HIE gives "could not execute: "

In stack cradle (more specifically, I use multi-cradle with stack), if a preprocessor executable is installed only locally but not globally, HIE says could not execute: <the name of the preprocessor>.
For my case, I use tasty-discover, and my package.yaml specify it as a dependency like this,
tests:
  test:
    main: Driver.hs
    source-dirs:
      - test
    dependencies:
      - hedgehog
      - tasty
      - tasty-discover
      - tasty-hedgehog

but HIE emits could not execute: tasty-discover.
Maybe this is related to mpickering#60 and will be solved by #1496, but I am not sure.

No, you have to have tasty-disover on the path for now. Maybe modify your path to point to the executable installed by stack.
Maybe we can change that in the future to be aware of the stack installed binaries.

The proper solution here is to execute hie using stack exec hie? Just like how nix users have to run ghc in the right nix-shell command.

While this is true, I dont think that this is really a viable solution for some users. E.g. we endorse the hie-wrapper, which can currently not handle this at all. But maybe we can fix that.

I think it is important that hie is able to build based on whatever environment the project is using. If it uses tasty-discover (or hspec-discover) based on a project config, we must honour this.

@fendor, @mpickering, @alanz, thank you all for the answers!
Unfortunately, as @fendor said, stack exec hie is not viable for me because of how lsp-haskell works (or at least I believe it is not), so I hope it will be handled by hie itself, if possible.

Oh, I see, it's hard to change the behaviour since it is the behaviour of GHC library.
So the following is a problematic part, right?
https://github.com/mpickering/hie-bios/blob/90955b3757f3fa65e7a96d54b677f5c936e3b782/src/HIE/Bios/Ghc/Api.hs#L84-L93
(The code calls G.depanal [] True and G.load' LoadAllTargets msg mod_graph consecutively to load all dependency modules, and I guess it's not easy to give some specific PATH to G.load')

Exactly. We would have to manipulate the path ourselves, which is doable, but definitely postponed for now.


Minor Fixes

Using python ping can require elevated permissions, even if ping does not. using requests solves that problem. also changing to makes it easier to test without installing using pip.

I changed it to relative imports because when imported from a folder it giving a import error. do you want to remove that from the pull request

I'm agree about the internet fix, it works well on mac

Changes the internet check from pythonping to socket
If you can check it on Mac and let me know it would be great


Various distance sensor fixes and enhancements

See individual commits for details

Codecov Report

Merging #617 into master will decrease coverage by 0.01%.
The diff coverage is 92%.

@@            Coverage Diff            @@
##           master    #617      +/-   ##
=========================================
- Coverage   84.71%   84.7%   -0.02%
=========================================
  Files          36      36
  Lines        6695    6700       +5
=========================================
+ Hits         5672    5675       +3
- Misses       1023    1025       +2

Impacted Files
Coverage Δ

gpiozero/exc.py
100% <100%> (ø)
:arrow_up:

gpiozero/input_devices.py
87.28% <91.66%> (-0.22%)
:arrow_down:

gpiozero/mixins.py
77.35% <0%> (-0.43%)
: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 458b4a8...e69f168. Read the comment docs.


TotalStorageCapacityInMegaBytes missing from redshift.describe_clusters()

Reporting Bugs
Please be aware of the following things when filing bug reports:

Avoid raising duplicate issues. Please use the GitHub issue search feature
to check whether your bug report or feature request has been mentioned in
the past.

When filing bug reports about exceptions or tracebacks, please include the
complete traceback. Partial tracebacks, or just the exception text, are
not helpful.

Make sure you provide a suitable amount of information to work with. This
means you should provide:

Guidance on how to reproduce the issue. Ideally, this should be a
small code sample that can be run immediately by the maintainers.
Failing that, let us know what you're doing, how often it happens, what
environment you're using, etc. Be thorough: it prevents us needing to ask
further questions.

Use @mock_redshift, and call boto3_client.describe_clusters().
TotalStorageCapacityInMegaBytes will be missing from the response.

Tell us what you expected to happen. When we run your example code,
what are we expecting to happen? What does "success" look like for your
code?
TotalStorageCapacityInMegaBytes should appear in the response

https://docs.aws.amazon.com/cli/latest/reference/redshift/describe-clusters.html

Tell us what actually happens. It's not helpful for you to say "it
doesn't work" or "it fails". Tell us how it fails: do you get an
exception? A hang? How was the actual result different from your expected
result?

Tell us what version of Moto you're using, and
how you installed it. Tell us whether you're using standalone server
mode or the Python mocks. If you are using the Python mocks, include the
version of boto/boto3/botocore.

Python 3.9.18
boto3                               1.26.165
boto3-stubs                         1.28.51
botocore                            1.29.165
botocore-stubs                      1.29.165
moto                                4.2.0
If you do not provide all of these things, it will take us much longer to
fix your problem.

Hi @mayk0gan! Marking it as an enhancement to add this field.
Considering we don't store any actual data for redshift, we'll probably just always return 0 here.


Consent error when acquire token for delegated api

Please follow the issue template below. Failure to do so will result in a delay in answering your question.

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Important: Please fill in your exact version number above, e.g<EMAIL_ADDRESS>Framework
React
Description
After signing in the user, I try to call acquireTokenSilent on scope of my delegated api, but I recevied a invalid_grant: AADSTS65001: The user or administrator has not consented to use the application with ID
And After I send an interact request to user, it says I have to login in as admin to consent.
Error Message
invalid_grant: AADSTS65001: The user or administrator has not consented to use the application with ID
Security

[ ] Is this issue security related?

Regression

[ ] Did this behavior work before?
Version:

I didn't try it with other versions, but I do request access token of the same scope, using the same clientId in ASP.NET using Microsoft.Identity.Web and it works.
MSAL Configuration
export const msalConfig: BrowserAuthOptions = {
  clientId: "{myid}",
  authority:
    "https://login.microsoftonline.com/{tenant}/",
};

Reproduction steps
// Provide relevant code snippets here.
// For Azure B2C issues, please include your policies.

Expected behavior
Browsers/Environment

[x] Chrome
[ ] Firefox
[x] Edge
[ ] Safari
[ ] IE
[ ] Other (Please add browser name here)

Is your application registered as a "SPA type" in the portal?


wire-desktop backport to release-20.09

Motivation for this change
Backport of #99941.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Yep, this definitely needs to be merged into 20.09, as this release fixes a high security vulnerability


simulation: Lorenz Attractor

Proposed Simulation Name : Lorenz Attractor
https://en.wikipedia.org/wiki/Lorenz_system
Category Proposed: Mathematics
Additional points/links:

Hi, I would like to give this issue a try for GSSOC '21.

Hi, I would like to give this issue a try for GSSOC '21.

@ritvij14 please go ahead!

@yashlamba @yashugarg could you please assign this to me and guide me a little so I can start working on this?

Hello @ritvij14 this is a 3D simulation and might not be possible with flutter right now.


Remove CodeClimate stuff

We aren't using this since our repo setup doesn't work with their tools. The jake task will still exist for manual code coverage support.

:+1:


Additional information to be added to Vault-mTLS configuration

Where is the problem?
https://docs.konghq.com/mesh/1.5.x/features/vault/
What happened?
I was looking for information on configuring Vault-mTLS integration, but ended up having to open a thread in Slack to get the necessary details. #3541 helps somewhat, but there are still a couple details that IMO would be worth adding:

We should document that the tls.caCert.secret should be stored in the "kong-mesh-system" namespace and be labeled with the name of the mesh to which it applies (in the format kuma.io/mesh: <mesh-name>.
We should document that the token found in auth.token.secret should also be stored in the "kong-mesh-system" namespace. If there are labeling requirements (it's labeled in my environment, but I don't know if the label is required) then we should include those as well. If this needs to be set as a Kuma secret (type: system.kuma.io/secret), then that also should be documented.

I'm happy to submit a PR to help with adding these details, if desired.
What did you expect to happen?
The documentation for configuring Vault-mTLS integration should have had all the necessary information for configuring this functionality.
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

We should document that the tls.caCert.secret should be stored in the "kong-mesh-system" namespace and be labeled with the name of the mesh to which it applies (in the format kuma.io/mesh: .
We should document that the token found in auth.token.secret should also be stored in the "kong-mesh-system" namespace. If there are labeling requirements (it's labeled in my environment, but I don't know if the label is required) then we should include those as well. If this needs to be set as a Kuma secret (type: system.kuma.io/secret), then that also should be documented.

I agree, it's not clear at the moment that this is a "Kong Mesh secret" thing. We do however have (Kuma) docs on those secrets, so we should link there as well as include a short hint on this page.

Also, under step 4, we advise the reader to use jq to parse the results of the vault token create command to show only the returned token. I found that Vault will in some cases return a token even if there is an error; for example, if the policy doesn't exist, Vault will still return a token. (The token won't work as expected.) It may be helpful to tell readers that the use of jq is an optional thing, so that errors are properly exposed.

@scottslowe Thanks for calling this out. You're welcome to go ahead and make a PR with these changes, the help is always appreciated.

@michaelbeaumont If you have time, can you have a look at the related PR to ensure that the updated wording is accurate/correct?


net/mail: mail ParseAddressList can bypass and panic

What version of Go are you using (go version)?

$ go version
go version go1.15.4 windows/amd64

What operating system and processor architecture are you using (go env)?
go env Output
$ go env
C:\Users\xx\go\src\awesomeProject1>go env
set GO111MODULE=on
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\xx\AppData\Local\go-build
set GOENV=C:\Users\xx\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\xx\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\xx\go
set GOPRIVATE=
set GOPROXY=direct
set GOROOT=C:\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=C:\Users\xx\go\src\awesomeProject1\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\xx\AppData\Local\Temp\go-build958360038=/tmp/go-build -gno-record-gcc-switches

What did you do?

https://play.golang.org/p/efQXFJo-TIC
package main

import (
	"net/mail"
	"strings"
)

func main() {
	//"":;
	data := "\"\":;"
	list, err := mail.ParseAddressList(data)
	if err != nil {
		return
	}
	var addrs []string
	for _, addr := range list {
		addrs = append(addrs, addr.String())
	}
	_, err = mail.ParseAddressList(strings.Join(addrs, ","))
	if err != nil {
		panic(err)
	}

}

What did you expect to see?
first  input  "":; can  bypass
if handleGroup {
   if p.consume(':') {
      return p.consumeGroupList()
   }
}

second  is  panic: mail: no address
What did you see instead?
panic: mail: no address

ParseAddressList do return an error "mail: no address" without panic as expected.
@cor0ps The panic is raised in your code not net/mail module
_, err = mail.ParseAddressList("")
if err != nil {
	panic(err)
}

ParseAddressList do return an error "mail: no address" without panic as expected.
@cor0ps The panic is raised in your code not net/mail module
_, err = mail.ParseAddressList("")
if err != nil {
	panic(err)
}

It seems we should check on addrs number at the end of parseAddressList
https://golang.org/src/net/mail/message.go?s=5170:5224#L307
		if p.empty() {
			break
		}
	}
	return list, nil // we shoud check it here

I can send a CL for this issue , Is it OK for you ? @cor0ps

It seems we should check on addrs number at the end of parseAddressList
https://golang.org/src/net/mail/message.go?s=5170:5224#L307
		if p.empty() {
			break
		}
	}
	return list, nil // we shoud check it here

