feat: Optimize the issuance of attestations

What does this PR do?
Optimizes various points to get cheaper attestation issuance:

Smaller events data
Less events
No useless returned value
Timestamp as uint64
Better packed struct definition

Related ticket
Fixes #145
Type of change

[X] Chore
[ ] Bug fix
[ ] New feature
[ ] Documentation update

Check list

[X] Unit tests for any smart contract change
[X] Contracts and functions are documented

Codecov Report

Merging #146 (0634b59) into dev (14628b4) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #146      +/-   ##
==========================================
- Coverage   96.75%   96.64%   -0.11%
==========================================
  Files           8        8
  Lines         154      149       -5
  Branches       37       37
==========================================
- Hits          149      144       -5
  Misses          4        4
  Partials        1        1

Files Changed
Coverage Δ

src/AttestationRegistry.sol
97.22% <100.00%> (-0.28%)
:arrow_down:

src/ModuleRegistry.sol
95.83% <100.00%> (ø)

src/PortalRegistry.sol
96.00% <100.00%> (ø)

src/example/EASPortal.sol
100.00% <100.00%> (ø)

src/interface/AbstractPortal.sol
94.44% <100.00%> (-0.30%)
:arrow_down:

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


creating test script for initial pull request

Maybe you can simplify it a bit? You rely on dictionaries etc, but there does not seem to be a reason to keep track of the ids.
Maybe you could just have that your function inputs a list of sequences, the alignment parameters, and the GA parameters) and just returns a matrix with the alignment. I think you can represent the alignment a Char/integer matrix with the number of rows, the number of sequences. You could even use nothing to indicate gaps.


Improve error message for Node.set_owner

Provides a less cryptic message than Condition "!owner_valid" is true.
While this is documented clearly I feel a clearer error message helps greatly here, as the check is for an unknown variable (from the perspective of a user) unlike something like ERR_FAIL_COND(p_owner == this) where the condition speak for itself (and users don't always look at the documentation when facing an error either, though arguably they should that's no guarantee, and it's easy to explain here)

Thanks!
I went ahead and merge the current version which is already a major improvement. We can fine tune the explanation of edge cases as pointed out by @kleonc in a follow-up, so to be fair I'm not sure how to describe it without being more confusing that necessary.

Thank you!
Agreed was going to suggest merging this and leaving the rest to further work

Cherry-picked for 4.1.1.


Bug in feature/ctre_lexer

Cannot load OE_Demo2.csl and car_gma950.csl. (press '2' or '4')
I can still load monkeys.csl and OE_Demo50MB.csl. (press '3' or '1')
I can load everything on the current master branch.
To test:

Run ./builddir/OE_Test
After the demo loads press '2' or '4'.

The above bug is fixed!
More fixes you can do:

Throw a proper exception if the requested file is not found.
Make physics engine
Try to fix loading of my_tanks.csl. (No Light found)
Make physics engine


Getting Use of undeclared type 'MFMailComposeViewController'

Hi,
I am getting the following error in automockable file which are as follows:
following two are from FBSDKShareKit

Use of undeclared type 'ShareDialog'
Use of undeclared type 'MessageDialog'

following two are from MessageUI

Use of undeclared type 'MFMessageComposeViewController'
Use of undeclared type 'MFMailComposeViewController'

Please let me know is there any changes which i need to do. For other pods it is working fine so unable to find the root cause.
Thanks!

have you imported those frameworks in the generated code you have? because this isn't Sourcery issue

have you imported those frameworks in the generated code you have? because this isn't Sourcery issue

I have imported frameworks in the code it is being build successfully. When i am running test case then is showing error to me in automockable file.

@HB-HemantBhatt You should modify the AutoMockable.stencil file, this part:
import Foundation
#if os(iOS) || os(tvOS) || os(watchOS)
import UIKit
#elseif os(OSX)
import AppKit
#endif

Just add import FBSDKShareKit and import MessageUI below UIKit.
And if you're not doing any OSX development, you might as well delete the whole #if #else #endif parts.


Create shop-calculator

Shop Calculator adds a calculator for bulk buying from NPC shops in the side panel

Don't close and open new pr's. Push to the same branch.


#120 Move SidebarLayout classes to tokens' configuration

Basic information

Tiller version: 1.7.0
Module: menu, theme

Description
Moved fixed classes into rewamped tokens' configuration for SidebarLayout component.
Related issue
Closes #120
Types of changes

Breaking change (fix, enhancement or feature that would cause existing functionality to change in backward-incompatible way)
Refactoring

Checklist

[x] I have read the project's CONTRIBUTING document
[x] I have checked my code with the project's static analysis tooling
[x] I have formatted my code with the project's Prettier code-style configuration
[x] I have checked my code for misspellings
[x] I have organized my changes in easy-to-follow commits
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added a story to cover my changes
[x] All new and existing story tests pass

Seems ok. @fbeceic you can merge this and publish it, but please increase minor version


Only work for specific lsp?

I install sumneko_lua, jedi, bashls, vimls, cssls, html, only sumneko lua work as expected.

same issue here

I guess I'm gonna use lsp-status meanwhile

Oh, not sure but aparently not all servers report status .. perhaps that's the issue


Create a book of best practices

We should provide a place where we can create/document short best practices snippets for concreate. This should be ideally in some wiki format where people can contribute easily.

I'll probably start using wiki on github - its probably good enough

Because we wanted this to be live and updated by community. It should be place where to share stuff. Or if we can enable part of read the docs to be updated with every commit it could serve too,

It is always updated after every commit: http://concreate.readthedocs.io/en/develop/ (develop vs. latest) Additionally GH gives a nice preview for these files and we can use PR's from community. I see it just as a win.

Closing, as decided, that this should be part of official docs.


One question

Hi, recently, I am confused by whether the training set should include outliers, I saw you said you trained the normal time series data, I do not know the meaning of normal.
Please reply to me when you saw it,
Many Thanks,
Finn

Hi Finn,
by normal I mean the observation which are not anomalies. All expected patterns/behaviours in data are normal observation. Observation(s) which show unusual behaviour are anomalies. The datasets I used had labelled anomalies. I removed these while training the prediction model. This is because we want the model to be learn the normal patterns. So that when the model encounters an anomaly, it makes a high prediction error, as the model has not been trained on such data. These high prediction errors on anomalies are then exploited to act as anomaly indicators.
I hope this explanation clears your doubt.


Add reference to logo conversation.

At the top of the page. We can remove this when we've got a logo :)

Sounds like a good idea to me. Going to merge this in. Thanks @simonv3


[FEATURE] Add show/hide snippets (browser)

Description
Snippets for showing and hiding elements, similar to jQuery's methods.
What does your PR belong to?

[ ] Website
[x] Snippets
[ ] General / Things regarding the repository (like CI Integration)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (non-breaking improvement of a snippet)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have checked that the changes are working properly
[x] I have checked that there isn't any PR doing the same
[x] I have read the CONTRIBUTING document.

timing is an annoyance when trying to approve and then request changes XD


When ANY user click to view profile of deleted User-> error is displayed.

Open "https://staging.fightpandemics.work"
Onboarding User -> Click on 'Offer Help' -> Volunteer
Enter zip code as '08536' -> skip

Issue: Most probably you will view the POST by the user having logo as 'NS' -> Click on this user logo to view the profile.
Below error is displayed:
Failed loading profile, reason: Not Found
Hence, this issue exist if user profile is deleted but the POST of the respective user still exist in the system.
Refer attached GIF:
https://share.getcloudapp.com/7Kuml1gz

Hi @rabbi08 - Right now there is no mechanism to delete users or organizations through the app. If we do manually in the DB then we should also delete posts I guess... (I will do so now for)
However, as a fallback, we have an issue to create a 404 page #1000 . If a user/ID , post/ID etc is not found we should redirect to that page (once it is created).
Later if we give users an ability to delete an organization or their own profile then we need to decide how to handle existing posts (disable comments, hide, totally delete?).
But there's nothing really actionable here code base wise so I added "on hold" label.


buildah-from: add effective value to mount propagation

According to the kernel documentation.
Signed-off-by: Zhou Hao<EMAIL_ADDRESS>

LGTM
Not sure why you would use this one, but it is possible.

LGTM

@rh-atomic-bot r+
Thanks @q384566678


Add tau to the math library

the current way of having the tau is like this:
local tau = math.pi * 2

however, it doesn’t go through all scripts
This method would make is useable in all scripts
math.tau -- math.tau = (math.pi * 2)

so Is there any chance this will be added?

Lines need to be added to luaopen_math method:
lua_pushnumber(L, TAU);
lua_setfield(L, -2, "tau");

@benpinpop I added that line of code at the end before the return ;)

I feel like a stronger usecase needs to be provided before Roblox will take this onboard, there are several constants you could have added to the math library, but wont be because there's no strong usecase.
tau as a concept doesn't serve a strong usecase over doing math.pi * 2, unless you really care about micro optimisations, no MULK op, only a GETIMPORT, but even here, there's barely a strong usecase to this.

I presume the biggest reason why others (myself included) would like math.tau is it's just nice to have. For the sake of code cleanliness and continuity, it'd be nice to see math.tau instead of a comparatively smaller tau; plus, then I don't have to define it in any of my potentially dozens of modules/scripts that already have a bunch of constants at the top. It's just there to use.

@tfilteredc Here is some use case, and it improves readability and performance, do you want to add 150 of your scripts this line??? local tau = (math.pi * 2) that’s daunting and time consuming
local TweenService = game:GetService(`TweenService`)

local part = script.Parent.Part
TweenService:Create(part, TweenInfo.new(math.tau), {CFrame = CFrame.new(math.tau, math.tau, math.tau)}):Play()
print(`Part has reached {CFrame.new(math.tau, math.tau, math.tau)} in {math.tau}`)

do you want to add 150 of your scripts this line??? local tau = (math.pi * 2) that’s daunting and time consuming

shared.tau = math.pi * 2?. (/j shared is a meme) Regardless, I could engineer a solution where I'm not writing this 150 times, but on convenience alone I see where you're coming from somewhat

a module, but the path can be messy, like this:
local Common = require(this.is.a.bad.lane.to.require.the.module.so.please.ignore.this.Common.lua)

