[memdb] Sending multiple concurrent write relationship requests silently fails

What platforms are affected?
linux
What architectures are affected?
amd64
Steps to Reproduce
Start spicedb in serve-testing mode
Send 200 writeRelationship requests in parallel.
Expected Result
either error responses or ok-reponses and relationships are written successfully
Actual Result
The requests are responded OK. but are actually not written. The spciedb process logs
1:37PM ERR received unexpected error error="serialization error"
1:37PM ERR received unexpected error error="serialization max retries exceeded"

@svenefftinge what size does the WriteRelationshipRequest have, one single relationship?
Do the logs shared here indicate that while SpiceDB logs an error on the server-side, a gRPC code OK is returned to the client? What does the SpiceDB log for the response look like?
Which SpiceDB version is this?

@svenefftinge I'm looking into why the error is not returned to the caller, but getting additional information about scale and usage would certainly be helpful

@svenefftinge I've verified that the datastore is correctly raising the error to the API call, so it shouldn't be failing silently. I've issued a PR to make the error nicer and contain more explaination


Typo "Azure key vault"→"Azure Key Vault"

https://docs.microsoft.com/en-us/azure/key-vault/secrets/quick-create-node

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

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

Hi @hyoshioka0128. The text you update refers to instances of the Azure Key Vault service. Lowercase "key vaults" is correct, so we'll close this pull request. Thank you for contributing to the documentation.

Hi @hyoshioka0128. The text you update refers to instances of the Azure Key Vault service. Lowercase "key vaults" is correct, so we'll close this pull request. Thank you for contributing to the documentation.


why not my application show this log ,my http is not work

[55:41:912] [INFO] - com.xxg.vxinv.client.handler.VxinvClientHandler.channelRead(VxinvClientHandler.java:32) - client read len  529 , id 2
/**
 * Created by wucao on 2019/3/2.
 */

public class LocalProxyHandler extends VxinvCommonHandler {

    Logger Log = LoggerFactory.getLogger(LocalProxyHandler.class);

    private Short id;

    public LocalProxyHandler(Short id) {
        this.id = id;
    }

    @Override
    public void channelRead(ChannelHandlerContext ctx, Object msg) throws Exception {
        byte[] data = (byte[]) msg;
        Log.info("LocalProxyHandler read  Client bind write {}", data.length);
        LengthMessage message = new LengthMessage();
        message.setData(data);
        message.setId(id);
        message.setLength(data.length);
        ChannelHolder.clientHandler.getCtx().writeAndFlush(message);
    }

    @Override
    public void channelActive(ChannelHandlerContext ctx) throws Exception {
        Log.info("LocalProxy {} active", id);
        super.channelActive(ctx);

        ChannelHolder.channelGroup.add(ctx.channel());
        ChannelHolder.pm.put(id, ctx.channel().id().asLongText());
    }

}

/**

存放 channel handler 引用
*/
public class ChannelHolder {
// 和client 链接 转发流量
public static VxinvClientHandler clientHandler;
// 监听本地连接的 channelHandler
public static Map<Short, LocalProxyHandler> proxyClientHandlerMap = new ConcurrentHashMap<>(4);
public static Map<Short, String> pm = new ConcurrentHashMap<>(4);
public static ChannelGroup channelGroup = new DefaultChannelGroup(GlobalEventExecutor.INSTANCE);
}

Sorry but you will need to provide some more details about the problem and what the exact questions is.

Hello , I use Netty for TCP forwarding , But I found a problem that I couldn't solve。When my program first ran, everything was fine，
But after a few seconds, the traffic forwarding failed，I watched many times before I failed，A log will be printed before the program fails  Here is the printed log :   [55:41:912] [INFO] - com.xxg.vxinv.client.handler.VxinvClientHandler.channelRead(VxinvClientHandler.java:32) - client read len 529 , id 2
Expected behavior
I expect my program can proxy http ，But the program will fail after a while，But it was OK at the beginning。
Actual behavior
I expect my program can proxy http ，But the program will fail after a while，But it was OK at the beginning。
Steps to reproduce
Minimal yet complete reproducer code (or URL to code)
myCode url：https://github.com/vxinv/octor
Netty version
4.1.42.Final
JVM version (e.g. java -version)
java 1.8
OS version (e.g. uname -a)
mac  10.13.6

I'm sure my HTTP program won't work after the log is printed, but TCP is normal

Sorry but you will need to provide more details in terms of a reproducer etc. We can't review your whole code-base. Also you are using a very old netty version. Ensure you at least upgrade to the latest one and see if the problem persists

OK, thank you very much for your reply，But I don't know what went wrong  ，Thank you again for your reply


Tx was submitted but was not yet found on the chain

Using both the latest release and the latest commti on main (650bccd) cannot setup a channel.
$ ibc-setup ics20
Used existing connections [gaiad-microtick-testnet, connection-1, 07-tendermint-1] <=> [microtick-testnet-rc6, connection-0, 07-tendermint-0]
info: Create channel with sender gaiad-microtick-testnet: transfer => transfer
(node:451751) UnhandledPromiseRejectionWarning: Error: Transaction with ID B92AE5B89D5BF39C63D59A5E22D7272201113FCB991AA155477D2B145FEF0628 was submitted but was not yet found on the chain. You might want to check later.
Correction: The transaction eventually does appear, but not before it times out apparently

This is probably fixed when upgrading CosmJS to 0.25.3.


Modify/Remove Hook

How do you modify or remove a hook during runtime?

Browsing the codebase I couldn't see something that allows to remove or even modify them. But I would like to see that feature too, especially good for proper test-hook cleanup. :+1:

i'd also like to see this feature to remove hooks during runtime, see #701

+1 for that idea ...
Would be awesome to be able to reload a hook (change its config) without having to restart the application.

Since there is no easy way to add this without breaking the api, i ended up using a logger instance instead of the package level logger and if a restart is needed, i just recreate it and add needed hooks again.

Related(somewhat) pull request: #782
With exposed lock it will be possible to safely modify the Hooks slice.

I am writing some unit tests, and I found this method to be useful. Though not a very efficient one if you want to use it in production runtime..
type LogHook struct {
	enabled bool
	entries []*logrus.Entry
}

func (h *LogHook) Clear() {
	h.entries = []*logrus.Entry{}
}

func (h *LogHook) GetEntries() []*logrus.Entry {
	return h.entries
}

func (h *LogHook) Levels() []logrus.Level {
	return []logrus.Level{logrus.ErrorLevel}
}

func (h *LogHook) Fire(entry *logrus.Entry) error {
	if h.enabled == false {
		return nil
	}

	h.entries = append(h.entries, entry)
	return nil
}

// Make sure hook implements the logrus.Hook interface
var _ logrus.Hook = (*LogHook)(nil)

var logHook *LogHook

func enableLogHook() {
	if logHook == nil {
		logHook = &LogHook{enabled: true}
		logrus.AddHook(logHook)
	}

	logHook.Clear()
}

func getLogHookEntries() []*logrus.Entry {
	if logHook == nil {
		return nil
	}

	return logHook.GetEntries()
}

func disableLogHook() {
	if logHook == nil {
		return
	}

	logHook.enabled = false
}


#8531: delete all makefiles

@vtangTT do we still need tt_metal/tracy.mk. Should we purge it as part of this commit as well?

@vtangTT do we still need tt_metal/tracy.mk. Should we purge it as part of this commit as well?

No I don't think we need it anymore.

@mo-tenstorrent pls check


Atom install failed on arch linux

