Selectable library loaders, including link-time

Resolves #16, as per my proposal there. Alternative to #22.
I wanted to do the first commit for OpenSSL 3 support anyway, and it provided an alternative solution for #16. It does feel slightly more invasive than I'd like.
@TurtleP can you check whether this works for you?

Alright, so I have set up my CMakeLists.txt to pull in your branch via FetchContent @bartbes (and set up LÖVE Potion as needed), but I am running into the following error (which might be where I need to patch things on my end):
G:/GitHub/C++/lovepotion/build/_deps/lua-https-src/src/common/HTTPS.cpp:70:
(.text._Z41__static_initialization_and_destruction_0v+0x4):
undefined reference to `LibraryLoader::GetCurrentProcessHandle()'

I've set LIBRARY_LOADER to linktime and USE_CURL_BACKEND to ON. I don't think I missed any files I need, either:
add_library(lua-https STATIC
    ${lua-https_SOURCE_DIR}/src/common/config.h
    ${lua-https_SOURCE_DIR}/src/common/LibraryLoader.h
    ${lua-https_SOURCE_DIR}/src/common/Connection.h
    ${lua-https_SOURCE_DIR}/src/common/ConnectionClient.h
    ${lua-https_SOURCE_DIR}/src/common/HTTPRequest.cpp
    ${lua-https_SOURCE_DIR}/src/common/HTTPRequest.h
    ${lua-https_SOURCE_DIR}/src/common/HTTPS.cpp
    ${lua-https_SOURCE_DIR}/src/common/HTTPS.h
    ${lua-https_SOURCE_DIR}/src/common/HTTPSClient.cpp
    ${lua-https_SOURCE_DIR}/src/common/HTTPSClient.h
    ${lua-https_SOURCE_DIR}/src/common/PlaintextConnection.cpp
    ${lua-https_SOURCE_DIR}/src/common/PlaintextConnection.h
    ${lua-https_SOURCE_DIR}/src/generic/LinktimeLibraryLoader.cpp
    ${lua-https_SOURCE_DIR}/src/generic/CurlClient.cpp
    ${lua-https_SOURCE_DIR}/src/generic/CurlClient.h
    ${lua-https_SOURCE_DIR}/src/generic/OpenSSLConnection.cpp
    ${lua-https_SOURCE_DIR}/src/generic/OpenSSLConnection.h
    ${lua-https_SOURCE_DIR}/src/lua/main.cpp
)

set(LIBRARY_LOADER "linktime")
set(USE_CURL_BACKEND ON)

I see you’ve set LIBRARY_LOADER, but that is a cmake-only variable. Is the define HTTPS_LIBRARY_LOADER_LINKTIME set? Otherwise LinktimeLibraryLoader.cpp will be ifdeffed out.

Ah, gotcha.. didn't see that part of it. It built the 3DS binary, will check if this works shortly.

Alright, it kind of works. I got the following error, which I vaguely remember how I solved it last time, but I'm not 100% sure. If memory serves it failed the connection test. I might be misremembering, though?
[LOVE] Error: [string "main.lua"]:5: No applicable HTTPS implementation found
stack traceback:
        [string "boot.lua"]:423: in function <[string "boot.lua"]:419>
        [C]: in function 'assert'
        [string "main.lua"]:5: in main chunk
        [C]: in function 'require'
        [string "boot.lua"]:391: in function <[string "boot.lua"]:143>
        [C]: in function 'xpcall'
        [string "boot.lua"]:433: in function <[string "boot.lua"]:426>
        [C]: in function 'xpcall'       1`

The version of cURL we have uses certificates that are placed on the SD Card, but I doubt it's expired by now, but I can also check that.

Digging further, it's not even enabling cURL as a backend:
HTTPSClient::Reply request(const HTTPSClient::Request& req)
{
    std::printf("request\n");
    for (size_t i = 0; clients[i]; ++i)
    {
        HTTPSClient& client = *clients[i];
        std::printf("client.valid()? %d\n", client.valid());
        if (client.valid())
            return client.request(req);
    }
    throw std::runtime_error("No applicable HTTPS implementation found");
}

This doesn't even print inside the for loop. I'm sure I missed something stupid, going to check the source a bit closer to see what it might be.

Okay, got it to work with the following:
set(LIBRARY_LOADER "linktime")

add_library(lua-https STATIC
    ${lua-https_SOURCE_DIR}/src/common/config.h
    ${lua-https_SOURCE_DIR}/src/common/LibraryLoader.h
    ${lua-https_SOURCE_DIR}/src/common/Connection.h
    ${lua-https_SOURCE_DIR}/src/common/ConnectionClient.h
    ${lua-https_SOURCE_DIR}/src/common/HTTPRequest.cpp
    ${lua-https_SOURCE_DIR}/src/common/HTTPRequest.h
    ${lua-https_SOURCE_DIR}/src/common/HTTPS.cpp
    ${lua-https_SOURCE_DIR}/src/common/HTTPS.h
    ${lua-https_SOURCE_DIR}/src/common/HTTPSClient.cpp
    ${lua-https_SOURCE_DIR}/src/common/HTTPSClient.h
    ${lua-https_SOURCE_DIR}/src/common/PlaintextConnection.cpp
    ${lua-https_SOURCE_DIR}/src/common/PlaintextConnection.h
    ${lua-https_SOURCE_DIR}/src/generic/LinktimeLibraryLoader.cpp
    ${lua-https_SOURCE_DIR}/src/generic/CurlClient.cpp
    ${lua-https_SOURCE_DIR}/src/generic/CurlClient.h
    ${lua-https_SOURCE_DIR}/src/generic/OpenSSLConnection.cpp
    ${lua-https_SOURCE_DIR}/src/generic/OpenSSLConnection.h
    ${lua-https_SOURCE_DIR}/src/lua/main.cpp
)

target_compile_definitions(lua-https PRIVATE
    HTTPS_LIBRARY_LOADER_LINKTIME
    HTTPS_BACKEND_CURL
)

# link curl
pkg_check_modules(libcurl REQUIRED IMPORTED_TARGET libcurl)
target_link_libraries(lua-https PRIVATE PkgConfig::lua51 PkgConfig::libcurl)

It's likely because of the way I'm doing this that lua-https isn't happy with certain things. If there's a better way for me to do this, let me know.


Support @blocks in CSS/SCSS

I realize CSS/SCSS isn't in the official list of supported languages, but it works, so apparently they are supported in some fashion. However, @blocks (e.g., @media queries in CSS and @include block-level mixins in SCSS) don't show up as part of the node tree. It would be great if they did.

That's up to whichever CSS/SCSS plugin you're using. Code Outline does not do any language parsing and relies on the symbol providers shipped with language support plugins.

There is no plugin, it seems to be built into VS Code. But thanks, that
makes sense.
--
Corey Caswick
Chatman Design
chatmandesign.com http://www.chatmandesign.com
On Fri, May 18, 2018 at 2:56 AM, Patryk Zawadzki<EMAIL_ADDRESS>wrote:

That's up to whichever CSS/SCSS plugin you're using. Code Outline does not
do any language parsing and relies on the symbol providers shipped with
language support plugins.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/patrys/vscode-code-outline/issues/128#issuecomment-390126662,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE1wfmhWT8-VmsgYM6-H315BFV92BwLwks5tzn6bgaJpZM4UD1V-
.


Port Scan

Wondering if you can help. im using a server at hetzner and tried using the scraper ( the old one before you updated ) and 5 mins later i got this email below.. now what do you reccomend? kind regards.
We have indications that there was an attack from your server.
Please take all necessary measures to avoid this in the future and to solve the issue.
We also request that you send a short response to us. This response should contain information about how this could have happened and what you intend to do about it.
In the event that the following steps are not completed successfully, your server can be blocked at any time after the 2018-06-25 14:27:04 +0200.
##########################################################################

Portscan detected from host  xxxx
##########################################################################
time                protocol src_ip src_port          dest_ip dest_port
Sun Jun 24 00:24:35 2018 UDP   xxxx 6881  =>      <IP_ADDRESS> 59336
Sun Jun 24 00:24:17 2018 UDP   xxxxxx 6881  =>   <IP_ADDRESS> 60693
Sun Jun 24 00:26:15 2018 UDP  xxxxx 6881  =>   <IP_ADDRESS> 60693
Sun Jun 24 00:22:54 2018 UDP   xxxxxx 6881  =>     <IP_ADDRESS> 49453

I recommend changing hosting providers... sometimes things can be somewhat "over secure", for a lack of better terms, and cause problems and headaches.
I'm guessing you can not disable this from your hosting control panel or whatever you use??? I would say the network nerds that work for this company monitor their entire network/data center and they detected it... which means it's nothing you can disable or do.
Before the support tickets get flying and emails get exchanging, it's probably not a good idea to raise much hell about it. I've personally been booted from 2 different hosting providers in the past for pretty much the same thing. Hosting companies don't even want the risk of hosting something like Alpha Reign (or any DHT torrent/movie/pirated content site).
 I've been terminated for violation of ToS, literally a few days after paying for a year in advance. Guess who didn't get a refund and threatened with legal actions when demanding my money back?
Yep, this guy... right here. Its really not worth the trouble and bs!

WarezAddict is probably right.
The thing is, is that AlphaReign pushes a lot of data, and that coming from one server on tons of different ports looks like an attack.  For the hosting provider, it's easier to just shut down the one server than look into it and investigate it.

