Windows: C++ ABI stability?

for example:
KUZU_API explicit Database(std::string databasePath);
the std::string-(template) parameter only works if you use the very same compiler with equal settings (debug/release,optimization) or else its more or less undefined behavior if the call works properbly
i'll switch to the C API because of this - what is the intention for having a non-100%-ABI-stable C++ interface?
or should the C++ API only be used if Kuzu is build as part of my project (is that possible?)

I think part of the issue is that MSVC's builds are less compatible between debug and release builds than on other toolchains and platforms. On Linux (or GCC and clang at least) you can usually link release builds with debug builds.

but its still undefined - so more or less nothing to build on forever

If the issue is primarily with std::string, we could probably provide overloaded versions of the functions which take C-strings

yeah that could help
at least the debug/release difference can happen to nearly every template based type - so most of STL or you're own templates

what is the intention for having a non-100%-ABI-stable C++ interface?

This is more an issue with C++ in general than with Kuzu in particular.

thats clear, i only talked about why Kuzu relies on this not very well defined stuff

If you need a more ABI-stable interface, then the C API is probably a better choice. Do note that none of the APIs are stable between different versions of kuzu at the moment, so don't expect the C API to provide ABI stability with other versions of kuzu except by coincidence.

these are two different things: API/ABI stability (so the parameters/return values beeing the same over serveral version) or missing ABI-stibility due to undefined behavior when using templates in the API

or should the C++ API only be used if Kuzu is build as part of my project (is that possible?)

Should be possible. You could include kuzu as an ExternalProject in a CMake build, for example. I'm not sure if this has been tested, but it probably will work. But you could also build kuzu using the toolchain and configuration you need and link manually against those binaries. You don't have to use the pre-built binaries.

im already building Kuzu from source - having it as inlined part of my project would force the same compiler(settings) :)

thats clear, i only talked about why Kuzu relies on this not very well defined stuff

Oh, okay. It wasn't clear from your original post what you were referring to.
We rely on the "not very well defined stuff" mostly because it hasn't been brought up as an issue before (again, we mostly work with compilers on other platforms where this is less of an issue), but also because I don't know of any good resource that defines exactly what isn't very well defined. I was aware that there are incompatibilities between debug and release ABIs on windows, but haven't been treating that as something that can be worked around.

the big difference between gcc/clang and MSVC is that MSVC activates the STL-debug-features per default
sizeof(std::string)
MSVC 2022
Release = 32
Debug = 40 (bigger due to validation members)
gcc 13.2
without -D_GLIBCXX_DEBUG = 24
with -D_GLIBCXX_DEBUG = 56 (bigger due to validation members)
and that could also change with different compiler versions/standard lib version - so prebuild libraries are definitly a problem
what about kuzu using libstdcpp and my application is using libc++ - could maybe work

#2553 improved this somewhat by replacing some unnecessary uses of std::string with std::string_view, which has the same size with stl debugging enabled.
To do more we'd also need to re-write the classes exposed in the API as PIMPL classes: instead of returning std::unique_ptr<T>, we return a PIMPLed public wrapper class which acts like a unique_ptr<T>, but lets us destroy the objects using code from kuzu instead of relying on the calling function's implementation of ~unique_ptr which needs to know the size of the type. We can also forward declare the actual implementations and omit them from the public headers, letting us omit as much non-public stuff as possible (primarily Database and QueryResult need to be changed; Connection may as well be changed too for consistency, but doesn't contain any types that get larger with STL debugging).
As mentioned in https://github.com/kuzudb/kuzu/pull/2553#discussion_r1427258835 (and following comment), this is a lot of work for relatively little gain, and the issue can be worked around by linking against different versions of kuzu depending on your configuration (which admittedly has a number of downsides). However a significant part of the work (the aforementioned change to fix the destructors) is a good practice which we should probably do anyway. Making sure that std::string isn't mandatory anywhere on the other hand may be more trouble than it's worth (e.g. the user defined function APIs use it a lot), but for the simpler and more often used functions we can try to make sure to at least provide overloads.

another idea is to do it the LLVM way - they do NOT write any wrapper for C++ to make the use safe but saying "you need to build and link with the same compiler when directly using the C++ interface" - only their C-API is language/runtime/compiler-version stable
maybe thats enough for Kuzu?

#2553 improved this somewhat by replacing some unnecessary uses of std::string with std::string_view, which has the same size with stl debugging enabled.

that is also in no way defined - could change any time - would not rely anthing on it - especially not a public API
but as i said - saying that the C++ API is only stable when consumed using the very same CRT and build-options - like LLVM does is maybe ok - but that must be stated clearly - and not based on "seems to work" mentality :)


Added CLIENT_ONLY to multi room audio

Hi,
As discussed here (https://forums.balena.io/t/balena-sound-disable-discoverability/88286/15) I added a Client-Only mode.
Currently everything works, in multiple variable combinations, over 4 Pi's in the fleet.
The only issue I am still having, is the system trying to restart the exited services, which of course exit immediately again. (Unlike DISABLE_MULTI_ROOM, which doesn't have that issue)

@tmigone Do you have a pointer how to fix that?

Hi @CaelynInc should be easy to fix. Just change the restart parameter of every container to on-failure instead of always here: https://github.com/balenalabs/balena-sound/blob/master/docker-compose.yml

@CaelynInc why do you want to disable all services (airplay/bluetooth/spotify) if multiroom is diabled?

@AlexProgrammerDE I don't want to disable them if multiroom is disabled (and this should not be possible with the code as I have tested it). I want to disable them to have a receiver only. With 4 devices in my house, it get's confusing which device to stream to as master (In my personal case, the Pi3 as he has the hardware to serve the multiroom setup, Pi2 has sometimes issues to get the stream consistent).

@CaelynInc i had just mistaken it. Sorry. ¯_(ツ)_/¯

@AlexProgrammerDE The restarting issue doesn't come up when you disable multi room, withour changing the parameters I believe. Shouldn't that be possible in this case as well?

@CaelynInc i had just mistaken it. Sorry. ¯_(ツ)_/¯

No worries!

@CaelynInc it is normal that the issue doesnt come up with multiroom disabled. The multiroom containers do only restart on failure. This is set in the compose file: https://github.com/balenalabs/balena-sound/blob/master/docker-compose.yml
You can set also the spotify, airplay and bluetooth containers to only restart on failure. The wont restart then.

I knew I should have gotten coffee first :laughing:
Thanks! I'll update it to fix that and test it out :)

@AlexProgrammerDE Thanks for the tip!

@CaelynInc you will need one tip more to get this with the balena ci working.

Open your terminal
Run:  git clone https://github.com/CaelynInc/balena-sound.git
RUN: cd balena-sound
Run: git reset HEAD~3
Run: git add .

Run: git push -f https://github.com/CaelynInc/balena-sound.git
Poste in this PR a comment: @balena-ci retest

@AlexProgrammerDE Thanks again! I followed the steps with a minor change due to use of SSH for upload.
Can you explain to me what it does, so that I can use it more often if needed? ;)

@balena-ci retest

@CaelynInc you will need one more tip to get this with the balena ci working.

Run: git push -f https://github.com/CaelynInc/balena-sound.git
Poste in this PR a comment: @balena-ci retest

@balena-ci retest

Balena uses its own CI in the PR checks. You should have only one commot with the right message. You must trigger the CI manually if you have done already more than one commit.

Check! Good to know and remember... Measure twice, commit one? :laughing:

@balena-ci retest

@AlexProgrammerDE and @CaelynInc thanks so much for working on this! Testing now :-)

Hey @tmigone ,

No problem, will do!
I tried killing the snapcast-server as well, but it gave issues with eiter snapcast-client or fleetsupervisor crashing and restarting (I can't remember which of the 2 it was :/). Still haven't figured out why. but working on it ;)
Yes, of course.

I'll check, and report back!

done
Testing now
done.

@tmigone When done, can I just commit or should I go through the series of commands again @AlexProgrammerDE posted before?

When I disable snapcast-server, fleet-supervisor is giving errors. Log excerpt below:
01.04.20 15:50:36 (+0200)  fleet-supervisor  ><EMAIL_ADDRESS>start /usr/src
01.04.20 15:50:36 (+0200)  fleet-supervisor  > node server.js
01.04.20 15:50:36 (+0200)  fleet-supervisor
01.04.20 15:50:38 (+0200)  fleet-supervisor
01.04.20 15:50:38 (+0200)  fleet-supervisor  Hello! I'm Fleet subscriber#54e07fdc-cf3c-49c5-8d0f-dac34a80c760
01.04.20 15:50:38 (+0200)  fleet-supervisor  ========================
01.04.20 15:50:38 (+0200)  fleet-supervisor
01.04.20 15:50:38 (+0200)  fleet-supervisor
01.04.20 15:50:38 (+0200)  fleet-supervisor  Hello! I'm Fleet publisher#59242ed7-1b2d-4097-b840-1b7b8b580e9e on 8000
01.04.20 15:50:38 (+0200)  fleet-supervisor  ========================
01.04.20 15:50:38 (+0200)  fleet-supervisor
01.04.20 15:50:38 (+0200)  fleet-supervisor  Fleet subscriber > service.online Fleet subscriber#7becb96b-0b3e-41ac-82f7-c8a6569c7f98
01.04.20 15:50:38 (+0200)  fleet-supervisor  Fleet subscriber > service.online Fleet publisher#cc2fc20b-de79-4868-96d6-6e01abd16bcb on 8000
01.04.20 15:50:38 (+0200)  fleet-supervisor  Fleet subscriber > service.online Fleet publisher#59242ed7-1b2d-4097-b840-1b7b8b580e9e on 8000
01.04.20 15:50:38 (+0200)  fleet-supervisor  events.js:187
01.04.20 15:50:38 (+0200)  fleet-supervisor        throw er; // Unhandled 'error' event
01.04.20 15:50:38 (+0200)  fleet-supervisor        ^
01.04.20 15:50:38 (+0200)  fleet-supervisor
01.04.20 15:50:38 (+0200)  fleet-supervisor  Error: connect ECONNREFUSED <IP_ADDRESS>:1705
01.04.20 15:50:38 (+0200)  fleet-supervisor      at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1129:14)
01.04.20 15:50:38 (+0200)  fleet-supervisor  Emitted 'error' event on Socket instance at:
01.04.20 15:50:38 (+0200)  fleet-supervisor      at emitErrorNT (internal/streams/destroy.js:84:8)
01.04.20 15:50:38 (+0200)  fleet-supervisor      at processTicksAndRejections (internal/process/task_queues.js:80:21) {
01.04.20 15:50:38 (+0200)  fleet-supervisor    errno: -111,
01.04.20 15:50:38 (+0200)  fleet-supervisor    code: 'ECONNREFUSED',
01.04.20 15:50:38 (+0200)  fleet-supervisor    syscall: 'connect',
01.04.20 15:50:38 (+0200)  fleet-supervisor    address: '<IP_ADDRESS>',
01.04.20 15:50:38 (+0200)  fleet-supervisor    port: 1705
01.04.20 15:50:38 (+0200)  fleet-supervisor  }
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR! code ELIFECYCLE
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR! errno 1
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR<EMAIL_ADDRESS>start: `node server.js`
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR! Exit status 1
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR!
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR! Failed at the<EMAIL_ADDRESS>start script.
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
01.04.20 15:50:38 (+0200)  fleet-supervisor
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR! A complete log of this run can be found in:
01.04.20 15:50:38 (+0200)  fleet-supervisor  npm ERR!     /root/.npm/_logs/2020-04-01T13_50_38_864Z-debug.log
01.04.20 15:50:40 (+0200)  fleet-supervisor
01.04.20 15:50:40 (+0200)  fleet-supervisor  2020-04-01T13:50:52.294148000Z OK

