Fade out previous slide

Here's the bug in question for #1342 : http://jsfiddle.net/7pvu8gv2/
Fixed: http://jsfiddle.net/m4dL92nq/
Another manifestation: http://codepen.io/ethanclevenger91/pen/PqNxOO
Another fixed: http://codepen.io/ethanclevenger91/pen/jPqQxQ
To say this was rigorously tested might be an overstatement. Let me know if updates should be made.

@ethanclevenger91 ... could you take a quick look over: https://github.com/kenwheeler/slick/pull/1344 ( https://github.com/nominalaeon/slick/commit/73c30873295b450f32a417f318b20940bee7da59 ).. does your PR mean that these z-index changes are unecessary, also you can see from @nominalaeon s commits that there's more z-index references than the ones you have erased, could you make sure those other z-index references are not conflicting, or affecting your PR please.
:smile:
Thanks , Simon.

the setFade() function (line 1665) is what makes fade transitions work if you're using the setPosition method to jump between slides. There the z-index ensures that the incoming slide is definitely behind the outgoing no matter what their mark-up order.
It does look like z-index is benign in both setFade() and fadeSlide() but, like I mentioned in #1344, I only noticed the z-index stuff in IE. The z-index declarations are probably just harmless safe-guards for IE compatibility. Chrome and FireFox are deceivingly competent in this department.
But someone's going to have to test in IE8 or 9 to confirm and I'm calling not-it right now.

 No reason I couldn't/shouldn't just revert. Then you guys can continue to address #1344 however you'd planned. Does that make sense?
If we want to be sure, I should have a VM handy for testing IE8.

The fix for #1344 is to make zIndex an arg you can pass in. There might be potential use cases for controlling the slide z-index, but that kind of functionality is totally scope creep for a slider.
If the transitions work fine without z-index being set for IE8 (or just IE9? Ken or Simon would have to speak to that), it would be worth dropping them out from both functions.

Hi @ethanclevenger91 , actually that's not what I was expecting :smile: -- your code addresses a rather prevalent issue which keeps getting raised, so I was hoping to merge it in! :smile:
But I wanted to make sure we cover those extra few z-index assignments:

https://github.com/ethanclevenger91/slick/blob/629cd590bdd88cbf58589a0da6b24a851e3d9044/slick/slick.js#L1677
https://github.com/ethanclevenger91/slick/blob/629cd590bdd88cbf58589a0da6b24a851e3d9044/slick/slick.js#L1685
https://github.com/ethanclevenger91/slick/blob/629cd590bdd88cbf58589a0da6b24a851e3d9044/slick/slick.js#L1692

If the z-indexes are un-needed now, we can remove them...
@nominalaeon PR is to allow control of the z-indexes... so if your PR removes the z-indexes, then there's no need for #1344. :) .....
Thus my question is: Could you investigate removing those other zIndex in your branch? :bow:
Si.

I will, actually I'll just take them out and see if the issue we had on our site is still resolved. brb

Cool, thanks so much! -- and don't forget to test all the scenarios where setPosition() is called :disappointed_relieved:

:\ Well without the z-index the slides do not cover my dropdown menu, however, they don't operate at all in IE8. Which is just a terrible thing to say out loud. on the internet. but the client wants compatibility.
Someone should double-check these results, but what I'm seeing is the last slide on top at full opacity. The dot navigation is blinking between slides, but the last slide is the only one that's ever visible.
Works fine in IE9 though! I have a slider that runs on a nav-tabs aside that uses setPosition() to jump around and I have the exact same results with it. Borked in IE8, amazing everywhere else.
I have my hack in place in my client's code-base, so that's covered. Ethan's change (when it updates to remove setFade() z-index'ing too) gets my vote. If you want IE8 compatibility, be warned fade-transition is not available. Seems fair to me.

So sounds like if I clear the rest of the z-index's we...
Gain:
Less intrusive on modern browsers
Lose:
IE8 compatibility for fade transitions.
I'm fine with that, but want to make sure before I update the commit.

ugh, there are a lot of IE8 mentions in the Issues though.
Ethan, does having the z-index property author-able help your situation? We could make a "none" setting too to completely ignore z-index.

It doesn't affect my original issue at all - like I said, I can revert to the original commit I made that doesn't alter the z-index'ing. If we wanted to add a passable z-index option, I would propose addressing that in another update.

Alright, no way we can f:cyclone:k up IE8 support, so if you can revert your zIndex commit, I'll give this PR a proper testing with a bunch of options and scenarios and hopefully merge it soon... then I'll work with @nominalaeon to rebase and test his branch, too :smiley:
Thanks for the effort, guys! :bow:
Si.


add hyper's BufList implementation

Nice! I'm not sure it has its place in h3::proto though. It will be used in h3::frame only.

yeah i was not sure about it being in h3::proto, moved it to h3::frame


[CVAT] Change signature generation

Description
Change signature generation
Motivation
To ensure compatability with JSON.stringify()
Implementation details
Use json.dumps() instead of str()

What's update here @Dzeranov

Completed but for some reason the issue didn't close automatically.


creative-mode: add TRAPI-log on final number of scored results

Right now, the execution of each template will have a log in the TRAPI response logs section that looks like this:
        {
            "timestamp": "2022-07-13T19:03:40.626Z",
            "level": "DEBUG",
            "message": "[Template-0]: Successfully scored 1052 results, couldn't score 1 results.",
            "code": null
        },

It's pretty useful, but creative-mode can involve merging result-sets from different templates.
It's then hard to tell how many of the final results (after this merge) have non-zero scores ("successfully scored"). It would be nice to have that as one of the final logs in the execution of creative-mode.
Optional:

May also be nice to report how many answers were "merged". At the moment, I calculate it by adding the number of results for each template and then subtracting the number of results in the final set (and I think that's accurate but maybe I'm doing simple math wrong...)

This should be pretty easy to implement thanks to an extra feature of the LogEntry class. You can attach a nearly-arbitrary data object to LogEntries, requiring only a type property. For the LogEntry that generates this log, type: "scoring" and some attribute to count the number scored/not scored can be added. See this example.
From there, that data can be used in the createSummaryLog() method in query_graph_handler to generate a summary of all results scored/not scored.

Resolved by https://github.com/biothings/bte_trapi_query_graph_handler/pull/114


Make 7.0 be the base version for builds

