MPEG-H support in ExoPlayer

MPEG-H (ISO/IEC 23008-3) support in ExoPlayer
The MPEG-H Audio system was already adopted by ATSC (ATSC 3.0 TV), DVB, TTA (Korean TV) and now is adopted by Brazilian SBTVD as the sole audio codec for Brazil's second-generation DTT called TV 3.0 (more details here: https://forumsbtvd.org.br/tv3_0/#panel-phase2). Since lots of TV sets in the Brazilian market run Android OS, I would like to request the support of MPEG-H in ExoPlayer. Is it underway? Is there a roadmap?
Best regards,
Carlos Cosme

https://github.com/androidx/media/tree/ts_mpegh_reader_patch
Done

Thanks for the feature request. There is already some work in progress to add parsing support for MPEG-H (see example linked above). Are you looking for any specific subset of MPEG-H features you would like to support in your app?

Thank you for pointing to the ongoing work on the parsing support for MPEG-H. We are trying to enable the capabilities according to the TV 3.0 standard, which include not only decoding but especially the user interactivity options and MPEG-H HDMI passthrough. I’m wondering how difficult it is use MPEG-H in media3 using the source software decoder available on GitHub (https://github.com/Fraunhofer-IIS/mpeghdec)

The initial support will likely involve an ExoPlayer extension published by Fraunhofer using the software decoder you linked. The changes we are making all ensure that such an extension can be easily written and used without any special forks of ExoPlayer. This setup should also support the interactive features and HDMI passthrough you mentioned.

Hi @cecosme,
We've added the MPEG-H extension to the library's main branch. Please note that it doesn't support the interactive features and HDMI passthrough you mentioned yet, but we're working on it and will keep you updated!

Hi @rohitjoins ,
This month MPEG-H HDMI passthrough was approved as a mandatory feature in the Brazilian’s TV 3.0 systems. Once you have any update regarding the support of this feature in the ExoPlayer please let me know. Thanks for the support!


Snap Fill - Resize snapped windows together

Summary of the new feature/enhancement
When I resize a snapped window, simultaneously resize any adjacent snapped window. This is part of snap assist in Win 10.  Would love to see this feature working with fancy zones.
Would love to see this implemented.

@srikanth003
thank you for your feedback.
We are tracking the feature in https://github.com/microsoft/PowerToys/issues/254
Closing this issue in favor of https://github.com/microsoft/PowerToys/issues/254
Thanks.


ThorVG: Broken SVG initial import, a reimport fixes it

Godot version
4.0.dev (9df9dc77a396bcdfe362365e9511de80c1e94fc0)
3.4.2 is not affected.
System information
Linux, Mageia 9 x86_64
Issue description
I noticed that when adding SVG images to a project (or forcing a reimport from scratch by deleting the .godot/ folder and their .import files), the images are broken:

These two should have been the Godot logo (two copies of the same logo with different names).
Note how they're also broken in the filesystem dock:

Hitting reimport for each SVG (possibly several times) seems to fix them (though not the thumbnails):

This might be specific to this SVG (icon.svg from the Godot repo), I haven't tried others to confirm.
Steps to reproduce

Copy this repo's icon.svg to a new Godot 4 project
Drag and drop on the 2D viewport to add it to a Sprite, or check it in the Inspector preview

Minimal reproduction project
SVGBrokenImport.zip

Here is a SVG that works fine in 3.4, but doesn't work at all in v4 no matter how often I click reimport

 ImageLoaderSVG can't create image.
  modules/svg/image_loader_svg.cpp:142 - Condition "p_image->is_empty()" is true. Returning: FAILED
  core/io/image_loader.cpp:65 - Error loading image: res://assets/UIpack_vector.svg
  editor/editor_file_system.cpp:1919 - Error importing 'res://assets/UIpack_vector.svg'.
  Failed loading resource: res://assets/UIpack_vector.svg. Make sure resources have been imported by opening the project in the editor at least once.

I can't reproduce this issue.


Implemented AutoCloseable

Implemeted AutoCloseAble for IoCTXStriper

Thanks for the code! Do you by any chance have Unit Tests which go with this piece of code?

@arnobroekhof: Ping? Did you see the previous message?

Sorry, did not see te message. I will push the test asap


panel macro ignores markup_contents if table_for presents

For example:
      panel "Children" do
        table_for(category.children) do
          column :name
        end

        markup_contents do
          h3 "Test"
        end
      end

will output only the table.
Expected behavior - output both: the table and the markup.

@smpallen99 maybe is there some insight, why so?

That's because of do_panel implementation: https://github.com/smpallen99/ex_admin/blob/master/lib/ex_admin/table.ex#L52

Yep, figured some recursion could fix that; after processing :table_for, remove that key from the map and call do_panel with remaining parts.

About to do a test on my existing app :)

@jwarlander would be great if u could add a test for this fix. Thanks... Steve

@jwarlander I would think your recursive approach might work. I'm just wondering what would/should happen if they have more than one table_for or markup_contents. Probably best if we make sure all are rendered, or we raise an appropriate exception.

My initial implementation just recurses until no remaining match,
concatenating the results along the way; the default catch-all clause
simply returns the result.
Will definitely add test(s) for it :-)
Right now, though, gotta get dinner ready before our daughter falls asleep
;-)
Den lör 14 maj 2016 18:25Steve Pallen<EMAIL_ADDRESS>skrev:

@jwarlander https://github.com/jwarlander I would think your recursive
approach might work. I'm just wondering what would/should happen if they
have more than one table_for or markup_contents. Probably best if we make
sure all are rendered, or we raise an appropriate exception.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/smpallen99/ex_admin/issues/112#issuecomment-219229361

@smpallen99 I think, the best option would be render all blocks specified inside panel macro regardless of their quantity.

I've added a quick and dirty fix to don't block associations stuff. But I'd like to end up with proper solution, that handle more than one table_for or markup_contents in any order.

Once I realized the HTML being built was kept in ETS, my recursion approach worked out well -- but the way the schema is passed (as a map, with :table_for or :contents as the key) doesn't allow multiple instances of each of those, just having one or both of them in either order..
  def do_panel(conn, %{table_for: %{resources: resources, columns: columns, opts: opts}} = schema, table_opts) do
    table(Dict.merge(table_opts, opts)) do
      # ...
    end
    new_schema = Map.delete(schema, :table_for)
    do_panel(conn, new_schema, table_opts)
  end
  def do_panel(conn, %{contents: %{contents: content}} = schema, table_opts) do
    div do
      content |> elem(1) |> Xain.text
    end
    new_schema = Map.delete(schema, :contents)
    do_panel(conn, new_schema, table_opts)
  end
  def do_panel(_conn, _schema, _table_opts) do
    ""
  end

If it were a keyword-style list instead, it could have multiples, and one could easily process it from beginning to end:
  def do_panel(conn, [{:table_for, %{resources: resources, columns: columns, opts: opts}}|tail], table_opts) do
    table(Dict.merge(table_opts, opts)) do
      # ...
    end
    do_panel(conn, tail, table_opts)
  end
  def do_panel(conn, [{:contents, %{contents: content}}|tail], table_opts) do
    div do
      content |> elem(1) |> Xain.text
    end
    do_panel(conn, tail, table_opts)
  end
  def do_panel(_conn, _schema, _table_opts) do
    ""
  end

Alternatively, the map key could be a string instead, and have some kind of unique identifier appended.. then you could match it like:
  def do_panel(conn, %{"table_for_" <> id, %{ ... }}, table_opts) do
    # ...
    new_schema = Map.delete(schema, "table_for_" <> id)
    do_panel(conn, new_schema, table_opts)
  end

It's not as elegant, but may be easier to implement depending on how the schema generation is organized (haven't looked yet).

I like a keyword list approach. This data structure fits perfect for this case.

I've got it working with keyword list.. will open a pull request, just going to sync against your latest changes :)

Oh, right.. you're on a separate branch; which one should I do a pull request against?

@jwarlander Give me a link to your branch. I'll cherry-pick commits from it to my branch and later (if all is ok) @smpallen99 will merge them all to ex_admin main repo.

You'll find it here:
https://github.com/jwarlander/ex_admin/tree/handle-multiple-panel-sections
It's just one commit - I haven't written any tests for it yet, as I just got it working. I should be able to get to that tomorrow though, once I figure out a good way to test. I'm hoping there will be some existing tests I can look at for inspiration ;)

Great! I like your commit. I'll merge it to my branch tomorrow.

@jwrlander. I think you can add a test or two in controller test. Simply add to one of the resources in test/support/admin_resources.exs. Should be pretty straight forward.

Done; pushed the test to the branch referred above :)
Parsing test failures when looking at HTML output would be a bit nicer with something like Floki, where you can pick out a section of the raw HTML and make a nested tuple / list data structure of it; but I'm not sure if it's worth adding a dependency for.

@jwarlander Looks good. I like your idea of using Floki. I've never used that package before. Would be a good addition. I'll look at on the next test I write that would benefit. Unless someone else beats me to it :).
I'll make sure all your stuff gets pulled in with @romul pull request which i'll be reviewing tonight.
Steve


Disallow unnamed (autonamed) migrations?

We could prevent autogenerated migration filenames, if we want:
https://adamj.eu/tech/2020/02/24/how-to-disallow-auto-named-django-migrations/

I like the idea of using django-test-migrations a lot better than overriding the management command or relying on commit hooks.
But I definitely think this is something we should do.

Good find, @Scotchester
It would also help just to spread knowledge oft he --name option. Worth a reference in https://github.com/cfpb/development/tree/master/standards?

If an automated check doesn't work for whatever reason, we can also add it to the "done" checklist in our PR template.

One minor request if we decide to implement this: can we make sure that devs can still use auto-named migrations when working locally? I often generate temporary migrations while working and only finalize when getting ready to commit/push up. We currently have some (admittedly hacky) logic that fails tox if there are missing migrations when run on Travis, but not when run locally -- perhaps we could do something similar with this check.


[Supporting react native web] Why stackNavigator no longer works with react-native-web after beta7?

Current Behavior
An error is thrown by PlatformHelpers return Promise.reject('Unsupported platform');
I am maintaining https://github.com/ericwooley/react-nativeish/ and trying to upgrade to RN 44, however, I am thwarted by this error. And I can't tell if it's necessary to throw it.
I have a branch with the error here. https://github.com/ericwooley/react-nativeish/tree/next
I have react navigation beta 7 working fine on web. Upgrading to 8+ throws this error.
Expected Behavior
I would propose that you do not determine whats available programmatically, but split packages.
EG react-navigation / react-navigation-web. Which allows react-native-web to use it as needed.
Your Environment
You can effectively create react-native-web.
For now I suppose I will have to create a .web.js version of every scene and create a different navigation setup for web.

software
version

react-navigation

react-native

node

npm or yarn

Not sure if that is a bug or error in react-navigation, those warnings are only shown on development mode, I mean by setting __DEV__: false by webpack.DefinePlugin and aliasing native navigators for the web  react-navigation': 'react-navigation/lib/react-navigation.js.
In a production environment those warnings are not triggered in the console.

