PvP: исключение в мета-действии

[ERROR 2015-06-10 12:41:21,184 workers 25905] Exception in worker "<the_tale.game.workers.logic.Worker object at 0x7f41e560ac90>"
Traceback (most recent call last):
  File "/home/tie/repos/mine/dext/dext/common/amqp_queues/workers.py", line 143, in process_cmd
    cmd(**cmd_data)
  File "/home/tie/repos/mine/the-tale/the_tale/game/workers/logic.py", line 118, in process_register_account
    self.storage.load_account_data(account)
  File "/home/tie/repos/mine/the-tale/the_tale/game/logic_storage.py", line 40, in load_account_data
    self._add_hero(hero)
  File "/home/tie/repos/mine/the-tale/the_tale/game/logic_storage.py", line 88, in _add_hero
    self.add_action(action)
  File "/home/tie/repos/mine/the-tale/the_tale/game/logic_storage.py", line 131, in add_action
    self.add_meta_action(get_meta_action_by_model(MetaAction.objects.get(id=action.meta_action_id)))
  File "/usr/local/lib/python2.7/dist-packages/django/db/models/manager.py", line 127, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/django/db/models/query.py", line 334, in get
    self.model._meta.object_name
DoesNotExist: MetaAction matching query does not exist.

Закрыто в польщу gh-1651


Small examples of unfold on vprops vs props, scenario for better error message

See https://github.com/FStarLang/steel/blob/nik_dice/share/steel/examples/pulse/bug-reports/UnfoldPure.fst

Thanks for the report!
Now, instead of just failing with "Impossible", the error message is:
UnfoldPure.fst(49,10-49,22): (Error) `fold` and `unfold` expect a single user-defined predicate as an argument, but pure (
        Prims.op_GreaterThan x 2) is a primitive term that cannot be folded or unfolded


'ziti/ziti.h' file not found

Hello, where is the Ziti library? How can I use the things in lib?For example, ZitiEnroller（Cannot find type 'uv_loop_t' in scope）. I feel like there is a lack of key libraries. Please let me know. Thank you very much

Hi. ziti-sdk-swift uses https://github.com/openziti/ziti-sdk-c, which is actually pulled in as a transitive dependency through https://github.com/openziti/ziti-tunnel-sdk-c. ziti-tunnel-sdk-c is referenced by this project as a git submodule in the deps/ subdirectory. Most of this info is in the build instructions, but here's a little more detail in case it helps:

You need to get submodules of the ziti-sdk-swift repo. You can do this when cloning with the  --recurse-submodules option. If you've already cloned then you can run git submodule update --recursive to get them.
Once your deps/ subdirectory contains ziti-tunnel-sdk-c you can build it using build_all.sh in this repo. Be sure to read the requirements section in build instructions. Specifically you'll need to have cmake and ninja installed (I use homebrew for these), and the VCPKG_ROOT environment variable exported and set to the path of a fetched and bootstrapped (built) clone of https://github.com/microsoft/vcpkg. (If you're running on Apple Silicon then you'll need to export andVCPKG_FORCE_SYSTEM_BINARIES to any value to satisfy vcpkg.)

Thank you very much for your help. I have successfully started the project.
I have reviewed the Building section below in detail and listened to your suggestions. I have installed cmake, ninja, vcpkg, and pkg-config respectively, and modified the VCPKG_ROOT in the files under deps.
I'm M1's computer, but I couldn't find VCPKG_ FORCE_ SYSTEM_BINARIES Where is set, it has not been modified.
VCPKG_ROOT have a problem. After installing it, I couldn't modify ENV and couldn't find the corresponding cmake, so I added the information about VCPKG_ROOT in the file uses the absolute path of files under vcpkg, and then goes to/bin/sh build_ all.sh, finally compiled successfully.
/bin/sh build_all.sh Many times during, I encountered Failed to connect to github.com port 443. Just turn off the agent and wait for a long time. Don't worry, I spent the whole morning, and finally succeeded
Thank you,
ZHANGFENG

Ok then. Thanks for letting me know! I'll watch this issue for any more questions that might come up. If you're all set thenljust let me know that it's ok to close this issue. Thanks!


Fixed undefined behavior when using TLS

The mutable reference created by UnsafeCell stayed alive across
the recursive call to inner_fn_call, which resulted in a mutable alias.
To ensure that this kind of issue results in a panic rather than
undefined behavior, I've switched to using a RefCell instead of an
UnsafeCell. The RefMut obtained from the RefCell is explicitly dropped
before we make the recursively call, ensuring that at most one is alive
at any time.

Good catch! I don't think it could cause problems as currently written, but it's always better to remove unsafe code when possible, especially that exposes a borrow rule violation.


Create PL-meetup-EthCC.md

@WinsCodes New Event - PL Meetup @ EthCC

@laeccles This request has been completed.


Feature/ab#8186 global search

https://github.com/bcgov/Unity/assets/97022120/22287754-ec97-4761-b487-f28f9c2f79fb


Fix all issues with the setup script once and for all

Clearly this is better than providing a script at all given that we're constantly getting complaints about it.
Most people don't use this anyway.

Since you removed the script, may be it would be good to document on the project wiki how to actually "install" things manually then? I.e. where to place / symlink the library (such as 32-bit and 64-bit versions)? Not everyone is familiar with it, especially when 32-bit location is confusingly called syswow64 and 64-bit location - system32 in wow64 prerix, and just system32 in 32-bit prefix.

@shmerl feel free to open a PR. Worst that can happen is that it gets closed again. 🙂

FYI: one more way to use DXVK:
export WINEPREFIX=/path/to/wineprefix
export WINEDLLOVERRIDES='*d3d9,*d3d11,*d3d10core,*dxgi=n'
export WINEPATH='Z:\\path\\to\\dxvk\\x64;Z:\\path\\to\\dxvk\\x32'

But, for this to work one should (re)move original d3d9.dll, d3d11.dll, etc. from wine prefix.
Likely an setup and/or launcher script needed for this trick, but it may come handy in cases like switching between multiple dxvk versions/builds for testing (by changing WINEPATH). Can't recommend this for regular users or for use on important wine prefixes.
p/s: Same can be done for d3dx*.dll, install all of them into separate wine prefix, move somewhere to /path/to/d3dx/x{32,64}, add to overrides and path (; separator for both) ...

Oh,  I didn't know about WINEPATH: https://gitlab.winehq.org/wine/wine/-/blob/master/loader/wine.man.in#L204
It's also a neat trick that it can mix 32-bit and 64-bit libraries in the same string.

Should you use WINEDLLPATH instead for the above?

Last time I checked WINEDLLPATH only handle builtin dlls (path to wine itself).

@shmerl as important as installing, is how one would remove dxvk then?
from the scrip it used to rename the original dlls to *.old before replacing them, and renaming them back when unistalling, perhaps this also should be mentioned.
I think it would also be good mention how one does that entirely from the command line without the need of winecfg (or maybe not?).
for example:
wine reg add 'HKEY_CURRENT_USER\Software\Wine\DllOverrides' /v d3d9 /d native /f

or
wine reg delete 'HKEY_CURRENT_USER\Software\Wine\DllOverrides' /v d3d9 /f

I think using .old files isn't a good approach, since by the time you might want to un-install things, they'd be outdated. I think the method with deleting and using wineboot -u that readme alraedy mentions is a better method.


Outputs version 2 instead of 3

My protobuf file starts with
syntax = "proto3";
but the generated swift file seems to contain version 2:
fileprivate struct _GeneratedWithProtocGenSwiftVersion: SwiftProtobuf.ProtobufAPIVersionCheck {
  struct _2: SwiftProtobuf.ProtobufAPIVersion_2 {}
  typealias Version = _2
}

This is a bit of a problem because I wrote my definition without the optional keyword, seeing as it's supposed to be redundant/implicit now, and none of the generated Swift struct members are optional (?).
I'm developing an iOS app on macOS 12.4 with Xcode 13.3.1 and Swift 5. I installed the protobuf tools with homebrew, the versions are:
protobuf: 3.19.4
swift-protobuf: 1.19.0
grpc-swift: 1.7.3
I generate the swift files from a Makefile with this rule:
	protoc angel.proto \
		--plugin=/opt/homebrew/bin/protoc-gen-swift \
		--swift_opt=Visibility=Public \
		--swift_out=$(SWIFT_OUTPUT) \
		--plugin=/opt/homebrew/bin/protoc-gen-grpc-swift \
		--grpc-swift_opt=Visibility=Public \
		--grpc-swift_out=$(SWIFT_OUTPUT)

There are no error messages or warnings. I've also tried the command without the grpc options, but it hasn't made any difference.
I'm generating Go code from the same files, and the struct tags include "proto3", so I think that's using the correct version. I was originally going to write the app with Flutter, and the dart bindings seemed OK, because the struct members were optional.

The Swift generated fields would never be Optionals: that's not how they're implemented. Per the proto3 spec:

When a message is parsed, if the encoded message does not contain a particular singular element, the corresponding field in the parsed object is set to the default value for that field.

The _2 version here is the version of the SwiftProtobuf library API.  It has nothing to do with the proto source file version.


Make getAccessToken & getProfile optional

It would be great if it was possible to pass down a param to only execute onLogin without calling getAccessToken and getProfile.
Essentially all that we need is the auth code that is coming back from the original authorization window, then we handle the rest server side, so no need to exchange the code and grab profile data. Will be great if you can make an option for that!

Thank you. I gonna do that soon.

This feature is complete, and I will release it soon

Awesome thank you 🙏🏼

🔖 2.5.5 version has been published

@cuongdevjs I saw that now the option you added was to make getProfile optional, but it still tries to exchange the code for an accessToken every time. It would be great if you can make that optional as well. All the we need is the code from the original authorization response, we then exchange the code for an authToken server side. We don't want to use any CORS bride, so prefer to handle it all server side.

oh, yes, you just need to pass props isOnlyGetToken to your component.

@cuongdevjs Well but that then still tries to exchange the code for an auth_token? We only need the code, no need to send any request to exchange the code for the auth_token in the users frontend.

ok. What components are you mentioning?

As far as I can tell, most of the components work the same:

User performs on-click, opens the authorization window and authorizes -> CODE
The Code is then being exchanged for an auth_token -> AUTH_TOKEN
The auth_token is then being used to fetch the profile of the user -> PROFILE

Before it was always doing all 3 steps. Now you made the new isOnlyGetToken parameter, which now is doing only step #1 and step #2. However, we only need step #1. All that we need is for example for the Twitter API to return us the CODE. We then handle the exchange ourselves server-side.

Ok. I got it. I will add one more prop to do only step #1.

Awesome thank you!

I've completed it, and it will be released soon.

Perfect thank you @cuongdevjs - please let me know when that has been released!

It has been released. please upgrade to 2.5.6 and then add prop isOnlyGetCode (read more on README.md)


Failure to import APIM Instances