BTW, I forgot to mention this....
The logs and IPs you see in that email are completely normal... since somebody, not you (remember?  Depending on the settings in your config, it also uses massive amounts of bandwidth and makes any monitor/admin go crazy!

I manged to fix this by dropping packets to reserved ips in iptables. Thanks for your support

Awesome!


Upgrade ttrpc to avoid fd leak

The ttrpc had a fd leak issue that was fixed by https://github.com/containerd/ttrpc/commit/96dcf73d208d67a3f88a173be3a23731cb62c2be. And the relate fix of containerd was https://github.com/containerd/containerd/pull/3496/commits/08483d18adea186c75463305d4040d218e5d68d4. But the fd leak will still happen when using kata because kata is using an old version of ttrpc.
/cc @egernst
/cc @YongjiXie

Hey @darfux — thanks for opening the issue. Will take a look - sounds like revendoring is needed.

@lifupan @bergwolf can you PTAL?


fix input of file path + add test to cover it

Checklist

[x] read and understood the contributing guidelines
[x] updated tests
[x] npm run build and npm test passes

(e2e failed due to missing exe at \packages\primitive\vendor\primitive-win32-x64.exe)

Affected core subsystem(s)

cli -> search

Description of change
fixed issue with single file path for CLI input, e.g.:
input: path/to/single.png

Without fix:
           ┌─────────────────┐
           │    All done!    │
           │                 │
           │  √ 0 completed  │
           └─────────────────┘

With fix:
           ┌─────────────────┐
           │    All done!    │
           │                 │
           │  √ 1 completed  │
           └─────────────────┘

I hope the tests are okay, I'm new to generators 😅

Thank you for the contribution, I really appreciate it!
I'll try to get on this early next week.


Add support for ng2 attributes

Was disregarding any html element that included angular2 input attributes [input]or directive attributes *ngFor.
We're using this in an angular 2 project with webpack, but it's currently skipping over everything we'd really like it to be scrutinizing.
Also, added a rule for attr-no-title-case, since angular 2 directives use camelCase, attr-lowercase doesn't quite cut it.

Submitted updated pull request (#114) with additional rules, also accommodated (outputs)


sink(ticdc): Support Multiple Downstream Addresses for MySQL

What problem does this PR solve?

Issue Number: close #11475
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/test verify
/test cdc-integration-kafka-test

/test verify

/test cdc-integration-kafka-test

/test verify

/test cdc-integration-mysql-test
/test dm-integration-test

/test cdc-integration-mysql-test
/test dm-integration-test

/test cdc-integration-mysql-test

/test cdc-integration-kafka-test

/test cdc-integration-pulsar-test


How to draw the top down view of the scene with graph connectivity?

Hi, thank you for sharing your code. I am just wondering how to get the top down view as shown in your paper?

Hi @Zhongxyyyy, we render the orthogonal view of the environment in MeshLab and obtain the top-down view by taking a snapshot from the top.

Thanks for your reply! I use MeshLab to render the *.ply and get the rendered building view. But how to draw the graph connectivity of the building in top-down view?

Thanks for your reply! I use MeshLab to render the *.ply and get the rendered building view. But how to draw the graph connectivity of the building in top-down view?

Hi, I have the same question here. Have you found out how to render the top-down view and visualize the connectivity?
Thanks a lot!


Cryptocurrency Prices

You can get historical cryptocurrency pricing from Quandl, but the MarketFlow pipeline needs to be modified to read directly from the data directory if no feeds are available.
Here are some sources of historical daily and intraday cryptocurrency data:

Daily : https://www.kaggle.com/sudalairajkumar/cryptocurrencypricehistory

Intraday (1-minute) : https://www.kaggle.com/smitad/bitcoin-trading-strategy-simulation/data

Our next data feed integration for cryptocurrencies:
https://www.cryptocompare.com/api/

is it possible to use pandas_datareader to snatch from yahoo instead? couldn't grab stocks from my country.

Cryptocurrency data will eventually be supported through the IEX, but historical data will have to be obtained from a difference source.
https://intercom.help/iexcloud/en/articles/3377978-cryptocurrency-data-on-iex-cloud

We can pull historical data through Binance, to be done in a future release.


[Java] MapVector written data does not meet expectations

What would you like help with?
    RootAllocator allocator = new RootAllocator(Long.MAX_VALUE);
    Map<String, String> metadata = new HashMap<>();
    metadata.put("K1", "V1");
    metadata.put("K2", "V2");
    Field a = new Field("key", FieldType.notNullable(new ArrowType.Int(32, true)), null);
    Field b = new Field("value", FieldType.nullable(new ArrowType.Int(32,true)), null);
    Schema schema = new Schema(asList(a, b), metadata);
    Field struct = new Field("struct", FieldType.notNullable(new ArrowType.Struct()), schema.getFields());
    StructVector structVector = new StructVector(Field.nullable("struct", struct.getType()), allocator, null);
    MapVector mapVector = new MapVector(Field.nullable("map", structVector.getField().getType()), allocator, null);
    mapVector.initializeChildrenFromFields(Collections.singletonList(struct));
    mapVector.allocateNew();

    UnionMapWriter writer = mapVector.getWriter();
    for (int i = 0; i < 10; i++) {
        writer.startMap();
        writer.setPosition(i);
        writer.key().writeInt(i);
        writer.value().writeInt(i * 10);
        writer.endMap();
    }
    writer.setValueCount(10);

when the types of key and value are different, data cannot be written .  Please provide an example of writing data to MapVector

I think this belongs on apache/arrow and not apache/arrow-adbc?

yes ,  here are the dependencies
        <dependency>
            <groupId>org.apache.arrow</groupId>
            <artifactId>arrow-vector</artifactId>
            <version>${arrow.version}</version>
        </dependency>
        <dependency>
            <groupId>org.apache.arrow</groupId>
            <artifactId>arrow-memory-netty</artifactId>
            <version>${arrow.version}</version>
        </dependency>

@lidavidm  Is there a plan to provide the correct MapVector example on the official website

@hawk9821 sorry for the delay, if I can find some time I will try to add a Cookbook example


[doc,hw] Entropy source IP initialization documentation

The initialization documentation of the entropy source has an argument rate which is not used. How do we use this properly?
More generally, how can I check that these initialization functions for other IP blocks are correct? For example, the initialization function of the CSRNG mentions CTRL_REG, where can I find this back in the code?

For the sceond part, I believe the name of the register comes before _REG part always. So if I would see anything that ends with _REG, I would copy the first part of it and search it throughout the register table in the documentation.

That clears up the second part, the only thing left to figure out is what to change in the entropy source initialization function.

For what it's worth, there are these two functions:

dif_entropy_src_init defined at https://github.com/lowRISC/opentitan/blob/master/sw/device/lib/dif/autogen/dif_entropy_src_autogen.c#L14
dif_entropy_src_configuredefined at https://github.com/lowRISC/opentitan/blob/master/sw/device/lib/dif/dif_entropy_src.c#L16

We could refer to those in the documentation?


func to the power

if you put a var in an () like (x) and give it a power like (x^2) it works fine, but if you add to that another power like (x^2)^2 it crushes,
this works with every power, i just did ^2 as an example

Okay

Fixed in 5359c8b6854e3f491a1d816bff92e00940253a75

if you're curious, the problem was the I created a new power function for each instance of it instead of alrady creating it once


"Info" button inactive: no info window displayed when clicked

Description: The "Info" button on the home page does not respond when clicked. Although the button is visually displayed and appears to be interactive, it does not perform any action when pressed.
Requirements:
Be in the home page
Steps:

Enter to the web page
When you are on the home page scroll down until you see the buttons on the footer.
Click on the button

Expected result:
When I click on  the button, a window should appear or redirect you to another place to be able to see information about the application
Actual result:
When you click on the button it does absolutely nothing.

@AngheloZambrana  email with feedback was sent to your way


Fixes #246: Add option --auto to set proxy option autoRewrite

add handling for --auto flag

@kwesterfeld looks like we have some merge conflicts to take a look at

Given the variety of options available in http-proxy, I'm going to close this in favor of the approach of passing through options to http-proxy itself in #459, which would solve this same issue


/tpa and /tpahere not appearing for receiving player

When I send a /tpa or /tpahere to anyone on the server, they don't receive anything. When I try to send another one it says there's already an outstanding request. Have tested on Atlas and Orion, haven't tried Titan.

Closing the issue as the messages can be seen and are working on all 3 servers.
If the issue re-occurs feel free to re-open the issue.


added post layout

(after reviewing the contribution guidelines I realized I should have posted a before and after of what the page would look like)
This what a normal blog post would look like using the default layout:

Added post layout from pages-themes/minimal (thanks @jmuccigr). Tested with scripts/cibuild (I don't think it actually ran any tests) and this is what posts now look like:

Npm:update:probot/@alethebat

|-c|{("show-con./files)}


Make the version info show MSIX package version

Description

Make the app's version info show the MSIX package version instead of assembly version.
Motivation and Context

I hope the app show the MSIX package version to reflect the version info in Microsoft Store.
How Has This Been Tested?

Tested with VS2022 on Windows 11 build 25357.
Screenshots (if appropriate):
none
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

@microsoft-github-policy-service agree

@realYulin Thanks for your PR.  Unfortunately, it would regress this fix: https://github.com/microsoft/WinUI-Gallery/commit/18c12764f3f1374bc846ebcf1fae481880a2711b
The version number reported by System.Reflection.Assembly.GetEntryAssembly().GetName().Version should be the same as that reported by Windows.ApplicationModel.Package.Current.Id.Version, or we have an issue somewhere else.

@Scottj1s If this regresses https://github.com/microsoft/WinUI-Gallery/commit/18c12764f3f1374bc846ebcf1fae481880a2711b, should this PR changed or rather closed if the intended behavior is not desirable?

@chingucoding It should at least be changed.  @realYulin, closing this PR - please feel free to open another (or an issue) if necessary.


Added the "InVision Design Engineering Handbook (PDF)"

I have added a link to the InVision Design Engineering Handbook (PDF) in the "Graphical User Interface" section. Please note that while the content is free to access, it does require users to provide their email addresses to obtain the PDF.
Reason for the Changes:
The InVision Design Engineering Handbook is a valuable resource, and I believe it could be beneficial to users of this repository. While it does request an email address, the content is provided for free, which aligns with the guidelines of this repository.
Additional Notes:
I have added a note next to the link to inform users that an email address is requested but the content is free. If there are any concerns or if this addition doesn't meet the repository's criteria, please let me know, and I can make any necessary adjustments.
What does this PR do?
Add resource(s)
For resources
Description
I've added a link to the "InVision Design Engineering Handbook (PDF)" in this pull request. This handbook provides valuable insights into design engineering practices.
Why is this valuable (or not)?
The "InVision Design Engineering Handbook" is valuable as it offers knowledge and best practices in the field of design engineering, which can be beneficial to developers and designers looking to enhance their skills and understanding of design processes.
How do we know it's really free?
The resource linked here is freely accessible without any direct cost. However, it does request users to provide an email address to access the PDF. While this may be seen as a minor requirement, it's important to note that the content itself is provided without charge.
For book lists, is it a book? For course lists, is it a course? etc.
This resource is a PDF document, which aligns with the guidelines for adding books to the list.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

I did not need to enter an email address.
Don't invent titles! Use the title given by the resource.
This book does not appear to be about "Graphical User Interfaces".

Oh true,lol,I misread the section. I will include it in the Misc section
and apply the needed changes then.
Thank you for your input.
On Wed, Sep 27, 2023 at 4:59 PM Eric Hellman @.***>
wrote:

I did not need to enter an email address.
Don't invent titles! Use the title given by the resource.
This book does not appear to be about "Graphical User Interfaces".
—
Reply to this email directly, view it on GitHub
https://github.com/EbookFoundation/free-programming-books/pull/9538#issuecomment-1737678475,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARX37OCVP23FFVG7C5N4SH3X4RENDANCNFSM6AAAAAA5ISYQGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


PLEASE STOP CHANGING URL every 90 days !

Guide
https://github.com/iptv-org/epg
Description
PLEASE STOP CHANGING URL every 90 days !  LEAVE it hasCOUNTRYURL FFS !

Thank you for your feedback. The Country URLs were more than trouble than it was originally anticipated hence why the project has gone back to the Source URLs.
You can help us make the project better by contributing to it.
See something that can be improved? Submit a pull request.
Thank you for investing your time in the project!


Added link ip6tables with /user/iptables as target

This is just to make ip6tables available in path with iptables.
Some containers accessing podman from inside of a container, needs ip6tables
Error response from daemon: error configuring network namespace for container blahblahblah: error adding pod drone-blahblah_drone-blahblah to CNI network "drone-blahblah": could not initialize iptables protocol 1: exec: "ip6tables": executable file not found in $PATH

More details documented in this blog post.

Thanks!


Implement PKCE

See https://snyk.io/vuln/npm:oauth2-server
https://github.com/oauthjs/node-oauth2-server/pull/452

Related: rfc7636

I started this here: #86

Hi,
if can be of any help, this is the full list of pr related to PKCE, in the original project:

https://github.com/oauthjs/node-oauth2-server/pull/452
https://github.com/oauthjs/node-oauth2-server/pull/658
https://github.com/oauthjs/node-oauth2-server/pull/659

Regards.

This issue is labeled with high priority. However, it have been open for 10 months. When do you plan to release this? Or must all issues in the v4.3 milestone be completed first? I don't want to stress anything, just would like to have this feature in the library 👍

Hi @nekman as I stated a few times, we need more people to test this with a real world setup. Just a few days ago someone did and revealed a missing feature which is now added as another PR.
If you want to get this feature faster merged then please help out with testing.
I have limited resources as I have only the authorization code workflow as setup to test. The other workflows can only be tested by me locally in an artificial environment.
You can also help out by reviewing existing pull requests regarding PKCE.
I will also try to get this repo into Hacktonerfest (never did this but I see some potential for getting some support).

Thanks for the information @jankapunkt. Of course we should not stress anything. Important to get it right and well tested. I will see what I can do!

implemented in 4.3.0 by PR #86


Display additional fields in the table and overview

[ ] Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)
fixes #2328
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[ ] No need to
[x] Manually (please, describe, if necessary)
[ ] Unit checks
[ ] Integration checks
[ ] Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)

@habrahamyanpro good to merge

@habrahamyanpro 🚢 it!


Correct the side mentioned with focus_neighbor_right and focus_neighbor_top.

A small typo with the definitions of focus_neighbor_right and focus_neighbor_top. Both use "bottom" rather than the correct side for each.

Take 2. Apologies for mistake with previous .rst pull request!

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 4.0.3.


For type errors is workspace.onDidChangeTextDocument intended to be used or is there a language provider I should be plugging in instead?

import * as path from "path";
import { window, ExtensionContext, languages, workspace, DiagnosticCollection, TextDocument, Diagnostic, DiagnosticSeverity, tasks, RelativePattern } from "vscode";

function randomInt(max : number) {
  return Math.floor(Math.random() * Math.floor(max));
}

const errorOnRandomLine = (d : DiagnosticCollection, doc : TextDocument) => {
    const line = doc.lineAt(randomInt(doc.lineCount));
    const diag = new Diagnostic(line.range,"random error",DiagnosticSeverity.Error);
    d.set(doc.uri,[diag])
};

export const activate = async (ctx : ExtensionContext) => {
    window.showInformationMessage("Spiral plugin is active.");
    const config_errors = languages.createDiagnosticCollection();
    ctx.subscriptions.push(
        config_errors,
        workspace.onDidChangeTextDocument(x => {
            if (path.extname(x.document.uri.path) === ".spiproj")
                errorOnRandomLine(config_errors, x.document);
        })
    );
};

I've finally figured out how to do messaging between Node and .NET and am starting to prototype. The above program just put an error in the random line inside .spiproj files. I am wondering whether I should be using a language provider instead. I think for a bunch of them there aren't any samples.

Since I haven't received an answer for 3 weeks, I'll just assume that this is the case. This question was somewhat rhetorical, but the error trace one is still important to me.


SICF: object type not self contained

Ideally, it should be possible to delete ZCL_ABAPGIT_OBJECT_SICF and still have a working abapGit.

suggest replacing with dynamic calls in ZCL_ABAPGIT_TADIR and catching exceptions


ktrain complaining about torch?!

Hi @amaiya and sorry to bother but ktrain stopped working recently, asking about torch
import ktrain
from ktrain import text
MODEL_NAME = 'distilbert-base-uncased'
t = text.Transformer(MODEL_NAME, maxlen=500, class_names=train_b.target_names)
trn = t.preprocess_train(x_train, y_train)
val = t.preprocess_test(x_test, y_test)
model = t.get_classifier()
learner = ktrain.get_learner(model, train_data=trn, val_data=val, batch_size=6)
learner.fit_onecycle(5e-5, 4)
learner.validate(class_names=t.get_classes()) # class_names must be string values
ModuleNotFoundError: No module named 'torch'

I only had used tensorflow with ktrain in the past, and everything worked great. Do you know what the issue is here?
Thanks!

Hello:  None of the commands you posted use torch.  I created a fresh virtual environment using the latest ktrain and only TensorFlow installed (no torch) and wasn't able to reproduce this problem.  So, I'm not sure where that error is coming from.
Perhaps try re-creating your virtual environment from scratch.

I will close this issue for now, but feel free to reply to this thread if needed.


Examples of usage with Play 2.6

Could you provide some examples of using this plugin with playframework 2.6?

Here are the docs:
https://deadbolt-java.readme.io/docs/
There is even a book you can read for free - however @schaloner would be very happy if you donate some cash :smile:
https://leanpub.com/deadbolt-2

I think we can close this issue.


Language Translator using GOOGLE translate API

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like.
We need to enter the text in English  and based on the language selected the text will be translated to the selected language and displayed to he user.
Here API used is google translate API
Describe alternatives you've considered.
nohing
Add any other context or screenshots about the feature request here.
no

Sir I was not added to the channel in discord kindly add me
On Tue, 17 Jan 2023, 21:36 Kushal Das, @.***> wrote:

Assigned #794
https://github.com/Kushal997-das/Project-Guidance/issues/794 to
@Harikaraja https://github.com/Harikaraja.
—
Reply to this email directly, view it on GitHub
https://github.com/Kushal997-das/Project-Guidance/issues/794#event-8247263626,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATCQWHQG3LLUZAN7BHCVJWDWS27PZANCNFSM6AAAAAAT4Q26MQ
.
You are receiving this because you were assigned.Message ID:
@.***
com>

Sir is this issue assigned to me , because I was unable to find the assigned label to this issue . kindly let me know sir so that I can submit my project .


Update auditing-solutions-overview.md

Updated the licensing requirements with Business Standard as standard auditing is available

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/8744

@JohanFreelancer9 Please copy review


Refactoring: Quit mixing Promises and async/await

Problem

There is a comment by @bcoe that says TODO. This comment has been here for two years.

https://github.com/googleapis/google-auth-library-nodejs/blob/main/src/auth/googleauth.ts#L206-L232

Suggestion

It would be better to use async/await only and reduce the length of code. It will make this repo easier to see through.
I am going to create PR when this change is accepted but beforehand, I put my plan below. Please use these code for your decision.

Before
// TODO: refactor the below code so that it doesn't mix and match
      // promises and async/await.
      this._getDefaultProjectIdPromise = new Promise(
        // eslint-disable-next-line no-async-promise-executor
        async (resolve, reject) => {
          try {
            const projectId =
              this.getProductionProjectId() ||
              (await this.getFileProjectId()) ||
              (await this.getDefaultServiceProjectId()) ||
              (await this.getGCEProjectId()) ||
              (await this.getExternalAccountClientProjectId());
            this._cachedProjectId = projectId;
            if (!projectId) {
              throw new Error(
                'Unable to detect a Project Id in the current environment. \n' +
                  'To learn more about authentication and Google APIs, visit: \n' +
                  'https://cloud.google.com/docs/authentication/getting-started'
              );
            }
            resolve(projectId);
          } catch (e) {
            reject(e);
          }
        }
      );
    }

After
this._getDefaultProjectIdPromise = (async () => {
        const projectId =
          this.getProductionProjectId() ||
          (await this.getFileProjectId()) ||
          (await this.getDefaultServiceProjectId()) ||
          (await this.getGCEProjectId()) ||
          (await this.getExternalAccountClientProjectId());
        this._cachedProjectId = projectId;
        if (!projectId) {
          throw new Error(
            'Unable to detect a Project Id in the current environment. \n' +
              'To learn more about authentication and Google APIs, visit: \n' +
              'https://cloud.google.com/docs/authentication/getting-started'
          );
        }
        return projectId;
      })();
    }

This looks good to me, as long as tests pass as well :) Thank you!

