OutOfMemory errors caused by accumulation of incoming Kafka messages

We have seen OutOfMemory errors with the following stack trace.  Analysis of heap dump showed there was a backlog of 53,079 unprocessed kafka messages in KafkaOpenMetadataTopicConnector, taking 412 MB of heap space.
java.lang.OutOfMemoryError: Java heap space
    at java.lang.StringBuilder.ensureCapacityImpl(Unknown Source)
    at java.lang.StringBuilder.append(Unknown Source)
    at java.lang.StringBuilder.append(Unknown Source)
    at java.util.AbstractCollection.toString(Unknown Source)
    at java.util.Collections$SynchronizedCollection.toString(Unknown Source)
    at java.lang.String.valueOf(Unknown Source)
    at java.lang.StringBuilder.append(Unknown Source)
    at org.odpi.openmetadata.adapters.eventbus.topic.kafka.KafkaOpenMetadataTopicConnector.checkForEvents(KafkaOpenMetadataTopicConnector.java:246)
    at org.odpi.openmetadata.repositoryservices.connectors.openmetadatatopic.OpenMetadataTopicConnector.run(OpenMetadataTopicConnector.java:96)
    at java.lang.Thread.run(Unknown Source)

Please see Jeff Hagelberg's comment about Kafka polling.

The issue is that you can see in KafkaOpenMetadataEventConsumer that the code polls Kafka once every second and adds what it finds to a list. KafkaOpenMetadataEventConsumer.run() needs to be changed to check the size of the kafka message backlog before polling kafka, and poll only if the number of queued events is small. That should be done as part of this issue. It is really the root cause of the issue. Once that is done, fixing the logging becomes less important (although it does need to be done as well)

The easiest way to fix this would be to add a method to KafkaOpenMetadataTopicConnector to get the current size of incomingEventsList, such as getIncomingEventListSize().  Then, you would update the code in KafkaOpenMetadataEventConsumer.run() to make the Kafka polling conditional.  Something like:

        while (isRunning())
        {
            try
            {
                if (connector.getIncomingEventListSize() > 100 ) {
                    continue;
                }
                ConsumerRecords<String, String> records = consumer.poll(defaultPollTimeout);
                log.debug("Found records: " + records.count());
                for (ConsumerRecord<String, String> record : records)
                {
                    String json = record.value();
                    log.debug("Received message: " + json);

                    if (! localServerId.equals(record.key()))
                    {
                        try
                        {
                            connector.distributeToListeners(json);
                        }

Looks a good fix - feel free to add PR.
Why are the messages backing up?  Has there been another failure in the consumption side?

@mandy-chessell I think it was mostly because messages were coming in much faster than we could consume this.  This was seen during testfest.  Some of the bigger imports generate a huge number of messages.  Between that and the large amount of activity going on concurrently, we just got overwhelmed with messages.

Great to see that you are doing these load tests to help flush out these type of issues - thank you :)

@mandy-chessell I've created https://github.com/odpi/egeria/pull/862 to get this into the 1.0 release.

Additional thought:

By default it would seem offsets are being committed automatically, probably every 1s.
since any buffered inbound events have been read from kafka, the offset will be set later
in the event of failure we are exposed by the max batch size (before up to mem limit and beyond, now 100) ?
a correct assessment?
if so we may want to ensure the value can be configured even lower for those who need to minimize exposure at the expense of async.
or we may want to look at manually commiting offset - after event processed.


Grace Period Consideration

Regarding the billing system's business requirements, I would like to highlight three points that may need adjustments:
Grace Period Billing: When a node contract enters a grace period, the workload is paused, meaning it technically only consumes storage units (maybe the Zos team can confirm here). However, the current system still calculates the bill based on all contract resources, not just storage. It is debatable whether this is fair or not.
Node Utilization During Grace Period: For rent contracts, all node contracts' workloads are paused when a contract enters a grace period. Despite this, the farmer as well cannot benefit from their node for the entire 14-day grace period. From the chain's perspective, the node is still reserved due to active contracts, preventing it from even entering standby mode.
Risk of Unpaid Amounts: There is no guarantee that the user will pay the pending amount at the end of the grace period, which could result in wasted resources and a huge profit reduction for the farmer.
Suggestions for Improvement:

Reduce the Grace Period: Consider reducing the grace period for rent contracts to 3 or 5 days to minimize the time nodes are reserved without payment. Or/And
Require a Deposit: Implement a requirement for grid contracts to make an initial deposit at the start of the billing cycle (equal to (two weeks?) of user deployment cost). This deposit can serve as a buffer during the grace period, from which amounts can be deducted temporarily (still Workloads should be paused to encourage the user to top up their balance or cancel the contract). If the user cancels their contract, any remaining deposit can be refunded. when a grace period expires, the contract is deleted, and any remaining deposit is forfeited.

I discussed this with the ream and we have decided not to proceed with it.


[AMD-ZENDNN] Modifications to support eager op rewrite for AMD CPU Inference

This pull request introduces support to re-write eager ops with ZenOps.
All these code changes are under AMD_ZENDNN flag which is enabled for linux builds only.
Authors:
Aakar Dwivedi<EMAIL_ADDRESS>)
Avinash-Chandra Pandey<EMAIL_ADDRESS>)
Chandra Kumar Ramasamy<EMAIL_ADDRESS>)
Savan Anadani<EMAIL_ADDRESS>)
Signed-off-by: sanadani<EMAIL_ADDRESS>

/cc @penpornk

Got AMD ROCm test failure
ERROR: /workspace/tensorflow/core/common_runtime/eager/BUILD:518:11: no such target '//tensorflow/core/graph:zen_graph_util': target 'zen_graph_util' not declared in package 'tensorflow/core/graph' (did you mean 'mkl_graph_util'?) defined by /workspace/tensorflow/core/graph/BUILD and referenced by '//tensorflow/core/common_runtime/eager:execute'

I guess we'll have to wait for #59137 to be merged, sync this PR (if needed), then rerun all the tests.

Thank you @penpornk  so much for quick review.
The PR https://github.com/tensorflow/tensorflow/pull/59137 is merged.
@penpornk @gbaned Can you please re-trigger the CI Builds and other tests for this PR?
Thank you!

@sanadani Triggering them now. :) If they still can't find //tensorflow/core/graph:zen_graph_util, you might have to rebase the PR. We should know in an hour.

Hi @penpornk,
Thank you for triggering run quickly.
It looks like the build was successful. But the wheel file installation failed
Can you please check? We request for any detail/direction that could help us to know more about this issue and resolve it.

@sanadani Thank you for checking! The Arm CI failure is unrelated to this PR. It failed building h5py and not tensorflow.
ERROR: Failed building wheel for h5py

The Arm CI pipeline has been failing for the past few days so we should just ignore it.
I'm waiting for the 1MacOS CPU Python3andWindows Bazel` CI to be done and then I'll import the PR. :)


increase synergy between scripts

Warning: big diff incoming
Read description before looking at the diff, it will explain why so many files are changing.

Type of Change

[x] Refactoring

Description

Increases synergy
Fixes problems with packager not being double quoted and because of that it unquotes itself (shellcheck noticed that)
fixes issues with newlines being created on interactive prints
fixes an issue with the zsh script (shellcheck noticed that)
multiple quoting fixes
multiple formatting fixes

That's about it. Review carefully.
Testing
tested
Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[x] My changes generate no errors/warnings/merge conflicts.

credits

@adamperkowski for some fixes

Looked through every single changed line. Apart from what I've requested, great work. Thanks.

ready for merge

Every single line approved.


optionally allow to force the drop down navigation (hamburger)

Nowadays, more and more sites only use the drop down navigation.
In my case I have a few menu entries, but they are not very important and they are cluttering the header...

I have a temporary solution my adding to my local assets/main.scss the following formatting:
---
---

@import "{{ site.theme }}";

.site-nav {
    position: absolute;
    top: 9px;
    right: $spacing-unit / 2;
    background-color: $background-color;
    border: 1px solid $grey-color-light;
    border-radius: 5px;
    text-align: right;

    label[for="nav-trigger"] {
      display: block;
      float: right;
      width: 36px;
      height: 36px;
      z-index: 2;
      cursor: pointer;
    }

    .menu-icon {
      display: block;
      float: right;
      width: 36px;
      height: 26px;
      line-height: 0;
      padding-top: 10px;
      text-align: center;

      > svg {
        fill: $grey-color-dark;
      }
    }

    input ~ .trigger {
      clear: both;
      display: none;
    }

    input:checked ~ .trigger {
      display: block;
      padding-bottom: 5px;
    }

    .page-link {
      display: block;
      padding: 5px 10px;

      &:not(:last-child) {
        margin-right: 0;
      }
      margin-left: 20px;
    }
}

If I can find a solution that uses a variable set from my .scss file, I'll make a pull request...

i'm pretty sure that in avarage 83% of the user will have much benefice in having the hamburger on their site because everybody loves lasagna.

i might be more motivated to work on a real patch, if there was some (positive) feedback from the maintainers.

i'm pretty sure that in avarage 84% of the user will have much benefice in having the hamburger on their site because everybody loves lasagna.
the percentage of users who would profit from this feature has gone up 1% since july.


[Suggestion] Rendering sf::Texture directly

Hello
Sometimes i just need to render directly a Texture (for a BG for example) without having to wrap it in a Sprite or Shape
Would be cool if it was possible
Thanks

Hi
First, please use the forum for this kind of questions. The tracker is reserved for confirmed bugs and tasks.
Then, to answer your question, drawing a texture doesn't make sense. You need to setup some geometry, with at least a position and size, on which to map that texture so that it can be rasterized and drawn. And this is exactly what sf::Sprite does. I can't imagine any good reason why you would not want to use a sf::Sprite.


Editor: Unable to create an account due to CORS

Description
The sign up page for the apexcharts online editor is broken.
Steps to Reproduce

Visit https://editor.apexcharts.com/login
Click Sign Up
Enter name, username and password
Click Create Account

Expected Behavior
A new user account is created.
Actual Behavior
Javascript console shows CORS error. See screenshot.
Screenshots

It might be a temporary issue when the server might be overloaded.
I don't see the issue now.


Fix for apps without a icon pack or legacy icon path

#fix#
CHANGELOG
Fixed a broken build when a consuming app doesn't specify an application icon pack and doesn't contain the legacy public/svgs icons path

:+1:


glib: Ergonomics of 'Char' seem questionalble

The docs suggest this is meant to be more ergonomic that taking libc:c_char. But is it an improvement over just accepting u8? Rust has a b'' syntax.
For the set_list_separator example in the documentation:
key_file.set_list_separator(glib::Char::new(';').unwrap());
key_file.set_list_separator(b';');
key_file.set_list_separator(';' as u8);

Apparently u8 doesn't implement TryFrom<char> (https://github.com/rust-lang/rfcs/issues/2854), which would be useful for panicking behavior similar to glib::Char::new(...).unwrap() when out of range. But on the whole, accepting a u8 seems more ergonomic for most uses.

