Validation rules in integrity zome

Validations

[ ] base of the TA link MUST be the (orig pubkey) of the DHT-level creator of the link
[ ] link tag

[ ] MUST follow all of the rules of the data format: https://github.com/trustgraph/trustgraph-holochain#data-format
[ ] MUST have exactly 4 null bytes (no more or less)

[ ] only agent who created a TA can delete it
[ ] IMPL: anywhere we have existing validation code

[ ] move it to integrity zome
[ ] if appropriate, still call it from the coord zome

-CRUD operations can only be accessed by the acting agent
-max data size for entries and linktags
-check that TA exists
-registration to check that agent exists in DHT
-permissions for shareable TGs

-CRUD operations can only be accessed by the acting agent -max data size for entries and linktags -check that TA exists -registration to check that agent exists in DHT -permissions for shareable TGs

Awesome thanks @dauphin3  -- incorporated some of these into description.

@harlantwood for the first box of validation above, would not this only be true for forward TA's... as the base of a reverse could be AnyLinkableHash

@harlantwood for the first box above in validations, would this only be true for forward TA's?  Since the base of reverse could be AnyLinkableHash
in which perhaps this could open up a potential use case for LinkTypes::TrustAtom(Forward)  ??
or just LinkTypes::Forward & LinkTypes::Reverse

@harlantwood for the first box above in validations, would this only be true for forward TA's? Since the base of reverse could be AnyLinkableHash

The conceptual TA base is as you say.  But the holochain link base is always the agent, regardless of direction.

correction: will need to handle forward & reverse

@harlantwood perhaps consider LinkType Directions, review found in #79


Revert "Merge pull request #2639 from Hixie/always_specify_types"

This reverts commit f41b3411da35929b09009e47cb52474389e42874, reversing
changes made to e33d8d96212f3e337a6660f1eb1118bffc945bf5.
This was a bad check-in due to my mangling uploading a new version of the branch from a different machine.
This reverts https://github.com/flutter/flutter/pull/2639 and will be replaced by https://github.com/flutter/flutter/pull/2640

TBR @abarth to fix tree


Convert .trt to .engine file

Hi,
I am starting with deepstream and tensorRT and I was wondering if you can explain or provide a clear code than the one provided on the NVIDIA official page tutorial on TensorRT to convert from .trt file to .engine file.
Thank you and sorry for the inconvenience

Hi @RogerAylagas, sorry the late response. Could provide a link to the page you are referring to?
Just to be clear, this repository does not require that feature since DeepStream directly can consume the onnx model file. But I am happy to take a look at the link of the tutorial page if you can share it here.

Closing due to lack of activity.


Auth UI sometimes get stuck (keeps loading, no crash, no error log) when using google sign in

Welcome to FirebaseUI and thanks for submitting an issue!
Please take a look at open issues, as well as resolved issues, to see if your issue is either already being addressed, or has been solved by someone else.
If not, please feel free to fill in the following info so we can help faster!
Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a GitHub issue.
For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: Moto OneZoom
Android OS version: 10
Google Play Services version: 21.24.13
Firebase/Play Services SDK version: bom 28.1.0
FirebaseUI version: 7.2.0

Step 3: Describe the problem:
When using google sign in method sometimes the progress get stuck after account selection, the UI keeps the loading animation, but don't complete the login, no error code.
Canceling the login and starting it again works fine, the login now completes successfully.
When using smartlock the screen is covered with a dimmer but the login never completes.
My app was working correctly some weeks ago, I didn't changed anything related to login, signatures, keys, etc.
Steps to reproduce:

Auth UI using Google, email and twitter
Start Auth UI
Try to login using google

Observed Results:

What happened?  This could be a description, logcat output, etc.

The account choose windows opens, but after selecting one the UI keeps loading and the login never completes.
Expected Results:

What did you expect to happen?

The login completes successfully
Relevant Code:
val providers = arrayListOf(
                    AuthUI.IdpConfig.EmailBuilder().build(),
                    AuthUI.IdpConfig.GoogleBuilder().build(),
                    AuthUI.IdpConfig.TwitterBuilder().build()
            )
startActivityForResult(
                    AuthUI.getInstance()
                            .createSignInIntentBuilder()
                            .setAvailableProviders(providers)
                            .setLogo(R.drawable.ic_launcher_login)
                            .setIsSmartLockEnabled(false)
                            .build()
                    , 1111
            )

When using google sign in method sometimes the progress get stuck after account selection, the UI keeps the loading animation, but don't complete the login, no error code.

In this case it sounds like something is going wrong with Google Sign In, which is not part of FirebaseUI (we just depend on it). Unfortunately without any logs or a way to reproduce this consistently in a test I don't think there is anything we can do about this. There's no way to know where the issue is.

I have received a bug report stating the exact same issue. Nothing happens when pressing the google sign in button. If it helps, the user was not using the built in 3g connection, but on the phone.


not able run this model in docker

Traceback (most recent call last):
  File "kfserving-absa.py", line 30, in <module>
    model.load()
  File "kfserving-absa.py", line 14, in load
    self.nlp = absa.load()
  File "/usr/local/lib/python3.7/site-packages/aspect_based_sentiment_analysis/loads.py", line 34, in load
    model = BertABSClassifier.from_pretrained(name, config=config)
  File "/usr/local/lib/python3.7/site-packages/transformers/modeling_tf_utils.py", line 730, in from_pretrained
    model(model.dummy_inputs, training=False)  # build the network with dummy inputs
  File "/usr/local/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer.py", line 968, in __call__
    outputs = self.call(cast_inputs, *args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/aspect_based_sentiment_analysis/models.py", line 150, in call
    logits = self.classifier(pooled_output)
  File "/usr/local/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer.py", line 964, in __call__
    self._maybe_build(inputs)
  File "/usr/local/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer.py", line 2398, in _maybe_build
    self.input_spec, inputs, self.name)
  File "/usr/local/lib/python3.7/site-packages/tensorflow/python/keras/engine/input_spec.py", line 196, in assert_input_compatibility
    str(x.shape.as_list()))
ValueError: Input 0 of layer classifier is incompatible with the layer: : expected min_ndim=2, found ndim=0. Full shape received: []

aspect-based-sentiment-analysis==2.0.1
tensorflow==2.2.0
tried different tf version but issue same

I'm having the same problem outside of Docker.

Please update to the version 2.0.2 - it should work now 😊


Bifrost Plot Sizing

Define the size of bifrost plot so that the faulting and adding color, opacity, or size doesn't invade the side bar

Fix the height of the side bar?


Right-side Hangouts chat (toggled in Advanced setting) has broken styling

This problem was reported in the Support section of the Chrome web store:

Moving the chat to the right-hand side (where it belongs) via the Advanced settings breaks the layout right now. Could you please look into that? Loving it otherwise!

I don't use Hangouts, so I wouldn't have known. It turns out that the background color is blue, but should be white. There's an element acting as a left border, and its background is set to blue. It looks more natural as #e5e5e5.
Also, the compose and reminder floating buttons display on top of it, but should be moved to the left instead.

Solved in PR #32 , should be merged soon


feat(coverage): option for raw V8 script coverage

What I did

Add includeRawScriptCoverage option to include full V8 precise coverage entry to result of page.coverage.stopJSCoverage()

Motivation
It's related to issue #985 . I want to map Puppeteer's coverage result with source-map with https://github.com/istanbuljs/v8-to-istanbul . But The Puppeteer's coverage result's range objects are edited by convertToDisjointRanges function and v8-to-istanbul can't map .
So, I want this option to get V8 raw coverage result, whose functions' ranges are not joined.
[FYI] The similar method in Praywrite returns raw V8 coverage result. https://playwright.dev/#version=v1.4.2&path=docs%2Fapi.md&q=chromiumcoveragestopjscoverage

hmmm, extra test CI job was failed. 🤔  Tell me how to fix it ?

@mathiasbynens @jackfranklin  Can you please merge this?

How about this? Let me know if anything needs to be fixed !


Add pgbench benchmark

Add pgbench benchmark and support for Postgres managed relational databases on AWS (RDS) and GCP (CloudSQL)

LGTM. Already pre-reviewed when the code is in private branch.
Merge whenever your tests pass and you feel it's ready.
Thanks for all the hard work! ;)


How to install on Windows Subsystem for Linux (WSL)?

I have some bash scripts that leverage az cli. I'd like to incorporate m365 cli. To run the bash scripts on Windows, I've enabled WSL (Ubuntu 20). I can't seem to install m365 cli in WSL though.
Steps to repro:
- From PowerShell prompt in Windows 10, verify node and npm are installed:
PS C:\Program Files\nodejs\node_modules\npm\bin> node -v
v14.15.5

PS C:\Program Files\nodejs\node_modules\npm\bin> npm -v
6.14.11

- Run wsl command from within C:\Program Files\nodejs\node_modules\npm\bin
PS C:\Program Files\nodejs\node_modules\npm\bin> wsl

To run a command as administrator (user "root"), use "sudo <command>".
See "man sudo_root" for details.

user@computer:/mnt/c/Program Files/nodejs/node_modules/npm/bin$

-Run the installation command: ERROR
user@computer:/mnt/c/Program Files/nodejs/node_modules/npm/bin$ npm i -g @pnp/cli-microsoft365

-bash: /mnt/c/Program Files/nodejs/npm: /bin/sh^M: bad interpreter: No such file or directory

-Research here suggests to add some stuff to the command: ERROR
user@computer:/mnt/c/Program Files/nodejs/node_modules/npm/bin$ node.exe 'npm-cli.js' npm install -g @pnp/cli-microsoft365

Usage: npm <command>

where <command> is one of:
    access, adduser, audit, bin, bugs, c, cache, ci, cit,
    clean-install, clean-install-test, completion, config,
    create, ddp, dedupe, deprecate, dist-tag, docs, doctor,
    edit, explore, fund, get, help, help-search, hook, i, init,
    install, install-ci-test, install-test, it, link, list, ln,
    login, logout, ls, org, outdated, owner, pack, ping, prefix,
    profile, prune, publish, rb, rebuild, repo, restart, root,
    run, run-script, s, se, search, set, shrinkwrap, star,
    stars, start, stop, t, team, test, token, tst, un,
    uninstall, unpublish, unstar, up, update, v, version, view,
    whoami

npm <command> -h  quick help on <command>
npm -l            display full usage info
npm help <term>   search for help on <term>
npm help npm      involved overview

Specify configs in the ini-formatted file:
    C:\Users\user7\.npmrc
or on the command line via: npm <command> --key value
Config info can be viewed via: npm help config