Hello!  We appreciate the submission, but we've decided not to add math.tau.
It's very easy for you to introduce a constants module and define tau exactly as you describe.  We do not think that the effort required to do so is so high as to justify a permanent addition to the standard library.

We do not think that the effort required to do so is so high as to justify a permanent addition to the standard library.

I just want to ask why you think the effort is too high for this addition? There's only 4 lines for the PR request, it's extremely small, and should no have any bearing/consequence on anything else in the codebase.
Additionally, the math library already has specialized methods with very few use cases, such as: math.frexp and math.ldexp
math.tau would have a way more common use-cases in comparison, as much logic consisting of rotations will often require accounting for a full rotation (2pi).
Not to mention the tau constant is used in official player module scripts (Orbital Camera Script).
I don't think it's entirely fair to dismiss the idea just because you can make it a constant yourself, It's just a nice convenience factor. Other game engines and math libraries such as Unity/Unreal/GLM/Godot/etc.. all support and define many more math constants to make lives of developers easier.

thanks for explaining @incetents ;)

You are free to define as many constants as you want in your tables.

@vegorov-rbx what do you mean?

Dropping my belated support on this. As an avid user of a wide and ever-diversifying range of other software packages where this is offered rightfully as a standard option, I am extremely disappointed that this feature request was shut down and urge those involved to reconsider their case and survey for broader sentiment among developers. This is a commonplace and unnecessary impediment of exactly the variety Luau should be solving for its developers, following exactly the precedent that prior approved additions to Luau have set (such as += operator shorthand, generalised iteration and string interpolation, to name only a few).
I fail to see any reason why this should not be added. Even in languages designed to rely on user code over a large standard library, such as Rust, their standard libraries include tau in acknowledgement of its wide usage and broad application to huge swathes of modern geometry and various algebrae relevant to computer graphics and physics. In my own work between Mathematica, Roblox and Rust, tau is the preferred constant to reach for every time due to how it best allows for unified forms of many computations and greatly simplifies the developer-side burden of having to track overcomplex coefficients, ratios and fractional divisions. I expect the mathematical libraries I work with to have adequate provisions to facilitate transcription of common expressions; it is simply not good enough that Roblox breaks step with industry by forgoing the inclusion of a fundamental constant that appears so universally.
When so much has already been done, why is a line inconsistently drawn at a single numeric constant in an immutable standard library table, with it's addition evidenced by in-field usage and bolstered with social proof?
I would like to see the case against this change made concrete.

I'll just note that we have a documented RFC process. We are not going to merge any PRs that add language or library features without an RFC, as specified in the contributing guide. For any change to happen, the change needs an RFC that explains the motivation - this change didn't have it (and also didn't add the feature correctly as it was missing type information changes), so it couldn't be accepted.
Also, does Mathematica define Tau? Pi is documented here https://reference.wolfram.com/language/ref/Pi.html but Tau isn't.


Automate pre release (WIP)

Description
Start to automate the manual steps for releasing a new image version
Motivation & context
Reduce mistakes and save time. This is a process we'll eventually roll out in the other projects too.
Type of change

Refactoring/debt (improvement to code design or tooling without changing behaviour)

Note to other contributors
If your change may impact future contributors, explain it here, and remember to update README.md and CONTRIBUTING.md accordingly.
Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

OK I'm getting somewhere, but I have a design decision to make.
Currently, we communicate the version number to be released by the release.yaml workflow by hard-coding it in that file, under the env.version key.
We've been manually updating this to do a release, but if we want our pre-release workflow to do this, it's going to need additional permissions, because by default the GITHUB_TOKEN can't update workflows.
The easy option would be to use a token with the workflows permission, but that seems like a bad idea for security reasons.
As an alternative, that release.yaml workflow needs to read the version-to-be-released from somewhere else. I guess we can use a file.

I think there might be another option where we pass the information about the version-to-release in the branch name, but I can't figure that out right now.

I've merged this in d7183772d966afe69c200fac76b7f67b3e50f640 - it's too difficult to play with in a branch.


interpolate tagging for media queries?

hello,
i try to lint my styled components with stylelint order plugin and i don't know how to tag my tagged template literals i use for media queries.
my example component:
const Button = styled.button`
  width: 100%;

  ${desktop`
    width: auto;
  `};
`

for this i get an Expected -styled-mixin0 to come before width error. as far as i understand i need something like ${/* sc-block */ desktop``} but it didn't help.
i can solve this by something like ${/* sc-custom '@media screen{a{top:1}}' */ desktop``} ('@media screen {}' would be an empty block so i need a as a short selector and top as a short property) but it looks stupid and i'd like to avoid comments like this.
is it possible to declare my own tags somehow? or do you know a better solution?

Ahhh, okay, the thing is what sc-block does is replace your interpolation with styled-mixin (which is also what our processor guesses is the right thing to do, so it's the same with or without the tag for now). I'm afraid that we do not support making your own blocks as of right now, and the best we offer is exactly the custom tag. So I see two possible solutions for you:

Use the custom tag, disable the order plugin, or order your interpolations at the place the linting rule wants you to even though it's not the intended place, basically make do with what you already have
Add the feature you request, I don't actually think it should be way too hard, you could just add a config option that is an object with keys being the tag and values being what the tag should be replaced with. And then just a small refactor of this https://github.com/styled-components/stylelint-processor-styled-components/blob/master/src/utils/tagged-template-literal.js#L49-L111 so that it's a map with a special case for the custom tag maybe. Really shouldn't be too bad.

I'll tag this as an enhancement now, but if you can't be bothered implementing it feel free to close the issue.

@gex I think sc-custom has given you the ability to declare anything that need to be replaced with. For you example, how about simply disable the linting of interpolation by /* stylelint-disable */ and recover by /* stylelint-enable */?
const Button = styled.button`
  width: 100%;

  /* stylelint-disable */
  ${desktop`
    width: auto;
  `};
  /* stylelint-enable */
`

@chinesedfan i was under the impression that the css rules inside my desktop interpolations are linted by stylelint. i had some issues with syntax highlighting in atom and i fixed that with these steps. now i understand it wasn't about syntax highlighting only.
@emilgoldsmith i'm gonna try to add a pull request but it would be nice to solve this other issue at once. having this code
const Button = styled.button`
  width: 100%;

  ${props =>
    props.danger
      ? css`
          background: red;
          color: white;
        `
      : '' /* sc-custom 'zoom: 1' */};

  ${desktop`
    width: auto;
  ` /* sc-custom '@media (min-width: 75rem) { a { top: 1; } }' */};
`

stylelint understands what's inside  css related to the danger prop but it doesn't know anything about what's inside the desktop interpolation. when i first read about interpolation tagging i assumed that's what sc-block does, instructing stylelint that it should check the template as well. is it possible to force the linting of a template content somehow? i think this sc-media we are talking about should also do that. what do you think?

Hmm I'm not sure I understand the details of what you're saying @gex but I'll clarify something I'm not sure whether you understand or not:
Our processor doesn't evaluate any Javascript whatsoever, it just reads your file as a static text file and parses the meaning of the Javascript without any evaluation and tries to extract the CSS to give to Stylelint so it can lint it. That means that we don't understand anything whatsoever inside any interpolation, as interpolations are javascript. Our processor just tries to replace interpolations with dummy CSS that won't make any stylelint errors but we will never be able to actually lint the contents of an interpolation. Then we'd need to follow imports and all kinds of crazy things, and it would suddenly possibly become very expensive in terms of RAM and CPU just for a linter,

@emilgoldsmith linting was always magical for me so thank you for making it more clear! :)
let me rephrase my questions. i'd like to solve two problems:

i'm using the stylelint order plugin and i didn't know how to tell the processor that my interpolation is a media query. i can move my mixin into the properties to keep the alphabetical order but that's not readable in my opinion or i can use the sc-custom tag but it feels painful for me because i have to provide a valid, not empty media query to prevent linting errors. and because of the length of that tag i think it would be great to have an sc-media tag. i haven't run into other cases where it would be beneficial to know that the interpolation is a media query but since there are several other stylelint rules that are related to media queries i think i or other people maybe will :)

// instead of this
const A = styled.a`
  color: blue;

  ${desktop`
    color: lightblue;
  ` /* sc-custom '@media screen { a { top: 1; } }' */}
`

// i'd prefer this
const A = styled.a`
  color: blue;

  ${/* sc-media */ desktop`
    color: lightblue;
  `}
`

i understand that the processor is just extracting the css code to give it to stylelint, that's how it understands what's inside a css interpolation but sometimes other interpolations also can contain a huge chunk of css and in these cases it would be great to be able to instruct the processor to extract that css as well. a proposed solution would be an sc-css tag:

// this doesn't have stylelint errors at the moment
const A = styled.a`
  color: blue;

  ${desktop`
    kolor: lightblue;
    text-align: centaur;
  `}
`

// this would have
const A = styled.a`
  color: blue;

  ${/* sc-css */ desktop`
    kolor: lightblue;
    text-align: centaur;
  `}
`

but since we know what could contain css, and i mean lintable css (sc-block and sc-media maybe sc-declaration?) and what not (sc-selector, sc-property and sc-value) an intelligent guessing would imply sc-css for the former.

...aaaand i just found #182 😇

I don't think we will ever support extracting anything from an interpolation to be honest, it'll be very hard to generalize it without evaluating CSS. I guess one could make a specific tag for breakpoints since it's a reocmmended SC pattern, and wraps it in a media query. It could then recurse on the TTL inside the interpolation. It would be a pretty custom case though, and not sure if enough users want it to support it? Maybe?
I'm completely okay with as I said above allowing a config option that can let you define your own tags as shortcuts, I think that could make some people's life quite a bit easier so you don't have to write the ugly custom tags.
Also beware though that processor support may be deprecated https://github.com/stylelint/stylelint/issues/3408
and you might want to consider contributing to https://github.com/gucong3000/postcss-jsx instead


Other necessary documentation for ovgs.proto

DeleteGroup(): What error is returned if a group is not empty?
AddUserRole(): What is returned if the username, org_id, user_type (ie: the specific user) does not exist?
*Serial(): clean-up language after the creation of Component.
GetOwnershipVoucher(): what is component or cert_id do not exist?