I can send a CL for this issue , Is it OK for you ? @cor0ps

Thank you for this report @cor0ps, thanks @mengzhuo and @ALTree for the triage! @cuonglm perhaps we should also place this on our list for fuzzing?

Thank you for this report @cor0ps, thanks @mengzhuo and @ALTree for the triage! @cuonglm perhaps we should also place this on our list for fuzzing?

@mengzhuo mengzhuo  you  can  make  a  issuse   and  have questions  can   contact with me  wechat：SuperTao99
	if p.empty() {
			break
		}
	}
	return list, nil // we shoud check it here

I  aggree with you

@mengzhuo mengzhuo  you  can  make  a  issuse   and  have questions  can   contact with me  wechat：SuperTao99
	if p.empty() {
			break
		}
	}
	return list, nil // we shoud check it here

I  aggree with you

@odeke-em  I  found  this  use  gofuzz  tools   and  i  think  this have  a   securtiy  problem  and   this  can  get  a  CVE id

@odeke-em  I  found  this  use  gofuzz  tools   and  i  think  this have  a   securtiy  problem  and   this  can  get  a  CVE id

@odeke-em I found this use gofuzz tools and i think this have a securtiy problem and this can get a CVE id ？ thanks

I don't think it's a CVE, RFC 5322 stated that empty group is allowed.
https://tools.ietf.org/html/rfc5322#section-3.4

@odeke-em I found this use gofuzz tools and i think this have a securtiy problem and this can get a CVE id ？ thanks

I don't think it's a CVE, RFC 5322 stated that empty group is allowed.
https://tools.ietf.org/html/rfc5322#section-3.4


Add whitespace error checking to GitHub Actions

Summary
Add a job that checks the project for whitespace errors (using git diff --check). The contribution guide already suggests no trailing whitespace; this helps enforce it.
The new job is called lint - my intention that in the future, further lint-like steps could be added to it.
Fix all existing errors in the codebase.
How to test
N/A

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG N/A - no user-visible changes
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

I'd want to move towards having some kind of automatic code formatting. Probably, this PR could be considered a first step.

The only this is that I'm not sure we want it for all files. Because some formats can rely on this.

It's possible to customize which checks are done for specific files (or file types) by adding Git attributes, so we can always add an exception if needed.

Let's exclude tests/assets/.

Let's exclude tests/assets/.

Why? None of the current files require trailing spaces.

Because such restriction has no foundation for these files.


Piezo frequencies could be floats

Hypothetically, piezo implementations may support more precision than an integer allows, and the frequencies of musical notes we provide in the Note enum could be specified to more precision than they currently are. It seems reasonable to use floats everywhere and make it the responsibility of the backend to round to an integer if it does not support fractional frequencies.
This should probably be implemented before or at the same time that #122 is, to avoid having to change things twice.