<EMAIL_ADDRESS>C:\Program Files\nodejs\node_modules\npm

How can I install m365 cli in WSL?
Thank you

Aha... figured the syntax out:

 node.exe 'npm-cli.js' install -g @pnp/cli-microsoft365.
This had to be run from within the /mnt/c/Program Files/nodejs/node_modules/npm/bin directory in WSL.

This was so much cleaner way to go! Just install on the Linux side of WSL! https://docs.microsoft.com/en-us/windows/nodejs/setup-on-wsl2


Download only once

Especially ethstats takes pretty long to get for multiple nodes, though then setup is more single machine specific.

Hi Peter,
Can you re-try running it again. Do you see it re-download ethstats? Docker should only do this once, once you don't wipe your docker images. It does take a while, they're over 1.3g in size combined:
buythewhale/ethstats_monitor   latest              5adf546983cd        5 months ago        477.2 MB
buythewhale/ethstats           latest              9019cb3b4556        5 months ago        899.8 MB

True


please upload more famous diffusion model about image to image，thanks

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

Hi @qianxiaqiyue, we are now trying to move these models into the core. The development is in the gen-ai-dev branch.
https://github.com/Project-MONAI/GenerativeModels/tree/main/generative/networks/nets
Thanks.


[PE-D][Tester D] Undo Command Does Not Work for Grade Command

Based on the UG, undo should undo commands that change data. However, this does not work for grade. It could be that the grade command was not saved in the command history? If grade is not meant to change data, then this is a feature flaw, as it works for adding assignment. Perhaps indicating which commands change the data (and thus can be undone) in the UG would also be helpul?
Steps to reproduce bug:

adda a/HELLO
grade 1 20 a/HELLO
undo

Expected: Undo undoes the grade command.
Actual: Undo undoes the command (add assignment in the example) before the grade command.

Labels: severity.Medium type.FunctionalityBug
original: getsquared/ped#4

Duplicate of #104


[clkmgr] Double check transactional clock enable logic

The transactional clock enable logic below may need a second look:
https://github.com/lowRISC/opentitan/blob/a7b30e63694cdf92ca70ba61f15492d6039a8e95/hw/ip/clkmgr/rtl/clkmgr_trans.sv#L31-L38
Specifically, the recent MUBI value change in #12153 leads to a lint error in this block now:
E   EQ_OP_BITLEN:   clkmgr_trans.sv:37   Bitlength mismatch between 'idle_cnt' length 3 and 'MuBi4True' length 4, instance 'clkmgr.u_clk_main_aes_trans' of module 'clkmgr_trans' (IdleCntWidth=3 ('$clog2(TransIdleCnt + 1)'),MuBi4Width=4,TransIdleCnt=6 ('int '(MuBi4True)'))                 New

This is because the new TRUE value does not need 4bits, but only 3bits to be represented correctly, and the code above sizes the counter accordingly.

ah yes thanks for the reminder. Let me have a look.
@vogelpi i think at one point suggested I put an assert to make sure it's greater than a certain value.
I THINK this still works, but will have a look.

Could you just do localparam int IdleCntWidth = int'($bits(MuBi4True))?
I was mostly worried that if the delay gets suddenly much smaller or much bigger because of modified values, this could trigger issues in the clock enable feature. Previously, we were counting to 10, now it's just to 6. It's probably still okay as @tjaychen says.
Alternatively, one could also reset the counter to MuBi4False = 0x9 and then count to 0x6 which would require 13 cycles. But I think the prim_count doesn't allow overflows.

i need to remind myself a bit on what my original intent was.
There's a small chance i wanted the "enable" value to be mubi like to guard
against glitches to all 0's and all 1's.. and the mubi's just happen to be
a convenient set for that.
It might make sense to just de-couple from Mubi from here, and declare my
own value.
On Fri, Apr 22, 2022 at 12:50 AM Pirmin Vogel @.***>
wrote:

Could you just do localparam int IdleCntWidth = int'($bits(MuBi4True))?
I was mostly worried that if the delay gets suddenly much smaller or much
bigger because of modified values, this could trigger issues in the clock
enable feature. Previously, we were counting to 10, now it's just to 6.
It's probably still okay as @tjaychen https://github.com/tjaychen says.
Alternatively, one could also reset the counter to MuBi4False = 0x9 and
then count to 0x6 which would require 13 cycles. But I think the
prim_count doesn't allow overflows.
—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/12274#issuecomment-1106121566,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXJ2Z3L46ZBQ43HZSLVGJK23ANCNFSM5UA5YUAA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Maybe it's better to de-couple and fix the values, but I must say once more your original design Tim is really smart because the output value of the counter is directly the Mubi value. There is no other logic in the path that could lead to a single-point of failure such as a single-bit comparison of the counter value that then generates the Mubi.


test aws arm images via cloud API

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

This is perfect! :)

LGTM, so are we merging this and adding 9.0-ga in a followup?

@jrusz Yes let's do that


Discord with Vencord not working/ app is damaged

Describe the bug
The Discord App is damaged when installed with Vencord.
Thanks in advance for looking into this :)
Steps To Reproduce
Steps to reproduce the behavior:

Install Discord like this:

{ pkgs, lib, vars, ... }:
{
  environment.systemPackages = with pkgs; [
    (discord.override {
      withVencord = true;
    })  # Discord with Vencord
  ];
}

Open Discord

Expected behavior
I expected the app to open normally as described here:
https://github.com/NixOS/nixpkgs/issues/259297#issuecomment-1752114901
Screenshots

