[SharedCache] Switch to SAX-based writer API for JSON serialization

Building up an in-memory representation of the JSON document was performing a lot of temporary memory allocations. Using the SAX-based writer API avoids this work by directly writing the desired types. This cuts the time spent serializing the state to JSON by around half.
One additional benefit of the SAX-based writer API is that it's now possible for serialization to operate on individual types rather than having to serialize a complex type in a single operation in order to assign it to an object field. Serialize is updated to work on a single value at a time, with function templates for types like std::pair, std::vector, and std::unordered_map delegating to Serialize overloads for the types they contain. This removes the repetition that was previously required for implementing serialization arrays or maps of different types.
Deserialization continues to use the document-based API as using the SAX-based reader API is cumbersome.
This branch builds on the work in #6127 and should be compared against it. You can use https://github.com/bdash/binaryninja-api/compare/dsc-serialization...bdash:binaryninja-api:dsc-serialization-2?expand=1 to view the diff excluding the serialization changes.

Merged via a2e5d061


Error in loading one of the pretrained model from Sentence-Transformers

I am trying to load below pre-trained model from Sentence-Transformers in the Embedding function of txtai
xlm-r-100langs-bert-base-nli-stsb-mean-tokens:
But i am getting not found error.
Regards,

It looks like that model isn't stored on Hugging Face's model hub yet. txtai only supports models there or local models.
You can run the following to pull the model locally
wget https://public.ukp.informatik.tu-darmstadt.de/reimers/sentence-transformers/v0.2/xlm-r-100langs-bert-base-nli-stsb-mean-tokens.zip
unzip xlm-r-100langs-bert-base-nli-stsb-mean-tokens.zip -d xlm-r-100langs-bert-base-nli-stsb-mean-tokens

Then use xlm-r-100langs-bert-base-nli-stsb-mean-tokens as the path when creating the Embeddings instance.

Hello,
thankyou for the reply
I tried with your solution. but now I am getting below error while loading the local model
OSError: Error no file named ['pytorch_model.bin', 'tf_model.h5', 'model.ckpt.index'] found in directory /content/model/xlm-r-100langs-bert-base-nli-stsb-mean-tokens or from_tf set to False

How about the path /content/model/xlm-r-100langs-bert-base-nli-stsb-mean-tokens/0_Transformer

yes you are right. I did it already. So, it's started working... now going ahead for searching.
Regards,


test PR with Github API

Please pull this in!

Docs Build status updates of commit 8c54747:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

tutorial-manage-data-disk.md
:warning:Warning

Details

tutorial-manage-data-disk.md

