Colorize t2 list by USB, LAN, PROVISIONED and NOT AUTHORIZED

As you could see at summit a lot of listed Tessel 2 will make it difficult to find your own.

So I made a difference from not authorized Tessels when no ~/.tessel/id_rsa is present. This will help the customer to find own Tessel-Name faster.

Tessel instructors can tell the users what red or blue means.

@Student007 Is this PR made to fix https://github.com/tessel/t2-start/issues/52? A better process for this would be to discuss a solution on the issue before implementing (I'm not sure that this solution fixes the problem).

@Frijol it is ineffective to wait for people sleep so long :laughing: so I provide a possible solution - either you take it or you change it - as I try to discuss in Slack-Chat.

@Frijol by the way: This is a quick fix, of cause - If I would take time to rework and discuss this, it wouldn't help the show at summit. For that reason I double set the same phrase for note. Also there are some things have to be done about t2 list and Tessel.isProvisioned() what causes in a totally different solution maybe (like caching known Tessels and a note if they could be found). But there are other major tasks to finish. Like t2 root I also stumbled over some things I have to work around.
Finally I couldn't test it with a USB Tessel - but it looks to me you could need this at the summit. So I wrote on the review call you also could rework and/or merge.

Hm, this fix doesn't make sense to me; I'm not sure what usability it is fixing. Can you explain? I can't tell what the red and blue denote.

https://github.com/tessel/t2-start/issues/52

New user here. I was following the instructions for "Find Your Tessel" and was a little confused when it returned multiple Tessels [...]

This will happen at events with free WLAN all the time ...

I didn't realize that other Tessels on the LAN would be returned.

@Frijol this is covered by your doc fix suggestion (I agree).

Maybe clarify that the USB connection is the one new users are looking for.

Mr. Speaker: the blue one is yours
(the red Tessels found are either others or your second one)
I though about to make the provisioned and authorized Tessels green but while testing several terminals I though it would better stay native.
t2 list should only show a blue USB-Tessel (not provisioned) and show not authorized Tessels red in general (e.g. LAN-Tessels). All others (short: your own Tessels) will stay native.

I think it's not a usability fix if the speaker has to say "the blue one is yours". It might even be more confusing: "why are these different colors?" "these red ones look like errors" etc.
Knowing that the USB one is the one connected to your computer is useful (good problem identification), but I think we should put more thought into the design.
There was some long discussion about this here: https://github.com/tessel/t2-cli/issues/86

I like the idea of coloring them. The words next to the readout should help clarify any confusion and after using it a few times the colors will become quite obvious, and useful. I think this is a nice addition.

@adkron me too - don't understand why @Frijol didn't love it. I vote for merging like it is - but if someone has a better wording, this could be done quickly.
Maybe @johnnyman727 and @rwaldron put their two cents in before we maybe have a bit more usability for events :heart_eyes:
At the moment voting is 2:1 for merging I think ?

@Student007 I think coloring the text could be useful but it's hard to do it properly with the way t2 list works as is. What I mean is that it would be great if we could color the default Tessel green, unprovisioned LAN devices as grey (not red because that symbolizes error), and all other devices some other unassuming color. Unfortunately, we don't know what the default Tessel is until the discovery process is complete.
Perhaps the best we can do for now is mark USB and authorized LAN devices as green and unauthorized devices as grey?

I think @johnnyman727 's proposed solution is good. Good in hackathon settings and home use cases alike.
I like the suggestion of green & gray as well; gray is a good "you're not authorized" indicator, as opposed to "you have an error" red

I very much agree on the "no red". Red makes me think that something went wrong when it didn't.
Even if we don't know which one is the default until the discovery process is complete, we could easily amend the list as soon as discovery is completed. log-update is a great package for this which I have used before.
Any time a tessel is discovered it would be added to a list, we would then call logUpdate(...) to print that list to the screen. When discovery is completed, we color one of them green and then call logUpdate again. Finally we call logUpdate.done() to signal that we are happy with the output and want to move on.

@LinusU +1
We also print out what the default Tessel is after listing them and we can color that name green as well.

by the way re @Student007

don't understand why @Frijol didn't love it

I love the enthusiasm, and the ideas behind it; I just want to make sure we fully think through the user-facing design decisions, since they have such a big impact.

I agree - red is misleading (I simple decide to use red because of best contrast on my tested terminals - but I also would agree, the user could change the terminal layout if the contrast is bad).

[...] log-update is a great package for this which I have used before.

@LinusU this would fix the contrast problem - great idea :+1:
@Frijol  as you wrote #86

       Rafael        LAN
       Bulbasaur     USB       (run `tessel provision` to authorize)
       Mewcat        LAN       (USB connect and run `tessel provision` to authorize)
       Jigglypuff    USB
       Jigglypuff    LAN

I would use (run 'tessel provision' to authorize) but in the case we stay using colors instead of @LinusU idea, colored blue.

Perhaps the best we can do for now is mark USB and authorized LAN devices as green and unauthorized devices as grey?

:+1:
I don't know if a USB connected Tessel causes to filter out its detected LAN-Version (I couldn't test yet) I would agree. I think a not provisioned USB Tessel should be blue (it will be used seldom but event visitors who use a Tessel the first time would get the point faster).
Finally I think a combination of less colors (but colors) and log-output would be a contrast save but event supporting solution.

As another story we could allow the user to pass --no-color to turn off color too

@adkron hmm... thinking about this the implementation needs to be made different. But this also could be reasonable for a presenting situation - the projectors silver screen and sun could crash the experience guests should have by following the course.
I like this option because I am not fine with forcing to use green for valid things - it simple could have problems with contrast (like I figured out with different terminals).
@Frijol I think you did the most public presentations - could you put in your two cents here (maybe there also had been problems with native colorization) ?
Blue I feel was the best color to accent things to point peoples nose on. But only on lighted terminals - no dark. Blue on black ... there would be a option "inversed" what will make white on blue per line.
@johnnyman727 because this PR was meant as a quick fix for the summit, we can close the PR now and reference it from a late milestone description. If I am right you are thinking about new milestone structure at the moment ?
I am focused on a different feature at the moment (t2 root and --/key).

@adkron I'd lean against adding another flag – better to make a design decision here and keep it standard to reduce confusion.
@Student007 I don't have strong knowledge of the effects of forcing colors on different terminals (I use a black terminal with transparency, but at workshops a lot of beginners use white terminals). I'm guessing there exists a canned solution for this.
However, I think it would be better to table this issue for now instead of taking time to discuss it further; it's not of central importance compared to building out the core functionalities of Tessel 2 and I'm wary of bikeshedding.

@Frijol :+1: .. but it has less priority on my list. First t2 key then t2 root... than this one


--get-filename not getting the final filename (was: get filename not working?)

"--get-filename" returns a different name from a normal execution.
~# youtube-dl -o 'test.%(ext)s' --prefer-ffmpeg --get-filename https://www.youtube.com/watch?v=f3gNU0rNIIo
test.webm
:~# youtube-dl -o 'test.%(ext)s' --prefer-ffmpeg  https://www.youtube.com/watch?v=f3gNU0rNIIo
[youtube] f3gNU0rNIIo: Downloading webpage
[youtube] f3gNU0rNIIo: Extracting video information
[youtube] f3gNU0rNIIo: Downloading DASH manifest
WARNING: You have requested formats incompatible for merge. The formats will be merged into mkv
[download] Destination: test.f248.webm
[download] 100% of 83.44MiB in 00:02
[download] Destination: test.f141.m4a
[download] 100% of 7.24MiB in 00:00
[ffmpeg] Merging formats into "test.mkv"
Deleting original file test.f248.webm (pass -k to keep)
Deleting original file test.f141.m4a (pass -k to keep)
~# ls
test.mkv
I am expecting the two commands to return the same filename (either .mkv or .webm).
What am I doing wrong?
Thanks