Hi @ericwooley @agrcrobles,
in the past few months this lib has been through a rollercoaster, now we are getting it back on track but sadly in order to prioritize the stability/core features we had to put web on hold.

@kelset at least could you add partial support for web without tab navigator and other stuff
just stack navigator for v1


DatePicker spawns an AutoFill suggestion

https://jsfiddle.net/tj2Lo4z7/
Your Environment

flatpickr version used:  /* flatpickr v4.6.13,, @license MIT */
Browser name and version: #
OS and version:  IOS
On mobile, tapping into the DatePicker spawns an AutoFill suggestion at the top-left of the screen.

Can you add jsfiddle screenshot with the issue? I don't have an iOS device right now but I will check later.

Please also provide iOS version and browser name. I assume this is on safari?

Reproduced using https://jsfiddle.net/armujahid/p3d6yxu8/
This issue isn't related to datepickr but it seems to be happening for readonly input fields in latest iOS (Tested in 17). happens only if non-native picker is used (which seems to always the case in some modes like range )
Not sure if there is a way to disable that behavior since this behavior is native to iOS

I have posted this here https://stackoverflow.com/questions/78997215/how-to-disable-ios-autofil-and-share-button-on-tapping-readonly-input-field to get feedback of community related to that behavior. this seems benign and doesn't affect any actual functionality of datepicker so I don't think this is a blocker.

ok tks lets see if any response.

From: Abdul Rauf @.>
Sent: Wednesday, September 18, 2024 4:23 AM
To: flatpickr/flatpickr @.>
Cc: Mostly Marketing @.>; Author @.>
Subject: Re: [flatpickr/flatpickr] DatePicker spawns an AutoFill suggestion (Issue #3046)
I have posted this here https://stackoverflow.com/questions/78997215/how-to-disable-ios-autofil-and-share-button-on-tapping-readonly-input-field to get feedback of community related to that behavior.
External<EMAIL_ADDRESS>Report This Emailhttps://protection.inkyphishfence.com/report?id=bmV0b3JnMTEzOTk5OTYvZGFuQG1vc3RseW1rdGcuY29tL2JhMThiNzE4ZDQ1ZjczYWQ4MDI5MWU2ZDI4OTljY2VjLzE3MjY2NDc3OTAuODI=#key=dc9a7e90a35d44d76612211eba196726  FAQhttps://www.godaddy.com/help/report-email-with-advanced-email-security-40813  GoDaddy Advanced Email Security, Powered by INKYhttps://www.inky.com/protection-by-inky
I have posted this here https://stackoverflow.com/questions/78997215/how-to-disable-ios-autofil-and-share-button-on-tapping-readonly-input-fieldhttps://shared.outlook.inky.com/link?domain=stackoverflow.com&t=h.eJxNj0uOgzAQRK8SsZ6OYyfBkFWu0v4AFsZNcHuiaDR3H8xqWrUqlZ5e_zRli83j1EzMa34IkRntTN9-GyK9z5YW8So-c6CUhe76Xit5FxO9gQlcyGiih0AZsDANIQImB3nCzYMpzJRgD-O6hjTC5tFRih8IaS0MQ_DRNV-nZq4CyTNto5TXfr9WOEzPhTLHzzLzeIgYlJ3RsnO3-6Cv6LqL6qVvndq1rPVWSK3a9qZ1fzl3qoL9ASYOQ7B4_PAcA0_FVF4duDr4V_3-AbqLWcA.MEYCIQDwOMyGLCAlVz4b-iTWNCNOv2l4z2CvPUwKJpivMkT8AAIhAIcAybH4M6MpcS-65-SiImYS2NmKr5pkyWnd4mLPtoSB to get feedback of community related to that behavior. this seems benign and doesn't affect any actual functionality of datepicker so I don't think this is a blocker.
—
Reply to this email directly, view it on GitHubhttps://shared.outlook.inky.com/link?domain=github.com&t=h.eJxNjssOgyAQRX_F0G0rAsrDlb-CgEpUMDIumqb_XnFTZ3Vzc3LufNCxL6gt0ASwpRbj0cN09KWJKx4WDZs3835LPqXDJcyqmj-ufIKrC_CirBGSSMUIehZozsrgIO4jIUydx7HVoVtjguW9zjBeC70mshdE2roZBNNWVlQRxy2VShnjDCaCcl4LoapS0ix2lziCH7zR4GNI3f_jDNgM3KrvD2rJRw0.MEUCIQCydCLW4id5kk4VoAbIl5y0aBdRypYS_s7IqtCkHaRXYwIgZLNgJdX1352MwoYNOxj-jVZ0yhIap-PQnEGkXzXpgbA, or unsubscribehttps://shared.outlook.inky.com/link?domain=github.com&t=h.eJxNkEtzgjAAhP-Kw7k1JqEheBKEgvKSh4DcICAwCnQgHJxO_3uLl7rHnZ1vdvdbmMe7sF0JDedf0xaAuuXNXKzZ0IF-4O21ZTlvh34Ccz_NxcTGtqje85k3QLVNzdJPmJhRcBGDfRD43tnQslTPSKDEe_czdIiySPVUZDliEpsHpIVO7CeHFNtOLGIvtCPPRYnl45MbJ4mFo7PpZJrhHlNTuUS-d4wMXXhbCbelZF_xYawhxPKfCCjzftcNE78_uhuvn52LHNJCgrQUP64Szku6QTKsSImoLDNWMQAlRIgoSfJmTdECrp7g16m7_w-WQLkEXqyfX-wpXyg.MEUCIQDPtK9YjlQ744_9yeORN79ys5iW3U9GXa_CU9Agqrw_ZwIgGZ88iI-MljR6TAxUiCbfpzGpaV5K2d0edeGwaqp9hpw.
You are receiving this because you authored the thread.Message ID: @.***>


[WIP] Fix grammar and make clarifications

More changes incoming tonight/this weekend. I'll squash commits at the end and re-request reviews


Fix broken URL to Bundesanstalt für Wasserbau netCDF pages

Hi, I was studying UGRID towards some visualisation work I am conducting for the library cf-plot and I noticed there is a broken link under the 'Known Issues' page:

subgrid data; the NetCDF pages by the Bundesanstalt für Wasserbau (BAW) contain some proposals on this topic (see their pages (in German)).

where the link in question that doesn't work is: http://www.baw.de/methoden/index.php5/NetCDF which leads to a '404 Not Found' error.
Based on the redirection that links takes to a new domain wiki.baw.de (though to a broken path under that), I found the NetCDF pages which I believe should be linked to as the equivalent URL to the previous, updated to the new domain and path under it. So I thought I'd put in a PR to fix it, since itis a one-line change (though I also noticed a typo in the README I've included a fix for). Note I've included a change to https from http as the secure protocol for a URL, though this updated link works with eithel protocol.
Thanks.

I think we should switch to the English page, but otherwise, this looks fine.
Thanks Sadie!

Looks great, thanks!

@sadielbartholomew , @ChrisBarker-NOAA Thanks for keeping the UGRID documentation and issues up-to-date 👍


always set port for git proxy uri.

#670

@TingluoHuang Could you send me the agent for this please?

@yugmac the agent package is too big to send, we will do a releases with it within 1-2 days.

@TingluoHuang What was in the email you sent? I am trying to get the email released from quarantine.

@yugmac  the .dlls build from this branch that contains the changes.

I managed to get it released and arrive into my mailbox! I added the dll into the bin folder along with the other files however this time when running through the config.cmd I get an error message "Could not load type 'Microsoft.TeamFoundation.DistributedTask.WebApi.TaskAgentPoolActionFilter' from assembly 'Microsoft.TeamFoundation.DistributedTask.WebApi, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'." when at the Register Agent stage.

@yugmac let me publish a new release with this fix.

@TingluoHuang Yeah that will be great! Thanks :+1:


istio: fix istio idempotent

Signed-off-by: Xieql<EMAIL_ADDRESS>What type of PR is this?
/kind bug
What this PR does / why we need it:
1 There is no kind in the return value of Casecret, which will cause an error when checking the type later, so a check is added.
2 After the createIstioOperator step has been performed, reinstalll will add an additional “{install.istio.io/v1alpha1 IstioOperator nil}”, causing the update to fail. Since the crd will be created later, so it is skipped here.
After these fixes, it has been tested that after the istio is forced to exit at any step, the subsequent reinstallation can be installed normally.
Which issue(s) this PR fixes:
Fixes #88
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

/lgtm


Test day tooling (OpenShift)

We're having regular test days but the process of standing up clusters for testing is manual (except KIND based), it could be beneficial to develop some automated tooling (eg some scripts or a binary) that will do a lot of the manual work for standing up the clusters.
E.G. to stand up a test env on AWS one needs to:

Deploy a couple of clusters on AWS, using the desired OpenShift version
Alter kubeconfigs for properly working with multiple clusters
Prepare the cloud for submariner deployment
Deploy submariner properly (e.g. globalnet if clusters were deployed with overlapping IPs, specific cable driver is needed)
Run subctl verify
Open an issue labeled with the testday label with subctl gather output in case of fasilure (of the verify or manual testing)

Seems like most of these steps can be rather easily automated in one or a few commands which will reduce user-errors and streamline test days. It also seems that these steps will be rather similar for different clouds when deploying with OpenShift.
This might also ease the testing process, thus encouraging participation in the test days.

I use the following Makefile to deploy on AWS, run cloud-prepare, and fix up the kubeconfigs:
deploy:
	for c in h 1 2; do \
		if [ ! -d awsocp$${c} ]; then \
			cp -a awsocp$${c}-template awsocp$${c}; \
			openshift-install create cluster --dir=awsocp$${c} > awsocp$${c}.log 2>&1 & \
		fi; \
	done
	tail -f awsocp?.log

cloud-prepare:
	for c in h 1 2; do \
		KUBECONFIG=awsocp$${c}/auth/kubeconfig subctl cloud prepare aws --ocp-metadata awsocp$${c}/metadata.json; \
	done

kubeconfig:
	for c in h 1 2; do \
		yq -i '.contexts[0].context.user = "admin-'$$c'" | .contexts[0].name = "admin-'$$c'" | .users[0].name = "admin-'$$c'" | .current-context = "admin-'$$c'"' awsocp$${c}/auth/kubeconfig; \
	done

clean:
	for c in h 1 2; do \
		[ -f awsocp$${c}/metadata.json ] && openshift-install destroy cluster --dir=awsocp$${c}; \
	done
	rm -rf awsocp?

Issue moved to submariner-io/shipyard #912 via ZenHub


Clicking "Sign up" with no email address shows the disposable email error

When clicking "Sign up" with no email address, an error saying that you can't use disposable emails is shown.

I believe it sholuld instead leave the validation to Flarum core.

Duplicate of #4

Addressed in v1.0.0


 It
    has the advantage of abstracting away this tangental concern
    for people that ramp-up on Akka rather than on Docker.

let me know if you are fine with the changes.

Thank you very much for this improvement, Vasile. That's very kind of you. I left a couple of suggestions you could follow. Just nitpicking. I hope you don't mind.