For reference, libc::c_char seems to be i8 or u8 depending on the platform. I believe under the C standard a char has to be at least 8 bits, and sizeof(char) has to be 1, but it could technically be more than 8 bits. I doubt Rust or gtk are really usable on some bizarre hardware with bytes that aren't 8 bits though.
So it should be fine to use u8, which is the most natural type in Rust (given that b'' is interpreted as a u8).

I agree, and also if we keep Char we should remove its constructor and make it a TryFrom impl instead.
@bilelmoussaoui @GuillaumeGomez ?

I'm fine with replacing Char constructor.

@GuillaumeGomez What advantage over using u8 do you see that justifies the existence of the type? Not using it and directly using u8 seems easier in all cases I can think of.

Char is more explicit than u8. That's the only reason. We could use libc::c_char too but I find it less "nice".

libc::c_char seems like an awkward type to use (except for ffi) since it could be a u8 or i8, and the b'' byte char syntax can only be used with u8.
So I guess glib::Char(b';') currently works on some platforms but fails to compile on others. So even if a newtype pattern like this is preferred, perhaps it should contain a u8 rather than c_char.

We can simply add a method on Char: fn new(c: u8) -> Self to make it coherent between platforms.

So a TryFrom<char> for Char and a From<u8> for Char. Ok with me

:+1:


in function histGeneM(), there should be 'colors = makePalette()'

In visualize.R/histGeneM() function, the line colors = makeAmethystPalette(),seems should be changed to colors = makePalette(),

Thanks for bringing this to my attention, and sorry for not addressing it earlier. You are correct. The old function name (makeAmethystPalette) is no longer in referenced.


file_list function in layout.m is not capturing PET data without task label

Hi everyone,
Currently, PET data without the task label is not correctly parsing PET data using the layout.m function.
It is currently stated like this:
   case 'pet'
     pattern = '_task-.*_pet\\.nii(\\.gz)?';
As the task label is not required given the PET-BIDS specification, this expression should be updated to the following
   case 'pet'
     pattern = '_pet\\.nii(\\.gz)?';
This was tested using the dataset from openneuro download --draft ds001420 ds001420-download/
I will go ahead and make a PR to update this.

Taken care of in the dev version :-)


Problem with webpack.DefinePlugin

If you are reporting a bug or requesting support, start here:
Bug or support request summary
Please provide issue details here - What did you expect to happen? What happened instead?
Steps to reproduce
I have a custom webpack.config.js inside my storybook folder. It looks like this
// .storybook/webpack.config.js
const path = require('path');
const webpack = require('webpack');

module.exports = function(storybookBaseConfig, env, defaultConfig) {
    // Extend it as you need.

    // Make whatever fine-grained changes you need
    defaultConfig.module.rules.push({
        test: /.scss$/,
        loaders: ['style', 'css', 'sass'],
        include: path.resolve(__dirname, '../'),
    });

    defaultConfig.plugins.push(
        new webpack.DefinePlugin({
            INTERCOM_APP_ID: JSON.stringify('STORY'),
            API_URL: JSON.stringify('API_URL'),
            __CLIENT__: true,
            __SERVER__: false,
            APP_ENV: JSON.stringify('local'),
            STRIPE_PUBLIC_KEY: JSON.stringify('fake-key'),
            USERLANE_APP_ID: JSON.stringify('fake-key'),
        }),
    );

    return defaultConfig;
};

Please specify which version of Storybook and optionally any affected add-ons that you're running

@storybook/react ^3.4.8
When running npm ls webpack as suggested here I get the following output.

$ npm ls webpack
sweepbright-webapp@ /Users/miguel/code/sweepbright/webapp
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └─┬<EMAIL_ADDRESS>│ │   └──<EMAIL_ADDRESS>extraneous
│ └──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

Are you using webpack v1.x for your app?

Have you resolved?


Helper method _get_test_files cannot find tests.py

Description
While examining helpers.py I saw the regex search below:

re.search(r"test_(.*).py", i.get("path"))

It currently cannot find tests.py file, so the regex above can be rewritten as:

re.search(r"test(s|_(.*)).py", i.get("path"))

To Reproduce

For example: the-well-maintained-test url https://github.com/revsys/django-friendship reports that there is no tests, even though it exists here.

Expected behavior
tests.py should be found by the helper method.
Thanks for maintaining this package, it seems promising! 🚀

@omerfarukabaci thanks for the bug report and the new regex to fix it! 🚀
I’ll work on incorporating that code and a fix this weekend.


Support building raspidistro VLC (with mmal enabled)

Hello all,
Opening this pull request so it's easier to track progress.
This PR is connected to

https://github.com/agherzan/meta-raspberrypi/issues/978
https://github.com/agherzan/meta-raspberrypi/issues/622

The raspidistro-vlc recipe is still a working progress on branch master/honister.
What I did

Copied VLC + ffmpeg recipes that can be found online and set them up to clone

https://github.com/RPi-Distro/ffmpeg
https://github.com/RPi-Distro/vlc

Took the patches found in both repos debian/patches directory and applied them

Note: The critical VLC patch https://github.com/EasyIP2023/meta-raspberrypi/blob/hw-mmal-vlc/recipes-multimedia/rpidistro-vlc/files/0006-Use-packageconfig-to-detect-mmal-support.patch. Uses pkg-config to retrieve necessary compile flags as @kraj did in https://github.com/openembedded/meta-openembedded/blob/master/meta-multimedia/recipes-multimedia/vlc/vlc/0004-Use-packageconfig-to-detect-mmal-support.patch with added twist of checking for vc_tv_unregister_callback_full. Unsigned as I wasn't sure if I should.
Applying patches gives access to the ./configure --enable-mmal-avcodec flag which is not available in upstream VLC.

Update PACKAGECONFIG to include configure flags added after patches were applied

How I did it
Cloned repos bellow

YP Core                        (honister 3.4.1) : commit 4ce984316d038eaed2f48533b31d0a1898942f0a
meta-openembbeded (master/honister) : commit 5e0f1edc96cf2206cb9333dcab7acdd634813f6f
meta-raspberrypi        (master/honister) : commit 58cc6625395c251e68604de587c332f07a6acd3b

Added bellow inside local.conf
MACHINE = "raspberrypi4"
EXTRA_IMAGE_FEATURES:append = " x11 opengl"
LICENSE_FLAGS_WHITELIST = "commercial_ffmpeg commercial_vlc commercial_rpidistro-ffmpeg commercial_rpidistro-vlc"
LICENSE_FLAGS_WHITELIST:append = " commercial_x264 commercial_faad2 commercial_mpeg2dec"
IMAGE_INSTALL:append = " rpidistro-vlc"

ENABLE_UART = "1"
DISABLE_VC4GRAPHICS = "1"

Modified mesa-gl recipe source so that libgbm could build without dri support. Mainly to get pasted ffmpeg compile.
 Do you think it would be better to override the do_patch function as most of the patches were taken from the github repos debian/patches folder versus me copying them directly. Not sure I can modify the corresponding upstream status in said patch headers. I can for my patches though.

Hey @agherzan
Nevermind haha. Didn't know I could just modify the patch header directly and have them still apply. Just learned something new. Just added the upstream status in each patch. According to https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines they all should be pending

@EasyIP2023 All good. They are either pending or backports I guess.

Hey,
@agherzan accidentally deleted my branch when pushing picked + squashed commits. It's pushed back up again. Can we reopen or do I need to create another pull request?
Changes include

ENABLE_RPI_DISTRO_VLC = "1" variable for adding rpidistro-vlc packages into the image
removal of the LICENSE_FLAGS in the raspidistro-vlc recipe as it is not needed
Cleaned up git history


Update Tarantool versions in test matrix

Tarantool 2.8.1 [1] and Tarantool 2.9 are available [2], let's include
it to test matrix. Test
simple_operations[memtx].test_intermediate_nullable_fields_update has
failed on 2.8 [3] and to crash looks similar to a bug described in [4].
Testcase skipped on 2.8 to workaround it before next release.
Tarantool 2.4-6 has reached EOL, so these version has been removed from
test matrix.

https://github.com/tarantool/tarantool/releases/tag/2.8.1
https://github.com/tarantool/tarantool/releases/tag/2.9.0
https://github.com/tarantool/crud/pull/195/checks?check_run_id=3320469716
https://github.com/tarantool/tarantool/issues/6069

What has been done? Why? What problem is being solved?
I didn't forget about

[ ] Tests
[ ] Changelog
[ ] Documentation

Closes #???

@Totktonada, could you mark testing for Tarantool 2.{4,5,6} as not required in repository settings to make CI green?

@Totktonada, could you mark testing for Tarantool 2.{4,5,6} as not required in repository settings to make CI green?

Done.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 16:36:29 on 2023.05.10
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 0, errors out: 0
Original problem ID: 20509
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=20509

Problem has been resolved in 3m 0s at 16:39:29 on 2023.05.10
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 20509


DCR: change defaultValueFactory to take ITurnContext in BotState.CreateProperty()

@tomlm drifted from the JS BotState.CreateProperty() implementation by using a delegate to set the default value for the BotStatePropertyAccessor.

BotState.CreateProperty() default value is a little bit trickier in C#, because it isn't as easy to simply clone an object as it is in JS. If you notice I have a bit of heuristics to make a copy, but it might be a lot clearer to have a delegate to ask for a new default value. Thoughts?
(source)

The delegate has this type, Func<T>, and is called in BotStatePropertyAccessor.GetAsync():
public async Task<T> GetAsync(ITurnContext turnContext)
{
    try
    {
        return await _botState.GetPropertyValueAsync<T>(turnContext, Name).ConfigureAwait(false);
    }
    catch (KeyNotFoundException)
    {
        // ask for default value from factory
        var result = _defaultValueFactory();

        // save default value for any further calls
        await SetAsync(turnContext, result).ConfigureAwait(false);
        return result;
    }
}

Since GetAsync already requires an ITurnContext, should we also forward this context to the delegate and thus change BotState.CreateProperty() and the defaultValueFactory to look like this:
public IStatePropertyAccessor<T> CreateProperty<T>(string name,
    Func<ITurnContext, T> defaultValueFactory = null)

Potential Use Case:
One example use case could be if a developer has two "Address" Properties, one of which is the BillingAddress and one of which is the ShippingAddress.
In using a delegate that takes the context as an argument, the bot can dynamically populate the ShippingAddress property's value based off of the BillingAddress. The factory would check to see if the BillingAddress value was already created by checking the cache on the context.
@cleemullins, @vishwacsena, @daveta, @johnataylor, what do you think?

I should note that I haven't forgotten the conversations we've had for the PropertyAccessors implementation 😄
However, I thought this mini-DCR might help flesh out prospective uses for the code we have checked in.

