uwu: remove compression

Content Encoding / Compression is done at Caddy / HAProxy level.
No need to compress images and videos with gzip as these are pre-compressed formats already.

Completed in v3.


Feature: wait2

This fixes #18 by exposing a wait2 method in DaemonRunner::ShellOut.

Thanks for adding this @fmitchell-r7


referer for AGSTokenSecurityHandler

I've forked ArcREST on the 20th April (db5f57304f5c20e9855c5f60e5eafe23e3476106) and am trying to use AGSTokenSecurityHandler with a referer based token and a GPTask. Unfortunately, although it retrieves a token correctly, it looks like it does not add the referer header to the submit job request.
I'm currently hacking it to get my POC working, but, firstly, wondering if I've missed something, as it looks like it's been implemented for the PortalTokenSecurityHandler, but AGSTokenSecurityHandler seems to be hard coded to requestip. Secondly, is this something that you could easily implement? I'm writing some pretty hacky code to get it working, probably partly because I'm struggling to understand the class hierarchy, and there appears to be urllib handlers and then ArcREST handlers, which I think are different things...
I'm happy to implement it if you can give me some guidance as to the best way, but my current code really isn't that great.
Cheers,
Adam

@adamkerz I'll take a look at it today.

Just committed a fix.  Please pull the latest.
import arcrest
if __name__ == "__main__":
    username = "username"
    password = "password"
    referer = "mysite.someotherpart.com"
    org_url = "https://mysite.someotherpart.com/arcgis"
    url = "https://mysite.someotherpart.com/arcgis/rest/services/SampleWorldCities/MapServer"
    sh = arcrest.AGSTokenSecurityHandler(username, password, org_url=org_url)
    sh.referer_url = referer
    ms = arcrest.ags.MapService(url=url, securityHandler=sh, initialize=True)
    print ms.capabilities


Redirect to plugin page after enabling to show configuration settings…

… if applicable.

I'm generally supportive, but if you have a new site or a new repository and try to mass enable a whole bucketload of plugins this is going to be very annoying.

There's also an issue that this will direct to the plugin detail page when you disable the plugin. I attempted to provide an improved block you could cut/paste but can't get markdown to not mangle it. So I'm just going to check it in. Basically it only redirects to the plugin detail page if the plugin has just been enabled and has an admin config block.

I knew there would be some details to hammer out. Thanks! I think this is an improvement. Also I wasn't aware of a bulk plugin activation method until this discussion.


[DF] Port Nested Subqueries to Datafusion

Is your feature request related to a problem? Please describe.
Nested subqueries need to be ported from the Apache Calcite data structures to the Apache Datafusion data structures.
Describe the solution you'd like
All references should be removed from select.py, and likewise any other locations where subqueries could be present. All instances of calcite references removed should be replaced with the correlating Apache Datafusion data structures.
Success criteria for this port being complete is all pytests passing for test_select.py
Describe alternatives you've considered
None
Additional context
None

This depends on https://github.com/apache/arrow-datafusion/issues/2219


Processing rules in TODO need to be updated

@Antti-L noticed that the following assumptions in TODO.md are incorrect:

PrimaryCG: if missing, use the output commodity
If Attribute=EFF: Change Comm-IN to ACT; Change Attribute to CEFF
If Attribute=OUTPUT: Change Comm-IN to Comm-OUT-A; Change Attribute to CEFF

If Attribute=OUTPUT: Change Comm-IN to Comm-OUT-A; Change Attribute to CEFF

Thanks @olejandro. If Attribute is OUTPUT, indeed mapping it to CEFF would be quite wrong.
The OUTPUT Attribute should be mapped to ACT_FLO (aka VDA_FLOP), like VEDA does. But if OUTPUT is defined for a PCG commodity, then it should be just ignored.  In addition, ACT_FLO (VDA_FLOP) should be supported as well, with the COM/CG optionally specified in Other_Indexes (instead of Comm_IN / Comm_OUT or using the ~COM syntax in a FI_T).

PrimaryCG: if missing, use the output commodity

Indeed that is incorrect, because there is a set of rules for determining the PCG if not specified, and it is much more complex than "use the output commodity".  For example, the default PCG may consist of several outputs or inputs, but only commodities of one single commodity type are included in th PCG (according to type priorities), and any auxiliaries are excluded.  Moreover, ouputs could be missing altogether, and then the PCG is determined from the input flows...


Define git sync strategy

The reason why I think we should sync on this is to incorporate the consequences of syncing with git later. It requires to organize the Soda Cloud editor platform also around concepts like proposals that have to match with PR's and a concept of a Soda Cloud repos (multiple) that has to match with git repos. And we need data sources and contracts to be configured in one of these repos.  Proposals need to group a bunch of file changes together like in a PR.
Initially we'll build a mechanism that stores all files in Soda Cloud.  But the target is that later we build a git repository synchronization.
If we want git synchronization, the models have to match.  That means that Soda Cloud will need to have a list of repositories.  Also in the case that only Soda Cloud storage is used.  Each of the repositories can contain data source and contract yaml files.  We will need editors for both file types.
When a user opens a contract editor, a 'clone' has to be performed of the full repository.  This means that for that user, a copy of the full repository needs to be made and associated to the user.  The user changes the cloned files in the editor session and when saving, a patch file is created.  This patch file can be applied immediately to the Soda Cloud repo files.   Or the patch file can be linked to a proposal for later merging.
When a patch is saved, the Soda Cloud repository files are updated.  In case a git repository is linked, the updates are pushed to git.
Concurrency handling: we need to decide if we allow for concurrent proposals and concurrent updates.  If we do this, it will complicate the solution significantly.  In that case we have to work with git patch files.  Potentially this results in conflicts to be resolved in the Soda Cloud web UI editor.  This pushes us in a direction of building a web UI git client to do merges.  It's complex and not core to our value.  Alternatively we could have a single-write-lock implemented in Soda Cloud editor UI so that there is max 1 editor at any time and conflicts are avoided.

CLOUD-8501

Moved https://sodadata.slite.com/api/s/GfW0Vv1_MfFBWQ/Git-synchronization-strategy


Twig issue with Validation reader