I am using arch linux with kernel 4.4.5-1 on a 64 bit machine
I am getting the following error while running
script/build
the atom editor in arch linux. I am following the following steps
https://github.com/atom/atom/blob/master/docs/build-instructions/linux.md
make: error while loading shared libraries: libatomic_ops.so.1: cannot open shared object file: No such file or directory
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 127
gyp ERR! stack     at ChildProcess.onExit (/home/hridayesh/atom/build/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at emitTwo (events.js:100:13)
gyp ERR! stack     at ChildProcess.emit (events.js:185:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)
gyp ERR! System Linux 4.4.5-1-ARCH
gyp ERR! command "/usr/bin/node" "/home/hridayesh/atom/build/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/hridayesh/atom/apm/node_modules/atom-package-manager/node_modules/keytar
gyp ERR! node -v v5.10.1
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
make: error while loading shared libraries: libatomic_ops.so.1: cannot open shared object file: No such file or directory
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 127
gyp ERR! stack     at ChildProcess.onExit (/home/hridayesh/atom/build/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at emitTwo (events.js:100:13)
gyp ERR! stack     at ChildProcess.emit (events.js:185:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)
gyp ERR! System Linux 4.4.5-1-ARCH
gyp ERR! command "/usr/bin/node" "/home/hridayesh/atom/build/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/hridayesh/atom/apm/node_modules/atom-package-manager/node_modules/first-mate/node_modules/oniguruma
gyp ERR! node -v v5.10.1
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
make: error while loading shared libraries: libatomic_ops.so.1: cannot open shared object file: No such file or directory
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 127
gyp ERR! stack     at ChildProcess.onExit (/home/hridayesh/atom/build/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at emitTwo (events.js:100:13)
gyp ERR! stack     at ChildProcess.emit (events.js:185:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)
gyp ERR! System Linux 4.4.5-1-ARCH
gyp ERR! command "/usr/bin/node" "/home/hridayesh/atom/build/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/hridayesh/atom/apm/node_modules/atom-package-manager/node_modules/git-utils
gyp ERR! node -v v5.10.1
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Linux 4.4.5-1-ARCH
npm ERR! argv "/usr/bin/node" "/home/hridayesh/atom/build/node_modules/.bin/npm" "--userconfig=/home/hridayesh/atom/build/.npmrc" "--target=0.10.35" "install" "--loglevel" "error"
npm ERR! node v5.10.1
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the keytar package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls keytar
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/hridayesh/atom/apm/npm-debug.log

If you want, you can install atom from the AUR:
https://aur.archlinux.org/packages/atom-editor/

Well Alexis-Martin Thanks for reply but the link that you provided, it is not working.
I have already tried to install atom-editor package by yaourt but it shows -
"atom-editor not found in AUR"

You right, if you copy-paste the link in a new tab it's work... I don't know why. :)
It's strange, for me the command yaourt -a atom-editor give me this :
1 aur/atom-editor 1.7.2-1 [installed] (555) (63,29)
    Chrome-based text editor from Github
2 aur/atom-editor-beta 1.8.0.beta2-1 (5) (1,53)
    Chrome-based text editor from Github - Beta Channel
3 aur/atom-editor-beta-bin 1.8.0.beta2-1 (21) (3,22)
    Chrome-based text editor from Github - Beta Channel - Precompiled binary from official repository
4 aur/atom-editor-bin 1.6.2-1 (Out of Date) (242) (26,64)
    Chrome-based text editor from Github - Precompiled binary from official repository
5 aur/atom-editor-git 1.1.0.beta.0.r178.g575d79a-2 (40) (0,32)

using the package from aur atom-editor or atom-editor-beta it happens the same error.
here is the log taked from yaourt
==> atom-editor-beta dependencies:
 - alsa-lib (already installed)
 - desktop-file-utils (already installed)
 - gconf (already installed)
 - gtk2 (already installed)
 - libgnome-keyring (already installed)
 - libnotify (already installed)
 - libxtst (already installed)
 - nodejs (already installed)
 - nss (already installed)
 - python2 (already installed)
 - git (already installed) [makedepend]
 - npm (already installed) [makedepend]

==> Building and installing package
==> Making package: atom-editor-beta 1.8.0.beta2-1 (Wed Apr 27 14:06:28 ART 2016)
==> Checking runtime dependencies...
==> Checking buildtime dependencies...
==> Retrieving sources...
  -> Downloading v1.8.0-beta2.tar.gz...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   123    0   123    0     0    161      0 --:--:-- --:--:-- --:--:--   161
100  9.8M  100  9.8M    0     0  1368k      0  0:00:07  0:00:07 --:--:-- 2037k
  -> Found atom-beta.desktop
  -> Found package.patch
==> Validating source files with sha256sums...
    v1.8.0-beta2.tar.gz ... Passed
    atom-beta.desktop ... Passed
    package.patch ... Passed
==> Extracting sources...
  -> Extracting v1.8.0-beta2.tar.gz with bsdtar
==> Starting prepare()...
patching file package.json
patching file spec/atom-environment-spec.coffee
patching file spec/module-cache-spec.coffee
patching file spec/window-event-handler-spec.coffee
Hunk #1 succeeded at 75 with fuzz 2.
patching file src/module-cache.coffee
==> Starting build()...
Node: v5.6.0
npm: v3.6.0
Installing build modules...
=> Took 200776ms.

Installing apm...
../src/main.cc: In function ‘void {anonymous}::Init(v8::Handle<v8::Object>)’:
../src/main.cc:52:53: error: no matching function for call to ‘SetMethod(v8::Handle<v8::Object>&, const char [12], Nan::NAN_METHOD_RETURN_TYPE (&)(Nan::NAN_METHOD_ARGS_TYPE))’
   Nan::SetMethod(exports, "getPassword", GetPassword);
                                                     ^
In file included from ../src/main.cc:1:0:
../node_modules/nan/nan.h:1853:17: note: candidate: void Nan::SetMethod(v8::Local<v8::Object>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1853:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::Object>’
../node_modules/nan/nan.h:1865:17: note: candidate: void Nan::SetMethod(v8::Local<v8::FunctionTemplate>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1865:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::FunctionTemplate>’
../node_modules/nan/nan.h:1876:17: note: candidate: void Nan::SetMethod(v8::Local<v8::ObjectTemplate>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1876:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::ObjectTemplate>’
../src/main.cc:53:53: error: no matching function for call to ‘SetMethod(v8::Handle<v8::Object>&, const char [12], Nan::NAN_METHOD_RETURN_TYPE (&)(Nan::NAN_METHOD_ARGS_TYPE))’
   Nan::SetMethod(exports, "addPassword", AddPassword);
                                                     ^
In file included from ../src/main.cc:1:0:
../node_modules/nan/nan.h:1853:17: note: candidate: void Nan::SetMethod(v8::Local<v8::Object>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1853:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::Object>’
../node_modules/nan/nan.h:1865:17: note: candidate: void Nan::SetMethod(v8::Local<v8::FunctionTemplate>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1865:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::FunctionTemplate>’
../node_modules/nan/nan.h:1876:17: note: candidate: void Nan::SetMethod(v8::Local<v8::ObjectTemplate>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1876:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::ObjectTemplate>’
../src/main.cc:54:59: error: no matching function for call to ‘SetMethod(v8::Handle<v8::Object>&, const char [15], Nan::NAN_METHOD_RETURN_TYPE (&)(Nan::NAN_METHOD_ARGS_TYPE))’
   Nan::SetMethod(exports, "deletePassword", DeletePassword);
                                                           ^
In file included from ../src/main.cc:1:0:
../node_modules/nan/nan.h:1853:17: note: candidate: void Nan::SetMethod(v8::Local<v8::Object>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1853:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::Object>’
../node_modules/nan/nan.h:1865:17: note: candidate: void Nan::SetMethod(v8::Local<v8::FunctionTemplate>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1865:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::FunctionTemplate>’
../node_modules/nan/nan.h:1876:17: note: candidate: void Nan::SetMethod(v8::Local<v8::ObjectTemplate>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1876:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::ObjectTemplate>’
../src/main.cc:55:55: error: no matching function for call to ‘SetMethod(v8::Handle<v8::Object>&, const char [13], Nan::NAN_METHOD_RETURN_TYPE (&)(Nan::NAN_METHOD_ARGS_TYPE))’
   Nan::SetMethod(exports, "findPassword", FindPassword);
                                                       ^
In file included from ../src/main.cc:1:0:
../node_modules/nan/nan.h:1853:17: note: candidate: void Nan::SetMethod(v8::Local<v8::Object>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1853:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::Object>’
../node_modules/nan/nan.h:1865:17: note: candidate: void Nan::SetMethod(v8::Local<v8::FunctionTemplate>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1865:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::FunctionTemplate>’
../node_modules/nan/nan.h:1876:17: note: candidate: void Nan::SetMethod(v8::Local<v8::ObjectTemplate>, const char*, Nan::FunctionCallback)
 NAN_INLINE void SetMethod(
                 ^
../node_modules/nan/nan.h:1876:17: note:   no known conversion for argument 1 from ‘v8::Handle<v8::Object>’ to ‘v8::Local<v8::ObjectTemplate>’
make: *** [Release/obj.target/keytar/src/main.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/tmp/yaourt-tmp-<user>/aur-atom-editor-beta/src/atom-1.8.0-beta2/build/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at emitTwo (events.js:100:13)
gyp ERR! stack     at ChildProcess.emit (events.js:185:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Linux 4.4.8-1-MANJARO
gyp ERR! command "/home/<user>/.nvm/versions/node/v5.6.0/bin/node" "/tmp/yaourt-tmp-<user>/aur-atom-editor-beta/src/atom-1.8.0-beta2/build/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /tmp/yaourt-tmp-<user>/aur-atom-editor-beta/src/atom-1.8.0-beta2/apm/node_modules/atom-package-manager/node_modules/keytar
gyp ERR! node -v v5.6.0
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok
npm ERR! Linux 4.4.8-1-MANJARO
npm ERR! argv "/home/<user>/.nvm/versions/node/v5.6.0/bin/node" "/tmp/yaourt-tmp-<user>/aur-atom-editor-beta/src/atom-1.8.0-beta2/build/node_modules/.bin/npm" "--userconfig=/tmp/yaourt-tmp-<user>/aur-atom-editor-beta/src/atom-1.8.0-beta2/build/.npmrc" "--target=0.10.35" "install" "--loglevel" "error"
npm ERR! node v5.6.0
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the keytar package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls keytar
npm ERR! There is likely additional logging output above.

@vyasriday updating the version of apm to 1.9.3 fix the problem, at least in my case. Please, try that.
Change the version in your atom repository /apm/package.json

Since this seems to have moved on to a discussion around the yaourt package, I'm going to close this.
Also, we're trying to control the amount of Issues that we have to triage. For general questions or discussions, the Atom message board is the best location.

Ok Thanks . well that is working now. Thanks tinchoz42 for that.
But got and issue again.
When trying to run
sudo scrip grunt/install i get the following error.
sudo script grunt/install
Script started, file is grunt/install
script: cannot open grunt/install: No such file or directory
Script done, file is grunt/install
Segmentation fault
Can you help me with that.
I think that might a small issue and i do not want to open a new issue just for that.

@vyasriday It's sudo script/grunt install.

@50Wliu see the next output which i got by running sudo script grunt/install

It is sudo script/grunt install, not sudo script grunt/install.  If you're still having trouble after this, please create a new topic on Discuss.

What's wrong with installing from source? (as a Gentoo user I'd say you should ALWAYS install from source when possible ;) )

@50Wliu  I am really sorry ,I couldnot see that.
But i am still having issues.
Thanks for your help


⚠ SceneTime has degraded performance

In 9a1275f, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1382 ms

Resolved: SceneTime performance has improved in 0829971 after 9 minutes.


/workflow//correlated/VERY SLOW if many workflows with that corr id

The ExecutionDAO.getWorkflowsByCorrelationId() method has only correlation id as a parameter. There is no additional method that also passes in the workflow name. So on the /workflow/wfname/correlated/corrid endpoint there is no way to do a query that joins in the workflow_def_to_workflow table to get back just the workflows for that workflow name that have that correlation id. This is a problem, especially if there are several records in the workflow table that have the same correlation id, even though they are for different workflow names.
The existing logic has to query the workflow table for all workflows with that correlation id, then get each workflow individually. If includeTasks is true all the tasks will be retrieved too. Only then do the results get filtered by workflow name.
In our case, the correlation id was '3' and there were 11,945 workflows with that correlation id but only 19 were for the workflow name specified. It took more than a minute to do that in our database that has millions of rows.
I would propose that an additional getWorkflowsByCorrelationId() method is added to all the ExecutionDAO-derived classes that takes the workflow name as a parameter. This would mean that all implementations would have to be changed. Is this something I should attempt to do? Is there a reason why this was not done?

@rickfish I believe that this can be accomplished using the /workflow/search endpoint. Both the correlationId and workflowType are indexed fields and can be used for this purpose. Would this suffice your use-case?

Yes, this can be accomplished with the /workflow/search endpoint. But that is like saying don't use the /workflow//correlated/ endpoint because it could be really slow. At that point, that endpoint would not be used so why even have it? That endpoint should be optimized I think. Since the workflow name is part of the endpoint path, why not pass it into the method that is supposed to retrieve those workflows?

@rickfish Ahh I misunderstood the earlier comment. I believe the method implementation needs to be changed to retrieve only the correlated workflows of a specific type. I have labeled this as a bug and we will look into fixing this API soon. Thanks for reporting.

Fixed in #1766


iOS - Add disclaimer on documents accepted for proof-of-address

Can't see the disclaimer on soralution from 16 build.

Can't see the disclaimer on soralution from 24 build.

25 build is ok


docs: Querying basics: remove what can be graphed

Put a scalar to query, it can be graphed.
And also, the range-queries, which used for graph, always returns a range vector https://promlabs.com/blog/2020/06/18/the-anatomy-of-a-promql-query/#range-queries.
So the doc says "an expression that returns an instant vector is the only type which can be graphed." is not correct?
it's confusing to read the statement, hence, remove it.

yes that's better, I updated


feat: update pcd/react packages to QRVerify circuits

Motivation
Change Summary

New QR code reader component
Refactor react package
Updated pcd package to QR code circuit
Fix CI

Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[x] PR title adheres to the conventional commits standard
[x] PR has been tagged with a change label(s) (i.e. documentation, feature, bug fix, or chore)
[ ] PR includes documentation if necessary.

@vuvoth I refactored from your comments
Now instead of extractWitness there is a new component generateArgs that will take care of manipulating the raw QR code and will directly output the args to pass to the prover.


Bug: Getting error after upgrading to >= 0.6.0 version of lexical from 0.5.0

Lexical version:
0.6.0 and above
Steps To Reproduce
upgrade lexical version from 0.5.0 to 0.6.0 and above
0.5.0 is working !

They are named exports now. See the playground for examples. For this case you can do:
import {RichTextPlugin} from '@lexical/react/RichTextPlugin';

@trueadm Thank you for help. I already use named imports . My code based on examples from docs and playground. But it not working after update from 0.5.0 -> 0.6.4 :(
import { LexicalComposer } from '@lexical/react/LexicalComposer';
import { RichTextPlugin } from '@lexical/react/LexicalRichTextPlugin';

Are all your versions for your references to other modules in your package.json files also 0.6.4?

"lexical": "0.6.4",
"@lexical/react": "0.6.4",
"@lexical/clipboard": "0.6.4",
"@lexical/code": "0.6.4",
"@lexical/file": "0.6.4",
"@lexical/hashtag": "0.6.4",
"@lexical/link": "0.6.4",
"@lexical/list": "0.6.4",
"@lexical/mark": "0.6.4",
"@lexical/overflow": "0.6.4",
"@lexical/plain-text": "0.6.4",
"@lexical/rich-text": "0.6.4",
"@lexical/selection": "0.6.4",
"@lexical/table": "0.6.4",
"@lexical/utils": "0.6.4",

Try clearing out your node_modules and doing npm install again?

deleting node_modules and reinstalling packages not helped me at all :(  Same error in console

Btw i use @swc/core and swc-loader for webpack . I updated them. And error dissapeared


Integration jar

Adding provide scope to brooklyn-tosca-transformer/brooklyn-all dependency and adding brooklyn-all dependency to brooklyn-tosca-dist module.
Uber jar requires brooklyn PR#1042 for running in dropins folder

@kiuby88 I'm not sure what your intention is here. What problem are you trying to solve?
brooklyn-tosca-transformer is logically distinct from the Alien runtime so should not contain configuration like alien4cloud-config.yml and elasticsearch.yml. We should continue to keep these in brooklyn-tosca-dist.
What does the addition of the maven-shade-plugin achieve over the brooklyn-tosca-dist module?

Thanks for your reply @sjcorbett
I am trying to add brooklyn-tosca support to brooklyn. Then, I thought it could be a good idea creating a uber jar which will be able to be added to brooklyn dropins folder. So, I used shade plugin to create this jar. I tried to optimize this uber jar, so I marked brooklyn-all as provided (then, I needed to add brooklyn-all to brooklyn-tosca-dist).
Probably, adding  alien4cloud-config.yml and elasticsearch.yml files to brooklyn-tosca-transformer is not the best idea.
Maybe, I could create a profile for generating this uber jar.
WDYT?

When we've used shaded jars in Brooklyn in the past we've run into strange errors that boiled down to problems with signed jars. So we use the assembly plugin to build a distribution.tar.gz instead (like brooklyn-tosca-dist).
Downstream projects can already include brooklyn-tosca-transformer in dropins/. I also expect downstream projects to provide their own elasticsearch.yml and alien4cloud-config.yml. We can provide basic ones to get started but can't predict suitable configuration for production.
As such, assuming you are working on a downstream project, I'd suggest:

Adding a dependency on brooklyn-tosca to the project.
Adding elasticsearch.yml and alien4cloud-config.yml to the project.
Following the pattern of brooklyn-tosca-dist to assemble everything.

Does this make sense?
At the same time as the first point we should start publishing brooklyn-tosca build artefacts.

Yes, I'd like to follow the first point, but I was not sure about if you wanted to publish the brooklyn-tosca artifacts. So, I built an uber jar, at least as a temporal solution. The uber jar contains all required dependencies (a4c, etc). So, I did not require to add more dependencies (a4c, etc) to dropins folder.
About config files. I added to brooklyn-tosca-transformer but as you have commented probably it was not a good idea. And they should be added final project too.
So, your suggestion makes sense.
I will use an uber jar in dropins as a temporal solution and I will use a maven dependency when the artifacts will be online. Moreover, I will added config files in my final project.
Thank you very much for your replies.
Probably, this PR could be closed.

@kiuby88 Thanks. I'm arranging for builds of the project to be published so will close this pull request.


plugins not available as dependencies of other plugins

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
If you add a plugin gem to another plugin as a (non-development) dependency in the gemspec, Fastlane does not detect it as an available plugin, and its actions are not available in the other_action context. The gem will be installed, and it is possible to call the action class directly, but it doesn't always work as expected.
Complete output when running fastlane, including the stack trace and command used
With spec.add_dependency 'fastlane-plugin-patch' in the gemspec:
[jdee@Jimmy-Dees-MacBookPro fastlane-plugin-branch (source-code-patching)]$ bundle exec fastlane lanes
+------------------------+---------+---------------------------------------+
|                               Used plugins                               |
+------------------------+---------+---------------------------------------+
| Plugin                 | Version | Action                                |
+------------------------+---------+---------------------------------------+
| fastlane-plugin-branch | 0.1.0   | setup_branch validate_universal_links |
+------------------------+---------+---------------------------------------+

--------- general---------
----- fastlane update
Add Branch keys and Universal Link domains to BranchPluginExample

----- fastlane update_and_commit
Add Branch keys and Universal Link domains to BranchPluginExample and commit to Git

----- fastlane validate
Validate Universal Link settings for a project

Execute using `fastlane [lane_name]`

With gem 'fastlane-plugin-patch' in the Pluginfile:
[jdee@Jimmy-Dees-MacBookPro fastlane-plugin-branch (source-code-patching)]$ bundle exec fastlane lanes
+------------------------+---------+---------------------------------------+
|                               Used plugins                               |
+------------------------+---------+---------------------------------------+
| Plugin                 | Version | Action                                |
+------------------------+---------+---------------------------------------+
| fastlane-plugin-branch | 0.1.0   | setup_branch validate_universal_links |
| fastlane-plugin-patch  | 0.1.0   | apply_patch revert_patch              |
+------------------------+---------+---------------------------------------+

--------- general---------
----- fastlane update
Add Branch keys and Universal Link domains to BranchPluginExample

----- fastlane update_and_commit
Add Branch keys and Universal Link domains to BranchPluginExample and commit to Git

----- fastlane validate
Validate Universal Link settings for a project

Execute using `fastlane [lane_name]`

The latter is desirable behavior. To reproduce yourself:

Clone any plugin repo.
Add any other plugin gem to the gemspec for that plugin using add_dependency or add_runtime_dependency.
bundle install
Verify via the Gemfile.lock that the dependency was installed.
bundle exec fastlane lanes: The other plugin will not appear in the list of Used plugins.
Move the other plugin gem from the gemspec to the Pluginfile and repeat. The other plugin will appear in the list of Used plugins.

The reason seems to be this:
https://github.com/fastlane/fastlane/blob/master/fastlane/lib/fastlane/plugins/plugin_manager.rb#L57
The call to Bundler::Dsl.evaluate(...).dependencies does not seem to return all dependencies at all levels, at least as called here. It only returns the contents of the Gemfile, essentially. The Pluginfile is effectively inserted into the Gemfile, so it registers dependencies listed in the Pluginfile. I think gems specified using add_development_dependency are essentially added to the Gemfile, so they are included.
I'm not quite sure how to fix this, or I'd just submit a PR.
For now it's not hard to work around it and warn users to install another plugin manually:
      def self.patch_app_delegate_swift(project)
        unless defined?(Fastlane::Actions::ApplyPatchAction)
          # plugins as dependencies of plugins not working well right now
          UI.important "The patch plugin is not available. Run fastlane add_plugin patch first to enable source-code patching."
          return
        end

        # ...

Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
✅ fastlane environment ✅

### Stack

| Key                         | Value                                       |
| --------------------------- | ------------------------------------------- |
| OS                          | 10.12.6                                     |
| Ruby                        | 2.4.1                                       |
| Bundler?                    | true                                        |
| Git                         | git version 2.11.0 (Apple Git-81)           |
| Installation Source         | ~/.rvm/gems/ruby-2.4.1/bin/fastlane         |
| Host                        | Mac OS X 10.12.6 (16G29)                    |
| Ruby Lib Dir                | ~/.rvm/rubies/ruby-2.4.1/lib                |
| OpenSSL Version             | OpenSSL 1.1.0f  25 May 2017                 |
| Is contained                | false                                       |
| Is homebrew                 | false                                       |
| Is installed via Fabric.app | false                                       |
| Xcode Path                  | /Applications/Xcode.app/Contents/Developer/ |
| Xcode Version               | 8.3.3                                       |

### System Locale

| Variable | Value       |   |
| -------- | ----------- | - |
| LANG     | en_US.UTF-8 | ✅ |
| LC_ALL   |             |   |
| LANGUAGE |             |   |

### fastlane files:

`./fastlane/Fastfile`

```ruby
desc "Add Branch keys and Universal Link domains to BranchPluginExample"
lane :update do
  setup_branch
end

desc "Add Branch keys and Universal Link domains to BranchPluginExample and commit to Git"
lane :update_and_commit do
  setup_branch commit: true # Use a string here for a custom commit message
end

desc "Validate Universal Link settings for a project"
lane :validate do
  # The :domains parameter is optional.
  # Without it, validation passes if at least one domain exists in the project and
  # all domains pass validation.
  # With this parameter, the applinks: domains in the project must match this list.
  # Order is irrelevant.
  # This app is not currently set up properly for test in the Dashboard. Don't include
  # test domains in the project for now.
  validate_universal_links domains: %w{
    bnctestbed.app.link
    bnctestbed-alternate.app.link
  }
end

**No Appfile found**

### fastlane gems

| Gem      | Version | Update-Status |
| -------- | ------- | ------------- |
| fastlane | 2.50.0  | ✅ Up-To-Date  |

### Loaded fastlane plugins:

| Plugin                 | Version | Update-Status  |
| ---------------------- | ------- | -------------- |
| fastlane-plugin-branch | 0.1.0   | 💥 Check failed |

Loaded gems

| Gem                       | Version      |
| ------------------------- | ------------ |
| did_you_mean              | 1.1.0        |
| io-console                | 0.4.6        |
| rake                      | 12.0.0       |
| CFPropertyList            | 2.3.5        |
| i18n                      | 0.8.6        |
| minitest                  | 5.10.3       |
| thread_safe               | 0.3.6        |
| tzinfo                    | 1.2.3        |
| activesupport             | 4.2.9        |
| public_suffix             | 2.0.5        |
| addressable               | 2.5.1        |
| ast                       | 2.3.0        |
| babosa                    | 1.0.2        |
| bundler                   | 1.15.3       |
| claide                    | 1.0.2        |
| fuzzy_match               | 2.0.4        |
| nap                       | 1.1.0        |
| cocoapods-core            | 1.2.1        |
| cocoapods-deintegrate     | 1.0.1        |
| cocoapods-downloader      | 1.1.3        |
| cocoapods-plugins         | 1.0.0        |
| cocoapods-search          | 1.0.0        |
| cocoapods-stats           | 1.0.0        |
| netrc                     | 0.7.8        |
| cocoapods-trunk           | 1.2.0        |
| cocoapods-try             | 1.1.0        |
| colored2                  | 3.1.2        |
| escape                    | 0.0.4        |
| fourflusher               | 2.0.1        |
| gh_inspector              | 1.0.3        |
| molinillo                 | 0.5.7        |
| ruby-macho                | 1.1.0        |
| nanaimo                   | 0.2.3        |
| xcodeproj                 | 1.5.1        |
| cocoapods                 | 1.2.1        |
| coderay                   | 1.1.1        |
| colored                   | 1.2          |
| highline                  | 1.7.8        |
| commander-fastlane        | 4.4.5        |
| declarative               | 0.0.9        |
| declarative-option        | 0.1.0        |
| diff-lcs                  | 1.3          |
| docile                    | 1.1.5        |
| unf_ext                   | <IP_ADDRESS>      |
| unf                       | 0.1.4        |
| domain_name               | 0.5.20170404 |
| dotenv                    | 2.2.1        |
| excon                     | 0.57.1       |
| multipart-post            | 2.0.0        |
| faraday                   | 0.12.2       |
| http-cookie               | 1.0.3        |
| faraday-cookie_jar        | 0.0.6        |
| faraday_middleware        | 0.12.0       |
| fastimage                 | 2.1.0        |
| jwt                       | 1.5.6        |
| little-plugger            | 1.1.4        |
| multi_json                | 1.12.1       |
| logging                   | 2.2.2        |
| memoist                   | 0.16.0       |
| os                        | 0.9.6        |
| signet                    | 0.7.3        |
| googleauth                | 0.5.3        |
| httpclient                | 2.8.3        |
| mime-types-data           | 3.2016.0521  |
| mime-types                | 3.1          |
| uber                      | 0.1.0        |
| representable             | 3.0.4        |
| retriable                 | 3.1.1        |
| google-api-client         | 0.12.0       |
| json                      | 2.1.0        |
| mini_magick               | 4.5.1        |
| multi_xml                 | 0.6.0        |
| plist                     | 3.3.0        |
| rubyzip                   | 1.2.1        |
| security                  | 0.1.3        |
| slack-notifier            | 1.5.1        |
| terminal-notifier         | 1.8.0        |
| unicode-display_width     | 1.3.0        |
| terminal-table            | 1.8.0        |
| tty-screen                | 0.5.0        |
| word_wrap                 | 1.0.0        |
| rouge                     | 2.0.7        |
| xcpretty                  | 0.2.8        |
| xcpretty-travis-formatter | 0.0.4        |
| fastlane-plugin-patch     | 0.1.0        |
| mini_portile2             | 2.2.0        |
| nokogiri                  | 1.8.0        |
| fastlane-plugin-branch    | 0.1.0        |
| method_source             | 0.8.2        |
| parallel                  | 1.12.0       |
| parser                    | <IP_ADDRESS>      |
| powerpack                 | 0.1.1        |
| slop                      | 3.6.0        |
| pry                       | 0.10.4       |
| rainbow                   | 2.2.2        |
| rspec-support             | 3.6.0        |
| rspec-core                | 3.6.0        |
| rspec-expectations        | 3.6.0        |
| rspec-mocks               | 3.6.0        |
| rspec                     | 3.6.0        |
| rspec-simplecov           | 0.2.2        |
| ruby-progressbar          | 1.8.1        |
| rubocop                   | 0.49.1       |
| simplecov-html            | 0.10.1       |
| simplecov                 | 0.14.1       |

*generated on:* **2017-08-01**

Hey @jdee - thanks for submitting this issue. Yeah I think when we were building the plugin architecture we did not have in mind that people would make plugins dependent on other plugins. That being said, this definitely does seem like a valid use-case. I think for the time being, the best course of action would be to either:

check if the dependent plugin is available early in the execution and fail with a nice error message asking the user to include the other plugin in their project, or
maybe try to submit a PR to the plugin that you are dependent on instead of creating another plugin all together

I think I would be in favor of the former option, but I'm curious to hear your thoughts! 👍 🐙

Thanks for the quick feedback, @ohayon. Two thoughts for you:

I realized just after opening the issue that I'm doing something dumb. I'm checking for the presence of ApplyPatchAction and then calling other_action.apply_patch if it's found. It's just a gem. I can add it as a dependency, require it in my code and call the action directly. So this should not hinder me at all. The only thing would be if someone notices the other plugin is in their bundle but they aren't able to use it in their Fastfile. That's probably a minor, limited problem with an easy workaround that can at least temporarily be explained and maybe addressed later.

As Fastlane grows and more plugins appear, using them may become awkward. If you have to install eight plugins to do some relatively common thing, setup becomes more complex. Of course, if those become very common use cases, those plugins could be contributed to Fastlane proper (if their authors are amenable). But there could be a case for making package plugins like fastlane-plugin-android that just pull in a number of common plugins for a common scenario, like Android development. This problem stops that idea in its tracks.

If anyone has any good suggestions on how to fix this, I'm happy to do the work. I assume if there's a Gemfile.lock (which there has to be after bundle install), the bundler must be able to infer every dependency at every level, without querying a server to resolve dependencies. It must be possible to get this information out somewhere. I haven't found good docs for the bundler gem API, so I'm left reading code and playing with irb. If I can figure it out, I'll submit a PR.

Thanks @jdee! Yeah I agree that things could eventually get out of control and maybe require something like meta-plugins or another solution to maintain plugins. I think we are clear for now 😄  Is it cool with you if I close this issue as it does not seem like there is a bug in fastlane? 🚀

I agree. This is not a bug in Fastlane. Closing.


Update mechanics guide for cloud capitalization

Adding HB guidance specific to using the word "Cloud" and when it should be capitalized.

Also removed "Code Intelligence" as a product name while I was updating this page.


Add Canvasboard to the projects list

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/india/blob/main/CONTRIBUTING.md.

Which change are you proposing?

[x] Adding/updating open source project/s
[ ] Adding/updating maintainer/s

Adding/updating open source project/s

[x] The project was built in or receives significant contributions from India
[x] The project has at least 100 stargazers

Why should this open source project be featured?

The project is build by students during the covid to enhance online teaching experience

Adding/updating maintainer/s

[x] Maintainer's location on their GitHub profile indicates as India
[x] Maintainer has approved this PR to be included on the website

What open source project/s does the individual maintain?

Please replace this line with GitHub URLs to projects that the individual/s maintains

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing page, or at least not a single one

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

News articles on canvasboard:
https://www.techcircle.in/2021/11/19/iiitb-presents-use-cases-for-differently-abled-and-e-learning-at-bengaluru-tech-summit/
https://indiaeducationdiary.in/iiitb-presents-innovations-to-improve-accessibility-and-help-the-differently-abled-at-the-bengaluru-tech-summit-2021/
https://digitalterminal.in/news/iiitb-showcased-innovations-to-help-differently-abled-at-bengaluru-tech-summit-2021/26022.html


adding directory option and updating readme

Coverage remained the same at 100.0% when pulling 4d88b12f1aba8d2e46f317a15f9272018cd851e8 on ay13:custom-permalink into da9b7b9eeddc7d85d4bac296d2ca3e7f19e83f84 on carrot:master.


Useless and invalid fixes from invalid mixin application

Given the following code:
void f() {
}
class X = Object;

the analyzer produces the following diagnostics:
error: Classes can only mix in mixins and classes. (mixin_of_non_class at )
error: Expected to find 'with'. (expected_token at )

There should only be one diagnostic here, but worse yet is the list of fixes suggested:

Import library 'dart:core'
Change to 'Object'
Create class 'Object'

The first is pointless because dart:core is already imported, the second changes 'Object' to 'Object', and the third creates a class that hides the Object from core. None of them actually fixes the problem.

The same two issues are reported today, and the same three fixes offered.


Change the signature of the Repository.Init method

See issue #101

Very nice job! It's been manually merged into vNext. Keep it up :)
BTW, It looks like you're using real tabs (\t) instead of spaces. As this hinders a bit the diffing, I've fixed this while merging.


Enhancement:Add network modal url needs hardcoded https://

As an user whom copy paste url,it seems to be cumbersome when its pasted into the url section when an user is adding a new network modal.
If the user did not add https:// to the url header,an error is thrown.It would be nice to have a http:// hardcoded in the url section

Would you then have the opposite problem if the user has copied an address with the protocol?

I'd recommend we simply detect upon submission if the URL value as an http or https prefix, and if not, add https automatically. That way the user doesn't have to deal with pre-filled values and benefits from protocol inclusion flexibility on entry.

We do provide an error message and I would think most users will copy/paste?

Yes,I would assume most users will copy and paste only the url .
However,if the url is missing https header,it does thrown an error as of now.


Transform and load dependencies from setup.cfg

Previously, requirements.txt was the only file cartography loaded dependencies from.
This PR adds support for setup.cfg, which is where many Python libraries define dependencies;
specifically in install_requires, setup_requires, and extras_require
Also tested locally by running cartography, ensuring the data was loaded correctly, and checking that I could query dependencies from setup.cfg.
Note: the GH dependency graph unfortunately does not support setup.cfg ingestion at the moment. Although setup.cfg support has been implement in dependabot, the two projects are completely separate with no timeline for merge at the moment.

Re testing, have you run this on a few setup.cfgs from some of  our bigger projects?

Re testing, have you run this on a few setup.cfgs from some of our bigger projects?

I tested on repos with setup.cfg's that had requirements across install_requires, setup_requires, and extras_require.

🚀


drivers: ethernet: Optimize RxBD buffer configuration

On IMX93_EVK A-core, the _nxp_enet_dma_buffer_section is configured to __nocache area. That makes the RxPkt performance very low (<= 50Mbps). By defining it to the cacheable area, the RxPkt performance is >10x better.
Example with Zperf:
build: west build -p always -b imx93_evk/mimx9352/a55 samples/net/zperf/
DUT command: zperf udp download 5001
PC command: iperf -u -c <IP_ADDRESS> -p 5001 -b 800M
Result before optimization:

Result after optimization:

is there any real guarantee of cache coherency here? wouldn't we have to invalidate the cache region after uDMA writes RX packet to memory?

is there any real guarantee of cache coherency here? wouldn't we have to invalidate the cache region after uDMA writes RX packet to memory?

[Shrek] Yes, in modules/hal/nxp/mcux/mcux-sdk/drivers/enet/fsl_enet.c the ENET_ReadFrame() code, the cacheable buffer will be invalidated if the 'driver_cache_maintain' is set to 'true'.

Also, this should be tested on the RT11XX and RT10XX as well, and more cases than the zperf download which is somewhat of a trivial case IMO
[Shrek] RT1170 was also tested but the code for RT1170 would go to the DTCM compiling macro part, so no impact. Besides zperf case, the IPv6-forwarding was also tested on MX93 and the performance also boosted a lot.


Synced version in bower.json to 0.1.1

@linkgod can you make a release version? This PR just syncs up the bower.json version.

I am very sorry to slow the process, because I was on vacation

No problem. As were everyone :)


AVRO-2114: Make missing value exceptions in nested structures easier to read.

With this patch the exception of a missing nested fields looks like this below.
The developer can now clearly see the path in the schema where the problem occurred.
Please review.
Field networkAddress type:STRING pos:1 not set and has no default value
Path in schema: --> connection --> networkAddress
	at org.apache.avro.generic.GenericData.getDefaultValue(GenericData.java:999)
	at org.apache.avro.data.RecordBuilderBase.defaultValue(RecordBuilderBase.java:138)
	at org.apache.avro.test.http.NetworkConnection$Builder.build(NetworkConnection.java:290)
	at org.apache.avro.test.http.Request$Builder.build(Request.java:440)
	at org.apache.avro.specific.TestSpecificBuilderTree.failOnIncompleteTree(TestSpecificBuilderTree.java:78)

Not sure if it is relevant, but not all generated Avro test classes are re-generated and included into this PR. The generated code for FooBarSpecificRecord, TestRecordWithLogicalTypes, TestRecordWithoutLogicalTypes classes will change.

@nandorKollar Do you know how TestRecordWithLogicalTypes and TestRecordWithoutLogicalTypes are to be generated? Or where I can find info on how to generate them correctly?

I found it:
https://github.com/apache/avro/blob/master/lang/java/avro/src/test/java/org/apache/avro/specific/TestSpecificLogicalTypes.java#L46

The classes should not be re-generated because they test compatibility of
Avro with existing Avro-generated sources. When using classes generated
before AVRO-1684, logical types should not be applied by the read or write
paths. Those files should behave as they did before.

So I will not regenerate these two classes.

I updated the commit to

Be rebased ontop of latest master
Include the newly generated version of FooBarSpecificRecord.java


Move date_part, date_trunc, date_bin functions to datafusion-functions

Is your feature request related to a problem or challenge?
As part of #9285 the date_* datetime functions should be migrated to the new datafusion-functions crate in the new structure.
Describe the solution you'd like
Functions are migrated to the new crate and use the new patterns as described in #9286 & #9216
Describe alternatives you've considered
No response
Additional context
No response

take


Course org specific certs setup

Allow definition of custom signatories and custom org overrides for the Advanced Setting 'cert_html_view_overrides', which can then be used in theme-variables.html cert config.
I never was able to resolve issues with tox setup for the Ginkgo env but this is some progress at least.

The tox setup is still messed up.  I don't have bandwidth to fix it right now though.


fields is blocking setSelectedText

Hey @johnlindquist,
I was trying to implement this script here:
let text = await getSelectedText();

let buildPreview = ([search, replacement]) => {
    if (!search) return md(text);
    if (!replacement)
        return md(
            `${text
                .replaceAll("\n", "
")
                .replaceAll(search, `<ins>${search}</ins>`)}`
        );
    return md(
        `${text
            .replaceAll("\n", "
")
            .replaceAll(search, `<ins>${replacement}</ins>`)}`
    );
};

let [search, replacement] = await fields({
    fields: ["Search", "Replacement"],
    preview: buildPreview([]),
    onChange: async (input, state) => {
        let preview = buildPreview(state?.value);
        setPreview(preview);
    },
});

let result = text.replaceAll(search, replacement);

// Output
await setSelectedText(result);

But it never finishes the process. There is a loading indicator after I submit the fields and that's where it gets stuck. If I add another arg afterwards, if works though. Here is a working example:
let [search, replacement] = await fields({
    fields: ["Search", "Replacement"],
    preview: buildPreview([]),
    onChange: async (input, state) => {
        let preview = buildPreview(state?.value);
        setPreview(preview);
    },
});
await arg("Dummy arg", ["foo"])
let result = text.replaceAll(search, replacement);

I assume there is something in fields that blocks the UI? 🤔

Here is a gif

Thanks @Schmedu
It's a flaw in the "hidden prompt" logic.
Before the prompt can hide, it needs to close the preview pane. (If not, you'll see a flash of the previous preview pane when the next prompt opens. This is because prompt renderer "sleeps" when hidden*).
setSelectedText tells the prompt to "hide", but doesn't tell the preview to close, so the prompt gets stuck waiting for the preview state to change.
I've got a fix in that I'll release in the build tonight. 👍
The easiest temp fix is to just toss in a call to div() which will automatically close the preview:
let result = text.replaceAll(search, replacement)

div() // <-- "hack" to close the preview pane by switch to a new prompt
await setSelectedText(result)

* "Sleeping renderer" - I wish it were possible to "render while hidden" to avoid this "wait for close" logic, but it's a limitation of the platform that causes some huge headaches when trying to make UIs that instantly open/close. I've run tests with setBackgroundThrottling, but unfortunately, they've all resulting in keeping the renderer active when hidden, which results in wasting CPU/GPU.
The goal is to have as close to 0% CPU/GPU as possible when hidden, with the only exception being the ~0.1% CPU for the script/data/keyboard watchers, which need to stay active for file changes.

Thanks John! 🙏


Uber Example

Hi - thanks for your efforts on this it looks great.
Can you please provide an example for how to launch the Uber app with deep linking to ride requests and optional parameters? This seems straightforward for iOS but I can't figure out what the options for android need to be.
https://developer.uber.com/docs/deep-linking

@armensg hi!
Test this:
startApp.set({ /* params */
    "action": "ACTION_VIEW",
    "uri": "uber://?action=setPickup&pickup=my_location&client_id=YOUR_CLIENT_ID"
}).start();

Thanks! I'm actually trying to use this with an ionic/Cordova app and having trouble figuring out how to install it :(

After doing Cordova plugin install do I need to include any JavaScript scripts on my index.html?

This is how im trying to use it in a more angular fashion
`
// link     :     https://github.com/lampaa/com.lampa.startapp
.factory('$startApp', ['$window', function ($window) {
'use strict';
var sApp;
return {
    set: function(params, extras){
        if($window.plugins.startApp){
            sApp = $window.plugins.startApp.set(params, extras);
        }else{console.error("startApp plugin was not found on window.");}
    },
    check: function(params, extras, successCallback, errorCallback){
        if($window.plugins.startApp){
            $window.plugins.startApp.set(params, extras).check(successCallback, errorCallback);
        }
    },
    start: function(params, extras, successCallback, errorCallback){
        if($window.plugins.startApp){
            $window.plugins.startApp.set(params, extras).start(successCallback, errorCallback);
        }
    }
};

}])`

Hi, @armensg.
method set() return methods start() and check():
app.factory('startAppFactory', ($window) {
    'use strict';

    return $window.plugins.startApp;
}
//use:
app.service('Service', function(MathService){
    this.sApp = startAppFactory.set(...);
});

I got deeplinks to work using the method you posted above for Uber:
Method 1: works!
startApp.set({ /* params */
    "action": "ACTION_VIEW",
    "uri": "uber://?action=setPickup&pickup=my_location&client_id=YOUR_CLIENT_ID"
}).start();

...but is there a way to do it using the uri parameters as extra key/value like in method 2 below:
Method 2: doesn't work
startApp.set({
   "action" : "ACTION_VIEW",
   "uri" : "uber://"
}, {
   "action" :  "setPickup",
   "pickup" :  my_location",
   "client_id" : YOUR_CLIENT_ID
}

The first method, that is working, forces me to create several different methods versus a universal (method 2) approach.  Same question/issue applies for standardizing the calls for iOS too.

Hi, @rolinger!
https://developer.uber.com/docs/riders/ride-requests/tutorials/deep-links/introduction
Parameters can be passed only through the Uri scheme.


compact/downsampling: Build chunks directly in object storage (without using disk)

It's totally doable now. Same with downsampling.

Do you mean read source block chunks from object store, merge them on the fly and stream the output to the compacted block directly to the object store without touching disk?

Correct. Or at least touching some constant amount of disk. Exploring this right now.

The most complex part of it is that we need to extend API with PartUpload to do such streaming: https://docs.aws.amazon.com/AmazonS3/latest/API/API_UploadPart.html
Or we can add even use PartCopy to avoid touching those bytes directly.

Another limit for S3 is Each part must be at least 5 MB in size, except the last part. There is no size limit on the last part of your multipart upload.
For GCS we could use just https://cloud.google.com/storage/docs/resumable-uploads because https://cloud.google.com/storage/docs/json_api/v1/how-tos/multipart-upload is used for something else (metadata upload).

Azure is very flexible for this: https://docs.microsoft.com/en-us/rest/api/storageservices/#blob-service
Damn it's a lot of work :cry:
But the gain is trivial: We don't need disk (or a lot of disks) to write chunks into chunk files. The problem of the disk is not that urging though.

Not sure if I agree with the "trivial" part. In bigger deployments, the size of produced blocks can easily go into the range of hundreds of gigabytes. It would be very nice to have this so hence reopening.

I actually had a troll-ish idea, it might be already feasible. What if we mounted, for example, an S3 bucket via s3fs (https://github.com/s3fs-fuse/s3fs-fuse) and then used the filesystem storage type for Thanos Compactor? In the compaction case, we download whole files so it's not an issue that Thanos is unaware of the underlying storage, right? :trollface:

That might work for a start! (: The problem with this approach is local disk data caching at the end might either end up downloading things on disk or make an additional number of calls unnecessarily. What we want is to have full control over those things.
I agree it's not trivial, but it's also not super difficult. Definitely high on priority list. Maybe good project for LFX summer? 🤗

Was looking for solutions to pull -> transform-> upload very large file to and from obj storage (S3 in my case), thought ther would be some implementations here on thanos :P looks like we're yet to implement this.
Used part upload for my purpose and it works fine, will try to onboard the same here. Will do a poc for S3 and then we can move forward for other services too.

Doing this for compaction is a bit trickier since Prometheus heavily relies on mmap. Downsampling is totally doable, though, since everything is under an interface.

Hey, I would like to resolve this issue and enhance compaction/downsampling. I have written a proposal for GSoC. It would be kind of you if you review and give feedback. @bwplotka  @yeya24 @matej-g
https://drive.google.com/file/d/1oZd3ENSZ7v2hONNf4pDL3yXr_0eHcwsV/view?usp=sharing

Still valid

@yeya24 @bwplotka @GiedriusS Doesn't it mean that for 2 blocks having 1M series each and all series are the same, you'd have to have 2M requests for S3 copying series A from block A and series A from block B chunks to the new block C chunk file?


Report + Negotiation Card: Update Report Content

#227 may propose updates to the content or organization of the report. This issue represents the work required to perform these updates. If no updates are required, this issue may be closed.

Subsumed by #227


sfdx force:source:deploy with RunSpecifiedTests fails with "Method not implemented" error

This bug was originally raised in https://github.com/forcedotcom/salesforcedx-vscode/issues/4056.  The initial issue was that when a deploy (using the extension) failed for the customer, an error was reported, but no additional information was reported.  The user also reported that when they ran:
sfdx force:source:deploy -p force-app/main/default -l RunSpecifiedTests -c -r --json

...from the CLI, they got:
{
  "status": 1,
  "name": "DeployFailed",
  "message": "Deploy failed.",
  "exitCode": 1,
  "commandName": "Deploy",
  "stack": "DeployFailed: Deploy failed.\n    at DeployResultFormatter.display (C:\\Users\\sryan\\AppData\\Local\\sfdx\\client\\7.147.1-1633b17\\node_modules\\@salesforce\\plugin-source\\lib\\formatters\\deployResultFormatter.js:62:19)\n    at Deploy.formatResult (C:\\Users\\sryan\\AppData\\Local\\sfdx\\client\\7.147.1-1633b17\\node_modules\\@salesforce\\plugin-source\\lib\\commands\\force\\source\\deploy.js:133:23)\n    at Deploy.run (C:\\Users\\sryan\\AppData\\Local\\sfdx\\client\\7.147.1-1633b17\\node_modules\\@salesforce\\plugin-source\\lib\\commands\\force\\source\\deploy.js:35:21)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)\n    at async Deploy._run (C:\\Users\\sryan\\AppData\\Local\\sfdx\\client\\7.147.1-1633b17\\node_modules\\@salesforce\\command\\lib\\sfdxCommand.js:81:40)\n    at async Config.runCommand (C:\\Users\\sryan\\AppData\\Local\\sfdx\\client\\7.147.1-1633b17\\node_modules\\@oclif\\config\\lib\\config.js:173:24)\n    at async SfdxMain.run (C:\\Users\\sryan\\AppData\\Local\\sfdx\\client\\7.147.1-1633b17\\node_modules\\@oclif\\command\\lib\\main.js:27:9)\n    at async SfdxMain._run (C:\\Users\\sryan\\AppData\\Local\\sfdx\\client\\7.147.1-1633b17\\node_modules\\@oclif\\command\\lib\\command.js:43:20)\n    at async Object.run (C:\\Users\\sryan\\AppData\\Local\\sfdx\\client\\7.147.1-1633b17\\dist\\cli.js:162:47)",
  "warnings": []
}

The (initial) issue the user was having with deploying using the VS Code extension is separate from this and is a different issue.
I was able to reproduce this myself running:
sfdx force:source:deploy -p force-app/main/default -l RunSpecifiedTests -c -r --json

...and I found that running just:
sfdx force:source:deploy -p force-app/main/default --json

...worked, so there appears there might be an issue with the RunSpecifiedTests option.

That is not a correct usage
sfdx force:source:deploy -p force-app/main/default -l RunSpecifiedTests -c -r --json is invalid because you said "run some specified tests" but didn't specify any tests after -r
That said, the CLI could have validated that flag combo upfront instead of doing the deploy and then having the test step fail because of bad inputs.

link to the underlying oclif bug https://github.com/oclif/core/issues/413

Closing this as the CLI's handle the flag validation better, the OCLIF bug still exists of reading the next arg into the array, but you can't replicate this anymore
 ➜  sf force source deploy -p force-app -l RunSpecifiedTests -c -r --json
{
  "code": 1,
  "context": "Deploy",
  "commandName": "Deploy",
  "message": "Flag --runtests expects a value",
...
 ➜  sf project deploy start --source-dir force-app/main/default --test-level RunSpecifiedTests --tests --json
{
  "code": 1,
  "context": "DeployMetadata",
  "commandName": "DeployMetadata",
  "message": "Flag --tests expects a value",


npm distribution incident

Incident Details:
Issue: empty npm registry package.
Description: This incident occurred due to issues with npm distribution during a switch from CommonJS to ES Modules. Initially, it was caused by a configuration problem where noEmit was set to false, resulting in the publishing of an empty package to the npm registry. The problem resurfaced during a release workflow where the npm build step was missing.
Incident Recap:
First Incident:
Cause: noEmit set to false, resulting in an empty package being published.
Resolution: Configuration issue identified and corrected after review.
Action Taken: Added a verification step in ci/cd which checks if build was successful or not by importing from /dist folder.
Second Incident:
Cause: Missing npm build step in release workflow. Since this is a separate workflow from the ci/cd, the build verification was not present in this workflow.
Resolution: Build step added after review.
Note: A verification step was already added to verify if the npm publish is successful or not. But this was not able to identify the bug in time. We need to revisit and fix it.
Observations:
PR review: No merge was without a PR review.
Over-Reliance on Automation: Both incidents highlight an over-reliance on automation without adequate manual verification.
Lack of QA: The absence of a quality assurance process contributed to the oversight of critical steps in the release workflow.
Action Items:

More details in the PR - Every PR should have a proof of QA by the author. We can have screenshots for manual verification. Without that PR should not be merged.
Implement Manual Verification: The manual testing should be done in form of a checklist. This checklist is to be created by the author of the PR and also should be added in the PR information.
Enhance Automation: Strengthen quality assurance procedures to prevent similar incidents in the future.
QA report in the ticket:  Once a PR is merged then before closing the ticket we should add manual entry about the QA which was done after merging the PR to main.

@dtornow @dfarr @guergabo  please review this and please add your inputs. Thank you.

Note: A verification step was already added to verify if the npm publish is successful or not. But this was not able to identify the bug in time. We need to revisit and fix it.

AFAIU the verification step "checks if build was successful or not by importing from /dist folder". Can you detail why this didn't trigger?

Note: A verification step was already added to verify if the npm publish is successful or not. But this was not able to identify the bug in time. We need to revisit and fix it.

AFAIU the verification step "checks if build was successful or not by importing from /dist folder". Can you detail why this didn't trigger?

Added the following information -
Since this is a separate workflow from the ci/cd, the build verification was not present in this workflow.

The verification step is included in the updated release workflow #88


1st of July is being considered a holiday in Portugal

Which country is affected?
Portugal.
Description
This is really a cool package! I'm using it for a personal project and, generating the holidays for Portugal in July 2019, it gives me the 1st of July as a holiday, which isn't. The only country i know it's a holiday is Canada.

This is not an error, it is a local holiday for Madeira.
You need to check the "Global" property to verify if it is national (true) or not (false)

@tiagossa1 Can you check my update?

New version is published


ReceiverDeterminationInService is missing

Issue description

ReceiverDeterminationInService is missing
Feedback Type (Optional)
content gaps
Page Title on SAP Help Portal (prefilled)
Add an SAP Process Orchestration System
Page URL on SAP Help Portal (prefilled)
https://help.sap.com/docs/integration-suite/sap-integration-suite/add-sap-process-orchestration-system#prerequisites

Hi @ottfro1, thank you for your feedback! We’ll look into it and come back to you if we have any questions.


Add building of TV example to gn_build.sh

Problem
TV example app is not built using gn_build.sh script
Summary of Changes

Updated the ./BUILD.gn to support building of tv-example-app

Test

Tested locally using chip-tool client and tv-server
Used the  ./gn_build.sh  to verify the building is successful

@saurabhst @andy31415 @msandstedt?

@Damian-Nordic @jmartinez-silabs ?


Reverts Zone.qualifier

Zone.qualifier was only introduced to support creation of Route53 hosted
zones. We were using this for the caller reference. Turns out that
caller references can never be reused. This means that if you delete a
zone, you can never create a new one with the same caller reference. As
such, we shouldn't expose this to users.
see #350

Live tests pass

failure is due to github DDOS I think. http://www.traviscistatus.com/incidents/tfk72kps8l7g


Satellite Assembly issue

Description
In our productive solution we encountered an issue in the behavior of MsTest.
Some UnitTest run well, but throws following error messages in the test output:
---------- Starting test run ----------
Problem bei der Testlaufbereitstellung: Fehler beim Abrufen der Satellitenassemblys für Bereitstellungselement "C:\Users\xxx\Desktop\SatelliteAssemblyTest\ClassLibrary.Tests\bin\Debug\ClassLibrary.dll": System.ArgumentOutOfRangeException: startIndex darf nicht länger als die Länge der Zeichenfolge sein.

We have made some researches on that and created a simple test solution to reproduce this error.
I will attach the test project to these issue for further analysis.
SatelliteAssemblyTest.zip
We found, the error gets raised if 2 conditions are met:

There is a project with assembly name “xxx.Resources”
The unit test uses the “DeploymentItem” attribute

Environment
The target framework is net 4.7.0
Package References:
<package id="MSTest.TestAdapter" version="2.1.1" targetFramework="net47" />
<package id="MSTest.TestFramework" version="2.1.1" targetFramework="net47" />

Running and debugging tests from Test Explorer in Visual Studio 16.7.2. Windows 10.0.19041
We also have tested it with the MSTest current version 2.1.2 but the issue occurs there to.

This should be fixed with the latest release of TestFx.


Remove java doc about overflow cases in Duration.toMillis().

Closes https://github.com/census-instrumentation/opencensus-java/issues/1133.
We have a limit on seconds and nanoseconds for Duration. That range is within (Long.MIN_VALUE, Long.MAX_VALUE) so the conversion should never cause overflow.

Codecov Report

Merging #1134 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1134      +/-   ##
============================================
- Coverage     82.48%   82.46%   -0.02%
  Complexity     1211     1211
============================================
  Files           187      187
  Lines          5767     5767
  Branches        523      523
============================================
- Hits           4757     4756       -1
  Misses          878      878
- Partials        132      133       +1

Impacted Files
Coverage Δ
Complexity Δ

...i/src/main/java/io/opencensus/common/Duration.java
100% <ø> (ø)
15 <0> (ø)
:arrow_down:

...census/implcore/trace/export/SpanExporterImpl.java
96.66% <0%> (-1.67%)
7% <0%> (ø)

Continue to review full report at Codecov.

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


fix(profile): fix missing profile on Android

Description
Upgrades sodium-react-native-direct to fix some methods on Android.
Linked Issues/Tasks
Fixes https://github.com/synonymdev/bitkit/issues/1460
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (improving code without creating new functionality)

Tests

[ ] Detox test
[ ] Unit test
[x] No test

I'm testing these changes but I still get this error https://github.com/synonymdev/bitkit/issues/1460


[BUG] currentBreak shows wrong position

Describe the bug
When dragged, currentBreak function show 'old' position
Also, when just touched, it triggers onDrag method
To Reproduce

Initially opened panel
Drag up
Console log should show 'top'

Expected behavior
When dragged up currentBreak should show 'top'
Screenshots
https://streamable.com/c58vwo
**Desktop (please complete the following information):
MacOS (Big Sur)
Chrome
Version 87.0.4280.67
**Smartphone (please complete the following information):
Huwaei Y6
Android 9
Chrome
87.0.4280.86
Additional context
Add any other context about the problem here.

Fix it with onTransitionEnd, even I think that on tap onDrag function should be triggered.

@sasafister All right, currentBreak changes after transition.


Fix unexpanded tildes in cache dirs

Makes defaults make sense, so you can run lakefs in rocks mode by setting just cataloger.type: rocks.

Thanks -- silly bug, caused me lots of grief.

Thanks -- silly bug, caused me lots of grief.

Codecov Report

Merging #1171 (313dce3) into master (575533a) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1171      +/-   ##
==========================================
- Coverage   46.93%   46.89%   -0.04%
==========================================
  Files         195      195
  Lines       15020    15023       +3
==========================================
- Hits         7049     7045       -4
- Misses       7044     7049       +5
- Partials      927      929       +2

Impacted Files
Coverage Δ

config/config.go
30.30% <0.00%> (-0.47%)
:arrow_down:

catalog/mvcc/cataloger_create_entry.go
94.73% <0.00%> (-5.27%)
:arrow_down:

catalog/mvcc/cataloger_create_branch.go
81.53% <0.00%> (-3.08%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #1171 (313dce3) into master (575533a) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1171      +/-   ##
==========================================
- Coverage   46.93%   46.89%   -0.04%
==========================================
  Files         195      195
  Lines       15020    15023       +3
==========================================
- Hits         7049     7045       -4
- Misses       7044     7049       +5
- Partials      927      929       +2

Impacted Files
Coverage Δ

config/config.go
30.30% <0.00%> (-0.47%)
:arrow_down:

catalog/mvcc/cataloger_create_entry.go
94.73% <0.00%> (-5.27%)
:arrow_down:

catalog/mvcc/cataloger_create_branch.go
81.53% <0.00%> (-3.08%)
:arrow_down:

Continue to review full report at Codecov.

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


Houdini Explicitly collect correct frame name even in case of single frame render when frameStart is provided

(cherry picked from commit 0f82a753560e362be731d0d14177c4f3edbf1db0)
Brief description
There was a bug in Houdini that whenever you do have a frame range set of just a single frame, e.g. 1001-1001 then collect_frames would actually eval the filename on the current frame instead of that start frame. Thus if you're writing to a file with $F the file would be named different than what the ROP node would actually output
Additional info
Here's an example error that could happen. The integrator here fails to copy the file becuase the ROP node actually rendered another file than was "collected" as expected resulting filename from the ROP node.
Instance:
vdbcachesmoke_room (160) [1001.0 - 1001.0]
Message:
[WinError -2147024894] The system cannot find the file specified
Line:
933
Traceback:
Traceback (most recent call last):
  File "C:\Program Files (x86)\OpenPype\dependencies\pyblish\plugin.py", line 522, in __explicit_process
    runner(*args)
  File "C:\Users\User\AppData\Local\pypeclub\openpype\openpype-v3.8.1\openpype\plugins\publish\integrate_new.py", line 138, in process
  File "C:\Program Files (x86)\OpenPype\dependencies\six.py", line 719, in reraise
    raise value
  File "C:\Users\User\AppData\Local\pypeclub\openpype\openpype-v3.8.1\openpype\plugins\publish\integrate_new.py", line 129, in process
  File "C:\Users\User\AppData\Local\pypeclub\openpype\openpype-v3.8.1\openpype\plugins\publish\integrate_new.py", line 597, in register
  File "C:\Users\User\AppData\Local\pypeclub\openpype\openpype-v3.8.1\openpype\plugins\publish\integrate_new.py", line 653, in integrate
  File "C:\Users\User\AppData\Local\pypeclub\openpype\openpype-v3.8.1\openpype\plugins\publish\integrate_new.py", line 699, in copy_file
  File "C:\Program Files (x86)\OpenPype\dependencies\speedcopy\__init__.py", line 289, in copyfile
    '\\\\?\\' + dest_file, None)
  File "_ctypes/callproc.c", line 933, in GetResult
OSError: [WinError -2147024894] The system cannot find the file specified

Testing notes:

Create a vdbCache instance to extract from frame 1001 to 1001.
Set current frame to 1002 in timeline.
Publish.

The render rop will render 1001 however the collect_frames and thus integrator thinks 1002 should be published.
Additional Context
I've also added a warning if it's rendering without a frame range set and thus renders current frame - since that is less reliable in a production environment and thus discouraged?

Sorry for that commit mess there - my app tried to be clever and exclude an edit from the commits each time. 🔥


On subsequent searches, filters appear selected but are not applied (react-search-refiners)

Category

[ ] Question
[x] Bug
[ ] Enhancement

Authors
@wobba
@FranckyC
Expected or Desired Behavior
When you search for something, apply filters, and then search for something else after, the search results should either:

Show the filters you previously applied as selected and apply them to the results (best option).
Clear all filters and show results with no filters (worse, but still clear to users).

Observed Behavior
When you search for something, apply filters, and then search for something else after, your previous set of filters appears to still be selected, but is not actually applied to search results.  To apply those filters again you need to manually clear and re-apply them.
Steps to Reproduce

Search for something.
Apply filters.
Without reloading the page, search again.
Observe that the filters are still listed as selected (shown at the top of the search results and checked in the filters panel), but they are not actually affecting the search results.
Manually uncheck and check the "selected" filters in the filters panel and observe that they are now being applied to the search results.

@mgordin should be resolved with the latest version.


set_charging_current not working

EU/Kia E-Niro 2020
Latest Integration 2.26.7
Car plugged into a ChargeAmps Halo charger and I'd like to be able to change the speed of charging.
When manually changing the current setting in the car (Charging current: AC charger portable) the power changes between 10.5/7.8/5.1 kW.
When using the service kia_uvo.set_charging_current it changes nothing but the service run as it should (no errors).
Its a bit tricky in the cars interface since you can change both "AC charger portable" and "AC charger stationary"
sidenote: Not clear what the difference is but when changing settings under "stationary" nothing happens so it is the 'portable' setting that seams to be the one to be fiddeling with.

Hello,
I have a MY20 Niro as well.
I asked on the Bluelinky discord, and it sounds like even though the API "accepts" this command for our model, it does not support this feature.

Ok, thanx a lot for the information


Revert "jssrc2cpg: Handle Destructured Params (#2883)"

This reverts commit b5336999a69a25cccc69531629af660e263d9999.
@DavidBakerEffendi Had to revert this as it creates identifier nodes not being included into the AST. That crashes many passes later on. Check e.g., tarpit-nodejs

@DavidBakerEffendi There is also another problem with some of your recent changes:
On tarpit-nodejs again:
joern> cpg.method.whereNot(_._astIn).size
res5: Int = 46

Could you checkt this and maybe push a fix directly at this PR here?

@max-leuthaeuser thanks, will do, I'll also see if I can create a unit test from that project to reproduce with.

FYI looking into this today

@max-leuthaeuser decided to do it in my own PR since it was a 2 line fix, then added 2 lines of tests to verify it

@max-leuthaeuser decided to do it in my own PR since it was a 2 line fix, then added 2 lines of tests to verify it

Then again, whether this doesn't create an invalid CPG is another question lol, but it seems fine with the bit of querying I did with the resulting CPGs

Described bug resolved by https://github.com/joernio/joern/pull/2979


zephyr: add minimal footprint configuration for nrf52

Added Kconfig project configuration whit shows the
minimal flash footprint on nRF2832 SoC targets.
Signed-off-by: Andrzej Puzdrowski<EMAIL_ADDRESS>

This is configuration I used as base for different revision comparison recently.


Implemented gradle github action for PR

#19

@AngieD8615 what version of JAVA are you using? I tried building against 17 but the build is failing with some errors.
https://github.com/AngieD8615/question/runs/6977696595?check_suite_focus=true
/home/runner/work/question/question/src/test/java/com/example/demo2/question/QuestionControllerTests.java:57: error: incompatible types: TopicEnum cannot be converted to List<TopicEntity>
                TopicEnum.ACCELERATION,
> Task :compileTestJava
                         ^
/home/runner/work/question/question/src/test/java/com/example/demo2/question/QuestionControllerTests.java:66: error: incompatible types: TopicEnum cannot be converted to List<TopicEntity>
                TopicEnum.ACCELERATION,
                         ^
/home/runner/work/question/question/src/test/java/com/example/demo2/question/QuestionControllerTests.java:86: error: incompatible types: TopicEnum cannot be converted to List<TopicEntity>
                TopicEnum.ACCELERATION,
                         ^
/home/runner/work/question/question/src/test/java/com/example/demo2/question/QuestionServiceTest.java:54: error: incompatible types: TopicEnum cannot be converted to List<TopicEntity>
                TopicEnum.ACCELERATION,
                         ^
/home/runner/work/question/question/src/test/java/com/example/demo2/question/QuestionServiceTest.java:64: error: incompatible types: TopicEnum cannot be converted to List<TopicEntity>
                TopicEnum.ACCELERATION,
                         ^
/home/runner/work/question/question/src/test/java/com/example/demo2/question/QuestionServiceTest.java:72: error: incompatible types: TopicEnum cannot be converted to List<TopicEntity>
                TopicEnum.ACCELERATION,
                         ^

@sangyum Yes, I am using Java 17. It looks like I merged to main without fixing some tests. I will fix the tests and update main shortly.


Kiwee UI

https://sanestinasblog.files.wordpress.com/2018/03/grapes-bowl.jpg?w=546&h=546&crop=1&zoom=2


Change the way server startup failure is forced to use illegal ports.

Description
One of the tests in KituraTests.TestServer was testing the lifecycle callback for server startup failure. The server startup failure was forced by starting the HTTP server and FastCGI server twice. This no longer fails as BlueSocket has added SO_REUSEPORT to the options set when listening on a port. The SO_REUSEPORT option allows multiple processes/threads to listen on the same port.
This PR changes the way the server startup failures are forced by using illegal ports.
Motivation and Context
Kitura's CI is failing.
How Has This Been Tested?
Kitura's unit tests were run locally.
Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

Codecov Report

Merging #1054 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1054   +/-   ##
=======================================
  Coverage   89.75%   89.75%
=======================================
  Files          35       35
  Lines        1698     1698
=======================================
  Hits         1524     1524
  Misses        174      174

Flag
Coverage Δ

#Kitura
89.75% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[ISSUE-4588] Refactor underscored functions

:ok_hand:

🆗
Side note : files must not contains many classes (PSR-1 : http://www.php-fig.org/psr/psr-1/#3-namespace-and-class-names) => TestResourceApi

Why scrutinizer is not plugged on this project ? (it can avoid this kind of problem)


PROD-579/do-not-display-streak-info-after-claiming-streak

@danijel-radomiljac are you looking into this?


chore(main): release cli 1.1.0

:robot: I have created a release beep boop
1.1.0 (2024-06-02)
Features

initial commit (d108533)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/flexydox/flexydox/releases/tag/cli%40v1.1.0 :sunflower:


DJI V2 goggles and AU 1 no audio

Hi, I've installed the v05 packages in both the AU and goggles v2. I'm not getting any audio.

I have the same issue, but it records to my SD Card if its inserted it wont work to the goggles, recording or not though..
SO.. I've figured out you can get the Live Audio to work on the V2's with these .IPK's if you just completely remove the SD Card from the Air Unit before power up, and you will get live Audio to the Goggles through the Headphone port. It doesn't record on the DVR Video file, and you cant record that nice 1080p60 AND have live audio.
So it seems like Live Audio and no recording on the Air Unit, OR you can have Audio recorded to the SD Card's videos..
I cant get it do both no matter how i install the packages, set the volume through CLI, etc. The Live Audio does indeed work but only because i happened to try it first without my SD Card by accident and then realized the second flight, it was the SD being physically inserted that caused the issues for me. I think i even made sure to stop the auto record on take off to try that and see if it worked when not recording, but it doesnt.
So basically i got lucky and figured this out by pure chance, and since i almost always run an SD Card im sure i would of just given up after a while because no one has OG Air units anymore and no one seemed to care that much about this feature or didnt even know it existed the whole time..

I can confirm that if I remove SD card from Air Unit, audio-out starts to work when IPK's are installed. It even looks like that there is no need to have recording running.
One more note to installation process: I only could install air-side IPK when there was already system package wtfos-modloader 0.2.2 installed  in the unit - e.g. via package manager in browser.


When using search geometry to refine and search geometry is selected, maintain the edit mode until the edit is done

After finishing moving a vertex, the shape needs to be clicked again for the vertices to show up, can we keep it in edit mode until user finishes editing (perhaps when user clicks somewhere outside of the shape to indicate finish editing)

Verified


Update and rename LICENSE.md to LICENSE-Apache.md

See https://github.com/pluto/aes-proof/pull/68#issuecomment-2343888000


fix: do no allow min or max fraction digits to go out of range

Related issues 🔗
Closes #1803

Test fails are incorrect, likely due to branch management changes. Merging anyway for this reason


Busted documentation page

The documentation page for PERF203 advises readers to "... [u]se instead" source which is not only fails to be stylish, but does not even compile (roughly, it has break where pass perhaps was intended).  Also, early experiments suggest that Ruff fires PERF203 even for Py3.11 and 3.12, contrary to the text.
I have little sense of how Ruff documentation is maintained.  Perhaps someone's already planning corrections for this page.  If not, and if I learn the appropriate workflow, I'll happily submit my own suggestions for revisions.

Thanks! Are you specifying a target-version? Ruff doesn’t rely on the currently-running Python version, it picks up the minimum-supported version either from your pyproject.toml or the target-version setting.
The example could definitely be improved — PR welcome, it would only require editing the Markdown around here.


[Service BUG] Service regression resulting in updates on error code and error messages value

Reference: https://github.com/Azure/azure-sdk-for-java/pull/11999/files#diff-520506b294f3494cdf4d1cd7f0b75623R139
Only impacting in westcentralus region.
It should not be different and should be fixed by preview 4.

.NET issue tracking this with service team - Azure/azure-sdk-for-net#12478

Moving this to P5


使用Swift 5.0.1编译的模块不能被Swift 5.2.2编译器导入:

使用Swift 5.0.1编译的模块不能被Swift 5.2.2编译器导入:/riot-ios-develop/Pods/JitsiMeetSDK/Frameworks/JitsiMeet.framework/Modules/JitsiMeet.swiftmodule/arm64.swiftmodule

what?

Showing Recent Messages
'/Pods/JitsiMeetSDK/Frameworks/JitsiMeet.framework/JitsiMeet' does not contain bitcode. You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target. file '/Pods/JitsiMeetSDK/Frameworks/JitsiMeet.framework/JitsiMeet' for architecture arm64
disable bitcode for this target and report it.

what?
disable bitcode for this target and report it.
Showing Recent Messages
'/Pods/JitsiMeetSDK/Frameworks/JitsiMeet.framework/JitsiMeet' does not contain bitcode. You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target. file '/Pods/JitsiMeetSDK/Frameworks/JitsiMeet.framework/JitsiMeet' for architecture arm64


Full name field..

This is a feature request...
Can we have a full name field added to the user form?  Logins are usually nicknames or first initial and last name.  It makes it easier if we can see the full name of the person so later when we look at it we can remember who that person was.
Thank you!!!
~ Merlin

Yeah why not. Will do that in v2.0

I really love you people!

While you are at it.. Perhaps add "Company Name" and also "eMail Address"..
Now I know you are thinking.. "Well, I have the eMail address in the SOA and that is redundant.."
But at some point you might be asked to add a password reset system.. and that can only be done if the account has an eMail address tied to it.  Also.. when the admin is looking at the user and realizes they want to contact them.  It is one less step to get to the address.  (:  Or leave it out.. I am just thinking out loud here.

You know on the eMail address... Maybe a new issue for this but if we could use the "@" sign and period in the login name we could literally have the option of using ether the login name OR eMail address and that would work out wonderfully.

Email seems to be a good idea, the company will be clear from the Email.


Include Swift type hinting

Please include type hinting to make implementation in Swift easier

Check #25

Just bumped #25


Error 0x80370109 when using WSL

Windows Version
Microsoft Windows [Version 10.0.22631.4249]
WSL Version
WSL version: <IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
Kernel version: <IP_ADDRESS>-2
Distro Version
n/a
Other Software
No response
Repro Steps
Attempt to start a Ubuntu terminal
Expected Behavior
Ubuntu starts
Actual Behavior
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80370109
Error: 0x80370109 The operation timed out because a response was not received from the virtual machine or container.
Diagnostic Logs
WslLogs-2024-09-29_12-24-46.zip

/dupe #12090


Generate OPIL from Measurement table

to completely transition a SR to OPIL, IP will need to generate OPIL from a measurement table

@jakebeal I'm not sure that generating OPIL from a measurement table is the right thing to do here. I recall in one of our old discussions, we talked about using SBOL3's combinatorial derivation object to encode this information. Is this correct? If not, what type of OPIL Parameter object does this fall under? A MeasureParameter?

@tramyn  This isn't the Parameter objects of OPIL, but the sampleSet properties, which point to SBOL3 CombinatorialDerivation objects in the UML diagrams.  So yes, you're right that it should be CombinatorialDerivation, and I'm right that this is included in the OPIL structures. :-)


Support inexact tarball name specification for loading

When loading a container one should be able to specify a partial name for the tarball. For example let's assume /usr/share/suse-docker-images/native/aws-cli.x86_64-0.1.1-Build1.1.docker.tar exists but the file is to be loaded through a %post action in an RPM package where we cannot determine the version string. As such being able to specify
flake-ctl podman load --oci /usr/share/suse-docker-images/native/aws-cli
would be very convenient and flake-ctl can sort out the container to load. If multiple tarballs that match the glob exist the latest version should be loaded to match the default behavior of container runtimes which also always use what is tagged as latest.

Fixed with d89451c6fb3a16b761bb104c3cddeb691fbb3690


Add support for healthcheck in compose to swarm

Fixes #28520
Adds support for healtcheck in the composefile v3 format, used it docker stack deploy --compose-file 🐮.
version: "3"

services:
  foo:
    image: busybox
    command: top
    healthcheck:
      test: cat /etc/password
      interval: 10s
      timeout: 2s
      retries: 3

/cc @dnephin @aanand @icecrime
This bumps compose-file to a70cb6ea5c403c383b5117dcd1dea78880a47824.
🐸

@aanand @dnephin updated with latest changes (commant -> test, disable, …)

LGTM


Allow settings of custom gitlab.rb config file

Gives the ability to set the gitlab.rb config file to your own puppet path overriding hiera.

Thanks for your PR, good idea. Would you mind checking the failing tests and update them accordingly to your change?


Disabling legacy console mode does not show any text on cmd console

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd]
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce

Expected behavior

Actual behavior

Hi,
I wanted to disable non-legacy mode of the command prompt so I unchecked the Legacy mode option. After reopening cmd window I cannot see what I typed.  Then switching back to Legacy mode and reopening a cmd tab makes the newly typed content  visible.

Explanation and workarounds in /dup #2319.