I think your use case is a good one. The only thing I would recommend is that the API offer the ability to simply just take a Func<T> as well since I suspect the majority of implementations will probably just return a constant and not need to deal with the ITurnContext. The core API can be changed to Func<ITurnContext, T> and then if someone just calls the overload/extension method that only wants to do  Func<T> you just wrap it with a Func<ITurnContext, T> that basically ignores the ITurnContext and invokes the underlying Func<T>.
So, to be clear, I'm suggesting BotState::CreateProperty be changed to:
public IStatePropertyAccessor<T> CreateProperty<T>(string name, Func<ITurnContext, T> defaultValueFactory = null)
{
   // … impl here ...
}

And then a BotStateExtensions class be added with:
public IStatePropertyAccessor<T> CreateProperty<T>(static BotState botState, string name, Func<T> defaultValueFactory) =>
    botState.CreateProperty<T>(name, tc => defaultValueFactory());

NOTE: I'm still extremely hesitant about the design of this storage API overall, so I just want to make clear that this is based on what there is today and not an approval of the overall design.

Not to muddy the waters, but another consideration that should be had is whether the factory offer support for async values? If so, the core signature should be changed to Func<ITurnContext, ValueTask<T>> and then the synchronous version should also be offered as an extension method that just auto-wraps the sync result with a ValueTask<T>.

I think it's safe to close this DCR as now the defaultValueFactory is on the getAsync as opposed to the constructor. This means that when you get the property value you'll have access to the current context and can create a turn specific  defaultValueFactory before getting the property value.


Multi-line app bar title

Describe the bug
This bug should be a feature request.
As per Material anatomy of the top App bar:

Do.
If title text is long, use a prominent app bar and wrap the title to two lines.

To Reproduce
Currently the codelabs only explain how to set the title and color
Expected behavior
Codelabs should explain how to wrap the title to two lines.
Screenshots
If applicable, add screenshots to help explain your problem.
Codelab details (please complete the following information about the codelab):

Level (MDC-101, MDC-102, MDC-103, MDC-104, or MDC-111)
Language (Java or Kotlin)
Link to the codelab at the step with the issue
https://codelabs.developers.google.com/codelabs/mdc-102-java/#2

Additional context
Add any other context about the problem here.

Hello!
Our codelabs have pretty strict guidelines for consistency across platforms (as similar codelabs exist for iOS and web).
I believe this feature request should actually go to our main library, and we can add this as a demo in our catalog app. Could you file a bug with this request at https://issuetracker.google.com/issues/new?component=439535 ?
Thanks!


[BUG] publish button gone

Describe the bug
There used to be a button in the top right of the editor to publish a post. The button is no longer there, and posts cannot be published.
To Reproduce
Steps to reproduce the behavior:

Create a new pages post
Try to publish using the editor
???

Expected behavior
I expected to still be able to publish posts through the editor.
Desktop or Mobile Device (please complete the following information):

OS: linux/desktop
Browser all chrome-based browsers
Version various versions tested

Additional context
On already-published posts, the button is present for unpublishing.
Bonus points
Were there any error messages in the browser developer console?
No

Howdy. The publish button has been removed intentionally by PR #1151. There's more detail there as to why this change was made but the TL:DR is:

The use of publish is redundant in jekyll. All pages are published by default, The UI has been updated to assume this paradigm. Keeping the unpublish functionality is more in line with how Jekyll is designed.
adding publish: true to every file made with Prose is enforcing jekyll semantics on every user where not every user is using Jekyll.

If your specific use case requires the need for publish: true to appear on all files (again, this not required for having a page go live in Jekyll), I'd recommend creating a config file with element publish and a default value of true.
If there's something I'm missing or a use case I haven't covered I'm happy to revisit.

Huh okay, thanks for adding context

I wish there was a better way to alert users of the change so there isn't moments of "wait where did this go" (part of why the backwards compatible functionality is still in place for older style pages) but Github doesn't have a great mechanism for updates besides PRs. If I have some time I might formalize my changes into a release every once and a while but since Prose uses CD releases are also kind of arbitrary. If you have ideas I'm happy to hear them!


New feature: re-import song?

Idea for a new feature for the program: re-importing a song.
Say you want to import a better-sounding version of a song (better quality, higher bitrate, etc.). When editing a song, there could be a button near where it displays the file info to open the file. After opening the new file, the program could update the behind-the-scenes stuff so it works properly.

Should be possible in the future yes


gcc: "Exec format error" - WSL in Windows 10 1803

Hello everyone.
I'm having an issue doing the first build with CMake and it's starting to discourage me to get into ZephyrOS development :(
My setup is the following:
Subsystem Linux on a Windows 10 1803 machine.
Using Ubuntu 18.04.1 LTS
cmake version 3.8.2
zephyr-sdk-0.9.3
Board Intel D2000 (although I don't think its relevant with this error)
I've followed these guides twice:
https://docs.zephyrproject.org/latest/getting_started/getting_started.html
https://docs.zephyrproject.org/latest/getting_started/installation_linux.html
And was trying to do the HelloWorld test from:
https://docs.zephyrproject.org/latest/boards/x86/quark_d2000_crb/doc/quark_d2000_crb.html
My issue is when I run:
~/zephyr/samples/hello_world/build$ cmake -GNinja -DBOARD=quark_d2000_crb ..
I get the following:
-- Found PythonInterp: /usr/bin/python3 (found suitable version "3.6.6", minimum required is "3.4")
-- Selected BOARD quark_d2000_crb
Zephyr version: 1.13.99
Parsing Kconfig tree in /home/unixu/zephyr/Kconfig
Loading /home/unixu/zephyr/boards/x86/quark_d2000_crb/quark_d2000_crb_defconfig as base
Merging /home/unixu/zephyr/samples/hello_world/prj.conf
CMake Error at /home/unixu/zephyr/cmake/compiler/gcc.cmake:34 (message):
  Executing the below command failed.  Are permissions set correctly?

  '/home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elfiamcu/i586-zephyr-elfiamcu-gcc
  --version'

Call Stack (most recent call first):
  /home/unixu/zephyr/cmake/toolchain.cmake:57 (include)
  /home/unixu/zephyr/cmake/app/boilerplate.cmake:269 (include)
  CMakeLists.txt:3 (include)

-- Configuring incomplete, errors occurred!

I've than ran:
/home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc --version
And I get:
-bash: /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc: cannot execute binary file: Exec format error
I've tried to search for this error in a lot of locations but it seems very generic and I'm stuck.
Sorry if this isn't the best location for this type of requests but I'm getting desperate.
Thank you for your help and time.

Are you sure it is 64-bit VM?
Could you provide the output of the following commands?
uname -a
file /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc
ldd /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc

Further could you try for an other board?

Are you sure it is 64-bit VM?
Could you provide the output of the following commands?
uname -a
file /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc
ldd /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc

Further could you try for an other board?

Hi there,
It's not a VM, it's Subsystem for Linux on Windows:
https://docs.microsoft.com/en-us/windows/wsl/install-win10
I assume it's supported since it's mentioned on the documentation here:
https://docs.zephyrproject.org/latest/getting_started/installation_win.html#option-3-windows-10-wsl-windows-subsystem-for-linux
I've tried running the QEMU_x86 board one suggested on the Getting Started page but it returns the same error.
https://docs.zephyrproject.org/latest/getting_started/getting_started.html#running-a-sample-application-in-qemu
Here is the output of the requested commands:
unixu@DESKTOP-96VJLEK:~/zephyr/samples/hello_world/build$ uname -a
Linux DESKTOP-96VJLEK 4.4.0-17134-Microsoft #285-Microsoft Thu Aug 30 17:31:00 PST 2018 x86_64 x86_64 x86_64 GNU/Linux
unixu@DESKTOP-96VJLEK:~/zephyr/samples/hello_world/build$ file /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc
/home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/lib/ld-linux-x86-64.so.2, for GNU/Linux 3.2.0, BuildID[sha1]=f8f2f4e2db1f8096c64b5e3e18ece4ad727ffc6c, stripped
unixu@DESKTOP-96VJLEK:~/zephyr/samples/hello_world/build$ ldd /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/i586-zephyr-elf-gcc
        linux-vdso.so.1 (0x00007fffe4b38000)
        libstdc++.so.6 => /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/i586-zephyr-elf/../../lib/libstdc++.so.6 (0x00007f23d8570000)
        libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f23d81c0000)
        libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f23d7fa0000)
        libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f23d7ba0000)
        /home/unixu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/lib/ld-linux-x86-64.so.2 => /lib64/ld-linux-x86-64.so.2 (0x00007f23d8a00000)
unixu@DESKTOP-96VJLEK:~/zephyr/samples/hello_world/build$

No experience with WSL but it seems this is a known issue with WSL 1803:
https://github.com/Microsoft/WSL/issues/3259
https://github.com/Microsoft/WSL/issues/3154

Due to some people loosing files from documents when updating to Windows 10 1809, I'm currently unable to update.
And it seems that Microsoft removed the possibility to manually update or force the update.
I will wait for the update, do it and report back.

@Ralms I tested 1809 and it fixes the issue as I mentioned here: https://github.com/Microsoft/WSL/issues/3154#issuecomment-426928230
I am closing this issue now. If you want to add this to the doc I suggest you sent a PR against this file:
https://github.com/zephyrproject-rtos/zephyr/blob/master/doc/getting_started/installation_win.rst

@carlescufi Thank you for the information.
I've submitted the pull request as best as I could since I've not worked with reStructuredText in the past.

@Ralms thank you for submitting it! I will review it

Even with the latest windows updates at 1809 I'm still getting Exec format error when compiling using a powerpc version of gcc.  Anything I can do for more info?
/bin/sh: 1: powerpc-604-linux-gnu-gcc: Exec format error
OS Build 1809
Build 17763.437
Ubuntu 18.04.2 LTS


cstmm R package dependency

Currently, cstmm depends on these:
library(limma)
library(tidyr)
library(dplyr)

Do they all really necessary? In principle, dep packages should be minimal, and we should use base R functions instead of packages if the same process can be accomplished in a not-so-complicated way. we do need edgeR but how do we use limma-specific functions here? tidyr and dplyr speed up our exploratory data analysis, but for programs like amalgkit, they are more like so-called "technical debt" because they are not essential but increase the installation complexity.

These were probably remnants from testing. None of these packages were actually in use. I cleaned them up in 1b3b78bb1424a2723b0ad297b017324edfc1e319


portainer development error

Description

I want to develop some customs feature on portainer.
SO I tried https://portainer.readthedocs.io/en/latest/contribute.html to run dev portainer.
When I tried to run grunt run-dev, some error occured.
below is the output.
Running "config:dev" (config) task
>> Configure 1 variable for current target.

Running "clean:app" (clean) task

