Support Window AI

This PR adds support for using the Window AI extension with GPTeam: https://windowai.io/
The ChatWindowAI python class talks to Window via websockets.
Testing:
I tested with GPT-4 and Claude using the OpenRouter provider in Window, with the command poetry run world --window. I noticed that after a while, I often encountered the following errors:
Model: openai/gpt-4
Console error: Uncaught (in promise) MODEL_REJECTED_REQUEST: undefined: AxiosError: timeout of 42000ms exceeded

Model: anthropic/claude-v1
Console error: MODEL_REJECTED_REQUEST: 405: AxiosError: Request failed with status code 405

I'm not sure if this has to do with my implementation or Window/OpenRouter.

"status code 405" may be getting returned by the APIs upstream - what's the request you're using?
"timeout of 42000ms exceeded" -> this issue might be due to extremely long non-streamed outputs that take longer than 42s to generate. Are you streaming?

"status code 405" - happened when calling window.ai.generateText(...) with the following message:
[
    {
        "role": "user",
        "content": "You are a role-playing AI, playing the role of Ricardo in front of a live audience. Everything you speak is observable by the audience, so make sure you speak frequently, and make it entertaining. You cannot interact with the audience directly.\n\nGiven the following context and tools, proceed as if you were Ricardo. Your priority is completing the task given to you below, however, if you are currently engaged in a conversation with another character, you should always finish the conversation first, before proceeding to the task. Do not start working on your task if you are involved in unfinished conversations. Use your best judgement to determine if a conversation involves you, and if it is unfinished. You do not need to respond to every message you get.\n\nHere is your character's bio:\nRicardo is a compassionate improviser known for his endearing characters and heartfelt scenes. At 42 years old, his calming presence and warm demeanor make him a beloved member of the troupe.  Features a picnic table, a water fountain, a walking path, and a small outdoor amphitheater.\nNearby people: \n- Marty: Marty, the ringmaster of our improv troupe, keeps the team together with his infectious energy and endless humor. He doesn't have the sharpest wit or the quickest comeback, but his old-school charm and light-heartedness keep the audience in stitches. He is always ready to make a fool of himself for a laugh.\n- Rebecca: Rebecca, the sharp-witted member of our troupe, is known for her quick comebacks and clever banter. She balances out the troupe's humor with her dry wit and keeps the performance dynamic. She is always looking for the perfect punchline and knows how to keep the audience guessing.\n\n\nHere are some relevant memories:\n```\nRebecca said to everyone in the Park: 'Hey guys!  Any ideas for what we should do?' [2023-05-28 17:15:06.036920+00:00]\nRicardo said to everyone in the Park: 'I'm excited to be here and ready to perform some improv! What ideas do you have in mind, Rebecca and Marty?' [2023-05-28 17:15:20.208730+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!' [2023-05-28 17:26:35.554079+00:00]\nRicardo said to everyone in the Park: 'Welcome to our time travel adventure! I am Ricardo, and I will be playing the role of a scientist who has just invented a time machine. Who wants to join me on this journey through time?' [2023-06-06 16:11:44.648748+00:00]\nMarty said to everyone in the Park: 'Alright everyone, Ricardo suggested a time travel adventure for our performance today! Let's get started and create some amazing scenes!' [2023-06-06 16:12:01.593042+00:00]\nMarty said to everyone in the Park: 'Alright everyone, Ricardo suggested a time travel adventure for our performance today! Let's get started and create some amazing scenes!' [2023-06-06 16:12:01.593042+00:00]\nMarty said to everyone in the Park: 'Alright everyone, Ricardo suggested a time travel adventure for our performance today! Let's get started and create some amazing scenes!' [2023-06-06 16:12:01.593042+00:00]\nMarty said to everyone in the Park: 'Alright everyone, Ricardo suggested a time travel adventure for our performance today! Let's get started and create some amazing scenes!' [2023-06-06 16:12:01.593042+00:00]\n```\n\nHere is some relevant conversation history:\n```\n\n```\n\nHere are the tools you have access to:\nspeak: say something in the Park. The following people are also in the Park and are the only people who will hear what you say: [Marty, Rebecca] You can say something to everyone in the Park, or address a specific person at your location. Input should be a json string with two keys: \"recipient\" and \"message\". The value of \"recipient\" should be a string of the recipients name or \"everyone\" if speaking to everyone, and the value of \"message\" should be a string. If you are waiting for a response, just keep using the 'wait' tool. Example input: {\"recipient\": \"Jonathan\", \"message\": \"Hello Jonathan! 😄\"}\nwait: Useful for when you are waiting for something to happen. Input a very detailed description of what exactly you are waiting for. Start your input with 'I am waiting for...' (e.g. I am waiting for any type of meeting to start in the conference room).\nhuman: You can ask a human for guidance when you think you got stuck or you are not sure what to do next. The input should be a question for the human.\ncompany-directory: A directory of all the people you can speak with, detailing their names and bios. Useful for when you need help from another person. Takes an empty string as input.\nsave-document: Write text to an existing document or create a new one. Useful for when you need to save a document for later use. Input should be a json string with two keys: \"title\" and \"document\". The value of \"title\" should be a string, and the value of \"document\" should be a string.\nread-document: Read text from an existing document. Useful for when you need to read a document that you have saved.\nInput should be a json string with one key: \"title\". The value of \"title\" should be a string.\nsearch-documents: Search previously saved documents. Useful for when you need to read a document who's exact name you forgot.\nInput should be a json string with one key: \"query\". The value of \"query\" should be a string.\n\nYour response should use the following format:\n\nTask: the task you must complete\nThought: your thoughts about what you should do\nAction: the action to take, which must be one of these words: [speak, wait, human, company-directory, save-document, read-document, search-documents]\nAction Input: the input to the action\nObservation: the result of the action\n... (this Thought/Action/Action Input/Observation can repeated N times)\nThought: 'I have completed the task'\nFinal Response: the final response to the task\n\nIf you are not ready with a final response, then you must take an action.\n\nIf you determine that you are incapable of completing the task with the tools you have been given, return 'Final Response: Need Help', however, if you are in dialog with another character, responses such as 'I don't know' are valid responses. When in dialog, you should never, under any circumstance, break character, or admit that you are an AI.\nIf the task is complete and no specific response is needed, return 'Final Response: Done'\nBegin!\n\nTask: \nDo this: Participate in an improv performance at the park.\nAt this location: Park\nStop when this happens: The performance has ended.\nIf do not finish within 1.0 hours, stop.\n\nThought: This is going to be fun! I love performing improv with the troupe. Let's see what ideas Rebecca and Marty have for us today.\nAction: speak\nAction Input: {\"recipient\": \"everyone\", \"message\": \"I'm excited to be here and ready to perform some improv! What ideas do you have in mind, Rebecca and Marty?\"}\nObservation: Ricardo said to everyone in the Park: 'I'm excited to be here and ready to perform some improv! What ideas do you have in mind, Rebecca and Marty?'\nThought: I should wait for Rebecca and Marty to respond before suggesting any ideas of my own.\nAction: wait\nAction Input: I am waiting for Rebecca and Marty to respond with their ideas for the improv performance.\nObservation: The event I was waiting for has not happened yet. Waiting...\nThought: It seems like Rebecca is suggesting some ideas. I should listen and see if any of them spark my creativity.\nAction: speak\nAction Input: {\"recipient\": \"everyone\", \"message\": \"A haunted house or a spy thriller sound like great ideas, Rebecca! Let's hear some more suggestions from the rest of the troupe.\"}\nObservation: Ricardo said to everyone in the Park: 'A haunted house or a spy thriller sound like great ideas, Rebecca! Let's hear some more suggestions from the rest of the troupe.'\nThought: I wonder if anyone else has any ideas for the performance. Maybe I can suggest something too.\nAction: speak\nAction Input: {\"recipient\": \"everyone\", \"message\": \"Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!\"}\nObservation: Ricardo said to everyone in the Park: 'Does anyone else have any ideas for the performance? I was thinking maybe we could do a time travel adventure or a musical!'\nThought: It seems like everyone is on board with the time travel adventure idea. I should start thinking about what character I want to play and what kind of scene I want to create.\nAction: speak\nAction Input: {\"recipient\": \"everyone\", \"message\": \"Alright, it looks like we're doing a time travel adventure! Let's get started and create some amazing scenes!\"}\nObservation: Ricardo said to everyone in the Park: 'Alright, it looks like we're doing a time travel adventure! Let's get started and create some amazing scenes!'\nThought: I should start the performance by setting the scene and introducing my character.\nAction: speak\nAction Input: {\"recipient\": \"everyone\", \"message\": \"Welcome to our time travel adventure! I am Ricardo, and I will be playing the role of a scientist who has just invented a time machine. Who wants to join me on this journey through time?\"}\nObservation: Ricardo said to everyone in the Park: 'Welcome to our time travel adventure! I am Ricardo, and I will be playing the role of a scientist who has just invented a time machine. Who wants to join me on this journey through time?'\nThought: I should wait for someone to join me in the scene and see how the story unfolds.\nAction: wait\nAction Input: I am waiting for someone to join me in the time travel adventure scene.\nObservation: Marty said to everyone in the Park: 'I'll join you, Ricardo! I'll be playing the role of a curious historian, eager to explore the past and uncover hidden secrets!'\nThought: Great, Marty is joining the scene. Let's continue the performance and see where our time travel adventure takes us.\nAction: speak\nAction Input: {\"recipient\": \"everyone\", \"message\": \"Welcome aboard, Marty the historian! Let's set the time machine to our first destination. Where in time would you like to go first?\"}\nObservation: Ricardo said to everyone in the Park: 'Welcome aboard, Marty the historian! Let's set the time machine to our first destination. Where in time would you like to go first?'\nObservation: The event I was waiting for occured at 2023-06-06 16:12:01.593042+00:00. No need to wait anymore.\nThought: "
    }
]

"timeout of 42000ms exceeded" - I'm not streaming, I think that's the problem. I'll make a code change to use streaming.

I'm not able to repro the 405 with that prompt and any anthropic models - lmk if there are other details!

I enabled streaming in the window.ai.generateText(...) call, and now both the errors ("status code 405" and "timeout of 42000ms exceeded") are no longer occurring! :)


Hack Session 628 ✼ Remotely

