feature request: Add a setting to embed .cab file into the .msi file

I would say that should be the default setting and Wax should make it easy.

As you may have noticed Wax currently only manages files, not properties.
To have defaults is so easy with WiX - just build a template with your preferences and include it - so I wonder if it's worth building a UI for this, since there are only very few properties for which this would  make sense.
Any suggestions how this could be handled?

@flyingxu If you think this should be the default setting for WiX, you should post a feature request here: https://github.com/wixtoolset/issues, not give a bad rating for a tool with a totally different scope.

This feature is not really in the scope of Wax.


When adding a grant manually to a submission. that grant is not getting attached to the submission

Hence it doesn't shows up in various places of the workflow and in the submission details page.

This has been fixed in the editable submissions PR


How to update a value from my collection using 2 unique identifiers?

I want to query the collection with 2 unique identifiers. How do I query the collection using 2 unique identifiers and modify the value of one of the fields?
Sorry, I might not be understanding how MeteorCollection works.

The short answer is, you'll need to implement this yourself by subclassing or extending MeteorCollection.
The longer answer is:  MeteorJS is tied to MongoDB on both the client and server. There's nothing approximating MongoDB in iOS, and for serious applications you'll really want to use Core Data, Realm, SQLite or perhaps Couchbase Mobile as your iOS datastore. In this more advanced scenario, SwiftDDP would handle the communication - method calls, pubs and subs - between your client and your mobile datastore. You can see examples of this in the Todos examples.
MeteorCollection is intended to provide scaffolding to create more powerful collections backed by other datastores by providing a built-in object that responds to server changes. While it has a few convenience methods, it's not a full-fledged client datastore like minimongo.
What you're trying to do is not hard. Just create a method that does it via extension or subclassing.


How to get "CD AB" order from ByteBuf?

How can I get a "CD AB" ordered float from ByteBuf? Like:

I know very little about modbus. Can you please help me? Thanks.

I'm not sure what that software would consider the order to be by default (maybe DC BA), but try something like this:
short cd = Short.reverseBytes(buffer.readShort());
short ab = Short.reverseBytes(buffer.readShort());
float f = Float.intBitsToFloat(cd << 16 | ab);

Assuming the buffer contains at least 4 bytes (meaning you read at least 2 registers, because each register is 16-bit).

@kevinherron Thank you for your reply. I'm trying using your code, but I got the value equal to order BA DC rather than DC BA. Is it because of different default order?

Maybe this then?
short ab = buffer.readShort();
short cd = buffer.readShort();
float f = Float.intBitsToFloat((cd << 16 | ab));

It works! Thanks!


Make sign up page

Do all work for this task in a branch called issue-08

allows user to enter their info to create a profile
go back to sign in page if they already have an account

This page is already created from the template. Unless we plan to customize the background/layout of the sign up page we can mark this complete.


Polymer template repeat on table row is extremely slow

Compiled to JavaScript, Observatory's profiler tree expansion is extremely slow.
I have a list of precomputed table cell strings and do the following in my HTML:

  {{row.columns[0]}} ...
  {{row.columns[1]}} ...

It takes 50 seconds to update the display when inserting ~1,000 rows.
Chrome 27.

This issue has been moved to dart-lang/polymer-dart#196.


Add USB_F_MIDI Kernel Module

Hi, I've been playing with your work on the rg35xx-plus, but mostly for LittleGPTracker.
One question I have is would it be possible to build the USB MIDI kernel module using your script? I had a shot but my knowledge of the kernel build system isn't good enough to figure it out.

Hi, I've been playing with your work on the rg35xx-plus, but mostly for LittleGPTracker.
One question I have is would it be possible to build the USB MIDI kernel module using your script? I had a shot but my knowledge of the kernel build system isn't good enough to figure it out.

Yes, it is possible to build the USB MIDI kernel module, but it requires some adjustments to the kernel configuration and build process. You would need to enable the necessary kernel modules and rebuild the kernel to include USB MIDI support.
Here's a general approach you can take to achieve this:
Steps to Enable USB MIDI in the Kernel:

Modify the Kernel Configuration:
You'll need to make sure the required kernel modules for USB MIDI are enabled in the kernel config.
In your case, since you're working with the linux-sunxi64-legacy.config, you need to enable the USB MIDI module in that configuration file or during the kernel build process.
Add or modify these lines in your kernel configuration (linux-sunxi64-legacy.config):
CONFIG_SND=y
CONFIG_SND_RAWMIDI=y
CONFIG_SND_SEQ_DEVICE=y
CONFIG_SND_USB=y
CONFIG_SND_USB_AUDIO=m
CONFIG_SND_USB_MIDI=m

If these options are set to m, the modules will be built as loadable kernel modules, which is likely what you want for USB MIDI.

Prepare for Kernel Build:
In your build_script.sh, after downloading the kernel and preparing the environment, you can include steps to ensure that the configuration is applied:
# Apply the custom config for enabling USB MIDI
cp /build/linux-sunxi64-legacy.config /build/linux-4.9.170/.config
cd /build/linux-4.9.170

# Update configuration to enable USB MIDI
make ARCH=arm64 oldconfig

Build the Kernel and Modules:
Once the configuration is updated, you can build the kernel and the specific USB MIDI modules:
# Build the kernel and the modules
make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- -j$(nproc) zImage modules

# Install the modules
make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- modules_install INSTALL_MOD_PATH=/path/to/your/output

Load the USB MIDI Module:
After the build is complete and the kernel is flashed onto your device, you can load the snd-usb-audio module to activate USB MIDI support:
modprobe snd-usb-audio

Or, if you've built the module as part of the kernel image, it should be available automatically once the device detects a USB MIDI device.

Update Your Build Script:
You may need to extend the build_script.sh to handle the kernel configuration and the module installation process. You could also integrate the steps above into your Dockerfile if you want to automate the entire process.
If you'd like more specific guidance or if you have a more detailed build script, feel free to share it and I can help tailor the instructions!

@simotek Any progress on this? I too, am not knowledgable on the build process TBH!


(maint) fix broken link on setup page

Description Of Changes
The link to Download + Powershell Method has been changed. I changed the broken link into the correct link
Motivation and Context
Broken links are useless.
Testing

[ ] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[x] Minor documentation fix (typos etc.).
[ ] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.
[ ] The change I have made should have a video added, and I have raised an issue for this.

Issue #

Change Checklist

[ ] Requires a change to menu structure (top or left hand side)/
[ ] Menu structure has been updated

Related Issue
None

@mkevenaar awesome work! Thanks for your contribution!


"Stage changes" command doesn't work on Merge Changes

Have a merge conflict in a git repo, file is in the "Merge Changes" section of the scm viewlet
Hover the + button on the file in "Merge Changes", it says "Stage changes cmd+shift+S" with my keybinding
Press that shortcut, or use the command via command palette - nothing happens
It works as expected when the file is not staged

@joaomoreno I can work on this one if possible.

I believe this is directly related to #24919, which has a pull-request pending.  I wasn't able to receive the tooltip keybinding as @roblourens explained.  So that may be an issue, I don't know at this point.
Something to be mindful about, this ticket was created a day after the 'extensions/merge-conflicts' was added to master.  I don't think that should of changed the lack of behavior as @roblourens mentioned but again, just be mindful about it.

@roblourens are you choosing a file and then press the shortcut? If you don't do that then nothing will happen. Stage changes only looks for the files that are selected then adds it to the stage. Please let me know if that works for you.

same error here! its there a workaround?

To keep the number of issues in our inbox on a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: We look at the number of votes the issue has received and the number of duplicates issues filed.
Thanks for your understanding. Happy coding!


Port ma fto gvcf updates

