Remove unused package reference to ReactiveUI

Unless i"m missing something, there does not appears to be a need to pull in the ReactiveUI package.

Thanks for the catch!


NTR sent to OMRSE: social economics related

See:
https://github.com/ufbmi/OMRSE/issues/62

household wealth index
poverty line
socioeconomic factor
household assets
household identifier
family identifier
marital status
family income
education
occupation
job
employment
salary
number of hours per day of employment
human traveling information

Some of the terms have been added in OMRSE. Need to check whether those terms have been imported in VEuPathDB ontology.
For those have not been added, we need to check whether we still need the terms and follow up the progress in OMRSE.


subset_nhdplus() catchments overlapping boundaries

Came across this while trying to get the nhdcatchments via comid. The topology seems to be off.... Is this normal?
It strikes me as some kind of rounding issue. If I look at the raw data that is loaded in the database, am investigating what might be up.

So this is a new one to me. Usually a 0 buffer doesn't do anything like this, but the S2 implementation does? Will have to do a little digging on what might be going on here.
p <- sf::read_sf("https://labs.waterdata.usgs.gov/geoserver/wmadata/ows?service=WFS&version=1.0.0&request=GetFeature&typeName=wmadata%3Acatchmentsp&maxFeatures=1&outputFormat=application%2Fjson")
p <- sf::st_make_valid(p)
buffer <- sf::st_buffer(p, 0)
plot(sf::st_geometry(p))
plot(sf::st_geometry(buffer), add = TRUE)

Created on 2022-01-03 by the reprex package (v2.0.0)

Hey @dblodgett-usgs thanks for taking a look! Also does it for the huc12 and huc08 as well. That's weird st_buffer(., 0) does that...

Looks like it's related to this. https://github.com/r-spatial/sf/issues/1692
You should be able to "fix" by setting sf::sf_use_s2(FALSE) for the near term.
It looks like I need to stop using a 0 buffer as a geometry cleaning tool.

Yup, perfect! That will work for now. Thanks :)


ModelElementFace.tintIndex -> ModelElementFace.layer

Second parameter to the ModelElementFace here is derived from the textures provided for the item in it's model json.

It is the "layer" passed to this method, aka, the numerical index corresponding to the constants layer0 -> 0, layer1 -> 1, etc... defined at the top of this class

Of course this means several downstream field names and method parameter names may need to be changed as well.
ModelElementFace(cullFace, tintIndex, textureId, textureData) -> ModelElementFace(cullFace, layer, textureId, textureData)
ModelElementFace.tintIndex -> ModelElementFace.layer
BakedQuad.colorIndex -> BakedQuad.layer
BakedQuad(vertexData, colorIndex, face, sprite, shade) -> BakedQuad(vertexData, layer, face, sprite, shade)
BakedQuad.hasColor() -> BakedQuad.isBaseLayer()
BakedQuad.getColorIndex() -> BakedQuad.getLayerIndex()
ItemColors.getColor(item, tintIndex) -> ItemColors.getColor(stack, layer)
ItemColorProvider.getColor(var1, var2) -> ItemColorProvider.getColor(stack, layer)
One complication, though, current name seems to be based on the crobbed Mojang name in the deserializer.
protected int deserializeTintIndex(JsonObject object) {
     return JsonHelper.getInt(object, "tintindex", -1);
}

Though IMO I'd rather go with it being the layer, or layerIndex, rather than tintindex because keeping it as it is creates confusion about whether it's meant to be a colour.

No, the tint index does not correspond exactly to a layer in all scenarios, and multiple faces can share the same tint index. Layer faces are an exception as they are generated from a texture.


Provider resource editor

https://github.com/digirati-co-uk/iiif-manifest-editor/issues/85 is a child of this task - you need an Agent as part of a Provider.


Azure default max no. of pods differs from the original provisioned through Azure Portal

While provisioning an AKS cluster from portal using Azure CNI as default network plugin the default max no. of pods provisioned is 110. While in the document it says 30.

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

ID: 37ba0343-7e43-6e2e-e564-5d3a0f523a75
Version Independent ID: 0ba581a1-3e4a-27b4-c667-4a9b969a3ade
Content: Configure Azure CNI networking in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/configure-azure-cni.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@sujaypillai
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@sujaypillai
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@zr-msft @mlearned
Can you please check and add your comments on this doc update request as applicable.

@zr-msft @mlearned
Can you please check and add your comments on this doc update request as applicable.

Thank you @sujaypillai for the feedback!
The article has been updated to incorporate your feedback.
#please-close


[ARPA P&E Reporting] Audit Report

Submitted Description:
Include all expenditure data reported by project ID, and by subaward #, and by quarter - to assist State's in finding calculation issues as reports are loaded into the Reporter (instead of waiting on Treasury validations) and to assist in reconciling projects to our accounting system. This is Ohio's #1 priority!
(Optional) Uploaded file - click to download:

Closing in favor of #1303


Trying to run the Google Colab notebook as is, getting inundated with an error

_/usr/lib/python3.9/encodings/ascii.py in decode(self, input, final)
24 class IncrementalDecoder(codecs.IncrementalDecoder):
25     def decode(self, input, final=False):
---> 26         return codecs.ascii_decode(input, self.errors)[0]
27
28 class StreamWriter(Codec,codecs.StreamWriter):
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 149: ordinal not in range(128)_
I've tried:

changing the input file to my own
resaving my own explicitly as utf-8 encoded
reading in my file as a df then saving back to TXT with encoding on
None made a difference, please help.

Have you tried f = open(file_path, 'r+', encoding="utf-8")

Thanks for the reply Lucy, yes I have.  Error remained the same.

From: Lucy Li @.>
Sent: Monday, April 24, 2023 3:42 PM
To: booknlp/booknlp @.>
Cc: Gregory P. K. Richardson @.>; Author @.>
Subject: Re: [booknlp/booknlp] Trying to run the Google Colab notebook as is, getting inundated with an error (Issue #14)
Have you tried f = open(file_path, 'r+', encoding="utf-8")
—
Reply to this email directly, view it on GitHubhttps://github.com/booknlp/booknlp/issues/14#issuecomment-1520799172, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHCQUONFX6NJRG7WO5ZOSPDXC3QSLANCNFSM6AAAAAAXB6OCDM.
You are receiving this because you authored the thread.Message ID: @.***>


Changes to support customization of delta file names

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: rymurr:x: jacques-nYou have signed the CLA already but the status is still pending? Let us recheck it.


Vector path command parameters are missing for stickers

I'm requesting the messages in a chat using GetChatHistoryAsync. One of the message's content is a sticker data. I can download the sticker and play the animation, but before that I'd like to display to the user the Outline vector. I receive the following object:

So I get a list of VectorPathCommand, any of then has a DataType, for ex. vectorPathCommandCubicBezierCurve. But there are no parameters like start_control_point or end_control_point as the documentations says.
My question is that is this a bug in tdlib? Or I need to call another specific method to get the parameters?
I'm using the tdlib in C#.

Okay, and what is the way to convert the array entries?
But anyway as you can see from the screenshot in the response received directly from the tdlib, there are no control points at all.
Would it be possible that the tdlib has a conversion error when parsing the response from the telegram servers?

that the tdlib has a conversion error when parsing the response from the telegram servers?

No. TDLib API is strictly typed, and can't have fields missing.

Okay, and what is the way to convert the array entries?

You need to check wrapper documentation, or search for the definiton of VectorPathCommandCubicBezierCurve class.

Thanks for the response.
I checked the tdlib code and I found that for creating the Commands array the get_sticker_minithumbnail method is used in StickersManager:1933.
The return value type is vector<td_api::object_ptr<td_api::closedVectorPath>> and indeed the values are set correctly. So I move forward to check the C# api.
However I see that there is commented code, which converts the commands to an svg string in StickersManager:2123-2164. Would it be possible to add this code and create a method which returns this information?

The code exists only for testing. App can generate SVG itself, if this is the simplest way to show the outline for you.

okay thank you


AoT: Remember if an app has Always on Top set, turn on when reopened

Description of the new feature / enhancement
Be able to always designate a window as always on top, even if its dismissed, so that next time it comes up, it still always on top
Scenario when this would be used?
Outlook reminders window.  When you dismiss reminders, the window closes.  The next time there is a Reminder, its no longer always on top.
Supporting information
No response

It's a different use case.  There are several different needs for always on top, if you look around the web.  A lot of users want to target a particular window like Outlook reminders to always be on top.  Another is that an app, like notepad, is always on top.  Your use case of choosing which I stance of an app always being on top is a third use case.  Powertoys should look I to if or how it could address the different use caces.

I came looking for this. I really want outlook calendar reminders to always be on top, but after closing the first one the new ones won't come on top. Maybe add a configuration option to make it reoccurring or not?

@SeraphimaZykova I see a number of requests for this. Do you have an estimate about the possibilities and cost, maybe?

There is no problem with memorizing a list of aot apps, it'll take 2-3 days. My main concern is that all app windows will be aot, not just particular, e.g. Outlook reminder window, as it's requested here.

This would be really nice, is there any update on it?  I personally would use it for calculator and atm especially for developing a python gui application that only takes up a portion of my screen and that I'd like to stay on top when I restart it

This would be great, but also would need some configuration options to enable remembering to AOT full apps or specific windows, and that would be the tricky part.  Perhaps something cheesy like recognizing a string in the title, so that Outlook itself would not always be on top, but any window it spawns that has the word "Reminder" in its title.  In my case, I would like certain Edge-spawned web apps to be AOT upon opening (di.fm makes for a great Edge app!). 👍


Fix checkstyle warnings

@tldavies here are the code style fixes you asked for this morning.

+1

@benbp

:+1:

was that travis-ci build failure resolved?

Pretty sure that failure is not related to the code style fixes. Most likely a false negative.

Rebased.

@RackHD/corecommitters Please merge :-)


Document v1.VM and v1.DomainSpec

By adding comments to the structs v1.VM and v1.DomainSpec, the swagger documentation is updated. We miss documentation almost everywhere. This is important, so that we can automatically generate documentation in different formats later on.

Hints on the meaning of the fields can be found here:
https://libvirt.org/formatdomain.html

The PR #347 mostly documented v1.VM. Given that https://github.com/libvirt/libvirt-go-xml exists and we may eventually switch to it, I think the issue could be reduced to v1.VM documentation. Documenting api.DomainSpec could be wasted, easily outsourced (to the other project) effort.


