character represented by universal-character-name 'char' cannot be represented in the current code page

    if (ret >= 256)
    {
        buffer_to_use = new char[ret + 1];
        snprintf(buffer_to_use, 256, "%s \u002A %-7s \u002A %s\n", time_buffer, Level, Log);
        buffer_to_use[ret] = '\0';
    }

Program won't compile with the \u00B3 UTF-8 character. I also tried L\u00B3 and u8\u00B3. Any ideas? It works if I use another char, for example * > \u002A.
Also side question, I have noticed that the Loader.exe in your Release v<IP_ADDRESS>-main package is sitting at a whopping 147MB and the folder only contains two files, Loader.exe and Loader.pdb.
My build compiles as following, but seems to work just fine. Is it safe to assume that you are compiling the dependences directly into the executable file? Could you walk me through the process?
15. 09. 2021  09:26               414 Loader.deps.json
15. 09. 2021  08:28           740.352 Loader.dll
15. 09. 2021  08:28           311.296 Loader.exe
15. 09. 2021  08:28            19.972 Loader.pdb
15. 09. 2021  09:26               265 Loader.runtimeconfig.dev.json
15. 09. 2021  09:26               154 Loader.runtimeconfig.json
15. 09. 2021  09:28    <DIR>          ref
15. 09. 2021  08:28           198.144 Loader.dll

Interesting, I'm not sure about that, what codepage does your system run on? You could just replace it with a | if needed, its only using the box-drawing line character to avoid gaps.
As for the large exe, it's because its generated as a standalone exe. Under .NET 5.0 this causes it to package parts of the .NET runtime with the exe rather than the user having to install a runtime seperately. This may or may not be the best way to handle this, .NET development is not really my area, but it seemed like the best choice for compatibility when I was setting up the release build scripts. I'm open to better suggestions.
You can see how the packaging is done here, the "Package DotNet" is the one you would be interested in.
https://github.com/TLeonardUK/ds3os/blob/main/.github/workflows/release.yml

Also as an aside. There is an option when packaging .NET applications to trim the executable by removing methods in the packed assemblies that are not actively used by the application. I experimented a bit with this before and it reduced the size by about half, but unfortunately it also made the executables crash almost immediately. Might be something to investigate further.

Also side question, I have noticed that the Loader.exe in your Release v<IP_ADDRESS>-main package is sitting at a whopping 147MB and the folder only contains two files, Loader.exe and Loader.pdb.
My build compiles as following, but seems to work just fine. Is it safe to assume that you are compiling the dependences directly into the executable file? Could you walk me through the process?

To compile a singel exe file u need to change some settings in the .csproj:
For a Single exe file with no build in runtime (needs .Net 5.0 installed on OS) (~ 1 MB):
add under  PropertyGroup the following statments
<RuntimeIdentifier>win-x64</RuntimeIdentifier>
<SelfContained>False</SelfContained>
<PublishSingleFile>true</PublishSingleFile>
<IncludeAllContentForSelfExtract>true</IncludeAllContentForSelfExtract>

For a Single exe file with build in runtime (~ 150 MB untrimmed , ~70 trimmed):
add under  PropertyGroup the following statments
<RuntimeIdentifier>win-x64</RuntimeIdentifier>
<SelfContained>true</SelfContained>
<IncludeAllContentForSelfExtract>true</IncludeAllContentForSelfExtract>
<PublishSingleFile>true</PublishSingleFile>
<PublishTrimmed>true</PublishTrimmed>
<TrimMode>Link</TrimMode>

U can delete PublishTrimmed and TrimMode if u want to go untrimmed, sometimes its better, it can cause weird behavior.
Since u mention the big size of the Loader I would suggest to just compile it with no build in runtime and make sure u have .Net 5.0 installed on your OS.
Note:
to compile the publish settings u need to compile it via a publish command like the CLI version donet publish

Interesting, I'm not sure about that, what codepage does your system run on? You could just replace it with a | if needed, its only using the box-drawing line character to avoid gaps.
As for the large exe, it's because its generated as a standalone exe. Under .NET 5.0 this causes it to package parts of the .NET runtime with the exe rather than the user having to install a runtime seperately. This may or may not be the best way to handle this, .NET development is not really my area, but it seemed like the best choice for compatibility when I was setting up the release build scripts. I'm open to better suggestions.
You can see how the packaging is done here, the "Package DotNet" is the one you would be interested in.
https://github.com/TLeonardUK/ds3os/blob/main/.github/workflows/release.yml

I'm running on 852 | Slavic (Latin II)

Try with latest, I switched those characters for the hex utf-8 chars. From what I read on the internet, that should solve this.

Works perfectly now.


botocore.exceptions.ClientError: An error occurred (InvalidClientTokenId) when calling the AssumeRole operation: The security token included in the request is invalid.

I am having an issue with authentication. This is my code:
from sp_api.api import Orders
from sp_api.api import Reports
from sp_api.api import Feeds
from sp_api.base import SellingApiException
from sp_api.base.reportTypes import ReportType
from datetime import datetime, timedelta

credentials = dict(
    refresh_token='Atzr|XXX,
    lwa_app_id='amzn1.application-oa2-client.XXXX',
    lwa_client_secret='9287bXXX',
    aws_secret_key='AKXXX',
    aws_access_key='BAWXXX',
    role_arn='arn:aws:iam::45XXX:role/XXX'
)

try:
    res = Orders(credentials=credentials).get_orders(CreatedAfter=(
        datetime.utcnow() - timedelta(days=7)).isoformat())
    print(res.payload)  # json data
except SellingApiException as ex:
    print(ex)

And I get the error
botocore.exceptions.ClientError: An error occurred (InvalidClientTokenId) when calling the AssumeRole operation: The security token included in the request is invalid.
I believe I followed through the steps that if said on the github, so please let me know if there is a solution.

There are multiple threads about Authentication problems in the discussions.

@saleweaver I am getting the same error can you please let me know what is the issue and which threads this issue discussion?

i'm having the same issue, how have you solved this problem please?


Local Clipboard Corruption on Linux when Non-Text Items are Copied

Problem Description:
When starting Parsec on a Linux system, the local clipboard is corrupted when placing images, files or other items other than text. This issue results in the loss of data and makes it difficult for users to transfer data between different applications.
Steps to Reproduce:

Start Parsec on a Linux system.
Copy an image or any other non-text item to the clipboard.
Attempt to paste the copied item into another application.

Expected behaviour:
The copied item should be pasted as expected, without loss of data.
Actual behaviour:
The pasted item is replaced by the last copied text, and the image/file/... is lost.
Environment:

Operating System: All X11 system
Parsec version: All
libmatoya version: All

Additional Information:
The problem comes from the X11 stock api which has no event on clipboard modifications.
The libmatoya library, to work around this lack, retrieves the clipboard property each time it loses its ownership to another application. This is done to detect any modification of the clipboard content by other applications.
When the libmatoya library recovers the clipboard ownership, it puts back in the clipboard what was added by the previous application. However, when the content is read, it is converted to UTF-8, which results in data loss if the content is not textual.
As a result, the local clipboard is corrupted when non-textual items, such as images or files, are copied.

Issue and PR reopen on root libmatoya repo: Here


Optimize imports and remove unused functions

This pull request is to remove unwanted imports and functions from the classes.
@nasthagiri @rohan-dhamal-clarice - Please review

Looks good! :+1:


Fix/simplify fetching the current user ID

Overview/Summary
Use a more direct method to retrieve the object ID in PowerShell. Current implementation failed for me and this approach is more transparent.
This PR fixes/adds/changes/removes

Instructions for executing the template using PowerShell.

Breaking Changes
None
Testing Evidence
Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
As part of this Pull Request I have

[ ] Checked for duplicate Pull Requests
[ ] Associated it with relevant issues, for tracking and closure.
[* ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Performed testing and provided evidence.
[* ] Updated relevant and associated documentation.
[ ] Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md)

Hi @richard087,
Thanks for taking the time to create this PR.
However, the proposed changes may actually lead to some unexpected behaviour as shown below:

As you can see in the above the proposed PowerShell changes may actually return a list of objects for some users hence the existing command taking a specific UPN from the current Az context to ensure the correct user is returned on its own.
Therefore I will close this PR as we will not be making these changes as they do work correctly today, as intended.
Please ensure you are using PowerShell 7 (https://docs.microsoft.com/powershell/scripting/install/installing-powershell?view=powershell-7.2) to run these commands and the latest version of the Az module (https://docs.microsoft.com/powershell/azure/install-az-ps?view=azps-7.0.0#update-the-azure-powershell-module).
If you have any issues please feel free to create an issue on this repo with all the information you can provide and we will assist 👍


layer: make clear that diffs should not cross mountpoints

From #492, I noticed that there was a concern about whether Config.Volumes should be included in layer changesets.  My gut says no, but we need to make this explicit (and what mountpoint we should start at). There's also a platform-dependence issue.

While #492 is defining the conversion of Volume -> mounts it's not a hard requirement, which means that either we have to i) talk about Volumes in layer.md b) make the requirement hard.

WDYT? Especially @vbatts who has had strong opinions about changesets in the past. :wink:

On Wed, Dec 14, 2016 at 02:32:16AM -0800, Aleksa Sarai wrote:

Should layer changesets ever cross a mountpoint? My gut says no,
but we need to make this explicit (and what mountpoint we should
start at).

I think we should start at the rootfs directory.  If somebody has
mounted something on top of that directory, we'd walk it.  So:
{…, "Volumes": {"/": {}}}
might end up in a volume getting walked.  But unless something like
#407 lands we don't care about the empty-layers use case anyway.

While #492 is defining the conversion of Volume -> mounts
it's not a hard requirement, which means that either we have
to i) talk about Volumes in layer.md b) make the requirement
hard.

Or we can accept that folks who provide Volumes in non-mount ways can
either modify the layer-generation algorithm to compensate or slurp up
volumes when they generate new layers.
If potential volume-slurping for SHOULD-breaking implementations is
not acceptable, we can get weaker config/layer coupling than you had
in 01fce0682 with a layer.md addition like:
Implementations must provide a way to ignore subtrees when
generating layers.
And then (somewhere?) require that implementations creating new layers
with a particular config in mind include that config's Volumes keys in
the ignored set.  But I'm really not excited about crafting enforcable
language for “creating new layers with a particular config in mind”.