15.05.2024
Wrap-up of our Gōng-fu I/O Weekly · Hack Session #628 ✼ Remotely ✼ meetup.
Projects
• Astro › Content Collections — @olange and @rudifa
• Flutter — @andreaskundig and @david-hodgetts
See also

#627 Previous session
#629 Next session

Review of @rudifa's work on migrating the descriptions and comments of the issues of this repository, using the GitHub GraphQL API, to a plain JSON file; and discussion about how to pivot this data to the upcoming website of Gōng-fu DEV, built with Astro (gongfudev/website#13).


Constraints for cpu and memory

Hi!
Is there a reason why tile-generator do not create constraints for cpu/mem?
https://github.com/cf-platform-eng/tile-generator/blob/bccaf3128310ececafbe356d55f4b8b20ed27f84/tile_generator/templates/tile/metadata.yml#L451
It seems that it is easy to fix so I wondering maybe this was not done by some reason.
Thanks.

Hi @antonsoroko,
You're right - we should probably be consistent and not allow users to dial those settings down below what the tile builder specified, just like we do for persistent disk.
Thanks for reporting it!

Fix is working its way through CI before release. Should be in tile-generator 9.0.3.


Update base.py

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[xx] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue

How Has This Been Tested (if it applies)

Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[x] All tests passed, and additional code has been covered with new tests.

I don't think that would solve it, but that seems ok at least

This definitely solves a bug.


Require is not defined

Hello,
I am following the example here: https://modern-web.dev/docs/dev-server/plugins/storybook/
I receive the following error from my firefox browser:
Uncaught ReferenceError: require is not defined
    <anonymous> http://<IP_ADDRESS>:8000/node_modules/@storybook/addon-knobs/register.js:1

My config file:
import { storybookPlugin } from '@web/dev-server-storybook';
export default {
  plugins: [storybookPlugin({ type: 'web-components' })],
};

I run the wds with
./node_modules/.bin/web-dev-server --config path/to/config --node-resolve
Perhaps i missed something?

Do you have a .storybook/main.js file? This is loaded as well, and might contain some configuration for regular storybook.

Yes, I stripped everything to keep it simple:
module.exports = {
  stories: ['../src/**/*.stories.ts'],
};

seems related to the knobs plugin.  I don't mind disabling all the plugins to just have some bare storybook running.  Is it possible to disable plugins?

I notice the "knobs" addon isn't part of the prebuilt, yet some how my setup is adding it.  Hmm. The index.html that is loaded to my browser is
<!DOCTYPE html>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <title>Storybook</title>

So the knobs addon is coming from my html. Not sure why though

I used the --app-index arg to pass the index page explicitly. So I passed the same html but deleted the import knob addon from the script tag. But then I receive this error
Loading module from<EMAIL_ADDRESS>was blocked because of a disallowed MIME type (“text/plain”).

That's strange, this import must come from somewhere. You're certain there is no main.js file somewhere interfering? I have added an example project here: https://github.com/modernweb-dev/example-projects/tree/master/storybook

Yes I was running with the assumption of the wrong .storybook config dir. I passed the .storybook config fold explicitly in the plugin and it worked.  Whoops!  :)


Crash on iOS 12.5.4 and Ti SDK 10.0.0.GA

Hi guys, I have a problem with this module since I upgraded to Ti SDK 10. On iOS 12.5.4(tested with iP6) it crashes right after startup. I found logs via xcode:
erminating app due to uncaught exception ‘UIViewControllerHierarchyInconsistency’, reason: ‘child view controller:<UINavigationController: 0x103037600> should have parent view controller:<TiViewController: 0x1020911a0> but actual parent is:<TiRootViewController: 0x10282de00>’
Building app with 9.3.2.GA works fine. Tested with newest 2.1.1 version of the module... Please can it be fixed? Thanks

https://github.com/mbender74/NappDrawer
I fixed that error!

@mbender74 , I'm hearing that this is still an issue when you put a NavigationWindow within a NappDrawer. It sounds like the NappDrawer code needs to change grab the next view controller up in the hierarchy kind of like how it's handled below. What do you think?
https://github.com/appcelerator/titanium_mobile/pull/12930/files

works great with NavigationWindow!
Sure, you need to use 10.0.1.GA or 10.1.0 (master).....
Using NavigationWindows as centerWindow no problem...

@mbender74, I submitted a PR to fix the specific case Josh was referring to https://github.com/mbender74/NappDrawer/pull/1


fmu-v4_cannode stuck in boot loop

Sometime recently the cannode build target for the Pixracer (fmu-v4) was broken.  It will boot, but upon connecting a Zubax Babel board to the CAN port, the Babel will show that the cannode gets stuck in an infinite reboot loop.  It will briefly show up as initializing, then power off and disappear from the UAVCAN GUI, show up again as initializing, etc.
This commit still works: pull/14444/d14fc944
But as of last Friday at least (and I tried yesterday as well) master does not work.
@dakejahl

Can you describe your test setup? I just tested this on a Pixracer with a GPS attached and everything works as expected.

Hmm.  Using commit 2b82b471? I'll give that another try.

Yup whatever the latest commit is

Still doesn't work for me; I just get this in the UAVCAN GUI when connected to a Babel:

It stays in the "INITIALIZATION" mode for a second or two then goes away.  And now after a few minutes, it's no longer rebooting itself at all is doesn't appear on the CAN bus at all.
Using an mRobotics Pixracer; brand-new purchase from last week.

Oh, interesting.  I removed the micro-USB cable, replug the CAN cable, and now it works 🤔

Somewhere the handling of USB power on this must have changed.  All of my prior testing has been with just leaving the USB cord plugged in.
At any rate, it's looking good now!

Do you have a FTDI to hook into the console? The USB cable being plugged in should make no difference, I've got mine plugged in with a USB as well.

I do; which connector/pins expose the console on the Pixracer?

It's part of the pixhawk debug port
https://dev.px4.io/v1.9.0/en/debug/system_console.html

Ah, I see it.  I have a DroneCode Probe; I'll give that a try later when I have time.

Well I rebased @dakejahl's airspeed branch on top of master (actually this branch here: https://github.com/volans-i/Firmware/tree/pr-uavcan-airspeed) and it all works now 🤷♂


Move instance resources upstream

All of the resources in
https://github.com/kubeflow/gcp-blueprints/tree/master/kubeflow/instance/gcp_config
Could probably move to manifests upstream. kpt setters and substitutions now give us a really good way to easily sure everything
is set correctly.
This would also be more consistent as opposed to having the resources spread out in multiple places.

/priority p2


Add sensible extension

Description

Adds ability to people to extract data from their documents using Sensible.so right from Raycast.
Requires a free Sensible account + API key to get started. (Instructions in the readme)
Screencast

Demo on Youtube
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Looks like our new Sensible SDK isn't exporting all the types correctly. I'll fix that on our side and update the SDK dependency to fix the issue tomorrow.

Alright, this should be ready for review now.


Update to Xorm v0.7.9

As title
Should fix #8337

Sorry, I didn't see #8354

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0d93430). Click here to learn what that means.
The diff coverage is 93.2%.

@@            Coverage Diff            @@
##             master    #8355   +/-   ##
=========================================
  Coverage          ?   41.75%
=========================================
  Files             ?      496
  Lines             ?    65585
  Branches          ?        0
=========================================
  Hits              ?    27386
  Misses            ?    34685
  Partials          ?     3514

Impacted Files
Coverage Δ

routers/repo/webhook.go
1.38% <0%> (ø)

routers/routes/routes.go
82.27% <100%> (ø)

models/repo.go
48.18% <100%> (ø)

routers/api/v1/repo/repo.go
68.22% <95.58%> (ø)

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 0d93430...ca0db2e. Read the comment docs.


No create new connection button

Bug Description
No create new connection button
Steps to Reproduce

On a fresh install of the UI
Spin everything up and log in
Navigate to the tab datastore connection

Expected behavior
Button should be available
Screenshots

Environment

Version:
OS:
Python Version:
Docker Version:

Additional context
Add any other context about the problem here.