Thank you for the suggested improvements, I've merged them. JIC I've double checked that docker compose up starts with no issues and that the init scripts are executed. All is well ✅ 👍🏽
...
postgres-betting-db-1  | 2024-06-24 22:13:59.593 UTC [1] LOG:  starting PostgreSQL 16.3 (Debian 16.3-1.pgdg120+1) on x86_64-pc-linux-gnu, compiled by gcc (Debian 12.2.0-14) 12.2.0, 64-bit
postgres-betting-db-1  | 2024-06-24 22:13:59.593 UTC [1] LOG:  listening on IPv4 address "<IP_ADDRESS>", port 5432
postgres-betting-db-1  | 2024-06-24 22:13:59.593 UTC [1] LOG:  listening on IPv6 address "::", port 5432
postgres-betting-db-1  | 2024-06-24 22:13:59.606 UTC [1] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
postgres-betting-db-1  | 2024-06-24 22:13:59.623 UTC [227] LOG:  database system was shut down at 2024-06-24 22:13:59 UTC
postgres-betting-db-1  | 2024-06-24 22:13:59.634 UTC [1] LOG:  database system is ready to accept connections
...

Thank you very much for your contribution @vasigorc :)


Get-SPOFolderItem and an SPO PSProvider

I often find that I need to know what files and folders I have in my site and a PowerShell cmdlet would come in handy. So wrote the Get-SPOFolderItem command that lists folder and file at a given path.
However even better would be navigate the site as a regular file store and be able to use commands like Get-Item, Get-ChildItem, Set-Location, Set-Content and so on. So I wrote a custom PowerShell (Drive) Provider that allow just that.
A drive can be created with the -CreateDrive switch at Connect-SPOnline, which default creates a drive named spo:, or via the regular New-PSDrive command.
Most regular provider commands are supported, thou right now copy/move actions between sites is not supported, but piping Get-Content and Set-Content can accomplish the same thing.
To be able to show File and Folder object in the same output a new format-section was needed that handles both object at the same time (which unfortunately will take over the output format for single file/folder object as well).

What a GREAT addition, Mårten!


cluster: increment idcounter immediately after create pod

@xiang90

ref: https://github.com/coreos/etcd-operator/issues/234#issuecomment-260484171

I actually feel the current code is better. we should increase the id counter in start/add (which is etcd focused) func not in the pod create path (which is k8s focused).

@xiang90
I actually feel the code is cleaner this way. We are putting the create logic together here.
Another way we can better handle this is to have id counter incremented every time.

@xiang90
Can we move forward with this?

I still dislike this idea. CreatePodAndService should not modify the id counter.

You can add a func called newName or something like that to create the name and increase the counter. Just do not hide it into a k8s focused func.

It's "cluster.CreatePodAndService()", not "k8sutil. CreatePodAndService()".
Does that make more sense?
Not putting idCounter++ immediately after kubecli.Pod().Create is not a good idea too

Does that make more sense?

No. It does not. cluster.CreatePodAndService is still just a k8s wrapper. There is no reason to hide id counter into that function at all. What we should do is to increase that counter when a new name is ever generated.

What we should do is to increase that counter when a new name is ever generated.
OK. This is my another proposal. I think that would be safer.

Closing this right now.


fix(badges): add badges to readme

Status
READY
Description
Looks like this on your README.MD now

Thanks


Betaflight cant connect over WiFi

First of all, many thanks to @rtlopez for a great project!
I try to connect Betaflight to esp32 over WiFi, but receive error Failed to open serial port.
In serial console all work perfect:

# status
ESPF ESP32 v0.0.0 0000000 Jan 22 2024 15:11:44 8.4.0 201103
STATUS:
    cpu freq: 240 MHz
  gyro clock: 8000 Hz
   gyro rate: 2000 Hz
   loop rate: 250 Hz
  mixer rate: 250 Hz
  accel rate: 500 Hz
   baro rate: 83 Hz
    mag rate: 75 Hz

 gyro device: MPU6050/I2C
 baro device: BMP280/I2C
  mag device: HMC5883L/I2C
     rx rate: 43
     rx lpfs: 0.00, 0.14

 arming disabled: 16777222

Also, Betaflight configurator work fine over COM port.
To connect over WiFi I set in COM port console SSID, password, wifi_mode to 1. After save and reboot. On this step, esp32 present in router clients list. Also, I can ping it. In betaflight configurator I select Manual selection on Welcome page. In port I enter esp32 ip adress: tcp://<IP_ADDRESS>. And receive described above arror.
Betaflight configurator ver. 10.8 and 10.9 were tried.

You need to specify port numer with address, by default it is 1111: tcp://<IP_ADDRESS>:1111


Heatmap from a table

People often want tables of numbers with heatmap style colouring on. Produce a generic solution for this using something like {reactable} or {kableExtra}

Note that this is done for discrete (RAG) categories here https://github.com/nhs-r-community/demos-and-how-tos/tree/main/conditional_table_formatting

There are now two solutions to this problem https://github.com/nhs-r-community/demos-and-how-tos/tree/main/conditional_table_formatting (although more are possible :smile: )


newsequentialid is broken in mssql-server-linux

I am using mssql-server-linux in combination with Entity Framework where the entities have Guid as Primary Key with DatabaseGeneratedOption.Identity option. This results in a default constraint on the primary key columns with newsequentialid() as default value/function.
However, newsequentialid() is generating duplicate values in mssql-server-linux, and that results naturally in PK violation.
P.S.: In the meantime, is there any workaround in this situation?

Have you tried newid() instead of newsequentialid()?

It seems that NEWID is your only option until this bug is fixed in EF. If you are open to using other data types for your PK you could do an int/bigint instead.

I am confused... Why do you mention EF? Did you mean Entity Framework? This issue has nothing related to Entity Framework, it's plain MSSQL for linux.

I see.  My bad.  Does NewID() work?
I'll check on the status of this bug.

Yes, newid() works correctly afaik.

Is there anything new to add to this issue?

OK good to hear.  Closing this out then.


workspacePrompt doesn't seem to work with fuzzy matching

Problem Description
It seems I can't make workspacePrompt working with fuzzy matching.
Upon bringing up the prompt I expect it to behave as described in the XMonad.Prompt.FuzzyMatch module, instead a match is not found unless the entered text is a prefix of the workspace's name.
Configuration File
N/A
The relevant parts are:
Prompt config:
myXPConfig :: XPConfig
myXPConfig = def { font            = "xft:PRagmataPro:pixelsize=32"
                 , height          = 39
                 , bgColor         = black
                 , fgColor         = white
                 , searchPredicate = fuzzyMatch
                 , sorter          = fuzzySort
                 , alwaysHighlight = True
                 }

Bindings:
       , ("M-S-g", workspacePrompt myXPConfig (windows . W.greedyView))
       , ( "M-S-f"
         , windowPrompt myXPConfig Goto allWindows
         )

I included the windowPrompt one as I confirmed that one behaves as expected.
When I have I will try and create a minimal reproducible example. Please feel free to close this until a minimal example is provided if that's a hard requirement.
Checklist

[ ] I've read CONTRIBUTING.md

[ ] I tested my configuration with xmonad-testing

I reduced my config to:
module Main where

import           Data.Map                       ( Map )
import           System.Exit
import           XMonad
import           XMonad.Prompt.Workspace
import           XMonad.Actions.DynamicWorkspaces
import           XMonad.Prompt.FuzzyMatch
import           XMonad.Config.Desktop
import           XMonad.Hooks.EwmhDesktops
import           XMonad.Prompt
import qualified XMonad.StackSet               as W
import           XMonad.Util.EZConfig

myWorkspaces :: [String]
myWorkspaces = map show ([1 .. 5] :: [Integer]) ++ [ "foobar" ]

myXPConfig :: XPConfig
myXPConfig = def { font            = "xft:PRagmataPro:pixelsize=32"
                 , height          = 39
                 , bgColor         = black
                 , fgColor         = white
                 , searchPredicate = fuzzyMatch
                 , sorter          = fuzzySort
                 }

myKeymap :: XConfig Layout -> Map (KeyMask, KeySym) (X ())
myKeymap c =
  mkKeymap c
    $  [
        ("M-S-c"    , kill)
       , ("M-S-n", appendWorkspacePrompt myXPConfig)
       , ("M-c"  , removeEmptyWorkspace)
       , ("M-S-g", workspacePrompt myXPConfig (windows . W.greedyView))

       , ("M-<Tab>"   , windows W.focusDown)
       , ("M-j"       , windows W.focusDown)
       , ("M-k"       , windows W.focusUp)
       , ("M-m"       , windows W.focusMaster)
       , ("M-<Return>", windows W.swapMaster)
       , ("M-S-j"     , windows W.swapDown)
       , ("M-S-k"     , windows W.swapUp)
       , ("M-S-q"     , io exitSuccess)
       , ( "M-q"
         , restart "xmonadrc" True
         )
       ]
    ++ [ (m ++ i, windows $ f j)
       | (i, j) <- zip (map show ([1 .. 9] :: [Integer])) (XMonad.workspaces c)
       , (m, f) <- [("M-", W.greedyView), ("M-S-", W.shift)]
       ]
    ++ [ (m ++ key, screenWorkspace sc >>= flip whenJust (windows . f))
       | (key, sc) <- zip ["w", "e", "r"] [0 ..]
       , (m  , f ) <- [("M-", W.view), ("M-S-", W.shift)]
       ]

main :: IO ()
main = launch $ ewmh defaults

white :: String
white = "#fbf8ef"

black :: String
black = "#383a42"

defaults =
  desktopConfig
        {
         workspaces = myWorkspaces
         , keys               =  myKeymap
        }

and the issue is still reproducible.

OK, NVM, this has been fixed in 28b3e34fd77c19e6b935c01a40e66cbf90abf69a Sorry for the noise.


Support for out-of-line CFFI modules

Out-of-line CFFI modules appear differently to inline ones - they are all of type builtin_function_or_method. This PR allows registering an out-of-line CFFI module with Numba so that it can build a mapping of types for all functions in the module, and therefore compile calls to them in jitted functions.

I've updated this to work with user-defined functions as per your patch, and added a test calling the foo function.


fix bug with change flag

Fixes #
Proposed Changes
Currently change flag in kafka_lib_topic may be set to False even though it was already set to True
To fix this, let the change flag remain true by:
changed = changed or len(topics_changed) > 0

Thank you, looks like a good catch. Just need to check if we can add some testing around this too.

Hi Stephen. Again thank you for your work on this so usefull ansible modules.
Do you think about merging this fix ?


Mokeypatching visdom

Hi,
I'm trying to patch visdom to take in a name variable on instantiation, but I keep getting an error. This is probably a stupid question, but what is the proper way to do this?
class Dashboard(Visdom):
    def __init_(self, name):
        super(Dashboard, self).__init__()
        self.env = name

