Shell completion

Hi, are there any plans to make shell completion?

Good point, shell completion is a nice feature to have, I will have a look at it.

@akemrir Hi, shell completion is implemented in pull request #11, you may try the latest release v0.8.0, any feedback is welcome.

@jxskiss Hi, I will surely give it a ride :) Just saw email about.
Any plans for fish shell also?

Ok. Just checked. Completion works just fine :) It's a nice feature.
I can see command and it's flags.
Checked some combinations around that:
Subcommands with different flags
Main commands without groups
Different completion command name

Everything worked.
Do you plan to expose descriptions also?
Like here?

Cobra also allows to specify function to provide valid arguments for completion. To complete values for parameters.

Nonethenless very nice feature :)
I like mcli for it's simplicity.

Command without group is not suggested.
// A sub-command can also be added without registering the group.
mcli.Add("group3 sub1 subsub1", runGroup3Sub1Subsub1, "Blah blah Blah"

Maybe because of space?

After adding a group, it works like previous group.

description - will check for bash and fish - I don't have accesss to powershell
no no, not argument validation. But suggestion of arguments for example
--port TAB TAB -> 3000, 4000, 5000
or with use of own function

But I don't know where to put it properly.

nice ~
the most commonly used arg suggestion may be providing choice from a list (enum), for suggestion of this type, suggetion and validation are somehow related.
For more general arg suggestion, the use-case could be pretty complicated, I'm not sure how much effort does it need and whether it worth the complication.
If you want to give it a try, maybe a new struct tag comp:"xxxx", then passing arg completion functions by adding a new ParseOpt is a good start point.

Yep. Wanna try.
In most cases (at least for me) suggestions are local, based for example on configuration.
So in this case I would load config right before parsing and make completion function aware of that.
That's the plan. Will see how it will go.

Could you show me output of:
issue  --mcli-generate-completion

Ok. Found it why I don't see descriptions.
It's based on shell set in env not the one used in background.
In my case even when I launch terminal -e zsh I would have SHELL=/usr/bin/fish
I will analyse and maybe prepare PR for this.
Later for arguments completion.

@akemrir The way to check shell type is borrowed from other cli library (without much thinking), maybe there is better way to do this. Perhaps we can pass the shell type in command line, since we use seperate completion script for each shell type, we have full control on the command line arguments.

That's actually good idea. And based on that type change separator contents text and description.
--mcli-generate-completion bash
--mcli-generate-completion zsh

Fish shell uses \t
Zsh uses :
We need to check bash

ok, I have mvp for unixes

Now I need to work on tests.
Do you plan to suggest also help params?

Yes, I think help params should also be suggested.

Ok, I've made second PR.
I wrote shell completion with descriptions.
It needs review. I made two comments on purpose.
Trying to figure out how to handle unsupported shells.
It should not happen, because we supply proper completion parameter upfront.
But if someone would change it, I think we should handle it somehow.
Maybe to throw error? (TODO in code)

Do you plan any more changes? I will start in a free time with flag value suggestions.

@akemrir No. I have tagged a new version v0.8.3, you can start from the latest main branch.

Thanks for info.

Hi, I was doing research and checking out this parse options.
You were right. It's quite complicated to do. Till now I haven't got it working.
Beside of that, I checked how cobra is doing this.
mcli currently have suggestions for commands and flags.
On the other hand cobra is passing completion function as argument, doing it this way:

This is on command creation level.
Next when doing completion for current command it executes this function to get suggestions:

Only when dash is present it suggests flags for command:

Maybe we could do the same or similar?

I know this is something opposite. Flag vs arguments completion.
I am not stopping approach for flags values suggestion.
What do you think about this idea?

I haven't understand the target use-case, could you state the user problem you are trying to solve in more details, better with real-world examples.

Ok. Let's see.
We have three topics:

flags completion
help completion
arguments completion

This is for third topic.
For example you would have this kind of configuration (taskfile.dev).

And you would like to have many user predefined commands/tasks.
You would need to have help to show all of them, or suggest as valid arguments.
task <tab>
task hello
task hello <tab>
task hello world

hello and world are both tasks to execute

By "arguments completion", do you mean positional arguments? Do we need to suggest value for a flag which takes argument?

Yes positional arguments. hello and world came from function outside of mcli lib.
Flags always start with dash -
So procedure would be like this:

do I have subcommand to suggest?
if there is no subcommands and no flags suggested execute command arguments function
if we type dash - try to suggest flags
if last already suggested is flag check if it has function and execute it to suggest value

When we have exact command we could always try to suggest or arguments or flags or flag values.
Because we could have dash condition.
To be more precise and complete, as in example mcli/examples/github-cli/main.go
Subcommand suggestion (done)
gh <tab>
gh browse

(command function) Function suggesting positional argument for command, e.g. pull request number:
gh browse <tab>
gh browse 217

(argument function) Function suggesting branches names for flags:
gh browse -b <tab>
gh browse -b PROJ-123

Our normal flag suggestion (done)
gh browse -<tab>
gh browse --settings

(argument function) at the end value for settings:
gh browse --settings <tab>

In general, I want to pass completion function for command to get positional arguments.
And second one, per flag to get flag value. To simplify it should receive only one value.

I see, powerful argument suggestion is very helpful to end user indeed.
It should be quite straightforward for flag value completion, but for positional arguments, it's a little complicated to decide which argument to suggest. Also boolean flag accepts flag value optionally, we may need to really parse the comamnd line to make a correct decision.
e.g.
type args struct {
	Approve bool   `cli:"-a"`
	Name    string `cli:"-n"`
	Branch  string `cli:"branch"`
	Message string `cli:"message"`
}

For this definition, these commands are all equal, which means Approve=true, Name=Tom.

program -a -b=Tom
program -a=1 -b=Tom
program -a=true -b Tom

Go ahead. I'm glad to give a hand if needed.

I added some comments in branch tmp_arg_suggestion_draft, you may check it to get some thoughts.

I had similar ideas. https://github.com/akemrir/mcli/tree/PR-003
Now I need to read through over all of these changes.

Hi, I have mvp. But I have question:
Do we want to allow more than one appearance of same flag? or this was a bug during completion?
On my current state mcli will not suggest already suggested flag.
I made also a change, which would require user to type hyphen to suggest flags.
We need also review this -a=1 situation. Suggestions right now don't make use of = character.
Right now I am on the stage of writing tests to cover various cases.

ok, found answer for double appearance of flag

I want to discuss directives.
Let's take a look on these examples:
cobra/zsh_completions.go at main · spf13/cobra
cobra/bash_completions.go at main · spf13/cobra
cobra/fish_completions.go at main · spf13/cobra
With current code we enforce developer to fill completion arguments with descriptions.
When description is empty string we leave only left side, option name.
When I take a look on these directives, they feel a bit obsolete.
    local shellCompDirectiveError=%[3]d
    local shellCompDirectiveNoSpace=%[4]d
    local shellCompDirectiveNoFileComp=%[5]d
    local shellCompDirectiveFilterFileExt=%[6]d
    local shellCompDirectiveFilterDirs=%[7]d
    local shellCompDirectiveKeepOrder=%[8]d

The plan is to provide completion by function at start.
But all of these extra options could be done inside of go code.
Only in cases when we would like to drop completion to shell in some special simple cases:

only directories
only files
files filtered by extension.

These simple cases could be too complex also. Which app/utility to use to handle them? find, ls?
What to do if it would be missing? (windows, minimal docker image)
When someone would like to have extra logic changing the behavior, he needs to write some code nonetheless.
On the other hand, you suggested this:
`comp:"nofile"`
`comp:"enums=a,b,c,d;"`
`comp:"fn=funcName"`
`comp:"listExt=json,yaml,toml"`
`comp:"listFiles"`  // and listDirs, listDirsAndFiles, etc.

So in fact they comply with internal functions?
Or we expect to drop handling to shell?
I see these enormous shell functions as over complication.
It should be simple, developer should be responsible for handling complex cases.
Your examples should be simple functions too, from mcli side.
To not expose them to performance degradation.
If someone wants to customize, do traversal, have faster execution etc., he should write own function.
By the way, I used double nested slices of strings in code, to be able to pass description.
mcli know which shell is used, hence knows which separator to use.
TODO: Fix bash, only which stoppped to work properly.

@akemrir Yes, I also think we should keep it simple on mcli side. By the draft code I given above, it is very incomplete and just providing some thoughts.
I will take a look at the PR, thank you for the contribution and deep thinking of this.

Ok, please review,so I will address all remakrs at once. During update I will clean directives from code.

Please take a look on bugfix. I had to update fish shell completion.

@akemrir Thank you very much for all the effort on this feature, I have merged the new PR.
I think this feature is almost finished, I am going to close this if I you also agree.

I agree. Just tested with 0.9.1 and worked like charm :)
We could plan help suggestions but in separate issue, if needed.
Thank you very much for help with development.


Use federated OIDC permissions with AWS to read S3 buckets.

What does this change?

How to test

How can we measure success?

Have we considered potential risks?

Part of https://github.com/wellcomecollection/platform-infrastructure/issues/431


Bus error on Raspberry Pi when using mongodb

What steps will reproduce the problem?

run dart file on command line using mongodb

----------------------------mongotest.dart-----------------------------
import 'package:mongo_dart/mongo_dart.dart';
import 'package:logging/logging.dart';
void main() {
dbConnection();
}
void dbConnection() {
Logger.root.level=Level.ALL;
Logger.root.onRecord.listen((LogRecord rec) {
print('${rec.level.name}: ${rec.time}: ${rec.message}');
});
var db = new Db('mongodb://<IP_ADDRESS>/test');
print("Here");
db.open().then((success) {
print("Here B");
if (success) {
var _bios = db.collection("std");
Cursor cur = _bios.find();
cur.each((Map bio) {
print(bio);
});
}
});
}
----------------------------pubspec.yaml-----------------------------
name: MongoTest
description: A sample command-line application
dependencies:
mongo_dart: any
What is the expected output? What do you see instead?
Here
Here B
(and data in the db)
But I got
Here
[  724.430982] Unhandled fault: alignment exception (0x001) at 0x75dc6bc5
Bus-Zugriffsfehler
What version of the product are you using?
Dart VM version: 1.12.0-edge.9f00e0591816edd4cca0cf349e87b7b4f82cd2e6 (Sun Jun 28 16:19:06 2015) on "linux_arm"
On what operating system?
Debian raspbian wheezy on Raspberry Pi 2, but I got the same error late last year when still using Raspberry Pi Model B;
What browser (if applicable)?
N/A command line app
Please provide any additional information below.
I had hoped that this were resolved by https://github.com/dart-lang/sdk/issues/22384, but apparently it wasn't
cat /proc/cpu/alignment before
User:       0
System:     0 (  (null))
Skipped:    0
Half:       0
Word:       0
DWord:      0
Multi:      0
User faults:    2 (fixup)
cat /proc/cpu/alignment after
User:       3
System:     0 (  (null))
Skipped:    1
Half:       0
Word:       0
DWord:      1
Multi:      1
User faults:    2 (fixup)

