docs: in-place removal of source block macros to fix docs build

Release notes
[rn:skip]
What does this PR do?
Fixes the docs build.
TIL that [source] blocks are handled as subdocuments in asciidoc, and their ability to link to the outer doc that contains them is implementation-specific. While my local docs tools can resolve the references, the docs build was broken by them 😩
Why is it important/What is the impact to the user?
It removes a broken thing from the docs used by the combined LS docs build.
We will need to mutate the tag held in github for the versioned plugin reference to pick up this fix since it is sourcing from rubygems version number (which is less than ideal), but with net zero code changes on this pre-1.0 project I am comfortable with the ramifications.
Once merged, we will need to push a tag deletion to github:
git push --delete origin v0.1.0

And then this will need to be re-tagged (preferably including a cryptographic signature).
git fetch origin
git checkout ${PR_MERGE_COMMIT}
git tag -s v0.10
git push origin v0.1.0

@yaauie, thanks for the fix.  Merging so that correct formatting is safely in place in plugin source.
Because this is a docs-only change in a pre-release version, I don't believe that the above gymnastics are necessary. Let's discuss.


How to use animateNodes in pure javascript

Hello, is it possible to import animateNodes function in pure javascript?
I have tried using Sigma.animateNodes, but I keep running into the error, Sigma.animateNodes is not a function.

animateNodes is pure function, it's not a function that is part of the Sigma object.
SO you just have to import it : import { animateNodes } from "sigma/utils/animate";
You can take a look at this code : https://github.com/jacomyal/sigma.js/blob/main/examples/layouts/index.ts#L95

I am unable to import that in a pure javascript file. I am not using a framework like react, but just javascript. Is is possible to import animateNodes using ES6 syntax?

import { animateNodes } from "sigma/utils/animate"; is an ES6 syntax, so I don't understand your question.
Please provide some code (or better a code sandbox) to see what you have tried so far.

Here is a code sandbox link for the issue I am having. https://codesandbox.io/s/black-pond-4qrdfx
I tried the ES6 import method but I get the error Uncaught TypeError: Failed to resolve module specifier "sigma/utils/animate". Relative references must start with either "/", "./", or "../".
Adding a "./" also does not fix the error. I get CORS error on codesandbox but on VSCode I just get 404 error.
Furthermore I have tried using the function as is, and using Sigma.utils.animate.animateNodes & Sigma.utils.animateNodes

Where did you find the sigma.js file ?
Why are you not using npm for dependencies in your example ? (same question for graphology)

What do you mean where did I find it, its literally in the releases of this github repo.
And I have been saying that this is for pure javascript and not node or react.

This is a pure JS example which use npm for dependencies, and that feets your needs : https://codesandbox.io/s/competent-leaf-6triuf

Hi @vinaysb, did you solve the problem? I'm trying too but I'm not having success

Hey @AndreaCirina No I did not, I ended up switching to d3.js due to better documentation.
Sadly even the "pure js" example presented here is still using a package.json file which my usecase (in python django) would not work.

thanks for the reply, i am also creating a web application with python, in case i will change too. Thanks again and have a nice evening!


Would be awesome to get the Material animation from the spec sheet.

The spec sheet recommend using this kind  of animation.
Would be awesome if you could implement it.
Thanks for your work BTW, it's a great lib.

Will definitely implement that somehow.
And thanks, I really appreciate it.

FragmentTransaction transaction = getSupportFragmentManager().beginTransaction();

// we just would have to create 2 anim xml files with the animation.
transaction.setCustomAnimations(android.R.anim.fade_in, android.R.anim.fade_out);
transaction.replace(R.id.fragmentContainer, fragment);
transaction.commit();

Does somebody already has a similar animation?

I've tried to get the same transition that we have in the spec. It looks more or less identical! Here my code:
anim/fade_out.xml
<?xml version="1.0" encoding="utf-8"?>
<alpha
    xmlns:android="http://schemas.android.com/apk/res/android"
    android:duration="@android:integer/config_shortAnimTime"
    android:fromAlpha="1"
    android:toAlpha="0"/>

anim/slide_in_up.xml
<?xml version="1.0" encoding="utf-8"?>
<set xmlns:android="http://schemas.android.com/apk/res/android">
    <translate
        android:duration="@android:integer/config_shortAnimTime"
        android:fromYDelta="5%p"
        android:toYDelta="0%p"/>

    <alpha
        android:duration="@android:integer/config_shortAnimTime"
        android:fromAlpha="0"
        android:toAlpha="1"/>
</set>

And in the fragmentManager set:
transaction.setCustomAnimations(R.anim.slide_in_up, R.anim.fade_out);


Sign Karura Transaction - warning

When using @substrate/txwrapper-polkadot to sign a Karura transaction, it throws the following warning:
REGISTRY: Unknown signed extensions SetEvmOrigin found, treating them as no-effect

at several occassions:
getRegistry, methods.balances.transferKeepAlive, construct.signedTx.
How to eliminate this warning?

This warning happens when the polkadot-js TypeRegistry is setting the metadata.
I wrote the above PR to fix the following.

@will-yijinin You can now add the userExtension as part of the options param to each function.
So to silence the warning in your case is would look something like:
userExtensions: { 'SetEvmOrigin': { payload: {}, extrinsic: {} } };

@TarikGul Thanks. Which version do I need to upgrade my dependency to?

@will-yijinin A new release tomorrow is going out early tomorrow or later today which will have the reflected changes for you.


Feature results format

Note this is for a PR in the Hygieia project - setup support for mocha js spec file results for publisher plugin

required for https://github.com/Hygieia/Hygieia/pull/2799

looks good to me

Thanks for the +1’s so far... what are we missing?


whatsapp-web.js / whatss

I have followed the tutorial correctly at every stage and I am using node v22, but after installing whatsapp-web.js and creating the main code, when I run the program it appears "TypeError: Cannot read properties of null (reading '1')", so what do I have to do? Thank You

I'm facing the same problem here... think that whatsapp made same changes that has broken the wwebjs...


BREAKING: rename the syncronous lintText method to lintTextSync

Fixes: https://github.com/feross/standard/issues/807

BREAKING: rename the syncronous lintText method to lintTextSync
Add an asyncronous lintText method (that just calls lintTextSync
internally)

This effectively undoes the breaking change introduced in 6.0.0, making it safe to upgrade from standard-engine 5.x to 7.x without introducing any breaking changes.
But we also keep the new option to lint source text synchronously, just under a new lintTextSync() method. This is better than overloading lintText to behave differently depending on whether a callback is passed in or not, IMO.
Related: https://github.com/Flet/standard-engine/issues/156

Also, @Flet, looks like @bcomnes is not a collaborator on this repo. Want to add him?

Invited @bcomnes to the repo

@Flet By the way, the context for this is that I want to upgrade from 5.x to 7.x in the standard 10.0.0 release that I'm about to do, but I don't want to break any of the editor plugins.

@feross makes a ton of sense! Thanks!

lgtm!

7.0.0


Add client_id parameter to sign-out request

When using the Application isolation SSO configuration https://docs.microsoft.com/en-us/azure/active-directory-b2c/active-directory-b2c-token-session-sso#single-sign-on-sso-configuration, attempting to signout without specifying the client_id parameter results in a successful navigation to your post_logout_redirect_uri, but does not clear the cookie on the b2clogin site.  A subsequent login attempt will immediately succeed, without a credential re-prompt.

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

I'm not sure if this is the correct forum for this discussion, but taking a step back from this particular documentation change;
The client_id parameter seems redundant given the value can be inferred from the aud attribute in the id_token_hint.   My personal preference would be for the B2C RP to examine the id_token_hint in order to determine which session to remove, rather than introduce an additional parameter. This would seem to more closely align with https://openid.net/specs/openid-connect-session-1_0.html#RPLogout

Hi @eric-winkler, thanks for the contribution! I've engaged the team on this one, will update here when/if I have more info or questions. Will need to test on this end.

After further testing and some offline conversations with @WhippsP we've concluded that the client_id parameter is only required when both the following apply;

'Application' isolation SSO configuration
'Require ID Token' is set to 'no'

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

Thanks @eric-winkler! LGTM.
#sign-off

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


Unable to make scopes work.

I've an Asp.Net Core WebAPI and a .Net Core console client. Both properly registered as Azure AD applications.
No matter what I tried, I can not make it work with scopes.
Following is a simplified version of my code, with three cases.
The third is the only one working.
// Settings.TenantId = the Directory ID from AAD Properties
// Settings.ClientId = a registered client app ClientId
// Settings.ClientSecret = a registered client app ClientSecret
// Settings.Scope = a registered WebAPI app Application ID Uri/a scope name , e.g "api://GUID/SCOPE_NAME
// For Authority  I tried a) without any url path after the tenant b) .../oauth2/v2.0/token c) .../v2.0 and many more
string Authority = $"https://login.microsoftonline.com/{Settings.TenantId}/oauth2/token";
// Case 1. Not working
IConfidentialClientApplication AppBuilder = ConfidentialClientApplicationBuilder.Create(Settings.ClientId)
.WithClientSecret(Settings.ClientSecret)
.WithAuthority(Authority)
.Build();
// Settings.Scope tried with HttpUtility.UrlEncode() too
AuthenticationResult AuthResult = AppBuilder.AcquireTokenForClient(new string[] { Settings.Scope })
.ExecuteAsync().Result;
...and keep getting the message
"(AADSTS70011: The provided request must include a 'scope' input parameter.
The provided value for the input parameter 'scope' is not valid. The SETTINGS.SCOPE HERE is not valid."
// Case 2. Not working - First Response returns OK, but if I use the returned token further I get "Unauthorized".
Dictionary<string, string> FormData = new Dictionary<string, string>();
FormData["client_id"] = Settings.ClientId;
FormData["scope"] = System.Web.HttpUtility.UrlEncode(Settings.Scope);  // and without url-encoding
FormData["client_secret"] = Settings.ClientSecret;
FormData["grant_type"] = "client_credentials";
Uri Uri = new Uri(Authority);
string AccessToken = null;
string JsonText;
using (var Client = new HttpClient())
{
HttpResponseMessage Response = await Client.PostAsync(Uri, new FormUrlEncodedContent(FormData));
JsonText = await Response.Content.ReadAsStringAsync();
var JO = JsonConvert.DeserializeObject(JsonText) as dynamic;
AccessToken = JO.access_token;
//... using AccessToken to call the WebAPI returns "Unauthorized"
}
// Case 3. Works
Authority = $"https://login.microsoftonline.com/{Settings.TenantId}";
var context = new AuthenticationContext(Authority);
var credentials = new ClientCredential(Settings.ClientId, Settings.ClientSecret);
AuthenticationResult result = await context.AcquireTokenAsync(Settings.WebApiAppIdUri, credentials);
return result.AccessToken;
Any pointers?

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

ID: 906f1086-776e-9abe-b6cc-bf93e84dc238
Version Independent ID: 50b27c9e-8f8a-3425-a1bb-b5d30e9d95e4
Content: Protected web API - app code configuration
Content Source: articles/active-directory/develop/scenario-protected-web-api-app-configuration.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@tbebekis
Thanks for the question! We are investigating and will update you shortly.

