Weird text in terminal

I am gettin the weird things all over the place. I read the common problems section in your description and ran the command, but the TERM is set to linux it seems, yet I still get the weird thinks printed. Here is a copy of the printout:
133;C;133;D;11337;RemoteHost=AlexBet@AlexBets-Mac-Pro.local1337;CurrentDir=/Volumes/ServerWebs/Sites/AlexBet/modules.v5133;AAlexBets-Mac-Pro:modules.v5 Al exBet$ 133;B                                                                                133;C;133;D;11337;RemoteHost=AlexBet@AlexBets-Mac-Pro.local1337;CurrentDir=/Volumes/ServerWebs/Sites/AlexBet/modules.v5133;AAlexBets-Mac-Pro:modules.v5 Al exBet$ 133;B                                                                                133;C;133;D;11337;RemoteHost=AlexBet@AlexBets-Mac-Pro.local1337;CurrentDir=/Volumes/ServerWebs/Sites/AlexBet/modules.v5133;AAlexBets-Mac-Pro:modules.v5 Al exBet$ 133;B                                                                                133;C;133;D;11337;RemoteHost=AlexBet@AlexBets-Mac-Pro.local1337;CurrentDir=/Volumes/ServerWebs/Sites/AlexBet/modules.v5133;AAlexBets-Mac-Pro:modules.v5 Al exBet$ 133;B                                                                                133;C;133;D;11337;RemoteHost=AlexBet@AlexBets-Mac-Pro.local1337;CurrentDir=/Volumes/ServerWebs/Sites/AlexBet/modules.v5133;AAlexBets-Mac-Pro:modules.v5 Al exBet$ 133;B                                                                                133;C;133;D;11337;RemoteHost=AlexBet@AlexBets-Mac-Pro.local1337;CurrentDir=/Volumes/ServerWebs/Sites/AlexBet/modules.v5133;AAlexBets-Mac-Pro:modules.v5 Al exBet$ 133;Benv | grep TERM                                                                                133;C;TERM=linux                                                                                ITERM_ORIG_PS1=\h:\W \u\$                                                                                ITERM_PREV_PS1=\[133;D;$?1337;RemoteHost=AlexBet@AlexBets-Mac-Pro.local1337;CurrentDir=/Volumes/ServerWebs/Sites/AlexBet/modules.v5133;A\]\h:\W \u\$ \[133 ;B\]
You can see where I ran the "env | grep TERM" command and the result. Hope you can fix this. Thanks!

Looks like an iTerm issue. Unfortunately some applications do not care what type of terminal is connected and simply assumes xterm capabilities. Current devel plans include support for xterm but it requires changing the underlying terminal emulator so it's not easily done. Until then only fix is to disable iTerm when using TerminalView - if that's possible I don;'t know (not mac user).

Hmmmm, that is weird, because it works on my laptop with iTerm, but it doesn't work on my desktop with iTerm. How could that be?

Maybe I should clarify, I'm not saying that it is an iTerm issue - it just looks like it. Could also be some other third party app messing with your bash prompt. I think @randy3k has seen something similar maybe he can pinpoint it. My best suggestion is to try to disable e.g. iTerm or other third party apps to see if that helps.
Do you normaly use bash ? Otherwise you could also try another shell.

@alexbet
try replacing the source command of the iTerm script in your .bash_profile or .bashrc by the following
if [ "$TERM" = "xterm-256color" ] && [ -z "$INSIDE_EMACS" ]; then
    [ -f "~/.iterm2_shell_integration.bash" ] && source ~/.iterm2_shell_integration.bash
fi

Hmmmm, that is weird, because it works on my laptop with iTerm, but it doesn't work on my desktop with iTerm. How could that be?

Sorry I think i misunderstood you. Are you confirming that turning off iTerm fixes the issue ?

I am pretty sure it is caused by the .iterm2_shell_integration.bash given the output of the console. Disabling sourcing the script should immediately fix the issue. The matter is to figure out where the source command is hidden (most probably is .bash_profile).

The root of the issue is because pyte 0.5.2 does not support OSC sequence, I tried switching to pyte 0.7.0-dev and it fixed the issue. However, I couldn't figure out how to fix the resize-window issue.

@randy3k - Adding that code in the .bash_profile file fixed the issue. Thank you!

@randy3k Thx


Required permissions to create a managed endpoint

In the following sites, the resource group doesn't require permissions to create a managed online endpoint, but it actually requires "Microsoft.Resources/deployments/write".
https://learn.microsoft.com/en-us/azure/machine-learning/how-to-assign-roles?tabs=labeler#common-scenarios

Creating a managed batch endpoint requires similar permissions, but it's not even mentioned in the table. Please add and correct each of them.

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

ID: 0ca3ea12-4f8c-72e4-4703-ea2ef9dcee0f
Version Independent ID: a4b7c496-22f9-0fd6-cd7d-0fd613f1a1ec
Content: Manage roles in your workspace - Azure Machine Learning
Content Source: articles/machine-learning/how-to-assign-roles.md
Service: machine-learning
Sub-service: enterprise-readiness
GitHub Login: @meyetman
Microsoft Alias: meyetman

@damochiz
Thanks for your feedback! We will investigate and update as appropriate.

@meyetman
Could you please review on this, update as appropriate.

@damochiz
Thank you for bringing this to our attention.
I've delegated this to content author @meyetman, who will review it and offer their insightful opinions.

@damochiz Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue. #please-close


Delete old disqus threads

In disqus thread list appears many discussion that was created for testing, with old address (localhost, <IP_ADDRESS> etc.) as you can see here:
https://disqus.com/home/forum/anyway-feedback/
The only way tot delete disqus thread is their api
https://disqus.com/api/docs/threads/remove/
We need to request API key and preform that.
https://disqus.com/api/applications/

Were deleted on #261

I'm not talking about the discussion list in our database, I'm talking about the list of discussions in Disqus system
go to https://disqus.com/home/forum/anyway-feedback/ and scroll all the way down, notice the some of the links pointing to localhost and so on.
Those discussion appear to everyone in the box under every discussion so we need to delete them


Possible deadlock on WS close