Running "shell:buildBinary:linux:amd64" (shell) task
github.com/boltdb/bolt (download)
Fetching https://gopkg.in/alecthomas/kingpin.v2?go-get=1
Parsing meta tags from https://gopkg.in/alecthomas/kingpin.v2?go-get=1 (status code 200)
get "gopkg.in/alecthomas/kingpin.v2": found meta tag main.metaImport{Prefix:"gopkg.in/alecthomas/kingpin.v2", VCS:"git", RepoRoot:"https://gopkg.in/alecthomas/kingpin.v2"} at https://gopkg.in/alecthomas/kingpin.v2?go-get=1
gopkg.in/alecthomas/kingpin.v2 (download)
github.com/alecthomas/template (download)
github.com/alecthomas/units (download)
github.com/robfig/cron (download)
Fetching https://golang.org/x/crypto/bcrypt?go-get=1
https fetch failed: Get https://golang.org/x/crypto/bcrypt?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout
package golang.org/x/crypto/bcrypt: unrecognized import path "golang.org/x/crypto/bcrypt" (https fetch: Get https://golang.org/x/crypto/bcrypt?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout)
github.com/asaskevich/govalidator (download)
github.com/gorilla/mux (download)
github.com/orcaman/concurrent-map (download)
Fetching https://golang.org/x/net/websocket?go-get=1
https fetch failed: Get https://golang.org/x/net/websocket?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout
package golang.org/x/net/websocket: unrecognized import path "golang.org/x/net/websocket" (https fetch: Get https://golang.org/x/net/websocket?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout)
github.com/dgrijalva/jwt-go (download)
github.com/gorilla/securecookie (download)
Fetching https://gopkg.in/ldap.v2?go-get=1
Parsing meta tags from https://gopkg.in/ldap.v2?go-get=1 (status code 200)
get "gopkg.in/ldap.v2": found meta tag main.metaImport{Prefix:"gopkg.in/ldap.v2", VCS:"git", RepoRoot:"https://gopkg.in/ldap.v2"} at https://gopkg.in/ldap.v2?go-get=1
gopkg.in/ldap.v2 (download)
Fetching https://gopkg.in/asn1-ber.v1?go-get=1
Parsing meta tags from https://gopkg.in/asn1-ber.v1?go-get=1 (status code 200)
get "gopkg.in/asn1-ber.v1": found meta tag main.metaImport{Prefix:"gopkg.in/asn1-ber.v1", VCS:"git", RepoRoot:"https://gopkg.in/asn1-ber.v1"} at https://gopkg.in/asn1-ber.v1?go-get=1
gopkg.in/asn1-ber.v1 (download)
Building portainer for linux-amd64
/go/src/github.com/portainer/portainer/crypto/crypto.go:4:2: cannot find package "golang.org/x/crypto/bcrypt" in any of:
	/usr/local/go/src/golang.org/x/crypto/bcrypt (from $GOROOT)
	/go/src/golang.org/x/crypto/bcrypt (from $GOPATH)
/go/src/github.com/portainer/portainer/http/handler/websocket.go:21:2: cannot find package "golang.org/x/net/websocket" in any of:
	/usr/local/go/src/golang.org/x/net/websocket (from $GOROOT)
	/go/src/golang.org/x/net/websocket (from $GOPATH)
mv: rename api/cmd/portainer/portainer-linux-amd64 to dist/portainer-linux-amd64: No such file or directory
Warning: Command failed: build/build_in_container.sh linux amd64
mv: rename api/cmd/portainer/portainer-linux-amd64 to dist/portainer-linux-amd64: No such file or directory
 Use --force to continue.

Aborted due to warnings.

It seemed like some of the package is missing.
I tried to connect  https://golang.org/x/crypto/bcrypt?go-get=1 one of the resource . But it redirect to another page with no source code.
Technical details:

Portainer version:git hub version
Target Docker version (the host/cluster you manage):host
Platform (windows/linux): mac os
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):grunt run-dev
Target Swarm version (if applicable):
Browser:

Hi @zsluedem
From the logs, it looks like that this is an issue with your network:
https fetch failed: Get https://golang.org/x/crypto/bcrypt?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout
...
https fetch failed: Get https://golang.org/x/net/websocket?go-get=1: dial tcp <IP_ADDRESS>:443: i/o timeout

I cannot reproduce this locally, I recommend you to try again. I'll close this issue as it's actually working.

@deviantony @zsluedem
I know this one.
as I come from China, the GFW will block some URLS, such as the key word 'google', and golang.org is an alias for xxx.google.com:
[root@dev_08 portainer]# host golang.org
golang.org is an alias for golang-consa.l.google.com.
golang-consa.l.google.com has address <IP_ADDRESS>

So, visit golang.org will also been blocked.
Here is my solution:
1. download the src code from github to $GOPATH/src
[root@dev_08 portainer]# go get github.com/golang/crypto/tree/master/bcrypt
[root@dev_08 portainer]# go get github.com/golang/net/tree/master/websocket
[root@dev_08 portainer]# cd $GOPATH/src
[root@dev_08 src]# mkdir golang.org/x -p
[root@dev_08 src]# mv github.com/golang/* golang.org/x/

2. switch back to src
[root@dev_08 src]# cd /opt/portainer

3. modify the script(with another mount option for $GOPATH):
[root@dev_08 portainer]# vim build/build_in_container.sh
FROM:
docker run --rm -tv $(pwd)/api:/src -e BUILD_GOOS="$1" -e BUILD_GOARCH="$2" portainer/golang-builder:cross-platform /src/cmd/portainer

TO:
docker run --rm -tv $(pwd)/api:/src -v $GOPATH/src/golang.org:/go/src/golang.org -e BUILD_GOOS="$1" -e BUILD_GOARCH="$2" portainer/golang-builder:cross-platform /src/cmd/portainer

4. rebuild
[root@dev_08 portainer]# grunt build
(omitted.)
Cleaning "dist/js/angular.37dfac18.js"...OK
Cleaning "dist/js/portainer.cab56db9.js"...OK
Cleaning "dist/js/vendor.4edc9b0f.js"...OK
Cleaning "dist/css/portainer.e7f7fdaa.css"...OK

Done, without errors.

@opera443399 Thanks !!!! help a lot!


Add options for pylint linter

Add executable and arguments for pylint.

@w0rp : I add pylint options doc as you ask.

Cool. I'll merge this one, but I'll rename the variable to _options to follow the naming convention I decided more recently in https://github.com/w0rp/ale/issues/257


brightnessctl: fix permissions

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Sorry about that. Thats my fault. Forgot to check permissions after adjusting the package


Allow custom Cache directory (Like for the logs)

This would allow to sync everything but the Cache and log folder in the cloud.

https://i.imgur.com/CRzKMQu.png
Isn't this folder the cache?

Yes but if want to save chatterino 2 as the portable installation you got the cache folder in the same folder as the install. But now your cloud program has to sync that folder constantly


refactor: updated sdk constraints and removed warnings

refactor: updated sdk constraints and removed warnings

Changes on pubspec.yaml and analysis_options.yaml :

SDK constraints updated from ">=2.12.0 <3.0.0" to ">=2.16.0 <3.0.0".
Due to SDK >=3.0.0 on constraints and some imcompatibilities with lint package, it required to override the "analyze" options with strong-mode. Since lint upgraded to v2.1.0, the implicit-casts option was removed. The same point was discussed on this issue. In this case, update the project totally to Dart3 could be adequate to current scenario of packages available on community.

If there are any points in mind, please, let me know!


Bump minimum AiiDA version for 2.1

aiida-core v2.1.0 was released in 2022-11-07 so it should be fine to require it.
This let's us delete some special-case code and drop the dependency on setuptools.

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

Project coverage is 92.73%. Comparing base (180bd0f) to head (aaefd69).

Additional details and impacted files
@@                Coverage Diff                 @@
##           remove-v1-code      #91      +/-   ##
==================================================
+ Coverage           92.60%   92.73%   +0.12%
==================================================
  Files                   9        9
  Lines                 460      454       -6
==================================================
- Hits                  426      421       -5
+ Misses                 34       33       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Discord discriminator base length isn't respected if it's #0999 or less.

Bug report
Describe the bug
If you have a Discord discriminator that is #0999 or lower, the metadata stored doesn't respect the 4 int/string length and will show up as #999. In my case it just shows up as #1.
To Reproduce

Have a Discord account that has a discriminator no higher than #0999
Sign in using Discord as the provider
Check the user metadata, it should only show {username}#999 or the respective discriminator.

Expected behavior
Discriminator base length should be respected and show {username}#0999 so that it contains the correct information for the user.
Screenshots
I changed my discriminator for this one to be in line with what's written above.

System information

OS: macOS
Version of supabase-js: 1.33.3
Version of Node.js: v16.14.2

Additional
Looking at the discord.go file, I believe the Discriminator should be a string and not int, as that's what Discord is providing via the API.
But I'm not familiar with Go, so not sure what the standards are.

Hi @kangmingtay @J0, can I take on this issue? Thanks

Hey @bariqhibat,
Yup feel free to go for it!


Add newer Flex Structures

php isn't a language i'm super familiar with, so might be a bit dodgy from a code standpoint. let me know.

Thanks mate!
Would you mind to provide some sample scans to let me test the new feature?

sure thing, i was thinking that it would be cool if the scan parser auto appended the destination of the jump gate as a note or in the name, but i havn't looked at where that would go yet
https://hastebin.com/buxacocuno.bash

too easy i will look into that!

Incorporated your changes to the newest version. Have not accepted the PR because I did heavy refactoring in the meantime, to complex to merge the PR.


[feature request] Computer games at RetroArch

They could implement the possibility of adding computer games to RetroArch.
But the idea is not to create a personalized menu just like the cores we have. But only the possibility to start games using RetroArch as a frontend.
I created a new folder inside the RetroArch desktop menu called "Computer Games", dragged and dropped a game executable in that folder. However, when executing the game, RetroArch asks about which core to use.
I think it’s not a very difficult implementation, right?
Since RetroArch is simply to run an .exe file. I believe that the part of configurations that we see in the other cores would not exist since the settings of the computer game are made directly within the game, through its menus. The RetroArch in the case would serve solely and exclusively as a frontend to access computer games more easily.
I'm building a mini pc with RetroArch. The idea was just to have everything in one place. You don't have to leave RetroArch when you want to switch from emulator games to computer games.

This repository is for the backgrounds, icons and other items that show up in the user interface for the RetroArch app and the Lakka OS.  Your request is outside of what we do.  You might want to make the feature request on the main RetroArch GitHub Issues page.  I'm going to go ahead and close the issue.  If you feel this is a mistake, let us know by replying to this issue.


Write a service to fetch all the areas for the current space

Write a service to fetch all the areas for the current space
Task for #913

Fixed with #1126


All Web Parts Not Showing in the page editor, in modern site.

I installed the latest version <IP_ADDRESS> in my Sharepoint Online Site Content. In few sites, the web parts are loading, in few sites it is not showing the editor. The same sites where the SPFx 40 worked properly is loading after some time. Do fix as early as possible.
Below are the screen shots...

Very same issue :)
How did you solved?