Commented out lines removed.
From: Zack Miller @.>
Date: Monday, November 6, 2023 at 8:22 AM
To: maize-genetics/BioKotlin @.>
Cc: Lynn Carol Johnson @.>, Assign @.>
Subject: Re: [maize-genetics/BioKotlin] Port ma fto gvcf updates (PR #13)
@zrm22 commented on this pull request.

In src/main/kotlin/biokotlin/genome/MAFToGVCF.kthttps://github.com/maize-genetics/BioKotlin/pull/13#discussion_r1383324170:

                           varinfo.chr,

                           Pair(previousInfo.endPos + 1, varinfo.startPos - 1),

                           if (sameChr) varinfo.asmChrom else "",

                           asmPositions,

                           ""

                       )

                   )

               }

               else {

                   val chrSeq = refGenomeSequence!![varinfo.chr]

                   filledVariantList.add(

                       buildMissingRegion(varinfo.chr,

                           previousInfo.endPos+1,

                           varinfo.startPos-1,

                           chrSeq!![previousInfo.endPos, previousInfo.endPos].toString(), // -1,  NucSeq is 0-based

                           //refGenomeSequence.genotypeAsString(varinfo.chr, previousInfo.endPos+1,previousInfo.endPos+1),

Is this commented out part still valid? Should we remove it?
—
Reply to this email directly, view it on GitHubhttps://github.com/maize-genetics/BioKotlin/pull/13#pullrequestreview-1715163433, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB7P3VC6TKAJBDNHP3TZ2ZLYDDQBDAVCNFSM6AAAAAA67LS7L6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMJVGE3DGNBTGM.
You are receiving this because you were assigned.Message ID: @.***>


Coveralls and Ruby 2.4

coveralls 0.8.19 depends on simplecov ~> 0.12.0.
simplecov 0.12.0 raises deprecation warnings when running under Ruby 2.4. This was fixed today with the new 0.13.0 release.
I would appreciate if you update coveralls to depend on simplecov ~> 0.13.0. That would allow coveralls to run under Ruby 2.4 without deprecation warnings.
If you want me to submit a PR for this change, please ping me.

I'm also available to submit a PR to speed up things

Made this PR: #117


Missing dss.exe SPF results?

Hello - in running the latest DSS.exe build and spot checking with EasyDmarc's Domain Scanner, I noticed that there appears to be SPF information consistently missing in some of the results.

Example 1 - Missing SPF information:
dss scan eisac.com -t120
scanResult:
domain: eisac.com
dmarc: v=DMARC1; p=quarantine<EMAIL_ADDRESS>mailto:4f5d1955<EMAIL_ADDRESS>ruf=mailto:4f5d1955@forensics.dmarc-report.com,mailto:it_dmarc_report@eisac.com;pct=100
mx:
- esa1.nerc.iphmx.com.
- esa2.nerc.iphmx.com.
duration: 861.5467ms
---  Update 09/07/2023
EasyDMARC specific SPF results:
https://easydmarc.com/tools/spf-lookup?domain=eisac.com

Example 2 - Returns SPF information:
dss scan cpi.coop -t120
scanResult:
domain: cpi.coop
dkim: v=DKIM1; k=rsa; p=MIGfMA0GCSqGSIb3DQEBAQUAA4GNADCBiQKBgQCb40a/s7UGrW5XdJqwk6Ins6Maya+tsu4mRKTCSccmY/IFHK4LVW9tInfZobj9y/9UB9grJs4/MywhJvkaN1rKzDsdG03rZv7b4LG/Q386c7MN7/UpbmIEij4c1S7qY0rwS4Czofi68/9TKXA7rgTvK3u4+XoNtEVN096Z2LejDQIDAQAB;
dmarc: v=DMARC1; p=quarantine<EMAIL_ADDRESS>mx:
- my-mx.email-protect.gosecure.net.
spf: v=spf1 mx a:mail.cpi.coop a:mx.cpi.coop a:www.cpi.coop include:spf.sendwp.com include:spf.protection.outlook.com ~all
duration: 1.1513876s
---  Update 09/07/2023
EasyDMARC specific SPF results: https://easydmarc.com/tools/spf-lookup?domain=cpi.coop

Hey @OrlandoMD, thanks for opening this issue. It took me a little while to figure out what was going wrong. The underlying library we use to make DNS requests (https://github.com/miekg/dns) makes use of the RFC 1035 spec, which specifies that plain old DNS responses should be 512 bytes. The issue is that eisac.com's TXT records combined consume more than 512 bytes. I've added an override to circumvent this, and will release this shortly.

This should now be fixed. I've added a new commandline option: dnsBuffer. I've increased the default buffer from 512 to 1024, but you can use this option to increase the value if needed. If you experience any further issues, please let me know!


Travis change for greater tracking of RPM issues

Issues:
Fixes #930
Problem:

RPM packages are getting created in Travis

With a rate of about 20%, random parsing errors can occur with deps

Analysis:

The test protion of travis scripting was moved to 'script' section

This is a mission-critical section for declaring a test failure
Upon test failure, work around is to rebuild the test to get rpm

Tests:

This is test code

@richbrowne
What's this change do?
Moves the testing portion of Travis's testing code for RPM package building (and debian by proxy) into the Travis's script section.  This should cause the tests to fail if/when a package is not properly built.
Where should the reviewer start?
.travis.yml file
Any background context?
This is only to address our tests failing if/when a travis build fails.

The rest of the commit looks fine.  Though I don't know why the packaging failed.

So there's generally no reason to keep the setup.cfg and the stdeb.cfg files around even as templates... if later we add them to gain some additional settings, then that's fine, the existing handling will use them; however, I've set up defaults for these files to be used if they're not present.

@richbrowne ready for re-review.  Please see latest commit for changes.  Put it in a separate commit to isolate non-critical updates.


disable enforcing successrate based outlier detection

Fixes https://github.com/istio/istio/issues/25220
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Can this please also be backported to 1.5?

In response to a cherrypick label: #25534 failed to apply on top of branch "release-1.5":
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/v1alpha3/cluster.go
M	pilot/pkg/networking/core/v1alpha3/cluster_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/networking/core/v1alpha3/cluster_test.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/cluster_test.go
Auto-merging pilot/pkg/networking/core/v1alpha3/cluster.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/cluster.go
Patch failed at 0001 disable enforcing successrate based outlier detection

In response to a cherrypick label: new issue created for failed cherrypick: #25542

In response to a cherrypick label: #25534 failed to apply on top of branch "release-1.6":
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pilot/pkg/networking/core/v1alpha3/cluster.go
M	pilot/pkg/networking/core/v1alpha3/cluster_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/networking/core/v1alpha3/cluster_test.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/cluster_test.go
Auto-merging pilot/pkg/networking/core/v1alpha3/cluster.go
CONFLICT (content): Merge conflict in pilot/pkg/networking/core/v1alpha3/cluster.go
Patch failed at 0001 disable enforcing successrate based outlier detection

In response to a cherrypick label: new issue created for failed cherrypick: #25543

@Stono automatic cherrypick is failing. I will create manual PRs tomorrow for this in 1.6 and 1.5


button: flat style regression

Bug
Expected behavior
Actual behavior
Flat appearance
Wrong color
Wrong hover color and bgColor
Steps to repro

Can see on docs site

Related packages

button@latest (since Position release with Button .0 release)

Environment

OS - MacOS 10.12
Browser version - Chrome v60

This is a problem with a withTheme and React.forwardRef compatibility.
We fixed withTheme.  And this is leading to the creation of a useTheme hook and upgrading of likely many components (any that use  but not withTheme).


Add support for default argument variables (JS, others?)

function foo(param = 123) {

should get documented as:
@param {[type]} [val=true] [description]

or better yet, automatic type detection:
@param {Number} [val=true] [description]

#129 implemented this feature. :smile:
@nikhilkalige can you please close this issue.


Permissions on resolv.conf restricted user access

Since the use of F36, the container base uses the following permissions on resolv.conf:
[root@WinT14 ~]# ls -al /etc/resolv.conf
-rwx------ 1 root root 198 Jul 14 13:19 /etc/resolv.conf

this means that a new user is unable to use the DNS resolver:
[root@WinT14 gbraad]# su - core
[core@WinT14 ~]$ curl google.com
curl: (6) Could not resolve host: google.com

This could be fixed by setting:
[root@WinT14 gbraad]# chmod 644 /etc/resolv.conf

This is not a direct issue for our usecase, but would prevent people who want to use a non-root user inside the WSL environment.
Suggested to make this part of the image for now.

https://bugzilla.redhat.com/show_bug.cgi?id=2106999

@n1hility I saw some changes you made to fix this, but it seems there is now an issue with successive builds ?

@n1hility yes will fix in a sec

Upstream didn't reply as they changed behaviour. Will probably reach out to some of the other base maintainers.
They used to include a prepopulated file, with entries from NetworkManager. While not correct for the container, this did create a file with the correct permissions. Since F36 they seem to have dropped with file, but I was unable to track where this change happened. The bootstrap files on Pagure of Fedora do not have this change.

root cause is that anaconda has been changed: https://github.com/rhinstaller/anaconda/pull/3818

Filed a PR at pagure for this: https://pagure.io/fedora-kickstarts/pull-request/898

I would argue that this is potentially an issue with the container runtime. For example.. Why does podman not have the same issue?
$ podman run -it --rm registry.fedoraproject.org/fedora:36 bash
[root@d87d7bfb8951 /]# ls -l /etc/resolv.conf
-rw-r--r--. 1 root root 45 Jul 20 17:42 /etc/resolv.conf

I think the goal by removing the /etc/resolv.conf from anaconda in https://github.com/rhinstaller/anaconda/pull/3818 is to toss the management of it to something else, which arguably is the right thing to do.

@dustymabe it's not an issue for podman since it bind mounts over whatever would be in the image. The WSL issue occurs since they truncate and overwrite preserving the bad perms.  I agree it should be fine for fedora not to include it. The underlying issue seems to be the interaction of the container runtime and the podman export process for example:
$ podman run --name ftry -it fedora:36 bash
[root@02476962449d /]# ls -l /etc/resolv.conf
-rw-r--r--. 1 root root 25 Jul 20 20:18 /etc/resolv.conf
[root@02476962449d /]#
exit
$ podman export ftry -o ftry.tar
$ tar -tvf ftry.tar | grep resolv.conf
-rwx------  0 root   root        0 Jul 20 15:18 etc/resolv.conf

I believe what's happening here is that when the bind mount for resolv.conf is created by crun, the location is first touched with 700 perms, which when exported (either live or post shutdown) the file is present leading to the problem.
Note this doesn't happen if you disable the resolv.conf bind:
$ podman run --name ftry --dns none -it fedora:36 bash
[root@6e99489d2188 /]# ls -l /etc/resolv.conf
ls: cannot access '/etc/resolv.conf': No such file or directory
[root@6e99489d2188 /]#
exit
$ podman export ftry -o ftry.tar
$ tar -tvf ftry.tar | grep resolv.conf
$ echo $?
1

Also runc appears to use a different umask, so works in this case:
$ podman --runtime=runc run --name ftry -it fedora:36 sh
sh-5.1# ls -l /etc/resolv.conf
-rw-r--r--. 1 root root 25 Jul 20 20:49 /etc/resolv.conf
sh-5.1#
exit
$ tar -tvf f.tar  | grep resolv.conf
-rwxr-xr-x root/root         0 2022-07-20 15:49 etc/resolv.conf

Not sure if we should propose change to crun or just accept it as a known issue.

Not sure if we should propose a change to crun or just accept it as a known issue.

The known issue is something that would have to be documented for the use of Fedora as a non-root container image. I think the majority of people will not realize what is happening.

@dustymabe it's not an issue for podman since it bind mounts over whatever would be in the image. The WSL issue occurs since they truncate and overwrite preserving the bad perms. I agree it should be fine for fedora not to include it. The underlying issue seems to be the interaction of the container runtime and the podman export process for example:

There are a lot of small details here that I'm missing. Is WSL just taking the Fedora container image directly and using it? If that's the case then how does "truncate and overwrite preserving the bad perms" happen because the file doesn't exist at all right?
From your comments it seems like maybe we massage the image before handing it to WSL?
Find me on IRC (i'm in #podman) if you'd like to talk more.

@dustymabe it's not an issue for podman since it bind mounts over whatever would be in the image. The WSL issue occurs since they truncate and overwrite preserving the bad perms. I agree it should be fine for fedora not to include it. The underlying issue seems to be the interaction of the container runtime and the podman export process for example:

There are a lot of small details here that I'm missing. Is WSL just taking the Fedora container image directly and using it? If that's the case then how does "truncate and overwrite preserving the bad perms" happen because the file doesn't exist at all right?
From your comments it seems like maybe we massage the image before handing it to WSL?

That's right, we run the image in podman to add packages and export a new rootfs tar, which WSL needs for imports.(workflow is here if you are curious). Since the file doesn't exist in the image, the act of running it in a container leads to the creation of the empty resolv.conf to bind mount a proper network config. When we get to the export at the end the resolv.conf with bad perms ends up included in the resulting rootfs tar.
To workaround we edit the tar and delete the entry, and add in an empty placeholder with standard perms just in case.

Find me on IRC (i'm in #podman) if you'd like to talk more.

Sure will do.

Right. I'm thinking we should revert the https://pagure.io/fedora-kickstarts/pull-request/898 PR and file a bug against podman (assuming there is one) and in the short term fixup the permissions on this file in your tooling that creates the WSL image.

+1 to revert the kickstart hackery once this is fixed elsewhere, in the meantime if this is helping to have WSL working I would be fine leaving the hack in place.
Note the change was only made in rawhide so far, @gbraad where you able to test the rawhide image to make sure we the kickstart change worked?

Not yet. Will do so asap.
Is the rawhide container published in the fp.o registry?

We have fixed the WSL toolchain by adding a workaround.
I'll  also file an issue against gitpod

Not yet. Will do so asap. Is the rawhide container published in the fp.o registry?

yes and quay.io using the rawhide tag
podman pull quay.io/fedora/fedora:rawhide


urbackup-client: fix gcc-13 build failure

Description of changes
Without the change build fails on` master as
https://hydra.nixos.org/build/249116447:
blockalign_src/crc.cpp: In function 'bool cryptopp_crc::IsAlignedOn(const void*, unsigned int)':
blockalign_src/crc.cpp:191:23: error: 'uintptr_t' does not name a type
  191 |                 const uintptr_t x = reinterpret_cast<uintptr_t>(ptr);
      |                       ^~~~~~~~~

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks for the fix!


Wireframe the site

Create wireframes to figure out the layout of the site.

We will need wireframes for:

[ ] home
[ ] individual method page
[ ] contribute
[ ] about


The Temurin download pages incorrectly points to Eclipse Foundation bug reporter

Describe the bug
I wanted to report a problem with the web page https://adoptium.net/temurin/releases/?version=8.
At the bottom at that page, there are headings Eclipse Foundation and Useful Links.
Under Useful Links there is a link Report a Bug, but when I used that link to report a bug with the Temurin web page, my bug report was rejected with "This bug tracker should only be used to report issues with services and infrastructure provided by the Eclipse Foundation.".
Expected behavior
The Adoptium web pages should not point at the wrong bug tracker.
Also, the footer gives the impression that the site supplies "services and infrastructure provided by the Eclipse Foundation", but apparently this is not the case.
Screenshots

@chrisguindon would there be any issue with us updating this link? The footer was copied from the EF template

@gdams Should be fine. The required links are listed in this policy:
https://www.eclipse.org/org/documents/eclipse-foundation-hosted-services-privacy-and-acceptable-usage-policy.pdf

To ensure all web pages related to operation of the server use either the standard
Eclipse.org footer template, or a footer that prominently contains a copyright notice, and
the following set of links:

Main Eclipse Foundation website (https://www.eclipse.org)
Privacy policy (https://www.eclipse.org/legal/privacy.php)
Website terms of use (https://www.eclipse.org/legal/termsofuse.php)
Copyright agent (https://www.eclipse.org/legal/copyright.php)
Legal (https://www.eclipse.org/legal)

@gdams @chrisguindon which one should be the right link to get pointed at?

@gdams @chrisguindon which one should be the right link to get pointed at?

So I think the links that need changing are:
Report a Bug => https://github.com/adoptium/adoptium-support/issues
Documentation => /docs
How to Contribute => /contributing


02.02.01

As a claimant, I want the list of expense claims to be sorted by starting date of travel, in order from most recent to oldest, so that ongoing or recent travel expenses are quickly accessible.

Basic Flow:

User logs in 2. User is taken to viewlist activity 7. Claims are automatically sorted by date

Exceptions:

There is no claim in claim menu

Use Case is written.
No UI test needed in part 2, revisit in part 3.


Delete Tooltip from Large CPU Widget

Delete Tooltip from large CPU Widget

added manually in 4b89ead11, thanks!


jump to most recent message

Regarding the iOS app. Is there any way to make it jump to the most recent message? I know there is the little arrow at the bottom, but that only jumps to the most recent message that is loaded -- which is a problem if you're like me and only use the app once every few weeks or so.

Hi @probabilistic, thanks for your question. I'm sorry to say this is an issue we're aware of, but a fix hasn't landed yet. To focus conversation and progress on it, I'm closing this as a duplicate of https://github.com/zulip/zulip-mobile/issues/1974.


router is not intercepting messages.. namespace specific problem

I'm trying to configure this library in my app. The configuration is like the following:
var router = require('socket.io-events')();
router.on(function(socket, args, next){
  console.log('the middleware...');
  // next();
});

var server = http.Server(app);
var io = IO(server);
io.use(passportSocketIo.authorize({
  cookieParser: cp,
  key: config.cookieKey,       	// the name of the express cookie
  secret:      config.sessionSecret,    	// the session_secret to parse the cookie
  store:       mStore,        			// mongo session storage
  success:     onAuthorizeSuccess,  		// *optional* callback on success
  fail:        onAuthorizeFail,     		// *optional* callback on fail/error
}));

io.use(router);
app.io=io;
server.listen(config.port);

As you can see, I'm using express and also I'm using passportSocketio to authorize socket connections, but I don't know why the router.on function is not working.. no console.log is logged and also my messages arrive directly with my 'method'.
Am I doing something wrong in this configuration ?

Hi to both of you, I think I have a similar problem with namespaces. I am using socket.io-events in conjunction with data.io which uses namespaces. Basically if I remove the namespace from the data.io code it works, when I add it back, it does not. I was wondering what it is that you did to add the router to the namespace.

Ok I did something like io.of('namespace').use(router) and that seemed to solve the problem (turns out this library does not work well with data.io but that is an unrelated issue). It would be nice to add this to the documentation of this wonderful library. As a suggestion request, there could be an option to also use a wildcard mechanism for the namespace but I guess it is a less frequent use case.

Sure I can take a look at it and and see what I can I do.
On Sunday, May 10, 2015, brucou<EMAIL_ADDRESS>wrote:

Ok I did something like io.of('namespace').use(router) and that seemed to
solve the problem (turns out this library does not work well with data.io
but that is an unrelated issue). It would be nice to add this to the
documentation of this wonderful library. As a suggestion request, there
could be an option to also use a wildcard mechanism for the namespace but I
guess it is a less frequent use case.
—
Reply to this email directly or view it on GitHub
https://github.com/turbonetix/socket.io-events/issues/13#issuecomment-100697783
.


Fix/ipct1 333/add sagas to community

This PR fixes [IPCT1-333] at https://impactmarket.atlassian.net/browse/IPCT1-333
Description
This PR implements REDUX SAGA approach on listCommunities and findById methods.
Type of change

New feature (adds functionality)

How Has This Been Tested?

[x] Manually

[x] BLU Advance L5
[ ] SLIDE SP4514
[ ] Asus ZenFone 3 Max
[ ] iPhone 6
[x] Other REDMI 5Plus

[ ] Automated

Screenshots/Videos
https://user-images.githubusercontent.com/44679989/129243587-a0f03698-e39c-4f23-99a5-7c79aeb6b28f.mp4

Suggested and approved one change.


Add a staging buffer manager

Without direct storage, we first need to copy data from the storage to the CPU's RAM first and then copy that into a CPU visible buffer. The CPU visible buffer is is called the staging buffer. We can use it that way in the GPU but it is slower than GPU only memory. So, for things which don't need any interactions with the CPU afterwards, it's better to copy them from a CPU visible buffer to a GPU only one.
The idea for now is to have a Staging Buffer Manager class where I will keep all the staging buffers. A GPU only buffer/texture will provide the manager with necessary details to create a staging buffer for it. The manager will also store necessary information so it can also make the CPU visible to GPU only copy later on and also the ownership stuff.

For now the plan is to have different functions for 2D and 3D images and Mip Levels as default arguments. One more function for BufferToBuffer. And in general take the offsets and copy size as arguments. Then have a batch copy and a single copy functions. Also need to store the dst Buffer/TextView references. These buffers would mostly be read only, so all of the CommandBuffers would share a single buffer, so I would need to wait for all of frames to be finished before updating these. CPU only ones wouldn't need staging buffers. Replacing the counter value of would require the upload buffer to be not temporary, so wouldn't need to worry about that. So, it should be fine to only subscribe to the QueueFinishEvent and then copy.

Made an initial implementation.


github-runner: 2.292.0 -> 2.293.0

Description of changes

Release notes: https://github.com/actions/runner/releases/tag/v2.293.0
diff: https://github.com/actions/runner/compare/v2.292.0...v2.293.0
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
N/A x86_64-darwin
N/A aarch64-darwin

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

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

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

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/526


Failed to compile when run yarn next build

Failed to compile.
./framework/ordercloud/utils/product.ts:16:5
Type error: Type '{ id: string; options: { id: string; __typename: "MultipleChoiceOption"; displayName: string; values: { label: string; }[]; }[]; }[]' is not assignable to type 'ProductVariant[]'.
Property 'price' is missing in type '{ id: string; options: { id: string; __typename: "MultipleChoiceOption"; displayName: string; values: { label: string; }[]; }[]; }' but required in type 'ProductVariant'.
14 |       currencyCode: product.xp.PriceCurrency,
15 |     },

16 |     variants: product.xp.Variants?.length
|     ^
17 |       ? product.xp.Variants.map((variant) => ({
18 |           id: variant.ID,
19 |           options: variant.Specs.map((spec) => ({
error Command failed with exit code 1.

Hey there! Thank you for opening this issue. We have decided to take Next.js Commerce in a new direction and will be closing out current PRs and issues due to this change. Please see this PR for more details: https://github.com/vercel/commerce/pull/966


seeAuthentication() not working

Hi,
i have a Problem with $I->seeAuthentication().
I become only false on my Test but when i do it directly on my browser, all is fine
http://laravel.io/bin/qQLy6
I hope you can help. Sorry for my English
Kind Regards

Ok now it ist working for me thank you


Make string/character auto-escaping more prominent

This functionality is unexpected and buried in between examples down the page. This adds a note just below where compound specifiers are first introduced to make it more prominent for anyone reading the docs.

Only change I'd make is to replace "add the - flag to %(" with something like "use %-( instead of %("—both because it's more direct, and because the - in %-( is not actually a flag, grammatically.

Only change I'd make is to replace "add the - flag to %(" with something like "use %-( instead of %("—both because it's more direct, and because the - in %-( is not actually a flag, grammatically.

I'm pretty much just copying what was already there and moving it up a bit. However, changing it to make it more clear is a good idea so I'll do that while I'm at it.

For both the dlang.phobos failures, it shows the following error message:
@azure-pipelinesazure-pipelines
/ dlang.phobos (Windows_VisualD x86-mscoff)
Build log #L36
Git fetch failed with exit code: 128

The issue seems unrelated to this PR, which only changes the docs.

I restarted the tests and they're green now.

CircleCI seems to be stuck "waiting for status to be reported". Does anyone know how to give it a kick to get things moving again?

Force push.


grpc: 1.35.0 -> 1.36.0

Motivation for this change
https://github.com/grpc/grpc/releases/tag/v1.36.0
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build or-tools

Last I checked, OR-Tools requires Abseil built with CMAKE_CXX_STANDARD=17, so we may need to provide both variants.

Yeah, this makes or-tools fail with /nix/store/cp1sa3xxvl71cypiinw2c62i5s33chlr-binutils-2.35.1/bin/ld: /nix/store/wv2mvvxpi6ic18j93vhbqgglffw0raqh-abseil-cpp-20200923.3/lib/libabsl_status.a(status.cc.o):(.rodata+0xc0): multiple definition of `absl::lts_2020_09_23::Status::kMovedFromString'; objs/lp_data/mps_reader.o:(.rodata._ZN4absl14lts_2020_09_236Status16kMovedFromStringE[_ZN4absl14lts_2020_09_236Status16kMovedFromStringE]+0x0): first defined here.

Thanks for reviewing! yes we probably need to provide two variants
@GrahamcOfBorg build or-tools

@andersk I dropped the changes to abseil.
IIUC we should build abseil with
[ "-DCMAKE_CXX_STANDARD=17" "-DBUILD_SHARED_LIBS=ON" ]

and offer an static variant for or-tools


Discrepancies in the documentation

Hi,
There is a confusing statement in this page.
There is no "All billing scopes" menu item in the left pane.
There is a "Billing scopes" menu item instead.
Also, it reads that "Properties" should exist under "Settings" but it doesn't. There is "Billing properties" instead, but an "Account Admin" box doesn't exist there either.
Please update the page.
Thank you.
Kind regards,
Eugene

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

ID: 665ef070-9879-3f6b-d528-ff64bd5236ff
Version Independent ID: 9ca6d22b-e684-4e65-cab3-c272189a3ddc
Content: Add or change Azure subscription administrators - Microsoft Cost Management
Content Source: articles/cost-management-billing/manage/add-change-subscription-administrator.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@eg-indeemo
Thanks for your feedback! We will investigate and update as appropriate.

Hi @eg-indeemo, Thanks for sharing the feedback. You are correct about Billing scopes vs All Billing scopes mismatch so we have verified and updated the document as required with regards to it. However, your point about below paragraph is not correct as I see the Account Admin as per the instructions provided in the paragraph. For more clarification, please take a look at below screenshot. Let me know if you have any further queries with regards to this topic. Thanks again!
If you're not sure who the account administrator is for a subscription, visit the Subscriptions page in Azure portal. Then select the subscription you want to check, and then look under Settings. Select Properties and the account administrator of the subscription is shown in the Account Admin box.

@eg-indeemo I see. What do you see under "Billing properties" tile under "Settings"?

@eg-indeemo Not sure what's the issue is. Tagging @bandersmsft requesting to provide any insights here. Otherwise you could raise a support ticket and work with Azure technical support team and get back here with any document enhancement feedback. Thanks.


browse_popler() documentation

Update documentation for the following changes:

Argument group_by has been changed to group_factors
Function popler() has been updated to browse_popler()

And include the argument options (e.g., "study", "species") in the documentation

Wow!
Been a long time! Now popler() has become browse() and it'll soon become something along the lines of  pop_browse()` (to try compliance with the rOpenSci naming standards).
Hence, I we can consider this issue closed! :)


feat: load the model from local

Feature request
can't do it.can you tell which file write this,i want to rewrite it.
Motivation
No response
Other
No response

You can already load the model from local path with --model-id

Please reopen if there is bug with loading from local path.


Security definition missing in Swagger JSON.

Thanks for providing this gem, it helped me a lot and I have something up and running.
During the process of setting everything up (Grape, WineBouncer, Swagger and SwaggerUI) I ran into the issue, that in the generated Swagger definition the "security" declaration for my secured methods are missing:
module API
  module V1
    class Me < API::V1::Base
      resource :me do
        desc "Return information of authenticated user"
        oauth2
        get do
          present UserRepresenter.prepare(resource_owner)
        end
      end
    end
  end
end

Is generating the following JSON:
"/api/v1/me": {
  "get": {
    "summary": "Return information of authenticated user",
    "description": "Return information of authenticated user",
    "produces": [
      "application/vnd.api+json"
    ],
    "responses": {
      "200": {
        "description": "Return information of authenticated user"
      }
    },
    "tags": [
      "me"
    ],
    "operationId": "getApiV1Me"
  }
},

When I add desc "Return information of authenticated user", security: [{ "oauth2": []}] to the definition of the description of the method, everything works fine, but I expected WineBounce would take care of that automatically, so I wouldn't need to declare it twice.
As Oauth2 definiton I have the following setup:
"securityDefinitions": {
  "oauth2": {
    "type": "oauth2",
    "description": "The API is secured by OAuth 2.0",
    "tokenUrl": "...",
    "authorizationUrl": "...",
    "flow": "password",
    "scope": []
  }
},

And I'm using the Swagger strategy:
WineBouncer.configure do |config|
  config.auth_strategy = :swagger
  config.define_resource_owner do
    User.find(doorkeeper_access_token.resource_owner_id) if doorkeeper_access_token
  end
end

I'm currently unsure if this is a bug like discussed in https://github.com/antek-drzewiecki/wine_bouncer/issues/32 or https://github.com/antek-drzewiecki/wine_bouncer/issues/51 or an error in my setup?
Would be great to get some feedback, because I tried a lot, but didn't found a proper setup.
Thanks for your time and support!

Hi @nlsrchtr,
Thanks for the issue. At the time of writing WineBouncer tried to be API 1,x compliant. No securityDefinitions existed at that point. It shouldn't be hard to that it would generate securityDefinitions or both authorizations.
The only issue is when multiple authorization strategies are defined such as oauth and basic http authentication. Feel free to see if you can make it spec 2 and 3 completable :). I might have a moment later on this/next week if you don't succeed!


Force UTF-8 encoding of lsb_release output

It's possible that the output may be encoded ASCII-8BIT, which cannot be automatically encoded to UTF-8 by Ruby's JSON library.
In this particular case, elementaryOS 6.1 returns Jólnir as a part of the lsb_release Description field.
Ruby's IO.popen would return this whole output in ASCII-8BIT.
When this description field was passed to homebrew-bundle to be written in a JSON lockfile,
Ruby throws an exception because it could not automatically reencode from ASCII-8BIT to UTF-8.
This may actually affect other output of the calls to the underlying Utils.popen_read but this may be the first thing to trigger it since
JSON requires UTF-8 output and nothing else using that function may require UTF-8.
Fixes #12689
Fixes Homebrew/homebrew-bundle#1043

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

I verified this works on my own installation. I'll add some tests if maintainers think this is an acceptable way to fix this.

Review period will end on 2022-01-11 at 02:15:13 UTC.

Review period skipped due to critical label.

🥳🥳🥳🥳


Added stickers

Seems to be missing a .gitsubmodule and stickers.html didn't seem to exist in the HHEU branch of the hexbin repo.


apps sc: falco alert group-by

What this PR does / why we need it:
We had the issue of that all FalcoAlerts came in the same alert, not separated by a rule. Now, a separation of rules has been made. There are 12 different falco-rules.
Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes #696
Public facing documentation PR (if applicable)

Special notes for reviewer:
Checklist:

[x] Added relevant notes to WIP-CHANGELOG.md
[x] Proper commit message prefix on all commits
[x] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[ ] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

Chart checklist (pick exactly one):

[x] I upgraded no Chart.
[ ] I upgraded a Chart and determined that no migration steps are needed.
[ ] I upgraded a Chart and added migration steps.

Pipeline config (if applicable)
If you change some config options (e.g. add/rename variable or change the default value) you may need to update the config used by the pipeline in pipeline/config.

I think I previously discussed with someone about testing to remove all of the group-by values, so that everything comes in separate alerts.
Did you try that? If not, could you try that and add some thoughts on which variant works best for us? I'm happy to help with this testing if you want.

I think I previously discussed with someone about testing to remove all of the group-by values, so that everything comes in separate alerts. Did you try that? If not, could you try that and add some thoughts on which variant works best for us? I'm happy to help with this testing if you want.

After some conversation, we decided that less grouping is what we want for more details.

Now it should be correct!


Users/arjgupta/iis mg update2

There is an issue on Ln 17.  Visual Studio creates transforms as Web..config.  Line 17 is adding the environment config to the transformation array as .config, which is not the same as Web..config.  The 3.0 release task uses this library and my environmental configs are not being fired and I believe it's because of this.


Rollback symlinks incorrectly every other rollback

Q
A

Issue Type
Bug

Deployer Version
4.0.1

Local Machine OS
Windows (git bash)

Remote Machine OS
Ubuntu (vagrant)

Description
Performing multiple rollbacks causes the current symlink to not link correctly on every second rollback.

Steps to reproduce
Deploy application multiple times. Then perform a rollback.
After each rollback, perform a ls -al on the destination server, app directory. The 'current' symlink fails on every other rollback. It links to the directory 3 for example, instead of releases/3. But on the next rollback it will link correctly.
Content of deploy.php
<?php
namespace Deployer;
require 'recipe/laravel.php';

// Configuration

set('repository', 'git@git.SITE.com:kirk.docker/status-page.git');

server('vagrant', '<IP_ADDRESS>', '2222')
    ->user('vagrant')
    ->password('vagrant')
    ->set('deploy_path', '/var/www/local-status.SITE.com');

Output log
$ dep rollback -vvv
Ô×ñ Executing task rollback
[vagrant] > cd /var/www/local-status.SITE.com && ([ -d releases ] && [ "$(ls -A releases)" ] && echo "true" || echo "false")
[vagrant] < true
[vagrant] > cd /var/www/local-status.SITE.com && (cd releases && ls -t -d */)
[vagrant] < 5/
[vagrant] < 4/
[vagrant] < 3/
[vagrant] < 2/
[vagrant] < 1/
[vagrant] > cd /var/www/local-status.SITE.com && (if [ -f .dep/releases ]; then echo "true"; fi)
[vagrant] < true
[vagrant] > cd /var/www/local-status.SITE.com && (tail -n 10 .dep/releases)
[vagrant] <<PHONE_NUMBER>3722,1
[vagrant] <<PHONE_NUMBER>4037,1
[vagrant] <<PHONE_NUMBER>4633,1
[vagrant] <<PHONE_NUMBER>4955,2
[vagrant] <<PHONE_NUMBER>1343,2
[vagrant] <<PHONE_NUMBER>5527,3
[vagrant] <<PHONE_NUMBER>5542,4
[vagrant] <<PHONE_NUMBER>5555,5
[vagrant] > cd /var/www/local-status.SITE.com && (if [[ "$(man ln)" =~ "--relative" ]]; then echo "true"; fi)
[vagrant] < true
[vagrant] > cd /var/www/local-status.SITE.com && (cd /var/www/local-status.SITE.com && ln -nfs --relative /var/www/local-status.SITE.com/releases/4 current)
[vagrant] > cd /var/www/local-status.SITE.com && (rm -rf /var/www/local-status.SITE.com/releases/5)
Rollback to `4` release was successful.
ÔÇó done on [vagrant]
Ô£ö Ok [518ms]

KirkD@HRT-PC07 MINGW64 /c/xampp/htdocs/status-page (deployment)
$ dep rollback -vvv
Ô×ñ Executing task rollback
[vagrant] > cd /var/www/local-status.SITE.com && ([ -d releases ] && [ "$(ls -A releases)" ] && echo "true" || echo "false")
[vagrant] < true
[vagrant] > cd /var/www/local-status.SITE.com && (cd releases && ls -t -d */)
[vagrant] < 4/
[vagrant] < 3/
[vagrant] < 2/
[vagrant] < 1/
[vagrant] > cd /var/www/local-status.SITE.com && (if [ -f .dep/releases ]; then echo "true"; fi)
[vagrant] < true
[vagrant] > cd /var/www/local-status.SITE.com && (tail -n 10 .dep/releases)
[vagrant] <<PHONE_NUMBER>3722,1
[vagrant] <<PHONE_NUMBER>4037,1
[vagrant] <<PHONE_NUMBER>4633,1
[vagrant] <<PHONE_NUMBER>4955,2
[vagrant] <<PHONE_NUMBER>1343,2
[vagrant] <<PHONE_NUMBER>5527,3
[vagrant] <<PHONE_NUMBER>5542,4
[vagrant] <<PHONE_NUMBER>5555,5
[vagrant] > cd /var/www/local-status.SITE.com && (if [[ "$(man ln)" =~ "--relative" ]]; then echo "true"; fi)
[vagrant] < true
[vagrant] > cd /var/www/local-status.SITE.com && (cd /var/www/local-status.SITE.com && ln -nfs --relative /var/www/local-status.SITE.com/releases/3 current)
[vagrant] > cd /var/www/local-status.SITE.com && (rm -rf /var/www/local-status.SITE.com/releases/4)
Rollback to `3` release was successful.
ÔÇó done on [vagrant]
Ô£ö Ok [545ms]

I've diffed the output and each rollback produces essentially identical info. Does this happen to anyone else, or is it something to do with my config?

Can't reproduce it locally. Can you debug it a little more? Please, add output of releases_list inside this task.

I'm not entirely sure what you mean, so I did the following: (note that version numbers will be different than original post):
desc('Get releases_list');
task('releases_list', function(){
    print_r(get('releases_list'));
});

I get the following output:
Ô×ñ Executing task releases_list
Array
(
    [0] => 4
    [1] => 3
    [2] => 2
    [3] => 1.1
Ô£ö

Let me know  if you meant something else.
Thank you.

Hmmm, difficult to say. Maybe you can repeat all steps with this task to reproduce error?

I've narrowed it down a bit. If I run the following code directly on the server this causes the 'flipping' behavior.
cd /var/www/local-status.SITE.com && (cd /var/www/local-status.SITE.com && ln -nfs --relative /var/www/local-status.SITE.com/releases/1.1 current)
And if I narrow it down further to this part, the issue still happens:
ln -nfs --relative /var/www/local-status.SITE.com/releases/1.1 current
Those commands were taken directly from the debug output of the rollback command.
Results:
vagrant@vagrant-ubuntu-trusty-64:/var/www/local-status.SITE.com$ ls -al
total 20
drwxrwxr-x 5 vagrant vagrant 4096 Nov 25 14:28 .
drwxr-xr-x 4 vagrant vagrant 4096 Nov 25 12:08 ..
lrwxrwxrwx 1 vagrant vagrant   12 Nov 25 14:28 current -> releases/1.1
drwxrwxr-x 2 vagrant vagrant 4096 Nov 25 14:10 .dep
drwxrwxr-x 4 vagrant vagrant 4096 Nov 25 14:22 releases
drwxrwxr-x 3 vagrant vagrant 4096 Nov 25 12:08 shared
vagrant@vagrant-ubuntu-trusty-64:/var/www/local-status.SITE.com$ ln -nfs --relative /var/www/local-status.SITE.com/releases/1.1 current
vagrant@vagrant-ubuntu-trusty-64:/var/www/local-status.SITE.com$ ls -al
total 20
drwxrwxr-x 5 vagrant vagrant 4096 Nov 25 14:28 .
drwxr-xr-x 4 vagrant vagrant 4096 Nov 25 12:08 ..
lrwxrwxrwx 1 vagrant vagrant    3 Nov 25 14:28 current -> 1.1
drwxrwxr-x 2 vagrant vagrant 4096 Nov 25 14:10 .dep
drwxrwxr-x 4 vagrant vagrant 4096 Nov 25 14:22 releases
drwxrwxr-x 3 vagrant vagrant 4096 Nov 25 12:08 shared
vagrant@vagrant-ubuntu-trusty-64:/var/www/local-status.SITE.com$ ln -nfs --relative /var/www/local-status.SITE.com/releases/1.1 current
vagrant@vagrant-ubuntu-trusty-64:/var/www/local-status.SITE.com$ ls -al
total 20
drwxrwxr-x 5 vagrant vagrant 4096 Nov 25 14:28 .
drwxr-xr-x 4 vagrant vagrant 4096 Nov 25 12:08 ..
lrwxrwxrwx 1 vagrant vagrant   12 Nov 25 14:28 current -> releases/1.1
drwxrwxr-x 2 vagrant vagrant 4096 Nov 25 14:10 .dep
drwxrwxr-x 4 vagrant vagrant 4096 Nov 25 14:22 releases
drwxrwxr-x 3 vagrant vagrant 4096 Nov 25 12:08 shared
vagrant@vagrant-ubuntu-trusty-64:/var/www/local-status.SITE.com$ ln -nfs --relative /var/www/local-status.SITE.com/releases/1.1 current
vagrant@vagrant-ubuntu-trusty-64:/var/www/local-status.SITE.com$ ls -al
total 20
drwxrwxr-x 5 vagrant vagrant 4096 Nov 25 14:30 .
drwxr-xr-x 4 vagrant vagrant 4096 Nov 25 12:08 ..
lrwxrwxrwx 1 vagrant vagrant    3 Nov 25 14:30 current -> 1.1
drwxrwxr-x 2 vagrant vagrant 4096 Nov 25 14:10 .dep
drwxrwxr-x 4 vagrant vagrant 4096 Nov 25 14:22 releases
drwxrwxr-x 3 vagrant vagrant 4096 Nov 25 12:08 shared

So I guess it's an OS thing? I've no idea why this is happening. Any ideas?
Thanks

I think it maybe a --relative option. Please, try with set('use_relative_symlink', false);.

Removing --relative fixes the issue on the command line.
I've just tried using deployer and this works correctly now as well.
Thanks for your help 👍

So it's bug then --relative is on. Will fix it.

@kirkdocker can you try using this?
ln -nfs --relative /var/www/local-status.SITE.com/releases/1.1 /var/www/local-status.SITE.com/current
I tried to reproduce your issue but it was impossible to me using xenial-64. But when you do a release, a similar symlink is created, and that works for you right?

@jordisala1991 Good catch with that one.
set('use_relative_symlink', false);
fixed the issue for me.
Cheers.

Beware that -r is a GNUism, i.e. non POSIX so won't work in the OP case as the standard OS X ln command is BSD based.
Also, Ubuntu 14.04.5 LTS is still using coreutils 8.21-1ubuntu5.4
So, I am not sure how good idea is --relative option to be default one.

@stz184 deployer detect is system supports relative symlinks and then use it. Otherwise use in without this option.


Default price probability for Market Trading

In v1 we had 50% probability by default for untraded markets.
Now with Invalid as a tradable outcome what should be the probability for newly created market?
revised: return last trade price for price or null if outcome hasn't been traded.

WE should show a dash instead of any probability for untraded markets.....Showing any number (even the range divisable by the total number of outcomes) is misleading

Also we may need to consider handling invalid differently as well....there is most likely never going to be a good price to sell invalid.....the bid of invalid is what is most useful imo

For untraded outcomes I think it should default to the halfway point between the best bid and best ask. If there are no bids and/or no asks, then a dash. This is what I think is most useful to users + the closest thing to what the market is saying in terms of price/probability and since markets will only be visible if they have close bids/asks, this data should usually be present.

For untraded outcomes I think it should default to the halfway point between the best bid and best ask.- No this is misleading to the user.....and showing bad information is much worse than showing no information, when there is nothing to show....As a trader showing me nothing has traded is more useful than showing me a false price pt

this data should usually be present....that is wrong....on any centralized exchange, that data would never be present.  It is only represented by a dash (or left empty) until it does.

@Chwy5 @matt-bullock should we do this now and get it out of the way? Doesn't seem a big change to trading/market cards.

@cillianhunter0 moved this item into the design list in Wrike, if this gets worked now it will get added to a timeslot.


update responsibleai and raiwidgets to erroranalysis 0.4.5

Description
update responsibleai and raiwidgets to erroranalysis 0.4.5
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2264 (264c8ad) into main (0f54933) will decrease coverage by 4.96%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2264      +/-   ##
==========================================
- Coverage   93.10%   88.15%   -4.96%
==========================================
  Files         103       17      -86
  Lines        5179      287    -4892
==========================================
- Hits         4822      253    -4569
+ Misses        357       34     -323

Flag
Coverage Δ

unittests
88.15% <ø> (-4.96%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 86 files with indirect coverage changes


Multiline item's separator doesn't use '\n'

Category

[ ] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[x] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[x] fish

I'm trying to get multiline history completion work by history -z | fzf --read0. However it generates items that are not separated by \n but \? which breaks command line substitution.

fzf does not officially support multi-line entries, and with the premise, a new line character is not a valid character fzf can properly display on screen, and fzf prints it as ? character instead. I know it is not ideal, but it's how it works right now.
fzf displays those characters as ?, but that does not affect the final output of fzf, so you can still depend on it.
> echo -en "foo\nbar" | fzf --read0 --select-1 | wc -l
       2

Notice that we have no issues with multi-line commands in bash and zsh, as history and fc -l 1 commands of them are smart enough to transform a multi-line command into a single-line entry.
$ a() {
> 1
> 2
> }
$ HISTTIMEFORMAT= history | tail -2
94918  a() { 1; 2; }
94919  HISTTIMEFORMAT= history | tail -2
$

Maybe fish can do something similar?

ah, fish's read command also needs a -z flag.  This works history -z | eval (__fzfcmd) --read0 -q '$str' | read -lz result


Clearer example on extending UI

Description
Is it possible to give a simple example of how to extend the UI?
https://docs.atoti.io/latest/lib/atoti.config.session_config.html#atoti.config.session_config.SessionConfig.app_extensions
config = {
    "app_extensions": {
        "some-extension": "../extensions/some-extension-package-directory/dist"
    }
}

After configuring some-extension, how do we make use of it in atoti?
Would be nice to have a sample .tsx extension example in the template folder.

Hi, after running npm run build on the template project and generating the dist folder, I pointed the config to the dist folder but I am receiving server error 500 when attempting to access the atoti embedded web server. The config I am using is
config = {
    "app_extensions": {
        "random-name": "path-to-dist"
    }
}

Are there any steps that I missed? Thanks.

Is it possible to give a simple example of how to extend the UI?

Isn't the code you pasted exactly that?

After configuring some-extension, how do we make use of it in atoti?

There is nothing special to do, once the extension is added to the app_extensions mapping, it will be enabled automatically when you open the app.

Would be nice to have a sample .tsx extension example in the template folder.

The app extension template linked just above the code example in the docs has the following contents:

The .ts and .tsx files show what you would like no?

Are there any steps that I missed?

Do you have an Atoti+ license and the atoti-plus plugin installed? They are required to use app extensions as indicated in the docs. If yes, can you share your sessions logs (at session.logs_path) so we can see the HTTP errors you're getting?

Hi, thanks for pointing me to the location of the logs. I realised that I had the incorrect "some-extension" name, I replaced it with the name property in package.json and there are no problems now. Thanks for your time.

name and path will be validated and a clear error thrown if necessary to avoid such issues in the next release


Index Sources & Publish Symbols task fails on TLS 1.2 only communication

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Index Sources & Publish Symbols (PublishSymbols)
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines

Account name: microsoft
team project name: WDATP
build definition name/build number: 35451

Agent - Private

OS: windows server 2016 Datacenter
agent version: 2.165.0

Issue Description
Since we updated our private build agents servers to work only with TLS 1.2 (and disable older TLS or SSL) this task is failing.
Error:
##[debug]Caught exception from task script.
##[debug]Error record:
##[debug]Invoke-WebRequest : The underlying connection was closed: An unexpected error occurred on a receive.
##[debug]At C:\Agent\_work\_tasks\PublishSymbols_0675668a-7bba-4ccb-901d-5ad6554ca653\2.0.16\PublishSymbols.ps1:20 char:15
##[debug]+ ...    $result = Invoke-WebRequest $serviceDefinitionUri -UseBasicParsing
##[debug]+                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
##[debug]    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:Htt    pWebRequest) [Invoke-WebRequest], WebException
##[debug]    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShe    ll.Commands.InvokeWebRequestCommand
##[debug]
##[debug]Script stack trace:
##[debug]at Get-SymbolServiceUri, C:\Agent\_work\_tasks\PublishSymbols_0675668a-7bba-4ccb-901d-5ad6554ca653\2.0.16\PublishSymbols.ps1: line 20
##[debug]at <ScriptBlock>, C:\Agent\_work\_tasks\PublishSymbols_0675668a-7bba-4ccb-901d-5ad6554ca653\2.0.16\PublishSymbols.ps1: line 202
##[debug]at <ScriptBlock>, <No file>: line 1
##[debug]at <ScriptBlock>, <No file>: line 22
##[debug]at <ScriptBlock>, <No file>: line 18
##[debug]at <ScriptBlock>, <No file>: line 1
##[debug]Exception:
##[debug]System.Net.WebException: The underlying connection was closed: An unexpected error occurred on a receive. ---> System.ComponentModel.Win32Exception: The client and server cannot communicate, because they do not possess a common algorithm
##[debug]   at System.Net.SSPIWrapper.AcquireCredentialsHandle(SSPIInterface SecModule, String package, CredentialUse intent, SecureCredential scc)
##[debug]   at System.Net.Security.SecureChannel.AcquireCredentialsHandle(CredentialUse credUsage, SecureCredential& secureCredential)
##[debug]   at System.Net.Security.SecureChannel.AcquireClientCredentials(Byte[]& thumbPrint)
##[debug]   at System.Net.Security.SecureChannel.GenerateToken(Byte[] input, Int32 offset, Int32 count, Byte[]& output)
##[debug]   at System.Net.Security.SecureChannel.NextMessage(Byte[] incoming, Int32 offset, Int32 count)
##[debug]   at System.Net.Security.SslState.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest)
##[debug]   at System.Net.Security.SslState.ForceAuthentication(Boolean receiveFirst, Byte[] buffer, AsyncProtocolRequest asyncRequest)
##[debug]   at System.Net.Security.SslState.ProcessAuthentication(LazyAsyncResult lazyResult)
##[debug]   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
##[debug]   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
##[debug]   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
##[debug]   at System.Net.TlsStream.ProcessAuthentication(LazyAsyncResult result)
##[debug]   at System.Net.TlsStream.BeginWrite(Byte[] buffer, Int32 offset, Int32 size, AsyncCallback asyncCallback, Object asyncState)
##[debug]   at System.Net.ConnectStream.WriteHeaders(Boolean async)
##[debug]   --- End of inner exception stack trace ---
##[debug]   at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.GetResponse(WebRequest request)
##[debug]   at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.ProcessRecord()
##[error]The underlying connection was closed: An unexpected error occurred on a receive.
##[debug]Processed: ##vso[task.logissue type=error]The underlying connection was closed: An unexpected error occurred on a receive.
##[debug]Processed: ##vso[task.complete result=Failed]

Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]

Hi, any workaround for this?

Still waiting. Any update on this?

still waiting for a response...

still waiting for a response...

I have the same issue.

see: https://github.com/microsoft/azure-pipelines-tasks/issues/15139


[SPARK-23289][CORE]OneForOneBlockFetcher.DownloadCallback.onData should write the buffer fully

What changes were proposed in this pull request?
channel.write(buf) may not write the whole buffer since the underlying channel is a FileChannel, we should retry until the whole buffer is written.
How was this patch tested?
Jenkins

Test build #86892 has finished for PR 20461 at commit c186a6b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86892/
Test PASSed.

lgtm

LGTM, also cc @jinxing64

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/466/
Test PASSed.

Merged build finished. Test PASSed.

Test build #86917 has started for PR 20461 at commit fed6dc2.

Test build #86917 has finished for PR 20461 at commit fed6dc2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86917/
Test PASSed.

@cloud-fan thanks a lot for ping. LGTM

thanks, merging to master/2.3!


Unknown Commands

I assume that this is to be updated later? export and autocomplete isn't working

Yes, these are our plans for the future and are not in the V1.4 release.

User Guide has been updated to address this issue #134


Automated cherry pick of #8133: fix(scheduler): filter InstanceGroup whose enabled is false

Cherry pick of #8133 on release/3.4.
#8133: fix(scheduler): filter InstanceGroup whose enabled is false

/lgtm
/approve


CSI WekaFS Driver - mount on servers

Hi, using the driver requires me to mount WEKA file system on node as prerequisites ? the CSI driver can mount the volume for me ?

Hello,
Kubernetes node must have the Weka client software installed and be already a member of Weka cluster.
Either using a legacy Weka client installation (then no need to mount any filesystem on the node), or using a stateless Weka client installation  (in such case, installation and registration of the node in Weka cluster will be done seamlessly upon a mount)


(PDB-2625) Upgrade raynes.fs to 1.4.6

This harmonizes the version of raynes.fs with the one used in other
projects.
This requires upgrading kitchensink and trapperkeeper as well, because
raynes.fs 1.4.6 is backward-incompatible with 1.4.5. Specifically, the
absolute-path and normalized-path functions are removed. Those functions
have been added to kitchensink 1.3.0, so we now use the kitchensink
version. Because trapperkeeper 1.2.0 also depended on fs 1.4.5 (and is
thus broken by upgrading it), it's also now version 1.3.1.

Should go along with puppetlabs/pe-puppetdb-extensions#180.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/619/
Test PASSed.

@nicklewis looks like unit tests are broken https://travis-ci.org/puppetlabs/puppetdb/jobs/123131263

Ughhh I guess there must be other breaking changes. Will look into it...

@nicklewis this might help:
https://github.com/puppetlabs/pe-file-sync/pull/207/files
The one that bit me there is that list-dir started returning file objects instead of strings, so I needed to add calls to fs/base-name, and/or remove calls to fs/file.

@cprice404 Thanks. I was considering that change as a possible culprit, but that function doesn't appear to be used except in different tests than the ones that are failing. Even then, it's only checked for emptiness.
I can't find any other functions whose behavior seems different, so I'm wondering if it could be in some dependency that's using raynes.fs...

Alright I figured it out and pushed up a fix.
kitchensink/normalized-path does not do the same thing that fs/normalized-path did, even though kitchensink/absolute-path does do the same thing that fs/absolute-path did. So the correct way to upgrade from fs 1.4.5 to fs 1.4.6 is to change fs/absolute-path to kitchensink/absolute-path and to change fs/normalized-path to fs/normalized.

@nicklewis ugh.  How does normalized-path differ between ks and old fs?

oh.  so in the old fs, absolute-path returned a String and normalized-path returned a File.  That seems dumb.

@cprice404 Yeah. I think the new behaviors in fs and kitchensink make sense; it just means updating uses of the two functions is different.

gtk, will remember this in case it trips anyone else up.  thx

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/620/
Test PASSed.

We need to make sure we're HUPable before merging this.

@ajroetker I'm not sure I understand the HUP'able requirement, is this because we're bumping to latest TK in this patch also? What work needs to be done to get this past the line, do we need an extra patch or a set of tests, afaik we've done no work on HUP support as yet.

@kbarber precisely, this bumps the TK version to HUP, so I thought we had agreed in standup that this patch was blocked by that ticket.

Cool, makes sense. This work is tracked in: https://tickets.puppetlabs.com/browse/PDB-2546.

I've merged this into my branch for HUP support in PDB-2546, since that depends on this bump. It'll get merged when my patch goes in.


Add MultipleChoiceSingleCharacterTask (MMLU, etc.)

Depends on https://github.com/EleutherAI/lm-evaluation-harness/pull/497.

I’m worried that the notion of “single character” isn’t tokenizer- and language-invariant. Why is this an important thing to add to the framework?

I was thinking calling it MultipleChoiceSimpleTask instead, as it isn't just "single character", but any cases when acc == acc_norm (or when acc_norm isn't needed), i.e. when completion_len values are all the same:
https://github.com/EleutherAI/lm-evaluation-harness/blob/23f30926f3ce738e3eee4e6be5c29fb3467e3a6e/lm_eval/base.py#L726-L728
It is (Unicode) character-level because doc["choices"] is a (Python) string, so completion_len is the numbers of (Unicode) characters.
And the reasons to add this:

Some tasks like MMLU don't need acc_norm, so they need something like MultipleChoiceSimpleTask
Faster runs (e.g. 4x faster for MMLU, though there are some TODOs about making MultipleChoiceTask faster in general, but that can be a separate thing)
Avoiding some precision errors (results logprobs for different answers can be the same, though rarely)
Potentially allowing MultipleChoiceTask tasks to use MultipleChoiceSimpleTask with some command-line parameter (e.g. we don't want acc_norm and want faster runs).

Renamed to MultipleChoiceExactTask, so it is good for tasks that can just check for exact matches in the correct completion, like the new MMLU, instead of scoring all of the completions.


[FLINK-5364] Rework JAAS configuration to support user-supplied entries

Fixes FLINK-5364, FLINK-5361, FLINK-5350, FLINK-5055
CC @tillrohrmann

@EronWright thanks a lot for this fix, I am looking though it now.
From the description in the JIRAs, I take that this adds the logic that reads custom JAAS security configurations and uses Flink's internal one (Hadoop UGI based) for fallback default?
This should be relevant for the master and the 1.2 release branch, correct?

@StephanEwen yes, this PR does the following:

isolate the configuration of kerberos credentials from how those credentials are used
modularize the security code to reflect the independent aspects of configuring Hadoop vs. JAAS based on the credential.
make explicit as to which JAAS login contexts are provided the cluster's credential
incorporate the user-supplied JAAS (to satisfy the MapR scenario)
update config.md and internals/security.md

The main user impact is that one must explicitly share the credential with specific contexts - the 'KafkaClient' login context, ZooKeeper 'Client' context, etc, based on whether the corresponding service is kerberized.  Earlier we had tried to always share the credential to all JAAS contexts, but this caused problems.
With this patch, I believe the design goal is met of allowing Kerberos to be independently enabled for any external connection.  For example, a Kerberized Kafka + a non-Kerberized Hadoop is a valid scenario.

@EronWright Thanks for explaining. It figured it out concurrently by looking at the updated documentation you wrote ;-) The docs are good, helped a lot!

Thanks for the review, I'll address these comments ASAP.

@StephanEwen updated based on feedback, thanks again.


above: init at 2.7

Invisible network protocol sniffer
https://github.com/casterbyte/Above
Related https://github.com/NixOS/nixpkgs/issues/81418

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 359228

x86_64-linux

:white_check_mark: 2 packages built:

above
above.dist


[Sprint: 46] XD-2853 Deployment message consumer ordering

Make sure the ZK distributed queue consumer for
processing deployment messages is initialized after
the path cache is setup for module, stream and job
deployment requests paths.

The update makes sense to me.  But would want someone who has further knowledge of Leadership logic to greenlight it.

Squashed and merged as 4ee44a00e998224b163acd3660a34b3cb7f932a5


Windows CI for Python-Only Implementation

Get CI up and running for Windows implementation using only the Python runtime

Windows build is now successful with this branch: windows_CI, however the smoke-test fails due to a missing .dll file, as here

Actually cudnn_adv_train64_8.dll exists in that directory and isn't missing. The reason is that tensorrt and torch load their own cuDNN DLLs and the version conflicts. Until the PyTorch devs upgrade the bundled cuDNN version from 8.8 to 8.9 in the windows wheel, the only workaround I can think of is to manually copy the DLLs from site-packages/nvidia/cudnn/bin to site-packages/torch/lib and overwrites them.

@HolyWu - thank you for the suggestion on this. I will try out the manual copy in the smoke test to get the CI tests running in the meantime.


L7 rule and policy status updates to neutron fail

update_l7policy_status() and update_l7rule_status() in plugin_rpc.py fail because they do not define operating_status parameter.

Will close with branch collapse

Fixed by PR #301


ignore config.json

what do you mean?

Config.json is in .gitignore twice now.

From: Miguel González-Fierro<EMAIL_ADDRESS>Sent: Wednesday, June 12, 2019 4:45:29 AM
To: microsoft/recommenders
Cc: Scott Graham; Review requested
Subject: Re: [microsoft/recommenders] ignore config.json (#828)
what do you mean?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Frecommenders%2Fpull%2F828%3Femail_source%3Dnotifications%26email_token%3DABLUTWN47BJMI3YGVYUY6MDP2CZSTA5CNFSM4HW4BO22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXPWFDQ%23issuecomment-501179022&data=02|01|scott.graham%40microsoft.com|b29406dc79ef4d58046c08d6ef12530a|72f988bf86f141af91ab2d7cd011db47|1|0|636959259320486304&sdata=RuSgc2Gv4N3%2F1aFxavfMuo088c47svAnYOTzYnZAQwU%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABLUTWIZ6FVLUANCBAC6GQLP2CZSTANCNFSM4HW4BO2Q&data=02|01|scott.graham%40microsoft.com|b29406dc79ef4d58046c08d6ef12530a|72f988bf86f141af91ab2d7cd011db47|1|0|636959259320486304&sdata=qHKWfiKXxvjh3p5l4PGJeBxQlhWdnBrUP%2B4rr94PJVY%3D&reserved=0.

oh I see, I'll fix it


Issue with fhir-server/samples/templates/default-azuredeploy-docker-sql.json

Describe the bug
The image Tag "release" is causing an issue when I deploy the Azure App service in my environment.
To Reproduce

**What fixed
I changed the line 233
From
"linuxFxVersion": "[concat('DOCKER|', variables('registryName'), '/', toLower(parameters('fhirVersion')), '_fhir-server',':', parameters('imageTag'))]"
To
"linuxFxVersion": "[concat('DOCKER|', variables('registryName'), '/', toLower(parameters('fhirVersion')), '_fhir-server')]",
**Shouldn't the tag be "latest" instead of "release".
The api site is now running fine for me and I see that it downloaded the latest image from the registry.

@LTA-Thinking can you look at this issue?

@sethuraj, thanks for bringing this to our attention. latest is not really the best tag because it could lead to some unexpected behavior, if the apps service restarts you could upgrade unexpectedly. We probably need to make some changes to snap to a specific release in the template. So we will take a look.
That said, the tag release should actually be there, but we have an issue with the release pipeline and it is not putting that tag out, so we will take a look at that too.

Hi @sethuraj
The release tag is meant to be the latest version of the fhir server that has been officially released. The issue with the latest tag is that this repository is used for PR testing, so the latest image sent to it could be a PR build and not a version of the master branch.
We have had some issues with the release pipeline and as you saw there is currently no image with the release tag. We are working on remedying this.
I'll update this when we have a release tag up.
In the meantime I would recommend using the tag master-20200702-3.

There is now a release tag for each FHIR version, and the release pipeline has been fixed so it will update it whenever a new release is made.

There is now a release tag for each FHIR version, and the release pipeline has been fixed so it will update it whenever a new release is made.

Great thanks, will try the release version. Thanks


Switch IntensityMap backend to DimensionalData.jl

AxisKeys maintenance is done by a single author and it can take a while to get requests merged. DimensionalData seems more actively maintained and is used by a large number of significant packages at this point. We should switch the backend.

Solved


Linux_android spell_check_test is consistently failing, "The processing instruction target matching "[xX][mM][lL]" is not allowed."

[spell_check_test] [STDOUT] stderr: [        ] FAILURE: Build failed with an exception.
[spell_check_test] [STDOUT] stderr: [        ] * What went wrong:
[spell_check_test] [STDOUT] stderr: [        ] Execution failed for task ':app:mergeDebugResources'.
[spell_check_test] [STDOUT] stderr: [        ] > /opt/s/w/ir/x/w/recipe_cleanup/tmpg4n5wqkj/flutter sdk/dev/integration_tests/spell_check/android/app/src/main/res/values/styles.xml:5:6: Error: The processing instruction target matching "[xX][mM][lL]" is not allowed.
[spell_check_test] [STDOUT] stderr: [        ] * Try:

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8804378514533463825/+/u/run_spell_check_test/test_stdout
@camsim99 testowner

This is still failing but on another file:
[spell_check_test] [STDOUT] stderr: [        ] > /opt/s/w/ir/x/w/recipe_cleanup/tmpnduaklna/flutter sdk/dev/integration_tests/spell_check/android/app/src/main/res/values/styles.xml:5:6: Error: The processing instruction target matching "[xX][mM][lL]" is not allowed.

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8804283279601867281/+/u/run_spell_check_test/test_stdout
Please validate the test passes locally on your machine on the next fix. 🙂

Woot, it's green, thanks @camsim99!


Add option to permit jvm to attach to itself for BMUnit tests.

A PR #1527 was created to implement this optimization. But due to a necessary change in the workflow  it has been abandoned.


scale image when using googlenet

Should I scale the input image to 0-1 if i use googlenet?

I found this link that says otherwise.
https://stackoverflow.com/questions/44262567/caffe-googlenet-model-predictions-are-always-the-same


Add support for NIP-55

This pull request adds support for NIP-55 as per coracle-social/coracle#421

[X]  Add Nip55Signer to @welshman/signer with dependency injection for sending/receiving intents based on an interface
[X]  Add support to @welshman/app's getSigner function

This should probably result in a version number bump as well, once the PR is accepted

The prepare script for the plugin wasn't running properly on npm install and the package-lock.json was retaining an old commit hash. The latest push builds for me from a fresh pull of the issue#421 branch.
Adding the plugin as a peer dependency avoids having to convert the signer package to a capacitor plugin since it doesn't call any native code directly.
There's a bug that is preventing Coracle from fully loading with this version, but I suspect it's in the session code for Coracle itself, so I'm going to dig a bit deeper into that.

Got it, that all makes sense. Peer dependency it is.

There's a bug that is preventing Coracle from fully loading with this version, but I suspect it's in the session code for Coracle itself, so I'm going to dig a bit deeper into that.

Looks like you just forgot to pass the package name in when calling addSession. This fixes it:
export const loginWithAmber = pubkey =>
  addSession({method: "nip55", pubkey, package: 'com.greenart7c3.nostrsigner'})

That brings up another question though — is it possible to avoid hard-coding the specific package? Maybe by querying for  nostrsigner:` handlers?

That brings up another question though — is it possible to avoid hard-coding the specific package? Maybe by querying for nostrsigner: handlers?

Yes it is! The same query to the Package Manager returns enough info to create a list of the Packages, their name and their icon. I'm pushing an update to the plugin shortly that will include this feature as well as support for the Content Resolver to reduce the amount of round trips using Intents which will improve the UX greatly.

This has been merged: https://github.com/coracle-social/welshman/commit/e1f8d6a4e03c6aeaa697c9d9e2d9975da9b1c238


error when playing mkv audio videos

I found a problem in this script when opening an X video with MKV format plays the video but the audio does not sound when the file is dual audio example: Spanish - English

#71


[named_blobs] Add NamedBlobDB interface

Add an interface and a skeleton implementation of NamedBlobDB.
This interface will enable interaction with a mutable mapping layer for
named blobs.

Codecov Report

Merging #1666 (0f2712c) into master (214835b) will increase coverage by 2.98%.
The diff coverage is 80.14%.

@@             Coverage Diff              @@
##             master    #1666      +/-   ##
============================================
+ Coverage     70.75%   73.74%   +2.98%
- Complexity     8520     8909     +389
============================================
  Files           633      638       +5
  Lines         48655    48796     +141
  Branches       6098     6115      +17
============================================
+ Hits          34427    35985    +1558
+ Misses        12366    10923    -1443
- Partials       1862     1888      +26

Impacted Files
Coverage Δ
Complexity Δ

...github/ambry/config/MySqlAccountServiceConfig.java
100.00% <ø> (ø)
1.00 <0.00> (ø)

...n/java/com/github/ambry/named/NamedBlobRecord.java
65.00% <65.00%> (ø)
7.00 <7.00> (?)

.../java/com/github/ambry/named/MySqlNamedBlobDb.java
77.17% <77.17%> (ø)
20.00 <20.00> (?)

...om/github/ambry/config/MySqlNamedBlobDbConfig.java
100.00% <100.00%> (ø)
1.00 <1.00> (?)

...api/src/main/java/com/github/ambry/named/Page.java
100.00% <100.00%> (ø)
3.00 <3.00> (?)

...om/github/ambry/named/MySqlNamedBlobDbFactory.java
100.00% <100.00%> (ø)
3.00 <3.00> (?)

...hub/ambry/protocol/CatchupStatusAdminResponse.java
75.00% <0.00%> (-6.25%)
6.00% <0.00%> (-1.00%)

...in/java/com/github/ambry/clustermap/Partition.java
79.12% <0.00%> (-3.30%)
33.00% <0.00%> (-1.00%)

...github/ambry/rest/AsyncRequestResponseHandler.java
87.87% <0.00%> (-2.36%)
22.00% <0.00%> (ø%)

...a/com/github/ambry/clustermap/ClusterMapUtils.java
92.23% <0.00%> (-0.46%)
60.00% <0.00%> (ø%)

... and 104 more

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 214835b...0f2712c. Read the comment docs.


Issues when running WebQuestions processing pipeline

Hi Siva,
I'm trying to run the pipeline specified in Makefile in order to generate deplambda data from raw questions. I encountered some error when running: make create_data_en:
zcat: can't stat: data/freebase/mid_to_key.txt.gz
Am I missing something here?
Thanks!

Hi Luheng,
Glad you are trying UDepLambda :). Appreciate it. Generating entity annotations and running the whole pipeline involves lot of resources like CoreNLP, Freebase API, Freebase endpoint and some additional files. Freebase API no longer works so some parts of the pipeline is broken. The code also supports KG API (the replacement of Freebase API) but it's accuracy is lower than Freebase API. The documentation for that is missing.
If you are interested in using UDepLambda without going through Freebase API, this is the best place to start https://github.com/sivareddyg/udeplambda#usage
If you are interested in already entity annotated/linked data along with UDepLambda parses, try this https://github.com/sivareddyg/UDepLambda/blob/master/doc/FreebaseExperiments.md#multiligual-webquestions-and-graphquestions-data
But if you are interested in Freebase entity annotating and linking new data and then UDepLambda parsing, I can help in person since I am not sure what works and doesn't in your system. Even if I share data/freebase/mid_to_key.txt.gz, something else will be missing. Let's discuss over email/skype and set things up if this is important to you (PM me) . After that you can add the required documentation to the repo if you like.
Siva

Thanks Siva! I emailed you. Closing this issue now :)


Question about jupyterlab / jetcam

Hi,
first of all thanks a lot for this repo. I'm a newbie in this field and it's good to understand.
I'm trying to do some experiment with jupyterlab notebook but I haven't found a way to "convert" the my-detection sample in a good way.
For what I have understood the display.IsOpen() use the OpenGL / CV library and need a real monitor, so I'm trying to use the jetcam module and ipywidgets/IPython but I haven't figure out yet how to render the detection overlay.
I'm doing that on the jupyterlab notebook:

Import and configure the camera:

from jetcam.csi_camera import CSICamera
camera = CSICamera(width=320, height=240)

Display the first image:

image = camera.read()
import ipywidgets
from IPython.display import display
from jetcam.utils import bgr8_to_jpeg
image_widget = ipywidgets.Image(format='jpeg')
image_widget.value = bgr8_to_jpeg(image)
display(image_widget)

Load the detect net:

import jetson.inference
import jetson.utils
net = jetson.inference.detectNet("ssd-mobilenet-v2", threshold=0.5)

Loop:

camera.running = True
def update_image(change):
    image = change['new']
    detections = net.Detect(jetson.utils.cudaFromNumpy(image), 320, 240)
    image_widget.value = bgr8_to_jpeg(image)
camera.observe(update_image, names='value')

The image is correctly refreshed but I was not able to place the overlay with detection on the image.
For what I have understood the net.Detect take as input a PyCapsule objet and cudaFromNumpy() should do the conversion but maybe I'm wrong.
Any ideas ?

@GeneralFox Hello, do you have any update for this? Thank you!

@jiefengsun I think it needs to be like this instead:
camera.running = True
def update_image(change):
    image = change['new']
    cuda_img = jetson.utils.cudaFromNumpy(image)
    detections = net.Detect(cuda_img, 320, 240)
    jetson.utils.cudaDeviceSynchronize()
    image = jetson.utils.cudaToNumpy(cuda_img)
    image_widget.value = bgr8_to_jpeg(image)
camera.observe(update_image, names='value')

I've also added more interoperability interfaces for jetson_utils.cudaImage that may make this easier/faster:   https://github.com/dusty-nv/jetson-inference/blob/master/docs/aux-image.md#array-interfaces
I've also added WebRTC support for streaming video headlessly to browser, and it will give you higher resolution/framerate and lower latency than Jupyter would:   https://github.com/dusty-nv/jetson-inference#webapp-frameworks


[Android] Develop the UI Test for OTA/BDX in android chip-tool

Feature description
After the OTA provider enablement and BDX Java/JNI API landing, we need the reference implementation in android chip-tool to demo how to use these new APIs.
Platform
android
Platform Version(s)
No response
Anything else?
No response

https://github.com/project-chip/connectedhomeip/pull/30145


Revert "SHOW TRANSACTION only possible within a transaction"

This reverts commit d2bfa92354a37d636aaf4ae6d18ad383c6ea32ba.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/59a2204c8e3fdbdf7b1ba6c0edea136925972f1c/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e5602785b93c5133bca24844a5cbcea64ed3db8e/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5e39a9b3bbead222e07f100441ec2d1089535622/


Fix broken synths view

Dynamic width scaling for this page and all synths is broken. The right panel does not move with the screen size but is cropped.

@blokku-chan we need to handle the mint/settle... sidebar on different views to be viewed more dynamically

a note on page resizing, the content will get broken and this is normal as it is functionality related, once @ElmoArmy finalize on the sdk integration stuff on these pages it will be automatically fixed

@blokku-chan we want to check how to handle the mint/settle... sidebar on different views for it to potentially look more dynamic

a note on page resizing, the content will get broken and this is normal as it is functionality related, once @ElmoArmy finalize on the sdk integration stuff on these pages it will be automatically fixed


Use supervisord for process supervision

I see you guys are using tini as a very lightweight process supervisor.
I'm creating a derived image and I'd like to lauch another process along with jenkins.  I'm not familiar with tini, but I know this can be done with supervisord easily.  For example, I want to install the consul agent and launch it along with Jenkins for a derived image.  I can install supervisord, but cannot use it since there is already an ENTRYPOINT being used with tini to launch the Jenkins war as a process.
I'd like to see TINI replaced with supervisord to make derived images much more extensible.
If you know a way to do this with tini, I'll accept that as a solution as well.

The best practice is to only launch one process in the container. You can create your own image to override the ENTRYPOINT to anything you like

You are right on best practice.  In the event that I needed to run 2 processes in a derived image, do you know of a way to run 2 processes using TINI?  This way, I could replicate the existing ENTRYPOINT you have and just add another launched process (consul in my case).

Hey @jbpadgett (I'm the author of Tini)
If you absolutely nee to launch two children processes, you can have Tini launch supervisord for you:
ENTRYPOINT ["/bin/tini", "--", "supervisord"]

And then, have supervisord launch /usr/local/bin/jenkins.sh and Consul.

You could also have Supervisord launch Tini and have Tini launch Jenkins. That would work provided you enable subreaping in Tini (there's an option flag for this, try tini -h for more details). Getting rid of Tini altogether means you need to address https://github.com/jenkinsci/docker/issues/54 in some other way (supervisor does not reap zombies).
Hope this helps (though I realize I'm late to the party!).
Cheers,

Perfect.  Thanks so much for taking the time to answer.  I'll give this a try.