[Fonts] Icon request: water_drop

Hi there!
I hope everyone is doing just fine, I am a flutter developer from Pakistan and sometimes it is very hectic to find or design png icons that matches the flutter material icons. Specially the water drop kind. It will be very helpful if this icon also available on flutter along with the others.
Have a great day!
Thanks
Ayesha Iftikhar

Hi @AyeshaIftikhar, Thanks for filing the issue. Please ensure that you are using the latest stable version of flutter by running flutter upgrade. The new release of flutter already contains the following water_drop icons

flutter doctor -v (mac)
[✓] Flutter (Channel stable, 2.10.4, on macOS 12.3 21E230 darwin-arm, locale en-GB)
    • Flutter version 2.10.4 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c860cba910 (3 days ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.65.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.37.20220301

[✓] Connected device (4 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:55555                    • android-arm64  • Android 11 (API 30)
    • iPhone 12 Pro (mobile) • 535317A4-921C-488F-B234-C2CC6D4AE5A1 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 99.0.4844.83

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

@maheshmnj
Thanks for responding quickly. I was using flutter version 2.10.3. Now, after updating to the latest version I have found the icons.


Error: Failed to get attached devices. on Visual Studio Code with Azure Sphere UI Run Start Debugging

Error: Failed to get attached devices.  on Visual Studio Code with Azure Sphere UI Run Start Debugging

@mcousine Thanks for the feedback! We are currently investigating and will update you shortly.

@mcousine Apologies for the long delay. Can you please share the documentation you are following for the steps?
I'm unable to reproduce this issue. Can you confirm your device OS and SDK versions? Make sure you have updated the OS to the latest version.
To verify the installed OS version on an attached device, use the following command:

azsphere device show-os-version

To check which version of the SDK is installed on your local computer, use the following command:

azsphere show-version

Also I suggest you follow Troubleshoot Azure Sphere problems for common issues and resolutions.

@mcousine This channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements. After reviewing the ask, it seems like generic query and not a doc feedback.
Hence, we request you to please reach out in forums like Microsoft Q&A We have our engineers and other community members who will be happy to assist. Forums have a better suited audience for such queries. Once your issue is posted in forum, please do share the forum thread link here for tracking.
If you have a document which you are following and if you think it needs an update, please let us know about that here.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.
Thank you


Optimise GetBoundCopy to not rely on reflection to create bindable instances

Closes #4685

Method
Mean
Error
StdDev
Ratio
RatioSD

GetBoundCopyOld
2.352 us
0.0425 us
0.0840 us
1.00
0.00

GetBoundCopy
1.071 us
0.0212 us
0.0343 us
0.46
0.02

While this avoids reflection-based metods of creating a bindable instance, the new protected CreateInstance method has to check that the returned instance matches the same type as the original bindable which is to be bound against, and throw if it doesn't match.
Doing that causes JIT to disable inlining on the method (see https://github.com/ppy/osu-framework/issues/3470). I've defined a ThrowHelper class for that purpose, as otherwise GetBoundCopy would've been less performant.

The fact that all inheritors of Bindables need to override CreateInstance is a bit disturbing, but since the chances of making a new bindable class is rare, I believe it's worth the optimisation.
I've left LocalisedBindableString from implementing CreateInstance on it as it doesn't seem possible to do so, and it's already broken before so I don't see any way of supporting that currently. Leaving it for later when deemed required.

Why does the throw need to be there? Is this intended to protect third parties that may be implementing their own bindable incorrectly? Not sure we need that.

In addition, I'm not sure why you've listed four types of invocation in the benchmark when there should only be two compared - what we were using and what the new code is providing. It's unclear immediately which two we should be looking at (guessing the first two, where the... second is the old?), especially since the baseline is set incorrectly to the new one, rather than the old.
Here's a better way (in my opinion) to create benchmark comparisons for PR descriptions:

Copy the pre-changes version of Bindable to BindableOld
Create two benchmark methods with the same content, one instantiating a Bindable and the other BindableOld. Set the old one as the baseline

Why does the throw need to be there? Is this intended to protect third parties that may be implementing their own bindable incorrectly? Not sure we need that.

Added that for two reasons:

Would act same as what happens when invoking Activator.CreateInstance on a bindable which doesn't implement a matching constructor.
Avoids the chances of forgetting to override CreateInstance, which can happen silently and result in unexpected behaviour. I can imagine that potentially happening osu!-side, with bindables that override the Value setter for example.

That said, I'm wondering if there's a reason you're against having it?

In addition, I'm not sure why you've listed four types of invocation in the benchmark when there should only be two compared - what we were using and what the new code is providing. It's unclear immediately which two we should be looking at (guessing the first two, where the... second is the old?), especially since the baseline is set incorrectly to the new one, rather than the old.
Here's a better way (in my opinion) to create benchmark comparisons for PR descriptions:

Copy the pre-changes version of Bindable to BindableOld
Create two benchmark methods with the same content, except that one instantiates a Bindable and the other BindableOld. Set the old one as the baseline

I thought Baseline is supposed to be set on the pure way of creating a bindable, then the other benchmarks would scale based on it to show the ratio between the pure way and the other methods. But perhaps not.
Your way sounds good, I can't exactly copy the pre-changed version of Bindable, but I can at least make a bindable that overrides to create the instance via Activator to reflect the overhead.

Benchmarks updated, shows a ratio of 0.50 on the new way.

I guess the throwing is required due to the virtual implementation in Bindable, seems fine to leave it.
The new benchmarks make me question whether this was required in the first place. At 2.3us this method likely does not need this change.

I've refactored this to remove the back-and-forth casting.

IBindable exposes a GetBoundCopy() method.
Custom interfaces like IBindable<T> can expose their own typed variant of GetBoundCopy()
The final implementation (e.g. Bindable<T>) implements GetBoundCopy() via IBindable.GetBoundCopyImplementation(this).
IBindable.GetBoundCopyImplementation() is only available within IBindable subclasses.
IBindable.CreateInstance() is now only protected instead of protected internal.

Method
Mean
Error
StdDev
Ratio

GetBoundCopyOld
1,729.3 ns
2.92 ns
2.59 ns
1.00

GetBoundCopy
797.9 ns
7.01 ns
6.22 ns
0.46

Relative performance hasn't changed from the OP.

Code seems good to me. I've added a breaking change note to the PR description.
@frenzibyte before I go merging this, do you have the corresponding game-side changes ready? because this will break custom bindable types and the game has a few of those.

Yeah I'll ready up one now.


Implement Simplify, GetPoints

Implement Simplify, GetPoints

Thank you for the PR.
This package implements a Go wrapper for functions in the GEOS library. Geom.GetPoints is not part of the GEOS library, and so should not be included.
Geom.Simplify is already part implemented and so can also be removed.

Without those 2 things this PR seems like it can simply be closed. Perhaps a solution using GEOSWKTWriter_write_r or similar might be a more suitable alternative to GetPoints?

OK.
Geom.ToWKT already uses GEOSWKTWriter_write_r, although you mentioned in another communication that this crashes when used concurrently. Do you have an example that reproduces this?


[Feature Request] add new backup file support

Why/User Benefit/User Problem
Works with new backup files(proto.gz)
What/Requirements
In the backup section there will be 3 options:

Create Backup (using the new .proto.gz format)
Create Legacy Backup (using the original .json format)
Restore Backup (which will work with .proto.gz and with .json)

In the newest release


Add Link component and component context

Visual
The Link component would be mostly functional and would have no specific visual styles itself.
Context
The Link component would be used wherever an <a> element would normally be used, such as in the Button or Navigation components. It would be particularly useful for more complex components on the molecule or organism stage in the Atomic Design methodology.
In order for the Link component to remain application-agnostic, there needs to be a way to customize it. This is where the component context comes into play. It allows to override the Link component with a custom one, e.g. from react-router, reach-router or Next.js.
The Link component API should follow the APIs of common routing libraries.
Here's an example of how the pieces fit together:
// Navigation.js in Circuit UI

export default function Navigation({ links }) {
  const { Link } = useComponents();

  return (
    <nav>
      {links.map(({ url, label }, i) => (
        <Link key={i} to={url}>
          {label}
        </Link>
      ))}
    </nav>
  );
}

// CustomNavigation.js in application

import { ComponentContext, Navigation } from '@sumup/circuit-ui';
import { Link } from '@reach/router';

export default function CustomNavigation(props) {
  return (
    <ComponentContext.Provider value={{ Link }}>
      <Navigation {...props} />
    </ComponentContext.Provider>
  );
}

The component context could be used for other components as well, e.g. a Head component. I'm successfully using this approach in my personal component library Bamboo UI.
State
Does this component have state? No.
Progressive enhancement
By default, the Link component renders an <a> element with an href attribute so a user is still able to navigate when JavaScript is disabled or crashes.
⚠ This would likely require a breaking change.

But how different is this from a regular <a> then?

Let's take the Sidebar as an example. The navigation links need to be styled but are also deep in the sidebar structure and not easily accessible/customizable from the outside.
You could just render an a element, but that would do a normal browser navigation and trigger a full reload. In a SPA, you want to intercept the click event and handle the navigation programmatically. This is usually done with a Link component.
Often, these Link components do more than just attaching an onClick handler, such as transforming a to prop for a route into a full href. So a Link component that can be customised is the more powerful solution.
Does that make it clearer?

:tada: This issue has been resolved in version 1.2.2-canary.5 :tada:
The release is available on:

npm package (@canary dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update thread_win.c to support Embarcadero's clang based compilers

CLA: trivial

Checklist

To support Embarcadero's clang-based compilers the following code was added to the file crypto/thread/arch/thread_win.c just before function void ossl_crypto_mem_barrier(void).
#if defined(__BORLANDC__) && defined(__clang__) && defined(_WIN32)
__forceinline void MemoryBarrier(void)
{
    long barrier;
    __asm {
        xchg barrier, eax
    }
}
#endif

Without this change, library creation fails because it cannot find the _MemoryBarrier function,

Thanks for your contribution, @gcardi!
Instead of just repeating your patch in the description, it would be helpful if you could provide some background information, since we know little about the embarcadero context.  E.g.,

What is the original error and in which context does it occur?
Does embarcadereo also have C/C++ toolchains or does the problem occur in some Delphi wrapper around OpenSSL?
Why does the MemoryBarrier have to be implemented by the Application/Library? Shouldn't it be a compiler intrinsic or a library function of the embarcadero framework?  A quick search only revealed that there is some  System.MemoryBarrier intrinsic in embarcadero Delphi.

Please also note, that in order to accept the pull request, you need to sign and send us a Contributor License Agreement (CLA).

What is the original error and in which context does it occur?

It's a linker error:
Error: Unresolved external '_MemoryBarrier' referenced from C:\USERS\GIULIANO\DOCUMENTS\EMBARCADERO\STUDIO\PROJECTS\OPENSSL_BCC32\CRYPTO\THREAD\ARCH\LIBCRYPTO-SHLIB-THREAD_WIN.OBJ
Error: Unable to perform link

It occurred building the library with
make -N

Does Embarcadereo also have C/C++ toolchains or does the problem occur in some Delphi wrapper around OpenSSL?

I honestly don't know. I tend to use Delphi as little as possible.

Why does the MemoryBarrier have to be implemented by the Application/Library? Shouldn't it be a compiler intrinsic or a library function of the Embarcadero framework?  A quick search only revealed that there is some  System.MemoryBarrier intrinsic in Embarcadero Delphi.

If I am not mistaken, the OpenSSL library is written in pure C. Or am I wrong? To consume objects or generally code written Delphi you have to use either Delphi itself or C++, to my knowledge. I don't think you can easily link Delphi code from a C codebase with C++Builder, unless that code is compiled into a dynamic library (DLL). But I could be wrong.

Please also note, that in order to accept the pull request, you need to sign and send us a Contributor License Agreement (CLA).

Just done.
Regards

From: Tomáš Mráz
Sent: Friday, October 13, 2023 1:12 PM
To: openssl/openssl
Cc: Giuliano Cardinali ; Mention
Subject: Re: [openssl/openssl] Update thread_win.c to support Embarcadero's clang based compilers (PR #22361)
@t8m requested changes on this pull request.

We will need a CLA signed.

Already sent

In crypto/thread/arch/thread_win.c:

@@ -32,6 +32,17 @@ static unsigned __stdcall thread_start_thunk(LPVOID vthread)
return 0;
}

+// To support Embarcadero's clang based compilers

Please do not use C++ style comments.

Ok
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I'm a bit unsure on this.  From what I read here:
https://learn.microsoft.com/en-us/windows/win32/api/winnt/nf-winnt-memorybarrier
MemoryBarrier is a windows platform macro defined in winnt.h
So normally we pull that in by including windows.h in thread_win.c, which (at least on the msys clang and mingw64 platforms get the definition via windows.h->windef.h->winnt.h
If embarcaderos compiler somehow has a different header structure, wouldn't it be better to do something like this in thread_win.c:
#if defined(OPENSSL_THREADS_WINNT)
# include <process.h>
# include <windows.h>
+#if defined(__BORLANDC__) && defined(__clang__) && defined(_WIN32)
+#include <winnt.h>
+#endif

Rather than redefining the function to something else (winnt.h defined MemoryBarrier as __mm_fence), while this implementation does an xchng.  The former serializes all memory operations, while I think the latter only guarantees that reads/writes to the desitnation address are serialized)


Add a CSS text editor

Idea is to have an option to toggle between GUI and stylesheet.

it would also be lovely to have the CSS appear in an editor on the right too, as well as toggles/GUI style
@Harjot1Singh

Resources:

https://github.com/Microsoft/monaco-editor
https://codemirror.net
http://www.apnapedia.com/create-html-online-editor/

https://github.com/JedWatson/react-codemirror

https://github.com/FormidableLabs/react-live


refactor(linter): use meaningful names for diagnostic parameters

#5564  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite


feat(core): Add support for ollama module

Added a new class OllamaContainer with few methods to handle the Ollama container.

The _check_and_add_gpu_capabilities method checks if the host has GPUs and adds the necessary capabilities to the container.

The commit_to_image allows to save somehow the state of a container into an image so that we can reuse it, especially for the ones having some models pulled.

Added tests to check the functionality of the new class.

Note: I inspired myself from the java implementation of the Ollama module.

resolving the issue #617


Delay to change a scene if the current scene has many objects

Version
3.17

Phaser Version:

windows 7

Operating system:

Browser:

Description

there are 7200 tiles object, tile size are 352x64 (separate with 4 sprite) to cover the game map.
If i switch scenes or start another scenes, it’s take about 7 seconds until it’s moved to target scene.
but starting game scene are quite fast, look like there are no delay to process.
My test link http://cdn.redfoc.com/test/p3_tiles
Example Test Code

class load extends Phaser.Scene {
constructor(){
super('load');
}
preload(){
this.load.spritesheet('tiled', 'tiles.png', {frameWidth: 88, frameHeight: 64});
}
create(){
this.scene.start('menu')
}
}
class main_menu extends Phaser.Scene {
constructor(){
super('menu');
}
create(){
this.add.text(20,20, 'Press A to change scene',{fontFamily: 'Arial', fontSize: '30px',color: '#fff'});
this.input.keyboard.on('keydown', function(e,f){
if(e.key == 'a'){
this.scene.start('game')
}
}, this);
}
}
class main_game extends Phaser.Scene {
constructor(){
super('game');
}
preload(){
this.load.spritesheet('tiled', 'tiles.png', {frameWidth: 88, frameHeight: 64});
}
create(){
var size = {
w: 60,
h: 120
}
for(var y=0; y<size.h;y++){
for(var x=0; x<size.w; x++){
var o = this.add.sprite(x88,y64,'tiled');
o.setFrame(Math.round(Math.random()*3));
}
}
this.add.text(20,20, 'Press B to back to previous scene',{fontFamily: 'Arial', fontSize: '30px',color: '#fff'});
this.input.keyboard.on('keydown', function(e,f){
if(e.key == 'b'){
this.scene.start('menu')
}
}, this);
}
}
var config = {
type: Phaser.AUTO,
width: 1280,
height: 720,
scene: [load, main_menu, main_game],
}
var game = new Phaser.Game(config);
Additional Information

"Try adding o.destroy = function(){}; in your nested for loop, like so:
for(var y=0; y<size.h;y++){
for(var x=0; x<size.w; x++){
var o = this.add.sprite(x88,y64,'tiled');
o.setFrame(Math.round(Math.random()*3));
o.destroy = function(){};
}
}"
"Unfortunately this is just how Phaser is processing the destruction of it’s GameObjects. I would suggest creating an issue on Phaser’s Github so others can look into this. Worst case, this is a necessary evil and not an issue. If that is the case, you can choose whether to preference memory (not using destroy = function(){}; or performance (using destroy = function(){};)."
by Jake.Caron

I don't really get what this issue is about. Are you asking why it takes a few seconds to destroy thousands and thousands of Sprites? A better question may be, why are you creating that many in the first place and why destroy them? :)

Actually i use regular sprite objects as a tilemap to draw/fill game map, i'm not using built-in tilemap since my game style are isometric.
I think if i change another scenes, current secene objects are destroyed, is that right ?
Someone from the forum tell me to use "Blitter" instead of sprite objects, now my problem are fixed.


Renaming a symbol in a file shared by two projects only affects the files in one

If you open two typescript projects sharing a common file with a symlink, and then do a "Rename Symbol" in the shared file, ONLY the files in the project were you did the operation  will be affected by the rename, rendering the other project files inconsistent. Steps to reproduce:
1- Create /tmp/a and /tmp/b projects and /tmp/c shared directory
cd /tmp ; mkdir a b c
cd a ; npm init ; tsc --init
ln -s ../c c
cd ..
cd b ; npm init ; tsc --init
ln -s ../c c

2- Create files:
/* /tmp/c/fc.ts */
export const C = 8

/* /tmp/a/a.ts */
import {C} from "./c/fc"
console.log(C)

/* /tmp/b/b.ts */
import {C} from "./c/fc"
console.log(C)

3- Now open both folders a and b in VS Code, then open a/c/fc.ts and do a "Rename symbol" on C. Only the files at the a folder will be affected. In the b project fc.ts will remain unchanged until you save it in the a project, then it will show the changes but it will not notice is a symbol change, so the b files will not be updated with the new symbol.

VSCode Version: 1.16.1 27492b6bf3acb0775d82d2f87b25a93490673c6d
OS Version: Ubuntu 17.04

This issue was moved to Microsoft/TypeScript#20021


Avoid spurious 'Upgraded tool environment' in uv tool upgrade

Summary
Closes https://github.com/astral-sh/uv/issues/9869.

Thanks a bunch !


story insight ideas

taylor swift low diva score
inaguration high score on "FLAG"
jazmine sullivan percentile, good != chaos

Autotune


Added DDP for contact-floating model: it is working! - [closed]

In GitLab by @nmansard on Jan 18, 2019, 17:09
Merges topic/refact -> master

In GitLab by @nmansard on Jan 18, 2019, 18:51
added 3 commits
ebf1a0f6 - [refact] added weights to the state cost.3fde55f4 - [refact] add armature.0cbd923e - [refact] Make test ddp more complex with end-eff contact.
Compare with previous version

In GitLab by @cmastalli on Jan 19, 2019, 13:55
Please see comments on PR !43

In GitLab by @proyan on Jan 19, 2019, 18:59
@nmansard
It is difficult to review the merge request, since it is merging 52 commits to the master branch.
Could you please change the destination branch to topic/refact?

In GitLab by @proyan on Jan 20, 2019, 13:22
I'm closing this PR and opening another which takes your commits and merges them to the topic/refact branch

In GitLab by @proyan on Jan 20, 2019, 13:24
mentioned in merge request !44


[Feature Request] Add opensearch-node subcommand to print current metadata

Is your feature request related to a problem? Please describe
Hi,
I have a use-case where I want to automatically, once I attach a new volume to a given node: detect if a given disk already has data from OpenSearch indices and if that data corresponds to the existing cluster or is coming from another cluster.
The goal is, in the event of a disaster, to recover by attaching the disks to another cluster and recovering the original data.
For that, I need to be able to print the metadata available in the disk.
Describe the solution you'd like
I have crafted a small opensearch-node-like CLI command that prints the metadata here: https://github.com/phvalguima/opensearch-metadata-cli
I'd love to evolve it to print a json-formatted output with the metadata of the current node and contribute it back to OpenSearch. Is it something the community is interested on?
Related component
Cluster Manager
Describe alternatives you've considered
No response
Additional context
No response

[Catch All Triage - 1, 2, 3, 4]

Hi @dblock I was working on that change btw. If that is something of interest, I'd contribute that standalone command back to OpenSearch.

Please do! Assigned this issue to you.


Improve the UI refresh rate

Allow the UI to refresh the display when any user input changes the UI state, but throttle the UI refresh rate to 5hz/200ms. Any clock input or reset input will force the UI to update regardless of UI refresh rate.

Thanks Adam. I think we're really getting close now.


Error while attempting to download

I am using the latest build. I was downloading successfully earlier, then I had an error with one manga. After closing the command prompt and launching the script again, I get the following error message
G:\Program Files (x86)\Fakku Downloader>python main.py
Using cookies file: cookies.pickle
DevTools listening on ws://<IP_ADDRESS>:52748/devtools/browser/27c763ee-900c-451f-9a36-e4e9921fcfb8
<class 'str'>
Traceback (most recent call last):
File "G:\Program Files (x86)\Fakku Downloader\main.py", line 142, in
main()
File "G:\Program Files (x86)\Fakku Downloader\main.py", line 138, in main
loader.load_all()
File "G:\Program Files (x86)\Fakku Downloader\downloader.py", line 221, in load_all
page_count = self.__get_page_count(self.browser.page_source)
File "G:\Program Files (x86)\Fakku Downloader\downloader.py", line 306, in __get_page_count
return int(pages_info[0])
ValueError: invalid literal for int() with base 10: 'No'

Nevermind, one day later and the issue appears resolved.

Error appeared to have disappeared, but after downloading one manga normally, the issue has returned


Cache dataset to accelerate trials

Is your feature request related to a problem? Please describe.
Currently, trial runner will reload dataset every time a new trial starts, that's quite inefficient for large dataset. ML.Net provides data cache to eliminate thus inefficiency.
Describe the solution you'd like
A clear and concise description of what you want to happen.
cache dataset to improve training speed.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Dev Work Tracking Task - Nothing to validate.


LDAP login appears to authenticate, but login does not occur

LDAP authentication may fail to save the user information in the session if it fails to correctly store the user information from the LDAP server into the session.  I am not sure what exactly causes this to happen on mine (perhaps some binary properties or just raw size) but it is definitely something with the user object returned.
The authentication does occur and everything is set on the req._passport.session.user object, but upon the next request, the req._passport.session.user object is undefined again causing the login to appear successful (no error) except that the resulting page shows the user still not logged in.
Manually filtering out everything except the displayName and mail properties from the resulting user object works around this, but there does not appear to be a way to do this from the config.yaml file at this time.

Pinging @everpcpc on this one

This should be closed. https://github.com/claudioc/jingo/pull/179 fixes this problem.
I experienced the same problem. Adding the attributes section to the config worked like a charm:

searchAttributes:
 - displayName
 - mail


Log Error if link is down (invalid speed)

If the link is down (AdapterSpeed is 0/NULL) few variations  like *MaxRSSProcessors, *NumRSSQueues and NicSwitch(*SRIOV) will fail due to $thisDefinitionPath resolved to NULL. The code assumes the previous parameter's definition and eventually validation fails.

Adding check for AdapterSpeed
Reinitializing $thisDefinitionPath inside switch.

Thank you @sandilyabhagi !!! This was very helpful. Apologies for the length of time that passed between you first opening this.

@dcuomo, much obliged to contribute.


Scrapoxy test fails

After running
scrapoxy start conf.json -d
and wait 3 minutes, I run
scrapoxy test http://localhost:8888
it gives me this response
2018-02-03T22:53:48.034Z - error: [Test] Error: Cannot get IP address: 500: [Master] Error: request error from target (GET http://api. ipify.org/ on instance 81206430@<IP_ADDRESS>:3128): Error: read ECONNRESET
What's the problem here?
Mostafa

duplicate of https://github.com/fabienvauchelles/scrapoxy/issues/97


[vault-secrets-webhook] Failure to deploy when pod is re-created on another node

Twice now I have observed that after installing the vault-secrets-webhook helm chart into its own namespace, that it fails to deploy again when moved to another worker node. This then fails all other deployments as they cannot reach the admission controller. I don't see anything wrong with the initial deployment or any settings to modify. It has been placed into an isolated namespace per the installation guidance. Any thoughts? We are running on EKS (Kubernetes v1.10).

@bonifaido Since re-deploying with the updated guidance everything now looks good. I've been running the webhook for several days without any of the aforementioned issues.

Great @jshrack-ssi , I will close this down then, feel free to open/reopen if you encounter any issues again.


Preventing negative indexes

Addressing (#198)
This preserves ordering when offest is negative. Using Math.Abs would reverse the ordering when offset is negative.
There is a discontinuity when offset transitions from positive to negative. This should be acceptable "unfairness" over 2B requests.
For example with endpoints.Count of 5:
offset		computed index
2147483640	0
2147483641	1
2147483642	2
2147483643	3
2147483644	4
2147483645	0
2147483646	1
2147483647	2
-2147483648	0
-2147483647	1
-2147483646	2
-2147483645	3
-2147483644	4

Woops, doesn't quite compile.

It sure feels like a Monday...
Is return endpoints[(int)((uint)offset % endpoints.Count)]; too much casting? Or just enough?

Your choice.

Masking with (i & 0x7FFFFFFF) % count: sharplab.io
    L0000: push esi
    L0001: mov esi, edx
    L0003: mov eax, ecx
    L0005: and eax, 0x7fffffff
    L000a: cdq
    L000b: idiv esi
    L000d: mov eax, edx
    L000f: pop esi
    L0010: ret

Multiple casts with (int)((uint)i % count): sharplab.io
    L0000: push ebp
    L0001: mov ebp, esp
    L0003: sub esp, 8
    L0006: push 0
    L0008: push ecx
    L0009: mov eax, edx
    L000b: sar eax, 0x1f
    L000e: push eax
    L000f: push edx
    L0010: call 0x0f8e4b50
    L0015: mov [ebp-8], eax
    L0018: mov [ebp-4], edx
    L001b: mov eax, [ebp-8]
    L001e: mov edx, [ebp-4]
    L0021: mov esp, ebp
    L0023: pop ebp
    L0024: ret

Thanks for trying a few variations.


[Actions] Adding a MacOS build workflow, fixing several MacOS specific compilation errors and warnings

Well done, small remark, but now we have a warning free build on Apple as well ?

Indeed we have! At least for Thunder, but other repos will follow shortly 😄


fix(material-experimental/mdc-chips): chip set not picking up indirect descendant chips

Fixes the MDC-based chip set not picking up indirect descendant chips.

mdc-chips is actually in use by a team internally, so marking as not merge-safe


Support for Rails 5.2.3

The latest version of Rails5.x is 5.2.3.
https://rubygems.org/gems/rails/versions

Close this PR because @negito6 fixed


Query on effect of connlimit on Control plane topology and data plane topology

If we have n nodes , and have connlimit to default of 100 ,
does it mean control plane topology will have n^2 connections ie a full mesh  but data plane topology will have only 100 connections, ie a partial mesh . ?
will increasing connlimit to n^2 , does it mean we move from partial mesh to full mesh in data plane as well ?

This question is better addressed to the weaveworks mesh project, which is what the weave net router uses. I believe the limit applies to both planes.

ok thanks .


prefix an executable with a command

How can I prefix wine with a command in the apprun file?
I want to add this line __GL_ExtensionStringVersion=17700
so the command looks like __GL_ExtensionStringVersion=17700 wine

Hi @Blad3forc3, on this line:
"$YQ" --exit-status 'tag == "!!map" or tag == "!!seq"' "$YAML" &>/dev/null || echo "cmd: \"{wine} {exec}\"" > "$YAML"

You can change it to:
"$YQ" --exit-status 'tag == "!!map" or tag == "!!seq"' "$YAML" &>/dev/null || echo "cmd: \"__GL_ExtensionStringVersion=17700 {wine} {exec}\"" > "$YAML"

Thanks that worked. Any chance of updating the Staging wine package to 8.2?

Nice, sure I just started a manual pipeline in github-ci, you can follow the progress here

Great thanks. Any news on being able to use the older wine 1.50?

No problem. Still working on it, but facing some issues to make it work.

OK. Still testing this so if anything crops up i'll create a new issue. Thanks again


Why the number of features is 2 for each station?

X = X.transpose((1, 2, 0))
Why the number of features (observation) is 2 at a specific time for each station?

X = X.transpose((1, 2, 0))
Why the number of features (observation) is 2 at a specific time for each station?

老哥知道为什么是2了吗？我也不是很懂，请教一下


Implementation of HPKE

This PR implements HPKE in the current version of the specs. It also uses @protz's tactic to automatically multiplex between different implementations of Curve, ChachaPoly, and SHA256 and SHA512.

Thanks for the comments. Some replies below:

Regarding assume vals: I agree it would be nicer, but we unfortunately don't have (yet?) implementations for all algorithms we are multiplexing on (for instance Curve448). I'd be in favour of an assume val that is well-understood (happy to add a comment to it if needed) than a match with admits because we don't have full implementations yet.

Regarding sequence associativity: It seems that the assertions indeed trigger the lemmas in the correct way. This proof does not seem to be flaky. The issues are more with functions that perform a lot of allocations on the stack (and hence need to be split into push/pop + allocations and actual body of the function).

Regarding merging soon: I'd be in favour as well. @beurdouche reported that the spec would change one last time in the next few days before being stable for a while. I'm happy to wait until it changes, propagate changes to this branch and then merge into dev

Thanks Santiago for the comments. I added an interface for the spec and addressed the smaller items as well.
Since the spec has not moved in the passed week on dev, I'm merging this now


Fix Issue 14870 Add new "abort" internal module and function

See discussion here<EMAIL_ADDRESS>abort(msg) Will ALWAYS print msg, unlike assert(0, msg) which does not when compiled in -release mode. abort will eventually call assert(0), so the end result is similar, but with a good message for debugging.
The only drawback is that assert(0, msg) is pure, but this function cannot be, since it has no mutable references as parameters or return, and the functions it calls to output the message are not pure.
So we can't replace all assert(0, msg) calls with abort(msg) (I had to leave out one case of assert(0, msg) in core.time because it was in a pure function).
In addition, the optimizer may not see that abort(msg) does not terminate, I'm not sure.
I have reworked core.time to use this internal module instead. I also had to write (another) integer to string conversion routine (following Walter's lead in his presentation for dconf2015, it doesn't allocate). Is there other needs for that anywhere in druntime? Mabye it should go in core.internal.convert, but that seems to be focused on floating point.
The large changes in core.time were regarding a bizarre "Something threw when nothing should have" try/catch scope, and I found that removing the try/catch did not cause any errors. Probably at one point, the internal/referenced code wasn't properly nothrow-annotated.
If this is approved, I'll work on the other modules in druntime that assert(0) with a message, and probably phobos as well.

I have reworked core.time to use this internal module instead. I also had to write (another) integer to string conversion routine (following Walter's lead in his presentation for dconf2015, it doesn't allocate). Is there other needs for that anywhere in druntime? Mabye it should go in core.internal.convert, but that seems to be focused on floating point.

There is something simillar in core.exception for Exception toString IIRC.

There is something simillar in core.exception

Close enough, I found it, actually part of Throwable in object.d. It's in rt/util/string, I'll try and rewrite my code to use it.

So long story short, I had based my branch on an old version of the master by accident. The rt/util/string.d has been moved, and since I already messed it up, I had to rebuild the branch. In any case, it should be OK now.

If you're so interested in a message always being printed even in release mode, why not just throw an Error?

LGTM

why not just throw an Error?

I could do that, but this is not dependent on any external code. It can be called from a signal handler, or just about anywhere. And if the system is corrupted, it should still work.

Well, there are already plenty of places in druntime that throw various Error types, and they're going to have to be caught correctly by _Dmain or whatever to be printed. It seems like abort is just specifically for the assert(0) case, and I don't see anything special about druntime in that regard. The current language design for it is that it's not going to print anything when built with -release, and no one should be seeing those fail except in highly abnormal circumstances anyway. So, this seems like it's purely designed to work around Walter not being willing to have assert(0, msg) print before the HLT instruction in release mode.
In the case of core.time, it probably should be throwing an Error, not asserting 0. It was written with the assumption that things were seriously broken if those calls failed, but since there are legitimate cases where they can fail where it's not a bug in druntime at all, an Error would make more sense, and then you will get a message in that case just like with any of the other Errors that we already throw in druntime.

This is intended to fix https://issues.dlang.org/show_bug.cgi?id=14870

I think that the cases in core.time aside from those related to _d_initMonoTime should just be changed to throw Errors on failure, since they fail when a system call fails, which isn't necessarily a program bug. The program is screwed in that case, but it's probably not its fault.

By the way, for those reviewing this, adding ?w=1 to the end of the URL for the diff definitely helps, because some of the changes altered the indentation, and so a number of the changes are whitespaces changes, and they increase the number of lines in the diff considerably.

number of the changes are whitespaces changes

Yeah, those are the cases where I removed the now-unnecessary try-catch.

OK, updated to take care of comments. Removed numToString from standard code, now uses core.internal.string, changed numToString for doubles to use core.internal.string (this was a unittest-only function).
One odd note, I have found that:
char[] x = "blah".dup;
auto y = "hi, " ~ x;
assert(is(typeof(y == char[])));

This messes up a lot of code, which is why you see idups in places that seemingly shouldn't need them (namely for throwing errors, which only accept string as the message parameter). It would be nice if the compiler simply treated the result of a concatenation as any constancy, since it's clearly a new piece of memory. For cases where the result was used in real code (not unittest), I updated the code to use proper concatenation (it's faster anyway).

I think that the cases in core.time aside from those related to _d_initMonoTime should just be changed to throw Errors on failure, since they fail when a system call fails, which isn't necessarily a program bug. The program is screwed in that case, but it's probably not its fault.

It could be a memory corruption bug (if the 0 value came from corruption instead of unsupported clocks). I'm really on the fence for this, we could go either way.

I'm not really a fan. From the places that are changed in this PR it looks more appropriate to have errors thrown.

It could be a memory corruption bug (if the 0 value came from corruption instead of unsupported clocks).

If it's corruption that could occur through external factors, and not some internal state of core.time, then it should still be an Error to alert of programmer bug.

It could be a memory corruption bug (if the 0 value came from corruption instead of unsupported clocks). I'm really on the fence for this, we could go either way.

I don't see much point in worrying about that. If a memory corruption starts screwing with stuff, then all bets are off anyway, and really, it doesn't matter particularly why the system call failed. We can't continue if it does regardless of why it failed. And since it's not a bug in druntime itself at that point, an assertion isn't appropriate. When I wrote that code, I was pretty much assuming that there was no way that the call would fail, and the assertion was just there to kill the program in case things went seriously wrong somehow, and it did fail. But even then, throwing an Error probably would have been better, and we've now seen a case where clock_gettime can fail due to a normally valid clock type being used on a system where it's not valid, because that system's kernel is too old to support it. So, regardless of whether an assertion would be appropriate if we could reasonably assume that the call would succeed, we now know that there are cases where the call will legitimately fail, but we can't continue if it does, so I think that falls pretty squarely in Error territory.
assert(0) is really supposed to be for truly unreachable code where it can't be proven that it's unreachable (and thus a serious bug if it is reached), and that's not the case with clock_gettime failing.

don't see much point in worrying about that. If a memory corruption starts screwing with stuff, then all bets are off anyway, and really, it doesn't matter particularly why the system call failed.

But abort can be called even when memory is corrupted. It uses no global state or heap memory, only system calls and new stack space. I think this is the best/safest way to print anything if you have reached a place that should not be reached. Throwing an error requires the GC work properly, and that the stack has not been corrupted.

From the places that are changed in this PR it looks more appropriate to have errors thrown.

The usage of abort may or may not be appropriate for those locations in core.time. And I plan to go throughout druntime and phobos and fix all the assert(0) that print messages (but don't really because we compile them in release mode). But I think the abort function has good use for a "last resort" way to communicate what went wrong to the user. It could be used (for example) in the signal handler for seg fault.

It could be used (for example) in the signal handler for seg fault.

Let us hope that the signal handler isn't recursive. ;-)

Let us hope that the signal handler isn't recursive. ;-)

From my understanding, if you trigger a recursive signal, it will simply exit the program.

Updated to use c abort instead of assert(0), moved non-allocating struct for getting temp string outside of the unsignedToTempString function because it's used for both signed and unsigned to avoid confusion with comments.
I also marked core.stdc.stdlib.abort as @safe instead of calling it in a @trusted lambda, because the documentation says it basically terminates the program immediately, so I can't see how it could corrupt memory. If this seems too much, I can do it the other way.

Fix failures on the auto-tester?

I also marked core.stdc.stdlib.abort as @safe instead of calling it in a @trusted lambda, because the documentation says it basically terminates the program immediately, so I can't see how it could corrupt memory. If this seems too much, I can do it the other way.

Makes sense

Fix failures on the auto-tester?

Attempt made, but I also want to fix the issue where the unsigned struct takes into account the negative sign (which reads weird). So hold off for now.

Good to go?

Huh, I guess I already moved the struct. My damned memory :)
So yeah, good to go.

Auto-merge toggled on

Changelog?

Changelog?

Are we using the individual changelogs now, or should I update dlang.org changelog? I see there's a changelog.dd in druntime source...

Use the individual ones (now on stable) please.

Working on this, but I'm wondering, should this even be in the changelog? We don't document core.internal at all.

ping @MartinNowak

I thought it was a public function, no need to mention core.internal and the change to always print errors is quite small too.

My bad - I thought abort was a new public primitive.

It could be made public if it makes sense.


op name couldn't contain '_', '/' ?

Hi :
Name of the node - tower_0/Concat/concat/axis
Name of the node - tower_0/Concat/concat
Name of the node - tower_0/MatMul
Name of the node - tower_0/Add
Name of the node - tower_0/MatMul_1
Name of the node - tower_0/Add_1
Name of the node - tower_0/MatMul_2
Name of the node - tower_0/Add_2
Name of the node - tower_0/Sigmoid
Name of the node - tower_0/pre_2/shape
Name of the node - tower_0/pre_2
Name of the node - tower_0/MatMul_3
Name of the node - tower_0/Add_3
Name of the node - tower_0/MatMul_4
Name of the node - tower_0/Add_4
Name of the node - tower_0/MatMul_5

The above is part of op names in my model. When I use tensorflow c api:TF_GraphGetOpDef to get op detail, I found a problem:
The op name which contain '_' or '/' couldn't be found, the message is ：Op type not registered 'tower_0/Add_4' in binary running on xxx. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.) tf.contrib.resampler should be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.
The op name which do not contain '_' or '/' could be found.  Why?

Both _ and / are reserved characters.

Because some node names were generated by tensorflow self, How c api access these node with name contain '_' or '/' ?

@haolujun Can you please provide some reproducible code for us to expedite the process. Thanks!

@hajuho Are you still facing the issue?

Closing this issue as it has been inactive for more than two weeks. Please add additional comments and we can open this issue again. Thanks!


[string] Only bridge tagged NSStrings to small string form.

Non-tagged NSStrings carry identity separate from their
value. Continue to bridge them lazily, even if they could fit in small
form, to respect this and avoid potential information loss.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - e42c94f25403424bf0a549e91361f9cd0197fe7d

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 5f1ba83e8d8d84ba5266a25912e3580900f233b7

Linux PR testing is borked currently

@swift-ci please smoke test Linux platform

@swift-ci please smoke test linux platform

Whoops, it already passed smoke testing. I meant:
@swift-ci please test linux platform

Build failed
Swift Test Linux Platform
Git Sha - 5f1ba83e8d8d84ba5266a25912e3580900f233b7


Add support for Build Tools 15 (Visual Studio 2017)

Unfortunately this is not so trivial since VS2017 doesn't create those registry entries anymore and keeps the build tools within its installation directory.  Instead, you're supposed to use a specific interop assembly to find the instances installed (or use this PowerShell module). Cake solved this by assuming a base-directory, e.g. C:\Program Files (x86)\Microsoft Visual Studio\2017 and then scanning within that directory for subfolders named Professional, Enterprise or Community. I'm willing to add this as a PR if you're interested.

I tried to call Msbuild for Visual studio 2017 with full path. When i was directly calling Msbuild.exe and compile , it compiled by excluding Unloaded projects. But when i was doing it via Psake ,it tried to build unloaded projects and show compilation issue in them. We unloaded it so that no need to build.

I am trying to call directly the MSbuild with full path for now and it works

Heres a simple fix
Windows Registry Editor Version 5.00
[HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\MSBuild\ToolsVersions\14.0]
"MsBuildToolsPath"="C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\Bin"

Another possible (but maybe not great) solution would be to implement similar to the gulp-msbuild did it.

Never mind. I've decided to try Cake instead.

@dennisdoomen said...
Never mind. I've decided to try Cake instead.

This is pretty much where I am as well.  When I became co-maintainer of the psake project, I had every intention of actively developing/maintaining it, but since then, I have jumped in with Cake, and I haven't looked back since.  I have very much enjoyed using psake, but perhaps it is time to let someone else take over the project.  Perhaps I will open up an issue to see if anyone is interested.

I would appreciate the support in psake. I have too much to move to cake in a timely fashion.

@taylorjonl did that allready, pull request is still open...
See PullRequest: https://github.com/psake/psake/pull/209
We implementet it and its working fine for us now.

There is a tool named vswhere can help you to find the installation path of VS2017 and newer vesion.
https://github.com/Microsoft/vswhere

I use VSSetup module and a special cmdlet - Initialize-MSBuild. I call it from pre-build psake task.

a side question - what other v14 tools will be used if this is only bypassed by directly invoking msbuild.exe from the correct location?

@Kreism, when you planning to release this fix? I do need of it.

Thx @dermeister0 ! Works perfectly for me!

@dermeister0 Regarding "I use VSSetup module and a special cmdlet - Initialize-MSBuild. I call it from pre-build psake task." How do you setup psake to actually use the msbuild from path?

@floekke looks like this line does it. https://github.com/Saritasa/PSGallery/blob/master/modules/Saritasa.Build/Saritasa.Build.psm1#L306


SMT solver fails on a simple case

Hey.
Consider a type
type set (a : Type { hasEq a }) : nat -> Type =
    | Empty : set a 0
    | Next : #n:nat -> (hd : a) -> (tl : set a n) -> set a (n + 1)

Basically, my own list.
Now, function contains:
let rec contains (#n : nat) (#a : Type { hasEq a }) (v : a) (s : set a n) =
    match s with
    | Empty -> false
    | Next hd tl -> hd = v || (contains v tl)

Now consider three "tests":
let containsWorks1 = assert(~(contains 3 Empty))
let containsWorks2 = assert(contains 3 (Next 3 Empty))
let containsWorks3 = assert(contains 5 (Next 3 (Next 5 Empty)))
let containsWorks4 = assert(contains 3 (Next 3 (Next 5 Empty)))
let containsWorks5 = assert(~(contains 4 (Next 3 (Next 5 Empty))))

The only one that can't be solved is containsWorks2 which is no more complicated than others.
However, there's a fix.
let containsWorks2 = assert(contains 3 (Next 3 Empty))

->
let v = (Next 3 Empty)
let containsWorks2 = assert(contains 3 v)

I just took the set out of there into a variable, now it's solved. I guess it's not expected behaviour, is it?

Thanks for the report, it should work now in the master branch.

Thanks!


Merge layers do not pass masks to the compute_mask function

After Keras & TensorFlow upgrade to 3.3 and 2.16.1 I encountered a bug with mask propagation in merge layers. A base Layer class in __call__ function takes a first argument and treats it as a previous_mask (link to source). It produces an incorrect mask propagation in Merge layers where arguments are passed as a single list of tensors (and list doesn't have _keras_mask attribute).
Example code to reproduce unexpected behavior:
from keras import layers, random, ops, Model

a = random.uniform([1, 2, 256], dtype="float32")
b = random.uniform([1, 2, 256], dtype="float32")

a._keras_mask = ops.convert_to_tensor([[True, False]])
b._keras_mask = ops.convert_to_tensor([[True, False]])

add_op = layers.Add()

x = add_op([a, b])
expected_mask = add_op.compute_mask([a, b], [a._keras_mask, b._keras_mask])

assert hasattr(x, "_keras_mask")

I see, that the code part is marked as TODO, are there any plans to fix it?
=== Edit ===
Similar code works with Keras 2.15 and TensorFlow 2.15:
import tensorflow as tf

a = tf.random.uniform([1, 2, 256], dtype="float32")
b = tf.random.uniform([1, 2, 256], dtype="float32")

a._keras_mask = tf.convert_to_tensor([[True, False]])
b._keras_mask = tf.convert_to_tensor([[True, False]])

add_op = tf.keras.layers.Add()

x = add_op([a, b])
expected_mask = add_op.compute_mask([a, b], [a._keras_mask, b._keras_mask])

assert hasattr(x, "_keras_mask")
assert tf.reduce_all(x._keras_mask == expected_mask)

The issue has been fixed in the main branch, but it hasn't been released yet.
Related PR: #19685

@jbed94 , You can use the Keras nightly till the next release, here is the working Gist using Keras-nightly


Restore ability to recover account

EOS removed account recovery ability, but able to use wrap to change authority of an account. We need to restore account recovery

original issue: https://github.com/EOSIO/eos/issues/2854 and missing commit (wrong tag): https://github.com/EOSIO/eos/commit/e2ee6c6c6d3d216d1d083fd1ce0c56bc51d63a1e

Decided to use build-in multisig with waits mechanism instead of restoring recovery.
PR #638


Build task doesn't reuse restored task terminal

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Open VS Code repo in an out dated Insiders
Ctrl/Cmd+Shift+B to run default build task
Update Insiders
Ctrl/Cmd+Shift+B again to run default build task

/duplicate #155163
should be fixed in insider's tomorrow if my PR checks ever pass

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


SourceKit: sever dependency on TSCUtility

The dependency on TSCUtility was strictly for identifying the platform of execution. This logic is relatively self-contained and effectively an extension over an enumeration. Replicate this logic with updates for new syntactic improvements. This allows us to partially reduce dependency on swift-tools-support-core. The dependency on TSCBasic is more complicated to remove due to the extensive use of AbsolutePath.

@swift-ci please test

Yes, except, it seems that this functionality is used only in LSP.  The other piece of the interface is in SPM only.  As a result we can split it up this way and deprecate it in TSC.

@swift-ci please test

https://github.com/apple/swift-package-manager/pull/6011
https://github.com/apple/swift-tools-support-core/pull/378
Are the other parts of this change that together deprecate TSCUtility.Platform.  It really is unfortunate that the symbol has to be public as it is used across multiple modules.

@swift-ci please test macOS platform


Add Key Vault Extension changes

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify: Adding examples and suggestions to use the Azure Key Vault virtual machine extension when adding certificates to a VM instance. This method is more secure and we want to point customers to perform the action in this manner instead of manually adding the secret.

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet. N/A
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.: Whenever the next publish cycle goes out is fine.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published. I am pushing updates for REST, PS and CLI each with individual PRs

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@smotwani FYI

@ArcturusZhang , the Go breaking changes appear unrelated to this PR, and are from older versions of the swagger (merged a while ago). I think they can be ignored for this PR.
@mimckitt

@sandido @ArcturusZang do we need anything else in this PR in order to get it merged?


Class generated by make:data-table extends from AbstractType instead of AbstractDataTableType

How to reproduce:
# bin/console make:data-table

 Choose a name for your data table class (e.g. VictoriousPopsicleType):
 > OfferDataTableType

 created: src/DataTable/Type/OfferDataTableType.php
  Success!

 Next: Open your new data table class and add some columns and filters!
#

Generated code:
<?php

declare(strict_types=1);

namespace App\DataTable\Type;

use Kreyu\Bundle\DataTableBundle\DataTableBuilderInterface;
use Kreyu\Bundle\DataTableBundle\Type\AbstractDataTableType;

class OfferDataTableType extends AbstractType
{
    public function buildDataTable(DataTableBuilderInterface $builder, array $options): void
    {
    }
}

As you can see, the class extends from AbstractType instead of AbstractDataTableType.

Thanks! Fixed in v0.9.2 :)


Invalid detections on comments

warning: FILE: D:\other\LifePlay\LifePlay\Content\Modules\vin_Base\Quests\bggw\Scenes\bggw_episodes.lpscene  LINE: 17  WARN: INVALID_OPERAND - Repaired invalid operand: &&
line 17:     If stage == 1 // Ashley & Iris & Eric

you could simply strip everything from a line starting with the // before processing it.

you could simply strip everything from a line starting with the // before processing it.

The detection also is wrong for strings containing operand characters, so you have to ignore them as well first

The detection also is wrong for strings containing operand characters, so you have to ignore them as well first

edge case normally ignore them will take a look

edge case normally ignore them will take a look

also comments are fixed, I just for got to close it

also comments are fixed, I just for got to close it

fixed

fixed


Entries for AI and generative AI, team collaboration

I create the first version of the READMEs for AI and generative AI using the template.
I added some assets in the team collaboration section.

I have read the CLA Document and I hereby sign the CLA


Update Readme

Update readme with project contribution, project goals, and a checklist of suggested features.

I can do this assign this to me?

closed by #10


Portrait Pictures shown streched to fit screen

the preview in browser shows the picture in wrong format.
it is portrait, but it streches it to landscape.
try with this picture:

Funny thing: This happens here on GitHub as well, right? ;-)
The "problem" is, that your image has the "portrait orientation" set as an EXIF metadate:

(sorry, german only, but you can read the marked entry as "Orientation: 6 (90° rotated counter-clockwise)"
And Redmine and other systems can't handle this metadate so they ignore it and present the image "as is", which means in landscape without the rotation.
On the other hand, the Windows image viewer, Mac OS Viewer and alike can handle this metadate and rotate the image properly. So they're making us think that "something is wrong with Redmine, Github, etc." but it isn't.
Normally, when rotating an image in those Desktop programs, the Orientation attribute is reset to 1 upon saving.
So I don't know why this isn't happening in your case. I assume the image is directly from the camera and hasn't been edited yet.
A similar question has been raised on StackOverflow: https://stackoverflow.com/questions/24658365/img-tag-displays-wrong-orientation
I could implement the CSS rule image-orientation: from-image; but CanIUse says it's not widely supported: https://caniuse.com/#search=image-orientation
From my observation, Firefox is doing this anyway, resulting in a stretched view of the image, but with correct orientation:

While Chrome isn't implementing this rule, resulting in an non-rotated view which is what you call "wrong format":

Testing and seeing this, I will add image-orientation: initial; to a further release in order to maintain the landscape view in Firefox to match the behaviour in other browsers and do not stretch and distort the image.
In the end, I'm stuck doing more because the lightbox library "fancybox" does not have any more options for this. I assume it's a bug how they retrieve the image size from source and I will open an issue at their bugtracker.
In the meantime I suggest you rotate and save the image in a desktop application and re-upload on your site. Sorry.

Testing and seeing this, I will add image-orientation: initial; to a further release in order to maintain the landscape view in Firefox to match the behaviour in other browsers and do not stretch and distort the image.

Done for Redmine 3.4 on branch redmine-3.4.
Done for Redmine 4.0 on branch master.
No new version has been released yet. Still looking for ways to read the EXIF meta data and find a workaround for fancybox.

A similar fancybox issue got closed: https://github.com/fancyapps/fancybox/issues/2293
So I guess this won't be implemented/fixed and I can'T do anything here at the moment...


fix: restrict topology domain universe by provisioner

Fixes: https://github.com/aws/karpenter/issues/3239
Description
We were incorrectly expanding our universe of topology domains based on instance type requirements.  These needs to be intersected  with the provisioner requirements to ensure that restricting domains by provisioner works correctly.
How was this change tested?
Unit testing and deployed to EKS.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 77.607%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
5649

Relevant Lines:
7279

💛  - Coveralls

Nice!


Add support for anonymous migrations

Adds support for anonymous migrations as at the moment it just skips them.
I couldn't find any tests in regard to this piece of functionality so was not sure where to add any 🤷

thanks for the PR!


Revert "Generate build environment SBOM"

Manual revert for PR #256

I am sorry @puerco 😭

I can take one for the cause 🫡


alertify

alertify.log("Will stay until clicked", "", 0);
is still closing automatically..

Can you provide a jsfiddle/jsbin example of it not working? From what I can tell based on the examples on the website, it's working fine there, so an example to reproduce the issue would be helpful.
Might be worth noting in version 1 and up that it will close if there are multiple log messages and it gets "pushed" into hiding, that's part of the design.

http://jsfiddle.net/mpqqhrc6/

Thanks. Turns out that the website documentation is outdated, the code example needs to be updated. It should be:
alertify.delay(0).log("Will stay until clicked")

Changes are being pushed to the website now, so closing this. Thanks!

thx.. but after doing the changes based on that.. the alerts can't be dismissed even when they are clicked...
http://jsfiddle.net/mpqqhrc6/1/

Yeah, close on click isn't the default in this version, so:
alertify.closeLogOnClick(true).delay(0).log("Will stay until clicked");

It can be set once, too, if that's the default functionality you're looking for, as in your example:
alertify.closeLogOnClick(true);

$('button').click(function() {
	alertify.delay(0).log("Will stay until clicked");
});


[#414] Add thinking time in test runner to reduce conversation restarted errors

Addresses # 414
Summary
This PRs adds a thinking time in the test runner before sending activities to the bot to give it time to properly save the state, reducing the occurrences of the conversation restarted issue.
Specific changes
Added a delay in the TestRunner.cs of 500ms before sending the activity messages to the bot with the user role.
Additional details
By adding this delay the test scenarios pipelines increase their time from 10-20 minutes to 20-30 minutes.
Because of this, we will consider this change as temporary to allow the tests to pass while we look for better approaches that don't impact as much in the total time.
Testing
To test the change we run a total of 35 test scenarios pipelines, where no occurrence of the conversation restarted issue showed up.
Below are some examples.

Closed in favor of #25


Weather Service Go version on Openshift 4

Deploy the Weather Service Go version on Openshift 4 using a custom S2I entry.

Decided to not incorporate this service into Bee Travels


re-subroutinizing SourceSansPro-Regular.otf yields slightly bigger file

I tried downloading SourceSansPro-Regular.otf and run python -m cffsubr on it.
Comparing the resulting CFF table, I see that the original table is smaller than the one produced after running cffsubr.
I was wondering why this is the case?
Is SourceSansPro-Regular using some different library to do the subroutinization than the one used by tx tool? Or is it passing different options that I am not aware of?
How do you explain the diff?

/cc @khaledhosny who is our only know user so far

Source Sans Pro uses makeotf, so it might be difference from makeotf subroutinizer.

I'm looking into this so I can explain in more detail but basically what @khaledhosny is the root of it: Source Sans Pro was built with makeotf and although the core subroutinization code is more or less identical between makeotfexe and tx, the result of subroutinization can be affected by the order in which the input glyphs are processed as part of subroutinization (not necessarily font glyph order -- it's the order in which the glyphs are analyzed for subroutinization).
On a related note: I built the latest SSP (from master branch) with the current AFDKO/makeotf, then ran the result of that through cffsubr. In that case, the original table is slightly larger than the one produced by cffsubr.

Historical note: I ported tx's faster subroutinizer code to makeotf in PR #882, which went out in AFDKO 3.0.0. Prior to that, makeotf used different subroutinizer code.

As Josh mentioned, the tx and makeotf subroutinizers are nearly identical. If anyone wants to compare them, they are in these two files:
afdko/c/public/lib/source/cffwrite/cffwrite_subr.c
afdko/c/makeotf/makeotf_lib/source/typecomp/subr.c

Thank you for the insights. In cffsubr I am calling tx with the option to keep the order, I thought it would be required to ensure that I can then reinsert the modified cff table back in the sfnt container. But maybe that's not the case and I can let tx find the most optimal order?

You definitely need the +b "preserve glyph order" option for tx, otherwise you'll have problems like cmap mapping Unicodes to the wrong glyph indices.

Yeah, what @cjchapman said. Don't remove +b unless you want a whole new set of problems to deal with (and likely still have size differences from makeotf).
To reiterate: the difference in the subroutinization result seems to be caused by the order in which glyphs are analyzed for subroutinization -- which is not necessarily the font's glyph order (I'm working up a test case to demonstrate/prove/describe this in detail).

Some additional information and data for this:

tx appears to perform the analysis for subroutinization based on the Adobe Standard Encoding order (that is: it looks at those glyphs, if present, first), regardless of the font's glyph order.
makeotfexe probably performs the analysis based on glyph order
Source Sans Pro is not in Adobe Standard Encoding order
When the analysis order is the same, you get the same subroutinization results. Therefore, we can say conclusively that the core makeotfexe subroutinization is the same as tx. It's the analysis order that contributes to the differences you see.

I've attached some test files that support the above (I did not bother trying to prove makeotfexe's behavior exhaustively as I think it can be inferred from the other findings). The file SSP-limited.otf is SourceSansPro built with makeotfexe using a modified GOADB, containing mostly glyphs not in Adobe Standard Encoding. SSP-limited-tx.otf is the result of running tx -cff +S +b on SSP-limited.otf, then stuffing the resulting CFF table into the file. There are still some slight differences in the CFF table, but the subrs and glyph charstrings are identical.
To sum up: the difference in this particular case is because the font's glyph set is not in Adobe Standard Encoding order. makeotfexe (apparently) analyzes for subroutinization based on font glyph order, whereas tx analyzes based on Adobe Standard Encoding order. If you want the same subroutinization results between makeotfexe and tx, you need to have the font's glyphs in Adobe Standard Encoding order.
SSP-limited-test.zip

Thanks for the analysis.

tx appears to perform the analysis for subroutinization based on the Adobe Standard Encoding order...
makeotfexe probably performs the analysis based on glyph order

Any particular reasons why tx and makeotf should differ in this regard?

There are still some slight differences in the CFF table

I noticed those too. In paticular, tx seems to use "ExpertEncoding", whereas makeotf prefers "StandardEncoding". I don't know what that means, but I wonder if that has any relationship with that difference you noticed in subroutinization order.
Another difference is that tx is dropping FamilyBlues and FamilyOtherBlues, apparently in this font these have the same values as the BlueValues and OtherBlues respectively. Maybe tx thinks they are redundant and flushes them away? Is that good/safe?

If FamilyBlues and FamilyOtherBlues have the same values as BlueValues and OtherBlues, it's correct to not include the Family set in the font.
As for the usage of ExpertEncoding, I find that strange. Not sure what consequences it may have.

Any particular reasons why tx and makeotf should differ in this regard?

I don't know for sure; this all happened way before my time at Adobe and I think the people who made those decisions are not around anymore. My guess would be because Adobe Standard Encoding provided a consistent starting point for compressing the character/glyph sets that were popular at the time this scheme was developed (in tx, anyway).

As for the usage of ExpertEncoding, I find that strange. Not sure what consequences it may have.

I suspect that's an anomaly from the somewhat unusual set that I chose for this experiment. Probably tx uses some heuristic like % of charset present to set "StandardEncoding" and this font has a very small percentage of that. I would not expect to see this change in more normal cases. And I'm not sure it has any real consequences in an OpenType font anyway: the font's 'cmap' subtables will ultimately determine the character set(s) and encoding(s).
Bringing all of this back around to cffsubr: to really solve this well I think we need to build only the subroutinizer (from code that @cjchapman mentions here) into either a standalone executable or C Extension so we truly isolate subroutinization from other parts of the CFF/CFF2 tables.
A shorter-term workaround might be to see if we can extract only the relevant data from the tx-subroutinized CFF table (local & global subrs, charstrings, maybe some other bits) and stuff those back into the pre-subroutinized CFF, rather than take the entire converted table which might have other undesired diffs.


Add FEATURE_SERIALIZATION

Added FEATURE_SERIALIZATION.
I did not wrap [Serializable] attribute at this time because it builds on netstandard, when using Serialization.Formatters package.

System.Runtime.Serialization.Formatters cool. Looks good.


[repl] New commands for cbom

Add new commands for various component types
Add a command to print the provides tree

New commands added.
.provides
.cryptos


Typescript compiler ignores wrong parameter type in callback

Code
// A *self-contained* demonstration of the problem follows...
class A<T> {
    methodA(callback: (value : T) => void){
    }
}
class B<T>{}

var a1 = new A<String>();
a1.methodA((value : number) => { });

var a2 = new A<B<String>>();
a2.methodA((value : number) => { });

Expected behavior:
Both invocations of methodA have compiler warning.
Actual behavior:
Only first invocation of methodA have compiler  warning.

Unused generic type parameters have no impact on the type of the object. See https://github.com/Microsoft/TypeScript/wiki/FAQ#why-is-astring-assignable-to-anumber-for-interface-at--

@RyanCavanaugh Generic type parameter is not unused. It's used in callback: callback: (value : T) => void
Also, I'm not trying to compare 2 objects or assign one object to other, I just call a method. Moreover, validation works ok when for the first statement!

Generic type parameter is not unused. It's used in callback: callback: (value : T) => void

It is not used in B<T>. that is why it works for the first and not the second.
change the declaration of B to be:
class B<T>{
    _a: T
}

and you should see the expected error.


[Question]: How to use it with websockets response

What happened?
I'm not able to get the response when using WebSockets. I'm using h2oGPT. The Gratio Client in Python works fine, but here I'm struggling to make it work.
In PHP when calling the code:
$result = $client->predict([['instruction_nochat' => 'Who are you?', 'stream_output' => 'True']], apiName: 'submit_nochat_api');

I get the response like this:
object(SergiX44\Gradio\DTO\Output)#239 (4) {
  ["is_generating"]=>
  bool(false)
  ["duration"]=>
  float(0)
  ["average_duration"]=>
  float(0)
  ["data"]=>
  array(0) {
  }
}

When I tried to debug what was actually returned from websockets I got this:
string(19) "{"msg":"send_hash"}"

string(190) "{"msg":"estimation","rank":0,"queue_size":1,"avg_event_process_time":3.7834130906127212,"avg_event_concurrent_process_time":0.05911582954082377,"rank_eta":3.7834130906127212,"queue_eta":0.0}"

string(19) "{"msg":"send_data"}"

string(24) "{"msg":"process_starts"}"

string(365) "{"msg":"process_completed","output":{"error":"An event handler (fn) didn't receive enough input values (needed: 5, got: 1).\nCheck if the event handler calls a Javascript function, and make sure its return value is correct.\nWanted inputs:\n [state, state, state, state, textbox]\nReceived inputs:\n [{'instruction_nochat': 'Who are you?'}]"},"success":false}"

I'm not sure if this is my fault with configuration, or specific h2oGPT implementation which is not compatible with this library in this case. I would really appreciate the hint, to make it work.
How to reproduce the bug
require __DIR__ . '/vendor/autoload.php';

use SergiX44\Gradio\Client;

$client = new Client('https://gpt.h2o.ai');
//$client = new Client('http://localhost:7860');
$result = $client->predict([['instruction_nochat' => 'Who are you?', 'stream_output' => 'True']], apiName: 'submit_nochat_api');

echo '<pre>';
var_dump($result);
echo '</pre>';

Package Version
0.0.2
PHP Version
8.2.11
Which operating systems does with happen with?
macOS
Notes
No response

Do you have a list of the endpoints supported by that model?
It seems that you not passing enough params 🤔
The websocket mode should be transparent to the end user, if supported by the model it will be used instead of http mode.
I'll take a look and try to reproduce the issue

Do you have a list of the endpoints supported by that model?

I only have this: https://gpt.h2o.ai/config

It seems that you not passing enough params 🤔

Yes, I know that the error indicates that there are too few params (the config above also states there should be 5 of them), although the same params as an input in the Python client work fine.
Python example:
from gradio_client import Client
import ast
import time

HOST = 'http://localhost:7860'
client = Client(HOST)
api_name = '/submit_nochat_api'
prompt = "Who are you?"
kwargs = dict(instruction_nochat=prompt, stream_output=True)

job = client.submit(str(dict(kwargs)), api_name=api_name)

text_old = ''
while not job.done():
    outputs_list = job.communicator.job.outputs
    if outputs_list:
        res = job.communicator.job.outputs[-1]
        res_dict = ast.literal_eval(res)
        text = res_dict['response']
        new_text = text[len(text_old):]
        if new_text:
            print(new_text, end='', flush=True)
            text_old = text
        time.sleep(0.01)
# handle case if never got streaming response and already done
res_final = job.outputs()
if len(res_final) > 0:
    res = res_final[-1]
    res_dict = ast.literal_eval(res)
    text = res_dict['response']
    new_text = text[len(text_old):]
    print(new_text)

To be fair, I'm very new to using AI models, so it might be that this is my fault... anyway thank you for looking into this. Much appreciate.


Baseline WARNINGS/ERROR in master

Since version 4.4.0 is available from the Adobe Maven Repo  baselining in latest master fails with one error and several warnings:
[ERROR] com.adobe.acs.commons.mcp.model: Version increase required; detected 4.2.0, suggested 4.3.0
[WARNING] com.adobe.acs.commons.cloudconfig: Excessive version increase; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.cloudconfig: Version has been increased but analysis detected no changes; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.filefetch: Excessive version increase; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.filefetch: Version has been increased but analysis detected no changes; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.marketo: Excessive version increase; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.marketo: Version has been increased but analysis detected no changes; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.marketo.client: Excessive version increase; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.marketo.client: Version has been increased but analysis detected no changes; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.mcp.form.workflow: Excessive version increase; detected 4.4.1, suggested 4.4.0
[WARNING] com.adobe.acs.commons.mcp.form.workflow: Version has been increased but analysis detected no changes; detected 4.4.1, suggested 4.4.0
[INFO] Baseline analysis complete, 1 error(s), 10 warning(s)

The error is a regression of https://github.com/Adobe-Consulting-Services/acs-aem-commons/commit/ee1674c994d4eb9b61e8596ededf82f87299bddd
The warnings are due to the fact that those packages are not versioned explicitly and therefore get the version of the bundle. Every exported package should get a dedicated package version!

@paul-bjorkstrand  fixed a number of these warnings with 0dc233a36271b9308343374ce48cebd89605161f

I raised #2179 to address the remaining baseline warning.


Provisioning process for existing device on field?

This document very well described on boarding process of device, but how do I on board a device which is already in field. How does I onboard these devices in to DPS? do I need to install some service manually on each devices?

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

ID: f21f48a7-a140-b21f-e699-df8e9fb000a9
Version Independent ID: 32965fad-9f8b-81b2-ac3d-d532b517182b
Content: Overview of Azure IoT Hub Device Provisioning Service
Content Source: articles/iot-dps/about-iot-dps.md
Service: iot-dps
GitHub Login: @nberdy
Microsoft Alias: nberdy

@isharehappycode thank you for your feedback! There are different approaches you can take to provision a device that is already in the field. You can have a look at Azure IoT Reference Architecture Guide for insights.
If you need more advisory please file a support request @ https://aka.ms/azsupt and reference this issue in the description. If you do not have access to a support plan, please reach out<EMAIL_ADDRESS>with a link to this Doc/Issue as well as your subscription ID and we can help get the support ticket opened for this issue.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Cache location suddenly empty

I really love the aerials, great work.
Since the few months that I use the screensaver, I recognized a dozen times that the cache location (user/library/caches/aerial) is empty and I have to download the videos again. I use a macbook pro (OS 10.13.4) and since I'm not always connected to the internet or the connection isn't always good enough to cache the videos as they play, I like to download them.
When I see a black screen instead of the screensaver, I have to download the videos again. Unfortunately, I couldn't see any pattern of how this happens. Any help?
Thanks in advance.

My guess is that your drive is running out of space. Don't let it run out of space.
Can't find documentation for this right now but I'm pretty sure if your Mac is runing out of space the operating system will automatically delete files from the ~/Library/Caches folder to make room.

There's a wee note from Apple talking about this in iOS. I'm sure macOS behaves the same way.
Note that the system may delete the Caches/ directory to free up disk space, so your app must be able to re-create or download these files as needed.
https://developer.apple.com/library/archive/documentation/FileManagement/Conceptual/FileSystemProgrammingGuide/FileSystemOverview/FileSystemOverview.html

I got a 500gb disk and still 250gb available, so not exactly  "running out".
But thanks for the hint, I change the cache path to something different and will see.
thx & best
D.

The only other thing that comes to mind is that there are tons of applications that do a bunch of housekeeping tasks to the operating system. For example OnyX or CleanMyMac. One of the things they usually do is delete things from the Cache folder.


Start in Accordion mode

Hi.
Your horizontalTab is great!
But I want know if is possible that "horizontalTab" start in accordion mode in desktop screen?
thanks for your answer.

Hi there,
I'm not sure I understand your wish, you want to have an accordion mode on a desktop width? Then how would you use the normal tabs?

Hello Velsargo,
All you have to do is comment out 2 media queries and a 'display: none;' in the CSS shown here...
.r-tabs {      .r-tabs-nav {         margin: 0;         padding: 0;         /* Accordion responsive breakpoint */         // @media only screen and (max-width: 768px) {             display: none;         // }     }     .r-tabs-tab {         display: inline-block;         margin: 0;         list-style: none;     }     .r-tabs-panel {         padding: 15px;         display: none;     }     .r-tabs-panel.r-tabs-state-active {         display: block;     }     .r-tabs-accordion-title {         // display: none;         // @media only screen and (max-width: 768px) {             display: block;         // }     } }

I'm closing the issue since there has been no activity for a while. Feel free to reopen if you feel this is still unresolved.