Traceback (most recent call last):
  File "/home/user/anaconda3/envs/deep-learning/lib/python3.6/site-packages/visdom/__init__.py", line 220, in _send
    data=json.dumps(msg),
  File "/home/user/anaconda3/envs/deep-learning/lib/python3.6/site-packages/requests/api.py", line 110, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/home/user/anaconda3/envs/deep-learning/lib/python3.6/site-packages/requests/api.py", line 56, in request
    return session.request(method=method, url=url, **kwargs)
  File "/home/user/anaconda3/envs/deep-learning/lib/python3.6/site-packages/requests/sessions.py", line 488, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/user/anaconda3/envs/deep-learning/lib/python3.6/site-packages/requests/sessions.py", line 603, in send
    adapter = self.get_adapter(url=request.url)
  File "/home/user/anaconda3/envs/deep-learning/lib/python3.6/site-packages/requests/sessions.py", line 685, in get_adapter
    raise InvalidSchema("No connection adapters were found for '%s'" % url)
requests.exceptions.InvalidSchema: No connection adapters were found for 'seq-to-seq-experiment:8097/events'

Ha, I missed an underscore in the function name.


Feature/refactor inventory demo

Codecov Report

Merging #276 (e644ce7) into master (5181d68) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #276   +/-   ##
=======================================
  Coverage   96.37%   96.37%
=======================================
  Files         175      175
  Lines        4003     4004    +1
=======================================
+ Hits         3858     3859    +1
  Misses        145      145

Impacted Files
Coverage Δ

.../jamtemplate/common/tilemap/tilemap_cache_null.cpp
100.00% <ø> (ø)

impl/jamtemplate/sfml/text.cpp
100.00% <ø> (ø)

...l/jamtemplate/common/box2dwrapper/box2d_object.cpp
100.00% <100.00%> (ø)

...late/common/box2dwrapper/box2d_world_interface.hpp
100.00% <100.00%> (ø)

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


On Wayland, app repaints every frame if IME output is set

Describe the bug
On Linux with Wayland, when an egui app using eframe has ui.output(|o| o.ime) not equal to None, the app continuously repaints.
One example of when ui.output(|o| o.ime) is not None is when an egui::widgets::TextEdit is focused because it is set by the widget in order to allow IMEs to determine where the cursor is.
I haven't tested this on non-Linux machines, but WebAssembly apps in browsers on Linux don't seem to have this problem, nor do apps for Windows running in Wine on Linux. Linux with X11 doesn't have this problem either.
To Reproduce
Steps to reproduce the behavior:

Clone egui and checkout to the 0.27.0 tag
cargo run -p egui_demo_app
Make sure "Backend" is selected in the top-left corner of the app so that the left pane is visible
Set the rendering mode to "Reactive" in the left pane so that the app doesn't repaint unless it has to
Focus the text edit in the widget gallery or any other text edit
Observe that the "Mean CPU usage" in the left pane constantly updates, meaning the app is repainting every frame

Expected behavior
The app should not repaint every frame when the text edit is selected.
Desktop (please complete the following information):

OS: Arch Linux with GNOME 46
Version: Linux unknown 6.8.1-zen1-1-zen #1 ZEN SMP PREEMPT_DYNAMIC Sat, 16 Mar 2024 17:15:23 +0000 x86_64 GNU/Linux

I don't see anything wrong with egui that would be causing this to happen. Rather, it's being caused by this call here to the function winit::window::Window::set_ime_cursor_area:
https://github.com/emilk/egui/blob/4d4cb3d20db0517e74ca6ab6d3b646f78fd8123b/crates/egui-winit/src/lib.rs#L819-L833
The Wayland implementation of that function is somehow triggering egui apps to repaint every time it's called. I'll try to get this fixed in winit.


I have a Mayflash Magic S Pro to make DS4 work, can I use DS4Windows?

I just want to see my battery life of the controller and if possible change the amount of time I have to hold PS button to turn off controller from 10 seconds to like 5 seconds.

#654


JS http module causing subo build to fail (macOS only?)

When re-creating Flaki's subtoot project, repo for my copy, I came upon the error.
Commenting out use and importing of the http module from @suborbital/runnable allows the compilation to work, so I suspect there's a problem with this module.
Tested on macOS 12.14 with M1.
Error:
ℹ  🐳 using Docker toolchain
⏩ START: building runnable: sendtoot (javascript)

> prebuild
> webpack

asset index.js 11.2 KiB [compared for emit] [minimized] (name: main) 1 related asset
runtime modules 670 bytes 3 modules
modules by path<EMAIL_ADDRESS>9.06 KiB
<EMAIL_ADDRESS>1.29 KiB [built] [code generated]
<EMAIL_ADDRESS>183 bytes [built] [code generated]
<EMAIL_ADDRESS>703 bytes [built] [code generated]
<EMAIL_ADDRESS>437 bytes [built] [code generated]
<EMAIL_ADDRESS>595 bytes [built] [code generated]
<EMAIL_ADDRESS>2.01 KiB [built] [code generated]
<EMAIL_ADDRESS>598 bytes [built] [code generated]
  + 4 modules
modules by path<EMAIL_ADDRESS>10.2 KiB
<EMAIL_ADDRESS>8.96 KiB [built] [code generated]
<EMAIL_ADDRESS>1.25 KiB [built] [code generated]
./src/index.js 1.2 KiB [built] [code generated]
<EMAIL_ADDRESS>1.09 KiB [built] [code generated]
webpack 5.73.0 compiled successfully in 852 ms

> build
> javy build/index.js -o sendtoot.wasm

thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: Uncaught ReferenceError: 'TextEncoder' is not defined
    at 445 (script.js)
    at r (script.js)
    at 427 (script.js)
    at r (script.js)
    at 259 (script.js)
    at r (script.js)
    at 169 (script.js)
    at r (script.js)
    at <anonymous> (script.js)
    at <anonymous> (script.js)
    at <anonymous> (script.js)
    at <anonymous> (script.js)
    at <eval> (script.js)
', crates/core/src/lib.rs:228:61
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Error: the `wizer.initialize` function trapped

Caused by:
    wasm trap: unreachable
    wasm backtrace:
        0: 0xb40c7 - <unknown>!__rust_start_panic
        1: 0xb3e8c - <unknown>!rust_panic
        2: 0xb3af9 - <unknown>!std::panicking::rust_panic_with_hook::h94d61e41dc3cb549
        3: 0xb314c - <unknown>!std::panicking::begin_panic_handler::{{closure}}::hfb2a7b493f925953
        4: 0xb309b - <unknown>!std::sys_common::backtrace::__rust_end_short_backtrace::h9529007da4de429b
        5: 0xb3647 - <unknown>!rust_begin_unwind
        6: 0xc27fd - <unknown>!core::panicking::panic_fmt::h8a189464cad24b3b
        7: 0xc36df - <unknown>!core::result::unwrap_failed::hc9e17ac6ec77d8f1
        8: 0x45ed - <unknown>!wizer.initialize
        9: 0xc78f1 - <unknown>!wizer.initialize.command_export
    note: using the `WASMTIME_BACKTRACE_DETAILS=1` environment variable to may show more debugging information