The problem is that we output the filename before deciding if the formats have to be merged into an mkv file.
The workaround is to use -f best (you won't get the DASH) or -f 'bestvideo[ext=mp4]+bestaudio[ext=m4a]/bestvideo[ext=webm]+bestaudio[ext=webm]/best'

Thanks, the second solution worked for me, although I'm planning to just read the generated filename for now.

@jaimeMF If there is no extension used, such as simply -o output, shouldn't it not matter whether any formats have to be merged and just use the output name? Instead of appending any extensions?
Or an option to force not using an extension so we can trust what we pass to the -o option will be whats written?

@eriktjacobsen if not extension is used ffmpeg is not able to merge.

@dstftw I see...  Ok, just trying to get a handle on whether devs see this as something working as intended or something to be fixed. Couldn't youtube-dl just wait until ffmpeg finishes the merge, and then if no extension is listed in the -o output option, the file is renamed from whatever extension it used to the supplied output name?
Then at least it could be trusted to have some predictable output.

@eriktjacobsen there wouldn't be a bug label on the issue if it were expected.

The problem is that the final filename may not be determined until actual downloading. For example, FFmpegExtractAudioPP uses ffprobe to determine the output file extension. This semantic needs real files.
If anyone else have a good idea, feel free to post it.

I too am running into the need for this functionality. It would be great if there was a way to pass the output file onto another script after downloading. If I'm missing something obvious, please let me know!

Hi everyone, is there anyway I can get the final output filename (correctly) up to now?
ps. I would like the best video+best audio, even if they're always merged into mkv doesn't matter to me

info_dict = ydl.extract_info(url, download=True)
fn = ydl.prepare_filename(info_dict)

This seems can only get the filename "prepared to be", i.e. maybe get something.mp4 while the actual filename is something.mkv

Brute force solution piece for a script:
class MyLogger(object):
    def debug(self, msg):
        match = re.search(r'.ffmpeg..[Dd]estination..(.*?)$', msg)
        if match:
            final_path = match.group(1)

And to cut down on msg:
ydl_opts = {
    'noprogress': True,

this might be a stupid question, but can't I just  'outtmpl': 'file.mkv', since it always falls back to mkv (assuming I don't care about audio-files)?
Or to phrase it differently: are there cases in which youtubedl won't give me a mkv-file?

Uh.. depends on the site/extractor?
I mostly have webm (from YT) and mp4 (others)..

A brave suggestion: Implement an additional (command-line) parameter or alter the existing one to print the true and final filename after all processing happened.
It's not really fun to write special cases for every extension-change that can happen between --get-filename and the process end.

Or — and not having looked at the code, I have no idea how much work it'd be to implement — how about outputting all of youtube-dl's "ok, I'm downloading stuff, here's how it's going" chatter to stderr, and after all is said and done, printing out the final output filename to stdout? It's a nice UNIX-y way to do things that's easy to deal with in just about any programming or scripting language. It seems like a crying shame that, how amazing this program is at grabbing video from so many different sources, that it can't report back on what it just downloaded.

https://github.com/ytdl-org/youtube-dl/issues/5710#issuecomment-369284152: --exec ...


Full invoicing plugin, invoicing module and job refactor

Refactor of invoices

[X] Move Invoices to it's own module
[X] Refactor Invoices to use Job interface
[x] Change plugin according to PR changes in https://github.com/mirumee/saleor/pull/5656

Ready for review, there are two major questions that I pointed out and moved from previous discussion.
Additional notes for reviewers:

You can notice that we are dropping Invoice model from Order app. The reason it was there in first place was that it (model only) was merged before feature branch. It should not contain any data at all since we did not connect it anywhere on master so it should be safe with dropping that and re-creating by 0001_initial.py in invoices module.

Sorry for long PR, jumping between few branches was a nightmare with refactor in progress, so that PR solves SP-629, SP-1512 and SP-1514.

Impact

[X] New migrations
[X] New/Updated API fields or mutations
[X] Deprecated API fields or mutations
[X] Removed API types, fields, or mutations

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Do we plan to have more types that implement the InvoideJobInterface? Maybe we can move the implementation straight into InvoiceJob?

Dropped implementation of InvoiceJobInterface entirely, simplified to use just Job as that suits well after some refactor.

@maarcingebala @fowczarek please check last commit where I add missing tests, otherwise it's all done.


Build Windows and macOS on GitHub Actions / TEST

Motivation:
Central Dogma only uses GitHub Actions for Linux build.
GitHub Actions provides macOS and Windows by default.
There is no reason to maintain two configurations, GitHub Actions and Appveyor.
Modifications:

Mostly fork Armeria's GitHub Actions configuration

Thanks to @minwoox for working on it first

Remove Appveyor configuration

Result:
Simpler CI environment

Codecov Report

Merging #2 (dbdd057) into master (6bfbaef) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master       #2      +/-   ##
============================================
- Coverage     70.01%   70.00%   -0.01%
  Complexity     3330     3330
============================================
  Files           336      336
  Lines         13319    13319
  Branches       1440     1440
============================================
- Hits           9325     9324       -1
- Misses         3099     3100       +1
  Partials        895      895

Impacted Files
Coverage Δ
Complexity Δ

...server/internal/thrift/CentralDogmaExceptions.java
50.00% <0.00%> (-13.64%)
2.00% <0.00%> (ø%)

...centraldogma/server/internal/api/WatchService.java
76.27% <0.00%> (+3.38%)
12.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

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


Distribution packaging

Over at at reuse-tool we're looking into adopting this library for the SPDX handling. As reuse-tool is present in many distributions, we have to be considerate to our distribution packaging maintainers when adopting new dependencies.
At the moment python:spdx-tools is only packaged for NixOS and FreeBSD. This is a practical barrier to adoption. This might be the case for other projects too.
I hope this issue will motivate distribution packaging and can help coordinating it.

I can do the Fedora packaging for this. Not sure when I'll have time, but feel free to poke me when it comes to Fedora.

Ah, interesting. We have not yet thought about maintaining packages on top of pip. Good to know that this is right now blocking adoption. We will discuss that in our next weekly sync.

FTR this package is in Fedora now.
https://packages.fedoraproject.org/pkgs/python-spdx-tools/


Build variants documentation.

Add documentation of the variants of DAPLink target build output.

Thanks for the doc updates! Could have one of the main pages link to this?

Table of contents is a good idea.  As for this, I would link to it somewhere from the developers guide.

@mbed-bot: TEST

Thanks for the doc updates! There no link to the index, but this can be added later.

@c1728p9 Which index do you mean?

I meant there is no link from the main page README.txt to the table of contents in docs/README.md.

Oh, right. That would be useful…


updated login page #99

Issue that this pull request solves
Closes: #99 (issue number)
Proposed changes
updated the login page
Brief description of what is fixed or changed
The problem which i am facing is the nav icons on the login page are not working and also not able to decrease the size of login button.

@Amit366 import  import Login from './components/auth/login/login'; at line 16 of app.js file i.e here

@Amit366 import  import Login from './components/auth/login/login'; at line 16 of app.js file i.e here


Postgres - Add option for high availability

Currently when you use the postgres component it will only have 1 instance in 1 AZ. Would be nice to have options for high availability running in multi-az so it can failover if needed

If you've already posted your issue on Discord, make sure to leave a link to it here.

@fwang figure out priority for this?

@thdxr any updates for this?


PIXI SWC issue

Hi Josh, when I try to compile this example http://nextgenactionscript.com/tutorials/dts2as-typescript-definitions-with-actionscript/ I got an error. In result I've just one line in the js-release/Main.js: //# sourceMappingURL=./Main.js.map.
Here is the compilation logs (same issue on OS X and Windows):
Aymerics-iMac:PixiAS3 Aymeric$ /Users/Aymeric/Documents/Workspace/SDKs/flex_js/js/bin/asjsc --external-library-path+=lib/pixijs.swc src/Main.as
Using Falcon codebase: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/js/bin/../..
Using Flex SDK: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/js/bin/../..
Picked up _JAVA_OPTIONS: -Xms1024m -Xmx4096m -XX:MaxPermSize=1024
Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=1024; support was removed in 8.0
Compiling file: /Users/Aymeric/Documents/Workspace/HTML5/PixiAS3/bin/js-debug/Main.js
using extern: /Users/Aymeric/Documents/Workspace/HTML5/PixiAS3/bin/js-debug/externs/svg.js
using extern: /Users/Aymeric/Documents/Workspace/HTML5/PixiAS3/bin/js-debug/externs/pixijs.js
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/js/libs/js.swc
using SWC: /Users/Aymeric/Documents/Workspace/HTML5/PixiAS3/lib/pixijs.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Binding.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Charts.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Collections.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Core.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/CreateJS.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/DragDrop.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Effects.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Flat.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Formatters.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/GoogleMaps.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Graphics.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/HTML.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/HTML5.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/JQuery.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Mobile.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Network.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Reflection.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/Storage.swc
using SWC: /Users/Aymeric/Documents/Workspace/SDKs/flex_js/frameworks/libs/player/21.0/playerglobal.swc
Dependencies calculated for '/Users/Aymeric/Documents/Workspace/HTML5/PixiAS3/bin/js-debug/Main.js'
Dependencies calculated for '/Users/Aymeric/Documents/Workspace/HTML5/PixiAS3/bin/js-debug/org/apache/flex/utils/Language.js'
mai 26, 2016 9:25:14 AM com.google.javascript.jscomp.LoggerErrorManager println
GRAVE: /Users/Aymeric/Documents/Workspace/HTML5/PixiAS3/bin/js-debug/externs/pixijs.js:3613: ERROR - Parse error. Duplicate parameter name "crossorigin"
PIXI.extras.TilingSprite.fromImage = function(imageId, crossorigin, scaleMode, crossorigin, scaleMode) {};
                                             ^

mai 26, 2016 9:25:14 AM com.google.javascript.jscomp.LoggerErrorManager println
GRAVE: /Users/Aymeric/Documents/Workspace/HTML5/PixiAS3/bin/js-debug/externs/pixijs.js:3613: ERROR - Parse error. Duplicate parameter name "scaleMode"
PIXI.extras.TilingSprite.fromImage = function(imageId, crossorigin, scaleMode, crossorigin, scaleMode) {};
                                             ^

mai 26, 2016 9:25:14 AM com.google.javascript.jscomp.LoggerErrorManager printSummary
AVERTISSEMENT: 2 error(s), 0 warning(s)
The project 'Main' has been successfully compiled and optimized.
``

The TypeScript definitions must have added more overloads of TilingSprite.fromImage() since the last time I tried. It looks like I need to modify parameter names when I merge signatures for ActionScript to avoid duplicates. Thanks for letting me know!

Thanks again! It will be fixed in the next update.

👍


exception when "sbt clean dist"

[warn] There may be incompatibilities among your library dependencies.
[warn] Here are some of the libraries that were evicted:
[warn] 	* org.webjars:jquery:(2.1.3, 1.11.1) -> 2.1.4
[warn] Run 'evicted' to see detailed eviction warnings
java.lang.NoSuchMethodError: spray.json.JsArray.(Lscala/collection/immutable/List;)V
at com.typesafe.sbt.less.SbtLess$$anonfun$2.apply(SbtLess.scala:64)
at com.typesafe.sbt.less.SbtLess$$anonfun$2.apply(SbtLess.scala:55)
at scala.Function1$$anonfun$compose$1.apply(Function1.scala:47)
at sbt.$tilde$greater$$anonfun$$u2219$1.apply(TypeFunctions.scala:40)
at sbt.std.Transform$$anon$4.work(System.scala:63)
at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:226)
at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:226)
at sbt.ErrorHandling$.wideConvert(ErrorHandling.scala:17)
at sbt.Execute.work(Execute.scala:235)
at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:226)
at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:226)
at sbt.ConcurrentRestrictions$$anon$4$$anonfun$1.apply(ConcurrentRestrictions.scala:159)
at sbt.CompletionService$$anon$2.call(CompletionService.scala:28)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
[error] (web-assets:less::jstaskJsOptions) java.lang.NoSuchMethodError: spray.json.JsArray.(Lscala/collection/immutable/List;)V

Not sure what this error is about.  Try cleaning your ivy cache:
rm -Rf ~/.ivy2/cache

And then do a pull on master to make sure you have latest and try again.

Any update here?  Can this be closed?

Sorry, i have try to rm ivy2 cache, and update sbt, it just doesn't work.


Update version and change log for release 3.4.0

What? and Why?
Release for v3.4.0 MBS transform

Codecov Report

