Support for encoding with adaptive filtering

This can considerably reduce the size of some images, but takes longer to execute.
The 4th paragraph of this section in the PNG standard recommends adaptive filtering for best compression.
https://www.w3.org/TR/PNG/#12Filter-selection
This paragraph better describes the method for adaptive filtering.
http://www.libpng.org/pub/png/book/chapter09.html#INDEX-688

For grayscale and truecolor images of 8 or more bits per sample, with or without alpha channels, dynamic filtering is almost always beneficial. The approach that has by now become standard is known as the minimum sum of absolute differences heuristic and was first proposed by Lee Daniel Crocker in February 1995. In this approach, the filtered bytes are treated as signed values--that is, any value over 127 is treated as negative; 128 becomes -128 and 255 becomes -1. The absolute value of each is then summed, and the filter type that produces the smallest sum is chosen. This approach effectively gives preference to sequences that are close to zero and therefore is biased against filter type None.

It's worth noting that https://github.com/brion/mtpng seems to already support this

I've implemented adaptive filtering but it needs some design work. I can submit a PR for review if you think it's in the right direction.
Here's the branch: https://github.com/okaneco/image-png/commit/0104fd716f3ac6e558f68c7187b8cbd532efd6d9
FilterType enum
I've added Adaptive to the enum with a value of 5. An issue I see is that Adaptive is not a valid decoding filter type. As an encoding filter, it delegates to the first five algorithms: NoFilter, Sub, Up, Avg, and Paeth and is used in filter(...).
unfilter(...) in filter.rs
Return an Error on matching with Adaptive. Should filters be non-exhaustive? I'm not aware of heuristics that perform better than these 5+1 algorithms, and it doesn't make sense to match on Adaptive in decoding. Having an EncodingFilterType and DecodingFilterType seems redundant.
filter(...) in filter.rs
current signature
pub(crate) fn filter(method: FilterType, bpp: BytesPerPixel, previous: &[u8], current: &mut [u8])

modified signature
pub(crate) fn filter(method: FilterType, bpp: BytesPerPixel, previous: &[u8], current: &mut [u8]) -> u8

I've changed the function signature so it returns a u8 to signal the filter type. I wasn't sure of the best way to communicate this back to the writers that call filter so that they can write the filter byte to the scanline. This could also return a FilterType and cast to u8 at the caller sites.
I've moved the row operations for each filter into their own functions to avoid code duplication in the Adaptive match arm.
Tests/Benchmarks
I'm unsure of a test design or validation for the results.
I've compared all the filters for size and it seems to work (this is how I discovered #263) as expected. It's not the smallest file size for every image but it's consistently towards the better performing filters or even with Paeth. I've only tested with default compression level and Best level, the advantages of Adaptive diminish with the higher compression levels. It matches my past experience with using imagemagick's -quality for pngs where -quality 95 should be roughly equivalent to Best compression and Adaptive filtering.

Another option would be to pass around an Option<FilterType> and have the None value mean adaptive. I'd also favor using our FilterType enum rather than a raw u8 where possible.


MMap with "NoThreads" Flag Crashes with 0xc00000225

Hello Darthton,
I'm having issues with using your MMap with thread hijacking. Every time the injection runs in my application it will crash the target program and then return 0xc00000225.
I've tried the thread hijacking Mmap with many different dlls and many different nonWOW64 processes but I always keep crashing. Perhaps there is a bug in the code, I'm not sure.

++ Problem with last widnows version and kernel injection
From log: Started on Windows <IP_ADDRESS>99 x64. Driver status: 0xC0000225

firstly google that
For Kernel MMap this status
0xC0000225
STATUS_NOT_FOUND
means that it can not find the driver file
For inject without driver, compile blackbone in debug mode and get trace log from it

I'm not using the driver, I'm not sure why boris and Roma seem to think that I am using the driver for manual map when the driver does not even support the "NoThreads" flag.
I figured out that the 0xC0000225 error was due not being able to resolve the dependencies but I have since fixed this issue. Now I am getting the 0x8000000d error with every process I seem to inject into with the NoThreads option since every process I inject into seems to crash.
Here are the logs:
 BlackBone: Allocate: Allocating at address 0x00000276C3810000 (0x1000 bytes) BlackBone: Allocate: Allocating at address 0x00000276C3820000 (0x4000 bytes) BlackBone: Allocate: Allocating at address 0x00000276C3830000 (0x1000 bytes) BlackBone: ManualMap: Hijacking Thread! BlackBone: ManualMap: Created APC Thread! BlackBone: ManualMap: Mapping image 'TEST.dll' with flags 0x81 BlackBone: ManualMap: Loading new image 'TEST.dll' BlackBone: Allocate: Allocating at address 0x00007FFA03680000 (0x7000 bytes) BlackBone: ManualMap: Image base allocated at 0x00007ffa03680000 BlackBone: Allocate: Allocating at address 0x00000276C3C70000 (0x200 bytes) BlackBone: Free: Free at address 0x00007FFA03680000 BlackBone: Free: Free at address 0x00000276C3C70000 BlackBone: Free: Free at address 0x00000276C3830000 BlackBone: Free: Free at address 0x00000276C3820000 BlackBone: Free: Free at address 0x00000276C3810000 Couldn't map with error: 0x8000000d

It seems that this line
   thd->Resume();     if (NT_SUCCESS( status ))     {         WaitForSingleObject( _hWaitEvent, 20 * 1000/*INFINITE*/ );         status = _userData.Read( RET_OFFSET, callResult ); <------------------- this line     }
Is causing the crash in the process. This code has consistently triggered every x64 process that I have injected into (Chrome, Firefox, etc) to crash.  any ideas on why this might be crashing?

I've just tested "NoThreads" flag with my x64 dll and everything worked flawlessly.
Perhaps your dll is causing this? Does it work without the flag or using native injection?

The injection seems to work fine without the "NoThreads" flag

Is there any way you could write a test for the "NoThreads" flag with the exact code that you're using with Google Chrome as the target process?


Check blob upload progress?

We are using the azure-storage Python package in our Django app to upload large files. I found the following documentation for showing the progress via node:
https://github.com/Azure/azure-storage-node/issues/310
What is the analog for doing the same in Python?
Thanks!

Hi @bbirand, the create_blob_from* methods take in an argument called progress_callback.
And here is a sample.


a problem on Lustre grammar supported by Kind2

Hi, I tried to use kind2 on my own lustre code. Although I followed the rules in syntax_test.lus and decided to declare some types like
type sint8 = subrange [-128, 127] of int;
type uint8 = subrange [0, 255] of int;

but there seems to be a syntax error in line2:5, which makes me confused.
Is there anything wrong with my code?
Thank you!

uint8 is a reserved word for the type of an unsigned machine integer of width 8. I'll look if we can provide a more informative message.

The issue has been resolved after I change uint8 to usint8.
Thanks!


[feat] Theme setter

Describe the problem
since Tauri offers you the current system theme using appWindow.theme(), we should have a Theme setter so that we can easily make Theme systems in our apps and especially ones that are adapting to Tauri from example Electron.
Describe the solution you'd like
Add a theme setter on appWindow, for example: appWindow.theme = "light".
Alternatives considered
Currently only HTML/CSS based ones.
Additional context
No response

duplicate of https://github.com/tauri-apps/tauri/issues/5279


Revert "Fix deploy config"

Reverts armory/docs-cdaas-sample#14

The title of this Pull Request needs to be changed to match the format:
<type>(<scope>): <subject>

See more details here: https://www.spinnaker.io/community/contributing/submitting/#commit-message-conventions
Some common examples are:
feat(armory.io): add docs.armory.io
fix(parser): grammar and spelling
style(loadingPage): made it super pretty!
refactor(helpMessages): changed order to make more sense


Improve future::get() and future::wait()

Previously, future::get() and future::wait() would acquire a mutex whenever they were invoked. This pull request adds double-checked locking to reduce overhead when calling these functions after the result has been made ready.
Other improvements in this PR are:

Deferred functions will no longer hold a mutex while notifying other threads about function completion.
Type flags have been cleaned up, and should be easier to understand.
Tests of future now check to make sure that asynchronous and deferred tasks do, in fact, complete as would be expected from those execution policies (asynchronous on a different thread, deferred on the thread that calls wait() or get()).

Thanks @nmcclatchey, apologies for the delay!


⚠ Bofrost.at has degraded performance

In 3ce58e4, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 4307 ms

Resolved: Bofrost.at performance has improved in ad33e08 after 20 minutes.


Update documentation

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   95.23%   95.23%
=======================================
  Files          16       16
  Lines         147      147
=======================================
  Hits          140      140
  Misses          7        7

Continue to review full report at Codecov.

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


Backports of hashtable fixes

https://phabricator.endlessm.com/T13987

Did not find anything evidently wrong in the code, check tests passed, and other tested passed as well (e.g. our ekl tests). The main difference with these patches is that I no longer get segfaults when running emeus  ascii art examples.
Should someone else take a look?

According to the documentation g_direct_hash is still called when you pass NULL, so I left that as it was, but g_direct_equal is inlined, so I replaced that with NULL. Also posted to the upstream bug

CR pass.


Implicit attribute defaults/overrides inside package definition

Issue description
Currently, in order to override a package's arguments in order to i.e. pin a version of a library, you must do so at the callPackage location, usually all-packages.nix. This has a few issues:

all-packages is huge

Easy to run into conflicts
Chaotic/Confusing
Hard to work with (can crash some LSPs)

Source of truth is split

It is no longer obvious from looking at a package definition file what its arguments are (they might be overridden)
Maintainers might forget/never now that overrides are in place (who here frequently looks inside all-packages for their packages?)
It just feels wrong to have this info in a separate place

Requires explicit imports

Works against https://github.com/NixOS/rfcs/pull/140
The current workaround is to explicitly callPackage the by-name path in all-packages (officially documented)

I've been a fan of explicit requirements; i.e. depending on ffmpeg_5 rather than ffmpeg with an ffmpeg = ffmpeg_5; override in all-packages. This also has issues, the primary one being that overrides aren't stable anymore and are even less intuitive as you'd have to override i.e. ffmpeg_5 = ffmpeg_6;.
IMHO, the benefits outweighed the downsides as overrides aren't really a stable thing to begin with but I'd prefer to fix this issue "proper". In order to do that we must either put the package definition into all-packages (yeah, no) or put the overrides into package definitions.
This issue exists as a collection point for ideas on how to realise this.
Related discussion: https://github.com/NixOS/nixpkgs/issues/204303

The first issue has another point - github won't even index the file. Though it should be massively improved with the move to by-name

Yeah this problem has been discussed with the Nixpkgs Architecture Team throughout RFC 140. I don't have any concrete proposals nor the resources to come up with one for now, but if anybody else does, it would be great to synchronise with the NAT for that.

dream2nix deps pattern may be interesting to at least pull the default overrides into the package file.

This also has issues, the primary one being that overrides aren't stable anymore and are even less intuitive as you'd have to override i.e. ffmpeg_5 = ffmpeg_6;.

