flutter: Bad state: No element

The plugin is raising this error (Bad state: No element) when executing the decode methond -> Excel.decodeBytes(bytes);
I can open the excel file perfectly in my PC but loading it from the assets raises this issue.

Can you please post the error log ?

@ArielFerreiro As it's seems that your error might be resolved, as we didn't hear from you.
Feel free to reopen the issue, if you finds any trouble.

Long Live Open Source

@ArielFerreiro As it's seems that your error might be resolved, as we didn't hear from you.
Feel free to reopen the issue, if you finds any trouble.

Long Live Open Source

Hi Singh (@justkawal) sorry I was on vacations and still a bit on it. The thing is in the file itself I think, as excel is opening it but before raises a warning telling me that will try to recover the data and finally it opens it correctly. So I guess it is something with the file itself. I will try to check it with my backend colleagues and in case come back.

Hi Singh (@justkawal) sorry I was on vacations and still a bit on it. The thing is in the file itself I think, as excel is opening it but before raises a warning telling me that will try to recover the data and finally it opens it correctly. So I guess it is something with the file itself. I will try to check it with my backend colleagues and in case come back.


Zoom In/Out UI

After adding rules and if the user zooming in/out, the first row is getting invisible partially. Please check the below screenshot.

Hello, @QAInsights
Since we changed the UI for the Correlation Rules, this issue looks fixed on our end but, this might not be your case on your machine. Please check out the last release, and let us know if you have any issues.
Just like the other ones, I'm closing this issue but, don't hesitate to open it again if you experience this kind of behavior with the new implementation.
Happy hacking & enjoy!


Minimized Header (identity card)

Just to have more space to work with, and also since the identity card is meant to just be a quick overview of, well, identity, it should be minimizable.
Then when user wants to see more things, or actually do something more involved with managing their identity, they can expand and go about their business.

+1, I like this. I think forget & backup are buttons that people rarely use, so I'd put them into a settings menu or something like that.

nice, would make sense to have the chain/block info next to the syncing status as well


Panic when instrumentation-opt-in enabled

Is there an existing issue for this?

[X] I have searched the existing issues

Output of ddev debug test
No response
Expected Behavior
Don't throw an error.
Actual Behavior
$ ddev version
 ITEM             VALUE
 DDEV version     v1.22.0-alpha3-29-g9b3b0d9b
 architecture     amd64
 db               ddev/ddev-dbserver-mariadb-10.4:v1.22.0-beta1
 ddev-ssh-agent   ddev/ddev-ssh-agent:v1.22.0-beta1
 docker           24.0.2
 docker-compose   v2.18.1
 docker-platform  dev
 mutagen          0.17.1
 os               linux
 router           traefik:v2.10
 web              ddev/ddev-webserver:v1.22.0-beta1

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x1d8 pc=0x91d79b]

goroutine 1 [running]:
github.com/ddev/ddev/pkg/ddevapp.(*DdevApp).GetOmittedContainers(...)
        /home/runner/work/ddev/ddev/pkg/ddevapp/ddevapp.go:371
github.com/ddev/ddev/pkg/ddevapp.(*DdevApp).TrackProject(0x0)
        /home/runner/work/ddev/ddev/pkg/ddevapp/amplitude_project.go:36 +0x7b
github.com/ddev/ddev/cmd/ddev/cmd.glob..func48(0x10e72c0, {0x112cda0?, 0x0?, 0x0?})
        /home/runner/work/ddev/ddev/cmd/ddev/cmd/root.go:98 +0x86
github.com/spf13/cobra.(*Command).execute(0x10e72c0, {0x112cda0, 0x0, 0x0})
        /home/runner/work/ddev/ddev/vendor/github.com/spf13/cobra/command.go:876 +0x770
github.com/spf13/cobra.(*Command).ExecuteC(0x10ea4c0)
        /home/runner/work/ddev/ddev/vendor/github.com/spf13/cobra/command.go:974 +0x3bd
github.com/spf13/cobra.(*Command).Execute(...)
        /home/runner/work/ddev/ddev/vendor/github.com/spf13/cobra/command.go:902
github.com/ddev/ddev/cmd/ddev/cmd.Execute()
        /home/runner/work/ddev/ddev/cmd/ddev/cmd/root.go:152 +0x25
main.main()
        /home/runner/work/ddev/ddev/cmd/ddev/main.go:28 +0xc5

Steps To Reproduce

Use the latest ddev binary.
ddev config global --instrumentation-opt-in=true
Run ddev version or ddev config global.

Anything else?
No response

Thanks for testing this path!

Thanks for the report @stasadev ! New binaries fixing this issue are available at https://github.com/ddev/ddev/pull/5066#issuecomment-1618958866

And thanks so much @stasadev for using/testing HEAD! Helps so much.

Glad to help!

In fact, the error occurred all the time with analytics enabled.
I don't know why I emphasized the disabled internet.
Anyway, the fix from @gilbertsoft works as it should.

That's weird @stasadev . I wonder why it didn't show up for others then. I'm sure we can completely prevent it, but surprised that it was happening to you all the time.

Maybe because I'm on HEAD, and the env variables for analytics are empty there.

That's an interesting theory. You must be installing HEAD by building yourself?

No, I used nighly.link.

OK, those should have the AmplitudeAPIKey in them. They don't? They warn you about not having it?

I've only seen this warning about AmplitudeAPIKey in PR that solves this problem.

Instrumentation is opted in, but AmplitudeAPIKey is not available. This usually means you have a locally-built ddev binary or one from a PR build. It's not an error. Please report it if you're using an official release build.

I just assumed that maybe there was no AmplitudeAPIKey on HEAD and DDEV was trying to connect with an empty key, the instrumentationApp was not initialized, and there was a panic error. But I could be wrong.

Ah thanks. Yeah, the PR doesn't get built with it (because built from a fork) but the HEAD build has it.


Tiny typo

(feel free to close this PR and perform the change yourself as you see fit)

Thanks @ffflabs !


Switch req file workflow

Nothing major, just reflecting a needed change to the naming of the root requirements file in the workflow, so nothing breaks!
Resolves #69

Trusting you on this


Error for avatar_url

What went wrong?
When I add a song on the queue, the bot gives me this error:
[ERROR:bot] Exception in on_message
Traceback (most recent call last):
  File "/home/debian/DiscordBots/LocandadiArroway-MusicBot-1-stanza1/musicbot/bot.py", line 4064, in on_message
    content = self._gen_embed()
  File "/home/debian/DiscordBots/LocandadiArroway-MusicBot-1-stanza1/musicbot/bot.py", line 1361, in _gen_embed
    icon_url=self.user.avatar_url,
AttributeError: 'ClientUser' object has no attribute 'avatar_url'

I downloaded all the repo some minutes ago for check if the error was fixed in the latest versions, but it didn't.
How do we reproduce this?
Just add a song on the queue with embed enables.
Which version of the bot are you using?
release-151221
Which operating system are you using?
Ubuntu 18.04 or higher
Just checking...

[X] I have confirmed that my issue has not already been reported.
[X] I am using Python 3.8 or higher.
[X] I have updated my Python dependencies.

Please provide the output of python -m discord --version

Python v3.9.2-final
discord.py v1.7.3-final
aiohttp v3.7.4.post0
system info: Linux 5.10.0-11-cloud-amd64 #1 SMP Debian 5.10.92-1 (2022-01-18)

I'm having this error too when I try to skip a song:
Traceback (most recent call last):
  File "/home/debian/.local/lib/python3.9/site-packages/discord/player.py", line 616, in _call_after
    self.after(error)
  File "/home/debian/DiscordBots/LocandadiArroway-MusicBot-2-stanza2/musicbot/player.py", line 247, in _playback_finished
    if e.winerror == 32:  # File is in use
AttributeError: 'PermissionError' object has no attribute 'winerror'

Are you sure that their that you have given your bot appropriate perms on Discord?

Yes. It has full permissions everywhere

Ok
Hmm this should not be happening

Ok I also this issue. Lets see

you need to use avatar.url because this is most likely using discord.py master branch

you need to use avatar.url because this is most likely using discord.py master branch
The bot uses pycord

I cannot reproduce this, could you remove your token in option.ini and zip the entire Musicbot folder and post it here?
Also, please run python3.9 -m pip list and post that here too so I know what modules are installed.

Closed due to no reply


 This appears to be because it runs each test individually. If I run them from command line it takes about 3 seconds to run 900 tests. Running from vscode instead takes about 3 minutes.
Reasons for requesting this feature
Tests are orders of magnitude slower to execute from vscode vs commandline.
Describe alternatives you've considered
Running tests in the command line is fine I guess, but it would be nice to have the integrated test runner work better.

do you have a code repo or sample project you can share?


ReferenceError: window is not defined

Hello,
I'm using NextJS and was getting:
ReferenceError: window is not defined

I solved this by using a dynamic import for the Sigma container:
import dynamic from "next/dynamic";

const SigmaContainer = dynamic(import("react-sigma-v2").then(mod => mod.SigmaContainer), {ssr: false})

Maybe it would be a good idea to add this to the documentation for people using NextJS?

Actually this didn't solve the issue, I'm still getting this with NextJS.

NextJS do server side rendering, so yes window is not defined, and Sigma  this lib are using it for some features.
I don't know how it can be solved... This more linked to NextJS than to this lib.
If you find a way  to let it works with NextJs, please share the detail, and I will do a doc about that !