@tbebekis
Case 1: My assumption is the scope you passed is not being recognized by Azure AD. Are you passing a custom scope? is this scope present in the App manifest?
Case2: Can you parse the token in https://jwt.ms and see that the scope you sent is coming in the token. It is possible that the operation requires a higher privilege or some other scope to perform that action.
Case 3: is the API call working in this scenario?

Case 1 and 2: I removed that scope already.
Case 3: This is what I'm using right now and it works. With the addition of a "service". E.g.
Dictionary<string, string> FormData = new Dictionary<string, string>();             FormData["client_id"] = Settings.ClientId;             FormData["client_secret"] = Settings.ClientSecret;             FormData["grant_type"] = "client_credentials";             FormData["resource"] = Settings.WebApiAppIdUri;             string Url = $"https://login.microsoftonline.com/{Settings.TenantId}/oauth2/token";
So no scopes for now.
I'll try to add scopes again and let you know the results.
For now I'm just closing this ticket.
Regards
Theo.


Use jQuery .on() to listen for events, rather than .click()

This change allows the Knowledge Panel to reload upon being clicked a second time.
Per https://stackoverflow.com/a/14187154/8748686

This solution enables generating the popover and executing the AJAX queries each time the button is clicked.
Merits a little more sleuthing with respect to what is happening (e.g. is it that the popup is created once with the html first used to create it and then not regenerated?).


Trace Filter support

Describe the feature
tracking issue for trace_fitler which is very useful for indexing
ref https://github.com/paradigmxyz/reth/issues/3661#issuecomment-1735506192
trace_filter returns parity traces similar to eth_getLogs but for traces
https://github.com/paradigmxyz/reth/blob/7024e9a8e91e9546ffd20b5aae65e85888f591a6/crates/rpc/rpc-types/src/eth/trace/filter.rs#L9-L24
all filter fields can be checked via Transactions alone, if I understand the filter options correctly.
Supporting the block range shouldn't be too difficult, at worst this is essentially a batch of replay_block requests.
Not sure how to support address fields without limiting blocks, because atm we don't have indices for that
@tjayrush what is a usual trace_filter request?
Additional context
No response

what's a typical block range?
since this is quite similar to replay_block with some extra steps, I can at least add support for block filters

Again, no real typical value. We would probably adjust at whatever value is the fastest (after experimenting). We batch 5,000 blocks at a time, but I'm sure that's way too many, so I've been thinking of breaking it up into 100 50-block chunks. (Currently, we query each block separately with trace_block). But we'd adjust.

sg!
I'll implement it with some limitations:
block range required
block range limited (needs some testing)

I'll test it (once I'm synced...)

Is this still an open issue?


HDWallet supports other Coins

How to suppert other coins ?
// m / purpose' / coin_type' / account' / change / address_index
Such as LTC: m/44'/2..., DOGE: m/44'/3...

https://github.com/satoshilabs/slips/blob/master/slip-0044.md


Documentation for how to package meteor applications

Would be nice to see a streamlined way to package and distribute a meteor application. There doesn't seem to be any solid frameworks anywhere.

@AlexFrazer Have you seen this:
https://github.com/sircharleswatson/Electrometeor

Closing since there are excellent article plus example now, @sircharleswatson you can also add it to https://github.com/sindresorhus/awesome-electron.

Here is a very cool example video showcasing the ease with which you might run your Meteor app on Electron:
https://www.youtube.com/watch?v=1OpsJp1_OK4


Set-Content : Access to the path 'C:\Test.html' is denied.

Set-Content : Access to the path 'C:\Test.html' is denied.
At C:\Users\przemyslaw.klys\Documents\WindowsPowerShell\Modules\PSWriteHTML\PSWriteHTML.psm1:12230 char:17
+ ...     $HTML | Set-Content -LiteralPath $FilePath -Force -Encoding $Enco ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : PermissionDenied: (C:\Test.html:String) [Set-Content], UnauthorizedAccessException
    + FullyQualifiedErrorId : GetContentWriterUnauthorizedAccessError,Microsoft.PowerShell.Commands.SetContentCommand

Provide workaround in case access to path is denied when saving for EMail/Save-HTML

Would the desktop of the current user be a good place to save it to?
# Access Denied
if((Test-Path -LiteralPath $FilePath) -eq $true){
    Write-Verbose "Save-HTML - Saved HTML to file $FilePath"
}
else{
    $NewFilePath = Split-Path -Path "$FilePath" -Leaf
    $ErrorMessage = $_.Exception.Message -replace "`n", " " -replace "`r", " " # Not working for me
    Write-Warning "Save-HTML - Failed with error: $ErrorMessage" # Not working for me
    Write-Verbose "Save-HTML - Saving HTML to file $env:USERPROFILE\Desktop\$NewFilePath"
    $HTML | Set-Content -LiteralPath "$env:USERPROFILE\Desktop\$NewFilePath" -Force -Encoding $Encoding
    if((Test-Path -LiteralPath $NewFilePath) -eq $true){
        Write-Verbose "Save-HTML - Saved HTML to file $env:USERPROFILE\Desktop\$NewFilePath"
    }
    else{
        Write-Verbose "Save-HTML - Failed"
    }
}

You may overwrite someone's file. I guess Temp Directory is better place with random file name but with information to screen where it was saved. You also need try/catch. Would you submit this and other stuff as PR's? :-)

Thank you @xlrod


Fix HIP, hwloc and test_install

HIP overhaul, Hwloc and test_install fixes.
On hwloc
There were a few issues with the CMake hwloc setup.

hwloc was always built even when a valid version already existed on
the system. To make this work properly it is required to unset a few
HWLOC_XXX variables such that we always redo the detection of whether
a system version is available or not.
The FindHWLOC module was also searching in the build directory whereas
it should not, hwloc available in a build directory is invalid as a
system dependency and would only work with Ginkgo.
hwloc is a public dependency of Ginkgo but was not installed when we
use the one from our build directory, that would create errors with
missing libraries in some cases.
The HWLOC_XMLFILE should not be managed by Ginkgo, hwloc already
takes care of it, if the environment variable HWLOC_XMLFILE is set, it
uses that as base, otherwise it uses the machine topology it can
detect.

On test install
Test install was missing a HIP version which I added. In addition, the
commands were not properly triggered thus only the base (non CUDA
version) was tested, that is now fixed.
On HIP
This part saw a lot of changes.

ROCm < 3.5 support is dropped as it is now old and all HPC clusters
always update the ROCm version since they know the ecosystem is somewhat
unstable.
Instead of linking against hip::device in ginkgo_hip, I link
against libamdhip64.so publicly.
In general, the purpose of this rework is to depend more on CMake
transitive properties such that we remove special cases. Thanks to this
and the previous point as well, it is possible to remove entirely the
hip_helpers.cmake file and remove all of the bans we were doing.
Thanks to relying on transitive properties, there is no need for the
special_linkage test case anymore.
Instead of building the RPATH_FOR_HIP chain, in Ginkgo everything
uses the CXX language for linking, which properly takes into account
RPATH information unlike when liking with the HIP language. For users,
a simple string GINKGO_INSTALL_RPATH_FOR_HIP is provided which is a
linking flag to be given to the HIP language linker after installing
Ginkgo.
In general, there is also better management or relying more on the
defaults for some environment variables and HIP variables.