When I do (taken from https://github.com/endroid/QrCode#twig-extension):

... I get:
An exception has been thrown during the rendering of a template ("Validation reader read "" instead of "lorem ipsum"").

I have twig/twig 2.3.2 with Symfony 2.8.19.

I updated the exception message so it is more descriptive. If you want to use validation and think the qr code should be readable, please create a failing test case in tests/QrCodeTest.php so I can see if it's something in the library or in one of the dependencies.

You can't reproduce?
When I do

or

I now get (v2.0.5):
An exception has been thrown during the rendering of a template ("Warning: strlen() expects parameter 1 to be string, array given")

...with a reference to vendor\bacon\bacon-qr-code\src\BaconQrCode\Writer.php at line 78:
if (strlen($content) === 0) {

Fixed with https://github.com/endroid/QrCode/pull/99


GEOMESA-1165 Allow configurable shards in z2 (and z3)

Signed-off-by: Charles Kelly<EMAIL_ADDRESS>

Emilio merged the WIP PR, so I'm closing this now.


Fixed the WARNING: Task running is currently disabled. message appear…

Fixed the WARNING: Task running is currently disabled. message appearing even when tasks are enabled.

Description
This warning message is wrongfully displayed because the setting that's being check is Tasks.logPerformance instead of Tasks.enabled
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.673%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Commands/Lister.php
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
332

Relevant Lines:
422

💛  - Coveralls

@ChoMPi0 Thank you!


If from Action "Apply on shipping " is selected as yes then the discounted amount is applying on both product and shipping charge.

Bug report
Title
If from Action "Apply on shipping " is selected as yes then the discounted amount is applying on both product and shipping charge.
Issue Description
If from Action "Apply on shipping " is selected as yes then the discounted amount is applying on both product and shipping charge.
Preconditions
2. Commit id.-

Steps to reproduce
1. step1- Create a cart rule and in action select yes for Apply on shipping.
2. step2- Place an order by selecting flat rate shipping.

Expected result
Discount should only apply on shipping charge
Actual result
Discount gets applied on both shipping and product

Framework Version: 0.1.9

Fixed


Images with no annotations

The torchvision model will throw an error if images with no annotations are passed to it. I think we need to preemptively  remove these images.
Note that a transform (like random crop or zoom) can remove the items from an image (potentially leaving it with no objects).
Related to this and this

A solution discussed in the forum given by the link is to create a fake bounding box with label 0 (background) when no object is present. Quoting the forum:
"I set the values to be (0,1,2,3) for label “0”."

And the images with no annotations should already by removed by the DataParser, so this problem only happens if a transform removes all objects from the image


csi: add nodeExpandSecret KEP details

Ref#
KEP: https://github.com/kubernetes/enhancements/pull/3173/
Implementation: https://github.com/kubernetes/kubernetes/pull/105963
Blog: https://github.com/kubernetes/website/pull/33979
Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

Cc @xing-yang @sftim

/assign

@sftim do we need to get this merged before any deadline ?

The deadline for merge is the v1.25 release date. However, there is also a deadline for having PRs ready to review (which is this week, I think).

/milestone 1.25
/assign @krol3
/cc @kcmartin

/lgtm

I believe this is good to go. Lift the hold if that is true.
/approve
/hold

@tengqm LGTM too, any improve or we can remomve the hold?

/hold cancel


run hapi-fhir-jpaserver-example in STU3 mode

As of hapi-fhir version 2.1-SNAPSHOT, please modify hapi-fhir-jpaserver-example to run in STU3 mode.

Hi @joelsch, now that STU3 is released I'm going to merge this. Thanks for the contribution!


[GithubCommitsSince GithubDownloads GithubRelease] GithubDownloads supports latest release by SemVer

Resolves #4682.

Warnings

:warning:
This PR modified service code for github but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @hjagodzinski!

  Generated by :no_entry_sign: dangerJS against a23e79c1376ac2d6df7dcc605bd10f1e9c843128

Thank you for the PR @hjagodzinski and apologies for the delay in review. Would you mind adding a couple tests to give us some validation of the behavior?
In this case I think using mocked service tests would probably be easiest in this case because:

test conditions too difficult to find in a real repo, and doesn't seem worth setting up a dummy repo with actual releases to use as the test target
there has to be so much logic within handle and trying to mock everything out with a unit test would be too painful

@calebcartwright I understand completely you guys have other obligations as well and finding time is a challenge at times.
I was felling uncomfortable with this changed not having any test. Having in mind our previous discussion I was not sure how should I approach this and decided I will follow your guidance.
I assume there won't be much more overhead introduced by the service tests than by the unit tests. Provided a service request mocking is inevitable in both cases the service tests seem more appealing as being easier to write (as you already noted). I have covered with new tests all four cases that this change regards.

I was felling uncomfortable with this change not having any test. Having in mind our previous discussion I was not sure how exactly should I approach this and decided I will follow your guidance.

These are among our most popular badges (I believe Downloads/Release are top 10 actually) so the additional tests are great, thanks so much!

I assume there won't be much more overhead introduced by the service tests than by the unit tests. Provided a service request mocking is inevitable in both cases the service tests seem more appealing as being easier to write (as you already noted). I have covered with new tests all four cases that this change regards.

Understood. It's more of a general strategy and preference for leveraging unit tests when possible whereas historically the only way anything was tested within the class was with complex mocked service tests. Most of the time using unit tests for logic and service tests for high level functional and integration validation works out for the best, but there's definitely still cases like this one where we just really need the mocked service tests :+1:


Fix TileSetAtlasSourceEditor performing many expensive redraws of the atlas grid.

Editing decently sized tile sets is really laggy and painful to use. This commit tries to reduce that friction in two ways.

Clipping intensive rendering of the potentially thousands of drawn rects to only the visible area and
Avoiding some calls to tile_atlas_view->queue_redraw() when dragging around in the atlas view editor.

Supersedes #84963 by maintaining the existing rendering logic.
A more comprehensive fix would be to perhaps batch and invalidate this expensive rendering in a more piecewise fashion in some kind of quadtree structure -- in the mean time this PR fixes the performance when zoomed in to reasonable values.

Dang it, just saw https://github.com/godotengine/godot/pull/92765! I guess both have their different advantages.


Adding arm64 to our release binaries

We should probably test on arm64.  Otherwise... surprises.

The drive was to have something working on M1 -- so I just run some basic commands to see that it is working natively.
For using lakeFS on arm64 we can use AWS Linux for testing.


tls context: minimum and maximum TLS protocol version

Description
This PR adds configuration for setting minimum and maximum TLS protocol version.
Related Issues
https://github.com/datawire/ambassador/issues/689
Todos

[ yes] Tests (needs improvement)
[ no] Documentation

@gsagula tests are failing, can you check? ;)

We definitely need better tests for this, but taking for 0.61 as we've verified manually as well as with the extant tests.


apiserver/controller/service.go has third party library

In GitLab by @Robin-Mao on Apr 19, 2018, 16:24
If you are reporting a problem, please make sure the following information are provided:

Version of board (last git commit number) , docker engine and docker-compose.
Config files of board, you can get them by packaging "board.cfg" and files in the same directory, including subdirectory.
Log files, you can get them by package the /var/log/board/.
Other necessery information.

In GitLab by @Robin-Mao on Nov 8, 2019, 07:49
"k8s.io/client-go/kubernetes"
"k8s.io/client-go/pkg/api/v1"

In GitLab by @luotian-github on Nov 8, 2019, 07:49
Fixing

In GitLab by @luotian-github on Nov 8, 2019, 07:49
move cleank8s and stopk8s to backend service from controller

In GitLab by @luotian-github on Nov 8, 2019, 07:49
PR#1234

In GitLab by @Robin-Mao on Nov 8, 2019, 07:49
fixed

In GitLab by @sokril on Nov 8, 2019, 08:21
closed

In GitLab by @sokril on Nov 8, 2019, 08:21
assigned to @luotian-github


Remove unneeded stubs from tests

The stubs aren't needed in the tests and can be replaced by a simple Connection mock.
This will make it easier to create tests for an upcoming PSR-7 PR.

Updated. Checkout the newes commits.


RealSense Viewer Crashes when opening bag file

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
<Describe your issue / question / feature request / etc..>
Hi, I am currently using a d415 for some data collection that gets uploaded into dropbox, and then retrieved at a different computer.
When I try to open the recorded .bag files, the realsense viewer software closes with no error message, it just closes and stops. Any idea why this could happen?

Hi @ortego26
ROS-bag file formatter is suffering from an issue where if the recording is stopped abruptly, the resulting bag file would not get indexed. I suspect this may be the case.
I would also try copying the file to main hard-drive (as opposed to network drive)
One of the tools within the SDK is called rosbag-inspector and is accessing files using ROS-bag library directly (not via SDK API). It can also offer some information about the file

Hi @dorodnic
That's what I was suspecting, but when the data is collected locally, it can be accessed no problem but then after upload and download in a different computer, even after moving the file to the hard drive, it seems to crash the viewer.
Here is a snippet of the code that I suspect is not stopping the recording abruptly.
Get current time to name the file
    Date = time.asctime(time.localtime(time.time())).replace(':','-')
    file_path = bagfiles_dir_path_new + str(int(time.time())) + ' ' + Date + '.bag'

    # Configure depth and color streams
    pipeline = rs.pipeline()
    config = rs.config()
    config.enable_stream(rs.stream.depth, 1280, 720, rs.format.z16, CAMERA_FRAMERATE)
    config.enable_stream(rs.stream.color, 640, 480, rs.format.bgr8, CAMERA_FRAMERATE)
    config.enable_record_to_file(file_path)

    # Start streaming
    print("creating bag file "+file_path,flush=True)
    pipeline.start(config)
    t0 = time.time()

    try:
        while True:
            # Wait for a coherent pair of frames: depth and color
            frames = pipeline.wait_for_frames()
            depth_frame = frames.get_depth_frame()
            color_frame = frames.get_color_frame()
            if not depth_frame or not color_frame:
                continue
            if (time.time() - t0) >= SECONDS_PER_RECORDING:
                print("saving bag file "+file_path,flush=True)
                break
    finally:
        # Stop streaming
        pipeline.stop()
        del(pipeline)
        del(config)

That's very strange, perhaps permissions issue? Linux / Windows?

Hi @dorodnic. I found the issue. On the system that was working before (Windows) the version of the pyrealsense was <IP_ADDRESS>5
When I set up a new system for data collection, I created a new environment that was using the pyrealsense2 version 2.33.1336
Reverting to the original fixed the bag files and they do not crash anymore.
Maybe this was an issue with that update?

Hi ortego26,
Will you be needing further help with this?
Are you still experiencing this on latest librealsense? As a general rule of thumb I would keep the versions of the SDK on the system that's recording the bag file and the one that's playing it the same.
If we don’t hear from you in 7 days, this issue will be closed.
Thanks


Refactor discrete zooming into delayed continous zooming

The discrete zooming capabilities could be improved significantly. The problems with apple's magic mouse (#4410) and smooth-scrolling touchpads are becoming greater; the scrollwheel events have a time delay right now; and other map libraries allow for keeping the zoom buttons pressed for a smooth zooming effect.
IMHO, all the zooming handlers should rely on the same method: L.Map._move, the one that Map.Handler.TouchZoom uses right now. Ideally this would allow for better zoom snapping and a scroll-zooming with faster UI response.
The idea is to make a scroll-zoom interaction the same as a touch-zoom interaction: when it starts, fire a zoomstart, fire zoom on each wheel event, and fire zoomend (and snap) after the wheelDebounceTime timeout.
Similar approach for keyboard or L.Control.Zoom: fire zoomstart on keydown/mousedown, fire zoom every frame, then zoomend (and snap) on keyup/mouseup.
Unless the Safari issue is very pressing, this can be pushed back into the queue.
See also #4610

Would this also mean ditching CSS Transitions based zoom animations?

@mourner Probably yes. I'm not sure CSS transitions offer a big performance boost over resetting the CSS transform every animation frame.

@IvanSanchez I found the nicest experience for continuous zooming by turning off animation altogether and just letting each (small) zoom event make a small zoom level change, by using a different sigmoid for trackpads / magic mouse and a different for discreet zooming.
Have a look at my older branch here:
https://github.com/hyperknot/Leaflet/blob/smooth-zoom/src/map/handler/Map.ScrollWheelZoom.js

I'm not sure CSS transitions offer a big performance boost over resetting the CSS transform every animation frame.

@IvanSanchez they do, a massive performance boost. At least they did last time I checked in different browsers with this test case.

Is there any updates on this? I'm still getting this problem on 1.3.4. Does this exist in 1.4.0 still?

Perhaps we don't understand what is required to implement this correctly (e.g. base layer providers may not host appropriately detailed image tiles, or smooth zooming requires heavy processing/resources) but it would be great to understand the reasons why this cannot be implemented in Leaflet;
It would be incredible to see "smooth zoom" in a planned release, even if it is optional or comes with a higher data/resource-processing cost; sometimes heavier is better.

Perhaps we don't understand what is required to implement this correctly

Maintainer's time.

Hey there, any plans to work on this?

Hey there, any plans to work on this?
Also, can anybody share a good smooth scrolling configuration (or link to one)? (one like Google maps is probably what we all want)

Hey! Not sure if this is right or not, thought I'd share (here)

@AndreasAtakan thank you for sharing. This is a nice plugin.

Not sure if this is helpful but this is what I did to smooth out wheel zooming:
const pauseClass = (ele: HTMLElement, className: string, unpause = false) => {
    const from = className.concat(unpause ? "-pause" : "")
    const to = className.concat(unpause ? "" : "-pause")
    Array.from(ele.getElementsByClassName(from)).forEach((ele) => ele.classList.replace(from, to))
}

const stopZoomAnimation = () => {
    let paused = false
    let timeoutId: ReturnType<typeof setTimeout>
    const animationClass = "leaflet-zoom-animated"
    return function (e: WheelEvent) {
        // pause animation if it isn't already paused
        if (!paused) {
            pauseClass(e.target as HTMLElement, animationClass)
            paused = true
            console.log("pausing zoom animations");
        }
        // unpause after 500ms
        clearTimeout(timeoutId);
        timeoutId = setTimeout(() => {
            pauseClass(e.target as HTMLElement, animationClass, true)
            console.log("zoom animations enabled");
            paused = false
        }, 500);
    };
}

and then
map.getContainer().onwheel = stopZoomAnimation()

Basically I just pause the zoom animations by temporarily renaming the leaflet-zoom-animated class while wheel zooming. Kind of a hack but it works pretty well.
You can see the full code change here:
https://github.com/hypergig/LeafyVTT/commit/8039df4d9e0c74d60390ecdc1357906b6e524700


Wallet Disconnect

Allow wallet to disconnect, ensuring rust side code frees up memory in the process

done at https://github.com/wireless90/IotaWallet.NET/commit/2961bfff9560980e8c9af02f84415876f995ec52


[VMware Horizon] Fix release cycles

Many release cycles on the horizon page had the name '8', which resulted in invalid data for the API (some files are named after the revision name).
Release label has been simplified too (following #2068).

links are breaking

@captn3m0, fixed, should be good now.


Unsure RelayX Browser Extension works in Brave

I am having a weird issue when trying to boost on Pow.co using the RelayX Browser Extension.
I am signed into the Relay Browser Extension.
So theoretically, I wanted to test "Shields Up" on Brave to make sure that it is using the Extension and not the regular page login.
But it keeps asking me to log in over and over again when I boost, and it won't do the boost.
Then when I put "Shields Down" (the old way), and log in, it boosts fine.
So I am not able to use the Browser Extension as intended.
Video:
https://user-images.githubusercontent.com/4019818/219051196-54a3ff54-015f-4026-af7d-6e33fd595a24.mp4

Just tested and it works. You should see connected to extension toaster on relayx.com when everything is setup correctly.


Allow show() to trigger while ghost-text is visible

This is a fix for https://github.com/Saghen/blink.cmp/issues/762: an issue where manually triggering show() via a key combo (for those of us that don't use auto_show) isn't possible while ghost text is visible.
Note that the current "fix" is very low effort. I simply looked at the definition for cmp.is_visible() and decided to just copy one half of that function without the "ghost text is visible part". This is on purpose! You can interpret this as a draft PR. I'd be glad to refactor this a bit even if that only means splitting a cmp.is_menu_visible() helper function. But...
...I first want to make sure that this fix is desired. I can verify after running this locally that this fixes my issue. But I'm unsure if this breaks anything else.
I'm happy to polish this further, write a test, you name it! Let me know 😆
Thanks for all of the work on blink.cmp. Thanks to it I now finally have a completion setup that I'm happy with!

Looks good as is, thanks!


Improve and fix VS Code new project instruction across getting started tutorials

In Visual Studio Code  section Run the app the first run says to use the combination of control + F5 but it does not run the app, is does nothing, the right should be only F5 (with debugger) because in the first run the Visual Studio Code will build the project .vscode directory, so the subsequents runs can use Control + F5
--- Added by WadePickett
Expanded this to reviewing and improving all the getting started tutorials for setup and run steps for VS Code.  (Except a separate one was made for Blazor getting started)
Move common instruction where I can into an included.
Address steps from new project, to accept authors, to accept assets to create the .vscode folder with generated launch.json and tasks.json files and verify through Ctrl + F5 step, the first run of the app.

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

ID: a25bf940-8977-fec4-461b-542a9f550fc8
Version Independent ID: 5492c641-af15-64b8-beaa-ddf511be73e6
Content: Tutorial: Get started with Razor Pages in ASP.NET Core
Content Source: aspnetcore/tutorials/razor-pages/razor-pages-start.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @wadepickett
Microsoft Alias: wpickett

Thanks very much @EACUAMBA for taking the time to point out this issue.  We will get it fixed and I will look for other instances of this issue in the tutorials.

No problem, it's a pleasure help to benefit everyone.

There are missing steps in the tutorial to add but a little different than originally thought:

code -r RazorPagesMovie opens the newly created project folder in VS Code.

The Trust the authors? dialog is displayed and trusted.

A prompt to add the required assets to build and debug appears but is not mentioned in the tutorial and should be.

After selectingYes, Visual Studio Code automatically adds the .vscode folder with generated launch.json and tasks.json files.

Selecting Ctrl+F5 results in the VS Code building and running without debugging.

There are at least two common cases and one outlier I can see where that prompt to create build and debug assets would not appear, (provided the prerequisites were followed), that would result in Ctrl+F5 (or F5) not immediately building and running the project:

C# Dev Kit is installed: It can cause a conflict where the prompt of debug assets will not occur. (Just tested that, still occurs).
The user accidentally or intentionally selected Don't Ask Again at the build assets in the past, so no it will no longer prompt.
(Not as common but folks have done this) Following the tutorial, the 2nd command line instruction code -r RazorPagesMovie was accidentally not run with a return.

The fix needed I think is to:

Call out the add assets prompt, which occurs after the trust the authors dialog.
Add something like the following in case the prompt is unable to run: " If this step was missed or unavailable, select View > Command Palette and enter .NET: Generate Assets for Build and Debug.

Ended up reviewing, and updating, testing, 6 tutorials, versions 5 through 8 of each.

Expanded this to reviewing, updating, testing, 6 tutorials, versions 5 through 8 of each, and using an include where we can to keep these consistant.

It will help a lot of people, when that stuck in this situation.


Update 'Yahoo Advertising'

Yahoo advertising is now Verizon advertising
Current detection -
    "Yahoo Advertising": {
      "cats": [
        36
      ],
      "html": [
        "<iframe[^>]+adserver\\.yahoo\\.com",
        "<img[^>]+clicks\\.beap\\.bc\\.yahoo\\.com"
      ],
      "icon": "yahoo.png",
      "js": {
        "adxinserthtml": ""
      },
      "scripts": "adinterax\\.com",
      "website": "http://advertising.yahoo.com"
    },

Keep old detection methods and make following changes -

Update website to https://www.verizonmedia.com/advertising
check for JS object 'dotq'
check for script https://s.yimg.com/wi/ytc.js
Update Yahoo logo to Verizon Media logo
Change the name from 'Yahoo Advertising' to 'Verizon Media'

Hey @rockeynebhwani need your hints
1)We have other technologies that start at Yahoo

Should we reimplement other technologies in Verizon.
2) We also have the already added  Verizon Media.
https://www.wappalyzer.com/technologies/advertising/verizon-media/
3 ) Yahoo Advertising probably need to be deleted since it does not have
https://www.wappalyzer.com/technologies/advertising/yahoo-advertising/
4)  https://s.yimg.com/wi/ytc.js it is necessary that this particular script is completely carried out fo Verizon Media or yimg.com/ ok

