fix: consistently use test namespace for all e2e tests

The one exception is the collector_pipeline test, which is deployed using the manifest. Absent a way to generate a manifest with an alternative namespace, it's easier to just use the eraser-system namespace when deploying from manifest.
Signed-off-by: Peter Engelbert<EMAIL_ADDRESS>What this PR does / why we need it:
Additionally fixes #529 by removing hardcoded reference to the eraser-system namespace.
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #529
This should be merged before #510

Codecov Report

Merging #530 (6152922) into main (5d4e8d7) will increase coverage by 0.31%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #530      +/-   ##
==========================================
+ Coverage   14.47%   14.79%   +0.31%
==========================================
  Files          12       12
  Lines        1547     1514      -33
==========================================
  Hits          224      224
+ Misses       1295     1262      -33
  Partials       28       28

Flag
Coverage Δ

unittests
14.79% <0.00%> (+0.31%)
:arrow_up:

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

Impacted Files
Coverage Δ

test/e2e/util/kubectl.go
0.00% <0.00%> (ø)

test/e2e/util/utils.go
7.46% <0.00%> (+0.58%)
:arrow_up:

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


Set or Remember Selection

Hi,
Is there a way to set the selected drawer menu item so that when a user views the drawer, the area of the app they are in is selected?

Can you expose the selectedPageIndex property so that users can set the selected menu item?

Hi, hope you are fine.
Yes I'll implement this feature someting like a setter for  selectedPageIndex property


Make CEA parser & decoder as plugins

Have you read the FAQ and checked for duplicate open issues?
Yes
Is your feature request related to a problem? Please describe.

Shaka currently has plugin system, i.e. to decide in individual build shall some text parsers be included. However, it doesn't work like that for closed captions - all of related parsers etc. are attached to the core build. If someone doesn't need TS parser or captions at all, he needs to live with that.
Describe the solution you'd like

Prepare plugin registry similar to text parsers plugins:

shaka.cea.ICeaParser/ICeaDecoder:

change to externs

shaka.media.ClosedCaptionParser:

add static exported methods for adding, removing and getting CEA Parser/Decoder plugins
get proper parser by a MIME type from plugin registry
fallback to shaka.cea.DummyCeaParser/Decoder if can not find plugin

shaka.cea.Mp4CeaParser/TsCeaParser/CeaDecoder:

register as a plugin (using static methods mentioned above)
exclude from @core

Describe alternatives you've considered

Forking and removing unneeded dependencies manually.
Additional context

I did some experiments already and I'm happy to work on that if Shaka team thinks this idea is valuable for the project.

For me it's ok!

Sounds good.  Thanks!


Updated SwiftASTContext to find a single module with Swift flags.

This makes LLDB get flags from the following sources (in order of preference):

the main executable;
a unit test bundle; or
the first module that has a Swift AST.

We used to take the union of all the Swift flags if the first two cases didn't apply.
This was a bad approach and caused a lot of user frustration (mostly of the form
"po doesn't work.")
rdar://problem/29906378

The approach sounds good to me, except that I think the unit test bundle ought to win over the main executable. I get that that's trickier, though.

@swift-ci please test

@scallanan A request: Can you rebase on your branch rather than merging in? It makes a nicer git history when you merge.

@gottesmm If that's standard procedure, I'm happy to follow suit.

@scallanan Just to be clear I am only talking on this branch. You should merge it in normally (not squash). That is the procedure we are using in general terms.

Close as nobody is working actively on this. We can reopen if we need to.


add swift fork bomb :trollface:

doesnt work on IOS tho

TIL you can add images in pull request titles

Nice, seems like a real great addition since nobody would ever think of using swift for a non-ios application


Implement PyMemory::get_range_as_ints().

 Add method PyMemory::get_range_as_ints()

Codecov Report

Merging #147 (1dfa563) into main (56490ea) will increase coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
+ Coverage   71.09%   71.27%   +0.17%
==========================================
  Files          12       12
  Lines         640      644       +4
==========================================
+ Hits          455      459       +4
  Misses        185      185

Impacted Files
Coverage Δ

src/memory.rs
90.47% <100.00%> (+2.24%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add "approved by" field to Leader requests

The DB should have the notion of which logged-in user approved/rejected a leader request.

@daganomri wanna take this?

Yeah I got it, sounds simple


Change Modifications to be its own class

This PR changes the way we write modifications in the urdf-modifiers library.
Modification is now a class part of urdfModifiers.core with methods for adding the different modifications.
A constructor for creating this class from a section of an ini file has also been added.
Below some examples of the changes:
Previous way of creating modifications:
modifications= {}
modifications[utils.geometry.Modification.DIMENSION] = [0.2, utils.geometry.Modification.ABSOLUTE]
modifications[utils.geometry.Modification.DENSITY] = [2.0, utils.geometry.Modification.MULTIPLIER]

New method:
modifications = Modification()
modifications.add_dimension(0.2, absolute=True)
modifications.add_density(2.0, absolute=False)

As you can see it is now clearer and the code is cleaner.
cc @CarlottaSartore

fyi @fabiodinatale

@GrmanRodriguez @vvasco can we tag the version of the library before this PR as v0.0.1 or something similar? we have some scripts that use the old modifications API, and while we could refer to specific checkout, it may be easier to refer to a given tag. No release is necessary, just a small tag is ok.

good idea @traversaro thanks

In that case I'd say first accept #12 since it's non-breaking and adding it to the tag

In that case I'd say first accept #12 since it's non-breaking and adding it to the tag

I agree


Looking for a new maintainer?

Hey @gandm, I think this is a very important package in the react community. If you would like to pass this project on instead of not supporting it anymore, I would like to offer to maintain it. The only way I could get in contact with you is by opening a PR. Please contact me here or via email<EMAIL_ADDRESS>if you would consider letting me take over. Thank you.

Thanks for the offer but I think it would be prudent to leave things as they are. I will ensure that the current code runs on Atom1.13 but I won't be fixing any issues. Part of the reason for this is my understanding that the Facebook Nuclide team may well introduce a new grammar for flow, js, jsx that will be based upon vscode's new typescript grammar. If this does turn out to be true, then that will prove to be better solution long term. Also, I have found some of the comments in the issues raised rather curt and not terribly supportive.
I'll close this for the time being and if things don't work out as I expect I will be happy to pass the code base over. Although I would advice against it. Looking at the star rating for the react package also gives me optimism that this would address most users needs.

Sounds good, thanks for all of your hard work on this.

Thank you @gandm. Your work on this package and keeping up with crazy JS syntax has been heroic. 💪
I'm really interested in this news about a Facebook-supported nuclide/atom grammar. Where did you hear about this?

@jonboiser thanks for the kind words. I believe Andres Suarez is working on this but as with most things in life it may come to nothing. I believe it will be done first for Sublime Text and then Atom and VScode. see his aims for the package


Zooming on map causes UI to become unresponsive

Hi,
More of a question than an issue, library works great.
It's just when you click the zoom button a couple of time or use scroll wheel on mouse to zoom, the map crashes and UI becomes unresponsive. Exceptions being thrown are

java.lang.IllegalArgumentException: Texture must be non-null

java.lang.IllegalStateException: Operation requires resource lock

From Googling, it seems this could be an issue with the JavaFX webview itself. Although, most of the issues are closed and JavaFX say the issue has being resolved.
So I'm just wondering if you guys came across this and would have any advice on how to workaround it:)
I'm thinking I could disable the scroll wheel and try prevent user from clicking zoom button too often!! Lost to be honest, would adding these 2 vm arguments help
-Dprism.order=sw
-Xmx2048m

I have experienced this in the past, but I just tried to recreate it by rapidly zooming and panning and couldn't. This was on both the supplied demo app and my own one, though they are compiled from some code that hasn't made it to release yet so it could be that some changes over the recent past have resolved this. It could also be related to particular platforms and versions of Java? Sorry I can't offer more in the way of workarounds, perhaps if Rob is seeing the problem too he might have some ideas.

Hello,
I have experienced this issue too. Rapid zooming with mouse wheel causes a few second unresponsiveness in the zooming process, while the mapView loads the new parts.
My env is Netbeans 8.0.2; win 8.1; maven 3.05; jdk1.8.0_65
Also prints out some interesting things:
Outstanding resource locks detected:
D3D Vram Pool: 536,146,668 used (99.9%), 536,870,912 target (100.0%), 536,870,912 max
515 total resources being managed
average resource age is 0.0 frames
0 resources at maximum supported age (0.0%)
511 resources marked permanent (99.2%)
2 resources have had mismatched locks (0.4%)
2 resources locked (0.4%)
511 resources contain interesting data (99.2%)
0 resources disappeared (0.0%)

After the exception below is raised, the application just goes on, you can work with the map.
If forced too much or again, it may raise exception:
java.lang.NullPointerException
	at com.sun.prism.impl.BaseContext.updateMaskTexture(BaseContext.java:231)
	at com.sun.prism.impl.ps.BaseShaderGraphics.renderShape(BaseShaderGraphics.java:482)
	at com.sun.prism.impl.BaseGraphics.fill(BaseGraphics.java:372)
	at com.sun.javafx.webkit.prism.WCGraphicsPrismContext$ClipLayer.render(WCGraphicsPrismContext.java:1442)
	at com.sun.javafx.webkit.prism.WCGraphicsPrismContext.renderLayer(WCGraphicsPrismContext.java:202)
	at com.sun.javafx.webkit.prism.WCGraphicsPrismContext.restoreStateInternal(WCGraphicsPrismContext.java:219)
	at com.sun.javafx.webkit.prism.WCGraphicsPrismContext.restoreState(WCGraphicsPrismContext.java:233)
	at com.sun.webkit.WebPage.paint2GC(WebPage.java:699)
	at com.sun.webkit.WebPage.paint(WebPage.java:665)
	at com.sun.javafx.sg.prism.web.NGWebView.renderContent(NGWebView.java:96)
	at com.sun.javafx.sg.prism.NGNode.doRender(NGNode.java:2053)
	at com.sun.javafx.sg.prism.NGNode.render(NGNode.java:1945)
	at com.sun.javafx.sg.prism.NGGroup.renderContent(NGGroup.java:235)
	at com.sun.javafx.sg.prism.NGRegion.renderContent(NGRegion.java:576)
	at com.sun.javafx.sg.prism.NGNode.doRender(NGNode.java:2053)
	at com.sun.javafx.sg.prism.NGNode.render(NGNode.java:1945)
	at com.sun.javafx.sg.prism.NGGroup.renderContent(NGGroup.java:235)
	at com.sun.javafx.sg.prism.NGRegion.renderContent(NGRegion.java:576)
	at com.sun.javafx.sg.prism.NGNode.doRender(NGNode.java:2053)
	at com.sun.javafx.sg.prism.NGNode.render(NGNode.java:1945)
	at com.sun.javafx.tk.quantum.ViewPainter.doPaint(ViewPainter.java:477)
	at com.sun.javafx.tk.quantum.ViewPainter.paintImpl(ViewPainter.java:330)
	at com.sun.javafx.tk.quantum.PresentingPainter.run(PresentingPainter.java:91)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308)
	at com.sun.javafx.tk.RenderJob.run(RenderJob.java:58)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at com.sun.javafx.tk.quantum.QuantumRenderer$PipelineRunnable.run(QuantumRenderer.java:125)
	at java.lang.Thread.run(Thread.java:745)