Open issues
An annoying issue (still), although it's mitigated thanks to linking
publicly against libamdhip64.so which doesn't create inconvenience to
the user, is that we have to include hip/hip_runtime.hpp in our public
headers (to find __forceinline__ as well as HIP_DEVICE_COMPILE, see
https://github.com/ginkgo-project/ginkgo/blob/develop/include/ginkgo/core/base/types.hpp#L46-L85.
In addition, the GKO_ASSERT cannot be reworked as well otherwise it
creates mysterious issues in the check_headers test, see:
https://gitlab.com/ginkgo-project/ginkgo-public-ci/-/jobs/1073846520

This should now be ready for review.

I added some explicit RPATH management for hwloc & the HIP libraries but I'm not sure if it should stay: I'm not sure if CMake is actually populating the RUNPATH in which case it's fine, or if it's the RPATH which the user cannot configure (as would be needed when copying the libraries to another platform).

There is one extra thing to consider for this PR: what default RPATH behavior we want to consider for our dependencies, e.g. HIP, and if we want optional "convenient" features to manage the RPATH directly.
Context
The RPATH is a library information field that states into which PATH the dependencies can be searched into first and foremost. You can usually read that information by doing objdump -x libginkgo.so for example.
On Linux, the order of searching is somewhat like this:

DT_RPATH (the one we said above, from the dynamic library itself)
LD_LIBRARY_PATH (the user defined environment variable)
DR_RUNPATH.

This means that if we do not provide any RPATH information by default, the user needs to provide all the correct PATH of our dependencies (e.g. hipSPARSE, rocSPARSE, ...) into the LD_LIBRARY_PATH environment variable.
Issue: an installed library should be relocatable. Should it always though?
The title summarizes the main problem, when we bundle RPATH information (since I do not know how to provide the RUNPATH in CMake and it's also not as powerful as RPATH due to some other differences), we basically prevent the user from customizing the dependencies, e.g. copying an installed ginkgo library into another platform with potentially different paths/libraries and have it automatically adapt to this new setup (through the LD_LIBRARY_PATH). On the other hand, putting RPATH information for some dependencies is convenient for users which can live with a "broken" system with missing LD_LIBRARY_PATH settings.
Question
Thus the question is, which path should we take forward:

Leave things as is (let users always populate a correct LD_LIBRARY_PATH)
Have a CMake setting for addingRPATH information and make it default.
Have a CMake setting for adding RPATH information and isn't default.

Note that actually setting all the RPATH information of our dependencies isn't always enough for the users' convenience, because Ginkgo itself will depend on libhipsparse.so, but libhipsparse.so doesn't bundle any RPATH information and thus will fail to find, for example, librocsparse.so. If you do an install of Ginkgo the commit e6505da on  a HIP based system (e.g. amdci) and use lddtree with an empty LD_LIBRARY_PATH you will see that happen.

As some extra consideration on the topic of RPATH and what Ginkgo's behavior should be, it seems that MFEM and libCEED both encode all the RPATH information into their library by default, thus creating non relocatable libraries. If many prominent packages decided to remove possibility for customization/relocatable code in order to simplify their user's life, maybe we should consider going that way as well.

I just tried with the latest version of this branch and also the latest version of develop; they both behave the same way.  If I compile with gcc (9.2.0), I see the errors, but if I compile with ROCm clang, I don't.  It only happens when building the examples, e.g.
[ 84%] Linking CXX executable ginkgo-ranges
../../hip/libginkgo_hip.so.1.3.0: undefined reference to `gko::MachineTopology::get_pci_device(std::string const&) const'
collect2: error: ld returned 1 exit status
make[2]: *** [examples/ginkgo-ranges/ginkgo-ranges] Error 1
make[1]: *** [examples/ginkgo-ranges/CMakeFiles/ginkgo-ranges.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 84%] Linking CXX executable cb-gmres
../../hip/libginkgo_hip.so.1.3.0: undefined reference to `gko::MachineTopology::get_pci_device(std::string const&) const'
collect2: error: ld returned 1 exit status
make[2]: *** [examples/cb-gmres/cb-gmres] Error 1
make[1]: *** [examples/cb-gmres/CMakeFiles/cb-gmres.dir/all] Error 2

So whatever is happening, whether it's my configuration on this machine or the hwloc I'm using or something else, it's not related to any changes in this branch.

I see. thanks. I will look into that.


Revert "Fix bugs in conv functions"

Reverts openai/iaf#8
Apparently causes NaN's:
https://github.com/openai/iaf/issues/11

I am executing tf_train.py (num_gpus=1). In forward function for two for loops it is running fine for i=0,j=0;i=0,j=1; but continuously it is taking huge memory. For i=1,j=0 of for loop in forward function it calls sub_layer.up which in turn calls conv2d, it got almost very slow and memory space is increasing at execution of line no. 49 (layer.py) of con2d function. Anyone please help me to resolve this increased memory issue.


Upgrade ffi to 1.0.0

current master and dev branch of flutter is not compatible with ffi 0.1.3, almost all packages depends on ffi and ffi itself will build failed. It was warning before not error.
...../win32-1.7.4/lib/src/structs.dart:2999:7: Error: Struct 'NLM_SIMULATED_PROFILE_INFO' is empty. Empty structs are undefined behavior.
class NLM_SIMULATED_PROFILE_INFO extends Struct {}
      ^
...../ffi-0.1.3/lib/src/allocation.dart:47:33: Error: Expected type 'T' to be a valid and instantiated subtype of 'NativeType'.
  final int totalSize = count * sizeOf<T>();

The top level functions free and allocate are moved into abstract class in ffi 1.0.0

maybe related issue
https://github.com/dart-lang/sdk/issues/44622
https://github.com/dart-lang/sdk/issues/44621

Thanks!

BTW, when building/hot restart an app with this qjs package, a lot of warnings (mostly number type related like long, int, size_t) will be generated.
Is this usual and allowed for c/c++ program without type cast? I am not familiar with c++, just try to clear the doubts. Thanks.

Wonderful project!
Have you encounter array problem in Struct, i.e. https://github.com/dart-lang/ffigen/issues/68#issuecomment-784656084
// C struct
struct TW_VERSION {
  char[34] info;
}

// dart class
class TW_VERSION extends Stuct {}

I could get the struct address but fail to perform static_cast like operation on it


Fix stuck build in openshift-ci

upgrade is waiting for an input
Transaction Summary:
Installing: 0 packages
Reinstalling: 0 packages
Upgrading: 3 packages
Obsoleting: 0 packages
Removing: 0 packages
Downgrading: 0 packages
Is this ok [y/N]:

/test servicemesh-ocp4.14-lp-rosa-classic-images


Diagram using dark theme

Some of the diagram is really hard to read in dark mode. Grey title on dark background is not ideal.

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

ID: d432d5bb-24a7-3bec-1e17-910c8129226e
Version Independent ID: fa6385e8-91ad-7949-382f-ca8e03f8201c
Content: Domain events. design and implementation
Content Source: docs/standard/microservices-architecture/microservice-ddd-cqrs-patterns/domain-events-design-implementation.md
Product: dotnet
GitHub Login: @CESARDELATORRE
Microsoft Alias: wiwagn

Thanks for your feedback @daesolus. This is a duplicate of #6547, so I'll be closing this one for that reason.

Duplicate of #6547


fix: info menu overflows on mobile, issue #221

a fix for #221
Seems to fix the "popping in" issue described:

This doesn't seem to work, the issue occurs if you open the menu while the segment data is still being loaded, then the menu get wider after that and it then overflows
Originally posted by @jwooning in https://github.com/commaai/connect/issues/223#issuecomment-1173558034

width < 400 truncates currentSegment and reduces font/button sizes
Sorry if this doesn't fix the issue, I tried my best to match the width originally described in the issue (~380px) and to simulate the loading.. it also seems to work down to 300px width.
https://user-images.githubusercontent.com/21132103/177290925-ff71c087-efdc-4e06-94ca-bd605dcae8ea.mov

I wrote a more complete description of the actual issue:
https://github.com/commaai/connect/issues/221#issuecomment-1174864971


Stopped Receiving:    Received event with handler LAN client handler

Using with Envisalink 3 with 193 firmware. Seemed to be working and getting LAN client responses.
After upgrade of EnvisaLink firmware to 01.12.202 now seem to never received "  Received event with handler LAN client handler"
and status of the sensors etc are not updated.
After deleting the DSC Primary Device and reconnecting, it seemed to work again.

Hello - Thanks for reporting the issue.  It seems the connection with the Envisalink could not be established after your firmware update.  It's difficult to tell what happened without some log output.  But I'm glad it's working for you again.  In the future if a similar problem arises, rather than deleting the device or reinstalling the driver, what I would suggest is to temporarily change the Envisalink LAN address in device Settings to something else, wait 10 seconds or so, then change it back to the correct address.  This will force a reset of the connection to Envisalink.


chore: bump mui to the create-app resolved version, to lose Accordion warnings

Yeah I would prefer merging this and then having them rebase on top

Fixes #2238


[TEST][TPCDS] Introduce schema check for q7

Why are the changes needed?

Introduce schema check for q7
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #578 (b4010d0) into master (f311211) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #578   +/-   ##
=======================================
  Coverage   79.86%   79.86%
=======================================
  Files         118      118
  Lines        4545     4545
  Branches      549      549
=======================================
  Hits         3630     3630
  Misses        608      608
  Partials      307      307

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 f311211...b4010d0. Read the comment docs.

merged to master, thanks


modifies layout of item pages

addresses #65 .  Restores the layout and look from the earlier site.

I think this is a definite improvement. But I think it would be better to stick with displaying the Catalogo entry text at the top of the item pages. And once we're otherwise happy, we should fully update the static site (generate the HTML pages, and run Jekyll to update the _site directory).


Custom Class definitions shouldn't be required

I get "System.Collections.Generic.KeyNotFoundException: 'The given key '(class name)' was not present in the dictionary.'" if I set the class property on anything in a map. I'm not sure what dictionary this is referring to, or how to add to it, and it's unclear how to proceed.
While I'm interested to learn the intended way to handle this, in my opinion, the "Class" property should be able to be used simply as a string that you can do whatever with, rather than something which requires a very strict implementation. You can type anything into a Class field after all, and using the Custom Types Editor isn't a requirement. It's just XML at the end of the day, and implementation should be flexible.

If you want some more clarity on why these are required, or just want to learn the recommended way to define and handle these custom types, I recommend you check out this documentation on custom types. TL;DR: Whenever DotTiled encounters non-empty class/type strings, it will look up that corresponding defined type and do some stuff to fill in values in properties.
If you don't want to have to worry about this, or just want to write strings that don't correspond to defined custom types, you can do the following:
var classDefinition = new CustomClassDefinition
{
  Name = "YourClassName",
  UseAs = CustomClassUseAs.All
};
var loader = Loader.DefaultWith(customTypeDefinitions: [classDefinition]);

This will make it so the loader will find your class type and therefore not throw an exception - but you will not gain any of the power of custom types. If you have many of these classes, you can do something like:
string[] classes = [
  "MyClass1",
  "Monster",
  "Chest"
];
var classDefinitions = classes.Select(c => new CustomClassDefinition { Name = c });
var loader = Loader.DefaultWith(customTypeDefinitions: classDefinitions);

That should solve your issue. However, I do understand your argument that it should be possible to just have any arbitrary string. In the future, I may decide to add some kind of configuration option for allowing unresolved classes/types, but until then, this is the solution.
Please let me know if you still have issues getting it to work.

Thanks for the reply! This workaround is fine with me. I'm hoping to translate a large project from another tiled library, and this will aid that process because I'm using the class property in my existing maps in a pretty loose, contextual way. I'm definitely going to try giving it a go the intended way though. I actually was only defining custom enums and bools etc in the types editor, and was using Templates in the way I could have been using custom classes, so it's probably good that I ran into this roadblock.

Great!
I'll keep this issue open until we have a decision on how/when a potential configuration option for unresolved classes and types is implemented.

One other thought: I think part of what bugged me is that this is the first instance of the LoadMap function not working given any (valid) output from Tiled. So maybe if there are other cases where some (valid) output from a .tmx file will throw an exception on default LoadMap, that's something that should be looked at and considered.

I just encountered the same issue as well. Managed to load my TMX just fine in the past with TiledCS because I also followed OP's logic concerning the class name. It is just a random string that can be interpreted in any way by the user's game logic.

One other thought: I think part of what bugged me is that this is the first instance I've encountered of the LoadMap function not working given any (valid) output from Tiled. So maybe if there are other cases where some (valid) output from a .tmx file will throw an exception on default LoadMap, that's something that should be looked at and considered.

I agree with this. Imho you should not treat the class property like it is supposed to be matched to an actual class in C#. Admitted, the name class is a bit.. confusing. You may not know this, but the class property didn't exist in older Tiled versions. In fact, if I recall correctly it was called type instead of class. I remember I had to make a modification in TiledCS for that.
It was introduced to provide a little more specification to an object to make it easier to mass import/read/select objects from a tiled map. Nothing more, nothing less.

It seems like there's no way to have a blank CustomEnumDefinition with just the Name field in the same way you've outlined for a CustomClassDefinition. Also, the FromEnum method defaults to expecting Integers instead of strings, which is different than the default for Tiled. I added a small fix for this in my PR, adding CustomEnumStorageType as a parameter to FromEnum method.

After quite a bit of deliberation, I understand that the fact that custom types are currently required to be defined can potentially cause a lot of confusion for many users of DotTiled. I therefore suggest that we make the custom types not required, and instead only attempt to find it during parsing to get correct default values etc.
However, this means that if a user makes use of custom types with default values, without setting the Resolve object types and properties option for exported maps, users will likely get a different map loaded with DotTiled than what they expect. All we can really do in this case is to provide a warning in the documentation about the behaviour - which isn't too bad.

PR #57 contains an initial implementation for the above suggestion.

Testing this out, and all seems well except for a detail with enums. The custom enums seem to only work with "Number" selected in the custom types editor (and an object created that uses the type), but I understand that's a separate issue.

Testing this out, and all seems well except for a detail with enums. The custom enums seem to only work with "Number" selected in the custom types editor (and an object created that uses the type), but I understand that's a separate issue.

Interesting, would you be able to open up a separate issue for the problem you are facing? Please include some reproducible example that I can work with to do some debugging :)


jsonrpc node over the evm instance used

given an implementer of the EVM trait mentioned in #23 and a database type (probably in-memory map), we should be able to run an RPC node that "acts" like a blockchain.
the logic under service could act as inspiration https://github.com/brockelmore/rust-cevm/blob/35cdefb760d41197ccfadc8c446343f20eba9080/service/src/EVM/mod.rs, but if possible we should try doing this with the tokio/hyper stack for performance reasons and the ability to inject extra middleware etc., e.g. https://github.com/tokio-rs/axum#usage-example with a jsonrpc module like how https://github.com/AtsukiTak/warp-json-rpc would work with https://github.com/seanmonstar/warp

Non-standard RPC method wishlist:

evm_snapshot – useful for externally-managed testing environments (JS SDK tests are maybe the most common example)
trace_replayTransaction – fork mainnet and get state diffs for actions you otherwise wouldn't be able to perform (eg run a governance proposal for a protocol that's behind a governance delay and observe the variable changes)