In order to not break Note being of type IntEnum when it is moved to the Piezo component (#122), we would need to support Union[int,float]

Ideally Note shouldn't have to be an IntEnum, because frequencies should be real numbers rather than integers. It sucks that we have to use this workaround to make the typechecking pass (see this thread for reference).

@HU90m I feel like you might want this one too?


Docs and docker setup

basic docker build
docker compose setup to allow running a second instance
some read me

I have added clap argument support that allows configuring host and port via app args or env vars. In addition the frontend code now uses relative urls so it will automatically request the api it is served from.
These changes now finally allow multiple instances of the app to run on the same system as well as running and configuring it for container based infrastructures.


Clarify status and semantics of object ('O') data type in storage spec

The current storage spec in the section on “data type encoding” describes how the data type of an array should be encoded in the array metadata. Here is the content from the beginning of the section:
Simple data types are encoded within the array metadata as a string,
following the `NumPy array protocol type string (typestr) format
<http://docs.scipy.org/doc/numpy/reference/arrays.interface.html>`_. The format
consists of 3 parts:

* One character describing the byteorder of the data (``"<"``: little-endian;
  ``">"``: big-endian; ``"|"``: not-relevant)
* One character code giving the basic type of the array (``"b"``: Boolean (integer
  type where all values are only True or False); ``"i"``: integer; ``"u"``: unsigned
  integer; ``"f"``: floating point; ``"c"``: complex floating point; ``"m"``: timedelta;
  ``"M"``: datetime; ``"S"``: string (fixed-length sequence of char); ``"U"``: unicode
  (fixed-length sequence of Py_UNICODE); ``"V"``: other (void * – each item is a
  fixed-size chunk of memory))
* An integer specifying the number of bytes the type uses.

The byte order MUST be specified. E.g., ``"<f8"``, ``">i4"``, ``"|b1"`` and
``"|S12"`` are valid data type encodings.

The spec then goes on to describe how datetime and timedelta data types are encoded:
For datetime64 ("M") and timedelta64 ("m") data types, these MUST also include the
units within square brackets. A list of valid units and their definitions are given in
the `NumPy documentation on Datetimes and Timedeltas
<https://docs.scipy.org/doc/numpy/reference/arrays.datetime.html#datetime-units>`_.
For example, ``"<M8[ns]"`` specifies a datetime64 data type with nanosecond time units.

...and also how structured data data types are encoded:
Structured data types (i.e., with multiple named fields) are encoded as a list
of two-element lists, following `NumPy array protocol type descriptions (descr)
<http://docs.scipy.org/doc/numpy/reference/arrays.interface.html#>`_. For
example, the JSON list ``[["r", "|u1"], ["g", "|u1"], ["b", "|u1"]]`` defines a
data type composed of three single-byte unsigned integers labelled "r", "g" and
"b".

Implicit in all of this is that the spec is inheriting the numpy definition of data types, and deferring to the numpy documentation as much as possible.
In addition to fixed-memory data types, numpy also defines an “object” data type (character code ‘O’). In numpy, an array with object data type is an array of memory pointers, where each pointer dereferences to a Python object. Although the object data type is described in the numpy documentation, it is not mentioned at all in the zarr storage spec. It is therefore unclear whether it is or is not a valid data type for use in a zarr array, and if it is, what it’s semantics are.
The Python zarr implementation has in fact fully supported the object data type since version 2.2 (zarr-developers/zarr#212). The implementation follows numpy in the sense that, when data are retrieved from a zarr array with object data type, they are returned to the user as a numpy array with object data type, i.e., as an array of Python objects.
However, this does not mean that the encoded zarr data are necessarily Python-specific. When storing data into a zarr array with object data type, how the objects are encoded is delegated to the first codec in the filter chain. For example, if the first codec in the filter chain is the MsgPack codec, then data will be encoded using MessagePack encoding, which is a language-independent encoding and can in principle be decoded in a variety of programming languages. Similarly, if the array contains only string objects, then the VLenUTF8 codec can be used, which will encode data in a format similar to parquet encoding, and which could be decoded in any programming language. Further examples are given in the sections on string arrays and object arrays in the zarr tutorial.
In the longer term, the community may want to revisit the approach to encoding of arrays with variable-length data types, and to produce a new major revision of the storage spec. However, I suggest that we first aim to resolve this issue by adding some clarifying text to the version 2 storage spec, to make explicit the status and semantics of the object data type. As precedent, we have previously made a number of edits to the version 2 storage spec to make clarifications, see the changes section of the spec. For example, we added clarifications regarding the datatime and timedelta data types, and we added clarifications regarding the encoding of fill values, so I am hoping for a similar resolution here.

Atm, object data type is not supported in v3 and might be added as an extension. So this is mostly a v2 issue for now I assume

The issue here is whether V3 aims to be fully backwards compatible with V2. In the memory-layout discussion, the desire for backwards compatibility was cited as a reason to allow both C and F order. If V3 will not support "O" data type, then this will break backwards compatibility with existing V2 data.
The extension would be an obvious way around this.
I personally support the idea of dropping "O" dtype in the core spec and then re-adding it immediately as an extension.

Am also in favor of dropping the "O" dtype. Adding through an extension sounds reasonable

Definitely agreed that it should be dropped for now.
It could be added back as a Python-specific  python-object data type for use with a pickle codec, but not all uses of the zarr v2 data type should be migrated to it.
For example:
With json codec it should be migrated to a new json data type.
With vlenutf8 codec it should be migrated to a new vlen Unicode string data type.
etc.
The end result would be that remaining uses of python-object are likely pretty rare.


LibWasm: Generate all spec-tests properly

Pulled from https://github.com/SerenityOS/serenity/pull/24502. At this point, all the crashing spec-tests have been fixed (I hope), so I just need to add SIMD support.
Edit:
Just kidding, apparently there's a spec test that doesn't pass through ubsan

SIMD support has been added! I believe it's ready to merge now.

This is much nicer, thank you!


NotepadService must complete it's deferral

@paulmon,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@paulmon, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

I tested this change, it looks fine because the NotepadService doesn't close when a client detaches.
The only doubt is relative to the client service task created by the NotepadService when the client attaches: it seems that the client service task doesn't close when the client detaches.
Maybe a further action is required to close the client task?

That is by design since the task type=startup in the appxmanifest.
If you would like the appservice to exit when it has no clients I suggest you take a look at this sample.
https://github.com/Microsoft/Windows-universal-samples/tree/master/Samples/AppServices


AHB enabled cores incorrect when using SQL Elastic Pools

🐛 Problem
When I check the "SQL DB AHB Consumed Cores per VM" it seems like it adds up all databases on a SQL Elastic Pool ?  For instance: we have 23 databases on a SQL Elastic pool with 6vcores.  Then 23x6 is calculated as SQL DB AHB Consumed Cores ?  (while of course the AHB is enabled on the Elastic Pool level).  This means the overview is unusable to correctly reflect the REAL number of vcores consumed with AHB
👣 Repro steps

Go to Rate Optimization - Azure Hybrid Benefit - SQL DB

🤔 Expected
I expect the value to be correct.
📷 Screenshots

🙋♀ Ask for the community

We could use your help:

Please vote this issue up (👍) to prioritize it.
Leave comments to help us solidify the vision.

Hello @pvansteenkiste
Acknowledged.  Thank you for submitting this.
I will work on this and revert back with the fix.

Hello @pvansteenkiste
There was an error on the queries used to generate these pie charts. I've changed it and it will be published to Advisor at the end of this month.

Let me know if you can't wait until the end of the month to have this fixed and I can make it available on the Toolkit (Which has a faster publishing process).

Hi Arthur,
No no, I can wait until the end of the month 😊. Thank you for your effort !!
Best regards
Philippe
Van: Arthur Clares @.>
Datum: woensdag, 17 april 2024 om 16:56
Aan: microsoft/finops-toolkit @.>
CC: Philippe Vansteenkiste @.>, Mention @.>
Onderwerp: Re: [microsoft/finops-toolkit] AHB enabled cores incorrect when using SQL Elastic Pools (Issue #685)
Let me know if you can't wait until the end of the month to have this fixed and I can make it available on the Toolkit (Which has a faster publishing process).
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/finops-toolkit/issues/685#issuecomment-2061460092, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIPK27GVWOTTGNJGFVOMVGTY52ESTAVCNFSM6AAAAABF77HYEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRRGQ3DAMBZGI.
You are receiving this because you were mentioned.Message ID: @.***>

Hello Arthur,
Any idea on when it will be fixed in Azure Advisor ?
best regards,
Philippe

HI Arthur,
When should it be fixed in Azure Advisor ?
Best Regards,
Philippe
From: Arthur Clares @.>
Sent: Wednesday, 17 April 2024 16:57
To: microsoft/finops-toolkit @.>
Cc: Philippe Vansteenkiste @.>; Mention @.>
Subject: Re: [microsoft/finops-toolkit] AHB enabled cores incorrect when using SQL Elastic Pools (Issue #685)
Let me know if you can't wait until the end of the month to have this fixed and I can make it available on the Toolkit (Which has a faster publishing process).
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/finops-toolkit/issues/685#issuecomment-2061460092, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIPK27GVWOTTGNJGFVOMVGTY52ESTAVCNFSM6AAAAABF77HYEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRRGQ3DAMBZGI.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@pvansteenkiste The portal should be updated. Can you confirm if you see the SQL Elastic Pool tab under SQL Databases?

Closing this as this query has been added to May's update.


Model Loading Error with Bi-directional LSTM layer

I trained a model using tf==2.16.1 and keras 3.0.5.
While loading the same model through the same tf and keras versions (.keras format), the following error is encountered
File "/root/miniconda3/envs/model_training/lib/python3.11/site-packages/keras/src/saving/saving_api.py", line 176, in load_model
return saving_lib.load_model(
^^^^^^^^^^^^^^^^^^^^^^
File "/root/miniconda3/envs/model_training/lib/python3.11/site-packages/keras/src/saving/saving_lib.py", line 192, in load_model
_raise_loading_failure(error_msgs)
File "/root/miniconda3/envs/model_training/lib/python3.11/site-packages/keras/src/saving/saving_lib.py", line 273, in _raise_loading_failure
raise ValueError(msg)
ValueError: A total of 2 objects could not be loaded. Example error message for object :
Layer 'lstm_cell' expected 3 variables, but received 0 variables during loading. Expected: ['kernel', 'recurrent_kernel', 'bias']
List of objects that could not be loaded:
[, ]
Investigation mostly suggests it might be a version compatibility issue train vs load, but since both of these are being done in the same conda env I do no think it might be the case.

@nilesh-sengupta Please use the appropriate loading function based on the saving format. For example, tf.keras.models.load_model for HDF5 and tf.saved_model.load for SavedModel.
Thank you!

@sushreebarsa The model is in .keras format, it is saved using model.save('model_name.keras'), loading it using tf.saved_model.load gives the following exception:
Traceback (most recent call last):
File "/usr/local/bin/mldnn/dnn_1.6/predictor.py", line 26, in
model = tf.saved_model.load('deep_neural_network.keras')
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/miniconda3/envs/model_training/lib/python3.11/site-packages/tensorflow/python/saved_model/load.py", line 912, in load
result = load_partial(export_dir, None, tags, options)["root"]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/miniconda3/envs/model_training/lib/python3.11/site-packages/tensorflow/python/saved_model/load.py", line 1016, in load_partial
loader_impl.parse_saved_model_with_debug_info(export_dir))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/miniconda3/envs/model_training/lib/python3.11/site-packages/tensorflow/python/saved_model/loader_impl.py", line 59, in parse_saved_model_with_debug_info
saved_model = parse_saved_model(export_dir)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/miniconda3/envs/model_training/lib/python3.11/site-packages/tensorflow/python/saved_model/loader_impl.py", line 119, in parse_saved_model
raise IOError(
OSError: SavedModel file does not exist at: deep_neural_network.keras/{saved_model.pbtxt|saved_model.pb}
The previous error is encountered while using tf.keras.models.load_model

@nilesh-sengupta The error arises because you're using tf.saved_model.load to load a model saved in Keras format (.keras). This function is designed for SavedModel format, not Keras format.
The loading function should be as follows;
model = tf.keras.models.load_model('DL_network.keras')

Thank you!

@nilesh-sengupta
Could you try to re-save your Keras model as a SavedModel:
tf.saved_model.save(model, 'path/to/save/model')  # Use a distinct path to avoid overwriting

Since your model is in the .keras format, please use  tf.keras.models.load_model to load it:
model = tf.saved_model.load('path/to/save/model')

This function is specifically designed to handle models saved in the Keras format.
Thank you!


Transactions tab only displaying "normal" transactions, i.e. excludes ERC20s

According to https://etherscan.io/apis#accounts:
"Normal":
http://api.etherscan.io/api?module=account&action=txlist&address=0xddbd2b932c763ba5b1b7ae3b362eac3e8d40121a&startblock=0&endblock=99999999&sort=asc&apikey=YourApiKeyToken
ERC20:
http://api.etherscan.io/api?module=account&action=tokentx&address=0x4e83362442b8d1bec281594cea3050c8eb01311c&startblock=0&endblock=999999999&sort=asc&apikey=YourApiKeyToken
Looks like we'll need to make 2 separate calls and combine them.
Need to verify for ERC721/ERC875.

@hboon this is not a problem, the api call includes transactions to contracts. We can switch to RPC however.


Backport "Fix listing release versions for bisect script." to LTS

Backports #19356 to the LTS branch.
PR submitted by the release tooling.

No regressions detected in the community build up to lts-19356.
Reference


Minor updates to getting started doc

Spelling fixes, minor command fixes, and a few small grammar changes.
Checklist:

[ ] cargo +nightly fmt --all
[ ] Modify CHANGELOG.md if applicable

Thank you very much!


Update sbt-sourcegraph and re-enable SBT build tool tests

Now that we can run auto-indexing for SBT and support Scala 3, it's time to re-enable the tests
Test plan

Re-enabled disabled tests
Expanded testing matrix

@olafurpg
Note that I have some changes locally to be able to run build tool tests in parallel, but it seems to suffer from a concurrency bug, where MUnit prints all suite names first and then all test results

FTR, the diff is
diff --git a/build.sbt b/build.sbt
index 4b2a864..52066b7 100644
--- a/build.sbt
+++ b/build.sbt
@@ -509,7 +509,8 @@ lazy val buildTools = project
       Map(
         "SCIP_JAVA_CLI" -> ((cli / pack).value / "bin" / "scip-java").toString
       ),
-    Test / fork := true
+    Test / fork := true,
+    Test / testForkedParallel := true
   )
   .dependsOn(agent, unit)

diff --git a/tests/buildTools/src/test/scala/tests/BaseBuildToolSuite.scala b/tests/buildTools/src/test/scala/tests/BaseBuildToolSuite.scala
index 80fe64a..d4401fb 100644
--- a/tests/buildTools/src/test/scala/tests/BaseBuildToolSuite.scala
+++ b/tests/buildTools/src/test/scala/tests/BaseBuildToolSuite.scala
@@ -21,6 +21,7 @@ import os.Shellable
 object Java8Only extends munit.Tag("Java8Only")

 abstract class BaseBuildToolSuite extends MopedSuite(ScipJava.app) {
+  self =>
   override def environmentVariables: Map[String, String] = sys.env

   def tags = List.empty[Tag]
@@ -49,8 +50,15 @@ abstract class BaseBuildToolSuite extends MopedSuite(ScipJava.app) {
   // NOTE(olafur): workaround for https://github.com/scalameta/moped/issues/18
   override val temporaryDirectory: DirectoryFixture =
     new DirectoryFixture {
-      private val path = BuildInfo.temporaryDirectory.toPath
-      override def apply(): Path = path
+      private val path = BuildInfo
+        .temporaryDirectory
+        .toPath
+        .resolve(self.getClass().getSimpleName())
+
+      override def apply(): Path = {
+        Files.createDirectories(path)
+        path
+      }
       override def beforeEach(context: BeforeEach): Unit = {
         DeleteVisitor.deleteRecursively(path)
       }

@keynmol what happens if you remove the -b option here? IIRC, MUnit supports using the sbt test logger, which groups by suite https://github.com/sourcegraph/scip-java/blob/main/build.sbt#L571

That seems to have helped - I made buildTools the only module running in parallel

Ok, I made the tests sequential again on CI only - GHA workers have such tiny vCPUs and barely two of them, that it's not worth the increased GC overhead


🐛 [Bug] Call RefreshDatabase and Artisan run migrate from  the Test file return error

Before feedback (在反馈之前)

[X] I've searched the existing Issues, Discussions and Google (我已经搜索了现有的 Issues, Discussions 和 Google)
[ ] The problem can be stably reproduced (这个问题可以被稳定复现)
[ ] The problem is generated after upgrading (问题是在升级之后产生的)

Operating System (操作系统)
MacOS
Golang Version (Go 版本)
1.21.x
Goravel Version (Goravel 版本)
1.14.7
Describe The Problem (描述问题)
Hi I create feature testing for some features in my app. The idea is I wanted to refresh database / run artisan migrate when run the test script. But I got this error :
=== RUN   TestAuthControllerTestSuite/TestLoginValidationRequest
    application.go:88: test panicked: failed to open source, "file://./database/migrations": open .: no such file or directory
        goroutine 9 [running]:

Any idea on this?
Reproduction Code (复现代码或截图)
// SetupTest will run before each test in the suite.
func (s *AuthControllerTestSuite) SetupTest() {
	s.RefreshDatabase()
        facades.Artisan().Call("migrate:fresh")
}

Could you try this case in your local? https://github.com/goravel/example/blob/master/tests/services/main_test.go

Could you create a PR for your failed case to goravel/example? The test cases of goravel/example are tested by CI, they should pass. Not sure what's happening in your local.


DST 440: end to end tests

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

Please upload report for BASE (main@8a991ad). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #159   +/-   ##
=======================================
  Coverage        ?   81.21%
=======================================
  Files           ?       64
  Lines           ?     2140
  Branches        ?        0
=======================================
  Hits            ?     1738
  Misses          ?      402
  Partials        ?        0

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


Update README.md

Default now uses Turbo instead of Small

Noticed it uses Turbo instead of Small by now, had me confused so I changed the Readme to reflect that.


译文搬运

Input Pipeline Performance Guide的译文

#24 almost down

Translation Of Input Pipeline Performance Guide(输入流水线性能指南).md

second submission

failed to merge, reset to 6ebf3ff6412e713cc08c88c07ea7c8febd863c2a


[e2e-tests] Add security scan check to quay plugin suite

Description
Extend quay plugin e2e suite to include the changes contributed by RHTAP
Which issue(s) does this PR fix
https://issues.redhat.com/browse/RHIDP-1112
PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/lgtm


View trash bin of disabled project space gives 404

ownCloud Web UI 5.7.0-rc.10
Infinite Scale 2.0.0-beta.7
Steps:

admin creates space
admin creates folder
admin deletes folder
admin disables space
admin trie to see trash bin before deleting space

Actual: PROPFIND https://localhost:9200/remote.php/dav/spaces/trash-bin/1284d238-aa92-42ce-bdc4-0b0000009157%242c0e3fa4-e4d3-4a3f-b69f-6ea08beced33 gives 404
server log: 2022-08-24T15:49:48+02:00 ERR failed to list recycle error="error: not found: 2c0e3fa4-e4d3-4a3f-b69f-6ea08beced33" key=/ pkg=rgrpc reference={"path":".","resource_id":{"opaque_id":"2c0e3fa4-e4d3-4a3f-b69f-6ea08beced33","space_id":"2c0e3fa4-e4d3-4a3f-b69f-6ea08beced33","storage_id":"1284d238-aa92-42ce-bdc4-0b0000009157"}} service=storage-users status={"code":6,"message":"path not found when listing recycle","trace":"00000000000000000000000000000000"} traceid=00000000000000000000000000000000
Expected:
I expect 2 options:

user can view trash bin of disabled project space

or:

the "Delete Files" button should not be available for the disabled space (web ticket)

it's not relevant.
we haven't button deleted files in menu now and user cannot request trash bin of the disabled space
in the Deleted files tab we haven't ability to see trashbin of the disabled space


[branch-2.1] Picks "fix Fix delete stmt on MOW table doesn't use partial update in Nereids planner #38751"

Proposed changes
picks https://github.com/apache/doris/pull/38751

run buildall


s2i build --as-dockerfile not using local .s2i/[assemble, run] when builder image present

When running s2i build with both local script files and a valid builder image, I'm not seeing the resulting
dockerfile having the correct RUN and CMD statements. However, when pointing to an invalid builder image
the expected set of statements and other elements (in the debug log) appear present in the container build file.
Version: 1.3.2 (Fedora 38) and 1.3.8
Reproduction:
(This repository contains both .s2i/bin/run and .s2i/bin/assemble
$ git clone https://github.com/cuppett/laravel-s2i.git
$ cd laravel-s2i
$ s2i build . registry.access.redhat.com/ubi8/php-80 --as-dockerfile Containerfile.invalid --loglevel=5

I0807 07:26:59.242962  100612 build.go:52] Running S2I version "v1.3.8"
I0807 07:26:59.243069  100612 util.go:70] Getting docker credentials for registry.access.redhat.com/ubi8/php-80
I0807 07:27:00.156576  100612 build.go:170]
Source:				.
Output Image Tag:
Environment:
Labels:
Incremental Build:		disabled
Remove Old Build:		disabled
Builder Pull Policy:		if-not-present
Previous Image Pull Policy:	if-not-present
Quiet:				disabled
Layered Build:			disabled
Docker Endpoint:		unix:///run/user/1000/podman/podman.sock
Docker Pull Config:		/home/scuppett/.docker/config.json
Docker Pull User:

I0807 07:27:00.156769  100612 scm.go:20] DownloadForSource .
I0807 07:27:00.157914  100612 clone.go:37] Downloading "." ...
I0807 07:27:00.157922  100612 clone.go:41] Cloning sources into "upload/src"
I0807 07:27:00.198650  100612 clone.go:57] Checked out "HEAD"
I0807 07:27:00.213746  100612 clone.go:63] Updated submodules for "HEAD"
I0807 07:27:00.219627  100612 install.go:269] Using "assemble" installed from "<source-dir>/.s2i/bin/assemble"
I0807 07:27:00.219660  100612 install.go:269] Using "run" installed from "<source-dir>/.s2i/bin/run"
I0807 07:27:00.219680  100612 install.go:269] Using "save-artifacts" installed from "image:///usr/libexec/s2i/save-artifacts"
I0807 07:27:00.219698  100612 ignore.go:84] .s2iignore lists a file spec of .dockerignore
I0807 07:27:00.219703  100612 ignore.go:116] using globspec upload/src/.dockerignore
I0807 07:27:00.219711  100612 ignore.go:122] Glob result upload/src/.dockerignore
I0807 07:27:00.219715  100612 ignore.go:84] .s2iignore lists a file spec of .env*
I0807 07:27:00.219720  100612 ignore.go:116] using globspec upload/src/.env*
I0807 07:27:00.219751  100612 ignore.go:122] Glob result upload/src/.env.example
I0807 07:27:00.219756  100612 ignore.go:84] .s2iignore lists a file spec of .git
I0807 07:27:00.219761  100612 ignore.go:116] using globspec upload/src/.git
I0807 07:27:00.219768  100612 ignore.go:122] Glob result upload/src/.git
I0807 07:27:00.219772  100612 ignore.go:84] .s2iignore lists a file spec of .idea
I0807 07:27:00.219776  100612 ignore.go:116] using globspec upload/src/.idea
I0807 07:27:00.219785  100612 ignore.go:84] .s2iignore lists a file spec of .s2iignore
I0807 07:27:00.219789  100612 ignore.go:116] using globspec upload/src/.s2iignore
I0807 07:27:00.219796  100612 ignore.go:122] Glob result upload/src/.s2iignore
I0807 07:27:00.219800  100612 ignore.go:84] .s2iignore lists a file spec of Dockerfile.s2i-fake
I0807 07:27:00.219804  100612 ignore.go:116] using globspec upload/src/Dockerfile.s2i-fake
I0807 07:27:00.219811  100612 ignore.go:84] .s2iignore lists a file spec of LICENSE
I0807 07:27:00.219816  100612 ignore.go:116] using globspec upload/src/LICENSE
I0807 07:27:00.219823  100612 ignore.go:84] .s2iignore lists a file spec of README.md
I0807 07:27:00.219827  100612 ignore.go:116] using globspec upload/src/README.md
I0807 07:27:00.219834  100612 ignore.go:122] Glob result upload/src/README.md
I0807 07:27:00.219839  100612 ignore.go:84] .s2iignore lists a file spec of vendor
I0807 07:27:00.219843  100612 ignore.go:116] using globspec upload/src/vendor
I0807 07:27:00.219853  100612 ignore.go:45] attempting to remove file upload/src/.dockerignore
I0807 07:27:00.219883  100612 ignore.go:45] attempting to remove file upload/src/.env.example
I0807 07:27:00.219905  100612 ignore.go:45] attempting to remove file upload/src/.git
I0807 07:27:00.223812  100612 ignore.go:45] attempting to remove file upload/src/.s2iignore
I0807 07:27:00.223836  100612 ignore.go:45] attempting to remove file upload/src/README.md
I0807 07:27:00.223855  100612 dockerfile.go:115] Constructing image build context directory at .
I0807 07:27:00.223893  100612 environment.go:41] Setting 7 environment variables provided by environment file in sources
I0807 07:27:00.223912  100612 dockerfile.go:223] Processing injected inputs: api.VolumeList(nil)
I0807 07:27:00.223918  100612 dockerfile.go:225] Processed injected inputs: api.VolumeList(nil)
I0807 07:27:00.223969  100612 dockerfile.go:296] Wrote custom Dockerfile to Containerfile.invalid
I0807 07:27:00.223977  100612 build.go:180] Application dockerfile generated in Containerfile.invalid