CreateGroup(): what if a group with the same parent & description exists?


Enh. Req. for sap.m.PlanningCalendar: Provide an option to select an interval range

For the purpose of creating a new entry it would be helpful, if there would be an option that the user can select an interval range by dragging across the interval header.

Hi,
I informed the Product Owner about this proposal. Maybe it's even planned anyway, as it sounds like a logical next feature of this control.
In case of questions or significant news he will come back to you, but as this issue list is mainly meant for bug reports, there will not necessarily be a follow-up activity (unless you or someone else decides to contribute the feature ;-))
Regards
Andreas

Hi @DerGuteWolf,
This should work already, see example: https://openui5nightly.hana.ondemand.com/entity/sap.m.PlanningCalendar/sample/sap.m.sample.PlanningCalendarDnD
Best regards,
Petya


htaccess

Hello, how do I use Rewrite in htaccess?
example:
https://foodomaa.club/index.php?topic=3.msg6
for
https://foodomaa.club/Whitescreenproblems

If they are exact like that, you could use ``Redirect``` directives.
Redirect 301 /Whitescreenproblems /index.php?topic=3.msg6

But there are several topics and there is no way to create a redirect for each one, in case you would have an example for me.

Not having more details its hard to help.  Rewrite  is a good choice if you have a specific pattern to rewrite.  If there is not really a pattern, then several Redirect are an easy route.

@Radicaliza are you having some topics that show only a white screen or are you trying to obtain a sort of semantic URL or SEO-friendly url (or whatever they are called nowadays)?

Closing as stale and really is a help topic better suited for the site.


Local Docker Registry - SSL

Environment
Device and OS: Ubuntu 20.04
App version: v0.17.0
Kubernetes distro being used: kind
Other:
Steps to reproduce

Install the self signed certificate to /etc/docker/certs.d/hostname:port/ca.crt. Once this is installed, restart docker.
Setup a new package to include an image from a docker registry that is hosted on site with a self signed cert.

Expected result
Make a tarball
Actual Result
ryoung@zarf:~/zarf/examples/sliceup$ zarf package create --confirm
kind: ZarfPackageConfig
metadata:
name: sliceup
description: Sliceup Deployment
architecture: amd64
build:
terminal: zarf.local.sliceup.co
user: ryoung
architecture: amd64
timestamp: Sat, 02 Apr 2022 13:26:19 +0000
string:  v0.17.0
components:

name: baseline
required: true
manifests:

name: sliceup
files:

manifests/chserver.yml
manifests/clickhouseapi.yml
manifests/clickhouse_int.yml
manifests/jobmanager.yml
manifests/postgres.yml
manifests/slicef.yml
manifests/sqlcron.yml
manifests/taskmanager.yml
manifests/vector.yml
manifests/yandex-client.yml
images:

defenseunicorns/zarf-game:multi-tile-dark
nexus.local.sliceup.co:443/chserver_master:8
nexus.local.sliceup.co:443/clickhouseapi_dev:9
nexus.local.sliceup.co:443/clickhouse_int_release-1.0.0:1
confluentinc/cp-enterprise-control-center:6.0.1
nexus.local.sliceup.co:443/flink_sliceup:latest
nexus.local.sliceup.co:443/flink-dev:working
nexus.local.sliceup.co:443/postgres_master:1
nexus.local.sliceup.co:443/sliceup-f:v3
nexus.local.sliceup.co:443/sqlcron_master:16
nexus.local.sliceup.co:443/flink-dev:working
hlebalbau/kafka-manager:stable
timberio/vector:0.15.1-debian
yandex/clickhouse-client
•  Create Zarf package confirmed

📦 BASELINE COMPONENT
✔  Copying manifests/chserver.yml
✔  Copying manifests/clickhouseapi.yml
✔  Copying manifests/clickhouse_int.yml
✔  Copying manifests/jobmanager.yml
✔  Copying manifests/postgres.yml
✔  Copying manifests/slicef.yml
✔  Copying manifests/sqlcron.yml
✔  Copying manifests/taskmanager.yml
✔  Copying manifests/vector.yml
✔  Copying manifests/yandex-client.yml
✔  Loading 1 manifests
ERROR:  Unable to pull the image nexus.local.sliceup.co:443/flink_sliceup:latest
ryoung@zarf:~/zarf/examples/sliceup$ zarf -l debug package create --confirm

Log level set to debug
DEBUG   Loading zarf config zarf.yaml
DEBUG   Creating temp path /tmp/zarf-1106993399
DEBUG   config.IsZarfInitConfig
DEBUG   config.GetSeedImage()
DEBUG   config.IsZarfInitConfig
DEBUG   config.BuildConfig(/tmp/zarf-1106993399/zarf.yaml)
kind: ZarfPackageConfig
metadata:
name: sliceup
description: Sliceup Deployment
architecture: amd64
build:
terminal: zarf.local.sliceup.co
user: ryoung
architecture: amd64
timestamp: Sat, 02 Apr 2022 13:26:24 +0000
string:  v0.17.0
components:

name: baseline
required: true
manifests:

name: sliceup
files:

manifests/chserver.yml
manifests/clickhouseapi.yml
manifests/clickhouse_int.yml
manifests/jobmanager.yml
manifests/postgres.yml
manifests/slicef.yml
manifests/sqlcron.yml
manifests/taskmanager.yml
manifests/vector.yml
manifests/yandex-client.yml
images:

defenseunicorns/zarf-game:multi-tile-dark
nexus.local.sliceup.co:443/chserver_master:8
nexus.local.sliceup.co:443/clickhouseapi_dev:9
nexus.local.sliceup.co:443/clickhouse_int_release-1.0.0:1
confluentinc/cp-enterprise-control-center:6.0.1
nexus.local.sliceup.co:443/flink_sliceup:latest
nexus.local.sliceup.co:443/flink-dev:working
nexus.local.sliceup.co:443/postgres_master:1
nexus.local.sliceup.co:443/sliceup-f:v3
nexus.local.sliceup.co:443/sqlcron_master:16
nexus.local.sliceup.co:443/flink-dev:working
hlebalbau/kafka-manager:stable
timberio/vector:0.15.1-debian
yandex/clickhouse-client
•  Create Zarf package confirmed

📦 BASELINE COMPONENT
✔  Copying manifests/chserver.yml
✔  Copying manifests/clickhouseapi.yml
✔  Copying manifests/clickhouse_int.yml
✔  Copying manifests/jobmanager.yml
✔  Copying manifests/postgres.yml
✔  Copying manifests/slicef.yml
✔  Copying manifests/sqlcron.yml
✔  Copying manifests/taskmanager.yml
✔  Copying manifests/vector.yml
✔  Copying manifests/yandex-client.yml
✔  Loading 1 manifests
DEBUG   Get "https://nexus.local.sliceup.co:443/v2/": x509: certificate signed by unknown authority
ERROR:  Unable to pull the image nexus.local.sliceup.co:443/postgres_master:1

Visual Proof (screenshots, videos, text, etc)
Severity/Priority
Additional Context
I have performed a docker pull on all the images on my local workstation, and those images were pulled successfully. My question is where do we input the CA certificate when building the init container? I am assuming we put it there. Any information on how to install a CA.crt file in the environment so I can package up my application, would be most helpful.

Thanks @ryansliceup, that's a good find. Under the hood we use Crane, which it seems does not support  the docker private CA.  However, on Linux they seem to indicate you can use SSL_CERT_FILE to handle this.  Alternatively, if the private registry is trusted on your system, then it should work as well.  A related thread that discussed this is here.
We definitely need to add a note to the docs about this once we settle on the right solution.  Since you're facing the issue right now, would welcome your recommendations on the best approach to take.

Thanks for the information! I was able to get past this issue. When you mean a private registry is trusted, do you mean from the operating system perspective, or docker? For instance if you put the certificate in /etc/docker/certs.d/hostname:port/ca.crt, then only docker will trust it. I figured I would be okay here, but that was not the case.
I should have tried to put the cert in /usr/local/share/ca-certificates and then ran update-ca-certificates.

Yeah that's the issue, zarf doesn't use docker itself (which is some ways is good because it means it's not a dependency), and the tool we use doesn't support the docker cert path. So you have to do an online override or update the OS cert list.


fix select input error and add amoozeshyar payment url and update manifest version

…ifest version to version 3

Hi there! Thank you so much for contributing to my project.
I'm reviewing your merge request, and I've noticed an issue with the removal of the DOMContentLoaded event listener. Here's my concern:
The script, as written, attempts to select and modify input elements. However, without DOMContentLoaded, it might run before the target elements are actually available in the DOM. This could result in missed event listeners, as elements might not be found.
I'm open to feedback, so if you have a different perspective or can offer a convincing justification for the removal, please share your thoughts!
Looking forward to your response!


ListenableBuilder duplicate prevents building with flutter stable

Despite https://github.com/superlistapp/super_editor/pull/880, _listenable_builder.dart itself imports flutter, which currently results in a compile error for newly released Flutter stable, 3.10.
Resolved by chaning to import 'package:flutter/widgets.dart' hide ListenableBuilder; on line one.
To reproduce:

Have the following in pubspec.yaml for any app

environment:
  sdk: ">=3.0.0 <4.0.0"
  flutter: ^3.10.0

dependencies:
  flutter:
    sdk: flutter
  super_editor: ^0.2.4

try building any platform.
results in various compile errors
Error: 'ListenableBuilder' is imported from both 'package:flutter/src/widgets/transitions.dart' and 'package:super_editor/src/infrastructure/_listenable_builder.dart'

Can you please describe which versions of super_editor are failing to compile?

v0.2.4-dev.1 on Pub
v0.2.4 on Pub
GitHub main branch
GitHub stable branch

Also, can you please file an issue for this PR so that we have documented history for a fix?

I'm going to close this issue in favor of https://github.com/superlistapp/super_editor/issues/1131 - there are multiple things we need to fix, and we need to run new releases.


declare module for git.d.ts

This PR fixes https://github.com/microsoft/vscode/issues/120448. This makes it easier for other extensions to use the git extension's apis so they can add a reference to the git module instead of copying the git.d.ts file to each extension and needing to update it when there are changes.

@kisantia How exactly does this make it any easier? Doesn't a third-party extension still need to copy paste git.d.ts?

Closing this pull request as the benefits of thise work is not clear.


Add Medieval Weapons Pack for KIS from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of jebheadmugkerman, to add Medieval Weapons Pack for KIS to CKAN.
Mod details:
name = /mod/1109/Medieval%20Weapons%20Pack%20for%20KIS
author = jebheadmugkerman
abstract = In this mod you use instead of guns medieval weeapons!
license = MIT
Homepage =
description = Tired of Big noisy ugly guns? then this mod is for YOU! Now you can just creep quietly behind your enemy and stab them in the back when they are not looking!
   ATTENTION! to have your kerbal have these parts in there hand you need the Kerbal Inventory System mod.

Please direct questions about this pull request to jebheadmugkerman.

Fixed


GROOVY-8907: Correcting @ClosureParams to correct type

https://issues.apache.org/jira/browse/GROOVY-8907

Merged. Thanks. Sorry for the delay. I knew there were problems in the early commits but hadn't noticed that you had corrected them all.

Thanks. No problem :)


Add minified bundle to NPM or bower

It would be nice to download minified version not from demo site, but from NPM.

Got the same problem today, and added it to NPM cause nobody acted.
But it didn't helped me a lot. I don't see ad anyway.
Link to NPM
npm install angular-dfp

Thanks for adding it! Sorry we didn't respond faster.

Hi guys! ^_^
Our intention was to complete the documentation and write a thorough suite of tests before releasing it on npm.


VHDL flow not working as ghdl not found inside yosys

Description
A user tried to use the VHDL flow in OpenLane in our IIC-OSIC-TOOLS image, see https://github.com/iic-jku/IIC-OSIC-TOOLS/issues/43.
The error is that the command ghdl is not working inside Yosys. If I start Yosys just by
> yosys

then running ghdl inside, it throws the error ERROR: No such command: ghdl (type 'help' for a command overview).
If I start Yosys with
> yosys -m ghdl

then running ghdl inside Yosys works fine!
Question: What is wrong with the OpenLane setup in our image?
Expected Behavior
VHDLClassic flow should work.
Environment report
Failed to get Docker info: [Errno 2] No such file or directory: 'docker'
Failed to get Nix info: [Errno 2] No such file or directory: 'nix'
kernel: Linux
kernel_version: 6.6.32-linuxkit
supported: True
distro: ubuntu
distro_version: 22.04
python_version: 3.10.12
python_path:
  - /usr/local/lib/python3.10/dist-packages/openlane
  - /headless/.local/lib/python3.10/site-packages
  - /usr/lib/python310.zip
  - /usr/lib/python3.10
  - /usr/lib/python3.10/lib-dynload
  - /usr/local/lib/python3.10/dist-packages
  - /usr/lib/python3/dist-packages
  - /usr/lib/python3.10/dist-packages
  - /foss/tools/ngspyce/154a272/local/lib/python3.10/dist-packages
  - /foss/tools/pyopus/0.11/local/lib/python3.10/dist-packages
  - /foss/tools/yosys/share/yosys/python3
  - /foss/tools/klayout/v0.29.4/pymod
tkinter: True
container_info: None
nix_info: None

Reproduction material
None needed.
Relevant log output
See above.

@donn Maybe the above issue just boils down to the question: How is OpenLane calling yosys, and how do you load modules like ghdl?

I don't understand nix, but it looks like there is some kind of wrapper involved?

We implemented now a wrapper similar to the above one in our IIC-OSIC-TOOLS. An alternative way, I think cleaner, is that OpenLane adapts the calling of yosys. Like using yosys for Verilog, or using yosys -m gdhl for VHDL, or yosys -m systemverilog.
Or just use yosys -m ghdl -m systemverilog in any case, if this modules are available.
@donn What do you think?

Doable- will think about it, I'm messing with Yosys stuff atm

Please notify me when you change things because then I will remove the Yosys wrapper again :-)

2.2.0 loads plugins explicitly, however, it introduces another issue because you need to pull in https://github.com/YosysHQ/yosys/pull/4553 until it gets merged
Anyways, should be in the next release. Thank you for your patience


[REQUEST] The Great Ace Attorney Chronicles PC

There are two version of this, one JP only, and one WW. The worldwide one was enhanced by a mod to support JP text as well.
https://store.steampowered.com/app/1158850/The_Great_Ace_Attorney_Chronicles/
Mod for JP
https://steamcommunity.com/sharedfiles/filedetails/?id=2567101744
The request would be to make both Great Ace Attorney games included hookable from the original Japanese via the modded WW version.

@aquiab wondering if you might find this similar to the Ace Attorney script you made

@aquiab wondering if you might find this similar to the Ace Attorney script you made
this uses mt framework instead of unity jit like the trilogy, it's probably a big jump in difficulty but i'll try to look up resources on how to do it and try my hand at it sometime


Typo fix

'getRegistrableDomain' should be 'getRegisterableDomain'

I don't think this is a type.  For one, "registrable" and not "registerable" is the actual name of the method within the library, not just in this instance but throughout.  Second, I believe "registrable" is the preferred spelling.
http://www.merriam-webster.com/dictionary/registrable

see also #85

Thanks for the PR! As @mikegreiling said, the corrected spelling is, uh, correct for what will become the latest version of php-domain-parser.

Ahh ok, got it. It's already changed in the code for the next release, but not on packagist ;-)
A bit confusing, since packagist grabs an older version and people check github for the docs. The docs in the package are correct. This PR can indeed be closed.

A bit confusing, since packagist grabs an older version and people check github for the docs.

Totally understand. Sorry for the confusion.


Fixed typo and grammer!

Just some text reviews and improved readability for other users.

I submitted the CLA.


how to use the command line to get it work at specific time of day ？

the arguments  /SetTime   only can “ Sets the time for the countdown.”
thinks

There is currently not the option to use time of day in the CLI, enabling that requires some light internal changes first.

I'll leave this issue open until the CLI is updated.


favicon added

Fixes #125
sample favicon logo added

Hi @sarthak77! The format for Fixes # is Fixes #n where n is the issue number. This way the issue automatically closes when the pull request is merged.

Hi @sarthak77! The format for Fixes # is Fixes #n where n is the issue number. This way the issue automatically closes when the pull request is merged.

Oh sorry!

Did you design this favicon? @sarthak77

Just a reference from there.
That’s why named samplefavicon.ico
Are you looking for something like that?

I think that the favicon should be made on similar principles as the big brands, like a couple of letter which can uniquely identify the tab. For example, Facebook, LinkedIn.

OK.
Will do that

Sir is this prototype somewhat like what you are thinking?

If we put this as a favicon, do you think it would distinguishingly identify CircuitVerse?

I think the favicon should be simple yet unique like the github octocat.

Sir is this prototype somewhat like what you are thinking?

If we put this as a favicon, do you think it would distinguishingly identify CircuitVerse?

Yeah, I guess it would be as the letters C and V would be more thick and would stand apart

I think the favicon should be simple yet unique like the github octocat.

Sir but Parikshit sir had intended to use letters in the favicon like facebook linkedin etc do

I think the favicon should be simple yet unique like the github octocat.

Sir but Parikshit sir had intended to use letters in the favicon like facebook linkedin etc do

Any updates on this would be helpful.

@sarthak77, Others and I have tried making smaller logo versions and favicons with "CV" as the design. However it doesn't look very good and when it is scaled down to the favicon size, it is downright ugly.
Your current design looks good, can we remove the part highlighted in red?

I am quite happy with the current logo, It is elegant, simple and circuitlike. Even when scaled down to favicon size, the details are visible. The major problem is it was not designed for CircuitVerse and was copied from another website, this was in the initial stages of the project and was fine. However, as we are establishing as an opensource community, we should have our own logo void of any copyright issues.

I did an image search and found the source:
https://www.pinclipart.com/pindetail/oRxoRo_circuit-icon-clipart/
One option is to find the designer and request licensing, another option is to have a similar logo designed by contributors.

Yeah I can remove it and improve upon so it does not look bad on down scaling
Thanks for the feedback

Sir its a rough prototype
Can you confirm it so that I start on it with appropriate tools

@sarthak77 don't address anyone as Sir, the new design looks good.
@satu0king I don't think we will be in any trouble cause both the designs are derives using the PCB circuits, most of the org working on electronics tend to have a similar design either with a complex schematic or PCB structure. Still I would like to know your thoughts.

@armudgal can you review the pr

@sarthak77 Hi, I am waiting for @satu0king approval on this.

Further conversation to be carried out on #203


docs out of date (still referencing old config style)

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
feature request
If this is a FEATURE REQUEST, please:

Describe in detail the feature/behavior/change you'd like to see.

Documentation (main README at least)  is out of date on chunks of this file. Several places are still referencing the old config style, example in the section about Environment Variable Expansion:
deployment:
  cluster: production-cluster
  keypair:
    -
      name: key
      publickeyFile:
      privatekeyFile:
      providerConfig:
        username:
        serviceAccount<EMAIL_ADDRESS>        serviceAccountKeyFile: "$MY_SERVICE_ACCOUNT_KEYFILE"

PR: https://github.com/samsung-cnct/kraken/pull/190


Change that -LogsOnly to Type = All, Data, Log

For Invoke-DbaDatabaseShrink and deprecate LogsOnly

@potatoqualitee, do you mind if I tackle this one?  I have a keen interest in this command.  I'm not sure if there is a process for assigning these?

yes please! that'd be fabulous. you'd assign yourself, let me add you to the sqlcollab group so that you can do that

You've invited Doug Meyers to SQL Server Community Collaborative ! They'll be receiving an email shortly. They can also visit https://github.com/sqlcollaborative to accept the invitation.

yesssssss


ci: adjust some actions

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #109 (c38db64) into master (29eb969) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #109   +/-   ##
=======================================
  Coverage   35.38%   35.38%
=======================================
  Files          69       69
  Lines       10998    10998
=======================================
  Hits         3892     3892
  Misses       6664     6664
  Partials      442      442

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 29eb969...c38db64. Read the comment docs.


ESP32-S2: The SPI MOSI idle level is not the same between 5.1.2 and 5.2-dev (IDFGH-11646)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hi,
I'm currently working on a WS2812b driver using the SPI peripheral.
When using the 5.1.2 version of the SDK, the idle level of the MOSI output is low, but when using the 5.2-dev version of the SDK, the idle level of the MOSI output is high. It is a bit problematic in my case, since I need the idle level of MOSI to be low, as it is used as the reset instruction by the WS2812B LED.
The idle level is low again with the 5.2-dev version using the following code:
esp_rom_gpio_connect_out_signal(gpio, spi_periph_signal[host].spid_out, true, false);

But then I need to also invert my output:
data[i] = ~data[i];

I got this trick from here: https://esp32.com/viewtopic.php?t=22496
Here is my initialization code of the SPI peripheral:
struct spi_device_handle_t handle;
struct spi_bus_config_t busConfig;
std::memset(&busConfig, 0, sizeof(spi_bus_config_t));
busConfig.miso_io_num = -1;
busConfig.mosi_io_num = GPIO_NUM_39;
busConfig.sclk_io_num = -1;
busConfig.quadwp_io_num = -1;
busConfig.quadhd_io_num = -1;
busConfig.max_transfer_sz = 18000;

struct spi_device_interface_config_t devConfig;
std::memset(&devConfig, 0, sizeof(spi_device_interface_config_t));
devConfig.mode = 0;
devConfig.clock_speed_hz = 2500000;
devConfig.queue_size = 1;
devConfig.spics_io_num = -1;
devConfig.post_cb = cb

spi_bus_initialize(SPI3_HOST, &busConfig, SPI_DMA_CH_AUTO);
spi_bus_add_device(SPI3_HOST, &devConfig, &handle);

// the call resolving the problem for 5.2-dev
esp_rom_gpio_connect_out_signal(GPIO_NUM_39, spi_periph_signal[SPI3_HOST].spid_out, true, false);

This call seems a bit hacky, and I'm not sure this is the right solution.
What can I do to tackle this behaviour?
Thanks!

8f5851d0646cf7fbccc66929b2e307f0e4b438e3
I see the MOSI IDLE level was set to low level already.

On the 5.2-dev version of this file (here), this line does not exist. What is the difference between the branch release/5.2 and the tag 5.2-dev?

@mcaralp "5.2-dev" is not a release tag at all. It's just a tag that tells the start point where we're kicking off esp-idf/v5.2.
BTW, v5.2-beta1 release will be on Github soon. Before a 5.2 release tag was pushed to the Github, I suggest you using the release/v5.2 branch.

I'm currently working on a WS2812b driver using the SPI peripheral.

BTW, have you tried https://components.espressif.com/components/espressif/led_strip component? It supports both RMT and SPI as its backend controller

@mcaralp "5.2-dev" is not a release tag at all. It's just a tag that tells the start point where we're kicking off esp-idf/v5.2.
BTW, v5.2-beta1 release will be on Github soon. Before a 5.2 release tag was pushed to the Github, I suggest you using the release/v5.2 branch.

Ok, thanks for the info. I thought it was a release since there is a page for it (https://github.com/espressif/esp-idf/releases/tag/v5.2-dev). It's more clear now.

BTW, have you tried https://components.espressif.com/components/espressif/led_strip component? It supports both RMT and SPI as its backend controller

I didn't know ESP registry, it's  a nice tool! I will check the ledstrip component, thanks!


Update diffusers to support InstructPix2Pix pipeline

https://huggingface.co/docs/diffusers/main/en/api/pipelines/stable_diffusion/pix2pix

Thanks for the PR, @Hades32!
I've also had my eye on this one for a while `:)  So thanks for this.
I'll be merging a lot of stuff into dev later this week (everything from the split branch, to finally move forward with "v1"), and will rebase this into that straight afterwards.  I'll keep you posted!
Thanks again :)

