Muted video plays with sound when AirPlay connected

Bug
Platform
Which player are you experiencing the problem on:

iOS

Environment info
React native info output:
System:
    OS: macOS 13.2
    CPU: (12) arm64 Apple M2 Pro
    Memory: 1.06 GB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 19.5.0 - /var/folders/f0/m44tfz1x2fq_ctwzwr91hm_40000gn/T/yarn--1676371817037-0.7719960446779974/node
    Yarn: 1.22.19 - /var/folders/f0/m44tfz1x2fq_ctwzwr91hm_40000gn/T/yarn--1676371817037-0.7719960446779974/yarn
    npm: 9.3.1 - /opt/homebrew/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
    Android SDK: Not Found
  IDEs:
    Android Studio: 2022.1 AI-221.6008.13.2211.9477386
    Xcode: 14.2/14C18 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.18 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.2.0 => 18.2.0
    react-native: 0.71.2 => 0.71.2
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Library version: 5.2.1 and 6.0.0-alpha.5
Steps To Reproduce

Connect to the AirPlay video receiver
Switch videos by tapping on the screen

Expected behavior
Video respects muted prop and plays on AirPlay without sound
Reproducible sample code
import React, {useState, useCallback} from 'react';
import {Pressable} from 'react-native';

import Video from 'react-native-video';

const videos = [
  'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4',
  'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerBlazes.mp4',
  'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerEscapes.mp4',
  'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerFun.mp4',
  'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/SubaruOutbackOnStreetAndDirt.mp4',
];

const VideoTest = () => {
  const [videoIndex, setVideoIndex] = useState(0);

  const onPress = useCallback(() => {
    if (videoIndex === videos.length - 1) {
      setVideoIndex(0);
    } else {
      setVideoIndex(videoIndex + 1);
    }
  }, [videoIndex]);

  return (
    <Pressable style={{flex: 1}} onPress={onPress}>
      <Video
        muted
        volume={0}
        allowsExternalPlayback
        source={{uri: videos[videoIndex]}}
        style={{flex: 1}}
      />
    </Pressable>
  );
};

export default VideoTest;

@misha-belokon  Change the 'muted' property after you switch to external player. 'onExternalPlaybackChange' callback function could help.

@vargajacint thanks for your reply. Your suggestion works, but only for one video source. When the source is changing without interrupting AirPlay the sound is not muted again. I'll try to solve it using unmute/mute on every source change next week, hopefully, it will work.

This will help https://www.youtube.com/watch?v=vVI7ZAZq5e0

@misha-belokon did you manage to solve this?

@dianamelga Yes, I did. It's a tricky solution, but it works. It requires to fix 2 issues.

Mute state is not respected if video starts on mute when AirPlay was already connected. The fix is on JS side, works by setting mute state to false and back to true.

import React, { useState, useCallback, useEffect } from 'react';
import { Pressable } from 'react-native';

import Video from 'react-native-video';
import { useAirplayConnectivity } from 'react-airplay';

import MuteButton from './MuteButton';
import PauseButton from './PauseButton';

const videos = [
    'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4',
    'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerBlazes.mp4',
    'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerEscapes.mp4',
    'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ForBiggerFun.mp4',
    'https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/SubaruOutbackOnStreetAndDirt.mp4',
];

const VideoTest = () => {
    const [videoIndex, setVideoIndex] = useState(0);
    const [muted, setMuted] = useState(true);
    const [paused, setPaused] = useState(true);
    const airplayConnected = useAirplayConnectivity();

    const onPress = useCallback(() => {
        if (videoIndex === videos.length - 1) {
            setVideoIndex(0);
        } else {
            setVideoIndex(videoIndex + 1);
        }
    }, [videoIndex]);

    useEffect(() => {
        // fix of mute state on initial casting start
        if (airplayConnected && muted && !paused) {
            setMuted(false);
            setTimeout(() => {
                setMuted(true);
            });
        }
    }, [muted, airplayConnected, paused]);

    return (
        <Pressable style={{ flex: 1 }} onPress={onPress}>
            <Video
                muted={muted}
                volume={0}
                paused={paused}
                allowsExternalPlayback
                source={{ uri: videos[videoIndex] }}
                style={{ flex: 1 }}
            />
            <MuteButton onChange={setMuted} muted={muted} />
            <PauseButton onChange={setPaused} paused={paused} />
        </Pressable>
    );
};

export default VideoTest;

Mute state is not respected when changing source prop. The fix I made is on the native side, you can use it with patch-package