Assigning this one to @TheAndrewJackson: I think I might see the root cause here from poking around elsewhere...
It looks like the flag.json file in the merged UI might have the wrong commit history. It showing that it was renamed like so:
clients/ctl/admin-ui/src/mocks/browser.ts -> clients/admin-ui/src/flags.json
(see https://github.com/ethyca/fides/commits/unified-fides-2/clients/admin-ui/src/flags.json)
Whereas I'd expect this to have been moved from:
clients/ops/admin-ui/src/flags.json -> clients/admin-ui/src/flags.json
(see https://github.com/ethyca/fides/commits/unified-fides-2/clients/ops/admin-ui/src/flags.json)

@TheAndrewJackson I think I see the issue here - it's not related to unified, it's a different problem. It looks like the "empty state" for this page isn't rendering like we expect, which prevents showing the button.

@NevilleS This may have been fixed after merging 1.8.1 in yesterday. This is what I see after running the updated admin ui.

Do you see it working with the newest version as well?

Yes, confirmed that this is working now! Thanks for checking 👍


Archive issue Xcode 12 beta

I failed to archive app with 12.0 beta 2 and I found that other libraries have similar issues.
https://github.com/Alamofire/Alamofire/issues/3240
https://github.com/SwiftyJSON/SwiftyJSON/issues/1066

PromiseKit version 6.13
It's installed using CocoaPods
The error is the following:

Command CompileSwift failed with a nonzero exit code

i also got the same issue.
Undefined symbols for architecture i386: "type metadata for Swift._StringObject.Variant"

Hello @ElenaVielva @mohamadakmalshukri
Try to install Xcode 12 from AppStore.

Hello @ElenaVielva @mohamadakmalshukri
Please close the issue if you cannot reproduce it with Xcode 12.

Hello @ElenaVielva @mohamadakmalshukri
update also Promisekit to the latest version

Archiving from Xcode 12 (no beta) now works fine for me, thanks!


graphicsdriver/opengl: Use PBO on Android and browsers

https://godoc.org/golang.org/x/mobile/gl#Context3 Unfortunately glMapBuffer and the other things are not implemented yet... :-/

https://stackoverflow.com/questions/7786187/opengl-texture-upload-unsigned-byte-vs-unsigned-int-8-8-8-8

I'd like to give up gomobile build so that any OpenGL ES functions can be called. Any opinions?

We can use glBufferSubData. This is available as of WebGL 2 (not 1).

Blocker: https://github.com/golang/go/issues/36355


Can it adapt to small and medium-sized screens

Can it adapt to small and medium-sized screens

I had to pull version 1.2.0 because it was letting people view other people's messages, so I'm reopening this.

Waiting for new version🤔

I've just released v1.2.1, which should fix this :)

According to an user at FreeFlarum, this issue does not seem to be fixed yet. This seems to happen when the message is too long, and the text overflows

I've just released v1.3.0, which should vastly improve mobile support! :)  Please give it a try and let me know if you find any issues!

seems v1.3.0 was not released? still 1.2.1 and mobile view issues still exists.


onboarding: Don't send zulip update message during new realm creation.

To help users focus on the onboarding experience, we no longer send the introductory "Zulip updates" message as a part of onboarding.
Now, we send the introductory message just before the first update message.
Fixes: #30053.
We diverted a bit from what the issue suggests after discussion -- CZO Discussion

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @prakhar1144!
The logic for checking if there's an existing message is not infinitely robust against sending the message more than once, given i18n or configuration changes may cause the topic to move, but it also doesn't seem wrong to me to always have a topic containing these start with that copy, so I think this logic is what we want.


wasmer: 2.3.0 -> 3.0.0

Description of changes
wasmer allows selecting backend at runtime, we can enable all available ones for respective platforms.

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, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Getting assorted test failures on darwin, guess the guards are still necessary.

obsoletes https://github.com/NixOS/nixpkgs/pull/195707 and https://github.com/NixOS/nixpkgs/pull/189600
closes https://github.com/NixOS/nixpkgs/issues/138538

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

1 package built:

wasmer

Result of nixpkgs-review pr 202340 run on aarch64-darwin 1

1 package built:

wasmer

Result of nixpkgs-review pr 202340 run on x86_64-darwin 1

1 package built:

wasmer

Just a note: 3.0.1 is already out.

nixpkgs-review happy, aarch64-linux


how to use chakra_trace_link?

Please provide a detailed description of your question or the information you seek.
i have follow your https://github.com/mlcommons/chakra/blob/main/USER_GUIDE.md to install the chakra.

however i don't know how to run your code.
$ chakra_trace_link
--pytorch-et-file /path/to/pytorch_et
--kineto-file /path/to/kineto
--output-file /path/to/merged_et
What does this mean?  should i import or run in bash or something?

It is too tough, is there any existing Chakra ET available？

Hi, @tyn513. Thank you for your interest. Once you install Chakra following the USER_GUIDE.md, you can run chakra_trace_link from your shell.


InlineValidator now binds $method to $model in 2.0.36

As of b642a384fa0184c58343f1d07e3d7007639fd700, InlineValidator is now binding closures to the model being validated:
https://github.com/yiisoft/yii2/blob/700ac0270644f8c7fe7a52f67ecfe8546ef86f37/framework/validators/InlineValidator.php#L76-L78
This is a change in behavior from prior versions which was not mentioned in the upgrade notes. It can lead to PHP errors from validation methods that were registered from a different class.
For example, Craft allows event-based rule registration with an EVENT_DEFINE_RULES event in craft\base\Model::rules():
https://github.com/craftcms/cms/blob/68d7e40efc7630fc319a27e0536426028d7baa50/src/base/Model.php#L78-L89
So now if an inline validation rule is added via that event, like this:
Event::on(Entry::class, Entry::EVENT_DEFINE_RULES, function(DefineRulesEvent $e) {
    $e->rules[] = ['title', function() {
        $this->someMethod();
    }];
});

A PHP error will occur, since $this within the callable is no longer a reference to the same class that the event handler was registered in; it will be the Entry object.
What steps will reproduce the problem?

Add a way for other classes to register validation rules to a model’s rules() method.
From a different class, register an inline validation rule on that model, which expects $this to be the rule was originally defined in.
Validate the model.

What is the expected result?
$this references the class the rule was defined in.
What do you get instead?
$this references the model’s class.
Additional info

Q
A

Yii version
2.0.36

PHP version
n/a

Operating system
n/a

I was able to work around this by wrapping inline validators’ closures in another closure (craftcms/cms@9f4105ca265c2e778bd4c679f852dda2b18c5c9d), but the change should still at least be documented in the upgrade guide.

I used inline validators and js validation, that is broken with 2.0.36.
In the class "yii\validators\InlineValidator" the method "clientValidateAttribute" has a bug. The line:
$method = $this->method->bindTo($model);
must be:
$method = $method->bindTo($model);

@cs-crewsAndCaptains Probably worth posting that as a separate issue since it’s more of a direct bug as opposed to a gap in the documentation.

@cs-crewsAndCaptains Probably worth posting that as a separate issue since it’s more of a direct bug as opposed to a gap in the documentation.

@brandonkelly okay, if have add: #18204


Use object type

Fixes #13493

@sandersn @RyanCavanaugh @mhegazy Please review.

@sandersn @RyanCavanaugh @mhegazy @DanielRosenwasser

Let's swap @sandersn with @rbuckton. I'm not fluent enough with runtime semantics to be a useful reviewer here.

I see, thanks.

@rbuckton Could you?

Thanks!


docs: document what are possible values of format option

Description

References

Review Checklist

[x] I have clicked on "allow edits by maintainers".
[x] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[x] I have added tests to validate that the change in functionality is working as expected

Approved, but converting to draft so that we do not accidentally merge. We need to enable the Linux Foundation's EasyCLA bot on this repo before we can merge PRs.
Will ping you once we have done so!

Ok thanks

/easycla

@big-kahuna-burger mind signing the CLA above? Once it is signed, we should be good to merge!

Done 😊

Thanks!


Introduce Vimeo video channels to jvm-bloggers

Vimeo provides rss feed with videos, so existing solution works there too.


[zsh-completion] Known issues

[x] ~USERNAME**<TAB>
[x] ~INVALID_USERNAME**<TAB>

nonomatch

[ ] foo\ bar**<TAB>
[ ] setopt vi resets TAB binding from fzf

Thanks for all this @junegunn ! Also I think we should add support for configuring a different bound key and having no trigger at all. Specifically, the use case I'd like to support is: start writing a path name, use tab completion as usual up to a certain point, and then use a different shortcut (e.g. CTRL+P) to trigger fzf from that point on (without ever typing a specific trigger).


Fix a crash on connection migration to the advertised preferred address

This closes #7045

This fixes the last known crash on 9.0.x at the moment.

[approve ci autest]

[approve ci autest]

[approve ci autest]

Cherry-picked to v9.0.x branch.


Report ZCHECK_NOTE_3089413 Incomplete check for note 3287611

Note 3287611 shows green instead of yellow or red result if only version 2 instead of version 4 is installed. The corresponding message shows: "Completely implemented version 0002"

Solved with https://github.com/SAP-samples/security-services-tools/commit/b7e2ae77e2df048a59b622619d8b1a50f92999ba on 13.03.2023


Add Ownership and Tenancy Mixins to Authentication

This adds ownership and tenancy to authentications allowing for key
pairs added by EmsRefresh to be associated with users in manageiq.
Depends: https://github.com/ManageIQ/manageiq-schema/pull/237
https://bugzilla.redhat.com/show_bug.cgi?id=1589766

Gaprindashvili backport: https://github.com/ManageIQ/manageiq/pull/17732

@agrare if this can be backported, please add the gaprindashvili/yes label.

I already created a gaprindashvili backport PR
On Thu, Jul 26, 2018 at 2:46 PM, John Prause<EMAIL_ADDRESS>wrote:

@agrare if this can be backported, please add the gaprindashvili/yes
label.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@Ladas so could we set these during EmsRefresh if the parser returns the username?

@agrare we should probably focus on manual assignment for now, plus assignment during creation. Manual and refresh assignment will probably not work together well, since they will be overwriting each other.

@gtanzillo this is ready, can you take a look?

This should fix the CIs https://github.com/ManageIQ/manageiq/pull/17998


feat: export runtime metric to promethues

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
export tokio runtime metrics
Please explain IN DETAIL what the changes are in this PR and why they are needed:

use tokio-metric to collect runtime metrics
use tokio-metric-collector to expose to promethues

Checklist

[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)
https://github.com/GreptimeTeam/greptimedb/issues/1529

@MichaelScofield PTAL

@SSebo can you attach some output as an example to illustrate the tokio runtime metrics look like?

@SSebo can you attach some output as an example to illustrate the tokio runtime metrics look like?

Of course. How about this ?
            assert!(metric_text.contains("runtime_0_tokio_budget_forced_yield_count 0"));
            assert!(metric_text.contains("runtime_0_tokio_injection_queue_depth 0"));
            assert!(metric_text.contains("runtime_0_tokio_workers_count 5"));


smash.gg integration

Adding smash.gg integration will make pulling standings from SSE tournaments a lot simpler. I don't think that this would be done in the theme, but rather in a Ghost API, but it still needs to be determined on how we're gonna do this.

The theme would not be able to do this, rather the app its self would have to.

The theme would not be able to do this, rather the app its self would have to.

Good to know. We'll go ahead and fork Ghost for this.


Page HOC concept (refactoring)

ref #3286
OK, here is one refactoring I wanted to come to for some time and mentioned in #3286. Currently just for one page as a demo. Eventually I'd suggest to migrate all pages to this approach.
Now all pages subclass gui_page which is cluttered: trickier to maintain, necessity to call super, confusion between renderable~render and render_content, redefinitions.
Idea: all generic page stuff - html head, footer, scripts - has to be a HOC (high order component). Which accepts a child component to render inside itself.
Consider changes to zcl_abapgit_gui_page_addonline - no redefinitions, no super->on_event,  proper reuse of interfaces,  completely independent and clean. Well, create knows about the gui_page but it is more for convenience sake.
zcl_abapgit_gui_page_hoc is a temporary piece - as soon as all pages will follow the proposed pattern it can be merged with gui_page (which will become non-abstract final class)

I like it. "HOC" seems a bit abstract. I would call it LAYOUT. The current HOC/LAYOUT would be a "full-page" i.e. a container for one child component. but we could have other layouts like "main and side panels" (two children) or "grid" (several kids) etc.
Maybe wrap the child content in a div for css:
ri_html = '' && mi_child->render( ) && .

HOC - it is from react and js world - very elegant concept - https://reactjs.org/docs/higher-order-components.html
It does not much matter how it is called though because this class is temporary - should merge with gui_page someday.
wrap in div: potentially ... but I'd wait for a practical application case. Currently the code replicates render_content logic 1-to-1. Most (or all) of pages return a single tag currently. Also I don't thin that container should impact styles of subcomponents (css class). So let's really wait for practical need.

@FreHu what do you think ? :)

Definitely looks simpler, and I had pretty much all the confusions you mentioned. As for how frontend is done these days, I don't really know as my FE career went from jquery to a frontend running inside abap backend :)

@FreHu: :+1: thanks !
P.S. FE carrer: yeah, I'm far from expert in this area actually, just picking nice things here and there trying to make stuff convenient and ordered :)


predict() does not correct for unbalanced classes in dataset

Would be nice to have support for balancing classes in dataset when building svm

This should work if you pass sklearn style kwargs to the method. For example:
dat.predict(algorithm='svm',cv_dict={'type':'kfolds','n_fold':5), {'class_weight':'balanced'})
The {'class_weight':'balanced'} part is a dictionary specified according to the sklearn spec for a particular algorithm, e.g. in this case SVM

You're right, that works. Actually figured it out a little while ago, should've closed the issue, sorry! And thanks!


Imported target "TIFF::TIFF" includes non-existent path C:/bld/qt6-main_1696944674905/_h_env/Library/lib/cmake/tiff/include

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
when building qt6 on windows with 4.6.0 I get this error:
CMake Error in qtimageformats/src/plugins/imageformats/tiff/CMakeLists.txt:
2023-10-10T13:54:39.7768892Z   Imported target "TIFF::TIFF" includes non-existent path
2023-10-10T13:54:39.7905566Z
2023-10-10T13:54:39.9591525Z     "C:/bld/qt6-main_1696944674905/_h_env/Library/lib/cmake/tiff/include"

I looked at the Library/lib/cmake/tiff/TiffTargets.cmake but it looks correct:
# Compute the installation prefix relative to this file.
get_filename_component(_IMPORT_PREFIX "${CMAKE_CURRENT_LIST_FILE}" PATH)
get_filename_component(_IMPORT_PREFIX "${_IMPORT_PREFIX}" PATH)
get_filename_component(_IMPORT_PREFIX "${_IMPORT_PREFIX}" PATH)
get_filename_component(_IMPORT_PREFIX "${_IMPORT_PREFIX}" PATH)
if(_IMPORT_PREFIX STREQUAL "/")
  set(_IMPORT_PREFIX "")
endif()

# Create imported target TIFF::tiff
add_library(TIFF::tiff SHARED IMPORTED)

set_target_properties(TIFF::tiff PROPERTIES
  INTERFACE_INCLUDE_DIRECTORIES "${_IMPORT_PREFIX}/include"
)

any idea ?
Installed packages
libtiff 4.6

Environment info
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=800840&view=logs&jobId=a70f640f-cc53-5cd3-6cdc-236a1aa90802&j=a70f640f-cc53-5cd3-6cdc-236a1aa90802&t=6119ccbe-9301-594f-7c27-f792b80a7fcc

it seems to be including the build time path on windows. how does that typically get addressed there?

set_target_properties(TIFF::tiff PROPERTIES
  INTERFACE_INCLUDE_DIRECTORIES "${_IMPORT_PREFIX}/include"
)

^^^ Is the above line from qt or from libtiff? if it is from qt, it seems to be wrong and likely assuming vendoring of some kind

I'm not an expert with cmake, but is _IMPORT_PREFIX equivalent to ${PREFIX}, or ${PREFIX}/lib/cmake/tiff?

oh yes, that's the build path, maybe with "detect_binary_files_with_prefix: true" then
_IMPORT_PREFIX should have a value of ${PREFIX} at the end (%LIBRARY_PREFIX% on windows)

looks that TiffConfig.cmake is bogus

https://gitlab.com/libtiff/libtiff/-/merge_requests/541

xid you want to make a PR here to fix it in the feedstocks?
I'm very ok fixing build related stuff when an upstream issue or merge request is created

lets see what libdiff says first, I already have a simple workaround in the qt feedstock


There is some problem after running the "conda env create -f environment_win.yml"

Hello!
Sorry to bother you. Thanks for this brilliant paper and all the work. I find the idea of applying MARL to distribution network is very enlightening and try to reproduce it.
But when I try to install the dependencies, I execute the command conda env create -f environment_win.yml and get error codes:
Using cached gym-0.21.0.tar.gz (1.5 MB)
Preparing metadata (setup.py): started
Preparing metadata (setup.py): finished with status 'error'
Pip subprocess error:
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [1 lines of output]
error in gym setup command: 'extras_require' must be a dictionary whose values are strings or lists of strings containing valid project/version requirement specifiers.
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
failed
CondaEnvException: Pip failed
Is there something wrong with the pip? I'm new to python, and this question bothers me a lot.
Could you please help me with this? Thank you so much!
Best wishes

me using windows10 and python 3.9

Is this fixed? I have the same issue.

Is this fixed? I have the same issue.

No😭Do you have any solutions?


Get private ssh key from bitwarden

use bitwarden cli. Test by making a commit to github

Can not be done. No file support on Bitwarden.


RAD-X3: maybe don't need text about sending to more than one Mgr

Section Number https://github.com/IHE/RAD.RTC-IMR/blob/Editorial-updates-lynn2/input/pagecontent/rad-x3.md#24x341-open-report-context-request-message
Issue There is standard text that reads, "The Sender shall support sending such messages to more than one Manager."   In this case that's probably not needed?   Wouldn't there be just one manager in a deployment?   If you agree, I did not consider whether this comment applies to other transactions.
Proposed Change Consider removing the sentence.
Priority:

Low: Typo or other minor classification that an editor can manage. Requires no group discussion.

We discussed this in Rad Tech. We will keep the generic statement. Although it is rare to have multiple managers, there may be multiple Managers (e.g. a reading group needs to use different reporting systems during the day vs during night read. There may be different Managers in this case).


default Cascading checkbox to TRUE

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

Fixed in next version.

Solved, when creating new setup, the cascading checkbox is default ON.


Features/#239 config override object

connects #239

note: plugins section config is breaking from previous commits and this is acceptable on development. Anyone who is using develop branch need to update accordingly.

no time for review, merged.


Imaginary part of the wave function

In the time independent harmonic oscillator from example, the eigen states doesn't have any imaginary part contrary to what could be expected by the analytical resolution of the harmonic oscillator. (exp(imphi))
Do you have any idea why, and how I could fix it ?

Hi,
The harmonic oscillator, as well as any other system subjected to a real value Hamiltonian, only has real-valued eigenstates up to an arbitrary complex constant with a module equal to 1.
Eigenstates of the 1D harmonic oscillator are:

Which they only have real part.

Yes but the 3D harmonic oscillator has the spherical harmonics function :

The complex spherical harmonic basis is not the unique basis of the 3D harmonic oscillator due to energy degeneracy. This solver will normally converge separately to the real  $\frac{Y^{m}_l + Y^{-m}_l}{2}$ and imaginary part   $\frac{Y^{m}_l - Y^{-m}_l}{2i}$ of the wavefunction you cite, which form a basis with the same span as the m = +-1 spherical harmonics $Y^{m}_l$ and $Y^{-m}_l$
For the hydrogen atom, it will happen exactly the same: instead of converging to the complex spherical harmonics, it will converge to the basis it's usually used in chemistry books to represent p orbitals.   However, if you turn on a magnetic field, as illustrated in the Zeeman effect example, the degeneracy breaks and the eigenstates will converge to wavefunctions with the angular part approaching the complex spherical harmonics.


Project Compile Failed

I downloaded project and when start compile then it failed.
Please suggest me to work properly.

Yup +1 for the above !!!

You need to update to Xcode beta 2.

@Dimillian I'm on Xcode beta 2. I'm facing below issue in console while running app in Simulator.
[general] CFLocaleCopyCurrent() failed to look up current locale via 'AppleLocale' and 'AppleLanguages' in user preferences; falling back to locale identifier 'en_US' as the default. Consider checking Console for sandbox violations from this process for reading from preferences, or enabling CoreFoundation debug logging for more information. This will only be logged once.

There is ton on output in console because it’s a beta built. But if the app load and you can see stuff on screen it should all be good :)