@cyphar From the perspective of the image spec, Volumes is really just a changeset mask. This is an odd way of looking at it but from an image, that is all it is, practice. I am not sure if we need anything other than "Children of the the volume should not be included in the changeset". Requiring some sort of mount/volume duality may be overbearing for the runtime.
As far as crossing mountpoints, I am not sure what the implications are. There are going to issues in overlay, where files in sibling files in different layers may actually be a part of different devices (I think). It certainly makes creating the changesets much simpler, as you don't need to deal with problems that arise in these cases. How is crossing mountpoints represent in a tar file?

On Wed, Dec 14, 2016 at 01:10:33PM -0800, Stephen Day wrote:

How is crossing mountpoints represent in a tar file?

It isn't.  The tar archive will just hold the mount-flattened tree (or
as deep into that mount-flattened tree as you want to go, I think we
should stop after creating a stub file/directory to be a mount
destination).

From the perspective of the image spec, Volumes is really just a changeset mask. This is an odd way of looking at it but from an image, that is all it is, practice.

I agree with that way of wording it (mountpoints were not a good choice of wording) -- but from what I can tell there's nothing like that in the spec? Should I add it to config.md or layers.md or both?

As far as crossing mountpoints, I am not sure what the implications are.

Neither am I. Let's just drop it and focus on Volumes.

@cyphar This probably belongs in config.md. Note that there is some behavior about how the volume content may be "copied up" from the layer. The masking effect is more about the next layer created from a previous image from the volume.

@stevvooe I've put forth some sample wording in #504.

@cyphar Does PR #504 satisfy this issue?

@RobDolinMS Yes, but the PR hasn't been merged yet so we should keep this open for the moment.

This has effectively been hanlded by #694.


:ghost: use pf5 select for set-targets

Also pulled the select out of the text-content wrapper which was throwing off some styles.

interesting... i don't have it running to test and have been rolled onto another project for now. i'll see if i can look tomorrow.. maybe just clearing on open will be good instead... hopefully don't need to restort to setTimeout


[question] ant-design support or example?

i hava try many times to support antd in razzle. but not work now. so anyone have a idea?

Hey!
Have you tried changing your .babelrc to:
{
  "presets": ["razzle/babel"],
  "plugins": [
    "transform-decorators-legacy",
    [
      "import",
      {
        "libraryName": "antd",
        "style": "css"
      }
    ]
  ]
}

Can you specify your error after trying this ?

@SaraVieira hi
I added this
{
  "presets": [
    "razzle/babel"
  ],
  "plugins": [
    [
      "import",
      {
        "libraryName": "antd",
        "libraryDirectory": "es",
        "style": "css"
      }
    ]
  ]
}