My application involves creating multiple SHIP devices and repeatedly starting/stopping the service. Every now and then, when I close the service, the application gets stuck. when replacing all sync.Mutex with deadlock.Mutex in websocket.go, the following trace logs appear:
POTENTIAL DEADLOCK:
Previous place where the lock was grabbed
goroutine 22687 lock 0xc0069cb758
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:256 ws.(*WebsocketConnection).WriteMessageToWebsocketConnection { w.muxShipWrite.Lock() } <<<<<
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:255 ws.(*WebsocketConnection).WriteMessageToWebsocketConnection { func (w *WebsocketConnection) WriteMessageToWebsocketConnection(message []byte) error { }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\connection.go:380 ship.(*ShipConnection).sendShipModel { err = c.dataWriter.WriteMessageToWebsocketConnection(shipMsg) }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\hs_hello.go:256 ship.(*ShipConnection).handshakeHelloSend {  }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\hs_hello.go:88 ship.(*ShipConnection).handshakeHello_Abort {  }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\handshake.go:151 ship.(*ShipConnection).handleState { case model.SmeHelloStateAbort: }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\handshake.go:199 ship.(*ShipConnection).setAndHandleState { c.handleState(false, nil) }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\hs_hello.go:108 ship.(*ShipConnection).handshakeHello_PendingInit { } }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\handshake.go:142 ship.(*ShipConnection).handleState { case model.SmeHelloStatePendingInit: }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\handshake.go:133 ship.(*ShipConnection).handleState { } }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\handshake.go:199 ship.(*ShipConnection).setAndHandleState { c.handleState(false, nil) }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\hs_init.go:42 ship.(*ShipConnection).handshakeInit_cmiStateServerWait { c.setAndHandleState(model.SmeHelloState) }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\handshake.go:116 ship.(*ShipConnection).handleState { } }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\handshake.go:44 ship.(*ShipConnection).handleShipMessage { c.handleState(timeout, message) }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\connection.go:245 ship.(*ShipConnection).HandleIncomingWebsocketMessage { c.handleShipMessage(false, message) }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:157 ws.(*WebsocketConnection).readShipPump { select { }

Have been trying to lock it again for more than 30s
goroutine 22688 lock 0xc0069cb758
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:327 ws.(*WebsocketConnection).closeShipWriteChannel { w.muxShipWrite.Lock() } <<<<<
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:326 ws.(*WebsocketConnection).closeShipWriteChannel { func (w *WebsocketConnection) closeShipWriteChannel() { }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:97 ws.(*WebsocketConnection).writeShipPump.func1 { /*go*/ w.closeShipWriteChannel() }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:106 ws.(*WebsocketConnection).writeShipPump { if w.isConnClosed() { }

Here is what goroutine 22687 doing now
goroutine 22687 [chan send]:
github.com/enbility/ship-go/ws.(*WebsocketConnection).WriteMessageToWebsocketConnection(0xc0069cb700, {0xc004164c30, 0x2a, 0x30})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ws/websocket.go:263 +0x10b
github.com/enbility/ship-go/ship.(*ShipConnection).sendShipModel(0xc006139b30, 0x78?, {0x1328c00?, 0xc0012beb20?})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/connection.go:380 +0x49
github.com/enbility/ship-go/ship.(*ShipConnection).handshakeHelloSend(0xc006139b30, {0x144aff4, 0x7}, 0x0, 0x0)
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/hs_hello.go:257 +0xd5
github.com/enbility/ship-go/ship.(*ShipConnection).handshakeHello_Abort(0xc006139b30)
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/hs_hello.go:89 +0x32
github.com/enbility/ship-go/ship.(*ShipConnection).handleState(0xc006139b30, 0x0, {0x0, 0x0, 0x0})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/handshake.go:152 +0x254
github.com/enbility/ship-go/ship.(*ShipConnection).setAndHandleState(0xc006139b30, 0xc000fa1860?)
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/handshake.go:199 +0x2f
github.com/enbility/ship-go/ship.(*ShipConnection).handshakeHello_PendingInit(0xc006139b30)
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/hs_hello.go:107 +0x7a
github.com/enbility/ship-go/ship.(*ShipConnection).handleState(0xc006139b30, 0x0, {0x0, 0x0, 0x0})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/handshake.go:143 +0x20f
github.com/enbility/ship-go/ship.(*ShipConnection).handleState(0xc006139b30, 0x0, {0x0, 0x0, 0x0})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/handshake.go:134 +0x1c9
github.com/enbility/ship-go/ship.(*ShipConnection).setAndHandleState(0xc006139b30, 0x1f935f2?)
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/handshake.go:199 +0x2f
github.com/enbility/ship-go/ship.(*ShipConnection).handshakeInit_cmiStateServerWait(0xc006139b30, {0xc0026d3e00, 0x2, 0x200})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/hs_init.go:42 +0xa5
github.com/enbility/ship-go/ship.(*ShipConnection).handleState(0xc006139b30, 0x0, {0xc0026d3e00, 0x2, 0x200})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/handshake.go:117 +0xe5
github.com/enbility/ship-go/ship.(*ShipConnection).handleShipMessage(0xc0026d3e00?, 0xb0?, {0xc0026d3e00?, 0xc000bb7ee8?, 0x28e33842ea8?})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/handshake.go:44 +0x225
github.com/enbility/ship-go/ship.(*ShipConnection).HandleIncomingWebsocketMessage(0xc006139b30, {0xc0026d3e00, 0x2, 0x200})
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ship/connection.go:245 +0x26f
github.com/enbility/ship-go/ws.(*WebsocketConnection).readShipPump(0xc0069cb700)
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ws/websocket.go:183 +0x127
created by github.com/enbility/ship-go/ws.(*WebsocketConnection).run in goroutine 23442
        C:/Users/mohamed.eltawel/go/pkg/mod/github.com/enbility/ship-go@v0.6.0/ws/websocket.go:88 +0xc5
Other goroutines holding locks:
goroutine 114 lock 0xc001ab73d8
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:256 ws.(*WebsocketConnection).WriteMessageToWebsocketConnection { w.muxShipWrite.Lock() } <<<<<
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ws\websocket.go:255 ws.(*WebsocketConnection).WriteMessageToWebsocketConnection { func (w *WebsocketConnection) WriteMessageToWebsocketConnection(message []byte) error { }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\connection.go:380 ship.(*ShipConnection).sendShipModel { err = c.dataWriter.WriteMessageToWebsocketConnection(shipMsg) }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\connection.go:174 ship.(*ShipConnection).CloseConnection.func1 {  }
C:/Program Files/Go/src/sync/once.go:74 sync.(*Once).doSlow { if o.done.Load() == 0 { }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\connection.go:193 ship.(*ShipConnection).CloseConnection { }) }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\ship\connection.go:151 ship.(*ShipConnection).CloseConnection { func (c *ShipConnection) CloseConnection(safe bool, code int, reason string) { }
..\..\..\go\pkg\mod\github.com\enbility\ship-go@v0.6.0\hub\hub_pairing.go:150 hub.(*Hub).DisconnectSKI { con.CloseConnection(true, 0, reason) }
internal\energyguard.go:617 internal.(*EnergyGuard).UnregisterRemoteSKI { h.myService.DisconnectSKI(remoteSki, "Disconnected by CEM") }
internal\energyguard.go:616 internal.(*EnergyGuard).UnregisterRemoteSKI { defer h.mu.Unlock() }
cmd\eebus-hub\cemApi.go:677 main.init.func25 { // }) }
internal\restHandler.go:91 internal.(*RestHandler).PushRoutes.func1 { // Insert endpoint to the group handler }
..\..\..\go\pkg\mod\github.com\gin-gonic\gin@v1.10.0\context.go:185 gin.(*Context).Next { c.handlers[c.index](c) }
..\..\..\go\pkg\mod\github.com\gin-gonic\gin@v1.10.0\context.go:182 gin.(*Context).Next { func (c *Context) Next() { }
..\..\..\go\pkg\mod\github.com\gin-gonic\gin@v1.10.0\context.go:185 gin.(*Context).Next { c.handlers[c.index](c) }
..\..\..\go\pkg\mod\github.com\gin-gonic\gin@v1.10.0\context.go:182 gin.(*Context).Next { func (c *Context) Next() { }
..\..\..\go\pkg\mod\github.com\gin-gonic\gin@v1.10.0\gin.go:590 gin.(*Engine).ServeHTTP {  }
C:/Program Files/Go/src/net/http/server.go:3210 http.serverHandler.ServeHTTP { handler.ServeHTTP(rw, req) }
C:/Program Files/Go/src/net/http/server.go:2092 http.(*conn).serve { serverHandler{c.server}.ServeHTTP(w, w.req) }

Note: code lines are shifted by one line
The possible reason for this problem is that the function WriteMessageToWebsocketConnection is blocked in Line 261 waiting for the channel to be emptied, and the Line of code which is supposed to consume the channel is never called upon closing the channel.

Thanks a lot for the bug report!
Would you be interested in providing a unit test that shows the issue? As an idea: It doesn't have to happen every time in the test, but a at least 1 out of 10 would be great and then make the test run at least 10 times.


Updated to work with ign-plugin

This PR depends and needs a release from this other PR https://github.com/ignitionrobotics/ign-plugin/pull/30
All the symbols are loaded in the executable because of the RTLD_GLOBAL. This logic will filter the symbols
Signed-off-by: ahcorde<EMAIL_ADDRESS>

Let me just check if I understood why this PR is needed. If https://github.com/ignitionrobotics/ign-plugin/pull/30 gets in, ign-rendering won't work properly anymore unless this filtering is done?
This sounds like a breaking change on ign-plugin, that will required a major version bump on that library.
I'm also curious about what other pluginNames are showing up here that don't contain rendering. Do you remember?

Let me just check if I understood why this PR is needed. If https://github.com/ignitionrobotics/ign-plugin/pull/30 gets in, ign-rendering won't work properly anymore unless this filtering is done?
This sounds like a breaking change on ign-plugin, that will required a major version bump on that library.
I'm also curious about what other pluginNames are showing up here that don't contain rendering. Do you remember?

Yes, this may require some more mayor version bumps: ign-gazebo, ign-gui, ign-rendering, sensors.

I'm also curious about what other pluginNames are showing up here that don't contain rendering. Do you remember?

I will check it.

With the proposal on https://github.com/ignitionrobotics/ign-plugin/pull/35, I think this PR isn't necessary.

Since https://github.com/ignitionrobotics/ign-rendering/pull/236/, we're looking explicitly for plugins that implement the ignition::rendering::RenderEnginePlugin interface, and in case we find several, we load the first one and print a warning to the user.
So I'm closing this one because I think it is no longer necessary. Please speak up if you think otherwise, @ahcorde .


How do I set object's named or numerically indexed properties?

I'd like to implement a Vector object that takes numeric values, such as vector[12] = 24. I'd like to use getters and setters in C++.
I see NAN_GETTER, NAN_SETTER, NAN_PROPERTY_GETTER, NAN_PROPERTY_SETTER, with the property-specific methods looking most applicable, but I could not see any examples of how to use it, or enough further explanation of how to put it to use.

Why not just do this in javascript with an array?
NAN_GETTER and NAN_SETTER are accessors, https://developers.google.com/v8/embed#accesssors
NAN_PROPERTY_GETTER and NAN_PROPERTY_SETTER are named property interceptors, while NAN_INDEX_GETTER and NAN_INDEX_SETTER are indexed property interceptors https://developers.google.com/v8/embed#interceptors
https://v8docs.nodesource.com/io.js-1.2/db/d5f/classv8_1_1_object_template.html#aa90691622f01269c6a11391d372ca0c5
https://github.com/rvagg/nan/blob/master/test/cpp/settergetter.cpp

@kkoopa I want to do this in C++ because that's where I'm planning on having the object residing. I'm interested in getting objects/classes written in C++ working within JavaScript. Just doing it in JavaScript would likely be slower. At least I'd like to compare implementations.
Thankyou for providing those references.
NAN_INDEX_SETTER looks like what I'm looking for. I've written some code using it, it compiles, but I don't know where to find the index that it was given.
It does not look like others have been using this feature as my search for its use on Google only turned up NAN itself.
NAN_INDEX_SETTER is not currently covered by tests either.

https://github.com/rvagg/nan#api_nan_index_getter

You get a uint32_t index and an appropriately typed args object that can act similar to the args argument to a NAN_METHOD call.

Also, I know that it is in use. https://github.com/TooTallNate/node-weak

Thanks for showing me that example. Unfortunately I'm running into problems with SetIndexedPropertyHandler. From what I can tell, a major difference between the node-weak code and what I'm looking for is that the addon provides a constructer function that gets called in JavaScript, then the property of the instance gets set through JavaScript, with the interception taking place in C++. I did not understand the V8 guide in terms of how it would translate to use of NAN
Here is the code I have that is not successfully binding the .IndexedPropertyHandler:
#include "c_vector.h"
#include <iostream>
#include <stdlib.h>

using namespace v8;
using namespace std;

Persistent<Function> CVector::constructor;

CVector::CVector(double value) : value_(value) {
}

CVector::~CVector() {
}

void CVector::Init(Handle<Object> exports) {
  NanScope();

  // Prepare constructor template
  Local<FunctionTemplate> tpl = NanNew<FunctionTemplate>(New);
  tpl->SetClassName(NanNew("CVector"));
  tpl->InstanceTemplate()->SetInternalFieldCount(1);

  tpl->SetIndexedPropertyHandler(CVector::IndexedSet, CVector::IndexedGet)
  // Prototype
  NODE_SET_PROTOTYPE_METHOD(tpl, "plusOne", PlusOne);

  NanAssignPersistent(constructor, tpl->GetFunction());
  exports->Set(NanNew("CVector"), tpl->GetFunction());
}

NAN_METHOD(CVector::New) {
  NanScope();

  if (args.IsConstructCall()) {
    // Invoked as constructor: `new MyObject(...)`
    double value = args[0]->IsUndefined() ? 0 : args[0]->NumberValue();
    CVector* obj = new CVector(value);
    obj->Wrap(args.This());

    NanReturnValue(args.This());
  } else {
    // Invoked as plain function `MyObject(...)`, turn into construct call.
    const int argc = 1;
    Local<Value> argv[argc] = { args[0] };
    Local<Function> cons = NanNew<Function>(constructor);
    NanReturnValue(cons->NewInstance(argc, argv));
  }
}

NAN_METHOD(CVector::PlusOne) {
  NanScope();
  CVector* obj = ObjectWrap::Unwrap<CVector>(args.Holder());
  obj->value_ += 1;

  NanReturnValue(NanNew(obj->value_));
}

NAN_INDEX_SETTER(CVector::IndexedSet) {
    NanScope();
    cout << "NAN_INDEX_SETTER" << endl;
    NanReturnNull();
}

NAN_INDEX_GETTER(CVector::IndexedGet) {
    NanScope();
    cout << "NAN_INDEX_GETTER" << endl;
    NanReturnNull();
}

And in the header:
#ifndef CVECTOR_H
#define CVECTOR_H

#include <nan.h>

class CVector : public node::ObjectWrap {
 public:
  static void Init(v8::Handle<v8::Object> exports);

 private:
  explicit CVector(double value = 0);
  ~CVector();

  static NAN_METHOD(New);
  static NAN_METHOD(PlusOne);
  static NAN_INDEX_SETTER(IndexedSet);
  static NAN_INDEX_GETTER(IndexedGet);
  //static NAN_INDEX_SETTER
  static v8::Persistent<v8::Function> constructor;
  double value_;
};

#endif

You have swapped the arguments to SetIndexedPropertyHandler, which in turn is not directly available from a FunctionTemplate, you need to access either InstanceTemplate() or PrototypeTemplate().
#include "cvector.h"
#include <iostream>
#include <cstdlib>

using namespace v8;
using namespace std;

Persistent<Function> CVector::constructor;

CVector::CVector(double value) : value_(value) {
}

CVector::~CVector() {
}

void CVector::Init(Handle<Object> exports) {
  NanScope();

  // Prepare constructor template
  Local<FunctionTemplate> tpl = NanNew<FunctionTemplate>(New);
  tpl->SetClassName(NanNew("CVector"));
  tpl->InstanceTemplate()->SetInternalFieldCount(1);

  tpl->PrototypeTemplate()->SetIndexedPropertyHandler(CVector::IndexedGet, CVector::IndexedSet);
  // Prototype
  NODE_SET_PROTOTYPE_METHOD(tpl, "plusOne", PlusOne);

  NanAssignPersistent(constructor, tpl->GetFunction());
  exports->Set(NanNew("CVector"), tpl->GetFunction());
}

NAN_METHOD(CVector::New) {
  NanScope();

  if (args.IsConstructCall()) {
    // Invoked as constructor: `new MyObject(...)`
    double value = args[0]->IsUndefined() ? 0 : args[0]->NumberValue();
    CVector* obj = new CVector(value);
    obj->Wrap(args.This());

    NanReturnValue(args.This());
  } else {
    // Invoked as plain function `MyObject(...)`, turn into construct call.
    const int argc = 1;
    Local<Value> argv[argc] = { args[0] };
    Local<Function> cons = NanNew<Function>(constructor);
    NanReturnValue(cons->NewInstance(argc, argv));
  }
}

NAN_METHOD(CVector::PlusOne) {
  NanScope();
  CVector* obj = ObjectWrap::Unwrap<CVector>(args.Holder());
  obj->value_ += 1;

  NanReturnValue(NanNew(obj->value_));
}

NAN_INDEX_SETTER(CVector::IndexedSet) {
    NanScope();
    cout << "NAN_INDEX_SETTER" << endl;
    NanReturnNull();
}

NAN_INDEX_GETTER(CVector::IndexedGet) {
    NanScope();
    cout << "NAN_INDEX_GETTER" << endl;
    NanReturnNull();
}

NODE_MODULE(cvector, CVector::Init)

this link: https://developers.google.com/v8/embed#accesssors appears all over the place. it's broken.


Feat/interactive founder hover card

https://github.com/HimanshuNarware/CareerZunction_Intern/issues/188 [Feature Request]:Adding the eye-catchy hover effect to the founder card
Feat https://github.com/HimanshuNarware/CareerZunction_Intern/issues/188
under gssoc'24
updation done on issue #188 and pull request #284
https://github.com/HimanshuNarware/CareerZunction_Intern/assets/116994649/0e5d20d9-fb93-4570-a937-d80fead42b76

https://github.com/HimanshuNarware/CareerZunction_Intern/pull/332  done this

mark the label as per gssoc'24

mark label and level to count under gssoc'24


darwin: fix generate-sdk-packages.sh

Description of changes
The script is broken, since https://opensource.apple.com/text/macos-1101.txt is no longer available.
Also, #201348: the packages seem to have different sha256 now.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1545

One nice addition would be printing usage information instead of erroring when run without arguments. I'm OK with merging this as-is though.

@wegank, would that be a simple addition?

@toonn Sorry, I'm not good at writing docs. Only added a dummy one.


cutoff ratio of excluded volume

magic number 0.4 is used in the function calc_force().

the numbers are removed.


http requests for monitoring

number of unique tunnels
number of tunnel connections
number of pending requests
number of completed requests
number of errored requests

+1


Update the notebooks

Update the notebooks with the most recent code.

[ ] KBG
[ ] SUOX
[ ] MAPK8IP3
[ ] RPGRIP1

TODO

make a folder with 3 notebooks
the folder has a subfolder phenopackets

KBG
SUOX
...

TODO - finish


Docs: new design

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

For the sake of confrontation, I think Vue 2.0 can also be Vue 2.


Fallback handler could not load library

I wanna start this off by saying I appreciate your / you guys' (unsure) work a ton!
There's a bug that's been plaguing my game ever since I was using the FakeByte version; when the transport loads the .dlls, it doesn't actually sanitize the filepath; should your filepath have a semicolon at any point in any place that leads to the dlls, the entire loading procedure fails and EOS is terminally broken.
#2 Reproduction steps:

Have a game with an EOS Transport
Set any folder names to contain a ';' char (image 1)
Open the game and see the plugin fail to load dlls

Stacktrace of what exactly is broken by it attached in my player.log after launching the game like this.
Player.log

There is some more edge cases that also make this loading prodecure fail; Sometimes the game has to be launched in an administrator mode or this will fail; There is also one (or more) scenario where this fails that I have yet failed to reproduce, (so I wasn't able to help my user even with all of the steps aformentioned and the game just doesn't work for them).

If you'd like to chat further with the use that has had this problem unpreproduced, here is a discord thread about it, where I failed to help them sadly. I could also show you in detail how I have all this set up, should it help. This has happened to them on my previous release (where the prevailing transport was that of FakeByte), but as far as I understood, the loading code is the same here and there. I can also provide them with my indev release where I have the not newest, but a recent version of your plugin installed instead.


pull request for e2e test at 12/19/2016 08:26:18

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Precompilation warning about identifier conflict

On current Decapodes.jl main:
  1 dependency had warnings during precompilation:
┌ Decapodes [679ab3ea-c928-4fe6-8d59-fd451142d391]
│  WARNING: using LinearAlgebra.⋅ in module Decapodes conflicts with an existing identifier.
└

I believe that Decapodes does not need to export \cdot anymore, so I am going to quickly check whether a "one-line fix" can be performed.


feat: (5291) Add section for doing CRUD on product tags (Folksonomy)

What
Adds UI support for performing CRUD operations on product tags using the Folksonomy api. Targeted fields on a tag are currently k, v and comments properties. Adds a new card to product page, but only shows the first 5 tags if there are more than 5 as the api does not paginate. Clicking into the card will show the full list where you add, edit and delete tags.

Screenshot

Part of

#5219

@g123k @Pierre-Monier Do you mind taking a look at this and see if I am going in the right direction or if theres any changes you think we need here. Put the PR in Draft for now.

Visually we'll need to adapt it to the new design that just landed, but we will do that in a subsequent pr.

@gciluffo What if the device is offline? Do you cache the values? Do you make it possible to edit values even if offline?

Little merge conflict to solve @gciluffo

@g123k @monsieurtanuki I think its ready for another review. Updated it with the latest styling changes. Still trying to figure out the internationalization. Not sure if the verbage I have in this is what we want. In other words is the term "Product Tags" what we want to use?

@monsieurtanuki Fair question, I have not taken into account connectivity issues for this yet. Is there a pattern already established in the app for syncing data when going online/offline that I can look at? Are most read/write features in the app able to sync when online/offline? (im a bit new to this code base). I could address this in a followup PR.

It would indeed be more appropriate in another PR.
We store product data in a local sqflite database table, write product data through what we call "background tasks", and cache pending product edits in another table of the database.
We'll see that later.

@g123k @teolemon I think its ready for another review. Updated it with the latest styling changes. Still trying to figure out the internationalization. Not sure if the verbage I have in this is what we want. In other words is the term "Product Tags" what we want to use?

@gciluffo I have zero interest in UI/UX (or let's say I have lower standards than demanded here on that project), so as long as the offline code is another PR I'll have a look at that other PR instead ;)

@gciluffo 2 little merge conflicts to solve.
@g123k @monsieurtanuki this look okay to me, perhaps as an enable option (that we can add after this is merged), so that we can get initial feedback and iterate a bit ?

@teolemon As already said I'll focus later on the offline mode for folksonomy, not on the UI. That said, an enable option in dev mode would indeed make sense.

@teolemon Updated

Codecov Report
Attention: Patch coverage is 0% with 205 lines in your changes missing coverage. Please review.

Project coverage is 6.86%. Comparing base (4d9c7fc) to head (842899e).
Report is 469 commits behind head on develop.

Files with missing lines
Patch %
Lines

...ooth_app/lib/pages/folksonomy/folksonomy_page.dart
0.00%
86 Missing :warning:

...ooth_app/lib/pages/folksonomy/folksonomy_card.dart
0.00%
55 Missing :warning:

..._app/lib/pages/folksonomy/folksonomy_provider.dart
0.00%
52 Missing :warning:

packages/smooth_app/lib/pages/folksonomy/tag.dart
0.00%
11 Missing :warning:

...b/pages/product/product_page/new_product_page.dart
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5768      +/-   ##
==========================================
- Coverage     9.54%   6.86%   -2.69%
==========================================
  Files          325     426     +101
  Lines        16411   23434    +7023
==========================================
+ Hits          1567    1609      +42
- Misses       14844   21825    +6981

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

@g123k @monsieurtanuki Really appreciate the feedback, thanks for dealing with my Flutter noob-ness.

@monsieurtanuki @g123k Not sure if totally understand the concept of "dev mode". Is there an example PR I can look at to get idea of what I need to do? Thanks.

@monsieurtanuki @g123k Not sure if totally understand the concept of dev mode, is it sort of like a feature flag? Is there an example PR I can look at to get idea of what the setup looks like? Thanks.

Go to the user preferences dev mode file, just add an entry for a bool, and use this bool value to enable folksonomy or not.

@monsieurtanuki @g123k Not sure if totally understand the concept of dev mode, is it sort of like a feature flag? Is there an example PR I can look at to get idea of what the setup looks like? Thanks.

Go to the user preferences dev mode file, just add an entry for a bool, and use this bool value to enable folksonomy or not.

Got it thanks. Updated.

Got it thanks. Added dev mode flag.

Thank you @gciluffo! I think the default bool value should be "don't show folksonomy", so that only advanced users use folksonomy in a first phase.

ok, congratulation on your first (and substantial) contribution to @openfoodfacts @gciluffo 🍾


Documentation for testing in jsdom environments

🙋 Documentation Request
I recently ran into an issue where unit tests were failing after a certain threshold of items was added to a Spectrum ListView component. The underlying issue turned out to be that useVirtualizerItem depends on scrollHeight which always returns 0 in a jsdom testing environment. This was causing "Maximum update depth exceeded" errors in tests. I was able to resolve it by mocking HTMLElement.prototype.scrollHeight, but it took some time to figure out what the underlying issue was.
It would be helpful for the testing docs (https://react-spectrum.adobe.com/react-spectrum/testing.html) to include additional details of any limitations / workarounds that might exist for such cases.
🧢 Your Company/Team
No response

In progress @ https://github.com/adobe/react-spectrum/issues/6146. Preview here.

Great, looks like the #virtualized-components section was exactly what I needed.

Great! Closing as a duplicate of https://github.com/adobe/react-spectrum/issues/3703


Fix CocoaPods Build issue

This fixes the issue building with CocoaPods in issue #2.
The #include <CommonCrypto/CommonCrypto.h> line needed to be moved from the header file to the implementation file to remove the build error.
I've tested this in my application and it works as expected.

I'm not 100% sure what's going on.
Obfuscator.m includes Obfuscator.h which includes <CommonCrypto/CommonCrypto.h>
I'm not sure why <CommonCrypto/CommonCrypto.h> must get moved to Obfuscator.h?

To clarify. It must be moved FROM Obfuscator.h not to it. It looks like it's a module-include error that is related to the way in which CocoaPods links system frameworks. I can't tell you exactly why this works, but I can guarantee you that it does. I've already archived and shipped a version of my application using this fix.

Is this PR going to be merged or rejected? I'd really like to move off of using an explicit dependency on my fork of this repository, but the current pod version is not building for me with Swift projects.


Bring “Page Tables” post up to date again

This PR:

removes the testing of the translate method since the stack overflow no longer occurs (as we're using a 16KiB stack now)
removes an unneeded unsafe block
updates the x86_64 crate to version 0.1.2, which provides the int3 function that we need in “Handling Exceptions” (this was reported in #303)

Travis fails because of https://github.com/rust-lang-nursery/rustup.rs/issues/1062. It works locally, so I merge it.


Draft: Support untyped nodes

Related to https://github.com/microsoft/kiota-abstractions-dotnet/pull/177

Support for untyped nodes without extending interfaces IParseNode and ISerializationWriter.
Added tests for serialization/deserialization, both tests passed.

TODO before merge

Bump abstractions dependency to 1.8.0 after merge of https://github.com/microsoft/kiota-abstractions-dotnet/pull/177
Cleanup csproj files to remove local refs
Bump version and add changelog entry

@andrueastman Hi, my plan was to bump dependency to 1.8.0, clean up csproj etc. and create a new pull request and close this draft.

No worries @MartinM85 . We can go with that plan after https://github.com/microsoft/kiota-abstractions-dotnet/pull/177 is merged.

@andrueastman I've merged with master and I'm referencing Microsoft.Kiota.Abstractions 1.8.0

There are a couple of comments from @baywet still pending. Any chance you can resolve that before the merge?

@andrueastman Hi, my plan was to bump dependency to 1.8.0, clean up csproj etc. and create a new pull request and close this draft.

Also, do you still plan on creating a new PR or should we still use this one and mark this ready for review?

@andrueastman I will create a new PR.


Hide dropdown.

Fixes #15, could use a little CSS animation tuning (disappears instantly, not exactly sure why, but I may have been editing peacock's nav.less accidentally)

:+1:


[$500] Chat - The bottom of the action and emoji popups are not in line with each other

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to any chat
Open the action menu from the composer
Open the emoji menu

Expected Result:
They should be in line above the composer.
Actual Result:
The bottom of the action and emoji popups are not in line with each other. The emoji one overlaps the composer a little bit.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e967311edf8b5655
Upwork Job ID:<PHONE_NUMBER>246729728
Last Price Increase: 2024-01-10

Proposal
Please re-state the problem that we are trying to solve in this issue.
The bottom of the action and emoji popups are not in line with each other
What is the root cause of that problem?
EmojiPicker calculate the posirion of tha anchor and display the UI just above it.
What changes do you think we should make in order to solve the problem?
On the function calculateAnchorPosition
https://github.com/Expensify/App/blob/f15ed75b0b4133a8c02617d20a09e29d72825a4d/src/components/EmojiPicker/EmojiPicker.js#L71-L78
We need to:
1- check if the anchor is the one on the input on the buttom.
The function is used with many emojies and in alot other places in the UI, if we change it without checking, all the other emojies will be displayed in a wrong position:
if (emojiPopoverAnchor.current.getAttribute('aria-label') ===  'Emoji') {
 position.vertical  -=  12;
}

2- Adjust the values of the new anchor position:
setEmojiPopoverAnchorPosition(position);

The final code will be here:
calculateAnchorPosition(emojiPopoverAnchor.current, anchorOriginValue).then((value) => {

  const  position  = {
    horizontal:  value.horizontal,
    vertical:  value.vertical,
  };

  if (emojiPopoverAnchor.current.getAttribute('aria-label') ===  'Emoji') {
    position.vertical  -=  12;
  }

  setEmojiPopoverAnchorPosition(position);
});

What alternative solutions did you explore? (Optional)
We can change the position from a parent component.

I had never noticed this either—doesn't seem like a super huge deal to me but I also don't really see the harm in making them the same. Curious if the other designers have any other or additional thoughts.

I feel the same as Danny.

Yeah, I'm with Shawn and Danny, don't feel too strongly but doesn't hurt making them the same.

@ntdiary what do you think of the proposals - anything we can implement?

@AndrewGable yours to sign off ^^

@suneox - Solution looks good, but I'd rather do this math in the code instead of having a magic value (e.g. -14 or -12) in case any of these values change.

@ntdiary bump thanks!

Payment Summary:
Contributor @suneox $500 payment issued in Upwork
C+ review @ntdiary @$500 payment issued in Upwok

@ntdiary please advise on regression steps thanks


ARROW-9814: [Python] Fix crash in test_parquet::test_read_partitioned_directory_s3fs

Failures are brew installation failures, so ignoring those

you should either pass a URI ("s3://...) and no filesystem, or either a filesystem object with a "normal" path (without scheme prefix), but never the combo of filesystem object ànd a URI

Right!


Conda install error for python 3.8

@tlambert03 CI failed when I merged the autoupdate PR. It seems like the conda on macos-latest doesn't hold a gurobi that is compatible with python 3.8. Should we just stop supporting/testing 3.8? Pin the macos version? Something else?
https://github.com/funkelab/ilpy/actions/runs/8882608765/job/24387679572

it's likely due to the recent change in the macos-latest runner on github (a couple weeks ago they moved to silicon-based runners and it requires a number of changes to be made).  fixing in #61


feat: Added Rectangle, RoundedRectangle, Capsule, Circle

// ShapesSection.tsx

import React from 'react';
import {
  Rectangle,
  RoundedRectangle,
  Capsule,
  Circle,
  HStack,
  List,
} from 'swiftui-react-native';

/**
 * A list of SwiftUI Shapes
 */
export const ShapeSection = () => {
  return (
    <List inset header="Shapes">
      <HStack>
        <Rectangle width={50} height={100} backgroundColor="pink" />
        <Rectangle width={20} backgroundColor="blue" />
        <Rectangle width={50} height={50} backgroundColor="red" />
        <Rectangle height={150} backgroundColor="green" />
      </HStack>
      <HStack>
        <RoundedRectangle
          cornerRadius={10}
          width={50}
          height={100}
          backgroundColor="pink"
        />
        <RoundedRectangle cornerRadius={2} width={20} backgroundColor="blue" />
        <RoundedRectangle
          cornerRadius={10}
          width={50}
          height={50}
          backgroundColor="red"
        />
        <RoundedRectangle
          cornerRadius={10}
          height={150}
          backgroundColor="green"
        />
      </HStack>
      <HStack>
        <Capsule
          cornerRadius={10}
          width={50}
          height={100}
          backgroundColor="pink"
        />
        <Capsule cornerRadius={10} width={20} backgroundColor="blue" />
        <Capsule
          cornerRadius={10}
          width={50}
          height={50}
          backgroundColor="red"
        />
        <Capsule cornerRadius={10} height={150} backgroundColor="green" />
      </HStack>
      <HStack>
        <Circle radius={100} backgroundColor="pink" />
        <Circle radius={20} backgroundColor="blue" />
        <Circle radius={50} backgroundColor="red" />
        <Circle backgroundColor="green" />
      </HStack>
    </List>
  );
};

Could use some assistance getting this in better shape (get it? 😄):

How shapes are done under the hood is quite hacky.
Had some trouble getting shapes to just fill all available space to them, so instead I required width/height in props.
Circle I used radius in the props, which is not SwiftUI-like. But it was the best I could do to get predictable behavior.

Here it is in SwiftUI for frame behavior comparison:

Had some trouble getting shapes to just fill all available space to them, so instead I required width/height in props.

Are we able to use the frame prop to supply the width and height arguments? I think SwiftUI is using frame in your example and I would like to stay as consistent as possible.

Circle I used radius in the props, which is not SwiftUI-like. But it was the best I could do to get predictable behavior.

For this, I think using frame.width/2 as a border radius will always guarantee that its circular, but let me know if that doesn't work.
Other notes:

Can RoundedRectangle use Rectangle under the hood to avoid duplication? Basically use the Rectangle component with a default radius of 15.
When you want to do default values, I think its easier to read if you supply the default values when you destructure the props in the function params. Like:

export const Capsule: React.FC<CapsuleProps> = ({
    backgroundColor,
    cornerRadius = 15,
    opacity,
    width,
    height,
    scaleEffect,
    padding,
    border,
    shadow,
    zIndex,
    style,
    onAppear,
    onDisappear,
}) = { ... }

@andrew-levy

Are we able to use the frame prop to supply the width and height arguments? I think SwiftUI is using frame in your example and I would like to stay as consistent as possible.

Definitely, I think that would be most SwiftUI-like. How would we be able to fill all available space by default if frame is not specified? That's the piece that makes complex shape composition possible/powerful. You can just put a frame on the outermost portion, and pad or stack inwards as needed.

For this, I think using frame.width/2 as a border radius will always guarantee that its circular, but let me know if that doesn't work.

See a few lines up as to how we know the width, but yes. If the shape fills available space, you'd take Math.max(frame.width,frame.height)/2 so a full circle is always showing.  Capsule is almost the same, but Math.min instead of max.

Can RoundedRectangle use Rectangle under the hood to avoid duplication? Basically use the Rectangle component with a default radius of 15.

Yupp!

When you want to do default values, I think its easier to read if you supply the default values when you destructure the props in the function params

Roger that!

Can you provide an example of what you mean by filling the space if no frame is provided in swiftui?

Can you provide an example of what you mean by filling the space if no frame is provided in swiftui?

Here's where we are now. SwiftUI shows "partial framing"

I recommend puting this code into an empty SwiftUI app's ContentView to see the behavior. When you click on a shape, refer to it's highlighted frame on the simulator preview panel.
 Thanks!

Try this SwiftUI Code out with the Preview open to see some of the behavior a bit better:
struct ShapeBehaviors: View {
    var body: some View {
        List {
            Section("Rectangles") {
                ShapeCases(s: Rectangle())
            }
            Section("Circles") {
                ShapeCases(s: Circle())
            }
            Section("Capsules") {
                ShapeCases(s: Capsule())
            }
            Section("Icon we mentioned") {
                ZStack {
                    Circle()
                        .stroke(Color.green, lineWidth: 10)
                    HStack(spacing: 5) {
                        ForEach(0..<6) { i in
                            Capsule()
                                .padding(.vertical, [10,15,20,25].randomElement()!)
                        }
                    }
                    .padding()
                }
                .frame(width: 100, height: 100)
            }
        }
    }
}

fileprivate struct ShapeCases<S: Shape>: View {

    let s: S

    var body: some View {
        HStack(spacing: 0) {
            s
                .fill(Color.pink)
                .frame(width: 50, height: 100)
                .border(Color.black)
            s
                .fill(Color.green)
                .frame(height: 150)
                .border(Color.black)
        }

        HStack(spacing: 0) {
            s
                .fill(Color.pink)
                .border(Color.black)
            s
                .fill(Color.green)
                .border(Color.black)
        }

        ZStack {
            ForEach([0.0,1,2,3,4,5,6,7,8,9,10], id:\.self) { i in
                s
                    .fill(Color.blue.opacity(0.1*i))
                    .padding(5*i)
                    .border(Color.black)
            }
        }

        HStack(spacing: 0) {
            VStack(spacing: 0) {
                s.fill(Color.pink)
                s.fill(Color.green)
            }
            s.fill(Color.pink)
            s.fill(Color.green)
        }
    }
}

struct ShapeBehaviors_Previews: PreviewProvider {
    static var previews: some View {
        ShapeBehaviors()
    }
}

Getting somewhere 🚀

What do you think about accepting a UIColor | string for fill? I don't think you get code completion on fill atm.

if so, can we add UIColor.primary and put it for the default fill of Rectangle?

Let's make sure we document the behavior of circle width/height when time comes. But I like how it's working.

What do you think about accepting a UIColor | string for fill? I don't think you get code completion on fill atm.

Can you show me how you are not getting code completion? I'm able to get it working.

if so, can we add UIColor.primary and put it for the default fill of Rectangle?

Yeah we can add this.

Let's make sure we document the behavior of circle width/height when time comes. But I like how it's working.

Definitely!

What do you think about accepting a UIColor | string for fill? I don't think you get code completion on fill atm.

Can you show me how you are not getting code completion? I'm able to get it working.

The way it's currently set up is using the UIColor object. So if you wanted primary you would have to import the object or use the hook, then do UIColor.primary. Currently there's logic to handle the case where you just put the string "primary". Though we can add this! Does that make sense kinda?

Makes sense. Are you open to converting UIColor to an enum?

That might conflict with how the whole dynamic color theming works right now, but I will create a separate issue for this so we can brainstorm.

Do you want to take first crack at README updates for shapes, or should I?
I suppose we may want to do rotationEffect as well.

Go for it. I think i added the part for rotationEffect in the last PR but i don't remember.

Start with that for now, but I'm hoping to talk you into an example snippet for them.

fill is required on Capsule but not the others it looks like, did we mean to make it optional?


Remove Content-Length header from proxy inspector response

Summary:
This change removes Content-Length header from proxy inspector response.
The presence of this header was resulting in the response being cropped under some circumstances because of erroneously calculated length.
The Content-Length header value represents the number of bytes in the response. In the code, string.length was used to calculate that value, but in JavaScript it gives the number of characters in a string instead of its size in bytes. Specifically, if there are some UTF characters in the string that occupy more than byte, there would be a mismatch in this size. This mismatch resulted in the response being cropped.
The easiest way to reproduce this problem is to set the simulator name to contain a two-byte UTF character.
This change works according to the HTTP spec, which states that when Content-Length is not present, the end of the response stream indicates the end of the response. Since in the code response.end(data) is use, it terminates the stream and hence there is no need to provide the length in the header.
Changelog:
[GENERAL] [FIXED] - fix issue with debugger not working when device name contain two-byte UTF characters
Test Plan:

Change your iOS simulator name to contain some two-byte UTF character (for example this one: "–")
Run metro and connect your app with it
Go to http://localhost:8081/json/list in your browser – see the response being marked invalid as it is cropped
Apply the change and see that the resulting JSON in the response is now correct
Open debugger workflow to confirm it sees the connected device

Thanks @kmagiera, appreciate the fix!


[BUG] Modify the README uninstallation section in the chart available for rancher v2.5

Describe the bug
The chart's README uninstallation section should be modified in the chart https://git.rancher.io/charts branch - dev 2.5 as it says From Rancher UI, navigate to Catalog Apps tab and delete Longhorn app.
https://github.com/rancher/charts/blob/dev-v2.5/charts/longhorn/README.md
The above chart is used in the rancher Dashboard where there is no catalog apps tab present and README says to delete the app from the catalog apps tab.

Validation: PASSED


Format code with black and isort

This is a proposal to switch to black (and isort, as black does not sort imports yet.)
@danielmitterdorfer said this in #1276:

black is on our radar but I fear that we won't have much capacity at the moment to work on it.

I'm not sure whether it was the actual pull request that was the issue, or the work needed to adapt (eg. everyone will have to configure their editors). My experience is that black allows to focus on the interesting parts of the code, which is highly valuable. Here's what Mike Bayer, creator and maintainer of SQLAlchemy says about black:

I would say that I can't think of any single tool in my entire programming career that has given me a bigger productivity increase by its introduction.   commits like https://github.com/sqlalchemy/sqlalchemy/commit/ebbbac0a76b3327a829864afb26ee1b7ff1dc780, Black did 90% of the work for me that previously we'd spend days formatting by hand

This time, lint passes (I was confused because lint fails on master with Python 3.9). The actual black run is in its own commit, so this pull request is easy to rebase. We can also tell git to ignore the black commit in git blame.

@elasticmachine test this please

I'm not sure whether it was the actual pull request that was the issue, or the work needed to adapt (eg. everyone will have to configure their editors).

It's actually a combination of both. :) Thanks for raising the PR to move this forward.

I fixed the pytest.mark.benchmark formatting, applied your suggestions to the Makefile, explained how to configure black and isort, and fixed the conflict.
Please take another look

@elasticmachine test this please

We're down from 17k new lines to 11k new lines thanks to the line length fix. 🎉 This is ready for another round of review!

@elasticmachine test this please

Wow, you really left 200+ comments! I would have been happy with only your main review. I'll fix everything, but not sure when it will be done, maybe next week.

If it's possible with black I'd probably even go as far as to exclude these files from formatting entirely (although there are places where black has improved formatting IMHO).

It's certainly possible to give to the black tool a specific list of files in make lint and make format, but then everyone would have to try to configure their editor to do that, so I'd rather avoid that and just put the hard work now. (Even if I have to admit this is more involved than I would have thought.)

Wow, you really left 200+ comments! I would have been happy with only your main review. I'll fix everything, but not sure when it will be done, maybe next week.

Honestly, I was also surprised that it added up to that many comments but I wanted not only mention the high-level feedback but also provide specific guidance around what formatting I think needs to be kept. Nevertheless I suggest you focus on my high-level comments; no need to address each and every one of them.

If it's possible with black I'd probably even go as far as to exclude these files from formatting entirely (although there are places where black has improved formatting IMHO).

It's certainly possible to give to the black tool a specific list of files in make lint and make format, but then everyone would have to try to configure their editor to do that, so I'd rather avoid that and just put the hard work now. (Even if I have to admit this is more involved than I would have thought.)

Good point, let's not exclude certain files then. The intention behind my proposal was to avoid that you need to manually take care of formatting of the many dicts that we use in the tests in these files.

I addressed comments in all files with less than 1k lines, will do the rest tomorrow.

@elasticmachine test this please

@elasticmachine ok to test

Thanks! I'm finally ready for another iteration. I marked all comments are resolved as it was the only to keep track of them, but I can unresolve them if you want me to.
When the pull request looks good to you, I'd like to rebase it on top of master to avoid messing up too much with the history.

(The tests ran fine but GitHub does not know it.)

Thank you for your review @dliappis!
@pquentin with two approvals on this PR we should be good to move forward. Can you please, as a last step before we merge this, push this as two commits as I've described above? As soon as this is done and the CI checks ran again, I'll merge this. Once more, thank you very much for all the effort that you put into it!

@dliappis Thanks for the review!

I currently use Monaco on macOS, and am not familiar with emoji handling on Linux, but I think you need a fallback emoji font such as noto-fonts-emoji. I have ordered a Linux laptop, so I might face the same issue at some point! This does display correctly in the CI, though:

Another option is to use --quiet but I think the emojis are a nice visual way to see success (when they work!).

Sure, I'll add a check-all command. But I really recommand to configure black and isort in your editor (even though black is slow on the largest files in the repo).

@danielmitterdorfer Sure, I'll do that. I'll also be happy to help with rebasing any outstanding pull requests that will be affected by this.

@danielmitterdorfer Done!

And now that CI has passed, it's merged. Thanks for your massive effort and persistence @pquentin!

🎉
Thank you! I was happy with my initial commits, and had no idea of the value of going through every single change like you did. You estimated the effort to do this much more accurately than me, even though I did that multiple times in the past. I'm really impressed.


Moving folders

Hey, i miss the feature to move a folder to another location or move it as sub-folder under another folder.

Hi, is that even possible within the IMAP specification?

It should definitly be possible because it is also possible in many
other webmail or desktop clients. :)

Perhaps they just create a new folder, move the messages and then remove the old folder. Something you could do yourself until the feature gets implemented if it gets enough votes at http://rainloop.uservoice.com/
If you did not register the feature request there, please go ahead and do so. This Github page is more for bugs then requests.

It's just a rename not a move
Eg.  Inbox.myfolder renamed to inbox.newfolder
Inbox.anotherfolder to inbox.newfolder.anotherfolder
Sent from my iPhone

On 11 Feb 2016, at 7:51 PM, Ralf<EMAIL_ADDRESS>wrote:
Perhaps they just create a new folder, move the messages and then remove the old folder. Something you could do yourself until the feature gets implemented if it gets enough votes at http://rainloop.uservoice.com/
If you did not register the feature request there, please go ahead and do so. This Github page is more for bugs then requests.
—
Reply to this email directly or view it on GitHub.

Any chance this will be added?  I move folders all the time, depending on if a project or client is active or not.
Thanks, and great job... been using for a few weeks now and enjoying it.

Still using rainloop, but having to open roundcube to change the folder locations to organize my mail every week.
It would be nice to hear back from someone to see if this in the works or not.
Thanks again.

This feature is very standard on all mails clients.. must be added to rainloop too!
Saludos!

Is this really still not a part of Rainloop? This is a very integral part of an email client.

bump.. to the top .. +1

I'd also like to bump this. I think it's a reasonable functionality that any other mail client has and since Rainloop imho is the best of them from a usability point of view, i think it would be a great benefit to provide this functionality. Especially if you're coping with large amounts of mail, it's quite usual to move around folders all the time.
I'm a little bit afraid that this will not get enough traction on uservoice since it's somehow an integral part of a mail client and not anything fancy or special..

Anyone minds if I implement this showing a simple dropdown for new parent selection in a dialog just like when you create a folder and submit it as a PR? I have a roughly working prototype but I'm not sure about development standards since I was not able to find any kind of developer documentation on this...

Is someone still working on that?

How are you doing @dprandzioch ?


drivers: watchdog: Watchdog API redesign [RFC]

New API enables setting watchdog timeout in the unit of microseconds.
It is possible to configure watchdog timer behavior in CPU sleep state
as well as when it is halted by the debugger.
It supports watchdogs with multiple reload channels.
Fixes #3994
Signed-off-by: Michał Kruszewski<EMAIL_ADDRESS>

Thanks for taking a look at the API. The cycles-vs-timeout thing is really welcome.
It would be nice to set and get values that are retained during a reset caused by the watchdog.  Some devices support this (Quark and ESP32 AFAIK).  This is useful specially for testing purposes.
The ESP32 watchdog offers a way to determine the boot reason, as well, but I'm not sure if this fits in a generic watchdog interface.

I have one more comment. Currently watchdog API is missing from the official Zephyr API Documentation. I think this PR should also update one of the files in doc/api/. I'm however not sure what's the best section to put it.
We should also run doxygen to verify that all doc strings are rendered as expected. I couldn't find the official documentation explaining how it should be done. As far as I remember one has to run
make -C doc htmldocs

from Zephyr's home directory. There may be some dependencies which need to be installed.

$ pip3 install --user -r scripts/requirements.txt
that will install all deps needed to build documentation

https://github.com/zephyrproject-rtos/zephyr/blob/master/doc/README.rst  tell how to setup and build documentation

P.S. Most of the English "issues" are around missing articles ("the", "an", "a") or language that's a bit too formal or passive, "it is required to install" can just be "install".  Otherwise things look pretty good and  I appreciate that you're writing documentation.

@nashif @mnkp @durub I have applied your concerns.

do not rename the current test (wdt_deprecated), leave it as is and it should be fixed to use the new APIs and probably made more generic, right now it is very qmsi specific.

At this point, I feel like this is now 1.11, any concerns related to pushing to 1.11?

This PR is all but ready. As far as I can tell all issues have already been addressed, only not yet approved by respective reviewers. It would be great if everyone followed up on the issues they reported. The status of the PR would be clearer.
That said it probably doesn't make sense to merge this PR in 1.10 if there are no drivers which implement it. If no one reports new issues we should merge this PR as soon as the window for 1.11 release is open. This will give us enough time to update the watchdog drivers. Or should the process be different?
There is also a question on how we should split the work. Shall we create an issue for every watchdog driver that needs to be converted to the new API so any developer interested in working on the task can take ownership of it?

Looks like this is not getting into 1.11 (it's in pre-release freeze already). Changing milestone to 1.12.

this was superseded by another PR


Overview page requires multiple refresh attempts before provider data is displayed

Describe the bug
When accessing the overview page after first login, the "Add Provider" page is displayed indicating there is no provider data even though one or more providers returned by the api. The page requires 2-3 refresh attempts before the provider data is displayed in the UI
To Reproduce
Steps to reproduce the behavior:

Login to Insights and access Cost Management
Overview page only displays text for adding a provider because no provider was found
Refresh the page 2-3 times to populate the UI w/ provider data

Expected behavior
Provider data is queried and displayed when providers are present
Desktop (please complete the following information):

Browser Chrome 69 and Firefox 62

Additional context
Once data is cached in the session there are no issues

This is an issue with the Insights platform related to tokens

Insights team has provided a promise that we can wrap our API calls in to make sure we always have a token:
https://docs.google.com/document/d/1xjITNYbp1aw9Xu3lYLQppKQl-Rtj6_BaThwy1Zkazm4/edit#heading=h.8aucqk7x2oqb
We need to implement this in our code.

+1
This is still happening @dlabrecq I can show you if you need to see it

it happens the same to me

I've opened a new Insights issue.
https://github.com/RedHatInsights/insights-chrome/issues/76

This is not completely blocking UI automation but is holding it up a lot, is there anything we can do to resolve it sooner? I will raise priority to high.

I messaged into frontend channel on slack to try and get some eyes on the issue.

As mentioned in https://github.com/RedHatInsights/insights-chrome/issues/76
I believe the cause of this is the timeouts between the frontend and the backend.
We can solve this in two ways:

Stretch the timeout for 7 seconds. This is good for short-term as this might fix the problem for 90% but what happens if the latency will be higher. In addition, this is not really user friendly to wait longer.

Try to fetch the data in 3 tries and if it keeps failing display an error message with the option to try again. For example, if in the first try the request failed due to timeout a new request will be sent. If the error is not because of timeout or this was the 3rd try - the error will be displayed (f.e. "Failed fetching providers data from sever due to ...") with an action button to try again and fetch the data. This option looks good to me because it is more robust and more user friendly.

WDYT?

Probably a combination of both. I'd like to see us add the retries, in addition to extending the timeout to address the latency problem. I don't know if the fetch button is necessary with the retries, but will defer to @nlcwong?

At the last November 16 Insights demo meeting the Platform team demo showed off notifications. Might be a useful mechanism to display timeout errors after we hit the retry limit.


In modded environments, custom world type additions aren't acknowledged in the world type slider

When playing with mods that add their own world type (for example, Modern Beta or BetterNether/End), the create world screen doesn't include their respective modded world type addition(s) in its selection, limiting folks who play with those sorts of mods to just vanilla's world types.

It's a known issue, because of the modified Create World screen, there is no good way to use the default world presets, due to using the vanilla Create World Screen for this, and even if I used these default world presets, modify the vanilla presets screen would be harder.

I see :0
I do hope it can get worked out soon, this whole thing's pretty exciting as a fellow Legacy Console fan :3 Take as much time as you need to figure that out

I hope so too, but it will probably take a while


Invisible file tree for FileDialog

Lorien version:

Lorien 5.0 dev (1e11cd7dcfad954219873de4e6e9c05de70617d5)
OS/device including version:

Linux (Pop OS 21.10)
Issue description:

FileDialog doesn't display anything when focused
Steps to reproduce:
Open the FileDialog and have the file tree in focus
Its because the focus style in tree for the dark theme is applied on top of the actual file tree. See https://github.com/godotengine/godot/issues/54107.
I have a fix for this and can start a pull request later.

Yeah i noticed that as well since Godot 3.4. Thanks for figuring it out!


(maint) Merge 6.4.x to master

CLA signed by all contributors.

@kris-bosland  @joshcooper I fixed the Travis build

CLA signed by all contributors.


Is motan rpc supported from skywaking version 5.x?

Hi. Is motan rpc supported from skywaking version 5.x?
Because 5.x is not stable version, I download the version 3.2.6, but not found the motan support. Then I looked over the issue history, it seems motan rpc is not supported before skywalking version 5.x according to the issue timestamp.
Expect to your answer, 3q much.

BTW, is there some docs about how to integrate motan rpc into skywalking? I still not found the motan config in application.yml from apache-skywalking-apm-incubating-5.0.0-beta2.tar.gz that the latest version, or I am in the wrong way?

I think 5.0.0-beta2 is stable enough. Feel free to use that. And for SkyWalking auto instrument, no need to set anything particular for motan, just follow documents to set agent, the motan will effect automatically.
:P Magic part of auto instrument agent.

Actually, I still want to know if skywalking version 3.2.6 support motan rpc, sorry to disturb you again, I am the obsession...3q~

I can't remember that. You can switch to that tag, if no plugin source codes, that is the answer.

I looked over the code according to your advice, version 3.2.6 support motan, and deploy it to validate, it works. 3QS a lot~~

You are welcome. Have fun in APM world.

After you used in your product or test env, welcome to share the user info with us. It is important to the project and community.


BOOST_SYMBOL_VISIBLE cannot be used for enums with Sun Studio

https://github.com/boostorg/test/blob/b9d74d1f55be2d70cdaee9bdb3ebe9445ce5c7b3/include/boost/test/detail/log_level.hpp#L25
In sun_cc, BOOST_SYMBOL_VISIBLE expands to __global. Unfortunately, that attribute cannot be used with enums.
According to the documentation, "The specifier, in this case, follows the struct, class, or union keyword"
This change effectively breaks all builds in Sun Studio
Tested with the following versions of SUNWspro:

12.3
12.4
12.5
12.6

I can send a PR to either remove BOOST_SYMBOL_VISIBLE of ifdef it based on __SUNPRO_CC, whatever works best...
I can't send a godbolt since it doesn't support sun_cc, but this snippet shows the problem:
enum __global foo {bar = 1}; int main() {}

CC main.cpp
"main.cpp", line 1: Error: Identifier expected instead of "__global".
"main.cpp", line 1: Error: "," expected instead of "{".
2 Error(s) detected.

Thanks for the report and sorry for the issue/regression. Unfortunately we do not have any regression testing for this platform.
In all cases, a PR with an ifdef would be the best. We are pretty advanced in the release process and I am not sure this will get in for 1.70. In the meantime you will have to patch or take the next master.


Could not find suitable distribution for Requirement.parse('httpsig')

Trying to use httpsig as install_requires in my setup.py but python setup.py install keeps failing with:
Could not find suitable distribution for Requirement.parse('httpsig')
Happens with both py2.7 and py3.4
Testing with:
from setuptools import setup

setup(
    name='test-httpsig',
    install_requires=[
        'httpsig'
    ]
)

pip install httpsig works fine though

pip install httpsig fails for me, looks like it's 404ing on PyPI:
Exception information:
Traceback (most recent call last):
  File "/Users/max/.virtualenvs/updownpic/lib/python2.7/site-packages/pip/basecommand.py", line 134, in main
    status = self.run(options, args)
  File "/Users/max/.virtualenvs/updownpic/lib/python2.7/site-packages/pip/commands/install.py", line 236, in run
    requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
  File "/Users/max/.virtualenvs/updownpic/lib/python2.7/site-packages/pip/req.py", line 1085, in prepare_files
    url = finder.find_requirement(req_to_install, upgrade=self.upgrade)
  File "/Users/max/.virtualenvs/updownpic/lib/python2.7/site-packages/pip/index.py", line 265, in find_requirement
    raise DistributionNotFound('No distributions at all found for %s' % req)
DistributionNotFound: No distributions at all found for httpsig==1.1.0

@mmautner what version of pip are you using? if pip -V is older than 6.0 you probably need to upgrade pip in order to get httpsig installed

thanks @tbarbugli -- that was the issue!

any plan of fixing the source of this problem? this problem really breaks libs/apps depending on httpsig.
if you release a python package and add httpsig in the requirements (setup.py) chances are that python setup.py install will fail

I'm not sure what the source of it is at the moment.  I'll look into it, but if you know, do share.

This issue occurs because there is only a wheel distributable available at pypi. pip handles the .whl package correctly but somehow setuptools refuses to install the wheel distributable. As a test I registered my own version of httpsig on pypi which is uploaded by running python setup.py sdist upload and this packages does not suffer from the same issue. Is there any reason you only distribute a wheel package, could you also publish it as a tarball?

@ahknight any update on this?

Uploaded this.  Let me know how it goes on your end.


New tokenize-complete hook

Motivation
Some languages have custom tokenizers implemented as hooks (e.g. JSX). Right now, they all use the after-tokenize hook. This is a problem because components like Markup Templating don't run this hook (because they can't, see the implementation).
Therefore, we need a new hook that gets run before after-tokenize and inside the Prism.tokenize function itself.
Description
Since the hook is supposed to be called inside Prism.tokenize, we need to change the function a little. It needs to take a new optional argument: the id of the grammar (the language string).
Only if this argument is present, will the hook be run. This ensures that the hook is only run for top-level grammars and not inside grammars.
Kinda like this:
function tokenize(text: string, grammar: Grammar, language?: string): TokenStream {
	// ...
	matchGrammar(text, tokenList, grammar, tokenList.head, 0);

	var tokens = toArray(tokenList);

	if (language) {
		var env = { tokens, language };
		Prism.hooks.run('tokenize-complete', env);
		return env.tokens;
	} else {
		return tokens;
	}
}

This approach will also make it easy to implement language-specific parsers.

Together with #2948, these languages specific parsers will also work as embedded languages.

Closed in favor of #3539.


corrected pub agent doc

Hi Joel, I am a little bit uncertain as to what to do with this one...it is a file that is in a 1.7 subdirectory. It seems like if I merge this, "bad things may happen." Let me know the best way to proceed. Maybe I am missing something here. I don't have that directory anymore in my master. Thanks!

The path was missing a slash. I have corrected this and pushed the change.


Enable new registry rename for Insteon (#17170)

## Description:
This is a simple change to the InsteonEntity what supports the unique_id property in order to allow renames with the new entity registry.
Currently the name returns a unique name thats used for the entity_id.  This is just being called now by the unique_id property in order to allow for the registry to support the rename configuration options.
Related issue (if applicable): fixes #17170
Pull request in home-assistant.io with documentation (if applicable): None
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed: None Needed
If the code communicates with devices, web services, or third-party tools: None Needed
If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @wonderslug,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Closing for Re-Run of CI after Error (not failure)

Reopen for Re-Run of CI after Error (not failure)

Name changes added to #17194 and PR #17195

I just came here to file an issue about this. I think this is exactly what I was looking for!


Warn user when generic plugin is detected in diagnose

Although "generic" network plugin works with many CNIs,
there is no guarantee that it will work with all.
So, instead of reporting success, this PR displays
appropriate warning.
Signed-off-by: Sridhar Gaddam<EMAIL_ADDRESS>

@skitt can you please review?


Documentation in spearmanr

I think it would be great if the line in spearmanr function: "Unlike the Pearson correlation, the Spearman correlation does not assume that both datasets are normally distributed" is corrected, because Pearson does not assume normality for the calculation of the coefficient, only for the p-values calculation. I think an specific mention to the p-value would avoid this source of potential confusion.
Here, commenting that Pearson's coefficient does not assume normality: https://stats.stackexchange.com/questions/3730/pearsons-or-spearmans-correlation-with-non-normal-data

Hi @davidsuculum thank you for reporting. Any documentation update is welcomed. I think we would need a reference from a reference paper since this is something people argue about.

Hi, there's this article but I can't readit: https://psycnet.apa.org/doiLanding?doi=10.1037%2Fh0048216. In Wikipedia's article you find no mention to normality of the datasets for Pearson's correlation, except when doing the inference and calculating the p-values. I have also checked in my copy of "Foundations of Agnostic Statistics" (Aronow, Miller), where the correlation coefficient is introduced and no mention to normality of the variables is commented. Could all this help?

As a descriptive statistic, Pearson correlation only measures linear association while spearmanr is for monotonic association.
Descriptive statistics like variance and correlation (pearson, spearman, kendalltau) do not depend on any distributional assumptions.
For the descriptive statistic the only relationship to normal distribution is that the conditional mean of a bivariate normal distribution is linear.
For inference like p-values the distributional assumptions are important.
(but I don't know how pvalues for spearmanr are derived.)

I don't think changing the docstring and dropping the mention of a normality assumption for the correlation itself needs a reference.
We are using (prarson) correlation on any kind of data all the time.

All right then if there is no disagreement. @davidsuculum feel free to make a PR 😃

Picking low hanging fruit: gh-16470.


[Feature] map 'select.itemStyle' color areacolor also need support 'inherit'

What problem does this feature solve?
areacolor as normal because it use visualMap color set
What does the proposed API look like?
itemStyle.areaColor='inherit'

just like https://github.com/apache/echarts/pull/15172

Set select.itemStyle.areaColor to null should work

not work, it still use the middle color of visual map to show selected area
before select

after select

configuartion

Set select.itemStyle.color to null seems work.

tks, it works


feat: add sheepla/gofind

#5757 sheepla/gofind: A command line pkg.go.dev searcher and go get helper
$ aqua g -i sheepla/gofind

Thanks.

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.39.0


docs: document stage1 requirements

Now that stage1s are swappable (with the --stage1-image flag), we need to document what the exact requirements are for people wishing to build their own stage1s. This document becomes the contract between stage0 (rkt) and stage1.
@vcaputo would you mind taking the lead on this?

sure


potential code refactor

This PR includes:

Use is instead of ==

It is recommended to use identity test ( is ) instead of equality test ( == ) when you need to compare types of two objects.

Simplify boolean expression
Remove self

The method doesn't use its bound instance. Decorate this method with @staticmethod decorator, so that Python does not have to instantiate a bound method for every instance of this class thereby saving memory and computation.

The Instagram GraphQL implementation has changed recently (as mentioned in the README) and finding and adapting to the new endpoints will take some time. Till then don't make changes to any Instagram based methods. It is still work in progress

The Instagram GraphQL implementation has changed recently (as mentioned in the README) and finding and adapting to the new endpoints will take some time. Till then don't make changes to any Instagram based methods. It is still work in progress


Feature Request: Ability to override/omit subchart templates

I’ve written a helm parent chart to wrap a stable public chart. The public chart didn’t provide for the creation of a basic auth secret, and so my wrapper chart adds that. This works well.
However, now I’d like to see some features added to the child chart — such as anti-affinity, more than 1 replica, and so on.
Is there any way from a parent chart to override/omit the child chart templates, beyond what is exposed in values.yaml? In this example, child chart doesn’t parameterize the replica integer. I can always contribute parameterization upstream, but the intent is to not fork upstream for too long. And contributing upstream has lag time. Thoughts?

Hi @Imunhatep. Have you looked at past issues/tickets asking for similar feature requests? I believe this has been asked in the past and there are some features that help address these cases such as the post-renderer flag. Hope that helps.
in any case, we ask that any proposals to the project be written up as a Helm Improvement Proposal. Have a look at the Contributing guide for next steps. Thanks!


Local (Lightweight/Transactional) DB Support.

Is your feature request related to a problem? Please describe.
Hi Everyone,
As an Enhancement, it would be nice to have support for popular local/proprietary DBMS installations to be used as Long-Term Memory Storage.
Describe the solution you'd like
Since we already store our apps. transactional/business data in a DBMS e.g: PostgreSQL , it would be nice to have the ability to use that as the AI-Assistance Memory Storage too, for sake of consistency and ease of use/deployment.
So adding support for PostgreSQL (pgvector) in combination with Chromadb specifically, will be appreciated.
On the other hand, having the ability to use some lightweight storage like SQLite will be a good option for prototyping/test development scenarios.
Describe alternatives you've considered
Letta (formerly MemGPT) has this feature by default.
Additional context
All The Best

@farzbood this repo is now deprecated. It was initially intended as a POC but has been left unmaintained since 2023. For any feature requests, please refer to the original haystack repo at a haystack . Best


Fix undefined method before_filter

Fix compatibility on Redmine 4.0.
before_filter has been deprecated in Rails 5.0, removed in 5.1 and replaced by before_action method.
Signed-off-by: Antoine Chabert<EMAIL_ADDRESS>

Why is it closed?


2FA SMS verification

Would be great to check how SMS verification works for 2fa and implement it.
Thank you!

Lol you'd probably need a 3rd party api key to framework to manage otps
On Sun, 6 Jun 2021, 12:15 pm edisax, @.***> wrote:

Would be great to check how SMS verification works for 2fa and implement
it.
Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/adw0rd/instagrapi/issues/196, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFB4NEXAHSTMJSZ4ZAM5KYDTRNKGFANCNFSM46F2E3IA
.

Lol you'd probably need a 3rd party api key to framework to manage otps
…
On Sun, 6 Jun 2021, 12:15 pm edisax, @.***> wrote: Would be great to check how SMS verification works for 2fa and implement it. Thank you! — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#196>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFB4NEXAHSTMJSZ4ZAM5KYDTRNKGFANCNFSM46F2E3IA .

Well @adw0rd already implemented 2fa with TOPT ;)

Please sponsor the development of this feature if you need it.
5 cups of coffee


Model

i can not find this file . please help me,thank you
OSError: Can't load the configuration of '/root/data/LaKo/checkpoint/vqa2.0_large_backbone/checkpoint/best_dev'. If you were trying to load it from 'https://huggingface.co/models', make sure you don't have a local directory with the same name. Otherwise, make sure '/root/data/LaKo/checkpoint/vqa2.0_large_backbone/checkpoint/best_dev' is the correct path to a directory containing a config.json file

It seems that you evaluate this model without training and saving it first?
This DIR (/root/data/LaKo/checkpoint/) should be the place where the trained model saves.

But this is the error that pops up when I run this command.  bash run_okvqa_train.sh

Maybe i need pre-train LaKo (large version) on VQA2.0 ?  How to pre-train LaKo (large version) on VQA2.0?  I need your help thank you

Maybe i need pre-train LaKo (large version) on VQA2.0 ? How to pre-train LaKo (large version) on VQA2.0? I need your help thank you

You can also train LaKo from scratch.

All the parameters in run_okvqa_train.sh are modifiable.

I got this error when I was training。 could you help me
Training | epoch 1:   0%|                                                                                | 0/1042 [00:00<?, ?it/s]
Traceback (most recent call last):
File "train_reader.py", line 271, in
train(
File "train_reader.py", line 67, in train
train_loss = model(
File "/root/miniconda3/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/root/autodl-tmp/LaKo-main/src/model.py", line 47, in forward
return super().forward(
File "/root/miniconda3/lib/python3.8/site-packages/transformers/models/t5/modeling_t5.py", line 1710, in forward
encoder_last_hidden_state=encoder_outputs.last_hidden_state,
AttributeError: 'tuple' object has no attribute 'last_hidden_state'

Have you installed Transformers with version 3.0.2?
Try pip install Transformers==3.0.2 ?
@Zhaihanlei

My version is Transformers==4.22.1

Yes, the latest Transformers have some differences from the older version.
You can build a virtual environment for testing.

t5-large-pytorch_ The model.bin link is too slow to download the model. Which folder should I put it in after I download it locally

Maybe this folder?
~/.cache/huggingface/transformers/.
You may be able to download to any path and then let the model read in the corresponding place by specifying the path. You can use Google to search for the specific method since I am a little fuzzy about this part.

Have you installed Transformers with version 3.0.2?
Try pip install Transformers==3.0.2 ?
@Zhaihanlei
I have installed transformers==3.0.2, but the following error always occurs when running the command:bash run_okvqa_train.sh.
AttributeError: module 'transformers' has no attribute 'TFFiDT5'

Please give me the complete error log.

| |
王凤娟
|
|
@.***
|
---- Replied Message ----
| From | Zhuo @.> |
| Date | 2/23/2023 12:06 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [hackerchenzhuo/LaKo] Model (Issue #10) |
Please give me the complete error log.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Please give me the complete error log.

Maybe i need pre-train LaKo (large version) on VQA2.0 ? How to pre-train LaKo (large version) on VQA2.0? I need your help thank you

You can also train LaKo from scratch by setting load_path="none";

@wangfengjuan Try this.

Maybe i need pre-train LaKo (large version) on VQA2.0 ? How to pre-train LaKo (large version) on VQA2.0? I need your help thank you

You can also train LaKo from scratch by setting load_path="none";

@wangfengjuan Try this.
If load_path="none"l, the following error will occur

Maybe i need pre-train LaKo (large version) on VQA2.0 ? How to pre-train LaKo (large version) on VQA2.0? I need your help thank you

You can also train LaKo from scratch by setting load_path="none";

@wangfengjuan Try this.
if load_path="none"，The following problems will occur again

I sincerely suggest that maybe you could first understand some of the key codes and the working principles of this paper, which can help you avoid many potential bugs and problems. For example, the issue that you are raising now can be avoided by reducing the amount of context, reducing the batch size, or switching to the base model, which are all the default principles of model learning.

Moreover, there are also some other people releasing related issues which could be available by searching the keyword in this project.

This can lighten my workload as I am also very busy recently.

Just mortifying the corresponding parameters in the bash file is enough to train VQA2.0  from scratch,  but in fact, you do not have to train VQA2.0. Only training on OKVQA is also acceptable and makes sense.


adding localisation support  - merging into 23.0 release branch

localisation enhancement -https://github.com/oracle/javavscode/issues/253
for simplified chinese and japanese

thanks for reviewing @Achal1607 & @sid-srini  will take a look and address these comments


src/render.rs: handle CfgExpr::Any(vec![])

Apparently CfgExpr::Any(vec![]) occurs in the wild.  Encountered while crate2nixifying cosmic-comp.

Thanks @amjoseph-nixpkgs. Would you mind creating a test for this? And could you do All with length 0 too?

Oh #289 right after this included All and the tests, by shear coincidence. Going with that instead of this.


React babel issue on reload

I am currently experiencing this issue with a fresh install of zero running a JSX file:
bundle.js:6450 Warning: Prop `className` did not match. Server: "MuiListItem-root Navigator-firebase-1387 Navigator-item-1385 Navigator-itemCategory-1386 MuiListItem-gutters" Client: "MuiListItem-root Navigator-firebase-25 Navigator-item-23 Navigator-itemCategory-24 MuiListItem-gutters" warningWithoutStack @ bundle.js:6450
I used a simple template from Material UI.
Initial load is fine, subsequent loads or reloads produces the above error in the console.
From investigation I believe this is an issue with babel, similar to this issue:
https://github.com/zeit/next.js/issues/7322
Any help would be appreciated.

I will have a look in some time. If you can, please share the repo so I can reproduce this quickly.

Sweet I used the code from this repo:
https://themes.material-ui.com/themes/paperbase/
https://github.com/mui-org/material-ui/tree/master/docs/src/pages/premium-themes/paperbase

While playing around again with the code I realised that after changing and saving a file the live reload / rebuild also seems to fix the issue. This is definitely something to do with the bundeler.


Loop initiziation interval greater of 1 for compiler v >=19.1

With the newer versions of the compiler (19.1 or 19.2) the II of loops is greater than 1.
This happens for both SMI device code as well as for loops in the user application.

This was due to the fmax compiler flag. Newer versions of the compiler, seem to enforce design frequency no matter what.
Removed the compilation flag in b7369d6c94c1


"The client_id parameter is missing" Error

I followed the install instructions, but when I try to make the template using the following code:
{{ vanillaforumsSSO() }}
It returns the following error:
{"error":"invalid_request","message":"The client_id parameter is missing."}
However, when I use this code:
var vanilla_sso = '{{ vanillaforumsSSOEmbed() }}';
It works correctly and returns the proper response. Tried re-entering the Client ID and Secret just in case to no avail. Any help would be greatly appreciated. Thank you, and thank you for the awesome plugin!

Sorry, somehow I missed this. Have you solved the problem?

Hi  Khalwat.
I filled the two values in the plugin.
But I got same problem after following the process step by step.
{"error":"invalid_request","message":"The client_id parameter is missing."}

It's working on my test install. Is something else somehow not set up properly?


Mongo DB Support

Love what you guys do!!!
On this current project I am on I have to use MongoDB, I am connecting with Prisma.
Within my schema.prisma file many of the associated types are not being defined by zod-prisma
Is there a workaround for this?
Here is an example of one of the missed types
type DailyReportContinent {
  AF Int
  AS Int
  EU Int
  NA Int
  OC Int
  SA Int
}

same here, any news on this?


Add python support for deployment.

LGTM!
Let's merge and release a beta version.

@smaEti
Please ask @jarqvi or @parishahla about how to deploy a beta version.


AE2 and AE2 Wireless Terminal Library block sort

Mod pack is https://www.curseforge.com/minecraft/modpacks/sky-greg with Inventory Essentials, Inventory Profiles Next, and stack refill disabled.  I added Inv Tweaks Refoxed and the sort tries to sort in wireless terminals.  Which removes armor, off-hand, and terminal upgrades.  I'm not sure what to try to add these terminals to the block list.
video clip to demonstrate what happens when sort button is clicked.  https://docs.google.com/document/d/1udOQp6H9ZR-ETgl_1bGjndapEugrz9UNphmSCoNz4S0/edit?usp=sharing

This should cover most of the AE2 terminals in the pack - replace the existing sorting.containerOverrides with this:
	[[sorting.containerOverrides]]
		containerClass = "tfar.craftingstation.CraftingStationMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "tfar.dankstorage.container.DankContainers"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "mcjty.rftoolsutility.modules.crafter.blocks.CrafterContainer"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "appeng.menu.me.items.CraftingTermMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "appeng.menu.implementations.InterfaceMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "appeng.menu.implementations.DriveMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "appeng.menu.implementations.WirelessTermContainer"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "appeng.menu.implementations.PatternProviderMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "appeng.menu.me.items.PatternEncodingTermMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "appeng.menu.me.common.MEStorageMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "gripe._90.megacells.menu.MEGAInterfaceMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "de.mari_023.ae2wtlib.wct.WCTMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "de.mari_023.ae2wtlib.wet.WETMenu"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "com.glodblock.github.extendedae.container.ContainerExInterface"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "com.glodblock.github.extendedae.container.ContainerExPatternProvider"
		sortRange = ""

	[[sorting.containerOverrides]]
		containerClass = "com.glodblock.github.extendedae.container.ContainerExDrive"
		sortRange = ""

Thank you.  That works perfectly.


Document enhanced HTTP retry environment variables

Document new environment variables for configuring enhanced HTTP retry.
Rendered: https://github.com/NuGet/docs.microsoft.com-nuget/blob/8f15af977ce90cf66b1505cbe0208bd97ad95336/docs/reference/cli-reference/cli-ref-environment-variables.md
Environment variables are named here: https://github.com/NuGet/NuGet.Client/pull/4436

Docs Build status updates of commit 8f15af9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/cli-reference/cli-ref-environment-variables.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel