Invalid Status Code: 403 - While Attempting to Update Various Mods

I use the ModManager frequently to update the mods that I have installed, I use the latest version of the mod that matches my game version (currently version 1.2.3 for Minecraft Fabric 1.18.2). When I attempt to update various different mods I get the same error message repeatedly (see picture). This issue only occurs for me with some mods, not all of them.

The issue persists even if I tell it to try again. I tried checking if this issue still presists when I have Minecraft Fabric 1.18.2 (with Kotlin Fabric Language, 1.7.2 or 1.7.1, and the currently latest Fabric Loader version, 0.14.6, and latest Fabric API, 0.53.4) installed in the default location, and it still does.
After the error message appears I find an empty .jar file in my mods folder named after the new version of the mod ModManager attempted to update to (the file size is always 0 KB). If I try to relaunch the game, it will crash due to having an empty .jar file.
Note: I'm not entirely sure if I remember correctly, but I think that this issue occurs on previous versions of the game the mod was made for too, such as 1.18.1, with version ModManager 1.2.2 (1.2.2+1.18).
#151

I checked if the Cloudflare DNS I have been using was the cause of my problem, as mentioned in #149. I switched to my ISP's DNS from the Cloudflare DNS, and despite that, the issue still persisted afterwards. Maybe the problem will eventually auto-solve itself. Here's the log of the issue:
2022-05-28-5.log

Will be fixed with Core#1

I checked if the Cloudflare DNS I have been using was the cause of my problem, as mentioned in #149. I switched to my ISP's DNS from the Cloudflare DNS, and despite that, the issue still persisted afterwards. Maybe the problem will eventually auto-solve itself. Here's the log of the issue: 2022-05-28-5.log

Here's the part you may be interested in @DeathsGun
[13:13:57] [DefaultDispatcher-worker-2/INFO]: Updating Ok Zoomer
[13:13:57] [DefaultDispatcher-worker-2/INFO]: Error while deleting C:\Users\Lenovo\AppData\Roaming\.minecraft\mods\okzoomer-5.0.0-beta.5+1.18.2.jar trying on restart again
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: xyz.deathsgun.modmanager.api.http.HttpClient$InvalidStatusCodeException: Received invalid status code: 403
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at xyz.deathsgun.modmanager.api.http.HttpClient.download(HttpClient.kt:48)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at xyz.deathsgun.modmanager.update.UpdateManager.installVersion(UpdateManager.kt:302)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at xyz.deathsgun.modmanager.update.UpdateManager.updateMod(UpdateManager.kt:340)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at xyz.deathsgun.modmanager.update.UpdateManager.updateMod$default(UpdateManager.kt:329)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at xyz.deathsgun.modmanager.gui.ModProgressScreen.updateMod(ModProgressScreen.kt:137)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at xyz.deathsgun.modmanager.gui.ModProgressScreen.access$updateMod(ModProgressScreen.kt:39)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at xyz.deathsgun.modmanager.gui.ModProgressScreen$init$2.invokeSuspend(ModProgressScreen.kt:69)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:571)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:750)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:678)
[13:13:59] [DefaultDispatcher-worker-2/INFO]: [STDERR]: 	at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:665)
[13:13:59] [DefaultDispatcher-worker-2/ERROR]: modmanager.error.unknown.update


Caused by: org.postgresql.util.PSQLException: Connection to db:5432 refused

Hi team,
When you try to integrate healenium-backend using docker compose command, the images are being created but while starting the container it is exiting with exit code 1. The console error messages are posted below for your reference. Any leads on this will be helpful.
PS: 5432 port and 7878 ports are not occupied in local system.

Hello @naveenkmr2310 !
Please, provide your docker-compose file


Support of "group" by genbil

Currently genbi and genbiL have no support of the feature "group", would be a great addition to be able to create groups from genbiL.

Overtaken by events ... genbiL support intelligent groups.


Updated RBAC doc

For 1.6 only
Builds on https://github.com/kubernetes/kubernetes.github.io/pull/2169/

This change is

Marking this "Do not merge" since it relates to 1.6

@jaredbhatti any timeframe for the 1.6 branch? pointing people to PR previews is suboptimal

@liggitt FYI, the kubernetes:release-1.6 branch is now available. Please change the base branch to that. Thx

ok, updated the section title. I'll have a lot of follow up PRs to split the other pages out, hoist common information to the overview page, and expand the RBAC page, but I'd like to go ahead and get this merged to get the content on the preview site

lgtm

@liggitt Ok, sounds good.

thanks. @chenopis @jaredbhatti, do you know how often http://kubernetes-io-vnext-staging.netlify.com/docs/admin/authorization/ gets updated from the release-1.6 branch?

@liggitt I believe vnext staging gets updated whenever there's a new merge to release-1.6, but @devin-donnelly would know for sure.

@chenopis  @devin-donnelly doesn't appear to, this merged and http://kubernetes-io-vnext-staging.netlify.com/docs/admin/authorization/ was not updated

@liggitt I believe vnext staging gets updated whenever there's a new merge to release-1.6, but @devin-donnelly would know for sure.

@chenopis @devin-donnelly doesn't appear to, this merged and http://kubernetes-io-vnext-staging.netlify.com/docs/admin/authorization/ was not updated

@liggitt I asked @devin-donnelly in the SIG Docs meeting just now, and the vnext preview should get setup today for 1.6. There was a config setting that hadn't gotten updated yet.


Additional static pages with HTMLWebpackPlugin

I'm attempting to re-use the HTMLWebpackPlugin as specified in https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/config/webpack.config.dev.js#L226-L229 to produce a static maintenance.html file that uses the Interpolated values from the InterpolateHtmlPlugin that is specified above it. When following the docs to create an additional file:
new HtmlWebpackPlugin({
  inject: true,
  filename: paths.appMaintenance,
  template: paths.appHtml,
  minify: {
    minifyCSS: true,
  },
}),

I'm getting an error that it cannot decode the parameters that are available in the standard index.html:
URIError: Failed to decode param '/%PUBLIC_URL%/favicon-32x32.png'
Is there something special I need to do to pass those interpolated values down to additional static files? Thanks!

Do you ejected or building your own webpack config? I feel that you'll get better response at reactiflux / stackoverflow / etc for these kind of use cases :)

Yeah that sounds like a good idea. Thanks!

hey I'm looking for the answer to this as well


Update active-directory-ds-admin-guide-join-rhel-linux-vm.md

Hi,
I'm a PFE from the Services organization and I'm suggesting some changes in order to allow this procedure to work with CentOS flawlessly and to make it clear that this procedure also works with a regular Active Directory domain.
Regards
Paulo

@mahesh-unnikrishnan - Please review the proposed changes.  Thanks

Thanks a bunch for your feedback. This article has changed and we now have distribution specific articles for RHEL, Ubuntu, CoreOS & CentOS. I've incorporated your feedback into the CentOS article.


[Security] IDP shouldn't answer to SP saml request with unsupported name ID formats

in https://github.com/OTA-Insight/djangosaml2idp/blob/master/djangosaml2idp/views.py#L232
or in https://github.com/askvortsov1/djangosaml2idp/blob/master/djangosaml2idp/views.py#L146
The IDP return an UNSPECIFIED format if neither the sp or the idp nameid format was found.
In addition, a check about the availability of the requested name id format, idp side, hasn't been made.
This should raise an handled exception

This was merged and released with version 0.6.0


[Bug?][Fabric 1.21.1] Sunken skeletons drown underwater

I don’t know if this is intentional or not but it seems unlikely that it is but Sunken skeleton variants seem to drown after being underwater for an extended period of time.

Yes that's a bug im trying to get to, wasn't able to work in stuff, all bugs will be fixed as soon as I can work again


Request timeout middleware dns.HandlerFunc

Is there any desire to include a sort of middleware dns.HandlerFunc which provides support for request timeouts via a request context?
It's a pretty straightforward thing to make and I've pulled something together which I'm happy to contribute via a PR (and get rid of my pkg).  Let me know if there's any interest and I'll open a PR.
Here's what I'm proposing to contribute: github.com/jimlambrt/respwriter

coredns implements a whole bunch of middlewares, I don't think it makes sense to pull more into this package. I mean if we pull this in, where do we stop?


updating the screenshot

Like iota, but screenshot looks old (because in header there is rdit). Would be nice if you could update this.

Yeah, that screenshot was from ages ago!

Updated there, its not as pretty as the other screenshot, but I'm not on my mac at the moment to use its fancy screenshot effects :wink:
Will update again later if I can!

But looks also very good. Like it and no problem :)


Set column width for one column (auto size the rest)

Current Behavior
for tailwind - they have a table-fixed option that will let you set the width instead of auto-sizing your columns for you
and I would like to set the width for the first column and have every other column get "auto-sized"
I have tried setting !w-[50px] on simultaneously on the table header and table data - but it just ignores it (specifically for the very first column)
the only way I can get it to allow me to set the width is by changing the table class (but that jacks up the formatting pretty badly and I would have to size all of the columns myself...)
Steps To Reproduce
No response
Anything else?
It was suggested I make a bug ticket for this issue in the following discord conversation:
https://discord.com/channels/1003691521280856084/1078736064237281393/1078736064237281393

@rconnol I apologize for not thinking of this sooner, but we do actually offer a way to do this is you're just creating a native HTML table and using our Tailwind Element styles for it:
https://www.skeleton.dev/elements/tables