Ok I moved up my timetable a bit :sweat_smile:
Merged everything to dev :muscle:
The resulting merge conflict from your PR wasn't terrible but I wanted to update to the latest diffusers commit hash too, so unfortunately I won't get to merge this... sorry about that... but you really did succeed in nudging me to move things forward :raised_hands:
Just note that everything I merged today bumps the dev branch from v0 to v1 (not officially released yet, but getting there).  There is a dev-v0-final branch pointing to the last dev commit of v0 if relevant.  Otherwise see Upgrading from v0 to v1.
For latest diffusers (in my commit https://github.com/kiri-art/docker-diffusers-api/commit/87632aa2c32faddfeb049fe969884b568066edd3), you can either build from source in dev or simply use the docker image with dev tag, i.e. docker pull gadicc/diffusers-api:dev.  On next merge to main, it will get tagged with appropriate semantic version too.
Hope that's all clear, not sure if you're ready to try out v1 yet since it's not officially released... but I'm available for anything you need.  Thanks for  your continued support! :pray:

Haha, it was just a single line, but very happy to help give some extra motivation :D
Thanks - as always - for your great work! I'll let you know when I stumble over any issues

🙏


Cannot read property 'on' of undefined

When I run the tests, what was by ember generated. I get the
cannot read property 'on' of undefined error.
How I find this is in the ember-simple-auth/mixins/application-route-mixin at line 89:
_this.get('session').on(event, _ember['default'].run.bind(_this, function () {      _this[method].apply(_this, _arguments);  }));
In that code the _this.get('session') is undefined, but _this.session is the injected service.
Why and how can I solve this?

Please post the full source of your test here. Also which versions of Ember.js, Ember Data, Ember Simple Auth and Ember CLI are you using?

Please reopen if the problem persists but then also provide the info as listed above.

I encountered the same problem, is there any solution for it?

Please post the full source of your test here. Also which versions of Ember.js, Ember Data, Ember Simple Auth and Ember CLI are you using?

I encountered too.

Ember 2.7.1
Ember DATA 2.7.0
Ember Simple Auth 1.1.0
Ember CLI 2.7.0

I also use ember-cli-auth-token 1.1.1.
The test code is just auto generated by ember-cli.
import { moduleFor, test } from 'ember-qunit';

moduleFor('route:application', 'Unit | Route | application', {
  needs: ['service:session']
});

test('it exists', function(assert) {
  let route = this.subject();
  assert.ok(route);
});


[8.x] Feature: Environment conditional routes

We all have come to situation that we want some routes to work locally and some on production and some on both. I have added this little feature to determine a route to be working locally, on production or on both.
This route will be working locally only
Route::group(['env' => 'local'], function(){
    Route::get("/will-work-locally-only", function(){ return "local";})
});

This route will be working on production only
Route::group(['env' => 'prod'], function(){
    Route::get("/will-work-on-prod-only", function(){ return "production";})
});

This route will be working on both local and production
// set env to null or don't define env
Route::group(['env' =>  null], function(){
    Route::get("/will-work-on-both", function(){ return "production and local";})
});

You could just do this:
if (app()->environment('local')) {
    Route::get("/will-work-locally-only", function(){ return "local";})
}

See above.


Resize is broken for grids with preventCollision: true, fixes #655

See issue #655 for example and description

Great work @betalb!
@STRML Could you please take a look at this PR?


Allow executing multiple timelines at once

Right now all the timeline execution is done by the dialogic game handler. While for many games one timeline at a time is the most logical thing to do, sometimes executing multiple timelines might be desired.
This would likely mean the following:

move much of the timeline execution into a new "TimelineHandler" class
make the dgh instance this class when you do start()

There is a couple of problems though:

Information is currently universal (e.g. current_state_info). Even if each timeline handler had it's own state info, the subsystems and events would have to know which one to use.
If you run multiple timelines at once, you likely want them to be displayed at the same time as well. However currently the DialogicNodes and Subsystems have no way of doing that. This could be solved by giving each dialogic node a layer (like visibility or shadows) and then give each timelinehandler a layer as well, so it will only target dialogic nodes of that layer. But not sure if that is the best way to do it or overkill.

Need to think about this more. Also I don't really know how needed this is, as I can only imagine some usecases for this. What could be way more useful is to allow stopping a timeline at any point and resuming the timeline later from there (like if you go away from a conversation and go to another one but later come back).
Just a big rant that serves as a note for later ;).