https://github.com/dart-lang/sdk/issues/22151


Resolved CPHD Version 1.0 bug for PVP parsing

Many thanks for finding that bug! This should have only be an issue for the CPHD version 1.0, so the correct fix was to ensure that CPHD Version 1 PVP vectors were returning offset in bytes versus field count. That was the intent from the start, I just made a mistake.


Use a more permissive license

Problem
The license of this package (being AGPL) is prohibitive that it cannot be used in a lot of environments. Would it be possible to relicense this under MIT/BSD/Apache2?
Solution
Pick a more liberal license.
Alternatives
No response

Please correct me if I'm wrong, but as far as I'm aware, the AGPL license's main prohibition is developing forks of the library without making it public or contributing back to the original. I'm absolutely willing to work things out, could you please explain what kind of environments wouldn't work with the current license?

I think I might have found what you mean actually...

conditioned on making available complete source code of licensed works and modifications, which include larger works using a licensed work, under the same license.

The license has been changed as of a6ff33d and will be in effect in the next release :+1:

LGPL works for my experimental case. However in general I think you will run into similar issues with LGPL further down the line.

Also going to add in a voice for using a more permissive license such as MIT or Apache 2.0.
All the GPL style licenses have some kinda of condition on whether the copyleft clause applies which usually means in a business setting at either my work or in my own business I can't be sure it's safe to use without consulting a lawyer or the legal department (many company's have a blanket ban for the use of GPL style licenses in their software in the general case). LGPL is the most permissive of them but my understanding (not a laywer so make sure you do your own research) is it can in some circumstances lead to legal issues. this blog post and synks gpl explained give a good overview.
It's also worth noting that the openai python lib is MIT licensed and other big libraries like AWS-SDK and Github SDK's have either apache 2.0 or MIT in general.
Appreciate the work you're putting into this library!

Thank you for your input, @nagibyro.
The reason I chose the GPL license (and then the LGPL license) is because I want to make sure that developers make contributions that everyone benefits from, however that blog post made a good point: it's not very nice to be required to contribute, if any changes are made.
I think it's valid to say that pretty much everyone likes to see an MIT license, and because of that it's the one I should use. Even OpenAI suggests the MIT license specifically in libraries made for their API. So while it disappoints me a little to think that a company might fork a proprietary version of my library, I will trust that more developers will share their improvements.
Thank you again, it looks like the license will be changing once more 😅

Aaaand done, as of 309e2d0


Missing device configuration: 0x019b:0x0004:0x000f:4.1

Checklist

[X] It is not in the configuration DB

[X] It was not merged recently or has a pending PR

Which device is missing?
HeatIt Unknown
Manufacturer ID
0x019b
Product Type
0x0004
Product ID
0x000f
Firmware Version
4.1
Is the device listed on the Z-Wave Alliance website?
No response
Do you have a manual?
Reported by Telemetry:

capability
value

associationGroups
["Lifeline"]

supportsAGI
true

supportsConfigCCV3
false

supportsZWavePlus
true

zWavePlusVersion
1

If anyone plans on helping out with this file, please read https://zwave-js.github.io/node-zwave-js/#/config-files/using-telemetry-data first.

I've bought tis Heatit Z-Push Wall Controller. I want to help with the files for this controller but don't know where to start?
Is adding a Device Configuration File sufficient ?
The Z-Wave product info: https://products.z-wavealliance.org/products/4028
Greetings Ralph

Hope that answers some of your questions. I'll let the bot create an import from the Z-Wave Alliance DB which can serve as a starting point.
@zwave-js-bot import config 4028 from zwa

fixed by https://github.com/zwave-js/node-zwave-js/pull/3987


Support polling

Silicon's Zware/ZIP libraries poll the main device properties every 30 seconds. Perhaps having an option for enabling polling for a specific device would be even better, since you'd only enable it for devices, that would need this, to avoid unneeded traffic.

I think this should configurable very granularly:

Which nodes?
For each node: Which CC?
For each node: Which endpoints? Default: 0
For each node: how often? Maybe even as a CRON, so we can spread out the requests

AFAIK, about the "HOW OFTEN", you should set a pollInterval greather then the number of nodes in seconds. If we have 10 nodes then the pollInterval should be 10 or more seconds. Is there anything about this in specs?

There is this:
https://www.silabs.com/documents/login/miscellaneous/SDS11846-Z-Wave-Plus-Role-Type-Specification.pdf
From section 3.7 onwards.

The following requirements apply to the normal case where a polling request is successful.
 CommandTime MUST be measured by the application
 The application MUST wait PollTime before polling any other node
 PollTime SHOULD be 10 seconds + CommandTime or more
 PollTime MUST NOT be less than 1 second + CommandTime

There's also this:
https://www.silabs.com/community/wireless/z-wave/forum.topic.html/zware_hcapi_defaultdevicepolling-Ikwq

When using ZWare service (or any application built on top of ZWare HCAPI) I've noticed that Z-Wave devices are polled every 30 seconds (active get status).

Z-wares polling behavior is compliant with the Z-wave Plus polling requirements that sets a maximum rate for polling. This means that as more device are included in the network, the polling of each devices will be longer interval to prevent network congestion. For Z-wave plus v.2 devices that are required to send true status lifeline reporting Z-ware will not poll at all, as it can be certain that the devices will report any state change automatically.
The are pros and cons for polling, and Z-ware has chosen an approach, where it can keep the UI as updated as possible within the confines of the Z-wave spec. Reasons for less polling would typically be concerns about network congestion, but if fulfilling the Z-wave spec requirements, as Z-ware does, it should not be a problem.

FWIW this is the only thing preventing me from switching over from OZW. I have a couple of old devices that do not report state changes and need to be polled.

FWIW this is the only thing preventing me from switching over from OZW. I have a couple of old devices that do not report state changes and need to be polled.

https://github.com/zwave-js/node-zwave-js/pull/1479 adds the basic support for polling. I'm hesitant to do polling automatically and if something happens, it has to be separated from the driver. On top of that, polling should be used very sparingly, so for now:
If you need regular polling, handle it yourself as part of an automation. The tools are there.

https://github.com/zwave-js/node-zwave-js/pull/1479 adds the basic support for polling. I'm hesitant to do polling automatically and if something happens, it has to be separated from the driver. On top of that, polling should be used very sparingly, so for now:
If you need regular polling, handle it yourself as part of an automation. The tools are there.


Fixes #282 pointing to correct version of ambassador docs

Fixes #282
Documentation change, pointing to correct version of the ambassador docs.

LGTM


FileDescriptor.dependencies only iterates through public dependencies

Background
Public dependencies are a protobuf feature for re-exporting symbols from different proto files, like here. The linked docs suggest this is meant for migrating between .proto file paths:

By default, you can use definitions only from directly imported .proto files. However, sometimes you may need to move a .proto file to a  new location. Instead of moving the .proto file directly and updating all the call sites in a single change, you can put a placeholder .proto file in the old location to forward all the imports to the new location using the import public notion.

Notwithstanding, some repos seem to prefer import public over import, like here or here.
Current state
FileDescriptor.dependencies only lists public dependencies.
Proposed change
Perhaps FileDescriptor.dependencies should iterate through the entire dependencies list, as opposed to only the public dependencies?
I'm more than happy to make this change and put up a PR, just wondering if there's a reason for only listing public dependencies that I missed

Wow, not sure what I was thinking when I named that method 😮. I agree that I'd expect it to iterate all the dependencies, public or otherwise.
I doubt this method is widely used, so I think we can just change the behaviour in the next major version along with a note in the changelog. Maybe the existing method should be renamed public_dependencies() as well

I've made this change in version 0.10.0


.json() returns ObjC objects that aren't compatible with js

This isn't a true polyfill for fetch() then, since NSJSONSerializer -> JSON.stringify() doesn't work (dictionary and object kinda works but not string values for some reason). This is probably a limitation in Mocha / cocoascript itself, but I think we should use JSON.parse() instead of NSJSONSerialization to ensure JS libraries work as intended.
const json = '"F3F4F4"';

const jsonData = NSString.stringWithString(json).dataUsingEncoding(NSUTF8StringEncoding);
const parsed = NSJSONSerialization.JSONObjectWithData_options_error(jsonData, NSJSONReadingAllowFragments, null);

log(json)
log(jsonData)
log(JSON.stringify(parsed))

returns
"F3F4F4"
<22463346 34463422>
{}

Script executed in 0.015787s

I encountered this when trying to convert react-sketchapp-skpm-example to fetch its data instead of using an inline literal


Use just the content hash for cached publication

This builds on top of the great addition in PR #342 by @tomlm.
I didn't understand the benefits of maintaining two hashes in #342, one of the script's path and the other of the script's content. This PR uses just the latter (hash of the script content) as part of the path to the cached DLL, yielding the following advantages:

The script can be moved or copied around the file system and the cache will be hit so long as it's the same content!

It avoids locking issues when updating the source of long-running scripts (which includes simple scripts having a call to Console.ReadLine()at the end). For example, if a script takes some time to run, I'm not able to edit and update it. The implementation in #345 fails due to locking issues, at least on Windows. With this PR, the change in the content will mean a separately published/cached version will run without interfering with existing one!

These advantages are even more pronounced when you think of a repo of C# scripts where you can switch between branches & run different versions of scripts without affecting those already in flight. The cache will help even more when switching between different versions.
This PR also changes the encoding of the hash bytes in text from Base64 to the more conventional hexadecimal. If it needs to be made shorter then Base32 would be a better compromise.

Sorry, this was meant to be a PR, not an issue.
See PR #353 instead.


VLLM Compatibility Update

Changes

update api_server to be compatible with latest version of vllm
removed deprecated param cot_table_alias from run_model_cot.sh

Testing
Tested with:
./run_model_cot.sh defog/sqlcoder-8b-dare-ties

Results changed slightly due to running on an A40 with more memory, but overall still similar to past runs:

Model
sqlcoder-8b-dare-ties

V1 Accuracy
88.10%

Basic Accuracy
100%

Advanced Accuracy
90.62%

Overall Accuracy
90.1%

Cool, thanks! Will also pin the version in requirements.txt if you think this post1 version will be the standard. Or can keep as it is for backward compatibility!

Thanks for the suggestion! We have some environments (e.g. the lambda labs H100) where it might not be convenient to upgrade the version due to existing hosted endpoints/models that we would need to test for regressions, hence will not pin the vllm version for now. 😄


Switching to wrong window on pressing space button or clicking "Close" button in overlay.

I'm playing in windowed fullscreen mode.
To reproduce:

Open overlay with any shortcut (tried with Ctrl+D and Ctrl+X)
Press Space or click "Close" button in overlay.
Previous window opens instead of game.

To prevent this behavior you can do one of this:

Use "Close Overlay With Mouse Click" option.
(I use this) Use any shortcut to open overlay, go to "All Settings", click on game window. "All Settings" window now is open but hidden behind game window. Keep it open. Now you can use shotrcuts to open another instance of overlay and close it with Space or "Close" button without any issue.
Minimize all opened windows of all programs other than game.

I'm pretty mush sure it is a bug, but at least it can be handled.

Linked to #410
@leMicin The 2nd point is interesting.


[CON-287] Ontraport connector

Closes #
Configuration

Catalog variables
<Does the provider's info include any template variables such as {{.workspace}}>
Workspace:
Testing
GET
URL: http://localhost:4444/v2/some-api-call
Postman screenshot (must show the request URL, the response status code & body clearly)
POST
URL: http://localhost:4444/v2/some-api-call
Postman screenshot (must show the request URL, the response status code & body clearly)
<Please add the same information for all other methods available - PUT, PATCH, DELETE, etc>
Pagination
Please add screenshots that show successful pagination using the connector.
Raw token response

Logo & Icons
<Add screenshot of what icons, logos where selected>

#1208  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Some of the links are not read with TalkBack on android

Describe the Bug
Most of the links are not read/skipped by TalkBack on android native. Also links are not read in a context of a text area. Usually it is read separately when it is clicked.
Steps to Reproduce

Go to 'Categories'
Click on 'text areas where the links are located'

Expected Behavior
It should be read in context of a text, not separately only when the link is clicked.
Actual Behavior
A link is skipped by TalkBack. Below there is the text area where the link is before the . sign. The link inside text area is not read, but . and then the whole sentence after is read
Additional Information

Possibly related to/duplicating #1236.

Why the <a> tag got that size?

https://integreat.app/augsburg/en/anlaufstellen/the-kultursozialticket-tickets-for-refugees-theatre-for-1

Why the <a> tag got that size?
https://integreat.app/augsburg/en/anlaufstellen/the-kultursozialticket-tickets-for-refugees-theatre-for-1

I think it is just wrongly highlighted by chrome dev tools, can't reproduce the same behavior in firefox. Also, only the "actual" link is clickable, so I think this is fine. Or am I missing anything here?

Yeah you are right.. in firefox it looks fine.


couldn't train network on different dataset

I tried train the network on different dataset but it didn't work, every time I start the training it throw the following error

can you please help me to solve this issue ?
Regards

哥，你这中式英语够可以的


gftools colorv1 should show colors

Rn proofs are all black
https://github.com/google/fonts/pull/4529

gftools-gen-html.py proof ../fonts/ofl/blakaink/BlakaInk-Regular.ttf gives me this:

Definite bits of yellow in there, so I believe this is fixed?

LGTM. I think it was black due to lack of colr v1 support in Rosa's Chrome.


Support non-disjoint classes in FightingWords

The modal application of the Fighting Words method is to compare two disjoint sets of texts to see which words are characteristic of each. However, the fact that the sets should be disjoint is neither specified in ConvoKit's documentation nor imposed in the paper (Monroe et al (2008) p. 385). But it is implied in the code at the beginning of FightingWords.fit():
        class1, class2 = [], []
        for obj in corpus.iter_objs(self.obj_type, selector):
            if class1_func(obj):
                class1.append(obj)
            elif class2_func(obj):
                class2.append(obj)

where corpus components are classified by means of two provided functions. This leads to the error ValueError: class2_func returned 0 valid corpus components if class2 is a subset of class1 (and undocumented behavior if the converse). But this is a useful case when we're comparing more than two sets of texts in a "one-vs-all" manner.
So if I'm understanding correctly, it might make sense to replace the elif statement here with a plain if.

Good catch. I think this was a condition we specification we imposed because it helped catch cases of overlaps. Though you're right that FightingWords does not require that the two sets of texts be disjoint -- this is often how it's used in practice to evaluate linguistic patterns in two different communities.
That being said, the lack of commutativity you pointed out is problematic...I'm okay with making this change and having users decide for themselves whether they want disjoint sets of texts.
Will make the change tomorrow. LMK if you have any thoughts @cristiandnm

Great catch!  I agree we should let the user decide.  We could throw a
warning if the two sets are overlapping to help the user realize that this
is a non-standard use
On Sat, Dec 10, 2022 at 09:38 Caleb Chiam @.***> wrote:

Good catch. I think this was a condition we specification we imposed
because it helped catch cases of overlaps. Though you're right that
FightingWords does not require that the two sets of texts be disjoint --
this is often how it's used in practice to evaluate linguistic patterns in
two different communities.
That being said, the lack of commutativity you pointed out is
problematic...I'm okay with making this change and having users decide for
themselves whether they want disjoint sets of texts.
Will make the change tomorrow. LMK if you have any thoughts @cristiandnm
https://github.com/cristiandnm
—
Reply to this email directly, view it on GitHub
https://github.com/CornellNLP/ConvoKit/issues/188#issuecomment-1345256055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACCMFE4NAQ5XAYTPCRVIJBLWMR2SVANCNFSM6AAAAAASZXEUB4
.
You are receiving this because you were mentioned.Message ID:
@.***>


ADD: Project 1

Basic tests are made, skeleton code is provided.
TODO: add specific documentation about the functions

Todo is not done


Replace titanic dataset

Fixes issues #21  and #162 (partially).

Awesome @alexzwanenburg! Thank you for this fix/contribution!


Add support for Azure OpenAI

resolves #87
We created a base gpt class that allows us to inherit from it to create the Azure OpenAI interface as well as the traditional OpenAI interface.

Merged in<PHONE_NUMBER>085212b587329ff951dd30a2a6527e


Support muted_topics format for pre-Zulip Server 6

In my initial implementation of muted streams and topics #346, I intend to leave out support for the old muted_topics format. It's not fundamentally hard to do, but it'll be a bit annoying.
The impact of this is that when using Zulip Server 5.x and older, the client won't be aware of any muted topics. (There won't be any unmuted or followed topics in any case, because those servers don't have those features.)
Zulip Server 6 was released 2022-11-17, just over a year ago. So we're a few months away from dropping support for 5.x (in a successor issue to #268). If we approach a full production launch before that time, then we'll come back and deal with this.

We're not going to implement this at this point — we'll drop support for those now-ancient server versions instead.


About GPU memory?

Hi! I'm using two A100 GPUs, each with 40GB of memory. This is the GPU memory utilization for my training. I'm almost reaching over 90% memory utilization on both A100 GPUs.

Is this normal? Here's the configuration:
accelerate launch --config_file=./pipeline/accelerate_configs/accelerate_config_fsdp.yaml
pipeline/train/instruction_following.py
--pretrained_model_name_or_path=luodian/OTTER-LLaMA7B-INIT  \
--mimicit_path="path/to/SN_instruction.json"
--images_path="path/to/SN.json"
--batch_size=4
--num_epochs=9
--report_to_wandb
--wandb_entity=ntu-slab
--run_name=OTTER-LLaMA7B-densecaption
--wandb_project=OTTER-LLaMA7B
--workers=1
--lr_scheduler=cosine
--learning_rate=1e-5
--warmup_steps_ratio=0.01
and here's the accelerate config(accelerate_config_fsdp.yaml):
compute_environment: LOCAL_MACHINE
distributed_type: no
downcast_bf16: false
machine_rank: 0
main_training_function: main
mixed_precision: fp16
num_machines: 1
num_processes: 1
rdzv_backend: static
same_network: false
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false
main_process_port: 20687

And during training, the system prompts me with "Using dtype torch.float32."

I think the GPU usage looks normal but you could try with deepspeed_zero2 config and set the num_processes=2.

using zero2 will be much faster than fsdp if you have 80G A100.

when i use accelerate_config_zero2.yaml, i meet the error:
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 2.08 GiB (GPU 0; 39.59 GiB total capacity; 29.10 GiB already allocated; 1.91 GiB free; 35.46 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF


Feature: Define position in journal

Added a option to select where in the journal note the tracker info should be added.
Also I changed the Notice function to look a bit nicer.

@Brazier85 hey, thanks for the PR! I will check it when I have time 🫡

@Brazier85 it seems like if there is another header, this feature doesn't work. Please consider fixing this 😅

@dartungar function should be fixed by now.. I made a super stupid mistake :D

@dartungar I think I got all the possible position errors by now :)


unmated reads

Greetings,
I see that samblaster has an --ignoreUnmated flag. It seems to require that the bam file be sorted by query name. However, it seems like it also ought to work with a bam file resulting from
bwa mem ... | samtools view -b > blah.bam

From quickly looking at the source code, it seems like using the --ignoreUnmated flag on blah.bam should work since their query names are together.
What do you think? It would be nice to skip the samtools sort -n blah.bam step.

If you read the README carefully, it says that the input file must be read-id grouped not necessarily read-id sorted. It also shows that the typical usage scenario for samblaster is indeed in a pipe right after the aligner as you suggest. This is true with paired reads or singletons. No sort is needed if used in such a pipe.


Question: How to update charging costs

Hello,
How can I update the charging costs for a period of time? My bill is not always a fixed $/kwh every month but I don't know until the end of the month how much it costs. I know I can update the cost moving forward, but how can I update the costs of the charges from last month?

See #82


Create SECURITY.md

A security policy to publish security email contacts to report issues privately would be great. Where can I report a security issues privately?
Thanks.

Where can I share details privately about a vulnerability security?

Not necessary...


Users/v nagarajku/fixfor unableto access outputvariable app service application url

Task name: AzureRmWebAppDeployment@4
Description: Implementation for removing cosmetic API call in 4.205.13v was blocking AppServiceApplicationUrl output variable to be set. Fixed for setting up the same in updateDeploymentStatus()
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) N
Attached related issue: (Y/N) N
Checklist:

[x ] Task version was bumped - please check instruction how to do it
[x ] Checked that applied changes work as expected

Is it possible to expedite this PR?
The AzureRmWebAppDeployment@4 v4.208.0 doesn't have output variables anymore.   Seems to be the same issue as with 4.205.13.  See Issue: #16467


Kernel Hangs when Asking for User Input

Read-Host 'Enter input here:'
results in a cell running forever and hanging.

We detect similar situation for the incomplete lines. I guess we can detect it and at least print the error telling they need to restart the kernel.
It's pretty annoying that we cannot run in a non-interactive mode so it errors out on such problems.


epoch_acc[0] += accuracy(y_pred, y, topk=(1, 3, 5))

1，3，5代表什么
1，3，5代表什么
l don't understand What does 1, 3, 5 stand for?Why to define epoch+acc=np.array([[0,0,0],[0,0,0],[0,0,0]])

@wangtao2668129173 Hi, topk=(1,3,5) means our accuracy() returns top-1, top-3, and top-5 accuracy in an np.array() with size (3,). You may set topk=(1,) if you want top-1 accuracy only.


Repaired regex

Repaired regex so it won't catch words like "hello" or "objects/exercise"
What issue is this solving?

Closes #139
Description

Updated the regex so that it ensures were the start and end of the word is.