Thank you. I am going to create PR 👍

Fixed in https://github.com/googleapis/google-auth-library-nodejs/pull/1436


Get GH Pages Documentation going

We need to run a GH action to convert docstrings into markdown

Doneskies! Although I can't remember when: https://github.com/in03/proxima


ProxyHandler.apply method

lib Update Request

Configuration Check

My compilation target is ESNext and my lib is ESNext.
The method is in the file: lib.es2015.proxy.d.ts
[According to vscode command (Go to Definition)]
Missing / Incorrect Definition

This is about an incorrect method:
ProxyHandler.apply()
(The second argument to the Proxy constructor)
Current type definition:
    apply? (target: T, thisArg: any, argArray?: any): any;

Actual type definition:
    apply? (target: T, thisArg: any, argArray?: any[]): any;

Sample Code
<!--
  What's some code using this that should work, but doesn't?
-->

This only makes this method even stricter,
so the only effect will be to make more code fail.

Documentation Link

https://tc39.es/ecma262/#sec-proxy-object-internal-methods-and-internal-slots-call-thisargument-argumentslist
Step 7 specifically creates an array, which is passed as the third argument to the ProxyHandler.apply method.
Edit
This also applies to ProxyHandler.construct

This was already fixed by https://github.com/microsoft/TypeScript/pull/35594 which will be released in TypeScript 4.2.