But I get an error
(function (exports, require, module, __filename, __dirname) { import './index.css';
SyntaxError: Unexpected string

Do you know what the problem is?

@SaraVieira hi
I added this to .babelrc
{
  "presets": [
    "razzle/babel"
  ],
  "plugins": [
    [
      "import",
      {
        "libraryName": "antd",
        "libraryDirectory": "es",
        "style": "css"
      }
    ]
  ]
}

But I get an error
(function (exports, require, module, __filename, __dirname) { import './index.css';
SyntaxError: Unexpected string

Do you know what the problem is?

do you solve the problem?


[microNPU] Fix skip tests when Vela is not present

This was failing locally due to an indirect import in infra.py. Also ensure both tests are run by using the standard tvm.testing.main() function to run the tests.
cc @arina-grovety @sergio-grovety @ekalda

Thanks @lhutton1!


HockeyApp app sign-in

I am looking at the transition now. But my old hockey app sign-in credentials are honored only on appcenter web, not when I try to sign in to the new App Center / HockeyApp mobile app (android).
Are the existing HockeyApp api tokens honored when uploading to app center via a build tool like jenkins?

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

ID: 27acd64c-5106-51ce-628c-8a1a17269e95
Version Independent ID: d57cf8a3-d9de-4a8a-b746-e1f278e2c80c
Content: HockeyApp Transition - Visual Studio App Center
Content Source: docs/transition/index.md
Service: vs-appcenter
GitHub Login: @derpixeldan
Microsoft Alias: daadam

@hkodungallur please reach out to us via support (blue bubble).

The Android mobile app requires the same credentials as App Center web.
No, create new ones to use the App Center API


[Form Recognizer] Secure Supply Chain Analysis failing in nightly runs

Form Recognizer nightly test runs are failing with:

Error message:
Starting DockerFile Analysis:
sdk/identity/identity/test/manual-integration/Kubernetes/Dockerfile - Container usage from external registry 'docker.io' found.
sdk/identity/identity/test/manual-integration/Kubernetes/Dockerfile - Container usage from external registry 'docker.io' found.
sdk/identity/identity/test/manual-integration/Kubernetes/Dockerfile - Container usage from external registry 'docker.io' found.
sdk/identity/identity/test/manual-integration/Kubernetes/Dockerfile - Container usage from external registry 'docker.io' found.
sdk/identity/identity/test/manual-integration/Kubernetes/Dockerfile - Container usage from external registry 'docker.io' found.
sdk/servicebus/service-bus/test/stress/Dockerfile - Container usage from external registry 'docker.io' found.
sdk/test-utils/perf/Dockerfile - Container usage from external registry 'docker.io' found.
Azure Artifacts Configuration Analysis found 1 vulnerable NPM package manifest in the repository. Visit http://aka.ms/azure-artifacts-configuration-analysis for more details.
##[warning]Container security analysis found 3 violations. This repo has one or more docker files having references to images from external registries. Please review https://aka.ms/containers-security-guidance to remove the reference of container images from external registries. Please reach out via teams (https://aka.ms/cssc-teams) or email<EMAIL_ADDRESS>for any questions or clarifications.
Finishing: Secure Supply Chain Analysis (auto-injected by policy)

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1547440&view=results

This should already be fixed for the whole repo


componentWillMount, componentWillReceiveProps are depricated

These 2 methods should be removed from the code.

WARN  Warning: componentWillMount has been renamed, and is not recommended for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

Move code with side effects to componentDidMount, and set initial state in the constructor.
Rename componentWillMount to UNSAFE_componentWillMount to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run npx react-codemod rename-unsafe-lifecycles in your project source folder.

Please update the following components: HTML
WARN  Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

Move data fetching code or side effects to componentDidUpdate.
If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state
Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run npx react-codemod rename-unsafe-lifecycles in your project source folder.

Please update the following components: HTML

same issue here

I am using react-native0.60.5 and getting warning-
"componentWillReceiveProps is deprecated and will be removed in the next major version".
Can I get confirmation to resolve this issue in the next release, so that I could continue using this library without any problem in the next version of react-native also?

same issue here

.... and same.

and same issue here

same

@ThomasHenrissat Please update

Same here. The issue is in MyProjectName/node_modules/react-native-render-html/src/HTMLImage.js at line 43 and MyProjectName/node_modules/react-native-render-html/src/HTML.js at lines 80 and 88. I added UNSAFE_ to the names on these lines as the warning from React told me. This stops the warning from popping up and seems to work but I'm not sure if this will keep on working if I have to update my project again later. I think React is going to get rid of this option in the future. According to React: "These methods are considered legacy and you should avoid them in new code" (https://reactjs.org/docs/react-component.html).

It looks like a few people have already added pull requests with fixes for this. See:
https://github.com/archriss/react-native-render-html/pull/136
https://github.com/archriss/react-native-render-html/pull/256/files
https://github.com/archriss/react-native-render-html/pull/297
https://github.com/archriss/react-native-render-html/pull/306
https://github.com/archriss/react-native-render-html/pull/313
I have not gone over the code in all these to see which is best.

same

Fix available in version 4.2.0.


Adding MAROE Prefix verification.

MAROE Prefix verification added for msg/32 and msg/64 for
EPID devices.
Added verifyMaroePrefix function in cryptoService class.

Signed-off-by: Darshini Parikh<EMAIL_ADDRESS>

I assume the prefix value was verified? I know it differs slightly from the actual applet appId due to some mapping in the HostApp.

The prefix value was taken from the JSON based implementation. I don't have the prefix for the CBOR based implementation. If you have the value, I can update the PR.


Add an option to link an example to prompt from trace

Feature request
Since langsmith is already detecting correctly that I'm using a prompt template from the private hub please make it so that I can easily add an example to the prompt playground from the trace view just like I can do when running the prompt directly.
Motivation
This will make things a lot more easier to us since prompt engineers could easily add an example to the prompt playground from a given trace and he or she can use it to optimize the prompt

Could you elaborate a bit more on what your desired UX is?
You can add any run to a dataset, and you can open datasets in the prompt playground when iterating on a prompt.

Thank you for your response! I appreciate the ability to add a run to a dataset, but I've noticed that it doesn't retain the input template of the original prompt. For example, if I've defined a template with variables like format_instructions and others, it merges these variables into one input when adding an example to the dataset, which doesn't align with my intended workflow.
Additionally, since we're discussing the prompt playground, it would be helpful to have the option to switch the prompt template type between chat and completion modes and vice versa. It would also be great to be able to add an output schema after the prompt is created, through the edit functionality, rather than only during the initial creation.


WHOIS

WHOIS is tricky, since all the IRCd's do it differently.
I'm thinking that it follows the general pattern:
WHOIS foobar
:server 3xx mynick foobar <lots of other stuff>
<rinse and repeat>

But the details vary for every single IRCd I've tested. It isn't feasible to parse the entire whois response into a useful, ircd-generic form, so just passing the parameters and having clients/bots deal with the specifics is the best that can be done, I guess. Working on this now.

Sounds good to me! Thank you.


[Feature] Support to delete the row in aggregation merge engine

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
Currently, the aggregation merge engine only produce INSERT row. However, if all the input of a key has been retract, it should produce a DELETE row.
Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Now 0.8, Spark SQL supports COW DELETE for agg table.


VSCode sui-move-analyzer plugin problem

When I use the sui-move-analyzer plugin in VSCode, I encountered the following issues:
parse manifest '"/home/user/.move/https___github_com_MystenLabs_sui_git_framework__testnet/crates/sui-framework/packages/sui-framework" 'failed.
 addr must exactly 32 length or start with '0x' like '0x2'
Error parsing '[package]' section of manifest

Caused by:
    Invalid 'edition'. Unsupported edition "2024.beta". Current supported editions include: "legacy", "2024.alpha"

My Move.toml :
[package]
name = "task1"

version = "1.0.0"
edition = "2024.alpha"
license = "MIT"
authors = ["Joe Smith (joesmith@noemail.com)"]

[dependencies]
Sui = { git = "https://github.com/MystenLabs/sui.git", subdir = "crates/sui-framework/packages/sui-framework", rev = "framework/testnet" }

[addresses]
task1 = "0x0"

[dev-dependencies]

[dev-addresses]

My VSCode settings.json:
{
  "sui-move-analyzer.server.path": "~/.cargo/bin/sui-move-analyzer",
  "move-analyzer.server.path": "~/.cargo/bin/move-analyzer",
  "move.server.path": "~/.cargo/bin/move-analyzer"
}

The command executed during the installation of sui-move-analyzer is:
cargo install --git https://github.com/movebit/move --branch sui_move_analyzer sui-move-analyzer

I made some attempts to resolve this issue, and eventually, I found that installing sui-move-analyzer with another command, while keeping all configurations unchanged, solved the problem.
cargo install --git https://github.com/movebit/move --branch move-analyzer2 move-analyzer

By using this command to install move-analyzer and then renaming it to sui-move-analyzer, the issue was perfectly resolved.
So, is there a bug in the sui_move_analyzer branch?

for the beta version you should mystan labs move analyzer


Phishlets hidden url

PLEASE READ THE POSTING GUIDELINES AND ANSWER THE QUESTION BEFORE POSTING, OTHERWISE ISSUE WILL BE CLOSED AND MARKED AS INVALID

I hereby declare the following issue is a [tool specific question/bug report] and it is NOT a help request about creating a phishlet.
I am fully aware that this is not a customer support portal, I can't demand answers and I'm aware I am using a free tool.
I am not going to use Evilginx to hax my girlfriend's account or use it for any other illegal purpose.
I am not trying to set up a domain on FreeNOM (also read the sentence above again).
I am not a robot.

Please type in "I CONFIRM" below if you confirm the sentences above or otherwise make some funny remark:
I CONFIRM
Thanks!
--
The phipshlets hide url block all the domain.com (example). You can't make it to block and redirect the domain.com and let go ahead the domain.com/random? Thanks

So all the lures path can go to ahea and will never been blocked, phishlets hidden command to block only the hostname.  Thanks

PLEASE READ THE POSTING GUIDELINES AND ANSWER THE QUESTION BEFORE POSTING, OTHERWISE ISSUE WILL BE CLOSED AND MARKED AS INVALID

I hereby declare the following issue is a [tool specific question/bug report] and it is NOT a help request about creating a phishlet.
I am fully aware that this is not a customer support portal, I can't demand answers and I'm aware I am using a free tool.
I am not going to use Evilginx to hax my girlfriend's account or use it for any other illegal purpose.
I am not trying to set up a domain on FreeNOM (also read the sentence above again).
I am not a robot.

Please type in "I CONFIRM" below if you confirm the sentences above or otherwise make some funny remark:
I CONFIRM
Thanks!
--
The phipshlets hide url block all the domain.com (example). You can't make it to block and redirect the domain.com and let go ahead the domain.com/random? Thanks

i can help you with the<EMAIL_ADDRESS>


Add julia language support.

Is it as simple as this?

Coverage remained the same at 100.0% when pulling 7126536d8b84c2f324b8089916443b9364f826dd on sbromberger:patch-1 into bb5c9dc3a9679262ff26c4fca701f9e7b347555e on deerawan:master.

Coverage remained the same at 100.0% when pulling 3a5b77899169d34716cf7565d1000af02c16b04d on sbromberger:patch-1 into bb5c9dc3a9679262ff26c4fca701f9e7b347555e on deerawan:master.


Add more API documentation for generated swagger docs

Now that the swagger documentation is available on api.deps.cloud, we should add more documentation to the protobufs based on the service, functionality, and message bodies.

Where is it? https://api.deps.cloud returns nothing (#175)


too much cpu

Hi, I do not know why, but I had to disable the fuzzy-finder because it required too much CPU (100%) without it, atom works fine...

@andre-dietrich Can you share a bit more information?

https://github.com/atom/fuzzy-finder/issues/216

@izuzak https://github.com/atom/fuzzy-finder/issues/216

@andre-dietrich @izuzak https://github.com/atom/fuzzy-finder/issues/216

@sagarpanchal please stop this spam.
You can edit your previous Github messages.

@pierrre
I did't spam, I deleted them.

@sagarpanchal yes, but when you delete and create a new message, it sends a new email/notification.
That's why it's preferable to edit a message.

@sagarpanchal yes, but when you delete and create a new message, it sends a new email/notification.
That's why it's preferable to edit a message.

@sagarpanchal yes, but when you delete and create a new message, it sends a new email/notification.
That's why it's preferable to edit a message.

@sagarpanchal yes, but when you delete and create a new message, it sends a new email/notification.
That's why it's preferable to edit a message.

@sagarpanchal yes, but when you delete and create a new message, it sends a new email/notification.
That's why it's preferable to edit a message.

Closing as no new information has been provided.


select parameter constraint

    pub fn load_select<Selection, T>(nid: i64, selection: Selection, conn: &PgConnection) -> QueryResult<T>
        where Selection: ???,
              T: ???
    {
        node::table.find(nid)
                   .select(selection)
                   .first(conn)
    }

Probably T: Queryable<Selection::SqlType>, Selection: SelectableExpression<node::table>. Follow the compiler errors and ask more specific questions if you don't understand the constraints it claims aren't holding.


phyHandling: fix compilation with Clang, fix incorrect log output

Description

fix variable 'ulPHYLinkStatus' is uninitialized when used here [-Werror,-Wuninitialized] on Clang 17
fix incorrect operation (|= instead of &) in printf, which leads to the always high output

Checklist:

[x] I have tested my changes. No regression in existing tests.
[ ] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@htibosch fixed according to review

@htibosch
I also had to remove ulPHYLinkStatus completely, otherwise it causes error: variable ‘ulPHYLinkStatus’ set but not used [-Werror=unused-but-set-variable] when configPRINTF is no-op.


ci: Reduce the number of Fluent version to run for each test

Following are the 3 categories of unittests:

Tests which do not require to run Fluent at all or can be run with any stable Fluent version:  Such tests aren't marked with the fluent_version marker and will be run only with the latest released Fluent version during PRs and with the latest released and the current development Fluent versions during nightly. If these tests get broken due to a recent Fluent API change, that will be caught during nightly.
Tests which depend on the current development Fluent version or are tracking Fluent API change: Such tests are marked as fluent_version("develop") and will be run only with the current development Fluent version.
Tests which depend on a specific released Fluent version: Such tests are marked as fluent_version("==<version>") and will be run only with that specific Fluent version.

After this PR is merged, we only need to add the fluent_version marker for tests which depends on the current development Fluent version or a specific released Fluent version. We should make sure a test is not marked to run with more than one Fluent version unless there is very specific scenario.

@prmukherj There is a Fluent version check in the CompundTask.add_child_and_update method in workflow.py. Let me know if we need to add specific Fluent version marker in some tests to cover that code.

@prmukherj There is a Fluent version check in the CompundTask.add_child_and_update method in workflow.py. Let me know if we need to add specific Fluent version marker in some tests to cover that code.

Not required @mkundu1, that is to just provide a warning to users using that argument for earlier versions. We don't need to test it for any specific versions.

Superseded by #2708


Standardize Keypress Menus

@OfficialEsco @vedantmgoyal2009
I noticed most of the menus were similar, so I made a function to help reduce duplicated code. Please review

Did https://github.com/Trenly/winget-pkgs/pull/11/commits/9ef559c5b808497de68f05e6fbe93d676c9f04c4 fix this? 😁
Have you signed the Contributor License Agreement (CLA)?
Reference Link: https://cla.opensource.microsoft.com/microsoft/winget-pkgs
  [Y] Yes
  [N] No

Enter Choice (default is 'N'):

Have you checked that there aren't other open pull requests for the same manifest update/change?
Reference Link: https://github.com/microsoft/winget-pkgs/pulls
  [Y] Yes
  [N] No

Enter Choice (default is 'N'): yMethod invocation failed because [System.ConsoleKey] does not contain a method named 'Replace'.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:888 char:27
+ ...         'Y' { $PrBodyContentReply += $_.Replace("[ ]", "[X]"), "`n" }
+                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : MethodNotFound

Automatic manifest validation failed. Check your manifest and try again

Have you validated your manifest locally with 'winget validate --manifest <path>'

  [Y] Yes
  [N] No

Enter Choice (default is 'N'): yMethod invocation failed because [System.ConsoleKey] does not contain a method named 'Replace'.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:907 char:31
+ ...         'Y' { $PrBodyContentReply += $_.Replace("[ ]", "[X]"), "`n" }
+                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : MethodNotFound

You did not test your Manifest in Windows Sandbox previously.

Have you tested your manifest locally with 'winget install --manifest <path>'

  [Y] Yes
  [N] No

Enter Choice (default is 'N'): n

Does your manifest conform to the 1.0 schema?
Reference Link: https://github.com/microsoft/winget-cli/blob/master/doc/ManifestSpecv1.0.md
  [Y] Yes
  [N] No

Enter Choice (default is 'N'): yMethod invocation failed because [System.ConsoleKey] does not contain a method named 'Replace'.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:942 char:27
+ ...         'Y' { $PrBodyContentReply += $_.Replace("[ ]", "[X]"), "`n" }
+                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : MethodNotFound

Does this pull request resolve any issues?

  [Y] Yes
  [N] No

Enter Choice (default is 'N'): n
Method invocation failed because [System.ConsoleKey] does not contain a method named 'Trim'.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:980 char:5
+     $PrBodyContentReply = ($PrBodyContentReply.Trim() -ne '')
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : MethodNotFound

Output =
Backspace

N

Did 9ef559c fix this? 😁
Have you signed the Contributor License Agreement (CLA)?
Reference Link: https://cla.opensource.microsoft.com/microsoft/winget-pkgs
  [Y] Yes
  [N] No

Enter Choice (default is 'N'):

Have you checked that there aren't other open pull requests for the same manifest update/change?
Reference Link: https://github.com/microsoft/winget-pkgs/pulls
  [Y] Yes
  [N] No

Enter Choice (default is 'N'): yMethod invocation failed because [System.ConsoleKey] does not contain a method named 'Replace'.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:888 char:27
+ ...         'Y' { $PrBodyContentReply += $_.Replace("[ ]", "[X]"), "`n" }
+                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : MethodNotFound

Automatic manifest validation failed. Check your manifest and try again

Have you validated your manifest locally with 'winget validate --manifest <path>'

  [Y] Yes
  [N] No

Enter Choice (default is 'N'): yMethod invocation failed because [System.ConsoleKey] does not contain a method named 'Replace'.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:907 char:31
+ ...         'Y' { $PrBodyContentReply += $_.Replace("[ ]", "[X]"), "`n" }
+                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : MethodNotFound

You did not test your Manifest in Windows Sandbox previously.

Have you tested your manifest locally with 'winget install --manifest <path>'

  [Y] Yes
  [N] No

Enter Choice (default is 'N'): n

Does your manifest conform to the 1.0 schema?
Reference Link: https://github.com/microsoft/winget-cli/blob/master/doc/ManifestSpecv1.0.md
  [Y] Yes
  [N] No

Enter Choice (default is 'N'): yMethod invocation failed because [System.ConsoleKey] does not contain a method named 'Replace'.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:942 char:27
+ ...         'Y' { $PrBodyContentReply += $_.Replace("[ ]", "[X]"), "`n" }
+                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : MethodNotFound

Does this pull request resolve any issues?

  [Y] Yes
  [N] No

Enter Choice (default is 'N'): n
Method invocation failed because [System.ConsoleKey] does not contain a method named 'Trim'.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:980 char:5
+     $PrBodyContentReply = ($PrBodyContentReply.Trim() -ne '')
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : MethodNotFound

Output =
Backspace

N

I believe it did

Yapyap that did, haven't tested the alternative routes tho, but i'm guessing this fixed every similar string

Yapyap that did, haven't tested the alternative routes tho, but i'm guessing this fixed every similar string

I think it did. I’m not sure why the pipeline output was the iterand and not a variable since it can really mess things up, but yeah, should be fixed

Seems okay, just a bit weird in the start compared to the last 2 version's we have had of the prompts 😁
Also just noticed mine is different compared to the screenshots, when the Help Variable it empty it outputs a space
Have you tested your manifest locally with 'winget install --manifest <path>'

  [Y] Yes
  [N] No

Enter Choice (default is 'N'):

Can you add a quick fix to this PR?
Fixing this spacing
Do you want to submit your PR now?

  [Y] Yes
  [N] No

Enter Choice (default is 'Y'): yHEAD is now at 78f8884e5 New version: Telegram.TelegramDesktop version 2.9.2 (#24434)
warning: LF will be replaced by CRLF in manifests/j/Joplin/Joplin/2.2.7/Joplin.Joplin.installer.yaml.

This also happens when you accept to test in Sandbox, those are the 2 places i remember atm
Just noticed another issue when i said mine looked different, this prompt is messed up
You did not test your Manifest in Windows Sandbox previously.

Have you tested your manifest locally with 'winget install --manifest <path>'

  [Y] Yes
  [N] No

Enter Choice (default is 'N'):

I'll look at this and make a single commit for your suggestions; Do you think it would be better to have the error messages as the prompt like your suggestion . . .
$_menu = @{
    entries       = @("[Y] Yes"; "*[N] No")
    Prompt        = "Automatic manifest validation failed.
    DefaultString = "N"
 }

or as the help text, since it isn't the actual question being asked like this -
$_menu = @{
    entries       = @("[Y] Yes"; "*[N] No")
    Prompt        = "Have you validated your manifest locally with 'winget validate --manifest <path>'?"
    HelpText      = "Automatic manifest validation failed. Check your manifest and try again"
    DefaultString = "N"
 }

I'll look at this and make a single commit for your suggestions; Do you think it would be better to have the error messages as the prompt like your suggestion . . .

or as the help text, since it isn't the actual question being asked like this -

Yeye thats fine, i think it should be at the top with maybe red text (something noticeable) if possible, i just worked with what i had 😁 Which.. failed
Have you checked that there aren't other open pull requests for the same manifest update/change?
Reference Link: https://github.com/microsoft/winget-pkgs/pulls
  [Y] Yes
  [N] No

Enter Choice (default is 'N'): y

You did not test your Manifest in Windows Sandbox previously.

  [Y] Yes
  [N] No

Enter Choice (default is 'N'):

Lets leave it like that for now, can you add do this quick change for cleanness sake
        git fetch upstream master -q
        git switch -d upstream/master
        if ($LASTEXITCODE -eq '0') {
            git add -A
            git commit -m "$CommitType`: $PackageIdentifier version $PackageVersion"
            git switch -c "$PackageIdentifier-$PackageVersion"
            git push --set-upstream origin "$PackageIdentifier-$PackageVersion"

to
        git fetch upstream master --quiet
        git switch -d upstream/master
        if ($LASTEXITCODE -eq '0') {
            git add -A
            git commit -m "$CommitType`: $PackageIdentifier version $PackageVersion" --quiet

            git switch -c "$PackageIdentifier-$PackageVersion" --quiet
            git push --set-upstream origin "$PackageIdentifier-$PackageVersion" --quiet

need the first git switch to not be silent to get this line HEAD is now at b73a14ccd gitthermal.thermal version 0.0.4 (#24445)
warning: LF will be replaced by CRLF in manifests/s/SoftwareFreedomConservancy/QEMU/6.0.93/SoftwareFreedomConservancy.QEMU.yaml.
The file will have its original line endings in your working directory

Is probably from git add which cannot be silenced
remote:
remote: Create a pull request for 'SoftwareFreedomConservancy.QEMU-6.0.93' on GitHub by visiting:
remote:      https://github.com/OfficialEsco/winget-pkgs/pull/new/SoftwareFreedomConservancy.QEMU-6.0.93
remote:

is believe is from gh pr create which does not have a silent switch
It looks a lot better tho

Think i've found a new issue which might have appeared after the -exclude problem popped up
[Required] Enter the Package Identifier, in the following format <Publisher shortname.Application shortname>. For example: Microsoft.Excel
PackageIdentifier: TorProject.TorBrowser

[Required] Enter the version. for example: 1.33.7
Version: 10.5.4
Found Existing Version: 10.5a17
Get-ChildItem : Cannot find path 'C:\Users\Esco\GitHub\winget-pkgs\manifests\t\TorProject\TorBrowser\10.5a17' because it does not
exist.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:1286 char:36
+ ... cript:OldManifests = Get-ChildItem -Path "$AppFolder\..\$LastVersion"
+                          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Users\Esco\...Browser\10.5a17:String) [Get-ChildItem], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.GetChildItemCommand

Error: Version 10.5a17 does not contain the required manifests
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:1305 char:9
+         Throw "Error: Version $LastVersion does not contain the requi ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Error: Version ...uired manifests:String) [], RuntimeException
    + FullyQualifiedErrorId : Error: Version 10.5a17 does not contain the required manifests

It picked up \Manifests\t\TorProject\TorBrowser\Alpha\10.5a17\ instead of \Manifests\t\TorProject\TorBrowser\10.5.2

Think i've found a new issue which might have appeared after the -exclude problem popped up
[Required] Enter the Package Identifier, in the following format <Publisher shortname.Application shortname>. For example: Microsoft.Excel
PackageIdentifier: TorProject.TorBrowser

[Required] Enter the version. for example: 1.33.7
Version: 10.5.4
Found Existing Version: 10.5a17
Get-ChildItem : Cannot find path 'C:\Users\Esco\GitHub\winget-pkgs\manifests\t\TorProject\TorBrowser\10.5a17' because it does not
exist.
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:1286 char:36
+ ... cript:OldManifests = Get-ChildItem -Path "$AppFolder\..\$LastVersion"
+                          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Users\Esco\...Browser\10.5a17:String) [Get-ChildItem], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.GetChildItemCommand

Error: Version 10.5a17 does not contain the required manifests
At C:\Users\Esco\GitHub\winget-pkgs\Tools\YamlCreate.ps1:1305 char:9
+         Throw "Error: Version $LastVersion does not contain the requi ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Error: Version ...uired manifests:String) [], RuntimeException
    + FullyQualifiedErrorId : Error: Version 10.5a17 does not contain the required manifests

It picked up \Manifests\t\TorProject\TorBrowser\Alpha\10.5a17\ instead of \Manifests\t\TorProject\TorBrowser\10.5.2

Looks like exclude overrides the depth flag

Edit - Can we assume that all manifest files match the pattern *.yaml ?

Very much yes, i think i've actually thought about that before when i was looking trough the script, but didn't look much into it since it actually worked..

Edit - Can we assume that all manifest files match the pattern *.yaml ?

Very much yes, i think i've actually thought about that before when i was looking trough the script, but didn't look much into it since it actually worked..

Okay; I think I fixed the filter.
I also changed the git code to be what you requested, and also suppressed the crlf warnings

Well... There appears to be a new issue where it dosn't detect the manifests in the sub folder >.<
Was trying to update Google.Chrome.Beta and it didn't fetch any metadata...

@OfficialEsco I see an email that you posted a comment; Did you delete the comment?

Yes, i was silly and on the wrong branch after fixing a Manifest🤦♂ Somehow the script in the official Repo (v1.1.5) maintain some of the functions from the new script so it looks identical, however it does not fetch the new method of fetching metadata, i've checked and realized before, but now i've added ;git switch YamlParsing so i was 99% sure i was on the right branch... So i realized once i opened GitHub Desktop and about to switch to your branch 🤦♂🤦♂

All good! Just wanted to maks sure I wasn’t missing something


Calling angular3/typescript codes inside elementClick function

`multibar: {
      dispatch:{
        elementClick: function(t, u){
          //call angular4/typscript variables here.
        }
      }

}
`
Hi,
How can I call a typescript variables/codes inside my dispatch elementClick()? I was trying to call my variables like this.chartType  but it's always undefined. I also plan to call a typeScript  modal inside the elementClick function. Is that possible?
Thanks!

I figured it out! Closing this issue..


FPGA Ethernet Support

The DE2-115 dev board has an ethernet controller. If there were a hardware block that could expose this to software running on the core, it could be used to load programs onto the board much faster than the serial port, by having the boot loader connect to the host over TFTP or some other protocol.

The dev board I'm using uses the following 88E1111 Ethernet transceiver (https://www.marvell.com/transceivers/assets/Marvell-Alaska-Ultra-88E1111-GbE.pdf), which can use either the MII or RGMII modes depending on a jumper on the board.
This could benefit from #151 to reduce overhead of copying packet data.


Pesticide Exam - Revert Exam Type Changes

Client has asked that the default exam type change that was made in commit
12d4014 be reverted since there are now going to be 3 different pesticide exam types (accounting for duration). A sort of the exam types has also been applied such that the list will now mirror functionality of all other exam type selections in all other add exam modal types.

Please push the other pesticide changes through first so I can test this commit before it's merged.
Thanks!


How to enabed torrentz2 provider?

i have changed enabled status as true. but still its not showing in burst?
kindly help!!! all the false also i put as true but no help.. pls suggest on how to enable the provider.
"torrentz": {         "anime_keywords": "{title} s{season:2}e{episode:2}",         "anime_keywords2": "{title} {season:2}x01|S{season:2}",         "anime_keywords_fallback": "{title} s{season:2}",         "anime_query": "",         "base_url": "https://torrentz2.nz/search?f=QUERYEXTRA",         "color": "FFFFB090",         "enabled": true,         "filter_title": true,         "general_extra": "",         "general_keywords": "{title}",         "general_query": "",         "language": null,         "languages": "en",         "login_failed": "",         "login_object": "",         "login_path": null,         "movie_extra": "",         "movie_keywords": "{title} {year}",         "movie_query": "",         "name": "TorrentZ",         "parser": {             "infohash": "item(tag='a', attribute='href', divider=('/', 1))",             "name": "item('a')",             "peers": "item(tag='span', order=5)",             "row": "find_once('div', ('class', 'results')).find_all('dl')",             "seeds": "item(tag='span', order=4)",             "size": "item(tag='span', order=3)",             "torrent": "'magnet:?xt=urn:btih:%s' % item(tag='a', attribute='href', divider=('/', 1))"         },         "predefined": false,         "private": false,         "season_extra": "",         "season_extra2": "",         "season_keywords": "{title} season {season:2}",         "season_keywords2": "{title} s{season:2}",         "season_query": "",         "separator": "%20",         "show_query": "",         "subpage": false,         "tv_extra": "",         "tv_extra2": "",         "tv_keywords": "{title} s{season:2}e{episode:2}",         "tv_keywords2": "{title} {season:2}x01|S{season:2}",         "tv_keywords_fallback": "{title} s{season:2}"     },

@naveen1hot
If you do development (you cloned git repo), then easy - run https://github.com/elgatito/script.elementum.burst/blob/master/scripts/generate_settings.py inside your repo (python 3 scripts/generate_settings.py).
If you do local changes - then manually copy/paste/edit settings lines for provider in your local https://github.com/elgatito/script.elementum.burst/blob/master/resources/settings.xml file (resources/settings.xml).
Although, it was disabled for a reason. You would need to fix it to use it.

I hello @antonsoroko .. thanks for your quick response.. I tried same. But it's not appearing in elementum burst..

You tried what exactly? You need to change providers.json and settings.xml and you need to check that you did changes in xml file correctly.

Thanks a lot @antonsoroko. With your guidance i am able to enabled torrentz and its appear in burst also..
as you said it has some problem.
but while checked with new domain (https://torrentz2.nz/) its working fine. but not showing the results. could you pls check it if  possible.

@naveen1hot i added it, but i am disapointed. open any link, scroll down to bottom and you will see that this torrentz2 is just a different UI for bitserach #351.
Also it uses "bad" numbers for seeds/peers, like 1.2K so very popular torrents will be skipped b/c of this wrong format.

Thanks greatful for you efforts and response.  It's working now.. yeah seems it's replication of bitsearch. But torrentz working very well for me. Thanks so much @antonsoroko ...


Add is_vehicle_tracking_enabled on Vehicle

Proposed change

Add is_vehicle_tracking_enabled property on ConnectedDriveVehicle (see https://github.com/bimmerconnected/bimmer_connected/issues/318#issuecomment-981981112).
Type of change

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to this library)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:

Checklist

[x] The code change is tested and works locally.
[x] Tests have been added to verify that the new code works.

Codecov Report

Merging #370 (6c113ee) into master (28fe969) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #370      +/-   ##
==========================================
+ Coverage   96.42%   96.43%   +0.01%
==========================================
  Files           8        8
  Lines         894      897       +3
==========================================
+ Hits          862      865       +3
  Misses         32       32

Flag
Coverage Δ

3.10.x
96.32% <100.00%> (+0.01%)
:arrow_up:

3.6
96.41% <100.00%> (+0.01%)
:arrow_up:

3.7
96.30% <100.00%> (+0.01%)
:arrow_up:

3.8
96.31% <100.00%> (+0.01%)
:arrow_up:

3.9
96.31% <100.00%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

bimmer_connected/vehicle.py
98.29% <100.00%> (+0.04%)
:arrow_up:

bimmer_connected/vehicle_status.py
97.93% <100.00%> (ø)

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 28fe969...6c113ee. Read the comment docs.


Update for v1.0.1

Signed-off-by: Christopher Schröder<EMAIL_ADDRESS>
Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] ~Bumped the build number (if the~ version is unchanged~)~
[x] Reset the build number to 0 (if the version changed)
[ ] ~Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)~
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender


Fix typo

@nigel5 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


"Number of participants to see statistics" not correct

Your website tells at least 500 participants
But in facts, I have a channel, and I were able to see the stats since I got 75 members (still need to confirm this)

You are confusing limits for chats and channels, there's two of them:
https://limits.tginfo.me/en/statistics
For channels, the limit is 50 participants. for groups — 500.


Transfer Hotspot not working

Transfer Hotspot not working
When I start the transfer the receiver gets a notification as shown in the image but when I click the "view transaction" button nothing happens.
I have tried to redownload my app and log in again but it didn`t help.

Device: [Samsung galaxy A52]

App Version [v 4.2.1]

Additional context
Add any other context about the problem here.

I am having a similar issue, has this been resolved?


Supporting RPC (client-server interaction)?

Do you guys have plans to support Remote Procedure Calls (RPC)? Websharper already has that for F# and recently also for C#! C# Remoting. ScalaJs has it as well using Autowire.

Hey @Zaid-Ajaj Im not with bridge, but how could they support Websharper as they are in beta??

Hi @Zaid-Ajaj,
We had discussions within the team about something like this, a great feature.
No firm time frame for this feature, however.


Why isn't there a definition for the term 'tenant' on this page?

[Enter feedback here]

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

ID: f9d5b3f1-0d66-0abc-3114-6fc51c707271
Version Independent ID: ba3beb78-6546-8154-41b9-b7fd20156666
Content: Azure glossary - Azure dictionary
Content Source: articles/azure-glossary-cloud-terminology.md
Service: multiple
GitHub Login: @MonicaRush
Microsoft Alias: monicar

@Crispness Thanks for the feedback. We shall evaluate and update the document as appropriate.

@Crispness Thanks for bringing this to our attention. I have submitted a PR to fix the issue that's in review. The changes should reflect post the merge.


add CommonHTML configuration example.

I'm undecided on how to handle the fonts though. But I would like to add at least one working example.

Non-working attempt is on https://github.com/pkra/MathJax-single-file/tree/texhtml

The branch is now working. But it still needs a config option to rewrite the font URL.
Can be hacked by pointing https://github.com/pkra/MathJax-single-file/blob/texhtml/dist/TeXHTML/MathJax-TeXHTML.js#L3442 to whatever URL you like (e.g., cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.0/fonts//HTML-CSS).

Ok. The branch is ready (though on v2.7.0).


Test Out COVID-19 HealthAlert Helpline

https://paper.dropbox.com/doc/COVID-19-HealthAlert-Helpline-0iwYNF9LYErvZcFJXNkSR

@schowdhury8 18th March
It's rather good (they work with the WHO) for information dispersion (updated in realtime) and curbs quite a few myths and misconceptions to tackle misinfo. I have a call with them tomorrow so if anyone has any questions they want me to bring up, lmk!!

Tested it out, but there's a lot of better alternatives getting made every day. Since it doesn't seem particularly relevant right now, we'll revisit if we ever need to.


Dependency/repo updates

Changelog
None
Docs
None
Description
Based on https://github.com/foxglove/message-definition/pull/6

Upgrade yarn to v4
Upgrade ESLint to v9
Upgrade other dependencies
Add provenance attestation for npm publish
Add dependabot

"provenance attestation" sounds so impressive


Make vsphere cluster configurable

What this PR does / why we need it:
Fixes our e2e tests

one nit and this is lgtm
@mrIncompetent thank you for fixing our e2e tests on vsphere !


More than 1 pin with custom pin icon throws exception

        var assembly = typeof(MapPage).GetTypeInfo().Assembly;
        var fileStream = assembly.GetManifestResourceStream($"iMHD.sample.png");

        foreach (var vehicle in ServerCommunicatorClient.GetAllVehicles())
        {
            var pin = new Pin
            {
                Position = new Position(vehicle.Position.Latitude, vehicle.Position.Longitude),
                Label = $"Line {vehicle.Line}",
                Address = $"Delay: {vehicle.Delay/60} minutes",
                Icon = BitmapDescriptorFactory.FromStream(fileStream)
            };

            MhdMap.Pins.Add(pin);
        }

In second loop it throws and exception Java.Lang.NullPointerException: null reference
If I do same just with 1 item it works

Also with the same code iOS does not work, it just renderes default pins.

Also with the same code iOS does not work, it just renderes default pins.

Thank you for your reporting.
I'll check it.
I wonder can't use same fileStream instance.

I wanted to debug by my self, but without logging I think I am not able to debug.

I tested with some changes:
        var assembly = typeof(MapPage).GetTypeInfo().Assembly;

        foreach (var vehicle in ServerCommunicatorClient.GetAllVehicles())
        {
            var fileStream = assembly.GetManifestResourceStream($"iMHD.sample.png");

            var pin = new Pin
            {
                Position = new Position(vehicle.Position.Latitude, vehicle.Position.Longitude),
                Label = $"Line",
                Address = $"Delay: 0 minutes",
                Icon = BitmapDescriptorFactory.FromStream(fileStream)
            };

            MhdMap.Pins.Add(pin);
        }

and it works, but I think it is still bug.
Anyway is it possible to create image from code? I want to draw triangle with number inside

You can user BitmapDescriptorFactory.FromView what can make Pin from your custom View.
Sample code is here!

Is it possible to use
var iconImage = BitmapDescriptorFactory.FromBundle("symbol_fore_close.png"); ?
Because it does not work for me, even if i change Build Action for image to Embeded Resource.
Or the only way is to create view with my image/ Get stream of image

Probably memery is destroyed after it is read.


Request: smart contract bytecode GET endpoint

It will be quite handy to have an endpoint in the middleware that returns the smart contract's bytecode. e.g. /middleware/contracts/<address>/bytecode
This will could be used for verifying the smart contract functionality.
Example use-case:
Aepp Developer provides smart contract code in github
-> user compiles it locally
-> verifies if the compiled bytecode matches the one on the provided address by developer
-> safely interact with smart contract.
P.S. Now when I'm writing this issue I think this could be further improved by having a meta-info endpoint - which could return the smart contract owner, the bytecode and other info that might end up in the meta. You can consider that as well.

OK to close this, since the info is provided by /contracts/transactions/address/ ?
regarding your postscript, the owner, if not the creator and in the logic, is not knowable by the middleware, as things currently stand.
re-open if this is not OK!


Project help

hi i am having trouble passing this test case and i cant figure out why,  can anyone help me out?

Check your arithmetic for the different events. Ensure the correct time is passed.

it looks like an issue with your self check queue, 13 should be at the front of the queue however for your code 15 is at the front of the queue instead and is being served first

Maybe something went wrong with your queuing mechanism. Perhaps you can look at which events you add 1 to Q and which events you remove 1 from queue and also whether the correct version of the server is being used to add/remove from the Q.

note that at self-check 3, it is first self check out counter where customers queue. At line 37, notice that 13 serves at self-check 3. In other words, 13 waited and was also served by 3. The expected output for this test case wants ur code to be flexible for 13 to accept any SCC (self-check 3,4,5 ... onwards). Therefore they wanted customer 13 to be served at self check 4 as at event time(0.862), it was the most readily available SCC to serve customer 13


Authentication

@dubcanada commented on Wed Feb 24 2016
Hello,
I am in need of the Authentication provider, so I figured rather then just build one for myself I would build it and contribute it back. I was reading the Trello card for the Authentication https://trello.com/c/wns5DzT3/28-authentication-provider
Are you wanting to separate it out into it's own repo like you did with the mail provider https://github.com/adonisjs/adonis-mail ? I plan on copying that to start with as that has your driver structure done.
I'm going to start tomorrow, so I'll probably send a pr before the end of this week. At least for basic/session, JWT may take a bit more as I don't need that atm.
Thanks!

@thetutlage commented on Wed Feb 24 2016
@dubcanada Nice, it will be great to have initial authentication system for Adonis. I am not sure if i would be able to make it the official Auth provider, because the actual one i have in my mind is quite complex. Here is a brief.

Authentication system will have UserProviders, whose job will be to validate credentials. Now these can be username,password or can be a token. UserProvider has no idea where the credentials will come from.

There will be multiple schemes like session, basic-auth. Now these scheme knows where to look for credentials.

Finally there will a driver which will read credentials from schemes and will validate them with the UserProviders.

Also there will be mechanism for guest logins, which means you can dummy login visitors and once they signup, you can simply join their guest tokens with their user profile. It will be helpful to know what a visitor did before becoming a user.
Obviously you are not bound to implement all these things. But it will be great if you can come up with a normal authentication provider to help others until i come up with the fully fledged one.

@dubcanada commented on Tue Mar 01 2016
I'll do a basic one for session and basic-auth drivers and leave it at that. I'll try and copy how you do things currently along with the Laravel auth provider and see how it turns out. Hopefully it will work until you do the larger version with JWT and such.

@thetutlage commented on Tue Mar 01 2016
:+1:

Cool I'll commit it here, I am still working on it.

Closing as it has been done


Report file empty

🐞Bug Report
Bug Description
when running gremlins unleashed command on main.go file, the report file is empty.
To Reproduce
Steps to reproduce the behavior:

Execute Gremlins unleash on main.go in the cmd /gremlins: gremlins unleash -o test.json2.
I see the correct log in the terminal

Starting...
Gathering coverage... done in 330.058424ms
 NOT COVERED CONDITIONALS_NEGATION at main.go:46:9

Mutation testing completed in 82 milliseconds 751 microseconds
Killed: 0, Lived: 0, Not covered: 1
Timed out: 0, Not viable: 0
Test efficacy: NaN%
Mutator coverage: 0.00%
ERROR: below mutant coverage-threshold

Found behaviour
test.json file is empty
Expected behaviour
test.json should contain the a json with the correct info
Operating System

OS: Mac

Hello @rzeAkbari.
This has to be expected because main.go has no coverage. Maybe we can improve the reports when there is no actual coverage.

@rzeAkbari can you take care of this?

gladly :)

The problem comes form a division by zero issue. In case we have only not covered mutations report, efficacy becomes NAN, hence facing marshalling issues later on.
rep.tEfficacy = float64(rep.killed) / float64(rep.killed+rep.lived) * 100

tests efficacy should be calculated only if the number of killed is grater than zero, otherwise it keeps its zero value, so we avoid division by zero.
does this make sense to you @k3rn31 ?

Closed by #188


Allow Replay of Achievement Animations Only for Achievements Already Acquired

Idea Description
The feature request is to enable users to replay the animation of achievements they've already acquired. Users should only be able to replay animations for achievements listed in their "Achievements" section as already obtained.
Reference images and links

🙋♂

@Aniokrait Thanks! Assigned!


Update gradle version

Current gradle version is 0.12.2 which is 2 years old. Android studio complains for the gradle version and recommends 2.2.3.

Did you get chance to try upgrading it? Any idea about the work involved?

#35 This issue can be closed


Can I specify bucket custom path for the upload?

Question
Hello there,
I want to upload an objet to GCS bucket with a specified path, with tusd server connected to my bucket with the command tusd -gcs-bucket=my_bucket.
When I upload an objet, this objet name is a generate value on the bucket.
So I want to get something like My_bucket/custom_path/filename or My_bucket/custom_path/generate_id for the upload filename
How can I specify the name of the upload?

You can control the folder on GCS, in which the upload will be saved using -gcs-object-prefix. The filename will always be generated by tusd and cannot be controlled by you. If you want to have a specific filename on GCS, you must rename/move the object after the upload is complete.

it is possible to have multiple prefix ? with

No, the prefix must be specific when starting tusd and cannot be changed at runtime. In addition, there can only be one prefix.

Interesting, thank you. And is there a way to deploy TusD which allows a greater flexibility regarding the prefix or the file naming ? Maybe we have to use another version of TUS ?

If you want fully customized paths, we recommend to use tusd to upload file in one folder and them move/rename them according to your preferences. This way you have the stability of tusd but the flexibility you want.

We are always interested in PRs :) However, it would make sense to first discussion the actual feature proposal and its implementation in a separate issue.

Parameterizing the object key for s3store might be possible in the future using https://github.com/tus/tusd/pull/962, although this is not implemented yet.

This will be implemented in v3 (release date unknown yet) with https://github.com/tus/tusd/pull/1167.


fix: Make themeGet curry

Codecov Report

Merging #23 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #23   +/-   ##
=======================================
  Coverage   71.91%   71.91%
=======================================
  Files          12       12
  Lines         146      146
  Branches       19       20    +1
=======================================
  Hits          105      105
  Misses         31       31
  Partials       10       10

Impacted Files
Coverage Δ

src/Card/Card.js
100% <ø> (ø)
:arrow_up:

src/utils.js
100% <100%> (ø)
: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 045ae6a...bf11073. Read the comment docs.


MQTT Option

https://github.com/nats-io/jwt/pull/221

This is already supported, because the API here is only a string.


fix: cond.wait() not in for condition loop

Usually, cond.wait() needs to be placed in a loop because it is not possible to know whether the condition has been satisfied when the cond is awakened. This is also explained in the comments of cond.wait() in the official library.
Based on my test case, the following concurrency errors may occur if the code is not modified:

Reference:
https://pkg.go.dev/sync#Cond.Wait
https://www.reddit.com/r/golang/comments/s6tuq3/why_synccond_needs_a_condition_in_for_loop_why/

@adrianbrad

Good catch @haoheipi, indeed the cond.Wait() should be wrapped in a loop, I didn't know that until now, but it actually makes a lot of sense. I will merge and publish a new release.
Cheers!


remove duplicate article.rb-related content in expert template - caused ...

...template install to fail
There was some repeated text in the expert template which was causing the install to fail.
It was attempting to add text to the article.rb file before that file existed.  I excised the offending code.

bump.
The 03-expert.rb template still doesn't work, and in the mean time it seems more template rows have been added (like a gsub on a non-existing article_tests.rb file) that cause the template to still fail.

Are you running the templates in order?
The 02, 03... Templates won't run by themselves.
You have to run 01, then 02 on top of that, then 03 on top of that and so on.

@binaryphile @epologee Yeah, @dabit is right -- you have to run the templates on top of each other.
I think there's maybe an issue already open to put some kind of guards there, which would enforce it, and fail loudly. I must look into that.

I was indeed not running the templates progressively. Haven't had the time to check, but I assume all will work as intended once I do so.

@epologee Thanks! I'll put in some checks to prevent running the templates out of order, keeping this one open until it happens.

The templates now have checks to help steer people towards running these in order, so per @karmi's last comment I think we're good to close this now. 👍


Finalize preparations for CRAN

This includes things like:

Update final license information in DESCRIPTION
Pick a version number
R CMD check
Double check matching CRAN policies
Add README.md
Submit to CRAN
Plan publicizing (e.g., Journal of Statistical Software?)

Add CC0 license


Can test framework run tests towards radicale?

References #57

I wonder if this got stuck on #2?  Should look into it one of those days

Test code runs towards radicale now, if the following flags are set:

nofreebusy
noproxy
nodefaultcalendar
nocalendarnotfound

Should check out:

nofreebusy - is it a bug in caldav, a bug in radicale, or a missing feature?
noproxy - it's written something at https://radicale.org/2.1.html#documentation/reverse-proxy, is that relevant?

freebusy is on the road map

Closing this, considering radicale to be well-enough supported as for now.


Improve documentation

Currently it is a bit hard to understand the steps to reproduce in order to create a tunnel.
Especially because the commands are not specified to be for the local or for the remote environment
Having a step-by-step detailed documentation would help a lot.

We have a README which contains everything you need to start, basically, it's 2 commands:

kubetunnel install
sudo -E kubetunnel create-tunnel -p '8080:80' svc_name

Did you manage to use it? if not please use our slack channel and we'll gladly help


TypeError: forward() got an unexpected keyword argument 'images'

I'm trying to set up the finetuning script locally, but am getting this error when testing the model in train.py. I'm manually passing in a single sample from the lazy dataset, and the error makes sense - I don't see where in the code we actually adapt the model's forward function to actually use the vision tower. Am I missing a setup step or something?

Just ran the finetuning script fully, and am getting TypeError: forward() got an unexpected keyword argument 'images'

Hi, thank you for your interest in our work.  To better understand the issue, can you share the command that you use?

Yup - it's pretty much what you have outlined in the repo for finetuning:
torchrun --nnodes=1 --nproc_per_node=8 --master_port=25001 \\
        llava/train/train_mem.py \\
        --model_name_or_path {VICUNA_LOCAL_PATH} \\
        --data_path {LLAVA_DATA_LOCAL_PATH} \\
        --instruct_data_path {SELF_INSTRUCT_DATA_LOCAL_PATH} \\
        --use_pretrain_data True \\
        --instruct_data_proportion 0.25 \\
        --pretrain_data_proportion 0.25 \\
        --image_folder {COCO_2014_LOCAL_PATH} \\
        --vision_tower openai/clip-vit-large-patch14 \\
        --pretrain_mm_mlp_adapter {PRETRAIN_ADAPTER_LOCAL_PATH} \\
        --mm_vision_select_layer -2 \\
        --mm_use_im_start_end True \\
        --bf16 True \\
        --output_dir /opt/ml/checkpoints/mix_pretrain_0.25_instruct_0.25 \\
        --num_train_epochs 3 \\
        --per_device_train_batch_size 4 \\
        --per_device_eval_batch_size 4 \\
        --gradient_accumulation_steps 1 \\
        --evaluation_strategy 'no' \\
        --save_strategy 'steps' \\
        --save_steps 5000 \\
        --save_total_limit 3 \\
        --learning_rate 2e-5 \\
        --weight_decay 0. \\
        --warmup_ratio 0.03 \\
        --lr_scheduler_type 'cosine' \\
        --logging_steps 1 \\
        --tf32 True \\
        --fsdp 'full_shard auto_wrap' \\
        --fsdp_transformer_layer_cls_to_wrap 'LlamaDecoderLayer' \\
        --model_max_length 1024 \\
        --gradient_checkpointing True \\
        --lazy_preprocess True \\
        --report_to wandb

But yeah, I just don't see how the model is constructed to take in 'images' in the forward call, since we're using the LlamaForCausalLM class which doesn't handle multimodal inputs

Hi @jihan-yin, are you using the latest code base?  You can see from here that it will use LlavaLlamaForCausalLM when you provide --vision_tower in argument.
If you are using the latest code base, please share the full error log so that I can better understand the issue.  Thanks.

Ah, I am not using the latest version. Looks like I need to update - thanks

Yep, please try upgrading the code base, as for the older code base, you need to install our customized transformers package here.
Upgrading to the latest code base removes that dependency, and we can make upgrades directly in this main code base.  It also supports more Vicuna models and new features :)
Please let me know if you have further questions, thanks.

Hey @haotian-liu, now I am getting an error stating
File "/workspace/llava/llava/model/llava.py", line 430, in initialize_vision_tokenizer    assert input_embeddings.shape == embed_tokens_weight.shape | Traceback (most recent call last): File "llava/train/train_mem.py", line 13, in <module> train() File "/workspace/llava/llava/train/train.py", line 615, in train model.initialize_vision_tokenizer( File "/workspace/llava/llava/model/llava.py", line 430, in initialize_vision_tokenizer assert input_embeddings.shape == embed_tokens_weight.shape
```.

I am passing in `--pretrain_mm_mlp_adapter` as the pretrain checkpoint listed in the repo at https://huggingface.co/liuhaotian/LLaVA-13b-pretrain-projector-v0/blob/main/LLaVA-13b-pretrain-projector-v0-CC3M-595K-original_caption.bin. I'm still running with the same commands as above - should I be changing something?

@jihan-yin My apologies, there is a fix that was not merged to the latest code base.  I just applied that fix, could you please pull the latest code base and try again?  Thank you.

Ah I ended up applying the same fix locally, glad to see it's in main as well! Thanks for the fast response too, will patch and try again


[BUG] Queues keep linking to youtube link even if you load/play a spotify playlist/album (from link)

There is bug when you try to see queue of all spotify tracks.. they are displaying as youtube link sometimes.. not from spotify itself. This current bug have same issue with savedqueue commands for spotify playlist and the timestamps sometimes mentioned as NaN instead of how long the duration of one video/music.
This might happen on Now Playing stuff too (when the bot posting a new music that will be played). This could be an issue from database files.
NaN stuff

There is bug when you try to see queue of all spotify tracks.. they are displaying as youtube link sometimes.. not from spotify itself. This current bug have same issue with savedqueue commands for spotify playlist and the timestamps sometimes mentioned as NaN instead of how long the duration of one video/music.
This might happen on Now Playing stuff too (when the bot posting a new music that will be played). This could be an issue from database files.
NaN stuff

playing songs with spotify links will scrape data and find it on youtube, so technically it is not a bug

That's not a bug as already explained,
Spotify tracks are UNRESOLVED. which means just the title and requester are available
When they get LOADED (aka on "turn", aka queue pos 0) they are fetched, which means only than they get link, duration etc. (from youtube fetched)


Build is showing Mobile VR in Oculus Quest

Im having a weird issue.  I imported the Unity Package, built the desert scene, pushed it to my server and loaded the page up in my Oculus Quest.  When I hit the "headset" button, it launches into a mobile VR view and not a full immersive Quest view.  Anything I am missing?  Did I miss a setup step?  using Unity 2019.3.7f1 - using Oculus Browser

Ok, so I had to switch browsers apparently in Oculus Browser to "Request Desktop Version" in the top right, instead of "Request Mobile Version" FYI.

SOLVED:  You MUST use HTTPS in the URL, not HTTP.  For my problem, I had to go to adavanced settings on the page, allow access to the page, hit the VR button and allow the activation.  Voila, it worked.  No resolution issues either.  Verified - Unity 2019.3.7f1 - URP - Oculus Quest - Oculus Browser - Mobile Mode


grammar/spelling in readme

fixes #68
this fixes the grammar and spelling in README.md as request in #68

done as requested

@Adarsh-jaiss @Ebukachuqz Please review and will merge, looks good to me.

All good. Ready to be merged.

Thanks.

@05e Thanks for your contribution, make sure you join this organization/community by creating a issue in our repo become-a-member. Thanks again :)!

looks good :)


Bug: Different DB drivers have different types of Exceptions

Ref #4331
When DBDebug is true.
If you set it false, the Exception will be gone and the query returns false.
MySQLi:
1) CodeIgniter\Database\Live\DbErrorTest::testForeignKeyConstraintFailsDBDebugTrue
mysqli_sql_exception: Cannot add or update a child row: a foreign key constraint fails (`test`.`db_auth_identities`, CONSTRAINT `db_auth_identities_user_id_foreign` FOREIGN KEY (`user_id`) REFERENCES `db_auth_users` (`id`) ON DELETE CASCADE)

SQLite3:
1) CodeIgniter\Database\Live\DbErrorTest::testForeignKeyConstraintFailsDBDebugTrue
ErrorException: SQLite3::exec(): FOREIGN KEY constraint failed

Postgre:
1) CodeIgniter\Database\Live\DbErrorTest::testForeignKeyConstraintFailsDBDebugTrue
ErrorException: pg_query(): Query failed: ERROR:  insert or update on table "db_auth_identities" violates foreign key constraint "db_auth_identities_user_id_foreign"
DETAIL:  Key (user_id)=(2) is not present in table "db_auth_users".