@nurbek91 - Since I raised this issue, I realise that Verizon has sold Yahoo and Yahoo again is going to be an independent brand - https://www.capacitymedia.com/articles/3829495/yahoo-due-back-today-after-five-year-gap-as-verizon-sells-for-5bn

Hey @AliasIO , might have to close.


Moon fixed frame transformations

Description
Adds Transforms.computeIcrfToMoonFixedMatrix and related functions to enable the sun position to be computed for the Moon by setting:
Cesium.Transforms.computeIcrfToCentralBodyFixedMatrix = Cesium.Transforms.computeIcrfToMoonFixedMatrix;

Issue number and link
N/A
Testing plan

Ensure this Sandcastle shows moon phases over the course of about a month
Ensure lighting angle was not otherwise changed by default in other examples

Author checklist

[x] I have submitted a Contributor License Agreement
[x] I have added my name to CONTRIBUTORS.md
[x] I have updated CHANGES.md with a short summary of my change
[x] I have added or updated unit tests to ensure consistent code coverage
[x] I have updated the inline documentation, and included code examples where relevant
[x] I have performed a self-review of my code

@jjspace Could you please review?

The more of these that get added the easier it'll be to miss one or not know about one you have to set... My gut just says this should be easier for other devs.

Yes, I agree. I was considering whether to set it based on the default ellipsoid, but I don't want to conflate them and couple them together. I was thinking more along the lines of a helper function, like what you mention for setUpForMoon. Can we circle back to that after this PR?

Can we circle back to that after this PR?

That's fine, can you open an issue so we don't forget if it's worth doing?
Otherwise the code looked good and I'll trust the calculations so I'm fine to merge this.


Trimming and organizing imports

In visual studio you kan type ctrl+k+e to delete unnessesary using statements.
It would be nice if that keyboard combination could be used for what  shift+alt+o  does in vscode.

You can add a custom keybinding for that https://code.visualstudio.com/docs/getstarted/keybindings#_keyboard-shortcuts-editor


Microstuttering when switching the primary playback device (persistent)

OS: WIN10
GPMDP Version: 4.7.1
Issue Descriptions: I am experiencing microstuttering after changing between two DAC devices (was on fiio olympus 2, changed to a focusrite scarlet 2i2). This is the only thing that I changed, and the issue persists between restarts. I have verified that GPMDP is the only app with this issue, and it does seem persist through restarts. I even tried to disable anything else that might be interfere with the playback devices, and manually set the playback device - no dice.
Steps to Reproduce: Switch DAC devices on windows 10.
I will update this post if the issue resolves itself or if I find the cause.

Found the issue, it had to do with the DAC drivers. Oddly enough only this and one completely random Steam game were suffering from this issue, but as soon as it happened on that game I figured it was probably something else. Reinstalling/udpating the drivers fixed it.


Do we have a plan to upgrade sam2 to sam2.1?

Do we have a plan to upgrade sam2 to sam2.1?

Sorry for the delay!  Yeah, we're planning to move to sam2.1, but it's been a bit hectic, so I haven't been able to prioritize it.  If you're keen to help out though, feel free to send a PR!

@zh-nj I've been using a fork of this repo for object tracking through frames. I've updated the network YAML for 2.1 and loaded a new model. It seems to be running fine. @Gy920 do you know if anything else is needed?

@zh-nj @pevers  I recently submitted a commit that updated to SAM2.1. Please try it out.

@zh-nj I've been using a fork of this repo for object tracking through frames. I've updated the network YAML for 2.1 and loaded a new model. It seems to be running fine. @Gy920 do you know if anything else is needed?

@zh-nj do you have your fork shared so others can try it? the master branch here is not really working on my side (at least the demo.py)

ok. I couldn't make this branch work on my machine. Not sure why. the install throws me a cuda related error.


Add user accounts.

 AAU I want to register an account.

 AAU I want to log out of my account.

 AAU I want to log in to my account.

 AAU I want to update my password.

 AAU I want to set a public display name for my account, because I want a more fitting brand for my content than an account index.


How can I issue a request to a LSP endpoint that contains underscore?

The C# LSP server (Omnisharp) contains an endpoint for tests discovery (o#/v2/discover_tests), and it contains an underscore. As far as I can see, all underscores are replaced to slashes, so I can't do something like:
	local err, result = client.request['o#_v2_discover_tests'](lsp_request)

Is there a way to instruct the client to not replace the last underscore?

Nevermind, the endpoint was discovertests without an underscore, I was just confused for a little while 😄


No .exe file found to install

i have downloaded the zip file and extracted it as per the instructions in the readme file, but there is no TaskGroups.exe or any such file found.

Same problem here

Okay it was a noobie problem lol, we have to download the TaskbarGroups.zip from there