Add crate support for shared types

Issue summary
Some types are shared in multiple packages, move them into one crate.
Other information and links

#51


[Feature] Add task_modules type hint

Motivation
Add task_modules type hint to the code and make it more readable.
Modification
add type hints to mmdet3d/model/task_modules
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your PR. Please review this PR. @Xiangxu-0103

Hi @A-new-b , please update this PR according to the comments and fix the lint issue.


[communication] update to core-tracing 1.0

Packages impacted by this PR

[communication chat]
[communication common]
[communication identity]
[communication network traversal]
[communication sms]

Issues associated with this PR

#21199

Describe the problem that is addressed by this PR
Updating to core tracing 1.0 from the beta release.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[x] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.

/check-enforcer reset


Add Packing Station Support

This is an attempt to add packing station support. As soon as this is merged, I would also like to add support for PO boxes ("Postfach"), post offices ("Postfiliale") and big recipients ("Großempfänger").
But first, please review if the code is fine for you or if you need changes :-)
To be honest, I did not check the tests at all. Do you think we need to update / extend the tests to reflect these changes?

Any chance that this gets reviewed and eventually merged? Thanks!

Do you think we need to update / extend the tests to reflect these changes?

Tests are the best way to demonstrate how a feature is supposed to work. So yes, adding a test for the Packstation use case seems to be beneficial. We will take care of that.
Did you ever succeed to get Packstation data corrected by the web service? In my tests, it just mirrors the wrong data.

Thank you for your contribution, @sprankhub – you did an amazing job here figuring out what needs to be done and where. Sorry it took such a long time to get reviewed.

Thanks for the kind words and thanks for finishing and merging it!


Export: Change "qemu-img" to use disk as buffer, and then copy output to GCS gssutil

This CL changes the way of using qemu-img:
Original: qemu-img output file to GCS by gcsfuse
Now: qemu-img output file to disk, then copy to GCS by gsutil
It achieves 3 purposes:

When qemu-img failed due to disk space or network issue, the workflow will fail. Previously it failed silently and the workflow got a success, and thus output a corrupt image.
Much higher success rate for large-size image, due that gsutil is much more reliable than gcsfuse, especially when network is not very stable. For a 198GB image, old method success rate is 29% while new method is 100% per my testing.
It made the code ready for next improvement: dynamically enlarge disk space to export >200GB image.

Conclusion: this CL resolved problems for <200GB image export, and technically prepared for >200GB improvement.
Concern of time efficiency: there was a concern that new method may double the time consuming. Below test result shows the concern is invalid:

Updated accordingly

This will fail for anything over 200Gb correct? If that's the case we need to be sure we don';t push this in a release. How close is the follow up PR? I would prefer to have the full fix in one PR so there isn't a period of time where export code will simply not work.

This will fail for anything over 200Gb correct? If that's the case we need to be sure we don';t push this in a release. How close is the follow up PR? I would prefer to have the full fix in one PR so there isn't a period of time where export code will simply not work.

Hi Andrew, This will fail for anything over 200Gb, just as how it behaves today. It won't fail any case that is passing today.

This will fail for anything over 200Gb correct? If that's the case we need to be sure we don';t push this in a release. How close is the follow up PR? I would prefer to have the full fix in one PR so there isn't a period of time where export code will simply not work.

Hi Andrew, This will fail for anything over 200Gb, just as how it behaves today. It won't fail any case that is passing today.

to be accurate, it will fail anything >200Gb after compression


fix: audit scanner not skipping namespaces.

Description
Due a bug in the audit scanner command generation, the namespace ignore list was not being properly translated into command line flags for the audit scanner command. This commit fixes that by splitting the line used to add the -i flag into two. Therefore the cronjob definition define the command line flags like:
[...]
  - -i
  - skip-namespace
[...]

Instead of:
[...]
  - -i skip-namespace
[...]

NOTE: This fix was proposed by @viccuad. Thanks!
Fix #264

Problem with the json notation here is that it renders and otherworldly large line.


Add access endpoint phase to the report

Added support for auth ('none', 'basic', 'bearer' and 'passport') to /access endpoint in the good_mock_server
compliance tests are run for the 4 different test drs_ids stored in the "/data" directory
Added "/data" directory which houses all the test DRS objects, access_url mapping and passport mapping
These test DRS objects are required to be loaded into a DRS implementation before running the drs_compliance_suite on it
Added access endpoint phase to the report

@emre-f please let me know if you need any clarifications.

Hey @yash-puligundla, I was trying to run the virtual environment following the README.md. Could you add a final line on how to enter and exit the virtual environment like this:
source {path_to_virtual_env}/bin/activate

and to exit the virtual environment:
deactivate

For the tests, I uninstalled all pip packages in the virtualenv, then re-installed them via pip install -r requirements.txt. But even then when I tried to run pytest -v I got 3 errors. Am I doing something wrong?

@emre-f Thank you for the feedback. I have updated the README to include the commands for activating and deactivating the venv. The unittests are broken in this PR and this is expected. These errors are fixed as a part of PR 24


Re-enable permissions in Firefox

Previously permissions were broken and/or disabled since the start due to slight differences in the extension API, which seem to be cleared up now.
unfortunately the diff is quite bad. Closes #1060, closes #1169  and probably others that I'll find.
Changes

Show the permissions tab in Firefox again
Change firefox manifest to no longer have wildcard permission by default
Change firefox manifest to have a wildcard permission in optional_permissions, in the same vein as Chrome. This allows us to request permission on any origin at runtime (i.e. turn on/off domains and add custom domains).
Remove redundant conditional Firefox code in the settings module, consolidating Chrome/Firefox code as they function exactly the same now.

Other:

Extract each permissions event handlers to its own named function, and remove/add the listener on each new render rather than adding more and more duplicate listeners over time. Fixes awful issues in Firefox where clicking a button could cause progressively more listeners to be created. The root cause is buried somewhere in the way the settings page is rendered, this is a clean fix until such a time as we re-write it.
Refactor permission filtering in a similar fashion to be a little more manageable.
Minor html change to settings page layout and input

Testing
This needs a bunch of testing and playing around with in Firefox, but also Chrome.
I managed to test a custom Jira domain by installing Jira server solution locally.. I wouldn't worry about testing custom domains again like that, just that you can add and remove them correctly in the settings.

Yep that's what I did. So I tried again... and this time the functionality does work. I think it might have been something about the confirmation window not showing up on my system for some reason. The only thing is that it still looks weird styling-wise. Here is a picture with the error logs:

@bryanph can you inspect the settings' iframe, using this dropdown in the firefox debugger for the extension:

and see what happened to the html? should just be a list..

@tcrammond I changed the styles a bit so it looks better in firefox. There are still some overflow issues though but that seems to happen because in firefox the settings page doesn't have a viewport height for some reason. I suppose it is injected with a fixed height or something :man_shrugging:
In firefox:

In chrome:

@bryanph nice, some issues though
We've lost styling of the custom permissions list items:
Before:

After:
(buttons wrap on normal permissions too now, not really a deal breaker, mainly want to keep the list item styles)

We also have an issue with tooltip on wide viewport settings (read: Firefox, at least on macOS) again I think you may have undone some width limits on the inner content that I had set up for this reason (see commit 3b98d33  Improve styles on wider width viewports (Firefox)). We don't wanna have to reinvent the help text at this point

@tcrammond thanks, should be good now :crossed_fingers:


OTA Issues in Pre Release Version 4.0.8 - 2024/10/03

Boards
AMB82-MINI
External Hardware
USB Connection to PC
Hardware Configuration
None
Version
latest dev (checkout manually)
IDE Name
Arduino IDE 2.3.3 & arduino-cli 1.0.0-4c.1 Commit: ba19a2d3
Operating System
Ubuntu 22.04.4 LTS
Auto Flash Mode
Disable
Erase All Flash Memory (16MB)
Disable
Standard Lib
Arduino_STD_PRINTF
Upload Speed
2000000
Description
I am having issues using the ameba-OTA-UI to properly run OTA when linked to the example OTA project in the following dev release 4.0.8-build20241003.
Frist I changed the ip address in the ota_drv.c file shown below:

That made me able to get my device to show up in the UI as shown below:

However, i can't seem to get the OTA to trigger using the UI.  to further clarify the following is the response on the device side:
[http_update_ota] Download new firmware begin, total size : 5447680

[http_update_ota] Current firmware index is 1
..
[ota_flash_NOR] target_fw_addr=0x520000, target_fw_len=0x10000

[http_update_ota] ota flash failed

On the UI side the folloing request seems to error out:

for further context the connectedclients request seems to be working properly:
{
    "recentEntries": [
        {
            "ip": "<IP_ADDRESS>",
            "timestamp":<PHONE_NUMBER>117,
            "OTA_state": "OTA_STATE_DOWNLOAD_FIRMWARE_COMPLETED"
        }
    ]
}

But the start_ota request seems to have an issue in getting a proper response:

Any help would be appreciated. I plan to take a deeper look at the OTA drivers, but was wondering if there was something obvious i was missing before doing so.
Sketch
#include "ota_thread.h"
#include "WiFi.h"

char ssid[] = "Network_SSID";    // your network SSID (name)
char pass[] = "Password";        // your network password
int status = WL_IDLE_STATUS;

void setup()
{
    Serial.begin(115200);

    // Connection to internet
    while (status != WL_CONNECTED) {
        Serial.print("Attempting to connect to WPA SSID: ");
        Serial.println(ssid);
        status = WiFi.begin(ssid, pass);
        delay(2000);
    }

    // Set up the threads
    start_OTA_threads();
}

void loop()
{
    // Empty or add non-blocking code here
}

Error/Debug Message
See Description (No Compile or upload error).

Reproduce remarks
No response
I have checked online documentation, FAQ, GitHub Wiki and existing/closed issues.

[X] I confirm I have checked online documentation, FAQ, GitHub Wiki and existing/closed issues.

hi @TeaBear5 , I think you have missed out the following steps in creating the OTA firmware image.

Set NN Model Load From SD Card. “Tools” -> “NN Model Load From” -> “SD Card”

Enable the OTA mode. “Tools” -> “OTA Mode” -> “Enable”

Compile the code again using Arduino IDE, you will then find the ota.bin file in the /home/username/.arduino15/packages/Realtek/tools/ameba_d_tools/x.x.x/ folder

This is the .bin file to be uploaded to your UI server for OTA trigger.
Thank you.

@kev0077 Thanks!  This works great!
For anyone else's reference this can also be accomplished using the arduino-cli tool with a command similar to the following:
arduino-cli compile --fqbn realtek:AmebaPro2:Ameba_AMB82-MINI --output-dir ./build \
--build-property build.ota_mode_val=Enable \
--build-property build.ota_mode_flags=-DArduino_OTA_MODE \
--build-property build.Arduino_LOAD_MODEL_flags=-DArduino_LOAD_MODEL_SD \
--build-property build.model_src=LoadFromSD \
OTA_Test


Lowercase Architecture and ErrorValues

This is a simple rename to match the recent casing discussion. I think these two, and ParsedJson, are the only names that will be affected. As all of this is subjective naming crap, I'll explain my reasoning but I'm super malleable on all of it.
It merges into #485 rather than master at the moment, just because I didn't want to deal with merge conflicts once we land things.
I whipped this up because I wanted to add some more architecture functions and simdjson.h seemed like it would start getting crowded, and because I've been trying to use ErrorValues as a type recently and finding the plural getting to me (ErrorValues error_code suggests multiple error codes, not one).

Renamed ErrorValues to error_code because (a) making it singular makes it possible to use as a type, and seems to be recommended for enum types, and (b) error_code is used as the name for the concept in the standard library as well. Doing that required renaming a few variables from error_code to error, but I think I am the one that named them that way (and most of them I did in #485, which isn't even checked in yet :)).
Renamed Architecture -> architecture, necessitating a few variable renames from architecture -> arch.
Moved error codes and exception definition into error.cpp / error.h
Moved architecture type and selection functions into architecture.h

Both types have backcompat layers.

We can also leave this for a few days (or however long it takes me) while I work up the vtable-based architecture, and see how that looks and feels. Probably won't change error_code, so if we go the waiting way (which is fine), consider this a WIP preview :)

Looks uncontroversial to me.
cc @DBJDBJ

There we go. It got confused when I rebased it on master, at first.

Oh--I didn't see the above comment as a thumbs up to merge, was it? No rush still, I know your week is awfully busy.

Yes!

I mean: I reviewed it. Good stuff.


Deprecate Github discussions and link to kedro slack

Github discussions are unnoticed and poorly referenced.

[ ]  Can discussion be made read only? I'd like to keep the track of past ones, jsut prevent from creating new ones
[ ] Add a badge to kedro slack on README

"Read only" can only be done at organisation level :
https://github.com/kedro-org/kedro/issues/3134#issuecomment-1819221067


feat: extension library

WIP - Bolt Extension Library (https://github.com/dustinwloring1988/extension-library)
WIP - Templates (https://github.com/bolt-starter-templates)
Demo
URL: https://www.useme.cc/
Goal
To enhance the user experience and community involvement
Why This Matters
Currently, the UI can feel overwhelming with providers, models, and options that might not be relevant to every user. This new library aims to streamline the interface by consolidating providers, models, prompts, and starter templates into one centralized, easy-to-navigate resource.
Key Features

User-Friendly Setup: For prompts, users will be able to paste a link in the settings, which will automatically download and place the JSON file in the right spot.
Starter Templates: These will initially be handled as Git clones for a straightforward implementation.

What’s Next?
Starting with the prompts and will expand the library as the foundation is built.
https://github.com/user-attachments/assets/6fc07723-794c-41e5-b069-eacdc83ed1b8

More prompts will still be added and a way to import them from github.
The Models tab will be for maybe breaking download what models come with the providers then have a extra model pack for it (just a thought maybe delete this tab)
Once the providers tab is done and implemented I was thinking of the following set up. What is everyone's thoughts:
With bolt.diy by default
Anthropic
Google
Mistral
OpenAI
Addable by the Extension library
OpenRouter
Deepseek
Cohere
xAI
Together
Groq
Perplexity
HuggingFace
OpenAILike
LMStudio
Ollama

I think the marketplace should be a separate app, where people can sign in and upload their own stuff using a git url.
so the code will stay in their git repo and they just link it here, and based on the stars and forks we can rank them in the portal for others to use

@thecodacus thanks for the feedback.

The marketplace is its own app and you can link any Starters template from any github url to it (just look in the mockData file).
I will work on adding the stars and forks as I love that idea.
For the logging in part do we want to login or should we have it so that the user adds there extension in the form of a PR and it will update the file that way?

For the logging in part do we want to login or should we have it so that the user adds there extension in the form of a PR and it will update the file that way?

again PR will be a hurdle for the community, maybe integrate with supabase cloud service for logging.


AllotmentHandle  does not have resize function

The doc shows that we can use ref.current.resize([123,344]) however the AllotmentHandle  does not have resize function
export declare type AllotmentHandle = {
reset: () => void;
};
Any help?

Could you just confirm which version of the library you've tested with?

I'm closing this as stale. Please ping me or create a new issue if this is still a problem.


Enhanced markdown editor

The current textarea works well but could be enhanced to make editing and previewing changes a more pleasant user experience.
There are a couple of project out there that look interesting though given the bootstrap nature of daux, perhaps this could be a quick win in that area: https://github.com/toopay/bootstrap-markdown.

Yes I think a better markdown editor is clearly a good idea when we add it back, I will add this to milestone 1.1

Closing this issue as we move the project to an organization: https://github.com/dauxio/daux.io, the packagist package name is now daux/daux.io and we even have a Docker image.
If you are still interested by that feature, please open a new issue in the organization and we'll look into it.


feat: use ethers and Promises All to consult data

Please, someone review and test this code before.
In local computer, only consume 40 MB.

whole app uses ~400MB on heroku with getVaults disabled.
i'm wondering what it was before the changes, can it be related with eth-multicall.
anyways i'm willing to keep current implementation, there is no leak so we can bump RAM, i dont think there is a big memory savings, is there?
multicall solution is better, as with promises we make 2 requests per each vault (thats 2500 requests), so some RPC might fail especially when it overlaps with prices or apys requests.
there is ethers-multicall libs but i dont think it can be significantly better.
some optimization could be to combine strategies and lastHarvests in 1 multicall, as this functions used only in getVaults.

you are right. Multicall resolve request faster:

now I testing current prod version (with getVaults enable) and refactor version and both seems to use same memory space.


Atrace not getting collected on Moto G40 Phone Device

WHAT
I used the  custom script to profile an android app. The app is in release build with the profileable tag enabled. On debugging and going a little deep into the source code i realised that the atrace was not getting collected. On debugging a little more i see the error log
Unable to open file5037 (.package.appname) S 1113 1113 0 0 -1<PHONE_NUMBER> 2417899 0 63 0 45820 17497 0 0 20 0 117 0 119065447<PHONE_NUMBER>8 58295<PHONE_NUMBER>3709551615 1 1 0 0 0 0 4612 1<PHONE_NUMBER> 0 0 0 17 3 0 0 0 0 0 0 0 0 0 0 0 0 0

The Unable to open file comes from line number https://github.com/bamlab/flashlight/blob/94c07a10f7811e83f8e7dcc01537e310e9957b87/packages/android-performance-profiler/cpp-profiler/src/atrace.cpp#L56
On checking the path of the systrace file the source code mentions
std::string traceOutputPath = "/sys/kernel/debug/tracing/trace_pipe";

However on my device i see the trace file being created at the path /sys/kernel/tracing/trace_pipe. To verify this on one terminal i ran the command adb shell atrace -c view -t 999 and in another terminal did a simple cat /sys/kernel/tracing/trace_pipe . And i was able to get the atrace..
What am i to do here? is it possible the trace file locations vary as per devices? Also how do i fix the current case?

HI @nitish24p, thanks for raising this issue, and apologies for the delay 🙏
Nice debugging! Nice that you were able to find the proper file path!
One thought: the app doesn't need to be profileable. That shouldn't matter but is it still /sys/kernel/tracing/trace_pipe if the app is not profileable?
Might be depending on the version of ftrace according to this.
If we need to handle multiple paths depending on the device, we'll handle all of them. In case you'd like to contribute and test it out locally, there's a guide here on how to build/test the C++ executable, but let me know if you have any questions!
In any case, I'll run a script at some point on our AWS device farm to check those paths work for all devices
Also just adding Unable to open file in case of error is kind of horrendous on our side, so we'll need better error handling 😅

Hey @Almouro , Sure can help out if need be. Presently for my case, i made a fork and pushed a new tag with the path updated to the right one and built the binary files again and its working as expected.

Hi @nitish24p, at last, this should be fixed, I was able to reproduce on other devices.
Can you confirm on your end? 🙏

Do i need to update the version to latest?

Hi @nitish24p , doing a bit of cleanup on Github issues, so I'm closing this one for now for lack of activity ☺
I think this is fixed now, but do feel free to reopen if the problem is still there! 🙏


Issue-254 (Chore): Fixed Sidebar in mobile view

Signed-off-by: Abhijay Jain<EMAIL_ADDRESS>Description
Fixed Sidebar in mobile view, made it rectangle and placed it to the edge
This PR fixes #254
Signed commits

[x] Yes, I signed my commits.

PS : sorry for the late PR 😅

Looks good to me!
Keep up the good work @Abhijay007

@Abhijay007 It would be great if you move the button to the right when the menu is expanded. Some shadow would look really nice. the button is cut off from below??

Ya I will move it to the right on expansion and the button is not getting cut ,it is due to preview link 😅

@Abhijay007 any update on this? Can we discuss this on the call tomorrow?

@Abhijay007 any update on this? Can we discuss this on the call tomorrow?

@debo19 I am getting some errors while running the nighthawk project locally and to resolve that I need to switch my installed ruby version, I will update you on this :)


strange problem when start vim

When I start vim, I always get a strange problem, the bottom line is always duplicate like this:
NORMAL ››  1:                                                          ‹‹ [unix] ‹ 100% :   0:  0
1:
1:
Press ENTER or type command to continue
I have tried update or reinstall, but nothing helps.
I run "vim -V9vim.log" to see the log of startup. I found the following error message!
Searching for "/usr/share/vim/vim73/ftoff.vim"
chdir(/usr/share/vim/vim73)
fchdir() to previous dir
line 57: sourcing "/usr/share/vim/vim73/ftoff.vim"
Error detected while processing /usr/share/vim/vim73/ftoff.vim:
line   11:
E216: No such group or event: filetypedetect *
finished sourcing /usr/share/vim/vim73/ftoff.vim
continuing in /Users/liudonghua/.vimrc.bundles
My OS is Mac 10.10.1
vim --version
VIM - Vi IMproved 7.3 (2010 Aug 15, compiled Sep  9 2014 16:30:51)
Compiled by<EMAIL_ADDRESS>......
Can anyone help me!

I tried this and it woks. It can be closed.

@spf13 : are we still supposed to work on vim 7.3?  I wonder if we just just prevent installation totallly on a vim 7.3 install if no one will be dedicated to making sure that it continues to work there.

Fine with me. We should ask for volunteers first though.
Steve Francia
spf13.com
@spf13

On Mar 7, 2015, at 3:54 AM, Johnny Robeson<EMAIL_ADDRESS>wrote:
@spf13 : are we still supposed to work on vim 7.3? I wonder if we just just prevent installation totallly on a vim 7.3 install if no one will be dedicated to making sure that it continues to work there.
—
Reply to this email directly or view it on GitHub.


Improve withdraw time for parachain

Describe the task
Automatically display the withdraw button when the timeout expires
Actual: Need to reload staking

Resolved in #2830


ValueError: Unknown layer: GCNLayer

Hi thanks for sharing the code, I have been trying to reproduce the results presented in the paper.
To do this I have trained the codegnnbilstm model from scratch using the instructions and code provided in the repository. The model trains fine. However, when I try to run the predict.py file, I run into the following error.
ValueError: Unknown layer: GCNLayer

hi I am trying to reproduce the code and I am constantly getting is error
Traceback (most recent call last):   File "train.py", line 12, in <module>     from utils.model import create_model   File "/content/ICPC2020_GNN/utils/model.py", line 1, in <module>     from models.codegnngru import CodeGNNGRU   File "/content/ICPC2020_GNN/models/codegnngru.py", line 6, in <module>     from tensorflow.keras.layers import CuDNNGRU ImportError: cannot import name 'CuDNNGRU'
Thank You

When you load the model you may need to change keras.load_model({model file}, custom_objects=GCNLayer) and make sure you import GCNLayer

Hi, Thanks for your reply, I have the correct model and GCN layer imported in the file, however still I run into this error. I am using the exact same code provided in the repository. I am not able to understand the possible cause for the same.
from models.custom.graphlayer import GCNLayer)
keras.models.load_model(modelfile, custom_objects={"tf":tf, "keras":keras,'AlexGraphLayer':GCNLayer})

I also encounter this problem, have you solved it?

Sorry, I just solved it, just change the {"tf":tf, "keras":keras,'AlexGraphLayer':GCNLayer}) to custom_objects={"tf":tf, "keras":keras,'GCNLayer':GCNLayer}, then the network can recognize the customized GCNLayer


