Importing via APIM CLI fails

APIM-CLI version
1.10.1
API-Management version
7.7.20201130
Bug description
Customer is using APIM CLI 1.10.1 to import updated API's and imports are failing on one of the environments.
4209   ERROR APIManagerAPIAdapter| Error deleting API-Proxy. Response-Code: 500, Response: '{"errors":[
{"code":500,"message":"Internal server error"}
]}'
4209   ERROR     RollbackAPIProxy| Error while deleting FE-API to roll it back
com.axway.apim.lib.errorHandling.AppException: Cannot delete API-Proxy.
at com.axway.apim.adapter.apis.APIManagerAPIAdapter.deleteAPIProxy(APIManagerAPIAdapter.java:658) ~[apimcli-apim-adapter-1.10.1.jar:1.10.1]
Attached logs and APIM CLI Output.
81d56f62ec5dc7355c7ad262.txt
81d56f62ef5dbc3491ab9189.txt
81d56f62f05de13e46838168.txt
cmd_debug_02052022.txt
Tested with Customer by creating a Basic API and reproduced issue
Imported a back-up of the customer KPS in test environment. Ran into export issues when exporting the client's API's.
Created Basic API with Frontend and Backend and exported and imported successfully with APIM CLI.
Checked Cassandra table: api_portal_portaltimestamp and Policy Studio config if "id" filed is set to auto-generated.
Steps to reproduce
No response
Relevant log output
No response

Thanks for reporting this issue. Can you please check, the Cassandra consistency level is configured properly?

Replication strategy is Network Topology  Strategy with READ and WRITE Consistency to LOCAL_Quorum.
We checked consistency across nodes with nodetool status and it was 100% on all 3 nodes.

I strongly believe it relates to the Cassandra communication, as the APIM-CLI works pretty well for many other customers. The error behavior points me into the direction of Cassandra consistency level, which only occurs when using the API-Manager REST-API quite fast, as the APIM-CLI is doing.
The nodetool may show all good, as the Cassandra DB has replicated the changes in the meantime, but when reading information written for instance only to Cassandra node 1 and is read a millisecond later from node 2 or 3 it may result wrong information.
Please confirm you checked the consistency level not only on the general Cassandra-Settings, but also on the KPS-Settings for the API-Manager collections:

Hi, the settings in KPS for the API Server, OAuth and API Portal have read and write consistency ONE

Hi Chris,
After setting the collections to Local Quorum, the imports are still failing.
Any thoughts?
Attached trace
latest_trace.txt

Hi Chris,
From my testing the issue comes from a corrupted kps. I've managed to import the api's successful to another env and doing the export with changes and importing back the API's there are no issues.
You can close the case for now. Thank you


Using indirect eval call with (0,eval) syntax triggers bad invocation without line info

If you place the following code into JSHint:
(0, eval)('alert(1);');

You'll get the following warning:

file.js: line 0, col 0, Bad invocation. (W067)

However, there is no line information given, which can lead to wasting time trying to find the source of the warning. It does not matter what function it is, whether it's eval, alert or any other function, nor the preceding values. Should this be fixed?

Is @Qantas94Heavy reporting that jshint shouldn't throw W067 or that it doesn't report the position?

I'm not sure whether it was intended to throw an error, but the lack of position makes it difficult to debug and the warning cannot be suppressed by using a comment like // jshint -W067.

I'm not sure whether it was intended to throw an error

I think it is, as this code triggers the warning:
function a() {}

(0, a)();

but the lack of position

This has been fixed (tested on jshint.com)

As a workaround, the following allows you to use indirect eval without getting a "bad invocation" warning:
[eval][0]('alert(1);');

This is not just for indirect eval. (0, a[b])() is a pattern to prevent a from being the this for the call to a[b] (such as when a is just a collection or record containing a function and not an object-with-methods).


Removing custom metric/command line defaults

Looks like there were some leftovers in there for custom metrics and command line flags that didn't really make sense as defaults. This PR removes them.

Closing in favor of major refactoring


OGCS: Use portable zip instead of nupkg

I found it seems nupkg will be released later than zip. Yesterday there is only zip and exe. And in the past only beta version has nupkg. Don't know will there be a nupkg in the next release.


Add branching strategy diagram to docs site

The diagram aims to provide context on the workflow surrounding bug fixes, feature development, and how things make their way across multiple simultaneous branches.

complete


🐛:  Theme change does not work properly, light mode is only applied to the sidebar.

:red_circle: Title : Improve theme change, i.e. switching between light mode and dark mode
:red_circle: Bug : Currently, the theme change only happens in the sidebar, the rest of the website remains black (in dark mode).
:red_circle: Changes :  Implement theme change throughout the webapp.
Ensure the switch between the two themes is applied evenly across the webapp.
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name :  GARIMA
What is your participant role?: GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

/assign

@techy4shri go ahead.

@techy4shri What's the update?

@techy4shri What's the update?

Testing themes for proper visibility. Some parts were excluded in the previous code so trying to implement golbal control properly.

Use this context api. I've already created these 2  isDarkMode, setDarkMode, parameters. It will help to declare it globally.
Do it fast within 2 days.


UI improvements

[ ] TUI improvements
[ ] Abstract TUI into separate file
[ ] GUI

For the TUI, do you plan on writing an interop handler in another language, or using something like ncurses to capture key presses and alter the state of the window?

Honestly I'm not even up to considering implementation yet, I just know that I want to make the TUI more user friendly (ie. the current T:x:y,z UX I stole from Corp is not the greatest) but I haven't even come up with ideas yet on how to do that.

Well, if you'd like to discuss ideas, and would appreciate some assistance, you know where to contact me. 👍


Add support for database migrations

Sometimes change is necessary, and once Relay exits beta, we'll need a concept of migrations for this.

Added in 63de638f53e34208b43b8fa25720f3a3f38bf461.


Add support for IAMServiceAccountCredentials API

Adds support for the IAM Service Account Credentials API, specifically the signJwt endpoint.
Usage:
func signJWT() {
    let client = try IAMServiceAccountCredentialsClient(credentials: credentialsConfiguration,
                                           config: IAMServiceAccountCredentialsConfiguration,
                                           httpClient: client,
                                           eventLoop: myEventLoop)

    let payload: JWTPayload = MyPayload(name: "key", value: "value")

    client.api.signJWT(payload, serviceAccount: "my-service-account@my-domain.com").map { response in
        print(response.signedJwt) // Prints JWT signed with the given service account's credentials
    }
}

@nathanfallet can we get the tags mentioned in #53 set up please (or at least the 1.0) - I've got a corresponding PR ready to go for #54 that wraps this new functionality in vapor-community/google-cloud but it needs a tag to point to 🙂


Check ignored errors

Check previously-ignored errors in issue_milestone.go

CI build failed

Codecov Report

Merging #3213 into master will decrease coverage by 0.01%.
The diff coverage is 48.71%.

@@            Coverage Diff             @@
##           master    #3213      +/-   ##
==========================================
- Coverage   34.69%   34.68%   -0.02%
==========================================
  Files         277      277
  Lines       39990    40014      +24
==========================================
+ Hits        13876    13879       +3
- Misses      24116    24132      +16
- Partials     1998     2003       +5

Impacted Files
Coverage Δ

routers/repo/issue.go
32.78% <0%> (-0.12%)
:arrow_down:

models/issue_milestone.go
57.8% <55.88%> (-3.03%)
:arrow_down:

models/repo_list.go
65.62% <0%> (-1.57%)
:arrow_down:

models/repo.go
41.47% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM

LGTM


Standard GitHub actions

Changes

isort in requirements at setup.py
lint yml workflow for pythons from 3.6 to 3.9
locust yml workflow
PyPI yml release workflow

How to test these changes?
Common files
Related issues
Resolves https://github.com/bugout-dev/moonworm/issues/8

@bugout-dev check

[ ] Require myoy fixes mypy moonworm/
[ ] isort fun isort moonworm
[ ] Add secrets BUGOUT_SECRET for locust
[ ] Add secrets PYPI_USERNAME and PYPI_PASSWORD for PyPI relese

@bugout-dev check

[ ] Require myoy fixes mypy moonworm/
[ ] isort fun isort moonworm
[ ] Add secrets BUGOUT_SECRET for locust
[ ] Add secrets PYPI_USERNAME and PYPI_PASSWORD for PyPI relese

@bugout-dev accept  Require myoy fixes mypy moonworm/


🙌 Wrong warning message when listing on unemeta.com

Our website is not in AllowList, and, our nft listing logic matched this code, It causes the error message displayed on ui, which makes users feel quite confused.
I also check this #12, Hope to add support for unemeta.com

Thanks! I'll have to add contract address checks to the signature rather than just looking at the structure of the message. Should be do-able. Thanks for the report, will try to get this done soon.

Do you know the verifyingContract address for unemeta?

Do you know the verifyingContract address for unemeta?

0x2445A4E934AF7C786755931610Af099554Ba1354

This has been fixed in 0.2.6


PNG, JPG note rendered

I try to render some pictures with html2Pdf.js lib.
But on PDF generation, i get no pictures.
Here, a simple exemple where  or  are ignored by the lib (i guess) :
https://jsfiddle.net/Gaetanbrl/jkzu1b9g/

Fixed by :
https://github.com/eKoopmans/html2pdf.js/issues/514#issuecomment-1430786741
With :
html2canvas:  {useCORS:true},


[GRPC] Handle non-existent transaction in server interceptor

What does this pull request do?
This change handles the case when APM is not recording/disabled and client.begin_transaction() just returns None. I wasn't able to figure out how to add a test for this. Do you have an advice/hint?
Related issues

