i can't use any of crouton scripts

Please describe your issue:
In the folder where the crouton scripts run (scripts like enter-chroot, edit-chroot,startcli, etc)  i can't run any of those scripts instead i got these:
(Note) i can't use sudo (script name) directly it gives this error: sudo: edit-chroot: command not found
sudo sh ./edit-chroot -all
sh: 0: getcwd() failed: No such file or directory
sh: 0: Refusing to exec ./edit-chroot from noexec mount; see https://chromium.googlesource.com/chromiumos/docs/+/master/security/noexec_shell_scripts.md
It goes on with a similar result in the rest of the scripts
By the way, I am running crouton from an external drive and the only way for me to access the chroot is the chroot command. I am using bionic as the release for the chroot
Reddit post here (that user that posted is me): https://www.reddit.com/r/Crouton/comments/v4h7kr/i_cant_run_a_crouton_script/?utm_source=share&utm_medium=web2x&context=3

See my previous response here: https://github.com/dnschneid/crouton/issues/4748#issuecomment-1146432532
Also, you may need to install or reinstall the crouton scripts under /usr/local/bin/, you can do that using crouton's -b option, see below:

-b          Restore crouton scripts in PREFIX/bin, as required by the               chroots currently installed in PREFIX/chroots.

-DennisLfromGA

This guy is having issues because he install crouton wrong. He refused to fix it. He installed it with -p and on an external drive, so all the scripts are there and that drive is not in his path environment variable.
He can't run enter-chroot because the script is on his drive with no-exec. I tried to exain all of this on his last post but he never listed. Here are the results. He deleted the post as well, because why not
https://www.reddit.com/r/Crouton/comments/uyai9r/cannot_execute_binbash_permission_denied_need/?utm_medium=android_app&utm_source=share

@Masong19hippows,
I understand.
If he installs the crouton scripts on his internal drive under /usr/local/bin/ then he can launch them on external media by specifying the -c option with his 'start' scripts.
-DennisLfromGA

how do I use -b, what I know about -b is that it runs in the background. If i try this: sudo crouton -u -n bionic_two -b -p /media/removable/USB\ Drive :
Downloading latest crouton installer...
############################################################################################################################################## 100.0%
crouton [options] -t targets
crouton [options] -f backup_tarball
crouton [options] -d -f bootstrap_tarball
Constructs a chroot for running a more standard userspace alongside Chromium OS.
If run with -f, where the tarball is a backup previously made using edit-chroot,
the chroot is restored and relevant scripts installed.
If run with -d, a bootstrap tarball is created to speed up chroot creation in
the future. You can use bootstrap tarballs generated this way by passing them
to -f the next time you create a chroot with the same architecture and release.
crouton must be run as root unless -d is specified AND fakeroot is
installed AND /tmp is mounted exec and dev.
It is highly recommended to run this from a crosh shell (Ctrl+Alt+T), not VT2.
Options:
-a ARCH     The architecture to prepare a new chroot or bootstrap for.
Default: autodetected for the current chroot or system.
-b          Restore crouton scripts in PREFIX/bin, as required by the
chroots currently installed in PREFIX/chroots.
-d          Downloads the bootstrap tarball but does not prepare the chroot.
-e          Encrypt the chroot with ecryptfs using a passphrase.
If specified twice, prompt to change the encryption passphrase.
-f TARBALL  The bootstrap or backup tarball to use, or to download to (-d).
When using an existing tarball, -a and -r are ignored.
-k KEYFILE  File or directory to store the (encrypted) encryption keys in.
If unspecified, the keys will be stored in the chroot if doing a
first encryption, or auto-detected on existing chroots.
-m MIRROR   Mirror to use for bootstrapping and package installation.
Default depends on the release chosen.
Can only be specified during chroot creation and forced updates
(-u -u). After installation, the mirror can be modified using
the distribution's recommended way.
-M MIRROR2  A secondary mirror, often used for security updates.
Can only be specified alongside -m.
-n NAME     Name of the chroot. Default is the release name.
Cannot contain any slash (/).
-p PREFIX   The root directory in which to install the bin and chroot
subdirectories and data.
Default: /usr/local, with /usr/local/chroots linked to
/mnt/stateful_partition/crouton/chroots.
-P PROXY    Set an HTTP proxy for the chroot; effectively sets http_proxy.
Specify an empty string to remove a proxy when updating.
-r RELEASE  Name of the distribution release. Default: xenial,
or auto-detected if upgrading a chroot and -n is specified.
Specify 'help' or 'list' to print out recognized releases.
-t TARGETS  Comma-separated list of environment targets to install.
Specify 'help' or 'list' to print out potential targets.
-T TARGETFILE  Path to a custom target definition file that gets applied to
the chroot as if it were a target in the crouton bundle.
-u          If the chroot exists, runs the preparation step again.
You can use this to install new targets or update old ones.
Passing this parameter twice will force an update even if the
specified release does not match the one already installed.
-V          Prints the version of the installer to stdout.
...

@Arvin2A
The -b option does not run it in the background. The guy above explained exactly what it does.
Basically, whenever you installed crouton with -p, it installed everything on your external drive. It installed the scripts as well as the chroot all on that external drive. This is why you are having so many issues. The external drive is a non-exec mount and it is not in your path. This is what is preventing you from using the enter-chroot command. You can't use the normal commands including enter-chroot because your os has no way to find these commands and no way to execute them . This is what I tired to explain while you were being rude in the reddit post.
The -b option fixes everything. I could be wrong, but all you should need to run is sudo crouton -b nothing after or before that. This will install the scripts under the right directory and fix the install that was broken using -p. After this, you should be able to execute sudo enter-chroot, and be able to enter your chroot normally.

If allof your chroots on installed on external media you will have to use the '-p' option along with the '-b' option to let crouton know where your chroots are so it can then determine the targets, etc. and script to restore.

Ope, my bad

i did sudo enter-chroot and then this happened: No chroots found in /mnt/stateful_partition/crouton/chroots

Ok, i successfully entered the chroot but now i am getting an installation error


Adding Confetti Effect on Contributors Page

Description
This PR introduces a confetti effect on the ContributorsPage that activates when contributor data is successfully fetched. The confetti runs briefly for 6 seconds, creating a celebratory moment when users view the list of contributors.
Implementation Details

State Management

A state variable manages when the confetti is displayed and automatically toggles off after 6 seconds using a timeout.

Responsive Window Handling

Confetti dimensions are adjusted based on window size, ensuring it remains responsive during window resize events.

Confetti Component Integration

The confetti effect is integrated using a React component, dynamically adjusting its visibility based on the component's loading state.

Screenshot

Desktop

https://github.com/user-attachments/assets/d77ce40c-a9ae-434d-8544-08d95d49233b

Mobile

https://github.com/user-attachments/assets/705274e8-b5fa-4f97-8ec7-d072d71b491e

Checklist

[x] Confetti effect added
[x] Effect stops after 6 seconds
[x] Responsive dimensions on window resize
[x] Code reviewed and tested

Linked Issues

Closes #1054

Hey @IkkiOcean, this is really nice, but there's a problem. When we click on the GitHub Contributors button, it doesn’t automatically scroll up; we have to manually scroll, which feels a bit weird. Please correct this so that when a user clicks the GitHub Contributors button, the footer shows, allowing every contributor to be seen without manual scrolling. Hope you understand.

@mdazfar2 I'm sorry I didn't understand that. Right now when we click GitHub contributor on the footer it redirects to the page at the top position as default. What do you need correction in it?

@mdazfar2 I'm sorry I didn't understand that. Right now when we click GitHub contributor on the footer it redirects to the page at the top position as default. What do you need correction in it

That's amazing! Actually, I was wrong, actually it takes time to reload. No problem, it's fine.

Hey @IkkiOcean, I noticed that I asked you to display only 4 contributor profiles in a row, but no changes were made for the desktop screen size at 100%. Could you please fix this in a new issue?

@mdazfar2 okay sure. I thought you agreed for cards per row depending on the screen size that's why I didnt touch it. I will raise the issue right away


Unable to convert account to multi-signature

@OlegMakarenko public key is known when there will be outgoing transaction. This address has only incoming transaction.
Is more like bug in explorer because it shows invalid public key for such address ... For sure it's not 000....0000

I reported bug for explorer: https://github.com/nemgrouplimited/symbol-explorer/issues/663

When I enter the accounts public keys from the account tab, the wallet doesn't show any warning and announce HASH_LOCK transaction

To create multisig you need to have public keys of cosigners.
From address you can't query public key if it's not announced to network.

@OlegMakarenko public key is known when you do first outgoing transaction. This address has only incoming transaction. From wallet perspective all is IMO OK.
@bassemmagdy @yilmazbahadir @evias please confirm.
Is more like bug in explorer because it shows an invalid public key for such an address ... For sure it's not 000....0000

I believe that's correct.

@OlegMakarenko public key is known when you do first outgoing transaction. This address has only incoming transaction. From wallet perspective all is IMO OK.
@bassemmagdy @yilmazbahadir @evias please confirm.
Is more like bug in explorer because it shows an invalid public key for such an address ... For sure it's not 000....0000

Confirmed.

@cryptoBeliever @yilmazbahadir @bassemmagdy This should work even if the cosignatory candidate accounts don't have any outgoing transaction. I have just double-check it with SDK and it works.
http://explorer-0.10.0.x-01.symboldev.network/accounts/TDOSSZJVNHGFTDJ3LE7YTB5N4VU353WCLBIGIEI

@OlegMakarenko in SDK you did it using public keys or addresses? IMO public keys.
If you will use public keys directly in cosigner input you can do this as well from wallet.
You can't calculate public key from address. It's not possible. That's why wallet app must ask network (/accounts/) to get public key from address. If address don't have any outgoing transaction it has no public key known.

@cryptoBeliever addresses.
Check this: https://docs.symbolplatform.com/symbol-sdk-typescript-javascript/0.21.1/classes/model_transaction_multisigaccountmodificationtransaction.multisigaccountmodificationtransaction.html#addressadditions

I was under the impression that public key is needed to convert. If not I leave it to verify by @bassemmagdy / @yilmazbahadir :)

@evias @bassemmagdy @yilmazbahadir
So there are 2 bugs I've found:

The form doesn't allow you to add cosignatory candidate accounts address (if it doesn't have outgoing transaction)
The wallet announce incorrect HASH_LOCK transaction. Should be HASH_LOCK transaction followed by a AGGREGATE_BONDED transaction

@evias @bassemmagdy @yilmazbahadir
So there are 2 bugs I've found:

The form doesn't allow you to add cosignatory candidate accounts address (if it doesn't have outgoing transaction)
The wallet announce incorrect HASH_LOCK transaction. Should be HASH_LOCK transaction followed by a AGGREGATE_BONDED transaction

@OlegMakarenko

As I explained in my previous comment, if this verification step is unnecessary then we can omit this step, it is an easy fix.
There is a bug already reported with https://github.com/nemgrouplimited/symbol-desktop-wallet/issues/513 (which I'm currently working on) are you referring to the same issue here?

Fixed.


Test: generalized keybindings for navigating input / result of widgets

Refs #179967

[ ] Windows @joyceerhl
[ ] macOS @hbons
[ ] Linux @alexr00

Complexity: 4
Create Issue

For screen reader users, consistency across components is very helpful.
We now have a generalized way of navigating to / from the results / input of widgets.
Test that this works and doesn't break any functionality in the following areas:

Extensions
Keybindings editor
Comments
Problems
Debug

I think this is ctrl+up arrow and ctrl+down arrow should let you navigate between an input and it's results. @meganrogge is that what we should be testing?

On macOS Ctrl+DownArrow and Ctrl+DownArrow opens Mission Control. When I use the Cmd+Down key instead from Search I can navigate to the files to include and files to exclude fields and after that the search results list. I can navigate back to the previous fields using Cmd+Up.

Sorry for the unclear instructions - yes, cmd (mac) or ctrl + up/down arrow should allow you to go from the input to the results for each of the views


App fails to start - I only see the "This app requires WiFi connection" toast.

I'm running RemoteVideoCam on 3 different phones all connected to the same broadcast domain:
Motorola Moto G (XT1032) works perfectly
Samsung Galaxy S4 (SGH-M919) works perfectly
OnePlus 5T (A5010) fails to start - all I get is the "This app requires WiFi connection..." toast.
All three phones are using v2.7 from f-droid (apk md5sum starts a140...).
Would a logcat help?

Are you sure your OnePlus is connected to your WiFi network? If it is,
maybe the telecom network takes precidence. Can you temporarily disable
your telephone network and try again?
On Fri., Oct. 28, 2022, 12:05 p.m. Jaime, @.***> wrote:

I'm running RemoteVideoCam on 3 different phones all connected to the same
broadcast domain:
Motorola Moto G (XT1032) works perfectly
Samsung Galaxy S4 (SGH-M919) works perfectly
OnePlus 5T (A5010) fails to start - all I get is the "This app requires
WiFi connection..." toast
https://github.com/izivkov/RemoteVideoCam/blob/master/app/src/main/java/org/avmedia/remotevideocam/MainActivity.kt#L44
.
All three phones are using v2.7 from f-droid
https://f-droid.org/repo/org.avmedia.remotevideocam_27.apk (apk md5sum
starts a140...).
Would a logcat help?
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/RemoteVideoCam/issues/8, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37SOJEXD45OWO664Y5LWFP2TDANCNFSM6AAAAAARRGWMKY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

One thing you can try is to go the Android Settings -> Apps and
Notification -> RemoteVideoCam and Clear Storage. Then run the app again.
I will check why the app does not start after disconnecting and
connecting back on WiFi.
Thanks for reporting.
Ivo
On Fri, Oct 28, 2022 at 1:58 PM Ivo Zivkov @.***> wrote:

Are you sure your OnePlus is connected to your WiFi network? If it is,
maybe the telecom network takes precidence. Can you temporarily disable
your telephone network and try again?
On Fri., Oct. 28, 2022, 12:05 p.m. Jaime, @.***>
wrote:

I'm running RemoteVideoCam on 3 different phones all connected to the
same broadcast domain:
Motorola Moto G (XT1032) works perfectly
Samsung Galaxy S4 (SGH-M919) works perfectly
OnePlus 5T (A5010) fails to start - all I get is the "This app requires
WiFi connection..." toast
https://github.com/izivkov/RemoteVideoCam/blob/master/app/src/main/java/org/avmedia/remotevideocam/MainActivity.kt#L44
.
All three phones are using v2.7 from f-droid
https://f-droid.org/repo/org.avmedia.remotevideocam_27.apk (apk md5sum
starts a140...).
Would a logcat help?
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/RemoteVideoCam/issues/8, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37SOJEXD45OWO664Y5LWFP2TDANCNFSM6AAAAAARRGWMKY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Are you sure your OnePlus is connected to your WiFi network? If it is, maybe the telecom network takes precidence. Can you temporarily disable your telephone network and try again?

This is really weird. I temporarily disabled my telephone network and I was able to start the app successfully. I then thought "Great - that is the solution." But...
Then I re-enabled my telephone network and the app did not start again.
Then I re-disabled my telephone network (hence only wifi now) and still the app did not start. Huh?
Disable and re-enable wifi (telephone network still disabled from previous step hence only wifi is connected) - still the app does not start. Huh?
After a "Clear cache", still the app does not start.
After a "Clear storage", the app does start.
Here is a logcat "failure" session (piped through "grep remotevideocam") - hopefully this helps.

Thanks, I'll look into it.

This happens when the phone is using its mobile connection instead of the WiFi connection. It is different for different phones, but maybe disabling your mobile connection temporary will solve the problem. The app has no control over that, so I am closing this issue.

I have the same problem on a Xiaomi Mi 9 Lite (Android 9).
I have only WiFi active. Cellular data is not active and also disabled on the properties of the app (No mobile data usage).

Is it working with other phones for you?
On Thu, Jun 6, 2024 at 7:14 AM Quantum-Future @.***>
wrote:

I have the same problem on a Xiaomi Mi 9 Lite (Android 9).
I have only WiFi active. Cellular data is not active and also disabled on
the properties of the app (No mobile data usage).
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/RemoteVideoCam/issues/8#issuecomment-2152072207,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37QEJIV6KXBJBM5RJVLZGBAAVAVCNFSM6AAAAABI4QICWKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJSGA3TEMRQG4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I tested on a newer Xiaomi phone with Android 13 and yes, it works.
Well, at least the error message does not appears, but I couldn't test video functionality because I had no other phone available at that moment.

I upgraded to Ver 3.2 and noticed this:

When the app starts with WiFi disabled, the message about WiFi required appears.
Then I enabled WiFi and started the app again, but the same message appears (does not detect that WiFi is active).
I forced the stop of the app and then re-startet it.  Now it detects that WiFi is active and works as expected.

I hope this helps to solve the problem in the next version.

Is the OnePlus connected to WiFi? Are you running only 2 phones at a time?
On Sun, Jul 21, 2024, 3:58 AM Quantum-Future @.***>
wrote:

I upgraded to Ver 3.2 and noticed this:

When the app starts with WiFi disabled, the message about WiFi
required appears.
Then I enabled WiFi and started the app again, but the same message
appears (does not detect that WiFi is active).
I forced the stop of the app and then re-startet it. Now it detects
that WiFi is active and works as expected.

I hope this helps to solve the problem in the next version.
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/RemoteVideoCam/issues/8#issuecomment-2241516936,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37VT7KSIWE7IFRS2EEDZNNS3VAVCNFSM6AAAAABI4QICWKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBRGUYTMOJTGY
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I don't have a OnePlus, I have a Xiaomi.
The comment about the OnePlus and other 2 phones, was from another user (jaimet).
I started to comment in this thread, because I had the same problem as Jaimet.
My previous post was using only the Xiaomi.

I'm not sure how to reproduce the issue. I'll live it open for a while, to
see if anyone can pick it up and try to address it.
On Mon, Jul 22, 2024, 4:47 PM Quantum-Future @.***>
wrote:

I don't have a OnePlus, I have a Xiaomi.
The comment about the OnePlus and other 2 phones, was from another user
(jaimet).
I started to comment in this thread, because I had the same problem as
Jaimet.
My previous post was using only the Xiaomi.
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/RemoteVideoCam/issues/8#issuecomment-2243786370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37VWQTRUQTVAZJJGMBDZNVVVBAVCNFSM6AAAAABI4QICWKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBTG44DMMZXGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I think the problem does not happen on all Android versions. As I said before, I tested it on another Xiaomi phone with Android 13, and this message doesn't appear.
The problem happens on a Xiaomi with Android 9.
Can you obtain a phone with android 9 to test it?
PS: This issue is not open.


Lack of --root option, broken setup.py makes flit unusable for creating Void, Arch and Alpine Linux system packages

As described in pypa/packaging#363, the packaging systems of Void, Arch and Alpine Linux (as well as others, I'm sure) rely on the ability to install software to otherwise empty destination directories that are used to define the contents of distribution packages. For Python packages using setuptools or distutils, this is accomplished by providing a --root argument to setup.py.
flit provides no comparable mechanism, instead allowing installation only into a per-user environment, the system environment or a virtual environment by passing the path to the python executable within that environment. The only possible workaround to this limitation is populating a destination directory with a virtual environment, using flit install to install to that environment, and then removing all parts of the virtual environment except the specific Python package to be wrapped in a distribution package. Obviously, this is not a viable solution.
The flit build or flit publish commands provide a partial workaround, because they produce source tarballs with a basic setup.py. However, the flit project discourages reliance on these scripts, even explicitly breaking them by importing distutils.core.setup rather than setuptools.setup despite their reliance on setuptools-specific keywords like install_requires, python_requires and extras_require. An attempt to prefer setuptools was explicitly rejected in #333. Reliance on pip to do the right thing isn't generally appropriate; especially for distribution packaging, using pip is a recipe for disaster because it deprives the packager of a lot of control (or at least makes the build process more difficult to manage).
If flit is to be usable by distribution packaging systems, then a couple of enhancements should be considered:

Reconsider #333, allowing flit to produce a setup.py that works properly with setuptools, so projects can still provide source tarballs that are properly buildable and relocatable without flit.
Support a --root argument in flit to allow installation with respect to an arbitrary (and most likely empty) $DESTDIR.

The first option seems to be low-hanging fruit (particularly since the work is done). The second option may be complicated if flit seeks eventually to support more advanced build procedures. For example, if flit intends to support compiled extensions in CPython, the compilation environment should be easily controllable because packaging systems like Void's xbps-src are designed to compile or cross compile for several different architectures.

Oh what joy for a Sunday morning.
The flit command line interface is meant for people working on packages. The canonical way to install an arbitrary Python package from source is to make a wheel (with standard APIs specified in PEP 517), and then install the wheel, which is a documented format. pip can be used for that, and there's lower-level tooling in development in the pypa-build project and @pradyunsg's WIP installer project. flit install already generates a wheel and then calls pip to install it, so you're not doing anything more low-level by running that.

