https://maven.twttr.com does not have 2.4.0

It looks like the latest 2.4.0 is missing:
[warn] ==== Twitter Maven: tried
[warn]   https://maven.twttr.com/com/twitter/finatra/finatra-http_2.11/2.4.0/finatra-http_2.11-2.4.0.pom
[info] Resolving com.twitter.finatra#finatra-httpclient_2.11;2.4.0 ...
[warn] 	module not found: com.twitter.finatra#finatra-httpclient_2.11;2.4.0
[warn] ==== Twitter Maven: tried
[warn]   https://maven.twttr.com/com/twitter/finatra/finatra-httpclient_2.11/2.4.0/finatra-httpclient_2.11-2.4.0.pom
[info] Resolving com.twitter.finatra#finatra-jackson_2.11;2.4.0 ...
[warn] 	module not found: com.twitter.finatra#finatra-jackson_2.11;2.4.0
[warn] ==== Twitter Maven: tried
[warn]   https://maven.twttr.com/com/twitter/finatra/finatra-jackson_2.11/2.4.0/finatra-jackson_2.11-2.4.0.pom
[info] Resolving com.twitter.inject#inject-server_2.11;2.4.0 ...
[warn] 	module not found: com.twitter.inject#inject-server_2.11;2.4.0
[warn] ==== Twitter Maven: tried
[warn]   https://maven.twttr.com/com/twitter/inject/inject-server_2.11/2.4.0/inject-server_2.11-2.4.0.pom
[info] Resolving com.twitter.inject#inject-app_2.11;2.4.0 ...
[warn] 	module not found: com.twitter.inject#inject-app_2.11;2.4.0
[warn] ==== Twitter Maven: tried
[warn]   https://maven.twttr.com/com/twitter/inject/inject-app_2.11/2.4.0/inject-app_2.11-2.4.0.pom
[info] Resolving com.twitter.inject#inject-core_2.11;2.4.0 ...
[warn] 	module not found: com.twitter.inject#inject-core_2.11;2.4.0
[warn] ==== Twitter Maven: tried
[warn]   https://maven.twttr.com/com/twitter/inject/inject-core_2.11/2.4.0/inject-core_2.11-2.4.0.pom
[info] Resolving com.twitter.inject#inject-modules_2.11;2.4.0 ...
[warn] 	module not found: com.twitter.inject#inject-modules_2.11;2.4.0
[warn] ==== Twitter Maven: tried
[warn]   https://maven.twttr.com/com/twitter/inject/inject-modules_2.11/2.4.0/inject-modules_2.11-2.4.0.pom
[info] Resolving jline#jline;2.12.1 ...
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: com.twitter.finatra#finatra-http_2.11;2.4.0: not found
[warn] 	:: com.twitter.finatra#finatra-httpclient_2.11;2.4.0: not found
[warn] 	:: com.twitter.finatra#finatra-jackson_2.11;2.4.0: not found
[warn] 	:: com.twitter.inject#inject-server_2.11;2.4.0: not found
[warn] 	:: com.twitter.inject#inject-app_2.11;2.4.0: not found
[warn] 	:: com.twitter.inject#inject-core_2.11;2.4.0: not found
[warn] 	:: com.twitter.inject#inject-modules_2.11;2.4.0: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[error] (common/*:update) sbt.ResolveException: unresolved dependency: com.twitter.finatra#finatra-http_2.11;2.4.0: not found
[error] unresolved dependency: com.twitter.finatra#finatra-httpclient_2.11;2.4.0: not found
[error] unresolved dependency: com.twitter.finatra#finatra-jackson_2.11;2.4.0: not found
[error] unresolved dependency: com.twitter.inject#inject-server_2.11;2.4.0: not found
[error] unresolved dependency: com.twitter.inject#inject-app_2.11;2.4.0: not found
[error] unresolved dependency: com.twitter.inject#inject-core_2.11;2.4.0: not found
[error] unresolved dependency: com.twitter.inject#inject-modules_2.11;2.4.0: not found
[error] Total time: 4 s, completed Oct 9, 2016 6:12:45 PM
`

@codeape it's on maven central, not at maven.twttr.com: http://search.maven.org/#search|gav|1|g%3A"com.twitter" AND a%3A"finatra-httpclient_2.11"

Ahh thanks.
Why do all finatra/example have this in build.sbt?:
resolvers ++= Seq(
Resolver.sonatypeRepo("releases"),
"Twitter Maven" at "https://maven.twttr.com"
)

@codeape There are still some artifacts that require maven.twttr.com, but we're actively working on removing those dependencies or moving them to maven central. @mosesn did some work on this just last week actually.


Custom scroll container

Thanks for writing jump.js, it has a nice API!
I have a use case where I would like to scroll not the window as a whole, but the contents of a specific div. It would be great if I could configure a custom container element rather than using window. As far as I can see that's currently not possible or am I overlooking something?

I see an old PR, #24, adding support for a custom scroll container. Would love to see that in the library :D

I created a PR too, see #51

Any plans for this to land in one of the next versions?

Yes please merge this PR :)

Any chance of getting this merged in? Can we do something to help with this?

Disappointed this hasn't been addressed. PR #24 doesn't appear to bloat the library which was the initial concern, and this feature is necessary for SPAs

Yea, I had to switch to Velocity.js which is unfortunate as it comes with plenty of features I never use.

Would be great to get this merged for @mikebywaters reasonings. I have a div that I want to scroll instead of window.

Please merge this PR!

Please merge this PR!

Any idea why this PR is not getting merged? Seems like a useful one!

For what it's worth for anyone else coming across here, I personally decided to use Anime.js (only around 6KB).
See an example of how to smooth scroll here: https://codepen.io/rukiadia/pen/qYyLdw
As you can see a custom container element is quite easy to setup.
Since this PR hasn't been merged or looked at for so long I worry that something has happened to the original creator. I hope he/she is doing okay but I would consider this library effectively dead.


⚠ Bofrost.at has degraded performance

In b69b155, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1517 ms

Resolved: Bofrost.at performance has improved in 150786f.


fix(router-valibot-adapter): support for v1 dist-tags

Looks like the npm install broke with the upgrade to Valibot v1. This PR should fix that.

we don't really need the adapter anymore, but let's merge this for those who still use it

Sounds good! But keep in mind that the Standard Schema spec is in beta and some things may change in the next few weeks before we release our RC.


exclude failing indirect_addressing tests

make test_{read,write}_data_dim-indirect_addressing xfail

bors try

bors try

bors try

bors try-

bors try

bors try


doesn't work with Ukrainian language

Thanks for the report!
Will investigate.

Checked localization resources, looks like it might be broken for czech, finnish, french, greek, tchinese, thai, turkish, ukrainian, vietnamese and japanese!
@frailtyofbeing, will be fixed in the next update!

Well, finally got a PC to run tests, 1.5 is up
Thank you for your patience!

I didn't have a chance to test it before. It still doesn't work, even after the update. I even tried reinstalling dota. Does not work with French, Traditional Chinese, Japanese, Thai, Finnish, Czech, Turkish, Greek, Ukrainian, Vietnamese languages.

@frailtyofbeing, another user have confirmed it working on Ukrainian so if it works there it works for all those I've mentioned too.
Maybe there's other issue?
Are you sure the dota_mods/pak01_dir.vpk was refreshed? I mean, the ui background should have changed to the default one (the simple gray one, without last match or bali major)

Yes, your right. I forgot about this step. It works fine now


Getting more people to use recollect?

What are you plans/thoughts/ideas to get more people to use recollect? I think it's got a lot of promise and would be good to see some more activity!

I've just release v4.0.0 and I'll do a blog post about it. Are you still using Recollect?

Unfortunately no 😢


[Bug] (pie): incorrect response area of pie piece when applied selectedOffset

Version
5.0.0
Link to Minimal Reproduction
Demo for Reproduction
Steps to Reproduce
firstly, click one section:

Then click the section again to make it return the original position:

At present the clickable area is not returned, but still:

Then the clickable area is not correct for ever.
Then I run the visual test, but the visual test about pie all passed, not cover this case.
Current Behavior

Expected Behavior
click area should be correct.
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

The problem lies in animation.
you can circumvent this issue through animation attributes.
My demo


API returns 502 proxy error for DOI lookups

Did some of the API URLs change, or their format?

Nevermind, seems to be working fine now. Thanks!


Update your About

Update your About section to include details on the repo and what the project is about. Keep up the good work on setting up your repository!

Image:

Thank you!!

Issue resolved and updated, thank you

Update made:


Mk40 save still corrupting file

Working on trying to build the "Material Sample File"
Taking something from Inkscaspe >> MK40 works fine.
Saving really messes the file up....
DEFINATELY can't take it back into Inkscaspe
Looks like you cant edit POSITION in Mk40T, it doesn't place things even close....
This is what I "arranged" in Mk40

This is what printed out
(The Box fills are likely on my InkScape edits)

It also injected a few PAUSES/Waits or something where everything "PAUSED" for several seconds... The computer was non-responsive/slow, and then after a few seconds more it picked up and kept going.
It was between different RASTER sections.
Lastly, when it finished the Raser operation, it did the Engrave, but it SLAMMED the cartridge around as if the software expected infinite acceleration/de-acceleration.   SLAM-engrave -SLAM next start point, engrave. SLAM next start, engrave for all 5 engrave lines...
The only thing "different" on this run is that I pulled the 2nd group of Raser function up higher in the operations list so all of the Raster operations would be done together THEN the Engraving.  Maybe that move caught something it shouldn't have...
Today I have time to "play" with MK40T...
No error files so far...

Ok, I created a sample File.
A box with the word "Sample" in it
Saved it in inkscape and PRINTED the SVG file
Opened it in MK40T, Put in "Notes for Sample" in the notes section, Saved THAT, and printed THAT.
Uhm... the two printouts weren't even CLOSE to each other.
PLEASE add some s and formatting.
Inkscape was easy to follow
TRYING to pull some discrepancies out I saw the following:
Viewspace and dimensions don't agree
Most of the Sections that Inskscspe uses are missing, Defs, Sodipod, metadata, etc... none match up...
It looks like the first portion of the MK40T save file is the settings of the operation menu, but VERY hard to follow without s and SOME formatting
The next section looks like the NOTES
then the two elements should follow, the rectangle and the text "Sample"
Dimensions don't agree for elements
Stroke -Opacity is missing
Transform Matrix values don't agree (Scale?? is this enough precision to scale?)
Text id is missing  (id="text1331")
trying to a "Stare and compare" was useless because of the HUGE structural differences. I tried to find specific elements, and see if they matched values... Most weren't even close.  Since it -is- an SVG file, it should look SIMILAR to the SVG generated by Inkscape... maybe with the Operations and Notes at the front with a header... After the Notes section, everything should look the same!

You can bypass the text location issue by editing the file in MeerK40t after saving it in MeerK40t the first time.
So; create a file in Inkscape. Open MeerK40t, select New from File menu (to clear out any existing operations). Open the inkscape file in MeerK40t. Save as in MeerK40t under a different filename. Open the newly saved file. Then move the text objects back where they belong, set any operation powers and speeds, notes, etc.; and save the file in MeerK40t again.
After you do that; you'll be able to open the file in Inkscape and keep working back and forth between Inkscape and MeerK40t. Even the text will remain text. Changing colors, adding new elements, etc.; should continue to work in MeerK40t following that setup. You may need to manually classify text as a Raster operation, but it will save that property too in MeerK40t.
If you use the attached file; remember the File->New option in MeerK40t. All the extra Operations this file uses will stick around, and may cause confusion or other issues. But generally speaking; this is how I work with MeerK40t on a daily basis.
BobTest5_Mk8.svg.zip

I will give that a try... seems odd, but I will give it a try
Please don't wait for me to respond when booking an inspection. I turn off
my devices when on appointments and someone else could get your spot.
Please CALL our service or book yourself online to save the spot as soon as
you know the time you want.
Bob Sisson ACI, BVI
Inspections by Bob
ASHI Member # 212016
MD Home inspectors License #29666*
*MAC-ASHI Chapter President 2010-2012
ASHI Mid-Atlantic Group Leader 2009-2012
ASHI National Board Of Directors 2012-2015, 2017,2018-2020
<EMAIL_ADDRESS>www.inspectionsbybob.com
(301) 208-8289 Scheduling/Pricing
On Mon, Feb 8, 2021 at 5:05 PM Joseph Lane<EMAIL_ADDRESS>wrote:

You can bypass the text location issue by editing the file in MeerK40t
after saving it in MeerK40t the first time.
So; create a file in Inkscape. Open MeerK40t, select New from File menu
(to clear out any existing operations). Open the inkscape file in MeerK40t.
Save as in MeerK40t under a different filename. Open the newly saved file.
Then move the text objects back where they belong, set any operation powers
and speeds, notes, etc.; and save the file in MeerK40t again.
After you do that; you'll be able to open the file in Inkscape and keep
working back and forth between Inkscape and MeerK40t. Even the text will
remain text. Changing colors, adding new elements, etc.; should continue to
work in MeerK40t following that setup. You may need to manually classify
text as a Raster operation, but it will save that property too in MeerK40t.
If you use the attached file; remember the File->New option in MeerK40t.
All the extra Operations this file uses will stick around, and may cause
confusion or other issues. But generally speaking; this is how I work with
MeerK40t on a daily basis.
BobTest5_Mk8.svg.zip
https://github.com/meerk40t/meerk40t/files/5947086/BobTest5_Mk8.svg.zip
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/meerk40t/meerk40t/issues/347#issuecomment-775494108,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ2GYYYIOPLJ2YP7YBFPYLS6BNZZANCNFSM4XJS4TIQ
.

The squares on the right are designed to be set at different speeds to see what is required to cut through a material. They hang off the panel edge on purpose, so one can look sideways and see how deep each one penetrated. I often go a bit slower than that on natural wood because its not 100% uniform. This was just cardstock though since it's fast and easy to test with.

OK, tried it, and it is still screwy
Created an Inkscape file

Brought it into MK40

saved it under a new name, closed, and opened the new file to:

Made lots of edits, including getting rid of a fragment, and tried to save, and got an error...
So much for my edits...
Simple stuff works... but complex edits crash, or get completely garbled.
Enclosed is the original Inkscape file, and the Mk40 version
New GridM.zip
What I am trying to do, is very close to what you did.  A material file with some standard cuts/text
Dial in the MAX current on the front dial
Pick a Medium high speed (300mm/sec, 300PPI) for Rastering Text
Set the Step up a notch so it finishes FAST
Print/cut the same thing at a number of different PPI
When I try this,  it garbles things   -AND- it slams the head around as if acceleration is set wrong if I let it run anyway.

You've lost parts outside the bed area (why head crashes). Every component needs to be accounted for after reloading the MeerK40t Save as file.
Usually just work from a template I made months/years ago....  I only draw new when I'm making something new; not trying to laser for the first time. Every project I make requires a few revisions; and MeerK40t settings before it becomes a one click laser job. If you want red,blue,black only jobs; that's pretty much baked in if MeerK40t is setup for that (something you would do on your own).

Why PPI? You are going to care about speed between materials? It was even marked on your initial test job (Speed, Material, Current). The file does need to be tuned for each material. Paper does not cut like wood. Acrylic is different still. Each one could be tuned for a 1-click job once you understand those settings for your specific machine.
It can not guess. Running a wood job on paper might catch fire... Sure won't look good.

MOST of the time I set a Medium High Speed, and work DOWN from that.
If it won't give me the look/cut I want at 1000 PPI I know I need to slow
down by a factor so that PPI will give me the range I want.
The Material Sample sheet was -SUPPOSED- to be a simple way to drop a
sample on the bed and run the job
It was supposed to have enough range to be able to pick the real jobs
parameters from that ONE sample job...
It was also supposed to be a way to learn how MK40 behaved and interacted
with Inkscape and the K40...
In my playing I have worked with Acrylic, cardboard, chip board, thin Birch
plywood, Pine 2x4's. and 1mm hardwood Veneers and even wooden spoons.  I
have been surprised a LOT by what DOESN'T cut easily, and what needs very
very little power to blow through completely.
Not being able to go between MeerK40t and Inkscape is a bother. Not
insurmountable, but a pain.  But when stuff comes across missing text, or
missing attributes, THAT is an error.
On this file, the SAMPLE TEXT all came across as BLACK.  It was supposed to
be the same color as the stars.  Also, the 1000PPI line is MISSING the
"Sample text" line completely... SO... how do I build it in InkScape when
it ignores Inkscape attributes?
On Mon, Feb 8, 2021 at 8:13 PM Joseph Lane<EMAIL_ADDRESS>wrote:

Why PPI? You are going to care about speed between materials? It was even
marked on your initial test job (Speed, Material, Current). The file does
need to be tuned for each material. Paper does not cut like wood. Acrylic
is different still. Each one could be tuned for a 1-click job once you
understand those settings for your specific machine.
It can not guess. Running a wood job on paper might catch fire... Sure
won't look good.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/meerk40t/meerk40t/issues/347#issuecomment-775578131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ2GYY2STVP7YSV7FCCKMDS6CD3JANCNFSM4XJS4TIQ
.

A considerable amount of this is corrected in the beta. It shows the text in the correct font and doesn't mangle between saves. More work might still be needed to be done. But, this should be significantly in the correct direction.
https://github.com/meerk40t/meerk40t/releases/tag/0.6.19-beta2

To what degree after 0.6.19 is this still an issue? Leaving this open until that's correctly settled.

I saved an SVG on 6.20 and it seemed to reload just fine when previously it didn't. Obviously I was not testing saving of all types of data, so I cannot confirm that it works in all situations.


List of pkgs supporting azure identity is not available

We no longer maintain the list of packages supporting @azure/identity explicitly in the readme of the package. So, removing this link.
cc @diberry

#sign-off


remove log files

Remove log files after build.
Jira - https://ccp.sys.comcast.net/browse/RDK-36957

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


should use gI instead of gi in config

Description
https://github.com/neovim/nvim-lspconfig/blob/d12140c5687a1186b95b3f42dbc6cc769df0cf0d/README.md?plain=1#L75
gi is mapped to go to last insert place and enter insert mode by default, which is quite useful, so this config should use gI instead

gI also mapped. it's just an exmaple config .


Fieldset with checkboxes

This is basically a follow-up to #400, but it raises the question if this is an actual bug, or a misconception from my part.
The main issue relies on the fact that I have a form and some of the questions have predefined options (displayed as checkboxes) and an optional text field for the user to write its own answer ("If other, please identify" - check #400 for images)
After going through the docs, I realized that the best solution for our case would be to have a type: "object" schema for the question, defining two properties: Options as an array (later to be mapped as checkboxes), and Answer as a string. Here's the relevant snippet of the schema object:
{
    "type": "object",
    "title": "Survey",
    "properties": {

        "4": {
            "type": "object",
            "title": "Magazine Subscriptions?",
            "items": {
                "properties": {
                    "Options": {
                        "type": "array"
                    },
                    "Answer": {
                        "type": "string",
                        "title": "If other, please identify"
                    }
                }
            }
        }
    }
}

and here's the form object for that question:
[
    {
        "key": "4",
        "title": "",
        "disableSuccessState": true,
        "type": "fieldset",
        "items":
        [
            {
                "key": "4.Options",
                "title": "Magazine Subscriptions?",
                "type": "checkboxes",
                "titleMap": [
                    {
                        "value": "16",
                        "name": "Men's Health"
                    },
                    {
                        "value": "17",
                        "name": "Women's Health"
                    },
                    {
                        "value": "21",
                        "name": "None"
                     }
                ]
            },
            {
                "key": "4.Answer",
                "title": "If other, please identify",
                "type": "string",
                "htmlClass": "col-md-6"
            }
        ]
    }
]

Maybe I'm doing something wrong, because even though the form is displayed correctly, when I click any of the checkboxes, I get the following error:
TypeError: Cannot read property 'length' of undefined
    at schema-form.js:1613
    at Object.fn (angular.min.js:109)
    at h.$get.h.$digest (angular.min.js:110)
    at h.$get.h.$apply (angular.min.js:113)
    at HTMLInputElement.<anonymous> (angular.min.js:187)
    at HTMLInputElement.n.event.dispatch (jquery-1.11.0.min.js:3)
    at HTMLInputElement.n.event.add.r.handle (jquery-1.11.0.min.js:3)

Can someone point me in the right direction?

I think the problem might be the key  "4". Using just a number might confuse  ng-model and sfSelect, mistaking it for an index in an array when updating the model. Try switching that out our just add some character, like "option4".

Please re-open if you do not feel the question has been answered sufficiently.

I'm just wondering if there is a way to use dot notation with the fieldset object? In our system we must use dot notation and I have created a custom ASF control that works like fieldset and makes ASF work with dot notation. I really don't want to have to create a custom "fieldset" object just to make this work so if anyone has an idea I would appreciate it.


surface_nets lighting artifacts from normal interpolation

It seems like there are two issues contributing to the artifacts:

Non-manifold complexes
Interpolation between "unlit" normals and "lit" normals

When you see random isolated black spots that seem like they should be lit, it's because the vertex is actually non-manifold, and its normal was estimated to point more in the "unlit" direction than the "lit" direction.
Then the interpolation will make that darkness bleed out where it might not makes sense. In particular, on sharp edges. If the normal is estimated as "unlit" that will bleed around the edge.
I think the sharp edge issue is actually a bigger problem. Non-manifold complexes are always undesirable. But sharp edges are certainly something we would like to look correct.

BTW, this should not be an issue if flat shading is used.


feat(deploy): deploy to google

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

v1.2.0
GitHub release

Your semantic-release bot :package::rocket:


chore: test

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

Test done!  Closing.


antd-globals-hiding-hack.css property outdated

Seems like this file uses a property text-decoration-skip with value ink and the spec seems to had been changed, providing a possibility to use a property called text-decoration-skip-ink with value auto instead.
PS: create-react-app compiles with warnings because of this.

Seems like antd has fixed this issue in their repo: 46a7a
Any possibiilty to update this dependency inside react-big-scheduler? Thanks!

@reenan I'll take a look, thanks for the feedback.

Hi @StephenChou1017, have you had the chance to update this in the npm package yet?
I'm still getting this warning.
Anything I can do to help :)?

Also having this issue

@StephenChou1017 Any updatation regarding this issue?


Make "reportable" setting consistent in zap file to avoid zap instability.

Problem
ZAP generation is unstable regarding "reportable" output, seen  in matter IDL files like https://github.com/project-chip/connectedhomeip/runs/4843394902?check_suite_focus=true
and
https://github.com/project-chip/connectedhomeip/runs/4838407145?check_suite_focus=true
Change overview
Change the struct to be consistent between client & server. Note that this was manually edited (not sure what zap UI changes would be required to make this consistent).
Testing
CI will validate this.

NOTE: I plan to add more to this by checking ToT a bit to see what other things are unstable.
I expect 2-3 zap files to need fixing in general.

I will merge this as a hotfix: zap is massively non-deterministic without this change and the change looks safe enough: I do not believe our regular codegen actually cares about reportable state (otherwise we would have seen non-determinism before).


#113821767 Create Initial DRF Boilerplate

What does this PR do
It creates the initial boilerplate for the Django Rest Framework.
Description of Task to be completed
After installation of the DRF module, the various apps should have an api.py for the API views and serializers.py for their respective models.
How should this be manually tested
This cannot be manually tested since no endpoints are defined yet.
What are the relevant pivotal tracker stories
#113821767

There's nothing here to show how the URL formats are going to be and where they will be added. Also would be nice to put that information on the code as well as on this PR description.


Add support to specifying hostname for create_container

The image parameter needs a tag as well, for example: ubuntu:latest.
Signed-off-by: Walinton Cambronero<EMAIL_ADDRESS>

your second commit doesn't fit the pull request description

That's right. My bad, I changed the title of the Pull Req.

A few changes to merge this request:

Squash changes.

Set documentation for image as:
:param str image: The image to run on this node, in the form ``repository:tag``.

Set default value for hostname keyword argument to None to match the Docker-py API: https://github.com/docker/docker-py/blob/master/docker/api/container.py#L94

Thanks Carlos, Done. Please review.


SolverError: Either candidate conic solvers (['CVXOPT']) do not support the cones output by the problem (ExpCone), or there are not enough constraints in the problem.

hello,i just Installation this use
pip install<EMAIL_ADDRESS>
and run some test code :
from maxrect import get_intersection, get_maximal_rectangle, rect2poly

# For a given convex polygon
coordinates1 = [[0,0],[0,1287.206-527.796],[1576.252,1287.206],[1815.126,1287.206],[0,0]]

# get the maximally inscribed rectangle
ll, ur = get_maximal_rectangle(coordinates1)

# casting the rectangle to a GeoJSON-friendly closed polygon
rect2poly(ll, ur)

but there have some error
(base) tomin@ubuntu:~/python$ /home/tomin/anaconda3/bin/python /home/tomin/python/test.py
Traceback (most recent call last):
  File "/home/tomin/python/test.py", line 8, in <module>
    ll, ur = get_maximal_rectangle(coordinates1)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/maxrect/__init__.py", line 139, in get_maximal_rectangle
    prob.solve(solver=cvxpy.CVXOPT, verbose=False, max_iters=1000, reltol=1e-9)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 290, in solve
    return solve_func(self, *args, **kwargs)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 568, in _solve
    self._construct_chains(solver=solver, gp=gp)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 511, in _construct_chains
    raise e
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 497, in _construct_chains
    gp=gp)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 434, in _find_candidate_solvers
    raise error.SolverError("The solver %s is not installed." % solver)
cvxpy.error.SolverError: The solver CVXOPT is not installed.

so i use conda install cvxopt
conda install CVXOPT

but there have some new errors
(base) tomin@ubuntu:~/python$ /home/tomin/anaconda3/bin/python /home/tomin/python/test.py
Traceback (most recent call last):
  File "/home/tomin/python/test.py", line 8, in <module>
    ll, ur = get_maximal_rectangle(coordinates1)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/maxrect/__init__.py", line 139, in get_maximal_rectangle
    prob.solve(solver=cvxpy.CVXOPT, verbose=False, max_iters=1000, reltol=1e-9)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 290, in solve
    return solve_func(self, *args, **kwargs)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 568, in _solve
    self._construct_chains(solver=solver, gp=gp)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 511, in _construct_chains
    raise e
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/problems/problem.py", line 506, in _construct_chains
    candidate_solvers)
  File "/home/tomin/anaconda3/lib/python3.7/site-packages/cvxpy/reductions/solvers/solving_chain.py", line 97, in construct_solving_chain
    ", ".join([cone.__name__ for cone in cones])))
cvxpy.error.SolverError: Either candidate conic solvers (['CVXOPT']) do not support the cones output by the problem (ExpCone), or there are not enough constraints in the problem.

can you help me ?thanks!

i maybe solve this problem.
go to read :
https://www.cvxpy.org/tutorial/advanced/index.html#solvers
there have some info:

Here EXP refers to problems with exponential cone constraints. The exponential cone is defined as
{(x,y,z)∣y>0,yexp(x/y)≤z}∪{(x,y,z)∣x≤0,y=0,z≥0}.
Most users will never specify cone constraints directly. Instead, cone constraints are added when CVXPY converts the problem into standard form.
By default CVXPY calls the solver most specialized to the problem type.

so i change the code in init.py at line 139
prob.solve(solver=cvxpy.CVXOPT, verbose=False, max_iters=1000, reltol=1e-9)

to
prob.solve()

now,it work.maybe there still have some problem,but it can be run.

found I also needed to change the return:
    #return list(bottom_left[0]), list(top_right[0])
    return bottom_left, top_right


Make Inlay hint length configurable

Solves https://github.com/microsoft/vscode/issues/205708
This will let us define how long we want the hints, I use rust quite a bit and rely a lot on this, the 43 limit was quite small so I made it configurable, examples:

Oh, sad to see that it got pushed to another milestone. I was really looking forward to be able to properly use rust in vscode again :(

@rafamerlin Thanks for staying with for so long. I updated this PR with main and it is now ready to be merged. I also pushed some little fixed and changed my mind and simplified this to a single setting again. I believe, for now, most folks will use this to disable trimming anyways


Unable to create new function project in VS code

I've tried to create a new function app project with runtime as python
selected python interpreter as python 3.75
As the first function I've selected a Queue triggered function, Here is the error I'm getting
WARNING: Python version 3.8+ is not supported with version "~2" of the func CLI. Update to at least "3.0.2245" for support. 2:56:52 pm: Running command: "git init"... Initialized empty Git repository in c:/Hari/Project-POC/ICFunction/.git/ 2:56:55 pm: Finished running command: "git init". 2:56:57 pm: Running command: "C:\Users\Harikishore.KA\AppData\Local\Programs\Python\Python37\python.exe -m venv .venv"... Error: 'list' object has no attribute 'read' 2:57:00 pm: Error: Failed to run "C:\Users\Harikishore.KA\AppData\Local\Programs\Python\Python37\python.exe" command. Check output window for more details.
I'm using python 3.7 only, don't know why I'm getting the first warning.
Repro Steps:

Create a new function app project using the VS code extension

You can ignore the warning at the top. It's just there in case someone is confused why 3.8 doesn't show up during our prompts
There seems to be an error running the command python.exe -m venv .venv. I can't repro myself. You can try running the command manually in a terminal to see if you get the same error. If so, it's something to do with your python setup - you may be able to find more guidance online


netDxf.DxfDocument.Load(System.IO.Compression.DeflateStream) returns null

I zipped dxf file then unzip it on memory and tried to load dxf by
public static DxfDocument Load(Stream stream);
which every time returns null.
//-------------------
//make zip from dxf.
using var memoryStream = new MemoryStream();
using (var archive = new ZipArchive(memoryStream, ZipArchiveMode.Create, true, Encoding.UTF8);
var entryFile = archive.CreateEntry("data");
using var entryStream = entryFile.Open();
using var fs = new FileStream(dxfFilePath, FileMode.Open);
fs.CopyTo(entryStream);
memoryStream.Seek(0, SeekOrigin.Begin);
var zipBytes= memoryStream.ToArray();

//unzip and try read dxf from zip entry
using var zipSource = new MemoryStream(zipBytes);
using var archive = new ZipArchive(zipSource, ZipArchiveMode.Read);
var entry = archive.Entries.First();
using var stream = entry.Open();
//it fails here.
Trace.Assert(netDxf.DxfDocument.Load(stream) != null);

//workaround is copy deflatestream to memorystream then load from it.
//-------------------
using var ms = new MemoryStream();
stream.CopyTo(ms);
ms.Seek(0, SeekOrigin.Begin);
netDxf.DxfDocument.Load(ms);

I have taken a look at your code and I have run into a couple problems. The code you have shown does not work as it is. In the line
using (var archive = new ZipArchive(memoryStream, ZipArchiveMode.Create, true, Encoding.UTF8);

the first parenthesis is not needed. Also there is a duplicate name error for the variable "archive". I deleted the unneeded parenthesis and I renamed them as archiveIn and archiveOut, but even then, after providing a sample DXF file for the dxfFilePath variable, your sample code keeps crashing in the line
using var archiveOut = new ZipArchive(zipSource, ZipArchiveMode.Read);

with the exception
System.IO.InvalidDataException: 'End of Central Directory record could not be found.'

before the loading of the DXF file. I am not familiar with the new Zip compression classes in the net framework so I do not know what is the problem here.
Try to post a full working sample and give me the details of what version of the library you are using, I only have uploaded compiled version of the library for the Net Framework 4.5 and in that version the Net classes for managing ZIP files were not available. It would be much better if you download the latest code and compile the library yourself in Debug mode. The latest code is a multitarget project predefined for Net Framework 4.5, Net Standard 2.1, Net Core 3.1, and NET 5.0.

Thank you for replying.
I prepared full code.(.Net Core 3.1 MS Unit Test)
using Microsoft.VisualStudio.TestTools.UnitTesting;
using System.Diagnostics;
using System.IO;
using System.IO.Compression;
using System.Linq;
using System.Text;
namespace DeflateStreamTest
{
    [TestClass]
    public class NetDxfDeflateTest
    {
        [TestMethod]
        public void check_method_with_latest_source()
        {
            string outputPath = @"empty.dxf";
            //make empty dxf
            {
                var dxf = new netDxf.DxfDocument();
                dxf.Save(outputPath);
            }
            byte[] zipBytes;
            //make zip from dxf.
            {
                var dxfFilePath =outputPath;
                using var memoryStream = new MemoryStream();
                using (var archive = new ZipArchive(memoryStream, ZipArchiveMode.Create, true, Encoding.UTF8))
                {
                    var entryFile = archive.CreateEntry("data");
                    using var entryStream = entryFile.Open();
                    using var fs = new FileStream(dxfFilePath, FileMode.Open);
                    fs.CopyTo(entryStream);
                }
                memoryStream.Seek(0, SeekOrigin.Begin);
                zipBytes = memoryStream.ToArray();
            }
            //unzip and try read dxf from zip entry
            {
                using var zipSource = new MemoryStream(zipBytes);
                using var archive = new ZipArchive(zipSource, ZipArchiveMode.Read);
                var entry = archive.Entries.First();
                using var stream = entry.Open();
                //it fails here.
                Trace.Assert(netDxf.DxfDocument.Load(stream) != null);
            }
        }
    }
}

DeflateStream.Position Property
That page says 'This property is not supported and always throws a NotSupportedException.'
I have seen some other classes which public abstract bool CanSeek{get;} returns false,
HttpWebResponse.GetResponseStream()
returns ConnectStream whcih does not support Seek/Position.
ConnectStream

Yeah, the problem is that the DefalteStream class do not allow changing the position in the stream. One of the first things that the reading process does is to read the first few bytes of the DXF file to check is it is binary or text, so the proper reader is used a BinaryReader or a TextReader, then the position is reset to the beginning to start the loading process. If there is a better way to do it, I do not know.
At the moment the only thing I will do is raise an exception for streams that do not support changing the position. So, you have two choices copying the data to a memory stream or saving the unzipped DXF file to a temporary directory.


feat(python): add python workshop content

Implements Issue #33
Description of changes:
Used the JavaScript/Node.js folder as a template, and updated the content specific to Python (code samples, screenshots, tool references, buildspec, etc).
Some changes to the content include

using main instead of master for git branch in CodeCommit
collapsed the git config steps from "Configure credentials" into the previous step
in "Setup a CDK Project" upgraded the version of node from 10.x using nvm install lts/fermium to avoid CDK warning messages
changed flow of "Introduce an error on purpose" to introduce a runtime error that occurs in Lambda but not in the test (looking specific ENV var).  This removes the need to comment out the test code.
tweak to the "Invoke the canary" script to fix an error when multiple lambda function names contain "sam-app-HelloWorldFunction"
changed some dependencies on jq to use --query on the aws cli command instead.
added cleanup step to delete the CodeCommit repository
minor formatting and typo fixes
"full screen" screenshots were taken in widescreen (16:10 ratio 1280x800)

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

@nmoutschen good catch on repo name -I fixed all those locally and pushed the commit


Delete models from local disk when they are not in LocalModelNode spec

What this PR does / why we need it:

Update localmodel agent to delete models from local disk when they are not in LocalModelNode spec
Added a test to make sure the correct folder is deleted.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Added a test for the controller.
Tested with local kind cluster
{"level":"info","ts":"2024-11-26T23:01:07Z","logger":"v1alpha1Controllers.LocalModelNode","msg":"Found model(s) to remove","num of models":2}
{"level":"info","ts":"2024-11-26T23:01:07Z","logger":"v1alpha1Controllers.LocalModelNode","msg":"Removing model","model":"iris"}
{"level":"info","ts":"2024-11-26T23:01:07Z","logger":"v1alpha1Controllers.LocalModelNode","msg":"Removing model","model":"iris2"}

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Re-running failed tests

/rerun-all - rerun all failed workflows.
/rerun-workflow <workflow name> - rerun a specific failed workflow. Only one workflow name can be specified. Multiple /rerun-workflow commands are allowed per comment.

/rerun-all


Sync read page progress with trackers

Describe your suggested feature
Currently the behavior of only syncing when a chapter is read is limiting. If mihon could sync per page, you could read something on your phone, then pick up where you left off on a desktop or other device.
There has been interest voiced for this with Komga, suwayomi, to name a few.
Original issue was here https://github.com/tachiyomiorg/tachiyomi/issues/5978
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.16.1.
[X] I will fill out all of the requested information in this form.

This would be a great addition. Happens quite often I might be reading a comic on one device, then I want to finish it on another device

I don't think trackers are meant for syncing read progress across devices at such granularity. That said, I tried to implement a workable version of page-based Komga tracker for myself. It's not a elegant solution though: to sync you have to click tracker icon, wait for a few seconds before close the dialog for the request to be done and page progress writen to the DB.

app-dev-arm64-v8a-preview.apk.zip

making a reverse proxy for it to high jack on syncs

For Komga it would be easier to just send read-progress requests in app to achieve sync. In this implementation, you can see the page progress get updated in Komga web app each time you turned page in Mihon, cuz it uses a event stream to get updates from server in real time. However to sync server progress to Mihon you need to manually trigger tracker refreshing.

It's working great so far. Thank you so much for sharing.

I just installed your Mihon apk but to me the tracking does not work. Should I do anything in particular? I am reading a manga from Komga. when I try to enable "Tracking" I have the "Update progress after reading" enabled as well as the Komga 'enhanced tracker'. However, it seems no page progress is synced to Komga :/

Update: I managed to get the syncing but it seems only one way: only mihon sends page reading progress to komga but if I advance reading on komga the progress is not synced back to mihon. Is this the supposed behavior?

However to sync server progress to Mihon you need to manually trigger tracker refreshing.

To do this, click the tracker button for the comic, it should show the kkmga sync progress which then syncs mihon with the komga seever in the background

@kanjieater yes I had managed the syncing but I wrote a later reply where I got the syncing work but it seems only one way: Mihon sends page progress to komga web interface but if I progress a comic from the web interface that is not synced back in Mihon. But I don't know if that is the expected behavior?

@kanjieater yes I had managed the syncing but I wrote a later reply where I got the syncing work but it seems only one way: Mihon sends page progress to komga web interface but if I progress a comic from the web interface that is not synced back in Mihon. But I don't know if that is the expected behavior?

No - im telling you you have to click the tracker manually each time to get komga to sync to mihon.

Thank you it worked. I really had to spend some time in understanding this tracker button to tap because it doesn't look like a button at all. In an ideal world, I would have liked that there should be no need to "tap to sync" but I understand that at the moment this is the best that it is. Thank you for the APK and I hope your PR gets pushed through!


TextCommandBarFlyout flickers open if empty

Describe the bug
The context menu on text boxes flashes a small rectangle when you try to open it if there are no commands.
Steps to reproduce the bug

Steps to reproduce the behavior:

Make sure your clipboard is empty (Win+V and clear works well for this)
Right-click an empty TextBox (otherwise Undo or Select All will be in the menu)
Watch as an empty rectangle flashes on screen before disappearing

Expected behavior
Nothing should appear if there are no menu items.
Screenshots

Version Info

NuGet package version:
Microsoft.UI.Xaml 2.8.5
Microsoft.WindowsAppSDK 1.3.230602002
Microsoft.WindowsAppSDK 1.4.230628000-preview1
Affects both WinUI 3 and WinUI 2

This issue still affects WinUI 2 and WinUI 3. Bumping to keep this from getting automatically closed.

Three years later and this seemingly trivial but bothering issue still not get fixed.


CFI range generation with start and end at different nodes level

The CFI generated for a selection starting and ending at different node level are incorrect.
For the following document :

Title

Some very important text

The CFI for "important text" should be : /2/4, /2/1:5, /3:5
But the actual CFI generated is : /2/4, /2/1:5, /4/3:5
The CFI for "Some very" should be : /2/4, /1:0, /2/1:4
But the actual CFI generated is : /2/4, /4/1:0, /2/1:4
The extra step come from the parent path.
Spec : http://www.idpf.org/epub/linking/cfi/epub-cfi.html#sec-ranges

Here it is !
@danielweck Yes, i'm @chardouin as well. I should get rid of one of my account :)

A fix has been merged in.


dm/subvert-case

The change in behavior is shown by this test:
  left: `"spam_eggs, SpamEggs, SPAMEGGS and spam-eggs"`,
 right: `"spam_eggs, SpamEggs, SPAM_EGGS and spam-eggs"`', src/line_patcher.rs:79:9

I do not think it's a good change, so let's close this


[Machine Config] Unable to create Dev Box environments if multiple users are logged in Dev Home

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Make sure at least two users are logged in Dev Home, and one of them doesn't have Dev Boxes configured.
One of the simplest ways to check this is to open Environments page; it should show an error message stating "")
Try to create an Environment
Expected result
No response
Actual result
The operations fails with

And the logs/logging out the unconfigured user confirms the root cause of the issue.
Included System Information
No response
Included Extensions Information
No response

This will probably need some design as well since there will need to be a way to select the account a user might want to use.

This also repros when running on a fresh setup, with no users logged in Dev Home. In this case, the error message should state this like in Environments page.


How do i run this locally

I am not sure of how to view this locally for offline use...
Please help..Thanks

Each page is in Markdown, so each page is also human-readable off the bat.
You can also view each page through a Markdown viewer (VS Code works pretty nicely). If you want to export to other formats, you can run it through Jekyll, GitBook, or Pandoc to create web pages, PDFs, etc.


Smart Editor: Widget fails to allow reopening

v 2.0.1
Selecting the widget, then a feature allows for editing. However, closing the widget then re-opening fails to allow me to select a feature to edit and simply opens the features popup.

That is the current process.  Once you select a feature, you have to close the attribute inspector to select another feature for editing.  This allows you to explore the map will editing.
Please close if this resolved the issue and if not, lets log an enhancement to allow the selection of new feature for editing while editing,


draft: Update report unreleased command

closing in favor of https://github.com/microsoft/FluidFramework/pull/20722


Use handlers instead of callbacks

The recent changes in Neovim head cause loading this file to create an error. These changes have resolved that on my machine.

I suspect you have to fix the obsolete callbacks references in lua/nvim_lsp/util.lua and lua/nvim_lsp/configs.lua as well...? It might work without obvious errors, but I have a feeling some functionality is just going into a void since they're never called anymore.

 I missed those. Gonna change them and commit.

Changed the rest of them.

@tjdevries

@Iron-E Sorry I've merged this PR. Could you rebase?
https://github.com/neovim/nvim-lspconfig/pull/348

No problem. Good time to get that one through

LGTM I think

@Iron-E Thank you for your work :)

Happy we could do it!


Gradle 5.0 RC1 doesn't build my composite build anymore.

Two projects A and B, A is providing plugins to B and B includes A using includeBuild in settings.gradle. B is a java-library and with Gradle 4.10 builds properly, but after upgrading to Gradle 5.0 RC1 it doesn't anymore. It seems to stop after executing the task processResources, but without any error. The only thing of interest logged is the following warning:

Ignoring listeners of task graph ready event, as this build (:A) has already executed work.

Expected Behavior

Project B should continue to build properly, resulting in the creation of a JAR file.
Current Behavior

No JAR file is created, the build stops after  build\resources\main\META-INF\gradle-plugins\[...] has been created.
Context

Project A is providing helper plugins to multiple other projects, project B is consuming those plugins and at the same time provides one on its own to one specific other project C. In the end, project A and B are simply a variant of buildSrc for individual projects, only externalized into their own repos/projects and therefore included by their consumers.
Gradle 5.0 RC1 needs to be used because it fixes another issue for me.
During debugging I was able to work around that problem not only by switching versions of Gradle, but as well by reducing my build.gradle of project B to either not apply my plugin from project A anymore or keep applying that plugin and instead not using the plugins of A which project B provides to project C. The main difference in both cases is the classpath for buildscript vs. implementation. that might or might not have to do with it... What I couldn't find was what actually triggers the execution the warning mentions, I don't have the feeling that that depends on my own code. And it works with 4.10 unchanged as well, so shouldn't be that wrong.
Steps to Reproduce (for bugs)

cd de.am_soft.docbeam.raw.server.build
gradlew wrapper --gradle-version=4.10
gradlew clean build
gradlew wrapper --gradle-version=5.0-rc-1
gradlew clean build
Your Environment

Build scan URL:
Windows 10 1803 x86-64
java version "1.8.0_181"

gradle_4_10.txt
gradle_5_0_rc1.txt
GitHub.zip

I can reproduce

A work around, until there's a fix, is to use the java plugin instead of the java-library plugin in the plugin build.


use smallzstd and sync pool

[x] read the CONTRIBUTING guidelines
[ ] raised a GitHub issue or discussed it on the projects chat beforehand
[ ] added unit tests
[ ] added integration tests
[ ] updated documentation if needed
[ ] updated CHANGELOG.md

This PR tweaks the zstd encoding by using smallzstd package from tailscale and a sync pool, instead of creating a new encoder.
While this won't fix #1014 entirely, it could reduce CPU usage when sending mapresponses to the clients

Hi @jsiebens, first and foremost... thanks! Very happy to see you around here :)
Can you have a look into the linting/build errors? Thanks!

@jsiebens the sha256 sum in vendorSha256 of flake. nix needs to be updated:
error: hash mismatch in fixed-output derivation '/nix/store/m9mr72wj7hb0qwgjf3hv7gaxvld4hajb-headscale-48e2a82-go-modules.drv':
         specified: sha256-SuKT+b8g6xEK15ry2IAmpS/vwDG+zJqK9nfsWpHNXuU=
            got:    sha256-8p5NFxXKaZPsW4B6NMzfi0pqfVroIahSgA0fukvB3JI=

ok, I'll try to find the time to set up a nix environment, never done that before :-)


Incorrect diff for dates before and after March 10th

Describe the bug
If you use diff for dates that include March 10th, the difference in hours is off by one.
To Reproduce
I am just using js fiddle as an example, but this issue is currently affecting my live production app in React Native.
const now = moment("2024-03-09T15:32:23")
const future = moment("2024-03-11T15:40:23")

const days = future.diff(now, 'days')
const hours = future.diff(now, 'hours') - (days * 24)

console.log("Difference: " + days + " days " + hours + " hours")
console.log("Total hours: " + future.diff(now, 'hours'))

You can use this code to see the error. When the date range includes the 10th, it gives a -1 for hours. If you change the date range to 2024-03-11T15:32:23 through 2024-03-13T15:40:23 for example, the hours difference is correct.
Screenshots
Incorrect hours when using code provided above:

The correct time when moving dates by two days:

Desktop (please complete the following information):

OS: MacOS
Browser: Chrome
Version: 122.0.6261.94

Moment-specific environment

The time zone setting of the machine the code is running on: PST
The time and date at which the code was run: March 4th, 2024
Other libraries in use: N/A

Please run the following code in your environment and include the output:
"Mon Mar 04 2024 17:40:49 GMT-0800 (Pacific Standard Time)"
"3/4/2024, 5:40:49 PM"
480
"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"
"2.2.1"

Nevermind, this was just an issue with daylight savings


example package version bug

https://github.com/erikras/redux-form-material-ui/blob/master/example/package.json#L24
"redux-form-material-ui": "file:../",

sorry

It's good to see you, vGhost2000 :)


[c++] Array-open internals for stateful support

This is an effort-tracking task for any C++ support that may (or may not) be needed for the statefult API (exp.open() / exp.close()).

Duplicate of #740


Error 200: Power BI Desktop process is opening or the Analysis Services instance is not yet ready.

Describe the problem
Error 200: Power BI Desktop process is opening or the Analysis Services instance is not yet ready.
Bravo version
0.9.5 (0.9.8190.17886)
Tabular model
Power BI Desktop, import mode
Power BI
2.108.997.0 64-bit (august2022)
Steps to reproduce the behavior

Open Bravo for Power BI
Open menu
Browse in "Active Reports on Power BI Desktop"
Open model on your local drive

Wait and see the error.
Same happens when I open  power bi report first, then open power bi bravo... when I want to manage dates it shows model is availabe only for a few seconds and then switch to "this model is no longer available".
I attached both screens
Additional context
No response
Screenshots

Anything else ?
No response

@albertospelta report was already saved, but I just checked and the same issue happens when I open bravo for new (unsaved) report

@donkocon It would be helpful to see the detail of the diagnostic messages.
Can you expand the Object nodes highlighted in the in the image and share the contents here ?

@albertospelta here it is :)

Does this problem occur always and with all PBI reports ?
What is the name of the PBI report you used in the image you shared before ? Could you take a snapshot of the PBIDesktop window title and the PBIDesktop process (visible in the Windows Task Manager), as in the example below ?
Thanks!

Hey,
Sorry for late reply,
Here are screens:

@donkocon we have an internal build available that tries to fix this issue, it would be great if you could test it and let us know if the issue still persists.
To upgrade Bravo to this version you can change the update channel from Stable to Dev within the options panel as shown in the following image. After this change the Downlaod button should allow you to download this new build ( 0.9.8285.24909 )

I am closing this issue as the fix is available in version 0.9.6 but feel free to post back here and we can reopen it if the issue still exists.


Compilation errors with Boost 1.64

Currently in Beta, Boost 1.64 breaks Beast
Using VC14.1 (Visual Studio 2017)
Beast\include\beast/websocket/detail/decorator.hpp(105): error C2280: 'boost::basic_string_ref<char,std::char_traits<char>>::basic_string_ref<std::allocator<char>>(std::basic_string<char,std::char_traits<char>,std::allocator<char>> &&)': attempting to reference a deleted function
Beast\include\beast/websocket/detail/decorator.hpp(118): error C2280: 'boost::basic_string_ref<char,std::char_traits<char>>::basic_string_ref<std::allocator<char>>(std::basic_string<char,std::char_traits<char>,std::allocator<char>> &&)': attempting to reference a deleted function
Beast\include\beast/websocket/impl/stream.ipp(189): error C2280: 'boost::basic_string_ref<char,std::char_traits<char>>::basic_string_ref<std::allocator<char>>(std::basic_string<char,std::char_traits<char>,std::allocator<char>> &&)': attempting to reference a deleted function
Beast\include\beast/http/basic_fields.hpp(266): error C2280: 'boost::basic_string_ref<char,std::char_traits<char>>::basic_string_ref<std::allocator<char>>(std::basic_string<char,std::char_traits<char>,std::allocator<char>> &&)': attempting to reference a deleted function

This should be fixed by this: http://lists.boost.org/Archives/boost/2017/03/234022.php but isn't yet.

Is this still an issue in the release version of Boost 1.64.0?

It looks like this is fixed so I'll close the issue. Please re-open it if a problem remains, thanks!


What is the underlying C# library that AzCopy uses?

I am initiating a storage account to storage account transfer of 1 large file that's 60GB (a .vhd file). AzCopy completes it in 2.4 minutes whereas Azure.Storage.Blobs library in C# completes it in 6 mins.
Which version of the AzCopy was used?
10.21.0
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
./azcopy.exe copy "storageaccount-sastoken" "storageaccount-sastoken" --overwrite=prompt --from-to=BlobBlob --s2s-preserve-access-tier=false --check-length=true --include-directory-stub=false --s2s-preserve-blob-tags=false --recursive --log-level=INFO;
Test code
using Azure.Storage.Blobs;
using Azure.Storage.Blobs.Models;
using Azure.Storage.Blobs.Specialized;

static async Task sdkcopy(Uri sourceBlobSASURI, string filename)
{
    int gibCopied = 60;

    var destinationBlob = new PageBlobClient("connectionstring", "test", filename);

    var start = DateTime.Now;
    Console.WriteLine(start);

    CopyFromUriOperation copyOperation = await destinationBlob.StartCopyFromUriAsync(sourceBlobSASURI);
    await copyOperation.WaitForCompletionAsync();

    var end = DateTime.Now;
    Console.WriteLine(end);

    var total = end - start;

    Console.WriteLine($"Copying {gibCopied} GiB took {total.TotalMinutes} mins");
    Console.WriteLine($"{total.TotalSeconds / gibCopied} secs/GiB");
    Console.WriteLine($"{total.TotalMinutes / gibCopied} mins/GiB");
}

var sas1 = new Uri("sasuri");
sdkcopy(sas1, "copywsdk1.vhd").Wait();

Hi @kecheung AzCopy is written in GoLang and backed by the Azure Storage SDK for Go

https://github.com/Azure/azure-sdk-for-go


Renamed file and updated naming scheme to also include names used in academia.

Renamed spectre_v1_indirect_jumps.cc to spectre_v2.cc as this is actually an implementation of Spectre-V2/Spectre-BTB, not Spectre-V1/Spectre-PHT. This can be verified by adding an lfence before line 137. If it was indeed a Spectre-PHT attack, speculation would end here but instead the data is still leaked. By adding the lfence before line 51 we can actually prevent leakage. Therefore, we mistrain the Branch Target Buffer, making it a v2/BTB attack.
Also updated the names used in the demos/README.md file to also include the names used by academia. This change is mostly to keep names across industry and academia consistent.

@googlebot I signed it!

@googlebot I fixed it.

I would need to re-read the paper to discuss what BTB vs PHT mean, but I want to specifically mention why I believe that the current demo should be classified with variant 1 and not variant 2 of Spectre.
Variant 2 of Spectre has an important distinct property from variant 1: it involves injecting a target into the predictor that no correct execution of a branch could possibly reach. This is extremely different from simply training the predictor to go to one valid target, and then leveraging this trained target to trigger a misprediction.
The original naming from Intel also specifically highlights this: Branch Target Injection (BTI).
This is an extremely important distinction for a number of reasons:

It is impossible to conclude through static analysis of the program whether or not there is a variant 2 findable leak gadget because the injected target may be to an instruction stream not in the program source (similar to ROP-gadgets).
Mitigating variant 2 in software requires blocking misspeculation of the branch (retpoline, even LFENCE isn't perfect due to a race). Whereas mitigating a mistrained (but not injected) target for an indirect branch can be done in many other ways (and in fact SLH implements one of these other mitigations).
The Intel microcode mitigations for variant 2 only mitigate against injection, not against mistraining.

This is why I think it is really important to classify this as variant 1. Hope that explanation helps.

After checking the code, I have to agree with @cc0x1f here. This is an instance of Spectre-BTB (aka Spectre variant 2).
I disagree with your distinction. The distinction is based on which predictor is exploited (see [1], [2], [3]). Spectre-PHT (aka Spectre variant 1) targets the Pattern History Table which predicts the outcome of a conditional branch [2]. Spectre-BTB targets the Branch Target Buffer which predicts the location of an indirect call or jump. Similarly, Spectre-STL (aka Spectre variant 4) targets the memory disambiguation for predicting Store To Load (STL) data dependencies, and Spectre-RSB (aka Spectre variant 5) targets the Return Stack Buffer predicting the return address.
In all Spectre variants, the branch predictor is manipulated (which you call injecting). For both Spectre-PHT and Spectre-BTB, there are 4 different variants of injecting a different prediction (see [2], Figure 3).
In this specific case here, the BTB is manipulated in-place in the same address space (specifically Spectre-BTB-SA-IP in the academic naming convention). This version of Spectre-BTB is especially nice, as it is so easy to understand and reproduce as PoC. This is why we always use it in slides to illustrate Spectre-BTB (e.g., [4], slide 26 (page 184)), and we also have it in our repository ([5]).
Regarding your reasons:

I agree that Spectre-BTB is much more powerful in the sense that you can use it ROP style. And in this PoC, you can also do that. The PoC contains an indirect jump/call which can also be influenced from a different application, jumping to an arbitrary destination in the PoC.
In-place (what you call mistrained) and out-of-place (what you call injected) require different mitigations. Still, the underlying mechanism is the same, only the way to exploit it differs, and the current mitigations mainly prevent the exploitation, not the underlying mechanism.
That is not entirely correct. They only prevent injection from a different privilege level. One user-space application can still manipulate the branch prediction for a different user-space application. It also works inside the same application (think of JS sandboxes), this is not prevented by the microcode updates. Preventing in-place exploitation is a lot more difficult.

To conclude, this PoC is, as @cc0x1f said, Spectre-BTB not Spectre-PHT.
[1] https://spectreattack.com/spectre.pdf
[2] https://misc0110.net/web/files/transient_execution_attacks.pdf
[3] https://msrc-blog.microsoft.com/2018/03/15/mitigating-speculative-execution-side-channel-hardware-vulnerabilities/
[4] https://misc0110.net/web/files/beyond_belief.pdf
[5] https://github.com/IAIK/transientfail/tree/master/pocs/spectre/BTB/sa_ip

After checking the code, I have to agree with @cc0x1f here. This is an instance of Spectre-BTB (aka Spectre variant 2).

Spectre-BTB is much more broad than "Spectre variant 2" as it is used pervasively in the rest of the industry. "variant 2" is a subset of "Spectre-BTB" which is all I've been suggesting. I'm not saying anything about "Spectre-PHT" here...
And jumping a head a bit:

To conclude, this PoC is, as @cc0x1f said, Spectre-BTB not Spectre-PHT.

I'm not arguing about this classification as I've tried to say several times. I think in this regard, we are talking past each other a bit.
My firm position from working with all of the HW vendors and other parties as part of the cross-industry mitigation effort is that "variant 2" refers very specifically to the core issue of CVE-2017-5715 which Intel describes as "Branch Target Injection". The injection combined with the BTB are the two critical components for something to be "variant 2". Using the academic classification you have proposed, this is Spectre-BTB-CA-OP.
I do not think that "variant 2" should be used to refer to any other issues. I am very happy to provide a mapping from the classification you use in the research paper, but I is essential that "variant 2" only refer to the injection into the BTB.
The reason why this is so crucial is that there are mitigation techniques that are specifically documented as "mitigating variant 2" which only mitigate the injection into the BTB. They do not mitigate the issue in this demo! We cannot call this "variant 2" without creating serious and harmful confusion among those deploying mitigations. And we cannot go change that documentation to use a different classification scheme.
The specific mitigation strategy I refer to here is the use of IBRS and/or STIBP in conjunction with operating system changes to prevent cross-address space BTB injection. We have strong reasons to believe this is the most critical aspect to mitigate, and we have strong mitigations widely available and well documented as handling "variant 2".
I hope this makes clear why I am insisting on this narrow definition of the term "variant 2".
I want to again note that I'm not disagreeing that this demo is classified in a related space exactly as you indicate:

In this specific case here, the BTB is manipulated in-place in the same address space (specifically Spectre-BTB-SA-IP in the academic naming convention).

I 100% agree with this. I simply disagree that we can call this "variant 2". That will create active and harmful confusion as it is not mitigated by STIBP or similar hardware techniques.
I also wanted to respond to some of the other comments here:

Regarding your reasons:

I agree that Spectre-BTB is much more powerful in the sense that you can use it ROP style. And in this PoC, you can also do that. The PoC contains an indirect jump/call which can also be influenced from a different application, jumping to an arbitrary destination in the PoC.

The demo isn't about what unrelated vulnerabilities might exist, it is about which specific one is demonstrated. The existence of an indirect branch doesn't make this a meaningful demo of any other variation on Spectre-BTB, it is just the one you identified: Spectre-BTB-SA-IP. If we want to demo the others, we should craft narrow and specific demos of them.

In-place (what you call mistrained) and out-of-place (what you call injected) require different mitigations. Still, the underlying mechanism is the same, only the way to exploit it differs, and the current mitigations mainly prevent the exploitation, not the underlying mechanism.

Yes, exactly. But see above: the mitigations are documented to cover "variant 2", and due their narrow nature, it is essential we don't put other things into that set.

That is not entirely correct. They only prevent injection from a different privilege level. One user-space application can still manipulate the branch prediction for a different user-space application. It also works inside the same application (think of JS sandboxes), this is not prevented by the microcode updates.

This is not accurate.
First, the mitigations I'm referring to are not a single flag, but a set of changes to HW and system software. It is the conjunction of system software changes and availability of IBPB/IBRS/STIBP (in different combinations based on HW generation) for Intel parts for example.
Second, while some have elected to not fully deploy these mitigations for user-space applications due to the severe performance impact on older HW parts, newer HW parts from Intel and other vendors do not have that problem. There, even user-space applications have "variant 2" (as I described it above) fully mitigated when combined with the system software changes.

Preventing in-place exploitation is a lot more difficult.

Agreed. This is specifically why I do not want it classified in any way as "variant 2" which is widely believed to be thoroughly and easily mitigated.

I think the main argument was that this is not Spectre v1, which is clearly described as "bounds check bypass".
Also see https://software.intel.com/security-software-guidance/insights/deep-dive-analyzing-potential-bounds-check-bypass-vulnerabilities
Spectre v1 is clearly only Spectre-PHT.
Also see https://software.intel.com/security-software-guidance/api-app/sites/default/files/336983-Intel-Analysis-of-Speculative-Execution-Side-Channels-White-Paper.pdf
So, calling anything that is not Spectre-PHT "Spectre v1" is misleading.
Better, because the v1,v2,... naming scheme is unclear to many, we should altogether avoid it.

I think the main argument was that this is not Spectre v1, which is clearly described as "bounds check bypass".
Also see https://software.intel.com/security-software-guidance/insights/deep-dive-analyzing-potential-bounds-check-bypass-vulnerabilities

The original discussions with Intel and everyone else around this made it abundantly clear it wasn't just about bounds checks. No one is arguing "bounds check bypass" is good terminology.
I don't think "variant 1" vs. "Spectre-PHT" is more or less clear. I don't find "PHT" especially useful (there are too many forms of branch predictor for me to like naming after each one of them) and so I somewhat prefer the perhaps overly generic "variant 1".

Spectre v1 is clearly only Spectre-PHT.

Only on some microarchitectures which use a pattern history table? There are other branch predictors that are still susceptible to the same core idea...

Also see https://software.intel.com/security-software-guidance/api-app/sites/default/files/336983-Intel-Analysis-of-Speculative-Execution-Side-Channels-White-Paper.pdf
So, calling anything that is not Spectre-PHT "Spectre v1" is misleading.

Some find "variant 1" misleading. Some find the "PHT" taxonomy misleading....

Better, because the v1,v2,... naming scheme is unclear to many, we should altogether avoid it.

I don't think we will find a perfect naming scheme.
But I think we might simply have a nice table that tracks all of the variations here and explains how they relate to each other.

The original discussions with Intel and everyone else around this made it abundantly clear it wasn't just about bounds checks. No one is arguing "bounds check bypass" is good terminology.

But that's what variant 1 was about.

I don't think "variant 1" vs. "Spectre-PHT" is more or less clear. I don't find "PHT" especially useful (there are too many forms of branch predictor for me to like naming after each one of them) and so I somewhat prefer the perhaps overly generic "variant 1".

But these branch predictors might need different kinds of mitigations.

Only on some microarchitectures which use a pattern history table? There are other branch predictors that are still susceptible to the same core idea...

Yes, other architectures might have a microarchitectural element with a different name serving the same purpose.

Better, because the v1,v2,... naming scheme is unclear to many, we should altogether avoid it.

I don't think we will find a perfect naming scheme.
But I think we might simply have a nice table that tracks all of the variations here and explains how they relate to each other.

My impression is that what you actually want is something like this:

You care about the in-place same-address-space variants, regardless which branch predictor that is (as shown on the slide). Neither v1 nor v2 express that, both mean other things.
The only naming scheme (that I'm aware of) that allows to make this distinction is the one you call the academic one (though there is no alternative non-academic one), and there it would be Spectre-*-SA-IP.

Hey sorry about the huge delay getting back to you, it was a really busy time of year. (Cppcon, performance review, ...)  I wanted to sit on this reply to give others a chance to respond, but they didn't and I feel bad that I've left you hanging.
I really appreciate this issue being raised, since I also was super confused about the exact boundaries being drawn around Spectre v1 vs v2, and had a couple iterations of this kind of discussion in private without gaining as much understanding of the issue as this thread has brought me. So, thank you!

But that's what variant 1 was about.

This seems like a major part of the disagreement worth elaborating on. Calling it "bounds check bypass" is IMO definitely overspecifying it, and that term/concept is worth abandoning forever. The specific code pattern of a direct branch based on a comparison of index against array length, as opposed to any other direct branch, does not change how the vulnerability works or how it can be removed. The same vulnerability/mitigations will work whenever a memory address can be made to point to a secret by mistraining the direct branch predictor. (Much of the discussion here revolves around indirect branches, but at least direct branches are all the same microarchitecturally, right?) SLH works on more than just bounds checks, for example!
(We get into messier territory if we want to talk about predicted branches that lead to a secret, but not via a speculative buffer overflow. IDK, what do you call that?)
As an example, many months ago I modified our spectre v1 example in a local copy to instead never use (or, well, rely on the use of) an out-of-range index, not even speculatively. Instead I wanted to execute a speculative bypass of the condition below, the "short string optimization":
class MyShortStringOptimization {
  size_t size_;
  union Data {char* p; char s[sizeof(p)];};
  Data data_;

 public:
  size_t size() const { return size_; }
  const char& operator[](size_t n) const {
    if (size_ >= sizeof(data_.s)) {
      // is this so big we need to store in the heap instead?
      return data_.p[n];
    }
    return data_.s[n];
  }
};

It worked perfectly, and was prevented perfectly by SLH!
In this example, we can train the branch predictor to think that the string is short, and then pass it a string (and index) that is larger than kShortLength, and get a speculative buffer overflow. This is true even if the index is always within bounds. This shows that from a certain point of view, it doesn't matter what the condition is, what matters is that the address is user-controlled so that they can manipulate it to point to the secret that they want to extract -- and they can do this without ever doing a bounds check bypass. SLH works on that generalized basis, and it's that generalized problem that we are interested in addressing.

But these branch predictors might need different kinds of mitigations.

This is a good point. I hope we do that as little as possible, though.
Even though indirect and direct branches are different on the CPU, they are not different in the C++ programming language (or any other high level programming language). Any if statement can become an indirect branch, and any virtual function call can become a direct branch -- not just the direct/indirect branches that we expect, respectively. As a result, we likely don't want to have separate mitigations for the two cases, even if we do need separate test cases. (Which are necessarily flawed for the same reason, in some way we might need to fix.)
For what it's worth, I think that the original naming scheme had a not-unreasonable basis, and mapped cleanly to our intent. The new naming scheme doesn't completely capture the right conceptual boundaries, and also is a lot of opaque letters. Also, it will be hecking merge conflicts. So I am -1 on renaming the file unless we have a really solid name everyone can agree is perfect.
The README change I am more sympathetic to, but I think we need a dedicated index rather than putting it in the README. There's too many overlapping names and this would get big, fast.

I'm aware that SLH works on more than just direct branches (depending on the specific case). But a systematization of attacks should not be tailored to a specific defense, right? Otherwise we could tailor a systematization also to retpoline and it might not look more useful then either.
I agree that it doesn't make sense to group it by the bounds check bypass, but that's also not what we propose.
But I would argue that it would be difficult to find a proper definition for Spectre v1 and Spectre v2 that captures all the things you want to mitigate under the umbrella term Spectre v1 and Spectre v2 respectively.

The same vulnerability/mitigations will work whenever a memory address can be made to point to a secret by mistraining the direct branch predictor.
There are direct-branch Spectre gadgets that can be exploited that don't fulfill that property. For instance, leaking data from registers. Leaking data from a static memory address that cannot be influenced by the attacker.

And I agree, branches are not always directly exposed to the programming language. But the systematization of attacks should not be based on programming languages as that would mean that we need different systematization for every programming language!?
The systematization on predictor and mistraining location is better as it is oblivious to the programming language and specific attack examples.


New Rule: prefer-return-typedef

Rule suggestion
I sometimes see unnecessary typedefs or type assertions in return statements that instead belong on the method signature directly:
// bad
function makeBlockScope() {
    return {
        ...
    } as IBlockScope;
}

// good
function makeBlockScope(): IBlockScope {
    return {
        ...
    };
}

+1

@adidahiya This is somewhat what I was searching. My rule suggestion would be a rule, that forces a return type definition on each function/method. Can you also agree to this?
Or do you specifically want to alert on returns with typedefs without matching typedefs on the function/method?
My good/bad example would be this:

// bad
function something() {
  console.log('bla')
}

// good
function something(): void {
  console.log('bla')
}

My use case is, that the developer tends to forget to write the function typedef.

@adidahiya ARGH. I missed that and was looking for it for a long time. Thanks, mate!

Note: per #4534, this issue will be closed in less than a month if no PR is sent to add the rule. If you really need the rule, custom rules are always an option and can be maintained outside this repo!

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for new rules. See #4534. 😱
If you'd like to see this rule implemented, you have two choices:

Recommended: Use typescript-eslint to lint your TypeScript code with ESLint ✅
Legacy: Write it as a custom TSLint rule 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


Increase prometheus timeout

What type of PR is this?
/kind bug
What this PR does / why we need it:
We are seeing that in some cases, the stateful pod (like prometheus) can be rescheduled and it can take even ~7m to start again due to volume remounting/6m force disk detach timeout.
Let's increase timeout to avoid failing test when this happens.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/assign @azylinski

/assign @marseel


Revert "Rename seconds to Ms for IM subscribe"

In spec, it do use seconds instead of Ms.
Reverts project-chip/connectedhomeip#8285

Thank you. The other problems with this code (not matching the IM spec, using the wrong tag values) are still outstanding. :(

it is fixed here https://github.com/project-chip/connectedhomeip/pull/8311


The curation API notebooks do not print human readable error message for failures

See [curation-api}(https://czi-sci.slack.com/archives/C02UGUV5R3J/p1659380082383059).
PUT /v1/collections documents that a BAD REQUEST status code includes diagnostic information will be returned such as:
{'detail': "'description' is a required property",
 'status': 400,
 'title': 'Bad Request',
 'type': 'about:blank'}

Currently, the HTTP request is caught, but is then re-thrown in failure:
def failure(logger: logging.Logger, e: Exception, message=None):
logger.error("\n\033[1m\033[38;5;9mFAILED\033[0m")  # 'FAILED' in bold red
if message:
logger.error(message)
raise e
resulting in:
FAILED
HTTPError                                 Traceback (most recent call last)
Input In [28], in <cell line: 1>()
----> 1 create_collection(collection_form_metadata)
File ~/Notebooks/curation_api/python/src/collection.py:25, in create_collection(collection_form_metadata)
23     data = res.json()
24 except Exception as e:
---> 25     failure(logger, e)
26 else:
27     collection_id = data.get("collection_id")
File ~/Notebooks/curation_api/python/src/utils/logger.py:43, in failure(logger, e, message)
41 if message:
42     logger.error(message)
---> 43 raise e
File ~/Notebooks/curation_api/python/src/collection.py:22, in create_collection(collection_form_metadata)
20 try:
21     res = requests.post(url, data=json.dumps(collection_form_metadata), headers=headers)
---> 22     res.raise_for_status()
23     data = res.json()
24 except Exception as e:
File ~/.pyenv/versions/3.9.11/envs/env/lib/python3.9/site-packages/requests/models.py:960, in Response.raise_for_status(self)
957     http_error_msg = u'%s Server Error: %s for url: %s' % (self.status_code, reason, self.url)
959 if http_error_msg:
--> 960     raise HTTPError(http_error_msg, response=self)
HTTPError: 400 Client Error: BAD REQUEST for url: https://api.cellxgene.dev.single-cell.czi.technology/curation/v1/collections

This is discussed here. It actually is not due to my catching and rethrowing -- it's a Python issue that requires that error messages be extracted manually.

Thanks for the clarification Dan.


Show extension recommendations

Feature
Show most recommended extensions in the extension center by default. For now instead of automating this based on analytics (as extension center don't have a good analytics system in place yet), this will be done manually.

[ ] create a separate recommended.toml within the extension repository
[ ] fetch this file also with the original extensions.toml
[ ] show the recommended content only in extensions view by default instead of showing all the extensions
[ ] #409

There doesn't seem to be a help wanted label, is this issue open for contributors?

@shirsakm issue is now fully documented, would you like to try implementing this?

Hi, yes I would love to work on the issue, but I need some pointers to get started.
Firstly, is there an extensions.toml file, because I cannot seem to find it at all. Secondly, a pointer to where exactly the extensions are being displayed will be helpful.

@shirsakm pardon, I forgot to include the changes to be done to the extension manager before this issue will be worked on. I have edited this issue to include https://github.com/tomlin7/biscuit/issues/438

There seems to be a quite an overhead on this issue. Perhaps, #438 should be worked on first. I will look into it.

@shirsakm #438 is done, have a look at the changes as well!

@tomlin7 Looks like the prerequisites are done, I will start working on this in a few days.


Add Volume controls to interactions that need it

To be added to:

Editor
Player (VR and flat screen)
Video interactions
Audio interactions


fix: fix unit test failure

Signed-off-by: moweiwei<EMAIL_ADDRESS>
What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #8
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

Codecov Report

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

@@            Coverage Diff            @@
##             master       #9   +/-   ##
=========================================
  Coverage          ?   48.33%
=========================================
  Files             ?        6
  Lines             ?      542
  Branches          ?      150
=========================================
  Hits              ?      262
  Misses            ?      219
  Partials          ?       61

/assign lysign

/lgtm
/approve


Flexibility to set device number < total device count

Before submitting

[X] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #152. However, this implementation is not fully correct, it will break if there is more than one node.

@tchaton is there a way to get the devices variable?

the problem might be more complicated than I thought, usually devices is set in the lightning Trainer class, but _DistributedEnv is called before that


Changed popup window

I think if the background is transparent its hard to see on white background for example.
I also removed the titlebar cuz the options yes and no are enought.
Finally i changed some basic styles.
Preview image:

@Waradu thanks for the pr


Velos Observation extract has duplicates

From Kevin: One small issue with the Velos Observation extract.  There are some duplicates – i.e. rows with the same SOURCE_PK that appear multiple times (76 rows).  In reviewing the data, I think a join to the provider table is causing the problem because in every case the only thing that differs on the row is the value of PROVIDER_ID.   My recommended fix would be to add “-“||PROVIDER_ID to the end of the source_pk value.  Your call though.

Per Kevin: Duplicates being returned for unique Study-id, person-code (MRN) and Patient-Study_status-id due to multiple PIs being returned.
Resolution: The problem is that there is only a PI-name in MINIVELOS.DM_PATIENT_STATUSES, not an ID; So we must link on PI-name to get PI-id from the eVelos User table.  However, a PI has multiple entries in the User table based on the site at which the study is being conducted.
-- Ex: Dr McGuire User-ID= 2292 for care-site 50 (UNM), but she has id = 472 for care-site 58 (Veteran's admin)
Solution:  added  condition usr.FK_siteID = 50 to Join between MINIVELOS.DM_PATIENT_STATUSES and MINIVELOS.ER_USER (usr)
Data ready for upload 4/21/22


TR-12	Ineffective storage and encryption

@nbrown-ScottLogic  @gkocak-scottlogic as per MS review (https://github.com/finos/ai-readiness/issues/79):
TR12 (Ineffective Storage and Encryption) overlaps with confidentiality/integrity controls (e.g., Control 1). Could these be streamlined or better cross-referenced?
Consider adding “data in transit” encryption if “data at rest” encryption is mentioned, for consistency. Alternatively, remove generic controls not specific to AI to avoid duplication


Unable to configure handler 'sentry' with raven-python 6.1.0 and django 1.5

I have tried upgrading the raven client from 6.0.0 to 6.1.0 on my Django 1.5 app (yes, it’s old :) and hit the following error (when running it with UWSGI):
Error in sys.excepthook:
Traceback (most recent call last):
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/base.py", line 276, in handle_exception
    self.captureException(exc_info=exc_info, level='fatal')
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/base.py", line 818, in captureException
    'raven.events.Exception', exc_info=exc_info, **kwargs)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/contrib/django/client.py", line 292, in capture
    result = super(DjangoClient, self).capture(event_type, **kwargs)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/base.py", line 638, in capture
    **kwargs)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/contrib/django/client.py", line 233, in build_msg
    data = super(DjangoClient, self).build_msg(*args, **kwargs)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/base.py", line 451, in build_msg
    data['modules'] = self.get_module_versions()
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/base.py", line 303, in get_module_versions
    modules = get_versions(self.include_paths)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/utils/__init__.py", line 112, in get_versions
    __import__(module_name)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/contrib/django/__init__.py", line 19, in <module>
    initialize()
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/contrib/django/models.py", line 257, in initialize
    get_client()  # NOQA
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/contrib/django/models.py", line 133, in get_client
    instance = Client(**options)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/contrib/django/client.py", line 140, in __init__
    self.install_sql_hook()
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/contrib/django/client.py", line 143, in install_sql_hook
    install_sql_hook()
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/utils/__init__.py", line 185, in new_func
    rv = func(*args, **kwargs)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/raven/contrib/django/client.py", line 81, in install_sql_hook
    from django.db.backends.utils import CursorWrapper
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/db/__init__.py", line 40, in <module>
    backend = load_backend(connection.settings_dict['ENGINE'])
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/db/__init__.py", line 34, in __getattr__
    return getattr(connections[DEFAULT_DB_ALIAS], item)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/db/utils.py", line 93, in __getitem__
    backend = load_backend(db['ENGINE'])
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/db/utils.py", line 45, in load_backend
    raise ImproperlyConfigured(error_msg)
django.core.exceptions.ImproperlyConfigured: 'django.db.backends.dummy' isn't an available database backend.
Try using 'django.db.backends.XXX', where XXX is one of:
    u'mysql', u'oracle', u'postgresql_psycopg2', u'sqlite3'
Error was: cannot import name util

Original exception was:
Traceback (most recent call last):
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/myapp/wsgi.py", line 23, in <module>
    import django.test
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/test/__init__.py", line 5, in <module>
    from django.test.client import Client, RequestFactory
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/test/client.py", line 21, in <module>
    from django.db import close_connection
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/db/__init__.py", line 11, in <module>
    if settings.DATABASES and DEFAULT_DB_ALIAS not in settings.DATABASES:
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/conf/__init__.py", line 52, in __getattr__
    self._setup(name)
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/conf/__init__.py", line 48, in _setup
    self._configure_logging()
  File "/usr/lib/myapp/local/lib/python2.7/site-packages/django/conf/__init__.py", line 82, in _configure_logging
    logging_config_func(self.LOGGING)
  File "/usr/lib/python2.7/logging/config.py", line 794, in dictConfig
    dictConfigClass(config).configure()
  File "/usr/lib/python2.7/logging/config.py", line 576, in configure
    '%r: %s' % (name, e))
ValueError: Unable to configure handler 'sentry': Cannot resolve 'raven.contrib.django.handlers.SentryHandler': cannot import name util

Downgrading to 6.0.0 solved the issue.

This was fixed in #1033.

Actually, #1033 does stop this issue - however it just masks the real problem. Exceptions from manage.py commands don't get reported.
The ImportError occurs at: https://github.com/getsentry/raven-python/blob/master/raven/contrib/django/serializers.py#L63, but gets caught by https://github.com/getsentry/raven-python/blob/master/raven/contrib/django/models.py#L262
I'll see if I can diagnose it further and raise a new issue for it.


Duplicate restart words in Additional Options section?

I didn't notice this in 1.6.1, but in 1.6.2 - there appears to be duplicates of shutdown and power off in the "Additional options" section.  Is this a typo or there by design?  If by design, what are the differences?

Should be fixed in the latest version

Confirmed.  Thanks!


Direct SSH to PWD instance not working

Trying to ssh to a fresh instance, but got connection refused.
ssh -vv<EMAIL_ADDRESS>debug1: Reading configuration data /etc/ssh/ssh_config
debug2: resolving "direct.labs.play-with-docker.com" port 22
debug2: ssh_connect_direct
debug1: Connecting to direct.labs.play-with-docker.com [<IP_ADDRESS>] port 22.
debug1: connect to address <IP_ADDRESS> port 22: Connection refused
ssh: connect to host direct.labs.play-with-docker.com port 22: Connection refused

Already check some similar closed issues about ssh not working, but got no solution.

Fixed

Fixed

I also have the same problem.
debug1: Reading configuration data /etc/ssh/ssh_config
debug2: resolving "direct.labs.play-with-docker.com" port 22
debug2: ssh_connect_direct
debug1: Connecting to direct.labs.play-with-docker.com [<IP_ADDRESS>] port 22.
debug1: connect to address <IP_ADDRESS> port 22: Connection refused
ssh: connect to host direct.labs.play-with-docker.com port 22: Connection refused


Automated cherry pick of #2558: fix: sync and set secgroup public scope

Cherry pick of #2558 on release/2.11.
#2558: fix: sync and set secgroup public scope

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@a43839c). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #2561   +/-   ##
==============================================
  Coverage                ?   7.03%
==============================================
  Files                   ?     455
  Lines                   ?   72136
  Branches                ?       0
==============================================
  Hits                    ?    5076
  Misses                  ?   66642
  Partials                ?     418

Flag
Coverage Δ

#aFlag
7.03% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/guests.go
0.67% <0%> (ø)

pkg/compute/models/secgroups.go
2.42% <0%> (ø)

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 a43839c...63d582a. Read the comment docs.

/lgtm

/approve