Any helpful knowledge/context for the reviewer?

Is a re-seeding of the database necessary?  No
Any new dependencies to install?  No
Any special requirements to test?  Must be lower than SU

(e.g., admin perms, alt account, etc.)

Please make sure you've attempted to meet the following coding standards

[x] Code has been tested and does not produce errors
[x] Code is readable and formatted
[x] There isn't any unnecessary commented-out code

I'm going to go ahead and merge this, I don't think it needs further approval for adding 4 characters.


decouple motion capture tracking package

Currently when crazyswarm2 has to be installed from scratch, the motion capture tracking package is mandatory due to this:

CMake Error at CMakeLists.txt:17 (find_package):
...
  Could not find a package configuration file provided by
  "motion_capture_tracking_interfaces" with any of the following names:

    motion_capture_tracking_interfacesConfig.cmake
    motion_capture_tracking_interfaces-config.cmake

Shouldn't this be an option based on what people are using as a positioning system? Perhaps also in the combination of what I mentioned in #67 ?
Not an urgency now but a point of discussion in the future :)

another reason to decouple it.. I am now trying out crazyswarm2 on 22.04 and humble and getting the following issue of motion capture:
CMake Error at deps/libmotioncapture/deps/pybind11/tools/pybind11NewTools.cmake:196 (python3_add_library):
  Unknown CMake command "python3_add_library".
Call Stack (most recent call first):
  deps/libmotioncapture/CMakeLists.txt:158 (pybind11_add_module)

We are always needing the motion_capture_interfaces package, because the server needs to include and link against it. Given that this is now released in humble/iron/rolling, I think this will be less annoying and I vote for closing this issue.

absolutely. So we could change the installation instructions now I assume? https://imrclab.github.io/crazyswarm2/installation.html

I think we would need wait for the next humble sync. It's in iron and rolling, but didn't make the cut for humble binary release, yet.

ah gotcha. I'm closing the issue but let's not forget to update the documentation once it has been synced


Mention Google Oauth plugin as requirement

Thanks @bodepd! Once the CLA is signed I'll merge this in. How did things go aside from this?

It'll take me a few days to figure out all of the contacts for the CLA. While I was able to follow the instructions and get things working, there were a few bits of detail about how things work that weren't entirely clear. Is there a mailing list where I could ask some follow up questions?

@viglesiasce my CLA is now taken care of, but it is listed under my company email address (which is listed on my account, but not the email address that I use for open source contributions.)

Thanks @bodepd! I found it.
As far as follow up questions, if they are related directly to the content in this repo open up an issue to spark up the discussion.
For container and Kubernetes related inquiries check out:
https://groups.google.com/forum/#!forum/kubernetes-dev
https://groups.google.com/forum/#!forum/google-containers
Thanks again for the contribution!


I-11441: Add cross-browser support for appearance

Description

appearance property is used to remove default browser styling for certain html elements like input tags.  Some browsers, like Safari iOS do not support it.  This PR adds cross-browser support for appearance.

Closes https://github.com/zurb/foundation-sites/issues/11441

Motivation and Context

Adds cross-browser styling support for appearance property.
Screenshots (if appropriate):
Types of changes

[ ] Documentation
[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 change)

Checklist (all required):

[x ] I have read and follow the CONTRIBUTING document.
[x ] There are no other pull request similar to this one.
[x ] The pull request title is descriptive.
[x ] The template is fully and correctly filled.
[x ] The pull request targets the right branch (develop or develop-v...).
[x ] My commits are correctly titled and contain all relevant information.
[x ] My code follows the code style of this project.
[x ] I have updated the documentation accordingly to my changes (if relevant).
[x ] I have added tests to cover my changes (if relevant).
[x ] All new and existing tests passed.

It is already there
https://github.com/zurb/foundation-sites/blob/develop/dist/css/foundation.css#L2376

Also the others are not needed as these are for very old browsers.

Keep in mind that we already use browserslist and a few other solutions to vendor-prefix such properties


The security of a page with swagger (iGotify Notification Assist API) on the web.

How safe is an open page with swagger (iGotify Notification Assist API) on the web? If I hide it from public access, the service stops working. Also, you can't put any protection method on it, can you leave it that way it's safe and protected from intruders? I use the authentik authentication method in many services, but as I understand it, it will not work in this case. I am using a configuration with traefik. Thank you for your hard work, you helped a lot.

Hi, yes it's save because it's handle nothing big that's securely relevant is.
It's only there for testing methods and other.
In the next 1.3.0.X release it will be by default disabled.
When you attach a authentication before the websocket connection closed that's the reason it don't work


fix: make request body and response have consistent name rule

Signed-off-by: Allen Sun<EMAIL_ADDRESS>1.Describe what this PR did
This PR makes HTTP POST request body have same name rules.
For a request body, we have rule of ObjectActionConfig, like: NetworkCreateConfig, ContainerCreateConfig, VolumeCreateConfig, ExecCreateConfig and ExecStartConfig.
For a response struct, we have a rule of ObjectCreateResp, like ContainerCreateResp, NetworkCreateResp.
However, VolumeCreateResp in not in the scope, since POST /volumes/create API returns VolumeInfo which is a inconsistency in Moby's API.
/cc @rudyfly @skoo87 @Letty5411
2.Does this pull request fix one issue?
NONE
3.Describe how you did it
NONE
4.Describe how to verify it
NONE
5.Special notes for reviews
NONE

LGTM


500 error while adding connection to ElasticSearch

How to reproduce the bug

Go to 'Data -> Databases -> +Database'
Click on 'Other'
Enter SQLAlchemy URL: elasticsearch+http://localhost:9200/
get a 500 error in a popup. Action Log shows test_connection_error.ModuleNotFoundError

Expected results
A connection to ElasticSearch. If I run superset shell in my docker container I can connect to elastic search so elasticsearch-dbapi is installed and the url is correct
Actual results
An error in the UI. I have no log file I can find to provide more. There is no /app/super_home/superset.log file???
Environment
(please complete the following information):

browser type and version: Firefox
superset version: Superset 0.0.0dev
python version: Python 3.8.12
node.js version: node: command not found
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any. -> where is the log? I can't find it, There is no /app/super_home/superset.log file, and docker logs gives me nothing
[X] I have reproduced the issue with at least the latest released version of superset. Well I built the docker image as indicated at https://hub.docker.com/r/apache/superset
[X] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Running one docker container, this is my dockerfile:
FROM apache/superset
USER root
RUN pip install psycopg2-binary
RUN pip install elasticsearch-dbapi
USER superset

RUN superset fab create-admin \
               --username admin \
               --firstname Superset \
               --lastname Admin \
               --email<EMAIL_ADDRESS>\
               --password admin

RUN superset db upgrade

RUN superset init

OK, by not running upgrade/init in the dockerfile and loading properly the python config file I've managed to get it to work.


Calling make_interpreter() crashes the sample classify_image.py script with no error message

Description
Hello,
I have a Windows 10 machine and I could only execute the Coral example using the old drivers (edgetpu_runtime_20210119), which caused abnormal runtimes of ~1800ms per image.
Later, I fixed the install.bat script in the latest version (edgetpu_runtime_20210726):
Before change
start /wait msiexec /i "%ROOTDIR%\third_party\usbdk\UsbDk_*.msi" /quiet /qb! /norestart

After change
start /wait msiexec /i "%ROOTDIR%\third_party\usbdk\UsbDk_1.0.22_x64.msi" /quiet /qb! /norestart

Now the failed to load delegate from edgetpu.dll error message has disappeared, yet a new error started appearing, where the script hangs up with no error message when it calls interpreter = make_interpreter(*args.model.split('@')).
Insights

It appears that it hangs when it calls: return tflite.load_delegate(_EDGETPU_SHARED_LIB, options or {}).
The 3rd party libusb-1.0.dll file of the latest edgetpu_runtime release makes the problems. If I copy the file from the 20210119 version to C:\Windows\System32, then I fallback to the previous problem of very slow model execution.

Any advice?
Click to expand!
Issue Type
Build/Install, Support
Operating System
Windows 10
Coral Device
USB Accelerator
Other Devices
No response
Programming Language
Python 3.7
Relevant Log Output
No response

Wow, thanks! I'm pretty shocked... wasn't aware of the fact that there were two files with similar names.
The updated output:
C:\programs\coral\pycoral>python .\examples\classify_image.py --model .\test_data\mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflite --labels .\test_data\inat_bird_labels.txt --input .\test_data\parrot.jpg
----INFERENCE TIME----
Note: The first inference on Edge TPU is slow because it includes loading the model into Edge TPU memory.
14.4ms
4.8ms
4.9ms
4.8ms
4.7ms
-------RESULTS--------
Ara macao (Scarlet Macaw): 0.76953

Problem's solved 😃


Allow for changing the okhttpclientprovider builder

Motivation
This is another attempt at reviving OkHttpClientProvider as a way of controlling the OkHttpClients used by react native. See previous attempts #14068 and #14675, and the original change here: https://github.com/facebook/react-native/commit/0a71f48b1349ed09bcb6e76ba9ff8eb388518b15 .
Test Plan
Unit test
Release Notes

[ANDROID] [ENHANCEMENT] [Networking] - Added support for swapping out OkHttpClient.Builder in OkHttpClientProvider

Any feedback or comments on this PR, @hramos or @mkonicek ? I'm open to alternative approaches, but since injecting new behavior into the networking code is possible on iOS it would be really useful to re-establish this possibility on Android - especially given that the current API gives you the impression that it's possible even though it isn't 😄

I don't have anything to add right now, other than there are some conflicts you might want to look at.
I'm not particularly familiar with this area of the codebase -- I'll let another core contributor jump in. Please ping me once this has been reviewed.


Importing some models fails if Eigen3 is not present

I get name 'LDA' is not defined when I installed Shogun without Eigen3 and the same for QDA and MCLDA.
So I think it should be made clear at Wiki that Eigen3 is required to use these models or may be Eigen3 can be a required library if there is many models depending on it.