Line 51: [Warning] Invalid file link:(~/sizes.md).
Line 68: [Warning] Invalid file link:(~/sizes.md).
Line 134: [Warning] Invalid file link:(~/tutorial-automate-vm-deployment.md).
Line 182: [Warning] Invalid file link:(~/tutorial-automate-vm-deployment.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Proposal to add a EcctrlProp to toggle camera follow

Ecctrl is taking the default camera using const { scene, camera } = useThree();
This means if we add another camera and mark it as the default camera Ecctrl will immediately take control of the camera.
But there are a use case where you want to cut to a different camera (let's say a flying map overview) and go back to the player's camera. So basically you want your new camera not to be controlled by ecctrl.
Therefore I would like to introduce a prop like camFollow: boolean then in Ecctrl.tsx line 989 we could do something like this:
/**
     *  Camera movement
     */
    if(camFollow) {
      pivotPosition.set(
        currentPos.x + camTargetPos.x,
        currentPos.y + (camTargetPos.y || (capsuleHalfHeight + capsuleRadius / 2)),
        currentPos.z + camTargetPos.z
      );
      pivot.position.lerp(pivotPosition, 1 - Math.exp(-camFollowMult * delta));
      state.camera.lookAt(pivot.position);
    }

and in useFollowCam.tsx we skip the mouse / touch events:
// Mouse move event
const onDocumentMouseMove = (e: MouseEvent) => {
    if(!camFollow) return false;
    ...

 // Mouse scroll event
  const onDocumentMouseWheel = (e: Event) => {
    if(!camFollow) return false;

    ...

// Touch move event
  const onTouchMove = (e: TouchEvent) => {
    // prevent swipe to navigate gesture
    e.preventDefault();
    e.stopImmediatePropagation();

    if(!camFollow) return false;

That would help us cutting to different camera view when needed.
If you think it's a good idea I can create a pull request!

Thank you, @valentincognito ! That's a really good point! Actually, I'm already working on this feature and it will be released in the next update!

Awesome news! I'm closing this issue then. Thanks for your work :)


Better error message when trying to change a final field of a struct

The following code snippet:
// struct with final fields:
struct Spaceship {
  name str;
  numWings int;
}

fn main() export {
  ship = Spaceship("Serenity", 2);
  println(ship.name);              // => Serenity
  ship.name = "Enterprise";        // <--
}

correctly gives a compiler error in the line indicated with <-- because the name field is final.
The error is:  expected ; or } after expression, but found: = "Enterprise";
This message should be made more clear to indicate that there was an attempt to change a final field.
Of course, defining the field as
                    name! str;

allows to change its contents with:
                    mut ship.name = "Enterprise";

Probably related to #121

Added a better error message for when someone tries to change a field without the mut keyword:
/Users/verdagon/main.vale:10:3: Changing a struct's member must start with the `mut` keyword.

And, when we do add the mut keyword, we now have a message about changing a final member:
/Users/verdagon/main.vale:10:7: Cannot mutate final member 'name' of struct Spaceship

Will mark this fixed when this release hits the site. Thanks Ivo!

Just tried it again, and I get the same result as last time.
I'm guessing you're using a different version, or I mixed up the release last time or something.
Will also test on windows before next release. Thanks Ivo!


How to save the projection?

After using bff-viewer, I found the desired projection from 3D to 2D surface. How could I save the projected surface? and how can I export the correspondence/relation of the mapping?

I have the same question. I cannot find any way to export the 2D surface result as an OBJ. How can the file be exported?

The application now outputs 2D surface by default. It was originally being stored using the "vt" flag in the OBJ format.


Implement Prim's algorithm with heaps

Implement Prim's algorithm to calculate MST of a graph using heaps (priority queue).

Resolved by PR #41


include libraries: Heart v6.0.0, Minie v3.0.0, Wes v0.6.0

Thanks for caring about the SDK! :)

I'm curious: what are your intentions regarding the SDK? What should I tell users and potential users?

I really want to release a 3.3 version, but there are a few fundamental changes that haven't been really tested and I'd want to move to a working update center again, so that's a bit "critical" to just release, which is why I'd need testing, but there aren't many available that would care about testing.
For instance, I've hacked 3.3 Animation System support in, but we have yet to see if there are more bugs than only #252.
And the update system together with maybe cutting out modules like nifty to be not part of the standard distribution, requires even more testing, of course. But it could help us with future updates.
Maybe I should just do a testing build and see what happens

If there's a build you want tested, I'll install and test it. However, I might not test it very thoroughly.


Migration to MAVLink organisation

Copying @hamishwillee's message on the PX4 Slack, to keep a record of the discussion.
The MAVLink dev team accepted the "MAVLink RFC Proposal" so we'll be taking the Dronecode SDK into the MAVLink organisation and renaming it to the "MAVSDK": https://github.com/mavlink/rfcs/blob/master/text/0011-mavlink-sdk.md
Code

[ ] Agree new names for SDK repos
[ ] Move SDK repos into github mavlink org
[ ] Agree new names for namespaces and other code items
[ ] Update code in line with new names
[ ] Update READMEs with appropriate new links to repos and docs
[ ] Make an official first release. Would this be 1.0?
[ ] Update all the proto/language repos
[ ] Cut release for those too - e.g. Python

Docs

[ ] Confirm name of docs repo in mavlink org [I think sdk_docs to mavsdk_docs]
[ ] Move docs repo into mavlink org
[ ] Update head revision docs API reference
[ ] Update docs guide materials
[ ] Review/update docs on marketing/positioning. Ie where it is hosted and the goals.
[ ] Decide what to do about previous releases/docs. These all will refer to the Dronecode SDK.
[ ] Propose that we delete all the old ones and start from scratch with a master release and initial release
[ ] Re-sync via gitbook
[ ] Get new domain/URL.
Current: https://sdk.dronecode.org/en/. Proposed: https://mavsdk.mavlink.io/en/. This is probably easier than try to do mavlink.io/mavlink, mavlink.io/mavsdk etc.
[ ] Themes updates
The theme for the SDK becomes more like the mavlink.io one: https://mavlink.io/en/
[ ] Other dronecode sites will need to have theme updated to point to SDK and any notes about SDK updated too.

DroneCode Website

[ ] Update top bar
This is likely to be SLOW process.
[ ] Update SDK page and other mentions.

SDK repos: I propose mavsdk-proto, mavsdk, mavsdk-swift, mavsdk-python, mavsdk-java, mavsdk-js.
Namespaces, code items: I would use "mavsdk" (lowercase) as much as possible.
First release: I wouldn't synchronize the github releases between the different repos. Also, I wouldn't make 1.0 just because of a name change. I would move to 1.0 once all the API is autogenerated (C++ is the next and last one coming for that). However, we should release packages when possible (I'm thinking about CocoaPods and python-pip here).

For the SDK repos I'd prefer MAVSDK-Proto. Although it depends on the font and logo as well there.

@julianoes and @JonasVautherin Why don't you sit down together and agree what you want to do - and update the post/s above appropriately :-)
FWIW

SDK repos. I'm ambivalent. I slightly lean towards MAVSDK  in text when referring to the project and product. However I don't extend that desire to package or repo names. Ie. We could use mavsdk-proto for the repo URL and still refer to it in the readme as MAVSDK-Proto.

We need a new URL. Anyone got objection to https://mavsdk.mavlink.io/en/ (or better suggestions)?
We need a new Logo, presumably with a new tagline - maybe something "like":

@LorenzMeier @mrpollo Are either of you able to help with the above - ie resourcing/making it happen?

Maybe the slack channel #sdk also need rename to #MAVSDK

Just so we are clear @hamishwillee, you are asking for help creating a logo for MAVSDK?

Just so we are clear @hamishwillee, you are asking for help creating a logo for MAVSDK?

@mrpollo Yes.
That logo might include a tagline like "MAVLink APP and Integration Toolkit". If it includes a tagline that also need to be discussed/agreed.

"MAVLink APP and Integration Toolkit"

APP -> App, it's not an acronym. I'm not super happy with the tagline but I can't think of something better yet.

I made a logo for you to choose. Hope you like it.

I made a logo for you to choose, hope you like it.

@julianoes I'm quite happy with @bys1123 "MAVLink Integration Toolkit" (though to me "APP" is fine in the previous case, because it is well understood.
I'd love a designer in on the logo to confirm colours etc. Stone, I don't hate the icon, but "what are you trying to say" - ie this is not like the plane with radio comms logo.

@hamishwillee Yes, the color not well adjustment yet.
The reason why it's looks like this:
0. When people talk about drone, This first thing come out in their mind is Quadcopter. So, the main logo is quad arrow. So, it makes people feels right on the outline. (We don't need to say this public)

This sdk is about control most of time, makes your MAV go some where .
Out of box.
Colors means it support different programming languages, also have many good features.
User-friendly. Lots of programmer in other industry start learning control drones through this SDK.

Consistent:

MAVLink logo is made about ten years ago. The draft feeling is not fit the power of this brand and project.
Adobe, Autodest, STM32 changed lots of logo in past years, "industrial feeling" is changed.
So, this MAVSDK logo is for fit the new trend of the industrial logo design. I tried my best make it will fit the new version MAVLink logo that will come out at some point by someone.

This thread is already discussing a lot of different topics, and the conversations are tough to follow. Thanks, @JonasVautherin, and @hamishwillee for putting this together, there are lots of areas of concern here, I recommend we create a Project Board and move the focused discussions elsewhere, per-issue if possible, creating the least overhead to everyone involved.

Thanks for the high energy and fast responses @bys1123, at this point, we are not ready for contributions, for the logo, tagline, or any branding collateral.

, I recommend we create a Project Board and move the focused discussions elsewhere, per-issue if possible, creating the least overhead to everyone involved.

Thanks @mrpollo
On the technical aspects @julianoes and @JonasVautherin should just make some decisions and let us know. That is enough for all the technical and docs side to proceed and does not require discussion with anyone. Though if they want to rubber stamp those, we can discuss in next weeks MAVLink meeting.
Julian, Jonas, can you confirm the names of repos, and name changes you want? Essentially summarise and provide a combined response for https://github.com/Dronecode/DronecodeSDK/issues/749#issuecomment-492962221 and https://github.com/Dronecode/DronecodeSDK/issues/749#issuecomment-492974967
For non technical aspects, yes, makes sense to at least have issues for separate tracking (because these require a lot of chit-chat).
I have created https://github.com/Dronecode/DronecodeSDK/issues/750 for the logo. It has one question that should be answered before it gets discussed - who is going to do the final design work.

I edited the issue after discussions with @julianoes. I believe we are aligned on the "code" points, and will start renaming after #724 is merged (should be next week). I believe we could start moving before that, as the redirection should make it transparent.

Julian, Jonas, can you confirm the names of repos, and name changes you want?

That's edited in the issue:

The official name for communications/docs is MAVSDK.
Repos will be named MAVSDK, MAVSDK-Proto, MAVSDK-Swift, MAVSDK-Python, etc.
In the code, we will prefer lowercase for readability (e.g. mavsdk:: as a namespace, and probably class MavSdk for a class.

@mrpollo: Would you mind renaming #sdk to #mavsdk on Slack (last point on the list above)?

@hamishwillee: I mostly agree with the Documentation points above, I'd like clarification on:

Decide what to do about previous releases/docs. These all will refer to the Dronecode SDK.
Propose that we delete all the old ones and start from scratch with a master release and initial release

So you suggest that the mavsdk docs start with master on the mavsdk repo, and we remove all the previous releases because the documentation says "DronecodeSDK" everywhere and we don't want to create confusion or go edit all occurrences there. Correct? I would be fine with that as well.

So you suggest that the mavsdk docs start with master on the mavsdk repo, and we remove all the previous releases because the documentation says "DronecodeSDK" everywhere and we don't want to create confusion or go edit all occurrences there. Correct? I would be fine with that as well.

@JonasVautherin Yes! I'd also like to do an official release "as an SDK" as soon as possible once we move to MAVSDK.
Re the docs, are you OK with repo move from sdk_docs to mavsdk_docs or would you prefer consistency MAVSDK-Docs?
FYI I like the consistency. The reason I tend to use underscores because when you select a bunch of text for copying it tends to break on "-" but not _ (may seem trivial, but I do a lot of text selection/copy/paste).

I moved the Java SDK for a first test, seems alright: https://github.com/mavlink/MAVSDK-Java

I moved the docs repo https://github.com/mavlink/MAVSDK-docs but now have lost admin rights and have broken auto builds from legacy gitbook.com. Chasing @dagar to build locally.

@hamishwillee I made you admin for MAVSDK-docs.

@hamishwillee: FYI, all the repos are now in the mavlink org :+1:.

Awesome. @JonasVautherin FYI I can't tick off my items above now because I don't have needed rights in this repo. I have moved docs to MAVSDK-docs. Note that we can now no longer build them. Waiting on Dagar for support.

@hamishwillee: Just gave you the rights!

@hamishwillee this page now 404s https://sdk.dronecode.org/ Is it possible to have it forward to the new docs page, best case including the path so that each link goes to its new page?

@julianoes Not by me (limitations of legacy gitbook). See if you can get @LorenzMeier to do the redirect at the domain level.

@JonasVautherin This can be closed now I think!

Agreed. There is still some renaming going on in the code but that will come eventually :-).


protobuf is required for all (Apple Silicon & intel clip as well)

https://substrate.stackexchange.com/questions/4015/failed-to-run-custom-build-command-for-libp2p-core-v0-34-0
error: failed to run custom build command for libp2p-core v0.38.0
Caused by:
process didn't exit successfully: /Users/adibir/Desktop/Impactility/substrate-node-template/target/release/build/libp2p-core-af21bf2b03805b28/build-script-build (exit status: 101)
--- stderr
thread 'main' panicked at 'Could not find protoc installation and this build crate cannot proceed without
this knowledge. If protoc is installed and this crate had trouble finding
it, you can set the PROTOC environment variable with the specific path to your
installed protoc binary.You could try running brew install protobuf or downloading it from https://github.com/protocolbuffers/protobuf/releases
For more information: https://docs.rs/prost-build/#sourcing-protoc
', /Users/adibir/.cargo/registry/src/index.crates.io-6f17d22bba15001f/prost-build-0.11.9/src/lib.rs:1457:10
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...

@gavofyork
take a look


Format Document messes up indent in Blazor razor editor

This issue has been moved from a ticket on Developer Community.

Hi there, when writing code blocks inside the Blazor razor editor, the indenting goes a bit funny.  Please see recording

Original Comments
Feedback Bot on 4/30/2023, 05:31 AM:
(private comment, text removed)

Original Solutions
(no solutions)

Looks like we did a bad job migrating this one, and have lost the recording mentioned. There is a comment on the internal issue that says "The HTML inside of the C# Switch inside of an HTML Div inside of a C# If is indented the wrong amount.  Switch uses modern "range" cases, like <30." but I am unable to repro bad formatting behaviour with that:

Going to close this as no repro.


Deploying Rancher on k3s with traefik has nginx annotations in ingress object

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Deploy Rancher on a k3s cluster with default options

Result:
The ingress object created by the Rancher Helm Chart creates nginx annotations:
apiVersion: v1
items:
- apiVersion: extensions/v1beta1
  kind: Ingress
  metadata:
    annotations:
      field.cattle.io/publicEndpoints: '[{"addresses":["<IP_ADDRESS>"],"port":80,"protocol":"HTTP","serviceName":"cattle-system:rancher","ingressName":"cattle-system:rancher","hostname":"myhost","allNodes":false}]'
      nginx.ingress.kubernetes.io/proxy-connect-timeout: "30"
      nginx.ingress.kubernetes.io/proxy-read-timeout: "1800"
      nginx.ingress.kubernetes.io/proxy-send-timeout: "1800"
      nginx.ingress.kubernetes.io/ssl-redirect: "false"
    creationTimestamp: "2020-04-14T04:43:27Z"
    generation: 1
    labels:
      app: rancher
      chart: rancher-2.4.2
      heritage: Helm
      release: rancher
    name: rancher
    namespace: cattle-system
    resourceVersion: "2477"
    selfLink: /apis/extensions/v1beta1/namespaces/cattle-system/ingresses/rancher
    uid: 49dcaf1e-09ea-425a-912c-78314a9e19b8
  spec:
    rules:
    - host: myhost
      http:
        paths:
        - backend:
            serviceName: rancher
            servicePort: 80
  status:
    loadBalancer:
      ingress:
      - ip: <IP_ADDRESS>

When deploying the Helm Chart on k3s & traefik, these annotations should not be added since nginx is not used as the ingress controller. If there are traefik equivalent settings for proxy and redirects, those should be used instead.
Other details that may be helpful:
This doesn't seem to cause a noticeable problem, but should be cleaned up.
See https://github.com/rancher/rancher/blob/master/chart/values.yaml#L50-L52
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v2.4.2
Installation option (single install/HA):
HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
k3s
Machine type (cloud/VM/metal) and specifications (CPU/memory):
AWS - m5.large - 8GB RAM, 2vCPU
Kubernetes version (use kubectl version):

# kubectl version
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.4+k3s1", GitCommit:"3eee8ac3a1cf0a216c8a660571329d4bda3bdf77", GitTreeState:"clean", BuildDate:"2020-03-25T16:13:25Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.4+k3s1", GitCommit:"3eee8ac3a1cf0a216c8a660571329d4bda3bdf77", GitTreeState:"clean", BuildDate:"2020-03-25T16:13:25Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):
NA

I ran across this (and a couple others) while trying to troubleshoot the very noticeable problem of not being able to connect to the Rancher UI after deploying in a k3s cluster. I think this is fixed by #26870 and made it into v2.4.3-rc4.

These annotations are still there :D


vsce/asvetliakov.vscode-neovim: init at 0.0.82

Motivation for this change
We have vscodevim but not vscode-neovim
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build vscode-extensions.asvetliakov.vscode-neovim

Result of nixpkgs-review pr 139452 run on x86_64-linux 1

1 package built:

vscode-extensions.asvetliakov.vscode-neovim


[API] Weight Decay into Layer Class

Make Weight Decay Parameter for Layer Class. From Now on Weight Decay
can be defined layer by layer.
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #89. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ec2-54-180-96-14.ap-northeast-2.compute.amazonaws.com/.


Layout - planning phase

https://app.diagrams.net/#G1MWBMQZ8l6J2Q7banhtw1ebdR1zvZLgWX
-moved the header pic to the front of the navbar because for mobile I want them to display block vs in the wider pages it should be display inline.

the name 40Kwartools does not seem to be taken.  just referenced for Dark Tide so I think we are good on the name.
So the header will always take us to home where it will show the mission statement.
-the VP tracker will be basic to begin with but needs to account for game setup.
Army Builder will be a dead link for now.
Contact Us will be pending.
So we need a media query for the qidest of phones.  ipads and tablets can see this in portrait.

Having issues with it lining up the way I want.  Flex ain't helping though in it's current form this is the layout for the mobile.  Though the text on the banners need to be smaller.
Also trying to get a border on this.


JavaScript error

What version of WebTorrent Desktop? 0.3.3
What operating system and version? OSX 10.10.5
What did you do? Hit CMD+Q while the app was downloading 3 torrents
What did you expect to happen? Quit the app without errors
What actually happened? This error:

Confirmed. I quit the app repeatedly and managed to trigger this error.

Fixed

Thank you @feross ! :)

This will be released in the next version of WebTorrent. The app will automatically update when the new version is released.
Cheers!


Proper location for the binary

It seems to be fairly unconventional to store the unlock binary in /Library/LaunchDaemons/. Given that homebrew sets crazy permissions on many /usr/local/ directories, and all of /System and /sbin are locked down by SIP, perhaps /Library/PrivilegedHelperTools/ would be a reasonable place to put the binary?

Given that homebrew sets crazy permissions on many /usr/local/ directories

We run as root during install, so that shouldn't be an issue. Want to open a PR?

Installers that write to /usr/local/bin/ work fine with Homebrew as long as it doesn't change ownership or permissions of that directory.

On Jan 27, 2016, at 3:48 PM, Justin Ridgewell<EMAIL_ADDRESS>wrote:
Given that homebrew sets crazy permissions on many /usr/local/ directories
We run as root during install, so that shouldn't be an issue. Want to open a PR?
—
Reply to this email directly or view it on GitHub.

To be clear, I'm not saying that homebrew's permissions make it tricky to install in /usr/local, I'm saying that they make it inappropriate to install anything in there which will be run as root, on boot :)


Allow for track changes

Just send to the server track changes. No point displaying them

Closed by 9a1416cd1ec9fbce3b77af50e675eb0367d4aeec


Update conda_mirror.py

Added an optional command line parameter to choose the temporary download location for applying the checks before moving the packages to the target location, which was hardcoded to the default temporary directory before. Now this is the default location, when not explicitly specifying the temp_directory parameter.
This should fix issue #19

Current coverage is 92.48% (diff: 100%)

Merging #21 into master will increase coverage by 0.03%

@@             master        #21   diff @@
==========================================
  Files             2          2
  Lines           212        213     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            196        197     +1
  Misses           16         16
  Partials          0          0

Powered by Codecov. Last update e201b61...332b24e

LGTM. Thanks @jneines . I'll cut a new release


Update fake backend to 4.x

Coverage remained the same at 100.0% when pulling db171e944f2311bdb89efea59d126ed52f461661 on update-fake-backend-to-4.x into 4df3ba1f88959746182c2099a41e55a165edf1bb on master.


Documentation additions that seem to be necessary to make code samples

syntactically legal.

Great, thank you!


TypeError: Cannot set properties of undefined (setting 'jwtClient')

I was following this tutorial on Youtube
When I came across this error:

Does anyone know how to fix this error?
Its the first time i see this error and no one get this too
This is the code:
const { GoogleSpreadsheet } = require('google-spreadsheet')
const credentials = require('./credentials.json')
const { promisify } = require("util")

const docID = '1XXAXjJva1kX00_A71dH4YU8QtAnfsFMJAKz07EakpM'

const accessSheet = async() => {
    const doc = new GoogleSpreadsheet(docID)
    await promisify(doc.useServiceAccountAuth)(credentials)
    const worksheet = info.worksheets[0]
    const info = await promisify(doc.getInfo)()
    const rows = await promisify(worksheet.getRows)({})
    console.log(rows)
}
accessSheet()

Try this solution
[https://stackoverflow.com/questions/64410856/react-js-and-google-spreadsheets-cannot-read-property-2-of-null]

I'm writing this just in case some other person encounters this error
Do not use promisify without binding
// incorrect code
promisify(doc.useServiceAccountAuth)(credentials)

// correct code
doc.useServiceAccountAuth(credentials)
// or
promiseify(doc.useServiceAccountAuth.bind(doc))(credentials)

I just released a new version which changes how auth is set up. Please take a look and reopen if you are still having issues.


add runner accessors to Command

Fixes #121
Please note the intended 2.0 goal mentioned by #120 - once runner initialization is added to the command, we can have a default runner result of Peridot\Runner\Runner, similar to how the default loader mechanism works in the command.

@austinsmorris can I get your thoughts on all the things mentioned here + a merge if things seem cool to you?

Coverage increased (+1.57%) to 96.38% when pulling e0a4ec6995d75a408a607f78cab2544ac60fe228 on brianium:feature/command-set-runner into a100d757f1a8dbc48927b82edab18fe25d8cc1c1 on peridot-php:master.

Looks good, but why did the last commit drop coverage by 5%?

@austinsmorris not sure on coverage details, as usual - hhvm coverage seems to be flawed. something with lib we are using? I definitely don't want that on the repo page. I'll look into it.

@austinsmorris yeah, it looks like there is an open issue on hhvm debug things, so I just reran the build and it put it back at 95. This should be good to merge.


feat(batch): vnode check for batch queries

Is your feature request related to a problem? Please describe.
The vnode must be correct for Scan executors of batch queries. We should add an assertion here.

I think the vnode check should be done in upper layer when creating the batch executor.
white_check_mark
When creating the batch executor / batch storage table, we should check whether vnode info matches with MV.

See more on Link to Slack discussion
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

@wcy-fdu Can we do this on state table layer? If not, please feel free to assign to others.

cc @xxhZs This issue is relevant to reading from non-checkpoint epoch.

Should be done on relational table layer.


ztunnel trust_domain not respecting Istio setting

Bug Description
When running Istio in Ambient mode and configuring routing to a waypoint proxy, I have noticed that the SAN that ztunnel expects from the waypoint proxy does not reflect the trustDomain set in the IstioOperator object:
apiVersion: install.istio.io/v1alpha1
  kind: IstioOperator
  metadata:
    name: ew-gateway-mgr
    namespace: gm-iop
  spec:
    meshConfig:
      trustDomain: cluster-1

Error:
2023-02-28T21:56:23.949875Z  INFO xds{id=20}: ztunnel::xds::client: received response type_url="type.googleapis.com/istio.workload.Workload" size=1
2023-02-28T21:56:23.949995Z  INFO xds{id=20}: ztunnel::workload: handling delete <IP_ADDRESS>
2023-02-28T21:56:42.551643Z  INFO ztunnel::proxy::inbound_passthrough: accepted connection source=<IP_ADDRESS>:29450 destination=<IP_ADDRESS>:443 component="inbound plaintext"
2023-02-28T21:56:43.488518Z  INFO xds{id=20}: ztunnel::xds::client: received response type_url="type.googleapis.com/istio.workload.Workload" size=1
2023-02-28T21:56:45.357910Z  INFO outbound{id=7979f9edf74ec2a700309c20e5bcfa58}: ztunnel::proxy::outbound: proxying to <IP_ADDRESS>:9080 using node local fast path
2023-02-28T21:56:45.358265Z  INFO outbound{id=7979f9edf74ec2a700309c20e5bcfa58}: ztunnel::proxy::outbound: complete dur=445µs
2023-02-28T21:56:45.540204Z  INFO xds{id=20}: ztunnel::xds::client: received response type_url="type.googleapis.com/istio.workload.Workload" size=2
2023-02-28T21:56:49.994931Z  INFO outbound{id=dba006d10fcfc010944a26796970a8f4}: ztunnel::proxy::outbound: proxy to <IP_ADDRESS>:9080 using HBONE via <IP_ADDRESS>:15008 type ToServerWaypoint
2023-02-28T21:56:49.998062Z  INFO outbound{id=dba006d10fcfc010944a26796970a8f4}: ztunnel::tls::boring: failed verifying TLS: san verification error: remote did not present the expected SAN (spiffe://cluster.local/ns/bookinfo/sa/bookinfo-ratings-istio-waypoint), got [Spiffe { trust_domain: "cluster-1", namespace: "bookinfo", service_account: "bookinfo-ratings-istio-waypoint" }]
2023-02-28T21:56:49.998214Z  WARN outbound{id=dba006d10fcfc010944a26796970a8f4}: ztunnel::proxy::outbound: failed dur=3.364375ms err=tls handshake failed: Failure(MidHandshakeSslStream { stream: SslStream { stream: WrappedStream { inner: PollEvented { io: Some(TcpStream { addr: <IP_ADDRESS>:37484, peer: <IP_ADDRESS>:15008, fd: 40 }) }, ext: None }, ssl: Ssl { state: "TLS 1.3 client read_server_certificate_verify", verify_result: X509VerifyResult { code: 1, error: "unknown certificate verification error" } } }, error: Error { code: ErrorCode(1), cause: Some(Ssl(ErrorStack([Error { code: 268435581, library: "SSL routines", function: "OPENSSL_internal", reason: "CERTIFICATE_VERIFY_FAILED", file: "../ssl/handshake.cc", line: 385 }]))) } })

ztunnel config_dump:
ztunnel_config.txt
Version
$ ./out/darwin_arm64/istioctl version
client version: 1.18-dev
control plane version: 1.18-alpha.54a87be0b45d834e26da33224cad0770b342d3ab
data plane version:  (3 proxies), 1.18-alpha.54a87be0b45d834e26da33224cad0770b342d3ab (5 proxies)

Additional Information
No response

Fixed by https://github.com/istio/ztunnel/pull/437


Generate Genomics England evidence as genetic_assocation type

Currently we generate the Genomics England evidence as genetic_literature data type and then the pipeline hardcoded replaces genetic_literature with genetic_association.
We should generate it correctly as genetic_association in the first place. Note this will change how it is validated by the schema, so we need to ensure it is still valid after this change.

genetic_literature refers to evidence strings that do not provide variant information. Therefore they need to be validated against a different JSON schema than genetic_association.


Need to add target_group_arns to ASG configuration

In the case of adding an Application Load Balancer (ALB) to front the Consul cluster, we need to have the ALB's Target Group pointing to the ASG. Since target_group_arns is not part of ASG's configuration, the Terraform module is trying to remove any attached Target Group, which is not what we want, e.g.
Terraform will perform the following actions:

  ~ module.prod_consul_1.module.consul_servers.aws_autoscaling_group.autoscaling_group
      target_group_arns.#:         "1" => "0"
      target_group_arns.842221997: "arn:aws:elasticloadbalancing:us-east-2:423487953385:targetgroup/my-lb-tg/2f4d7259f471d85a" => ""

https://github.com/hashicorp/terraform-aws-consul/pull/71

Turns out this is an issue in v0.2.2 and has been addressed in this change by removing the target_group_arns argument from the ASG. See above closed pull request for details.


usec timestamps option is not honoured on MySQL migration

When a column is specified as utc_datetime or native_datetime in a migration, having usec: true does not cause the creation of a datetime(6) type column, but rather a plain datetime. In mysql 5.7 (which supports sub-second resolution) the latter truncates sub-second values (while the former preserves the sub-second information).
Now this may be intentional - there's a note about usecs in the mysql adapter, but it just notes differences between versions and advises that we consult the documentation :) I can't see any "right" way to get the appropriate type set on the column beyond hardcoding some raw SQL into the migration.
Environment

Elixir version: 1.4.1
Database and version: MySQL 5.7
Ecto version: 2.1.4
Database adapter and version: mariaex 0.8.1
Operating system: Ubuntu 16.04

Yes, the best is to update the usec notes about MySQL and say they can
explicitly pass "datetime(6)" type in migrations.
José Valim
www.plataformatec.com.br
Skype: jv.ptec
Founder and Director of R&D

Is there a way to specify that as part of a "normal" Ecto type declaration? In particular I want to use it for the timestamps() shortcut to add created_at and updated_at, but I can't just set:
@timestamps_opts [type: "datetime(6)"]

You do not define it as part of your schema but rather as part of migrations. There you can do:
add :created_at, :datetime, size: 6

I am not sure if timestamps(size: 6) in migrations is supported. If not, we should.

I am not sure if timestamps(size: 6) in migrations is supported. If not, we should.

A quick test suggests it's not supported. I might take a stab at getting that working. Thanks very much for the pointers.

A more correct test suggests it is in fact supported, just not as part of the @timestamps_opts attribute :) So I've just updated the docs to explain it. The PR is #2013. Thanks very much for your help :)

I will keep this open though because I think we can do a better job here.

Closing it in favor of #2018.


AnimeSama: Fix Website changes

Checklist:

[X] Updated extVersionCode value in build.gradle for individual extensions
[ ] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[ ] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[ ] Added the isNsfw = true flag in build.gradle when appropriate
[X] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[X] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

Not finished yet, i just discovered a bug

Fixed in this version. Can be deployed.

I've tested a few things, you let me know if it's better, in any case thanks you've taught me a lot of stuff, I've never really touched kotlin before and I've done very little java so it's a bit new.


Added Android library

Added a link to the android library for katex rendering.

Please note that correct capitalization of the name is KaTeX, with capital T and X, just like TeX.

Please note that correct capitalization of the name is KaTeX, with capital T and X, just like TeX.

@ylemkimon  Hey thanks for pointing that out, I'm a little confused as to where you would like me to change that, should I change that in the md file or the library name and description itself?

@ylemkimon made the changes, let me know if anything is out of order.

[clabot:check]

[clabot:check]


Pagination

Pagination Implementation
Summary
We need to implement pagination to handle large sets of data efficiently.
Detailed Description
Currently, our application retrieves all data entries at once, which can lead to performance issues and long loading times. Implementing pagination will allow us to load data in smaller chunks, improving the user experience and reducing the load on the server.
Requirements

Fetch data from the backend using pagination parameters.
Display the data in a paginated format.
Implement pagination controls (next, previous, and page numbers).

Additional Notes

Ensure compatibility with existing data fetching mechanisms.
Maintain consistent user experience while navigating through paginated data.
Consider edge cases such as empty pages, single-page datasets, and large datasets.

Completed. The final design is a bit different from the proposed one though.


Update pom.xml

@jpwhitemn, @tsconn23, need one of your review since I am not a committer on this repo.

I forgot to sign off..let me fix


Remove deprecated application from publick8s

Remove some deprecated helm charts from publick8s which were used for Jenkins election 2019.
Once removed here, I'll manually delete them from the cluster.
Signed-off-by: Olivier Vernin<EMAIL_ADDRESS>

The oauth-proxy helm chart is not supported anymore -> https://github.com/helm/charts/blob/master/stable/oauth2-proxy/
The dex helm chart is not supported anymore -> https://github.com/helm/charts/tree/master/stable/dex
In the current state, the poll helm chart won't be used anymore


Implemented the Message Queue Server in Java

It is fixed.
On Mon, 18 Nov, 2024, 3:17 pm Ashu Kumar, @.***> wrote:

@.**** commented on this pull request.
In src/main/java/com/clustercrew/messagequeue/BookKeeperClient.java
https://github.com/yadavpa1/DistributedMessageQueue/pull/10#discussion_r1847287726
:

           "password".getBytes(StandardCharsets.UTF_8))) {

       System.out.println("Ledger created for topic: " + topic + ", partition: " + partition + ", ID: " + ledger.getId());

   }

}

/**

* Writes a message to the specified topic partition ledger.

*

* @param topic     The topic name.

* @param partition The partition number.

* @param message   The message to write.

* @throws Exception If an error occurs while writing the message.

*/

public void writeMessage(String topic, int partition, Message message) throws Exception {

   try (LedgerHandle ledger = bookKeeper.openLedger(

           generateLedgerId(topic, partition),

How can we use custom genereateLedgerId() if creating a ledger is not
using a topic and partition to create the id.
—
Reply to this email directly, view it on GitHub
https://github.com/yadavpa1/DistributedMessageQueue/pull/10#pullrequestreview-2443751134,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKC6J7KG5YNUXRKTKSBD5PT2BJKOLAVCNFSM6AAAAABR7FY2XCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINBTG42TCMJTGQ
.
You are receiving this because you were assigned.Message ID:
@.***>


Update golang version in Dockerfile

Pipeline in drone use go1.11.
This PR update go version to actual in Dockerfile.

Codecov Report

Merging #5246 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5246      +/-   ##
==========================================
+ Coverage   37.62%   37.65%   +0.03%
==========================================
  Files         310      310
  Lines       46039    46039
==========================================
+ Hits        17320    17338      +18
+ Misses      26240    26223      -17
+ Partials     2479     2478       -1

Impacted Files
Coverage Δ

models/repo_list.go
63.29% <0%> (-1.27%)
:arrow_down:

models/repo_indexer.go
50% <0%> (+5.5%)
:arrow_up:

modules/indexer/repo.go
69.84% <0%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00533d3...f21bbdd. Read the comment docs.


Added content for mail inbox- forgot password

Message and subject for the mail if user forgets password and demands OTP.
This is with respect to issue #147

Hey @Jigyasa-G, the content looks good, however we need to add it over here, so that the mail received contains your written text.


Check validity of admin_code in boundary layer params

Currently, we do not check for the validity of admin_code in boundary definitions. This can lead to mismatch down the road, and break calls to the analysis API.
One easy solution would be to add a check in fetchBoundaryLayerData that the admin_code parameter indeed appears in the geojson properties that we are loading.

Linked to #994


ECS provider panic

What version of Traefik are you using (traefik version)?
v1.2.0-rc1
What is your environment & configuration (arguments, toml...)?
[web]
address = ":8080"
[ecs]
Cluster = "staging"
Watch = true
RefreshSeconds = 15
ExposedByDefault = false
Region = "us-east-1"
What did you do?
Scaled the amount of tasks running that have containers with these labels:
traefik.frontend.rule=Host:test.traefik.io
traefik.enable=true
What did you expect to see?
I expected to see the new tasks to be registered in traefik and for traefik to route traffic to them.
What did you see instead?
goroutine 81 [running]:
runtime/debug.Stack(0x4e808f, 0xc42008e240, 0x1d9a271)
	/usr/local/go/src/runtime/debug/stack.go:24 +0x79
runtime/debug.PrintStack()
	/usr/local/go/src/runtime/debug/stack.go:16 +0x22
github.com/containous/traefik/safe.defaultRecoverGoroutine(0x1ada8e0, 0xc420012030)
	/go/src/github.com/containous/traefik/safe/routine.go:147 +0x7d
github.com/containous/traefik/safe.OperationWithRecover.func1.1(0xc4203d3e60)
	/go/src/github.com/containous/traefik/safe/routine.go:155 +0x56
panic(0x1ada8e0, 0xc420012030)
	/usr/local/go/src/runtime/panic.go:458 +0x243
text/template.errRecover(0xc4203d3a50)
	/usr/local/go/src/text/template/exec.go:140 +0x2ad
panic(0x1ada8e0, 0xc420012030)
	/usr/local/go/src/runtime/panic.go:458 +0x243
github.com/containous/traefik/provider.(*ecsInstance).Host(0xc4207bb3c8, 0x0, 0x0)
	<autogenerated>:38 +0x7b
reflect.Value.call(0x1c17300, 0xc4207bb3c8, 0x613, 0x1d78247, 0x4, 0x2b8cdd8, 0x0, 0x0, 0x5e9c03, 0x2b6bba0, ...)
	/usr/local/go/src/reflect/value.go:434 +0x5c8
reflect.Value.Call(0x1c17300, 0xc4207bb3c8, 0x613, 0x2b8cdd8, 0x0, 0x0, 0xc420028fb8, 0x7, 0x7)
	/usr/local/go/src/reflect/value.go:302 +0xa4
text/template.(*state).evalCall(0xc4203d39d0, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0x1c17300, 0xc4207bb3c8, 0x613, 0x2a85ea0, 0xc4208d2630, 0xc42046e087, ...)
	/usr/local/go/src/text/template/exec.go:658 +0x530
text/template.(*state).evalField(0xc4203d39d0, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0xc42046e087, 0x4, 0x2a85ea0, 0xc4208d2630, 0xc420012dc0, 0x1, ...)
	/usr/local/go/src/text/template/exec.go:549 +0xdb6
text/template.(*state).evalFieldChain(0xc4203d39d0, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0x2a85ea0, 0xc4208d2630, 0xc420012db0, ...)
	/usr/local/go/src/text/template/exec.go:520 +0x22f
text/template.(*state).evalFieldNode(0xc4203d39d0, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0xc4208d2630, 0xc420012dc0, 0x1, 0x1, 0x0, 0x0, ...)
	/usr/local/go/src/text/template/exec.go:484 +0x114
text/template.(*state).evalCommand(0xc4203d39d0, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0xc4208d2600, 0x0, 0x0, 0x0, 0xc4203d3568, 0x78ac3f, ...)
	/usr/local/go/src/text/template/exec.go:422 +0x866
text/template.(*state).evalPipeline(0xc4203d39d0, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0xc42069be50, 0x0, 0x0, 0x98)
	/usr/local/go/src/text/template/exec.go:400 +0xee
text/template.(*state).walk(0xc4203d39d0, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0x2a85c60, 0xc4208d2660)
	/usr/local/go/src/text/template/exec.go:226 +0x423
text/template.(*state).walk(0xc4203d39d0, 0x1d0b1a0, 0xc4207bb3c8, 0x199, 0x2a85fc0, 0xc4208d22d0)
	/usr/local/go/src/text/template/exec.go:234 +0x13d
text/template.(*state).walkRange.func1(0x19a9060, 0xc4200130f0, 0x82, 0x1d0b1a0, 0xc4207bb3c8, 0x199)
	/usr/local/go/src/text/template/exec.go:321 +0x132
text/template.(*state).walkRange(0xc4203d39d0, 0x1b06920, 0xc42051f260, 0x99, 0xc4209ff8c0)
	/usr/local/go/src/text/template/exec.go:330 +0x300
text/template.(*state).walk(0xc4203d39d0, 0x1b06920, 0xc42051f260, 0x99, 0x2a86140, 0xc4209ff8c0)
	/usr/local/go/src/text/template/exec.go:237 +0x59c
text/template.(*state).walk(0xc4203d39d0, 0x1b06920, 0xc42051f260, 0x99, 0x2a85fc0, 0xc4208d2210)
	/usr/local/go/src/text/template/exec.go:234 +0x13d
text/template.(*Template).execute(0xc4209ff7c0, 0x2a67e80, 0xc4201e0070, 0x1b06920, 0xc42051f260, 0x0, 0x0)
	/usr/local/go/src/text/template/exec.go:189 +0x204
text/template.(*Template).Execute(0xc4209ff7c0, 0x2a67e80, 0xc4201e0070, 0x1b06920, 0xc42051f260, 0x0, 0x0)
	/usr/local/go/src/text/template/exec.go:175 +0x53
github.com/containous/traefik/provider.(*BaseProvider).getConfiguration(0xc42017eea0, 0x1d91332, 0x12, 0xc42053f5c0, 0x1b06920, 0xc42051f260, 0x20, 0x0, 0x0)
	/go/src/github.com/containous/traefik/provider/provider.go:90 +0x561
github.com/containous/traefik/provider.(*ECS).loadECSConfig(0xc42017eea0, 0x2a804c0, 0xc42008ea00, 0xc4209ceab0, 0x1e930b5, 0x1e930b4, 0xc41ffd12f5)
	/go/src/github.com/containous/traefik/provider/ecs.go:189 +0x3f4
github.com/containous/traefik/provider.(*ECS).Provide.func2.2(0x0, 0x0)
	/go/src/github.com/containous/traefik/provider/ecs.go:121 +0xc4
github.com/containous/traefik/safe.OperationWithRecover.func1(0x0, 0x0)
	/go/src/github.com/containous/traefik/safe/routine.go:159 +0x75
github.com/containous/traefik/vendor/github.com/cenk/backoff.RetryNotify(0xc4203fe370, 0x2a73a80, 0xc4203fe380, 0x1e96e28, 0xc4203fe360, 0x0)
	/go/src/github.com/containous/traefik/vendor/github.com/cenk/backoff/retry.go:32 +0x3f
github.com/containous/traefik/provider.(*ECS).Provide.func2(0xc420444620)
	/go/src/github.com/containous/traefik/provider/ecs.go:158 +0x1d2
github.com/containous/traefik/safe.(*Pool).Go.func1()
	/go/src/github.com/containous/traefik/safe/routine.go:77 +0x3a
github.com/containous/traefik/safe.GoWithRecover.func1(0x1e96ec8, 0xc4204ee000)
	/go/src/github.com/containous/traefik/safe/routine.go:141 +0x4d
created by github.com/containous/traefik/safe.GoWithRecover
	/go/src/github.com/containous/traefik/safe/routine.go:142 +0x49
time="2017-02-08T19:12:08Z" level=error msg="ECS connection error Panic in operation: %!s(<nil>), retrying in 1.105608358s"```

Ping @lpetre :'(

Yikes I'll take a look asap

Attempting to address this here: https://github.com/containous/traefik/pull/1143
@emilevauge the panic is happing on this line:
https://github.com/containous/traefik/blob/master/provider/ecs.go#L381
Unfortunately the AWS Go SDK uses pointers for absolutely everything, and isn't really clear on when to expect nil values. I've added a bunch of additional checking and filtering, so it shouldn't be possible to get this panic in the future.  Let me know what you think.

Hi @emilevauge, this is fixed by PR #1143. @tskinn tested it locally. Let me know if you need anything done to get it merged.


[General Questions] Rough estimates for training time for pre-training CoAtNet?

Hi, 👋 Thanks for such an amazing library and taking out the time to implement so many parts of the CoatNet paper!
In your CoAtNet README, you mentioned you use TPU accelerators. Could you provide a ballpark for the amount of time it took for you to train the biggest models and the corresponding accelerators? I have a task for which I wish to use scaled-up models, but I'd have to pre-train on Imagenet first because of low data amount (<5-10M) and squeeze out maximum accuracy from fine-tuning.
I assume there might've been a few bottlenecks also, perhaps data? 🤔 If you could describe your setup, it would be very helpful to my experiments!
Sorry for bothering you with minor questions, and again thank you for all your work!

Thanks for quick response! 🤗 What was the GPU you were using?

For TPU, have no experience.
Cool, I'll be training the model in a few weeks' time. If I manage to get it working on TPUs, or have any hiccups I'll surely share it here on a separate thread.

Have a great day!

For input_shape 160, it's by 1 RTX 2080 Ti, for input_shape 224, it's RTX 8000. Looking forward for your results then. :)

I just did a pretty simple trial run on Colab w/ TPUs to see if its working:-
!pip install keras_cv_attention_models
!git clone https://github.com/leondgarse/keras_cv_attention_models.git
!cd keras_cv_attention_models; TF_XLA_FLAGS="--tf_xla_auto_jit=2" python3 ./train_script.py -m coatnet.CoAtNet0 --seed 0 --batch_size 128 -s CoAtNet0_160 --TPU -d 'cifar10' --disable_float16

for reproduction, and error:-
>>>> ALl args: Namespace(TPU=True, additional_model_kwargs={}, basic_save_name='CoAtNet0_160', batch_size=128, bce_threshold=0.2, cutmix_alpha=1.0, data_name='cifar10', disable_antialias=False, disable_float16=False, disable_positional_related_ops=False, distill_loss_weight=1, distill_temperature=10, enable_float16=True, epochs=-1, eval_central_crop=0.95, freeze_backbone=False, freeze_norm_layers=False, initial_epoch=0, input_shape=160, label_smoothing=0, lr_base_512=0.008, lr_cooldown_steps=5, lr_decay_on_batch=False, lr_decay_steps=100, lr_m_mul=0.5, lr_min=1e-06, lr_t_mul=2, lr_warmup=0.0001, lr_warmup_steps=5, magnitude=6, mixup_alpha=0.1, model='coatnet.CoAtNet0', num_layers=2, optimizer='LAMB', pretrained=None, random_crop_min=0.08, random_erasing_prob=0, rescale_mode='torch', resize_method='bicubic', restore_path=None, seed=0, summary=False, teacher_model=None, teacher_model_input_shape=-1, teacher_model_pretrained='imagenet', tensorboard_logs='auto', token_label_file=None, token_label_loss_weight=0.5, weight_decay=0.02)
2022-05-21 23:25:23.558107: E tensorflow/stream_executor/cuda/cuda_driver.cc:271] failed call to cuInit: CUDA_ERROR_NO_DEVICE: no CUDA-capable device is detected
[TPU] All devices:  [LogicalDevice(name='/job:worker/replica:0/task:0/device:TPU:0', device_type='TPU'), LogicalDevice(name='/job:worker/replica:0/task:0/device:TPU:1', device_type='TPU'), LogicalDevice(name='/job:worker/replica:0/task:0/device:TPU:2', device_type='TPU'), LogicalDevice(name='/job:worker/replica:0/task:0/device:TPU:3', device_type='TPU'), LogicalDevice(name='/job:worker/replica:0/task:0/device:TPU:4', device_type='TPU'), LogicalDevice(name='/job:worker/replica:0/task:0/device:TPU:5', device_type='TPU'), LogicalDevice(name='/job:worker/replica:0/task:0/device:TPU:6', device_type='TPU'), LogicalDevice(name='/job:worker/replica:0/task:0/device:TPU:7', device_type='TPU')]
>>>> Set random seed: 0
2022-05-21 23:25:40.850248: W tensorflow/core/platform/cloud/google_auth_provider.cc:184] All attempts to get a Google authentication bearer token failed, returning an empty token. Retrieving token from files failed with "NOT_FOUND: Could not locate the credentials file.". Retrieving token from GCE failed with "NOT_FOUND: Error executing an HTTP request: HTTP response code 404".
>>>> init_model kwargs: {'input_shape': (160, 160, 3)}
>>>> Built model name: coatnet0
>>>> RandAugment: magnitude = 6, translate_const = 0.450000, cutout_const = 28.800000
>>>> Both mixup_alpha and cutmix_alpha provided: mixup_alpha = 0.1, cutmix_alpha = 1.0
>>>> Loss: BinaryCrossEntropyTimm, Optimizer: LAMB
>>>> basic_save_name = CoAtNet0_160
>>>> TensorBoard log path: logs/CoAtNet0_160_20220521-232552
Traceback (most recent call last):
  File "./train_script.py", line 223, in <module>
    run_training_by_args(args)
  File "./train_script.py", line 214, in run_training_by_args
    model, epochs, train_dataset, test_dataset, args.initial_epoch, lr_scheduler, args.basic_save_name, logs=args.tensorboard_logs
  File "/content/keras_cv_attention_models/keras_cv_attention_models/imagenet/train_func.py", line 251, in train
    workers=8,
  File "/usr/local/lib/python3.7/dist-packages/keras/utils/traceback_utils.py", line 67, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/usr/local/lib/python3.7/dist-packages/tensorflow/python/framework/ops.py", line 1315, in graph
    "Tensor.graph is undefined when eager execution is enabled.")
AttributeError: Tensor.graph is undefined when eager execution is enabled.
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/dist-packages/tensorflow/python/eager/context.py", line 2685, in async_wait
    context().sync_executors()
  File "/usr/local/lib/python3.7/dist-packages/tensorflow/python/eager/context.py", line 740, in sync_executors
    pywrap_tfe.TFE_ContextSyncExecutors(self._context_handle)
tensorflow.python.framework.errors_impl.NotFoundError: Resource tpu_worker/_AnonymousVar7044/N10tensorflow22SummaryWriterInterfaceE does not exist.
	Encountered when executing an operation using EagerExecutor. This error cancels all future operations and poisons their output tensors.
2022-05-21 23:25:53.361613: W ./tensorflow/core/distributed_runtime/eager/destroy_tensor_handle_node.h:57] Ignoring an error encountered when deleting remote tensors handles: INVALID_ARGUMENT: Unable to find the relevant tensor remote_handle: Op ID: 6693, Output num: 0
Additional GRPC error information from remote target /job:worker/replica:0/task:0:
:{"created":"@1653175553.358294014","description":"Error received from peer ipv4:<IP_ADDRESS>:8470","file":"external/com_github_grpc_grpc/src/core/lib/surface/call.cc","file_line":1056,"grpc_message":"Unable to find the relevant tensor remote_handle: Op ID: 6693, Output num: 0","grpc_status":3}

I'll keep working to fix it in my spare time otherwise I'll have to fully commit to debugging this when my exams are over 😉

It's my fault added a keras.callbacks.TensorBoard callback recently, and set it enabled writing to a local path by default. Just changed it disabled by default, and updated basic training result on MNIST in kecam_test.ipynb, as MNIST is available in public gcs. Please also note for other dataset like cifar10, needs a custom data loading process from uploaded gcs path, which I'm not very familiar with...

Thanks for the speedy reply! You're right, apparently, GCP requires one to use a bucket to train on TPUs, or use a dataset available at TFDS. swapping CIFAR-10 to the flowers dataset yields no problems. Amazing work! 🌟

You may open another issue or discussion if anything new. Will just close this ATM. :)

works! I'm just fixing some stuff, and integrating WandB support. I must say, your codebase is remarkably clean and modular considering you're working with TF! pretty terrific, makes debugging very easy 👍

I'm rather glad heard this!


Modify grammar

Modified the sentence such that it doesn't end with a preposition.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/skills/overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


chore: Upgrading Node.js to v20

I have deprecation messages from GitHub )

@t-botz ping :)

Sorry for the delay, was slightly more complicated.
Checkout out v2.2.0


Add new user to "Headquarter" office by default

Any update about this?
How to create a new user? or How to register?

@krisnaw  Registration is through invitation, from admin page you can invite new user via email

@Hasnayeen
Thanks for the responses
How to access Admin page if you don't have any users? Any console command to create the first user?

Did you used the single file installer to install the app? there should be a admin user already if you installed via installer, otherwise you've to run seeder

I installed it manually.
My problem is solved :)
Thanks @Hasnayeen


Add information about custom builder image

Updating the readme to add commands for how to build the custom image
Also going to test if I can auto-merge with ci bot commands

/lgtm

/lgtm

/approve

/retest

/approve


cross-platform: Compile into module blocks

Combines static libraries from sub projects. (Linux, macOS)
ChakraCore consumer now links to 3 libs instead of 18.

Defines AutoSystemInfo etc. initializers builtin for ease of embedding

LGTM overall. One thing I don't understand -- can CH link to the bigger combined libs instead of the "Singular" ones? It'll be nice if we can skip building those Singular libs.

can CH link to the bigger combined libs instead of the "Singular" ones? It'll be nice if we can skip building those Singular libs.

The reason it doesn't use the combined lib for shared build is that, in such case CH would be already embedding everything available from shared. There we have to deal with AutoSystemInfo initializer etc....

Looks like my newly introduced dependency of std::map is causing build break in static Linux build, and your change should be able to fix that. Can you retarget the change to master so I can merge your change in? thx

rebased to latest.
@Yongqu unfortunately this is not for master. However, since there is no big change since the last merge we must be able merge linux to master painlessly (especially before introducing something big to linux branch) /cc @digitalinfinity  ?

@jianchun Thanks for the review.


chore: Update Caustics logo

With PR https://github.com/Ciela-Institute/caustics/pull/121, I don't think the logo needs updating unless it's for README.


Live

Docs Build status updates of commit 1d9531c:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Object connect and emit_signal could be unexposed as superseded by Signal methods

Godot version
v4.0.alpha1.official [31a7ddbf8]
System information
Win 64
Issue description
In the current official alpha build, I can use both GDScript1 and GDScript2 methods to connect and emit signals.
This makes it harder to discover and learn GDScript2 because I'm not pushed by the engine to change habits.
GDScript1 will also not be marked red while writing the code. Meaning there is no indication for a user this is going to be obsolete going forward.
Alpha1 also allows a mix of both, so as a user who wants to use GDScript2 I never quite know for sure whether I got it all or if part of it is still GDScript1 because I don't get any feedback.
Steps to reproduce
both of these work:
emit_signal("a")
connect("a", self, "react_to_a")

a.emit()
a.connect(self.react_to_a)

Minimal reproduction project
signal_test_G4_alpha1.zip

We should probably show a deprecation warning. Or unexpose connect(), there's no use-case where it's needed instead of the new syntax (aside from backwards-compatibility, but it can be auto-converted).

It's worth discussing indeed, but removing these methods does remove a handy and well-known option to connect and emit signals using strings, which can sometimes be a requirement (e.g. if you need to create the string programmatically "do_action_player%s" % player_id).
Without these methods, it's still possible, but not the most straightforward:
Signal(self, "mysignal").connect(Callable(self, "mymethod"))
Signal(self, "mysignal").emit()

"do_action_player%s" % player_id

That's what signal binds are for:
signal.connect(do_action_player.bind(player_id))
tbh I can't think of a use case where you need to use a constructed string for method/signal name.

I don't know what's the best solution for dynamic naming is since I have almost no GDScript2 experience. But what I do know is that signals are an essential part of Godot and one which is easily among the top3 hurdles (things being asked for help in the community channels) by beginners.
Since it is so essential and integral to Godot, I believe it's very important to make it the least confusing it can be. More syntax styles are therefore not really ideal or helping. I think we would have much less confusion amoungst beginners learning godot, if they need to learn about one syntax logic, not two, the old and the new.
Coding beginners will see no benefit of using less strings. They will only wonder "Why do I have to use strings here, but write in a completely different manner elsewhere? Why can't I use the string method for everything?".
Learning these abstract rules of logic is straining enough. Learning multiple variations of doing almost the same thing for no apparent reason makes it even harder.

"do_action_player%s" % player_id

That's what connection binds are for:
signal.connect(do_action_player.bind(player_id))
tbh I can't think of a use case where you need to use a constructed string for method/signal name.

Yeah from the top of my mind I can't really think about a good example either. I just wanted to raise it for discussion but that's pretty niche IMO - and thus it might be niche enough that we don't need to keep the compat methods for it.
Using constructed strings to access identifiers (e.g. action names for local multiplayer) is more common, but for signals and method names I never had to resort to such hacks myself.

To be honest I knew that I can connect signal in new way. But it is first time someone told me I can emit signal by using "signalname.emit()". So golddotasksquestions is kinda right.

There's also Object.call() and Object.call_deferred() which can be replaced with Callable.call() and Callable.call_deferred().
But thinking about it some more, I'm not sure this is a good idea because I'm not sure this is practical/possible to change. The Object methods are still needed in the core codebase, and the bindings are probably still relevant for GDExtension and C#.

I can't seem to make the first example work, if anybody could check this I would appreciate it.
I would like to use the old syntax for making a custom Visual Script Node. The current method of making a callable variable is a bit cumbersome and I would like to bypass it if possible by using the old connection method.
As well I sometimes use Object,call("method", arguments) when making custom nodes as the current VS call node is broken and will not pass arguments.

connect actually can't be unexposed, because it's used in connection dialog for UndoRedo. We could rename it to _connect(), to hide it from the view as internal method.

Whichever you decide to do, please update the docs accordingly. I had to fix my connection checks by trial and error as the docs are still somewhat lacking.

Whichever you decide to do, please update the docs accordingly. I had to fix my connection checks by trial and error as the docs are still somewhat lacking.

Can you point to the parts which are lacking, and describe what you'd like to see added?

Most tutorials, as you mentioned. The only way to figure it out was to go to class docs, and it took me embarrassingly long to figure out I needed to look at Callable docs for the last piece of the puzzle.

I personally am in favour of keeping the string based methods exposed, since I already know that sooner or later I will end up in a situation where I have to construct/modufy the string. But maybe we could hide them somehow?
Don't really use gdscript much so I am not certain if it already supports static methods, but what about the following?
´´´
Signal.emit_signal("a")
Signal.connect("a", self, "react_to_a")
´´´
In addition some warnings could be added to the documentation, informing people about the new way to do this.

I already know that sooner or later I will end up in a situation where I have to construct/modufy the string.

Does this assumption have any base? Did you ever need to construct a signal/method name? Concrete examples would be useful.
Also this would still be possible, but in a different way.

Also this would still be possible, but in a different way.

Please document it somewhere.
My use case: I have two or three factions in my game. I could do
func _on_whatever() and pass it the group (or do a chain of ifs within the function checking for it)
OR (with constructing a method name)
func _on_whatever_ally() and _on_whatever_enemy()
No risk of confusion/forgetting which one is which and/or forgetting to account for a new faction

Also this would still be possible, but in a different way.

Please document it somewhere.

x.signalname.connect(Callable(self, "some_func_string"))

unfortunately since methods of built-in types cannot be passed as callables (https://github.com/godotengine/godot/issues/58912), I am still stuck using emit_signal in places like tween.tween_callback(emit_signal.bind("my_signal")).

We've discussed the relevant PR for this issue, and decided it's better to still allow the old methods to exist.

Why? @YuriSizov

See Juan's comment in the linked PR https://github.com/godotengine/godot/pull/57431#issuecomment-1240938231

This does not explain why both methods should be exposed to GDScript.

It does. Methods need to be exposed for language bindings to have access to them. Once exposed, it's available for all languages. We can't unexpose them for GDScript only.


Update rod-trent.yml

Docs Build status updates of commit e9ea5d6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/rod-trent.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[Event Request] table 5964 "Service Contract Line" - "Service Item No."

We added a field to this type to make it possible to have have a service contract line with something other than a service item. But  the Code is not that extendable at the moment. The biggest issue is the Validate of "Service Item No." because we need different get on different tables.
We added the following fields in a TableExtension
fields
    {
        modify("Service Item No.")
        {
            TableRelation = if ("Line Type" = const(Equipment)) Equipment where("Lease Equipment" = const(true))
            else
            if ("Line Type" = const(Object)) "Building Object";

        }

        field(11002000; "Line Type"; Option)
        {
            Caption = 'Line Type';
            Description = 'BV A806';
            OptionCaption = 'Service Item,Equipment,Object';
            OptionMembers = "Service Item",Equipment,"Object";

            trigger OnValidate()
            begin
                //ServLineMgt.LineTypeOnValidate(xRec, Rec); //BV A893
            end;
        }
    }

As you can imagine we need to bypass certain Standard BC code. So I would like to add certain events
field(5; "Service Item No."; Code[20])
{
    Caption = 'Service Item No.';
    TableRelation = "Service Item";

    trigger OnValidate()
    begin
       .... Line 56
      if (ServContractHeader.Status = ServContractHeader.Status::Signed) and
         (not "New Line")
      then
        Error(Text013, FieldCaption("Service Item No."));

     OnBeforeGetServiceItem(Rec,IsHandled); //NEW

     if ("Service Item No." <> '') and not IsHandled then begin //MODIFY

//NEW
local [IntegrationEvent] OnBeforeGetServiceItem(VAR ServiceContractLine : Record "Service Contract Line";VAR IsHandled : Boolean)
begin
end;
//NEW

Next there is a part where the service contract line is rebuild. In this function we want to add our own field, but there is no event.
This could be solved in to put this code in a different function and give an option to create our own transfer, or add an event to add field to the Rec. In BC18 there is an Event OnValidateServiceItemNoOnBeforeModify, but this does not the ServContractLine. So maybe we can add these to the Event.
... line 124
ServContractLine := Rec;
INIT;
"Line Type" := ServContractLine."Line Type"; //This is the Line we need to add.

The following issue is that the ServItem is copied to Rec. It would be nice if we are able to skip this. But I think this code does not cause an error it everything in blanc. It is just not necessary to be executed.
... line 138
if "Service Item No." = '' then
    exit;

IsHandled := FALSE; //NEW
OnBeforeTransferServiceItem(Rec,IsHandled); //NEW

if not IsHandled then begin //NEW
  Description := ServItem.Description;
  "Serial No." := ServItem."Serial No.";

//NEW
local [IntegrationEvent] OnBeforeTransferServiceItem(VAR ServiceContracLine : Record "Service Contract Line";VAR IsHandled : Boolean)
begin
end;

//This is the Validate we want to do in OnBeforeTransferServiceItem and set IsHandled to true
CASE Rec."Line Type" OF
  Rec."Line Type"::Equipment:
    BEGIN
     //Code
    END;
  Rec."Line Type"::Object:
    BEGIN
       //Code
    END;
END;

The first part of request will be realized like this:
         if (ServContractHeader.Status = ServContractHeader.Status::Signed) and
            (not "New Line")
         then
             Error(Text013, FieldCaption("Service Item No."));

         ServiceItemNoIsNotEmpty := "Service Item No." <> '';
         OnValidateServiceItemNoOnAfterCalcServiceItemNoIsNotEmpty(Rec, ServiceItemNoIsNotEmpty);
         if ServiceItemNoIsNotEmpty then begin
             GetServItem;
             TestField("Customer No.");

As for OnValidateServiceItemNoOnBeforeModify, it defined with IncludeSender = true:

    [IntegrationEvent(true, false)]
    local procedure OnValidateServiceItemNoOnBeforeModify(ServItem: Record "Service Item"; ServContractHeader: Record "Service Contract Header")
    begin
    end;

So you can use parameter Sender as a Service Contract Line Rec.
Is it ok for you to have only 1 new event OnValidateServiceItemNoOnAfterCalcServiceItemNoIsNotEmpty from p.1 and using Sender from p. 2 for the second part of your request?

Hi Oleg,
I tested your solution and it works. I missed the Sender parameter in the 2nd part so that works as well. I'm glad with this. Because We overrule the Get on the Service Item, so the code below is performed with a blanc service item.
Line 137..
               if "Service Item No." = '' then
                    exit;

               Description := ServItem.Description;
                "Serial No." := ServItem."Serial No.";
                "Service Item Group Code" := ServItem."Service Item Group Code";
                "Item No." := ServItem."Item No.";
                "Variant Code" := ServItem."Variant Code";
                "Unit of Measure Code" := ServItem."Unit of Measure Code";
                if (ServContractHeader."Response Time (Hours)" < ServItem."Response Time (Hours)") and
                   (ServContractHeader."Response Time (Hours)" <> 0)
                then
                    "Response Time (Hours)" := ServContractHeader."Response Time (Hours)"
                else
                    if ServItem."Response Time (Hours)" <> 0 then
                        "Response Time (Hours)" := ServItem."Response Time (Hours)"
                    else
                        "Response Time (Hours)" := ServContractHeader."Response Time (Hours)";
                ServMgtSetup.Get();
                Validate("Line Cost", ServItem."Default Contract Cost");
                Validate("Line Value", ServItem."Default Contract Value");
                Validate("Line Discount %", ServItem."Default Contract Discount %");

maybe change this:
if "Service Item No." = '' then
   exit;

Description := ServItem.Description;
...

to this:
if "Service Item No." = '' then begin
  Description := ServItem.Description;
  Serial No." := ServItem."Serial No.";
...

This way no unnecessary code is executed.

What about:

Add parameter ServContractLine to event OnValidateServiceItemNoOnBeforeModify

or add event
         ServContractLine := Rec;
         Init;
         OnValidateServiceItemNoOnAfterInit(Rec, ServContractLine);
         "Starting Date" := ServContractLine."Starting Date";
         "Contract Expiration Date" := ServContractLine."Contract Expiration Date";

Thanks for the quick update. I checked the the options and I my preference goes to option 2.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


bug: Only the first of multiple wildcard domain certificates works

Current Behavior
I use cert-manager to create a multi wildcard domain cert like this:
apiVersion: cert-manager.io/v1
kind: Certificate
metadata:
  name: apisix-default-cert
  namespace: <namespace>
spec:
  dnsNames:
    - '*.staging.domain.com'
    - '*.prod.domain.com'
  issuerRef:
    kind: ClusterIssuer
    name: cloudflare
  secretName: apisix-default-cert

Then I would get a secret apisix-default-cert with two wildcard domains. And use cloud native way to inject secret to apisix by apisix ingress controllor TLS crd.
apiVersion: apisix.apache.org/v2
kind: ApisixTls
metadata:
  name: apisix-tls
  namespace: <namespace>
spec:
  hosts:
    - "*.staging.domain.com"
    - "*.prod.domain.com"
  secret:
    name: apisix-default-cert
    namespace: <some ns>

After this, I can see tls have been applied in apisix by call admin api
there is some reponse.
{"snis": ["*.staging.domain.com","*.prod.domain.com"]}

Then I try some host by curl but got error.
staging
this return error msg show ssl worked.
$ curl https://test.staging.domain.com
{"error_msg":"404 Route Not Found"}

prod
this return ssl error.
$ curl https://test.prod.domain.com
curl: (35) LibreSSL/3.3.6: error:1404B438:SSL routines:ST_CONNECT:tlsv1 alert internal error

Expected Behavior
I thought both wildcard domain should take effect, instead of the first.
If multiple wildcard domain are not supported, neither apisix nor apisix-ingress-controller reported any errors. It might be denied at some stage of apisix or apisix-ingress-controller ?
Error Logs
No response
Steps to Reproduce

Create a multi wildcard domain cert.
Apply cert to apisix
Use other than the first domain name to access apisix

Environment

APISIX version (run apisix version): 3.6.0
APISIX Docker version: 3.6.0-debian
OpenResty / Nginx version (run openresty -V or nginx -V): openresty/<IP_ADDRESS>
APISIX Ingress Controller version: 1.7.0

@incubator4 Can you also paste your ApisixRoute configuration?

@incubator4 The error is that route not found because you haven't created a route for that host maybe. You must have a route configuration configured for prod.* already

@incubator4 The error is that route not found because you haven't created a route for that host maybe. You must have a route configuration configured for prod.* already

yeah but I'm not talking about route error but tls error.

@incubator4 Okay I get it. Let me take a look

@incubator4 Can you share some logs from APISIX when you send these requests?

@incubator4 If you only use prod and not staging in hosts, do you get the same error?

@incubator4 If you only use prod and not staging in hosts, do you get the same error?

no， standalone wildcard domain would not get error


Adding a memory resource

Force all the calls to malloc/free to go through a memory resource object.
This can be used to use cudaMalloc instead of cudaMallocAsync on platforms where the later is not supported
Or use a memory pool...

just looking through this. like the concept of a memory resource, just having problems merging because I changed a lot of the template interfaces to allow other data types and dimensoins. will merge by hand.

merged into master.
note however there's been a lot of other changes in master, hope that doesn't break anything


one pixel difference between "Bar Fill" end and the handle start

http://jsfiddle.net/mq93ag31/1/

Well spotted! I'll look into that.

I think this one is closed :+1:

Indeed I fixed it myself by mistake in this commit: 819127a
I had this code in my local repo when I tried your fix and forgot to remove it! ^^

:+1:


Allow to  "Run Update Copyright..." on saved files

When I save files, I would like copyright statement be updated according to rules I specify in "Update Copyright" preference. This item is available under Code->Update Copyright...

Hey @tandr. I really like the idea of having a "update copyright" save action.
I checked how we could implement this:

Use com.maddyhome.idea.copyright.actions.UpdateCopyrightAction
Use com.maddyhome.idea.copyright.actions.UpdateCopyrightProcessor

I think the second option is better, since it would work like other processor we're using, like com.dubreuia.processors.OptimizeImportsProcessor. We could instanciate it there com.dubreuia.processors.ProcessorFactory, making it available for all Jetbrain's product.
I'm not sure I'll have time to develop that, but you can submit a PR anytime, anything you need to know for contributing is here https://github.com/dubreuia/intellij-plugin-save-actions#contributing

IntelliJ has an embedded action https://www.jetbrains.com/help/idea/generating-and-updating-copyright-notice.html (the same as organize imports etc). I don't remember installing a special plugin for it.
Would it be possible to run actions somehow by just selecting something from dynamic checklist box of ALL available refactors/code issue finders instead of  having a limited predefined list of actions?

Executing any IDE action was submitted as an improvement a while ago, see #118.
IMO this issue is doable in a short amount of time, but implementing #118 is really a lot of work.

I will be happy with having at least copyright updated if it is possible to have it sooner :)

@dubreuia I couldn't find the library com.maddyhome.idea.* in the classpath
Where did you get those classes?

There are many more missing classes including one called com.intellij.copyright.CopyrightManager which is, I guess, part of the updated SDK (not found in 2016.3).
Therefore, first we need to update the SDK then rely on the code you mentioned above

Too bad. I suppose copying the whole package com.maddyhome.idea.copyright is a bit too dirty: https://github.com/JetBrains/intellij-community/tree/0e91550e746c94ab6afe5c0d36450b93b587dd2d/plugins/copyright/src/com/maddyhome/idea/copyright
Doing a multi release jar is a lot of work for just this processor. I'm not against it since I feel we'll have to do it at some point, but this particular processor just isn't worth it IMO.

@dubreuia It cannot be used well for two reasons:

There isn't a quick list action to update copyright to current file, only to the project so the dialog pops up asking which files to update copyright for (See screenshot below)
Quick list items cannot be checked into repository, they are not project specific settings


better check for node

This check will work regardless of a process shim.
If in browser, Object.prototype.toString.call(global.process) -> “[object Undefined]”
If in node, returns “[object process]”

Will resolve #32


Code generator does not properly handle keyword-named properties

Certain JSON schemas define properties that have names like "class" (for example, the EXT_feature_metadata that is currently used as the input for the generator).
The code generator apparently tries to solve the resulting problems by renaming these reserved words and appending the string Property to the name. However, this is not properly taken into account in the reader classes. For example, the reader now seems to expect the string classProperty in the JSON input, even though it will only find class there.
I extended the respective test case in https://github.com/CesiumGS/cesium-native/tree/keyword-named-properties-test-2 , which shows that even though it should read the value "tree" into the featureTables["tree"].classProperty , this property has no value. (When this issue is addressed, the test cases should be extended further, but this change shows the basic issue)

@kring iirc you fixed this at some point right? Especially considering we now have EXT_feature_metadata auto-generated and working correctly. Closing this, reopen if I'm mistaken.

Both the readers and writers look to be correctly generated now. For example, in the writer for MetadataEntity:
jsonWriter.Key("class");
writeJson(obj.classProperty, jsonWriter, context);

And in the reader:
if ("class"s == str)
  return property("class", this->_classProperty, o.classProperty);

So I'm closing this. We can deal with the more evil scenario if and when we have a spec that does something like that.


Provide good example and rationale behind the rule

Version 2.4.0
Robocop print warnings and errors given rules. When we read those warnings and errors, sometimes it is hard to understand why there is this rule or how to fix it in a proper way.
For example:
https://robocop.readthedocs.io/en/stable/rules.html#comments
We would like to add a "TODO: do not forget to do this and this." in my code. Robocop print the warning "todo-in-comment". I check the documentation and I do not understand why.
For example, I moved the line from the test cases section to the header and still have the warning. Yes, I'm trying (silly ?) things.
I have no doubt this warning is useful. Checkers/linters are wonderful tools to learn best practices. For example, I really love shellcheck for this help to the learning process:
https://www.shellcheck.net/wiki/SC1118
Is it possible to add to the robot documentation the rationale and a good example to follow?

It's a good idea - it's one of the reasons why I've recently extended our docs with the ability to add such extended docs (previously all our external docs were automatically generated without examples). One of the examples where we put "why this rule exist" is https://robocop.readthedocs.io/en/stable/rules.html#tag-with-or-and .
However there are currently workshops ongoing that are trying to create Robot Framework official style guide - it's very likely that some of our rules will be updated/reworded to match official style guide. When it will happen I will try to also reuse examples from style guide in our documentation (or link directly from the Robocop rule to style guide rule/recommendation).
But I will take a look into our rules and try to update missing docs in the meantime too. There are still several rules that are missing examples / reasons for existence.

And to comment on the rule mentioned in your post - TODO in the comments is fine and Robocop rule for discovering it is just a handy tool to see any TODO/FIXMEs in your code. You can't "fix" it by moving the comment, the only way of "fixing" it is resolving TODO task :). But we could use this extra explanation in our docs - that it's just visual guidence for users to see all of their TODOs, not necessarily rule prohibiting using TODO in the comment.

I think we should revisit the documentation for each rule and provide more meaningful, real-world examples, to make it easier for the users to understand how to use the rules. Currently, in many places, we have some generic sample code which has nothing to do with the code that is produced in a real environment.
It'll take time, because it's not that easy each time to come up with a good example, but I think it's worth doing. Again, we can split the work here, @bhirsz. Also, it's not very urgent, so we can do it gradually.

2 out of 9 rule categories were updated in #917:

naming
documentation


doc: Update Docs For Storage Tiering Testing

manual documentation update for how to test storage tiering feature

Fixes: #543

Rendered Locally:


Add method to merge PDF from InputStream source

Introduced a new addSource method that accepts an InputStream as a parameter, enabling merging of PDFs directly from streams keeping the developer from having to understand the RandomAccessRead interfaces and use a common way of accessing file data.
Ideally this is merged with the 3.0 branch as well.

I'd prefer to improve the javadoc. Also for Loader.loadPDF() which has the same problem.

I fully support @THausherr. We are using the RandomAccessRead interfaces to give the user maximum flexibility w.r.t. the used access method and possible optimization


Fixed Responsiveness of View Code Pane

#67

Thanks for this PR.
This issue has been fixed in https://github.com/foss42/api-dash/commit/2276fcfe1a66474f40f46b3893465b3979bc5a8c


WP Plugin location

Should wp-tipsy-payment-info be in a separate repository?

Closing out due to inactivity.


”Ilieni”, ”Dobolii de Jos”, ”Sâncraiu” are listed in the drop-down menu

Description:
When user writes ”Ilieni”, ”Dobolii de Jos”, ”Sâncraiu” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Ilieni"

Expected results
The location appears in the drop-down menu as "Sat Ilieni, COVASNA (Ilieni)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Sâncraiu"

Expected results
The location appears in the drop-down menu as "Sat Sâncraiu, COVASNA (Ilieni)".


Add cast method to Quaternion

I'm not sure if the doc comment is correct (I just copied it from the vector method).

Cheers!


Clone KVM Template

Hi There!
I'm trying to clone a KVM template (template id: 100) This is the code i made but the only output im getting is "boolean false"

What do you get when you try to run the same request using pvesh?  The code itself looks correct, but the API is outside this project's scope.

I will try that out in a bit, But the wierd part is, It doesnt give any error output.

Well, to be fair, the client lib doesn't actually do much more than handle authentication on each request.  Otherwise, it's basically just a cURL wrapper.  So it won't actually provide error details on most things.

I understand, But its retarded that promox doesnt output any errors.

The PVE2 API probably does, actually.  The client lib simply doesn't report them.

Hy guys ,did you finder any infos about this ?  Thank you

The last commit as added the function : clone_vm ($node,$vmid)
Hope this can help to solve your problem.

Thanks @lucapiccio - closing this as completed! :-)


Update setup

Description
Add project links to setup.py and update DOWNLOAD_URL to that of PyPi.
Type of change
Please check options that are relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Not tested, waiting for next release workflow to see if links are well displayed on TestPyPi.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Hello @gmartinonQM! Thanks for opening this PR.

In the file doc/conf.py:

Line 18:1: F401 'sphinx_gallery' imported but unused
Line 52:1: E402 module level import not at top of file
Line 53:1: E402 module level import not at top of file

Do see the Hitchhiker's guide to code style


GRPC Configuration Documentation, Defaults, and Examples

Is your feature request related to a problem? Please describe.
We are attempting to optimize Triton as a GRPC server.  We noticed a couple of parameters that we felt should be tuned.  And, without trial and error, we don't know the best approach for optimizing these values.
The values we think might need to be adjusted are:
--grpc-infer-thread-count	Number of threads handling GRPC inference requests.
--grpc-infer-allocation-pool-size	The maximum number of inference request/response objects that remain allocated for reuse. As long as the number of in-flight requests doesn't exceed this value there will be no allocation/deallocation of request/response objects.
--pinned-memory-pool-byte-size	The total byte size that can be allocated as pinned system memory. If GPU support is enabled, the server will allocate pinned system memory to accelerate data transfer between host and devices until it exceeds the specified byte size. This option will not affect the allocation conducted by the backend frameworks. Default is 256 MB.

--grpc-infer-thread-count - is this the concurrent requests that can be processed?  I thought I saw ticket somewhere that mentioned the GRPC interface was non-blocking.  If so, should this be set to the number of CPU cores?
Should --grpc-infer-allocation-pool-size and --pinned-memory-pool-byte-size be set to the same size?  Or, should --pinned-memory-pool-byte-size be larger to enable temp space?
Assuming we have:

1 MB request/response payload
8 CPUs
Maximum of 32 requests per second
50 GB of RAM
4 model instances

What is the most efficient Triton configuration for GRPC for this type of model/inference payloads?  Are there other parameters that need to be known?
Describe the solution you'd like
It would be great to have a bit more details into these configuration parameters, default values, and a couple of examples on how best to configure them.
Describe alternatives you've considered
We have tried the default configuration and --grpc-infer-thread-count=1024 --grpc-infer-allocation-pool-size=1024 --pinned-memory-pool-byte-size=1073741824.  Both seem to work.
Additional context
We are running in the Triton Docker container.

Don't use --grpc-infer-thread-count. It has no impact and is being removed in Triton V2. As you note the grpc endpoint is completely async and can handle any number of concurrent requests.
--grpc-infer-allocation-pool-size can be important if you have large request size (which you do). GRPC uses protobuf which can be inefficient in their create/delete. So this parameter lets you have a pool of request protobuf objects that are reused instead of created/deleted. You likely can see a perf difference if you set it to 1 compared with setting it higher (default is 8). The downside of using a large value is that memory usage will increase since the server will keep the pool of that many request protobufs around even if not in use.
--pinned-memory-pool-byte-size is not related to the grpc endpoint. It is the maximum amount of pinned memory (see CUDA docs) that Triton will use. Pinned memory will accelerate DtoH and HtoD copies. Since your model has large input it can be helpful. You can experiment with it set to 0 and a larger number, like 64MB. I wouldn't set it to 1GB since it does have an overall system impact because it locks virtual pages into memory.

Thank you!
I have lowered my numbers and I am seeing about the same performance with perf_client.  I'll keep them lowered and monitor in production.

Great!. Closing, reopen if more feedback.


Remove dependency on @solana/spl-token v0.1.8

I was having difficulties using the gem-farm app as the wallet adapter kept complaining about getMinBalanceRentForExemptMint not existing when trying to mint a test token, using @gemworks/gem-farm-ts v0.26.4. Digging a bit I found out that the wallet adapters have a dependency on @solana/spl-token v0.1.8, which is a version from 2 years ago, so I've done a bit of refactoring in order to remove that dependency.
The change consists mostly about deleting old imports and re-importing the correct functions, with correct parameters and correct types from the newest version of the spl-token library, and fixing the tests that complained.
The change ended up being bigger than I expected, so I may have fucked up something :D the tests all pass correctly and as far as I tried, the app wallet adapter works correctly now when pointed to the local version of the library.
Let me know if the change is ok or anything needs to be modified :)

If you end up merging this lmk if you would mind a few more PRs :) I've been cleaning up a bit since I needed to use it for smth, changes for now are:

cleaned up typescript errors (mostly missing casts)
split claim into claim and buildClaim like the rest of calls
(soon) removed dependency to @metaplex/js as it's deprecated

Does this fix the current ongoing issue @0xnetfox ?

@peterschwarzdev sorry, just seen the comment
This PR isn't related to that issue. My uneducated guess is that the linked issue is related to the changes that ilmoi did to accomodate programmable NFTs at the end of January. Those changes shouldn't break existing farms as far as I know so I'd bet it's an issue on the SDK.
If updating the @gemworks/gem-farm-ts library to the latest version (0.26.4) doesn't fix the issue, someone will need to dig exactly why it is happening and if it is fixable :)


Hard drive SMART data?

Issue Type

[ ] Bug
[X] Feature Request
[ ] Question

Expected behavior
It would be awesome if this project could collect the SMART data from hard drives. I'm currently using this project in a custom remote monitoring system for endpoint PCs and at the moment the only hard drive related warning I have set is for running out of space. I would really love it if I could also set it up for SMART errors/messages.
Specifications

operating system: Linux, Mac, Windows
hardware: Any

Hi @tankerkiller125 , I also would like to see that in this package. Unfortunately I did not found a reliable way to get smart status without certain dependencies. But I will try to find a smart way to retrieve S.M.A.R.T. status when needed libraries are installed.

Proper S.M.A.R.T. detection always needs the smartmontools, so you
They are available on all systems.
In the end its the same as with the temperature settings.
For most people it will be okay if they get what they get - and are able to get more information like temperature or SMART data when installing additional software.
If you like to tinker around with some C library and the node addon interface or what it is called, you could take a look at this here: http://git.0pointer.net/libatasmart.git/tree/
But otherwise I would just do some "retrieve values if smartmontools are installed, otherwise ignore". For retrieving the values, you could take a look at pySMART, how they did it: https://pypi.org/project/pySMART/#description
But yes, overall +1 from me, would be very cool

@mpathy - you are absolutely right! Will have a look on it.

@mpathy the only problem I find with this solution is windows and Mac support, both are very important where I work and it seems that smartmontools isn't a thing for windows.

@tankerkiller125 It is available for every system: https://www.smartmontools.org/wiki/Download#InstalltheWindowspackage

@mpathy In that case ignore what I said, I'm all for this solution then.

Just published a new version which supports detecting SMART status (diskLayout() --> [...].smartStatus). Be sure to install smartmontools if you are running on linux. See version 3.45.0