$ cat Containerfile.invalid
FROM registry.access.redhat.com/ubi8/php-80:latest
LABEL "io.openshift.s2i.build.commit.date"="Sun Mar 20 08:16:49 2022 -0400" \
      "io.openshift.s2i.build.commit.id"="fefad4475746a38a4197f36bff312dd056155f48" \
      "io.openshift.s2i.build.commit.ref"="master" \
      "io.openshift.s2i.build.commit.message"="Upgrade Laravel 9 (Laravel 8.68.1 -> 9.5.1)" \
      "io.openshift.s2i.build.source-location"="/home/scuppett/Development/php/laravel-s2i/."  Builder image must support chown command.
RUN chown -R 1001:0 /tmp/src
USER 1001
# Assemble script sourced from builder image based on user input or image metadata.
# If this file does not exist in the image, the build will fail.
RUN /usr/libexec/s2i/assemble
# Run script sourced from builder image based on user input or image metadata.
# If this file does not exist in the image, the build will fail.
CMD /usr/libexec/s2i/run

I would expect the RUN and CMD to point to the local files, not the originals for the builder image.
Using a nonexistant builder image (with the same repository) yields what I believe is the correct results.
$ s2i build . registry.access.redhat.com/ubi8/php-83 --as-dockerfile Containerfile.valid --loglevel=5
I0807 07:29:37.883152  100867 build.go:52] Running S2I version "v1.3.8"
I0807 07:29:37.883197  100867 util.go:70] Getting docker credentials for registry.access.redhat.com/ubi8/php-83
W0807 07:29:38.222461  100867 build.go:167] could not inspect the builder image for labels: reading manifest latest in registry.access.redhat.com/ubi8/php-83: name unknown: Repo not found
I0807 07:29:38.222513  100867 build.go:170]
Source:				.
Output Image Tag:
Environment:
Labels:
Incremental Build:		disabled
Remove Old Build:		disabled
Builder Pull Policy:		if-not-present
Previous Image Pull Policy:	if-not-present
Quiet:				disabled
Layered Build:			disabled
Docker Endpoint:		unix:///run/user/1000/podman/podman.sock
Docker Pull Config:		/home/scuppett/.docker/config.json
Docker Pull User:

I0807 07:29:38.222701  100867 install.go:269] Using "assemble" installed from "image:///usr/libexec/s2i/assemble"
I0807 07:29:38.222715  100867 install.go:269] Using "run" installed from "image:///usr/libexec/s2i/run"
I0807 07:29:38.222724  100867 install.go:269] Using "save-artifacts" installed from "image:///usr/libexec/s2i/save-artifacts"
I0807 07:29:38.222732  100867 scm.go:20] DownloadForSource .
I0807 07:29:38.223926  100867 clone.go:37] Downloading "." ...
I0807 07:29:38.223934  100867 clone.go:41] Cloning sources into "upload/src"
I0807 07:29:38.240659  100867 clone.go:57] Checked out "HEAD"
I0807 07:29:38.255107  100867 clone.go:63] Updated submodules for "HEAD"
I0807 07:29:38.261764  100867 install.go:269] Using "assemble" installed from "<source-dir>/.s2i/bin/assemble"
I0807 07:29:38.261790  100867 install.go:269] Using "run" installed from "<source-dir>/.s2i/bin/run"
I0807 07:29:38.261810  100867 ignore.go:84] .s2iignore lists a file spec of .dockerignore
I0807 07:29:38.261815  100867 ignore.go:116] using globspec upload/src/.dockerignore
I0807 07:29:38.261821  100867 ignore.go:122] Glob result upload/src/.dockerignore
I0807 07:29:38.261825  100867 ignore.go:84] .s2iignore lists a file spec of .env*
I0807 07:29:38.261829  100867 ignore.go:116] using globspec upload/src/.env*
I0807 07:29:38.261853  100867 ignore.go:122] Glob result upload/src/.env.example
I0807 07:29:38.261857  100867 ignore.go:84] .s2iignore lists a file spec of .git
I0807 07:29:38.261860  100867 ignore.go:116] using globspec upload/src/.git
I0807 07:29:38.261865  100867 ignore.go:122] Glob result upload/src/.git
I0807 07:29:38.261868  100867 ignore.go:84] .s2iignore lists a file spec of .idea
I0807 07:29:38.261871  100867 ignore.go:116] using globspec upload/src/.idea
I0807 07:29:38.261877  100867 ignore.go:84] .s2iignore lists a file spec of .s2iignore
I0807 07:29:38.261880  100867 ignore.go:116] using globspec upload/src/.s2iignore
I0807 07:29:38.261885  100867 ignore.go:122] Glob result upload/src/.s2iignore
I0807 07:29:38.261888  100867 ignore.go:84] .s2iignore lists a file spec of Dockerfile.s2i-fake
I0807 07:29:38.261891  100867 ignore.go:116] using globspec upload/src/Dockerfile.s2i-fake
I0807 07:29:38.261896  100867 ignore.go:84] .s2iignore lists a file spec of LICENSE
I0807 07:29:38.261899  100867 ignore.go:116] using globspec upload/src/LICENSE
I0807 07:29:38.261904  100867 ignore.go:84] .s2iignore lists a file spec of README.md
I0807 07:29:38.261907  100867 ignore.go:116] using globspec upload/src/README.md
I0807 07:29:38.261911  100867 ignore.go:122] Glob result upload/src/README.md
I0807 07:29:38.261915  100867 ignore.go:84] .s2iignore lists a file spec of vendor
I0807 07:29:38.261918  100867 ignore.go:116] using globspec upload/src/vendor
I0807 07:29:38.261925  100867 ignore.go:45] attempting to remove file upload/src/.env.example
I0807 07:29:38.261951  100867 ignore.go:45] attempting to remove file upload/src/.git
I0807 07:29:38.264608  100867 ignore.go:45] attempting to remove file upload/src/.s2iignore
I0807 07:29:38.264624  100867 ignore.go:45] attempting to remove file upload/src/README.md
I0807 07:29:38.264636  100867 ignore.go:45] attempting to remove file upload/src/.dockerignore
I0807 07:29:38.264648  100867 dockerfile.go:115] Constructing image build context directory at .
I0807 07:29:38.264663  100867 dockerfile.go:456] found override script file assemble
I0807 07:29:38.264667  100867 dockerfile.go:456] found override script file run
I0807 07:29:38.264689  100867 environment.go:41] Setting 7 environment variables provided by environment file in sources
I0807 07:29:38.264699  100867 dockerfile.go:209] Override scripts are included in directory "upload/scripts"
I0807 07:29:38.264705  100867 dockerfile.go:223] Processing injected inputs: api.VolumeList(nil)
I0807 07:29:38.264709  100867 dockerfile.go:225] Processed injected inputs: api.VolumeList(nil)
I0807 07:29:38.264845  100867 dockerfile.go:296] Wrote custom Dockerfile to Containerfile.valid
I0807 07:29:38.264850  100867 build.go:180] Application dockerfile generated in Containerfile.valid

$ cat Containerfile.valid
FROM registry.access.redhat.com/ubi8/php-83:latest
LABEL "io.openshift.s2i.build.image"="registry.access.redhat.com/ubi8/php-83:latest" \
      "io.openshift.s2i.build.commit.author"="Stephen Cuppett<EMAIL_ADDRESS>\
      "io.openshift.s2i.build.commit.date"="Sun Mar 20 08:16:49 2022 -0400" \
      "io.openshift.s2i.build.commit.id"="fefad4475746a38a4197f36bff312dd056155f48" \
      "io.openshift.s2i.build.commit.ref"="master" \
      "io.openshift.s2i.build.commit.message"="Upgrade Laravel 9 (Laravel 8.68.1 -> 9.5.1)" \
      "io.openshift.s2i.build.source-location"="/home/scuppett/Development/php/laravel-s2i/."
ENV DOCUMENTROOT="/public" \
    APP_NAME="Laravel" \
    APP_DEBUG="true" \
    APP_URL="http://localhost:8080" \
    LOG_CHANNEL="errorlog" \
    DB_CONNECTION="sqlite" \
    DB_DATABASE=":memory:"
USER root
# Copying in override assemble/run scripts
COPY upload/scripts /tmp/scripts
# Copying in source code
COPY upload/src /tmp/src
# Change file ownership to the assemble user. Builder image must support chown command.
RUN chown -R 1001:0 /tmp/scripts /tmp/src
USER 1001
RUN /tmp/scripts/assemble
CMD /tmp/scripts/run

I'm not sure if I'm invoking this right or if I'm missing some other key attribute or behavior here.

looks like a long standing bug in https://github.com/openshift/source-to-image/blob/980ca195116928b3beb61b25d5939d0044b3040b/pkg/build/strategies/dockerfile/dockerfile.go#L130-L134
since the image itself as a label that indicates it provides scripts, "hasAllScripts" ends up true and so we never scan the local repository for override scripts.
the logic looks almost deliberate, which is interesting but i'm pretty sure the behavior is incorrect.  Looks like it went in here: https://github.com/openshift/source-to-image/commit/6815fa6d92037acdfacf42dbe496813c12bdaff0
I reviewed/approved it but i think we missed this scenario when thinking about it.

/hold cancel

I see that the last release does not include those fixes -- v1;3.9 from Oct 19th, while PR got merged after
Last images we can get from RH registry would still suffer from that bug:
https://catalog.redhat.com/software/containers/source-to-image/source-to-image-rhel8/64ee172747850a7cc4b335b6?architecture=amd64&image=6543cd204a64b4ac1990e32e
Any ETA for next release?

... oh... nevermind ...
I guess it's not yet full working, ... https://github.com/openshift/source-to-image/issues/1141 ... assuming that 1;3.9-dirty was built including fix mentioned here ... case filed 3 days ago ... odds are: it was ...


chore: remove ui-dist folder

@harshpatel-crest I do not see any usage of ui-dist folder in the repo. Should we remove it?

Codecov Report

Merging #340 (766ecbe) into main (d35446d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #340   +/-   ##
=======================================
  Coverage   79.48%   79.48%
=======================================
  Files          28       28
  Lines        1867     1867
=======================================
  Hits         1484     1484
  Misses        383      383

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 d35446d...766ecbe. Read the comment docs.

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

v5.9.0
GitHub release

Your semantic-release bot :package::rocket:


Question about complex(?) relationships

Hello there!
I have been struggling to implement this kind of relationship:

Here is the code which I came up with:
///switch_link.rs:

...

#[derive(Copy, Clone, Debug, EnumIter)]
pub enum Relation {
    SwitchPort,
}

impl RelationTrait for Relation {
    fn def(&self) -> RelationDef {
        match self {
            Self::SwitchPortOne => Entity::belongs_to(super::switch_port::Entity)
                .from((Column::SwitchIdOne, Column::PortIdOne))
                .to((super::switch_port::Column::SwitchId, super::switch_port::Column::PortId))
                .from((Column::SwitchIdTwo, Column::PortIdTwo))
                .to((super::switch_port::Column::SwitchId, super::switch_port::Column::PortId))
                .into(),
        }
    }
}

impl Related<super::switch_port::Entity> for Entity {
    fn to() -> RelationDef {
        Relation::SwitchPort.def()
    }
}

///switch_port.rs:

...

#[derive(Copy, Clone, Debug, EnumIter)]
pub enum Relation {
    SwitchLink,
}

impl RelationTrait for Relation {
    fn def(&self) -> RelationDef {
        match self {
            Self::SwitchLink => Entity::has_one(super::switch_link::Entity).into(),
        }
    }
}

impl Related<super::switch_link::Entity> for Entity {
     fn to() -> RelationDef {
         Relation::SwitchLink.def()
     }
}

...

The problem is that if implementing it this way only relationships switch_id_two -> switch_id and port_id_two -> port_id are built.
So my question is: how can I make this work?
Thank you!

Hey @videobitva, you could use Linked for this.
https://www.sea-ql.org/SeaORM/docs/relation/chained-relations

Thank you, I tried to look into that. As far as I could understood Linked is used when you already have multiple tables with relations and you want to have a more comfortable way of making joins, am I correct?

Correct!

Ok, so my question remains the same: how (or can I even) define both switch_id_one -> switch_id, port_id_one -> port_id and switch_id_two -> switch_id, port_id_two -> port_id? Like, I can't use Linked before I have any relation between switch_port and switch_link defined. Or can I?

No, I still don't get it.
In order to make relations between switch_link and switch_port I need to implement this thing:
impl Related<super::switch_port::Entity> for Entity {
    fn to() -> RelationDef {
        Relation::SwitchPort.def()
    }
}

but I can't, because in order to do that I need to properly implement this:
impl RelationTrait for Relation {
    fn def(&self) -> RelationDef {
        match self {
            Self::SwitchPort => Entity::belongs_to(super::switch_port::Entity)...
        }
    }
}

Use Linked trait with https://docs.rs/sea-orm/0.4.2/sea_orm/?search=_linked find_linked methods

Sorry, gonna sleep now. Let's continue the conversation tmr loll

Thank you, have a good sleep =)


GCNs Perform Bad on These Datasets