Merging #159 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #159   +/-   ##
=======================================
  Coverage   84.72%   84.72%
=======================================
  Files          19       19
  Lines        1021     1021
=======================================
  Hits          865      865
  Misses        156      156

Impacted Files
Coverage Δ

transform/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Is it possible to construct a VCache or VSpace mock value in a pure way for testing purposes?

Mainly asking for https://github.com/dmbarbour/vcache-trie/issues/2, but could be nice for other data types too.

I made the interface a type class in https://github.com/Ericson2314/haskell-vcache. That could help.

@Ericson2314 Hmm, interesting, thanks for letting me know! It's here, right? This may help.
On an unrelated note, I wonder, are the two of us the only ones who use/have used vcache? :)
I wonder how come the adoption is as low as it currently is, i.e. in contrast to acid-state.

I use vcache for EtaMOO, so there are at least three of us.

Yeah, that and the previous commit.
Unfortunately I never got around to actually using VCache for anything. (My hope was to make an alternative backend with vrefs content-addressed to work with something like IPFS, hence the type classes, and needless to say I never finished the project.) It's a shame though, because I did and still think the concept is super good.

@Ericson2314 Super good indeed. I do use it in an app, almost daily, and I quite enjoy it's effortless and transparent persistence. :)
Also, never say never; maybe just haven't yet. ;)

@Wizek well, I think it's important to be honest to oneself and others about stalled hobby projects. Let the presumption be I'll never return to it, and maybe we'll be pleasantly surprised :).

Our team is considering using vcache for the persistence layer of various web applications. The goal is to use Haskell types to define "schema", and have them be compatible with our application layer (i.e. a process vending data over a JSON API). We're not sure how feasible this would be given the high concurrency demands of web apps (LMDB seems well-suited to this task), and the need to eventually scale the persistence layer independently of the application layer depending on process usage. It seems like there would a fair amount of work involved to make this achievable, but the payoff of having type safety from persistence to API could make it worthwhile.

Please take over the project and maintain it yourself if you want to use it.
I mostly abandoned it because Haskell's performance wasn't good enough for my purposes. Not performance of vcache but in other areas - plugins, runtime compilation.

Yeah, I too have mostly abandoned Haskell, partly for performance, and partly for other reasons too. And I miss VCache-like database approaches in other ecosystems.

@dmbarbour Thanks for the update. We are still reviewing various approaches, but I will reach out to you if we decide to use VCache.


Html extraction

Hi, Adrien, awesome library, works like a charm.
In my use case I need not a text, but an html extract. Seems like I look into an API, and issues, but I didn't find any way to get an extract of document not as plain text, but as html containing that text. Am I missing something?

What was I expecting is get this result usign extract(downloaded, output_format='html'). But it just return text, and as I know understand it should raise an exception :)
BTW, it's very confusing that extract(downloaded, output_format='json')
return a string. Yes I know that I can use bare_extraction, but unexpected.
Overall thanks for your work, I could make a PR. If this will be approved.

Hi @slavaGanzin, thanks for your feedback!

You could definitely write a PR to raise an exception for unknown output formats.
HTML as output won't be possible as is because the content is converted to XML internally. However, you could use XML as output format and work on the nodes as you'd do with HTML.
The output has to be a string sooner or later because there are people using the package on the command-line. I didn't think of actually using something else than a string for consistency reasons, maybe that's something we could tackle later.

Thanks Adrien


Update OVA generation code

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [bpradipt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm
code looks more cleaner now
/hold
It will be good if you can fix the author info it shows my name as that machine is configured with a global username and email with mine,

New changes are detected. LGTM label has been removed.

@mkumatag updated it.. ptal

/lgtm
/unhold


[FRONTEND] Design a Text Box and Submit button

Add a Text Box and Submit button as shown in the preview
Here :-

I want to work on this issue under JWOC

Go ahead @aaheli8


[bug] web view not launched in Ubuntu

Describe the bug
While launching the app, a Gdk-ERROR occurred:
(WebKitWebProcess:119326): Gdk-ERROR **: 16:41:25.523: The program 'WebKitWebProcess' received an X Window System error.
This probably reflects a bug in the program.
The error was 'BadValue (integer parameter out of range for operation)'.
  (Details: serial 219 error_code 2 request_code 152 (GLX) minor_code 34)
  (Note to programmers: normally, X errors are reported asynchronously;
   that is, you will receive the error a while after causing it.
   To debug your program, run it with the GDK_SYNCHRONIZE environment
   variable to change this behavior. You can then get a meaningful
   backtrace from your debugger if you break on the gdk_x_error() function.)

Reproduction

Create a new tauri app with pnpm create tauri-app, choosing pnpm as pm and vanilla as ui.
cd tauri-app && pnpm install
pnpm tauri dev, and the error occured.

Expected behavior
No response
Platform and versions
pnpm tauri info:
Environment
  › OS: Ubuntu 22.04 X64
  › Node.js: 16.13.1
  › npm: 8.8.0
  › pnpm: 7.1.2
  › yarn: Not installed!
  › rustup: 1.25.1
  › rustc: 1.63.0
  › cargo: 1.63.0
  › Rust toolchain: stable-x86_64-unknown-linux-gnu

Packages
  › @tauri-apps/cli [NPM]: 1.0.5

(I don't know why it stuck here.) Additional version info:

Cargo tauri: 1.0.5
Cargo tauri-build: 1.0.4
Cargo tao: 0.12.2
Cargo wry: 0.19.0

Stack trace
No response
Additional context
No response

Errors like this are basically impossible to debug (for us/me) tbh, but a few questions anyway:

WSL or an actual Linux installation?
What GPU are you using?
If AMD: Are you using the open-source drivers (that are part of the kernel) or amdgpu-pro?
Are all your system dependencies up-to-date?

Errors like this are basically impossible to debug (for us/me) tbh, but a few questions anyway:

WSL or an actual Linux installation?
What GPU are you using?
If AMD: Are you using the open-source drivers (that are part of the kernel) or amdgpu-pro?
Are all your system dependencies up-to-date?

Edit: Another thing i saw on the internet, can you try setting this env var: WEBKIT_FORCE_SANDBOX=0 ? Probably doesn't work but worth a try :/

Oops, after some apt update and reboot, it works now.
Anyway, the detailed info are:

Actual Linux in physical machine
Nvidia w/ driver 510.85.02
Not up-to-date at that moment


Adds python script for incremental partition insertion.

insert_dynamic_partitions.py A python script to insert partitions incrementally into the partitioned table. Used by the e2e unbounded source testing script.

/gcbrun


8-bit quantization support

I was wondering if it supports 8-bit quantization. The example code is as below.
python main.py \   --model_name "" \   --bits 8 \   --group_size 128 \   --train_bs 8 \   --gradient_accumulate_steps 8 \   --deployment_device 'gpu' \   --output_dir "./save_ckpt"

Hi,
As an algorithm, Autoround could supports 2-8 bits for quantization, even other bits if we change the code a little.  However, we leverage the kernels from other amazing repositories like AutoGPTQ and ITREX for real deployment which may have some limitation for the quantization configures which we have not thought tested, just for example, triton backend in AutoGptq does not support 3 bits.
However, fortunately, I  just tested facebook/opt-125m with your command, and it performed well and the saved model could run on GPU.
One soft reminder is we use the default config "--train_bs 8  \ --gradient_accumulate_steps 1 ",  and the config  "--train_bs 1  \ --gradient_accumulate_steps 8 " for save memory. We haven't tried "--train_bs 8  \ --gradient_accumulate_steps 8 " before

Thanks! i will try.


Feature/update production build github action

Related Issues:

CSBAPP-442

Main Changes:

Add a step to the 'Production Build' GitHub Action workflow to remove the NCES JSON files, as they're fetched from an S3 bucket in production, so don't need to be in the deployed codebase

Steps To Test:

Create a new production build, and ensure the NCES JSON files aren't included.

The change looks good to me, thanks!


Bump y18n from 4.0.0 to 4.0.1

Bumps y18n from 4.0.0 to 4.0.1.

Release notes
Changelog
Commits

⚠ ALL CHECKOBOXES MUST BE CHECKED BEFORE SUBBMITING A PULL REQUEST ⚠

[ ] I confirm that everything below is true
[ ] I understand that I'm not supposed to put solution to this repository
[ ] I confirm that It is not a solution to the task, but fix in task itself
[ ] In case if it is a solution I agree to get -100 points for the task

запрос


🥳[Feature] Parity: Show seed in Settings

Overview
Allow user to  see their seed in Settings
Why?
Is your request related to a problem? Or, do you have an idea to improve the app? Please describe.

#1 problem with Litewallet is people cannot access their seed.
What?
User can go to Hamburger > Settings > Show my seed phrase
Then see this:

User sees seed

Additional context

Done


Fix unhandled promise rejection warning on npm start

Run flow to see type issues, then resolve and see if the problem persists


[4.11.0][Breakpoints]Breakpoints between method provides wrong value for widest screen value

Media query generated by theme.breakpoints.between() are invalid.
Error occurs only when second parameter is widest value.In default theme it is theme.breakpoints.between('sm','xl')

[ ] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Method does not generate media query for widest media screen

Expected Behavior 🤔
Generate media query for widest screen

Steps to Reproduce 🕹

Codesandbox
Context 🔦

Your Environment 🌎

This issue does not occur on latest version of material-ui.Should this be fixed in 4.11.0 version? i would like to fix this if its even in consideration.
Thanks

Tech
Version

Material-UI
4.11.0

This issue does not occur on latest version of material-ui

Do you mean it was fixed in v5? We can close if it's the case. We only backport major fixes.

Yeah OK, it's fixed in v5, we can't fix it in v4, it would be a breaking change.

Duplicate of #13448


Reimplement JPEG and GIF parsers to discard BufReader usage

BufReader is used internally in JPEG and GIF metadata parsers which may lead to surprising results. Ideally immeta should read as little as possible from the stream; if BufRead is used inside, it may read a lot more than necessary.

JPEG can't be reimplemented sensibly without BufReader, and while GIF can be, I've eventually decided to always require BufRead instead of just Read. I don't think this would make any significant inconveniences (any reader could be wrapped into a BufReader), however, this would be a performance win in many cases (lots of formats require skipping through large amounts of data, which is more performant with a buffered reader).
I've also added another method to LoadableMetadata trait, read_from_seek(), which accepts R: BufRead + Seek and by default delegates to read<R: BufRead>(); however, implementations may override it to provide more efficient parsing using seeking.


--with-metadata does not correctly analyze build scripts

I understand this is somewhat of a known issue, but I was a bit surprised that cargo-machete prompted to use --with-metadata for "more accuracy" but when I did that it ended up incorrectly flagging build dependencies (that don't also appear as normal dependencies in Cargo.toml) as unused.
I'm not sure what's considered verboten or in- vs out-of-scope for machete, but there are quite a few approaches that could be used specifically for the case of build dependencies, and you normally have their exact path (either build.rs or the build field from Cargo.toml).
Just some food for thought. Thanks for making this tool :)

+1


Moving tests to use transactions

Attempting to use transactions for tests to try to increase speed / reduce the need to rebuild all fixtures for each test

Whoops a lot of extra errors here. :)

OK - I've got the tests passing locally. With this patch the framework test suite passes in 4.5 mins, without it takes 10.2mins - so we are seeing a 56% improvement in speed. It also saved 8MB of memory (on PHP 7.1 - savings on 5.x will be much higher).
Whilst I appreciate the changing of methods to static is a semver violation, we should try to explore how we can get this into 4

Awesome... how much of it is due to the static fixture factory do you think?

Whilst I appreciate the changing of methods to static is a semver violation, we should try to explore how we can get this into 4

Just make the new static methods a different name, and deprecate the old ones. ;)