You'll need to apply .table-cell-fit to both the header cell, as well as the matching body cell. When you do so, that column will then adjust to fill the width of the contents within.
Here's a quick example:

				Position
				Name
				Symbol
				Weight

			{#each tableArr as row, i}

					{row.position}
					{row.name}
					{row.symbol}
					{row.weight}

			{/each}

If you need something similar for the Table component, then let me know and we'll flip this from a bug ticket to a feature request, as that's something that will need to be implemented as an addition.
Additionally, with the right selectors, you should be able to apply the styles for this using vanilla CSS. Here's how the class works:
https://github.com/skeletonlabs/skeleton/blob/dev/src/lib/styles/elements/tables.css#L100
And here's the reference source:
https://stackoverflow.com/questions/11267154/fit-cell-width-to-content


Update Windows App Lifecycle [#93059106]

Enables running non IIS web applications.
Signed-off-by: Dave_Goddard<EMAIL_ADDRESS>

Hi Ben!
Thank you for submitting this pull request.  Is there a reason to keep the old windows app lifecycle blob around?
https://github.com/cloudfoundry-incubator/diego-release/blob/d42e664f9c492bf34f7d47663eac14827b7ef29b/config/blobs.yml#L26-L29

@leochu no there's no reason to. in fact, we will push a new version of the windows app lifecycle shortly and remove the old one from the blob as well.

@jvshahid @benmoss: should we just close out this PR, then?

Yep!


Pending Orders and Judgments accordion has extra content

(domestic, test, Mason County)
I deleted my only "incident". Now it shows up like this:

It looks like we need to trigger the incidents accordion even when there are none in at least some circumstances.

(though maybe this would be fixed if we triggered minimum incident gathering right away instead of after hitting the resume button)  so maybe try fixing #232


problems creating formula for gprof2dot

I think the source of the issue is that poet is trying to fetch information for urllib2 which appears to be part of the standard python library. Give it a try yourself to take a look at the debugging output:
$ poet gprof2dot
Traceback (most recent call last):
  File "/usr/local/bin/poet", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/site-packages/poet/poet.py", line 203, in main
    print(resources_for(package))
  File "/usr/local/lib/python2.7/site-packages/poet/poet.py", line 157, in resources_for
    nodes = make_graph(package)
  File "/usr/local/lib/python2.7/site-packages/poet/poet.py", line 131, in make_graph
    package_data = research_package(package, dependencies[package]['version'])
  File "/usr/local/lib/python2.7/site-packages/poet/poet.py", line 95, in research_package
    format(name, version or ''))
  File "/usr/local/homebrew/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 154, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/local/homebrew/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 437, in open
    response = meth(req, response)
  File "/usr/local/homebrew/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 550, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/local/homebrew/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 475, in error
    return self._call_chain(*args)
  File "/usr/local/homebrew/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 409, in _call_chain
    result = func(*args)
  File "/usr/local/homebrew/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/urllib2.py", line 558, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
urllib2.HTTPError: HTTP Error 404: Not Found

gprof2dot is available through pip/PyPi

The problem is that gprof2dot's versions aren't PEP 440-compliant. pip returns a normalized version from pip.get_installed_distributions() (2015.2.3 instead of 2015.02.03) but pypi doesn't accept a normalized version in its API (https://pypi.python.org/pypi/gprof2dot/2015.2.3/json 404s, https://pypi.python.org/pypi/gprof2dot/2015.02.03/json does not).
poet could try to work around this by making a more generic call to pypi and working harder to interpret it but I'm not feeling inspired.
I really should add better error handling, though.

The problem is that gprof2dot's versions aren't PEP 440-compliant.

https://www.python.org/dev/peps/pep-0440/ states that "Date based release segments are also permitted. " and gives a bunch of examples such as "2012.04".
So PEP 440 compliance doesn't look like the problem to me.

That's fair; I shouldn't have said "compliant." This is allowed and works normally for version comparison purposes. I should have said that the normalized and canonical versions are non-identical, following the rule at https://www.python.org/dev/peps/pep-0440/#integer-normalization.

I'm closing this since I was able to get a Formula worked up... not sure if there are any issues that need further attention, however. Feel free to re-open, @tdsmith

It really is a poet bug, so I'll leave it open.

I think the best way to normalize the version is to steal this code from pip:
https://github.com/pypa/pip/blob/a6e329fef41a95d688533886b4e0fe24a8551621/pip/_vendor/pkg_resources/__init__.py#L1360-L1369
def safe_version(version):
    """
    Convert an arbitrary string to a standard version string
    """
    try:
        # normalize the version
        return str(packaging.version.Version(version))
    except packaging.version.InvalidVersion:
        version = version.replace(' ','.')
        return re.sub('[^A-Za-z0-9.]+', '-', version)


Paras Faucet Integration

Problem
Our current wallet onboarding required users to fund new accounts by making an initial NEAR deposit. This is a barrier to entry for users who are not already holding NEAR, and there is no way currently for users to fund new accounts with other tokens they might be holding such as ETH.
How did we discover this problem?
The Paras faucet has proven to be a valuable enabler for Rainbow Bridge users coming from Ethereum who have yet to create a NEAR account since it allows them to do so without the prerequisite of holding NEAR. There is a clear opportunity here to make an experience native to the wallet that enables users to create new accounts by leveraging their existing assets as "social proof".
Job Story(s)

When I create a new account, I want to avoid having to fund my account by leveraging my existing assets to authorize my use of the faucet, granting me an account free of charge.

Scope
What are we planning to do about it?
During wallet onboarding, users should be able to select an option to receive a funded account by connecting an Ethereum wallet containing a minimum balance of 0.05 ETH as a kind of social proof.
What are we not planning to do about it?
We are not allowing users to fund their new account with ETH or other tokens. Connecting their Ethereum wallet during account creation is simply a means to authenticate the use of the paras faucet, and will not actually cost them.
Measuring Success
We should monitor the following scenarios to determine the success of this integration:

When using the rainbow bridge without an account, users regularly navigate to the wallet to create their new NEAR account by connecting an Ethereum wallet.
Independent of the bridge, users creating a new account on wallet.near.org regularly take advantage of the ability to connect their Ethereum wallet in order to gain access to the Paras faucet as a means to fund their new account.

Resources
There was discussion of making this integration native to the bridge front-end, and designs have been explored around how to accommodate this here: https://github.com/aurora-is-near/rainbow-bridge-frontend/pull/215
While we may not use the exact same UX, this can act as a reference and starting point for how it might be integrated within the wallet.
UX & UI Design

https://www.figma.com/file/0LZELExyOkQmL6X1R8q6zM/1826-–-Paras-Faucet-Integration?node-id=1%3A2

Design complete.
One thing to keep in mind about interacting with Ethereum Wallets for the purpose of signing txns on the Ethereum side:
On mobile, the only way a user will be able to leverage this faucet is by accessing NEAR wallet from inside a Ðapp browser. Outside of a Ðapp browser, there is no way for a user to connect to a wallet to confirm their Ethereum transactions. I suggest that if there is a way to detect that the user is using a standard browser on mobile, that we hide the option to connect their Ethereum account since it is ultimately inaccessible.

Closing as issue is outdated


Autocompletion not working when invoking a function and records

Hello,
Loving the extension so far. Here are few things I have found:
Using: https://github.com/ryyppy/reason-lnd/blob/master/frontend/src/game.re#L122, I am not seeing autocompletion not shown for function.

In this case, since the function is defined in the same module, it is not as big a deal, but when opening modules, it would be nice to see autocompletion for types that can be passed in.
Similar issue for records, where I see all options instead of ones that can be looked up on the record.

I am comfortable with JavaScript and TypeScript, not so much with TS language server, but I am willing to investigate and help with this if you point me where to start.
Cheers,
Nish

Related to https://github.com/reasonml-editor/vscode-reasonml/issues/4 and https://github.com/reasonml-editor/vscode-reasonml/issues/67


feat: add yamlfix

Add yamlfix to uniformise yaml formatting everywhere.
Related ClickUp, GitHub or Jira tickets : ALEPH-180
Self proofreading checklist

[x] New code does not duplicate the functions of builtin or popular libraries.

How to test
Just run:
hatch run lint:style
hatch run lint:fmt

I find the result less readable without the empty lines. What is your opinion on this ?

I find the result less readable without the empty lines. What is your opinion on this ?

It makes sens to keep at least one whiteline when there is one yes, let's do that https://lyz-code.github.io/yamlfix/#whitelines-adjusting

Done, I've also added section_whitelines = 2 to match python behavior


link to lectures and practices

as an option: add the "Link" field, where you can insert links yourself and remember them; and as an addition, the name of the field may depend on the link itself (for example, if the link is on zoom, then the field is called "Zoom")

#2 need to be implemented first


Add custom timestamps.

Why do you need it? If timestamp is zero, it's not sent over the wire, so trapper uses current time.

In my case metrics are being collected in real time for precise graphs, but it produces high network load when I try send metrics right after collection. So I use metric buffering and send a pack of metrics in every 60 seconds.

Well, current behaviour of MakeDataItems can't be changed. May you update this pull request to keep only new function?


Extend Metadata schema

It'd be great if we could easily extend/override the Metadata schema for the issuers.
I've tried a couple of OIDC configurations (ADB2C & Duende IdentityServer) and both have custom claims_supported, grant_types_supported, response_types_supported etc..
I haven't found an easy way to override that at the moment.

I agree this is really needed as I don't seem to be able to authenticate with OneLogin due to Zod and schema matching issues.

my attempts to use this to add Microsoft OIDC SSO suffer the same problem here

Does this basically mean these providers are simply not supported at all? Dreading recreating OIDC just because the zod schema doesn't match (although maybe the claims/grants need to be actually supported).


Ingress Trait to support an array on the path

Requirement
Would like to set multiple paths and/or aliases for an integration.
Problem
Currently unable to utilize the ingress trait on an integration to set multiple paths for the ingress to consume.
It appears this is possible/supported via kubernetes documentation:
https://kubernetes.io/docs/concepts/services-networking/ingress/#examples
Proposal
Allow and array on ingress.path ([]string):
https://camel.apache.org/camel-k/2.5.x/apis/camel-k.html#_camel_apache_org_v1_trait_IngressTrait
Open questions
No response

Thanks for the feature request. Are you planning to work on this by any chance?

@squakez - I would very much like to try -- I'll take a walk through the developer's guide and see if I can work through how to implement the changes.

Good process so far, documentation has been excellent. I should be able to get a PR submitted in the coming days.

Good progress so far, documentation has been excellent. I should be able to get a PR submitted in the coming days.

Excellent. Just let us know if you have any doubt or you need any help!

TL;DR - I think this is bubbling up into a bigger issue.
@squakez  I have a working build, allowing a string array on the ingress.path, however, I have some doubts...

This would become a breaking change == any integration having an ingress.path=<string> set would break.
Possibly some code could be implemented to auto-convert string into array... but maybe not worth the effort based on below.
This isn't fully solving the problem... IMO, the IngressTrait should follow the APIs hierarchy, starting from here:
https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.31/#ingressspec-v1-networking-k8s-io
This would allow any number of IngressRules (which has it's own sub-structure), meaning, you could specify multiple hosts, and for those hosts, multiple paths, which may also have their own associated pathType.

Best visual example of the hierarchy (from https://v1-30.docs.kubernetes.io/docs/concepts/services-networking/ingress/#hostname-wildcards):
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: ingress-wildcard-host
spec:
  rules:
  - host: "foo.bar.com"
    http:
      paths:
      - pathType: Prefix
        path: "/bar"
        backend:
          service:
            name: service1
            port:
              number: 80
  - host: "*.foo.com"
    http:
      paths:
      - pathType: Prefix
        path: "/foo"
        backend:
          service:
            name: service2
            port:
              number: 80

Please let me know your thoughts.

First of all thanks for the effort to put on a draft @cfitzw, really appreciated. About the specific problems you're mentioning:

Indeed, we cannot transform the usage of path as it introduce a compatibility problem. What we can do, however, is to introduce a new one, ie, paths that will hold the new logic. Eventually we can deprecate path in favor of paths if we see it makes sense.
If the goal of the feature is to support multiple paths and hosts, then, we may rework entirely the feature and allow the user to introduce a given syntax, eg, -t ingress.rules=foo.bar.com/bar -t ingress.rules=*.foo.com/foo which would translate into the Ingress structure you've highlighted.

I am not entirely sure what exactly you need to support, but I think we can separate this issue to work both on 1 and, if that makes sense, on 2, providing a different parameter for each feature we want.

@squakez -- thank you for the feedback.
I've implemented number 1 and committed to the PR.
In regards to number 2, this is not a desired need at-the-moment, so let's focus on adding support for number 1 and this can become a new issue/PR at a later point-in-time.


Icons!

Instead of a big drop-down, use a grid of icons, aping the in-game one, to select items.

Done!


kubernetes/helm provider dependencies on local file

Hi there,
I'm sorry if a similar ticket has been raised already. I did search but couldn't find my expectation. Please correct me.
Basically we have kubeconfig files stored on aws/ssm-parameter store. And we generate local file then give them to kubernetes provider, as below
# Terragrunt module A
locals {
  k3s_config_ssm_path = "config.yaml"
}

data "aws_ssm_parameter" "k3s" {
  name            = "/some/path/on/aws/ssm"
  with_decryption = true
}

resource "local_file" "k3s_config" {
  content         = data.aws_ssm_parameter.k3s.value
  filename        = local.k3s_config_local_path
  file_permission = "0644"
}

From Terragrunt module B which depends on module A
# terragrunt.hcl has block that declares a dependency on A.

provider "kubernetes" {
  config_path = local.k3s_config_local_path
}

As far as I know, terragrunt run-all apply will work well, however, terragrunt run-all plan doesn't work on a fresh check-out of the source tree. Because the local file doesn't exist before kubernetes provider starts its job.
Is there any work-around for this problem with helps from terragrunt?
Thanks for your reading.

Hi, I was thinking that usage of output variables and mock_outputs on dependencies can help
If I have this setup:
├── a
│   ├── main.tf
│   └── terragrunt.hcl
└── b
    ├── main.tf
    └── terragrunt.hcl

a/main.tf

provider "aws" {
  region  = "us-east-1"
}

data "aws_ssm_parameter" "k3s" {
  name = "test"
  with_decryption = true
}

resource "local_file" "k3s_config" {
  content         = "module a config"
  filename        = data.aws_ssm_parameter.k3s.value
  file_permission = "0644"
}

output "config_file" {
  value = local_file.k3s_config.filename
  sensitive = true
}

b/terragrunt.hcl

dependency "a" {
  config_path = "../a"
  mock_outputs = {
    config_file = "file.yaml"
  }
}

inputs = {
  k8s_config = dependency.a.outputs.config_file
}

b/main.tf

variable "k8s_config" {
  type = string

}

resource "local_file" "provider_file" {
  content         = "k8s_config  ${var.k8s_config}"
  filename        = "module_b_provider_file.txt"
  file_permission = "0644"
}

and run terragrunt run-all plan first time, will get:
<module a output>
Terraform will perform the following actions:

  # local_file.k3s_config will be created
  + resource "local_file" "k3s_config" {
      + content              = "module a config"
      + directory_permission = "0777"
      + file_permission      = "0644"
      + filename             = (sensitive)
      + id                   = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Changes to Outputs:
  + config_file = (sensitive value)

<module b output>
Terraform will perform the following actions:

  # local_file.provider_file will be created
  + resource "local_file" "provider_file" {
      + content              = "k8s_config  file.yaml"
      + directory_permission = "0777"
      + file_permission      = "0644"
      + filename             = "module_b_provider_file.txt"
      + id                   = (known after apply)
    }

Plan: 1 to add, 0 to change, 0 to destroy.

So it will reference value ${var.k8s_config} from A to B

Hi, I was thinking that usage of output variables and mock_outputs on dependencies can help

@denis256 Thanks for you feedback. I think the problem with mocking is that it doesn't give any real data. At the planning time, helm/kubernetes provider is actually requiring some real configuration to access to the cluster. Without a correct configuration for k8s, helm likely reports an error attempting to access k8s api. This is kinda an annoying thing.
What I'm doing now is to execute a simple command to generate k8s configuration, which is alternative of the terragrunt apply form the module A (FYI, I'm using before_script within a Gitlab build context.)  I think it's great if terragrunt can give some pre-hook command and/or to execute an actual terragrunt apply command before continuing. But it's my 2 cents maybe there is any better way.

Hi, most probably hooks in terragrunt.hcl can be used to get a similar result:
# terragrunt.hcl
terraform {
  before_hook "fetch_kubernetes_config" {
    commands = ["apply", "plan"]
    execute  = ["echo", "Fetching Kubernetes config"]
  }
}

https://terragrunt.gruntwork.io/docs/reference/config-blocks-and-attributes/#terraform

@denis256 That's a nice idea. I will look how it works with our workflow. Thanks for your information.

Sorry for this late feedback. We now completely forget the local file and use provider's parameter instead. It's not very convenient in some way (look at [0]), but it works in our case. The whole code is as below; we hope that helps.
locals {
  k3s_config_ssm_path = "/some/path/on/aws/ssm/config.yaml"
}

data "aws_ssm_parameter" "k3s" {
  name            = local.k3s_config_ssm_path
  with_decryption = true
}

locals {
  k8s_config_as_json = yamldecode(data.aws_ssm_parameter.k3s.value)
  k8s_config = {
    host                   = local.k8s_config_as_json["clusters"][0]["cluster"]["server"]
    cluster_ca_certificate = base64decode(local.k8s_config_as_json["clusters"][0]["cluster"]["certificate-authority-data"])
    client_certificate     = base64decode(local.k8s_config_as_json["users"][0]["user"]["client-certificate-data"])
    client_key             = base64decode(local.k8s_config_as_json["users"][0]["user"]["client-key-data"])
  }
}

provider "kubernetes" {
  host                   = local.k8s_config["host"]
  cluster_ca_certificate = local.k8s_config["cluster_ca_certificate"]
  client_certificate     = local.k8s_config["client_certificate"]
  client_key             = local.k8s_config["client_key"]
}

provider "helm" {
  kubernetes {
    host                   = local.k8s_config["host"]
    cluster_ca_certificate = local.k8s_config["cluster_ca_certificate"]
    client_certificate     = local.k8s_config["client_certificate"]
    client_key             = local.k8s_config["client_key"]
  }
}


Responsive gridLayout padding is add to every row of hight

Hi,
I found a problem with resize items in the grid. Everytime when I resize height of component I would expect just double it value of the "rowHeight" property plus margin in "containerMargin.y".
But in the real scenario the pattern is littlebit different. I found something like this;
on resize the height is does :
rowHeight * h + containerMargin.y * (height - 1);
for example:
rowHeight = 30;
containerMargin = 5;
h = 1;
so the component of the height 1  ... have height 30px ;
30 * 1  + 5 * (1 - 1) = 30;
h = 2;
30 * 2  + 5 * (2 - 1) = 65 px;
h = 2
with height 3 is gap even bigger ;
30 * 3  + 5 * (3 - 1) = 100 px;
and so on....
So I think the margin should be just + 2 times every time (1 times on each side);
Somethink like this
rowHeight * heightOfComponent + containerMargin.y * 2;
30 * 2 + 5 * 2 = 70px;
30 * 3 + 5 * 2 = 100px;
30 * 4 + 5 * 2 = 130px;
and so on ...
or just rowHeight * h
add the margins between components and thats it :)
Cheers have a good day

add the margins between components and thats it

That's what we do. Take some time to look at how large components line up to small ones. Margin is added for each additional span. If it weren't, the grid would be tagged.


Creating a new terminal does not automatically activate the chosen conda interpreter

Environment data

VS Code version: Version: 1.46.1
Commit: cd9ea6488829f560dc949a8b2fb789f3cdc05f5d
Date: 2020-06-17T21:17:14.222Z
Electron: 7.3.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 18.7.0

Extension version (available under the Extensions sidebar): XXX

OS and version: MacOS Mojave

Python version (& distribution if applicable, e.g. Anaconda): conda 4.8.3 python 3.7

Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda

Relevant/affected Python packages and their versions: XXX

Relevant/affected Python-related VS Code extensions and their versions: XXX

Value of the python.languageServer setting: XXX

Expected behaviour
My understanding is that when creating a new terminal it should automatically load the chosen interpreter
Actual behaviour
It starts terminal with base conda environment.
Steps to reproduce:
Not sure what's relevant here. But I installed conda, setup an env in an outside terminal. Then restarted VSCode and chose the correct interpreter. But terminals still open with base conda env

Hi @rahul003 , I would need additional info for this. Please provide the logs as mentioned in the issue template, and also, the extension version.
Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)

XXXX

Does having this in user setting work?
  "python.experiments.optOutFrom": ["DeprecatePythonPath - experiment"]

Closing as no further info was provided.


Add push/set JSON with String parameter

I remove this JSON string support since v 2.6.x as I state in the doc here

Why did decide to remove them?

Better and alternative to use FirebaseJson which already has function setJsonData if someone want to set JSON string to it. And due to I want to remove the setXXX, pushXXX and getXXX to use the generic (template) function as simple set, push and get which String parameters of JSON string is not consistent between get and push or set.

Thank you. Close PR.


[SP-2842] - Backport of PDI-15007 - Many errors (NPEs, Infinite loops…

…. bad data) in Const utility methods (6.1 Suite)
@mchen-len-son, here is backport of  https://github.com/pentaho/pentaho-kettle/commit/43f7e39d34714306f04b97d3557d828ce2b4d43d to 6.1. Thanks.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f core/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f core/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  core/src/org/pentaho/di/core/Const.java
  core/test-src/org/pentaho/di/core/ConstTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.Const

Branch Change: -.67%:small_red_triangle_down:
Complexity Change: -1.47%:small_red_triangle_down:
Instruction Change: -.46%:small_red_triangle_down:
Line Change: -.42%:small_red_triangle_down:
Method Change: -.67%:small_red_triangle_down:

Double-checked locally.
Tests are passing, checkstyle is OK.
Merging.


BUG: Command prompt no longer prints

There is a line of code (xpd_puts(">>> ");) that is being executed with no output so the shell is not prompting the user properly for command with >>> .
This line was being printed correctly at one point, but I'm not sure when exactly this line stopped being printed.
The code around it is being executed because reading input works, so I have no clue what the issue is (although I suspect this is a problem with the XPD driver).

I am 99% sure this is a problem with the XPD.h driver. After adding random print statements in places, it prints sometimes. I'm not sure how to fix.

Resolving because we have deduced that it is just bad behavior on the part of the driver


GTM Rogue Referral issue

There's an issue that crops up in SPAs that's dubbed the Rogue Referral issue, you can read about it in depth over here...

Simo Ahava: Fix the rogue referral problem with single page sites

Right now there are two open PRs to try to address this but they've been abandoned and are no longer in-sync with the current release of gtm-module.

#74 (open)
#86 (open)

(There's also issue #68 which was around this as well but was closed without an actual resolution.)
What is the best way to get this moving forward again?
I was starting to put together a new PR against the current base branch, but the two existing PRs worked in different files and I wasn't clear on what the actual best practice was (if any), or what the difference between the two approaches was (if any).
(This situation a blocker to me being able to use Nuxt for any work at ye olde day-job since it means paid traffic can't be accurately tracked for performance purposes.)

Looks like this also snakes back to September 2019 in this issue that was filed over on nuxt/nuxt.js originally.


Lint on broken recursion structure

Issue description
It's possible to write something like:
{somePackageName, stdenv}:
stdenv.mkDerivation {
  pname = "some-package-name";
  # ...
  passthru.someOtherThing = pkgs.callPackage ./something.nix { inherit somePackageName; };
}

where something in passthru doesn't follow the containing derivation's overrideAttrs properly due to broken recursion structure. The correct code here would be:
{somePackageName, stdenv}:
stdenv.mkDerivation (finalAttrs: {
  pname = "some-package-name";
  # ...
  passthru.someOtherThing = pkgs.callPackage ./something.nix { somePackageName = finalAttrs.finalPackage; };
})

I would like to have an automated lint that rejects code that exhibits this broken recursion structure: passthru referring to the parent derivation but the passthru attrs get the parent derivation from other means than finalAttrs.finalPackage.

Why? Also, shouldn't we ensure all the builders support the finalAttrs pattern in the first place?

Also, shouldn't we ensure all the builders support the finalAttrs pattern in the first place?

yeah, there's a lot of very broken builders that have very broken overrides in general. we should at least fix the standard mkDerivation though.


Feat: endpoint for soft deletion of a customer by admin user

Implement DELETE Endpoint for Customer Deletion
Description
This pull request introduces a new endpoint for deleting a customer from the database. The endpoint ensures that only admin users can perform the deletion. The following changes have been made:

Endpoint Implementation: Added a DELETE api/v1/customers/{customer_id} endpoint to handle customer deletion.
Authorization Check: Implemented authorization logic to ensure that only admin users can delete a customer.
Soft Deletion: Used soft deletion to mark customers as deleted without physically removing them from the database.
Response Handling: Added proper response messages for success and error cases.

Related Issue
This change is related to issue #69, where we need to provide functionality for customer deletion with proper authorization checks.
Motivation and Context
The application requires functionality to delete customers, but with safeguards to ensure that only admins can perform this action. Implementing soft deletion helps in maintaining data integrity and auditability by retaining deleted records for future reference.
How Has This Been Tested?
The following tests were performed to ensure that the new endpoint works correctly:

Unit Tests: Added unit tests for the DELETE api/v1/customers/{customer_id} endpoint to verify its behavior under various scenarios:

Successful deletion by an admin user.
Failure when a non-admin user attempts to delete a customer.
Handling of cases where the customer does not exist.

Integration Tests: Verified the integration of the new endpoint with the existing authentication and authorization mechanisms.

Testing Environment:

Local development environment
PostgreSQL version: 13
Python version: 3.10
FastAPI version: 0.111.1

Screenshots (if appropriate - Postman, etc):
Example Request:
all postman screenshots are in the doc/screenshots folder
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[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.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Fix conflicts

@Dev-wonderful
resolved


Set android camera to autofocus

This change makes the Camera widget on Android to autofocus. This avoids the blurry images that appear when using this widget.

I also changed the fragment shader to rotate the texture for android devices.

Except the pep8, looks good!


Highlight chop destination

Is your feature request related to a problem? Please describe.
People often chop a feature or resource, and are immediately bummed to realize that the tile was owned by the wrong city, thereby sending production or food to a place they didn't intend.
Describe the solution you'd like
Anytime you have a builder with 1+ movement selected on top of a resource that would give production/food, or alternatively (if possible) anytime you hover over the toolbox bulldozer that would remove such a feature, display a white hexagon around the city center plot that will receive the production. This way, the destination is 100% clear and the player knows to swap the tile if needed.

I realized I never replied to this but I like this idea.  Not sure if it's possible, but I'll look into it at some point.


resolve how to produce perfomance tests after a change automatically

not to be fully solved until #8,
now inside notebook -- see 22caea2baf48a9b79a046660cfbf304c8f7121ef

probably blocked by #12

not imporant, in my opinion

not imporant, in my opinion

the thing of the issue is to have also an automatised test by means of one-click running (running each setting separately in the notebook), to run it when a change appears in the core is a simple git hook.


ThreadLocalSpan.CURRENT_TRACER  is null

Span span = ThreadLocalSpan.CURRENT_TRACER.next();    span is null

this issue was cross pasted a lot. please follow up here https://github.com/openzipkin/sleuth-webmvc-example/issues/13


animateLayoutChanges option in useSortable

i love to learn anything by doing. while i was at your this story RemovableItems you used this
export const RemovableItems = () => {
  const animateLayoutChanges: AnimateLayoutChanges = (args) =>
    defaultAnimateLayoutChanges({ ...args, wasDragging: true });

// ...rest.......
};

animateLayoutChanges and defaultAnimateLayoutChanges options but these are not in your useSorting hook documentation and search engine

Copy there package.json file and check your versions.

Didn't you mean useSortable hook? I'm using "@dnd-kit/sortable": "^7.0.1" and it's right there:
const {
    attributes,
    isDragging,
    listeners,
    setNodeRef,
    transform,
    transition,
  } = useSortable({
    id,
    animateLayoutChanges,
  });

But that's true, the docs are missing these sections. @clauderic could you please consider updating the docs? I'm trying to extend the animation duration and thanks to useSortable.d.ts declaration file, however it'd be great if it's covered in the docs :)

Btw docs still missing it.


Release 2.18.4 has 2.18.4-DEV in PhpCsFixer\Console\Application::VERSION

Shouldn't this line be
    const VERSION = '2.18.4';

instead of
    const VERSION = '2.18.4-DEV';

?

Hey, thanks for spotting. the only way to fix it is to actually release 2.18.5, which is planned anyway, so i gonna close this github issue.

Just a hint for future releases 😉

For reference:

06f764e3cb6d60822d8f5135205f9d32b5508a31 was the commit that was tagged as v2.18.4
e0326187520ce098bad34a424669cbc09d610613 (the next commit) was the commit that should have been tagged as v2.18.4, as it included the changelog and version bump


Updated docker-compose.yaml to fix rpc bug

In order to unlock accounts with Seth-rpc, the rpc service and the cli-go service need to share a volume where the aliases are stored. This PR introduces that volume.

I think you are missing the sign-offs in your commit message. You can fix it by running
$ git commit -s --amend

While you are there, please add some more context to your commit message, following this guide for writing commit messages: https://chris.beams.io/posts/git-commit/
After you've done this, force push your branch.

Can you add more context to the commit message itself? "In order to unlock accounts with Seth-rpc, the rpc service and the cli-go service need to share a volume where the aliases are stored. This PR introduces that volume." would work.

This PR cannot be merged because the commit doesn't pass the DCO check (see comment from Apr 25, 2019) and we have a request to update the commit message (see comment from Jan 4, 2019). AFAIK these are the only things holding up this PR.

@Mansoor-AR I see 3 commits in place of one for this change. One option could be to cherry-pick the commit with DCO and then force push to the same branch name.

This repository will be archived soon so I'm closing this PR. Thanks for your contribution!


Update ValidatorTests.java

person.setFirstName("again empty");

@MichaelKim2000 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Wrong path to SmartScreenSDKConfig.json in macOS Quick Start Guide

Briefly summarize your issue:
Command to run the sample app (Step 8) contains incorrect path to SmartScreenSDKConfig.json.
https://developer.amazon.com/en-US/docs/alexa/alexa-smart-screen-sdk/mac-os.html#step-8-run-the-alexa-smart-screen-sdk-sample-app
What is the expected behavior?
The path could be for example:
~/sdk_folder/alexa-smart-screen-sdk/modules/GUI/config/SmartScreenSDKConfig.json
What behavior are you observing?
SmartScreenSDKConfig.json doesn't exists in specified path.
Provide the steps to reproduce the issue, if applicable:
n/a
Tell us about your environment:

What version of the AVS Device SDK are you using?
2.0.1
Tell us what hardware you're using:

[x] Desktop / Laptop
[ ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[x] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi maciejmiarecki,
Thanks for bringing this to our attention. We will be fixing the documentation with the next release of the SDK.
Thanks
Priya


[dv/flash_ctrl] Adding he_en to mp cfg and randomization

Hi,
This PR is adding the randomization and handling of high endurance enable in the memory protection configurations of the flash_ctrl DV.
Thanks!
Signed-off-by: Eitan Shapira<EMAIL_ADDRESS>

Thank!
If all is good here, can you please merge this PR?


replace deprecated exallocatepool* APIs with their modern replacement

Re-doing this PR against the develop branch.

@JakobL-MSFT Correct target branch this round?

Yes, right branch :-)  Thank you so much.   I have an unrelated comment I will post separately.

@JakobL-MSFT This change is broken. You cannot replace ExAllocatePoolWithTag with ExAllocatePool2 directly because the argument type changed from POOL_TYPE to POOL_FLAGS. The driver will refuse to load with Code 31 (STATUS_NO_MEMORY). May I ask how this change was tested by Microsoft before being merged?


wt doesn't start in directory given from explorer

Environment
Windows build number: 10.0.18363.1256
Windows Terminal version (if applicable): 1.4.3243.0

Any other software?

I don't think it matters but I am using clink in cmd and a profile in powershell. Same stuff happens without them.

Steps to reproduce
If I open wt from a directory via Windows Explorer, it doesn't start in the correct directory. This behavior is different from cmd/powershell.
Expected behavior
If i run wt in Windows Explorer in a directory, it should start in that directory.
Actual behavior
wt seems to start in the user's home directory.
Attaching a video of the behavior difference
https://user-images.githubusercontent.com/5749838/103238307-233d1c80-48ff-11eb-8b5b-d435035fce89.mp4

The recommended way to make this work is by entering wt -d . in the explorer bar. For more info see https://github.com/microsoft/terminal/issues/878#issuecomment-579508860.

Yep, this is by design. /dup #878

Yep, this is by design. /dup #878


arithmetic operation overflowed

There's a problem upstream for mac users that stops iron apps from starting up.
I've created an issue out of courtesy in case your users stumble across this.
https://github.com/seanmonstar/num_cpus/issues/2

Fixed in the latest version of num_cpus.


Update README.md for clearer instructions

Only Second Generation is supported for CloudSQL Proxy
On GCE you need to target b.gcr.io/cloudsql-docker/gce-proxy:1.05 because of this Issue: GoogleCloudPlatform#35

Thanks much for the doc updates!


package.json not found

Describe the bug
The examples and apidocs try to load the current version from https://raw.githubusercontent.com/openlayers/openlayers.github.io/build/package.json
but it returns a 404 error.
If you are browsing an older version you no longer get the info to switch to the latest
To Reproduce
Steps to reproduce the behavior:

Go to https://openlayers.org/en/v6.15.1/examples/accessible.html
No info about outdated version
See error in console

Expected behavior
The package.json should be available at the URL above.

I started to fix this up by propagating the change in #14025 to the content in the openlayers.github.io repo, but needed to board a plane.  I'll continue making changes when back on the ground unless someone else has fixed it up already.

The scripts for deployed versions of the API docs have been updated.  Examples still need updating.

Examples updated with https://github.com/openlayers/openlayers.github.io/commit/f48da45e5955292715d6b2ea9e54744ec88e1387.  When that deploys, things should be working again.


Use json-stringify-safe implementation of stringify

I was getting occasional circular reference errors when saving objects.  It appears that json-stringify-safe was loaded, but the calls were still using JSON.stringify.

Thanks for figuring this out!  That was an annoyance.


"Packed Cell" encoding

The current cell encoding is rather heavyweight in its memory use. Each cell consists of a tagged 8-byte pointer to a 24-byte struct, consisting of:

8 bytes of metadata (for alignment, only 4 bytes are needed for mugs)
8 bytes to store the head of the cell
8 bytes to store the tail of the cell

All structure in nouns is encoded with cells, so such a heavyweight encoding is likely to create memory pressure. There are 2 major inefficiencies:

4 bytes are wasted for alignment purposes to store the metadata.
8 bytes each are used to store a pointer to something which, given the use of a bump allocator and copying collector, is likely very nearly adjacent.

It may be possible to produce a "packed" representation of a cell which need not be a tagged pointer, but rather packs the mug and offsets from itself to its children into a 64 bit word. Such a representation could not be directly loaded into a register or local variable, as the offset information would then be meaningless. It could however be dynamically converted, since we use accessor methods for cells already. Probably this form would never be created by user code calling some variant of Cell::new(), but would instead be created by the copier after checking the constraints. Since the copier is also responsible for laying out a copied noun in memory, combining this representation with a breadth-first traversal for copying nouns could result in nearly 3x savings in cell memory allocations in the best case.

It'd be great to experiment with this. I'm not sure if it'd be worth including the mug -- that's a lot of bits. Another possibility is to include head/tail atoms directly where possible -- non-null +units (and maybe some tagged unions) could effectively be encoding with tagged pointers.

I'm not sure if it'd be worth including the mug
It's necessary because otherwise you have to copy everything below the cell back to the current frame to mug it, and the mug cache won't get propagated back to the original copy.


Architecture of BioD3®

Architecture of BioD3®:

Architecture of Scientific Data Visualisation Framework "BioD3",
Modular Structure Design,
Dependency Design,
A high level structures of a software system based on CoffeeScript and SASS.

Note: Defining a structured solution that meets all of the technical and operational requirements, while optimizing common quality attributes such as performance, reusability, interactivity, modularity and manageability.

Architecture of BioD3® started and Modular Structure Design - is done.

@require: require.js
@require: r.js


PR template

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
PR template
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

It can have Medium as the GSSOC label


Fix problem with detection of endpoint termination

When a endpoint is terminated a EndpointTerminatedEvent is emitted using the eventstream.
When this message is processed it seems to be wrapped in a slice. This leads to a processing inside of sendEnvelopes which causes the actor to fail and be restarted.
To avoid this problem I added a additional handling inside of sendEnvelopes. This can only be considered as a temporary solution. It seems that the event should not be wrapped in the first place.

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

To reproduce the problem start a cluster with random ports and send some messages.
Terminate one member and restart it with a different port.
As a result the endpoint never stops to try to reconnect.

Good catch,
The EndpointWriter uses a batching mailbox, thus any message sent to it will be wrapped in a slice.
(except for ProtoActor built in messages that are not delivered via the mailbox)
I will look into this deeper to see what can be done about the whole process of remoting


[Feature Request] Custom packet priorities

Each packet has been assigned a priority, but it’s unclear how these priorities were determined. When I want to change the priority of a packet , it’s not very straightforward; it always requires making changes in the source code. Would it be possible to set priorities for each packet  in an .ini section as shown in the example? When the sphere starts, it would load a packet priority list and maintain that priority value for the packet as long as the server is running. If no priority is specified for a packet , the sphere would assign the default priority value. This would be great. I have no idea how feasible this is; we can discuss this further under this topic.
eg;
// PRI_LOW = 1
// PRI_NORMAL = 2
// PRI_HIGH = 3
// PRI_HIGHEST = 4
[PACKETPRIORITIES]
PACKET0x88=4 // PacketPaperdoll
PACKET0x86=1 // PacketCharacterListUpdate
or decimal
PACKET136=4
PACKET134=1

If a priority is entered for that packet in the list above, that packet will run at that priority. If no priority is entered, Sphere will continue to run with its default priority value.
instead of function there will be priority value (1-4).
It could be a structure similar to "PACKET FILTER: Hooked packet"
///////////////////////////////////////////////////////////////
//////// General Information
///////////////////////////////////////////////////////////////

[SPHERE]
// Both
// PACKET12=F_PACKET_0x0C
// PACKET21=F_PACKET_0x15
// PACKET34=F_PACKET_0x22

Or can it be added to the existing structure? For example, a variable that can be read/written under the hooked function
LOCAL.PRIORITY=1
similar
eg;
[FUNCTION F_PACKET_0x0C]
SERV.LOG Priority: <DLOCAL.PRIORITY>
LOCAL.PRIORITY=1

Do the priority change really something? Each packet are send reallt fast no?

Do the priority change really something? Each packet are send reallt fast no?

Theres also packet priority sphere.ini setting, not sure what that change. 🤔

Do we know what determines the current and previous packages? I want a package to be given the highest priority, is there a requirement to change the code for this?

Do the priority change really something? Each packet are send reallt fast no?

Actually, this is not a matter of whether it is sent fast or not. Let's think of two different packets with the same length and think that we send them at the same time. When sphere starts begintransaction, it processes and sends the packet that is the priority first.


Use jq for better error reporting with turborepo

What happens?
Currently when there's an error during build, turbo repo just stops and you need to scroll up to find the error
What did you expect to happen?
Display the errors nicely at the bottom
We can use jq and summarize to format the errors nicely
https://github.com/vercel/turborepo/discussions/8381#discussioncomment-9934147

Looks like turbo was upgraded and they went back to the old UI, this is no longer needed. I kind of miss it though


Fix naming issues in workflows

Purpose

$subject.

Codecov Report

Merging #23694 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #23694   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 2da6e08...3338ac0. Read the comment docs.


Issues #17034: Changed Dead Mob DrawDepth from FloorObjects to Items

About the PR

As per #17034, made dead mobs change to the "Items" DrawDepth instead of "FloorObjects", which should put them above things like puddles, but below things like living mobs.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

fix: Dead mobs now appear above puddles!

I think maybe we need a drawdepth tag between FloorObjects and SmallMobs instead.

Sure, should it just serve dead mobs, or did you have anything more general in mind for the layer?

Changing to a layer between "FloorObjects" and "SmallMobs" leads to dead mobs being drawn under tables/machines. Yey or ney on the change?

under tables seems fine
also vaulting should prob force a thing onto a layer above the table, so say if a mouse jumps ontop of a table people can see it

Sure, should it just serve dead mobs, or did you have anything more general in mind for the layer?

Dead mobs is fine, small mobs should just draw over them. We need more drawdepths anyway as it makes the renderer faster.

Sure, should it just serve dead mobs, or did you have anything more general in mind for the layer?

Dead mobs is fine, small mobs should just draw over them. We need more drawdepths anyway as it makes the renderer faster.

New Layer has been added!


azcopy 2012

does azcopy v10 work on windows 2012 R2? I have been trying to get simple stuff working for a while and it does nothing. Tried the cp and sync options and the command sees to take but no output in the command shell, just sits there until I kill it.

@gatornation11  Please help with the link of document on which above document feedback request is created. In case your query in not related to document content, open a new forum thread in Azure Community forum as it could be better place to get help on your scenario. These forum community members could provide their expert guidance on your scenario based on their experience. Thanks.


[Lambda] Add Lambda support for Skygear Data Type

Add new $type=record in serializer/deserializer to Record data type.
Modify lambda.go to use mapconv deserializers for converting parameters
to payload.
Lambda result in LambdaHandler should be interface{}, make sure it supports:

slice
map
Record
complex data types (such as skydb.Location)
simple data types (such as number)

Add test case for testing new input/output data types.

Assets handling

If user call lambda with asset, SDK will upload the asset (if new) and skygear server will sign the asset before passing to plugin.
If lambda return asset, skygear server will sign the asset object before returning to the SDK

Invalid data handling

Skygear server will return 400 directly when calling the plugin if the data invalid. For example, unrecognized $type, or missing required data like $type=number with no number ...etc.


MINOR: Rename RecordBatch to ProducerBatch to allow reuse in KIP-98

cc @ijuma, as discussed in #2614.

Looks like this needs a rebase?

@ijuma Thanks for reviewing. Agreed on IncompleteRecordBatches. I will rename it as well.


distribute_params and random seeds -- a truly random random seed?

The distribute_params method is designed to let you clone an AgentType while distributing the new Agents over a distribution over a single parameter, such as DiscFac.
https://github.com/econ-ark/HARK/blob/910bff34d6e1d59e686029076ca82a6cf81553c2/HARK/core.py#L1664
Using this feature, I've been surprised when the resulting AgentTypes all have perfectly correlated income shocks.
In hindsight, I should not have been surprised: these AgentTypes all have the same random seed.
However, correcting for this issue required some hacky custom code, looping over the agents, changing their seeds, reseting their RNGs, etc.
While having default and steady random seeds is very useful for testing and development, in Monte Carlo simulations with mutliple runs of the same model, you want actually random random seeds.
HARK could better support this with some kind of flag in the AgentType constructor that, if True, gave the agent a random seed (using the larger computing system to provide the entropy).
A corresponding argument could be added to `distribute_params``

I think this issue was resolved (and the missing connection to central RNG as well). To the extent there's something left to do, it's on the documentation side, directing users on how to set seeds for AgentType instances.


Change username's default allowed characters to confirm better to email spec

Since the default configuration in the samples, and presumably in quite a lot of systems will be that the username is the email address, it threw me off a little bit when discovering that the + sign is not an allowed character in a username.
I was thinking that it would be helpful to offer the email's allowed list (see this answer of someone diving into the allowed characters).
If this is not feasible (I can imagine that a username should be something remotely readable, although one could argue that an email should be as well), would it be possible to offer both versions of these strings as a const string in the user options class, so that this behavior is not repeatedly implemented by every application? In either case, the current implementation suggests it allows email addresses (the @ and .) but it doesn't actually fully implement this.
The string I use in my application (as far as I know I'm only missing the 'quoted' character requirement):
abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789!#$%&'*+-/=?^_`{|}~.@

@blowdart the delta appears to be the following characters, how many of these do you want to enable by default, some of these look pretty wonky to be valid user name characters by default...
!#$%&'*+/=?^`{|}~

Oh that is bad isn't it. Lots of invalid characters for emails. This is strange. Either we update the samples so it's not emails, or we pare it down a bit.

I can imagine that a username should be something remotely readable, although one could argue that an email should be as well

@janpieterz Well... the original RFC compliant regex doesn't argue. That is if you want to validate as per the RFC.
There is many characters that are available in an email (()[]+-<> among others).
Good read if you want to go further.
One thing for sure... refusing a valid email is plainly frustrating to most users.

I do use the + sign in my Gmail address. Allows me to tag emails I use to register to some services.

We won't be using regular expressions for validation. We made a conscious decision to remove that approach.

For reference, AllowedUserNameCharacters is here ...
https://github.com/aspnet/Identity/blob/c8a276e716064ac40df5a56512e0217b14b48060/src/Microsoft.AspNetCore.Identity/UserOptions.cs#L17
abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789-._@
Is there any reason why that can't be overridden no matter what the team settles on here?

@GuardRex  It can (quite easily even) be overridden. Below does the trick. The issue (more an issue with the template and a suggestion for improvement for Identity) isn't the override, but more the fact that the default templates use an email as a username (and I think this is a very common usage pattern) and this fails when certain characters are used, for example the + (I found this out as well when testing with a + sign). The options then are to either always allow all email chars
or have two statics strings (one with email, one with 'normal') or change the templates.
 services.AddIdentity<ApplicationUser, IdentityRole>(options =>
    {
        //Below UserOptions is my own class
        options.User.AllowedUserNameCharacters = UserOptions.EmailAllowedCharacters;
    })

How about we just add + since that is fairly common, users can always add whatever additional characters they really want?

@HaoK that would fix 90+% of all scenarios I think.

Works for me!

it is, thanks!


Update terms-of-service.php

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

Summary by CodeRabbit

New Features

Updated the hyperlink for the Privacy Policy in the Terms of Service for improved clarity and accessibility.

@gstraccini help

@gstraccini help


CI: run tests on aarch64

Run the integration tests on aarch64 in the same way as they are run on x86_64.
The orchestration should be the same, but we may need to teach some tests to opt-out of certain cases depending on the architecture (not all image types are available everywhere).
To the extent schutzbot hardcodes any policies about which tests to run, this should be pushed down into the test-package to make this easier.

Once #826 is merged, we should be able to use Fedora's aarch64 cloud image and the Cloud Access image for RHEL 8.2 aarch64 in AWS. The support for the new Graviton instances is in the latest version of the AWS SDK + EC2 plugin for Jenkins.
This will likely require a few tweaks to get package names right and test manifests working.

Oof, we have to do qemu-img converts with one subroutine per https://bugs.launchpad.net/qemu/+bug/1805256
Command:

qemu-img convert -f qcow2 -O qcow2 ./disk01.qcow2 ./output.qcow2

Hangs indefinitely approximately 30% of the runs.

----

Workaround:

qemu-img convert -m 1 -f qcow2 -O qcow2 ./disk01.qcow2 ./output.qcow2

Run "qemu-img convert" with "a single coroutine" to avoid this issue.

Hm, well, this is sub-optimal. I guess this should go in as a fix to osbuild? @gicmo @dvdhrm?

I ended up adding this to osbuild-composer's image-info script temporarily: https://github.com/osbuild/osbuild-composer/pull/882/commits/ac360f821993956c148ddffef889ffc694f5209c

Hm, well, this is sub-optimal. I guess this should go in as a fix to osbuild? @gicmo @dvdhrm?

Yes, totally. I guess someone should file an issue there? :D

@gicmo how about an issue and a PR? https://github.com/osbuild/osbuild/pull/483

We're currently running as many tests as we can on aarch64 and Ondrej is experimenting to see if we can enable a few more.
That said there are limitations with the existing environment: no VPN, no nested virt which we are aware of and tracking internally. I am closing this issue. In the case something is missing and can be added let's open a more specific issue for it.


fix: README.md todo

TLDR;

[] not working
[ ] working

Thanks for this PR. There is a long list of todo i will update soon.


Fix linting issues introducd by #14

I've fixed the original linting issue (was a missing blank line). I'm seeing a few strange issues which appear to be caused by the ecosystem. Right now I'm bumping up against this issue:
https://github.com/psf/black/issues/3447
Which appears to be because there isn't a version of black which works with 3.10 and 3.11. I'm reading around the issue now.
The python 3,12 issue is because of the removal of distutils from the standard library, which has been deprecated since 3.10.  I'll look at that when I've resolved the issue with black.

I've just realised how we got here. Github actions is configured to run on pull_request and on push, but push is filtered to the main branch. This means that CI runs when a PR is raised, but not when a PR is updated. I'll tweak that now.

Great- that's resolved everything except the 3.12 build. Am looking at that now.

I have at least managed to replicate it locally:
[rvodden:~/Source/maaspower]└2 [maaspower] fix_linting(+3/-1)+* 2s ± pipenv install --python 3.12 --dev --skip-lock
The flag --skip-lock has been reintroduced (but is not recommended).  Without the lock resolver it is difficult to manage multiple package indexes, and
hash checking is not provided.  However it can help manage installs with current deficiencies in locking across platforms.
Creating a virtualenv for this project...
Pipfile: /home/rvodden/Source/maaspower/Pipfile
Using /home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/bin/python3 (3.12.0) to create virtualenv...
⠼ Creating virtual environment...created virtual environment CPython3.12.0.final.0-64 in 353ms
  creator CPython3Posix(dest=/home/rvodden/.local/share/virtualenvs/maaspower-Z0oYyxSK, clear=False, no_vcs_ignore=False, global=False)
  seeder FromAppData(download=False, pip=bundle, via=copy, app_data_dir=/home/rvodden/.local/share/virtualenv)
    added seed packages: pip==23.3.1
  activators BashActivator,CShellActivator,FishActivator,NushellActivator,PowerShellActivator,PythonActivator

✔ Successfully created virtual environment!
Virtualenv location: /home/rvodden/.local/share/virtualenvs/maaspower-Z0oYyxSK
Installing dependencies from Pipfile...
[pipenv.exceptions.InstallError]: Obtaining file:///home/rvodden/Source/maaspower (from -r /tmp/pipenv-hflwope7-requirements/pipenv-uo28p775-reqs.txt (line 1))
[pipenv.exceptions.InstallError]:   Installing build dependencies: started
[pipenv.exceptions.InstallError]:   Installing build dependencies: finished with status 'done'
[pipenv.exceptions.InstallError]:   Checking if build backend supports build_editable: started
[pipenv.exceptions.InstallError]:   Checking if build backend supports build_editable: finished with status 'done'
[pipenv.exceptions.InstallError]: ERROR: Exception:
[pipenv.exceptions.InstallError]: Traceback (most recent call last):
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/cli/base_command.py", line 180, in exc_logging_wrapper
[pipenv.exceptions.InstallError]:     status = run_func(*args)
[pipenv.exceptions.InstallError]:              ^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/cli/req_command.py", line 245, in wrapper
[pipenv.exceptions.InstallError]:     return func(self, options, args)
[pipenv.exceptions.InstallError]:            ^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/commands/install.py", line 377, in run
[pipenv.exceptions.InstallError]:     requirement_set = resolver.resolve(
[pipenv.exceptions.InstallError]:                       ^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/resolution/resolvelib/resolver.py", line 76, in resolve
[pipenv.exceptions.InstallError]:     collected = self.factory.collect_root_requirements(root_reqs)
[pipenv.exceptions.InstallError]:                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/resolution/resolvelib/factory.py", line 513, in collect_root_requirements
[pipenv.exceptions.InstallError]:     reqs = list(
[pipenv.exceptions.InstallError]:            ^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/resolution/resolvelib/factory.py", line 474, in _make_requirements_from_install_req
[pipenv.exceptions.InstallError]:     cand = self._make_candidate_from_link(
[pipenv.exceptions.InstallError]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/resolution/resolvelib/factory.py", line 190, in _make_candidate_from_link
[pipenv.exceptions.InstallError]:     self._editable_candidate_cache[link] = EditableCandidate(
[pipenv.exceptions.InstallError]:                                            ^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/resolution/resolvelib/candidates.py", line 322, in __init__
[pipenv.exceptions.InstallError]:     super().__init__(
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/resolution/resolvelib/candidates.py", line 157, in __init__
[pipenv.exceptions.InstallError]:     self.dist = self._prepare()
[pipenv.exceptions.InstallError]:                 ^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/resolution/resolvelib/candidates.py", line 226, in _prepare
[pipenv.exceptions.InstallError]:     dist = self._prepare_distribution()
[pipenv.exceptions.InstallError]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/resolution/resolvelib/candidates.py", line 332, in _prepare_distribution
[pipenv.exceptions.InstallError]:     return self._factory.preparer.prepare_editable_requirement(self._ireq)
[pipenv.exceptions.InstallError]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/operations/prepare.py", line 696, in prepare_editable_requirement
[pipenv.exceptions.InstallError]:     dist = _get_prepared_distribution(
[pipenv.exceptions.InstallError]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/operations/prepare.py", line 71, in _get_prepared_distribution
[pipenv.exceptions.InstallError]:     abstract_dist.prepare_distribution_metadata(
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/distributions/sdist.py", line 52, in prepare_distribution_metadata
[pipenv.exceptions.InstallError]:     self.req.isolated_editable_sanity_check()
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/req/req_install.py", line 544, in isolated_editable_sanity_check
[pipenv.exceptions.InstallError]:     and not self.supports_pyproject_editable()
[pipenv.exceptions.InstallError]:             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_internal/req/req_install.py", line 247, in supports_pyproject_editable
[pipenv.exceptions.InstallError]:     return "build_editable" in self.pep517_backend._supported_features()
[pipenv.exceptions.InstallError]:                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_vendor/pyproject_hooks/_impl.py", line 153, in _supported_features
[pipenv.exceptions.InstallError]:     return self._call_hook('_supported_features', {})
[pipenv.exceptions.InstallError]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_vendor/pyproject_hooks/_impl.py", line 321, in _call_hook
[pipenv.exceptions.InstallError]:     raise BackendUnavailable(data.get('traceback', ''))
[pipenv.exceptions.InstallError]: pipenv.patched.pip._vendor.pyproject_hooks._impl.BackendUnavailable: Traceback (most recent call last):
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/envs/maaspower/lib/python3.12/site-packages/pipenv/patched/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 77, in _build_backend
[pipenv.exceptions.InstallError]:     obj = import_module(mod_path)
[pipenv.exceptions.InstallError]:           ^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "/home/rvodden/.pyenv/versions/3.12.0/lib/python3.12/importlib/__init__.py", line 90, in import_module
[pipenv.exceptions.InstallError]:     return _bootstrap._gcd_import(name[level:], package, level)
[pipenv.exceptions.InstallError]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 1381, in _gcd_import
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 1354, in _find_and_load
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 1304, in _find_and_load_unlocked
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 1381, in _gcd_import
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 1354, in _find_and_load
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 1325, in _find_and_load_unlocked
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 929, in _load_unlocked
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap_external>", line 994, in exec_module
[pipenv.exceptions.InstallError]:   File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
[pipenv.exceptions.InstallError]:   File "/tmp/pip-build-env-wdxmwq35/overlay/lib/python3.12/site-packages/setuptools/__init__.py", line 10, in <module>
[pipenv.exceptions.InstallError]:     import distutils.core
[pipenv.exceptions.InstallError]: ModuleNotFoundError: No module named 'distutils'
ERROR: Couldn't install package: {}

It looks like pipenv has a bug which means that pipenv lock doesn't work when it's executing under python 3.12. We force pipenv lock to be run every time by using the --skip-lock option. This is suboptimal IMHO as it means that we're not sure which versions of our dependencies are employed in each build. I've removed --skip-lock and now the build is succeeding. Have also raised this issue with pipenv: https://github.com/pypa/pipenv/issues/6010 ; however I suggest that even when this is resolved, we continue to run without --skip-lock`
@gilesknap over to you - comments, as always, more than welcome.

OK something odd is going on with pipenv and Python 3.12. I've raised this with pipenv: https://github.com/pypa/setuptools/issues/3661 and temporarily removed 3.12 from the build matrix (our code works under 3.12, so I've not removed it from setup). I'll keep an eye on that bug and raise a new PR when its resolved.
@gilesknap over to you - comments, as always, more than welcome.

(as the 3.12 action is no longer running this will no longer succeed, so that check will always show as failed - you'll need to force merge it if you're happy

Thanks @rvodden. Since creating this project the 'skeleton' framework it is based on has moved on from pipenv which is a bit dead now. We just use pip these days.
So I'll take an action to get this project updated to the latest skeleton framework. I'm really embedded in another project right now so will get to this in a few weeks.

Great stuff. I've got some ideas about future enhancements , and I've got a few engineers I can potentially enlist for a bit of help. How would you like to discuss these? If you don't have an existing pattern, I suggest that I open a GitHub discussion and we can noodle around on that ahead of opening issues?

Sure. Let's use this https://github.com/gilesknap/maaspower/discussions/18


address state machine PR comments

This should go with my previous PR, but since the repo mode changed from private to public i was not able to make changes to that PR.

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


OrderBy and Limit - wrong order?

In the GratefulDeadConcerts database (Version 2.1.4) the query
select expand(out('sung_by')) from v order by name desc

calculates the result (1):

rid
name

9:156 | Weir_Hart
9:61 | Weir
... | ...
When adding a limit
select expand(out('sung_by')) from v order by name desc *limit 2*

i expected to receive the first two records from (1). Instead the query returns

rid
name

9:74 | Spencer_Davis
9:8 | Garcia
Is this a bug or is there an error in the query?

Hi @andreashannig
which OrientDB version are you using?
I fixed a similar issue a few days ago, could you please try again with latest release?
Thanks
Luigi

Hi @luigidellaquila,
i'm using version 2.1.4 which i downloaded from the orientdb homepage this morning.
(I found the problem in my current application and i wanted to check, if i can reproduce the issue in the GratefulDeadConcerts database)
Best,
Andreas

Hi @andreashannig
you're right, my bad, I fixed it after 2.1.4 was released, so it's still in snapshot.
If you have a chance to build from sources, you can try it on your won. Anyway I'll do it here
Thanks
Luigi

Hi @andreashannig
no way, it was a different issue, I have to verify it in depth.
In the meantime, as a work-around, you can use this:
select from (select expand(out('sung_by')) from v order by name desc) limit 2

Thanks
Luigi

Hi @luigidellaquila,
many thanks for your quick response, - and thank you for the workaround.
Best regards
Andreas

Hi @andreashannig
it's fixed in 2.1.x branch, now I'm porting it to develop (2.2)
Thanks
Luigi

fixed also in develop branch.
Closing, please reopen if I missed something
Thanks
Luigi

Hello everybody!
just FYI, I've also found a weird behaviour which I think is related to what @andreashannig  reported (copying from http://stackoverflow.com/questions/33239118/orientdb-weird-behaviour-of-limit?noredirect=1#comment54283435_33239118)
the query select expand(in()) from Employee returns 0 item(s) found. Query executed in 0.0 sec(s)., while the query select expand(in()) from Employee limit 1000 returns 52 item(s) found. Query executed in 0.062 sec(s).
Unfortunately I cannot share the data I'm using, however, my Employee class has > 15k instances of which only ~1400 have an in-edge.

Hi @albertotonon
you have two problems here: the first one was a wrong behavior of expand() operator when combined with limit (fixed in 2.1.x branch), the second one is that you are probably executing the query from Studio, that automatically applies a LIMIT 20. Try to change this default from the lower right corner of the query panel, setting it to 1000, you will probably see the same results


Update the release 0.0.2 deployment data

Update the artifact https://github.com/kabanero-io/kabanero-operator/releases/download/0.0.2/kabanero-operators.yaml
to:

use the kabanero-operator:0.0.2 tag instead of latest
use the proper namespacing

Then reattach the update to the release and notify other team members (such as doc)

Done.


Implemented transform-inline-consecutive-adds plugin.

This plugin inlines consecutive attribute definitions.

This looks great! Couple things:

Can we utilize computed properties? let x = {}; x[y] = 1; → let x = { [y]: 1 }
Should we look into comma operator too? let x = { }; x.a = 1, x.b = 2
Can we make it work with arrays too?

So, we aren't optimizing this, right ?
var a = {};
a.b = {};
a.b.x = 1;
a.b.y = 2;
a.c = 3;

@boopathi not currently -- fixing bugs in the simple version right now:
e.g.
var a = {};
a.a = 4;
a.b = foo();
...
function foo() {
  console.log(a);
  return 0;
}

Naturally, there may be more levels of indirection between a.b and foo, and we can't inline a.b in these cases. This is similar to that other PR where we need to resolve function calls in the rval recursively.

@kangax:

computed properties should be ok, since I think we can just reference check the computation to make sure there aren't bad side effects like:

var foo = {};
foo.a = 4;
foo[bar()] = 5;
...
function bar() {
  console.log(foo);
  return 0;
}

this is possible as well, if we only take consecutive statements of form:

statement => assignment, assignments
assignment => id1.id2 = expr | none

This can also work for arrays (probably easier since there are no lvals to check in statements like foo.push(bar)).

Awesome, thanks!
Let's just rename the plugin to "transform-consecutive-property-assignments" since "attributes" usually refer to something else in ES

@boopathi mind taking a look at the implementation details?

Sure ... I'll take a look tomorrow.

This should be added to preset. But I guess, this was forked before preset options was merged to master. So we can merge this and I can add it to preset separately or @shinew, mind adding it to preset with some option name ?

LGTM from me now, @boopathi please merge if all good on your end

@boopathi I think I merged the plugin with the preset wrongly.
@kangax working on the array property-based assignments right now -- do you know how to make a copy of an AST node?

@shinew Sorry, I accidentally commented directly on your last commit — https://github.com/babel/babili/commit/5c85e78 Can you see the comments?

@shinew could you check if our size benchmark reports any positive size differences with this transform?

@shinew could you please also fix the tests and we should be ready to go

@kangax I tried running the benchmark (./scripts/benchmark.js<EMAIL_ADDRESS>react/dist/react.js), but I got an error (below). This was on master.

/Users/shinew/dev/babili/node_modules/babel-core/lib/transformation/file/index.js:600
      throw err;
      ^

TypeError: unknown: Property right of AssignmentExpression expected node to be of a type ["Expression"] but instead got undefined
    at Object.validate (/Users/shinew/dev/babili/node_modules/babel-types/lib/definitions/index.js:109:13)
    at validate (/Users/shinew/dev/babili/node_modules/babel-types/lib/index.js:511:9)
    at Object.builder (/Users/shinew/dev/babili/node_modules/babel-types/lib/index.js:472:7)
    at liftDeclaration (/Users/shinew/dev/babili/packages/babel-plugin-transform-merge-sibling-variables/lib/index.js:31:29)
    at PluginPass.ForStatement (/Users/shinew/dev/babili/packages/babel-plugin-transform-merge-sibling-variables/lib/index.js:44:9)
    at newFn (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/visitors.js:276:21)
    at NodePath._call (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:76:18)
    at NodePath.call (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:48:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:105:12)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitMultiple (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:103:17)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:190:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitSingle (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:108:19)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:192:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitMultiple (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:103:17)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:190:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitSingle (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:108:19)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:192:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitMultiple (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:103:17)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:190:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitSingle (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:108:19)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:192:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitMultiple (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:103:17)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:190:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitMultiple (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:103:17)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:190:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitSingle (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:108:19)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:192:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitSingle (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:108:19)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:192:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)
    at TraversalContext.visitQueue (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:150:16)
    at TraversalContext.visitMultiple (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:103:17)
    at TraversalContext.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/context.js:190:19)
    at Function.traverse.node (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/index.js:114:17)
    at NodePath.visit (/Users/shinew/dev/babili/node_modules/babel-traverse/lib/path/context.js:115:19)

Will update the test suite for this plugin.

Ok, based on the benchmark error, looks like we have an issue with recently implemented var lifting behavior /cc @vigneshshanmugam

Damn it, forgot to squash before merging. @hzoo @boopathi is there a way to squash now that it's merged?

Published to NPM

I squashed. Take a pull git pull --rebase


fix memory leak at JNI layer, and trigger partition cleanup by Spark

delet[] for Mat byte array
consume all examples in a partition

+1


How to add filter on headers and edit inline of datatable

I try to maker filter on header of every column and edit inline of datatable but don't know how to do this

Hi @amasahara
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you