Hi, I trained a GCN on cloth dataset. But it performs really bad.
I used the code from https://github.com/ki-ljl/GNNs-for-Link-Prediction/blob/main/homogeneous/gcn.py.
And I use sentence-transformers/all-MiniLM-L6-v2 to extract features from raw texts.
Here is how I deal with the raw data.
import json
import os
base_dir = "data/"
dname = "cloth"
edge_paths = [os.path.join(base_dir, dname, "link_prediction", f"{mode}.text.{n}.jsonl") for (mode, n) in
              [('train', 32), ('val', 32), ('test', 10000)]]
text_path = os.path.join(base_dir, dname, "corpus.txt")
print(edge_paths)
print(text_path)
text2id = {}
texts = []
i = 0
cnt = 0
with open(text_path, 'r') as f:
    line = f.readline().strip('\n').split('\t')[-1]
    text2id[line] = i
    texts.append(line)
    i += 1
    cnt += 1
    while line:
        cnt += 1
        line = f.readline().strip('\n').split('\t')[-1]
        if line not in text2id:
            text2id[line] = i
            i += 1
            texts.append(line)
print(f"{i} unique texts, {cnt} lines in total.")
# 667161 unique texts, 987246 lines in total.
edges = {"train": [], "val": [], "test": []}
for p in edge_paths:
    if "train" in p:
        mode = "train"
    elif "val" in p:
        mode = "val"
    elif "test" in p:
        mode = "test"
    else:
        raise ValueError("Invalid path")
    with open(p, 'r') as f:
        lines = f.readlines()
        for l in lines:
            l = json.loads(l)
            edges[mode].append([
                text2id[l['q_text']],
                text2id[l['k_text']]
            ])
with open(os.path.join(base_dir, dname, 'edges.json'), 'w') as f:
    json.dump(edges, f)
with open(os.path.join(base_dir, dname, 'text.txt'), 'w') as f:
    f.writelines(texts)
from sentence_transformers import SentenceTransformer
import numpy as np

model = SentenceTransformer('sentence-transformers/all-MiniLM-L6-v2')
embeddings = model.encode(texts, show_progress_bar=True)
np.save(os.path.join(base_dir, dname, 'embeddings.npy'), embeddings)

After all, I use these data for link prediction. But it got an accuracy of 50% (neg_sample_ratio=1.). It's just randomly guesses.
Do you conducted any experiments on GNNs? How do them perform?
Looking forward to your reply!

Hi there,
Thank you for your message and I'm really sorry for my late reply.
In our implementation of GNN baselines, we adopt the same in batch negative infoNCE loss compared with Patton. I believe the optimization objective matters a lot. We previously also tried the 0/1 classification loss but found it's not that effective compared with infoNCE loss. In addition, increasing the number of negatives (increasing the batch size in our case) can also help.
Hope the information helps.
Best,
Bowen


chore(main): release 2.6.3

:robot: I have created a release beep boop
2.6.3 (2024-08-13)
Bug Fixes