Thanks for clarifying the scope of flit.


Removing the $n function

Description of the Change
This function makes it difficult for IDEs to work correctly as they’re dynamically defined. It also doesn’t play nice with static analysis systems.

Closes #246
How to test the Change
Tested Locally
Changelog Entry

Removed - $n function, in preference of passing the namespace manually

Checklist:

[x] I agree to follow this project's Code of Conduct.
[ ] ~I have updated the documentation accordingly.~
[ ] ~I have added Critical Flows, Test Cases, and/or End-to-End Tests to cover my change.~
[x] All new and existing tests pass.

This makes me so happy ❤  IDE integration and dead code detection ftw!


Add a note about the home directory in container

There seems to be issues depending on what users are doing in the
container - the added content should prevent them.
As a pull request owner and reviewers, I checked that:

[X] Appropriate documentation exists and/or is up-to-date:

/lgtm

/approve


playerProfileRequest mainly returns null and readCString error

Since the start of using the 'package', I've been trying to recieve player profile information; however, all requests no matter what valid accoundID provided returns null for the majority of the fields:
{
  "request_id": null,
  "account_profiles": [
    {
      "account_id": 396346719,
      "ongoingmatch": null,
      "global_stats": null,
      "penalty_seconds": null,
      "penalty_reason": null,
      "vac_banned": null,
      "ranking": null,
      "commendation": {
        "cmd_friendly": 19,
        "cmd_teaching": 16,
        "cmd_leader": 18
      },
      "medals": {
        "medal_team": null,
        "medal_combat": null,
        "medal_weapon": null,
        "medal_global": null,
        "medal_arms": null,
        "display_items_defidx": [
          4674,
          970,
          1367
        ],
        "featured_display_item_defidx": 4674
      },
      "my_current_event": null,
      "my_current_event_teams": [],
      "my_current_team": null,
      "my_current_event_stages": [],
      "survey_vote": null,
      "activity": null,
      "player_level": 14,
      "player_cur_xp": 327681231,
      "player_xp_bonus_flags": null
    }
  ]
}

The logs return:
13 Dec 23:37:01 - Launching CS:GO
13 Dec 23:37:03 - Sending ClientHello
13 Dec 23:37:03 - CS:GO fromGC: 4004
13 Dec 23:37:03 - Received client welcome.
13 Dec 23:37:03 - Sending player profile request
13 Dec 23:37:04 - CS:GO fromGC: 9173
13 Dec 23:37:04 - CS:GO fromGC: 9128
13 Dec 23:37:04 - Received player profile

Can confirm CS GO is running, as it established a successful connection and:

In addition, I've had the client crash to a random error:
/root/**********/node_modules/csgo/helpers/VDF.js:28
            var name = buffer.readCString();
                              ^

TypeError: buffer.readCString is not a function
    at Object.exports.decode (/root/**********/node_modules/csgo/helpers/VDF.js:28:31)
    at CSGOClient.handlers.<computed> (/root/**********/node_modules/csgo/handlers/rich_presence.js:44:20)
    at SteamClient.<anonymous> (/root/**********/node_modules/csgo/index.js:50:30)
    at SteamClient.emit (events.js:326:22)
    at SteamClient._netMsgReceived (/root/**********/node_modules/steam/lib/steam_client.js:175:10)
    at SteamClient.handlers.<computed> (/root/**********/node_modules/steam/lib/steam_client.js:262:10)
    at SteamClient._netMsgReceived (/root/**********/node_modules/steam/lib/steam_client.js:160:26)
    at Connection.emit (events.js:314:20)
    at Connection._readPacket (/root/**********/node_modules/steam/lib/connection.js:50:8)
    at Connection.emit (events.js:314:20)
    at emitReadable_ (_stream_readable.js:557:12)
    at processTicksAndRejections (internal/process/task_queues.js:83:21)

Can provide more information if needed!

Did you find a fix?

Did you find a fix?

What was the solution to var name = buffer.readCString();
I've stated to have this same issue?

@LegendNed, were you able to solve this problem? I'm trying to get the same answer. Players who have rank are coming with the value null.

@LegendNed, were you able to solve this problem? I'm trying to get the same answer. Players who have rank are coming with the value null.

Quite a time ago, however, if I remember correctly there was no proper solution, some user ranks were not retrieved at all returning the value as null


Add internal jax.lax.asarray utility

jnp.asarray is a huge API, and we don't always need its full power when converting ArrayLike to Array.
Putting this in lax makes sense to avoid circular imports when breaking up the lax_numpy.py monolith.

I suppose this is the canonical asarray to use after check_arraylike in the future?

I suppose this is the canonical asarray to use after check_arraylike in the future?

Yes, that's what I have in mind. We'll probably end up rolling the two together when addressing #14921


 One cannot totally automate the font generation b.c. the tool used to generate the intermediate svgfont doesn't do a good job of setting the font metrics correctly.
In #13239, I thought that the correct descent value was 6.25% of the fontHeight. Using that number caused the icons to be offset and blurry.
However setting the same ascent and descent values using https://www.fontsquirrel.com/tools/webfont-generator yields better results.
This is because fontsquirrel sets some x and y max values better than the ttf generator does, thus leaving the glyphs aligned with precise pixel values.

@adamjspooner


Add operation ID to headers

PR summary
Fixes: #5
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the
Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Thank you for the review, @vmatyusGitHub and @ricellis . I'm about to merge this.


Multicategory

Works fine for me with multiple categories (different keyName, templates and hitPerPage).
Props flow is changed pretty much so I guess it should go with version 2.0.0
For single index search just pass one object in array.

Really nice PR and refactor thanks @KafHat . Can you please squash your commits into one?
I'll test and merge it today or tomorrow.


Description of "File Types" makes no mention of syntax

There are two important things to understand about the syntax of the text entered into this field:

multiple file types can be entered by separating them by semi-colons (;)
You can exclude file types by specifying an exclamation point (!) before the file type

For example, suppose you want to search all *.txt files, but exclude files named *-foo.txt, you could enter
*.txt;!*-foo.txt
To restrict the search to all .txt files that don't have "-foo" at the end of the filename.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0e27881f-04bc-b7ba-e217-8360278a14f8
Version Independent ID: d31c8050-339a-688c-869e-ff7942fd3f74
Content: Find in Files - Visual Studio
Content Source: docs/ide/find-in-files.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@LegalizeAdulthood Thanks for your inquiry and apologies for our delayed response. We've recently revised the Find in Files page, which now includes the updates you've suggested to the File types section.
We'll close this issue now with our further thanks to you for helping us improve the documentation.


[Feature] Docker container

Feature description
Dockerize the code so that processing could be run from the command line, e.g. docker run <TF2.4image>
Describe the solution
This would simply put the OSIPI code inside of a docker container that could be run on the command line. Something like this for now: https://stackoverflow.com/questions/42625947/docker-input-output-outside-the-container
Just to run one and exit. Future work would be to keep it alive for future connections.
Describe alternatives
The current approach doesn't use the docker container. This would allow for someone to run the processing without installing anything other than docker and downloading our image. Further work could make it more of a server architecture, but this is the first step.
Other containerization could be used as well, such as singularity. Perhaps both could be supported in the future.
Additional context
No response
Are you working on this?
None

@etpeterson, following up with this, I have some questions.

By dockerizing the code, we are basically dockerizing this wrap script created in https://github.com/OSIPI/TF2.4_IVIM-MRI_CodeCollection/issues/58?
This is regarding how the input and the output data will be mounted when running the docker image, I am thinking bind mounting might be better since the input and output might keep changing, and I do not think we have all the input data in the repository as some are gotten from zenodo. What do you think ?

Yes, I think you're correct for both. However, the data should be provided and not downloaded from zenodo. We can assume people will send data in - essentially point to some files or folders. In fact that should be a requirement.


Create prefix.tag

I added a warning on what could happen if a Bedrock and Java player join the server at the same time with the same name.


"Odobasca","Petreanu","Podu Lacului","Târâtu" are listed in the drop-down menu

Description:
"Odobasca","Petreanu","Podu Lacului","Târâtu" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and running. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Odobasca"

Expected results
The location appears in the drop-down menu as "Sat Odobasca,  VRANCEA (Poiana Cristei)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Petreanu"

Expected results
The location appears in the drop-down menu as "Sat Petreanu,  VRANCEA (Poiana Cristei)"

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Podu Lacului"

Expected results
The location appears in the drop-down menu as "Sat Podu Lacului,  VRANCEA (Poiana Cristei)"

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Târâtu"

Expected results
The location appears in the drop-down menu as "Sat Târâtu,  VRANCEA (Poiana Cristei)"


Issuer validation failed. Issuer did not match.

Describe the bug
When trying retrieve blob data from the azure storage I get an error
Expected behavior
Being authenticated and retrieving data
Actual behavior (include Exception or Stack Trace)
Status: 403 (Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.)
ErrorCode: AuthenticationFailed
In fiddler I can see: Issuer validation failed. Issuer did not match.
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)
environment variable:
AZURE_USERNAME: {my azure ad account logged into visualstudio that does work for sqlconnections for example}
new BlobServiceClient(new Uri(configuration.GetConnectionString("AzureStorage")), new DefaultAzureCredential());

var container = blobServiceClient.GetBlobContainerClient("apk");

            foreach (var blob in container.GetBlobs()) //fails at .GetBlobs()

Environment:

Name and version of the Library package used:

Azure.Identity 1.1.1
Azure.Storage.Blobs 12.3.0

Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects):
Windows 10

IDE and version : [e.g. Visual Studio 16.3]

Hi,
Your AAD identity might not have the rights to read the blobs. So you may be able to authenticate to sqlconnections but if your ad account might not have access to storage account or read rights. Please recheck if you've given the rights to the storage account to the aad user.

Hi,
Sorry took a while to get back here.
The AAD identity definitely has rights since I can do this:

Thanks for the confirmation.
Could you rerun this this again and provide the x-ms-request-id? I would use the request id that you originally screenshotted but unfortunately the backend logs for these are flushed every 2 days.

Hi Sorry for the super late reply, but with covid and everything I didn't get to it, the code also wasn't in sourcecontrol yet so...
Anyway the x-ms-request-id = 10f103cd-c01e-009b-2ec6-0ce6ad000000

Also just updated to 12.4.1.
Same issue: 4374cdfc-e01e-0040-3ec7-0c427b000000 x-ms-request-id

Sorry about this, could you re run this and get the request id again?

Definitely: 4e0a9657-d01e-0007-6510-139d10000000

Can I get a status update for this one? :)