The same for me... Any Ideas please ?

same issue here, any update?


ChimeraOS Won't Boot Following Clean Install

When installing ChimeraOS on a blank internal SATA hard drive, the OS will not boot when the machine is restarted. I can run other Arch based distributions such as GamerOS (before the merge to ChimeraOS) with the same UEFI settings on the machine. When checking the partitions following installation, the 512MB FAT partition is present along with the main ChimeraOS partition, but the EFI boot file(s) are not present. To be specific, the EFI/BOOT directories were not installed normally containing syslinux.efi and bootx64.efi.
I would like for someone to confirm if the below change to the frzr download/update utility may have caused it. Or if I'm doing something wrong. I was, however, able to get the OS to boot by installing on top of an existing Ubuntu drive because the required EFI partition, directories, and files were already present.
frzr-bootstrap
don't install to EFI/syslinux anymore

Maybe this commit is the one to blame?

frzr-bootstrap now installs to EFI/BOOT instead of EFI/syslinux
Perhaps you still have the old UEFI boot entry which is trying to look in the old location?

Maybe this commit is the one to blame?

Yes, that's the one I identified.

frzr-bootstrap now installs to EFI/BOOT instead of EFI/syslinux
Perhaps you still have the old UEFI boot entry which is trying to look in the old location?

Possibly, but the UEFI boot list on the PC showed two frzr entries and neither worked. I deleted those and tried to add a new one then determined the EFI/BOOT directories were not present after installation.

There should be no frzr boot entries. To boot Chimeraos you should select the disk you installed it to in the bios boot options.

Possibly, but the UEFI boot list on the PC showed two frzr entries and neither worked. I deleted those and tried to add a new one then discovered the EFI/BOOT directories were not present after installation.

Double check you are looking at the right partition and that you didn't perform a legacy BIOS install.

I was not able to reproduce. Most likely the issue is that you selected the old boot entries. Another possibility is it could be hardware/bios specific.

Good to know. I didn't see any other place where this was mentioned so I will close this issue. Thanks for the insight.


Selller Products Page

Create the seller products page
this is the link for the design to implement
https://demo.activeitzone.com/ecommerce/seller/products

Will like to work on this

Great....I have assigned it to you AbdulGaniy

@alialaba any update on this?

@alialaba Thanks for the completion


BUG:lack of margin in literary realms page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
text is not visible due to lack of margin in literary realms page

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

i want to work on this issue please assign me ?
@huamanraj

it looks great to me

please explain what's the issue clearly

Hi @RitiChandak,
Could you please check again, not on the home page but in the literary realms section? Thank you so much!
Best regards,
Nikhil NIshad
video-
https://github.com/anuragverma108/SwapReads/assets/155576040/7a0dc42c-9699-48c2-b903-49cb87151643

Hi @nikhilnishadoo7 I'll contact you regarding this

can i work on this issue @RitiChandak @huamanraj

hi @nikhilnishadoo7 as there are 2 same pages delete one and raise the new PR


Open GitHub Discussions

Could we open GitHub Discussions? People on Slack aren't very responsive and I don't consider Slack channels to be "open" in the context of open source. This way we could have discussions about NetNewsWire without polluting the Issues more than it already is.

I’d rather not have yet another channel for discussion. I’ve been happy with using a mix of Slack and GitHub issues.


feat: upgrade to new version of pinecone with serverless api

Brings changes inline with new pinecone api so we can access serverless indexes

Hey @seanmor5, happy to take it over for you 😄  I'm using it actively in a startup I'm running so I'm more than happy to maintain the library and keep it updated.
Happy to create a test setup for this all though to keep it in good condition, I can give it the standard library package treatment I do with my other libs!

@MikaAK Sounds good to me :) I will transfer it to you now! Thanks for taking it on from me! I'm glad somebody will be able to keep it up to date

Ah so apparently I cannot transfer it to you because you already have a forked version, so instead I've just invited you to the repo and I will make you a repo owner. If you have another org or something you'd like it transferred to in the future I'm happy to do that as well

Oh, strange, I'll get this passing and we can merge it in, then we can do the transfer after I delete my existing fork 😵💫 I'll try to have it all wrapped up by EoD

@seanmor5  I've removed my fork on github 😄


feat: add support for go dep and go modules

Issue #, if available:
#73
Description of changes:
This change adds support for go dep and go modules. It also updates travis to use dep in go versions 1.9 and 1.10, while using go modules in go versions 1.11 and 1.12 (and tip).
I did not remove support for glide yet, and it's still used with go versions 1.7 and 1.8 in travis.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Good call-out, I'll update the readme.


Make material changes interactive

Hello,
This change adds the ability to change object's material interactively during Active Shade rendering mode. Please note changes to existing materials are reflected in active shade rendering. Applying new materials requires restarting of rendering.
Thanks for review!

A general comment:
Since Max 2019 update 3 Max has the possibility to interact/edit and render simultaneously in the same ActiveShade viewport. The SDK has for this a new IActiveShadeFragmentManager class: http://help.autodesk.com/view/3DSMAX/2020/ENU/?guid=__developer_what_s_new_what_s_new_3ds_max_2020_sdk_html#rendering &
http://help.autodesk.com/view/3DSMAX/2020/ENU/?guid=__cpp_ref_class_max_s_d_k_1_1_i_active_shade_fragment_manager_html

Thanks @Mango-3 , I didn't know about this! This is so cool!
Actually starting 2017 they changed interactive renderer so that some things like material changes handled by SDK. But we already have our own logic built so I thought rather than rewrite whole interactive renderer I'll just add material changes.

Hi @dictoon,
Sorry, I was working on this change for sometime without updating the PR. Some of the changes here are irrelevant by now, but I'll fix your comments and then push couple of commits or probably do another PR.


FR: Provide access to Material Responsive UI breakpoints

https://material.io/guidelines/layout/responsive-ui.html outlines layout guidelines for various screen sizes.
Flutter should provide access to this data based on the screen size provided by MediaQueryData. One possible design could be
enum MobileDeviceType {handset, tablet}
enum MobileDeviceSize {small, medium, large}

class MaterialResponsiveUiData {
  static MaterialResponsiveUiData of(BuildContext context) {
     // calls MediaQuery.of(context) and computes various fields, or stores MediaQueryData
     // and computes on demand
  }
  MobileDeviceType deviceType;
  MobileDeviceSize deviceSize;
  int columns;
  double gutter;
}

Our project could us the gutter definition, as well as MobileDeviceType to implement different behavior depending on whether the device is regarded as a handset or a tablet.
I would be contribute PR to implement this.

In case anyone stumbles upon this, there's the adaptive_breakpoints package by the Material Design team that does exactly this.

Similar behavior is added in a package referenced in this PR. The package offered by material mentioned earlier seems to also provide this functionality. I don't believe there are currently any plans to add this functionality directly to the framework due to the opinionated nature so I'm closing this issue for now. If anyone else has further input, feel free to reopen.


Beta cloud: PLT_GW_MISSING_BUNDLE

By deploying this PR: https://github.com/Eomm/pokedex/pull/1 (commit 54290be)
I get:
{
  statusCode: 404,
  code: "PLT_GW_MISSING_BUNDLE",
  error: "Not Found",
  message: "There is no bundle deployed on that machine"
}

Turning on/off the graphiql/openapi stuff did not solve