@karlnapf should I update the Wiki or should I change the build to make Eigen3 a required library ?

I think we can make eigen3 a requirement. If not present, we can bundle it easily. We had many discussions about this before and converged onto this without ever doing it.
A patch would remove all of the #ifdef HAVE_EIGEN3 statements (and more, grep for eigen3) , and modify the cmake build to use eigen3 by default. If not available, it should either give an error message asking to bundle it, or turn bundling on straight away.
If you send a pr to start this, we can discuss in there. This would be a great contribution!
@lisitsyn @vigsterkr @besser82 will have comments.

Done in https://github.com/shogun-toolbox/shogun/tree/feature/eigen3_bundling which will be merged soon


The reference to "schema/schema.json" is bad

This is probably talking about 1, which should have been updated in
#481 or the #498 fixup.  Are you working on a PR, or shall I submit
one?

hmm. @hapnermw do you have any more specifics?


Run OmniJ tests with all build targets except OS X

The Windows binaries are "routed" through wine, which works surprisingly well with a few minor adjustments.
The build/test time may be longer though.

Storing extra information about a crowdsale, when closing one, provides no gain, given that this information can also be retrieved via RPC.

I'm not against removing it per se (though ideally it'd be nice to know why it crashes 32-bit builds) but perhaps it might be better to shift it into msc_debug_sp or a new msc_debug_crowdsale flag rather than getting rid of it completely (in the case that we want to verify log vs RPC data)

Ah, please note that I added the crowdsale commit to this PR only temporary, to test Travis. :)
Travis is really awesome, and I think we should always run the tests, if possible, but unfortunally this has a negative impact on the test time with about 3-12 minutes more per build target.
So far it looks like, assuming there are no other builds in the queue, the total time is around 16-20 minutes. I guess that's not horrible, and ideally builds are tested locally anyway, but it still makes a difference.
What do you think?

What do you think?

I would always sacrifice build time for more testing.   In a heartbeat.

Well then, let's figure out how to deal with the crowdsale dumping issue and get it in then. :)


Deletable Runs

Make runs

[ ] Deletable (even before they are started)
[ ] Cancelable

Frankly, I'm not exactly sure what is meant by "deletable" and "cancelable". Do you mean the ability to terminate a run? And if yes, where's the difference between "deletable" and "cancelable"?

I don't see the difference here either. There is already the possibility to end a run, so having the possibility to subsequently also delete it should be sufficient, right?

I agree with you. I guess the two bullets are an artefact of our discussions.
The system would benefit from deletable runs, which, however, are always deletable. Hence, non-started runs would be _cancelable 😉

With the changes in #62 it should now be possible to terminate a run in ever state (unless a task is being executed). Furthermore, there is now a CLI command to delete runs.


full ubuntu theming

Is your feature request related to a problem? Please describe.
I can't get used to how the shell window will never feel like a ubuntu shell. It just feels like I am using the windows command prompt with additional ubuntu support. No matter what workaround I use to try to make it mostly similar it's not the same as using gnome shell in terms of looks and functionality.
Describe the solution you'd like
I would like to see there be an option to set the shell to look exactly like the Ubuntu gnome terminal shell because for people like me, I hate using and looking at a windows command line. I've always been around the ubuntu gnome shell and I just can't get used to the command line theme that it has by default. Even using alternative methods by installing the ubuntu fonts and changing the colors from the preferences, it just isn't the same. Especially when it comes to something light right clicking in the shell, I would like to be able to have it open a menu instead of pasting what ever is in the clipboard.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

ubuntu shell

Let's start of being clear here. Ubuntu doesn't have a shell. GNU bash is a shell, among many shell alternatives such as zsh. Likewise gnome-terminal isn't a shell. The gnome-terminal is a (wait for it) terminal, among many terminal alternatives such as enlightenment.
You are being pretty specific here; you want gnome-terminal. Which is a reasonable ask, but not a WSL ask. Supporting GUI apps naively (which would include but is not specific to gnome-terminal) is duplicative #829, #938, others.
You might want to check out the new terminal. It is converging on modern terminal looks and functionality expectations.
Alternative is to use a third-party X11 Server like VcXsrv, others.


about items from the augmented Ender Chest

Timings link
No response
Expected behavior
Can we make the additional slots be dropped on ground when player loses the permission for more ender chest slots?
Observed/Actual behavior
When players lose permission to have a larger ender chest, items that were in the additional slots are deleted.
Steps/models to reproduce
1 Have more space on ender chest
2 Use the additional slots
3 Remove your permission to have it
4 You lost the items that were in the additional space
Purpur version
Checking version, please wait...
This server is running Purpur version git-Purpur-1521 (MC: 1.18.1) (Implementing API version 1.18.1-R0.1-SNAPSHOT) (Git: 4365741 on HEAD)
You are 11 version(s) behind
Download the new version at: https://purpurmc.org/downloads
Previous version: git-Purpur-1519 (MC: 1.18.1)
Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
I'm not using the latest version of Purpur, but my issue has nothing to do with it

With the way containers are implemented, this isn't really possible. It's the same problem if you decided to switch to another fork instead of Purpur. All the extra rows simply will not exist in order to drop the contents to the ground.

What do you can suggest for me then? maybe there is a plugin that can handle this?


Error to display OAuth Card in Web Chat

Project
Vistual Assistant Bot - Error to display OAuth Card in Web Chat which is embedded in Azure portal
Description
Error to display OAuth Card in Web Chat which is embedded in Azure portal
To Reproduce

Clone Code and deploy and run
Deploy it to Azure Bot Service
Chat the bot by using Web Chat which is embedded in Azure portal
type "email to me"
Bot do not properly display the OAuth Card

Expected behavior
Bot should display the Sign-in card and then user can type their credentials
Screenshots

Additional context
I found a link which is maybe related to this issue on Github
https://github.com/Microsoft/BotBuilder/issues/4632

Hi @angie4u, the web chat in the "Test in Web Chat" tab of the Azure portal is out of date and does not support oauth cards. The latest web chat information can be found here: https://github.com/Microsoft/BotFramework-WebChat

Alternatively please use the Emulator and open the Bot file we create for you in your project directory

@David201406 the authentication code is used for channels that do not support sending the token back via an event. You should be able to paste the code back in to the conversation to finish authenticating. Does that not work for you?

@lauren-mills  No, if I enter the code in the chat, I get an error.

@David201406 thanks for the screenshot. Would you mind trying this same flow in the emulator and sending me the stack trace from the trace message? You should be able to see it in the bottom right window when the error occurs. Another option is to look at your appInsights log in the Azure portal for exceptions. Either of those will help me a lot in figuring it out.

Sorry, I just saw in your previous comment that you can't repro in the emulator. If you have appInsights set up for your published app please try that otherwise let me know.

@lauren-mills I startet the chatbot in the emulator and it worked fine, I could sign in and send emails. Afterwards, in Chrome and Edge even after restart the PC I could send email without the need to sign in. I started even in Azure the App Service. I still could send email from the chatbot without sign in. So I guess, that somewhere and somehow the Token is saved. But how can I reset everything, to try to reproduce the issue.

You should be able to type "logout" to clear your token cache

Entering "logout" I get "OAuthPrompt.SignOutUser(): not supported by the current adapter |    at EmailSkill.MainDialog.OnLogout(DialogContext". Is there a other possibility to clear the token Cache?

Hi David,
Logout isn't a scenario we currently support at the VA level but we do have a reset user event to trigger the flow we envisage. You can try this by typing this into your conversation:
/event:{Name:"IPA.ResetUser"}
https://github.com/Microsoft/AI/blob/master/solutions/Virtual-Assistant/docs/virtualassistant-events.md
Sent from my 📱

From: David201406<EMAIL_ADDRESS>Sent: Monday, December 3, 2018 8:15:14 PM
To: Microsoft/AI
Cc: Darren Jefford; Mention
Subject: Re: [Microsoft/AI] Error to display OAuth Card in Web Chat (#210)
Entering "logout" I get "OAuthPrompt.SignOutUser(): not supported by the current adapter | at EmailSkill.MainDialog.OnLogout(DialogContext". Is there a other possibility to clear the token Cache?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FAI%2Fissues%2F210%23issuecomment-443854227&data=02|01|darrenj%40microsoft.com|e1e6823619b34fdb258808d6595c096e|72f988bf86f141af91ab2d7cd011db47|1|0|636794649165838769&sdata=pnEx%2Fbggsc%2BX4w%2F1OPUSywO7Co3Z%2BR3XXR%2Bbgi2%2Fy24%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAf93oMAUfu51PycYHKdyUxRcMfDQEdorks5u1YZSgaJpZM4YFxlk&data=02|01|darrenj%40microsoft.com|e1e6823619b34fdb258808d6595c096e|72f988bf86f141af91ab2d7cd011db47|1|0|636794649165848786&sdata=q3khzCwNozSZ52qcl4SBlkumbC9wC8S5JkCgX3Psk3c%3D&reserved=0.

Exception.zip
Entering /event:{Name:"IPA.ResetUser"} AND reboot of PC helped to clear the token Cache.
I could reproduce the issue (see above screenshot). I attach the current used SkillDialog.cs and the ExceptionLog of AppInsight.

I discovered. I need to refresh the HTML page and restart the conversation. Afterwards it works sending an email.


How to change URL of section page?

Putting url = "some-url" in the _index.md does not seem to work

Hmm I didn't think this would be needed. I was expecting people to use folders for the section path. Is it no possible in your case?

I have section that is used internally only. All posts have custom URLs directly under the page root (e.g. "/post1", "/post2"). Now I want a custom URL for the section page, too (e.g. "/extra" instead of "/parent-category/extra-content").

if all the URLs are custom, what's preventing you from putting it directly in extra in the filesystem? Not against it, just trying to figure out if it's possible to avoid adding it

It's just a bit cleaner, I think. I have the main posts in a posts section and extra posts in an extra section. A few of these extra posts build on each other, so I put them in a subsection with manual ordering. These posts have custom URLs for historical reasons (they were part of the main posts once), so the additional directories for the section page seem strange. Moving them out of extra is certainly possible, but it feels like a workaround and would clutter the content folder.

To be clear, this is not a large problem for me, just a nice-to-have feature :).

I'll think about it

I am currently investigating gutenberg and ran into the same question I guess. I don't really want a 1:1 relationship of the sections name and the output. Ideally I would want to turn posts like this:
content/posts/2017-some-topic

or
content/posts/2017/some-topic

into /2017/some-topic.
Am I correct that this is currently not possible?

This isn't currently possible no. Any reason you don't/can't put your files into content/2017?

@Keats then I would have sections called 2017 and 2016 and so on ...and I would not be able to iterate through all posts in templates, no?

