debug complete

[x] correctly formatting the response from http GET '5000:/cow?msg=test test test'
[x] correctly formatting the response from  http POST '5000:/cow msg=test test test' and return it in the json response


Broken partition size

Partition is size is calculated improperly
To Reproduce

Build any project with BOOTLOADER_MCUBOOT enabled
Observe improper flash size

Expected behavior
Size must be

Impact
No application could be build with bootloader support.
Environment (please complete the following information):

ubuntu 18.04
zephyr sdk
7d7fed0d2b

Additional context
Problem is created by extra 0x before the decimal value. partition size is defined as 0x5b800 (374784)
./zephyr/misc/generated/configs.c:256:GEN_ABSOLUTE_SYM(CONFIG_FLASH_LOAD_SIZE, 0x374784);
./zephyr/.config:813:CONFIG_FLASH_LOAD_SIZE=374784
./zephyr/include/generated/autoconf.h:247:#define CONFIG_FLASH_LOAD_SIZE 0x374784

https://github.com/zephyrproject-rtos/zephyr/pull/19878


bug using npm run dev`

OS:Nobara Linux 38
DE: GNOME 44.2
Shell: bash 5.2.21
When try to use npm run dev
I have this error sh: line 1: next: command not found

I forgot to install g++
Sorry


Add support for single end data and switch to python3

Hello everyone,
First of all, thanks for the fantastic package. This PR contains the changes mentioned in the title. Both did not require a lot of code changes.
Updating to python3 allows using current versions of various dependencies. It also helps overcome some problems with bundling for bioconda (related PR).
Adding support for single-stranded datasets allows wider usage of the package. I understand that this needs to be thoroughly evaluated before it can make it into production.
If you are interested in adding these changes to the package, I am eager to contribute. If not, I will most likely keep maintaining a fork, as these functionalities are important for my use case.

Hi @nictru , thanks a lot for the helpful PR. There may be some concerns about the code for identifying single-ended / paired-ended BSJ reads. I will look into this, but maybe not very soon. Once done, I will be very happy to merge these codes into the next release.

Maybe I can perform an evaluation similar to the one in the paper on single-stranded data to increase confidence


Missing channel announcement

While working on PR #1493 I believe I discovered a bug related to missing announcements to connected peer.
Problem can be see here
The integration tests failed with an error "channel not opened before timeout".
This error is not consistent and has a probability for showing up (my guess is 25%).
Let me explain the situation using 3 nodes:Alice, Bob, Carol

Alice (P2Pport 19555) connects with Bob (19558). They open a channel between them.

Carol (19561) connects with Alice.

Bob restarts. At startup both Alice and Bob tries to restore the connection. Lets assume that Bob managed to create the connection, from port 59756 to 19555. Alice detected that she does not need to continue trying to set a connection with Bob.

Alice log:
2018-07-02 11:28:57.038 [DBG] CMGR: Attempting to connect to 0397268f5962e3cdf0fe3d9f8acac4b2384f74a8ace109ed400e19727b4cc9da6f@<IP_ADDRESS>:19558 (reqid 13)
2018-07-02 11:28:57.039 [DBG] CMGR: Failed to connect to 0397268f5962e3cdf0fe3d9f8acac4b2384f74a8ace109ed400e19727b4cc9da6f@<IP_ADDRESS>:19558 (reqid 13): dial tcp <IP_ADDRESS>:19558: connect: connection refused
2018-07-02 11:28:57.039 [DBG] CMGR: Retrying connection to 0397268f5962e3cdf0fe3d9f8acac4b2384f74a8ace109ed400e19727b4cc9da6f@<IP_ADDRESS>:19558 (reqid 13) in 5s
2018-07-02 11:28:57.539 [INF] SRVR: New inbound connection from <IP_ADDRESS>:59756
2018-07-02 11:28:57.539 [INF] SRVR: finalizing connection to 0397268f5962e3cdf0fe3d9f8acac4b2384f74a8ace109ed400e19727b4cc9da6f, inbound=true
2018-07-02 11:28:57.539 [DBG] PEER: Sending Init to <IP_ADDRESS>:59756
2018-07-02 11:28:57.539 [DBG] CMGR: Canceling: 0397268f5962e3cdf0fe3d9f8acac4b2384f74a8ace109ed400e19727b4cc9da6f@<IP_ADDRESS>:19558 (reqid 13)
2018-07-02 11:28:57.540 [DBG] PEER: Received Init from <IP_ADDRESS>:59756

Now, Carol opens a channel with Alice. All good so Alice tries to broadcast the channel announcement to Bob

Alice log:

2018-07-02 11:29:01.680 [INF] DISC: Broadcasting batch of 4 new announcements
2018-07-02 11:29:01.680 [DBG] SRVR: Broadcasting 1 messages
2018-07-02 11:29:01.680 [DBG] SRVR: Broadcasting 1 messages
2018-07-02 11:29:01.680 [DBG] SRVR: Broadcasting 1 messages
2018-07-02 11:29:01.680 [DBG] SRVR: Broadcasting 1 messages
2018-07-02 11:29:02.039 [DBG] CMGR: Attempting to connect to 0397268f5962e3cdf0fe3d9f8acac4b2384f74a8ace109ed400e19727b4cc9da6f@<IP_ADDRESS>:19558 (reqid 13)
2018-07-02 11:29:02.042 [DBG] CMGR: Ignoring connection for canceled connreq=0397268f5962e3cdf0fe3d9f8acac4b2384f74a8ace109ed400e19727b4cc9da6f@<IP_ADDRESS>:19558 (reqid 13)

Result - Bob is not aware of the channel between Carol and Alice.
This is exactly the situation in the integration test:

Alice and Bob connected.
testSwitchCircuitPersistence - restarts Alice and Bob. They will restore the connection by not always it will be by Alice
In case connection restored by Bob, testSwitchOfflineDelivery will fail.

As Part of #1493 I provided a temporary fix for the situation by reseting Alice's P2Pport (so that Bob will not be able to connect to Alice). This fix should be remove once this issue resolved.

@offerm I'm not able to reproduce this, and it looks like the build where the test was failing is gone (e751e030f53b976c9558fb6f7c490aa8bf7d4efd).
Can you still reproduce this?

@halseth Yes. I think I know how to reproduce this one.
Let me also try to reproduce #1632 as I think these are related.

@halseth I managed to reproduce it. Please see PR #1730.
The PR basically changes Bob's P2P port and restart Bob's node while Bob has a channel with Alice. When Bob starts Alice can't connect it (since she has its old port number) and Bob is the one to restore the connection.
You can see the impact by running integration tests on your development machine.
You can also see the build https://travis-ci.org/lightningnetwork/lnd/builds/416489274 which shows 2 failures. The second  https://travis-ci.org/lightningnetwork/lnd/jobs/416489277 is what we are looking for:

--- FAIL: TestLightningNetworkDaemon/switch_circuit_persistence (29.10s)
lnd_test.go:74: Failed: (switch circuit persistence): exited with error:
*errors.errorString Bob(0): timeout waiting for channel(2e5b51f05a965f7580f6636fa1a2c08c24ca5142a45ead19ed97bd6ba8c13558:0) open: channel not opened before timeout

@offerm Thanks again, really appreciate you babysitting our integration tests! 🤓
I will make sure we look into this after we get v0.5 out the door 👍

@halseth Sure thing!
See the discussion in #1733.
I think #1733 solves this issue.


Binding to */ causes a lot of messages in traces and interferes with other interceptors

I've just started using Polly with Playwright and noticed 2 things about the use of */ interception.
Firstly it seemed to cause a problem with an existing interceptor I had on a more specific route. Depending on the order of attachment polly-playwright-adapter seemed to grab the request and continue it meaning my route interceptor never got a chance so I had to mess around with the order of things to get my test to work again.
Additionally in traces or the (awesome) new UI view I see a lot of route.continue() because I'm only recording a small subset of my requests using shouldHandleRequest.

Would you consider allowing for configuration of the route pattern that this adapter attaches to?  I'm happy to create a PR for it if so.
p.s. Absolutely love this, it's great, thank you so much for creating it!

hey @murphybob,
no problem, thanks for raising this issue and shedding light on the problem! i'd be onboard with adding an extra configuration option to customize the route pattern used for interception - a pr would be totally welcome and appreciated for this.

Thanks 👍 I’m on holiday this week, I’ll put something together next week.
On Sat, 1 Apr 2023 at 23:07, reda @.***> wrote:

hey @murphybob https://github.com/murphybob,
no problem, thanks for raising this issue and shedding light on the
problem! i'd be onboard with adding an extra configuration option to
customize the route pattern used for interception - a pr would be totally
welcome and appreciated for this.
—
Reply to this email directly, view it on GitHub
https://github.com/redabacha/polly-adapter-playwright/issues/61#issuecomment-1493138728,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL2NXOVA3MJPWYP7ZEGHFLW7CRJ5ANCNFSM6AAAAAAWJNIDDY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Any movement on this?

sorry i haven't checked on this in a while. i've introduced a new option now called routesToIntercept which will let you configure this.
merry christmas! :christmas_tree:


Print the "not authorized" error returned from CC when creating service key

Fix a bug that only the spacedeveloper or admin can create a service key. CC will return "Not authorized error" and CLI need to report the error and print out the error message.
Besides, for other errors reported from CC side, we need also to report and print them in CLI side
this is a fix for story [#87480878]
Signed-off-by: Edward Zhang<EMAIL_ADDRESS>

@shalako this PR fixes the issue with story [#87480878] causing the rejection. Hopefully this change should allow you to accept story.

Thank you, @maximilien. Please see my questions in the story: https://www.pivotaltracker.com/story/show/87480878

Thanks, all good and merged


Final fix

primary styling done
followed gitflow

Good job 👍💯 on fixing the issue on the project.
Nothing else to say than, it's time to merge


use net core 1.1 to support more platforms

See https://github.com/dotnet/netcorecli-fsc/issues/76

I'm actually a bit confused here. When I change to:
<TargetFramework>netcoreapp1.1</TargetFramework>

I get:
Error occurred while restoring NuGet packages: The operation failed as details for project SourceLink.Create.GitHub could not be loaded.
Restoring NuGet packages...
Project dotnet-sourcelink is not compatible with netcoreapp1.0 (.NETCoreApp,Version=v1.0). Project dotnet-sourcelink supports: netcoreapp1.1 (.NETCoreApp,Version=v1.1)
Project dotnet-sourcelink-git is not compatible with netcoreapp1.0 (.NETCoreApp,Version=v1.0). Project dotnet-sourcelink-git supports: netcoreapp1.1 (.NETCoreApp,Version=v1.1)
One or more projects are incompatible with .NETCoreApp,Version=v1.0.
NuGet package restore failed.
Time Elapsed: 00:00:01.8636236

I also tried adding a reference to:
https://www.nuget.org/packages/Microsoft.NETCore.App
<PackageReference Include="Microsoft.NETCore.App" Version="1.1.0" />

When I changed back to netcoreapp1.0, I mistakenly left the reference in and got:
1>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.Sdk.DefaultItems.targets(101,5): warning : A PackageReference for 'Microsoft.NETCore.App' was included in your project. This package is implicitly referenced by the .NET SDK and you do not typically need to reference it from your project. For more information, see https://aka.ms/sdkimplicitrefs

https://aka.ms/sdkimplicitrefs

I just tried again with bc-csharp #175, but got:
Errors in C:\projects\bc-csharp\crypto\src\crypto.csproj
      Package dotnet-sourcelink-git 2.1.0-b435 is not compatible with netcoreapp1.0 (.NETCoreApp,Version=v1.0). Package dotnet-sourcelink-git 2.1.0-b435 supports: netcoreapp1.1 (.NETCoreApp,Version=v1.1)
      One or more packages are incompatible with .NETCoreApp,Version=v1.0.

A DotNetCliToolReference shouldn't have an impact like that on the project being built. Anyone have a link to a relevant .NET bug?


Greenkeeper/eslint plugin node 9.0.1

:tada: This PR is included in version 1.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


pygame into requirements

Hi,
First of all, thanks for maintaining pyvgdl.
This is the first of two issues I want to bring to your attention.
It seems that pygame is required to run py-vgdl, of course, but it isn't in the requirements.txt.
I just cloned your repo, installed it in a clean conda environment of Python 3.6 with pip install -e '.[all]', and when trying to run python -m vgdl.util.humanplay.play_vgdl vgdl/games/aliens_lvl0.txt I ran into a ModuleNotFoundError, saying pygame should be installed.
Cheers.

Alright that's a good catch. I could simply add it to requirements, but I don't think any version of pygame is compatible. So I'll make work of it, but I need to take a proper look and try out some versions.


unexpected behavior/errors when changing storePathCommitLog directory

What did you do (The steps to reproduce)?
1) stop master broker
2) change ${storePathCommitLog} directory setting from '/home/rocketmq' to '/rocketmq'
3) mv /home/rocketmq/* /rocketmq/
4) restart master broker
repeat the above steps for slave broker.

What did you see instead?
after restart I observed the following:
1) the Topic and Consumer offsets are no longer changing
2) stop async between master and slave broker

Please tell us about your environment:
Java: 1.8
RocketMQ: 4.3.2

Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):
storeerror.log:
WARN WriteSocketService - Offset not matched. Request offset:<PHONE_NUMBER>51, firstOffset: 0, lastOffset:<PHONE_NUMBER>4, mappedFileSize:<PHONE_NUMBER>, mappedFiles count: 56

@xhkyyy it seems that you just changed the storePathCommitLog, but moved all store message to this path, maybe you should change the storePathRootDir config.

@xhkyyy it seems that you just changed the storePathCommitLog, but moved all store message to this path, maybe you should change the storePathRootDir config.

@duhenglucky I am sure I've set the 'storePathRootDir'.
While reviewing the default value of 'storePathRootDir', I suddenly realized that the 'storePathRootDir' uses the 'store' directory as a subdirectory.
Here's the configuration I use:
storePathRootDir=/rocketmq

I mean, If I want to change the directory, I believe the correct configuration should be this:
storePathRootDir=/rocketmq/store

This is very subtle.

@xhkyyy rocketmq will create a new empty store directory in the subdirectory of your config as it's storage directory for safe, and It seems that developers in the community are also used to this usage, so I close this issue first, but please feel free to reopen it if you have any other questions.


System.InvalidOperationException: The method 'OnMessage' or 'OnMessageAsync' has already been called.

Is this a bug report?
Yes.  Looks like a recurrence of #747 's first issue.
Can you also reproduce the problem with the latest version?
We experienced the problem in 5.1.0.  We have deployed 5.1.2 and are waiting to see if the problem occurs again.
Environment

.NET 4.7.1
Console EXEs and ASP.NET MVC web app on IIS
Built with Visual Studio 2017 15.7.2
Running on Azure App Service x64
MassTransit using Azure Service Bus
3 processes connected to bus
Each process receives some broadcast messages via per-process queues
One process receives competing-consumer messages via a shared queue

Steps to Reproduce
Wait until problem occurs.  No specific trigger is known.
Expected Behavior
MassTransit remains connected to Azure Service Bus indefinitely.
Actual Behavior
An exception from a Task is unobserved and rethrown from the finalizer thread.  Afterward, MassTransit disconnects from Azure Service Bus and receives no more messages until the process is restarted.  This occurs around the same time (within 1hr 30min) for all 3 of our processes connected to the bus.
Reproducible Demo
Would love to have one myself.
Stack Trace
System.AggregateException: A Task's exception(s) were not observed either by Waiting on the Task or accessing its Exception property. As a result, the unobserved exception was rethrown by the finalizer thread.
System.InvalidOperationException: The method 'OnMessage' or 'OnMessageAsync' has already been called.
   at Microsoft.ServiceBus.Messaging.MessageReceiver.OnMessage (Microsoft.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)
   at Microsoft.ServiceBus.Messaging.QueueClient.OnMessageAsync (Microsoft.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)
   at MassTransit.AzureServiceBusTransport.Transport.Receiver.Start (MassTransit.AzureServiceBusTransport, Version=<IP_ADDRESS>6, Culture=neutral, PublicKeyToken=b8e0e9f2f1e657fa)
   at MassTransit.AzureServiceBusTransport.Pipeline.MessageReceiverFilter+<GreenPipes-IFilter<MassTransit-AzureServiceBusTransport-ClientContext>-Send>d__7.MoveNext (MassTransit.AzureServiceBusTransport, Version=<IP_ADDRESS>6, Culture=neutral, PublicKeyToken=b8e0e9f2f1e657fa)
   at GreenPipes.Agents.PipeContextSupervisor`1+<GreenPipes-IPipeContextSource<TContext>-Send>d__8.MoveNext (GreenPipes, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b800c4cfcdeea87b)
   at GreenPipes.Agents.PipeContextSupervisor`1+<GreenPipes-IPipeContextSource<TContext>-Send>d__8.MoveNext (GreenPipes, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b800c4cfcdeea87b)
   at GreenPipes.Agents.PipeContextSupervisor`1+<GreenPipes-IPipeContextSource<TContext>-Send>d__8.MoveNext (GreenPipes, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b800c4cfcdeea87b)
   at MassTransit.AzureServiceBusTransport.Transport.ReceiveTransport+<>c__DisplayClass16_0+<<Receiver>b__0>d.MoveNext (MassTransit.AzureServiceBusTransport, Version=<IP_ADDRESS>6, Culture=neutral, PublicKeyToken=b8e0e9f2f1e657fa)
   at MassTransit.AzureServiceBusTransport.Transport.ReceiveTransport+<>c__DisplayClass16_0+<<Receiver>b__0>d.MoveNext (MassTransit.AzureServiceBusTransport, Version=<IP_ADDRESS>6, Culture=neutral, PublicKeyToken=b8e0e9f2f1e657fa)
   at MassTransit.Policies.PipeRetryExtensions+<Retry>d__1.MoveNext (MassTransit, Version=<IP_ADDRESS>6, Culture=neutral, PublicKeyToken=b8e0e9f2f1e657fa)
   at MassTransit.Policies.PipeRetryExtensions+<Retry>d__1.MoveNext (MassTransit, Version=<IP_ADDRESS>6, Culture=neutral, PublicKeyToken=b8e0e9f2f1e657fa)
   at MassTransit.AzureServiceBusTransport.Transport.ReceiveTransport+<Receiver>d__16.MoveNext (MassTransit.AzureServiceBusTransport, Version=<IP_ADDRESS>6, Culture=neutral, PublicKeyToken=b8e0e9f2f1e657fa)