One additional problem arises because you want to specify a RegExp to support multiple origins. However, JSON does not allow RegExp to be parsed through, we'll need to add a new option. We'll need to support all the combinations of the origin option in `fastify-cors.
For your tests, you can try setting the origin to *, it should work for now.

This has two bugs for us:

escape env variables correctly so it's valid JSON
make sure CORS origin can be specified correctly


cannot read property '0' of undefined

getting the same error as described here:
http://twitter.com/#!/ionutbilica/status/89242719079239680
also as he said there are no visible "problems" and kind of hard to reproduce, the chrome console just spits it out every once in awhile, but it's just polluting our error logs so would be nice to have it go away.

(we were using a slightly old version so just updated and gonna see if that changes anything)

haven't seen the error since I upgraded so closing :P


Correct get_map_boundaries call

The call requires a map ID otherwise it fails: https://developers.neatorobotics.com/api/robot-remote-protocol/maps
>>> boundary = robot.get_map_boundaries().json()
>>> print (boundary)
{'version': 1, 'reqId': '1', 'result': 'not_found',

with Map ID provides:
>>> boundary = robot.get_map_boundaries('2018-06-26T02:45:10Z').json()
>>> print (boundary)
{'version': 1, 'reqId': '1', 'result': 'ok', 'data': {'boundaries': [{'id': '009cdabd-5b38-4804-bdf2-da7de2e3f2d7', 'type': 'polygon', 'vertices': [[0.7169, 0.3447], [0.8103, 0.3447], [0.8103, 0.5725], [0.7169, 0.5725]], 'name': 'Hallway', 'color': '#7BB472', 'enabled': True}, {'id': '8447e383-2ad8-41af-a141-29a8770cfc0d', 'type': 'polygon', 'vertices': [[0.4096, 0.2074], [0.6993, 0.2074], [0.6993, 0.4955], [0.4096, 0.4955]], 'name': 'Living room', 'color': '#5B7BA5', 'enabled': True}, {'id': 'aa6c2a3f-5b9d-430a-8dd4-37a4f2b38e3c', 'type': 'polygon', 'vertices': [[0.3413, 0.5005], [0.6255, 0.5005], [0.6255, 0.7785], [0.3413, 0.7785]], 'name': 'Kitchen', 'color': '#E49770', 'enabled': True}]}}

Looks great. Merged into dev  branch together with #20, #32 and #33 for now. Will do some more testing before merging dev to master.
It would be great if you guys with robots that actually support persistent maps check that everything seems to be working fine before I merge.

Made a small change to the parameter name (changed to map_id) for PEP8 compliance in 749720b.


This project needs a license

Hey @alfremedpal,
First I wanted to thank you for writing this app, it's just great and I wanted to contribute by adding a patch to add an option to select the desired number of players (not minimum or maximum, but how many players are going to play the game). I also wanted to host an instance locally for me to use.
However, one thing I noticed is that you didn't add any license at all.
Would you mind to add a license so that others know how can they contribute and use your software?
Interesting read about picking a license: https://blog.codinghorror.com/pick-a-license-any-license/
Thanks!

Hey @mvicomoya !
Thank you for the comments, and I'm glad you found this little project useful 😄 . You are definitely right, I should add a license, I can't believe I forgot about it! I'll be adding it this week and notifying you (spoiler alert: it will be a very permissive license like MIT or GNU GPLv3). Thanks again.

@mvicomoya forget about what I said about adding the license during this week haha, it's already up. The license is MIT so you are free to do pretty much anything you want. I'll be waiting for your patch 😉 .

Awesome! Thanks a lot @alfremedpal!


Providing no argument to del erases the first file on disk

Booting from Floppy...
bootOS
$dir
fbird
pillman
invaders
basic
textmode
$del
$dir
pillman
invaders
basic
textmode
$

Now it generates an error when the filename isn't found.


Including on every pages throws ErrorException: Undefined variable: apiJsUrl

To reproduce simply put {!!  GoogleReCaptchaV3::init() !!} in the footer of your page layout with the default configuration (background mode on) and the site will crash.
This is happening because https://github.com/RyanDaDeng/laravel-google-recaptcha-v3/blob/master/src/GoogleReCaptchaV3.php#L67 which prepares the variables for the view does not contain apiJsUrl as a view variable which is required for the plugin to initialise.

@martsie Hi, omg, this is critical and this is my bad. I merged this feature from one of contributors without thinking about the initialisation. I should have made some test cases to cover it. I have updated the code, please run update again, you should get version ~2.2.14

No worries, it happens - good to catch it early!


chore(main): release 1.0.1

:robot: I have created a release beep boop
1.0.1 (2024-11-12)
Features

migrated from monorepo (c331004)

Bug Fixes

corrected changelog (9009321)

Miscellaneous Chores

release 1.0.1 (8a08d48)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/nico-i/prettier-config/releases/tag/v1.0.1 :sunflower:


Update domains-admin-takeover.md

There is a wrong cmdlet on the step 6 of "external admin takeover". The step instructs to run the Confirm-MsolDomain cmdlet, but the cmdlet in the example is another one (Confirm-MsolEmailVerifiedDomain). The cmdlet Confirm-MsolEmailVerifiedDomain might be used on the "internal admin takeover" procedure, and for now it's under "external admin takeover".
I also fixed the quoting code of the step 3 that has the "For example" within the code block and space in the beginning of each cmdlet.

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

#sign-off


Remove docs site version, include SHA and link to codebase

At the bottom of the docs site you'll see "Copyright 2020 Chocolatey Software", then something like "0.0.0-preview.0.143". See image below:

We don't version this site, so I would prefer instead to see a Git SHA with a link back to this codebase. This is what we do on the Chocolatey.org site (even though the text is hidden by default). See image below:

I don't think we need to hide the text, but it should be using the SHA if we are going to show anything.

I am going to modify this to the following:


Any plans to update this module to cldr rules v40

There are a few changes that might be worth updating this module to the latest v40.
Eg. french has now beside "one, other" also the "many" category.
Would be awesome if you find the time to get an update out to the npm registry. Thanks a lot for providing this module.

released 2.0.0

awesome...thanks a lot 👍

No problem, thank you for reminder about notable changes in CLDR.
Could you tell, how do you use this lib? It's very "low level". Usually people prefer more high abstraction, something like babelfish.

We rely on the plural.forms(lng) function in our application which does basically add text fields needed for pluralforms in the target language based on the source language.

Thanks!


Value Generator tool does not appear in tool list

Microsoft PowerToys version
0.72.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
Unless I missed something, no steps are needed.
Just open the power toys tool list. The Values Generator tool is not present:

✔ Expected Behavior
The new tool should appear on the tool list
❌ Actual Behavior
The new tool does not appear on the list. It also does not appear on the shortcuts list:

Other Software
No response

Hi @Nippius
Value Generator is plugin for PowerToys Run. You can find it under the PowerToys Run settings.

/needinfo

That's weird. Your screenshot doesn't also have plugin icons.
Can you attach a /bugreport ?

Every plugin in the list is supposed to have an icon placed on the left.
Look at my screenshot.

Nothing weird from the bug report.
Can you confirm that you have this folder on your PC? C:\Program Files\PowerToys\RunPlugins\ValueGenerator\Images
And the PowerToys settings you are looking into is 0.72.0 (first page)?

I came here with the same problem after upgrade. Before upgrading I didn't have PowerToys Run enabled at all.
"Value Generator" did not show up on the list, but I also had duplicate entries in the plugin list, such as "Calculator" and "Folder".
I restarted PowerToys as administrator (on the General tab), and the plugin list now is correct without duplicates and "Value Generator" showing. I exited and launched again in user mode, and it was still fine.

Ok I don't know what I did but it's fixed now. I just left it alone for a few hours.
@davidegiacometti  I do have the C:\Program Files\PowerToys\RunPlugins\ValueGenerator\Images folder and I can see the plugin icons. Also I'm running version 0.72.0 (it's in the screenshot on my first post as well)
One thing that I know I did is that I enabled the powertoys.run tool right before taking those screenshots in my last message. Maybe it took a little longer to initialize the new plugins?
I didn't try running as admin but that was probably a good call @ngbrown .
Anyway as it seems to be fixed, I'm going to go ahead and close this.
Thank you both.


add asdf-racket team

Do you plan to maintain this repository?
https://github.com/asdf-community/asdf-racket

Yes, I do.

@nandalopes Fantastic! Can you link the team to the repository in terraform/github/repositories.tf?


SecurityRegistryImpl file persistance refactoring

This is a refactorization suggestion for SecurityRegistryImpl file persistance which looked quite wonky. If the patch has potential, please suggest approach for testing it. Currently the file persistance part of SecurityRegistryImpl isn't covered by the integration tests.
Specifically the previous implementation writes to the file while in a loop reading objects from it (loadFromFile did add which did saveToFile), it also had some funky tripple-nested exception handling.

Agree on the wonkyness, now why keep "concurrent hashmap" if we use synchronize ?
I would prefer two "normal" maps and a ReadWrite lock. WDYT?
This class is not tested because it's used for running leshan-demo, it's not mean to be used in production (at least we are not for AirVantage), but the location (server-core) and the doc is not explicit.
So yes, let's fix it.

I agree if we use synchronize we don't need concurrent hashmap anymore.
I'm ok that load/save method are a bit strange, I rewrote it for the BootstrapStoreImpl, but I don't take time to apply this change to SecurityRegistryImpl.
Adding SecurityInfo[] and filename to load and save method seems not really necessary? what is the purpose ?
You missed to close the FileInputStream in loadFromFile.

Thanks a lot for your feedback, I have pushed an update based on your comments. I reverted the changes to the saveToFile/loadFromFile method signatures too, I prefer input parameters but I agree it wasn't a necessary change.

Currently, the file creation is done on load. I think we should do it on save, like we do here. We can copy/paste a large part of the save/load method.

Agreed, actually this is how it's done in the current patch. :)

you're right ^^ !
I integrated you patch in master (commit 4a00f36ec47ef7d5acdb3a110b836cfea680aa0e).
I just change a log.info to log.debug.
And merge all the catching exception in one clause.
Thx @oflisback

Thanks guys!


Rename uiImage to uiImageAsset

See #30

Thank you for your work on patching up libui.
If it is not too much maintenance work on your part, would it be possible to
mention renaming API changes in the CHANGELOG file as well? The
one at https://github.com/libui-ng/libui-ng/blob/master/CHANGELOG.md.
I believe this may be helpful for "downstream" language binding maintainers
in different programming languages, simply to see what they may have to
update specifically.

If it is not too much maintenance work on your part, would it be possible to mention renaming API changes in the CHANGELOG file as well?

Absolutely. There are no API changes listed in the changelog yet, only because none have been made yet.
The changelog is using the Keep a Changelog standard format, which requires the Changed tag for all breaking API changes. Also any API changes proposed will be clearly marked as such with the label api breaking change, as you can see here on this issue #32.

I'm trying to delay all work that involves breaking API changes, until we've merged in everything outstanding that does not.


Downgrade some log messages.

This lowers the log level of several "info" messages. I find that these can be quite noisy when using log messages, and I don't think need to be such a high log level.

r? @alexcrichton
(rust-highfive has picked a reviewer for you, use r? to override)

:pushpin: Commit f0aae40f00f7cc60263ae6a0f9b195cdc4f6cb24 has been approved by alexcrichton

:hourglass: Testing commit f0aae40f00f7cc60263ae6a0f9b195cdc4f6cb24 with merge 03e24bcf67696ba35d3aa2b0dd01f97bcfdf91a7...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 03e24bcf67696ba35d3aa2b0dd01f97bcfdf91a7 to master...


Fix: GsQuickStageCurrentFileCommitCommand is also a WindowCommand

Fix #1000
1000! What a great milestone.
@stoivo Sorry for the bug. We will need to release another hot fix release.

Should this be a MR to dev?
and checkout 2.17.5 and cherrypick this commit and release 2.17.6?


License for CSS and HTML

At the bottom of the License file it says:
   All code in any directories or sub-directories that end with *.html or
   *.css is licensed under the Creative Commons Attribution International
   4.0 License, which full text can be found here:
https://creativecommons.org/licenses/by/4.0/legalcode.

However the CSS files have a Apache License header, for example material-design-lite/docs/_assets/customizer.css has the Apache license.
Does this mean the CSS file is under the Apache license or the CC Attribution License?

Possibly both? I'm not going to pretend to have any legal knowledge but of of the top of my head they are similar and are summarised as "do what you want but credit us"


Suggestions about the module

I have few remarks regarding how the module is tested and build
I did not see any build or CD information in the module. There is no badge, no build process.
If you look at the final product, the one you get from the PowerShell Gallery, include the same folder as the GitHub repos. This include the source, the .git, the .vscode and the images folders, readme.md  and the test files.
My suggestions:

Create a build process (Azure DevOps ? but perhaps it exists elsewhere)
Include a module creation with one psm1 file and a class folder for type.ps1
Test the newly created module (test the module and the scripts)
Include a badge on the readme page
Automate module publishing

I am working (slowly) on CI/CD for this (and simultaneously other) PowerShell module repos.  I also just created #22 for testing the psm1.

one psm1 file and a class folder for type.ps1
I see value in removing the development files (.git, .vscode, imgs, etc), but are you saying all the script files (except types.ps1) should be merged into a single psm1?  If so, what is that value of that over dot-sourced scripts?

Hi
The values of a unique psm1 file are

fast loading (but it's not the case now)
easier to debug and to manipulate
take a look at
https://github.com/lazywinadmin/AdsiPS/blob/master/build/tasks.build.ps1


V1 return correct cert if multiple

I've only looked very briefly, not done a proper review, but this code doesn't seem to handle multiple SignerInfos correctly and doesn't handle Signed Attributes at all. See:
https://android.googlesource.com/platform/tools/apksig/+/refs/tags/platform-tools-34.0.5/src/main/java/com/android/apksig/internal/apk/v1/V1SchemeVerifier.java#712
https://android.googlesource.com/platform/tools/apksig/+/refs/tags/platform-tools-34.0.5/src/main/java/com/android/apksig/internal/apk/v1/V1SchemeVerifier.java#522

There's should be APKs with Signed Attributes in the apksig test suite FYI. Though IIRC none with multiple certificates for one signer (only multiple signers).

Also please don't ping me in issues/PRs involving eighthave as he's left me no choice but to block him from interacting with me after I was given no choice but to leave F-Droid.

This code returns the last certificate that has a public key that can be used to successfully verify the signature in the first SignerInfo (though it mishandles verification with Signed Attributes).
It should loop over either only the first (android < N) or all SignerInfos (>= N) and verify the signature using the pubkey of the first certificate matching (i.e. having the same issuer and serial number) that SignerInfo (ignoring all other certificates). And then return the certificate that was used to verify the first SignerInfo that successfully verified, if any.

Prior to Android N, Android attempts to verify only the first SignerInfo. From N onwards, Android attempts to verify all SignerInfos and then picks the first verified SignerInfo.

https://android.googlesource.com/platform/tools/apksig/+/refs/tags/platform-tools-34.0.5/src/main/java/com/android/apksig/internal/x509/Certificate.java#65

@obfusk ♥ thank you so much for this detailed and constructive feedback.
I understand and respect your request not to be involved in issues where eighthave is involved. I want to assure you that my decision to reach out to you was solely based on your exceptional experience and the quality of your work(which I stumbled upon while searching for tackling this issue). Considering the complexity of the issue and the challenges presented in both the initial PR and my own, I felt your expertise was essential for developing a robust solution. The initial PR, which had significant shortcomings, was closed due to its quality. I decided to pick up the issue myself as it sparked my interest in learning more about it. Therefore, you could consider this PR as having evolved independently of the initial one from eighthave.
Having said that, I will update it and hopefully you can take another look.

Happy to take another look. Just haven't had time yet :)

FYI: Android/apksigner also checks for unsupported critical extensions and key usage.
And I think recent Android versions differ from apksigner in that they silently skip SignerInfos if the certificate cannot be found or the signed attributes are missing a message digest:
https://android.googlesource.com/platform/libcore/+/refs/heads/android14-release/ojluni/src/main/java/sun/security/pkcs/SignerInfo.java#338

Note that unsupported critical extensions and key usage are not verified!

I think that apart from that it's correct now. But it's been a while so I'd have to do another full review of the final version to be sure I didn't forget or overlook anything.
There's also the matter of attribution and licensing for the canonicalisation code I wrote (since I licensed that under GPLv3+ like most of my code).

There's also the matter of attribution and licensing for the canonicalisation code I wrote (since I licensed that under GPLv3+ like most of my code).

ofc, I did not take that into account. Due to the code being licensed GPLv3+, it is incompatible with androguard's Apache License 2.0. Not sure what is the best angle here.

ofc, I did not take that into account. Due to the code being licensed GPLv3+, it is incompatible with androguard's Apache License 2.0. Not sure what is the best angle here.

Under the circumstances I am willing to make an exception to my policy of only using strong copyleft licenses here and dual-license it so you can use it :)

hey @obfusk feel free to take another look and maybe i can merge it

Might take a little while before I can get to it. But it's on my to-do list :)

Still on my to-do list. But:

I found a bug in my canonical_name() code I haven't had time to look into yet which I'd like to fix first.
It seems more useful to review this after it's been modified to not use oscrypto as you just removed that dependency.

I updated the canonical_name gist. I could have missed more edge cases but at least I fixed the one I found. Will try to review the PR not too long after you update :)

I updated and moved the code to https://github.com/obfusk/x509_canonical_name.py

Merged without waiting for me to review?

Unfortunately @ping2A  merged it before discussing with me or checking the comments. @obfusk please feel free to check it even now and I can push any updates later on.


Bot Key

Hi, I've just implemented your library and i would like to know how do you make a bot key so people can post without a github account

Create a new GitHub account. (You have to use a unique email address.)

Go to https://github.com/settings/tokens and create a new token using Generate new token. (You don't need to give the bot any other rights than posting issues.)

Copy the OAuth access token you get at the end of the setup

Override getGuestToken() like this:
@Override
public String getGuestToken() {
    return "<your token here>";
}

Create a new GitHub account. (You have to use a unique email address.)

Go to https://github.com/settings/tokens and create a new token using Generate new token. (You don't need to give the bot any other rights than posting issues.)

Copy the OAuth access token you get at the end of the setup

Override getGuestToken() like this:
@Override
public String getGuestToken() {
    return "<your token here>";
}

I'll close this issue once I add this description to the README.md.

Hi @HeinrichReimer, thanks for making this library!
While the instructions you added here went a long way toward helping me create a bot to use with the library, I ended up getting stuck for a bit trying to figure out which of the available permissions would allow the bot to create issues.
It might help to clarify in the docs that the permission required for this (on open source repos, at least) is the public_repo permission.
Thanks again!

Clarified that in the README (see 269f6fc57d11e27dd0496be039cab194b6d25425).


Question about Training Set

May I ask whether the endoscopy data in your training data set include CVC-ClinicDB (including 612 images) [1] and Kvasir-SEG (including 1000 images) [2]?
[1] Bernal, Jorge, et al. "WM-DOVA maps for accurate polyp highlighting in colonoscopy: Validation vs. saliency maps from physicians." Computerized medical imaging and graphics 43 (2015): 99-111.
[2] Jha, Debesh, et al. "Kvasir-seg: A segmented polyp dataset." MultiMedia Modeling: 26th International Conference, MMM 2020, Daejeon, South Korea, January 5–8, 2020, Proceedings, Part II 26. Springer International Publishing, 2020.

Our training set contains the above datasets.


Update machine-learning-faq.md

just some grammar fixes.

Hai

Hi @hning86, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Unable to verify the first certificate

Windows Build Number
Microsoft Windows [Version 10.0.19042.1110]
WSL Version

[ ] WSL 2
[X] WSL 1

Kernel Version
Linux version 4.4.0-19041-Microsoft / gcc version 5.4.0
Distro Version
Cent OS
Other Software
visual code 1.59.1
Repro Steps
I have a brand new laptop with VS code version 1.59.1. First of all, I was unable to download python and pylance locally (but I could download CMake, Jupyter, etc locally). So, I went to the market place, downloaded the ms-python*.vsix and did the "install from vsix" thing, and it was installed fine. After that, I tried to install it in WSL for centos 7, and I saw the error "unable to verify the first certificate". Google showed to add "http.proxyStrictSSL": false, in settings.jason. I did that, restarted VS code, and tried again, but I kept seeing the same "unable to verify the first certificate" error.
I don't have this issue on my older laptop though.
Expected Behavior
I could install Python/pylance into WSL, just like other extensions such as C++, cmake, etc.
Actual Behavior
Python and pylance couldn't be installed in WSL, with error showing "unable to verify the first certificate".
Diagnostic Logs
No response

If you try this on another distribution like Ubuntu are you able to do that? I'm trying to determine whether this is a Linux binary specific error or a WSL platform related error, so far it seems like a Linux binary specific error.


Update dev-best-practices.md

See https://forum.mendix.com/link/questions/112765. If we can't convince the UX'ers, let them convince us :-) and let' adopt their standard

@ConnorLand I'd talk to Front-End about this and also verify with Academy if they are teaching anything like this already.

@ConnorLand I'd talk to Front-End about this and also verify with Academy if they are teaching anything like this already.

Will do.

Hi @TimvanSteenbergen . After discussing your proposal internally, we have added naming best practices. I did it in a PR outside this one, which you can view here https://github.com/mendix/docs/pull/4480/files
Thank you for being a part of the Community conversation and improving our documentation on this issue.


Moving a window to a workspace assigned to a monitor, will create a new workspace

Hyprland Version
0.30
Bug or Regression?
Bug
Description
Moving a window to a workspace, will sometimes move it to a new workspace with the name of the workspace you were trying to move the window to.
Check the botton left on the image. You can see two workspaces with the name 2 after moving the window.

The issue has been present since I started using hyprland (at least for 5 months).
How to reproduce
Using the next config

login into hyprland
open a window
move it to another workspace
move it back to the previous workspace

# Monitor aliases
$screen1 = DP-1
$screen2 = HDMI-A-3
$screen3 = HDMI-A-1
$screen4 = DP-3
$screen5 = DP-4
$screen6 = DP-5

# SCREEN            RESOLUTION    POS        ON  COLOR
monitor = $screen1, 3840x2160@60, 0x0,       1,  bitdepth,8
monitor = $screen2, 3840x2160@60, 3840x0,    1,  bitdepth,8
monitor = $screen3, 3840x2160@60, 7680x0,    1,  bitdepth,8
monitor = $screen4, 3840x2160@60, 0x2160,    1,  bitdepth,8
monitor = $screen5, 3840x2160@60, 3840x2160, 1,  bitdepth,8
monitor = $screen6, 3840x2160@60, 7680x2160, 1,  bitdepth,8

# WORKSPACE       MONITOR           IS DEFAULT
workspace=name:1, monitor:$screen4, default:true
workspace=name:2, monitor:$screen5, default:true
workspace=name:3, monitor:$screen6, default:true
workspace=name:4, monitor:$screen1, default:true
workspace=name:5, monitor:$screen2, default:true
workspace=name:6, monitor:$screen3, default:true

You can find the full config here
Crash reports, logs, images, videos
Please request any info you need. I can record a video too.

What are you using to "move" the window?
Also, sidenote:
# SCREEN            RESOLUTION    POS        ON  COLOR
monitor = $screen1, 3840x2160@60, 0x0,       1,  bitdepth,8

it's not ON but scale...

looking at your config:
bind =    $mod SHIFT,    1,        movetoworkspace, 1
bind =    $mod SHIFT,    2,        movetoworkspace, 2
bind =    $mod SHIFT,    3,        movetoworkspace, 3
bind =    $mod SHIFT,    4,        movetoworkspace, 4
bind =    $mod SHIFT,    5,        movetoworkspace, 5
bind =    $mod SHIFT,    6,        movetoworkspace, 6

workspace name:6 != 6. 6 is numbered, name:6 is named.
I recommend dropping the name: from your workspace keywords...

Thank you so much for that, it makes sense. Probably the issue was movetoworkspace 1 was moving to workspace=1 (a integer?) while my workspace was defined as workspace=name:1 (a string?), resulting in two different workspaces.
I've removed name:. I'm gonna test a couple days and report back.

Ok I've been testing for some days.
This config works fine
# See: https://wiki.hyprland.org/Configuring/Workspace-Rules/
# WORKSPACE       MONITOR           IS DEFAULT
workspace=1,      monitor:$screen4, default:true
workspace=2,      monitor:$screen5, default:true
workspace=3,      monitor:$screen6, default:true
workspace=4,      monitor:$screen1, default:true
workspace=5,      monitor:$screen2, default:true
workspace=6,      monitor:$screen3, default:true

# - Switch to workspace
#   Note:  workspaces are asociated ot monitors on ../hardware.d/displays
# BIND     MOD            KEY       DISPATCHER      VALUE
bind =     $mod,          1,        workspace,      1
bind =     $mod,          2,        workspace,      2
bind =     $mod,          3,        workspace,      3
bind =     $mod,          4,        workspace,      4
bind =     $mod,          5,        workspace,      5
bind =     $mod,          6,        workspace,      6

# - Move focussed window to workspace
# BIND    MOD            KEY       DISPATCHER      VALUE
bind =    $mod SHIFT,    1,        movetoworkspace, 1
bind =    $mod SHIFT,    2,        movetoworkspace, 2
bind =    $mod SHIFT,    3,        movetoworkspace, 3
bind =    $mod SHIFT,    4,        movetoworkspace, 4
bind =    $mod SHIFT,    5,        movetoworkspace, 5
bind =    $mod SHIFT,    6,        movetoworkspace, 6

This config causes the issue
# See: https://wiki.hyprland.org/Configuring/Workspace-Rules/
# WORKSPACE       MONITOR           IS DEFAULT
workspace=name:1,      monitor:$screen4, default:true
workspace=name:2,      monitor:$screen5, default:true
workspace=name:3,      monitor:$screen6, default:true
workspace=name:4,      monitor:$screen1, default:true
workspace=name:5,      monitor:$screen2, default:true
workspace=name:6,      monitor:$screen3, default:true

# - Switch to workspace
#   Note:  workspaces are asociated ot monitors on ../hardware.d/displays
# BIND     MOD            KEY       DISPATCHER      VALUE
bind =     $mod,          1,        workspace,      name:1
bind =     $mod,          2,        workspace,      name:2
bind =     $mod,          3,        workspace,      name:3
bind =     $mod,          4,        workspace,      name:4
bind =     $mod,          5,        workspace,      name:5
bind =     $mod,          6,        workspace,      name:6

# - Move focussed window to workspace
# BIND    MOD            KEY       DISPATCHER      VALUE
bind =    $mod SHIFT,    1,        movetoworkspace, name:1
bind =    $mod SHIFT,    2,        movetoworkspace, name:2
bind =    $mod SHIFT,    3,        movetoworkspace, name:3
bind =    $mod SHIFT,    4,        movetoworkspace, name:4
bind =    $mod SHIFT,    5,        movetoworkspace, name:5
bind =    $mod SHIFT,    6,        movetoworkspace, name:6

This config also causes the issue (breaking it on purpose)
# See: https://wiki.hyprland.org/Configuring/Workspace-Rules/
# WORKSPACE       MONITOR           IS DEFAULT
workspace=name:1,      monitor:$screen4, default:true
workspace=name:2,      monitor:$screen5, default:true
workspace=name:3,      monitor:$screen6, default:true
workspace=name:4,      monitor:$screen1, default:true
workspace=name:5,      monitor:$screen2, default:true
workspace=name:6,      monitor:$screen3, default:true

# - Switch to workspace
#   Note:  workspaces are asociated ot monitors on ../hardware.d/displays
# BIND     MOD            KEY       DISPATCHER      VALUE
bind =     $mod,          1,        workspace,      1
bind =     $mod,          2,        workspace,      2
bind =     $mod,          3,        workspace,      3
bind =     $mod,          4,        workspace,      4
bind =     $mod,          5,        workspace,      5
bind =     $mod,          6,        workspace,      6

# - Move focussed window to workspace
# BIND    MOD            KEY       DISPATCHER      VALUE
bind =    $mod SHIFT,    1,        movetoworkspace, 1
bind =    $mod SHIFT,    2,        movetoworkspace, 2
bind =    $mod SHIFT,    3,        movetoworkspace, 3
bind =    $mod SHIFT,    4,        movetoworkspace, 4
bind =    $mod SHIFT,    5,        movetoworkspace, 5
bind =    $mod SHIFT,    6,        movetoworkspace, 6

In resume, name: seems to cause it. In case someone wanna try to repro.

should this be closed?

I think we can yes. At least there is a working config.


New Contributor

Hi! I am new to open source. I would like to contribute to this project. Please do tell if there is anything I can help you with.

Will do, currently I don't have much of a plan on what will be added in the future but if you have suggestions or questions in the meantime lmk


Penny: Clicking "Your profile" after login redirect to a broken page

step to reproduce

start a recipe instance with:
npx community-solid-server -c config-penny.json -f ~/Documents/ -p 3005

go to localhost:3005/idp/register/ and create a new account, choose the option " Create a new WebID for my Pod." and fill up the form, choose bar as a pod name

after registration, the "You've been signed up" page shows a link to the Pod URL localhost:3005/bar/ , click the URL

Now you should be back on Penny UI, click the Connect button and enter your freshly created credential ( you might encounter #3 but don't mind)

clicking the "Your profile" button on the top right corner return an error:

expected
See some profile page
actual behaviour
redirect to http://localhost:3005/bar/profile/card/#http%3A%2F%2Flocalhost%3A3005%2Fbar%2Fprofile%2Fcard%23me
with  "This Resource does not exist."

potential solution
Maybe the "Your profile" link should to http://localhost:3005/bar/profile/ instead ?
Is it penny that is built around solidcommunity.net's pod, which fills up profile/card/ with data contrary to SCS that leave profile/card/ empty that cause the bug?

I think this is a Penny bug, could you report it to @Vinnl via https://gitlab.com/vincenttunru/penny?

Unfortunately this seems to be caused by the framework Penny is using automatically appending a trailing slash, with the only other option being automatic removal of trailing slashes (breaking the viewing of Containers): https://github.com/vercel/next.js/discussions/23988
Tracking this for Penny here: https://gitlab.com/vincenttunru/penny/-/issues/17


Plastic panel thickness

Not sure if this is the right place, but I am trying to figure out how thick the plastic should be for the panels for the enclosure. My current guess is 5mm but it would be great if you could confirm this for me. Many thanks.

Sorry about the delay, the thickness is 3 mm if you are using the V1 design files. But we are currently working in a second iteration of foldable cardboard.


[Workers for Platforms] Add C3

Work in progress.

Created a new PR: https://github.com/cloudflare/cloudflare-docs/pull/13578


Add Custom Error for cancel task reject

I need to have a different code path for cancellation than for normal errors. This pr adds support for properly scoping using a custom error instance:
    .catch(error => {
      const hasBeenCanceled = error instanceof ReactNativeBlobUtil.CanceledFetchError

Since the cancel function rejects the promise, I believe it is better to use a custom error like this to allow people to filter it easily, without resorting to error string comparison.

Thank you sounds legit. But please don't include a version bump in the pull request. This might cause issues, and it might be required to do change the version in another way based on other commits in the update.


[TEST] - Test unit - Fix failing UI unit test main branch

Description
Related #5212

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan ui

@utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5220 (SL-JID-390)

@blueorangutan ui

@utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5220 (SL-JID-422)

@utchoang can you please fix the conflicts ?

@blueorangutan ui

@utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5220 (SL-JID-435)

@utchoang please merge latest 'main' branch and address any new failures/conflicts, thanks.

Alternative PR https://github.com/apache/cloudstack/pull/5262


Expose library commands

Currently, the package is most useful as a CLI, but we cannot import most of its functionalities into python code.
I have created this PR to expose the most useful commands:

code_complexity to compute the complexity of any snippet (string) of code
file_complexity to compute the complexity of a file

@andrewdea would be great if we have this functions in the documentation, if you want you can add it, or I can do it if you want to.

@rohaquinlop great point! I'll go ahead and add docs today

@rohaquinlop I think we can close this thanks to the merge.
Once we have the docs done, we'll be ready for a new release? Just checking if there's any additional steps


Enables advanced filtering

Let user filter dataset by:

languages
data collection protocol
task

I think we can consider this one as done for now 🥇


Get input data from handlebars /profile page to appear on handlebars /pets

Get input data from handlebars /profile page to appear on handlebars /pets

done!


set-eh: fix compile error

The type of function accepted by util.ModifyManifest changed with the
testing PR that got merged, and I forgot to git pull --rebase in the
set-event-handler PR before merging it.

lgtm, if only we had a CI test....


Cleanup build.gradle

The build scripts (build.gradle) in the repository have been chaotic. It was partially because of the bintray Gradle plugin, but we now dropped JCenter/Bintray (#1401). It's time to cleanup build.gradle in the repository!
To be easier to review/trace, and not to make mistakes, this pull request consists of several Git commits for step-by-step changes.

Thanks!

Resolved merge conflict -- master has been 0.10.29-SNAPSHOT while this branch has been expecting 0.10.28-SNAPSHOT.


Expose shadow terminator correction factor

Implements a control for changing the shadow terminator correction factor within the appleseed object modifier.
Depends on https://github.com/appleseedhq/appleseed/pull/2720

Thanks!


How to use Perplexity

Can you provide us with an example of how Perplexity can be used? Thank you for your help.

from pytgpt.perplexity import PERPLEXITY

bot = PERPLEXITY()

ai_respone = bot.chat("hello")

print(
    ai_respone
)


sqlcipher: fix cross build

Motivation for this change
pkgsStatic.sqlcipher wasn't building because tcl was incorrectly in buildInputs and the Makefile didn't detect the build CC correctly.
This also fixes pkgsCross.aarch64-multiplatform.sqlcipher and so on.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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

[x] Fits CONTRIBUTING.md.

@ofborg build pkgsStatic.sqlcipher

@ofborg build pkgsCross.aarch64-multiplatform.sqlcipher

@ofborg build pkgsCross.aarch64-multiplatform.sqlcipher pkgsStatic.sqlCipher

@ofborg build pkgsStatic.sqlcipher

Result of nixpkgs-review pr 132573 at 3e3e445d run on x86_64-linux 1

1 package failed to build:

python38Packages.patator

7 packages built successfully:

kmymoney
python38Packages.pysqlcipher3
qtox
retroshare
signal-desktop
skrooge
sqlcipher

1 suggestion:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/libraries/sqlcipher/default.nix:32:3:
   |
32 |   meta = with lib; {
   |   ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

$ nix build .#pkgsCross.aarch64-multiplatform.sqlcipher
$ nix shell .#qemu
$ qemu-aarch64 ./result/bin/sqlcipher
SQLite version 3.34.1 2021-01-20 14:10:07 (SQLCipher 4.4.3 community)
Enter ".help" for usage hints.
Connected to a transient in-memory database.
Use ".open FILENAME" to reopen on a persistent database.
sqlite>


provide a way to authorize oneself (enter a voucher) without javascript

here with @hiurequeiroz we just came across an android device 4.4.4 that could not display the bundle developed by @luandro
browsing to http://thisnode.info/pirania/ just showed a blank page (it did not even say You need to enable JavaScript to run this app.). Disabling javascript in the android browser did correctly show the You need to enable JavaScript to run this app., indicating that the blank page is due to a failed javascript (quite possibly an outdated browser. But we'll definitely find this in the wild)
We need to provide a way to authorize oneself (enter a voucher) without javascript
details:
model: G620S-L01
version: Android 4.4.4

In this case another device with Javascript should enable for the device.