Importing into Unreal Engine difficulties

Amazing project you have here! We would love to give it a try :heart:
We are using Unreal Engine 4 but we can't get the world to load. Unpackaging using 7zip succeeded and importing the fbx file also seemed to work. But when loading the map into the world we get the following.

Would you be able to provide a small instruction on how to get this world loaded into UE4? Or maybe point me in the direction where we can learn how to do this?

After a tip from one of the creators (thanks!) I found this page with a tutorial on how to import The trick is to select "FBX Scene Import" in the 'import' file selection window. This will show you a bunch of new options. The tutorial linked shows screenshots with recommended settings. Using those it works!

You're welcome. :)

Hi,
@SijmenHuizenga How exactly did you unpack those files? I have unpacked 7zip archive, which results in a folder, not a fbx file. Is there some kind of process to get fbx file from those unpacked files?

Hi @Lu7913, It feels ages ago I did this :sweat_smile:. From a quick look I think the fbx file we used is ./Assets/ZZ_opendrive/FBX/Zalaegerszeg.fbx.

Been looking for fbx file in folders, apparently not enough :disappointed_relieved:. Thanks.


fix: ensure query is prepared by listening to changes stream

Fixes #225

Codecov Report

Merging #226 (bd49e33) into main (01be522) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:exclamation: Current head bd49e33 differs from pull request most recent head a7806e1. Consider uploading reports for the commit a7806e1 to get more accurate results

@@            Coverage Diff             @@
##             main     #226      +/-   ##
==========================================
- Coverage   89.90%   89.88%   -0.02%
==========================================
  Files         116      116
  Lines        7490     7487       -3
==========================================
- Hits         6734     6730       -4
- Misses        756      757       +1

Flag
Coverage Δ

e2e.cbl
89.51% <100.00%> (-0.02%)
:arrow_down:

unit.cbl
2.92% <0.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

packages/cbl/lib/src/query/ffi_query.dart
96.36% <100.00%> (+0.20%)
:arrow_up:

packages/cbl/lib/src/query/proxy_query.dart
97.36% <100.00%> (+0.14%)
:arrow_up:

packages/cbl/lib/src/query/query.dart
92.15% <100.00%> (-1.60%)
:arrow_down:

packages/cbl_ffi/lib/src/libraries.dart
70.00% <0.00%> (-3.34%)
:arrow_down:

Continue to review full report at Codecov.

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


Earlgrey: prim_fifo_sync.sv doesn't work in Verilator

Parsing prim_fifo_sync.sv file passes in Verilator, but resulting Earlgrey simulation hangs in infinite loop.

Removed from the list in https://github.com/antmicro/verilator/pull/496


Resource retrieve! should not get index for resource

Scenario/Intent
Improve the performance of the resource #retrieve! method.
Environment Details

OneView SDK Version: All, but current is 5.2.0
OneView Appliance Version: All
OneView Client API Version: All
Ruby Version: All
Platform: All

Steps to Reproduce
call #retrieve! on any resource and follow the REST calls being made
Expected Result
Do an intelligent lookup of the resource. Instead of getting the entire list of resources and filtering with Ruby code, it should send the query in the OneView API request, so only that resource is returned. If the URI is set, it can even just do a refresh.
Another option is to create an additional method, but I believe the idea of #retrieve! is to be a performant, safe retrieval using unique attributes.
Actual Result
This method is really not much more than a wrapper for ::find_by(*args).first. When there are large numbers of resources, the current implementation is slow and potentially dangerously memory-intensive.

This SDK has limited support and will not be actively developed for the foreseeable future.


Pol vendor ref number

Create new LDP derived table po_lines_vendor_reference_numbers.

I am not sure why this PR deletes po_instance.sql.  I will try to undo that.

Merging commits manually and closing this PR to avoid deleting po_instance.sql


Don't use postinstall for rust installation

It makes debugging rustup or openssl errors impossible.
Fixes #10

I'm not super familiar with the now builder api, so it's possible there's a better hook than simply build that this should use instead. Apologies if so.

Ah whoops, the version config option is being ignored here.

Er, actually it seems to have been ignored already before this anyway?

Thanks for this as well! This seems like a better option anyway 😄

This should now be up as<EMAIL_ADDRESS>Also you've been added to the contributors list at the bottom of the README :)


Delegate with covariant type parameter in Expression

Bridge throws an exception when trying to cast Expression.Body to an actual expression type - (NewExpression in the sample below). The error is reproducible for a delegate with a covariant generic type parameter, when a derived type was used as a type argument.
Steps To Reproduce
https://deck.net/8bdd3fbbf50b642134db1c9642a52568
using System;
using System.Linq.Expressions;
using Bridge;

public class Program
{
    public static void Main()
    {
        var name1 = GetTypeName(() => new A());
        Console.WriteLine(name1);

        var name2 = GetTypeName(() => new B());
        Console.WriteLine(name2);
    }

    public static string GetTypeName(Expression<Func<A>> expr)
    {
        var newExpr = (NewExpression) expr.Body;
        return newExpr.Type.FullName;
    }
}

public class A
{
}

public class B : A
{
}

Expected Result
https://dotnetfiddle.net/OwmJov
Console output
A
B