I think the usage @tcurdt is looking for is exactly what brought me to this thread. content/posts/date-prefixed-file.md becoming /date/based/path/slug.html is precisely what I'd need to have to convert a jekyll blog to use gutenberg instead :confetti_ball:
I haven't tried this yet, but I wonder if the components (year, month, day) could be used as replacements in the path for a given page. That should get me most of the way there.

Sorry I forgot to reply initially.
You can use get_section to grab each year pages one by one but there wouldn't be a way to grab all of them in one call.

content/posts/date-prefixed-file.md becoming /date/based/path/slug.html is precisely what I'd need to have to convert a jekyll blog to use gutenberg instead

I am not planning to add that feature (or the :year/:month/:day/:title kind of URL scheme) anytime soon.

Closing this as I'm not planning to add a way to change the URL of a section right now


Two errors were raised when using mirrored strategy.

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow):
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if
the issue happens on mobile device:
TensorFlow installed from (source or
binary): using conda
TensorFlow version (use command below): 2.1.0
Python version: 3.7.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from
source):
CUDA/cuDNN version: 10.1.243 / 7.6.5
GPU model and memory: Tesla T4 16GB

TypeError: An op outside of the function building code is being passed
a "Graph" tensor. It is possible to have Graph tensors
leak out of the function building context by including a
tf.init_scope in your function building code.
For example, the following function will fail:
  @tf.function
  def has_init_scope():
    my_constant = tf.constant(1.)
    with tf.init_scope():
      added = my_constant * 2
The graph tensor has name: Dec_VAE_VDraw_Var/Identity:0

During handling of the above exception, another exception occurred:
...
_SymbolicException: Inputs to eager execution function cannot be Keras symbolic tensors, but found [<tf.Tensor 'Dec_VAE_VDraw_Var/Identity:0' shape=(None, 128) dtype=float32>, <tf.Tensor 'Dec_VAE_VDraw_Mean/Identity:0' shape=(None, 128) dtype=float32>]

reproducible notebook files:
https://drive.google.com/file/d/1DzES68HT3-7dlrpH6oo13Q9MgybdjcGm/view?usp=sharing
When i didn't use mirrored strategy and added experimental_tf_function = False as an argument to model.compile, the errors were not raised.
It would be glad if you would help me to solve this errors.

Would it be possible to create a small Colab notebook that reproduces the issue?

Would it be possible to create a small Colab notebook that reproduces the issue?

I know that google colab does not support multiple gpu. Does mirrored strategy work on one gpu?

Hi @Crispy13, yes you can run MirroredStrategy with one GPU in colab and for this scenario I think we should be able to reproduce the error. If you are still facing this issue, please provide a colab notebook so we can help troubleshoot.


Create vendor_compromise_cred_theft.yml

Description
Email contains credential theft language, a short body, a suspicious link, and an additional suspicious indicator, all from a known sender in your environment. Highly indicative of vendor compromise.
Associated samples

https://platform.sublime.security/messages/242c30e6c47b2bee71f96bd0445eecf16680cf0920aed001dbe1db6b0739c0dd

/update-test-rules


Can i get the video file with success reponse?

When the liveness detection is completed and verified, i need the whole video of it.
please help me here

??


bn256.hashedMessage panics for some messages

For some messages M, hashedMessage function panics.
Example:
message = []byte("I am the byzantine general.")
Reason:
Under the hood bn256.RandomG1(reader)  is using big.Int(rand io.Reader, max *big.Int) function.
This function assumes implementation of io.Reader can be called
many times and each time reads up fresh data.
We fail to satisfy this condition as we create reader with :
reader := bytes.NewBuffer(hashed) this reader has finite capacity and
_, err = io.ReadFull(reader, bytes) will return error when the reader is exhausted. We don't handel this error hence the bug.

fixed in:
https://github.com/ConsenSys/handel/pull/125


[FEATURE REQUEST]The Issue Regarding OIIO Support for ARW File Format

When I use the OIIO library to read ARW file format, it shows an error indicating unsupported file format or not a valid raw file format. However, upon checking, I found that LibRaw does support parsing this format. Therefore, I hope to see if it's possible to add support for parsing ARW file format.

I have uploaded a sample file in ARW file format, hoping it will be helpful for analyzing this file format.

All formats that libraw reads are the same to OIIO; there's no place in the code where we include or exclude individual formats that libraw supports.
Can you tell us the exact error message?
(And is there an arw file you can point us at, to give it a try on our end?)

@lgritz
Sorry, just found out there's a size limit for uploading files on GitHub. Could you please provide your email so I can send it to you privately? Also, please do not disclose this file, as it is the client's confidential document.

<EMAIL_ADDRESS>ought to get to me. Anything you send will be 100% confidential and be deleted as soon as I've identified the problem.

@lgritz Hello, the required ARW file sample has been sent to your email. Please check your inbox. My email has the prefix "jmp" and is a Google email.

Works for me, opens fine with OIIO on m Mac using libraw 0.21.2 and OIIO master.
Let's take a step back. Can you tell us the precise error you got, and the command you tried?

@lgritz The version of oiio I use is #define OIIO_VERSION_MAJOR 2
#define OIIO_VERSION_MINOR 4
#define OIIO_VERSION_PATCH 13
#define OIIO_VERSION_TWEAK 0 has_error() returns true when calling ImageBuf to read The compiled libraw supports DNGSDK I'm not sure if it's the compiled libraw that's the problem?

The version of oiio I'm using is. When calling ImageBuf to read, has_error() returns true. The compiled libraw supports DNGSDK. I'm not sure if there are any issues with the compiled libraw.
#define OIIO_VERSION_MAJOR 2
#define OIIO_VERSION_MINOR 4
#define OIIO_VERSION_PATCH 13
#define OIIO_VERSION_TWEAK 0

has_error() returns true, but does geterror() return any useful message that gives us a clue?
What if we take your app out of the equation for the moment, does it work with a pure OIIO codebase pipeline? Like
oiiotool --info -v -stats yourfile.arw

does it work, or does it give an error?
Also,
oiiotool --help

should print the library versions (near the bottom, where it lists "Dependent libraries:"). Can you verify that libraw is there, and what version it's using?

@lgritz Sorry, it might be due to issues with the compilation of the libraw library on my end during debugging. I apologize. Are there any specific things to pay attention to when compiling the libraw library for use with oiio?

@lgritz  I think I've found the answer. Support for the arw file format in the libraw library began with version 0.20.2. The oiio library I compiled relies on the source code of libraw version 0.20.2. I apologize for the inconvenience, but at least finding the reason is always a positive outcome. I'm sharing this information in the hope that it may serve as a reference for others who encounter the same issue in the future.
These are the results of my program tested with libraw version 0.20.2.

@cool2528 I'm glad we quickly got to a solution. I've deleted the file you sent me.


Release v0.1.1

Bug fixes for T::Array
Support T::Boolean

Bug fixes for T::Array
Support T::Boolean


Add missing class string type annotations

This is more in line with what's been done here for example: https://github.com/filamentphp/filament/blob/3.x/packages/actions/src/Exports/Exporter.php#L66
There are a few more places where this could also be added. If needed/wanted I can add them to the PR too.

Can you please investigate the PHPstan failures as a result of this PR?

I have had to remove the resource annotation, PHP CS Fixer incorrectly changes it from class-string<Resource> to class-string<resource>


Add tests for pitchfork

👋 Is saw this linked in some recent rails PR and thought it pretty useful.
Pitchfork is a unicorn fork, compatible with Rack 3, a few new features and a few other removed. It's not quite as popular as the other servers here but a perfect canditate for people still stuck on unicorn today.
I'm not quite sure on the requirements for inclusion here so I'm just giving this a shot. There are some failures, especially around Rack 3 since it isn't Rack 3.1 ready yet (release pending).
Current failures are (when using git source for rack 3.1 compat):

hijack.rb it can stream a response (rack 2 & 3)
body.rb it can stream a response (rack 3)
websocket.rb it can establish a websocket connection (rack 3)

I assume these are legit failures. Especially the hijack one doesn't occur with unicorn so a bisect should lead to the cause. I will try to understand these failures to get them fixed upstream.
I will put this into draft for now until these are adresses. Let me know what you think about this in general. Just an (obvious) FYI that I'm not really involved in the project and byroot probably doesn't know about this yet but I'll give a heads up once I do my first issue/PR about the above conformance issues.
Types of Changes

New feature.

Contribution

[ ] I added tests for my changes.
[x] I tested my changes locally.
[x] I agree to the Developer's Certificate of Origin 1.1.

Sorry, I missed this, I was giving a talk about rack-conform today and realised I should add it!