Bumping this issue following a thread on the discord.
My use case for this feature is implement both voice and textbubble "barks" (e.g. in Unity) as Dialogic timelines. For example, in an RPG, as you walk through a town, various NPCs will perform short dialogue lines as you walk by them. These dialogue lines should be able to be performed concurrently. If an NPC was in the middle of delivering a bark and the player walked up and interacted with them, they would finish the bark line before moving onto the rest of the dialogue. If the player decides to leave an interactive dialogue and regain movement control, the NPC wouldn't just cut off, but finish the dialogue line while the player moves away.


Improved docs for infer_datetime_format

Fixes #12152

pls add to doc-string of pd.to_datetime as well.

Thanks - I think I've sorted all of those now.

maybe show an example where this helps (in Examples) mainly for pd.to_datetime

thanks!


Kconfig: CONFIG_DEBUG_INFO misleading/broken

Is your enhancement proposal related to a problem? Please describe.
The Kconfig option CONFIG_DEBUG_INFO under zephyr/subsys/debug is at least misleading and seems to be broken. Going by the commit history of this option's definition, it seems that the option affects x86 only, however there is nothing in the option description to indicate this and the option is not defined in an x86-specific location. I would have expected from the option's name and description that it affected the inclusion of debug symbols (-g compiler flag) but this is not the case. Furthermore, I can't see anywhere that this option is actually used. Perhaps its use has been removed but the option definition remained by mistake?
Describe the solution you'd like

CONFIG_DEBUG_INFO option definition moved to architecture-specific location and name/description clarified if this option is still used.
CONFIG_DEBUG_INFO option deprecated if no longer used.

Perhaps a Kconfig option that does control the inclusion of debug symbols (-g compiler flag) would also be useful. That should probably be done as a separate issue if such an option is desirable.
Describe alternatives you've considered
Leave the option as it is in its currently misleading state.

Thanks for the quick reply, but I don't think that's true. CONFIG_DEBUG_OPTIMIZATIONS sets the -Og flag, as it's one of the optimization options, but it does not set the -g flag, which controls the inclusion of debug symbols and is somewhat unrelated to optimization. It seems that the -g flag is set regardless of optimization level (CONFIG_*_OPTIMIZATIONS).
Besides this, my points about CONFIG_DEBUG_INFO still stand.