Also after closing the app, maven says:
Final Memory: 18M/169M
I'm not sure from where this final memory thing is taken, I'm quite new to maven and javaFX also.

This looks like a JavaFX problem. Similar things have shown up http://stackoverflow.com/questions/23467797/javafx-nullpointerexception-when-rendering-big-image for example, and the suggestion is setting the VM arguments as in the original post. A bit of slowness is to be expected, even Google Maps on the browser can experience this when zooming quickly, but the NPE is not so good. You should also update your java and see if that helps, it's up to 112 now, so 65 might be missing some fixes.

Thanks, I will surely update my version.
Do you have any idea how could someone disable the mousewheel usage at the map (or in general), to avoid this? I couldn't find the javafx (or mapView or map or scene) function for it. I also couldn't find where this behavior (event handler or something) can be found, that makes this the way it is. Googling resulted that there is a function in Maps API v3 to achieve/disable this.
Could you guide me where to look to change gmapsfx to make this work?
Mine is a desktop java app (at the moment).
In the meantime I will try to find how to make those parameters work in my situation =]

You can set scrollwheel to false, this worked for me. Zooming using buttons is ok and doesn't cause above issue.

Add this method to MapOptions:
public MapOptions scrollWheel(boolean scrollControl ) {
setProperty( "scrollwheel", scrollControl );
this.scrollControl = scrollControl;
return this;
}
Then in mapInitialized(), can set options
@Override
public void mapInitialized() {
    LatLong center = new LatLong(53.3856292, -6.3759797);
    MapOptions options = new MapOptions();
    options.center(center)
            .mapMarker(true)
            .zoom(12)
            .scrollWheel(false)
            .overviewMapControl(false)
            .panControl(false)
            .rotateControl(false)
            .scaleControl(false)
            .streetViewControl(false)
            .zoomControl(true)
            .mapType(MapTypeIdEnum.ROADMAP);

    map = googleMapView.createMap(options, false);

}

Thanks @johnmc5810 I've just committed a change to MapOptions to add this in. Looking at the docs there are still a few parameters in MapOptions that we haven't included yet.

No worries, thanks for the library:) if i come across anymore. I'll send them on


Fix polyglot number comparison

Pull Request Description
Fix polyglot number comparison/equality.
Important Notes
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

Equals_Spec.enso is failing and #5744 should help with that. Waiting until #5744 is merged.

Wait until #5778 is merged.

Closing in favor of #5832.


Docker Hostname as Dynamic Endpoint Address

Title: Enable using docker hostname as address for dynamic EDS
Description:
Docker service name for a static cluster address defined in envoy config yaml works well and resolves fine, as cluster type is logical_dns.
Snippet:
static_resources:
  clusters:
  - connect_timeout: 1s
    hosts:
    - socket_address:
        address: my-docker-service
        port_value: 8888
    name: ads_cluster
    type: logical_dns

For dynamic Envoy config, cluster type = EDS, and multiple dynamic cluster with its dynamic Endpoints. If I provide normal IP Address and port for each Endpoint, it works fine.
But when using docker containers' Hostname as address for EDS, it throws a warning even though EDS type (resolver_name) is defined as LOGICAL_DNS.
Initialized Endpoint as
endpoints := make([]cache.Resource, 1)
var eps []endpoint.LbEndpoint
eps = append(eps, endpoint.LbEndpoint{Endpoint: &endpoint.Endpoint{
	Address: &core.Address{
		Address: &core.Address_SocketAddress{
			SocketAddress: &core.SocketAddress{
				Protocol: core.TCP,
				Address: "webapp_dev_webapp.1.no9rqsfuz73k026uod3hsjk83",
				ResolverName: "LOGICAL_DNS",
				PortSpecifier: &core.SocketAddress_PortValue{
					PortValue: uint32(8080),
				},
			},
		},
	},
},
})

endpoints[0] = &v2.ClusterLoadAssignment{
	ClusterName: "my_cluster",
	Endpoints: []endpoint.LocalityLbEndpoints{{
		LbEndpoints: eps,
	}},
}

var edsSource *core.ConfigSource
edsSource = &core.ConfigSource{
	ConfigSourceSpecifier: &core.ConfigSource_Ads{
		Ads: &core.AggregatedConfigSource{},
	},
}
cluster := &v2.Cluster{
	Name:          "my_cluster",
	ConnectTimeout: 5 * time.Second,
	Type:           v2.Cluster_EDS,
	EdsClusterConfig: &v2.Cluster_EdsClusterConfig{
		EdsConfig: edsSource,
	},
}

// some listener and routes too

After generating and setting snapshot with above settings, envoys updates LDS, RDS, CDS but throws error in EDS as
[warning][config] bazel-out/k8-opt/bin/source/common/config/_virtual_includes/grpc_mux_subscription_lib/common/config/grpc_mux_subscription_impl.h:70] gRPC config for type.googleapis.com/envoy.api.v2.ClusterLoadAssignment rejected: Unknown address resolver: LOGICAL_DNS. Consider setting resolver_name or setting cluster type to 'STRICT_DNS' or 'LOGICAL_DNS'.
All service and envoy itself is running in the same docker network.

SOLVED
Below code worked for me. webapp-service is my docker service name. This way envoy resolves webapp-service to as many IPs as its replica count.
var hosts []*core.Address
dnsRefreshRate := 15 * time.Second
hosts = append(hosts, &core.Address{
	Address: &core.Address_SocketAddress{
		SocketAddress: &core.SocketAddress{
			Protocol:     core.TCP,
			Address:      "webapp-service", // webapp-service is a docker service name
			ResolverName: "STRICT_DNS",
			PortSpecifier: &core.SocketAddress_PortValue{
				PortValue: uint32(8080),
			},
		},
	},
})
cluster := &v2.Cluster{
	Name:            "my_cluster",
	ConnectTimeout:  5 * time.Second,
	Type:            v2.Cluster_STRICT_DNS,
	LbPolicy:        v2.Cluster_ROUND_ROBIN,
	Hosts:           hosts,
	DnsLookupFamily: v2.Cluster_V4_ONLY,
	DnsRefreshRate:  &dnsRefreshRate,
}


read grid file path from recipe if provided

Problem
For complicated meshes, the grid file takes a long time to create. There is currently no way to reuse grids for the same compartments without renaming the grid file.
#204
Solution
I added a check for "grid_file_path" in the recipe which loads from the specified grid file if it is present.
Type of change

New feature (non-breaking change which adds functionality)

Change summary:

read grid file from recipe if provided and if file exists

Steps to Verify:

Rename previously created grid file for a recipe
Add an option in the recipe with "grid_file_path": PATH_TO_RENAMED_GRID_FILE
Use a config with "load_from_grid_file": true

cellPACK should load the grid from the specified path instead of the default
Keyfiles:

Environment.py

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (aae7ed1) 98.52% compared to head (2d00f18) 98.52%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #205   +/-   ##
=======================================
  Coverage   98.52%   98.52%
=======================================
  Files          16       16
  Lines         476      476
=======================================
  Hits          469      469
  Misses          7        7

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


[Design] Decouple tooling from MVC assemblies

I had to hack up a bunch of stuff to get this going, but it works!

This is looking really cool! 🍾 👏

Updated and removed design label


Spotify only load black screen

I manage applying the theme without any errors. But when i open spotify this black screen showup.

Spotify version : <IP_ADDRESS>9.gc253025e
Spicetify version : 2.9.6

FIXED! 👍🏻
disable "spicetify-marketplace"
set check for updates to "1"

I've been having the same blank screen on spotify 1.1.80 and spicetify 2.9.7. There are a number of issues that I have been tracking on https://github.com/spicetify/spicetify-cli/issues. It looks like once we get 1.1.83 the issues will be resolved.
The steps I did to get thinks working was installing 1.1.83. For arch I had to get it from the spotify-edge package from AUR .

Uninstall current spotify
Install latest spotify
spicetify backup apply
Profit

Using the spotify-edge AUR package also fixed it for me

Nah, it didn't work for me. On my case, the spicetify-marketplace is the
culprit. I just remove that from my xpui config and it fixed.
On Tue, Apr 26, 2022 at 9:03 PM Nico @.***> wrote:

Using the spotify-edge AUR package also fixed it for me
—
Reply to this email directly, view it on GitHub
https://github.com/catppuccin/spicetify/issues/14#issuecomment-1109838579,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5JVMOSXXVFOBBIADKRGNDVG7ZTBANCNFSM5UGDWPDQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Native notebooks: Remote kernels are not refreshed every time a notebook is opened.

Since remote supports connecting to an existing kernel, every time we create a new kernel we should mark the kernel provider as requiring a refresh.

Adding to triage as this should probably be fixed as part of making kernels work better. The kernel list needs to be updated when opening a notebook.

Closing this so that it gets verified as part of the iteration


VeraStatus doesn't detect drives mounted as a service