I guess you should delete and recreate the simulator maybe too, it should help.

Update to Xcode beta2 and now it all works. Thanks

I just downloaded and ran in Xcode Beta 2, works perfect!

There is ton on output in console because it’s a beta build. But if the app load and you can see stuff on screen it should all be good :)

Thanks for the info. Seems like https://www.themoviedb.org/ is not accessible from my end. May be blocked by ISP.

@Dimillian Thank you for quick response.
Really appreciated!!!!

I'll close this one, don't hesitate to open a new one if any problem occur.


[MiCS] [Fix] saving and loading model checkpoint logic for MiCS sharding

Only the first partition group will save the model checkpoints
Need to avoid call dist.barrier on the WORLD group.
Including the support for loading the partitioned model checkpoint as ZeRO-3

Hi there, one of the unit test is failed, but I didn't see the corresponding error.
Can someone help to retry with the nv-torch19-v100 / unit-tests

Hi @samadejacobs @tjruwase
would you mind taking a look at this PR? It is an easy fix.
The only thing that can affect the current logic of ZeRO-3 is the in deepspeed/runtime/engine.py: _create_zero_checkpoint_files, in which I let the dist.barrier function get called on ranks within the self.optimizer.dp_process_group.

@zarzen, I am unable to rebase this PR. Did you set some restrictions on your branch?

I think it is due to the permission policy on dmlc account.
I just rebased with the master branch. Will take a look at the permission setup.


Duplicate Keys with different Labels

If I add two entries with the same Key but different value and Label, I am not able to access these duplicate Keys from the .Net Core App/Azure Function.
What is the use of Label? Is this similar to Version?
If that is the case, still shouldn't all the Keys with different values still be accessible?