Perhaps that using the dynamic import with the  check that if the cod eis running on server/client side will work :
const isBrowser = () => typeof window !== "undefined"
if(isBrowser) {
   const SigmaContainer = dynamic(import("react-sigma-v2").then(mod => mod.SigmaContainer))
  ...

I added my graph (useSigma and addNode etc) to client side as well:
const SigmaContainer = dynamic(import("react-sigma-v2").then(mod => mod.SigmaContainer), {ssr: false})
const NetworkGraph = dynamic(import("../components/graph").then(mod => mod.NetworkGraph), {ssr: false})

<SigmaContainer>
     <NetworkGraph/>
</SigmaContainer>

But I'm not getting the following error:
Error: Sigma: container has no height.

Any idea how to solve this?

I added the CSS:
import "react-sigma-v2/lib/react-sigma-v2.css";

But I'm still getting:
Error: Sigma: container has no height.

The div that contains sigma (ie. the <div class="sigma-container">) must have a height.
Normally the CSS in the lib put it at 100% of the parent container.
Can you try this : <SigmaContainer style={{height: "500px"}}> ?

I tried it with
<SigmaContainer style={{ height: "50%", width: "50%"}}/>

But your version works!
The CSS isn't making it the parent container's height and width for some reason.
No worries.
Thanks for your quick response I highly appreciate it.
Maybe an idea to reference this issue for people wanting to use NextJS with this library.
Kind regards,
Marten Mooij

I'm glad that it finally works !
Yes I need to improve the documentation, like for example that you needs to defined a size (width & height) on your container.
And to create a section about NextJS  easy from what I understood, it's just the dynamic loading when you are in the browser, right ?

FYI: https://github.com/sim51/react-sigma-v2#faq


NN Based Contextual Bandit for large datasets

Datasets we can use -

This collection has datasets (like Bibtex) with a large label space (>100)
Datasets that this paper tested on used datasets for much smaller (<10) label spaces.

Another difference between the two is in (2) they had to select a single label. Whereas for (1) each datapoint might have more than one label. So would we have to predict a very high dimnesional binary label vector.
The approach for (2) is training an NN to output parameters for a distribution over reward for each label which would be the logical extension of the simple tabular policies we implemented for categorial CBs.
For (1) this is intractable so we can implement other embedding based approaches.
So I think we should start with (2) first, what are your thoughts @Sharad24?
I am currently working on implementing a variation on the linear posterior method from 2

Where should I add the bandit agents for this? In deep/agents/bandit or classical/bandit/contextual ?


Captcha Dialog problem with MyJDownloader

Hello,
I have a problem with the docker instance of JDownloader.
The Captchas are not displayed properly at MyJDwonloader anymore.
The captcha notification pops up for a split second, but even if i manage to click in the notification in that short time frame the captcha solving window does not open.
The captcha solving was working before, but i haven't used JDownloader for a month or two and now it stopped working. Unfortunately I don't know if it is due to a system update or an update of the docker image that I made in the meantime when i wasn't using JDownloader.
I can still access the MyJDownloader GUI besides the captcha dialog, so i can see 'Downloads', 'LinkGrabber' etc. and i can download files without captcha protection.
I'm using it on linux x86_64 Ubuntu 18.04.2 LTS, kernel: 4.15.0-52-generic, docker v18.09.6-build481bc77, docker-compose v1.23.2.
The Captcha dialog in MyJDownloader still works with my Windows laptop JD instance.
Here is a little selection of my output.log file:

THREAD: 40
40|Log.L.log 6/21/19 2:39:18 PM - INFO [ org.appwork.storage.JsonKeyValueStorage() ] -> Prefer (merged) JSon Storage from File: /config/cfg/org.jdownloader.api.captcha.CaptchaMyJDownloaderRemoteSolverSettings.json
40|Log.L.log 6/21/19 2:39:18 PM - INFO [ org.appwork.loggingv3.LogV3(info) ] -> Try to register API namespace /captcha = interface org.jdownloader.api.captcha.CaptchaAPI
40|Log.L.log 6/21/19 2:39:18 PM - INFO [ org.appwork.loggingv3.LogV3(info) ] -> Try to register API namespace /captchaforward = interface org.jdownloader.captcha.api.CaptchaForwarderAPIInterface
40|Log.L.log 6/21/19 2:39:18 PM - INFO [ org.appwork.loggingv3.LogV3(info) ] -> Try to register API namespace /jd = interface org.jdownloader.api.jd.JDAPI
40|Log.L.log 6/21/19 2:39:18 PM - INFO [ org.appwork.loggingv3.LogV3(info) ] -> Try to register API namespace /downloads = interface org.jdownloader.api.downloads.DownloadsAPI
40|Log.L.log 6/21/19 2:39:19 PM - INFO [ org.appwork.loggingv3.LogV3(info) ] -> Try to register API namespace /downloadsV2 = interface org.jdownloader.api.downloads.v2.DownloadsAPIV2
40|Log.L.log 6/21/19 2:39:19 PM - INFO [ org.appwork.loggingv3.LogV3(info) ] -> Try to register API namespace /downloadcontroller = interface org.jdownloader.api.downloads.v2.DownloadWatchdogAPI
40|Log.L.log 6/21/19 2:39:19 PM - INFO [ org.appwork.loggingv3.LogV3(info) ] -> Try to register API namespace /config = interface org.jdownloader.api.config.AdvancedConfigManagerAPI
40|Log.L.log 6/21/19 2:39:19 PM - INFO [ org.appwork.loggingv3.LogV3(info) ] -> Try to register API namespace /toolbar = interface org.jdownloader.api.toolbar.JDownloaderToolBarAPI

Is this part of the log-file helpful? I don't really know what to look for in this file. Is it helpful to post the whole output.log or other log-files? If yes which?
Do you know what could be the problem here and how can i debug/fix it?
thank you very much for your help!
Best regards,
Bob

Are you using the MyJDownloader browser plugin?

Yes, I'm using the Browser Plugin. I tested it with Chrome and Firefox, both with Plugins installed on a Win 10 PC. I also tested the MyJDownloader Android App. I get the same results on all machines. Captchas of my Docker JD Host are not displayed, despite the fact that the captchas of the normal Windows JD instance work. Both are in the same network, so i think it doesn't seem to be a router firewall issue either.
Is there anything i can check/try to narrow the problem down?
Thanks!

I would make sure that:

JD is up-to-date: Help->Find Updates
You are properly logged to My.JDownloader: Settings -> My.JDownloader
The My.JDownload captcha plugin is enabled: Settings -> Captchas

Also, you could try to start the container with a clean appdata folder: Stop the container, rename the folder you mapped to /config and then start the container.

Hello jlesage,
i tried everything that you recommended but it still doesn't work.
I deleted the /config folder and started the container fresh. The captchas are still not displayed at MyJDownloader. Do you have any more suggestions? Do you knows what else could be the reason?
Thanks for your help!
Rob

Hi,
i just installed the current ubuntu 18.04 updates of this week and captcha solving does work again.
I don't know what has caused it in detail but it seems like it got fixed with the current updates.
Thank you for your help!
best regards,
Rob

Great, glad to see that it's now working for you !


Document monadic operators for Decoded.

Documents the monadic operators (flatMap, map, apply) for Decoded.
Option-clicking operators will show documentation in a popover.

:notebook_with_decorative_cover: :notebook_with_decorative_cover:   :notebook_with_decorative_cover:

Looks good! thanks for doing this man. Just a few comments.

:ok_hand::question:
:eyes: :pray:

Thanks @jdhealy!! Merged in here: e469524e153daee6accb16aea8a9502afb0c6e49


OpenGov. Delegations. Delegation summary

Add modal with 3 tabs

List of delegations
Voted last 30 days
Voted for all time list

figma link

Status verification: ❌
What has been verified:

https://github.com/novasamatech/nova-spektr/issues/2369
https://github.com/novasamatech/nova-spektr/issues/2368

Status verification: ✅


Merge pull request #4 from Netflix/fix/apply-plugin

1-Fix generation of java code when using the plugins block to apply the…

Hi @ftservice, could you rebase this PR so that it's clear what changed exactly?

Hi @ftservice, could you rebase this PR so that it's clear what changed exactly?

Closing because of inactivity.

Closing because of inactivity.


Fails to download file from YouTube.

Checklist

[x ] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x ] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar issues including closed ones

Verbose log

PASTE VERBOSE LOG HERE

Description

Unable to download video from YouTube.

#30839.


Added eth_ecrecover function

ecrecover recovers the ethereum address used to sign a message with ecsign (which is already implemented in repository), and is useful for verification. ecrecover has already been used in the repo as part of a test for ecsign in tests/requests.js (both are called from the ethereumjs-util library), and so is implemented using the same style.
thanks!

hi @benjamincburns, would you be able to give this a look? thanks.


fix: make aliases.drushrc.php.stub PHP 8 compatible

Checklist

[x] Affected Issues have been mentioned in the Closing issues section
[x] Documentation has been written/updated
[ ] PR title is ready for changelog and subsystem label(s) applied

Closing issues
Closes https://github.com/uselagoon/lagoon/issues/3047

I'n not sure about the PR title. The explanation was too much for me 😅


Add validator flags for configurable hours of blob data

This PR adds validator flags to make the hours of blob data collected configurable.
The flags are BLOCKS_PER_MINUTE of type int, and HOURS_OF_BLOB_DATA of type int.
The hours of blob data collected is calculated as BLOCKS_PER_MINUTE * HOURS_OF_BLOB_DATA * 60.

Review Error for danyalprout @ 2024-06-18 16:49:15 UTC
User failed mfa authentication, see go/mfa-help

This Pull Request was merged without enough reviews
Please go here to review and +1

This Pull Request was merged without enough reviews
Please go here to review and +1


Set meshtastic service working directory to data dir

Self-signed certs for HTTPS seem to get saved in whatever meshtasticd's current working directory is, so setting that to the configured data directory seems reasonable?

