How to completely uninstall

Doing whats listed in the readme doesn't actually do it

You’ll have to elaborate; the readme instructions definitely work. What did you try, and what leads you to claim it “doesn’t actually do it”?

According to you're readme I should do rm -rf "$NVM_DIR" and then remove the export line my ~/.bashrc and that should remove it, but I run nvm --version to check if it's gone and I still shows' the version ❯ nvm --version 0.37.2

According to you're readme I should do rm -rf "$NVM_DIR" and then remove the export line my ~/.bashrc and that should remove it, but I run nvm --version to check if it's gone and I still shows' the version ❯ nvm --version 0.37.2

what show
echo $NVM_DIR?

the .nvm folder in the user directory that doesn't exist

@bckelley nvm is a sourced shell function, so you have to restart your terminal after that, or, run nvm deactivate, for nvm itself to be gone.

I did source my .bashrc

@bckelley yes, and all new shells will lack nvm after running the uninstall instructions. To remove it from the current shell, you either have to close that shell, or run nvm deactivate.

something on linux that have to take in mind is that everything stay on memory, including the shells, if you do a change, to get the new one, you can on the same terminal, write bash, and this will load the new changes, or close the terminal and open a new one.


MINOR: Remove redundant SuppressIntegrationTests

The removed tests have counterparts covered by SuppressScenarioTest using the TopologyTestDriver.
This will speed up the build and improve stability in the CPU-constrained Jenkins environment.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/17609/
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/17610/

hah. Looks like I only needed one run to fail my own target: https://builds.apache.org/job/kafka-pr-jdk11-scala2.12/552/testReport/junit/org.apache.kafka.streams.integration/SuppressionIntegrationTest/shouldShutdownWhenRecordConstraintIsViolated/

@bbejeck I share your concern, and I guess it's an important thing to hash out up front...
This PR is essentially switching test coverage from Integration tests to TopologyTestDriver tests. Doing so trades covering some failure modes for increased test speed and (presumably) decreased flakiness.
Do you think this tradeoff is worth it? In particular, are we giving up meaningful test coverage and losing the value of testing if we make changes like this?

Java8:
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/17615/
Java11:
https://builds.apache.org/job/kafka-pr-jdk11-scala2.12/555/

The single remaining SuppressionIntegrationTest still timed out after 30s, so I'm also experimenting with bumping the timeout to 60s.

no java 8 fails.
java 11 fails:
kafka.api.PlaintextConsumerTest.testLowMaxFetchSizeForRequestAndPartition
kafka.api.SaslPlainPlaintextConsumerTest.testCoordinatorFailover
kafka.api.SaslSslAdminClientIntegrationTest.testAclDescribe
kafka.api.SaslSslAdminClientIntegrationTest.testLegacyAclOpsNeverAffectOrReturnPrefixed
kafka.server.epoch.EpochDrivenReplicationProtocolAcceptanceTest.shouldSurviveFastLeaderChange
kafka.admin.ResetConsumerGroupOffsetTest.testResetOffsetsNotExistingGroup
kafka.api.CustomQuotaCallbackTest.testCustomQuotaCallback
kafka.tools.MirrorMakerIntegrationTest.testCommaSeparatedRegex
org.apache.kafka.streams.integration.ResetIntegrationTest.shouldNotAllowToResetWhenInputTopicAbsent
org.apache.kafka.streams.integration.ResetIntegrationWithSslTest.testReprocessingFromScratchAfterResetWithIntermediateUserTopic
retest this, please.

no java 8 fails.
java 11 fails:
https://builds.apache.org/job/kafka-pr-jdk11-scala2.12/567/
kafka.admin.ResetConsumerGroupOffsetTest.testResetOffsetsNotExistingGroup
kafka.api.CustomQuotaCallbackTest.testCustomQuotaCallback
kafka.api.PlaintextConsumerTest.testLowMaxFetchSizeForRequestAndPartition
kafka.api.PlaintextConsumerTest.testCoordinatorFailover
kafka.api.SaslSslAdminClientIntegrationTest.testAclDescribe
kafka.api.SaslSslAdminClientIntegrationTest.testLegacyAclOpsNeverAffectOrReturnPrefixed
kafka.server.ListOffsetsRequestTest.testResponseIncludesLeaderEpoch
org.apache.kafka.streams.integration.RepartitionWithMergeOptimizingIntegrationTest.shouldSendCorrectRecords_OPTIMIZED
org.apache.kafka.streams.integration.ResetIntegrationTest.shouldNotAllowToResetWhenInputTopicAbsent
org.apache.kafka.streams.integration.ResetIntegrationWithSslTest.testReprocessingFromScratchAfterResetWithIntermediateUserTopic
kafka.api.SaslPlainPlaintextConsumerTest.testCoordinatorFailover
kafka.tools.MirrorMakerIntegrationTest.testCommaSeparatedRegex

retest this, please

I personally don't believe that the risk for this particular test is large. It's testing with a single partition and everything is based on event-time.
The only difference I see is, that using TopologyTestDriver we flush the KTable cache after each records. But this should not impact the test.
Btw: I am wondering, why we don't remove the whole test? The two remaining error scenarios can be tested as unit-test, too, IMHO.

I wanted to actually test that Streams shuts down, since that's what the feature is advertised to do. My reasoning was that I could have a unit test that verifies it throws an exception, but the only way to make sure that Streams shuts down is actually to run the whole thing.
Does this seem like good thinking, or am I overengineering?

Java 11 fails:
https://builds.apache.org/job/kafka-pr-jdk11-scala2.12/575/
kafka.api.PlaintextConsumerTest.testLowMaxFetchSizeForRequestAndPartition
kafka.api.PlaintextConsumerTest.testCoordinatorFailover
kafka.api.SaslPlainPlaintextConsumerTest.testCoordinatorFailover
kafka.api.SaslSslAdminClientIntegrationTest.testAclAuthorizationDenied
kafka.api.SaslSslAdminClientIntegrationTest.testAclOperations
kafka.api.SaslSslAdminClientIntegrationTest.testInvalidAlterConfigs
kafka.api.SaslSslAdminClientIntegrationTest.testAlterLogDirsAfterDeleteRecords
kafka.api.SaslSslAdminClientIntegrationTest.testConsumeAfterDeleteRecords
kafka.api.SaslSslAdminClientIntegrationTest.testReplicaCanFetchFromLogStartOffsetAfterDeleteRecords
kafka.api.SslConsumerTest.testCoordinatorFailover
kafka.admin.ResetConsumerGroupOffsetTest.testResetOffsetsNotExistingGroup
kafka.api.CustomQuotaCallbackTest.testCustomQuotaCallback
kafka.api.SaslSslAdminClientIntegrationTest.testAclDescribe
kafka.api.SaslSslAdminClientIntegrationTest.testLegacyAclOpsNeverAffectOrReturnPrefixed
kafka.api.SaslSslConsumerTest.testCoordinatorFailover
kafka.api.TransactionsTest.testFencingOnTransactionExpiration
kafka.tools.MirrorMakerIntegrationTest.testCommaSeparatedRegex
org.apache.kafka.streams.integration.RepartitionWithMergeOptimizingIntegrationTest.shouldSendCorrectRecords_OPTIMIZED
org.apache.kafka.streams.integration.ResetIntegrationTest.shouldNotAllowToResetWhenInputTopicAbsent
org.apache.kafka.streams.integration.ResetIntegrationWithSslTest.testReprocessingFromScratchAfterResetWithIntermediateUserTopic
retest this, please.

Does this seem like good thinking, or am I overengineering?

Depends if we have a test KafkaStreamsTest#shouldShutDownOnUncaughtException or similar -- If yes, and if you believe in decomposition, it should be sufficient to test that an exception is thrown.
Note: if we don't have this case, it might be worth to add this test to get rid of the other two. I personally believe in decomposition, and both tests, basically tests from this class have a huge overlap that the new test could cover at once.