How do you feel about moving the entire test fixture system into a TestState? I had originally wanted to do this, but it was going to be a huge amount of work. It would reduce the SapphireTest api surface area a great deal. :)

how much of it is due to the static fixture factory do you think?

I'm not sure - I think it's mostly savings because we aren't trashing, and re-inserting, the fixtures to the DB. I'm sure it's more a DB bottleneck than a YAML parsing one (though it may well be both).

Just make the new static methods a different name, and deprecate the old ones. ;)

that doesn't stop it being a breaking change. Any classes that have overridden the current method will no longer have the effect they used to, therefore breaking people's code.

How do you feel about moving the entire test fixture system into a TestState? I had originally wanted to do this, but it was going to be a huge amount of work. It would reduce the SapphireTest api surface area a great deal. :)

This was what I was thinking too... but it does seem like a lot of work too :P - it's an interesting idea.
This PR at the moment was a semi-proof-of-concept to see just what kind of speed up we could get.

OK - moved to a TestState and made minimal API changes to SapphireTest - I've had to add a few APIs to both SapphireTestState and SapphireTest to allow the FixtureTestState to access the information it needs about tests.

I think I need to do more testing.

@chillu can we get some input on this - it's a real shame that this has missed a large number of builds (especially CWP ones) that could have highlighted any edge cases with this fix (as well as saving a lot of build time)


Reboot step missing in documentation

deploy.ps1 relies on the "az" command being present in the current PATH, which is not true until after rebooting the system following the installation of the Azure CLI module.
If you follow directions as posted, the script will fail with the following error:
### AZURE AD APP CREATION ###
Creating Azure AD App - 'Request-a-team'...
Error occured while creating an Azure AD App: The term 'az' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or
if a path was included, verify that the path is correct and try again.

Thanks @ericpskl - I will update the documentation.


Load Values (FIXME): order after invalidate

Type of change: Clarification.
Explanation:
Constrain the unpredictability of an invalidate followed by a load.

I incorporated the intent of this pull request into the draft spec #34


Reset model also if there is a WSConnectError

fixes #219

@Gerrrr can you take a look? Please also note the discussion in the issue.


Remove version: Jackett.Jackett version 0.20.2614

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.20.2614/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


iOS: add SwiftPM example

And validate that it builds successfully on CI.
Ideally we'd have a whole suite of example integrations that validate all our supported installation methods, but this should be a good start.
I reached out to the owner of https://catfact.ninja to make sure this usage is within the terms of service and will update to use something else if necessary.
https://user-images.githubusercontent.com/474794/171312056-2167333a-fb88-4cfe-aa30-353cceaa49a0.mp4
Signed-off-by: JP Simard<EMAIL_ADDRESS>

Strangely, I can't even resolve the domain catfacts.ninja from my local ISP here.

Strangely, I can't even resolve the domain catfacts.ninja from my local ISP here.

that’s the wrong domain. https://catfact.ninja


[FEATURE] Add like button

It would be nice ti have a like button like below:

For this story please just create the UI, functionality of this button will be discussed on another ticket.

Like button should be visible on hover state.

Like button should be visible if image has like.

User can like an image by hovering the image and clicking the like button

User can like an image by double clicking AND double tapping.

When user likes it, the heart icon should be filled.

Icons to use:

<FontAwesomeIcon icon="fa-regular fa-heart" />
<FontAwesomeIcon icon="fa-solid fa-heart" />

Hey I would  like to contribute to this issue

Hi @1san1  seems like you removed your PR on this. I was planning work on your PR now. Please let me know if you are still interested maybe we can make it work together 💪 If not, I think I will start a new branch and assign to myself.

hello @furkanmutlu  since you were unresponsive throughout the day yesterday I thought my code PR was not up to the expectation and just removed it sorry for that but I restored it if you are still interested.  By the way, it totally got off my mind that it was Christmas yesterday Merry Christmas.


chore: update to latest sdk version

Type of change

Improvement (refactoring, restructuring repository, cleaning tech debt, ...)

Changes
The following changes have been made:

Updated the escrow tests to work with the newest sdk version

Could you add a test with more than 2 assets since there is the use of a vec now?

I modified the deposit_to_two_escrows test to test for this case.  lmk if you think this should be separated into its own test case instead

Could you add a test with more than 2 assets since there is the use of a vec now?

I modified the deposit_to_two_escrows test to test for this case. lmk if you think this should be separated into its own test case instead

The cases that we'd like to cover are:

No assets revert
1 asset
2 assets in 1 escrow
1 asset per escrow, we only need 2 different escrows

This way we can cover unit and system testing

Could you add a test with more than 2 assets since there is the use of a vec now?

I modified the deposit_to_two_escrows test to test for this case. lmk if you think this should be separated into its own test case instead

The cases that we'd like to cover are:

No assets revert
1 asset
2 assets in 1 escrow
1 asset per escrow, we only need 2 different escrows

This way we can cover unit and system testing

when_assets_are_not_specified should already take care of the first test case you mentioned.  Also I think it is appropriate to remove the TODO on line 154 now.
The rest of the tests you mentioned should be covered now in the success mod in create_escrow.  Although I used 3 assets instead of 2 for the 3 test case you mentioned.


Hardware Requirements

Do you NEED to have a micro sd? I would love to try this out but I only have the esp32-wroom-32d.

yes this project depends on the SD Card
also esp32-wroom-32d has no USB support, you need an ESP32-S2


Code Quality: Fixed Sidebar alignment

Resolved / Related Issues
Internal
Steps used to test these changes
Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

Open Files app
Try as many states as possible (ItemCollapse, PaneCollapse, Minimal, etc)

Ready

It looks like we're missing some spacing between "Home" and "Pinned".

Done.
Ready for review.

Is this correct?

/format

Is this correct?

I would say so, if you want the space between Home and the other headers?

I meant in the collapsed mode, it looks a bit off from my viewpoint.


Add nullable

Adds nullable: true to all non-guid response fields. When the list
users endpoint on the Python library, I would receive an error if the
field was NoneType and it was expecting a string. This resolves that
error.
Error
mx_platform_python.exceptions.ApiTypeError: Invalid type for variable 'id'. Required value type is str and passed type was NoneType at ['received_data']['users'][1]['id']

We should be able to set the pythonAttrNoneIfUnset configuration option in our config file to get around this error, right?
https://openapi-generator.tech/docs/generators/python

Yeah, I tried that hoping it would solve it as well. No luck though.

Good to know. :shipit:


problem with deleting fetch's user_data

Hi Andre
I'm trying to send a heap allocated object as user_data to fetch to later use it in the fetch callback. however in fetch callback after I'm done and want to delete the object it crashes. complaints that the object is not heap allocated. is there anything I can do about it?
an example
sending the request
FetchUserData *udata1 = new FetchUserData();
.
.
.
sfetch_request_t fetch_request = {
    .path = udata1->file_name->data(),
    .callback = fetch_callback,
    .buffer_ptr = fileBuffer_,
    .buffer_size = sizeof(fileBuffer_),
    .user_data_ptr = udata1,
    .user_data_size = sizeof(FetchUserData)};
sfetch_send(&fetch_request);

callback function
static void fetch_callback(const sfetch_response_t *response)
{
    auto udata = (FetchUserData *)response->user_data;
    if (response->fetched)
    {
        int img_width, img_height, num_channels;
        const int desired_channels = 4;
        stbi_uc *pixels = stbi_load_from_memory(
            (const stbi_uc *)response->buffer_ptr,
            (int)response->fetched_size,
            &img_width, &img_height,
            &num_channels, desired_channels);

        if (pixels)
        {
            /* initialize the sokol-gfx texture */
            sg_image_desc idesc = {
                .width = img_width,
                .height = img_height,
                .pixel_format = SG_PIXELFORMAT_RGBA8,
                .min_filter = SG_FILTER_LINEAR,
                .mag_filter = SG_FILTER_LINEAR};
            idesc.data.subimage[0][0] = {
                .ptr = pixels,
                .size = (size_t)(img_width * img_height * desired_channels),
            };
            sg_init_image(*udata->image_id, &idesc);
            stbi_image_free(pixels);
            delete udata;     //crashes here!
        }
    }
}