Hi @srd424, thank you for your PR!
Yes, I think that makes perfect sense. Could you please just make one more minor change in the commit?

Yup, no worries - will sort tomorrow, festivities permitting!

Sure, have a lovely holiday!


(Sorry for so much issues, YouTube keeps having updates 🙃) A new "Shorts" section in the watch page now

So I just discovered this today, but YouTube added a new "Shorts" section in the watch page (Maybe it's not new? I'm not really sure)
Anyways, here's a screenshot:

Personally, I'm not a big fan of it, so I turned on the "Hide Shorts" option. Here's the outcome.

Anyways, you don't really have to do anything yet, this issue can also be unfixable if the update hasn't rolled out yet for you! I just issued this so if it rolls out, you'll know what to expect!
(This is just another visual thing that takes away the old YouTube vibe lol, doesn't ruin viewing experience.)
That's all, and thank you!

Since i haven't received any updates from YouTube, could you please right-click the shorts area, select "inspect," and send me the screenshots?

Okay, sure! This update seems to only be present in one of my accounts, (the one that has Premium), I'm not so sure if this update is being tested for Premium user currently:
Anyways:


The CH grid should be the same as the LH one

The chaoshead grid starts in the top left, while the LH one starts in the bottom left. It would be nice if they were lined up so that I could use coordinates of items in LH easily in CH

@slothybutt Do you mean how they are displayed, or how they are used in scripts?
The current approach was chosen to make it easier to draw the objects, changing how positions are displayed can simply be done by adding a setting, changing how it's sued in script would be an annoying and breaking change.

I'm gonna make a new issue since I don't think this is a good idea at all and I'd rather just make a new one


[Synapse] - Generate SDK based on the latest Swagger spec

README.md link: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/synapse/resource-manager/readme.md
Tag: package-2019-06-01-preview

Hi @markcowl ,

There are too many PRs to list here. Please check link here: https://github.com/Azure/azure-rest-api-specs/commits/master/specification/synapse/resource-manager/Microsoft.Synapse/preview/2019-06-01-preview . The corresponding PRs are the ones since 10.12.
I will update the test recording. These new features are not core features. Can we add test cases soon since we really in a rush state?
there is no API version change.


Copy over two metric functions from Goalie

The functions determine_metric_complexity and intersect_on_boundary fit better here.
Closes #4.

Linked PR: https://github.com/pyroteus/goalie/pull/13.


docker cp cause /dev/pts to be unmounted.

docker version:
[root@ip-172-31-25-218 porx]# docker version
Client:
Version:      1.10.1
API version:  1.22
Go version:   go1.5.3
Git commit:   9e83765
Built:        Thu Feb 11 20:39:58 2016
OS/Arch:      linux/amd64
Server:
Version:      1.10.1
API version:  1.22
Go version:   go1.5.3
Git commit:   9e83765
Built:        Thu Feb 11 20:39:58 2016
OS/Arch:      linux/amd64
docker info:
[root@ip-172-31-25-218 porx]# docker info
Containers: 3
Running: 2
Paused: 0
Stopped: 1
Images: 3
Server Version: 1.10.1
Storage Driver: btrfs
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
Volume: local pxd
Network: null host bridge
Kernel Version: 3.19.3-1.el7.elrepo.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 7.304 GiB
Name: ip-172-31-25-218.us-west-1.compute.internal
ID: XRC7:XGJG:5VVJ:MO5H:LS4N:KTXO:HEKL:W57E:DNHE:STUN:QQ57:D6Q7
Username: XYZ
Registry: https://index.docker.io/v1/
Provide additional environment details (AWS, VirtualBox, physical, etc.):
AWS, EC2.
List the steps to reproduce the issue:

[root@ip-172-31-25-218 porx]# mount |grep pts
devpts on /dev/pts type devpts (rw,nosuid,noexec,relatime,gid=5,mode=620,ptmxmode=000)

[root@ip-172-31-25-218 porx]# docker run -it --name test_container -v /dev/:/dev/ busybox

[root@ip-172-31-25-218 porx]# docker cp test_container:/bin/sh /tmp

[root@ip-172-31-25-218 porx]# mount |grep pts

Describe the results you received:
/dev/pts is unmounted.
Describe the results you expected:
I ran -v /dev:/dev to map it all the host's devices into the container.  After docker cp, I expect the device to still be mounted on the host.
/dev/pts should not be unmount.

I can also deterministically reproduce the bug, with any storage drivers (aufs/overlay/btrfs) on Ubuntu 15.10 amd64 on AWS.
(docker/docker@dfebb6074f62891368cb3c668380c186f3bebb33, kernel 4.2.0-30-generic)
However, I cannot reproduce the bug on these environments:

Boot2Docker 1.10.2 on VirtualBox
Ubuntu 15.10 amd64 on baremetal box (docker 1.10.2-rc1, kernel 4.2.0-27-generic)

/dev/pts is unmounted within detachMounted("/var/lib/docker/aufs/mnt/foobar/dev")
Call stack:

https://github.com/docker/docker/blob/v1.10.2/container/container_unix.go#L655
https://github.com/docker/docker/blob/v1.10.2/container/container_unix.go#L682
https://github.com/docker/docker/blob/v1.10.2/daemon/archive.go#L157
https://github.com/docker/docker/blob/v1.10.2/daemon/archive.go#L57
..

Printf-debug patch:
diff --git a/container/container_unix.go b/container/container_unix.go
index b8bae23..140e783 100644
--- a/container/container_unix.go
+++ b/container/container_unix.go
@@ -7,7 +7,9 @@ import (
        "io/ioutil"
        "net"
        "os"
+       "os/exec"
        "path/filepath"
+       "runtime/debug"
        "strconv"
        "strings"
        "syscall"
@@ -653,7 +655,19 @@ func (container *Container) UpdateContainer(hostConfig *containertypes.HostConfi
 }

 func detachMounted(path string) error {
-       return syscall.Unmount(path, syscall.MNT_DETACH)
+       grepDEV(fmt.Sprintf("detachMounted(%s) ENTER", path))
+       logrus.Debugf("STACK: %s", debug.Stack())
+       err := syscall.Unmount(path, syscall.MNT_DETACH)
+       grepDEV(fmt.Sprintf("detachMounted(%s) LEAVE", path))
+       return err
+}
+
+func grepDEV(id string) {
+       out, err := exec.Command("sh","-c","mount | grep \"on /dev\"").Output()
+       if err != nil {
+               panic(err)
+       }
+       logrus.Debugf("grepDEV(%s):\n%s\n", id, out)
 }

 // UnmountVolumes unmounts all volumes

I can reproduce the bug only when I run the /usr/bin/docker daemon -s XXX directly from the prompt.
I can not hit the bug when I run the docker service from systemd.
/lib/systemd/system/docker.service (default configuration, 1.10.2-0~wily)
[Unit]
Description=Docker Application Container Engine
Documentation=https://docs.docker.com
After=network.target docker.socket
Requires=docker.socket

[Service]
Type=notify
ExecStart=/usr/bin/docker daemon -H fd:// -s XXX
MountFlags=slave
LimitNOFILE=1048576
LimitNPROC=1048576
LimitCORE=infinity
TimeoutStartSec=0

[Install]
WantedBy=multi-user.target

@AkihiroSuda so, this is a packaging issue? The 1.10.2 service file uses "slave" https://github.com/docker/docker/blob/v1.10.2/contrib/init/systemd/docker.service - where does the service file on AWS come from?

@thaJeztah
I was manually running the daemon.
So this is not a packaging issue.
I think docker should not unmount the host pts even when running without systemd,
because user cannot login to the host after pts is unmounted.
Can we move MountFlags=slave logic from the systemd service into the docker daemon itself?
(As in POC AkihiroSuda/docker@f2158c1?)

@AkihiroSuda can you open a PR with that change? We can then discuss the change on that PR

@AkihiroSuda sounds good..removing Mountflags=slave is not an option because mount propagation doesn’t work with that flag: https://github.com/docker/docker/issues/14630'

@thaJeztah Sure, #20712

Just to note, this is also affecting users of Docker 1.10.3 on Fedora 24 systems. I have filed https://bugzilla.redhat.com/show_bug.cgi?id=1326532 downstream.

The issue is still reproducible with docker/docker@348d90276802aac6c76126f2fe6866c521190ecd + opencontainers/runc@445d184c81685337fd640fc3da930244ae3c8e95.

non-docker repro instruction
$ mkdir /tmp/t1
$ mount -o rbind,rw /dev /tmp/t1/dev  # this mimics `daemon/container_operations_unix.go:mountVolumes()`
$ mount | grep pts
none on /dev/pts type devpts (rw,relatime,mode=600,ptmxmode=000)
none on /tmp/t1/dev/pts type devpts (rw,relatime,mode=600,ptmxmode=000)
$ umount -l /tmp/t1/dev # this mimics `daemon/container_operations_unix.go:detachMounted()`
$ mount | grep pts # /dev/pts should be kept, but we get an empty output here

I'm looking into this issue again.

I have proposed a PR #22009 to solve this issue. See if it makes sense.


Add handling for null PostgreSQL Range column values

It appears that NPE throws while getting null ranges from a table. PR tries to handle this situation.

Thanks. I'll review it when I have some time.

Applied upstream, thanks


Support modern python and django versions

This PR adds support for versions up to python 3.6.4 and django 2.0.1.  I ran into compatibility problems when using this library in my project so I decided to update it to support. Thanks for the library!

Sorry for the delay. Thanks for updating a supporting Python 3.x and Django 2.0+.
I was about to do the same thing. :-)


Value Added Tax (add field to model)

This change is

Coverage increased (+0.04%) to 46.474% when pulling 190a3839ed8d130756d8c7c6c0ed0a54f55340cc on ITVaan:value_added_tax into 3d7aa25119f7e7a0aae4f80b1579cba8cdda1bee on openprocurement:master.

Coverage increased (+0.04%) to 46.474% when pulling 190a3839ed8d130756d8c7c6c0ed0a54f55340cc on ITVaan:value_added_tax into 3d7aa25119f7e7a0aae4f80b1579cba8cdda1bee on openprocurement:master.

Coverage increased (+0.04%) to 46.474% when pulling 79eedc6c27d80cdd0a29682021c49cefbbf90af2 on ITVaan:value_added_tax into 3d7aa25119f7e7a0aae4f80b1579cba8cdda1bee on openprocurement:master.


Add different BIGINT data for load_table tests

@andrewseidl I added a new column for tests .. but not sure if it was what you mentioned before.
is there a specific test you want I add to the tests?

Yup, that works. The bug in omniscidb was such that something like [[1, 3], [4, 2]] would be imported as [[1, 2], [1, 2]], which the original test didn't catch (since the source data was [[1, 2], [1, 2]]).

I see, thanks for the explanation! :)