4.2: SoftDeletingScope prevents restore on Eloquent model

I have initially posted my issue on Stackoverflow detailing my findings:
http://stackoverflow.com/questions/33171033/laravel-restore-trashed-not-working?noredirect=1#comment54150454_33171033
To summarise, I have a model that uses the SoftDeleteTrait and I am trying to execute the restore method to restore the record in the database.
The restore method executes the following SQL:
update UserClasses set updated_by = ?, deleted_at = ? where UserClasses.deleted_at is null and id = ?
{"bindings":["1",null,"2086"],"time":0.72}
If you look at the WHERE statement, you will see "deleted_at is null" which is being injected by the SoftDeletingScope and preventing the record from being updated.
I can get my code to work by manually commenting out $builder->whereNull within the "apply" method of the SoftDeletingScope class.

4.2 not supported

Given that it looks as though the same issue is also present in the 5.0 and 5.1 branches, I'd have thought you'd be a little less dismissive of a bug report
And I'm sure that a lot of Laravel users aren't aware that 4.2 is no longer supported, then perhaps it's also time to consider making it more obvious to them what versions are and aren't supported.

I was under the impression that with 4.2 documentation still on the Laravel website, it was still supported...


timer: add support for sigev_thread flag in zephyr timer posix api

add support for the sigev_thread flag in zephyr timer posix api, following the behavior described in the linux man page. with this enhancement, a single thread is created to receive all notifications from timers

This is probably the specification we should refer to in this pr
https://pubs.opengroup.org/onlinepubs/9699919799/functions/timer_create.html

@arun-mani-tech - can you please rebase on latest main and resolve conflicts?
The tests that are failing are unrelated and passing on main

@arun-mani-tech - you may have better luck using pool allocated thread stacks rather than heap allocated thread stacks. Heap allocation can require more memory resources than what is available on certain platforms to satisfy alignments, MPU, etc.
See e.g. #60759
CONFIG_POSIX_API=y
CONFIG_THREAD_STACK_INFO=y
CONFIG_DYNAMIC_THREAD=y
CONFIG_DYNAMIC_THREAD_POOL_SIZE=2

@arun-mani-tech - you may have better luck using pool allocated thread stacks rather than heap allocated thread stacks. Heap allocation can require more memory resources than what is available on certain platforms to satisfy alignments, MPU, etc.
See e.g. #60759
CONFIG_POSIX_API=y
CONFIG_THREAD_STACK_INFO=y
CONFIG_DYNAMIC_THREAD=y
CONFIG_DYNAMIC_THREAD_POOL_SIZE=2

One of the test failures leads me to believe that we should be filtering out arch_posix in a testcase.yaml file.
2023-09-08T14:56:32.5934159Z INFO    - -+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
2023-09-08T14:56:32.5934859Z INFO    - The following issues were found (showing the top 10 items):
2023-09-08T14:56:32.5935559Z INFO    - 1) tests/posix/common/portability.posix.common.picolibc on nrf52_bsim failed (Failed)
2023-09-08T14:56:32.5935994Z INFO    -
2023-09-08T14:56:32.5936484Z INFO    - To rerun the tests, call twister using the following commandline:
2023-09-08T14:56:32.5937047Z INFO    - west twister -p <PLATFORM> -s <TEST ID>, for example:
2023-09-08T14:56:32.5937443Z INFO    -
2023-09-08T14:56:32.5937929Z INFO    - west twister -p nrf52_bsim -s tests/posix/common/portability.posix.common.picolibc
2023-09-08T14:56:32.5938363Z INFO    - or with west:

it seems to fail on mps2_an385
In file included from /__w/zephyr/zephyr/lib/posix/timer.c:9:
2023-09-08T14:33:56.1660296Z /__w/zephyr/zephyr/include/zephyr/posix/pthread.h:445:18: error: unknown type name 'pthread_once_t'; did you mean 'pthread_cond_t'?
2023-09-08T14:33:56.1660840Z   445 | int pthread_once(pthread_once_t *once, void (*initFunc)(void));
2023-09-08T14:33:56.1661195Z       |                  ^~~~~~~~~~~~~~

it seems to fail on mps2_an385
In file included from /__w/zephyr/zephyr/lib/posix/timer.c:9:
2023-09-08T14:33:56.1660296Z /__w/zephyr/zephyr/include/zephyr/posix/pthread.h:445:18: error: unknown type name 'pthread_once_t'; did you mean 'pthread_cond_t'?
2023-09-08T14:33:56.1660840Z   445 | int pthread_once(pthread_once_t *once, void (*initFunc)(void));
2023-09-08T14:33:56.1661195Z       |                  ^~~~~~~~~~~~~~

@cfriedt Changes have been made please revisit

@ycsin please review

@povergoing @jgl-meta Please review this

Not a fan of single line if-conditionals without curly braces, otherwise LGTM


enh: open PDF citations on the associated page

Description

If you use PDFs, the file is now opened on the right page of the text snippet citation on supported browsers.
Styling changes to clarify to the user that the link is clickable.
Additional "tippyOptions" property to add specific properties to the Tooltip (needed to remove the off-mount delay for scrolling containers).
"page" is now a i18n key
German locales

Changelog Entry
Added

📄 PDF Citation Support & UI Enhancements: PDFs now open directly at the referenced page in supported browsers, improving navigation. Additionally, clickable links in the citation modal have updated styling to make them more visually clear and user-friendly.

Thanks!


Timing