I use sc to run VeraCrypt as a service and mount a drive at startup with this command:
sc create veracryptauto binpath= "\"C:\Program Files\VeraCrypt\VeraCrypt.exe\" /v C:\EncryptedDrive\EncryptedDrive.hc /l a /p MySecretPassword /a /s /q" type= own start= auto displayname= veracryptauto
When I log in to my administrative account and run VeraStatus.exe A: I get:
Drive letter A: doesn't correspond to a mounted VeraCrypt volume.
I am using VeraCrypt version 1.23-Hotfix-2 (64-bit) on a 64-bit Windows Server 2008 R2 Datacenter with Service Pack 1.

My mistake. I wasn't running VeraStatus as administrator. The drive details showed up once I ran it in an administrator command prompt.


Changed $timeout to $interval

Changed the use of $timeout to use $interval in order to pass protractor tests. I ran the test suite prior to changes and protractor was failing to synchronize with the page. Tests now pass with use of $interval. #58

Thanks for the contribution @aheys. Do you happen to know if it fixes the issue #54?

No it doesn't. I wasn't able to track down that bug.


Workflow saves an empty "settings.json" file, and stops working.

Every once in a while, the workflow saves the settings.json file as an empty file / 0 bytes. When this happens, the workflow does not work anymore.
Usually (but not every time), the error occurs when I enter some invalid input, like "cal <fweiuf" , then I add/remove characters from the query string. The error might also occur when I type the input a bit too fast.
When I run the main script via iPython:
In [1]: from main import Main

In [2]: main = Main("")

In [3]: main.execute()
16:18:58 workflow.py:2070 DEBUG    Workflow version : 1.2.1
16:18:58 workflow.py:1473 DEBUG    Reading settings from `/Users/XXXXX/Library/Application Support/Alfred 2/Workflow Data/zmin.calendar/settings.json` ...
16:18:58 workflow.py:2088 ERROR    No JSON object could be decoded
Traceback (most recent call last):
  File "workflow/workflow.py", line 2081, in run
    func(self)
  File "main.py", line 39, in main
    self.init_settings()
  File "main.py", line 29, in init_settings
    self.minus = self.wf.settings.setdefault('minus', self.minus_default)
  File "workflow/workflow.py", line 1475, in settings
    self._default_settings)
  File "workflow/workflow.py", line 900, in __init__
    self._load()
  File "workflow/workflow.py", line 911, in _load
    for key, value in json.load(file_obj, encoding='utf-8').items():
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/__init__.py", line 290, in load
    **kw)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/__init__.py", line 351, in loads
    return cls(encoding=encoding, **kw).decode(s)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/decoder.py", line 365, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/json/decoder.py", line 383, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
16:18:58 workflow.py:2106 DEBUG    Workflow finished in 0.009 seconds.
An exception has occurred, use %tb to see the full traceback.

.
And the UI shows:

When I edit the settings.json file to contain an empty JSON object, the workflow starts working again.

Hi @boris-chervenkov,
Have you tried the latest update? This issue should be fixed by v1.2.1. Please let me know if the problem persists.

Hi, @owenwater,
I thought I was using the latest version of the workflow, but apparently I was not. I'll let you know if something comes up.
Thank you again for the wonderful Alfred workflow :o)

Boris


zIndex for Groups

I have the situation where I'd like to order Groups by their zIndex. I have a menubar at the top of the screen and a ListView below it. Now that menubar should always stay on top of other elements, which it doesn't because it is drawn first.
In the source I found that Layers are in fact ordered by their zIndex, but Groups are not. Would that ordering be applicable to Groups, too?

This should work. Do you have a reproducible test case?


Harmonize fg/bg/session check intervals with iOS

This has been done

This involved changing the iOS tracker - was a no-op in this tracker.


WALinuxAgent (waagent.conf) docs update for AutoUpdate and EnableSwapEncryption

Corrected walinuxagent sed search string: #93251
ResourceDisk.EnableSwapEncryption (only FreBSD.): https://github.com/Azure/WALinuxAgent/issues/2628

@soyfrien : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MsGabsta
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@soyfrien, thanks for this request.  We are reviewing it internally.

Is this change still needed? If not, we can close it by adding a comment that says # please-close (take out the space after the #).

They will update the docs instead of merging this.
I can't seem search the repo for "AutoUpdate.Enabled=y" right now, but it would be good to make sure other articles aren't also cloudy. Troubleshoot the Azure Linux Agent is another one that would make people decide to edit /etc/waagent.conf to remove the comment.
Just for context here is what the conf file looks like:
# Enable checking RDMA driver version and update
# OS.CheckRdmaDriver=y

# Enable or disable goal state processing auto-update, default is enabled
# AutoUpdate.Enabled=y

# Determine the update family, this should not be changed
# AutoUpdate.GAFamily=Prod

#please-close

Invalid command: '#please-close'. Only Microsoft employees can use this command.

I can't seem search the repo for "AutoUpdate.Enabled=y" right now
So, I got a chance to search the repo:

WALinuxAgent$ grep -e '\# AutoUpdate.Enabled=y' -rn .
./tests/data/test_waagent.conf:120:# AutoUpdate.Enabled=y
./config/openbsd/waagent.conf:99:# AutoUpdate.Enabled=y
./config/alpine/waagent.conf:78:# AutoUpdate.Enabled=y
./config/waagent.conf:128:# AutoUpdate.Enabled=y
./config/coreos/waagent.conf:107:# AutoUpdate.Enabled=y
./config/suse/waagent.conf:116:# AutoUpdate.Enabled=y
./config/arch/waagent.conf:103:# AutoUpdate.Enabled=y
./config/ubuntu/waagent.conf:104:# AutoUpdate.Enabled=y
./config/freebsd/waagent.conf:105:# AutoUpdate.Enabled=y
./config/debian/waagent.conf:113:# AutoUpdate.Enabled=y

WALinuxAgent$ grep -e 'AutoUpdate.Enabled=n.*' -rn .
./config/nsbsd/waagent.conf:101:AutoUpdate.Enabled=n
./config/gaia/waagent.conf:104:AutoUpdate.Enabled=n

WALinuxAgent$ grep -e '# AutoUpdate.Enabled=n' -rn .
[no result]

There are three because there are actually that many variations in the docs with the last two appearing four or five times. So even though my edit was adding the first one, I still missed all the instances of sed on that page.
The last, empty result will never occur. Probably not even in the wild. And this is the part where someone says it is correct. 😉
Really, I don't know what to make of that, I'm obviously new this. But I hope this helps...
PS - Here is a sed command that can catch whatever the docs say:


linting broken in Travis builds

Need to figure out what changed with linting - breaking for all builds now, seems like.

@sjsadowski Seems like a linter issue with import sorting. Let me take a look

@sjsadowski It was a minor import order issue related to webscoket package in testing utility. Opened a new PR to fix it.


Fix non PE-D documentation bugs in UG

==ready to merge==
You can refer to the UG here
Changes made:

Add glossary and student command field table
Mention that course/group/task/session can have special characters
Add design decisions

Nice good job! LGTM 👍


[Remove FontAwesome] Replace icons in the side drawer

A continuation of https://github.com/bluesky-social/social-app/pull/3438. Here we are just replacing the same icons from the previous PR in the side drawer as well on native.
Test Plan
Open the drawer, select the various icons. In the instances where you can still open the drawer from the page you navigate to, verify that the icons become filled.

We did it!


Generate Midi data from Elm data

Hi. Nice library! I really like how the midi message types nicely translate to Elm union types.
I'm working on a browser based step sequencer and I want some way to save the step sequencer phrases to disk. At first I was going to use Json.Decode, but that will be a headache to maintain as this is a work in process, and I'm likely to change the internal data format, rendering the saved files obsolete. I figure MIDI might be a better serialization format -  I should be able to map it to my internal step sequencer note data, and if I abandon my project, any saved data will still be readable as a MIDI file in any another software that can read midi files.
Would it make sense to add this functionality to this package, or do you think it should be a separate one? The main thing that would be common between the two would be the types in MidiTypes.elm. I'm happy to have a go at building the generator in a fork of this repo - I'll let you know how I go! If I get somewhere with it, I'll submit a pull request.

Hi, @mbylstra - many thanks!
I think using MidiTypes.elm as a serialization format makes sense.
At the moment, I'd prefer it to be a separate package and have it dedicated to being a MIDI parser.  However, I have something of the same problem, given that my midi-player and abc-player packages also depend on MidiTypes.elm.
Perhaps there are two ways to go with this.  One way would be to publish MidiTypes.elm as a separate package - I'd be happy to do this.  Another might be for you to go ahead with your generator and then see if there is enough coherence between our various projects in order somehow to combine them into some sort of uber-MIDI library.  At the moment I think I prefer the former, but I'd be very interested to see what direction your project will take.

I'm also interested in this and I agree that splitting MidiTypes.elm into a separate package seems like a good way to go about it.
I might even have time to write the generator. I'll take a look at it today.

My preferred option would be to be able to live in an Elm world where there was a single place to go in http://package.elm-lang.org/ for comprehensive MIDI support. This would require Elm to have a straightforward story to tell on binary data and also for it to climb down from its position on the publication of native modules which only wrap a browser API such as web-midi.  Unfortunately, I can't see this happening.
This probably being the case, I'd be prepared to have a separate MidiTypes package.
I'd be interested to hear @mbylstra's thoughts.

Yeah, I was pretty surprised to learn Elm didn't have any kind of real support for binary data. Splitting things is just the best of the options I realistically see available.

Agreed.  (sigh).

I've gotten far enough along on a project that I could really use a library to generate "binary" MIDI data from MidiTypes and I like the idea of MIDI support existing in one place. How would you feel about a PR that restructured it like this?
Midi.Types (MidiTypes)
Midi.Parse (CoMidi)
Midi.Generate (what I'll write)

I'd be happy with this now, I think.  Would you be able to liaise with @mbylstra to see whether he is still interested in the idea and whether your requirements match his?

I was assuming @mbylstra wasn't interested anymore given the lack of response to this thread. Either ways, my plan was to add two functions.
Midi.Generate.event: Midi.Types.MidiEvent -> List Int
Midi.Generate.event: Midi.Types.MidiRecording -> List Int
I think the latter should handle pretty much all cases where you want to serialize MIDI data, including @mbylstra 's case. My own application also has a sequencer which will probably make use of this too (though I'm primarily more interested in sending MidiEvents).