@CaelynInc no, what Alex posted was to "fix" your previous commits. Our CI runs some checks every-time you push a commit. One of those checks needs to find at least one commit with the Change-type information (though it's good practise to have it in all commits). Since your first 2 commits originally didn't have that you had to amend it with the steps Alex provided. After that was fixed, by doing "(at)balena-ci retest" you let our CI know that you want the checks to run again. Next time you push a commit the CI will be automatically triggered.
Regarding item 2 on your list, that error with fleet-supervisor is because it's trying to connect to the local snapcast server which we killed. When running in client only mode we should skip that.
You can find the logic for that in this file: https://github.com/balenalabs/balena-sound/blob/master/fleet-supervisor/server.js#L14
In case you are not familiar with nodejs, that should read like this to work:
if (process.env.BALENA_DEVICE_TYPE !== 'raspberry-pi' && !process.env.CLIENT_ONLY_MULTI_ROOM) {
  snapcastServer.connect()
  snapcastServer.onPlayback((data) => {
    fleetPublisher.publish('fleet-update', { master: getIPAddress() })
  })
}

@tmigone Awesome! Thnx. I am indeed unfamiliar with node.js (yet :laughing:), so that would have taken me a while to figure out...
It works on my setup, with only fleet-supervisor and snapcast-client running :smile:
Thanks again!

I don't mind nitpicking ^^ Has to be clean and tidy!

@balena-ci retest

This is looking great! Is there anything we can do to stop a client only device filling the log with the below on first boot? By default the client tries to connect to itself so generates a lot of error messages until a source server is set.
2020-04-01 16-52-28 [Error] (stop) Error in socket shutdown: Transport endpoint is not connected
01.04.20 17:52:29 (+0100)  snapcast-client  2020-04-01 16-52-29 [Error] (worker) Exception in Controller::worker(): connect: Connection refused

I wanted to change the README.md as @chrisys mentioned, somehow I see now:
Merge branch 'master' of github.com:CaelynInc/balena-sound
9c18346
I didn't expect that, I might have made an error somewhere... Or is that okay?
(README.md should be updated though...)

@CaelynInc we can fix that later.

I've just tested it and it's working great with a Pi Zero in client only mode!

This is looking great! Is there anything we can do to stop a client only device filling the log with the below on first boot? By default the client tries to connect to itself so generates a lot of error messages until a source server is set.
2020-04-01 16-52-28 [Error] (stop) Error in socket shutdown: Transport endpoint is not connected
01.04.20 17:52:29 (+0100)  snapcast-client  2020-04-01 16-52-29 [Error] (worker) Exception in Controller::worker(): connect: Connection refused

I've just tested it and it's working great with a Pi Zero in client only mode!

Yay. That saves a lot on resources for the Pi Zero Owners ;)

This is looking great! Is there anything we can do to stop a client only device filling the log with the below on first boot? By default the client tries to connect to itself so generates a lot of error messages until a source server is set.
2020-04-01 16-52-28 [Error] (stop) Error in socket shutdown: Transport endpoint is not connected
01.04.20 17:52:29 (+0100)  snapcast-client  2020-04-01 16-52-29 [Error] (worker) Exception in Controller::worker(): connect: Connection refused

Good question... I'll take a look, but I can't promise anything yet :laughing:

Do I understand right, that this bit in snapserver-client/start.sh sets the initial location (localhost) and that it changes when an other master is announced?
Start snapclient
SNAPCAST_SERVER=$(curl --silent http://localhost:3000)
echo -e "Starting snapclient...\nTarget snapcast server: $SNAPCAST_SERVER"
snapclient -h $SNAPCAST_SERVER
(and isn't that also the variable you could use in #122 ? )

Yep that's right @CaelynInc as the server would normally be on the same device as the client, setting it to localhost is a good starting point, but now we've disabled the server it's not required. I don't think this needs to be fixed before we can merge but would be a good improvement for later.

@CaelynInc if the user uses spaces would the if command fail.

Spaces where?

@CaelynInc if the user uses spaces would the if command fail.

Spaces where?

In the variable

I wouldn't really know how, as the variable itself is set, and needs to be entered in the dashboard as it is in the script to pick up, if I am not mistaken. The user has no choice in the name, only in the value

@chrisys I can take a look at it, but I agree. It doesn't block function, more of an aesthetic.

Is there still something waiting for me to do? Or am I just being confused xD (First time working with Git, so it's all quite a learning curve)

@balena-ci retest

@AlexProgrammerDE Awesome! Thanks for your help!

@CaelynInc thank you - send me an email at<EMAIL_ADDRESS>and we'll get you some swag

We'll have to wait for @tmigone to update his review but after that Alex feel free to merge.

This is looking great! Is there anything we can do to stop a client only device filling the log with the below on first boot? By default the client tries to connect to itself so generates a lot of error messages until a source server is set.
2020-04-01 16-52-28 [Error] (stop) Error in socket shutdown: Transport endpoint is not connected
01.04.20 17:52:29 (+0100)  snapcast-client  2020-04-01 16-52-29 [Error] (worker) Exception in Controller::worker(): connect: Connection refused

Good question... I'll take a look, but I can't promise anything yet 😆

I remember I looked at this briefly when developing multi-room with no good results. Maybe there is an option to make the output less verbose which I didn't find? Anyways, it's worth another set of eyes on it.

@balena-ci retest

@CaelynInc can you please run this commands again?

Open your terminal
Run:  git clone https://github.com/CaelynInc/balena-sound.git
RUN: cd balena-sound
Run: git reset HEAD~9
Run: git add .

@CaelynInc sorry i think i have done something wrong. Can you try it with that please?

Open your terminal
Run:  git clone https://github.com/CaelynInc/balena-sound.git
RUN: cd balena-sound
Run: git reset HEAD~1
Run: git add .
Run: git commit --amend
The file should be saved like this:

Add CLIENT_ONLY to multi room audio
Change-type: minor

Run: git push -f https://github.com/CaelynInc/balena-sound.git
Poste in this PR a comment: @balena-ci retest

@AlexProgrammerDE Sure! Hold on...

@balena-ci retest

@balena-ci retest

@CaelynInc sorry for the complication. I will merge now. 👍

@AlexProgrammerDE No worries! Thanks! :grin:


Workaround for Blank Skills

This is definitely not a full fix but will at least hide the unused skills until a proper fix is made. #941

I'm not going to merge this as I'd rather just get a proper fix in, but I'll leave this PR open for now as a way to easily show people a "quick fix" they can do locally if they want it.
Proper fix is being worked on as part of the refactor this weekend.

Any news on this?


/cards give card, card doesn't exist.

Describe the bug
/cards give card, card doesn't exist.
To Reproduce
Steps to reproduce the behavior:
/cards give card
Expected behavior
A card to be given.
Error Logs
Paste any errors here.
Screenshots
If applicable, add screenshots to help explain your problem.
Server & Plugin Versions:

Server Version: Purpur version git-Purpur-1336
Minecraft: 1.17.1
XenTradingCards: 5.4.0-SNAPSHOT

Fixed in #27


Letting wallets to define failure

Instead of guarding insufficient value in simulator, lets wallets themselves define insufficient funds in selectCoins by returning None
I needed to tear apart selectCoins and spend for this.
TODO: measuring time. I was too lazy to do this

The result is that the wallets are spending utxos sooner, since I removed the (outgoingPaymentsQueue.front.value * (-1) + 2 * WalletConstants.CENT) < currentLowestBalance) from most wallets (it's still in the wallets that I don't understand how they work :D).
The results are slightly worse and I am not sure why. I think it's because they are spening utxos sooner, the next transaction have utxo set with smaller utxos and that somehow hurt? Really, I am not sure at this point.
However it matches user's behavior more I think
(note that this PR is nor mergaeble at the same time as https://github.com/Xekyo/CoinSelectionSimulator/pull/11 ; whichever is merged first, the second will have to be edited :))

This probably slows down the whole thing a bit too, but anyway, when combined with the other PR with "failure modes" it makes even more sense


Box coordinate confusion

I'm probably confused about this so this is more for clarification than to indicate an issue with the code.
As far as I can from Yolo's documentation, Yolo should be return for an object some X, Y, W, H values. Apparently these correspond to the center x coordinate, the center y coordinate, and some ratio of width to height data. Your YoloItem class however has the method Center() which seems to imply that the X, and Y correspond to the top left of an object, as opposed to the center.
Could you please respond indicating what X, Y, Width, and Height specify with regards to a YoloItem?
Much appreciated!

the X and Y coordinates belong to the upper right corner of the detected box.
the width is the pixels to the right from the X coordinate.
The height is the pixels down from the Y coordinate.

Thanks a ton!

Yes, it is the upper left, I was wrong in the wording, excuse me


organizational revamp and multicore feature

Big Picture
This pull request is twofold:  it introduces a new parallel processing feature to SVTyper and is a major organizational revamp.  On a high level, it attempts to do the following:

make SVTyper pip installable
make SVTyper library callable (svtyper now can be embedded into any python program)
breakdown svtyper into smaller reusable components
convert the test directory into a formal testing suite paradigm
introduces a multicore version of SVTyper (for just a single sample): svtyper-sso

Detailed Breakdown

move the core svtyper script into svtyper/classic.py
move the Vcf, Variant, Info, Genotype, QueryPos, SamFragment, Sample, Library, SplitRead, and SplitPiece classes into svtyper/parsers.py
move the statistical related functions to svtyper/statistics.py
move common utility functions into svtyper/utils.py
move the verison information out into svtyper/version.py
convert the svtyper command line script into a pip console script
convert the multiple svtyper tests into a single python test module tests/test_svtyper.py
add setup.py, MANIFEST.in, and setup.cfg to make installation and testing work with pip and setuptools
add a Makefile to help facilitate testing and development management
add svtyper/singlesample.py: a module to optimize genotyping a single sample via multicore parallelization.  This has potential to speedup genotyping by 2-15x compared to the original svtyper.
add an alternative command line script svtyper-sso to interface with svtyper/singlesample.py
add an integration test for svtyper-sso, tests/test_singlesample.py
introduce a few new helper methods into the Vcf, and Variant classes in svtyper/parsers.py,  and in svtyper/utils.py.  These are used in svtyper/singlesample.py
updated README.md to describe svtyper-sso and new installation process.

Other Notes
The core svtyper algorithm and implementation has not been altered.  The user interface to svtyper is the same as in prior versions.  The average user shouldn't notice any difference in calling svtyper.
The commit history contains many experimental deadends that were attempted to improve svtyper performance.  Some parts of these ideas could possibly be used in the future, so I've decided to leave them in the commit history.
If these changes are desired, it is probably best to have a code review of this pull request before integrating into the main master branch.

@ernfrid : It might be good to do a code review sometime relatively soon (perhaps sometime next week) before I forget all the details.

A few benchmark number done with the in-house compute at the MGI.
cores   batch-size      max-reads-threshold     LSF-JOB-ID      num-batches     start   finish  elapsed_time    min-batch-elapsed-time(secs)    max-batch-elapsed-time(secs)   memory(bytes)   memory(gb)
1       -       -       1894321  1      11:39:02        02:12:11        14h33m09s       -       -       14155776*       0.0132*
2       1000    1000    7373887  614    12:38:22        20:00:23        7h42m01s        65.1300 183.1593<PHONE_NUMBER>      4.1208
2       2000    1000    7373888  307    12:38:22        21:55:21        9h16m59s        163.7969        338.3813<PHONE_NUMBER>      4.1256
2       3000    1000    7373889  205    12:38:21        23:20:23        10h42m02s       215.4711        643.2028<PHONE_NUMBER>      2.8796
2       4000    1000    7373890  154    12:38:22        19:31:48        6h53m26s        162.8096        466.0055<PHONE_NUMBER>      4.1492
4       1000    1000    7335723  614    00:46:03        05:47:09        5h01m06s        65.5910 403.2647<PHONE_NUMBER>      5.7202
4       2000    1000    7335724  307    00:46:03        06:15:25        5h29m22s        139.6992        815.6095<PHONE_NUMBER>      5.7486
4       3000    1000    7335725  205    00:46:03        06:26:56        5h40m53s        160.5561        1170.6312<PHONE_NUMBER>      5.7486
4       4000    1000    7335726  154    00:46:03        06:33:30        5h47m27s        147.9888        1633.3055<PHONE_NUMBER>      5.7745
8       1000    1000    7335727  614    00:47:09        04:46:53        3h59m43s        98.0087 517.0530<PHONE_NUMBER>      8.952
8       2000    1000    7335728  307    00:47:35        04:47:24        3h59m49s        195.2631        938.3752<PHONE_NUMBER>      8.9864
8       3000    1000    7368558  205    11:37:04        14:10:01        2h42m57s        197.9738        494.9243<PHONE_NUMBER>      8.6677
8       4000    1000    7368559  154    11:37:05        13:51:11        2h24m06s        175.0392        737.7193<PHONE_NUMBER>      8.5199
8       12000   1000    7369163  52     11:43:15        13:58:18        2h15m03s        190.0644        1728.9017<PHONE_NUMBER>      9.0496

Total # SVs in VCF: 778534
Actual # of SVs to genotype: 613579
# of SVs SKIPPED (surpassed max-reads threshold): 6659


Wrong hystrix event type for fallback missing

In my annotated hystrix command I have not defined any fallback method. But when my command execution gets fail due to force fully open circuit, it gives me
 eventNotifier.markEvent(HystrixEventType.FALLBACK_FAILURE, commandKey);

But I think this should be
 eventNotifier.markEvent(HystrixEventType.FALLBACK_MISSING, commandKey);

which has a check for if (fe instanceof UnsupportedOperationException) {} before making event for FALLBACK_MISSING.
Can somebody confirm whether this is expected? Any help would be appreciated.

@dmgcodevil Do you know if it is possible to support FALLBACK_MISSING in Javanica?

@mattrjacobs will take a look shortly

https://github.com/Netflix/Hystrix/pull/1616

@dmgcodevil I also found the cause and tested yesterday. I was about to provide information here to seek possibilities to contribute :)

@mattrjacobs  in which release we can expect to be available ? :)

This is released in v1.5.13 now


Tag 1.2.0 missing from docker

Warning: Docker pull failed with exit code 1, back off 6.118 seconds before retry.
  /usr/bin/docker pull quay.io/terraform-docs/gh-actions:1.2.0
  Error response from daemon: manifest for quay.io/terraform-docs/gh-actions:1.2.0 not found: manifest unknown: manifest unknown
Error: Docker pull failed with exit code 1

I have a github action that uses this project, its only pinned to @main
    - name: Render terraform docs and push changes back to PR
      uses: terraform-docs/gh-actions@main

Its failing because it cant find the latest version which was released this morning.
I can just pin back to a previous version, but I wanted to drop a note here as others may experience this same issue.
Thanks for all your work on this great project.

This has started to fail for us also. We are not able to pull the 1.2.0 tag. Should we use the edge release ?

We've just cut the v1.2.0 release. Can you try again?

Worked great, thanks!

For everyone else having the same issue, point the action to v1 instead of main
    - name: Render terraform docs and push changes back to PR
      uses: terraform-docs/gh-actions@v1
      with:


[3.x] Add the ability to remove project contents from the remove project dialog

#47200 is a cherry-pick of this (since my laptop doesn't support vulkan), and since there were some merge conflicts I figured it'd save a little work if I just made a separate PR for 3.x

Remember to apply changes suggested here too: https://github.com/godotengine/godot/pull/47200#pullrequestreview-616944132

@Calinou is this good now 🙂?

Thanks!


⚠ Secret Website 2 has degraded performance

In dfa6aa9, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 3927 ms

Resolved: Secret Website 2 performance has improved in 58b08c1.


Typo

TYPO
Hi, I noticed in main.py, a small error in the --help section (line ~200):
RUN EXAMPLES:
Required:           koverage run --assembly [file] --reads [dir]

Should be:
RUN EXAMPLES:
Required:           koverage run --ref [file] --reads [dir]

Well spotted Andrea, thanks! Fix will be pushed in next release.


hyperdrive 8

(WIP PR)

[x] replicate
[ ] replicate (only latest + sparse)
[x] unlink
[x] history
[x] checkout/version
[x] readdir
[x] readFile/writeFile
[x] stat
[x] lstat
[x] createReadStream/createWriteStream
[x] mkdir
[x] rmdir
[x] access
[x] exists
[x] tests
[x] write files as normal files

WOOOOOOOT ❤

unlink

:O I've been waiting for this for so long it's finally out :D
The others look very useful too
BTW you might want to close #99 now

:tada: :tada: :tada: :tada: :tada:

Will be released on NPM in a few hours \o/

Awesome!!
So what's the status on the two unchecked boxes, "replicate (only latest + sparse)" and "tests" ?

@pfrazee tests and sparse mode in there. it is missing "only latest" but i'm adding that in a minor bump asap to just get this out the door

You can just use sparse mode to download latest anyway

Cool

8 is now on npm!


[ci] Add MacOS job to test shared object copying

Issue and/or context:

Follow up to #1937
Linked Issue: #2210

Changes:
Expands SO copying pipeline to include a macOS job
Notes for Reviewer:
I have good news and bad news. The good news is that I added a macOS job, and its behavior is correct! The bad news of course is that this means it doesn't explain the feedstock failures where the macOS builds fail to find the shared objects.
I suspect this is because the feedstock is still using the older setup.py with the flag --libtiledbsoma (script). This was required pre-#1937. However, now that #1937 has been merged, we can now upgrade to the more modern alternative:
export TILEDB_PATH=$PREFIX
export TILEDBSOMA_PATH=$PREFIX
$PYTHON -m pip install --no-deps . -vv

update: replaced jinja2 {{ PYTHON }} (for use in meta.yaml) with $PYTHON (for use in separate build script)

Backports to release-1.8 and release-1.7 have failed to launch in CI multiple times, e.g.
https://github.com/single-cell-data/TileDB-SOMA/actions/runs/8176608919
Backport
Error: The process '/usr/bin/git' failed with exit code 128
Backport
HttpError: Resource not accessible by integration
Has anyone seen this before? I can debug but am already debugging multiple things simultaneously in an attempt to unblock multiple teams, so I thought I'd crowdsource first ...

I'm not convinced this is urgently needed in the branches


kie-issues#652: Document Dev deployments and its requirements on KIE Sandbox

Closes https://github.com/apache/incubator-kie-issues/issues/652

Thank you @thiagoelg, that's much better.
Can you please better clarify the image usage with an example?
eg. It expects a Quarkus application to be running at the same host, which it will use to fetch information to render a form that let's users interact with a Decision.
That means we need a canvas_dev_deployment_quarkus_blank_app ?

Thank you @thiagoelg, that's much better.
Can you please better clarify the image usage with an example?
eg. It expects a Quarkus application to be running at the same host, which it will use to fetch information to render a form that let's users interact with a Decision.
That means we need a canvas_dev_deployment_quarkus_blank_app ?

I've fixed some things and added a better README to both the dmn-form-webapp and its image, can you check if it's better now?

New PR with code changes to dev-deployment-dmn-form-webapp: https://github.com/apache/incubator-kie-tools/pull/2438


node_modules library css fo rc-time-picker

I am trying to include rc-time-picker css, how can i achieve this? How to enable any side library css ?

just require('/path/to/rc-time-picker.css') on top.

Dear @ZeroCho  where should i place  if (isClient) {   require('!style-loader!css-loader!prismjs/themes/prism-coy.css'); } ?

@ZeroCho I ve just got it  ! Just put on top of my jsx file
`const isBrowser = typeof window !== 'undefined';
if (isBrowser) {
require('!style-loader!css-loader!rc-time-picker/assets/index.css');
}
`
Thank you!

@rihannarickeminem 👍


Provide examples in documentation for wrappers

          Understandable, I think adding easy to read examples will be the most helpful. I'll make a documentation issue to add examples for the wrappers.

Originally posted by @stephenberry in https://github.com/stephenberry/glaze/issues/869#issuecomment-2018209543

Wrappers are now much better documented.


Healthcheck

Describe your changes
Issue ticket number and link
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] The pipeline is passing

Closing duplicate. Same as #166.


Remove exclusive_creation argument from write() function

This will revert the changes from #77.
Raising an error by default is useful in some cases, but it can also be harmful in other cases.
As far as I know, in most APIs the "write" function overwrites by default.
Also, e.g., the Unix commands cp and mv by default overwrite their target.
I think it's reasonable to assume that users expect overwriting behavior, therefore we should give it to them.
Once the default is changed to overwriting behavior, the argument exclusive_creation isn't that relevant anymore and it should be removed.
If users want to make sure that a file is not overwritten, they'll either have to check the existence first (e.g. with os.path.exists()) or they can use f = SoundFile(..., mode='x').

I added a commit that should implement the changes: 1ebcb891a3717866b9648a04ecdf4ff83e2dc95b

Shouldn't this change a test as well? If not, it should have ;-)
Anyway, looks good to me.