Since opening this ticket, the Java11 version has been upgraded (https://issues.apache.org/jira/browse/INFRA-17235), and an issue with our build configuration has also been fixed, so this PR may not be needed to improve stability.
I'll put the question to the reviewers whether we should still proceed.
Retest this, please.

That is an unrelated failure:
kafka.admin.ReassignPartitionsClusterTest.shouldExecuteThrottledReassignment
and it's the same for java 8 and 11.
retest this, please.


Make stock bluefin less container focused

Note that I mean from a user experience perspective, not a technical level. Now that we have homebrew on the image we can do cooler stuff and gives us some new options that I think we should explore. bluefin-dx remains unchanged of course, this will be for our bluefin image that we give to our friends.
First things first I'd like to:

Remove Containers from the logo menu.
Remove boxbuddy
Strongly de emphasize distrobox and containers in general.

Let's keep all the container stuff in -dx, there's definitely an audience that loves this brew-first setup, and after trying every combination this one is the one that feels like the sweet spot. I've also totally moved to this model, I keep the devcontainers as part of my IDE and then I have everything else. Feels like we might be overrotating on the need to learn containers, it's supposed to be invisible anyway lol.
When I need a distrobox we ship 2 keyboard shortcuts to ubuntu and fedora containers, and those get cleaned up every update. And all of those things will still be there, we just won't show them as part of the default experience.

Are there any GUI to manage homebrew packages?
Also, one-click install .deb packages with BoxBuddy felt very user and noob friendly

Is this still of interest? Would be a lot easier to do now after the default container purge

Everything mentioned here seems to be addressed by now? And boxbuddy also seems like a pretty nice entrypoint for the "how to container" things

I'll close this for now, let's reinvestigate at a later date.

@castrojo wouldn't closing as not planned be better for now?


add minimal word length setting

Adds a new setting to the setup screen to select the minimal word length.
Shorter words entered by the user will not be checked. The grid above the board displays a white 'x' instead of the number and the results page will not display shorter words.
Current possible values as '2+', '3+' and '4+'.

I like the idea. But I have some issues with this:

It breaks the unit tests. They do not compile anymore. They would not to be changed to cover the new behavior anyway.
With longer minimal length of allowed words, chances increase that there are no solutions at all. Does this need to be addressed?
I think you found the best possible icon for the slider, but it is definitely less intuitive than the other two. But this is not a deal breaker for me.

I'll fix the tests. Forgot about those. Possibly, we need to regenerate the board of there are no or not enough solutions

The commits in between are already in master. The only relevant changeset is https://github.com/bennorichters/bnoggles/pull/17/commits/4c9e0567435983b62314a8f0b8deb4012fbe57c0


Classifier missing Python3.9

Package Name: azure-mgmt-resourcegraph
Package Version: 8.0.0
Operating System: linux
Python Version: 3.9

Describe the bug
A clear and concise description of what the bug is. Classifier for this library is missing Python3.9 hence getting No module found error.
To Reproduce
Steps to reproduce the behavior:

https://pypi.org/project/azure-mgmt-resourcegraph/
check Programming Language section it does not have Python 3.9

Expected behavior
A clear and concise description of what you expected to happen.
Is it deprecated? If yes what is alternative package?
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Docs:0.37469435,Azure.Core:0.12538026,Graph:0.02912764'

can you please respond? Its very urgent

Thanks for reaching out @sravantiaketi1! @msyyc can you please take a look ASAP?

We will release a new version for azure-mgmt-resourcegraph the doc shall be updated with clarify for all supported python version including py3.9

Hi @sravantiaketi1
We have released new package about azure-mgmt-resourcegraph.
https://pypi.org/project/azure-mgmt-resourcegraph/8.1.0b2/
Please try the new azure-mgmt-resourcegraph to solve your problem.
Hope it can help you.


ReadMe

User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

Finished by filling out the readme with relevant information to the project.


Dapr1.0.1 with Istio 1.5.9 -Error in loading components Issue.

/area runtime
/area operator
/area placement
/area docs
/area test-and-release

What version of Dapr?
1.0.1
Istio version 1.5.9
Expected Behavior
daprd sidecar should load the statestore componenet successfully when istio enabled .
Actual Behavior
daprd sidecar  throwing error to load the component
"failed to load components: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial tcp Ip:80: connect: connection refused"
Steps to Reproduce the Problem
1.enabled istio injection in namespace istio-test.
2. Deployed redis instance and redis component by taking from Dapr/quickstart repo.
3. Deployed pythonapp and nodeapp to test the flow.
4. python app throwing error that not able to persist the state and daprd sidecar pod logs following error is dispalyed.
"failed to load components: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing dial tcp Ip:80: connect: connection refused\
Appliations and redis instnace and component yaml deplyoed in same namesapce istio-test where istio-injection is enabled.

This is not a Dapr issue. Istio is blocking traffic to Redis, you should use Istio Traffic Policies to allow the traffic to the Redis service and port.

Thanks a lot @yaron2 for clarifying this.
I am new to this can you provide any link to enable traffic setting pls.
Thanks,
kamesh

@kamesh1677 depending on your Istio configuration there could be a few things going on here.
Istio blocks all outbound connections from pods until the envoy sidecar is started. Because all containers in the pod start at the same time if any of the pods requires outbound access at start-up then it will be blocked. If the pods fail to perform an internal retry then it will simply continue crashing and restarting the pod.
Potential fixes (Istio 1.8.x and newer, refer to the Istio documentation for how to implement in Istio 1.5.9):
Istio added a setting which you can apply at the Istio mesh level:
defaultConfig:
holdApplicationUntilProxyStarts: true
or by adding this annotation to the pods.
proxy.istio.io/config: '{ "holdApplicationUntilProxyStarts": true }'

approach discuss at  App container unable to connect to network before sidecar is fully running


Exceptions and asserts thrown by QRDecoder

We're trying to find QR codes in the camera image. The attached images cause 3 different detection errors.
We try to load the images as follows ZXingReader.exe -format QRCode qrcode_1280x960_bw_out_of_range.png.  With the -fast switch the detection does not fail.
For the image qrcode_1280x960_bw_out_of_range.png (first image) the exception std::out_of_range is thrown from std::vector (it is visible when loading from the web https://nu-book.github.io/zxing-cpp/demo_reader.html)
The other 2 images are causing asserts.

I could not reproduce the issue with the first image. Did you try with HEAD or the latest release?
The other two images work now.

You're right, the problem with the first image was solved in #267 and #251. Thank you for the quick solution

I found another images that causes the same assert (assert(cur.isBlack());)

I'm sorry, but the problem still persists, I have attached more pictures

The floor part of your samples is almost random -> not very well tested with current sample set. Thanks for bringing it to my attention. If you have more, bring it on! :)

The last modification seems to be stable, I'll let the test run for a few more days. For now, thank you


Move the edit button

My question is pretty simple. How to change the position of the edit button that appears on the top left of the page.

Hi @ahmedatya. The ignition buttons are positioned purely using CSS, you can see the code that positions them here.
The easiest way to position them elsewhere is to change the CSS (or override it) for .ct-ignition, depending on where you position the buttons you may also need to modify the individual .ct-ignition__button--busy, --confirm, --cancel, --edit buttons to.

Thanks a lot


New feature request - Allow custom workflow task executors

Currently workflow task executor threads are launched from the thread that schedules the workflow.
This created a situation, where when a workflow thread was created from a thread handling a user operation, it inherited security context data (we are using Apache Shiro which uses an InheritableThreadLocal) from the user operation thread. Since the workflow task executors are subsequently reused, this can lead to problems due to cached thread local data.
A solution would be to allow passing in a custom ExecutorService, so users can implement their own logic in ThreadPoolExecutor.beforeExecute  and / or ThreadPoolExecutor.afterExecute.

@JimBugwadia please review and let me know that this is OK and I'll make a release.

Thanks for the quick turnaround! The interface looks good.
A few questions on the behavior:

If a custom executor is specified, how does this impact task allocation. For example if an implementation declares it can handle 10 concurrent tasks, are we using task completion to throttle, or the number of tasks scheduled / running?

Are there any restrictions / guidelines on the custom executor's behavior? Is it best to have a fixed pool size to match the concurrency level?

The Executor is expected to execute the task and return. The calling thread waits for it to complete using a FutureTask.
If using a pool, the pool should probably have as many threads as configured Workflow Executors. Otherwise, the Executors will block until a thread is available.

Ok, makes sense. Thanks!


Support deleting properties from objects

Jfrog cli should support removing properties from objects as well as setting/updating them as requested in #43

@labrown,
New jfrog rt set-props and jfrog rt delete-props commands have been recently added to JFrog CLI.
Feel free to share with us your feedback.


fix: Add missing translations

Add translations defined in #1559

Heyy @g123k looks good, I'd say we don't need the dev mode to be translated as it's just visible to the devs anyways but that's not a strong preference

Do you mean I should remove translations for the dev mode? Or should we stay 'as-is'?

Mergeable again 👍

Tests are now OK, you can merge this PR 👍

Codecov Report

Merging #1572 (ddee3f8) into develop (2ea0da3) will decrease coverage by 0.04%.
The diff coverage is 4.54%.

@@            Coverage Diff             @@
##           develop   #1572      +/-   ##
==========================================
- Coverage     8.86%   8.81%   -0.05%
==========================================
  Files          161     162       +1
  Lines         6623    6792     +169
==========================================
+ Hits           587     599      +12
- Misses        6036    6193     +157

Impacted Files
Coverage Δ

...s/smooth_app/lib/data_models/user_preferences.dart
34.42% <ø> (+2.11%)
:arrow_up:

...mooth_app/lib/generic_lib/widgets/smooth_card.dart
0.00% <ø> (ø)

...p/lib/pages/onboarding/consent_analytics_page.dart
1.53% <0.00%> (-0.03%)
:arrow_down:

...oth_app/lib/pages/onboarding/country_selector.dart
0.00% <0.00%> (ø)

...ages/onboarding/knowledge_panel_page_template.dart
0.00% <0.00%> (ø)

...oth_app/lib/pages/onboarding/preferences_page.dart
0.00% <0.00%> (ø)

...th_app/lib/pages/product/category_picker_page.dart
0.00% <0.00%> (ø)

...p/lib/pages/product/common/product_query_page.dart
0.00% <0.00%> (ø)

...h_app/lib/pages/product/edit_ingredients_page.dart
0.00% <0.00%> (ø)

...ges/smooth_app/lib/pages/product/summary_card.dart
0.00% <0.00%> (ø)

... and 12 more

Continue to review full report at Codecov.

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

info • 'canLaunch' is deprecated and shouldn't be used. Use canLaunchUrl instead • packages/smooth_app/lib/helpers/launch_url_helper.dart:20:15 • deprecated_member_use
info • 'launch' is deprecated and shouldn't be used. Use launchUrl instead • packages/smooth_app/lib/helpers/launch_url_helper.dart:21:13 • deprecated_member_use
info • 'canLaunch' is deprecated and shouldn't be used. Use canLaunchUrl instead • packages/smooth_app/lib/pages/user_management/sign_up_page.dart:203:37 • deprecated_member_use
info • 'launch' is deprecated and shouldn't be used. Use launchUrl instead • packages/smooth_app/lib/pages/user_management/sign_up_page.dart:204:35 • deprecated_member_use
info • 'launch' is deprecated and shouldn't be used. Use launchUrl instead • packages/smooth_app/lib/pages/user_preferences_profile.dart:152:19 • deprecated_member_use
info • 'launch' is deprecated and shouldn't be used. Use launchUrl instead • packages/smooth_app/lib/views/bottom_sheet_views/social_handle_view.dart:66:23 • deprecated_member_use

Should be OK now

purrffect @g123k  merging


RealConnection causes Unrecognized SSL message

What kind of issue is this?

[ ] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests
get fixed. Here’s an example: https://gist.github.com/swankjesse/981fcae102f513eb13ed

version:  okhttp 3.11
The following bug test fails with

[ERROR] test(com.mw.test.OkHttpBug)  Time elapsed: 1.218 s  <<< ERROR!
javax.net.ssl.SSLException: Unrecognized SSL message, plaintext connection?
at com.mw.test.OkHttpBug.test(OkHttpBug.java:39)

The error suggests a connection is being made via a non SSL port, but this is not the case.  Access via Chrome 67 works without issue.
The address in the code: "https://www.mytestsite.com/" can be replaced with an external address or a local address running on the laptop.  The output is always the same.
package com.mw.test;

import okhttp3.Call;
import okhttp3.ConnectionSpec;
import okhttp3.OkHttpClient;
import okhttp3.Protocol;
import okhttp3.Request;
import okhttp3.Response;
import okhttp3.mockwebserver.MockResponse;
import okhttp3.mockwebserver.MockWebServer;
import okhttp3.mockwebserver.RecordedRequest;
import org.junit.Rule;
import org.junit.Test;
import static org.junit.Assert.assertEquals;
import java.util.ArrayList;
import java.util.List;
import javax.net.SocketFactory;
import javax.net.ssl.SSLSocketFactory;

public class OkHttpBug{

    @Test
    public void test() throws Exception {

        List<Protocol> protocols = new ArrayList<Protocol>();
        protocols.add(Protocol.HTTP_2);
        protocols.add(Protocol.HTTP_1_1);

        List<ConnectionSpec> connectionSpecs = new ArrayList<ConnectionSpec>();
        connectionSpecs.add(ConnectionSpec.MODERN_TLS);

        SocketFactory socketFactory = SSLSocketFactory.getDefault();
        OkHttpClient client = new OkHttpClient.Builder().socketFactory(socketFactory).protocols(protocols)
                .connectionSpecs(connectionSpecs).build();

        Request request = new Request.Builder().url("https://www.mytestsite.com/").build();

        Response response = client.newCall(request).execute();

        assertEquals("text/html; charset=ISO-8859-1", response.header("Content-Type"));

        assertEquals("<!doctype", response.body().string().substring(0, 9));
    }
}

I have tried multiple JVM vendors and versions; and different networks, wifi hot spots and phone connections.
If the code in the okhttp3.internal.connection.RealConnection is updated,
Line 307 from:
      sslSocket = (SSLSocket) sslSocketFactory.createSocket(
          rawSocket, address.url().host(), address.url().port(), true /* autoClose */);

to:
SSLSocketFactory sslSocketFactory = (SSLSocketFactory) SSLSocketFactory.getDefault();
sslSocket = (SSLSocket) ((SSLSocketFactory) sslSocketFactory)
      .createSocket(address.url().host(), address.url().port());

The issue is resolved and a connection is made.  However, I do not consider this an acceptable resolution as the rawSocket would be orphaned, since it is not closed when the SSLSocket is.
Any suggestions?
The following standalone application demonstrates the issue further.  You might consider this suggests a problem outside okHttp, however, it does suggest why okHttp uses this mechanism:
package com.mw.web.http2;

import java.io.IOException;
import java.net.InetSocketAddress;
import java.net.Proxy;
import java.net.Socket;
import java.net.SocketAddress;
import java.util.ArrayList;
import java.util.List;

import javax.net.SocketFactory;
import javax.net.ssl.SSLSocket;
import javax.net.ssl.SSLSocketFactory;

import okhttp3.ConnectionSpec;
import okhttp3.OkHttpClient;
import okhttp3.Protocol;
import okhttp3.Request;
import okhttp3.Response;
import okhttp3.internal.platform.Platform;

public class SocketTest {

    public static void main(String[] args) {
        String host = args[0];
        int port = Integer.parseInt(args[1]);
        sslSocketFactoryTest(host, port);
        sslFIXSocketFactoryTest(host, port);

    }

    public static void sslSocketFactoryTest(String host, int port) {
        try {
            SocketFactory sslSocketFactory = SSLSocketFactory.getDefault();
            Socket rawSocket = sslSocketFactory.createSocket();
            InetSocketAddress inetSocketAddress = new InetSocketAddress(host, port);
            rawSocket.connect(inetSocketAddress, 5000);

            SSLSocket sslSocket = (SSLSocket) ((SSLSocketFactory) sslSocketFactory).createSocket(rawSocket, host, port,
                    true);

            sslSocket.startHandshake();
            System.out.println("handshake");

        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    public static void sslFIXSocketFactoryTest(String host, int port) {

        try {
            SocketFactory sslSocketFactory = SSLSocketFactory.getDefault();

            SSLSocket sslSocket = (SSLSocket) ((SSLSocketFactory) sslSocketFactory).createSocket(host,
                    port );

            sslSocket.startHandshake();
            System.out.println("FIXED handshake");

        } catch (Exception e) {
            e.printStackTrace();

        }

    }
}

FIXED
Bad request was being caused because an SSLSocketFactory was used to create both the rawsocket and the SSLSocket.
Thanks

Thank you denvercoder9
Raised this PR to make this more obvious - https://github.com/square/okhttp/pull/4398/files


Remove unused dependencies that are non-compliant from Java test apps

Signed-off-by: Shubham Sharma<EMAIL_ADDRESS>Description
Cleans up three dependencies from test apps that are non-compliant with CNCF guidelines.

Issue reference

Please reference the issue this PR will close: #4200
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[ ] Created/updated tests (NA)
[ ] Unit tests passing (NA)
[ ] End-to-end tests passing (NA)
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number] (NA)
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number] (NA)
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number] (NA)