OCI8:
 1) CodeIgniter\Database\Live\DbErrorTest::testForeignKeyConstraintFailsDBDebugTrue
ErrorException: oci_execute(): ORA-00942: table or view does not exist

SQLSRV:
 1) CodeIgniter\Database\Live\DbErrorTest::testForeignKeyConstraintFailsDBDebugTrue
Exception: [Microsoft][ODBC Driver 17 for SQL Server][SQL Server]The INSERT statement conflicted with the FOREIGN KEY constraint "db_auth_identities_user_id_foreign". The conflict occurred in database "test", table "dbo.db_auth_users", column 'id'.

Fixed by #6163


[Linux] Not clearing backbuffer before drawing.

On the main branch, I created this very simple Flutter app:
import 'package:flutter/material.dart';

/// Flutter code sample for [MenuAnchor].

void main() {
  debugFocusChanges = false;
  runApp(const MenuApp());
}

class MenuApp extends StatelessWidget {
  const MenuApp({super.key});

  @override
  Widget build(BuildContext context) {
    final ThemeData theme = ThemeData.from(colorScheme: const ColorScheme.light());

    return MaterialApp(
        theme: theme,
        home: Center(
          child: Column(
            children: <Widget>[
              FilledButton.tonal(
                onPressed: () { },
                child: const Text('Tonal'),
              ),
              FilledButton.tonalIcon(
                onPressed: () { },
                icon: const Icon(Icons.add),
                label: const Text('TonalIcon'),
              ),
            ],
          ),
        ),
      );
  }
}