Ah yes, that's a bit awkward. sokol_fetch.h copies the provided user data blob into a small per-request-buffer:
https://github.com/floooh/sokol/blob/89b1b6b1b844fd968537e633a3ded7662bb63641/sokol_fetch.h#L1097-L1099
Also see the explanation here:
https://github.com/floooh/sokol/blob/89b1b6b1b844fd968537e633a3ded7662bb63641/sokol_fetch.h#L256-L264
...so this works differently than typical user_data pointers.
What you need to do is pass a pointer to the FetchUserData pointer to be copied into the user data block, and later copy this out again:
FetchUserData *udata1 = new FetchUserData();
//...
sfetch_request_t fetch_request = {
    .path = udata1->file_name->data(),
    .callback = fetch_callback,
    .buffer_ptr = fileBuffer_,
    .buffer_size = sizeof(fileBuffer_),
    .user_data_ptr = &udata1,
    .user_data_size = sizeof(udata1)};
sfetch_send(&fetch_request);

...note how the address and size of the udata1 pointer is passed to sokol_fetch.h, so 8 bytes (on a 64-bit system) are copied into the user data block.
...and later in the fetch callback extract the original FetchUserData pointer from the request's user data block:
FetchUserData* udata = *(FetchUserData**)response->user_data;
//...
delete udata;

...this should work.

Thanks alot, shame I didn't read the docs properly :p


Fluentd behind haproxy: Massive amount of 'SD' errors (server disconnect)

Versions
td-agent 3, with fluentd gem updated to 1.1.3
haproxy 1.8
Everything on Ubuntu 16.04LTS.
haproxy using tcp forwarding to fluentd backends
Your configuration
Relevant fluentd config:
<source>
  @id                       tls_forwarder_input
  @type                     forward
  port                      2514

  <transport tls>
    cert_path               /etc/td-agent/ca_cert.pem
    private_key_path        /etc/td-agent/ca_key.pem
    private_key_passphrase  passphrase
  </transport>

  <security>
    self_hostname           hostname
    user_auth               true
    shared_key              key
    <user>
      username {{ name }}
      password {{ password }}
    </user>
  </security>
</source>

haproxy config:
global
    maxconn 50000
    chroot /var/lib/haproxy
    log /dev/log local0
    log /dev/log local1 notice
    stats socket /run/haproxy/admin.sock mode 660 level admin expose-fd listeners
    stats timeout 30s
    user haproxy
    group haproxy
    daemon

defaults
    balance roundrobin
    log global
    mode tcp
    timeout tunnel  1h
    timeout connect 30m
    timeout client  30m
    timeout server  30m
    timeout check   10s
    option dontlognull
    option log-health-checks
    option redispatch
    option tcplog
    option tcpka

frontend td-agent_frontend_2514
    bind :2514
    default_backend td-agent_backend_2514

backend td-agent_backend_2514
    server <host> <ip> check inter 5s
    server <host> <ip> check inter 5s
    server <host> <ip> check inter 5s
    server <host> <ip> check inter 5s
    server <host> <ip> check inter 5s
    server <host> <ip> check inter 5s

Explanation
We've got several fluentd hosts configured behind an haproxy using tcp load balancing. Everything seems to work fine, we're doing excellent throughput up to 2 million documents a minute.
That said, we're seeing massive numbers of SD (server disconnect) messages in the haproxy logs. According to the haproxy documentation:

S : the TCP session was unexpectedly aborted by the server, or the
server explicitly refused it.
D : the session was killed by haproxy because the server was detected
as down and was configured to kill all connections when going down.

So it appears that fluentd behind the haproxy is terminating connections early.
This doesn't seem to be interfering with our ingest, but it is troublesome.
Questions

Is this normal?

If this is normal, why?

What might be causing this?
What should I look at to further troubleshoot this?
Are there any fluentd config changes I can make that might improve this situation?
Are there any haproxy config changes I can make that might improve this situation?
Why isn't the client fluentd host making a long-term persistent connection?

hello,@chs-bnet  ,you can try  in_forward Parameters: linger_timeout

True. https://docs.fluentd.org/v/0.12/input/forward#linger_timeout can help it.
also, it's a stale issue. I'm closing.


Fix EnvSort struct

Fixes #627 and unblocks #518
cc: @surajssd @cdrage

Tests(unit tests, functional tests)

Ping @kadel please review.

LGTM other than the nitpick from @kadel 's comment. I agree with adding a few comments.

Thanks @kadel ;)


Update set-output commands

This PR updates the use of set-outputcommand in shipping package. Now uses the new recommended way: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Removes these warnings on the checks:

I don't know if the cached is even used. The output shows the following message both before and after this PR:

Thanks @adevade If we get this in we can always revisit, I think this is something to do with the scope in which the cache is available so might just need some tweaks on the repo itself :)


add readonly to the field

make the select field disabled if the field meta has readonly attribute

@dillingham any plans to merge this PR?


FileNotFound during model.writefiles

Running examples/elegant_fodo_example on 91cd28a3761f334d5a8dec8b74f25316605b63fd consistently results in an error with model.write_files unable to write out the elegant run files in the libEnsemble worker's working directory. I'm not sure how to diagnose this error and trying to make sure os.getcwd did not raise an error before proceeding just resulted in a timeout. I can consistently reproduce this error on my home computer but not on Jupyter. Error from ensemble.log:
[0] libensemble.libE (ERROR): ---- Received error message from 2 ----
[0] libensemble.libE (ERROR): Message: [Errno 2] No such file or directory
[0] libensemble.libE (ERROR): Traceback (most recent call last):
  File "/home/vagrant/.pyenv/versions/py3/lib/python3.7/site-packages/libensemble/libE_worker.py", line 465, in run
    response = self._handle(Work)
  File "/home/vagrant/.pyenv/versions/py3/lib/python3.7/site-packages/libensemble/libE_worker.py", line 435, in _handle
    calc_out, persis_info, calc_status = self._handle_calc(Work, calc_in)
  File "/home/vagrant/.pyenv/versions/py3/lib/python3.7/site-packages/libensemble/libE_worker.py", line 363, in _handle_calc
    out = self._determine_dir_then_calc(Work, calc_type, calc_in, calc)
  File "/home/vagrant/.pyenv/versions/py3/lib/python3.7/site-packages/libensemble/libE_worker.py", line 321, in _determine_dir_then_calc
    return calc(calc_in, Work['persis_info'], Work['libE_info'])
  File "/home/vagrant/.pyenv/versions/py3/lib/python3.7/site-packages/libensemble/libE_worker.py", line 220, in run_sim
    return sim_f(calc_in, persis_info, sim_specs, libE_info)
  File "/opt/project/rsopt/simulation.py", line 84, in __call__
    job._setup.generate_input_file(kwargs, '.')  # TODO: Worker needs to be in their own directory
  File "/opt/project/rsopt/configuration/setup.py", line 232, in generate_input_file
    model.write_files(directory)
  File "/home/vagrant/.pyenv/versions/py3/lib/python3.7/site-packages/sirepo/lib.py", line 58, in write_files
    pykern.pkio.py_path(dest_dir),
  File "/home/vagrant/.pyenv/versions/py3/lib/python3.7/site-packages/pykern/pkio.py", line 130, in py_path
    res = py.path.local(path, expanduser=True)
  File "/home/vagrant/.pyenv/versions/py3/lib/python3.7/site-packages/py/_path/local.py", line 163, in __init__
    self.strpath = abspath(path)
  File "/home/vagrant/.pyenv/versions/3.7.2/lib/python3.7/posixpath.py", line 383, in abspath
    cwd = os.getcwd()
FileNotFoundError: [Errno 2] No such file or directory

[0] libensemble.libE (ERROR): Manager exception raised .. aborting ensemble:
[0] libensemble.libE (ERROR): Dumping ensemble history with 10 sims evaluated:

Does libensemble remove directories? I don't think sirepo.lib removes anything so the current directory should always be valid. Busy today but I'll check it out tomorrow.

I have to ask: why are you running a container on windows?

Mostly I've found the Docker containers work pretty well on Windows, so it's been a good environment to write code in.
If it seems reasonable that this bug might be connected to using Windows I think there is no need to spend time on it unless someone else reports experiencing it.

That's interesting. What about Windows is different than a Mac? I personally don't bother with Docker, because I like working in a VM, which is easier to manage things...

Closing this since it is not a major concern at this point.
I also haven't noticed it happening in quite a while.


feat(bun): add bun sqlite support

Resolves #70
This PR adds support for bun-sqlite. This allows the bun:sqlite driver to be used with Bun when using the file protocol.
There is some difference in the implementation with better-sqlite3:

BigInt support is different, it's not possible to retrieve a bigint from a query : https://github.com/oven-sh/bun/issues/1536
No lastInsertRowId support as there is no equivalent to the info returned by better-sqlite3 run
Argument syntax is more strict and some combinations are not supported
No native support for multiple line statements, therefore multiline statements will use batch under the hood.

Additionally, I did not implemement any of the server related tests, as I believe this driver should only be used for local databases.
There's one outstanding question regarding the bun lockfile which I did not include, and whether we should bother taking care of both lockfiles (This can be done automatically by the CI)
Before moving forward I would appreciate a review 🙏🏽
TODO :

[ ] Run tests with CI
[ ] Add documentation

Thanks for the review @honzasp, I will implement these changes.

I left a few unresolved conversations, waiting for further feedback.
Do you have any thoughts regarding the bun lockfile ? I think it's fine if we don't commit it.
It's really cumbersome to have to duplicate all these tests, however in the future maybe it will be possible to unify them ?
I've deployed this PR because I needed it : https://www.npmjs.com/package/@hebilicious/libsql-client, I can confirm it works (tested with drizzle + bun)

Thanks for the review @honzasp. I resolved all of our conversations. Please let me know if I missed anything.

formatting:  I took the liberty of adding a prettier file that roughly mirror the existing configuration to make it easier to format stuff in this PR.
lockfile: Yes the bun lockfile is similar to the npm one. Tbh as things are pretty new with bun, I am not aware of what the best practice would be in this scenario. I would assume that when running bun install in the ci, the package.json file should be read and understood.
I agree it's not ideal, I did like this to avoid doing a major re-factor of the existing tests and to make sure bun:sqlite was working as intended. The bun:sqlite module has native code bindings that won't work with node. Therefore we need to write accordingly. Bun support for jest is still WIP (?), and to avoid issues and too much refactor in this PR, I went with the RY route. I'm interested in what @penberg has to say, but I think there's 3 options :

keep the node and bun tests separated, but extract all common logic in a testing framework agnostic way.
test everything with the bun test runner and the bun runtime
test everything with jest (or equivalent) with both node and bun runtimes
In the meantime this gives us confidence that the library is working.