Thanks for trying this out @kamalsivalingam.
By default, the config provider of the App Configuration will load keys with no labels for simplicity. When labels are used, they can be loaded with the Use function. The example below loads all keys with the label mylabel.
configBuilder.AddAzureAppConfiguration(options => options
    .Connect(configuration["connection_string"])
    .Use("*", "mylabel")

Labels provide an extra dimension to key-values, so each key can have multiple values at the same time. It can be used to differentiate config for different environments (dev, test, ...), different microservices (app1, app2, ...), different regions and so on so forth. We also expect it will be a very typical use of labels for versioning. For example, I have config labeled with v1 and v2. In my app, I want the v2 config overwrites the v1 config when they have the same key. Then I will have something like this
configBuilder.AddAzureAppConfiguration(options => options
    .Connect(configuration["connection_string"])
    .Use("*", "v1")
    .Use("*", "v2")

Please check out the AzureAppConfigurationOptions class, which provides APIs for filtering key-values, watching for changes, connecting with managed identity and so on.
Hope this helps. Please let me know if you have any other questions.

I've added 2 keys with same name  (poc-api:myvalue), one with label "Development", the other with no label.
With this code :
public static IWebHostBuilder CreateWebHostBuilder(string[] args) =>
        WebHost.CreateDefaultBuilder(args).ConfigureAppConfiguration((hostingContext, config) =>
            {
                var settings = config.Build();
                config.AddAzureAppConfiguration(options =>
                {
                    options.Connect(settings["ConnectionStrings:AppConfiguration"])
                        .Use(KeyFilter.Any, LabelFilter.Null)
                        .Use(KeyFilter.Any, "Development")
                        .UseFeatureFlags(optionsff =>
                        {
                            optionsff.PollInterval = new TimeSpan(0, 0, 5);
                        });
                    options.Watch("poc-api:myvalue",new TimeSpan(0,0,5));

                }); ;
            })
            .UseStartup<Startup>();

It seems I cannot get the value for the key with label "Development", i always get the one without the label
using
<PackageReference Include="Microsoft.Extensions.Configuration.AzureAppConfiguration" Version="1.0.0-preview-008920001-990" />

Code "Inspired" by
https://docs.microsoft.com/en-us/azure/azure-app-configuration/howto-best-practices


Don't scroll for fitting new comment/reply input if input is already on-screen.

What fixed #555 also broke the experience for non-mobile users. Just go to any topic with some decent level of activity and try to reply to an argument.
Automated scrolling should:

Only work if the input + submit/cancel buttons are partially off-screen.
Be smooth. Add some easing or something to the auto-scroll.

I was looking at component/scroll-to. It seems to do what you described but it uses the window as the scrolling element while in DemocracyOS it is the #browser section what's need to be scrolled.

@solde9 I guess you may fork scroll-to and make the scrolling element part of an opts param, and using window as default.

This is already fixed.


Memory accumulating during docking runs

Description
Not sure if this is a lilac thing, an OpenEye thing, a multiprocessing thing, or just an unrelated set of errors, but @JenkeScheen and I have been getting some core dumps and I'm wondering if in part it is due to the way in which memory is being allocated (and not overwritten by OpenEye).
Potentially Related

#181

Error Messages
/scratch/lsftmp/1680789260.16343999.shell: line 17:  7538 Bus error

TERM_MEMLIMIT: job killed after reaching LSF memory usage limit.
Exited with signal termination: 25.

Resource usage summary:

    CPU time :                                   1893.00 sec.
    Max Memory :                                 144 GB
    Average Memory :                             65.79 GB
    Total Requested Memory :                     144.00 GB
    Delta Memory :                               0.00 GB
    Max Swap :                                   -
    Max Processes :                              76
    Max Threads :                                88
    Run time :                                   1000 sec.
    Turnaround time :                            1591 sec.

Things to try:
If they're checked off and this issue is still open that means I tried it and it didn't improve the slow increase of memory use over time.
I/O management

[x] load_openeye_sdf, etc functions should be closing the i/o streams so I don't think this is it
[ ] use with statements for file reading and writing to ensure no memory leak from file objects staying open

pooling parameters

[ ] use maxtasksperchild=1 to force each child process to close
[ ] (?)

make sure all openeye objects are being deleted (?)

[ ] the generated OE objects might not be getting deleted

Questions

[ ] how should I go about debugging this efficiently?
[ ] what can we do in the meantime

looping in @hmacdope @ijpulidos - we should be able to inspect the core dumps using gdb : https://stackoverflow.com/questions/5115613/core-dump-file-analysis

I've observed core dumps on lilac with non-docking code as well (happened with find_all_mcs.py).

Another thing to try is valgrind (I can try it at some point). It will be slow so use a small amount of data if possible.
export PYTHONMALLOC=malloc
valgrind python find_all_mcs.py

Instead of actually uncovering the problem, running smaller segments of jobs in job arrays on lilac seems to be working fine for me


Documentation updates for Dockerfile and README files

Signed-off-by: Rajan Ravi<EMAIL_ADDRESS>

LGTM


WSL2 driver directory is using to much disk space

I searched Google for a solution to reduce WSL2 disk space and found a guide on shrinking the virtual disk. It helped me reduce 44GB to 32GB now.
But the main issue is that my physical disk is running low and the WSL2 driver directory uses 224GB out of 237 GB. It;'s the same as my C driver.
I can't any guide or any discussion on the WSL2 driver directory that used so much disk space.
Does anyone have any idea if it is possible to delete it or clean it up?

/question

/usr/lib/wsl/drivers is automatically mounted and linked into your WSL filesystem from your Windows DriverStore, at C:\Windows\System32\DriverStore\.
It is not taking up space in your WSL filesystem's virtual file system, e.g. in the Ubuntu .VHDX that contains your WSL installation.
If you notice, the used and available space listed by df is actually the same as C:\.

Indeed @jadentohjm: /usr/lib/wsl/drivers is just a mountpoint that doesn't actually take space inside WSL. If you want to know where space is being taken, running something like du -sh would be the easiest way.


Track cash transfers as asset transaction type

moved to: https://github.com/centrifuge/pools-subql/issues/134


[stable/grafana] Sidecar Dashboard "ADD" files but there is no dashboard in Grafana

Hello,
Describe the bug
I'm deploying Grafana with Helm. I enabled sidecars for dashboards, datasources and notifiers. When I'm deploying the chart, all the sidecars are running and the logs telling me all is working.
Datasources is well configured, same for notifiers. But I cannot find my dashboards.
My dashboards are in config maps.
Here is the logs of grafana-sc-dashboard
$ kubectl logs -n monitoring grafana-559dd97b9d-n8pvh grafana-sc-dashboard
[2021-03-08 14:39:26] Starting collector
[2021-03-08 14:39:26] No folder annotation was provided, defaulting to k8s-sidecar-target-directory
[2021-03-08 14:39:26] Selected resource type: ('secret', 'configmap')
[2021-03-08 14:39:26] Config for cluster api loaded...
[2021-03-08 14:39:26] Unique filenames will not be enforced.
[2021-03-08 14:39:31] Working on configmap monitoring/wbs-grafana-dashboard-traefik2-cm
[2021-03-08 14:39:31] File in configmap traefik2.yaml ADDED
[2021-03-08 14:39:31] Working on configmap monitoring/wbs-grafana-dashboard-nodes-cm
[2021-03-08 14:39:31] Found a folder override annotation, placing the wbs-grafana-dashboard-nodes-cm in: /tmp/dashboards
[2021-03-08 14:39:31] File in configmap nodes.yaml ADDED
[2021-03-08 14:39:31] Working on configmap monitoring/wbs-grafana-dashboard-cluster-total-cm
[2021-03-08 14:39:31] File in configmap cluster-total.yaml ADDED

Version of Helm and Kubernetes:
Helm: v3.5.0
Kubernetes: v1.20.2
Which chart:
Stable/Grafana
What happened:
I have no dashboard in my Grafana.
What you expected to happen:
To have dashboard in my Grafana ^^
How to reproduce it (as minimally and precisely as possible):
I don't know how you can reproduce it.
Just deploy a grafana chart, in the values.yaml file, only configure "sidecars.dashboard".

Migrate the issue to grafana/helm-charts.


Use real notes for the binding page

This should use the proper NOTE component for notes on {Binding}

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

#sign-off


Development nextcloudv2

Continuation of #930
The api docs were in the way so I made a new branch and set gitignore with docs/api/
I also updated the work with the new changes with disks.
@AhmedHanafy725
@MohamedElmdary
I tag you here since you did already help a lot in the previous PR, so you know the ongoing work.
Thanks.

This now works and all checks are passed.
Two things we might want to improve before publishing:

Have the IPv4 address on by default, and that it cannot be set off

Reason: the deployment needs an IPv4 address

Remove the micro VM drop down menu so users cannot change to "Other" and change the FList

@AhmedHanafy725
@MohamedElmdary
Now, the IPv4 is there by default and users can't change it. So it should always work.
Also the VM image can't be changed, it's always the Nextcloud AIO flist.
I might do some deployment testing and I invite you and others to test it if needed.
Thanks.

TODO:

[ ] Remove option to chose "Other" as a VM Image (or remove completely the VM image menu)
[ ] Add IPV4 in Contract table after deployment

@AhmedHanafy725
@MohamedElmdary
Apart from needing to promote the FList (current tf-images issue), I think this is ready to go.
Please, if you want, have a test and let me know if I need to adjust anything.
Thanks!

@MohamedElmdary
Do you think you could have a look?
On my end it did work perfectly when I deployed the playground locally.
I think new changes in the dev branch made the Full Clients Lint test not working.
Other than that, I would think it's ready.
Thank you.

@Mik-TF Lint workflow is passed now. All I had to do is to re-run it.

@zaelgohary
@AhmedHanafy725
@MohamedElmdary
I did all the changes requested.

Additional note for branch development_ncgateway
I am also testing the same nextcloud weblet, but this time with the possibility of setting a gateway and the gateway domain. Much like it is possible with CasperLabs. I took the CasperLabs.vue template for this other branch.
The gateway version is available in this branch: https://github.com/threefoldtech/tfgrid-sdk-ts/tree/development_ncgateway
Perhaps it would be possible to set it correctly. But I haven't succeeded yet (started not long ago).
So far here is what works with this other branch (ncgateway):

ipv4 and custom domain

it works on both ipv4:8080 and domain:8443

ipv4 and gateway domain

it works on ipv4:8080 but not gateway_domain:8443

gateway and gateway domain

doesn't work in both case so far

This was requested by some TF users that is why I try with gateway + gateway domains.

@Mik-TF Deploy button is not disabled when there is no node chosen.

@zaelgohary
Thanks for the feedback. I set the deploy button as false by default and set the form-validator so it checks all the deployment's parameters (as the standard with other playground weblets).

Thanks @zaelgohary for all your help.
Thanks also to @AhmedHanafy725 @MohamedElmdary


Merge to dev: Restore ModelMetadata.PropertyName != null behaviour

#7413 part 2 of 2
add ModelMetadata.Name and ParameterName

use Name instead of PropertyName in most cases

update ModelMetadata.ContainerType and other property use

choose using MetadataKind almost everywhere; support all possibilties

usually parameter metadata was possible but not handled
worst case was one or two potential NREs, especially ContainerType.* dereferences

improve MvcCoreLoggerExtensions metadata handling

add three new debug messages, one for type metadata and two for parameter metadata

update ModelMetadata.ContainerMetadata, ContainerType and PropertyName doc comments
no changes needed in Microsoft.AspNetCore.Mvc.ViewFeatures because parameters aren't viewed

nits:

add missing TestModelMetadataProvider.ForParameter(...) method
remove unused EmptyModelMetadataProvider instances in ModelMetadataTest
refactor ModelValidationResultComparer out of DataAnnotationsModelValidatorTest`
take VS suggestions, mostly related to variable inlining and object initializers

FYI only. Will merge once tests complete.

Tests are complete!

:ok:


Faulty triangles

Lighting and reflections with triangles are not calculated as expected

The edges of the triangles are notched

I am working on this problem now

I fixed a bug with a distorted shadow (it seems like we need to test everything)

It turns out that it was necessary to normalize the vector, I will tell you the details later

It remains to solve the problem with jagged edges

I fixed a bug with faulty faulty from point light sources, and also fixed a bug with shadows in reflections.
Before:

After:

Maybe there are more bugs in the reitracer, I'll check, I figured it out and it pleases!


Update .gitattributes

Fixes https://github.com/github/linguist/issues/2332

LGTM +1

We've got a number of .jsm files in the repo, and those need to be recognised as JavaScript too.
@arfon Could you please amend the commit to also cover the *.jsm case; and also please update the commit message to something a little more descriptive, e.g.
Update .gitattributes to fix GitHub file type detection for JavaScript.

@Snuffleupagus done.

Thank you for the patch!

works :-)


ci: Pre-release v0.25.0

What do these changes do?

Related issue number

Fixes

Codecov Report

Merging #3175 (9811e83) into main (9a6597c) will decrease coverage by 6.80%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3175      +/-   ##
==========================================
- Coverage   42.29%   35.49%   -6.80%
==========================================
  Files         101      101
  Lines       10907    10916       +9
==========================================
- Hits         4613     3875     -738
- Misses       6294     7041     +747

see 29 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


Config Schemas for all my packs

Pretty self-explanatory

Nice!
Trying to get all those commits, huh :P

LGTM, +1


Final touches

The main activity was to improve the look and feel of the website using CSS styling.
The following changes were added to achieve that:

[ ] Adding vibrant colors.
[ ]Making the website screen responsive.

STATUS: APPROVED!!!
Hi @stephenkati,
Good job !!!
To Highlight!
heavy_check_mark Good commit messages heavy_check_mark Descriptive Readme file
Your project is complete! Great work on making the changes requested by the previous reviewer. There is nothing else to say other than... it's time to merge it :shipit: Congratulations! tada tada tada champagne champagne champagne

Optional suggestions
Check the comments under the review.
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
N/A
Cheers and Happy coding!clapclapclap
Feel free to leave any questions or comments in the PR thread if something is not 100% clear or you can get to me on slack using this handle @Mihndim.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you, @Mihndim2020

Thank you, @Mihndim2020


[Feature Proposal] Display multiple data sets and fits together

Many times we are required to plot similar data sets (and their fits) together, similar to plot overlay. For example, the relation between voltage and current at different temperatures. We imagine loading many data sets, plotting them + any number of fitted function. An example of the desired output is attached in the screenshot: two different data sets, with different fits, are displayed together.

First, I agree that this is an important ability that should be supported in a future version of Eddington-GUI.
However, this is a big change in the Eddington-GUI workflow: at the moment, Eddington-GUI is designed to work with a single data file at a time, and changing that means a big change in the user experience of the software.
At the moment, I add this ability to the 0.1.0 release milestone, since it is a huge change.
We may consider pulling it sooner if we could find a way to make that ability possible within the current workflow.

First, I agree that this is an important ability that should be supported in a future version of Eddington-GUI.
However, this is a big change in the Eddington-GUI workflow: at the moment, Eddington-GUI is designed to work with a single data file at a time, and changing that means a big change in the user experience of the software.
At the moment, I add this ability to the 0.1.0 release milestone, since it is a huge change.
We may consider pulling it sooner if we could find a way to make that ability possible within the current workflow.


Bones fail to colorize even though property has changed

Steps to reproduce
Load any model and change the color of the geometry, e.g. bones.
Expected result
Expect color of property to be respected and have bones change color.
Actual result

Environment and GUI version
OpenSim 4.0 release posted on SimTK. Both MacOSX and Windows.

@aseth1 I am only able to reproduce this behavior when I have had OpenSim open for a long time and my computer has gone to sleep, woken up. I think this could be related to #523 and not an issue on its own.

I cannot edit the color of all or multiple bones even after restarting OpenSim and loading a model. I am having the same issue on my Mac. Even though the color property appears to be set correctly, for the color to appear I must edit the color of the individual geometry to some other color and then set the value back to the bone color for it to take effect.

I am testing with my Mac and the jumper model we used for the webinar. I can change the color of a body or the individual geometry (one at a time or via a multi-select) via the navigator Display -> Color dropdowns. Color gets updated immediately. I can also edit colors of geometry individually or multi-select via the Property Editor.
@aymanhab Any ideas on what could cause differences between what Ajay and I are seeing?

My guess is that he's not doing GUI editing, instead he's modifying the file. It's possible the initialization sequence is different so bones show up first time in default colors. If you can save and load model with non default colors of bones that would prove one way or the other.

I can save a load a model with non default bone colors.

Could be typo or format of edited colors then. Can we diff the files?
On Mon, Mar 4, 2019, 4:30 PM Jen Hicks<EMAIL_ADDRESS>wrote:

I can save a load a model with non default bone colors.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opensim-org/opensim-gui/issues/1104#issuecomment-469484153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADyccEjSZeruzqhKM4WsT46v5BIb-QG9ks5vTbq9gaJpZM4aznTa
.

My guess is that he's not doing GUI editing, instead he's modifying the file.

Please look at the image in this issue. I use the GUI and the color picker to change colors. You can see that the values for "boney" bone color is correct but the bones remain white. I can change  Geometry individually and even multiselect Geometry under a Body, it turns out, but I cannot  change color via bodies->Display->Color although the properties do update. Has nothing to do with files.

Thanks for the additional detail @aseth1 . Confirming, as noted above that I don't see this behavior on my machine with the jumper model. Might there be differences due to the model used? Maybe it is easier to take a look in person.

Since this doesn't reproduce for me or @jenhicks on arbitrary models, I assume this is model specific (@aseth1 can you check a different model).
Bodies do not have Geometry of their own, the GUI instead collects Geometry attached to the body and applies the color/preference/etc. to them. This process apparently breaks for this specific model. Will need to have the model in hand to investigate, but likely the issue is related to how the various Geometry is attached to respective bodies or the presence of unexpected Geometry that breaks the "collection/traversal" process.

The issue occurst when there is a missing geometry file anywhere in the model. In the case of the upper-extremity model, I didn't care about the hand geometry visualizing so I was ignoring the missing vtp file warnings. Apparently we cannot do that anymore.


provided user data is not executable  (detected type: text/plain; charset=utf-8)

Hello, I am seeing this error in the following file: /var/log/amazon/ec2/ec2-macos-init.log
I do see a file written to disk located at: /usr/local/aws/ec2-macos-init/instances/<instance-id>/ named: userdata (No extension).
I can take this script and execute it directly and it will run to completion without failure.  Additionally, I have added another module to the init.toml after copying the userdata script to another location.  This module runs my script successfully without issue.  (I would use the same location but it gets overwritten as new userdata is pulled freshly from Amazon for each run it seems.).
Please let me know what I can do to help troubleshoot this issue.

Hi @dpreble-cisco, thanks for opening this issue to address this unexpected behavior you have been seeing.
Could you provide some more information to help us troubleshoot this issue? Some useful information would be:

AMI ID you used to launch the instance
What method you used to launch the instance with user-data
What user-data is being included (please scrub any sensitive data)
Output of sudo /usr/local/bin/ec2-macos-init version
The module you added to copy the userdata file
How the user-data is being manually executed

Thanks for getting back to me.  I am executing on two different instance families/ami IDs so I will answer for each separately using mac1.metal and mac2.metal
mac1.metal

AMI ID: ami-0665a0b2ea8636d1d
Launch Method: Auto-Scaling Group using Dedicated host provisioning based on the following method https://aws.amazon.com/blogs/compute/implementing-autoscaling-for-ec2-mac-instances/
User Data: I'd rather not provide this immediately.  I can take a user data script you provide and attempt to run the process with it.
Version output:

EC2 macOS Init
Version: 1.5.4 [2022-10-18 10:48:47 -0700]
https://github.com/aws/ec2-macos-init
Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.

The module you added to copy the userdata file:  My understanding is the init.toml is already configured based on this AMI provided by amazon to execute a UserData block.  The block I find in my unmodified init.toml is

### Group 5 ###
## Finally, run user data.

# Attempt to execute userdata, if provided
[[Module]]
    Name = "ExecuteUserData"
    PriorityGroup = 5 # Fifth group
    RunPerInstance = true # Run once per instance
    FatalOnError = false # Best effort, don't fatal on error
    [Module.UserData]
        ExecuteUserData = true # Execute the userdata

I modified the init.toml by adding the following block
[[Module]]
    Name = "TestUserData"
    PriorityGroup = 4 # Second group
    RunPerBoot = true
    FatalOnError = false # Best effort, don't fatal on error
    [Module.Command]
        Cmd = ["/bin/sh", "-c", "/Users/ec2-user/userdata.sh"]

Then I run the following commands:
ec2-macos-init clean -all
ec2-macos-init run

I can then see the successful execution of my script reflected in the log located at: /var/log/amazon/ec2/ec2-macos-init.log

How the user-data is being manually executed:

cp /usr/local/aws/ec2-macos-init/instances/i-080313263a0577882/userdata .
cp userdata userdata.sh
./userdata.sh >> userdata_exec.log &
tail -f userdata_exec.log

mac2.metal

AMI ID: ami-013846afc111c94b0
Launch Method: Same as mac1.metal
User Data: Same as mac1.metal
Version output: Same as mac1.metal
The module you added to copy the userdata file: I did not add a module to copy the file?: N/A, I didn't test this method on the mac2.metal yet.
How the user-data is being manually executed: Same as mac1.metal

@dpreble-cisco
Make sure the userdata script starts with either #!/bin/bash or #!/bin/zsh. The executeCommand function that is used by the UserData module attempts to detect the content mimetype and the underlying go/http implementation is somewhat limited. The test you've implemented which uses the Command module does not attempt to perform this same type of check.
https://github.com/aws/ec2-macos-init/blob/9460c024ac68de3bfca4e78102a691b4b7f1a22e/lib/ec2macosinit/userdata.go#L65
out, err := executeCommand([]string{userDataFile}, "", []string{})
	if err != nil {
		if strings.Contains(err.Error(), "exec format error") {
			contentType := http.DetectContentType([]byte(ud))

https://pkg.go.dev/net/http#DetectContentType
https://mimesniff.spec.whatwg.org/

@dpreble-cisco
I tried to recreate this issue using the same information that you provided, but I was not able to. I used the following script for my userdata:
#!/bin/bash
whoami > /tmp/userdata.log

I was able to test that the userdata works as expected.
ec2-user ~ %  cat /tmp/userdata.log
root

I believe the problem could be in the userdata you are using, like the other comment mentioned. Could you try running with this userdata and let me know if this works? Please let me know if this solution will work for you. Thank you!

@thimslugga Hi! Sorry for the late reply, testing this is a difficult process due to the boot time of mac os ec2 instances!
This was indeed the issue, as I had the correct directive but my templating of the script resulted in an empty line at the top of the file.  Sorry for such a simple issue!
Once this resolved the issue I ran into other issues as the userData script executes as root.  It's all working now, thanks for your help!
In regards to iteration time, is it possible to edit the userData without incurring the cost of booting a new mac ec2 instance?  My current iteration workflow is to update my launch template with my new user data, scale up a new instance to test the user data's success.  This takes at least 40+ minutes.  Of course, I try to use the instance to debug as much as I can, but it seems there are differences between calling the Module.UserData and Module.Command.  If you directly edit the userdata file located under /usr/local/... whenever you re-execute the init process with ec2-macos-init run it downloads the userData from what is configured on the instance.  I can only change this if I reboot which takes a significant amount of time for mac instances and cannot be done for auto-scaling as it will timeout and a new instance will be allocated.  Any help reducing the iteration time here would be greatly appreciated!


feat: Kakarot Rpc method "eth_getLogs"

eth_getLogs
Metadata

name: getLogs
prefix: eth
state: ⚠
specification

Specification Description
Describe the method
Parameters

None

Returns

uint64 - e.g. blockNumber

Kakarot Logic
Describe the interaction with Kakarot and/or Starknet.
e.g. "this method does not interact with the kakarot contract or any other starknet contract."
Kakarot methods
Which Kakarot methods are called?
Starknet methods
Which Starknet RPC methods are called? e.g. "Would be calling the corresponding starknet_getLogs https://github.com/starkware-libs/starknet-specs/blob/master/starknet_vs_ethereum_node_apis.md"

Check kakarot repo, in testing @ClementWalter has adapted starknet logs to be readable in evm format

Currently, logs in transaction receipt might be malformed, there'll need to be some amount of exploration.
eth_getLogs can check how logs were done inside the codebase already.

see this https://github.com/sayajin-labs/kakarot/blob/85a6fcaab5bcfc54b3cae46bba612a9ebbcbb484/tests/integration/helpers/wrap_kakarot.py#L65
there is a try/catch because there is no guarantee that the event is an EVM event and not a Starknet one.


agency used in colourShade and coreStrengthCode, version 2.0.0. agency vs assignedBy?

papiNet should be consistent when designing schemas.
In current work there are now both agency and assignedBy used with the same type of enumerations.
Either agency or assignedBy should be used, not both of them.
agency:
type: string
enum:
- Buyer
- Customer
- EndUser
- Seller
- Supplier

We decided to rename agency into assignedBy.

Done, see commit aeb0982.


Add output option for a hosts file

Something like terraform.py --hostfile. It would output the hosts between comment blocks for easy addition and removal: terraform.py --hostfile >> /etc/hosts should be a reasonable thing to do.

:+1:  Wow, did this get implemented this is great.

Yes, it's in now. Check the output of --help on your version for details and revisions since the original merge.

On Feb 12, 2016, at 8:46 PM, Ross Edman<EMAIL_ADDRESS>wrote:
Wow, did this get implemented this is great.
—
Reply to this email directly or view it on GitHub.

@BrianHicks Worked great for me. Thanks for this.


Resource rename

Those are quite generic names for the Resource types - we really don't want to have any additional clarity about them?
In other popular operators, I'm seeing "VaultService" or "EtcdCluster" or "PgCluster" or such.


Please add dark mode.

I love this index. Can u please add dark mode too. It will be great.

+1.
Dark theme would be awsome

https://github.com/alx-xlx/goindex
until official dark mode comes out :)

https://github.com/alx-xlx/goindex
until official dark mode comes out :)

Thanks for this.
One more thing.
Is it possible to add subtitles feature in player?

https://github.com/alx-xlx/goindex
until official dark mode comes out :)

Amazing, but the update pop-up is annoying in this one. Can you somehow disable it?


{get:side} and {set:side} Function in Tag 54

Would be great to have possibility to get and set value of Side tag 54 from and in the FIX Message.
Currently its only drop down list.

This is fixed in https://github.com/yaalalabs/fixyl/releases/tag/v1.0.9


lxd/db/networks: Reworks GetNetworkInAnyState and its ilk to split the functionality out into separate functions

This will pave the way for adding a GetNetworkInByIDAnyState function and avoid duplicating everything.
Back port of:
314575c723b900117ffd1422f07773b85a6ec600
ef0b86f305bde10c76d2b86883e9cd041229b527
4612e52aaea862f38a5e8bffb99f0a2c044a58bf
94565f4c51b315783c17267464dada20b7092684
3cdca5022f2a42b7ee07f52cb707e6ed7157b501
45b482f75edd79917f2dc74ca8eed797a2c06011
dfd55eb325dd18b6373eeaf9d474489fc3476c3d
Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

Testsuite passed


Set cairo version in manifest

Problem
Contracts might be dependent on a particular version of the corelib
Proposed Solution
Allow users to set the cairo corelib version to use in the Scarb manifest, providing a rev which could be a tag or commit hash
Notes
No response

Hey! This seems to duplicate #90. Closing this in favour of the original issue. It's a great suggestion to allow specifying a rev here!


how to improvee the rendering speed

Superset version
0.28.1
Expected results
quick / page table
Actual results
long time rendering , all rows in one page.
Steps to reproduce
I wrote a query which return about 93k rows . in psql console , that query only takes few seconds to return the result.
but in superset , with different browsers , the rendering take long time from 2:24(chrome) to 3:38(firefox), and take high single core CPU loading
here is the screen recording
firefox: https://www.youtube.com/watch?v=s_SH8U67_Dg
Braver(chrome): https://www.youtube.com/watch?v=X1WUxNrF_YA
OS: Ubuntu 18.04.2 LTS
CPU<EMAIL_ADDRESS>RAM: 8G
H.D.D: Pleextor M6P 128G SSD
is there anyway to improve the rendering speed ?
maybe take 1K rows or 100 rows per page ?
I search the issues and there`s a topic talk about "limit button" in superset , but I dont see that button in my page , how to enable the limit option ?

A few things here, first is that the term rendering is misleading. Assuming your database is in async mode, here's some of what's happening. You can track some of this using statsd logging if it's configured

query executes:sqllab.query.time_executing_query
fetching data: sqllab.query.time_fetching_results
state is changed to rendering
data is pushed to results backend: sqllab.query.results_backend_write
client is polling
data is available, request is sent to get the full result set
fetching over network
actual rendering on the browser starts

If you can just limit your query, all of this will be much faster.
The reason why the postgres client is fast, is that it doesn't fetch all the data upfront, at step 2 (fetching data), it fetches a page and shows it. The cursor is kept alive and can be used to retrieve more rows as you scroll.
We could improve all this to be more seamless/faster, but operating in async mode makes all of this more complicated. I think we'll improve this over time. One approach would be to deliver the data in two phases, with a new state partially-delivered or something like that.

Look for other sql-engine. For instance, there are some sql-engines especially for data vizualization, such as omnisciDB. Give it a try.


Add testnet support for btc.com

@KateNegrienko please confirm that all tests are passes on npm run test-browser


TerminalRenderer API

Suggestion: Provider TerminalRenderer API.
Reason:
Sometimes Whack Whack terminal may be used only as UI to render some ANSI terminal running somewhere else. In this case, only data stream may be available. A solution to this so far has been to pipe this stream through stdio of a broker console app, and make Whack Whack run this app.
This solution doesn't work prior to Windows 10 because Windows doesn't support ANSI. Any ANSI escape chars got transformed into garbage when winpty (in node-pty) reads them from the console buffer of the broker app.
The suggestion:
Provide an analog of TerminalRenderer API as in this VSCode Suggestion that would allow to:

Write data to xterm
Fire an event when user inputs data in xterm
Fire an event when user resizes the terminal window

Additionally, external terminal may not follow the xterm's window size and has its own dimensions.
Ideally, the TerminalRenderer API would need to account for that by:

Allowing xterm UI control to have specified size and do not auto fit the tool window
Allow callers to resize the xterm control

I have a couple of questions about the resizing:

What is the UI experience supposed to be like when the terminal is resized to smaller than the available window?
Do the consumers of this API expect to be able to resize the terminal to larger than the current window will allow?

@dgriffen

If the terminal is resized to smaller than the available window, I'd expect the empty space to be filled with background
Ideally, the consumers of the API should be able to resize the terminal to larger than the current window allows. VSCode guys have reservations about that, they're concerned about scroll bars, so I'm ok with not allowing the terminal to resize to larger dimensions. If multiple users share the same terminal, we'll pick the smallest dimensions to fit all users' windows. To satisfy this, we'd need to know the terminal window dimensions, and when user resizes it.

I have reservations as well about allowing resizing to larger than the available window space. Mostly for the reason that stacked scrollbars are going to be difficult to manage. (if the user scrolls their mouse wheel, which one should scroll?). In my case though it might be harder to get the maximum dimensions since I don't implement custom logic for determining size, I just reuse the fit() function from xterm.js.

Work in progress on the API is in my clone of the repo:
https://github.com/IlyaBiryukov/WhackWhackTerminal/commit/7f1ef57ed3cc166928c696c445fad6c4b55621d2
@dgriffen If you have any feedback on the API, I'd appreciate it.

@IlyaBiryukov I've looked at the API and have a few comments

It looks like the ITerminalWindow is just ITerminal without the ChangeWorkingDirectoryAsync method. Should you just reuse the ITerminal interface?

ITerminalRenderer has a few setters, I think it would make more sense to have those be settable at creation through the ITerminalRendererService

@dgriffen Here is the latest iteration that should cover your second comment.
https://github.com/IlyaBiryukov/WhackWhackTerminal/commit/50e563a421e0b0a213536751852eed266eaffbfa
I guess we can reuse ITerminal instead of ITerminalWindow. What should we do with ChangeWorkingDirectoryAsync called on an instance of TerminalRenderer though? Ignore, throw, or add an event to handle it?

The new version of the ITerminalRenderer API looks good to me. I'm conflicted about what to do for the ChangeWorkingDirectoryAsync method. Adding an event to handle it doesn't make much sense for the liveshare usecase since the solution directory won't change for the duration of the session. I think its best to throw for now and perhaps come back to it later.


2组<PHONE_NUMBER>0徐文龙 add XDU生产实习 房价预测

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


There are a lot of query cannot be killed

one of the query elapsed like this:
com.cn :) select elapsed from system.processes limit 1;
SELECT elapsed
FROM system.processes
LIMIT 1
┌───────────elapsed─┐
│ 1628712.238700054 │
└───────────────────┘
1 rows in set. Elapsed: 0.005 sec.
com.cn :) select count() from system.processes  where elapsed > 1628712;
SELECT count()
FROM system.processes
WHERE elapsed > 1628712
┌─count()─┐
│      80 │
└─────────┘
1 rows in set. Elapsed: 0.128 sec.
when i use kill query where elapsed > 1628712, all the status is waiting.
my clickhouse version is 18.14.13.
how can i kill this query where  elapsed is too long? thank you

