Adds property accessors to the OHLCTooltip with fallbacks

This code is, from what I can tell, backwards-compatible with the current implementation; however, it provides more control to the user. For my use case, currentItem is actually an ID, not a data object. My code requires open/high/low/close fields be resolved via the accessor functions. Direct item.key access, even after calling accessor(), is insufficient.
Feel free to comment if you feel this breaks the current API!

cant you use displayValuesFor for this?

@rrag That would require instantiating a new object with open, high, low, close and volume keys. I would prefer to use the accessors so I can resolve the data myself (my accessors map ID to data).
Do you object to this PR? If so, I'm happy to make changes to it to meet your desires! Just let me know what you would like it to be, instead.

this is not a scalable solution, if the currentItem is an id, then each XXXTooltip would need this kind of a change. using displayValuesFor solves this problem for this and other usecases,
I also do not believe creating and throwing away objects is a problem, in-fact I recommend that objects should be short lived.


Add ruff lint

upgrade checkout to v4
add ruff lint

Will do this along with preparation of release 1.0.0.


[release/3.2] Issue #1814 Remove findbugs warning

Suppressing the warning as the type is used in a specific context
where exposing that byte[] is not an issue.

test this please

Could reproduce the test failure locally - investigating

test this please
Facing some unstable tests - see #1782


Japanese Health And Sports Day rename to Sports Day since 2020.

Japanese [Health And Sports Day(体育の日)]  rename to [Sports Day(スポーツの日)] since 2020.
https://www8.cao.go.jp/chosei/shukujitsu/gaiyou.html

@cookie-maker Just wanted to check if you were able to review my comments.

sorry I'm late.
Thanks, but I could not find your review comment.
Could you tell me where you commented on?

You can find my comments here on Github in this PR.

I am sorry I do not know how to use Github.
I still do not know the location of the comment.
Conversation Tab? Files Changed Tab?
Can I have a picture of the place where the comment is written?

No problem. My apologies, as I don't always realize not everybody uses Github often.
On this page you can scroll down the page to see the changes and also comments and feedback from others. See this screenshot:

You can there respond or provide other comments.
Hopefully that clarifies it.
Cheers! Sacha

Thank you for your kindness.
I understood the problem of the code by image file.
However, I can not find your review comment from above Github.
This is the screen that I see.

Maybe, the submit review button is not pressed.

@cookie-maker Thanks for the screenshot. Nothings is wrong. You are absolutely right, I forgot to press Submit.
I did it now, so hopefully you can see it.
Cheers! Sacha

ありがとうございます！
I found your comment!
Thank you for review.
I Fixed the wrong point just now.
よろしくおねがいいたします！

いつもありがとうございます！すぐにあなたの他のPRを検証します。


Add RB_USER_INSTALL to preserved ENV keys

This PR adds the environment variable RB_USER_INSTALL to the list of preserved keys. That variable is used on FreeBSD.

see #6013

Without this change, Bundler's usage of it with the shared helpers to set a preserved env would raise this:

ArgumentError: new key RB_USER_INSTALL

(This seems to be the only env var which is used but not listed in the preserved keys list.)

Perhaps we can check for usages:
$ egrep -h -o -R 'set_env "([A-Z_])+"' lib  | sort | uniq
set_env "BUNDLER_VERSION"
set_env "BUNDLE_BIN_PATH"
set_env "BUNDLE_GEMFILE"
set_env "GEM_HOME"
set_env "MANPATH"
set_env "PATH"
set_env "RB_USER_INSTALL"
set_env "RUBYLIB"
set_env "RUBYOPT"

Thanks!
@bundlerbot r+

@bundlerbot retry


функционал, связанный с курсами

Codecov Report

Merging #3 (e026016) into master (1028993) will increase coverage by 0.83%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   23.43%   24.26%   +0.83%
==========================================
  Files          12       12
  Lines         239      239
  Branches        4        2       -2
==========================================
+ Hits           56       58       +2
+ Misses        183      181       -2

Impacted Files
Coverage Δ

app/controllers/ProfileController.scala
0.00% <0.00%> (ø)

app/domain/service/ProfileServiceImpl.scala
96.87% <100.00%> (+0.20%)
:arrow_up:

Continue to review full report at Codecov.

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


Arquivo que contem o CPF

Poderia informar?

Eu também gostaria de saber

Arquivos disponiveis nao contem CPF

Codigo fonte do projeto?

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
marcusrbrown, @jammarmalade, @tersapp, @NicolasLopes7, @DapengLan, @simonbuechi, @alberrrttt, @jnylen, @Requiemu, @riptl, @AnthonyAkentiev, @roson9527, @xingxinglian, @omarladan-Dev, @kolyasapphire, @mozila80, @DiversoServicies, @veox, @bergren, @qftgtr


TableRowColumn Not Wrapping Text

There doesn't appear to be any way (that I can find) to wrap text in a TableRowColumn. I have tried with <p>,<span> and <div> as children of the TableRowColumn and the text never wraps.
At best I can get ... at the end of the text (which appears to be default behaviour) but that doesn't actually allow the user to view the larger text, it only seems to indicate that the text is longer than the column
Is there any way to make text in a TableRowColumn wrap within?
PS. I have tried setting the height of the Rows to something large as well and that still doesn't wrap.

@JasonWeise Could you try applying the following style to the TableRowColumn element?
<TableRowColumn style={{wordWrap: 'break-word', whiteSpace: 'normal'}}></TableRowColumn>

Source: http://stackoverflow.com/questions/1057574/html-td-wrap-text

@newoga Thank you so much, that got it. I was CERTAIN I tried that as I found it on a forum somewhere, but was getting so frustrated that heaven only knows what I ended up trying.
Think I just needed to walk away for a bit... you are a saviour :+1:

No problem @JasonWeise!
@oliviertassinari @alitaheri I know we plan on adding functionality to the Table at some point based on the roadmap. Do you think this use case should be a configurable prop? I can argue both ways.

@newoga It's debatable, I think the use case is valid! but a configuration prop might complex the logic within the component. if we support composability better this would be much easier to implement.

@alitaheri Since Table is already a little composable, we may be able to put this logic in the getStyles() for <TableRowColumn />.
I think we get a way with doing it in one line of code with something like:
      root: {
        fontWeight: 'normal',
        fontSize: 12,
        paddingLeft: theme.spacing,
        paddingRight: theme.spacing,
        height: theme.height,
        textAlign: 'left',
        whiteSpace: this.props.wrap ? 'normal' : 'nowrap', // changed line
        textOverflow: 'ellipsis',
        color: this.getTheme().textColor,
        position: 'relative',
      },

That sounds like a good idea :+1:


updated README

Various updates to the readme file...

Coverage remained the same at 96.96% when pulling e45be6529902fd8e87333690c1b9ffac7591d2a5 on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.

Coverage remained the same at 96.96% when pulling e45be6529902fd8e87333690c1b9ffac7591d2a5 on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.

Coverage remained the same at 96.96% when pulling fdcce4fe94c3f435810c594fd1ffc46412636f17 on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.

Coverage remained the same at 96.96% when pulling 57c9228f5603ce17a98b7f7b62ab121b657e7c70 on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.

Coverage remained the same at 96.96% when pulling 57c9228f5603ce17a98b7f7b62ab121b657e7c70 on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.

Coverage remained the same at 96.96% when pulling 57c9228f5603ce17a98b7f7b62ab121b657e7c70 on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.

Coverage remained the same at 96.96% when pulling 443cc47ec786b97aef878d093b8962a44026fc0d on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.

Coverage remained the same at 96.96% when pulling 4595ef5bbdb6cceff76fce3605d5258f79a152b1 on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.

Coverage remained the same at 96.96% when pulling d3799c5d112a9cab455bce193994d1aee8c8e8b8 on update-readme into 22b633ce4524b114cbc2db38da6fe5d2a167ba3d on master.


[WIP] Initial support for generating externally verifiable tokens

no-issue

@allouis think this PR is 95% done at this point. Needs couple minor tweaks around naming and some tests fixed after there was a new /ghost/.well-known/ endpoint introduced. Can you take over that so we could maybe shit this whole change on Monday?

@allouis tried permission migrations on MySQL and SQLite, both worked fine :+1: Left couple comments for copy change :wink:


codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@f8499b8). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   50.00%
=======================================
  Files           ?        1
  Lines           ?        4
  Branches        ?        0
=======================================
  Hits            ?        2
  Misses          ?        2
  Partials        ?        0

Continue to review full report at Codecov.

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


tfm: Harden and Enforce TF-M Initial Attestion Key

Enforce that the initial attestation partition has the required
initial attestation key provisioned.
If the initial attestation key (IAK) is not present during boot of
TF-M the system will panic during initialization.
Introduce Kconfig option in zephyr build system that reflects the TF-M
cmake config variable with the same default value for dummy provisioning
and have it satisfy the IAK present requirement.
This configuration is not suitable for production, and by having this
in zephyr configuration we can have this as part of the hardened
configuration check.

@d3zd3z @microbuilder The issues discovered in this PR appears to be existing in main branch as well.
Would you be able to have a look at this?

@d3zd3z please take a look

Rebased the PR. Hopefully the CI issues are gone now.

Whether the HW / SDK supports this key. Given that self-provisioning of the IAK is not something that should be done according PSA, this means that it is not really part of the build, but part of the whole "system" of this platforms build.

@joerchan Can you point me to where it's mentioned that self-provisioning is something to avoid? I've never looked in detail, so just started poking around to understand the expected workflow.
In this PSA document on attestation, I see:

Source: https://arm-software.github.io/psa-api/attestation/1.0/IHI0085-PSA_Certified_Attestation_API-1.0.3.pdf

During such an enrolment process, a device might generate a new key pair
and create a Certificate Signing Request (CSR) or equivalent, containing:
∙ The public key of the generated key-pair.
∙ A proof of possession of the corresponding private key (in general this is
   the public key signed by the private key). This protects against
   man-in-the-middle attacks where an attacker can hijack the enrolment
   to insert their own public key into the device request.
∙ An initial attestation, in order for the recipient to assess how that
   particular combination of hardware and firmware can be trusted.

The CSR is then passed to a Certification Authority who can assign it an identity with the new service and
then return an identity certificate signed using the private key of the Certification Authority. The
Certification Authority may be operated by the company who owns the devices or operated by a trusted
third party. Creating extra identities on devices is expected to be a routine operation.