RelationGroup should contain target identifier, currently only target object

Prerequisites

[x] I have written a descriptive issue title
[x] I have verified that I am running the latest version of the ReqIFSharp
[x] I have searched open and closed issues to ensure it has not already been reported

Description

Feature request: Access to target identifier in RelationGroup(if target object isn't available in *.reqif file, external link)
Given a *.reqif file with a RelationGroup where the target object isn't part of the reqif file (external link). It would be nice to get the identifier of that target object (SpecObject).
New/Changed Interface:
string CoreContent[i].SpecRelations.TargetIdentifier or so
The xml snippet. The identifier 'x0400000001330720' isn't specified in *.reqif. It's part of another file.
<SPEC-RELATION IDENTIFIER="x1500000001330753" LAST-CHANGE="2015-03-31T16:33:20+02:00">     <TARGET>         <SPEC-OBJECT-REF>x0400000001330720</SPEC-OBJECT-REF>     </TARGET>     <SOURCE>         <SPEC-OBJECT-REF>x040000000133073F</SPEC-OBJECT-REF>     </SOURCE>     <TYPE>         <SPEC-RELATION-TYPE-REF>SPECREL</SPEC-RELATION-TYPE-REF>     </TYPE> </SPEC-RELATION>
Steps to Reproduce

only Interface extension.
System Configuration

ReqIFSharp version:

[1.3.2 ] ReqIFSharp:
[ ] Other:

Environment (Operating system, version and so on): windows 10
.NET Framework version: net 4.6.2
Additional information:

in the next release, such not-found references will not be null anymore but will represent an object of the right type (Specification for RelationGroup and SpecObject for SpecRelation) that only has their identifier set with a description mentioning that the reference could not be found.

Dear @Helmut-Ortmann, we published a new release on nuget which includes the requested functionality https://www.nuget.org/packages/ReqIFSharp/1.4.0 let us know if you need anything else

Thanks a lot for your fast and competent action.
Best regards,
Helmut


Add ".Values.extraEnvs" to Deployment and set default .extraEnvs to {}

Hi Camunda Team !
I have a issue when deploy and run camunda.  It's here
https://forum.camunda.org/t/your-driver-may-not-support-getautocommit-or-setautocommit/16469/9
and solution are set testOnBorrow="true" . It's can be done with set DB_VALIDATE_ON_BORROW=true to container. And i was added extraEnvs to can be to customize more config over Container Environment, please merge my commit .
Many Thank !

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Nguyen Thanh Trung seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @scila1996 thanks for your contribution, it's pretty useful :+1:
Could you please rebase your change with the latest changes in the main branch?
Also could you please document your change in the chart README file?

This feature has been added via PR no. #13


adds basic vector types

Random thoughts on vector type gen

there are bunch of functions ready to use.  Don't remember from top of my head now. Can discuss tomorrow :)


Add emojis, change README, added '@everyone' before embed

Added emoji images for Ice Golem, Stone Slammer, Bat Spell, BH9 & Hog Glider.
Added additional emoji information to README.md
Left their ID #s blank in index.js since I am not hosting the bot publicly.
Added '@everyone' before the missing attacks embed, as suggested in issue #3.

Will take a look at this sometime this week. Been a while since I updated stuff.

my bad, 2 emojis had weird resolutions, closing to make a new pull


[DO NOT MERGE] Failed test marked with no_xdist passes github action python test

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

CC: @AnandInguva this would block the current approach of #17569


Investigate Go 1.16 test failures

https://travis-ci.org/github/gohugoio/hugo/jobs/740152430
Potentially some breaking upstream changes...?

FYI https://github.com/golang/go/commit/6af088bfc66c13143c9ef46b4cf0805df77a8fbe
it seems to be a breaking change, though it only changes prefix of the error message.

Shouldn't this issue be closed since v0.81.0 uses Go 1.16?

Yes, thanks.


Add missing EventType property

:wave:
Opening this pull request just to add one property that might be useful for some users when they are using this library.
Power select library allows passingeventType prop down to specify types of the handler -> https://github.com/cibernox/ember-power-select/blob/master/addon/components/power-select.hbs#L29
And it looks like this library is missing that.
Please let me know if there is a better way of handling that PR!

just a little bump for @cibernox in case this PR got lost


[Bug][Env] add datasource driver name into env

add datasource driver name into env

Codecov Report

Merging #8100 (c770e78) into dev (e0a99dd) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8100      +/-   ##
============================================
- Coverage     41.46%   41.38%   -0.08%
+ Complexity     3736     3730       -6
============================================
  Files           637      637
  Lines         27141    27141
  Branches       3089     3089
============================================
- Hits          11253    11233      -20
- Misses        14803    14824      +21
+ Partials       1085     1084       -1

Impacted Files
Coverage Δ

...er/master/processor/queue/TaskResponseService.java
50.00% <0.00%> (-18.19%)
:arrow_down:

...rver/master/processor/queue/TaskResponseEvent.java
93.54% <0.00%> (-6.46%)
:arrow_down:

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 e0a99dd...c770e78. Read the comment docs.


Unarchiving of the job to the temporary folder was unsuccessful.

Hi,
I'm having the following problem with some of my portals:
Unarchiving of the job to the temporary folder was unsuccessful.
The file that caused this issue was 4GB, I have seen larger files process without error.
Invalid XML document: C:/Users//AppData/Roaming/Enfocus/Switch Server/temp/275/ScriptElement/1752/365/unarchiveTemp0ARHA\jobTicket\ticket.xml
Error in line 137 of script : Error. File 'C:/Users//AppData/Roaming/Enfocus/Switch Server/temp/275/ScriptElement/1752/365/unarchiveTemp0ARHA\jobTicket\ticket.xml' does not exist
I did see that a similar error was possibly closed a while ago.
Thanks
Pete

I have the same error with a customer now.
The error happens because there is a zip in the ether folder where there is no ticket.xml file. Line 137 which is mentioned in the error is the line where the he tries to read this not existing file. I had to manually remove the job from the ether folder in order to get the flows running again.
If I check my messages for the history of that job, then I also see that the outgoing portal outputted this job earlier with some suffixes which suggest the job was not fully written at that point. Filenames like this:
Intermarche_Bache-190x490_HighRes.pdf.5dreq01.partial
strategie-3355f5.zip.e6ffwtf.partial
The file that triggered the issue here is 500 MB, so it does seem to be related to big jobs.
Domonick made one github contribution in the last year. He seems more active on the Enfocus forums, but there he also mentions that he hasen't used Switch in a while:
[https://forum.enfocus.com/viewtopic.php?f=13&t=2722&p=9022#p9022]
So to me it looks like he is no longer maintaining these Apps since he is no longer working with Switch.
@dominickp, am I correct in this assumption?
For now I think I'll move back to the old system of interconnecting folders and keeping the unique prefixes instead of portals.
Maybe one day if I have sufficient time I'll create a similar project with a different mechanism to know if files are already fully written.

Ok, same problem here, but i've also notice that it also trigger this fail, when you have a long enough directory outcapaciting windows max length.
@joyd42, there is a test built in switch :
hasArrived( fileName : String, stabilitySeconds : Number ) : Boolean [static]
https://www.enfocus.com/manuals/DeveloperGuide/SW/19/home.html#en-us/common/sw/reference/r_file_class.html#checking_for_arrival
I'm working on code review for the both outgoing and incoming script, and will try a fix in a few day

It would be really nice to have a fixed version of the Portals App, I haven't had the time to write and test a good fix myself.
Using hasArrived will most likely avoid the issue, however it will cause an additional delay in processing caused by the time it waits to be 100% sure the file is fully written.
I think an alternative could be:
1 Write a lock file in the ether folder before moving the job file with datasets and metadata file there
2 Move the job file, datasets and metadata file to the ether folder. There is no need to zip them since this will cause an additional delay
3 Remove the lock file again once the files are fully written
4 In the portal which is picking up jobs, check in timerfired for all jobs where there is no lock file next to and bring them in the flow.
This way the delay between the incoming and outgoing portal will be:
time to move jobs to the ether folder + a part of the timerfired interval + time to move jobs to the next flow
Using hasArrived the delay will be:
time to move jobs to the ether folder + a part of the timerfired interval + time that the hasArrived waits +time to move jobs to the next flow
You could even avoid the timerfired interval delay by using webhooks as a signal between portal Apps when jobs are ready to be picked up. (the incoming portal sends a http request when a job is ready in the ether folder to be picked up. the outgoing portal receives the http request as a webhook and knows it can start to process the file. ) Downside of this method is that the webhooks port can be changed in the Switch preferences. So you would have to assume that it is not changed from default, or have a setting in the portal app where users have to fill in the webhook port.
While my suggestions have merits, they also require bigger code changes compared to your suggestion. So please feel free to write your script just the way you want it to :)

Thanks for the feedbacks @joyd42
hasArrived have a typical time of 1second, and it's not much comparing to the 20sc from timeFired.
And it's a lot easier to implement!
When arguing about the topic with colleague, i'll also implement "try/catch" when reading the XML in order to "pass" to the next job if a fail happend.

If you go the hasArrived route, then I think it is best to test with really big files and see if the filesize of the zip constantly increases.
I haven't checked the results of the switch zip function, but I've seen other software where writing a big file doesn't mean that the filesize constantly increases. There can be "breaks" in the increasing of the filesize which are longer then a second.

Hi,
I managed to trigger the bug even on small file (when setting the time fired at 1s, and with a lot of the same dummy file).
So it's seems to be really tied with the "outgoing" trying to access a file not fully written at the moment.
You'll find attached a new version, with some minor tweak :

Better log (called with a function) => readability/easy to improve later
Some minor fixes in test and syntax (space, simple/double quote...)
Added a "check file integrity" function => return true if file exist and file has arrived
Added time parameters for "Has arrived" and "TimeFired" => @joyd42  it will adress your precedent comment

At the moment if an "old" zipped problem file is still present, it will trigger the same error (at line 149 now). I'm working on a test in order to check if the zip is ok, and if not skipp the file with a error logd
Outgoing_V4.zip
)