Some types of queries can't be killed (for example if it contains scalar subquery).
Some of queries check the 'killed' flag only on certain moments of execution and you need to wait till query will come to that moment.
In versions before 18.16.0 where was no possibility to kill the query which was not started yet (for example due to waiting for locks).

thank you

Related #1576


Increase content length for uploads

@amirardalan9473 flagged that he cannot upload a file with 43mb size. We initially thought it was related to the ingress controller that hosts the backend but looking at the logs from the backend we see that its the web server that is giving the issue.
Screenshot from nachet-backend logs (thank you argoCD) :

Looking at the code, I see we are using Quart as a web framework. Parsing default values i see that the limit for content is set as 16 mb :

reference :
https://github.com/pallets/quart/blob/8a0cdd5a650c29d036dfec3000818b0e06ce3e0c/src/quart/config.py#L19
We would need to increase this value to at least 200mb by doing something like :
from flask import Flask, request

app = Flask(__name__)

app.config['MAX_CONTENT_LENGTH'] = 200 * 1024 * 1024

for testing with a file, simply ask Amir or me and well provide it.

for testing with a file, simply ask Amir or me and we'll provide it.

@SonOfLope I would like to have a test file if you can provide one :)

@amirardalan9473 flagged that he cannot upload a file with 43mb size. We initially thought it was related to the ingress controller that hosts the backend but looking at the logs from the backend we see that its the web server that is giving the issue.
Screenshot from nachet-backend logs (thank you argoCD) :  Looking at the code, I see we are using Quart as a web framework. Parsing default values i see that the limit for content is set as 16 mb :  reference : https://github.com/pallets/quart/blob/8a0cdd5a650c29d036dfec3000818b0e06ce3e0c/src/quart/config.py#L19
We would need to increase this value to at least 200mb by doing something like :
from flask import Flask, request

