Greetings

Your repository conveniently has all the code I needed to register my program as a browser, so I decided to fork it and use it as my starting line.
Here is my fork, I made sure to properly credit you and I decided to stick with just the MIT licence.
I just wanted to ask if you have any issues of me doing so.
I have removed all the branding, only your name, a link to the original repo and your email are left.

PS: I am a beginner in the rust programming language, so please don't kill me for my bad code ;-;

@ArjixWasTaken Absolutely OK, I appreciate the crediting! It seems like something that might be easier to implement as a browser extension instead of a full browser replacement, but I'm happy to have my code used as a basis for your work. :)
Let me know if you have any questions about any of the code in bichrome!

@ArjixWasTaken Absolutely OK, I appreciate the crediting! It seems like something that might be easier to implement as a browser extension instead of a full browser replacement, but I'm happy to have my code used as a basis for your work. :)
Let me know if you have any questions about any of the code in bichrome!

Awesome!!
I assume you don't know what osu is, so I'll explain why I'm doing this and not a browser extension.
Basically, osu is a music game, and you can play beatmaps in it.
The game has multiplayer, and if you don't have a beatmap it launches the browser so that you can download it.
I wanted to make a browser proxy, if it is given a beatmap link then it will download the beatmap w/o the need for a browser.
If the link is not a beatmap, then it passes the link to a browser like normal.

Awesome!!
I assume you don't know what osu is, so I'll explain why I'm doing this and not a browser extension.
Basically, osu is a music game, and you can play beatmaps in it.
The game has multiplayer, and if you don't have a beatmap it launches the browser so that you can download it.
I wanted to make a browser proxy, if it is given a beatmap link then it will download the beatmap w/o the need for a browser.
If the link is not a beatmap, then it passes the link to a browser like normal.

Ah, that's reasonable! Looks like osu is open source, wouldn't it be easier to send a pull request to the game itself? :)

osu!lazer is open source, osu!stable is not
osu!lazer is going to replace osu!stable at some point

Also, my program is useless for osu!lazer, that is because it downloads them in-game

Aah, understandable workaround then!


New SLA Industries 2ed sheet

first attempt at a sheet, with dice roller on skills..
Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

confused by the error .. it is asking me to correct the errors, but none are listed

I have game specific information in the sheet, but i have permission from the game team -
nobble — 03/14/2021
Just to clarify Roll20 as a default automatically removes game system data, and does not allow it to be put into there character sheets on the platform.  To include this info it needs to be allowed by the game production team
Mark R — Yesterday at 4:12 PM
The game production company say yes please...go for it!

Greetings @seregrauko,
We no longer accept characters sheets with layouts built using table tags:, https://roll20.zendesk.com/hc/en-us/articles/360037773453. These elements will need to be replaced to meet the require programming standards.


import-sort via prettier

I've always found it awkward to run import-sort and prettier in the editor since chaining these two is somewhat error prone and can easily have both running at the same time.
So I wrote a prettier plugin https://github.com/ggascoigne/prettier-plugin-import-sort that allows import-sort to run as a preprocessor to prettier.  I'll admit that it's not a very sophisticated approach and could most likely be improved by trying to reuse the prettier parser, but I didn't have time for that.
Anyway, this led me to wonder if it's worth officially supporting a prettier-plugin in this repo, or whether it's better to keep it separate.

Hi @ggascoigne
that's pretty cool! I wasn't aware that prettier now has support for plugins. I will take a closer look to learn how it works.
Your plugin being part of this repository definitely makes sense to me; at the very least we should mention it in the README.

+1 bumping because it looks like this is outstanding.


Version 0.10 macro-style lib.rs breaks code completion in intellij

I was trying out the 0.10.0-alpha.2 release inside of my intellij IDE and none of the type information was coming through to the editor. I don't think intellij knows how to expand that if_hyper style macro.
Downgrading to 0.9 got me back in business.

Is it an older version of Intellij? The style of the if_hyper macro is commonly used in a lot of places, such as futures.

No, this is the latest plugin version. I know it works well with #[cfg(...)] style guards, but regular macro expansion is not as reliable.

Do you have the experimental macro expansion engine enabled? (Settings -> Languages & Frameworks -> Rust)
The old engine is bad at macros. With new engine I appear to be seeing correct type information with 0.10.0-alpha.2.

Good question, I did have the old expansion engine. I switched and it's better! I'll close this for now. Thanks!


setting target 'web' in renderer webpack config also make the preload script compile with 'web' target

Preflight Checklist

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Issue Details

Electron Forge Version:

6.0.0-beta.52

Electron Version:

v9.1.0

Operating System:

macOS 10.14.6

Last Known Working Electron Forge version::

--

Expected Behavior
preload script should use the target: 'electron-preload',
Actual Behavior
I am not sure if this is an expected behaviour or bug. I am porting my electron app to electron forge. In the app I have the nodeIntegration false and I expose some native features with a  preload script. So basically the renderer will target will be similar to a modern browser.
When I tried the current default webpack configuration, the compiling fails with errors like
Uncaught ReferenceError: require is not defined
    at Object.events (external "events":1)
    at __webpack_require__ (bootstrap:789)

with which I came to a conclusion that the webpack is compiling the renderer code similar to a node environment, which actually is not a node environment as node integration is false (refer this comment)
then I tried setting target to web in webpack.renderer.config.js
module.exports = {
  ...
  target: 'web',
  ...
}

that led me to this issue.
setting target in renderer config sets the same target for preload as well. when I dig into the codebase of Webpack plugin
this line 191 in this function is setting the same web target for preload as well.
https://github.com/electron-userland/electron-forge/blob/d025878664a662a2c35cd1e140444b36a9f8dabb/packages/plugin/webpack/src/WebpackConfig.ts#L169-L191
Is this an expected behaviour? I think it's better to set target: 'electron-preload', for preload in this case as well.
To Reproduce

I've explained the use case above. If a sample repo is required, I can create one.

Additional Information
N/A

@malept @MarshallOfSound any thoughts?

@vazra could we not take those changes in the forked plugin and make a PR for the main plugin?  Many more people would benefit from it.


populate 'type' field of metrics object

/containers/cna/metrics/0/other/type is mandatory, but somehow went missing in CVE-2023-41313.

we haven't seen this recently


parity-secp256k1 relies on a library that is no longer maintained. Should we switch to rust-secp256k1?

Updating crates.io index

error: no matching package named parity-secp256k1 found
location searched: registry crates-io
required by package near-crypto v0.14.0
... which satisfies dependency near-crypto = "^0.14" of package near-sdk v4.1.1
... which satisfies dependency near-sdk = "^4.1.1" of package hello_near v1.0.0 (/workspaces/hello-near)

Temporary workaround if one need to use it asap before the team solve this issue:
Add this to your cargo.toml at the end
[patch.crates-io]
parity-secp256k1 = {git = "https://github.com/paritytech/rust-secp256k1"}

It'll get the installation of the yanked package directly from github, hence installation proceeds without getting stuck.
For those with cargo.lock created, you could pass --locked to cargo build, one didn't tried but some say it'll work with yanked package. The above strategy more for new project, or if you changed cargo.toml in the meantime...