That's fine, guess I was just ahead of the curve. It's better if someone with a deep understanding of it all does it anyways, though I am trying (;
For the other 2 failures on rack 3, I too suspect the Chunked middleware. It doesn't seem to take into account that the response type is a proc and shouldn't/can't be chunked.

I really appreciate that you had a go at adding it, and I think your PR is great, maybe even better than mine!


[Group 4][Accessible Pressable] Migrate EmojiReactionBubble

HOLD on https://github.com/Expensify/App/issues/16988
Description
Migrate EmojiReactionBubble to use PressableWithFeedback

Triggered auto assignment to @arielgreen (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero process steps for feature requests for more details.

I will do this one :D

@roryabraham I think it will be resolved after migrating PressableWithSecondaryInteraction #17047

I want to work on it.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_4fe30dnh5h/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_4fe30dnh5h/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_4fe30dnh5h/clone/recipe/meta.yaml:44: invalid_url: http://opensource.perlig.de/rjsmin/doc-1.0/index.html : Not reachable: 404
Errors were found


Allow per_try_timeout when global route timeout is disabled.

This allows, for instance, a per_try_timeout (which is only in
effect until the response to downstream begins) with streaming endpoints,
for which an overall timeout doesn't make sense.
Signed-off-by: Greg Greenway<EMAIL_ADDRESS>Risk Level: Low
Testing: New tests written
Docs Changes: I don't think any are needed. The docs say This timeout must be <= the global route timeout, and I think this makes it a bugfix, because in this case a zero value actually means infinity, not zero.
Release Notes: TODO

In addition to clarify the option docs themselves, some discussion in https://www.envoyproxy.io/docs/envoy/latest/faq/configuration/timeouts would be great also.

LGTM, but seems like CI is red

Weird. No code changes at all, but it failed to compile. I'll merge master and see if that fixes it.


[BUG] get_trips triggers an exception when connections do not have a duration set

Describe the bug
The method get_trips triggers an exception upon retrieving a trip that contains a connection that does not have the connection.duration attribute set.
Apparently, the VVS API does not set the duration property in case a connection takes less than a minute, i.e. the duration would be 0. This is at least what I assume to be reason for not adding the duration.
Expected behavior
The request should not fail. If no duration is given, it probably is 0 and thus does not need to be part of the final sum.
Proof of Concept
The following request is one that triggers the exception. One part of the trip contains a connection with only one stop, taking less than a minute.
from vvspy import get_trips
from datetime import datetime

trip = get_trips("de:08111:341", "de:08111:6118", trip_date=datetime(2022, 8, 1, 10, 15, 0, 0))
print(trip)

Code fragment
The problematic code can be found in https://github.com/zaanposni/vvspy/blob/master/vvspy/obj/trip.py#L33:
    def __init__(self, **kwargs):
        self.connections = []
        for connection in kwargs.get("legs", []):
            self.connections.append(Connection(**connection))

        self.duration = sum([x.duration for x in self.connections])

Suggestion
It has to be checked whether the duration attribute exists. If it does not exist, (I assume that) it is equal to 0 and therefore can be left out.
        self.duration = sum([x.duration for x in self.connections if x.duration])

Hi @chrrel
I have uploaded the new version (1.1.4) to pypi.
I have added you as a contributor to the readme file, I hope this is ok with you.
Thank you for your detailed report and the quick fix :smile:

Thanks, @zaanposni. That was really fast!


Spelling Mistake

https://github.com/facebook/docusaurus/blob/306d3afc0a79340999a8e8bb2568c4965611a555/packages/docusaurus-theme-classic/src/theme/hooks/useAnnouncementBar.ts#L20
should have been
themeConfig: {announcementBar: {id = 'announcement-bar'} = {}} = {},

thanks, will be fixed in https://github.com/facebook/docusaurus/pull/3343
also related: https://github.com/facebook/docusaurus/issues/334


How to set a monthly pick-up?

Before you submit a new bug report, please check that

[X] I have read frequently asked questions and limitations

Write your question here
I love the updated integration. I have it successfully tracking my garbage collection (twice weekly) and my recycling (every other week) using 2 separate integration instances added. I have a 3rd I'm trying to set that occurs on the 1st of every month (not a specific weekday, but rather Jan 1, Feb 1, Mar 1, Apr 1, etc.); I have tried all of the configuration options in the integration setup I can think of and can NOT get it to show on the proper day. Any chance you can provide guidance on what the setup should be to make that work?

The regular configuration is not designed for that. If you want to do this, you can use the "blank" frequency, setup the integration with a "Manual update" turn on, and have an automation that will fill-in the schedule automatically on each update. Do you want to do that? I'd think about a blueprint to do that. I think I saw more people who wanted to do that. I usually tell them to use google calendar integration instead of the garbage integration, but it comes up regularly :).

I have just realized that I did the blueprint already. Here it is:
https://github.com/bruxy70/Garbage-Collection#monthly-on-a-fixed-date

Thank you! I was going to look into this today to answer your question from Friday. I'll see if I can figure out how to implement the blueprint now.
In the meantime, to reply to your other question, I don't yet have Google Calendar integrated and per Jeff @ SlackerLabs (https://www.youtube.com/watch?v=NQH0c6YGQqY), it's not going to be straightforward to make any of it work, but question back to you...
IF I make Google calendar work in general, would I still use Garbage Integration for the collection reminders or just set a recurring event in Google and go from there?

No, I guess you's use the google calendar integration instead of garbage collection. Or you can use the blueprint. Up to you.

Got it. Well, I used the blueprint and it was VERY easy. I have it working and really appreciate it.
I find it weird that there's no "Add Event" feature in the calendar generally and it just pulls in dates from these other integrations, but that's beyond the point for now. Thank you again!


lqip support

In https://github.com/cyrilwanner/next-optimized-images/issues/120 you mentioned that this Img Component should also support an lqip placeholder. Is there any progress on this? Would love to try it out! :)

It is one of the features I'll be working on over the next weeks, but it isn't in canary yet. I'll notify you once it can get tested.

Are there any plans for trace too?
I think an <ImgLoader> component for the placeholders would be cool.
I could see that getting a lot of use.
Like this, but with same props as v3  like webp and resize.
/** Placeholder component for next-optimized-images v2 using image-trace-loader
    Credit given to https://github.com/EmilTholin/image-trace-loader **/

const ImgLoader = ({ traceObject, alt }: Props) => {
  const [imageLoaded, setImageLoaded] = useState(false);

  return (

       setImageLoaded(true)}
      />

  );
};

WebP + resize + your choice of placeholder is a potent combo indeed!

Are there any plans for trace too?

Yes, I plan to provide a component for all lazy loading techniques, like <BlurUp>, <Trace>, etc.
The API will probably be like this so you can use all normal properties of the  tag and use resizing or webp conversion as you like:
<BlurUp>

</BlurUp>

I already created a Placeholder component on my own in the meantime. Feel free to reuse anything you want from this component for this project: https://github.com/tilman/react-intersection-placeholder
Also if you need help I'm glad to help :)
It uses an Intersection Observer to only display the child if it is near the viewport.
import Img from 'react-optimized-image';
import TestImg from 'test.png';
import TestImgLqip from 'test.png?lqip';
...
<IntersectionPlaceholder
    src={TestImg} // original image needed for having the right dimensions
    lqip={TestImgLqip} // pass your placeholder image here
    observerOptions={{ threshold: 0.5 }} // you can also pass options for the intersection observer. Use the same config object as you would use to init the intersection observer.
>

</IntersectionPlaceholder>


Allow setting the module name for a pyclass

Following #474, this PR adds an argument for #[pyclass] that lets the user specify the name of the module the class is part of. I also added the right module for all builtin types (either builtins or datetime), with a few tests.
Maintenance

Run cargo fmt (This is checked by travis ci) :white_check_mark:
Run cargo clippy and check there are no hard errors (There are a bunch of existing warnings; This is also checked by travis)
If applicable, add an entry in the changelog. :white_check_mark:
If applicable, add documentation to all new items and extend the guide. :white_check_mark:
If applicable, add tests for all new or fixed functions :white_check_mark:

You might want to run tox (pip install tox) locally to check compatibility with all supported python versions. If you're using linux or mac you might find the Makefile helpful for testing.

Thanks, I think it's a really meaningful enhancement for us, but left comments about the coding.
In addition, I want a test in which we check that the module name is correctly set.

I forgot to push the latest commits with changelog expansion and tests for the module name, will do that when I can.

Thanks! :tada:


SafeProxy contract is not deployed on the current network

I am trying to send ETH from my safe wallet.
Safe address : 0x159bD467fafcfA348D2b5f357b23aA1d70E814A0

.proposeTransaction({
        safeAddress: checksumSafeAddresss,
        safeTransactionData: safeTransaction.data,
        safeTxHash,
        senderAddress: await owner1Signer.getAddress(),
        senderSignature: senderSignature.data,
      });

I am getting the error "SafeProxy contract is not deployed on the current network"
On Sepolia Network.
Note : I am doing it using code not using UI of safe.

Hi Gnosis team, kindly consider looking into this issue which is not resolved yet.
@katspaugh @rudolfs @vorot93 @koeppelmann

Hi Gnosis team, kindly consider looking into this issue which is not resolved yet.
@katspaugh @rudolfs @vorot93 @koeppelmann

Are you sure that the RPC you use is correct and that your SAFE_ADDRESS configured in the configService is correct?
If you look at the code in the safeDeploymentContracts.ts:
https://github.com/safe-global/safe-core-sdk/blob/main/packages/protocol-kit/src/contracts/safeDeploymentContracts.ts#L128
It's tryign to fetch the contract code from the network. Apparently for your safe this fails - you are either are passign wrong address or the RPC call is failing.

@faisal-anwar825
have a look at this please

It can happen if your wallet is on the wrong chain.

@katspaugh  thats not the case here!

Replace where?

@katspaugh !

In what code?

@katspaugh please check the attachments in start convo!

There’s no tx service URL in the attached code, so I’m not sure what exactly you’re replacing.
Are you following this guide? https://github.com/safe-global/safe-core-sdk/blob/main/guides/integrating-the-safe-core-sdk.md

The error you’re getting has nothing to do with proposing or the transaction service.
It cannot find a ProxyFactory contract on the chain your provider is on.
So as compojoom indicated above, check your provider and its RPC URL. L

are you ref to this one : Are you following this guide? https://github.com/safe-global/safe-core-sdk/blob/main/guides/integrating-the-safe-core-sdk.md ?

@katspaugh  Thank you so much !


EXPLAIN gives odd results with sub query

Given the query
EXPLAIN select from (select from Vertex) LIMIT 1

I get the results
elapsed: 2514.5999
recordReads: 1
documentReads: 1
resultSize: 1
evaluated: 1

Given the query took 2.5seconds I would have expected some of those numbers to be a lot higher (recordReads, documentReads, resultSize, evaluated)

This is more a question than an issue. Please post in Community Group where more than 2k users can answer to your question.

@lvca I saw it more as a bug / feature request, but I'll raise it in the forum. Out of interest would the orientdb team prefer all issues / feature request to go via the forum first, and is there a doc explaining what to raise where for newbies?

Raised for discussion in forum https://groups.google.com/forum/#!topic/orient-database/BSuiP8a9nqQ


Bump String Dependency

Bumping a vulnerable dependency:
Tests all passing locally:

@meetearnest/pl-grow
@meetearnest/decide

@wennergr that's a good point

Updated to a major version bump - looking through the commit history on the library there was a change to the method we use so I'll audit the monolith when I merge it in.


Runing SynthSR on Mac(M1)

Hello SynthSR team,
I am trying to run SynthSR on a Macbook Air (M1) using the development version of Freesurfer (freesurfer-macOS-darwin_arm64-dev.pkg).I'm using brain1.nii.gz (located in SynthSR's date folder) as my input, but the output shows there are two errors about the input dimensions. Do I have to convert the input dimensions to 4D in order to run it on my laptop? What method should I use to do the conversion (e.g. tensorflow or something else)?
Errors:
2 root error(s) found.
(0) INVALID_ARGUMENT:  input must be 4-dimensional[1,160,192,160,24]
[[{{node unet/unet_bn_down_0/FusedBatchNormV3}}]]
[[unet/unet_conv_downarm_2_0/BiasAdd/_38]]
(1) INVALID_ARGUMENT:  input must be 4-dimensional[1,160,192,160,24]
[[{{node unet/unet_bn_down_0/FusedBatchNormV3}}]]
0 successful operations.
0 derived errors ignored. [Op:__inference_predict_function_1938]