Bus Configuration
private IBusControl CreateBusUsingAzureServiceBus()
{
    return Bus.Factory.CreateUsingAzureServiceBus(c =>
    {
        // Normalize the URI
        var uri = Configuration.HostUri;
            uri = ServiceBusEnvironment.CreateServiceUri(AzureServiceBusScheme, uri.Host, "");

        // Configure connection to Azure Service Bus
        var host = c.Host(uri, h =>
        {
            h.SharedAccessSignature(s =>
            {
                s.KeyName         = Configuration.Secret?.UserName;
                s.SharedAccessKey = Configuration.Secret?.Password;
                s.TokenTimeToLive = TimeSpan.FromDays(1);
                s.TokenScope      = TokenScope.Namespace;
            });
        });

        // Disable retries by default
        c.UseRetry(a => a.None());

        // Everything below is for message reception
        if (!Configuration.IsReceiveEnabled)
            return;

        // Ensure long-running consumers will not lose their lock on the message
        c.UseRenewLock();

        // Configure the request queue (shared among all instances, persistent)
        c.ReceiveEndpoint(host, Configuration.QueueName, r =>
        {
            ConfigureFilters(r);
            LoadRequestConsumers(r);
        });

        // Configure the event queue (per-instance, auto-deleted)
        c.ReceiveEndpoint(host, r =>
        {
            // NOTE: Though the queue will auto-delete after 5 min, the
            // subcriptions for it will remain, and worse, will fill up
            // with messages. An external maintenance script should run
            // periodically to clean up those go-nowhere subscriptions.
            // Source: https://github.com/MassTransit/MassTransit/issues/553
            ConfigureFilters(r);
            LoadEventConsumers(r);
        });

        // If the normal plumbing fails, MassTransit will move messages
        // to an error queue.  We currently do not monitor the error
        // queue, so we just need to prevent it from filling up.
        c.ReceiveEndpoint(host, Configuration.QueueName + "_error", r =>
        {
            // Log any request in the error queue
            r.Consumer<LoggingConsumer>();
        });
    });
}