By the way, feel free to test this one and give your FeedBack.
@dominickp can you update this to the Enfocus Store if the test are successfull?
Or did you mind if i contact Enfocus in order to update?

@DeschampsThomasSobook this is fantastic. I'm so glad that someone is interested in further developing Portals since I've lost my access to Switch.
Do you mind making a pull request for these changes? That way, we can all review the diff and merge it into the code easily if we are happy with the test results.
If you are unfamiliar:

Click the "Fork" button on the top right
On your version of the repo, apply the changes

Make sure to add the .sscript file to ./Packed as well as the .js file (you have to extract the .sscript file with an unarchiver, then copy the contents of Outgoing_v4.js into ./Unpacked)
If there's no breaking changes with v3 Portals, I'd recommend you leave the version at v3 so we can see a comparison. If there are breaking changes, its fine to increment the version number.

Create a Pull Request back to this repo

I work for an Enfocus reseller so I don't have a production Switch of my own where I could test this I'm afraid.
My 2 customers where I used the portals apps and had issues only had them once every 3-4 weeks or so, so a test to see if it still works in their environment will take to long.
Thanks for all your work in this!

@Dominick-Peluso-Bose I'm not yet familiar with git, i'll try :)
@joyd42
You could test this by :
Creating a special flow with the element "dummy job clock"  for a new file each few second
Injecting a "big" file (i tested with 500 mo)
Using portal, with the new outgoing setting property : scan for new file each 1s

I tested this week with method and had ~10k file without error.

@Dominick-Peluso-Bose Ok, i think i used the fork correctly :)

Ok, I have a test running. I'll let you know the result.
One minor point of feedback: there is a typo in the tooltip of the new "Process file only after" setting. "vebore" most likely needs to be "before" :)

In my test I injected the PitStop Server installer, which is slightly over 1 GB. I started the test when I left the office and apparently my Switch shut down after half an hour because there was not enough disk space. The reason the disk is full is that the ether folder is 60 GB. @DeschampsThomasSobook, is it possible that the cleanup mechanism for the ether folder is not working ok yet?

In case you want to check them, here are my logs (I removed stuff from other flows) and my exported flow.
20190903170303.xlsx
New flow.sflow.zip

Oops, bad typo, not my native language,  :), will fix this soon.
Tested your flow on my Switch (Windows) without error, the cleanup of the ether file is well done.
Maybe it's tied to the clock setting,  sending 2 file each 10s (txt, and injected) and the flow take 5s for each in order to check "hasArrived".
That said, it's fixing the initial error, but take way too long for big number of file.
I use portal extensively, ~20k time per day (700 file/day multiply by at least 30 portal jump) and for a 5s check, it represent at lot of downtime.
I'll explore your other idea (lock file) by the end of the week if i have time.

@DeschampsThomasSobook I created a PR from your fork here https://github.com/open-automation/switch-portals/pull/28
Changes you make in your fork will show up here and eventually they can be merged into this project once tested and approved.

Hi everyOne.
Back after a week on another project.
After few test, some odd bug that we could not fix (CallBack function and variable), and efficiency problem, we decided to fully rewrote the API.
We splitted a lot of big function in smaller one, moved some action (rm the zip file) at other places.
Rewrotte some "If/Else" with ternary operator for readability etc etc....
As my coworker said :

we are going to write this as if a psychopath were going to read it

Hi @DeschampsThomasSobook,
I've just started a test flow. Some notes:

I started off with replacing the old script with the new script in my testflow. It looked like it took a while until jobs arrived in the outgoing portal. Then I cleaned up the Ether folder and it was ok. Should the script behave ok when potential new app users upgrade and there are still jobs in the Ether folder from the previous version of the script?
I see a lot of "object" and "undefined" warning messages. Is there a reason for them?
I think the "process files only after" setting needs a default value. Appstore users won't know on their own what to put in there.

I haven't checked the code itself since I'm not really familiar with the inner working of the script. I'll leave that to @Dominick-Peluso-Bose


[Sync Status] If verification fails the first time, then we re-downlaod the whole ledger from scratch and then re-verify again

This is wrong, we should verify first, and then download the ledger

Dupe of #4073


README has some methods which are missing

I tried methods and properties listed in README and some are missing:

hash
prefix

Sorry about that, it is in fact multihash and not hash and prefix is not a method, just a property, updated the readme now https://github.com/ipld/js-cid/commit/fe344b67d22fa71477d52900bca96a7cf8cfdf82

Thanks!


[NCL-7980] Integrate entity attributes into Product pages

@patrikk0123 Let's skip them for now, there is just one rule - if tooltips are available on one page (for example create page), then they have to applied also on all other pages (detail, list, etc).


Large area processing: Dynamically/adaptively start sub-jobs?

In the current "partitioned jobs" implementation all sub-jobs are started at once when the main job is started by the user.
To better take back-end load/availability into account, it might be necessary to start jobs in a more on the fly, adaptive approach

Also to cover here: retrying of jobs (e.g. on same back-end, or on another back-end?)

Could a more generic version of this issue be to simply support the case where a user submits a lot of jobs (e.g. 10k), which then need to be queued adequately?

You mean that the aggregator would then act as an intermediate scheduler to not overload the upstream back-ends? That would indeed be a more generic solution for the original idea of this issue

Exactly, and it would be a useful intermediate step towards supporting large scale processing if I can do splitting myself, and let tracking be handled by the aggregator.

Another thing to consider in this context: in cross-backend processing use cases, some jobs can only be created when other dependency jobs are created or successfully finished, because a valid job id/metadata URL is necesarry in a load_result/load_stac node


Running kustomize against a remote git server fails

Given this is an issue exclusive to this snap package I feel inclined to drop it here instead of the main kustomize repo but feel free to correct me if wrong.
I currently can't use kustomize against remote repos with the following error
miguel@miguel-XPS-15-9560:~$ kustomize build https://github.com/kubernetes-sigs/kustomize.git/examples/helloWorld?ref=v1.0.6
Error: git cmd = '/snap/kustomize/28/usr/bin/git init': exit status 1

Here is the details of my current install on ubuntu 20.04
miguel@miguel-XPS-15-9560:~$ snap info kustomize
name:      kustomize
summary:   Kubernetes native configuration management
publisher: abskmj
store-url: https://snapcraft.io/kustomize
contact:   https://github.com/abskmj/snapcraft-kustomize/issues
license:   unset
description: |
  Kustomize introduces a template-free way to customize application configuration
  that simplifies the use of off-the-shelf applications.
commands:
  - kustomize
snap-id:      Jj5ZG7PyLsqV1ByLBquH4izp4u3i4vaj
tracking:     latest/stable
refresh-date: hoy a las 01:12 CEST
channels:
  latest/stable:    4.1.3 2021-05-21 (28) 33MB -
  latest/candidate: ↑
  latest/beta:      ↑
  latest/edge:      ↑
installed:          4.1.3            (28) 33MB -

Happens to me, as well, on Ubuntu 20.04.4. However, the kustomize executable is fine. It's just that its git can't do the trick.
{ovan}~/trash/kust> ls -al
total 8
drwxr-xr-x  2 dkacar dkacar 4096 Aug 19 11:00 ./
drwxr-xr-x 12 dkacar dkacar 4096 Aug 19 10:29 ../
{ovan}~/trash/kust> type -a kustomize
kustomize is /snap/bin/kustomize

{ovan}~/trash/kust> kustomize build  "https://github.com/kubernetes-sigs/kustomize//examples/multibases/dev/?ref=v1.0.6"
Error: git cmd = '/snap/kustomize/34/usr/bin/git init': exit status 127

{ovan}~/trash/kust> cp /snap/kustomize/current/kustomize .
{ovan}~/trash/kust> ./kustomize build  "https://github.com/kubernetes-sigs/kustomize//examples/multibases/dev/?ref=v1.0.6"
apiVersion: v1
kind: Pod
metadata:
  labels:
    app: myapp
  name: dev-myapp-pod
spec:
  containers:
  - image: nginx:1.7.9
    name: nginx

{ovan}~/trash/kust> type -a git
git is /usr/bin/git
{ovan}~/trash/kust> git --version
git version 2.25.1
{ovan}~/trash/kust> /snap/kustomize/34/usr/bin/git --version
git version 2.25.1