For a listed expression e.g.
servicedesk.brighton.ac.uk/*
– when, exactly, will deletion occur?
The list aside: if a period e.g. 60 days is set, when will deletion occur?
At or shortly after application start time?
Are deletions periodic – every minute or so, maybe – whilst the application runs?
At application quit time?

Found:

History is cleared on startup and for every hour the browser is open. Does a manual cleanup when turning on for the first time.


Correct the sql.jinja2 load path

Fix issue where template would not get loaded unless
create-sql-functions was run from the cwd
@zerebubuth could you review please?

:+1:


Suboptimal decompiler output with array of structs

Test case
In the test case, the function _main first initializes r12 to point to a global array of structs:
100003f51  4c8d25b8400000     lea     r12, [rel _a_list]

Then, in a loop, it gets two fields from the nth entry and ORs them together:
100003f5f  410fb704dc         movzx   eax, word [r12+rbx*8]
100003f64  450fb76cdc02       movzx   r13d, word [r12+rbx*8+0x2]
100003f6a  4909c5             or      r13, rax

Binary Ninja (2.2.2551-dev) decompiles this as:
100003f76      rax_2 = _asdf(0, zx.q(*((rbx << 3) + 0x100008012)) | zx.q(*(_a_list + (rbx << 3))), &var_38)

where 0x100008012 points to _a_list + 2.
This is suboptimal in two ways:

Displaying the raw number 0x100008012 instead of _a_list + 2 is unhelpful.
-> If I manually create a struct with appropriate fields and mark _a_list as a variable of that struct type, 0x100008012 changes to _a_list.field_2, which is better, but isn't it missing an &?  Regardless, BN should show the symbol without needing this.

Ideally the decompiler would merge the duplicate instances of r12+rbx*8 and end up with something like:
entry = _a_list + (rbx << 3)
… *entry | *(entry + 2) …

I've heard that this is  being worked on, will we be seeing it in dev builds anytime soon?

Very soon, the feature is being tested currently

Awesome, thanks for the update @plafosse , I definitely won't miss https://i.gyazo.com/7a9bb5aca7ddcd47d8efeede3205ead5.png

Array resolution support has been added in build 2760. HLIL output will now include array references from array and pointer types. Accesses to global structure members have been improved as well.
After defining a structure and applying the structure to the global array in the given example, the output is now like this:
100003f76      rax_2 = _asdf(0, zx.q((*_a_list)[rbx].field_2) | zx.q((*_a_list)[rbx].field_0), &var_38)

The need to dereference global addresses may be improved at a later date.


remove npm release check for testing provider mock

could do with @forgetso taking a look at this. Not sure if there was a reason why the provider mock was waiting on npm releases


Fix mapping bugs (mainly related to slicing)

PanelMembers maps to related was setting related cardinality to 0..1 (since that is PanelMembers cardinality).  This isn't what we want.
Mappings to component were slicing on coding.code, which isn't a valid relative path from component.  It should be code.coding.code.
Also adds a .gitignore to ignore any local VSCode workspace settings.

@cmoesel -- When we want to set cardinality to  0..0 related, would removing PanelMembers maps to related make that impossible?

@markkramerus -- that's a good point.  No, it would not.  But the problem is, we can't have it both ways -- either it's mapped and controls the cardinality (in which case 0..1 is a real problem) or it's not mapped (and as you stated, now 0..0 doesn't come through).
You could always make related 0..0 in the mapping itself, as the situation warrants -- although I admit, that's not idea.  The other thing we could do is revisit why we're doing the PanelMembers.Observation pattern anyway, as opposed to a non-nested variant (like we do w/ EvaluationComponent).


Request for svg sprite which can be used with <use xlink:href

Hi devs, is it possible to create an svg file as part of release in format -
<svg xmlns="http://www.w3.org/2000/svg">
  <defs>
    <symbol id="activity" viewBox="0 0 24 24">
      <polyline points="22 12 18 12 15 21 9 3 6 12 2 12" />
    </symbol>
   ...
  </defs>
</svg>

This will allow adding an icon similar to this feather request - https://github.com/feathericons/feather/pull/319

@boscharun Welcome to the Lucide Project!
Yess we have an SVG sprite available, but it is included in the Lucide Static package.
https://www.jsdelivr.com/package/npm/lucide-static

Thanks @ericfennis for the quick response, this is excellent. Would you please consider adding this in readme. The more obvious svg sprite inside the release zip files is different.

Sorry, @boscharun , do svg sprite need load an large svg file? or need a lot of request? I think that use lucide with Vue/React/... is a better idea...
Just question >_<

@PeterlitsZo , I have a python/django based project and don't use Vue/React. The full sprite is ~100Kb but is only 22Kb with zipped transfer which is very acceptable and much faster than individual svgs or our usecase. Thanks.

Added it to the Readme file!


Use s:path with s:rtp always

Having a trailing slash with &rtp entries is problematic when removing
them later: if loading on demand is used, s:reorg_rtp might fail to
remove the previous runtime paths.
Test case has been using maktaba (which triggers unsetting s:middle in
s:reorg_rtp), but because of the trailing slashes being used, the
previous rtp entries were not removed.

Can you provide the steps to reproduce the case?

I've noticed it when using maktaba, which triggers changing the &rtp:
    Plug 'google/vim-maktaba'
    Plug 'google/vim-coverage'

    Plug 'Vimjas/vim-python-pep8-indent', {'for': ['python']}

When opening some Python file the &rtp should get doubled, having entries with and without a trailing slash.
It looked a bit like at some point Vim/Neovim would normalize the trailing slashes away.

Rebased.
Just noticed this again.
Without this patch/PR I get 188 entries in my &rtp, using this fix it's only 76.  A set of entries are contained therein 3 times (first with trailing slash, then without, and then with a trailing slash again).
Both start with /home/user/.config/nvim, and then the version without this patch has entries ending with a slash up to /home/user/.dotfiles/vim/plugged/vim-coverage, and then again a list without slashes after /home/user/.dotfiles/vim/plugged/vim-maktaba (making those appear 3 times).

@blueyed Can you verify if the problem exists? I'm interested in a smaller &rtp. Can you add a test as well?

@janlazo
Have you checked your rtp with and without this patch?
(rebased for now)

Works for me.


Move translations and the renderer engine out of the renderer models

https://jira.dev.bbc.co.uk/browse/CONNPOL-3647

This is to separate the translations and renderer engines out of the render model hierarchy in order to:

Allow alternative implementations of the translations
Allow translations to work in the JSON renderer
Allow us to render snippets of HTML from the JSON renderer
Separate concerns

Nice. Should have been done this way from the start.

Nice! 👍

Looking good :+1:

@dazoakley, can't @kodikos test this locally, by pointing the Gemfile at this branch, or is that not an option?

@samfrench she has been ;)

Looks good to me 👍

👍

Okay, going to stop messing around with this PR now!! Please final review and let's get it merged, thanks :) In particular, note the issue with self.template_path in the description and let me know if you have any problems in your codebase (I already have a fix for election-2016-renderer to go in)

👍


Fixes Sprite's texture listener not getting removed

Fixes #4930
Adds a unit-test that fails on dev

Maybe call it only if texture wasn't loaded, we dont want to add extra iterations there.
if (!value.baseTexture.hasLoaded) ...

Its up to you.

Seems kinda silly to optimize with that if-check in the destroy function. Not a performance critical place in the code.

I just upgraded to a version of Pixi that includes this change, and my code ended up breaking because I was relying on being able to call destroy multiple times. Now, calling it more than once will result in an undefined error for this._texture.off(...).
I don't think that what I'm doing should be considered a Pixi bug, but I just wanted to add my two cents here. I'll likely end up wrapping my destroy calls with a check to see if texture exists.


docs: add local environment setup instructions

fixing #639

@yknl could you help me identify the changes needed after monorepo work is done?

[ ] use StacksMocknet
[ ] change dependencies

At a high level, we should use the testnet demo app for a "template" of clarity web app development. This way, you don't have to fiddle with React code. https://github.com/blockstack/ux/tree/master/packages/test-app

In my practise it is much easier to restart mocknet and redeploy the contract than finding the code for the contract addresses and replace them. In particular if you have more than one.
The yarn lerna..  command for the authenticator should be simplified to something like yarn start-authenticator
Could the explorer be part of stacks api docker image? Or have a mocknet docker image?

@dantrevino and @psq - any input for this? what is your local contract setup like?

@agraebe ty for reaching out.  Instructions work and seem clear enough, however ... I feel like this is useful for either a) those getting started from scratch, and/or b) bigger teams that have a real distinction between their QA and dev teams.  Either that or I personally lack a good process, which is highly likely.
One nitpick ... align on a single node package manager. Instructions are mixed between npm and yarn.
None of this precludes the PRs usefulness.
For me personally, I have the following issues:

Setting up my own authenticator means I have to track a completely different set of different user accounts in a different set of browsers.  Probably good practice, but not very likely for lone developers.  I.e. I see a lot of value in testing with my real-world (test) accounts that I use across multiple apps, including apps that are not mine.  This could become even more required if Collections ever becomes reality.
Instructions for doing an npm init clarity-starter within an existing project would likely be easier for devs who want to bring Clarity to their existing applications.  It's not hard to do manually but would lower friction for their entry to Stacks, imho.

@agraebe my setup looks different, and probably more involved, but allows for running up to date, or even patched components.

postgres runs natively, and I manually clear/recreate database to reset
use the docker image to run bitcoind (the only thing that really never needs updating
run stacks-node (in mining mocknet mode) from git with suitable branch (docker images are not updated fast enough for me)
run stacks-node-api from git from suitable branch if needed
(so that's 3 terminal windows, with full access to the logs, and easy to restart)
I set all the ports explicitly, either via environment variable or configuration rather than rely on having the startup script to pick a new one each time (try that a 100 times and let me know how if felt 😂)
I some times have more than one environment running, so again, all ports need to be set explicitly, no guessing.

I usually don't run the explorer, while a great tools when things work, it does not expose enough to diagnose problems when there are errors, so I rely on logs.
To deploy processes, I rely on my own javascript code, when working on swapr+flexr, I need to redeploy 10 contracts, in the right order (https://github.com/psq/flexr/blob/master/deploy-contracts.ts#L86-L95)
I would rather see explanations on how you can change the auth url and api url in your app code rather than rely on pseudo magic code that makes the choice for you.  In practice, it rarely does pick the right ones and explaining how to use explicit values would save people some time when it makes the wrong choice.  Plus if you have to run on a server with your custom node (auth or api), learning how to set the url explicitly is something you have to do anyway.
And I sometimes have to run with my own authenticator, so that's one more dimension to add.
Also, I would recommend people start working on their clarity contracts using the js sdk, not even mocknet, it is a lot easier to get something up and running this way (example here https://github.com/psq/flexr/blob/master/test/unit/flexr.ts, @friedger probably has simpler examples)

@agraebe , like psq I am not using testnet explorer locally.
For mocknet,

I am running the docket image of stacks-blockchain-api
I run a script to deploy my contracts and restart the docker image whenever I want to retest
I use one or more of the four test accounts that have initial balances (with a flag in the script to switch between test accounts and my "real" testnet accounts)

@psq With the recent update of the authenticator, a local instance is not necessary anymore, see #834

@psq With the recent update of the authenticator, a local instance is not necessary anymore, see #834

yes, I noticed, but that was necessary earlier

Wow, thanks everyone for the input! I'm updating the PR and wanted to respond to some comments

This way, you don't have to fiddle with React code. https://github.com/blockstack/ux/tree/master/packages/test-app

@hstove I looked at it and figured it would be more difficult for folks (can't resolve multiple packages, need to clone the whole ux repo, not documented as well as the other app). Sticking to an app that has a tutorial is probably best for onboarding purposes

In my practise it is much easier to restart mocknet and redeploy the contract than finding the code for the contract addresses and replace them. In particular if you have more than one.

@friedger Good point! I added instructions for that. Instead of the script, one can restart the mocknet using docker

The yarn lerna.. command for the authenticator should be simplified to something like yarn start-authenticator

I removed the local authenticator setup now that the network can be set

Could the explorer be part of stacks api docker image? Or have a mocknet docker image?

Yes! Thanks to @wileyj, the docker compose project includes the node+API+explorer

One nitpick ... align on a single node package manager. Instructions are mixed between npm and yarn.

Thanks @dantrevino, I fixed that!

Setting up my own authenticator means I have to track a completely different set of different user accounts in a different set of browsers

This is not required anymore, so we can avoid that complexity

Instructions for doing an npm init clarity-starter within an existing project would likely be easier for devs who want to bring Clarity to their existing applications.

That's true! It is possible to just run the command inside a project folder, too. However, in my instructions I can basically say that one should complete the counter tutorial before doing this one. So, I can skip over the contract creation and focus on the goal of this tutorial: setup for development.

I usually don't run the explorer, while a great tools when things work, it does not expose enough to diagnose problems when there are errors, so I rely on logs.

@psq and @friedger Yeah, I recognized that and removed the Explorer step/requirement. It is served through the docker compose, but I don't think I explicitly need to tell devs to set it up... also, thanks for sharing your local setup details! I believe it's probably too much for the regular smart contract dev, but still good for my understanding!

I would rather see explanations on how you can change the auth url and api url in your app code rather than rely on pseudo magic code that makes the choice for you

I updated the instructions to cover that

Also, I would recommend people start working on their clarity contracts using the js sdk, not even mocknet, it is a lot easier to get something up and running this way

Agreed! This tutorial requires the counter tutorial to be completed, which in fact uses the clarity project generator that comes with the js sdk :) So devs can work in TDD fashion before ever deploying to mocknet. I added that to the instructions too

I use one or more of the four test accounts that have initial balances (with a flag in the script to switch between test accounts and my "real" testnet accounts)

@friedger thanks for sharing that! The new docker compose uses test accounts and I added instructions for deployment using one of them as well (with a note so folks know about it). The script seems nice! I'm not sure it's the right place to mention it but let me think about it

Current blockers:

[ ] merging https://github.com/blockstack/stacks-local-dev/pull/2

@yknl this can't be merged until https://github.com/blockstack/stacks.js/issues/878 is resolved. could you take a look please?

this is quite outdated by now and there seems to be no strong demand for this type of tutorial right now. closing for now.


Cmdline error when immediately exiting after opening

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
i map qq -> :q\<CR> to quit neovim.
when i qq, an error from blink.cmp always bothers me like this:

Relevant configuration
return {
  "saghen/blink.cmp",
  enabled = vim.g.cmp == "blink",
  lazy = false,
  dependencies = { "rafamadriz/friendly-snippets" },
  version = "*",

  opts = {
    keymap = {
      ["<M-j>"] = { "select_next", "fallback" },
      ["<M-k>"] = { "select_prev", "fallback" },
      ["<Tab>"] = { "select_and_accept", "fallback" },
      ["<C-c>"] = { "hide", "fallback" },
      ["<M-Space>"] = { "show", "show_documentation", "hide_documentation" },
      ["<C-u>"] = { "scroll_documentation_up", "fallback" },
      ["<C-d>"] = { "scroll_documentation_down", "fallback" },
    },
    appearance = {
      use_nvim_cmp_as_default = true,
      nerd_font_variant = "mono",
    },
    sources = {
      default = { "lsp", "path", "snippets", "buffer" },
    },
    signature = { enabled = true },
  },
  opts_extend = { "sources.default" },
}

neovim version
NVIM v0.10.3 Build type: Release LuaJIT 2.1.1731601260 Run "nvim -V1 -v" for more info
blink.cmp version: branch, tag, or commit
version 0.8.1 , tag: v0.8.1, branch: main, commit: a9a0f96

Do you mind sharing your full code for the mapping? I'm not able to reproduce this

just this:
vim.keymap.set("n", "qq", ":q<CR>", { silent = true })

but i zz to quit neovim, no errror, everything well

I'm not able to reproduce, do you mind sharing a minimal repro? https://lazy.folke.io/developers#reprolua

It works well now after upgrading plugin.
Thanks！


Invalid value for attribute in AMP tag 'controls' in tag 'amp-video'

@orballo is this solved?
If it's not, could you please recreate the issue in https://github.com/frontity/saturn-theme/issues?

I don't know, but if nobody complained I'd dare to say is fixed. Do you know any site with native videos to check it?

Ok, let's leave it for now then.


Is it possible to explicitly set black version in pre-commit?

Quoting https://github.com/pydata/xarray/pull/6290#issuecomment-1046291891

FYI something that's a bit surprising is that blackdoc passes in pre-commit locally but not in pre-commit.ci. My guess is that it doesn't specify the version of black it uses and so on my local machine it has an older version of black. I'm not sure whether that's possible with additional_dependencies...

Again — thank you for the excellent tool!

this should definitely be possible, but I'm not quite sure where to put it. I would probably suggest to do that in the project's .pre-commit-config.yaml:
repos:
- repo: https://github.com/psf/black
  rev: 22.1.0
  hooks:
  - id: black

- repo: https://github.com/keewis/blackdoc
  rev: v0.3.4
  hooks:
  - id: blackdoc
    additional_dependencies: ["black==22.1.0"]
  - id: sync-black-version

where the sync-black-version hook will keep it in sync with the hook version of black. That way, I won't have to release a new version every time black does (although that might be better given that blackdoc is using some of the internal API of black).

I agree it shouldn't be in the setup.cfg requirements of blackdoc (that would pin the version of black that consumers use). What do you think about putting the additional_dependencies in the blackdoc pre-commit configs? Although it would mean the black version would lag a bit (until you did the next release), it would mean that people would get a decent option by default. Having a slightly old version of black for blackdoc is fine, but having inconsistent versions between people running it isn't good.
(TBC, it's completely your call and I'm asking because I use blackdoc in lots of repos and we hopefully know each other well enough for me to ask; it's fine to say "OK but no"! :) )

My main concern was that black would introduce changes to the code style, but I guess now that black is stable and the most important changes happen once per year this is not so important anymore. It would also avoid having the hook break because black (or tomli) changed, although that might also be avoided by increasing the test coverage...

notes to myself:
it looks like additional_dependencies (or dependencies of hooks in general) are a issue with pre-commit: they are only updated whenever a hook is installed or updated. I can probably write a github action (or a local hook that pre-commit.ci can run) to update black whenever that is necessary, and release shortly afterwards. That might require CalVer, though...


Saving edited PDFs on Android not seamless

Since the PDFs are opened in editors through the share menu, the file's original location isn't exposed and so the editor can't save changes back into the Obsidian Vault without extra steps. WPS works OK because it'll ask you to choose your save location, but some editors don't do this (e.g. Neoreader for Boox tablets). Is there a different way to get PDFs opened up in external editors?

Yes, this is an issue that I am aware. Because of how most editors in Android are coded, the share menu creates a copy of the file. In the README, you have a list of editors that don't do that and do work.
For me, Xodo is the best by far, you can install it in the App Store. You can use it without a subscription but some minor features are locked. If you are comfortable sideloading an APK, you can search online for an outdated version that does 99% of the things it does now and has no paywall.


update to 0.28.0

https://github.com/describo/crate-builder-component/releases/tag/v0.28.0

Done.


EISDIR: illegal operation on a [particular] directory

While attempting to open a folder, I got the error below. After renaming, it opened just fine. Making a new, empty folder with the same name yielded the same error.
I'm on OS X 10.10.4 using 0.199.0. The path is still read- and usable to other processes, and just before manually installing 0.199.0, I had the folder open and working fine with 0.194.0.
Rebooting didn't help, so I'll look for references to the path in Atom's config files and on my system.
index.js:62 Error: EISDIR: illegal operation on a directory, read
    at Error (native)
    at Object.fs.readSync (fs.js:596:19)
    at Object.fs.readFileSync (fs.js:428:28)
    at Object.fs.readFileSync (ATOM_SHELL_ASAR.js:369:29)
    at File.module.exports.File.readSync (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/pathwatcher/lib/file.js:248:36)
    at TextBuffer.module.exports.TextBuffer.updateCachedDiskContentsSync (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:940:85)
    at TextBuffer.module.exports.TextBuffer.loadSync (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:969:12)
    at Project.module.exports.Project.buildBufferSync (/Applications/Atom.app/Contents/Resources/app.asar/src/project.js:431:14)
    at Project.module.exports.Project.bufferForPathSync (/Applications/Atom.app/Contents/Resources/app.asar/src/project.js:407:61)
    at TokenizedBuffer.module.exports.TokenizedBuffer.deserializeParams (/Applications/Atom.app/Contents/Resources/app.asar/src/tokenized-buffer.js:82:36)index.js:62 handleSetupError

Ok, deleting editor* files referencing the path in ~/.atom/storage/ fixed it.

FYI, I had the exact same problem and @jiku's solution worked for me as well. I'm on OS X 10.10.3 and I could reproduce the issue in at least 0.198 and 0.199 (it used to work, but can't say which version).


Spike: Sensitive Data Exposure via Directory Listing Enabled

Context
As apart of the scope of this ticket research how to fix this issue and remediate it.
This issue needs to be remediated before 2023-08-31.
Sensitive Data Exposure via Directory Listing Enabled
Overview of the Vulnerability
Sensitive data can be exposed by web servers which list the contents of directories that do not have an index page, which increases the exposure of files that are not intended to be accessed. Within this application, sensitive data has been exposed through a directory listing being enabled. This allows an attacker to quickly identify resources of a specific path, or gain access to data stored in the directory by browsing to the directory listing.
Business Impact
Data exposure could result in reputational damage for the business through the impact to customers’ trust. The severity of the impact to the business is dependent on the sensitivity of the data being stored in the directory listing.
Steps to Reproduce

Using a browser, navigate to the following URL to find that directory listing is enabled:
https://campd.epa.gov/env-config.js
Proof of Concept (PoC)
The screenshot below demonstrates the sensitive data found:
{{screenshot}}

vrtversion 1.10.1
Server Security Misconfiguration > Directory Listing Enabled > Sensitive Data Exposure
Target
*.epa.gov— Website Testing
Bug URL
https://campd.epa.gov/env-config.js
Remediation
Restrict directory listings being displayed from the server configuration.
Example for Apache:

Edit the server configuration file or edit/create directory .htaccess
Add the following line: Options -Indexes
If it is the last line, make sure you have a new line after it.
'

References
•	http://projects.webappsec.org/w/page/13246922/Directory Indexing
•	https://www.owasp.org/index.php/Top_10-2017_A6-Security_Misconfiguration
•	http://projects.webappsec.org/w/page/13246959/Server Misconfiguration
•	https://cwe.mitre.org/data/definitions/548
•	https://cwe.mitre.org/data/definitions/16

Based on 7/18 discussion, this is a non-issue. Will wait on direction from Mike H. when to close this issue out.

Ticket no longer needed. This was a non issue.


Documentation for columns.data

Can you please add a note to [http://datatables.net/reference/option/columns.data#function] similar to [http://datatables.net/reference/option/columns.render#function].

Note that this function might be called multiple times, as DataTables will call it for the different data types that it needs - sorting, filtering and display.

Committed. Thanks!


T641: Any nonempty compact $T_0$ space has a closed point

This replaces the old T641, which asserts that $T_0$ spaces with a focal point has a closed point. Note that having a focal point implies compactness, and this is already known to pi-Base: https://topology.pi-base.org/spaces?q=Has+a+point+with+a+unique+neighborhood+%2B+not+Compact

In fact, having a nonempty closed compact subset is enough. I'll see what the weakest condition in pi-Base implying this is.

Very nice generalization.
S49: As a result, we can remove the trait for P24 (locally relatively compact).


The great renaming

Naming is a notoriously hard problem, but we can probably do better than the current status quo.

Should we include View in view names? Or should we try to have views::Edit, views::Text, layout::Linear?
views::SizedView is confusing; should probably be called SizeTrackedView, SizeTracker, or LastSizeView.
views::ViewBox should probably be views::BoxedView
views::BoxView should probably be views::SizedView

Some of the expansion traits would need to adapt as well.
It's not a lot of types to renames, but they are very commonly used.

Some modules refactor may come at the same time.

Most common types should at least be re-exported in the top-level namespace (looking at you, vec::Vec2).
The view module is a bit of an arbitrary mashup of various components, we may want to split it out.
The utils modules may need to be split into other top-level modules - maybe layout::{lines, span} and markup.
Some generic parts could be moved to separate crates?

XY,Rect, Vec2? Most of utils?

The most controvertial change here is the removal of the View suffix from view names. To better understand the implications, here is the exact renaming it would include:

BoxView -> views::Sized
DebugView -> views::Debug
EditView -> views::Edit
EnableableView -> views::Enableable
HideableView -> views::Hideable
IdView -> views::Id
LinearLayout -> layouts::Linear
ListView -> views::List
OnEventView -> views::OnEvent
PaddedView -> views::Padded
ScrollView -> views::Scroll
SelectView -> views::Select
ShadowView -> views::Shadow
SizedView -> views::LastSize
SliderView -> views::Slider
StackView -> views::Stack
TextView -> views::Text
TrackedView -> views::Tracked
ViewBox -> views::Boxed

Most of them look good, though views::Debug, views::Id, views::LastSize still don't seem quite right. Also views::Sized may seem weird. Aaah, naming...

Haven't stripped the View suffix yet, but the first phase of the renaming has been done:
BoxView -> ResizedView
SizedView -> LastSizeView
ViewBox -> BoxedView
IdView -> NamedView
Selector::Id -> Selector::Name
Identifiable -> Nameable
*_id -> *_name
Old names are still exported, but deprecated.


Cmd-d of single character followed by typing same character as has no effect

VSCode Version: 1.7.2

This is a odd one. Steps to reproduce:

Create document x x.
Hit cmd-D twice to select both.
Type x.
Nothing will happen.
Type y.
All instances will be replaced by y.

What I expected would happen: All instances should be replaced with x and the cursor moved after it, so that when you then type y, you end up with xy xy.
If it sounds minor, this happens to me all the time. For example, when replacing one-letter loop variables.

wow, nice find! 👍

Just in case the above came across as a rare edge case: Let's say you have the variable i. You want to replace it with iterator. To accomplish this after hitting cmd-D, you have to type iiterator — an extra "i" as the first on has no effect. Pretty annoying.

Additional steps from #25101:

New Untitled File
Press: a
Select: a
Press: a
Expected: Selection disappears and cursor is to the right of a.
Actual: Selection stays.

This only happens when overwriting with the same character.


Properly detect if the environment is Node

This fixes how we currently determine if the running environment is Node, by also ensuring that window is not defined. Simply checking for process is insufficient in browser environments such as those provided by NW.js or Electron, which also define a window.process.
This issue is related to this PR in Ember.

@stefanpenner - We need to update the version of RSVP that Ember vendors to fix this for Ember apps.  Do you have any objections to tagging a 3.0.18 and updating Ember?

Do you have any objections to tagging a 3.0.18 and updating Ember?

nope none, ill doit in the next 30 min

released, now running the ember tests.

I have an issue in NW.js context since this PR.
In nw context "window" is an object and his type is not undefined.
So useMutationObserver() is executed instead of useNextTick()
I get Uncaught ReferenceError: document is not defined in node_modules/rsvp/dist/rsvp.js

@brzpegasus can you take a look?

Argh, I have missed this particular use case. I was using rsvp in NW.js, but in the browser, via Ember. So I was getting both window and process in the same scope, and there were issues with the app running due to Ember thinking I was in Node and not in the browser.
You're running rsvp in NW.js as well, but in Node's context (via node-main or as a node module in your app). So you're also getting both window and process because NW.js injects window onto Node's global object. That sounds about right, @vdumontier?
If so, we should have been checking for typeof document === 'undefined'.

Something like that.
Yes typeof document === 'undefined' is true
and we also have an object in process.versions with that
{
ares "1.10.0-DEV",
chromium: "41.0.2272.76",
http_parser: "2.3.0",
modules: "43",
node: "1.2.0",
node-webkit: "0.12.0",
nw-commit-id: "f6c7f79-10b9c03-be948af-459755a-2bdc251-c114bb7"
openssl: "1.0.1k"
uv: "1.4.0"
v8: "3.32.7"
}
The node-webkit key can be usefull
Thanks

Well, what we need is to be able to distinguish between Node and the browser. process.versions['node-webkit'] wouldn't be a differentiator. It's also too specific to the technology (think of how this would apply to Electron for example).
@stefanpenner Do you see any impact on other areas of the lib if we were to change the check from window to document?

@stefanpenner Do you see any impact on other areas of the lib if we were to change the check from window to document?

In RSVP we typically check self, as self is available in WebWorkers and regular web frames.

Gotcha. self should work too. Will PR something later this evening to get this fixed.

Gotcha. self should work too. Will PR something later this evening to get this fixed.

:+1:

@brzpegasus @stefanpenner thanks !


Audio for Samsung Chromebook Plus (v2)

Samsung Chromebook Plus (v2)
NAUTILUS
XE521QAB-K01US
Kaby Lake
thank you!

@namportuhkee can you do upload an alsa-info please.
sudo alsa-info

Closing because no answer, join our discord to make audio working (kabylake support is done).
https://discord.gg/WatsaYhXCa


Add lattice index to Element class.

Coverage increased (+0.06%) to 93.315% when pulling 31a68fd4bca7eeab877e2a683df4bc37ae17f142 on element-index into ed00e52e419618c3a717616368bd2919e6b7d0d4 on master.


Internal mode set by executeScript / closing ressources by executeScript

Hi,
first of all, thanks for this library!
It greatly facilitates access to PowerShell.
There seems to be an issue when executeCommand is called after executeScript during the same session.
In executeScript an internal flag 'scriptMode' is set to true but never reset to false on method exit.
So following plain calls to executeCommand use PowerShellCommandProcessor as if it runs in scriptMode.
These calls usually fail.
Another point is, that in executeScript(String scriptPath, String params), a BufferedReader is opened but seems to be never closed.
Could you please check both issues.
Thanks
Juergen

Hello Juergen,
Thanks for reporting it. I will check & fix it when possible.
Best regards

Both bugs are fixed in the new version: https://github.com/profesorfalken/jPowerShell/releases/tag/v3.1.1
So I close this issue.
Thank you!


Data Loading: Handle errors in data response from server

Let's ignore the response from server if we can't parse it for some reasons or we detect data corruption

Currently its implemented,
each post has "must have" properties, if it fails getting them from API response, Post entity is not created, thus it gets ignored.
No error is shown for that case.
Note: cache still caches invalid response, but it does not affect the feed.

I'd like a clarification on this issue. Are we talking about a situation where we get a success HTTP response from the server, but the response body is not parseable into the appropriate response structure? Is this for GET actions, or is this also for POST actions?

@sharadagarwal
"Are we talking about a situation where we get a success HTTP response from the server, but the response body is not parseable into the appropriate response structure?"
Thats right. I meant this case.

If you see this behavior often, then there is a bug somewhere, either on the server or on the client. Otherwise, perhaps this could happen when the network connection fails, but even that seems unlikely because of checksums in the lower layers of the network stack. Can you tell me more about when you saw these errors?
What is the behavior you have implemented for GETs, POSTs, PUTs, and DELETEs? Is the operation retried?


CB-10284 Update Azure coordinates got from the az client

See detailed description in the commit message.

I think this should be 2.35 or master


Release 4.4

Please make sure you've read the pull request section of our code contribution guidelines.
When filling in a section please remove the help text and the above text.
Summary
PLT-8118 - Account settings items misaligned
PLT-8106 - Fixing edited indicator on compact view
Ticket Link
https://mattermost.atlassian.net/browse/PLT-8118
https://mattermost.atlassian.net/browse/PLT-8106
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Ran make check-style to check for style errors (required for all pull requests)
[ ] Added or updated unit tests (required for all new features)
[ ] Has server changes (please link)
[ ] Has redux changes (please link)
[x] Has UI changes
[ ] Includes text changes and localization file (.../i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

@asaadmahmood Fixing the edited indicator looks like a lot of changes. Is that risky to take this late to the release? https://github.com/mattermost/mattermost-webapp/pull/283/commits/a4fd93daf6cbbab6a66c59ffcc6e6af430cfcfd3

@jasonblais Have a look at it now.


Delete External Database do not delete physical file

If i create external database with command manager.createAtPath(dbName, myPath) and then delete it with manager.delete(dbName) i found that the physical file is not deleted. But if i create with manager.create(dbName) and then manager.delete(dbName) then the physical file is gone. Did i miss some setup? I'm still using geopackage android 3.2.0. Thank you

GeoPackages through external links are not managed in the app's memory space.  Upon deletion, a GeoPackage internal to the app's memory space is deleted.  However for an externally linked GeoPackage where the file exists outside of the app's memory space, only the link to the GeoPackage is deleted.  The same thing would happen if you deleted the app.
When the file path is provided, the GeoPackage is treated as external.
External Creation Methods:

createAtPath(String database, File path)
createFile(File file)
createFile(String database, File file)
importGeoPackageAsExternalLink(File path, String database)
importGeoPackageAsExternalLink(File path, String database, boolean override)
importGeoPackageAsExternalLink(String path, String database)
importGeoPackageAsExternalLink(String path, String database, boolean override)

There are a number of external related methods to determine if a GeoPackager is external.  For those, you could try delete the file on your own assuming your app has write access.

isExternal(String database)
existsAtExternalFile(File file)
existsAtExternalPath(String path)
getPath(String database)
getFile(String database)
getDatabaseAtExternalFile(File file)
getDatabaseAtExternalPath(String path)

Also, if you would like to import / copy the GeoPackage internally into the app space memory:

importGeoPackage(File file, boolean override)
importGeoPackage(String database, InputStream stream)
importGeoPackage(String database, InputStream stream, GeoPackageProgress progress)
importGeoPackage(String database, InputStream stream, boolean override)
importGeoPackage(String database, InputStream stream, boolean override, GeoPackageProgress progress)
importGeoPackage(String name, File file)
importGeoPackage(String name, File file, boolean override)
importGeoPackage(String name, URL url)
importGeoPackage(String name, URL url, GeoPackageProgress progress)
importGeoPackage(String name, URL url, boolean override)
importGeoPackage(String name, URL url, boolean override, GeoPackageProgress progress)


#1 - Dependencies bump

Dependencies bump

@olegmoz/z this pull request is too small, just 7 lines changed (less than 10), there will be no formal code review, see §53 and §28; in the future, try to make sure your pull requests are not too small; @g4s8/z please review this and merge or reject

Job gh:artipie/vertx-server#28 is not assigned, can't get performer


Conflict w/ ember-cli-sass

Hi,
I seem to have a conflict when using both ember-cli-less and ember-cli-sass. I have an app.scss file located under app/styles/app.css, but whenever I run ember server, I receive the following error:
Error: File not found: app/styles/app.scss
in any of the following include paths:
  /Users/jamesdixon/Projects/scout/platform/business/web/tmp/broccoli_merge_trees-output_path-veln3PGt.tmp
    at Object.findFileSync (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/node_modules/include-path-searcher/index.js:12:9)
    at SassCompiler.<anonymous> (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/index.js:51:33)
    at lib$rsvp$$internal$$initializePromise (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/node_modules/rsvp/dist/rsvp.js:541:9)
    at new lib$rsvp$promise$$Promise (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/node_modules/rsvp/dist/rsvp.js:757:9)
    at SassCompiler.updateCache (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/index.js:42:10)
    at /Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/node_modules/broccoli-caching-writer/index.js:96:34
    at lib$rsvp$$internal$$tryCatch (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/node_modules/rsvp/dist/rsvp.js:493:16)
    at lib$rsvp$$internal$$invokeCallback (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/node_modules/rsvp/dist/rsvp.js:505:17)
    at lib$rsvp$$internal$$publish (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/node_modules/rsvp/dist/rsvp.js:476:11)
    at lib$rsvp$asap$$flush (/Users/jamesdixon/Projects/scout/platform/business/web/node_modules/ember-cli-sass/node_modules/broccoli-sass-source-maps/node_modules/rsvp/dist/rsvp.js:1198:9)
    at process._tickCallback (node.js:355:11)

Removing ember-cli-less causes things to work once again. I tried setting the include paths of both addons to different directories, but unfortunately, I end up with a blank CSS file.
Here's my version info:
version: 1.13.8
node: 0.12.2
npm: 2.13.4
os: darwin x64

and my dependencies in package.json:
  "devDependencies": {
    "broccoli-asset-rev": "^2.1.2",
    "dotenv": "1.2.0",
    "ember-cli": "1.13.8",
    "ember-cli-app-version": "0.5.0",
    "ember-cli-babel": "^5.1.3",
    "ember-cli-content-security-policy": "0.4.0",
    "ember-cli-dependency-checker": "^1.0.1",
    "ember-cli-deploy": "^0.4.2",
    "ember-cli-deploy-azure": "0.0.11",
    "ember-cli-dotenv": "1.0.0",
    "ember-cli-flash": "1.3.1",
    "ember-cli-htmlbars": "0.7.9",
    "ember-cli-htmlbars-inline-precompile": "^0.2.0",
    "ember-cli-ic-ajax": "0.2.1",
    "ember-cli-inject-live-reload": "^1.3.1",
    "ember-cli-less": "1.3.3",
    "ember-cli-mirage": "0.1.7",
    "ember-cli-mocha": "0.9.1",
    "ember-cli-release": "0.2.3",
    "ember-cli-sass": "4.1.0",
    "ember-cli-simple-auth": "0.8.0",
    "ember-cli-simple-auth-testing": "0.8.0",
    "ember-cli-simple-auth-token": "https://github.com/jamesdixon/ember-cli-simple-auth-token.git",
    "ember-cli-sri": "^1.0.3",
    "ember-cli-uglify": "^1.2.0",
    "ember-data": "1.13.8",
    "ember-deploy-redis": "0.0.6",
    "ember-disable-proxy-controllers": "^1.0.0",
    "ember-export-application-global": "^1.0.3",
    "ember-paper": "https://github.com/peec/ember-paper.git#feature/paper-menu",
    "ember-validations": "2.0.0-alpha.3"
  }

Appreciate your help!
Best,
James

i have the same problem?

I'm pretty sure that the way that css preprocessors work at the moment doesn't allow using two together..

This might be possible in the future with embroider work, but at this time is not with the current ember-cli architecture.


Update SIL VERSION_MINOR for new SIL instruction.

I added a SIL instruction in the previous commit!

@swift-ci smoke test and merge.


Incorrect credential gets picked when creating container with private registry that has more than 1 credential.

Server version - V0.17.0
Steps to reproduce the problem:
Create a registry with 2 credentials.
1st one is a valid credential.
2nd is invalid one.
Now launch add container/add service page.
Add image for the private registry and pick the valid credential from the dropdown ( the default was the invalid one - 2nd credential list).
Container fails to launch since it tried to use the invalid credential:
Request sent from API indicates the incorrect credential being used.
{startOnCreate: true, publishAllPorts: false, privileged: false, stdinOpen: true, tty: true,…}
allocationState: null
capAdd: []
capDrop: []
commandArgs: []
count: null
cpuSet: null
cpuShares: null
created: null
dataVolumes: []
dataVolumesFrom: []
description: "testp"
devices: []
directory: null
dns: []
dnsSearch: []
domainName: null
environment: {}
externalId: null
firstRunning: null
hostname: null
imageUuid: "docker:quay.io/ubuntu:14.04.1"
instanceLinks: {}
kind: null
memory: null
memorySwap: null
name: "testp"
networkIds: ["1n2"]
ports: []
privileged: false
publishAllPorts: false
registryCredentialId: "1c10"
removeTime: null
removed: null
restartPolicy: {name: "no"}
startOnCreate: true
stdinOpen: true
systemContainer: null
token: null
tty: true
user: null
uuid: null

You cannot create more than one credential per registry in the UI as of rancherio/ui#152, and the backend automatically picks the first (oldest) matching registry/credential.

v0.20.1
Verified fixed


[SPARK-44634][SQL][3.4] Encoders.bean does no longer support nested beans with type arguments

What changes were proposed in this pull request?
This is a port of 42327
This PR fixes a regression introduced in Spark 3.4.x  where  Encoders.bean is no longer able to process nested beans having type arguments. For example:
class A<T> {
   T value;
   // value getter and setter
}

class B {
   A<String> stringHolder;
   // stringHolder getter and setter
}

Encoders.bean(B.class); // throws "SparkUnsupportedOperationException: [ENCODER_NOT_FOUND]..."

Why are the changes needed?
JavaTypeInference.encoderFor main match does not manage ParameterizedType and TypeVariable cases. I think this is a regression introduced after getting rid of usage of guava TypeToken: SPARK-42093 SQL Move JavaTypeInference to AgnosticEncoders @hvanhovell @cloud-fan
In this PR I'm leveraging commons lang3 TypeUtils functionalities to solve ParameterizedType type arguments for classes
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing tests have been extended to check correct encoding of a nested bean having type arguments.

Merged this PR to 3.4. Thanks!


Allow Katan to be run inside a Docker container

Is your feature request related to a problem? Please describe.
The only current way to successfully run the Katan server is on the local host, with privileges. This is because we have not yet provided a solution to explore the files on the servers without being directly connected to them, outside the Docker environment.
Note: container created by Katan have a common network (katan0) that is created the first time that Katan is run, it can serve something.
Additional context
To do this you will need to create a new configuration option and a new FileSystem implementation and switch it during the initialization of the file system module.

Why not use docker volumes and mount Host volume to the (Katan) container? Or all containers have a shared volume between them?

Why not use docker volumes and mount Host volume to the (Katan) container? Or all containers have a shared volume between them?

Why not use docker volumes and mount Host volume to the (Katan) container? Or all containers have a shared volume between them?

This was considered, however, the question is how Katan (inside his container) will access data from the other containers created ince the server container cannot be linked to the host, unlike Katan itself.

Why not use docker volumes and mount Host volume to the (Katan) container? Or all containers have a shared volume between them?

This was considered, however, the question is how Katan (inside his container) will access data from the other containers created ince the server container cannot be linked to the host, unlike Katan itself.

Well, you stated that the main issue is navigating to files and doing actions in others containers right? Why you can't do this with ssh?
You can expose the docker socket for the Katan containter too, like following example:
docker run -v /var/run/docker.sock:/var/run/docker.sock --name containerB myimage ...
Iff Katan container has any docker client, you will control the others containers from him.

Well, you stated that the main issue is navigating to files and doing actions in others containers right? Why you can't do this with ssh?
You can expose the docker socket for the Katan containter too, like following example:
docker run -v /var/run/docker.sock:/var/run/docker.sock --name containerB myimage ...
Iff Katan container has any docker client, you will control the others containers from him.

Well, you stated that the main issue is navigating to files and doing actions in others containers right? Why you can't do this with ssh?

SSH is a good option, I will start to think about how to create his implementation.
since I marked this issue as a low priority but it is something to have in Katan as soon as possible.

You can expose the docker socket for the Katan containter too, like following example:
docker run -v /var/run/docker.sock:/var/run/docker.sock --name containerB myimage ...
Iff Katan container has any docker client, you will control the others containers from him.

yes, I am aware of this, thanks.

Well, you stated that the main issue is navigating to files and doing actions in others containers right? Why you can't do this with ssh?

SSH is a good option, I will start to think about how to create his implementation.
since I marked this issue as a low priority but it is something to have in Katan as soon as possible.

You can expose the docker socket for the Katan containter too, like following example:
docker run -v /var/run/docker.sock:/var/run/docker.sock --name containerB myimage ...
Iff Katan container has any docker client, you will control the others containers from him.

yes, I am aware of this, thanks.


Remove Python upper version constraint.

Description
Removes the upper Python version constraint. See #33.
Checklist

[x] tox runs successfully
[x] Docs updated
[x] Version updated in pyproject.toml
[x] Version updated in changelog
[ ] Branch merged
[ ] Tag created and pushed
[ ] Published
[ ] Github release created

Thanks @Fuyukai for taking the time to open the issue and fix this. I'm just adding in the CHANGELOG and updated version and I can roll this out.


Cannot pass immutable value as inout argument

I try to compile a simple Protocol Buffer:
message ProtoMessage {
    required uint32 payloadType = 1;
    ...
}

The protoc compiler generates the Swift files but when I try to build the project I get the following error:
/.../Sources/Messages.pb.swift:93:93: error: cannot pass immutable value of type 'inout UInt32?' (aka 'inout Optional<UInt32>') as inout argument
    case 1: handled = try setter.decodeSingularField(fieldType: ProtobufUInt32.self, value: &_payloadType)
```

Any ideas?

  There was a recent change around how fields are modeled in general that might have made this go away if you were on a tag.

I just followed the instructions here:
https://github.com/apple/swift-protobuf/
So, I guess, I am using master. Any other ideas?

I am using:
$ protoc-gen-swift --version
protoc-gen-swift 0.9.24

There's a few directions there; swift build (using a tag), CocoaPods (uses master), a clone a build (uses master).

Re:

dependencies: [
    .Package(url: "https://github.com/apple/swift-protobuf.git", majorVersion:0)
]

I believe that just says use the newest 0.x, so you are using an older tag where things might not have been working.

There also is a testing file that covers this:
https://github.com/apple/swift-protobuf/blob/master/Protos/unittest_swift_all_required_types.proto#L57
  required   uint32 required_uint32   =  3;

Generated:
https://github.com/apple/swift-protobuf/blob/master/Tests/SwiftProtobufTests/unittest_swift_all_required_types.pb.swift
So it is built into the tests.

Actually, I think what is happening is you are mixing versions.
protoc-gen-swift is likely from trunk, so it is generating for the current version of the runtime, but the Package file is pulling a tagged version.  So the sources you are generating don't work with it.
You need to build protoc-gen-swift from the same 0.9.24 tag that the package manager is including in your app.

Indeed, I built from the 0.9.24 tag and it worked.
Maybe the documentation should be updated to make this point more explicit.
In any case, thanks for your help :)

np.  Yea, the docs have a small nod to it, but the problem is our master reports the same things as our last branch.  ☹

yeah, the version on the master should be updated or something.
On Thu, Oct 27, 2016 at 12:23 AM Thomas Van Lenten<EMAIL_ADDRESS>wrote:

np. Yea, the docs have a small nod
https://github.com/apple/swift-protobuf#building-your-project-with-swift-build
to it, but the problem is our master reports the same things as our last
branch. ☹
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apple/swift-protobuf/issues/120#issuecomment-256481176,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACR3nDxGR6vAWylF62MSWMuAMQkgA4mks5q38S2gaJpZM4Kg2KH
.

yeah, the version on the master should be updated or something.

Yup, bumped it in PR #122 right after closing this yesterday.