private static void ConfigureFilters(IPipeConfigurator<ConsumeContext> c)
{
    c.UseFilter(new LoggingFilter            <ConsumeContext>());
    c.UseFilter(new ApplicationInsightsFilter<ConsumeContext>());
}

protected virtual void LoadRequestConsumers(IReceiveEndpointConfigurator r)
{
    // Messages that use competing-consumer
    r.LoadConsumersFrom(_context, IsRequestMessage);
}

protected virtual void LoadEventConsumers(IReceiveEndpointConfigurator r)
{
    // Messages that are broadcast
    r.LoadConsumersFrom(_context, IsEventMessage);
}

Hopefully not, this wouldn't be a good thing to see this occurring again.

We are having the same issue in a very similar environment with MassTransit version 5.1.2

I have the same issue on Azure Web jobs. Mass transit version 5.1.3

I'm fairly certain this commit will fix it, since I reproduced it by deleting the queue underneath and saw it properly reconnect and reapply the topology to create the queue, and a new QueueClient.

Are you gonna release this fix with a new nuget package version?

Soon, next day or two - been a busy week.

Good to hear.  We deployed 5.1.2 and 5.1.3 and still see the problem.

5.1.4 is now building, and should deploy shortly.

This was released, if you still see the issue let me know, otherwise for now I'm going to close this issue.

I expect we'll deploy this update today or tomorrow.  We'll know if it fixes the issue in a few days.

Cool, I was able to reproduce the issue, so I'm hopeful it's resolved.

Is anybody still having this issue? We thought of deploying 5.1.2/5.1.3 to production but @sharpjs methoned he reproduced again in those version. @sharpjs, @teosangio79 or anyone can confirm this issue in not re-produced in 5.1.4 ?

Hi, to me the issue is solved. My message consumer is running for 15 days without any issue at the moment.

I haven’t had it reported with 5.1.4, the commit addressing this was in that release.

Long-term report: it remains solved for us. cc @kailashravuri


JTAppleCalender PreMonth selection issue

IMPORTANT - BEFORE CREATING YOUR ISSUE PLEASE READ THE FOLLOWING:

PLEASE STATE THE VERSION NUMBER you are using.
Ans:       6.0

Are you using the latest version of JTAppleCalendar? Latest version is currently 7.1.5
Ans:   No, I updated latest but its gives many errors so are degrade to 6.0

I have implemented JTAppleCalender in my iOS project. for to select start date adnd end date.
Now the my problem is that When I open the calender popup from current month its not going to back month, ie. current month is April 2018 and I want to select the date 15 March 2017 its not possible right now.  Please help me.

Version 7 of this library has been out for some time now (over a year)
Going back to support version 6 would be too difficult for me.
I can help you port your 6.0 project to version 7.
Also version 7 has many videos which could help porting over from version 6 to 7.