Looks like I missed something when I created this issue and said

I can't see anywhere that this option is actually used.

It seems it is in fact used. However, I stand by my point that it affects x86 only and that the name/description are a bit misleading, making it sound like a much more generic option if you don't know otherwise.


Appease clippy

Note: The first commit is essentially the same as #565 as warnings are what initially spurred me to attempt the iterator simplification PR. If this PR goes in, I'll rebase #565 and remove its first commit.


How do I set up a network proxy?

Due to the inability of the local computer to connect to the API, I need to use a SOCKS5 proxy. Otherwise, the software will freeze and not function properly.

You can join our discord for this problem. https://discord.com/invite/bn4t2Jy6TT


Ensure flow type checking passes

@GreenGremlin @blainekasten This fixes the regression introduced in FormidableLabs/enzyme-matchers#306
Fix getDisplayName() function to match its type signature.
Add yarn run typecheck step to travis gating and rework.
Fix lint errors using eslint-import-resolver-alias and new config.
Update flow-bin to latest version.
Tweak .flowconfig so all imports are recognised.
Fixes FormidableLabs/enzyme-matchers#309

@blainekasten Have you got some time to review this? Would be good to get the gating working as intended.

@bz2 This is brilliant!
Thank you so much. I'm sorry I missed this! I'll get a release out

@blainekasten Thanks! Have pulled the new release and removed the workaround, all is fixed.


Lucas Number in C++

💥 Proposal
Lucas Number in C++
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


Updated project

Nancy, Nancy.Authentication.Forms and Nancy.Testing were missing from the *.Next.sln, causing it to fail to build
Bumped Newtonsoft.Json dependency to 8.0.3

Can you also update the nuspec?


Add wrapper for pubtools-marketplacesvm

This commit introduces a wrapper for pubtools-marketplacesvm which aims to be eventually used to support VM images pushes on various cloudmarketplaces (such as AWS and Azure).
With this wrapper it will be possible to easily integrate the tooling into Tekton CI using the Cloud Staged structure as input.
Refers to SPSTRAT-464

I defer to @release-service-maintainers on this.

You're failing flake8 now for some lines being too long. You are also failing gitlint for your commit title. Maybe something like feat(SPSTRAT-464): add wrapper for pubtools-marketplacesvm instead? Otherwise lgtm

@johnbieren I've fixed most of the issues with flake except the URL which I've added a # noqa since it can't be shortened.
For the commit name I was a bit afraid it woul bee too long to put what you suggested (and fail again on gitlinkt) then I've shortened pubtools-marketplacesvm just to marketplacesvm.. Hope it works for you

Failing gitlint now for 1: CC1 Body does not contain a 'Signed-off-by' line git commit -s should do that for you. Python checks pass now, great :slightly_smiling_face:
Can you fix the gitlint thing and rebase? Then I will trigger the e2e and approve. If it looks good to Martin in his morning he can merge

Done rebasing and signing it off.
@mmalina please check everything and merge if it's ok to you 😄

/ok-to-test

@mmalina I noticed some small nitpiks while writing the Task:

The starmap dir should be inside the staged dir, it's not part of its base directory, thus I'll remove it from here and validate it in the task.
There's one missing argument for our tooling, called --offline which is meant to prevent the tooling to invoke the StArMap server when the --repo-file is set.

I'll create an additional commit with such fixes, then you may squash them on merging if you prefer 😄
Just a minute, working on that

@johnbieren @mmalina I've added the necessary fixes on bdb3b59c01fbed3bde9a3bd5ac09f7ac4c72040c
Feel free to squash it when merging and please re-run the checks + approval if ok to you
Sorry for the inconvenience

@JAVGan sorry for the delay. Can you rebase and squash the commits yourself? The second one fails gitlint now because of capital letter after feat:. It looks good to me now. But Let's wait for Johnny as well (should be here when US wakes up).

/ok-to-test


machine learning?

Does this perform any machine learning / artificial neural network function? Prior README commits show it definitely did this at some point, current iteration doesn't directly mention this being a feature. I've looked through the code and admittedly am not great with node, but it appears that the trade logic is completely hard coded and the zenbrain library, map and reduce, simply pulls in data for one to manually run the simulator.
Is my understanding correct and if so does anyone know why this changed?

I think you are correct and that carlos8f took the functionality out
temporarily.  But then he mentioned being in jail and hasn't been seen for
a while.
On Sat, Mar 25, 2017 at 6:16 PM Ben Van Doren<EMAIL_ADDRESS>wrote:

Does this perform any machine learning / artificial neural network
function? Prior README commits show it definitely did this at some point,
current iteration doesn't directly mention this being a feature. I've
looked through the code and admittedly am not great with node, but it
appears that the trade logic is completely hard coded and the zenbrain
library, map and reduce, simply pulls in data for one to manually run the
simulator.
Is my understanding correct and if so does anyone know why this changed?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/carlos8f/zenbot/issues/128, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACU0beXOkS8Tszp6BZYIRKpjW_Kb7sCOks5rpZJZgaJpZM4MpPFu
.

Indeed the last trace of activity I can find of him is traced back in September.
However the bot is definitely branded as IA driven so I'm not sure. (I don't use it, just curious about the project).

Can I use 2.0 version with machine learning?

Originally the idea for zenbot was to exclusively use machine learning, but the neural network part never panned out. As far as I have found, there really isn't a practical pure neural network model for trading, at least unless you want to pull in the price of gold and speculate on a thousand other variables. So in that way I made a mistake on the branding of it :) The machine learning code was pulled because I moved from a volume-sensing strategy to an RSI-based one, and the volume strategy required constant machine learning to run. With RSI being a simple bounded indicator, one-size-fits-all config replaced the learned variables.
Default strategy in 4.x will (tentatively) require at least one unbounded variable, and I may write a "tuner" that will find some optimal configuration, but the tuner won't have to run constantly for trading to work, one might run it every few months or so.

You are thinking about role of machine learning in this case as analog for similar functionality to find the best parameters for strategy in Meta Trader?

How about something especially now?
Could we analyze all history, find fractals and patterns and try to use only them without realtime indicators?

@7flash I suppose so, although I haven't used Meta Trader. Machine learning would automate the process of mutation of params, and looping of sims. Not a neural network, but a trainer.

Could we analyze all history, find fractals and patterns and try to use only them without realtime indicators?

See above, I don't know of a practical way to do this. Neural networks can be built, but require a huge amount of data and is far beyond the scope of what I'm doing. And in theory, some basic technical analysis should perform as good as a fancy neural network, and require way less resources.


feat: port AlgebraicTopology.DoldKan.Projections

[ ] depends on: #3526

This PR/issue depends on:

leanprover-community/mathlib4#3526
By Dependent Issues (🤖). Happy coding!

bors d+
Please merge once dependent PRs have been merged.

This PR/issue depends on:

leanprover-community/mathlib4#3526
By Dependent Issues (🤖). Happy coding!

Thanks @semorrison for these reviews!
bors r+


Simplify ChangeConstants

Since BASEX is a constant and radix is a uint32_t radix, there is nothing to be gained from having the limit be 64 bit.
In addition, the g_ratio += !g_ratio only applies if g_ratio is 0, so I simplified that too.

Another way to simplify this would be to use a closed-form expression: g_ratio = static_cast<int32_t>(ceil(log2(BASEX) / log2(radix))) - 1;. But I think this change is OK as-is.


[HW]  Hardik Talati

Name:  Hardik Talati
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/hardik-talati-49b73b192/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:

Hey Hardik Talati, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/oalcTOWARWCXfEHj1ZXDSQ


Add new Issuer and Principal abstractions

Summary
Adds identity package with Issuer and Principal abstractions as well as an IssuerPool implementation that matches a token against a pool is issuers to authenticate. This work is part of a larger refactor of our identity issuer code as described here.
Future work includes:

[ ] Reimplementing our existing issuer abstractions to fit this interface including
[ ] Changing our server to use the issuer pool instead of the current implementation
[ ] Removing all the dead code once the new abstraction is in place

Ticket Link
Partially addresses #275 along with the other future works describe above
Release Note
NONE

Codecov Report

Merging #558 (b69f357) into main (05aa4bb) will increase coverage by 1.16%.
The diff coverage is 100.00%.

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

@@            Coverage Diff             @@
##             main     #558      +/-   ##
==========================================
+ Coverage   35.68%   36.84%   +1.16%
==========================================
  Files          18       19       +1
  Lines        1415     1441      +26
==========================================
+ Hits          505      531      +26
  Misses        851      851
  Partials       59       59

Impacted Files
Coverage Δ

pkg/identity/issuerpool.go
100.00% <100.00%> (ø)

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 05aa4bb...3bd5424. Read the comment docs.

Huh I can't merge. Maybe because I rerequested Dan's review? I don't have permissions to dismiss that unfortunately

Merged, it needs write permissions.


Prepare and construct index from col metadata for skipping pages at reading

Which issue does this PR close?
Closes #1761 .
Rationale for this change
Get this info in memory then we can apply page-level filter in future.
What changes are included in this PR?
Add an option to read page index in parquet/src/file/serialized_reader.rs.
Are there any user-facing changes?
In  parquet-testing only data_index_bloom_encoding_stats.parquet has one rowgroup with pageIndex.
I will generate test file base on alltypes_plain.parquet (this file  not contains any pageindex) in repo parquet-testing, and support multi-RG in Next Pr.
 parquet-tools column-index ./alltypes_plain.parquet
row group 0:
column index for column id:
NONE
offset index for column id:
NONE
.
.
.
column index for column timestamp_col:
NONE
offset index for column timestamp_col:

Codecov Report

Merging #1762 (3468a21) into master (722fcfc) will increase coverage by 0.07%.
The diff coverage is 51.91%.

@@            Coverage Diff             @@
##           master    #1762      +/-   ##
==========================================
+ Coverage   83.27%   83.35%   +0.07%
==========================================
  Files         195      198       +3
  Lines       55896    56156     +260
==========================================
+ Hits        46549    46808     +259
- Misses       9347     9348       +1

