fix: missing client validation call

Summary
What's changed?

Validate the client

Why do we need this?

Create more meaningful error messages

How have you tested it?

Tested manually

Checklist

[x] I have reviewed this myself
[ ] There is a unit test covering every change in this PR
[ ] I have updated the relevant documentation

Details

Asking for a rereview because I had to modify a lot of stuff to change the tests :pray:
I'm not exactly sure why my PR was what broke it and why it was complaining about the missing config file as opposed to the client being wrong, but clearly were loading a scaleway_client regardless of if we were deployed or in docker compose, so I refactored to avoid it.


Colours depending on value

One-line summary [问题简述]
Any way we can assign colours to the charts series (i.e bars) depending on the value of the data?
Version & Environment [版本及环境]

ECharts version [ECharts 版本]: 3.6.2
Browser version [浏览器类型和版本]: Edge, Chrome
OS Version [操作系统类型和版本]: Windows 10

Expected behaviour [期望结果]
Example:

In this case a gradient of colours but also considering something predefined like what happens in the gauge:
series: [{
			  axisLine: {
				show: true,
				lineStyle: {
				  color: [
					[0.25, 'red'],
					[0.75, 'orange'],
					[1, 'green']
				  ],

ECharts option [ECharts配置项]

option = {

}

Other comments [其他信息]

@LMJS1974
Hi
Did you got a solution for this issue?


TASK 25736691: Publish package should use same version as nuget package

Uses the nbgv value as the version so it matches the nuget package version in the statically compiled provider for Bicep

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

Comparison is base (25b9a07) 90.00% compared to head (58311df) 90.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2004   +/-   ##
=======================================
  Coverage   90.00%   90.00%
=======================================
  Files           2        2
  Lines          20       20
  Branches        6        6
=======================================
  Hits           18       18
  Misses          1        1
  Partials        1        1

Flag
Coverage Δ

dotnet
90.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


feat: recognise config.options.map (options.sourceMap)

Currently loader options can be overridden by config options loaded by postcss-load-config.
So we should use the merged options.map here instead of sourceMap.

@michael-ciniawsky What do your think about this? Should we prioritize loader options over config?

@evilebottnawi
This is not prioritizing loader options over config. Loader options are already extended by config here:
https://github.com/postcss/postcss-loader/blob/271ab9a86799a97e3b230815e8365728b4072615/lib/index.js#L101-L109
This PR is just making sure that we can enable source map from config.
Although I think loader options should have higher priority, considering changing this might include a big breaking change, I'm not gonna suggest you to change current logic.

The reason I avoided this is that in a normal setup you need to enable sourceMap in various places (style-loader, css-loader, sass-loader etc) to work correctly and all possible PostCSS Sourcemap Options are not 1:1 compatible with how webpack handles sourcemaps internally. config.options.map would need to have a safeguard at least to ensure it is either { inline: false, annotation: false } || { inline: true , annotation: false }. The current approach in this PR can lead to subtile bugs. I'm not really in favour of this change tbh.

This also needs a (separate) test

Needs tests and a safeguard to ensure config.options.map is consistent with the two sourcemap options currently possible for webpack setups ( true|'inline')

I'm not sure if I understand this correctly:

What I'm confusing about here is postcss-loader prioritize some options from config files but ignore some other options from them at all. I have to configure in both places and be aware of the details under the hood if I want to put things into config files.

What I'm confusing about here is postcss-loader prioritize some options from config files but ignore some other options from them at all.

options.from && options.to is handle by webpack as it is resposible for File I/O. For using the bare PostCSS Node API + postcss-load-config options.from && options.to can be useful, albeit the usecase is even limited there. For postcss-loader or gulp-postcss these two options don't make sense at all so they are ignored

I have to configure in both places and be aware of the details under the hood if I want to put things into config files.

Yeah agreed it is in one way or the other not optimal, but the standard way for sourceMap in webpack.config.js is  { sourceMap: {Boolean} } and webpack sets requirements of how a sourcemap file needs to look like to work correctly, so not all supported options.map values are valid for postcss-loader
webpack.config.js (No options.map possible [Current])
[
   { loader: 'style-loader', options: { sourceMap: true } },
   { loader: 'css-loadder', options: { sourceMap: true } },
   { loader: 'postcss-loader', options: { sourceMap: true } },
   { loader: 'sass-loader', options: { sourceMap: true } }
]

webpack.config.js (with options.map possible [Proposed])
[
   { loader: 'style-loader', options: { sourceMap: true } },
   { loader: 'css-loadder', options: { sourceMap: true } },
   'postcss-loader', // <=  "Inconsistent" for options.sourceMap (as the point of view)
   { loader: 'sass-loader', options: { sourceMap: true } }
]

Since { sourceMap: .. } needs to be set on all loaders within a rule, the optioniated decision was to only allow sourceMap to be set via webpack.config.js for consistency reasons (nits), since both approaches are not 💯 sufficient anyways.

Thanks for the explanation. So rather than changing current behavior, I think we'd better clarify this in the docs to prevent confusion like "I've configured source maps in my config file, why do postcss-loader still giving warnings?".

I gave this some though and this two paths are possible
L99+
Warning
if (config.options.map && sourceMap) delete config.options.map

if (config.options.map && !sourceMap) {
  this.emitWarning(`\n\n ⚠  PostCSS Loader\n\nSetting source map via 'postcss.config.js' isn't supported due to inconsistent behaviour between source map options, please use options.sourceMap in 'webpack.config.js' instead.\nThe loader will not generate a source map with the current configuration.\nSee https://github.com/postcss/postcss-loader#sourcemap for more information.\n\n`)

  delete config.options.map
}

Fallback (no inline)
if (config.options.map && sourceMap) delete config.options.map

 if (config.options.map && !sourceMap)  {
  delete config.options.map
  sourceMap = true // !  Needs a change from const to let for sourceMap (L50)
}

Or a mixture of both, totally open for suggestions here cc @evilebottnawi 😛

@michael-ciniawsky
I think updating warning should be enough for my case. The fallback solution my add extra complication here IMO. I'm closing this PR and thanks for your explanation. 👍


PHOENIX-5238 Provide an option to pass hints with PhoenixRDD and Data…

…source v2

used [setCacheBlocks|https://hbase.apache.org/apidocs/org/apache/hadoop/hbase/client/Scan.html#setCacheBlocks-boolean-] to disable the block cache.
Current implementation deprecated the PhoenixRDD.scala file, so I modified PhoenixDataSource.java with an additional option.


Easier Installation Process

We need to improve the installation process of Melissa, so that non-developers and beginners can also easily run Melissa on their machines.

In order to invite more developers as well as non-technical users to try out Melissa, we should create an easier installation process. Thoughts? @neilnelson

The key issue I had when installing Melissa is that it errored at pywapi for which you provided a link. I downloaded pywapi from that link, did a manual install and then reran the Melissa install procedure that completed the install. I have not looked at the install procedure but automating the download of that particular link and its install appears to be the key factor for the current installation procedure.
When you say easier installation procedure you likely have a target in mind you could direct us with. In general, Melissa depends on quite a bit of software as given in the requirements.txt file, a file used to get software in the installation process. Since Melissa is user-computer oriented as against server-interface oriented, as is Wit and Api.ai, the many current and likely future additional software components are inherently required that under the current procedure will compound the installation issue.
The Ubuntu and RStudio method to provide for an apparently unlimited number of software/package additions is for the user to manually install particular packages when needed. In saying non-technical users my thinking is that we might avoid a manually initiated install (not necessarily a complete manual install) by installing a package at the moment it was needed. And/or perhaps by having a screen on which the user could check software items in a list at the end of which an install button could be used to complete those installs. Except for having to build a screen, I expect the current install procedure could be fairly easily adapted to either of these cases.
A wrinkle in the prior install of only those packages that are needed for the particular user is that Melissa then has to know what packages are installed and avoid modules using the not-yet-installed packages (the second case using check list on a screen) or know when a package is not installed and start a sequence to install the package when a user makes a request using that package. Another note here would be that there will likely be software available to Windows or Linux and not both that can be accounted for in either of these two sequences.
Of course we would like to maintain the current design where we do not have to provide a server that Melissa connects to in order for Melissa to function, as is done with Wit and Api.ai. However, and I only note this as a solution possibility in that we can also reduce some of the installation issues by offloading activity to a server. I doubt anyone at the moment would volunteer for doing that but is something to keep in mind.

When you say easier installation procedure you likely have a target in mind you could direct us with.

I was thinking about a Shell Script that can automate the git cloning, installation of requirements.txt, weather API, building and compilation of PortAudio and PyAudio as well as installation of the requisite music players and TTS engines. This would serve as an interim solution for Linux/OS X users for now.

@neilnelson Can you confirm if this works for you in Ubuntu on a fresh installation?

Solved by https://github.com/Melissa-AI/Melissa-Core/commit/a333cae4eadc7c43cebf6f9ddb62fa35ce8e8c5c.

Dumped previous approach and used it in Travis-CI integration, looking for a better solution.

This is not a priority right now and can be reopened at a later stage.


Module not imported for meta-data references.

From https://github.com/Microsoft/TypeScript/issues/12888#issuecomment-272100943
after upgrading to TypeScript 2.1.5.
the issue regressed in different place.
when 'strictNullCheck' option is false the following typescript emitt invalid javascript code.
file: auxiliry.ts
export class SomeClass {
    field: string;
}

file: main.ts
import { SomeClass } from './auxiliry';

function annotation(): PropertyDecorator {
    return (target: any): void => { };
}

export class ClassA {
    @annotation() array: SomeClass | null;
}

Emitted code:
"use strict";
var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
    return c > 3 && r && Object.defineProperty(target, key, r), r;
};
var __metadata = (this && this.__metadata) || function (k, v) {
    if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
};
function annotation() {
    return function (target) { };
}
var ClassA = (function () {
    function ClassA() {
    }
    return ClassA;
}());
__decorate([
    annotation(),
    __metadata("design:type", auxiliry_1.SomeClass) // <-- auxiliry_1 is not emitted as require statement
], ClassA.prototype, "array", void 0);
exports.ClassA = ClassA;

I'm having the same problem with strictNullChecks turned on.
@Input() employee: Employee | undefined; // This crashes at runtime
@Input() employee?: Employee; // Workaround

@Input() set employee(value: Employee | undefined) { // This crashes at runtime
    ...
}
@Input() set employee(value?: Employee) { // Compiler error
    ...
}

I can work around the problem on regular fields (line 1 & 2), but the workaround isn't applicable for properties as they don't allow optional parameters.
Will there be a bugfix for this in 2.2, or do I have to wait for 2.3? I'm currently stuck at 2.1 because of this problem.

Just wanted to note we have more than ten instances of this bug coming up within Google in various user code, which means we are needing to teach our users to sprinkle
const workaround = Import;

in their code to make some Import symbol not get dropped.

When will this be released?


Fix minor typo in usage.adoc

acem to acme.

@ilopmar This PR is a minor typo fix and seems like it could easily be merged.

Thanks!


[Refactor] Fix style in example

Replaces #125616
I just fixed "IMAGES" to "images" in examples/image_list.
Currently, "PREFER uses the lower case of the constant name ".
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

@goderbauer PTAL

LGTM

thank you. I think we can marge this.

It still needs a test exception (see the bot message above for how to get one) and a second team member needs to LGTM it.

test-exempt: code refactor with no semantic change

Thanks for the contribution!


adal.js has been deprecated by msal.js

adal.js has been deprecated by msal.js
Please update the example to use msal.js.
Thank you.

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

ID: 9a51d2ac-2b1c-d708-dd48-3b1ec982f61c
Version Independent ID: bc3282d9-d484-23d6-6d68-d374d14d2935
Content: Walkthrough: Registering and configuring a SimpleSPA application with adal.js (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/walkthrough-registering-configuring-simplespa-application-adal-js.md
Service: powerapps
GitHub Login: @paulliew
Microsoft Alias: nabuthuk

@Vivihung Yes, we are updating our samples. Thanks for the feedback.


refactor(Cluster): modify startup method

Codecov Report

Merging #46 (c3154c8) into main (0719f85) will increase coverage by 0.08%.
The diff coverage is 81.81%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
+ Coverage   79.59%   79.67%   +0.08%
==========================================
  Files          12       13       +1
  Lines        1176     1181       +5
==========================================
+ Hits          936      941       +5
  Misses        164      164
  Partials       76       76

Files
Coverage Δ

cluster.go
87.80% <100.00%> (ø)

services/cluster_rpc.go
84.21% <100.00%> (ø)

services/rpc.go
90.00% <ø> (-0.63%)
:arrow_down:

services/cluster.go
77.77% <77.77%> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Data model improvements

Change log:

Renames fiftyone.core._sample._Sample to fiftyone.core.document.Document, to reflect the fact that this class is a base class for all user-facing objects backed by documents in the DB
Adds a Document.merge() method for merging fields of arbitrary fiftyone.core.document.Document instances together
Updates Dataset.merge_samples() to properly merge the frame-by-frame contents of video samples
Fixes a bug where sample.copy() would not create a copy of the frames of a video sample
Adds a variety of usability improvements to sample.frames for video samples, including:

print(sample.frames) will print the actual frame contents (repr(sample.frames) still just shows # frames, for readability)
frame_number in sample.frames # True/False now works
Adds a sample.frames.update(new_frames) method to set entire Frames
Adds a sample.frames.merge(other_frames) method, which will merge fields of Frames

Raises informative errors for frames-related operations that are not yet supported for video samples, such as clearing, setting the entire object, etc
Removes various bits of duplicate/unnecessary code throughout

I'll note that all of this functionality really needs thorough unit tests. The video release introduced quite a few bugs relating to video samples and the Dataset/Sample interfaces, but no unit tests existed to really enforce the details of desired behavior around copying, merging, getting/settting/iterating fields, etc...
Example of merging video samples:

import fiftyone as fo

sample1 = fo.Sample(filepath="video.mp4")
sample1.frames[1] = fo.Frame(
    gt=fo.Classification(label="cat"),
    pred=fo.Classification(label="dog"),
)

sample2 = fo.Sample(filepath="video.mp4")
sample2.frames[1] = fo.Frame(
    gt=fo.Classification(label="kitten"),
    other_pred=fo.Classification(label="doggo"),
)

dataset = fo.Dataset()
dataset.add_sample(sample1)

# Merge the samples, which will merge the frame-by-frame labels, NOT
# overwriting any existing values
dataset.merge_samples([sample2])
print(dataset.first())
print(dataset.first().frames)

# Merge again, but this time overwrite existing values
dataset.merge_samples([sample2], overwrite=True)
print(dataset.first())
print(dataset.first().frames)

@lethosor any idea what's up with the failing Windows test here? Looks unrelated to the changes in this PR, so I'm going to go ahead and merge

Yeah, that's unrelated. The steps starting with "Post" at the end aren't steps that we control (I would like to make them not required to pass, but I haven't found anywhere to do that).
In the future, if you're not sure, you can re-run the tests from the log page:
https://github.com/voxel51/fiftyone/runs/1240425126


Improve request caching performance

This PR introduces the following changes:

Request cache is always in-memory
Replace request cache implementation to https://github.com/hashicorp/golang-lru which is a threadsafe fixed-size LRU cache
Cache response data without serialization. #788 shows that byte array decoding has a huge impact on CPU time.

Nice PR. Other than some commented out code in tests, my only suggestion is that you put the LRU cachebehind an interface. There's been changes over time towards caching in Hoverfly and maintaining an interface ensures we can do that with some speed.
It looks like this could be a new cache interface. This could the existing cache interface for the purposes it currently serves, which way may also need to be reviewed at some point down the line.

Thanks @benjih I have created a new cache interface as you suggested.


Trouble accessing xorg.freedesktop.org (was Crouton script can't handle a 302?)

(Sorry about all these posts.  I'm a newbie to crouton).
I've run into a problem using ...
sudo sh ~/Downloads/crouton -t gnome,chrome,xiwi,keyboard -r trusty -n trusty

I got this error...
Download Xorg dummy driver...
--2015-03-28 16:12:30--  http://xorg.freedesktop.org/releases/individual/driver//xf86-video-dummy-0.3.7.tar.gz
Resolving xorg.freedesktop.org (xorg.freedesktop.org)... failed: Name or service not known.
wget: unable to resolve host address 'xorg.freedesktop.org'
Failed to complete chroot setup.

I get this this when I do curl xorg.freedesktop.org in the crosh shell .
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>302 Found</title>

Found

The document has moved here.

<address>Apache/2.2.22 (Debian) Server at xorg.freedesktop.org Port 80</address>

I assume some script can't handle a 302?

when I did wget http://xorg.freedesktop.org/releases/individual/driver//xf86-video-dummy-0.3.7.tar.gz in my chros shelll everything worked fine and I got the small file.  So it has nothing to do with a 302.

The link works so maybe it was a glitch, did you try it again?
On Sat, Mar 28, 2015 at 7:28 PM, mark-hahn<EMAIL_ADDRESS>wrote:

(Sorry about all these posts. I'm a newbie to crouton).
I've run into a problem using ...
sudo sh ~/Downloads/crouton -t gnome,chrome,xiwi,keyboard -r trusty -n trusty
I got this error...
Download Xorg dummy driver...
--2015-03-28 16:12:30--  http://xorg.freedesktop.org/releases/individual/driver//xf86-video-dummy-0.3.7.tar.gz
Resolving xorg.freedesktop.org (xorg.freedesktop.org)... failed: Name or service not known.
wget: unable to resolve host address 'xorg.freedesktop.org'
Failed to complete chroot setup.
I get this this when I do curl xorg.freedesktop.org in the crosh shell .

302 Found

Found
The document has moved here.

Apache/2.2.22 (Debian) Server at xorg.freedesktop.org Port 80

I assume some script can't handle a 302?
—
Reply to this email directly or view it on GitHub
https://github.com/dnschneid/crouton/issues/1555.

--
DennyL@GMail


rpc deps are pinned to old versions

The deps of rpc are pinned to older versions. rpc is in use by several other projects; the older deps mean that they are selecting older package versions when running pub get / pub upgrade, and tihs increases the risk of version lock when one of the other transitive dependencies requires a version newer then one that rpc supports.

Upgrade version could be cool but also migrate unittest to test an rewrite some test in order to use pub run test

We've migraded from package:unittest to package:test and widened several constraints.


Testing model on other Datasets

@a1600012888
Thanks for sharing your work.
Is it possible to test the model only on cross-view pedestrian tracking datasets such as WILDRACK or MvMHAT?
Thanks in advance.

Hi, I think you can definitely test it.
But generally monoculer 3D models is hard to generlize across camera parameters, especiallys for cameras with different focal length.

Hello, thanks for sharing your work.
Can the code run through nuscenes-mini?


New Feedback design

📒 Description
This PR introduces the new design for Feedback screen.
⚠ Warning
This PR is branched off from #2791, so it's critical to merge the parent firstly.
🕶 Types of changes
New feature (non-breaking change which adds functionality)
🤯 List of changes

Implement new design for Feedback
Refactor the original logics

👫 Relationships
Closes #2790
🔎 Review hints

Able to submit the feedback
Follow the design
Feedback validator works well

Fixed some alignment issue but other than that all good!


request.file method

Sails version: 0.12.1
Node version: 5.7.1
NPM version: 3.6.0
Operating system: OS X 10.10

https://github.com/balderdashy/sails/blob/master/lib/router/req.js#L125
1) `res` is undefined
~~2) why `res.send(500, 'Streaming file uploads via req.file() are only available over HTTP with Skipper.')` ? Where skipper? i mean `console.log(request.file + "")` prints `function(){res.send(500...`~~ i do `if (request.file)` in my code, now `request.file` is always exist.

@dangreen
Need more description

@Nishchit14 in request.file method res variable is undefined.
https://github.com/balderdashy/sails/blob/master/lib/router/req.js#L125

https://github.com/balderdashy/sails/blob/master/lib/router/req.js#L125

@dangreen thanks for the report!  We'll take care of it in the next patch release.  We added that stub method to help make it clearer that req.file() is specific to the default setup and skipper when working on the body parser, but clearly forgot to put res in place.


Windows 10 and Office 365 deployment lab kit missing HYD-APP1

The most recent package of Windows 10 and Office 365 deployment lab kit is missing HYD-APP1 vm.  Is this on purpose?   Can you please tell us to which article your feedback applies? This information will help us route the issue to the correct writer/team.
Thanks!

@denisebmsft  thanks for the prompt response.  The windows 10 and office 365 deployment tool kit "description" states that the lab kit has Microsoft BitLocker Administration and Monitoring which according to the lab guide is contained in HYD-APP1  (https://www.microsoft.com/en-us/evalcenter/evaluate-lab-kit#evaluation_4723) but the recent upload of the lab kit has no such VM.  Hope this answers your question.

@greg-lindsay Please help us in resolving this issue. Thanks

@greg-lindsay Gentle reminder

I'm having the same issue and would like to know how to resolve it. This VM is necessary to setup the lab environment

@Teddy24B I think Microsoft is looking into it.  Hopefully, we will receive an update soon that the package has been updated. @yogkumgit did send a friendly reminder.

@greg-lindsay Please review, comment and resolve this issue. Thanks.

Is this still an issue? Because both of the last two iterations of the lab Win_10_21H2_lab_guides and Microsoft365DeviceLabKit do not generate that Bitlocker VM

Definitely an issue for the MECM course I am trying to follow using this lab. The first real hands-on practical lab is to configure the app server.

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Update noble

:recycle: Current situation
https://github.com/abandonware/noble/issues/303
:bulb: Proposed solution
https://github.com/abandonware/noble/pull/304

I currently have this version in the beta branch


Tests fix imodel react hooks

Some of the tests for useFeatureOverrides in the package imodel-react-hooks were skipped during a previous upgrade. These changes fix and re-enable them.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Fixing exists tests that were commented out is one thing, but I don't think I have the bandwidth or the expertise to add a new test that does something like that.
FWIW I gave it a try, but its an internal variable that disappears when the component unmounts; something clever will have to be done to get access to it without exposing it to consumers of the package.


Reopen all scenes that were open last run

Operating system or device:
Issue description (what happened, and what was expected):
When opening a project it would be nice if the editor re-opened all the scenes that were open previously.  It only opens the $main_scene.
Steps to reproduce:
Link to minimal example project (optional but very welcome):

@akien-mga This request is included in #3333, but IMO it's better as a separated issue.

#2385

Closing it then, as it's a duplicate of #2385


Recovery point restriction

According the to FAQ there is actually a restriction of 9999 recovery points.  https://docs.microsoft.com/en-us/azure/backup/backup-azure-backup-faq

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

ID: 444d32c6-9a0f-d6b3-c078-0abdbc415de1
Version Independent ID: a931023d-69c0-e6dd-369e-211670a238f3
Content: Use Azure Backup to replace your tape infrastructure
Content Source: articles/backup/backup-azure-backup-cloud-as-tape.md
Service: backup
GitHub Login: @trinadhk
Microsoft Alias: saurse

Thanks for the feedback and apologies for the delay. I've updated the article. The change will show up in the next few days.
#please-close


[Enhancement] Add Warning message when submodule has pending changes or commits which arent pushed to remote when trying to push top module

Could you add a hint window, when someone is trying to push a repository which includes submodules which are in a dirty state or the commit of the submodule is not yet on a remote branch?
The hint popup window should have the ability to ignore that there are pending changes or/and a button to jump to the submodule which is affected
Like:

Can someone give me a script (or git hooks) for this feature?

https://git-scm.com/book/en/v2/Git-Tools-Submodules#:~:text=Those changes will only exist,or “on-demand”.
https://stackoverflow.com/questions/29924360/how-to-list-git-submodules-that-have-not-been-pushed-to-origin

an option has been added to enable --recurse-submodules=check on push.


fix: add .gitignore

Simple add, wanted to make sure we dont accidentally push cache/residue files related to python.

Closing as superseded by #31


Fix installing to system directories on linux

On Linux, when you install to /usr/ or /usr/local/, the app doesn't work because it doesn't have permissions to create a props.json file in those directories.
This fixes the issue by instead storing the user app state in the standard directory on Linux. If the props.json already exists in the app install directory, then it will continue to use that one so as to not affect existing installations.
For reference, here is the specification for storing program files on Linux: https://specifications.freedesktop.org/basedir-spec/latest/#variables.

The $XDG_STATE_HOME contains state data that should persist between (application) restarts, but that is not important or portable enough to the user that it should be stored in $XDG_DATA_HOME. It may contain:

actions history (logs, history, recently used files, …)
current state of the application that can be reused on a restart (view, layout, open files, undo history, …)

Closes #102

Thank you!

Thanks!


Extending Laravel 11 EventServiceProvider

Hi,
i used the Package in Laravel 8 and updated it now to Laravel 11, my Package is currently still on Version 8.2, but i think the Problem is more related to the Documentation here: https://laravelmodules.com/docs/v11/registering-module-events#creating-an-eventserviceprovider
In Laravel 11 the EventServiceProvider got a new Function called configureEmailVerification, this function is automatically called which leads to multiple request for the sendEmailVerificationNotification Event, which means for every Module that has the EventServiceProvider extended another email is being generated. For example i had 3 Modules active which all had events registered, thus i received 4 emails.
A simple fix is to create an abstract class and then let the EventServiceProvider of these Modules extend from this class instead of extending the Core Class. Then add an empty Function into it. The Code for this abstract class can be found here:
https://github.com/laravel/framework/issues/50783#issuecomment-2072458671
I think it would make sense to either add this info to the documentation, or even better, add a default EventServiceProvider when creating a Module, which already has this fix in it.

thanks I've added a make:event-provider command and added this to module creation.
the events service provider contains the configureEmailVerification method to override the default.


Inverse default hoogfile creation behavior

At the moment the captain tries to create a new hoogfile each time the user does hoog install.
This forces the user to select "n" each time - if you want to have 4 hoogs in your repository, you have to type "n" 4 times.
From my point of view this is annoying and thus I suggest to change the behavior.
Only create a new hoogfile if the install command gets called with a --create (or something) option and otherwise only create the hook within .git

It would suggest kind of strange behaviour - user has to install the captain and additional has to say let's create the hoogfile for me. I would suggest the other way around to have a --skip-hoogfile flag or an update command.

Fine with that.


eval.py generates plain black images

Hello, thank you for sharing your great work! I have a question about eval.py.
I used vid4 test dataset calendar images for evaluation. I found that eval.py generates plain black images with x4 sizes.
I tried to save images before applying converting to uint8  (eval.py last line), and it somehow generates calendar images but with wrong colors.
May I ask how could I fix this?
Thank you.

Hi YoungJoong,
Thanks for your interest. I just did a test using video frames from the calendar set. It works well. I have uploaded the test images into the repo in the data/Vid4 folder (PLEASE change test data path accordingly when you do testing) and the results in the res folder. Could you please try the uploaded frames? I generated testing images using a script similar to https://github.com/yulunzhang/RDN/blob/master/RDN_IR/RDN_TestCode/RDN_SR_RGB/Prepare_TestData_HR_LR.m.
If there are still some issues, please let me know.
Best,
Yapeng

Thank you for your fast reply!
I have tried change test data path to "data\Vid4" and now I got this error.
Traceback (most recent call last):
File "eval.py", line 101, in
image = io.imread(os.path.join(LR, ims[0]))
"Could not find a format to read the specified file in %s mode" % modename
ValueError: Could not find a format to read the specified file in single-image mode
May I ask how could I fix this? I have found that when I set the test data path as "data", the network generates results well on SPMCs.

Could you remove ".DS_Store" in the Vid4/calendar and try the code again?

It works! Thank you a lot for helping me :)


Not all commands visible when Live Preview web view is active

Testing microsoft/vscode#158835
If the Live Preview tab is focused, this is what clicking on the status bar yields:

vs when the html file is focused:

ok maybe the "Show Preview Internal" doesn't need to be there... but I feel like the others could? Maybe I wanna jump from opening internal to external.

verif steps

Install Live Preview prerelease
Open an internal preview for a file
Have focus on the internal preview iframe
Open command palette
Live Preview: Show Preview (External Browser) should be an option


Should not activate on pages that have their own dark mode

If I visit e.g. GitHub with system dark mode enabled and GitHub configured to obey the system preference, Nightshift will still activate and try to apply its own generated dark mode.
This conflicts with the "real" dark mode and seems to appear worse than Nightshift's generated dark mode on sites that do not have their own dark mode.
In fact, it probably should not activate on any page which has a dark background.

I'm not sure how trivial it would be to detect whether the page is dark enough to not enable a dark mode. Also, I don't think there is a way to check styles for dark mode.
Maybe the solution suggested in #1 makes more sense?

I suppose you could have check like, converting the original background color in greyscale and if greyscale is inferior to 128 then invert?


question: SAML authentication using samltest.id IdP

I've tried to figure out if this plugin is able to authenticate users using the IdP provided by samltest.id?
On the SAML section of the documentation you state:

The plugin supports the following SAML identity providers (IdP):
Azure Active Directory (Office 365) Applications

However, there is a Jumpcloud integration descriped, too. So maybe this plugin also supports generic providers?
I've tried to create a configuration for samltest.id following the Jumpcloud configuration. However, there is a need to upload a metadata XML file to samltest.id. Is this plugin able to generate such a file?
Right now, I'm stuck with the following (non-working) configuration.
{
    order authenticate before respond

    security {
        saml identity provider generic {
            realm app
            driver generic
            idp_metadata_location /etc/caddy/samltest.xml
            idp_sign_cert_location /etc/caddy/samltest.pem
            idp_login_url https://samltest.id/idp/profile/SAML2/Redirect/SSO
            entity_id urn:oasis:names:tc:SAML:attribute:subject-id
            acs_url http://localhost:8082
        }

        authentication portal myapp {
            enable identity provider generic
        }
    }
}

:8082 {
    log

    handle /ping {
        respond "pong"
    }

    handle {
    	authenticate with myapp
        root * /var/www/html/public
        file_server
    }
}

Maybe you could provide a step-by-step guide for configuring generic providers?
Thank you.

@stlrnz , please search this repo for the issues related to Azure: https://github.com/greenpau/caddy-security/issues?q=is%3Aissue+azure
Try asking people for help. For example, @calebcoverdale.

@greenpau Maybe there is a misunderstanding. I'm not looking for help related to Azure. I'm trying to use the plugin in combination with a generic IdP (for examle, samltest.id). Is this even possible?


Use JSON in special/_generate_pyx.py?

Currently the signatures of the special functions are listed in a big string, whose entries look like this:
eval_laguerre --    eval_laguerre[double]: dd->d,     eval_laguerre[double complex]: dD->D,     eval_laguerre_l:ld->d -- orthogonal_eval.pxd

As the functions gain more dispatches this becomes hard to read. Are there objections to instead putting the signatures in a JSON file, where the previous string would become something like this:
"eval_laguerre": {
    "orthogonal_eval.pxd": {
	"eval_laguerre[double]: dd->d",
	"eval_laguerre[double complex]: dD->D",
	"eval_laguerre_l: ld->d"
    }
}

Maybe leave the verbose version as a comment too for quick glance-through?

Seems fine to me


With Tags ticked file merge files fails to create

Exported from each device .jwlibrary backup
Load JW Merge (great tool thanks)
Drop in 3 files to merge

As shown in image when Tags is ticked, but no merge files is created.

As shown in image when Tags is not ticked, the merge files is created.
It seems to be an issue with my Tags, but unsure how I can fix this.

@IDSAU First check that you are using the latest version of JwlMerge. If you are, please post the log file which is in Documents\JWLMerge\Logs

Thanks for reply
log-20210414.txt
Latest Version  JWLMerge <IP_ADDRESS>

I can see that there was an error merging the location data. Would it be possible to send me the backup files please? It would only be me who looks at them and I would delete them asap. If so, please email<EMAIL_ADDRESS>

@IDSAU Thanks for the files. Identified as a corruption in one of the source databases. The Location table contains 2 rows that violate one of the databases unique constraints. These must have been inserted before the constraint was added to the database in a JW Library upgrade.

Fix is to perform a source validation check, stripping out "duplicate" Locations rows if needed.


How to chain query helpers

I use query helpers
animalSchema.query.byName = function(name) {
    return this.find({ name: new RegExp(name, 'i') });
};

var Animal = mongoose.model('Animal', animalSchema);
Animal.find().byName('fido').exec(function(err, animals) {
    console.log(animals);
});

How can i mock the byName call?

Hi @ms007, I think it's not a good practice to append methods to the query. Instead you could (and probably should) use statics.
Your code will look like
animalSchema.statics.findByName = function(name) {
    return this.find({ name: new RegExp(name, 'i') });
};

var Animal = mongoose.model('Animal', animalSchema);
Animal.findByName('fido').exec(function(err, animals) {
    console.log(animals);
});

Then you can test it doing the following
sinon.mock(Animal)
  .expects('findByName')
  .withArgs('fido')
  .chain('exec')
  .resolves('SOME_VALUE');

If i'm not clear enough just tell me!

Hello @gaguirre , thank you for teaching me some mongoose best practice. I'm getting more and more familiar with it.

 Hope this library helps you too.


Code in documentation is broken

I saw this: http://www.juliafem.org/JuliaFEM.jl/latest/packages/FEMBase/basis/ and thought i let you know.

Yes, it has been more or less broken all the time. I'm working with the idea of using Literate.jl to generate documentation from unit tests hoping it would be easier to maintain. Because now the situation is that if something in documentation is outdated or broken, we cannot catch that in CI.
http://www.juliafem.org/FEMBase.jl/latest/fields/
It's going to be something like this.


ring-mqtt

add ring-mqtt chart
Interested if this would run better in my cluster rather than inside home-assistant itself

https://github.com/ankushg/home-cluster/tree/main/cluster/apps/home/ring-mqtt


Invalid pointer error when using cbc

What version of OR-Tools and what language are you using?
Version: v9.1
Language: C++
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
cbc
What operating system (Linux, Windows, ...) and version?
Ubuntu 20.04.6 LTS
What did you do?
When I run the following line of code, I get an "invalid pointer" error. What could be the possible reasons for this issue, and I would appreciate any suggestions that might help resolve this error. Thanks!
Code:
callCbc(absl::StrCat("-threads ", num_threads_, " -solve "), model)

Error:

Anything else we should know about your project / environment
The code is related to the paper NeAT:https://arxiv.org/abs/2202.02171

don't use cbc. It is mostly unmainained, and very unreliable.

What version of OR-Tools and what language are you using? Version: v9.1 Language: C++
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi) cbc
What operating system (Linux, Windows, ...) and version? Ubuntu 20.04.6 LTS
What did you do? When I run the following line of code, I get an "invalid pointer" error. What could be the possible reasons for this issue, and I would appreciate any suggestions that might help resolve this error. Thanks!
Code: callCbc(absl::StrCat("-threads ", num_threads_, " -solve "), model)  Error:
Anything else we should know about your project / environment The code is related to the paper NeAT:https://arxiv.org/abs/2202.02171

Hi, did you solve this problem? I also met same error

I think CBC is not compiled with multi-thread support.
Which is reasonable, as it does not help.
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le mar. 9 juil. 2024 à 14:12, Lu Chen @.***> a écrit :

What version of OR-Tools and what language are you using? Version: v9.1
Language: C++
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP,
Gurobi) cbc
What operating system (Linux, Windows, ...) and version? Ubuntu 20.04.6
LTS
What did you do? When I run the following line of code, I get an
"invalid pointer" error. What could be the possible reasons for this issue,
and I would appreciate any suggestions that might help resolve this error.
Thanks!
Code: callCbc(absl::StrCat("-threads ", num_threads_, " -solve "), model) [image:
image]
https://private-user-images.githubusercontent.com/40981550/322177461-1390e748-fd5c-4f03-b068-2aedef589ede.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1MjcyOTcsIm5iZiI6MTcyMDUyNjk5NywicGF0aCI6Ii80MDk4MTU1MC8zMjIxNzc0NjEtMTM5MGU3NDgtZmQ1Yy00ZjAzLWIwNjgtMmFlZGVmNTg5ZWRlLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA5VDEyMDk1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWMwYmUzYmZiZTgyNzk0NGEwNDJmOWRiMWJjNmNkMGI3YWExMWExZGQ0N2U4ZThjZjVkZmNlYjY5YjgxNDAwZDcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.Vip-FmLJStqOTLPyhESb5IPTJsKbu-uyT7WsYx5IpIw
Error: [image: image]
https://private-user-images.githubusercontent.com/40981550/322177684-e9dea8b8-476b-4501-8c47-117c1031e8a7.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1MjcyOTcsIm5iZiI6MTcyMDUyNjk5NywicGF0aCI6Ii80MDk4MTU1MC8zMjIxNzc2ODQtZTlkZWE4YjgtNDc2Yi00NTAxLThjNDctMTE3YzEwMzFlOGE3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA5VDEyMDk1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTdjMGUyNzllZDM4YjJkZjhlYjgxMjFlZTMxNWI0NjRhMjU2YTJlYTg2NWZjMjhjMzBkMzBiNjE1YmVhYzBhOGQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.yEG_peHC0IW_r0v9ynWSzqoGF3llhNZvDCsZVdkIKAQ
Anything else we should know about your project / environment The code
is related to the paper NeAT:https://arxiv.org/abs/2202.02171
Hi, did you solve this problem? I also met same error
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4181#issuecomment-2217482847,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3KHI46VYUGWSDMHAATZLPHTPAVCNFSM6AAAAABGFE2X5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJXGQ4DEOBUG4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I think CBC is not compiled with multi-thread support. Which is reasonable, as it does not help. Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53 00 Le mar. 9 juil. 2024 à 14:12, Lu Chen @.> a écrit :
…
What version of OR-Tools and what language are you using? Version: v9.1 Language: C++ Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi) cbc What operating system (Linux, Windows, ...) and version? Ubuntu 20.04.6 LTS What did you do? When I run the following line of code, I get an "invalid pointer" error. What could be the possible reasons for this issue, and I would appreciate any suggestions that might help resolve this error. Thanks! Code: callCbc(absl::StrCat("-threads ", num_threads_, " -solve "), model) [image: image] https://private-user-images.githubusercontent.com/40981550/322177461-1390e748-fd5c-4f03-b068-2aedef589ede.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1MjcyOTcsIm5iZiI6MTcyMDUyNjk5NywicGF0aCI6Ii80MDk4MTU1MC8zMjIxNzc0NjEtMTM5MGU3NDgtZmQ1Yy00ZjAzLWIwNjgtMmFlZGVmNTg5ZWRlLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA5VDEyMDk1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWMwYmUzYmZiZTgyNzk0NGEwNDJmOWRiMWJjNmNkMGI3YWExMWExZGQ0N2U4ZThjZjVkZmNlYjY5YjgxNDAwZDcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.Vip-FmLJStqOTLPyhESb5IPTJsKbu-uyT7WsYx5IpIw Error: [image: image] https://private-user-images.githubusercontent.com/40981550/322177684-e9dea8b8-476b-4501-8c47-117c1031e8a7.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1MjcyOTcsIm5iZiI6MTcyMDUyNjk5NywicGF0aCI6Ii80MDk4MTU1MC8zMjIxNzc2ODQtZTlkZWE4YjgtNDc2Yi00NTAxLThjNDctMTE3YzEwMzFlOGE3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA5VDEyMDk1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTdjMGUyNzllZDM4YjJkZjhlYjgxMjFlZTMxNWI0NjRhMjU2YTJlYTg2NWZjMjhjMzBkMzBiNjE1YmVhYzBhOGQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.yEG_peHC0IW_r0v9ynWSzqoGF3llhNZvDCsZVdkIKAQ Anything else we should know about your project / environment The code is related to the paper NeAT:https://arxiv.org/abs/2202.02171 Hi, did you solve this problem? I also met same error — Reply to this email directly, view it on GitHub <#4181 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACUPL3KHI46VYUGWSDMHAATZLPHTPAVCNFSM6AAAAABGFE2X5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJXGQ4DEOBUG4 . You are receiving this because you modified the open/close state.Message ID: @.>

So, If the Threads number shows 1 in Figure 2, can the problem be solved?

It should.
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le mar. 9 juil. 2024 à 14:53, Lu Chen @.***> a écrit :

I think CBC is not compiled with multi-thread support. Which is
reasonable, as it does not help. Laurent Perron | Operations Research |
@.*** | (33) 1 42 68 53 00 Le mar. 9 juil. 2024 à 14:12, Lu Chen @.
> a écrit : … <#m_8861080050972741638_> What version of OR-Tools and what
language are you using? Version: v9.1 Language: C++ Which solver are you
using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi) cbc What operating
system (Linux, Windows, ...) and version? Ubuntu 20.04.6 LTS What did you
do? When I run the following line of code, I get an "invalid pointer"
error. What could be the possible reasons for this issue, and I would
appreciate any suggestions that might help resolve this error. Thanks!
Code: callCbc(absl::StrCat("-threads ", num_threads_, " -solve "), model)
[image: image]
https://private-user-images.githubusercontent.com/40981550/322177461-1390e748-fd5c-4f03-b068-2aedef589ede.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1MjcyOTcsIm5iZiI6MTcyMDUyNjk5NywicGF0aCI6Ii80MDk4MTU1MC8zMjIxNzc0NjEtMTM5MGU3NDgtZmQ1Yy00ZjAzLWIwNjgtMmFlZGVmNTg5ZWRlLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA5VDEyMDk1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWMwYmUzYmZiZTgyNzk0NGEwNDJmOWRiMWJjNmNkMGI3YWExMWExZGQ0N2U4ZThjZjVkZmNlYjY5YjgxNDAwZDcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.Vip-FmLJStqOTLPyhESb5IPTJsKbu-uyT7WsYx5IpIw
https://private-user-images.githubusercontent.com/40981550/322177461-1390e748-fd5c-4f03-b068-2aedef589ede.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1MjcyOTcsIm5iZiI6MTcyMDUyNjk5NywicGF0aCI6Ii80MDk4MTU1MC8zMjIxNzc0NjEtMTM5MGU3NDgtZmQ1Yy00ZjAzLWIwNjgtMmFlZGVmNTg5ZWRlLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA5VDEyMDk1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWMwYmUzYmZiZTgyNzk0NGEwNDJmOWRiMWJjNmNkMGI3YWExMWExZGQ0N2U4ZThjZjVkZmNlYjY5YjgxNDAwZDcmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.Vip-FmLJStqOTLPyhESb5IPTJsKbu-uyT7WsYx5IpIw
Error: [image: image]
https://private-user-images.githubusercontent.com/40981550/322177684-e9dea8b8-476b-4501-8c47-117c1031e8a7.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1MjcyOTcsIm5iZiI6MTcyMDUyNjk5NywicGF0aCI6Ii80MDk4MTU1MC8zMjIxNzc2ODQtZTlkZWE4YjgtNDc2Yi00NTAxLThjNDctMTE3YzEwMzFlOGE3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA5VDEyMDk1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTdjMGUyNzllZDM4YjJkZjhlYjgxMjFlZTMxNWI0NjRhMjU2YTJlYTg2NWZjMjhjMzBkMzBiNjE1YmVhYzBhOGQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.yEG_peHC0IW_r0v9ynWSzqoGF3llhNZvDCsZVdkIKAQ
https://private-user-images.githubusercontent.com/40981550/322177684-e9dea8b8-476b-4501-8c47-117c1031e8a7.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1MjcyOTcsIm5iZiI6MTcyMDUyNjk5NywicGF0aCI6Ii80MDk4MTU1MC8zMjIxNzc2ODQtZTlkZWE4YjgtNDc2Yi00NTAxLThjNDctMTE3YzEwMzFlOGE3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA5VDEyMDk1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTdjMGUyNzllZDM4YjJkZjhlYjgxMjFlZTMxNWI0NjRhMjU2YTJlYTg2NWZjMjhjMzBkMzBiNjE1YmVhYzBhOGQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.yEG_peHC0IW_r0v9ynWSzqoGF3llhNZvDCsZVdkIKAQ
Anything else we should know about your project / environment The code is
related to the paper NeAT:https://arxiv.org/abs/2202.02171
https://arxiv.org/abs/2202.02171 Hi, did you solve this problem? I also
met same error — Reply to this email directly, view it on GitHub <#4181
(comment)
https://github.com/google/or-tools/issues/4181#issuecomment-2217482847>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3KHI46VYUGWSDMHAATZLPHTPAVCNFSM6AAAAABGFE2X5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJXGQ4DEOBUG4
https://github.com/notifications/unsubscribe-auth/ACUPL3KHI46VYUGWSDMHAATZLPHTPAVCNFSM6AAAAABGFE2X5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJXGQ4DEOBUG4
. You are receiving this because you modified the open/close state.Message
ID: @.>
So, If the Threads number shows 1 in Figure 2, can the problem be solved?
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4181#issuecomment-2217652363,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3K3J5R2GKXUHXVB5TDZLPMOBAVCNFSM6AAAAABGFE2X5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJXGY2TEMZWGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I still got this error! And when I change the solver to 'SCIP‘ or 'GLPK', it shows:


Error during "go get"

Hi, just simple to fix error. Prevents building.
$ go get -u github.com/Boostport/avatica
# github.com/Boostport/avatica
./driver.go:55:28: multiple-value uuid.NewV4() in single-value context

uuid.NewV4() returns also error.

I've opened #53 to fix this. The go.uuid package is pinned to master on the commit 36e9d2ebbde5e3f13ab2e2. However, I would still recommend using dep to install the library, in case a breaking change is pushed to master again.


fix(frontend): Remove conversion action when not in transaction page

Motivation
There was a bug so that when we set the token store to a convertible one in the main page, the conversion button would appear (since it is linked to the token store and its possibility to be converted).
To quickly fix the visual aspect of the issue, we filter that button to appear ONLY in when the user already selected the token, meaning, when the user is in a transaction page. In that case, in fact, the token store is set to be the pageToken.
NOTE
To re-iterate, this does not fixes the issue completely, it is just a visual fix. The correct long-term fix is to remove the deprecated token store and adapt the code.
Tests
Before
We notice that the conversion button appears and stays when we select a convertible token in the main page.
https://github.com/user-attachments/assets/4067543a-cdbe-4827-bee5-e66966598626
After
https://github.com/user-attachments/assets/fdc2947f-8358-4ad6-9c78-e123aae89544

[OISY-98]


Unable to authenticate using bcrypt salt

Took a moment to figure out how to set up the plugin to work properly.
which works with md5 hashed passwords.
The problem happens when I try to use bcrypt
with 10 cycle salted passwords:
here is my conf:
.mongo.auth_query.password_hash = salt bcrypt 10
tried also
auth.mongo.auth_query.password_hash = salt bcrypt
and
auth.mongo.auth_query.password_hash = 10 salt bcrypt
my bcrypt hashed password is:
$2a$10$xKRR.n9Mqej0jubXh9iFiOvywcpn9sBLiKalc6UVXI8guzqdc6M0.
The clear text password is:
TEST-PASS-USA
to validate online and generate bcrypt passwords:
https://www.dailycred.com/article/bcrypt-calculator
Getting this error:
2017-06-13 17:23:20.584 [info] <0.1389.0>@emqttd_protocol:trace:347 Client(undefined@<IP_ADDRESS>:55584): RECV CONNECT(Q0, R0, D0, ClientId=6dbb2d6f0e66ecde4706a14922e79304, ProtoName=MQTT, ProtoVsn=4, CleanSess=true, KeepAlive=60, Username=usa-user-test, Password=******)
2017-06-13 17:23:20.587 [error] <0.1389.0>@emqttd_protocol:process:214 Client(6dbb2d6f0e66ecde4706a14922e79304@<IP_ADDRESS>:55584): Username 'usa-user-test' login failed for {function_clause,[{emqttd_auth_mod,passwd_hash,[{salt,bcrypt},<<"TEST-PASS-USA">>],[{file,"src/emqttd_auth_mod.erl"},{line,55}]},{emq_auth_mongo,check_pass,3,[{file,"src/emq_auth_mongo.erl"},{line,65}]},{emq_auth_mongo,check,3,[{file,"src/emq_auth_mongo.erl"},{line,54}]},{emqttd_access_control,auth,3,[{file,"src/emqttd_access_control.erl"},{line,60}]},{emqttd_protocol,authenticate,2,[{file,"src/emqttd_protocol.erl"},{line,513}]},{emqttd_protocol,process,2,[{file,"src/emqttd_protocol.erl"},{line,194}]},{emqttd_client,received,2,[{file,"src/emqttd_client.erl"},{line,315}]},{gen_server2,handle_msg,2,[{file,"src/gen_server2.erl"},{line,1046}]}]}

PS. Is there a guide somewhere how to run the whole EMQ project
locally from IntelliJ Idea and debug it?
I'm quite new to Erlang and would be much more efficient to Debug
the code and learn while doing so..

I think I might have found the issue.....
the default erlang bcrypt uses 12 cycle salt:
from:
https://github.com/smarkets/erlang-bcrypt

default_log_rounds
Sets the default number of rounds which define the complexity of the hash function. Defaults to 12.

While I'm trying to use 10 cycle.
I will test and report.

@syberkitten https://github.com/smarkets/erlang-bcrypt/blob/master/src/bcrypt.app.src#L10

@turtleDeng yeah, thanks for pointing this out -) but it still does not solve my problem.
I'm creating a dummy user before the test with 12 cycle bcrypt salt (using nodejs bcrypt)
but still getting back:
Connection refused: Bad username or password

Thank you for treating this seriously. :)
I would love to help, so if any more info or tests
give me a lead, and I'll try to check...
Cheers 👍

I am getting similar error while using bcrypt with pgsql. Authentication fails while using bcrypt passwords. Tried 10 & 12 cycles, but no luck.
I am using emqtt stable version 2.2 on ubuntu 16.04.

@zacsonskaria It looks like the closest would be to use the emq-auth-http plugin
and make rest HTTP calls for every connecting client, the rest endpoint
is responsible to Authenticate the user or not.
Of course it is more elegant to have the broker do it,|
but until it's fixed ...

@syberkitten Thats definitely a work around, but for me it will bring unnecessary overheads.
I have tried numerous combinations to somehow make bcrypt work (extracting hash and salt separately, changing cycles, prefix hash with salt etc) but nothing works. I heard bcrypt got enabled recently in 2.2 and wish it works even with some limitations.

@turtleDeng
Is there any news on this issue?
Should I currently use the emq-auth-http plugin?
Perhaps the issue is simply resolved by allow a different
number of cycles for the bcrypt (currently it's 12)
but we are using 10.
12 cycles takes too long on our server infrastructure,
so we would like to keep it at max 10.
this is the ref:
https://github.com/smarkets/erlang-bcrypt/blob/master/src/bcrypt.app.src#L10

I have the same problem, looking forward to the solution, thank you!

@Bruce-sz Please provide error.log.

in emq_auth_mysql.conf
=====but use bcrypt is failure, username="13701234567" login.===
Password hash: plain, md5, sha, sha256, bcrypt
#auth.mysql.password_hash = sha256
sha256 with salt prefix
auth.mysql.password_hash = salt,sha256
bcrypt with salt only prefix
auth.mysql.password_hash = salt,bcrypt
=====but use sha256 is ok, username="RTW-100-000002" login.===
Password hash: plain, md5, sha, sha256, bcrypt
#auth.mysql.password_hash = sha256
sha256 with salt prefix
auth.mysql.password_hash = salt,sha256
bcrypt with salt only prefix
auth.mysql.password_hash = salt,bcrypt
===============
I always restart emqtt server when re-setup config

@Bruce-sz The salt filed is incorrect! The bcrypt cipher-text structure is:
$2a$(2 chars work cost)$(22 chars salt )(31 chars hash)

So the cipher-text has contained salt!
The salt field of user "13701234567" should be $2a$12$GgGmmP8xAMnummMdCeKn9. (prefixed 29 characters)
Besides, you should ensure configuration files (emq_auth_mysql.conf) is correct:
## config auth fields is `password` and `salt`
auth.mysql.auth_query = select password, salt from mqtt_user where username = '%u' limit 1

## config auth method is `salt,bcrypt`
auth.mysql.password_hash = salt,bcrypt

@HJianBo Thanks!

In EMQ 2.3 and emq-auth-mongo, support any bcrypt rounds. Because the server only checks whether the chiper-text is equal to the result which is encrypted by the plaintext & salt.
So you first have to make sure that the configuration file is correct (emq_auth_mongo.conf):
## config server <IP_ADDRESS>:27017 or other address
auth.mongo.server = <IP_ADDRESS>:27017

##  auth_query
auth.mongo.auth_query.password_field = password,salt

## bcrypt with salt prefix
auth.mongo.auth_query.password_hash = salt,bcrypt

Then generate salt & cipher-text by bcrypt, and saving to mongodb. For example:
> use mqtt

# the clear password is TEST-PASS-USA
> db.mqtt_user.insert({username: 'test-mongo', password: '$2a$10$xKRR.n9Mqej0jubXh9iFiOvywcpn9sBLiKalc6UVXI8guzqdc6M0.', salt: '$2a$10$xKRR.n9Mqej0jubXh9iFiO'})

Try login again ! 👻
How generate salt & cipher-text? by nodejs
Or taking the first 29 characters of the cipher-text

@syberkitten @Bruce-sz @HJianBo Closed. Reopen it if any issue still.

@syberkitten
In EMQ 2.3 and emq-auth-mongo, support any bcrypt rounds. Because the server only checks whether the chiper-text is equal to the result which is encrypted by the plaintext & salt.
So you first have to make sure that the configuration file is correct (emq_auth_mongo.conf):
## config server <IP_ADDRESS>:27017 or other address
auth.mongo.server = <IP_ADDRESS>:27017

##  auth_query
auth.mongo.auth_query.password_field = password,salt

## bcrypt with salt prefix
auth.mongo.auth_query.password_hash = salt,bcrypt

Then generate salt & cipher-text by bcrypt, and saving to mongodb. For example:
> use mqtt

# the clear password is TEST-PASS-USA
> db.mqtt_user.insert({username: 'test-mongo', password: '$2a$10$xKRR.n9Mqej0jubXh9iFiOvywcpn9sBLiKalc6UVXI8guzqdc6M0.', salt: '$2a$10$xKRR.n9Mqej0jubXh9iFiO'})

Try login again ! 👻
How generate salt & cipher-text? by nodejs
Or taking the first 29 characters of the cipher-text

I using Windows10 with MongoDB 4.02 anb salt,bcrypt. it login fails as show log in emqx console as below:-
19:34:16.052 [error] MQTT(test-server@<IP_ADDRESS>:61244): Username 'test-mongo' login failed for {{badmatch,{error,{bcrypt,{"no such file or directory","bcrypt.app"}}}},[{emqx_passwd,hash,2,[{file,"src/emqx_passwd.erl"},{line,41}]},{emqx_auth_mongo,check_pass,4,[{file,"src/emqx_auth_mongo.erl"},{line,60}]},{emqx_auth_mongo,check,3,[{file,"src/emqx_auth_mongo.erl"},{line,50}]},{emqx_access_control,authenticate,3,[{file,"src/emqx_access_control.erl"},{line,71}]},{emqx_protocol,authenticate,2,[{file,"src/emqx_protocol.erl"},{line,613}]},{emqx_protocol,process_packet,2,[{file,"src/emqx_protocol.erl"},{line,289}]},{emqx_connection,handle_packet,2,[{file,"src/emqx_connection.erl"},{line,327}]},{gen_server,try_dispatch,4,[{file,"gen_server.erl"},{line,637}]}]}
what the problem and how can i fix it?

Yes, I testing both EMQ 2.3.11 and EMQX 3.0 beta 3 (current version download from website) but unlucky it doesn’t works

@seksity From the error log: {{badmatch,{error,{bcrypt,{"no such file or directory","bcrypt.app"}}}, it is likely that your release does not have bcrypt or it is not compiled successfully. Could you check if there is bcrypt under deps folder and if bcrypt/ebin has the compiled beam files?

@seksity From the error log: {{badmatch,{error,{bcrypt,{"no such file or directory","bcrypt.app"}}}, it is likely that your release does not have bcrypt or it is not compiled successfully. Could you check if there is bcrypt under deps folder and if bcrypt/ebin has the compiled beam files?
For the salt field, we will support bcrypt without "salt" field in the near future.

Thanks @ngjaying, cause of i download EMQX from website, so that means I need to git clone from Github and compile by make file instance, right?
great job for without "salt" field soon

@seksity From the error log: {{badmatch,{error,{bcrypt,{"no such file or directory","bcrypt.app"}}}, it is likely that your release does not have bcrypt or it is not compiled successfully. Could you check if there is bcrypt under deps folder and if bcrypt/ebin has the compiled beam files?
For the salt field, we will support bcrypt without "salt" field in the near future.

Thanks @ngjaying, cause of i download EMQX from website, so that means I need to git clone from Github and compile by make file instance, right?
great job for without "salt" field soon

@seksity windows version does not support bcrypt. I recommend you to use other OS or use WSL(Windows subsystem for linux). Thanks.

it seems that this problem has been solved.


Status support: Event notification support for build and application status for IDE integration for devfile scenarios

What type of PR is this?
/kind feature
What does does this PR do / why we need it:
Short answer: This watches the status of odo-managed components and outputs status changes as machine readable JSON events, for use by consuming tools (such as IDEs)
Long answer: Design proposal is here.
Parent issue:
https://github.com/openshift/odo/issues/2550
How to test changes:
# Set experimental mode
# Ensure a k8s/openshift context is set in your environment

git clone https://github.com/odo-devfiles/nodejs-ex
cd nodejs-ex
odo create nodejs
odo url create --port 3000 --host (your host)
odo push
odo component status -o json -f

# JSON events, corresponding to the status of odo-managed cluster resources, will now be output to the container:
# - k8s resource changes: will be output only whenever they change
# - URL reachability changes: will be output only if it switches states (eg reachable <-> not-reachable)
# - supervisord program status: will be output every X seconds

You can now (for example) do the following, and see it reflected in the JSON events:

Kill the k8s pod, and watch the odo JSON events that show the pod terminating and then be recreated; you will also see that even though the pod restarts, the supervisord program does not (you must issue another odo push to restart it)
Issuing odo push -f will restart the supervisord program, and you will see this reflected in the status events
Exec into the container and kill the node process (you will see that supervisord program status switches to STOPPED from RUNNING)
Disable your network/VPN connection and see that an error is reported; re-enable, and see the Kubernetes status and supervisord status will be updated

Special notes to the reviewer:

I am intentionally suppressing a gosec warning with #nosec, see the comment for my reasoning.

Note: For any IDEs/consuming tools that wish to use the event data generated by odo component status -o json --follow or odo push -o json, see the following example code: https://github.com/jgwest/snippets/tree/odo-event-consumer/odo-event-consumer

This example reads the JSON event data from these commands, unmarshalls them from JSON into objects, and then uses an algorithm to calculate the component status based on those events.

/ok-to-test

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

@jgwest Can you rebase?

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

we should move this out of experimental mode

@girishramnani Done, is now out of experimental and is restricted to support configuration (eg devfiles, not s2i).

Unit tests on windows are constantly failing on
FAIL	github.com/openshift/odo/pkg/url [build failed]
# github.com/openshift/odo/pkg/url [github.com/openshift/odo/pkg/url.test]
pkg\url\status_test.go:139:45: not enough arguments in call to "github.com/openshift/odo/pkg/kclient/fake".GetIngressListWithMultiple
	have (string)
	want (string, string)

/retest

/retest

@kadel

Unit tests on windows are constantly failing on

Looks like this was caused (on all platforms) by changes that have merged since I last rebased...
I have (again) rebased and the unit and integration tests are now passing. HOWEVER, it looks like https://github.com/openshift/odo/pull/4121 broke the RPM build of ci/prow/unit last night, so they won't be green here until that is fixed.

/retest

I have (again) rebased and the unit and integration tests are now passing. HOWEVER, it looks like #4121 broke the RPM build of ci/prow/unit last night, so they won't be green here until that is fixed.

that should be resolved now, the changes that broke it were reverted

Review comments addressed and rest of code looks good to me
/lgtm

adding back
/lgtm
after rebase

Waiting for travis. If it is green, we can merge it

/approve


atf_inttest.test_cleanup_crash failed to find necessary failure condition string

From https://jenkins.freebsd.org/job/FreeBSD_HEAD-tests/1053/testReport/junit/local.kyua.testers/atf_inttest/test__cleanup__crash/ :

Error Message

testers/atf_inttest.c:207: atf_utils_compare_file("test-result", "broken: Test case cleanup received signal 6 (core dumped)\n") not met
Standard Output

subprocess stderr: About to die due to SIGABRT!
subprocess stderr: Subprocess timed out; sending KILL signal...
subprocess stderr: Process with PID 46776 dumped core; attempting to gather stack trace
subprocess stderr: (no debugging symbols found)...Core was generated by `atf_helpers'.
subprocess stderr: Program terminated with signal 6, Aborted.
subprocess stderr: (no debugging symbols found)...(no debugging symbols found)...(no debugging symbols found)...#0  0x0000000800b7567a in thr_kill () from /lib/libc.so.7
subprocess stderr: GDB exited successfully

I'm going to close this as obsolete. All of the testers code has been removed so the problem above is either gone or has moved; a new release with all the hot shininess is due soon. (It's hard for me to tell if the bug is gone because I do not know what caused the issue...)


No server start button

I installed steamcmd and then the server, but I have no server start button

On the windows server I had very low resolution and that was the issue.


[WIP] cmd/govim: use signs for diagnostics

Closes #284
This PR will place one sign on each line where there is a quickfix entry.

FYI - https://github.com/vim/vim/commit/809ce4d317fe12db0b2c17f16b4f77200fb060c4 just landed.

Closed in favour of #382 that uses sign_placelist() and sign_unplacelist() to place/remove several signs at once.


ADD: Advanced gas setting

Is your feature request related to a problem? Please describe.
Advanced users would like to set their own gas & nonce settings when sending Tx.
Describe the solution you'd like
At minimum, provide the same settings like in Trust Wallet (check the available settings when sending e.g. ETH and BTC)

@vultisig/android See the figma design for it to implement.

@yvebe can you add the icon in top right too? or is there already a pull for it?

A new issue has been created for this ,https://github.com/vultisig/vultisig-android/issues/1166
Let's move this one to done


753 - Descheduling framework

For https://github.com/kubernetes-sigs/descheduler/issues/753. Combination of https://docs.google.com/document/d/1fr5tfSaCkqGZVXUMJA9pUkIy6GGRwJumPN5MeZ8oewo/edit#heading=h.xgjl2srtytjt and https://docs.google.com/document/d/1tWpXoZ6YF3ksnmueM2POwASdxoiLW8y7TXuWFxT8NSo/edit#heading=h.xgjl2srtytjt.

Thank you Mike for the review. Comments addressed.

really very good picture.


"300 degC" wrongly converted to "300 K"

Summary
"300 degC" is wrongly converted to "300 K". Probably it is a bug in mathjs. I tested with (10.2.4).
Steps to Reproduce
to reproduce it one can store a scalar value with 'degC' unit or  simpler in nodejs
 unit("300 degC").toSI().toJSON()

### Current Behaviour

```json
{ mathjs: 'Unit', value: 300, unit: 'K', fixPrefix: true }

Expected Behaviour
{ mathjs: 'Unit', value: 573.15, unit: 'K', fixPrefix: true }

Extra Details
Workaround for the problem could be to change the above line of the code in convertToSI to something like
 unit(inputValue, inputUnit).to(unit(inputUnit).toSI()).toJSON()

which gives the correct result
{ mathjs: 'Unit', value: 573.15, unit: 'K', fixPrefix: true }

I've added the patch to #925

@jkotan given that the discussion regarding arrays in Scientific Metadata requires more time to be resolved, could you please isolate the code to solve the problem highlighted in this issue in a separate PR?
Thanks


Eretana2 branch

Expected: validation fail
hmtl  tags in portfolio require Text Here however are not needed in my code


Add macro for defining scopes

The current mechanism for creating LocalScopes relies on a function and closures. This change adds a macro named mpi::define_scope! for defining the LocalScope instead, which allows the user to stay in their local function context, rather than having a separate function. This enables more rich flow control when dealing with scopes, and also unifies scope creation for when I add async support.
For example, define_scope! can be used like this:
// define send and receive buffers here
{
    // you may define send buffers here
    mpi::define_scope!(scope);
    // use scope like normal with receive and send buffers
}
// use receive buffers here

The macro works and is safe because the created &LocalScope shadows the stack-pinned LocalScope, making it impossible to forget, or even reference, the underlying LocalScope.
I also changed all the examples to use mpi::define_scope!(scope);.
There's probably a better name than mpi::define_scope, btw. Maybe just mpi::scope!? Or mpi::request_scope!?

Hm... defining Scope for pinned LocalScope isn't actually sound and I'm not sure why I thought it was. Marking as WIP.
I think the macro is fine, but trying to use pinning to solve the issue doesn't work.

To be honest, the macro-based approach is a bit too magical for my taste. Scope/closure based API are a standard idiom when lifetimes need to be controlled so this change could make rsmpi harder to learn. Injecting an unsafe block into user code could also unfavorably interact with things like #![deny(unsafe)] AFAIU.

Injecting an unsafe block into user code could also unfavorably interact with things like #![deny(unsafe)] AFAIU.

Should be fixable with an #[allow(unsafe)], right?

To be honest, the macro-based approach is a bit too magical for my taste.

Well, I'm not super attached to this change, but the big reason I want it is for async/await (which I'd like to tackle seriously for 0.7). It would unify defining scopes for both sync code and async code. We can maybe push this off for 0.7.

Should be fixable with an #[allow(unsafe)], right?

Yes, I think so.

We can maybe push this off for 0.7.

I would be glad if we did as the interactions with pinning should be worked out first. Polling a write on a socket for completion basically has the same problem of borrowing a buffer and then yielding. Hence an implementation of immediate operations based on async/await might not have this problem at all.

Is the main benefit here that you can have overlapping scopes? For example, in an atmospheric simulation, you can have a scope for communication with horizontal neighbors and another scope to communicate with vertical neighbors. With this macro, they can overlap, but they have to be either nested or disjoint if using a closure. I probably need to experiment.


users should not be encouraged to copy their seed phrases

this is where I wanted to assign you :)


Cart additional arguments

Hey guys,
I think this is an important issue, in case lets say we want to add a coupon discount to a cart, or any other information relative to the cart, we should be able to in a additional array.

Hi,
Please look at issue #238 it's basically a WIP

You are welcome to share your thoughts on my proposal: https://github.com/Crinsane/LaravelShoppingcart/issues/238


Name Change rq

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
Change my token name #21333

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

@keone

hey dude ive the same problem
duplicate tokenadress do you know how to fix that i dont have github skills need help from the devs but they only send me the link https://github.com/solana-labs/token-list#duplicate-token and I still dont know what to do

closing this issue. please edit the existing token instead of adding it in again and then submit a new PR


Update Japanese info text of mqtt out node

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

This PR updates Japanese info text of MQTT out node according to changes on original English info text(https://github.com/node-red/node-red/commit/aa6c0b9d6e8b5601b22d543c700ead9166f10903).
Also fixes some typos.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 77.586% when pulling e7c2ff3bd2befe64a5b457a3db0c7eaf9adecfce on node-red-hitachi:fix-mqtt-jp into b81be8f35864e50c0589eca149d46f52c31059b4 on node-red:master.


Connection WebSocketChannelException in flutter

Getting error in flutter deepgram_speech_to_text: ^2.2.2
Connection to 'https://api.deepgram.com:0/v1/listen?model=aura-asteria-en&encoding=linear16&container=wav&detect_language=true&filler_words=false&punctuation=true#' was not upgraded to websocket
late DeepgramLiveTranscriber transcriber;
Future<void> initDeepgramStt() async {
  await cubit.initDeepgram(isInit: true);

  final Stream<List<int>> micStream = await AudioRecorder().startStream(
    const RecordConfig(
      encoder: AudioEncoder.pcm16bits,
      sampleRate: 16000,
      numChannels: 1,
    ),
  );
  transcriber = cubit.deepgram.createLiveTranscriber(micStream);

  transcriber.stream.listen((DeepgramSttResult res) {
    Utility.showLog('${res.transcript}');
  });
}

Getting error when call start method:
transcriber.start();
Flutter doctor result is following :
[✓] Flutter (Channel stable, 3.22.2, on macOS 14.5 23F79 darwin-arm64 (Rosetta), locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] Connected device (4 available)
[✓] Network resources

hey @amit-iconflux as mentioned in the readme "...was not upgraded to websocket" errors often mean that your parameters are invalid, in the url I see that its the case, you used params that belong to TTS instead of STT, those are the one you should use :
final streamParams = {
  'detect_language': false, // not supported by streaming API
  'language': 'en',
  // must specify encoding and sample_rate according to the audio stream
  'encoding': 'linear16',
  'sample_rate': 16000,
};

for STT streaming you can find valid params here : https://developers.deepgram.com/reference/listen-live


feat: Middleware example to docs

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

New section with an example on how to use a middleware for auth redirect
https://github.com/Intevel/nuxt-directus/issues/8
Checklist:

[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes (if not applicable, please state why)

Thanks for this PR 💚
But please call it "Middleware example"

Fix 👍

And you have to take a look at this: https://www.conventionalcommits.org/en/v1.0.0/
Fix your PR and commit messages, I cant merge it until you did.

Fixed it, merged! Thanks for your work

Sorry,
lunch time, next time i will follow your suggestion, so that you don't need to spent too much time on it.
👍

No problem


Deleting row from ngTable when it's the last item on page

When the last item is removed from a table on page 2 or greater, the paginate navigation disappears.
    $scope.tableParams = new ngTableParams({
        page: 1,            // show first page
        count: 10,           // count per page
        sorting: {
            firstName: 'asc'     // initial sorting
        }
    }, {
        total: 0,
        getData: function ($defer, params) {
            $scope.myData = angular.copy($scope.clients, []);
            var filteredData = params.filter() ?  $filter('filter')($scope.myData, params.filter()) : $scope.myData;
            var orderedData = params.sorting() ? $filter('orderBy')(filteredData, params.orderBy()) : filteredData;

            params.total(orderedData.length);
            var data = orderedData.slice((params.page() - 1) * params.count(), params.page() * params.count());

            //EXTRA CHECK
            if (data.length === 0 && params.page() !== 1){
                params.page(params.page()-1);
                filteredData = params.filter() ?  $filter('filter')($scope.myData, params.filter()) : $scope.myData;
                orderedData = params.sorting() ? $filter('orderBy')(filteredData, params.orderBy()) : filteredData;
                params.total(orderedData.length);
                data = orderedData.slice((params.page() - 1) * params.count(), params.page() * params.count());
            }

            $defer.resolve(data);

        }
    });

I got around it with the above check to see if the filtered data comes back empty and if we're not on page one. Is there a built in way to manage this that I am overlooking?

+1. I've got the same issue as above.

As a workaround, you can ckeck If the current row is the last one on that page and it's not the first page
if ($scope.tableParams.data.length == 1 && vm.tableParams.page() != 1) {      $scope..tableParams.page(vm.tableParams.page() - 1); } $scope..tableParams.reload();

That's what I've done in the past.
I don't think ngTable is going to be monitoring the dataset any time soon
(this would introduce potential performance problems). So I would suggest
going with this as a solution.
On Tue, Feb 7, 2017 at 9:46 AM, M.Msallati<EMAIL_ADDRESS>wrote:

As a workaround, you can ckeck If the current row is the last one on that
page and it's not the first page
if ($scope.tableParams.data.length == 1 && vm.tableParams.page() != 1) {
$scope..tableParams.page(vm.tableParams.page() - 1); }
$scope..tableParams.reload();
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/esvit/ng-table/issues/720#issuecomment-277949435, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA2HPnt6g1_cHsg4NhwX9VKNbwHuzASsks5raD2HgaJpZM4GLPEq
.

I've also used @msallati workaround in the past. Works pretty good but it would be nice to add this fix to the ngTable as it is a little bit annoying... It's only one check during deleting operation so I think it won't be a performance hit for overall library.

The problem is that ngTable doesn't "know" about the delete operation.
ngTable would need to watch the dataset for changes - this would be a
performance problem
On Tue, Feb 7, 2017 at 10:04 AM, Jakub Powierza<EMAIL_ADDRESS>wrote:

I've also used @msallati https://github.com/msallati workaround in the
past. Works pretty good but it would be nice to add this fix to the ngTable
as it is a little bit annoying... It's only one check during deleting
operation so I think it won't be a performance hit for overall library.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/esvit/ng-table/issues/720#issuecomment-277953666, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA2HPo3igPmhmaPxxusbi-rJ1YreOLGqks5raEGigaJpZM4GLPEq
.

Thanks for your explanation! Now I can fully understand what's going on :)


No Windows support for latest version for RustDDS

Hi,
I tried to compile the latest version of RustDDS, however src/mio_source/mod.rs uses os::fd::{AsRawFd, FromRawFd, OwnedFd} which is not available under Windows.

Yes, this is a semi-accidental change due to mio-0.8 support.
We need to send notifications from the background thread to the application thread. In order to poll these notifications with mio-0.8, there needs to be an OS file descriptor that can be polled to detect a notification. We use a socketpair to do this, so a dummy piece of data is sent over a socketpair. This can then be polled with mio to detect that there is an event available.
The setup of socket pairs to do this was not fully portable from Linux to Windows. Do you have means to fix this and propose a PR? It should be sufficient to fix only the mio_source module.

Was this fixed by https://github.com/jhelovuo/RustDDS/pull/290?

Sadly no.  With a new project ("cargo new dds_test"), rust 1.71.0 (stable, not nightly), on Windows 10, with the ONLY dependency being rustdds="0.8.4" you get this error message:
error[E0432]: unresolved import `std::os::fd`
 --> C:\Users\Kevin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\rustdds-0.8.4\src\mio_source.rs:4:7
  |
4 |   os::fd::{AsRawFd, FromRawFd, OwnedFd},
  |       ^^ could not find `fd` in `os`

error[E0599]: no function or associated item named `from_raw_fd` found for struct `mio::net::TcpStream` in the current scope
  --> C:\Users\Kevin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\rustdds-0.8.4\src\mio_source.rs:53:57
   |
53 |   let rec_mio_socket = unsafe { mio_08::net::TcpStream::from_raw_fd(rec_sps.as_raw_fd()) };
   |                                                         ^^^^^^^^^^^ function or associated item not found in `TcpStream`

error[E0599]: no method named `as_raw_fd` found for struct `socketpair::SocketpairStream` in the current scope
  --> C:\Users\Kevin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\rustdds-0.8.4\src\mio_source.rs:53:77
   |
53 |   let rec_mio_socket = unsafe { mio_08::net::TcpStream::from_raw_fd(rec_sps.as_raw_fd()) };
   |                                                                             ^^^^^^^^^ method not found in `SocketpairStream`

error[E0599]: no function or associated item named `from_raw_fd` found for struct `mio::net::TcpStream` in the current scope
  --> C:\Users\Kevin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\rustdds-0.8.4\src\mio_source.rs:54:58
   |
54 |   let send_mio_socket = unsafe { mio_08::net::TcpStream::from_raw_fd(send_sps.as_raw_fd()) };
   |                                                          ^^^^^^^^^^^ function or associated item not found in `TcpStream`

error[E0599]: no method named `as_raw_fd` found for struct `socketpair::SocketpairStream` in the current scope
  --> C:\Users\Kevin\.cargo\registry\src\index.crates.io-6f17d22bba15001f\rustdds-0.8.4\src\mio_source.rs:54:79
   |
54 |   let send_mio_socket = unsafe { mio_08::net::TcpStream::from_raw_fd(send_sps.as_raw_fd()) };
   |                                                                               ^^^^^^^^^ method not found in `SocketpairStream`

Some errors have detailed explanations: E0432, E0599.
For more information about an error, try `rustc --explain E0432`.
error: could not compile `rustdds` (lib) due to 5 previous errors

So I don't know what people are compiling, but with a very basic test, it doesn't build, even with no code present except the dependency.
What's the latest version that actually builds in Windows?  0.7.x?

My previous comment was useless.  When you specify the dependency to the github repository, it appears to build.  to do that in Cargo.toml, use a line similar to this:
rustdds= { git="https://github.com/jhelovuo/RustDDS.git" }

Seems to build now from the latest master.  Probably that PR that did it, but it's not released either, so you can't pull just from crates.io.

The master branch works for me on Windows. Perhaps it's time for a minor version release on crates.io?


Imaginary clinic world and hotel world

This PR includes the following contents:

imaginary clinic map and launch scripts
hotel map and launch scripts
HospitalRobot model and adapter launch script
edits in DeliveryRobot inertial parameters
request_lift task script
updates in README.md

This PR should only be merged after https://github.com/osrf/traffic_editor/pull/221 and https://github.com/osrf/rmf_core/pull/139 are merged.

I have updated the lift configurations according to changes in https://github.com/osrf/traffic_editor/pull/220 and https://github.com/osrf/traffic_editor/pull/221. It should be ready for review now

Thanks for getting this ready! A high level comment: can we change the map name from "imaginary_clinic" to "clinic"?


Pin orchestra to v0.1 release

Pin the orchestra dependency to v0.1 release so changes to orchestra will not break future v2.8.x patch releases.

Thanks :+1:


Add pagination to devotionals page

I'm going to close this. Either we will be removing devotionals, or combining devotionals, sermons, and videos all into one resources page.


internal/ui: update cursor/touch positions right after the layout is determined

Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[X] Android
[X] iOS
[ ] Nintendo Switch
[ ] Xbox
[X] Web Browsers

What feature would you like to be added?
Cursor and touch positions are affected by the current layout (see clientPositionToLogicalPosition). These should be updated right after the layout is determined (see layoutGame call in updateFrameImpl) to reflect the latest state.
This fix is needed especially for mobiles and browsers. For desktops, I'll create a commit soon.
Why is this needed?
In order to provide more exact cursor/touch positions to the user.

This fix is needed for #2475


Statically link the C runtime in the agent

Problem
We've had issues in the past where there was a mismatch of c runtime versions between our CI environment and the fuzz environment.
In the best case, this can cause confusion for our users of this unknown (to them) dependency. In the worst case, it can stop us from meeting our users in the environments they need to fuzz in.
Solution
By statically linking the c runtime, we wouldn't have those issues and we take a step in decoupling the agent from our CI environment which will make a future Bring Your Own Compute feature easier to implement.
Helpful link: https://doc.rust-lang.org/reference/linkage.html#static-and-dynamic-c-runtimes
AB#40239843

Note that I tried this with libmusl already and it seemed that some of our dependencies were incompatible with it (sorry, can't recall which off the top of my head).

Unfortunately, it wasn't as easy as adding crt-static. I explored a bit but in the end I couldn't get it to compile without going down a rabbit hole. I'm sure it's possible but I think we need to explicitly call out a bunch of libs that rust automatically handles for us and in the end there's still some dynamic linking that has to happen if we choose to use glibc.
I think we should put a pin in this for now until we decide that we absolutely need this functionality since it's a non-trivial change.


PWM??

Can I use pwm with this module. I want to supply half voltage to the pins.
I tried:
gpio.write(pin.number, 0.5, function() {});
but it does not seem to work.

This module just wraps the built-in GPIO functionality on the Raspberry Pi, and as far as I'm aware there's no options for fine-control of the voltage like this.


chore: set consensus version to 1

This will prevent accidentally mixing up the first release with federations previously deployed off random master commits.
Related to #3053

Should we also update module versions?

Debugging the test failure:

The client seems to assume all its modules are incompatible, so tries to initialize without a primary module, which fails
Using websocat I get the following version info:

{"jsonrpc": "2.0", "method": "version", "id":"0", "params": [{"params": null}]}
{"jsonrpc":"2.0","result":{"core":{"core_consensus":1,"api":[{"major":0,"minor":0}]},"modules":{"0":{"core_consensus":0,"module_consensus":0,"api":[{"major":0,"minor":0}]},"1":{"core_consensus":0,"module_consensus":0,"api":[{"major":0,"minor":0}]},"2":{"core_consensus":0,"module_consensus":0,"api":[{"major":0,"minor":0}]}}},"id":"0"}

Formatted result
{
  "core": {
    "core_consensus": 1,
    "api": [
      {
        "major": 0,
        "minor": 0
      }
    ]
  },
  "modules": {
    "0": {
      "core_consensus": 0,
      "module_consensus": 0,
      "api": [
        {
          "major": 0,
          "minor": 0
        }
      ]
    },
    "1": {
      "core_consensus": 0,
      "module_consensus": 0,
      "api": [
        {
          "major": 0,
          "minor": 0
        }
      ]
    },
    "2": {
      "core_consensus": 0,
      "module_consensus": 0,
      "api": [
        {
          "major": 0,
          "minor": 0
        }
      ]
    }
  }
}

* I guess the per-module `core_consensus` value is wrong? Why is it needed anyway? Is it ever expected to be not in line with the main `core_consensus`? Should such a `fedimintd` even start (or compile)?

This and this looks wrong. We should be taking these from the constant of the build we're building.


Avoid package upgrades during CI

In GitLab by @samthursfield on Oct 2, 2017, 15:40
The .gitlab.ci.yml currently does all kinds of dnf install and pip install calls which mean that each run of the CI can have different results depending on what stuff the package managers install.
We should ensure that the Docker image used for CI contains all the dependencies that are needed. This may require having a separate tag used for running the tests, as we don't want to bloat the image that we provide for people who just want to run BuildStream too much.

In GitLab by @samthursfield on Oct 2, 2017, 15:40
assigned to @samthursfield

In GitLab by @samthursfield on Oct 9, 2017, 15:50
mentioned in merge request !105

In GitLab by @samthursfield on Oct 10, 2017, 12:42
closed

In GitLab by @tristanvb on Oct 11, 2017, 04:54
I think this is incomplete, it seems to only cover the dnf installs.
We should be changing the .gitlab-ci.yml to invoke pip install in such a way that no random things from the internet are getting downloaded.
When installing buildstream, this can be done with e.g.:
pip install --no-index .

If this by itself does not break integration tests, that is good news but we should still add the protection to ensure there are no random things being installed from the internet.
If the integration tests break, it means we need to pre install some more dependencies in the docker image.

In GitLab by @tristanvb on Oct 11, 2017, 04:54
reopened

In GitLab by @samthursfield on Oct 13, 2017, 12:25
mentioned in merge request buildstream-docker-images!2

In GitLab by @samthursfield on Oct 13, 2017, 12:53
mentioned in merge request !110

In GitLab by @samthursfield on Oct 18, 2017, 05:27
closed via commit fd91491a7619964e01d1fc8dede1c07cb68a433f

In GitLab by @samthursfield on Oct 18, 2017, 05:46
mentioned in commit fd91491a7619964e01d1fc8dede1c07cb68a433f

In GitLab by @tristanvb on Oct 18, 2017, 06:39
mentioned in commit 0d4d88a9cef7a00fcb49a594ec6b4d56f54c1e84

In GitLab by @tristanvb on Oct 18, 2017, 06:39
mentioned in merge request !114

In GitLab by @tristanvb on Oct 18, 2017, 06:44
More whackamole going on here.
Looks like previous merges still did not cover the pytest and pytest_unix cases, since these indirectly download random stuff from the internet.
This is addressed by !114, but as is indicated by this recent pipeline log with the latest related patches applied, the docker image will need to be updated with any python packages which are required by setup.py's tests_require entries before we can merge it.

In GitLab by @samthursfield on Oct 18, 2017, 09:56
Oops, well spotted. I didn't realise setup.py test would also need to be told to not download anything!

In GitLab by @samthursfield on Oct 18, 2017, 12:08
mentioned in merge request !115

In GitLab by @samthursfield on Oct 18, 2017, 15:11
mentioned in commit a243a3335b7c113a49b1d5f0daa2c11efe2299bd


Removed PS prompt from Example 1 & updated WhatIf text.

OPS Build status updates of commit 4f7e359:
:clock10: Preparing: average preparing time is 1 min(s) 12 sec(s)

OPS Build status updates of commit 4f7e359:
:clock10: Incremental building: average incremental building time is 12 min(s) 27 sec(s)

OPS Build status updates of commit 4f7e359:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Remove-CsAdditionalInternalDomain.md
:white_check_mark:Succeeded
View (skype-ps)

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