closing issue.
Let me know when you need help porting to version 7


Fix: llama2 fp8 quantization, fix invalid transformers syntax for padding/truncating to length

Fix for wrong syntax in fp8 llama2 quantization script:
https://github.com/NVIDIA/TensorRT-LLM/issues/288

Hi @0xymoro , we have included your fix in the latest update of the main branch, and we added you as the co-author, thanks very much for your great contribution!
I'm closing this PR, please feel free to let us know if you have any questions, thanks.


Design Separate Compilation

Description
While pseudo-linkage (#40) is sufficient for using our polyfill system, we need a way to deal with the more complex notions of linkage that arise in our source languages. This work can potentially be made part of the Cairo compiler proper, giving Cairo a notion of cross-module linkage and separate compilation.

Closed as this was accounted for by #25.


Version 31.1.3

31.1.3

Feedback component visual updates (PR #2894)

I think this warrants a minor version rather than a patch bump, as in I think this is a 31.2.0

I think this warrants a minor version rather than a patch bump, as in I think this is a 31.2.0

do i need to update branch name or is that fine

It doesn't matter so up to you.


[Feature Request] Add a "router" attribute on Menu-Item

Existing Component
是
Component Name
Menu-Item
Description
I have a Menu, and there are 20  items under it. With "router" set to true, it's very convenient to route. But for only a few menu items, I want to use my own @click. So I suggest to add an attribute "router" on MenuItem, to prevent the behavior inherits from Menu. And maybe it can also be added to Menu-Group.

Duplicate https://github.com/ElemeFE/element/issues/13292


Fix if-else to keep newline character

Removed trailing - characters from if-else in helm chart to keep newline characters in the generated yaml.
fixes https://github.com/jenkins-x/jx/issues/3494

Tested both w/ and w/o a value in Global.EnvVars.DOCKER_REGISTRY key of myvalues.yaml, using command helm install --dry-run --debug D:\jx\jenkins-x-platform. The output is the expected one:
---
# Source: jenkins-x-platform/templates/jenkins-x-docker-registry.yaml
apiVersion: v1
kind: ConfigMap
metadata:
  name: jenkins-x-docker-registry
  annotations:
    expose.config.fabric8.io/clusterip-port-if-empty-key: docker.registry
data:
  docker.registry: test:5000

/assign @jenkins-x-bot


Python 3.13 support

Hi Connector-x team
I am a happy user of Connector-x, but like to more forward to new python version soon after they are released. Is it possible that you could update connector-x to support python 3.13 now that it has been released.
Best regards Kristian

We are waiting for all our dependencies like numpy to release the newest version (0.22 on cargo in this case). Once they are done, we can release a new stable version that supports python 3.13.

Python 3.13 is now supported from version 0.4.0! Please feel free to have a try.


server with db create/update & delete fn done

No problem.
On Mon, Mar 27, 2023 at 7:35 AM RohitShah1706 @.***>
wrote:

@tessamero https://github.com/tessamero
This PR was made by mistake and I am yet add many functionalities yet
I request you to not close this PR
Once again, sorry for the mistake.
—
Reply to this email directly, view it on GitHub
https://github.com/appwrite-octernship/devrel-task-2/pull/3#issuecomment-1485230511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANWMBGQWKWPARA5C2OUVM3W6GQU7ANCNFSM6AAAAAAWJHBG3Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Just a reminder you can make comments/questions here on the Octernship Discussions https://github.com/orgs/appwrite-octernship/discussions


gp output scaler

Added output scaler option for GP's
There is repeated code for the Pydantic validator so we probably need to think about a better way to do this in the future.

Hi Johannes. That is a possible idea about would be suitable for now I think. I'm not sure about how pydantic2 works though and how it changes things...
For now I'll make your suggested change and commit it when i get the chance

Hi @jduerholt,
I've made the suggested change by adding a new TrainableBotorchSurrogate data model.
In addition, the random forest data model was also modified because it originally inherits from BotorchSurrogate and TrainableSurrogate too. Whilst this is not completely necessary, I changed it to inherit TrainableBotorchSurrogate so RandomForestSurrogate will now also take scaler and output_scalar keywords arguments, which both default to  ScalerEnum.IDENTITY

Thx, I will have a look over the course of the week!


fix SSM UI don't show "Add Rule" and "Run Action" in the first login.

@MajorJason & @lipppppp, the patch looks good to me. Will commit the patch soon, thanks so much!


Session handling

This pull request updates dependencies to their latest versions to mitigate security vulnerabilities. This pull request also and fixes an issue that would cause the client login method to not save the new session on creation.

Coverage increased (+0.0003%) to 99.767% when pulling 996f78caaa07c4c23c5873c85a176fff5e234984 on session-handling into eaa8b927ca78b98db71adc07c0e755cc660951e7 on master.


FFmpeg filter support?

Is there any way to use FFmpeg filters (such as those done using the -vf flag on the ffmpeg CLI) with this crate? If not, is there a way to interop with ffmpeg-next to do this?

That is unfortunately not in the scope of this library. ffmpeg-next does have it I think. You can pull raw frames and interop with ffmpeg-next, that would work most likely (I haven't tried).


Deleting the wrong rule

Created 4 rules for a ruleset.
Selected the last in the list and deleted. It deleted the second last one instead.

Deleting the second rule in a list of 4 also doesn't work

Fix verified


Release: specify arch for base image and use GA version of Python SDK

Will update the release page.


where is memcached??

link is broken

try this https://blog.couchbase.com/memcached-144-windows-32-bit-binary-now-available/


Duplicated requests in app details (tabs)

WIP
Trying to remove the duplicated requests when selecting a tab (variants, analytics, etc), it seems that both the click and the controller activation execute the same request(s) twice.
I removed the "ng-link" directive and added a check on the $watch function to verify if the new value is different from the old one
@lholmquist @sebastienblanc @edewit @matzew

For me, I still see the duplicated requests - I don't really know too much of Angular 1.x
Perhaps @lfryc has a thought ?

@lholmquist @sebastienblanc @edewit anyone able to take a look ?

Taking the reins of this :horse:


Cache gems into ~/.bundle/cache [WIP]

We probably also want to test that this works when separate local bundles are installed on the same machine, i.e., that they don't overwrite the .gems in the cache, although I'm unsure how to do this just yet.

Only gems with no extensions can be safely cached globally, I think.

@segiddins this cache is for the .gem files, not for the installed gems. so all gems can be cached here

This implementation looks good to me, but as I read it I realized that I think we can't do global caching this way. Because load_global_cache reads every cached .gem file, ungzipping and untarring it to extract the .gemspec file and then evaluating the gemspec, it's going to get slower and slower as the global cache grows.
Instead of loading gemspecs from the global cache, I think we will have to use it as a pure download cache instead—when the RubyGems source is asked to download a .gem file, it checks the global cache and provides the cached .gem if present, and downloads the gem into the global cache if it is not. Make sense?

See https://github.com/bundler/bundler/pull/3983/


Check that default_chat_template is also None

The chat template is currently overwritten if the chat_template attribute is None. Some tokenizers are implemented using default_chat_template, and do not set chat_template. I noticed this in my use case with CodeLlama.
Bug report: https://github.com/huggingface/alignment-handbook/issues/84

Great catch, cheers! If you're open to writing a unit test against your codellama example, that would be great to ensure we don't accidentally introduce a regression in future :)

No worries @lewtun ! Unit test added.
The test includes a docstring for clarity, but none of the others do - if you prefer the code to be "self-documented" let me know and I can remove it.
Let me know if any other changes are needed before you can merge :)


prefilter capture, capture packet count increases

bug in the capture code where captured bytes is correct, but the captured packet counts the raw captured packets, not # of packets after the prefilter capture

fixed, needs regression test

prefilter regresion test updated to check the counts


pix2pix.py is modified to work on tensorflow 2.0 or higher

I upgrade the pix2pix.py with tf_upgrade_v2 command to make compatible with tensorflow2.0. pix2pix.py is now running perfectly fine on freshly installed tensorflow2.2 with anaconda and google colab.

Would love to see this merged! Just went through a similar change myself to get the code to work.
@mehshankhan was wondering if there was a reason you chose to use tf.compat.v1 rather than changing to the v2 version of each method.

Please fork the repo and make a PR to link to your fork in the README


grunt-contrib-cssmin creating min.css empty files

I have the following configuration:
cssmin: {
	  target: {
		files: [{
		  expand: true,
		  cwd: css_dir,
		  src: ['**/*.css',  '**/!*.min.css'],
		  dest: css_dir,
		  ext: '.min.css'
		}]
	  }
	},

I am runnit this grunt task with ANT,
When i execute the tasks the result is:
[exec] ←[4mRunning "cssmin:target" (cssmin) task←[24m
[exec] ←[32mDone, without errors.←[39m
And the output directory has the .min.css files but those files are all empty.

@scuenca can you please ask raise it in cssmin? It rather has nothing to do with clean-css and I don't maintain any plugins. Thanks!

sorry!
In the cssmin page: https://github.com/gruntjs/grunt-contrib-cssmin
Says:

Issues with the output should be reported on the clean-css issue tracker.

Well, I don't know what's the input so unless you share it I can't really figure it out.

Ok, let me explain please,
The css_dir variable is equals to E:\project\src\main\webapp\css
In this css directory there is only one file:
printable.css
with the following content:
* {
	overflow: visible !important;
	float: none !important;
	position: relative;

}
div {
	width: auto !important;
}
.ui-widget-content, .ui-state-default {
	border:1px solid #000000 !important;
}
.botones_tabla_datos, .tabla_datos {
	display: none !important;
}

When i execute the task cssmin (grunt-contrib-cssmin) that is calling cleancss, the result is:
[exec] ←[4mRunning "cssmin:target" (cssmin) task←[24m
[exec] ←[32mDone, without errors.←[39m
And the output directory (E:\project\src\main\webapp\css) has the printable.min.css files but this file is empty.

Great, thanks for sharing!
Please also let me know which version of clean-css you run, under node_modules/grunt-contrib-cssmin/package.json in "dependencies" section.

It's the version 3.1.0

I've just minified the file with cleancss binary and it works fine on 3.1 so it has to be something different. Are you sure only that file gets matched by src: ['**/*.css',  '**/!*.min.css']?

Hi Jakub,
I have specified the cwd parameter of cssmin that is the init directory, in this case: webapp\css.
In this directory i only have one file, printable.css with the content that you have seen. Its encoding is UTF-8 without boom.

I'm sorry but that's got to be a grunt plugin issue then. As a last resort, could you point your package.json to version 0.11 instead of 0.12 and see what happens?

You are right Jakub!
With the version 0.11, it works.
*{overflow:visible!important;float:none!important;position:relative}div{width:auto!important}.ui-state-default,.ui-widget-content{border:1px solid #000!important}.botones_tabla_datos,.tabla_datos{display:none!important}

Thank you very much!!, and sorry for your time. I will report the error to them.

Sure, no probs. Let's see where it goes!


Check if cilium leaks goroutines with kafka requests

In tests there were 15 goroutines still running blocked in vendor/github.com/optiopay/kafka/proto/serialization.go:70 might be worth checking if we are leaking goroutins
syscall.Syscall(0x0, 0x43, 0xc423e10000, 0x4, 0x0, 0xc422d917f8, 0x4abd50)
	/usr/local/go/src/syscall/asm_linux_amd64.s:18 +0x5
syscall.read(0x43, 0xc423e10000, 0x4, 0x400, 0xc422d91800, 0x0, 0x0)
	/usr/local/go/src/syscall/zsyscall_linux_amd64.go:756 +0x55
syscall.Read(0x43, 0xc423e10000, 0x4, 0x400, 0x0, 0x0, 0x402)
	/usr/local/go/src/syscall/syscall_unix.go:162 +0x49
internal/poll.(*FD).Read(0xc421d8f780, 0xc423e10000, 0x4, 0x400, 0x0, 0x0, 0x0)
	/usr/local/go/src/internal/poll/fd_unix.go:122 +0x125
net.(*netFD).Read(0xc421d8f780, 0xc423e10000, 0x4, 0x400, 0x8, 0xc4200efbc0, 0xc422d91998)
	/usr/local/go/src/net/fd_unix.go:202 +0x52
net.(*conn).Read(0xc420465a70, 0xc423e10000, 0x4, 0x400, 0x0, 0x0, 0x0)
	/usr/local/go/src/net/net.go:176 +0x6d
io.ReadAtLeast(0x7f0d1b6a5178, 0xc420465a70, 0xc423e10000, 0x4, 0x400, 0x4, 0x17018ec, 0xc42150f800, 0x7f0d1b81d000)
	/usr/local/go/src/io/io.go:309 +0x86
io.ReadFull(0x7f0d1b6a5178, 0xc420465a70, 0xc423e10000, 0x4, 0x400, 0x400, 0x19a5ba0, 0xc422d91a01)
	/usr/local/go/src/io/io.go:327 +0x58
github.com/cilium/cilium/vendor/github.com/optiopay/kafka/proto.(*decoder).DecodeInt32(0xc422d91af0, 0x400)
	/home/vagrant/go/src/github.com/cilium/cilium/vendor/github.com/optiopay/kafka/proto/serialization.go:70 +0x72
github.com/cilium/cilium/vendor/github.com/optiopay/kafka/proto.ReadResp(0x7f0d1b6a5178, 0xc420465a70, 0x0, 0x0, 0x1000000000001, 0xc422d91b90, 0x4112a8, 0x1a97040)
	/home/vagrant/go/src/github.com/cilium/cilium/vendor/github.com/optiopay/kafka/proto/messages.go:174 +0xd7
github.com/cilium/cilium/pkg/kafka.ReadResponse(0x7f0d1b6a5178, 0xc420465a70, 0xc420465a70, 0x7f0d1b6a5178, 0xc420465a70)
	/home/vagrant/go/src/github.com/cilium/cilium/pkg/kafka/response.go:51 +0x53
github.com/cilium/cilium/pkg/proxy.(*kafkaRedirect).handleResponses(0xc422eb1da0, 0xc423320cc0, 0xc4239939e0, 0xc4239cd590, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/home/vagrant/go/src/github.com/cilium/cilium/pkg/proxy/kafka.go:327 +0x1f6
github.com/cilium/cilium/pkg/proxy.(*kafkaRedirect).handleResponseConnection(0xc422eb1da0, 0xc4239939e0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/home/vagrant/go/src/github.com/cilium/cilium/pkg/proxy/kafka.go:367 +0x1f7
created by github.com/cilium/cilium/pkg/proxy.(*kafkaRedirect).handleRequest
	/home/vagrant/go/src/github.com/cilium/cilium/pkg/proxy/kafka.go:280 +0xbbf

My interpretation would be that we leak connections. Eg the policy was changed from L7 + Kafka rules to an L3 policy (or at least with no kafka rules) but the established connections haven’t been closed (RST) when their entries have been removed from conntrack, or the entries haven’t been removed conntrack yet.

Hmmm actually, the trace above is for the goroutine handling a response connection. When a redirect is removed, we close the request connections immediately. But we give a 1-minute grace period for the response connection to close after that.
Maybe you took that trace less than one minute after reconfiguring a policy to remove Kafka rules.

They may be coming from pkg/proxy/kafka_test.go

They may be coming from pkg/proxy/kafka_test.go

The unit tests aren't ran in the same environment as the runtime tests, which is where the stacktrace @aanm posted is from. The unit tests are ran via Docker Compose on the host on packet.net in CI. Thus, I do not think they are from the unit test.

OK, it wasn't clear to me that this is from a runtime test. How long after the test was the backtrace taken? Was the policy unloaded and thus the redirect removed?
It may be the case that we tear down the network plumbing before the container is killed so the container can no longer send out a FIN to close the connection gracefully and we won't see an EOF but rely on either closing the connection ourselves on redirect removal or waiting for Linux to give up.
func (k *kafkaRedirect) handleResponses(done <-chan struct{}, pair *connectionPair, c *proxyConnection, record kafkaLogRecord, handler kafkaRespMessageHander) {
        defer c.Close()
        scopedLog := log.WithField(fieldID, pair.String())
        for {
                rsp, err := kafka.ReadResponse(c.conn)

                // Ignore any error if the listen socket has been closed, i.e. the
                // port redirect has been removed.
                select {
                case <-done:
                        scopedLog.Debug("Redirect removed; closing Kafka response connection")
                        return
                default:
                }

                if err != nil {
                        record.log(accesslog.TypeResponse, accesslog.VerdictError,
                                kafka.ErrInvalidMessage,
                                fmt.Sprintf("Unable to parse Kafka response: %s", err))
                        scopedLog.WithError(err).Error("Unable to parse Kafka response; closing Kafka response connection")
                        return
                }

                record.log(accesslog.TypeResponse, accesslog.VerdictForwarded, kafka.ErrNone, "")

                handler(pair, rsp)
        }
}


Empty message placeholder text customizable

Pull Request
Hi,
Thanks for helping us improve the Predix UI platform by submitting a Pull Request.
To help us merge your Pull Request as fast as possible, please fill out the following sections:

A description of the changes proposed in the pull request:

Empty message placeholder text customizable

A reference to a related issue (if applicable):

@mentions of the person or team responsible for reviewing proposed changes:

@randyaskin

working tests:

The tests need to be functional and/or unit tests, written for the wct framework, and placed in the test folder, following our testing guidelines.

The preferred/standard approach is to provide a localization hook for hardcoded strings rather than exposing a property. You are correct that "No Results" should have been included in that localization hook, which I've rectified in v5.1.9. You can now localize / customize the "No Results" text (and other strings) using the resources property, e.g.
<px-data-table resources='{"en":{"No Results":"Nothing Found"}}' table-data="...">
</px-data-table>

Hello Randy!
I've decided to implement this property because the REST service is replying different results whether the service is available, or there is privilege problem or really there is no records. To solve this through localization seems strained to me.
Regards,
Csaba

From: Randy Askin<EMAIL_ADDRESS>Sent: Monday, December 11, 2017 5:51:12 PM
To: PredixDev/px-data-table
Cc: Szell, Csaba (GE Transportation, consultant); Author
Subject: EXT: Re: [PredixDev/px-data-table] Empty message placeholder text customizable (#186)
The preferred/standard approach is to provide a localization hook for hardcoded strings rather than exposing a property. You are correct that "No Results" should have been included in that localization hook, which I've rectified in v5.1.9. You can now localize / customize the "No Results" text (and other strings) using the resources property, e.g.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/PredixDev/px-data-table/pull/186#issuecomment-350784372, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Agv8ixSmDKLnfAFGqGtsBNReE8FvOdDtks5s_V2AgaJpZM4Q9kfg.


malicious?

It's not malicious. This link will answer your question: https://stackoverflow.com/questions/43777106/program-made-with-pyinstaller-now-seen-as-a-trojan-horse-by-avg?noredirect=1&lq=1


Fix #184: Add streamwrapper to read container->log() response

My patch is failing because of master branch is not stable enough   :-(

I'm not a fan on how it's handled here, IMO this should be done like others managers with the following workflow :

If user ask a stream (FETCH_STREAM) return the DockerRawStream
If user ask an object (FETCH_OBJECT) return an object containing the stdin / stderr / stdout, the process to create the object must be inside the logs methods (object should not be aware of the stream like now)

Fixed with 9cb5794d4bcb11993384f071e9539850b7cc3704 Thanks for the work !


1.8.0 release failed

@srvaroa https://github.com/srvaroa/labeler/actions/runs/6943962687 the release didn't work properly so the actions weren't created. Referring to the missing actions tar in the release downloads here: https://github.com/srvaroa/labeler/releases/tag/v1.8.0
Here's the failing action: https://github.com/srvaroa/labeler/actions/runs/6943962687
Originally posted by @mjfwebb in https://github.com/srvaroa/labeler/issues/113#issuecomment-1821009505

Looks like there is an issue in the runners that doesn't populate the GITHUB_REF variable sporadically, which makes the go-release action fail. I changed it to a manual release so I can feed the tag by hand. It looks like it works now, and i've updated the v1 floating reference, as well as the default version in Dockerfile
I'll wait before closing this in case someone screams, otherwise close soon.
Thanks @mjfwebb!

I've ran a test PR with the latest release and things look ok.
I'll close for now, but let me know if any issue appears. https://github.com/srvaroa/labeler/actions/runs/6945949126/job/18896413696?pr=116


Individual deploy test error

Describe the issue:
Individual deploy test error
Repro Steps:
1.Run command: bash ./test-templates.sh -c false -t <template_name> -b pr/551.
2.Change “complete” to “completed” in UI
3.Run azd deploy  --service web
Error Message:

Environment:
OS: Codespace.
Templates:
https://github.com/Azure-Samples/todo-csharp-sql-swa-func
https://github.com/Azure-Samples/todo-python-mongo-swa-func
Branch: pr/551
Expected behavior:
Run azd deploy  --service web  can run successfully.
@jongio for notification.

cc: @danieljurek for Codespace environment

Yes, I ran into this to.  You need to use a bigger codespace spec.  You can go to https://github.com/codespaces/ and change it.

This reads exactly like an out of memory error that we encountered previously. Moving up to an 8GB VM should get you past this problem. We have a PR to move the machine size up by default.

@jongio, @danieljurek - We have changed the machine size to 4 core 8GB, but it failed when we executed bash ./test-templates.sh -c false -t Azure-Samples/todo-csharp-sql-swa-func this time. The error message was shown as below:

@weikanglim do you see this issue when running this template in Codespaces?

@zedy-wj That was caused by a separate issue that's now been fixed. Can you verify if everything is working as expected, and close if necessary?

@weikanglim - We have verified with 4 core 8GB and this issue is no longer reproduced. Besides, the smallest size we can create now is 4 core 8GB. We will close it, thanks.


loader does not support reading file from url

We should be able to load a raml file from a URL, and not just a file.  Note that this changes the way in which includes are processed.

@derrickburns just clarifying, are you wanting to load a main RAML file from URL (e.g. ramlfications.parse("https://api.example.com") or ramlfications.load("https://api.example.com"))? or are you wanting to load a local RAML file, that has an !include pointing to a URL?
The latter, I agree, should be supported (and I think is for remote JSON files IIRC).

The former. In that case, the loader must treat includes as references to URLs
Derrick
On Jun 7, 2016, 9:41 AM -0700, Lynn<EMAIL_ADDRESS>wrote:

@derrickburns(https://github.com/derrickburns)just clarifying, are you wanting to load a main RAML file from URL (e.g.ramlfications.parse("https://api.example.com")orramlfications.load("https://api.example.com"))? or are you wanting to load a local RAML file, that has an!includepointing to a URL?
The latter, I agree, should be supported (and I think is for remote JSON files IIRC).
—
You are receiving this because you were mentioned.
Reply to this email directly,view it on GitHub(https://github.com/spotify/ramlfications/issues/104#issuecomment-224340156), ormute the thread(https://github.com/notifications/unsubscribe/AAjo-hEXjVca8Coek3lY-yrp5uhBQZ-yks5qJZ8_gaJpZM4IwDma).

Include pointing to a URL is not in the raml spec if the raml is local
Derrick
On Jun 7, 2016, 9:41 AM -0700, Lynn<EMAIL_ADDRESS>wrote:

@derrickburns(https://github.com/derrickburns)just clarifying, are you wanting to load a main RAML file from URL (e.g.ramlfications.parse("https://api.example.com")orramlfications.load("https://api.example.com"))? or are you wanting to load a local RAML file, that has an!includepointing to a URL?
The latter, I agree, should be supported (and I think is for remote JSON files IIRC).
—
You are receiving this because you were mentioned.
Reply to this email directly,view it on GitHub(https://github.com/spotify/ramlfications/issues/104#issuecomment-224340156), ormute the thread(https://github.com/notifications/unsubscribe/AAjo-hEXjVca8Coek3lY-yrp5uhBQZ-yks5qJZ8_gaJpZM4IwDma).

@derrickburns hmm so, this isn't actually a requirement in the spec - I don't see it in 0.8 or 1.0 (please correct me if I'm wrong). I'm also hesitant to actually add this feature, as it seems a bit insecure.
Looking at the json library in the stdlib in Python, loading remote JSON files do not seem to be supported. And it seems that the pyyaml package doesn't either (not without implementing your own loader, and hopefully a SafeLoader).  I understand how it would be a nice feature to have, as it could get quite annoying to download files that you need.
I will keep this open, but this feature will be low priority, behind implementing RAML1.0 and any other missing functionality that is required of the spec.

Here is the key paragraph from the 0.8 spec:
"If a relative path is used for the included file, the path is interpreted
relative to the location of the original (including) file. If the original
file is fetched as an HTTP resource, the included file SHOULD be fetched over
HTTP."
https://github.com/raml-org/raml-
spec/blob/master/versions/raml-08/raml-08.md#includes
Sent from Nylas N1, the extensible, open source
mail client.
On Jun 7 2016, at 12:31 pm, Lynn Root<EMAIL_ADDRESS>wrote:

@derrickburns hmm so, this isn't actually
a requirement in the spec - I don't see it in 0.8 or 1.0 (please correct me if
I'm wrong). I'm also hesitant to actually add this feature, as it seems a bit
insecure.

Looking at the json library in the stdlib in Python, loading remote JSON
files do not seem to be supported. And it seems that the pyyaml package
doesn't either (not without implementing your own loader, and hopefully a
SafeLoader). I understand how it would be a nice feature to have, as it could
get quite annoying to download files that you need.

I will keep this open, but this feature will be low priority, behind
implementing RAML1.0 and any other missing functionality that is required of
the spec.

—
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](https://github.com/spotify/r
amlfications/issues/104#issuecomment-224388860), or [mute the
thread](https://github.com/notifications/unsubscribe/AAjo-mv4NhwhJVpZaWUoD2Y4B
N3QV6hVks5qJccrgaJpZM4IwDma).![](https://github.com/notifications/beacon/AAjo-
lTI3ct0rEDh_Aczt9lZ6dWSCY5fks5qJccrgaJpZM4IwDma.gif)

I understand and agree with the priorities.  However, it means that one cannot
parse a RAML file that is served from a web server.  Copying the file is
insufficient, because the include links would all be broken.  Thus, it is not
possible to write a service that checks for compliance with a remotely served
raml file without also downloading all the files separately.
Sent from Nylas N1, the extensible, open source
mail client.
On Jun 7 2016, at 5:44 pm, Derrick Burns<EMAIL_ADDRESS>wrote:

Here is the key paragraph from the 0.8 spec:

"If a relative path is used for the included file, the path is interpreted
relative to the location of the original (including) file. If the original
file is fetched as an HTTP resource, the included file SHOULD be fetched over
HTTP."

https://github.com/raml-org/raml-
spec/blob/master/versions/raml-08/raml-08.md#includes

Sent from Nylas N1, the extensible, open source
mail client.

On Jun 7 2016, at 12:31 pm, Lynn Root<EMAIL_ADDRESS>wrote:

@derrickburns hmm so, this isn't
actually a requirement in the spec - I don't see it in 0.8 or 1.0 (please
correct me if I'm wrong). I'm also hesitant to actually add this feature, as
it seems a bit insecure.

Looking at the json library in the stdlib in Python, loading remote JSON
files do not seem to be supported. And it seems that the pyyaml package
doesn't either (not without implementing your own loader, and hopefully a
SafeLoader). I understand how it would be a nice feature to have, as it could
get quite annoying to download files that you need.

I will keep this open, but this feature will be low priority, behind
implementing RAML1.0 and any other missing functionality that is required of
the spec.

—
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](https://github.com/spotify/r
amlfications/issues/104#issuecomment-224388860), or [mute the
thread](https://github.com/notifications/unsubscribe/AAjo-mv4NhwhJVpZaWUoD2Y4B
N3QV6hVks5qJccrgaJpZM4IwDma).![](https://github.com/notifications/beacon/AAjo-
lTI3ct0rEDh_Aczt9lZ6dWSCY5fks5qJccrgaJpZM4IwDma.gif)

A simple option is to provide a hook for processing "!include" directives....
Sent from Nylas N1, the extensible, open source
mail client.
On Jun 7 2016, at 5:46 pm, Derrick Burns<EMAIL_ADDRESS>wrote:

I understand and agree with the priorities.  However, it means that one
cannot parse a RAML file that is served from a web server.  Copying the file
is insufficient, because the include links would all be broken.  Thus, it is
not possible to write a service that checks for compliance with a remotely
served raml file without also downloading all the files separately.

Sent from Nylas N1, the extensible, open source
mail client.

On Jun 7 2016, at 5:44 pm, Derrick Burns<EMAIL_ADDRESS>wrote:

Here is the key paragraph from the 0.8 spec:

"If a relative path is used for the included file, the path is interpreted
relative to the location of the original (including) file. If the original
file is fetched as an HTTP resource, the included file SHOULD be fetched over
HTTP."

https://github.com/raml-org/raml-
spec/blob/master/versions/raml-08/raml-08.md#includes

Sent from Nylas N1, the extensible, open source
mail client.

On Jun 7 2016, at 12:31 pm, Lynn Root<EMAIL_ADDRESS>wrote:

@derrickburns hmm so, this isn't
actually a requirement in the spec - I don't see it in 0.8 or 1.0 (please
correct me if I'm wrong). I'm also hesitant to actually add this feature, as
it seems a bit insecure.

Looking at the json library in the stdlib in Python, loading remote JSON
files do not seem to be supported. And it seems that the pyyaml package
doesn't either (not without implementing your own loader, and hopefully a
SafeLoader). I understand how it would be a nice feature to have, as it could
get quite annoying to download files that you need.

I will keep this open, but this feature will be low priority, behind
implementing RAML1.0 and any other missing functionality that is required of
the spec.

—
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](https://github.com/spotify/r
amlfications/issues/104#issuecomment-224388860), or [mute the
thread](https://github.com/notifications/unsubscribe/AAjo-mv4NhwhJVpZaWUoD2Y4B
N3QV6hVks5qJccrgaJpZM4IwDma).![](https://github.com/notifications/beacon/AAjo-
lTI3ct0rEDh_Aczt9lZ6dWSCY5fks5qJccrgaJpZM4IwDma.gif)

Ah okay good catch, thanks for correcting me. I'll re-prioritize this for the 0.2.0 release.

Would you like a PR from me for such a "hook"?
Sent from Nylas N1, the extensible, open source
mail client.
On Jun 7 2016, at 5:49 pm, Lynn Root<EMAIL_ADDRESS>wrote:

Ah okay good catch, thanks for correcting me. I'll re-prioritize this for
the 0.2.0 release.

—
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](https://github.com/spotify/r
amlfications/issues/104#issuecomment-224456894), or [mute the
thread](https://github.com/notifications/unsubscribe/AAjo-
tDxz2phZnlV2HV7tjOsAS0WKN-
Dks5qJhGHgaJpZM4IwDma).![](https://github.com/notifications/beacon/AAjo-
gpPbJpgrvfGu6LVhFmRqTKwH9r7ks5qJhGHgaJpZM4IwDma.gif)

I would gladly accept any PRs! Be sure to branch off of the v0.2.0-dev branch.

Thx. I'll have a look to see what I can do.


[StyleCop] Fix warnings in ApplicationInsights projects

Fix spacing
Add trailing comma
Fix documentation

Promoted to MS # 1439


[Release] Stage to Main

https://github.com/adobecom/milo/pull/1963

Why is this commit showing up again?

From CC team we validated bulk publish sanity flows and CAAS card reference and they seem to be fine on STAGE.
bulk publish V2 : https://stage--cc--adobecom.hlx.page/tools/bulk-publish-v2

but we dont see personalization (MEP) working as expected in CC milo product page. request to some one look into this and confirm. ex:
login as all cc apps paid user<EMAIL_ADDRESS>/ Bap@d0be
page: https://www.stage.adobe.com/products/illustrator.html
before login:

after login:

still 'buy now'cta appers even login as paid user
@Blainegunn @3ch023 @vivgoodrich @spadmasa

Looks good from our side

Looks good in DC.

We don’t have a CC QE available to do a thorough testing. However I took a a look at the PR and it only has code for a new block caas-marquee-data which I have verified with authors is not being used in CC pages yet. Also confirmed with @cmiqueo this new block code doesn't impact any existing caas blocks that we are using. hence approving the PR as no impact on CC. cc @sigadamvenkata

@Ruchika4 see the comments above. we tested all the features part of this PR and raised concern to check the personalization flow which seems not working in stage (product pages)

Why is this commit showing up again? Is it due to the outdated branch? 🤔

i'd guess prev PR was merged with 'squash'
will be fixed once we merge with one with 'merge' option


linux: optimize inotify tree lookups/insertions

When there are lots of uv_fs_event_t file watchers on a busy file tree, say on the order of 50,000-100,000, the overhead of tree lookups and insertion/removal starts getting noticeable.
Descriptors returned by inotify_add_watch() are monotonically increasing so a file descriptor-like table won't work but a hash table will, and should hopefully improve performance at the cost of additional memory.
One minor complication is that there can be multiple uv_fs_event_t watchers with the same inotify descriptor. Currently that's handled by linking watchers together using a QUEUE. Maybe that's not necessary when storing them in a hash table.

I have some working code for this (bnoordhuis/libuv@23225a7) but it's more complex than the existing binary tree and the performance issue isn't that big so... let's not?
I'll go ahead and close this but we can revisit if there are more uses for hash tables in libuv (or if someone feels it's a good idea after all, of course; just holler.)


Replace the test framework with PyTest

#3324 has been reverted by #3589 because it included some unrelated changes.
So this PR is for re-applying the changes only related to the purpose which is to use PyTest.

LGTM!


Black does join import lines, but does not remove brackets

Code to format:
from a import (
    aa,
    ab,
)

Black output:
from a import (aa, ab)

Expected output:
from a import aa, ab

Operating system: Ubuntu 17.10
Python version: 3.6.3
Black version: 18.3a2
Does also happen on master: yes

This is a duplicate of #4 and a planned feature :-)

It would be nice if it was consistent.
Now code like:
from a import (
    aa,
    ab,
)
from b import ba, bb

is transformed to
from a import (aa, ab)
from b import ba, bb

which looks kind of strange.

Agreed, this will be fixed soon.

In the mean time, as a workaround, if you manually remove the unnecessary punctuation, Black won't add it back ;-)

The reason I'm leaving a trailing comma for single-element parenthesized expressions is that it looks like a tuple and removing it would change what your code means.
In the import case it's a false positive of course :-)

@ambv Is there anything specifically actionable here that isn't covered by #4? Closing as a dupe; it's still linked from #4 if you want the examples.


Fix sample deduplication in chainSampleIterator

Fixes https://github.com/prometheus/prometheus/issues/8558
Signed-off-by: Julius Volz<EMAIL_ADDRESS>

Thanks!

Thanks for fixing @juliusv  👍🏽


header-menu: Add translation to Children's Guide link

French, Italian, and Portuguese translations - the latter two generated from online translations tools, I don't speak these languages.

I'm actually a bit torn on this one. Should we translate it because we currently don't offer the book in those languages and having it translated in the drop down might confuse people because they could expect that the book is in that language too.
This is the same reason I don't translate the lab descriptions because the rest of the lab is in English and it might mislead people if we describe it in their local language https://ebpf.io/fr-fr/labs/

One thing I was considering was mentioning it in the link itself, something like:
          'fr-fr': 'Livre pour enfants sur eBPF (en anglais)',

(Read in English.) It would make the column a bit wider, but maybe OK?
I'm also fine with dropping the change if you think it's better that way.

Sorry somehow lost this PR. I'm fine to add it with the (en anglais) part


Noob question

I intalled pysnmp and if I open an example ai get some errors when running:
AttributeError: module 'pysnmp.carrier.asyncore.dgram.udp' has no attribute 'DOMAIN_NAME'
or
NameError: name 'USM_AUTH_HMAC96_SHA' is not defined
Please, tell me what I do wrong.

Could you share which pysnmp version you have installed and what example you trying to run?

listen-on-ipv4-and-ipv6-interfaces.py
python 3.7.3. as for pysnmp i just pip installed a few minutes ago

I suspect you have taken the example from GitHub master. The you should take examples from release-4.4.10 branch.

Thanks, i will try to install 5

Just keep in mind that pysnmp 4 is stable, pysnmp 5 is still in development (though it should work).

Ok. Thanks.

One more think, when i run the example provided by you above a got this error (<IP_ADDRESS>):
WinError 10013] An attempt was made to access a socket in a way forbidden by its access permissions
when I put a printer IP isnotead of 127....: [WinError 10049] The requested address is not valid in its context

I think the system prevents you from binding port 161 unless you are a superuser.
Printer IP perhaps is a remote IP, you can only listen on local IP interfaces.
You may try ('<IP_ADDRESS>', 1024), that should avoid both of the above mentioned problems.

You can also change DOMAIN_NAME to domainName in the code.


Acronyms

Allows \gls{} function for presentations.
Note: #3

This looks good. My only requested change is that the aux file should be removed.
I understand now what you were trying to say yesterday! I thought you were telling me it failed to build because you need to install glossaries for it to run. (which was confusing to me because I didn't use the glossaries package in the template). I see now that you meant you were just trying to add acronyms and had to find the acros tex file, not that you had to actually install anything. Glad it wasn't a platform issue or anything wrong with the template.
Please note that I try to maintain the master list of useful nuclear acronyms here and pull requests are always welcome.


Conda forge

Hello! I have successfully put this package on to Conda Forge, and I am extending the invitation for the owner/maintainers of this package to be maintainers on Conda Forge as well. Let me know if you are interested! Thanks.
https://github.com/conda-forge/picologging-feedstock/

Perfect, thank you, please add me to the feedstock repo


Reformat some BarcodeItem code to aid the reader

I was looking over the BarcodeItem class and noticed that much of my effort to understand it could be eliminated by a more strategic use of horizontal and vertical whitespace. This PR includes the changes I made that I think would be helpful.
I also wanted to left align the hash values in to_h, but Rubocop would not permit it.
RSpec and Rubocop tests are still successful after this change.

Looks like there was a CI failure here that might be a real one. Can you investigate @keithrbennett ?

@mdworken, thanks for responding. I've restarted the build to try again. This really surprises me. The differences between this feature branch and master, when whitespace is ignored, consist only of the addition of backslashes to indicate line continuations, and do/end's that replace {}'s. I'm looking into this further.

@mdworken This build passed. I think the error on the previous run was a false positive, caused by something else. I'll trigger another build to get another data point.