app = Flask(__name__)

app.config['MAX_CONTENT_LENGTH'] = 200 * 1024 * 1024

The assumption is right. I add this line:
app.config["MAX_CONTENT_LENGTH"] = 200 * 1024 * 1024  # 200MB

The system will be able to process the picture. However, tiff image is not displayed by the frontend relating to

https://github.com/ai-cfia/nachet-frontend/issues/89

related to #58


Keptn CLI completation activation fails (zsh)

When executing keptn completion zsh it fails with a lengthy error message.
Environment

Client OS (e.g., Linux, OSX, Windows): Mac OSX 11.2
Keptn Version (keptn version): Keptn CLI version (0.8.0-dev.202102091052) and Keptn cluster version (0.8.0-dev)
Kubernetes Cloud Provider (e.g., GKE, AKS): GKE
Kubernetes version (kubectl version): v1.17.14-gke.1600

Affected Component

[ ] Docs
[x] CLI
[ ] Kubernetes Integration
[ ] Openshift Integration
[ ] Helm
[ ] Istio
[ ] Bridge
[ ] Gatekeeper
[ ] Datastore
[ ] REST API
[ ] eventbroker / distributor
[ ] jmeter

Script output (excerpt):
./keptn completion zsh
* Warning: Your Keptn CLI version (0.8.0-dev.202102091052) and Keptn cluster version (0.8.0-dev) don't match. This can lead to problems. Please make sure to use the same versions.
#compdef _keptn keptn