When I'm working on PRs, the version of PowerShell that gets built is 6.0.0-alpha.10. I'd like to change that to 7.0.0-alpha.10 (or whatever...I just want it to be 7.0 since I'm working on 7.0-related functionality). Where is that base version set so that I can change it?

I believe you need to just run: git fetch --all --tags

Wonderful, that was exactly what I needed. Thanks @SteveL-MSFT!


Make Stats Replicator job idempotent

Tracked by https://github.com/NuGet/NuGetGallery/issues/2237


Perfomance

There are huge performance lack. Is a good architecture decision - store list of anonymous function with closures?


Fix incorrect spelling of wgslLanguageFeatures (should be WGSLLanguageFeatures)

Bug reproduction:
Install<EMAIL_ADDRESS>and navigate to node_modules/@babylonjs/core/Engines/engine.d.ts:1272
Actual:

Error message:

Expected:

The erroneous code is defined in packages/dev/core/src/LibDeclarations/webgpu.d.ts:71

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@sebavan @Popov72 @RaananW
If you are available, I think this PR needs to get merged and published ASAP since it is preventing any builds with Typescript.

Snapshot stored with reference name:
refs/pull/14403/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14403/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14403/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14403/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14403/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14403/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14403/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14403/merge#BCU1XR#0


Add paths-based reducer for new experimental pipeline [phab:D900]

Created by schroederc at 2016-05-18 21:22:28:
Instead of the ad-hoc streams of protos used in the current pipeline,
the new experimental pipeline will consist entirely of sorted Path
messages built from chains of Kythe edges.  By building various edge
chains, one can construct the various serving artifacts.
For instance, the current FileDecorations may be constructed from Paths
matching the following template:
file <[childof]- anchor -[*]> target

And the serving data for cross-references are their reverses:
node <[*]- anchor -[childof]> file

The new paradigm allows for easier construction of new serving data
through simple reductions that either add or remove edges to its input's
Paths.  In particular, it will be much easier to add target definitions
to each FileDecoration_Decoration by appending more edges to get Paths
matching the following:
file <[childof]- anchor -[*]> target <[defines/binding]- anchor -[childof]> file

zarko wrote at 2016-05-18 21:24:49:
Pre-review comment: can this handle replacement edges? What about relationships that require fixpoint computation (like Callers)?

schroederc wrote at 2016-05-18 22:18:02:

! In (earlier comment), zarko wrote:
Pre-review comment: can this handle replacement edges? What about relationships that require fixpoint computation (like Callers)?

This is just a general framework for handling reductions over *ipb.Path streams.  It win't necessarily be the only method constructing serving data.  The file tree, for instance, can be built directly from each file node's /kythe/node/kind *spb.Entry.
However, one way to handle replacement edges could be to have a chain of initial reducers that make the following path transformations:
source -[]> target
||  // reverse "replaces" edges; sort them first
/
replacement <[replaces]- node
replacement -[]> target
|| // when pivot is a replacement node, emit (node -[]> target) for every non-"replaces" edge
/
node -[]> target
Callers can be done iteratively to build paths like this (where we specialize callables and anchors):
node -[calleableas]> callable <[ref/call]- anchor -[childof]> parent

zarko commented inline (context lost during migration) at 2016-05-19 20:54:29:
It would be useful to include an example here encoded as a text proto (maybe the FileDecorations one from the repo comment).

zarko commented inline (context lost during migration) at 2016-05-19 20:54:29:
nit: node

zarko commented inline (context lost during migration) at 2016-05-19 20:54:29:
nit: their (instead of its)

zarko commented inline (context lost during migration) at 2016-05-19 20:54:29:
nit: no apostrophe in keys

zarko commented inline (context lost during migration) at 2016-05-19 20:54:29:
nit: returns and? coerces

zarko commented inline (context lost during migration) at 2016-05-19 20:54:29:
Should this be here?

zarko wrote at 2016-05-19 20:54:29:
nit: The syntax in the diff message is confusing. I spent a while trying to parse <[childof]- anchor -[*]> as a <>-delimited string and was assuming that * was a Kleene star.

zarko accepted this revision at 2016-05-19 20:54:29

schroederc wrote at 2016-05-19 22:02:13:
Code review

schroederc wrote at 2016-05-19 22:03:54:
Done

schroederc closed via commit 604ceb79965397dd78dbf05b4566db123fb228be at 2016-05-19 22:04:09


oath-toolkit 2.6.7 fails to build on Darwin due to lack of pam_modutil.h

The oath-toolkit package doesn't build on Darwin due to a dependency on pam_modutil.h which doesn't exist in OpenPAM:
checking security/pam_modutil.h usability... no
checking security/pam_modutil.h presence... no
checking for security/pam_modutil.h... no
configure: error: PAM header files not found, install libpam-dev.
configure: error: ./configure failed for pam_oath
builder for '/nix/store/vh071q2g59dqc97mshxh104kmjdjqf65-oath-toolkit-2.6.7.drv' failed with exit code 1

There is an upstream issue to potentially replace the use of functions from pam_modutil.h with something else on platforms where it doesn't exist.
Based on this, I think the intention is not to use PAM on Darwin anyway:
let
  securityDependency =
    if stdenv.isDarwin then xmlsec
    else pam;

So a reasonable fix could be simply to pass --disable-pam when building on Darwin, which solves the build failure.
Notify maintainers
@schnusch
Metadata
 - system: `"aarch64-darwin"`
 - host os: `Darwin 20.5.0, macOS 11.4`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.14`
 - channels: `"nixpkgs-21.11pre301805.dac74fead87"`
 - nixpkgs: `~/.nix-defexpr/channels/nixpkgs`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

test this
diff --git a/pkgs/tools/security/oath-toolkit/default.nix b/pkgs/tools/security/oath-toolkit/default.nix
index a925d07cb7c..fd6a938b2f2 100644
--- a/pkgs/tools/security/oath-toolkit/default.nix
+++ b/pkgs/tools/security/oath-toolkit/default.nix
@@ -1,6 +1,7 @@
 { lib, stdenv, fetchurl, pam, xmlsec }:

 let
+  #TODO Switch to OpenPAM once https://gitlab.com/oath-toolkit/oath-toolkit/-/issues/26 is addressed
   securityDependency =
     if stdenv.isDarwin then xmlsec
     else pam;
@@ -16,6 +17,8 @@ in stdenv.mkDerivation rec {

   buildInputs = [ securityDependency ];

+  configureFlags = lib.optional stdenv.isDarwin [ "--disable-pam" ];
+
   passthru.updateScript = ./update.sh;

   meta = with lib; {

Yup, works.


How to pass parameter in @select="method( extra )" ?

First, I would like to thank for your great vue component. But I have a problem to watch an array object property.
My component code looks like this.
data: function () {
    return {
        products: {
            product: [
                { product_id: '', price: ''},
                { product_id: '', price: ''},
            ],
            another_field: '',
        },
    }
},

Here 'product_id' is a vue multiselect field. Now I want to get object index when 'product_id' changed. I didn't get any solution in vue to get the object index. Now I'm thinking to pass object index in @select="method( key )" when v-for loop in templating.
Is there any way to pass an extra parameter in @select="" or any other solution to solve my problem.
Thanks

Hey! Thanks for the kind words.
I would probably suggest a different data architecture. The products property looks as if it should be an array. It also misses a selectedProduct property that could store the selected element. I don’t really see why you need to use @select? Please take a look at the docs https://vue-multiselect.js.org/

Please see it https://stackoverflow.com/questions/54877945/vue-js-how-to-watch-property-of-nested-object-and-get-index  I think you will understand

I see. The events are emitted with an additional param, called id that you can pass as :id="key". Then your method will receive it as the 2nd param. See docs: https://vue-multiselect.js.org/#sub-events


Add Migaloo assets

ampwhale, bwhale, fable

Found this PR after checking other one. This is looking good! What is needed for merge? I could close the other two in favour of this

Found this PR after checking other one. This is looking good! What is needed for merge? I could close the other two in favour of this

Just wanted your 'OK' to signal that I can merge. I noticed that the two fable images are slightly different (one has a body, and the other is just a head);should I get rid of the png version--the one with the body?

Ahh, my apologies didnt realise. Yes all good to go on this. During the changes for fable image it got changed into something more akin to a coin. Yes the version with the body could go its no longer needed.
Might seek you out on TG or something appreciate all the effort you've put in the help the PR


[WIP] Use ExamQuestionSubmissionService in exam

Resolves #40

@Johnny850807 I have added the comment.
There is still several conversations that are not resolved. Should I just squash and merge?


scrollbar

I wish to position scrollbar on the left

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9440cd51-f00a-2a66-c895-0c093c6c883d
Version Independent ID: 77cf076b-949f-ce9a-882f-7a0c02d268d0
Content: Scroll bar map mode and bar mode - Visual Studio
Content Source: docs/ide/how-to-track-your-code-by-customizing-the-scrollbar.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@Yrp1994 -- than you for for your question. Here are a few options you might consider:

https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

You are welcome to use the Report a Problem tool to report the issue.


Use EncodeUri() if part of the URL path or EncodeUriComponent() if used as a URL parameter

https://github.com/OpticksSecurity/gtm-sensor-template/blob/00d97afc86891c80d665cc375960155783c73183/template.tpl#L108
When reviewing your template we've noticed that it is using parameters to build a url. It's a best practice to use EncodeUri() if part of the URL path or EncodeUriComponent() if used as a URL parameter.

Thank you for your insights @semisight , the issue has been resolved in https://github.com/OpticksSecurity/gtm-sensor-template/commit/40b7ba031d977fba2362d937241971b55ae7e295.


fix reference error

The "parentForm" variable is not initialized therefore leads to:
ReferenceError: parentForm is not defined

All merged and release, thanks for the PR :)
Here's the latest version 1.5.23 and on NPM


swiftTypeCheckTimes and swiftFunctionTimes only appearing for main application target

Using xclogparser v0.2.18
I've been playing around with using -Xfrontend -debug-time-expression-type-checking -Xfrontend -debug-time-function-bodies to analyze what functions and type checkers are slowing down our build times.
I've noticed that only the source files in the main application target seem to have swiftTypeCheckTimes and swiftFunctionTimes populated. All the source files in our dynamic frameworks and static libraries don't though. I've confirmed those targets are also built with the correct compiler flags, and that the method/typechecks + durations are in the build logs in xcode.
Any ideas? Thanks!

It does look like the information is also in the output from running xclogparser dump

Hi! do you have an xcactivitylog to reproduce the error? If you can't provide one, can you share the overall structure of your project to try to mimic it?

Thank you for the quick reply!
Unfortunately I can't share our xcactivitylog. But general structure:

App target (iOS 11 min)
Internal Dependencies:
- static libs
- dynamic frameworks
External Dependencies:
- Carthage
- a few pre-compiled frameworks

Not sure if it matters, but we use XcodeGen as well
Let me know if you need anymore info!

Hmmmm I actually created a sample project to try and get an xcactivitylog with a similar setup and it worked fine. Must be something weird in our setup. May also just be a complexity issue since our repo is pretty large with tons of chained dependencies

I might try to clone XCLogParser and see if I can figure out what's not working. Any hints on where in the codebase to focus on?

Did some more digging.  Turns out being in frameworks was a red herring. Ended up being related to spaces in file paths. Put up a PR here to fix: https://github.com/spotify/XCLogParser/pull/102

Thank you for the PR!


Update virtual-machines-managed-disks-benchmarking.md

The original link to download Iometer takes you to the x86 version. That will not work with a 200GB file like the tutorial wants the learner to create. Adjusting in case this proves problematic for others trying to understand storage benchmarks on Azure Disks.

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

@sbkuehn Thanks for helping improve our docs, we appreciate it! This change looks good so:
#sign-off


Error 0xc000007b

I am getting an error...
{Bad Image}
%hs is either not designed to run on Windows or it contains an error. Try installing the program again using the original installation media or contact your system administrator or the software vendor for support. Error status 0x

So I guess it's indeed not a PE

It's a weird behavior. Sometimes it occurs and sometimes it doesn't...

Problem is in fact not with BlackBone, sorry...


Keep only 18.04 testing stage with scripts/check-ci check

Right now, we run the scripts/check-ci script as part of two testing stages (on Ubuntu 16.04 and Ubuntu 18.04). This is part of Check dev flows stage inside the def checkDevFlows(String version), which is run on both platforms here.
The scripts/check-ci script checks license headers, runs spellcheck and does a dry run of clang-format. Besides the fact that it's not needed to run this twice, the binaries versions (spellcheck and clang-format) from Ubuntu 16.04 and Ubuntu 18.04 may differ and we can end up with inconsistent results.
We need to remove the scripts/check-ci testing stage run from the checkDevFlows method, and have it run in the CI only for the Ubuntu 18.04 flavor.

Let's also set a specific version for each of the tools that we use and make it a requirement for our contributor environment

@johnkord I created #1552 to fix this, and we'll treat the package version pinning in ansible in a separate PR.

I opened https://github.com/Microsoft/openenclave/issues/1555 to continue tracking the packages versions pinning in Ansible.


How to run a single Mocha test

When I stand inside a Mocha file and click "Ctrl+Shift+B", I get an error "ReferenceError: describe is not defined".    Apparently, Script is trying to run the file with nodejs. How can I tell it to run such files with mocha (without having to select a "mocha" profile each time)?

hi, @erelsgl
Sorry, there is no way other than use profiles or configure script options window.
But you can change the grammars.coffee in you local copy of atom-script installation.

What is  grammars.coffee ? Where is it documented?

this file: https://github.com/rgbkrk/atom-script/blob/master/lib/grammars.coffee

@erelsgl Have a look at https://wallabyjs.com/ if you want a smart test runner.


Prerequisites for Configuration Server only

What are the prerequisites for a Configuration Server only deployment?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7a699efb-c78c-2dc6-2d0d-6ffc0a2918ce
Version Independent ID: 36a048d5-71b5-ce1d-75d7-bd5182f4386b
Content: Deploy the configuration server for VMware disaster recovery with Azure Site Recovery
Content Source: articles/site-recovery/vmware-azure-deploy-configuration-server.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@nickbrown1968 Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi @nickbrown1968, can you describe what you mean by "configuration server" only. This document has the prereqs for the config server only. Thanks! Rayne

Under the prerequisites heading there is a line:
"Configuration/Process server requirements"
Later on there is a line:
"Configuration/Process server sizing requirements"
Both of which suggest that the requirements are for a server running
both the Configuration and Process server roles, rather than the
Configuration server only.
Regards, Nick.
On 12/09/2018 14:13, Rayne Wiselman wrote:

Hi @nickbrown1968 https://github.com/nickbrown1968, can you describe
what you mean by "configuration server" only. This document has the
prereqs for the config server only. Thanks! Rayne
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/14723#issuecomment-420643120,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApIeztbyLnKj9faMW3_bEEp40eTPhk33ks5uaQh_gaJpZM4Wg1QQ.

This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus

@nickbrown1968 Kinda dated, but this article for migrating to a CSP shows both broken out in separate tables. https://docs.microsoft.com/en-us/azure/cloud-solution-provider/migration/on-premises-to-azure-csp/asr-capacity-planning


restler-fuzzer connect server with http rather than https

Description
In swagger.json, i have server like this format

servers:

url: https://xx.xx.xx.xx:xxxx/example/

After compile, in grammar.py, i saw host like this

primitives.restler_static_string("Host: xx.xx.xx.xx:xxxx\r\n"),

In grammar.json, i saw this

    [
      "Host",
      "xx.xx.xx.xx:xxxx"
    ]

After test:
I got "Request coverage (successful / total): 0 / 7" in RestlerLogs/restler-20230206-100623.log
got "**Empty response received.  Re-creating connection and re-trying.**" in Test/RestlerResults/experiment265302/logs/network.testing.140076775822912.1.txt
Used wireshark, i saw all requests were sent out in http rather than https.

I found the demo server also have similar issue
I setup demo_server following https://github.com/microsoft/restler-fuzzer/blob/main/docs/user-guide/TutorialDemoServer.md
and I saw the swagger.json has host is https
 "servers": [         {             "url": "https://localhost:8888",             "description": "Staging environment"         }
but from the sniffer i saw, the fuzzer connect server with http rather than https

Hello @NanFeng009,
For your own service, could you please share more details about your scenario?  Is authentication configured for your service?  There may be several reasons for the "empty response received" error.  If it is possible for you to attach the same trace for a working trace of your service (you can remove the endpoints), that would be helpful.
The demo_server traffic you captured looks correct to me.  The Swagger file was auto-generated by FastAPI, good catch that is a bug in the demo_server code and it should be http://localhost:8888.  However, the https is stripped off for the host name (as you saw in the grammar), so there is no issue connecting.
Thanks,
Marina

The demo_server traffic you captured looks correct to me. The Swagger file was auto-generated by FastAPI, good catch that is a bug in the demo_server code and it should be http://localhost:8888. However, the https is stripped off for the host name (as you saw in the grammar), so there is no issue connecting.

Hello @marina-p,
Shouldn't the URL for FastAPI still be set to http? Otherwise, users won't be able to try the endpoints from the interactive API docs in the browser(http://localhost:8888/docs).


Folder placement

In the install instructions, it says to unzip both BepInEx and WurstMod into the H3 folder - but in the provided image, there's no WurstMod folder, and a CustomLevels folder that's not in this download or ever mentioned in the installation process. Is Wurstmod meant to be copied into BepInEx, and if so, where?

The file in releases is an archive containing the folder structure WurstMod needs, including the CustomLevels folder.
It sounds to me like your archive tool is making parent folders when you extract things. If you're getting a WurstMod folder when you extract, you need to put everything in that folder into the H3VR folder.
If you use 7Zip, the context option to do this directly is "Extract Here" with WurstMod_<IP_ADDRESS>.zip in your H3VR folder.

Thanks! The one I downloaded gave me the folders "generic" "properties" and "TNH" in a folder named "WurstMod".
I might just not know how GitHub works.


ternary search audio

Description 📜
I have included the audio explanation of ternary search.
Fixes #3638

Type of change 📝

[x] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)

Domain of Contribution 📊

[x] DSA

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@Sarthak-Agg plz review and merge


the model learns very badly on cifar10 32*32

Hello! Firstly, thank you for the awesome video and code which explain so well how the diffusion models are implemented in code.
I met some issues during reimplementing your code. I' m wondering if you could give me some advice on how to make it work.
I tried to use your code to generate conditional cifar10 (32*32 resolution), but so far the results look kind of bad. In my training, I changed the Unet input size and output size to 32, the corresponding resolution in self-attention, and batch-size to 256. The number of down and up blocks, as well as the bottleneck layers were kept the same as your original setting. After training 400 epochs, the generated images were almost pure color.

Then, I tried add a warmup learning schedule to 10 times as big as the original lr (1e-4) in the first 1/10 epochs, and a cosine annealing for following epochs, and trained it for 1800 epochs. But the final results still look the same as earlier

Do you have some ideas on what's wrong with my version of reimplementation ? I would really appreciate any insights.

Hi, I found a way to make this work. The problem is in the sampling process. During sampling, if I clamp x_t to [-1, 1] in every recurrent timestep, the results looks better (at least now it looks like photos)

But, still, there might be some other problems. The picture looks very hazy, feels like covered by neon or something.

@RowenaHe same problem here, did you mean fix it by something like this?:
def noise_images(self, x, t):
    sqrt_alpha_hat = torch.sqrt(self.alpha_hat[t])[:, None, None, None]
    sqrt_one_minus_alpha_hat = torch.sqrt(1 - self.alpha_hat[t])[:, None, None, None]
    Ɛ = torch.randn_like(x)
    return (sqrt_alpha_hat * x + sqrt_one_minus_alpha_hat * Ɛ).clamp(-1, 1), Ɛ

@RowenaHe same problem here, did you mean fix it by something like this?:
def noise_images(self, x, t):
    sqrt_alpha_hat = torch.sqrt(self.alpha_hat[t])[:, None, None, None]
    sqrt_one_minus_alpha_hat = torch.sqrt(1 - self.alpha_hat[t])[:, None, None, None]
    Ɛ = torch.randn_like(x)
    return (sqrt_alpha_hat * x + sqrt_one_minus_alpha_hat * Ɛ).clamp(-1, 1), Ɛ

I output the intermediate data to look for reason. And I think the reason is that, for some unknown reason, during the recurrent sampling process, the model cannot keep the the generated data around / within [-1,1].
If no clamping in the recurrent sampling process, after 1000 inference steps, almost every single sample is out of the range of [-1,1]. So simply clamping once after the whole sampling process is over would inevitably get the picture of pure color.

I am experiencing the same type of issue with 64x64 landscapes. Half of the images are beautiful, but the other half are solid colors, mostly black and white. I believe the reason for this runaway effect where the model can't keep the values between [-1, 1] is actually simply due to the choice of hyperparameters, and the model may also be trying to overfit. Maybe increase the weight decay and decrease the learning rate? The hyperparameters that worked for the 64x64 model may not necessarily work for your 32x32 model.

@volcanik Thank you for your insight! I will notice you if I can somehow solve this problem in my following experiments.

Hi! I found that using the complex, residual sampling procedure (the one Open-AI use in diffuser-samlpers), which was supposed to get the same results as the simple one-line sampling, in fact gets better results! @potatoker @volcanik @dome272
one-line sampling result:

open-AI DDPM 'residual' sampler result (from the same model):

@RowenaHe that is very interesting! I have also done a few small experiments on CIFAR10 64x64. I only trained for around 50 epochs, but I found that increasing the weight decay of AdamW to 0.1 and decreasing the learning rate to 1e-4 completely removed solid color images with the regular sampling algorithm. I think this runaway effect is most likely caused by excessively large weights. As for the learning rate, I'm not really sure if it has made any difference. Please tell me where exactly you got this new sampling procedure? BTW your results look really good now!

@volcanik OpenAI has a public repository, and there are several available plug-in schedulers, including DDPMScheduler, DDIMScheduler, etc. You can simply use "pip install diffusers" to download them. To use them only need use "from diffusers import DDPMScheduler" and "scheduler = DDPMScheduler()", and then replace the sampling with another loop like this:

There are some great jupyter notebooks demonstrating how we can use the diffusers. I put the link here for you to check out. https://github.com/huggingface/diffusion-models-class

@RowenaHe Thank you for your suggestions. We are also trying to get this DDPM version to work on our own dataset, and get very bright colors. We will try to clamp x = x.clamp(-1,1) at the end of each timestep (i.e. we do it on x_{t-1}, which we have gotten from x_t).
The scheduler you mention works on a conditional model. Is that the DDPM_conditional from this github? I don't see what y is for example, is that the conditional text_embedding? It looks like a global variable, or am I mistaken?

You guys are so awesome! Was having all of the above issues and you helped me solve them ❤ @RowenaHe @thomasnicolet

@RowenaHe Thank you for your suggestions. We are also trying to get this DDPM version to work on our own dataset, and get very bright colors. We will try to clamp x = x.clamp(-1,1) at the end of each timestep (i.e. we do it on x_{t-1}, which we have gotten from x_t).
The scheduler you mention works on a conditional model. Is that the DDPM_conditional from this github? I don't see what y is for example, is that the conditional text_embedding? It looks like a global variable, or am I mistaken?

I think y is the conditional label

@Dart120, which solution was the final one? I have already tried using the AdamW and the learning rate suggested by @volcanik. Still, while I have seen an improvement in the general quality of the generated samples, I continue obtaining some images full of one colour.

@Dart120, which solution was the final one? I have already tried using the AdamW and the learning rate suggested by @volcanik. Still, while I have seen an improvement in the general quality of the generated samples, I continue obtaining some images full of one colour.

Hiya! So what worked for me was clamping the output between -1 and 1 for each timestep when using the cosine schedule. This produces the greyish images that were posted earlier in this thread. Then to fix the greyish images I used code from the hugging face diffusers package which is also a technique posted about in this thread.
I would post my code but it's part of coursework that's currently being marked so I can't for obvious reasons but doing the above two things really helped 😁

Thank you very much for your answer, @Dart120; I'll try to apply this information to check if I can improve my results. Good luck with your coursework!

I have finally implemented what you said @Dart120, and it works as expected. Thank you again!
For anyone reaching this thread in the future, the scheduler of diffusion library seems to avoid the problem of values outside range [-1, 1] by using more sophisticated ways.
https://github.com/huggingface/diffusers/blob/d2fc5ebb958ad8c967752dba5b23c43563cb3159/src/diffusers/schedulers/scheduling_ddpm.py#L462-L468


Atom won't save window size when it exits

Prerequisites

[X ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I've just installed atom for a while. In the beginning, it works fine, and it will save the window size and position when it exits. However, later I started to find that no matter how I resized and positioned the text editor, the window is always fullscreened when I open atom again.
Steps to Reproduce
Expected behavior:
The window will stay on its saved window size and position
Actual behavior:
Every time it's just fullscreened, and I have to resize it again
Versions
Atom    : 1.23.3 x64
OS: Windows 10 64-bit

Thanks for reporting this!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/13909 which was fixed in https://github.com/atom/atom/pull/16295. This fix is currently scheduled to be released in Atom 1.25


Enhanced Percentage Calculator UI

Fixes Issue🛠

Closes #541
Description👨💻
Changed the UI and UX of percentage calculator page.
Type of change📄

[x] New feature (non-breaking change which adds functionality)

How this has been tested✅

Works on live server and ran locally on server.
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷

Merged, Congrats on getting your PR merged to this repo under SWOC S4 🎉🎉 @VIDHISHAK15

Nice, Merged!! Congrats on getting your PR merged to this repo under SWOC S4 🎉🎉 @VIDHISHAK15


How to disable hints?

Is the hint enabled on default?
How could you disable the hint if it is selectable?
Thanks.

There is no default hint. Just supply the array and the Spinner will be populated and display the first element.


docs: external C code

Updated URL for the "embedding Python" link in external_C_code.rst
in the userguide.
The freenet.org.nz domain has clearly expired at some point and now
the original link goes spam links, but there are copies in the
Wayback Machine.
The updated (fixed) URL uses the most recent archive for the
original page (author: June, 2004; web archive: Feb, 2012).

The replacement link goes here.
It's probably worth moving stuff like this into the project wiki since at least that way it will follow the project wherever it goes, rather than leaving end users depending on sites which don't last.  While the Wayback Machine is great, it shouldn't be relied on for documentation.

Thanks. Note that there's also a dedicated documentation (stub) page now, which I've started to collect documentation about embedding - knowing how bad the general state of affairs is on that topic.
Improvement PRs for that page are welcome! :)


Changing Camera2D position and zoom simultaneously in _process is out of sync

Tested versions
v4.2.1.stable.official [b09f793f5]
System information
Godot v4.2.1.stable - Windows 10.0.26058 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1070 (NVIDIA; <IP_ADDRESS>33) - Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 Threads)
Issue description
When changing position and zoom of Camera2D simultaneously in _process() callback, both do not update in the same frame. It appears as if zoom changes immediately and position changes in the next frame. The issue does not exist when changing the zoom and position in _physics_process() callback.
Adding videos to explain the issue.
When zoom and position are updated in _process():
https://github.com/godotengine/godot/assets/6249061/4793e3d8-ce9d-482a-93a2-b9912154d064
When zoom and position are updated in _physics_process():
https://github.com/godotengine/godot/assets/6249061/12fdcbd4-9d48-4ad1-a843-84823d5a4660
Found another issue that might have the same underlying bug, but not sure so opening another issue: https://github.com/godotengine/godot/issues/73298
Steps to reproduce
Use the following script attached to Camera2D to test:
extends Camera2D

func _process(delta: float) -> void:
	#oscillate zoom value between 0.5 and 1.0
	var z = 0.5 + abs(fmod(Time.get_ticks_msec()/1000.0 + 0.5, 1.0) - 0.5)
	zoom = Vector2(z, z)

	#position camera top-left to 0,0
	#replace 540 with whatever is viewport height
	position.y = 540 / z / 2.0

Minimal reproduction project (MRP)
Camera2DTest.zip

Just tested on Godot 4.3-dev4, can confirm that the issue is resolved. Thank you!


Listening requires the "press" command to be in lower case

Listening requires the "press" command to be in lower case.

Fixed in #76


Block: Add test that double-spending blocks are rejected

@aszepieniec Let me know if these small rewrites conflict with your current work.
TODO (for future commits, not for this PR): Block's is_valid function should return an error code indicating why the block was rejected. That would make negative tests of block validity much more robust. I'll add an issue to that effect.
Includes a rewrite of helper function for constructing a new block that makes it easier to create a new block from a BlockPrimitiveWitness. Specifically, a new function block_template_from_primitive_witness is added which is used in this test as well as in the call graph for creating a new block from a transaction and a predecessor.
Also adds an "unsafe" version of MutatorSetUpdate's "apply" function that allows the caller to calculate a new MS accumulator which ignores double spends. This was needed to allow the BlockPrimitiveWitness constructor to complete the construction of a block with a double- spending transaction.

Merged in 4746a778eee2a2dd17f236d62f89aba00d195dad


ci: add windows coverage

Fixes #97

Ugh! I can't reproduce this windows failure locally, even with
bazel clean --expunge; bazel --bazelrc=../../.github/workflows/ci.bazelrc --bazelrc=.bazelrc test --enable_bzlmod --noenable_runfiles //... --nocache_test_results
it is still green for me. That spot that passes the $(rlocation) for tarball.tar so the test failure isn't making sense. I'll come back to it later maybe.

@thesayyn this failure doesn't make sense to me from looking at the code, do you have any ideas?

@thesayyn this failure doesn't make sense to me from looking at the code, do you have any ideas?

it might repro if you delete docker.config at $HOME. Also it might help if you rebase as i am not sure if this branch was run against the main that has some fixes for docker config discovery.


Update guidance URLs

Brief overview of changes
We give guidance links in a number of messages on our tests, have updated those to the analysts guide URL
Why are these changes being made?
The rsconnect site hasn't been updated for years and was binned altogether earlier this year / last year (a while ago!).
Detailed description of changes
Mass find and replace of the domain name for the new domain, all pages and sections have similar titles now so didn't change those.
Additional information for reviewers
None
Issue ticket number/s and link
None

 Update guidance URLs on data screener


ngf-select not working on all android browsers

If i remove ngf-select then working fine.

Believe it's related to https://github.com/danialfarid/ng-file-upload/issues/768

Yes i test it the demo work fine. But my code is working in ios, blackberry devices but not in android.


Remove Keybase chat button from facebook profile

Hey there
I'm in a big trouble. Please help me out.
After opening an account I've included facebook account on Keybase.
But after sometime I've completely deleted the key provided from Keybase and also deleted my account from Keybase. But the chat button still appears on my facebook profile.
How can I remove that button?

Delete the Keybase Chrome Extension.
The Keybase icon appears on everyone’s profile, even if they never had a keybase account. This is because the Keybase Chrome Extension overlays the button on the profile.


Update ecc.md

This pull request contains changes to improve clarity, correctness and structure.
Description correction:
Corrected comapnion to companion
Please review the changes and let me know if any additional changes are needed.

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

Looks good. However, can you please sign the CLA for being able to merge? I would first merge it into the aggregate branch fix/typos-v0.15 which will be merged to master before next release.


Heroku app google login not working

The google auth on the heroku app is not working.

The OAuth client was deleted.

This is fixed.


Connect linalg on tensors to rest of the CPU side.

Connecting the code generated from Linalg on tensors to rest of the
CPU side requires setting the tile size (that is dynamic when tile +
distribute happens in flow level) to the static tile size that is
computed using LaunchConfig.

This is ready for review. But making this a draft review. Please look at just the second commit here. The first commit is https://github.com/google/iree/pull/4597 which is ready to land, but waiting on integrate.

This is ready for review. But making this a draft review. Please look at just the second commit here. The first commit is https://github.com/google/iree/pull/4597 which is ready to land, but waiting on integrate.

As a tip, if you hit edit you can set the base branch:

In this case to MaheshRavishankar:linalg_on_tensors. Then github shows only the commits added on top of that. When that branch gets merged github will automatically update this PR to point to main. It's kind of like DIFFBASE.

As a tip, if you hit edit you can set the base branch:

In this case to MaheshRavishankar:linalg_on_tensors. Then github shows only the commits added on top of that. When that branch gets merged github will automatically update this PR to point to main. It's kind of like DIFFBASE.

As a tip, if you hit edit you can set the base branch:

In this case to MaheshRavishankar:linalg_on_tensors. Then github shows only the commits added on top of that. When that branch gets merged github will automatically update this PR to point to main. It's kind of like DIFFBASE.
NEVERMIND: that only works on branches within the repo ;(

Would be good there was a way I can merge individual commits of sequence of commits without having to merge the whole branch....

As a tip, if you hit edit you can set the base branch:

In this case to MaheshRavishankar:linalg_on_tensors. Then github shows only the commits added on top of that. When that branch gets merged github will automatically update this PR to point to main. It's kind of like DIFFBASE.
NEVERMIND: that only works on branches within the repo ;(

Would be good there was a way I can merge individual commits of sequence of commits without having to merge the whole branch....


Track changed Image assets and use them to update image caches.

Previously, since the bind groups do not get updated every frame, modified image assets would not be rebound. This caused hot reloading to no longer work. Similarly, the TextureArrayCache (when not using atlas) would not invalidate modified assets, so the textures would become out-of-date.
Note there can be other reasons that an asset can be modified, so this should keep the assets more up-to-date in general.
Fixes #430.

Rebased onto main to fix cargo clippy which was broken at HEAD at the time.

Can it be merged to main? This commit fixes the issue with updating texture on a map.

I will rebase this once #440 goes through. I still need a review though for the overall structure of this solution.

I've rebased this onto main after #440. I tested this with a little example and it still works.

@StarArawn Is there anything I should change?

@StarArawn can it be merged?

@rparrett can you review this for me?

@andriyDev is it possible to use https://docs.rs/bevy/latest/bevy/render/render_asset/struct.ExtractedAssets.html instead of doing the manual collecting/extraction?

It seems like the answer to the above is probably no.
Sorry,but I don't think I can provide a meaningful review. I don't think I'll be able to get up-to-speed on this area of bevy_ecs_tilemap / bevy any time soon.
But this does fix hot reloading for me and doesn't seem to cause other issues as far as I can tell.

In theory I think I could extract the EventReader<AssetEvent<Image>> resource, then do the collection in a render system, but since we need the modified images in queue_material_tilemap_meshes and remove_modified_textures, we'd still need the ModifiedImageHandles resource. It's not clear to me that that would be worth it? But if that seems nicer, I can make that change!

Looks like CI is just failing due to new let-else formatting in rust 1.72 in an unrelated file.

What should I do in regards to https://github.com/StarArawn/bevy_ecs_tilemap/pull/431#issuecomment-1732363616 ? I'm happy to change it if that looks cleaner. They both look roughly the same to me.

They both look roughly the same to me.

I agree. Seems fine as-is IMO.

@andriyDev Can we get the conflicts fixed up so we can get this merged in? :+1:

Alright I rebased onto main. I also used AssetIds in place of Handles, since AssetIds seem to be the more "canonical" identifier for assets. I tested this locally and it still works for hot reloading!
@StarArawn I think this is good to go :)

Hey @andriyDev, could I bug you to update the branch again? It looks like a relatively straightforward rebase. We should be in a better position to try to get this merged now.
If not, I'd be happy to adopt the PR.

Merged the adopted PR. Thanks @andriyDev.


[Data Chain KIT] Remove API Plug-In

Please refer to https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/issues/970 to see tasks and acceptance criteria.

Ill add this to my todos. Thank you @danielmiehle for assigning


On posting data with no data

hi ,
I am send the request from SLbatchrequest  with ProductionOrders(123)/Cancel  this is Post Method But with no content i.e data = null and i am always getting bad format as a response

Hi, @sandy5791. I've seen this before and for some reason, requests within a batch require a body, even if an empty one.
Try sending "{}" as the request body and it should work.

thank you it worked.


Can't press Ctrl+F5 to start without debugging in VsCode on Linux OS

Environment
Operating System:Linux
JDK version:14.0.2
Visual Studio Code version:1.52.1
Java extension version:0.74.0
Java Debugger extension version:0.31.0
Steps To Reproduce

Open a  js file  in VS Code
Set a breakpoint
Press Ctrl+F5to start without debugging

Actual Result
The program cannot be run and vscode windows will be closed auto.
Expected Result
The program starts without hitting the breakpoint.

Thanks for filling this issue. Can you reproduce using Visual Studio Code Insiders https://code.visualstudio.com/insiders/
I can not reproduce

@isidorn , the issue can repro in Visual Studio Code Insiders on Linux OS  with using above steps

I just tried this on my Ubuntu 20 and this works like a charm with JavaScript.
@testforstephen can you reproduce this? If yes, is java doing something funky with the keybindings, or is it supporting run without debugging mode?
@SunnyChen1 if you do this with a simple javascript file do you see the same. Here's an example file
console.log('hello there!')

Sorry, i don't have linux on hand yet. I cannot reproduce it in macOS.
And yes, Java debugger supports run without debugging mode.


兼容dart 2.15.0虚拟机问题

虚拟机运行xcode报错
ld: in /Users/flutter/flutter/.pub-cache/hosted/pub.dartlang.org/tencent_kit-2.1.0/ios/Libraries/TencentOpenAPI.framework/TencentOpenAPI(QQObject.o),
building for iOS Simulator, but linking in object file built for iOS, for architecture arm64

[✓] Flutter (Channel stable, 2.8.0, on macOS 11.4 20F71 darwin-arm, locale
zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[!] Xcode - develop for iOS and macOS (Xcode 12.5.1)
! Flutter recommends a minimum Xcode version of 13.0.0.
Download the latest version or update via the Mac App Store.
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.57.1)
[✓] Connected device (3 available)

我重新打开该issue
因为我需要在该插件打开arm64 架构

issues#54


Error setting up entry - Failed to set up

Just started getting this error, I checked the logs on alarm.com and it seems to have logged in. 2fa disabled
Error setting up entry ADT Security
This error originated from a custom integration.
Logger: homeassistant.config_entries
Source: custom_components/alarmdotcom/init.py:117
Integration: Alarm.com (documentation, issues)
First occurred: 9:50:24 AM (5 occurrences)
Last logged: 6:49:03 PM
Error setting up entry ADT Security Australia:XXXXfor alarmdotcom
Error setting up entry ADT Security Australia:XXXX for alarmdotcom
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 387, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/alarmdotcom/init.py", line 117, in async_setup_entry
hass.config_entries.async_setup_platforms(config_entry, PLATFORMS)
AttributeError: 'ConfigEntries' object has no attribute 'async_setup_platforms'

any help would be great

Dupe of #257


ARROW-12785: [CI] the r-devdocs build errors when brew installing gcc

@github-actions crossbow submit test-r-devdocs


[REQUEST] remember QM/Hero league preference

A minor inconvenience having to always set it back to hero league.
Also can you let it swap the MMR display if we switch the display when a game has loaded?

Do you use latest version?   I just updated to 0.4.3 and will try it out.  Thanks for responding.


⚠ Bofrost.lu has degraded performance

In 128c10b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 803 ms

Resolved: Bofrost.lu performance has improved in 0aa4535 after 6 minutes.


evaluation issues

Hi, thanks for your work!
If training and validating with a custom dataset, how are the evaluation metrics acc_occpair and miu generated?
Is it the predicted amodal mask and the generated amodal mask annotations ? BecauseThe custom dataset does not contain amodal and order annotations
Can you answer my question? Thanks a lot

Do you excult his codes on the COCOA or KINS datasets?

On custom dataset, if you want to evaluate quantitatively, you have to prepare modal and amodal masks, as well as order annotations by yourself. You may also run on given datasets COCOA and KINS.


Roadmap to v1

Just jotting down some thoughts on how to get PythonCall to v1.

Internally refactor the code into more modules, PythonCall_X where X is CPython, Base (Py and related functionality), PyList, PySet, ..., JuliaValue (pyjl), PyMacro (@py), Convert (pyconvert), MultiMedia, JuliaCall, etc. This will make it easier to reason about the package in smaller chunks and make the dependencies between various parts more explicit. In the future these could be split off into separate packages.
Maybe change how PyArray is parameterised.
Change pyconvert to use a simpler explicit branching logic (branching on the target type and the python type).
Maybe change pyconvert to prefer copying conversion over wrapping.
Change pyconvert to take keyword arguments to more finely control the conversion. For example copy=true to force copying conversion.
Configure with Preferences.jl.
Rewrite @py, splitting branches out into separate functions and removing MacroTools.
Add pydel!_of_Py_is_safe(x) returning true if pydel!(Py(x)) is safe. Use this to automatically pydel! stuff where possible, e.g. in @py.
Put unsafe_ prefix on unsafe functions like pydel! and pynew() and/or remove them from the API. Recommend using @pyconst instead.
Check for threadid() == 1 in finalisers.
Make all values in JuliaCall be of the same JuliaValue type (or Jl?), just as all Python values in PythonCall are of the same Py type. Implement wrappers on top of this.
Add a constructor JuliaValue(x).

just want to say we really need Preferences.jl based solution

Closing because I now have a v1 roadmap with issues covering all of these points.


Add Queue::peek.

Hi @jmgao, interesting addition to heapless.
Could you add some doctests to the new methods?
Furthermore, I am not sure about a return type of Option<T>, because of non-copy types. Would it be more correct to return Option<&T> ?

Yeah, returning T by-value doesn't look correct (without a T: Copy bound). Returning an Option<&mut T> should also work, though, right? That would be a bit more flexible than returning an immutable reference.

Oops, it hadn't occurred to me that there wasn't already a T: Copy bound.
I can return an Option<&mut T>, but it seems to me that that should be called peek_mut or something along those lines. I went with Option<&T> in the latest revision of the patch, do you want me to add peek_mut as well?

@jmgao thanks a lot for coming back and addressing the issues
@japaric could you take a look at this PR and maybe run bors
@bors try

Why do the peek methods take &mut self? That would prevent the following two operations, which seem perfectly fine to me, from compiling:
fn foo(q: &mut Queue<i32, U2>) {
    let x = q.peek();
    let y = q.is_empty();
}

fn foo(c: &mut Consumer<i32, U2>) {
    let x = c.peek();
    let y = c.ready();
}

The peek operation does not mutate the head or tail pointer so it should be allowed through a shared reference (&self).

Why do the peek methods take &mut self? That would prevent the following two operations, which seem perfectly fine to me, from compiling:

Latest revision switches the methods to &self.

@jmgao thanks!
(Please next time push new commits into the branch instead of force-pushing. It's hard to see what has changed from revision to revision if you do the latter)
bors r+

(Please next time push new commits into the branch instead of force-pushing. It's hard to see what has changed from revision to revision if you do the latter)

Apologies, I'm used to review tools like gerrit/phabricator(/pretty much everything that isn't github pull requests) that make it easy to diff between revisions. I'll push separate commits in the future.


move to JobConfig subdir

Dont we need to change the include file paths?
On Fri, May 28, 2021 at 1:13 PM Ray Culbertson @.***>
wrote:

Merged #2 https://github.com/Mu2e/Production/pull/2 into main.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Mu2e/Production/pull/2#event-4814594267, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAH576GS4TBX7KW2ILSAUTTP72PZANCNFSM45XMAEFA
.

--
David Nathan Brown            @.***
Office Phone<PHONE_NUMBER>  Fax  495-2957
Lawrence Berkeley National Lab
MS 50R5008 (50-6026C)  Berkeley, CA 94720

This method has lost the history.  Somehow the history was there before "git mv" and is not there after.

Hi Ray,
Is there any content in Production that would be lost if you wiped it and started over?

Rob
From: Ray Culbertson @.>
Reply-To: Mu2e/Production @.>
Date: Friday, May 28, 2021 at 3:23 PM
This method has lost the history. Somehow the history was there before "git mv" and is not there after.

OK, losing the history seems to be normal with git mv.  It is there with "--follow".. Is this OK?  The method I'm using doesn't obviously have a way to move the directory with keeping the directory.  it moves the content out of the directory.  Note that I could not get the most modern methods of splitting repos to work, I'd have to download tools.  If you want, we could stop and investigate.
Rob, nothing in Production needs to be kept, can easily start over

History is there.  After a move you must use:
git log --follow
Dave
On Fri, May 28, 2021 at 1:23 PM Ray Culbertson @.***>
wrote:

This method has lost the history. Somehow the history was there before
"git mv" and is not there after.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Mu2e/Production/pull/2#issuecomment-850652007, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAH57Y6JOXI2YH5FJ5S7CTTP73S3ANCNFSM45XMAEFA
.

--
David Nathan Brown            @.***
Office Phone<PHONE_NUMBER>  Fax  495-2957
Lawrence Berkeley National Lab
MS 50R5008 (50-6026C)  Berkeley, CA 94720

I just cloned Production and did a git log.  I see 956 commits dating back to 2013.  I did not need –follow .
Rob
From: Ray Culbertson @.>
Reply-To: Mu2e/Production @.>
Date: Friday, May 28, 2021 at 3:27 PM
To: Mu2e/Production @.>
Cc: Rob Kutschke @.>, Comment @.***>
Subject: Re: [Mu2e/Production] move to JobConfig subdir (#2)
OK, losing the history seems to be normal with git mv. It is there with "--follow".. Is this OK? The method I'm using doesn't obviously have a way to move the directory with keeping the directory. it moves the content out of the directory. Note that I could not get the most modern methods of splitting repos to work, I'd have to download tools. If you want, we could stop and investigate.
Rob, nothing in Production needs to be kept, can easily start over
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Mu2e_Production_pull_2-23issuecomment-2D850653986&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=Zi5IH-k2_JiK_Wipiv3pwwSXO9UNaHiLP8sisfPz__k&m=4K-aT-TwyGfsOyezi2h853Mm8lxaQZ0tYiiInRj2R_U&s=nXvkrRbTGkfbWcnkfspBmOeeTbGkYEc_JaZGL18tcRQ&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ABHY5ZRHAMVTL6ZS7356QHTTP74EBANCNFSM45XMAEFA&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=Zi5IH-k2_JiK_Wipiv3pwwSXO9UNaHiLP8sisfPz__k&m=4K-aT-TwyGfsOyezi2h853Mm8lxaQZ0tYiiInRj2R_U&s=HZI3tRrNpmwE8UY0ZDuq7EcgCspDxZ_WjX_sv_LQ5j4&e=.

The pattern seems to be "git log" without a file name does not need --follow, with a file it does.
Do you want me to stop and investigate how to move a whole intact directory rather than insert an unnecessary mv?  It might take a few days, talking to experts.

I think this is a transient think.  As time goes on we will care less and less about old history.  We will only remember to use –follow for short time.
From: Ray Culbertson @.>
Reply-To: Mu2e/Production @.>
Date: Friday, May 28, 2021 at 3:46 PM
To: Mu2e/Production @.>
Cc: Rob Kutschke @.>, Comment @.***>
Subject: Re: [Mu2e/Production] move to JobConfig subdir (#2)
The pattern seems to be "git log" without a file name does not need --follow, with a file it does.
Do you want me to stop and investigate how to move a whole intact directory rather than insert an unnecessary mv? It might take a few days, talking to experts.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Mu2e_Production_pull_2-23issuecomment-2D850661543&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=Zi5IH-k2_JiK_Wipiv3pwwSXO9UNaHiLP8sisfPz__k&m=fS5_gBJqzPcHBKkqdfcy2uxKSki50xA_d6bQsN9Dbgk&s=i5lMRhANvHge2YiR9Z9fvZyk9marJWoQ23C-DS3LjhM&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ABHY5ZVIFJX7E57M3C5LD43TP76KTANCNFSM45XMAEFA&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=Zi5IH-k2_JiK_Wipiv3pwwSXO9UNaHiLP8sisfPz__k&m=fS5_gBJqzPcHBKkqdfcy2uxKSki50xA_d6bQsN9Dbgk&s=TpfbnjXRYb5Azww-5rKmp40npBdEQqBj-diOZbd9ZVI&e=.

I don't think the final result will change, so I don't think it matters.
On Fri, May 28, 2021 at 1:46 PM Ray Culbertson @.***>
wrote:

The pattern seems to be "git log" without a file name does not need
--follow, with a file it does.
Do you want me to stop and investigate how to move a whole intact
directory rather than insert an unnecessary mv? It might take a few days,
talking to experts.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Mu2e/Production/pull/2#issuecomment-850661543, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAH57YDM6BSGKYSLEPFSS3TP76KTANCNFSM45XMAEFA
.

--
David Nathan Brown            @.***
Office Phone<PHONE_NUMBER>  Fax  495-2957
Lawrence Berkeley National Lab
MS 50R5008 (50-6026C)  Berkeley, CA 94720


Add default favicon for favicon.png and favicon.ico

With this PR, spin-fileserver is able to serve either favicon.ico or favicon.png if the user did not provide a corresponding file.
Instead of altering many parts of the existing FileServer implementation, I came up with a ResourceServer struct responsible for reading and compressing the favicon from fallback favicon graphics embedded into the module.
This also required modifying the return value of FileServer::resolve from Option<PathBuf> to a custom enum that I called FileServerPath. The FileServerPath comes with different variants to represent physical resources (Physical(PathBuf)), embedded resources (Embedded(&'static [u8]))) and non-existing resources (None).
fixes #33

Quick addition: I added favicon.png because it would also work in the case someone links the favicon as png without actually providing it.
This is totally optional, removing it would of course reduce the amount of modifications.

This will be a handy feature, thank you @ThorstenHans!
Found a small bug in my testing, noted in comments.
Additionally, I wanted to bring up the scenario of when an app uses this component but sets a trigger route other than root (/...), eg:
[component.trigger]
route = "/static/..."

With such an app, the favicon fallback logic works as expected when the request is e.g. http://localhost:3000/static/favicon.ico (where PATH_INFO_HEADER is relative to the trigger route, so /favicon.ico) -- yet, favicons are always requested at the root of the website*, so requests to e.g. http://localhost:3000/favicon.ico still encounter a 404. I wonder what we could do to support favicon fallback when arbitrary trigger routes are used?
edit: perhaps both /favicon.* and /images/favicon.* are supported by browsers? Is there a canonical spec where they are expected to live?
(I stumbled upon this on spin new using the spin-fileserver template and cruising through the prompts; the default for the trigger route ("HTTP path") happens to be /static/...

I looked at the HTLM 4.0.1 and XHTML 1.0 spec to spot any further information about favicon requests. I found this - quite dated - blog post also talking about the loading favicons from different sources... However, everything I found assumes that users actually specify a custom favicon.
According to the HTML5 spec (https://html.spec.whatwg.org/multipage/links.html#linkTypes) using <link rel="icon" ...> is treated as External Resource. I quickly read through those pages, but haven't found any spec that explains more...
Regarding the with either specifying
trigger = { type = "http", base = "/foo" }
# ...
route = "/..."

or
trigger = { type = "http", base = "" }
# ...
route = "/bar/..."

TBH, I don't know how we could address this when the user doesn't specify any favicon in HTML.  Obviously, it'll serve the fallback icon for both configurations shown before when user specifies
<link rel="shortcut icon" href="favicon.ico" type="image/x-icon">

(See href not starting with a leading /.)

Thanks for the research on conventions and preferred methods of specifying a favicon.  Like you've pointed out, it does seem that HTML representation is recommended over varying browser conventions for auto-lookup.

TBH, I don't know how we could address this when the user doesn't specify any favicon in HTML. Obviously, it'll serve the fallback icon for both configurations shown before when user specifies

Indeed, with how Spin's trigger mechanism is designed, I believe you are right.

Sure, I'll add a corresponding section to the README. Will look into it tomorrow morning


Property 'init' does not exist on type 'typeof Geocoder'.

Types are wrong configured, because there is a typescript warning in all methods but works

Please release new version to update index.d.ts file.

Hi,
I think this issue is fixed in 0.5.0.


chore: Remove unecessary calls to pedersen__init

Related to #3029 and resolves #3077 .
The generator context is initializing and extending generator data which outlives the lifetime of the function (it stays alive for the lifetime of wasm instance). There may be an argument in the future to add a method that will arbitrarily extend the generators, so that they do not get extended in the middle of a pedersen call, if this is wanted we can open up a separate issue.
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 15db6bf3 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,444
179,588
716,132

l1_rollup_calldata_gas
222,960
868,268
3,449,456

l1_rollup_execution_gas
842,047
3,595,376
22,204,825

l2_block_processing_time_in_ms
1,996 (+1%)
7,510
29,616

note_successful_decrypting_time_in_ms
298 (+1%)
860
3,140

note_trial_decrypting_time_in_ms
102 (+6%)
70.0 (+4%)
134

l2_block_building_time_in_ms
13,109
51,987
207,663

l2_block_rollup_simulation_time_in_ms
11,801
46,785
186,766

l2_block_public_tx_process_time_in_ms
1,267
5,074
20,422

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
21,485
42,399

note_history_successful_decrypting_time_in_ms
2,015 (+1%)
3,936

note_history_trial_decrypting_time_in_ms
120
142 (-1%)

node_database_size_in_bytes
1,630,633
1,096,398

pxe_database_size_in_bytes
27,188
54,187

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
72.2
61,697
18,841

private-kernel-ordering
43.4
24,233
8,089

base-rollup
1,770
656,309
811

root-rollup
77.3 (-6%)
4,072
1,097

private-kernel-inner
53.2 (+1%)
81,504
18,841

public-kernel-private-input
41.6 (+1%)
41,455
18,841

public-kernel-non-first-iteration
26.5 (+1%)
41,497
18,841

merge-rollup
0.884 (-2%)
2,592
873

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
8,723
27,240


Display 3D trajectories, with a grid interface

Please base yourself from master :)

Naa, ill wait for #180 to go in :P

I will get a CLI also :P

I seemed to have forgotten to add the example. Added it now. @Daval-G you are good to go for review whenever you get time.


Border with radius creates colored boxes on corners

Checklist

[x] I have read the appropriate section in the contributing
guidelines
[x] I believe this issue is a problem with polybar itself and not a misconfiguration on my part.
[x] I have searched for other open and closed issues that
may have already reported this problem.
[x] I have checked the known issues page for this problem.

Describe the bug

Recently, PR #2359 was merged, which added curved corners. I tried this out, and saw that, although the borders were curved as they should be, small rectangles were also appearing on the corners of my bar.
Expected behavior:

No colored boxes should appear on the inner corners of the bar. See picture for what should be happening:

Actual behavior:

Colored boxes are appearing on the inner corners of the bar. See picture for what is happening:

Was it working before?
No, since this feature hadn't yet been implemented in polybar.

Did you also experience this bug in an earlier version of polybar (yes/no/don't know)?

No, since the feature hadn't been implemented before.
To Reproduce

[bar/main]

width = 60%
height = 52

offset-x = 20%
offset-y = 0.5%

background = #70182232

radius-top = 16.0
radius-bottom = 16.0

border-bottom-size = 2
border-top-size = 2
border-left-size = 2
border-right-size = 2
border-color = #4088aaff

font-0 = "SF Pro:size=15;2"
modules-right = test

[module/test]
type = custom/text
content = "test"

Running the above config with polybar main -c config.ini & should reproduce the problem.
Polybar Log

notice: Parsing config file: ./test_config.ini
notice: Loaded font "SF Pro:size=15" (name=SF Pro, offset=2, file=/usr/share/fonts/TTF/SF-Pro.ttf)

Screenshots

See screenshots above
Environment:

WM: bspwm
Distro: Arch Linux
Output of polybar -vvv:

polybar 3.5.5-113-gd3797ace

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags: -D_FORTIFY_SOURCE=2 -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -Wall -Wextra -Wpedantic -Wsuggest-override -O3 -DNDEBUG
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

Additional context

I have forked the repo and have a pull request ready to go that should fix this issue. If you can reproduce this and think the problem should be fixed, just let me know and I'll file it for review. The screenshot under Expected Behavior is from polybar running on my fork.
Also, I don't know if it's relevant  (since the bar is partially transparent), but I use picom for compositing.

I can reproduce this.
Looks like it happens only if the bar background is transparent. Weird that we never caught this.
Yes, please open a PR with your fix. Thanks :smiley:

Just opened #2407; thanks for the quick response :)


Add a caching layer for password auth

Continuing the conversation from #9615, I think a better approach for reducing the CPU overhead of authenticating with a password that wouldn't reduce the difficulty of brute-forcing the passwords should the database leak would be to cache successful authentication requests.
My proposed scheme would be:

Strongly recommend bcrypt.DefaultCost as the difficulty parameter
The first time a user successfully authenticates against the bcrypt hash, do the following

Generate a random salt
Hash the salted password with a basic hash (SHA256, for example)
Cache (bcrypt hash, salt, hash result) for the user

Every time a user connects and they have an entry in the cache, do the following:

If the current bcrypt hash matches the cached bcrypt hash, hash the password with the salt (<1us) to authenticate the user.
If the current bcrypt hash differs from the cached bcrypt hash, remove the cache entry and proceed with the normal (slow, ~60ms) bcrypt authentication, caching on success.

(Maybe?) Preemptively disseminate (bcrypt hash, salt, hash result) between the nodes (in-memory only) to mitigate the cost of having to take the bcrypt time penalty on each server in the cluster.
(Maybe?) Preemptively purge cache entries when a user's password is changed (infrequent).

Some benefits of this include:

The bcrypt cost is maintained at the recommended difficulty in the event of a database (on-disk) breach.
Subsequent authentication attempts to nodes with a current cache entry are fast, and do not steal all the CPU in a busy cluster.
The additional risk of an adversary acquiring the salt+hash is the same as an adversary being able to read the process' memory, in which case the plaintext password would also be accessible.

/cc @jxuan @gyuho @ericchiang @mitake

This seems like a fine solution. It's low overhead, doesn't introduce new APIs. etc.
On a side note, bcrypt is supposed to be a bottle neck, and the fact that we're working around that probably indicates that bcrypt isn't a good fit for this use case. As I mentioned in the original issue (https://github.com/coreos/etcd/issues/9615), I think a different kind of credential would be a more wholistic solution. Random values, signed random values, whatever. Just not bcrypt.

I like this idea, but I am afraid that it can be very complex to share in memory cache among etcd server nodes. I like ericchiang's comments here.
Several other things I'd like to suggest to alleviate this problem:

Clients do not use a fixed order to connect to etcd server nodes. Each client should choose a random server node to connect to each time. Otherwise, if all clients share the same list of etcd nodes, it can bring down the first node in the list during a re-election if there are a large number of clients.
Should uses JWT token instead of simple token. In case of a re-election, if the client presents a valid JWT token, the client should connect without having to go through the bcrypt algorithm on the server side.


Allow sorting in log history view

Would be nice to allow users to sort based on column when viewing a bin's log history.

:+1:


Update cryptographic-functions-transact-sql.md - fix typo in function name

Fix typo 'VERIGYSIGNEDBYCERT' in function name. The link had the name spelled correctly so clicking on it still worked as expected.

Learn Build status updates of commit f9660e1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/functions/cryptographic-functions-transact-sql.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

I spotted another typo in the same file: VERIFYSIGNEDBYASMKEY should be VERIFYSIGNEDBYASYMKEY. Let me see if I can update or add another commit to the pull request here in the github web interface.

Learn Build status updates of commit 8b89818:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/functions/cryptographic-functions-transact-sql.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@VanMSFT
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Just checking there's no holdup from me - is the change trivial enough that no CLA is required?

Hi @max-webster - No action for you at the moment. The CLA check has passed. Thanks!

@vanmsft
Can you review this proposed update to your article and enter #sign-off in a comment if it's ready for merge?

Sorry for the delay.
#sign-off


[Request, Releases] Providing atleast one hash per file

Description
More and more projects canceling by force of corpse. When this happens - a lot of "mirrors" appears. Some of them is fair-play, some of them contains malicious software. Providing at least one not yet compromised hash for each file under Assest in the end of the description(in text form) give users an option to validate content of "mirrors".
One or two of following hash function should be enough
SHA2-256
SHA2-512
SHA3-256
SHA3-512
personally i would prefer pair of SHA2-256 + SHA3-512
Additional context
Try to find kex official hashes, all you will find is by users. And for dl the archive is slow and not always works
btw CF can also be asked for a necessary actions

I like this idea. I'll do it for 126 R4 and Superfox.

@xixhub
SHA2-256 is actually SHA-256, i just explicitely specify version of SHA like in SHA3 :sweat_smile:
First - the purpose of this request is providing trusted hash to verify the file's authenticity from untrusted sources
Now explaination why SHA3
When we talk about targeted attack it's pretty easy to spoof compromised hashes, SHA-1 and MD-5 both is compromised and can't be used in security concerns, only for checking integrity from trusted sources.
Providing a size can slightly mitigate spoofing, but again there 100+ MB area for this
Single uncompromised hash has two issues: natural collision with looow chance and compromising (not all publicly available ... yet; SHA2 is about 23 years, SHA1 was publicly compromised at year 16 and the attack was published 6 more years later)
Two uncompromised hashes has same issues but with chance of chance
Finding a common collision for several compromised hashes can be expensive but i think it's still possible

VT is simply providing "fingerprint" (hash and other meta-info) of the files just for reference and without any warranty (you must not rely on hashes, you must check any susp).
Btw security is most important part when you are using obsolete software and/or officially unsupported/unavailable. so in XP case i will prefer to spend even 30 min with single core Athlon XP rather than having any consequences of not doing so.

Yep, i check and whole SHA3 family not work correctly in HashTab v<IP_ADDRESS> (latest for XP)
As another option BLAKE2sp can be considered

it's completely different hash function
it's released 12 years ago
it's available in latest 7zip that works under Win2K+
according to wiki it faster than SHA3 but has similar core-properties

HashTab issue

@win32ss ...
me when i realise R4 release :smiley:
me when i saw hashes

... seems i should had to provide some sort of template
but first - where did you get from those SHA(2)-256 hashes? seems, they are all incorrect, at least for me (see example, if this was not mistake i'm interested in tool, you used)
what about this?
 tag is native HTML5 spoiler, so, even browser not support it, its content will be displayed as plain text. I don't insist on it, just like it.

 is also native HTML and prevent some of text formatting, additionally github hadling it as code block so font is monospace.  XD there not much tools that even known about this... bookmarked...
wiki (end of second paragraph)

... SHA-512/256 are also truncated versions of SHA-512, but the initial values are generated using the method described in Federal Information Processing Standards (FIPS) PUB 180-4.

i suggested BLAKE2sp as an alternative to SHA3 by your initiative.
not so many offline tools supports SHA3, even fewer works on win7- (if we are talking about Win2K support it might be important)
on other side BLAKE2sp is a successor of BLAKE(finalist of SHA3 competition) and correctly works atleast in two famous tools - 7zip(Win2K+) and HashTab(v5,WinXP).
but i don't insist
btw, mentioned LAMY's tool not support BLACKE2sp, but can cuz underlying library does (you can even try yourself, but proper environment setup here probably more hard than adding hash support, and necessary examples can be found in tests, as always)

@win32ss, 1013 closed so i post here
ok, i confirmed my doubts that grabbing all from the directory was not the best idea
supermium_126_*_setup_premium.exe :smile:
lack of supermium_*_syms.7z
so here is the updated version of the script, posted in #1013 (comment)
- script and hasher tool should be placed in the same dir, but the targets can be anywhere
- name of the hasher tool should be hash.exe
- name of the targets can be any
- script configured for lamy' tool
new options
- script accepts targets as arguments, so simple drag&drop will work
- the include pattern can be disabled(still supermium_; comment the 7th line to use fallback)
- the exclude regexp(premium by default; comment the 8th line to disable; see findstr help)
- targets can be files and/or directories
- directories are non-recursively scanned for the files matching the include pattern
- all files(founded or inputs) are filtered by the exclude regexp, but not by the include pattern
- if exists, result can be appended - useful when all files cannot be lazy drag&droped in one pass
note: the include pattern can be easily changed to or accompanied with regexp and engaged in files filtering
@echo off & setlocal
set "searchIncPtrn=*"
set hasherTool="%~dp0hash.exe"
set newOutFile="%~dp0hash.out.txt"

rem next two lines can be commented to disable related features
set "searchIncPtrn=supermium_*"
set "filesExcRegEx=_premium"

if not exist %hasherTool% (
	echo can't find hasher tool %hasherTool%
	set /p "=press any key to exit "<nul & pause>nul & echo.
	goto :eof
)

if not "%filesExcRegEx%"=="" (
	rem BE CAREFUL with this line; it's not pritable but executable
	set "excludeCmd=(echo %%i|findstr /i /r "%filesExcRegEx%" 1>nul) ||"
)

if not exist %newOutFile% goto reset_result
rem uncomment for auto-append to result file, when it exists, and to avoid user interaction
rem goto append_result

set /p "resultAppend=Append to existing result? [y/any] "
if /i "%resultAppend%"=="y" goto append_result

:reset_result
rem overwrites result with new one; starts with putting necessary header
echo ^^Hashes^>%newOutFile%
echo.>>%newOutFile%
echo ^
>>%newOutFile%
echo.>>%newOutFile%

goto args_loop

:append_result
rem appends to result; copies original file, opens this copy and puts all, except footer, back to original file
set oldOutFile=%newOutFile:~0,-1%.old"
copy /y %newOutFile% %oldOutFile% >NUL
findstr /i /v /c:"
" %oldOutFile%>%newOutFile%

rem arguments for-like loop
:args_loop
set "targetObj=%~f1"
if "%targetObj%"=="" goto args_loop_end
if not exist "%targetObj%" goto args_loop_next

rem directory check; expects you have read permissions
if exist "%targetObj%\*" (
rem	echo is dir: "%targetObj%"
	set "targetObj=%targetObj%\%searchIncPtrn%"
)

for %%i in ("%targetObj%") do (
	%excludeCmd% (
rem	call echo [%%time: =0%%] processing %%~nxi
	echo %%~nxi : %%~zi bytes>>%newOutFile%
rem	call echo [%%time: =0%%] calculating sha2-256
	for /f %%j in ('start "" /b /wait %hasherTool% -a sha256 -f "%%i"') do echo SHA2-256 ^: %%j>>%newOutFile%
rem	call echo [%%time: =0%%] calculating sha3-512
	for /f %%j in ('start "" /b /wait %hasherTool% -a sha3/512 -f "%%i"') do echo SHA3-512 ^: %%j>>%newOutFile%
	echo.>>%newOutFile%
rem	echo.
))

:args_loop_next
rem echo processed "%~f1" & echo.
shift & goto args_loop
:args_loop_end

echo ^
^>>%newOutFile%
rem call echo [%%time: =0%%] done

set /p "=press any key to copy the result to the clipboard, close to exit "<nul & pause>nul & echo.
clip<%newOutFile%

endlocal


Need a section on Document Content profiles

Likely new section or sub-section on section 2.5
Point at wiki pages for content profiles (CDA, XDW, and FHIR-Document)
Start with Last paragraph in section 2.5
Add XDS-I

I propose that the existing text and the additional reading has covered this space well enough.


Android: Mark-on-Scroll, Beta Bugfixes

Implements the new mark-on-scrol option and fixes several bugs present in prod and beta.  Also did a pass of cleanup on warnings/lint/style.
Note that I did finally give in and bump our minimum platform version from 4.1 to 4.4 in line with what GOOG have stopped supporting.

Interesting, you chose to mark the story as read as soon as the top of it gets to the top line, not the bottom of the story (as we do on iOS and web). In other words, on android you see stories being marked as read as they scroll. On iOS and web, you don't. I think I may change it the Android method.

Release as 5.2.0b4.

Interesting, you chose to mark the story as read as soon as the top of it gets to the top line, not the bottom of the story (as we do on iOS and web). In other words, on android you see stories being marked as read as they scroll.

Yeah, pinged a few threads about the feature for tips and didn't get much of a response, so went with my gut.  I figured that if people generally read top-down, they are "read" as soon as they are seen?  I couldn't get the mark-on-scroll option to work on the web, so I went with that.
If it turns out to not be a great approach, the other impls. I could swap to without too much time cost would be to mark when a story (assuming top-down reading withing the list):

fully passed out of view to the top of the screen
hits a midpoint in the list
shows up in part at the bottom of the screen

No, I love it this way. I changed iOS to mark as read as soon as half of the title of the top story goes above the viewport.


Docs: please clarify how to vmap nnx.Module over batch dimension

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): wsl2 ubuntu 22.04
Flax, jax, jaxlib versions (obtain with pip show flax jax jaxlib: 0.8.5, 0.4.30, 0.4.30
Python version: 3.12 and whatever colab uses today
GPU/TPU model and memory: nvidia rtx 4090 24gb
CUDA version (if applicable): 12.5

Problem you have encountered:
made a nnx.Module to apply to single (1) input output pair example at a time using vmap
the model works on the first pass, but then on the 2nd pass it crashes
What you expected to happen:
expected vmapping a nnx.Module over a batch of data would not pass some shape on the first call and a different shape on the second call
Thought the docs could help but there's no mention of batch handling in there, seems like the batching for the given mnist example is assumed, which implies autobatching, but that just crashed when i tried it
tried various combinations of nnx jit, jax jit, and they all crashed with various shape bugs
doing one example of mnist at a time with no batching worked, but batching is pretty necessary
dont want to hard code models for some particular batch size, can you please clarify in docs how can we map a nnx.Module over a batch without needing to hardcode the batch size in the kernel dimensions? or, what am i doing wrong in the colab?
side note: i also hit bugs with dropout, the rngs context would crash, deterministic or not
i also wanted to make a dynamic length SGLD scan and early stop when the hessian indicated an optima but jax can't do this apparently :(
Logs, error messages, etc:

with jax.vmap

with nnx.vmap

Steps to reproduce:
Whenever possible, please provide a minimal example. Please consider submitting it as a Colab link.
NOTE: TO SAVE YOUR TIME, you can skip reading the 2nd codeblock as it's just copypasta to load mnist
https://colab.research.google.com/drive/11gKSydAn3p_fLdQaqw7pkdHmC2zyW12k?usp=sharing

Hey! In the NNX Transforms section I think we make it clear that you must use the nnx.* version of the transform when using NNX objects. Is there something we should improve here?

yes, you're right, the docs are clear to use the nnx transforms
I hit an issue where, when using these transforms, the first time I call the transformed function, it receives a single slice of the batch, which is what I was going for, but then when it gets called again, it receives the whole batch at once, which caused it to crash due to a shape issue in a call to "concatenate"
I reckon it's using an XLA version of the same function on the second invocation
For what to improve, could the docs have a section dedicated to advice about how to work with batches of data?
I was looking for more focused troubleshooting docs to deal with shape bugs in handling batches with vmap, or advice on how to apply a function over slices of a batch independently without having to change the function to handle the whole batch at once
(it's an EBM / energy based model, so I don't want to inappropriately mix energy gradients across the members of the same batch, and it breaks if I try to apply the single-item function to the whole batch)
Might just be my general inexperience with jax and a vmap issue and not a nnx issue; maybe i am missing something about how vmap works

I think currently you want this:
(loss_values, (y_preds, the_energies, accuracies)), grads = nnx.vmap(
  loss_value_and_grad_fn,
  in_axes=(None, 0, 0),
  state_axes={...: None},  # <<== add this
)

But in the near future of #3963 , you will be able to do
(loss_values, (y_preds, the_energies, accuracies)), grads = nnx.experimental.vmap(
  loss_value_and_grad_fn,
  in_axes=(None, 0, 0), # <<== just works
)

and hopefully experimental becomes the norm


An error occurred: unknown flag: --snapshotVolumes

What steps did you take and what happened:
./ark backup create mano-brown_20181130_1 --snapshotVolumes
What did you expect to happen:
To create volume snapshots
Environment:

Ark version (use ark version): v0.10.0
Kubernetes version (use kubectl version): v1.11.2-gke.18

@gandarez the flag you're looking for is --snapshot-volumes (though you don't really even need to specify it explicitly - as long as you have a VolumeSnapshotLocation configured, snapshots will automatically be taken).

voila!


checkJs: Tuple - optional elements

Hello,
it seems like type checking in JS files provided by VS Code cannot handle optional elements in tuples.
Code
/**
 * @param {[string, number, number?]} val1
 */
function f(val1){}

f(['hi', 4])  <-- error

Error message:
Argument of type '[string, number]' is not assignable to parameter of type '[string, number, number]'.
Property '2' is missing in type '[string, number]' but required in type '[string, number, number]'.ts(2345)
TypeScript itself is handling it as expected: Playground Link

Search Terms:
tuple optional
VS Code Info:
Version: 1.44.0 (user setup)
Commit: 2aae1f26c72891c399f860409176fe435a154b13
Date: 2020-04-07T23:31:18.860Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

Fixed by https://github.com/microsoft/TypeScript/pull/39123, which means that #38747 was a duplicate of this. Sorry I missed this before.


Cut integer parsing

Changes Unknown when pulling 699035dd12cd7abaf479e7a5f602af68243e96c6 on Code4SA:cut-integer-parsing into ** on openspending:master**.

This ensures that a cut value such as "field:2015M01" is treated as ("field", ":", "2015M01") and not ("field", ":", 2015) as is currently the case.

This has the limitation that the value 01 is parsed as the integer 1 and not the string 01, so it won't match 01 in a string column in the DB as expected. See https://github.com/Code4SA/babbage/pull/4 for an alternative to this.


Multitarget support

🚀 Feature Requests
Multitarget support when executing nukeeper
Contextualize the feature
Option when execute CLI to allow different behavior on different target when multitarget is used
Describe the feature
I have this king of configuration and I would like to address this in NuKeeper. Of course I could exclude those package but maybe it could be done by excluding specific target? Or maybe you have better idea - in this sample I have nothing to update in 3.1 but in other cases maybe there would be some packages that I will want to update but not touch those in 2.1 target.
<ItemGroup Condition="'$(TargetFramework)' == 'netcoreapp3.1'">
        <FrameworkReference Include="Microsoft.AspNetCore.App" />
    </ItemGroup>
    <ItemGroup Condition=" '$(TargetFramework)' != 'netcoreapp3.1'">
        <PackageReference Include="Microsoft.Extensions.Configuration" Version="2.1.1" />
        <PackageReference Include="Microsoft.Extensions.Logging" Version="2.1.1" />
        <PackageReference Include="Microsoft.Extensions.Options" Version="2.1.1" />
    </ItemGroup>

Platforms affected (mark all that apply)

[X ] :hammer_and_wrench: NuKeeper CLI

I think supporting this use case is too specific; I can imagine that there are many more conditions than just target framework that could result in requiring specific packages to be installed. I'm thinking of maybe feature flags so compiling with certain features enabled or not (I haven't personally worked on a project that had such a requirement, or supported something like this, so I'm sure other people might be able to come up with additional examples).
What would make sense in my opinion is to add extensibility points (hooks) in nukeeper where you would be able to modify for example, which packages actually end up getting selected. In your case, you would be able to write a hook for a target framework.
What do you think?

Extensibility point sound good - I understand you don't want to handle each case.
ps. This case is obsolete for me now  becouse we now have only 3.1 ;)


Add support for date +/- interval year month Presto functions

Summary:
Add support for Prestissimo queries like
... date + interval '1' year
... date - interval '6' month

Differential Revision: D54344518

It might also worth adding +/- IntervalYearMonth support to Timestamp type: https://github.com/facebookincubator/velox/pull/8848