Impacted Files
Coverage Δ

parquet/src/basic.rs
91.49% <ø> (ø)

parquet/src/data_type.rs
75.84% <ø> (ø)

parquet/src/file/page_index/index.rs
16.96% <16.96%> (ø)

parquet/src/file/page_index/index_reader.rs
79.16% <79.16%> (ø)

parquet/src/file/serialized_reader.rs
94.98% <88.63%> (-0.67%)
:arrow_down:

parquet/src/file/metadata.rs
95.15% <100.00%> (+0.72%)
:arrow_up:

arrow/src/array/array_struct.rs
88.40% <0.00%> (-0.05%)
:arrow_down:

...arquet/src/arrow/array_reader/dictionary_buffer.rs
94.44% <0.00%> (-0.04%)
:arrow_down:

parquet/src/arrow/array_reader/offset_buffer.rs
94.79% <0.00%> (-0.03%)
:arrow_down:

arrow/src/array/array_binary.rs
93.27% <0.00%> (-0.03%)
:arrow_down:

... and 14 more

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 2ba1ef4...3468a21. Read the comment docs.

@alamb @tustvold plz have a review

Awesome, will review tomorrow 😀

I intend to merge this once CI clears

🎉


Can mathjax expression load without MathJax.js ?

Hi ,
We want to load following mathjax html string only with mathjax css styling.
Is there anyway to load without following snippet in our html
""
We dont want to load js script since it would take some rendering time in our client native apps(iOS/Android).
Please let us know if there any solution for native mobile apps to load MathjAx in WebView.

I'm sorry, but I'm confused by your question.  You say

We want to load following mathjax html string

but the only strong that I see is the <script> tag that loads the MathJax.js itself, which you also say you don't want to do.  So I'm confused about what you want.
If you want to preprocess your page using MathJax so that it doesn't have to run in the browser, you might consider mathjax-node, or more specifically, the mathjax-node-page wrapper for it.
You may also want to consider MathJax version 3, which is currently in beta with a final release being prepared.  There are examples of how to run MathJax via the command line on individual expressions or on an entire page in the MathJax 3 node demos repository.  MathJax 3 renders considerably faster than MathJax 2, so perhaps using it in the browser would be acceptable for you.  There are examples in the MathJax 3 browser demos repository.


After removing inheritance, _type field still being set

I'm not sure if this is a bug, but I am stumped with the following issue related to inheritance:

I used to have a class Affiliate that inherited from AdminUser
I removed the AdminUser class so that Affiliate no longer inherits from any classes
I updated all of the _type fields for documents in the Affiliate class to nil
Upon calling Affiliate.new, the _type field is still being set to "Affiliate", yet Affiliate no longer inherits from any class and has no descendants.

I'd appreciate if anybody had any ideas on why this is happening.

I cannot duplicate, and no activity so closing.


[1pt] Create app and install basic dependencies - Setup (group task)

Create React App.
Install React Redux, Redux Logger and React Router.

It's Done, Branch setup-project. https://github.com/mrcbq/space-travelers-capstone/tree/setup-project


Greenkeeper/rollup 2.12.1

Hi! Like Mercury reader. Have some issues:
Doesn't open links in reader mode. Example links in this article when use Mercury reader:
https://www.makeuseof.com/tag/stop-switching-todo-apps/
Need text size between middle and big.

@vovation can you please submit an issue here for the problem you are facing?
https://github.com/postlight/mercury-parser/issues


[MINOR][SQL] Improve examples for percentile_approx()

What changes were proposed in this pull request?
In the PR, I propose to replace current examples for percentile_approx() with only one input value by example with multiple values in the input column.
Why are the changes needed?
Current examples are pretty trivial, and don't demonstrate function's behaviour on a sequence of values.
Does this PR introduce any user-facing change?
No
How was this patch tested?

by running ExpressionInfoSuite
./dev/scalastyle

Test build #128990 has started for PR 29841 at commit e513ab5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33612/
Test PASSed.

Test build #128994 has started for PR 29841 at commit d236c7e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33616/
Test PASSed.

Test build #128990 has finished for PR 29841 at commit e513ab5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128990/
Test FAILed.

Merged to master.

Max, I think it might be better to backport to branch-2.4 and 3.0 as well.

Test build #128994 has finished for PR 29841 at commit d236c7e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128994/
Test PASSed.

Here are PRs for:

2.4 https://github.com/apache/spark/pull/29849
3.0 https://github.com/apache/spark/pull/29848


Python: Append comment: missing extra space before #

When writing python code <leader>cA only adds one space after the code:
<some code> # <comment starts here>
However, according to PEP 8  at least 2 spaces are required
<some code>  # <comment starts here>
Please note that this is not the same as #104 which can be solved by let g:NERDSpaceDelims = 1

See also #240. Something is wonky here. Are you sure you're still not getting a space before the delimiter? A lot has changed in the code since this report and I'd like to hear an update on whether it is working as expected or not. Thanks!

I can't reproduce the OP behavior.
<leader>cA correctly appends two ' ', one '#' and one ' ' at the end of the line.
As expected, let g:NERDSpaceDelims = 1 is mandatory in order to get the last space.

Is this problem solved?


Fix keycloak ingress

On deploying CNOE, the keycloak ingress does not work properly. To fix the issue:

Set KC_HOSTNAME to the subdomain for keycloak
Port 8081 is not exposed by the keycloak service. Switch to port 8080
Set path / for keycloak ingress to match a Prefix, otherwise path based routing doesn't work.

These changes are tested working in my EKS environment.
I'm still trying to sort out how all the templating works, so please check my work on whether the value for KC_HOSTNAME is set correctly

Closing this due to inactivity. Please feel free to re-open.


feat: Adding fr compatibility to smt variables api

This pr updates FFTerm and FFITerm classes. Also CMakeLists are changed a little.
FFTerm

Added Fr compatibility. Now you can perform arithmetic operations with symbolic variables using bb::fr rather than strings or FFTerm::Const

FFITerm

Added .mod() method, that performs the operation of taking the remainder on terms. It leads to less constrained system of equations and helps the solver to work faster. However, now you have to do it manually.
Added Fr compatiblitity(same here)
Added xor, $\ge, \gt, \le, \lt$ operations
Changed Constant variable initialization a little

Since you need these classes to behave similarly, they now both have identical methods. However, most of them are just placeholders in FFTerm class.
Tests
Added simple unit tests for both  FFTerm and FFIterm to test that all the arithmetic operations are implemented properly.
updt
Terms

changed division method to be more readable

changed the hash value that is used as a dvision symbolic variable
name to md5(Aztec)

Removed the mod operations from inequalities methods in FFITerm

Modified == and != methods in FFITerm so now there won't be any
redundant mod operations

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit e899e56e and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,700
18,884
36,452

l1_rollup_calldata_gas
66,108
238,904
469,712

l1_rollup_execution_gas
194,056
500,210
908,970

l2_block_processing_time_in_ms
1,200 (-1%)
4,512
8,997

note_successful_decrypting_time_in_ms
205 (+1%)
554 (-1%)
1,013 (-3%)

note_trial_decrypting_time_in_ms
9.39 (-12%)
72.9 (-12%)
50.9 (-7%)

l2_block_building_time_in_ms
16,383 (+1%)
64,752
130,045

l2_block_rollup_simulation_time_in_ms
12,446 (+1%)
49,217
98,941

l2_block_public_tx_process_time_in_ms
3,907 (+1%)
15,465
30,939

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,353
26,951 (+1%)

note_history_successful_decrypting_time_in_ms
1,254
2,313 (-4%)

note_history_trial_decrypting_time_in_ms
106 (+17%)
111 (-20%)

node_database_size_in_bytes
18,767,952
35,590,224

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
252
44,736
28,001

private-kernel-ordering
193
52,625
14,627

base-rollup
1,303 (-1%)
177,932
933

root-rollup
70.1
4,192
825

private-kernel-inner
318 (-1%)
73,715
28,001

public-kernel-app-logic
193 (-2%)
32,254
25,379

merge-rollup
5.70 (-2%)
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts

tx_size_in_bytes
19,179

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
2,582 (-2%)
1,372

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.0330 (-1%)
472 (-1%)


[Feature Request] Swipe through episode details.

As a user, I would find it useful to, once in the details screen for a TV Show episode, swipe left or right to navigate to other episode details for lower and higher episode numbers without having to back out of the episode details screen to the TV show seasons screen and enter the next lowest or highest episode.

:+1:


Top toolbar now has dramatically reduced clickability.

Issue Type: Bug
This latest update has shrunk the clickable area of the top toolbar so that it now no longer touches the top of the screen. This means if you press your mouse to the top of the screen you cannot click on the 'file', 'edit', 'selection', etc... buttons.
For more information on why edge contact is important "Fitt's Law" is the term to search. Buttons that touch the edge of the screen require much less precision to click than one pulled even one pixel away because of the ability to press your cursor to the edge of the screen.
Sorry if this is the wrong category, i'm not sure where "UI Regression" is supposed to go.
 We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

Working on a fix #151250

Fixed via #151250. The hit area of the button will now fill the entire height of the title bar even with the new styling.


I have removed the restriction to getting all the posts because Ben, …

…who is part of my yeam, asked me so he can access them

Thank you Petar @PressureandTime


⚠ Adacta-fintech.com has degraded performance