A 3rd party key generated off device and added during provisioning also seems normal -- since you need to get the public key into a database as well to verify token signatures -- but I didn't see a statement that self-provisioning (such as deriving a new key from the HUK) should be avoided?

joerchan Can you point me to where it's mentioned that self-provisioning is something to avoid? I've never looked in detail, so just started poking around to understand the expected workflow.

@microbuilder It's not that they something to avoid, but provisioning by a different image is something they recommend, and it is our current documented approach.
https://tf-m-user-guide.trustedfirmware.org/integration_guide/platform_provisioning.html#provisioning-production-hardware

Jumping out of the blue and possibly a dumb question, but I had a try and (I can confirm this doesn't break anything but) I'm still seeing the following warning when running TF-M samples:
[WRN] This device was provisioned with dummy keys. This device is NOT SECURE

Is that expected ?

Jumping out of the blue and possibly a dumb question, but I had a try and (I can confirm this doesn't break anything but) I'm still seeing the following warning when running TF-M samples:
[WRN] This device was provisioned with dummy keys. This device is NOT SECURE

Is that expected ?

Yes, pretty much all of the boards are dummy provisioned, for the nordic boards you need to use our SDK in addition to get it provisioned.


implement alignment-safe ntoh16p() && ntoh32p()

Cortex-M platforms don't allow unaligned uint16_t or uint32_t accesses, leading to hard-faults.
This PR provides portable implementations for ntoh16p() and ntoh32p() using malloc memcpy.

(Actually, it is Cortex-M0 platforms that don't support unaligned accesses.  Cortex-M3, for instance, do.)

So it probably would be useful to keep the unaligned-access-supported alternatives based on a compile-time switch.

(Actually, it is Cortex-M0 platforms that don't support unaligned accesses. Cortex-M3, for instance, do.)

I'm not sure it is that simple. According to ARM, Cortex-M3+ support unaligned accesses for some of the available load/store operations. I don't know what others are there, but can we be sure the compiler is not gonna use them?

So it probably would be useful to keep the unaligned-access-supported alternatives based on a compile-time switch.

Plese ignore my last comment. Do you want one switch per size or is one switch for both 16bit & 32bit enough for now?

I don't think we need to do this fine-grained; a single "can do unaligned accesses" switch should be enough.  (If there are weird intermediary cases, the library just isn't optimized for them, but it seems worthwhile to optimize for Cortex-M3 type environments.)

I've added the cmake option (align_reads), disabled by default.

@cabo what's the status here? Context: we are waiting for some fix here to merge some PR in RIOT... and the COSE work also depends on this... you get the picture ;)

Hm, shouldn't those fixup commits have been squashed before merging this?


roblox - mangohud support stopped working

Recently mangohud stopped working.
Vulkan and mangohud functionality is now incorporated in mkWindowsApp, and it works fine for Sable but for some reason mangohud doesn't execute at all for Roblox.

I removed the roblox package due to it being unrealiable.


Tweak how we transform the holdings data

Remove the description; add its value as another entry in enumeration. Often they contain similar-looking data, and this makes it easier for the front-end to render. Closes https://github.com/wellcomecollection/platform/issues/5082
De-duplicate holdings records. A lot of the holdings data is private info we don't expose in the transformed Work, so we can de-duplicate safely. Closes https://github.com/wellcomecollection/platform/issues/5083

This feels like a change that can be applied in-place – because the description field is already optional, we can reindex affected works and the already-deployed API will just stop showing that field.
A catalogue snapshot suggests it's ~26k works.


Compilation failure on Alpine Linux

MFEM fails to compile on Alpine Linux. I use Alpine Linux for my CI builds, so it isn't a major problem.  I've been able to repeat the error on alpine:latest as well as an older version (alpine:3.12). Below is the error message.
[ 16%] Building CXX object CMakeFiles/mfem.dir/linalg/constraints.cpp.o
In file included from /mfem/linalg/../fem/../mesh/mesh.hpp:25,
                 from /mfem/linalg/../fem/fespace.hpp:17,
                 from /mfem/linalg/constraints.cpp:14:
/mfem/linalg/../fem/../mesh/../general/zstr.hpp: In function 'std::string strict_fstream::strerror()':
/mfem/linalg/../fem/../mesh/../general/zstr.hpp:83:35: error: invalid conversion from 'int' to 'const char*' [-fpermissive]
   83 |    std::string tmp(p, std::strlen(p));
      |                                   ^
      |                                   |
      |                                   int

Below are the steps for repeating the error (I'm using ubuntu 20.04 Docker 20.10.7)

docker run -it alpine:latest /bin/sh
apk update && apk upgrade && apk add build-base cmake git blas-dev lapack-dev openmpi-dev curl-dev
Then I clone the repo, configure, and build MFEM. Failure occurs during the build process.

git clone https://github.com/mfem/mfem.git
cd mfem
cmake -S. -B./build/
cmake --build ./build/ --config Release

Thanks for reporting this @reine249, I was actually not aware of Alpine Linux.
I believe the issue is likely due to the following check not passing
#elif (_POSIX_C_SOURCE >= 200112L || _XOPEN_SOURCE >= 600) && ! _GNU_SOURCE || \
      defined(__APPLE__) || defined(__FreeBSD__) || defined(__OpenBSD__) || \
      defined(__NetBSD__) || defined(__DragonFly__) || defined(__EMSCRIPTEN__)

Do you know if Alpine Linux defines a similar macro that we can check here?

Maybe __MUSL__?

It looks like __MUSL__ isn't defined.  After doing some digging in /usr/include/string.h, the only macro I could find that was defined in Alpine and not in Ubuntu was __NEED_locale_t.  However, I don't know how specific this macro is to Alpine.
Sorry for using such an obscure linux distribution.  Gitlab's examples for CI/CD use Alpine, and I found that Alpine was faster for my CI builds than Ubuntu.

@reine249 Can you check that #2995 resolves this problem?

Yes, #2995 resolves the problem.  Thank you for the quick fix!

Resolved in #2995


Heads Up: Moving the repository

As part of the transition to the Node.js Foundation Express top level project, the strongloop/express repository will be moved to expressjs/express. I will be looking to make the move later on today. Github should take care of the API redirects.

Any specific time range in mind? I can stand by to make sure the integrations are up and working, like AppVeyor, Travis CI, and Coveralls.

Another 30 minutes or so? Finishing up another task first.

Ok, other task complete. Will start the process of moving the repo now.

Moved. There shouldn't be any breakage but let's use this issue to track anything that does break.

/cc @Fishrock123 @mikeal

I added tc-agenda and tsc-agenda labels to the issue tracker to provide a means of escalating issues per the "default" processes. I'll get the governance draft PR update tomorrow so we can iterate on those.
@Fishrock123 ... were you going to get the TC invite put together?

The Travis CI push integration is broken; going to disable it and re-enable it to get the hooks straighted out.

Ok, Travis CI & AppVeyor hooks are working now.

+1 awesome.

Just trying to figure out Coveralls now.

Alright. Coveralls working now. Integrations seem to be good to go now.

+1 thanks for getting those working

I added tc-agenda and tsc-agenda labels to the issue tracker to provide a means of escalating issues per the "default" processes.

@jasnell what is the difference between those two labels?

Also, I guess label discussion seem off-topic for the subject of this issue: "Heads Up: Moving the repository". Since the repository is moved now, the issue can probably be closed.

tc-agenda puts the issue on the Express TC meeting agenda. tsc-agenda puts it on the Node.js TSC meeting agenda. The goal is to never use the tsc-agenda one ;-).
Will close this now that the move is done and things appear to be working :-)


DOCSP-8723: Add devhub directives

[JIRA]

Adds support for default_domain field in snooty.toml
Adds the following directives to the devhub domain:

meta-description
series
author
author-image
aft-image
type
level
tags
products
introduction
prerequisites
content
summary
related

Adds the following directives to the empty domain:

pubdate
twitter
youtube
twitch

Should author-image be an option of author? cc @schmalliso

Is Date something we are going to automate or have users input?
Also, would there be a specific directive for Author Bio

I would happy for author-image to be an option of author!

Just a note about these author modifier fields -- we actually don't want to encourage authors to put their data into each article. I'm assuming we are capturing this here so that we make it possible to create an author document within the devrel content. But that we would perhaps include that document in the render template based on an author doc lookup. Are we all in agreement?

Just a note about these author modifier fields -- we actually don't want to encourage authors to put their data into each article. I'm assuming we are capturing this here so that we make it possible to create an author document within the devrel content. But that we would perhaps include that document in the render template based on an author doc lookup. Are we all in agreement?

@skerschb That makes sense. So for example, articles could include a document called includes/authors/author-name.rst that includes something like this:
.. author: Author Name
   :image: path/to/image.jpg
   :bio: Short bio here

@eshwer AFAIK they don't want date to be automatically generated based on when the doc is created. They will write the date here and we can massage it into an accurate date time format in the parser level. And see above, I think we can include bio as an option of author.


Allow overriding Cowboy HTTP protocol options

Closes #1266

Pull Request Test Coverage Report for Build de8112f0bfa91ade7f0fff0a81b5cbd61e885a3c-PR-1474

0 of 0   changed or added relevant lines in 0 files are covered.
67 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 66.495%

Files with Coverage Reduction
New Missed Lines
%

apps/fz_http/lib/fz_http/config/definitions.ex
67
19.63%

Totals

Change from base Build 467d6c309528806766293b571f0507c48dc73ae5:
-0.03%

Covered Lines:
1675

Relevant Lines:
2519

💛  - Coveralls


Add support for Sagemath

Sagemath is a mathematics language built on top of python  https://www.sagemath.org/

So, the binary from their site is like 5+ gb uncompressed.. kinda big and may not get through the github actions.

Is there a reason it needs to be that big?
Is there any extra "fluff" that we can exclude? E.g. we definitely wouldn't need a UI if that's included in there
If so, please propose what is a good build option for just getting the core functionality into Piston, if it's still wanted


Tests enhancement

Description

No relevant changes to the UI/UX or the features. Mainly enhancing our test coverage due to feedback on last retrospective presentation.
Tests

UI Changes
Issues

Additional Notes
Add any other information about the PR here.

Once PR is ready to merge, please Squash then merge as I have a lot of irrelevant commit due to reverting and such.

@saanbe16 Can you check what is the issues with the server pipeline testing ? I merge main into my branch and I don't understand why it's not passing here but on main it is.

@saanbe16 Can you check what is the issues with the server pipeline testing ? I merge main into my branch and I don't understand why it's not passing here but on main it is.

Yeah im looking into it


add required traces to the endpoint package

#ziohackathon

I wasn't sure if the macro needed to be added to all files as a form of future proofing or only to the ones that have effects currently in them.

@paulpdaniels This looks great! We can move this over to zio/zio-http now, as my pull request has been merged.


SelectPanel2: Export props + pass props

Closes https://github.com/github/primer/issues/3024

Rollout strategy

[x] Patch release

Testing & Reviewing

Merge checklist

[dont have tests yet!] Added/updated tests
[x] Changes are SSR compatible
[x] Tested in Chrome
NA] (GitHub staff only) Integration tests pass at github/github (Learn more about how to run integration tests)

@siddharthkp we would also need the SelectPanelSecondaryActionProps to be exported to use it with the item picker for cases where we need secondary actions (related: https://github.com/github/collaboration-workflows-flex/issues/423, https://github.com/github/github/pull/310856/files).
Do you want me to create another issue to track that or can that also go as a part of this PR?

@ayy-bc Sure thing, I can add them to this PR


feat(extend): regard extended enum as exclusive union type of instances

Fix #7

Codecov Report

Merging #22 (87523c7) into dev (82f3a85) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               dev       #22   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          173       189   +16
  Branches        12        12
=========================================
+ Hits           173       189   +16

Impacted Files
Coverage Δ

src/extend.ts
100.00% <100.00%> (ø)


Add detection for spotify-launcher install location on Arch

On Arch you can either install Spotify using the AUR 'spotify' package or the community repository 'spotify-launcher' package. SpotX successfully detects the AUR package Spotify install, but not the spotify-launcher package install. Using the path argument works as a workaround, but it would be nice for the directory to be automatically detected.
spotify-launcher installs Spotify to this directory:
~/.local/share/spotify-launcher/install/usr/share/spotify

+1
It used to be detected after the Spotify not found in PATH. Searching for Spotify directory..
Now it doesn't.

have you tried using curl
curl -sS https://download.spotify.com/debian/pubkey_7A3A762FAFD4A51F.gpg | sudo gpg --dearmor --yes -o /etc/apt/trusted.gpg.d/spotify.gpg
echo "deb http://repository.spotify.com stable non-free" | sudo tee /etc/apt/sources.list.d/spotify.list
then install the client
sudo apt-get update && sudo apt-get install spotify-client

@vsdhaliwal They are on Arch, not a Debian based distribution.

@akumaaayush you on arch too ? Have you added it to the path

@vsdhaliwal, Yes I'm on arch too. The spotify-launcher is already on the path upon install.

Until SpotX finds it, you can specify where Spotify is located: $HOME/.local/share/spotify-launcher/install/usr/share/spotify
Following the README example, you can do : bash <(curl -sSL https://raw.githubusercontent.com/SpotX-CLI/SpotX-Linux/main/install.sh) -ceP $HOME/.local/share/spotify-launcher/install/usr/share/spotify

The command fir my steam deck was : bash <(curl -sSL https://raw.githubusercontent.com/SpotX-CLI/SpotX-Linux/main/install.sh) -ceP /var/lib/flatpak/app/com.spotify.Client/x86_64/stable/7a095d1e1ffc3a1aa42a58579a9a733950c8eac306b1b2cd38495c2811bb4744/files/extra/share/spotify

The command fir my steam deck was : bash <(curl -sSL https://raw.githubusercontent.com/SpotX-CLI/SpotX-Linux/main/install.sh) -ceP /var/lib/flatpak/app/com.spotify.Client/x86_64/stable/7a095d1e1ffc3a1aa42a58579a9a733950c8eac306b1b2cd38495c2811bb4744/files/extra/share/spotify

You're using the Flatpak version of Spotify, not spotify-launcher.

You're using the Flatpak version of Spotify, not spotify-launcher.

Will SpotX work with the flatpak version? I cannot find the path for this. I am using arch.

Will SpotX work with the flatpak version? I cannot find the path for this. I am using arch.

@devolskii Does this part of the README not work for you? I don't use Spotify or Arch, but that should work.
If you installed Spotify for your user instead of system, replace /var/lib/flatpak/app with /home/$USER/.var/app/ instead from that command.

Until SpotX finds it, you can specify where Spotify is located: $HOME/.local/share/spotify-launcher/install/usr/share/spotify
Following the README example, you can do : bash <(curl -sSL https://raw.githubusercontent.com/SpotX-CLI/SpotX-Linux/main/install.sh) -ceP $HOME/.local/share/spotify-launcher/install/usr/share/spotify

this one works. Thanks man.

+1 for this issue.
The solution posted by @Mylloon worked for me. I used spotify from the AUR (yay -S spotify) and then bash <(curl -sSL https://raw.githubusercontent.com/SpotX-CLI/SpotX-Linux/main/install.sh) -ceP $HOME/.local/share/spotify-launcher/install/usr/share/spotify

This repo is inactive.
Everyone should use the new version instead where the spotify-launcher issue is fixed:
https://github.com/SpotX-Official/SpotX-Bash


[Plot] add "baseline" option in addCurve

This is the PR related to #2714 to add a baseline parameter.
this include

[x] matlab backend
[x] opengl backend
[x] unittest
[x] example of usage -> not sure this is mandatory

If such a parameter is added then we could reconsider the usage of the 'fill parameter'.
/close #2714
small script to test, for now fails in the addHistogram, with the current values:
from silx.gui import qt
from silx.gui.plot import Plot1D
import numpy

qapp = qt.QApplication([])

x = numpy.arange(0, 10, step=0.1)
my_sin = numpy.sin(x)
y = numpy.arange(-4, 6, step=0.1) + my_sin
mean = numpy.arange(-5, 5, step=0.1) + my_sin
baseline = numpy.arange(-6, 4, step=0.1) + my_sin
edges = x[y>=3.2]
histo = mean[y>=3.2] - 1.8

# opengl plot
plot_ogl = Plot1D(backend='opengl')
plot_ogl.addCurve(x=x, y=y, baseline=baseline, color='grey', legend='std-curve', fill=True)
plot_ogl.addCurve(x=x, y=mean, color='red', legend='mean')
plot_ogl.addHistogram(histogram=histo, edges=edges, color='red', legend='mean2', fill=True)

# matplotlib plot
plot_mtl = Plot1D()
plot_mtl.addCurve(x=x, y=y, baseline=baseline, color='grey', legend='std-curve')
plot_mtl.addCurve(x=x, y=mean, color='red', legend='mean')
plot_mtl.addHistogram(histogram=histo, edges=edges, color='red', legend='mean2', fill=True)

plot_mtl.show()
plot_ogl.show()

qapp.exec_()

For the OpenGL implementation, did you make it work?
The approach needs to be completely changed when the baseline is no longer a single value.

If this features is a kind of way to create stacked histogram?

If this features is a kind of way to create stacked histogram?

was not the original idea, but I guess we could have something like that from it. But would probably require a dedicated api or some thinking around the existing one.
Is it a need ?

For the OpenGL implementation, did you make it work?
The approach needs to be completely changed when the baseline is no longer a single value.

I make it work on my case, but there is some code for duplicating the last point (connecting degenerated triangle) that I don't fully get and is currently removed.
Maybe you also have some 'corner cases' to test and that can be interesting.

As now the order of rendering is that of addition of items to the plot, I don't think you need to use the baseline to render stacked histograms.
Stacked histogram is simple to do (a cum sum of the values of the histograms) and you provide it in the examples, so to me it does not need to be added to silx.

I put a few comments in the review

There is still a small issue with setBaseline, one can provide an array with a different length as the data. Maybe this method can be private.
Also, the Curve does not have a baseline argument in setData.

BTW, what means a curve with a baseline? As a curve is supposed to be a line, there is maybe an inconsistency problem with the picking API. Then that's maybe not a curve, but another object which have a surface.

From my point of view a baseline is more a 'property' of a curve. Having a specific object would complexity the code and the usage for no reason. (in the case we want to first have a curve, then have an associated baseline). It is also how most 'ploting' libraries work I guess.
Anyway it would make sense to keep consistency. We can have a setData with the baseline and the getData which does not return the baseline. In this case the user would have to specifically call the getBaseline function. But the setData and the getData would not be 'homogeneous'... :confused:

After a discussion with @t20100, second option is the best one. We won't modify getData, only setData with a baseline parameter and a default value set to None.

LGTM, @payno is there anything else to do on it?


test case assign feature of yangwei in June

E00005 assign IP to job/pod for ipv4, ipv6 and dual-stack case
R00006 finally succeed to recovery a pod whose original node is power-off
D00004 if the pod's ip in one ippool is occupied by the pod, it fails to delete the ippool, vice verse
E000014 succeed to create and delete ippool in batches

resolve

v

https://github.com/spidernet-io/spiderpool/pull/422


docs: clarify where to look for AWS MSK zip

clarify where to look for AWS MSK zip
add client.environment to property table

I have to copied changes to https://github.com/ably/kafka-connect-ably/pull/162, because workflow doesn't start for external PRs, I will check it later


Update tween.js dependency

The latest major version of tween.js results in build warnings:
  Build/Cesium/Cesium.js  3.7mb ⚠
⚡ Done in 1556ms
▲ [WARNING] Import "Easing" will always be undefined because the file<EMAIL_ADDRESS>has no exports [import-is-undefined]
    packages/engine/Source/Core/EasingFunction.js:1:9:
      1 │ import { Easing } from<EMAIL_ADDRESS>        ╵          ~~~~~~
▲ [WARNING] Import "Tween" will always be undefined because the file<EMAIL_ADDRESS>has no exports [import-is-undefined]
    packages/engine/Source/Scene/TweenCollection.js:8:9:
      8 │ import { Tween as TweenJS } from<EMAIL_ADDRESS>        ╵          ~~~~~

These warnings will need to be resolved before updating the dependency.

Based on the docs, it looks like we'd want to make sure the import points to tween.es.js.


Improve project setup

Add airbase as parent to enable release in future
Add necessary details for release
Add airbase property overrides to skip various checks to allow passing build
Move test code into correct path
Centralize property declarations in parent pom
Improve dependency declarations and add version overrides as minimally needed

Fixes #3
Fixes #14

Currently there are test failures related to some classpath issues .. @willmostly and myself are looking but if anybody else wants to chime in that would help

@mosabua can you split this PR into multiple commits?

@mosabua can you split this PR into multiple commits?

Not really .. this current setup is necessary for it to build. If I split it up the commits in between wont build.

I have a script running some tasks against the gateway with changes in this PR and it seems to be running fine. It's good for me.

Rebase #17 from main into here now .. everything still working.


Add test for creating items with position.

It seems the entity creation interface has a really surprising nonuniformity here. There's likely a need to chase this further with all objects, but for now this at least documents one problem.
TEST=TestEntity.test_create_with_position

@redruin1 I'd have been happier to fix this, but absent that, this at least is very solid documentation of the issue.
Is this present in the main branch?


Add inline assembly sym operands as GCC input operands

Updates <Builder as AsmBuilderMethods>::codegen_inline_asm to convert sym operands into GCC "X" (&func_or_static) input operands to indicate the dependency on the referenced symbols and prevent them from being eliminated.
Fixes #157.

I think it would be better to do it the same way as the LLVM codegen, that is by setting the attribute __attribute__((__used__)).
Is this a blocker for you?

The LLVM codegen does add_compiler_used_global only for sym operands in global_asm!, not those in asm!, unless I'm missing something.
https://github.com/rust-lang/rust/blob/d8e59edbfa47ff38e23e6dedab6bedd3b41895e0/compiler/rustc_codegen_llvm/src/asm.rs#L345-L347

Oh, you're right indeed!
Have you tried using the same constraint (s) as the LLVM codegen to see if it fixes this issue?

Also, could you please add an option (maybe an environment variable) in the tests to allow running the tests with optimizations in the CI?

Have you tried using the same constraint (s) as the LLVM codegen to see if it fixes this issue?

GCC seems to employ some sort of check that rejects this when position independent code is enabled. https://cpp.godbolt.org/z/jdGbxKzcq

Also, could you please add an option (maybe an environment variable) in the tests to allow running the tests with optimizations in the CI?

Since lang_tester doesn't provide a good support for test matrices, I split tests/lib.rs into tests/lang_tests_{debug, release}.rs so that  cargo test would test both with delimited outputs. Does this look good for you? https://github.com/rust-lang/rustc_codegen_gcc/pull/163/commits/c5b80404d299160d4b941182f0c2eea578a1610e
$ cargo test
    ⋮
     Running tests/lang_tests_debug.rs (target/debug/deps/lang_tests_debug-9d19ab0362e92e44)

running 21 tests
test lang_tests::slice.rs ... ok
test lang_tests::static.rs ... ok
    ⋮

test result: ok. 21 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

     Running tests/lang_tests_release.rs (target/debug/deps/lang_tests_release-bff42339b6b0d08f)

running 21 tests
test lang_tests::fun_ptr.rs ... ok
test lang_tests::static.rs ... ok
    ⋮

test result: ok. 21 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

Addressed your feedback in 63ffdfdd1776afa2e82bbd3d2ff8ff7b7f0d5b67.


⚠ jaswdr.dev has degraded performance

In f19f0ca, jaswdr.dev (https://jaswdr.dev/) experienced degraded performance:

HTTP code: 200
Response time: 668 ms

Resolved: jaswdr.dev performance has improved in b48e288.


Auto-generated form drop-list width

This width seems to be computed from the longest item in the list.
For long lists, the vertical scrollbar appears, and causes the horizontal scrollbar appear unnecessarily.
Would it be possible to increase the computed width by scrollbar width?

Good observation, I have noticed it too. Will fix.

Fixed this issue in the current master. There was already logic to adjust width based on scrollbar. I have just fixed it. I will keep an eye on it to make sure it works without unnecessary scrollbars.


Kitchen Sink : Pokemon App

Describe the feature
This app will show all the pokemons and their stats and Evolutions.
Please assign me this for the hacktoberfest_2023
Additional information

[x] Would you be willing to help implement this feature?

Go ahead @sushilmagare10!

Any updates on this @sushilmagare10?


Share code with https://github.com/jrmuizel/etw-profiler?

I just found out about blondie and thought you might be interested in some parts of https://github.com/jrmuizel/etw-profiler/tree/main/etw-gecko
It consumes ETW files and produces profiles like: https://share.firefox.dev/32sqKU4
It uses https://crates.io/crates/pdb-addr2line instead of dbghelp to symbolicate.

etw-gecko looks really interesting! Thanks for mentioning it. Do you know if https://profiler.firefox.com is meant just for firefox/webapps (Like the landing page implies), or if it is intended to be used with arbitrary native applications?
The symbols are being resolved in profiler-symbol-server/profiler-get-symbols, right? I really like how it seems to have support for symsrv symbol servers!
Currently a big part of the code for blondie is related to manually starting an ETW tracing session and getting the events. etw-gecko instead requires using xperf for trace collection, and then processes the etl file. I wanted to avoid depending on xperf for this project to make it usable without installing the Windows Performance Toolkit.
Getting rid of the dependency on dbghelp.dll would be nice, although not something I'm too worried about. Was the reason you avoided it the requirement (As mentiond in https://docs.rs/profiler-get-symbols/0.14.0/profiler_get_symbols/ ) to run in WASM? dbghelp seems to work fine in my tests, and it was pretty easy to get up and running.
Someone else suggested I could use the DIA SDK , but a rust-native solution like pdb-addr2line does look like a better match.
My only problem would be mapping the image files to PDB's, since I'm not using xperf I don't get DbgIDRSDS events with the PDB names. I'm not too sure but maybe I can just use the https://crates.io/crates/object 's pdb_info method on PE files.
All in all, I'm not sure there's much code that could be shared with specifically the code in etw-gecko (Because I'm not using xperf), but I could probably use object/pdb-addr2line . The one thing that did look useful to me is the symrv env var parsing/lookup/download code in profiler-symbol-server in symsrv.rs

Do you know if https://profiler.firefox.com is meant just for firefox/webapps (Like the landing page implies), or if it is intended to be used with arbitrary native applications?

Supporting arbitrary native applications was one of my eventual goals when I started the project, and the current maintainers are also happy to accept patches to serve that use case. For example, here are some issues I filed to polish off some imperfections you run into when using it with non-Firefox profiles.

Currently a big part of the code for blondie is related to manually starting an ETW tracing session and getting the events. etw-gecko instead requires using xperf for trace collection, and then processes the etl file. I wanted to avoid depending on xperf for this project to make it usable without installing the Windows Performance Toolkit.

etw-gecko only requires xperf at the moment because that was the fastest way to get something working. Removing the xperf dependency is definitely a goal. There's some code to make this work but it's not very far along yet, as far as I know.

Getting rid of the dependency on dbghelp.dll would be nice, although not something I'm too worried about. Was the reason you avoided it the requirement (As mentiond in https://docs.rs/profiler-get-symbols/0.14.0/profiler_get_symbols/ ) to run in WASM? dbghelp seems to work fine in my tests, and it was pretty easy to get up and running.

No, WASM was not a concern - we just ended up reusing profiler-symbol-server, which I had originally created for the macOS profiler I'm working on, and which just happens to reuse code I had written for Firefox profiling.
pdb-addr2line gives you inline stacks and good performance. I invite you to use dbghelp or the DIA SDK on a big binary like Chrome or Firefox - you'll probably end up waiting a long time for your symbol information. pdb-addr2line has been optimized for those big binaries.

My only problem would be mapping the image files to PDB's, since I'm not using xperf I don't get DbgIDRSDS events with the PDB names.

Oh, those events are synthesized by xperf, and can't just be enabled by setting the right flags when starting the trace? I wonder if there are other events like that that we would need to emulate.

I'm not too sure but maybe I can just use the https://crates.io/crates/object 's pdb_info method on PE files.

Yes, definitely, that works. It gives you the path, the guid and the age.

The one thing that did look useful to me is the symrv env var parsing/lookup/download code in profiler-symbol-server in symsrv.rs

That part would ideally live in its own crate. Would you like to extract it?

pdb-addr2line gives you inline stacks and good performance. I invite you to use dbghelp or the DIA SDK on a big binary like Chrome or Firefox - you'll probably end up waiting a long time for your symbol information. pdb-addr2line has been optimized for those big binaries.

Ouch, I didn't know they were that slow.

Oh, those events are synthesized by xperf, and can't just be enabled by setting the right flags when starting the trace? I wonder if there are other events like that that we would need to emulate.

That's my understanding, based on this perfview comment
https://github.com/microsoft/perfview/blob/6a02a08e0b31531de6933304fcab18be52a6708d/src/TraceEvent/Parsers/SymbolTraceEventParser.cs#L11-L20
    /// Kernel traces have information about images that are loaded, however they don't have enough information
    /// in the events themselves to unambigously look up PDBs without looking at the data inside the images.
    /// This means that symbols can't be resolved unless you are on the same machine on which you gathered the data.
    ///
    /// XPERF solves this problem by adding new 'synthetic' events that it creates by looking at the trace and then
    /// opening each DLL mentioned and extracting the information needed to look PDBS up on a symbol server (this
    /// includes the PE file's TimeDateStamp as well as a PDB Guid, and 'pdbAge' that can be found in the DLLs header.
    ///
    /// These new events are added when XPERF runs the 'merge' command (or -d flag is passed).  It is also exposed
    /// through the KernelTraceControl.dll!CreateMergedTraceFile API.

Although it seems there's an API to do that (And some other post processing like mapping paths like “\Device\HarddiskVolume1\” to the MS-DOS paths like “C:\”), Although it seems to take in a filename, meaning I don't think it can work from an in-memory trace.

Would you like to extract it?

Yes, would it be ok if I send a PR to https://github.com/mstange/profiler-symbol-server splitting that part into a separate crate with a cargo path dependency, and you can then publish that yourself (So you keep ownership) and once published you can change the path dependency to a crates io version?

Sure, that's a good idea!

That's my understanding, based on this perfview comment

Ah, interesting.

v0.4 uses symsrv, object and addr2line 🎉


Discuss reporting of bugs in gov.uk services by the public

Topic Suggestion
I think that it would be great and beneficial to the public if Cross Government Software Engineering Community discussed ways to enable adding the public's voice to the discussion. One area the public would greatly benefit from is bug reporting. It might be a good one to kick start the software engineering meetings, which seem to have died in 2022.
Problem
The current implementation of reporting problems with services seem to be aimed at making it as difficult and off-putting for the public to engage as possible. The engagement includes reporting issues interfering with the end user's experience with the services.
Let's look at an example. Apply for a passport is supposedly a BETA (=new) service that doesn't need much explaining. Any comments from the public are gathered through this feedback form. That form is a one way route that offers no acknowledgement of the receipt of the feedback, no reference number, no nothing. Its use is actively discouraged through a banner stating:
We can’t reply to your message, but feedback helps improve the service

The user has no idea what happens to their feedback, whether it's been read by anyone or even reached anyone.
However, this is probably meant for general comments or maybe things like enhancement requests, although submitting the latter is kind of pointless since no transparent conversation of any sort follows.
What is then available to the user if something goes wrong with the service? Nothing really. The only other way to reach someone is to use the contacts listed on the Help page. The contacts there are not meant for technical support though, they are to "Get help with my application.". We will still try to use it later, though.
Example Issue
Here is an example issue that's been affecting the service since at least 6th of July 2024 (at the time of writing is 9th of July), that's when I first encountered it. It's possible the issue has been around for much longer than that since there is no way to report it and as a result no engineer is probably aware of it. If they were the service may have been taken offline or a warning displayed, one would expect so anyway.
The issue happens when the user chooses the "I’ll take or upload a digital photo" option during the passport application process. On the "Upload your photo" form they select a picture to be uploaded and after "Processing image" runs the service fails with the following error:
Something went wrong

To keep this service secure, any information you’ve entered hasn’t been saved.

You’ll need to start again.

The message leaves the user non the wiser as to what went wrong. The UX here is particularly bad, with the long form getting cleared and the user forced to start from scratch. Digging deeper, not something a regular user of the service would do, reveals  504 Gateway Timeout error code and also x-cache Error from cloudfront response header which may or may not have something to do with the failure.

Calling the helpline is of no use, the operator is adamant that the service works and suggest using a photo booth instead.
That's it when it comes to the assistance available to the user. Meanwhile the service remains broken. Or so it appears as it could be that the service is running so no one gets alerted to the problem and the actual issue is a poor input validation feedback. Who knows, it doesn't matter since none of that is communicated to the user, only that "something went wrong".
For the user it's a dead end. They can fall back on the inconvenient options such as using the photo booth, with no guarantee that the photo will be accepted despite such claims made in those booths, or pay extra and use a paper form instead. There is also an option to complain but (a) it doesn't solve anything, (b) it takes up to 3 weeks for the initial response. That's a pretty poor SLA imposed on the customer and they have no other choice than to put up with it since the arrangement is basically a monopoly as they cannot go and have a passport issued by a different provider.
Those fallback options wouldn't be necessary if there was a way to report the problem to the right person but either through the lack of foresight or a deliberate introduction of barriers this is not offered.
Incidentally, jokes like the x-clacks-overhead GNU Terry Pratchett response header are only funny when the software works. When it doesn't they become jarring, as in, shouldn't the effort have been put into the actual software that's now broken rather than making silly jokes?
Final Thoughts
A bug reporting facility that is open to the public is a powerful and free resource that can be very effective in improving software and as a result the end user experience and satisfaction. Yet, gov.uk brand doesn't take advantage of it without offering any good alternative.

Why is that and can it be done in a more user friendly way?
These are services that everyone will use at one point or another, how are the engineers who implemented the services are planning to report problems when they themselves become the end users? Have they put any thought into that? Are they happy with the options that are available?
Can the barriers to discussion be removed and a lightweight solution that offers bi-directional communication put in place instead?
Should the existing helpline operators have access to the technical people in charge of maintenance of the product they help with?
Could a 'public request' category be created in the existing ticketing system with the ticket id's that can be shared with the public reporting the issues for future reference?

These are some questions to kick start the discussion. It would be great to learn the outcome of it once it happens. Also, if anyone from the passport application service could look into the problem given as an example here that would be great.

thanks for raising the issue ... will discuss at our Lean Coffee  sessions


feat(linter/tree-shaking): support JSX

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3139  👈
#3138
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mysteryven and the rest of your teammates on  Graphite

Merge activity

Apr 29, 11:23 PM EDT: @Boshen started a stack merge that includes this pull request via Graphite.


Error on WordBeamSearch Inference

Hello,
I am encountering some problems in running the code in Inference Mode and specifically with the decoder type 'wordbeamsearch'.
I have successfully cloned the repository CTCWordBeamSearch (https://github.com/githubharald/CTCWordBeamSearch) and the test script attached correctly runs.
However, when I try to make inference on the provided test image (word.png) with the given net weights, I get the following error:
Traceback (most recent call last):   File "main.py", line 194, in <module>     main()   File "main.py", line 190, in main     infer(model, args.img_file)   File "main.py", line 133, in infer     recognized, probability = model.infer_batch(batch, True)   File "/opt/notebooks/HTR/SimpleHTR/src/model.py", line 294, in infer_batch     loss_vals = self.sess.run(eval_list, feed_dict)   File "/root/.conda/envs/MXNetGPU/lib/python3.7/site-packages/tensorflow/python/client/session.py", line 968, in run     run_metadata_ptr)   File "/root/.conda/envs/MXNetGPU/lib/python3.7/site-packages/tensorflow/python/client/session.py", line 1167, in _run     (np_val.shape, subfeed_t.name, str(subfeed_t.get_shape()))) ValueError: Cannot feed value of shape (0,) for Tensor 'Placeholder_1:0', which has shape '(None, 2)' Segmentation fault (core dumped)
Please notice that there may some inconsistence on the code lines number only due to a few simple comments I have written in the script.
Can someone provide me some hints to solve this problem?
Thank you in advance.
Regards
Jacopo Paris

this already happens before even calling the word beam search decoder.
Can you checkout exactly the code version as it is on github and then just run python main.py?

Thank you Harald,
I have recloned the repository from scratch and now it seems to be working!
Thank you for the support.
Jacopo Paris


Added Google Firebase Authentication setup#33

Issue Number
#33

Describe the changes you've made
Installed And Setup google firebase auth for login
Describe if there is any unusual behavior of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

@nanda-mik Pls Review if any Changes need pls Let me know
and I have made Setup For auth.

@pixan198 Pls review if you are free

@Priyanshu-official it was under review please wait a bit!!

@nanda-mik ok

@nanda-mik let me Know if have to add get element too..

Hey @Priyanshu-official
Until @nanda-mik is reviewing your PR, Please resolve all the merge conflicts with your code

@pixan198 Thank you reply
I am also not able to understand the conflict. can you help me to find when you have time
till then I am trying to find.

Yeah @Priyanshu-official
For the index.js file you don't need to change anything as I see the code
And for the App.js file, can we have a seperate component which deals with sign in and sign up only
I mean it would be good right?

@pixan198 Ok you are right
I will do it!


[CUBRID] Display Partition Tables

Make a folder Partitions under table and display all partitions which is related to each table.

error is shown on attempt to open partition tab

Test table
CREATE TABLE participant2 (
    host_year INT,
    nation CHAR(3),
    gold INT,
    silver INT,
    bronze INT
)
PARTITION BY RANGE (host_year) (
    PARTITION before_2000 VALUES LESS THAN (2000),
    PARTITION before_2008 VALUES LESS THAN (2008)
);

error is shown on attempt to open partition tab
Test table
CREATE TABLE participant2 (
    host_year INT,
    nation CHAR(3),
    gold INT,
    silver INT,
    bronze INT
)
PARTITION BY RANGE (host_year) (
    PARTITION before_2000 VALUES LESS THAN (2000),
    PARTITION before_2008 VALUES LESS THAN (2008)
);

Thank you for checking!
I found out, It happens on server 11.0. Because it doesn't support multi schema.
I will fix it soon.

verified


Sintayew4 wish to run a sealer node

Fill this form to declare your intention to becoming a node operator in the production network
Company

Full legal name: __sintayew Gashaw
Company website: ___http://www.sintayew4.com
Representative contact name: ___sintayew
Representative contact email: ___sintayewgashaw4<EMAIL_ADDRESS>
I hereby recognize that our company does not already have a Node in this network
Sealer Node

Sealer address: <0x1244...0087> (ensure you have the ownership and safekeeping of the private key)
Link to your website to prove that your company owns this address

Motivations
Explain your reason for joining the network

@Sintayew4
Thanks for the interest but your company name cannot be identified and you have not stated your motivation for joining
Also I recommend that you first join in Kerleano testnet
Without proper reply this request will be rejected
Regards


Very minor typo in readme

In the examples section:
<app-datepicker><app-datepicker>

should read instead
<app-datepicker></app-datepicker>

@santanubasu Thank you for reporting!


add video demo of temporal web

i have often wanted this in docs when explaining to candidates - what do you think?

doesn't hurt to have. Should we also consider embedding a simple animation in README?

it would take some time to make that animation...


feat(sfu): add join config to signals

At the moment, it is not possible to use new configs (https://github.com/pion/ion-sfu/commit/54cd1c2d6132b0ba9ec4cf8a59b85f6013902015) with signal servers

supported in sfu #529
need https://github.com/pion/ion-sdk-js/issues/180

Mark:https://github.com/pion/ion-sdk-go/pull/33


Verification of spied methods fails in PowerMockito

From<EMAIL_ADDRESS>on September 04, 2009 08:16:26
See description in code
Original issue: http://code.google.com/p/powermock/issues/detail?id=147

From<EMAIL_ADDRESS>on September 19, 2009 12:38:28
Status: Fixed


Allow arithmetic between single/double and decimal values.

Adds support for +, -, *, /, % and unit tests for all permutations.

this comes up a lot.
the way I understand the original intent of the creator, is that NCalc should behave as .net.
in .net you cannot use a decimal and a floating point number in the same operation.
The error is the same as in NCalc
e.g. Operator Y cannot be applied to operands of type 'decimal' and 'double'
For this reason I dont think I will merge this PR.
The user should decide "a priori" if the calculations will be done with decimal or floating point numbers

If that was the original intent, the author was not consistent with it. You can see in Numbers.cs;9b27146b, the conversion was already taking place for some operators (addition of Single or Double to a Decimal and vice versa). They even created a test (ShouldAddDoubleAndDecimal) to ensure that you could add the two together. I figured they just abandoned the project before finishing up with the rest of the operators.
So right now, you're in this weird state where you can only add together a single/double and a decimal.

you are correct. Let me think about it

If you do decide to incorporate this change, would you be able to roll out a new nuget package? I'd love to use this within my project rather than copying the code over.

Yes. Should this be a minor or a patch?
--
Pedro Marques
No dia 16/06/2016, às 23:48, Mikey Cooper<EMAIL_ADDRESS>escreveu:

If you do decide to incorporate this change, would you be able to roll out a new nuget package? I'd love to use this within my project rather than copying the code over.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

No new APIs at all, so I'd say patch.

I've decided to include this PR in the source.
The current state is not consistent and a Convert.ToDecimal seems reasonable to me since it was already there for add operations.
Thanks for your contribution
Thanks for all the tests


Revisit error validation messaging (maybe ranges themselves, too)

Some discussion here: https://github.com/sleepepi/sleepinnovate.org/issues/30
Right now an error on a numeric field might look like:

We are moving more and more toward having direct participant entry, so validations like this might confuse more than help in terms of letting people what they need to do next. Perhaps we could eliminate both the tooltip that displays range and rewrite the error messaging to include the range within the message, e.g. "Please enter a value between 80 and 200."
We would want the messaging to be understandable by study staff and participants alike. There are lots of different permutations of range checking, which might require different sorts of phrasing and error messaging. These could be gone through 1-by-1.
Other potentially related issues/discussions to have down the line:

Ranges? Do we want soft and hard ranges long term? Or only one range?
A possibly neat innovation for a date range would be to allow "today" to be set as a minimum or maximum. If I have a variable like, "When were you diagnosed with insomnia?" the logical maximum is the current date (the minimum is harder to discern). People might run with this sort of request and then want to do "today + 10 days".

One issue to think about with the "today" as a range would be someone updating the sheet later, would it enforce the new date? Validate against when the value was entered initially, or when the sheet was created?

Also, just a note the validation passes back two values, a display of the value with units and an optional error message

One more thought, I've been wondering if the error message should replace a field note, instead of pushing it down.

Hm, yes, I hadn't thought of how such a validation might work for both the initial creation and later editing!
Replacing the field note on error seems like it would be worth trying.

Thinking about this a bit further, we COULD have the following setting for date variables, that wouldn't break as the date changes: A checkbox that says "Disallow future dates".

"Disallow future dates" seems pretty clear/concise.

Disallow future dates issue now here https://github.com/remomueller/slice/issues/734.


microsoft visual c++ runtime library

Godot version
Godot_v4.0-beta1_win64.exe
System information
Intel Core i3 2.20GHz RAM 8GB Windows10 64bits Video: GT520 1GHz.  Although GLES3 works. In godot3.5.1 I use GLES2 to avoid problems. But the error happened in godot4.0b1.
Issue description
I usually use version 3.5.1 downloaded by Steam which by the way works very well. But today I decided to test "Godot_v4.0-beta1_win64.exe" and it didn't open. Just got an error screen: "microsoft visual c++ runtime library". Image link: https://photos.app.goo.gl/uM38eM7S5J6a22Ao8
https://photos.app.goo.gl/NTsZ518nGccr9VG18
I clicked the "Repetir" button (Retry button). and then Godot closed.
I made other attempts and they all failed.
I reinstalled the "microsoft visual c++ runtime library" package but the error remains.
If anyone here knows of any extra procedures involved in the damn microsoft librarian I would be grateful to know.
I hope this error can be fixed in the future. Thanks!

Steps to reproduce
I downloaded godot4 from the link:
https://downloads.tuxfamily.org/godotengine/4.0/beta1 from page: https://godotengine.org/article/dev-snapshot-godot-4-0-beta-1#downloads
unzipped in windows 10
I ran the "Godot_v4.0-beta1_win64.exe" file
and the error screen appeared
reinstalled the microsoft package
https://www.microsoft.com/en-us/download/confirmation.aspx?id=102159
restart the computer
I ran the "Godot_v4.0-beta1_win64.exe" file again
and the error screen appeared again
Minimal reproduction project
Godot4 did not open.

There is a hint in the popup, and it says more in the console: your graphics card seems to be too old to support Vulkan. I checked here and it says your card model Vulkan support is N/A. Other sources like here or the absence of your card in this page suggest the same.

Duplicate of #58927 (same cause, that bug tracks how it should be resolved).

Thanks for the answer!


Display authors on the posts index and give them some more visibility

This change moves the information about post authorship to the post's
Front Matter instead of having it as a hardcoded paragraph at the end.
This allows programmatic access to the authorship information, which
we then use to display author names in the post index and more
prominently next to the post title.
To ensure author details are always consistent (name and URL), this
moves all people data to a table in _config.yml and references it
using usernames from the Front Matter.

Sample preview of how this looks like:
Posts index: https://screenshot.googleplex.com/8yxPF5H9Zds
One complex post with more than one author: https://screenshot.googleplex.com/T7cCAZvPoXT


Nominate Han-Ru Chen (Future-Outlier) as Flyte committer

Han-Ru has consistently demonstrated exceptional dedication in contributing to the Flyte project, making invaluable contributions to the community.
Highlighted contributions

ChatGPT agent
Databricks agent
Sync plugin
FlyteChatGPTSummaryBot
Email notification in the sandbox
Improve test coverage

https://github.com/flyteorg/flyte/commit/033276d42d00e508d9e66299683a5ea18efaec2f
https://github.com/flyteorg/flyte/commit/ac4ec7d84e88fe8a00a4b850c30a078deaee7f3e

Added Flyte sensor example.
Added GPU enabled sandbox image
Tons of PRs to improve the Flyte agent
Tons of PRs to improve the Kubeflow plugin
Flyte blog, tutorial, video.

Developing an LLM app with Flyte Agents

More detail

https://github.com/flyteorg/flyte/commits?author=Future-Outlier
https://github.com/flyteorg/flytekit/commits?author=Future-Outlier

Have you been an elected Contributor for at least 6 months?
flyte (monorepo), flytesnacks
What SIG/WG are you part of?
Any other context you wish to share?

+1
Long overdue :)

+10. i assume you mean "core contributor" in the ladder. should we replace "committer" with "core contributor" to clarify the term?

+1
Thank you for all the work, @Future-Outlier. You've been an amazing contributor to Flyte. I hope we get to work a lot more in the future!

+1 <3

+1, thanks for all your contributions!

Approved!


Arrays not working in start/end buttons properties

Using A-Frame 0.9.1 with super-hands 3.0.0
Passing a single value to grabStartButtons works fine. Passing a schema array does not.
Example:
This works:
super-hands="grabStartButtons: triggerdown"
This does not:
super-hands="grabStartButtons: triggerdown gripclose"
When you attempt to use an array no buttons are mapped and grabbing no longer works

Array values need to be comma separated
https://aframe.io/docs/0.9.0/core/component.html#schema_array

Yep, I was checking for gripclose instead of gripdown, thanks for the great library!


Add task to refresh migration DB from production

Context
We have amigration environment that we will be using for the ECF -> NPQ migration testing. Prior to running this in production we want to test it in a production-like environment.
Changes proposed in this pull request

Add task to refresh migration DB from production

Add a GitHub action to allow us to populate the migration database from the production database. Update Makefile to include command to connect to the migration DB.
Guidance to review
I've ran the GitHub action manually as part of this PR so the migration DB is populated. You can connect to it with the new make commands:
make migration konduit

You can also connect to the snapshot DB:
CONFIRM_PRODUCTION=yes make production konduit-snapshot

might be able to port the same logic to the make file you did in ECF, but not a blocker :)

Doh! I did, I just hadn't pushed it 😞


Unable to log in. (Udemy for business custom company domain)

Hi,
I am completely unable to log in. It is probably due to the account being Udemy for business, and on a company domain. "companyx.udemy.com". Is there perhaps a workaround?
I keep getting either "Invalid Username/Password" or "Connection Failiure".
Yes, I am on the latest version. (1.6)

Hello @GabrielCoetzee What do you type in Udemy Business Name field?
It should only be the name not entire domain name. For example for apple.udemy.com, you have to type apple only.

Hi @FaisalUmair ,
I am only typing in the "company" part in "company,udemy.com" as you indicated.

Few days ago, I tried creating a trial business account for testing the app against it.
But it appeared to work fine, I don't know if a real working business account is different from that. But I guess it should work for real business account if it works for trial account.
And I also found that Udemy creates two separate account for you, one associated with Udemy business and other with normal Udemy website. Did you try skipping the Udmey business field? In order to check if the normal account works.

I did try that too. I`ve also tried a variable of lowercase and uppercase characters in both my username and 'Udemy Business' field in case it is case-sensitive, but also no success.
I think something must be different with my Udemy For Business account then. It is company provided, so I can`t use my credentials to log into normal "www.udemy.com". I have to log in through my browser at "company_name_omitted.udemy.com". It only works there.
Should I maybe try also creating a 'normal' udemy account, using the same credentials as my 'Udemy For Business' account, and give that a shot? Maybe both are required for it to work?

Ive also tried both the "x86" and "x64" versions of Udeler, as well as running in elevated mode vs not running in elevated mode. Doesnt seem to have an effect,

Yeah, just try to create a normal account. If the normal account doesn't work for you. Then the login is not working for you at all.
Since you mentioned that sometimes you get Connection Failed error, it makes me think it could be connection related issue. Are you using a proxy server?

I can`t create a normal account since the email address is already in use.
No, I`m not using a proxy server.

Just create a account with another email address, in order to check if the login works for you using normal account.

I have same issue. I can log in with my normal account, but not with my company. Previously worked very well, I even managed to download some courses. Also when I'm with my normal account in the last version 1.6 it not listing the courses I have in my account.

Hi @FaisalUmair - I created a normal udemy account, and I can successfully log in. However, my 'Udemy for Business' account obviously still persists.
Hi @donjordano - Can you please elaborate? Are you saying uninstall the old version and do a clean install of the new version? (I`m on Windows 10)

Well, I am trying with my business account and it appears to work fine. If I enter correct username and password but incorrect business name, it still logs in but shows no course. But when I login with the same credentials and type correct business name it shows me courses from my business account. If I uncheck/skip Udemy business, it just login in to normal account.
The login function since v1.5.4 has changed. So that could interfere with business account. It could also be possible that your business account and the trial business account I created are different.
Now in order for me to fix this, I would have to reproduce this issue on my side. If anyone here can share their credentials (through email), which work on Udemy website but not on Udeler, I might be able to resolve this issue.

Hi @FaisalUmair,
Yeah there must be something different with my account vs your trial business account then. There is clearly no normal udemy account linked to mine so no login works for me, business or otherwise. Whether I enter company name, no company name, uncheck "Udemy Business?", I`ve never had a successful login.
I completely understand that in order to fix it, you need to reproduce the issue. However, my udemy for business account is provided by the company I work for and providing you with the credentials would be the highest possible breach of security. I understand this means we are at a dead-end, but its fine. Ill have to download the videos one at a time with a chrome extension. :)

@GabrielCoetzee I understand the issue with sharing credentials. Unfortunately for issues related to Udemy account and specific paid courses, I would obviously need to have access to reproduce.
I will see what else I can do about this.

So as far as I could debug I have enabled the devtools in the app. It seems the authentication endpoint in the app is POST https://www.udemy.com/api-2.0/auth/udemy-auth/login/?fields[user]=access_token, but in my business page is POST to https://<business>.udemy.com/join/login-popup/.

@jcortejoso  I received your email. Thanks for providing you credentials to debug.
I will explain the difference in endpoints, https://www.udemy.com/api-2.0/auth/udemy-auth/login/?fields[user]=access_token is used to authenticate no matter what your account type is.
Now for retrieving the course if we use https://www.udemy.com/api-2.0/users/me/subscribed-courses?page_size=1 it will try to get courses from normal Udemy account, but since your account is a business account you need to request https://sngular.udemy.com/api-2.0/users/me/subscribed-courses?page_size=1. This functionality is already implemented in the app.
I was able to login and view courses from you business account. if you check the is_business checkbox and enter sngular in the field you will be able to view your courses. However if you leave it blank or don't check is_business at all, you will still be able to login but it wont show you any courses as your normal account doesn't have any courses.
Let me know if that is also the case with you.
And others please let me know what error you are getting and make sure you are entering correct business name in the field.
I understand that the business login in the app is quite deceiving sometimes you think you were able to login because you already typed correct details. But as I mentioned before you will still be able to login even if you enter incorrect business name but correct credentials.

I'm getting the same issue. Incorrect username/ password. But actually, it's correct.

Same issue here

He has not solved this issue?

Hi @akhilpadgilwar No one supplied me with their login credentials to debug the issue. I requested it multiple times. I got one email recently asking if I need the credentials, I replied to it, but got no response.
And your business account seems to use a different login technique than other accounts. (I am not sure about this), but if you can also send me the credentials, I might be able to get some clue.
I will mention this again, I tried to create a business account and it worked for me. So whoever is facing this issue and want the issue to be fixed, kindly email me your login credentials so that I can debug the issue.
And I am not able to work on the issues raised here, due to my busy schedule :( I hope to get some free time soon, to work on a new update and fix majority of issues.

Hi @theWebArtisans!
what is supposed to be written at Business box? just "myCompany" or "mycompany.udemy.com"?
Thanks in advance!

hope you get back to us with good news for udemy business accounts

When I am going to log in with the email for the udemy business it is opening another window and when I returned to the previous screen it again opened another window. This continued over and over again. Hence I couldn`t just type the email.

Ok go to udemy right click or F12 for Inspect
Choose Application tab, then choose Cookies
Take access_token form there
https://user-images.githubusercontent.com/50474221/57762361-e6eb8700-76ee-11e9-954f-e75dfe0cab5d.png
Whşle loggin in udeler use Access Token and paste it
https://i.imgur.com/nITcv04.jpg

So as far as I could debug I have enabled the devtools in the app. It seems the authentication endpoint in the app is POST https://www.udemy.com/api-2.0/auth/udemy-auth/login/?fields[user]=access_token, but in my business page is POST to https://<business>.udemy.com/join/login-popup/.

Hello dude can u say how u ssl unpinn udemy app plz


Long term access token

How do I get long term access token or is it possible to extend my short term access token to long term tokens? Please let me know.

Closing because this is not an bug with the SDK. Please send support questions to the Google Group or Stack Overflow.


[AMBARI-23666] Hive Summary page does not have the correct 'Go To Vie…

…w' links for Views.
What changes were proposed in this pull request?
Implementation of "goToView" function should include:
    if (event.context) {
      window.open(event.context.get('internalAmbariUrl'));
    }

How was this patch tested?
Manually Tested.
The Build was successful:
[INFO] Ambari Main ........................................ SUCCESS [  1.174 s]
[INFO] Apache Ambari Project POM .......................... SUCCESS [  0.013 s]
[INFO] Ambari Web ......................................... SUCCESS [01:52 min]
[INFO] Ambari Views ....................................... SUCCESS [  1.921 s]
[INFO] Ambari Admin View .................................. SUCCESS [  7.818 s]
[INFO] ambari-utility ..................................... SUCCESS [  3.477 s]
.
.
[INFO] Ambari Functional Tests ............................ SUCCESS [  0.414 s]
[INFO] Ambari Agent ....................................... SUCCESS [ 50.632 s]

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1954/
Test PASSed.

@atkach @adoroszlai  can you please review this Pull request.

Closing PR as it is over 6 months old.


Create a callback to remove all notifications from the screen

It would be nice to a callback to remove all notific8 from the screen

I am using this way:
$('.jquery-notific8-notification').remove();

Is this for just the visible notifications or for the plug-in overall?

Visible only

A new function called remove will remove all visible notifications on the screen instantly while leaving the plug-in's foot print. It can be called with $.notific8('remove');.
The destroy function has also been completed. That will remove all of the notifications and the plug-in's foot print. It can be called with $.notific8('destroy');.


Diffrent line collors

Hi
Thanks for your great work.
i was wondering if there is away to have different colors for the line (before and after the Grip)

Thank's allot
Avi

You can now add a second color to the curve.color value to add a color to either side of the grip demo:
curve: {
  width: 2,
  color: ["#f00", "#00f"],
  cap: "round"
}

For additional tweaks, a drawCanvas callback was added. A code example can be seen in issue #8.


chore: release 0.4.2

:robot: I have created a release beep boop
0.4.2 (2024-11-02)
Miscellaneous Chores

release 0.4.2 (356b8e9)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.4.2
:sunflower:


Not convenient to import a sql database since you must specify the RequestedServiceObjectiveId, which is more of an internal id.

Importing a Database resource fails, since a matching requestedServiceObjectiveId: "" setting is missing and is flagged as deleted. This should not be required to be set, since what you really want to configure is the RequestedServiceObjectiveName and the corresponding RequestedServiceObjectiveId should be inferred from that. Not sure if this only relates to import or if the Id must also be specified when applying resource without ImportId specified. But the RequestedServiceObjectiveId should never be required to be specified.
http://blog.itaysk.com/2015/09/04/create-sql-db-from-arm-template-requestedserviceobjectiveid

Could you please add a repro? Thank you.

I just updated the description, since I was a bit wrong in my first statement.

Thank you for the clarification. Does that also mean that you can change the title from "not possible" to "not convenient" or something like that?
We generate resource models from API definitions and I see nothing in the API definition to make such inference a simple task for automation...

Wouldn't it be possible to see how Terraform does it, since you do not need to specify the RequestedServiceObjectiveId there.

Our current policy is to fully rely on Azure Open API specs and automation to decide which properties are needed and which not, with no manual overrides or helpers. It has upsides and downsides but it's unlikely we'll change anything manually here.
Broader improvements regarding import strictness are tracked in https://github.com/pulumi/pulumi/issues/5878, so I'll go ahead and close this issue for now.

Our current policy is to fully rely on Azure Open API specs and automation to decide which properties are needed and which not, with no manual overrides or helpers. It has upsides and downsides but it's unlikely we'll change anything manually here.
Broader improvements regarding import strictness are tracked in https://github.com/pulumi/pulumi/issues/5878, so I'll go ahead and close this issue for now.


How to know if the menu is already open ?

Hi,
Thanks for your circle menu !
Is it possible to know the status (open or close) of the menu? If it's possible, what is the best way to find out if the menu is already open or not?
Minipomme.

Hello, @Minipomme. There is no such feature. You can use EventListener. Look closer to the sample app.


(PDB-5482) Log queries before parsing

Adds query logging before parsing so parsing failures are easier to debug. Please see commit messages for more info.

Rebased onto origin/6.x recheck CI

A single travis test is failing because it's "exceeding maximum log length" so I'm ignoring that one -_-

that sounds reasonable to me. We should probably look to rein in the log length of puppetdb tests eventually.


gapi.load callbackOrConfig missing "config" property docs

callbackOrConfig argument is documented here
But as you can see in the source this object may also have config property. And config property may also have update property.
Can anyone add documentation regarding this, please?

@grant could you, please, take a look at this issue?


Fix for issue 855

Hi team,
This PR is to solve the issue #855 (Request Body Internal Ref Not Resolved)
Please review the PR and merge the same.
Thanks,
Mohammed

@ymohdriz thanks!


Update README.md

Ticket
Link to Github Issue
Problem description
Provide context for the problem.
What's changed
Describe the approach used to solve the problem.
Summarize the changes made and its impact.
Checklist

[ ] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Thanks @tt-aho let me work on them


Production ready?

I fell in love when I started using this. Would you say it is still in a POC state or could you reliably use it in an actual project?

I see it as a little more than a POC but, to be honest, it's only 34 commits and has still to be proven in production. I documented some missing features but I do not know of bigger problems or issues that would rule out production usage.

Is this still good to go?

Don't use it in production! @wtfiwtz documented current shortcomings on #8. (Thanks a lot!)


Update docker scripts

About this change - What it does
Provides a script that allows users to build and deploy docker images locally of latest changes.
It also provides the ability to deploy a kafka and schema registry locally for testing and validation.
Resolves: #xxxxx
Why this way
It should allow developers to work more quickly and flexibly without sharing environments and resources.

maven spring plugin mvn spring-boot:build-image kind of does the same ?

maven spring plugin mvn spring-boot:build-image kind of does the same ?

Using spring-boot:build-image The klaw jar and api jar are ~100mb and 150mb smaller but they pull in a builder image a run image  that are an additional ~1.3GB. There is a slightly faster build time for recompiles using the spring boot method though.
We also still require the ability to deploy those images as it only builds them.

Thanks for the work! I was just doing a similar one yesterday.
It's really great to externalize the configurations since the default ones (such as writing klawprodb.*.db into /) might not meet security controls in some cases.


Function with no arguments (sometimes?) recognized as varargs function

I've been exploring AVBin as an alternative to libav.  One thing I've noticed is that some functions with no arguments there are recognized as varargs functions and skipped.
I'm a little confused why, as I did not see the same thing when wrapping libav (but maybe there are fewer no-arg functions).
I can get around this by disabling that check in wrap_c.jl, but there's probably a better fix.  Not sure when I'll have time to look into it.
For testing: https://github.com/kmsquire/AV.jl/blob/master/util/wrap_avbin.jl (and AVbin installed, of course).

If I had come across this issue a little earlier, that would have saved me some debugging work.
The code I wrote to handle this case is part of my pull request: dfdfbc0
In case you don't like my quick fix, some kind of handling of this would still be nice to have.

Already fixed in dfdfbc0, so closed this.


Remove RTD in favor of documentation download comment

ReadtheDocs does not work for us anymore (see #589). Here, I am removing the build and adding an (automatic) comment to the discussion for a download of the latest docs.
Contributor (creator of pull-request) checklist

~[ ] Tests updated (for new features and bugfixes)?~
~[ ] Documentation updated (for new features)?~
~[ ] Issue referenced (for PRs that solve an issue)?~

Reviewer checklist

~[ ] CHANGELOG updated with public API or any other important changes?~

I tested the code on a test repo and it seems to work.
The action will create a download link for the docs and (if executed) for the wheels.
If it is working and we are happy it may make sense to write an independent Github action. But not now. I had enough javascript for today...

Yes good improvements thanks!


Rename the config package to gitconfig

@charlierudolph I finally found some time to work undistracted on Git Town. I had to physically sit on a plane without wifi. I am sending a number of PRs with structural improvements.
This PR renames the github.com/Originate/git-town/lib/config package to .../gitconfig. This has several reasons:

config is a too generic name. I want to use it for configuration variables and structs later
this package really only is concerned with reading and writing configuration out of Git metadata
in the future we might allow providing additional configuration information via the command line

I don't like gitconfig since we have a git folder as well. I'd rather we just move the config file into the git folder.

Hmm, I don't understand your comment. Do you want to merge the git and gitconfig package into one called git? The former has mostly methods that I could see in the latter, like

HasRemote
GetMainBranch
EnsureIsFeatureBranch
GetChildBranches
etc.

Maybe we start by moving these methods into the git package?

Yes, I am saying merge the gitconfig and git pacakages.

All comments addressed


96 command wrapper

In short, this commit adds an initial wrapper around Katago that handles the stdin, stdout, stderr well.
It will need to be refactored a bit so it can be tested, but I wanted to get this in as-is for review.
Fixes: https://github.com/otrego/clamshell/issues/96

Branch has conflicts. Can you merge in upstream/master?


Implement GraphML output for graphs

This PR implements GraphML output for graphs. It is a XML based format, thus it add the new dependency on xml-rs, although I put everything behind feature flags.
This PR is not in a final state, as nothing is documented it. I will add some documentation, but I want to get some feedback on the API design first.
You can see some examples for usage in the included tests. The definition of the function which converts the weights (for nodes and edges) into attributes is a bit complex, but I don't really see a better way.
The functions have to be (PrintWeight type definition):
for<'a> Fn(&'a W) -> Vec<(Cow<'static, str>, Cow<'a, str>)>

The reasons for this are:

Attributes are arbitrary key-value pairs in GraphML, therefore the function has to return a vector.
It should be possible to generate the attribute lists, thus owned values have to be returned.
I wanted to avoid dynamic allocations if possible, otherwise a simpler Vec<(String, String)> would suffice as return type.

Right now there is a to_string() -> String and a to_writer(writer: Writer) -> Result<_> method to produce the output.
I'd love to get some feedback on this.

Hi, that looks useful. I would prefer if this was published in a separate crate. As in: building on top of petgraph, not being a part of it.

Oh sure, thanks for the feedback. I will put it in its own crate then. I thought it might be a fit, because petgraph already contains dot output functionality.


Use standard jsonio

merging now cause it is needed for browser-sol test to pass. (and tests are ok here)


Potential race condition in Summary window rotation

The summary.update() might be called from multiple threads.
So the following lines in Summary rotate() might have race condition due to currentWindow:
timeWindow[currentWindow] = new TimeWindow();
if (++currentWindow >= timeWindow.length) {

FYI @golgotha

I take another look at this, and I see now that current version Summary metric was a quick implementation to keep the parity, before implementing a proper fast algorithm for online quantiles. Probably makes sense to rather rewrite it all instead of trying to fix.

I take another look at this, and I see now that current version Summary metric was a quick implementation to keep the parity, before implementing a proper fast algorithm for online quantiles. Probably makes sense to rather rewrite it all instead of trying to fix.

I will think about after faster algorithm. Do you have some ideas?


Use contract function from Tasit SDK to transfer land

This can also be used to move it to the user’s wallet after having saved the address of the external, “proper” wallet in the Tasit app


Allow custom ffmpeg rtmp output

I want youtube-dl to push a youtube livestream via ffmpeg with rtmp module to custom rtmp url.

D:\Programs\ffmpeg\bin>youtube-dl.exe --verbose -f best "https://www.youtube.com/watch?v=VIDEO_NUMBER" --no-part --encoding -c copy -f flv "rtmp://MY_SERVICE/MY_KEY_HERE"

I get an error "LookupError: unknown encoding: -c"
Is there a way to work it arond and force custom ffmpeg settings inside youtube-dl?

That is not possible but you can extract stream URL with youtube-dl -g ... and then call ffmpeg yourself.

Ok, I managed to grab video only so far by:

ffmpeg -i https://manifest.googlevideo.com/api/manifest/hls_playlist/id/YkuAziUi_50.410/itag/96/source/yt_live_broadcast/requiressl/yes/ratebypass/yes/live/1/cmbypass/yes/goi/160/sgoap/itag%3D140/sgovp/itag%3D137/hls_chunk_host/r6---sn-jvhnu5g-bvwe.googlevideo.com/playlist_type/DVR/tx/962801/txs/962800,962801,962802/gcr/ru/mm/32/mn/sn-jvhnu5g-bvwe/ms/lv/mv/u/pcm2cms/yes/pl/20/dover/3/sver/3/fexp/9405973,9408497,9416126,9417157,9417703,9418223,9420452,9422596,9423348,9423661,9423662,9424135,9424336,9424489,9427902,9427959,9429245,9429739,9431012,9431047,9432583/upn/ZtSkTKEQdzo/mt/1458674497/ip/<IP_ADDRESS>/ipbits/0/expire/1458696677/sparams/ip,ipbits,expire,id,itag,source,requiressl,ratebypass,live,cmbypass,goi,sgoap,sgovp,hls_chunk_host,playlist_type,tx,txs,gcr,mm,mn,ms,mv,pcm2cms,pl/signature/150C9CDC228A9022DFD9D58EFBBFC0BEF0A20690.46201CB6626D06602895FC3B06E22DB8F137AEB7/key/dg_yt0/playlist/index.m3u8 -c copy -bsf:a aac_adtstoasc -f flv rtmp://premium.cybergame.tv:1953/premium/sneaky4oe?key=MY_KEY_HERE

But there are two flaws - there's no audio, and it plays stream since the beginning of vsh 4 hours.
So, please help to enable audio and grab it from realtime, not past, please.

Found a solution:

ffmpeg -i https://manifest.googlevideo.com/LONG_GOOGLE_URL/playlist/index.m3u8 -c copy -acodec aac -strict -2 -bsf:a aac_adtstoasc -f flv rtmp://MY_SERVICE/MY_KEY_HERE

Should manually encode to aac, but that seems to fix the issue. Thanks for help.

-acodec copy should be enough.

Ok, I'll have to polish it later. Thank you for sending me to "-g". :-)

Tested it. -acodec copy doesn't work. But that doesn't matter either.


Could't created apk Android for device armeabi-v7a

I Think it's impossible to create apk for device with ABI armeabi-v7a. I check into the ply store, at moment around the world there are 7K device with ABI aarch64  and 13K of Device with armeabi-v7a. do you tink in the Feature we can create app for armeabi-v7a?

GluonFX plugin relies on GraalVM for creating a native image, and it doesn't have support for 32 bits for any platform so far.
As an alternative it is possible to combine two APKs into one bundle, one for each type of arch. For armeabi-v7a you could use the old JavaFXPorts plugin, and for arm64 the GluonFX plugin. It is ugly but so far there are no better options.

How can I add into the bundle the second apk?

Sorry, I wasn't very clear: Actually you don't bundle them together in one APK, you can upload two APKs to Google Play for the same app but for different architectures. See https://developer.android.com/google/play/publishing/multiple-apks

First I'll try to generate apk with javafxport to test the upload on the store; then I create aab file, create the new release with this aab file, than I attach to the same release and I receive this error:
Hai caricato un APK con una firma non valida (ulteriori informazioni sulle firme). Errore dello strumento apksigner: ERROR: MIN_SIG_SCHEME_FOR_TARGET_SDK_NOT_MET: Target SDK version 30 requires a minimum of signature scheme v2; the APK is not signed with this or a later signature scheme
I use the same keystore. what I'm wrong?


Several site changes.

Added changes:

Changed info for the ff: index.html, hannah.html, and pranchi.html
It's my birthday! Replaced every number 16 to 17.
Several fixes.


fix copy to the linux system clipboard

Hello! Thank you for your project, I liked it, but immediately I came across that the clipboard of the system was not working. I found the reason why this happens - the instance of arboard clipboard must run throughout the lifetime of the entire program..
Instead of copying text, signals are sent in the vim module, which are processed in the app, as well as components.

(you can ignore this bot's feedback for now, it's just to flag anything else i might want to look at)
@greptileai

hm after taking another look at vim_state and the command_tx registration, i think it might be better to always initialize vim_state with a pointer to a function in editor.rs that does the message sending instead since it already has the command_tx initialized.
i can make the edits directly after doing some testing if you dont mind

hm after taking another look at vim_state and the command_tx registration, i think it might be better to always initialize vim_state with a pointer to a function in editor.rs that does the message sending instead since it already has the command_tx initialized.
i can make the edits directly after doing some testing if you dont mind

editor.rs

Yes of course! Perhaps this is the best solution

i couldn't get it to work with giving the vim_state a function pointer or closure to a function in editor, so i'll stick with your approach of having the command_tx registered in vim_state :)
did some profiling and cloning the command_tx every transition doesnt have any performance impact

i couldn't get it to work with giving the vim_state a function pointer or closure to a function in editor, so i'll stick with your approach of having the command_tx registered in vim_state :)
did some profiling and cloning the command_tx every transition doesnt have any performance impact

Yes, cloning command_tx as far as I know just copies the reference, so the number of sender can be created a lot