[MVP-1519] Intercom chat window

Use mock data for now(will remove mock data in this pr when embed firing API):
Incoming messages:
Should be a gray bubble
Show username
Show time
Outgoing messages (messages from you):
Blue bubble
Should not show username
Show time
Send message:
The message should not have a character limit
It should be scrollable if it is a long message
The Send button should be gray when the user has nothing in the message box.
The Send button should be blue when the user has sent something in the message box.
will working on the alert setting page next, so the alert setting button is disable now.
Test:

Story: https://notifi.atlassian.net/browse/MVP-1519
https://notifi.atlassian.net/browse/MVP-1558
https://notifi.atlassian.net/browse/MVP-1564
https://notifi.atlassian.net/browse/MVP-1560
https://notifi.atlassian.net/browse/MVP-1563
https://notifi.atlassian.net/browse/MVP-1561

Hey Hao, can you please update the screenshot/video when you are done making the updates?

Hey Hao, can you please update the screenshot/video when you are done making the updates?

updated with the screenshot to show the padding change on the message, added Virtuoso but haven
't done with the set up yet, will do and test more when connecting with backend API.

Hey Hao, can you please update the screenshot/video when you are done making the updates?

updated with the screenshot to show the padding change on the message, added Virtuoso but haven 't done with the set up yet, will do and test more when connecting with backend API.

The virtual container looks good (for now), you can always copy and paste the same mock data to make a long array to test.
Also if these are easy updates:

Update the font of the placeholder
If there is a way to increase the padding more between the message and message box. You will see that when you type something into the message box, the message overlaps a bit around the blue border of the message box.


LICENSE

Is MIT License required by the codeathon?
Otherwise we could switch to Artistic 2.0 which is used by mia and other packages. This would be better compatible.

MIT License was the default for all projects. Perhaps I can update it.


Microsoft Azure Storage Explorer

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.22.0
Regression From
No response
Architecture
x64
Storage Explorer Build Number
No response
Platform
Windows
OS Version
No response
Bug Description
When I load the software, a message pops up (see attached)
Steps to Reproduce
1 - Launch Microsoft Azure Storage Explorer
2 - Click on Blob Containers
3 - After few seconds will pop up an error message

Actual Experience

Expected Experience
No response
Additional Context
No response

@bigojessie Is this error message the same as the one you shared in https://github.com/microsoft/AzureStorageExplorer/issues/5302? If yes, let's close this one and keep the conversation there since it has more detailed information.


Validate BioData Catalyst logos

Is your feature request related to a problem? Please describe.
BioData Catalyst® has recently updated its logo and name usage.
https://www.biodatacatalyst.org/consortium-resources/bdcatalyst-branding/
Describe the solution you'd like
Validate that Dockstore is complying with the new requirements.
Describe alternatives you've considered
NA
Additional context
NA
┆Issue is synchronized with this Jira Story
┆Attachments: BDC-Logos.zip | BioDataCatalyst-StyleGuide.pdf | Color Icons.zip | Gray Icons.zip
┆Fix Versions: Dockstore 1.14
┆Issue Number: DOCK-2291
┆Sprint: 104 - Blue Nile
┆Issue Type: Story

Confirmed that logos are correctly updated.
[Domain] - https://qa.dockstore.org
Webservice - fb9ebdf
UI - 2.10.1-107-ga76609e6
Compose Setup - 6f68328
Deploy - 1.13.2-31-g140c057
cwlParsingLambdaVersion: n/a
wdlParsingLambdaVersion: n/a
nextflowParsingLambdaVersion: n/a
galaxyParsingLambdaVersion: 0.0.6
checkUrlLambdaVersion: n/a


Create definitive visuals (skybox, ambient light, toon shader) (Godot)

[x] Make graphics settings customizable

Task finished on https://github.com/decentraland/godot-explorer/pull/407


Update ?

Is there any chance, the nuget package will be updated to current (2024) RDkit version ? We must some detect-bond futures not present in the 2021 release ...

https://www.nuget.org/profiles/Priyanku_Biswas
RDKit C sharp wrapper for RDKit Release_2023_03_2

Sorry, I am not active in this project.


Bug: undefined is displayed for "Currently Assigned" counter, when API  token has expired

Which version of Node.js are you running (node -v)?
v6.9.5
Which version of urcli are you running (urcli -V)?
1.5.1
What behavior were you expecting to see?
The "Currently Assigned" counter to display a number.
What was the actual behavior?
"Currently Assigned" counter displays undefined.
Describe the steps needed to reproduce the problem?

Wait for token to expire
Wait for review to be assigned.

What is your Operating System (e.g., Linux Ubuntu, MacOS, Windows)?
Windows 10

Hi @Izodine. Thanks for reporting. The latest version of urcli is 2.5.0. Could you update it and check if the error persist?

Hi @Izodine, thanks for reporting this. Since this is a v1 issue, I'm going to close this, but please open a new issue if it persists in v2.


Flash always disabled on Linux Mint 20