Hi @woutervs, sorry for the delay and the repeated asking for client ids and not looking them up in time.
I tried to reproduce your issue but I just wanted to clarify a few things. Are you actually using new DefaultAzureCredential() in initalizing the BlobServiceClient?
This looks related to this issue (https://github.com/Azure/azure-sdk-for-net/issues/8658).
Is it possible for you to try the preview 1.2.0-preview of Azure.Identity package to see if that resolves your problem?

So I basically do this:
services.AddSingleton(s =>
            {
                var configuration = s.GetService<IConfiguration>();
                return new BlobServiceClient(new Uri(configuration.GetConnectionString("AzureStorage")), new DefaultAzureCredential());
            });

I've tried with the preview version but still the same error.
x-ms-request-id: fc011faa-401e-00ce-1114-200dda000000

So I first tried with 1.2.0-preview.2 that failed with the same error.
I then read in #8658 that some people had some success with the 1.2.0-preview.1 package -> I tried that too first getting this

Then I did an az login in powershell getting me a new token.
And at last I can retrieve data from storage.
Though only in the 1.2.0-preview.1 version.

Hi,
Thanks for providing the request id (I was able to confirm it's just an identity auth issue) and confirming that the Azure.Identity package v1.2.0-preview.1 is giving you a more helpful error to get a new token to access your storage container.
I'm going to relabel this with Azure.Identity and see if they can better help your issue, because that's strange that the newer version of the preview does not have the fix that preview.1 has.

@woutervs Thanks for including the full error and stack. It seems like what's going on here is that your falling back to authenticating with the AzureCliCredential which in the second error you posted is failing because the refresh token has expired and you would have to re-run az login to re-authenticate via the CLI.
However, this doesn't explain why we failed to authenticate via the account logged into Visual Studio, which should supersede the Azure CLI. Could you possibly try this on our latest 1.2.0 preview? There have been several fixes to DefaultAzureCredential including the introduction of the VisualStudioCredential to provide more reliable authentication with Visual Studio. Also it would probably help to re-authenticate your Azure Service Account in VS to ensure it's up to date and the correct account.

I'll see if I can find some time to squeeze in another test. At some point I did manage to get it to work and from there I deployed to production as this is not a customer critical component, but an internal tool.

Hi, I am also encountering this issue when trying to authenticate a BlobCloudClient using MSI.
HTTP/1.1 401 Server failed to authenticate the request. Please refer to the information in the www-authenticate header.
Content-Length: 402
Content-Type: application/xml
Server: Microsoft-HTTPAPI/2.0
x-ms-request-id: 604a9acc-201e-006a-3382-86c184000000
x-ms-error-code: InvalidAuthenticationInfo
WWW-Authenticate: Bearer authorization_uri=https://login.microsoftonline.com/8dc638e2-f657-4a1c-bf43-6da5804f5f57/oauth2/authorize resource_id=https://storage.azure.com
Date: Wed, 09 Sep 2020 08:26:21 GMT

<?xml version="1.0" encoding="utf-8"?><Error>InvalidAuthenticationInfo<Message>Server failed to authenticate the request. Please refer to the information in the www-authenticate header.
RequestId:604a9acc-201e-006a-3382-86c184000000
Time:2020-09-09T08:26:22.3758046Z</Message><AuthenticationErrorDetail>Issuer validation failed. Issuer did not match.</AuthenticationErrorDetail></Error>

I am running this locally in Visual Studio and am logged in using an account that has been granted the contribuator role to the storage account. I am able to access the storage account as expected from Azure Storage Explorer.
My user account is in a CSP AAD and has been added as a guest to the tenant where the storage account exists. The tenant ID referenced in the WWW-authenticate header returned is that for the tenant where the storage account exists rather than for the CSP tenant where my user account exists if that makes any difference?

@woutervs Is this still a problem for you?

@woutervs Is this still a problem for you?

I haven't developed on this project anymore as we launched in production (the issue was only local during development) I think it was resolved, or we used a version locally where it worked. So no idea if it still occurs in the latest version.
If you want I can allocate some time to try it again during work hours tomorrow.

I haven't developed on this project anymore as we launched in production (the issue was only local during development) I think it was resolved, or we used a version locally where it worked. So no idea if it still occurs in the latest version.
If you want I can allocate some time to try it again during work hours tomorrow.

Thanks for the update @woutervs! I just wanted to make sure the issue has been resolved.
@mattosaurus, are you still experiencing the problem?

Thanks for the update @woutervs! I just wanted to make sure the issue has been resolved.
@mattosaurus, are you still experiencing the problem?

Hi, I changed my code to access Azure blob storage using a SAS key rather than MSI so I'm unsure if this has been resolved or not.

Hi, I changed my code to access Azure blob storage using a SAS key rather than MSI so I'm unsure if this has been resolved or not.

I see. Thanks for the updates! I'll close the issue for now. If there's more question regarding to the problems mentioned in the issue, please reopen.

I see. Thanks for the updates! I'll close the issue for now. If there's more question regarding to the problems mentioned in the issue, please reopen.

I see this ticket is closed, but is the issue actually resolved?
I am facing this now using:

Azure.Identity 1.4.0
Azure.Storage.Blobs (12.9.1)

I have tried setting my user both though VS -> Tools -> Options -> Azure Service Authentication, and via an AZURE_USERNAME environment variable, none of which has worked.  The error I am getting is:
[2021-07-04T10:14:54.086Z] Executed 'ListFiles' (Failed, Id=554faeca-1e68-4f7d-b8e6-75ad6e76d03c, Duration=3565ms)
[2021-07-04T10:14:54.088Z] System.Private.CoreLib: Exception while executing function: ListFiles. Azure.Storage.Blobs: Server failed to authenticate the request. Please refer to the information in the www-authenticate header.
RequestId:bee61514-501e-00b7-29bd-700f62000000
Time:2021-07-04T10:15:12.7427965Z
[2021-07-04T10:14:54.089Z] Status: 401 (Server failed to authenticate the request. Please refer to the information in the www-authenticate header.)
[2021-07-04T10:14:54.092Z] ErrorCode: InvalidAuthenticationInfo
[2021-07-04T10:14:54.093Z]
[2021-07-04T10:14:54.093Z] Additional Information:
[2021-07-04T10:14:54.094Z] AuthenticationErrorDetail: Issuer validation failed. Issuer did not match.
[2021-07-04T10:14:54.095Z]
[2021-07-04T10:14:54.095Z] Content:
[2021-07-04T10:14:54.096Z] <?xml version="1.0" encoding="utf-8"?><Error>InvalidAuthenticationInfo<Message>Server failed to authenticate the request. Please refer to the information in the www-authenticate header.
RequestId:bee61514-501e-00b7-29bd-700f62000000
Time:2021-07-04T10:15:12.7427965Z</Message><AuthenticationErrorDetail>Issuer validation failed. Issuer did not match.</AuthenticationErrorDetail></Error>
[2021-07-04T10:14:54.097Z]
[2021-07-04T10:14:54.098Z] Headers:
[2021-07-04T10:14:54.099Z] Server: Microsoft-HTTPAPI/2.0
[2021-07-04T10:14:54.100Z] x-ms-request-id: bee61514-501e-00b7-29bd-700f62000000
[2021-07-04T10:14:54.101Z] x-ms-error-code: InvalidAuthenticationInfo
[2021-07-04T10:14:54.102Z] WWW-Authenticate: REDACTED
[2021-07-04T10:14:54.106Z] Date: Sun, 04 Jul 2021 10:15:12 GMT
[2021-07-04T10:14:54.107Z] Content-Length: 402
[2021-07-04T10:14:54.108Z] Content-Type: application/xml
[2021-07-04T10:14:54.109Z] .

Can we reopen this ticket or should I create a new one?

I have the same issue, @danielmackay did you open a new issue or not?

@Expecho - I did not open a new issue, but I managed to solve the issue.
In my case I was using a user who had access to multiple subscriptions/tenants.  I had the right user selected in Visual Studio, but I think it was defaulting to the wrong tenant.
I solved this by explicitly using the AzureCliCredential() and Azure CLI to login to the exact tenant.  You could probably also do something similar by using EnvironmentCredential()

@danielmackay I ended up with the same "Issuer does not match" error. I came across this issue via Google search for that error message. I was able to solve this by specifying the tenantId to the options of DefaultAzureCredential:
new DefaultAzureCredential(new DefaultAzureCredentialOptions
        {
            VisualStudioTenantId = "7e5a038c-dd78-4130-b562-f0ea12548732"
        })

There are similar options for TenantId for other auth mechanisms. Unfortunately, there's no overarching generic TenantId option, so it must be specified for each potential auth mechanism.

Good to know we can lock in the tenant that way. Thanks for the info!
On Fri, 14 Jan 2022, 5:26 am Andrew Scott, @.***> wrote:

@danielmackay https://github.com/danielmackay I ended up with the same
"Issuer does not match" error. I came across this issue via Google search
for that error message. I was able to solve this by specifying the tenantId
to the options of DefaultAzureCredential:
new DefaultAzureCredential(new DefaultAzureCredentialOptions
{
VisualStudioTenantId = "7e5a038c-dd78-4130-b562-f0ea12548732"
})
There are similar options for TenantId for other auth mechanisms.
Unfortunately, there's no overarching generic TenantId option, so it must
be specified for each potential auth mechanism.
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-net/issues/10529#issuecomment-1012439433,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUDWYABGS6WSO622A34N5LUV4RMPANCNFSM4LFQY27Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

But for guest accounts, we still get this issue!

The weird thing is using "DefaultAzureCredential()" to do the authetication when retrieving Key Vault, it succeeds while for Azure blob storage, it shows "Issuer validation failed", do you have any clue?

I am running into this same issue today using an ASP.NET Core web app with .net version 6 and the latest Azure Identity and Azure blobs SDKs (1.8.0 and 12.14.1 respectively.
Here is my code to initialize the client
            var credentials = new DefaultAzureCredential(includeInteractiveCredentials: true);             var uri = new Uri(_Configuration["Azure:Blob:URI"]!);             var client = new BlobServiceClient(uri, credentials);

@hirejohnloveland - Try using Azure CLI to login.  I've found I have a lot more control than when trying to leave the authentication up to visual studio.
By default the Azure CLI will not let you login to an tenant without a subscription (which is common if your tenant is only used for AAD).  In that case try logging on with
az login -t myb2ctenant.onmicrosoft.com --allow-no-subscriptions

After logging out and clearing all VS account info and restarting, the CLI method is working for me.  I did also have to specify the AZURE_TENANT_ID with the tenant I am requesting for.  IMO this is a bug because the sdk should infer the requesting tenant from the uri of the storage account.  Or the SDK should allow you to turn on interactive credentials AND specify DefaultCredentialOptions with Tenant ID in the constructor. If I use the options, I can't turn on interactive credentials and I hit MFA issues.  If I turn on interactive credentials instead of putting the tenant in the options, I have to add the environmental variable.


GO F-P link data

GOC annotations from Function -process links...
Where do we get this data from?
I have completely forgotten where this is documented.....

@kimrutherford @mah11

This is an old issue but it might be the location
https://github.com/pombase/pombase-chado/issues/126

Thats a very old link...
I looked on the GO website documentation/FAQ and did a google search for
"gaf-inference" which was  in the old path but I still can't locate it.....

Is this what you mean?:
http://build.berkeleybop.org/view/GAF/job/gaf-check-pombase/lastSuccessfulBuild/artifact/gene_association.pombase.inf.gaf

that' the one

Chris recommends this location
https://github.com/pombase/website/issues/263#issuecomment-295110040
https://build.berkeleybop.org/job/go-gaf-release-snapshot/lastSuccessfulBuild/artifact/pipeline/target/groups/pombase/

Chris recommends this location

I can't see a .inf file there.  Which file do we need?

seems to be
pombase-gafcheck-prediction.gaf
I'm not sure what some of them are...

That file is a bit odd.  It's identical to the other file (gene_association.pombase.inf.gaf) except that:
Column 9 ("Aspect") is always "C" or "F" and never "P".  The GAF spec says it should be the aspect of the ID in column 5.
Column 9 seems redundant since you can work out the Aspect from the ID.  But it makes me slightly suspicious that it's wrong here.
As an example, the term in the first annotation in both that file and the old gene_association.pombase.inf.gaf file is: GO:0006357 - regulation of transcription from RNA polymerase II promoter
The old file has "P" in column 9 and the new file has "C".
Is it worth reporting to Chris?
For us it's not a problem because we ignore the aspect column so I'll change the load script to use the new one instead.

I'll change the load script to use the new one instead

Done.

Hi Chris, see Kim's comment above...looks like a problem?
@cmungall

Yes that does seem odd.
v


Not able to install on AWS linux

When I run npm install after generating the app, it throws a large amount of WARNS (see log.txt)
log.txt
When I then run npm start, it doesn't work properly and outputs the following:
npmstart.txt
Is this expected behavior? When I follow the same steps on my Mac, it works as expected.
I'm using npm 3.10.3 version and nodejs version 6.6.0

Hi,
The warnings shouldn't be a problem. Instead of 'npm start' can you try executing :
npm run serve
Thank you

Hello Mario,
This is the output when I run that:
`$ npm run serve

<EMAIL_ADDRESS>serve /home/ec2-user/test
wsrv -o -s -l -p 3000 -x ./server/versions.js

Server running at: http://localhost:3000
Opening start page: http://localhost:3000
(node:3712) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Exited with code 3`

@jtalfresco I think the issue is related to npm start script, it tries opening default browser when you are probably running a headless linux configuration. For testing purposes I would suggest removing -o (open browser) and -l (live reload) switches.
Please note that npm start is mainly developer-oriented, you may need something more complex for production.

Thank you for the help. Once I did that and defined the host with -a, it worked.


Update doc Parse and Format Date Function Calls

Add M to the table and add a note about the inconsistent behaviors.
Related to GitHub issue: https://github.com/mendix/docs/issues/5438

Hi Yiyun Here are my comments (again - don't know what happened to the ones I did yesterday). Nice response to the customer in the original issue. I've made some suggestions to make it easier for the customer to decide more quickly whether it is important for them. The commit suggestion is more to point out which "and" I meant than to resolve all my comments. Let me know if you disagree or have any questions. Mark

Hi Mark,
I just updated the note based on your comments (see commit 1b195db6eb910607c66753b8ccfcafeda6931a06). Thanks for the nice comments.
Let me know if there is more that needs to be improved.
Yiyun


Pull lyrics in correct language using the "language" tag as a guide

My lyrics config is the following
lyrics:
fallback: 'No lyrics found'
google_API_key: AIzaSyDxkosmJMLvt6bAJIy_VkwcM01E_A-AqRg
I find that for a few albums the lyrics are pulled in the wrong languages.  The Poets of the Fall album "Jealous Gods" is pulled in French and Bon Jovi's Slippery When Wet is pulled in Spanish, even though in both cases "Language:" is set to "eng" in the tags.
Shouldn't Beets be looking at the Language tag and pulling the lyrics based on that, if a default isn't set?
Then again, you can't set a default language either.

Yes, getting the wrong lyrics language sounds frustrating—but at the moment, I don't see a straightforward way to request a specific language from any of our backends. Maybe @Kraymer has more insight, but I think we just get text back from the databases—there's no indication as to what language it's in.

Thank you. For now, removing Google as a search engine seems to take care of it.

One possible solution to have it both ways would be to identify which website source is responsible for serving you non english lyrics (by adding a logging.debug()), and remove these sources from the google results by defining your own custom search engine (and setting google_engine_ID in plugin's configuration).
Sources used in beets CSE are listed at
https://github.com/sampsyo/beets/blob/master/test/test_lyrics.py#L232

It seems that for the time being, removing Google as a source fixes this. It makes sense as Google will pull from everything where as the other sources are pure lyrics repositories and as such will more likely only have the lyrics in the original language.


Cal.com

https://github.com/calcom/docker

@andrasbacsai   Kindly Add and Vote for this request here: https://feedback.coolify.io/

Closing this one in favor of: https://feedback.coolify.io/posts/33/service-cal-com


refactor: rename txHashList to txHash

Description
Issue link
In the proto file for outboundtracker:
message OutBoundTracker {
  string index = 1; // format: "chain-nonce"
  int64 chain_id = 2;
  uint64 nonce = 3;
  repeated TxHashList hash_list = 4;
}

It doesn't make sense to call TxHashList since this is the element of the list. Should be renamed TxHash

How Has This Been Tested?

[ ] Tested CCTX in localnet
[ ] Tested in development environment
[ ] Go unit tests
[ ] Go integration tests
[ ] Tested via GitHub Actions

Summary by CodeRabbit

Refactor

Renamed various data structures and fields from TxHashList to TxHash to improve naming consistency.
Updated the OpenAPI specification to reflect the renaming of crosschainTxHashList to crosschainTxHash for clarity.

@lumtis I need to change the same variable name in all the places I think. And running make generate only removed most of the ts files, but I only commited the changes that were made to outbound_tracker.pb.go file.
I will fix it by EOD.

@lumtis I need to change the same variable name in all the places I think. And running make generate only removed most of the ts files, but I only commited the changes that were made to outbound_tracker.pb.go file. I will fix it by EOD.

Yeah, everything changes by make generate needs to be committed, as the CI run the same command and check if there is a difference in the diff

There are 441 files deleted because of it, is this expected? 😓

There are 441 files deleted because of it, is this expected? 😓

Now, I think there is an issue when you run make generate
Can you show the logs printed by the command? Thanks

It was some issue with file not found, but now it resolved by itself.
➜ make generate
--> Formatting Protobuf files
--> Removing old Go types
--> Generating Protobuf files
Generating gogo proto code
--> Generating OpenAPI specs
rm: ./docs/openapi/zetachain.swagger.yaml: No such file or directory
--> Generating module documentation
--> Generating TypeScript bindings
./scripts/protoc-gen-typescript.sh: line 9: typescript/package.json: No such file or directory
--> Generating zetacored documentation
--> Generating mocks
Mocks generated.
Fixing long lines...
Formatting code...
Code is formatted

I need one more approval before merging this PR. @kingpinXD can you please check this too? Thanks!


Slashing.signing_info_by_address doesn't behave as expected

Jigu version: 0.1.2
Python version: 3.7.7
Operating System: macOS Mojave 10.14.6 (18G3020)

Description
Describe what you were trying to get done.
Tell us what happened, what went wrong, and what you expected to happen.
list all signing info as indicated here.
What I Did
>>> from jigu import Terra
>>> terra = Terra("columbus-3", "https://lcd.terra.dev")
>>> terra.is_connected()
True
>>> terra.slashing.signing_info_by_address()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/etienne/Library/Caches/pypoetry/virtualenvs/valmailer-tZO3ahg3-py3.7/lib/python3.7/site-packages/jigu/client/lcd/api/modules/slashing.py", line 36, in signing_info_by_address
    validate_val_consaddress(address)
  File "/Users/etienne/Library/Caches/pypoetry/virtualenvs/valmailer-tZO3ahg3-py3.7/lib/python3.7/site-packages/jigu/util/validation.py", line 84, in validate_val_consaddress
    if not is_val_consaddress(data):
  File "/Users/etienne/Library/Caches/pypoetry/virtualenvs/valmailer-tZO3ahg3-py3.7/lib/python3.7/site-packages/jigu/util/validation.py", line 45, in is_val_consaddress
    is_bech32(data)
  File "/Users/etienne/Library/Caches/pypoetry/virtualenvs/valmailer-tZO3ahg3-py3.7/lib/python3.7/site-packages/jigu/util/validation.py", line 20, in is_bech32
    return bech32.bech32_decode(data) != (None, None,)
  File "/Users/etienne/Library/Caches/pypoetry/virtualenvs/valmailer-tZO3ahg3-py3.7/lib/python3.7/site-packages/bech32/__init__.py", line 66, in bech32_decode
    if (any(ord(x) < 33 or ord(x) > 126 for x in bech)) or (
TypeError: 'NoneType' object is not iterable

Fixed in v0.1.4.


xparsedate can't parse nanoseconds

It seems xparsedate intends to support parsing a timestamp with nanosecond / 9-digit precision, but it actually doesn't work.
$ find -newerct @123.123456789
find: bad date @123.123456789

Thanks for the quick fix again!

is anyone interested in passing a fix along to BusyBox? (even if its not a patch)

Busybox and toybox are completely different codebases. (I started over from scratch when I left. I ported a few of the commands I'd written and thus owned the copyright for, and submitted a couple of toybox commands to busybox over the years, but there was signifcant porting work each way, and for stuff like sed and mount I just started over from scratch even though I wrote most of busybox's version.)
I check the busybox list every few weeks to see if they have interesting bug reports or feature requests that might apply to toybox as well, but I don't read their code without a good excuse (because license). Recently got embarassed by that, actually: http://lists.busybox.net/pipermail/busybox/2022-February/089462.html

(I only posted becuase BusyBox does not handle "nanoseconds", and only if someone was interested - I am not)

Busybox and toybox are completely different codebases.

Yep, I knew that, hence "even if its not a patch"

Recently got embarassed by that

;) Oh the joys of Life ..


Graphql insert mutation: omitting fields with default values

Hello! I've encountered an issue with the crate: the type generated by the crate from the schema doesn't allow me to omit fields that have a default value generated by the database (for example, a serial id or creation and update timestamps), but it requires me to provide values for them. If I try to give a value of None, Rust sends a query populating the fields as null, and the database returns an error because the fields are not nullable (and I don't want to insert null values anyway).
The following example replicates the mutation I tried to execute using two mock tables that mimic the behavior of the ones I need to manage:
create table hobbies (
    id serial primary key,
    creation_timestamp timestamp with time zone not null default now(),
    username text not null,
    type text not null,
);

I want to run this mutation with Hasura GraphQL:
mutation InsertData(
    $hobbies: [hobbies_insert_input!]!
) {
    insert_hobbies(
        objects: $hobbies
    ) {
        affected_rows
    }
}

With the following input:
{
    "hobbies": [
        {
            "username": "user1",
            "type": "reading"
        },
        {
            "username": "user2",
            "type": "gaming"
        }
    ]
}

Where hobbies_insert_input is defined in the auto-generated schema that I added to the repository files using graphqurl.
When I execute this through Rust, however, I encounter the following problem: the schema forces the type used in Rust to require that I provide a value for all fields in the row, including id, creation_timestamp, and update_timestamp. If I set these fields to None, rust translates them to null, and the database gives me an error because the columns cannot be null. If I omit those values, the compiler does not compile, and manually providing a value when those fields are populated by default by the database doesn't seem like a very elegant solution.
For now the only solution I found was to create a separate module for this mutation and manually create the schema that this module uses.

Been a while since this issue was opened, but FWIW a possible workaround is to use the "skip_serializing_none" flag on queries that may have None values that you don't want to send in the mutation. This more or less works for my case.
It would still not help for situations where either Null or Unspecified could be valid values of a field. It's probably be good to have something like an enum SpecifiableOption which has Some, None, and Unspecified as possible values, so we could differentiate in updates whether we intend to not provide a new value for a field, or to actually set the field to null.


Rework debugger backend

Have the Daffodil debugger backend zip added to the VSIX and have the extension use this file instead of trying to download the zip file from the releases url of the repo

How do you envision supporting multiple Daffodil versions?

I'm just wondering if this PR simplifies Daffodil debugger development, but as a side effect also reduces some usability while debugging during schema development.
Instead of removing the download capability intended for marketplace installs, could we just provide an improved method of installation for development environments?

I have to say, I hoped to be able to use the debugger with various
different Daffodil versions for support reasons.
My needs can be accommodated, however, by just having 2 (or more) different
VSCode installs, each with a different debugger + daffodil combination.
At some point I'd like to see if the debugger "dap" interface can be
back-ported to Daffodil 2.7.0. or even 2.6.0. We even have customers on
2.4.0.
I expect the cybersecurity community that is deploying daffodil to "scrub"
data, will all have this sort of broad set of releases to support as an
issue.
E.g., I expect to deliver training on DFDL/Daffodil in January to a
user/customer community and by then I'd hope to be able to use the
debugger. That customer will be using Daffodil 2.7.0.
On Wed, Oct 20, 2021 at 7:36 AM Steve Lawrence @.***>
wrote:

I would just consider that an implementation detail. Release notes might
want to mention what version of Daffodil is used for a release, but I think
adding the daffodil version to the vscode extension version just adds
confusion.
Note that if we go with an option where it is expected that the user must
install Daffodil, and the extension just finds it (or is told where it is
via a config option), then this isn't an issue at all, since the Daffodil
version isn't tied to the extension. This might get more complicated if
this extension grows to the point where it does depend on a certain
version. Distributing it in a zip resoles that issue.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/daffodil-vscode/issues/39#issuecomment-947580350,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALUDA5LMFF6N2MZEJ4A7V3UH2SVTANCNFSM5GHKBF7Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Up until this discussion I had been under the impression that there was value in conveniently running multiple versions of Daffodil in the IDE.  Pretty much like Mike described above.
I am not convinced that (1) users only want latest, or (2) it's acceptable to tell people to reinstall to get a different version, or that (3) the compiler should be distributed with the IDE, but I would like to get this moving forward.
So I'm good with taking the simple approach here and bundling the backend for the 1.0.0 version of the extension.  We only have a single supported Daffodil version at the moment anyways (3.1.0).
We do need to revisit this after 1.0.0 and address the lingering concerns.  I'll add a separate issue for that.


Carousels on home page

Looks fine, but hulls for large datasets very slow, so set those as featured = NULL in the database so the home page will load. Shelving that performance issue for now.


QP Separator Items: Show buttons when in section

The buttons on workspaceChunkSearch.ts should show since I am in that section like so:

Verification:

Open "Quick Search"

Arrow down through the items, notice that the button on the file name shows up when the active item is in that section:

Now hover over another section in the picker, notice that the button on the file name item shows up when your hover is on a search result in that file:

Try hovering & changing the active item in a few ways to make sure that the button in the file name item does go away when a search result in its section is no longer hovered and no longer has the active item.


Cursor not a the good position when trying to add cariage return.

Hello,
I tried to set my tex.vim like this (with the ^M in the second line, I also tried \^M) :
let b:pear_tree_pairs = extend(deepcopy(g:pear_tree_pairs), {
            \ '\\begin{*}': {'closer': '^M\\end{*}', 'until': '[{}[:space:]]'},
            \ '\\left(': {'closer': '\\right)'},
            \ '\\left[': {'closer': '\\right]'},
            \ '\\left|': {'closer': '\\right|'},
            \ '\\left\\{': {'closer': '\\right\\}'},
            \ '\\left.': {'closer': '\\right.'},
            \ '$$': {'closer': '$$'},
            \ '\\{': {'closer': '\\}'},
            \ '\\(': {'closer': '\\)'},
            \ '\\[': {'closer': '\\]'},
            \ '$': {'closer': '$'}
            \ }, 'keep')

The expected result is:
\begin{anything}|
\end{anything}

Instead I obtain:
\begin{anything}
|\end{anything}

Am I doing it the wrong way ? Is there a proper way to append a new line ? I don’t understand the behavior.
Thanks.

Hello!
Control characters like ^M are not supported in pair definitions, and opening/closing pair strings need to span a single line.
I'm not too familiar with TEX, but my impression is that most use cases for \begin and \end would be like:
\begin{anything}
% Some more stuff
\end{anything}

(with \begin and \end on separate lines)
OR
\begin{anything} Some more stuff \end{anything}

(with \begin and \end on the same line)
Pear Tree's behavior has those use cases in mind. If you remove ^M from the closing pair, simply hitting <CR> after \begin will move your cursor to the next line and push \end to the line below:
\begin{anything}|\end{anything}

after hitting <CR> becomes:
\begin{anything}
|
\end{anything}

I'm closing the issue as this exact use case is not supported, but I think the existing <CR> behavior works well enough if you remove ^M from the closing pair.

Thanks for your answer.
Actually, most of text commands take parameters:
\begin{minipage}[t]{\linegoal}
    content of the minipage
\end{minipage}

so, it would be more ergonomic to let the user go to the next line when he want.
As you said this use case is not supported, you can consider label it as a feature request.

I see.
I could potentially come up with a better way of enabling this, but a workaround with the capabilities Pear Tree has now would be to use the following mapping:
imap } <Plug>(PearTreeOpener_})<Plug>(PearTreeExpand)<BS>

Explanation:

<Plug>(PearTreeOpener_}) triggers the auto-close for the \begin{*} when } is pressed
<Plug>(PearTreeExpand) immediately expands the pair after it is inserted.
 moves back to the previous line

I've only played around with it a little bit, so there could be some quirks, but it appears to work alright.
Note: if g:pear_tree_repeatable_expand = 1, then the \end line will disappear until you exit insert mode


Update s3, sdk-core, sqs to 2.20.5

Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sdk-core
software.amazon.awssdk:sqs

from 2.17.295 to 2.20.5.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

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

Superseded by #572.


How to set defaultPositionUs of a Window in the Timeline?

I want to set default start positions of my playlist items (basically to resume from where the user left), to use when playlist item auto transitions to next item.
As per my understanding (I am not sure), the default start position is of the window in the timeline.
From Docs: Timeline.Window.defaultPositionUs. But, I can't find a way to set this defaultPositionUs parameter.
I use this function to set playlist items: setMediaItems. My playlist items are mp3 files (local and https source).

I think you can do this only if you own the media. This is AFAIK used with live streams.
The feature is a duplicate of #6373 and #1041 I think.
You can use MediaItem.ClippingProperties to clip a media item. However, this removes the part before the clipping start position. A user can't seek back beyond this because it's clipped. So this isn't exactly the same as a start position but about the same as a default position. :)
If this isn't helpful you need to use a seek for this. This was discussed in #1041

I got a solution based on this https://github.com/google/ExoPlayer/issues/9122#issuecomment-1436942499:
class MyMediaSource(mediaSource: MediaSource, private val startPositionMs: Long): WrappingMediaSource(mediaSource) {
    override fun onChildSourceInfoRefreshed(newTimeline: Timeline) {
        super.onChildSourceInfoRefreshed(MyTimeline(newTimeline, startPositionMs))
    }

    class MyTimeline(timeline: Timeline, private val startPositionMs: Long): ForwardingTimeline(timeline) {
        override fun getWindow(windowIndex: Int, window: Window, defaultPositionProjectionUs: Long): Window {
            return super.getWindow(windowIndex, window, defaultPositionProjectionUs).apply {
                defaultPositionUs = Util.msToUs(startPositionMs)
            }
        }
    }
}

val defaultMediaSourceFactory = DefaultMediaSourceFactory(this)
val mediaSourceFactory = object : MediaSource.Factory {
    override fun setDrmSessionManagerProvider(drmSessionManagerProvider: DrmSessionManagerProvider): MediaSource.Factory {
        return defaultMediaSourceFactory.setDrmSessionManagerProvider(drmSessionManagerProvider)
    }

    override fun setLoadErrorHandlingPolicy(loadErrorHandlingPolicy: LoadErrorHandlingPolicy): MediaSource.Factory {
        return defaultMediaSourceFactory.setLoadErrorHandlingPolicy(loadErrorHandlingPolicy)
    }

    override fun getSupportedTypes(): IntArray {
        return defaultMediaSourceFactory.supportedTypes
    }

    override fun createMediaSource(mediaItem: MediaItem): MediaSource {
        return MyMediaSource(
            defaultMediaSourceFactory.createMediaSource(mediaItem),
            mediaItem.mediaMetadata.extras?.getLong("startPositionMs") ?: 0,
        )
    }
}

player = ExoPlayer.Builder(this)
    .setMediaSourceFactory(mediaSourceFactory)
    .build()


Test dummy4

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #296 (a305a61) into main (78ea8be) will decrease coverage by 0.08%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##               main     #296      +/-   ##
============================================
- Coverage     36.96%   36.88%   -0.09%
+ Complexity      887      886       -1
============================================
  Files           179      179
  Lines          7010     7028      +18
  Branches        855      857       +2
============================================
+ Hits           2591     2592       +1
- Misses         4182     4198      +16
- Partials        237      238       +1

Impacted Files
Coverage Δ

...rch/securityanalytics/action/GetAlertsRequest.java
0.00% <0.00%> (ø)

...h/securityanalytics/action/GetFindingsRequest.java
0.00% <0.00%> (ø)

...curityanalytics/action/GetMappingsViewRequest.java
0.00% <0.00%> (ø)

...securityanalytics/action/ValidateRulesRequest.java
57.14% <0.00%> (ø)

...securityanalytics/mapper/IndexTemplateManager.java
0.00% <0.00%> (ø)

...search/securityanalytics/mapper/MapperService.java
4.61% <0.00%> (+0.06%)
:arrow_up:

...earch/securityanalytics/model/DetectorTrigger.java
56.14% <0.00%> (ø)

...a/org/opensearch/securityanalytics/model/Rule.java
0.00% <0.00%> (ø)

...ytics/resthandler/RestAcknowledgeAlertsAction.java
0.00% <ø> (ø)

...nalytics/resthandler/RestDeleteDetectorAction.java
0.00% <0.00%> (ø)

... and 25 more

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


Document that user, password, and db name should be urlencoded

I ran into an issue today deploying usage of dj-database-url to production, which had a password that included a non-url-safe character. Since putting usernames / passwords into urls is not something I've done a lot of, I didn't know that quoting was necessary (and I wasn't even sure it would work.)
I ended up finding this web page, which mentioned that quoting was needed: https://support.brightcove.com/special-characters-usernames-and-passwords
The error I got was:
File "/usr/lib/python2.7/urlparse.py", line 214, in urlsplit
    raise ValueError("Invalid IPv6 URL")

It would be great if the documentation for dj-database-url could include a note about this. It doesn't seem like there would be a good way to catch this error and re-raise with more helpful information, unfortunately, but if I'm mistaken then that would also be great.

This requirement has been added to the readme.rst in commit 836740dd814878c


Support several expected values for the Accept header

This PR is mostly refactoring, with a rough pass at the functionality needed for #342. I haven't added tests yet since I'd like feedback on the approach. If it looks good, I'll add a commit to this PR with tests.
cc @jsancio @BenWhitehead

Can we add a test with some case classes and MediaTypes that are disconnected from all our existing model and use them to create a test that shows an endpoint can return different responses? Looking at the signature of EndpointHandler I'm failing to see how multiple response types will/would be expressed.

Yep @BenWhitehead that's the one thing I haven't dealt with yet. This change will work for "superficial" modifications to the response body, like omitting or including an optional field. I believe we could technically get away with that for 1.8 if we added minDcosReleaseVersion to our DescribeResponse model as an optional field, which we always fill in for "v2" responses.
In the long term though we'll need to allow for arbitrary response objects, so I'll look into the test you suggested.

Status update: after the team discussion today, I'm looking into changing the approach slightly. I'll add a comment here once I verify that the new approach works.

I have a way forward, just tying up some loose ends now. New commits coming by EOD.

@jsancio @BenWhitehead updated. This should satisfy the requirement of having separate case classes for each version of the response object. Please review and I can make changes as necessary.

I'm out of time today, but I'll make sure to resolve the conflicts and merge this first thing tomorrow morning.

Yes! 🎉 🎈


Fix offsets for Class Properties

Detailed explanation:
https://github.com/loginov-rocks/UbxGps/issues/27

Closing as per https://github.com/loginov-rocks/UbxGps/pull/26#issuecomment-1504275417


Animations

Adding in simple page level animations. Nothing fancy but just setting it up.
Don't forget to do an "npm install" as this uses a new package "react-addons-css-transition-group"

Let's DRY this up a little bit and make the transition group a component on it's own. Make a new component file (copy/paste one of the simple ones).
In the new component's render method, have it return <ReactCSSTransitionGroup transitionAppear={true} transitionAppearTimeout={400} transitionLeave={false} transitionEnterTimeout={400} component="div" transitionName={transitionNames}>{this.props.children}</ReactCSSTransitionGroup> and make sure you are expecting children in your PropTypes.
After that, in the router, file, wrap all the  in a path-less component.

Now, all the routes are "children" of this transition component, and will be rendered inside the {this.props.children}.

@chrisheninger,
I set up a new react component the way you described for the animations and pushed up my changes to the Animations branch. It sorta works on initial load but it is not triggering during DOM changes. I think this stackoverflow is relevant.... http://stackoverflow.com/questions/29478579/reactcsstransitiongroup-doesnt-seem-to-be-applying-animations
Advice? hahah.


new line under Windows

Under Windows 10, goawk only output 0x0a for a newline. Windows splits the carriage return and new line characters. Under Windows, goawk should output 0x0d 0x0a
In other words, it need to output \r\n under Windows instead of just \n
Reference:
https://en.wikipedia.org/wiki/Newline#Representation
https://stackoverflow.com/questions/14493867/what-is-the-most-portable-cross-platform-way-to-represent-a-newline-in-go-golang

It looks like the problem may stem from interp.go:137
p.outputRecordSep = "\n"

Is there a way to make this cross-platform?

Thanks. Fixed in https://github.com/benhoyt/goawk/commit/b13da806d9e3b9d5770e0ebb0f10cfb33af97f6c

This is working well.
The only issue I see is:
works correctly:
dir c:\windows\system32 /s/a-d/-c | .\goawk "{sum+=$4} END {print sum}"

still only outputs 0x0a and not 0x0d 0x0a:
dir c:\windows\system32 /s/a-d/-c | .\goawk "{sum+=$4} END {printf('%s\n', sum)}"

I realize that I can get proper output by using \r\n instead of just \n however I believe expected behavior dictates that \n under Windows truly outputs \r\n because \n is traditionally represents newline character(s) regardless of OS.

Good call. Fixed in https://github.com/benhoyt/goawk/commit/7c84d80620243796b2e1511e2bcb85c5fb229b36


Add a chi example for sync users

I submit a yuml template for migrating users when expanding capacity, so that people in need can use it.

@Slach Hello, I have revised and submitted it according to your comments,thanks.

@ccsxs thanks for contribution


Footer layout

How can i change the footer layout?
I would like to add items side-by-side, as mentioned in the issue #111.
I would really appreciate if someone could help me with this one. :)

@giuwolf Have you tried the code I posted in that issue? https://github.com/eddiesigner/liebling/issues/111#issuecomment-571581176

It worked! Thanks a lot :)


add async and defer attributes to js requirements

as discussed in  #4427 rebased to master and replaced parameters for combine_files() with options array.
Please let me know if you need any more changes.
Cheers.

OK, final thing, can we add some docs for this? see https://docs.silverstripe.org/en/3.2/developer_guides/templates/requirements/

@chillu @dhensby @tractorcow how does this overlap with the JS work y'all been doing lately?

This is still relevant, since Requirements can be used for all websites, not just the CMS (where async and defer likely won't play a role)
I tried to rebase this for 15 minutes on master, and there's substantial conflicts due the addition of $options and provides functionality to Requirements since this PR was raised. I'm going to close it for now, but @xini is welcome to reopen with a fixed version. I promise we'll look at getting it over the line quicker this time!

I had completely forgotten about this one. I'll see what I can do.


local Bing correctly, but server  does not work

I'm running into some issues,
When I deploy locally bing returns correctly,
When I deploy on the server but no results are returned,
Have you ever encountered this? I guarantee the code is exactly the same

On the server, try doing a curl GET request to https://www.bing.com/turing/conversation/create. You should see Sorry, you are not allowed to access this service if it works, but maybe Bing blocks some IPs from VPS networks entirely.

I tried it and found that my host is cn.bing.com, so can I support the configuration of host?

I tried it and found that my host is cn.bing.com, so can I support the configuration of host?

@yuwangi Can you first turn on debug mode for the Bing client and try it on the server?

国内ip会强制跳转cn.bing.com不支持bingAI  你可以尝试host能否解决 如果不行就需要翻墙

@waylaidwanderer
I tried it, changing the server code to cn.bing.com is ok.
I originally wanted to try to use nginx reverse proxy directly, but the cookie and authentication information will be discarded, so I hope to support host customization, thks

In setting.js I defined the  host: 'https://cn.bing.com',
But it is not work
ReferenceError: host is not defined
at BingAIClient.createNewConversation (file:///var/chatgpt-api/src/BingAIClient.js:19:41)
at BingAIClient.sendMessage (file:///var/chatgpt-api/src/BingAIClient.js:120:62)
at Object. (file:///var/chatgpt-api/bin/server.js:97:31)
at preHandlerCallback (/var/chatgpt-api/node_modules/fastify/lib/handleRequest.js:128:37)
at preValidationCallback (/var/chatgpt-api/node_modules/fastify/lib/handleRequest.js:112:5)
at handler (/var/chatgpt-api/node_modules/fastify/lib/handleRequest.js:76:7)
at /var/chatgpt-api/node_modules/fastify/lib/contentTypeParser.js:192:9
at AsyncResource.runInAsyncScope (node:async_hooks:203:9)
at done (/var/chatgpt-api/node_modules/fastify/lib/contentTypeParser.js:186:14)
at Parser.defaultJsonParser [as fn] (/var/chatgpt-api/node_modules/fastify/lib/contentTypeParser.js:292:5)
ReferenceError: host is not defined
at BingAIClient.createNewConversation (file:///var/chatgpt-api/src/BingAIClient.js:19:41)
at BingAIClient.sendMessage (file:///var/chatgpt-api/src/BingAIClient.js:120:62)
at Object. (file:///var/chatgpt-api/bin/server.js:97:31)
at preHandlerCallback (/var/chatgpt-api/node_modules/fastify/lib/handleRequest.js:128:37)
at preValidationCallback (/var/chatgpt-api/node_modules/fastify/lib/handleRequest.js:112:5)
at handler (/var/chatgpt-api/node_modules/fastify/lib/handleRequest.js:76:7)
at /var/chatgpt-api/node_modules/fastify/lib/contentTypeParser.js:192:9
at AsyncResource.runInAsyncScope (node:async_hooks:203:9)
at done (/var/chatgpt-api/node_modules/fastify/lib/contentTypeParser.js:186:14)
at Parser.defaultJsonParser [as fn] (/var/chatgpt-api/node_modules/fastify/lib/contentTypeParser.js:292:5)

@eu09 Oops, thanks for catching that. Fixed in 1.17.6.


test: GH-132: Add test for SpringPayloadAnnotationTypeExtractor

Add log message to indicate mismatch java versions
Co-Authored-By<EMAIL_ADDRESS>

Seems like I misled you, I already merged it yesterday (Timon also created this PR earlier)


voice recognition added

Added voice input feature for textboxes

kindly pull and resolve the conflicts.
Btw, does this fix the voice command? @adityabisoi


changing the name of react to rudra

In the react version still the name is React App should I chang this to RUDRA

In public/index.html file?

yes

@Ayush2020012016 go for it

I want to contribute to this issue. can you assign me??

@gargkrishna the issue is assigned to other person, so let wait till tonight, if @Ayush2020012016 didn't do any PR I will assign this issue to you.

@gargkrishna go for it, try to submit PR till midnight

I want to contribute to this issue.Can you assign me?

its giving 403 when i am pushing.


Can't build emscripten example

Hi, I'm sure it's trivial (I'm new to C/C++) but when I follow the instructions on the README I get fatal error: 'glfm.h' file not found:

If I manually copy theglfm.h into the source directory the example builds but I get WARNING:root:emcc: cannot find library "glfm" and the emscripten example loads in browser but just shows black square for content.

I'm guessing this error happens when using the CMakeLists.txt in the example directory directly. Instead use the CMakeLists.txt from the root directory with the -DGLFM_BUILD_EXAMPLE=ON flag. (The root CMakeLists.txt includes the example CMakeLists.txt, not the other way around)

Ah yes, I thought I had to run this from the example directory for some reason. Running from root fixed everything, thanks.


fix: building sphinx rtd theme

Better follow the rtd config instructions from
https://docs.readthedocs.io/en/stable/config-file/v2.html#packages
This has the advantage of not duplicating the requirements specification

Sorry @clintonb I think I just understood your comment on the docs ci job! Hopefully this will fix it...


Confusing notification text on "stop logging" action

Version 0.5.19 private kit from play store, verified exists in development code as well.

Type of phone: Android
OS Version: 9
Model: Moto X4

Steps to reproduce
Tap settings icon in top left to get to the dashboard
Tap the "stop logging" button.
Notification text reads: "Location tracking was Disabled" (good) followed by "PrivateKit requires location services" (technically true, but confusing, as it implies the app does not have access to location services.)
Expected Behavior
Suggest we change the second line to: "Stopped at user request" in the case tracking was disabled by the user as opposed to the loss of location services. [Or just "Stopped" if the stop action can be triggered by things outside of user requests...]

I suspect this is caused by a lazy reuse of the label.location_disabled_message in the app/services/LocationService.js file for both the stop action and the checkStatus action.
Suggested fix: Add a new label "location_stopped_message", get it translated for all languages, and differentiate between a stop action and an "if (!locationServicesEnabled)" check failure.

I can start looking this issue.

yes please


Support workspaces for multi-state migrations

Implements the workspace feature request from #3
Main logic is in setupWorkDir() where I create the local workspace folder required before calling OverrideBackendToLocal()
Would love some feedback on this PR @minamijoyo

@dhaven Thank you for working on this! I'll review this after studying how workspace works in Terraform core. Please wait.

Hi, @dhaven. Sorry for the late review.
Although creating a local workspace directory terraform.tfstate.d without the workspace new command depends on undocumented behaviors, I agree it's a reasonable compromise under the current technical limitations.
I think your implementation is almost good. I left some minor comments. Please check. Thanks again!

@dhaven Shipped it in v0.2.6. Thank you for your great work 🎉


Fixed shouldBeAbleUpdateBudget

The stubbed post and put request are now called properly

Good stuff, Put your commit in my PR over at #5


gyp: don't print xcodebuild not found errors

As node-gyp rebuild doesn't seem to need xcodebuild, we don't need to be printing the error every time GYP is run.
Refs: https://github.com/nodejs/node-gyp/pull/1057
Refs: https://chromium-review.googlesource.com/c/492046/
Refs: https://github.com/nodejs/node-gyp/issues/569
As the previous attempts haven't landed, I thought I'd aim for a minimal diff. This seems to remove all node-gyp build warnings during npm install of a native module, and errors from running configure in node core, although for some reason the first line in Node's built-in gyp uses xcrun rather than xcodebuild.
This is caused by me running a "getting started with node" course, and having half the audience stuck because "there was an error in the build".


Auto-suggest's addition of 'dependsOn' can corrupt existing task lines when adding a new ID

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
When auto-suggest is adding a dependency, and needs to add an ID to the task being newly-depended on, the original data in that task should not be corrupted.
Current behaviour
When:

auto-suggest is adding a dependency,
and it needs to add an ID to the task being newly-depended on,
then if:

that line had extra spaces between the existing emojis
and the task is in the same file as the task getting the new dependency....

the original data in that task is corrupted.
Steps to reproduce

Run a build of Tasks with the auto-suggest for dependencies enabled, by setting showDependencySuggestionsDefault to true in src/Suggestor/Suggestor.ts
Create a new note, with any name
Paste in this content - the 4 spaces before the calendar emoji are important:

- [ ] #task Task 1    📅 2024-05-27
- [ ] #task Task 2

Put the cursor at the end of Task 2
Type [space] dep
Select ⛔ Task depends on id
Select Task 1 from the popup menu

Expected result:
- [ ] #task Task 1 🆔 bbaupb 📅 2024-05-27
- [ ] #task Task 2 ⛔ bbaupb

Actual result - note the additional -27 at the end of the line
- [ ] #task Task 1 🆔 bbaupb 📅 2024-05-27-27
- [ ] #task Task 2 ⛔ bbaupb

Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[X] macOS
[ ] Windows

Obsidian Version
1.6.1
Tasks Plugin Version
Post 7.3.0 development version
Checks

[ ] I have tried it with all other plugins disabled and the error still occurs

Possible solution
The problem occurs when the original line was longer than the new line that Tasks would write its place, because of the additional whitespace.
In the following code, toFileLineString() should be changed to originalMarkdown.
https://github.com/obsidian-tasks-group/obsidian-tasks/blob/8c4880fa54aff6118f20e1546c16241b2880e76b/src/Suggestor/EditorSuggestorPopup.ts#L101-L109
This also needs to be tested with tasks in Callouts and Blockquotes - non-nested and nested.

My notes from testing my proposed bug-fix:
Original example
- [ ] #task Task 1    📅 2024-05-27
- [ ] #task Task 2

After the fix, the new result is correct (the new ID is random, so will usually differ on each run):
- [ ] #task Task 1 🆔 8t4oie 📅 2024-05-27
- [ ] #task Task 2 ⛔ 8t4oie

block quote in a callout:
Tested with a block quote in a callout:
> [!NOTE]
> > - [ ] #task Task 1    📅 2024-05-27
> > - [ ] #task Task 2

Gives:
> [!NOTE]
> > - [ ] #task Task 1 🆔 dom7kt 📅 2024-05-27
> > - [ ] #task Task 2 ⛔ dom7kt

Nested tasks
Tested with
- Stuff
    - asdfasd
        - [ ] #task Task 1    📅 2024-05-27
    - [ ] #task Task 2

Also fine:
- Stuff
    - asdfasd
        - [ ] #task Task 1 🆔 0uqqph 📅 2024-05-27
    - [ ] #task Task 2 ⛔ 0uqqph


Links to questions are not permanent

Editing a question generates a new slug for the URL, so all links pointing to the question are then broken.
Example:

I create a question "How to access the management console of a Google Cloud project?"
The link to the question is "https://faq.zenika.com/#/q/how-to-access-the-management-console-of-a-google-cloud-project"
I post the link to Workplace
I edit the question to "How to access the management console of a Google Cloud project? Hello"
The link becomes "https://faq.zenika.com/#/q/how-to-access-the-management-console-of-a-google-cloud-project-hello"
The old link does not work anymore, so the link on Workplace is broken

However having the title in the URL is nice.
A common solution is to add add the id of the linked resource at the end of the URL :
https://faq.zenika.com/#/q/how-to-access-the-management-console-of-a-google-cloud-project-hello-123456
This is used in many news web sites.

So if I follow what you are saying, we should have a url like this: https://faq.zenika.com/#/q/my-title-123456
And I should only take the uid at the end of the url to fetch the ressource ? This way, whatever the title is, we still get the right ressource.
Should I add a redirection if the title is wrong ? /q/my-title-123456 => /q/my-new-title-123456
So that people doesn't try to share the old url.

Yes that's what I meant.
A 301 redicrect is not crucial, but if it can be done easily it's a plus :+1:.


Mouse-pasting text with embedded HTML anchor yields unrenderable page and disrupted browsing session

Describe the bug

Mouse-pasting text which contains an HTML anchor between regular text will corrupt the page when you save it.
The browser will show the  content area with an error message and javascript traceback (details below).
Navigating to other pages will continue to show the error until you refresh the browser.
Returning to the page with the mouse-pasted anchor text will still fail to render properly, and require refreshing again when you navigate to other pages.

To Reproduce
Steps to reproduce the behavior:

Copy the indicated text on sample paste irritant page into a new Volto page and save the page.

The content area of the resulting page will have a traceback, starting with:
Sorry, something went wrong with your request
Cannot read properties of null (reading 'toLowerCase')
...

Navigate to some other page on the site and continue to see the traceback until you refresh your browser view.

Return to the new page to once again see the traceback.

I haven't found a way to recover a page with the pasted text. If I pasted text that has an anchor into a page with other text, that's lost unless I can undo to before the paste.

Expected behavior
The pasted text should be reproduced with an identical anchor.
Software (please complete the following information):

OS: Windows, Android, probably all others
Browser: Chrome, Edge, etc
Volto Version 16.18.0
Plone Version 6.0.4
Plone REST API Version 8.37.0

It looks like the problem is not present in a more recent Volto frontend. The anchor past problem does not occur when pasting into Volto 16.21.2. The anchor is changed in the paste, but it does not corrupt the page. I am closing this issue as fixed.

Hmm. Trying again to close with comment.