This might have something to do with apparmor refusing to let git from the snap package to operate freely. I don't know how to solve that one.

I got advice to try using --classic installation method, but it's not allowed:
# snap install --classic kustomize
Warning: flag --classic ignored for strictly confined snap kustomize

kustomize 4.5.7 from abskmj installed

So if you could make that work?


GA cookies not removed when user opts-out

What
Checking "Do not use cookies that measure my website use" and saving changes on the GOV.UK cookie settings page does not remove all Google Analytics cookies.

Some work was recently done on this. Based on a brief test on the live site it looks like this issue has been resolved. Accepting all cookies with the cookie banner, then changing consent on the cookies page to reject everything results in the GA cookies being removed from my browser (note: testing this in Chrome the list of cookies doesn't always auto-update, if you click away from the item shown in the screenshot, then back again, it should refresh).
Also just as an assurance even if the GA cookies persist, tracking does not occur if consent has been withdrawn.


Implicit branch creation

Allow implicit branch creation when the specified branch does not exist.
Related Issue.

[x] Add a boolean switch (e.g. create_branch_ok) to the LakeFSFileSystem constructor, with documentation. Code
[x] Add the switch to the LakeFSFileSystem.scope() context manager, same as the other two options. Code
[x] Create the branch via client.branches_api.create_branch before an additive operation if create_branch_ok is True - either by creating if it does not exist (incurs a branch listing, which might be expensive for repos with many branches) OR by branch-ensuring via an unconditional create (needs client-side error handling). Code
[x] Create the branch in the same way for the LakeFSFile if it is opened in write mode, e.g. in init if mode == "wb" OR in _upload_chunk directly before the put_object call). Code

In making this PR, i found two issues.

When using LakeFSFileSystem.put() the prefix lakefs:// in the resource identification string throws an error, e.g. : ValueError: expected path with structure <repo>/<ref>/<resource>, got 'lakefs://quickstart/main/lakesHead.csv'. This is not the case for the get() method. I think we should support it with the prefix. I am unsure as to what causes this deviant behaviour as both methods use the same parse() function

Pushing changes with postcommit = True results in an error when no changes are on the branch. The error:

HTTP response headers: HTTPHeaderDict({'Content-Type': 'application/json', 'X-Content-Type-Options': 'nosniff', 'X-Request-Id': '490cc54c-a8f9-41c4-974b-25e56c9b9aa3', 'Date': 'Tue, 22 Aug 2023 13:54:27 GMT', 'Content-Length': '33'})
HTTP response body: {"message":"commit: no changes"}

I think we should handle that error since we also have a logging statement informing that there was no update to the resource.

Let's add a follow-up issue for a create_branch_ok test (and, more importantly, a "create branch not ok" failure) - otherwise, we're set!

New issue created here


Enhancement: Is Alive Command

Use Case: as a system admin, it would be helpful to have a command that can be used to ping all systems to get a current status.

The flip side is whether this feature is already covered by the heartbeat?  If so, would it make sense to enable an interface to get this?


(InvitingGuests) Incorrect notification when verifying your email address for the first time

Minor stuff but when you verify your email address for the first time the notification goes Email verification ... Your old one is no longer in use. This isn't entirely correct as it's your first email address.

Fixed 5040a94


[REEF-483] Add configuration in allowing other ways to preserve running evaluators aside from DFS

This addressed the issue by

Creating DriverRestartManager interface and implementing YarnDriverRestartManager.
Binding ServiceHandlers for operations that require the DriverRestartManager to record evaluators.

JIRA:
REEF-483

I'll do a pass.

@afchung I have done a pass. A couple of high level notes:

Tang doesn't respect private on constructors. Hence, you can make them private if they are only ever to be invoked via Tang.
Please add JavaDoc on all public methods, at least those defined in interfaces.
Please add JavaDoc to all public classes, interfaces and enums.
DriverRestartManager could be structured like a Driver, with the various event handlers as inner classes. Do you think that would be beneficial to the code structure?
DFSEvaluatorPreserver  needs some refactoring. The code doesn't make use of the fact that it is now allowed to maintain state between invocations of the record...() methods. For one, we could hang on to a FileSystem instance. Or, better yet, a Writer to record these messages. We would decide how to implement the writer (append or append-by-overwrite) once in the constructor of DFSEvaluatorPreserver instead of on every call to the record...() methods.

Also, this PR needs testing on actual YARN clusters before merging. Let's drive it to code-level consensus first, and then ask the mailing list for help with testing.

@markusweimer refactored and rebased. Please have another look. Note that I have not yet opted to open a JIRA for the restart handler issue with regards to recovering only after the restart handler has been called. Thanks!

@afchung I've done a syntax pass, will do a logic one later. Overall feedback:

Please add JavaDocs to all public classes, interfaces and members.
Please add the @DriverSide / @ClientSide / @TaskSide annotations as well as the audience level annotations (@Unstable, @Private, ...)

@markusweimer Fixed issues in comments. Please have another look. Thanks!

@afchung This looks good overall, minus the few comments I have left.

@markusweimer found out there are a few comments I missed to address. Will be pushing another commit.

@markusweimer Addressed all comments. Thanks!

I'll test locally and merge once the build servers and my local test give the go-ahead.

By "locally" I mean on an HDInsight instance. This has the chance of breaking things on YARN.

Tests pass on HDI. I will merge as soon as the build servers give the go-ahead.


feat (Algebra/Order/AddTorsor) : OrderedVAdd, OrderedCancelVAdd

This PR defines vector addition operations that preserve or reflect order, and provides instances relating it to ordered commutative monoids and to covariant/contravariant classes.  In a future PR, we expect to use this to generalize Hahn module structures, for example giving an R((z))-module structure to z^s V((z)) for V an R-module.

Thank you for letting me know.  I will rewrite the classes as prop-valued mixins.


No need to include local type vars

See https://www.dartlang.org/guides/language/effective-dart/usage#consider-omitting-the-types-for-local-variables

I'm 50/50 on this, for short blocks I am fine removing them, for complex ones it may be better for readability.

Agree with @isoos, instead of inferring always if we can, we should only do it where we save space and not reduce readability. Mainly if the right hand side makes it clear for the reader what type the expression has, then it's ok to let the inference do it.
For example, your change includes
final apiHandler = await setupServices(activeConfiguration)

Is it clear what apiHandler is, what kind of arguments it takes? It's probably a function type, but could be anything. a shelf.Handler apiHandler = ... will make this clear.

Closing this out. Clear there is no way to lean here that a linter can help w/


Add addin to website to allow for automatic documentation

i.e. get your addin listed here:
http://cakebuild.net/addins
Have a look at this PR:
cake-build/website#152
This assumes that you have added XML Documentation to your addin, as described here:
http://cakebuild.net/docs/contributing/documentation
Let me know if you have any questions. Thanks!

@Ashthos just wanted to touch base to see if you had thought about doing this. Thanks!


Typescript

This PR begins the process of migrating the Temporal polyfill's sources to TypeScript.
I'm less familiar with the impact of this change on the distributed NPM bundle / changes to package.json and package-lock.json, so a review from someone with that expertise would be greatly appreciated.
In follow-up PRs I will migrate various parts of the source to more canonical representations (e.g. Calendars should probably be based on an abstract class hierarchy).

I'll review this PR tomorrow. One immediate suggestion is to make sure to rebase on top of the most recent polyfill code from proposal-temporal because a few minor PRs have landed there recently. Also, are there any changes that you made that should be back-ported to the old MJS polyfill?

I opened https://github.com/js-temporal/temporal-polyfill/pull/22 to port over the normative changes I saw in the proposal repo, though I'm not entirely sure how they will translate over to TS (unsure how you'd assign those properties to what are currently module exports). Once that is merged then I'll rebase this.
Regarding backporting: there's a fair number of changes that could be backported, but I'd prefer to wait for this to be merged before starting that. I'd probably consider backporting:

a fair number of let->const changes that the linter decided it wanted fixing (though why it didn't seem to trigger on the existing mjs source is confusing to me).
I adjusted how the __debug__ worked to make tsc happy (it doesn't understand the global comments, and I'd prefer not to pollute global typings by re-opening the window/global namespace to include that symbol).
various copies of the bigIntIfAvailable function needed to check for BigInt off of globalThis (instead of doing so implicitly)
renaming the default timezone function (this seems to greatly confuse tsc, but was clearly valid code before)

Everything else are changes that are in TypeScript type space (or are adjustments to use tsc modules instead of Node mjs), so I don't think can be backported unless we moved that impl to TS source as well (which could be done if people felt it was useful?).

I'm afraid backporting anything that wasn't clearly a bug fix to the old impl would end up being contentious within TC39.

Ah right. In that case, I don't think there are any bug-fix-like changes to backport.

Here's another, more minor sourcemap isssue: I use a webpack plugin to detect differences between code stored inline in the  sourcemap vs original source files. My plugin complained about the two lines below. Any idea what's going on here?
/Users/justingrant/Documents/hdev/h3/web/node_modules/tslib/tslib.es6.js doesn't match sourcemap, starting at line 20, col 48
Original Line 19:         ({ __proto__: [] } instanceof Array && function (d, b) { d.__proto__ = b; }) ||
     Map Line 19:         ({ __proto__: [] } instanceof Array && function (d, b) { d.__proto__ = b; }) ||
Original Line 20:         function (d, b) { for (var p in b) if (b.hasOwnProperty(p)) d[p] = b[p]; };
     Map Line 20:         function (d, b) { for (var p in b) if (Object.prototype.hasOwnProperty.call(b, p)) d[p] = b[p]; };
Original Line 21:     return extendStatics(d, b);
     Map Line 21:     return extendStatics(d, b);

/Users/justingrant/Documents/hdev/h3/web/node_modules/is-date-object/index.js doesn't match sourcemap, starting at line 15, col 54
Original Line 14: var dateClass = '[object Date]';
     Map Line 14: var dateClass = '[object Date]';
Original Line 15: var hasToStringTag = typeof Symbol === 'function' && !!Symbol.toStringTag;
     Map Line 15: var hasToStringTag = typeof Symbol === 'function' && typeof Symbol.toStringTag === 'symbol';
Original Line 16:
     Map Line 16:

A final sourcemap problem is related to the fact that our polyfill when installed in a project will be 2 levels under node_modules (first @js-temporal and seond polyfill).  So either the sourcemap should be rewritten to add an extra ../ for all references to node_modules code, or we could try rewriting the path to ~/ in place of ../node_modules.  The ~/ is AFAICT an undocumented synonym for "the app's node_modules folder", although I don't know if it has universal support. See https://github.com/justingrant/amplify-js/blob/768ad63b2689d377eddafe3802c50a31f509e5ab/packages/aws-amplify/webpack-utils.js#L56-L60 for an example of this rewriting that I did an an AWS PR a while ago.
The other line of investigation on this issue is why we're including code in our sourcemaps from our dependency libraries. Does this mean that rollup and/or TS includes es-abstract, big-integer, and their transitive dependencies in our bundle?  That's probably not what we want. Is there a rollup and/or TS setting to prevent that?

Does this mean that rollup and/or TS includes es-abstract, big-integer, and their transitive dependencies in our bundle?

BTW, here's a relatively modern TS library and the rollup config they use: https://github.com/mongodb/js-bson/blob/master/rollup.config.js.
If you npm i bson and look at the sourcemaps (and likely the bundled JS files too) you'll see that bson.bundle.js.map contains "../node_modules/buffer/index.js" (it bundles its dependencies) but bson.esm.js.map does not bundle dependencies. The latter output file comes from a rollup config with format: 'es'that also uses theexternal` config option which presumably excludes dependencies.
Would a similar ESM config be worth adding to this polyfill?

I've pushed another commit that I believe addresses your comments - it's heavily based on the commit you sent me with some small adjustments. In particular, we don't need to have two tsconfig files as the rollup TypeScript plugin can use the package tsconfig as a base and seems to adjust various settings automatically to work with the rest of the build tools.
I'll wait until #25 is merged before rebasing and squashing / cleaning up the commits.

@12wrigja - Actually, one more thing: I did some research about bundling dependencies, and I think CJS should not bundle its dependencies. I think that this will also solve any sourcemap paths issues for anyone using non-ESM bundles in node.
The change would be simple:
export default [
  {
    input,
    external,
    output: [
      outputEntry(pkg.module, 'es'),
      outputEntry(pkg.main, 'cjs')
    ],
    plugins
  },
  {
    input,
    output: [
      outputEntry(pkg.browser, 'umd'),
      outputEntry(pkg.module.replace('.esm', '.browser.esm'), 'es')
    ],
    plugins
  }
];

What do you think?

@justingrant Done. Pity about the linter formatting though.

I've rebased this on top of main and squashed in the adjustments to the rollup config. Is there anything left to resolve, or can we merge this now?


⚠ Bofrost.lu has degraded performance

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

HTTP code: 200
Response time: 1592 ms

Resolved: Bofrost.lu performance has improved in 4d831a0 after 15 minutes.


Fix aligned memory allocation

Closes #1192

Reworked sol::detail::aligned_space_for<Ts...>

If no type is over-aligned (alignof(Ts) <= alignof(std::max_align_t) for all Ts...), calculate maximum possible required size assuming that allocator can return absolutely any address
Otherwise assume that every single type in pack has initial address 0x1 (the worst alignment)
Make it constexpr

Always do only one allocation with required memory size. "aligned allocation failed" error should never happen anymore
Remove unused arguments/variables (it was probably a legacy code)

This actually does look good to me, albeit the tests will always fail for one or two platforms because we're running into a hard regression thanks to lua_error thrashing the stack in some cases of optimized code. Can't escape it, so the CI will always go red.
I'm gonna run this locally and try to fix things up, so forgive me if merging this takes a little bit!!!

Lllllooooooooooooooooooooooks good to me and makes things much faster / more consistent when alignment gets weird. Thanks so much for this!
I was originally going to promote these functions out of the detail namespace since they're used but I'll do that later, since that's technically a breaking change.

Ssssskskskskdskdjakdasjk I closed it and almost panicked because the merge button disappeared
I AM A PROFESSIONAL I PROMISE!!


EllipsoidDomain is_inside method

Describe the bug
The is_inside method of the EllipsoidDomain class doesn't seem to work.
To Reproduce
ellipsoid1 = EllipsoidDomain({'x': [0, 1], 'y': [0, 1]})
pt_1 = LabelTensor(torch.tensor([[0.5, 0.5]]), ['x', 'y'])
print(ellipsoid1.is_inside(pt_1))

Expected behavior
True
Output
False
Additional context
Add any other context about the problem here.

Solving in #118

Solved in #118 .


can graphite retrieving metrics from remote and local carbon-daemon

Hi !
I've been working on the graphite clustering recently, and i have tried the CLUSTER_SERVER config which used to collecting metrics from remote graphite hosts.
However, i started wondering, instead of having a dedicated graphite-web host querying metrics from two carbon-daemons, can i host graphite web on local along with carbon-daemon running, and retrieve metrics not only just from local carbon-daemon but also remote graphite hosts(carbon-daemon running) to combine metrics and to present result?
That way i don't need to maintaining one extra graphite-web layer.
So, my question is, can i use CARBONLINK to listen on remote carbons? ( such as [<IP_ADDRESS>:7002 , <IP_ADDRESS>:7002] ) Or, would there be any other way to achieve that?
Your help is greatly appreciated! 👍
Best,
David

Hello @caldavid,
Unfortunately, it will probably not work. Carbonlink was not designed as clustering protocol. But OTOH you don't need a dedicated rendering server, you can use only two graphite-web instances and point them to each other.

Hi Deniszh,
Thanks for your reply.
That said, I could use CLUSTER_SERVER config to let two graphite-web servers talk to each other? Like this way?
CLUSTER_SERVERS=['<IP_ADDRESS>:80', 'xxx.xxx.xxx.xxx:80']
Best,
David

Yes, indeed. You can probably remove local part completely, so, CLUSTER_SERVERS=['xxx.xxx.xxx.xxx:80'] - local cluster should use carbonlink and direct access to whisper files.

So the CLUSTER_SERVERS and CARBONLINK can be used together? I tried that, but didn't seem worked.

CARBONLINK is for connecting graphite-web to carbon cache, usually localhost and port 700x.
CLUSTER_SERVERS is for clustering graphite-web instances, protocol is http - so they are not interchangeable but ofc can be used together.

gotcha! Thanks a lot!  👍


Xeltalliv/simple3D: Feature update v1.1.0

Adds support for SharkPool's "Layer Control" extension.
Fixes bug where it was always using centroid interpolation for v_color.
Fixes [set clear color R:() G:() B:() A:()] to take premultiplied alpha into a count.
Adds 2 new blocks.
Adds new blending mode for drawing behind what is already drawn.
Adds ability to measure text's horizontal advance distance, which is different from text's bounding box width.
Adds Math.ceil to text borders to prevent text from jumping around when smoothly scaling border.
Adds PenguinMod and Snail IDE extension removal support.
Clarifies which parts are MPL-2.0 and which are BSD-3-Clause.
Changes default near plane to be further to reduce Z-fighting.

wowie

have some opinions about how other people decided to do extension removal but ill leave it in because i dont have a better thing yet


Spike - Explore "Flask Admin" library to enable Instance Admin Portal

Overview
Instance admin users should be able to modify certain entities in the back end.

Overview
Flask Admin can be configured in the existing ReportAPI which will enable the admin interface to be accessed in /admin url.
Need to configure keycloak and enable access to the users with Instance Admin role.
It is possible to enable admin ui for selected database models


feat: AI playbook actions

resolves: https://github.com/flanksource/mission-control-registry/issues/361

Can we break this out it into its own chart ?


Issue with FASTA reference for arrow

Hi,
I'm running iterations of Arrow and Pilon, and I'm running into some sort of problem with my FASTA or .fai reference from the output of Pilon (which I have modified to restore some large INDELs corrected by Pilon). I've done simple checks to validate the FASTA file and all looks ok. Any help would be much appreciated!
Command is:
arrow -j16 c_incerta_canu_v2.pbalign_v2.bam -r ../pilon_1/c_incerta.canu_v2.arrow_pilon_v1.fa -o c_incerta.canu_v2.arrow_v2.fa -o c_incerta.canu_v2.arrow_v2.fq -o c_incerta.canu_v2.arrow_v2.variants.gff
Error message is:
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcommand/cli/core.py", line 138, in _pacbio_main_runner
    return_code = exe_main_func(*args, **kwargs)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/GenomicConsensus/main.py", line 340, in args_runner
    return tr.main()
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/GenomicConsensus/main.py", line 267, in main
    self._loadReference(peekFile)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/GenomicConsensus/main.py", line 125, in _loadReference
    reference.loadFromFile(options.referenceFilename, alnFile)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/GenomicConsensus/reference.py", line 97, in loadFromFile
    f = ReferenceSet(filename_)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4336, in __init__
    super(ReferenceSet, self).__init__(*files, **kwargs)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 3887, in __init__
    super(ContigSet, self).__init__(*files, **kwargs)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 450, in __init__
    self.updateCounts()
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4092, in updateCounts
    if not self.isIndexed:
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4241, in isIndexed
    lambda x: isinstance(x, IndexedFastaReader))
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 1832, in _pollResources
    return [func(resource) for resource in self.resourceReaders()]
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4200, in resourceReaders
    self._openFiles()
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4155, in _openFiles
    resource = self._openFile(urlparse(extRes.resourceId).path)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4170, in _openFile
    resource = IndexedFastaReader(location)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/FastaIO.py", line 410, in __init__
    self.fai = loadFastaIndex(self.faiFilename, self.view)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/FastaIO.py", line 267, in loadFastaIndex
    assert (header_[0] == ">" and header_[-1] == "\n")