function _keptn {
  local -a commands

  _arguments -C \
    '--mock[Disables communication to a Keptn endpoint]' \
    '(-n --namespace)'{-n,--namespace}'[Specify the namespace where Keptn should be installed, used and uninstalled in (default keptn).]:' \
    '(-q --quiet)'{-q,--quiet}'[Suppresses debug and info messages]' \
    '(-v --verbose)'{-v,--verbose}'[Enables verbose logging to print debug messages]' \
    "1: :->cmnds" \
    "*::arg:->args"

  case $state in
  cmnds)
    commands=(
      "add-resource:Adds a local resource to a service within your project in the specified stage"
      "auth:Authenticates the Keptn CLI against a Keptn installation"
      "completion:Generate completion script"
      "configure:Configures one of the specified parts of Keptn"
      "create:Creates a new project or service"
      "delete:Deletes a project"
      "generate:Generates the markdown CLI documentation or a support archive"
      "get:Displays an event or Keptn entities such as project, stage, or service"
      "help:Help about any command"
      "install:Installs Keptn on a Kubernetes cluster"
      "onboard:Creates a new service and uploads its Helm chart to the branches in the Git repository"
      "send:Sends an event to Keptn"
      "set:Sets flags of the CLI configuration"
      "status:Checks the status of the CLI"
      "uninstall:Uninstalls Keptn from a Kubernetes cluster"
      "update:Updates an existing Keptn project"
      "upgrade:Upgrades Keptn on a Kubernetes cluster"
      "version:Shows the version of Keptn and Keptn CLI"
    )
    _describe "command" commands
    ;;
  esac

  case "$words[1]" in
  add-resource)
    _keptn_add-resource
    ;;
  auth)
    _keptn_auth
    ;;
  completion)
    _keptn_completion
    ;;
  configure)
    _keptn_configure
    ;;
  create)
    _keptn_create
    ;;
  delete)
    _keptn_delete
    ;;
  generate)
    _keptn_generate
    ;;
  get)
    _keptn_get
    ;;
  help)
    _keptn_help
    ;;
  install)
    _keptn_install
    ;;
  onboard)
    _keptn_onboard
    ;;
  send)
    _keptn_send
    ;;
  set)
    _keptn_set
    ;;
  status)
    _keptn_status
    ;;
  uninstall)
    _keptn_uninstall
    ;;
  update)
    _keptn_update
    ;;
  upgrade)
    _keptn_upgrade
    ;;
  version)
    _keptn_version
    ;;
  esac
}

...

Command to be executed should actually be:
source <(keptn completion zsh)

Doing so, it does not result in this "error" message, but nevertheless, I couldn't get it to work.

Documentation also needs to be updated to explain this feature: https://github.com/keptn/keptn.github.io/issues/729

IMHO the feature is implemented similar to what Kubernetes and Helm offer, e.g.:
$ kubectl completion
error: Shell not specified.
See 'kubectl completion -h' for help and examples

$ kubectl completion -h
Output shell completion code for the specified shell (bash or zsh). The shell
code must be evaluated to provide interactive completion of kubectl commands.
This can be done by sourcing it from the .bash_profile.

 Detailed instructions on how to do this are available here:
https://kubernetes.io/docs/tasks/tools/install-kubectl/#enabling-shell-autocompletion

$ helm completion

Generate autocompletion scripts for Helm for the specified shell.

Usage:
  helm completion [command]

Available Commands:
  bash        generate autocompletion script for bash
  fish        generate autocompletion script for fish
  zsh         generate autocompletion script for zsh

Flags:
  -h, --help   help for completion

But I agree that some more information might be helpful, especially considering that running keptn completion SHELLNAME outputs a lot of code.
While this is not a bug, I would like to ask for the following improvement:

Docs for keptn completion available, as defined in https://github.com/keptn/keptn.github.io/issues/729
When running keptn completion (without -h), users should see a link to the docs

@christian-kreuzberger-dtx you are absolutely right, I had followed similar to what Helm and kubectl offer while implementing this feature.
@jetzlstorfer I believe you had missed the command to autoload the completion on startup, which can be done by executing keptn completion zsh > "${fpath[1]}/_keptn
At the moment the docs for the same can be accessed when you run the following command:
keptn completion -h
It will let you know how to enable completion depending on your shell and OS, like this:

Hence I think, once keptn/keptn.github.io#729 been resolved, we can then provide the link to docs when users run the command without the -h flag.