It's looking good, and a natural fit with the parser in that we have its inverse operation.  I like the fact that you are round-tripping and using QuickCheck style checks.  I'm very tempted eventually to port this over (with acknowledgement) to my PureScript version if you have no objections.

No objections at all. I'll be happy to see the code have more impact.
I love how amenable this kind of code is to QuickCheck style tests. It feels almost like it belong in a textbook with how well it works out.


Return Stream Struct from Streaming Method/RPC

Feature Request
Motivation
To prevent excess cpu/mem usage caused by re-instantiation of streaming requests/connections when messages are sent from another thread.
Proposal
When the client-streaming method is invoked, the Go gRPC implementation returns a grpc.ClientStream struct, on which "Send(request)" can be called as messages are readied. This allows for a persistent connection and true streaming of requests from service to service. From what I've seen with tonic, streams can only be created from iterables, forcing messages to be sent in batches, and the connections end up being relatively short-lived.
Ideally, I'd like to have a client stream that can send individual messages as they're received over an mpsc channel without dropping the stream's connection each time.
If I'm overlooking something that can be done as a workaround, please let me know! I've searched for a while and haven't found anything so far.
Thanks

A lot of the examples use iterators but a Stream in rust can be anything that async returns items, like a channel or like how you described with go. https://tokio.rs/tokio/tutorial/streams is a good place to start with streams. What you described as the ideal situation is possible.

For anyone else looking for the answer in the future, it's here
I was over-complicating things by forcing different behavior than intended. Later I found out that I was attempting to reinvent the tokio_stream::wrappers::ReceiverStream struct as another solution. Everything works well with ReceiverStream in place.
I hope this helps someone else with similar questions. No action needed, just leaving this here for posterity. Thanks again, @LucioFranco!


Expected Performance on Mobile?

Reading Issue #26 - https://github.com/jeeliz/jeelizWeboji/issues/26
"It should work nicely with a Samsung S7..."
When i run the Raccoon Demo on a Samsung Note 10 i am only getting around 5-12 FPS with Chrome, and there arent any errors in the console. Im assuming i should be getting better performance if it should work nicely on a 3 generations old phone?

Hi
Unfortunately I don't have this device.
If you send me a galaxy note 10 I could take a look.
Best
Xavier

Unfortunately i dont have a spare $1500 to supply a device.
I know there can be a 100 + 1 device specific issues/variable. Relating to the comment on issue #26 with it working nicely on an S7 ;
https://www.gadgetsnow.com/compare-mobile-phones/Samsung-Galaxy-Note-10-Plus-5G-vs-Samsung-Galaxy-S7
Technically speaking from what i can see, there is the hardware requirements available to run the library on a Note 10 when compared to the S7. So if there is a handful of old devices which get 30 FPS+ then i know i have a device specific issue. Otherwise if working nicely means around 10 FPS on older devices, im not any more clear on benchmarks
I will try to get a hold of a range of devices andabout posting what benchmarks i can

Unfortunately I don't have time to solve your issue (my time is money).

Apologies for misunderstanding, i don't wish for you to solve my issues. Thats why i didn't ask for a fix, commit or a timeline of completion.
I was just wondering what benchmark you may have done/what the basis was for that comment 👍
"It should work nicely with a Samsung S7..."
Was that just a guess?
Just trying to get an understanding*, not asking for free work.
*Typo

Hi,
The source of the problem can be:

the webRTC implementation: maybe the video resolution is too high. SInce many getUserMedia API are bad, especially handling media constraints, if we don't manage to get a low resolution image we try with other media constraints, then with no media constraints at all. In this latest case the video resolution can be high. I had this problem with a version of IOS 12 and some specific media constraints (the bug has been fixed since),
the WebGL implementation may introduce bottlenecks, especially when refreshing the video texture. This operation may involve the CPU, which makes thing slows,
2 WebGL contexts are created: 1 for the neural network processing, it is also the same used to display the video with the user face and the frame, and the other with THREE.js. Maybe the WebGL implementation struggles with switching contexts,
maybe the resolution of the THREE.js canvas is too large,
maybe other applications are running in the background, or other tabs are opened.

So there are different ways to explore to find the problem.
I think the GPU is powerful enough, there is a bottleneck somewhere.
If the cartman demo ( https://jeeliz.com/demos/weboji/demos/svg/ ) works well we can already exclude cause due to getUserMedia implementation or video texture refresh.
When I tested on the Samsung S7 it was not running at 10FPS, but faster enough to be usable (at least 30FPS), with Chrome. It was not my phone so I cannot retest now. On My 4 yo Sony Xperia ZPerformance, which is quite dumb, it works well. On Iphone6 it is a bit slow. On Iphone XR/Ipad pro it works nicely.  These are the latest tests.

Amazing! Thank you for the breakdown, gives me some great options to explore. Ill be sure to return anything interesting i come across looking into it.
Cheers!

Hi @werez-mgr
I have pushed a new neural network, faster and lighter than the previous one. It makes a good difference on my medium range mobile device (4yo sony Xperia, same computing power than Iphone6). I hope it can help.


YIFY releases (not YTS) are not categorized as LQ (Radarr collection of custom formats)

Radarr collection of custom formats - LQ format:
Releases that have YIFY in the name are not categorized as LQ despite LQ having YIFY as one of its words. YTS works fine (it is categorized correctly).

Not a bug - Xxxxxx has not release group
The standard is -XXXXX with some additional support for XXXXXXX) and XXXXX]
@TRaSH- should we split LQ into LQ Group & LQ Rls Name for groups that name sanely vs groups/terms to avoid?

So you mean like 2 LQ cf?

Yup

darn, that means we need to be very carefully what we add to that CF, because of some rlsgrp's have common words.
let's bring this in to discord, being it's easier. i will open a topic

how frequently is - YIFY encountered vs. -YIFY or YIFY)  seems like it's a renamed release and that's the root cause

Fixed with https://github.com/Radarr/Radarr/pull/8883

https://github.com/Radarr/Radarr/compare/7f3e7b360bdb...d4fada9b4ec7
Fixed in radarr


Fix for issue 2370, show machine state after kill is executed #2386

Hi,
This fixes #2370 "kill does not return any message to the user #2370". docker-machine kill displays machine(s) state after kill is executed.
Thanks...

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "2370-kill-return-message" git@github.com:ksasi/machine.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Thanks for the contribution @ksasi.  Unfortunately I have to say, this method of implementing it doesn't look like how I'd like to solve the issue.
Instead of adding extra code to this command, why don't we just echo out the name of the machine after an action has been performed on it?  It would then be similar to how docker rm, docker start, etc. work and we'd only have to update the code in one place.

Fully agree with @nathanleclaire. What we expect is feedback on what was done more than the list of remaining machines.

Thanks for the feedback @nathanleclaire and @dgageot . I will explore on how we can  just echo out the name of the machine after an action has been performed on it.

@ksasi Something like Machine 'target' has successfully been killed is sufficient

It should be sufficient to do something like:
log.Infof("Successfully ran action %q on machine %q", action, host.Name)

in runAction before returning nil.

@nathanleclaire adding log.Info before returning nil in runAction helped. log.Info doesn't seem get executed in case of multiple arguments i.e. docker-kill default tobekilled where default in "Running" and tobekilled already "stopped" unless len(errs) == len(hosts) is also included as below
if errs := runActionForeachMachine(actionName, hosts); len(errs) > 0 && len(errs) == len(hosts) {
return consolidateErrs(errs)
}

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "2370-kill-return-message" git@github.com:ksasi/machine.git somewhere
$ cd somewhere
$ git rebase -i HEAD~4
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Hi @ksasi the PR contains two commits that are not related to your work. You should try to rebate your commit on top of master branch

Hi @dgageot ,
Not returning an error if len(errs) != len(hosts)
$ ./docker-machine ls
NAME          ACTIVE   DRIVER       STATE     URL                         SWARM                ERRORS
default       -        virtualbox   Running   tcp://<IP_ADDRESS>:2376
tobekilled    -        virtualbox   Stopped
$ ./docker-machine kill default tobekilled
Machine "tobekilled" is already stopped.
Sasikanths-MacBook-Pro-2:bin sasikanthkotti$ ./docker-machine ls
NAME          ACTIVE   DRIVER       STATE     URL   SWARM                ERRORS
default       -        virtualbox   Stopped
tobekilled    -        virtualbox   Stopped
As shown above,it appears when we are using docker kill for multiple machines with different statuses, returning an error is causing to
display details of few machines.
Returning an error only if len(errs) == len(hosts)
$ ./docker-machine ls
NAME          ACTIVE   DRIVER       STATE     URL                         SWARM                ERRORS
default       -        virtualbox   Running   tcp://<IP_ADDRESS>:2376
tobekilled    -        virtualbox   Stopped
$ ./docker-machine kill default tobekilled
Successfully ran action "kill" on machine "default"
Successfully ran action "kill" on machine "tobekilled"
$ ./docker-machine ls
NAME          ACTIVE   DRIVER       STATE     URL   SWARM                ERRORS
default       -        virtualbox   Stopped
tobekilled    -        virtualbox   Stopped
Maybe ideal solution would be identify the machine(s) returning error and exclude them from the loop printing the status
and returning error after the loop.
Please advise.

Hey @ksasi, I think the original issue was not clear. I'm going to fix it here: https://github.com/docker/machine/pull/2645
I might have to close this PR afterwards. Sorry and thanks for the contribution

Fixed with https://github.com/docker/machine/pull/2645


adding the ability to get cluster resource needs

@mesosphere-ci retest this please

@sschneid is this good to go?

LGTM, do we want to wait on #110 to land before rolling 1.1.14?


Slow/laggy when opening large (~200K) files

VSCode Version: 0.10.11
OS Version: Win 10

Steps to Reproduce:

Open folder containing this file: https://github.com/kjur/jsrsasign/blob/master/npm/lib/jsrsasign.js
Open file and scroll around a bit. VS Code gets really slow/laggy.

@brockallen Thanks a lot for filing this! :+1:
.. Looks like most time is spent in decodeTextMateToken:

This is now fixed on our side by optimizing the decodeTextMateToken function, but we still need to investigate what makes the grammar go crazy and generate 20+ embedded scopes in this case

Create a file full of (as in half a millon lines):
0123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789
VS Code starts to work slowly (Version 1.0.0 in Windows).
Also if you try to do Ctrl + A to select all the code it crashes.
There are many ways to break VS Code or make it crash when using very large files...

VSCode Version: 1.1.0
OS Version: Win 10 x64

Trying to open a large JS file (~85K SLOC) makes window crash.
Steps to reproduce:

Open a large JS file (e.g., http://developer-autodesk.github.io/viewer3d/2.2/viewer3D.js)
Wait a few seconds to see editor crash.

@digeff I have just pushed a fix in ca2ca08c63c88f7c0bc2735a37a97ff3b8f600fc that fixes that the <textarea> sync we had for accessibility purposes (screen readers) was unbounded and we still have some issues with memory consumption when opening a file (the file gets allocated minimum 2 times), but it gets GC'd after a while (so for your testcase I see the renderer process at 200MB) and with the fix I can select all and have no issue:

@sergeykhrykov Thank you for the repro, I can reproduce the crash immediately. I have opened #6916 to better track it. On a first look, it seems it goes OOM.

@aeschli Where do we get the JavaScript grammar from? https://github.com/Microsoft/TypeScript-Sublime-Plugin ?

It's the TypeScriptReact grammar from https://github.com/Microsoft/TypeScript-TmLanguage

The slowness on our side is dealt with for this example, let's use this to track Microsoft/TypeScript-TmLanguage#174

Since all issues on our side are fixed (no more crashes for these test cases), I would prefer to close this issue. The JavaScript grammar is still broken, but that is being tracked in Microsoft/TypeScript-TmLanguage#174


BUG: Curve Leverage Bot Created, but no Mybots showing and no bot creation alerts

User created the bot: https://etherscan.io/tx/0x8615b0786260539b93618560cb2e8d418363d26e5ab84194aaea45b29fc58cba
However, no bots show up on Mybots page and no bot alerts.
User is unable to adjust the bot.

I checked db and there have been no leverage bots since Jan 19. Please check backend script. @taariq @rootedbox


Fix Ctrl+Q bug

Hi, I found some issues related to not-working Ctrl+Q/File->Quit flows:
Lens just closes the window but still works, however the same button in the tray fully closes the app
So, IMHO the deal in the wrong event, where the system prevents app.exit() call on windows closing.
The fix has tested on Ubuntu, in theory, it should not affect Mac users because of app.exit() calling on Ctrl+Q

Hi @ixrock, thank you for your answer, sorry, it seems there is some misunderstanding from my side...
I suppose that in this case event.preventDefault() function fully prevents termination process, isn't it?
I think we should terminate the app after  Ctrl+Q instead of just closing the window
As an option we can call app.exit() after clusterManager?.stop() line, it will stop clusters and terminate app.

I did some renaming of File menu items in #1420. They should now reflect better the role of those items.

Looked throw  @nevalla  PRs, It seems he has fixed this problem. I'll close the PR


Warning: @deprecated("Use JavaTests instead", since = "Mill 0.11.10")

I just upgraded to mill 0.11.11, and see a warning for trait JavaModuleTests in trait JavaModule is deprecated even though our project isn't using that deprecated trait (JavaModuleTests).
I also saw this warning in a recent issue (https://github.com/com-lihaoyi/mill/issues/3359#issuecomment-2282284235), so I figured the warning may come from mill itself.
I think it's coming from the first line here:
  type JavaTests = JavaModuleTests
  @deprecated("Use JavaTests instead", since = "Mill 0.11.10")
  trait JavaModuleTests extends JavaModule with TestModule {

https://github.com/com-lihaoyi/mill/blob/main/scalalib/src/mill/scalalib/JavaModule.scala#L39-L41
It can be reproduced on main with e.g. ./mill -i dev.run anything, even if you update all JavaModuleTests  to JavaTests.
I had a try with some @nowarns on the non-deprecated type in the repo, and removing references to the deprecated JavaModuleTests, but didn't manage to fix it. It looks like it could help though (quick check in Scastie), but no luck trying it in the actual repo:

(Tangent: somehow our -Xfatal-warnings aren't triggered by the warning, so that's good - but we might just have something misconfigured :sweat_smile: )

Same here. I reverted to 0.11.9 for now.


feature Request: PDF/A support

It would be nice to have PDF/A Format supported. Any plans to implement this anytime soon?

I agree this would be a nice feature. There are currently no plans to implement this. We are happy about pull requests, though.

Duplicate of #2882.


CS0103 Error -> BuildMethodException

Getting a BuildMethodException / Can't make standalone build because of these two errors. Seems to be related to #if UNITY_EDITOR. Any suggestions?

Assets/3rd Party Assets/WaveFunctionCollapse/Code/WaveFunctionCollapse/ModuleData.cs(126,9): error CS0103: The name 'EditorUtility' does not exist in the current context
Assets/3rd Party Assets/WaveFunctionCollapse/Code/WaveFunctionCollapse/ModuleData.cs(127,9): error CS0103: The name 'AssetDatabase' does not exist in the current context

Got a working solution from https://github.com/vertxxyz
"using UnityEditor;" must be wrapped in #if UNITY_EDITOR
#endif must be after "SavePrototypes()", but before "OnBeforeSerialize()" and "OnAfterDeserialize()"

Thank you!


Refactor test_pkcs_11.py

It doesn't make sense to call from system_test.py the script as a command, add the option to import and use it directly

done, merged in #99


sds: pause SDS requests while parsing CDS and LDS resources

Commit Message: pause SDS requests while parsing CDS and LDS resources
Additional Description:
Prevent sending a per parsed cluster/listener SDS request. Envoy will send a single SDS request for all the resources.
Risk Level: Medium - modifying typical SDS workflow.
Testing: Added an integration test for the CDS case.
Docs Changes: None.
Release Notes: Added.
Platform Specific Features: N/A.
[Optional Runtime guard:]
Fixes #21518.
Possibly related to: #21492 and #10793
Signed-off-by: Adi Suissa-Peleg<EMAIL_ADDRESS>

cc @howardjohn @mattklein123 @htuch
LMK if you think this should be runtime guarded.

I'd suggest runtime guarding.

/retest

Sorry for the delay here, seems like this needs a main merge to resolve the changelog


TypeScript types generated as optional but should be required

First of all, I want to thank you for building a really great tool that wilI make development easier 👍
I am using Kiota to generate TypeScript code. It is really easy to use, but I am getting problems with the generated types.
OpenAPI schema
schemas:
  Station:
    type: object
    required:
      - name
      - id

OpenAPI schema with nullable
  schemas:
    Station:
      type: object
      required:
        - name
        - id
      properties:
        id:
          nullable: false
          type: integer
          format: int64
          example: 17
        name:
          nullable: false
          type: string
          example: Trelleborg

Swagger

Kiota generated types
export interface Station extends AdditionalDataHolder, Parsable {
    /**
     * Stores additional data not described in the OpenAPI description found when deserializing. Can be used for serialization as well.
     */
    additionalData?: Record<string, unknown>;
    /**
     * The id property
     */
    id?: number | undefined;
    /**
     * The name property
     */
    name?: string | undefined;
}

Notice that both id and name are defined as: name?: string | undefined.

Questions

Why are they generated as optional when they are, in OpenAPI, defined as required and non-nullable?
Why is generated both with ? and undefined

Thank you so much for an awesome tool! I do appreciate all feedback on this, so that I an progress.
Additional references

When I try openapi-generator-cli to generate, this works as expected.
Looking at the samples here, https://github.com/microsoft/kiota-samples/tree/main/petstore/typescript/src/models, ll properties on the model types are also undefined/optional.

Hi @nonameolsson
Thanks for your interest in kiota and for reaching out.

This is a design decision, lots of APIs allow for field selection via query parameters, or partial representations during updates and since the models can be used both as response and request body, having nullable properties makes them more reusable.
You are correct, this is redundant, on properties we could loose the undefined, we need to keep it on setters since they don't support the ? notation.

More generally we should implement some kind of generated signal to avoid triggering linters for generated code, if this is available for eslint/ts parser. In the meantime we could fix that ad-hoc redundancy, would you be willing to submit a pull request for that?
The code that emits the property is located here
https://github.com/microsoft/kiota/blob/4eb8187f0183f0fac8e16ddade2dbedcb441fbff/src/Kiota.Builder/Writers/TypeScript/CodePropertyWriter.cs#L46

As per the generated annotation, this can be done through comments
/* eslint-disable */
/* generated by ... */

...

/* eslint-enable */

This would need to be implemented here (and in enum and in interface and in function)
https://github.com/microsoft/kiota/blob/ef8f5086a1adb6373c04f09ad0a9b35f7815b0e8/src/Kiota.Builder/Writers/TypeScript/CodeClassDeclarationWriter.cs#L20
As well as the closing equivalent

Assigning to myself and moving to 1.7 since @nonameolsson doesn't seem to want to submit a pull request.

Authored #3318 to address the concerns above.

@nonameolsson There's a follow-up issue at #3911, please upvote if you're still interested.

Not today no

anything in the roadmap?

yes https://github.com/microsoft/kiota/issues/3911


Fix support for Media Query syntax in StyleX shim

Describe the issue
The correct syntax for Media Queries in StyleX is not supported in the native shim. We only support a legacy syntax.
Expected behavior
See https://stylexjs.com/docs/learn/styling-ui/defining-styles/#media-queries-and-other--rules
Steps to reproduce
N/A
Test case
No response
Additional comments
No response

Should be straightforward — I can take this on


Add get user actions support for audit service

let administrators or people like CISOs get detailed information about user actions

Coverage decreased (-0.2%) to 55.121% when pulling bbcb6f3aa781b2c8e6974d5c9874b093efb17a9f on feature/audit-service-get-user-actions into 1e9bb952b3bccf6615359c17f0a7686821e0b566 on main.

when reviewing please note the TODO I left and let me know what you think? i would leave it as is for now and when the user starts waiting too long for the answer we can consider it if not already done, but meanwhile, I believe this is enough.


Muted posts can cause the notifications tab to become empty.

It is with great regret that I must report: I accidentally posted a banger post. In an attempt to reclaim my notifications, I muted the offending thread. What Bluesky appears to do is load the most recent notifications, see that the majority of them belong to the post I muted, and toss them away. The problem seems to start here.
Because I'm not currently receiving enough recent notifications from other posts to fill the tab, I'm unable to scroll the tab and tell Bluesky to load older notifications. So my notifications tab ends up looking more or less empty on most attempts to load it, and so I'm blocked from seeing notifications for other post if they happen to be too old to be loaded on the first attempt.
Screenshots

At the very most, I might see a few lines of notifications while the tab attempts to load more.

But eventually, it will settle on only the most recent notifications from the first page, and stop loading.
Details
Windows browsers, Firefox & Chrome.
The Android app has no problems here: It will continue to load notifications page by page until the screen is full, and then continue loading them as I scroll.

Additional Context
The last time I played myself by shitposting online, I ended up submitting #1974. That issue also deals with client notification loading behavior, and I don't think I'd be too far off the mark by suggesting that there's a deeper root cause. In this case, I'm assuming what's happening is that my first page of notifications is loaded, the client goes "these notifications belong to a post that is muted", and then as a result of the first page of notifications almost entirely belonging to a muted post, nearly everything on the page is thrown out.
The mobile client's behavior, continually loading notification pages until the visible tab is full, appears to be missing from the web client. That would solve the issue of having an empty tab. But it feels like the client is listening to notifications for all posts, regardless of whether they're muted or not, and then it decides what to do with them. Either the client should request the server to exclude muted posts, or the server should check whether a post is muted before delivering pages to the client.
In the meantime, I have to simply leave this post unmuted on desktop so that I can actually receive notifications.

This should happen less frequently now but it's still a fundamental issue with client-side filtering that we need to work out.


404 Using Postgres DB

I was able to successfully create a simple form using sqlite3 db. I switched to a Postgres db, ran python manage.py migrate with no problems and tried the same code and I come up with a 404 error.
Page not found (404)
Request Method:	POST
Request URL:	http://<IP_ADDRESS>:8000/en/forms/submit/
Raised by:	cms.views.details
Using the URLconf defined in jsg.urls, Django tried these URL patterns, in this order:
^media/(?P<path>.*)$
^static\/(?P<path>.*)$
^en/ ^admin/
^en/ ^sitemap\.xml$
^en/ ^select2/
^en/ ^ ^cms_wizard/
^en/ ^ ^(?P<slug>[0-9A-Za-z-_.//]+)/$ [name='pages-details-by-slug']
^en/ ^ ^$ [name='pages-root']
^en/ ^ ^forms/submit/$ [name='djangocms_forms_submissions']
The current URL, /en/forms/submit/, didn't match any of these.
I went back to the original default template and it is still the same problem.

As I understand it, you swapped db backend from sqlite to postgresql. How did you include form submission page urls?
Did you add djangocms_forms.urls to your project's urls, like this:
urlpatterns = patterns(
    url(r'^forms/', include('djangocms_forms.urls')),
)

or create a django CMS page to hook the application into?
From the traceback above I suspect its the latter scenario .. for that you'll need to recreate the page and add djangocms_forms as an AppHook

You're completely right that I didn't add the app hook. Thank you so much for your help and sorry for bothering you for such a silly mistake. Ramadan Kareem.

As I understand it.. you swapped db backends. How did you attach tthe

Don't forget to 🌟 this repo 😄


Add support for fastchess

made it generic so it can work with both cutechess and fastchess binaries

@gahtan-syarif can you update akimbo-main example along with the format? (run cargo fmt)

that should fix the formatting i think

I'm gonna follow this up with a commit that changes how this is done slightly, but otherwise lgtm.


Added redirection for AzPS 9.0.0 overview page

PR Summary
Added redirection for AzPS 9.0.0 overview page
PR Checklist

[x] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[x] Summary: This PR's summary describes the scope and intent of the change.
[x] Contributor's Guide: I have read the contributors guide.
[x] Style: This PR adheres to the style guide.

Learn Build status updates of commit 7ef6122:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Cannot order them

I would like to have this feature implemented to be able to order them later

Thank you for reaching out.
I have a similar need to organize habit lists, but I implemented it by using stars to prioritize habits:

Yes I saw that but that option will just push the to top ... I want to order them like drag them on the position I would like to see in the list
Thank you for your great work here

I see, will figure out a workaround to order them.

Hi @astingengo, a "secret" page was created to order habits: /gui/order :")

Please check v0.3.0 for this new feature, you can simply drag any habit to reorder it.
BTW, it's still an experimental feature. I will refine the page and integrate it into the main workflow in the future.

Hi, this feature has been released and is now available through the menu. Please don't hesitate to reply if you have any other questions.


Build conda step taking very long on build server

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior, code snippets encouraged
Expected behavior
A clear and concise description of what you expected to happen.
Info (please complete the following information):

SynapseML Version: [e.g. v0.17]
Spark Version [e.g. 2.4.3]
Spark Platform [e.g. Databricks]

** Stacktrace**
Please post the stacktrace here if applicable

If the bug pertains to a specific feature please tag the appropriate CODEOWNER for better visibility
Additional context
Add any other context about the problem here.
AB#1745597

Fixed with caching and pinning versions!


Update .gitignore

Mostly taken from https://github.com/achow101/btcinformation.org/blob/master/.gitignore

I added a number of those things to the original Bitcoin site file and I know they don't apply to this site, but I don't really see the harm in including them here, so utACK.


Update SublimeLinter-contrib-reek plugin

This update introduces better highlighting of code, introduced in this PR.
Repo link: https://github.com/codequest-eu/SublimeLinter-contrib-reek

You seem to have made a release, which automatically pushes to package control within an hours time. It seems to be version 1.1.0 which includes the PR.
If I missed something in this issue let me know, but I don't need to do anything to update
Thanks

You are correct, I was confused by the sentence "Then do the steps above with the new version." - I figured I had to create an issue. Sorry about that!

No problem, glad we figured it out. Thanks again!


Fix up default issue/PR templates

Right now, we have just the generic templates. I'm positive we can simplify these, and thus, not introduce needless friction on otherwise good ideas / good bug reports.

I promise I'll get to this!

Fixed!


cursor blinks and cannot type commands.

I have installed zsh and oh-my-zsh in wsl.

I hava the same issue.

install from scoop
first launch and default config
want to open a powershell terminal
nothing show except cursor

Please open Devtools from Settings - Devtools and capture console log at the time of the issue

Please open Devtools from Settings - Devtools and capture console log at the time of the issue

Actually, after times restart, the termianl works well, so maybe it's down just in the first few times

@Eugeny thanks for you help.

Could you please capture the log with a fresh nightly build?


fix: Error Handler soft media requirement

fix: Error Handler soft media requirement
remove tests

#1085  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Cobalt0s and the rest of your teammates on  Graphite


Combined manual trigger with time trigger

Would this do the trick on having a time based trigger as well as a manual trigger at the same time?

Thanks for your PR, but unfortunately that won't work: The read command blocks the further execution of the script until it finds a new-line character on stdin. So now we're basically always stuck in 'manual mode'.

I was thinking you could add a timeout -t to the read command. Also it might be good to shorten the sleep 60 a bit be ause the script itself takes time to run, lower the chanse you it misses a timed trigger.


Warn before leaving user edit page. Fix #1184

adds additional useEffect to EditUserInfo.js in Pages that tracks dataWasChanged, alerts if true.

Fixes #1184

Fixed DiscordID not properly being updated for the User.


yt-dlp sign in to confirm you’re not a bot.

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
EUA
Provide a description that is worded well enough to be understood
yt-dlp sign in to confirm you’re not a bot.
Even when changing VPS, changing server I always have this problem, has anyone managed to solve it?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp sign in to confirm you’re not a bot.

Duplicate of the issue that's pinned to the top of the tracker


Add files via upload

"Fixed" the dpi steps of the g300 to 50 to allow for finer adjustment.

Are you sure this is supported by the device?

I'm allowed to do this in the Logitech Gaming Software on windows and it
works so I don't see why not.
On Sat, Jul 25, 2020, 9:14 AM Filipe Laíns<EMAIL_ADDRESS>wrote:

Are you sure this is supported by the device?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/libratbag/libratbag/pull/1035#issuecomment-663854364,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRYR4JG3WWQPNFHUCJ6GJDR5LLE3ANCNFSM4PHMNWGQ
.


%%host%% interpolation not honored in conf.d

Describe what happened:
With configmap
data:
  conf: |-
    init_config:
    instances:
      - url: "http://%%env_NODE_IP%%:4001"
      - url: "http://%%env_NODE_IP%%:4002"

mounted as
            - mountPath: /conf.d/etcd.d/
              name: dd-agent-conf-masters
              readOnly: true

and
        - name: dd-agent-conf-masters
          configMap:
            name: dd-agent-conf-masters
            items:
              - key: conf
                path: conf.yaml

with env var set thusly
            - name: NODE_IP
              valueFrom:
                fieldRef:
                  fieldPath: status.hostIP

we receive the following error:
    etcd (1.6.1)
    ------------
      Instance ID: etcd:60991aae3a74c004 [ERROR]
      Total Runs: 3
      Metric Samples: Last Run: 0, Total: 0
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 2, Total: 6
      Average Execution Time : 89ms
      Error: HTTPConnectionPool(host='%25%25env_node_ip%25%25', port=4001): Max retries exceeded with url: /v2/stats/self (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fcd35e19d50>: Failed to establish a new connection: [Errno -2] Name or service not known',))
      Traceback (most recent call last):
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/base.py", line 387, in run
          self.check(copy.deepcopy(self.instances[0]))
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 105, in check
          self.check_pre_v3(instance)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 214, in check_pre_v3
          self_response = self._get_self_metrics(url, ssl_params, timeout, critical_tags)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 287, in _get_self_metrics
          return self._get_json(url, "/v2/stats/self", ssl_params, timeout, tags)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 308, in _get_json
          r = self._perform_request(url, path, ssl_params, timeout)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 303, in _perform_request
          url + path, verify=verify, cert=certificate, timeout=timeout, headers=headers(self.agentConfig)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py", line 75, in get
          return request('get', url, params=params, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py", line 60, in request
          return session.request(method=method, url=url, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py", line 533, in request
          resp = self.send(prep, **send_kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py", line 646, in send
          r = adapter.send(request, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/adapters.py", line 516, in send
          raise ConnectionError(e, request=request)
      ConnectionError: HTTPConnectionPool(host='%25%25env_node_ip%25%25', port=4001): Max retries exceeded with url: /v2/stats/self (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fcd35e19d50>: Failed to establish a new connection: [Errno -2] Name or service not known',))
      Warning: In Agent 6.10 this check will only support ETCD v3+. If you wish to preview the new version, set `use_preview` to `true`.

      Instance ID: etcd:7ec1f415e0b4fbbc [ERROR]
      Total Runs: 2
      Metric Samples: Last Run: 0, Total: 0
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 2, Total: 4
      Average Execution Time : 49ms
      Error: HTTPConnectionPool(host='%25%25env_node_ip%25%25', port=4002): Max retries exceeded with url: /v2/stats/self (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fcd35e19d90>: Failed to establish a new connection: [Errno -2] Name or service not known',))
      Traceback (most recent call last):
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/base.py", line 387, in run
          self.check(copy.deepcopy(self.instances[0]))
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 105, in check
          self.check_pre_v3(instance)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 214, in check_pre_v3
          self_response = self._get_self_metrics(url, ssl_params, timeout, critical_tags)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 287, in _get_self_metrics
          return self._get_json(url, "/v2/stats/self", ssl_params, timeout, tags)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 308, in _get_json
          r = self._perform_request(url, path, ssl_params, timeout)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/etcd/etcd.py", line 303, in _perform_request
          url + path, verify=verify, cert=certificate, timeout=timeout, headers=headers(self.agentConfig)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py", line 75, in get
          return request('get', url, params=params, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py", line 60, in request
          return session.request(method=method, url=url, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py", line 533, in request
          resp = self.send(prep, **send_kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py", line 646, in send
          r = adapter.send(request, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/adapters.py", line 516, in send
          raise ConnectionError(e, request=request)
      ConnectionError: HTTPConnectionPool(host='%25%25env_node_ip%25%25', port=4002): Max retries exceeded with url: /v2/stats/self (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fcd35e19d90>: Failed to establish a new connection: [Errno -2] Name or service not known',))
      Warning: In Agent 6.10 this check will only support ETCD v3+. If you wish to preview the new version, set `use_preview` to `true`.

Describe what you expected:
We expected the %%env_XXXX%% to expand the environment variable to get the node's IP address per https://github.com/DataDog/datadog-agent/pull/1339 . We also tried %%host%%  and got the same results. When hardcoding the node's IP address as a test, things worked as expected, but is untenable to deploy in our setup.
Steps to reproduce the issue:
See above.
Additional environment details (Operating System, Cloud provider, etc):
Kops gererated Kube cluster 1.10.x running in AWS, Datadog agent 6.9.0

Hey there,
Thank you for reaching out!
I think the issue here is that the template you are passing in your configmap is processed as a regular integration template.
Could you add the ad_identifiers field as follows:
https://github.com/DataDog/integrations-core/blob/master/etcd/datadog_checks/etcd/data/auto_conf.yaml#L1
?
With that, the agent should understand that it's supposed to process the template as an autodiscovery one and thus interpolate the template variables.
Let us know if that helps.
Best,
.C

Same problem, but how are you supposed to do this for the postgres integration since it doesn't match any containers.
## All options defined here are available to all instances.
      #
      init_config:

          ## @param service - string - optional
          ## Attach the tag `service:<SERVICE>` to every metric, event, and service check emitted by this integration.
          ##
          ## Additionally, this sets the default `service` for every log source.
          #
          # service: <SERVICE>

      ## Every instance is scheduled independent of the others.
      #
      instances:

          ## @param host - string - required
          ## The hostname to connect to.
          ## NOTE: Even if the server name is `localhost`, the Agent connects to PostgreSQL using TCP/IP unless you also
          ## provide a value for the sock key.
          #
        - host: psql-masterytrack-production.cyspzu9k3jsf.us-west-2.rds.amazonaws.com

          ## @param port - integer - optional - default: 5432
          ## The port to use when connecting to PostgreSQL.
          #
          # port: 5432

          ## @param username - string - required
          ## The Datadog username created to connect to PostgreSQL.
          #
          username: datadog

          ## @param password - string - optional
          ## The password associated with the Datadog user.
          #
          password: "%%env_DATADOG_PGPASSWORD%%"

this doesn't work, if I hardcode that password line it works fine.


should  mention about two different command in ReadTwitterFromEventHub

Hi team,
For the first try, I combine two block code of ReadTwitterFromEventHub into one and run, then only binary block tweet display. Then, I separate two blockcode of ReadTwitterFromEventHub into two different cmd. Then run each of them. Finally, I got it worked.
Thanks for excellence tutorial!
Huy,

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

ID: 4137b0cc-898b-b25c-6321-fe98f22f803d
Version Independent ID: 4ec1e187-7e69-e5da-b6dd-cb03b6db9366
Content: Tutorial: Stream data into Azure Databricks using Event Hubs
Content Source: articles/azure-databricks/databricks-stream-from-eventhubs.md
Service: azure-databricks
GitHub Login: @lenadroid
Microsoft Alias: alehall

@huyngo81 Thanks for the feedback. Could you please elaborate more if you have faced any issue while following the document? Also please confirm if no action is required, so that we will close this issue.

I think I'm not familiar with scala then facing minor issue like abive. No action require, close ussue.
Thanks,
Huy

@huyngo81 Thanks for confirming!


collection mode - collectionTileMargin:0 still has margin between images

Hi,
I'm trying to use collectionMode with multiple images and I want the margin between them to be zero or very small.  I'm currently using the most recent release (2.4) and I've done a lot of google searches about this issue and thought i was resolved with https://github.com/openseadragon/openseadragon/pull/446.
I've looked at the following examples and the problem does not exist:
https://bl.ocks.org/fabiovalse/7705d5acb1cdd8678d39a74031e07580
http://iangilman.com/osd/test/demo/m2/
So obviously I'm doing something wrong and I'm about to lose whatever hair I have left :(
I've tried passing x and y coordinates and these seem non-responsive no matter what I set it to.  Also, I'm using simple jpg images and not dzi if that makes any difference.  Any help/guidance is much appreciated.  Thanks.
Here is how I instantiate OSD:
var viewer = OpenSeadragon({
id:            "osd",
prefixUrl:     "/includes/js/openseadragon-bin-2.4.0/images/",
constrainDuringPan: true,
visibilityRatio: 1,
blendTime: 0.2,
zoomPerClick: 1.5,
nextButton:     "nextPage",
previousButton: "previousPage",
preserveViewport: true,
collectionMode:       true,
collectionRows:       1,
collectionTileSize: 256,
collectionTileMargin: 0,
tileSources: imageSource
});

Are your images square? If not, there will always be gaps between them using collectionMode, since it puts the images in a square grid, centering each image in its grid cell.
If that's not the issue, can you share a screenshot or a link?

I might add: the collectionMode arrangement is pretty naive... if you want something more relevant to your needs, you might want to build one yourself. Here's the code that collectionMode uses:
https://github.com/openseadragon/openseadragon/blob/master/src/world.js#L321
You basically just go through each of the images and set its position however you want.

Thanks for the quick response.  Okay, that explains it, my images are not squares.  It's like a portrait where the height is bigger than the width.  I will look into modifying the code in arrange function.  Just a quick question about the coordinates (x,y, width, height, position) used in the function, are these measured in px?

Good question! No, they are in viewport coordinates, which is an arbitrary coordinate system.
http://openseadragon.github.io/examples/viewport-coordinates/
That said, since it's arbitrary, you can define it however you want. Just use setWidth on your tiledImages and that's how wide they are in viewport coordinates.


Vercel AI SDK with Ollama - not for production

What is the issue?
The blog post about how Vercel AI SDK can be integrated with Ollama should be updated to include that it's NOT for production and only works locally.
Calls from NextJS api routes can't be proxied to localhost. When this route is called on a hosted instance, it runs on the server and can't access the Ollama running on a local machine.
What did you expect to see?
I expected to be able to host it through Vercel and still be able to hit my locally running Ollama. It doesn't state that this isn't possible and since the Vercel AI SDK is there, it implies for it to work.
Steps to reproduce
No response
Are there any recent changes that introduced the issue?
No response
OS
Windows
Architecture
No response
Platform
No response
Ollama version
No response
GPU
No response
GPU info
No response
CPU
No response
Other software
Vercel

Openai sdk can be accessed from local machine, remote machine (via python), and remote machine (curl). However, once it is hosted on the vercel, it failed.
Any clue or idea? I have already added it into my production on the vercel.

Openai sdk can be accessed from local machine, remote machine (via python), and remote machine (curl). However, once it is hosted on the vercel, it failed. Any clue or idea? I have already added it into my production on the vercel.

`import { CHAT_SETTING_LIMITS } from "@/lib/chat-setting-limits"
import { checkApiKey, getServerProfile } from "@/lib/server/server-chat-helpers"
import { ChatSettings } from "@/types"
import { OpenAIStream, StreamingTextResponse } from "ai"
import OpenAI from "openai"
import { ChatCompletionCreateParamsBase } from "openai/resources/chat/completions.mjs"
import { ServerRuntime } from "next"
export const runtime: ServerRuntime = "edge"
export async function POST(request: Request) {
const json = await request.json()
const { chatSettings, messages } = json as {
chatSettings: ChatSettings
messages: any[]
}
try {
const openai = new OpenAI({
apiKey: "any",
baseURL: ${process.env.NEXT_PUBLIC_OLLAMA_URL}/v1
})
console.log(messages)

const response = await openai.chat.completions.create({
  model: chatSettings.model as ChatCompletionCreateParamsBase["model"],
  messages: messages as ChatCompletionCreateParamsBase["messages"],
  temperature: chatSettings.temperature,
  max_tokens:
    CHAT_SETTING_LIMITS[chatSettings.model].MAX_TOKEN_OUTPUT_LENGTH,
  stream: true
})

const stream = OpenAIStream(response)

return new StreamingTextResponse(stream)

} catch (error: any) {
const errorMessage = error.error?.message || "An unexpected error occurred"
const errorCode = error.status || 500
return new Response(JSON.stringify({ message: errorMessage }), {
status: errorCode
})
}
}
`

Is this still a problem?  Does it require adjusting the OLLAMA_ORIGINS setting?
https://github.com/ollama/ollama/blob/main/docs/faq.md#how-can-i-allow-additional-web-origins-to-access-ollama

I'm going to close this, but we can reopen it if it's still a problem.


Add WellcomeHttpApp to http lib

This is duplicated between API services in storage & search - we should share it.

For https://github.com/wellcomecollection/platform/issues/5162


Orleans is suitable for this project

See https://docs.microsoft.com/en-us/learn/modules/orleans-build-your-first-app/4-exercise-create-grains-silos

This is great.


Improve CircleCI infrastructure

Things done in this PR:

Just specify node version 6, and let CircleCI default to the most recent version
Add very basic Jest tests to NavigationPlayground
Fix the existing Jest tests on ReduxExample
Run the example app tests during the CI

Tests are run in a post script after the main tests have passed, so this shouldn't slow that part down
Because of the extra tests, the total CI length does increase by a few minutes; if this becomes problematic, it could be solved by using more parallel builds, or a service like Travis which runs all tests concurrently by default

Remove LinkingExample entirely, because it's now part of NavigationPlayground

Can we get this branch rebased and conflicts resolved? Also, let's not include LinkingExample in the test pipeline, it was meant to be removed.
Actually I was planning to do it soon, but I think you could try including it in this PR.

@grabbou LinkingExample is not being tested. I'll remove it entirely!

Rebased again after changes to circle.yml. Should be good to merge.

Looks good to me. @ericvicenti that looks good to me, what you think?


TS-4719: Make AIO threads interleave memory across NUMA nodes.

(cherry picked from commit e5261f4d5edf557fe03571b46eacc61d430425e3)

Back port of #868

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1035/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1142/ for details.


Improve code folding

Moves code folding into a separate file, refactors the test existing cases for it and fixes issues mentioned in https://github.com/apple/sourcekit-lsp/issues/898.
Can be reviewed commit by commit
Fixes https://github.com/apple/sourcekit-lsp/issues/898
rdar://116877123

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test Windows


[terra-disclosure-manager] Prevent progressive disclosure from occurring if no key is provided during disclosure request

Feature Request
Description
The disclosure manager's disclose API expects a unique key to be provided alongside the component to be disclosed. This key is used for internal mappings and bounds checking within the stack of disclosed components.
We do not currently check for the presence of that key during the disclose request. If a key is not provided, the disclose request should be rejected, and the user should be warned in development mode.

Thought about this some more. I think we should remove the key from the API altogether and generate it internally. This will guarantee presence and uniqueness while making consumption easier for our consumers.  If we do this, then we wouldn't necessarily need a Major Version Bump; the key provided by consumers would just go unused.


iam: find roles by tags

Describe the feature
In order to find roles with certain tags one currently has to utilise aws_sdk_iam::client::Client:

list roles https://docs.rs/aws-sdk-iam/latest/aws_sdk_iam/client/struct.Client.html#method.list_roles
list_role_tags(&self) for each role

list_roles does not have a filter to only return roles matching a certain tag key value pair. This leads to a potentially high amount of API calls when many roles have to be checked.
Use Case
Example use case:
AWS RDS IAM based authentication does not allow to trace back a certain IAM role (from postgres perspective) to the user which assumed a role. A workaround is to create dedicated IAM roles for each user that wants to access the db via IAM auth. As IAM role names are limited one needs to utilise tags to store additional meta information such as the user's user id, the corresponding db etc.
Creating a program that needs to identify the role which has to be assumed is difficult without using tags.
Proposed Solution
Provide a filter parameter that allows filtering against a tag key value pair.
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

A note for the community

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

I created a feature request in the cross sdk repo here. Closing this issue


MapView.AnimatedRegion blank on real device

I've tried the sample code for MapView.AnimatedRegion then it worked well on Simulator but renders blank on real device.

Should be fixed in #1889


disable completion for specific cmdline eg: Manw

Why the feature is needed?

@Shougo I currently enabled cmdline completion using this snippet from README.
  cmp.setup.cmdline(':', {
    sources = cmp.config.sources({
      { name = 'path' }
    }, {
      { name = 'cmdline' }
    })
  })

However neovim is choking when completion commands like :Man I figured this is because man pages are usually a very long list. Is there anyway to disable cmdline completion say for example only on the Man command?

Hm..  You can use keyword_pattern for it.
https://github.com/hrsh7th/nvim-cmp/issues/821#issuecomment-1056160923

I'm not quite familiar with regular expression, could you give a sample.

I think it should work.
cmp.setup.cmdline(':', {
  sources = {
    {name = 'path'},
    -- Do not show completion for words starting with '!'
    {name = 'cmdline', keyword_pattern = [[\!\@<!Man\s]]},
  }
})


Added concurrent parsing feature for genbank files

This adds simple concurrent parsing for genbank files. I added two functions:

ParseConcurrent: Parses a mutli-genbank file into a channel
ParseFlatConcurrent: Parses a flat-genbank file into a channel

I also added an MD5 hash into the Sequence struct. This will be used in allbase to see if genbank files should be updated.

Resolved the naming issue (changed from MD5 to CheckSum). Now uses blake3. Additionally added this as a feature into the gff parser.
Interestingly enough, GFF actually produces the exact file (as hashed) as its input, meaning it is far more reliable than Genbank.


depresolve: Fix incorrect mapping for grpc import paths.

grpc imports paths do not follow the same pattern as the general google.golang.org/... case.
This is a quick and easy solution, to add a special case for grpc. A future investigation can see if it's viable to remove the special case optimization, or if it's still worth having it for performance reasons.

Sure, let's fix CI, but that's independent of the change in this PR. Merging because this improves correctness and is needed to resolve a downstream issue.

We've fixed CI with help from @chris-ramon. /cc @jpoler


Confirm if voting power can use uint or needs big.Int

Similar to the concern raised in #22.

It uses big.Int. See this, this and this.

It does, but it probably doesn't need to if we're willing to lose some precision.

Is there a benefit to losing precision? I do not see it currently

Smaller power table updates. E.g., if I can truncate to the nearest 32GiB, I can save ~4 bytes per entry. I can save 5 bytes per entry if I can limit power increments to 1TiB.
This will also let us limit the number of power table updates (as long as an update wouldn't cross over a 1TiB boundary).

Truncating power is a good design question which has occurred to us, but not been investigated in depth. I'll retitle this issue to reflect that, and add to our meeting agenda.

I think changing the go-f3 representation to big int right now is the safest thing to do. Even if we truncate, we might still want a bigint to future-proof it.

So, goals are are:

Small power-table diffs: small power values, fewer power updates.
Avoid "rewrite everything" updates to power tables stored, e.g., in bridge smart contracts.

One interesting issue will be, e.g., if power shrinks appreciably. We'd need some way to "add back" precision and probably want some way to avoid constantly changing how we truncate.

If we truncate the power by one byte (divide by 256) each time we reach some threshold, we can:

Remove precision in smart contracts by masking power values (avoids expensive state updates).
Add precision back by transmitting a single byte-array (least significant bytes for all powers). However, this update will be expensive to process.

I'd like to avoid having any unstable points where it's possible to repeatedly trigger steps 1 & 2, but that's going to require some hysteresis. I guess that's fine?

See also #5 , which will probably resolve this too

For completeness, I looked into scaling power to the range 0x0-0xFFFF based on the total fraction of power. The idea was to avoid "cliffs" where we'd need to suddenly "add back" precision.
Unfortunately:

By scaling this way, we get some epochs with > 600 power changes (over the course of 1000 epochs). Not unexpected, but unfortunate.
If we don't scale at all, we get 0-4 power changes over that same 1000 epochs.

So I'm going to discount scaling as a possible solution to reducing the size of the deltas, but it may still have a place...
Additionally, given the low frequency of power changes, I'm inclined to discard truncation as well (for now). If necessary, we can smooth out power changes in the Filecoin protocol itself at some later point.

So, current thoughts on scaling/truncation:

Represent power changes, new keys, etc. in terms of raw (unscaled, untruncated) power changes. Do NOT commit to these.
Commit to a full power table (unscaled/untruncated) for now.
Make consensus decisions after scaling all power to the range 0-0xFFFF. Specifically, truncate((0xFFFF * Power)/TotalPower). The cost of scaling power on the fly should be minimal and it'll make some future optimizations possible.

Specifically, I want to scale power for consensus decisions because it'll let us (eventually) commit to a scaled power table which:

Will be small (2 bytes per participant). This will make it a good fixed-ish size fallback in case we get large power swings.
May work better in zero knowledge proofs (fixed size)?


WIP: Add File-System Based Components Pack Generation Functionality

Summary
This PR intends to add a file system-based automated component registration mechanism to the shakapacker.
NOTE

This is still a work in progress PR; more details will be added later.

@justin808 Closing this PR. Work done here is moved to the react_on_rails gem.
Link to Pull Request https://github.com/shakacode/react_on_rails/pull/1455


[v9rc4] Fresh install with Atomik or Elemental theme fails

After install message just says "Installation failed" and "undefined"
348 tables were created in the database
Can we have an install log, to be able to check what went wrong
I assume it is the same error as here #9908

Was never able to reproduce this as described – but I can verify that installation seems fine in 9.0.2 and the latest develop, so I'm closing this.


[SPARK-45870][INFRA] Upgrade action/checkout to v4

What changes were proposed in this pull request?
The pr aims to upgrade action/checkout from v3(node16) to v4(node20) (point to v4.1.1 now).
Why are the changes needed?

After V4.0.0, the action/checkout default runtime is node20.

Node 16 has reached its end of life

https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @HyukjinKwon @dongjoon-hyun