Flash is always disabled
chrome://version/
Chromium | 84.0.4147.135 (Developer Build) built on Ubuntu focal/20.04, running on Linuxmint ulyana/20 (64-bit)
Revision | c42bd09b3f24da1698d71d3b4f57402137163566-refs/branch-heads/4147@{#1102}
OS | Linux
JavaScript | V8 8.4.371.23
Flash | (Disabled)
User Agent | Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/84.0.4147.135 Safari/537.36
Command Line | /usr/lib/chromium/chromium --show-component-extension-options --enable-gpu-rasterization --no-default-browser-check --media-router=0 --flag-switches-begin --flag-switches-end --origin-trial-disabled-features=MeasureMemory --file-url-path-alias=/gen=/usr/lib/chromium/gen
Executable Path | /usr/lib/chromium/chromium
Profile Path | /home/me/.config/chromium/Default
Flash running on other browsers like firefox with no problem

Why do you need to run Flash? It is deprecated:

Starting from Chrome 76 and Firefox 69, Flash is disabled by default and browsers do not even show a prompt to activate Flash content. Users who want to play Flash content need to manually set a browser to prompt for Flash content, and then during each browser session enable Flash plugin for every site individually. Furthermore, browsers show warnings about the removal of Flash entirely after December 2020. Microsoft Edge based on Chromium will follow the same plan as Google Chrome.[47]
Google Chrome will block Flash plugin as "out of date" in January 2021 and eventually remove it from the source code.[48] Also in December 2020 Flash support will be completely removed from Firefox.[49] Apple will drop Flash Player support from Safari 14 alongside the release of macOS Big Sur.[50] In a move to further reduce the number of Flash Player installations, Adobe announced plans to add a "time bomb" to Flash to prevent existing installations past End-of-Life date, to prompt users to uninstall Flash, and to remove all existing download links for Flash installers.

There are still lots of games for flash.

Even so with Chromium phasing it out I'm not sure what you expect us to do about it. I don't think anyone here cares enough about Flash to continue trying to support it.

First question is, should it work or not?
In last "normal" chromium I used, it worked.

Flash is always disabled
chrome://version/
Chromium | 84.0.4147.135 (Developer Build) built on Ubuntu focal/20.04, running on Linuxmint ulyana/20 (64-bit)
Revision | c42bd09b3f24da1698d71d3b4f57402137163566-refs/branch-heads/4147@{#1102}
OS | Linux
JavaScript | V8 8.4.371.23
Flash | (Disabled)
User Agent | Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/84.0.4147.135 Safari/537.36
Command Line | /usr/lib/chromium/chromium --show-component-extension-options --enable-gpu-rasterization --no-default-browser-check --media-router=0 --flag-switches-begin --flag-switches-end --origin-trial-disabled-features=MeasureMemory --file-url-path-alias=/gen=/usr/lib/chromium/gen
Executable Path | /usr/lib/chromium/chromium
Profile Path | /home/me/.config/chromium/Default
Flash running on other browsers like firefox with no problem

sudo apt-get install pepperflashplugin-nonfree
sudo su
sudo sed -i -e '/Exec=/usr/bin/chromium/s/$/ --ppapi-flash-path=/usr/lib/pepperflashplugin-nonfree/libpepflashplayer.so --ppapi-flash-version=<IP_ADDRESS>3 /' /usr/share/applications/chromium.desktop
Launch chromium enable flash chrome://settings/content/flash
Test flash https://helpx.adobe.com/flash-player.html

sudo sed -i -e '/Exec=/usr/bin/chromium/s/$/ --ppapi-flash-path=/usr/lib/pepperflashplugin-nonfree/libpepflashplayer.so --ppapi-flash-version=<IP_ADDRESS>3 /' /usr/share/applications/chromium.desktop

That works. Thanks very much!


tqdm-contrib dependancies

hello team from Berkeley -- good work and very interested to see this.  The python3 tqdm packaged for GNU-Debian-Ubuntu 22.04 LTS (jammy) does not have a contrib section? With a quick look using standard packaged python3, I saw no way to satisfy the the tqdm.contrib.concurrent thread_map feature extra dependency. In my test for data download I got enough bandwidth with a single stream.  ( ~18MB/sec at OSUOSL)

Thanks for reporting.
Let me try to understand better.
Installing geo-benchmark has dependencies already satisfied for tqdm and it's from your ubuntu tqdm package?
does `pip install tqdm' solve the problem?

some context :  installing python .. on GNU-Debian-Linux, there are python3 packages in dot-deb format. Most of those packages are very closely in sync with modern updates, especially numpy pandas and other popular data-science. So it is desirable to stay close to these standard installs for maximum portability.  It is true that there is no way that Debian or Ubuntu python packages can be "cutting edge" it is too much work for the packagers. Contrast to the benefits of python dot-deb.. integration into a binary ecosystem with many tens of thousands of applications available, tested and compatible. For the security-minded, signed packages from a known source are valuable or even required.
You as project implementer choose your own contents, to meet your own goals -- 100% agree --  "do-ocracy" speaks.  And, when you make choices for your package dependencies, you can consider the compatibility and portability of the library.
I am telling you today that on a popular and stable Linux OS, the tqdm.contrib.concurrent is not available, including thread_map  on the LTS version 22.04.  Maybe I missed it ?

we're considering moving the dataset to hugging face so this would mostly solve the problem once we change to a new download script.
I mainly need to know. Is there a quick workaround for your situation? I.e. if you do a manual pip install geo-benchmark

my situation is -- I am senior editor for a Linux distribution on maps, mapping and spatial data;  the purpose of the exercise is to use FAIR and Reproducible Science principles to recreate the results you will show with GEO-Bench -- not on the cloud, not on HuggingFace, but reproduce with authentic inputs.
It took me ten minutes total to remove the global environment variable, remove the tqdm thread_map and revert to a single-queue, which I did.
In my case, I ran on a VM in the Oregon State University Open Source Lab (OSUOSL) getting about 18MB/sec download speed for the data.  That worked fine with single stream I/O.

better ref  'Toward Foundation Models for Earth Monitoring: Proposal for a Climate Change Benchmark' dec2021  3.1 Benchmark -

better ref 'Toward Foundation Models for Earth Monitoring: Proposal for a Climate Change Benchmark' dec2021 3.1 Benchmark -

This is us too. We'll reference it in the camera ready of the paper.

#from geobench import config

I don't understand why config.GEO_BENCH_DIR is an issue

not on the cloud, not on HuggingFace

The benchmark data will likely be hosted on huggingface soon. The download script will download from there instead of Zenodo so it will solve the problems. You won't have to "run in the cloud".

getting about 18MB/sec download speed for the data

zenodo is not consistent on download speed. We frequently are throttled at ~300k. the parallel download is a workaround to get a ~10x speedup.

from tqdm import tqdm #from tqdm.contrib.concurrent import thread_map

I'll switch to single thread. This will solve the problem (with occasional slow download speed until we move to hugginface)


multiprocessing fails in windows - cannot pickle thread.lock

During Openroast 1.2 development, I have assessed that v0.2.0 still fails to operate in a windows cx_Freeze application, as well as from a standard python windows install (as a non-msi, non-frozen python app).
This is because auto_connect() spawns a thread to try to repeatedly connect to hardware, and when the hardware is found, two processes are started.  Starting the processes in Windows forces a pickle of the freshroastsr700 object instance, which now has an active thread running (the auto_connect thread), which cannot be pickled.
I have a workaround that I am developing in a v0.2.1 branch for eventual submission.

Fixed in v0.2.1.


Multi-language code fences in Statiq Docs

Allow defining Markdown code fences (and/or HTML code blocks) for multiple languages and then automatically tabifying them - not sure if this should be a module (by combining adjacent code fences) or shortcode (by declaring contained code fences are part of a group)

If you did it with a Shortcode, would it be feasible to implement it as a general purpose tab system so it is not limited to only code blocks?

Maybe... the biggest problem is that most CSS libraries (Bootstrap, Foundation, Tailwind, etc.) don’t apply only to plain semantic HTML without classes. They all require either a certain “flavor” of markup and/or specific classes. The most portal solution would probably be to ignore any CSS framework in use and instead generate markup and CSS classes just for this.


Add pub (Dart) as supported package manager

Add support for the pub dependency manager to the analyzer module.
https://github.com/dart-lang/pub
Possible test project using pub: https://github.com/sass/dart-sass

Key question: Is it possible in Dart to get a dependency tree that includes package names, version, scopes licenses and source locations?
the package dependency tree can be generated using
pub deps --style=list

which provides the following output (where NameOfPackage is the name of the package this command is run for):
Dart SDK 2.2.1-dev.4.0.flutter-None
Flutter SDK 1.4.9-hotfix.1
NameOfPackage 0.8.0

dependencies:
- protobuf 0.13.11
  - fixnum ^0.10.9
- flutter 0.0.0
  - sky_engine any
  - vector_math 2.0.8
  - collection 1.14.11
  - typed_data 1.1.6
  - meta 1.1.6

dev dependencies:
- test 1.6.1
  - async >=1.13.0 <3.0.0
  - shelf_static ^0.2.6
  - analyzer >=0.26.4 <0.37.0

transitive dependencies:
- args 1.5.1
- async 2.1.0
  - collection ^1.5.0
- yaml 2.1.15
  - charcode ^1.1.0
  - source_span >=1.0.0 <2.0.0
  - string_scanner >=0.1.4 <2.0.0
  - collection

(note: I truncated the output).
Another option is to look into the pubspec.lock (https://dart.dev/tools/pub/glossary#lockfile) file.
Unlike the pubspec, which only lists immediate dependencies and allows version ranges, the lock file comprehensively pins down the entire dependency graph to specific versions of packages.
A lockfile ensures that you can recreate the exact configuration of packages used by an application.
The package hosting location is also stored in the pubspec.lock file.
cat pubspec.lock

Generated by pub
See https://www.dartlang.org/tools/pub/glossary#lockfile
packages:
  async:
    dependency: transitive
    description:
      name: async
      url: "https://pub.dartlang.org"
    source: hosted
    version: "2.0.8"
  dart_config:
    dependency: transitive
    description:
      path: "."
      ref: HEAD
      resolved-ref: e26916e095244a7e5ea61315b030d298d127ed26
      url: "https://github.com/MarkOSullivan94/dart_config.git"
    source: git
    version: "0.5.0"
  dart_style:
    dependency: transitive
    description:
      name: dart_style
      url: "https://pub.dartlang.org"
    source: hosted
    version: "1.2.2"

(note: I truncated the output)
packages are installed to
~/.pub-cache/hosted/pub.dartlang.org/
e.g. the package contents 'async' with version '2.2.0' will be stored in
~/.pub-cache/hosted/pub.dartlang.org/async-2.2.0
licenses are part of package and the file name is always LICENSE. As far as I know, a LICENSE file is a mandatory requirement to publish packages on pub.dartlang.org.
cat ~/.pub-cache/hosted/pub.dartlang.org/async-2.2.0/LICENSE

One more thing to consider for supporting Flutter packages
A Dart package which is a flutter plugins can contain native iOS and/or Android code. Flutter is currently using Cocoapods on iOS and Gradle for Android to install the native dependencies for each platform. While Gradle is already support, cocoapods are not yet supported.

Done in #1539.


Add support for classifiers for jar

Many builds with natives have support for maven classifiers (used to provide separate jars depending on OS). This reduces download size etc. Would be good to add this as more and more platforms are added.

Seems reasonable, do you have an example I can look at?

Sure, https://github.com/netty/netty-tcnative/blob/master/pom.xml is 1 example from Netty.
You can check the classifier related parts.
https://geekzguru.wordpress.com/2008/02/16/maven-tag/ is an article explaining the maven tag.
Sorry, not familiar with sbt to help on that though.

Yes, I will look how to provide alternative jars with classifiers.

I made the build.sbt changes and it looks they are producing correct jars.  Verified by "publishLocal". With the next release I will start publishing them to maven central.

zstd-jni-1.3.4-11 just landed on Maven Central with the classified jars.


changes for Monocular Image data

Hey,
Thanks for open sourcing this awesome work.
I wanted to train your C+L  fusion model on my own dataset but i have a monocular camera, thus only one image view. Can you point me to what changes i would have to make to train the model successfully.

Hi @RG2806,
I think BEVFusion will still work with monocular camera + LiDAR inputs. The main difference between single camera v.s. surrounding camera is that the img variable will be 1x3x256x704 instead of 6x3x256x704, lidar2camera will be something like 1x3x3 instead of 6x3x3, similar for other variables.
Best,
Haotian

Hi @kentang-mit
Thanks, also the size of nuscenes images is 1600 × 1200.
Then how do you arive at 256×704?
My dataset has images of size 4096×3000. Apart from default.yml. how would i use it?

I'm just using these numbers as examples. You can have any resolution you want. Simply changing image_size and augment_2d parameters in this file will be a good idea.
Note that this line means: the training time resolution is sampled in the range of 0.38x to 0.55x of the original resolution, and the test time resolution scaler is sampled from the range of 0.48x to 0.48x of the original resolution.

Thanks for the clarification


feat: next polyfils (including global fetch)

Works like a charm. Big thanks to @Vadorequest

@Vadorequest, I'm gonna merge this into alpha which should auto-release onto NPM. Would you mind testing this out when you get a chance?
I've also added you as a co-author to this PR :D


Generated file/module names inconsistency?

Currently, for:

file Shapes.proto with package named interproc
we get purs file Shapes.Interproc.purs with module name Interproc.Shapes

Was it intended to be this way?

I like Shapes.proto and Shapes.Interproc.purs to be close to each other in alphabetical order for file browsing.

I currently put proto files in a separate folder, and generated files into the src, also I usually have files sorted by extension files first in IDE exlorer.
Anyway, to me it just seems kinda strange to see file Shapes.Interproc.purs with module Interproc.Shapes inside.
I believe you also prefer to use small caps so purs file is called shapes.Interproc.purs.
I'm not sure how to make it better, I just pointed to this confusing moment in naming.


urgent linux port needed

this should be a crossplatform masterpiece

see #8

@sctech-tr Do same steps that described in https://github.com/face-hh/wattesigma/issues/7 but choose the Linux release

ok i'm working on it


[BUG]VM created by VM Template with Access Credentials use the same secret

Describe the bug

To Reproduce
Steps to reproduce the behavior:

Create a VM A
Add Access Credentials to VM A
Generate VM Template
Create VM B via the VM Template
The VM B created by VM Template use the same secret to VM A

Expected behavior

VM created by VM Template could create secrets correctly
Support bundle

Environment:

Harvester ISO version:
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context
Add any other context about the problem here.

Verified this bug has been fixed.
VM created by template will not have any Access Credentials.
Test Information

Environment: qemu/KVM 3 nodes
Harvester Version: master-75cc3ced-head
ui-source Option: Auto

Verify Steps:

Follow Steps to Reproduce in https://github.com/harvester/harvester/issues/1989#issue-1164669600
The VM in step 5. should not have any Access Credentials


bug: import functions from h3 with relative path, breaks with pnpm and no shamefully hoist

Environment
tried with<EMAIL_ADDRESS>Reproduction
https://stackblitz.com/edit/github-bshclz?file=.nuxt%2Ftypes%2Fnitro-imports.d.ts%3AL128
Describe the bug
Some declarations use just 'named' imports, which cause resolution problems when the project is restored with pnpm and no shamefully-hoist as the dependencies are not really present in the node_modules.
This includes files like nitro-imports.d.ts (h3) nitro-nuxt.d.ts (nuxt/schema, h3, ...) nitro-config.d.ts (defu) nitro-routes.d.ts (nitropack)
Additional context
I partially tried to fix this in PR 2207, but noticed since then that there are a lot more of occurrences. I find this strange because of nuxt/nuxt 14146.
With this problem, I can not create server handler with defineEventHandler without type problems.
Logs
No response

This issue should be resolved in the latest version of Nuxt with https://github.com/nuxt/nuxt/pull/26085.

@danielroe what about nitro usage alone?

Nitro usage alone is fine - the issue IIRC is that Nuxt is using nitropack types, but TS can't find nitropack .

thanks for confirming. then let's track for nuxt.

btw related discussion for nitro itself (direct import types don't work today with hosting): https://github.com/unjs/nitro/issues/2232


IATP: Request a self-issued ID token from the STS

Feature Request
A participant agent makes an OAuth request to a Secure Token Service (STS) to obtain a self-issued ID token, containing an access_token, which in turn contains auth scopes (#3492) can subsequently be used to request credentials.
Which Areas Would Be Affected?
iam
Why Is the Feature Desired?
Adoption of the IATP specification

closing as it is a duplicate of #3493


TUISpec

coverage: 60.968% (+8.4%) from 52.597%
when pulling f946c0f301ecf0344b45b42b41e2da1c4f6f550d on se-09-GUI-Libraries
into 92bc0d15b190e8b43009c6a8265f077400b140c3 on main.


Turned the respawn command into 2 seperate commands for easier usage

Instead of having 1 command to respawn the player in place there's now 2. One that respawn him in place "/Respawnstay" and one that respawns him in the normal civilian spawn "/Respawn"

"stay" should be the default /respawn behavior. The way it was done in Clockwork is simply unacceptable.

Sorry but no, the current respawning behavior is by design, and is not an oversight.


EE-1207: make definition of system account AccountHash consistent

This change will change the AccountHash bytes of the system account to:
blake2b(b"system0")

(as defined by AccountHash::from_public_key)

bors r+


Table parsing needs a refactor

The function that parses table is badly written and buggy, it should be remade before 3.0, I'll fix it myself sometime soon

Turns out the tables aren't bugged themselves, and something bugged was making tables seem bugged.
Thus, there is no longer urgency in refactoring the function

maybe there is no need to do this in hindsight


Remove SimpleRequest::breakCycle

This patch removes the need for Request objects to provide a default
cycle-breaking value, instead opting to return llvm::Expected so clients
must handle a cycle failure explicitly.
Currently, all clients do the 'default' behavior, but this opens the
possibility for future requests to handle failures explicitly.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@DougGregor In these tests, since cycle breaking is an external action, the cycle-break result is never cached, and the final test always shows (dependency not evaluated). Should that be what we test? That's a functional change, now.

Yes, it's fine to test for (dependency not evaluated); we don't need to record that a cycle happened beyond the evaluator's reporting mechanisms.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


Pass all classes to the component, not only root

Hi, I have a code below, and in this case, minHeight:32 will be added to styles on the page but not used by the component.
I propose to pass all classes to component
What do you think?
import {Tab} from '@mui/material';
import {withStyles} from 'tss-react/mui';

const StyledTab = withStyles(Tab, (theme: Theme, _params, classes) => ({
  root: {
    display: 'flex',
  },
  labelIcon: {
    minHeight: 32,
}));

Hi @osa911,
Thanks for the PR.
I am not sure I understand, why just not doing:
const StyledTab = withStyles(Tab, (theme: Theme, _params, classes) => ({
  root: {
    display: 'flex',
    minHeight: 32,
  },
  labelIcon: {
    minHeight: 32,
}));

?

I found in mui types this explanation related to Tab.
For different components this is different.
export interface TabClasses {
    /** Styles applied to the root element. */
    root: string;
    /** Styles applied to the root element if both `icon` and `label` are provided. */
    labelIcon: string;
    /** Styles applied to the root element if the parent [`Tabs`](/api/tabs/) has `textColor="inherit"`. */
    textColorInherit: string;
    /** Styles applied to the root element if the parent [`Tabs`](/api/tabs/) has `textColor="primary"`. */
    textColorPrimary: string;
    /** Styles applied to the root element if the parent [`Tabs`](/api/tabs/) has `textColor="secondary"`. */
    textColorSecondary: string;
    /** State class applied to the root element if `selected={true}` (controlled by the Tabs component). */
    selected: string;
    /** State class applied to the root element if `disabled={true}` (controlled by the Tabs component). */
    disabled: string;
    /** Styles applied to the root element if `fullWidth={true}` (controlled by the Tabs component). */
    fullWidth: string;
    /** Styles applied to the root element if `wrapped={true}`. */
    wrapped: string;
    /** Styles applied to the wrapper element of `icon` if `icon` is provided. */
    iconWrapper: string;
}

Why do you accept only root?
So in my project, I use root and labelIcon together.
Let me show you a full example:
const StyledTab = withStyles(Tab, (theme: Theme, _params, classes) => ({
  root: {
    display: 'flex',
    textAlign: 'left',
    textTransform: 'capitalize',
    paddingLeft: 0,
    paddingRight: 0,
    flexDirection: 'row',
    justifyContent: 'flex-start',
    minHeight: 'inherit',     <--------
    '&:not(:last-of-type)': {
      marginRight: theme.spacing(3),
    },
    [theme.breakpoints.up('md')]: {
      padding: 0,
      paddingLeft: theme.spacing(3),
      '&:not(:last-of-type)': {
        marginBottom: theme.spacing(2),
      },
    },
  },
  labelIcon: {
    minHeight: 32,       <--------
    transition: theme.transitions.create('color'),
    [`&.${classes.selected}`]: {
      color: theme.palette.primary.main,
    },
    '&:hover': {
      color: theme.palette.primary.main,
    },
    '&:active': {
      color: theme.palette.primary.dark,
    },
  },
}));

Hi @osa911,
I took the time to look into it but things seems to be worked as they supposed to:
https://user-images.githubusercontent.com/6702424/183224718-10f18fc3-6afc-4f6f-9263-45a8f041ec3a.mov

We have the correct styles completion for the <Tab /> component and the correct classes names  are passed to the underlying classes prop of the <Tab /> component.
Feel free to create a test repo that clearly exhibit what is the expected behaviour and what's not rendering as you expect.
If you do, please show how you expect withStyles to behave  without using withStyles but directly the classes props.
Best regards,

Sorry, my fault
I'm using version "4.0.0-beta.3"
And if you will use Tab like below, you will see the issue;
import Tab from "@mui/material/Tab";
import MenuIcon from '@mui/icons-material/Menu';

const StyledTab = withStyles(
  Tab,
  (theme, props) => ({
    "root": {
      "height": 10,
      "backgroundColor": "red"
    },
    "labelIcon": {
      "height": 150,
      "backgroundColor": "blue"
    }
  })
);

<StyledTab label="Just label" />
        <StyledTab label="label and icon" icon={<MenuIcon />} />

P.S. I'm using version 4 because with version 3 I had the issue with sharing cache for mui on next SSR.

Btw with my proposal, it works as expected.
Probably we need to check mergeClasses() again...

Thanks for the explanations, make sence.
Thanks for the example, I am looking into it (but I am in vacation so I work only when I got some time to kill)
PS: Note that TSS v4 is only compatible with the latest release of MUI: 5.9.3

Afer investigations, I found out this this is a problem that needs to be addressed on the MUI's side.
I have opened a PR:  https://github.com/mui/material-ui/pull/33866

Thanks
Probably I can close this PR, right?

Yes,
Even if it's not merged your input have been of great help for finding out a big problem in v4 before it's released.

Hi @osa911,
Just to keep you up to date,
The PR I opened on MUI won't be merged. It's not deemed important enough to be fixed (for understandable reasons).
Nevertheless I will be able to fix it on TSS side by using classes.root instead of classeName in withStyles (when there is a classe that is not root )
I'll ping you when it's implemented.

Hi @osa911,
It should have been resolved in 4.0.0-beta.4


LUN-274: Select profession on initial start

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword LUN.

There are still some comments that are hidden and are not addressed yet (maybe also on purpose, not sure)

I think I addressed all comments. Also fixed the bottom margin @f1sh1918 mentioned.


Update README with initial dev instructions

Context
The current Readme is the default FastAPI/NEXT readme that came with the template we used, so we need to erase this Readme and put our initial configuration details in here
Definition of Done

Readme only contains information related this project and how to build and run it locally

Engineering Details
We need to have the following:

A new title that represents the working title/name of our application
a short summary of how the code is laid out (frontend, backend, streamlit)
local environment setup and install instructions*
how to run the app locally

*This will likely change soon as we get ENV variables setup to deploy our app and we'll also likely be setting up Docker to help manage these as well

This is handled via @agennadi 's PR and #66


Document setting up mock models with multiple rows

Currently the examples are very limited and it is not clear how one could have multiple rows in their mock data. Neither the readme nor the documentation seem to cover how that would be possible.
For example the below example only describes the usage of a single row
var UserMock = DBConnectionMock.define('users', {
		'email'<EMAIL_ADDRESS>		'username': 'blink',
		'picture': 'user-picture.jpg',
	}, {
		instanceMethods: {
			myTestFunc: function () {
				return 'Test User';
			},
		},
	});

Whereas directly below we have a WHERE example which is completely unclear on how to actually reproduce. This is because there is neither information on adding multiple rows, nor is the example condition and fields valid for the above row.
UserMock.findOne({
	where: {
		username: 'my-user',
	},
}).then(function (user) {
	// `user` is a Sequelize Model-like object
	user.get('id');         // Auto-Incrementing ID available on all Models
	user.get('email');      //<EMAIL_ADDRESS>Pulled from default values
	user.get('username');   // 'my-user'; Pulled from the `where` in the query

	user.myTestFunc();      // Will return 'Test User' as defined above
});

I met the same problem when init model with multiple records. I have tried to pass the data as an array, but the result from query was wrong. Does any way solve this?

Any updates on this?
I can't use this lib without ability to define a collections of mock instances.... *(

I am facing the same problem. hope to get it fixed soon.

Took me a few hours to find but I found $queueResult aswell. My implementation for my test was like this:
jest.mock('./../../models/my_news_model', () => () => {
  var SequelizeMock = require('sequelize-mock');
  var dbMock = new SequelizeMock();
  var NewsModel = dbMock.define('news_model');
  NewsModel.$queueResult([
    NewsModel.build({
      id: 1,
      title: 'The Test Title',
      description: 'This is the news story',
      basemedia: "5cf5069bd8ad580bf03728a7.jpg",
      link: 'https://newsfeed.com/somestory',
      createdAt: '2019-06-03 13:13:45',
      updatedAt: '2019-06-03 13:13:45'
    }),
    NewsModel.build({
      id: 2,
      title: 'The Test Title 2',
      description: 'This is the news story 2',
      basemedia: "5cf5069bd8ad580bf03728a72.jpg",
      link: 'https://newsfeed.com/somestory2',
      createdAt: '2019-06-02 13:13:45',
      updatedAt: '2019-06-02 13:13:45'
    })
  ]);
  return NewsModel;
});

There are still many unanswered questions, and this been the plug and play package I was hoping for is not the case


Supporting API to list all log analytic workspaces

List All Log analytic workspaces
I need support for below API
https://learn.microsoft.com/en-us/rest/api/loganalytics/workspaces/list

Hi slifer2015! Good news, we already have an SDK for this API. WorkspacesClient.NewListPager method in the armoperationalinsights module hits the api you referenced your comment.
Other Log Analytics management plane functions can also be found in that module. There have been name changes over the years, so sometimes it's hard to find. Please let me know if there is anything else you are looking for.


Doesn't install godoc

I am brand new to Go (installed it today...), and I see that GVM doesn't install other utilities such as godoc. How do I install them? If I have do an apt-get install golang-stable to get them, then how does GVM help?

Makes sense to install godoc once under the system version, since it's a database, not a versioned package and is therefore immune to the need to be managed as a build dependency.
Installing under system $GOPATH makes godoc command universally available for reference during local development, regardless of the active gvm-managed go version.
Above could be default behavior when running gvm-installer script, with a command flag option added to install without godoc.
Also for reference for future googlers, current install path is
$go get golang.org/x/tools/cmd/godoc

ref New import paths for Go sub-respositories

I will likely close this issue in the near future unless I hear otherwise.

I met the same issue. Since I use Go1.18 under GVM to write generics codes, my system go binary is go1.13.15. The Godoc is for system go binary and cannot support new generics functions.
I didn't find a way to use godoc under GVM specific version.


Add multiple files support.

It can be useful to target several files, I don't know if it is in the philosophy of the library.

@jdalton fixed

@jdalton do you agree?

I'll need some time to seriously mull it over. Christmas Eve is probably not the time. 🎅

@jdalton take your time, Merry Christmas!🎅

This is also something I'm interested in for some of my projects. I was actually planning on making a pull request until I saw this one.
Is there something that concerns you about supporting multiple files?

@jdalton did you take a look?


v1.13 Backports 2024-05-24

[ ] #32084 (@pchaigno) :warning: resolved conflicts
[x] #32328 (@joestringer)
[ ] #32464 (@darox) :warning: resolved conflicts
[x] #32553 (@michi-covalent) :warning: resolved conflicts
[x] #32564 (@jibi) :warning: resolved conflicts
[ ] #32548 (@squeed)
[x] #32570 (@mhofstetter)
[x] #32600 (@mhofstetter) :warning: resolved conflicts
[ ] #32302 (@marseel) :warning: resolved conflicts
[ ] #32629 (@antonipp) :warning: resolved conflicts

PRs skipped due to conflicts:

#32307 (@ferozsalam)
#32450 (@jschwinger233)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 32084 32328 32464 32553 32564 32548 32570 32600 32302 32629

It looks like you backported my PR twice, with the second backport being empty.

Looks like I messed it up when I redo the process once. Fixed now.

/test-backport-1.13

Rebasing to pull-in bfbb9be3ed5f21ee8c4a9452c139d22578894aef.

/test-1.17-4.19

/test-1.19-4.19

/test-1.25-4.19

/test-1.17-4.19


Cirrus: Adjust destination branch name

Signed-off-by: Chris Evich<EMAIL_ADDRESS>

LGTM once things go green

/approve

/lgtm
/hold

/hold cancel


Update README.md for uninstall modification

The PR should be made to v0.5.0 release. Master is still v0.4.0 so uninstall is different.
The steps for uninstallation wasn't updated. It was pointed to the longhorn-system to watch the uninstallation job.

Please double check.


[Program]: Write a Swift program to convert time from 24-hour to 12-hour format

Description
Write a Swift program to convert time from 24-hour to 12-hour format
Input  : 12:05:45
Output : 12:05:45PM

How to contribute

Save the solution in program/program/convert-time-from-24-hour-to-12-hour-format/program/ConvertTimeFrom24-hourTo12-hourFormat.swift file

!assign


[IMPROVEMENT] : Change the dark social media icons to colored icons

Type of the Issue

[ ] BUG
[ ] FEATURE REQUEST
[ ] DOCUMENTATION
[X] OTHERS

Description of the Issue
Change the dark social media icons in the footer to colored icons
Reproducing the issue - [BUG]
Not required
Outcome/Advantages of the implementing/fixing this feature/bug - [BUG/FEATURE]
The colored icons looks better with both the dark/light background but dark icons don't look good with dark background
Screenshots

Additional Info
--

@777ahesh @atharva0300 Can I fix this ??

I can fix this. Let me try.

Can I fix this issue? Please assign me this issue.

Can I Fix this issue? Please assign


LitElment extends HTMLElement but TS type definitions get lost within mixin

Description
LitElment extends HTMLElement but TS type definitions get lost within mixin

Live Demo
https://stackblitz.com/edit/typescript-iv8yk6
Steps to Reproduce

In the demo above, I'm showing two things:

A mixin expecting to take a HTMLElement-ish constructor.
Adding an event listener within a LitElement-created element.

The following things can be observed:

LitElement isn't reconized as a class that extends HTMLElement, even though it does. Applying the mixin triggers the Typescript compiler; otherwise this is ambiguous.
Trying to do standard HTMLElement stuff like this.addEventListener produces an error: Property 'addEventListener' does not exist on type 'MyElement'.

Expected Results
LitElement inheritance of HTMLElement should come through in type definitions.
This is likely an upstream issue with PropertiesMixin from @polymer/polymer. An issue could be opened in that repo as well, although the type definitions could probably be fixed within this project.
Actual Results
LitElement should be typed show it inherits HTMLElement.
Browsers Affected
This isn't browser related but more a type definitions issue.

[ ] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari 11
[ ] Safari 10
[ ] IE 11

Versions

lit-element: v0.5.2
webcomponents: v2.0.4

Versions above reflected in demo. Note the demo only applies the es5 adapter.

This should be fixed in master and the current dev release.


Jupyter Book documentation site for v0

With the initial Jupyter Book site setup completed in #89, we can now start to add more sections/sub-sections to the Table of Contents. The website will be hosted at https://clay-foundation.github.io/model/ until we get a proper domain name.
For v0 (due before Christmas 2023), I'm planning the following structure:
Getting started

[x] Installation (installation.md)

Data Preparation

[x] Sampling locations (data_sampling.md) #124
[x] Preparing the datacube (data_datacube.md) #97
[x] Preparing labels for evaluating model performance (data_labels.md) #100

Running the model

[ ] Self-supervised pretraining (model_ssl.md)
[x] Generating embeddings (model_embeddings.md) #98
[x] Finetuning on labels (model_finetuning.md) #99

Note that the markdown files for the 'Data' and 'Model' sections are prefixed with data_ and model_ respectively to organize things a bit. Feel free though to suggest changes or edit the above directly.

PR opened to complete v0 documentation on #118

I think this is already live and mostly complete for a first pass.


Add "normal scale" resize button (Scale to 1)

with button tooltips

Middle button

Player panel infos adjustment in order to make russian translations fit again.


TextField: Add experimental show/hide password button

Currently TextField doesn't give users any way of viewing what they've typed so far when type="password". This is a frustrating UX and is likely leading to failed logins. This PR adds an experimental "show/hide password" button to solve this problem.

I was playing with it locally, and it seems to  me weird that the eye icon is display on password mode, but the hidden-eye icon is display on text  mode.
<img width="337" alt="Screen Shot 2022-03-30 at 6 39 16 PM" src="https://user-images.githubusercontent.com/10593890/160942615-d014c0f0-37af-4dd7-bd92-f3b46d13059f.
I see that Microsoft follows the pattern I  see here.   I also looked into  other platforms all seem to go the other way:
Facebook

png">

TikTok

Can we add some context from where we made our decision, just for context?

Just for context, can we add some information/context from how we made this decision, decided to go this way?

@AlbertCarreras Good point. I went with what made sense to me, which designers approved but may not have been paying attention to which icon is used for a given state. I'll ask in the team channel to double-check.


Trigger a re-index/index load in a macro

:wave: I've built a macro that generates an NPC for me and posts it to chat. Part of that is picking a random image for them and I'm using game.moulinette.cache.cache["moulinette/images/custom/index.json"] as the source of my images. However, it appears that cache isn't populated until I open one of the UI dialogs that would search it. So, every time I log into the UI I need to open one of those before my macro will work.
Is there already a function I can call to build that cache? If not, can the function that generates it be exposed on game.moulinette somewhere?
cc https://github.com/crimsonknave/foundryvtt-macros/pull/1 (where I use the cache)

Hi!
There is no such function but I'll add one in the future to let other modules/macros use it.
In the mean time, you can use the following snippet
const user = await game.moulinette.applications.Moulinette.getUser()
const index = await game.moulinette.applications.MoulinetteFileUtil.buildAssetIndex([
      game.moulinette.applications.MoulinetteClient.SERVER_URL + "/assets/" + game.moulinette.user.id])


Generic call execution

This PR introduces the generic call execution. This feature automatically resolves the service definition and the method name from the gRPC service based on the input protobuf argument. This essentially implies that the gRPC MUST be generated with the protobuf files because it provides the automatic RequestMethodPB discovery.


Added error 355

Rules 141 & 147 enforce that DECOM and DEC are in the format dd/MM/yyyy so I think we're fine testing as a string.
astype(str) probably isn't needed, but won't matter.

@dezog Done

Closes #327


Add project skeleton

We can introduce some tests in order to restrict the scope of the project

@volodya-lombrozo the puzzle #3 is still not solved.

@volodya-lombrozo the only puzzle #3 is solved here.


support execution of trivial examples (see explanation)

Currently a python file must contain exactly one function (and nothing else) on the top level to be correctly translated to EO by our transpiler. We need to support at least some trivial examples, which don't have functions, like print "Hello, world", say.
In order to do that we a going to

wrap a trivial example into a function
implement printing of string literals

done in #264


Add the image for a weapon (if there is one) next to the weapon where the D20 icon is right now

Would be nice to have the image for a weapon appear next to it.

I've added this as part of the drag from sheet refactor, weapons and armor need something to drag, and I'd prefer not to make textboxes draggable since it messes with selecting text.

Released in v0.26.0


Cannot build LIST OF LIST OF ENTITY

More specifically, if one replaces the relevant file in the master branch with the following, the build will not pass.
https://github.com/ricosjp/ruststep/blob/vec-vec-entity/ruststep/tests/list.rs

This issue is resolved in #183.


Import issues with Svelte REPL

Describe the bug
I am able to use old versions of svelte-query in https://svelte.dev/repl/ using the following import:
import { QueryClient, useQuery } from '<EMAIL_ADDRESS>
However if I try to import a newer version like @1.6.0 I see errors like
Error: Could not load<EMAIL_ADDRESS>(imported by https://unpkg.com/broadcast-channel@4.11.0/dist/esnode/methods/node.js): Cannot find "/index.js" in<EMAIL_ADDRESS>
I'm not sure if this is an issue with the REPL, unpkg.com, or svelte-query, but as older versions work I think it will at least partially involve changes here.
Reproduction

Open a new REPL
add import { QueryClient, useQuery } from '<EMAIL_ADDRESS>watch for console errors

https://svelte.dev/repl/be8980cad4244f72b6eae7b6ab726c1c
Logs
Error
    at R (index.3fa06cef.js:21:834)
    at tn (index.3fa06cef.js:51:3364)
    at index.3fa06cef.js:51:21283
    at async tc (index.3fa06cef.js:51:4123)
    at async ic.addModuleSource (index.3fa06cef.js:51:9326)
Mp @ index.3fa06cef.js:60

Error: Could not load<EMAIL_ADDRESS>(imported by https://unpkg.com/broadcast-channel@4.11.0/dist/esnode/methods/node.js): Cannot find "/index.js" in<EMAIL_ADDRESS>    at index.3fa06cef.js:60:1469
    at async Object.load (index.3fa06cef.js:60:2977)
    at async index.3fa06cef.js:51:8510
    at async Zu.work (index.3fa06cef.js:51:2752)
Mp @ index.3fa06cef.js:60

System Info
This is in the REPL, and the browser shouldn't make a difference, but my user agent is
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/101.0.4951.54 Safari/537.36

Severity
annoyance

closing since the site has been rewritten. please file at https://github.com/sveltejs/svelte.dev/issues/new if the issue persists
I'd first recommend putting any packages that don't work into publint.dev


feat: remove decryption domains from the deprecation list

Why
Decryption domains should not be on the deprecated list
How

Removed as deprecated option

Checklist

[X] Commit messages are formatted as per the convention, and breaking changes are marked

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Damcn/i card/base treasures

The base treasures are implemented as a Decorator. May change in the future, but trying out actions will better reveal that. Hopefully, a future change would only/mostly impact the database.

Ready to squash merge when you are.
You have 24 hours. Good luck.
:D


release: fix duplicate publish

release: fix duplicate publish

:tada: This PR is included in version 1.0.50 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add DistanceManager and DistanceData for easy retrieval of distance and travel time data

With the current state of Course and Distance classes, the optimisation algorithm will now have all information available to use. Please ask Jacky or Joshua for any questions on usage. Sample code available as part of commit(s).
Summary

Add DistanceManager, which provides a high-level interface for accessing distance information between course sessions.
Add DistanceData, a simple class containing to and from building codes, and the distance & travel time between them
Add DistanceDemo, to test a simple working scenario to calculate all possible distances between two courses. See screenshot below for CSC207 & COG250. ✈
Modify Session, to add a ", Toronto, Ontario, Canada" suffix at the end of building addresses. This increases accuracy within Google Maps.
Move Course and Session into their dedicated package, to differentiate between major functionalities.
Modify .gitignore to better ignore Intellij-specific files.

Currently, the API key for calling the Google Maps API is restricted to specific IP addresses. You can use your own key (free) or send me (Joshua) your external IP address via DM.


correct end of support date for jdk-11

Hey guys
Description of change
Isn't it supposed to be Oct 24 instead of Oct 27?
If i am wrong sorry for bothering.
At least also RH hast full support only until Oct 24.
-Mario

I think we can close this in that case.


Add remove

Steps Taken

Updated styles page
Made the list items draggable by adding the draggable attribute to the component
Created the edit static helper method
Created a delete method to help in the removing of item(s)
Created a remove function to handle the removal of todo list item
Created the clearAll Static method to remove all completed todos
Adjusted previous methods and functions to adhere to requirements
Included styles in the list form element and other parts of the pages
Implemented the rotate functionality of the refresh button
Included drag.js file to handle all drag/drop related events
Updated README.md file

Screenshots
Before

After

thanks Daniel


Added Remove-SPOListItem cmdlet.

Added Remove-SPOListItem cmdlet.

Hi @Laskewitz, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Thanks Daniel!


[R] custom iter in model training

I am working to let people use a customized iter in model training, and it provides more flexibility in inferencing the shape (see #4462).
A simple MF demo with a customized iter is also provided.
Can you give me some opinions on this? @lxn2 @p2004r @vk99

I'm wondering if the concatenation of two CSViter results in performance issues from the redundancy of the labels from each CSViter?
Would it be feasible to have a CSViter reworked so has to have something like a list of CSV files as argument rather than the current fixed data and label? This would facilitate the use of mixed or multiple data sources. It is something fairly straightforward to perform through R array iter right now, but ability to use a CSV iter would be a nice feature for larger datasets. I'm also thinking that such flexible CSV iter might be used to feed varying length RNN (assuming data has been previously processed into specific CSVs based on their bucket).
Finally, is it desirable to assume there's an argument named label? Just like for other inputs that could be named user and product rather than data, I feel like it would improve model flexibility to have input arguments binded based on whatever arguments are fed by the iter value() rather than on some naming assumptions.
Anyway, just some personal thoughts; thanks a lot for your work on this!

@jeremiedb Thanks for your advice.
Maybe we can open another issue for the CSViter.
For the "label", it is possible to let users use other names. However, this needs some more careful check since there are many codes with the assumption of that we have always have "label".


Update Velero docs

Summary
A concise description of the changes being introduced. Please review the pre-merge checklist section to validate this pull request is ready for review and merge. If it is not ready, please mark the pull request as a draft.
The owners of this repo are not experts in the subject matter of the quickstarts. We review for the quickstart to be functional and for security risks. If you are seeking feedback on the content of the quickstart, please seek out a subject matter expert. If you are not an internal NR contributor, we can do our best to connect you with a content reviewer.

Pre merge checklist

[ ] Did you check you NRQL syntax? - Does it work?
[ ] Did you include an InstallPlan and Documentation reference?
[ ] Are all documentation links publically accessible?
[ ] Did you check your descriptive content for voice and tone?
[ ] Did you check your descriptive content for spelling and grammar errors?
[ ] Did you review your content with a subject matter expert? (e.g. a Browser agent quickstart is reviewed with a member of the Browser Agent team)

Dashboards

[ ] Does the PR contain a screenshot for each of your dashboards?
[ ] Do your screenshots show data?
[ ] Has the sanitization script been run on each dashboard?

Alerts

[ ] Did you check that your alerts actually work?

Hey @mangulonr looking good so far, just need to resolve the merge conflict here with your content changes. I see you have a similar PR in draft state over here:
https://github.com/newrelic/newrelic-quickstarts/pull/1651
Do we want to close that other one?


Suggestions: Method overriding and event prioritizing

I've been really missing a few key features in NAV when trying to write clean and separated code.
Over the years, most VARs (and even ISVs) have been heavily modifying the NAV core while writing add-ons. Obviously this cannot continue in a cloud-first approach. Which is a good thing.
I come from a c#/c++/java background. While creating customizations and add-ons for multiple other platforms, I'm always able to achieve everything I need in a clean manner. But NAV has some important limitations.
Events:
I'd like to see a prioritizing system for event subscribers. Tenants would have the option to prioritize subscribers, not the coders (because we'll all just end up using the highest possible priority for our add-on ;)). That way the tenant can decide in what order to invoke.
Method overriding:
I'd like full method overriding possibilities. For example, in table 18 I want to be able to completely override the GetBillToCustomerNo method, with the possibility to call the base method if for some reason I need to fall back on it. These overrides should of course have the same kind of prioritizing system as mentioned for events.
I know this is a huge deal and a big ask, based on the current NAV tech. But it would be a massively powerful addition.

Hi! This issue is beyond the scope of the AL language and development tools. Please go ahead and post this to our Ideas forum at https://aka.ms/BusinessCentralideas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


chore(release): v0.9.1

:robot: I have created a release beep boop
0.9.1 (2023-03-12)
Bug Fixes

tailwind: prompt component grid area spacing. (2ebe3e8)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sira-design/sira/releases/tag/v0.9.1 :sunflower:


Undefined parameters for triggers

Class parameters for services are not defined in triggers. I.e.:
https://github.com/ongr-io/MagentoConnectorBundle/blob/master/Resources/config/triggers/content.yml#L10

Will be closed by #41 .


Bug: Unable to reset user password

What were you trying to achieve?
I want to reset my user password but I get an error
What are the expected results?
Password reset successfully
What are the received results?
{
"error": "",
"message": "entity not found"
}
Steps To Reproduce
No response
In what environment did you encounter the issue?
Ubuntu 22.10
Docker 24.0.5
Golang 1.21.1
Additional information you deem important
No response

This issue has been resolved via https://github.com/absmach/magistrala/pull/2135.


Remove tests from the package distribution

Problem
Originally raised in #244, which was tracking the split of tests, but now closed. I'm in favour of getting the tests excluded from the distribution, I personally think testing code should not be present on Production machines. However, they seem to be needed to be able to package for RPM (see #230 @Germano0 ?)
The official packaging doc suggests it, but it doesn't state it clearly.
Testing code is around 2/3 of the code in this repo (1031 lines vs. 2369 from a quick estimate), so that would also make the distribution lighter.

I'm +1 to remove them from the distribution. I guess that, if we remove the test-related arguments in setup.py and exclude the models_utils.tests folder, it should be enough.
And ideally, it would have even been better to move the tests folder outside of the model_utils folder, and not even have to exclude anything in setup.py.


Adds the "create_top_pins.sh" script

Adds the "create_top_pins.sh" script which is essentially the same script from sky130 caravel which uses the top-level LEF file to annotate the top level layout with the padframe pins.  Also: Added a GDS file reference to the simple_por layout so that it will import the GDS verbatim when generating the final GDS output for the chip, which is a technique used on the sky130 caravel to get the full hierarchical GDS generation on the POR while allowing the rest of the chip (digital) to be output without hierarchical processing.

@RTimothyEdwards we need the caravel.lef in order to get this script working, can you please provide it?


Show field api name on record page

Hi Thomas,
This pull request introduces a new feature that enhances the user experience when viewing record pages on Salesforce. It enables to display the Field API Name, providing valuable information for administrators and developers.
It works on Layouts & Dynamic Forms.

Merci 😁

Hi @MD931,
Thanks for this PR !
I've already seen this functionality on other extensions but never had the idea to implement it on sfir.
Could you describe a use case where this is more relevant than the show all data page ?
Except some particular use cases where you have multiples fields with same label, I can't really figure it out.
If we found use cases, I would include the button just after the show all data, to only display it on record page and not all pages

@tprouvot I can make an update to display the button after the "Show All Data" button on a record page.

@tprouvot I can make an update to display the button after the "Show All Data" button on a record page.

That would be great !
I was thinking that we may have too many buttons on the popup so I created a new feature request that would help to reduce the number of buttons displayed

Hi @tprouvot
I've made the changes, and now the button is displayed exclusively on a Lightning record page. I also took the opportunity to show the API Name of the fields within a quick action of a related record.

Thank you 🙂

Hi @MD931,
I added a new element of 'mark' type to highlight the api name.
During the test I've found a behaviour that may affect users when trying to copy the api name:
On double click (to select the api name) the UI triggers the edit mode which removes the api name

Hi @MD931, I added a new element of 'mark' type to highlight the api name. During the test I've found a behaviour that may affect users when trying to copy the api name: On double click (to select the api name) the UI triggers the edit mode which removes the api name

Yes, it's tricky, and unfortunately, it removes the API Name. I don't have a solution for that. What I could do is store the API Name to the clipboard with a single click on the API Name

Hi @MD931, I added a new element of 'mark' type to highlight the api name. During the test I've found a behaviour that may affect users when trying to copy the api name: On double click (to select the api name) the UI triggers the edit mode which removes the api name

Yes, it's tricky, and unfortunately, it removes the API Name. I don't have a solution for that. What I could do is store the API Name to the clipboard with a single click on the API Name

Copy the api name to the clipboard could be a good option but we need to give this information to the user.
I think that a cursor over the api name would prevent users to double click and I'm also thinking to use a copy icon as cursor.
Just wondering if the UI won't be too ugly 😄

Hi @MD931, I added a new element of 'mark' type to highlight the api name. During the test I've found a behaviour that may affect users when trying to copy the api name: On double click (to select the api name) the UI triggers the edit mode which removes the api name

Yes, it's tricky, and unfortunately, it removes the API Name. I don't have a solution for that. What I could do is store the API Name to the clipboard with a single click on the API Name

Copy the api name to the clipboard could be a good option but we need to give this information to the user. I think that a cursor over the api name would prevent users to double click and I'm also thinking to use a copy icon as cursor. Just wondering if the UI won't be too ugly 😄

I have just made a new commit that incorporates the copy cursor and also enables copying the API Name with a single click.

Hi @tprouvot
I have just made some changes. Now, the button is no longer visible when no field is found.
Thanks 😁

@MD931 That's great ! thanks for this 👏
Now I have just one use case missing: the console.
When enabled, the show all data is not available, could you check if you can find an element in the page which can be used with and without the console enabled ?

Keep going, you're close to the end !

Hi @tprouvot
I just modified the code by adding the necessary selector for the console mode.
Thank you 😅

Hi @MD931
Unfortunately I don't have the button with the console:

Hi @tprouvot
Sorry. I don't know why some console apps have a selector 'div.content' and others don't. I just updated it by changing the selector to 'section.oneConsoleTab'

Hi @MD931
No updates from my side 😢 :

fix #202


Issue adding Jellyfin as provider (2.0.0b93)

What version of Music Assistant has the issue?
2.0.0b93
The problem
Trying to add a Jellyfin server I get error:
"List index out of range"
How to reproduce
Update to 2.0.0b93, attempt to add jellyfin sever at Tailscale IP in range 100.93.XXX.XXX (normal 192.XXX.XXX.XXX IP works)
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.1.6
What type of installation are you running?
Home Assistant Container
On what type of hardware are you running?
Linux

Please add a snippet of the error that is logged in the music assistant server log

We will close this soon due to no follow up

Log entry:
2024-02-16 12:56:01.031 INFO (ThreadPoolExecutor-0_15) [JELLYFIN.jellyfin_apiclient_python.api] Sending get request to system/info/public
2024-02-16 12:56:06.040 ERROR (ThreadPoolExecutor-0_15) [JELLYFIN.jellyfin_apiclient_python.connection_manager] connect_to_address http://<IP_ADDRESS>:8096/ failed
2024-02-16 12:56:06.040 INFO (ThreadPoolExecutor-0_15) [JELLYFIN.jellyfin_apiclient_python.api] Trying to login to http://<IP_ADDRESS>:8096//Users/AuthenticateByName as homeassistant
2024-02-16 12:56:06.040 INFO (ThreadPoolExecutor-0_15) [JELLYFIN.jellyfin_apiclient_python.api] Sending post request to Users/AuthenticateByName
2024-02-16 12:56:11.046 ERROR (ThreadPoolExecutor-0_15) [JELLYFIN.jellyfin_apiclient_python.api] HTTPConnectionPool(host='<IP_ADDRESS>', port=8096): Max retries exceeded with url: //Users/AuthenticateByName (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x7f3c624bb2d0>, 'Connection to <IP_ADDRESS> timed out. (connect timeout=5)'))
2024-02-16 12:56:11.046 INFO (ThreadPoolExecutor-0_15) [JELLYFIN.jellyfin_apiclient_python.connection_manager] Failed to login as homeassistant
2024-02-16 12:56:11.046 INFO (ThreadPoolExecutor-0_15) [JELLYFIN.jellyfin_apiclient_python.credentials] {}
2024-02-16 12:56:11.047 ERROR (MainThread) [music_assistant.webserver] [139897324275856] Error handling message: CommandMessage(message_id=122, command='config/providers/save', args={'provider_domain': 'jellyfin', 'values': {'url': 'http://<IP_ADDRESS>:8096/', 'username': 'homeassistant', 'password': 'XXXXXXXXX', 'log_level': 'GLOBAL'}})
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/controllers/webserver.py", line 374, in _run_handler
result = await result
^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/controllers/config.py", line 261, in save_provider_config
config = await self._add_provider_config(provider_domain, values)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/controllers/config.py", line 777, in _add_provider_config
await self.mass.load_provider(config)
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/server.py", line 427, in load_provider
provider = await prov_mod.setup(self, prov_manifest, conf)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/providers/jellyfin/init.py", line 117, in setup
await prov.handle_setup()
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/providers/jellyfin/init.py", line 197, in handle_setup
self._jellyfin_server = await self._run_async(connect)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/providers/jellyfin/init.py", line 226, in _run_async
return await self.mass.create_task(call, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/asyncio/threads.py", line 25, in to_thread
return await loop.run_in_executor(None, func_call)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/music_assistant/server/providers/jellyfin/init.py", line 188, in connect
server = credentials["Servers"][0]
~~~~~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range

@lokiberra

@usbrit 'homeassistant' is a user on your jellyfin server?

Yes, independent user and password (password blanked with X's)
On Fri, 16 Feb 2024, 22:32 lokiberra, @.***> wrote:

@usbrit https://github.com/usbrit 'homeassistant' is a user on your
jellyfin server?
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/1900#issuecomment-1949427044,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMXTKQ7CGCZUPHT52EJAN7DYT7M6PAVCNFSM6AAAAABDIFCCO2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBZGQZDOMBUGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

This might be a networking problem at your end in which case we won’t be able to assist.

We will close this soon unless you can confirm this is not a tail scale issue.

I'm sorry I didn't realise you were wanting a reply.
Tailscale works fine to access the same machine from browser to Jellyfin (I
use the Tailscale networks for internal and external access) so I don't
believe it is an issue.
Is it possible that the current media assistant code does not accept 100.
style IP's?
On Tue, Feb 20, 2024 at 3:00 PM OzGav @.***> wrote:

We will close this soon unless you can confirm this is not a tail scale
issue.
—
Reply to this email directly, view it on GitHub
https://github.com/music-assistant/hass-music-assistant/issues/1900#issuecomment-1954403547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMXTKQ7FQ4L4TKP7HKOZ33TYUS3CBAVCNFSM6AAAAABDIFCCO2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJUGQYDGNJUG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Theoretically I don't think it should matter if it has a tailscale IP address. My best guess is that the docker networks make it unusable with tailscale. I tried setting up Jellyfin and Music Assistant in two docker containers with Tailscale clients in both and I couldn't get them to even ping each other. There might be a way to do this but it is beyond my current networking knowledge. Music Assistant doesn't seem able to connect to your IP at all. Is Home Assistant able to connect to your tailscale IP using the Jellyfin component?

We will close this soon due to no followup


Feature/parallelise store

I did a small local test where I found that this was significantly faster than the status quo. I think we might benefit from these changes.

Sure; it's quite a simple test. Just push a graph through .store with and without these changes and see how it compares.

I tested it on Mattis with SF10 conversion from Neo4j to Parquet.
Without this PR this takes 23 minutes, with this PR it's 9 minutes. Of those 9 2 minutes are schema procedure in Neo4j


Partners Terms of Service UX / Link Issue

Putting a link to the TOS vs the TOS itself seems like an unintended pattern of this component. It also doesn't open in a new tab, so you have to back track to accept the terms.

no0 longer present


displaying PDF files always uses fullscreen

upload a PDF to OS
display it

result: OS uses always fullscreen. You can press the button to turn off fullscreen but nothing will happen.

It would also be nice if one could set a default opening behavior (open always in full screen, or not in full screen). Maybe separate options for PDF / pictures / videos.

It would also be nice if one could set a default opening behavior (open always in full screen, or not in full screen). Maybe separate options for PDF / pictures / videos.

Could you please report this as separate issue? Because these are enhancements and should not be forgotten when this this bug will be closed/fixed.

2 suggestions:

Change the fullscreen icon in pdf presentation control panel to a "reset" icon (this action resets the zoom level to default (fullscreen))
Project each image in fullscreen by default (fullscreen button is enabled).

I close this issue and keep #3552 still open.


Allow the use of private IPs when connecting to the Packer VM.

Public IP addresses aren't allowed in some AWS secret regions, such as
us-iso-east-1.  This flag allows users in those environments to override the
default behavior and launch the packer VM without a public IP.  In that case,
packer will manage the VM using the private IP.

We will close this PR for now, until we finish all the work needed to use the builder in offline environments.


Check if token is present before authorizing with it

The correct fix here might be to figure out why the user is being saved without an oauth token on my machine.
@MikeMcQuaid would you verify that oauth tokens are being saved for you when you log in to the app?

The correct fix here might be to figure out why the user is being saved without an oauth token on my machine.

Nice spot @kytrinyx 💖. Fixed in https://github.com/ossfriday/ossfriday/pull/70.


Remove white space after autofill

When using the autofill function on a mobile device, it will create a white/blank space after the inserted text, resulting in errors in authentication or when crating things if not noticed. We need to add an extra flag added to our TextInput and Textarea components to automatically check if there is a white/blank space behind the word and remove it automatically
Image of the current problem:

Can i take this up?

Can i take this up?

Sure, thanks for your interest 👍

sorry, pls assign to someone else


Per message config for elasticdump

What is this PR trying to achieve?
Configure elasticdump on a per message basis (fixes #1868 )
Who is this change for?
☄ 💻
Have the following been considered/are they needed?

[ ] Deployed new versions
[ ] Run terraform apply.

DO NOT MERG! FLAKY TESTS DETECTED!


Dashboard: Add pagination to the family list on the frontpage

We're now dealing with almost a thousand font families. This is too much to show at once at the dashboard front page, so it would be good to add a pagination system to show just a few families per page. Maybe 50 or so...

The webapp still available from http://fontbakery.com dealt with this already, with some nice lazy-loading code. I hope you can use it :)

Migrating to fontbakery-dashboard issue tracker per #1383


🦺[Tech Debt] Reduce Cocoapods and Increase Swift Packages

Goal
Describe the work to be done

Check to see where current Cocoapods cn be converted to Swift Packages
Current Podfile:
  pod 'UnstoppableDomainsResolution', '~> 4.0.0'
  pod 'KeychainAccess', '~> 4.2'
  pod 'Firebase/Analytics'
  pod 'Firebase/Crashlytics'

Definition of Done

[ ] Do this incrementally
[ ] Update the Bitrise CI/CD Step to remove the Podfile

@Iferencak :  This is in the backlog for reference but it is a low priority for now


DL4JInvalidConfigException: Invalid configuration for layer

I got an error "Invalid configuration for layer" when I tried to import a keras model (based on VGG16).
[main] INFO org.nd4j.linalg.factory.Nd4jBackend - Loaded [CpuBackend] backend
[main] INFO org.nd4j.nativeblas.NativeOpsHolder - Number of threads used for NativeOps: 2
[main] INFO org.nd4j.nativeblas.Nd4jBlas - Number of threads used for BLAS: 2
[main] INFO org.nd4j.linalg.api.ops.executioner.DefaultOpExecutioner - Backend used: [CPU]; OS: [Linux]
[main] INFO org.nd4j.linalg.api.ops.executioner.DefaultOpExecutioner - Cores: [4]; Memory: [4.3GB];
[main] INFO org.nd4j.linalg.api.ops.executioner.DefaultOpExecutioner - Blas vendor: [OPENBLAS]
Exception in thread "main" org.deeplearning4j.exception.DL4JInvalidConfigException: Invalid configuration for layer (idx=-1, name=block2_conv1, type=ConvolutionLayer) for width dimension:  Invalid input configuration for kernel width. Require 0 < kW <= inWidth + 2*padW; got (kW=3, inWidth=1, padW=0)
Input type = InputTypeConvolutional(h=112,w=1,c=64), kernel = [3, 3], strides = [1, 1], padding = [0, 0], layer size (output channels) = 128, convolution mode = Same
        at org.deeplearning4j.nn.conf.layers.InputTypeUtil.getOutputTypeCnnLayers(InputTypeUtil.java:349)
        at org.deeplearning4j.nn.conf.layers.ConvolutionLayer.getOutputType(ConvolutionLayer.java:181)
        at org.deeplearning4j.nn.modelimport.keras.layers.convolutional.KerasConvolution2D.getOutputType(KerasConvolution2D.java:150)
        at org.deeplearning4j.nn.modelimport.keras.KerasModel.inferOutputTypes(KerasModel.java:306)
        at org.deeplearning4j.nn.modelimport.keras.KerasModel.<init>(KerasModel.java:181)
        at org.deeplearning4j.nn.modelimport.keras.KerasModel.<init>(KerasModel.java:98)
        at org.deeplearning4j.nn.modelimport.keras.utils.KerasModelBuilder.buildModel(KerasModelBuilder.java:305)
        at org.deeplearning4j.nn.modelimport.keras.KerasModelImport.importKerasModelAndWeights(KerasModelImport.java:108)
        at DL4J.main(DL4J.java:4)

I'm using DL4J 1.0.0-beta2 and Java 1.8.0_181 on Ubuntu18.04
$ cat DL4J.java
class DL4J {
    public static void main(String... args) throws Exception {
        org.deeplearning4j.nn.modelimport.keras.KerasModelImport
            .importKerasModelAndWeights("my_model.hdf5",
                                        new int[]{224, 224, 3}, false);
            // .importKerasModelAndWeights("my_model.hdf5"); // failed too
    }
}
$ java -version
java version "1.8.0_181"
Java(TM) SE Runtime Environment (build 1.8.0_181-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.181-b13, mixed mode)

This is the model(74MB) I used.
Thank you in advance.

thanks for reporting. will take a look next week

Whats news?

Hi,
I got a very similar error to that reported by u-ryo.
I am trying to import an Keras functional model and this is the error message that I get:
21:12:26.182 [main] WARN org.deeplearning4j.nn.modelimport.keras.KerasModel - If enforceTrainingConfig is true, a training configuration object has to be provided. Usually the only practical way to do this is to store your keras model with `model.save('model_path.h5'. If you store model config and weights separately no training configuration is attached.
Exception in thread "main" org.deeplearning4j.exception.DL4JInvalidConfigException: Invalid configuration for layer (idx=-1, name=conv2d_9, type=ConvolutionLayer) for width dimension:  Invalid input configuration for kernel width. Require 0 < kW <= inWidth + 2*padW; got (kW=3, inWidth=2, padW=0)
Input type = InputTypeConvolutional(h=16,w=2,c=256), kernel = [3, 3], strides = [1, 1], padding = [0, 0], layer size (output channels) = 512, convolution mode = Same
	at org.deeplearning4j.nn.conf.layers.InputTypeUtil.getOutputTypeCnnLayers(InputTypeUtil.java:334)
	at org.deeplearning4j.nn.conf.layers.ConvolutionLayer.getOutputType(ConvolutionLayer.java:191)
	at org.deeplearning4j.nn.modelimport.keras.layers.convolutional.KerasConvolution2D.getOutputType(KerasConvolution2D.java:148)
	at org.deeplearning4j.nn.modelimport.keras.KerasModel.inferOutputTypes(KerasModel.java:304)
	at org.deeplearning4j.nn.modelimport.keras.KerasModel.<init>(KerasModel.java:179)
	at org.deeplearning4j.nn.modelimport.keras.KerasModel.<init>(KerasModel.java:96)
	at org.deeplearning4j.nn.modelimport.keras.utils.KerasModelBuilder.buildModel(KerasModelBuilder.java:307)
	at org.deeplearning4j.nn.modelimport.keras.KerasModelImport.importKerasModelAndWeights(KerasModelImport.java:262)
	at main.java.com.moma.SimpleSequentialMlpImport.main(SimpleSequentialMlpImport.java:96)

Java version output, though I am running with Java 1.8 configuration for the project in Eclipse:
$ java --version
openjdk 11.0.3 2019-04-16
OpenJDK Runtime Environment (build 11.0.3+7-Ubuntu-1ubuntu219.04.1)
OpenJDK 64-Bit Server VM (build 11.0.3+7-Ubuntu-1ubuntu219.04.1, mixed mode, sharing)

I also tried the model that is linked by u-ryo and I get the same error message as he does.
I can provide the model, if needed, etc., if needed.
I would be grateful, if you could provide any pointers on where the issue may lay.
This is the Keras model (it is a U-Net model):
    inputs = Input((img_rows, img_cols, dims))
    conv1 = Conv2D(32, (3, 3), activation='relu', padding='same')(inputs)
    conv1 = Conv2D(32, (3, 3), activation='relu', padding='same')(conv1)
    pool1 = MaxPooling2D(pool_size=(2, 2))(conv1)

    conv2 = Conv2D(64, (3, 3), activation='relu', padding='same')(pool1)
    conv2 = Conv2D(64, (3, 3), activation='relu', padding='same')(conv2)
    pool2 = MaxPooling2D(pool_size=(2, 2))(conv2)

    conv3 = Conv2D(128, (3, 3), activation='relu', padding='same')(pool2)
    conv3 = Conv2D(128, (3, 3), activation='relu', padding='same')(conv3)
    pool3 = MaxPooling2D(pool_size=(2, 2))(conv3)

    conv4 = Conv2D(256, (3, 3), activation='relu', padding='same')(pool3)
    conv4 = Conv2D(256, (3, 3), activation='relu', padding='same')(conv4)
    pool4 = MaxPooling2D(pool_size=(2, 2))(conv4)

    conv5 = Conv2D(512, (3, 3), activation='relu', padding='same')(pool4)
    conv5 = Conv2D(512, (3, 3), activation='relu', padding='same')(conv5)

    up6 = concatenate([Conv2DTranspose(256, (2, 2), strides=(2, 2), padding='same')(conv5), conv4], axis=3)
    conv6 = Conv2D(256, (3, 3), activation='relu', padding='same')(up6)
    conv6 = Conv2D(256, (3, 3), activation='relu', padding='same')(conv6)

    up7 = concatenate([Conv2DTranspose(128, (2, 2), strides=(2, 2), padding='same')(conv6), conv3], axis=3)
    conv7 = Conv2D(128, (3, 3), activation='relu', padding='same')(up7)
    conv7 = Conv2D(128, (3, 3), activation='relu', padding='same')(conv7)

    up8 = concatenate([Conv2DTranspose(64, (2, 2), strides=(2, 2), padding='same')(conv7), conv2], axis=3)
    conv8 = Conv2D(64, (3, 3), activation='relu', padding='same')(up8)
    conv8 = Conv2D(64, (3, 3), activation='relu', padding='same')(conv8)

    up9 = concatenate([Conv2DTranspose(32, (2, 2), strides=(2, 2), padding='same')(conv8), conv1], axis=3)
    conv9 = Conv2D(32, (3, 3), activation='relu', padding='same')(up9)
    conv9 = Conv2D(32, (3, 3), activation='relu', padding='same')(conv9)

    conv10 = Conv2D(1, (1, 1), activation='sigmoid')(conv9)

    conv11 = Reshape((img_rows*img_cols,1),input_shape=(img_rows,img_cols,1))(conv10)

    model = Model(inputs=[inputs], outputs=[conv11])

    model.compile(optimizer='adam', loss='binary_crossentropy', metrics=[dice_coef], sample_weight_mode='temporal')

Wondering if there is any update on this? I can load my model and run it in python enviroment successfully, but met this error message in DL4J. Is there any version difference between python and java? Thanks.

Wondering if there is any update on this? I can load my model and run it in python enviroment successfully, but met this error message in DL4J. Is there any version difference between python and java? Thanks.

@FanJiangDev  mind opening a new issue and I'll take a look? Include your version etc. I'll close this in favor of that issue.

@FanJiangDev  mind opening a new issue and I'll take a look? Include your version etc. I'll close this in favor of that issue.


doc: update howto guide for generated libraries

The check-api build immediately starts testing new GA libraries, we need to create the API baseline as we create the new library.

This change is

Done


[PR BOT] Update base image tag file(s)

Issue #, if available:
Description of changes:
One last issue with the pr opening code, but the builds all succeeded and this is the correct tag file update.  Will fix the pr code tomorrow.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Everyone - Write All?

Are you able to elaborate on the "Enable auditing on an Exchange object" section.  As in, what making this change is doing and why we're giving the Everyone group Write All?  In 2023 that doesn't read like something we should be doing anymore in a only give the least privileges required world of IT.

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

ID: b032a29d-2d3a-d737-28fb-3e6632a667e2
Version Independent ID: b75dd591-4cd7-259a-1fe7-a581d6e356c9
Content: Configure Windows Event collection - Microsoft Defender for Identity
Content Source: ATPDocs/configure-windows-event-collection.md
Service: microsoft-defender-for-identity
GitHub Login: @batamig
Microsoft Alias: bagol

Thank you for your comment. We'll investigate and get back to you.

Please follow the instructions carefully (step 7).
The "Everyone group Write All" is not a DACL (who has access to an object and who is denied access) requirement, but a SACL (which access-granted or access-denied events are to be recorded for auditing) requirement.

Hi @mattsmacblog, please let us know if this solves your issue. Thanks!

We haven't heard back from you, so we will now close this issue. If this remains an issue, please reply and we will gladly continue the discussion.
#please-close


📖 Document management upgrade tests

Docs update for e2e upgrade

/test-centos-e2e-integration-main
/test-ubuntu-integration-main


Upload Failure

Every time I try to upload something, it fails with the following error on both flac and mp3.
Fragments of Time.flac | GetUploadSession: 404 Client Error: Not Found
(requests kwargs: {'url': 'https://uploadsj.clients.google.com/uploadsj/rupio', 'headers': {'Authorization': '<omitted>', 'User-agent': 'Music Manage
r (1, 0, 55, 7425 HTTPS - Windows)'}, 'data': '{"protocolVersion": "0.8", "clientId": "Jumper Uploader", "createSessionRequest": {"fields": [{"extern
al": {"put": {}, "name": "/home/media/Music/Random Access Memories/Fragments of Time.flac", "filename": "Fragments of Time.flac"}}, {"inlined": {"con
tent": "true", "name": "SyncNow"}}, {"inlined": {"content": "1", "name": "ClientTotalSongCount"}}, {"inlined": {"content": "Fragments of Time", "name
": "CurrentUploadingTrack"}}, {"inlined": {"content": "0", "name": "CurrentTotalUploadedCount"}}, {"inlined": {"content": "jumper-uploader-title-42",
 "name": "title"}}, {"inlined": {"content": "false", "name": "TrackDoNotRematch"}}, {"inlined": {"content": "bcd8c90b-28e7-331c-85b6-badce2344c33", $
name": "ServerId"}}, {"inlined": {"content": "04:7D:7B:76:54:87", "name": "UploaderId"}}, {"inlined": {"content": "831", "name": "TrackBitRate"}}, {$
inlined": {"content": "bXaoZhty0bjdfAhjB2T4vw", "name": "ClientId"}}]}}', 'method': 'POST'})
(response was: 'Not Found')

Pretty Printed:
{
  'url':'https://uploadsj.clients.google.com/uploadsj/rupio',
  'headers': {
    'Authorization': '<omitted>',
    'User-agent': 'Music Manager (1, 0, 55, 7425 HTTPS - Windows)'
  },
  'data': '{
    "protocolVersion": "0.8",
    "clientId": "Jumper Uploader",
    "createSessionRequest": {
      "fields": [{
        "external": {
          "put": {},
          "name": "/home/media/Music/Random Access Memories/Fragments of Time.flac",
          "filename": "Fragments of Time.flac"
        }
      },
      {
        "inlined": {
          "content": "true",
          "name": "SyncNow"
        }
      },
      {
        "inlined": {
          "content": "1",
          "name": "ClientTotalSongCount"
        }
      },
      {
        "inlined": {
          "content": "Fragments of Time",
          "name": "CurrentUploadingTrack"
        }
      },
      {
        "inlined": {
          "content": "0",
          "name": "CurrentTotalUploadedCount"
        }
      },
      {
        "inlined": {
          "content": "jumper-uploader-title-42",
          "name": "title"
        }
      },
      {
        "inlined": {
          "content": "false",
          "name": "TrackDoNotRematch"
        }
      },
      {
        "inlined": {
          "content": "bcd8c90b-28e7-331c-85b6-badce2344c33",
          $name": "ServerId"
        }
      },
      {
        "inlined": {
          "content": "<retracted>",
          "name": "UploaderId"
        }
      }, {
        "inlined": {
          "content": "831", "name": "TrackBitRate"
        }
      }, {
        $inlined": {
          "content": "<retracted>",
          "name": "ClientId"
        }
      }
    ]
  }
}', 'method': 'POST'}

Update gmusicapi with 'pip install --upgrade gmusicapi'. Google gave a heads up that they would be moving the uploading endpoint, so Simon already has it updated. Seems this is the first reported case of the old endpoint being disabled.

It seems to be working now! Thanks!


[Translatable] Can't query the non-current locale

I need to translate dynamic url data for a lang switcher. I have to query an entity and get the slug for the "other locale" but it always only gives me back the current locale data. What I do is that I query once the translated object and then use the id to query the other locale like so :
 $qb                 ->select('e, et')                 ->join('e.translations', 'et', 'WITH', 'et.locale = :locale')->setParameter('locale', $locale)                 ->where('e.id = :tid')->setParameter('tid', $identifier->getId())                 ;
However it only return the current locale data. Even if I try to do $entity->translate($locale) on my entity, it still only give me the translation for the locale that I don't want.
Is there any way to achieve what I want?
Thanks

The only way I found to workaround is a native SQL query using doctrie's prepared statement (of course you should use parameters and not variable directly inside a query) :
$query = 'SELECT * FROM '.$tableName.' WHERE translatable_id = '.$entity->getId().' AND locale = "'.$locale.'" LIMIT 1';                 $stmt = $this->em->getConnection()->prepare($query);                 $stmt->execute();                 $results = $stmt->fetchAll();
But it kind of looks unclean so I'm not completely satisfied but at least it's some work around.

The only way I found to workaround is a native SQL query using doctrie's prepared statement (of course you should use parameters and not variable directly inside a query) :
$query = 'SELECT * FROM '.$tableName.' WHERE translatable_id = '.$entity->getId().' AND locale = "'.$locale.'" LIMIT 1'; $stmt = $this->em->getConnection()->prepare($query); $stmt->execute(); $results = $stmt->fetchAll();
But it kind of looks unclean so I'm not completely satisfied but at least it's some work around.

I'm going through the old issues/PRs and closing most of them so we can focus on now.
This one is missing any feedback or reasoning so I cannot deduce anything from it.
Thanks for you though, but closing for reasons above.
If the bug still remains, please send failing test case to verify it. Only with test we'll be albe to help.


Lack of support for server URL basepath templated variables in Connexion library

Description
The Connexion library, which helps create APIs using OpenAPI, has a problem when using variables in the server URL. The problem is when we use variables in the openapi.yaml file, the library automatically chooses a default value for the variable instead of letting us choose the value in the Swagger UI. This means we can't change the value of the variable easily, and also it doesn't allow us to receive the variable value in the request callback.
Here's an example of how variables are defined in the openapi.yaml file:
openapi: 3.0.0
info:
  title: "Example API"
  description: ''
  version: 1.0.0
servers:
  - description: ""
    url: https://example.com/
  - description: ""
    url: /{version}/example
    variables:
      version:
        description: The version number
        enum:
          - 1
          - 2
        default: 1

As you can see in the above example, we have defined a variable version with possible values 1 and 2 and a default value of 1. But the problem is, we can not change the value of the variable in the Swagger UI, and also we can not receive the variable value in the request callback.
Expected behaviour
We should be able to choose the value of the variable in the Swagger UI and also receive the variable value in the request callback.
Actual behaviour
The library automatically chooses a default value for the variable, we can not change the value of the variable easily, and also we can not receive the variable value in the request callback.
Steps to reproduce

Create a new project using the Connexion library
Add a variable in the server URL in the openapi.yaml file similar to the above example
Try to change the value of the variable in the Swagger UI
Observe that the default value is chosen and cannot be changed
Try to access the variable value in the request callback and observe that it's not accessible

Additional info:
Output of the commands:
python --version : Python 3.10.9
pip show connexion | grep "^Version:" : Version: 2.14.1

Hi @moshego189 , I am not sure I can fully reproduce the issue:
using the provided yaml as is in the hello world example raises an InvalidSpecification exception
connexion.exceptions.InvalidSpecification: 1 is not of type 'string'

Failed validating 'type' in schema['properties']['servers']['items']['properties']['variables']['additionalProperties']['properties']['enum']['items']:
    {'type': 'string'}

On instance['servers'][1]['variables']['version']['enum'][0]:
    1

When I change it to the string variants by adding quotes, I indeed see that it's not possible to select a value:

However, when removing the first element of the servers list, I can access the UI on the following URL: http://<IP_ADDRESS>:9090/1/example/ui/

but not on http://<IP_ADDRESS>:9090/2/example/ui/
Unfortunately I'll have to dig a bit deeper on this, I believe I'm simply overlooking something here :)

you are right , i will update the example


Added FormatPainter Plugin

Hi There,
I've added Format Painter plugin in this PR
This plugin is commonly used in google docs, so tried to implement it here
I'm open to discussions and any changes if required

Thank you for your contribution.
But, If just simply style it, formatting becomes a problem.
I'll add this feature in version 3.0.0.


Action is failing on deploy: unknown flag: --cache-artifacts

Hi all, I'm using the action below and having two problems:

the action is not respecting the -v debug flag
The action is failing. The reason being logged is unknown flag: --cache-artifacts but I am not setting this flag in the action, and setting cache: false makes no difference.
I am running the same skaffold version locally and can run the deploy command with arguments successfully.

Run<EMAIL_ADDRESS>  with:
    command: deploy -v debug
    repository: gcr.io/PROJECT
    tag: 8e47aa2272f6284241bd2fe36ad705487c0fb9d2
    skaffold-version: 1.32.0
    container-structure-test-version: 1.10.0
    cache: true
    filename: skaffold.yaml
    skip-tests: true
  env:
    PROJECT_ID: PROJECT
    GKE_CLUSTER: CLUSTER
    GKE_ZONE: us-west2
    CLOUDSDK_METRICS_ENVIRONMENT: github-actions-setup-gcloud
    GCLOUD_PROJECT: PROJECT
    GOOGLE_APPLICATION_CREDENTIALS: REDACTED
    KUBECONFIG: REDACTED
/usr/bin/chmod +x /home/runner/actions-runner/_work/github-reporter/github-reporter/bin/skaffold
/home/runner/actions-runner/_work/github-reporter/github-reporter/bin/skaffold deploy -v debug --cache-artifacts=true --default-repo=gcr.io/PROJECT --tag=8e47aa2272f6284241bd2fe36ad705487c0fb9d2 --filename=skaffold.yaml --skip-tests=true
unknown flag: --cache-artifacts
See 'skaffold deploy --help' for usage.
Error: The process '/home/runner/actions-runner/_work/github-reporter/github-reporter/bin/skaffold' failed with exit code 127

For further context, I'm running on a self hosted runner running Ubuntu 20.04

@donnyv12 I ran into this myself, there's a workaround to not specify a cache for deploy-only command - use
with:
  cache-artifacts=''

in your action call - this will tell the action to drop the CLI argument, as it tries to pass the argument to deploy by default, which doesn't take an argument. I'm working on a fix in my fork.


User should verify his changed email/mobile

When user saves new email or mobile we send him verification codes
Email/mobile fields should say unverified and have an icon to verify
clicking on icon leads to same screen as on signup waiting for user to copy paste the code

@sirpy I think that user cannot change his email/mobile without verifying them on the spot, so once he is typing a new one, he automatically directed to the same screen as on signup waiting for user to copy-paste the code.
Maybe just a popup message that will inform the user that he needs to verify his mobile/email and then direct him to the signup screen with the code...

@LiavGut lets think of the exact flow...

@LiavGut need exact flow

I'll work on that

@sirpy

User is editing his phone number/email.
User clicking outside of text field / clicking the save button
User is automatically directed to one of these screens (NO SAVING the new phone/email without verifying them):
https://xd.adobe.com/spec/8f259bcd-550e-4f0c-40bb-bdce2ed36fe5-f914/screen/b95c7ad6-5fd8-4f5c-9f29-605c61b501f0/YOUR-Profile-Edit-Verify-Phone

https://xd.adobe.com/spec/8f259bcd-550e-4f0c-40bb-bdce2ed36fe5-f914/screen/e254547c-6dc0-43b8-ae68-d6b655cb43cd/YOUR-Profile-Edit-Verify-Email
4. The user then continue the signup funnel of the code verification process.
5. If completed successfully - User directed to User profile screen (and the new number/email is shown in his details. If NOT - User directed to User edit profile screen (and the old number/email is shown in his details.

@sirpy Do you think we need a success/failed popups? or just what we have in the original signup flow?

@LiavGut we will need an option to exit the verification. during registration you must succeed. here we need some cancel button.

@LiavGut is that a two step? we first show him the we need to verify you email/mobile message only then we send him the email/sms? we can't just send him directly to "we sent an sms/email, type the code"?

@sirpy Yes, it's a two-step process.
First, the user will confirm it's number/email and that he really want to change it.
Then, after he confirms, we'll send him the code.

@Nordwhale this is ready for devs

@LiavGut So should we add some kind of Back or Cancel button on the Step2?
https://xd.adobe.com/spec/8f259bcd-550e-4f0c-40bb-bdce2ed36fe5-f914/screen/0dcfb5c7-054d-4d13-a542-e761d54d15e9/Sign-Up-19
for ex. if a user didn't get SMS or email for some reason, there isn't a possibility to go back and continue using GD with old email/phone.
Please advise.

@LiavGut Sorry. found the answer in your flow. If entered code is incorrect, then we open User profile and show old phone/email.
But I still think that we should have some sort of notifications ( message or icon) of successful or failed verification. It can be difficult for users to not get any error and notice that a phone wasn't changed. I expect to see what went wrong if my new phone is not saved.

@Nordwhale Yes, But I think you still right and we should add a back arrow in this case.
And we should also add success/fail popup message.

@LiavGut please provide text for success/fail popup.
When a user presses the back button on 'confirm change number/email' or 'type verification code' screen - he should be redirected back to edit profile page and see his old number/email?

Link to GoodServer's PR - https://github.com/GoodDollar/GoodServer/pull/91

@yaroslav-fedyshyn-nordwhale
Let's leave it for now without the Success/Fail messages.
Thanks :)

@LiavGut I came up with the text ин myself - it can be a temporary solution. We can change it at any time in the future.

@yaroslav-fedyshyn-nordwhale Thanks, but we decided that we don't need these messages.

@LiavGut but what will happen if the user enters the wrong verification code by mistake? what should happen then?
so in case of success verification just redirect a user to profile page?

@yaroslav-fedyshyn-nordwhale In case of an error, he will receive the regular errors, like in the sign-up flow (ie red text error beneath the input field)