AssertionError
[ERROR]
Traceback (most recent call last):
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcommand/cli/core.py", line 138, in _pacbio_main_runner
    return_code = exe_main_func(*args, **kwargs)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/GenomicConsensus/main.py", line 340, in args_runner
    return tr.main()
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/GenomicConsensus/main.py", line 267, in main
    self._loadReference(peekFile)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/GenomicConsensus/main.py", line 125, in _loadReference
    reference.loadFromFile(options.referenceFilename, alnFile)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/GenomicConsensus/reference.py", line 97, in loadFromFile
    f = ReferenceSet(filename_)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4336, in __init__
    super(ReferenceSet, self).__init__(*files, **kwargs)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 3887, in __init__
    super(ContigSet, self).__init__(*files, **kwargs)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 450, in __init__
    self.updateCounts()
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4092, in updateCounts
    if not self.isIndexed:
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4241, in isIndexed
    lambda x: isinstance(x, IndexedFastaReader))
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 1832, in _pollResources
    return [func(resource) for resource in self.resourceReaders()]
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4200, in resourceReaders
    self._openFiles()
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4155, in _openFiles
    resource = self._openFile(urlparse(extRes.resourceId).path)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/dataset/DataSetIO.py", line 4170, in _openFile
    resource = IndexedFastaReader(location)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/FastaIO.py", line 410, in __init__
    self.fai = loadFastaIndex(self.faiFilename, self.view)
  File "/home/craigror/anaconda2/lib/python2.7/site-packages/pbcore/io/FastaIO.py", line 267, in loadFastaIndex
    assert (header_[0] == ">" and header_[-1] == "\n")
AssertionError```

Could it be that your reference fasta file is not valid? The error message
assert (header_[0] == ">" and header_[-1] == "\n")

tells that there is a line starting with > but is not followed by another line with sequence data.

Thanks for clarifying the error message, I had a bug in my script that was manipulating the Pilon output that was occasionally duplicating a new line, I should've caught that. Arrow is now working with my file, thanks again.


AFNetworking still use non watch supported usages in 2.6.1

Hi I saw the issue https://github.com/AFNetworking/AFNetworking/issues/2849 with use of SystemConfiguration/SystemConfiguration.h in Reachability Manager, and I saw you solved it with
#if !TARGET_OS_WATCH
But there are still more other non watch supported usages in other classes like another use of SystemConfiguration/SystemConfiguration.h in AFHTTPRequestOperationManager.h or use of UIApplication in AFURLConnectionOperation.m

NSURLConnection is not supported on watchOS, so those files are not included in the watch target. You can see in the podspec watch support is not available for those classes, therefore there is no need to mark up that file for watch support


ERR_GET_FUNC should be removed or marked deprecated

Handling of encrypted private keys in mod_ssl was broken by the change to remove function codes, since it checked for a specific function returning errors in:
https://github.com/apache/httpd/blob/trunk/modules/ssl/ssl_engine_init.c#L1484
IMO if function codes are removed it would be preferable to remove ERR_GET_FUNC completely - the ABI is changed here, better to have it break at compile time than at runtime in an obscure way.  Or at least ERR_GET_FUNC should be marked deprecated so we get a gcc warning, which it isn't currently?

Unfortunately it is too late to add deprecation markers after 3.0-beta1 has been released.

Thinking about this more, this might cause quite serious breakage in apps so perhaps adding the deprecation marker could be treated as a bug fix.

OTC will have to decide.

@notroj would the bug be avoided if ERR_GET_FUNC() returned -1 for example instead of 0?

In this case it would yes, the problem was both that ERR_GET_FUNC() always returned 0 and all the _F_ constants were also defined to 0.

Unfortunately adding the deprecation marker would require to change the way the deprecation marker is defined as it cannot be otherwise applied to static inline functions.

.... as it cannot be otherwise applied to static inline functions.

Oh‽

Oh‽

We define it to extern when OPENSSL_SUPPRESS_DEPRECATED is defined or for compilers that do not support deprecation markers. And extern static is invalid.

So make the function a regular function, not static inline.  The extra cost for a deprecated function is not a big deal.  Breaking mod_ssl however, is a big deal.

OTC have voted for removal.


BasketMiddleware.merge_baskets and checkout process

Under some circumstances, baskets are merged together, where it would actually make sense for a more recent basket to replace older baskets.
Here's an example for when it is (almost certainly) inappropriate to merge two baskets together:

A logged in user adds an item to their basket
The user logs out (or uses a different computer)
User adds a different item to their basket
Basket now shows only the 2nd item
User proceeds to checkout (Obviously if a user presses the checkout button at this point, they intend to checkout with the basket in its current state.)
User is prompted to log in
User logs in and is redirected to first step of checkout process
BasketMiddleware has merged the two baskets together, so now there is a basket with both items in it.
User fills out shipping information, billing information, etc.
Only then is the user presented with this new basket that has (surprise) both items in it

Instead of silently merging the two carts, maybe the user should be notified that there are two carts, and be prompted to pick one or somehow at least draw attention to the fact that they have been merged. If the first cart is quite old, it might be safe to discard the first cart silently.
Some discussion elsewhere:
http://stackoverflow.com/questions/11942764/is-it-appropriate-to-merge-guest-user-cart-items-with-items-in-the-cart-when-a-u
http://ux.stackexchange.com/questions/52785/should-shopping-carts-from-different-sessions-be-merged-or-not

I've had the same issue on a project I was working on. We chose to always use cookie-based baskets, which also solved the same problem.
See also #1317

@mvantellingen Can you provide me with some more information regarding the cookie-based baskets? I'm having a hard time finding info about it in the docs.
Thanks.

Fixed by PR #1871.


cache functions/transformations used

This is to avoid performing multiple transformations after one another, which can cause some faulty statements and states

Perhaps MFL can be used in someway to aid in this cacheing by extracting the MFL string from the model in question


CR Leader List: have it show unique CRs in last 14 days

The Leader list was not that helpful. It was showing the total
CR signature count across all pulls in pulldasher's memory (not the DB).
This wasn't that useful of a metric because that was an abritrary number
of pulls and 7 CRs on one pull isn't the same as 1 CR on 7 pulls.
Now:

Only include open pulls and those merged less than 14 days ago.
Only include signatures made in the last 14 days
Count only the number of pulls each user has CRed (not the number of
signatures)
Avatars show username on hover

Future work:

Change pulldasher to load even closed pulls from the DB so
these numbers stay the same on restart.

Right now it only loads open pulls on restart

Maybe: drop list and only show how you compare to mean or median of the
group.

This is closer to your spec @davidrans

Current:

Dev:

The number of inactive CR's are now being included, is this intended?

(in progress) 🟢
Avatars show username on hover ☑
Count # of unique CR stamps ☑
Limit signatures to 14 days ☑
Count for only open pulls & pulls merged in the past 14 days*
_I can say for sure that open pulls are being counted, but it doesn't appear that merged pull signatures are

No longer shown in CR column:

So, I went for it. This now keeps recently merged pulls in memory, sends them to the client, and culls too-old ones every hour.
Looking good!

Thanks @ardelato for looking at this and pointing out an improvement!

Count with avatars appears correctly.
Name shows on hover.
Shows count of unique CR's.
Counts stamps of open and up to last fourteen days.

QA 😺

CR :goat: through 3464c78
Cool!


CMDistributionPoint:  Adding the capability to create standard Distribution Points

Pull Request (PR) description

This Pull Request (PR) fixes the following issues
Task list

[ ] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ x] Resource documentation added/updated in README.md.
[X ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ X] Comment-based help added/updated.
[ X] Localization strings added/updated in all localization files as appropriate.
[X ] Examples appropriately added/updated.
[X ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[ X] New/changed code adheres to DSC Community Style Guidelines.

This change is

Tested and everything seems to work, including boundary group issues being fixed. I appreciate you anticipating someone doing a null array of boundary groups with match when someone wants to remove all.


lefty and leftx not working on Mac

Do lefty and leftx work on Mac? I keep getting taken out of Play mode when I hit A or S (which each activate the tool associated with the default keyboard shortcut) and W and D do nothing.
I am trying to follow this tutorial. https://youtu.be/U4eXsz1fi0M?t=138

Yes, keyboard controls don't work on Mac OS X at the moment.

Is there anything you can think of that might allow me to control a Servo Joint? That's pretty critical for my project.

You can use a slider controller instead. Open UI, select the servo joint, and click on the "generate slider controller" button and adjust the values afterwards.

Hi, Richard,
Keyboard input, asside from events, should work now with 0.9.5.
Anton


Can't find awsscv in pip

Hi,
The lambda common layers reference a library that can't be found in pip or Conda. Please could you confirm if this is a private package?
https://github.com/amazon-connect/amazon-connect-salesforce-scv/blob/d8eedba2205d32fbd7ca21455a32914214f11d28/Common/AWSSCV-CommonLayers/Code/python-layer/python/lib/python3.8/site-packages/requirements.txt#L1

It is a private package developed specifically for these use cases.