@yaron2 could help trigger /ok-to-test?

/ok-to-test


Rename DocumentWidget -> DocumentWrapper for clarity

cc @jasongrout

Code looks fine, and it still seems to work, and what few tests we have pass. Thanks!


feature: support glob and webpack 4, Close #15

Coverage decreased (-9.3%) to 69.863% when pulling eacd66f247e8e9fcfc8dfb5598867c144a657bda on feat-support-glob-and-webpack-4 into 9e7bb96ca47a75e4d79285650acc643727c0f85b on master.


Some updates for UX improvement

Firstly, kudos 🎉  and thanks' for the app 🙏
Being one of the user of your app, I thought I can contribute to your public work. 🎨
Here is my small effort to considering , myself more as a user than developer.
P.S: If you believe this PR is to be ignored. No hard feelings from my side, still all good 👍

Hey @eaccmk, that's super cool - you're my first contributor 😍
Love already the picture on the watch, the rest of the code changes I'll test on the weekend on my own device and if that looks good I'll certainly merge 👍
Will let you know once I took a look :)

Thx again for your PR, I merged it and just adapted a couple of things on spelling and code cleaniness, hopfe that's fine for you:
https://github.com/SaskiaKeil/fitbit-wikipedia/commit/90484301f581e64dbac7bc7bce767e3da0b3824e
It's also pushed to the store, but it requires a check by FitBit, so it should show up in the next days in the store :)