[package]
name = "contract"
version = "1.0.0"
authors = ["Near Inc<EMAIL_ADDRESS>edition = "2021"
[lib]
crate-type = ["cdylib"]
[dependencies]
near-sdk = "4.1.1"
uint = { version = "0.9.3", default-features = false }
[patch.crates-io]
parity-secp256k1 = {git = "https://github.com/paritytech/rust-secp256k1"}
[profile.release]
codegen-units = 1
opt-level = "z"
lto = true
debug = false
panic = "abort"
overflow-checks = true
[workspace]
members = []

Your suggestion is very effective, thank you


Add Clear Method

I'm sure this has been asked a million times, but why don't we have a Clear method on MemoryCache? I understand adding it to IMemoryCache would be problematic, as it would be a breaking change.
Thanks!

Can you elaborate on the purpose for which you want to clear the MemoryCache?
Originally, IMemoryCache was designed to be a shared cache between all aspnet components. As such, having a Clear method would be counterproductive since one component can remove the cache entries that was added by another component. However, we now recommend instantiating a separate MemoryCache instance for each component to better control memory usage and isolate the data stored. While we don't have a Clear method, we do have Compact: https://github.com/aspnet/Caching/blob/970e255413466eb05c83339bb10f57465db4f3eb/src/Microsoft.Extensions.Caching.Memory/MemoryCache.cs#L358-L369. This may be sufficient, depending on your use case.
I'll close the issue for now since I don't see any immediate action items. If there is any specific functionality you would like to propose please feel free to file another issue and include a sample use case. Thanks!


README.md: fix typos, clarify tnow's unit

Units of tnow is actually arbitrary—the algorithm only uses the ratio of tnow / t (where t is currently the third parameter in models), so if you start with model = defaultModel(0.5); predictRecall(model, 1.0), the units on 0.5 and 1 are entirely arbitrary and depend on your convention: one hour, one day, etc.
I tend to use hours for reasonable accuracy in floating point—there are 8760 hours in a year—and to encourage people to move away from the one day minimum increment in Anki, but Ebisu is agnostic to units.
Thank you for the very thorough editing! I now realize I must have lacked spellcheck when I wrote the bulk of this. If you update the pull request to omit (or clarify) the discussion of units, I'll be most happy to merge.

Ah, ok. Yes, I guessed hours based on one of your quiz apps.
How does this look?

Again, many many thanks for your generosity with your time and your hard work!


Support auth in RestTemplateFactory

Coverage decreased (-0.2%) to 45.21% when pulling f36d948c980637943cca658b4c3ffe304f95ecde on yiming187:rest_factory into fd08a71725fea558d92231ab6b36891fb5d34f6b on ctripcorp:master.


contributing.md missing

Describe the bug
The contributing.md file mentioned in the readme.md is missing
Expected behaviour
If clicked on the contributing.md guidelines should appear
Additional context
Readme.md: https://github.com/Ananya-0306/Cybersecurity/blob/main/README.md

Hi Hritik Roy, Thankyou for updating, now i added the file please have a
look https://github.com/Ananya-0306/Cybersecurity/blob/main/CONTRIBUTING.md
On Sat, Apr 17, 2021 at 12:32 PM Hrittik Roy @.***>
wrote:

Describe the bug
The contributing.md file mentioned in the readme.md is missing
Expected behaviour
If clicked on the contributing.md guidelines should appear
Additional context
Readme.md:
https://github.com/Ananya-0306/Cybersecurity/blob/main/README.md
[image: image]
https://user-images.githubusercontent.com/67012359/115104854-de55f800-9f78-11eb-801e-9b479d39dbbf.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Ananya-0306/Cybersecurity/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARWJ5CUEKRETGBRES5S77ULTJEXAPANCNFSM43CYUGTQ
.


OkHttp library default Idle connection timeout

Hi,
What is the default idle connection timeout value when using OkHttp library ? Is this configurable ?
Note:  I had opened a thread earlier , and then cloudant had a server side timeout of 2.5 minutes, but based on our recent communication with cloudant support , we found that they have higher limits now , upto 1 hr but subject to change.
Thanks.

Hi @sagardiwan, we set a client timeout of 2.5 minutes on our load balancers. The 60 minute timeout that Cloudant Support were referring to must have been the server timeout. This is for requests between the load balancer and backend database nodes.
The idle timeout when using OkHttp is set to 3 minutes here. Bear in mind that this is not currently configurable. This was chosen in order to keep the connections open for as long as possible (noting that they will be terminated by the server at 2.5 minutes).
We'll do some further investigating our side to ensure the client isn't failing to detect closed connections (or similar) and will be in touch shortly with our findings.

Thanks @smithsz
We are using netscaler LB , and have set client timeout = 3 mins and server side timeout = 6mins
With those settings , our applications sees frequent connections resets , almost  8-10 times an hour.

I’ve run through some test scenarios and with respect to connection lifetimes in the pool the OkHttp library appears to detect connections closed by the Cloudant LBs without any problem, successfully re-opening new connections when required. See also my comment on another ConnectionReset issue. I don’t know about how NetScaler handles connections or keep-alive, but you may want to try and identify where the TCP RST is coming from as that may make it clear what is happening. It appears that if you can capture the packet there are some codes available.
Given that it’s possible this isn’t caused by pooling at all I think identifying the RST is your best course of action. Failing that an alternative may be to allow the client to take control of closing the connections to NetScaler. As discussed the client will close idle connections after 3 minutes so if you set your NetScaler client side keep-alive to > 3 min then the client might be able to purge them first. You may also want to set your NetScaler server side timeout to < 2.5 min so that it can close the connection with Cloudant before Cloudant does.
How many connections are you trying to maintain between client and NetScaler (i.e. ClientBuilder#maxConnections()) and how many CloudantClient instances do you have? How many connections are you maintaining between NetScaler and your Cloudant account(s)?

Closing as stale, please re-open if you provide answers to the above questions.


As a developer, I want to constrain the length of the cheeps to fit the requirements of the application.

Text from lecture:
Constrain cheeps so that only cheeps of length 160 characters are considered valid. Your Chirp! application should not accept cheeps that are longer that 160 character. Such cheeps should never be stored to the database.
Using the mechanisms presented in chapter 18 of the course book Andrew Lock ASP.NET Core in Action, validate that only cheeps up to 160 characters length can be sent.
Acceptance criteria:

Cheeps can only be 160 characters long.
There should be a relevant error message if cheep gets too long, or the user should be prompted when reaching the maximum characters.

We restricted the length of cheeps, so it isn't possible to write more than 160 characters. There isn't any error message and we aren't going to implement that.


Credit to Michael Jackson?

I see the initial code is taken from his codepen, should we include that in the Readme?

PR welcome

https://github.com/stevemao/react-classic-calculator/pull/2

Thanks for your contribution. You are now promoted to be a collaborator :)

what about we create a new org named so-classic and we implement all classic project under that name? 😄 .. https://github.com/so-classic

maybe later implement,  angular-classic-calculator, vue-classic-calculator...bla,bla,bla, anything that is classic... ✨


Deserialize<'_> is not implemented for xdg::ServerInformation

https://github.com/hoodie/notify-rust/blob/c253a82f53c24ad29d4aeecec84952e53a886d52/src/xdg/zbus_rs.rs#L217
This line makes the compiler unhappy if I use notify-rust with default-features = false, features = ["zbus"].
You may want to check the feature gates on this particular code piece :wink:

Full error:
    Checking notify-rust v4.11.0
error[E0277]: the trait bound `xdg::ServerInformation: DynamicDeserialize<'_>` is not satisfied
   --> /home/m/.cargo/registry/src/index.crates.io-6f17d22bba15001f/notify-rust-4.11.0/src/xdg/zbus_rs.rs:217:10
    |
217 |         .deserialize()?;
    |          ^^^^^^^^^^^ the trait `Deserialize<'_>` is not implemented for `xdg::ServerInformation`, which is required by `xdg::ServerInformation: DynamicDeserialize<'_>`
    |
    = help: the following other types implement trait `DynamicDeserialize<'de>`:
              <DynamicTuple<(T0, T1)> as DynamicDeserialize<'de>>
              <DynamicTuple<(T0, T1, T2)> as DynamicDeserialize<'de>>
              <DynamicTuple<(T0, T1, T2, T3)> as DynamicDeserialize<'de>>
              <DynamicTuple<(T0, T1, T2, T3, T4)> as DynamicDeserialize<'de>>
              <DynamicTuple<(T0, T1, T2, T3, T4, T5)> as DynamicDeserialize<'de>>
              <DynamicTuple<(T0, T1, T2, T3, T4, T5, T6)> as DynamicDeserialize<'de>>
              <DynamicTuple<(T0, T1, T2, T3, T4, T5, T6, T7)> as DynamicDeserialize<'de>>
              <DynamicTuple<(T0, T1, T2, T3, T4, T5, T6, T7, T8)> as DynamicDeserialize<'de>>
            and 11 others
    = note: required for `xdg::ServerInformation` to implement `DynamicDeserialize<'_>`
note: required by a bound in `Body::deserialize`
   --> /home/m/.cargo/registry/src/index.crates.io-6f17d22bba15001f/zbus-4.3.1/src/message/body.rs:25:12
    |
23  |     pub fn deserialize<'s, B>(&'s self) -> Result<B>
    |            ----------- required by a bound in this associated function
24  |     where
25  |         B: zvariant::DynamicDeserialize<'s>,
    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ required by this bound in `Body::deserialize`

Thanks for letting me know, I was a bit confused how this was possible until I looked at the combination of things.
The feature flag for using zbus is "z" not "zbus". I think I could technically just change that. Or document that better. Not sure. Any suggestion.