You're right, I forgot the tests, my bad!
There are actually 2 tests which fail now.
I'll fix the tests ...

I amended my commit, fixing one test and removing another (d39b332d39d26e34d74d10da6bede7c71a557578).
There is already a test case test_if_open_with_mode_w_truncates() which tests if 'w' and 'w+' truncate "normal" files, so I guess we don't have to make a separate test for sf.write(), right?
I hope now it's OK?

Looks good to me. If you're happy with it, merge.


Changes to IPython setup code not present when creating new notebook

I'd expect the setup code to consist of imports that every analysis should have. But, changes to setup code aren't present when starting a new notebook. Only the default setup code is there. Perhaps I've set things up incorrectly or misunderstood the purpose of setup code.

the setup code is per notebook, so every time you make a new notebook, you get the default setup code.
sounds like what you want to do is update your default notebook.  set your setup code and active plugins and any boilerplate all saved in a regular notebook file, and then copy it to ~/.beaker/v1/config/default.bkr and then next time you click "new default noteobok" you'll get what you want.


Python 3.7 --- model prediction and object detection

🐛 Bug
I got this error
~/.local/share/virtualenvs/cell_instance_segm-TWYpWSyU/lib/python3.7/site-packages/flash/core/integrations/icevision/adapter.py in (.0)
86             DefaultDataKeys.INPUT: collate_fn(
---> 87                 [to_icevision_record({**sample, DefaultDataKeys.METADATA: m}) for sample, m in zip(samples, metadata)]
88             ),
TypeError: 'str' object is not a mapping
To Reproduce
Launch model prediction for object-detection, master version of flash and python3.7