Event listener added to both input field and click button, querystrin…

…g to pass on the country name and fetch data moved to countryinfo.js, set the ccountryinfo.html background to empty for now, for easier access and country data verification.

Changes made, creating a new pull request soon.


Runtime modules should be able to configure default excludes in dagger.json

Motivation
I have to constantly add "exclude": ["sdk", ".venv"]  to dagger.json after dagger init.
I don't expect these files to be uploaded. They're not completely replaced in codegen. During local dev, some files can be created in there and they will stay after codegen in the container. It affects the Source dir cache and slows down module loading.
Solution
The engine already loads dagger.json first. Load .gitignore too. Prepend the .gitignore patterns to the exclude patterns from dagger.json when collecting the patterns for loading the module’s files.
As an escape hatch, if the user needs to include something in .gitignore they can add the same pattern manually to include in dagger.json.
Rationale
This seems like the easiest solution, with less work for the user but keeping control.
Doesn’t require extra work. Most modules in the Daggerverse have the automatically generated .gitignore but no patterns in "exclude". By using .gitignore, those modules would benefit automatically. Of course, this is only necessary when dagger developing modules locally.
When a module is loaded via a git reference, those patterns are already applied because the git clone won’t have those files. So using .gitignore is actually more consistent because it avoids differences in dirty working copies.
It’s still possible to have dirty woking copies because global ignores won’t be picked up, but rather than adding some of those to exclude, users might as well just add them to .gitignore which also helps other users contributing to a module that don’t have the same global ignores.
It’s also possible for files in a module to be ignored by some other .gitignore up the file tree until the git repo root, but we could automatically find those and add their patterns, prefixed with their file path relative to the context directory, to the module’s excludes.

@sipsma, I updated the description. Previously, my thinking was just to add the ignore patterns to the execlude in dagger.json in dagger { init | develop } --sdk=, but wdyt about loading from .gitignore?

This seems reasonable to me but I may not be aware of all the subtleties of the design.
In general I'm in favor of adopting the standard gitignore format whenever we need to express filters on directories (see for example the ignore part of #7647 ). This is one step further in the same general direction.
How would the default behavior be disabled or customized?

@sipsma, I updated the description. Previously, my thinking was just to add the ignore patterns to the execlude in dagger.json in dagger { init | develop } --sdk=, but wdyt about loading from .gitignore?

@drmikecrowe love to get your thoughts ☝

I guess my base question would be:   Why wouldn't you adopt .gitignore as the standard?  In my case, I'm controlling the release target contents via .gitignore.  There's no situation where I want to use local files in dagger that are outside .gitignore because they will never be in my target environment (unless I'm missing something)

I guess my base question would be: Why wouldn't you adopt .gitignore as the standard? In my case, I'm controlling the release target contents via .gitignore. There's no situation where I want to use local files in dagger that are outside .gitignore because they will never be in my target environment (unless I'm missing something).

Yes, that's what I'm suggesting. I'll reformulate the description when I can to provide more details on how I see this working, including picking up on global git excludes and parent .gitignore files too.
Also, notice this part of the rationale:

When a module is loaded via a git reference, those patterns are already applied because the git clone won’t have those files. So using .gitignore is actually more consistent because it avoids differences in dirty working copies.

So in the same line of thought, when we support other VCSs, I'd do the equivalent to .gitignore in those cases if possible.

Remember, dagger init will not run if there is a dagger.json present -- options 1 and 3 don't seem possible.  I think the second option is the only  choice of those 3.
Alternatively, scope out a .daggerignore?
Also, does dagger respect .dockerignore?

Remember, dagger init will not run if there is a dagger.json present -- options 1 and 3 don't seem possible. I think the second option is the only choice of those 3.

Those options aren't targetting dagger init, it's mostly dagger call. Remember, those options are for disabling the default use of .gitignore files, but your use case doesn't call for disabling. If you did want to disable, you could do it after dagger init as I can't think why a dagger init would fail based on something you have on .gitignore. Even if that's desirable, a CLI flag can be added there as well.

Also, does dagger respect .dockerignore?

No, it doesn't. I don't see a need for it other than possibly on getting a directory for a docker build, which is different use case, however we use a common Directory for that, uncoupled to the docker build.

Those options aren't targetting dagger init, it's mostly dagger call.

Help me understand please:

When would you want to include files in dagger init and then exclude them in dagger call?
Conversely, when would you need to include files in dagger call that you previously excluded in dagger init?

I'm trying to understand the different use-cases (and to date, my angst has been with excluding and dagger init, so I haven't had time to experience the need in dagger call yet)

I'm trying to understand the different use-cases (and to date, my angst has been with excluding and dagger init, so I haven't had time to experience the need in dagger call yet)

Well, today, if your dagger init takes 6m and you don't proceed to adding excludes in dagger.json, depending on the changes you make triggering cache invalidation, can take you the same 6m or more in a dagger call.
dagger init creates initial files based on a template. That's controlled by the SDK and fixed for each release. You can add any number of stuff afterwards depending on your needs. Add dependencies, add files to the source directory, run local tools that create new stuff that's not generated in dagger init but that needs excluding for subsequent dagger call commands.
A simple yarn install in your host is a common example of this. It'll create a node_modules directory no generated in dagger init that you don't want to upload when loading the module on a dagger call. It's not that dagger init includes node_modules, it just doesn’t exclude anything. The engine, by default, adds a <source dir>/**/* include. That’s today.
Let's say you already have a .gitignore in your repo (or even a global git exclude if this is possible for Dagger to use), with the patterns .yarn, node_modules, etc. Then with this proposal, dagger init would pick up on that automatically, thus solving the problem that you had.
The same would be kept for  dagger call without having to manually add "exclude" patterns to dagger.json. And if you happen to use direnv during development, and add .direnv to .gitignore because you don't want to commit that, then there's nothing else you need to do to exclude it in Dagger too.