Awesome, yea typos are features - not a bug :)
Feel free to have a look at few of the apps I have built (https://gallery.fitbit.com/developer/9b6764a5-63f1-4c35-b497-1dc3cd9be12b)
Another 1 coming soon (under fitbit review) that one I took some references from your wiki app
(private/ pre publish link)  - https://gallery.fitbit.com/details/5cc641b2-86d5-4962-97f1-e39a7a83165b


Filter by owner

Heya,
Would it be possible to have an env var so we can limit which owners we could watch?
e.g. REPO_OWNER=modem7,vladkens
That way it'd only show the owners that we want (especially useful for owners who have a lot of public repos).
I guess this request is similar to #1 .

Hi, @modem7.
Can you elaborate on how exactly you want to filter? Eg:

Remove statistics collection for certain repositories
Remove some repositories from the public list (but keep data collection)
Collect / show only certain repositories (include)
Remove only certain repositories (exclude)
Something else?

Right now ghstats is done as a private service, so we need to think about how to set this up properly.
I also have repositories that I don't care about / belong to organizations. I've done for now just collecting data so not to lose it.

@vladkens Heya,
My thinking is 1 + 3 (but maybe with owner and/or repo option. E.g. everything public by modem7).
My thought process is as follows:
I've got co-maintainer on a lot of other repos bar mine, and I'm not overly interested in long term stats for them, nor for others to see those in my own private ghstats.

Released in v0.3.0

Heya,
Thanks for this!
I'm trying to implement, but doesn't seem to be playing ball.
  ghstats:
    image: ghcr.io/vladkens/ghstats:latest
    # image: ghstats:test
    container_name: GHStats
    environment:
      - GITHUB_TOKEN=$GHSTATS_GITHUB_TOKEN
      - GHS_FILTER=modem7/*

Also tried:
  ghstats:
    image: ghcr.io/vladkens/ghstats:latest
    # image: ghstats:test
    container_name: GHStats
    environment:
      - GITHUB_TOKEN=$GHSTATS_GITHUB_TOKEN
      - GHS_FILTER=modem7/*,!borgmatic-collective/*,!petersem/*,!willfarrell/*

But still getting:

@modem7 fixed in v0.3.1.

@modem7 fixed in v0.3.1.

Works great, thank you!


Fix Facebook google and GitHub social login

This is  a bug | feature request.

Prerequisites

[ ] Are you running the latest version?
[ ] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/familytree365)
[ ] Did you check the documentation?
[ ] Did you perform a cursory search?

Description

Steps to Reproduce

Expected behavior

Actual behavior

permission error causes this, the system cant write into cache( check permission for storage folder)


Cannot use my own SSL certificate with Enable-Https command

Since the Preview and GA launch of the Standard_Microsoft CDN tier, we've been able, through the portal, and via the use of the Key Vault, to store and use our own SSL certificates.
Azure CLI hasn't been yet updated to support this feature (nor is it supporting the new Standard_Microsoft CDN Tier profile creation, but I've already submitted this elsewhere), so we cannot fully automate endpoint setup w/ custom https domain using Azure CLI.

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

ID: ed8c61db-b294-77fb-0e92-eae01d07b776
Version Independent ID: 958b6e8d-609b-2a58-dbe1-240d3fd894bb
Content: az cdn custom-domain
Content Source: src/command_modules/azure-cli-cdn/azure/cli/command_modules/cdn/_help.py
Service: cdn
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Haven't heard anything back in over two weeks.
Any idea when this will be implemented ?
Thanks.

Moved this to backlog @devigned

I'm not going to get to this until next week. #8855 will provide access to these operations via the updated management library. From there, the custom-https-enable and disable will need to be created allowing for either a provided certificate or a Key Vault source.

We are currently working on adding this support. It should be available by the end of next week.

Hi,
could you please advise when the feature will be available for production purposes?

Closing as a duplicate of #9894. A fix for this is in progress.


geos 3.10.6

geos 3.10.6
Destination channel: defaults
Links

PKG-6068
dev_url (conda-forge): https://github.com/libgeos/geos/tree/3.10.6
conda-forge: https://github.com/conda-forge/geos-feedstock/blob/main/recipe/meta.yaml
conda-forge at version 3.10.3: https://github.com/conda-forge/geos-feedstock/tree/881e2f45065e17852a2c760d47691aaa6a652101
pypi: not there

Explanation of changes:

bump version
remove not needed patches
update build scripts
folder cleanup
⚠ update the conda_build_config.yaml with the last geos 3.10.6 version, after merging and deploying geos: https://github.com/AnacondaRecipes/aggregate/pull/325

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_d0h80f68j_/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:29: missing_tests: No tests were found.
clone/recipe/meta.yaml:41: http_url: http://trac.osgeo.org/geos/ is not https
clone/recipe/meta.yaml:41: missing_license_family: The recipe is missing the about/license_family key.
===== Final Report: =====
3 Errors and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_b4y27xras8/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:41: missing_license_family: The recipe is missing the about/license_family key.
clone/recipe/meta.yaml:29: missing_tests: No tests were found.
===== Final Report: =====
2 Errors and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_08v4tys7ad/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:29: missing_tests: No tests were found.
===== Final Report: =====
1 Error and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_4bhpiy1w23/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:27: patch_must_be_in_build: patch must be in build when source/patches is set.
clone/recipe/meta.yaml:31: missing_tests: No tests were found.
===== Final Report: =====
2 Errors and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_00ra8v1bei/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:27: patch_must_be_in_build: patch must be in build when source/patches is set.
clone/recipe/meta.yaml:31: missing_tests: No tests were found.
===== Final Report: =====
2 Errors and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_6c9cqfm_xd/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:33: missing_tests: No tests were found.
===== Final Report: =====
1 Error and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_7ejojmkbv7/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:33: missing_tests: No tests were found.
===== Final Report: =====
1 Error and 0 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(125): `conda run conda-lint /tmp/abs_0b3ln6qmh9/clone` failed. (See above for error)
The following problems have been found:
===== ERRORS =====

clone/recipe/meta.yaml:33: missing_tests: No tests were found.
===== Final Report: =====
1 Error and 0 Warnings were found

I updated the conda_build_config.yaml with the last geos 3.10.6 version


Watermark Maker(For Images)

Description

Apart from the previous PR #628  Added user defined position option and textwrap  of multiple lines

Fixes #546
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[x] No

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@Kushal997-das

@kaustubhgupta Almost all scripts don't follow PEP8 requirements. I think deploying a autopep8 bot on the repo would save us a ton of time. Here's the link to it

@kaustubhgupta Almost all scripts don't follow PEP8 requirements. I think deploying a autopep8 bot on the repo would save us a ton of time. Here's the link to it

Yes, this is a good idea. @avinashkranjan can you include this in workflows or should we make an issue for the participants to contribute?

Made the changes as requested. Kindly review .

@antrikshmisri


duplicate token: token address H8bFy62L3ZmUfA3Q4anvneF9nbGjQGce7QTsiXkpCk9y is already used

duplicate token: token address H8bFy62L3ZmUfA3Q4anvneF9nbGjQGce7QTsiXkpCk9y is already used
Originally posted by @keone in https://github.com/solana-labs/token-list/issues/13083#issuecomment-1008350176
how do i get passed this or can someone manually merge my changes to the main?

same problem idk..

let me know if you found solution please

added comment in PR


Syntax highlighting is weird for Yaml escape sequences

In YAML double-quoted strings, escaped sequences are highlighted with a red background on Github, which is used elsewhere to identify invalid markup. This is not a good experience as escape sequences are not invalid.
I checked the Atom grammar (which is used for YAML), and it looks like it should properly identify these escape sequences as constant.character.escape.yaml.
I tried using this YAML file in Atom and it properly detects the token type (and highlights it by putting the text in blue):
test: "some string\_with and escape sequence"

A quick test using the TextMate grammar shows this is likely to be a problem with the Atom grammar itself.

Atom grammar showing problem
TextMate grammar not showing problem

As this appears to be a problem with the grammar itself, I'm going to close this. Please open an issue in https://github.com/atom/language-yaml/ for this issue.

Well, I tried using Atom itself to see how it was parsing things previously (and it was detecting things as constant.character.escape.yaml properly). But I just discovered that Atom updates just don't seem to work on ubuntu, as it always tells me that there is no update of packages available, even though the YAML package is not at 0.30.0 which is the latest release 😞

@stof atom/language-yaml#88 should fix this. We'll update the reference to the grammar in Linguist with the next release after the fix is merged.


Because this example app is so small, I don't think we need this section in the README

https://github.com/MindscapeHQ/raygun4node-aws-lambda/blob/230e9e928158475a420cff6dede94126be8d9de4/example/README.md?plain=1#L76-L78

:+1: happy to remove that


HIVE-28623: Implement RuntimeContext to decouple runtime data from TezTask

What changes were proposed in this pull request?
Encapsulate runtime related info to a separate class for later usage.
Why are the changes needed?
Code refactor.
Does this PR introduce any user-facing change?
No.
Is the change a dependency upgrade?
No.
How was this patch tested?
Easy refactor, precommit tests should pass.

LGTM +1

LGTM +1

thanks for the review, pushed one more commit to address that

thanks @deniskuzZ, addressed comments in https://github.com/apache/hive/pull/5538/commits/cd391822537e7a6dfeed5b7944990823c53ef5b6


Fixed daysOfWeekDisabled not working with non-sequential days

Disabling weekends with daysOfWeekDisabled={[0,6]} would not work due to the for loop getting confused.
Instead, using a indexOf simplifies the loop / break and makes the intention clearer.

:+1:

I just pushed 0.0.19


Default to System.Diagnostics tracing when no console available (and no other providers)

Wouldn't that make sense?

maybe not. i just switched to serilog and use their sys.diag support

As you mention, most (all?) of the supported libraries support System.Diagnostics tracing. They all support console too of course...
Will leave this open, see if anyone else weighs in with yays or nays.

As discussed in #62, LibLog shouldn't be doing any actual logging. Closing as wontfix.


License ?

Hello, under what license is this project released under ? Thank-you.

Hi,
I added MIT license to this project. Thank you.
Best,
Yaqi
From: suprafun<EMAIL_ADDRESS>Sent: Wednesday, March 20, 2019 7:33 PM
To: zhangyaqi1989/Gcode-Reader<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [zhangyaqi1989/Gcode-Reader] License ? (#1)
Hello, under what license is this project released under ? Thank-you.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/zhangyaqi1989/Gcode-Reader/issues/1, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AI5Z54HjHXnjotujJF2pBI_rPZZaR5Vdks5vYtMjgaJpZM4cAhlj.


Origin Class New Paremeter

https://github.com/spatie/ray/commit/5574bd6c31ab1ec84bd2e8fbd80e01189bb4d982#diff-6d81839d864b60747fcfade6ab6910594d607fc9ccf6ec63d2d5dc048e796ee1R11

I don't this PR is needed as it's an optional parameter in Ray now.


kitchen-docker does not properly detect Docker CLI tool on Ubuntu 14.04

The Docker CLI tool is called docker.io in the new Ubuntu 14.04 Docker package.  When I try using the kitchen-docker plugin on Ubuntu 14.04, I get the following error because it is looking for docker instead of docker.io:
kitchen list
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::UserError
>>>>>> Message: You must first install the Docker CLI tool     http://www.docker.io/gettingstarted/
>>>>>> ----------------------
>>>>>> Please see .kitchen/logs/kitchen.log for more details
>>>>>> Also try running `kitchen diagnose --all` for configuration

use_sudo: false worked for me but I don't know why . . . anyone can explain?

@marzzz21 That's because by default on Ubuntu installation, docker is run under sudo until you add your user to docker group.
However, kitchen doesn't know that, hence by default, to use docker, kitchen tries to run docker as sudo.
Now if you don't run kitchen as sudo(like sudo kitchen list), then it cannot run docker as sudo, hence it throws an error.
When you put use_sudo:false, then kitchen tries to run docker without sudo and if you have configured dokcer to run with your user, kitchen will run just fine!

@rkgade I see, that explains it. Thanks for taking the time to elaborate.

For the record, I got same issue under Debian 9 with a user able to run docker commands and a local socket. My user is member of sudo and docker group and I specify use_sudo: false in my kitchen.yml.
After adding NOPASSWD in my sudoers file:
%sudo   ALL=(ALL:ALL) NOPASSWD:ALL

it works.


Pagination

Add the skip_rows and n_rows capability

There is a problem here. Out of bound is not handled. I will look into it.

Made the change so that if skip_rows  > height then it should give back a empty Dataframe. But now that's in place I'm not convinced that this is the best design.
Wouldn't it be more explicit to raise a error so that the user know that he has reached the limit ?

@jGundermann I tend to prefer the error as well. At least it's explicit and people can always catch it to fallback on an empty dataframe

I'm fine with both, although this will be a RuntimeError for now.

We should ad maybe a custom errors in the futures. I just saw that you already add it. #45


PS-3126 Identify sandbox types which uses Python

https://keboola.atlassian.net/browse/PS-3126
Needed for: https://github.com/keboola/sandboxes/pull/294

Sorry, wrong PR, this is not ready for review yet.

Ready now

I have a simpler solution - let's just move these arya constants (type arrays) from Sandboxes API PHP client to sandboxes :) The API client really doesn't need to know abything about it, it's sandboxes component stuff :)
I just already spent more than enough time on this initiaitive and had to leave something as is :smile:


Fixed gizmo bug when Skeleton3DEditor is re-generated

An inspector is re-generated internally when changing rotation mode is changed (#54084), but Skeleton3DEditor destructor doesn't deselect bone. So I fixed it.
Note:
The bone selection information is held by the Skeleton3DEditor, and is synchronized with the Node3DSubGizmo selection while the Skeleton3DEditor is existing.
Once the Skeleton3DEditor is destroyed due to the inspector's re-generating, the synchronization is broken and there is no other way to deselect it. The Node3DSubGizmo still contains selection information, but the inspector does not notify what reason it was generated, so it is dangerous for the Skeleton3DEditor to revert to edit mode based only on the state of the Node3DSubGizmo. Since rotation mode changes are not often done, I think this implementation is not a problem.
And, if we want to implement in the future that bones keep their own individual rotation mode, the behavior of re-generating when redrawing the inspector may need to be changed. Probably don't worry about this line, I've discussed and agreed with reduz to implement the element for Quaternion within Inspector.

Before:
https://user-images.githubusercontent.com/61938263/143719238-63b401ce-9cba-4e4e-b944-4e07a1320918.mov
After:
https://user-images.githubusercontent.com/61938263/143719246-5f92b9d1-452d-4c84-a127-12756698549e.mov

Thanks!


Remove TransformApplicationsForTesting

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

This field is mutated but never queried.
Remove Pipeline#addValueInternal
This method is never called and not suitable for use.

R: @kennknowles

LGTM if this compiles. And/or wait for Jenkins.


Housekeeping: merge release/0.2.x back into master

Thank you Hidde. Do you want to drop a note on the original Kubernetes issue to inform people looking for this functionality that we now have a tentative solution?
I presume most people facing this problem will land on that page.

@mterron good idea, I have updated my original comment with a note about host independent timezone database support.


radxa-RockPi4: Fix USB keyboard not working

Note: On ROCK Pi 4B+, the keyboard must be connected to one of the blue USB ports.
Fixes #147

This change AFAICT should only change the operating system boot order, it shouldn't affect USB functionality in other ways. Unless for some reason looking for a USB boot device works around a problem.
Have you tested the upcoming update (#178) to see if there is some issues that got solved by upstream? There might be, considering other RK3399 had some USB-related issues fixed by it.

This change AFAICT should only change the operating system boot order, it shouldn't affect USB functionality in other ways.

Yeah, it’s weird, but it really fixed the problem with USB keyboard.

Have you tested the upcoming update (https://github.com/Tow-Boot/Tow-Boot/pull/178) to see if there is some issues that got solved by upstream?

Yes, that was the first thing I did.

Note that it would be undesirable to prefer booting operating system form USB within Tow-Boot.

I know, it’s suboptimal but still better than the current state in which Tow-Boot is unusable on this device. I’ll try to find another way though.

Okay, I found a better way, CONFIG_PREBOOT="usb start". I’ve also backported a patch from u-boot (https://github.com/u-boot/u-boot/commit/d5daa02d8d9e7c403a3339db1966e8413e64e408) to fix a reboot loop occuring with some USB devices.

Notes to self: the changes to drivers/usb/host/xhci-ring.c are part of 2022.07, so the patch is not needed.
The radxa-rockpi4 still doesn't enable PREBOOT with usb start, so this will be needed to be addressed.


Kakao Login was recently added, can I use it?

i really want to use it

@kokihoon
Yup, we currently only support web based login, but you can perform it like this.
await supabase.auth.signInWithOAuth(Provider.kakao);

oh thank you bro~


gem install: eco-source is --pre; eco is not

It's a bit tricky to install the gem right now. I have to run gem install --pre eco-source to install eco-source. gem install --pre eco won't work, though, because eco is stable. So I have to run the following in succession:
gem install --pre eco-source
gem install eco

I'm sure this will be fixed when eco 1.1.0 is released. Just leaving this here for the next person.

Thanks, that helped.  Doing a remote search didn't show the pre version.  Glad this was here.

gem install --pre eco-source works but when I do gem install eco
I'm getting this:  gem install eco ERROR:  Could not find a valid gem 'eco-source' (>= 0) in any repository ERROR:  Possible alternatives: to_source, chsource, taosource, epf-source, outsource

There is a problem with eco-source when installing the backbone-on-rails gem

gem install --pre eco-source works but when I do gem install eco
I'm getting this: gem install eco ERROR: Could not find a valid gem 'eco-source' (>= 0) in any repository ERROR: Possible alternatives: to_source, chsource, taosource, epf-source, outsource

Same for me

This is still unresolved.

Download the new [gem] (https://www.dropbox.com/s/akzf19kwnd41jn8/eco-1.0.0.gem?dl=0) and install it locally to resolve the dependency issue. I agree it is band aid and not solution but for something that's open for 4 years, I think this should suffice.

+1


Improve the user journey for a language change

Is your feature request related to a problem? Please describe.
From the municipality survey it was mentioned that users struggle with finding the possibilty to change the language inside the app.
Describe the solution you'd like
Possibly the icon should be re-evaluated?
Describe alternatives you've considered
Add lanugage changes to the context menu?!

#2549

As @hauf-toni already wrote this duplicates a ticket therefore this will be closed.


data: vishnu-mouli-102408

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

delete the last semicolon

Just delete the last semicolon

thanks i did it.

Just delete the last semicolon

thanks i did it.

please tell me if there is anything wrong so i can fix it.


Range using 2 inputs?

Hello,
I'm aware of the range plugin but what's the best way to create a range with 2 date fields?
Specifically, when the "Start" date field is selected, the "End" date input shouldn't allow selections earlier than that date. Same goes the other way around.
The disabled option supports using a function, but unfortunately it only passes the date, and not the input element (making it hard to determine the field in question).
Thanks

Here is a supposed solution from another person:

but it doesn't work for me.
It looks like flatpickr is undefined and I cannot define it (new to JS).
Help would be appreciated (new to JS).


sdwebimage4.0 支持ipv6吗？

为什么我上传app store，被拒了，Guideline 2.1 - Performance，求解决
Guideline 2.1 - Performance
We discovered one or more bugs in your app when reviewed on iPad and iPhone running iOS 10.3.1 on Wi-Fi connected to an IPv6 network.
Specifically, your app still did not respond to tap on any 拼图 in app. Additionally, same kitten puzzles were listed in app.
Please see attached screenshots for details.
Next Steps
To resolve this issue, please run your app on a device while connected to an IPv6 network (all apps must support IPv6) to identify any issues, then revise and resubmit your app for review.
If we misunderstood the intended behavior of your app, please reply to this message in Resolution Center to provide information on how these features were intended to work.
For new apps, uninstall all previous versions of your app from a device, then install and follow the steps to reproduce the issue. For updates, install the new version as an update to the previous version, then follow the steps to reproduce the issue.
Resources
For information about supporting IPv6 Networks, please review Supporting IPv6 DNS64/NAT64 Networks and About Networking.

version 4 supports ipv6. also here https://github.com/rs/SDWebImage/issues/1566

@gpluscat do you think others can understand chinese question? 怎么直接写中文呢。。？

As I commented on #1867

Guys, I'm pretty sure SDWebImage does not cause this issue. Thousands apps use it and don't have issues at submission (including a few I saw with my own eyes). There must be something with your own setup (app or server).

closing this


Checklist Save Feature

As a user I want a way to save the checklist items that I've completed to keep track of my progress.

Details in #28


Invalid gonny reponse structure

the json response fields are camelcase but should be snake case
For reference, the Telraam java class that parses a Ronny detection
   5   │ public class RonnyDetection {
   6   │     public int id;
   7   │     public String mac;
   8   │     public int rssi;
   9   │     public float battery;
  10   │     @JsonProperty("uptime_ms")
  11   │     public long uptimeMs;
  12   │     @JsonProperty("detection_timestamp")
  13   │     public double detectionTimestamp;
  14   │ }

Has been fixed in 16067be


Flow Setting Controller as Sequential Before Unit Pipe Insertion

For example:
'PROBLEM.Foreach(N by 1){ i =>
  Foreach(...){...}
  sram(i) = ...
}

When this gets staged, PROBLEM only has one child and therefore its schedule is set to be sequential, even though the sram write will eventually be put in a unit pipe and then it will contain two stages that should be scheduled in a pipeline

Ah, good call. The analysis should also check if there are any primitive operations directly under that controller that will become other stages.

Assigned to myself but if you get to it before I do, should be an easy fix

I thought it may be expensive to check every statement in every flow so I
made another case locally where I convert sequenced to pipelined if there
are more than one child, it’s an outer pipe and has no user schedule. This
eventually catches the case in unrolling I think. Maybe it’s a dangerous
assumption but I couldn’t think of any example. Should I just redo it to
use your method?
On Mon, Jul 23, 2018 at 1:45 PM David Koeplinger<EMAIL_ADDRESS>wrote:

Assigned to myself but if you get to it before I do, should be an easy fix
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stanford-ppl/spatial/issues/68#issuecomment-407142438,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIKq_bh3gEALE6Aj7892fYDivEZTwSPmks5uJgukgaJpZM4VbNwe
.


Allow changing the Houdini generated output directory

Fixes #391
Configurable output directory
This PR adds a config value to configure the output directory name for Houdini's generated files ($houdini) by default.
TODO

[x] fixup cmd/init to work properly
[x] somehow drag config into is_root_route in houdini-svelte/fsPatch
[x] update e2e to use .houdini
[x] changset?

uh-oh, can you add .houdini/ to the gitignore? 😅

whoops

Running pnpm test on this branch and main results in the same 2 tests failing, and E2E is failing in GHA on build but this also succeeds locally, any insight?

https://github.com/sveltejs/kit/issues/13002 blocks fixing the failing thing properly but I can just switch the e2e to use the barrel exports. Let me know what you want done re: getting this done, if you want to merge the functionality ASAP I see a few options:

Disable the client option and merge the plumbing with $houdini as the default
Merge as-is, making .houdini the new default; add a note to docs about using the barreled exports.
Merge as-is, but leaving $houdini as the default for new projects
Wait until the SvelteKit bug is addressed to proceed (likely gonna hit a huge conflict when it comes time to resurrect unless I go and fix this upstream myself ASAP)

Tests should pass now that I've fixed the path resolving bug, and I've updated docs to the best of my knowledge, please let me know if I've done it wrong or if that file is somehow autogenerated.


Added QuickSort [C#]

Fixes #493
By submitting this pull request I confirm I've read and complied with the below declarations.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[x] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.

Hi! Thanks for showing interest in contributing to this repository.
Please make sure that you have ticked the points mentioned in PR description correctly.
Thanks again!

Any idea why Travis is failing? can the test be restarted somehow?


chore: Integrate Poetry for dependency management and click for cli interface

Refactor package structure for PyPI readiness

Update README with Poetry usage instructions

Add .idea to .gitignore

Replace yaml configuration with toml for enhanced consistency and readability.

Integrate tomlkit for efficient config parsing and manipulation.

Adapt file paths to absolute for reliable root directory execution.

Introduce 'configure' and 'run' commands to the CLI, enabling dynamic configuration updates and command execution.

运行正常

命令展示

修改前后

TODO:

考虑 commit 独立性，未完成移除原 yml 与 CONFIG


Upgrade to Vue 3

As part of https://github.com/numberscope/frontscope/issues/24, we should prioritize upgrading Vue from Vue 2 to Vue 3. To do this, we need to write unit tests for most (all?) Vue components so that we can be reasonably sure they aren't broken by the upgrade.

Completed via PR #123.


Fixed project name in xdebug warning

Copy paste spotted! :-P

following composer who warns it users about the impact of xdebug this will warn users in a similar way.

(https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1552#issue-119223739)
bit to similar ;)
:+1:

:hatched_chick:

Thank you @gharlan.

:+1: thanks @gharlan


How can I do CTRL+W to select current word?

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

cpptools
ms-vscode
0.11.4

Steps to Reproduce:

Open editor and type some words
Put cursor to the start, end or inside word
By pressing for example CTRL+W i would like to select current word only (like in Intellij products)

Can your do it or how I can do it?
THANX

Maybe this extension will work for you.

YES, thank you very much!!!
2017-06-16 15:29 GMT+03:00 Wouter Hardeman<EMAIL_ADDRESS>
Maybe this extension
https://marketplace.visualstudio.com/items?itemName=k--kato.intellij-idea-keybindings
will work for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/28878#issuecomment-309013893,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWOxHdvQVKRMiFRqhyRTyWPOijj1Tby8ks5sEnUwgaJpZM4N8X_d
.


Resolve mismatched clusters name on envoy_stats without overwriting

Description
Envoy stats are being sent and built with the incorrect name inside the snapshot due to cluster name mismatches, this one being formatted as <name>, instead of using the envoy name <cluster>_<name>_<namespace>.
This change seeks to make both cluster names similar temporarily in order to check if the given cluster really exists in envoy_stats
Befere
The old logic creates an aux variable self.cluster_names which the only goal is to store the envoy_names mapped from self.diag.clusters; then the self.cstats variable does a second map over self.cluster_names passing envoy_name to self.estat.cluster_stats and returns a Dict with key-cluster-name formatted as  <cluster>_<name>_<namespace>:
self.cluster_names = [cluster.envoy_name for cluster in self.diag.clusters.values()]
self.cstats = {name: self.estat.cluster_stats(name) for name in self.cluster_names}

In the following pic, the routes are in orange color, meaning that the cluster names didn't match

So in consequence the self.cstats returns undefined cluster stats

After
I removed the self.cluster_names aux variable in favor of directly using cluster.stats_name from self.diag.clusters as parameter for self.estat.cluster_stats because this one has the right format <cluster_name> accepted by the method
self.cstats = {cluster.name: self.estat.cluster_stats(cluster.stats_name) for cluster in self.diag.clusters.values()}

In this case, the cluster names are matching (I've created pod called weby to test traffic shown in green color)

So this output is way different from the original, due to cluster names matching avoiding the condition
https://github.com/emissary-ingress/emissary/blob/77a348c34617acf59b934aa71b74d2f510967b41/python/ambassador/diagnostics/envoy_stats.py#L110

UPDATE:
I've changed cluster.envoy_name in favor of cluster.name due to issues when envoy_name surpasses the 60 characters gets truncated causing undefined stats

Related Issues
List related issues.
Testing
Manual, built a kubeception cluster, then make a couple of requests and trigger a 5xx error to see the health percent change.
Checklist

[ ] I made sure to update CHANGELOG.md.
Remember, the CHANGELOG needs to mention:

Any new features
Any changes to our included version of Envoy
Any non-backward-compatible changes
Any deprecations

[x] This is unlikely to impact how Ambassador performs at scale.
Remember, things that might have an impact at scale include:

Any significant changes in memory use that might require adjusting the memory limits
Any significant changes in CPU use that might require adjusting the CPU limits
Anything that might change how many replicas users should use
Changes that impact data-plane latency/scalability

[x] My change is adequately tested.
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[ ] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently.

[ ] The changes in this PR have been reviewed for security concerns and adherence to security best practices.

change wise this looks fine with me pending CI

@LukeShu As we discussed on Friday with Lance, Diagnostics UI doesn't have any tests yet, I agree to create tests but not yet because that implies a lot of time.  This time the manual test should be enough until we create unit tests. Regarding changelog, I'm not sure in which version this little change would go.

@alexeikmun - This will only go in 3.2 because the diagnostics feature wasn't introduced into the Agent until version 3.2 so clould can only support it as a going forward feature.

@alexeikmun - it looks like one test is failing. Normally, I would say it is one of the flaky ones but looking at this:
https://github.com/emissary-ingress/emissary/actions/runs/3084534857/jobs/5004150996#step:5:3727
It is getting a 500 when trying to call the diagd endpoint so it might be related. Let me know if you need help debugging. You can find the logs at the bottom of the summary page for download:
https://github.com/emissary-ingress/emissary/actions/runs/3084534857

Looks like the last fix did it, if we could just run it and verify that does it then we can get this merged.

I verified it is working as well.
UI:

Here is the cluster with the long service name causing the cluster name to be mangled but retaining the long name for the alt_stat_name
{
        "alt_stat_name": "quote_with_reallly_loooooooooooooooooooooooong_naaaaaaamme_default",
        "connect_timeout": "3.000s",
        "dns_lookup_family": "V4_ONLY",
        "lb_policy": "ROUND_ROBIN",
        "load_assignment": {
          "cluster_name": "cluster_quote_with_reallly_loooooooooooo-1F2DC8766E451E56-0",
          "endpoints": [
            {
              "lb_endpoints": [
                {
                  "endpoint": {
                    "address": {
                      "socket_address": {
                        "address": "quote-with-reallly-loooooooooooooooooooooooong-naaaaaaamme.default",
                        "port_value": 80,
                        "protocol": "TCP"
                      }
                    }
                  }
                }
              ]
            }
          ]
        },
        "name": "cluster_quote_with_reallly_loooooooooooo-1F2DC8766E451E56-0",
        "type": "STRICT_DNS"
      }

One interesting thing to note is that they all show up grey until at least one call via the LB endpoint goes through. So, probably in most cases the ambassador/v0/check_ready and ambassador/v0/check_alive will be grey.
A side note, we should investigate if those are routes are being actively used or just hanging around from the old python entrypoint.


Understanding blockchain as a service

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

`This closes #3612

Hello @crezyj .
@ahmadmardeni1 , @hectorkambow can I review this one now that the other article was closed.

yes please
On Mon, Sep 20, 2021 at 12:36 PM Elly @.***> wrote:

Hello @crezyj https://github.com/crezyj .
@ahmadmardeni1 https://github.com/ahmadmardeni1 , @hectorkambow
https://github.com/hectorkambow can I review this one now that the
other article was closed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/3927#issuecomment-922769500,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVNOJHIC2E66XUDIVG7WC53UC36AVANCNFSM5ELJ5UTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I am glad that you asked
On Mon, Sep 20, 2021 at 12:37 PM john waigwa @.***> wrote:

yes please
On Mon, Sep 20, 2021 at 12:36 PM Elly @.***> wrote:

Hello @crezyj https://github.com/crezyj .
@ahmadmardeni1 https://github.com/ahmadmardeni1 , @hectorkambow
https://github.com/hectorkambow can I review this one now that the
other article was closed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/3927#issuecomment-922769500,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVNOJHIC2E66XUDIVG7WC53UC36AVANCNFSM5ELJ5UTA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I am glad that you asked
On Mon, Sep 20, 2021 at 12:37 PM john waigwa @.> wrote: yes please On Mon, Sep 20, 2021 at 12:36 PM Elly @.> wrote: > Hello @crezyj https://github.com/crezyj . > > @ahmadmardeni1 https://github.com/ahmadmardeni1 , @hectorkambow > https://github.com/hectorkambow can I review this one now that the > other article was closed. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#3927 (comment)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AVNOJHIC2E66XUDIVG7WC53UC36AVANCNFSM5ELJ5UTA > . > Triage notifications on the go with GitHub Mobile for iOS > https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 > or Android > https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. > >
Someone kindly review my work

@ahmadmardeni1 , what is the status of this.
Kindly.

@crezyj
Awesome.
Thanks for making the changes.
I'll start my next round of review, as soon as the plagiarism check is done.
Thanks.

👀
@srishilesh
Lets be sure to gauge the level of quality before jumping in for a full review.
If it needs revisions - please let the author know and we can close the PR to ease the backog.

Hello @ahmadmardeni1 @hectorkambow Please kindly assign someone my work so that I may make changes on any errors present

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published.
Our team of peer reviewers do not provide revisions services at this time in order to review them all in a timely manner.
All revisions and edits should be completed on your own forked repo (as to not take up room in the queue).
Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

🤔  was this PR closed and re-opened at some point?
Or was it in our queue the whole time?


Create a test to validate that bundles updates do not move dependencies outside of safe ranges

This should mimic the behavior we have with the host dependency validation tests.

[x] Changes in v4.x branch - https://github.com/Azure/azure-functions-extension-bundles/pull/310
[ ] Changes in v4.x-preview branch -
[ ] Changes in v3.x branch - https://github.com/Azure/azure-functions-extension-bundles/pull/317

Changes in v4.x branch

[x] https://github.com/Azure/azure-functions-extension-bundles/pull/310
[ ] https://github.com/Azure/azure-functions-extension-bundles/pull/324

Changes in v4.x-preview branch

[x] https://github.com/Azure/azure-functions-extension-bundles/pull/318

Changes in v3.x branch

[x] https://github.com/Azure/azure-functions-extension-bundles/pull/317


allow loading of relative config paths

Currently it is not possible to load the config from a relative directory.
For example using … -m config.json … the searched paths contain the working directory which works fine (but many node_modules folders will get searched unnecessary).
When using … -m ./config.json … (or something else relative) only paths relative to the global install point will be used (/usr/lib/node_modules/homekit2mqtt/config.json)
The exact paths that will be tried can be found out with require.resolve.paths.
Saving the config is already done with the fs API. Was there a reason to use require instead of JSON.parse and the fs API?

Sorry for the long delay... Will merge your PRs asap, please give me a week or two.


ensure bindings are installed

when deploying node functions with azure devops how can we make sure bindings are installed?
I just get Function (my function) Error: The binding type(s) 'blobTrigger' are not registered. Please ensure the type is correct and the binding extension is installed.
when I veiw in the portal

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

ID: b47363e3-c36f-0755-819c-26946013d673
Version Independent ID: 197c63a1-9f72-1049-f27e-8c9c672ec472
Content: JavaScript developer reference for Azure Functions
Content Source: articles/azure-functions/functions-reference-node.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@CastleArg Thank you posting your question. We will review this and provide and update as appropriate.

@CastleArg Can you share the screenshot/steps of you DevOps pipeline?
Also give a try with the steps as shown below and see if that helps?

@jeffhollan do you know what task(s) must be added to a Node.js pipeline to get binding extensions installed?

The above is accurate except you'll also need to make sure you have the .NET Core SDK on the agent as well.  "func extensions install" has a dependency on .NET Core SDK 2 (for the time being).
That said - @dariagrigoriu I believe has been working with the DevOps teams on some custom build tasks for functions that I don't know if may simplify this.

Hi guys,
I eventually got this by installing core tools into my project and then
adding an npm script to run func extension install in the pipeline.
Cheers,
Mark
On Sat, Feb 23, 2019, 1:36 PM Jeff Hollan<EMAIL_ADDRESS>wrote:

The above is accurate except you'll also need to make sure you have the
.NET Core SDK on the agent as well. "func extensions install" has a
dependency on .NET Core SDK 2 (for the time being).
That said - @dariagrigoriu https://github.com/dariagrigoriu I believe
has been working with the DevOps teams on some custom build tasks for
functions that I don't know if may simplify this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/25434#issuecomment-466595441,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AObxMeGKzWXmzXiel0qSEk3LC64-_H1Cks5vQI0kgaJpZM4bG38w
.

@CastleArg Thanks for the confirmation. Glad that it worked. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@KetanChawda-MSFT
I'm still having an issue here after running funcpack to treeshake and package my code.
I'm still getting the blobtrigger are not registered error.  Can someone point me to what files are actually produced and where by func extensions install command so I can make sure they're getting into my final package?

@CastleArg  Can you share the screenshot of your Build and Release pipeline from Azure Devops with arguments ?

[image: image.png]
All functions, proxies etc look fine in the portal and zip file is correct
with compiled javascript.
Only the bindings error is a problem.  What is actually missing to cause
this?
Thanks for the help.  It looks like I should have gone with c# or a
standard webapi..
On Tue, Mar 5, 2019 at 5:36 PM KetanChawda-MSFT<EMAIL_ADDRESS>wrote:

@CastleArg https://github.com/CastleArg Can you share the screenshot of
your Build and Release pipeline from Azure Devops with arguments ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/25434#issuecomment-469534211,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AObxMXdEf4rxDlv2PyiKM8YMRYkiGHKYks5vTfRCgaJpZM4bG38w
.

@CastleArg Not sure if you have posted the screenshot correctly as I cant see them. just checking if you have followed the steps mentioned in the above post ?

Ok so I've finally anwsered my question of:
what files are actually produced and where by func extensions install command?
the answer is - bin and obj folders with dlls - even if you're running a nodeJs azure function.
This was not obvious to me immediately as they were not displaying in vsCode and were gitIgnored.
After copying these as part of my build process the extensions now work, although obviously the package is larger.
Hope this helps someone in the future!
https://github.com/MicrosoftDocs/azure-docs/issues/25727 discusses better docs around build process for JS devs.


Update docs with Git + encryption changes

Do Readme Driven Development to flush out the ideas for:

Using Git for locking
Automatic state management with Git
Using KMS for encryption

Update: this PR is now very out of date, so it can't be merged as-is. However, the general ideas and discussion over whether Git is a good locking mechanism are still worth considering.

Any chance this feature can be merged in yet to master after dealing with the conflicts?

This PR was just put up for discussion. It has not actually been implemented. Comments and feedback welcome :)

I'm confused - why would KMS encryption only be usable with the Git backend? There's no discussion why I wouldn't be able to use KMS encryption with the S3 backend.

There's no discussion why I wouldn't be able to use KMS encryption with the S3 backend.

The S3 backend already has encryption if you set the encrypt config to true.

Since Terraform now natively supports locking, no longer keeps a local cache of the tfstate file, and has more plans related to security on the horizon, going forward with the plan in this PR makes no sense, so I'm going to close it.


[vue] No sizes attribute set when using preload="true"

Reproduction
<UnLazyImage
  :src-set="sourceSet"
  :placeholder-src="placeholderSrc"
  :auto-sizes="true"
  :width="image.width"
  :height="image.height"
  :alt="image.alt"
  :title="image.title"
  :preload="true"
  class="u-object-cover"
/>

Describe the bug
When using :auto-sizes="true" in conjunction with :preload="true" the sizes attribute is not set.
This happens because UnLazyImage.vue checks if props.preload === true and uses the loadImage method to load the image directly. But the loadImage function is not checking, if the sizes attribute should be calculated.
Additional context
No response
Logs
No response

Great catch! Thanks for the thorough bug report. Should be fixed in https://github.com/johannschopplich/unlazy/commit/6903fe814cf10ab3491f60ead09337bd6a6fb08a.

Thanks for the quick fix! 🎉


Fix for Deepgram async websocket

Proposed changes
The Python websockets library seems to accept additional_headers not extra_headers:
https://github.com/python-websockets/websockets/blob/main/src/websockets/asyncio/client.py#L79
Fixes deepgram/deepgram-python-sdk#483
Types of changes
What types of changes does your code introduce to the community Python SDK?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update or tests (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have lint'ed all of my code using repo standards
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

Hi @xzys thanks for the PR!
We'll need to update the version of the websocket dependency in the relevant files (requirements.txt, setup.py, and pyproject.toml) in addition to making the change you've already made.
If you want to make that change, the rest of our team will review this. Otherwise we'll work off your changes.

@jjmaldonis Added the changes to upgrade websockets to 14.1. Didn't add any additional tests, but existing tests are passing (except for some minor transcription errors, aka not picking up commas)

@xzys  We have a concern that this change will be breaking and we'll need to consider how to offer backwards compatibility for users still using older versions of the the Python websockets library.
Ideally both  additional_headers and extra_headers would work depending on what version the user is using.  Is that something you'd be able to help with on this PR?
cc @jjmaldonis


Feature/dockerize mauth proxy

Making changes suggested by @austek

@johngluckmdsol looks like that last change broke the .travis.yml

Could you remove the secure tokens as they are setup on travis now and don't need to be part of the yaml file anymore.
Previous commit broke the travis.yml as you've removed -cache incorrectly

@austek Done, ready to merge.


Fix complex.new

A thought: move TryToFloat to PythonOps?

I looked into the CPython's code and roughly speaking the functionality of TryToFloat resides inside PyNumber_Float, except that the latter also does string parsing, so I suppose is more or less equivalent to NewFloat. PyNumber_Float is used in a few places but it seems that apart from of the float and complex constructors, only to parse a string. So I suppose no need to move TryToFloat to PythonOps.


Custom Svelte component content blocks?

This seems new and cool and promising :-)
I have been looking around at different editors and am curious about this...
The biggest thing I want to be able to do is to make and offer custom content components to my users...
Is something like that possible with lexical or with your version of lexical?
Any plans on something like that in the future?
Thanks! XD

Hi @rchrdnsh ,
We can have custom components , but they need to implement the svelte-lexical plugin interface. In fact, most of the functionality in the editor is implemented using plugins. Image plugin, Keyword, Hashtag, Horizontal Rule, Table and others are examples of such plugins.
By implementing the plugin interface, we ensure that import/export, undo/redo and other features work for the custom components.


How to add BLE device in HA with ZNDMWG03LM?

How can i connect a Rubik's cube (XMMF01JQD) to my HA with ZNDMWG03LM? I want to use my cube as an sensor for my smarthome as shown in https://www.youtube.com/watch?v=4D_vqvUre_0.
Thanks for your help
Florian

It should be added to Mi Home. And there are instructions on how to enable automation. You need to rotate one of the faces quickly 4 times. After this, events from the cube will begin to come to the HA.

Thanks, initial rotation 4 times helped to get my cube working in mi home, events are correctly registered, but no event is pushed to HA, how long does is take usually to receive events in HA?

Instantly. Try reboot gw.

perfect, great job, reboot of HA and hub helped...


Fail to rename local (Emulator) blob container which contains multiple blobs

Storage Explorer Version: 1.8.0_20190507.2
Platform/OS: Windows 10/macOS High Sierra/Ubuntu 18.04
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Make sure Storage Emulator is started.
Open 'Connect' dialog -> Attach to a local emulator.
Expand 'Local & Attached' -> 'Storage Accounts' -> 'local-1(Key)' -> 'Blob Containers'.
Create a new blob container then upload multiple files to it(Or upload a blob folder which contains multiple files)
After uploading successfully, Try to rename the blob container.

Expect Experience:
Succeed to rename the blob container without any error.
Actual Experience:
Fail to rename the blob container with the below error.

More info:
This issue still reproduces when renaming a local table which contains multiple entities.

@v-xuanzh , which emulator did you test against? Azurite or the legacy emulator.

Hi @MRayermannMSFT ,We used Storage Emulator version is <IP_ADDRESS>

Ok, we're pretty sure this is an emulator issue. Closing.


Fix comment

Fixes a comment for MONAI 101 tutorial.
https://github.com/Project-MONAI/tutorials/pull/1303#discussion_r1166713750
Description
A few sentences describing the changes proposed in this pull request.
Checks

[x] Avoid including large-size files in the PR.
[x] Clean up long text outputs from code cells in the notebook.
[x] For security purposes, please check the contents and remove any sensitive info such as user names and private key.
[x] Ensure (1) hyperlinks and markdown anchors are working (2) use relative paths for tutorial repo files (3) put figure and graphs in the ./figure folder
[x] Notebook runs automatically ./runner.sh -t <path to .ipynb file>

Closing it as the merge history is not clean


Does not compile when directly using PlatformIO's built in library manager

There's a mbedl_5 kind of error which relates to an old library that hasn't been updated.
The workaround is to just directly download and place this library in your /lib folder.

It has been fixed, however there is an issue with getting the latest version using PlatformIO repository. If you put
lib_deps = https://github.com/me-no-dev/ESPAsyncWebServer.git
in the platformio.ini I believe it will work.

Thank you!


Menu Funtions now working (Adding, removing, updating & deleting)

Surprise!!!
Now the basic functions to manage the tables are functional, there are aspects that I have not yet been able to complete due to the complexity that it has (but I hope to have it soon), for the moment I leave these features like this.
I hope you like it @HarbingerOfFire

Looks great! Thank you!

Looks great! Thank you!


Show only blocks that contains certain tags inside page and hide others

This plugin is perfect, and I've been frying my brain trying to solve a particular case (that can be easily achieved in Roam with a filter).
With Style Carousel I can toggle certain tags "on" (show) and "off" (hide).
So if I have

[[A]]
[[B]]
[[C]]

...and I have a button associated with B, I can make B disappear or come back.
But what I want is having a button associated with B, and when I press it, all blocks that does not contains [[B]] will be hidden.
Is there any way to achieve this? It'd be perfect!
Thank you very much Mr Lanza!

It is certainly doable to some extent but you're always going to have edge cases and things you haven't anticipated.  In short whenever I try to get what I'm after it involves trial and error.
I'll get you started, but you're going to have to work out your own details.
Add a new button to the plugin settings:
    ...
    "bee": {
      "desc": "Toggles the visibility of bee",
      "disabled": false,
      "refreshRate": 5,
      "hits": [
        "text-decoration: underline;",
        ""
      ],
      "styles": [
        {
          "tooltip": "Everything",
          "char": "\\ea69",
          "status": "everything",
          "hits": "div#main-content-container div[data-refs-self*='\"bee\"']"
        },
        {
          "tooltip": "Bee only",
          "char": "\\ea67",
          "status": "bee-only",
          "hits": "div#main-content-container div[data-refs-self*='\"bee\"']"
        },
        {
          "tooltip": "Other than bee",
          "char": "\\ea68",
          "status": "other-than-bee",
          "hits": "div#main-content-container div[data-refs-self*='\"bee\"']"
        }
      ]
    },
    ...

Add styles to your custom.css:
body[data-sc-bee="bee-only"] div[data-refs-self] {
  display: none;
}
body[data-sc-bee="bee-only"] div[data-refs-self*='\"bee\"'] {
  display: inherit;
}
body[data-sc-bee="other-than-bee"] div[data-refs-self] {
  display: inherit;
}
body[data-sc-bee="other-than-bee"] div[data-refs-self*='\"bee\"'] {
  display: none;
}

You spoke of having a button which makes B disappear and come back.  Then you indicated interest in having another button which makes all-other-than-B disappear and come back.  This puts the two buttons at odds with each other.  Fortunately, Style Carousel was designed for more than toggling.  You have a button with three possible states.  That's what I set up for you.  Each state explains itself.
Unfortunately, the only way tweak all this involves looking under the hood.  I had to open the Developer Tools and look at the DOM to get you started.    There's no shortcut and no way around it.  Plus, you're going to have to CSS.  Carousel just puts the appropriate hooks in place.  The developer is still needed to determine what styles should be installed.
With Carousel and Classy you can do just about anything.  It's really a matter of your time and patience.


Weird behavior if interval parameter is string

Try this
var rule = new RRule({
freq: RRule.DAILY,
interval: '1',
dtstart: new Date(2016, 1, 1, 10, 30),
until: new Date(2016, 1, 12)
});
It should have generated 11 repeats from January 1 to January 12
But this is what we receive if call rule.all()
[Mon Feb 01 2016 10:30:00 GMT+0200 (EET), Thu Feb 11 2016 10:30:00 GMT+0200 (EET)]

interval It's an integer argument, so passing a string incorrect

interval It's an integer argument, so passing a string incorrect


Use code coverage

Just wasted an hour trying to find some way of running it through actual grownup visual studio.
I am not willing to waste further time.
WONTFIX.


Generalize TargetRefExists() and UpdatePolicyStatus() methods for Policy CRDs

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs from aws-gateway-controller showing the issue:
Testing done on this change:

Automation added to e2e:

Will this PR introduce any new dependencies?:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I dont like the direction of this PR. Not clear description, not clear why moving code into different place makes it cleaner/better/easier to understand. Dont see path forward.

There might be some generic parts in two functions, it would be easier to seem them when we actually implement both of them, and common patterns emerge. Right now I dont see these common patterns in original code and this PR to make decision. These utility functions looks to specific to me, to be in generic utils.

There might be some generic parts in two: AccessLog and Auth, it would be easier to see them when we actually implement both of them, and common patterns emerge. Right now I dont see these common patterns in original code and this PR to make decision. These utility functions looks to specific to me, to be in generic utils.

My later PR will include that code in iamauthpolicy_controller.go:
func (r *authPolicyReconciler) reconcileUpsert(ctx context.Context, iap *anv1alpha1.IAMAuthPolicy) error {
	if err := r.finalizerManager.AddFinalizers(ctx, iap, authPolicyFinalizer); err != nil {
		r.eventRecorder.Event(iap, corev1.EventTypeWarning,
			k8s.IAMAuthPolicyEventReasonFailedAddFinalizer, fmt.Sprintf("Failed to add finalizer due to %s", err))
		return err
	}

	if !core.AuthPolicyTargetRefExists(ctx, r.client, r.log, iap) {
		r.log.Infof("Could not find Auth Log Policy targetRef %s %s",
			iap.Spec.TargetRef.Kind, iap.Spec.TargetRef.Name)
		err := utils_policy.UpdatePolicyStatus(ctx, r.client, iap, gwv1alpha2.PolicyReasonTargetNotFound)
		if err != nil {
			return fmt.Errorf("failed to update Auth Log Policy status, %w", err)
		}
		return nil
	}

	err := r.buildAndDeployModel(ctx, iap)
	if err != nil {
		// TODO: Error handling
		return utils_policy.UpdatePolicyStatus(ctx, r.client, iap, gwv1alpha2.PolicyReasonInvalid)
	}
	err = utils_policy.UpdatePolicyStatus(ctx, r.client, iap, gwv1alpha2.PolicyReasonAccepted)
	if err != nil {
		return fmt.Errorf("failed to update Access Log Policy status, %w", err)
	}
	return nil
}

That use UpdatePolicyStatus AuthPolicyTargetRefExists, should I include that in this PR?


Use pyodbc with SQLAlchemy / Teradata?

Is it possible to use pyodbc with SQLAlchemy for query Teradata?

ODBC drivers for Teradata are available: https://downloads.teradata.com/download/connectivity

@v-chojas
Thank you! But I'm asking about integration with SQLAlchemy.

Closing this due to age.  I'm assuming this was asked of the SQLAlchemy group.  Please add any more information you found for others - thanks.


Added examples for Dataloader creation

Added Jupyter notebook and modified the examples and README file. Please review and suggest any changes

Thanks man !!


Remove/update m-form-field modifiers

There are no styles associated with m-form-field__selectable (should be m-form-field--selectable). This was likely added for consistency because we do have m-form-field--checkbox and m-form-field--radio, however, we don't have a modifier for every single type of form input (e.g. text inputs), so it's probably better if we add this back if its needed. There's an argument this could be helpful in the crawler, but in that case the internal select a-select or m-multiselect classes could be queried instead.
Removals

Unstyled m-form-field__select class.

Changes

Update m-form-field__file-input to m-form-field--file-input because it's a modifier not an element.

How to test this PR

PR checks should pass.

SonarCloud failure is just because I touched files it now sees as duplicate.


Bulk operations crash picotorrent consistently

Trying to perform any bulk cancellations crashes picotorrent.
Reproduce by:

Selecting more than one torrent
right click and click remove
crash should occur

Device info:
Windows 10 home edition
No errors come up in event viewer

Additionally, the app tends to delete unselected and unfinished downloads but keeps them in the list and begins re-downloading the next time I start it. It is easily reproducible.

@erayrafet Please make a separate ticket for unrelated issues like that.

@VideoPlayerCode It's not exactly unrelated because it is still a bulk action.

@erayrafet Can you give more detail? I can reproduce the original issue but not yours.

Let's say that I have 5 torrents - 2 of them are finished and the other 3 are still running. If I try to delete/remove the downloaded torrents, the app crashes and upon the next launch, it starts downloading the unfinished ones from scratch.


What datasets are used for pretraining?

The pretrained model works amazingly well on the real-life photos! What datasets are used for pretraining?
Can you please provide the training details of the pretrained model? Thanks!

As described in Sec 4.1 in the paper, in addition to our rendered dataset, we collect major public datasets for training including Sceneflow, Falling Things, Airsim, etc. No pre-training steps are implemented.