We're in 2016 (Happy new year !).

Bump year in NOTICE.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/378/
Hooray Jenkins reported success with all tests good!

Thanks! Belated Happy New Year :D
Good reminder, I added the tweak for the applet's NOTICE as well, and to the IntelliJ template. We'll get the actual headers in source files whenever somebody changes the files for some other reason and remembers.


Triggering / sending react-email templates from server side

Hi there, referencing the issue in react-email: https://github.com/resend/react-email/discussions/570#discussioncomment-10580756
My use case is a Next.js application from which I need to send emails triggered by an external cronjob scheduler (using https://github.com/kelektiv/node-cron) and nodemailer. I would love to be able to build the emails using react-email for its ergonomics. But my understanding is that those emails JSX would need to be pre-compiled into static HTML if needed to be sent out by nodemailer, from outside Next.js.
This is the way I would like to send emails. The code resides in the Next.js codebase but is called from node-cron and therefore bypasses JSX compilation:
// src/lib/email/index.ts
import type { Locale } from "-/src/i18n";
import type { Post, Profile } from "@prisma/client";
import { convert as convertToText } from "html-to-text";
import { sendNodemailerEmail } from "./nodemailer";
import emailTemplate, { type EmailTemplateType } from "./templates";

async function sendEmailTemplate({
  type,
  post,
  recipientProfile,
}: {
  type: EmailTemplateType;
  post?: Partial<Post>;
  recipientProfile: Profile;
}) {
  const { template, subject } = await emailTemplate({
    type,
    locale: recipientProfile.locale.toLowerCase() as Locale,
  });
  const html = template({ post, recipientProfile });

  return await sendNodemailerEmail({
    to: recipientProfile.email,
    subject,
    text: convertToText(html),
    html,
  });
}

export { sendEmailTemplate };

Is there a better / more correct approach to this? Thanks a lot.

Hi @anfragment thanks so much for your help.
I think react-email's render only works when called from within a JSX environment - but I need to call it from simple .ts file since it's gonna be triggered from node-cron...

Check out https://www.typescriptlang.org/tsconfig/#jsx. Your project environment is probably already configured to allow importing JSX from anywhere, otherwise it is still pretty easy to make it work.


Problems when sentence is mix of Chinese and English

When using CosyVoice in a mix of Chinese and English, if the English text is in uppercase, it reads a single word as multiple words, resulting in a strong no English accent. This issue needs to be addressed.

for example， the following sentence in this code，result in very differenct voice!!
text = "大家好，这里给大家介绍一篇名为AN IMAGE IS WORTH 16X16 WORDS: TRANSFORMERS FOR IMAGE RECOGNITION AT SCALE的论文，"
output = cosyvoice.inference_sft(
text.replace("-", ""),
"中文女",
)
for i, j in enumerate(output):
torchaudio.save("sft_uppercase.wav".format(i), j["tts_speech"], 22050)
output = cosyvoice.inference_sft(
text.replace("-", "").lower(),
"中文女",
)
for i, j in enumerate(output):
torchaudio.save("sft_lowercase.wav".format(i), j["tts_speech"], 22050)

手动或者工具转写为小写，我们的训练数据也会归一化，并不是一点也不处理


bugfix:Generate requirement test when param --requirement_test

Check param --requirement_test before generating requirement tests.
Which will resolve - https://app.circleci.com/pipelines/github/splunk/splunk-add-on-for-microsoft-hyper-v/1235/workflows/0dba5943-2c23-4c70-9b13-64f76717a55d/jobs/4597/tests

I have read the CLA Document and I hereby sign the CLA

@nandinivij Since versioning of PSA is now automatically determined, can we please update the commit message to follow the conventional commit standards https://www.conventionalcommits.org/en/v1.0.0/

:tada: This PR is included in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Breakpoints do not stop on record assignment

Set a breakpoint in IntelliJ (well, you can't yet, but will be able to after the next plugin release) on an assignment statement with a RHS that is a record. Run to the breakpoint. Observe that execution stops at the next non-record-assignment.
The code in the screen shot was copied from the patterns spec, if you need it. Notice that all three assignments have been performed. I did not single-step at all. The breakpoint is on line 2; execution stopped on line 5.

TBH I'm not sure this is a real problem. But it looks odd.
@johnniwinther

cc @alexmarkov

Fyi @bkonyi
Steve have you tested that code with DDC? It would be interesting to see if that is a VM only issue or not.


acceptTap doesnt seem to work anymore

I installed 2.0 and acceptTap doesnt seem to close the drawer anymore. As I understand it, acceptTap allows you to close the drawer by tapping on the area of the screen that is not overlaid by the drawer.

Tried lots of things, but it really doesn't work. I see this happen in the phone log when I tap the area next to the open menu:
May  3 12:37:53 Marcs-MacBook-Pro assertiond[88244]: assertion failed: 15E56a 13C75: assertiond + 12188 [8CF1968D-3466-38B7-B225-3F6F5B64C552]: 0x1

Ok, got it working again. I had
panCloseMask={1}

Not sure if this worked before and stopped working after

Glad you got it working. Right now panCloseMask handles both integers (pixels) and decimals (treated as %). Do you think it would be more intuitive if we only accept one number format and allow the rest to be handled with callback style?

Not sure. But the way it is, is indeed counter intuitive


test: ensure .test.ts ending for test code files #1137

fixes #1137
Signed-off-by: Tommesha Wiggins<EMAIL_ADDRESS>

Codecov Report

Merging #1147 (9cd769d) into main (506002c) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1147      +/-   ##
==========================================
- Coverage   73.14%   73.12%   -0.03%
==========================================
  Files         246      246
  Lines        8617     8617
  Branches      996      996
==========================================
- Hits         6303     6301       -2
- Misses       1783     1785       +2
  Partials      531      531

Impacted Files
Coverage Δ

...t-tooling/src/main/typescript/common/containers.ts
74.48% <0.00%> (-0.83%)
: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 506002c...9cd769d. Read the comment docs.


DRY Placement Route Helpers

⚠ Blocked on merging #330
Context
There was a lot of repetition when looking at the case statements for the placement route helpers, using polymorphic_path offered a way to achieve the same results with less code.
Changes proposed in this pull request
Replace all case statements (where possible) with polymorphic_path
Adds a new service that returns and instance of a passed object as its base class. This was required due to Rails being too clever and not matching our routes when given a class that uses STI.
Guidance to review
There should be no noticeable changes to how the application functions.
Link to Trello card
N/A I don't have access
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated or added to the Azure KeyVault

I'm closing this as we've agreed it's probably an unnecessary change at this point.
It would be helpful if we were expecting many more organisation route helpers, because it cuts down on boilerplate by using polymorphic_path. However in actual fact, we don't expect this file to get much bigger than it currently is. All of the 'Organisation' (i.e. School and Provider) shared functionality is already implemented – e.g. the users list, organisation details, etc.
So we're going to leave this as is for now. It could be something to consider in future if we have more things to add here.


metals version update

This is just the latest Version update for metals.

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

[ x ] 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"
[ x ] 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
[ ] Fits CONTRIBUTING.md.

This version is already in master and will be soon on nixpkgs-unstable. Please follow the contributing guide when creating your changes.

This version is already in master and will be soon on nixpkgs-unstable. Please follow the contributing guide when creating your changes.


list Minikube under "local-machine solutions"

List Minikube under "local-machine solutions" as the recommended solution.

Minikube should definitely be in there.
@dlorenc @aronchick @aaron-prindle anywhere else minikube should be advocated in the docs? is there an @minikube to ping on gh?

also, dup of #851 ?

Thanks, LGTM!


Enable TLS for communication between control plane components and the local apiserver

This is a first step towards disabling the apiserver insecure port binding entirely; in this PR, we generate TLS client certificates for kube-controller-manager and kube-scheduler, and change those manifests to communicate with the apiserver via the encrypted (authenticated) port.
The kube-proxy pods on controller nodes were also using the apiserver insecure port, and this PR changes that as well.
Ref: #1291

Codecov Report

Merging #1292 into master will increase coverage by 0.45%.
The diff coverage is 91.57%.

@@            Coverage Diff             @@
##           master    #1292      +/-   ##
==========================================
+ Coverage   36.63%   37.08%   +0.45%
==========================================
  Files          63       63
  Lines        3882     3923      +41
==========================================
+ Hits         1422     1455      +33
- Misses       2242     2248       +6
- Partials      218      220       +2

Impacted Files
Coverage Δ

core/controlplane/config/encrypted_assets.go
71.15% <91.57%> (+0.78%)
:arrow_up:

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 a55c665...762e658. Read the comment docs.

Conformance test results:
https://scanner.heptio.com/fdfe8f4cbb8e41efcaf0fe7e836fc969/diagnostics/

@danielfm Thank you so much for your continuous efforts on security 👍
LGTM.


WIP: Improve Support for Liftoff's Preloaders

Revises the Knex CLI's initialization logic so that it takes better advantage of Liftoff's module pre-loading.  More specifically:

Removed the unsupported "knexfile" and "knexpath" arguments from the call to Liftoff#launch(..).
Removed Knex's custom logic for config file discovery.  (In certain cases, this logic would fail to discover configuration files unless they were either Javascript or Typescript.  Furthermore, this logic would never invoke Liftoff's module pre-loaders, which could then lead to syntax errors when opening the files)

Importantly: this PR changes the CLI's behavior when both the --cwd and --knexfile parameters are specified.  Previously, the location of the --knexfile was always relative to the value specified via --cwd.  Once this PR is merged, the two parameters will be independent of one another.  This better aligns w/ the semantics used by Liftoff.  For example:
knex --knexfile /path/to/shared/knexfile.js --cwd /var/www/project1 migrate:latest
knex --knexfile /path/to/shared/knexfile.js --cwd /var/www/project2 migrate:latest

FYI: This code currently breaks 1 unit test.  Specifically:
  6) CLI tests
       resolves knexfile correctly with cwd specified:
     node /home/brian/src/arclia/knex/bin/cli.js migrate:latest --cwd=test/jake-util/knexfile --knexfile=knexfile.js -> FAIL.