body_type default value is zero and not None (#271) (bf6b4f0)
c3: silent level as string (#270) (c851e33)

This PR was generated with Release Please. See documentation.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 37.76%. Comparing base (c851e33) to head (a64a2f5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #272   +/-   ##
=======================================
  Coverage   37.76%   37.76%
=======================================
  Files          84       84
  Lines        7470     7470
=======================================
  Hits         2821     2821
  Misses       4649     4649

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

:robot: Created releases:

v2.6.3
:sunflower:


How to run a motor in the opposite direction?

How to run a motor in the opposite direction ?
-PWM_FWD7 ?? Not working
Best Regards.

Sorry for the late response, but PWM_REV7 didn't work here?


HL2 skybox doesn't work properly with CSM on multiplayer session

It works as intended on singleplayer session but it doesn't work very well on multiplayer session and skybox is bugging with half of black texture.

This bug doesn't appear in maps with painted skybox (like default GMod maps are using) but i think that it requires fixing support of CSM for HL2 skyboxes.

I identified that the problem leads to that CSM uses commands that are cheat-protected on servers and it's required to unlock them by sv_cheats command but it would be cool if mod would use necessary commands without requiring to unlock them by sv_cheats.

This bug doesn't appear in maps with painted skybox (like default GMod maps are using) even on multiplayer session but i think that it requires fixing support of CSM for HL2 skyboxes.

Also i'm changing this because it happens on all skyboxes.

Fixed


expression did not evaluate constant Point tmpPoly[(int)this->PolyPoints.size()];

Hello friend,
I am trying to make in vs2015 c++ project and getting the subject error while building. Request your help to identify this. I tried a lot but nothing worked for me. snap shot below.

You can substitute lines 430 to 435 with:
std::vector<Point> tmpPoly = this->PolyPoints;

but I don't think this code can be used for production purposes. It is something that I have written a long time ago when I was still trying to figuring out how to code in c++. It has a lot of mistakes (and this line is one of them) and should be rewritten from scratch.


Separate SRAM DATA & ECC to distinct busses at Caliptra I/F

The SRAM data and ECC bit locations in SRAM bus are non-obvious at Caliptra boundary.
This creates unnecessary difficulty delineating data/ecc in the bus.
Integrators seeking to manage integration level machine check or reliability infrastructure with respect to SRAM data must redundantly include Caliptra ECC codes, increasing integrator area and timing overhead for SRAM and associated reliability wrappers.
Request:  Separate SRAM data and ECC to distinct busses at Caliptra interface.
/////begin description
The following EL2 memory interface should be defined with data and ecc as separate logic.  The existing iccm_data* and dccm_data* 39-bit wide signal should be partitioned into 32-bit data and 7-bit ECC.

Similarly the mailbox SRAM data should be partitioned into data and ECC signals at the caliptra_top.

Recommend that this is changed to:
    //SRAM interface for mbox
    output logic mbox_sram_cs,
    output logic mbox_sram_we,
    output logic [MBOX_ADDR_W-1:0] mbox_sram_addr,
    output logic [31:0] mbox_sram_wdata,
    output logic [6:0] mbox_sram_wecc,
    input  logic [31:0] mbox_sram_rdata,
   input  logic [7:0] mbox_sram_recc,

always_comb begin
    mbox_sram_cs = mbox_sram_req.cs;
    mbox_sram_we = mbox_sram_req.we;
    mbox_sram_addr = mbox_sram_req.addr;
    mbox_sram_wdata = mbox_sram_req.wdata.data;
    mbox_sram_wecc = mbox_sram_req.wdata.ecc;
    mbox_sram_resp.rdata.data = mbox_sram_rdata;
    mbox_sram_resp.rdata.ecc = mbox_sram_recc;
end

@nstewart-amd - I would suggest AMD to make the change of design & val collateral and turn it into the repo as its an enhancement as its a good change that would benefit everyone.
@andreslagarcavilla @pkwidzin-amd @Bryankel @nvpachou as FYI

Moving to Future as there is no update on the thread


Update Schema Locations for Absolute Pathing

The schema locations were not nicely addressable, such as those for wallet rendering (e.g. "https://identity.foundation/wallet-rendering/schemas/entity-styles.json")
Credential Manifest relies on absolute schema paths, and includes a currently incomplete/broken reference to the Presentation Definition and Presentation Submission schemas: https://identity.foundation/credential-manifest/#credential-manifest-3
This allows the Presentation Definition schemas to have well-known fully qualified URLs

@JaceHensley @csuwildcat hope this is OK. Just mostly shuffling things about for easier linkage.

@JaceHensley @csuwildcat hope this is OK. Just mostly shuffling things about for easier linkage.


Dice Sample

A few days ago I got my first GoDice and tested it right away in Foundry. Everything works except for a /r 1d20+1d20+1d20 test. The GoDice module recognizes the dice correctly, but Foundry does not display them correctly

Hey, same issue here. Do you use them with the DSA5 (The Dark Eye) System from Ulisses? I try to roll 3d20 rolls, and the Foundry GoDice Screen shows the right results, but in the Chat Window other results were printed.

Yes, exactly have the problem with DSA5. The developer of DSA5 and the people of GoDice see the problem with the module "Unfulfilled-rolls" which does not implement the dice test 1W20+1W20+1W20 correctly.
Von: Hijah82 @.>
Gesendet: Dienstag, 11. Juli 2023 00:47
An: foundryvtt/unfulfilled-rolls @.>
Cc: BunteDimensionen @.>; Author @.>
Betreff: Re: [foundryvtt/unfulfilled-rolls] Dice Sample (Issue #23)
Hey, same issue here. Do you use them with the DSA5 (The Dark Eye) System from Ulisses? I try to roll 3d20 rolls, and the Foundry GoDice Screen shows the right results, but in the Chat Window other results were printed.

Reply to this email directly, view it on GitHubhttps://github.com/foundryvtt/unfulfilled-rolls/issues/23#issuecomment-1629831006, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATWW2JHG2RH2MCJVFKE2BXDXPSA5BANCNFSM6AAAAAA2FDRHDI.
You are receiving this because you authored the thread.Message ID: @.***>


Fix integration-cli suite setup requirements

Moves the testRequires check from within setupRegistry
and setupNotary to when the Suite itself starts up.
This fixes a bug where testRequires in setupRegistry would
succeed and start registry, but testRequires in setupNotary
would fail. This immediately exits the goroutine with registry
still running.
Signed-off-by: Christopher Jones<EMAIL_ADDRESS>

LGTM

LGTM


ConfigurationExtensions.Get() should support TypeConverters

The current implementation uses Convert.ChangeType which only handles types that implement IConvertible.
A small change to allow the use of TypeConverters would improve mapping values to other types.
e.g.
#if NET45 || ASPNET50 || ASPNETCORE50
        public static T Get<T>(this IConfiguration configuration, string key)
        {
            var type = typeof(T);
            var value = configuration.Get(key);
#if ASPNETCORE50
            return (T)Convert.ChangeType(value, type);
#elif NET45 || ASPNET50
            if (typeof(IConvertible).IsAssignableFrom(type))
            {
                return (T)Convert.ChangeType(value, type);
            }
            else
            {
                var converter = TypeDescriptor.GetConverter(type);
                if (converter == null) throw new InvalidCastException("Could not find a valid TypeConverter for type " + type.FullName);

                return (T)converter.ConvertFromInvariantString(value);
            }
#endif
        }
#endif

Hey all, I guess with #199 in place, this can be closed?

POCO binding was moved from Options into the Configuration repo recently (see https://github.com/aspnet/Configuration/pull/205). It is currently in Microsoft.Framework.ConfigurationModel although I think it is misplaced there (see #210). Regardless of the specific location, would any part of this improvement apply to the new code?

@divega I'd think the logic when using the full runtime could still be applied at https://github.com/aspnet/Configuration/blob/dev/src/Microsoft.Framework.ConfigurationModel/ConfigurationBinder.cs#L217.  Essentially, anywhere Convert.ChangeType is used TypeConverters should also be considered.

This has been done

3476fad293466b2a574133428b37667d9c9574a6

Brilliant!


Fix compass no network issue

We want the confirm button to be disabled until the new location is properly reverse geocoded + saved,
This logic (=the deleted) caused the confirm button to be enabled before that
This is handle here already - only after network is available.

who changes it to false atm?
This is handle here already - only after network is available.


GWEI problem with Polygon

When using the polygon network the GWEI (Max priority fee aka miner tip) value that's pushed to the wallet is 2.5 and the transaction hangs and fails.
If it was 60 it would be OK and the transaction wouldn't take long to complete either.
blockchain.smartContract.methods       .mint(mintAmount)       .send({         gasLimit: String(totalGasLimit),         to: CONFIG.CONTRACT_ADDRESS,         from: blockchain.account,         value: totalCostWei,       })

Adding this changed the value and transaction go through now.
blockchain.smartContract.methods
.mint(mintAmount)
.send({
gasPrice:<PHONE_NUMBER>0,                            <==================== GWEI
gasLimit: String(totalGasLimit),
to: CONFIG.CONTRACT_ADDRESS,
from: blockchain.account,
value: totalCostWei,
})


Use job name improve CWL tool debugging (connected to #1343)

jenkins, test this please

@chapmanb @mr-c would either of you mind taking a look at how I'm attempting to label CWL jobs? Any suggestions on which CWL tool attributes to use for naming would be appreciated!

Christopher;
Thanks again for taking this on. This looks good for the jobName. I'd suggest using the executed command for the other debugging slot:
self.unitName = " ".join(cwltool.tool.get("baseCommand", []))

We don't have access to sample information at this point since it we're dealing with the workflow definition (in CWL) not the samples that are passing through, so don't have a way to set that. If Toil somehow knows to set unitName correctly we could instead concatenate the command information to the id and leave sample setting in unitName as is.
Thanks again for doing this. Once you get it integrated I'll work on rebasing it against my cwltool branch update.

jenkins, test this please

jenkins, test this please

@chapmanb it now handles both strings & lists for the baseCommand attribute - do you want to take another look before I merge it?

@chapmanb thanks, I'll make that change & we should have this merged today!


feat(site): add a security.txt well-known file

Description
Add a security.txt file to the .well-known directory, as per security.txt.
Motivation
Allow security researchers to properly report findings related to this project.
Additional Notes
closes #50

This doesn't look like it's actually deploying: https://reach4help.org/.well_known/security.txt
(perhaps gatsby doesn't copy hidden files in static by default?)
Also it should be .well-known not .well_known

Uhh nope, looks like it's a problem with netlify: https://community.netlify.com/t/hidden-files-removed-in-zip-deploy/8997
We could add a redirect rule in cloudflare and change the path? @mdeous


Verify GTMSessionFetcher SwiftPM fix

Test https://github.com/google/gtm-session-fetcher/pull/220
Testing only. Do not merge.
#no-changelog

Force pushing the base branch messed up the diffs. See the last few commits for the real necessary changes.


Sema: remove TypeResolver::diagnoseMoveOnlyGeneric (+ un-XFAIL more things)

Part of a series of PR's merging old and new infrastructure for noncopyable types.

@swift-ci smoke test

@swift-ci smoke test


[FEATURE] - Give Rich Text Field a fixed height with scroll and a draggable corner to adjust size

Summary
Right now if a rich text field has a large body of text, it fills up a lot of space on the view.
It would be preferable if it had a max height by default with a scroll bar, and had a draggable corner so a user can manually expand it if they want.
Basic Example
see summary
Drawbacks
No response
Unresolved questions
No response

This feature is available on v4.4.2 🚀
Feel free to give us feedback

This feature is available on v4.4.2 🚀
Feel free to give us feedback

Thanks @cregourd !
Just one minor issue. The draggable corner is somehow not draggable. 😃


Update CONTRIBUTING.md

Update contribution guidelines
Description
Add guideline on size of contributions.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run check

/cbmc run checks


同步element-plus

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow contributing guide English | (中文 | Español | Français).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer to relative issues for your PR.

同步element-plus代码


logout action not working on browser refresh

logout is working If I didn't refresh the browser.After refeshing the browser logout not working. Is there any solution for it?
Below is the saga code that I had written for the logout action
//...function

    export function* logout() {
          yield put({ type: 'SENDING_REQUEST', sending: true });

          try {
            let response = yield call(auth.logout);
            yield put({ type: 'SENDING_REQUEST', sending: false });
          } catch (error) {
            yield put({ type: 'REQUEST_ERROR', error: error.message });
          }
        }

        export function* logoutFlow() {
          while (true) {
            yield take('LOGOUT');
            yield put({ type: 'SET_AUTH', newAuthState: false });
            yield call(logout);
            forwardTo('/login');
          }
        }

        export function* logoutRoot() {
          yield fork(logoutFlow);
        }

        export function* watcherRoot() {
          const watcher2 = yield fork(logoutRoot);
        }

        export default [
          mainRoot,
          watcherRoot,
        ];

Hello @prudhvisays, what is auth.logout doing ? Is it possible that you actually are setting the authstate to false on logout, but some 'autologin' is happening (on reload) because of the fact that localstorage is checked for some data (maybe a token ?) when the store is initiated - which happens on reload ?

It is correct and working fine now.
Problem is with global saga. saga should be in app level(global) to work. Earlier injected auth saga in component level thats why it didn't work after browser refresh


The "generated-examples" folder is always empty for Kotlin projects

I noticed that for Kotlin projects the codegen does not generate the "...generated-examples" folder. It is always empty.
A good example is https://github.com/Netflix/dgs-examples-kotlin

the generated-examples is empty
for java projects such as this one: https://github.com/apollographql-education/intro-dgs.git
it generates the examples fine:

@twilczek - we actually don't generate data fetchers for kotlin, only support for java exists at the moment. We are planning to drop support for java, in order to reduce our maintenance overhead. This feature hasn't really been widely used/adopted so far.

thanks for the answer!


fix(api-v1): Optimise link value queries for Fuseki (DSP-1243)

This PR improves the performance of the API v1 SPARQL queries that find link values when using Fuseki.

[x] Use IRI literals instead of BIND.
[x] Use FILTER NOT EXISTS to check knora-base:isDeleted (as in #1739).
[x] Remove unnecessary graph patterns.

resolves DSP-1243

Thanks for reviewing!

Thanks for reviewing!


Implement Fabric platform support

Closes #307

Temporary draft status, but I am not sure if there is still a need to add or modify content

Hey @huanmeng-qwq nice to see another PR grom you.
I am afraid that translations of brigadier will not work properly in some cases e.g. when I will use a multilevel argument etc.

Also I have problem with this issue: https://github.com/Rollczi/LiteCommands/issues/349
I think we can resolve this as in the minestom platform (one full string argument represents all arguments)

Also I have problem with this issue: #349
I think we can resolve this as in the minestom platform (one full string argument represents all arguments)

It does look good, let me give it a try.
Also, it seems that Fabric does not support directly introducing sub projects as mod dependencies, which has led to the failure of CI

Also, it seems that Fabric does not support directly introducing sub projects as mod dependencies, which has led to the failure of CI

I think it is an issue with JDK version. I have tried to index your project with JDK 17 and it also failed.

我认为这是JDK版本的问题。我尝试使用 JDK 17 为您的项目编制索引，但它也失败了。

The reason why the problem only occurs on jdk17 is because the project requires 17 to run, so it was ignored in jdk8

I think it is an issue with JDK version. I have tried to index your project with JDK 17 and it also failed.

The reason why the problem only occurs on jdk17 is because the project requires 17 to run, so it was ignored in jdk8

Ah okay, you are right

It seems that my writing style is incorrect. I obtained the correct way through reporting issues

It should be possible now

It seems that my writing style is incorrect.

Don't worry, I will review your code :P

require assets/litecommands/icon.png
@Rollczi

require assets/litecommands/icon.png @Rollczi

How can I test the icon?

require assets/litecommands/icon.png @Rollczi

How can I test the icon?

exampes/fabric/run/mods: ModMenu mod
examples:fabric:runClient

modImplementation("dev.rollczi:litecommands-fabric:3.3.4+1.20.4") also work when I publish the litecommands to maven local.

require assets/litecommands/icon.png @Rollczi

How can I test the icon?

exampes/fabric/run/mods: ModMenu mod examples:fabric:runClient

I mean how to check icon

ModMenu supports you to view the icons of installed mods in the game

also work when I publish the litecommands to maven local.

implementation(project(path = ":litecommands-fabric", configuration = "namedElements")) It directly references local sub projects, supports debugging hot modifications, and can ensure that the project code is the latest

also work when I publish the litecommands to maven local.

implementation(project(path = ":litecommands-fabric", configuration = "namedElements")) It directly references local sub projects, supports debugging hot modifications, and can ensure that the project code is the latest

Yes you arę right.
I compared
modImplementation("dev.rollczi:litecommands-fabric:3.3.4+1.20.4")
with
modImplementation(include("dev.rollczi:litecommands-fabric:3.3.4+1.20.4")!!)

But I compared modImplementation("dev.rollczi:litecommands-fabric:3.3.4+1.20.4") with modImplementation(include("dev.rollczi:litecommands-fabric:3.3.4+1.20.4")!!)

include means that the dependency will be packaged into the META-INF/jars directory of the jar

Okay, I think all is done. But last issue. LiteCommands should be displayed in the mod menu?. (Currently it is only displayed example mod)

Okay, I think all is done. But last issue. LiteCommands should be displayed in the mod menu?. (Currently it is only displayed example mod)

Because of this paragraph, it will not be displayed by default
    "custom": {
        "modmenu": {
            "badges": [
                "library"
            ]
        }
    }

open:

Okay, I think all is done. But last issue. LiteCommands should be displayed in the mod menu?. (Currently it is only displayed example mod)

Because of this paragraph, it will not be displayed by default

thanks <3

@huanmeng-qwq I added a few changes. Please check if they suit you and everything is okay.

Okay, it can be merged

Thank you again ❤

Looks awesome, can't wait to test.

Looks awesome, can't wait to test.

@creatorfromhell You can checkout new snapshot 3.4.0-SNAPSHOT in the repository with snapshots:
maven("https://repo.eternalcode.pl/snapshots")

Looks awesome, can't wait to test.

@creatorfromhell You can checkout new snapshot 3.4.0-SNAPSHOT in the repository with snapshots ;P
maven("https://repo.eternalcode.pl/snapshots")

Cheers. The project I need it for I still have to port over from the previous lib I used xD


Add compatibility with aeson-1.1

@sjakobi Thanks for the PR! I have added some review comments, apart from those it looks good (and CI verifies backwards compatibility with aeson-0.9 and aeson-0.11).

I have added some review comments

I can't see these?!

Right. Because I entered them, but forgot to submit the review :-)

@fmthoma: I have added a commit that addresses your comments!

Merged. @sjakobi, thank you for yor contribution!

Thanks for the discussion and review! This is a much better solution than my first PR!


Allow Competition APIs to be used with womIds

Currently the competition API's require usernames to create or update teams. This works great for the website where users are expected to search, and add the player that way. 3rd party services however, would probably prefer to store wom's internal player id as the users name is not stable.
This however means that currently 3rd party services have to request the players username for every mutation they intend on doing, leading to extra API calls (a lot of them if this competition is not bound to a group).
My proposal is to add a new set of endpoints (or maybe an extra field in the request body?) So that IDs can be passed instead of names.

I think it's best if third party services use usernames as the player identifiers instead of player IDs.
What's your use case? Why are you using player IDs, and how?

We continued this conversation on discord (https://discord.com/channels/679454777708380161/696219254076342312/1209863709955194910) and I don't think the value outweighs the added complexity, closing.


Optimizing Bundle Size

I'm currently trying to optimize my bundle size and this module has become a sticking point. I've follow the instructions in the README but @sanity/client is still being included in the bundle.
If I remove the import import { createPreviewSubscriptionHook } from "next-sanity"; from my sanity.ts and add
export const usePreviewSubscription = (query: string, options: PreviewOptions) => { //createPreviewSubscriptionHook(config);
  return {
    data: options.initialData,
  };
}

in place of export const usePreviewSubscription = createPreviewSubscriptionHook(config);
I no longer see @sanity/client included in the bundle.
I'm testing this with nextjs-12.2.3-canary.10 using webpack5 configuration, unmodified from the default and @next/bundle-analyzer-12.1.6

I am also experiencing this.

Hi, we're working on a new version of preview mode that is easier to keep out of your bundle until it's actually needed.
It's an early version so we haven't added the code-splitting we want to yet (it loads a hefty eventsource polyfill that's really only needed if Safari fails to use cookies with the groq subscription).
Feel free to share your thoughts on what the API should look like etc 🙂


delete only waiting jobs

follow-up to #2403

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

Comparison is base (2d2827d) 91.14% compared to head (b606959) 90.44%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2426      +/-   ##
==========================================
- Coverage   91.14%   90.44%   -0.71%
==========================================
  Files         275      208      -67
  Lines       16501    10540    -5961
==========================================
- Hits        15040     9533    -5507
+ Misses       1461     1007     -454

Flag
Coverage Δ

jobs_cache_maintenance
94.40% <ø> (ø)

jobs_mongodb_migration
86.43% <ø> (ø)

libs_libapi
?

libs_libcommon
?

services_admin
87.60% <ø> (ø)

services_api
85.99% <ø> (ø)

services_rows
82.95% <ø> (ø)

services_search
80.95% <ø> (ø)

services_sse-api
94.19% <ø> (ø)

services_worker
92.68% <ø> (-0.07%)
:arrow_down:

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

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


Cannot drag on Android Chrome

Device: Samsung Galaxy s10 Android Chrome

Seems to be broken on Android Chrome.
Current Behavior:
Dragging just causes the page to scroll up and down
Expected Behavior: Able to drag the SVG children without page moving up and down

I was able to reproduce this problem on a recent ChromeBook, although I noticed that double-tapping the shape allowed me to drag it as expected.
I was able to get it to work correctly on the ChromeBook by adding "touch-action:none;" to the css of the svg, e.g. changing docs.css to style svg objects as follows:
svg {
width: 100%;
touch-action:none;
}
I'll investigate this further to see if there might be a better solution. Thanks-

This issue is fixed in version  0.1.4 by setting .style.touchAction = "none" on the target svg. Closing now.


Propose Logo

Hi @SSbug696 .. I'm a graphic designer and an open source enthusiast. I would like to contribute to your awesome project by proposing logo design for it if you allow me. I'll be doing it as a gift for free to your awesome project.
Best regard. Mirza zulfan.

Thanks for your response sir. I'll show it to you.

Hi @SSbug696 .. I've designed a logo for you project,, what do you think ? if you have an idea please share with me, it would be better for me.

let's take the blue one!)

Hi @SSbug696, What do you think about this?

2 number!) right

Hi, @SSbug696 I've created a pr. #2
Thanks! If you need to change in the future, dont hesitate to cantact me, Email<EMAIL_ADDRESS>


Cherab development branch is incompatible with Raysect development branch due to material API changes

Since Raysect 0.7.1 is released, I think until the new version of Cherab is ready we should set ci.yml to build with Raysect master branch.

Oops, created in wrong repository. Sorry.


fixed the layout issues on guidance index

@sparkdevelopment can you review and sign off?

@marcincichon Looks good


SOTO scroll item can't be added to a layout

The new xpac's hub teleport item [Wizard's Tower Teleportation Scroll] can't be added to a layout - it can't be found in the item list when I type its name into the 'find item' box. Not sure if the module needs an update to add new items to its list, or whether this is an API issue with automatically finding new items.

Issue resolved and should be available in latest release: https://github.com/tentose/Gw2QuickItem/releases/tag/0.1.0-6-main


Various documentation fixes

Changes
I see Task - TaskRun, and Pipeline - PipelineRun as pairs, so I move PipelineResource to the bottom of the table.
/kind misc

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.

/ok-to-test

/lgtm


Tensorboard does not display more then 100 bounding boxes

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No. (I've prepared pipeline.config, classes.bptxt and one tfrecord
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04.01 (Linux smok 4.15.0-45-generic #48-Ubuntu SMP Tue Jan 29 16:28:13 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux)
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):
v1.12.0-0-ga6d8ffae09 1.12.0
(I've also tried b'v1.12.0-6120-gdaab2673f2' 1.13.0-dev20190116 and results are the same)
Python version: 3.5.2
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:10.0
GPU model and memory: GTX 1080Ti, RTX 2080

Describe the current behavior
I run training
python object_detection/model_main.py --pipeline_config_path=/mnt/data/environments/bag/100bboxes/pipeline.config --model_dir=/mnt/data/environments/bag/100bboxes/exp

This is only one step training and its goal is to visualize bounding boxes in tensorboard images tam.
On the "IMAGES" tab in tensorboard I don't see all bounding boxes which are in the tfrecord file.
Options provided in the pipeline.config
num_visualizations: 200
max_num_boxes_to_visualize: 200

have no effect on this.
Describe the expected behavior
I see all bounding boxes (ground thruth) in the tensorboard>images (pictures on the right)
Code to reproduce the issue
I've created repo where you can find

pipeline.config - one step training,  the same tfrecord for training and evaluation
classes.pbtxt - two sample classes
t.tfrecord - containg two white images: one with 100 bounding boxes and second with 200 bounding boxes
notebook-images-view.png - view from notebook when I've tried visualise bboxes
tensorboard-images-view.png - tensorboard images view

For faster resolution please post the issue on TF-tensorboard
repository.Thanks!

I've just posted https://github.com/tensorflow/tensorboard/issues/2413

@ravikyram I believe this is an issue with the tensorflow/models/research/object_detection. I tried to debug the issue but I do lack a lot of context here. Can you please reassign it to the person from the owners listed here? Thanks!

Today I made also simple experiments.

In first experiment I prepared 100 images with less then 100 objects on every and labeled it automaticaly. It was training set. Similarly I prepared tfrecord with 30 images for evaluation
In second experiment I prepared 100 images with more then 150 objects on every and labeled it automaticaly. It was training set. Similarly I prepared tfrecord with 30 images for evaluation

I trained faster rcnn (resnet101) (10000 steps) on each set.
On this image  you can see comparison of mAP for first (red one) and second (rose one) experiment. I'm almost sure that there is also problem with training/evaluation.
On second image  you can see sample images from tensorboard images tab for set with more then 150 objects and on third  for set with less then 100.
Will it helps if I upload somewhere this tfrecords and pipelines (its about 50M) ?

Hi,
I've made a config and tfrecord's for this issue on my drive You can download it, change paths in pipeline.config and run training (run.sh - set CUDA_VISIBLE_DEVICES and paths).
Can I help you in some way to solve this issue?

I've tried tensorpack lib and succesfully got   results for image with more then 100 bboxes

I’ve struggle the same issue and done some investigation around this topis. It looks like the issue is only in the evaluation step of a training process.
Experiment 1 - training on pictures with less than 100 objects
I’ve done an experiment where training set had only images with less than 100 objects. I’ve done the training, which went fine - mAP around 0.82.
Then, when I was exporting the model for inference, I’ve changed in pipeline.config, in section second_stage_post_processing { batch_non_max_suppression { }}value of parameter max_detections_per_class to 300.
I’ve checked the predictions for a photo where there was over 100 objects in the picture. The number of predictions was correct - it marked all objects.
Experiment 2 - training on pictures with more than 100 objects
I’ve then done the same experiment, but with a training set having some of the images with over 100 objects, and parameter max_detections_per_class set to 300 from the beginning. The final mAP was around a 0.3 - much less than in the first experiment. But, the training loss curves looked similar. This means for me, that the training process is going fine, only the evaluation step has some issues.
Metrics
Legend:
Orange - evaluation, images with less than 100 objects
Red - evaluation, images with more than 100 objects
Gray - training, images with less than 100 objects
Blue - training, images with more than 100 objects
Evaluation mAP

Evaluation loss

Training loss

Evaluator Issue
I’ve checked evaluator sorce code. The COCO evaluator has a fixed value of 100 maximum detections (link to source code). This causes, that mAP and other metrics calculations are incorrect, because in the evaluation dataset there are examples, where in Ground Truth photos there is over 100 objects, and during evaluation only 100 detections are done.
Any ideas how to tackle the problem with maxDets limit in COCO evaluator?
I've already tried setting the maxDets parameter to [1, 10, 300] for box_evaluator (after this line), but this caused, that mAP was calculated as -1.000, so something were not working fine.

https://github.com/tensorflow/models/issues/5465

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


erase all getters and setters

after convo with @ClementWalter, erase all getters and setters
### Tasks
- [x] erase all getters and setters, use directly properties (that you'lll turn from private to public)
- [x] add toString() to relocatable

111


failed to run preseeded command / wget

ubuntu-16.04.4-server-amd64-unattended.iso is failing to complete unattended install with this error message:

The url should be:
https://raw.githubusercontent.com/netson/ubuntu-unattended/master/start.sh

changing the path is not enough I still get the error even with correct path, I think wget is missing in the target

changed from wget to curl and now it works

Thanks for the PR, merged and closing now