ah, that's an excellent clarification @helderco -- I'm now in sync with your proposal and fully agree

There's a POC here:

https://github.com/dagger/dagger/pull/8501

I still need to update this issue's description with the proposal, add details and examples for design review. It's not forgotten, I'm just backlogged.

There's a POC here:

POC: Apply .gitignore patterns on module load by default #8501

I still need to update this issue's description with the proposal, add details and examples for design review. It's not forgotten, I'm just backlogged.

@helderco This idea sounds good but I just thought of a potential failure case - how will Github large file support be handled? We have a bunch of repositories which have an included gitversion.exe for devops purposes, it's 160mb, and it's not the only example.. we also have repositories with stub databases and suchlike.
Github encourage this approach (vendor lock in) so it's impossible to prevent developers doing so. How would one handle this? These artifacts won't be git excluded but we definitely don't want that junk sent to docker on every dagger ... invocation.

How would one handle this? These artifacts won't be git excluded but we definitely don't want that junk sent to docker on every dagger ... invocation.

The idea is that the .gitignore files contribute to the final exclude patterns for loading a module. You still have dagger.json exclude to ignore more stuff that's not in any .gitignore, or even to override a pattern that was added by .gitignore.
For example, let's say you have a .gitignore with this:
.idea

And in your my-module/dagger.json:
{
  "name": "my-module",
  "sdk": "go",
  "exclude": [
    "gitversion.exe"
  ]
}

Then the final exclude patterns for loading your module looks like this:
excludes := []string{
     "**/.git",                   // always added by the engine
     "**/.idea",                  // from .gitignore
     "my-module/dagger.gen.go",   // from my-module/.gitignore
     "my-module/internal/...",    // from my-module/.gitignore
     "my-module/gitversion.exe",  // from my-module/dagger.json
}

@helderco that's great - thanks for clarifying, still familiarizing myself with dagger.

I'm currently working on some performance improvements to module loading that will result in enough changes/simplifications to the relevant code that I'm just going to rope this in. Re-assigning to me (cc @helderco)


Colcon clean does not work

Building works fine, but I get the following output when I run colcon clean workspace in a subfolder (/homes/17vahl/colcon_ws_mono/src/bitbots_meta in the traceback)
colcon clean workspace
Using workspace root /homes/17vahl/colcon_ws_mono
Paths:
[1.892s] colcon ERROR colcon clean: '/homes/17vahl/colcon_ws_mono/build' is not in the subpath of '/homes/17vahl/colcon_ws_mono/src/bitbots_meta' OR one path is relative and the other is absolute.
Traceback (most recent call last):
  File "/homes/17vahl/.local/lib/python3.10/site-packages/colcon_core/command.py", line 522, in verb_main
    rc = context.args.main(context=context)
  File "/homes/17vahl/.local/lib/python3.10/site-packages/colcon_clean/subverb/workspace.py", line 58, in main
    clean_paths(
  File "/homes/17vahl/.local/lib/python3.10/site-packages/colcon_clean/subverb/__init__.py", line 235, in clean_paths
    path = path.relative_to(cwd_path)
  File "/usr/lib/python3.10/pathlib.py", line 818, in relative_to
    raise ValueError("{!r} is not in the subpath of {!r}"
ValueError: '/homes/17vahl/colcon_ws_mono/build' is not in the subpath of '/homes/17vahl/colcon_ws_mono/src/bitbots_meta' OR one path is relative and the other is absolute.

This was resolved yesterday by a change to colcon-clean.
Duplicate of colcon/colcon-clean#37


Extra single quote appeared after use safeDump

I use safeDump method to create a YAML document and here is my code

const yamlContent = yaml.safeDump({
  build: [
    'npm -v && npm install',
    'npm run build -- --json --verbose --outputPath ./dist --config ./webpack.config.js --no-compress ' // here is a white space in last
  ]
}, { lineWidth: 120 });

console.log(yamlContent);
// build:
//  - npm -v && npm install
//  - 'npm run build -- --json --verbose --outputPath ./dist --config ./webpack.config.js --no-compress '

then i find single quote disappeared after i trim my string in array

const yamlContent = yaml.safeDump({
  build: [
    'npm -v && npm install',
    'npm run build -- --json --verbose --outputPath ./dist --config ./webpack.config.js --no-compress '.trim()
  ]
}, { lineWidth: 120 });

console.log(yamlContent);
// build:
//  - npm -v && npm install
//  - npm run build -- --json --verbose --outputPath ./dist --config ./webpack.config.js --no-compress

I ran into a similar situation, but with an object instead of an array.
I'm not familiar enough with YAML to know how older version of yaml worked, but it looks like the extra quotes get added in "compatability mode".  When I set noCompatMode to true when dumping, I no longer get the extra quotes.
Relevant code seems to be in lib/js-yaml/dumper on line 323.  This might be related to what you're seeing.

Dumper tries to add quotes only when needed. You can see details about quotes in spec. Result is valid.


feat(techdocs-common): add custom docker image support

Signed-off-by: Andrew Thauer<EMAIL_ADDRESS>Hey, I just made a Pull Request!
This adds support for using a custom docker image in the techdocs generator step. This is a per-requisite for https://github.com/backstage/techdocs-cli/pull/70 which will add custom docker image support to the techdocs-cli generate command.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

It seems kind of redundant to have techdocs.generators.techdocs (I can't imagine what other keys would ever be under generators -- maybe @OrkoHunter has context?).

It was added way in the beginning, expecting folks would want to plug in and out there generators, and techdocs generator being the one based on mkdocs.
However I'm a big +1 for making it simpler since it doesn't seem likely to happen or even feasible at some extent.

It seems kind of redundant to have techdocs.generators.techdocs (I can't imagine what other keys would ever be under generators -- maybe @OrkoHunter has context?).

It was added way in the beginning, expecting folks would want to plug in and out there generators, and techdocs generator being the one based on mkdocs.
However I'm a big +1 for making it simpler since it doesn't seem likely to happen or even feasible at some extent.

I'm not opposed to this assuming there's little to no chance that other generators will be supported.  So it would end up being singular generator which would match the other keys (builder & publisher). This could also make it easier to tag the existing plural version as deprecated as well.
techdocs:
  builder: local
  generator:
    runIn: docker
    dockerImage: my-org/techdocs
  publisher:
    type: local

@iamEAP / @OrkoHunter - This is now ready for review.
After thinking about it I think runIn is actually more clear then type for this use case. It's not really a type of generate but rather where it's run. I'm happy to rename this if you think of something better or prefer type.
I also added a new optional pulllImage config setting since the docker image could be located on a private repository that requires authentication. In this case you would need to ensure the image is pulled locally first. I've been using this approach internally in both CI and a custom docs CLI wrapper around techdocs-cli so it works out decently well. In the future some native support for docker logins would be nice, but that would be a much larger effort.

@backstage/reviewers - Looks like this needs an additional review, probably for the backend-common changes.


some questions about individuality

hello,I want to change the the navigation bar below.But I can not find the place to change.Could you solve the problem?

Do you want this?
https://github.com/icylogic/carbon/blob/master/source/css/_partial/unique.styl#L105


wsl stops responding after suspend/hibernate wakeup vmmem 100% cpu

Version
Microsoft Windows [Version 10.0.22000.978]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu-22.04
Other Software
No response
Repro Steps
Run WSL with graphic application such as xfce4-terminal.
Suspend/hibernate a few times.
WSL stops responding.
Expected Behavior
WSL operational as it was few days ago.
Actual Behavior
WSL is not responding.
vmmem (vmwp.exe) takes 100% CPU
> wsl.exe true does not respond.
> wsl.exe --shutdown takes a lot of time to run (minutes)
Diagnostic Logs
Worked up to 2022-09-23.
Recent updates:

KB5017328
KB5017497

Relatively recent updates:

KB5016629
KB5012170

Reproduced first with release preview:
>wsl --status
Default Distribution: Ubuntu-22.04
Default Version: 2
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.42
MSRDC version: 1.2.3401
Direct3D version: 1.606.4
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.22000.978

Then downgraded to stable and also reproduced.
>wsl --status
Default Distribution: Ubuntu-22.04
Default Version: 2

Windows Subsystem for Linux was last updated on 24/09/2022
WSL automatic updates are on.

Kernel version: <IP_ADDRESS>

Related to https://github.com/docker/for-win/issues/12968

I can confirm, I've been running shutdown for 10 minutes now, tried to restart lxssmanager in the meantime but no avail :(

Duplicate of #6982

Duplicate of https://github.com/microsoft/WSL/issues/6982

Related-To: #8930


QA Checks - For Duplicate Records

The scope of this ticket is to address the following Checks to verify any duplicate records being input:
QA and Certification Data Checks

[x] SEVNDAY-29 - Duplicate Calibration Test
[ ] APPE-46 - Duplicate Appendix E Test
[ ] APPE-48 - Duplicate Appendix E Summary
[ ] APPE-49 - Duplicate Appendix E Run
[ ] APPE-50 - Duplicate Appendix E Oil Record
[ ] APPE-51 - Duplicate Appendix E Gas Record
[ ] CYCLE-19 - Duplicate Cycle Time Test
[ ] CYCLE-20 - Duplicate Cycle Time Injection

SEVNDAY-29-A POST

SEVNDAY-29-A IMPORT

SEVNDAY-29-A POST

SEVNDAY-29-A IMPORT

Closing and the rest of the checks shall be tested as part of 4709 and 4710


treewide: add meta.mainProgram to many packages

Description of changes
Following on from my efforts in #166865, I refined my hacky script and I've been able to find an additional 167 packages that provide a single binary whose name differs from the package's name in nixpkgs (or the package's pname) and so don't work with nix run.
The mainProgram additions in 16e15fa68f0247d1ccc9bf94e2906097bcaa2d5c should be pretty boring and uncontroversial.
I separated out additions of mainProgram to files located in pkgs/development/libraries/ into a separate commit (9e4be532219685cdd81e4db54f9fc061d250ba25) since a lot of these required more judgement. I think it makes sense for many of these packages to include a mainProgram as they often provide a single binary utility as part of the package that folks might want to use. However, in some cases it's pretty marginal. For those cases, I looked into the functionality of the binary and what it's used for, and made a judgment call. (I did err on the side of adding mainProgram in a lot of the marginal cases since it does little to no harm, but there were many cases where I left it off since it made no sense.)
Finally I made separate commit for nimble-unwrapped (d8da6a153b7951f55f4e0243047ad4ae644b4ed5) since I added a new meta attribute set since it was missing entirely.
I've gone through these changes pretty carefully, but a couple more skims for others is probably in order as a sanity check.
(While I was at it, I also ended up tweaking the order of the meta attributes to my understanding of which order is best practice, as well as updating some of the homepages and adding some when they were missing.)
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 171602 run on aarch64-darwin 1

Result of nixpkgs-review pr 171602 run on aarch64-darwin 1


Startup crash when trying to run it on KDE

System: Kubuntu 23
When I try to open the app, after the login screen appears, it crashes and I get this error related to the app trying to fetch some gtk specific property (which doesn't exist on KDE. Same happened in Manjaro with KDE.
set 24 09:55:27 dom-ThinkPad-T14s-Gen-3 greenlight[5233]: Theme parsing error: gtk.css:1649:16: '-gtk-icon-size' is not a valid property name

Similar situation in KDE 5.27 on Fedora 38

I have tested this more, It started after the beta 13, beta 12 and previous versions work fine. It also happens on Gnome. I've tested it on Ubuntu and was able to reproduce the error. Try version 11 or 12 as a workaround.

I believe this issue should be fixed but I'm unable to verify it. Can someone let me know if this issue is fixed?

I will take a look and let you know.
On Thu, Dec 21, 2023 at 3:17 PM UnknownSKL @.***> wrote:

I believe this issue should be fixed but I'm unable to verify it. Can
someone let me know if this issue is fixed?
—
Reply to this email directly, view it on GitHub
https://github.com/unknownskl/greenlight/issues/962#issuecomment-1866739320,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFUPGKBD7SJJ7WH65ILJ44LYKR4NLAVCNFSM6AAAAAA5E55PCOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRWG4ZTSMZSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Finish mobile version

In this project, I did the following:

Modify the index file to finish the mobile version page.
Modify the CSS file to finish styling the mobile version page.

Hi Assiati Magana,
Thank you, but I really don't understand what you are talking about. All
those required changes you have indicated,  I have done them on my last
commit.
See attached screenshots from my CSS file and tell me what else I exactly
need to do.
On Fri, Jan 13, 2023 at 12:30 PM Assiati Magana @.***>
wrote:

@.**** requested changes on this pull request.
STATUS: Required Changes ♻♻♻ Hi @WilhelmK109
https://github.com/WilhelmK109 , I'm @MaganaAsiati
https://github.com/MaganaAsiati, your reviewer for this PR 👋
Good job so far!
There are some issues that you still need to work on to go to the next
project but you are almost there!
[image: so-close-gif-4]
https://user-images.githubusercontent.com/95297251/191952218-b7185270-c02b-4e4d-89af-eae77fcd8bea.gif
Highlights

Descriptive README file ✔👨🏻💻
Great work using the right GitHub Flow.✔
No Linter errors ✔

Required Changes ♻

Kindly I suggest you apply the Inter font family for the paragraph
of your all card as required by the previous as shown you in the picture
below you still have Roboto as the font family please fix that

[image: image]
https://user-images.githubusercontent.com/95297251/212292750-9323d932-6f82-4775-9a84-6b4237c49215.png
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you take them into
account as they can make your code better.

[OPTIONAL] Please make sure to make your commit messages more
descriptive in the future. ✔👨🏻💻
kindly Add a meaningful description to your pull request in the
future which gives more information about what you implement in this
milestone ✔👨🏻💻

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.

In styles.css
https://github.com/WilhelmK109/my-portfolio-website/pull/2#discussion_r1069189778
:

+.post-stories {

display: flex;

flex-direction: column;

+}

kindly use the appropriate color for this section as required above
by the previous I suggest you to use this color: #091E42; instead of
black it will make your design stick to the Figma design as required in the
project

In index.html
https://github.com/WilhelmK109/my-portfolio-website/pull/2#discussion_r1069214169
:

About me

Hello I’m a software developer! I can help you build a product , feature or website.

   Look through some of my work and experience! If you like what you see and have a project you need coded,

   don’t hesitate to contact me.

 <button class="btn3" type="submit">Get My Resume</button>

Kindly use the appropriate background color for your about section
as required by the previous reviews I suggest you use this one background:
#1C1A19; it will make your design stick to the Figma design 👍
Kindy also apply the appropriate font family for Languages,
Frameworks, and Skills as required by the previous reviewer it will make
your design stick more to the Figma design 👍

—
Reply to this email directly, view it on GitHub
https://github.com/WilhelmK109/my-portfolio-website/pull/2#pullrequestreview-1247292253,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARYYJW3IFHQXJ5WV4ZTXCPDWSEVF3ANCNFSM6AAAAAATZRW3OE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@.**, Thank you a lot, sir.
On Fri, Jan 13, 2023 at 3:51 PM Francisco Borja Lobato <
@.***> wrote:

@.**** approved this pull request.
Hi @WilhelmK109 https://github.com/WilhelmK109,
STATUS: APPROVED 😄
[image: git-merge]
https://user-images.githubusercontent.com/58642949/212335334-564b64f4-2073-4a35-b275-8a863ba68b0a.gif
Your project is complete! There is nothing else to say other than... it's
time to merge it [image: :shipit:]
Congratulations! 🎉
Highlights

No linter errors ✔
Correct Github flow ✔
Descriptive PR ✔
Professional README file ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this
PR. However, I strongly recommend you to take them into account as they can
make your code better. Some of them were simply missed by the previous
reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
—
Reply to this email directly, view it on GitHub
https://github.com/WilhelmK109/my-portfolio-website/pull/2#pullrequestreview-1247669230,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARYYJW3M7BERO2UGGQF2ARTWSFMUPANCNFSM6AAAAAATZRW3OE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Kik Review process declined - Bot not responding to 'start-chatting' message

Bot Info

Node.js
SDK Version: latest from npm (3.15.0)
Skype, Telegram, Kik
Azure Bot Service, Azure App Service

Issue Description
Our Bot is declined by the review process of Kik because it is not responding to the "start-chatting" messages.
We are catching all events (and logging them for debugging purposes) but we don't get any "start-chatting" events.
This event seems to "disappear" somewhere in the Bot Framework. Any events triggered by sending a message 'actively' from Kik are received and handled. So the general connection is existing and working.
Variants with  'conversationUpdate' and 'contactRelationUpdate' as events (working with Skype, taken from here: https://docs.microsoft.com/en-us/azure/bot-service/nodejs/bot-builder-nodejs-handle-conversation-events?view=azure-bot-service-3.0) are not working with Kik.
Expected Behavior
Either receive the message of "start-chatting" so we can handle it, or receiving 'conversationUpdate' or 'contactRelationUpdate' as events to handle this.
Actual Results

In my understanding Kik expects the Bot to respond to this event without any prior interaction by humans. Therefore we can't handly anything as we don't get our code triggered until it is too late.

The Bot Framework supports start-chatting messages by sending a ContactRelationUpdateActivity to your bot and responding appropriately.  Since you are not seeing this, please provide your bot handle and i'll look into our logs and see if I can determine where the problem is

Bothandle: novi-app-bot
App Id: 82b2e58f-dec7-4704-89ef-702edfd1aad3

I looked through our logs and I see over 100 messages from Kik to your bot and they all returned 200's back to Kik so i'm not sure why they would be reporting that your bot is not responding.
Is it possible that there could have been messages sent before you configured your bot in the Azure portal (i.e. set the name and api key)?
Did you try to resubmit your bat and see if it's working better now?

Yes, the bot is generally working and responds to messages.
But not to the KIK message type "start-chatting".
From the submission status: "Your bot failed to respond to messages of the following types: start-chatting"

I'd suggest resubmitting your bot and see if Kik rejects it again.  If they do I'd then ask them for more details about the failing calls because I don't see any failed calls in our logs.

We already resubmitted the bot to Kik and it was reject again.
Does the botframework/botbuilder handle this message type: https://dev.kik.com/#/docs/messaging#start-chatting? We do not see this or a similar event in the logs.

We do handle start-chatting messages and send them to the bot as ContactRelationUpdate activities.

We solved this issue. So closing it. Thanks for your effort!

We finally fixed the issue with a extra workaround for kik.
If kik sends the "start-chatting" message. It will be received by the bot twice.

a empty text message (type:message). Within the sourceEvent you see it is a "start-chatting" message.
a contactRelationUpdate event but without the expected "action" property (https://docs.botframework.com/en-us/node/builder/chat-reference/interfaces/botbuilder_d.icontactrelationupdate.html, https://docs.microsoft.com/en-us/azure/bot-service/nodejs/bot-builder-nodejs-handle-conversation-events?view=azure-bot-service-3.0)

Our quickfix:

check the message.sourceEvent for "start-chatting" and discard that message
on "contactRelationUpdate" beside the check (message.action === 'add') for the other platforms, we check if the source is 'kik' and then respond with a welcome message.

While this solution works, the "contactRelationUpdate" event still behaves differently on "kik".

Thanks for the feedback.  It sounds like we should add the Action='add' and include the raw Kik message in the ContactRelationUpdate message we send.  I'll open a work item on our end to get this done.