Stdout:
Error: /home/brian/src/arclia/knex/bin/utils/cli-config-utils.js:11
    throw new Error(
    ^

This error occurs because Liftoff treats --knexfile and --cwd as orthogonal parameters.  More specifically, --knexfile is not evaluated relative to --cwd.

@briandamaged I am somewhat concerned about breaking scripts for existing users, but I guess just one case with cwd is not too bad, as long as we publish it as a major version. Can you add one more cli-testlab-based test that was failing prior to this change, so that we don't break your fix again in the future? I guess adding ts file would work.

@kibertoad : Sure!  I just wanted to get your feedback on the breaking change before moving forward w/ the other test cases.  I should have some time to write them in the next day or so.

@kibertoad : I've added some new tests, but I'm noticing a discrepancy.
When both the --cwd and --knexfile flags are specified, it is changing the process's CWD to path.dirname(knexfile) (instead of honoring the --cwd value that was passed in explicitly).  It looks like this is potentially a bug in the Liftoff library itself:
https://github.com/js-cli/js-liftoff/issues/55
Anyway, I'll do a bit more digging into this tomorrow.

@kibertoad : A closely-related question: what is the intention of the cwd value?  Is it supposed to represent:

The directory that contains the Knexfile?
The directory that contains the developer's application?  (ie: project root)
An arbitrary directory based upon the specific project's needs?
Something else?

@briandamaged Prior discussions on this subject: https://github.com/knex/knex/pull/2959 and https://github.com/knex/knex/issues/2952
I would say that CWD is arbitrary point that is the root from which other paths are calculated. Important behaviour to preserve as this is how knex used to work historically - paths to seeds and migrations are resolved relatively to knexfile. Path to knexfile, however, should be resolved relatively to CWD, whatever user decides to pass as such.

@kibertoad : Howdy again!  Just following up to see if there's anything you need on my side.  Thx!

@briandamaged Sorry, just started at new job and got distracted. This looks great and I would be happy to merge this. Any reasons why you kept WIP in title?

@kibertoad : Ah -- nope.  I just forgot to remove the WIP: prefix.  Lemme take care of that now...
Good luck on your new job, btw!  Doing something interesting, I hope?

@briandamaged Thanks! Oh yeah, brand new team, greenfield project, I'm extremely hyped about it. The only unfortunate thing is that I switched stacks and returned to Java world, so there'll be less eating of own's dog food, unfortunately.

@kibertoad: Ah -- oh well.  I've heard that the Java language / ecosystem has improved significantly over the last several years.  So, it might not be that painful.  Plus, there are several languages that compile into Java bytecode, such as Kotlin and Scala.  So, you might be able to integrate those into whatever you're building.

Well yeah, convincing folks over there to use Kotlin is one of my current dreams :D


Fix UG accordings to the suggestions

Closes #296
Closes #250
Closes #252
Closes #298
Closes #270
Closes #272
Closes #302
Closes #303
Closes #259

This PR also closes #249


Listening for change events

Hi,
I want to do something (e.g. persist the current grid layout) after a tile has been moved/resized. Is there a way to do this currently, or would the element need to be updated to fire an event on tile changes, which a containing element could then listen for?
Thanks,
Chris.

Thanks for feedback @lamplightdev. Definitely doable and useful. I will add it in the next release.

@lamplightdev @vpusher you can add an observer on the databinding on the property and do whatever you want to do there

@yorrd @vpusher thanks for the tip. I'm using polymer-redux however using only one-way data binding so an observer wouldn't work in this case without adding some extra properties to track the tile properties and observing them to then update my redux state which I'd rather avoid

@lamplightdev are the following events enough for you:

resize (once the tile gripper is released ~touchend/mouseup)
move (once the tile is released ~touchend/mouseup)
?

@vpusher yes, those would be ideal, thanks!

Delivered in v1.2.0. Enjoy.


refactor($parse): do not pass scope,locals to interceptor fns

After @lgalfaso's comment I thought it would be a good idea to remove these params from interceptors. All the internal uses of interceptors are for doing things like data conversion/normalizing, never reading state out of the scope/locals. I think this is the intended use, and makes them like filters (which receive the input value + args).
Note that scope/locals still get passed through to the expression being wrapped/intercepted, it's just the interceptor-fn itself that now only receives the value.

@jbedard can you please add the info from the PR to the commit message? This will be helpful for the next person who works on this code. While it's a refactor because it doesn't touch a public API, by including tests we change the expectations for how the internals should work.

Updated the commit message.

I'd also vote to only put this into 1.7 since it does change an internal but publicly accessible API

1.7 is fine with me

Commit msg look ok before merging?

LGTM


Sort Tables for docs

Tries a hand at #5965
Currently It looks dirty. And the sorting doesn't work for params column (JS and string sorting is a programming meme in itself).
Vanilla Dummy tables
https://output.circle-artifacts.com/output/job/ef871bb1-aeaf-4ecb-8157-b4f56f78c939/artifacts/0/docs/dummy_table.html
New dummy tables with tweaks
https://output.circle-artifacts.com/output/job/c6fd27dc-11ed-41fa-bb34-74368be7869e/artifacts/0/docs/dummy_table.html
Tweaked output on our docs
https://output.circle-artifacts.com/output/job/c6f9484c-b105-48ae-8a93-a6e2d3fcb6b0/artifacts/0/docs/models_new.html
Use class rst
https://output.circle-artifacts.com/output/job/7703674c-7553-4a09-8dad-6a6126e96999/artifacts/0/docs/models_new.html

I believe this feature is very cool. As it's painful to find the best model from the list (and our library will keep adding models) + potentially other statistics such as say run-time time etc. Let's try to hack some way :)

Update:
So a few tweaks to the CSS in Jquery made it work :smiley:
Although the sad thing is something is still broken with classification table. Also some tweak can probably make it work on parms / disable it on params.
Ref
https://datatables.net/examples/basic_init/filter_only.html
https://datatables.net/reference/option/
https://stackoverflow.com/questions/13290626/datatable-hide-the-show-entries-dropdown-but-keep-the-search-box

@oke-aditya Just confirming the intention of this PR. I understand that currently you try to show-case the sorting functionality. You just copy-paste the accuracy tables for convenience and plan later to adopt @NicolasHug's solution that builds these tables dynamically form the weight meta-data, right?

Yes, I will build upon @NicolasHug solution. I inentionally took the tables out for dry run and to check how the functionality works.

Note that sorting still doesn't work well in Params column.

Note that sorting still doesn't work well in Params column.

Do you know if we can provide a custom sort function for that specific column?

It should be possible. But I will need to search and try.

Sorry I still can't find the fix for it :( Maybe I'm doing something wrong. But my skills with JQuery and JS aren't that great either.

No problem at all @oke-aditya , thanks a lot for trying!!
I'll try to see if someone can help internally with this


testing action v1.38

#3  👈
main

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

Join @alangenfeld and the rest of your teammates on  Graphite


Fix #457, Add build and run app reusable workflow

Checklist (Please check before submitting)

[x] I reviewed the Contributing Guide.
[x] I signed and emailed the appropriate Contributor License agreement to<EMAIL_ADDRESS>and copied<EMAIL_ADDRESS>
Describe the contribution

Fix #457

Testing performed
Passed CI with modified workflow (pointed to skliper merge branch) here: https://github.com/skliper/CF/runs/6065276796?check_suite_focus=true
Expected behavior changes
None, CI only
System(s) tested on
CI
Additional context
None
Code contributions
The cFS repository is provided to bundle the cFS Framework.  It is utilized for bundling submodules, continuous integration testing, and version management and does not contain any software.  Code contributions should be directed to the appropriate submodule.
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

CCB:2022-04-20 APPROVED with Changes

change don't skip from "push" to "pull request"

CCB:2022-04-20 APPROVED with Changes

change don't skip from "push" to "pull request"

Fixed in 0adfd3c


Request: Most recent pre-computed SpliceAI scores

Hello,
I am working on developing a Slivar pipeline to run parallel to our lab's Seqr pipeline, and one remaining area of discrepancy is our SpliceAI annotations. Would it be possible to obtain the most recent (gencode v44 basic) masked spliceAI pre-computed annotations that SpliceAI-lookup uses?
Thank you for your time and effort,
William Gordon (Chong Lab @ UW-CRDR)

Hi William,
SpliceAI-lookup doesn't use precomputed scores. The model is called directly for each new variant.
There are newly precomputed SpliceAI and Pangolin delta scores for gnomAD variants that are available publicly within the  gnomAD v4 hail table:
ipython3
In [1]: import hail as hl
In [2]: ht = hl.read_table("gs://gcp-public-data--gnomad/release/4.0/ht/exomes/gnomad.exomes.v4.0.sites.ht")
In [3]: ht = ht.filter(hl.is_defined(ht.in_silico_predictors.spliceai_ds_max), keep=True)
In [4]: ht.in_silico_predictors.spliceai_ds_max.take(5)
In [5]: ht.in_silico_predictors.pangolin_largest_ds.take(5)

They are based on gencode v39, max distance = 500 and mask=True.
SpliceAI-lookup doesn't use them though.
-Ben

Hi Ben,
Thanks for clearing that up for me and pointing me towards the gnomAD v4 hail table!
Cheers,
William


I need to know what is the best data base connector for Django with MongoDB  project

I need to know what is the best data base connector for Django with MongoDB  project

I found a few solutions:-
MongoEngine, Djongo


Option to import github repository similar to the cloud environment

Option to import github repository similar to the cloud environment:
https://bolt.new/~/github.com/vinodsharma10x/saas-founder-hub
http://localhost:5173/~/github.com/vinodsharma10x/saas-founder-hub
When I try I get a 404 not found error.
I really need a way to import my projects, as it became unfeasible to continue with the bolt.new plan.

This is definitely on the radar and prioritized, with a few pull request submissions related to it. Thank you for your patience while it's worked out.

we need thiss 🥲

Where do you need help? Are you trying to keep the same model of appending the github url to the url path?

yes, I keep the same pattern but I get error 404


IDbQueryCache for ef core 3.1

I'm trying to upgrade to ef core 3.1 in one of my projects but as a result I am getting an error as IDbQueryCache is missing.
Has this been moved into an extension package or is there an alternative that can be used?

@AdamWorley DbQuery has been removed in EF Core 3. Query types are now just entity types that have no key defined. So the "replacement" for IDbQueryCache is IDbSetCache. See https://docs.microsoft.com/en-us/ef/core/what-is-new/ef-core-3.0/breaking-changes#qt

@ajcvickers Great, thanks for the update


Retreiving consumer status

Hello everyone!
I've faced a problem with confluent .NET consumer, which I believe is quite common. Yet still I haven't found any solutions on the internet... So I'm here :)
Is there any way I can check my consumer status in real-time? I'm trying to implement kubertenes health-check for Kafka consumer readiness in my service. Ideally, I'd like to know if controller broker considers my consumer alive (does it receive heartbeats from my consumer or not).
There can be other ways, of course. You can implement a sort of check with SetPartitionsAssignedHandler, or simply check if your consumer received any messages during last N minutes. But it won't work with consumer group which has more consumers than partitions, which in my case is crucial.
So, maybe there are some best practices that I'm missing? Or that exact feature is already implemented in your API?
Thanks!

