Prooph\ServiceBus\PluginServiceLocatorPluginis not compatible with theEventBus`

Prooph\ServiceBus\Plugin\ServiceLocatorPlugin relies onProoph\ServiceBus\MessageBus::EVENT_PARAM_MESSAGE_HANDLER, but the Prooph\ServiceBus\EventBus has its own EVENT_PARAM_EVENT_LISTENERS
This means that registering a ServiceLocatorPlugin against an EventBus is not causing any effect, as no events are intercepted.

Oh, that would be a really bad bug. I guess this line claryfies the issue?
The event bus is just special in allowing multiple handlers for one message (aka listeners for an event)
Should we make it more explicit?

@codeliner I am aware of the fact that they behave differently, but since the type system allows me to register any plugin against an EventBus, this looks kinda weird.

@Ocramius The idea is is the following (which is also explained in the docs and in the code):
CommandBus, QueryBus and EventBus are all some kind of message bus. That is the reason why they share basic logic (by extending Prooph\ServiceBus\MessageBus).
That said, a plugin can be a "message bus plugin" and be attached to every bus type.
The ServiceLocatorPlugin is the best example. Why should it be different for the EventBus?
The event bus triggers the same Prooph\ServiceBus\MessageBus::EVENT_PARAM_MESSAGE_HANDLER
event, just for every listener. For the ServiceLocatorPlugin it does not matter if a command handler should be lazy loaded or multiple event listeners. The plugin just asks the container if it knows the message handler. That's it.
What would you change so that it looks less weird?

@codeliner I managed to simply write my own service locator plugin for now, but here's what is quick and simple improvement to be applied in order to make this less confusing to debug/use: make the listener registration process throw exceptions if registration of unknown event names is used.
Basically, a CommandBus and an EventBus have different internal events, but they are all pre-defined upfront (we know which events will be used): attaching to an event that is unknown may throw an exception when running in a "stricter" mode.
This can all be hidden in the private API of the package: as long as the event system is not shared across two buses, this should be fine.

@Ocramius Very good idea. We'll do that. Thank you.
But to clarify things: The ServiceLocatorPlugin shipped with prooph/service-bus works with the event bus. See my first comment and the linked line in the event bus.
Do I understand your use case wrong? Can you provide me a gist of your ServiceLocatorPlugin?

Here are the two factories that I use to setup the event store and event bus in my project:
<?php

declare(strict_types=1);

namespace MyProject\Factory\Infrastructure\Messaging;

use Doctrine\DBAL\Connection;
use Interop\Container\ContainerInterface;
use Prooph\Common\Event\ActionEvent;
use Prooph\Common\Event\ActionEventEmitter;
use Prooph\Common\Event\ActionEventListenerAggregate;
use Prooph\Common\Event\ProophActionEventEmitter;
use Prooph\Common\Messaging\FQCNMessageFactory;
use Prooph\Common\Messaging\NoOpMessageConverter;
use Prooph\EventStore\Adapter\Doctrine\DoctrineEventStoreAdapter;
use Prooph\EventStore\Adapter\PayloadSerializer\JsonPayloadSerializer;
use Prooph\EventStore\EventStore;
use Prooph\EventStoreBusBridge\EventPublisher;
use Prooph\ServiceBus\EventBus;
use Prooph\ServiceBus\MessageBus;

final class EventStoreFactory
{
    public function __invoke(ContainerInterface $container) : EventStore
    {
        /* @var $connection Connection */
        $connection = $container->get(Connection::class);

        $eventStore = new EventStore(
            new DoctrineEventStoreAdapter(
                $connection,
                new FQCNMessageFactory(),
                new NoOpMessageConverter(),
                new JsonPayloadSerializer()
            ),
            new ProophActionEventEmitter()
        );

        $eventBus = new EventBus();

        $eventBus->utilize($this->buildProjectorRouter($container));

        (new EventPublisher($eventBus))->setUp($eventStore);

        return $eventStore;
    }

    // this is where I replaced the base plugin to make it work with the event store
    // each service inside this container is a `callable[]`
    private function buildProjectorRouter(ContainerInterface $container) : ActionEventListenerAggregate
    {
        return new class ($container) implements ActionEventListenerAggregate
        {
            /**
             * @var ContainerInterface
             */
            private $projectors;

            public function __construct(ContainerInterface $projectors)
            {
                $this->projectors = $projectors;
            }

            public function attach(ActionEventEmitter $dispatcher)
            {
                $dispatcher->attachListener(MessageBus::EVENT_ROUTE, [$this, 'onRoute']);
            }

            public function detach(ActionEventEmitter $dispatcher)
            {
                throw new \BadMethodCallException('Not implemented');
            }

            public function onRoute(ActionEvent $actionEvent)
            {
                $messageName = (string) $actionEvent->getParam(MessageBus::EVENT_PARAM_MESSAGE_NAME);

                if ($this->projectors->has($messageName)) {
                    $actionEvent->setParam(
                        EventBus::EVENT_PARAM_EVENT_LISTENERS,
                        $this->projectors->get($messageName)
                    );
                }
            }
        };
    }
}

<?php

declare(strict_types=1);

namespace CodeReviewsIo\Factory\Infrastructure\Messaging;

use Interop\Container\ContainerInterface;
use Prooph\Common\Event\ActionEvent;
use Prooph\Common\Event\ActionEventEmitter;
use Prooph\Common\Event\ActionEventListenerAggregate;
use Prooph\EventStore\EventStore;
use Prooph\EventStoreBusBridge\TransactionManager;
use Prooph\ServiceBus\CommandBus;
use Prooph\ServiceBus\MessageBus;
use Prooph\ServiceBus\Plugin\ServiceLocatorPlugin;

final class CommandBusFactory
{
    public function __invoke(ContainerInterface $container) : callable
    {
        $commandBus = new CommandBus();

        // just using a `ServiceLocatorPlugin`
        $commandBus->utilize(new ServiceLocatorPlugin($container));
        // this one simply takes the command name and makes it the handler name
        $commandBus->utilize($this->buildCommandRouter());

        $transactionManager = new TransactionManager();

        $transactionManager->setUp($container->get(EventStore::class));

        $commandBus->utilize($transactionManager);

        return function ($command) use ($commandBus) {
            $commandBus->dispatch($command);
        };
    }

    private function buildCommandRouter() : ActionEventListenerAggregate
    {
        return new class implements ActionEventListenerAggregate {
            public function attach(ActionEventEmitter $dispatcher)
            {
                $dispatcher->attachListener(MessageBus::EVENT_ROUTE, [$this, "onRoute"]);
            }

            public function detach(ActionEventEmitter $dispatcher)
            {
                throw new \BadMethodCallException('Not implemented');
            }

            public function onRoute(ActionEvent $actionEvent)
            {
                $actionEvent->setParam(
                    MessageBus::EVENT_PARAM_MESSAGE_HANDLER,
                    (string) $actionEvent->getParam(MessageBus::EVENT_PARAM_MESSAGE_NAME)
                );
            }
        };
    }
}

@Ocramius The ProjectorRouter looks good. Could be a new core plugin. @prolic What do you think?
Question:
if ($this->projectors->has($messageName)) {
                    $actionEvent->setParam(
                        EventBus::EVENT_PARAM_EVENT_LISTENERS,
                        $this->projectors->get($messageName)
                    );
                }

Your container (projectors) returns an array of projectors, right?
Now look at this loop from the event bus:
foreach ($actionEvent->getParam(self::EVENT_PARAM_EVENT_LISTENERS, []) as $eventListener) {
                $actionEvent->setParam(self::EVENT_PARAM_MESSAGE_HANDLER, $eventListener);
                if (is_string($eventListener) && ! is_callable($eventListener)) {
                    $actionEvent->setName(self::EVENT_LOCATE_HANDLER);
                    $this->trigger($actionEvent);
                }
                $actionEvent->setName(self::EVENT_INVOKE_HANDLER);
                $this->trigger($actionEvent);
            }

As you can see self::EVENT_LOCATE_HANDLER is triggered for each event listener, if it is a string!
So the default ServiceLocatorPlugin works the same as for command handlers. No exception.

Your container (projectors) returns an array of projectors, right?

Correct

As you can see self::EVENT_LOCATE_HANDLER is triggered for each event listener, if it is a string!
So the default ServiceLocatorPlugin works the same as for command handlers. No exception.

Ah yes, there is some confusion here, sorry (my bad, I opened this issue after working around it, so I didn't really recall it). The problem is that the event parameters are indeed different here.

Ok, so we have no bug but a list of suggestions and new ideas. Well done. Thank you @Ocramius :)

:+1:

I've created two new issue. I think we can close this one now.

Linking #106 and #105


change default option selected for amount received account from bank to credit

[x] under purchase order to bill conversion

issue fixed!


Consistent number formatting for stacks

Adds a util class with three formatter functions:

runescape style (which matches the one in game)
"pretty" style with locale-aware formatting and 3 sig figs
string to number for converting the other direction (open to criticism, it was a little tricky to find a good method)

Open to suggestions for a better name. ItemManager called it "stack size" so that's what I went with.
Additionally, it updates plugins using gp to use the new formatter.
Before

After

I wish I had a better variety of stuff to show it off on but I'm a poor noob

I'll need to change some of the test cases since they won't pass depending on what locale they are run in

Hmm, probably examine should not be changed, you usually want to see pretty accurate price there.

Maybe examine and the price check command can keep the whole price but ground items and grand exchange offers could show the abbreviated version.

I agree with ground items, but grand exchange offers probably should be accurate as well.

as of now:
stack formatter for

ground items
blast furnace coffer

locale formatter for

grand exchange
examine
price command


Add support for Kibana 7.11

https://github.com/elastic/kibana/releases/tag/v7.11.1
Loving enhanced table on an older version of Kibana. Are we ok to upgrade Kibana to the latest release; will enhanced tables still work?
Thanks!

Hi,
I have to do some tests to say that :)
In the better case, the current code is compatible, and I just have to publish packages for Kibana 7.11.0 and 7.11.1.
In the worst case, the current code is not compatible, and I have to spend some time to find and fix the required changes.

Hi  fbaligand,
Yeah really loving enhanced table on an older version of Kibana. May I know when you are planning to release a version of this plugin which will have added support for kibana 7.11.1?

Hi fbaligand,
It would be very helpful if you let us know when you are planning to release the new version of the enhanced table for Kibana 7.11.0.

Hi fbaligand,
It would be very helpful if you let us know when you are planning to release the new version of the enhanced table for Kibana 7.11.0.

Hi,
I'm currently working on it.
There are some breaking changes in Kibana 7.11 and I try to find how to deal with these breaking changes.

Hi all,
I'm glad to announce you that enhanced-table v1.11.1 is out, with Kibana 7.11 support!
You can get it there:
https://github.com/fbaligand/kibana-enhanced-table/releases/tag/v1.11.1
Enjoy :)


[CanonicalizeDoublyStridedOp] Add support for unit dimension folding with offsets

This case comes up when supporting larger L2 buffers, prefetching it here.
With this change, there are two cases being handled in foldUnitDimension:

If a dimension has size == 1 and offset == 0, the dimension can be
folded entirely.
If a dimension has size == 1 and offset != 0, it can be folded into
another dimension with the same stride if that exists.

Example for case 1:
offsets: [0, 0, 0], sizes: [32, 1, 8], strides: [32, 1024, 1]

will be transformed into:
offsets: [0, 0], sizes: [32, 8], strides: [32, 1]

Example for case 2:
offsets: [1, 0, 1, 0], sizes: [1, 32, 1, 8], strides: [1024, 32, 1024, 1]

will be transformed into:
offsets: [2, 0, 0], sizes: [1, 32, 8], strides: [1024, 32, 1]

Note that the dimensions are merged into the outermost one. Heuristically,
this works out best with other strided access pattern transformations, but
could be made an option in the future.

One question pertaining to the design choice: Why don't we choose to implement these as part of the ops' folder using hasFolder = 1 ?

A couple of reasons:

Originally, the idea was that you might not want to do these canonicalizations initially and perform transformations on the non-canonicalized form that is derived from the pack operations. This helps/is likely needed for example for the current logical objectFifo splitting transformations.
The canonicalization can be hardware-aware or not, leading to different results. While non-hardware aware folding is what you would typically expect in the op folder, we use hardware-aware canonicalization by default and this would mean that you have to look up the device model within the ops' folder themselves. This can be done, but I am not convinced that it's a good idea to make ops functions dependent on the device model.


Fix: Experemental filters AnyFilter working bug

Fixes #2942
instead of passing f.any filter as argument in newAnyFilter() it was passing f.all which was causing this issue
Proposed Changes

passing required arguments to create a proper Any filter.
put the test to check if any filter gets its format or not

/ok-to-test


Default selection / Programmatically changing selection

I have tried all possible ways to set the default value of select box (single). none worked.
I have tried:-
[active]="{id:'1' , text : 'foo' }"
[active]="[{id:'1' , text : 'foo' }]"

[data]="{id:'1' , text : 'foo' }"
[data]="[{id:'1' , text : 'foo' }]"

Is there a way to change selection Programmatically  ?
angular 2.0.0-rc.4
ng2-select 1.0.3

Same issue for me as well. There is no way to set the value of ng2-select programatically.

you can do something like this. Its not a clean solution but it works.
@ViewChild(SelectComponent) select: SelectComponent;

changeSelection() {
     if (this.select) {
          this.select.initData = [{id: 1, text: 'new value'}];
          this.select.ngOnInit();
     }
}

@littleStudent  does this work on version 1.0.3 ? because I think initdata property was removed. still , I will give it a try.

@littleStudent  confirmed. working for now.  :+1:

@littleStudent any idea on how to do it for multiple select boxes ?

maybe something like this? did not try it, but this way you should be able to get the reference for
multiple selects
<select #select1></select>
<select #select2></select>

@ViewChild('select1') select1: SelectComponent;
@ViewChild('select2') select2: SelectComponent;

this works but ! but first selectbox shows correct selected value , but second one is blank not even placeholder.

I also have this issue in v1.1.0 / Angular 2.0.0.
littleStudent's workaround fixed it for me (in case it's not obvious, I had to call changeSelection() from ngOnInit()).
Also, you have to assign value not initData
this.select.value = [{id: 1, text: 'new value'}];

Hopefully we can get a more robust solution soon.

@matthewhegarty   I tried everything , and now what works for me is :-
this.select.initData  will work only when component initializes.
After , that if you need to change selection then this works,
this.select.active= [{id: 1, text: 'new value'}];

hello, I solve this point to easy, in version 1.0.3 is initData in 1.1 is active
example:
<ng-select [initData]="initDataTeacher" [items]="teachers" placeholder="Teacher Name"  (selected)="setTeacher($event,classes)" >
in my component.ts
this.initDataTeacher = [{ 'id': 12, 'text': 'teacherName'}];

What's the current guidance on this with angular 2.1.2?

@littleStudent Your method saved my day. You are lifesaver. Thank you so much!!!!

Another method, that also works with selecting the placeholder:
<ng-select *ngIf="show" ng-model="model" ...>
...
//pass undefined to reset to placeholder
private setSelectTo(value) {
this.model = value;
this.show = false;
setTimeout(() => {
this.show = true
}, 0);
}

another way to clear is
@ViewChild(SelectComponent) select: SelectComponent;
select.writeValue(undefined)

My 5 cents. It was tough!
template
<ng-select name="dealers"
[active]="activeDealer"
[multiple]="false"
[items]="dealers"
(data)="refreshValue($event)"//schould be present in code
placeholder="dealers"
>

// delcarations
dealers : Array < any > = []
activeDealer:Array =[];
// somewhere in code .(async operation) The funny part is with _temp var.
// receiving AJAX response as responseData
let _temp = [];
responseData.result
.forEach((item, index) => {
_temp
.push({id: item.id, text: item.title})
});
this.dealers = _temp;
this.activeDealer=[this.dealers[1]]

Try this
this.items = this._selectService.mapObjectProperty([{_id:1,dealerName:'One',code:'O'},
    {_id:2,dealerName:'Two',code:'T'}, {_id:3,dealerName:'Three'}]);
//in service use the function as
  public mapObjectProperty(objectArray){
        let returnObject:any=[];
         _.forEach(objectArray, function (itemObj, index) {
             let mapObject ={id:'', text:''}
             _.forOwn(itemObj, function(value, key) {
               if(key == '_id'){
                  mapObject.id= value;
               }else if(key == 'dealerName' || key == 'branchName' || key == 'value'){
                  mapObject.text= value;
               }
             });
            returnObject.push(mapObject);
        });
        return returnObject;
    }


Radial Bar Chart not rendering correctly Firefox

Description
It seems the following works correctly in Chrome but not in Firefox. Having the chart div within a flex container
causes issues rendering. Specifically when the padding-left or padding-right values are set, the parent container is a flex container, and the child (chart) container doesn't have the flex property set
Steps to Reproduce

Create a Radial bar chart anywhere on the page. I used the exact one of the radial bar demo page
Create a flex container the contains our 'chart' container within it.
Set the padding-left and padding-right for the chart div to some values other than 0 (ex. 12px)
Open up the page using that chart and don't resize it. The bar chart will only render partially.

Expected Behavior
The Radial bar chart renders completly and the animation of the bar 'filling up' to the desired value
Actual Behavior
The Radial bar chart renders partially and most is cut off.
Screenshots

Reproduction Link
Setting the 'flex' property (the one thats commented out in the codepen) for the child (chart) seems to make the radial bar render correctly.
https://codepen.io/duecknoah/pen/BadmjNB

As tested on v3.54.1, this works fine now


Wrong detection of jQuery version

I use jQuery in version "1.11.0" and got error: "Tidy-Table requires jQuery 1.8.3 or greater."
if (!window.jQuery || (window.jQuery && window.jQuery.fn.jquery < '1.8.3')) {
	throw new Error('Tidy-Table requires jQuery 1.8.3 or greater.');
}

This will raise error for version of jQuery "1.10.*", "1.11.*"

This issue has been resolved in the latest release v2.0.5
Closing this ticket.


Add support of CMAKE_OSX_DEPLOYMENT_TARGET option

In order to compile object files adapted to older macos versions, the CMAKE_OSX_DEPLOYMENT_TARGET option is generally add to the CMakeLists.txt as below,
if(APPLE)
    set(CMAKE_OSX_DEPLOYMENT_TARGET "11.0")
endif()

But this only works in CMake projects. As the openssl uses the ./configure file, the -mmacosx-version-min flag is not actually passed to the ./configure command. I've tried to add the following part to openssl-cmake/CMakeLists.txt, and it works,
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 5343bb4..e7d60e1 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -185,6 +185,10 @@ if(OPENSSL_USE_STATIC_LIBS)
     list(APPEND OPENSSL_CONFIGURE_OPTIONS no-shared)
 endif()

+if(APPLE AND CMAKE_OSX_DEPLOYMENT_TARGET)
+    list(APPEND OPENSSL_CONFIGURE_OPTIONS -mmacosx-version-min=${CMAKE_OSX_DEPLOYMENT_TARGET})
+endif()
+
 list(SORT OPENSSL_CONFIGURE_OPTIONS)
 list(REMOVE_DUPLICATES OPENSSL_CONFIGURE_OPTIONS)
 list(PREPEND OPENSSL_CONFIGURE_OPTIONS ${OPENSSL_TARGET_PLATFORM})

I wonder if there is any more suitable implements could help supporting the CMAKE_OSX_DEPLOYMENT_TARGET option.

Thanks for suggestion.
But I won't add those options because they could be duplicated or conflict with other users' options.
And your modification is applied to iOS targets incorrectly since they need -mios-version-min or -mios-simulator-version-min.
If you really want to target macOS only, don't use the APPLE variable but CMAKE_SYSTEM_NAME.
if(CMAKE_SYSTEM_NAME STREQUAL "Darwin")
    set(CMAKE_OSX_DEPLOYMENT_TARGET 11.0)
    list(APPEND OPENSSL_CONFIGURE_OPTIONS -mmacosx-version-min=${CMAKE_OSX_DEPLOYMENT_TARGET})
endif()

if(CMAKE_SYSTEM_NAME STREQUAL "Darwin")
    list(APPEND OPENSSL_CONFIGURE_OPTIONS -mmacosx-version-min=${CMAKE_OSX_DEPLOYMENT_TARGET})
endif()

Thanks for your suggestions. I've tried to add these to the CMakeLists.txt of the top project, but if(CMAKE_SYSTEM_NAME STREQUAL "Darwin") cannot work in my macOS. After changed to if(APPLE), the list command can work.

Could you find how CMAKE_SYSTEM_NAME is actually defined on your system?

Could you find how CMAKE_SYSTEM_NAME is actually defined on your system?

$ cmake --system-information | grep CMAKE_SYSTEM_NAME
CMAKE_SYSTEM_NAME == "Darwin"
CMAKE_SYSTEM_NAME "Darwin"
PRESET_CMAKE_SYSTEM_NAME "FALSE"

But it's weird the var CMAKE_SYSTEM_NAME outputs "" in cmake config stage (cmake -B build).

I think the Xcode generator isn't selected. Could you try cmake -B build -G Xcode?

I think the Xcode generator isn't selected. Could you try cmake -B build -G Xcode?

Yes, Ninja is used bty default. The CMAKE_SYSTEM_NAME can be set correctly only if the Xcode generator is selected?

It could be empty before project() command. If CMAKE_SYSTEM_NAME is empty, it is set to CMAKE_HOST_SYSTEM_NAME after project(). I guess Xcode generator sets CMAKE_SYSTEM_NAME to Darwin.

It could be empty before project() command. If CMAKE_SYSTEM_NAME is empty, it is set to CMAKE_HOST_SYSTEM_NAME after project(). I guess Xcode generator sets CMAKE_SYSTEM_NAME to Darwin.

Yes! After project(), the CMAKE_SYSTEM_NAME is set even though Ninja is used. But the CMAKE doc ask to set CMAKE_OSX_DEPLOYMENT_TARGET prior to the first project(). How to solve it?

Through a cache variable. cmake -B build -DCMAKE_OSX_DEPLOYMENT_TARGET=11.0

Through a cache variable. cmake -B build -DCMAKE_OSX_DEPLOYMENT_TARGET=11.0

But the project needs to be compiled both in macOS and linux, if the CMAKE_OSX_DEPLOYMENT_TARGET is set by command line, it'll need to use different build scripts in different systems.

It should be different command for each target platform, but I think CMAKE_OSX_DEPLOYMENT_TARGET will not affect other platforms.

It should be different command for each target platform, but I think CMAKE_OSX_DEPLOYMENT_TARGET will not affect other platforms.

Yes for a pure CMake project, but -mmacosx-version-min=${CMAKE_OSX_DEPLOYMENT_TARGET} is manually added to the openssl C_FLAGS by detecting CMAKE_OSX_DEPLOYMENT_TARGET.

Isn't it okay if you append it under if(CMAKE_SYSTEM_NAME STREQUAL "Darwin")?

Isn't it okay if you append it under if(CMAKE_SYSTEM_NAME STREQUAL "Darwin")?

Great idea, thanks a lot!


Add safekeeper scrubber test

The test is very rudimentary, it only checks that before and after tenant deletion, we can run scan_metadata for the safekeeper node kind. Also, we don't actually expect any uploaded data, for that we don't have enough WAL (needs to create at least one S3-uploaded file, the scrubber doesn't recognize partial files yet).
The scan_metadata scrubber subcommand is extended to support either specifying a database connection string, which was previously the only way, and required a database to be present, or specifying the timeline information manually via json. This is ideal for testing scenarios because in those, the number of timelines is usually limited.

LGTM, @yliang412 might want to add this to the cron job if necessary?

I think we can do it if the checks are stable + do not emit false positive. As part of #8128, I only checked into pageserver's metadata scan code.


feat: account for secondary reserved namespace range

Closes https://github.com/celestiaorg/celestia-node/issues/2561
Blocked on https://github.com/celestiaorg/celestia-app/pull/2257

Codecov Report

Merging #2562 (c56d1de) into main (87e9500) will increase coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2562      +/-   ##
==========================================
+ Coverage   51.05%   51.23%   +0.17%
==========================================
  Files         158      158
  Lines       10398    10398
==========================================
+ Hits         5309     5327      +18
+ Misses       4622     4607      -15
+ Partials      467      464       -3

Files Changed
Coverage Δ

share/namespace.go
61.25% <100.00%> (+6.25%)
:arrow_up:

... and 4 files with indirect coverage changes


Supports max clients in server pool

The server pool has the max client connections config, but it never works.

[ ] Supports max clients in server pool

[ ]  Shrink the mbuf when exceeding the max numbers.


How to publish message with Headers?

How to publish message with Headers?
Like below:
amqp.Publishing{
Headers:     headers,
ContentType: "text/plain",
Body:        data,
},

And how to publish messages with retries???

Please see the Godoc: WithPublishOptionsHeaders
Retries are not a built-in functionality to the client - your application should handle them if you need them.

Please see the Godoc: WithPublishOptionsHeaders
Retries are not a built-in functionality to the client - your application should handle them if you need them.

I download latest version, and use WithConsumeOptionsBindingExchangeName does not work well.


How to do "Take Profit % from total volume"?

As the title said, in 3Commas there's such option:

I wonder how to implement this in strat_dca.py.
Thank you, stash86

Use minimal_roi


TypeError: needs a 'context' argument.

I am trying to run a test for a ember application using mocha-jsdom and chai. (Here is my test file http://pastebin.com/0rE8aai8)
Node version:  v0.10.35
Npm verson:  2.1.17
Machine:  OSX Yosemite (v10.10)
When I run $ mocha test.js it gives me the following error:
test
 1) "before all" hook

0 passing (1s)
1 failing

1) test "before all" hook:
   TypeError: needs a 'context' argument.
    at new Window (/Users/sagar/zlemma/new-design-zcloud/static/zcloud/node_modules/jsdom/lib/jsdom/browser/Window.js:39:26)
    at Object.exports.jsdom (/Users/sagar/zlemma/new-design-zcloud/static/zcloud/node_modules/jsdom/lib/jsdom.js:46:16)
    at processHTML (/Users/sagar/zlemma/new-design-zcloud/static/zcloud/node_modules/jsdom/lib/jsdom.js:220:24)
    at Object.exports.env.exports.jsdom.env (/Users/sagar/zlemma/new-design-zcloud/static/zcloud/node_modules/jsdom/lib/jsdom.js:122:5)
    at Context.<anonymous> (/Users/sagar/zlemma/new-design-zcloud/static/zcloud/node_modules/mocha-jsdom/index.js:43:22)
    at Hook.Runnable.run (/usr/local/lib/node_modules/mocha/lib/runnable.js:218:15)
    at next (/usr/local/lib/node_modules/mocha/lib/runner.js:259:10)
    at Object._onImmediate (/usr/local/lib/node_modules/mocha/lib/runner.js:276:5)
    at processImmediate [as _immediateCallback] (timers.js:354:15)

And this works on my colleague's machine (same configuration) but doesn't work on my machine.

Try deleting node_modules and reinstalling your modules (npm install).

Nope. Still getting the same error.
Package json file http://pastebin.com/qVEYE0xE

Getting same issue.

I edited mocha-jsdom's package.json and replaced "jsdom": "^1.0.0-pre.7", with "jsdom": "^3.1.2",.
This fixed this issue, but I'm not sure why mocha-jsdom was pinned to such an old version of jsdom, so maybe there's side effects that I'm unaware of.

Yes, it worked for me too.

This is rather strange: mocha-jsdom has its jsdom dependency marked as *, so it'll use whatever version is the latest. The version 1.0.0 was a devDependency only to be installed when actually working on mocha-jsdom.
Nonetheless, it's been bumped (thanks @nacyon!) and will be available as 0.3.0 pretty soon.


Chrome console Warning: Failed prop type: The prop closed is marked as required in Section, but its value is undefined"

Chrome console Warning: Failed prop type: The prop closed is marked as required in Section, but its value is undefined". I don't get Flow errors which is correct BUT why is the Chrome browser complaining?
e.g. "closed" is my State not Props. Why is this a warning?
type PropsType = {|
title: TitleType,
children: React.Node,
|};
type StateType = {
closed: boolean,
};
export default class Section extends React.PureComponent<PropsType, StateType> {
constructor(props: PropsType) {
super(props);
this.state =  {closed: false};

}
}
e.g. However, if set closed as an option Chrome won't complain.
type PropsType = {|
title: TitleType,
children: React.Node,
|};
type StateType = {
closed?: boolean,
};
export default class Section extends React.PureComponent<PropsType, StateType> {
constructor(props: PropsType) {
super(props);
this.state =  {closed: false};

}
}

[https://github.com/facebook/flow/issues/4707] - similar

Similar Issue


utility: Add performance annotation library

Description:
This is broken out from #2615 as it likely deserves its own review.  This provides a mechanism to annotate and measure the costs of functions that are data-dependent, e.g. regexes.
This is step 3a-ish, in the plan to improve startup performance. This is another step toward addressing #2373
Adds perf annotation library that can be used to instrument code but disappear completely from generated code unless enabled with bazel --define=perf_annotation=enabled
Produces tables in this format.
Duration(us)  # Calls  Mean(ns)  StdDev(ns)  Min(ns)  Max(ns)  Category  Description
        4600        4   1150000      129099  1000000  1300000     alpha            1
         200        1    200000         nan   200000   200000     gamma            2
          87        3     29000        1000    28000    30000      beta            3

Instrumentation can be coded into the system but is turned off via compiler macros so there is zero cost in production.
Risk Level: Low -- new utility library not used by anything yet.
Release Notes: N/A

@mrice32 had several comments in #2615 which are applicable here, and repeated here:

Is there a reason we're not using monotonic clocks/times here and throughout since we're just computing durations?

Add doxygen annotation for params to PerfAnnotationContext::record() and elsewhere

Is there any reason we can't just subclass the ThreadSafeSingleton to keep this singleton implementation consistent with others like it across Envoy?

There don't appear to be any tests for PerfOperation. Can we add a few? I think just using a second timer to ensure that the recorded time will be greater than some value will suffice.

RE @mrice32 comments:

monotonic clocks: done
doxygen annotation: done
ThreadSafeSingleton; does not appear to be used consistently now.  I'm actually not sure I want to switch to this because the explicit instantiation used currently works well with the tests, and also I'm thinking that if we want to use this for performance critical code, we might want to actually put it in thread-local storage and have a way to aggregate across the silos only when someone interactively requests output.
PerfOperation (and all this new code) has 100% coverage via PerfAnnotationTest.testMacros (which is the recommended mechanism for code to do annotations).   Do you think it's important to also test PerfOperation explicitly (as opposed to via PERF_OPERATION)?

You're probably right that it's not used in all cases, but I don't think that's necessarily reason to not use it as it is the most common and supported implementation.  In both cases there will be leakage across tests that is cleaned up with the CLEAR macro - IIUC, the only difference would be calling a different getter method?  (Feel free to correct me if I'm wrong)  As for thread-local aggregation, the singleton implementation we use here will be overhauled regardless in that case, so I see no reason one would be preferred over the other.

This is ultimately up to you as it's a relatively small detail, but I think it's generally good practice to try to reuse common libraries, especially when they are designed for your use-case, as they increase functional predictability and general readability across the codebase.  The argument isn't necessarily that a special implementation of a singleton will be wrong, but more that it might differ in subtle ways that ends up violating others' assumptions about the object - especially with something like singletons.  These sorts of small misunderstandings can lead to much larger bugs.

Nope, you're totally right.  I must've just glanced right past that test.  Thanks for pointing that out.

@mrice32 per f2f, one reason I'd prefer not to use ThreadSafeSingleton is that in the current state, the object remains completely uninstantiated unless enabled at compile-time. I actually was torn over whether to use a singleton at all for this, and went this way because I thought it would be easier to instrument code with zero production effect. In the future it may be worth putting the object into an appropriate context object so it's available where needed -- maybe using one of the thread-local objects.

I think all comments are addressed; PTAL.  One that I addressed really with just a comment was the possibility of taking another dependency for a library that's got some overlap with this:  https://github.com/LLNL/Caliper.  I did have a quick look at that and I agree there's a lot of overlap but I don't think the output was what I was looking for in #2615.  Still it might be nice to think about utilizing that one as well.

ping :) @dnoe


Autoload JS files from /scripts/autoload/ doesn't work with import "./autoload/**/*" from main.js

Submit a feature request or bug report

[ ] I've read the guidelines for Contributing to Roots Projects
[ ] This is a feature request
[ ] This is a bug report
[ ] This request isn't a duplicate of an existing issue
[ ] I've read the docs and NPM Debugging Guidelines post and followed them (if applicable)
[ ] This is not a personal support request that should be posted on the Roots Discourse forums

What is the current behavior?
Files from resources/scripts/autoload/ are not being loaded by the import "./autoload/**/*" from main.js
I'm receiving an error on a fresh install. Nothing has been changed.
Here's the error I'm receiving in Terminal:
ERROR  Failed to compile with 1 errors                                                                                8:35:29 PM
error  in ./resources/assets/scripts/main.js
Module build failed: Module failed because of a eslint error.
/resources/assets/scripts/main.js
5:43  error  Parsing error: Unterminated string constant
✖ 1 problem (1 error, 0 warnings)
What is the expected or desired behavior?
All files from resources/scripts/autoload/ should be loaded and for yarn build to be successful

Bug report

Please provide steps to reproduce, including full log output:
import "./autoload/**/*" frommain.js seems not to load files under /autoload/.
Please describe your local environment:
WordPress version: 4.9.6
OS: Ubuntu (64 bit) on Mac OS High Sierra Version 10.13.5
NPM/Node version: Node: v10.6.0
Where did the bug happen? Development or remote servers?
dev local
Is there a related Discourse thread or were any utilized (please link them)?
https://github.com/roots/sage/issues/1956

Make sure you save resources/assets/scripts/main.js after introducing/removing files from autoload directory

NPM/Node version: Node: v10.6.0

Use a lower version of Node, Sage currently is not compatible with Node 10.x (#2067).
I would recommend using nvm to switch between Node versions — as mentioned in the NPM Debugging Guidelines Post in the issue checklist that you did not fill out. Use NVM to install and use the current LTS version of Node (nvm install lts/* then nvm use lts/*).
I received a similar error, however, in my case it was because I broke the Webpack config file.

This hasn’t seen any action in a while. I’m closing it for now but we can re-open if the issue is still current.


Exception with  CSpect <IP_ADDRESS>

I've tried the plugin with the current version of CSpect (<IP_ADDRESS>) and I got:
************************* #CSpect logfile 17-12-23 19:04:05 *************************
Error in DLL: E:\SpectrumNext\Tools\CSpect\DeZogPlugin.dll
Exception: Unable to load one or more of the requested types. Retrieve the LoaderExceptions property for more information.
Exception: Method 'OSTick' in type 'DeZogPlugin.Main' from assembly 'DeZogPlugin, Version=<IP_ADDRESS>79, Culture=neutral, PublicKeyToken=null' does not have an implementation.

I've tried to build from the source and have opened the Plugin.dll that is provided with CSpect.
I get:
`#region Assembly Plugin, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
// D:\Users\ediazfer\Documents\Projects\DeZogPlugin\Plugin.dll
// Decompiled with ICSharpCode.Decompiler <IP_ADDRESS>43
#endregion
using System.Collections.Generic;
namespace Plugin
{
public interface iPlugin
{
void Tick();
    void OSTick();