Error: Couldn't create wasm from input
npm notice
npm notice New minor version of npm available! 8.11.0 -> 8.12.2
npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.12.2>
npm notice Run `npm install -g<EMAIL_ADDRESS>to update!
npm notice
Error: failed to BuildWithToolchain: 🚫 failed to build sendtoot: failed to RunInDir: failed to Run command: exit status 1
Usage:
  subo build [dir] [flags]

Flags:
      --builder-tag string   use the provided tag for builder images
      --docker               build your project's Dockerfile. It will be tagged {identifier}:{appVersion}
  -h, --help                 help for build
      --langs strings        build only Runnables for the listed languages (comma-seperated)

The issue here is rooted in the honest mistake of Ramón accidentally overwriting the wrong file in the JS runnable template generated by subo. Instead of putting the runnable code in lib.js, the code overwrites the "magic" boilerplate in index.js that we use to instrument the imports and load the TextEncoder polyfill missing from the javy engine.
With the above, this is not a "bug" in subo build, per se, but I'd argue there are various things we could (and should) improve here, and I would propose that we rename this issue to track the improvement of subo build error messages, and file a new issue for reducing the magic boilerplate and documenting the Reactr JS API boundary to demistify the interaction between index.js-lib.js.
If I read the above error message correctly there are two issues here (and a third potential one from experience working with JS runnables that should also be addressed).

the javy build fails due to a parse-time JS reference error (our HTTP API relies on the TextEncoder but we fail to import the polyfill), this triggers a build failure in javy which is a Rust binary and we are seeing the backtrace of the Rust panic
we use wizer to preinitialize/snapshot the generate wasm runnable for faster startup times, it seems like we are not picking up on the build failure and still trying to run wizer on the non-existent wasm output

A third issue that is not triggered here is a preceding step, even before we run javy we use webpack to bundle all the code into a single JS file that we compile afterwards. This step can also fail with mysterious error messages.
Note that these are all compile-time issues and runtime errors are very cryptic and hard to debug themselves, but that DX issue should be handled and improved on separately.

Thank you so much for investigating this!!


Ignore expected warning during kubernetes get_job test

This PR adds an ignore flag to the warning that the kubernetes output directory is a local path and that output data will not be available, since the use of a local path is by design in this test (and we're not trying to get output data). This PR returns the fv3config tests to a state where no warnings should be printed when they run.
I'd consider this change too small to document in the HISTORY.rst.

It could save someone a lot of trouble at some point, we should revisit removing it at some point in the future if it shows up a lot in cases where it doesn't matter.


Align API path endpoints with specification

This aligns the newly defined openapi specification with the naming used in the specification document.  The specification document inconsistently uses capitalization and underscores in the naming of the API endpoint paths, but based on the conformance tests and the specific sections of the documents that deal with each of the individual endpoints it seems clear that the intent is for them to start with "o2ims-" rather than "O2ims_".

/lgtm
/approve


DB/1 Store suggested sentences as separate entries with separate ids (all corresponding to the same sentence id)

Currently, all suggested sentences are under one id. In JSON file, they appear as just one entry, comma-spearated. This is not ideal for further data processing.
We want to store the suggested sentences as separate entries (a sentence per JSON line) with separate ids.

PR #38

While there are some issues with this PR I will commit it as it includes a couple important architectural changes that should be in main. The ticket should be kept open to fix a couple remaining issues as a new PR:
We still need a good database setup script. We have the schema file now, but it is not runnable (if you just pass it into postgres it will throw syntax errors), and is missing setup of the seed data (the initial sentences)

We should be using IDs throughout. There are two places where this becomes important:

In the selected_suggestion column we still store the full suggestion. This should be the id of the relevant entry in thesuggestion table
We send the full sentence to the frontend but not it's ID. Then, when we click a suggestion we send the full sentence back and ask the Database to find it's ID. What we should do is send both the sentence AND it's ID to the frontend, and the frontend should send BOTH back. The database can then look up the sentence by ID (and verify it is correct by matching it to the sentence). This will be still safe, but also performant. (Note that sending back the full sentence is not actually technically necessary, the ID is enough for most purposes)

When getting a sentence from the database we query the entirety of the database and selecting a random one from that. Postgres is clever enough to have features (please have a look on your own) to provide you with a single random sentence - use that feature. The current implementation would lose performance once the sentence database gets huge.


Problem with import workbook from code

Bicep version
Bicep CLI version 0.4.613 (d826ce8411)
Describe the bug
I created some workbooks in the portal, when I try do the same steps from I have error code:
{"error":{"code":"InvalidTemplate","message":"Deployment template language expression evaluation failed: 'The language expression length limit exceeded. Limit: '81920' and actual: '127368'.'. Please see https://aka.ms/arm-template-expressions for usage details.","additionalInfo":[{"type":"TemplateViolation","info":{"lineNumber":0,"linePosition":0,"path":""}}]}}

To Reproduce
Create some larger workbook in the portal a try create the same workbook in the bicep

Duplicate of #4293


Add ',' separator for currency

Hey! Love the library - I was wondering how hard it would be to add a comma separator for the currency input. I understand that this would be more of a dynamic thing, but I'm curious where you would start on adding something like this?

@tapdancemobile Hey. I think about formatted input as well and will try to implement it with "Affine masks" (https://github.com/RedMadRobot/input-mask-ios#affine-masks"), but I'm not sure yet it would solve the issue.
Here is a good library for React (https://github.com/s-yadav/react-number-format) that I'd like to implement for RN. Any ideas/pr are welcome :)

@ivanzotov - Thanks! I am currently having trouble working with the masks as they are static and inline currency formatting requires dynamic insertion of the commas. I have a RN solution for inline formatting - my issue is that the input blink and it is not smooth as all the updates are async in RN. That is why I am looking for a native solution...


Write ndt7 blog post

Once https://github.com/m-lab/mlab-speedtest/issues/10 is complete, it would be nice to write an accompanying blog post saying more about ndt7.
@bassosimone will do that and someone else will review (we'll figure out that later).

I'm a bottleneck here, so better to remove myself.

Assigning my self to this one. We need to get a post out soon about the current ndt, and what ndt-server will provide, and how things will be different in both testing and data.

Calling this one done with two recent posts.


Improve interface for changing local Config Path

The intended behavior is evidently for editing the text at Project Settings > Addons > Project Builder > Config Path to automatically move the file to the new location, but this is buggy and has lots of problems.
1
When the file is moved or renamed, it's not updated in the filesystem immediately, and accessing it gives an error that it could not be loaded. Making Godot lose and regain focus updates it.
2
When trying to move to a new folder that doesn't exist, the file is not moved and the new folder is not created. But the Config Path field shows the invalid path, and Project Builder then can't find any routines.
3
Pressing the revert icon next to the config path sets it to . It also removes Project Builder and this setting from the Project Settings so it can't be set back. Reloading the plugin from the Project Settings fixes this and sets it back to the default res://project_builds_config.txt.
4
Entering a path to an existing file overwrites that file. This is a little dangerous. Though it's perhaps unlikely that it would hurt anyone, users won't expect this automatic overwriting behavior, so their guard would be down. Fortunately, Godot doesn't allow dragging files to this field to overwrite them (at least for me). But how much can this behavior be counted on?
Pressing the file icon to select a file instead of writing the filepath gives a popup to browse the filesystem and select files. When a file is selected through here it gives the popup File "..." already exists. Do you want to overwrite it? It's good that it tells users exactly what would happen, but when would it be desired to overwrite a file here? This is basically file-shredding functionality. Though it is possible to navigate to the existing folder you want, type in your filename, and save.
5
Also, the same issue as #3 is present, aborting moving between partitions on some systems. (I personally don't mind this at all.)

There might be a way to make this functionality work, but I don't think that's a good plan. Most users aren't going to expect or try to use automatic renaming, and it opens safety concerns, so my suggestion is to drop that feature. I think it would be better to copy the functionality such as in Application > Config > Icon and Application > Run > Main Scene. These update the field in the Project Settings when the set file is moved to a new folder. But even if the user has to reselect the file manually through the Project Settings, I think it would be an improvement.
That would fix problems 1, 2, 4, and 5. My guess is the remaining problem 3 would be easy to fix by setting a hint somewhere to make the default filepath be the same as the one that is automatically generated when Project Builder first opens a project.
Another option could be to track this config path through the Project Builder's Config > Local Config settings instead of the Project Settings through the plugin. The plugin could then be kept as a limited, optional feature for adding an interface into the Godot Editor. And this feature could be valuable in the Project Builder apart from the plugin. But if I'm not mistaken that would be settings in the file for the file to find itself, so it might not be feasible. Perhaps something in global settings could map projects to their local config files? It's hard for me to tell how viable or good this idea would be. One issue is that this could make sharing files more complicated.
Another option would be to require the local config file to be at the root of the project and remove the feature of setting a path to it completely. I think this would be my preference, since I don't have any use for this feature and it would simplify things. But maybe someone else would want this feature.

Tested and I approve this new design. I like the implementation idea of storing the setting as a hidden variable in project.godot.
I once managed to enter a state in which I moved the local config path somewhere else while Project Builder didn't find it and created a new empty local config instead. But this wasn't dangerous and it was easy to fix when it happened. I think this was due to first updating the local config path from Project Builder and then updating the plugin. So I think this is a one-time upgrade inconvenience and not a problem worth fixing.
I noted some minor usability improvements. I don't consider any of them essential.

The file dialog could look for and save as .txt instead of "All Files".
The file dialog could set a default filename that is the current filename. This would make it easy to go to the desired directory and click Save without having to think about what to call the file. It already sets the default filename this way after using the file dialog to move the file once.
Selecting the current config file in the file dialog gives the confirmation popup "File ... already exists. Do you want to overwrite it?" Everything works fine, but the confirmation popup doesn't reflect what is actually happening. It would be better if there was no confirmation popup here, since there's already safety against overwriting.
It seems like it doesn't allow you to save the local config file outside of the Godot project folder. I don't know if this is intended, but I think it's a good feature. The problem is the result of trying to do this is it shows an absolute filepath and then gives a generic error. It would be better if it gave an error saying immediately that moving the file outside of the Godot project folder is not allowed.

I managed to again enter the state in which Project Builder can't find the local config file and so creates a new empty one instead. This happens whenever you move the local config file without using Project Builder's interface, so it's not just a one-time upgrade inconvenience.
In order to exit this state, you must exit Project Builder, delete the empty config file, then move and rename the non-empty config file to the place of where the empty config file was. If you do this without exiting Project Builder first, then it will overwrite the non-empty config file with the empty config file. (This is the worst part!)
My guess is it would be a minority (though a substantial one) of users who would try to move local config file outside of the Project Builder interface first. Apart from that, something like the same problem will be encountered by anyone moving a local config file from one project to another, which includes users sharing their local config files.
However, I doubt there is any way to substantially improve this without taking some noticeable usability hit somewhere else, given the nature of the problem. So I still consider this issue solved.
Thinking something like this might happen is why it was my preference to remove the feature completely. Standardizing the filename and location would make sharing and reusing files simpler. Is there a similarly valuable use-case for moving the local config file?

The file dialog could look for and save as .txt instead of "All Files".

The config file can use any extension, but maybe providing default one is better, idk.

It would be better if there was no confirmation popup here, since there's already safety against overwriting.

That's a limitation of FileDialog. You can only select new files in Save mode, but it comes with the warning that can't be disabled.

Is there a similarly valuable use-case for moving the local config file?

It's valuable for project organization. I have all editor-related metadata stored in a single directory (except export_presets.cfg, which can't be moved yet).

It's valuable for project organization. I have all editor-related metadata stored in a single directory (except export_presets.cfg, which can't be moved yet).

Okay, that's a fine use-case. I just wanted to make sure there was some definite use for this feature.
It occurs to me that supporting your use-case could be done by standardizing the filename while only letting users select the folder to find it in. The logic would go:
If selected folder does not contain a local config file:
    Move current local config file there

Else if selected folder does contain a local config file:
    If current local config file is empty:
        Delete current local config file

    Else if current local config file is not empty:
        Do not delete current local config file

    Either way, begin to use local config file in selected folder

Benefits

Simpler interface. No Apply button or confirmations.
Easier maintainance (I think). No overwrite protections or hacks using the file dialog in an unintended way.
Users can move local config files outside the Project Builder interface without apparent bugs.
Users can reuse and share local config files with less hassle.

Downsides

No ability to change the filename.
(IDK, maybe there's some other problems with this I can't think of.)

So then I think the question is: Is there a valuable use-case for changing the filename? If there is none, then I think this could be a substantially better interface.
Though again, the way the interface is now is good enough if you don't want to change it anymore.

2d2464b removes Apply button and makes the path non-editable. You can change it by picking a new folder, the file name is hard-coded.

I think it's an improvement, but it doesn't implement the full functionality I suggested. So it's missing these 2 benefits:

Users can move local config files outside the Project Builder interface without apparent bugs.
Users can reuse and share local config files with less hassle.

Maybe the checklist I made from testing will be helpful:

[x] Creates and finds local config when past location is in root folder of project
[x] Creates and finds local config when past location is in subfolder
[x] Move to new folder using Project Builder interface
[x] Move to same folder (do nothing) using Project Builder interface
[ ] Move without Project Builder interface and select new folder using Project Builder interface
[x] Cannot move outside project folder
[ ] Select a different local config in a different folder

[ ] Delete old local config if it's empty
[ ] Do not delete old local config if it's not empty

[x] Can delete local config in current location and add new local config over it to switch without Project Builder interface
[x] Does not overwrite a local config when trying to select it

Check after c587eb0

Looks good! Everything on the checklist now works.
Just 2 notes on slightly weird behavior:

Selecting the same local config with no change gives a message. I would expect it to do nothing like canceling.
Selecting either the same or a different local config when it was changed saves the changes since the last time the routine editor was exited and drops changes after it. This behavior is just hard to predict. You can't rely on either reloading the same file to revert changes, or to save the file when selecting a new one. But no one should be relying on these things anyway.

I'd say these don't matter and that this issue is complete.

42d8765


How to increase svg width based on text

I'm using this component in my app in many places. recently i found some difficulty with the usage. if i have long text the width of the svg must be adjustable based on the text and the text must be in the center of the box. For suppose

My code is something like below
<View style={{marginTop: 50,width:'100%', height: 50, justifyContent: 'center', alignItems: 'center', flexDirection:'row'}}>                         <Svg  width={200} height={50}  style={{backgroundColor:'yellow'}}>                             <SvgLinearGradient id="Gradient" x1="0%" x2="100%" y1="0%" y2="0%">                                 <Stop stopColor="#C2515D" offset="0%"/>                                 <Stop stopColor="#853C5E" offset="100%"/>                             </SvgLinearGradient>                             <Text fill="url(#Gradient)"                                   x="50"                                   y="20"                                   fontSize="18"                                   fontWeight="700"                                   textAnchor="start" >                                 hi hello hi hih hi hi hi hi hi hi hi hi hi hi hi hi hi hi hi hi i hi                             </Text>                         </Svg>             </View>
here based on the text the width of the svg must be increased how?

I tried to write this example with hooks and TS. But I'm getting this error in the tag

Unknown event handler property onLayout

I need to change the font size to adjust it to the size of the container.


Add support for desec.io

Please add support for https://desec.io the modern DDNS service w/ IPv6 support

deSEC uses the No-IP protocol, so by setting the endpoint to https://update.dedyn.io, you can technically get DsDDNS to work with that service.
But, here's the rub: If deSEC doesn't receive both the IPv4 and IPv6 addresses in the same update request, it will delete the A or AAAA record that corresponds to the address that wasn't received in that request. DsDDNS doesn't currently support simultaneous updates like this; it requires A and AAAA records to be configured and updated separately, with two separate requests.
So my current thought is to keep the current configuration format, but allow DsDDNS to combine A/AAAA record pairs for the same hostname into single update operations. This would not only allow deSEC to be implemented, but also halve the number of requests for current Google Domains & No-IP users.


Update generate-consume-asynchronous-stream.md

Wrong naming convention was spotted, method names should be in PascalCase by conventions mentioned here https://docs.microsoft.com/en-us/dotnet/standard/design-guidelines/capitalization-conventions
Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Thanks for submitting this change @PhilipYordanov
Before we merge it, can you change that method name throughout the article and the samples. It appears in the text in a few more places. Also, in the sample code the method name should change there as well, in both the starter and finished versions. (Link points to your current branch)

@BillWagner  I've just created pull requests for changing method name in both finished and start tutorial snippets, i'm leaving the links below:
https://github.com/dotnet/docs/pull/27389
https://github.com/dotnet/docs/pull/27390


[Feature Request] - Clarify Flagged content actions

Mohamed Kashkoush — Today at 7:37 AM
I'm assuming the green check means "this is not spam, keep the post" and the red trash bin means "yes this is spam, delete post" but I wanted to be sure. Perhaps we can just use words rather than images/logos, especially since the spam feature is only editor-facing, so I don't think a clean aesthetic is as important compared to the user-facing main interface
(concern is that the opposite interpretation is reasonably valid: could be interpreted as green check = "yes this is correctly identified as spam, delete post" and red bin = "this is not spam, keep post and delete spam report")

We will add a tooltip for clarification of these different actions


FancyZones editor shows zone frosted zone preview on wrong monitor (monitors 3/4 flipped) while positions in the graphic are correct

ℹ Computer information

PowerToys version: 0.29.3
PowerToy Utility: FancyZones
Running PowerToys as Admin: No
Windows build number: 18362.1256

📝 Provide detailed reproduction steps (if any)

Open Fancyzones editor again (because it STILL cannot remember the right shapes -- filed that bug twice, resolved/fixed twice -- its back, I sorta give up on that)
Select monitors 3 or 4 (only those are misplaced / misidentified in my case
Notice that the frosted highlights are backwards
The small graphic has them in the right place. The monitors are positioned correctly. Only the frosted preview is wrong.

✔ Expected result
Frosted zone preview should be on the correct monitor.
❌ Actual result
See screen shots.
[NOTE: I specifically left the app spanned across the monitors to prove they are arranged correctly. That does not change the behavior. Their arrangement matches the graphic (and matches what windows things), but the frosted preview does not.]
Second note: as mentioned above, I have to keep doing this over and over again because Fanzy zones keeps forgetting its configuration. This is the 3rd or 4th time now. I have filed the bug twice... I wont anymore, but I hit this one while actually having to go set them again. ALSO note that my saved favorite patterns are also gone this time.
📷 Screenshots

@jp-stewart
is this problem still happening with 0.31.2?

@enricogior yes. Left side frosted even though Monitor 4 selected.

Hi, just installed 0.33.1 -- Still wrong, even with all the new graphics / shading, its backwards.

@jp-stewart
sorry for the delay, we have been busy fixing some older crashing bug, we'll try to investigate this one but I can't promise it will be for 0.35.

@SeraphimaZ what is needed to debug this further?  A bug report zip?

@SeraphimaZ what is needed to debug this further? A bug report zip?

Yes, I need to take a look at it.

/bugreport

Hi there!We need a bit more information to really debug this issue. Can you add a "Report Bug" zip file here?  Then drag the zipfile from your desktop onto the GitHub comment box in this issue. Thanks!
PowerToysReport_2022-01-02-12-21-06.zip

Attached. Sorry for delays -- the issue is mostly impacting work... but Github doesn't allow me to create an account with my work email.. so I have to remember to check personal email for this issue occaisionally.

As an additional note, I should add. As of today, its two different monitors that are flipped from the ones in the images above. (but still have some flipped)
Also just adding... that again I had to go set all of the layouts since they keep getting lost... over and over. I filed issues here on that twice... both being closed, so gave up on that.
But yeah, even in the zip above, at least two of the screen positions should be flipped.

Feel like https://github.com/microsoft/PowerToys/issues/11870 and this are arelated

Is this still an issue with the latest version? /needinfo


Fixing V gene offset

Modified V gene offset to handle a case where the matched portion of the query is shorter than the germline V gene.

The v gene offset commit is causing the CI to fail, looks like a syntax issue with the if statement. Removing the parenthesis will probably fix it. Would you update and resubmit the pull request? I'll accept if it passes CI.


Add support for NSG for control plane

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 or have submitted a pull request, please leave a comment

Description
It is now possible to specify a list of nsgs for the control plane.
Proposed enhancement:

Continue using security list if and only if worker nodes are public i.e. use security lists to control North-South traffic
Switch default to NSGs for private worker nodes i.e. use NSGs to control East-West traffic

New or Affected Resource(s)

Potential Terraform Configuration

# Copy-paste any Terraform configurations for how the requested feature may be used.

References

Closed in #346


[EdfPricesBridge] fixing bridge

Hi,
Small changes has been done on jechange.org.
Fixing issue #4351

Github action seems to be broken because it works fine one my computer


Add DAP capabilities to BSP

This allows the client to run/debug code using the Debug Adapter Protocol

This was superseded by #96 #97 and #98.
Those changes are much simpler as #96 and #97 are in no way related to the DAP and would probably be proposed somewhere in the future as they are natural extensions to the BSP.
#98 on the other hand introduces a simple way of communication that the process requested by the client using run/test endpoints is ready to accept connections and communicate in whatever way is appropriate for this debuggee (e.g. Java Debug Wire Protocol for jvm applications)


Allow number formatting

Guidelines
Please note that GitHub issues are only meant for bug reports/feature requests.
Feature request template
Numbers are formatted to use thousands separator which is not always desirable, for instance when representing phone numbers. Coercing the integer to a string removes the comma, but messes with the sorting, as it sorts from the left and not from the right, so 900 is seen as "smaller" than 98. It would be better to have number formatting options available, such as thousands separator and currency.

I take this one

+1


Bumped version to 0.0.12

Yes, it would be great if you could bump the version and re-release a new version on npm.  I installed it with npm and it is still using jade.compile() instead of jade.render(), so the output is a js function not html.
The package.json lists "version": "0.0.11" although package.json here in master shows 0.0.8, so I'm not sure exactly what happened but bumping the version above 11 should do it.  Bumping the minor or major version seems more appropriate, for what its worth.

@ojdx can you merge into master? Thanks!


Add support for Ansible YAML executable playbook.

It would be nice when a Ansible Playbook start with this line:
!/usr/bin/env ansible-playbook playbook.yml
That the script module execute it.

https://github.com/rgbkrk/atom-script/pull/899

Closing since there's a way, though if we should re-open as a statement on #! lines feel free!

This is what I was looking for. Didn't knew you could set another "subset" of a YAML. Thanks!

Sorry to bring this back, but how exactly do you make this work? I'm relatively new to Atom.

Geeeee. This one is back from the dead..... lol
Trying to wrap my head around it back again....
If I recall correctly,
You can create a normal playbook file and at the top, add this -->
!/usr/bin/env ansible-playbook playbook.yml
This would run the playbook file that you are within like a script that can be started by just running
./playbook.yaml


Fix #388. Ignore parent/prior text nodes.

This change is

PTAL @RoboErikG

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Plex integration not working

Hi, I am having issues integrating plex. I have read the previous issue post's and followed the steps. I tested the http:htpcip:port/plex/myPlexSignin/yourusername/yourpassword which gave me "Logged in to myPlex"
Logs are -
2015-12-07 22:11:59	modules.plex	ERROR	Unable to fetch currently playing information! Exception: HTTP Error 401: Unauthorized
2015-12-07 22:11:55	modules.plex	ERROR	Unable to fetch all movies! Exception: HTTP Error 401: Unauthorized
2015-12-07 22:11:54	modules.plex	INFO	GDM: Servers discovered: 1
2015-12-07 22:11:53	modules.plex	INFO	Sending discovery message: M-SEARCH * HTTP/1.0
2015-12-07 22:11:36	modules.plex	ERROR	Unable to contact Plex via http://<IP_ADDRESS>:32400 error HTTP Error 401: Unauthorized
2015-12-07 22:11:35	modules.plex	INFO	GDM: Servers discovered: 1
2015-12-07 22:11:34	modules.plex	INFO	Sending discovery message: M-SEARCH * HTTP/1.0
2015-12-07 22:11:28	modules.plex	ERROR	Unable to contact Plex via http://<IP_ADDRESS>:32400 error HTTP Error 401: Unauthorized
2015-12-07 22:11:26	modules.plex	INFO	GDM: Servers discovered: 1
2015-12-07 22:11:25	modules.plex	INFO	Sending discovery message: M-SEARCH * HTTP/1.0
2015-12-07 22:06:24	modules.plex	ERROR	Unable to contact Plex via http://<IP_ADDRESS>:32400 error HTTP Error 401: Unauthorized
2015-12-07 22:06:24	modules.plex	ERROR	Unable to contact Plex via http://<IP_ADDRESS>:32400 error HTTP Error 401: Unauthorized
2015-12-07 22:06:03	modules.plex	INFO	GDM: Servers discovered: 1
2015-12-07 22:06:02	modules.plex	INFO	Sending discovery message: M-SEARCH * HTTP/1.0
2015-12-07 21:09:33	modules.plex	ERROR	Unable to contact Plex via http://<IP_ADDRESS>:32400 error HTTP Error 401: Unauthorized
I have opened the DB in SQlite but i cannot find a settings page which list's the auth token, maybe i am not looking in the right place.
Any help would be greatly appreciated, thanks, Nick

See the picture :)

Hi, I followed the picture guide and it all seems fine, It has a Auth_token and the username and password are ok, I am using the plex pass version of PMS (ver <IP_ADDRESS>) could this be an issue? Could i try deleting the auth token form the db and see if it generates a new one?

I don't know if there is any difference between the pms version. Any special chars in the password. Have you changed the password after you set it initially.
You can try to delete the token

So I reverted back to the main PMS and bobs your uncle its sorted, somewhere in there i broke the psutil but i am sure i can get that going again, I just need to figure out how i broke it.

Haha, well you need a psutil version higher then 3.

All sorted I just reinstalled psutil and its all good.  I will close this now.

Im happy to help :)


Nightly fuzzer job failed on an expression using array_position and week_of_year

Bug description
Nightly linux-presto-fuzzer-run job failed on an expression using array_position and week_of_year:
[Expected behavior] and [actual behavior].
I0118 03:34:23.887835  5909 ExpressionFuzzer.cpp:877] ==============================> Started iteration 59891 (seed: 342267654)
I0118 03:34:23.887974  5909 ExpressionVerifier.cpp:80] Executing expression: array_position("c0","c1",week_of_year(1999-03-05T15:14:57.350817728))
I0118 03:34:23.888015  5909 ExpressionVerifier.cpp:32] 2 vectors as input:
I0118 03:34:23.888017  5909 ExpressionVerifier.cpp:34]  [DICTIONARY ARRAY<BOOLEAN>: 100 elements, 20 nulls], [ARRAY ARRAY<BOOLEAN>: 100 elements, no nulls]
I0118 03:34:23.888025  5909 ExpressionVerifier.cpp:34]  [CONSTANT BOOLEAN: 100 elements, 0]
I0118 03:34:23.888176  5909 ExpressionVerifier.cpp:133] Modified inputs for common eval path:
I0118 03:34:23.888183  5909 ExpressionVerifier.cpp:32] 2 vectors as input:
I0118 03:34:23.888186  5909 ExpressionVerifier.cpp:34]  [LAZY ARRAY<BOOLEAN>: 100 elements, no nulls]
I0118 03:34:23.888191  5909 ExpressionVerifier.cpp:34]  [CONSTANT BOOLEAN: 100 elements, 0]
E0118 03:34:23.888625  5909 Exceptions.h:68] Line: ../../velox/expression/tests/ExpressionVerifier.cpp:62, Function:compareVectors, Expression: left->equalValueAt(right.get(), i, i) Different results at idx '0': '[0->4] 0' vs. '9', Source: RUNTIME, ErrorCode: INVALID_STATE
terminate called after throwing an instance of 'facebook::velox::VeloxRuntimeError'
  what():  Exception: VeloxRuntimeError
Error Source: RUNTIME
Error Code: INVALID_STATE
Reason: Different results at idx '0': '[0->4] 0' vs. '9'
Retriable: False
Expression: left->equalValueAt(right.get(), i, i)
Function: compareVectors
File: ../../velox/expression/tests/ExpressionVerifier.cpp
Line: 62

System information
CI
Relevant logs
n/a

@kagamiori @kgpai Wei, Krishna, I'm seeing that nightly Fuzzer job uploads the fuzzer log as artifact, but it doesn't upload repro files. Should we modify the job to produce and upload repro files?

velox_expressionVerifier_xPCY3Q.tar.gz


feat(Model): improve Date serialization in Model class

The new check ensures that the value is an instance of Date in the serializeValue function of Model.ts.
Adding a condition like if ('toISOString' in value') to check for the existence of a method on an object can lead to issues in JavaScript because it does not accurately check whether the method actually exists and can be called. This is because objects in JavaScript can have inherited properties, and the in operator also checks for properties in the object's prototype chain. As a result, using the in operator to check for the existence of a method on an object can sometimes result in unexpected behaviour.
In my personal experience, I had a big headache with the in operator in Python, and since then, I have never used in in any language with similar behaviour. Because it affected performance much and resulted in unexpected outcomes.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/in#Caveats :
The in operator returns true if the specified property is in the specified object or its prototype chain.
A few examples:
const myObj = {
  toISOString: function () {
    return 'some value';
  }
};

// This will return true, even though myObj is not a Date object
console.log('toISOString' in myObj);

// This will return false because myObj is not a valid Date object
console.log(myObj instanceof Date && !isNaN(myObj.getTime()) && typeof myObj.toISOString === 'function');

class MyObject {
  toISOString() {
    return 'some value';
  }
}

class MyChildObject extends MyObject {

}

const myObj = new MyChildObject();

// This will return true, because myObj's prototype chain includes the toISOString method
console.log('toISOString' in myObj);

// This will return false because myObj is not a valid Date object
console.log(myObj instanceof Date && !isNaN(myObj.getTime()) && typeof myObj.toISOString === 'function');

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
The new check ensures that the value is an instance of Date, that its getTime() method returns a valid number, and that its toISOString() method returns a valid ISO 8601 string.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (99fc915) 99.64% compared to head (b8a90e4) 99.64%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1121   +/-   ##
=======================================
  Coverage   99.64%   99.64%
=======================================
  Files          88       88
  Lines        5875     5878    +3
  Branches      536      537    +1
=======================================
+ Hits         5854     5857    +3
  Misses         19       19
  Partials        2        2

Impacted Files
Coverage Δ

packages/pinia-orm/src/model/Model.ts
100.00% <100.00%> (ø)

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

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

Thanks for this PR @sergerdn. 🥇 . That's a good notice.


Added SessionReadOnly

This PR provides a new interface ISessionReadonly that allows to create sessions only for read operation. It doesn't use transactions and it doesn't cache entities at identity map.
It is useful for helping devs to reduce the number of locks when they use YesSql and for  reducing the probability of interlocks as it is explained in this issue https://github.com/OrchardCMS/OrchardCore/issues/8705

Tests on this pr finally are fixed. Problem was I reproduced the interlock only for sqllite which was the only db I've testes the pr.  For sqlserver that test was not valid cause it throws an exception cause it doesn't accepts to have multiple transactions open for one connection. So, I've created only a test for sql server which creates two stores to simulate the scenario of two threads locking one to the other.

It doesn't use transactions

I think I have read that it's important to do reads in transactions. Do you have another opinion?

You are right I didn't remember that point. I read somewhere the cost of opening a trasaction is important, and when you don't add a transaction every single query is created by the server with a transaction that is closed after it ends to ensure the atomicity of the operation. So, that's why are cheaper longer transactions.
Thinking on the problems we are facing I would add to that idea that it is important to measure the counterpart of having longer transactions, which is it is more likely they lock other operations. So the initial performance gain you would obtain with longer transactions in an scenario with low interactions between transactions will be lost and maybe it will be worse in an scenario with more locks interactions caused by those long transactions. So, it looks it is important to have tools for both scenarios.
In my case all the integration test create at least one user and request an access token to openid module, thats's why for me was enough to reinmplement UserStore, and openiddict stores using readonly sessions for read methods, and regular sessions for methods that store changes on db. By the momento no other changes are needed on my db.

and it doesn't cache entities at identity map

For my use case there is no problem on caching also for readonlysessions. I removed caching when I was writting the tests for this pr, cause I found weird that after another session writes sth at the db if you read from the db the data readed didn't have the change. But I understand that it could be valid depending on the scenario.
I found also strange to use the identitymap in a session that was not though for tracking changes. And because I saw that no matter if an entity is cached or not, the query to db is always done.

Is that a performance issue?
I didn't though on that, I see that to use the identity map cache saves the session from  deserialize the entity, so to remove the cache has to have a penalty. I've meassured how much my 185 integration tests takes to execute. with and without identity map cache and times are almost the same. Maybe with test that handles bigger documents and queries on same request accessing same documents this difference is meaningful, but at least for my test it is not meaningful.

Using a custom interface is a solution, but maybe there are other options to brainstorm.
Like what if there was a mode where it would not create a transaction if there are no updates. Wouldn't that be better, since it would be transparent?
Or a mode where identitymap is not used.
Could even be options on the CreateSession() still, but return the same interface. Technically I don't think a custom interface is necessary since this is the same contract.

Yes, interface is only a helper to help users of the library which methods they should use for the mode they selected when the session was created. It is not a must have.
To have more modes on the creation of the session is a great help for supporting any scenario, the problem is it will lead to have multiple kinds of Session registered for IoC each one configured with one different mode.
What I would like is to use an approach like EF which allows to define the default mode in the creaton of the session but also to set the mode for one query, with extensiong like .AsNoTracking()
https://docs.microsoft.com/es-es/ef/core/querying/tracking
We could also add another one called .WithTransaction().
And for Session.GetAsync() queries we could provide optional params for doing the same

In this way Orchard only will have a Session class registered for IoC that will be valid for using on every scenario we need

Problem is a connection cannot execute queries without transactions when there is a transaction open, so it will need two connections per session, one for queries with transaction and the other for queries without transaction.

Adding documentOID to indexes problems of this issue https://github.com/sebastienros/yessql/pull/349 dissapeared.
My integration tests are running on my own machine. I've the feeling that interlock problems would come back in production with more Azure instances running the app under heavy load. So, it could be worth to have sth like what we are talking about on YesSql to provide support for scenarios where indexes are not enough to avoid interlocks


Adding HTTPStatusEvent Listener "Non-null type parameter" error

Hey all. I get this issue when setting an HTTPStatusEvent listener using the below code:
	public function login(username:String, password:String):Void
	{
		var loginVars:URLVariables = new URLVariables();
		loginVars.username = username;
		loginVars.password = password;

		var loginRequest:URLRequest = new URLRequest(Endpoints.SIGNIN);
		loginRequest.method = "POST";
		loginRequest.data = loginVars;

		var loginClient:URLLoader = new URLLoader();
		loginClient.addEventListener(HTTPStatusEvent.HTTP_RESPONSE_STATUS, restfulHandler);
		loginClient.load(loginRequest);
	}

Error: [Fault] exception, information=TypeError: Error #2007: Parameter type must be non-null.
If you set the event listener using the literal value instead of the field, "httpResponseStatus" it works just fine.
Target platform is Flash. Haven't tested on other platforms, will do if necessary.

According to the Adobe Flash documentation, httpResponseStatus is only supported on AIR platform.
https://help.adobe.com/fr_FR/FlashPlatform/reference/actionscript/3/flash/net/URLLoader.html#event:httpResponseStatus

Yes I’m sorry; you’re correct! Seems rather silly to make this AIR-only. Oh well..

Hunter

On Dec 30, 2018, at 12:39 AM, tour1st <notifications<EMAIL_ADDRESS>wrote:
According to the Adobe Flash documentation, httpResponseStatus is only supported on AIR platform.
https://help.adobe.com/fr_FR/FlashPlatform/reference/actionscript/3/flash/net/URLLoader.html#event:httpResponseStatus
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/openfl/openfl/issues/2113#issuecomment-450542417, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AG0ox_DvNm0jKkUsYZHnNpL-5kK2euOsks5u-F-mgaJpZM4ZkFeA.

Well, AIR-only means that Flash Player can’t do it, as far as AS3. For Haxe, everything that was AIR-only could probably be implemented on every target besides Flash.


support for Statement request size

Java driver has requestSizeInBytes
to give a higher estimate of Statement size for counting the batch size
https://docs.datastax.com/en/drivers/java/3.7/com/datastax/driver/core/Statement.html#requestSizeInBytes-com.datastax.driver.core.ProtocolVersion-com.datastax.driver.core.CodecRegistry-
go has only Batch::Size()
so when client wants to do exact batch splitting, it's hard to count the statement size
Could above be ported over from java to go?
Java logic is in:
https://github.com/scylladb/java-driver/blob/3.7.1-scylla-2/driver-core/src/main/java/com/datastax/driver/core/BatchStatement.java#L203
or
https://github.com/scylladb/java-driver/blob/3.7.1-scylla-2/driver-core/src/main/java/com/datastax/driver/core/RegularStatement.java#L185
or
https://github.com/scylladb/java-driver/blob/3.7.1-scylla-2/driver-core/src/main/java/com/datastax/driver/core/BoundStatement.java#L307
(and similar, just query java driver tree for requestSizeInBytes)

Perhaps this belongs in the driver? Not sure what is needed is available
outside of the driver package.
On Tue, 21 Jul 2020, 18:02 Lubos Kosco<EMAIL_ADDRESS>wrote:

Java driver has requestSizeInBytes
to give a higher estimate of Statement size for counting the batch size
https://docs.datastax.com/en/drivers/java/3.7/com/datastax/driver/core/Statement.html#requestSizeInBytes-com.datastax.driver.core.ProtocolVersion-com.datastax.driver.core.CodecRegistry-
go has only Batch::Size()
so when client wants to do exact batch splitting, it's hard to count the
statement size
Could above be ported over from java to go?
Java logic is in:
https://github.com/scylladb/java-driver/blob/3.7.1-scylla-2/driver-core/src/main/java/com/datastax/driver/core/BatchStatement.java#L203
or
https://github.com/scylladb/java-driver/blob/3.7.1-scylla-2/driver-core/src/main/java/com/datastax/driver/core/RegularStatement.java#L185
or
https://github.com/scylladb/java-driver/blob/3.7.1-scylla-2/driver-core/src/main/java/com/datastax/driver/core/BoundStatement.java#L307
(and similar, just query java driver tree for requestSizeInBytes)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scylladb/gocqlx/issues/169, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJYU45ITTWY6NF5TEQMATR4W32DANCNFSM4PDX6E5Q
.

closing in favour of https://github.com/scylladb/gocql/issues/51


rework staking::reap_stash

Changes reap-stash to be able to also reap an account that due to any reason have a ledger.active less than existential deposit. Also makes the transaction signed, but free in case of successful execution, for better protection.

const ED = new BN(10000000000);
const ledgers = await api.query.staking.ledger.entries();
let count = 0;
let stale = 0;
for (let [ctrl, ledger] of ledgers) {
	const total = ledger.unwrapOrDefault().total;
	count += 1;
	if (total.toBn().lte(ED)) {
		stale += 1;
		console.log(`🚨 ${ctrl.args[0].toHuman()} has ledger ${api.createType('Balance', total).toHuman()}.`)
	}
}

console.log(`${stale} / ${count} are stale`);

unravels that there are 2275 out of 41990 garbage ledgers in polkadot, for instance.

bot merge

bot merge


AmqpMessageConverter: support AbsoluteExpiryTime larger than max DateTimeOffset

AbsoluteExpiryTime from Microsoft.Azure.Amqp is a DateTime. On the other hand, AmqpMessage.Properties.AbsoluteExpiryTime is a DateTimeOffset. Some DateTime cannot be converted to DateTimeOffset, causing ArgumentOutOfRangeException ("The UTC time represented when the offset is applied must be between year 0 and 10,000."). This has been observed on a message that was sent and received using Azure.Messaging.ServiceBus between 2 processes running in the EST timezone.
Here the AmqpMessageConverter is adjusted to work when the DateTime is out of range.
A unit test is added. This unit test fails without the change to the AmqpMessageConverter, and when the timezone of the local machine is set to a timezone that lags behind UTC.

Thanks for the fix @tlecomte!

Thanks @JoshLove-msft ! Looking forward to the next release 😄


Auth support?

I'd like to Proxy DigitalOcean Spaces (S3 clone) with authorization.
Does this module allow me to use access keys to retrieve private files? Thank you!

Storing access keys in Caddyfile itself is probably not a great idea. You can use the features of the underlying AWS SDK though. If you set any of the standard environment variables for AWS SDK/CLI, it should work.
You can also use multiple profiles and set the profile in your Caddyfile.
Hope that helps.

Thanks for the quick reply! So theoretically I could set env variables like:
AWS_ACCESS_KEY_ID=AKIAIOSFODNN7EXAMPLE
AWS_SECRET_ACCESS_KEY=wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY

Then I would be able to use that like:
example.com

file_server {
	fs s3 {
		bucket mybucket
		region nyc3.digitaloceanspaces.com

		'credentials' => [
                      'key'    => {AWS_ACCESS_KEY_ID},
                      'secret' => {AWS_SECRET_ACCESS_KEY},
                ],
	}
}

Not sure exactly how to format the syntax for that kind of request. Thank you!

You don't need the credentials block. Setting the env cars should be enough.

Oh perfect! I guess I'd only need that if I wanted to use a different env variable name?
example.com

file_server {
	fs s3 {
		bucket mybucket
		region nyc3.digitaloceanspaces.com
                accessKeyId {DO_ACCESS_KEY}
                secretAccessKey {DO_SECRET_KEY}
	}
}

Is there a reference I could look at for getting the keys / syntax correct? Thanks!

This is all you need:
example.com

file_server {
	fs s3 {
		bucket mybucket
		endpoint nyc3.digitaloceanspaces.com
		force_path_style
	}
}

export AWS_ACCESS_KEY_ID=...
export AWS_SECRET_ACCESS_KEY=...

caddy

Here is the reference: https://github.com/sagikazarmark/caddy-fs-s3#usage

Is this fixed? I'm trying this currently and getting an error. I've set up my key environment variables according to your last message, but still can't retrieve my file to do "no EC2 IMDS role found". This is from a public bucket, stored on Supabase, and if I go directly to the file's URL there is no problem.
server@localhost ~ % ./caddy version
v2.8.4 h1:q3pe0wpBj1OcHFZ3n/1nl4V4bxBrYoSoab7rL9BMYNk=

When I try to access the resource I get this error:
2024/11/18 04:01:19.670	ERROR	http.log.error	open test-uid-001/index.html: operation error S3: GetObject, get identity: get credentials: failed to refresh cached credentials, no EC2 IMDS role found, operation error ec2imds: GetMetadata, http response error StatusCode: 404, request to EC2 IMDS failed	{"request": {"remote_ip": "<MY IP ADDRESS>", "remote_port": "61630", "client_ip": "<MY IP ADDRESS>", "proto": "HTTP/1.1", "method": "GET", "host": "<SERVER IP ADDRESS>", "uri": "/test-uid-001/index.html", "headers": {"Accept-Encoding": ["gzip, deflate"], "Connection": ["keep-alive"], "Upgrade-Insecure-Requests": ["1"], "Accept": ["text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8"], "User-Agent": ["Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.5 Safari/605.1.15"], "Accept-Language": ["en-AU,en;q=0.9"]}}, "duration": 0.001138951, "status": 500, "err_id": "629qiubiv", "err_trace": "fileserver.(*FileServer).ServeHTTP (staticfiles.go:285)"}

Are there other ways of doing this I should be trying?

Honestly, I haven't tried this with public buckets. I'm not quite sure if accessing the bucket through the S3 API is supposed to work without authentication.
AFAIK a bucket's "public" nature is just an additional HTTP interface to access the objects, and going through the API (which is what this plugin does) may still require authentication...but I could be wrong there.
I wonder, though: if it's a public bucket, why do you need this plugin? I mean, if you wanted to hide the bucket behind a custom URL, you could just use the reverse proxy component of Caddy, couldn't you?

hey, sorry for digging this up, however it would be nice if this information was provided in the readme. I spent quite some time reading through the source code and eventually looking all the open and closed issues before figuring out how to set keys/secrets.
I appreciate the module, however I was so close to just writing my own as this was my 3rd module I'd been trying with almost no documentation on how to use it.

Added a section about authentication in the documentation: https://github.com/sagikazarmark/caddy-fs-s3?tab=readme-ov-file#authentication


Property test for vector bounds.

Added a property test that fails catastrophically do to an index out of bounds access.
$ stack test
+++ OK, passed 100 tests.
+++ OK, passed 100 tests.
+++ OK, passed 100 tests.
+++ OK, passed 100 tests.
+++ OK, passed 100 tests.
+++ OK, passed 100 tests.
+++ OK, passed 100 tests.
+++ OK, passed 100 tests.

Completed 2 action(s).
Test suite failure for package succinct-vector-1.0.0
    tests:  exited with: ExitFailure (-11)
Logs printed to console

If it helps, the code in my repo shows the line that fails:
https://github.com/haskell-works/hw-succinct/tree/debugging-index-out-of-bounds
$ stack test
hw-succinct-<IP_ADDRESS>: test (suite: hw-succinct-test)

HaskellWorks.Data.Succinct.RankSelect.Rank9
  HaskellWorks.Data.SuccinctSpec
iii11sts)
*** Failed! Exception: 'K' (after 1 test):
VectorRank [1] 64

instance SuccinctNativeBitVector NativeBitVector where
    partialRank (NativeBitVector rank9_ _ bits_) p0 = (c1 + c2 + c3, p3)
      where
        (c1, p1) = partialRankLevel0 rank9_     p0
        (c2, p2) = partialRank       basicBlock p1
        (c3, p3) = partialRank       w64        p2
        basicBlock = BasicBlock (safeIndexJ rank9_ (2 * (getPosition p0 `quot` 512) + 1))
        w64        = trace ("iii" ++ show iii ++ show (Unboxed.length bits_)) $ safeIndexK bits_ iii --- <-----------
        iii = getPosition p0 `quot` 64

Also, what's the motivation for the equality in the second test on this line?
rank :: SuccinctBitVector a => a -> Position -> Maybe Count
rank sbv p0 = do
    guard (0 <= p0 && p0 <= size sbv)    ---- <-------------
    let (c1, p1) = partialRank sbv p0
    guard (p1 == 0)
    return c1

The equality test is supposed to represent the values of p0 that are supposed to be safe arguments to unsafeRank, so your test indicates a genuine bug


chore: add codeql-analysis workflow

Fixes #69

Codecov Report

Merging #70 (63eef1a) into main (1eeeaf0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #70   +/-   ##
=======================================
  Coverage   85.22%   85.22%
=======================================
  Files          56       56
  Lines        4439     4439
  Branches      458      458
=======================================
  Hits         3783     3783
  Misses        568      568
  Partials       88       88

Continue to review full report at Codecov.

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


WithFields - Spaces after log message

Hi
I'm trying to use the 'WithFields' function to supply fields to the log messages. Somehow this introduces additional spaces after the actual log message ('msg'):
log.WithFields(log.Fields {
    "test": "something"}).Error("DataExchangeClient could not connect server")

Which results in:

ERRO[0000] DataExchangeClient could not connect server   test=something

There are 3 whitespaces after 'server'. Looking at other messages there are even more whitespaces. Is there a reason for this? I didn't find any hint in the documentation...
Regards

Hi @inzanez ,
Logrus default logger "std" use TextFormatter. When output to terminal, TextFormatter will call printColored function in text_formatter.go.
fmt.Fprintf(b, "\x1b[%dm%s\x1b[0m[%04d] %-44s ", levelColor, levelText, int(entry.Time.Sub(baseTimestamp)/time.Second), entry.Message)
"%-44s" explains your issue.

@nianglao
Perfect, many thanks for that detailed answer! I will take a closer look at that!

Is there a way to fix this?


Fix ant build error: Unexpected attribute "additionalparam"

additionalparam attribute should be on javadoc element

I received this error as well on Gentoo Linux:
el@defiant ~/bin/ABAGAIL $ ant
Buildfile: /home/el/bin/ABAGAIL/build.xml

BUILD FAILED
/home/el/bin/ABAGAIL/build.xml:40: Unexpected attribute "additionalparam"

Total time: 0 seconds

I'm also seeing this on OS X (10.11.3). This PR works to resolve it.

My master branch has diverged