https://github.com/elastic/apm-agent-python/issues/1739
Introduced in the initial gRPC instrumentation (https://github.com/elastic/apm-agent-python/pull/1703)

huh. not sure why this was auto-merged before the tests completed :/


Remove -xs infixes

Description
Bootstrap 4 deprecates the -xs infix for columns as this is now the new implied default.
Tasks

[ ] Remove all instances of -xs

Non-issue.


Stone Portal + TargetPortal problem

Hi and thanks for the mod!
The problem I'm having is this: when using Stone Portal with TargetPortal, the "teleport everything" option of Stone Portal doesn't work.
It's a bummer, because I use Stone Portal as a second, really expensive portal that can teleport metals.
Not sure which side this is on, I've reported this bug to the author of TargetPortal as well.

Probably need to write some custom code for the compatibility. I can take a look at some point if TargetPortal has sources available.

Just wanted to say I'm experiencing the same issue.  Target Portal has the ability to allow teleportation of all items with the normal portal, but like Urfin13, I wanted to have the stone portal be able to do it since it costs way more.  Anyway, Smoothbrain wrote Target Portal, and he has source code on his github here: https://github.com/blaxxun-boop/TargetPortal
Would be great if these 2 mods could be used together properly!


Add kv_shared_past_present option

In the config, there is a new bool option: { "model": { "decoder": { "kv_shared_past_present": true } } } that will cause the KV cache buffers to be allocated once to the max size, and to have the past & present buffers be the same buffers (so no memory allocations after the first iteration, and only half of the max memory).
Does not apply to KV_Cache_Combined.
Tested with phi2 model on cuda

could you please set it true in the genai_config.json for cuda case while building the model with builder.py?


Use Ticket Events on Frontend

Questions:

Do we want to sort events in-between notes or show them in a separate section?  I prefer to sort between notes.
Do we want to create "pseudo-events" (ones that don't exist in the database) to be displayed for things like the ticket being opened?  Or should that be an actual event?

Added in 8f940e2ca3b2f6de515344f63e48787309235070, just needs to be styled.  Events show between notes.


After upgrade strange errors

Describe the bug
After updating the package to the latest version from the store, we get strange errors. I am asking for help in daignosis, which should be improved.
[IMPORTANT] How can we reproduce the issue, step by step:
Upgrade from previous version to current one in Unity 2021.
Expected behavior
Errors from Mirror source code.
(0,0): error Action can't be deserialized because it has no default constructor. Don't use Action in [SyncVar]s, Rpcs, Cmds, etc. (at System.Action)

(0,0): error Cannot generate reader for generic variable Func`1. Use a supported type or provide a custom reader (at System.Func`1<System.Boolean>)

(0,0): error IsRecording has an unsupported type (at System.Func`1<System.Boolean> Mirror.SyncObject::IsRecording)

(0,0): error Cannot generate writer for generic type Func`1. Use a supported type or provide a custom writer (at System.Func`1<System.Boolean>)

Assembly 'Library/ScriptAssemblies/Assembly-CSharp-Editor.dll' will not be loaded due to errors:
Unable to resolve reference 'Assembly-CSharp'. Is the assembly missing or incompatible with the current platform?
Reference validation can be disabled in the Plugin Inspector.

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows
Build target: Android (Oculus/VR)
Unity version: 2021.1.22f1
Mirror branch: Unity store / current version

I see that IsRecording is in Mirror.SyncObject and code is here:
public Func<bool> IsRecording = () => true;

Support is available in our Discord, jump and someone will help.


Cannot change to flatpak steam_root

When i try to set steam_root the program always want to check native installation.
protonutils config steam_root ~/.var/app/com.valvesoftware.Steam/data/Steam/
Steam root not a directory: /home/<user>/.local/share/Steam

Thanks for reporting! I fixed the issue and released v1.6.7. I already closed the issue but please let me know if it does not work for you!


runs search indexer at container's start

as a www-data execute a php bin/indexer.php -c (--clear) to create search index for existing wiki pages... otherwise search wouldn't work!

Thanks!


Add standard daily weather statistics

Add additional standard weather statistics such as:

Rainfall amount since midnight
Date/time of the start of the most recent storm
Date/time of the end of the most recent storm
Total rainfall of the most recent storm
High temp since midnight
Low temp since midnight

Create a new stored procedure and include it in the setup instructions (best if most SP's are created via a single script).
  Will be included in the next merge to master.


Norminette infinite loop with this student code.

Describe the bug
When the workers(student iMacs) accepts this code for review infinite loops occurs and high cpu load appears.
Erroneous code
Check pinned files
Additional infos

OS: ProductName:	Mac OS X
ProductVersion:	10.14.6
BuildVersion:	18G9216
python --version:  Python 2.7.16
norminette -v: 3.3.0

Additional context

ft_print_comb.txt

Hello,
I apologize but that's beyond the scope of the norminette and would require quite a lot of work to fix.
Norminette is expected to work on files that compile. Using it on files that don't compile may containt unexpected behaviour, and that's a design choice.
Also, you are using python2.7 and norminette isn't developped for 2.7. Please use a more up to date version (3.7, 3.8, 3.9..)


SUID sandbox helper breaks packages in Single User Installation

Describe the bug
When using Nix as "Single User Installation", the SUID sandbox helpers break packages, e.g. Chromium.
SUID sanbox helpers do not make any sense when using Nix as "Single User Installation", and should be fixed/ignored there.
Example:
$ chromium
...:FATAL:setuid_sandbox_host.cc(158)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /nix/store/jp4jzpbnbsyb748vrqqxablf9sgspr5q-chromium-90.0.4430.93-sandbox/bin/__chromium-suid-sandbox is owned by root and has mode 4755.
Trace/breakpoint trap

To Reproduce
Steps to reproduce the behavior:

Install Nix as "Single User Installation"
Install Chromium (nix-env -iA nixpkgs.chromium)
Try to start Chromium

Notify maintainers
@primeos @thefloweringash @bendlas
Metadata

system: "x86_64-linux"
multi-user?: no
sandbox: no
version: nix-env (Nix) 2.3.10
channels(user): "nixpkgs-21.05pre286914.3cc34f9e559"
nixpkgs: /home/user/.nix-defexpr/channels/nixpkgs

Huh, that's strange. IIRC it should only be used as a fallback so I assume you're on a system with user namespaces disabled (on Debian based system there's sysctl kernel.unprivileged_userns_clone).
Personally I'd like to drop the legacy/deprecated SUID sandbox helper binary entirely but AFAIK it would still break some NixOS setups (e.g. for users of a hardened kernel).
Anyway, there are probably three options but all come with security implications:

sysctl -w kernel.unprivileged_userns_clone=1
Setting CHROME_DEVEL_SANDBOX (needs changes in Nixpkgs, s. below)
Using --no-sandbox - But I wouldn't recommend that... :D

The sandboxing can be verified via chrome://sandbox/.
As for the Nixpkgs packaging: We should likely remove the following two lines:
https://github.com/NixOS/nixpkgs/blob/2b76957b3954318904d1cacd0bff9d16bba9169c/pkgs/applications/networking/browsers/chromium/default.nix#L180-L181
That always refers to a Nix store path so I think this fallback should never actually work.

I'm getting the same error. See #308128.
It also affects youtube-music.
Also, it seems related, ping, which fails with ping: Lacking privilege for icmp socket, unless I change its permission bits.
My kernel.unprivileged_userns_clone flag is already set to 1.
I'm running Ubuntu with home-manager.
In all these cases, if I set the permission bits to 4555 (with chmod 4555 <file>) they all work. They are initially set to 0555.
Any suggestions on how I could work around these problems?

Another standalone home-manager user here. I just updated my Ubuntu box  from 23.10 -> 24.04 and am now running into this issue for Chromium and Slack (probably anything chromium or electron based). kernel.unprivileged_userns_clone is also already set to 1. Setting the permission of the SUID sandbox binary to 4555 does workaround this for now.
I didn't have any issues when on Ubuntu 23.10, this only started happening after the upgrade to 24.04.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.8.0-31-generic, Ubuntu, 24.04 LTS (Noble Numbat), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.21.2`
 - nixpkgs: `/nix/store/7sn1rdxkdszc3mmqk61fhf3vmvakqnhb-source`

@camperboy1000 I'm also on 24.04.

I'm getting the same error. See #308128. It also affects youtube-music. Also, it seems related, ping, which fails with ping: Lacking privilege for icmp socket, unless I change its permission bits.
My kernel.unprivileged_userns_clone flag is already set to 1.
I'm running Ubuntu with home-manager.
In all these cases, if I set the permission bits to 4555 (with chmod 4555 <file>) they all work. They are initially set to 0555.
Any suggestions on how I could work around these problems?

I'm experiencing the same issues. Affected programs are the Brave browser and Obsidian. I'm running a fresh system with Kubuntu 24.04 (haven't tried 23.10).
 - system: `"x86_64-linux"`
 - host os: `Linux 6.8.0-31-generic, Ubuntu, 24.04 LTS (Noble Numbat), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.21.2`
 - nixpkgs: `/nix/store/fjs1x9gz2n1dvjqcyyl6qsw110qxc2zn-source`

It looks like this is related to apparmor being enabled by default on Ubuntu 24.04.
Workaround: for one session, do:
echo 0 | sudo tee /proc/sys/kernel/apparmor_restrict_unprivileged_userns

This works until the next reboot, I have tried it and it fixes the problems we are having.
They mention a permanent way to disable it, which is how it was before 24.04:

Disable this restriction using a persistent setting by adding a new file (/etc/sysctl.d/60-apparmor-namespace.conf) with the following contents:
kernel.apparmor_restrict_unprivileged_userns=0

Read more:
https://discourse.ubuntu.com/t/ubuntu-24-04-lts-noble-numbat-release-notes/39890#unprivileged-user-namespace-restrictions-15

I'm back. I just updated my Ubuntu 24.04 and I'm having the problems again. My settings did not change, as we have discussed in the issue, but the problem is happening again. If anyone has a way to fix it this time, please let us know.

Can confirm that I am seeing the same thing after updating to Ubuntu 24.04.  For me the failing program is codium.
$ codium --verbose
[39172:0924/234538.836078:FATAL:setuid_sandbox_host.cc(158)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /nix/store/abha8v0h5rj1gjx0pr9g3y85s50rrnj3-vscodium-<IP_ADDRESS>56/lib/vscode/chrome-sandbox is owned by root and has mode 4755.

Hello! did someone managed to create a specific AppArmor generic rule, inspired from the one stated in release note https://discourse.ubuntu.com/t/ubuntu-24-04-lts-noble-numbat-release-notes/39890#p-99950-unprivileged-user-namespace-restrictions ?
Thanks

This might be related and it contains a rule that works with the Nix store but the question is about vscodium.
https://stackoverflow.com/questions/79220945/nix-with-home-manager-on-ubuntu-error-with-sandboxing

Creating AppArmor profiles in /etc/apparmor.d/ worked for me. Without these, Google Chrome wouldn't launch due to the same error from SUID sandbox helper after upgrading to Ubuntu 24.04 and Firefox would show a warning banner that said something like "some of Firefox’s security features may offer less protection" with a link to this help article.
/etc/apparmor.d/nix-firefox for example:
abi <abi/4.0>,

include <tunables/global>

profile nix-firefox /nix/store/*-firefox-*/**/* flags=(default_allow) {
  userns,
}


Only mutate tested code

Right now, all code is being mutated. Code coverage has been collected before code is mutated. This makes it possible to only mutate code that has actually been tested. It is possible that only mutating tested code is slower than mutating everything due to the fact that code coverage has to be analysed.
This feature should not replace the current 'mutate everything' system. It should be an option (or the default, making 'mutate everything' an option).

There will be a mutator plugin in the future which will make supporting this more difficult because plugin writers will have to support this. The performance impact of mutating everything is neglegable


HTML code used in initialData cannot load scripts and CSS from assets

[x] I have read the README
[x] I have already searched for the same problem
[x] I have read and done the setup for Android: IMPORTANT Note for Android and iOS and IMPORTANT Note for Android
[x] I have read and done the setup for iOS: IMPORTANT Note for Android and iOS and IMPORTANT Note for iOS
[x] I have ran the sample app and it does not work there

Environment

Technology
Version

Flutter version
1.26

Plugin version
4.0.0+4

Android version
11

iOS version
N/A

Xcode version
N/A

Device information: Galaxy S8
Description
Expected behavior:
Scripts and CSS loaded from assets
Current behavior:
Scripts and CSS not loaded, and even with debuggingEnabled I do not see anything in logs
Steps to reproduce
InAppWebView(
                initialData: InAppWebViewInitialData(
                  data: """
<!DOCTYPE html>

        <meta charset="UTF-8">
        <meta name="viewport" content="width=device-width, user-scalable=no, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
<link href=\"assets/css-file.css\" rel=\"stylesheet\">

TEST

                  """
                ),
                initialOptions: InAppWebViewGroupOptions(
                    crossPlatform: InAppWebViewOptions(
                      debuggingEnabled: true,
                    )
                ),
              ),

and make some random CSS file in assets that modifes the h1.
Observe that the css will not be applied to the h1.
Maybe I'm doing something wrong?

If you  are using InAppWebViewInitialData you also need to pass baseUrl to load stuff in like this
initialData: InAppWebViewInitialData(data: "your html", baseUrl: yourUrl).
If you are hoping to load your local css or asset into that html I think the best way to do so is by using the InAppWebViewController.loadFile() function

Oh wow I didn't even know that was a method on the controller. I will try that and close this if it works. Thanks!

We're struggling with this same issue, we can't use loadFile(). We're building a document viewer that gets the document html remotely, but has the css, js and images locally (that stuff is always the same and the server expects us to have it locally). We need a way to integrate the assets "on the fly" and it worked perfectly in earlier versions using loadData().

@tneotia with initialData you don't have a base href as you would have in a normal website loaded from a domain or using the localhost server. So, you need to specify in that case the full file path.
For example, on Android, using your file path, it should to be file:///android_asset/flutter_assets/assets/css-file.css, that is
<link href="file:///android_asset/flutter_assets/assets/css-file.css" rel="stylesheet">

Or you can create and start a localhost server, so the path to your css file would be http://localhost:8080/assets/css-file.css, or you can use the initialFile to load an HTML file inside your assets directory (in this case, css-file.css would be ok if the HTML file is at the same folder level)

I'm fairly certain I tried the full asset path like that but I will try it again. Thanks!

I tested it on the plugin example project and it worked. Be sure that is the right path of your file

@tneotia with initialData you don't have a base href as you would have in a normal website loaded from a domain or using the localhost server. So, you need to specify in that case the full file path.
For example, on Android, using your file path, it should to be file:///android_asset/flutter_assets/assets/css-file.css, that is
<link href="file:///android_asset/flutter_assets/assets/css-file.css" rel="stylesheet">

Or you can create and start a localhost server, so the path to your css file would be http://localhost:8080/assets/css-file.css, or you can use the initialFile to load an HTML file inside your assets directory (in this case, css-file.css would be ok if the HTML file is at the same folder level)
How to use in iOS

@pichillilorenzo just got around to testing this and it does work correctly, thanks. How should I approach the file path for iOS? The android_asset directory surely doesn't apply for iOS, correct?

iOS still not work current.
This is my request issue: https://github.com/tneotia/html-editor-enhanced/issues/169

Is there a way to load remote css and javascript files from loadData other than from localhost?


Leading zero-width no-break space in keys

Operating System: Mac OS 10.14.5
Node Version: 10.15.3
NPM Version: 6.9.0
csv-parser Version: 2.3.0

Expected Behavior
With a CSV looking like:
name,start_date,end_date
some - name,2019-08-25,2019-12-31
another - name,2019-08-05,2019-12-31

after parsing, accessing row.name should return a string
Actual Behavior
The row property doesn't actually have name, but rather name preceeded by the utf char 65279, or a zero-width no-break space. I've created the CSV by hand, so I have no idea why that character would be in there
How Do We Reproduce?
fs.createReadStream(input_path)
        .pipe(parse())
        .on('data', (data) => console.log(data.name)) // undefined :(
        .on('end', () => {});

Please use JSON.stringify on each row and paste the result here. Chances are your editor is inserting a Byte Order Mark (BOM), in which case you should strip that character before sending data to csv-parser, but let's have a look at that JSON first.

I believe that is the case. I'm using VS Code. Is there a way to not save the BOM? Or do I just need to strip that leading character?

I'm not a VS Code user, so I'm afraid you'd have to ask those folks. But you can use intermediate tools like https://github.com/sindresorhus/strip-bom and https://github.com/sindresorhus/strip-bom-cli to do that as part of the process.

I found this issue as well. The CSVs were created using Excel. First saved as xlsx, then as CSV. Printing the JSON.stringify() showed the header as ' USER_NAME' instead of the expected 'USER_NAME'. I wasn't able to get around this by modifying the CSV file, but I could instead trim the headers using the included mapHeaders function:
.pipe(csv({
    mapHeaders: ({ header, index }) => header.trim()
  }))

this is exactly what I needed, thank you! @fedpettinella


ActiveDirectoryDsc: Add notice to pull request template about wiki content

I think we should add a notice to the pull request template saying that the wiki content is generate automatically from certain files (schema, resource README.md, and resource examples files) so that new contributors know that they do not need to manually update the wiki, and doing so could remove the manually added content in the next release update.

Also maybe a notice should be added to the first page of the Wiki? Maybe re-word this section

This wiki is currently updated manually by a maintainer (using automatically generated documentation by the CI pipeline), so there can be some delay before this wiki is updated for the latest release.

Definitely. I didn't realize that the Wiki was open for anyone to edit. Is that by default?

https://help.github.com/en/articles/changing-access-permissions-for-wikis

Yes, default it is open for everyone. I don't have admin rights so can't change the permission (just collaborator). But having it open is okay I think, if we just could help avoid changes to files that is generated automatically so no one makes edits in the wrong place. Every change is logged in the normal commit history, so it could be traced (and reverted if need be). But if we feel we should change the permission that is possible too.

Does anyone get notified if the Wiki is updated?

No, there ate nog GitHub notifikations for that I think. :thinking: Probably need a function that clones the wiki repo and look at the commit history.

I would recommend getting it set to collaborator update only then. Someone could change a page, and no one notices.

My thought was that hopefully the community notices, and if a change is made to a markdown file that is automatically generated, that change will be reverted as soon as the new markdown file is published (assuming we have this automated in the near future).

But if we have it fully automated then we could switch the permission :thinking:

@gaelcolas what are you thoughts here?

Obvious answer is to lock down the permissions, but in light of this, would you want the wording under the top section (thanks for submitting) or the Task List? Any objections to the below?

Please note that the wiki content is automatically generated using the information contained within the schema, resource README.md, and resource examples files. Manual editing of the wiki should not be performed as changes can be overwritten following the next release update.

Closing this as the Wiki is now only editable by dsccommunity team members.


canGetCookieForPInvokeCalliSig not implemented

I was trying to compile this project with CoreRT: https://github.com/mellinoe/veldrid/tree/master/src/NeoDemo.
I added the latest Microsoft.DotNet.ILCompiler package and simply ran dotnet publish -r win-x64.
  ILCompiler.CodeGenerationFailedException: Code generation failed ---> System.NotImplementedException: canGetCookieForPInvokeCalliSig
     at Internal.JitInterface.CorInfoImpl.canGetCookieForPInvokeCalliSig(CORINFO_SIG_INFO* szMetaSig)
     at Internal.JitInterface.CorInfoImpl._canGetCookieForPInvokeCalliSig(IntPtr thisHandle, IntPtr* ppException, CORINFO_SIG_INFO* szMetaSig)
     --- End of inner exception stack trace ---
     at Internal.JitInterface.CorInfoImpl.CompileMethod(IMethodCodeNode methodCodeNodeNeedingCode, MethodIL methodIL)
     at ILCompiler.RyuJitCompilation.ComputeDependencyNodeDependencies(List`1 obj)
     at ILCompiler.DependencyAnalysisFramework.DependencyAnalyzer`2.ComputeMarkedNodes()
     at ILCompiler.RyuJitCompilation.CompileInternal(String outputFile, ObjectDumper dumper)
     at ILCompiler.Compilation.ILCompiler.ICompilation.Compile(String outputFile, ObjectDumper dumper)
     at ILCompiler.Program.Run(String[] args)
     at ILCompiler.Program.Main(String[] args)

Any plans to implement this functionality, or is it tracked anywhere else? I searched briefly but didn't find anything.

Thanks for reporting it! We don't track this anywhere because we didn't hit it yet. Seems like this is from a calli to a p/invoke. I'll have a look if we can do something easy there in the next couple of days.

Seems like this is from a calli to a p/invoke.

Yes, that's likely, and it's probably from this library. If that's the case, then a simpler repro could probably be created from that package alone.

@MichalStrehovsky +1, can't use SharpDX due this :-\

@MichalStrehovsky +1 we are hitting this issue aswell in our project. Would greatly appreciate a fix

@mellinoe @Krakean @feliwir The CoreRT packages with the fix are available now. Please give it a try and please let us know if you run into other blocking issues for you project.

@jkotas I get a lot further now, and most of my code works. I still have an issue with SharpDX which I filed over in #6042 .


fix: bin cleanup

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

#244  👈
#243
main

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

Join @refcell and the rest of your teammates on  Graphite


[Refactor] Consolidate the replica related maps such as bdev map, mode map and address map in SPDK engine

Is your improvement request related to a feature? Please describe (👍 if you like this request)

Consolidate the maps in longhorn-spdk-engine and make them easy to be maintained.
https://github.com/longhorn/longhorn-spdk-engine/pull/205
Describe the solution you'd like

Describe alternatives you've considered

Additional context

@derekbit can we add some context to the description? thanks.

@derekbit can we add some context to the description? thanks.

Added. @shuo-wu and I discussed the replica related maps in longhorn-spdk-engine yesterday. The separate maps are not easy to maintain, so we need to refactor them a bit.

@shuo-wu The integration test for the latest longhorn-spdk-engine commit (longhorn-spdk-engine 114edff) failed: https://github.com/longhorn/longhorn-spdk-engine/actions/runs/11996443739/job/33440890249.
Could you help take a look? Thank you!

Keep repeating the unit test on longhorn-spdk-engine 114edff, sometimes (~1/5) gets the following error:
OOPS: 3 passed, 1 FAILED
    testing.go:1399: race detected during execution of test
--- FAIL: Test (295.30s)
time="2024-11-26T10:02:24Z" level=info msg="spdk gRPC server: stopped broadcasting instances due to the context done"
FAIL
coverage: [no statements]
FAIL	github.com/longhorn/longhorn-spdk-engine/pkg	295.359s
=== RUN   Test
OK: 1 passed
--- PASS: Test (0.00s)
PASS
coverage: 0.2% of statements
ok  	github.com/longhorn/longhorn-spdk-engine/pkg/spdk	1.072s	coverage: 0.2% of statements
FAIL
spdk_tgt process killed
FATA[0374] exit status 1
make: *** [Makefile:11: ci] Error 1

@derekbit the code has been merged, should we move this back to 1.8.0?

Yes, updated. Thank you.

@yangchiu Would you retry the unit test? There should be lots of modifications and fixes since your last test


Bug after polygon drawing

There is a bug after the polygon drawing while clicking the cancel button on the editor panel. I draw a polygon then the editor panel appears and when I click the cancel button the editor panel disappears but the polygon does not. The polygon still appears even if clicking the cancel button. The same steps work while drawing a rectangle, but it doesn't work by drawing a polygon.
Could you fix this bug, please?
(tested with chrome browser)

Fixed in the latest release.


冯版单字音

TODOs:

[x] Add a field to FengDocument for 单字音
[x] Display 单字音 on UI
[x] Integrate with TTS (#272)
[ ] Integrate with Search

/cc @DDFShanyu

Has it been integrated with search?
备考，以防我误解此词：

Has it been integrated with search?

Nope. Work in progress: #922


Upgrading kube2pulumi to use<EMAIL_ADDRESS>causes changes in YAML comments

What happened?
Upgrading kube2pulumi to use the latest Pulumi SDK v3.46.0 forces an upgrade from github.com/goccy/go-yaml v1.8.*  to github.com/goccy/go-yaml v1.9.*. Unfortunately, go-yaml seems to have changed how comments are represented in yaml AST.
pkg/yaml2pcl/yaml2pcl.go:490:43: comment.Value undefined (type *ast.CommentGroupNode has no field or method Value)
pkg/yaml2pcl/yaml2pcl.go:506:11: comment.Prev undefined (type *ast.CommentGroupNode has no field or method Prev)
pkg/yaml2pcl/yaml2pcl.go:507:11: comment.Next undefined (type *ast.CommentGroupNode has no field or method Next)

Steps to reproduce
see above
Expected Behavior
no change in behavior
Actual Behavior
change in comments
Output of pulumi about
No response
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

I dug into this, and it unfortunately appears to be a substantial breaking change in the parsing library. That got me thinking that it might be possible to swap out this conversion logic and use the library underpinning our Pulumi YAML support instead. Let's investigate that option before trying to fix these low-level changes.

After some further testing, I found that the breaking API change was in v1.9.0 of the go-yaml library, so I pinned the dependency to v1.8.10 to avoid the API change for now.


Jenkins master build failed at grpc-jenkins-windows-worker3 FileSystemException

The last relatively green master build is https://grpc-testing.appspot.com/job/gRPC_master/8345/ at Oct 15, 2015 8:42:01 PM (since last Thursday night, about 4 days ago). Ever since then (8346 and onward), all the master builds are failing at the very top with this stack trace
Started by timer
Building remotely on grpc-jenkins-windows-worker3 (x86_64 i386 windows) in workspace c:\jenkins\workspace\gRPC_master
Wiping out workspace first.
java.io.IOException: remote file operation failed: c:\jenkins\workspace\gRPC_master at hudson.remoting.Channel@6877b8a3:grpc-jenkins-windows-worker3: java.nio.file.FileSystemException: c:\jenkins\workspace\gRPC_master\config\dbg\language\c\platform\windows: The process cannot access the file because it is being used by another process.

	at hudson.FilePath.act(FilePath.java:987)
	at hudson.FilePath.act(FilePath.java:969)
	at hudson.FilePath.deleteContents(FilePath.java:1184)
	at hudson.plugins.git.extensions.impl.WipeWorkspace.beforeCheckout(WipeWorkspace.java:28)
	at hudson.plugins.git.GitSCM.checkout(GitSCM.java:1040)
	at hudson.scm.SCM.checkout(SCM.java:485)
	at hudson.model.AbstractProject.checkout(AbstractProject.java:1277)
	at hudson.model.AbstractBuild$AbstractBuildExecution.defaultCheckout(AbstractBuild.java:610)
	at jenkins.scm.SCMCheckoutStrategy.checkout(SCMCheckoutStrategy.java:86)
	at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:532)
	at hudson.model.Run.execute(Run.java:1741)
	at hudson.matrix.MatrixBuild.run(MatrixBuild.java:301)
	at hudson.model.ResourceController.execute(ResourceController.java:98)
	at hudson.model.Executor.run(Executor.java:408)
Caused by: java.nio.file.FileSystemException: c:\jenkins\workspace\gRPC_master\config\dbg\language\c\platform\windows: The process cannot access the file because it is being used by another process.

	at sun.nio.fs.WindowsException.translateToIOException(Unknown Source)
	at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source)
	at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source)
	at sun.nio.fs.WindowsFileSystemProvider.implDelete(Unknown Source)
	at sun.nio.fs.AbstractFileSystemProvider.delete(Unknown Source)

I tried to log into the instance but couldn't quite figure out what process is locking that file

I restarted the instance grpc-jenkins-windows-worker3 and looked like the problem goes away. But the issue could come back in the future.

The error only went away for 2 builds
https://grpc-testing.appspot.com/job/gRPC_master/10927/
https://grpc-testing.appspot.com/job/gRPC_master/10928/
The error starts happening again after
https://grpc-testing.appspot.com/job/gRPC_master/10929/

This thread seems relevant: https://issues.jenkins-ci.org/browse/JENKINS-19994. One suggestion in the middle of the thread is to disable the "Wipe out repository & force clone" options in the project configuration.

Seen this again today~

I don't know what was done but this seems to not happen anymore. When a build failed now it's most likely due to #3724.

Doesn't look like we're seeing this anymore. Closing this as part of buildpony bankruptcy.


[BUG] NPE in azure-storage-file-datalake-12.0.1

Describe the bug
NPE in azure-storage-file-datalake-12.0.1
Exception or Stack Trace
Caused by: java.lang.NullPointerException: text
	at java.base/java.util.Objects.requireNonNull(Objects.java:246) ~[na:na]
	at java.base/java.time.format.DateTimeFormatter.parse(DateTimeFormatter.java:1945) ~[na:na]
	at java.base/java.time.OffsetDateTime.parse(OffsetDateTime.java:402) ~[na:na]
	at com.azure.storage.file.datalake.Transforms.toPathItem(Transforms.java:258) ~[azure-storage-file-datalake-12.0.1.jar:na]

To Reproduce
List recursivly a datalake.
To be more specific, the path.getLastModified() return a null date which go to NPE while formatting at com.azure.storage.file.datalake.Transforms.toPathItem(Transforms.java:258).
Setup (please complete the following information):
Bug in version 12.0.1
Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

@ylemoigne Thank you for reporting this. @gapra-msft I think this is a known bug in the service, right? That it doesn't return the LMT when listing?

Hi @ylemoigne
Thank you for reporting this issue. I've created a PR to add a null check before parsing out the date.
You can expect this fix in an upcoming release.
Thanks


Triggering an update when data changes

Hi there,
Thanks a lot for this library! Just wondering if you have figured out how to trigger the graph to refresh when the data changes. It doesn't seem like the component reacts to any change.
Here's the code I have right now:

{$page.params.country}

<Plotly
	id="phaseout-graph"
	data={phaseout_data}
	layout={phaseout_layout}
	config={{ responsive: true }}
	bind:loaded
	{reloadPlot}
>
	Loading plotly..
</Plotly>

Any tips would be much appreciated!
Thanks

Nevermind, I figured it out. The issue was that the data itself wasn't being refreshed. The code below fixes that.

{$page.params.country}

<Plotly
	id="id"
	data={phaseout_data}
	layout={phaseout_layout}
	config={{ responsive: true }}
	bind:loaded
	{reloadPlot}
>
	Loading plotly..
</Plotly>

Thanks for the solution and for using the library! :)


Refactor Viewer to fit API Architecture

User story

As a product owner
I want to have the basic viewer structure
So that it matches the API documentation

Acceptance criteria

ViewerPanoAPI implements the view method and the scene property
ViewerViewState is implemented as described in the docu

Definition of done

Added only after week 5
The same for all features

As part of last weeks sprint, I had already created the ViewerViewState file in the correct location, and added the class with it's 3 variables. The class is also already imported in the main.js file. You can find the commit here.
However what still needs to be done, is ViewerViewState objects need to actually be used.
So certain parts of the implementation in the main.js need to be refactored.
For example the ´longitude´ and ´latitude´ variables should be gone, in exchange for a ViewerViewState object.
The same can be done with ´onPointerDownLon´ and -Lat.
Of course this means changing the code in other spots too, wherever the variables are used right now, it needs to be replaced with the correct use of a ViewerViewSate Object.


Add option to use suda automatically if needed

Fixes #7
EDIT:
This is kind of ugly but it works pretty good for me. Let me know what do you think, especially on those cabbreviations.

One more thing: I noticed the smart_read is not smart enough, it doesn't exclude directories for example. It handles truely unreadable files fine but from some reason, it makes normal, readable files appear as empty. I'm working on that now.

I've fixed the issue mentioned in my previous comment. One last thing I'm not totally sure of is about the new settings I've added in plugin/suda.vim:
I noticed that if I put for exmaple let g:suda_smart_read=1  in a file inside my &runtimepath's plugin/ directory, when plugin/suda.vim is loaded, exists(g:suda_smart_read) is false. But If I put my let g:suda_smart_read=1 in my init.vim it works fine. Perhaps we should use after/plugin/? BTW the same thing goes for suda#prefix as well.
Further more, maybe I should use get(g:,  instead? This will be more consistent with your way of reading settings..

Thanks for your help in making this feature a reality @lambdalisue. I'll mark the review above as resolved and push the changes accordingly.
Besides our discussed dilemmas above, I have one more UX issue I'd like to seek advice with you:
Besides this plugin, I have the plugin eunuch installed as well. I use it mostly for :Mkdir, :Rename and :Move. After experimenting with our new feature, I noticed that on there is a sort of collision with a command defined by eunuch (:W and :Wall). The problem is that from some reason I end up in Insert mode after every write. Additionally, the writes seem to be slow because they are wrapped in a function.
Anyway, I was thinking that maybe all of this write wrapper is not that necessary and maybe we can just make this whole feature a 1 setting thing and we'll replace buffers with their suda://s even if they are not writeable. In code, that means:
diff --git i/autoload/suda.vim w/autoload/suda.vim
index 350a44c..cc4765a 100644
--- i/autoload/suda.vim
+++ w/autoload/suda.vim
@@ -93,14 +93,6 @@ function! suda#read(expr, ...) abort range
   endtry
 endfunction

-function! suda#write_wrapper(write_cmd) abort
-  try
-    execute(a:write_cmd)
-  catch /^Vim\%((\a\+)\)\=:E45:/
-    execute(a:write_cmd . ' suda://%')
-  endtry
-endfunction
-
 function! suda#write(expr, ...) abort range
   let path = s:expand_expression(a:expr)
   let options = extend({
@@ -212,8 +204,8 @@ function! suda#BufEnter() abort
   if !empty(&buftype) || empty(bufname) || match(bufname, '^[a-z]\+://*') isnot# -1
     " Non file buffer
     return
-  elseif filereadable(bufname)
-    " File is readable
+  elseif filereadable(bufname) && filewritable(bufname)
+    " File is readable and writeable
     return
   elseif empty(getftype(bufname))
     " if file doesn't exist, we search for a all directories up it's path to
diff --git i/plugin/suda.vim w/plugin/suda.vim
index 2276acd..2e926fa 100644
--- i/plugin/suda.vim
+++ w/plugin/suda.vim
@@ -7,13 +7,6 @@ if get(g:, 'suda_startup', 1)
   call suda#init()
 endif

-if get(g:, 'suda_hijack_write')
-  cabbrev w SudaW
-  cabbrev wq SudaWq
-  command! SudaW call suda#write_wrapper('w')
-  command! SudaWq call suda#write_wrapper('wq')
-endif
-
 if get(g:, 'suda_smart_read')
   augroup suda_smart_read_primary
     autocmd!

After experimenting with our new feature, I noticed that on there is a sort of collision with a command defined by eunuch (:W and :Wall).

That's why I asked to add Suda prefix to our command. That's why I beleive that all plugins should NOT define this kind of alias. My advice for this is "Do NOT use tpope's plugins or do NOT use others' plugins 😇"

Anyway, I was thinking that maybe all of this write wrapper is not that necessary and maybe we can just make this whole feature a 1 setting thing and we'll replace buffers with their suda://s even if they are not writeable. In code, that means:

So suda will automatically use "suda://" prefix when

The file is not readable
The file is not writable

right? It sounds reasonable 👍

Oh right then, I think this PR is fully ready. I'm pushing the patch I've showed in my previous comment. There should be added some docs as well before this can be merged but I'm postponing this in the meantime because I might change my mind again as for some aspects of the behaviour as I'll be testing this in the next few following days..

There's one more edge case I'd like to discuss with you:
I noticed that with our current set of autocmds, if I open multiple unwriteable files with the new feature enabled, only one of them gets the suda:// prefix. This brings me back to our original discussion were I tend to think the BufNew approach, with an s: variable for caching checked buffers is better. For me, this method feels much more reassuring in it's technical design.. What do you think?

This brings me back to our original discussion were I tend to think the BufNew approach, with an s: variable for caching checked buffers is better. For me, this method feels much more reassuring in it's technical design.. What do you think?

Well, I believe that using s: is not the best design for this case. It seems simple but actually is complex. So I would love to try really hard to find a way without using s: approach.
Could you write step by step procedure for your case?

Could you write step by step procedure for your case?

Just open multiple unwriteable files in at once from the command line, e.g. nvim /etc/profile /etc/environment.

Just open multiple unwriteable files at once from the command line, e.g. nvim /etc/profile /etc/environment.

I got.
$ nvim ~/wk/unreadable ~/wk/unreadable2
BufEnter
VimEnter
:next
BufEnter

I'll think about this but now I feel cache style is required to handle this case with "user-friendly" interface...

I'll think about this but now I feel cache style is required to handle this case with "user-friendly" interface...

Please keep me updated. I'm changing the branch now to the way it was with only one autocmd and the dictionary cache based check, I'm not pushing it though. I'll keep you updated if I find multiple edge cases with that approach as well.

Oh and as for documentation, please see my last commit https://github.com/lambdalisue/suda.vim/pull/13/commits/5f0870a897393523c7f95a125820e7671f0b1452 .

Great work 🎉 Thanks to keeping the discussion 👍

It was a pleasure :)


Speeding up tests with --keepdb and --parallel

This PR introduces two improvements:

Use in-file database instead of in-memory one while running tests with --keepdb. This change lets us preserve database between test launches and, therefore, avoid the overhead of applying migrations during each test launch. Note that a new fresh in-memory database is still used by default, that is when you don't set --keepdb flag (i.e. python3 manage.py test).

TestCase.subTest cannot be used when tests are launched in parallel, that is with --parallel flag. To bypass this limitation, we use a dummy subTest mock when when tests are run in parallel.

The first improvement saves 30 seconds on my machine once the database is created. All test suite takes now 50 seconds instead of 80 seconds.
The second improvement in combination with the first one let you speed up tests twice (25 seconds instead of 50 seconds on my machine, my proc has 4 cores).

Unfortunately, squashing migrations doesn't make any difference. It still takes 30 seconds to apply migrations on my machine, even when first 87 migrations are squashed. So that was waste of time. :( See this branch: https://github.com/chrismedrela/amy/tree/squash-migrations
BTW, pro hint: if you get contenttype.name cannot be NULL IntegrityError when applying squashed migration, you need to add ('contenttypes', '0002_remove_content_type_name'), to Migration.dependencies.

Thanks - was the speed of tests a real obstacle to development?

@gvwilson, When I wanted to run one test, I had to wait half a minute for results. Now it runs in few seconds. This really slowed down writing tests, especially that I need to run each test a few times before I get it right.

@chrismedrela

TestCase.subTest cannot be used when tests are launched in parallel, that is with --parallel flag. To bypass this limitation, we use a dummy subTest mock when when tests are run in parallel.

Does that mean these sub tests are not run at all when using --parallel?

@pbanaszkiewicz, no, they are run, but if any of the subtest fails, other subtests are not checked and we go to the next test.

@chrismedrela please update this PR so that I can merge it

@chrismedrela ping

@pbanaszkiewicz pong


CupertinoPageRoute has no parallax effect when pushed on top of custom route

When using the CupertinoPageRoute to push a new route on top of a custom route the parallax effect is lost. If using MaterialPageRoute instead of CupertinoPageRoute, then the overlay is also lost.
This seems to be happening only when pushing on top of a custom route. Below is an example of my custom route that breaks following CupertinoPageRoute routes.
import 'package:flutter/material.dart';
import 'package:intui/utils/appearances.dart';

class BottomFadeRoute<T> extends PageRouteBuilder<T> {

  BottomFadeRoute({required BuildContext context, required Widget screen, RouteSettings? settings}) : super(
    opaque: false,
    barrierDismissible: true,
    barrierColor: Theme.of(context).primaryColor.withOpacity(0.5),
		transitionDuration: AppAnimation.defaultDuration,
    reverseTransitionDuration: AppAnimation.defaultReverseDuration,
		pageBuilder: (BuildContext context, _, __) {
      return screen;
    },
		transitionsBuilder: (_, Animation<double> animation, __, Widget child) {
      CurveTween curveTween;
      if (animation.status == AnimationStatus.reverse) {
        curveTween = CurveTween(curve: AppAnimation.defaultReverseCurve);
      } else {
        curveTween = CurveTween(curve: AppAnimation.defaultCurve);
      }
      final fadeTween = Tween<double>(begin: 0, end: 1).chain(curveTween);
      final slideTween = Tween<Offset>(begin: Offset(0, 0.4), end: Offset.zero).chain(curveTween);
			return FadeTransition(
				opacity: animation.drive(fadeTween),
				child: SlideTransition(
          position: animation.drive(slideTween),
          child: child,
        ),
			);
		},
    settings: settings,
	);

}

Hi @..., please provide a minimal, reproducible example and the output of flutter doctor -v.

Hi everyone, I noticed this problem too.
Here is a minimal example.
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Navigation Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: const FirstPage(),
    );
  }
}

class FirstPage extends StatelessWidget {
  const FirstPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('First Page')),
      body: Center(
        child: ElevatedButton(
          onPressed: () => Navigator.push(context, FadePageRoute(child: const SecondPage())),
          child: const Text('Push route'),
        ),
      ),
    );
  }
}

class SecondPage extends StatelessWidget {
  const SecondPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Second Page')),
      body: Center(
        child: ElevatedButton(
          onPressed: () => Navigator.push(context, MaterialPageRoute(builder: (context) => const ThirdPage())),
          child: const Text('Push route'),
        ),
      ),
    );
  }
}

class ThirdPage extends StatelessWidget {
  const ThirdPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(appBar: AppBar(title: const Text('Third page')));
  }
}

class FadePageRoute extends PageRouteBuilder {
  final Widget child;

  FadePageRoute({
    required this.child,
    RouteSettings? settings,
    Duration transitionDuration = const Duration(milliseconds: 300),
  }) : super(
          pageBuilder: (context, animation, secondaryAnimation) => child,
          settings: settings,
          transitionDuration: transitionDuration,
          reverseTransitionDuration: transitionDuration,
          transitionsBuilder: (context, animation, secondaryAnimation, child) => FadeTransition(
            opacity: animation,
            child: child,
          ),
        );
}

The missing parallax issue can be seen when navigating from SecondPage to ThirdPage.

Output of flutter doctor -v
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.5.1 21G83 darwin-arm, locale it-IT)
    • Flutter version 3.0.5 at /opt/homebrew/Caskroom/flutter/2.10.5/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (6 settimane fa), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[!] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/fabio/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] IntelliJ IDEA Ultimate Edition (version 2022.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 69.0.5
    • Dart plugin version 222.3739.24

[✓] VS Code (version 1.70.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.46.0

[✓] Connected device (3 available)
    [redacted]

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

I can reproduce this issue on stable and master

Video
https://user-images.githubusercontent.com/50555895/186600381-4daa5748-c4a4-4c24-9c40-906fcfa429e0.mov

Code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Navigation Demo',
      theme: ThemeData(primarySwatch: Colors.blue),
      home: const FirstPage(),
    );
  }
}

class FirstPage extends StatelessWidget {
  const FirstPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('First Page')),
      body: Center(
        child: ElevatedButton(
          onPressed: () => Navigator.push(
            context,
            FadePageRoute(child: const SecondPage()),
          ),
          child: const Text('Push route'),
        ),
      ),
    );
  }
}

class SecondPage extends StatelessWidget {
  const SecondPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Second Page')),
      body: Center(
        child: ElevatedButton(
          onPressed: () => Navigator.push(
            context,
            MaterialPageRoute(builder: (context) => const ThirdPage()),
          ),
          child: const Text('Push route'),
        ),
      ),
    );
  }
}

class ThirdPage extends StatelessWidget {
  const ThirdPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: const Text('Third Page')),
      body: Center(
        child: ElevatedButton(
          onPressed: () => Navigator.push(
            context,
            MaterialPageRoute(builder: (context) => const FourthPage()),
          ),
          child: const Text('Push route'),
        ),
      ),
    );
  }
}

class FourthPage extends StatelessWidget {
  const FourthPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(appBar: AppBar(title: const Text('Fourth Page')));
  }
}

class FadePageRoute extends PageRouteBuilder {
  final Widget child;

  FadePageRoute({
    required this.child,
    RouteSettings? settings,
    Duration transitionDuration = const Duration(milliseconds: 300),
  }) : super(
          pageBuilder: (context, animation, secondaryAnimation) => child,
          settings: settings,
          transitionDuration: transitionDuration,
          reverseTransitionDuration: transitionDuration,
          transitionsBuilder: (context, animation, secondaryAnimation, child) =>
              FadeTransition(
            opacity: animation,
            child: child,
          ),
        );
}

flutter doctor -v (Stable)
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.5.1 21G83 darwin-arm, locale en-US)
    • Flutter version 3.0.5 at /Users/nabeelparkar/fvm/versions/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (6 weeks ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nabeelparkar/Library/Android/sdk/
    • Platform android-33, build-tools 33.0.0
    • ANDROID_SDK_ROOT = /Users/nabeelparkar/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Brave Browser.app/Contents/MacOS/Brave Browser

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.1.3)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 69.0.4
    • Dart plugin version 221.5921.27

[✓] VS Code (version 1.69.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.44.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.5.1 21G83 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Brave Browser <IP_ADDRESS>

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

flutter doctor -v (Master)
[✓] Flutter (Channel master, 3.1.0-0.0.pre.2480, on macOS 12.5.1 21G83 darwin-arm, locale en-US)
    • Flutter version 3.1.0-0.0.pre.2480 on channel master at /Users/nabeelparkar/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a1e8d5d680 (2 hours ago), 2022-08-25 00:23:18 -0400
    • Engine revision 2ad79bea5b
    • Dart version 2.19.0 (build 2.19.0-135.0.dev)
    • DevTools version 2.16.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/nabeelparkar/Library/Android/sdk/
    • Platform android-33, build-tools 33.0.0
    • ANDROID_SDK_ROOT = /Users/nabeelparkar/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F100
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Brave Browser.app/Contents/MacOS/Brave Browser

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.1.3)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 69.0.4
    • Dart plugin version 221.5921.27

[✓] VS Code (version 1.69.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.44.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.5.1 21G83 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Brave Browser <IP_ADDRESS>

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!


Add Scoop installation method to the README

usql was recently added to Scoop's main bucket, which makes it easier/faster to install usql on Windows.

I appreciate the contribution. I don't really use Windows, how does scoop compare to chocolately? Originally I had intentions of writing the .choco script, but it was a giant pain, and more trouble than it was worth.

I don't really use Windows, how does scoop compare to chocolately?

See Chocolatey Comparison on the Scoop wiki :smiley:

Here is a chocolatey package:

https://chocolatey.org/packages/usql

It is automatic package that  will follow your updates here within single day


Name for grass loot pool

This PR adds a name to the added grass loot pool for compatibility with other mods to avoid empty loot tables.
https://github.com/enemeez/simplefarming/issues/68

Thank you for the contribution!


⚠ SceneTime has degraded performance

In 57dc157, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1038 ms

Resolved: SceneTime performance has improved in 435ea8b after 6 minutes.


[Question] java jar source code

Do you agree to expose the code of the library (UniAndroidPermission/Assets/UniAndroidPermission/uniAndroidPermission.jar) ?
Thank, Rémi.

Hi RemiLatapy,
I added a AndroidStudioProject folder at root. It's for create jar file.
Thank you for pointing it out.
Sanuki.

Hey!
Thank a lot!


Remove options from navbar

Currently the Navbar is looking quite clumsy due to so many options where the first 3 , Aboutus ,Services Reviews are non functional so can I remove them and rearrange the existing options
Participant under gssoc'22

Looks a lot cleaner now.

@ayush25102001 sounds really interesting
Could you please share how are you planning to rearrange the navbar items?


integration/inspektor-gadget: Clean up log files after NetworkPolicy test

Clean up log files after NetworkPolicy test
Avoid leaving log files after running the IG integration tests.

I committed these files couple of times already so thanks for it! smile

In this case, should we add *.log to .gitignore to prevent this problem to occur?

Yes, that works for me as well since I am more concerned about committing the files accidentally.

I committed these files couple of times already so thanks for it! smile

In this case, should we add *.log to .gitignore to prevent this problem to occur?

Yes, that works for me as well since I am more concerned about committing the files accidentally.

Yep, that would avoid committing them accidentally, but I would definitely prefer to leave things as they were before running the tests.


Standardize use of KBD tag on docs in docs.msft

Provide a description of requested docs changes
https://docs.microsoft.com/en-us/windows/powertoys/ has a bunch of different styles for keys.  We should standardize on KBD after the doc update from @dedavis in .41

That specific page looks fine. I assume you mean to check all the pages?

[ ] https://docs.microsoft.com/en-us/windows/powertoys/install
[ ] https://docs.microsoft.com/en-us/windows/powertoys/fancyzones
[ ] https://docs.microsoft.com/en-us/windows/powertoys/shortcut-guide

A different, small thing, but not 100% consistent:

"Next, open Windows File Explorer, select the View tab in the File Explorer ribbon, then select Preview Pane."

on https://docs.microsoft.com/en-us/windows/powertoys/file-explorer

I got two more ideas i could change, but i want to wait until after this pr is accepted.

in stead of using bullets in front of key/shortcut examples, I want to use a table, to match the appearance in the UI
I used "quotes" to show keyboard output (in contrast to kbd for physical input) but now that I think about it, this output is data just as well, so I think it would be better if I used data.


Record query row

Hi! I think would be very useful save a row that represents which line number is being running from the queries, like an id for the queries.
Thx!

hi @latot
this is already supported.

modify your input file to create a custom id according to the above format.
So your keywords should be like:
keyword1 #!#1
keyword2 #!#2

and the results will have this as an id.
let me know if that works for you
best

great thx! I didn't found! was in the docs all the time.
I'll reopen if I find something!


Support setting internaldate on IMAP appends.

I'm not too sure why this got overlooked before, but appendMessage() previously didn't allow the internaldate to be set, although Zend\Mail\Protocol\Imap\append() exposes that functionality.  This PR simply allows the internaldate to be set.

Need tests

Also needs a rebase before the next push (ping @ameir ).

@weierophinney I've rebased, as requested.  Let me know if you need any further changes to be made.

Closing; author has not responded to feedback or requests for changes.
If you would like to re-open, please address the feedback and re-submit.
Thanks!


Exception on boot that "entity id already exists for automation" is confusing

Home Assistant release with the issue:

0.82.1
Last working Home Assistant release (if known):
Not known
Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io
Component/platform:

entity
Description of problem:
When your configuration file contains entities with a duplicate alias, the error message on boot doesn't list the actual text of the duplicate alias.  This makes it impossible to search for using a text search.
In particular, in my example, it reported:
homeassistant.exceptions.HomeAssistantError: Entity id already exists: automation.secure_house_for_the_night
However, the actual problem was that the aliases were duplicate which had the name "Secure house for the night".
The desired behavior in this scenario would to be to report the alias name "Secure house for the night" (with spaces) as the problem.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
automation.yaml:
- id: automate_secure_house_for_night
  alias: 'Secure house for the night'
  trigger:
    platform: time
    at: '23:00:00'
  condition:
    condition: template
    value_template: '{{ now().month != 10 }}'
  action:
    - service: switch.turn_off
      entity_id: switch.garage_light_switch
    - service: scene.turn_on
      entity_id: scene.secure_house

- id: automate_secure_house_for_night_halloween
  alias: 'Secure house for the night'
  trigger:
    platform: time
    at: '23:00:00'
  condition:
    condition: template
    value_template: '{{ now().month == 10 }}'
  action:
    - service: switch.turn_off
      entity_id: switch.garage_light_switch
    - service: scene.turn_on
      entity_id: scene.secure_house_halloween

Traceback (if applicable):
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/homeassistant/helpers/entity_platform.py", line 346, in _async_add_entity
    msg)
homeassistant.exceptions.HomeAssistantError: Entity id already exists: automation.secure_house_for_the_night

Additional information:

As an end user, I don't have anything with the name: secure_house_for_the_night in any of my configuration files.  This means that I don't have an obvious way to track down the problem.
Additionally I have something called "id" that I assumed was the entity_id.  (I don't know what this id is for?)
I know that I personally wasted an embarrassing amount of time before I realized that the problem was with my alias that was getting transformed into a name with underscores.
I recognize that people familiar with how the home assistant code works may not have this problem.

As an end user, I don't have anything with the name: secure_house_for_the_night
All entity_id's replace spaces with underscores. If you call a light "Living room" it will be light.living_room for the entity_id.
Additionally I have something called "id" that I assumed was the entity_id. (I don't know what this id is for?)
I have no idea why you put that in if you're manually writing automations. The id key is for automations created/edited with the gui editor so it knows which one it's editing. It's not mentioned in the docs for manually creating automations at all.

Your example should be just one automation anyway...
- alias: 'Secure house for the night'
  trigger:
    platform: time
    at: '23:00:00
  action:
    - service: switch.turn_off
      entity_id: switch.garage_light_switch
    - service: scene.turn_on
      data_template:
        entity_id: >
          {% if now().month != 10 %} scene.secure_house
          {% else %} scene.secure_house_halloween {% endif %}

As an end user, I don't have anything with the name: secure_house_for_the_night
Additionally I have something called "id" that I assumed was the entity_id. (I don't know what this id is for?)
I have no idea why you put that in if you're manually writing automations. The id key is for automations created/edited with the gui editor so it knows which one it's editing. It's not mentioned in the docs for manually creating automations at all.

Thanks, that's good to know.  If I recall correctly, I tried to use the automation editor at one point, then found that it wasn't displaying some of the logic in my automations so I reverted to just editing the text files and preserved using the ids (that I imagine the editor had added).


fix: update project_options to v0.15.1

Codecov Report

Merging #192 (58caa1c) into main (ae04522) will decrease coverage by 41.53%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #192       +/-   ##
===========================================
- Coverage   80.00%   38.46%   -41.54%
===========================================
  Files           3        3
  Lines          35       26        -9
  Branches       18       18
===========================================
- Hits           28       10       -18
  Misses          6        6
- Partials        1       10        +9

Flag
Coverage Δ

Linux
38.46% <ø> (ø)

Windows
?

macOS
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

test/tests.cpp
50.00% <0.00%> (-50.00%)
:arrow_down:

test/constexpr_tests.cpp
50.00% <0.00%> (-50.00%)
:arrow_down:

src/main.cpp
20.00% <0.00%> (-26.16%)
:arrow_down:

Continue to review full report at Codecov.

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


Allow unregistering any package using emailed one time token

Allow to unregister any package by adding an option to email a one-time short-lived token to author(s) mentioned in package's bower.json, which they can use in existing CLI,
That is, check the token not only against github oauth, but also against some temporary store (an in-memory bounded-size queue) of [token, project] pairs..

Pretty much same as https://github.com/bower/bower/issues/2167 & ooold.
Closing.


Unicode decoding problems

Hey, I'm using OpenNMT with unicode languages and It works just fine until I send a special character such as: á. Then I get an encoding error. OpenNMT doesn't supports unicode or am I encoding it wrong?. Here's the stdout and stderror logs and the sample code I used to test. I am using the master branch, up to date.
==> /var/log/opennmt.out.log <==
[02/13/17 18:24:42 INFO] Loading model
[02/13/17 18:24:42 INFO] Using GPU(s): 1
[02/13/17 18:24:42 INFO] Loading '/home/ubuntu/my_model.t7'...
[02/13/17 18:24:45 INFO] Server initialized at tcp://<IP_ADDRESS>:8081
[02/13/17 18:25:00 INFO] Received... [{"src": "ol\u00e1"}]

==> /var/log/opennmt.err.log <==
/home/ubuntu/torch/install/bin/luajit: /home/ubuntu/torch/install/share/lua/5.1/json/json.lua:309: Failed to load string [ return "ol\u00e1"] in JSON4Lua.decode_scanString at position 10 : 20
stack traceback:
        [C]: in function 'assert'
        /home/ubuntu/torch/install/share/lua/5.1/json/json.lua:309: in function 'decode'
        /home/ubuntu/torch/install/share/lua/5.1/json/json.lua:270: in function 'decode'
        /home/ubuntu/torch/install/share/lua/5.1/json/json.lua:181: in function 'decode'
        tools/translation_server.lua:110: in function 'main'
        tools/translation_server.lua:130: in main chunk
        [C]: in function 'dofile'
        ...untu/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
        [C]: at 0x00406670

Sample code:
import zmq, sys, json
sock = zmq.Context().socket(zmq.REQ)
sock.connect("localhost:8081")
sock.send(json.dumps([{"src": "olá"}]))

I also tried the following, got the same problem
sock.send(json.dumps([{"src": "ol\u00e1"}]))
sock.send(json.dumps([{"src": u"olá"}]))
sock.send(json.dumps([{"src": u"ol\u00e1"}]))

Is this happening only using the zmq server or also while decoding with translate.lua ?

only zmq

It seems the json module that is used does not handle escaped unicode sequence. A workaround:
sock.send(json.dumps([{"src": "olá"}], ensure_ascii=False))

The real fix would be to use a JSON library that fully implements the JSON specifications.

Yeah, that workaround did the trick. I had some issues with the results, but it seems that my model's wrong, nothing to worry about.
Anyway, this should be closed after someone changes the json library, right?

ty!


Add UAC bypass to gain elevated prompt

This technique use the taskmgr trick to bypass the UAC and gain an a process executed with High Integrity level  !
Example with and without the option "privileged" :

This was added to latest version. You were credited in CHANGELOG.


Update CODEOWNERS for every mgmt package

As discussed in https://github.com/Azure/azure-sdk-tools/issues/1099, usage of glob patterns does not work for our bot. So adding codeowner for each of the mgmt plane package. I copied the list from https://github.com/Azure/azure-sdk-for-js/blob/feature/v4/eng/pipelines/mgmt-ci.yml

@ramya-rao-a  hey, just out of curious why remove this pattern match/sdk/**/arm-*/ @qiaozha @dw511214992

@ramya-rao-a hey, just out of curious why bot doesn't support this pattern match/sdk/**/arm-*/ @qiaozha @dw511214992

I see, the details from the azure sdk tools issue now. Will this be a issue in feature/v4 branch as well ?

Will this be a issue in feature/v4 branch as well

Yes, I made this PR for that branch: https://github.com/Azure/azure-sdk-for-js/pull/17061
Unfortunately the auto merge bot does not wait for PR approval if the PR is targeting a feature branch.
I have logged a separate issue for that https://github.com/Azure/azure-sdk-tools/issues/1933

/check-enforcer override


Error: must specify version

Dear @SimulatedGREG
When I build with
npm run build:darwin

one error occurs.
Building electron app(s)...

Error: must specify version
    at download (/Users/wxl/projects/bg/node_modules/electron-download/index.js:16:27)
    at Array.<anonymous> (/Users/wxl/projects/bg/node_modules/electron-packager/index.js:121:7)
    at each (/Users/wxl/projects/bg/node_modules/run-series/index.js:17:24)
    at next (/Users/wxl/projects/bg/node_modules/rimraf/rimraf.js:74:7)
    at CB (/Users/wxl/projects/bg/node_modules/rimraf/rimraf.js:110:9)
    at /Users/wxl/projects/bg/node_modules/rimraf/rimraf.js:136:14
    at FSReqWrap.oncomplete (fs.js:123:15)
Build(s) successful!
undefined

DONE

I think there is a problem with config.js, because when I change
building:{
  'app-version': pkg.version,
  ......
}

to
building:{
  version: '1.3.2', // pkg.version,
  ......
}

which is the version of electron-prebuilt. It works!!!

The default setup uses the version number from ./app/package.json. Can you confirm that is a proper semver? Do you have this in a repo I could look into as well?

When I only change "version" in package.json to "version": "1.1.1", it report error also.
When I add a new term version: '1.2.1'into building:{ }  in config.js, it download electron-v1.2.1 during the building process. So I think we have to specify the version of electron-prebuild.

@wuxianliang
Found a pull request with electron-packager that sort of broke getting the electron version automatically (it would seem), which is live in v7.5.0. For now, you can continue to manually set the electron version (like you demonstrated above), or you can install the previous version of electron-packager by doing npm i --save-dev<EMAIL_ADDRESS>Here's the PR...
https://github.com/electron-userland/electron-packager/pull/435#issuecomment-238036368

@wuxianliang
Did some further investigation and found what broke in the new update. (You discovered something kinda nasty for them in my opinion... 😁)

@wuxianliang
Can confirm a fix is being released rather soon in<EMAIL_ADDRESS>Closing this now.

<EMAIL_ADDRESS>did not fix the issue.
https://github.com/electron-userland/electron-packager/pull/440#issuecomment-238043603

Fixed with<EMAIL_ADDRESS>


Series to be downloaded and today

It will be great if you can add to Sonarr option to see aired today episodes and are they downloaded or no.

While it doesn't seem realistic to add this to the exportarr series, it is easy to add JSON data directly to Grafana without an exporter and prometheus, and build it into a dashboard.
I've decided to fiddle with a direct JSON dashboard for Servarr products, starting with this feature for Sonarr. Bare minimum information is present to get it running. Progress will probably be slow.
https://github.com/codefaux/grafana-dashboard-sonarr

Yea, +1 -- prometheus in general is meant for metrics, rather than text output. a
More info on a JSON datasource here:
https://grafana.com/grafana/plugins/simpod-json-datasource/


Upgrade to golangci-lint from gometalinter

Problem: gometallinter has been deprecated
Solution: Switch to gometalinter and enable to least number of linters
to be comparable to gometalinter.
See https://github.com/rancher/rancher/issues/18638

Per our discussion this pr should just get the new tool running using our current list of linters. Once this is merged we can look at turning on additional linters and fixing issues they create.

LGTM

Reverted by c0e17880540a115151c5edb8712e06dea1906da4


Fatal error. System.AccessViolationException

When using:
            string[] labels = { "0", "1", "2", "3", "4", "5", "6", "7", "8", "9" };

            using var scorer = new DecoderScorer(labels);
            var decoder = new CTCDecoder();
            var result = decoder.Decode(arrayResult, scorer);

"Fatal error. System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt."
1.0.4
VS22
Windows 10

Hi @Vippzz ,
What is arrayResult in your sample code? Could you provide this?


LBRY desktop auto-sets itself as default application for opening html files in Linux file managers like nemo after logout and login or a cold/warm boot

Bug
When opening an HTML file in a Linux File Manager like nemo, LBRY is launched instead of the default browser or application set to open HTML files. The defaults can be reverted in an HTML file properties but after a logout and login or a cold/warm boot, LBRY desktop sets itself as the default application for opening HTML files. Reverting to the default browser only lasts as long as one's PC is on and is no logout has occurred.
To Reproduce
Steps to reproduce the behaviour:

Download and install the lbry-desktop deb package and install it in any Debian/Ubuntu based OS.
Login to lbry via the app. (I am not sure if this contributes to the cause of the bug).
Logout and log back in or perform a warm/cold boot.
Create an HTML file with sample HTML code and launch/open it in the File Manager or launch/open an available HTML file in a File Manager
If lbry-desktop is launched instead of the default browser, revert to the default browser via the file properties and launch the file.
Logout and log back in or perform a warm/cold boot.
Launch the HTML file in a File Manager.

Expected behaviour
The default browser or applications like editors or IDEs should be used to launch/open an HTML file instead of lbry-desktop which currently auto-sets itself as the default application for launching/opening an HTML file after a cold/warm boot or a logout and login.
Screenshots
Below are the screenshots of the default web browser set

and when right clicking an html file in nemo (Cinnamon DE File Manager) and opens with LBRY by default.

System Configuration

LBRY Daemon version: 0.79.1
LBRY App version: 0.47.2
LBRY Installation ID: 5nMz3Kv8dG4XhXqzeWBNt4dnYMYQpbFhWEuMQnhgkQxBsVuSRHQCwfwqLvGn6ru34N
Operating system: Linux Mint 20 Cinnamon (Ubuntu 20.04 Base)

Full output of: curl 'http://localhost:5279' --data '{"method":"version"}':
{   "jsonrpc": "2.0",   "result": {     "build": "release",     "desktop": "X-Cinnamon",     "distro": {       "codename": "ulyana",       "id": "linuxmint",       "like": "ubuntu",       "version": "20",       "version_parts": {         "build_number": "",         "major": "20",         "minor": ""       }     },     "lbrynet_version": "0.79.1",     "os_release": "5.4.0-45-generic",     "os_system": "Linux",     "platform": "Linux-5.4.0-45-generic-x86_64-with-LinuxMint-20-ulyana",     "processor": "x86_64",     "python_version": "3.7.8",     "version": "0.79.1"   } }

Thanks for opening the issue. This is a strange bug with the underlying packaging we're using but it's not supposed to happen in our case. Hope it's fixed once we upgrade some of these components, and if not, we'll have to dig into it more: https://github.com/signalapp/Signal-Desktop/issues/3602

Same issue. So frustrating as a developer that I removed LBRY for now.

Also removed lbry-desktop and will reinstall when the issue has been fixed

Thanks for the feedback guys, we'll take a closer look at this soon!

Noted that Electron was upgraded to V9 in Version 0.48.1 but the issue is still not fixed

I can confirm, it still sets as default, but other Electron apps do not do this.

@jessopb any thoughts on why this is still happening or if it can be fixed by updating dependencies?

@jessopb any thoughts on why this is still happening or if it can be fixed by updating dependencies?

Just my own reply, I fixed it by simply removing LBRY and set my file-associations back to what they were before LBRY. I prevented LBRY from violating my file-associations by simply never installing it again.

I looked into this 2 weeks ago and tried other things, and no luck. This is a bug on xdg-utils https://github.com/balena-io/etcher/issues/3374#issuecomment-739558436

We can remove the LBRY protocol association for now on Linux until it's fixed.

I use the following work-around to prevent LBRY from becoming the default-handler:
I execute sudo chattr +i ~/.config/mimeapps.list when LBRY is currently not the default.
This prevents any changes to the defaults, but since mine don't change that often, I can manually do sudo chattr -i ~/.config/mimeapps.list to reverse it every time I want to change a default.
Simply removing the write permission to that file does not work for me, so I'm setting the immutable bit which is only changeable by root


Update Detekt to version 1.18.0+

There was a problem in #123 PR with the Detekt upgrade to version 1.18.0. Because of type resolution, many rules cannot work without --classpath, where the path to all the libraries used in the solution is required.
Most likely, when we start the tool, we must pass the path to the folder with the necessary libraries.

@vyahhi can we gather all the necessary libraries? Currently, we don't support type resolution since we don't have a list of the libraries


[Feature Request] Firebase AuthUI not allowing account linking between Mobile Phone auth and other providers

At the moment there is no way to perform account linking between mobile number accounts and other provider type accounts. I understand that it might be connected with the way Firebase Auth stores user details, specifically identifiers. Would it be possible to add additional optional identifiers to account by which the linking could be performed?
For example:

The user signs up using mobile phone and the account is created in the Firebase Authentication - Users with identifier +44 12345 678908 and some UID.

Then, in the application flow I could ask the user to provide e-mail, which I would then update with the code:

val user = FirebaseAuth.getInstance().currentUser
val profileUpdates = UserProfileChangeRequest.Builder()
.setDisplayName("Jane Q. User")
.setPhotoUri(Uri.parse("https://example.com/jane-q-user/profile.jpg"))
<EMAIL_ADDRESS>.build()
user?.updateProfile(profileUpdates)
?.addOnCompleteListener { task ->
if (task.isSuccessful) {
Log.d(TAG, "User profile updated.")
}
}

Once the user tries to sign in using for example Facebook provider, two accounts can be automatically linked.

One problem with this is that e-mail address would have to be verified first somehow to prevent following scenario:

Jane Q signs in using mobile phone and provides optional e-mail identifier belonging to Joe Schmoe (joe.schmoe@email.com).
Joe Schmoe not having any account signs in using his Facebook account which provides e-mail<EMAIL_ADDRESS>Joe Schmoe Facebook account is then linked with Jane Q mobile phone account.

This is just example of how this could work. Would it be possible to implement mobile phone account linking with other providers by this or any other means?

@Alimiel thanks for the feature request!  Unfortunately there is no way to make this work with Firebase Authentication.  Email and Phone Number are mutually exclusive unique identifiers for the account.  There is no way to say "this account is the unique owner of<PHONE_NUMBER> and<EMAIL_ADDRESS>cc @lsirac who is on the Firebase Authentication team and relay this feature request.  But for now, nothing we can do in FirebaseUI.


python312Packages.knx-frontend: 2<IP_ADDRESS>106 -> 2<IP_ADDRESS>307

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.knx-frontend is: Home Assistant Panel for managing the KNX integration
meta.homepage for python312Packages.knx-frontend is: https://github.com/XKNX/knx-frontend
meta.changelog for python312Packages.knx-frontend is: https://github.com/XKNX/knx-frontend/releases/tag/2<IP_ADDRESS>307
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2<IP_ADDRESS>307 with grep in /nix/store/am7br8ihd6kxicr8zahpqklzxm5s225f-python3.12-knx-frontend-2<IP_ADDRESS>307
found 2<IP_ADDRESS>307 in filename of file in /nix/store/am7br8ihd6kxicr8zahpqklzxm5s225f-python3.12-knx-frontend-2<IP_ADDRESS>307

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

home-assistant-component-tests.knx
python311Packages.knx-frontend
python312Packages.knx-frontend

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/am7br8ihd6kxicr8zahpqklzxm5s225f-python3.12-knx-frontend-2<IP_ADDRESS>307 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.knx-frontend https://github.com/r-ryantm/nixpkgs/archive/e03083764dce827151c33b121736d417f56986cd.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e03083764dce827151c33b121736d417f56986cd#python312Packages.knx-frontend

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/am7br8ihd6kxicr8zahpqklzxm5s225f-python3.12-knx-frontend-2<IP_ADDRESS>307
ls -la /nix/store/am7br8ihd6kxicr8zahpqklzxm5s225f-python3.12-knx-frontend-2<IP_ADDRESS>307/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.knx
python311Packages.knx-frontend
python311Packages.knx-frontend.dist
python312Packages.knx-frontend
python312Packages.knx-frontend.dist

Maintainer pings
cc @mweinelt for testing.

#333095


[13.4-stable] better isolation for eve user apps

Backport of https://github.com/lf-edge/eve/pull/4304

@OhmSpectator let's wait for the Eden tests to finish

yeah, sure... If they work =(


Addressed issue 152 in main repo.

Added ability to select default tab

More detail
Added a ListPreference to act in the settings page
and in the main activity worked with the navGraph to programmatically set the startDestination
Issue #152

@mueller-ma let me know what you think <3

@mueller-ma ok done all the requested changes, are they ok?

@Kardelio In addition to my review can you replace all user-facing and internal strings from "default tab" to "start page"?

@mueller-ma also i think the Github Actions build needs to be re-run


OLS-779: Bump-up OpenAI to version 1.35.7

Description
Bump-up OpenAI to version 1.35.7
(that library seems to progress very quickly, but the newest version are not compat with other packages ATM)
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[x] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Related Tickets & Documents

Related Issue #OLS-779

/approve

/override "Red Hat Konflux / lightspeed-service-on-pull-request"

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

Project coverage is 95.42%. Comparing base (587fa12) to head (0339951).
Report is 9 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1212      +/-   ##
==========================================
- Coverage   95.66%   95.42%   -0.25%
==========================================
  Files          56       56
  Lines        2586     2381     -205
==========================================
- Hits         2474     2272     -202
+ Misses        112      109       -3

see 7 files with indirect coverage changes