In 998d82a, Adacta-fintech.com (https://www.adacta-fintech.com) experienced degraded performance:

HTTP code: 502
Response time: 747 ms

Resolved: Adacta-fintech.com performance has improved in 386898f after 7 minutes.


feat(compiler): validate directive locations + undefined directives

partial of #246
This looks over each directive usage and checks whether it's in a correct location as specified by its original DirectiveDefinition. In case of incorrect usage UnsupportedLocation error is returned. Since we are already looking for the original DirectiveDefinition, this PR also adds a UndefinedDefinition error for undefined directives in case that DirectiveDefinition is not found.
While working on this validation rule, I realised we need to tidy up our ValidationDatabase before we implement any follow up rules. Primarily we are using two different paths for validating: check_* and validate_*, which should be unified in some way.
We are also creating a lot of let diagnostics = Vec::new() for pretty much each validation step, and it would be good to not keep allocating new vecs for each node as we add diagnostics.

sorry for 5000+ line additions! i swear most of these are tests 😅

We are also creating a lot of let diagnostics = Vec::new() for pretty much each validation step, and it would be good to not keep allocating new vecs for each node as we add diagnostics.

I think in this tree-walk design, it's almost certainly going to be faster with standalone functions that can borrow hir nodes and push into one shared diagnostics vec, even if it has to rerun the entire validation any time something changes. (compared to having the many cacheable queries with owned values everywhere). maaaybe it wouldn't make as much of a difference if we wrap everything in Arcs but i don't really have a good idea of how expensive atomic operations are.
this looks good tho 😄

yea definitely something for us to think about once we simplify this validation a little bit!


port async advancements patch to purpur

Just a note: I'm such a noob at this, so if this looks super bad or something, tell me.
I ported the async advancements patch from https://github.com/PaperMC/Paper/pull/4160 to Purpur.
But it doesn't seem to build and I can't figure it out. The error always is:
Caused by: org.eclipse.aether.transfer.ArtifactNotFoundException: Failure to find co.aikar:cleaner:jar:1.0-SNAPSHOT in https://hub.spigotmc.org/nexus/content/groups/public/ was cached in the local repository, resolution will not be reattempted until the update interval of spigotmc-public has elapsed or updates are forced

I can't seem to figure out how to fix that, so any help would be appreciated!

I've now fixed the issue I was having.

I did some testing w/o any issues.

Wow, is this a merger?

Wow, is this a merger?

No. The referenced PR in that commit is from Pufferfish.


closes #2 hex logo and various other updates from default admiraltemplate

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

[ ] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[ ] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[ ] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[ ] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[ ] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[ ] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ ] Address any updates needed for vignettes and/or templates
[ ] Update NEWS.md under the header # admiralpeds (development version) if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[ ] Build admiralpeds site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the Reference page.
[ ] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[ ] Link the issue in the Development Section on the right hand side.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done! Much love to your accomplishment!

@bms63 @manciniedoardo a bunch more minor updates from the admiraltemplate defaults. let me know if you spot any others. FYI i did create https://github.com/pharmaverse/admiralpeds/issues/7 which catches some more needed for later once people dive into this.


update deps to depend on major version only

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

Comparison is base (e307fd2) 75.80% compared to head (5d82bf8) 75.22%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   75.80%   75.22%   -0.59%
==========================================
  Files          10       10
  Lines        1021     1021
==========================================
- Hits          774      768       -6
- Misses        247      253       +6

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


Enhance buildah task to support RHEL Entitlement

Issue : https://issues.redhat.com/browse/SRVKP-4045

/retest

/retest

/approve
/lgtm


Improve Hash2Fragment by using a map to validate allowed sequence characters

Changes in this PR

Improves Hash2Fragment by using a map for more efficient validation of allowed sequence characters, discussion here.

Why are you making these changes?
General improvement.
Are any changes breaking? (IMPORTANT)
No
Pre-merge checklist
All of these must be satisfied before this PR is considered
ready for merging. Mergeable PRs will be prioritized for review.

[ ] New packages/exported functions have docstrings.
[x] New/changed functionality is thoroughly tested.
[ ] New/changed functionality has a function giving an example of its usage in the associated test file. See primers/primers_test.go for what this might look like.
[ ] Changes are documented in CHANGELOG.md in the [Unreleased] section.
[x] All code is properly formatted and linted.
[x] The PR template is filled out.

I saw the small alphabet size in the PR and was curious what that threshold might be when a map performs better list for another issue I'm working on.
I created some benchmarks just to see what would happen with sequences with different possible alphabets of varying length.  I wonder if caching starts to come into play with all the different data involved. Including the benchmark code and cpu profile in the comment.

cpu profile

Also, Contains seems to be more performant in the Small/Complete alphabet with no mistakes case, which seems like it would be the most common case.

Interesting! @matiasinsaurralde Do you have any thoughts on this? The use case of no-mistakes is definitely the most common by an order of magnitude or more

@Koeng101 Agree with that, feel free to discard/ignore my suggestion


Fix: the main page responsiveness #126

Fix: the main page responsiveness #126
closes issue no - #126
now the page will be responsive even for smaller displays.

@Rakshit-gen hello, hmm you have merged without labels. please resolve this./

Done @AE-Hertz

@Rakshit-gen thank you


d3-format fixed incorrectly rounds some numbers

When rounding positive and negative numbers, Math.round behaves according to mathematical convention:
> [40.5, -40.5].map(Math.round)
< (2) [41, -40]

However, this is not how d3.format behaves:
> [40.5, -40.5].map(d3.format('.0f'))
< ["41", "-41"]

Obviously, one can just use Math.round here, but if you're rounding to tenths or hundredths, using Math.round(n * 10 ** p) / 10 ** p) can introduce the same IEEE 754 rounding errors that d3-format is meant to fix. (Number.toFixed(0) produces the same result as d3, and it's unlikely JavaScript interpreters will fix that error.)
Negative numbers aren't the only place where d3.format has a problem with ambigious rounding: again following Number.toFixed, it will round up or down ambigious cases somewhat arbitrarily:
> [2.05, 2.15, 2.25, 2.35, 2.45, 2.55, 2.65, 2.75, 2.85, 2.95, 3.05].map(d3.format('.1f'))
< ["2.0", "2.1", "2.3", "2.4", "2.5", "2.5", "2.6", "2.8", "2.9", "3.0", "3.0"]

Sorry, no, I hadn't. I'd originally thought this was just to do with negative numbers so I missed it in my search.
Negative numbers do seem like they're a separate issue. 40.5 and -40.5 should both be exactly representable in binary, so I don't understand why d3 (and toFixed) should round -40.5 (or any other negative integer-and-a-half) down instead of up.

I’ve confirmed that d3-format’s behavior is consistent with number.toFixed, which isn’t too surprising because that method is the basis of the f format type:
https://observablehq.com/d/7957bdfa735d20f0
And I took another look at the specification for number.toFixed, and while I don’t have a smoking gun to explain the behavior here, I expect the reason for this behavior is in 9a where number.toFixed is intended to be symmetric around zero (unlike Math.round, which rounds 0.5 up):
https://tc39.es/ecma262/#sec-number.prototype.tofixed


proposal: x/crypto/blake2b: add BLAKE2X (BLAKE2Xs and BLAKE2Xb)

BLAKE2X is a construction based on BLAKE2 to produce arbitrary hash values (up to 2^16 -1 bytes for BLAKE2s and 2^32 -1 bytes for BLAKE2b) similar to the ShakeHash interface of the SHA-3 package.
Such long hash values can be used within signature schemes - for example, Ed521 signatures need a 1056-bit hash.
For a detailed description of BLAKE2X see this paper.

/cc @agl


new link for documentation

in the documentation example 'hello world' you linked to the SplObjectStorage class but the link is not working anymore. Here is the new link: https://www.php.net/manual/en/class.splobjectstorage.php if you want to update it :)
Edit: Incorporated clues comment

PRs welcome! :+1:
(nit: should probably link to English documentation here: https://www.php.net/manual/en/class.splobjectstorage.php)

Someone already did a request, nearly 2 years ago ;)
https://github.com/ratchetphp/socketo.me/pull/44


Update configure-ssl-web-site.md

Adding a note explaining the behavior when having both an IP Based SSL binding and an SNI binding with a different certificate for the same Web App.
In line 458 adding:
> [AZURE.NOTE] If you add an IP based SSL to a Web App that already had an SNI binding with a different certificate, as soon as IP SSL is enabled for the Web App, we'll remap the site's hostname to that IP address, so if any other hostname is CNAME'd to that site's hostname it will also get traffic on IP SSL address.
For such cases we created one more DNS entry: sni.<nameofyourWebApp>.azurewebsites.net where <nameofyourWebApp> is the name of your Azure App Service Web App. So, you should change your DNS records pointing to the name used in your SNI binding, so that it points to sni.<nameofyourWebApp>.azurewebsites.net instead.

Hi @cristhianu, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@cephalin - Please review this proposed change from @cristhianu in the public repository.

@cristhianu,
Thanks a bunch for your contribution! Can you do us a favor? Right now the XML tags in your note (like <...>) are not rendering properly. Can you change all instances of the < character to < ? That will make your note show up properly in the rendered article.
Great addition, by the way!

@cephalin
Done. Does it look good now?

@cristhianu I don't see the change in your fork yet.

Done, now. Can you see it?

@cristhianu  Awesome! Thanks!
@PRmerger Go ahead and merge.

@tysonn This is ready to merge.


Allow disabling of Scale handles in 2D editor

I would like the ability to disable the scale handles on 2D nodes, like Sprite for example. My team does not use them, as we prefer absolute and precise values over those the handles return, yet occasionally miss click on a handle when selecting other nodes, causing a tiny amount of scaling to occur that can be hard to catch until you dig into the inspector to find out why things no longer align. This also goes for the extends handles for collision shapes.

CC @groud

Closing out of lack of interest.


fix "Continue Reading..."

Hi, I found a bug in the blog display and fixed it.
Before this fix, "Continue Reading..." was displayed when the page.summary and page.content were equal.  The correct logic is the other way around, right?
Then I found a function safe_truncate_html that cuts down the number of characters without destroying the html tags, so I changed it to that. Twig Filters & Functions | Grav Documentation
Thanks for a very good theme! I love the simplicity of it.

I'm sorry, I must have been mistaken.
The problem was in the wrong place.  I'll check again.

Thanks for posting, I'll take a look as soon as I can.  I'd like to test this before merging into main.


Improve notification feature

Description:
The current notification feature feels particularly useless.
When software opened, many notifications popped up all at once.
It would be nice to occasionally pop up a single notification to remind you of today's unfinished tasks.

@cu186 could you please use the corresponding template for a feature request? Or open a discussion first, in order to work out the precise requirements.
And on a side note you might want to check the behavior of this pre-release: https://github.com/ransome1/sleek/releases/tag/v2.0.3-rc.3
It includes a mechanism, which supresses already sent notifications.

I'm sorry. I'll pay attention next time.

I'm sorry. I'll pay attention next time.

Don't worry, since this is a pre-release, it is very unlikely that you could have known.