    bool Write(eAccess _type, int _port, int _id, byte _value);

    byte Read(eAccess _type, int _port, int _id, out bool _isvalid);

    bool KeyPressed(int _id);

    List<sIO> Init(iCSpect _CSpect);

    void Reset();

    void Quit();
}

}
#if false // Decompilation log
'10' items in cache
Resolve: 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'
Found single assembly: 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'
Load from: 'C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework.NETFramework\v4.5.2\mscorlib.dll'
#endif`
So OSTick is present.
Rebuiding from the source (I've used VS 2022) fixes the issue.

CSpect is released nowadays with the Dezog plugin included.
So there isn't (shouldn't be) any need to compile the dezog plugin by yourself.
I tested CSpect <IP_ADDRESS> on macOS and on windows.
For me it runs (CSpect and plugin) without any problems.


Bug: Tasks are not rendered in a note when Obsidian starts

I took a quick look at the Issues here and it doesn't seem like anyone reported this issue, so please feel free to tell me if I am doing anything wrong.
Info
I am querying a list of tasks from the past 14 days in a note which I pin in the sidebar view. However, I see an error in the preview mode of that note instead. If I navigate from that note in the same pane to another note and then come back, then the dataview works as expected. This issue occurs even in a regular note that isn't pinned, but I'd include the steps relevant to me here.
Steps to reproduce
Given:

Obsidian v0.11.10 (installer v0.6.4)
Dataview plugin is installed.
And I query a list of tasks in a note using:

task
FROM ""
WHERE file.mtime >= date(today) - dur(14 day)
SORT file.mtime desc

When:

I start Obsidian.

Expectation:

I should see my list of tasks in the pinned note.

Reality:

I see this error in the preview mode.

Dataview: TypeError: Cannot read property 'get' of null

TypeError: Cannot read property 'get' of null
    at executeTask (eval at <anonymous> (app://obsidian.md/app.js:1:1149631), <anonymous>:10328:27)
    at eval (eval at <anonymous> (app://obsidian.md/app.js:1:1149631), <anonymous>:10538:47)
    at tryOrPropogate (eval at <anonymous> (app://obsidian.md/app.js:1:1149631), <anonymous>:10570:16)
    at DataviewTaskRenderer.eval (eval at <anonymous> (app://obsidian.md/app.js:1:1149631), <anonymous>:10538:26)
    at Generator.next (<anonymous>)
    at eval (eval at <anonymous> (app://obsidian.md/app.js:1:1149631), <anonymous>:26:71)
    at new Promise (<anonymous>)
    at __awaiter (eval at <anonymous> (app://obsidian.md/app.js:1:1149631), <anonymous>:22:12)
    at DataviewTaskRenderer.onload (eval at <anonymous> (app://obsidian.md/app.js:1:1149631), <anonymous>:10537:16)
    at DataviewTaskRenderer.e.load (app://obsidian.md/app.js:1:363101)

It works fine if I unpin the note, navigate to another note in that pane and go back
Things I tried
I tried disabling all other plugins except for Dataview, and the issue persists. I have several plugins installed, so I am hesitant to uninsatll all of them, but I can make an effort if it helps resolve the issue.

Thanks for the very detailed notes - I've seen this error before. It's due to the query engine trying to execute a query before the indices are ready (which store per-file metadata and need to be generated at vault startup). I tried fixing the issue a while back and got it to mostly go away, though it seems it stubbornly returns - likely a race condition.
Will try to improve the index initialization.

I've changed around initialization a few more times - @minhthanh3145 does this work on the latest version?

@blacksmithgu Confirmed that it's working on start-up in Obsidian 0.12.1 with the latest Dataview plugin. Great work!

Excellent! Hopefully that bug is finally squashed.


Form Components - Width Issue

Our form components need to be adjsuted to scale when the width of the host element is increased.
Example:
kyn-text-input {
  width: 100%;
}

This does not work because the internal input element is not 100% width or display block, and the input wrapper is set to display: inline-flex instead of flex.
We need to examine each form component to address this issue.

:tada: This issue has been resolved in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


OPENAI_API_KEY required when not using OpenAI since update

Here's how I have been instantiating a vector store using fromDocuments for ~6 months without issues:
    const index = await VectorStoreIndex.fromDocuments(
      [document],
      {
        serviceContext: {
          llm: new Ollama({
            model: 'mistral'
          }),
          embedModel: new OllamaEmbedding({
            model: 'mistral'
          }),
          promptHelper: new PromptHelper(),
          nodeParser: new SimpleNodeParser(),
          callbackManager: new CallbackManager()
        }
      }
    );

But when I updated llamaindexTS recently, I now get the dreaded:
Error: Set OpenAI Key in OPENAI_API_KEY env variable
    at new OpenAISession

Why?
https://github.com/run-llama/LlamaIndexTS/blame/f7b4e94231626f2dfb0f2b4edc2cb684b9e8b974/packages/llamaindex/src/internal/settings/EmbedModel.ts#L10
This code compiles to the following JavaScript:
let globalEmbeddedModel = null;

export function getEmbeddedModel() {
    if (globalEmbeddedModel === null) {
        globalEmbeddedModel = new OpenAIEmbedding();
    }
...

globalEmbeddedModel is always null so it always gets set to OpenAIEmbedding even if you aren't using OpenAI and even if you provide a service context.
I wanted to open a PR to fix it myself, but there were some architectural changes in that blamed PR that I don't quite understand, namely - why is a StorageContext required when I am not using storage?
The only reason OpenAI is being defaulted here is that storageContextFromDefaults is being called, but only because I don't provide one (never have). I have always used the in-memory vector store, and never had to provide a storage context.
@logan-markewich @marcusschiesser Any help would be highly appreciated thank you!

This is a bug since we are depracing the context service in the future and use Settings.xxx instead. I think context is missing when passing into the deep class or method

Could please provide more stack trace so I know where it's missing?

Could please provide more stack trace so I know where it's missing?

Here's the trace after calling the above fromDocuments:
Error: Set OpenAI Key in OPENAI_API_KEY env variable
    at new OpenAISession (/home/benny/Documents/Projects/ragdoll/node_modules/@llamaindex/openai/dist/index.cjs:512:23)
    at getOpenAISession (/home/benny/Documents/Projects/ragdoll/node_modules/@llamaindex/openai/dist/index.cjs:534:19)
    at new OpenAIEmbedding (/home/benny/Documents/Projects/ragdoll/node_modules/@llamaindex/openai/dist/index.cjs:986:45)
    at getEmbeddedModel (/home/benny/Documents/Projects/ragdoll/node_modules/llamaindex/dist/cjs/internal/settings/EmbedModel.js:28:31)
    at new VectorStoreBase (/home/benny/Documents/Projects/ragdoll/node_modules/llamaindex/dist/cjs/vector-store/types.js:62:74)
    at new SimpleVectorStore (/home/benny/Documents/Projects/ragdoll/node_modules/llamaindex/dist/cjs/vector-store/SimpleVectorStore.js:98:9)
    at storageContextFromDefaults (/home/benny/Documents/Projects/ragdoll/node_modules/llamaindex/dist/cjs/storage/StorageContext.js:24:70)
    at VectorStoreIndex.fromDocuments (/home/benny/Documents/Projects/ragdoll/node_modules/llamaindex/dist/cjs/indices/vectorStore/index.js:122:107)

seems like it tries to storageContextFromDefaults when one is not passed


Hotfixed security hole. Phew

Why are builds taking so long :thinking: ?

I think travis broke...

fixed #184


Trawl reports the last IPv4 as the address of an interface, not the first

If you have an interface with multiple IP addresses attached to it, trawl reports the last IPv4 address of the interface (and probably the last IPv6 interface). Traditionally one would report the first IP if you're only going to report one, because that is usually the primary IP address.
$ ip -4 addr list eno1
2: eno1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq state UP group default qlen 1000
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global eno1
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global secondary eno1:1
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global secondary eno1:2
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global secondary eno1:3
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global secondary eno1:4
       valid_lft forever preferred_lft forever
$ trawl -f eno1
eno1  <IP_ADDRESS>  <IP_ADDRESS>  <IP_ADDRESS>/24  1500  d4:ae:52:cb:c2:2c  fe80::d6ae:52ff:fecb:c22c/64

I think that the broad fix for this would be to change extractAddrs() to return a list of IPv4 and IPv6 addresses, instead of trying to return just one of each. The current New() could then use the first one of each.

Ah interesting, thanks @siebenmann for bringing this to my attention. I'll get this fixed, may not be until next week though. Unless you fancy submitting a PR?

I'll see if I can take a shot at this and successfully navigate the PR process to get the proposed change to you.
I just noticed that this applies to IPv6 addresses too, basically as expected. This is a bit more annoying in the IPv6 case, since every interface has a link-local IPv6 fe80::/10 address attached to it, which comes last on my Linux machine and so always overwrites any global IPv6 address in the report.

No pressure @siebenmann don't feel you have to fix this yourself, I'm happy to get to it in due course. I'm also happy to help with the PR process if you get stuck.

This is a bit more annoying in the IPv6 case

Yeah, for sure. I'll prioritise getting this fixed. Thanks again :+1:

closed by https://github.com/robphoenix/trawl/commit/432e7a2b4527cd8d5e423fbf0ab65c9a5d7ce399


Remove moot version property from bower.json

Per bower/bower.json-spec@a325da3

Per the contrib doc, please make your PRs to the development branch. PRs to the master will be closed.
Also, this code block will have to be removed
https://github.com/Eonasdan/bootstrap-datetimepicker/blob/master/tasks/bump_version.js#L43

Whatever

Also their maintainer says they probably won't ever use it: http://stackoverflow.com/questions/24844901/bowers-bower-json-file-version-property


Missing functionality

The behavior shown in the README doesn't appear possible in the current implementation. Most of the ability to customize has been closed off.
e.g. I cannot customize layers because because neither ConfettiLayer nor var confettiLayers are public/open. Was this intentionally removed or is this library simply incomplete?
In any case - is this library still "alive" enough that a PR adding the functionality would be approved?

The behavior shown in the README doesn't appear possible in the current implementation.

What behavior specifically?

Most of the ability to customize has been closed off. e.g. I cannot customize layers because because neither ConfettiLayer nor var confettiLayers are public/open.

The API was intentionally designed to support certain kinds of configuration, but not everything. What functionality are you looking for that isn't currently supported?

The behavior shown in the README doesn't appear possible in the current implementation.

What behavior specifically?

Most of the ability to customize has been closed off. e.g. I cannot customize layers because because neither ConfettiLayer nor var confettiLayers are public/open.

The API was intentionally designed to support certain kinds of configuration, but not everything. What functionality are you looking for that isn't currently supported?

Oh! My apologies. I was installing ConfettiView which appears to be another pod entirely. I double checked the README and noticed that this pod is actually ConfettiView-NSHipster.
So, in short, I installed the wrong pod.

Oh! My apologies. I was installing ConfettiView which appears to be another pod entirely. I double checked the README and noticed that this pod is actually ConfettiView-NSHipster.
So, in short, I installed the wrong pod.


Add backend support for 24-word seed phrases. Not enabled in UI

Offer support for implementing #3447.
No functional change introduced but:

Updated TrustWalletCore
Backend can handle (store, read, sign with) wallets created with 24-word seed phrases
Can import 24-word seed phrases, but the UI doesn't enable this yet
Added tests for wallets using 12-word and 24-word seed phrases

@oa-s this is a delicated part of the system; would you help to test this?
At least this:

Test signing with existing wallets that were imported prior to this PR, that are 12-word seed phrases
Create new wallet (12-words)
Import private key-based wallet
Import 12-words seed phrase
Test signing

@oa-s this is a delicated part of the system; would you help to test this?
yep, sure

@oa-s thanks. Let me know when it's ready to merge (or you can merge it)

@hboon these steps works as axpected:

Test signing with existing wallets that were imported prior to this PR, that are 12-word seed phrases
Create new wallet (12-words)
Import private key-based wallet
Import 12-words seed phrase
Test signing

Only thing has been found that For some reasons wallet backup doesn't work on simulator but i guess it doesn't related to these changes

@oa-s for the backup that doesn’t work, can you give an example? And is the import or creation done before this PR or after? And which type of wallet?

@oa-s for the backup that doesn’t work, can you give an example? And is the import or creation done before this PR or after? And which type of wallet?

Created a new issue https://github.com/AlphaWallet/alpha-wallet-ios/issues/3609

@oa-s for the backup that doesn’t work, can you give an example? And is the import or creation done before this PR or after? And which type of wallet?

Created a new issue #3609

Thanks! Might not be a problem but you are right, it's a separate issue from this PR.


Add interaction red-x clicks to histogram

From using the plugin I noticed it currently doesn't log/display red-x clicks to interact with objects. For example: clicking on a door, mining rock, or an NPC that isn't attackable.
This would be useful since many times your movements are based on red clicks (like when you click a door to run there) and in particular would be useful for users learning red-x PVM methods like 6:0 bandos or red-x ba-ba.

The trouble is that the timings for certain interactions might vary depending on what they are, ie. clicking a door might be handled sooner in the server update than mining a rock would (I don't know if they're different timings without testing it, that's just a possible example). I wouldn't want any general "all red-x interactions" display because it'd probably be wrong for some cases. It might be the case that all red-x timings are the same, but I wouldn't want to assume that.
What I think will be the best compromise would be to add a couple (5?) customizable options that can be setup for whatever interaction the player wants. This would mean the player would have to figure out the timings themselves, but at least it can support whatever the player wants.
I'm gonna get to work on that / start fiddling with it.

Just to keep this updated, I've implemented the changes and put them up for review to go into the plugin hub. Cheers!


ERROR "playPublish" is not a valid command!

Screenclip of error:

bubblewrap/cli/README.md give this information about play and playPublish:

Why this error?
I am using windows and the latest version of everything.
Do I need to install core for this command?
Source:
I was also reading about this command in #621

Hey! I've just created a PR resolving this one - hopefully it will get merged soon.
And until it does - in my repo I'm using pnpm patch to uncomment needed lines by myself. You can do the same on your side natively in pnpm/yarn or  if you need npm then you can try using this package.


Update player_data.mdx

Added the missing info the playerdata docs.
Some info is missing and I dont want to look for it, but this is way better than before

Have you seen https://docs.qbox.re/resources/qbx_core/types/player_entity? Maintaining the same info in two places is not ideal.

The exact types for PlayerData are defined here:
https://github.com/Qbox-project/qbx_core/blob/945dbd0f922aaa972374d2c6ef8b3ae3cf245a46/types.lua#L38
Some data is missing but most of it belongs to PlayerEntity and should be kept that way.

Oh thats where it is. I think my descriptions of tables would be better then

Oh thats where it is. I think my descriptions of tables would be better then

Change the PR to put the descriptions in the other file then where these tables are already located.


Remove duplicate package management code

Here is the payoff from adding the package manager classes!
http://tracker.ceph.com/issues/12644

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-deploy-pull-requests/1742/
Test PASSed.


added quizeo play

I have added the Quizeo-play
Fixes #115

@vasantisuthar When you take care of the review comments and questions, please resolve merge conflicts too.
After that can we do a quick call to finish the review? It is a bigger play, so just want to clarify some doubts before approving. It will be faster to merge. Need around 10-15 mins time. Let me know when based on your availability.

@vasantisuthar When you take care of the review comments and questions, please resolve merge conflicts too.
After that can we do a quick call to finish the review? It is a bigger play, so just want to clarify some doubts before approving. It will be faster to merge. Need around 10-15 mins time. Let me know when based on your availability.

Sure, can we have a meet today at 8:00 pm?

@vasantisuthar When you take care of the review comments and questions, please resolve merge conflicts too.
After that can we do a quick call to finish the review? It is a bigger play, so just want to clarify some doubts before approving. It will be faster to merge. Need around 10-15 mins time. Let me know when based on your availability.

@atapas tomorrow at 3:00 pm?

@atapas kindly review the changes

@vasantisuthar I picked it up for review. Please resolve the conflicts carefully.

@vasantisuthar Are you still making changes?

@vasantisuthar Are you still making changes?

No, its done now.It is complete, it was just a small change to remove the logger.

@vasantisuthar most of it looks good to me. I've noticed some impact on other plays due to conflicting styles. Could you please test it yourself using the preview URL?
To be at the safer side, better to add a unique prefix to all your styles defined in your play's CSS files.

@vasantisuthar most of it looks good to me. I've noticed some impact on other plays due to conflicting styles. Could you please test it yourself using the preview URL?
To be at the safer side, better to add a unique prefix to all your styles defined in your play's CSS files.

Let's plan to merge the beast this week 😁


GH-34472: [Go][FlightRPC] Drain result of DoAction in Flight SQL client

Rationale for this change
Unless we drain the result of DoAction, we can't be sure that the RPC actually completed.
What changes are included in this PR?
Always drain the result of DoAction.
Are these changes tested?
A new test was added and existing tests were modified.
Are there any user-facing changes?
This fixes potential nondeterministic behavior that users may have previously observed.

Closes: #34472

Cleaned things up:

CloseSend() needs to be called before Recv() always
Mocks were made more reliable via Once()
Addressed comments

Whoops, thanks.

I'll merge this now, CI failures are from #34488


Add Event Publisher in Purchase Header - OnBeforeAssistEdit()

Hi there,
we need a new Event Publisher OnBeforeAssistEdit() in Purchase Header:
    begin
        // NEW++
        Handled := false;
        OnBeforeAssistEdit(SalesHeader, OldSalesHeader);
        if Handled then
            exit;
        // NEW--
        GetPurchSetup;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeAssistEdit(var PurchHeader: Record "Purchase Header"; OldPurchHeader: Record "Purchase Header"; var IsHandled: Boolean)
    begin
    end;

Thanks,
Carsten

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Label Count with Multiple Targets

The label count for each entity should still be functional when there are multiple targets.
>>> label_times.count

             count
customer_id
1                4
2                4
3                4
4                4
5                4

We can now select an individual target before making the label count.


[Coverity CID :186049] Memory - corruptions in /samples/subsys/mgmt/mcumgr/smp_svr/src/main.c

Static code scan issues seen in File: /samples/subsys/mgmt/mcumgr/smp_svr/src/main.c
Category: Memory - corruptions
Function: main
Component: Samples
CID: 186049
Please fix or provide comments to square it off in coverity in the link: https://scan9.coverity.com/reports.htm#v32951/p12996

@nvlsianpu can you please take a look at this one?

sure

issue is false positive - memory corruption doesn't happen - just function for clearing the statistic data (void stats_reset(struct stats_hdr *hdr)) doesn't know the peculiar structure of  statistic group (and can not know as the function is generic).

Closing as false positive.  Marked the issue in Coverity as such as well.


Upgrade sbt from 0.13.2 to 0.13.7

The version of Scala is fixed for a specific sbt release and cannot be changed. For sbt 0.13.7, this version is Scala 2.10.4., and for sbt 0.13.2, this version is Scala 2.10.3. See sbt’s Scala version
In this project the file BuildSettings.scala has specified the Scala version to 2.10.4, but in build.properties the sbt's version is still 0.13.2 which is outdated, so when you run sbt assembly to compile this project it will still download scala-compiler-2.10.3.jar and use Scala 2.10.3 to compile the project. To enable compiling with Scala 2.10.4, we need to update the version of sbt in  the file build.properties.

Are you confusing the version of Scala used to compile the /project sub-folder (i.e. the SBT build definition), with the version used to compile the project itself?


Adding the Koola-Koola Variation:

Adding the Koola-Koola Variation:
https://en.wikipedia.org/wiki/List_of_chess_openings#A00–A09
https://www.chess.com/openings/Kadas-Opening-Koola-Koola-Variation
(There is this commit "Merge branch 'master' of https://github.com/Malorra/chess-openings", i don't know what it does) and why does it say that it is not sorted correctly? Kádas should be before Koola

Thanks! Will deal with the ordering of diacritics.


Versioning

The data structure of OSRM changes in time and the current osrmnet only works with v5.3. Is it possible to use the version of the OSRM data file and then use the correct libosrm related files?
Or, maybe easier and better, make subprojects within osrmnet per compiled osrm version (with included osrm's compiled exe/dll/lua). Kind of archive with all necessary files within.

Yes, I think at the minimum include all the tools (exe, lua profile).  This way user don't have to worry about building osrm version.


Dev Environment

What is the recommended development environment for this repo?
I kept running into weird version conflicts from either Python2/3 or Tensorflow1/2. A specification and a docker image would be much appreciated.

I made a docker environment: https://hub.docker.com/repository/docker/zzj0402/natural-questions-environment
Feel free.


DID Method Specifications?

Would it be useful to optionally designate which DID methods are supported by the participants?

do issuers and holders and verifiers each need specification of supported DID methods? Should they all be specified with only set of allowed DID methods?

example: issuers and verifiers use did:web or did:indy, holders use did:peer.


Make secrets optional when updating application

Sometimes you want to update an existing application but don't have the original secret values. Currently, an update that requires all manifests uses the PUT verb. Could the UPDATE verb be used to represent an update to certain parts of an application?
To specify that the secrets be removed using the UPDATEverb , provide a blank secrets file. Then all existing secrets would be removed and no new secrets would be created.
To specify that the secrets should remain unchanged up all other areas (pipeline, configuration, instance) be updated, don't provide any secrets file.
With this, you could create a separation of role. Where the platform operator creates the tenant, a security person "inits" an example application with all the needed secrets, and a developer updates with the intended function. The security person could provide a schema of secrets, for the developer to consume (but never see the actual values).

Already done


Automator: update ztunnel@master in istio/istio@master

Generated by Automator - 2024-11-19T23:36:20+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
2d80a9bea5c7d76586adf341113e9f0dbd786ce2
link
true
/test integ-distroless

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.


[Feature Request]: Ability to sync roles in single transaction

Add the ability to sync roles(assignAllRoles, forbidAllRoles together) in single transaction so that if anything goes wrong it will fallback to its original state

Hi @mdsadique-inam,
Thanks for the request, it will be a nice addition, will add

Hi @holoyan 👋
Also it will be great to have an optional parameter to pass the transaction instance like TransactionContract making it a part of different functions

Hi @mdsadique-inam
Transaction support added


Android - Stuck notification

Hi,
Some of my users reported that sometimes the notification cannot be removed from the notification center while nothing is playing. The only way to remove it is by killing the app. I've also experienced this on a Pixel 4 with Android 10.
I've updated the app to display the close button on the notification but it has no effect when this happens, otherwise the button works.
I don't have much information.
Do you have any idea what could do this?
Thanks

Make sure you have dismissable : true set when you create the controls, then the user can just swipe the notification away any time they want.

Otherwise the notification should be destroyed when the app is killed.


the setting with terminal doesn't work

VSCode Version: 1.37.1
OS Version: win10 1903

i set the terminal is  integrated,but it always open terminal with new windows ,just like this, what i can do to repair it .

Does this issue occur when all extensions are disabled?: No
i disable recent extension when this thing occur, but it doesn't work

no like it ,i want open a integrated terminal ,but vscode open a external terminal. terminal can normal work，the setting also is integrated。

/duplicate https://github.com/microsoft/vscode/issues/73790, you need to disable legacy console mode in cmd.exe/conhost's properties.


Incorporate corpus callosum to connect hemispheres.

Some brains seem to have large longitudinal fissures, which can cause the hemispheres to print separately.

I've started to address this in 865c907. Still need to fix a few things and then check the output.

The corpus callosum seems like it causes problems with the final stl file. Most of the brain looks really pixelated after adding CC.

Corpus callosum dropped for now. Unless someone finds a way to make it work, I'm going to close this issue.


Problem with returning data to view

I can show the popup where you enter the email and password and the text of the button changes, but it does not return the data to the component to display in the view or console. That could be happening?
I also put some messages in console in the function onGoogleSignInSuccess but it does not return them to me ...
Bye!

Could you upload an example to see how you implement it?

I have facing same issue please help me .
Object {error: "idpiframe_initialization_failed", details: "Not a valid origin for the client: http://localhos…itelist this origin for your project's client ID."}
details
:
"Not a valid origin for the client: http://localhost:4200 has not been whitelisted for client ID myclientId.apps.googleusercontent.com. Please go to https://console.developers.google.com/ and whitelist this origin for your project's client ID."
error
:
"idpiframe_initialization_failed"

this is component file code but not getting any response.
onGoogleSignInSuccess(event: GoogleSignInSuccess)
{
console.log(event);
let googleUser: gapi.auth2.GoogleUser = event.googleUser;
console.log(googleUser);
	    let id: string = googleUser.getId();
	    let profile: gapi.auth2.BasicProfile = googleUser.getBasicProfile();
	    console.log('ID: ' +
	      profile
	        .getId()); // Do not send to your backend! Use an ID token instead.
	    console.log('Name: ' + profile.getName());
}

Hi! I will look into it this weekend, bare with me!


Support semver version ranges for released decorator (and manifests)

Motivation
In Node.js we have multiple release lines (ie: v3.x, v4.x, etc...). Historically we would put the latest version in the system-tests manifest, without specifying the potentially backported versions.
Example:
Feature Test_Blocking_request_cookies has been released in v3.37.0 and v4.16.0.
Current manifest:
Test_Blocking_request_cookies: v4.16.0

Ideally we'd want to specify the full compatibility range like this:
Test_Blocking_request_cookies: ^3.37.0 || >=4.16.0

Where ^3.37.0 is equivalent to >=3.37.0 <4.0.0
Changes
Yeah basically you can do it now, thanks to the pip package semantic_version and yours truly.
I kept it compatible with the current vX.X.X syntax, so if the version declaration starts with v, it will use the classic "above or equal", else it will consider it as a semver spec.
TODO

[ ] Update documentation
[ ] Update manifest JSON Schema
[ ] Add tests ?

Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] Relevant label (run-parametric-scenario, run-profiling-scenario...) are presents
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present
[ ] To R&P team: locally build and push the image to hub.docker.com

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team
[ ] Once merged, add (or remove) it in system-test-dasboard nightly

I tested all the nominal cases manually and it works perfectly. But I had enough python today to write unit tests for this, please someone from the @DataDog/system-tests-core team take over the PR.

The 2 python parametric jobs failing are also failing in master :shrug:

@DataDog/system-tests-core

I am not sure we can easily reconcile this SemVer spec with the manifest schema in /manifests/parser/schema.json. Maybe the manifest validation should be done python side instead ?

./format.sh to fis the lint job


Feature: URL Aliases

I like to have pretty URLs enabled, but those can be taxing to say for example at the end of the podcast. I'd prefer to be able to say go to my-site.com/2 and have that be able to redirect to the actual post location.
To that end, I think it would be nice if you could put in sculpin_kernel a "redirects:" key that would contain key: value pairs of 'redirectPath': 'redirectTarget'. For generation this could generate just some html documents with a meta http-equiv refresh tag.

I don't know how well it works but you can try looking at this package:
https://github.com/mavimo/sculpin-redirect-bundle

Maybe I'm late, but for the record, you can implement this without an external bundle. Either create a new layout, or just put this snippet in your default layout's <head>:
{% if page.redirect %}
    <meta http-equiv="refresh" content="0;url={{ site.url ~ page.redirect }}" />
{% endif %}

Then create a new md or html file for the redirect, specify the layout you chose and also a redirect tag:
---
title: Redirecting...
layout: redirect
redirect: /example/url
---


MAISTRA-1251 Allow overriding Maistra Istio repo in download-charts.sh using env var

/test unittests

@dgn I added one small improvement. Can you review the second commit? Thanks.

@dgn there are now two small improvements :) sorry about that


Explanation for achieving better performance than original paper

Thanks for sharing your great work!
The MobileNet-v1 you trained achieved 72.9 top-1 acc. which surpasses the reported number (70.6) in original paper by a large margin. Could you please explain the reasons? Thanks!

Thanks!
That's a good question, and honestly I am not sure I completely know why! From the original MobileNets paper, it seems they use hyperparameters from the Inception papers, whereas I tried with the recent Nasnet paper hyperparameter (much large learning rate of ~0.2), which seem to give much better accuracy. I got the same good results with the MobileNets v2 models (and here, the reported numbers in the papers are pretty close as well).

Wow, that's a big discovery! This is a strong evidence that how important hyperparameters are in DL :-D Thanks for your explanation!

Hi @balancap, I'd like to run your souce codes for training MobileNet-v1/v2. I guess the training command should be python tf_cnn_benchmarks.py followed by hyperparameter settings. Could you please show me the list of hyperparameters you use? Or do you just follow Nasnet? Thanks!

Hi @balancap, could you please share more details of hyperparameters? Thanks a lot!

@balancap Do you mean that you were training MobileNet v1 using learning rate ~0.2 which achieved a better accuracy than original paper?
My learning rate was set to ~0.05, I tried 6000 to 10000 steps and only got 68% top-1 accuracy. Is it a problem of the small learning rate?


MSB4019

I'm getting the "MSB4019" error it used to work without any issues until i installed Visual Studio 2017.
I only have 2 folders in the sdk folder {1.0.0-preview2-1-003177, and 1.0.0} any help would be appreciated.
"C:\Users\Techn\OneDrive\Documents\Nadeko\NadekoBot\src\NadekoBot\NadekoBot.xproj(7,3): error MSB4019: The imported project "C:\Program Files\dotnet\sdk\1.0.0\Microsoft\VisualStudio\v14.0\DotNet\Microsoft.DotNet.Props" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk.
The build failed. Please fix the build errors and run again."

Same for me except mine says "C:\Program Files\dotnet\sdk\1.0.1\Microsoft\VisualStudio\v14.0\DotNet\Microsoft.DotNet.Props". I looked for the folders and got to 1.0.1 but there is no "Microsoft" folder within it.

.

Visual studio 2017 comes with msbuild version of 1.0. We're using project.json version of 1.1. Remove it from control panel and it will work. Didn't look into how to switch version but that's possible too, probably.


perlPackages.Appcpm: 0.994 -> 0.995

Motivation for this change
https://metacpan.org/release/SKAJI/App-cpm-0.995
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 105537 run on x86_64-darwin 1

2 packages built:

perl530Packages.Appcpm
perl532Packages.Appcpm

Thanks again! :tada:


Proposal: normalize IAM policies upon rendering

Problem
IAM policy documents that are generated sometimes get larger than the allowed size limit (in bytes); this happens especially when statements repeatedly (automatically) get added to it.
See https://github.com/awslabs/aws-cdk/issues/1777 for an example involving StepFunctions state machines.
Solution
Statements are modeled as a list, but they actually represent a set; duplicate elements have no meaning.
Therefore (upon resolve()ing of a policy document), any statement will only be emitted once.
Example:
const poldoc = new PolicyDocument();
poldoc.addStatement(new PolicyStatement()
  .addResource('arn:bla')
  .addAction('bla:Hello'));
poldoc.addStatement(new PolicyStatement()
  .addResource('arn:bla')
  .addAction('bla:Hello'));

console.log(poldoc.resolve()); // will contain only one copy of the 2 identical statements

Emphatically out of scope (or: why we think this is safe)
We are only going to drop statements that are exactly the same (deep equal of JSON structures). We are not going to try to analyze statements to see if they cover the same set of permissions.

Sounds good (and safe) to me. Let’s do it.

Made a ticket so Shaun has a place to comment

@rix0rrr If no one is already on it, I'm willing to take this.

Hi Jonathan, you are amazing! Thanks.


refactored codebase a/c napoleon style

Changed the whole codebase Refactored codebase to consistently use Napoleon-style docstrings.
Improves code readability and maintainability by providing clear function and class descriptions.

@williballenthin Please review the PR and suggest me the required changes , if  necessary .

Neat! Please run this through black and follow https://google.github.io/styleguide/pyguide.html#381-docstrings.

@mr-tz Please have a look . I have reformatted the codebase using black.

@williballenthin @mr-tz  I did the desired changes , please have a look.

@williballenthin Is it ok now ?

@williballenthin is all the changes ok for the PR to be merged ?

Please ensure all code style tests succeed and rebase to master.


New Tile38 1.0.4 formula

The failure is just a "too new" audit failure.
failed: brew audit tile38 --strict --online
Stacktrace
        Error: 1 problem in 1 formula
==> audit problems
tile38:
 * GitHub repository too new (<30 days old)

This is a very young project, just announced a week ago as far as I can tell. You can resubmit this in a month and it will go through. In the meantime, you or tile38 can set up your own tap to support installation via brew.

@BrewTestBot test this please

Calling this mature enough now; there's clearly enough interest on GitHub and it's a solid-looking release. Will merge this once the Jenkins build goes through.

Merged in https://github.com/Homebrew/homebrew-core/commit/d8ee49610ca4abf4ba30f7079cffe59239a63b48.
Thank you for your contribution to Homebrew!

Thanks @apjanke for leaving the PR open rather than asking me to re-submit. Glad that it's merged now.


Building Go from source tip at fde4b9ed14 failed on runtime TestGdbPython using gdb v8.2

What version of Go are you using (go version)?
go version devel +6e2a13535f Wed Oct 31 11:36:39 2018 +0700 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN="/home/ms/bin"
GOCACHE="/home/ms/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/ms/go"
GOPROXY=""
GORACE="history_size=7"
GOROOT="/home/ms/Programs/go"
GOTMPDIR=""
GOTOOLDIR="/home/ms/Programs/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build877907951=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Build the Go compiler from source.
What did you expect to see?
New Go compiler successfully build.
What did you see instead?
ok      regexp/syntax   0.618s
--- FAIL: TestGdbPython (1.71s)
    runtime-gdb_test.go:64: gdb version 8.2
    runtime-gdb_test.go:213: gdb output: Loading Go Runtime support.
        Loaded  Script
        Yes     /home/ms/git/go/src/runtime/runtime-gdb.py
        Breakpoint 1 at 0x47f770: file /home/ms/git/go/src/fmt/print.go, line 274.
        [New LWP 1749]
        [New LWP 1750]
        [New LWP 1751]
        [New LWP 1752]

        Thread 1 "a.exe" hit Breakpoint 1, fmt.Println (a=..., n=<optimized out>, err=...) at /home/ms/git/go/src/fmt/print.go:274
        274     func Println(a ...interface{}) (n int, err error) {
        BEGIN info goroutines
        * 1 running  runtime.systemstack_switch
          2 waiting  runtime.gopark
          3 waiting  runtime.gopark
          4 waiting  runtime.gopark
        END
        #1  0x0000000000485c47 in main.main () at /tmp/go-build133295970/main.go:14
        14              fmt.Println("hi")
        BEGIN print mapvar
        $1 = map[string]string = {[0x4b658d "abc"] = 0x4b6599 "def", [0x4b65a5 "ghi"] = 0x4b65ae "jkl"}
        END
        BEGIN print strvar
        $2 = 0x4b658d "abc"
        END
        BEGIN info locals
        mapvar = map[string]string = {[0x4b658d "abc"] = 0x4b6599 "def", [0x4b65a5 "ghi"] = 0x4b65ae "jkl"}
        strvar = 0x4b658d "abc"
        slicevar = {array = 0xc000096000, len = 4941197, cap = 3}
        END
        #0  fmt.Println (a=..., n=<optimized out>, err=...) at /home/ms/git/go/src/fmt/print.go:274
        274     func Println(a ...interface{}) (n int, err error) {
        BEGIN goroutine 1 bt
        #0  fmt.Println (a=..., n=<optimized out>, err=...) at /home/ms/git/go/src/fmt/print.go:274
        #1  0x0000000000485c47 in main.main () at /tmp/go-build133295970/main.go:14
        END
        BEGIN goroutine 2 bt
        #0  runtime.gopark (unlockf=<optimized out>, lock=0x551b50 <runtime.forcegc>, reason=16 '\020', traceEv=20 '\024', traceskip=1) at /home/ms/git/go/src/runtime/proc.go:303
        #1  0x0000000000428f03 in runtime.goparkunlock (lock=0x551b50 <runtime.forcegc>, reason=16 '\020', traceEv=20 '\024', traceskip=1) at /home/ms/git/go/src/runtime/proc.go:308
        #2  0x0000000000428cca in runtime.forcegchelper () at /home/ms/git/go/src/runtime/proc.go:251
        #3  0x000000000044fd41 in runtime.goexit () at /home/ms/git/go/src/runtime/asm_amd64.s:1340
        #4  0x0000000000000000 in ?? ()
        END
        Breakpoint 2 at 0x485c72: file /tmp/go-build133295970/main.go, line 19.
        hi

        Thread 1 "a.exe" hit Breakpoint 2, main.main () at /tmp/go-build133295970/main.go:19
        19      }  // END_OF_PROGRAM
        BEGIN goroutine 1 bt at the end
        #0  main.main () at /tmp/go-build133295970/main.go:19
        END

    runtime-gdb_test.go:274: info locals failed: mapvar = map[string]string = {[0x4b658d "abc"] = 0x4b6599 "def", [0x4b65a5 "ghi"] = 0x4b65ae "jkl"}
        strvar = 0x4b658d "abc"
        slicevar = {array = 0xc000096000, len = 4941197, cap = 3}
--- FAIL: TestGdbPythonCgo (2.25s)
    runtime-gdb_test.go:64: gdb version 8.2
    runtime-gdb_test.go:213: gdb output: Loading Go Runtime support.
        Loading Go Runtime support.
        Breakpoint 1 at 0x481180: file /home/ms/git/go/src/fmt/print.go, line 274.
        [Thread debugging using libthread_db enabled]
        Using host libthread_db library "/usr/lib/libthread_db.so.1".
        [New Thread 0x7ffff5b62700 (LWP 1959)]
        [New Thread 0x7fffeffff700 (LWP 1962)]
        [New Thread 0x7ffff5361700 (LWP 1964)]
        [New Thread 0x7ffff4b60700 (LWP 1965)]
        [New Thread 0x7fffef7fe700 (LWP 1966)]

        Thread 1 "a.exe" hit Breakpoint 1, fmt.Println (a=..., n=<optimized out>, err=...) at /home/ms/git/go/src/fmt/print.go:274
        274     func Println(a ...interface{}) (n int, err error) {
        BEGIN info goroutines
        * 1 running  runtime.systemstack_switch
          2 waiting  runtime.gopark
          3 waiting  runtime.gopark
          4 waiting  runtime.gopark
        END
        #1  0x0000000000487657 in main.main () at /tmp/go-build772824409/main.go:15
        15              fmt.Println("hi")
        BEGIN print mapvar
        $1 = map[string]string = {[0x4b95ad "abc"] = 0x4b95b9 "def", [0x4b95c5 "ghi"] = 0x4b95ce "jkl"}
        END
        BEGIN print strvar
        $2 = 0x4b95ad "abc"
        END
        BEGIN info locals
        mapvar = map[string]string = {[0x4b95ad "abc"] = 0x4b95b9 "def", [0x4b95c5 "ghi"] = 0x4b95ce "jkl"}
        strvar = 0x4b95ad "abc"
        slicevar = {array = 0xc0000a4000, len = 4953517, cap = 3}
        END
        #0  fmt.Println (a=..., n=<optimized out>, err=...) at /home/ms/git/go/src/fmt/print.go:274
        274     func Println(a ...interface{}) (n int, err error) {
        BEGIN goroutine 1 bt
        #0  fmt.Println (a=..., n=<optimized out>, err=...) at /home/ms/git/go/src/fmt/print.go:274
        #1  0x0000000000487657 in main.main () at /tmp/go-build772824409/main.go:15
        END
        BEGIN goroutine 2 bt
        #0  runtime.gopark (unlockf=<optimized out>, lock=0x556ca0 <runtime.forcegc>, reason=16 '\020', traceEv=20 '\024', traceskip=1) at /home/ms/git/go/src/runtime/proc.go:303
        #1  0x000000000042a843 in runtime.goparkunlock (lock=0x556ca0 <runtime.forcegc>, reason=16 '\020', traceEv=20 '\024', traceskip=1) at /home/ms/git/go/src/runtime/proc.go:308
        #2  0x000000000042a60a in runtime.forcegchelper () at /home/ms/git/go/src/runtime/proc.go:251
        #3  0x0000000000451811 in runtime.goexit () at /home/ms/git/go/src/runtime/asm_amd64.s:1340
        #4  0x0000000000000000 in ?? ()
        END
        Breakpoint 2 at 0x487682: file /tmp/go-build772824409/main.go, line 20.
        hi

        Thread 1 "a.exe" hit Breakpoint 2, main.main () at /tmp/go-build772824409/main.go:20
        20      }  // END_OF_PROGRAM
        BEGIN goroutine 1 bt at the end
        #0  main.main () at /tmp/go-build772824409/main.go:20
        END

    runtime-gdb_test.go:274: info locals failed: mapvar = map[string]string = {[0x4b95ad "abc"] = 0x4b95b9 "def", [0x4b95c5 "ghi"] = 0x4b95ce "jkl"}
        strvar = 0x4b95ad "abc"
        slicevar = {array = 0xc0000a4000, len = 4953517, cap = 3}
FAIL
FAIL    runtime 39.656s

likely a result of http://golang.org/cl/141822. Different gdb versions give different output (e.g. order and with and without slice content). As discussed in the CL we should test each variable individually
to also cover:
mapvar = map[string]string = {[0x4b95ad "abc"] = 0x4b95b9 "def", [0x4b95c5 "ghi"] = 0x4b95ce "jkl"}
strvar = 0x4b95ad "abc"
slicevar = {array = 0xc0000a4000, len = 4953517, cap = 3}

Renamed the issue appropriately. The compiler does not fail to build, just the test fails.

@martisch seems like the generated output is different since the last fix with gdb v8.2 [1].
The following patch has been tested work on my environment,
@@ -264,9 +264,9 @@ func testGdbPython(t *testing.T, cgo bool) {
 	// back to its original form.

 	infoLocalsRe1 := regexp.MustCompile(`slicevar *= *\[\]string *= *{"def"}`)
 	// Format output from gdb v8.2
-	infoLocalsRe2 := regexp.MustCompile(`^slicevar = .*\nmapvar = .*\nstrvar = 0x[0-9a-f]+ "abc"`)
+	infoLocalsRe2 := regexp.MustCompile(`^mapvar = .*\nstrvar = 0x[0-9a-f]+ "abc"\nslicevar = .*`)
 	// Format output from gdb v7.7
 	infoLocalsRe3 := regexp.MustCompile(`^mapvar = .*\nstrvar = "abc"\nslicevar *= *\[\]string`)
 	if bl := blocks["info locals"]; !infoLocalsRe1.MatchString(bl) &&
 		!infoLocalsRe2.MatchString(bl) &&

[1] https://go-review.googlesource.com/c/go/+/135055


How to port/ build openssl on Windows (Visual studio) from source file?

I'm am trying to port OpenSSL for windows (Visual Studio) from Source code , without using DLL and Lib.
Is there any porting guide available or could you give any inputs regarding this.

The installation/build guide is here:
https://github.com/openssl/openssl/blob/OpenSSL_1_1_1-stable/INSTALL
That guide covers all platforms. There are some additional Windows specific notes here:
https://github.com/openssl/openssl/blob/OpenSSL_1_1_1-stable/NOTES.WIN

The installation/build guide is here:
https://github.com/openssl/openssl/blob/OpenSSL_1_1_1-stable/INSTALL
That guide covers all platforms. There are some additional Windows specific notes here:
https://github.com/openssl/openssl/blob/OpenSSL_1_1_1-stable/NOTES.WIN

It would be very helpful if the OpenSSL community had a batch file to generate the project file like the solution used in Curl forum.
Reference: https://github.com/curl/curl/tree/master/projects
Expecially since I believe these two projects work very well together.

It would be very helpful if the OpenSSL community had a batch file to generate the project file like the solution used in Curl forum.
Reference: https://github.com/curl/curl/tree/master/projects
Expecially since I believe these two projects work very well together.


IT: Switch to OpenJDK8 base image. (#5060)

IT: Switch to OpenJDK8 base image.

Also split the Docker image into a base image and a child image, and
build the base image ahead of time for efficiency's sake. Also upgrade
ZK to 3.4.10.

Additional comments about ZK upgrades.

@leventov please use branches in a different repository than druid-io/druid, so we can keep the main repo relatively cleaner.

15 TC inspections failed, seemingly spuriously. Restarted that checker. Alternatively, we could just merge this without inspections passing, since nothing changed in this patch that should affect inspections anyway.


[Cloud Run] app.current_request can be rewritten by subsequent requests while function is running

Test I ran looked like
@app.pubsub_subscription(
    os.getenv("PUBSUB_TOPIC"),
    use_subscription=True,
)
def main(event):
    change_event = json.loads(event)
    attributes = app.current_request.json["message"].get("attributes")
    v1 = change_event["value"]
    v2 = attributes["value"]
    if v2 != v1:
        app.log.error(f"{v2}:{v1}")

sent pubsub messages all at once using
for i in range(1000):
    data = json.dumps(
        {
            "value": str(i)
        })
    publish_future = publisher.publish(topic_path, data.encode("utf-8"), value=str(i))

which logged:
ERROR:goblet.app:872:468


[bug]: Unable to lookup SRV records via primary seed on machine with multiple network interfaces

Background
022-12-05 17:11:21.778 [TRC] DISC: Unable to lookup SRV records via primary seed (nodes.lightning.directory): lookup _nodes._tcp.nodes.lightning.direct     ory on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:44770-><IP_ADDRESS>:53: i/o timeout

Your environment

version of lnd: 0.15.5-beta
which operating system (uname -a on *Nix) : debian 11 bullseye
version of btcd, bitcoind, or other backend: bitcoind v23.0.0
any other relevant environment details

There are 2 physical interfaces mapped to 6 ipv4 addresses. lnd is using an internal only ip address to perform the DNS lookup and there's no way to force it to use the other interface even though it is listening only on the correct address using --listen=.
Steps to reproduce
Setup an up to date linux machine with an internal and an external network. Setup lnd to listen only on the external network.
Expected behaviour
lnd should find peers using the external network
Actual behaviour
lnd is unable to find bootstrap peers

I think lnd is using the default dns resolver in pure go.
Are you using a custom docker image/setting GODEBUG=netdns=cgo? This is what we do it in our docker image.

That appears to have resolved it.


[Suggestion] Dark Steel Door

It would be awesome if you added a Dark Steel Door that could be painted much in the same way as the existing Dark Steel Trapdoor.

Suggestions are now being tracked on:
http://ideas.theideawall.com/EnderIO/
Please post your suggestion there.

Thanks for the hint, I just saw that it was already suggested :D


Port "Steady-State: Homodyned Jaynes-Cummings emission" tutorial to QuTiP 5.

The Homodyned Jaynes-Cummings emission tutorial is currently only available for QuTiP 4. It needs to be ported to QuTiP 5.
Steps:

Copy the Markdown notebook into the appropriate folder under  tutorials-v5.
Update the notebook to work with QuTiP 5.
Make any small improvements needed to the notebook.
Add some assertions to the end of the notebook to ensure it ran correctly (e.g. check some results).
Check that the notebook is included in the website build scripts.

Hi, I would like to work on this issue for the UnitaryHack.

I would like to work in this issue. It is still open?

@ArturDomingues I accepted and merged your PR (#98). Could you comment on this issue so that I can assign it to you? Then the UnitaryHACK administrators will be able to see that you claimed it.
@alan-nala @GustavoPetronilo Thank you for your submissions. Artur's PR was completed first.

Ok, what do I need to do with respect to UnitaryHACK?

Ok, what do I need to do with respect to UnitaryHACK?

Thanks for commenting. That allowed me to assign the completed issue to you and now the UnitaryHACK team will do the rest. All done from our side, I believe.

Nice, I hope I can help solve more things in the near future. Especially in the main qutip repo. I have used it a lot in my masters it is one of the nicest tools out there for simulating problems.

Looking forward to seeing your PRs to QuTiP and great to hear it was helpful during your MSc. There are likely still many small improvements to make in QuTiP 5 and many bigger projects to pursue.


Robot Update: Dialogue Box & Initiate Update

As a user, I would like to be given the option to update to the latest Robot server version.
Acceptance Criteria

'Update' | 'Updated' link displayed in 'Information' card on 'Robot Settings' page
-- If current Robot version matches latest version, display inactive 'Updated' text
-- If current Robot version is behind latest version, display 'Update' link'
Clicking button opens a robot update alert modal
-- Alert modal includes latest available server version number
-- Alert modal includes option to cancel ('not now') or 'update'
Selecting 'Update' initiates a robot update

Tasks (break down as stories?)

[ ] Add robot bundle to app shell (small)
[ ] Send robot bundle to api (robot server) (med)
[ ] UI including button, dialogue box, and confirmation (maybe confirmation is a separate story) (med)

Design

https://projects.invisionapp.com/d/main#/console/13154070/275374445/preview

API now supports update. See #912

UX review completed, feedback to be reviewed and addressed with additional tickets


Cannot Find module 'mkdirp'

My issue is based on my stackoverflow post
My environment has been updated to as follows:-
**- Node: v14.0.0

Yarn: 1.22.10
OS: Linux homestead 4.15.0-124-generic #127-Ubuntu SMP Fri Nov 6 10:54:43 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
-**

I cannot seem to isolate the issue and trying to install one by one seems to encounter other issues to resolve. Is this a Node version thing?

It seems the install script try to be executed before the complete node_modules install.
Some required modules are not yet installed when install.js run.

It's possible Yarn isn't waiting for all the dependencies to be installed before executing the install script. Unfortunately, it's not something we can address (if this wasn't something machine/time specific)


Feature Request: Show visual rackspace reservations in graphical view

Currently rack reservations show in the reservations table on a rack page, and when using mouse-over on the graphical rack view it displays the reservation as a comment.
Ideally we would also like to see some sort of shading or visual representation of rack-space reservations that are not yet filled with equipment in the graphical rack view. currently it is easy for someone untrained to believe the space is actually available when viewing the rack-space graphical view alone as without the mouse-over it does not show the space is reserved.

Rack reservations are represented with blue diagonal stripes, like this:

If you don't see that, try forcing a refresh of the cached CSS file by hitting ctrl+F5 (or equivalent) or navigating to /static/css/base.css directly.

Oh well that is embarrassing. I'm sure I tried a forced refresh, but clearly not.... /facepalm.
Thanks for the quick response


Account prohibited when using Tdlib

Recently, when logging in with a new account using tdlib, there is a phenomenon of withdrawal from that account. In addition, you will be withdrawn when you log in with an existing account, not a new one. Do you happen to know about any related documents or issues?
And if there is any improvement or solution, please let me know

This behavior is up to the server and isn't directly related to TDLib. It blocks all non-bot accounts with suspicious activity, which don't look like real users.

This behavior is up to the server and isn't directly related to TDLib. It blocks all non-bot accounts with suspicious activity, which don't look like real users.

It was blocked only by client login using tdlib, not suspicious behavior. Does it affect the tdlib version?

It is not directly related to app version. Not doing anything after logging in is also an action, ordinary apps do.


Add support for specifying ACME HTTP01 ingress name as an annotation on Certificates

What this PR does / why we need it:
Adds support for an annotation on Certificates http01.acme.certmanager.k8s.io/ingress-to-edit to manually specify the ingress to edit, to overcome the need to have access to adding a specific solver entry in the Issuer.
A new field AllowManuallySpecifiedIngress is available for Issuer solvers to enable this annotation to be used - if the annotation is set but the this option is not enabled, then the solver selection procedure will continue as if the annotation was not set.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #1666
Special notes for your reviewer:
The tests only ensure the solver is correctly set after creation, I am assuming that anything after this point is reliant on the correctness of the sync() function in general, and another PR should handle adding tests for the new solver.
Similarly, no changes have been made to the validating webhook since it doesn't seem to consider the new solver anyway and I think another PR would be better suited for doing this.
Release note:

Adds support for Certificate annotation `http01.acme.certmanager.k8s.io/ingress-to-edit` to manually specify the Ingress to edit, when the option is enabled under `AllowManuallySpecifiedIngress` in one of the Issuer's solvers.

/retest

/retest

Will be adding doc/validation now that the podTemplate PR has started the process

Closed in favour of #1880