Additional context
I tried different scenarios like installing both the override and discord, i removed every remains of discord and installed only discord and then afterward added the override and more, but everything i tried doesn't seem to work
Metadata

 - system: `"aarch64-darwin"`
 - host os: `Darwin 24.2.0, macOS 15.2`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/Users/user/.nix-defexpr/channels/nixpkgs`

Notify maintainers
@Artturin
@donatello
@Infinidoge
@jopejoe1
@Scrumplex

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

CC @FlafyDev @NotAShelf @DontEatOreo for Vencord
I don't have a darwin machine to test this sadly.

Try to open it so it fails, then
System Settings -> Privacy -> Scroll all way down -> Open Anyway -> Enter your user password

Try to open it so it fails, then
System Settings -> Privacy -> Scroll all way down -> Open Anyway -> Enter your user password

Thank you that worked, it opens now, but since it updated on launch there is no vencord.
Could you please update the discord package? Thank you very much in advance!
And i highly appreciate your fast responses! Didn't think i get an answer that fast lol

Thank you that worked, it opens now, but since it updated on launch there is no vencord.

When you open Discord did it ask you to enter your password (AFTER the prompt in system settings) in order to update Discord?
If that's the case, then you will have to log out of your user, account log in, then sudo nix store delete /nix/store/....-Discord and rebuild your system to download Discord again
To avoid all of this mess in the future, make sure to run Discord first from the terminal (by just running discord) so it runs the Python script (https://github.com/NixOS/nixpkgs/blob/7421ff3aa29d1a7da7bf0302b9e640cab064fb67/pkgs/applications/networking/instant-messengers/discord/disable-breaking-updates.py) that adds SKIP_HOST_UPDATE to settings.json (in ~/Library/Application\ Support/discord), so you don't get "tricked" into accidentally updating Discord on Nix and breaking your Vencord install

Could you please update the discord package?

There's already a PR (https://github.com/NixOS/nixpkgs/pull/355721)

If that's the case, then you will have to log out of macOS, log in, then sudo nix store delete /nix/store/....-Discord and rebuild your system to download Discord again

Ahh I see, okay, I will try that :) Just got some issue about the store still being active (even after logout)

To avoid all of this mess in the future, make sure to run Discord first from the terminal (by just running discord) so it runs the Python script (https://github.com/NixOS/nixpkgs/blob/7421ff3aa29d1a7da7bf0302b9e640cab064fb67/pkgs/applications/networking/instant-messengers/discord/disable-breaking-updates.py) that adds SKIP_HOST_UPDATE to settings.json (in ~/Library/Application\ Support/discord), so you don't get "tricked" into accidentally updating Discord on Nix and breaking your Vencord install

good to know, will do that, thanks for the info :)

So ive tried it now and it doesnt seem to work for me.
The issue is probably that i cant open discord again from the terminal after i hit "open anyway" in settings, i have to open it with the open button and then it will update

ah i see, you hit cancel, wait and then it starts but now i see this:

i guess thats related to https://github.com/NixOS/nixpkgs/issues/334141
maybe i should just use nixcord

So ive tried it now and it doesnt seem to work for me. The issue is probably that i cant open discord again from the terminal after i hit "open anyway" in settings, i have to open it with the open button and then it will update

In that case, you can first open it via System Settings, and when you are prompted again for the password in Discord to update, just cancel it, close Discord, and then launch it from Terminal.
It should start with:
[Nix] Disabling updates already done
[Vencord] Starting up...
[Vencord] Loading original Discord app.asar
...

I think the crash is related to discord updates, breaking some of Vencord's regex, it should be fix in v1.10.7

Okay, ill try again this evening because I didn't see that in the logs:
[Nix] Disabling updates already done
If that's not working, ill just use nixcord, it's more in style with nix anyway (declarative), I think, since with this, I have to have my vencloud container active, which I wouldn't need any more if I use nixcord

I feel like there is some missing part here that's causing the issue
But let's try this first:

Remove Discord entry from your nix config
Rebuild
Restart
Collect Garbage (both WITH and WITHOUT SUDO)
Add Discord again and rebuild again

Then, when you start Discord again from system settings, if Discord asks you to update, DO NOT and cancel the prompt for a password. Discord will force you to wait 10 seconds, and after that, it will restart itself and let you use it. At this point, you can shut down Discord and just start Discord from the terminal to trigger the Python script. Then you can again shut down Discord from terminal window and start it normally.

Thanks a bunch for the detailed instructions! I’ve tried everything you suggested, but I still can’t seem to make it work. It seems like the python script is not even being triggered.
I don't really know what's wrong with my config, but nixcord also won't work, tried the same steps for nixcord on two different machines. Pretty much the same issue there, when it opens, there is no vencord, the only difference is the nixcord app isnt broken

I'm not really sure how I can help you further, both nixpkgs and nixcord work fine, but I notice this:
https://github.com/ojsef39/nix-personal/blob/cc403ba411a148b8a7dad8f4632258f1262d60e0/hosts/darwin/system.nix#L33
Are those dotfiles, the ones you're using right now? If yes then you're using a completely different derivation (in this case vanilla discord) when launching Discord from the Dock...

I'm not really sure how I can help you further, both nixpkgs and nixcord work fine, but I notice this:

No problem, thank you very much tho for all your help! I appreciate it a lot :)

Are those dotfiles, the ones you're using right now? If yes then you're using a completely different derivation (in this case vanilla discord) when launching Discord from the Dock...

Yes correct, i'm aware of that, i just didn't push the changes for the right app in the dock yet (or its in dev)

@DontEatOreo GREAT NEWS :) I fixed it.
So the issue was i have to follow nixpkgs like this…
    nixcord = {
      url = "github:kaylorben/nixcord";
      inputs.nixpkgs.follows = "nixpkgs";
    };

Apparently, my home manager version and nixpkgs version are mismatched, and i didn't get an error for that until i added the follow.


chore(Landing): update rc-scroll-anim 2.0

Issue:
What I did
How to test

Deploy preview ready!
Built with commit 05c1ee0d214acabcb75569b9b13d298b16dbff4e
https://deploy-preview-465--mcs-lite-introduction.netlify.com

Deploy preview ready!
Built with commit 05c1ee0d214acabcb75569b9b13d298b16dbff4e
https://deploy-preview-465--mcslite.netlify.com


Use workspace as name for profiles created by modal token CLI

Describe your changes
A few things are happening in this PR:
First, I refactored the existing code underlying modal token new and modal token set:

I moved the underlying functions into modal/token_flow.py so the CLI code stays very thin.
_new_token now calls into _set_token after grabbing credentials from the server, to reduce duplicated code and ensure consistent console logging.
I converted the underlying functions to be async as they call into an async function for fetching the workspace name

Then, I changed the behavior of these functions to let us move away from having a "default" profile concept:

The default profile that is created / updated by these functions will be named after the workspace that the token points to, rather than being named "default"
If no profiles currently exist, the new profile will have its active metadata set to True
There's a new --activate flag for both CLI commands that will activate the profile that gets created / modified

Open questions:

Should _new_token and _set_token be public functions in the API?

One thing to consider is that the functions as written won't modify the config state in the active process, so I think we'd want to make them reload the config if we do that.

Maybe modal/token_flow.py should be modal/token.py now? This is safe to do as it does not currently contain public interfaces.
Would it make more sense to have activate=True by default?

Part of MOD-2168
Changelog
The modal token new and modal token set commands now create profiles that are more closely associated with workspaces, and they have more explicit profile activation behavior:

By default, these commands will create/update a profile named after the workspace that the token points to, rather than a profile named "default"
Both commands now have an --activate flag that will activate the profile associated with the new token
If no other profiles exist at the time of creation, the new profile will have its active metadata set to True

With these changes, we are moving away from the concept of a "default" profile. Implicit usage of the "default" profile will be deprecated in a future update.

Great cleanup! UX wise this makes a lot of sense! If we want to expose the functions in token(_flow).py (especially set_token, but maybe new as well?) we might want to refactor them to be "non-interactive" (optional console output, no input) at the same time, but that's probably not high prio at the moment.
Most important is that the CLI is intuitive and leads the user in the right direction, which I think this change does

we might want to refactor them to be "non-interactive"

Good point — I left the input() calls for set_token in the CLI layer, but we'd probably not want an API call that goes through a manual web browser input step. I do see some potential benefit for programmatic token creation / setting (e.g. there was a question from a user who is running modal in CI and wanted PRs from forks to be able to kick off jobs) but probably best to defer exposing it until we understand that use case better

When creating a new token on the website, it recommends using --profile with a different name (modal_labs instead of modal-labs).

If we're making this change, we should also update the website to either recommend --profile modal-labs, or just use modal token set without --profile (which will then check the token's workspace)

Great catch! I think that we can remove the --profile suggestion from this suggestion after this change lands.

We can't remove it immediately, since not all users will have the latest client version (older clients will apply it to the wrong profile if --profile isn't specified).
I think the website should suggest --profile=modal-labs for now, and in a few months after everyone has upgraded, we can remove --profile while keeping the behavior the same.

Good point — yeah, agreed, we should not normalize the suggested profile name here.


ADD Rosen Bridge Icons

Please take a look here @capt-nemo429 and lets merge together with #104  Thanks https://ergexplorer.com/token#8b08cdd5449a9592a9e79711d7d79249d7a03c535d17efaee83e216e80a44c4b

Seems duplicated with #104


help with scrcpy please ty

the bot is telling me scrcpy is not installed how can i fix this i think its my issue i see the folder in the bot folder and if i open console it clones my bluestacks instance but the bot is saying its not installed

WARNING:bot_logger:Starting bot...
INFO:bot_logger:scrcpy is not installed
DEBUG:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): github.com
DEBUG:requests.packages.urllib3.connectionpool:https://github.com:443 "GET /Genymobile/scrcpy/releases/download/v1.25/scrcpy-win64-v1.25.zip HTTP/1.1" 302 0
DEBUG:requests.packages.urllib3.connectionpool:Starting new HTTPS connection (1): objects.githubusercontent.com
DEBUG:requests.packages.urllib3.connectionpool:https://objects.githubusercontent.com:443 "GET /github-production-release-asset-2e65be/111583593/0a75b6d1-b575-40f8-ba8b-2123360e276a?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20230702%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230702T153822Z&X-Amz-Expires=300&X-Amz-Signature=14a9850632c4cc86bce8f2723988b13713c0a43aa8583b6b169db74b7e8257fb&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=111583593&response-content-disposition=attachment%3B%20filename%3Dscrcpy-win64-v1.25.zip&response-content-type=application%2Foctet-stream HTTP/1.1"<PHONE_NUMBER>5
INFO:bot_logger:Selected units: demon_hunter, bombardier, trapper, dryad, harlequin

well i think i jave scrcpy installed now but its notconnecting ofr some reason looks like the ports incorrext maybe how can i fix this?
C:\Users\12092\Downloads\Rush-Royale-Bot-main (1)\Rush-Royale-Bot-main>call .bot_env\Scripts\activate.bat

daemon not running; starting now at tcp:5037
daemon started successfully
Scanning <IP_ADDRESS> Ports 49500 - 55500
Started 2000 threads
failed to connect to <IP_ADDRESS>:49668
failed to connect to <IP_ADDRESS>:49671
failed to connect to <IP_ADDRESS>:49665
failed to connect to <IP_ADDRESS>:49664
Ports Open: [49665, 49668, 49671, 49664, 49666, 49670, 49667]
failed to connect to <IP_ADDRESS>:49666
disconnected <IP_ADDRESS>:49664
Exception in thread Thread-1:
Traceback (most recent call last):
File "C:\Users\12092\AppData\Local\Programs\Python\Python39\lib\threading.py", line 980, in _bootstrap_inner
disconnected <IP_ADDRESS>:49665
self.run()
File "C:\Users\12092\AppData\Local\Programs\Python\Python39\lib\threading.py", line 917, in run
self._target(*self._args, **self._kwargs)
File "C:\Users\12092\Downloads\Rush-Royale-Bot-main (1)\Rush-Royale-Bot-main\Src\gui.py", line 104, in start_bot
self.bot_instance = bot_handler.start_bot_class(self.logger)
File "C:\Users\12092\Downloads\Rush-Royale-Bot-main (1)\Rush-Royale-Bot-main\Src\bot_handler.py", line 64, in start_bot_class
bot = bot_core.Bot()
File "C:\Users\12092\Downloads\Rush-Royale-Bot-main (1)\Rush-Royale-Bot-main\Src\bot_core.py", line 27, in init
raise Exception("No device found!")
Exception: No device found!
disconnected <IP_ADDRESS>:49666
disconnected <IP_ADDRESS>:49667
failed to connect to <IP_ADDRESS>:49667
disconnected <IP_ADDRESS>:49668
disconnected <IP_ADDRESS>:49670
failed to connect to <IP_ADDRESS>:49670
disconnected <IP_ADDRESS>:49671
daemon not running; starting now at tcp:5037
daemon started successfully
Scanning <IP_ADDRESS> Ports 49500 - 55500
Started 2000 threads
error: protocol fault (couldn't read status length): No error
cannot connect to <IP_ADDRESS>:54945: No connection could be made because the target machine actively refused it. (10061)
failed to connect to <IP_ADDRESS>:49668
failed to connect to <IP_ADDRESS>:49671
failed to connect to <IP_ADDRESS>:49664
failed to connect to <IP_ADDRESS>:49666
Ports Open: [49665, 49668, 49671, 54945, 49664, 49666, 49670, 49667]
disconnected <IP_ADDRESS>:49664
Exception in thread Thread-2002:
Traceback (most recent call last):
File "C:\Users\12092\AppData\Local\Programs\Python\Python39\lib\threading.py", line 980, in _bootstrap_inner
self.run()
File "C:\Users\12092\AppData\Local\Programs\Python\Python39\lib\threading.py", line 917, in run
self._target(*self._args, **self._kwargs)
File "C:\Users\12092\Downloads\Rush-Royale-Bot-main (1)\Rush-Royale-Bot-main\Src\gui.py", line 104, in start_bot
self.bot_instance = bot_handler.start_bot_class(self.logger)
File "C:\Users\12092\Downloads\Rush-Royale-Bot-main (1)\Rush-Royale-Bot-main\Src\bot_handler.py", line 64, in start_bot_class
bot = bot_core.Bot()
File "C:\Users\12092\Downloads\Rush-Royale-Bot-main (1)\Rush-Royale-Bot-main\Src\bot_core.py", line 27, in init
raise Exception("No device found!")
Exception: No device found!
disconnected <IP_ADDRESS>:49666
disconnected <IP_ADDRESS>:49665
disconnected <IP_ADDRESS>:49667
failed to connect to <IP_ADDRESS>:49667
disconnected <IP_ADDRESS>:49668
disconnected <IP_ADDRESS>:49670
failed to connect to <IP_ADDRESS>:49670
disconnected <IP_ADDRESS>:49671


Remove class snippet comment

Provides fix for https://github.com/Microsoft/vscode/issues/21738.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for the contribution! I think we should take this over #23210 to make the snippets more consistent
The change should be in the first 1.12 insiders build once those start up again


Enum tag sizes are too large

https://github.com/zig-lang/zig/blob/53b18c8542d6d40c3aa41e9d97290a4ca46eaa15/src/analyze.cpp#L1392
currently picks how big the tag for an enum should be by using bits_needed_for_unsigned on the number of fields. bits_needed_for_unsigned(x) returns the smallest number of bits needed to hold the value x (unless x is 0, in which case it somewhat confusingly returns 0). That means if there are eg. two variants in an enum, enum { A, B }, the tag is a u2, the smallest int that can hold 2.
For an enum with 256 variants, this means it will take 2 bytes instead of 1.
Trying to change this got me errors involving @enumTagName when running the tests.

Also, fields with a type like enum {} probably shouldn't count towards the total. enum { A, B: enum {} } should be zero-sized, right?

Also, fields with a type like enum {} probably shouldn't count towards the total. enum { A, B: enum {} } should be zero-sized, right?

enum {} is the same as void, which is the same as the type of the A field in this example. I think the type of this example should be u1 - we still have to know whether it's A or B. If the u1 is 0 it's A; if the u1 is 1 it's B. Same as if the example were enum {A, B}.

I think this should be: get_smallest_unsigned_int_type(g, field_count - 1);. Let me write some more tests and get a fix going.

void is the type with one element isn't it? enum {} should have zero elements, so since enum { A, B: enum {} } can never be B, it only has one inhabitant and should be zero-sized.

There's no way to initialize an enum {} because the syntax is EnumType.Field { value } and there are no fields. But it works the same way as void, and you can initialize void like this: {}
const Foo = enum { A, B: void };
const Bar = enum { A, B};

Foo and Bar are exactly the same. Bar can be B like this: const b = Foo.B

Why is enum {} the same as void? Algebraically, I expect enum {} to be the sum type with no terms, ie. 0, while void is 1.

Hmm, I see what you're saying. I'm not sure zig is prepared to handle or should handle the algebraic concept of enum {}. We might just disallow that construct.

Actually 0 does already exists as noreturn, I guess. I think disallowing it is better than making it void.
Anyway, since enum {} isn't an empty type after all and you fixed the tag size, I'll close this 👍

A type with 0 values and a type with 1 value are kinda the same. They both contain 0 bits of information.
void is a 0-bit type. an array of length 0 is a 0-bit type. a pointer to a 0-bit type is also a 0-bit type, and an array of 0-bit elements is a 0-bit type. (here an array being like [n]T, not a slice.)
a struct with 0 members would be a 0-bit type, if it were allowed (not sure if it is or not). an enum with 0 or 1 member would be a 0-bit type. a u0 would be a 0-bit type (whose value is always 0).
Types are first class objects at comptime, so something like a pointer to void might happen as a result of passing void as a type parameter. For example, a set can be defined as a map from T to void.
However, there's currently no way to add or delete struct members or enum members with comptime code. So it makes more sense for both of those 0-member situations to be compile errors. I can't think of any reason to explicitly make a struct or enum that's completely useless.

Just looking at "number of bits of information", a singleton and an uninhabited type differ when they compose.  struct { x: i32, y: Z } has 32 bits of information when Z has one value, and 0 bits of information when Z has zero values. enum { x: i32, y: Z } has slightly more than 32 bits of information when Z has one value, but exactly 32 when it has zero.
You can use them in generic code. eg. if consumers of a generic feature must provide a type for what kind of errors they produce, a consumer can provide an empty type to signal that it never produces errors.
Unless they really make the implementation difficult, I think they should be allowed. struct {} currently works correctly so I don't think it should be changed. It's the same as void which already exists anyway.


[Tensor Operation] Unfold

Feature description
Support the unfold operation. See the PyTorch documentation for full description: https://pytorch.org/docs/stable/generated/torch.nn.Unfold.html#torch.nn.Unfold

@nathanielsimard I can try doing this, and I'm assuming you'd want to follow PyTorch's constraint of only allowing 4-dimensional input tensors? If that's the case, how would you get around the constraint of working with dimensionality of D? If I were to do this, I'd start by defining the output tensor with something like let mut output = Tensor::zeros([N, C * kernel_size[0] * kernel_size[1], H_out * W_out]);, and then basically it seems like you have to extract "patches" of the current tensor to project into the output, something like this:
let patch_tensor: Tensor<B, D, K> = self.select(n, indices);

output = output.select_assign(2, some_other_indices, patch_tensor); // will error because expects a Tensor<_, _, 3> but patch_tensor is Tensor<_, K, D>

Is there a way to ensure self.select(n, indices) returns a Tensor<_, _, 3> or to cast/convert it appropriately?

@agelas I'm unsure if we can implement this operations efficiently using existing tensor operations. It is similar to a convolution or a pooling operation, which are all implemented by backends with custom kernels.
Maybe we can implement this efficiently using conv2d with a custom weight matrix of ones and zeros, but probably not a select_assign or anything index related.


Add gw support to zsh completion

I use gdub, so this adds gw to the zsh compdef. Unsure how to do that safely with your bash method, maybe a conditional on the command being defined?
Looks great though. Have this running now, I've added to my dotfiles - https://github.com/DanielThomas/dotfiles/commit/336c5f7800f89bd49a4c1fc7902b683ee32533c6!

Thanks @DanielThomas! Merged.


Add a $PSAssignedParameters automatic variable

Summary of the new feature / enhancement
There is currently a $PSBoundParameters automatic variable which includes any parameters/values passed to the command. But this variable doesn't include parameters which have default values. There are times when users want any assigned parameter key/values passed to another function--including any default values if the user didn't pass a value to that parameter. Basically any parameter variables which hold a value, whether that value was provided by the user or a default value.
I'd like to propose a $PSAssignedParameters automatic variable which includes any parameters with assigned values, whether they be bound parameters passed by the user, or parameters not passed by the user, but with default values.
Current functionality:
PS > function Get-BoundParameters {
    param (
        $Param1 = "This is param1",
        $Param2,
        $Param3
    )
    [PSCustomObject] $PSBoundParameters
}
PS > Get-BoundParameters -Param2 "This is param2"

Param2
------
This is param2

Currently, only $Param2 is returned, whereas the desired result (for the proposed $PSAssignedParameters variable) is that both $Param1 and $Param2 are returned, since both variables have assigned values--$Param1 from the default value, and $Param2 from the value passed by the user.
Proposed technical implementation details (optional)
Desired functionality:
PS > function Get-AssignedParameters {
    param (
        $Param1 = "This is param1",
        $Param2,
        $Param3
    )
    [PSCustomObject] $PSAssignedParameters
}
PS > Get-AssignedParameters -Param2 "This is param2"

Param1         Param2
------         ------
This is param1 This is param2

The $PSAssignedParameters automatic variable would hold any parameters which have assigned values, whether that value was assigned by the user during runtime, or it's a default value. In this example, it returns $Param1 with the default value and $Param2 with the user-provided value, but doesn't return $Param3 because it doesn't have a value at all.
Possible challenges / considerations

How to approach switch/boolean parameters. If the switch/boolean is unused, should it be included in the $PSAssignedParameters hash with a $false value, or should it not be included at all?
How to handle dynamic parameters

@DevOpsJeremy thanks for submitting this!
@SteveL-MSFT this is something @Jaykul and I have discussed many times as being desired and valuable.
It greatly streamlines the process of splatting down to a command.

Good point, but your issue is a duplicate of this issue (from 2017, alas):

#3285

And what could be practical use? If the parameter has a default value then this parameter is already assigned and can be used without checking whether it is present or not. What is the logic in this?

@237dmitry if a number of parameter key/values are being passed to another cmdlet via splatting, the default values need to be manually added to the follow-on cmdlet since they aren't included in $PSDefaultParameters. With $PSAssignedParameters (or a similar functionality), any parameters with values would be included without additional code needed for handling default values.

Closing due to duplicate of #3285


Netherside world not load

Hello I am kilerv69 from aternos server I have been having a major issue as when I add all examples and things to netherside and then restart my server so the nether side world doesn't load I mean when I creates it it works and when I restart my server it never loads and delete whole plugin and did the same thing again but still not works also 34 warning from it which I can't view as I am bedrock but I'm sure it says that the newly added biome doesn't have the replacements in netherside .pls fix the issues :(  I'm on 1.19.1

I think you will just need to turn the netherside example off. Sorry!

Turned it off for always ???
Will it load world then also

It is resource intensive. If your server can’t handle it, you would need to turn it off and can’t use it.
You could try using a plugin like worldborder to pre generate, but if it won’t load in the first place there is nothing I can do.


Basic: make dirname work better on Windows

Use the Path APIs to compute the parent directory path on Windows.

@swift-ci test

@compnerd should we continue to work on this in light of recent discussions?

@tomerd - change of heart.  I have collapsed all the necessary path changes to get a bootstrapped version going into #114.  This should be considered obsoleted.


Make things unitless, not pint-dimensionless

@whoburg, ready for review

test models please

@whoburg, merging after 9 days, feel free to review!


Add Method/UI for Quitting From Within App

Feature Description
I'd like to be able to quit Franz when the app is open without having to interface with the system tray to quit. This could be either (or both) adding a keyboard shortcut (like Ctrl/Command + Q) or an option in the Alt menu that appears.
Motivation and Context

This streamlines the quitting process instead of having to find the system tray. Some OS's will hide the tray icon, some OS's need extra support for the tray icon to appear. It just seems like an intuitive feature that would be expected in any application. Even further, if users have multiple screens, the tray may only be located on one monitor (in my case, my system tray is about 4ft away from where the application is running).

Yeah agree. Not even Alt+F4 works on Windows.


fix: wrong path in PathSupport when param has http scheme

fix #1682

this is what i need

Thanks a lot for the PR!


Update Add an Org to a Channel tutorial

A few minor updates:

Indicate that NodeOU admin is now utilized rather than admincerts.
Indicate that leader election is not the default starting in v2.2.
Fix the endorsement to use Org2 and the new Org3 peer (2 out of 3 required).

Signed-off-by: David Enyeart<EMAIL_ADDRESS>

@mergifyio backport release-2.2


late listener registration causes WARN message

Since v2.0.1: There is an error in the listener initialization causing problems with the page tooling API.

https://github.com/LaxarJS/ax-developer-tools-widget/blob/master/ax-developer-tools-widget.js#L114
must be moved up.

Fixed on master (v2.1.0) and on release-2.0.x (v2.0.3).


pop_farm_fetch.py code

Good use of docopt but a test would be good. Excellent EDA code and nice figures. This may be personal preference, but a line of code like this:
pop = pd.read_csv('data/raw/API_SP.POP.TOTL_DS2_en_csv_v2_1678576.csv', skiprows=4).drop(     columns=['Country Code', 'Indicator Name', 'Indicator Code', '2020', 'Unnamed: 65']).melt(id_vars=['Country Name'],                                                                                var_name='Year',                                                                                value_name='Population')
can be hard to understand. Perhaps splittign the read_csv, drop, and melt would improve readability. And also assigning lists like ['Country Code', 'Indicator Name', 'Indicator Code', '2020', 'Unnamed: 65'] prior woul improve readability.

Thank you, Arjun! Really helpful comment for the future EDAs.


Add CircleCI config to push to GR charts repository

I wanted to follow up on #360 and set up CircleCI to push new releases to https://github.com/G-Research/charts.
I did a few things here:

Added commands for:

installing helm
checking out the charts repo
updating the charts and pushing the packaged charts  back to the charts repo

Put those new commands in the release job
Changed the index.yaml, Chart.yaml and values.yaml files to use 'LATEST' for version/appVersion to we can replace the value when we run the release.

I've run my steps in the local CircleCI environment and also in a test job in the proper CircleCI environment, so unless I've made some bonehead mistake moving my code into Armada's .circleci/config, it should all run fine.
The git clone and push commands rely on a deploy key from the charts repository.  I added the key as the ARMADA_CHART_UPDATE_KEY environment variable to the armada project in CircleCI so that it will run happily.  There was a problem with CircleCI eating line breaks in the key and I can't tell if I've put it in the right way or not.  If it breaks, I'll just have to change it to the version with \n's.
Also, just register my feelings about doing a replace on 'LATEST': I'd really prefer to check the updated code back in somewhere.  It feels wrong that someone couldn't just grab the code for a specific version and build it immediately.  I understand the reticence to have a CI job check code in, but I'd like to continue the conversation and see if there isn't a way that feels ok for everyone.
Would love any feedback you have for this.

Thanks, this looks great.
For the replace functionality, yes it unfortunate that you can't just clone the repo, we could maybe publish "latest" tag for all images so you could clone it and use latest? Other option would be to completely rework the release process. Currently it is initiated by tagging the repository, that effectively prevents you from doing any changes to the repo. You could reverse it and initiate the release from manual circle ci job, in that case I would not be oposed to commiting changes and tagging from the circleci job.

Actually one more thing, could you please change quick start accordingly? I think this change might break it.

we could maybe publish "latest" tag for all images

Nah, at least this way the charts that we publish will refer to a specific version.  It's halfway to what I'd like.  If we only ever point these charts at "latest" it will be a mess to go backwards and use an old chart, or maybe someone continues to use an old chart but starts pulling in a newer "latest" image... eh, it's not the way we want to go.

Other option would be to completely rework the release process.

Eventually, this is what I'd like to look at doing.  Do we have any allegiance to CircleCI?  I feel like Github Actions might be a more contained way of doing CI these days, but I don't know if we can do the whole multi-cluster setup like we need with GH Actions.  @jgiannuzzi would know.  If we decide to rework the process, I like your idea of "commiting changes and tagging from the [circleci/gh actions] job".

Quickstart needs an update before merging this.

Totally -- my whole goal with this change was to get back to simplifying the Quickstart guide, so I'm glad you asked!  Hopefully, this allows us to make it even simpler for new users.  I'll try and get to it tonight, or over the weekend at the worst.

I added a couple of commits that I believe bring the Quickstart guide up to speed with the changes in this PR.
I ran through the steps and everything seemed to work.  The only thing I had to do beyond just copying and pasting the steps in the quickstart was --set image.tag=v.0.1.9 when helm installing the server and the executor.  I've created a PR to add the v0.1.9 charts to the charts repo so that we can test without needing to add this command line parameter.  Once someone +1's that, the changes in this PR should be really straightforward to test.
Other thoughts:
That chunk of bash to download and install armadactl... I kinda hate to suggest it, but should we be looking at homebrew and chocolatey packages to make that solid chunk of ugliness go away?
Also, I could maybe simplify the helm install steps a little by making the redis, prometheus and executor-monitoring charts all dependencies for the server and executor charts.  However, because a lot of the parameters in the values files for the server and the executor have the same names, I can't just simply -f docs/quickstart/*.yaml.  The alternative is having a bunch of -f's for the helm install command or merging some of the separate values files into just a server and executor values file.  I'll muse on it a bit more, but any input is welcome.

I am all for creating packages for armadactl, we just have not prioritized it yet.
For helm, we are at a point where complete refactor of values is annoying but manageable, so if there is a need to nest all server and executor value under some key, lets do it now. Is there a way to have optional chart dependencies? Most production deployments will probably not want one chart deploying everything.


US5: Country Selection: At the start of each round, I want the game to randomly select a country for players to guess

Acceptance Criteria

[x] #57
[x] #58

all tasks are done


Easy measurements on schedules

What is the expected behavior?
It would be nice not to have to build a measurement schedule up every time. It's usually the same thing every time: measure some qubits, or measure all qubits. This requires a backend with pulse enabled.
How it is currently done
Measuring only a couple qubits, building the pulses
sched  = Schedule()
# ... add operations to sched
meas_pulse = pulse_lib.gaussian_square(duration=meas_samples,
                                       sigma=meas_sigma,
                                       amp=meas_amp,
                                       risefall=meas_risefall)
acq_cmd = pulse.Acquire(duration=meas_samples)
measure_schedule = meas_pulse(MeasureChannel(0))
                   + acq_cmd([AcquireChannel(i) for i in range(n_qubits)],
                             [MemorySlot(i) for i in range(nqubits)])
sched |= measure_schedule << schedule.duration

or measuring all of them, with info from backend defaults
sched = Schedule()
# ... add ops
sched_duration = schedule.duration

cmd_def = backend.defaults().circuit_instruction_map  # about to be renamed
meas_map = backend.configuration().meas_map
for i in range len(meas_map):
    sched |= cmd_def.get('measure', meas_map[i]) << schedule_duration
# in practice, meas_map is usually one element and just list(range(n_qubits))

API ideas
sched = sched.measure_all(backend)

and
schedule = schedule.measure(qubits_list, backend)

Second option:
sched = measure(schedule, backend, qubits)

Whatever API, the method should do something like what the scheduler does: take the default measure command, and remove the extra measurement pulses. Unlike the scheduler, this method can just do a trivial mapping from qubit index to MemorySlot (at least for this issue).

I do not think this should be a part of the Schedule as this is a utility method making assumptions such as the mapping to channels. I would prefer to keep the Schedule class as close as possible to a data structure for a pulse program and am open to providing a builder interface on top of it.

@taalexander okay, sounds good. This suggestion was for keeping the interface close to circuit.measure but I also didn't like depending on the backend. Also, looks like I've opened a duplicate issue, which you and @eggerdj have also made versions of! Good thing Soolu is working on it!

duplicates #3480  and #3572


Unable to debug C# Functions in VS2017 Preview after clean install

Description
After having to repave my machine, I tried to re-install my Azure Functions tooling. The order of the install was:

VS2017 Preview (15.3)
Node.js/NPM
Azure Functions Extension for VS2017 Preview
Azure Functions Core Tools (via NPM)

Tried to debug a C# Function but I got the following error:

I then installed the Azure Function CLI (old version) which failed to install due to bandwidth issues. To fix this I download it manually following the steps outlined here: https://github.com/Azure/Azure-Functions/issues/110.
I unzipped the files in the following location:
C:\Users\<userprofile>\AppData\Local\Azure.Functions.Cli\1.0.0-beta.9
This made VS2017 to stop complaining, but the Function failed to run since there was a mismatch between the CLI tools and the latest Functions project format (WebJobs based).
###Workaround
To make it work, I went in my NPM package folder and copied the executable and accompanying dlls to the directory that VS2017 expects to find them. To make it more explicit, I copied from here:
C:\Users<userprofile>\AppData\Roaming\npm\node_modules\azure-functions-core-tools\bin
to
C:\Users<userprofile>\AppData\Local\Azure.Functions.Cli\1.0.0-beta.9
With this workaround, I was able to run and debug the latest Functions code in VS2017
Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here:
Azure Functions Core tools ->NPM
Azure CLI -> manually
CLI Version: What version of the CLI and modules are installed? (Use az --version)
Answer here:
Azure Functions Core Tools (1.0.0-beta.99)
Function Runtime Version: 1.0.11002.0
OS Version: What OS and version are you using?
Answer here: Windows 10 Creator's update

When you run az --version what output do you get?

 Apologies for the confusion but this belongs to the Azure Functions CLI! arg, too many CLIs


multi-class visualize problem

Hi, my dataset have 13classes include background
and model.predict shape is [1,384,480,13]
how can i visualize it on one image?

Traceback (most recent call last):
File "C:/Users/SDR/Desktop/segmentation_models-master/train.py", line 427, in
pr_mask=pr_mask.squeeze(),
File "C:/Users/SDR/Desktop/segmentation_models-master/train.py", line 32, in visualize
plt.imshow(image)
File "C:\Anaconda3\envs\s_m\lib\site-packages\matplotlib\pyplot.py", line 2684, in imshow
None else {}), **kwargs)
File "C:\Anaconda3\envs\s_m\lib\site-packages\matplotlib_init_.py", line 1599, in inner
return func(ax, *map(sanitize_sequence, args), **kwargs)
File "C:\Anaconda3\envs\s_m\lib\site-packages\matplotlib\cbook\deprecation.py", line 369, in wrapper
return func(*args, **kwargs)
File "C:\Anaconda3\envs\s_m\lib\site-packages\matplotlib\cbook\deprecation.py", line 369, in wrapper
return func(*args, **kwargs)
File "C:\Anaconda3\envs\s_m\lib\site-packages\matplotlib\axes_axes.py", line 5679, in imshow
im.set_data(X)
File "C:\Anaconda3\envs\s_m\lib\site-packages\matplotlib\image.py", line 690, in set_data
.format(self._A.shape))
TypeError: Invalid shape (384, 480, 13) for image data

Matplotlib can't handle 13 channels image. You need to assign color to each class and then paint them on one RGB image.
You can try something like this:
colors = [(255,0,0),(0,255,0)] # etc, add 13 colors
out_image = np.zeros_like(original_rgb_image)
for i in range(classes):
    class_mask = image[...,i] # get the layer of class
    class_mask = np.stack((class_mask * colors[i][0], class_mask * colors[i][1],class_mask * colors[i][2]), axis=-1)
    # you can simply add values but I don't use this approach with multiple classes
    out_image = out_image + class_mask
    # so if the above doesn't work you can use opencv addWeighted
    out_image = cv2.addWeighted(out_image ,1.0,class_mask,0.7,0)
# now you can plt.imshow(out_image)

Also you can place copy of original image into out_image and get classes painted over original

Another option is to overlay the entire mask on the class by changing the alpha:
# turns the 13-channel mask into a 1-channel image
mask = np.argmax(y_pred, axis = 2)

plt.imshow(image)
plt.imshow(mask, alpha = .65)
plt.savefig(arr = mask, fname = "results.png")
plt.show()

or like @VladislavAD, is you only want to display a single class over the image:
plt.imshow(image)
plt.imshow(y_pred[:, :, class_index], alpha = .65)
plt.show()

thanks :)


CryptoTests_0 FAILED with unmappable character for US-ASCII in Test_openjdk11_hs_extended.functional_aarch64_linux

Failing with unmappable character (0xC4) for encoding US-ASCII
Test Info
Test Name: CryptoTests_0
Test Duration: 2 min 14 sec
Machine: test-docker-ubuntu2004-armv8l-2
TRSS link for the test output: https://trss.adoptium.net/output/test?id=63e85be6ed0b30773bfcc749
Build Info
Build Name: Test_openjdk11_hs_extended.functional_aarch64_linux
Jenkins Build start time: Feb 11 2023, 12:30 pm
Jenkins Build URL: https://ci.adoptopenjdk.net/job/Test_openjdk11_hs_extended.functional_aarch64_linux/377/
TRSS link for the build: https://trss.adoptium.net/allTestsInfo?buildId=63e85aeeed0b30773bfcb8ba
Java Version
openjdk version "11.0.18" 2023-01-17
OpenJDK Runtime Environment Temurin-11.0.19+1 (build 11.0.18+1)
OpenJDK 64-Bit Server VM Temurin-11.0.19+1 (build 11.0.18+1, mixed mode)
Rerun in Grinder
[2023-02-11T20:52:53.435Z] STDOUT:
[2023-02-11T20:52:53.435Z] STDERR:
[2023-02-11T20:52:53.436Z] /home/jenkins/workspace/Test_openjdk11_hs_extended.functional_aarch64_linux/aqa-tests/functional/security/Crypto/CryptoTest/cryptotest/tests/TerminalFactoryTests.java:54: error: unmappable character (0xC4) for encoding US-ASCII
[2023-02-11T20:52:53.436Z]  * @author Zden??k ??ambersk??, Petra Mikova
[2023-02-11T20:52:53.436Z]                ^
[2023-02-11T20:52:53.436Z] /home/jenkins/workspace/Test_openjdk11_hs_extended.functional_aarch64_linux/aqa-tests/functional/security/Crypto/CryptoTest/cryptotest/tests/TerminalFactoryTests.java:54: error: unmappable character (0x9B) for encoding US-ASCII
[2023-02-11T20:52:53.436Z]  * @author Zden??k ??ambersk??, Petra Mikova
[2023-02-11T20:52:53.436Z]                 ^
[2023-02-11T20:52:53.436Z] /home/jenkins/workspace/Test_openjdk11_hs_extended.functional_aarch64_linux/aqa-tests/functional/security/Crypto/CryptoTest/cryptotest/tests/TerminalFactoryTests.java:54: error: unmappable character (0xC5) for encoding US-ASCII
[2023-02-11T20:52:53.436Z]  * @author Zden??k ??ambersk??, Petra Mikova
[2023-02-11T20:52:53.436Z]                    ^
[2023-02-11T20:52:53.436Z] /home/jenkins/workspace/Test_openjdk11_hs_extended.functional_aarch64_linux/aqa-tests/functional/security/Crypto/CryptoTest/cryptotest/tests/TerminalFactoryTests.java:54: error: unmappable character (0xBD) for encoding US-ASCII
[2023-02-11T20:52:53.436Z]  * @author Zden??k ??ambersk??, Petra Mikova
[2023-02-11T20:52:53.436Z]                     ^
[2023-02-11T20:52:53.436Z] /home/jenkins/workspace/Test_openjdk11_hs_extended.functional_aarch64_linux/aqa-tests/functional/security/Crypto/CryptoTest/cryptotest/tests/TerminalFactoryTests.java:54: error: unmappable character (0xC3) for encoding US-ASCII
[2023-02-11T20:52:53.436Z]  * @author Zden??k ??ambersk??, Petra Mikova
[2023-02-11T20:52:53.436Z]                             ^
[2023-02-11T20:52:53.436Z] /home/jenkins/workspace/Test_openjdk11_hs_extended.functional_aarch64_linux/aqa-tests/functional/security/Crypto/CryptoTest/cryptotest/tests/TerminalFactoryTests.java:54: error: unmappable character (0xBD) for encoding US-ASCII
[2023-02-11T20:52:53.436Z]  * @author Zden??k ??ambersk??, Petra Mikova
[2023-02-11T20:52:53.436Z]                              ^
[2023-02-11T20:52:53.436Z] 6 errors
[2023-02-11T20:52:53.436Z]
[2023-02-11T20:52:53.436Z] TEST RESULT: Failed. Compilation failed: Compilation failed
[2023-02-11T20:52:53.436Z] --------------------------------------------------
[2023-02-11T20:52:53.436Z] Test results: passed: 28; failed: 3
[2023-02-11T20:52:53.436Z] Report written to /home/jenkins/workspace/Test_openjdk11_hs_extended.functional_aarch64_linux/aqa-tests/TKG/output_1676148636612/CryptoTests_0/report/html/report.html
[2023-02-11T20:52:53.436Z] Results written to /home/jenkins/workspace/Test_openjdk11_hs_extended.functional_aarch64_linux/aqa-tests/TKG/output_1676148636612/CryptoTests_0/work
[2023-02-11T20:52:53.436Z] Error: Some tests failed or other problems occurred.
[2023-02-11T20:52:53.436Z] -----------------------------------
[2023-02-11T20:52:53.436Z] CryptoTests_0_FAILED
[2023-02-11T20:52:53.436Z] -----------------------------------

The same test passed in openj9: https://openj9-jenkins.osuosl.org/job/Test_openjdk11_j9_extended.functional_aarch64_linux_Nightly/359/

Wonder what is the default encoding on machines (fails with US-ASCII)
Also seen on x86-64_linux
https://ci.adoptopenjdk.net/job/Test_openjdk11_hs_extended.functional_x86-64_linux/351/

Initial grinder on x86-64_linux (test-docker-ubi8-x64-1) passed https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/6619/ . The failed one is on test-docker-debian11-x64-2. Feels like there are setting differences between same platforms.

Seems like this issue may be avoidable, no matter what machine locale settings are used by either removing or escaping the characters that cause the compilation issue, similar to the advised handling of non-standard ASCII characters in jtr files.
See https://openjdk.org/jtreg/faq.html 3.17 Why are there extra \ characters in the output from a test in a .jtr file?
By design, the contents of a .jtr file, including any output from tests, is represented in a way that can be read back in again by jtreg and related tools. To that end, characters that are not standard ASCII characters (printable characters, and CR, LF, SP, HT) are encoded with escape sequences. Characters outside that set are represented by \uXXXX, and \ itself as written as \\. Anyone viewing the contents of a .jtr directly, such as in a plain-text editor, or using command-line tools like grep need to be aware of that encoding and take it into account.

FYI @zzambers - you have a non-standard name according to jtreg ;) - would you be able to update the test material?

The fix was merged into CryptoTest; can this issue be closed?


Test Orthographic Map Projection

https://en.m.wikipedia.org/wiki/Orthographic_map_projection#Mathematics

Closed in favor of https://github.com/rsadsb/adsb_deku/pull/74


Costumize the Readme title to Chimoney Community Projects

after the October period

and before the October month

Please assign me this one.


fix(default theme)/fix link in images block in cms component #432

Changes
closes #432
Acceptance criteria

[x] use SfLink component from Storefront-ui if we need to display a link
[x] click works internally with vue router if it's local link (should work natively in SfLink)
[x] external links are used, then <a> should be used (should work natively in SfLInk)
[x] open tab/not open tab should be considered and adjusted
[x] no link component if CMS content doesn't have it

getCmsLink helper added
Checklist

[x] I followed contributing guidelines

Link in image works fine but after adding links to the pictures in 'Three columns, rounded' block - the pictures are displayed incorrectly, as below:

fixed on all implemented images blocks.

I merged that with #931


Is this production ready?

Hi, We desperately need a solution that can detect root on android devices. Is this solution production ready?

Hi,
Can i know who you are?
On Mon, Feb 19, 2024, 11:06 AM Sunny @.***> wrote:

Hi, We desperately need a solution that can detect root on android
devices. Is this solution production ready?
—
Reply to this email directly, view it on GitHub
https://github.com/udithperera-dev/oneroot/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWCJPR3DL2O5D3BCM3M5FXDYULQD7AVCNFSM6AAAAABDO2PNGGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE2DCNJUG4YDONQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

App developer bro

yes, ios one still under development , but will rehearse soon
On Mon, Feb 19, 2024, 1:08 PM Sunny @.***> wrote:

App developer bro
—
Reply to this email directly, view it on GitHub
https://github.com/udithperera-dev/oneroot/issues/1#issuecomment-1951863671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWCJPRZQS5FK6TPHIIREDY3YUL6Q3AVCNFSM6AAAAABDO2PNGGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJRHA3DGNRXGE
.
You are receiving this because you commented.Message ID:
@.***>

For now I just needed to test the solution for android.
The solutions which already exist had 'false positive' issue.

i didnt understand
On Mon, Feb 19, 2024, 1:21 PM Sunny @.***> wrote:

For now I just needed to test the solution for android.
The solutions which already exist had 'false positive' issue.
—
Reply to this email directly, view it on GitHub
https://github.com/udithperera-dev/oneroot/issues/1#issuecomment-1951880548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWCJPR5F6JS7ACVQHE3FSY3YUMAA7AVCNFSM6AAAAABDO2PNGGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJRHA4DANJUHA
.
You are receiving this because you commented.Message ID:
@.***>

I mean the existing solution is not accurate with detection. Many non-rooted devices are detected as rooted which causes problems.

a wait. within 2 days ill update that. give me that time period
Thankz for information
On Mon, Feb 19, 2024, 2:29 PM Sunny @.***> wrote:

I mean the existing solution is not accurate with detection. Many
non-rooted devices are detected as rooted which causes problems.
—
Reply to this email directly, view it on GitHub
https://github.com/udithperera-dev/oneroot/issues/1#issuecomment-1951983380,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWCJPRZESETYGAOHFCIV36DYUMH7ZAVCNFSM6AAAAABDO2PNGGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJRHE4DGMZYGA
.
You are receiving this because you commented.Message ID:
@.***>

fix added for wrong root detection issue for below android 9


Do not create Index-Operations with the same name as ForeignKey constraints

This should fix Issue 113
In contrast to SQL Standard, MS Access will create an index together with a the FK constrains. No Operation for those indexes are generated anymore.
According to https://docs.microsoft.com/en-us/office/client-developer/access/desktop-database-reference/constraint-clause-microsoft-access-sql
this can be deactivated, however creating an index with the same name as an FK still results in an runtime error, therefor the index creation operation is skipped

[ ] A problem might be, that the generated model will not produce an index for another DBMS like SQL Server
[ ] I had run the existing tests locally and got a lot of errors, mostly due to missing drivers and infrastructure. Please verify that there are no unintended sideeffects

@MichaelSteinecke Thank you for your contribution!


Add content/de/docs/concepts/workloads/pods/_index.md

Added German translation for pod overview page

/cc @mkorbi as German reviewer ;)

Tim, thx for spotting some mistakes. I fixed them or tried to match the formatting of the original English text, for example : {{< /note >}}.
I also clicked on all the links and they seem to work.

Thx "mkorbi" for the review comments. For now I've settled on "Pod Vorlage".
In the last section "Nächste Schritte" I'm not 100% happy about translating "Learn about" to "Lerne...". "Lerne" just sounds a bit too commanding. I thought about "Mehr erfahren zum Thema ..." or simply "Mehr zum Thema...". Any thoughts?

thanks for the updates!
Yes I agree, think you can be creative:

Mehr zum Thema
Erfahre mehr
Verstehe den Lebenszyklus eines Pods
Finde mehr zu XZY heraus

/lgtm
/approve


Store CABL rules configuration in openedge-project.json

Until the connected mode can be used, it should be possible to have a local configuration file for the CABL rules. Probably in openedge-project.json, but could be another file.
Configuration is currently hard-coded in the language-server.

Done


Expose ZEPHYR_TOOLCHAIN_VARIANT to Kconfig to support openisa_rv32m1

Part of my reasoning for this is this will allow us to build test / regress the vega board and openisa_rv32m1 soc/drivers with the standard Zephyr toolchain (once 0.10.0) is official.

@SebastianBoe can you take a look

set(ENV{ZEPHYR_TOOLCHAIN_VARIANT} ${ZEPHYR_TOOLCHAIN_VARIANT})

I think its not needed since its already coming from the env.

Aha, then it is pretty close.
But it is valid (and has precedence) to set ZEPHYR_TOOLCHAIN_VARIANT as cmake -DZEPHYR_TOOLCHAIN_VARIANT=gnuarmemb.


Restruncure JSON files and solution files

Restructure solution folder so that unnecessary files is not seen
Added sorting in JSON files so that configs can easily be found

LGTM!


Crash On Login

As the title suggests, the server crashes on client log in.
SpongeForge: spongeforge-1.10.2-2281-5.2.0-BETA-2546.jar (This also happens with the latest build, spongeforge-1.10.2-2281-5.2.0-BETA-2571)
Java Version: 1.8.0_141
OS: Linux
Console Log: rawlog.txt
Crash Report: crash-2017-08-18_13.52.34-server.txt

@SjsolisZXA: Are you able to reproduce this consistently with a fresh world?

No, it seems like it's a problem with the world I had on my server, my apologies. Do you have any idea what the log is saying about the world?


v1.16 Backports 2024-10-22

[ ] #35179 (@wedaly)
[ ] #35337 (@julianwiedmann)
[ ] #35165 (@julianwiedmann)
[ ] #35400 (@sayboras)
[ ] #35399 (@aanm)
[ ] #35408 (@aanm)
[ ] #35381 (@jrajahalme)
[ ] #35333 (@rastislavs)
[ ] #35422 (@marseel) :warning: resolved conflicts

added any: prefix as with other labels because commit 1918908a174d ("Improve identity-relevant-labels.rst page") which removed these prefixes across all docs wasn't backported to v1.16.

[ ] #35457 (@aanm)

PRs skipped due to conflicts:

#33972 (@darox)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 35179 35337 35165 35400 35399 35408 35381 35333 35422 35457

/test-backport-1.16

lgtm, nonblocking: if you happen to update commits for whatever reason, now my commit points to incorrect commit, while PR description points to correct one: 1918908

Oops, sorry. I updated it locally but forgot to force-push apparently.

/test-backport-1.16


Passing data between GD.yield() and resume()

Discussed in https://github.com/gilzoide/godot-lua-pluginscript/discussions/13

Originally posted by Alikae January 27, 2022
Hi !
I would like to know if there is currently a way to pass data between between GD.yield() and the corresponding resume().
All my tests so far behave as if GD.yield() always return nil, regardless of how I call resume().
Wanted usage:
script a:

b.test()
b.resume('data')

script b:

function test()
a = GD.yield()
print(a) -- print data

Is there something I missed about how to use it ?
Is this maybe not an intended feature ? The wanted behavior is easily reproduced by changing

b.resume('data')

by

b.some_prop = 'data'
b.resume()

Lastly, I came to the sources to try to understand it by myself, but I stumbled upon coroutine.resume and coroutine.yield, that (in my understanding) doesn't seems to be defined anywhere. Does it come from Godot API ? If so, do you know where I can find some documentation about it ? Pretty hard to find Godot documentation that do more than scratching the surface.

This is most likely because of the lack of a return at the end of GD.yield: https://github.com/gilzoide/godot-lua-pluginscript/blob/main/src/late_globals.lua#L132


Roll Chromium 49.0.2623.110

This has the latest M49 stable from Chromium.
As usual, big thanks to everyone who has contributed to this rebase:
Lin Sun, Chuan Liu, Andrey Malyshev, Xiaosong Wei, Xu Xing, Hengzhi Wu,
Shao Boyan, Kenny Yuan and most importantly Olli Raula who did most of
the work which led to this milestone and last but not least
Raphael Kubo Da Costa for his relentless support and attention to detail
throughout past 3 months to get this release out the door.
Things to note:

The Presentation API implementation might still need some adjustments,
as tracked in XWALK-6262 and XWALK-6493.
Multidex configuration has been turned off temporarily on Android until
we figure out the build issue which can be tracked here, XWALK-6625.

For other Chromium features refer to it's release notes here,
http://blog.chromium.org/
Relevant upstream CLs we had to adapt to:
https://codereview.chromium.org/1540423004
https://codereview.chromium.org/1523133002
https://codereview.chromium.org/1533073005
https://codereview.chromium.org/1459473003
https://codereview.chromium.org/1569273002
https://codereview.chromium.org/1578953005
https://codereview.chromium.org/1467193003
https://codereview.chromium.org/1500993002
https://codereview.chromium.org/1341923002
https://codereview.chromium.org/1467603002
https://codereview.chromium.org/1475773002
https://codereview.chromium.org/1533073005
https://codereview.chromium.org/1497633002
https://codereview.chromium.org/1430413003
https://codereview.chromium.org/1466573002
https://codereview.chromium.org/1507743005
https://codereview.chromium.org/1474603004
https://codereview.chromium.org/1491063002
https://codereview.chromium.org/1491063002
https://codereview.chromium.org/1304373007
https://codereview.chromium.org/1806853005
https://codereview.chromium.org/1550063002
BUG=XWALK-5805

This is #3628 rebased on top of the latest master. I can't push to Mrunal's remote, so I had to open a new PR.

er, Mrunal didn't push his coding style fixes...

victory at last

rs=me


Upgrade to v2.10.4 pulumi-terraform-bridge to fix a regression on pulumi config set

Fixes: #720

v3.24.1 has been released


tests-ci : kernel: timer: tickless test_sleep_abs Failed

**Describe the bug**
tickless test is Failed on v3.1.0-rc1 on mimxrt1170_evk_cm7

see logs for details

**To Reproduce**
1.
```
scripts/twister --device-testing --device-serial /dev/ttyACM0 -p mimxrt1170_evk_cm7      --sub-test kernel.timer
```
2. See error

**Expected behavior**
test pass

**Impact**

**Logs and console output**
```
START - test_sleep_abs

Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/timer/timer_api/src/main.c:809: test_sleep_abs: (late >= 0 && late < k_us_to_ticks_ceil32(250) is false)
expected wakeup at 14481, got 14483 (late 2)
FAIL - test_sleep_abs in 0.32 seconds
```

**Environment (please complete the following information):**
 - OS: (e.g. Linux )
 - Toolchain (e.g Zephyr SDK)
 - Commit SHA or Version used: v3.1.0-rc1

@dleach02 issue found on v3.1.0-rc1

Unable to reproduce the failure on tag  v3.1.0-rc1. Below is the test result for test tests\kernel\timer\timer_api
*** Booting Zephyr OS build v3.1.0-rc1  ***
Running TESTSUITE timer_api
===================================================================
START - test_time_conversions
 PASS - test_time_conversions in 0.11 seconds
===================================================================
START - test_timer_duration_period
 PASS - test_timer_duration_period in 0.326 seconds
===================================================================
START - test_timer_restart
 PASS - test_timer_restart in 0.276 seconds
===================================================================
START - test_timer_period_0
 PASS - test_timer_period_0 in 0.202 seconds
===================================================================
START - test_timer_period_k_forever
 PASS - test_timer_period_k_forever in 0.202 seconds
===================================================================
START - test_timer_expirefn_null
 PASS - test_timer_expirefn_null in 0.326 seconds
===================================================================
START - test_timer_periodicity
 PASS - test_timer_periodicity in 0.252 seconds
===================================================================
START - test_timer_status_get
 PASS - test_timer_status_get in 0.1 seconds
===================================================================
START - test_timer_status_get_anytime
 PASS - test_timer_status_get_anytime in 0.326 seconds
===================================================================
START - test_timer_status_sync
 PASS - test_timer_status_sync in 0.352 seconds
===================================================================
START - test_timer_k_define
 PASS - test_timer_k_define in 0.702 seconds
===================================================================
START - test_timer_user_data
 PASS - test_timer_user_data in 0.301 seconds
===================================================================
START - test_timer_remaining
 PASS - test_timer_remaining in 0.51 seconds
===================================================================
START - test_timeout_abs
 PASS - test_timeout_abs in 0.2 seconds
===================================================================
START - test_sleep_abs
 PASS - test_sleep_abs in 0.13 seconds
===================================================================
TESTSUITE timer_api succeeded
===================================================================
PROJECT EXECUTION SUCCESSFUL

@mmahadevan108 , thanks for the check, just want to confirm do you build with below command line?
west build -b mimxrt1170_evk_cm7 -- -DCONF_FILE="prj_tickless.conf"

Thank you. I am able to reproduce the failure.

This is seen when when enable CONFIG_PM. MIMXRT1170 does not support Power Management yet. I have changed this to an enhancement.

@mmahadevan108 PM should be supported on the RT1170, see #41171


Implement $^S

Implement $^S ($EXCEPTIONS_BEING_CAUGHT) = 1 if executing an eval or try block.
Implement it by setting a flag in perllib each time we enter an Eval. Only get it if SpecialVarsUsed{‘$^S’}. Handle looking for it in import. Handle the English name too.

Implemented in v1.027


Repo name is incorrectly detected from pull request body

For example, for spyder-ide/spyder#6286, loghub is returning
* [PR 6286](https://github.com/spyder-ide/spyder/pull/6286) - PR: Install and run window manager in Travis to fix focus issues and unskip now-passing tests ([4151](https://github.com/similar issues with CircleCI in /issues/4151), [](https://github.com/a ton of frustration I'm having with a the tests in a number of recent PRs. First crack at it will just ensure the code runs without error, and then I'll try enabling tests that failed due to focus/issues/))

Fixes by #88.


Add a hidden file in the fixtures and modify the Sources.find matcher…

… to ignore files beginning with a dot (.)
Should fix #17

@awochna Thanks for opening this PR! I see that you have included a fixture, but not a test to prove this new behavior. Can you please add it, and we'll be happy to merge. Thank you!

The presence of a hidden file broke other tests, but I've added a test specifying the behavior in test/integration/compilers_test.rb. If it needs to be elsewhere, please let me know.

@awochna Thanks for the clarification, it works great, thank you! :+1:


Gitea wiki repo uses urlencoded file name when creating new pages online

Gitea version (or commit ref): 1.14.1 and 1.15.0+dev-399-gda057996d at try.github.io
Operating system: Debian and whatever used in try.github.io
Can you reproduce the bug at https://try.gitea.io:

Yes https://try.gitea.io/jxjxjxj/jxjxjxj/wiki/_pages

Description
When creating a new wiki page online, it will generate a urlencoded file name for that page to the wiki repo, instead of using the page title as-is, which can make local management (i.e. clone the wiki repo locally and use it as a regular git repo to manage documentation or notes) vary hard.
Not sure if that's intended, but for reference, GitHub, instead, will (at least try to) use the wiki page title as-is as the name of the new file.

List of related issues that need to be discussed/fixed

[x] wiki page with non-urlencoded filename cannot be edited
[x] wiki page with non-urlencoded filename cannot be deleted
[ ] wiki page contains space cannot be viewed from the Gitea wiki webpage (e.g. Test Test.md)
[ ] when can we drop the backward compatibility so we can remove the urlencoded behavior?
[ ] once we no longer use urlencoded page, how to deal with pages that contain / character?

This is indeed a problem.

While I try to write a patch for this issue, I found there is a bug related to this issue.
If you created a non-urlencoded markdown file and push it to the wiki repo, the file name can be displayed correctly, but if you edit that page online and save it with the save button, it will create a new page with urlencoded file name, and the old non-urlencoded file will still exist.
My patch is here but I haven't tested it due to some other issue I've got, anyway here is the patch. be aware it's NOT TESTED
diff --git a/services/wiki/wiki.go b/services/wiki/wiki.go
index 75b9d1d1f..dbf2771b3 100644
--- a/services/wiki/wiki.go
+++ b/services/wiki/wiki.go
@@ -44,6 +44,11 @@ func NormalizeWikiName(name string) string {
 }

 // NameToFilename converts a wiki name to its corresponding filename.
+func NameToUnescapedFilename(name string) string {
+	name = strings.ReplaceAll(name, " ", "-")
+	return name + ".md"
+}
+
 func NameToFilename(name string) string {
 	name = strings.ReplaceAll(name, " ", "-")
 	return url.QueryEscape(name) + ".md"
@@ -81,6 +86,19 @@ func InitWiki(repo *models.Repository) error {
 	return nil
 }

+func CheckFileExistence(gitRepo *git.Repository, filePath string) (bool, error) {
+	filesInIndex, err := gitRepo.LsFiles(filePath)
+	if err != nil {
+		log.Error("%v", err)
+		return false, err
+	}
+	if util.IsStringInSlice(filePath, filesInIndex) {
+		return true, nil
+	}
+
+	return false, nil
+}
+
 // updateWikiPage adds a new page to the repository wiki.
 func updateWikiPage(doer *models.User, repo *models.Repository, oldWikiName, newWikiName, content, message string, isNew bool) (err error) {
 	if err = nameAllowed(newWikiName); err != nil {
@@ -133,27 +151,40 @@ func updateWikiPage(doer *models.User, repo *models.Repository, oldWikiName, new
 		}
 	}

-	newWikiPath := NameToFilename(newWikiName)
+	newWikiPath := NameToUnescapedFilename(newWikiName)
+	isWikiExist, err := CheckFileExistence(gitRepo, newWikiName)
+	if err != nil {
+		return err
+	}
+	if !isWikiExist {
+		newWikiPath = NameToFilename(newWikiName)
+	}
+
 	if isNew {
-		filesInIndex, err := gitRepo.LsFiles(newWikiPath)
+		isNewWikiExist, err := CheckFileExistence(gitRepo, newWikiPath)
 		if err != nil {
-			log.Error("%v", err)
 			return err
 		}
-		if util.IsStringInSlice(newWikiPath, filesInIndex) {
+		if isNewWikiExist {
 			return models.ErrWikiAlreadyExist{
 				Title: newWikiPath,
 			}
 		}
 	} else {
-		oldWikiPath := NameToFilename(oldWikiName)
-		filesInIndex, err := gitRepo.LsFiles(oldWikiPath)
+		oldWikiPath := NameToUnescapedFilename(oldWikiName)
+		isOldWikiExist, err := CheckFileExistence(gitRepo, newWikiName)
 		if err != nil {
-			log.Error("%v", err)
 			return err
 		}
+		if !isOldWikiExist {
+			oldWikiPath = NameToFilename(newWikiName)
+			isOldWikiExist, err = CheckFileExistence(gitRepo, newWikiName)
+			if err != nil {
+				return err
+			}
+		}

-		if util.IsStringInSlice(oldWikiPath, filesInIndex) {
+		if isOldWikiExist {
 			err := gitRepo.RemoveFilesFromIndex(oldWikiPath)
 			if err != nil {
 				log.Error("%v", err)

This patch will try to find if there is a non-urlencoded file in the repo and try to update that file if there is, which may fixes the bug mention in this comment. This will of course not fix the main issue since it will not try to use the wiki page name as-is as the newly created file name. Using the page name as-is by default may probably break already existed wikis so I'm not sure about the right way to do it.

I updated the description to match the main point of this issue.
While working on the third issue (wiki page contains space cannot be viewed from the Gitea wiki webpage), I found it seems harder to fix. I cannot find a way to fix this issue and also keep the old behavior not break. Any suggestions?
To help others reproduce that issue, you can:

Clone the wiki repo locally
Create a file names test test.md, commit it and push it to the wiki repo
Browse the file online.

I tested GitHub Wiki's behavior for reference:
When creating wiki page online (with Markdown chose as its format):

Wiki page name entered online
Created file name in wiki repo
Page URL  (last part only)
Page title displayed online

Wiki Page
Wiki-Page.md
Wiki-Page
Wiki Page

学习 C++
学习-C--.md
学习-C--
学习 C

Chrome / Chromium
Chrome---Chromium.md
Chrome---Chromium
Chrome Chromium

安装指南
安装指南.md
安装指南
安装指南

Space( ) got converted to dash(-), and it seems characters unsupported by GitHub Wiki (+ and /) will also get converted to dash(-) and will no longer exist on the page title.
When creating wiki page locally, then commit and push it to wiki repo:

Wiki file name created locally
Page URL (last part only)
Page title displayed online

KDE Connect.md
KDE-Connect
KDE Connect

掌握 C++
掌握-C--
掌握 C++

Test%20test+test test.md
Test%2520test-test-test
Test%20test+test test

Page URL seems weird but the page title can still be displayed correctly.
TODO:

[ ] Try to edit the page that created locally online and see if it creates new page or correctly updated existed pages.
[ ] Try to create a wiki page that contains % online and see the created page name.

Some additional tests:
Create 学习 C.md、学习-C.md、学习 C--.md in the repo, according to the previous comment, their URL would be 学习-C、学习-C and 学习-C--(first two URL are the same, and both of these pages display with the name 学习 C).

If we try to visit the URL ends with 学习-C: 学习 C.md content will be shown instead of 学习-C.md
If we try to edit the page with URL 学习-C-- and leave the title field unchanged (学习 C  ): An error will be shown says Cannot rename, target already exists.

BTW, I don't think GitHub Wiki dees are good since users won't be able to use some characters in a wiki title which can be confused. A page with title names 学习 C++ will be not possible, for example. I think these need to be discussed before working on future bug fixes related to this issue.


The sending queue.

This is a totally rewritten sending queue than what I had before. Even with the mutex, I was still getting panics while blocking on the buffered reader about every 15 to 30 minutes.
I reworked the synchronization so the sending and receiving loop will never have their connections closed out from under them.  Only the sending loop closes connections, and it only does so when it is sure that the receiving loop is done with the connection as well.
This code is simpler than the previous version. Only two go routines instead of 3. No condition variable.
conn.go - wrapper around net.Conn. Writes JSON to connection and reads responses
logger.go - wrapper around log.logger. logs errors.
queue.go - The queue itself
request.go - One request
sender.go - The meat of this code. Contains the sending and receiving loops. This is the critical file where all the synchronization is.

Addressed these comments in senderloop2 branch.


Video doesn't play at all, but vast ads do

Hello,
I have tried to implement the video it all seems to work except that the video won't play, only the vast ad shows as soon as the page load!
the video can be seen here: https://huntergames.online/about.html  note: need to scroll to see it!
I would greatly appreciate anyone's help, thank you in advance!

Hi @huntergames79, it seems that your player is loading a youtube url as source, Fluid Player supports the same formats as HTML5 video (mp4, webm, ogg), as well as hls and dash for streaming.

So it means that a google drive share link doesn't work? You need to see this video that this person made it work: https://www.youtube.com/watch?v=v529dXuFRAA

If the source url responds with a supported and well formed media file it will work

@huntergames79 are you still having issues with this?

@huntergames79 is this still and issue? Can I close this ticket? Thank you!

yes, but you can close it! Thank you
Oscar Medina Matiz
On Mon, Feb 26, 2024 at 8:18 AM Sergio Sousa @.***>
wrote:

@huntergames79 https://github.com/huntergames79 is this still and
issue? Can I close this ticket? Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/fluid-player/fluid-player/issues/700#issuecomment-1963897835,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4HC3PIJSENCIA4KI53IAYDYVRVP7AVCNFSM6AAAAAASCKHA5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRTHA4TOOBTGU
.
You are receiving this because you were mentioned.Message ID:
@.***>