[Runing SynthSR on Mac.txt](https://github.com/BBillot/SynthSR/files/14315820/Runing.SynthSR.on.Mac.txt)
Best,
Siyan Wen

Hi, thanks for the interest in our tool.
just to be clear, this error is not due to SynthSR in particular, but to mac/tensorflow/GPU compatibility issues.
Please have a look at the answer provided by a member of our community for another one of our tool (SynthSeg).
https://github.com/BBillot/SynthSeg/issues/81#issuecomment-1947505992
A temporary solution is to force running the code on the cpu by using the --cpu flag.
The rest of the issue is independent to SynthSR, and will have to be solved externally (i.e., tensorflow support).
Hope this helps,
Benjamin


Broken Bypass on Ouo

Domain
https://ouo.io/
Link
https://ouo.io/9ZrsEG
Version
0.2251
What browsers are you seeing the problem on?
Microsoft Edge
What OS are you seeing the problem on?
Windows
Anything else?
It was working a week ago, but it no longer works.

Duplicate of #861


Remove deploymentName label on every ScaledObject

In #504 we've fixed an issue where we correctly log the missing deploymentName label on every ScaledObject.
However, do we still need the label? If so, what was the reason again?
Ideally, we get rid of this as it's duplicated data from spec.scaleTargetRef.deploymentName which is not ideal.

@tomkerkhove to be honest I don't see any reason for leaving that label.
As far as I can see the entire rest of the logic leverages on the spec.scaleTargetRef.

Yes, was somewhat the consensus on #242 but added to agenda for standup before we get things going as a final verification.

So from #242 it seems that we can remove the label as mandatory field to be specified by the user but the operator has to add it for the metrics server.

Chatted about this briefly in standup - sounds good. It doesn't sound like there is a downside to doing this

Are you up for it @ppatierno?

Yes, guys, I will take it over the weekend I guess.


Dynamic configuration

Hello Angie Developers,
We would like to dynamically manage the routes and/or upstream servers.
Saw You introduced API, but according to documentation it seems limited to read-only capabilities.
Is it possible currently or if planned any approximate expected eta?
I saw some references in closed items like #24 #30, but not very informative / up to date.
I'd like to get rid of Ocelot gateway service... )
Thanks in advice for such a great product!

+1
El lun., 6 nov. 2023 15:48, Dorin Marcoci @.***>
escribió:

Hello Angie Developers,
We would like to dynamically manage the routes and/or upstream servers.
Saw You introduced API, but according to documentation it seems limited to
read-only capabilities.
Is it possible currently or if planned any approximate expected eta?
I saw some references in closed items like #24
https://github.com/webserver-llc/angie/issues/24 #30
https://github.com/webserver-llc/angie/issues/30, but not very
informative / up to date.
I'd like to get rid of Ocelot gateway service... )
Thanks in advice for such a great product!
—
Reply to this email directly, view it on GitHub
https://github.com/webserver-llc/angie/issues/57, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMFCBLFT3IVAQLX53FCIU3DYDD2DZAVCNFSM6AAAAAA67UIQJ2VHI2DSMVQWIX3LMV43ASLTON2WKOZRHE3TSMZUHA3TKOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[Event Request] Table 49 "Invoice Post. Buffer" - OnBeforeAssignInvoicePostBuffer

Dear Microsoft,
We like to have to ability to have prepare PrepmtAdjInvPostBuffer our own way before it is assigned to InvoicePostBuffer.
Therefore we would appreciate the publisher OnBeforeAssignInvoicePostBuffer added as below:
procedure FillPrepmtAdjBuffer(var TempInvoicePostBuffer: Record "Invoice Post. Buffer" temporary; InvoicePostBuffer: Record "Invoice Post. Buffer"; GLAccountNo: Code[20]; AdjAmount: Decimal; RoundingEntry: Boolean)
var
    PrepmtAdjInvPostBuffer: Record "Invoice Post. Buffer";
begin
    with PrepmtAdjInvPostBuffer do begin
        Init;
        Type := Type::"Prepmt. Exch. Rate Difference";
        "G/L Account" := GLAccountNo;
        Amount := AdjAmount;
        if RoundingEntry then
            "Amount (ACY)" := AdjAmount
        else
            "Amount (ACY)" := 0;
        "Dimension Set ID" := InvoicePostBuffer."Dimension Set ID";
        "Global Dimension 1 Code" := InvoicePostBuffer."Global Dimension 1 Code";
        "Global Dimension 2 Code" := InvoicePostBuffer."Global Dimension 2 Code";
        "System-Created Entry" := true;
        "Entry Description" := InvoicePostBuffer."Entry Description";
        OnBeforeAssignInvoicePostBuffer(PrepmtAdjInvPostBuffer,InvoicePostBuffer); //<< New Publisher
        InvoicePostBuffer := PrepmtAdjInvPostBuffer;
        TempInvoicePostBuffer := InvoicePostBuffer;
        if TempInvoicePostBuffer.Find then begin
            TempInvoicePostBuffer.Amount += InvoicePostBuffer.Amount;
            TempInvoicePostBuffer."Amount (ACY)" += InvoicePostBuffer."Amount (ACY)";
            TempInvoicePostBuffer.Modify();
        end else begin
            TempInvoicePostBuffer := InvoicePostBuffer;
            TempInvoicePostBuffer.Insert();
        end;
    end;
end;

New publisher
[IntegrationEvent(false,false)]
local procedure OnBeforeAssignInvoicePostBuffer(var PrepmtAdjInvPostBuffer : Record "Invoice Post. Buffer";InvoicePostBuffer: Record "Invoice Post. Buffer")
begin
end;

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


[PSM Interop] Legacy test builds always pull the driver from master (v1.52.x backport)

Backport of #33699 to v1.52.x.
Similar to https://github.com/grpc/grpc-java/pull/8982.
All other languages already to this.
Needed for easier deployment of https://github.com/grpc/grpc/pull/33685.
ref b/274944592

[ ] grpc/core/v1.52.x/branch/linux/grpc_xds_v3
[ ] grpc/core/v1.52.x/branch/linux/grpc_xds_v3_python


Filter directories

Implements the filter_dir method in the neotest adapter interface
This reduces the initial load time of the neotest plugin in large repos by a substantial margin (20s to 9s in my work rails repo)
Currently using some simple rules. If the path has "spec" in it, it's good. If it doesn't, and it's got a "/" in it, it's bad.
These rules allow "engine/spec" to get picked up as well as "spec", but would miss "engine/something/spec"
afaik, that last case is not common or recommended? I'm fairly new to this whole ruby thing. So let me know if that's a problem.

Nice! Thanks for this and had been on my list for some time. 20s to 9s is an incredible improvement!


replace configure-builder with configure-run-image

I think we should allow users to specify additional run-image locations by adding local configuration for a run-image rather than for a builder. This will allow users to avoid specifying duplicate configuration for every builder that is based on the same stack. It also expresses the idea that all of the run images should be different locations for the same artifact. Finally It allows us to use the same run-image selection logic on both build and rebase.
set-run-image-mirrors
When I run pack set-run-image-mirrors <run-image> --mirror=<mirror-image-1> --mirror=<mirror-image-2>
Then I can see the following entry in my config.toml
[[run-images]]
 image = <run-image>
 mirrors =  [<mirror-image-1>, <mirror-image-2>]

create-builder
Given I have a stack and run image mirrors configured
[[stacks]]
 id = <stack-id>
 build-image = <build-image>
 run-image = <run-image>

[[run-images]]
 image = <run-image>
 mirrors =  [<mirror-image-1>, <mirror-image-2>]

When I run pack create-builder -s <stack-id>
Then my builder is generated with the follow metadata label which excludes the mirrors
{"runImage": { "image": "<run-image>", "mirrors": []} }

pack build
When  I run pack build <app-image> --run-image=<custom-run-image>
Then <custom-run-image is used as the base image for <app-image>. We will not look up mirrors
Given I have configured run image mirrors
[[run-images]]
 image = <run-image>
 mirrors =  [<mirror-image-1>, <mirror-image-2>]

Given my default builder has the following metadata label {"runImage": { "image": "<run-image>", "mirrors": [<builder-configured-mirror>]} }
When  I run pack build <app-image>
Then Then the run image is chosen by finding a image with a registry that matches the app-image preferring a locally configured mirror to a mirror from the builder metadata. (same behavior and order of precedence that we currently have)
pack inspect-builder
Given I have configured run image mirrors
[[run-images]]
 image = <run-image>
 mirrors =  [<mirror-image-1>, <mirror-image-2>]

Given my builder has the following metadata label {"runImage": { "image": "<run-image>", "mirrors": [<builder-configured-mirror>]} }
When  I run pack inspect builder <builder>
Then the run images section of the output should look like:
Run image: <run-image>
Run images:
  <mirror-image-1> (locally-configured)
  <mirror-image-2> (locally-configured)
  <run-image>
  <builder-configured-mirror>

tag was used instead of image in config.toml. We should avoid the word "tag," because:

Digest references are valid in this context (e.g., example.com/image@sha256:abc2345)
tag is often misused (even in official documentation) to refer to the part of the tag after the :

ref would also be acceptable, but we seem to use image consistently in other places, so I think we should use that here.


Is any analogue for Android?

Hi, @nurshark!
Thank you for your interest! As a Swift Package built with native Apple frameworks, this project is not available for Android development.
You could find similar projects for Android apps online. For example, I found this Dart package for Flutter that seems to implement something along the lines of this repo.
I'm sure you can find something good and proper for your stack. Have fun coding! 🍀


Include kytos/kytos_stats on Test Analytics Dashboard

We'll also need to include kytos/kytos_stats on Test Analytics Dashboard, and also create internal GitLab projects to run its unit tests on a nightly basis. Our team uses this dashboard to have an overview of the current tests status, and also we link it on the release notes, so having kytos/kytos_stats there it'd be great before we consolidate the upcoming release notes for 2023.1.
@gretelliz, can you help out with this one?

Sure

This issue has been solved by following the steps:
Adding a Napp to the AmLight Kytos-ng Test Analytics Dashboard and also to daily e-mail report

Make sure the Napp is included in Scrutinizer execution:

case: kytos_stats
Yes, it is https://scrutinizer-ci.com/g/kytos-ng/kytos_stats/

Add the unit tests to AmLight gitlab-ci daily execution
1.1) create a gitlab-ci execution:

Create a new project in https://gitlab.ampath.net/kytos and configure a new file (.gitlab-ci.yml)
Create a new schedule within the new project with the option CI/CD/Schedules.

1.2) add the new test to amlight daily report (crontab script)

Add the new Napp to the Test Analytics dashboard app:

Include the repo on app.py

Deploy a new version of the Test Analytics dashboard

Validate if the Test Analytics dashboard is working as expetectd


Replace lodash.get with local function; support array paths.

@olsonpm or @sudo-suhas for review, please.
cc @stof, @cspotcode

Looks good to me. 👍

I've got things goin on til sunday - will review then.