:warning: It would be really great if we could keep version numbers out of the attrnames.  You've pointed out one of the problems this causes; there are lots of others.
Analogy to multi-platform packages
This has close parallels with how we handle platforms.  We don't have separate toplevel ffmpeg_x86-linux and ffmpeg_aarch64-linux attributes -- instead, ffmpeg is a "spliced" collection of packages, one for each platform.
We should do the same thing with versions.  Each top level attribute should be not a single package (which hasn't been true since __spliced happened) but rather a collection of all available versions of that package.
Version overrides then become straightforward: you simply delete from the __versions set any versions you know won't work with your package.
Proof of Concept
I have a prototype here but it's nowhere near PR-ready.  I developed it against gcc, since that is by far the gnarliest case of multiversioned packaging in nixpkgs.
Some repl to give an idea of what it feels like:
nix-repl> gcc
«derivation /nix/store/20z7kwqc1fp8v4fmmm1p9jmvig3r6cys-gcc-wrapper-13.2.0.drv»

nix-repl> gcc.__versions.best
«derivation /nix/store/20z7kwqc1fp8v4fmmm1p9jmvig3r6cys-gcc-wrapper-13.2.0.drv»

nix-repl> gcc.version
"13.2.0"

nix-repl> builtins.attrNames gcc.__versions
[ "10.5.0" "11.4.0" "12.3.0" "13.2.0" "4.8.5" "4.9.4" "6.5.0" "7.5.0" "8.5.0" "9.5.0" "__unfix__" "best" "extends" "require" "unstable" ]

nix-repl> gcc.__versions.require "<10"
«derivation /nix/store/xc0hx0ijajgxjvdlrv9k92xlwfvrbb2b-gcc-wrapper-9.5.0.drv»

nix-repl> (gcc.__versions.require "<10").version
"9.5.0"

nix-repl> (gcc.__versions.require "<10").__versions.best.version
"9.5.0"

Notice how there is a reference cycle pkg.__versions.best.__versions.best... so you can always get a reference to the version-set from any of its members and vice-versa.
Highlight commits in the PoC

lib.versions.makeVersioned, where the magic happens.  This is just an early draft; there are several issues that need to be fixed but all of them seem straightforward.

Note in particular the __versions.unstable attribute, which generalizes pkgs.default-gcc-version into a mechanism that can be applied to any package instead of just gcc.

Conversion of pkgs/development/compilers/gcc to __versions style, which was easier than I expected.
what all-packages.nix looks like after converting gcc.  The remaining gcc${version} attributes are for legacy compatibility; they should then be inlined at their point of use, allowing to move the primary pkgs.gcc attribute out of all-packages.nix and into pkgs/by-name.

We really ought to have a place for abstract package information that isn't specific to a build of a package, or we'll keep incurring the performance and complexity cost of unused package evaluations / unnecessary overrides.
For instance, evaluating gcc.__versions.something requires that we evaluate gcc, as well as something, which has a real performance cost when applied at a scale that isn't tiny. Also overrideAttrs isn't particularly cheap. As for complexity, there's just less division of responsibilities, and the performance problem implies a strictness problem - e.g. gcc may fail to evaluate (well, more likely for other versioned packages perhaps). Also it's not possible to pass an exact version without the possibility that some code inspects its __versions to get something else instead. This could turn out a bit chaotic.
All that said, I'm glad to see some experimentation happen in this area.
#217243 was meant to mitigate some of this, but mkDerivation tech debt is bankrupt. I've put a plan in the bottom comments, but versioning should not be shoehorned into mkDerivation, regardless of whether we execute on such a plan or not.
nitpick: @amjoseph-nixpkgs It is my understanding that the convention around __ is that _ is reserved for libraries/frameworks/infra defined in expressions, where as __ is reserved for things defined and/or handled by Nix itself. There's definitely a gray area, but your __version is an instance of the prior.

For instance, evaluating gcc.__versions.something requires that we evaluate gcc, as well as something, which has a real performance cost when applied at a scale that isn't tiny.

With lazy attribute sets this could be improved. I'd like to pick up that PR again

I don't think we should invent language features to patch architectural flaws; the lack of a package concept that's not instantiated. I know it may have other uses, but this one basically doesn't count.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/why-is-there-only-one-version-in-nixpkgs/47773/3

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/why-does-the-nixos-infrastructure-have-to-be-hosted-in-a-centralized-way/46789/49

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/discourse-via-email/29/13

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/review-of-333575/50496/1

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/review-of-333575/50496/3

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/review-of-333575/50496/5


Bump install release versions for SLS Gallery.

Ready to deploy now OMERO 5.6.3 is released.

No, I'll go ahead and deploy.

:+1: otherwise as https://github.com/ome/prod-playbooks/pull/289#issuecomment-702658260

Now deployed.


Use RFCs and ADRs to record decisions

Bootstrap this repo by proposing it into existence!
The first commit is the proposal itself, the second commit is the proposed implementation.
This is loosely based on alphagov/govuk-rfcs as well as other internal RFC repos, adapting them to make them more suitable for a RFC process where we expect feedback from a wider community.
This proposal will remain open until 16 March 2018.

:+1: Looks great. "Proposal" and "Decision Record" are clearer names than RFC and ADR to my ears.

@nickcolley Good call – I've added links to alphagov's code of conduct.

Like the idea. I think it'd be helpful to add some links to similar repositories elsewhere. eg https://github.com/apple/swift-evolution

Do we need to include licensing information?

Do we need to include licensing information?

I'll do this as part of a separate PR.


Add pagination to channel pages

[x] Switched channel claims to lookup from claims.byId
[x] Added infinite scroll

For some reason the container doesn't auto resize though and I haven't been able to figure out why yet.

@6ea86b96 @kauffj @lyoshenka
I have some suggestions regarding this.

Instead of auto-scrolling it should be paginated, as infinite scrolling is a bit tedious if you have to find some thing for the nth time. So there should be page numbers at the bottom for navigation.
It should append some prompt on the address bar such as lbry://url&page=3 or lbry://url:3 so that user can type in that and directly reach to that page, or share the page with people.
(Super Optional) Maybe the user could set the no. of results to be displayed on one page?

P.S. #351 is dependent on this.

Good point. Pagination would be a lot more usable.

@6ea86b96 did you want me to review/merge this as-is? I got the impression you were planning to come back to it.

@kauffj Removed you as a reviewer for now. I like the pagination suggestion. I'm going to come back to this after the channel surfing rewards fix.

Added pagination, just need to get the css working and it's good to go.

@kauffj pagination is working now. Doesn't look that amazing. CSS is not my strong point :)

@6ea86b96 I was just trying it out, but I found one issue, it seems to work with the provided version of the lbrynet-daemon but not with the latest version(with the latest lbryum and schema). One possible reason I've singled out is that it may be happening due to this PR. Can you please check it out once?

@hackrush01 it worked fine for me after I rebased onto master.
@6ea86b96 merged. Added my own CSS, which I also don't like. I did clean up the class names, though.


Check for page loading bug in qt-wk experiments

With alfred 0.2b5, users would sometimes experience issues with page loading in qt-wk experiments. I will ask around for a more specific description, but as far as I remember, sometimes pages would not load in the webkit browser window and we would get a warning message from the flask server. A relatively easy fix was to just F5 or right click reload the empty page and the experiment would correctly continue.
As far as I know, there was no systematic pattern in the occurence of this glitch, but it never happened in pure web experiments. Maybe the issue is already resolved through the new webkit version we now use, but we have to keep an open eye for this type of issue un qt-wk experiments when testing.

We removed QT-WK.


Added null check for the officers service address elements

Added null check for the officers service address elements
Resolves BI-9268

This appears to have already been handled by the getString method, so closing this PR.

This appears to have already been handled by the getString method, so closing this PR.


GetUsedCCLanes() to return (Bank/Program Select) upon PC

Fix for #1344

:white_check_mark: Build sws 275-master completed (commit https://github.com/reaper-oss/sws/commit/082640d150 by @icalvo)

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=x86_64

build/sws-<IP_ADDRESS>-Linux-x86_64-082640d1.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=i686

build/sws-<IP_ADDRESS>-Linux-i686-082640d1.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Visual Studio 2019, ARCH=x64

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-082640d1.exe

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=aarch64

build/sws-<IP_ADDRESS>-Linux-aarch64-082640d1.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=armv7l

build/sws-<IP_ADDRESS>-Linux-armv7l-082640d1.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Visual Studio 2019, ARCH=x86

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-082640d1.exe

Thanks for the PR!
https://github.com/reaper-oss/sws/pull/1345/files#diff-a084b794bc0759e7a6b77810e01874f2
I once had a conversation with @cfillion about adding these to .gitignore, quoting his opinion here:

Makes sense. I didn't even know about .git/info/exclude, thanks! I will revert the .gitignore.

Can we add that .gitignore insights to the Wiki? Not sure how to do it myself.

:white_check_mark: Build sws 276-master completed (commit https://github.com/reaper-oss/sws/commit/4900d25f9c by @icalvo)

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Visual Studio 2019, ARCH=x64

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-a77ff043.exe

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=aarch64

build/sws-<IP_ADDRESS>-Linux-aarch64-a77ff043.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Visual Studio 2019, ARCH=x86

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-a77ff043.exe

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=armv7l

build/sws-<IP_ADDRESS>-Linux-armv7l-a77ff043.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=i686

build/sws-<IP_ADDRESS>-Linux-i686-a77ff043.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=x86_64

build/sws-<IP_ADDRESS>-Linux-x86_64-a77ff043.tar.xz

Can we add that .gitignore insights to the Wiki? Not sure how to do it myself.

Good idea.
Suggestion:

About included .gitignore file:
.gitignore only includes files generated as part of the build process.
So local config files/folders (such as .vscode, .vs/, CMakeSettings.json) are purposely not included.
Contributers may add those to their user-wide .gitignore or .git/info/exclude for sws-specific personal ignore rules (see e.g. here).

Anything to change/add?

I didn't refer to the wording (although yours seems perfect to me) but to how to add things to the Wiki. It doesn't let me edit the pages.

Thanks, added.

Merged into master as bb55a817de35bfbfab389930108aa056207cefd9.


VariableNameUtils#generateVariableName() should consider following statements

When using VariableNameUtils#generateVariableName() to generate variable names to be inserted at a given position in the code (like it is being used in the ZipSlip recipe of rewrite-java-security, which is the only use I found), then the logic should also consider the statements that follow the given cursor (which will roughly be the insertion point for the variable), otherwise when a new variable is inserted it could conflict with following variables.
In the following example the utility would for a base name of s calculate the name s1 even if that conflicts with a variable that follows. Thus, inserting a variable would render the class uncompilable.
class A {
  void test() {
    String s = null;
    ; // <-- insert new variable with base name "s" here
    String s1 = null;
  }
}

I am currently facing this issue in the InstanceOfPatternMatch recipe.

Also note that VariableNameUtils is currently unaware of the flow scope introduced by the instanceof pattern matching. See the following test, where the commented out test cases should actually also succeed. At least the probe1 case should be supported. I think the others are probably quite rare.
    @ParameterizedTest
    @CsvSource(value = {
//      "probe1:probe1,s1,o",
      "probe2:probe2,o",
      "probe3:probe3,probe2,o",
//      "probe4:probe4,n,probe2,o",
//      "probe5:probe5,n,probe2,o",
    }, delimiter = ':')
    void instanceOfPatternFlowScope(String scope, String result) {
        rewriteRun(
          baseTest(scope, result),
          java(
            """
              class Test {
                  void m(Object o) {
                      if (o instanceof String s1) {
                          Object probe1 = s1;
                      }
                      Object probe2;
                      if (!(o instanceof Number n)) {
                          Object probe3;
                          throw new IllegalStateException();
                      } else {
                          Object probe4 = n;
                      }
                      Object probe5 = n;
                  }
              }
              """
          )
        );
    }

@traceyyoshima I should probably report a separate issue for this. What do you think?

Also note that VariableNameUtils is currently unaware of the flow scope introduced by the instanceof pattern matching. See the following test, where the commented out test cases should actually also succeed. At least the probe1 case should be supported. I think the others are probably quite rare.

Hi @knutwannheden, yes, I think these should be two separate issues to make the work clearer.


[Home] Selective Pull-To-Refresh #tiny

This pr makes our pull-to-refresh implementation a bit smarter. By calling forceFetch on the Home container, it ensures all entirely new modules are fetched every time. It will then check for modules that are frequently updated and forceFetch them individually instead of refreshing every single rail every time.

          1 Error

:no_entry_sign:
No CHANGELOG added.

          1 Warning

:warning:
The following files do not have tests: lib/containers/home.js

  Generated by :no_entry_sign: danger

When metaphysics updates a rail like Recommended Works with new content, is the __id also updated?

When metaphysics updates a rail like Recommended Works with new content, is the __id also updated?

It does not. We could maybe do this by adding the ETag from the Gravity response, not sure right now.

So when do we refresh the rails that are not often updated? Once a day?

Ideally, yes. I haven't found a way to do that yet. If we can't figure it out by release day, maybe forceFetch for all rails every time makes more sense. It just feels inefficient and the little engineer person in my head is bothered by that.

It just feels inefficient and the little engineer person in my head is bothered by that.

Totes, I get that, but I’d prefer that we include those fixes before merging, just so we don’t accidentally ship something that could leave the home page ‘broken’ (out of date content).

I don’t think there’s a way from RN to know that a view became visible (e.g. when going back to the home tab in Eigen’s tab bar), so we might need to trigger something from the objc Home component VC. Then in that hook we check when the last refresh took place and force a full one if it’s over N time.

I’m closing this one, we’ll have to revisit it soon.

Hmmmm, my Safari extension auto deleted the branch, that wasn’t really my intention. The diff is still available, though, so we can always cherry-pick.


In tables make  "Selected Cell Type" more readable

Need
The Selected Cell Type section is very narrow at the moment and difficult to read.
Approach
Let's come up with some ways to widen this or to show more detail on hover or click of (+3 more ) or similar.

resolved in dx UI


Release status for 0.6.6

[ ] Dev Complete
[ ] Test Signoff
[ ] Perf Signoff
[ ] License Check
[ ] Build and Deploy docker Image

@ShishirH, are there any new packages / dependencies since the last release ?

No new packages or dependencies in this release


After the last update sometime it fills all input field with email id. note: I didn't change any settings.

Can you please provide a page and settings backup that reproduces this problem?


Determine heart rate

Story points: 0.5
Just read a senzor

folosind un fisier extern din care se citesc datele la senzor

depilcated issue #15

when finished please done also #35 - set and send sensor data to endpoint

Done here https://github.com/SmartSleepIoT/SmartSleepCoding/commit/d4f46191d0b7b255387072f8cf5681f78b6c42ea


Regarding segmentation task

Hi@simonJJJ I'm trying to inference the segmentation task of pretrained model using one-peace-vision.pkl with command  python test.py /home/Documents/ONE-PEACE/one_peace_vision/seg/configs/ade20k/mask2former_onepeace_adapter_g_896_40k_ade20k_ss.py "/home/Downloads/one-peace-vision.pkl" --show I encountered the below error. What could be the correct way to inference ?
Traceback (most recent call last):
File "/home/Documents/ONE-PEACE/one_peace_vision/seg/test.py", line 351, in
main()
File "/home/Documents/ONE-PEACE/one_peace_vision/seg/test.py", line 243, in main
checkpoint = load_checkpoint(model, args.checkpoint, map_location='cpu')
File "/home/.local/lib/python3.9/site-packages/mmcv/runner/checkpoint.py", line 581, in load_checkpoint
checkpoint = _load_checkpoint(filename, map_location, logger)
File "/home/.local/lib/python3.9/site-packages/mmcv/runner/checkpoint.py", line 520, in _load_checkpoint
return CheckpointLoader.load_checkpoint(filename, map_location, logger)
File "/home/.local/lib/python3.9/site-packages/mmcv/runner/checkpoint.py", line 285, in load_checkpoint
return checkpoint_loader(filename, map_location)
File "/home/.local/lib/python3.9/site-packages/mmcv/runner/checkpoint.py", line 302, in load_from_local
checkpoint = torch.load(filename, map_location=map_location)
File "/home/.local/lib/python3.9/site-packages/torch/serialization.py", line 815, in load
return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
File "/home/.local/lib/python3.9/site-packages/torch/serialization.py", line 1035, in _legacy_load
raise RuntimeError("Invalid magic number; corrupt file?")
RuntimeError: Invalid magic number; corrupt file?
I also checked with
python test.py --cfg-options model.pretrained="/home/Downloads/one-peace-vision.pkl" /home/Documents/ONE-PEACE/one_peace_vision/seg/configs/ade20k/mask2former_onepeace_adapter_g_896_40k_ade20k_ss.py --show
but getting error as
Traceback (most recent call last):
File "/home/Documents/ONE-PEACE/one_peace_vision/seg/test.py", line 351, in
main()
File "/home/Documents/ONE-PEACE/one_peace_vision/seg/test.py", line 123, in main
args = parse_args()
File "/home/Documents/ONE-PEACE/one_peace_vision/seg/test.py", line 105, in parse_args
args = parser.parse_args()
File "/home/.conda/envs/onepeace/lib/python3.9/argparse.py", line 1825, in parse_args
args, argv = self.parse_known_args(args, namespace)
File "/home/.conda/envs/onepeace/lib/python3.9/argparse.py", line 1858, in parse_known_args
namespace, args = self._parse_known_args(args, namespace)
File "/home/.conda/envs/onepeace/lib/python3.9/argparse.py", line 2067, in _parse_known_args
start_index = consume_optional(start_index)
File "/home/.conda/envs/onepeace/lib/python3.9/argparse.py", line 2007, in consume_optional
take_action(action, args, option_string)
File "/home/.conda/envs/onepeace/lib/python3.9/argparse.py", line 1935, in take_action
action(self, namespace, argument_values, option_string)
File "/home/.local/lib/python3.9/site-packages/mmcv/utils/config.py", line 711, in call
key, val = kv.split('=', maxsplit=1)
ValueError: not enough values to unpack (expected 2, got 1)

Hi @alekhyaparimala, you should use this  checkpoint for inference.
And one-peace-vision.pkl is for fine-tuning.

Hi @alekhyaparimala, you should use this checkpoint for inference. And one-peace-vision.pkl is for fine-tuning.

Thanks, but when I used this checkpoint file, i'm getting error as /home/T7724A1/Documents/ONE-PEACE/one_peace_vision/seg/test.py:299: UserWarning: SyncBN is only supported with DDP. To be compatible with DP, we convert SyncBN to BN. Please use dist_train.sh which can avoid this error.
warnings.warn(
Traceback (most recent call last):
File "/home/T7724A1/Documents/ONE-PEACE/one_peace_vision/seg/test.py", line 358, in
main()
File "/home/T7724A1/Documents/ONE-PEACE/one_peace_vision/seg/test.py", line 309, in main
model = build_dp(model, cfg.device, device_ids=cfg.gpu_ids)
File "/home/T7724A1/.local/lib/python3.9/site-packages/mmseg/utils/util_distribution.py", line 28, in build_dp
model = model.cuda()
File "/home/T7724A1/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 905, in cuda
return self._apply(lambda t: t.cuda(device))
File "/home/T7724A1/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 797, in _apply
module._apply(fn)
File "/home/T7724A1/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 797, in _apply
module._apply(fn)
File "/home/T7724A1/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 797, in _apply
module._apply(fn)
[Previous line repeated 1 more time]
File "/home/T7724A1/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 844, in _apply
self._buffers[key] = fn(buf)
File "/home/T7724A1/.local/lib/python3.9/site-packages/torch/nn/modules/module.py", line 905, in
return self._apply(lambda t: t.cuda(device))
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 76.00 MiB (GPU 0; 3.81 GiB total capacity; 2.99 GiB already allocated; 30.88 MiB free; 3.03 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
I checked with clearing the cuda cache, still same issue

@alekhyaparimala Your GPU memory seems lower than the inference requirement. I suggest using alibaba cloud GPU service or google colab for inference.

@alekhyaparimala Your GPU memory seems lower than the inference requirement. I suggest using alibaba cloud GPU service or google colab for inference.

Sure, can I use cpu instead?

@alekhyaparimala Sorry, we do not support cpu inference now.


⚠ GitLab Server has degraded performance

In 44f83aa, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 9116 ms

Resolved: GitLab Server performance has improved in 5804f30.


feat(KRY-624): A more intelligent SideKick

What it Does

Use additional info from QuickSilver to make more intelligent decisions on traffic routing

if cluster is unhealthy, automatically try to hit S3
if cluster is healthy, split traffic between crunch and s3 based on the traffic splitting param

Rename things all around since QuickSilver no longer just returning endpoints
Add tests

if cluster is healthy, split traffic between crunch and s3 based on the traffic splitting param

is this read or write traffic. Are we handling read and write separately?
Is the fallback behavior included in this PR?

if cluster is healthy, split traffic between crunch and s3 based on the traffic splitting param

is this read or write traffic. Are we handling read and write separately? Is the fallback behavior included in this PR?
Some speculation on Traffic Split for writes, fallback for reads for an inline crunch workflow If traffic split > 50% -ping Bolt first for reads -fallback to S3 for writes -fallback to S3 for reads

read and write are not handled separately. fallback is already included in SideKick and is controlled via a documented arg.
In this iteration let's just split all requests independent of what they are. We can iterate on this down the line.

+1


Fixes problem with attribution classes from Gravity

This PR is to remedy an ongoing incident.
Recent changes to the artwork modelling were deployed today, causing a long series of events that led users to see the auction view with few or zero lots. Only artworks with no attribution class were visible.
tl;dr

We changed the JSON Mantle mapping for attributionClass to be attributionClass.name, since it's nested in Metaphysics.
However, Gravity models don't return that nesting. When Mantle calls valueForKeyPath with attribution_class.name on Gravity models, it has an NSString for the attribution_class key path, but string is not key value coding compliant for the name key, so this try/catch re-throws and is later silently ignored.

The solution was to extend the existing pattern for handling collisions. I had to prefix attribution_class with mp_ since the type are different.

CI passed here, but Circle's being weird about reporting it: https://circleci.com/gh/artsy/eigen/5611

Pushed a change that I have tested in GraphiQL, but not yet inside the app as I didn’t have the time to get my local env up-to-date. I’ve also ran make deploy again.

i.e. please test the change in the app before releasing!!

It’s weird how as soon as I saw your commit message, I knew exactly what was wrong 😅 I guess this is why we get people to review our sleepy code! 💤

Yup! 😄

Okay, found one more slight problem that's keeping attribution classes from being displayed on the artwork view. I don't think I want to give up our place in queue for it, but will PR a fix separately.


Problems about reproduced results

Based on your detailed code files, I try to reproduce the results about multiple treatmentments and dosages (TCGA) by directly running the commands you provided. I have some problems now and look forward to your help.

For baselines Drnet, Tarnet and VCnet,  there is a big difference between the result in Table 5 and my reproduced one:
This attached figure is the results on test dataset and validation dataset under Drnet:

As for as I know, the last indicator represents AMSE and in different simulation experiments, the results are similar.
So, I wonder are there some hyper-parameters should be tuned for these three baselines?

For your proposed TransTEE, I can obtain a similar result as you provided in Table 5 by running the command:
python test_TransTEE.py --num_treatments=3 --treatment_selection_bias=2.0 --dosage_selection_bias=2.0  --batch_size=128 --h_dim=64 --h_inv_eqv_dim=64 --num_dosage_samples=5 --alpha=1.0 --model_name="TransTEE"
But if I tune the hyper-parameters based on Table 8, the results are much worse.
Are the parameter values in Table 8 not suitable for TCGA dataset in this scenario?

Hi, you might want to try the following hyperparams
For Drnet, Tarnet and VCnet
python test_DRNet.py --num_treatments=3 --treatment_selection_bias=2.0 --dosage_selection_bias=2.0  --batch_size=512 --h_dim=48 --h_inv_eqv_dim=64 --num_dosage_samples=5 --alpha=1.0 --model_name=['drnet', 'tarnet', 'vcnet']
For TransTEE
Namespace(alpha=1.0, batch_size=128, beta=0.5, cov_dim=100, dosage_selection_bias=2.0, h_dim=48, h_inv_eqv_dim=64, initialiser='xavier', log_interval=10, lr=0.001, lr_scheduler='cosine', max_epochs=200, model_name='TransTEE_tr', num_dosage_samples=5, num_treatments=3, optimizer='adam', p=2, rep=1, save_dataset=True, test_fraction=0.2, treatment_selection_bias=2.0, validation_fraction=0.1, verbose=False, weight_decay=0.0)
We reran and got 0.222893 ATE for TransTEE and 2.055841 for TARNet.

Hi, you might want to try the following hyperparams
For Drnet, Tarnet and VCnet python test_DRNet.py --num_treatments=3 --treatment_selection_bias=2.0 --dosage_selection_bias=2.0 --batch_size=512 --h_dim=48 --h_inv_eqv_dim=64 --num_dosage_samples=5 --alpha=1.0 --model_name=['drnet', 'tarnet', 'vcnet']
For TransTEE Namespace(alpha=1.0, batch_size=128, beta=0.5, cov_dim=100, dosage_selection_bias=2.0, h_dim=48, h_inv_eqv_dim=64, initialiser='xavier', log_interval=10, lr=0.001, lr_scheduler='cosine', max_epochs=200, model_name='TransTEE_tr', num_dosage_samples=5, num_treatments=3, optimizer='adam', p=2, rep=1, save_dataset=True, test_fraction=0.2, treatment_selection_bias=2.0, validation_fraction=0.1, verbose=False, weight_decay=0.0)
We reran and got 0.222893 ATE for TransTEE and 2.055841 for TARNet.

I observed the result of ATE is heavily impacted by random initialization from 0.16xx to 0.5xx. Is the 0.222893 ATE you reported a single rep or mean of 30 times ?

Thanks for reproducing. The results you present seems normal and we report the mean.


EqualsBuilder.AppendMany won't accept arrays of interfaces.

The actual build error is:
The type 'IInterface' cannot be used as type parameter 'T' in the generic type or method 'Capgemini.CommonObjectUtils.EqualsBuilder.AppendMany(System.Collections.Generic.IEnumerable, System.Collections.Generic.IEnumerable)'. There is no implicit reference conversion from 'IInterface' to 'System.IEquatable'

AppendMany will accept arrays of interfaces that inherit IEquatable.

The fix for issue #4 lets you call AppendMany with arrays of interface type provided that you also provide a comparer object.


Chem: Current value -> Use as filter adds an empty filter

applyState() is called 2 times on filter (assume this occurrs in api.grok_FilterGroup_Add) - first is correct one, second contains previous value.

Now Current Value > Use As Filter adds a molecule to the filter in a different orientation (compared to the cell)


Use defaults for package and type if not provided

Problem
The Package (-p) and MessageType (-t) arguments are currently mandatory, are often very predictable, and passing them every time makes Proton more work to use.
Solution
Use defaults for the Package and MessageType if not provided.
Package: if there is a package in the Proton file (-f), use that one if none provided
MessageType: use the MessageType of the first message in the Proton file (-f), if the proton file has a message, and none is provided.

Dear @peterklijn , would it make sense to add the enhancement to the readme or to the dedicated usage session?

Dear @peterklijn , would it make sense to add the enhancement to the readme or to the dedicated usage session?

Ah, good one Rodrigo! Will do


CRDs

I noticed that the CRDs of this helm chart are placed in the templates folder instead of the Helm 3 special crds folder. I wonder if this is intentional and will there be any gotchas when upgrading the helm release?
This is more of a question than a change request - I'm not familiar with the technical details of Helm. But I used several Helm charts that whenever a new version has changes to the CRDs, requires me to manually update the CRDs. e.g. https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack#from-35x-to-36x

There's a valid and quite common reason for needing this. When using GitOps tools like Flux or ArgoCD it's not uncommon to separate CRDs so they are are loaded before any other resources when bootstrapping into a clean cluster to prevent race conditions:

You can see an example of this here, the CRD directory is loaded before anything else during a new cluster bootstrap.
If you won't/can't consider moving to the Helm3 CRD directory, please consider adding a chart value that disables the CRD templates, for example cert-manager or Kyverno

I'll definitely add an option to disable crds through values, however This chart is deploying the operator itself, without using the crds directly, so it should not trigger any race condition. But I agree it doesn't cost much to add a value for that.


Unable to build the Alpine docker image

Thank you very much for your project! Very excited to use it for my project!
I have some issues though during my initial build of an alpine docker image from docker directory
After about 10-20 minutes of installation and configuration of npm packages, I received a bunch of deprecation warnings, but fortunately it didn't block me to continue. When the docker build process was successfully finished, docker composed ran all 3 images (Postgres, redis and lemur itself) and at the first attempt lemur exited with code 2 because it was unable to connect to Postgres db. After I restarted the container it returned me another error which to be honest I have no idea what to do.
Please take a look on the error and explain, what did I do wrong? Did I miss something? I would really like to have a nice alpine image to run.
lemur_1     |  ?column?
lemur_1     | ----------
lemur_1     |         1
lemur_1     | (1 row)
lemur_1     |
lemur_1     |  # Create Postgres trgm extension
lemur_1     | NOTICE:  extension "pg_trgm" already exists, skipping
lemur_1     | CREATE EXTENSION
lemur_1     |  # Done
lemur_1     |  # Running init
lemur_1     | Traceback (most recent call last):
lemur_1     |   File "/usr/bin/lemur", line 33, in <module>
lemur_1     |     sys.exit(load_entry_point('lemur', 'console_scripts', 'lemur')())
lemur_1     |   File "/usr/bin/lemur", line 25, in importlib_load_entry_point
lemur_1     |     return next(matches).load()
lemur_1     |   File "/usr/lib/python3.6/site-packages/importlib_metadata/__init__.py", line 105, in load
lemur_1     |     module = import_module(match.group('module'))
lemur_1     |   File "/usr/lib/python3.6/importlib/__init__.py", line 126, in import_module
lemur_1     |     return _bootstrap._gcd_import(name[level:], package, level)
lemur_1     |   File "<frozen importlib._bootstrap>", line 994, in _gcd_import
lemur_1     |   File "<frozen importlib._bootstrap>", line 971, in _find_and_load
lemur_1     |   File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
lemur_1     |   File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
lemur_1     |   File "<frozen importlib._bootstrap_external>", line 678, in exec_module
lemur_1     |   File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
lemur_1     |   File "/opt/lemur/lemur/manage.py", line 19, in <module>
lemur_1     |     from lemur.dns_providers.cli import manager as dns_provider_manager
lemur_1     |   File "/opt/lemur/lemur/dns_providers/cli.py", line 6, in <module>
lemur_1     |     from lemur.plugins.lemur_acme.acme_handlers import AcmeDnsHandler
lemur_1     |   File "/opt/lemur/lemur/plugins/lemur_acme/acme_handlers.py", line 19, in <module>
lemur_1     |     import OpenSSL.crypto
lemur_1     |   File "/usr/lib/python3.6/site-packages/OpenSSL/__init__.py", line 8, in <module>
lemur_1     |     from OpenSSL import crypto, SSL
lemur_1     |   File "/usr/lib/python3.6/site-packages/OpenSSL/crypto.py", line 15, in <module>
lemur_1     |     from OpenSSL._util import (
lemur_1     |   File "/usr/lib/python3.6/site-packages/OpenSSL/_util.py", line 6, in <module>
lemur_1     |     from cryptography.hazmat.bindings.openssl.binding import Binding
lemur_1     |   File "/usr/lib/python3.6/site-packages/cryptography/hazmat/bindings/openssl/binding.py", line 16, in <module>
lemur_1     |     from cryptography.hazmat.bindings._openssl import ffi, lib
lemur_1     | ImportError: Error relocating /usr/lib/python3.6/site-packages/cryptography/hazmat/bindings/_openssl.abi3.so: DTLS_client_method: symbol not found
docker_lemur_1 exited with code 1

I'd recommend using the docker files available at: https://github.com/Netflix/lemur-docker
They are more recent than the one you ran.

I invested some time to get the build working with alpine, but I think one of the issues is with the openssl/libressl switch that they reverted, and some packages (like python3-dev did switch over to libressl, but cryptography when build using pip not)
It would probably make sense, to switch the dockerfile to Ubuntu similar to the one used in https://github.com/Netflix/lemur-docker/blob/master/web/Dockerfile

Thx @sergerdn for the fix. I guess, the issue is resolved now


Figure out "Page build failure" warning/error

The page build failed with the following error:
The submodule registered for ./lib/libxml2 could not be cloned. Make sure it's using https:// and that it's a public repo. For more information, see https://help.github.com/articles/page-build-failed-invalid-submodule.
For information on troubleshooting Jekyll see:
https://help.github.com/articles/troubleshooting-jekyll-builds
If you have any questions you can contact us by replying to this email.

NOTE: libxml2 submodule reference link starts with git:// not https://, also GitBook maybe used instead of Jekyll, not sure!

Github Pages are disabled for now, lets see what happens. Documents will be written later!


Release/Revision Change_Requests_Declined.md

This issue was created to collect comments about the upcoming metrics release.
This thread is for comments about Change_Requests_Declined.md
GitHub location: https://github.com/chaoss/wg-evolution/blob/master/metrics/Change_Requests_Declined.md
See all release candidates at: https://chaoss.community/metrics/

Name change revision - From "reviews to "change request"

Change Request LGTM!
Signed-Off-By<EMAIL_ADDRESS>

closing for release

This is a metric that should be reviewed. This may be considered a filter on the Change Requests Metric, and we may want to revisit the way all our Change Requests metrics relate to each other.


Mobile design

The genome browser is probably best viewed on desktop but some updates could improve the view on mobile. For instance it might be better to stack the labels and their tracks.

I noticed that -- at least in our implementation -- it was hard to scroll on a tablet (touch screen can't tell if you are moving the window or the genome browser frame).  That said, I am not sure it rendered at all on that device (Kindle Fire)


updating slack notification no brackets

@benbp @VulpesArtificem @rolandpoulter

Same question about indentation.

:+1:

:+1:


Accelerated Video Playback Enablement

Hi, I just discovered that I need to add pkgs.mesa.drivers to hardware.opengl.extraPackages in order for the radeon libva driver to be available.
Do we have a module for broad-libva enablement, or per-{cpu/gpu/apu} libva settings? Any thoughts about what that should look like? I have gfx-intel.nix and gfx-amd.nix that I'm working on that I could share as a starting point.

Actually, I'm just wrong, it's there by default but for some reason firefox-nightly just can't load it, I guess.


QnA Parity: Adding Active Learning Feature

Fixes #241

Porting new QnA Maker feature, Active Learning, that was just introduced to C#/JS repos in 4.5, into Python
Adds:
Models:

FeedbackRecord
GenerateAnswerRequestBody
Metadata
QnAMakerTraceInfo
QueryResult
QueryResults
TrainRequestBody

Utils:

ActiveLearningUtils
GenerateAnswerUtils
HttpRequestUtils
QnATelemetryConstants
TrainUtils

Thanks for the PR.


nuget package ?

Hello,
Your plugin is excellent, pretty clean ! Good work. :)
Do you have a date for the nuget package? Or is it better to extract the sources for now ?

Hello,
Thank you for the nice words.
I thought I should add some more polishing before publishing it as a nuget package.
Seeing there is some interest, I may squeeze in some time to get it done sooner.
For now you are free to build it from source.
Also, if you have any suggestions/bug reports, feel free to let me know.

ok. thanks :)


Feature Request - DVD mode - Add /raw support for ASUS/LG drives for both DVDs and Blu Rays using the READ CACHE (F1 06) command

Describe the solution you'd like
To go off my previous issue about raw dumping support, it turns out that ASUS/LG drives with RibShark's modified firmware can read the raw, scrambled 2384 byte sector data from a DVD by using the command to read from the cache. I don't believe this has been implemented in DIC? But this would be great to have. Unfortunately, these drives do not read Wii/GC discs with this method due to the invalid PFI data so drives have a hard time recognizing the discs. It also won't allow you to read security sectors either. But at least you can dump DVDISO no issue. Nothing seems to have this implemented so far.
As per RibShark's comment on an issue I opened with redumper:
https://github.com/superg/redumper/issues/40#issuecomment-1529148183
ASUS/LG drives support raw 2384 reads via the same F1 06 command as CDs (alternately 3C 03 02 XX XX XX YY YY YY 00, also works on CDs).
Blu Rays can also be read, but I'm having a hard time determining the size of an actual sector (with the ECC and header data).

it turns out that ASUS/LG drives with RibShark's modified firmware can read the raw, scrambled 2384 byte sector data from a DVD by using the command to read from the cache

No, my modified firmware is not necessary, it just needs to be below a specific version.

DiscImageCreator_test.zip

added: 2384 bytes/sector raw dump and unscramble

Doesn't seem to work?
H:\Tools\DiscImageCreator\test2>DiscImageCreator.exe dvd I conker 24 /raw
AppVersion
32 bit, AnsiBuild, 20230502T011731
valid extension was omitted. -> set .raw
CurrentDirectory
H:\Tools\DiscImageCreator\test2
WorkingPath
Argument: conker.raw
FullPath: H:\Tools\DiscImageCreator\test2\conker.raw
Drive: H:
Directory: \Tools\DiscImageCreator\test2
Filename: conker
Extension: .raw
StartTime: 2023-05-01T18:31:13-0400
Set the drive speed: 0KB/sec
DiskSize of [H:\Tools\DiscImageCreator\test2]
Total:<PHONE_NUMBER>224 bytes
Used:<PHONE_NUMBER>576 bytes
---------------------------
Space:<PHONE_NUMBER>48 bytes
=> There is enough disk space for dumping
Rawdump command [0]:0x3c [1]:0x02 [2]:0000
LBA[000000, 0000000]: [F:ReadDVDRaw][L:951]
Opcode: 0x3c
ScsiStatus: 0x02 = CHECK_CONDITION
SenseData Key-Asc-Ascq: 05-24-00 = ILLEGAL_REQUEST - INVALID FIELD IN CDB
EndTime: 2023-05-01T18:31:16-0400
Logs:
test2.zip

	            VendorId: ASUS
	           ProductId: BW-16D1HT
	ProductRevisionLevel: 3.10

Downgrade is needed.

My bad, I thought it would work with Rib's patched firmware. I downgraded my firmware using this:
http://wiki.redump.org/index.php?title=Flashing_with_Asus_BW-16D1HT_3.02_firmware
Tested with my Xbox disc, dumped the DVD Video partition after downgrading. All seems to work. :)
H:\Tools\DiscImageCreator\test2>DiscImageCreator.exe dvd I conker 24 /raw
AppVersion
32 bit, AnsiBuild, 20230502T011731
valid extension was omitted. -> set .raw
CurrentDirectory
H:\Tools\DiscImageCreator\test2
WorkingPath
Argument: conker.raw
FullPath: H:\Tools\DiscImageCreator\test2\conker.raw
Drive: H:
Directory: \Tools\DiscImageCreator\test2
Filename: conker
Extension: .raw
StartTime: 2023-05-01T21:02:30-0400
Set the drive speed: 0KB/sec
DiskSize of [H:\Tools\DiscImageCreator\test2]
Total:<PHONE_NUMBER>224 bytes
Used:<PHONE_NUMBER>464 bytes
---------------------------
Space:<PHONE_NUMBER>60 bytes
=> There is enough disk space for dumping
Rawdump command [0]:0x3c [1]:0x02 [2]:0000
Creating raw(LBA)    6832/   6992
Layer is changed: 20 -> 01
Creating raw(LBA)    6992/   6992
GOD/WOD unscrambler 0.4.1<EMAIL_ADDRESS>This program is distributed under GPL license,
see the LICENSE file for more info.
caching seed 0080
caching seed 0005
caching seed 0100
caching seed 000a
caching seed 0200
caching seed 0014
caching seed 0400
caching seed 0028
caching seed 0801
caching seed 0050
caching seed 1002
caching seed 00a0
caching seed 2004
caching seed 0140
caching seed 4008
caching seed 0280
caching seed 0014
caching seed 0400
caching seed 0028
caching seed 0801
caching seed 0050
caching seed 1002
caching seed 00a0
caching seed 2004
caching seed 0140
caching seed 4008
image successfully unscrambled.
time elapsed: 3.00 seconds.
ret = 0
Hashing: conker.iso
EndTime: 2023-05-01T21:02:55-0400
Raw scrambled contents, should include ECC/EDC too.

Feel free to merge unless you need more testing.


Can decrypt older DBs on macos, but not on Linux/Docker

These tests fail in CI/CD but pass on macos:
header "Read an old database (v0.5.0)"

cp ./tests/testing-data/secrets-v0.5.0.db "$DB_FILE"
./bin/rucksack show db-version \
    --config-file "$CFG_FILE" \
    --db "$DB_FILE" \
    --db-pass 1234

./bin/rucksack list \
    --config-file "$CFG_FILE" \
    --db "$DB_FILE" \
    --db-pass 1234

header "Read an old database (v0.6.0)"

cp ./tests/testing-data/secrets-v0.6.0.db "$DB_FILE"
./bin/rucksack show db-version \
    --config-file "$CFG_FILE" \
    --db "$DB_FILE" \
    --db-pass 1234

./bin/rucksack list \
    --config-file "$CFG_FILE" \
    --db "$DB_FILE" \
    --db-pass 1234

Error output on CI/CD:
>> Read an old database (v0.5.0)
Error: Could not decrypt data: aead::Error
2023-01-31 05:59:08 ERROR [src/store/db/encrypted.rs:83 rucksack::store::db::encrypted] ▶ Could not decrypt data: aead::Error
2023-01-31 05:59:08 ERROR [src/store/db/encrypted.rs:83 rucksack::store::db::encrypted] ▶ Could not decrypt data: aead::Error
Error: Could not decrypt data: aead::Error
>> Read an old database (v0.6.0)
2023-01-31 05:59:08 ERROR [src/store/db/encrypted.rs:83 rucksack::store::db::encrypted] ▶ Could not decrypt data: aead::Error
Error: Could not decrypt data: aead::Error
2023-01-31 05:59:08 ERROR [src/store/db/encrypted.rs:83 rucksack::store::db::encrypted] ▶ Could not decrypt data: aead::Error
Error: Could not decrypt data: aead::Error

Repo: https://github.com/RustCrypto/AEADs

Backlogging this for now. It seems like db files are not totally cross-platform safe. Probably best not to move/copy a database file to a different OS for now ...


Add permission boundary attachment logic

what
This commit adds a variable and logic to attach permission boundaries to permission sets.
why
This functionality is useful in the context of deploying permission sets.
references
fixed #37
Notes
Just in time (?) provisioning for the boundary to the target account doesn't seem to work so the boundary policy needs to exist in the target account prior to attempting to attach it.

attaching boundary to Permission set is done by policy name, it requires this policy to exist on the target AWS Account when assignment will be done. and module doesn't create anything in target accounts. this at least has to be written somewhere in a doc, if this PR will get merged

@aaronrea also, when you're done with your updates, please run some automation to pass our tests by doing the following locally, adding + committing the result, and pushing to your branch.
make init
make readme

Thanks!


Regex tag and zoom in for histogram summary does not work properly

Since ResNet could create hundreds of layers and the initial activation value of layers are also very large, I found I need to zoom in multiple times, and also have hundreds of histogram summary in one tag group. So it would be critical to be able to zoom, and also break summary down according to regex.
However, I found though for scalar summary, it works pretty well, for histogram summary, I can only zoom in once in 0.9.0, and not even once in 0.10.0. Also the regex group function seems to not work at all (see the photo below, it seems all summary comes in whatever the regex is --- I write a "###" in this case).
At first I though it is just there are too many graphs, so my computer stuck. But after I tried some small network, the phenomenon held. Could it be fixed? Thanks a lot.

Zoom doesn't work on those histograms. That's currently working as intended. In a future release, we will rename the histograms to "distribution charts" and add in true histogram charts which are much nicer.
For the performance issues: in the future, we hope to improve performance for large numbers of charts in a single group, but it's not on the immediate roadmap. We are actively working on ways to make it easier to set up the tag groups.
You're right that the regex filter doesn't work on the histogram tab. We'll dig into it and report back.

Thanks for reaching back. Performance perhaps won't be an issue when regex
is working since we could filter relevant out each time.
As for zooming, I remember it works in 0.5, at least. It is nice to have
that feature around, since sometime, the vibration in the network may cause
activation or parameters to reach a far larger value than the values when
the network stablizes. To be able to zoom in sometimes helps a lot,
otherwise, the stable values are indiscernable.
On 11 August 2016 at 01:26, Daniel W Mane<EMAIL_ADDRESS>wrote:

Zoom doesn't work on those histograms. That's currently working as
intended. In a future release, we will rename the histograms to
"distribution charts" and add in true histogram charts which are much
nicer.
For the performance issues: in the future, we hope to improve performance
for large numbers of charts in a single group, but it's not on the
immediate roadmap. We are actively working on ways to make it easier to set
up the tag groups.
You're right that the regex filter doesn't work on the histogram tab.
We'll dig into it and report back.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/3592#issuecomment-238939630,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUNSZ0BIXc5niYyFkIxl2ar39q7_xtvks5qegnhgaJpZM4JY_b9
.

Closing due to inactivity. Feel free to open a new github issue if the problem is still current.


Philly Glow

What is your question?
Question is there any way to change the lights to make it so certain colors are chosen and it's not just random every time?

No.


Mess with dates after calendar scrolling

Version Number: Using JTAppleCalendar (7.1.7)
I'm trying to show the current month with the app start.
I call  calendarView.scrollToDate(Date(), animateScroll: false) in viewDidLoad and it gives me wrong dates order.

If I'll click the previous month and then return to the current calendar shows the right ones.

do you have a regular calendar view inside of a UIViewController?
or do you have a more complex setup, with views stacked inside views?

is it possible you can provide a sample program demonstrating this?

Regular calendarView inside MyViewController's root view :)
conferenceroomcalendar.zip

Also, I noticed that function calendar 'willScrollToDateSegmentWith' doesn`t get called with 'calendarView.scrollToSegment' so I couldn't update the monthLabel.

ok it looks like this behavior is expected.
You have the scrolling mode set to .none. Therefore if youre scroll to today's date (the 5th), you will see the 5th cell is scrolled to the left side.
If you have your scrolling mode set paged scrolling, then it would not stop at the 5th. it will scroll to the next paged section. Whish should be the behavior u want.
So either set it to paged mode in storyBoard, is set it in the ViewDidLoad in code.
If you set it in code, here are the possible options you can use --> https://patchthecode.com/jtapplecalendar-home/scrolling-modes/

Thank you a lot!! I didn't notice this scrolling options. What about 'willScrollToDateSegmentWith' function. I see that it doesn't get called in pair with 'scrollToSegment'. How can I update the month label in the right way?

built this project for 2+ years, and you found a bug. haha.
i forgot to call that delegate-function within the segment one.
Will fix later today.

since this is fixed. Closing  this issue.
Ive created a new one to reflect the calling of the delegate function here
https://github.com/patchthecode/JTAppleCalendar/issues/1070


espidf 5.1、4.4 运行ethernet的iperf示例出现insufficient TX buffer size (IDFGH-12604)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

lan8720已经正确配置，我已经尝试过很多板子，包括淘宝发售的WT32-ETH01。
只要运行iperf -u -c SERVER_IP -t 60 -i 3就会出现错误 esp.emac: emac_esp32_transmit(232): insufficient TX buffer size。能力有限，尝试调节了很多内存、队列相关的参数仍然没能解决。很头疼

Closing the issue, since this is expected behavior.


Suggestion: Function calls without parenthesis

To me the visual cue to signify a function call is the parenthesis. I can't stand when they are left off. It would be really nice if there were an option for this.

Cool, thanks for the feedback. Give me a couple of days to see what I can do.

Ok, I have a plan for this rule, though it requires a small patch to CoffeeScript before getting enabled. If that patch doesn't land relatively soon, I'll come up with something else. Stay tuned.

Ok, my patch landed on master, so once the next version of CoffeeScript is released, this rule will be enabled.

Fear not. This rule will be disabled by default.

+1 from ME HERE I AM DAN.

I'd actually prefer a warning when the unnecessary parentheses are included. They should only be used when you have an argument list. An empty argument list should just be omitted.


help text fix

On the help page:
The text below doesn't explain the defaults that are assumed if you don't include a specific reference sequence. The examples assume that. Most people assume that.
Searching
To search just start typing a variant in the search box and it will auto complete. Variants are identified using HGVS nomenclature, a standard created by the Human Genome Variation Society. Most genetic test results are reported in HGVS nomenclature. This nomenclature describes a variant by indicating (1) a reference sequence, (2) what kind of sequence it is (genomic, cDNA or protein), (3) the position of the variant in relation this reference sequence, and (4) the nucloetide or protein differences between the variant and the reference sequence. For example, NM_007294.3:c.5053A>G indicates a variant relative to reference sequence NM_007294.3, which is a cDNA sequence (as indicated by c), that the variant is in position 5053, and that it involves changing an A to a G.

In this line:
Benign variants are not associated with any markedly increased risk of disease.
you need to use the agreed upon term "benign /little clinical significance"

Added an LShift Trello card: https://trello.com/c/IdE0GJpS/27-add-more-menu-button-to-the-menu-bar


Scanned and found a malware

Bro i was scanning my device with malwarebyte and found this "Trogan.Ransum.Jisut.AP"
I don't know much about coding and hope for an simple explanation.
Thank you.

It is a scam, of course. Source code is public, so look, there is nothing bad there.
Or you downloaded APK from unofficial source  (i.e. not Xposed repo)

It is a scam, of course. Source code is public, so look, there is nothing bad there.
Or you downloaded APK from unofficial source (i.e. not Xposed repo)

Ok bro if you say so, but "malwarebytes" is one of the trusted app and i downloaded the module from the xposed installer itself. You can check my mail with the screenshot i sent you.
I will not bother you again
Thank you
☺

Here is an answer to your question from Malwarebytes forums.
https://forums.malwarebytes.com/topic/213147-xinternalsd/


batch_jacobian incorrect

System information

Have I written custom code: yes
OS Platform and Distribution

Windows 10, Anaconda Python 3.7.6, tensorflow 2.1.0
Debian GNU/Linux 8.11 (jessie), Anaconda Python 3.7.3, tensorflow 2.0.0

Standalone code to reproduce the issue
import numpy, tensorflow
x = tensorflow.Variable([[1], [1]], dtype="float64")
with tensorflow.GradientTape(persistent=True) as t:
    with tensorflow.GradientTape() as tt:
        obj = x[0]**2 + x[1]**2 + x[0]*x[1]
    dx = tt.gradient(obj, x)
print(t.batch_jacobian(dx, x).numpy())

output is
[[[3.]]

 [[3.]]]

which is not [jacobian(y[i], x[i]) for i in range(x.shape[0])] as stated here.
It is more like [gradient(sum(y), x[i]) for i in range(x.shape[0])], same as gradient of gradient:
import numpy, tensorflow
x = tensorflow.Variable([1, 1], dtype="float64")
with tensorflow.GradientTape(persistent=True) as t:
    with tensorflow.GradientTape() as tt:
        obj = x[0]**2 + x[1]**2 + x[0]*x[1]
    dx = tt.gradient(obj, x)
print(t.jacobian(dx, x).numpy())
print(t.gradient(dx, x).numpy())

output is
[[2. 1.]
 [1. 2.]]
[3. 3.]

which is perfectly fine
Other info / logs

f(x,y) = x^2+y^2+x y
df/dx = 2x+y
df/dy = 2y+x
ddf/dxdx = 2
ddf/dxdy = 1
ddf/dydy = 2

What I want is the diag(Hessian) without calculating full Hessian.

Could able to reproduce the issue with Tf2.2rc2 and Tf2.1.
Please find the gist here. Thanks

As per documentation of batch_jacobian, the function requires the following:
"target: A tensor with rank 2 or higher and with shape [b, y1, ..., y_n]. target[i,...] should only depend on source[i,...]."
In your case, the dependence requirement is not met.

I ran into this same issue today and thought it was a bug, but then saw @agarwal-ashish 's comment of the requirement not met (which is stated in the batch_jacobian API) and indeed it was the case for me.
That said, I think the documentation in the "Advanced Autodiff" guide could be improved by mentioning this, because right now that guide says:

Caution: tf.GradientTape.batch_jacobian only verifies that the first dimension of the source and target match. It doesn't check that the gradients are actually independent. It's up to you to make sure you only use batch_jacobian where it makes sense. For example, adding a tf.keras.layers.BatchNormalization destroys the independence, since it normalizes across the batch dimension.

Hence, it seems from this warning that batch_jacobian doesn't care about y[i] being dependent on only x[i], and would still work as stated.
This is just a soft suggestion though, maybe it was just me who misunderstood.


convert useStates in a single object state

Wanted to work on this

@Aadarsh805 Like this?

I should have explained this issue in detail, this one actually didn't need this
I was referring to those states which can be used as one like the username input and the question input
this is more of a code cleanup task, I made it for myself so that others could understand the code faster because there were too many states and all
Leave this issue to me, we'll do this later

ok @Aadarsh805


Fix script vars being dumb

This PR attempts to fix issues with script-var scripts not being closed correctly.

@buffet could you maybe look over this again? I'm really unsure if i understand all of this, and i'd want to make sure that it's now done and no longer an issue ^^

Remind me tomorrow.

@buffet there ya go, reminder here!

Thanks a ton, @buffet!


simple event addition : end of score EVENT

An "end of score" callback, useful for adapting the UI elements.
PlaybackEngine.ts
export enum PlaybackEvent {
  STATE_CHANGE = "state-change",
  ITERATION = "iteration",
  END_OF_SCORE = "end-of-score", // new event
}

// callback
  private endOfScoreCallback(message: string) {
    this.events.emit(PlaybackEvent.END_OF_SCORE, message);
  }

 // check for "end of score" and firing cb
private iterationCallback() {
    if (this.state !== PlaybackState.PLAYING) return;
    if (this.currentIterationStep > 0) this.cursor.next();
    ++this.currentIterationStep;
// Checking here
if (this.currentIterationStep > this.iterationSteps) {
      this.endOfScoreCallback("End of score");
    }
}

Hi!
Thanks, yes this could definitely useful. Feel free to open a PR with the change or I can integrate it myself soon.


PyPlot and ImageView

Hello, I don't know if it's a known bug, but I could reproduce it on three linux mint 17.2 and julia 0.3.11.
using PyPlot
matshow(rand(10, 10))  //Everything is good
using ImageView
matshow(rand(10, 10))  //Will crash with following message:

The crash error message is (it kills julia):
Fatal Python error: PyEval_RestoreThread: NULL tstate

signal (6): Abandon
gsignal at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
abort at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
unknown function (ip: -1235177474)
PyEval_RestoreThread at /usr/lib/x86_64-linux-gnu/libpython2.7.so (unknown line)
unknown function (ip: -1281970477)
TclInvokeStringCommand at /usr/lib/x86_64-linux-gnu/libtcl8.6.so (unknown line)
TclNRRunCallbacks at /usr/lib/x86_64-linux-gnu/libtcl8.6.so (unknown line)
unknown function (ip: -1095393792)
TclServiceIdle at /usr/lib/x86_64-linux-gnu/libtcl8.6.so (unknown line)
Tcl_DoOneEvent at /usr/lib/x86_64-linux-gnu/libtcl8.6.so (unknown line)
tcl_doevent at /home/light/.julia/v0.3/Tk/src/tkwidget.jl:20
tcl_doevent at /home/light/.julia/v0.3/Tk/src/tkwidget.jl:20
jlcall_tcl_doevent_21657 at  (unknown line)
jl_apply_generic at /usr/bin/../lib/x86_64-linux-gnu/julia/libjulia.so (unknown line)
anonymous at task.jl:73
jl_handle_stack_switch at /usr/bin/../lib/x86_64-linux-gnu/julia/libjulia.so (unknown line)
julia_trampoline at /usr/bin/../lib/x86_64-linux-gnu/julia/libjulia.so (unknown line)
unknown function (ip: 4199613)
__libc_start_main at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
unknown function (ip: 4199667)
unknown function (ip: 0)
[1]    14168 abort      julia

And just the fact to include ImageView is enough !
Do you have an idea ?

It's probably a Tk-related bug. I'm partway through a rewrite of ImageView based on Gtk, but it's not ready yet.

I've also been hit by this bug (but I can live with it). Any progress on the work with Gtk?

I'm completely overwhelmed with https://github.com/timholy/Images.jl/issues/542, which is a predecessor to the ImageView rewrite. Significant progress announced yesterday: https://github.com/JuliaImages/ImagesFiltering.jl/pull/5.

That's fine. As I said, I can live with this problem, no rush ;-)

Does not crash for me with current ImageView.


Add release strategy to community documentation

This adds the release strategy as discussed in the core team.
Please feel free to make further comments and suggestions before we fully commit to this strategy by merging it into master!
@nielsdrost, could you please let me know when you are done with the figure or just add it to this PR?
NB: If adopted, the next feature freeze is scheduled for 2020-10-05 and the next release will be overseen by @valeriupredoi, so I propose to merge this within the next two weeks if possible.

I added the people that were in the video call or had expressed interest in it as well as the PIs as reviewers.
However, in the spirit of an open community, everyone is welcome to comment.

Thanks @zklaus! @ESMValGroup/esmvaltool-developmentteam Any comments?

@nielsdrost, could you please let me know when you are done with the figure or just add it to this PR?
I added the figure to this branch. It's a diagrams.net (formally draw.io) image, with the vector image embedded in the png (holy backward metadata batman!)

First of all, nice work! I particularly like the definition of the terms in the glossary. Two comments on the current version:
(1) I think since this document will be part of the user's guide, the wording should be a bit more "determined". E.g. instead of "The goal of this document is to lay out [...]" simply "This document lays out [...]" or instead of "We propose to follow a strategy of timed releases." simply "We aim at following a strategy of timed releases.".
(2) I am still a bit worried that such frequent releases might be to much. Too much in terms of additional work and possibly also confusing for scientist users as there are new versions of the tool "all the time". But I am fine trying to follow the proposed release schedule and see how it goes.

First of all, nice work! I particularly like the definition of the terms in the glossary. Two comments on the current version:
(1) I think since this document will be part of the user's guide, the wording should be a bit more "determined". E.g. instead of "The goal of this document is to lay out [...]" simply "This document lays out [...]" or instead of "We propose to follow a strategy of timed releases." simply "We aim at following a strategy of timed releases.".

I second that comment, and added some text suggestions to "implement" this.

Thanks @axel-lauer for the suggestions and @nielsdrost for the figure and implementing the suggestion by Axel. I commited them.


Webauthn Login Failure

I used ashirt-deployments to bring up the tip of main sha-1bf765e
When registering a new user with only webauthn the process seems to complete, but initial login fails with a 403 to the client. The username in this case was lower-alpha only. The uses appear in the database/admin settings, but the user cannot login. If I impersonate the user with a recovery link and re-add the same key the user can login.
timestamp=2023-02-24T08:33:28.603963926Z ctx=fb404bcf-e62a-45c7-9918-06a3fd8e4ab9 msg="Error handling request" error="userHandle and User ID do not match" rootError="userHandle and User ID do not match" status=403 url=/auth/webauthn/login/finish

Fixed by #800


Release Delphi Epidata {next_tag}

Releasing Delphi Epidata {next_tag}.

deleting this to get versioning back in sync


Remove config file for ci.bazel.io

ci.bazel.io is not in use anymore, Bazel CI has switched to buildkite.
@greggdonovan

why is ci.bazel.io still running?

any thoughts @philwo

@philwo is on vacation, @fweikert  do you know how github presubmit is setup?

@fweikert is there a way to remove ci.bazel.io for presubmit?

@fweikert is there a way to remove ci.bazel.io for presubmit?

buchgr@ and I had a brief look at the project settings (I don't have the necessary permissions), but we did not find out how the integration was configured.

buchgr@ and I had a brief look at the project settings (I don't have the necessary permissions), but we did not find out how the integration was configured.

Fixed :D It was an old branch protection rule in the Settings that said that the "ci.bazel.io" status check must pass before it can be merged.
I removed it from there and picked the Buildkite check instead.

@philwo Thank you!!!

@petroseskinder Please merge this PR


event ID 61f71b770cab4a47b75349e2f0f177ae

EncodingError: Failed to execute 'decodeAudioData' on 'BaseAudioContext': Unable to decode audio data Error: Failed to execute 'decodeAudioData' on
at K (https://railannouncements.co.uk/9f38c194-39fbb7e3538bbd963356.js:1:2529) at u (https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:1527) at m (https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:4325) at div at https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:15306 at div at I (https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:8129) at https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:15035 at div at o (https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:9891) at I (https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:8129) at https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:10212 at https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:13048 at https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:15921 at div at https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:13048 at div at ne (https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:18386) at main at https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:13048 at main at https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:13048 at t (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:194756) at Qn (https://railannouncements.co.uk/76d8c668-26bc05d90cb7623e3f0b.js:1:38431) at RecoilRoot (https://railannouncements.co.uk/76d8c668-26bc05d90cb7623e3f0b.js:1:40398) at t (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:290224) at p (https://railannouncements.co.uk/689577f30886a7ec6d84f438f5451ec0a24ddfa1-ef846d46cd58fd54c58c.js:1:2977) at be (https://railannouncements.co.uk/beaa6cd174c4c823946845c5593ee41b3efafd57-37ec705af821455fa5f5.js:1:20931) at o (https://railannouncements.co.uk/component---src-pages-stations-amey-phil-sayer-tsx-df6ae0fdea26f7a861d0.js:1:488) at A (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:317613) at e at div at se (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:25302) at oe (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:25162) at ue (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:26036) at ce (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:25956) at t (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:290224) at t (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:321075) at o (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:324125) at t (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:200490) at t (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:321723) at https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:24050 at q (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:23261) at z (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:23530) at WithErrorBoundary() at ee (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:24644) at n (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:323606) at i (https://railannouncements.co.uk/app-f8f830f9ab44300b0e13.js:1:325807)

I think you caught GitHub at a moment where it was down, causing an audio file to fail to load correctly.
I can't find your exact event ID, but some others around the same time as your ticket look to be related to this too.
If you can still reproduce this issue, please share a link to the announcement which breaks, and make sure you try it again without any ad blocking or privacy protection tools active (or I won't get the error report).


report_tests_log_factory error: EXCEPTION: undefined local variable or method `keys'

I'm getting the following error related to report_tests_log_factory for the prerelease
🚧 Loaded project configuration at default location using ./project.yml
🪲 ERROR: Exception raised while trying to load plugin: report_tests_log_factory

🧨 EXCEPTION: undefined local variable or method `keys' for #<JsonTestsReporter:0x0000556af1166d98 @handle=:json, @filename="json.report", @config=nil, @config_walkinator=#<ConfigWalkinator:0x0000556af087dad8>>

Disabling the report_tests_log_factory plugin works as a temporary fix. It also affects junit and html.
I think the issue is here: https://github.com/ThrowTheSwitch/Ceedling/commit/4d75a3e80c7eda467b29b996942c74f0971cdf4b#diff-433207d3ec0d0e1561d8e3af1fca049b4aca2d942ea1a30735baa7c514d443c9

Thank you, @tyler-macinnis. Yes. That's an easy fix. Will get that bug squashed in a new prerelease soon.

@tyler-macinnis, this should be fixed in the latest prerelease.


R&D: Get error code info

Match error codes to solutions, add to seed spreadsheet.

@lauragonzalezzz  during communication with chatbot I am getting "default_choice" & "default_error" in response, though I didn't do any code or response texts with this strings, so can you tell me why I am getting this codes?

Hi @piyush-dholariya, I think you meant to post this comment to the Microsoft Bot Framework repo, this is a small project my team created which utilizes the framework. We cannot provide answers or support for the framework. Thanks!

@lauragonzalezzz  ohh I am sorry.


"Please copy this code, switch to your application and paste it there"

I am trying to use the quickstart.js for nodejs api but I am getting to the following screen and have no idea how to proceed further. There is a key below that copy on the screenshot, but I don't quite understand where should I add it. I am using OAuth2 to authenticate the user. When I run the quickstart script, I am getting to choose my user in the opened gmail tab, but after I authorise it I see the following key. The project I am trying to access is within our organization account. Please advise how to proceed further.
Meta
OS: macOS High Sierra 10.13.4
Chrome: 66.0.3359.139
Reference

Good afternoon ) So that was the first thing I was trying to do. I managed to make it work, but the original scripts here(https://github.com/google/google-api-nodejs-client/blob/master/samples/sheets/) did not work for me, so I had to dig into API and figure things out. It's working for me now though so the issue can be closed. I needed to change the auth option to service-to-service schema.


fix typo

multi-mode -> multi-modal

I had just added the @BUPT/chairs as the maintainer of this repository, so please feel free to help yourself to merge this PR! :smile:


Budget Alert not considering Bandwidth

I set an alert for 50 dollars in a resource group and its currently at 59.99 according to the budget alert it has only spent a little over 5 dollars; most of the cost is being generated by the bandwidth of a virtual machine,  which as of now is of 52 dollars. according to the cost Analysis tool. I should have received an alert by now, but because of budget alert is ignoring the bandwidth this has not happened yet

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

ID: 7349e660-9bb8-9676-6e48-17796a97b674
Version Independent ID: 3b80ab4a-f9c2-0b81-aa7b-99fff673d813
Content: Quickstart - Explore Azure costs with cost analysis
Content Source: articles/cost-management-billing/costs/quick-acm-cost-analysis.md
Service: cost-management-billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@burstingninja Can you clarify a bit so that I can help? What did you set the budget amount to? What value did you set the alert condition to? It sounds like 50. It sounds like your evaluated spend shown in budgets is $52.00. What's the current progress % for the budget?

At Cost Management > Budget:
I set a budget of 50 dollars.
The condition was to send me an email when it reached 90%.
According to the budget, my current evaluated spend is of $5.45 with a progress of 10.91%
The progress formula is correct; but the evaluated spend is wrong.
*From my understanding the evaluated spend is wrong compared to the information provided by cost Analysis.
When I go to Cost Analysis I get the following information:
I get that the Actual Cost is of  $59.99.
In my resource group I had a virtual machine and from those $59.99; $52.50 of them were generated by the bandwidth of this virtual machine.
I suspect the cost of the bandwidth is not being considered by the Budget calculation for the evaluated spend.

@burstingninja Investigating with the enginnering team. In the meantime, Bandwidth is a reported cost from meter category so it’s costs are shown in Cost Management. So, bandwidth costs (if used by resources in the budgeted resource group) should be monitored by the budget. I'll get back to you after team investigation.

@bandersmsft
Here I can provide additional screen captures:
From budgets

From Cost Analysis without applying the budget to the view:

From Cost Analysis, applying the budget to the view:

with this information I cannot explain which value is being taken as evaluated spent of $5.45.
Thanks in advance

@burstingninja Okay, we'll looking at your screenshots.
Budget evaluation runs every 20 hours. When you created the budget, under Budget Scoping, did you click Add filter > ResourceGroupName and select the resource group that contains your VM?

yes, the Budget Scoping is at ResourceGroupName level

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

@burstingninja Is this still an issue for you? It looks like cost analysis is getting updated faster than the budget evaluation run. Consequently, you're seeing a delayed budget evaluated spend.

I decided to delete the resources, so its not a problem related to costs anymore. On the other hand, is it possible to add to the documentation a note which mentions how often the budget alert runs (according to one of your messages, every 20 hours) and also ask for a feature to run it more often or adjust how often it run (with the purpose that it prevent unexpected charges) with the next run timestamp?
Thank you! @bandersmsft

@burstingninja Budget eval and notification timeline is documented at https://docs.microsoft.com/azure/cost-management-billing/costs/tutorial-acm-create-budgets. I'll pass along your request to the engineering team.
#please-close


Could not load the Qt platform plugin

Expected behaviour
Application with matplotlib should be able to show a plot window.
Actual behaviour
Segmentation fault during the plotting process.
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "/home/sadams/projects/meccanoFork/venv/lib/python3.8/site-packages/cv2/qt/plugins" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
Steps to reproduce

example code
`import cv2
import matplotlib.pyplot as plt
import matplotlib
import numpy as np

print(f'Build information: {cv2.getBuildInformation()}')
print(f'CV2 Version: {cv2.version}')
print(f'Matplotlib Version: {matplotlib.version}')
x = np.linspace(0,1,10)
plt.figure()
plt.plot(x)
plt.show()
`
See attached for build information.
Version control:               4.4.0-dirty
qt_debug.txt
Issue submission checklist

[ X] This is not a generic OpenCV usage question (looking for help for coding, other usage questions, homework etc.)

[ X] I have read the README of this repository and understand that this repository provides only an automated build toolchain for OpenCV Python packages (there is no actual OpenCV code here)

[ IDK] The issue is related to the build scripts in this repository, to the pre-built binaries or is a feature request (such as "please enable this additional dependency")

[ X] I'm using the latest version of opencv-python

I realize that this is similar to IMShow No Window - <IP_ADDRESS> MacOS Big Sur #423;
however, this shows that it is not just Mac related.

Duplicate of https://github.com/skvark/opencv-python/issues/386 and https://github.com/skvark/opencv-python/issues/404.


Move to proper Github organization / rename project

https://github.com/ReactNativeDesktop could be irrelevant since it's not about desktop in common sense now (or yet).

Might want to hold off for a bit, given that Facebook has announced the equivalent of React Native for Windows. It won't surprise me to see a version for Mac and Linux soon.

moved to react-native-macos


Allow destructors to be run directly on a memory location, so it is not impossible to handle location-dependent values

 Issue by huonw
Monday Aug 04, 2014 at 12:52 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/16242
This issue was labelled with:  in the Rust repository

Currently, the only way for a library to run a destructor on a value is to let it fall out of scope while on the stack, that is, types wrapping raw pointers (like Vec and Rc) have to call ptr::read and let the resulting value drop to destroy their contents. This moves values in memory, and thus makes it literally impossible to use Vec or Rc (or any other library defined container type) for types with destructors that depend on the location of the value in memory.
This could be avoid we had a destroy_directly(&mut T) intrinsic that destroyed that memory location, and so what is currently written ptr::read(x) (or drop(ptr::read(x))) would be written destroy_directly(&mut *x) and (for types with Drop impls) effectively be just calling (&mut *x).drop().
e.g. https://github.com/rust-lang/rust/blob/795f6ae829ab1bfd72394a5da9096e2717ec0f62/src/libcollections/vec.rs#L1539 (there's a variety of other ptr::reads for a similar purpose in this file, and in other low-level code).
As a real-world example, putting a series of std::comm::Handle into a preallocated Vec (to satisfy the requirement that they must not move in memory after calling .add) will still break, because the destructors asserts that the memory address is the same as it was and thus Vec moving things in memory upsets it. The value being location dependent is bad, but it seems very reasonable to just destroy things in place to allow it to mostly work (it's probably more efficient too, since I believe LLVM can't optimise away the memcpy out of the Vec in all situations).
(NB. one can currently work-around this by box-ing everything, because the compiler-magic of the Box type means its destructor destroys things in-place.)

Shouldn't "destroy_directly" take a raw pointer rather than a reference? A reference implies the value is being borrowed, which isn't the case here.

I originally gave it that signature to match drop, however I do think the semantics we want here are reasonably close to &mut (wrt aliasing and uniqueness).


Python version of osvr_json_to_c

I think we still want to use the C++ version by default, since it keeps dependencies low, but it makes cross-building for Android easier (one step rather than two) to have an equivalent. It's not complex to write, and is blocking the android merge.

If it's blocking the Android merge, let's do it.

Hi !
I'm pretty competent with Python, looking to contribute, and that .cpp file doesn't look very complicated
If this is still needed i'd be glad to write it.
I would need an example of expected input and output, and that's about it.

Ello,
If there's any menial Python tasks that need to be addressed I'm willing to help as well. I'm versed in C and C++ but I've used Python the most. So you need a python to c conversion or json to python?

Thanks for your interest! So this osvr_json_to_c tool works a bit like xxd if you've ever used that. Right now, using a locally-built tool makes cross-compiling (like for Android) harder, hence why a Python equivalent would be handy.
Here's what the tool does. You give it a JSON file, a variable name, and an output file. It reads in the JSON file (discarding comments - might need to use a more liberal json parser in Python since apparently the standard one is a bit pickier than jsoncpp), turns it back into a string in "compressed" format - that is, without unneeded whitespaces, etc. It then writes out a file that basically is a C/C++ source file defining a string array by converting every character to hex.
Sample input and output pair:

in: https://github.com/OSVR/OSVR-Core/blob/master/src/osvr/Client/display-HDK.json
out: https://gist.github.com/rpavlik/e392afc6770a9e046e4f

Still a welcome addition, but no longer blocking Android - the OSVR-Android-Build repo builds just osvr_json_to_c separately on the host.

Hi. This issue looks like it's a year old, but do still want help with this?  If so, can you provide an updated link to an example input file?  The link from the comment posted on April 1, 2015 (https://github.com/OSVR/OSVR-Core/blob/master/src/osvr/Client/display-HDK.json) seems to be no longer valid.
I'd like to take a shot a porting https://github.com/OSVR/OSVR-Core/blob/master/devtools/osvr_json_to_c.cpp to Python if you still need it done.

@malcolmathci  Any JSON file would serve as input. You could use https://github.com/OSVR/OSVR-Core/blob/master/apps/displays/OSVR_HDK_2_0.json as an example.
It reads the JSON file and converts it to a const char[]. For example, the following JSON file:
{
   "hello": "world"
}

would get converted to something like this:
static const char my_json[] = {
0x7b,0x22,0x68,0x65,0x6c,0x6c,0x6f,0x22,0x3a,0x22,0x77,0x6f,0x72,0x6c,0x64,0x22,0x7d,0x0a};

It just converts each character of the JSON input into its ASCII value and dumps it into an array.

@godbyk Ah, okay.  I got it.  Thanks for clearing that up for me.

@godbyk @rpavlik Can I have some feedback on my solution?  Is there anything that should be changed?  The output of this script matches the "hello world" example, except the EndOfFile newline character is missing the leading 0.  The script returns "0xa" instead of "0x0a", which may be the same for all single-digit hex values.  I'd have to test some more to be sure.  I'm not sure if that will be a problem on the C/C++ end.: https://github.com/malcolmathci/OSVR-Core/commit/912531da52ac52407d9a6dae1365cc84bbede93f

@godbyk @rpavlik I have a pull request at https://github.com/OSVR/OSVR-Core/pull/483, but I'd like to add a unit test for the following before you review it:
{    "hello": "world" }

@godbyk @rpavlik I added the unit tests, so my solution is ready for review now.  My branch fails the continuous integration check, and I'm looking into why.  I welcome any feedback you have.


Create no.json

Thanks, but there is another language file in frontend/localize/languages.. Can you translate that as well?

awesome thank you :)


Implemented logic for function

Description of your changes

Fixes #
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Added or updated unit tests for my change.

@art10m Please resolve lint and unit-test failings.


AvroConverter should support namespaces

AvroConverter does not pay attention to Avro namespaces right now.

Closed by #307