And ran it on my Linux machine,  I got a window that didn't clear the backbuffer when switching buffers, leaving an interesting artifact filled image.
https://github.com/flutter/flutter/assets/8867023/8758aa17-d1ea-4ea7-8173-ff86bdb38136

flutter doctor -v
[✓] Flutter (Channel main, 3.20.0-7.0.pre.31, on Debian GNU/Linux rodete 6.5.13-1rodete2-amd64, locale en_US.UTF-8)
    • Flutter version 3.20.0-7.0.pre.31 on channel main at /usr/local/google/home/gspencer/code/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (57 minutes ago), 2024-02-13 21:48:41 +0000
    • Engine revision 0091a4914d
    • Dart version 3.4.0 (build 3.4.0-138.0.dev)
    • DevTools version 2.33.0-dev.6

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /usr/local/google/home/gspencer/Android/Sdk
    • Platform android-33, build-tools 32.1.0-rc1
    • ANDROID_HOME = /usr/local/google/home/gspencer/Android/Sdk
    • Java binary at: /usr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+7-Debian-1)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Debian clang version 16.0.6 (16)
    • cmake version 3.27.7
    • ninja version 1.11.1
    • pkg-config version 1.8.1

[!] 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/linux#android-setup for detailed instructions).

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Debian GNU/Linux rodete 6.5.13-1rodete2-amd64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 121.0.6167.160

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

I could not reproduce this on my system using the example provided above.
Labeling based on the report shared above.
Screencast from 14-02-24 08:52:02.webm

flutter doctor -v
[!] Flutter (Channel master, 3.20.0-7.0.pre.43, on Ubuntu 22.04.3 LTS 6.5.0-17-generic, locale en_US.UTF-8)
    • Flutter version 3.20.0-7.0.pre.43 on channel master at /home/daniel/fvm/versions/master
    ! Warning: `flutter` on your path resolves to /home/daniel/fvm/versions/stable/bin/flutter, which is not inside your current Flutter SDK checkout at /home/daniel/fvm/versions/master. Consider adding /home/daniel/fvm/versions/master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /home/linuxbrew/.linuxbrew/Cellar/dart/3.2.2/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /home/daniel/fvm/versions/master. Consider adding /home/daniel/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 295eeaf100 (5 hours ago), 2024-02-13 19:58:58 -0800
    • Engine revision 3af336bfb2
    • Dart version 3.4.0 (build 3.4.0-140.0.dev)
    • DevTools version 2.33.0-dev.6
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/daniel/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /home/daniel/.local/share/JetBrains/Toolbox/apps/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1.1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2023.1)
    • Android Studio at /home/daniel/.local/share/JetBrains/Toolbox/apps/android-studio
    • 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 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.85.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.82.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 22.04.3 LTS 6.5.0-17-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 120.0.6099.129

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

I'm able to replicate this on Ubuntu 22.04 running with XDG_SESSION_TYPE=x11.

/cc @robert-ancell

Does it occur with the rendering changes in https://github.com/flutter/engine/pull/50754?

I can reproduce this using flutter master on Ubuntu 23.10 using Wayland.

Confirmed the new rendering changes don't have this issue - the background is now black:

I suggest we consider this resolved once that PR lands.

I'm using X11: XDG_SESSION_TYPE=x11

@cbracken I am using wayland

Awesome thanks. Sounds like once flutter/engine#50754 lands, this is likely to be closeable.

https://github.com/flutter/engine/pull/50754 has landed. @gspencergoog is this working for you at tip-of-tree (I don't have my linux machine here today).


Login with XING does not work

Hello,
I tried implementing the HybridAuth options.
I configured both LinkedIn and Xing.
And while LinkedIn works without problem Xing does not.
When I click the Button I get the following Messages Logged.
[2018-02-21 17-18-53] PHP Notice: Undefined index: oauth_token in /var/www/html/YAWIK/vendor/hybridauth/hybridauth/hybridauth/Hybrid/thirdparty/OAuth/OAuth1Client.php:85  @  http://stellenboerse.dev.example.com/login/hauth?hauth.start=XING&hauth.time=1519229932
[2018-02-21 17-18-53] PHP Notice: Undefined index: oauth_token_secret in /var/www/html/YAWIK/vendor/hybridauth/hybridauth/hybridauth/Hybrid/thirdparty/OAuth/OAuth1Client.php:85  @  http://stellenboerse.dev.example.com/login/hauth?hauth.start=XING&hauth.time=1519229932
[2018-02-21 17-18-53] Zend\ServiceManager\Exception\ServiceNotCreatedException: Service with name "HybridAuth" could not be created. Reason: Authentication failed! XING returned an error: 401 Unauthorized.. in /var/www/html/YAWIK/vendor/zendframework/zend-servicemanager/src/ServiceManager.php:765 caused by Exception: Authentication failed! XING returned an error: 401 Unauthorized.. in /var/www/html/YAWIK/vendor/hybridauth/hybridauth/hybridauth/Hybrid/Auth.php:169  @  http://stellenboerse.dev.example.com/login/xing?ref=%252Flogin%252Fxing  @@  exception--2018-02-16--18-26--3b16327f02.html

I triple checked the right key and secret (The fact that Xing is key and every thing else is id was a little disorienting as well)
But to be specific, I did:

Go to https://dev.xing.com/plugins/login_with/new and Input my URL
Copied the Info from Xing to the file autoload/module.auth.local.php
added xing to 'enableLogins' => ['xing', 'linkedin'],

I did so 5 days ago (and just now again to make sure I did it right).
Does anyone know what the two extra errors mean?
Also is there a way for me to see the actual call to Xing, to see if the right key etc. even gets there?
Sincerely
Marvin

YAWIK authenticates agains a XING app. Afaik XING does not support apps any more. At least it's not possible to add apps at  https://dev.xing.com.
Authentication in our Demo is currevtly working, because it uses a Xing app.
The YAWIK auth module has propably modified to support XING plugins.


Upgrade zlib

Details and background can be found in #51. This upgrades zlib to include fixes and additions up to node 6.
Closes #51

This is fixed in<EMAIL_ADDRESS>http://npmjs.com/browserify-zlib


In the case of consecutive calls of invalidate function, it behaves strange

Describe the bug
Actually I could not find any information about how the invalidate function works. I think of that it returns 'href's to the function you passed as paremeter, and you check the href, then decide to invalidate the href by returning true or false.
I have radio buttons that allows to change the language of my application. When a radio button changed, I am checking that href parameters comes from the invalidate function are in my multiLanguagePaths array or not.

The problem is that, href parameters comes correctly at the first call, but following subsequent calls, it does not return href parameters correctly. Although the parameters are not correct, it makes correct call.
Also it calls API's and page endpoints twice, every time.

Reproduction
  input.addEventListener('change', () => {
    $session.language = input.id
    document.cookie = `language=${input.id}; Max-Age=${60 * 60 * 24 * 30}; Path=/; SameSite=Lax;`

    // does not work for standalone endpoints, I have a load function that fetches data in my __layout.svelte
    // $multiLanguagePaths.forEach(path => {
    //   console.log('http://localhost:3000' + path)
    //   invalidate('http://localhost:3000' + path)
    // })

    console.log($multiLanguagePaths, '\n\n')

    invalidate(href => {
      const parts = href.split('/')

      console.log(href)
      for (let i = parts.length - 1; i > 0; i--)
        if ($multiLanguagePaths.includes('/' + parts[i])) {
          console.log('invalidate calling for ', `/${parts[i]}`)
          return true
        }

      return false
    })
  })

Logs
Array(3) [ "/", "/product", "/getInfoForLayout" ]

http://<IP_ADDRESS>:3000/api/getInfoForLayout
invalidate calling for  /getInfoForLayout
http://<IP_ADDRESS>:3000/api/getInfoForLayout
invalidate calling for  /getInfoForLayout
XHR GET http://<IP_ADDRESS>:3000/api/getInfoForLayout [HTTP/1.1 304 Not Modified 3ms]
XHR GET http://<IP_ADDRESS>:3000/api/getInfoForLayout [HTTP/1.1 304 Not Modified 3ms]

http://localhost:3000/product/Product2
invalidate calling for  /product
http://localhost:3000/product/Product2
invalidate calling for  /product
XHR GET http://localhost:3000/product/Product2/__data.json [HTTP/1.1 304 Not Modified 3ms]
XHR GET http://localhost:3000/product/Product2/__data.json [HTTP/1.1 304 Not Modified 3ms]

Array(3) [ "/", "/product", "/getInfoForLayout" ]

http://localhost:3000/product/Product2
invalidate calling for  /product
http://localhost:3000/product/Product2
invalidate calling for  /product
XHR GET http://<IP_ADDRESS>:3000/api/getInfoForLayout [HTTP/1.1 304 Not Modified 3ms]
XHR GET http://<IP_ADDRESS>:3000/api/getInfoForLayout [HTTP/1.1 304 Not Modified 3ms]

http://localhost:3000/product/Product2
invalidate calling for  /product
http://localhost:3000/product/Product2
invalidate calling for  /product
XHR GET http://localhost:3000/product/Product2/__data.json [HTTP/1.1 304 Not Modified 6ms]
XHR GET http://localhost:3000/product/Product2/__data.json [HTTP/1.1 304 Not Modified 3ms]

System Info
System:
    OS: Linux 5.18 Fedora Linux 36 (Workstation Edition)
    CPU: (2) x64 Intel(R) Pentium(R) CPU G2010 @ 2.80GHz
    Memory: 1.45 GB / 7.66 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 16.14.0 - /usr/bin/node
    npm: 8.3.1 - /usr/bin/npm
  Browsers:
    Firefox: 102.0
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.50
    @sveltejs/kit: next => 1.0.0-next.350
    svelte: ^3.44.0 => 3.48.0

Severity
serious, but I can work around it
Additional Information
No response

Actually I could not find any information about how the invalidate function works.

I haven't looked at this issue yet, but wanted to make sure you knew there is documentation for invalidate as it sounds like you didn't see it. Let us know if there's something unclear in the docs. https://kit.svelte.dev/docs/modules#$app-navigation-invalidate

A lot has changed since this was opened, and I cannot reproduce this given the reproduction. If this still happens, please provide a  full reproduction (ideally in form of a repository, for example through stackblitz).

Closing due to inactivity; this may have been fixed. If not, please open a new issue.


Online documentation structure

This ticket tracks the first design of the structure of the contents included in the online documentation

Addressed in #24