Use After Free error

Thanks for this project; I was able to migrate off of Redis 7.4 after I inadvertently upgraded my databases.
To get it to build under Debian I had to comment out -Werror from deps/redis/Makefile as it was complaining about a potential use-after-free error. With that commented out it built and ran fine (once I figured out I needed to clone hiredis; might want to add that to the documentation).

Hi @koehn,
I created PR that should resolve the hiredis issue.
Regarding the use-after-free can you dig little further please to see if this is a real issue?
Thank you.

Here’s what I get with the new Makefiles using a clean debian:latest docker image:
# apt update && apt-get install -y build-essential cmake make git
[snip]
# git clone https://github.com/redis/librdb.git
[cd into librdb and apply your changes to the makefiles]
# make example
git submodule update --init deps/hiredis
Submodule 'deps/hiredis' (https://github.com/redis/hiredis.git) registered for path 'deps/hiredis'
Cloning into '/librdb/deps/hiredis'...
Submodule path 'deps/hiredis': checked out '869f3d0ef1513dd0258ad7190c9914df16dcc4a4'
make -C deps  all
make[1]: Entering directory '/librdb/deps'
make -C redis all
make[2]: Entering directory '/librdb/deps/redis'
cc -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden -c crc64.c -o crc64.o -g3 -DDEBUG=1
cc -MM -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden crc64.c > crc64.d
cc -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden -c crcspeed.c -o crcspeed.o -g3 -DDEBUG=1
cc -MM -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden crcspeed.c > crcspeed.d
cc -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden -c endianconv.c -o endianconv.o -g3 -DDEBUG=1
cc -MM -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden endianconv.c > endianconv.d
cc -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden -c fpconv_dtoa.c -o fpconv_dtoa.o -g3 -DDEBUG=1
cc -MM -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden fpconv_dtoa.c > fpconv_dtoa.d
cc -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden -c intset.c -o intset.o -g3 -DDEBUG=1
cc -MM -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden intset.c > intset.d
cc -fPIC -O3 -std=c99 -Wall -Wextra -pedantic -Werror -fvisibility=hidden -c listpack.c -o listpack.o -g3 -DDEBUG=1
listpack.c: In function 'lpDeleteRangeWithEntry':
listpack.c:937:31: error: pointer 'lp' used after 'realloc' [-Werror=use-after-free]
  937 |     unsigned long poff = first-lp;
      |                          ~~~~~^~~
In file included from listpack.c:45:
In function 'lpShrinkToFit',
    inlined from 'lpDeleteRangeWithEntry' at listpack.c:945:10:
listpack_malloc.h:47:28: note: call to 'realloc' here
   47 | #define lp_realloc(ptr,sz) realloc(ptr,sz)
      |                            ^~~~~~~~~~~~~~~
listpack.c:177:16: note: in expansion of macro 'lp_realloc'
  177 |         return lp_realloc(lp, size);
      |                ^~~~~~~~~~
cc1: all warnings being treated as errors
make[2]: *** [Makefile:15: listpack.o] Error 1
make[2]: Leaving directory '/librdb/deps/redis'
make[1]: *** [Makefile:2: all] Error 2
make[1]: Leaving directory '/librdb/deps'
make: *** [Makefile:26: all] Error 2

So it’s cloning hiredis, which is good. Looks like I’m still needing to uncomment -Werror, which may be another issue.

Here’s a Dockerfile that will reproduce the test case:
FROM debian:latest

RUN apt-get update && apt-get install -y build-essential git make

RUN git clone https://github.com/redis/librdb.git

# Uncomment the line below to get a successful build
#RUN sed 's/^\(.*\) -Werror/\1/' -i librdb/deps/redis/Makefile

RUN cd librdb && make example```

Save that to a file named `Dockerfile`, then run `docker build .`

Uncomment the `RUN sed` line to make the build pass.

Looking at the offending code, it seems like the compiler is just pitching a fit.
    /* Store the offset of the element 'first', so that we can obtain its
     * address again after a reallocation. */
    unsigned long poff = first-lp;

    /* Move tail to the front of the listpack */
    memmove(first, tail, eofptr - tail + 1);
    lpSetTotalBytes(lp, bytes - (tail - first));
    uint32_t numele = lpGetNumElements(lp);
    if (numele != LP_HDR_NUMELE_UNKNOWN)
        lpSetNumElements(lp, numele-deleted);
    lp = lpShrinkToFit(lp);

    /* Store the entry. */
    *p = lp+poff;
    if ((*p)[0] == LP_EOF) *p = NULL;

    return lp;

My guess is (I haven’t done any serious C coding since the nineties) if you simply used a different variable to hold the results of lpShrinkToFit() instead of re-assigning it to lp, the compiler wouldn’t throw a fit about use after free. I’ll try to make a branch and test it out.

Hmm. Changing the assignment to a new variable didn’t help; the compiler is convinced that one of the preceding (inlined) functions is calling realloc, apparently in lpAssertValidEntry() (as that’s the last place it’s used).
I’m at a loss.

I reproduced the warnning. It doesn't look like a real problem.
Played a little with warnning flags of deps/Makefile. When I optimized with flag -flto=auto the issue disappear like magic.
Please verify it works for you as well.

That works for me!

Hi @moticless, what's your opinion? Hope this issue can be fixed ASAP. Thanks.

Hi @JunhongMao ,
I’m might get to this by the end of the week. Maybe you be able to assist with fixing it? Thanks.

Yes. My pleasure.

Hi @JunhongMao , I’m might get to this by the end of the week. Maybe you be able to assist with fixing it? Thanks.


client_connections doesn't work?

Hi, any
I'm using twemproxy in v0.4.1
# nutcracker -V
This is nutcracker-0.4.1

And I'm using this config
proxy01:
  listen: <IP_ADDRESS>:8001
  auto_eject_hosts: false
  hash: fnv1a_64
  distribution: modula
  redis: true
  preconnect: true
  timeout: 300
  client_connections: 1
  servers:
  - localhost:7001:1 redis7001
  - localhost:7002:1 redis7002

I've set client_connections to 1, but when I run redis-benchmark with -c flag to start multi clients like this
redis-benchmark -t get -c 5 -p 8001 -n 10000000
It didn't get any errors. The result of status port is like this

{
   "source" : "twemproxy01",
   "curr_connections" : 8,
   "version" : "0.4.1",
   "total_connections" : 48,
   "service" : "nutcracker",
   "proxy01" : {
      "redis7001" : {
         "request_bytes" : 0,
         "server_timedout" : 0,
         "requests" : 0,
         "server_ejected_at" : 0,
         "out_queue_bytes" : 0,
         "in_queue_bytes" : 0,
         "out_queue" : 0,
         "in_queue" : 0,
         "response_bytes" : 0,
         "server_err" : 0,
         "responses" : 0,
         "server_eof" : 0,
         "server_connections" : 1
      },
      "client_eof" : 35,
      "forward_error" : 0,
      "fragments" : 0,
      "redis7002" : {
         "request_bytes" : 23035068,
         "server_timedout" : 0,
         "requests" : 639863,
         "server_ejected_at" : 0,
         "out_queue_bytes" : 72,
         "in_queue_bytes" : 0,
         "out_queue" : 2,
         "in_queue" : 0,
         "response_bytes" : 3199305,
         "server_err" : 0,
         "responses" : 639861,
         "server_eof" : 0,
         "server_connections" : 1
      },
      "client_err" : 5,
      "client_connections" : 5,
      "server_ejects" : 0
   },
   "timestamp" :<PHONE_NUMBER>,
   "uptime" : 126
}

It looks like that client_connections does not work.

@Shanicky actually, it never works.

马晓栋已经收到您的来件，谢谢!


New version: Mythicsoft.AgentRansack version 9.2.3418.1

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: Open in CodeFlow

Manifest Error: Duplicate Entry is found.


Space around fancy zones doesn't reflect specified value

Microsoft PowerToys version
0.64.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
FancyZones
Steps to reproduce
After this update, the parameter "Space around zones" not working anymore.
✔ Expected Behavior
The space around zones should change according to the specified value, or set to 0 if disabled.
❌ Actual Behavior
The space around zones stay at default value (it seems to be 16px).
Other Software
No response

Downgrade to v0.63.0 works (need to uninstall v0.64 first)

same here: #21643

Duplicate of #21643

Related observations:
It is ignoring the "Space Around Zones" toggle in the layout editor, If you toggle space on (or if it's already on) and set it to zero (which is a clunky process with the slider - why can we not double-click and enter a value) it will look right, though you need to do this for every layout you use.
In short, it's the "Space Around Zones" toggle which is bugged.

@benzo8 thanks - for anyone else trying to get to 0px and overshooting with the mouse cursor, using the arrow keys is much easier.

/dup of https://github.com/microsoft/PowerToys/issues/21630


"Connect Wallet" button doesn't work & Contract information does not appear

What version are you using?
node 18.15.0
rustc 1.71.0
Two Freighter setup (same bug in both)
(1) Firefox & Freigher v5.2.3
(2) Brave & Freighter v5.1.2
What did you do?
(1)
In one terminal bash quickstart.sh standalone
In onother terminal
npm run clear
NETWORK=standalone npm run setup
npm run dev

What did you expect to see?

When pressing "Connect Wallet" button, Freigher should pop up.
After workaround (see down). I should see contract data and Dapp buttons (deposit, etc...)

What did you see instead?

When pressing "Connect Wallet", nothing happens.

However I did a workaround. Manually opened Freighter and type Password. Then Dapp showed my address, however, no Dapp information, neither buttons, just an infinite spinning circle:

Bb


 Items marked with ✓ are implemented in a PR I'm working on.
Current Behavior
import { lookup } from 'fp-ts/lib/ReadonlyArray'

lookup(0, [1, 2, 3])

Desired Behavior
pipe([1, 2, 3], lookup(0))

Suggested Solution
Add proper overloadings
export function lookup(i: number): <A>(as: ReadonlyArray<A>) => Option<A> // <= new signature
export function lookup<A>(i: number, as: ReadonlyArray<A>): Option<A> // <= old signature
export function lookup<A>(i: number, as?: ReadonlyArray<A>): Option<A> | ((as: ReadonlyArray<A>) => Option<A>) {
  if (as === undefined) {
    return (as) => lookup(i, as)
  } else {
    return isOutOfBound(i, as) ? none : some(as[i])
  }
}

Who does this impact? Who is this for?
<EMAIL_ADDRESS>users

Closed by https://github.com/gcanti/fp-ts/pull/1244


Add materialization to exhaust method in IonTests tests

Make sure that instead of just navigating over values we are actually materializing them (getInteger(), getTimestamp(), or whatever they're called)

PR #118 accomplishes this.


Fix not-null assertion in PdfDocumentReaderConfig

This appears to be a copy-and-paste issue.

Thanks @alexcheng1982


Test-PasswordQuality - key ShowPlainTextPasswords in not available

Hi!
In version 2.22 I was able copy plain text password in result file using the key -ShowPlainTextPasswords of Test-PasswordQuality cmdlet .
In version 3.0 this key is not available.
Can you please check?

Hello @kutsev, after much thinking, I have intentionally removed this feature in version 3.0:

The main use case is now checking passwords against HaveIBeenPwned, which only provides hashes, so the actual passwords cannot be shown.
Several IT managers have told me that they would prefer if the auditor did not see the actual passwords.

What is your opinion?

I understand your opinion, in most situation, I think, if password match some weak password database, it should be changed, no matter what password actually it is.
Nowadays I have a task from security team to audit users with certain passwords and know what certain password was used (because it was possible that someone of the IT team assigned such password to users and that's their responsibility).
Ok, in case I'll need the plain text password, I will use old version. Thank you

Thanks for your feedback. I might give it another thought in the future.
Yes, one approach is using the old version. Or you could split your dictionary into multiple files / command lines and run Test-PasswordQuality multiple times to identify users with those specific passwords that might have been assigned by administrators.


task/g55-alias-transactions: Added AddressAliasTransaction and MosaicAliasTransaction features

Added

Added models Alias, AliasType, AddressAlias, MosaicAlias, EmptyAlias
Added NamespaceHttp.getLinkedMosaicId(namespaceId)
Added NamespaceHttp.getLinkedAddress(namespaceId)
Added AddressAliasTransaction
Added MosaicAliasTransaction

Details

This PR is based on PR#63.
This PR is to achieve the Issue#55 - Alias

Pull Request Test Coverage Report for Build 140

94 of 126 (74.6%)  changed or added relevant lines in 15 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.1%) to 71.994%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/model/namespace/AddressAlias.ts
5
6
83.33%

src/model/namespace/EmptyAlias.ts
2
3
66.67%

src/model/namespace/MosaicAlias.ts
5
6
83.33%

src/model/mosaic/NetworkCurrencyMosaic.ts
19
21
90.48%

src/infrastructure/transaction/CreateTransactionFromDTO.ts
2
6
33.33%

src/model/mosaic/NetworkHarvestMosaic.ts
17
21
80.95%

src/infrastructure/NamespaceHttp.ts
5
24
20.83%

Totals

Change from base Build 138:
-1.1%

Covered Lines:
850

Relevant Lines:
1103

💛  - Coveralls


Model events for DNSResponse lifecycle

It recently occurred to me that the start and stop times for a DNSResponse are incorrect. The current time starts when the DNSRequest class is created, and ends when the DNSResponse#done event is called. This truncates a large portion of the request lifecycle largely contained in the network interface, including network transmission and serialization/deserialization.
We should, first of all, fix the timestamps for the start times to begin when the request first begins. Then, we should model an appropriate lifecycle of events and consider storing a timestamp for each across the lifecycle of the DNS packet (possibly configurable with a flag if you don't want to waste the modest compute on getting the current hrtime for each of these:

request begins
request all request data received
deserialization complete / handler chain begins (should be roughly identical)
plugin chain assigned (router has finished constructing handler chain)
plugin chain ends (possibly recording the latency for each plugin handler, a la #24).
serialization complete
all data sent / request finished

There may be a few of these we can justify cutting out. This may also merely be a duplicate of #24 and may be something we ONLY want to do if a detailed "profile" flag is enabled.
Regardless, we need to correct the implementation of the request start times and end times to occur in the network event, meaning we at least need to fix the DNSRequest#done event. Perhaps we can replace it with a DNSRequest#answer event, and remap DNSRequest#done to mean "the request has finished sending back to the client."

This is partially addressed by #27 , however, we'll probably still need some configurable way to add in profiling for this to be a completed issue.


fix(schematics): fix the scripts in package.json

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/scullyio/scully/blob/main/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?
After we run the build/serve script, it will get an error like the following:
$ npm run scully

><EMAIL_ADDRESS>scully /Users/leo/Documents/Angular小聚/2021/scully
> npm scully --

Usage: npm <command>

where <command> is one of:
    access, adduser, audit, bin, bugs, c, cache, ci, cit,
    clean-install, clean-install-test, completion, config,
    create, ddp, dedupe, deprecate, dist-tag, docs, doctor,
    edit, explore, fund, get, help, help-search, hook, i, init,
    install, install-ci-test, install-test, it, link, list, ln,
    login, logout, ls, org, outdated, owner, pack, ping, prefix,
    profile, prune, publish, rb, rebuild, repo, restart, root,
    run, run-script, s, se, search, set, shrinkwrap, star,
    stars, start, stop, t, team, test, token, tst, un,
    uninstall, unpublish, unstar, up, update, v, version, view,
    whoami

npm <command> -h  quick help on <command>
npm -l            display full usage info
npm help <term>   search for help on <term>
npm help npm      involved overview

Specify configs in the ini-formatted file:
    /Users/leo/.npmrc
or on the command line via: npm <command> --key value
Config info can be viewed via: npm help config

<EMAIL_ADDRESS>/Users/leo/.nvm/versions/node/v14.5.0/lib/node_modules/npm

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>scully: `npm scully --`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>scully script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/leo/.npm/_logs/2021-02-24T01_42_59_774Z-debug.log

What is the new behavior?
It will build/serve correctly.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

nope, this will not work if the project needs one or more params

I believe they will add -- in the script when people need one or more parameters, but the function modifyPackageJson will add the scripts in the package.json.
The original code will be confusing who the people used if they don't need any parameters.


How to work with Hashes inside Native Functions? e.g. Hash statHash

Greetings,
So i've been looking over the Native Trainer code and seen that the 'Add Cash' feature of the mod uses STAT_GET_INT and STAT_SET_INT to increase money by a certain amount:
				for (int i = 0; i < 3; i++)
				{
					char statNameFull[32];
					sprintf_s(statNameFull, "SP%d_TOTAL_CASH", i);
					Hash hash = GAMEPLAY::GET_HASH_KEY(statNameFull);
					int val;
					STATS::STAT_GET_INT(hash, &val, -1);
					val += 1000000;
					STATS::STAT_SET_INT(hash, val, 1);
				}

So i tried to do something similar in my code, the problem is that if i do something like:
Hash hash = Function.Call(GET_HASH_KEY, "someString");
int val;
Function.Call(Hash.STAT_GET_INT, hash, &val, -1);
I get an exception saying that no such overloaded method exist. The issue here is the hash part in the parameters i think, but how can i work with Hash keys then?

int hash = Function.Call(GET_HASH_KEY, "someString");
int val;
Function.Call(Hash.STAT_GET_INT, hash, &val, -1);

I.e. if you are using function that returns something - you should specify return type in <some_return_type>.
And Hash (in type.h from ScriptHookV SDK) it's usual UInt32 :wink: So, you should use UInt32 or int.

Thank you :-)
@IncoCode Could you try to convert the example i gave from native trainer into C# code? :-)

for ( int i = 0; i < 3; i++ )
unsafe
{
	int hash = Function.Call<int>( Hash.GET_HASH_KEY, "SP" + i + "_TOTAL_CASH" );
	int val;
	Function.Call( Hash.STAT_GET_INT, hash, &val, -1 );
	val += 1000000;
	Function.Call( Hash.STAT_SET_INT, hash, val, 1 );
}

@IncoCode Ah now i get it! Thanks man, the SP represents single player and the i represents the single player character. So i could do something like:
for ( int i = 0; i < 3; i++ )
unsafe
{
int hash = Function.Call( Hash.GET_HASH_KEY, "SP1_TOTAL_CASH" );
int val;
Function.Call( Hash.STAT_GET_INT, hash, &val, -1 );
val += 1000000;
Function.Call( Hash.STAT_SET_INT, hash, val, 1 );
}
For Franklin :-)
But where can i find these Hashes?

Is there any reason for doing this natively? We have a money property :-)

@Frede1012 Well it's not for the money, i'm just trying to get my head around hashes. I need it for other purposes :-) The money was just to put out an example which was easy to work with and test :-)

@frodzet yeah. Stats in spstatssetup.xml in one of the rpf-file.
@Frede1012 but if you want add money only to Franklin and your current character Trevor?

@frodzet but you forgot to add <int> after Call.
int hash = Function.Call<int>( Hash.GET_HASH_KEY, "SP1_TOTAL_CASH" );

@IncoCode I didn't forget to add it, somehow github removes it when i copy paste it from my code :-P I know that things with return types requires the opening and closing angle brackets :-) But thanks for the heads up! :-)

@frodzet use code blocks https://help.github.com/articles/github-flavored-markdown/#syntax-highlighting

@IncoCode Thanks :-)

@IncoCode Do you know why this isn't working properly:
    for ( int i = 0; i < 3; i++ )
    unsafe
    {
        int hash = Function.Call<int>(Hash.GET_HASH_KEY, "SP" + i + "_STAMINA");
        int test;
        Function.Call(Hash.STAT_GET_INT, hash, &test, -1);
        test = 100;
        Function.Call(Hash.STAT_SET_BOOL, hash, test, 1);
    }

I'm trying to set a stat value like Stamina, but only if i go to the Pause Menu and into Stats is it changed, as soon as i hit ALT my Stamina value have not changed, and if i go back into Stats it is back to it's original value.
Is there a way to bypass this? Do i have the right values in my parameters? The HASH.STAT_GET_INT is probably redundant for this particular case, since i don't care about what the current value is.

Stamina is int value. But you trying to set 100 as bool :grimacing:
Try to use STAT_SET_INT.

@IncoCode Oops, i know - just forgot to change to INT, already tried that with int instead, was trying to set STAMINA_MAXED with bool :-P

@frodzet resolved?

@JohnnyCrazy Unfortunately no. I can't seem to work out how to change a stat value of a player.

By the way, did you tried other stats? The stamina stat didn't work even in c++ :confused:

@IncoCode I did not! :-) Maybe i should try that out later :-)

Tested it, the results remains the same for any other stat as for the STAMINA. So i guess in order to change stats of "abilities" we need some other way to approach it! :-)


Database Instance Outage Report

The system needs to be able to produce the data for a database instance outage report.

Closing this for now.  This is still a target for the project, but is not something we are currently working directly on.


High-granularity in-game volume sliders

Is your feature request related to a problem? Please describe.
In-game sounds and music have volume sliders that usually leave the user wishing there were more than 5 volume levels.
Describe the solution you'd like
A continuous slider (as a bonus, with very high granularity towards the quieter volume).
Describe alternatives you've considered
Complaining to Mod Ash on Twitter

Duplicate #2213


[release-0.59] rpm: Update virtualization packages, add Istio Label, and disable custom SELinux policy on Centos 9 providers

What this PR does / why we need it:
This is a manual cherrypick of https://github.com/kubevirt/kubevirt/pull/9157 , https://github.com/kubevirt/kubevirt/pull/9159 , https://github.com/kubevirt/kubevirt/pull/9134 and https://github.com/kubevirt/kubevirt/pull/9248
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

This version of KubeVirt includes upgraded virtualization technology based on libvirt 9.0.0 and QEMU 7.2.0.
Each new release of libvirt and QEMU contains numerous improvements and bug fixes.

/cc @dhiller @acardace

/retest-required

/lgtm

@phoracek the network tests around passt seem to fail on this one: https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/9267/pull-kubevirt-e2e-k8s-1.26-sig-network-0.59/1626511129716985856

@phoracek the network tests around passt seem to fail on this one: https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/9267/pull-kubevirt-e2e-k8s-1.26-sig-network-0.59/1626511129716985856

Could it be related to https://github.com/kubevirt/kubevirt/pull/9134 or https://github.com/kubevirt/kubevirt/pull/9155?

@dhiller Tested locally: https://github.com/kubevirt/kubevirt/pull/9134 should resolve the passt failure
Added the commit

/lgtm
Thanks for addressing this. Would you please reflect those Passt backports in the PR title too?

Thanks for addressing this. Would you please reflect those Passt backports in the PR title too?

@phoracek Thanks! Sure, done!

/retest-required


GattService.GetCharacteristicAsync(BluetoothUuid) on iOS causing System.NullReferenceException due to race condition

Platform: net6.0-ios
Lib: InTheHand.BluetoothLE 4.0.25, 4.0.26
To reproduce:
// Example getting battery level from a connected BLE device
var service = await bluetoothDevice.Gatt.GetPrimaryServiceAsync(BluetoothUuid.FromShortId(0x180f));
if (service != null) {
  // NullReferenceException raised in call to GetCharacteristicAsync().
  var characteristic = await service.GetCharacteristicAsync(BluetoothUuid.FromShortId(0x2a19));
}

The crash log shows:
2022-11-02 11:27:09.935 TestBLELibrary[698:215217] Unhandled managed exception: Object reference not set to an instance of an object. (System.NullReferenceException)
   at InTheHand.Bluetooth.GattService.PlatformGetCharacteristic(BluetoothUuid characteristic) in C:\Users\Me\work\ThirdParty\32feet\InTheHand.BluetoothLE\Platforms\Apple\GattService.unified.cs:line 46
   at InTheHand.Bluetooth.GattService.GetCharacteristicAsync(BluetoothUuid characteristic) in C:\Users\Me\work\ThirdParty\32feet\InTheHand.BluetoothLE\GattService.cs:line 32
   at TestBLELibrary.MainPage.OnCounterClicked(Object sender, EventArgs e) in C:\Users\Me\work\TestBLELibrary\TestBLELibrary\MainPage.xaml.cs:line 80
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_0(Object )
   at Foundation.NSAsyncSynchronizationContextDispatcher.Apply()
   at UIKit.UIApplication.UIApplicationMain(Int32 , String[] , IntPtr , IntPtr )
   at UIKit.UIApplication.Main(String[] , Type , Type )
   at TestBLELibrary.Program.Main(String[] args) in C:\Users\Me\work\TestBLELibrary\TestBLELibrary\Platforms\iOS\Program.cs:line 13

Looking into PlatformGetCharacteristic(BluetoothUuid) shows that it calls ((CBPeripheral)Device).DiscoverCharacteristics() and uses the value of _service.Characteristics immediately afterwards.  It would seem that it should wait for a callback to DiscoverCharacteristics(), similar to PlatformGetCharacteristics().
A possible fix might be something like (shamelessly adapted from PlatformGetCharacteristics()):
        Task<GattCharacteristic> PlatformGetCharacteristic(BluetoothUuid characteristic)
        {
            TaskCompletionSource<GattCharacteristic> tcs = new TaskCompletionSource<GattCharacteristic>();
            CBPeripheral peripheral = Device;

            void handler(object sender, CBServiceEventArgs args)
            {
#if NET6_0_OR_GREATER
                peripheral.DiscoveredCharacteristics -= handler;
#else
                peripheral.DiscoveredCharacteristic -= handler;
#endif
                if (args.Error != null)
                    tcs.SetException(new Exception(args.Error.ToString()));

                GattCharacteristic matchingCharacteristic = null;
                foreach (CBCharacteristic cbcharacteristic in _service.Characteristics)
                {
                    if ((BluetoothUuid)cbcharacteristic.UUID == characteristic)
                    {
                        matchingCharacteristic = new GattCharacteristic(this, cbcharacteristic);
                        break;
                    }
                }

                tcs.SetResult(matchingCharacteristic);
            }

#if NET6_0_OR_GREATER
            peripheral.DiscoveredCharacteristics += handler;
#else
            peripheral.DiscoveredCharacteristic += handler;
#endif
            ((CBPeripheral)Device).DiscoverCharacteristics(new CBUUID[] { characteristic }, _service);

            return tcs.Task;
        }

Resolved in 4.0.32


Build fails on the commandline

Hi,
I'm trying to build this DLL on the commandline. I see a .sln file and two csproj files, but the osklib csproj file seems incomplete:
Microsoft (R) Build Engine version 14.0.24720.0
Copyright (C) Microsoft Corporation. All rights reserved.

Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
Build started 9/13/2017 12:03:20 PM.
Project "C:\osklib\Osklib.sln" on node 1 (default targets).
ValidateSolutionConfiguration:
  Building solution configuration "Debug|Any CPU".
Project "C:\osklib\Osklib.sln" (1) is building "C:\osklib\Osklib\Osklib.csproj" (2) on node 1 (default targets).
C:\osklib\Osklib\Osklib.csproj(1,1): error MSB4041: The default XML namespace of the project must be the MSBuild XML namespace. If the project is authored in the MSBuild 2003 format, please add xmlns="http://schemas.microsoft.com/developer/msbuild/2003" to the <Project> element. If the project has been authored in the old 1.0 or 1.2 format, please convert it to MSBuild 2003 format.
Done Building Project "C:\osklib\Osklib\Osklib.csproj" (default targets) -- FAILED.
The target "BeforeGenerateProjectPriFile" listed in a BeforeTargets attribute at "C:\Program Files (x86)\MSBuild\Microsoft\NuGet\Microsoft.NuGet.targets (186,61)" does not exist in the project, and will be ignored.
Project "C:\osklib\Osklib.sln" (1) is building "C:\osklib\ConsoleTester\ConsoleTester.csproj" (3) on node 1 (default targets).
PrepareForBuild:
  Creating directory "bin\Debug\".
  Creating directory "obj\Debug\".
Project "C:\osklib\ConsoleTester\ConsoleTester.csproj" (3) is building "C:\osklib\Osklib\Osklib.csproj" (2:2) on node 1 (default targets).
C:\osklib\Osklib\Osklib.csproj(1,1): error MSB4041: The default XML namespace of the project must be the MSBuild XML namespace. If the project is authored in the MSBuild 2003 format, please add xmlns="http://schemas.microsoft.com/developer/msbuild/2003" to the <Project> element. If the project has been authored in the old 1.0 or 1.2 format, please convert it to MSBuild 2003 format.
Done Building Project "C:\osklib\Osklib\Osklib.csproj" (default targets) -- FAILED.
Done Building Project "C:\osklib\ConsoleTester\ConsoleTester.csproj" (default targets) -- FAILED.
Done Building Project "C:\osklib\Osklib.sln" (default targets) -- FAILED.

Build FAILED.

"C:\osklib\Osklib.sln" (default target) (1) ->
"C:\osklib\Osklib\Osklib.csproj" (default target) (2) ->
  C:\osklib\Osklib\Osklib.csproj(1,1): error MSB4041: The default XML namespace of the project must be the MSBuild XML namespace. If the project is authored in the MSBuild 2003 format, please add xmlns="http://schemas.microsoft.com/developer/msbuild/2003" to the <Project> element. If the project has been authored in the old 1.0 or 1.2 format, please convert it to MSBuild 2003 format.

"C:\osklib\Osklib.sln" (default target) (1) ->
"C:\osklib\ConsoleTester\ConsoleTester.csproj" (default target) (3) ->
"C:\osklib\Osklib\Osklib.csproj" (default target) (2:2) ->
  C:\osklib\Osklib\Osklib.csproj(1,1): error MSB4041: The default XML namespace of the project must be the MSBuild XML namespace. If the project is authored in the MSBuild 2003 format, please add xmlns="http://schemas.microsoft.com/developer/msbuild/2003" to the <Project> element. If the project has been authored in the old 1.0 or 1.2 format, please convert it to MSBuild 2003 format.

    0 Warning(s)
    2 Error(s)

Time Elapsed 00:00:00.18

So it seems that the Osklib.csproj is outdated. I can add the xmlns to the Project element, but then I still get:
Microsoft (R) Build Engine version 14.0.24720.0
Copyright (C) Microsoft Corporation. All rights reserved.

Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
Build started 9/13/2017 12:04:21 PM.
Project "C:\osklib\Osklib.sln" on node 1 (default targets).
ValidateSolutionConfiguration:
  Building solution configuration "Debug|Any CPU".
Project "C:\osklib\Osklib.sln" (1) is building "C:\osklib\Osklib\Osklib.csproj" (2) on node 1 (default targets).
C:\osklib\Osklib\Osklib.csproj : error MSB4040: There is no target in the project.
Done Building Project "C:\osklib\Osklib\Osklib.csproj" (default targets) -- FAILED.
The target "BeforeGenerateProjectPriFile" listed in a BeforeTargets attribute at "C:\Program Files (x86)\MSBuild\Microsoft\NuGet\Microsoft.NuGet.targets (186,61)" does not exist in the project, and will be ignored.
Done Building Project "C:\osklib\Osklib.sln" (default targets) -- FAILED.

Build FAILED.

"C:\osklib\Osklib.sln" (default target) (1) ->
"C:\osklib\Osklib\Osklib.csproj" (default target) (2) ->
  C:\osklib\Osklib\Osklib.csproj : error MSB4040: There is no target in the project.

    0 Warning(s)
    1 Error(s)

Time Elapsed 00:00:00.23

With the following Osklib.csproj I can get it to compile:
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="15.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" Condition="Exists('$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props')" />
  <PropertyGroup>
    <TargetFrameworks>net40;net45;net461</TargetFrameworks>
    <Description>Library for controlling Windows 10 On Screen Keyboard</Description>
    <PackageTags>osk, OnScreenKeyboard, TabTip</PackageTags>
    <Authors>Alexei Shcherbakov</Authors>
    <GeneratePackageOnBuild>True</GeneratePackageOnBuild>
    <SignAssembly>True</SignAssembly>
    <DelaySign>False</DelaySign>
    <AssemblyOriginatorKeyFile>key.snk</AssemblyOriginatorKeyFile>
    <Version>1.0.0</Version>
		<AssemblyVersion><IP_ADDRESS></AssemblyVersion>
		<AssemblyFileVersion><IP_ADDRESS></AssemblyFileVersion>
    <Company />
    <PackageProjectUrl>https://github.com/AlexeiScherbakov/osklib</PackageProjectUrl>
    <RepositoryUrl>https://github.com/AlexeiScherbakov/osklib</RepositoryUrl>
  </PropertyGroup>
  <PropertyGroup>
    <Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
    <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
    <ProjectGuid>{9752EAF3-4855-4E2D-889B-961AF3CE1F73}</ProjectGuid>
    <OutputType>Library</OutputType>
    <RootNamespace>Osklib</RootNamespace>
    <AssemblyName>Osklib</AssemblyName>
    <TargetFrameworkVersion>v4.5</TargetFrameworkVersion>
    <FileAlignment>512</FileAlignment>
  </PropertyGroup>
  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
    <PlatformTarget>AnyCPU</PlatformTarget>
    <DebugSymbols>true</DebugSymbols>
    <DebugType>full</DebugType>
    <Optimize>false</Optimize>
    <OutputPath>bin\Debug\</OutputPath>
    <DefineConstants>DEBUG;TRACE</DefineConstants>
    <ErrorReport>prompt</ErrorReport>
    <WarningLevel>4</WarningLevel>
  </PropertyGroup>
  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Release|AnyCPU' ">
    <PlatformTarget>AnyCPU</PlatformTarget>
    <DebugType>pdbonly</DebugType>
    <Optimize>true</Optimize>
    <OutputPath>bin\Release\</OutputPath>
    <DefineConstants>TRACE</DefineConstants>
    <ErrorReport>prompt</ErrorReport>
    <WarningLevel>4</WarningLevel>
  </PropertyGroup>
  <ItemGroup>
    <Reference Include="System" />
    <Reference Include="System.Core" />
    <Reference Include="System.Xml.Linq" />
    <Reference Include="System.Data.DataSetExtensions" />
    <Reference Include="Microsoft.CSharp" />
    <Reference Include="System.Data" />
    <Reference Include="System.Net.Http" />
    <Reference Include="System.Xml" />
  </ItemGroup>
  <ItemGroup>
    <Compile Include="OnScreenKeyboard.cs" />
    <Compile Include="Interop\ITipInvocation.cs" />
    <Compile Include="Interop\NativeMethods.cs" />
  </ItemGroup>
  <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />

</Project>

And the ConsoleTester.exe works for me, which is awesome!

Solution can be:

You are using outdated Build Tools. (Project osklib is in a new VS2017 format)
You don't have .NET Core SDK installed (which contains MSBuild for new format)
I choosed new format because it makes nuget package automatically

Hi @AlexeiScherbakov, thanks for your response! You're right, I was using an older version of VS, I think VS2015. If you don't support that then this can be closed as-is, and if anybody needs to build this on an older version they can use the csproj I pasted above :)

No, I Build Tools 2017 and VS Community 2017 are free to download. NuGet
package is compiled for .NET 4.0+.NET45+.NET461 and .NET 4.0 is able to run
even in Windows XP SP3 :-) - so I think that adding support something older
is bad idea.
On Mon, Sep 18, 2017 at 5:39 PM, Sjors Gielen<EMAIL_ADDRESS>wrote:

Hi @AlexeiScherbakov https://github.com/alexeischerbakov, thanks for
your response! You're right, I was using an older version of VS, I think
VS2015. If you don't support that then this can be closed as-is, and if
anybody needs to build this on an older version they can use the csproj I
pasted above :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlexeiScherbakov/osklib/issues/1#issuecomment-330244223,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIQ7O-V7crozdvJ9UDLsjdDznjmsamXDks5sjoCCgaJpZM4PV3zk
.


[$500] Split  - "Tab Highlight" is not centered in Split bill flow

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

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

Access staging.new.expensify.com or open the Desktop app
Sign into a valid account
Create a group chat
Click on plus icon > split bill
Enter amount to go to confirm page
Click on checkbox of one member
Click on "Tab" and highlight the checkbox

Expected Result:
User expects the checkbox to be centered when highlighted
Actual Result:
The checkbox is completely off center and slightly cuts off the circular checkbox
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

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

Upwork Job URL: https://www.upwork.com/jobs/~01d2bb24f7d4def822
Upwork Job ID:<PHONE_NUMBER>022115840
Last Price Increase: 2024-01-15

Dupe of https://github.com/Expensify/App/issues/32400

Dupe of #32400

@miljakljajic We should put this issue on hold, referring to #32400

No problem - we can close and re-open if it is still an issue once https://github.com/Expensify/App/issues/32400 is resolved.


Strange code in ClaimHolder.sol

Seems like this line should be removed:
https://github.com/OriginProtocol/identity-playground/blob/48d06bb589e22c6e79eac94fe7a6ed5a8df18f20/contracts/ClaimHolder.sol#L24
Or whats its for?

Yup no need for that line... should be removed


Avalara integration

Currently, we support VAT calculations for EU with Vatlayer and manual tax configuration through the dashboard. Our goal, however, is to provide integration with third-party service and delegate all tax calculations there. Avalara seems to be the most promising option.
We have to investigate how we could integrate it with Saleor. Since Avalara is a paid solution, we may want to provide it optionally and leave the current implementation as the default one.

One note: AFAIK Avalara (unlike Vatlayer) does not expose its ruleset. Instead we have to report each transaction as it happens and update the fields according to Avalara's response. Their service tiers are based around the number of transactions served over a time period (whereas for Varlayer we only do one API call a day).

For USA sales taxes, a Saleor-based project I work on now uses a custom dashboard area where sales tax rates are entered manually state-by-state. This has worked for them for awhile, but an integration would obviously remove the manual labor for the staff user.
Maybe something simple and naive like that would be a base feature, and the integration could be optional to automate it?

Bump - we are looking forward to this integration.
ATM we are using tax calculation thru Stripe, but thats not sufficient solution for us.

@krzysztofwolski Could you or someone from your team briefly describe your use-case for Avalara? Just to better understand where in Saleor we would have to plug it. Or is there a single standard workflow for this integration?

ATM our client displays prices with taxes on the checkout page. The whole cart is sent to stripe which calculates tax for all items and then this info is presented to the user.
Problems with this approach:

every cart change triggers an additional call to stripe which means a performance hit
product price displayed in storefront does not include taxes. ATM we only sell in the USA and it's not a problem, but we want to go international and in Europe law enforces us to display prices with taxes included

What I need:

integration with Avalara which will dynamically pull prices with taxes
taxes need to be automatically adjusted to the country of the visitor
caching solution which will prevent us from doing too many calls to the API (afaik API is priced by the number of calls)
built-in solution for cart, which will save tax data in it
tax rates need to be also remembered in order object

What would be nice:

Update google feed to use tax rates: https://github.com/mirumee/saleor/issues/3832

Regarding the law, if we only sell to businesses (like the project we talk about), we could get away with net prices only until checkout but I don't think Saleor can make that assumption.

Note: not sure how this should work in the USA, where each state has different tax laws. Probably in this situation taxes are displayed only during checkout, after providing a shipping address.

Yes, for the US it's expected to only display taxes during checkout.

@krzysztofwolski - By chance, would you be able to share the Stripe / Sales Tax code you mentioned above or maybe suggest the best place in the Saleor codebase to integrate our own similar Stripe API code?  We're working on a U.S.-only project w/ Stripe, and the solution you mentioned above would do the job.  We're still getting up to speed on the architecture.  Thank you -- sorry for the off-topic post.


Support relative time

Support relative times:

in the past , eg "5 days ago";
in the future, eg "in 5 days";

API
Globalize.formatRelativeTime( value[, unit][, options] )

Examples:
.formatRelativeTime( inTwoDays ); // in 2 days
.formatRelativeTime( twoDaysAgo ); // 2 days ago
.formatRelativeTime( 2, "day" ); // in 2 days
.formatRelativeTime( twoDaysAgo, "hour" ); // 48 hours ago
.formatRelativeTime( -48, "hour" ); // 48 hours ago
.formatRelativeTime( twoDaysAgo, "hour", { form: "short" } ); // 48 hr. ago
.formatRelativeTime({ from: christmas2001, to: newyear2002 }, "day" ); // in 7 days

Value: Object with from and to keys and Date instance values, or a Date instance (considered to be the to value and from defaults to now (new Date())), or a Number (positive numbers represent future, negative numbers represent past).
Unit: Optional when value is a Date instance or an object of Date instances. Required when value is a Number. Possible values are (year, quarter, month, week, day, hour, minute, and second), e.g.,  "in 30 days", "in 4 weeks", or "in 1 month", ... Defaults to automatic detection when using Date instances, therefore implementing a simple approximation algorithm [1]. But, delegating any more complex things to a specific date manipulation library such as moment.js.
Options:

form (short, narrow), eg. "in 30 days", "in 4 wk.", or "in 1 mo."...

1:

"1 second ago" .. "59 seconds ago"; (1 .. 59 s)
"1 minute ago" .. "59 minutes ago"; (60 .. 60*60 s)
"1 hour ago" .. "59 hours ago";
"1 day ago" .. "59 days ago";
"1 week ago" .. "59 weeks ago";
"1 month ago" .. "59 months ago";
"1 year ago" .. "59 years ago";

(analogous for future)
CLDR
Relative time fields can be found on /dateFields.json.

http://www.unicode.org/reports/tr35/tr35-dates.html#Calendar_Fields
http://www.unicode.org/reports/tr35/tr35-dates.html#Date_Field_Symbol_Table

Other
References:

twitter-cldr
Rails;
Wikipedia [1] [2];

--
This issue succeeds #205 (discussion to get to the API above). For absolute durations, see .formatUnit() in #252.

On CLDR v26, different widths/forms appeared: short and narrow. So, I've updated the API accordingly.
I've also (a) made the API more similar to .formatUnit(), i.e., making unit an argument and having options.form to modify the default width/form; and (b) allowed value to be a Number, whose sign represents the past/future direction (according to https://github.com/jquery/globalize/pull/389#issuecomment-73994259).

Ecma proposal:

https://github.com/tc39/ecma402/issues/35


Cross-platform support

Right now only Linux is supported. The changes required to also support other *NIXes should be small.

Closing this for now since I only use Linux. Please start a discussion to let me know if you are interested in this.


Problem with SSD1351 . Nothing shown on display

Tried a few examples and cannot see anything on the display. No errors.
I have no problem with i2c displays.
Checked the SPI port has ok
ls -l /dev/spi*
crw-rw---- 1 root spi 153, 0 Apr 12 15:40 /dev/spidev0.0
crw-rw---- 1 root spi 153, 1 Apr 12 15:40 /dev/spidev0.1

The program runs without error, but I see nothing on the display :
:~/luma.examples$ python examples/perfloop.py -f conf/ssd1351.conf --framebuffer=full_frame
Testing display rendering performance
Press Ctrl-C to abort test

Version: luma.oled 2.3.2 (luma.core 1.7.2)
Display: ssd1351
Interface: spi
Dimensions: 128 x 128
------------------------------------------------------------
#### iter =   2945: render time = 43.67 ms, frame rate = 22.87 FPS

Hardware: Tried both Pi 3 Model B and Pi 3 Model B+. Same result
OS: Raspbian Release date:2018-03-13 Kernel version:4.9
Connections:

OLED Pin
OLED Name
RPi Pin
RPi Function
Remarks

1
GND
P01-20
GND
Ground

2
VCC
P01-17
3V3
+3.3V Power

3
SCL
P01-23
GPIO 11 (SCLK)
Serial Clock

4
SDA
P01-19
GPIO 10 (MOSI)
Serial Data

5
RES
P01-18
GPIO 24
Reset

6
DC
P01-22
GPIO 25
Data/Command

7
CS
P01-24
GPIO 8 (CE0)
Chip Select

Here are some photos. I have 2 displays of the same ssd1351 ic. Both yield the same results (nothing shown on display).
Thanks very much in advance!

I haven't tried a Raspberry 3 or the latest Raspian image; it is entirely possible that some SPI implementation detail changed (unlikely though).
Also since you have two boards, it is unlikely (or very unlucky) that both are dead. Looking at the pic, the soldering doesn't look too good on one of them. Also interesting is that one OLED display is smaller than the other - what size was it? Where did you buy them from?
One thing I found in the past though: using cheap breadboards can cause problems because the metal inserts dont grip the male pins properly (see this youtube video about 14mins in).
I always tend to use the female-female jumper cables like these https://www.ebay.co.uk/itm/10cm-Female-to-Female-Jumper-Wire-for-Breadboard-Prototyping-/262403793547

Finally had time to test this out.  I have took your advice and removed the need for breadboard, directly jumping with cable.
It went one step further, running below:
$ sudo modprobe fbtft_device name=freetronicsoled128
$ dmesg | grep fbtft_device
[   63.900658] fbtft_device: module is from the staging directory, the quality is unknown, you have been warned.
[   63.903323] fbtft_device: GPIOS used by 'freetronicsoled128':
[   63.903337] fbtft_device: 'reset' = GPIO24
[   63.903345] fbtft_device: 'dc' = GPIO25
$ con2fbmap 1 1
then do a "sudo reboot" , I managed to see something on screen, I guess that confirmed wiring is ok.

However, when I run any luma.examples, I still get a blank display without any error.
eg.
pi@raspberrypi:~/luma.examples $ python examples/maze.py -f conf/ssd1351.conf --framebuffer=full_frame
Version: luma.oled 2.4.0 (luma.core 1.7.2)
Display: ssd1351
Interface: spi
Dimensions: 128 x 128
pi@raspberrypi:~/luma.examples $ python examples/welcome.py -f conf/ssd1351.conf --framebuffer=full_frame
Version: luma.oled 2.4.0 (luma.core 1.7.2)
Display: ssd1351
Interface: spi
Dimensions: 128 x 128
Result: Nothing on display

Ok, what is interesting is that when you have it working with the kernel driver, you have pin 24 connected to reset, and pin 25 connected to the data/command line.
[   63.903337] fbtft_device: 'reset' = GPIO24
[   63.903345] fbtft_device: 'dc' = GPIO25

However, if you follow the instructions at https://luma-oled.readthedocs.io/en/latest/hardware.html#spi you will see that the default pins for these two lines are the other way around.
Therefore I would suggest either:

You connect the pins as suggested in the luma.oled documentation
You maintain your existing pin connections and supply --gpio-data-command=25 and --gpio-reset=24 to override the defaults when running the demos.

Either should work.

It's working!! Thanks so much for this brilliant library.
python examples/animated_gif.py -f conf/ssd1351.conf --framebuffer=full_frame --gpio-data-command=25 --gpio-reset=24 --framebuffer=full_frame

For me it's not working unfortunately.
I'm using following module:

It works fine with the sample on https://www.waveshare.com/wiki/1.5inch_RGB_OLED_Module
I figured that I connected GPIO-24 for DC and GPIO-25 for RST. Moreover the device is connected to CE0 which is supposedly spi-device 0. I'm using RaspberryPi 4 B and
Linux raspberrypi4 4.19.75-v7l+ #1270 SMP Tue Sep 24 18:51:41 BST 2019 armv7l GNU/Linux
With the luma example the display stays black.
python examples/demo.py -f conf/ssd1351.conf --framebuffer full_frame --gpio-data-command 24 --gpio-reset 25 --spi-port 0 --spi-device 0
Any ideas why luma example does not work although waveshare example does?


Backport2.16 hkdf test stack usage reduction

Description
Backport of #2192 to mbedtls-2.16
Status
READY

@hanno-arm could you review the last commits since your previous review? or @AndrzejKurek could you review it? (Please coordinate to avoid duplicated work!) This second review is the last thing holding back this backport and its parent PR. Thanks!

Approved and passing all CI, so labelling as 'ready for merge'.


Clustering

At the time of writing this the framework is single threaded. For version 0.1 it should atleast take advantage of cpu core on a single machine.
For version 0.2 (?) it should also be easy to setup clusting, where multiple application can talk to one another over a network.

Closing in favour of #188.


Update CI for python range and add flake8 validation

This should fail initially due to validation errors.

This will now require #16 to pass.


[Enhancement] Release the alter permission on the range hive service …

…to refresh the external table
Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.yangjunnan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Multiple errors with typeof

I'm trying to upgrade my project to the latest version of your hashmap (previously using version 2016-2018) , but after rewriting all instances to include new syntax, I'm running into multiple errors at compile time that basically seem to reduce to:
error: ‘typeof’ was not declared in this scope
Searching the internet seems to indicate that this is a GNU extension to C that doesn't work well with C++.  This is a mixed C/C++ project.  Is there anything I can do?

I was able to finally get all this resolved.  I don't know why gcc kept complaining about this, but it seems to be in the way I had declared the HASHMAP typedef with a pointer to char rather than char.  I think it is just an artifact of multiple nested macros making resolution of the actual problem very difficult.  It has taken me about a week of rewriting (we use hashmap a lot) but it all compiles cleanly and seems to work just fine.  Still lots of tests before I can sign off on the changes, but it looks OK.


Define and implement the process for tagging the docker image for release management

We have decided to have just one image: the last one! We don't have releases in our SLDC


Performance Test

Did someone has been able to perform a performance test of node media with high load?
1,000 concurrent conections or more
@illuspas @hthetiot @snowron @KM8Oz @nsthorat
Any feed back is apreciated
Regards

I will inform as soon as possible
On Mon, Feb 22, 2021, 12:29 AM asterisk360-admin<EMAIL_ADDRESS>wrote:

Did someone has been able to perform a performance test of node media with
high load?
1,000 concurrent conections or more
@illuspas https://github.com/illuspas @hthetiot
https://github.com/hthetiot @snowron https://github.com/snowron @KM8Oz
https://github.com/KM8Oz @nsthorat https://github.com/nsthorat
Any feed back is apreciated
Regards
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/illuspas/Node-Media-Server/issues/432, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKPIGZ5W634LZWZ344API3TAFUKHANCNFSM4X7KUBWQ
.

Hello any update?

@KM8Oz any update?

none brother we closed the website after a while. but as I told you last
time it's working fine for 100 stream max in a 20 cores server. in four
qualities 360/480/720 HLS.
On Thu, Oct 13, 2022, 22:39 gigabyteservice @.***>
wrote:

@KM8Oz https://github.com/KM8Oz any update?
—
Reply to this email directly, view it on GitHub
https://github.com/illuspas/Node-Media-Server/issues/432#issuecomment-1278098116,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKPIG36V6YWDLJ5K65KUDLWDBQQ5ANCNFSM4X7KUBWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

I tested with 1004 connections with 32 GB vCPU . No load . But there is an issue
https://github.com/illuspas/Node-Media-Server/issues/435 coming


[BUG] Deprovision doesn't remove the error field if it exists

What is the bug?
If provisioning has a failure on one of the steps we write to the error field in the state index. However when we deprovision that same workflow we don't remove the error message from the document in the workflow state index.
How can one reproduce the bug?

Provision with an expected failure (easy way is to provision a pipeline with a processor that isn't in your cluster)
2.Deprovision the same workflow:

resulting state index:
            {
                "_index": ".plugins-flow-framework-state",
                "_id": "QljrPY4BJDcxHyOJPUNd",
                "_score": 1.0,
                "_source": {
                    "workflow_id": "QljrPY4BJDcxHyOJPUNd",
                    "state": "NOT_STARTED",
                    "provisioning_progress": "NOT_STARTED",
                    "provision_start_time":<PHONE_NUMBER>112,
                    "resources_created": [],
                    "provision_end_time":<PHONE_NUMBER>966,
                    "error": "org.opensearch.flowframework.exception.WorkflowStepException during step create_ingest_pipeline, restStatus: BAD_REQUEST"
                }
            }

What is the expected behavior?
error field shouldn't be there anymore
Potential solution
Some potential solution

We can either simply place the error field as an empty string, however this isn't prefered because if the error field wasn't there to begin with now we have the field with just "error": "".
You can't directly remove the field through update, this has to be done through a script as far as I know. You also can't provide both a contentMap(doc) and a script in the same update request. This means we either have to just update the logic here https://github.com/opensearch-project/flow-framework/blob/main/src/main/java/org/opensearch/flowframework/transport/DeprovisionWorkflowTransportAction.java#L230 to update with a script that removes the error field if it exists (e.g "if (ctx._source.contains(error)) { ctx._source.remove('error)}") and updates the other fields accordingly or we have to make two update operations, one with the current way we do it with a new doc and another with the painless script

It's not a bug, it's a feature!  😁
Seriously, though, this is very interesting.  I'm not overly concerned a provisioning error hangs around after deprovisioning the failure (although should we re-populate it with deprovisioning failures, that I don't think we do?). More importantly if I provision a second time does the error remain even if provisioning was successful?

While implementing #631 I couldn't use the field-by-field update or script, so I added some helper methods to WorkflowState that would enable this:

Create a builder using the existing state
update the fields as necessary (e.g., set error to null)
just index the new document to overwrite the old one, instead of using update.

See https://github.com/opensearch-project/flow-framework/pull/631/files#diff-d49816ce76f874e242a545dd3de0515c2435ec7a6a315faf83ae28e4069f08e0, some of those changes should probably be kept.


Unknown contacts warning: Workflow, persistency

Hi,
official Apps present a notification bar with "Unknown contact" warning upon receiving a new chat with an unknown person. 3 choices can be made: Add to address book, report and do nothing (by pressing X). After any of these actions the warning will never appear again.
I assume this is a client-side marker that will be set, but how is this persisted across devices? Otherwise if this is a local option it would appear on any client that has not seen this chat. So it must be stored server-side?

It's called chatActionBar. You need to process the field chat.action_bar and the update updateChatActionBar. Everything other is done by TDLib, including action bar synchronization with other apps.

Oh wow that is convenient! Thanks!

For completeness, to close the chat action bar without any action, call removeChatActionBar function.

@levlam if the action, like in official Apps, would be "report and leave", we would in fact call 2 functions, one is to report spam, and 2nd is to leave the chat right?

we would in fact call 2 functions, one is to report spam, and 2nd is to leave the chat right?

Yes, reportChat and leaveChat.

Oh it seems I have to call removeChatActionBar for any action, I did an addContact now but the bar remained :)

Action bar is automatically removed by TDLib on any relevant action.


grammar revision

For the sake of improving clarity, I removed a word in that was probably written by accident.

thanks! for future reference, the php files are generated from the src here: https://github.com/processing/p5.js-website/tree/master/examples/examples_src. so you can make the changes to these files directly.


Tooling improvements

I have started to improve the basic tooling by replacing homegrown helper files with commonly used patterns. There's also some preparation for the introduction of a proper setup.py, which will need to happen anyway sometime in the feature to enable users to install InvenTree via package managers like pip.

Note that this will of course not yet result in a passing build; I created #20 to solve that.

@zakx so this should be merged before #20 ?
What advantage does flake8 offer over the pep8 utility I was using? I am fine to change it, just wondering what the rationale is.

Yeah, this should be merged first.
flakes8 ist basically a merger of pep8 (style checking only) and pyflakes (error checking only). See https://dancallahan.info/journal/python-flake8/ for more information and examples.
In #20 I have also removed (not fixed) two errors that a simple style check would not have found.

Ok, I am happy with this, it looks a lot more robust and forward-thinking. Thanks :)


Prevents apps from stubbing router service in unit tests

This addon eagerly instantiates its service in an instance-initializer:
https://github.com/DockYard/ember-router-scroll/blob/2f17728fd82d7d486888df9a09fdc4aae233f294/addon/instance-initializers/ember-router-scroll.js#L3
Which also eagerly instantiates the router service:
https://github.com/DockYard/ember-router-scroll/blob/2f17728fd82d7d486888df9a09fdc4aae233f294/addon/services/router-scroll.js#L72
But that means it's impossible to write a unit test that stubs the router service, because setupContext from @ember/test-hepers (which is what backs setupTest from 'ember-qunit') will run this initializer before users have a chance to register any overrides.
So if people have unit tests against a stubbed router service, and then they add this addon, those tests break.

One thing that you can do to go around this problem is to unregister the router service, before registering your mock
this.owner.unregister('service:router');
this.owner.register('service:router', class extends Service {});


"n lts" installs v18. suggest "n 16" or similar instead

What article is affected?
https://github.com/kadena-community/kadena.js#installing-prerequisites
What part(s) of the article would you like to see updated?
Requirements
node > LTS (node 16, as 18 isn't yet LTS)
install node
n lts // installs 18
suggest "n 16" instead
Additional information
No response

Added in https://github.com/kadena-community/kadena.js/commit/f75e8291c345913d907ee366f9f08697dff8acbc#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

Node 18 is LTS since October '22, 16 is Maintenance since then: https://github.com/nodejs/Release


Why is Searcher loading a second Checkpoint, i.e. the same model again?

When following the basic examples I noticed that the Searcher class when called through RAG.search will always load the pytorch model used in the index (e.g. colbert-ir/colbertv2.0) a second time, by calling self.checkpoint = Checkpoint(self.checkpoint, colbert_config=self.config, verbose=self.verbose). Is there a specific reason for doing this? Couldn't it just use the Checkpoint that was previously loaded when the index was loaded, to save on the extra loading time and avoid double the memory use?
Steps to reproduce, based on the example here
# will load the model `colbert-ir/colbertv2.0` the first time
RAG = RAGPretrainedModel.from_index(".ragatouille/colbert/indexes/Miyazaki/")

# will load the model a second time
RAG.search(query="What animation studio did Miyazaki found?")

This is a good point.
The issue is a bit convoluted, in that it can't currently re-use the already loaded checkpoint, due to the fact that upstream colbert-ai won't let you pass an already initiated checkpoint.
Our options would be to make changes upstream (but that'll take a bit more time), or to not load a checkpoint on RAGPretrainedModel.from_index() calls, but only do so when a relevant function is called (e.g. index in memory, or Searcher()). I think that option could make sense, it's probably fairly uncommon usage to load a model from an index to then use for purposes otehr than querying the index.

Our options would be to make changes upstream (but that'll take a bit more time), or to not load a checkpoint on RAGPretrainedModel.from_index() calls, but only do so when a relevant function is called (e.g. index in memory, or Searcher()). I think that option could make sense, it's probably fairly uncommon usage to load a model from an index to then use for purposes otehr than querying the index.

Wouldn't the latter break if someone loaded the model and then started adding/deleting entries without performing search first?


Sync broken: "Not Found","documentation_url":"https://developer.github.com/v3/gists/#edit-a-gist"

Describe the bug
Sync is broken, "Not Found","documentation_url":"https://developer.github.com/v3/gists/#edit-a-gist" in Developer Console (see screenshot below). "sync.gist" ID is correct. https://developer.github.com/v3/gists/#edit-a-gist opens OK in browser.
Visual Studio Code Version : 1.24.1
Code Settings Sync Version : 2.9.2
Operating System : macOS 10.13.5
Occurs On: Upload
Proxy Enabled: NO
Gist Id: b6e46411753ca3222c0f553b02042923
To Reproduce
Steps to reproduce the behavior:

Open command palette.
Execute Sync: Update/Upload settings.

Expected behaviour
Sync performed correctly.
Console Error Log

Additional context
None.

Duplicate of #177

see the solution there.


Where are the debug symbols and nuget packages?

I can't believe you expect us to work like this.

Here they are:

https://www.nuget.org/packages/System.Ben/1.0.0


⚠ Bofrost.be has degraded performance

In 47cfdaf, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2469 ms

Resolved: Bofrost.be performance has improved in fd8e87e.


Az login issue

I am trying to login with my azure account (on lab portal while using vs code) to use the azure storage resources, but I am unable to do so because of the following error.
browser window opens and shows an error saying login.microsoftonline.com sent an invalid response. Unable to continue the exercise without it. When tried using device code it shows SSL protocol error on the browser. Please suggest!
### Tasks
- [ ] https://github.com/microsoft/azcode/issues/67
- [ ] https://github.com/microsoft/azcode/issues/67

Hi @Sidzeppelin95, could you please provide us with more information about your situation? What do you mean by "on lab portal while using vs code"?

Hi Alex,
By lab portal I meant the one with vs code training access provided by
Microsoft for cloud skills challenge.
The same error occurred while doing the same on azure cloudshell.
Thanks
Siddhartha Sharma
On Wed, Jan 10, 2024, 02:17 Alex Weininger @.***> wrote:

Hi @Sidzeppelin95 https://github.com/Sidzeppelin95, could you please
provide us with more information about your situation? What do you mean by
"on lab portal while using vs code"?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azcode/issues/68#issuecomment-1883768309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMEY74WISTYT5Y56AWWTYKLYNWUFPAVCNFSM6AAAAABBM3P6SWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTG43DQMZQHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


[CodeCompletion] Missing init completions for dotExpr

Fixes non-visible inherited convenience initializers
Fixes non-visible initializers from protocol extensions
Fixes visible initializers on dynamic metatypes for postfix and parenPostfix expressions
Resolves SR-2206, SR-7256, SR-7789, the last being a bug found while fixing the other 2.
Note
The following logic causes global completion results (all of the literals, stdlib decls etc.) to show when there aren't any function completions found for ParenPostfix. This includes cases when nothing is found, some of which are directly related to SR-7789 – before, they showed initializers, now, being fixed, they show nothing (empty completion results) and thus fall under the case when global lookup is done and shown.
https://github.com/apple/swift/blob/ba9cc174e457704c1d9e959a0bac3cb28b9fd9d8/lib/IDE/CodeCompletion.cpp#L5404-L5407
Example:
class A { init(foo: Int) {}}
func test() {
  var B = A.self
  B(#^HERE^# // will show global completion results
}

IMO this is incorrect and there shouldn't be any completions at all if the sink is empty to begin with, but nevertheless, is this is supposed to be correct?
UDP
SR-7789 is fixed, however. I think this should be moved to a separate bug report if needed. I recently found out it's present in some cases which aren't related to SR-7789. In particular, parenPostfix initializer completions for some generic types.

\cc @benlangmuir

Roughly, I've fixed all of the following cases concerning SR-7789 and cleaned up the code.
class Foo {
  init() {}
  init(a : Int) {}
  init(foo: String) {
    self.#^HERE^# // shows initializers
  }
  class func foo() {
    self.#^HERE^# // doesn't show initializers
  }
}
class Derived: Foo {
  class func foo() {
    super.#^HERE^# // doesn't show initializers
  }
  convenience init() {
    super.#^HERE^# // shows initializers
  }
}

func foo() {
  var MT= Foo.self
  MT.#^HERE^# // no initializers (OK)
  MT(#^HERE^# // shows initializers
  MT#^HERE^#  // shows initializers
}
protocol P {
  init(bar: Int)
}
func test<S: P>(_ arg: S) {
  S.#^HERE^# // doesn't show initializers
}

But there's a problem: I can't make the
  class func foo() {
    super.#^HERE^# // shows super init completions
  }

test to pass in complete_after_super.swift. At the same time, super#^HERE^# passes fine.
Of course, all of them work as expected when I build and run swift-ide-test. I suspect the reason is the language version used to run the tests in this file, but I do not know how to find it out. I've tried using different versions with -swift-version on swift-ide-test, but there is no difference.
At this point I though you might have an idea of what's going on.

Of course, all of them work as expected when I build and run swift-ide-test. I suspect the reason is the language version used to run the tests in this file, but I do not know how to find it out. I've tried using different versions with -swift-version on swift-ide-test, but there is no difference.
At this point I though you might have an idea of what's going on.

When the test fails it should print out the exact command line that it runs for every RUN line, so you should be able to use that to reproduce.

Great...It didn't work because the two checks were in the same function.

Just in case, ready to proceed :)

@swift-ci please smoke test

Yup, thanks for the reminder!


Cannot test Spring Cloud Stream Batch Mode using the test binder and Spring Cloud Contract

Hi,
In Summary Spring Cloud Contract and spring cloud stream test binder in batch mode are not working quite right.  I have a boot 2.6.8 Cloud 2021.0.3 project https://github.com/davidmelia/spring-boot-webflux-avro-sink-only-batch which shows this issue in isolation:
I have two consumers:

StaticInstrumentEventBatchConsumer - consumes batch events
StaticInstrumentEventConsumer - consumes a single event

and two corresponding spring cloud stub runner integration tests:

FxRateBatchIntegrationFailingTest - fails because the payload is a Message<byte[]>  instead of a Message<List<AvroFxRateEvent>>
FxRateNoneBatchIntegrationTest - works fine

FxRateBatchIntegrationFailingTest shows the problem.  When running in batch mode the event is a Message<byte[]>  instead of a Message<List<AvroFxRateEvent>>.  In the logs you see
Processing Batch: GenericMessage [payload=byte[39], headers={
where the payload is a byte array which is incorrect
To prove this is only a test problem you can create a kafka 'test' topic and use the KafkaTopicPopulator to populate, start the app and you will see batch messages consumed with the correct payload in the log:
Processing Batch: GenericMessage [payload=[{"from": "GBP", "to": "USD", "rate": 0.109631}, {"from": "GBP", "to": "USD", "rate": 0.109631} ...
where the payload is correctly an array of AvroFxRateEvent objects.
Thanks
N.B this is actually the same issue https://github.com/spring-cloud/spring-cloud-stream/issues/1925 which didn't progress although no reason was given.

I would suggest not to use the test binder but go with testcontainers and an actual binder like we do it e.g. here https://github.com/spring-cloud-samples/spring-cloud-contract-samples/tree/3.1.x/producer_kafka_middleware

I'm running into the same issue. is there a way batch mode can be supported in test binders?

The short answer is no and we have no plans to do so. Here is why. . .
Batching often supplemented by the underlying functionality of the messaging system. For example, RabbitMQ prefetch. Similar things exist for Kafka etc. . .
And in fact if you look at extended binding properties just between Kafka and Rabbit you'll see quite a few differences which are all based on supported functionality.
So, basically there is no generic case that could be reliably implemented to emulate this or that system.
Further more, the TestBinder was initially created for internal use to primarily support internal testing of the core binder API and to replace the old MessageCollector that did not do that. We quickly realised that it could be used for  majority of the unit testing based on core functionality, so users can benefit from it as well. However TestBinder was never meant to replace or replicate a real binder tied to a particular messaging system. So this is where integration testing should really be done and between Docker images and embedded messaging systems (i.e., Kafka) I don't see a reason fort test binder to attempt to reproduce (re-implement) some of the functionality specific to a particular messaging system. Also how would we manage the subtle differences in such functionality between let's say Kafka and Rabbit? And what about binders we don't know about?


Building circuit breaker config for a service.

Use the paramters passed in to beehive.service/service to build the
circuit breaker.

Looks good.


Multitenancy and public DIDs

Hi all,
as I'm looking into acapy multitenancy a bit, I had some trouble registering public DIDs for sub-wallets.
It looks like I'm not the only one as there have been some issues here:
https://github.com/hyperledger/aries-cloudagent-python/issues/1460
and there
https://github.com/hyperledger/aries-cloudagent-python/issues/1358
I'm trying to summarize what I've learned so far:
Most important to understand:

/wallet/did/create and /wallet/did/public endpoints are NOT enough.
You need to use /ledger/register-nym IN BETWEEN those endpoints!

So far so good. The issue is, that:
Using this endpoint is not possible when using multitenancy. I think there are Options here:

Having a separate agent running that does the endorsement for you
Having a sub-wallet with endorser rights doing this for you
Maybe let the base-wallet do this for you

Currently I only see option 1.
Option 2 does not work because I would need to be able to pre-seed / fill the DID I create in a sub-wallet - which seems not possible right now.
/wallet/did/create does not allow to add a seed or existing (on the ledger) DID
/wallet/did/public does not allow to add seed, so I can NOT add an existing (on the ledger) DID
Option 3 would have been my first guess, but with multitenancy enabled, it requires authentication, and this I get only for a sub-wallet.
Can someone confirm my observations or give some hints where I might have some misunderstandings?
Thanks a lot!
Matthias

Last time I checked all the base wallet could do was create sub wallets. So I would say yes you need another agent, sub wallet or standalone, that does the endorsement for you.
As a remark there is also an open PR that endorses an ATTRIB transaction to set the endpoint on the ledger: https://github.com/hyperledger/aries-cloudagent-python/pull/1616 For holder wallets this is the best option imho.
In the end it depends on where to place the orchestration of the api calls. If I remember correctly the consent was to keep such logic in the controller (not aca-py). But what seems odd right now is that one part of the wallet initialisation logic lies in the sub wallet and one part in the base wallet. This means that there is a difference on what a freshly initialised sub- wallet controller must do vs a standalone controller. So maybe it would make sense to have the option to provide a did/seed when calling POST /multitenancy/wallet and as a second step have the option in the base wallet to write a NYM or ATTRIB transaction to the ledger (either directly or via an endorser). Like this all api calls would be in one place.

Thanks @etschelp
I've now pushed a simple proposal for Option 3. I don't oversee all possible implications here and I hope someone could have a look into this.
I personally would prefer to have just 1 instance running.

#1 is the recommended option.  With PR #1616 (should be completed soon) the sub-wallet can self-register their new DID via an endorser.  Right now it's recommended that the endorser be a separate aca-py instance (not another sub-wallet and not the base wallet) because it will have different startup params (you can start up an aca-py instance configured as an author or as an endorser) - in a multitenant scenario the aca-py startup params apply to all sub-wallets, there is no way to configure one sub-wallet differently from the others.
(Note - you can start up aca-py without any of the endorser parameters, but then author sub-wallets need to call all the aca-py endpoints with all the endorser parameters explicitly provided.)
So, as above, #2 is possible but will require some extra care and attention for the author in calling the aca-py endpoints
For #3, this relies on the controller calling the base wallet, so I'm not sure this would count as a "self serve" model (whereas connecting to an endorser would allow the author to manage their own ledger updates, and would be required for the schema and cred def updates anyways)


sortable for nearly anything

https://github.com/RubaXa/Sortable

That library looks awesome! :+1:

done with https://github.com/timelyportfolio/sortableR .  one of my favorites


can this obtain the  program id （pid） of the packet?

just like title, I want to which pid  the packet  belong

Can you share a .pcap file and maybe Wireshark screenshot that shows the pid?

it seem can not find a pcap contain pid..
do  You know how to mark the application so that I can follow the tag to know which application it is  ( like setting we can set  DSCP on windows qos)

I guess you could see what TCP connections each process has open using Windows API and then see what packets match these connections.


Allow custom getTags and getWrapperTag

This feature allows you to specify custom methods for getTags and getWrapperTag
Why not use blockRenderer?
This would work but I want to take advantage of renderBlockContent. Specifically its style and attribute processing.

Hi @jasonenglish. Thanks for your contribution! Sorry for the delay in reviewing this. I do not have any problem with this approach, however I like the approach mentioned in #37 better. Would it suit your use case? If not, I'm OK to land this one, just a few small cleanup issues: (1) the unchanged block that is marked as changed you noticed. (2) some naming his.blockTags = options.blockTags || getTags; has three names for essentially the same function. I'd call it getBlockTags to be clear.


GEODE-1617: Regions can be created with a variety of characters that …

…are unsupported
Added comprehensive test to validate region names

+1

I reworded your commit message:
GEODE-1617: add test for region names

Added comprehensive test to validate region names

This closes #285

Pushed to develop. Thanks!


Use getattr to load config settings so we can easily set a default

Hey demonrik! I saw this script and liked the idea, so I thought I'd do a few PRs and try to help out a bit.
This change makes renamedir, renamefile and forceupdate optional in the config and defaults them to False if not present.

Logic is a bit wonky. It is doing False, even if set. I'll update this.

Hi Natewalck
Just noticed this now - was on the road last week and didn't get any notification. Sorry for the delay :(
Great to see some more input.. I'll keep an eye on the project more now :)
TIA

@demonrik No worries! I realized I had to do some more refactoring for this to work, so I abandoned this specific PR. I will try to circle back around, but I'm not sure when exactly.


Problems developing on Windows

Hi!
First of all - congrats on an awesome project!
I wanted to contribute a plugin to support results from Everthing by voidtools (a great file indexer on Windows) and wrote a plugin https://github.com/reflog/cerebro-plugin-everything
But I cannot test it well, since the instructions are very Mac oriented and don't work in Windows land.
Opening dev tools doesn't work, since they are shown like this:

And there is no way (that I could find) to see console output from my plugin.
What's your recommended way of debugging on Windows?
Also - how can I mark my plugin as 'windows only' so that other platform users would avoid it?

Hey, to open dev tools properly, open the menu on the upper left and pop them out as a separate window.

Tried that, unable to click any menu :(
On Sat, Jan 28, 2017 at 4:47 PM Kageetai<EMAIL_ADDRESS>wrote:

Hey, to open dev tools properly, open the menu on the upper left and pop
them out as a separate window.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KELiON/cerebro/issues/45#issuecomment-275852213, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGtNJQyCgOnQxM09emUJNTgks8viMnPks5rW1UbgaJpZM4Lv66G
.

Oh sorry I meant the upper-right and the men with the three dots there

I know, but on Windows, this button (...) is non-clickable.
On Sat, Jan 28, 2017 at 5:54 PM Kageetai<EMAIL_ADDRESS>wrote:

Oh sorry I meant the upper-right and the men with the three dots there
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KELiON/cerebro/issues/45#issuecomment-275855918, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGtNLr11ujFxCpCnkghCfnbEA60EGtVks5rW2S6gaJpZM4Lv66G
.

@reflog yep, looks like it is a bug. To open dev-tools you can try to type something (to see some results in cerebro window), and open dev tools after that. In this window is bigger and you can resize it, so you can see dev tools (and try to detach dev tools windows from cerebro windows by clicking ... button).
Also, would be cool to fix it for other users.  There is an option for openDevTools in Cerebro and we should always open it in detached window: openDevTools('detach'). There are only two calls for that function: one and two. Would be awesome if you or someone else can fix it in PR

@KELiON - thanks, that worked! Managed to undock the devtools and now able to see the problems.
To communicate with Everything, my plugin is using node-ffi, which is compiled natively and has problems starting under Electron. Here's the log:
Plugin: cerebro-plugin-everything
Path: C:\Users\reflog\AppData\Roaming\Cerebro\plugins\node_modules\cerebro-plugin-everything...
Error loading
Error: Could not locate the bindings file. Tried:
 → d:\dev\cerebro-plugin-everything\build\binding.node
 → d:\dev\cerebro-plugin-everything\build\Debug\binding.node
 → d:\dev\cerebro-plugin-everything\build\Release\binding.node
 → d:\dev\cerebro-plugin-everything\out\Debug\binding.node
 → d:\dev\cerebro-plugin-everything\Debug\binding.node
 → d:\dev\cerebro-plugin-everything\out\Release\binding.node
 → d:\dev\cerebro-plugin-everything\Release\binding.node
 → d:\dev\cerebro-plugin-everything\build\default\binding.node
 → d:\dev\cerebro-plugin-everything\compiled\6.5.0\win32\x64\binding.node
    at r (d:\dev\cerebro-plugin-everything\dist\index.js:1:16695)
    at Object.<anonymous> (d:\dev\cerebro-plugin-everything\dist\index.js:1:783)
    at r (d:\dev\cerebro-plugin-everything\dist\index.js:1:194)
    at Object.<anonymous> (d:\dev\cerebro-plugin-everything\dist\index.js:1:26444)
    at r (d:\dev\cerebro-plugin-everything\dist\index.js:1:194)
    at Object.<anonymous> (d:\dev\cerebro-plugin-everything\dist\index.js:1:36093)
    at r (d:\dev\cerebro-plugin-everything\dist\index.js:1:194)
    at d:\dev\cerebro-plugin-everything\dist\index.js:1:570
    at Object.<anonymous> (d:\dev\cerebro-plugin-everything\dist\index.js:1:580)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)

Do you have any tips on how to package my module correctly so it would work under Cerebro? This is my first attempt at Electron plugin, so it's a new territory for me...
Thanks in advance!

@reflog can you check that this package installed and works? You can just run node from d:\dev\cerebro-plugin-everything\ and try to require it.

When I run the code with my local node it runs fine, the problem is with
electron's node runtime being different from my local one.
On Mon, Jan 30, 2017, 14:12 Alexandr Subbotin<EMAIL_ADDRESS>wrote:

@reflog https://github.com/reflog can you check that this package
installed and works? You can just run node from
d:\dev\cerebro-plugin-everything\ and try to require it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/KELiON/cerebro/issues/45#issuecomment-276047908, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGtNI7bQynRTSg1NZA8VMq_Jb0h_VVjks5rXdOagaJpZM4Lv66G
.

@reflog ok, another question: can you check that your local node/npm version are the same, that current electron version uses?
Also, you can try to rebuild your native extensions for electron's node/npm by command:
npm rebuild --runtime=electron --target=1.4.12 --disturl=https://atom.io/download/electron --abi=50

Yea, that's the path I'm on right now, tried doing the node-gyp rebuild, but after it's done the bindings are at:
c:\Users\reflog\AppData\Roaming\Cerebro\plugins\node_modules\cerebro-plugin-everything\node_modules\ffi\build\Release\ffi_bindings.node
And as you can see from the log I posted before, it's not where Electron is looking for them :(
Any ideas?

@reflog hm, no idea. Maybe this can help?

@reflog did you have any success with your plugin? Do you need any help?

Well, I've tried following all the steps in the guide that you've referenced with no luck, same result, cannot find the bindings.
If someone more experienced than me in the world of electron-win32-bindings would like to take a stab at it - that would be awesome, the repo is there and waiting :)

I believe this problem has already been solved with the new version. @dubisdev


Stop opening indexes to only check if they exists

We must not open indexes to check if an index exists but rather use a new method to do that without loading the index in memory.
https://github.com/meilisearch/meilisearch/blob/05cc2d1fac7554b8210086ccccd524010511dc00/meilisearch/src/routes/indexes/documents.rs#L306-L308

@Kerollmops
I used the existing function from index_mapper which is
https://github.com/meilisearch/meilisearch/blob/main/index-scheduler/src/index_mapper/mod.rs#L483
which is in turn using the get function from databases.
https://github.com/meilisearch/heed/blob/main/heed/src/database.rs#L342
I see that get function from database.rs will be decoding the content which we don't need for this use case.
I think it would be more easier and faster if we can create index_exists function in the database.rs which skips the decoding part.
If it isn't necessary my PR should suffice for now.
Thanks,
Bala.


Add Date type

Could you add a date type, for when we want dates without times?

Is this really useful? You can specify a custom date format and only parse the date components, then only use the date component of the date time

Is this really useful? Y ou can specify a custom date format and only parse the date components, then only use the date component of the date time

It's a small benefit.  Mostly semantic I guess.  But date != datetime, I like to be precise.  :)

More specifically:  datetime.date is a different type from datetime.datetime, and they are not interchangeable (e.g., some_date < some_datetime does not work).
If you have code that uses dates, and you want a user option that provides a date, with click.DateTime you need to do shenanigans somewhere to cast the datetime values into date, and the typing of your command methods is inaccurate (because the method signature needs to say datetime when what it really should get is date), etc.


Fix output for helm installation

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

@justindavies
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


[AVM Module Issue]: AVM Host Pool Schedules

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
Module Name
avm/res/desktop-virtualization/scaling-plan
(Optional) Module Version
0.1.1
Description
The AVM for Host pools does not support creating schedules for personal Host Pools.
Currently the modules are using the "2022-09-09" version of the API. This API version does not support creating scaling plan for personal host pools. Seeing as the rest of the AVD Azure Verified Modules support creating personal host pools, the scaling plans should do the same.
The first non preview API that supports the personal scaling plan schedules is "2023-09-05", breaking the personal host pool schedule type into its own API of Microsoft.DesktopVirtualization scalingPlans/personalSchedules
(Optional) Correlation Id
No response

@MikeHirtWB we are checking the issue and will get back to you.

Feature request been addressed by PR 2347 https://github.com/Azure/bicep-registry-modules/pull/2347/files

Pushing additional updates to separate APi calls of scaling plan creation from schedules definitions, this will account for Personal and pooled host pool type scenarios.


Remove unnecessary semi-colon from BrotliDecompressor#State enum

Motivation:
We don't need a semi-colon at the end of enum.
Modification:
Removed unnecessary semi-colon
Result:
Clean code

@chrisvest Can you take a look at failed build? Looks like 'Buffer` API error.
https://github.com/netty/netty/runs/6353499276?check_suite_focus=true


[FEATURE REQ] Track telemetry of requests make from Spring Lib through Azure Identity

Context
Azure Identity now supports customize UserAgent since 1.7.1
see

#32095
#32071

With this new feature, Spring Cloud Azure should measure adoption from an authentication perspective.
Goal
Spring Cloud Azure libraries using azure identity should have it s own UserAgent customized following on the Spring Cloud Azure Guidelines
And answer this question:

how many customers are using Spring Cloud Azure to perform authentication through Azure Identity.
Percentage of spring over java sdk.

Status
currently blocked by #32356

azure-identity:1.7.1  has released on 2022-11-18.

Goal:
Track  telemtry of requests make from Spring Lib through Azure Identity.
Sub Goal:

Azure Identity now supports customize UserAgent since 1.7.1, all spring cloud azure libraries using azure idenityt should have it s own UserAgent customized following on the Spring Cloud Azure Guidelines
Collect this kind of data and show it in our own BI system.

@saragluna please plan this as the highest priority in our next sprint, and this targets to Spring Cloud Azure not passwordless only.

Blocked by https://github.com/Azure/azure-sdk-for-java/issues/32356.

PoC https://github.com/Azure/azure-sdk-for-java/pull/33212


fixed: arguments in docs

the readme seems to use an outdated argument this just updates it

Thanks for catching this!


Do not delete

Includes #193; comments out line deleting Singularity requests.
This is not at all pretty; but I would like to get server back up-and-running ASAP, and people are relying on Sous in production now, so I want to avoid the possibility of catastrophe from this source.
For deleting things, we need to alert the owner of the deleted manifest that they have an unowned request that needs cleaning up, or else they need to add their manifest back.

cc @wilwade

Travis seems to have hung; but there's a failing test so looking at that now.

Current coverage is 59.28% (diff: 88.23%)

Merging #194 into master will increase coverage by 0.04%

@@             master       #194   diff @@
==========================================
  Files           119        119
  Lines          6259       6271    +12
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3708       3718    +10
- Misses         2237       2239     +2
  Partials        314        314

Powered by Codecov. Last update fcaefbc...75a89bc


fix: Feed page responsiveness

Description
Fixes #1453
Fixes #1336
Duplicate to Issue https://github.com/open-sauced/insights/issues/1336
What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
#1336
Mobile & Desktop Screenshots/Recordings
Before:

After:
https://github.com/open-sauced/insights/assets/106407013/11f56cc8-4cbf-4d8d-bb66-02f42044dc36
Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Reviewing this @RitaDee, in the meantime do append the Fixes term to the duplicate issue too so It can allow it automatically close on merge this PR.

@babblebey Done.

Thank you, @babblebey for providing the diagram and the specific tasks to make the /feed page responsive. I have carefully reviewed the requirements, and I'll make the necessary changes as suggested.

I acknowledge the suggestions provided by @babblebey. Thank you
Update:
I have improved the layout and styles of the /feed page to achieve a responsive design as outlined in the tasks provided. The changes focus on the main, left, and right sections, ensuring they adapt gracefully to different screen sizes.
I applied the container and px-2 md:px-16 classes to the main element, allowing proper padding and responsiveness. Also, used the appropriate tailwind class to achieve this.
Current behaviour:
https://github.com/open-sauced/insights/assets/106407013/25935c04-3d45-400b-b291-ce6ef0522101


Errors during

node master__Operator1 ( input1:int^3 ) returns ( output1:int^3^2 )
	var
		Edge_3:int^3;
		Edge_4:int^3^2;

	let
		Edge_3 = input1;
		Edge_4 = Edge_3 ^ 2;
		output1 = Edge_4;
		--%MAIN ;
		check output1[1][0] < 4;
	tel

I tried the code firstly with kind2 1.5.1, resulted in a unexpected counter example. So, I turned to kind2 1.8.0, which end up with several runtime errors. Is there anything wrong with the code above?
Thank you in advance!

Thank you for reporting this issue.
There is nothing wrong with your code. The code triggers a bug in both versions of Kind 2.
Support for arrays is an experimental feature. We are considering dropping its support and replacing arrays with other high-level data structures like maps. We would appreciate it if you could provide feedback on how you use arrays and what for.
Thank you!