Well for me it was unclear that z is the feature flag I need. You could add a compiler error if a user only enables zbus but not serde, but I am not sure whether this is actually an error... can the crate work without serde when `zbus is used? If not, why have it as seperate features?


[BUG]: OPT 6.7B inference is extremely slower than FasterTransformer

🐛 Describe the bug
Describe the bug
We run the OPT-6.7B inference using opt_fastapi.py, and found it extremely slow, say, 9 seconds per request on two A800 (with tensor parallel = 2) under batch size 1, 2 or 4.
And it became ever worse when batch size is 8 or higher, in which scenario, the VMEM reached 80GB and then hung forever.
We performed a comparison using FasterTransformer, and it can handle batch size of 96, at a speed around 1.7s per item.
To Reproduce
Download OPT-6.7B and run EnergonAI/example/opt/opt_fastapi.py.
Expected behavior
Comparable or better speed as advertised in your documentation.
Screenshots
N/A
Optional: Affiliation
N/A
Environment
CUDA 11.6
PyTorch 1.13
Python 3.8

@tthn we are actively developing inference engine. Please stay tuned.


Adding Banxa Bridge

Overview

A simple integration of Banxa, where you can click and be redirected to the Banxa site to purchase BOBA network on our network (I'm still working on the 100% integrated version in the app).
Refactoring of most functions/connectors.
Display fixes, especially on large screens, horizontal scrolling, and alignment issues in general.

Changes

Codecov Report
Patch and project coverage have no change.

Comparison is base (13b9b0c) 53.68% compared to head (1a9681d) 53.68%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #666   +/-   ##
========================================
  Coverage    53.68%   53.68%
========================================
  Files          156      156
  Lines         6454     6454
  Branches      1272     1272
========================================
  Hits          3465     3465
  Misses        2989     2989

Flag
Coverage Δ

coverage
53.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@InoMurko Currently, we are using the referral method for the bridge. I am working on a version that is 100% integrated into the current UI.
In the meantime, the current version is sending parameters such as:
defaults: {
fiat: 'usd',
token: 'boba',
network: 'boba network',
amount: '$35'
}
For the amount case, there are two options, either pass the minimum allowed by default, which in this case is $35, or simply leave it without a reference value.
In the case of Bancor, https://app.bancor.network/fiat, they don't even offer a default token or a reference value, leaving it 100% up to the user's decision. As for Arbitrum, we know that it offers Eth by default on the Arbitrum network, and the minimum amount is $100.


Add 'get' functions to Registry

This adds public functions get_counter, get_gauge, get_histogram, that return a clone of the requested metric stored in the internal Storage.
This allows retrieving a copy of a single metric without creating a full HashMap clone.

Released in<EMAIL_ADDRESS>Thanks again for your contribution!


Added NS view extension showcase app

I would recommend the following changes

Add some additional info in the readme to clarify that the Current Value is a normal MDK Simple Property field and only the slider is the NS Extension.
The Images/Slider.png should be updated to remove the Simple Property from the image.


[vtflib] Add new port

Describe the pull request Add the VTFLib package https://github.com/NeilJed/VTFLib

What does your PR fix? Fixes #

Which triplets are supported/not supported? Have you updated the CI baseline?

Does your PR follow the maintainer guide?

@dhkatz, thanks for the PR!
The following triplets failed on CI testing, could you help have a look? is this expected?
vtflib:x64-windows-static
vtflib:x64-uwp
vtflib:arm-uwp
vtflib:x64-osx
vtflib:x64-linux

@dhkatz, is work still being done for this PR?

Closing this PR since it seems that no progress is being made. Please reopen if work is still being done.


Make SubViewportContainer's adjustments optional

This PR improves documentation about properties modified by SubViewportContainer and makes this behavior optional by adding adjust_viewport_properties property.
Closes #56502

I would argue it probably would be better to simply the behavior by ensuring Godot is not changing the value of update_mode every time the visibility is changing.
It should either:

simply  disable the processing using internal flag which is not exposed to gdscript
or don't do anything since this can be achieved very easily manually in gdscript (and that's how I would expect it to work before this pr without looking at the implementation)

simply disable the processing using internal flag which is not exposed to gdscript

Update mode is handled on RenderingServer side. It can't be an internal flag.

Well from user perspective internal flag makes it behave exactly like they would expect (disabling processing when SubViewportContainer is hidden), while respecting update_mode (in current behavior exposing update_mode has no sense since no matter what programmer will set there, engine will always overwrite it). This way we would not need to introduce new a bit cryptic adjust_viewport_properties flag and the api would be cleaner

Rebased. I also tweaked _validate_property() to disable the properties handled by the container.


Improving the readme

Hi,
Interesting component 🙂
Here a template for your readme: iOS-readme-template

Hi, not sure exactly which part you're referencing but I do have plans to add support for package managers. Going to close this out for now, feel free to re-open if you had specific suggestions in mind.


Problem with module

Hi
I have a problem using video2bvh, when I try to run demo.ipynb I have this error in the first cell :
ModuleNotFoundError                       Traceback (most recent call last)
 in
----> 1 from pose_estimator_2d import openpose_estimator
2 from pose_estimator_3d import estimator_3d
3 from utils import smooth, vis, camera
4 from bvh_skeleton import openpose_skeleton, h36m_skeleton, cmu_skeleton
5
~/video2bvh/pose_estimator_2d/openpose_estimator.py in
1 from .estimator_2d import Estimator2D
----> 2 from openpose import pyopenpose as op
3
4
5 class OpenPoseEstimator(Estimator2D):
ModuleNotFoundError: No module named 'openpose'
But Openpose is installed and working, I think I forget to do something, do I have to specify where is located my Openpose folder ?
Thanks for your work

Hi,
I still have difficulties to make video2bvh works as I mentionned above, if anyone can help me, it would be great, I'm not expert with Jupyter Notebook
Thanks again !

Sorry, I've never encountered this problem before. Maybe you can ask OpenPose for help.

Thank you for answering my question, I'll ask them but I'm not sure they can help me about a project they didn't create like video2bvh
But can you just tell me if my structure should be able to work (see the screenshot), both folders "openpose" and "video2bvh" are in the root when I launch Jupyter Notebook

Did you install the openpose python module?

Yes, I think that's my problem, I can't see this module, I try to install it but I'm facing issues
thanks for your answer

I think the problem is because I'm on Mac OS, and the path to this module is not the same (on MAc OS /usr/local/python), but I don't see where I can modify it in your file.

See https://github.com/CMU-Perceptual-Computing-Lab/openpose/blob/master/doc/installation.md#python-api for help.
The only thing you need to specify in this project about OpenPose is the OpenPose model folder path.
e2d = openpose_estimator.OpenPoseEstimator(model_folder='/openpose/models/')

I finally solved my problem by adding these lines in the file pose_estimator_2d.py :
import sys sys.path.append('/usr/local/python');
Maybe it can help some MacOs users
Now I have a problem with Pytorch installation (during "Estimate 3D pose from 2D pose" step):
---> 47         raise AssertionError("Torch not compiled with CUDA enabled")
AssertionError: Torch not compiled with CUDA enabled
I think I need to re-install it


bug(checkbox): A small dot in the middle of an inactive radio button

Reproduction
Steps to reproduce:

https://material.angular.io/components/checkbox/examples

Expected Behavior
The unselected radio button should be without a small dot in the center.
Actual Behavior

Environment

Browser: Firefox 86.0 (64 bit)
Operating System:  Windows 10

I wasn't able to reproduce it on Firefox 86 and Windows 10. What is your system-level zoom?

125% (recommended)
When I set it to 100% the problem disappeared

Thanks, I'm able to reproduce it now.


TOC refers to absent anchor tags

The TOC at the top of the site assumes anchor tags on each entry, but they are not present.
For example:
https://functional.works-hub.com/blog/Functional-Programming-Jargon#partial-application

That site is not maintained by us. They're running some kind of fork of the readme and improperly compiling the markdown. The link for partial application on github seems to work fine so I don't think there's anything to fix on our end


Feature request: have active-class behave like class on router-link

What problem is this solving
Currently, the router-linke active-class attribute accepts only a string as accepted value.
e.g.:
  <router-link active-class="bg-teal-500">...</router-link>

There are some cases however where we have reusable components that can have a different color.  We would like to be able to do the following:
<router-link
  :active-class="{
    'bg-primary-800': color === 'primary',
    'bg-fuchsia-800': color === 'fuchsia',
    'bg-green-800': color === 'green',
    'bg-yellow-800': color === 'yellow',
    'bg-red-800': color === 'red',
    'bg-teal-800': color === 'teal',
    'bg-purple-800': color === 'purple',
  }"

similar to how the class property accepts these objects.
Proposed solution
Apply same behaviour / support for class to the router link's active-class
Describe alternatives you've considered
No response

Since you can build such string with a computed property, there is no need to add an extra abstraction layer for this. You can also create your own custom router link extension: https://router.vuejs.org/guide/advanced/extending-router-link.html#Extending-RouterLink

Hi @posva, I agree it can be build with a computed property, but something like tailwind won't like it as it won't be included in the tree shaking algorhitm.  Also, it would just align both property's behaviour.
Are you saying you won't accept a PR for this, or that you are open for it but don't have time for it at this time?  In the first case, I might invest some time - otherwise, I'll just customise on our end.

I'm not planning on adding this feature. FYI if I close as not planned I mean it that way.
Last time I tried, tailwind will still pick these classes from the js/ts part in vue files if they are included. In any case, it's usually more convenient to have a wrapper like mentioned in docs. Cheers.


Add manpage for scrcpy

I don't know if I put the file in the right directory, if not please tell me.

Thank you :+1:
I suggest the following changes:
diff --git a/app/scrcpy.1 b/app/scrcpy.1
index ddf2649..fda53e5 100644
--- a/app/scrcpy.1
+++ b/app/scrcpy.1
@@ -10,7 +10,7 @@ scrcpy \- Display and control your Android device

 .SH DESCRIPTION
 .B scrcpy
-provides display and control of Android devices connected on USB (or over TCP/IP). It does not require any root access. It works on GNU/Linux, Windows and MacOS.
+provides display and control of Android devices connected on USB (or over TCP/IP). It does not require any root access.

 .SH OPTIONS
@@ -34,7 +34,7 @@ value is computed on the cropped size.
 Start in fullscreen.

 .TP
-.B \-F, \-\-record\-format
+.BI "\-F, \-\-record\-format " format
 Force recording format (either mp4 or mkv).

 .TP
@@ -68,7 +68,7 @@ Set the target directory for pushing files to the device by drag & drop. It is p
 Default is "/sdcard/".

 .TP
-.BI "\-r, \-\-record " file.mp4
+.BI "\-r, \-\-record " file
 Record screen to file.

 The format is determined by the
@@ -106,7 +106,7 @@ Print the version of scrcpy.
 Set a custom window title.

-.SH Shortcuts
+.SH SHORTCUTS

 .TP
 .B Ctrl+f
@@ -196,16 +196,23 @@ Specify the path to adb.
 Specify the path to server binary.

-.SH AUTHOR
+.SH AUTHORS
 .B scrcpy
-is written by
-.UR https://www.genymobile.com
-Genymobile
-.UE
-and Romain Vimont.
-
-This manual page was written by
-.MT mmyangfl@\:gmail.com
-Yangfl
-.ME
-for the Debian Project (and may be used by others).
+is written by Romain Vimont.
+
+This manual page was written by Yangfl<EMAIL_ADDRESS>for the Debian
+Project (and may be used by others).
+
+
+.SH "REPORTING BUGS"
+Report bugs to <https://github.com/Genymobile/scrcpy/issues>.
+
+.SH COPYRIGHT
+Copyright \(co 2018 Genymobile <https://www.genymobile.com>
+.br
+Copyright \(co 2018\-2019 Romain Vimont<EMAIL_ADDRESS>+
+Licensed under the Apache License, Version 2.0.
+
+.SH WWW
+<https://github.com/Genymobile/scrcpy>

Is it ok for you?

Fine, but what's wrong with MT/ME and UR/UE macros?

what's wrong with MT/ME and UR/UE macros?

The main reason is that they use () instead of <> for URLs. But if it's better (I have no experience in writing manpages), I can use them:
diff --git a/app/scrcpy.1 b/app/scrcpy.1
index fda53e5..34b8168 100644
--- a/app/scrcpy.1
+++ b/app/scrcpy.1
@@ -200,19 +200,31 @@ Specify the path to server binary.
 .B scrcpy
 is written by Romain Vimont.

-This manual page was written by Yangfl<EMAIL_ADDRESS>for the Debian
-Project (and may be used by others).
+This manual page was written by
+.MT<EMAIL_ADDRESS>+Yangfl
+.ME
+for the Debian Project (and may be used by others).

 .SH "REPORTING BUGS"
-Report bugs to <https://github.com/Genymobile/scrcpy/issues>.
+Report bugs to
+.UR https://github.com/Genymobile/scrcpy/issues
+.UE

 .SH COPYRIGHT
-Copyright \(co 2018 Genymobile <https://www.genymobile.com>
+Copyright \(co 2018 Genymobile
+.UR https://www.genymobile.com
+Genymobile
+.UE
 .br
-Copyright \(co 2018\-2019 Romain Vimont<EMAIL_ADDRESS>+Copyright \(co 2018\-2019
+.MT<EMAIL_ADDRESS>+Romain Vimont
+.ME

 Licensed under the Apache License, Version 2.0.

 .SH WWW
-<https://github.com/Genymobile/scrcpy>
+.UR https://github.com/Genymobile/scrcpy
+.UE

What do you think?
Btw, how to add a . at the end of Report bugs to (https://github.com/Genymobile/scrcpy/issues)?  In the source file, it would need to appear alone on the line, but I can't fine a way to make it work (., (space)., \.…)

The main reason is that they use () instead of <> for URLs.

On my system (Debian) <> are also used for URLs. And a strong reason to use macros is they help rendering better pages in other formats, like HTML. (see linux.die.net)

Btw, how to add a . at the end of Report bugs to (https://github.com/Genymobile/scrcpy/issues)? In the source file, it would need to appear alone on the line, but I can't fine a way to make it work (., (space)., \.…)

.UE .
.ME .


Test compatibility with Photon OS 2.0

Based on the results of the investigation, we need to update our customer facing documentation.
Useful links -

https://blogs.vmware.com/cloudnative/2017/11/01/version-2-0-project-photon-os/
https://www.virtuallyghetto.com/2017/11/workarounds-for-deploying-photonos-2-0-on-vsphere-fusion-workstation.html

Ran all the end-to-end for vDVS against Photon 2.0 and all tests finished successfully. No issues were found while running theses tests with Photon 2.0.
Got the Photon 2.0 from the link - https://github.com/vmware/photon/wiki/Downloading-Photon-OS


Fix safari fab rendering issue

Fixes a couple of weird Safari only rendering issues with the FAB button:

On the list view the FAB was displaying in a random position on the page
On the map it was disappearing when the map loaded

In both cases, the button was still clickable at the original position, so it's only the browser rendering the visual part of the button in a different place 🥴

:tada: This PR is included in version 1.15.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Angular 12

Update to angular 12

You should have a look at "ngx-pinch-zoom/projects/ngx-pinch-zoom/package.json". There are two peerDependencies that must be updated too in order to do a clean update to angular 12.

https://github.com/CoderpillarNest/ngx-pinch-zoom/pull/1

Thank you. I will be posting an update for the strict mode of the Typescript soon.

Will the code also be updated to support Angular 12? But it will not work with this PR and @lozenzero also did not react to my PR on top of his changes. So maybe @drozhzhin-n-e you could commit own changes to support angular 12?

Hi, any update on this? Or angular 13?

@drozhzhin-n-e @considine still no update on this?


Update 03.02.data-management.md - Adding references

DOI PHIS publication & MIAPPE publication

@BrapiCoordinatorSelby , thanks for accepting this pull request, but I'm not sur I did properly. I wanted to add 2 references that will be referenced at the end of the article, but it doesn't seem to work. Could you tell me how to do it right? Thanks

@Isabelle-inrae, citation format can be found here. For your citations, they'd be formatted as [@doi:https://doi.org/10.1111/nph.15385]
@BrapiCoordinatorSelby or I will go through and ensure they're all formatted correctly during the editing process.

Ok Thanks a lot, I realize I forgot the "@" so I will push another pull request


Bump up version to v1.1.2

Issue #, if available:
Description of changes: Bump up version
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looks good, waiting for checks to complete.


Authorization Error

Hello,
When copying data to the b2fuse directory that is linked to b2 cloud storage I am getting permission denied:

Please let me know what you recommend.
Thanks!

b2fs4chia does not support writing of any kind. It's read-only. You should use b2 cli to upload your data, then mount b2fs4chia and add the path to the farmer.

Got it.  Thanks!
On Mon, Aug 16, 2021, 4:29 PM Paweł Polewicz @.***>
wrote:

b2fs4chia does not support writing of any kind. It's read-only. You should
use b2 cli [github.com]
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Backblaze_B2-5FCommand-5FLine-5FTool&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=IHQ5OnfCk2C8EMEhG1qk8vflTWCQAbYuxbH2EXQUtVo&m=7Li7ueS-jhwvR9yBHWnCgIsr0-lgU7uRDXQyMGv-4q0&s=MBhgTaQZAqUuuJXb3JJD_TGGwo9xXCPd2gJfLIZVbZc&e=
to upload your data, then mount b2fs4chia and add the path to the farmer.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [github.com]
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Backblaze-2DB2-2DSamples_b2fs4chia_issues_20-23issuecomment-2D899798498&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=IHQ5OnfCk2C8EMEhG1qk8vflTWCQAbYuxbH2EXQUtVo&m=7Li7ueS-jhwvR9yBHWnCgIsr0-lgU7uRDXQyMGv-4q0&s=GtZZ3QBvON74YCgNZki70_5yqG8C7e-2lK5q0fEIPnY&e=,
or unsubscribe [github.com]
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AGYVAFHUJZ6Q2FG2Y3E3XLTT5FYL3ANCNFSM5CIN6DTQ&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=IHQ5OnfCk2C8EMEhG1qk8vflTWCQAbYuxbH2EXQUtVo&m=7Li7ueS-jhwvR9yBHWnCgIsr0-lgU7uRDXQyMGv-4q0&s=qm3Ci30ORAQgDhLFduEC7lHO5hcUcZaFe1b4ZJpkVxw&e=
.
Triage notifications on the go with GitHub Mobile for iOS [apps.apple.com]
https://urldefense.proofpoint.com/v2/url?u=https-3A__apps.apple.com_app_apple-2Dstore_id1477376905-3Fct-3Dnotification-2Demail-26mt-3D8-26pt-3D524675&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=IHQ5OnfCk2C8EMEhG1qk8vflTWCQAbYuxbH2EXQUtVo&m=7Li7ueS-jhwvR9yBHWnCgIsr0-lgU7uRDXQyMGv-4q0&s=yopHAdripGZ1CR09Fm8uvNaHjxdS2FSLejTCfLp72e0&e=
or Android [play.google.com]
https://urldefense.proofpoint.com/v2/url?u=https-3A__play.google.com_store_apps_details-3Fid-3Dcom.github.android-26utm-5Fcampaign-3Dnotification-2Demail&d=DwMCaQ&c=s883GpUCOChKOHiocYtGcg&r=IHQ5OnfCk2C8EMEhG1qk8vflTWCQAbYuxbH2EXQUtVo&m=7Li7ueS-jhwvR9yBHWnCgIsr0-lgU7uRDXQyMGv-4q0&s=XvdqDefnxL3d1Tb6YNHtM1G6A1dfQ5QiGbv1Wsa2j0M&e=
.


safetensors!

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/runtime/ops_disk.py              10               6
------------------------------------------------------------
lines added in the tinygrad folder: 4

Played around with this for a bit here: https://github.com/wozeparrot/tinygrad/tree/bert_safetensors
from models.bert import BertForPreTraining
from tinygrad import state
from extra.utils import get_child

# convert to safetensors
mdl = BertForPreTraining()
mdl.load_from_pretrained()
state.safe_save(state.get_state_dict(mdl), "/tmp/bert_for_pt.safetensors")

del mdl

# try to load safetensors
mdl = BertForPreTraining()
state_dict = state.safe_load("/tmp/bert_for_pt.safetensors")
for k, v in state_dict.items():
  get_child(mdl, k).assign(v.numpy()).realize()

Seems to work.
Should probably also move get_child to state as well tho, because it is needed to load from the state_dict.

I don't like get_child. Try get_state_dict on mdl and assigning to those tensors.

Ok, yea that seems to work


Add downsampling modal for tasks

as a user
with admin
when i view my list of buckets
then i have the ability to create a rollup task from the data in that bucket in the ui
User would select the following options:

Time range: 5m, 1d, 1w, etc
Delay: 10s, 1m (to account for late data)
New bucket name and org: default to bucket_name_timerange in the same org
Aggregate Functions Multi Select Dropdown - User should be able to pick things like min, max, mean, last, etc

If the bucket doesn't exist, create it.
We would then create a task that would execute with those parameters.

https://xd.adobe.com/view/94aa5559-cfea-4880-6b3b-f31e032b9953-e29b/?fullscreen

+1
On Thu, Jul 25, 2019 at 5:10 PM stale[bot]<EMAIL_ADDRESS>wrote:

This issue has been automatically marked as stale because it has not had
recent activity. It will be closed if no further activity occurs. Thank you
for your contributions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/influxdata/influxdb/issues/12768?email_source=notifications&email_token=ABEVK3JSDUS37DMB32APGXLQBI6HDA5CNFSM4G7WY4JKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD23DUHI#issuecomment-515258909,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEVK3JBW3HZLFLKHWKEIITQBI6HDANCNFSM4G7WY4JA
.

--
Russ Savage
Product Manager • InfluxData
m<PHONE_NUMBER>     t @russellsavage     w influxdata.com
https://www.influxdata.com/


Roll Engine from 345a04b11f96 to 44f21fcb04d4 (2 revisions)

https://github.com/flutter/engine/compare/345a04b11f96...44f21fcb04d4
2022-02-17<EMAIL_ADDRESS>Roll Skia from e6fba0a50b47 to 5505a02ca1fb (1 revision) (flutter/engine#31512)
2022-02-17<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from ciPTp_Hy7... to pbHlKwXzl... (flutter/engine#31509)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from ciPTp_Hy7XM6 to pbHlKwXzl8WB
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


OSD: Fixed Pagination Alignment

Fixes #2497
Describe the changes you have made in this PR -
Update layout.scss to align pagination correctly
Screenshots of the changes (If any) -

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

@gr455

@Kriti-bit I guess the issue description means pagination should be directly below the above "search project" line.


upgrading to 0.14 on windows 10 fails at "react-native upgrade"

Been using react-native 0.13.* in windows 10 for 2-3 weeks I encountered this when trying to follow the upgrade-instrucions to upgrade to 0.14.0.
Steps in a working 0.13.0 project:

edit package.json to entering react-native 0.14.0 as a dep
running "npm install"
Running "react-native upgrade" gives me this:

'/**' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
child_process.js:464
    throw err;
    ^

Error: Command failed: C:\Users\ippa\Documents\Projects\contacts\node_modules\react-native\local-cli\setup_env.bat
'/**' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.
'*' is not recognized as an internal or external command,
operable program or batch file.

    at checkExecSyncError (child_process.js:441:13)
    at Object.execFileSync (child_process.js:461:13)
    at Object.run (C:\Users\ippa\Documents\Projects\contacts\node_modules\react-native\local-cli\cli.js:59:16)
    at Object.<anonymous> (C:\Users\ippa\AppData\Roaming\npm\node_modules\react-native-cli\index.js:73:7)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:311:12)
    at Function.Module.runMain (module.js:467:10)
    at startup (node.js:134:18)

I also tried nuking "node_nodules"-directory and running "npm install" before "react-native upgrade" with the same results.

Digging a bit further,  setup_env.bat contains only this:
/**
 * Copyright (c) 2015-present, Facebook, Inc.
 * All rights reserved.
 *
 * This source code is licensed under the BSD-style license found in the
 * LICENSE file in the root directory of this source tree. An additional grant
 * of patent rights can be found in the PATENTS file in the same directory.
 */

and windows gets confused trying to execute that.

@facebook-github-bot answered


How to install custom services in an image and then encapsulate them into an ISO image?

Can I use this script to install some services in the image?
Can you provide an example if possible?
Thanks.

You can place your script file into the iso_root/ directory and copy it into the /target directory as well as the iso_root/ubuntu.sudoers file.
I think that the curtin in-target command might be useful for customizing your installation.

https://canonical-subiquity.readthedocs-hosted.com/en/latest/reference/autoinstall-reference.html#late-commands

Regards,

Thank you, resolved.


bzlmod - how to use oci_pull?

I don't see any examples for how to use oci_pull with bzlmod. Is this not supported?
Typically it's an extension function (e.g. with rules_jvm_external there's a "maven" function with a maven.install method), but I couldn't see that being supported in rules_oci.

This came up yesterday in the "bzlmod" channel of the "Bazel" Slack workspace. There, I discovered that if I want to use the oci_pull rule, I need to define the corresponding targets in my WORKSPACE.bzlmod file, which means that I then also need to declare Bazel module dependencies in my MODULE.bazel file for any repositories loaded by rules_oci's oci_pull (e.g. aspect_bazel_lib).
That got me thinking that we'd benefit from a module extension function that would allow pulling OCI images from the MODULE.bazel file, which would then remove the need to use a WORKSPACE.bzlmod file. Are there any plans for such a module extension?
@alexeagle, I expect that you'll know best.

Fixed in #118

Example: https://github.com/aspect-build/bazel-examples/blob/main/oci_go_image/MODULE.bazel#L20-L28


Chat only works for first question

Chat only works for the first question and ends after first response. Voice is working as charm without any issues.

See - https://github.com/aws-samples/amazon-connect-contact-surveys/issues/14#issuecomment-2065479824
Please indicate what language is used in your Amazon Connect Flow when you invoke the survey module. Troubleshooting should point out that the Lex locale used is en_US. If you use are using a different locale in Amazon Connect, the bot will fail to match it.
See: https://docs.aws.amazon.com/connect/latest/adminguide/set-voice.html#set-voice-lexv2bot


Add dotenv/load shortcut file

So it can be used as:
require "dotenv/load"

as a shortcut for:
require "dotenv"
Dotenv.load

In similar fashion to Bundler's bundler/setup which requires bundler then calls Bundler.setup.

Thanks!


move DEFINE_VAR min/max from src to arg

new arg is (Variable, min as CONST, max as CONST). not exactly sure why some PTX kernels became simpler...

something to do with cmp_tuple and the use of DEFINE_VAR there. merging given that other CI + benchmarks kernels remained unchanged and PTX became simpler


Feature: Add support for KeyVault for AWS (AWSKMS)

I would suggest using the Signer architecture in https://github.com/secure-systems-lab/securesystemslib/pull/456 if it does get merged.
Also for reference, I have a initial GCP KMS implementation on the way https://github.com/secure-systems-lab/securesystemslib/pull/442 maybe useful for ideas.
The one thing that's not handled in those two is KMS key creation/import which I assume you'd like to have -- but I have it in my head already, let me know if you need it somewhere else :)

FYI: we have a pending AWS signer implementation in https://github.com/secure-systems-lab/securesystemslib/pull/609 (h/t @ianhundere).

We need to split this issue into two.

Storage Backend service AWS S3
Key Vault service AWS KMS

FYI: we have a pending AWS signer implementation in secure-systems-lab/securesystemslib#609 (h/t @ianhundere).

The above PR is merged 🚀
We can use in RSTUF the localstack in the development environment to create the tests

Task for implementation

[ ] https://github.com/repository-service-tuf/repository-service-tuf-worker/issues/344


Naming guidelines not available to all users. fixing #119

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

Project coverage is 87.79%. Comparing base (4507a06) to head (2b04778).
Report is 38 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #125      +/-   ##
===========================================
+ Coverage    87.76%   87.79%   +0.03%
===========================================
  Files           10       10
  Lines          793      795       +2
===========================================
+ Hits           696      698       +2
  Misses          97       97

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


Update aws-java-sdk-core to 1.12.773

About this PR
📦 Updates com.amazonaws:aws-java-sdk-core from 1.12.459 to 1.12.773
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #815.


No icons in terminal with brew installed emacs in MacOS 12.0.1 Darwin 21.1.0 arm6

What did you expect to happen?
I should see icons in the dashboard when I open emacs
What actually happened?
There is no icon showing in the dashboard when I open emacs
Describe your attempts to resolve the issue
No response
Steps to reproduce

brew install emacs
install doom as in the installation guide
open emacs in terminal

System Information
https://pastebin.com/raw/QPBV7Udc

That's because icons are not supported in the terminal, and there is no plan to change that (it's non-trivial to support, as too many packages depend on all-the-icons and offer no easy way to substitute them).


Square Instead of Circle (not an issue)

Yo this is just a variation of the code, but if anyone wanted to, they could change the step increment to 90. This would draw a square instead of a circle, but it works just the same. I learned from this Code Bullet video that the way the Neal.Fun code works is by checking if points are equidistant from the center, and because pyautogui can draw the square so fast, Neal.fun thinks that the square is a circle, passing 99.99% almost all the time.

Hi! Thanks for the issue.
I checked the video and that's impressive how Neal.fun works, I didn't know about that.
I tested here and is true but depends on how centered my mouse can be, using the already implemented method can reach 99,9% too.

Well, I think wouldn't be nice to set the step-increment to 90 because technically wouldn't be more a circle but a note could be added on README referring to this mentioned video or this PR explaining that if the step is 90 a square will be drawn and scoring the max possible.
Feel free to open a pull request.

Good idea! I just opened the pull request. You can add commits to it if needed, because wording or formatting may be a little weird.

Good idea! I just opened the pull request. You can add commits to it if needed, because wording or formatting may be a little weird.

I added a space, thanks a lot!


[iam_authenticator] make option to disable it

We need to have enabled variable for iam_authenticator

This is now out of date with the new reference architecture #286


Add support for ENV and CMD commands in Dockerfiles

Feature request summary
Right now the ENV and CMD commands in Dockerfiles are ignored by kraft cloud deploy. The idea is to add support for these, essentially:

ENV  --> Map to kraft cloud deploy's -e flag
CMD --> Use it to override the Kraftfile's cmd line (it one such line exists)

We introduce an additional ExporterEntry (type OCI) which saves the OCI artifact to a temporary location, then unpack the output tarball and read in config.json which contains all the information we want.
The issue is that this is embedded in function is part of a general interface which is only concerned about the generation of a rootfs and not any additional metadata. (Handling serializing directories in to CPIO archives, reading existing CPIO archives, and turning Dockerfiles into CPIO archives is its main purpose).
The generation of the rootfs is done at lots of different points throughout the project, depending on what you invoke, which is why we have this. Once solution could be to update the Initrd interface to handle these additional metadata attributes, which would ultimately return empty for other source inputs (directory, existing cpio archives, etc.). This would be the quickest solution.

Add new interface methods:
Env() []string
Args() []string // join ENTRYPOINT + CMD together
Ports() []string // EXPOSE
Volumes() []string // VOLUME

Add the ExporterEntry and then parse the temporary archive using stereoscope such that we can access the Metadata object.

Update all references where the Args are necessary.

Similar to the readme work, propagate this info back to our OCI image

Note, we should NOT automatically map ports or create volumes. This is not how docker behaves, it is only acting as a reference.
Describe alternatives
No response
Related architectures
None
Related platforms
None
Additional context
No response

This was done some time ago


fix: [#4202][botbuilder] TeamsActivityHandler — Prevent calling event handlers twice

Fixes #4202
Description
This change prevents calling TeamsActivityHandler event handlers twice for every event received.
Specific Changes
Why were the event handlers getting called twice?

teamsActivityHandler overwrites dispatchEventActivity
dispatchEventActivity is only called from within onEventActivity
onEventActivity already dispatches after calling all handlers
on teamsActivityHandler, when overwriting, it is also calling all handlers

How to fix it?

Prevent the handler call inside the dispatchEventHandler
This should have no effect on any consumer, since they should already be consuming only from within a onEventActivity call.

Testing
A test is added to prevent the behavior from reappearing
Note that to test the issue described, it is enough to run the changed test in main.
The test will fail when run in main currently, since since it is called twice.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.002%) to 84.485%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
19908

Relevant Lines:
22315

💛  - Coveralls

The original code appears to be the same code as is used to handle the ConversationUpdate activity.
Can you also verify that the behavior of ConversationUpdate? And if it has the same bug. And apply the fix and add a test for that there if necessary.
If this is a bug here and not in the ConversationUpdate scenario then why is that?

It appears you are right @johnataylor, I adjusted the code accordingly.
I reviewed other activity types, and I don't think they are vulnerable to the same issue.
However, I wonder if we should consider adding a test that runs for each activity type and makes sure they don't have double calls.

If those extra tests are wanted, I added an experimental branch, experimental/fix-double-call that contains an automated mechanism to test all event types that use the onEventType(handler) structure

Thanks for adding the extra test for conversationUpdate is good.
But if you are saying you can verify this isn't a problem in other cases that would be great.

I wrote some extra tests to verify all others, should I include them?

and we can always consider adding in the extra tests as a separate PR
at this stage the main concern is around this specific bug, and you are confident that this isn't a wider issue?

I added an extra commit.
Basically, I just mean writing tests inside a for loop dynamically. (You could extract the content of the for loop to a function, making that function a kind of factory)
for(const t of types) {
 it(`should use ${t}`, ...)
}

But yes, I have done some further testing as demonstrated on that third commit and believe it is not happening anywhere else for the TeamsActivityHandler.
I think it was just an issue with how overwriting a superclass method made it confusing, loosing track of whether the handler chain was called.
Besides, we saw this issue the first time we used a OnEvent handler. It is really hard to miss.

Hmmm

Error: .github/workflows/tests.yml (Line: 41, Col: 16):
Error: The template is not valid. .github/workflows/tests.yml (Line: 41, Col: 16): hashFiles('**/yarn.lock') couldn't finish within 120 seconds.

I am not sure what that error is referring to. But I will revert back, I guess.

Any updates?

Hey @johnataylor ,
Does this PR look good to you or do I need to adjust something else?

Hi @alexrecuenco
Thank you for finding this and providing a fix! Please do include your test for the generic 'ActivityTypes handlers should not be called twice'

Hmmm

Error: .github/workflows/tests.yml (Line: 41, Col: 16):
Error: The template is not valid. .github/workflows/tests.yml (Line: 41, Col: 16): hashFiles('**/yarn.lock') couldn't finish within 120 seconds.

I am not sure what that error is referring to. But I will revert back, I guess.

I got this weird error when trying to create a for loop that would create tests automatically for each type.
Are those tests necessary to prevent regressions in the future? In that case, I would need some help to understand how to prevent this issue. (Maybe it is an issue unrelated to my tests?)


feat: use ConfigOptions to transfer custom settings

Which issue does this PR close?
Closes #622
Rationale for this change
ConfigOptions is a structure for passing setting in datafusion, which supports passing custom setting.

What changes are included in this PR?
Use ConfigOptions to transfer custom settings.

Are there any user-facing changes?
No.

How does this change test
Add new test.

Codecov Report

Merging #639 (54d9df1) into main (47e4518) will increase coverage by 0.14%.
The diff coverage is 96.30%.

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

@@            Coverage Diff             @@
##             main     #639      +/-   ##
==========================================
+ Coverage   66.26%   66.41%   +0.14%
==========================================
  Files         285      283       -2
  Lines       44863    44259     -604
==========================================
- Hits        29728    29393     -335
+ Misses      15135    14866     -269

Impacted Files
Coverage Δ

analytic_engine/src/engine.rs
58.41% <ø> (-1.59%)
:arrow_down:

analytic_engine/src/instance/mod.rs
84.21% <ø> (-0.79%)
:arrow_down:

analytic_engine/src/lib.rs
98.76% <ø> (+0.08%)
:arrow_up:

analytic_engine/src/sst/reader.rs
92.30% <ø> (ø)

analytic_engine/src/table/data.rs
91.64% <ø> (ø)

df_operator/src/udfs/time_bucket.rs
0.00% <0.00%> (ø)

analytic_engine/src/sst/parquet/async_reader.rs
89.22% <66.66%> (-0.27%)
:arrow_down:

sql/src/provider.rs
35.71% <76.92%> (+0.93%)
:arrow_up:

analytic_engine/src/setup.rs
55.76% <77.27%> (+1.22%)
:arrow_up:

analytic_engine/src/manifest/meta_update.rs
95.97% <91.66%> (ø)

... and 72 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix:removed the callback from the logs where ever it is not necessary

Description
As per the issue , i have removed the unnecessary usage of the callback in logs  of msg_server.
closes: #3404
Commit Message / Changelog Entry
nit: remove "callback" from msg server logging, except where relevant.

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md).
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards and Go style guide.
[x] Wrote unit and integration tests.
[x] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[x] Added relevant godoc comments.
[x] Provide a commit message to be used for the changelog entry in the PR description for review.
[x] Re-reviewed Files changed in the Github PR explorer.
[x] Review Codecov Report in the comment section below once CI passes.

Thanks @crodriguezvega for adding removing the comments.

Thanks! Grepping through the pr locally I'm seeing three additional cases of callback being included when they probably shouldn't.
In RecvPacket:
https://github.com/cosmos/ibc-go/blob/b562f24a1dbda9ea0bf7eae0890a9868abd6c0a7/modules/core/keeper/msg_server.go#L483
In Timeout:
https://github.com/cosmos/ibc-go/blob/b562f24a1dbda9ea0bf7eae0890a9868abd6c0a7/modules/core/keeper/msg_server.go#L554
and in TimeoutOnClose:
https://github.com/cosmos/ibc-go/blob/b562f24a1dbda9ea0bf7eae0890a9868abd6c0a7/modules/core/keeper/msg_server.go#L628


Bug: Unexpected token error when encountering JSX

What did you do?

Open https://explorer.eslint.org/#eslint-explorer with default settings (which include JSX toggled on)
Put in an example including JSX, such as const MyComponent = () => <>foo</>;

What did you expect to happen?
To see successfully rendered AST and Code Path tabs.
What actually happened?
The AST and Code Path tabs say, "Error: Unexpected token < (1:27)"
Participation

[ ] I am willing to submit a pull request for this issue.

Additional comments
No response

Confirmed, I can reproduce this. Please feel free to submit a PR. Thanks!

Thanks. I unchecked "I am willing to submit a pull request for this issue" as I probably won't be able to prioritize troubleshooting and fixing this any time soon.

Okay, I'll take a look.


How to apply BalancedMSE for the d-dimensional regression task?

@jiawei-ren
I wondered how should I apply Balanced-MSE for the D-dimensional regression task? I checked the tutorial code, the example code is formatted for one-dimensional regression.
def bmc_loss(pred, target, noise_var):
    logits = - (pred - target.T).pow(2) / (2 * noise_var)
    loss = F.cross_entropy(logits, torch.arange(pred.shape[0]))
    loss = loss * (2 * noise_var).detach()  # optional: restore the loss scale

    return loss

For example, the pred shape is (N, D), and the target is also (N, D). N is the batch size, and D is the dimension for regressing tasks.
If following the above example, there is an error in the F.cross_entropy function.

Oops, my misunderstanding

@ChengBinJin


Order of written XML elements in header

We are getting some validation errors on XML output which seem to be about the order of the elements in the header. I suspect this is the fault of the validator we're using, but thought I would ask. I would not have expected that the order of XML elements was important.
Specifically, our validator apparently expects the first element to be "mes:ID". But here the "mes:Test" is written as the first element.
Here's one validation error that we're seeing:
Error While Processing XML : Expected element 'ID@http://www.sdmx.org/resources/sdmxml/schemas/v2_1/message' instead of 'Test@http://www.sdmx.org/resources/sdmxml/schemas/v2_1/message' here in element Header@http://www.sdmx.org/resources/sdmxml/schemas/v2_1/message

I skimmed through SDMX_2-1_SECTION_3A_PART_I_MESSAGE.doc and did not see any reason to believe that the order of those elements is expected to be a certain way. However it does always list "ID" before "Test" in that documentation. I would appreciate some feedback on whether our validator is to blame, or if the order really does matter. If the latter, I would be happy to submit a PR.

Could you please explain what "our validator" is? I think there are some XML schema (.xsd) files provided by the SDMX consortium; “Section 3B” on the standards page. Is your validator using these?
XSD definitely allows for order-sensitive vs. -insensitive schemas, so it's possible these could contain something that's not reflected in the plain-language spec.

Could you please explain what "our validator" is? I think there are some XML schema (.xsd) files provided by the SDMX consortium; “Section 3B” on the standards page. Is your validator using these?
XSD definitely allows for order-sensitive vs. -insensitive schemas, so it's possible these could contain something that's not reflected in the plain-language spec.

@khaeru Thanks for the reply. The validator in question is a Java application called SDMX Converter. Another member of my team already has SDMX Converter set up and has confirmed that manually switching the order of the "ID" and "Test" elements does allow the output to pass validation. Personally I am still struggling to get SDMX Converter to work on my computer (some sort of Java version issue I think).

@khaeru Thanks for the reply. The validator in question is a Java application called SDMX Converter. Another member of my team already has SDMX Converter set up and has confirmed that manually switching the order of the "ID" and "Test" elements does allow the output to pass validation. Personally I am still struggling to get SDMX Converter to work on my computer (some sort of Java version issue I think).

As I see it, three possibilities:

The standard fixes the order of these elements.
The order is not fixed.
The standard itself is inconsistent: (1) in some places and (2) in others.

If (1), then that tool is accurately implementing the standard. In this case, we'd definitely want to adjust this package to do the same.
If (2), then this is a bug in that tool. It is incorrectly rejecting XML that actually does comply with the standard (which in (2) allows these elements to occur in any order). Then we could adjust this package in the interest of compatibility, so long as that doesn't make this package violate the standard. But the real fix would be removing that non-standard limitation of the other tool.
If (3), someone ought to contact the SDMX standards group and get some clarification.
To determine which is the case, I'd suggest to inspect the source code of the other tool and/or contact its developer. Someone should also look at those XSD schema files I mentioned. If you aren't familiar with XSD, I can do that.

As I see it, three possibilities:

The standard fixes the order of these elements.
The order is not fixed.
The standard itself is inconsistent: (1) in some places and (2) in others.

If (1), then that tool is accurately implementing the standard. In this case, we'd definitely want to adjust this package to do the same.
If (2), then this is a bug in that tool. It is incorrectly rejecting XML that actually does comply with the standard (which in (2) allows these elements to occur in any order). Then we could adjust this package in the interest of compatibility, so long as that doesn't make this package violate the standard. But the real fix would be removing that non-standard limitation of the other tool.
If (3), someone ought to contact the SDMX standards group and get some clarification.
To determine which is the case, I'd suggest to inspect the source code of the other tool and/or contact its developer. Someone should also look at those XSD schema files I mentioned. If you aren't familiar with XSD, I can do that.

That all makes sense to me. I'll do some more investigation and report back soon.

That all makes sense to me. I'll do some more investigation and report back soon.

@khaeru Judging from these XSD files, it does look like those elements inside the Header are expected to be in a certain order. For example: https://github.com/sdmx-twg/sdmx-ml-v2_1/blob/master/schemas/SDMXMessage.xsd#L1118-L1119
Given that, I assume we're in situation 1 above, so I'll go ahead and a submit a PR in case it's helpful.

@khaeru Judging from these XSD files, it does look like those elements inside the Header are expected to be in a certain order. For example: https://github.com/sdmx-twg/sdmx-ml-v2_1/blob/master/schemas/SDMXMessage.xsd#L1118-L1119
Given that, I assume we're in situation 1 above, so I'll go ahead and a submit a PR in case it's helpful.


[apps/movex-docs] Clean up Hero file

Where
apps/movex-docs/modules/home/Hero.tsx
What
This file seems to have a lot of code smells, such as:

commented code
unused imports
unused code (aka currentTheme, darkColors)

Why
Readability & file size
Definition of done

[ ] the commented code are gone
[ ] the unused code is removed
[ ] the unused imports are removed
[ ] tests still pass yarn test

Can you assign this issue to me ?

Of course @VishalMyla! Thank you


TGD: Master Issue tracking TG issues that must be fixed in v2

Via @tznind "we have lost support for the FrameView because drag/move doesn't work properly with it." here
https://github.com/gui-cs/TerminalGuiDesigner/pull/189
May be same issue:  "Borders are disabled for now I think too for same reason (they break click drag)"

WIth #2537 any view will be sizable too.

My plan with #2537 is to leverage IDesignable such that if design mode is enabled, Border will be able to be "hidden" such that a View without a Border can be dragged.
Same for dragging multiple views.

Since the TGD as access to internal TG objects may be could have some kind of mechanism that allow TGD to enable drag and restore defaults when finish drag.

There is a LOT of code in TGD to do this drag.  Complex bits include

Preventing drag for axis which are not compatible Pos type (e.g. lock X to Pos.Right of Something but still drag along Y axis)
Undo/Redo
Drag to sub/super view
Multi view drag

It's probably a good 10% of the designer codebase.
Not saying we can't make it part of core, just that it's going to get complex fast 😀 but up for trying.
Regarding internal, I think policy is to try and avoid.

Since the TGD as access to internal TG objects may be could have some kind of mechanism that allow TGD to enable drag and restore defaults when finish drag.

As previously discussed, we want to move away from having TGD have internal access. We should make these mechanisms public.
Of course, we CAN do this over time, but I'd like to really try to make it happen before V2 release!

There is a LOT of code in TGD to do this drag. Complex bits include:

Preventing drag for axis which are not compatible Pos type (e.g. lock X to Pos.Right of Something but still drag along Y axis)
Undo/Redo
Drag to sub/super view
Multi view drag
Keyboard support

It's probably a good 10% of the designer codebase. And it's heavily coupled to other bits of code like

view selection system
Operation stack
Global mouse manager.

Not saying we can't make it part of core, just that it's going to get complex fast 😀 but up for trying.
Regarding internal, I think policy is to try and avoid.

How willing are you to migrate this code into TG as needed?

I can try but it is going to be difficult. And not fast because of scale.
Designer basically has a complete parallel mouse harding system based on
global handler and hit detection.
On Tue, 6 Aug 2024, 18:07 Tig, @.***> wrote:

There is a LOT of code in TGD to do this drag. Complex bits include:

Preventing drag for axis which are not compatible Pos type (e.g.
lock X to Pos.Right of Something but still drag along Y axis)
Undo/Redo
Drag to sub/super view
Multi view drag
Keyboard support

It's probably a good 10% of the designer codebase. And it's heavily
coupled to other bits of code like

view selection system
Operation stack
Global mouse manager.

Not saying we can't make it part of core, just that it's going to get
complex fast 😀 but up for trying.
Regarding internal, I think policy is to try and avoid.
How willing are you to migrate this code into TG as needed?
—
Reply to this email directly, view it on GitHub
https://github.com/gui-cs/Terminal.Gui/issues/3650#issuecomment-2271752581,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHO3C5AVOFD2G3DWYDF2WGTZQD7ENAVCNFSM6AAAAABMCU4BU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZRG42TENJYGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I can try but it is going to be difficult. And not fast because of scale. Designer basically has a complete parallel mouse handling system based on global handler and hit detection.

My head's still not clear on this, but I'm now thinking the place to rally here is around IDesignable.
We put it on View and expand it with whatever methods/properties TGD needs. Since I've moved most mouse handling code into Application and out of View & Toplevel (more do do), we can put the right logic in Application.Mouse.cs to enable calling out to TGD (and other designers). We can even go as far as adding a 4th Adornment that is only lit up when IDesignable.DesignMode == true that sits between Border and Padding that TGD can leverage.
The goal would be to enable TGD to minimize throwing away as much as that code as possible whilst leveraging the new architecture in TG V2.
Just spitballing of course to get the juices flowing. This will be an interesting challenge. I'll repeat again: We don't ship V2 until TGD is amazing on it.

Instead of adding a new Adornment why not enabling all the mouse features anywhere on a view if IDesignable.DesignMode == true. So, the mouse will have the same features on Margin, Border, Padding and Viewport.

I think we should split to 2 goals

Enable TGD to function like it did before with Borders/FrameView.
Migrate TGD resize/move code to core

The first will enable a polished use experience and full range of Views supported in TGD in relatively short period of time.
The second is far more ambitious and will take longer.
1. Enable FrameView and Borders as simply as possible
I will have to refresh my memory on the exact issue with Borders and FrameView but I think it was around hit detection and the 'out of the box' FrameView being moveable.  Probably can fix that in CreateSubControlDesign  where it explicitly makes everything focusable we can just make everything immoveable and let the original TGD move code take over.
There are already some Border related hacks that TGD applies in its extension version of FindDeepestView e.g. see UnpackHitView
/// <summary>
/// <para>
/// Sometimes <see cref="View.FindDeepestView"/> returns what the library considers
/// the clicked View rather than what the user would expect.  For example clicking in
/// the <see cref="Border"/> area of a <see cref="View"/>.
/// </para>
/// <para>This works out what the real view is.</para>
/// </summary>
/// <param name="hit"></param>
/// <returns></returns>
public static View? UnpackHitView(this View? hit)
{

      if (hit != null && hit.GetType().Name.Equals("TabRowView"))
      {
          hit = hit.SuperView;
      }

      // Translate clicks in the border as the real View being clicked
      if (hit is Border b)
      {
          hit = b.Parent;

      }

      // TabView nesting of 'fake' views goes:
      // TabView
      //   - TabViewRow
      //   - View (pane)
      //     - Border (note you need Parent not SuperView to find Border parent)

      if (hit?.SuperView is TabView tv)
      {
          hit = tv;
      }

      return hit;
  }

#2. Migrate 'design' code from TGD to Terminal.Gui
This is the Design class which is at the core of TGD.
It is how TGD distinguishes between Views the user created e.g. a TabView and those that are subcomponents that are not ineded to be treated seperately e.g. the TextField in a ColorPicker or the TabRowView in a TabView.
We might start tackling this feature with a new namespace Terminal.Gui.Designer rather than trying to put everything under an IDesignable interface which is on all views?

Re: UnpackHitView and FindDeepestView...
Adornments can have Subviews, and before v2 ships there will be at least several places where this happens:
Border:

#3407
#2814

Padding:

#2489

Note that Adornments is not finished yet, and my thinking is finishing the issues above will lead to fixing most of what's not finished. Some tracking issues:

#2994
#3406

This is relevant because FindDeepestView NEEDS to (and does) traverse into subviews of a Border, which, in-turn may have Borders with subviews...
I don't think TGD needs to support the design of such things, but it DOES need to be aware.
Instead of TGD having a special UnpackHitView, I'd prefer to figure out how FindDeepestView can do what an app like TGD needs: Find the deepest View, ignoring Adornments.
Right?

2. Migrate 'design' code from TGD to Terminal.Gui
This is the Design class which is at the core of TGD.
It is how TGD distinguishes between Views the user created e.g. a TabView and those that are subcomponents that are not ineded to be treated seperately e.g. the TextField in a ColorPicker or the TabRowView in a TabView.

Wow, I see a lot of issues in https://github.com/gui-cs/TerminalGuiDesigner/blob/v2/src/Design.cs that could be addressed with fixes in TG.
Text - should be an attribute of a View somehow, not a hard-coded list in TGD:
    /// <summary>
    /// View Types for which <see cref="View.Text"/> does not make sense as a user
    /// configurable field (e.g. there is a Title field instead).
    /// </summary>
    private readonly HashSet<Type> excludeTextPropertyFor = new()
    {

ALL Views have Title. If Border.Thickness.Top > 0 the Title will be displayed. AND I want it to work with Border.Thickness.Left > 0 too with a vertical-oriented Title.
It is true that not all views need a Text property. Back in v1, we moved Text into View and there were good reasons for that. But in v2, perhaps we back this out somewhat? For example, we could define ISupportsText with a default impl, similar to how I recently did IOrientation.
Would this allow you to simplify Design.cs?
Defn of Container is weird
First, this terminology is confusing
    /// <summary>
    /// Gets a value indicating whether <see cref="View"/> is a 'container' (i.e. designed
    /// to hold other sub-controls.
    /// </summary>

container -> SuperView
control -> View
Second, if a View is public whether it uses Subviews or direct code should be completely encapsulated and TGD should not need to know that. By knowing (or guessing) it means the View in question can't change it's implementation without TGD changing.
Can we figure out a better way for TGD to deal with this?
DesignableProperties should be discoverable
It seems cray to me that TGD has so much hard-coded code for what properties on a View are interesting for design or not.
It seems we could easily come up with a way for TGD to inspect a View for, say, an [DesignableProperty] attribute.

We might start tackling this feature with a new namespace Terminal.Gui.Designer rather than trying to put everything under an IDesignable interface which is on all views?

I now concur that just IDesignable is not enough. Not sure a new namespace is the solution either. I think we could tackle each of the three things I point out above relatively independently?

https://github.com/gui-cs/TerminalGuiDesigner/blob/86c83f8d6e2a920a6c6e6899efca470d058169b9/src/Design.cs#L170C1-L173C32
FWIW, I plan on addressing the cray that led to this in

#3627

https://github.com/gui-cs/TerminalGuiDesigner/blob/86c83f8d6e2a920a6c6e6899efca470d058169b9/src/Design.cs#L150C1-L158C6

From #3490 which I'm now closing as a dupe of this:
We should find all places in TGD where reflection is used to get to the internals of TG and open an Issue for each to address.

PosExtensions
DimExtensions
ColorSchemes - TGD.FindDeclaredColorScheme
ViewExtensions

etc...

Instead of TGD having a special UnpackHitView, I'd prefer to figure out how FindDeepestView can do what an app like TGD needs: Find the deepest View, ignoring Adornments.

I am not sure ignoring Adornments is exactly what is required.  I think TGD can be smarter about it.  Lets look at the use case.
When you click somewhere TGD wants to work out what the most 'user meaningful' thing(s) there are in the click hierarchy.   It does this by inspecting the hit and all the SuperViews:
So when we right click the View in a TabView we will show:

operations relevant to the TabView
but also the actual View right clicked (i.e. AddView).

Looking at the results of UnpackHitView we see we are 4 deep:

The only issue I think with Adornments is that they have Parent instead of SuperView.  Probably TGD can just do something like
var actualSuperView = v is Adornment ? v.Parent : v.SuperView

For example, we could define ISupportsText with a default impl, similar to how I recently did IOrientation.

Would this allow you to simplify Design.cs?

That sounds sensible, so that check for making Text designable would just be v is ISupportsText.
The only thing to be careful about is inheritance where a subview decides it doesn't want Text to be supported but not a big deal.

Defn of Container is weird [...] if a View is public whether it uses Subviews or direct code should be completely encapsulated and TGD should not need to know that.

Being a 'container' determines whether the user is able to drag and drop other views into it.
This covers the use case:

User adds a View (the actual class not a subclass) onto the design area
User then drags views into it
User then adds Border and Title to the View

If user drags a View over a TextField then instead of the dragged view becomming a subview of TextField it would just hover over it (i.e. siblings).
So we need to know by class what is designed to have stuff dropped into it at design time.  But having an attribute DesignedToHoldSubviews would probably would do the trick (probably needs a better name).

view.DesignVisibility.HasFlag(HideFromDesigners)

I don't think this is relevant to whether a view is a container.

It seems cray to me that TGD has so much hard-coded code for what properties on a View are interesting for design or not.

It seems we could easily come up with a way for TGD to inspect a View for, say, an [DesignableProperty] attribute.

I agree, it will simplify code and also potentially allow design of user custom Views later in a consistent manner.
One thing to be careful of is that TGD needs to robustly support the property that means:

TGD knows how to deal with the Type of the property
TGD needs to know if the Property is 'special'

Should it be designable but have a hard coded 'design time' value e.g. Visible (these use SuppressedProperty class instead of Property
The property doesn't do anything weird when changed at runtime e.g. throw exceptions

We also need to decorate 'sub properties' not just those on the root View itself e.g. those accessed via Style or other paths.
TGD can then use reflection to step into the sub properties and identify those that can be designed.

Background
The current approach of explicitly listing in TGD the properties comes from stability point of view.
It is important that TGD does not offer properties that then crash when you try to modify them or save/load them.
For this reason I was very conservative about adding properties and only enabling them when there was adequate unit testing and manual testing for each property


Include unary operators in Subtractable/Addable skills

Unary minus is needed to make user defined literals work nicely, since
the minus is not a part of the literal, but rather applied as a unary
operator afterwards. For instance:
using DegreesCelcius =
    fluent::NamedType<double, struct DegreesCelciusTag, fluent::Subtractable>;

DegreesCelcius operator"" _C(long double degreesCelcius)
{
    return DegreesCelcius(degreesCelcius);
}

DegreesCelcius minimumValue{-45._C};

The last line will first construct a DegreesCelcius{45.}, and then apply
operator- to it.
I also included unary plus in Addable for symmetry, even if it's a lot
less useful than unary minus.
(The compilation error in main.cpp is also fixed in #30, but I included it here as well so it's possible to run tests.)

Thanks for the PR!
I clearly see the point of adding those operators. To give more flexibility, what would you think of breaking down the skills into UnarySubtractable and BinarySubtractable, with Subtractable being the combination of the two?
I'll accept your PR, and break down Subtractable into two. Let me know what you think!

Fyi here is the commit to break them down: https://github.com/joboccara/NamedType/commit/d7828c6295863ff2f4af021682a1778807f98746

I like that idea! Thanks for accepting the PR.


Adding beans.xml to make the JAR accssible to CDI

https://issues.jboss.org/browse/AGPUSH-2174

@polinankoleva @dimitraz w/ the beans.xml file inside the JAR, CDI is not able to access the content / clazzez

From the log:
16:22:34,589 INFO  [org.jboss.aerogear.unifiedpush.kafka.consumers.InstallationMetricsKafkaConsumer] (EE-ManagedExecutorService-default-Thread-9) Update metric analytics for push message's ID 684d7626-02a5-4eaf-905b-6c13e318190f and variant's ID 9a7f4807-ae94-42bb-b834-49d59770b38f


Notifications in menu bar

It would be neat to have a count of unread messages in the menu bar. When my computer is muted I often don't notice new messages.
I wonder if there's an easy way to implement this.
Thoughts?

I know of one github project that allows for easy menubar tinkering: https://github.com/maxogden/menubar

The app icon is currently badged. I'm reluctant to introduce a status item into the system status bar as this app doesn't really need it.
And FTR, it's trivial to implement w/o any libraries or frameworks. Just a few lines of code (see -NSStatusBar statusItemWithLength)


Found a picture of naked woman in the FSOD dataset....Is that a surprise?

The picture‘s path is fsod/images/part_1/n02925666/n02925666_15426.jpgn02925666_15426.jpg.
I was running some test on the fsod dataset and output the pictures with detection results.
When I checked them one by one, that pic showed up and surprised me.
Fortunately no one was standing behide me then.

Very sorry about that, I will process that later.

Thank you for your report. I record your problem, and I will delete this issue in 1 hour to avoid to upset others. Is it on?

The FSOD dataset will be cleaned again after the CVPR DDL, and other functions will be added to FewX.


Remove ?, * and \ symbols from the folder names

Yesterday I had an issue with the repository on my Windows machine, because Windows doesn't allow ?, * and \ in the folder names.

Partly removed

I partly renamed all folders. Few moment ago tested now again on Windows and the issue is still exists.

I was thinking of doing this for the Python under the Language Skills. It'll be a long task

@moulick Well at the moment it's top priority task from my point of view. I want to fix it to allow Windows users to interact with repository. So if you have time, ease take a look at it
Thanks

@ummahusla Should I also add file numbers like 1. Welcome!.py ?

@Moulick I would suggest to make it something like 01 Welcome.py without dots or exclamation marks

! @ # $ % ^ & ( ) _ - + = { } ] [ ' ; and dots are fine in windows...

The following reserved characters:
< (less than)

(greater than)
: (colon)
" (double quote)
/ (forward slash)
\ (backslash)
| (vertical bar or pipe)
? (question mark)

(asterisk)

these are the problematic characters

I see, thanks for that. Even tho I would suggest not to use special characters.

Closed


Fixed crash when the text is too short.

The view crashed when the text is too short as the spannable index becomes negative.

This has also been reported as #10


OnebitAdam got an unexpected keyword 'cuda-aware'

I am running a distributed language modeling script, and it gives the following error:
File "train.py", line 74, in <module>
    training_data=None)
  File "/opt/conda/lib/python3.7/site-packages/deepspeed/__init__.py", line 119, in initialize
    config_params=config_params)
  File "/opt/conda/lib/python3.7/site-packages/deepspeed/runtime/engine.py", line 174, in __init__
    self._configure_optimizer(optimizer, model_parameters)
  File "/opt/conda/lib/python3.7/site-packages/deepspeed/runtime/engine.py", line 538, in _configure_optimizer
    basic_optimizer = self._configure_basic_optimizer(model_parameters)
  File "/opt/conda/lib/python3.7/site-packages/deepspeed/runtime/engine.py", line 613, in _configure_basic_optimizer
    optimizer = OnebitAdam(model_parameters, self, **optimizer_parameters)
TypeError: __init__() got an unexpected keyword argument 'cuda-aware'
Traceback (most recent call last):
  File "train.py", line 74, in <module>
    training_data=None)
  File "/opt/conda/lib/python3.7/site-packages/deepspeed/__init__.py", line 119, in initialize
    config_params=config_params)
  File "/opt/conda/lib/python3.7/site-packages/deepspeed/runtime/engine.py", line 174, in __init__
    self._configure_optimizer(optimizer, model_parameters)
  File "/opt/conda/lib/python3.7/site-packages/deepspeed/runtime/engine.py", line 538, in _configure_optimizer
    basic_optimizer = self._configure_basic_optimizer(model_parameters)
  File "/opt/conda/lib/python3.7/site-packages/deepspeed/runtime/engine.py", line 613, in _configure_basic_optimizer
    optimizer = OnebitAdam(model_parameters, self, **optimizer_parameters)
TypeError: __init__() got an unexpected keyword argument 'cuda-aware'

How do I approach this?

Hi, thank you for your interests in 1-bit Adam. For your error, I think it's because that the actual config name is "cuda_aware", not "cuda-aware". And please make sure to set this config correctly depending on whether your network is InfiniBand or Ethernet. Please find more information in this tutorial: https://www.deepspeed.ai/tutorials/onebit-adam/

Hi, thank you for your interests in 1-bit Adam. For your error, I think it's because that the actual config name is "cuda_aware", not "cuda-aware". And please make sure to set this config correctly depending on whether your network is InfiniBand or Ethernet. Please find more information in this tutorial: https://www.deepspeed.ai/tutorials/onebit-adam/

Close this now since the issue seems got resolved, please feel free to reopen if needed.


Can log into Voyager, but cannot access several routes

Laravel Version: 5.3.26
Voyager Version: 0.9.34
PHP Version: 7.0.8-0ubuntu<IP_ADDRESS>
Database Driver & Version: MySQL

Description:
On a fresh install I am able to log into at site.com/admin using the default credentials, however /admin/roles, /admin/users, /admin/posts/, /admin/pages, /admin/categories, and /admin/menus are not accessible with NotFoundHttpException in RouteCollection.php line 161 while /admin/settings, /admin/database, and /admin/media function fine.
Steps To Reproduce:

Complete fresh install according to documentation

@54656452 Did you using .env file?

All pages seem to give a NotFoundHTTPException since this commit on v0.10: https://github.com/the-control-group/voyager/blame/efd2db3eac6675bdea0b37649872b3329d317f51/publishable/routes/voyager.php#L13
(No longer an admin prefix in the route)

Whats the output of the command php artisan route:list.

+--------+----------+----------+------+---------+--------------+
| Domain | Method   | URI      | Name | Action  | Middleware   |
+--------+----------+----------+------+---------+--------------+
|        | GET|HEAD | /        |      | Closure | web          |
|        | GET|HEAD | api/user |      | Closure | api,auth:api |
+--------+----------+----------+------+---------+--------------+

re-running: php artisan voyager:install did the trick.

For others with same issue, this is because that you now how to add the following piece of code into your routes/web.php:
Route::group(['prefix' => 'admin'], function () {
    Voyager::routes();
});

It's however automatically done when running php artisan voyager:install.

@marktopper Unfortunately in my case re-running php artisan voyager:install did not change anything. Additionally, when adding the codeblock to routes/web.php I recieve the following error:
FatalThrowableError in web.php line 23:
Call to undefined method TCG\Voyager\Voyager::routes()

The necessary entry has been made inside config/app.php
         ...

         /*
         * Package Service Providers...
         */

        TCG\Voyager\VoyagerServiceProvider::class,

         /*
         * Application Service Providers...
         */
         ...

The output from running php artisan voyager:install:
root@monroeco:/var/www/html/monroecoproject# php artisan voyager:install
Generating the default authentication scaffolding
Publishing the Voyager assets, database, and config files
Migrating the database tables into your application
Dumping the autoloaded files and reloading all new files
Seeding data into the database
Adding the storage symlink to your public folder
Successfully installed Voyager! Enjoy :)

Is there a specific namespace that needs a use line in my routes file?

Hey @54656452, the issue that @craigharman had was with the alpha v0.10 version. You are on the v0.9 version. I recommend upgrading to v0.10 since a lot of issues and new features have been made, however soon there will be a migration guide to help you upgrade, but if this is a new project I recommend just starting over.

@marktopper so then my understanding is that I shouldn't be seeing this issue on 0.9.4 anyways? I'll try spinning up a fresh Laravel install and pull dev-master through composer to see if that gets me up and running.

The error should have been fixed in dev-master but also in the next version dev-release/v0.10.
If not please let me know, then I will reopen this issue.


Decrypt vaulted variables in yaml files

Appends decrypted value after the !vault tag.
Deletes encrypted string from the DOM.

Try to replicate existing semantics,
but no 'Decrypt' button (because I didn't need it).
See:
https://docs.ansible.com/ansible/latest/user_guide/vault.html#creating-encrypted-variables

@davetapley This doesn't seem to work for me atm; testing with this file (vault password is "test") https://github.com/gwatts/ansible-test/blob/main/keys.yaml

also I think we'd want to maintain the "decrypt" button semantics so it matches the rest of the extension behavior

@davetapley This doesn't seem to work for me atm; testing with this file (vault password is "test")
https://github.com/gwatts/ansible-test/blob/main/keys.yaml

@gwatts well that is peculiar, I see the same, I'm glad you tested it, I'll 👀

also I think we'd want to maintain the "decrypt" button semantics so it matches the rest of the extension behavior

That's fair. I'll see if I can figure something out.
It's a little awkward because I collapse multiple encrypted lines down to a single one, which is easy enough just deleting DOM nodes and inserting a new one, but going the other way might require some more cleverness.