To confirm that everything is working further, I deployed a fork of libsql-client-ts that I'm using with drizzle and bun https://www.npmjs.com/package/@hebilicious/libsql-client
import { drizzle } from "drizzle-orm/libsql"
import { createClient } from "@hebilicious/libsql-client"

const url = process.env.DATABASE_URL ?? "file:drizzle/local.db"
const authToken = process.env.DATABASE_AUTH_TOKEN

console.log(`Connecting to ${url}...`)
export const client = createClient({ url, authToken })
export const db = drizzle(client)

Hey @Hebilicious, please do add Bun to CI (otherwise we'll just end up breaking it) and also feel free to update the README.

Hey @Hebilicious, please do add Bun to CI (otherwise we'll just end up breaking it) and also feel free to update the README.

Hi @penberg, I will update this.
Do you have any insight regarding the tests themselves ?

I agree it's not ideal, I did it like this to avoid doing a major re-factor of the existing tests and to make sure bun:sqlite was working as intended. The bun:sqlite module has native code bindings that won't work with node, therefore we need to write the tests accordingly. Bun support for jest is still WIP (?), and to avoid issues and too much refactor in this PR, I went with the RY route. I'm interested in what @penberg has to say, but I think there's 3 options :

keep the node and bun tests separated, but extract all common logic in a testing framework agnostic way.
test everything with the bun test runner and the bun runtime
test everything with jest (or equivalent) with both node and bun runtimes

In the meantime this gives us confidence that the library is working. I personally think we should go with
1, as it will be more friendly to other runtimes such as Deno... Once a solution has been chose, I'm happy to do the refactor.

@Hebilicious I agree that we should keep Bun tests separate. Presumably in the future Bun gets compatible enough that the separate suite just becomes redundant and we'll switch to one suite.

Btw Vercel tests were using an expired token, which is sorted now. For Cloudflare, there seems to be some issue with the Hrana remote protocol, so I disabled the tests for now https://github.com/libsql/libsql-client-ts/commit/37382a773207e6a1d5a36b3c20e0e33f9c37a772
Both unrelated to this PR of course

@Hebilicious Btw, I switched from better-sqlite3 to a new libsql package today https://github.com/libsql/libsql-client-ts/commit/fb19cb7d31b99ce5fa4edac847d1648b0ecdcd13. And it seems to be working fine with Bun now that i tested it:
https://github.com/libsql/libsql-node/issues/10#issuecomment-1708250334
Do we still need to support bun:sqlite driver after the switch to libsql?

@penberg That's great !
For my use case, I would personally still prefer to usebun:sqlite given the choice, as it's written natively for bun and therefore should be more performant.
On a side note, I am a little bit worried about the potential confusion between libsql-client-ts and libsql. Perhaps moving them to the same package/repo/monorepo would be a good idea 🤔 ?
Regarding the tests, I have a suggestion : Now that bettersqlite3 has been replace by libsql, how about we update the test suite to use the Bun test runner (instead of Jest), which would make it significantly easier to test the bun:sqlite driver, while also massively improving the CI speed ?

I've tested the @hebilicious/libsql-client package and worked great. What need to be integrated in main repo?


ADOT EKS add-on Getting Started Guide Update

This PR updates the Getting Started Guide. Updates include:

updating the configs + related descriptions to match changes made in this PR
Call outs of the ClusterRole and ClusterRoleBinding used in some of the configs

From @Aneurysm9:

What does <YOUR-APPLICATION-METRIC> do? What if there are multiple metrics from the application? Isn't this going to scrape a lot of metrics only to throw most of them away?

This will indeed do the work to scrape a lot of metrics and <YOUR-APPLICATION-METRIC> will select one of them, throwing the rest away. This is because it matches up with the config used in this blog post, however for the purposes of the guide, I think it's better to remove the selector so a user can see all the metrics from the application.


[VL] Move all columnar rules to post-columnar transitions

This patch moves all columnar rules to post-columnar transition API. This works like we'll inject Gluten's custom rules to Spark's preparation rule list rather than in to columnar rules.
By doing this, we can treat all of Gluten's columnar rules as an individual Spark preparation rule that works on a complete Spark plan with C2Rs and R2Cs added . So we get full control of plan conversion rather than being clamped by Spark's rule ApplyColumnarRulesAndInsertTransitions. This will help on continuously improving Gluten's plan optimization capability and to make it not only limited to doing columnar conversions.

/Benchmark Velox

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_4790_time.csv
log/native_master_02_26_2024_6acd1b367_time.csv
difference
percentage

q1
32.75
33.25
0.493
101.50%

q2
24.72
24.24
-0.479
98.06%

q3
38.86
38.32
-0.535
98.62%

q4
37.76
37.90
0.144
100.38%

q5
68.52
72.46
3.937
105.75%

q6
7.22
6.95
-0.268
96.29%

q7
83.11
83.72
0.611
100.73%

q8
86.19
86.53
0.346
100.40%

q9
122.79
127.60
4.812
103.92%

q10
44.70
47.09
2.387
105.34%

q11
20.37
20.47
0.103
100.51%

q12
28.33
28.59
0.269
100.95%

q13
46.07
47.07
0.998
102.17%

q14
16.91
18.34
1.430
108.46%

q15
27.27
27.73
0.458
101.68%

q16
16.31
14.65
-1.661
89.81%

q17
103.08
101.67
-1.413
98.63%

q18
148.40
147.95
-0.448
99.70%

q19
13.70
15.14
1.434
110.46%

q20
26.66
27.29
0.631
102.37%

q21
227.09
228.47
1.378
100.61%

q22
13.56
13.67
0.108
100.80%

total
1234.35
1249.09
14.733
101.19%

/Benchmark Velox

@rui-mo @PHILO-HE

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_4790_time.csv
log/native_master_02_29_2024_22d9fe3c8_time.csv
difference
percentage

q1
32.79
33.44
0.659
102.01%

q2
24.40
27.05
2.650
110.86%

q3
38.23
35.95
-2.278
94.04%

q4
37.87
39.22
1.345
103.55%

q5
71.98
71.30
-0.682
99.05%

q6
6.88
7.43
0.550
108.00%

q7
84.42
85.83
1.412
101.67%

q8
85.77
85.20
-0.566
99.34%

q9
125.77
118.37
-7.399
94.12%

q10
43.03
43.82
0.782
101.82%

q11
20.86
20.13
-0.723
96.53%

q12
26.05
26.26
0.206
100.79%

q13
45.48
46.41
0.931
102.05%

q14
18.72
20.53
1.814
109.69%

q15
29.02
28.82
-0.206
99.29%

q16
12.92
12.90
-0.020
99.85%

q17
101.32
103.50
2.178
102.15%

q18
150.17
149.47
-0.709
99.53%

q19
15.42
12.55
-2.871
81.39%

q20
26.32
28.87
2.555
109.71%

q21
224.29
227.08
2.787
101.24%

q22
14.71
13.78
-0.933
93.66%

total
1236.44
1237.92
1.483
100.12%


squashfsTools: enable strictDeps

adds -n to gzip###### Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 170433 run on x86_64-linux 1

3 packages marked as broken and skipped:

authy
markmind
timedoctor

1 package blacklisted:

appimage-run-tests

1 package failed to build:

udocker

84 packages built:

Sylk
altair
anytype
appimage-run
appimagekit
apple-music-electron
beekeeper-studio
betterdiscord-installer
binwalk (python39Packages.binwalk)
blockbench-electron
bloomrpc
buttercup-desktop
caprine-bin
chrysalis
cider
cozy-drive
crypto-org-wallet
davinci-resolve
devdocs-desktop
diffoscope
diffoscopeMinimal
distrobuilder
electron-mail
electronplayer
firefly-desktop
flexoptix-app
freetube
fspy
heroic
irccloud
jitsi-meet-electron
joplin-desktop
keeweb
lbry
ledger-live-desktop
lens
logseq
losslesscut-bin
lunar-client
lxd
marktext
mathpix-snipping-tool
mobilecoin-wallet
molotov
mycrypto
neo4j-desktop
notable
notion-app-enhanced
nuclear
octant-desktop
p3x-onenote
pcloud
plexamp
python310Packages.binwalk
python310Packages.binwalk-full
python39Packages.binwalk-full
radicle-upstream
rambox-pro
ripcord
saleae-logic-2
session-desktop-appimage
singularity
sonixd
soulseekqt
spotify
spotify-unwrapped
spotifywm
squashfsTools
ssb-patchwork
standardnotes
station
super-productivity
termius
timeular
todoist-electron
trezor-suite
tusk
unityhub
via
vial
wootility
ytmdesktop
zettlr
zulip


[SRP] 2021-06-01 Swagger Api

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@Himanshg, please create a new PR following this item (and check the box when you submit): [ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. This is very important to get an quality and efficient ARM review.

@Himanshg, please create a new PR following this item (and check the box when you submit): [ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. This is very important to get an quality and efficient ARM review.

@mentat9, this is exactly how this PR has been prepared. Below the very first commit which is the directory copy from older version to the new version.
https://github.com/Azure/azure-rest-api-specs/pull/15627/commits/7e0f2751aed1705d5a1857979c629198f645e3fb

All the new commits thereafter are additional changes only for the new version as separate commts

@Himanshg Himanshu Gupta FTE, please create a new PR following this item (and check the box when you submit): [ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. This is very important to get an quality and efficient ARM review.

I don't see this message being ack'd, we cannot proceed with the API review until you follow these steps.

There is a cross version breaking change , please follow the action https://github.com/Azure/azure-rest-api-specs/pull/15627#issuecomment-906986199  to get approval


Rosepine Theme

Rosepine Theme
https://github.com/rose-pine/steam-os
Checklist:

[✓] I am the original author of this theme.
[✓] I am not bundling a part of another theme with my theme, to encourage mixing and matching themes
[✓] I have verified that my theme works properly on the latest versions of SteamOS, decky-loader and SDH-CssLoader.

Theme does not follow the file structure that is expected. Will not merge until fixed
Also from a quick peek at the theme.json you don't specify a target. This is required

fixed

Exception: Theme has no author
Please read trough the entire readme and see if you have everything


Local discovery doc generation

Note that this PR currently contains the commit for custom URLs as well, since I wanted to have that change in for discovery doc testing.

This PR has been cleaned up so that the custom URL stuff is no longer in it.

Current coverage is 79.88% (diff: 89.88%)

Merging #23 into master will increase coverage by 0.93%

@@             master        #23   diff @@
==========================================
  Files            34         36     +2
  Lines          4562       4987   +425
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3602       3984   +382
- Misses          960       1003    +43
  Partials          0          0

Powered by Codecov. Last update dd4ac75...ea8fe15

This PR now hooks up the local discovery generation to the framework, so that requests are actually handled locally from both endpointscfg.py and from HTTP requests.


Does this work without ionic?

I'm trying to figure out how to use WKWebView with Cordova and stumbled with this plugin.
Does it work with pure Cordova (no ionic)?

I don't see why not, did you try it?


The great evaluation_metrics_row refactor!

Right now, evaluation_metrics_row is kludgy and tries to do way too much stuff, which makes it (a) hard to know exactly what your outputs relate to and (b) hard to customize any internal subfunctions without rewriting the entire loop (or threading some new param everywhere). It also combines a bunch of different types of output metrics into a single hard-to-parse schema (EvaluationRow). We'd like to refactor this!
Plan (as devised w/ @ericphanson): Split current evaluation_metrics_row function into three separate (types of) functions:

a function that covers everything not dependent on a specific threshold OR on predicted_hard_labels (e.g., roc curves, pr curves, some calibration curves maybe)
function(s) that choose a threshold, given the output of step 1 - this would also allow choice of specific desired threshold, e.g., thresh for given specificity OR thresh based on calibration curves (default now) OR thresh based on ROC curve minimization (other default currently!)
functions that calculates derived stats that are based on hardened predictions (e.g., confusion matrix, ea kappas, etc) (multiclass AND per-class rows); two methods:

(convenience option) takes a hardening option + threshold + observations, compute predicted_hard_labels internally
takes predicted_hard_labels, goes from there

What is the output if you call all three steps??

Currently, we return a single EvaluationRow, which contains threshold-dependent AND threshold-independent metrics, and where each field contains results for all classes AND (when valid) a multiclass value. (i.e., some fields are per class, some fields are multiclass, some contain both; some fields threshold dependent, some are threshold independent
After refactor, those should all be separate things:

Should have separate schemas for threshold-dependent (from step 3) and threshold-independent statistics (from step 1)
Each class should have its own output row(s) - #row
Multiclass stats should be its own row - confusion matrix & multiclass expert-agreement kappa (if valid)

Other important points:

This doesn't necessarily account for specialized hardening (a la #68), but would make it clearer to implement. What needs a hardening function? Step 1 and the convenience option of step 3.
As much as possible we should reorganize the loop structure so that instead of doing each computation over all classes before moving on to the next metric, we instead do all computations for a single class and then iterate over all classes at the outer loop
make sure docs are clear about what values do and don't depend on predictions (i.e., ira does not!)
Can make a breaking release with all of these changes; depending on how much time it takes, can add a deprecation path to construct the current EvaluationRow from new row types (might be useful for testing that change adds no functional change, otherwise file it for later)
How does this interact with plotting? Plotting is its own special hell that we'll likely want to refactor also, but for now,

I think one nice benefit here is that it'll be easier to use inspect results with several thresholds. For example, I could make a report with first threshold-independent metrics like ROC curves, PR curves, etc. Then I could show metrics with several choices of threshold, e.g. "sensitivity >= s1" for a few choices of s1, and "minimize calibration error", or "max sensitivity * (1-fpr)" (i.e. from the ROC curve).


EXSWHTEC-75 - Implement tests for hipMemcpyAsync and derivatives

Implement positive and negative unit tests for the following APIs:
hipMemcpyAsync
hipMemcpyDtoHAsync
hipMemcpyHtoDAsync
hipMemcpyDtoDAsync.

@milos-mozetic / @music-dino : Can someone look into this PR. The previous resolution of the merge conflict https://github.com/ROCm-Developer-Tools/hip-tests/pull/18/commits/1ef552aa3eabebb0d8c23c67e22069b5b4372ae8 looks incorrect. There are some remnants of the merge conflict that can be seen in hipMemcpyAsync.cc.

@milos-mozetic / @music-dino : Can someone look into this PR. The previous resolution of the merge conflict 1ef552a looks incorrect. There are some remnants of the merge conflict that can be seen in hipMemcpyAsync.cc.

@mangupta: Sorry for the inconvenience, now the conflict is resolved appropriately within commit https://github.com/ROCm-Developer-Tools/hip-tests/pull/18/commits/dc236b82908c8c80f7bf7850656c516e1720b380. @music-dino: can you please double-check that the content of hipMemcpyAsync.cc is as expected from your side?

@milos-mozetic / @music-dino : Can someone look into this PR. The previous resolution of the merge conflict 1ef552a looks incorrect. There are some remnants of the merge conflict that can be seen in hipMemcpyAsync.cc.

@mangupta: Sorry for the inconvenience, now the conflict is resolved appropriately within commit dc236b8. @music-dino: can you please double-check that the content of hipMemcpyAsync.cc is as expected from your side?

Any update?

@milos-mozetic / @music-dino : Can someone look into this PR. The previous resolution of the merge conflict 1ef552a looks incorrect. There are some remnants of the merge conflict that can be seen in hipMemcpyAsync.cc.

@mangupta: Sorry for the inconvenience, now the conflict is resolved appropriately within commit dc236b8. @music-dino: can you please double-check that the content of hipMemcpyAsync.cc is as expected from your side?

Any update?

The contents seem to be correct. Please proceed with the merge.

@music-dino
Will you please resolve the merge conflicts for this PR?


ValueError: [dequantize] The matrix should be given as a uint32

After quantizing mlx-community/miqumaid-v3-70b with this command mlx_lm.convert --hf-path miqumaid-v3-70b --mlx-path miqumaid-v3-70b-4bit -q --qbits 4.
The model miqumaid-v3-70b-4bit cannot be inferred with mlx_lm.server. ValueError: [dequantize] The matrix should be given as a uint32 will show up.

I just downloaded NeverSleep/MiquMaid-v3-70B and quantized it and it seems to run fine. Perhaps there is something wrong with the unquantized download?

I just downloaded NeverSleep/MiquMaid-v3-70B and quantized it and it seems to run fine. Perhaps there is something wrong with the unquantized download?

Thanks for testing. But NeverSleep/MiquMaid-v3-70B is not the model I downloaded, I downloaded the mlx converted (unquantized) mlx-community/MiquMaid-v3-70B and tried to quantize it, but unsuccessful.

I see, did you dequantize first? Even though it is not in the name, that model is actually quantized at 8 bits.

I see, did you dequantize first? Even though it is not in the name, that model is actually quantized at 8 bits.

Thanks for pointing out it was already quantized, I didn't notice it. I requantized, quantize again with 4bit and it worked. Thanks again for your generous help!


Move emoji filter to own file

This belongs in it's own file
I'd also like to make other tweaks:

Rename to ReveakCK::Builder::EmojiFilter?
Ok to move align='absmiddle' to css?

Thoughts?
fyi jch/html-pipeline#195

Interesting, align='absmiddle' is not consistent across browsers. ref
Some people implement with:
img.emoji {
  display:table-cell;
  vertical-align:middle;
}

or
img.emoji {
  margin-bottom:.25em;
  vertical-align:middle;
}

I did notice that <small>x</small> and :emoji: were not aligning correctly for me.

Hi again!
Moving to its own file is a good idea-- not sure why I left it right there.
If it is not renamed, then it should live over at ./lib/html/pipeline. But if it is renamed, this is a good resting place. I don't see any reason to not rename it, so long as it can still inherit from that base filter and everything (tests, rubocop) still passes.
Regarding CSS-- and align="xxx" moving out makes sense, right? If the styles aren't inlined then people will have more control, eh? If I recall correctly reveal-ck defines its own CSS over in this file.

I was going down this path (re: own file) over in this branch as I initially considered your request: https://github.com/jedcn/reveal-ck/compare/emoji-and-the-alts?expand=1. Which is to say-- lets add some tests as part of breaking it out (even if we don't change its behavior).

As for next steps, unless I hear objections / further activity, I will:

Merge this change and do the rename to RevealCK::Builder::EmojiFilter
Bring in the specs (for existing behavior-- not new) I wrote from the emoji-and-the-alts branch
We'll treat the CSS separately.

Looking at all filters out there, they all keep in the correct namespace.
Don't like it, and a few issues over there are concerned about it.
But it seems like the way people are doing it.

I agree this belongs over in html/pipeline/reveal_ck_emoji_fulter.
If you want me to move here, let me know, otherwise just close.
Your other branch does a better job w/ specs and all.

Thanks. I took the commit you had here.. moved the file over to lib/html/pipeline, and added the specs from emoji-and-the-alts (since that won't get merged now).
I included your commit (rather than just using the stuff from emoji-and-the-alts) to make sure you ended up on the contributors page.

emoji filter has been changed. you may want to just overwrite emoji_image_tag


Allowing normal users to create repositories with LDAP authentication?

I'm kind of stuck at what to do here. I would like to allow users to have a non-admin account on gitblit, but still have the ability to create personal repositories. Is there any way to support this with ldap? Trying to enable it through an admin account results in a "Failed to update user." error thanks to the LDAP.

This should work, as long as you have one (or more) groups that all users belong to and to which you can give the can create privilege (for the team in Gitblit)

I have a team "developers" which I tried to give create priveleges to in gitblit's ui, and I got "Failed to update team!" from it. Is there a special way to do this?

Oh, don't know. Anything interesting in the server log? I think it worked for me through the UI, otherwise you can edit the users.conffile
[team "developers"]
        role = "#create"

Ah it looks like I had messed up the permissions for my system while transferring my gitblit to a new OS. Sorry about that.


auth/authorize login redirect not working

Hi,
the documentation says if the user is not logged in when opening auth/authorize he will be redirected to login. But it looks like this is not working, the server returns a 500 error code.

Should be fixed now.

Still not working for me

@RainerZA Does it work for you if the user is already logged in? I have tried it both with an already logged in user and with no logged in user and both appear to redirect fine for me.
I tried with both authorization pin and authorization code.

works if the user is already logged in, but if not I get 500


Update from upstream 2022 02 09

This PR updates from upstream to fix https://github.com/Kong/charts/pull/542
THIS PR SHOULD USE "CREATE MERGE COMMIT"

Yea Gateway API is great, but I'm unsure why this does not work in upgrade tests https://app.circleci.com/pipelines/github/giantswarm/kong-app/715/workflows/f499d28c-8684-40be-b6f1-63304f0e696b/jobs/952
maybe somethings fishy with creating/deleting resources
@webwurst do you see anything that I could be doing wrong? https://github.com/giantswarm/kong-app/blob/add-test-with-plugins-with-fix/tests/ats/test_basic_cluster.py

looks like kubecel apply --filename test-ingress.yaml fails with that?

admission webhook "validations.kong.konghq.com" denied the request without explanation

Looks like we're hitting https://github.com/Kong/go-kong/pull/81


Qim Info

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


Remove unused var in NativeAnimatedNodesManager

Should have been removed in https://github.com/facebook/react-native/commit/6f5433febe2c6dab5cc7b5799db7b95fc9992a11#commitcomment-19793695.
Simple enough fix.
cc/ @janicduplessis

@facebook-github-bot shipit


enzyme mount() fails for even a

Hi,
I have the following versions:
"react": "16.0.0-alpha.12",
"react-dom": "16.0.0-alpha.12"
"enzyme": "^2.9.1",
"jest": "20.0.4",
"jsdom": "^11.1.0"
I have initialized jsdom as mentioned in the enzyme docs.
And I get this error:
TypeError: Cannot read property 'getPublicInstance' of undefined
at any attempt to call mount:
mountedMulti = mount(

             );
I tried everything, searched all over the internet, but could not get around this.

enzyme 2 does not support react alphas or betas; you'll have to wait for v3 before you can use React 16 with enzyme.


Give up checking tag query key syntax statically

Static syntax checking of tag query keys (field names like #e in filters) has relied on list of possible (standardized) single-letter tag names. Managing such a list as a part of this library deems unreasonable because Nostr protocol evolves too quickly.
Instead, this PR make fetcher methods assert that a given filter doesn't have invalid tag query (e.g. #foo). Further, it make fetchLatestEventsPerKey() / fetchLastEventPerKey() warn when otherFilter have field that is also specified as main key since the filter condition specified by that field will be ignored in that situation (Formerly, situations like this are guaranteed not to occur by static type checking).

resolves #325.


fixed referenced entries publish

@shafeeqd959 , Security-scan failing due to tmp library. @cs-raj already merged the changes in development #1318 . We need to do the same in staging branch.


Add CNHC token

Project team background:
CNHC team is based in different countries such as China, Singapore, the United States, Brazil and Japan. Core team members are mostly from top Wall Street hedge funds, leading blockchain investment institutions, etc, with over 10 years of experience in the Internet, foreign trade and financial industries. Currently, Cnhc Group has launched a digital currency linked to offshore RMB.
Project basic information:
CNHC is a crypto currency that is CNH-pegged. Each CNHC is backed by 1 offshore RMB (CNH), in compliance with regulated U.S. trust and banks for fund custody. Audit reports will be published by an independent audit firm on a regular basis. CNHC is supported by top international investment institutions and Internet giants, and will be promoted globally, including but not limited to application scenarios such as international payment, global trade, and decentralized finance.

Website: https://www.cnhc.to/
Medium: https://medium.com/@cnhcto
Facebook: https://www.facebook.com/CNHCto
Twitter: https://twitter.com/CNHC93231472

Media publications:
Tradeable exchanges:
Recommended Gas Limit for transaction: 60000

为确保资料提交的真实性，请在贵方官网或其他经过认证的社交媒体渠道发布公告，并将此公告链接附于「官方公告」。推荐文案：
#imToken# #1636# 我们正在完善 XXX 在 imToken 中的代币相关资料。完成后，大家可以在 imToken 里看到 logo，更方便地管理 XXX，同时能看到更多代币信息。
注：

1636 为 pr 序号，如 https://github.com/consenlabs/token-profile/pull/1636 的序号为 1636，这是帮助 imToken 判断 pr 真实性的关键。
#imToken #1636 标签及其展示形式可自行调整。
若未提供「官方公告」，则无法显示代币信息。

https://twitter.com/CNHC_TO/status/1368772515082866690

非常抱歉，该官方公告Twitter与官网中账号不一致。

非常抱歉，该官方公告Twitter与官网中账号不一致。

sorry, updated.


Graph Data Modeling Using ArangoDB

[ ] 3.0
[ ] Windows

I have a business that very similar with a part of Telegram App:
multiple Users that have list of Contacts, and each user is member of multiple Group. each Group has a Admin , members.
part of my design:
a Doc Collection with Name User. a Edge Collection with name Contact( that two filed _from , _to fill out from User Id; for example  _from:User/1, _to:User/2).
1. this design is correct?!
2. for Group, I need a Edge Collection or Doc Collection?
I was guess two solution:
-Group is a doc collection that had name and admin field. Group's Users also save as a array filed in Group Doc Coll...

Group is a doc collection that had name and admin field. Group's Users save as a array filed in User Doc Coll...
which one? neither?!

I think I'm so confused about Graph data modeling! when and where use Edge Coll or Doc Coll?
my case similar below photo:

We try to explain that in our documentation: https://docs.arangodb.com/3.0/Manual/Graphs/index.html
In general ArangoDB isn't as strict in what data you're able to attach to vertices and edges as other solutions in the field. Vertices is the name for the items that you call doc col - thus in graphs they're aliased Vertex Collections. In ArangoDB Edge collections are regular collections with the two mandatory _from and _to attributes.
In a more traditional SQL way you would map m:n relations using pivot tables. You can think of edge collections as being similar. The edge usually is a relation between two objects. So if you have one collection of Users, and another collection of groups, you then would have an edge collections usersToGroups, in which you would have edges making the user a member of the group. Attributes to such an edge would i.e. be the privilege status in the group, the join date, etc.
The group document would i.e. have the name attribute, maybe a group topic, things that are common to the group, and not to its individual members.
Let me know if you like the explanations in the documentation, and if not what you've got problems with to comprehend from it.

@dothebart: with your comment with the documentation solved my problem.... thanks a lot

We added an example to the docs: https://github.com/arangodb/arangodb/commit/1c95b6829691f2a63e2ea8154b4b4c9caa1c097e


Can't change docker registration once I entered one successfully

Rancher versions: 2.0 master 4/17
Steps to Reproduce:

Create a pipeline and register into docker in the publish section
Create another pipeline and go to publish section

Results:  Since I'm already registered, it doesn't show that option anymore, but if I want to send to a different registry I can't do that.  We should make it so that they can switch (maybe have a button to bring back?)

It works now, I am not sure what's wrong before, maybe something wrong with the registry module. But it should work fine now.


Changed hover color for all links in light mode

#703 Issue Solved
I have changed the color when hovering over any link it gets white in color which was unable to read so I change the color.

Thanks for the contribution @MOHDNEHALKHAN but this will not solve it. I am closing the PR.
You changed the compiled css file, not the original configurations for the theme. Example: if I approve this one, my ci/cd pipelines will overwrite that file.


Users: If a user has forgotten their password, they can request a reset through email

Have a link on the login form which will enable a user to request to change their password if they have forgotten it. Upon clicking the link, they will be prompted to enter their username or email.
An email will be sent to them which includes a link they can follow to enter a new password.

Nudging this enthusiastically. I have a client emailing every other week to reset their password.

Bumping this to priority as well.

From the message in #1174, would this require a project-level config for apostrophe-email with a nodemailer account? Or is still still in process?

This feature is complete. There is a HOWTO.
On Mon, Jun 4, 2018 at 1:11 PM, Alex Bea<EMAIL_ADDRESS>wrote:

From the message in #1174
https://github.com/apostrophecms/apostrophe/issues/1174, would this
require a project-level config for apostrophe-email with a nodemailer
account? Or is still still in process?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/apostrophe/issues/288#issuecomment-394429628,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fTyWSP5QKfD2S3q4FFU1ng_dyB24ks5t5WpBgaJpZM4FUtS2
.

--
THOMAS BOUTELL, CHIEF SOFTWARE ARCHITECT
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

I see. https://apostrophecms.org/docs/tutorials/howtos/password-reset.html


Python3 support

pg2kinesis sould work with python3,
opened https://github.com/handshake/pg2kinesis/pull/10 to resolve.

https://github.com/handshake/pg2kinesis/pull/10

💯 thanks!


validate initial template values

Validates initial values and adds feedback form fields.
Solves https://github.com/cyclops-ui/cyclops/issues/130

rc version:
kubectl apply -f https://raw.githubusercontent.com/cyclops-ui/cyclops/v0.0.1-alpha.16-rc.1/install/cyclops-install.yaml

This looks good 👍
We should also add the same validation animation to the module name!


Match box border bg with box bg color

fixes https://github.com/rivo/tview/issues/519
Before change:

After change:

Notice the background color of the border around each frame box and the modal.
One mystery: if I checkout https://github.com/rivo/tview/commit/53d50e499bf914fbd2873aa7d8b1bcb6427dde0e and run the presentation demo, the modal color is a dark blue.  Unsure why it changes color, maybe something with tcell upgrade?
Unsure if this is the best way to fix this problem, so open to suggestions.  I guess an alternative would be to revert the switch to using the borderStyle property?  Cheers!

Sigh 🤦


fix: Use env variable name in calendar invite

Hey Tim, thanks for making this repo!
Found an instance of "Tim" instead of the user's name in the application. This pull request swaps that out for the environment variable instead. Cheers!

Thank you so much!


fix: fix a typo in shovel docs

@rauno56 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@rauno56 Thank you for signing the Contributor License Agreement!

Thank you!


nodemailer: use 1.x API instead of 0.x

nodemailer was asking to downgrade otherwise.

@cgestes Thank you very much for the PR! I will be working on a new release over the next couple weeks and will be handling all pull requests then.