diff --git a/node_modules/react-native-video/ios/Video/RCTVideo.m b/node_modules/react-native-video/ios/Video/RCTVideo.m
index a757c08..686cce2 100644
--- a/node_modules/react-native-video/ios/Video/RCTVideo.m
+++ b/node_modules/react-native-video/ios/Video/RCTVideo.m
@@ -696,6 +696,12 @@ - (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(N
         _videoLoadStarted = NO;

         [self attachListeners];
+        // hacky solution to fix mute state when AirPlay is active
+        // start
+        [_player setVolume:1];
+        [_player setMuted:NO];
+        // end
+
         [self applyModifiers];
       } else if (_playerItem.status == AVPlayerItemStatusFailed && self.onVideoError) {
         self.onVideoError(@{@"error": @{@"code": [NSNumber numberWithInteger: _playerItem.error.code],
@@ -738,6 +744,12 @@ - (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(N
         self.onVideoExternalPlaybackChange(@{@"isExternalPlaybackActive": [NSNumber numberWithBool:_player.isExternalPlaybackActive],
                                              @"target": self.reactTag});
       }
+      // hacky solution to fix mute state when AirPlay is active
+      // start
+      [_player setVolume:1];
+      [_player setMuted:NO];
+      [self applyModifiers];
+      // end
     }
   } else if (object == _playerViewController.contentOverlayView) {
       // when controls==true, this is a hack to reset the rootview when rotation happens in fullscreen


Unable to upgrade to latest Tomcat - Spring framework and RestEasy don't support Tomcat 10

Describe the bug
When attempting to upgrade Tomcat from 9.0.46 to 10.0.6 for #2685, I quickly found that in order to run this upgrade we would also need to upgrade Spring framework and RESTEasy - they use the javax.servlet classpath, which is incompatible in Tomcat 10 which migrates to use jakarta.servlet.
However, there are as of yet no released versions of either library that use the jakarta.servlet classpath. Because of this, it is impossible to update Tomcat to 10 without breaking RESTEasy and Spring framework, until both libraries do work to support Tomcat 10.
To Reproduce
Steps to reproduce the behavior:

Change TomcatVersion in equellaserver/build.sbt to 10.0.6.
Rename all in-code instances of javax.servlet to jakarta.servlet as per the Apache Tomcat 9 - 10 Migration guide.
Rebuild openEQUELLA. Note errors wherever Spring framework is used (E.G line 90 on RemoteInterceptor.java, where jakarta.servlet.HttpRequest and jakarta.servlet.HttpResponse are used and org.springframework.remoting.httpinvoker.HttpInvokerServiceExporter expects javax.servlet.HttpResponse and javax.servlet.HttpRequest).

Expected behavior
We need to wait until stable versions of org.springframework and resteasy are released that have support for jakarta.servlet classpaths, and then we can upgrade both of those and Tomcat.
See also: https://github.com/spring-projects/spring-framework/issues/25354
Platform:

OpenEquella Version: 2021.1+
OS: Agnostic
Browser: Agnostic

Additional context
Another slight problem with upgrading Tomcat is that the JDBCStore which has been deprecated for multiple versions is finally removed, and openEQUELLA relies on this JDBCStore in the TomcatServiceImpl. We will need to migrate to use the new (JNDI) format.

From the above we're currently waiting on two releases:

Spring 6.x+
RESTEasy 5.x+

Current situation is as follows.
Spring
Not yet available, the latest is still 5.3.14. When this does update, several updates will be required at once but at least we are currently running 5.3.14 and renovate is actively keeping us up to date.
Note: We don't use spring-core, we only use spring-{aop, context, context-support, jcl, jdbc, orm, tx, web}.
RESTEasy
Version 5 of RESTEasy is now available and includes use of the jakarta namespace. So at least this is ready, but I guess we also need to wait for spring.
That said though, we're also on a rather old version at RESTEasy with resteasy-jaxrs at 3.15.3.Final so it would be worthwhile at least getting this closer (say 4.7.x) in preparation.


Validation front end

#1381
Marks the cells when they're invalid
Changelog:

Slightly changed the format of validation error from the backend, when displaying errors about duplicates
Validation Message accordion at the top of the schedules
Marks the cells as red, if they're invalid
Changes the message accordion to green, when there are no errors
Highlights duplicate rows
Changes icons for row numbers depending on the row status

 Validation | Schedule A |5| v1.4.25

 Validation | Schedule B |16| v1.4.25


carrot just exits

I'm trying to run carrot + django + postgres + rabbitmq in a docker-compose, but carrot just bails out successfully with exit code 0...  My docker-compose.yml file looks like this
version: '2'

services:
  rabbit:
    image: 'rabbitmq:3.7-alpine'
    expose:
      - "5672"

  postgres:
    image: 'postgres:9.6'
    volumes:
      - postgres_data:/var/lib/postgresql/data

  django:
    restart: always
    build: ./
    depends_on:
      - postgres
      - rabbit
    expose:
      - "8000"
    ports:
      - "<IP_ADDRESS>:8000:8000"

  carrot:
    build: ./
    depends_on:
      - django
      - rabbit
    command:
      bash -c "sleep 8" && python manage.py carrot --traceback

volumes:
  postgres_data:

The container does not seem to produce any output whatsoever. There is no file in /var/log/carrot.log either. Please advice.
I fixed up another simpler django-leek yesterday, but today I saw you had fixed some stuff with the migrations, etc, so I thought I'd give carrot another try as I like the monitor and so on. However, I cannot seem to get it running at all.

Give me a few hours, will get a working docker container up for you

Sweet, thanks! I picked up the old django-queue project and hotted it up as django-leekso our needs are covered for now, so no stres. :-) But would gladly switch to carrot. 👍

This turned out to be a bit trickier than I had anticipated. In our setup, we have a central Rabbit server, with a number of standalone dockerized django apps that connect to it, and this is very easy to implement. Actually running RabbitMQ inside the same container as the Django app caused a few more issues. However, I've managed to get a working example up here: django-carrot-docker
Key things to note:

The default config for carrot assumes that the RabbitMQ instance runs at localhost, which doesn't work with a dockerized setup. You need to send the hostname argument to the rabbit container (e.g. rabbit), then update your CARROT settings as follows:

DEFAULT_BROKER = 'amqp://guest:guest@rabbit:5672/'

CARROT = {
    'default_broker': DEFAULT_BROKER,
    'queues': [
        {
            'name': 'default',
            'host': DEFAULT_BROKER
        }
    ]
}

Running rabbit in its own container causes issues accessing the database - the consumer raisesObjectDoesNotExist exceptions for every message it tries to consume that wasn't already in the queue when the consumer started. The solution to this is to launch django and carrot in the same container, using a bash script.

Anyway, the example config seems to work, and provides an empty app with a dummy task for testing


CSAF support short term hack

This is the same as #93 but includes the short-term hack.

Thank you for submitting this. Sorry for taking so long to look at it, I will review it tomorrow and then merge it!

@joker314 Is that PR ok or do I need to change anything else?

Thank you! I've merged so it's now deployed at https://securitytxt.org

@joker314 Thank you so much!


How to get expected pA length of all genes of each 10 grouped neighboring cells?

I'm wondering how to get expected pA length of all genes of each 10 grouped neighboring cells just like Figure 6D in your article? I haven't found a tutorial in git-hub.
Thanks!!

Hi,
Thanks for your interest on our work, this function was not included in our package, and please refer to the Expected pA length section in the method.
Ran


Better Integration with Greenshot

When I take a screenshot using Greenshot, the copy to clipboard and paste feature does not work with BookStack.  I have to save the file and then upload.  It would fix a very minor inconvenience to fix that. Pasting from other sources works just fine.

Hi @Ethanb00, I've just tested this and could not replicate the issue. I directly copied to clipboard and also opened up a screenshot in the greenshot image editor and copied from there and both pasted fine.
I'm on windows10 & using chrome. Can you confirm your OS and browser?

Thanks for getting back to me @ssddanbrown,
I should have clarified. I was using the 'Copy to Clipboard' option, and then trying to paste into the editor.
I haven't had other problems with copying and pasting images directly into the editor from other sources (like visio).

Hi @Ethanb00, Thanks for confirming those details. Yeah, I was also using the 'Copy to Clipboard' option. Possibly only an issue on Windows 7 then? I might have to download a win7 VM to test this.

Before you go through the trouble, let me try some test on my end. I'll let you know tomorrow what I find.
Thanks!

I'm not having issues with this anymore. I think a greenshot update fixed the problem. thanks for looking into it.

@Ethanb00 Awesome, Thanks for confirming.


ReactDemo doesnt add new items to the commentList

I created a new proyect, followed tutorial step by stem it didnt work , then downloaded the demo code, didnt work oob had to copy everything to a new project but it didnt work items added using the button dont get listed, the mvc AddComment action is not even called from the client side, anyone having the same issue?
thanks.

yes thats exactly whats happening, how do I fix it? should I remove Remarkable completely?
thanks!

any clue why the demos dont work? I need an example how to add third party js into the project,  thanks

I have a fix for this. I will create a PR. Add an additional function in tutorial.jsx
function createRemarkable() {     var remarkable = (("undefined" != typeof global) && (global.Remarkable)) ? global.Remarkable : window.Remarkable;    return new remarkable(); }
then use that to create the instance e.g.
var md = createRemarkable(); var rawMarkup = md.render(this.props.children.toString());

great! is working now, thank you.


Test $document decalaration in CapabilityStatement

Summary
Add $document test in CapabilityStatement
New behavior
Code changes
Testing guidance

Changes moved to ips-mustsupport branch


5/12 Publish

@Taojunshen,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-autogen/account.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/ad/app.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/ad/sp.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/disk.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/snapshot.pycliyml
:white_check_mark:Succeeded
View

docs-ref-autogen/vm.pycliyml
:white_check_mark:Succeeded
View

docs-ref-conceptual/interactive-azure-cli.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


All available methods for copying out a large string spanning multiple lines do not work.

Version and Platform (required):

Binary Ninja Version: 3.2.3870
OS: macOS
CPU Architecture: x86_64

Bug Description:
All available methods for copying out a multiline string are broken.
Steps To Reproduce:

Load the attached binary and navigate to the large/only string in the binary.
Attempt all available methods of copying the string out (including base64 / other transforms)

Most of these only copy the first two bytes, which does not even line up with where the string wraps.
string_copy.zip

Closing in favor of this issue: #2366


Autoplay audio

Question
Can I define some how that audio the bot "send" will  play automatically without needs to click on the "play" button?
Thanks!

Hi,
No, this can't be handled by bot.

Baga.. Thanks anyway 👍


Updated Extension to fix compiler errors with latest swift

Ran into some errors with longs and malloc, this cleaned up branch seems to work, hope it helps!

Thanks done.


lang/ruby: add featurep for rspec and minitest

The ruby's testing frameworks should be optional. When both are installed rspec's keybindings take precedence over minitest's.

I want to avoid adding more flags than necessary, especially on a per-package basis. Instead, I think the more appropriate solution would be to advise rspec-enable-appropriate-mode to no-op if {project-root}/spec doesn't exist. Can you make this change? If not, I'll look into it sometime within the next week or two.

Could it be something like this?
(advice-add 'rspec-enable-appropriate-mode :around
            (lambda (orig-fun)
              (when (file-exists-p (expand-file-name "spec/" (rspec-project-root)))
                (funcall orig-fun))))

I want to avoid adding more flags than necessary, especially on a per-package basis. Instead, I think the more appropriate solution would be to advise rspec-enable-appropriate-mode to no-op if {project-root}/spec doesn't exist. Can you make this change? If not, I'll look into it sometime within the next week or two.


TypeError: Cannot read properties of null (reading 'parentNode')

Environment

Operating System: Linux
Node Version:     v16.17.0
Nuxt Version:     3.6.1
Nitro Version:    2.5.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      devtools, modules, app, imports
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction

Open https://vs3g68-3000.csb.app/foo
click on Bar Layout link

Describe the bug
Layout loading fails
Additional context
See also #13309 (possibly not related)
Logs
No response

@danielroe should be https://codesandbox.io/p/sandbox/vs3g68

Sorry @danielroe
Thats correct. Thanks @manniL

I have the same with navigation back

The issue here is that you are changing the layout in a global store during the transition. So it starts changing the layout on the old page as well as the new one.
This is an upstream Vue bug: https://github.com/vuejs/core/issues/6463.
However you can probably resolve by not setting the layout in a single global store but instead on the metadata for that individual route, which will not change the layout on the old page while transitioning.

@danielroe Is it really the same Vue bug as in #22353? I don't use multiple layouts there and still get this error.

The issue isn't layouts as such, but rerendering a node within suspense while a transition is taking place.

@danielroe What would be the right way to fix the issue I'm dealing with (redirecting to the same route but adding/changing a query param in setup)? I don't seem to be implementing any transitions when navigating to the route as far as I know.

The issue here is that you are changing the layout in a global store during the transition. So it starts changing the layout on the old page as well as the new one.
This is an upstream Vue bug: vuejs/core#6463.
However you can probably resolve by not setting the layout in a single global store but instead on the metadata for that individual route, which will not change the layout on the old page while transitioning.

Ok, thank you for the info. I'll try the suggested approach

@danielroe
I managed to solve it, Thanks again for the tip.
However, I was not able to get it done with definePageMeta (which, if I understood it correctly, is what you suggested). Instead I needed to solve it with a nasty setTimeout. I think the reason is that the layout info comes from the API asynchronously.
I'll keep my eye on vuejs/core#6463, Maybe there will be no need for this workaround anytime soon.

Nuxt 3.7.3 @danielroe
Detailed reproduction:
https://stackblitz.com/edit/nuxt-starter-btlcfb?file=pages%2Fwaiting.vue

Related issue with nuxt 3.7.3
https://github.com/nuxt/nuxt/discussions/23221

the same issues as https://nuxt.com/blog.
when I fast click the menu...
and then  console got error

@Artem-Schander, would you please check if<EMAIL_ADDRESS>solves the issue?

@dargmuesli
For some reason my codesandbox has been reset :(
In my current project the beta.3 seems to solve the issue. However, the project is quite complex and I'm not sure if I can rely on this quick test. I'll try to get the repro back and let you know how it behaves there.

Closing here as most likely resolved via Vue 3.4.0
I'll open another issue that reflects what @ilyaDegtyarenko posted.


docker build on macOS asks for keychain credentials if I include "alexwlchan" in the image tag

Description
When I run docker build on my Mac and tag it with the string "alexwlchan", something within Docker is asking for access to entries in the system keychain.
Steps to reproduce the issue:

I have the following Dockerfile:
FROM alpine

CMD ["echo", "hello"]

I run a docker build command and tag the image with a name that contains a subset of my username:
$ docker build -t alexwlchan/foo .
Sending build context to Docker daemon 2.048 kB
Step 1/2 : FROM alpine
 ---> 4a415e366388
Step 2/2 : CMD echo hello
 ---> Using cache
 ---> 0542fdccd367
Successfully built 0542fdccd367

Describe the results you received:
I get a popup from OS X:

docker-credential-osxkeychain wants to use your confidential information stored in "sync.omnigroup.com (alexwlchan)" in your keychain.
Do you want to allow access to this item? [Always Allow / Deny / Allow]

This is reproducible.
I've also seen it ask for PyPI credentials once, although I didn't get a screenshot that time.
Describe the results you expected:
Docker should build the image without trying to get credentials it doesn’t need. I’m not pushing the image to a remote repository, just doing a local build. And even if I was pushing remotely, my Omni Sync Server and PyPI credentials aren’t what you’d need.
Additional information you deem important (e.g. issue happens only occasionally):
My username on this machine is "alexwlchan":
$ whoami
alexwlchan

That seems to be the problem string. The following names all hit the same issue:

alexwlchan/foo
foo/alexwlchan
alexwlchan

These other names don't:

alex/foo
foo/alex
alex

I also use that name for a lot of my online accounts – "alexwlchan" is the problem string, but whether that comes from my username or because it matches a lot of keychain entries, I’m not sure.
Output of docker version:
$ docker version
Client:
 Version:      17.03.0-ce
 API version:  1.26
 Go version:   go1.7.5
 Git commit:   60ccb22
 Built:        Thu Feb 23 10:40:59 2017
 OS/Arch:      darwin/amd64

Server:
 Version:      17.03.0-ce
 API version:  1.26 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   3a232c8
 Built:        Tue Feb 28 07:52:04 2017
 OS/Arch:      linux/amd64
 Experimental: true

Output of docker info:
$ docker info
Containers: 42
 Running: 1
 Paused: 0
 Stopped: 41
Images: 25
Server Version: 17.03.0-ce
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host ipvlan macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 977c511eda0925a723debdc94d09459af49d082a
runc version: a01dafd48bc1c7cc12bdb01206f9fea7dd6feb70
init version: 949e6fa
Security Options:
 seccomp
  Profile: default
Kernel Version: 4.9.12-moby
Operating System: Alpine Linux v3.5
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 1.952 GiB
Name: moby
ID: Z4EI:56LI:OF5L:AGCF:OXT4:ANTZ:QX4A:CQKR:23YY:67ZA:NGHL:C4EY
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 27
 Goroutines: 42
 System Time: 2017-04-07T17:52:00.32030131Z
 EventsListeners: 2
No Proxy: *.local, 169.254/16
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Additional environment details:
OS X 10.12.4 (16E195).

This is a duplicate of https://github.com/docker/for-mac/issues/1371, and should be fixed in the 17.03.1 version of Docker for Mac
I'll close this here, but let me know if you still have it after upgrading 👍


Remove Product entity from reviews-subgraph service

Remove the knowledge of Product entity from reviews-subgraph service while maintainging the same subgraph structure.

@weekens: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/


Attribute 'Topic' is not valid on this declaration type. It is valid on 'Method' declarations only.

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Hot Chocolate
Describe the bug
Documentation says:

We can even use the Topic attribute on dynamic arguments of the subscription field.
https://chillicream.com/docs/hotchocolate/v13/defining-a-schema/subscriptions#dynamic-topics

Visual Studio says:

Attribute 'Topic' is not valid on this declaration type. It is valid on 'Method' declarations only.

Steps to reproduce
Example:
public class Subscription
{
    [Subscribe]
    public Client ClientUpdated([Topic] int clientId, [EventMessage] Client client) => client;
}

Relevant log output
No response
Additional Context?
No response
Version
13.0.5

This was a breaking change as part of v13 as detailed here: https://chillicream.com/docs/hotchocolate/v13/migrating/migrate-from-12-to-13/#topicattribute
The documentation needs to be updated.

when the docs get updated you might want to note that nameof can be used to get the argument name
public class Subscription
{
    [Subscribe]
    // What's inbetween the curly braces must match an argument name.
    [Topic($"{nameof(author)}")]
    public Book BookPublished(string author, [EventMessage] Book book)
        => book;
}

when the docs get updated you might want to note that nameof can be used to get the argument name
public class Subscription
{
    [Subscribe]
    // What's inbetween the curly braces must match an argument name.
    [Topic($"{nameof(author)}")]
    public Book BookPublished(string author, [EventMessage] Book book)
        => book;
}

You need extra curly braces here: [Topic($"{{{nameof(author)}}}")]


Fix handling files with the same name

If there are multiple files with the same name, append random string to the name

https://github.com/felixge/node-formidable/issues/412

I think this is already implemented. Not sure by which PR, but it's on master.
Thanks anyway. :tada:

Doing this on the file event is too late.
Do we really want this feature ? File inputs in the browser make it impossible to have the same name.

With the new filename option you can loop over form.openedFiles and chose a different name if needed


LLOS: Observer Points/Target Points parameters do not handle coordinates entered in Long Lat order (ArcGIS Pro)

The Coordinate Conversion tool was modified to allow users to enter in coordinates in both Lat Long and Long Lat order. All other tools that allow user to type in coordinates should allow the same.
Tool accepts coordinates to be entered in Lat Long...

Tool does not accept coordinates to be entered in Long Lat...

Picking this up since it is similar to this issue

Fixed with #264 PR

I am not able to verify this due to this issue:
https://github.com/Esri/visibility-addin-dotnet/issues/265

Will be able to verify now since issue was fixed in this PR.

Changes to CC library affected this Long/Lat issue. Moving this back to In Progress

Needs verification. Re-opening issue.

This is working in Pro 2.1 and Visibility Add-in (119) - 6/13/18

Updating CMS doc for Visibility issues addressed.

verified added to the doc


Invalid parameter type. renameProperty must be of type: undefined.

What version of the module is the issue happening on? Does the issue happen on latest version?
0.14.2

What platform and Node.js version? (For example Node.js 6.9.1 on Mac OS X)
6.10.0

Does the action work when you manually perform request against mailgun using curl (or other means)?
yes

Sample source code or steps to reproduce
{ from: '"Bio-soft"<EMAIL_ADDRESS>to<EMAIL_ADDRESS>subject: 'Please change your password', html: '...'

(Write description of your issue here, stack traces from errors and code that reproduces the issue are helpful)
there is a error like 'Invalid parameter type. renameProperty must be of type: undefined.' when I send a email using mailgun-js function.
function like this.
mailgun.messages().send(data, function (error, body) {       if(error) {         console.log(error)       } else {         return res.end('success');       }     });

Hello, can you please provide a more detailed report with a stack trace. I was not able to reproduce this with the specific version of node and module. Thanks.

Closing due to lack of follow-up information.


feat: Add basic STRtree wrapper

Basic STRtree wrapper with insert and query methods. I'm fairly new to Rust, so I wasn't sure about the correct way to store (references to) items in the tree. The proposed implementation introduces a generic type parameter  corresponding to the type of items that can be inserted into the tree. Internally, items are boxed and the raw pointer is passed to the GEOS implementation. When the tree is dropped, the implementation ensures that those pointers are freed.

Sorry, didn't receive the notification until @amatissart requested a review from me... Thanks a lot for the PR!

You're welcome :) Thanks for merging!


fix: remove default imports

BREAKING CHANGE: Includes were removed by default as tree-shaking should do its job.
Checks

[x] Manual check in Codesandbox.

:tada: This PR is included in version 5.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Compiling Issue

Hi There,
I love your work good job.
I have downloaded the code and try to compile and send it to my Teensy4.1 but get the following errors, I wonder if you can assist in fixing this.
In file included from src\config.h:30,
from src\main.cpp:9:
src\eyes/EyeController.h: In instantiation of 'std::pair<float, float> EyeController<numEyes, Disp>::computeEyelids(Eye&) const [with unsigned int numEyes = 2; Disp = GC9A01A_Display]':
src\eyes/EyeController.h:682:43:   required from 'bool EyeController<numEyes, Disp>::renderFrame() [with unsigned int numEyes = 2; Disp = GC9A01A_Display]'
src\main.cpp:132:20:   required from here
src\eyes/EyeController.h:252:27: note: parameter passing for argument of type 'std::pair<float, float>' when C++17 is enabled changed to match C++14 in GCC 10.1
252 |   std::pair<float, float> computeEyelids(Eye &eye) const {
|
../.platformio/packages/toolchain-gccarmnoneeabi-teensy/bin/../lib/gcc/arm-none-eabi/11.3.1/../../../../arm-none-eabi/bin/ld.exe: .pio\build\eyes\firmware.elf section .text.csf' will not fit in region FLASH'
../.platformio/packages/toolchain-gccarmnoneeabi-teensy/bin/../lib/gcc/arm-none-eabi/11.3.1/../../../../arm-none-eabi/bin/ld.exe: region `FLASH' overflowed by 3072 bytes
collect2.exe: error: ld returned 1 exit status
*** [.pio\build\eyes\firmware.elf] Error 1
======================================================================================= [FAILED] Took 12.90 seconds =======================================================================================

The terminal process "C:\Users\frikk.platformio\penv\Scripts\platformio.exe 'run'" terminated with exit code: 1.
Terminal will be reused by tasks, press any key to close it.

I am using 2 x GC9A01A round displays with the teensy4.1

Hmm I'm not sure what's different as it fit onto my 4.1 OK, are you sure you have a 4.1 and not 4.0? (The 4.0 has less storage). You could try commenting out one (or more) additional eyes from config.h (you'll also have to decrease the 13 on line 46 to match the number of eyes you have remaining) and see if that helps.

Hi Chris,
Thanks for the response, I do have a teensy 4.1 and I have managed to get
it working, i just had to change the board in the platformio.ini from
teensy4 to teensy4.1.
And now it works great
Keep up the good work.
btw. have you converted this code to get it to work on an ESP32 or
Raspberry PI ,  Zero 2W or pi 4 or pi 5?
On Tue, 10 Dec 2024 at 10:47, Chris Miller @.***> wrote:

Hmm I'm not sure what's different as it fit onto my 4.1 OK, are you sure
you have a 4.1 and not 4.0? (The 4.0 has less storage). You could try
commenting out one (or more) additional eyes from config.h (you'll also
have to decrease the 13 on line 46 to match the number of eyes you have
remaining) and see if that helps.
—
Reply to this email directly, view it on GitHub
https://github.com/chrismiller/TeensyEyes/issues/4#issuecomment-2529818814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKSMZVX53XKV6JA3Z4RIJL2EYT23AVCNFSM6AAAAABSILH2JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRZHAYTQOBRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ah OK, I should probably change the board to 4.1 then, since the current number of enabled eyes is too many for the 4.0. Note that with the 4.1 you should be able to enable all of them at once.
No I haven't tried to port the code to other platforms, as I don't really have a use case for that personally and I think it would probably run too slow on an ESP32 anyway. I guess it wouldn't be too hard to port though, if someone was willing to try.

Hi Chris,
Yes, i have loaded all the eyes and added code for a press of a button to
switch between the different eyes,  and i added a light sensor for the
pupil, and a joystick to move the eyes,
I am just waiting for the Person Sensor by Useful Sensors to check if
the eye can track a person ...
In the meantime I will update the joystick  to a wireless joystick with
ESP32C3 Seeed Studio XIAO BLE WIFI. This esp is small 20mm x 20mm and cheap.
FYI
Game Controller R1 Mini Ring Bluetooth-Compatible 4.0 Rechargeable Wireless
VR Remote Joystick
https://www.aliexpress.com/item/1005004526148953.html?spm=a2g0o.order_list.order_list_main.64.50271802Wbzfwl
ESP32C3 Seeed Studio XIAO BLE WIFI
https://www.aliexpress.com/item/1005005382287176.html?spm=a2g0o.order_list.order_list_main.39.55bc1802VYqjl1
On Tue, 10 Dec 2024 at 11:17, Chris Miller @.***> wrote:

Ah OK, I should probably change the board to 4.1 then, since the current
number of enabled eyes is too many for the 4.0. Note that with the 4.1 you
should be able to enable all of them at once.
No I haven't tried to port the code to other platforms, as I don't really
have a use case for that personally and I think it would probably run too
slow on an ESP32 anyway. I guess it wouldn't be too hard to port though, if
someone was willing to try.
—
Reply to this email directly, view it on GitHub
https://github.com/chrismiller/TeensyEyes/issues/4#issuecomment-2529896014,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKSMZUVOR2KA6LKZPYBU232EYXHZAVCNFSM6AAAAABSILH2JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRZHA4TMMBRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi Chris,
How are you?
Quick question, on which pins on the teensy 4.1 should i connect the person
sensor, it has 4 wires
On Tue, 10 Dec 2024 at 11:17, Chris Miller @.***> wrote:

Ah OK, I should probably change the board to 4.1 then, since the current
number of enabled eyes is too many for the 4.0. Note that with the 4.1 you
should be able to enable all of them at once.
No I haven't tried to port the code to other platforms, as I don't really
have a use case for that personally and I think it would probably run too
slow on an ESP32 anyway. I guess it wouldn't be too hard to port though, if
someone was willing to try.
—
Reply to this email directly, view it on GitHub
https://github.com/chrismiller/TeensyEyes/issues/4#issuecomment-2529896014,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKSMZUVOR2KA6LKZPYBU232EYXHZAVCNFSM6AAAAABSILH2JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRZHA4TMMBRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks Chrismiller/Teensyeyes I will give it a bash and let you know.
On Tue, 17 Dec 2024 at 09:16, Chris Miller @.***> wrote:

The Person Sensor is an I2C device, so you want to connect its SCL and SDA
pins to one of the SCL and SDA pairs on thet Teensy (e.g. pins 18 and 19).
The other two pins go to ground and +3.3V (note not 5v!). The Person
Sensor docs for this are here
https://github.com/usefulsensors/person_sensor_docs/blob/main/README.md#connecting,
but if I recall correctly the diagrams they show are for the old v1.0
sensors, and their board layout has changed with their newer versions of
the Person Sensor, so take care to be sure you know which pin is which as I
remember it was pretty confusing due to the out of date diagram (I'm not
near mine so can't check right now sorry).
—
Reply to this email directly, view it on GitHub
https://github.com/chrismiller/TeensyEyes/issues/4#issuecomment-2546959593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKSMZTPN4775K33QZE7GRT2F5GLXAVCNFSM6AAAAABSILH2JCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBWHE2TSNJZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Force reloading

Hi!
I've been using cargo-watch since I started using Rust and I'm really really loving it. Thank you so much for all of your work so far.
Currently I'm building a library, and I'm also building a server that uses that library. I work on both simultaneously occasionally, and sometimes the library crashes the server, which I'm hot reloading with cargo watch. When I update code in the library, it doesn't cause the server to hot reload. This makes sense to me - a dependency, even local, changing shouldn't cause a sudden reload (in my opinion) unless I'm explicitly watching it.
However, it'd be cool to reload the watched code at any point in time, by pressing a key like r or <spacebar> in the terminal or something. Feels like it's something that might be useful to others as well. Also, if you're interested, I'd love to contribute the code for this if you can point me in the right direction.
Thanks!

Yeah, this would be great. Would have to go in watchexec: https://github.com/watchexec/watchexec/issues/67
The way to go, I think, would be to start here: https://github.com/watchexec/watchexec/blob/master/src/run.rs#L93 and upgrade the channel event to support both filesystem events and keyboard events, then spawn another thread to wait on STDIN.
Also upgrade on_manual https://github.com/watchexec/watchexec/blob/master/src/run.rs#L193 to take a restart: bool so rs<enter> always restarts the process. Ideally I'd like to support both entr's and nodemon's inputs (rs<enter> = restart, <space> = go if stopped, q = stop gracefully).
Of course it would also need a flag to disable (and maybe customise the keybindings if ambitious! at least via the Rust API) the behaviour, for programs that want to do their own STDIN handling.

Okay, awesome, thank you for the guidance! I'll get started on this later this week/end.

Hey! I haven't forgotten about this; I'm about 35%, but got a bit slammed with stuff for a week so have to put it on hold for a little.

That's perfectly alright, thanks for the status update!
On Fri, 19 Jul 2019, 03:14 Julien Tregoat<EMAIL_ADDRESS>wrote:

Hey! I haven't forgotten about this; I'm about 35%, but got a bit slammed
with stuff for a week so have to put it on hold for a little.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/passcod/cargo-watch/issues/123?email_source=notifications&email_token=AABGBCY2I7R655AGH4WZLMDQACCENA5CNFSM4H65SXBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2IZ25A#issuecomment-512859508,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABGBC5OJNR4R25U4LVC7UDQACCENANCNFSM4H65SXBA
.

Have you made any progress on this? I would love to see this functionality in watchexec

I got about halfway but then things got pretty busy this year and I haven't been able to tackle any of the OSS projects I've been wanting to. sorry bout the dealy! I still want this functionality as well and will try to finish it off in the next couple of months but someone's welcome to grab this if they want.

Ah right, I might have a look at it if I get some time to spare

Closing in favour of upstream issue.


TF 2.13 in docker images doesnt work with GPU

Describe the bug
Hello,
Thanks for putting these together, makes it easy to get started. One issue I found: TF doesnt use GPU even when its available. I think this is due to CUDA 12.1.1. When I switched to CUDA 11.7.1 - TF does use GPU.
ARG CUDA_VER=11.7.1
ARG UBUNTU_VER=22.04
ARG PYTHON_VER=3.10
Here is how I tested:
sudo docker run --rm --runtime=nvidia --gpus all  matifali/dockerdl python3 -c 'import tensorflow as tf; devices = [d.device_type for d in tf.config.list_physical_devices()]; print("Available devices: %s", devices)'
Output:
CUDA Version 12.1.1

Container image Copyright (c) 2016-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.

This container image and its contents are governed by the NVIDIA Deep Learning Container License.
By pulling and using the container, you accept the terms and conditions of this license:
https://developer.nvidia.com/ngc/nvidia-deep-learning-container-license

A copy of this license is made available in this container at /NGC-DL-CONTAINER-LICENSE for your convenience.

2023-07-13 15:38:30.515088: I tensorflow/tsl/cuda/cudart_stub.cc:28] Could not find cuda drivers on your machine, GPU will not be used.
2023-07-13 15:38:30.564382: I tensorflow/tsl/cuda/cudart_stub.cc:28] Could not find cuda drivers on your machine, GPU will not be used.
2023-07-13 15:38:30.565062: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-07-13 15:38:31.662344: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
2023-07-13 15:38:35.089274: I tensorflow/compiler/xla/stream_executor/cuda/cuda_gpu_executor.cc:995] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero. See more at https://github.com/torvalds/linux/blob/v6.0/Documentation/ABI/testing/sysfs-bus-pci#L344-L355
2023-07-13 15:38:35.122639: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1960] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
Available devices: %s ['CPU']

Output of same command when using CUDA 11.7.1:
sudo docker run --rm --runtime=nvidia --gpus all  my-tf-image python3 -c 'import tensorflow as tf; devices = [d.device_type for d in tf.config.list_physical_devices()]; print("Available devices: %s", devices)'

==========
== CUDA ==
==========

CUDA Version 11.7.1

Container image Copyright (c) 2016-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.

This container image and its contents are governed by the NVIDIA Deep Learning Container License.
By pulling and using the container, you accept the terms and conditions of this license:
https://developer.nvidia.com/ngc/nvidia-deep-learning-container-license

A copy of this license is made available in this container at /NGC-DL-CONTAINER-LICENSE for your convenience.

2023-07-13 15:50:44.533017: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-07-13 15:50:45.743829: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
2023-07-13 15:50:49.414871: I tensorflow/compiler/xla/stream_executor/cuda/cuda_gpu_executor.cc:995] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero. See more at https://github.com/torvalds/linux/blob/v6.0/Documentation/ABI/testing/sysfs-bus-pci#L344-L355
2023-07-13 15:50:49.461339: I tensorflow/compiler/xla/stream_executor/cuda/cuda_gpu_executor.cc:995] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero. See more at https://github.com/torvalds/linux/blob/v6.0/Documentation/ABI/testing/sysfs-bus-pci#L344-L355
2023-07-13 15:50:49.463117: I tensorflow/compiler/xla/stream_executor/cuda/cuda_gpu_executor.cc:995] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero. See more at https://github.com/torvalds/linux/blob/v6.0/Documentation/ABI/testing/sysfs-bus-pci#L344-L355
Available devices: %s ['CPU', 'GPU']

To Reproduce
Use docker command on VM configured to use GPU.
sudo docker run --rm --runtime=nvidia --gpus all  matifali/dockerdl python3 -c 'import tensorflow as tf; devices = [d.device_type for d in tf.config.list_physical_devices()]; print("Available devices: %s", devices)'
Expected behavior
GPU is in the list of supported devices

@realalexgalenko Thanks for raising the issue. I myself don't use TensorFlow actively so that I couldn't test the images.
There are open issues for CUDA 12.x support for TensorFlow pip packages.
https://github.com/tensorflow/tensorflow/issues/60943
https://github.com/tensorflow/tensorflow/issues/60691
Meanwhile, I can try to pin CUDA to 11.8 for TensorFlow images.

Yep, totally makes sense - keep cuda 12 for other images (maybe mention tf would use cpu only for those) and  change tf image to use 11 to enable gpu


Tensorflow only sees a fraction of Tensorflow memory

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NA
TensorFlow installed from (source or binary): source
TensorFlow version: 2.4.1
Python version: 3.8.8
Installed using virtualenv? pip? conda?: Virtualenv inside docker
Bazel version (if compiling from source): 3.1.0
GCC/Compiler version (if compiling from source): gcc 7.5.0
CUDA/cuDNN version: CUDA 11.3 (host) CUDA 11.2 (docker), cuDNN8
GPU model and memory: GT 730, 1gb

Describe the problem
My device has 1gb of memory but Tensorflow only sees 636mb:
+-----------------------------------------------------------------------------+
 | NVIDIA-SMI 465.19.01    Driver Version: 465.19.01    CUDA Version: 11.3     |
 |-------------------------------+----------------------+----------------------+
 | GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
 | Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
 |                               |                      |               MIG M. |
 |===============================+======================+======================|
 |   0  NVIDIA GeForce ...  On   | 00000000:01:00.0 N/A |                  N/A |
 | 30%   33C    P8    N/A /  N/A |    118MiB /   980MiB |     N/A      Default |
 |                               |                      |                  N/A |
 +-------------------------------+----------------------+----------------------+

>>> import tensorflow as tf
2021-04-24 12:33:23.660088: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.11.0
>>> tf.test.is_gpu_available()
WARNING:tensorflow:From <stdin>:1: is_gpu_available (from tensorflow.python.framework.test_util) is deprecated and will be removed in a future version.
Instructions for updating:
Use `tf.config.list_physical_devices('GPU')` instead.
2021-04-24 12:33:36.109192: I tensorflow/compiler/jit/xla_gpu_device.cc:99] Not creating XLA devices, tf_xla_enable_xla_devices not set
2021-04-24 12:33:36.112728: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcuda.so.1
2021-04-24 12:33:36.163350: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1720] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: NVIDIA GeForce GT 730 computeCapability: 3.5
coreClock: 0.9015GHz coreCount: 2 deviceMemorySize: 980.00MiB deviceMemoryBandwidth: 37.33GiB/s
2021-04-24 12:33:36.163429: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.11.0
2021-04-24 12:33:36.174055: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcublas.so.11
2021-04-24 12:33:36.174941: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcublasLt.so.11
2021-04-24 12:33:36.179867: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcufft.so.10
2021-04-24 12:33:36.181045: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcurand.so.10
2021-04-24 12:33:36.184262: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcusolver.so.11
2021-04-24 12:33:36.187118: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcusparse.so.11
2021-04-24 12:33:36.188172: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudnn.so.8
2021-04-24 12:33:36.189819: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1862] Adding visible gpu devices: 0
2021-04-24 12:33:36.190342: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.11.0
2021-04-24 12:33:37.537902: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1261] Device interconnect StreamExecutor with strength 1 edge matrix:
2021-04-24 12:33:37.537984: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1267]      0
2021-04-24 12:33:37.538003: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1280] 0:   N
2021-04-24 12:33:37.540130: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1406] Created TensorFlow device (/device:GPU:0 with 636 MB memory) -> physical GPU (device: 0, name: NVIDIA GeForce GT 730, p
ci bus id: 0000:01:00.0, compute capability: 3.5)
True

Could this be the result of build config?

These two lines are interesting:

2021-04-24 12:33:36.163350: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1720] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: NVIDIA GeForce GT 730 computeCapability: 3.5
coreClock: 0.9015GHz coreCount: 2 deviceMemorySize: 980.00MiB deviceMemoryBandwidth: 37.33GiB/s

vs.

2021-04-24 12:33:37.540130: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1406] Created TensorFlow device (/device:GPU:0 with 636 MB memory) -> physical GPU (device: 0, name: NVIDIA GeForce GT 730, p
ci bus id: 0000:01:00.0, compute capability: 3.5)

Not sure how to interpret these messages?

Yes, exactly! My supposition is that either the CUDA drivers or Tensorflow or both isn't representing the device properly.

Looks like a known issue: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/common_runtime/gpu/gpu_device.cc#L1405

@asteriske ,
Please take a look at moflo   comment and let us know if you are still facing the same issue? Thanks!

I'm not very experienced with GPUs so I may not be interpreting it correctly, but that comment seems to suggest that the mismatch is a result of reallocation?
I get these numbers in a fresh python process, so does that mean that some mechanism is enforcing sharing before I initialize the device?

TensorFlow has to reserve some amount of GPU memory for system libraries like cuDNN, cuBLAS etc.; that is likely the delta you're seeing here.

@asteriske,
TensorFlow allocates the  GPU memory conservatively and manages memory efficiently. It uses memory on demand strategy and only allocates GPU memory as required and doesnot pre-allocate the entire GPU memory to avoid wastage of the resources.
It might be useful when the multiple users or processes share a GPU, as it prevents that the only one user utilizes the entire GPU memory.
Also tensorflow is designed to reuse the GPU memory whenever it is required. When the tensors memory is no longer needed, it can be recycled for other tensors, reducing the overall memory.
https://www.tensorflow.org/guide/gpu
Thank you!


Add benchmark and reduce GPU memory for cudnn_conv and speed up cudnn_conv.

The code in paddle are extracted to pull  https://github.com/baidu/Paddle/pull/218

add benchmark
add ConvProjection to reduce GPU memory for googlenet
increase workspace limit size in cudnn_conv to speed training.
use TmpMatrix in cudnn_conv to reduce GPU memory.

can split this pull to some small pulls ? as there are 60 files changed, and is hard to review.


Subaru: Fix cancel backoff rate calculation

this check didn't do anything before, since frames are whole numbers. Need to verify if we can still cancel if we aren't spamming at the full 50hz

cancel seems to work fine on our outback: 54827bf84c38b14f|2023-08-15--09-54-57

I'm only seeing a couple dozen cancels in that route. We need to check that the distribution of cancellation times hasn't (meaningfully) changed.
The other option here is to remove the check that it's never hitting anyway instead of "fixing" it since that has been working fine.

I'm only seeing a couple dozen cancels in that route. We need to check that the distribution of cancellation times hasn't (meaningfully) changed.
The other option here is to remove the check that it's never hitting anyway instead of "fixing" it since that has been working fine.

If there's no particular reason that other car ports have a backoff rate, then I'll just remove it


The sdist doesn't currently include LICENSE

This could be as simple as adding include LICENSE to MANIFEST.in

I'll put up a PR now.


Do I need the backend code?

I am using react-social-network, and it seems like it's good standing alone, after connecting to a cloud firestore database. But someone else said this repo cannot be used stand alone unless pairing with the  https://github.com/Qolzam/firestore-social-backend repo.

Hello @easonychang , thanks for pointing this issue. I can tell you is stand alone repo why? because whole the services are using interfaces  from core services. It means you are able to implement your own service and inject to the container. Sure there are some challenge such as stream data which has it's own solution. I have implemented  AWS and ASP.NET Core some how but as it takes so much time it's developing so slowly.
If you are interested in any kind of backend to implement for this repository, I am happy to help so we can make it faster.

So I think my understanding was that this repo is stand alone, and I don't really need the firestore-social-backend repo along with it to be able to interact with the database that I have configured on firebase.

Sure at least you need to have a back-end and database to store the data. However for firestore-social-backend is only configuration using Firestore Database not really huge backend to manipulate the data. Almost manipulating data is done on this repository as a Firestore Data Client. With this you have kind of Serverless Architecture.
Hope it's helpful. Let me know if any question. Thanks!

I see. I followed the instructions on your medium post, was able to configure it with firestore even without firestore-social-backend repo. Which was why I was a little bit confused of why I needed the backend repo to have the app running.

I am going to close this issue. Let me know if any issue. Thanks!


bad guess for pyFAI

pyFAI is a package for reducing powder diffraction data. It has been published in Journal of Applied Crystallography, Powder diffraction, ...

Depsy is no longer being updated. See https://twitter.com/depsy_org/status/970376969782149120

duplicate of #101


Usage guide needs improvement

Just saying. Including the legacy script or the everything script, and store is undefined.
RequireJS can't find it without explicitly specifying the context, unless you happen to throw the project in the root of your website which would be highly strange.
I've spent 30 minutes just trying to figure out how to use this damn thing. Incredibly frustrating.
Wish I could say it was a breeze to utilize this, but it's not.
Just curious if anyone else has had a positive experience with installation? Or is it just me.

Indeed, I too have issues while exploring documentation. For example, the guide to use Expire pluging, simply sends you to the tests written for testing plugins.

@pavittarx For expire, I'm not sure if we have to run removeExpiredKeys or not. Have you found out?


Strange text on Attribute:

OUT-PATIENT_ATTENDANCE_INDICATOR_FOR_RADIOTHERAPY_DATA_SET: “then an Out-Patient Attendance recordis required, and National Code‘Out-Patient Attendance recorded’ applies.” National Codes:”

Looks like this is fixed?

It doesn't match the current version exactly.
There is an extra bullet between the two "Where the PATIENT" and the final part "then an" is not indented.
Can this be fixed?

Extra (empty) bullets also spotted on https://modelcatalogue.cs.ox.ac.uk/datadictionary/supporting_information/supporting_information/accident_and_emergency_investigation_table.html

Accident and Emergency Investigation Table will be retired in the October release.

Text still not displaying correctly in the beta version, looking to see if the original text be updated to resolve this.

Fixed in the October release.


[PE-D][Tester C] Past Dates Are Incorrectly Accepted for Event Scheduling

Description: The application currently allows users to schedule events with dates in the past. Since events are typically meant to be scheduled for future dates, allowing past dates can lead to errors and confusion, particularly for users who may not realize their mistake.
Steps to Reproduce:

Open the application and create an entry for an event.
Enter a date that is in the past, such as "01 Jan 2023".
Submit the event.

Impact: Allowing past dates for event scheduling can result in erroneous entries, creating potential issues in scheduling, reminders, and reporting. This may cause user frustration and data inconsistencies.
Attachment:
Input:
add -e n/wkdakw des/wkadk d/1997-01-01 c/0 v/6
Output:

Labels: severity.Medium type.FeatureFlaw
original: e1121208/ped#4

This is an expected behaviour, as users might want to add old events for reference.


WIP: Osxfs caching updates

Proposed changes

Update osxfs documentation with details about support for user-defined caching in 17.04
Add a new page, docker-for-mac/osxfs-caching, with a detailed specification of the new flags
Update the volumes tutorial with a note about the cached flag

Related issues (optional)
Related: https://github.com/docker/docker/pull/31047

I've updated the documentation to remove links to the (still-pending) blog post.  We can add links once the post is up.
This is now ready for merging.

Is anyone available to take a look at this (e.g. @londoncalling, @mstanleyjones, @johndmulhausen)?

@yallop I'll take a look now

@yallop I'm in midst of reading through now, but here are my suggestions:

I will defer to you on technical content, and ping some other folks to review, but they may not get back to it until tomorrow because of time zones

I would like to make editorial updates, and also incorporate the new content into the current File Sharing topic rather than create a  new one. (If we do keep the new topic, I need to add it into the TOC, which I can do, but I think this content belongs in the File Sharing topic.) In order to make the updates, I'm going to go ahead and contribute my updates to this PR on your branch. Sound good?

@londoncalling, thanks for your review and improvements, which all look good to me.  I've added some examples in a8c21ae.

@yallop @avsm all these changes look great! Just going through once more, I might add one or two things, then merge it if you guys are ready.

Work of art, merging.


transactions: added sent/received filter options, filtering reworked

I have tried to announce a transfer transaction than return back to the transaction list. it is still not working quite right. Once the unconfirmed transaction gets removed (become confirmed), the transaction list changed unexpectedly. Could you please have a look?


Indeterminate CircularProgress is rendering as a black circle

in my package,json:
"rmwc": "^5.5.2",
"material-components-web": "^2.3.1",
using webpack to build my frontend, have you experience using CircularProgress but it renders a black circle instead of the indeterminate circle? Thanks in advance to solve this. last time i just resorted to using the linear progress because that one works.
here is a screenshot, i was trying to make a suggestion menu for a text field, and im trying to show that the suggestions are "loading" that's why i want to use circular progress.

this screenshot is from using firefox developer edition on a linux os
in my component:
<MenuItem disabled>
       <CircularProgress />
</MenuItem>

I haven't been able to reproduce this. Can you repro it in the code sandbox?

@maurusrv is this still an issue?

Please reopen if you are still seeing this issue. Firefox is working correctly in my testing.


Clean up and remove true and false positives with -fsanatize=undefined errors in mock-array-big design

Description
Some of these look interesting w.r.t. that they could be real problems or produce rare non-determinism.
Build with:
./build_openroad.sh --local --clean-force --openroad-args -DCMAKE_CXX_FLAGS=-fsanitize=undefined

Then run:
make DESIGN_CONFIG=designs/asap7/mock-array-big/config.mk

Hits:
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gpl/src/nesterovBase.cpp:344:36: runtime error: signed integer overflow: -2147483648 -<PHONE_NUMBER> cannot be represented in type 'int'
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gpl/src/nesterovBase.cpp:344:50: runtime error: signed integer overflow: -2147483648 -<PHONE_NUMBER> cannot be represented in type 'int'
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/grt/src/fastroute/include/DataType.h:175:8: runtime error: load of value 85, which is not a valid value for type 'bool'
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/drt/src/gc/FlexGC_init.cpp:88:20: runtime error: downcast of address 0x7f0258010f90 which does not point to an object of type 'drShape'
0x7f0258010f90: note: object is of type 'fr::drVia'
 00 00 00 00  d0 10 33 51 a2 55 00 00  ff ff ff ff 3d 0e 00 00  25 26 00 00 02 7f 00 00  80 a7 5e 61
              ^~~~~~~~~~~~~~~~~~~~~~~
              vptr for 'fr::drVia'
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gui/src/gui.cpp:255:29: runtime error: member call on address 0x565503bbd800 which does not point to an object of type 'MainWindow'
0x565503bbd800: note: object is of type 'QWidget'
 00 00 00 00  48 31 e9 ef 6b 7f 00 00  20 ec b0 03 55 56 00 00  f8 32 e9 ef 6b 7f 00 00  00 00 00 00
              ^~~~~~~~~~~~~~~~~~~~~~~
              vptr for 'QWidget'
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gui/src/mainWindow.h:94:49: runtime error: member access within address 0x565503bbd800 which does not point to an object of type 'MainWindow'
0x565503bbd800: note: object is of type 'QWidget'
 00 00 00 00  48 31 e9 ef 6b 7f 00 00  20 ec b0 03 55 56 00 00  f8 32 e9 ef 6b 7f 00 00  00 00 00 00
              ^~~~~~~~~~~~~~~~~~~~~~~
              vptr for 'QWidget'
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gui/src/gui.cpp:267:22: runtime error: member call on address 0x565503bbd800 which does not point to an object of type 'MainWindow'
0x565503bbd800: note: object is of type 'QWidget'
 00 00 00 00  48 31 e9 ef 6b 7f 00 00  20 ec b0 03 55 56 00 00  f8 32 e9 ef 6b 7f 00 00  00 00 00 00
              ^~~~~~~~~~~~~~~~~~~~~~~
              vptr for 'QWidget'
[etc]

Suggested Solution
No response
Additional Context
No response

@vvbandeira please setup a sanitizer build pipeline

@gadfort would you look at the gui issues and @eder-matheus can look at the others.

@vvbandeira please setup a sanitizer build pipeline

I think there is a lot to be said for automatic failure of PRs upon sanatizer failures, but that does mean that false positives have to be dealt with.

FYI, if you see this one, it is a false positive:
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gpl/src/nesterovBase.cpp:344:36: runtime error: signed integer overflow: -2147483648 -<PHONE_NUMBER> cannot be represented in type 'int'
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gpl/src/nesterovBase.cpp:344:50: runtime error: signed integer overflow: -2147483648 -<PHONE_NUMBER> cannot be represented in type 'int'
[NesterovSolve] Iter: 1 overflow: 0.995667 HPWL: 7577714
[NesterovSolve] Iter: 10 overflow: 0.995671 HPWL: 2144047

The code can easily be rewritten not to trigger a false positive by using INT_MAX - 1 and INT_MIN + 1:

See also https://github.com/The-OpenROAD-Project/OpenROAD/issues/3244

@maliberty I'm not sure what I'm looking for here. I'm happy to fix anything, but I'm not 100% sure that the issue is

To be honest I don't see the problem either now that I look at it.  I wonder if it is confused by multiple inheritance.

A possible for fix the gui error: https://github.com/The-OpenROAD-Project/OpenROAD/pull/3251

@maliberty I think we're agreed that removing sanatizer issues and checking for them in CI would be great.
If so, then this issue can be closed. It doesn't contain anything specifically actionable.

@maliberty is this still a thing?


Migrate DragAndDrop/index.js to function component

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/16142
Tests

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/39824924/9dc6cb45-d48e-4d7e-8f61-4f6561f16c13

Mobile Web - Chrome
Not sure if drag and drop to chat feature can be tested on this platform.

Mobile Web - Safari
Not sure if drag and drop to chat feature can be tested on this platform.

Desktop
https://github.com/Expensify/App/assets/39824924/93ac7f37-5fa7-4448-9368-66297a1c2319

iOS
Not sure if drag and drop to chat feature can be tested on this platform.

Android
Not sure if drag and drop to chat feature can be tested on this platform.

Closing this as I was informed that @jczekalski is going to be working on other projects. Thanks for your work on this so far @jczekalski. I'm going to create a new PR based on this one incorporating my above feedback since this is HOLDing another PR that's a roadmap priority. @allroundexperts I'll request your review on my PR when it's ready


Tapping on the card below the top card makes bottom card come up and top card goes down

Tapping on the card below the top card makes bottom card come up and top card goes down and then instead of restricting user to topmost card, user can swipe card on 2nd position also instead of only the topmost card. Need to disallow moving any other card except the top card by the user.

This would be a great fix for us too!

same issue here


User Terminal  - category Management

Review table Structure for Catgegory and sub-Category Management

Change The user Terminal to show all the category from CATG_MSTR and CATG_SUB_CATG_MAP .

We have four possible status everywhere CHECK ('PNDG' or 'LIVE' or 'REJ' or 'DLTD').

Only LIVE Catg should be shown on page.If status is changed from Admin it be reflected om User Page and accordingly it should be updated on User Page.

If the Position is changed from admin Terminal it should also change on User side.

When hover on Root Category  Sub Category  Tree Should be Displayed .

When hover on Root Category  Image should be displayed in main Banner Sections .

When hover on Root Category Category Name Should be highlighted .

On User Home Page Below Best Sells section, Categories and Sub-Categories should be displaced  based on CATG_MSTR_DISP_PRIORITY >=5 order by CATG_MSTR_DISP_PRIORITY ascending.
On Hovering Image for category and Sub-Category should be Highlighted .
Time Required:1 Day

basic functions look ok . passed on for testing - In Test

hello sir,
i have made 3 templates in html
they are below
neha's_html.zip
in above template, with the use of tables i created templates.
after that i had try to change their background color, font color , font size ,etc.
database:
in  database i create all tables of the_theme_park
but i have an issue regarding this table creating...
i m unable to create foreign key of that table...
this are shown below:-


Modularize visualization and add MSE function

Modularized the visualize function by splitting it into compute_traj_pred_actual_actions and visualize, and added a compute_mse function that mimics the visualize function (looping through sampled trajectories from the datasets and calling compute_traj_pred_actual_actions for each trajectory), and logging the MSE metrics (that are part of the logging into wandb).
Just as a note, diffusion_policy.py and BC_Transformer in bc.py have the exact same 3 functions (compute_traj_pred_actual_actions, visualize, compute_mse). These 3 functions probably work for other classes in bc.py, so moving that to the base BC class may be a good idea.
Other files that are modified:

make_model_prediction_plot in algo.py: slight modification to accommodate changes in visualize
config/base_config.py, scripts/config_gen/helper.py: adding fields to self.experiment.vis and self.experiment.mse
bc_transformer.json, bc.json, diffusion_policy.json: adding mse fields

Added support for a validation set for R2D2 data. Changes are in:

config_gen/helper.py: Added a function split_datasets that splits the dataset_paths. At the bottom, set "experiment.validate" to True and add "experiment.validation_epoch_every_n_steps".
config_gen/bc_xfmr_gen.py: Added keys "train.hdf5_filter_key" and "train.hdf5_validation_filter_key", which calls split_datsets to generate the train set paths and val set paths
config_gen/diffusion_gen.py: Same modifications as bc_xfmr_gen.py
train_utils.py: In dataset_factory, set ds_kwargs["filter_by_attribute"] according to filter_by_attribute

Updated algo/bc.py and algo/diffusion_policy.py to support validation set in visualization and MSE calculation: inference_datasets_mapping = {"Train": training_sampled_data, "Valid": validation_sampled_data}  instead of just inference_datasets_mapping = {"Train": training_sampled_data}

I incorporated the comments. Specifically:

compute_traj_pred_actual_actions, visualize, compute_mse are removed from diffusion_policy.py and BC_Transformer in bc.py and moved to PolicyAlgo class in algo.py
Changes in config_gen/helper.py, config_gen/bc_xfmr_gen.py, config_gen/diffusion_gen.py, and train_utils.py for validation set have been removed


Removed a bug regarding #2253

Something I did not use at all so far is to put the link between the PR and the issue through the Development setting at the right. That is something I intent to do consistently and this is something PR submitters could do. I mean this:

I should put this in the guidelines somewhere.

Something else. I generate release notes from the PRs, so it is nice to have PR titles that read as release notes right away. Last month I added this section to the guidelines https://mapsui.com/documentation/contributors-guidelines.html#pr-titles-should-be-written-as-release-notes.
If a change is needed I prefer to do this in the source (the PR) so that release notes and PR titles are identical. I will need to do this frequently.


Succeed to create a table using the name of one table which is being deleted

Storage Explorer Version: 1.22.0-dev
Build Number: 20211121.3
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.0.1
Architecture: ia32/x64
How Found: From running test cases
Regression From: Previous release (1.21.3)
Steps to Reproduce

Expand one storage account -> Tables.
Create a table -> Delete the table -> Then create a table using the name of the deleted table.
Check whether fail to create the table.

Expected Experience

Fail to create the table.
An error indicating 'The specified table is being deleted' occurs.

Actual Experience

Succeed to create the table.
The table is auto opened and an error info bar prompts.

Error info:
{"odata.error":{"code":"TableNotFound","message":{"lang":"en-US","value":"The table specified does not exist.\nRequestId:ef3b5f9d-f002-004e-5871-df3cc7000000\nTime:2021-11-22T07:19:07.4894971Z"}}}
Additional Context
The created table disappears after manually refreshing.

Appears to be the same root cause as #5200. The createTable function doesn't throw an error as expected.
Creating a table with the same name as one that's been deleted is not likely to be a common scenario. Even so, just waiting a few seconds is usually long enough to try creating the table again. We can address this for a future release once the SDK comes out with a fix.

See Azure/azure-sdk-for-js#18865.

Verified this issue on the build main/20220212.2. Fixed.

The "undefined" part is still a little unpolished feeling, so I'll make one change to fix that.


requireOperatorBeforeLineBreak: doesn't work well for CallExpressions

See https://github.com/jscs-dev/node-jscs/issues/511#issuecomment-66161341.
Requires investigation. Check out the failing test: https://github.com/jdlrobson/node-jscs/commit/08f440372fb75e81ea6efb9d26a72a35fd5a5678.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/20941171-requireoperatorbeforelinebreak-doesn-t-work-well-for-callexpressions?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

. was removed as a binary operator so var defaultOperators = require('../utils').binaryOperators.slice(); doesn't include it in the rule

At this point only major and CST related bugs will be fixed.


Feature request: Theme star rating, filter

Choosing a theme is really difficult as there are too many to choose from - https://themes.gohugo.io
A star rating/review system and filtering by theme features will make it a lot easier for people to find a theme that suits them.

Please post theme site issues here:
https://github.com/gohugoio/hugoThemes/issues


[GSSOC'23]: adding timer feature in PCM quiz C++

Describe the Feature you want to end
adding timer feature in PCM quiz C++
Expected behavior
In this updated version, each question in the quiz will have a 20-second timer. When the quiz is displayed, the timer will start counting down from 20 seconds. If the player does not select an answer before the timer runs out, the next question will be displayed automatically.

@MohitGupta121 @TusharKesarwani sir kindly assign me this issue


Cross Iterator

This is the nutty one that's undefined and slow. The notion is to make graph-shapes more efficient, but it's unclear how that'll happen.

Moved to feature request forum: https://discourse.cayley.io/t/cross-iterators/205


[azurefleet] Change the name of the property

@rahuls-microsoft Could you help verify the type changes in this PR? Thanks!

Confirmed with @rahuls-microsoft about the type changes in the spec.

Hi, there is no breaking change, only documentation updates, so the line numbers changed.
Please approve and merge.


SWARM-1553 add parameter to skip all executions for multi-module projects

[yes] Have you followed the guidelines in our Contributing document?
[yes] Have you created a JIRA and used it in the commit message?
[yes] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[yes] Have you built the project locally prior to submission with mvn clean install?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

test this please

Thanks for the contribution @mzatko !

This appears to have broken the Maven plugin tests we have.
Could you take a look at the cause?

All 4 tests are passing for me, what is the failure or error?

@mzatko  See here https://github.com/wildfly-swarm/wildfly-swarm/pull/780#issuecomment-363099138

@kenfinnigan , @heiko-braun  I am afraid I will need some details about the failure as all tests pass without any failure or error

@kenfinnigan , @heiko-braun this is probably a dumb question but what are prerequisites for running tests in wildfly-swarm/testsuite/testsuite-maven-plugin? if I run it with mvn clean test I only get skipped tests
-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.wildfly.swarm.maven.plugin.MavenPluginTest
Tests run: 5, Failures: 0, Errors: 0, Skipped: 5, Time elapsed: 0.004 sec - in org.wildfly.swarm.maven.plugin.MavenPluginTest

Results :

Tests run: 5, Failures: 0, Errors: 0, Skipped: 5

If I recall it requires MAVEN_HOME to be set explicitly. Is that right @Ladicek ?

Yea the M2_HOME environment variable needs to be set.

yeah, that was it. Now I can see failures, it looks like there is assertion for max. 2 WARNINGs about mainClass deprecation. It was somehow missing in stop goal so I added it there too and now it is logged 3 times during 2 of tests...
Do you think it is OK just to change in MavenPluginTest.java
assertThat(count).as("There should only be 1 or 2 warnings").isIn(1, 2);
to
assertThat(count).as("There should only be 1 to 3 warnings").isIn(1, 2, 3);
?
thank you

I think that should be fine

I don't think it's intentional for the stop goal to print the warning, but then again, I don't think it's a big deal. So if we agree with that, then the proposed change to the test is the right thing to do.

So, I fixed it as proposed. Now even those additional tests are passing...

Awesome @mzatko, when you've pushed the changes and CI is green we will merge
Thanks again!

Thanks! I wonder if I have the Jenkins powers...

test this please


jitsi freezes

WARNING: impl.protocol.jabber.OperationSetBasicInstantMessagingJabberImpl.isCarbonSupported().775 Failed to retrieve carbon support.service-unavailable(503)
severe impl.protocol.jabber.InfoRetreiver.retrieveDetails().361 Cannot load details for contact<EMAIL_ADDRESS>service-unavailable.
then freeze.....
tried deb nightly and stable....
also tried the uncheck the box of tray/docker...
my desktop: ubuntu 15.10
/usr/lib/jvm/java-7-openjdk-amd64/jre/bin/java

before the content of post before this line is visible in the logs:
WARNING: impl.protocol.jabber.OperationSetBasicInstantMessagingJabberImpl.isCarbonSupported().775 Failed to retrieve carbon support.service-unavailable(503)

Incomplete bug report. Please see https://github.com/jitsi/jitsi/blob/master/CONTRIBUTING.md and https://jitsi.org/logs.

yeah, no.
i am not subscribing yet another mailinglist.
have a nice day figuring out yourself


Simplify/Create the AlertDialog API

It would be a good enhancement to create some common base classes/simple API for your AlertDialogs, so that much repeated, similar, mostly design boilerplate code can be eliminated from various places.
In the end we can go, for example, from this:
The entire DeleteWorkoutAlert.class
Plus
showDeleteWorkoutAlertDialog(BuildContext context) {
    RaisedGradientButton okButton = RaisedGradientButton(
      gradient: LinearGradient(
        colors: <Color>[Color(0xFF34D1C2), Color(0xFF31A6DC)],
      ),
      radius: 30,
      child: Text(
        "DELETE",
        style: buttonTextSmall,
      ),
      onPressed: () {
        deleteWorkout(workoutData.workout);
        Navigator.of(context).pop();
          Navigator.of(context).push(
                    MaterialPageRoute(
                      builder: (context) => CalendarPage(
                      ),
                    ),
                  );

      },
    );

    // set up the AlertDialog
    DeleteWorkoutAlert alert = DeleteWorkoutAlert(okButton);

    // show the dialog
    showDialog(
      barrierDismissible: false,
      context: context,
      builder: (BuildContext context) {
        return alert;
      },
    );
  }

To just these few lines of code
showDeleteWorkoutAlertDialog() async {
	final response = await showConfirmationDialog(title: 'Are you sure you want to delete this workout?');
	if(response) {
		// Delete workout
		// Navigate
	}
}

Like this we can eliminate some then redundant classes, and greatly cleanup the code that is inside the Pages calling for AlertDialogs

Great suggestion! The entire code base can definitely be refactored quite a bit

Can I Work On This Issue?? or is it Solved

@vivekdusad I am not sure if this was solved completely in the #97 PR. @jorre127 what do you think?

@KalleHallden @vivekdusad Hmm #97 did improve it by moving the alert to a mixin, but I don't think it's complete quite yet. currently. We still have separate methods to call different alerts like "showLogoutAlert", but this ideally should all be one method "showCustomAlert" or maybe a couple If we want really specific alerts. And then just customise the alert by passing the right properties like title, accept, cancel, body etc. So I don't think it's quite complete yet.

Okay nice then it's all yours if you still want it @vivekdusad

ok Thanks ;)

ok Thanks ;)

Did you start? I am already working on it since this morning. :/


Move @import in selectfield.scss to a separate SASS file

I am trying to use the mdl-selectfiled component im my own SASS workflow.
I have installed MDL and mdl-select field with Node JS and thus MDL is located under the node_modules folder, not under '../../bower_components/material-design-lite/src/variables' as specified in selectfield.scss. This means that it is not possible for me to use selectfield.scss in my own SASS workflow. If import directives were moved to a separate file, it will be possible to use selectfield.scss in any SASS workflow.
As a workaround, I've copied selectfield.scss into my own project and removed the import directives. This works great, but it would be best to use the component directly.

A SASS variable for the roo path would have been preferable, but that's not possible in SASS, see: https://github.com/sass/sass/issues/739
If you use webpack, then you can prefix import path with "~", as suggested here: https://github.com/webpack/css-loader/issues/12

regards

@leifoolsen, please check


Update NumpyRandomManager

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (c1b0af3) to head (92b8fb8).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #528   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           64        64
  Lines         1688      1692    +4
  Branches       196       196
=========================================
+ Hits          1688      1692    +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


qa_crowbarsetup: fail on manila instance failure

this should make error output more readable

+1

+1

:+1:


Feature: Magic Item and Spell Importer

As discussed in Discord, it would be helpful to have Magic Item and Spell importers based on the Monster Importer. I will start them now.

Closed with #692 and #681


Is the modal formatting supposed to break when you change the data-target?

I'm trying to run a modal through a foreach loop like shown in this post, but it's breaking my formatting for my head and body of the modal.

Hi there @marlonbrand0 ,
The example shown by you is used in a different way comparing to this one used be bootstrap and also us. If you want to implement it like that , take a closer look and take it step by step :)

`

×

DELETING THIS AFFECTS CLIENT PROFILES WITH THIS SELECTION

Are you sure?
Deleting this will remove this entry from all client profiles with this selected

Close
<a href="" class="btn btn-danger" role="button">Delete

`
All posts I see implement using a different modal id. Is it alright if you tell me which part is not implementing it the bootstrap way?

First thing would be that uses inline JS, try to use it exactly by default changing just texts if you want to run like our example, you find it in general_components.html page,

I updated my code. Forgot to include the top div hahaha


URL parameter "/?file_path=" added to load local model files

Hello!
I've been working with Netron for some time and there's a feature that I was willing to add: loading local model files, hosted on the server, using an URL parameter.
This is similar as the currently supported /?url parameter, allowing us to load a model file from the Internet. The /?file_path= is just as the same but tries to load it from the server local files instead.
By default, this feature is disabled (eg. no file will be accessed on the server) to keep the current Netron behavior. This can be enabled using a new parameter called --allow-local-files when starting the server.
At the moment, loading a local file on Netron requires a manual action (drag/drop or a click on the "Load model" button). Using an URL parameter instead allows scripts to automatically load them on Netron, this is the main benefit of my PR.

@0hDEADBEAF can you explain an example how this would be used.

Well, if I'm not mistaken, using the existing file argument, you can only serve one local file at a time. If you want to serve another file, you'd have to either restart the server with that new file (meaning that the previous file wouldn't be able to be accessed anymore from the server) or start another server instance with that new file. My PR allows us to dynamically load any local model file from the server without restarting it. This is especially useful when several people need to work on the same server instance that has many private models.

Would using a generic web server work for this scenario? For example, what are example steps and commands you or your team would use and what are you trying to optimize for?

I'm not sure to properly understand what you mean by "generic web server" here. However, I can give you few details about how the features I've added in this PR would be useful to us.
We generate a lot of models dynamically, models that we want to keep private (and therefore we cannot expose them to the Internet and access them through Netron with the /?url parameter). Those models are hosted on a shared server on which we have a Netron server running. Several people access the Netron interface through that server (http://the_server:8080).
We want everyone to be able to work on the numerous models that are automatically generated as the server is running. We don't want to restart the server each time a new model is generated as this is a process we deal with very often.
Using such feature is also pretty handy when it comes to share a model to someone: we just need to share the link (http://the_server:8080/?file_path=/models/auto_model178765.onnx) instead of passing the model file itself to everyone who needs it. Is is especially useful since the models are hosted on a server, which mean we would need to download them through (s)FTP first, then uploading on Teams to share it to others. It is much more convenient to share a link instead.
Moreover, if a model file gets updated, it just needs a F5 refresh on the browser instead of a server restart or a heavy process such as downloading the updated model through SFTP, sharing it to everyone else and making sure everyone is correctly using the updated version.

This is a workaround that could probably work indeed. However, it feels a bit inconvenient to run two servers when we could just use one, such as my PR allows us to do so 🙁
I've fixed the remaining Pylint warnings 🙂

The trade-off is between "a bit inconvenient" for a few users capable of running a web server and the complexity and security concerns this creates for many other users. There might be other approaches worth considering, like running a single server and explicitly adding models to that instance via an API, especially if the server side evolves into doing more than just serving files. Currently evolving the app into a full web server and addressing related issues is not a priority. There are many well-built and maintained options available to accomplish this.


helper-cli: Allow filtering the listed packages by severity

Merge despite the failing tests (are unrelated).


complete task 10 - CORE: View a separate page for each topic with a list of related articles

Task 10:

able to select different topics
able to re-render the articles based on selected topic
able to share url and only fetch the articles associated with that topic

Thanks for the feedback Ali 😁 ill go check out the backend stuff when i finished with my tasks.


Custom UIActivityIndicatorView

Hi, Would be awesome if we could have custom UIActivityIndicatorView for the refresh control

YES, the current design supports this feature, but exposes no direct API to do this. I'll try to implement this feature, providing better interfaces. Thanks for your advice.