The most recent release of this provider has me very excited and I have been experimenting with its usage for APIM Instances, but it appears that I cannot use the import command for the "anypoint_apim_mule4" resources.
Problem
It appears as though the terraform import anypoint_apim_mule4.<resource> <id> command generates a malformed URL for the APIM Instance in question, thereby resulting in a "NotFoundError" from Anypoint.
Symptoms
For the purposes of these symptoms, I am using dummy GUIDs and IDs. Be aware that I am using (to the best of my knowledge) accurate GUIDs and IDs for the platform I am working with and that these examples will need to be modified to use other values to work on an arbitrary machine. Despite this, the symptoms described will appear with these stub values.
I have a Terraform configuration with the following content describing an APIM Instance:
resource "anypoint_apim_mule4" "hello-world" {
    env_id = "12345678-90ab-cdef-1234-567890abcdef"
    org_id = "12345678-90ab-cdef-1234-567890abcdef"

    asset_group_id = "12345678-90ab-cdef-1234-567890abcdef"
    asset_id       = "my-api"
    asset_version  = "1.0.0"

    endpoint_uri   = "https://hello.world/"
}

Given this, when I attempt to run an import command such as terraform import anypoint_apim_mule4.hello-world 12345678 (the 12345678 can even be invalid such as abc), I get the following response:
% terraform import anypoint_apim_mule4.hello-world 12345678
anypoint_apim_mule4.hello-world: Importing from ID "12345678"...
anypoint_apim_mule4.hello-world: Import prepared!
  Prepared anypoint_apim_mule4 for import
anypoint_apim_mule4.hello-world: Refreshing state... [id=12345678]
╷
│ Error: Unable to get API manager's mule4 instance 12345678
│
│ {"name":"NotFoundError","message":"\"Organization\" could not be found using \"id: environments\""}
╵

Note that I can use the terraform plan command to get valid output:
% terraform plan

Terraform used the selected providers to generate the following execution plan.
Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # anypoint_apim_mule4.hello-world will be created
  + resource "anypoint_apim_mule4" "hello-world" {
      + asset_group_id              = "12345678-90ab-cdef-1234-567890abcdef"
      + asset_id                    = "my-api"
      + asset_version               = "1.0.0"
      + audit                       = (known after apply)
      + autodiscovery_instance_name = (known after apply)
      + deprecated                  = false
      + endpoint_audit              = (known after apply)
      + endpoint_deployment_type    = "HY"
      + endpoint_id                 = (known after apply)
      + endpoint_type               = (known after apply)
      + endpoint_uri                = "https://hello.world/"
      + env_id                      = "12345678-90ab-cdef-1234-567890abcdef"
      + id                          = (known after apply)
      + is_public                   = (known after apply)
      + last_active_date            = (known after apply)
      + last_updated                = (known after apply)
      + master_organization_id      = (known after apply)
      + order                       = (known after apply)
      + org_id                      = "12345678-90ab-cdef-1234-567890abcdef"
      + product_version             = (known after apply)
      + status                      = (known after apply)
      + technology                  = (known after apply)
    }

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

────────────────────────────────────────────────────────────────────────────────

Note: You didn't use the -out option to save this plan, so Terraform can't
guarantee to take exactly these actions if you run "terraform apply" now.

Theories
The output mentioning that the organisation ID be searched for is "environments", which is very odd. However, the line in the Anypoint Go Client that describes this URL template has /environments as the immediate next path element. My theory is that the org_id is somehow blanked out and so Anypoint just uses the next path element available. This looks feasible to me as the Anypoint Go Client has a function called parameterValueToString that contains logic to blank out the string under certain conditions.
I'm not super familiar with Go or this framework, so I would take any of this analysis with a grain of salt.
Technical Information
I am using the following Terraform providers for my testing:
terraform {
  required_providers {
    anypoint = {
      source = "mulesoft-anypoint/anypoint"
      version = "1.6.0"
    }
  }
}

provider "anypoint" {}

Anypoint Credentials are being passed in via the exported variables ANYPOINT_CLIENT_ID and ANYPOINT_CLIENT_SECRET and have been granted permission to the business group and environment in question.
When running terraform version I get the following output:
Terraform v1.7.3
on darwin_arm64
+ provider registry.terraform.io/mulesoft-anypoint/anypoint v1.6.0

Hi @TomChapple
for your example, you have to use a composed id. the resource APIM is located under other resources and the URL cannot be generated with only the id of the APIM.
here's an example:
# In order for the import to work, you should provide a ID composed of the following:
#  {ORG_ID}/{ENV_ID}/{API_ENV_ID}

terraform import \
  -var-file params.tfvars.json \    #variables file
  anypoint_apim_mule4.mule \                #resource name
  aa1f55d6-213d-4f60-845c-201282484cd1/7074fcdd-9b23-4ab6-97r8-5db5f4adf17d/19218070    #resource ID

replace with your own values.


docs: update CONTRIBUTING.md to point to the correct file

Fix broken CONTRIBUTING.md link in the examples subfolder to point to correct file located in root.

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


generate service must account for revision tokens

TL;DR
The generate service attempts to revise keys, but that is always failing since it doesn't handle revision tokens.

/assign


just preview local file system code and without backend

suggestion
new a branch, it can preview current machine current workspace code, and  without backend code-server. it only read local file.

similar project: https://github.com/conwnet/github1s
but that project can only open github repo files.

I don't think this is possible without a backend - mostly because VSCode relies on natively running language servers for Intellisense/auto-complete, and because local storage access is pretty limited in a web browser.

Yeah, I think @oxy is right. Probably not possible. Though someone internally was exploring a project similar to github1s.
https://coder1s.com/cdr/code-server
But it only works with GitHub repos.

but that project can only open github repo files.

What kind of projects would you like to open? GitLab? Something else?

@oxy @jpinnix
ok, First of all, I can tell you that many js lib can preview current system files without the backend(php, nodejs, java and so)
Let me give you some examples.

https://github.com/surfcodes/surf

this project can open github, gitlab, and any others file system.

baidu minder project

It is a javascript project, and without backend. it need a static server, like nginx, apache.

If it need nodejs backend, that's ok. I want it can package to a Electron app, and it also can start a nodejs server, which we can access it at LAN.