TypeError                                 Traceback (most recent call last)
~/.local/share/virtualenvs/cell_instance_segm-TWYpWSyU/lib/python3.7/site-packages/flash/core/data/batch.py in forward(self, samples)
230                     try:
--> 231                         samples = self.collate_fn(samples, metadata)
232                     except TypeError:
~/.local/share/virtualenvs/cell_instance_segm-TWYpWSyU/lib/python3.7/site-packages/torch/nn/modules/module.py in _call_impl(self, *input, **kwargs)
888         else:
--> 889             result = self.forward(*input, **kwargs)
890         for hook in itertools.chain(
~/.local/share/virtualenvs/cell_instance_segm-TWYpWSyU/lib/python3.7/site-packages/flash/core/data/utils.py in forward(self, *args, **kwargs)
192     def forward(self, *args, **kwargs) -> Any:
--> 193         return self.func(*args, **kwargs)
194
~/.local/share/virtualenvs/cell_instance_segm-TWYpWSyU/lib/python3.7/site-packages/flash/core/data/process.py in collate(self, samples, metadata)
439         if len(parameters) > 1 and DefaultDataKeys.METADATA in parameters:
--> 440             return collate_fn(samples, metadata)
441         return collate_fn(samples)
~/.local/share/virtualenvs/cell_instance_segm-TWYpWSyU/lib/python3.7/site-packages/flash/core/integrations/icevision/adapter.py in _collate_fn(collate_fn, samples, metadata)
86             DefaultDataKeys.INPUT: collate_fn(
---> 87                 [to_icevision_record({**sample, DefaultDataKeys.METADATA: m}) for sample, m in zip(samples, metadata)]
88             ),
~/.local/share/virtualenvs/cell_instance_segm-TWYpWSyU/lib/python3.7/site-packages/flash/core/integrations/icevision/adapter.py in (.0)
86             DefaultDataKeys.INPUT: collate_fn(
---> 87                 [to_icevision_record({**sample, DefaultDataKeys.METADATA: m}) for sample, m in zip(samples, metadata)]
88             ),
TypeError: 'str' object is not a mapping
Code sample

Expected behavior
Environment

PyTorch Version (e.g., 1.0):
OS (e.g., Linux): linux
How you installed PyTorch (conda, pip, source): pip
Build command you used (if compiling from source):
Python version: 3.7
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

Additional context

Hi @gianscarpe thanks for reporting this! I thought this should have been fixed following #857 - could you share a reproducible script which gives the error?

I can reproduce the error.
installation via:
pip install git+git://github.com/airctic/icevision.git#egg=icevision[all] --upgrade
pip install 'git+https://github.com/PyTorchLightning/lightning-flash.git#egg=lightning-flash[image]'
use the example program from https://lightning-flash.readthedocs.io/en/latest/reference/object_detection.html
append the following lines to the end of the file to load weights from saved file and attempt inference
model_2 = ObjectDetector.load_from_checkpoint(
    "object_detection_model.pt"
)

predictions = model_2.predict(
    [
        "data/coco128/images/train2017/000000000625.jpg",
        "data/coco128/images/train2017/000000000626.jpg",
        "data/coco128/images/train2017/000000000629.jpg",
    ]
)
print("with reloaded weights", predictions)

I can reproduce the error.
installation via: pip install git+git://github.com/airctic/icevision.git#egg=icevision[all] --upgrade pip install 'git+https://github.com/PyTorchLightning/lightning-flash.git#egg=lightning-flash[image]'
use the example program from https://lightning-flash.readthedocs.io/en/latest/reference/object_detection.html append the following lines to the end of the file to load weights from saved file and attempt inference
model_2 = ObjectDetector.load_from_checkpoint(
    "object_detection_model.pt"
)

predictions = model_2.predict(
    [
        "data/coco128/images/train2017/000000000625.jpg",
        "data/coco128/images/train2017/000000000626.jpg",
        "data/coco128/images/train2017/000000000629.jpg",
    ]
)
print("with reloaded weights", predictions)

Got same error with that code, any workaround using Trainer.predict?

Hey all, this issue should now be fixed as we now only support prediction via a Trainer with a datamodule where this error shouldn't occur. Closing this, but please feel free to re-open if needed :smiley:


Klog flags

This should resolve flag provided but not defined: -logtostderr error when clusterapi start machine controller

This code was based on the docs from https://github.com/kubernetes-sigs/cluster-api/tree/master/docs/book/provider_implementations
You may want to consider proposing this addition there, too.

Filed https://github.com/kubernetes-sigs/cluster-api-provider-openstack/issues/271 to track this. I'm not entirely sure if this PR would fix it, so didn't propose upstream PR yet

I tried this fix, but it doesn't seem to work for me.
bin/machine-controller-manager --logtostderr=true --v=3
flag provided but not defined: -logtostderr


Doen't work for me - Chrome 43 on OS X 10.10.4

Hi.
Having trouble with high res (4k) videos, in Chrome,
but in Safari it's fine. Found topic:
https://code.google.com/p/chromium/issues/detail?id=456513
with link to h264ify, installed, but no effect, youtube still loading vp9 in chrome.
Chrome 43.0.2357.134 (64-bit)
On page
http://www.youtube.com/html5

all checkmarks (including "WebM VP8" and "MSE & WebM VP9")

Look like the JS may not be injected properly. What happens when you run window.MediaSource.isTypeSupported("vp9") in the console?

Now it's work fine, found the problem - Adblock Plus extension. Deleting it (switched to uBlock) helps.
window.MediaSource.isTypeSupported("vp9")
false


passing launch config without a 'type' attribute to 'vscode.startDebug' does not return an error

testing https://github.com/Microsoft/vscode/issues/6248:

VSCode Version: 1.1.0 alpha
OS Version: OS X 10

Steps to Reproduce:

use the setup from https://github.com/Microsoft/vscode/issues/6806
remove the 'type' attribute
run 'Hello' command

Observe:
no error is reported.
'type' is a mandatory attribute that should result in error if missing.
But it should be possible to omit attribute 'name' without error.

Currently the command returns the errors, after consulting with Joh it is the extensions responsibility to handle this error in any way the extension writer wants. Closing as designed. More details:

@isidorn What error is returned if attribute 'type' is missing?

With the commit I just pushed it returns a proper error message 😊

Maybe we should tune the error message such that it does not refer to launch.json


Add Track MBID if present in the Plex Webhook POST

I sniffed the webhooks of Plex for fun... Turns out properly MBID-tagged tracks do have MBID UUIDs sent in the webhook... Weird that nobody noticed. Here's an example webhook POST content:
{
    "event": "media.play",
    "user": true,
    "owner": true,
    "Account": {
        [redacted]
    },
    "Server": {
        [redacted]
    },
    "Player": {
        [redacted]
    },
    "Metadata": {
        "librarySectionType": "artist",
        "ratingKey": "75741",
        "key": "/library/metadata/75741",
        "parentRatingKey": "75740",
        "grandparentRatingKey": "64551",
        "guid": "plex://track/6622d12bf03cec061155e4cb",
        "parentGuid": "plex://album/6622d11af03cec06115354f9",
        "grandparentGuid": "plex://artist/5d07bc87403c640290518e59",
        "parentStudio": "Drum&BassArena",
        "type": "track",
        "title": "Two Fang",
        "grandparentKey": "/library/metadata/64551",
        "parentKey": "/library/metadata/75740",
        "librarySectionTitle": "Drum and Bass",
        "librarySectionID": 12,
        "librarySectionKey": "/library/sections/12",
        "grandparentTitle": "Spor",
        "parentTitle": "Two Fang",
        "summary": "",
        "index": 1,
        "parentIndex": 1,
        "ratingCount": 1100,
        "userRating": 9.0,
        "viewCount": 1,
        "lastViewedAt":<PHONE_NUMBER>,
        "lastRatedAt":<PHONE_NUMBER>,
        "parentYear": 2024,
        "thumb": "/library/metadata/75740/thumb/1718401062",
        "parentThumb": "/library/metadata/75740/thumb/1718401062",
        "grandparentThumb": "/library/metadata/64551/thumb/1718310361",
        "addedAt":<PHONE_NUMBER>,
        "updatedAt":<PHONE_NUMBER>,
        "musicAnalysisVersion": "1",
        "Guid": [
            {
                "id": "mbid://d3c67e4c-6589-4612-9efb-d6fb6a7b4db4"
            }
        ]
    }
}

And indeed, Metadata.Guid[0].id contains a beautiful MBID for a track... Which means we can submit it to the API and get actual Album information to ListenBrainz!
Anyways, I've patched a few things to add support for that. I am not that big in the TS territory, so if there's any cleanup to do, I'd advise someone else help me with this!
Cheers.

Turns out ListenBrainz does not yet support reading from the MBIDs sent in the JSON. Weird of them... I'll open a ticket tomorrow.

In any case - this won't really do anything on LB side until https://tickets.metabrainz.org/browse/LB-1330 is resolved.

I've started another branch, to resolve the track_mbid into release_mbid and recording_mbid because as I said earlier, LB does not yet use track_mbid for matching.
https://github.com/defvs/eavesdrop.fm/tree/track-mbid-resolving if you want to have a look...

Just tested everything now and I can see the track_mbid being properly sent to ListenBrainz (through the "inspect listen").
I can also confirm that it isn't currently taken into account when matching tracks. Yet.
Example (Both tracks are properly tagged with MBIDs and have a different track_mbid linking to two different releases):
Track 1:

Track 2:


Support for setting the date/time

This adds the option to set the date/time in the settings app.
I'm not sure if it is desirable to merge this now already, since the kernel function to actually set the time (convertTimeToTicks) is not done. I'm creating this pull request now anyway to get some comments, and to have a place to discuss this feature.

First pass over the code looks good. Having a look at functionality now.

Functionality looks awesome. This is super cool.
One change I would make is to center the date/time interface on the screen, rather than aligning it to the left.

FYI, I added some features to settings today and you'll have to merge from upstream before this can be applied. Am I correct that this can be merged now and will Just Work when the relevant feature is added to the kernel?

Yes, it should start working when the kernel function is added.
At the moment there is one remaining issue, namely that the date wrap-around code assumes all months have 31 days. (So if you decrease the day when 2015-03-01 is selected, you get 2015-02-31 instead of 2015-02-28.) I'll hopefully have time to look into that today.

Cool. I'm going to start doing a weekly mailing list blast of "this week in KnightOS", and it'd be cool if this could be included in the first one on friday :wink:
It also looks like leap years are not handled - when you tidy up all those things, let me know and I'll give this a merge (without kernel support).

I'm working on the leap years right now :)

Great!

Head's up - you need to handle losing focus correctly (by redrawing the screen when you regain focus). You'll know that you've losed focus because corelib(app{Get,Wait}Key) will reset Z.

I think all the outstanding issues are resolved now, except for the suggestion to omit the clock option from the menu if the calculator doesn't have a clock. I agree that that would be nicer from a user perspective, and I can build that, but it would complicate the menu code in main.asm quite a lot, I think... Do you think that is worth it?

Yes

Yes, I think it's worth it. I would rather not merge this without. It should be just as simple as pcall(getClock), then load a different menu table if NZ.

We should have a clockSupported kernel call, by the way.

I have that kernel function on my list of useful kernel functions that I want to add :smiley:

How many more stuff needs to be done until ConvertTimeToTicks is finished?

Date/time support in general is being tracked here: https://github.com/KnightOS/KnightOS/issues/168

I think it is ready; the menu item is not shown anymore when the clock is unsupported :smile:

Awesome!


RecursionCircuit and updated BatchCircuit

This PR induce a new aggregation layer to aggregate multiple snarks from batch circuit, using an recursive scheme:
doc

is the bundle config file here is used for production?  if not, there should be some code to let recursion circuit use "layer5" config, where is that code?

withdraw root is "Additional public input that are unchanged over all the SNARKs".  not too clear to me.  You mean these are public inputs not to be "chained"?

is the bundle config file here is used for production?  if not, there should be some code to let recursion circuit use "layer5" config, where is that code?
withdraw root is "Additional public input that are unchanged over all the SNARKs".  not too clear to me.  You mean these are public inputs not to be "chained"?

If indicate the bundle_circuit.config file under aggregator/config: No. This file is only used by the unit test of recursion circuit, so we need 5 adv colums for this config since one unit test would recursive prove a recursion circuit itself. And maybe we should not naming it as "bundle" i think.

Recursiion circuit (bundle) use a "layer5" config here: https://github.com/scroll-tech/zkevm-circuits/blob/8846bd40a42fd995276ed9508d16c78fb938ac9f/prover/src/aggregator/prover.rs#L229

I think it is not "unchanged" but "do not involved in state transition". I consider this PI is "not require to be verified by state transition", because they are in fact involved with (or say, decided by) state root. The doc has some irriadiation about them:
https://www.notion.so/scrollzkp/Upgrade-4-Darwin-Documentation-05a3ecb59e9d4f288254701f8c888173?pvs=4#62a0b2058b9842188b5bd5e7267220ca

ok i find this line env::set_var("BUNDLE_CONFIG", layer_config_path(id));


Able to suppress INFO logs

Hi, it seems everytime a user instantiates a SupervisedNE problem, INFO logs appear in jupyter cell and can't seem to disable it with typical methods like logging.basicConfig or .setLevel(logging.WARNING).

Nevermind, worked with logging.disable('WARNING'). I wonder why...

Hi @maulberto3!
Sorry for replying to your issue very very late.
While replying to the issue #81, I noticed/remembered that this older issue you posted is mentioning the same thing.

Nevermind, worked with logging.disable('WARNING'). I wonder why...

It works, because logging.disable(WARNING) disables all the reports with levels up to and including WARNING. The regular reports generated by an evotorch.Problem instance are at the INFO level (a level that is below the level of WARNING), and therefore they are disabled by this trick you mentioned.
If you wish to suppress the INFO messages of only EvoTorch and not of any other library, you might want to try this:
from evotorch.tools import set_default_logger_config
import logging

set_default_logger_config(
    logger_level=logging.WARNING,  # only print the message if it is at least a WARNING
    override=True,  # override the previous logging settings of EvoTorch
)

Sorry again for my very late response.
Happy coding,
Engin


qna

Describe the solution you'd like

Describe alternatives you've considered

Additional context

The Bot Framework composer creates the QNA Maker Search service with a standard SKU by default (which runs at $250/month).  It would be great if it gave the option to specify which SKU you wanted, or defaulted to the Free SKU since you can't downgrade the service without deleting it and re-creating it.  Most deployments from the composer app will be for development environments, and it burns through the MSDN benefit rather quickly! :-)

The publishing profile defaults to S0 as there are limitations that can happen with doing F0 by default.  When provisioning for local development the default is F0.  So, this is by design.  You can change the setting in the Azure portal post provisioning and change the pricing tier to F0 if desired.  Just be aware that you will have a maximum of 3 KBs for the service.

Also, most of the cost is tied to the Search service, however you can use another search service instance for your QnA Maker instance. Here is an article to show you how to do this:
https://docs.microsoft.com/en-us/azure/cognitive-services/qnamaker/how-to/configure-qna-maker-resources?tabs=v1


Base wireframe

Wireframe of the base template design including navigation and footer

This is completed or tracked in other tickets.


noname=yes

I was willing to had the common noname=yes (300,000 occurrences worldwide) and was very disappointed to see the rare, undocumented validate:no_name=yes (1,000 occurrences worldwide or.... DC wide).
Please propose common tags.
https://taginfo.openstreetmap.org/compare/noname=yes/validate:no_name=yes

I really don't understand what you are asking us to do.. Can you attach a screenshot?

Today I tried again and when typing nona, noname was correctly proposed.
It must be a slightly different scenario where validate:no_name was proposed. I had to type noname entirely.
As I can't reproduce it I close it.


GLES2 cull mode inverted (wheel)

Godot version:
master
OS/device including version:
Antergos (Arch Linux)
Issue description:
In GLES2 the wheel disappears if I set the cull mode to back. The wheels are rendered correctly when using GLES3. In the editor it looks correct with GLES2 and GLES3.
Steps to reproduce:
See minimal reproduction project.

Minimal reproduction project:
https://hugegameartgd.github.io
https://hugegameartgd.itch.io/solus-stunts

Strange that this happens only on wheels. even for my project and @BastiaanOlij tutorial demo.
I think it has something to do with transformation matrix in the physics.

I have the exact same issue.
Godot 3.1 stable x64
Windows 10 x64

Can anyone here post steps to reproduce or an MRP? Without either one of those, no one is going to work on this bug. A video/packaged game is not an MRP. Here is a description of an MRP from stackoverflow https://stackoverflow.com/help/minimal-reproducible-example

@clayjohn
Here I made a small demo project, it's just importing things from Blender.
It looks fine in the editor but inverted in the game.
gles2_vehicle_wheel_bug.zip

@OBKF
Thanks, I updated the description

Okay, wow. Looking into this has exposed a whole mess in both the vehicle code and the rendering code.

In Vehicle_wheel sets the scale to a negative value for some reason. This inverts the geometry. And causes some other weird stuff https://github.com/godotengine/godot/issues/26627
GLES3 forces the scale to be positive so the objects render normally (kind of, normals still have issues: https://github.com/godotengine/godot/issues/26710, https://github.com/godotengine/godot/issues/25696)

To fix this particular issue, we either need to fix why wheels are using a negative scale. Or we need to make it so that negative scales are okay and will not screw up the geometry.
What we really need to do is both.
Regarding the negative scales @Timofffee can you take a look at that? Do wheels absolutely have to use negative scales?

Nice one @clayjohn , a lot of the vehicle code was copied from an old version of bullet and altered to work with Godot, I'm guessing they did that to make something work without noticing the consequences :)

The vehicle code is actually too bad and does not behave as good as in Bullet (at least the Blender version), so I gave up on it (I am making my own based on a unity opensourced project Tork, bit arcady but seems good).
I think @AndreaCatania mentioned that he will bind the vehicle code from bullet at some point but I am not sure.

A bit off-topic but reduz mentioned that the vehicle code won't be part of the engine in coming versions of Godot and instead implemented as a module (asset library).


GroupExt Additions

Hello,
Kindly requesting a couple functions for GroupExt:

clip_children()
draw_child()
draw_children()
update_child()

I'm still trying to create my own custom scroll group and reading through the FLTK source for FL_Scroll found a couple of these methods.
Would be much appreciated!

Hi
I'll check it out.

Thanks!!


fix(PaginatedMessageEmbedFields): custom template with fields

This was on purpose, but if you feel like it is needed, just add a check to be sure not to have more than 25 fields :)


Add release notes for 0.12.1

Signed-off-by: Janki Chhatbar<EMAIL_ADDRESS>

I was updating the GDoc as you sent this PR :stuck_out_tongue_winking_eye:. Can you copy the latest from there?

@Jaanki we are already working on releasing 0.12.2. Can you please get this one reviewed and merged as soon as possible? thanks!


delete all records then insert new one

if response is true then delete all data and insert new data otherwise ignore it.

Yap, we need this one!

Hello Kiaking
I think we need to support  option and method
delete all before create or insert
add option on Model.api
User.api().get(url, {
  deleteAll: true,
})

use deleteAll() on action.
actions: {
   method_name() {
      return this.deleteAll().post()
   }
}

User.insert({
  deleteAll: true,
  data: { id: 1, name: 'John' }
})
User.create({
  deleteAll: true,
  data: { id: 1, name: 'John' }
})

Thank you.

Yea I hear lots of request regarding this. I think we need it.

Would this be useful in the following case?:
I'm on a password reset page and am fetching the User via a token. When found, the user is loaded into VuexORM, I then fetch the user with first() (this seems slightly dodgy currently as it assumes an empty collection just prior to the fetch).
Also at the moment, navigating back to the password reset page doesn't show a 404 because the User is still in the collection and even though the second fetch returns an empty result, this doesn't clear the User collection.

I think this issue boils down to the user being able to leverage the persist methods and/or options.
The underlying save() method currently invokes User.insertOrUpdate but in reality, while this should be considered the default behaviour, users should be able to configure persistence to alleviate current limitations.

@mean-cj wrote:
I think we need to support option and method delete all before create or insert
User.api().get(url, {
  deleteAll: true,
})

An option such as deleteAll is useful but, as previously pointed out, this is something User.create handles nicely. However, the current implementation requires user intervention and additional boilerplate code.

The { save: boolean } option could be the ideal property to become configurable as it directly relates to persistence. Furthermore, the configuration of this option can work with the save method without additional validation.
For instance, the save option can be configured as follows:
save: boolean
save: true invokes Model.insertOrUpdate as per existing implementation.
save: false defers persistence as per existing implementation.
save: string
Where the string value can be any one of the persist methods, falling back to insertOrUpdate:
save: 'create' invokes Model.create
save: 'insert' invokes Model.insert
save: 'insertOrUpdate' invokes the default behaviour
Now, we have a way to delete before persistence without additional options.
Refresh entities per request:
User.api().get('url', { save: 'create' })

Refresh entities for all model requests:
static apiConfig = {
  save: 'create'
}

Refresh all entities globally:
VuexORM.use(VuexORMAxios, {
  axios,
  save: 'create'
})

What do you guys think?

@cuebit Yeah I think that's what we should be doing 👍

Hello @cuebit  @kiaking
Thank you for your detail.
it ok , but define save option current doesn't work with relation.
i thank we need option for delete model data with relation hasOne, hasMany beforeCreate
or if can define hook method on api option
Example
export default class User extends Model {
  static fields() {
    return {
      id: this.attr(null),
      orders: this.hasMany(OrderItem, 'order_id'),
    }
  }
}

User.api().get('url', { save:'create', beforeCreate:()=>{
 this.model.orders.deleteAll();
 this.model.deleteAll();
}})

Work for me,  i use hook for delete all relation
export default class User extends Model {

  static beforeCreate(model) {
    Order.deleteAll()
  }

  static fields() {
    return {
      id: this.attr(null),
      orders: this.hasMany(Order),
    }
  }
}

User.api().get('url', { save:'create'})

@mean-cj I don't think the plugin will have any intention of handling relations, let alone deleting them, simply because it just bridges axios and Vuex ORM. Until Vuex ORM supports cascade deletes, your solution is about as good as it gets.

Hello @cuebit
Thank you very much for your update and PR.

Hello @cuebit thank you so much.
I use entities per request doesn't work
ajax response is ok , last response not exists order_id
but orderModel not deleteAll data before create and not callback beforeCreate method after ajax response.
Order.api().cancleOrder()
export default class Order extends Model {

  static beforeCreate(model) {
    console.log('beforeCreate')
  }

  static apiConfig = {
    cancleOrder(order_id) {
      return this.post(  '/order/cancle', {
          order_id: order_id,
        },
        { save: 'create' }
      )
    },
  }
}

Hello @cuebit
deleteAll on beforeCreate doesn't work
because myModel append data before call to beforeCreate method
therefore if use deleteAll in beforeCreate, order relation is empty
  static beforeCreate(model) {
    Order.deleteAll()
  }

@mean-cj not sure I follow. Assuming you are using save: 'create', this will invoke Model.create() and any corresponding hooks should still fire. If your request also contains relations, those relations will also use the create method on their respective models.

Resolved by #119


chore(main): release 0.4.17

:robot: I have created a release beep boop
0.4.17 (2024-04-08)
Continuous Integration

build: Fix build syntax (7d3200a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/linrongbin16/cargo-ci-template.rs/releases/tag/v0.4.17 :sunflower:


fix: resource policy generation for {path+}

Why is this change necessary?

To transfrom the arn in the api gateway resource policy to be "*"
instead of the name of the path itself.

How does it address the issue?

Allow to address the appropriate resource.

What side effects does this change have?

None that are known at this point.

Issue #, if available:
https://github.com/awslabs/serverless-application-model/issues/1549
Description of changes:

Transform path of three kinds, {path+}, {proxy+} and {path} into "*"

Description of how you validated changes:

deployed the template without errors as referenced in https://github.com/awslabs/serverless-application-model/issues/1549

Checklist:

[x] Write/update tests
[x] make pr passes
[ ] Update documentation
[x] Verify transformed template deploys and application functions as expected
[ ] Add/update example to examples/2016-10-31

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sriram-mv
From your description: Transform path of three kinds, {path+}, {proxy+} and {path} into "*"
Should {path} really be resolved into *?

@jfuss Good catch on the PR description, will update that.


this fixes the siginfo_t compilation errors

@alexcrichton this fixes the siginfo_t problem with netbsd.

I also added some missing externs for netbsd.

@alexcrichton BTW, I'm 95% done with bootstrapping for NetBSD.  My dhuseby/rust-cross-toolkit is able to get through the first three stages for NetBSD.  It can build a stage0 rustc but the rustc segfaults when calling https://github.com/dhuseby/rust/blob/master/src/libstd/sys/unix/stack_overflow.rs#L131
For some reason, the compiler emits code to call the signal 13 compatibility trampoline code instead of just calling the main sigaction syscall.  That's as far as I've gotten in the debugging.  I need to install some debug symbols and debug the NetBSD kernel to see why it's returning ENOSYS and segfaulting on that call.  It's probably just some alignment issues or something.  The weirdest part is if I use clang to compile C code that calls sigaction exactly the same way stack_overflow.rs does, it generates the correct code to execute the sigaction syscall.
Once I can figure that out, I'm pretty sure I'll be able to build snapshots for NetBSD and set up a build bot for that OS too.

@dhuseby if you use master branch of rustc for building, be aware that llvm could produce bugged code for x86-64 architecture. It is what I hited with OpenBSD (https://github.com/rust-lang/rust/issues/31363). See also https://github.com/rust-lang/rust/issues/31505.

@semarie wow, those bugs are nasty.  I'm just trying to figure out why I'm getting signal 13 trampoline code generated for a call to sigaction.  I'll know more in a few days.

It looks like the build is failing because the cfg_if! macro isn't exactly the most robust in the world (it doesn't handle this #[cfg]-like situation.
Perhaps, though the struct could be defined as:
pub struct siginfo_t {
    pub si_signo: ::c_int,
    pub si_code: ::c_int,
    pub si_errno: ::c_int,
    #[cfg(target_pointer_width = "64")]
    __pad1: ::c_int,
    pub __pad: [u64; 13],
}

That should mirror what we do in Linux which is to not give you si_addr but rather where the union starts. The padding should only be there for 64 bit as well, I think, right?
This'll require more support code in std but it's a little more faithful to the representation on the system I think.

@alexcrichton I just removed the #[cfg] for now since we don't support 32-bit netbsd anyways.

rebased

Could you also squash the commits down into one as well?

@alexcrichton squashed and fixed.

@dhuseby Regarding the ENOSYS errors: Just a wild guess, but some compatibility system calls on NetBSD are just not available on x86_64, since they have been deprecated long since before x86_64 even existed. So you need to make sure not use the compatibility functions (most of them are really, really old).
The libc crate used to define the #[link_name] for the affected NetBSD functions to avoid calling the legacy versions (see PR rust-lang/rust#28543) for exactly for this reason, but unfortunately this got lost during the great libc refactoring.
I think the reason your code works in C, but not in Rust, is that NetBSD C header files defines alternative linker symbols for the replacements. If you call sigaction in C, it actually actually links against __sigaction14. So Rust needs to define #[link_name = "__sigaction14"] on the sigaction declaration. See also alexcrichton/ctest#1
Unfortunately I've been too busy in the last few months to maintain Rusts NetBSD port myself. I have some time now, so let me know if I can help out.

@gandro it was a #[link_name] problem.  I have a patch inbound.


MINOR: Add 3.6.1 to system tests

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mjsax @dajac @vvcephei Can you upload the 3.6.1 artifacts to the kafka-packages S3 bucket?
Steps are described in https://cwiki.apache.org/confluence/display/KAFKA/Release+Process#ReleaseProcess-Afterthevotepasses

@mjsax @dajac @vvcephei Can you upload the 3.6.1 artifacts to the kafka-packages S3 bucket? Steps are described in https://cwiki.apache.org/confluence/display/KAFKA/Release+Process#ReleaseProcess-Afterthevotepasses

Please also help upload 3.5.2 artifacts. Thank you!

Ping @dajac @mjsax @vvcephei Can you take a look at this? Thanks

Uploaded 3.5.2 and 3.6.1 artifacts.

Thanks @mjsax!
I've added 3.5.2 to this PR as well.

@mimaison -- should this be cherry-picked to 3.7 branch? -- And the bump to 3.5.2 split out for 3.6 branch?

I've backported to 3.7 branch, and created a PR: https://github.com/apache/kafka/pull/15187 to add v3.5.2 into 3.6 branch. Please help review. Thanks.


Fastnlight: Watch frb085664

Fastnlight requests the watch of the watch_keyword frb085664. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

frb085664 has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


DSL for Target Platform Type

Note: This isn't same as the type of target
In Xcake we have a way of configuring the supported platforms of a target, this defines if a app is iPhone Only, iPad Only or Universal.
I'm not sure if this is configurable in the Info.plist but we should look into how Xcode configures it and add it to the DSL if not.

We can do this in a configuration override.

iPhone-only is the default, no changes needed.
For iPad: add an override TARGETED_DEVICE_FAMILY: 2
For Universal: add an override TARGETED_DEVICE_FAMILY: 1,2

We should add a higher-level DSL for this as part of the main task to integrate an Xcake-like DSL superset.

Sounds good 😀 yeah a higher level dsl would be great as those numbers aren't understandable

@jcampbell05 Yup I agree! I guess Xcode developers got lazy that day 😉


10.1101/2021-10-22-464971

Croucher et al.
Single-cell transcriptional analysis of the immune tumour microenvironment during myeloma disease evolution
https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4971v1.abstract
Data:
https://www.ncbi.nlm.nih.gov/geo/query/acc.cgi?acc=GSE134370

can I choose this dataset? Thank you


Prefer unstable sort for primitive types

Unstable sort uses less memory and because this is a primitive type,
equal values are indistinguishable so stability is useless.

What problem does this PR solve?
Problem Summary:
What is changed and how it works?
What's Changed:
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

/lgtm

Please sign off your commits (with git commit -s)

Recommend to use --amend to overwrite your last commit.

@hbina you pushed an empty commit.

@andylokandy It seems that this PR have basically implemented this PR. Closing this.


Slurm collector (separate crate)

Replaces #141

Codecov Report

Merging #146 (4caa4bb) into main (e66a566) will decrease coverage by 4.49%.
The diff coverage is 0.60%.

@@            Coverage Diff             @@
##             main     #146      +/-   ##
==========================================
- Coverage   60.13%   55.64%   -4.50%
==========================================
  Files          32       33       +1
  Lines        2022     2187     +165
==========================================
+ Hits         1216     1217       +1
- Misses        806      970     +164

Impacted Files
Coverage Δ

collectors/slurm/src/main.rs
0.60% <0.60%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


x/dl/internal/version: add $GOROOT/bin to $PATH when running go run/generate

Tools run by go run/generate sometimes need to execute the go command, for example when using go/packages.
Adding $GOROOT/bin to $PATH ensures that the appropriate version of the go/gofmt commands are run when using a go command wrapper generated by x/build/version.
edit: and test. Anything that runs go code could exec the go tool.
See #28043 for background and discussion, specifically https://github.com/golang/go/issues/28043#issuecomment-427969204.
cc @bcmills

New CL isn't being linked for some reason, see: https://golang.org/cl/143545


Update BOS costs

We need to update BOS costs to reflect the new benchmarks.

This will include:

Adding excel sheet from Vignesh to repository
Adding script for fitting BOS costs to repository (this script should probably also build BOS preset-tree?)
Add BOS preset tree javascript file to repository
Make sure to add documentation regarding BOS costs: cite the system cost benchmark technical report, explain that the fits aren't perfect but are a good approximation, add caveat that if you are evaluating module sizes that are much larger than standard sizes in 2020, the BOS costs will be less accurate.

(or much smaller)


1.x Refactorings/replication batcher

A replacement for blitz4j batcher. A noticeable improvement over the latter one is request collapsing (removing old pending task, when a new one is submitted).

NetflixOSS » eureka » eureka-pull-requests #450 SUCCESS
This pull request looks good

NetflixOSS » eureka » eureka-pull-requests #460 SUCCESS
This pull request looks good


bump node 20.12.2 LTS

Applied mkpr on package.json
sed s/"node": ">=20.12.1"/"node": ">=20.12.2"/

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.138%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
404

Relevant Lines:
412

💛  - Coveralls


DEF files (version 5.8) written contain an obsolete NAMESCASESENSITIVE statement

A sample of such statements can be seen in the .defok files in most of the tools. For example:
https://github.com/The-OpenROAD-Project/OpenROAD/blob/openroad/src/replace/test/ar01.defok#L1-L2
https://github.com/The-OpenROAD-Project/OpenROAD/blob/openroad/src/resizer/test/buffer_ports1.defok#L1-L2
https://github.com/The-OpenROAD-Project/OpenROAD/blob/openroad/src/opendp/test/simple05.defok#L1-L2
For some more context, see this: https://github.com/The-OpenROAD-Project/OpenROAD/pull/452#pullrequestreview-472968410
Thanks.

Resolved in https://github.com/The-OpenROAD-Project/OpenDB/commit/da95adc746f7928660faaaa96fa46fe8ae17abc9.


[proposal] Z-index in Stack widget

Hello, yesterday I have reported spurious lack of functionality in flutter and without it I cannot complete game migration. I have tried all proposed ways proposed on stackoverflow, Flutter cummunity silent about solution also
I have found another similar feature request here  https://github.com/flutter/flutter/issues/70836 just its usecase is simple
So I am asking to reopen issue:
https://github.com/flutter/flutter/issues/70758#issuecomment-729675908
And implement it adding zindex to positioned and animated Position widgets or creating new widget ZIndex
It will simplify layouting dranatically as well as games cretion
Thank you

@vytautas-pranskunas-
self-duplicating issues, are not the best way to proceed,
next time please consider give more time to reopen a closed issue befor opening a new one
this seems to me a very limited use case for a feature request,
one could argue that you can already achieve what you describe using Transform and Matrix4 (example)
I'm also a bit confused by the involvment of the Stack,
children are "piled" on Z axes already, I'm not sure Positioned is the best candidate for the role
imho your feature request should be a 3rd party package, rather than part of the main framework

I was not sure if closed tasks did get an attention.
I dear to disagree a limitation of use because then we have to argue about limitation of use position: absolute and z-index usage on the web.
Stackoverflow answers are not working or involves anormous implementation complexity.
Do not understand why z-index might be irrelevant for framework. From my point of view it would dramatically simplify layouting.

I looked at example you gave - transforms are meant to scew and rotate objects. Would you be able to give me a code snippet that makes first container rendered above second container using matrix4D?

I see you put likes to your answers but hesitate to give a solution to existing problem.

Rearanging list arrays involves big difficulties when it comes to working with 56 cards when each of them can be dragged and stacked into combinations... Also it has negavtive effects to animations
I could not made elevation working with Material using STack layout it does not react to elevation
I have just tried Matrix4 - i was not able to get item with index 0 rendered on top of item with index 1 in a list.

So i would be more then happy if you could show how to do this because i am bit stuck and i cannot get answer from anywhere.
Otherwise i keep thiking that this problem did not exist on windows phone, it can be really easily solved on the web using CSS, ReactNative and NativeScript.
I do elieve in Flutter as a 1# framework to develop corss platform app but in order it to happen likes is not enough - it would be more helpful to provide example (if it is so easy) rather then keep liking showing that framework can do this and dev who requested that is just stupid. Or maye you will face same chalange and recognize the need of this or that feature.

@vytautas-pranskunas-

Rearanging list arrays involves big difficulties when it comes to working with 56 cards when each of them can be dragged and stacked into combinations... Also it has negavtive effects to animations

reordering the children of a stack works very well in principle
here's a minimal code sample
I believe using key properly would minimize if not eliminate completely any inconvenience with animations

Reordering 9 arrays and total 56 cards which can be stacked in combinations meaning that you might want to stack together indexes 0,15,37,50 together into one place leaving other cards in their previous places and relative to each other.. But how you can calculate this if 5 indexes just had angee places?
I understand that you example where you switched 2 arrays it is easy... I also understand that theoretically it is possible. But guys - you do not want to understand that it brings anormous amount of complexity where should not be ANY problem.
Z-index would solve all this emediatly.

@vytautas-pranskunas-
I wrote this for you, try it out, I've also updated the code sample
to see the code in action,
it's a very minimal helper, but could make the task easier

elevation helper

extension Elevete on Widget {
  Elevated elevate(num i) => Elevation(
        child: this,
        elevation: i,
      );
}

abstract class Elevated {
  const Elevated({this.elevation});
  final num elevation;
  Widget call();
}

class Elevation extends Elevated {
  const Elevation({@required this.elevation, @required this.child});
  final num elevation;
  @protected
  final Widget child;

  @override
  Widget call() => child;
}

abstract class AxisZInterface<T extends Elevated> {
  const AxisZInterface(this.childrenZ);
  final List<T> childrenZ;

  List<Widget> get orderedChildren;
}

class StackZ extends Stack implements AxisZInterface<Elevation> {
  StackZ({@required this.childrenZ});
  final List<Elevation> childrenZ;

  @override
  List<Widget> get orderedChildren {
    childrenZ.sort((a, b) => a.elevation.compareTo(b.elevation));
    return <Widget>[for (Elevated e in childrenZ) e()];
  }

  @override
  List<Widget> get children {
    childrenZ.sort((a, b) => a.elevation.compareTo(b.elevation));
    return <Widget>[for (Elevated e in childrenZ) e()];
  }
}

Hello,
@iapicca  thanx for your effort. I have tried .sort option 2 days ago but animation was not working properly (because basically what you build for me is ...cards.orderBy((c) => x.zIndex).toList()) so looking at your example i understood that maybe something wrong was with animation - after moving one card all cards were rearanging. After reading your comment about keys i assigned key to card class animation became normal.
So having index is not so crucial in my case and your extension and StackZ could be part of framework reducing amount of questions given about z-index especially for those who comes from web.
Thanks for your help that helped me identify problem.

I have some Platform widgets in a stack and when trying to rendering them using Keys, they still get reinitialised. So zIndex would be great!

I want to paint a marking box behind a text but it shall still catch pointer-events before the text. Z-Index seems to be the easiest solution IMO. I know I could use OverlayEntry for that. Also Stack would work if I duplicate the marking widget into two widgets, one behind the text and the other in front of the text. For now I'm using blendMode.

I would be great if all of the widget could apply zIndex which gives the flexibly of rearranging UI

try this [https://pub.dev/packages/indexed][1]

Online demo
Video demo
    Indexer(
        children: [
            Indexed(IndexedInterface
              // prefer add key when use AnimatedPositioned
              key: UniqueKey(),
              index: 100,
              child: Positioned(
                //...
              )
            ),
        ],
    );

+1

Why not wrap your widget with Material then change elevation value?

any updates?

any updates?

Yes, this package solves the problem elevated_flex

@lukas-pierce not exactly, you can manipulate index with conditions easily and have multiple elevated ekements above each other. With this package, as i understand, you can have only one above others. But it is better than nothing

@lukas-pierce not exactly, you can manipulate index with conditions easily and have multiple elevated ekements above each other. With this package, as i understand, you can have only one above others. But it is better than nothing

Yes, this package only for elevate relative items in column or row.
But as far as I understand from the statement of your problem, you need a Flow widget; which allows you to describe at a lower level the position and order of widgets in layers
https://youtu.be/NG6pvXpnIso?si=ylhxOxhETLZqUBZV

Yes thats may be good solution, but i have long time ago solved that
problem in my game somehow.
On Fri, 17 May 2024, 08:08 Lukas Pierce, @.***> wrote:

@lukas-pierce https://github.com/lukas-pierce not exactly, you can
manipulate index with conditions easily and have multiple elevated ekements
above each other. With this package, as i understand, you can have only one
above others. But it is better than nothing
Yes, this package only for elevate relative items in column or row.
But as far as I understand from the statement of your problem, you need a
Flow widget; which allows you to describe at a lower level the position and
order of widgets in layers
https://youtu.be/NG6pvXpnIso?si=ylhxOxhETLZqUBZV
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/70836#issuecomment-2116794929,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA54ZF7MCT7OHYL6IPXF7KLZCWNHFAVCNFSM4T3NKZQ2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJRGY3TSNBZGI4Q
.
You are receiving this because you were mentioned.Message ID:
@.***>


Added annotation based processor configuration

This should implement #196
It is now possible to add these annotations:

@Name to define block or macro names for BlockProcessors, BlockMacro- and InlineMacroProcessors
@ContentModel to define the content model of a BlockProcessor
@Contexts to define the contexts of a BlockProcessor
@DefaultAttribute to define default attributes to pass to the process method of a processor
@DefaultAttributes to define multiple @DefaultAttributes on one type
@Location to define where content generated by a DocinfoProcessor is to be placed.

@PositionalAttributes are missing yet because I am not aware what they should do.
In general the implementation adds options to the processors class object that serve as a default configuration for the instances. (see https://github.com/asciidoctor/asciidoctor/blob/master/lib/asciidoctor/extensions.rb#L84)
If this is a way to pursue docs and javadocs should be enhanced.
Also the javadocs of the new annotations should be extended with examples.

:scream:

I just skimmed through de code and it’s amazing, not only that, it’s 100% compatible with the current registration methods.

Thank you! :)
Staying compatible is imo a top priority.
But it is only possible because Alex has provided such an excellent test suite!

Appveyor failed again due to a timeout.
What about only testing one platform on Appveyor?

@robertpanzer, this is really amazing. Extensions for AsciidoctorJ just got a whole lot more awesome. I'm raising my glass to your work tonight.

Staying compatible is imo a top priority.

:+1: Certainly a tough challenge, but since a bulk of the Asciidoctor user base goes through AsciidoctorJ, giving them a clear path forward is crucial.

But it is only possible because Alex has provided such an excellent test suite!

:+1: :+1:
One for the test suite, and one for teamwork.

@PositionalAttributes are missing yet because I am not aware what they should do.

Positional attributes map indexed positions in the attribute list to attribute names. For instance, the positional attributes for the image block macro are "alt", "width", "height" so you can write a shorthand version of the block macro, like image::pic.jpg[Picture,100,200]. Effectively, the block macro gets interpreted as image::pic.jpg[alt=Picture, width=100, height=200]. The value of the annotation would then be @PositionalAttributes("alt", "width", "height") Does that make sense?

What about only testing one platform on Appveyor?

Yes, I think that is sufficient. Let's proceed with that change.

Am 22.06.2015 um 21:43 schrieb Dan Allen<EMAIL_ADDRESS>Does that make sense?

That makes sense. I will try to get that working tomorrow.

Am 22.06.2015 um 21:43 schrieb Dan Allen<EMAIL_ADDRESS>One for the test suite, and one for teamwork.

Hehe, just had a Pale 31...

@robertpanzer this is really awesome, my test suites are only this, test suites, necessary of course, but the real work is here and it works super

Added @PositionalAttributes annotation and added some Javadocs.

:+1:
...and more :beers:

@mojavelinux We should have some real :beers: together soon again!

:+1:


Callout Function throws error: Uncaught TypeError: Cannot read property 'getBoundingClientRect' of null

Copied and pasted the exact callout code from the Hopscotch demo to test in my page
var calloutMgr = hopscotch.getCalloutManager();
calloutMgr.createCallout({
  id: 'attach-icon',
  target: 'attach-btn',
  placement: 'bottom',
  title: 'Now you can share images & files!',
  content: 'Share a project you\'re proud of, a photo from a recent event, or an interesting presentation.'
});

this results in Chrome throwing errors: Uncaught TypeError: Cannot read property 'getBoundingClientRect' of null
and
Uncaught Error: Callout by that id already exists. Please choose a unique id.

Thank you for the issue report @txshan . This issue is happening because element that you have specified in target option does not exist on the page, hence Hopsoctch does not know where to position the callout.
This exception is not helpful though, since it's not telling you what actually happened.  I'll add this issue to our 0.2.5 milestone, to make sure that error message of the exception is more descriptive.

Hi @kate2753 ,
Thanks for your feedback. However. The element does exist on the page and is still throwing the error. I tried this with my own callout for my client project as well. I have had no issues with the typical tour, but receive this error with every callout instance I have attempted.

@txshan, could you try running this in the console of your page:
document.getElementById("attach-icon")
This is what hopscotch does under the hood to find the target element. If element can't be found using document.getElementById, then Hopscotch would not be able to position the callout.

It's located in the console, but not Hopscotch. See screenshot:

It seems like a timing issue. Your code to create HopscotchCallout must be executing before the element itself is rendered.
Try running this in console after page is loaded:
document.getElementById("attach-icon")

var calloutMgr = hopscotch.getCalloutManager();

calloutMgr.createCallout({ id: 'attach-icon',   target: 'attach-btn',   placement: 'bottom',  title: 'Now you can share images & files!',   content: 'Share a project you\'re proud of, a photo from a recent event, or an interesting presentation.' });

Tried it in a $(window).load(function({});  too and get the same error.

Is there any way you can include unminified version of hopscotch on your page? I'd love to see the stack trace of the exception with line numbers.
You could also try applying the change from pull request #178 to your hopscotch.js file to see if error message you are seeing is changing

I used the unminified css and applied the changes from the pull request, and all is well.

Glad to hear that #178 resolves it. Thank you for helping us catch this issue!


Feature request: Show lobby name in text area

Would like to request the ability to show the lobby name in the custom text area. I often forget what baal run # I'm on, and need to tab the normal map to check it. Small but appreciated convenience.

I'd like to add to this, display game IP in custom text area

Currently I don't know how to get lobby name from memory, need some research.
game IP is found, will be added later

Would also like to request the ability to add 'area level' to the text box

implemented in dd439eb


v1.10 backports 2021-07-08

#16363 -- conformance tests: Use hubble-relay-ci image (@michi-covalent)
#16715 -- ci/conformance: Various image-related fixes (@gandro)

AWS-CNI workflow doesn't exist on v1.10, had to remove it.

#16769 -- Revert "policy: Make selectorcache callbacks lock-free" (@aanm)
#16767 -- test: Redeploy DNS after endpointRoutes reconfiguration (@joestringer)
#16804 -- contrib: Explicitly set remote for backport branches (@twpayne)
#16782 -- docs: account for bandwidth manager now being disabled by default (@bmcustodio)
#15999 -- Improve logging when cgroupfs mount fails (@johngv2)
#16822 -- docs: Document dns visibility limitations (@joestringer)

Skipped because the PR it fixes (which introduces the new test file) is not backported yet:

#16775 -- test/Bookinfo: Collect full artifact in case of failure (@pchaigno)

Once this PR is merged, you can update the PR labels via:
$ for pr in 16363 16715 16769 16767 16775 16804 16782 15999 16822; do contrib/backporting/set-labels.py $pr done 1.10; done

test-backport-1.10

test-backport-1.10

test-upstream-k8s

test-backport-1.10

test-backport-1.10

test-1.21-4.9

Commit from #16711 seems to be missing.
Commit from #16830 seems to be missing.
One commit of #16787 (f98a99de3ae4afb9f22065d08a5cf5a7be168449) seems to be missing.

Other commits LGTM.

All GH workflows were tested here https://github.com/cilium/cilium/pull/16897 the CI was green before I pushed as the only new changes pushed were GH workflow related.


contains malware

why Port 7777/tcp is open on your firmware . we all know that backdoors use this port , so you need to explain about this !!

README
Advanced configuration has to be done via the command line on the console interface. This console is available either via the serial port at 115200 baud or via tcp port 7777 (e.g. "telnet <IP_ADDRESS> 7777" from a connected STA).
Use the following commands for an initial setup:
set ssid your_home_router's_SSID
set password your_home_router's_password
set ap_ssid ESP's_ssid
set ap_password ESP's_password
show (to check the parameters)
save
reset

alright and how to close this port ? is your project open source ?

You can disable it
https://github.com/martin-ger/esp_wifi_repeater/blob/5fdca67c74f66bc5787883de78e9412378c18bba/user/user_config.h#L69

You can also dynamically change or disable this port:

set config_port portno: sets the port number of the console login (default is 7777, 0 disables remote console config)

could you explain more about how to disable ports and enable them , and tell me is it possible to change the frontend of config page (<IP_ADDRESS>) ?

I suggest you to first read all documentation about the project. 🙂
A quarta, 17/06/2020, 12:33, polararian<EMAIL_ADDRESS>escreveu:

could you explain more about how to disable ports and enable them , and
tell me is it possible to change the frontend of config page (<IP_ADDRESS>) ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/martin-ger/esp_wifi_repeater/issues/404#issuecomment-645319280,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQMA2BMSS6QGGGMMQFTGMTRXCSXNANCNFSM4OALU54Q
.

i could not find anything about setting password on telnet or ssh , could you tell me the command ?
😉

There is non password other than the "lock" password.

lock [password]: saves and locks the current config, changes are not allowed. Password can be left open if already set before (Default is the password of the uplink WiFi)
unlock password: unlocks the config, requires password from the lock command

i know this but i want to set ssh with username and password

No way. There is no ssh demon for the ESP8266 - at least none that I am aware of.

thanks for support 💎


feat(permissions): add option to make file generated in docker shell …

Description
Allows you to specify that all files generated in the docker-shell with user laradock should be group writable, i.e. 775 instead of 755
Just update this flag in .env to true:
WORKSPACE_GROUP_WRITABLE=true

Motivation and Context
Multiple users may be using a Laradock installation workspace. You can already specify a group id in the .env file, i.e.
WORKSPACE_PGID=1002

But files created in the workspace will only be readable by the group. For example if you run
php artisan make:controller MyController

from a Laravel project, the file generated on the host won't be writable by all members of that group, and saving the file from a code editor will fail
Types of Changes

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

Definition of Done Checklist:

[x] I've read the Contribution Guide.
[ ] I've updated the documentation. (refer to this for how to do so).
[x] I enjoyed my time contributing and making developer's life easier :)

Note: there is no section in the documentation about permissions in general, or existing permission configurations like the env variable:
WORKSPACE_PGID=1002

So it updating the docs did not seem to apply. Let me know if this is not the  case


v2.0-dev x-axis showing ms for dataset in seconds

I have the following example
https://jsfiddle.net/d0txbo3e/1/
It adds 10 data points each a second a part. The issue is the x-axis is showing ms instead of what I'm expecting.
If i change
return moment().add(days, 's').format();

to
return moment().add(days, 'm').format();

It shows correct. My only guess is it might have something to do with the labels on the ticks and there aren't enough data points?
Thanks

Ok it does seem to be a lack of data issue. I updated with more data and getting the results I'd expect.
https://jsfiddle.net/d0txbo3e/2/

Also another question.. in the
https://jsfiddle.net/d0txbo3e/2/
You'll notice there is a padding where the last value isn't at the end of the graph. On my test with a lot more data it seems even worse

@mzupan you can set a hard end point with
scales: {
    xAxes: [{
        time: {
            max: // max time value here.
        }
    }]
}

@etimberg thanks for the quick reply. I updated this with the example with setting a max for time
https://jsfiddle.net/d0txbo3e/4/
I'm not seeing any changes but still see the shifting and showing empty data.

Might it be linked to https://github.com/nnnick/Chart.js/pull/1825 ?

@Dergonic looks like its the same issue

@mzupan take a look at #1950
I've updated your fiddle at https://jsfiddle.net/kfts0jdg/1/
As an immediate fix for the label issue you can add the following to your config
{
    time: {
        displayFormats: {
            millisecond: 'h:mm:ss.SSS a'
        }
    }
}

Closing as #1950 was merged


UI Enhance

Looks similar to the bug described by https://github.com/goplus/community/issues/195


Self preservation method on eureka console

Describe the bug
EMERGENCY! EUREKA MAY BE INCORRECTLY CLAIMING INSTANCES ARE UP WHEN THEY'RE NOT. RENEWALS ARE LESSER THAN THRESHOLD AND HENCE THE INSTANCES ARE NOT BEING EXPIRED JUST TO BE SAFE.

Hello I get anytime this Message, also if my Service are alltime online and I don't know why it does not reset. After an Restart it's normal and I have this Message not anymore, but I wanna not anytime to restart after an update by my Applications, so give it an Feature which make this Fail away?
I tried everything, if more Informations needed like pom or something else tells me
Thanks & Greetings
Justin

See https://github.com/Netflix/eureka/wiki/Server-Self-Preservation-Mode

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

It works for me thanks alot

It works for me thanks alot