Actual Result
Console output
A
System.Exception: Uncaught System.InvalidCastException: Unable to cast type System.Object to type Boolean
Error: Unable to cast type System.Object to type Boolean
    at ctor (https://deck.net/resources/js/bridge/bridge.min.js?16.8.2:7:88705)
    at new ctor (https://deck.net/resources/js/bridge/bridge.min.js?16.8.2:7:93891)
    at Object.cast (https://deck.net/resources/js/bridge/bridge.min.js?16.8.2:7:10290)
    at Function.GetTypeName (https://deck.net/RunHandler.ashx?h=-2138153307:22:59)
    at Function.Main (https://deck.net/RunHandler.ashx?h=-2138153307:15:33)
    at https://deck.net/resources/js/bridge/bridge.min.js?16.8.2:7:40150
    at HTMLDocument.i (https://deck.net/resources/js/bridge/bridge.min.js?16.8.2:7:5505)
    at i (https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js:2:27151)
    at Object.add [as done] (https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js:2:27450)
    at n.fn.init.n.fn.ready (https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js:2:29515)

When unsafe casting using As<>() method is used, a wrong value is returned:
https://deck.net/03a7460cf97dc44cbb7c78252245da33
public static string GetTypeName(Expression<Func<A>> expr)
{
    var newExpr = expr.Body.As<NewExpression>();
    return newExpr.Type.FullName;
}

Console output
A
A


Problem add ac grundig

Hi, I try to connect an ac unit grundig grphi120 but after I added in hacs I try to add homewhiz integration, submit but nothing happens. Any suggestions please!

Hi! Currently, only washing machines are supported 😢

I'll close this for now since there hadn't been any updates. @home89c3, you can try again with the latest branch and we can look at this issue again if the problem is still there!

Hi sorry for the late reply. Ac is with WiFi not Bluetooth. This is the
label with the model. Any suggestion how can I integrate in home assistant.
On Wed, 7 Dec 2022, 01:02 TechHummel, @.***> wrote:

I'll close this for now since there hadn't been any updates. @home89c3
https://github.com/home89c3, you can try again with the latest branch
and we can look at this issue again if the problem is still there!
—
Reply to this email directly, view it on GitHub
https://github.com/rowysock/home-assistant-HomeWhiz/issues/12#issuecomment-1340127675,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2QHWCPIPVMLZP3RNTMX273WL7AXNANCNFSM6AAAAAARTM6BKU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@home89c3 in the latest beta release we added support for wifi devices via cloud. Can you test that?

@rowysock I install the new version and works to added my cloud account. Was detected my ac grundig automatically without modification on configuration.yaml.
I have the interne temperature, target, status if is on /off and the mode. All the information are like a sensor.
Thx for the new version
Do you think will be possible to controll the ac on, off and maybe the target themperature

Glad it is working now!
What you are describing is already tracked with #53 , therefore I will close this issue 😄


[378] JWT long shelf life (clientside)

Context
JWT long shelf life
Changes proposed in this pull request
JWT tokens will be invalid due to

expiration time
mismatch

audience
issuer
subject

Guidance to review
EncodeService belongs nested in DecodeService that is by designed as it has very little to no reason to live.
This is the client side of the service, and consumes the JWT.
Should be reviewed alongside
see https://github.com/DFE-Digital/publish-teacher-training/pull/1429
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 [1 day] JWT long shelf life


Array type doesn't seem to allow clearing all options

When I clear the contents of a form array, an 'empty' array isn't posted back to the controller, so the clear doesn't happen. In fact, nothing is posted for the form value at all. This only happens when I'm removing every entry. Removing 1 of n works fine.  Adding items works fine as well.
It appears that there's some javascript in here deciding not to post the empty results, but I'm having trouble tracking down where the problem is.

@jeffdeville Can you post your model and admin resource file? I need a bit more context to understand. Thanks...

Sure:
Model
defmodule NHWeb.Customer do
  use NHWeb.Web, :model
  import Ecto.Query

  alias NHWeb.{Licensee,License,Company,Repo}

  schema "customers" do
    field :name, :string
    field :about, :string
    field :website, :string
    field :facebook, :string
    field :twitter, :string
    field :urls, {:array, :string}
    field :video, :string
    field :phones, {:array, :string} # unknown phone
    field :office_phone, :string
    field :mobile_phone, :string
    field :fax, :string
    field :featured_image, :string
    field :images, {:array, :string}
    field :valid, :boolean
    field :review_status, :string
    field :reviewed_by, :string
    field :editor_notes, :string

    has_many :customer_terms, NHWeb.CustomerTerm
    has_many :licenses, NHWeb.License
    belongs_to :company, NHWeb.Company
    has_one :listing, NHWeb.Listing
    # many_to_many :terms, NHWeb.Term, through: NHWeb.CustomerTerm

    timestamps()
  end

  def changeset(struct, params \\ %{}) do
    struct
    |> cast(params, [:name, :about, :website, :facebook,
                     :twitter, :video, :phones, :fax, :featured_image,
                     :images, :review_status, :reviewed_by, :editor_notes,
                     :urls])
    |> cast_assoc(:customer_terms)
    |> cast_assoc(:licenses)
    |> cast_assoc(:company)
    |> set_valid
    |> validate_required([:name])
    |> validate_inclusion(:review_status, ~w(new invalid in_process ready))
  end

  def set_valid(changeset) do
    put_change(changeset, :valid, valid?(changeset))
  end

  defp valid?(changeset) do
    do_valid?(Map.merge(changeset.data, changeset.changes))
  end
  defp do_valid?(%{featured_image: image}) when is_nil(image), do: false
  defp do_valid?(%{featured_image: image}), do: String.length(image) > 0
  defp do_valid?(_), do: false

  # **************************************
  # Move this to a context in 1.3 migration
  # **************************************
  @spec existing_licenses(list(map)) :: {:ok, list(NHVerify.License.t)}
  def existing_licenses([]), do: {:ok, []}
  def existing_licenses(licenses) do
    # I guess I break these up by region, otherwise my 'in' queries are messy
    region_lists = group_by_regions(licenses)

    existing_region_licenses = region_lists
    |> Enum.reduce(%{}, fn({region, license_ids}, acc) ->
      query = from l in License,
              where: l.region == ^region,
              where: l.num in ^license_ids,
              select: l
      Map.put(acc, region, Repo.all(query))
    end)
    |> Enum.reduce([], fn({_, vals}, acc) -> acc ++ vals end)
    {:ok, existing_region_licenses}
  end

  defp group_by_regions(licenses) do
    licenses
    |> Enum.reduce(%{}, fn (%{region: region, num: num}, acc) ->
      Map.put(acc, region, Map.get(acc, region, []) ++ [num])
    end)
  end
end

Resource File
defmodule NHWeb.ExAdmin.Customer do
  use ExAdmin.Register
  import Ecto.Query

  register_resource NHWeb.Customer do
    scope :all
    scope :new, fn(q) ->
      from c in q,
      where: c.review_status == "new"
    end
    scope :ready, fn(q) ->
      from c in q,
      where: c.review_status == "ready"
    end

    index do
      selectable_column()
      column :id
      column :name
      column :website
      column :phones
      column :office_phone
      column :featured_image
      # column :licenses, fields: [:num]
      column :review_status
      column :reviewed_by
    end

    form customer do
      inputs do
        input customer, :name
        input customer, :about, type: :text
        input customer, :website
        input customer, :facebook
        input customer, :twitter
        input customer, :urls
        input customer, :video
        input customer, :phones
        input customer, :office_phone
        input customer, :mobile_phone
        input customer, :fax
        input customer, :featured_image
        input customer, :images
        input customer, :editor_notes, type: :text
      end
    end

    member_action :ready, &__MODULE__.ready_action/2, label: "Ready!"

    def ready_action(conn, params) do
      current_user = conn.assigns.current_user

      # changeset_fn = Customer.changeset_fn(defn, :update)

      updated_params = %{review_status: "ready", reviewed_by: current_user.email}
      customer = NHWeb.Repo.get(NHWeb.Customer, params[:id])
      changeset = NHWeb.Customer.changeset(customer, updated_params)

      case NHWeb.Repo.update(changeset) do
        {:error, changeset} ->
          Phoenix.Controller.put_flash(conn, :error, "Customer could not be updated.")
        {:ok, resource} ->
          Phoenix.Controller.put_flash(conn, :notice, "Customer was successfully updated.")
      end
      |> Phoenix.Controller.redirect(to: ExAdmin.Utils.admin_resource_path(conn, :index))
    end
  end
end

And here's a screenshot.  If I try and delete this phone number, it won't work, because the phones field isn't posted with the params

Ok, I get how. Thanks for posting. I believe your best bet is to handle this in your changeset. If the data in the changeset has something different than the params, you know its a delete and could add a change to the changeset.
What do you think?

@smpallen99 The downside to that is that you would have to send back the current array with every single update, which isn't very REST-y.
I guess this works with HTML responders, but for this to be "correct", the client should only patch changed values. In this case, the empty array should be sent with the data.
(Just my unsolicited 2¢)

I need to look a little deeper into this, but not tonight.

seems to be fixed in changeset with this code
  def changeset(struct, params \\ %{}) do
    struct
    |> cast(params, [:title, :section, :array1, :array2, :order])
    |> put_change(:array1, Map.get(params, "array1", []))
    |> put_change(:array2, Map.get(params, "array2", []))
  end

@d4rk5eed your solution won't work, as if we call the changeset like on action edit, the params of that field be always be set to [], and when you just get to edit form, you can't see your selected value because it's always be blank


CPU hogging on wallet page

There is a lot of CPU usage when the application is on the wallet page. It is already mentioned on #197 but it is marked as completed.
I've tested the version 1.21.6 and the issue is still present.
Expected Behavior
My computer should not be burned by the app.
Current Behavior
My computer had 90º in a couple of minutes.
Steps to Reproduce
Just visit that page, but it seems like an environment specific issue.
Possible Solution
As far as I could check, the problem seem to involve a process named asgardex --type=renderer many of them quite heavy on CPU usage.
Your Environment - Release Version (skip if not using a release version)
I'm on a not so old Linux, in a recent computer but without video card (just the embedded one), the app version is  (appimage).

ASGARDEX desktop version: 1.21.6
Operating system and version: Trisquel 11 (Ubuntu based)

ive done some performance improvements on the asset page. A new release will be out shortly if the problem still exist let us know.


Option to interupt the task and give a new instruction

What problem or use case are you trying to solve?
Right now it always want to run things like a server or install the whole project via terminal.
It you should be able to interupt the Agent when giving a new instruction.
Describe the UX of the solution you'd like
Do you have thoughts on the technical implementation?
Describe alternatives you've considered
Additional context

Closing as a duplicate of adding a stop button which will effectively do the same thing. https://github.com/OpenDevin/OpenDevin/issues/2703

Will be fixed by https://github.com/OpenDevin/OpenDevin/pull/2602/ but takes a little time, waiting for other dependencies to go first.


XMLHttpRequest does not return status 200 for local files

The following does not work on a device with iOS 9.1 using cordova:
if (req.readyState === 4 && req.status === 200) fn(req.responseText);

in navigator.js line 394
This is because XMLHttpRequest does not return a status code when accessing local files.
One solution might be to omit the status check and check the responseText string length instead.

Thank you for reporting this problem.
Can you please tell me if it fixed this issue?
https://github.com/quark-dev/Phonon-Framework/releases/tag/v1.1.2
I updated the condition like so:
if(req.readyState === 4 && (req.status === 200 || !req.status && req.responseText.length))


[feature][array-type]support select ARRAY data type on vectorized engine

Proposed changes
Issue Number: close #7570
Problem Summary:

add column_array, data_type_array codes;
codes about data_type creation by Field, TabletColumn, TypeDescriptor, PColumnMeta move to DataTypeFactory;
support create data_type for ARRAY date type;
RowBlockV2::convert_to_vec_block support ARRAY date type;
VMysqlResultWriter::append_block support ARRAY date type;
vectorized::Block serialize and deserialize support ARRAY date type;

Checklist(Required)

Does it affect the original behavior: (No)
Has unit tests been added: (Yes)
Has document been added or modified: (No)
Does it need to update dependencies: (No)
Are there any changes that cannot be rolled back: (No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Can we do the select array after this PR is merged?
Could you give some sql example?

Can we do the select array after this PR is merged? Could you give some sql example?

yes.
sql example have added in this pr.


PathFollow 3D erratic rotations / wobbling / twitching

The object following pathfollow offset gets weird and erratic rotations but only on the first loop/lap. Tested on beta and RC 2.0 on linux systems.
Simple test inluded to confirm the issue.
followcurvetest.zip

Does this happen with the current master branch?

Using a build like a week old, the motion is similar to the original one, 3 points on the right side have a small "shake" when turning.
Here is the project for master:
curvetestv3.zip

That's not a bug in pathfollow3d. It's working as expected, following the tangent of the curve. If the curve tangent isn't smooth (as in the attached project), you'll get "erratic rotations".
The curve class can be changed to guarantee smooth derivatives/gradients or some helper could be added to the UI to guarantee that the tangent manifold is smooth maybe, but it's a lot work work for little gain. I'd just be careful when designing curves.

I know you can't really see the torsion on the UI but you can try to be extra careful about making it smooth

Well, I have just tested the sample project in both versions and that is what appears to be the problem mentioned in the OP, so you say that is an issue on the curve design then?
Too bad there is not possible to see the points on the inspector for fine tuning...

You can see the points and the curve itself quite well if you zoom in, but what matters is the torsion of the curve which is the second derivative.
Roughly speaking, if you see kinks or "sudden" changes in your curve when you zoom in, that's a bad sign.
Again, introducing a new type of curve based on a different type of polynomial which makes in hard to introduce large gradient changes would be a solution.

I'm on Godot v3.0.5 stable.
PathFollow2D rotation seems to be working properly.
PathFollow (3D) rotation is not working properly.
The behavior that I expect is that the rotation is perpendicular to the curve.
And the same offset would always produce the same rotation.
That is how PathFollow2D seems to work.
#14321 seemed to have fixed it for 2D.
Could 3D get the same treatment?

same with master build.
PathFollow node loop will result strange rotate.
I replace with OrientedPathFollow node,and it is work fine

I created a Path (3D) in capsule form and made a mesh move over it, using PathFollow.
The mesh has absolutely no rotation.
However, as you can see in the image, the mesh appears slightly rotated.
I do not know what causes this.
Any idea?

Closing the curve doesn't actually "Close" the curve, but instead places another point at the start point.
This causes the anchors to be seperate and unable to use the "Mirror Handles" options.
The main issue though is it causes a noticeable rotation jitter.
https://youtu.be/-4zX-wWV_L8

This bug report seems to be still valid due to the duplicate reported in #36258, but it needs an updated minimal reproduction project. Neither of the two projects posted above work in Godot 3.2.2.

This is a fixed example tested on 3.2.2beta4, the body glitches on turns and after one loop the rotation breaks completely.
curvetestv322.zip

I can still reproduce this on 3.3.2 and 3.4.beta c04521986, specifically this part:

the body glitches on turns and after one loop the rotation breaks completely.

https://user-images.githubusercontent.com/180032/124289886-179f0c80-db53-11eb-83c2-49a9549ecd64.mp4


[Question] Is custom configuration of mpv.conf available?

Since in my understanding this project uses libmpv as the backend is it possible to support custom configuration of mpv.conf?

This is a media playback library, not a media player. Let's have support for all platforms in 1.0.0 first.
The libmpv binaries that we're shipping are heavily stripped. Secondly, you can still implement a similar mechanism on your project.

Thankyou!
Yes i understand that this is a library, i was wondering because the PlayerConfiguration is limited and that's answered my question.

Not every feature from libmpv has been exposed in a common API, neither libmpv's exact API is kept, primarily becase:

libmpv's behavior changes across versions, this could break the usage & make things confusing for developers.
libmpv's API is substantially low-level as compared to Player class package:media_kit offers.
libmpv is just one of the backends that project has right now (because it's the best). On other platforms e.g. web where libmpv can't be used (or we decide to switch to something else), the features that we would have exposed wouldn't work as expected (because they're exclusive to libmpv). So, only a fine number of features which can be implemented across platforms have been exposed as part of Player class.

However, you can still easily modify properties/options specific to libmpv with following API:
final player = Player();
if (player.platform is libmpvPlayer) {
  await (player.platform as dynamic).setProperty('terminal', 'yes');
  await (player.platform as dynamic).setProperty('msg-level', 'all=v');
}


CERT-7874 Handle LLM key crash

https://certora.atlassian.net/browse/CERT-7874
Summary of Changes
Purpose:
Enhance the application's robustness by handling scenarios where the Anthropic API key (ANTHROPIC_API_KEY) is not configured. This ensures that dependent checks are gracefully skipped, and users are appropriately informed.

1. Quorum/checks/new_listing.py

Imported Configuration Module:

Added import Quorum.config as config to access configuration settings.

Conditional Execution in NewListingCheck:

Before Performing First Deposit Check:

Check for Anthropic API Key:

If ANTHROPIC_API_KEY is not set in the environment:

Print Warning:
Displays a warning message indicating that the first deposit check is skipped due to the missing API key.
Skip Further Processing:
Exits the new_listing_check method early to prevent executing dependent checks without the necessary API key.

Impact:
Prevents the tool from attempting to perform operations that rely on the Anthropic API when the API key is not available, thereby avoiding potential errors and informing the user of the skipped functionality.

2. Quorum/config.py

Imported Pretty Printer:

Added import Quorum.utils.pretty_printer as pp to utilize the pretty printing utility for displaying messages.

Configuration Validation for Anthropic API Key:

After Retrieving ANTHROPIC_API_KEY:

Check for API Key Presence:

If ANTHROPIC_API_KEY is not set:

Print Warning:
Displays a warning message informing the user that the ANTHROPIC_API_KEY environment variable is not set and that all dependent checks will be skipped.

Impact:
Alerts users during the configuration phase if the Anthropic API key is missing, ensuring they are aware that certain features will not be available until the API key is provided.

Overall Benefits:

Improved User Experience:
Users receive clear and immediate feedback about missing configurations, preventing confusion and unexpected behavior.

Enhanced Stability:
By conditionally skipping dependent checks when necessary, the application avoids runtime errors and maintains smooth operation.

Maintainable Codebase:
Centralizing the API key check in the configuration and conditionally handling it in dependent modules promotes cleaner and more maintainable code.

Example Warning Messages:

From config.py:
Warning: ANTHROPIC_API_KEY environment variable is not set. All dependent checks will be skipped.

From new_listing.py:
first deposit check is skipped. If you have an LLM API key, you can add it to your environment to enable this check

This comment will trigger CI?


Add target option to name project when created

This adds a dialog that pops up when a new project is created using the big button in the home screen. We've discussed that with David a few months back. This is controlled by appTheme option (to be enabled for Arcade).
With the new settings namespace, the storage resets when the name changes, and also stays on device when the name is the same on two programs. Because of that it's more important now to name projects.
If you hit "Create!" with no name, you get "Untitled".

I would remove 'cancel' button, they have close (x). What is the behavior of 'x'
I love your determination @Mmoskal.

We are moving away from cancel button in all other dialogs

Cancel is the same as X. Will remove

Looks great.  I think this feature has been on the backlog for almost 2 years now...


Implement support to rbl operator into libmodsecurity

Operator rbl needs to be implemented inside libmodsecurity.
The documentation of the operator can be found here:
https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#rbl
The file already exist in the repository:
https://github.com/SpiderLabs/ModSecurity/blob/libmodsecurity/src/operators/rbl.cc

Support added by: 049e4eb69df4d3f7cd17d0a5c853b132b1e239bb


[readme] Didnt found exemple code for Planetception

hi,
the link on the readme :
https://github.com/innFactory/react-planet/blob/master/src/stories/Planet.stories.tsx
Current behaviour:

Excpected behaviour : have a link  to see the code please 👍

Thank you. The link is: https://github.com/innFactory/react-planet/blob/master/src/stories/Planetception.stories.tsx
It's fixed in the readme.


:seedling: Unit tests fileparser/listing

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[x] PR title follows the guidelines defined in  https://github.com/ossf/scorecard/blob/main/CONTRIBUTING.md#pr-process

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?
Unit tests fileparser/listing
https://github.com/ossf/scorecard/issues/986

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Codecov Report

Merging #1607 (66a75af) into main (8733080) will increase coverage by 0.28%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1607      +/-   ##
==========================================
+ Coverage   54.67%   54.96%   +0.28%
==========================================
  Files          72       72
  Lines        6257     6257
==========================================
+ Hits         3421     3439      +18
+ Misses       2617     2595      -22
- Partials      219      223       +4

Thanks! Do you want to also test the error type? There's a function under utests/ called errCmp() for it. It's not exported but we could export it?

I will do that as a separate PR. Thanks


DNU tools only work on windows

DnuTool.cs defines only DNU.cmd as the tool executable name.
protected override IEnumerable<string> GetToolExecutableNames()
        {
            return new[] { "dnu.cmd" };
        }

This doesn't work on unix/osx.
Error: DNU: Could not locate executable.

I can fake it if i add the ToolPath manually in DNURestoreSettings in my build.cake, but this is not very portable :)
Task("Restore")
    .IsDependentOn("Clean")
    .Does(() =>
{
    DNURestore(new DNURestoreSettings { ToolPath = "/Users/acunningham/.dnx/runtimes/dnx-mono.1.0.0-rc1-final/bin/dnu" });
});

A possible solution might be to look in $PATH for dnu executable, since dnvm adds the current runtime to the $PATH...
acunningham@ACMBP $ dnvm install 1.0.0-rc1-update1
dnx-mono.1.0.0-rc1-update1 already installed in /Users/acunningham/.dnx
runtime=mono
Adding /Users/acunningham/.dnx/runtimes/dnx-mono.1.0.0-rc1-update1/bin to process PATH

Actually, looking at https://github.com/cake-build/cake/blob/develop/src/Cake.Core/Tooling/Tool.cs#L236, it seems all you'd need to do is add "dnu" to GetToolExecutableNames()... i'll submit a PR.

Merged via #690.


Update Readme

Readme can be updated with details of mentors,admins,slack channels and details of gssoc. Please assign me this issue.

@Sapna2001 I don't think details mentors,admins,etc etc is needed in readme. Mentors are permanent for the project. This is a normal project as other are there on GitHub. It is just a part of GSSOC20


Prestissimo composer plugin

It might be very possible to speed up, depending on servers bandwidth, composers vendor download.
The composer plugin "prestissimo" enables parallel downloads of vendors which should speed things up noticeably.
Ref:
https://github.com/hirak/prestissimo

Commit 90c367f15fe7cbc2d2c9b2142262c3f12606173b rewrites the Pockethold class, adding the backbone of the Prestissimo approach. HTML/JS needs a complete overhaul as well, coming shortly.

Looks like this might not be a viable option. (I am not skilled enough it seems to install and use a composer global plugin to be used with a non-installed composer).
I will leave the ticket open, and after next pockethold release, I will experiment more on the side.

Composer is now bundled with pockethold. Maybe I can manage to get this working now, though I am doubtful :( (In beta8-dev branch).
I would be glad for any input from more versed larvel/composer developers.


Update npm badge to go to correct location

closes #58

Pull Request Test Coverage Report for Build 168

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 167:
0.0%

Covered Lines:
175

Relevant Lines:
175

💛  - Coveralls


Print amount of space freed

I was wondering if there is a way to integrate df into the script to print a simple line at the end of the script that reads something like this:
...
Cleaned up 3GB of space
Everything is cleaned up :3

I will close this issues since @kamil98 added the feature. Thank you.


Channel mapping not updated with the playlist

Describe the bug
Creating a custom filters in order to do channel mapping works so far so good.
The problem is, once the playlist is updated the channel mapping is not. I.e. I have to delete the filter first and re-add it and do the ALL the mapping again if a channel in the M3U playlist decided to change position. (e.g. provider adjusts channels)
Server (please complete the following information):

Linux - Ubuntu 22.04
x86_64
2.2.0

To Reproduce
Steps to reproduce the behavior:
0. Move the entry of a channel in the m3u playlist (e.g. ZDF HD)

Update the playlist in xTeve
curl http://xteve:34401/m3u/xteve.m3u | grep -A 1 "ZDF HD"
Grab the "/stream/ac6da6e89c5c8f09316cb53d58f951a0" and open the stream, i.e. another curl call
Notice the wrong stream is called.

Expected behavior
The updated entries in the m3u would be updated with the according title of the program.

Thats more of how your provider is handling channel edits, instead of renaming they are deleting and adding as new.  Making it appear as a new channel instead of an old channel with a new/same name.
If they offer xtreamcodes, epg.best can help but it also depends on how they operate their updates


Compatibility with factorio 0.17

Sorry to bother you but I wanted to ask if you planned to look into supporting version .17 of factorio. OutpostPlanner truly is an excellent and useful mod, so if anything, you have my thanks for all the work that you previously put on it.

Thank you for the kind words!
I have just finished updating the mod, I'm going to upload it to the mod portal now :)
It looks like it's all working with just a few minor changes, but I'm going to be pretty busy for a couple of weeks now so if something is badly broken I might not be able to fix it for a while - use at your own risk!

This is awesome, thank you so much for taking the time to update it. I'll report back if I see anything but don't worry too much about it if so ;)


Bloated JXL Output/Final Output Size Not Matching Preview Estimate

Describe the bug
When converting to JXL, the preview estimate shown while adjusting settings will give several hundred kBs or lower (depending on quality and effort) for a lossy conversion, but then the resulting file is several MBs or larger when viewed in a file browser.
To Reproduce
Batch convert
Select large image (several MBs)
Select JXL
Choose Lossy
Lower Quality and raise Effort until preview estimate ~250kBs or less
Save
Check file in file manager
Expected behavior
Output size should match preview, or at least be smaller than input.
Screenshots

(Ignore the ghosting on the stitched long screenshot)
Smartphone (please complete the following information):

Device: Ocean (Moto G7 Power)
OS: Lineage 20 (Android 13)

For comparison, I do not have this issue with this app: https://play.google.com/store/apps/details?id=ebusky.jpeg.xl.image.viewer.converter

i will check 😃

Thanks! Also, the input file size is only 2.5MB! Not sure how it got so large, then. (12MP, if that has anything to do with it: 3024 x 4032)

Maybe that my fault, and i'm passing wrong args, but if not, that's may be related to libjxl or coder which i use

Hopefully it's an easy fix. I found this, because I'm trying to help with getting JXL support into Fossify Gallery and I'm helping QuackDoc test his PR. It seems that all viewing apps suffer from the decoder (my guess), because all of them show a lower-resolution version than the actual file. The only one that seems to show it correctly is https://github.com/oupson/jxlviewer except it doesn't zoom in past 100% and isn't a gallery app.
So, there could be something funky with the libjxl implementation for Android, in general. Maybe something changed recently, because the PlayStore app I linked was last updated in Feb. of this year.

Try to open it in image preview screen

Is that the +Magnifier button in the upper right (like in the second screenshot)?

It's in the third section

Got it.
Well, that's weird. I imported the JXL of the flowers and the JPG original and the JXL looked crappier. But, after going back and forth, then it looked fine at some point? Hmmm.
But, weirdly, one other file that I converted twice with this app had one that looked just as bad as I described, while the first attempt now looked fine(ish).
...okay, hold up. I just imported the original for that other image just to see (since I couldn't remember if I did anything different with the settings between the two conversion attempts) and I didn't notice anything different between it (the original) and the first attempt. So, I  checked the second attempt and...it looked just as fine as the other two. It definitely looked way worse before. And this is all in one session (I didn't exit out of the Image Preview during making this post).
That seems like it might be the decoder, after all. 🤔

interesting indeed, i have no idea what's wrong 🧮

Well, at least it's noted. But, I'll leave that aspect for the upstream parts, like Awxkee's coder and libjxl proper.
Thanks for bearing with me, regardless.

Theres update in libjxl and jxlcoder, maybe that will help

Okay. I'm using Obtainium. So, as soon as the new binary releases for ImageTooolbox, I'll test it out and let you know if it's working for me.

I think I need to either rename this issue or create a new one, because it isn't just JXL that's having this problem. I tested converting to MozJpeg, WebP, and Avif and all of them came out much larger than the preview file size claimed (I think it even low balls the input size, saying a 2.2MB PNG was 1.0MB). MozJoeg also came out massively bloated compared to what it should be. That 2.2MB PNG turned into a 597KB Jpeg when converting using Aves Gallery (which uses the system's built in libjpeg-turbo) at Q=93, while ImageToolbox output a 924KB Jpeg when using MozJpeg at Q=85! And the preview estimated it would be around 171KB.
Update: just tried converting that PNG to Jpeg (using the Jpg setting) in ImageToolbox using Q=93 and it output to 1MB (preview said 302KB), which is double what Aves output.

@T8RIN
Just updating to say that this issue is not resolved. I tested with the latest pre-release/alpha apk (FOSS version) and a 3MB Jpeg was turned into a 5MB JXL (Lossy at 80% quality and 7 Effort), while claiming the sample/preview size was ~97KB.
I also just tested it with Jpeg->Jpeg at 100% Quality and the result was it bumped it up from 3MB to 3.3MB...while claiming the preview size to be 579.6KB.
So, the preview estimate numbers seem to be broken across the board. And I've tested outputing to Jpeg, WebP, JXL, Avif, and MozJpeg, with only Jpeg and Avif not producing significantly bloated files.
(Just tested again with Avif and the numbers do seem to still be off, but not by nearly as much: 34.5KB for preview, 39.3KB for actual in one test)
(Tested that same image with JXL and got 66.4KB for preview, 610.6KB for actual...)

I've been using another app I found before yours to handle JXL conversions (this one) and I noticed similar bloating started happening with its outputs, as well. I realized that I had set it to RGBA, instead of the usual RGB. So, when I converted a file using one setting and then the other, I noticed the RGB one was really small and the RGBA one was larger (1.6MB) than the original PNG image (1.3MB).
I thought this must be what was going on in ImageToolbox, so I did the same test with the same image, but using this app. And...still quite large. It wasn't as big as the 1.3MB PNG, coming out at 842KB. The difference between RGB and RGBA from ImageToolbox was 0.3KB (842.2KB vs 842.5KB).
I'm not sure what the difference between the apps is, but your guess about the arguments being fed to the encoder might be it. I hope so, at least.
Again, this is using the Lossy mode only, not Lossless.

Just wanted to post here to confirm that on the latest build (51 FOSS RC) MozJpeg, WebP, and JXL and no longer producing bloated files and the difference between the estimate and actual output is only a few kilobytes and is sometimes lower than the estimate!
I know it's been "closed" or a while, but I can now officially considered it resolved.
Thank you for whatever you did!

Thanks for using 👀


Avoid error when a not yet handled elements is in the source

for instance when there is a rest element which is not yet handled, we store the json string corresponding to it and returning it back
fixes #500

it requires to add tests. i think we should add for rest but also for all other elements not yet vizualized, with a specific entities, like restconfiguration, routeConfiguration
I initially thought to provide a specific identity for the rest element but it would have left apart all the other cases. So sounds safer to have a generic entity to handle all elements that we just do not want to lose but for which we are not providing a vizualization

I added a fix for the tests: https://github.com/apupier/kaoto-next/pull/1

Provided branch from Ricardo is working. i was hitting a different issue, which was already present. this issue is that routes using top-level from and which does not have a steps element (so that are incomplete route) are not displayed in the canvas.
So it remains to provide new tests

@apupier is there something that I can lend a hand with? :smiley:

is there something that I can lend a hand with? 😃

it remains to provide new tests


How to update?

Hi all, I faced such a problem that I can not update npm when entering the combination npm install comes up a bunch of errors (you can see on the screenshot).

I think the bot needs to be updated. I also encountered many errors and I am using a different bot now. The bot developer has already removed the files. I hope they update it soon.

updated


Add missing <utility> include

optimization.h uses std::unreachable() if available but does not include <utility>, causing errors in absl/strings/ascii.cc.

I see this was reverted because it needs to be compilable as C. Looks like somebody edited the commit to make the include unconditional (6f972e2), I think the original should work as is?

We will handle this.

Now fixed in https://github.com/abseil/abseil-cpp/commit/a305e859b6d0abe6d88a3fe12c6891152cc6e859. Thanks!

Great, thanks!


Removing Mock server details from API base URL

Describe the bug
When I am adding the server details, the mock server details are being added by default. Can you help me with a way to remove these mock server details altogether from my public APIs.
To Reproduce
visit https://www.nutanix.dev/api_references/era/#/YXBpOjE2MjkwNjE5-api-reference

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (remove any that are not applicable):

Library version: [e.g. 3.0.0]
OS: [e.g. Windows 7]
Browser: [e.g. Chrome 61]
Platform: [desktop or web]

Removing mock server is a feature enhancement. An issue here is fine but can you also please add your insights here: https://roadmap.stoplight.io/c/160-show-hide-mocking-in-try-it
The default is the live server though.

Hi Nauman,
Currently our customers are seeing both Live servers and Mock servers. My requirement is to remove mock server details from the final published documents and keep it for internal testing purpose ONLY. So, basically we need to default the server as Live server and not mock server.
The details mentioned in the https://roadmap.stoplight.io/c/160-show-hide-mocking-in-try-it [roadmap.stoplight.io]https://urldefense.proofpoint.com/v2/url?u=https-3A__roadmap.stoplight.io_c_160-2Dshow-2Dhide-2Dmocking-2Din-2Dtry-2Dit&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=wIG1wHVnQNmSvu6d3Pe2qipPr9Ii25bI1HqHiOwDq7G-tt2raQLYevdLLi04JZev&s=ZkZSA32GWRBAeuLKBs4QJaVejomWzMQW_TpgMlhU_mg&e= seems to be different. I am not able to edit them.
Please edit them then raise for an enhancement request. Also, please share the timelines for this request to be fixed. We need to make our customers aware of the same.
Thanks.
From: Nauman @.>
Sent: 01 March 2022 00:48
To: stoplightio/elements @.>
Cc: Smita Bharadwaj @.>; Author @.>
Subject: Re: [stoplightio/elements] Removing Mock server details from API base URL (Issue #2042)
Removing mock server is a feature enhancement. An issue here is fine but can you also please add your insights here: https://roadmap.stoplight.io/c/160-show-hide-mocking-in-try-it [roadmap.stoplight.io]https://urldefense.proofpoint.com/v2/url?u=https-3A__roadmap.stoplight.io_c_160-2Dshow-2Dhide-2Dmocking-2Din-2Dtry-2Dit&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=wIG1wHVnQNmSvu6d3Pe2qipPr9Ii25bI1HqHiOwDq7G-tt2raQLYevdLLi04JZev&s=ZkZSA32GWRBAeuLKBs4QJaVejomWzMQW_TpgMlhU_mg&e=
The default is the live server though.
—
Reply to this email directly, view it on GitHub [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_stoplightio_elements_issues_2042-23issuecomment-2D1054581240&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=wIG1wHVnQNmSvu6d3Pe2qipPr9Ii25bI1HqHiOwDq7G-tt2raQLYevdLLi04JZev&s=4LqsUwDNKNJZeIXJd9qiK4r6cX7WwRbwgvOgBRMog9c&e=, or unsubscribe [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AP6VKLTQORMXHOI547EYYW3U5ULWNANCNFSM5PDXC6YA&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=wIG1wHVnQNmSvu6d3Pe2qipPr9Ii25bI1HqHiOwDq7G-tt2raQLYevdLLi04JZev&s=MNPXvmLCtLP549wcfj3UxVhsE2XCN8mlSFiTkw9-GJo&e=.
Triage notifications on the go with GitHub Mobile for iOS [apps.apple.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__apps.apple.com_app_apple-2Dstore_id1477376905-3Fct-3Dnotification-2Demail-26mt-3D8-26pt-3D524675&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=wIG1wHVnQNmSvu6d3Pe2qipPr9Ii25bI1HqHiOwDq7G-tt2raQLYevdLLi04JZev&s=oGuI2LLC-e7Hayz-5fHHlPSr0jwAo07a6XgrovOjUpc&e= or Android [play.google.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__play.google.com_store_apps_details-3Fid-3Dcom.github.android-26referrer-3Dutm-5Fcampaign-253Dnotification-2Demail-2526utm-5Fmedium-253Demail-2526utm-5Fsource-253Dgithub&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=wIG1wHVnQNmSvu6d3Pe2qipPr9Ii25bI1HqHiOwDq7G-tt2raQLYevdLLi04JZev&s=y5XkfhAnY4oJ-0UluhlncYAVSxakOfajOr8g4u0iT-g&e=.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

@allegra-rosenbaum My bad! Considering you are using elements you should be able to hide the hideMocking option. https://meta.stoplight.io/docs/elements/ZG9jOjMyNjU4OTU3#configuration to hide any mention of the mock server.

Thanks Nauman,
I tried setting this up, but then the Live server details also vanishes once we set the “hidemocking” true parameter in our html code.
So we have a project where we have set the hidemocking true. Here is the link of the API page– https://www.nutanix.dev/api_references/karbon/#/b3A6NDM0MDU2MjE-update-the-certificates-used-by-the-various-components-of-the-k8s-cluster, both the mockserver and live server vanishes from the send API request section(see red arrow below).
@.***
Once I remove the hidemocking: true value, both the live server and mock server shows up.
Am I setting something wrong?
Thanks,
Smita,
From: Nauman @.>
Sent: 02 March 2022 21:52
To: stoplightio/elements @.>
Cc: Smita Bharadwaj @.>; Author @.>
Subject: Re: [stoplightio/elements] Removing Mock server details from API base URL (Issue #2042)
@allegra-rosenbaum [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_allegra-2Drosenbaum&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=8lD8iAIoSgQHALK5MAy5_GBRhSJi3aAgbwQ3s2O5hh9cNQPtl_4iS31IJE6C83EX&s=rE3DoLiirSqA9GTTTnW5ayGd9OFhlUz5PLKHV0_h4t8&e= My bad! Considering you are using elements you should be able to hide the hideMocking option. https://meta.stoplight.io/docs/elements/ZG9jOjMyNjU4OTU3#configuration [meta.stoplight.io]https://urldefense.proofpoint.com/v2/url?u=https-3A__meta.stoplight.io_docs_elements_ZG9jOjMyNjU4OTU3-23configuration&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=8lD8iAIoSgQHALK5MAy5_GBRhSJi3aAgbwQ3s2O5hh9cNQPtl_4iS31IJE6C83EX&s=cHE1RNcv-xQDIJ1GVTpKl6rQV1Udpo12vHHSjxxIJTU&e= to hide any mention of the mock server.
—
Reply to this email directly, view it on GitHub [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_stoplightio_elements_issues_2042-23issuecomment-2D1057113081&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=8lD8iAIoSgQHALK5MAy5_GBRhSJi3aAgbwQ3s2O5hh9cNQPtl_4iS31IJE6C83EX&s=yU_TwHOJAYIFy4t_m9PHNEmrFl2yYvHCQkHfYPqP9v0&e=, or unsubscribe [github.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AP6VKLRMCBKUHB55FOUJWVLU56IR3ANCNFSM5PDXC6YA&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=8lD8iAIoSgQHALK5MAy5_GBRhSJi3aAgbwQ3s2O5hh9cNQPtl_4iS31IJE6C83EX&s=6x6elz7t_SPkwTgKKzka31bbSoDf_KRExdbG_DkC2ys&e=.
Triage notifications on the go with GitHub Mobile for iOS [apps.apple.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__apps.apple.com_app_apple-2Dstore_id1477376905-3Fct-3Dnotification-2Demail-26mt-3D8-26pt-3D524675&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=8lD8iAIoSgQHALK5MAy5_GBRhSJi3aAgbwQ3s2O5hh9cNQPtl_4iS31IJE6C83EX&s=TF4rmfp20HusjkHYg66lo5i8RuqEUWRDb1wlOMLwOGM&e= or Android [play.google.com]https://urldefense.proofpoint.com/v2/url?u=https-3A__play.google.com_store_apps_details-3Fid-3Dcom.github.android-26referrer-3Dutm-5Fcampaign-253Dnotification-2Demail-2526utm-5Fmedium-253Demail-2526utm-5Fsource-253Dgithub&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=UbNX3gjLAGfqIeW034NRCo6TFSpLuMGs8wS2LOBPGcg&m=8lD8iAIoSgQHALK5MAy5_GBRhSJi3aAgbwQ3s2O5hh9cNQPtl_4iS31IJE6C83EX&s=yk0rOk74wKtcQ4pHAsMM99qKNPS-Xxo6eiun6noQb1k&e=.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


What should I do with the file which exceeds 256000 characters?

I am uploading my secret file which is exceeding 25600 characters:
az keyvault secret set --vault-name mykeyvault --name mycreds --file mycreds.yml
And getting the following error:
Secret is beyond the maximum permitted length of 25600 characters.

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

ID: 39cfb8ed-c246-2ffb-9dff-070b90c18b21
Version Independent ID: c09c1597-1b6c-b0a2-b3fc-bdc16764d096
Content
Content Source
Service: key-vault

The simplest work around would be to split the file and store as multiple secrets.  If this isn't possible or undesirable, then I would suggest encrypting the file with a symmetric key, store the encrypted file outside of the vault, and store the symmetric key as a secret in the vault.

if you split it you might get back a corrupted file

see https://www.synopsys.com/apps/support/est/How_to_split_files_over_2GB.html

usage of split and cat and sha256 will prevent having a corrupted file


added support for oauth2 audience

Delomboking is done .. you need to rebase this now .. sorry .. also might want to wait a bit since we are about to get the checkstyle refactor in as well hopefully

Should all be good to rebase now .. lots of refactoring got in

Is this replaced by #164 and we can close this PR @andythsu ?

yes I'll close it


arbitrary file type selectors

Hi,
I'm trying to use this with the ROCM HIP/Thrust code base from AMD, but it seems to be having issues with sources like "ABC.inl"
I'm trying to configure a custom selector, but modifying the default to include source.inl |  but I'm not sure it's enough
any tips would be appreciated
thanks

In-line include files are usually not compible as standalone compilation units. The design of clangd requires this, so it won’t work well with those type of files.

it appears to propagate an issue to the my original source which includes it indirectly, is there someway to disregard errors in unrelated files, assuming I'm not using any symbols declared within?

What is the error you're seeing?

I'm seeing a few, but I have a feeling that one is caused by the other other
Thrust from AMD has some optional CUDA compatibility, which I am not using, yet clangd LSP is saying it can't find some cuda related header (cub specifically). When I compile normally, I do not get such an error.
It's quite possible I'm missing some defines in my .clangd file which the hipcc compiler expects.
The other error I'm seeing, which I don't really get, and it only appears in thrust headers, is cstddef header not found
but I can include it my own source and LSP shows me the correct gcc header

I don't really want to have LSP analyse .inl files, but are they still used as part of the AST for other files?

Also, since I've switched from compile_flags.txt to .clangd
relative includes from the project root directory no longer work, as I'm guessing, the working directory is changing to the source file directory, when the relative imports are no from there, maybe this could be separate issue, or you have some idea to support relative includes from the root project directory (where .sublime.project... are)


Corrected indexation in TS

Corrected IDE's indexation. In src/main.js or main.ts 8th line was marked as "unused", same with "app" class in index.html
On top of that, there was warning that /src/main.js or main.ts wasn't found by given path.

I think we want to keep it as is for now as all the template starts with / too, and it seems to be an IDE error only. Closing this for now to keep the current way.


Publish Tofu-Jet

Tofu-Jet is a generator that converts Terraform providers into TF-controller's primitive modules.
This ticket is to track the progress of opensourcing Tofu-Jet.

cc @steve-fraser

Looking to use it for Azure resource :)

Hello @chanwit! Do you have any timeline for when this will be open-sourced?

Unfortunately @akselleirv
We de-prioritized this feature and now are spending time to implement other features at the moment.

The generated modules are available here: https://github.com/tf-controller/aws-primitive-modules


Missing Headers in MultipartFormData

Moya version 14.0.1
What i do
let formData = MultipartFormData(provider: .data(data), name: name, mimeType: "text/plain")

What I expect
Name: parentObjType
Headers: content-disposition: form-data; name="parentObjType"
content-type: text/plain
content-transfer-encoding: 8bit
Data:
INTERACTION

What i get
Name: parentObjType
Headers: content-disposition: form-data; name="parentObjType"
content-type: text/plain
Data:
INTERACTION

The question is - How to add missing header ? content-transfer-encoding: 8bit

You could add this header in the headers property on your struct or class that's implementing TargetType.


Update versionconfig.html

Delete line 3 to remove Version banner:
{% assign versionBanner = "version_old" %}

PR build failed so deleting PR.


Not working for parent class @Subscribe methods.

Hi RxBus support!
I am having issues with parent class methods annotated with @Subscribe.
For example I have 2 presenter class.. BasePresenter and MainPresenter (which extends BasePresenter).
When I register with RxBus.register(this) methods annotated with @Subscribe in BasePresenter is not registered.
Could you give some suggestions how to implement this???
Thanks!

Sorry for bothering! I have already found solution in https://github.com/AndroidKnife/RxBus/issues/13
Thanks!


Single sign-on integration

Enable authentication via the single sign-on identity service.

Done by the new ckanext-openidconnect extension.


⚠ Avatar API Endpoint has degraded performance

In e225d9b, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 2051 ms

Resolved: Avatar API Endpoint performance has improved in 20e81d4 after 12 minutes.


GEODE-8742: fix list gateways command when dispatcher-threads is set …

…to 1
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

It's possible that these changes may overlap/conflict with https://github.com/apache/geode/pull/5770. It could be a good idea to rebase off that branch and confirm that the issue these changes are intended to fix is still present with the above PR in place.

It's possible that these changes may overlap/conflict with #5770. It could be a good idea to rebase off that branch and confirm that the issue these changes are intended to fix is still present with the above PR in place.

Hi @DonalEvans ,
I tried with this PR and the issue still exist.
With this change issue dissapear.
There is some test failing, I'm trying to fix it.

The new PR is opened #5803.


Consider adding defaultValue to @Argument

Similar as for traditional controllers the @RequestParam annotation supports a defaultValue, it would be nice to have this for the GraphQL @Argument annotation as well.
Rationale:
Imagine this Query definition:
seasons(first: Int = 10, offset: Int = 0): SeasonPage!

here default values are specified in the GraphQL schema, and if not provided by the user the defaults are used automatically.
But the above Season type could have the following field:
type Season {

   standings(first: Int, offset: Int = 0): StandingPage!
}

here the first argument has no default value in the GraphQL schema, basically meaning there is no limit and by default all standings are returned unless the limit is explicitly set by the client.
Now having a SchemaMapping for this like:
@SchemaMapping
public Page<Standing> standings(Season season, @Argument int first, @Argument int offset) { .. }

will fail as null is an illegal argument for the first argument.
It would be nice to define the SchemaMapping like:
@SchemaMapping
public Page<Standing> standings(Season season, @Argument(defaultValue = "99999") int first, @Argument int offset) { .. }

See the arbitrary 99999 as default value which in this proposal will be used when the argument is not passed, or set to null by the client.
I don't want the 99999 in the GraphQL schema itself as it is an arbitrary value without a real meaning,
unlike offset = 0 or first = 10 in the query mapping.
Without a defaultValue I can solve it on other ways to:

Like mentioned above, add an arbitrary value like 99999 to the GraphQL schema. But even then a client could pass null which will break the controller
Change the code to @Argument Integer first and test myself on null; this will also make sure the controller won't break.

Why not add the possibility of doing this:
@QueryMapping
public long idLongOrDefault(@Argument Optional<Long> id) {
  return id.orElse(-1);
}

So as to allow Optional<T> to be argument values, including OptionalInt, OptionalLong, and OptionalDouble for performance.
Spring Data only allows Optional<T>
Your proposal would introduce extra complexity, since there would be two places to define default values (the schema, and in your code).

@jord1e yes it's indeed in 2 places, but with different behaviour.
The one in the schema is public user facing, the one in the controller is more of a technical default value. But I agree it could be confusing... nevertheless I would use it if available ;-) (it would also be consistent with @RequestParam).
As I said I can easily workaround it by using Integer or maybe better Optional<Integer> as you mentioned.
Thinking of it, would it maybe be possible to use custom argument resolvers in GraphQL controllers?
I could introduce a custom Pageable (not to confuse with Spring Data's one) that holds the first and offset arguments and would globally assign default values if not provided...

Thinking of it, would it maybe be possible to use custom argument resolvers in GraphQL controllers?
I could introduce a custom Pageable (not to confuse with Spring Data's one) that holds the first and offset arguments and would globally assign default values if not provided...

This is currently not possible, the resolvers are hardcoded.
https://github.com/spring-projects/spring-graphql/blob/5919178d9157527ae8911709d219237ca17a3b1c/spring-graphql/src/main/java/org/springframework/graphql/data/method/annotation/support/AnnotatedControllerConfigurer.java#L122-L148
Better yet
type Query {
    test(id: Int): Int!
}

@QueryMapping
public int test(@Argument Optional<Integer> id) {
    return id.orElse(-1);
}

works
{ test(id: -2) } > -2
{ test } > -1
See
https://github.com/spring-projects/spring-graphql/blob/5919178d9157527ae8911709d219237ca17a3b1c/spring-graphql/src/main/java/org/springframework/graphql/data/GraphQlArgumentInitializer.java#L93

This feature not listed in the documentation:
https://docs.spring.io/spring-graphql/docs/current-SNAPSHOT/reference/html/#controllers-schema-mapping-argument

Thx for analyzing this 👍
So besides my proposal for defaultValue on @Argument (and questionable if desired per your comment) it would also nice to open up AnnotatedControllerConfigurer to be able to define custom argument resolvers ;-)

Explaining the bigger picture for my case and what I'm trying to achieve in a more declarative way:
In my GraphQL schema I have several collections fields which all return a Page containing the actual items and pageInfo.
Like:
type Query {
    seasons(first: Int = 10, offset: Int = 0): SeasonPage!
    teams(first: Int = 10, offset: Int = 0): TeamPage!
}

type Season {
    standings(first: Int, offset: Int = 0): StandingPage!
}

So it the Query type I have some top level collections which by default do paging with 10 items per page. Note I also want to limit it with a max value of 100 items per page.
The standings field within the Season will return by default all standing items, but clients could limit it only to 1; e.g. to display multiple seasons with just the champion.
I have many more similar collections and I want to avoid repetitively having to code checking for null, min value, max value etc.
I would love to define a custom argument handler like:
@QueryMapping
public Page<Season> seasons(@PageableDefault(first=10, maxFirst=100, offset=0) Pageable pageable) {
    ..
}

@SchemaMapping
public Page<Standing> standings(
        Season season,
        @PageableDefault(first=Integer.MAX VALUE, maxFirst=Integer.MAX VALUE, offset=0) Pageable pageable) {
    ..
}

Note that the Integer.MAX VALUE could also be used as default values in my @PageableDefault to avoid having that.
My my names used above are confusing as it is basically a copy from Spring Data....
but it at least points outs what I'm trying to accomplish is on a similar level already possible with MVC and Spring Data.

Just a clarification question for now. In the snippets, the query returns SeasonPage but I see no definition for a type called that. Instead, you have a type called Season. Is that meant to be SeasonPage, i.e. a typo?

Besides the GraphQL Season type I have also a SeasonPage type:
type SeasonPage {
    items: [Season!]!
    pageInfo: PageInfo!
}

type PageInfo {
    hasNextPage: Boolean!
    hasPreviousPage: Boolean!
    totalCount: Int!
}

Note this looks similar as the Relay connection spec, but it is not. It uses a simpler limit/offset paging model sufficient and more easier in my case.
If you would like I can share more of the schema.

It would be nice to understand Spring's team take on this (without having an immediate implementation).
That way I could choose a current approach for my project that that suits future directions of spring-graphql best.

There are several requests in this, so let me start with the original question only for now.
Initially we had both required and defaultValue attributes on @Argument, mirroring @RequestParam, but dropped those in #150 because we saw no reason for a duplicate mechanism that creates confusing choices. It also makes sense that clients should be aware of such constraints.
You raise a good point that a default value in the schema does not seem to apply to null and it's tempting to bring back defaultValue on @Argument, as it would serve a real purpose. Nevertheless, it still encourages either duplication of the default value, or not exposing it at all. I think neither is good a choice.
Perhaps we can consider some boolean flag on @Argument to ignore nulls, and that would cause the resolver to fall back on the schema-defined default value, which seems accessible through the GraphQLFieldDefinitionfromDataFetchingEnvironment`.

Perhaps we can consider some boolean flag on @Argument to ignore nulls, and that would cause the resolver to fall back on the schema-defined default value, which seems accessible through the GraphQLFieldDefinitionfromDataFetchingEnvironment`. What do you think?

Do you mean:
type query {
  a(myArg: Int = 5): Int!
}

with a resolver
@QueryMapping
public int a(@Argument(fallbackIfNull= true) int myArg) {
  return b;
}

Which would return
The value of myArg when it is not-null
and
5 when myArg is null
?
I would strongly favour just using Integer, or adding an argument resolver for OptionalInt. I really don't find this an ideal situation, why would you want this in the first place?.
There is even a difference between null and not specifiing the argument at all. This is being discussed in #140 .
As far as I am aware no other JVM or GraphQL framework implements this functionality.

Thx @rstoyanchev for replying.

You raise a good point that a default value in the schema does not seem to apply to null and it's tempting to bring back defaultValue on @Argument, as it would serve a real purpose. Nevertheless, it still encourages either duplication of the default value, or not exposing it at all. I think neither is good a choice in the general case.

Clients passing null is indeed a nasty issue at the moment for number types in the GraphQL schema... at least when defined as int in the controller methods.
I would almost tend to always use Optional<Integer> or as @jord1e mentioned OptionalInt (but I believe the latter is not supported at the moment).

Other question (and yes I understand in this threads many questions / alternatives are discussed 🙈 ) what is your take on the ability to provide / register custom custom argument resolvers as in normal MVC controllers?
E.g. to do something like:
@QueryMapping
public Page<Season> seasons(Pageable pageable) { .. }

or even:
@QueryMapping
public Page<Season> seasons(@PageableDefault(first=10, offset=0) Pageable pageable) { .. }

Custom argument resolvers is just something that hasn't come up but nothing against it. It does feel though like @Argument should be able to support creating a Pageable in this case. The capability is there and where ArgumentMethodArgumentResolver calls GraphQlArgumentInitializer, if it passed no name, the target object would be initialized from the full arguments map. This is already how it works for @ProjectedPayload and for Query by Example. So just a matter of having a way to express that this is what you want, and it should be easy to do.

Custom argument resolvers is just something that hasn't come up but nothing against it. It does feel though like @Argument should be able to support creating a Pageable in this case. The capability is there and where ArgumentMethodArgumentResolver calls GraphQlArgumentInitializer, if it passed no name, the target object would be initialized from the full arguments map. This is already how it works for @ProjectedPayload and for Query by Example. So just a matter of having a way to express that this is what you want, and it should be easy to do.

@jord1e, I understand no value vs null is an important distinction, but it is surprising that there is no a way to express an optional argument with a guaranteed value. You always have to deal with the possibility of null even though the distinction between null vs no value (or vs a default value) may be unnecessary.
This undermines the value of declaring a default value in the schema, especially for Java applications where there isn't a distinction between null vs no value. The end result is you have to check anyway and provide a fall back. Optional<Integer> doesn't solve this. It's just another flavor of handling it. The default value still needs to be duplicated.
I'm now thinking that @Argument should always fill in the default value from the schema, but also look for signs that null is expected, like on a method parameter that is Optional or marked @Nullable.

As this thread was growing and requesting multiple things I also created 2 separate issues:
https://github.com/spring-projects/spring-graphql/issues/258 Custom @Argument object to access root properties does not work
https://github.com/spring-projects/spring-graphql/issues/259 Add support for custom argument resolvers in GraphQL controllers

Team decision: null is an explicit input that requires additional effort from the client and if present it needs to be taken into account. Therefore application should be prepared to handle it. In practice, the extra effort required to send null means it's unlikely to be an issue if it makes no sense to send null like with first and offset.
In terms of options on how we can help, a defaultValue on @Argument is no good as it duplicates the schema mechanism. Transparently filling in the schema declared default value is also no good because we don't know enough. We would need something more explicit to get involved. The best place for that would be in the schema, e.g. argument directive is something we might consider in the future.

I agree and it's a difficult "problem".
For now probably best to never use primitives as arguments in GraphQL @Controller's as clients can send null explicitly,
and instead e.g. user Optional<Integer> or Integer.

Note reading this comment from @leebyron : https://github.com/graphql/graphql-js/issues/210#issuecomment-149738534

This is expected behavior for GraphQL. The library currently does not distinguish between undefined and null values for input. So both are replaced with a default value.

One nice thing about this from a server development point of view is that if you provide a default value for an argument, you can be guaranteed to always get a non-null value for that argument.

graphql-js seems to use the default value from the GraphQL schema when null is explicitly sent by client.

Note reading this comment from @leebyron : graphql/graphql-js#210 (comment)

This is expected behavior for GraphQL. The library currently does not distinguish between undefined and null values for input. So both are replaced with a default value.

One nice thing about this from a server development point of view is that if you provide a default value for an argument, you can be guaranteed to always get a non-null value for that argument.

graphql-js seems to use the default value from the GraphQL schema when null is explicitly sent by client.

That comment is from 2015 when no null literals were present in GraphQL
See https://github.com/graphql/graphql-js/pull/544 which negates the behavior you (and then Lee) are describing
This test case may also be of your interest:
https://github.com/graphql/graphql-js/blob/a91fdc600f2012a60e44356c373e51c5dd20ba81/src/execution/__tests__/variables-test.ts#L330


Update index.js: Add logging

Just adding some logging to be more safe.

The silent error is intentional. require() will throw an error when the module hasn't been loaded client-side yet. This might be improved later on via splitting up the client-side and server-side sync importing. Waiting on a require.id feature to be built into Webpack for this.


[5.6] Make inequality validation fail on different types rather than 500

There are four types of inequality validators: lt, lte, gt, gte. If these validators reference another attribute, the attribute must be of the same type. Before this change, if the payload has different types, a 500 error is given. However, the type is not determine by the rule configuration, but rather the HTTP request. It shouldn't be an Internal Server Error, when really the request was invalid.

5.6 is no longer supported. If accepted, as a breaking change, master would need to be targeted.
If considered a bug, 5.5 could possibly be targeted since it is accepting bug fixes until August.

Please send to 5.8 if this is a bug fix or master if it is a breaking change improvement.

I considered it a bug, because I believe an invalid payload should cause a validation failure, rather than an error to be thrown, that if uncaught will return a 500 internal server error. Because I believe it to be a bug, I was going to target the 5.5 (LTS) branch like the collaboration docs said, but the inequality validators were introduced in 5.6, so I targeted it to 5.6 instead. Based on what taylorotwell said, I'll target 5.8 instead.


removing some unnecessary argo rbacs

Signed-off-by: rajdas98<EMAIL_ADDRESS>
Proposed changes
removing some unnecessary argo rbacs
Types of changes
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the commit for DCO to be passed.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Dependency

Please add the links to the dependent PR need to be merged before this (if any).

Special notes for your reviewer:

Testing in progress

Works fine without these permision


Mask mode missing from Light2D

Tested versions
Reproducible in any 4.0 version
System information
Windows 10 - Godot 4.3
Issue description
Light2D nodes no longer have "mask" as one of their blend modes like they did in previous versions, making creating Line Of Sights/Field Of Views in 2D impossible. Some say to use CanvasItems/Groups but there is 0 documentation/tutorials about this subject so I'm not even sure if it's actually possible to get the same effect.
Steps to reproduce
Create a PointLight2D node
Try to change blend mode to mask
Notice that only add, substract and mix modes are available
Explode
Minimal reproduction project (MRP)
N/A

What functionality are you missing? In what way is this not possible in 4.x? Does this not work in this example?

What functionality are you missing? In what way is this not possible in 4.x? Does this not work in this example?

Hi, I'm trying to make a line of sight kinda like Darkwood/Zero sievert/Among Us, in the project you've sent they use a canvas material with light only, but that will show enemies and everything else also when they are not lit by the player, for example by any other light source.
Most of the workarounds I've found online and some I've created myself are too expensive and complicated for larger scenes and are generally too complex for an issue that didn't even exist in godot 3.

Have you tested the new Clip Children mode? This is intended to be the replacement as far as I can find, see:

https://github.com/godotengine/godot/issues/70358

For documenting the changes, please see:

https://github.com/godotengine/godot-docs/issues/5121

If this is still not what you need, please open a proposal as this is for bugs, not missing features (even if they were removed)
Thank you for your report!

Have you tested the new Clip Children mode? This is intended to be the replacement as far as I can find, see:

blend mode mask is missing in Light2D node #70358

For documenting the changes, please see:

Documentation pages to add or update for Godot 4.0 godot-docs#5121

If this is still not what you need, please open a proposal as this is for bugs, not missing features (even if they were removed)
Thank you for your report!

Yes, I've tried using clip children with subviewports but it's really heavy with larger scenes and kinda of a weird setup, I guess I'll open a proposal, thanks for your time.

Oh my bad you didn't mention clip children in your report so didn't know you had tried that

Oh my bad you didn't mention clip children in your report so didn't know you had tried that

Np, btw if you'd like to take a look at the way I implemented it I've added my project in the proposal https://github.com/godotengine/godot-proposals/issues/11082


Adding some return early to quieten type checker.

Well sorry. this is not so functional. Just to put away some warnings/errors by Pylance type checker.
I first thought I need to switch input -i options but what I wanted to do is obviously covered by your bandwidth -b update and the inputs might already be like its supposed to. Examples would be nice.

Btw: Another thing I was experiencing: When running this from a non-console python app: It does not terminate properly anymore? I actually had an issue with this in my initial standard subprocess implementation running ffmpeg and thought this one handled this. But apparently its something basic to do with these signals being sent only working when there is a console? 🤔


[BUG] Unidirectional Relationships imposing Bidirectional Relationships

Scope
Relationship Issue
New Unidirectional Relationships are creating Bidirectional Relationships in the connected relationship Elements.
To Reproduce
Steps to reproduce the behaviour:

Create a new unidirectional relationship.
Select the note you created the relationship to.
Check the linked Elements/Relationships and observe how it appears to be a bidirectional relationship.

Expected behaviour
When selecting a unidirectional relationship in the relationship manager, it is expected that the relationship remain a one-way relationship.
Video
Video Link here: https://youtu.be/qPyShGMSST0
Desktop (please complete the following information)
Obsidian 1.0.3
RPGM 3.3

Hi @sigrunixia
I tried to reproduce the bug, but can't. Also, the video is marked as private, so I can't see it :(


Avoid closing HTTP handle twice

WinHttpSyncHttpClient::~WinHttpSyncHttpClient() closes the handle, so does its base class' destructor. Annul the handle's pointer to avoid the second closing.
This also fixes the AWS core and S3 unit tests (when ran through AppVerifier or gflags, they crash).

@singku Done. (sorry for the first commit message, that was my first commit ever in GitHub and I expected to be prompted for commit message)

Merged https://github.com/awslabs/aws-sdk-cpp-staging/commit/36064ea3eeaacb5e45a6cc2a1866f63bec0a9434

how can I tell which version of SDK will have this fix?

1.3.28
On Thu, Dec 21, 2017 at 6:21 PM crusader-mike<EMAIL_ADDRESS>wrote:

how can I tell which version of SDK will have this fix?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/aws/aws-sdk-cpp/pull/763#issuecomment-353506247, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADTsCC5xGc-HvKrgfokWrQUeGNQYC0oHks5tCxIlgaJpZM4RIjwg
.

--
Best Regards!
Liang.
Software Engineer
Game Developer

Is there any way to tell without bothering you with questions? Like checking smth in github repository?

You can view the commits and find the tag info inside that page. The tags is the version number.
Usually it takes around 10 hrs publish to nuget from our pipeline. I just checked, our nuget client is out of date and many versions of libs are failed to publish. We just fixed it. It takes another 10 hours to see 1.3.28. on Nuget.

Thank you, @singku !


Force disable shaders of a mod

Describe your problem here.
Hi, i'm on MacOS and using GPTK (Wine) to run PsychEngine mods. Is there some sort of program argument (mod.exe --args) I can use to force disable the shaders? Because of the outdated OpenGL version on Mac, they make the game unable to launch.
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

Why has this been closed as completed when it isn't?

Why has this been closed as completed when it isn't?

the issue was stale


Use Photon

Would be interesting to check out how Photon Kit would work with this: https://github.com/connors/photon
:sparkles:

Wow it's seem promissing. That's a beautiful and smaller toolkit and we can do some tests and see how it work and linux and Windows


[Maintenance task] upgrade lots of libraries

Description

Pins latest dapr/dapr in certification tests.
Upgrades lots of libraries to their latest minor and patch versions.

/ok-to-test

Azure cert tests failing because of some changes in the azidentity library. Taking a look

/ok-to-test


Cosmos DB upsert documentation inaccurate - does not support partition key

Package Name: @azure/cosmos
Package Version: 3.15.1
Operating system: macOS
[x] nodejs

version: 14.18.3

[ ] browser

name/version:

[x] typescript

version: 4.6.2

Is the bug related to documentation in

[ ] README.md
[x] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
A clear and concise description of what the bug is.
The options for the Items.upsert function specifies that it can include a partition key. This, however seems inaccurate.
https://github.com/Azure/azure-sdk-for-js/blob/1007ebbd0ec6239e73395ad910313b132f0cb19a/sdk/cosmosdb/cosmos/src/client/Item/Items.ts#L309
The type for the request object is found here:
https://github.com/Azure/azure-sdk-for-js/blob/1007ebbd0ec6239e73395ad910313b132f0cb19a/sdk/cosmosdb/cosmos/src/request/RequestOptions.ts#L8-L47
To Reproduce
N/A
Expected behavior
A clear and concise description of what you expected to happen.
Either (preferred), the Items.upsert function should be updated with support for specifying partitionKey to conform to the documentation, or the documentation should be updated.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Current workaround is to use the batch API instead:
container.items.batch(
  [
    {
      operationType: 'Upsert',
      resourceBody: document as unknown as JSONObject,
    },
  ],
  document.somePartitionKey
)


Shortcut timeouts in PREPARE and COMMIT when justified

When sufficient justification is received do not wait for timeouts to expire in PREPARE and COMMIT phase.
Fixes #242

@anorth  I would love your early feedback on this please 🙏