We can code using our windows pc, besides if we share a link(http://<IP_ADDRESS>:8898) to you friends, our friends can also preview our code(not microsoft remote develop).

@oxy @jpinnix

I assume you meant to tag @jsjoeio
But this is an interesting conversation 😃


Fine-grain filtering for Graphite reporting

[Feature request]
I am on a small team attempting to use this library to integrate our Jersey2 project with HostedGraphite.
Because HostedGraphite operates on a quota based on the number of individual "metrics". Currently the GraphiteReporter in this library sends multiple metrics to Graphite as a part of a single Metric.
In order to keep costs under control, but still allow us to measure as many of our REST endpoints as possible, we would like to restrict which of the individual metrics. For example, on a Timer we would prefer to only keep "min", "max", and "mean" for now.
Is this already possible? If not, would it be possible to add this kind of functionality?

Just reopened #178. This is on the roadmap for v4 as it will require some refactoring.


Support wildcards in gradle-home-cache-excludes and gradle-home-cache-includes

Currently these parameters won't work quite as expected if specified as paths with wildcards.

This is not planned


set-security-context flag set to true does not work in tekton-pipelines namespace!

Tekton version:

v0.49.0

Question:

Pod creation in tekton-pipelines namespace is limited，when running pipelineruns, taskrun execution fails!
Do I have to add the containers.securityContext configuration manually?
Or change the namespace labels to pod-security.kubernetes.io/warn: restricted ?

Taskrun:
$ kubectl get taskrun -n tekton-pipelines pipelinerun-pfjr4-git-clone -o yaml

...
status:
  conditions:
  - lastTransitionTime: "2023-06-26T09:12:53Z"
    message: 'pods "pipelinerun-pfjr4-git-clone-pod" is forbidden: violates
      PodSecurity "restricted:latest": allowPrivilegeEscalation != false (container
      "step-clone" must set securityContext.allowPrivilegeEscalation=false), unrestricted
      capabilities (container "step-clone" must set securityContext.capabilities.drop=["ALL"]),
      runAsNonRoot != true (pod or container "step-clone" must set securityContext.runAsNonRoot=true),
      seccompProfile (pod or container "step-clone" must set securityContext.seccompProfile.type
      to "RuntimeDefault" or "Localhost")'
    reason: PodAdmissionFailed
    status: "False"
    type: Succeeded
  ...
  provenance:
    featureFlags:
      AwaitSidecarReadiness: true
      DisableAffinityAssistant: true
      DisableCredsInit: false
      EnableAPIFields: beta
      EnableProvenanceInStatus: true
      EnableTektonOCIBundles: false
      EnforceNonfalsifiability: ""
      MaxResultSize: 4096
      RequireGitSSHSecretKnownHosts: false
      ResultExtractionMethod: termination-message
      RunningInEnvWithInjectedSidecars: true
      ScopeWhenExpressionsToTask: false
      SendCloudEventsForRuns: false
      SetSecurityContext: true
      VerificationNoMatchPolicy: ignore

feature-flags:
$ kubectl get cm -n tekton-pipelines feature-flags -o yaml

apiVersion: v1
data:
  await-sidecar-readiness: "true"
  disable-affinity-assistant: "true"
  disable-creds-init: "false"
  enable-api-fields: beta
  enable-provenance-in-status: "true"
  enable-tekton-oci-bundles: "false"
  enforce-nonfalsifiablity: none
  require-git-ssh-secret-known-hosts: "false"
  results-from: termination-message
  running-in-environment-with-injected-sidecars: "true"
  send-cloudevents-for-runs: "false"
  set-security-context: "true"
  trusted-resources-verification-no-match-policy: ignore
kind: ConfigMap

After changing the namespace labels to pod-security.kubernetes.io/warn: restricted, I successfully created the Pod and checked the Pod manifests to see that set-security-context was in effect and the securityContext was injected into the initContainers, However, the containers are not injected with securityContext.

Thanks for the reply, my problem is solved !
Without modifying namespace labels, you must enable set-security-context and add the securityContext configuration to Task.spec.steps
For example:
edit feature-flags configmap:
$ kubectl patch cm feature-flags -n tekton-pipelines -p '{"data":{"set-security-context":"true"}}'

task:
apiVersion: tekton.dev/v1
kind: Task
metadata:
  name: git-clone
  namespace: tekton-pipelines
spec:
  steps:
    - name: clone
      securityContext:
        allowPrivilegeEscalation: false
        capabilities:
          drop:
            - ALL
        runAsNonRoot: true
        runAsUser: 1001
        seccompProfile:
          type: RuntimeDefault

But I have a question, why not support injecting securityContext for user-defined containers? Is it considered to support this feature?

Hi @AliDev0ps, thanks for your question. set-security-context only applies to init containers and sidecar containers injected by Tekton, not step/sidecar containers specified by the user. So what you are seeing is expected behavior. Here's an example of setting security context on a step and one for the pod. Running taskruns and pipelineruns in the tekton-pipelines namespace is not recommended. Please feel free to reopen if you have additional questions!

Thanks for the reply, my problem is solved !
Without modifying namespace labels, you must enable set-security-context and add the securityContext configuration to Task.spec.steps
For example:

edit feature-flags configmap:
$ kubectl patch cm feature-flags -n tekton-pipelines -p '{"data":{"set-security-context":"true"}}'

task:
apiVersion: tekton.dev/v1
kind: Task
metadata:
  name: git-clone
  namespace: tekton-pipelines
spec:
  steps:
    - name: clone
      securityContext:
        allowPrivilegeEscalation: false
        capabilities:
          drop:
            - ALL
        runAsNonRoot: true
        runAsUser: 1001
        seccompProfile:
          type: RuntimeDefault

But I have a question, why not support injecting securityContext for user-defined containers? Is it considered to support this feature?

But I have a question, why not support injecting securityContext for user-defined containers? Is it considered to support this feature?

Thanks for your question! I think it's unlikely we'd support this feature. Tekton controls the init containers and sidecar containers that it injects, and users have no ability to configure them, so it makes sense for us to set the most restrictive security context possible for those containers. (The only reason it's configurable is that securitycontext features don't work on all platforms, like openshift.) However, Tekton has no knowledge of user containers or what security context would be appropriate for them, and users do have the ability to configure this. Rather than trying to be too opinionated about what security context would be appropriate for user-specified containers, we'd prefer to just let users specify their security context.


Could you please consider adding a way to generate inline critical css as well?

@sanishkr This is a great idea. I haven’t found a good way to do this in Next.js. Any ideas?

@sanishkr Looks like there's a PR up on Next.js to add an experimental flag that does this: https://github.com/vercel/next.js/pull/16539. 🎉

Thanks @taylorbryant ,
for now, I did a setup of my own using styled-components and  twin.macro.


Add support for options in environment variables

This uses yargs' support for providing option values via the command line:  https://github.com/yargs/yargs/blob/main/docs/api.md#envprefix
My use case is that in CI scenarios I want raw output, but locally I want prefixes. I could duplicate all the scripts in my package.json: one with concurrently -r, one without - but I feel that is suboptimal and could lead to mistakes when devs forget to keep them in sync.
With this change, my CI script can now look like:
set -eux
export CONCURRENTLY_RAW=true

npm run check
npm run validate
npm run build-all
# etc..

Coverage remained the same at 98.618% when pulling 72067ede31a645e4fd81b9214487327fb30163cb on aidansteele:patch-1 into 66ed4bfea1b6c0f57e9a87d516a6520639708141 on open-cli-tools:master.

Sure thing, I'll try add those today :)


move subcommands into bin/cmd in the package

With this change the subcommands are moved into a
sub directory under the bin directory so they are
not available in the PATH if the bin directory is
added there, so they can not be called by accident.
The top level scripts to some additional configuration
and setup without it the subcommands can not be
executed in some environments.

One more thing. I tried to run the following command: /build/CodeChecker/bin/cmd/codechecker-analyze -h but I got the following exception:
Traceback (most recent call last):
  File "./build/CodeChecker/bin/cmd/codechecker-analyze", line 20, in <module>
    CodeChecker = spec_from_file_location('CodeChecker', CC).loader.load_module()
AttributeError: 'NoneType' object has no attribute 'loader'

I've updated the commit message.

One more thing. I tried to run the following command: /build/CodeChecker/bin/cmd/codechecker-analyze -h but I got the following exception:

In the current setup the sub commands should not be called directly, they will not work, the problem was that one user called them (because they were in the bin directory so they got into the PATH), do we want to make those sub commands callable?

you mean to make the sub command files non-executable?

I've checked and making the files only non executable (without moving them) but it solves to problem only in bash, in zsh the autocompletion still finds and auto completes the subcommands


[meetings] Created NewComers meeting attendence folder

Notes for Reviewers
This PR fixes #12011

Created a new directory week-of-10-14-2024 under the _meetings directory for weekly Newcomers meeting attendance.
The directory follows the required format (mm/dd/yyyy) for the start of the week.


username/password silently dropped from durable object request urls

Given the following worker:
export default {
  fetch(req, env) {
    let url = new URL(req.url)
    url.username = 'kentonv'
    console.log(url.href) // ie, http://kentonv@localhost:8080/
    return env.do.get(env.do.newUniqueId()).fetch(url, req)
  }
}

export class DO {
  fetch(req) {
    return new Response(req.url)
  }
}

The URL returned by the durable object's response does not contain the username set in the worker, despite serializing correctly (ie, shows up in url.href) inside the worker itself.
It seems as though username and password are being silently dropped, despite being valid URL components. Ideally URLs would be sent as-is, but if for some unfortunate reason workerd strips them intentionally, fetch should fail in the worker itself before reaching the DO (same as protocol errors a la fetch_refuses_unknown_protocols).

Hmm, I'm not sure. If you perform fetch() in a browser and gave it a URL with username/password, the username and password do not remain in the URL as seen on the server. Instead, the Authorization header is filled per HTTP basic auth. In fact, it wouldn't be possible to keep them in the URL since the HTTP protocol doesn't actually transmit the full URL as a single string, it breaks it up and puts all the parts in various places.
I would argue that it is a bug that we don't fill in Authorization. (Assuming the fetch() spec does in fact say we should, at least...)
If we did fill that in, would that help with your use case?

There's also an argument to be made that we should let you pass any string you want to fetch() when it's a worker-to-worker or worker-to-DO call, and that string should just show up unchanged on the receiving end. But I think we tried to implement that once and ran into too many problems.

I think adding an Authorization header would be fine. Personally I'd avoid arbitrary strings, since this would cause the already disparate DO and worker fetch arguments to further diverge from each other... If you want to relax the HTTP format, which I think is a good idea, I'd add another method on the DO altogether that looks more like postMessage.
FWIW, here's what the spec says:

This flag doesn't appear to be implemented in any browser I can find. fetching a URL like the above instead results in the following error, even when credentials is set to include: TypeError: Failed to execute 'fetch' on 'Window': Request cannot be constructed from a URL that includes credentials.

Honestly I prefer the brower's behavior here. We really should not be encouraging putting the username/password into URLs and should instead encourage developers to set the Authorization header explicitly.
Specifically, I'd prefer that this reject/throw<EMAIL_ADDRESS>

FWIW I think credentials = "include" is meant to tell the browser to use the ambient credentials, i.e. whatever the user has previously entered when prompted to log into the target site using HTTP basic auth. Of course, no one actually uses HTTP basic auth anymore and I think it's officially deprecated, even. But it makes sense to me that browser makers did not want to figure out how to reconcile ambient credentials with credentials specified in the URL.
Of course, in Workers, we don't have ambient credentials at all. But I agree we should do what the browsers do and throw an exception. We'll need a compat flag, of course.

Agreed, throwing is probably the safest bet (with a flag a la fetch_refuses_unknown_protocols, as per my original suggestion).
(HTTP basic auth is definitely not deprecated though, just support for username:password@ in URLs.)

Ok, specifically, this is tied to the fact that our fetch implementation is really not compliant with the spec, so we'll include the change here within that bucket. Specifically, if we look at the definition of the new Request(...) constructor, step 5 requires that we parse the input URL and throw if the parsed URL includes credentials.

In our current implementation of Request, we do not parse the input URL at all until immediately before the fetch impl is called.

@jasnell Sounds like you're saying there are two issues:

We parse at fetch(request) time and not new Request() time. (But this only makes a difference if you do fetch(url) without explicitly creating a Request object.)
When we do parse the URL (at fetch() time), we silently ignore credentials, when we should be throwing if they are present.

I could imagine these being two separate fixes with separate compat flags (if we care to fix them).


Rewrite src= path

Currently src= isn't being re-written like href= is.

Fixed

Per our conversation, doesn't seem to be working.


Enable support of HTTPS proxy

Is your feature request related to a problem? Please describe.
NewRelic agent is able to use HTTP/S proxy to send data to NewRelic endpoints, but jfr-daemon is not.
Feature Description
Add environment variables NEW_RELIC_PROXY_HOST and NEW_RELIC_PROXY_PORT to configure usage of HTTP proxy
Describe Alternatives
We can use -Dhttps.proxyHost and -Dhttps.proxyPort java args.
Priority
Nice to have

@alx13  This request is being tracked in #167 now. Thanks for suggesting it!


chore(master): release 1.1.156

:robot: I have created a release beep boop
1.1.156 (2023-08-17)
Miscellaneous

deps: update dependency archiver to v5.3.2 (4695f3b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/keywind/releases/tag/v1.1.156 :sunflower:


[MV] "Cannot read property of 'replace' of undefined." error (plugin command: rpc_replaceRow)

CHECKLIST
I've updated nwjs to 0.49.2 or higher: ✅
I've installed the discord-rpc node module: ✅
There's a node_modules folder in my project directory: ✅
Describe the bug
When using rpc_replaceRow1 OR rpc_replaceRow2, the game will throw an error stating it "Cannot read property of 'replace' of undefined."
To Reproduce
Steps to reproduce the behavior:

Install all pre-requisites. Set a text string in the plugin parameter for Row 1 or 2.
Created a plugin command in an event to replace a certain row (rpc_replaceRow1 OR rpc_replacerow2) followed by a string of text.
Error immediately occurs when the event executes the plugin command.

Expected behavior
Discord RPC should report the default text set within the plugin. Upon using the plugin command "rpc_replaceRow" (1 or 2) it should update according to the text string inputted.
Screenshots

Desktop (please complete the following information):

OS: Windows
Plugin version: 1.2

Additional context
Just bonus content to be patched as well:

Line 208 & 210 are missing "discord" in front of "parameters". Without "discord" in front I got errors, but after filling in the code, it works just fine.

Sorry about that. The previous issue you got was because I copy-pasted the 1.2 source code from MZ version which uses the variable parameters instead of discordParameters.
Yesterday then I just lazily replaced everything starting with parameters[, and didn't remember those two. I'll fix this soon.

No worries! It'll be good to patch out the parameters issue, but the main point of the bug/issue log is something to do a different line of code involving the plugin command.
The discordParameters issue doesn't seem to directly affecr this one...

I've made an update to another branch to see if it works now.
Link: https://github.com/00due/Discord-RPC-for-RMMZ/archive/refs/heads/test.zip

Two things:
First: Error shows up:

Second: The replaceRow1 & replaceRow2 code look to be mixed in code.

I'll see if I can read the code and try do some editing on my behalf.

I updated it once again.
Sorry for all this, but I can't test the MV version, since I only own MZ. Because of that, I don't know if the code will work or not.

Okay, I saw another issue there. I've updated it again.

That's more than okay! I'm happy to keep posting logs if you're happy to keep troubleshooting!


Update colors for darkmode in Other Projects page

When darkmode is activated in the other projects page the colors don't look too good for the heading and the current violet text, so make it a little darker color that will suit darkmode.

Done in commit bcac0c0


I want the "Homeschool" option to exist within the school picker drop down

Acceptance Criteria:

[ ] Add an option into the list of schools for "Homeschool"
[ ] Remove radio button from the list below

notion


Add faces for Elfeed

The default faces that the theme uses for elfeed are the bog standard colors like "#00FF00" that don't follow the rest of the color palette.

#24 Adds the "material" versions of the colors instead of the stock emacs colors (material orange instead of goldenrod, etc) for elfeed.


Cmdlet - ConvertFrom-Base64String

Summary of the new feature / enhancement
I have written this Basic Function to Decode a Base64 Encoded String. However I was hoping that it or something similar to it could be considered for an addition to the builtin Powershell CmdLets/Modules.
Understand, what I have included in the codeblock below in the Proposed technical implementation details is only an example and others may have their own opinions on how to better approach it, I am all for the free exchange of ideas to better this process and hope this simple concept can be included in a future release of powershell.
Proposed technical implementation details (optional)
A CmdLet or set of CmdLets to be added to Powershell that will allow Encoding/Decoding of Strings to and From Base64 and potentially even other Encoding Types. A basic example of this is included in the codeblock below
function ConvertFrom-Base64String {
    [CmdletBinding()]
    param (
        [Parameter(mandatory=$true,ValueFromPipeline=$true)]
        [String]
        $EncodedString
    )
    try {
        $decodedString = [System.Text.Encoding]::UTF8.GetString([System.Convert]::FromBase64String($EncodedString))
        return $decodedString
    }
    catch {
        throw "The provided string is not a valid Base64 encoded string."
    }
}

Hi,

I am all for the free exchange of ideas to better this process

Not all new ideas have to be added to PowerShell itself. The PowerShell Gallery is good place for sharing useful modules.

allow Encoding/Decoding of Strings to and From Base64 and potentially even other Encoding Types

The only type that makes sense to encode with Base64 is a sequence or array of bytes. Base64 is not intended as a way of encoding strings. Places where this is done, for example in HTTP Basic authentication, are the exception not the rule. All you end up with is a string that is 50% bigger. Any further encodings to and from byte sequences can be done with other cmdlets.
So has this been done before? Yes, there is a pair of cmdlets, ConvertTo-Base64String and ConvertFrom-Base64String available as a PowerShell Module. These convert individual records between byte arrays and strings.
There is another pair, ConvertTo-Base64 and ConvertFrom-Base64 which are intended for use in a streaming pipeline, there the entire content is treated as a single encoding. Also available in The Gallery.

Here is the Microsoft written module with those cmdlets. https://www.powershellgallery.com/packages/Microsoft.PowerShell.TextUtility/0.5.0

Since it is often used with Web based requests, that is why im asking.   AWS Also uses it with their AWS CLI for exporting CSV based reports through the CLI which is where I use modules currently, it would just be nice to have it be natively there, so that if i share scripts with others, I don't have to make specific modules required.

so that if we share scripts with others, we don't have to make specific modules required. especially for the Base64Decode.

Alas that approach really doesn't scale. Modern languages are all designed to be extendable and use Maven, NuGet, Pip, npm etc to provide additional functionality that can be shared and avoid reinventing the wheel. PowerShell has PSGallery for that.
Say you get Base64Decode added as a cmdlet to PowerShell 7.5, you won't get it until months away and anything that uses won't be backward compatible with existing PowerShell installations. If the cmdlet is in the Gallery then even PowerShell 5.1 can use it along any other version.
I suggest either make a dependency on any suitable implementation you choose from the Gallery or use the native method invocation as you are using.

My suggestion (best way) for you：
Abandon AWS CLI and use AWS PowerShell:
Manual root directory
https://docs.aws.amazon.com/powershell/index.html
All commands are in:
https://docs.aws.amazon.com/powershell/latest/reference/Index.html
Use the PowerShell script framework I wrote
If you place the script in the 'node_stcript' directory, it will be automatically copied to each controlled machine when first connected to it.

main site: https://gitee.com/chuanjiao10/kasini3000
site mirror: https://github.com/kasini3000/kasini3000 --- Welcome give star to project
win,linux devops automation batch script framework.(It is similar to Puppet，Ansible，pipeline)
Open source, free, cross-platform
English manual:
https://gitee.com/chuanjiao10/kasini3000/blob/master/docs/kasini3000_manual.xlsx
tech：
https://www.cnblogs.com/piapia/p/16534765.html
Features:
Transfer PowerShell objects (. net objects) over both WINRM and SSH between the main control machine and the controlled machine.This means that you can pass object level variables between hybrid clouds.


endless page refresh when running in development mode in Internet explorer

When running the application in development mode in internet explorer the pages keep refreshing somehow again and again. I tested this in different IE versions: 11, 10 and 9.
Every time this message appears and the page is rebuild and refrehsed:
restobonsweb_1  | Client pings, but there's no entry for page: /
restobonsweb_1  | > Building page: /
restobonsweb_1  |  DONE  Compiled successfully in 1946ms3:03:57 PM
restobonsweb_1  |
The next version I am using is: 2.2.0.

@Filipvdb321 can you try the latest version?

@timneutkens , on the latest version it works indeed. Thanks.

👌


DO NOT MERGE: add test pages for vanilla demo for different layers an…

…d how they respond to sizing constraints (or no sizing constraints). Update player styles.css to experiment with sizing behavior.
Added vanilla test pages for:

play ol' video w/hls
mux-video
mux-video + media chrome
mux-player

This was for reference. Solution resolved by different PR.


Accessibility enhancement for videos

Description
Hello, I noticed that the video block, and the responsively embed blocks don't have any option to add a "title" attribute. The title attribute should be included on all iframe elements to improve accessibility. Would it be possible to include this in the next concrete version?
Example
No response

If this were submitted as a pull request we would accept it.

If this were submitted as a pull request we would accept it.

Thanks. I wish I could do that, but I'm not skilled enough with PHP to code that and even less experienced to submit a pull request to core. I was hoping someone else (who also would benefit from this) would find it easy to code.

This has been fixed and will be released in 9.1.0!


VerticalOff twice in list

Describe the bug
A clear and concise description of what the bug is.
If possible attach the JSON info file for your devices.
Hi. In the image you can see that VerticalOff is there twice. Very first and very last. Other values looks like there are only once. Air condition bought and installed today. HA and HACS on latest version.

Expected behavior
If applicable, a clear and concise description of what you expected to happen.
Every option just once.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment details:

Environment (HASSIO, Raspbian, etc): Raspbian HA supervised
Home Assistant version installed: core-2023.3.6
Component version installed: 0.31.4
Last know working version: -
LG device type and model with issue: lg pc12sk
LG devices connected (list):

Output of HA logs
Paste the relavant output of the HA log here.

Additional context
Add any other context about the problem here.

;)

Please try if last release fix the issue.

Issue solved. Thank you for fixing such a low prio thing ;)


Pending functional tests for security-dashboards-plugin

Here is the list of functionalities that are yet to be tested page-wise:
Roles

[ ] Edit
[ ] Duplicate
[ ] Delete

Internal Users

[ ] Sorting by table headers
[ ] Edit
[ ] Duplicate
[ ] Delete
[ ] Search

Permissions

[ ] Search with all 3 filter criteria
[ ] Sorting by table headers
[ ] Create from selection

Tenants

[ ] Search
[ ] Edit
[ ] Duplicate
[ ] Delete
[ ] Private tenant
[ ] View Dashboards link
[ ] View Visualizations link

Audit Log

[ ] Modify General settings
[ ] Modify Compliance settings

Is there a way to know that this list is correct / isn't missing something?  I am hoping for some kind of code coverage metric that we could interrogate and drive upward

[Triage] Lets circle back on these at a case by case basis and see if we should implement them with feature work.


It looks like cr-touch is not connected properly

4 pin is floating

and looks like you don't use all of common ground cables

all goes through pin 7 of sprite cable

From what I can tell, pin 4 isn't even connected on the original official board. Mine doesn't seem to connect to any traces. I don't know enough about the BL/CR touch, but maybe the pin just...isn't necessary for normal functionality?


[BUG] [Remote State] Auto restoring decommission attributes after QL can lead to election loops

Describe the bug

Remote State auto restore trigger when all local disk state is lost, i.e. all cluster manager node are replaced
While restoring Remote State when new nodes comeup, we restore the decommission attributes but not the VotingConfiguration.
The decommissioned node will take part in election of new cluster manager since we did not restore the VotingConfiguration and new VotingConfiguration construction will allow the decommissioned node to take part in preVoting round.
If the election is successful by a vote from the decommissioned node, the first publish will fail as the decommissioned node vote would be considered invalid because the decommissioned node wont be able to join the elected cluster manager.
This will fail the elected cluster manager and another eligible cluster manager will start election.
The new election again can fail if the decommissioned node again votes.
The decommission node will not ever start election as we already have a check for nodeCommissioned in this flow.

To Reproduce
Steps to reproduce the behavior:

Create a 3 dedicated cluster manager remote store and remote state domain with force zone awareness enabled.
Weight out a dedicated cluster manager node using a node attr
Decommission the weighed out cluster manager node.
Replace all cluster manager nodes
The new cluster manager nodes will download the remote state with decommission attributes
Cluster would go into election loops

Expected behavior
Decommissioned node should not participate in  voting

Hi, are we on track for this to be released in 2.12 ?

Hi, are we on track for this to be released in 2.12 ?

Removed the 2.12 label.
@soosinha Can you add the correct target release label?


Edits to OCF's application page

Describe the bug
A few edits to make on OCF's hosting application (https://opencollective.com/foundation/apply/form)

The 'More info' link in the sidebar is dead. Please direct it instead to https://docs.opencollective.foundation/about/mission-and-values

This hover (?) doesn't show anything

Can we add a hover (?) saying "This can be edited later"

Please add to this title: "What does your initiative do? Please describe how you plan to use raised funds." -- either as a subheader, or in parenthesis; whichever looks best/is easiest.

In this highlighted gap, can we add a tip saying:

⚠ Once your page is submitted, please be sure to have at least 2 admin (with First, Last Name) on your initiative's Team to ensure timely application processing. Instructions for adding admin here.

Should more info actually just go to our main docs? Seems like that is separate from the mission/vision drop-down.

One thing the auditors pointed out is it would be good to explicitly document how the Collective's mission connects to OCF's mission areas (education, social impact, civic engagement). That would also be helpful for the approval process (e.g. an open source project with no specific social impact mission would be caught on that point and could be referred to OSC). Should we add a box for that?

@alanna one of the questions in the application, not shown in this issue, is "Please explain how your initiative furthers one or more of our mission impact areas. Do you think that is enough?
https://docs.opencollective.foundation/getting-started/how-to-apply/application-questions

Ah yeah sorry missed that. If it's already in there that's all good.

Should "More info" actually just go to our main docs? Seems like that is separate from the mission/vision drop-down.

Yea sure, works for me

I've done some quick changes in the PR above but I think we need to clarify some things. 😄

1. The 'More info' link in the sidebar is dead. Please direct it instead to https://docs.opencollective.foundation/

The more info link seems to be working now and it is directed to the OCF FAQ section; https://docs.opencollective.foundation/faq/basic-faq. Let me know if this is the best place we can redirect our users.

2. This hover (?) doesn't show anything
   ![image](https://user-images.githubusercontent.com/64920686/141583734-fefd1c8e-f75c-400f-94fc-6409f10ad532.png)

I've added the following to the tool-tip for now; Tell us more about your collective. This information is private and only used for internal purposes. Let me know if there's better wording for this one? 🤔

3. Can we add a hover (?) saying "This can be edited later"
   ![image](https://user-images.githubusercontent.com/64920686/141584732-0cc93823-0f8f-4c33-8c06-2842414a2ff7.png)

Maybe a parenthesis fits better since we are already using parenthesis for the one below and also the user can see it directly? 🤔

4. Please add to this title: "**What does your initiative do?** Please describe how you plan to use raised funds." -- either as a subheader, or in parenthesis; whichever looks best/is easiest.
   ![image](https://user-images.githubusercontent.com/64920686/141587500-c4627672-fcc8-4f6f-a1b8-ffc470ea2f72.png)

Done in parenthesis.

5. In this highlighted gap, can we add a tip saying:
   ![image](https://user-images.githubusercontent.com/64920686/141592611-3e4c9f6d-6ddc-499a-9b75-80efaedfe61f.png)

warning Once your page is submitted, please be sure to have at least 2 admin (with First, Last Name) on your initiative's Team to ensure timely application processing. Instructions for adding admin here.

Done. 👍🏽
The final result looks like this;


Animal create/update features

WIP.  Some irrelevant commits are pulled from the history

waiting for the Backend team changes regarding type structures


Add all ledgers endpoint

Modeled heavily off of All Assets, this endpoint represents the all ledgers endpoint for the stellar horizon server.
Since this is my first PR please critique mercilessly :)
Closes #57

Some tests failed, I will fix these up.

we also use cargo fmt to lint the repo.    Again thanks for joining and taking the initiative to dive right in.  This is awesome!

Thanks for jumping in with two feet! This is a great start :)

Hey so this seems to be the problem I am having:
thread 'main' panicked at 'called Result::unwrap() on an Err value: ParseError(ErrorImpl { code: Message("missing field prev_hash"), line: 51, column: 7 })'
And it is related to the doc tests.

So it appears prev_hash is not actually a value on a ledger, so that was causing tests to fail. Everything passes now!
https://horizon-testnet.stellar.org/ledgers?limit=1

@robertDurst I updated some of the ways I documented and arranged the modules in #69 if you want to take a look 😄
Hopefully those changes make sense and we can match these PRs. I don't mind refactoring yours after we merge though.

Sounds good! I'll check out this stuff and squash tomorrow!

Ok.... let me squash and this should be good to go.

Hey @kbacha so I am not very good at rebasing and squashing 😅 ... I did it once and I'm still not 100% sure I didn't mess it all up.
Essentially here, I assume I would need to move all my commits into one, after the two PR's that I pulled (and were submitted since this PR has been open).
Otherwise I reformatted the ledger docs and I think it all looks good.

Try following these steps @robertDurst :

Checkout and pull master from GitHub
checkout robertDurst:all-ledgers
git rebase master.  From the looks of it you will have conflicts to resolve.  You can also do git merge master here if you are more comfortable resolving conflicts in a merge commit.  Conflict resolution in a rebase is slightly different, so up to you on how many new things you want to try at once.

Now you should be on robertDurst:all-ledgers and be up to date with the changes in master.
This is the most explicit way I know of to squash old commits, there are fancier shortcuts out there you can learn about.
4.  6. Repeat step 5 until you have all the changes you want stagged for commit. git commit` should open a vim editor.  The first line should be the title of your commit followed by a few sentences describing with you did.
7. All your changes are saved with a single commit :beers:

You may want to use interactive rebase.
https://robots.thoughtbot.com/git-interactive-rebase-squash-amend-rewriting-history#squash-commits-together

Thanks @correlator and @kbacha! So I have squashed all down to one commit.
However it also squashed the merged code I had in the middle of my PR's...

I'm guessing that your merge commit may have pulled in some in-between state. You can probably revert those files by checking them out individually.
$ git checkout CONTRIBUTING.md

And then amending your commit:
$ git commit --amend
$ git push -f origin HEAD

btw, I'll merge this as soon as those extraneous changes are removed 👍

Learn something new about git everyday... thanks for your patience 👍

Now it is messed up again. Ill give this another go after work.

i may have made the checkout harder to understand, but if you do:
git checkout master -- <filename/directory>

then I think it works. Basically, this tells git not to checkout the branch but to checkout a particular file instead.

Got it down to two commits.

Started celebrating when rebase worked... and then I failed the tests. However, I am unable to reproduce this failure @kbacha. Help would be awesome :)

Rebase worked 👍  but then build broke 👎 . Having issues reproducing the error. Any help here would be awesome!

@robertDurst I'll rebuild. I wonder if the horizon test api blocked us? It looks like requests timed out.

Thanks @robertDurst !


Updated the correct URLs

The URL contained in the direct links no longer reloads the page to the page site but instead points to the main URL.

@abrilurena Could you look over these changes? Thank you


Syntax highlighting failure in GVim on Windows

GVim version: 8.0.839 x86
Windows: Win 10 x64
GVim 8.0.636 works fine, but all the highlighting disappeared when I move around the cursor in GVim 8.0.839. Console mode is also fine, only the GVim:

command to reproduce:
gvim -N -u ..\initrc.vim d:/acm/github/vim/asc/menu.vim

content of initrc.vim:
set nocompatible
syntax on

If I enter these two line above in the vim, the bug will not be reproduced, they must be in initrc.vim and load before gvim startup.

I can reproduce similar Vim script syntax fail on 8.0.1067 on Win 10 x64. In my case syntax breaks in gVim and CMD's Vim.
Steps to reproduce:

gvim -u NONE -i NONE -N "+so $VIMRUNTIME/plugin/matchparen.vim" "+syntax on" reproduce.vim
type the following in normal mode ffyiWjviwpk
syntax disappears

reproduce.vim
let a = foo()
let b = baz

If I disable matchparen with let g:loaded_matchparen  = 1 "$VIMRUNTIME/plugin/matchparen.vim
syntax will not break
Breakage was introduced in patch 8.0.0647. Before that commit I cannot reproduce.

Still exists in 8.0.1118

@xtal8 I also use Win 10 x64. But I don't reproduce it.

@mattn Do you use x64 gVim also? I can't reproduce on x86. My version is downloaded from here https://github.com/vim/vim-win32-installer/releases

Yes, I use x64. But I built vim with mingw compiler.

I always download gvim from
https://github.com/vim/vim-win32-installer/releases
And both gvim_8.0.1118_x64.zip and gvim_8.0.1118_x86.zip downloaded from
can reproduce this problem:

download my menu.vim from
https://github.com/skywind3000/vim/blob/master/asc/menu.vim

open menu.vim and

:normal gg$

move the cursor up and down by 'j' and 'k'

and than, You can encounter the issue.

I can reproduce issue with menu.vim only on gvim_8.0.1118_x64.zip. When I try to reproduce on x86. there's no syntax breakage.

strange, on my laptop:
gvim installed by gvim_8.0.1118_x86.exe has the problem.
gvim extracted from gvim_8.0.1118_x86.zip doesn't have the problem.

strange, on my laptop:
gvim installed by gvim_8.0.1118_x86.exe has the problem.
gvim extracted from gvim_8.0.1118_x86.zip doesn't have the problem.

Just checked it, could not reproduce it with either one.

This patch will fix the issue.
diff --git a/src/syntax.c b/src/syntax.c
index 6361bd7aa..0d5af0cc3 100644
--- a/src/syntax.c
+++ b/src/syntax.c
@@ -6573,9 +6573,14 @@ syn_get_id(
     else if (wp->w_buffer == syn_buf
 	    && lnum == current_lnum
 	    && col > current_col)
+    {
+	/* reset syn_tm here since this is not highlighting */
+	syn_tm = NULL;
+
 	/* next_match may not be correct when moving around, e.g. with the
 	 * "skip" expression in searchpair() */
 	next_match_idx = -1;
+    }

     (void)get_syntax_attr(col, spellp, keep_state);

syn_tm is set when start syntax to disable heavy syntax. matchparen call f_synstack(). And it call syn_get_id(), syn_get_attr(). This doesn't call syntax_call (this is correct pass). i.e. syn_tm is used unintentionally which is already timed-out.

@brammool This should be temporary updating like below?
diff --git a/src/syntax.c b/src/syntax.c
index 6361bd7aa..950ea7153 100644
--- a/src/syntax.c
+++ b/src/syntax.c
@@ -6564,6 +6564,8 @@ syn_get_id(
     int		*spellp,     /* return: can do spell checking */
     int		keep_state)  /* keep state of char at "col" */
 {
+    proftime_T	*save_tm = syn_tm;
+
     /* When the position is not after the current position and in the same
      * line of the same buffer, need to restart parsing. */
     if (wp->w_buffer != syn_buf
@@ -6573,12 +6575,19 @@ syn_get_id(
     else if (wp->w_buffer == syn_buf
 	    && lnum == current_lnum
 	    && col > current_col)
+    {
+	/* reset syn_tm here since this is not highlighting */
+	syn_tm = NULL;
+
 	/* next_match may not be correct when moving around, e.g. with the
 	 * "skip" expression in searchpair() */
 	next_match_idx = -1;
+    }

     (void)get_syntax_attr(col, spellp, keep_state);

+    syn_tm = save_tm;
+
     return (trans ? current_trans_id : current_id);
 }

Fixed by 8.0.1133


fix: move more links to the conda org from conda-incubator

Description

We missed some more conda-incubator mentions, also move to much from mamba-org.

Restarted the failed CI as I have no clue what these errors meant

Its unrelated, I have to fix that. Feel free to merge!


How to set configure options?

How to force './configure' to use jdk? Is it even possible?
Im not sure how this can happen, but for some reason the jdk is not being detected:
[INFO] checking dynamic linker characteristics... GNU/Linux ld.so
[INFO] checking how to hardcode library paths into programs... immediate
[INFO] checking whether stripping libraries is possible... yes
[INFO] checking if libtool supports shared libraries... yes
[INFO] checking whether to build shared libraries... yes
[INFO] checking whether to build static libraries... no
[INFO] checking how to run the C++ preprocessor... g++ -I/usr/include/wine/windows -E
[INFO] checking for ld used by g++ -I/usr/include/wine/windows... /usr/bin/ld
[INFO] checking if the linker (/usr/bin/ld) is GNU ld... yes
[INFO] checking whether the g++ -I/usr/include/wine/windows linker (/usr/bin/ld) supports shared libraries... yes
[INFO] checking for g++ -I/usr/include/wine/windows option to produce PIC... -fPIC -DPIC
[INFO] checking if g++ -I/usr/include/wine/windows PIC flag -fPIC -DPIC works... yes
[INFO] checking if g++ -I/usr/include/wine/windows static flag -static works... yes
[INFO] checking if g++ -I/usr/include/wine/windows supports -c -o file.o... yes
[INFO] checking if g++ -I/usr/include/wine/windows supports -c -o file.o... (cached) yes
[INFO] checking whether the g++ -I/usr/include/wine/windows linker (/usr/bin/ld) supports shared libraries... yes
[INFO] checking dynamic linker characteristics... (cached) GNU/Linux ld.so
[INFO] checking how to hardcode library paths into programs... immediate
[INFO] configure: javac was on your path, checking to see if it's part of a JDK we can use...
[INFO] checking if '/usr/lib/jvm/java-8-openjdk-amd64' is a JDK... no
[INFO] configure: Taking a guess as to where your OS installs the JDK by default...
[INFO] checking if '/usr' is a JDK... no
[INFO] configure: error: JDK not found. Please use the --with-jni-jdk option

'/usr/lib/jvm/java-8-openjdk-amd64' is present and is most definitely a JDK.
How can I set the '--with-jni-jdk ' option?

I encounter this kind issue by running mvn install.
I set it in my pom.xml and its like
<executions>
              <execution>
                <id>build-native-lib</id>
                <configuration>
                  <name>netty_resolver_dns_native_macos_${os.detected.arch}</name>
                  <nativeSourceDirectory>${nativeSourceDirectory}</nativeSourceDirectory>
                  <libDirectory>${project.build.outputDirectory}</libDirectory>
                  <!-- We use Maven's artifact classifier instead.
                       This hack will make the hawtjni plugin to put the native library
                       under 'META-INF/native' rather than 'META-INF/native/${platform}'. -->
                  <platform>.</platform>
                  <configureArgs>
                    <arg>${jni.compiler.args.ldflags}</arg>
                    <arg>${jni.compiler.args.cflags}</arg>
                    <arg>MACOSX_DEPLOYMENT_TARGET=10.9</arg>
                    **<arg>--with-jni-jdk=/Users/petterLee/Library/Java/JavaVirtualMachines/corretto-18.0.2/Contents/Home</arg>**
                  </configureArgs>
                </configuration>
                <goals>
                  <goal>generate</goal>
                  <goal>build</goal>
                </goals>
              </execution>
            </executions>


Allow java-test-fixtures to be applied standalone

Previously, java-test-fixtures requried the Java plugin to be applied to add any new behaviors. Instead, we now perform most of the work without requiring the Java plugin to be applied, but instead automatically apply the JavaBasePlugin.
We still require react to the Java plugin in order to add automatic dependencies between the test fixtures and the production code, and the tests and the test fixtures. We also react to the Java plugin in order to publish the test fixtures. However, both of these operations are possible to perform manually without the Java plugin being applied.
This is important for third-party ecosystems, like Kotlin and Android, using test fixtures without needing to also apply the Java plugin.

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.

Pre-tested commit build failed.

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.

Pre-tested commit build failed.

I've triggered a build for you. Click here to see all failures if there's any.


PMID 22952805 not syncing to Deposition App

I added this PMID to the Curation App on 31.01.23.
The PMID is not yet available when searching https://www.ebi.ac.uk/gwas/deposition/curation_queue
Please add this PMID to the Deposition App so I can curate.

This will be doen before the Data release

@earlEBI I can see the Pmid is available in the queue now , the issue appears to have resolved itself, kindly confirm

Confirmed, thank you. Ticket can be closed.


Support Trivy IaC Scanning

Trivy now supports IaC scanning. The VS Code extension should be able to also run Trivy in IaC mode and show results as appropriate.

FWIW this seems to work when I run this in VS Code.

This has been done


[SceneKit] Add apinotes for API renaming

What's in this pull request?

Add in apinotes, and associated tests, that rename many APIs to be
more consistent with the API design guidelines.

rdar://problem/26573775
Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

Add in apinotes, and associated tests, that rename many APIs to be
more consistent with the API design guidelines.

@swift-ci please test

Should be no test difference for Linux, as the test case only runs on macOS.
Linux bot: Build timed out (after 30 minutes). Marking the build as failed.
macOS bot: legitimate failure, I will update

@swift-ci please test

@swift-ci please test


perl fails to build with zlib-ng

Looks like zlib-ng (with zlib-compat)  isn't compatible with perl?
Perl fails to build:
In file included from Zlib.xs:34:
/usr/include/zlib.h:1781:31: error: unknown type name 'gzFile'
Z_EXTERN int Z_EXPORT gzgetc_(gzFile file);  /* backward compatibility */
                              ^
/usr/include/zlib.h:1818:13: error: unknown type name 'gzFile'
   Z_EXTERN gzFile Z_EXPORT gzopen(const char *, const char *);
            ^
/usr/include/zlib.h:1819:45: error: unexpected type name 'off_t': expected identifier
   Z_EXTERN z_off_t Z_EXPORT gzseek(gzFile, z_off_t, int);
                                            ^
/usr/include/zconf.h:159:21: note: expanded from macro 'z_off_t'
#    define z_off_t off_t
                    ^
In file included from Zlib.xs:34:
/usr/include/zlib.h:1819:54: error: expected identifier
   Z_EXTERN z_off_t Z_EXPORT gzseek(gzFile, z_off_t, int);
                                                     ^
/usr/include/zlib.h:1820:37: error: a parameter list without types is only allowed in a function definition
   Z_EXTERN z_off_t Z_EXPORT gztell(gzFile);
                                    ^
/usr/include/zlib.h:1821:39: error: a parameter list without types is only allowed in a function definition
   Z_EXTERN z_off_t Z_EXPORT gzoffset(gzFile);
                                      ^
/usr/include/zlib.h:1840:35: error: unknown type name 'gzFile'
Z_EXTERN int Z_EXPORTVA gzvprintf(gzFile file, const char *format, va_list va);
                                  ^
In file included from Zlib.xs:743:
./constants.h:370:18: warning: incompatible pointer to integer conversion assigning to 'IV' (aka 'long') from 'void *' [-Wint-conversion]
      *iv_return = Z_NULL;
                 ^ ~~~~~~

Looks like there is a patch for zlib-ng from OpenMandriva
https://github.com/OpenMandrivaAssociation/zlib-ng/blob/master/0001-Fix-Z_SOLO-mode.patch

Perl still fails. Only 2 of 7 errors fixed with patch.
In file included from Zlib.xs:34:
/usr/include/zlib.h:1820:13: error: unknown type name 'gzFile'
   Z_EXTERN gzFile Z_EXPORT gzopen(const char *, const char *);
            ^
/usr/include/zlib.h:1821:45: error: unexpected type name 'off_t': expected identifier
   Z_EXTERN z_off_t Z_EXPORT gzseek(gzFile, z_off_t, int);
                                            ^
/usr/include/zconf.h:159:21: note: expanded from macro 'z_off_t'
#    define z_off_t off_t
                    ^
In file included from Zlib.xs:34:
/usr/include/zlib.h:1821:54: error: expected identifier
   Z_EXTERN z_off_t Z_EXPORT gzseek(gzFile, z_off_t, int);
                                                     ^
/usr/include/zlib.h:1822:37: error: a parameter list without types is only allowed in a function definition
   Z_EXTERN z_off_t Z_EXPORT gztell(gzFile);
                                    ^
/usr/include/zlib.h:1823:39: error: a parameter list without types is only allowed in a function definition
   Z_EXTERN z_off_t Z_EXPORT gzoffset(gzFile);
                                      ^

For now, will allow Perl to use it's bundled zlib source:
# Ensure that we never accidentally bundle bzip2
rm -rf cpan/Compress-Raw-Bzip2/bzip2-src
sed -i '/\bzip2\-src/d' MANIFEST

Buid success without these commands.
https://github.com/dslm4515/CMLFS/blob/9ff93739462b3f75406f26c707ad819c28648975/3-chroot/042-perl#L10-L13

Without those commands, perl builds with the bundled bzip2 and zlib source in the perl source.
I assume LFS forced perl to not use the bundled sources (zlib & bzip2) in case those bundled sources are out of date.
Main issue is incompatible zlib-ng headers causes the Compress-Raw-Zlib perl module to not compile... I can't find anyone who has built perl with zlib-ng

Confusing ..

Normally when building a system from source, one would rather use the freshly built libraries than a package's bundled 3rd-party library source.
The perl source has bundled copies of the sourcecode for bzip2 and zlib. This is handy if building perl on a system that may not have nzip2 or zlib installed. Otherwise, its like compiling zlib and bzip2 twice: once for the system and again for perl.


[Bug]: Issue with ES Module Importing in Node

What happened?
Hello team,
I recently encountered an issue when trying to use the chromadb library in my project, which uses Node with ES Modules.
Problem:
When using the following import statement in my code:
import { ChromaClient } from "chromadb";
Node.js is referring to the module field in package.json, pointing towards the dist/module directory. Within that directory, there appears to be a mix of ES module and CommonJS conventions, specifically in the file OpenAIEmbeddingFunction.js.
File path where the error ocurred: node_modules/chromadb/dist/module/embeddings/OpenAIEmbeddingFunction.js:2
The package.json file has its main field pointing to "dist/main/index.js" and the module field pointing to "dist/module/index.js". Traditionally, main is reserved for CommonJS builds and module for ESM builds. However, in the case of chromadb, it appears that this convention might have been inverted or misconfigured:

The module directory seems to have a mix of CommonJS (usage of require) and ESM (usage of export), which causes issues in strict ESM environments.
The main directory seems to be more compliant with ESM as I was able to import from "chromadb/dist/main" without issues.

Temporary Workaround:
As a temporary fix, I've changed my import statement to point directly to the main build:
import { ChromaClient } from "chromadb/dist/main";
This works, but it's clearly a workaround and may not be obvious for other users since it's not even documented.
Suggestion:

Please review and ensure that the main field in package.json strictly points to a CommonJS build, and the module field strictly points to an ESM build. This will help ensure better compatibility across different environments and tools.
Ensure that each build (CommonJS and ESM) strictly adheres to its respective module system without mixing conventions.

Thank you for your attention to this matter. Looking forward to a resolution that makes integration seamless for all Node.js setups!
Versions
Chroma v1.5.6, Node v18.16
Relevant log output
(node:6580) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.

@juliandreas thanks for reporting this! I am working on it 👍

@juliandreas can you try 1.5.8 (just released today) and see if this addresses the issue you were seeing?

It's still not working unfortunately. And it looks like you're still mixing CommonJS and ESM. You're using require to import OpenAI, for example.

If you want to try to recreate it with a Node.js project using ESM, just add "type": "module" into the package.json. You should get the same error when importing and trying to use ChromaClient like this: import { ChromaClient } from "chromadb";. It's when you spin up the project that you get the error.

I've created a PR #1126 for this. Like I mentioned in the PR, this will only fix the issue for two of the embeddings. I didn't have time for it today to look at the rest. But it's a good start if you want to take a look at this yourselves. I can continue with the other embeddings as well, but not sure when I will have the time.


Decrease container startup time

grafana/otel-lgtm takes a long time to start. I wonder if this sequential starting of Loki, Prometheus, Tempo etc can not be parallelized somehow?
2024-11-11 12:49:41 Waiting for the OpenTelemetry collector and the Grafana LGTM stack to start up...
2024-11-11 12:49:41 Running Loki v3.2.1 logging=false
2024-11-11 12:49:41 Running Prometheus v2.55.0 logging=false
2024-11-11 12:49:41 Running OpenTelemetry Collector v0.112.0 logging=false
2024-11-11 12:49:41 Running Tempo v2.6.1 logging=false
2024-11-11 12:49:41 Running Grafana v11.3.0 logging=false
2024-11-11 12:49:41 Waiting for Grafana to start up...
2024-11-11 12:49:42 Waiting for Grafana to start up...
2024-11-11 12:49:43 Waiting for Grafana to start up...
2024-11-11 12:49:44 Waiting for Grafana to start up...
2024-11-11 12:49:45 Waiting for Grafana to start up...
2024-11-11 12:49:46 Waiting for Grafana to start up...
2024-11-11 12:49:47 Waiting for Grafana to start up...
2024-11-11 12:49:48 Waiting for Grafana to start up...
2024-11-11 12:49:49 Grafana is up and running.
2024-11-11 12:49:49 Waiting for Loki to start up...
2024-11-11 12:49:50 Waiting for Loki to start up...
2024-11-11 12:49:51 Waiting for Loki to start up...
2024-11-11 12:49:52 Waiting for Loki to start up...
2024-11-11 12:49:53 Waiting for Loki to start up...
2024-11-11 12:49:54 Waiting for Loki to start up...
2024-11-11 12:49:55 Waiting for Loki to start up...
2024-11-11 12:49:56 Waiting for Loki to start up...
2024-11-11 12:49:57 Waiting for Loki to start up...
2024-11-11 12:49:58 Waiting for Loki to start up...
2024-11-11 12:49:59 Waiting for Loki to start up...
2024-11-11 12:50:00 Waiting for Loki to start up...
2024-11-11 12:50:01 Waiting for Loki to start up...
2024-11-11 12:50:02 Waiting for Loki to start up...
2024-11-11 12:50:03 Waiting for Loki to start up...
2024-11-11 12:50:04 Loki is up and running.
2024-11-11 12:50:04 Prometheus is up and running.
2024-11-11 12:50:04 Waiting for Tempo to start up...
2024-11-11 12:50:05 Waiting for Tempo to start up...
2024-11-11 12:50:06 Waiting for Tempo to start up...
2024-11-11 12:50:08 Waiting for Tempo to start up...
2024-11-11 12:50:09 Waiting for Tempo to start up...
2024-11-11 12:50:10 Waiting for Tempo to start up...
2024-11-11 12:50:11 Waiting for Tempo to start up...
2024-11-11 12:50:12 Waiting for Tempo to start up...
2024-11-11 12:50:13 Waiting for Tempo to start up...
2024-11-11 12:50:14 Waiting for Tempo to start up...
2024-11-11 12:50:15 Waiting for Tempo to start up...
2024-11-11 12:50:16 Waiting for Tempo to start up...
2024-11-11 12:50:17 Waiting for Tempo to start up...
2024-11-11 12:50:18 Waiting for Tempo to start up...
2024-11-11 12:50:19 Waiting for Tempo to start up...
2024-11-11 12:50:20 Tempo is up and running.
2024-11-11 12:50:20 Waiting for the OpenTelemetry collector to start up...
2024-11-11 12:50:21 Waiting for the OpenTelemetry collector to start up...
2024-11-11 12:50:22 The OpenTelemetry collector and the Grafana LGTM stack are up and running. (created /tmp/ready)

all components are already started in parallel - see https://github.com/grafana/docker-otel-lgtm/blob/81745f67341e5106eea09e22e43b09dff4c3b888/docker/run-all.sh#L3-L7
Not waiting for Tempo and Loki would speed up startup - we could add a parameter for that


User friends - friends list

profile vue refactoring
bootstrap vue tabs
"social connections" tab
fiends list with pagination


Add the Agility Pyramid to the Agility skill calculator.

The Agility Pyramid rewards bonus XP depending on player level. This PR calculates the bonus XP based on the starting level entered into the calculator.

Rather than having code specific to the agility pyramid in the skill calculator, I'd rather see support for level-based variable-experience actions added generally, eg. #10076. If done correctly, this should mean that all of the implementation for such skills could be done in SkillAction or some child interface of that one so no one-off code needs to exist in the skill calculator class itself.
You can also refer to some of the discussion on #9940 to understand some of the work that would be involved. Mainly, such a solution should consider that intermediate levels have different XP requirements to level up, and also grant different XP for each actions completed.


Bug 2307909: Fix disable dr when VR failed validation

When deleting a primary VRG, we wait until the VR Completed condition is
met. However if a VR precondition failed, for example using a drpolicy
without flattening enabled when the PVC needs flattening, the VR will
never complete and the vrg and drpc deletion will never complete.
Since csi-addons 0.10.0 we have a new Validated VR condition, set to
true if pre conditions are met, and false if not. VR is can be deleted
safely in this state, since mirroring was not enabled.
This changes modifies deleted VRG processing to check the new VR
Validated status. If the condition exist and the condition status is
false, validateVRStatus() return true, signaling that the VR is in the
desired state, and ramen completes the delete flow.
If the VR does not report the Validated condition (e.g. old csi-addon
version) or the condition status is true (mirroring in progress), we
continue in the normal flow. The VR will be deleted only when the
Completed condition status is true.
This is a backport of:

https://github.com/RamenDR/ramen/pull/1542 - updating for API and dependencies changes in csi-addons 0.9.1
https://github.com/RamenDR/ramen/pull/1561 - for getting csi-addons 0.10.0 in ramen and drenv
https://github.com/RamenDR/ramen/pull/1570 - using the new Validated condition in csi-addons 0.10.0

Tested with:

[x] drenv with deployment using cloned pvc
[ ] drenv with vm using cloned pvc

Image for testing: quay.io/nirsof/ramen-operator:release-4.17-vr-validated-1

@ELENAGER can you review?

Unit tests failure is unrelated, but I cannot rerun the tests in this repo.

/cc @ELENAGER

Tested also with vm.

Should we merge it manually?


mention possibility to bundle SQLite

See https://github.com/diesel-rs/diesel/discussions/3255

I would rather like to suggest to depend on libsqlite3-sys

Of course. Updated the PR.

CI failures are unrelated to this change and caused by the clap 4.1 release: https://github.com/diesel-rs/diesel/pull/3479


I have this error

Warning: Missing argument 4 for buildPostsForPage(), called in /home/u899309222/public_html/bl-kernel/boot/rules/70.posts.php on line 66 and defined in /home/u899309222/public_html/bl-kernel/functions.php on line 95
Notice: Undefined variable: key in /home/u899309222/public_html/bl-kernel/functions.php on line 105

Is this a custom theme? did you modify any bludit files?

I think you are using the code from github?

Yes I use  code from github.

Can you try the lastest code ? I release a beta yesterday, https://github.com/dignajar/bludit/releases/tag/1.6-beta1

Last 1.6-beta1 work normal.


Pyinstaller missing import

It seems that an import is missing when I try to create a binary using pyinstaller
Traceback (most recent call last):
  File "flask\app.py", line 2070, in wsgi_app
  File "flask\app.py", line 1515, in full_dispatch_request
  File "flask_cors\extension.py", line 165, in wrapped_function
  File "flask\app.py", line 1513, in full_dispatch_request
  File "flask\app.py", line 1499, in dispatch_request
  File "flask_sock\__init__.py", line 24, in websocket_route
  File "simple_websocket\ws.py", line 136, in __init__
AttributeError: 'Input' object has no attribute 'raw'

The application runs great otherwise. Have you tried creating a binary using flask_sock or simple_socket? if so, how did you overcome this issue?

The version of gevent you were using wasn't being handled correctly. You can upgrade your gevent to the latest and everything should work. I have also committed a fix to handle the older versions.


Need API nodes separate from block proposers

We need separate API nodes to protect block proposers.

Done.


Use native aac format

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Fixed tickets
fixes #issuenum

Related issues/PRs
#issuenum

License
MIT

What's in this PR?
‘AAC’ is the native codec and ‘libfdk_aac’ only exists if compiled in during the build process. This PR switches the 'aac' to the default.
Why?
#567

No, the v1.x branch is currently a mess. I don't know exactly what causes this, but anything with the Progress Listener. No worries :)


BeginScope(TState state) feels like there is something incomplete there.

I maintain an Azure table logger since beta3, and because this thing is bugging me since then, I never published it.
BegingScope has to be implemented as a generic method, but it has no constraint of any kind. It feels almost pointless. Moreover, builtin loggers are not leveraging it.
My first guess was that is was intended as an easier way to allow us to do some cached code generation. But, from what I observed, it seems like the team committed to states deriving from IReadOnlyList<KeyValuePair<string, object>>. In that case add a constraint on TState, it's just an interface so it is easy to implement for anyone willing to provide a custom scope.
This would make it much easier and useful  when supporting scopes.

I noticed that entityframework team has done things a bit differently and is not implementing IReadOnlyList<KeyValuePair<string, object>. So we are back to reflection if we want to support scopes and are not satisfied with what ToString()` method provides.

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


Build svg

implement contract methods to build and return the svg image data

Yea should definitely run the linter. Thanks!


Clean up the old BloomFilter implementation where BloomFilters are attached to HashTables

This PR cleans up the old BloomFilter implementation where BloomFilters are attached to HashTables. In subsequent PRs we will have a systematic implementation for applying the generalized BloomFilters (i.e. LIPFilters).

Looks good, merging.

Already merged.. manually close this PR.


[Question]: How to set HuggingFaceLLM for "tiiuae/falcon-7b-instruct"?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I want to use "tiiuae/falcon-7b-instruct" as my CustomLLM.
The model card is here: https://huggingface.co/tiiuae/falcon-7b-instruct
The Huggingface implementation looks like this:
from transformers import AutoTokenizer, AutoModelForCausalLM
import transformers
import torch

model = "tiiuae/falcon-7b-instruct"

tokenizer = AutoTokenizer.from_pretrained(model)
pipeline = transformers.pipeline(
    "text-generation",
    model=model,
    tokenizer=tokenizer,
    torch_dtype=torch.bfloat16,
    trust_remote_code=True,
    device_map="auto",
)
sequences = pipeline(
   "Girafatron is obsessed with giraffes, the most glorious animal on the face of this Earth. Giraftron believes all other animals are irrelevant when compared to the glorious majesty of the giraffe.\nDaniel: Hello, Girafatron!\nGirafatron:",
    max_length=200,
    do_sample=True,
    top_k=10,
    num_return_sequences=1,
    eos_token_id=tokenizer.eos_token_id,
)
for seq in sequences:
    print(f"Result: {seq['generated_text']}")

How to adopt the example below to work with falcon-7b-instruct? I struggle with pipeline parameters, model kwargs, etc.
# setup prompts - specific to StableLM
from llama_index.prompts.prompts import SimpleInputPrompt

system_prompt = """<|SYSTEM|># StableLM Tuned (Alpha version)
- StableLM is a helpful and harmless open-source AI language model developed by StabilityAI.
- StableLM is excited to be able to help the user, but will refuse to do anything that could be considered harmful to the user.
- StableLM is more than just an information source, StableLM is also able to write poetry, short stories, and make jokes.
- StableLM will refuse to participate in anything that could harm a human.
"""

# This will wrap the default prompts that are internal to llama-index
query_wrapper_prompt = SimpleInputPrompt("<|USER|>{query_str}<|ASSISTANT|>")

import torch

llm = HuggingFaceLLM(
    context_window=4096,
    max_new_tokens=256,
    temperature=0.7,
    do_sample=False,
    system_prompt=system_prompt,
    query_wrapper_prompt=query_wrapper_prompt,
    tokenizer_name="StabilityAI/stablelm-tuned-alpha-3b",
    model_name="StabilityAI/stablelm-tuned-alpha-3b",
    device_map="auto",
    stopping_ids=[50278, 50279, 50277, 1, 0],
    tokenizer_kwargs={"max_length": 4096},
    # uncomment this if using CUDA to reduce memory usage
    model_kwargs={"torch_dtype": torch.float16}
)

My first attempt:
model="tiiuae/falcon-7b-instruct"

llm = HuggingFaceLLM(
    context_window=2048,
    max_new_tokens=256,
    tokenizer_name=model,
    model_name=model,
    device_map="auto",
    tokenizer_kwargs={"max_length": 2048},
    # uncomment this if using CUDA to reduce memory usage
    model_kwargs={"torch_dtype": torch.float16}
)

generates the following error:
ValueError: The following `model_kwargs` are not used by the model: ['token_type_ids'] (note: typos in the generate arguments will also show up in this list)

I didn't find token_type_ids in the CustomLLM and HuggingFaceLLM classes.
Any idea?

This is a parameter that the hugging face tokenizer returns automatically.
We have an additional kwarg to remove unused tokenizer outputs, to avoid this error
HuggingFaceLLM(..., tokenizer_outputs_to_remove=["token_type_ids"])

Thank You this worked for me.


The number of repositories are incorrectly

The number of items is displayed incorrectly, there is actually 1 item，but there are two shown above.
account name：LuccyD

Hi, I believe this is a question about GitHub.com, the website. This issue tracker is for a command-line utility called hub, not the website itself. For reporting bugs and requesting features for GitHub the website, please email<EMAIL_ADDRESS>Thank you!


Unicode's backslash lost problem

When I tries to convert a json to a Dictionary<string, object>, my Chinese characters (e.g. "\u94b1\u4e0d\u591f!") inside dictionary values will become "u94b1u4e0du591f!", thus make it not readable any more.

I couldn't reproduce the issue. I added those characters as a test case to the unit tests though and it passes:
https://github.com/zanders3/json/commit/6bf8bff4a7d4406f09666ddf460f70c4f9ae5d20


IOError: write: broken pipe (EPIPE) error in Julia API running in Production (Kubernetes Cluster)

I am encountering an IOError: write: broken pipe (EPIPE) error in my Julia API, which only appears when running in Production (in a Kubernetes Cluster), I have never seen this issue in local. The error occurs randomly, sometimes breaking the session when making a request and sometimes not. I have tried several things to debug the issue, but I believe it is a core issue in either Oxygen or HTTP.jl.
Here are some details about my setup:
•	Julia version: 1.9.0
•	Oxygen version: 1.1.9
•	HTTP.jl version: 1.9.6

The API is containerized and running in a Kubernetes cluster with adequate memory and vCPU resources allocated to the container. However, due to the nature of the application, some requests can be particularly resource-intensive, using a lot of memory. Despite the sufficient resources, I am encountering a IOError: write: broken pipe (EPIPE) error randomly when making requests to the API. To help diagnose the issue, I have attached the logs that are generated whenever this error occurs. Below you can find the logs I can see in Production:
┌ Error: handle_connection handler error
│   exception =
│    IOError: write: broken pipe (EPIPE)
│    Stacktrace:
│      [1] uv_write(s::Sockets.TCPSocket, p::Ptr{UInt8}, n::UInt64)
│        @ Base ./stream.jl:1066
│      [2] unsafe_write(s::Sockets.TCPSocket, p::Ptr{UInt8}, n::UInt64)
│        @ Base ./stream.jl:1120
│      [3] unsafe_write
│        @ /home/.julia/packages/HTTP/uAaAw/src/Connections.jl:129 [inlined]
│      [4] write
│        @ ./strings/io.jl:244 [inlined]
│      [5] write
│        @ ./io.jl:678 [inlined]
│      [6] unsafe_write(http::HTTP.Streams.Stream{HTTP.Messages.Request, HTTP.Connections.Connection{Sockets.TCPSocket}}, p::Ptr{UInt8}, n::UInt64)
│        @ HTTP.Streams /home/.julia/packages/HTTP/uAaAw/src/Streams.jl:95
│      [7] write
│        @ ./strings/io.jl:244 [inlined]
│      [8] write(io::HTTP.Streams.Stream{HTTP.Messages.Request, HTTP.Connections.Connection{Sockets.TCPSocket}}, s::Base.CodeUnits{UInt8, String})
│        @ Base ./strings/basic.jl:761
│      [9] (::HTTP.Handlers.var"#1#2"{Oxygen.Core.var"#6#8"{Oxygen.Core.AutoDoc.var"#2#4"{Oxygen.Core.var"#28#31"{HTTP.Handlers.Router{typeof(HTTP.Handlers.default404), typeof(HTTP.Handlers.default405), Nothing}, Bool}, HTTP.Handlers.Router{typeof(HTTP.Handlers.default404), typeof(HTTP.Handlers.default405), Nothing}, Vector{Function}}, Sockets.IPv4}})(stream::HTTP.Streams.Stream{HTTP.Messages.Request, HTTP.Connections.Connection{Sockets.TCPSocket}})
│        @ HTTP.Handlers /home/.julia/packages/HTTP/uAaAw/src/Handlers.jl:61
│     [10] (::Oxygen.Core.var"#9#10"{Oxygen.Core.AutoDoc.var"#2#4"{Oxygen.Core.var"#28#31"{HTTP.Handlers.Router{typeof(HTTP.Handlers.default404), typeof(HTTP.Handlers.default405), Nothing}, Bool}, HTTP.Handlers.Router{typeof(HTTP.Handlers.default404), typeof(HTTP.Handlers.default405), Nothing}, Vector{Function}}})(stream::HTTP.Streams.Stream{HTTP.Messages.Request, HTTP.Connections.Connection{Sockets.TCPSocket}})
│        @ Oxygen.Core /home/.julia/packages/Oxygen/wl2K0/src/core.jl:123
│     [11] #invokelatest#2
│        @ ./essentials.jl:816 [inlined]
│     [12] invokelatest
│        @ ./essentials.jl:813 [inlined]
│     [13] handle_connection(f::Function, c::HTTP.Connections.Connection{Sockets.TCPSocket}, listener::HTTP.Servers.Listener{Nothing, Sockets.TCPServer}, readtimeout::Int64, access_log::Nothing)
│        @ HTTP.Servers /home/.julia/packages/HTTP/uAaAw/src/Servers.jl:447
│     [14] macro expansion
│        @ /home/.julia/packages/HTTP/uAaAw/src/Servers.jl:385 [inlined]
│     [15] (::HTTP.Servers.var"#16#17"{Oxygen.Core.var"#9#10"{Oxygen.Core.AutoDoc.var"#2#4"{Oxygen.Core.var"#28#31"{HTTP.Handlers.Router{typeof(HTTP.Handlers.default404), typeof(HTTP.Handlers.default405), Nothing}, Bool}, HTTP.Handlers.Router{typeof(HTTP.Handlers.default404), typeof(HTTP.Handlers.default405), Nothing}, Vector{Function}}}, HTTP.Servers.Listener{Nothing, Sockets.TCPServer}, Set{HTTP.Connections.Connection}, Int64, Nothing, Base.Semaphore, HTTP.Connections.Connection{Sockets.TCPSocket}})()
│        @ HTTP.Servers ./task.jl:514
└ @ HTTP.Servers /home/.julia/packages/HTTP/uAaAw/src/Servers.jl:461

Despite my efforts, I have been unable to identify the root cause of the IOError: write: broken pipe (EPIPE) error that occurs randomly when making requests to the API. I would greatly appreciate any insights or suggestions on how to resolve this issue.
As an additional note, I have observed that the error may occur more frequently when using threads for any parallelization within the API (running serve() not serveparallel()) or when the request is particularly long. However, it still happens randomly and without any consistent pattern.

HI @BrandonEscamilla,
I think this issue has been brought up before. It looks very similar to this issue: broken pipe issue
After checking it out, let me know if you have any follow-up questions about this!

Hi @ndortega, just wanted to provide an update regarding the issue we previously discussed. Although I initially thought that the session disconnection was related to the errors appearing in the logs, it turns out that it was just a coincidence. The disconnection was actually caused by a timeout in the cluster's load balancer. While this issue has been resolved, the errors are still persisting. Just wanted to clarify that the session disconnection was not directly related to the errors.

Hi @BrandonEscamilla ,
Thanks for the clarification. Have you been able to address/pinpoint this issue? I plan on closing this issue unless we can find an Issue with the Oxygen.jl package.


User redirected back to the login page after no internet connection

When their is no internet connection the state of the Auth changes to the unauthenticated and the user redirects to the login page
After making the internet available back it goes directly to the main page and the auth state is then authenticated.
It should stay authenticated or at least should stay in main page after no internet connection.
from the UX perspective it looks not good that after no internet it redirects to the Login Page, but at the back its already authenticated.
My implementation of Auth
class MainWidget extends StatelessWidget {
  const MainWidget({super.key});

  @override
  Widget build(BuildContext context) {
    final authNotifier = context.authNotifier;
    Widget widget;
    switch (authNotifier.status) {
      case AuthStatus.authenticated:
        widget = const FeedPage();
        break;
      case AuthStatus.unauthenticated:
      case AuthStatus.authenticating:
        widget = const LoginPage();
        break;
      case AuthStatus.uninitialized:
        widget = const SplashScreenWidget();
        break;
      default:
        widget = const SplashScreenWidget();
        break;
    }
    return widget;
  }
}

any update to this?

is their anyone who can pick this issue and solve it. it would be great. Thank you

I forked the projekt and wanted to look into it but didnt found time for it. So no not solved yet. I am still waiting for that bug to be fixed actually.

where do i put these?

@FaisalMohammadi  From UX perspective, it's okay to redirect back to login when no internet, even if they are logged in in the server, without internet the app shouldn't know about it. I feel it's better to notify with internet checking plugin and error message.
But if you want to make it work offline this is a good solution to your problem, but this might have other design issues which I'm not ready to tackle in this package. Want to keep this package simple. Also offline working is not in the scoope of this package.
Let me know what you think.


Bypass JSR107 layer when creating cache cause it to be not found

If a cache is created directory on the wrapped Ehcache CacheManager, the JSR107 won' see it. See this code:
    @Test
    public void test() {
        CacheManager cacheManager = Caching.getCachingProvider().getCacheManager();
        org.ehcache.CacheManager ecm = cacheManager.unwrap(org.ehcache.CacheManager.class);
        ecm.createCache("foo", CacheConfigurationBuilder.newCacheConfigurationBuilder(Integer.class, String.class, ResourcePoolsBuilder.heap(1)));

        Cache<Integer, String> cache = cacheManager.getCache("foo", Integer.class, String.class);
        assertThat(cache).isNotNull(); // fails
    }

However, add a cacheManager.getCacheNames() before calling getCache repairs it since it causes a cache refresh.
I think it should work or at least be coherent.

There has been work in this area recently... I think this might be fixed. This should get reassessed in-house and closed if it really has been fixed.

@chrisdennis, I have validated the mentioned use case and found that issue is still there (sample JUnit is still failing)
The above code is logically correct, as the cache is been created post  unwrap() method call, So it should not be visible to javaxCacheManager.
Current implement allows visibility of such new cache which has been created post  unwrap() method call after a cache refresh, which can be achieved by calling - cacheManager.getCacheNames()
My stand here is logged issue is there, but the mentioned behavior is logically correct.
If we still want a fix for this, we need to add cache refresh logic there inside of .getCache() method

Fixed(daggy) in #3117, #3119