Seems the thing am also looking for. Did you get a way to figure out the approach to get the consumer status @anton-matsyshyn ?


Add paths in push trigger for accelerator workflows

Overview/Summary
Adding back paths in push trigger for accelerator workflows so not all workflows will run deployment regardless of where the change was made.
This PR fixes/adds/changes/removes

This fixed bug reported in #598

Breaking Changes
Testing Evidence
As part of this Pull Request I have

[ x ] Read the Contribution Guide and ensured this PR is compliant with the guide
[ x ] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[ x ] Checked for duplicate Pull Requests
[ x ] Associated it with relevant GitHub Issues
[ ] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[ x ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ x ] Performed testing and provided evidence.
[ ] Updated one or more of the following tests (if required)

Unit
Linting
E2E (End-To-End)
ValidateAzCloud (Base validation in Azure Cloud)
ValidateMcCloud (Base validation in Azure China Cloud)

[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)
[ ] If relevant, created or updated Code Tours here

Thanks @stalejohnsen, @oZakari can you take a look here and do the honours if all good?


HDDS-10192. Verify container checksum after downloaded

What changes were proposed in this pull request?
HDDS-10192 Verify container checksum after downloaded
Verify container checksum after downloaded to make sure the checksum matches the value stored in yaml;otherwise, throw error immediately.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10192
How was this patch tested?
unit test.

cc. @sumitagrawl @errose28 @jojochuang @umamaheswararao @nandakumar131 @sadanand48

Thanks @DaveTeng0 @adoroszlai


License

Hello
I appreciate your related blogpost and would like to reuse and possibly build upon this code. The usage would be commercial.
Could an MIT license (or equivalent) be attached to the project?

Sure! License was added :-)
Thanks for the hint!


Avalonia.Android  update 11.1.0  error

Is your feature request related to a problem? Please describe.
Avalonia.Android update 11.1.0
protected override void OnCreate(global::Android.OS.Bundle savedInstanceState)
savedInstanceState is null
System.InvalidOperationException: 'Could not create glyphTypeface. Font family: $Default (key: ). Style: Normal. Weight: Normal. Stretch: Normal'
Describe the solution you'd like

Describe alternatives you've considered
No response
Additional context
No response

Only default fonts can be used。

I believe the other package is not 11.1  compatible yet. Ask the author please

Normally, packages don't really need to be 11.1 compatible, as 11.1 is binary compatible with 11.0.


Network Test: Configuration of NTP / DNS Server

[ ] https://github.com/openbmc/openbmc/issues/2106  DNS Server

[ ] https://github.com/openbmc/openbmc/issues/2107 NTP Server

Today confirmed with the development team that it is for ESP seems. So we need to test and automate the same.

Closing as requested by @prkatti1


Guidance about how to proceed with Google fonts marked as Early Access

As #906 reflects, there are some fonts that appear in the Google fonts GitHub repo but are marked as Early Access fonts, according to Google the definition for this type of fonts is:

We look forward to adding support for many new languages to Google Fonts. To accelerate this process, we would like to get your feedback on our latest fonts.

These are the fonts marked as Early Access:

Lao Muang Don
Lao Muang Khong
Lao Sans Pro
Tharlon
Nanum Brush Script
Nanum Pen Script
Souliyo

So it seems these fonts can has bugs and undesired behaviours, so IMHO it would be better to make the user know it with something like a warning or avoid to add them to this repo.
I wanted to get some advice from the project collaborators in order to know how to proceeed
Thanks!

so IMHO it would be better to make the user know it with something like a warning or avoid to add them to this repo.

Nope. We purposefully do not do that. See our FAQ on the matter:

we’re not a discoverability service and our users are expected to have reasonable knowledge about the apps they’re installing through us before doing so.

You can go right ahead and add Early Access fonts just like any other. We already have some.


Implement retry-as-promised or equivalent

Feature Request
Hi! Is there a way to implement retry-as-promised or an equivalent into twurple-api? To give a usage example, I often have FetchError with the message connect ETIMEDOUT <IP_ADDRESS>:443 (after seeing with a Twitch moderator, it would be linked to a problem with Twitch load balancing), and retry the request one or two times could solve this kind of issue.

Would definitely be nice to have something that is able to handle common temporary errors. I've not quite run into this issue with Twurple yet but in my experience with the Discord API it's kind of tricky to know how to handle all temporary errors that can occur because you usually don't know to account for them until they happen.
You basically end up writing what amounts to boilerplate code that wraps around the API calls and checks for timeouts and things like that. I feel like in 99% of projects you just want those cases to be handled automatically, possibly with a retry: false type option.


HTTP 500 on nameserver lookup

HTTP 500 when running "[SW] Nameserver to Domains" on ns2.ropot.net.

Similar issues for analytics and adsense lookups. However, if regex doesn't match, then the correct error is returned.

2017-03-16T01:43:47.220294+00:00 app[web.1]:   File "server.py", line 68, in adsense
2017-03-16T01:43:47.220296+00:00 app[web.1]:     ent.setLinkLabel(data[name])  # date ID was associated with domain
2017-03-16T01:43:47.220297+00:00 app[web.1]: TypeError: string indices must be integers, not str


Add Embark and Bytecode Alliance to the registry

We haven't implemented the code to fetch the registry from the tool, but it's still a useful place for discoverability.

r? @mystor

lgtm


fix: properly convert plain text in brackets that is not a link

Proposed changes (including videos or screenshots)
Properly convert the following:

[Title 1] foo [TItle 2](http://bar.com) to html
[Title 1] foo Title 2
[Foo [Bar _emph_ Baz] Waz](http://bar.com) to html Foo [Bar emph Baz] Waz

Issue(s)
Closes #1131

Further comments

Tests included

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

CLA signing is not properly detected even after days have gone by. I have signed and rechecked multiple times.


Support for BPM volumes

#166404375
This includes:

bpm ephemeral disk support (uses a emptyDir volume)
bpm persistent disk support (ignored for the moment)
bpm additional volumes support (uses a emptyDir volume)
bpm unrestricted volumes support (uses a emptyDir volume)

For both additional_volumes and  unrestricted_volumes , paths under /var/vcap/store/ are not
supported, until we find a better way to handle persistentVolumesClaim.
This PR generates the volumeMounts for the above types, for the
containers(corev1.PodSpec)  and initContainers(only bpm hooks), and also a matching Volume for the pod definition.

looks good - please also update docs

@viovanov done, docs updated


The caddy2 version bahaves unexpectly.

I'm trying to simply set up an https proxy which has a basic auth check and proxy everything to a http proxy listening on localhost.
In the Caddyfile, I write something like:
https://my.domain:31082 {
    route {
        forward_proxy {
            basic_auth user passwd
            upstream http://localhost:31081
        }
    }
}

However when I do curl -x https://user:passwd@my.domain:31082 https://whatever.site, it results in an error:1408F10B:SSL routines:ssl3_get_record:wrong version number. When I do curl -x https://user:passwd@my.domain:31082 http://whatever.site, it just return an empty 200 response even if the "whatever.site" does not exist.
I have no idea why it behaves like this. Anyone help?

+1

I dig into the caddy server code and find out that this related to how caddy matches a request to handler. Caddy server uses http Host header to match domains defined in config file. However, when it comes to forward proxy scenario, the request's Host is always the target host which is not your proxy's Host. The following is a typical proxy request
`
[root@VM_0_4_centos ~]# curl -x https://example.com httpbin.org/ip -v

GET http://httpbin.org/ip HTTP/1.1
Host: httpbin.org
User-Agent: curl/7.81.0
Accept: /
Proxy-Connection: Keep-Alive
You can see in this case, the requestHost` header is httpbin.org while not example.com. When caddy server receives this request, it can't find any route of target host httpbin.org and then it just responses 200 to client.

I think what I've done with my Caddyfiles is something like:
:443 example.com

forwardproxy ...

in other words, specify a domain name along with a catch-all, port-only address. That should route the requests properly. I haven't finished updating this plugin for v2 due to lack of sponsorship resources to prioritize it. But I think that's basically how you want to do it.

I think what I've done with my Caddyfiles is something like:
:443 example.com

forwardproxy ...

in other words, specify a domain name along with a catch-all, port-only address. That should route the requests properly. I haven't finished updating this plugin for v2 due to lack of sponsorship resources to prioritize it. But I think that's basically how you want to do it.

This Caddyfile works. Thanks a ton!

No problem. You really only need the :443 (don't restrict the Host) but adding example.com (or rather, your domain) is an easy way to tell Caddy to automate the certificate for that domain.


Certain worlds aren't changing gamemode

I am using 1.14.4, newest released pwi version from spigot, and the following are my plugins and the configs i have:

The problem i'm having is that only one of the four creative worlds ('testing') sets the player to creative mode. There is no error in the console when i restart. I couldn't find a solution for why these worlds arent working anywhere and this is the last thing i have left to try.

Most likely it's MultiVerse doing it. There is a permission to bypass its GameMode enforcements, but I can never remember what it is off the top of my head. I would try that.

I found a permission for multiverse called "mv.bypass.gamemode.WORLD_NAME"
So if i give players the permission "mv.bypass.gamemode.creative" or "mv.bypass.gamemode.creativeflat" then they should change into creative mode when they travel to the creative world? This permission seems to be OP so i should use this command only for the creative worlds? (I will have to test this later, when there are no players on my server)
On a side note, i did try removing my two multiverse plugins (core and portals) and the creative world was still not in creative mode. Is there possibly something else that is overriding pwi?

"mv.bypass.gamemode.WORLD_NAME"

Yeah, that's the one. If you want PWI to manage gamemodes instead of MultiVerse, you'll have to set that on all worlds. That's just how MultiVerse works.

I added the permission "mv.bypass.gamemode.*" to the player and reloaded but the pwi still isnt working. I mentioned before that one of the worlds in my default-gamemode: CREATIVE is working. Why would only one of the four work? I checked to make sure that the worlds exist and are in the creative group so I cant think of why they wouldnt be working?

If you can't figure out a solution for this, i'm fine with closing this issue


Add Dockerfile

A little PR to add a Dockerfile and instructions in README
Closes #92

Thank you.


Change base class

New version (2.9) of factory_boy uses different base class for custom fields.
We are using OrderedDeclaration https://github.com/mimesis-lab/mimesis-factory/blob/master/mimesis_factory/main.py#L5
While the new version is using BaseDeclaration.
There are two possible ways:

To drop 2.8 support (is was initially made this way to make pytest-factoryboy work
To support both versions and choose base class conditionally
Find something we can safely inherit from

@sobolevn What do you think about it? Should we support both versions?

I don't know, it depends on what versions are the most popular.

@sobolevn Seems like 2.8.1.


Multi masternode support

Add multimasternode support.
This allows to manage an indefinite amount of masternodes and be able to easily remove, add, see the status of and vote.
Voting is done with all owned masternodes at once.

Closing as stale


Runnable in fp16 mode on DCU

Runnable in fp16 mode on DCU

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: sljlp:x: Your NameYour Name seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Can we suppress the subfolders that pages are generated into?

Search terms
subfolders
Question
When Typedoc generates its article pages for each type, it sorts these files into subfolders according to what each of them describes - enums, classes, interfaces, and so on.
Can we suppress this, or tell Typedoc to use the same subfolder for all pages?
Background: Based on the module name and type name (which should be sufficiently unique; after all, we use nothing else when importing from another module in TypeScript), I am trying to automatically determine the URL to our Typedoc documentation. The fact that for this to work, I need to know whether a given identifier is e.g. a class, an interface, or anything else, is the only blocker left for me.

Not yet supported - tracked in https://github.com/TypeStrong/typedoc/issues/2111

@Gerrit0: I appreciate the link and I happily accept if it somehow covers my requirements.
However, I did find and read that other ticket before posting my question, and I just read it again, and both times, I saw little definitive overlap between my question and the other ticket. #2111 appears to be about a linking problem, restricted to functions, somehow specific to Cloudflare (?).
The ticket title ("Support for custom output directory structure") and a single sentence in the ticket ("I guess, what I'd like to see is a way to have customization over the file/folder structure.") sound reminiscent of what I'm asking about, but even the detailed description of the latter ("Maybe functions can be nested under their modules?") goes into a totally different direction.

Fair enough - my eventual goal there is to support fully customizing the output folder structure to allow both flattened and nested structures, or somewhere in between like TypeDoc today, I haven't figured out what that looks like yet

I would love this too!


Error when attempting to delete a user

OperationalError: (1054, "Unknown column 'POS_posapplication_residents.ewsuser_id' in 'where clause'")
I've checked and the column indeed doesn't exist. There's a similar column called 'user_id' in the same table. I have run migrations.

That one must have been missed int he migrations. Deleting users probably shouldn't be supported given the number of relations to the user table. I'd recommend setting them to inactive and renaming the username if you need to re-use it.


Add events

It would be nice if flextabs get some event supports (notably tab-selected) so e.g. classes can be assigned.

@strarsis Could you go into more detail on this? I see how events might be useful in some situations, but for classes/styling can't you use the built-in ones listed at https://github.com/mdmoreau/flextabs#state?


MTDSA-14841: Delete RAML API docs

delete raml and references

I know the method in the DocumentationController is slightly different to in the other APIs but for consistency probably still worth renaming the method from file to asset

Also, might as well implement the third platops comment :)


added configurable 'z' type

Made type of 'z' configurable through macro
Added missing 'z' value set calls in offseter
Use user defined "preserve colinear" parameter value in offset final merge step.

Actually I still have a Z interpolation differences when compared to some polygon offset operations that were relying on Clipper1Lib.
Angus, it would be a good idea if you had a look at that maybe...

Firstly, I like your suggestion to include a defaultZ, though I think it's better done as a property of the Clipper object rather than semi-hardcoded as predefined macro.
However, I'm not persuaded that there's sufficient need to offer a configurable Z type. And I'm also unlikely to add Z support to polygon offsetting as again I'm not sure there's sufficient need to justify it. Of course you're welcome to do this in your own fork of the library. Cheers.


Add verbose option for plugins

Description
Updates plugin message log: By default, without errors, one line per plugin will print reporting successful operation. It is possible to pass a boolean variable, verbose, at next.config.js set to true to print previous informational messages.
Also errors are now handled regardless of message logging. Errors wont block the process as they are handled by try-catch.
One extra function is added to print colors at console, making easier to understand info messages from errors.
Screenshots

Info
Errors

Fix #100

this looks great @GuilleAngulo - thinking a quick comment above the verbose option would help people use it then we can get this merged in


Modifications to work with SDL2 on Android

These are actually quite minimal, and are all that is actually needed to work with SDL2. Some of the details will need to be updated later (e.g. changing the NewPythonActivity references, and maybe changing how Hardware.java works more than I already did), but I believe these can and should be merged now, it will make it easier to test the sdl2 build tools.
Edit: Actually, not to be merged yet.

In one place I left the old android module call for use with pygame, but added the sdl2 one with pyjnius. This is because I had to modify Hardware.java with the line at https://github.com/inclement/p4a-experiment/blob/master/bootstrap_templates/sdl2/src/org/renpy/android/Hardware.java#L166 in order for it to work...I haven't tested if this is necessary with pygame (is it done somewhere else?).

Replaced by #3648


Support OffsetDateTime columns

Hi,
It would be great if we could use OffsetDateTime which is often the new standard for handling dates with offset information, as a Column. I know they support it in Exposed and would love to see it here too. Or can I somehow extend with a new custom column type?

It's easy to extend a custom SQL type for Ktorm, you can have a try by yourself, here is an example:
/**
 * Define a column typed of [LocalDateTimeSqlType].
 */
public fun BaseTable<*>.datetime(name: String): Column<LocalDateTime> {
    return registerColumn(name, LocalDateTimeSqlType)
}

/**
 * [SqlType] implementation represents `datetime` SQL type.
 */
public object LocalDateTimeSqlType : SqlType<LocalDateTime>(Types.TIMESTAMP, "datetime") {
    override fun doSetParameter(ps: PreparedStatement, index: Int, parameter: LocalDateTime) {
        ps.setTimestamp(index, Timestamp.valueOf(parameter))
    }

    override fun doGetResult(rs: ResultSet, index: Int): LocalDateTime? {
        return rs.getTimestamp(index)?.toLocalDateTime()
    }
}

Thanks!


Create add.js

Your Pull Request Title
Pull request description. Describe what you changed or added here.
Type

$add

Another details

does this have anything to do with any of the issues? (if yes, tag the issue)

thanks for contributing again!


Hello, does this app not support http://?

The requestId cookie is empty in the case of http://, but normal in the case of https://. How to solve?

We use Secure flag for cookies which works only for https. We suggest using HTTPS for safe execution.


CRAN packages not using the correct compiler in configure scripts

That is
C50 Cubist GPBayes RDieHarder RPostgreSQL RcppGSL RhpcBLASctl Rmixmod
Rrdrand SimInf SymTS eaf excursions fRLR ff float getip git2r gsl gslnls
hdf5r island memuse mmap ore poismf randtoolbox rayrender rgeolocate
rmatio rnetcarto rngWELL scrypt survSNP xgboost
These show
checking for gcc... gcc
even on platforms using clang as the compiler.  Please re-read
https://cran.r-project.org/doc/manuals/r-devel/R-exts.html#Configure-and-cleanup
There need be no 'gcc' compiler, it might be faked (macOS) or very old
(Solaris).  The aim of a configure script is to test things using the
same setup as will be used to compile code in the package.
In several cases you are using a C compiler in configure but a C++
compiler to compile the code: they usually have access to different
headers.  Looks like this is the case in at least
GPBayes RcppGSL Rmixmod fRLR ff rayrender xgboost
The logs showing
    checking for gcc option to accept ISO C89... none needed

need configure rebuilt with a modern autoconf, which will check C11 (and
that or C17 are the default in all the current compilers we know of --
however, that old autoconf test will fail when C23 comes into use).
Please correct before 2023-01-26 to safely retain your package on CRAN.
--
Brian D. Ripley<EMAIL_ADDRESS>Emeritus Professor of Applied Statistics, University of Oxford

Since we've had trouble getting R tests working on XGBoost 1.7, I am skeptical that we'd be able to make the fix in time. We should prepare for the possibility of having XGBoost pulled from CRAN.

I think I can give it a go, at least for 1.6

Maybe we should consider using an alternative distribution mechanism, like R-universe.

I think it's fine, working on other distributions is not less time consuming than fixing the issues on CRAN. I will take another look into the tests today and see how much work is needed.

I agree with @trivialfis , it is really worth trying to stay on CRAN. And especially for {xgboost}... if {xgboost} is removed from CRAN, all of these other packages will have to either quickly release a new version not depending on {xgboost} or also be removed:

(https://cran.r-project.org/web/packages/xgboost/index.html)
As annoying as it is to get these "fix this warning or we evict you in two weeks" emails from them. I know...I'm currently working on the second "fix something small to stay on CRAN" release LightGBM has had to do in the last month 😫 .
I can look into this today or tomorrow too if you'd like some additional help!

I think I already see at least one possible issue.
{xgboost} isn't getting information about the C and C++ compilers from R's configuration.
It's just relying on environment variables CC and CXX.
https://github.com/dmlc/xgboost/blob/72ec0c54841c03f9c7a81b72137234f2519273ab/R-package/configure.ac#L43
I think that'll need to change to something like this:
https://github.com/microsoft/LightGBM/blob/23403a7c2c4d38961506968291f13c3f344ccb6f/R-package/configure.ac#L16-L28
That's recommended in "Writing R Extensions": https://cran.r-project.org/doc/manuals/R-exts.html#Configure-and-cleanup
I won't be able to investigate and test that until about 10 hours from now, so just posting in case you're working on it right now and it's helpful.

I'm aiming to publish the 1.7 release:

https://github.com/dmlc/xgboost/pull/8667
https://github.com/dmlc/xgboost/pull/8668
Still working on the configure script. The references shared by @jameslamb are incredibly helpful!

@trivialfis Can we publish a patched version of 1.6 first, to be safe? As I recall there were unresolved test failures related to 1.7.

@hcho3 I will give 1.7 another try after all backports are made. If it fails then I will backport https://github.com/dmlc/xgboost/pull/8669 to 1.6 and submit it to CRAN. This way we don't upset the CRAN maintainers by making too many updates. What do you think?

If you think we can pass all tests with 1.7, go ahead


decodeVerKeyDSIGN: wrong length, expected 32 bytes but got 0

I am trying to make a simple transaction (lovelace only) using generated payment keys.
CHAIN_CONTEXT = BlockFrostChainContext(
  project_id=BLOCKFROST_PROJECT_ID,
  network=Network.TESTNET,
  base_url="https://cardano-preprod.blockfrost.io/api",
)

payment_key_pair = PaymentKeyPair.generate()
payment_signing_key = payment_key_pair.signing_key
payment_verification_key = payment_key_pair.verification_key

stake_key_pair = StakeKeyPair.generate()
stake_signing_key = stake_key_pair.signing_key
stake_verification_key = stake_key_pair.verification_key

address = Address(payment_part=payment_verification_key.hash(),
                         staking_part=stake_verification_key.hash(),
                         network=Network.TESTNET)

toAddress = "..."
...
builder = TransactionBuilder(CHAIN_CONTEXT)
builder.add_input_address(address)
builder.add_output(TransactionOutput.from_primitive([toAddress, 10000000]))
builder.build_and_sign([payment_signing_key], change_address=address)

The tx can be built, but submitting it to Blockfrost throws this error
blockfrost.utils.ApiError: {'error': 'Bad Request', 'message': '"transaction read error RawCborDecodeError [DecoderErrorDeserialiseFailure \\"Byron Tx\\" (DeserialiseFailure 1 \\"Size mismatch when decoding TxAux.\\\\nExpected 2, but found 4.\\"),DecoderErrorDeserialiseFailure \\"Shelley Tx\\" (DeserialiseFailure 1 \\"Size mismatch when decoding \\\\nRecord RecD.\\\\nExpected 4, but found 3.\\"),DecoderErrorDeserialiseFailure \\"Shelley Tx\\" (DeserialiseFailure 1 \\"Size mismatch when decoding \\\\nRecord RecD.\\\\nExpected 4, but found 3.\\"),DecoderErrorDeserialiseFailure \\"Shelley Tx\\" (DeserialiseFailure 1 \\"Size mismatch when decoding \\\\nRecord RecD.\\\\nExpected 4, but found 3.\\"),DecoderErrorDeserialiseFailure \\"Shelley Tx\\" (DeserialiseFailure 193 \\"decodeVerKeyDSIGN: wrong length, expected 32 bytes but got 0\\"),DecoderErrorDeserialiseFailure \\"Shelley Tx\\" (Deseria', 'status_code': 400}

I can see it hast to do with the verification key and I bet I am doing something wrong. Any idea what it is? 🙇

Hi Robin, the code looks correct to me. Could you provide the CBOR, which can help debugging? It will be great if you can also provide the code that submits the transaction cbor.

Hey Jerry,
the cbor looks like this
84a400818258207c806478ff2de0c970523870cd57f508c0634c0dd0005ef835c16ef7a9c8001800018282583900b5b7e62899719388be48d412e1384311a68e44a01633d5185b203a643a746b909de8d2e7747ccb76125b25d98d198b9950f39fd70eb335611b0000000253bf98c082583900b56bb1f70feb383207c926af89ede6934f2faca85037f3d982801e5a4b1ab3fccc016ccdebd20006fe167d4e40bd1a12f934fdc4dd7574571a0049b9c3021a0002917d031a00acb03ca100818240584056be2bf9c7c3a6f8d429bfe31febb794b6ec4ccd64139dc179f15688330105c4cb8bb886da54f3fff3b7e342d856b688396c597d226ed6ff5d2a553ecb029408f5f6

and the submission is simple as this
tx = builder.build_and_sign([payment_signing_key], change_address=address)
CHAIN_CONTEXT.submit_tx(tx.to_cbor())

Thanks, this is very helpful. What I found is that the verification key in the witness of cbor is empty.
echo 84a400818258207c806478ff2de0c970523870cd57f508c0634c0dd0005ef835c16ef7a9c8001800018282583900b5b7e62899719388be48d412e1384311a68e44a01633d5185b203a643a746b909de8d2e7747ccb76125b25d98d198b9950f39fd70eb335611b0000000253bf98c082583900b56bb1f70feb383207c926af89ede6934f2faca85037f3d982801e5a4b1ab3fccc016ccdebd20006fe167d4e40bd1a12f934fdc4dd7574571a0049b9c3021a0002917d031a00acb03ca100818240584056be2bf9c7c3a6f8d429bfe31febb794b6ec4ccd64139dc179f15688330105c4cb8bb886da54f3fff3b7e342d856b688396c597d226ed6ff5d2a553ecb029408f5f6 | xxd -r -p | base64 | python3 -m cbor2.tool -d | jq
[
  {
    "0": [
      [
        "|\\x80dx\\xff-\\xe0\\xc9pR8p\\xcdW\\xf5\b\\xc0cL\r\\xd0\u0000^\\xf85\\xc1n\\xf7\\xa9\\xc8\u0000\u0018",
        0
      ]
    ],
    "1": [
      [
        "\u0000\\xb5\\xb7\\xe6(\\x99q\\x93\\x88\\xbeH\\xd4\u0012\\xe18C\u0011\\xa6\\x8eD\\xa0\u00163\\xd5\u0018[ :d:tk\\x90\\x9d\\xe8\\xd2\\xe7t|\\xcbv\u0012[%ٍ\u0019\\x8b\\x99P\\xf3\\x9f\\xd7\u000e\\xb35a",
<PHONE_NUMBER>
      ],
      [
        "\u0000\\xb5k\\xb1\\xf7\u000f\\xeb82\u0007\\xc9&\\xaf\\x89\\xed\\xe6\\x93O/\\xac\\xa8P7\\xf3ق\\x80\u001eZK\u001a\\xb3\\xfc\\xcc\u0001l\\xcd\\xeb\\xd2\u0000\u0006\\xfe\u0016}N@\\xbd\u001a\u0012\\xf94\\xfd\\xc4\\xddutW",
        4831683
      ]
    ],
    "2": 168317,
    "3": 11317308
  },
  {
    "0": [
      [
        "",    <-------- This is the problem
        "V\\xbe+\\xf9\\xc7æ\\xf8\\xd4)\\xbf\\xe3\u001f뷔\\xb6\\xecL\\xcdd\u0013\\x9d\\xc1y\\xf1V\\x883\u0001\u0005\\xc4ˋ\\xb8\\x86\\xdaT\\xf3\\xff\\xf3\\xb7\\xe3B\\xd8V\\xb6\\x889lY}\"n\\xd6\\xff]*U>\\xcb\u0002\\x94\b"
      ]
    ]
  },
  true,
  null
]

The verification key was created and added to witness set by this line, which calls to_verification_key that yielded an empty vkey. Could you call to_verification_key from signing key directly and check if it is empty?

Calling to_verification_key on the signing_key returns a proper key
<bound method SigningKey.to_verification_key of {"type": "PaymentSigningKeyShelley_ed25519", "description": "PaymentSigningKeyShelley_ed25519", "cborHex": "58204c50b442277d6b322b08fdb534f2eecf20e372fe70ae96c07bdcce2c55976812"}>

I tried using v0.6.3 and 0.7.0 by the way

Thanks Robin. Could you try constructing the signed transaction manually like this and share the output? This will help us understand which step went wrong.
...
tx_body = builder.build(change_address=address)
signature = payment_signing_key.sign(tx_body.hash())
vk = payment_signing_key.to_verification_key()
print(vk)
vk_witnesses = [VerificationKeyWitness(vk, signature)]
print(vk_witnesses)
tx = Transaction(tx_body, TransactionWitnessSet(vkey_witnesses=vk_witnesses))
print(tx)
tx_cbor = tx.to_cbor()
print(tx_cbor)

Thanks for your support Jerry!
This is the entire printed output.
{"type": "PaymentVerificationKeyShelley_ed25519", "description": "PaymentVerificationKeyShelley_ed25519", "cborHex": "5820ae60924a2a814453c0fc57a9a3e685643ed5a6f5af227298cb5105a12f25092e"}
[{
  'signature': b'\xfbwE\xfd\xa1Y\xf2YUN\xd3)\x1d\xfd\x80\xdaA\xbf\x8aY\x9d\xbdG\x8eml\xc1\x06;\xc8N\xfb\x88\x17L\xf3\xb2\xe6\xa0\x11\xbb>\x84\x99\x92\xc7\xa9r\xa6\x03\x03W~\n\xa5q\xb6*l\xc8\x01.v\r',
  'vkey': {"type": "PaymentVerificationKeyShelley_ed25519", "description": "PaymentVerificationKeyShelley_ed25519", "cborHex": "5820ae60924a2a814453c0fc57a9a3e685643ed5a6f5af227298cb5105a12f25092e"},
}]
{
  'auxiliary_data': None,
  'transaction_body': {
    'auxiliary_data_hash': None,
    'certificates': None,
    'collateral': None,
    'collateral_return': None,
    'fee': 168317,
    'inputs': [
      {
        'index': 0,
        'transaction_id': TransactionId(hex='7c806478ff2de0c970523870cd57f508c0634c0dd0005ef835c16ef7a9c80018'),
      },
    ],
    'mint': None,
    'network_id': None,
    'outputs': [
      {
        'address': addr_test1qz6m0e3gn9ce8z97fr2p9cfcgvg6drjy5qtr84gctvsr5ep6w34ep80g6tnhglxtwcf9kfwe35vchx2s7w0awr4nx4sstap56q,
        'amount': {'coin': 10000000, 'multi_asset': {}},
        'datum': None,
        'datum_hash': None,
        'post_alonzo': False,
        'script': None,
      },
      {
        'address': addr_test1qz6khv0hpl4nsvs8eyn2lz0du6f57tav4pgr0u7es2qpukjtr2elenqpdnx7h5sqqmlpvl2wgz735yhexn7ufht4w3tsqx0vgv,
        'amount': {'coin':<PHONE_NUMBER>, 'multi_asset': {}},
        'datum': None,
        'datum_hash': None,
        'post_alonzo': False,
        'script': None,
      },
    ],
    'reference_inputs': None,
    'required_signers': None,
    'script_data_hash': None,
    'total_collateral': None,
    'ttl': 11392138,
    'update': None,
    'validity_start': None,
    'withdraws': None,
  },
  'transaction_witness_set': {
    'bootstrap_witness': None,
    'native_scripts': None,
    'plutus_data': None,
    'plutus_v1_script': None,
    'plutus_v2_script': None,
    'redeemer': None,
    'vkey_witnesses': [
      {
        'signature': b'\xfbwE\xfd\xa1Y\xf2YUN\xd3)\x1d\xfd\x80\xdaA\xbf\x8aY\x9d\xbdG\x8eml\xc1\x06;\xc8N\xfb\x88\x17L\xf3\xb2\xe6\xa0\x11\xbb>\x84\x99\x92\xc7\xa9r\xa6\x03\x03W~\n\xa5q\xb6*l\xc8\x01.v\r',
        'vkey': {"type": "PaymentVerificationKeyShelley_ed25519", "description": "PaymentVerificationKeyShelley_ed25519", "cborHex": "5820ae60924a2a814453c0fc57a9a3e685643ed5a6f5af227298cb5105a12f25092e"},
      },
    ],
  },
  'valid': True,
}
84a400818258207c806478ff2de0c970523870cd57f508c0634c0dd0005ef835c16ef7a9c8001800018282583900b5b7e62899719388be48d412e1384311a68e44a01633d5185b203a643a746b909de8d2e7747ccb76125b25d98d198b9950f39fd70eb335611a0098968082583900b56bb1f70feb383207c926af89ede6934f2faca85037f3d982801e5a4b1ab3fccc016ccdebd20006fe167d4e40bd1a12f934fdc4dd7574571b000000025370bc03021a0002917d031a00add48aa10081825820ae60924a2a814453c0fc57a9a3e685643ed5a6f5af227298cb5105a12f25092e5840fb7745fda159f259554ed3291dfd80da41bf8a599dbd478e6d6cc1063bc84efb88174cf3b2e6a011bb3e849992c7a972a60303577e0aa571b62a6cc8012e760df5f6

Thanks Robin! This cbor looks correct. If you submit this transaction, it will go through. It is still a mystery to me why build_and_sign resulted in an empty verification key. Is this issue consistently reproducible?

Yeah, the tx went through.
Correct, I was consistently getting this error. I've tried about 5-6 times yesterday.
I just put build_and_sign back in, so it is now as it was when I got the error. I tried another transaction and it succeeded! To be honest, now I am completely lost.

Closing this issue as it is not reproducible now. @robinboening If you encounter this problem again, please feel free to reopen this issue.