[Feature Request] Send text to current terminal to run

Dear Team,
Integration with Terminal is a killing feature for VSC, and interaction between text editing and terminal would be better.
When working with VSC, I always keep both text editing window and terminal active. For example: connect to remote hive in the terminal meanwhile write some hql script in text window. Or some thing like writing python script meanwhile connect to ipython in terminal. I have to keep coping from text and pasting into terminal all the time
I think send selected text directly to current terminal maybe a better way to achieve REPL and inteactive programing.

Commands are already available for this 😃

Run selected text will run the current line if there is no selection. You can attach custom keybindings to them if you use them frequently.


[do not merge] Std wasm

Just to quickly apply/compare wasm with stdgo'
Should probably make it a makefile variable

applied this actually


Upper bound on ipywidgets causing issues

Hi @davidbrochart, thank you for your extensive work in open-source - your name pops up very often =)
Problem description
I'm a user of https://github.com/pangeo-data/pangeo-docker-images and have traced a regression in a high level user experience of those images. They have xarray_leaflet installed in them, which in turn depends on ipyspin.
The regression observed is that ipyspin is detected by JupyterLab as in need of a rebuild and prompts users about it when their jupyterlab session starts. The reason that this shows now is that ipyspin 0.1.2 has been installed instead of the previosly installed version of 0.1.6. The reason ipyspin got downgraded is because 0.1.2 didn't have an upper bound constraint like ipywidgets<8, but 0.1.5 and 0.1.6 did, and now that  ipywidgets=8 is used by other packages ipyspin got downgraded to a version that didn't make use of the excellent jupyterlab 3 extension packaging.
Suggested resolution
Unless there is a known issue to resolve to use this package with ipywidgets 8, I suggest the upper bound is entirely removed as compared to just bumped to version 9. I suggest that because it my experience so far in the Python ecosystem is that precautionary upper bounds often cause more maintenance effort for maintainers and often cause more problem than benefits for end users.
Cheers from Sweden
/ Erik

This is off topic, but I'd like to advocate that the new release is versioned 1.0.0 rather than 0.1.7 - but its entirely off topic.
I find that many projects would benefit from going to 1.0.0 as soon as they have passed a flurry of initial changes and gained a few users or dependent projects.
I'm influenced greatly by reading on https://semver.org, where they suggest one should quickly arrive at 1.0.0. Here is experpt from the section of How do I know when to release 1.0.0?.

If your software is being used in production, it should probably already be 1.0.0. If you have a stable API on which users have come to depend, you should be 1.0.0. If you’re worrying a lot about backwards compatibility, you should probably already be 1.0.0.

Thanks @consideRatio, I release version 1.0.1, let me know if this solves your issue!

I'm confident it will as soon as its packaged in conda-forge, if not I'll follow up!
Thank you @davidbrochart!!!

Issue confirmed to be resolved, thank you @davidbrochart!

Great, thanks for the feedback!


Private window does not follow Firefox dark mode colours

Description
See the difference in the toolbar/tabsbar and menus

Configuration

Firefox Version: 121
OS: (eg. Windows 11, macOS 14, Linux with GNOME 45, etc.) Fedora 39
Tweaks enabled:

uc.tweak.floating-tabs
uc.tweak.disable-drag-space
uc.tweak.rounded-corners
uc.tweak.show-tab-close-button-on-hover

This should be fixed in the latest release.

Thanks, seems to work!


Add support for group dms

Added group dms to the private message struct. Also added more fields to the ready event and switched the gateway browser to "Firefox". Addresses #4

this seems to do more (like affect the Ready struct?) but otherwise LGTM


Add a command to remove one entry of the logger

So one can remove some useless entry

done here: https://github.com/moosetechnology/Midas/commit/b728bd0cbc5f5d0ec755b2292ef88c47a6e8e678


IntelliJ 2018.3.1 Incompatible with the Intellij version

Hi, same discussion on the new version here:
PhpStorm incompatible with latest version

I performed some exploratory smoke tests for 2018.3.1. The plugin is basically working.

dup of #210


navigation issue about power formula with base contains parentheses

In this page，we can see navigation issue.
In that page, there are two formulas, each have the same navigation issue about power formula with base contains parentheses.
Following is the description of the issue with navigation keys mixed with speech viewer messages.
formula 1
formula 1

down-key
open paren  2           x         close paren squared     space

space-key(following superscript is not appropriate, base is more appropriate)
math
   in superscript         2           x

right-key
in superscript         2

left-key(Note: following base is appropriate)
   in base         2           x

up-key
   out of base         open paren  2           x         close paren squared

formula 2
no indent  formula 2

down-key
     e        raised to the exponent         negative 1 half  times         open paren         the fraction with numerator         x         minus  mu         and denominator sigma         close paren squared         end exponent     space

space-key
math
   in base         e

right-key
   in superscript         negative 1 half  times         open paren         the fraction with numerator         x         minus  mu         and denominator sigma         close paren squared

down-key
   negative 1 half

right-key
   times

right-key
   open paren         the fraction with numerator         x         minus  mu         and denominator sigma         close paren squared

down-key(following superscript is not appropriate as base)
   in superscript         the fraction with numerator         x         minus  mu         and denominator sigma

right-key
   in superscript         2

left-key(following base is appropriate)
   in base         the fraction with numerator         x         minus  mu         and denominator sigma

up-key
   out of base         open paren         the fraction with numerator         x         minus  mu         and denominator sigma         close paren squared

Sorry for taking to long to get to this. I've fixed it now.


fixed canvas issue for web

Does minshell.html need this same change?

Does minshell.html need this same change?

canvas.emscripten { border: 0px none; background-color: black;}

it does not have the width set, so it's already fine

closed as this changes the expected behavior instead of solving the underlying issue, reported to be https://github.com/emscripten-core/emscripten/issues/22847


Extract serializable ModuleMetadata model from ModuleMetadataJsonWriter

Another step toward making GenerateModuleMetadata compatible with the configuration cache.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


Can ResampleAlg be exported?

RasterBand::read_as(..) and the rest take Option<ResampleAlg>. But that enum is public-but-not-exported, so the only choice to the user is None.
Is this by design? I noticed that the tests use crate::raster::rasterband::ResampleAlg; so perhaps the re-export from mod raster was overlooked.
Thanks for your time.

I can't check right now, but it sounds like an oversight. Do you want to submit a PR?

I can -- but off the clock ;)


Write a test for ExerciseSet.Service().hasSet(for:)

Use swift testing to write a test for ExerciseSet.Service().hasSet(for:).

I'd love to tackle this one!

@mikaelacaron you got it!

awh just realized, I didn't get this done in time, unassigning myself, sorry about that! I had trouble with knowing how to do some of the async stuff


Aspect is not completed when using unit %

Steps to repro:
crop: { unit: '%', aspect: 1, width: 100 }

This should complete the crop but retain a square aspect, currently height is just going to 100% even if it doesn't make a square.

Fixed in 8.0.2


Query  not working in v5

//in v4 work fine
var value = data.FindOne(Query.And(Query.EQ("JobId", jobId), Query.EQ("Key", key)));

//in v5 not working ,value is null
var value = data.FindOne(Query.And(Query.EQ("JobId", jobId), Query.EQ("Key", key)));

Hi @akaylh, thanks. There is a bug in this Query.And and I will fix it. Query static methods now can be changed by BsonExpression string constructor:
var value = data.Find("JobId = @0 AND Key = @1", jobId, key)


Most important fixes from the forks

These are some patches from the forks which I think are crucial to integrate into your next version.
There are more interesting feature additions in the forks, but it is is impossible to merge them without getting conflicts.

@cranetm Can you have a look? Thanks.

@cranetm Hi, any chance that these changes can make it into master (if you continue to maintain this repo?)


Include Conda-install package functionality

How would this feature be useful?
Reading the documentation , nox provides automated pip-install packages feature ( that can be defined in the configuration file).
It would be useful to support conda-install as well , to be able to include installing miniconda/anaconda package install .
I would like to work on this as my first contribution , if anyone could help me get started, would be really helpful!

I would find this useful for testing pandas-gbq as well.
Would you create conda envs instead of virtual envs? I recall that using a virtual env inside conda is not supported.

Not sure if I understood your question right , but virtual env can be created using conda, and that should support conda install + pip install within that .

nox is intended to support multiple versions of Python, but within a conda env, you can only have a single Python executable available. This will cause problems when nox creates the virtualenv because it will likely find either no Python for the desired version or a Python that is not managed by conda .(See: https://github.com/theacodes/nox/blob/cede4472651c4902ea66d630bd52345885cd685c/nox/virtualenv.py#L205 for the command that creates the virtualenv)

I'm pretty sure that what is needed to support this use case is to be able to create conda envs instead of virtualenvs, that way conda can be used to pick the python interpreter rather than virtualenv. Alternatively, maybe some logic to detect what Python interpreters are available from conda and switch to the right one before creating the virtualenv.

I would love to see this feature, but I have limited knowledge of Conda. Someone else will need to sketch out what this should look like, but once y'all have an idea I'm happy to review the PR. :)

Thank you @theacodes and @tswast .
Could I get a heads up on what module/functionality should I look into to get started with ?

You'll probably just have a to make a subclass/sisterclass of our
virtualenv module. We already have a similar thing with ProcessEnv vs
VirtualEnv. I'd imagine we'd end up with ProcessEnv, VirtualEnv, and
CondaEnv.
On Fri, May 10, 2019 at 6:42 PM Pragnya Srinivasan<EMAIL_ADDRESS>wrote:

Thank you @theacodes https://github.com/theacodes and @tswast
https://github.com/tswast .
Could I get a heads up on what module/functionality should I look into to
get started with ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/theacodes/nox/issues/183#issuecomment-491469302, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAB5I42WBPKCJGOFK4DQDG3PUYP6XANCNFSM4HMFMVOA
.

One project that has made some traction with tox + conda is the  tox-conda plugin https://github.com/tox-dev/tox-conda.  This may be the sketch you're seeking @theacodes.
A shining feature when using conda over a pip backend is that downloading the scipy stack takes seconds to install instead of minutes.  Example: try installing panda, numpy, scipy, matplotlib,  and notebook using native tox (via pip).  Then try the same using tox-conda by installing the plugin and modifying the .ini file.  The benefits become self-evident.

Thanks for sharing the tox-conda stuff. I'm sure that'll be helpful.
I think we're all aware of the benefits of conda for scientific packages, which is why we're open to this. :)

I created a draft PR to add conda support. https://github.com/theacodes/nox/pull/215
I used the suggested venv_backend option from #199 and added a session.conda_install() function so that pip and conda can be used in the same session when needed.


Fix issue with parsing errors in Mapping agent

LLM was returning Markdown-formatted JSON, but that isn't valid JSON, so it failed to parse.

Out of curiosity (so I can understand why others might do the same) - we create your own mapping/mapping set data classes?

Ok thank you!


changing instance name so that profiler can match string to print out…

… header only once.

https://github.com/bespoke-silicon-group/bsg_manycore/blob/40f737306a263937c77042c328feddd8ec985b55/testbenches/common/v/vcache_profiler.v#L73
vcache_profiler needs to check the instance name so that only the first vcache prints out the header.


x/tools/gopls: fails to load std and cmd workspace packages

Gopls loads std and cmd with the go/packages queries "std/..." and "cmd/...", respectively, resulting in no workspace packages being loaded. As a result, the only packages considered part of the workspace are loaded via open files.
This seems to have been the case for a while, but may have become more noticeable with recent memory optimizations.
This is straightforward to fix, though it makes working in cmd and std a lot more resource intensive: 2.8G vs <1G for me, and find references on testing.T takes a good 5-10s.  I'd still prefer to fix it. If anyone wants the old behavior they can opt in to the degraded memory mode.
CC @stamblerre @mdempsky

How do you opt-in to the previous behavior?

How do you opt-in to the previous behavior?

"memoryMode": "DegradeClosed"
should do it.
We should verify this and move this setting out of experimental before releasing the attached change to std/cmd loading.

Whelp, degraded memory mode doesn't seem to help much, actually. I'll file a separate issue for that.


Remove unused dependencies

Use a tool like depcheck to find and remove all unused dependencies

Resolved in #577


Fix runtime not being python (cont.)

Context
This is a continuation of the work done in #745, which should address #741 by enabling non-Python runtimes to be specified in provider.runtime attribute of the serverless config.
Changes

Fix runtime not being python
Added test
Ignoring test-generated packages.
Adding .python-version file for the required version of Python.
Updating the CONTRIBUTING.md instructions.

Testing

Unit tested

No problem, glad to help! 😃

@pgrzesik Any chance of getting a release out with this fix in the near future? Thanks!

Hey @jameskbride - yes, I intend to prepare a new release later this week, sorry about the delay

No worries, thanks!

Any update on the release?


Implement a dedicated gRPC API for dispatch

Distributed dispatching currently reuses the existing gRPC API and uses headers to propagate context.
It might make sense to design an API dedicated for dispatching that encapsulates this context and is agnostic to the API with which the end-user made a request.

Fixed by #39


i can't use it to preview multi-noise dimensions jsons

.

Hello, would you mind explaining in depthc your issue?

The bug reporter did not respond for 23 days


Use of WithStyles forces "classes" attribute to be present in TypeScript [has workaround]

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When using withStyles to add style to a component, and WithStyles<typeof styles> to describe the props for the component, the user of the component is forced to provide the classes prop otherwise they get a type error.
Expected Behavior 🤔

The user of my component should not be required to provide a classes prop.
Steps to Reproduce 🕹

No repro to share yet, but looks like similar to https://github.com/mui-org/material-ui/issues/10022.
It isn't clear what to do to make classes optional. Seems like having to add a type for it in my props definition is too complicated (how am I supposed to know how to type it? I want to leave that to material-ui)
Unless I missed it, the docs don't explain this part.
Your Environment 🌎

Tech
Version

Material-UI
v3.5.1

React

Browser

TypeScript

etc.

Sorry is there an answer for this yet? How can we prevent every component wrapped with withStyles ( and using WithStyles) from giving compiler warnings complaining about "Element xxx doesn't have required attribute classes" ?
This is clearly not intended behavior yet its been tagged working as intended? Is it intended that the IDE is flooded with irrelevant warnings? Surely not.
The workaround I have been using is either ignoring the warnings or adding classes={{} as any} to every element in question, but both of those seem a little lame.
Plus I've seen some fairly complex solutions which seem to involve wrapping everything in a HOC with some elaborate Partial business, but that looks like overkill.
I've read https://material-ui.com/guides/typescript/ probably a dozen times and implemented it, but that doc is almost conversational rather than prescriptive, and does not mention this issue.

@moclei I agree the label was not helpful since I had to made some assumption about the used code. I assumed the component with WithStyles props was not wrapped with withStyles. Obviously the classes props would be required in that case.
However this is an assumption on my part and without seeing the code incomplete is more appropriate.
We can link from the typescript guide to one of our demos in typescript which illustrates actual usage if that is your issue. For example https://material-ui.com/demos/app-bar/#app-bar-with-buttons (click on "code open" and switch to "TS").
If you think you found a bug please open a separate issue and follow the issue template.

@eps1lon That is a great idea, and in fact it helped me solve this issue within minutes.

What solved the problem for me, from reading the typescript examples you linked to, was including the PropTypes definition:
CustomElement.propTypes = {     classes: PropTypes.object.isRequired, } as any;
I had removed that on advice of another React Typescript example from elsewhere. Below is the code showing everything.
`import * as React from "react";
import PropTypes from 'prop-types';
import withStyles, { WithStyles} from "@material-ui/core/styles/withStyles";
import {createStyles} from "@material-ui/core";
const styles = () => createStyles({
round: {
borderRadius: "30px"
},
block: {
width: "100% !important"
},
});
interface Props extends WithStyles {
sampleProp?: string;
}
const CustomElement = withStyles(styles)(
class extends React.Component {
static defaultProps  = {
sampleProp: 'Some text'
};
constructor(props: Props) {
super(props);
}
render() {
const {
classes,
sampleProp
} = this.props;
return (

{sampleProp}

);
}
});
CustomElement.propTypes = {
classes: PropTypes.object.isRequired,
} as any;
export default CustomElement;`
When this CustomElement is used in another component, the compiler no longer gives warnings "Element xxx doesn't have required attribute classes"
In retrospect I should have known about the Typescript examples, but linking them to the Typescript section might save a few people like me from similar frustration. Thanks again @eps1lon

propTypes can get in the way with TS, I agree.
However, this should trigger a runtime warning because you shouldn't be required to pass classes. The classes are not required on the CustomElement but on the anonymous class passed to withStyles. withStyles should inject them into the anonymous component. This is what the propTypes should check.

@eps1lon I've been stuck on this issue all day and I see it's been opened and closed a few times. Nothing I have tried worked yet.
Should I open one and maybe we can close it with a real answer and update the docs properly? Or should I provide some code and context here? What do you think?

I faced  this issue running this code:

import React from "react";
import { withStyles, createStyles, WithStyles, Theme } from "@material-ui/core";

const styles = (theme: Theme) =>
  createStyles({
    scrollableContainer: {
      background: (props: Props) => (props.example ? "black" : "blue"),
    },
  });

interface Props extends WithStyles<typeof styles> {
  example: boolean;
}

const Demo = withStyles(styles)(
  class extends React.PureComponent<Props> {
    render() {
      return (

dadadadada

      );
    }
  }
);

export default Demo;

See how |styles| and |Props| make a circular dependency. The error message is confusing since the problem is irrelevant to |classes| field.

In my case the solution is changing  background: (props: Props) to  background: (props: {example:boolean})


[Release 2.2.1]Add security

Signed-off-by: prudhvigodithi<EMAIL_ADDRESS>Description
[Release 2.2.1]Add security
Issues Resolved
Part of: https://github.com/opensearch-project/opensearch-build/issues/2481
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2494 (1598c89) into main (8b7bcbc) will increase coverage by 5.52%.
The diff coverage is n/a.

:exclamation: Current head 1598c89 differs from pull request most recent head d3c14a6. Consider uploading reports for the commit d3c14a6 to get more accurate results

@@              Coverage Diff              @@
##               main     #2494      +/-   ##
=============================================
+ Coverage     94.47%   100.00%   +5.52%
=============================================
  Files           218         6     -212
  Lines          4449       105    -4344
  Branches         29        19      -10
=============================================
- Hits           4203       105    -4098
+ Misses          240         0     -240
+ Partials          6         0       -6

Impacted Files
Coverage Δ

...jobs/PromoteArtifactsQualifier_actions_Jenkinsfile

src/ci_workflow/ci_args.py

src/jenkins/InputManifest.groovy

src/sign_workflow/signer.py

...jenkins/jobs/CreateGithubIssueExisting_Jenkinsfile

src/ci_workflow/ci_check_list_dist.py

...workflow/ci_check_gradle_publish_to_maven_local.py

tests/jenkins/jobs/CreateGithubIssue_Jenkinsfile

.../jenkins/jobs/PromoteArtifacts_actions_Jenkinsfile

...rc/test_workflow/test_result/test_suite_results.py

... and 202 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Debugger does not stop after May 19 Release

After the last BC update (14.0.29537.29654) I can not debugg any extension. Has it happened to someone the same? I have been searching for and I have not found anything.
VSCode version: 1.34.0
VSIX: Al Languaje 3.0.126610
I am developing directly in Sandbox and a second Sandbox.
Thank you.

Same thing happens to me. Debugging works in local container but not in cloud sandbox.

Hi! Are you trying to debug your extension or 1st party extensions/the base application?

I have gotten base application errors that should have been caught by the debugger. I have also placed breakpoints in my extension code that were ignored.

I Don´t understand you. What I'm doing is trying to debug my extension publishing it from VS Code.


Top menu bar disappears after couple of sliding

Here is the video (animated gif):

Thank you for finding the issue @maysamsh .
I want to know your spec about PageController. Now lastest version is 0.0.4.
Can you try?

I have tried it but it has the same problem unfortunately.

v.0.0.6 has the same problem.

Thanks @maysamsh .
I try to do this issue. If you find a point about this issue, please show me.
Thanks again.

I did try to do this issue, but I could not reproduce this issue.

Seems it has a problem with specific data (which is "cope" here). I've used another db and works fine yet.

This is the test case: https://dl.dropboxusercontent.com/u/67205192/sample.zip
Go forward to the "ascetic" word then move back.

@maysamsh , Thanks for uploading zip.
I found bugs about this issue.
I will try to bugfix it.
Thanks again.

I fixed by https://github.com/hirohisa/PageController/pull/5 .
My friend(Software Quality Assurance Engineer) did try to test.


Proper support for ERROR dispatch

Detailed Description

Context

See #488
Possible Implementation

Your Environment

Version used:
Link to your project:

Closing this until someone requests it


ja: Make docs/tasks/administer-cluster/enabling-endpointslices.md follow v1.18 of the original text

This is a Feature Request

What would you like to be added
Update docs/tasks/administer-cluster/enabling-endpointslices.md to follow v1.18 of the original (English) text.
Why is this needed
content/ja/docs/tasks/administer-cluster/enabling-endpointslices.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.18-ja.1/content/ja/docs/tasks/administer-cluster/enabling-endpointslices.md
Original
https://github.com/kubernetes/website/blob/fb6364d/content/en/docs/tasks/administer-cluster/enabling-endpointslices.md
diff between translated and v1.18
https://gist.github.com/0b00da20f92061e907d681a30f0880f4
Note
Currently, we use dev-1.18-ja.1 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel

/assign

/close


[PT Run] Add an opacity setting for the search box

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
---I would like to add an opacity change option to the function of 'PowerToys Run' so that users can search documents while looking at them.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

Neat idea.

Already tracked in https://github.com/microsoft/PowerToys/issues/4794


Adding a new chart to raw.density from the d3 Gallery

Hi guys I'm new to Raw and Git Hub. There seems to be a provision to add a chart to raw, however, when you click on Add chart it takes you to Git Hub but where how you add the chart is so far a mystery. Can anyone help?? (I'm trying to create a graphic from a sample of excel data).

Sounds like a question for RAW, not D3.


[WIP] refactor codebase with functor APIs

I found filter specification greatly reduces the complexity of code structure and improves the style consistency when I played with ImageQualityIndexes and ImageNoise. Check my GSoC 2019 blog The principles of Images.jl: Part I as an example and illustration. Also, see Proposal: add abstract types ImageAlgorithm and ImageFilter for a unified API that enables extensibility without loss of API consistency.
To make this PR easy to review, the first eight commits serve as a start point, future commits are submitted in the form of PRs.
TODO:

[ ] move implementations into concrete functors, swap arguments, and refactor docstring (closes #23 )

[x] AdaptiveThreshold PR: https://github.com/zygmuntszpak/ImageBinarization.jl/pull/30
[x] Balanced PR: #32
[x] Entropy PR: #33
[x] Intermodes PR: #34
[x] MinimumError PR: #35
[x] MinimumIntermodes PR: #36
[x] Moments PR: #37
[ ] Niblack PR: #42
[x] Otsu PR: #38
[ ] Polysegment
[ ] Sauvola
[x] UnimodalRosin PR: #39
[x] Yen PR: #40

[ ] update documentation and README (closes #25)

Codecov Report

Merging #29 into master will decrease coverage by 23.92%.
The diff coverage is 95.95%.

@@             Coverage Diff             @@
##           master      #29       +/-   ##
===========================================
- Coverage   94.47%   70.55%   -23.93%
===========================================
  Files          16       20        +4
  Lines         163      180       +17
===========================================
- Hits          154      127       -27
- Misses          9       53       +44

Impacted Files
Coverage Δ

src/sauvola.jl
0% <ø> (-93.75%)
:arrow_down:

src/niblack.jl
0% <ø> (-93.75%)
:arrow_down:

src/ImageBinarization.jl
100% <ø> (ø)
:arrow_up:

src/polysegment.jl
0% <ø> (-91.67%)
:arrow_down:

src/minimum_error.jl
100% <100%> (ø)
:arrow_up:

src/intermodes.jl
100% <100%> (ø)
:arrow_up:

src/otsu.jl
100% <100%> (ø)
:arrow_up:

src/moments.jl
100% <100%> (ø)
:arrow_up:

src/BinarizationAPI/binarize.jl
100% <100%> (ø)

src/unimodal.jl
100% <100%> (ø)
:arrow_up:

... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ff0e10...88202fe. Read the comment docs.

Codecov Report

Merging #29 into master will decrease coverage by 23.92%.
The diff coverage is 95.95%.

@@             Coverage Diff             @@
##           master      #29       +/-   ##
===========================================
- Coverage   94.47%   70.55%   -23.93%
===========================================
  Files          16       20        +4
  Lines         163      180       +17
===========================================
- Hits          154      127       -27
- Misses          9       53       +44

Impacted Files
Coverage Δ

src/sauvola.jl
0% <ø> (-93.75%)
:arrow_down:

src/niblack.jl
0% <ø> (-93.75%)
:arrow_down:

src/ImageBinarization.jl
100% <ø> (ø)
:arrow_up:

src/polysegment.jl
0% <ø> (-91.67%)
:arrow_down:

src/minimum_error.jl
100% <100%> (ø)
:arrow_up:

src/intermodes.jl
100% <100%> (ø)
:arrow_up:

src/otsu.jl
100% <100%> (ø)
:arrow_up:

src/moments.jl
100% <100%> (ø)
:arrow_up:

src/BinarizationAPI/binarize.jl
100% <100%> (ø)

src/unimodal.jl
100% <100%> (ø)
:arrow_up:

... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ff0e10...88202fe. Read the comment docs.

To summary up, this PR does:

refactor the codebase using the functor API discussed in #26
add in-place function binarize!
support Color3 inputs
add more test codes
slightly enhance the documentation

Breaking changes (Deprecated in 0.3):

swap the argument order #23 ( https://github.com/zygmuntszpak/ImageBinarization.jl/pull/29/commits/d1f8309ef5fa35f46f52aa748e9de0307edd8fc6)
unexport recommend_size in favor of #41, i.e., AdaptiveThreshold(img) instead of recommend_size(img) ( PR: #30 #45)
made window_size of AdaptiveThreshold not an optional argument. ( PR: #45 )


Automate tests

Just to be sure I understand, is it OK if I add tests with Jest and create automation using a GitHub workflow?


Client project name unique

Notion card
Summary

Preview
https://www.loom.com/share/912ab5787fcd4e70b388a9e4271e6b32
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[x] I have manually tested all workflows
[ ] I have performed a self-review of my own code
[x] I have added automated tests for my code

@apoorv-mishra I'm assigning it to you for a review.

Closing this for now . Will see if its required later


Generation of terminal codes is broken

Terminal codes listed in generator.yaml files for RDS is not properly being listed in the output of the terminalAWSError() resource manager function. Not sure when this stopped working.

Never mind... I was wrong. Terminal code generation is working properly.


Codespace: keep getting notifications of enableing widget downloads

Environment data

VS Code version: 1.60-Insiders
Jupyter Extension version (available under the Extensions sidebar): 2021.8.2027072902

I kept getting this notification even if I click "Enable" downloads
Widgets require us to download supporting files from a 3rd party website. (Error loading keplergl-jupyter:^0.3.0).

It works after reloading the whole workspace, however the notification only asks me to restart the kernel.


Additional info message while waiting for IP address to be assigned to the Droplet.

It takes a lot of time to assign the IP to droplet, so informing user
about this hang looks like a good idea.

LGTM
cc @nathanleclaire

I guess it's OK.  In general drivers aren't really supposed to be doing this sort of IP polling on their own though.


Remove deprecated plugins

These two plugins were deprecated and consolidated into stylelint-order a long time ago.

Thanks


Converting a nested Archetype Model

Here is the sample code I'm working on.
private IEnumerable<ConvertToModel> ConvertFromArchetype(ArchetypeModel value)
{
    var converter = new AnotherConverter();
    List<ConvertToModel> model = new List<ConvertToModel>();

    value.ForEach(x => model.Add(new ConvertToModel()
    {
        Value = x.GetValue<string>("Title"),
        AnotherValue = (AnotherType)converter.ConvertFrom(x.GetValue("ValueToConvert")))
    }));

    return model;
}

The problem with nested Archetype Models is when you try to get the value ValueToConvert it gives you json style string instead of the actual ArchetypeModel.
The question is: it is possible to do this without creating another class to deserialize the json string?

Hi @ordepdev,
Within Archetype it internally makes a call to Umbraco's TryConvertTo<T> method, so you could try...
AnotherValue = x.GetValue<AnotherType>("ValueToConvert")

But yes, I believe you'd still need another TypeConverter to convert from an ArchetypeModel to whatever AnotherType would be.
Hope this helps?

So, I already have the second TypeConverter to convert the type of ValueToConvert to AnotherType and the property AnotherValue is decorated with [TypeConverter(typeof(AnotherConverter))].
Meanwhile, AnotherValue = x.GetValue<AnotherType>("ValueToConvert") does't hit the AnotherConverter code and returns null.

The TypeConverter attribute needs to be on the AnotherType class itself, (so that .NET's TypeDescriptor can discover it - it doesn't look for the attribute on properties).

Oh right! Anyway, I was using GetValue<IEnumerable<AnotherType>> so... no hit.
Thanks Lee :+1:

If you want to register a TypeConverter for a generic type, you could add this to your app start-up...
TypeDescriptor.AddAttributes(typeof(IEnumerable<AnotherType>), new TypeConverterAttribute(typeof(AnotherTypeConverter)));


Validate payload pass to the api methods

@alavers do you think it would be a good idea to validate right away the format of the payload passed? Unless we think it might be a problem if we updating it on the remote server. My main concern would be having a wrong validation client side and we can't change it unless the client is updated.
But then... we get a bug report and we fix it.

TBD


Double function of mouse wheel

When scrolling a drop down list with the mouse wheel, the flow in background zooms too.

Confirmed. I will try to see if I can change this.

changed in version <IP_ADDRESS>


start-stop-daemon: give user an error if calling --start with --retry or --stop with --wait

Found while trying to figure out why a long running, slow to start up java webapp was always showing as 'crashed' when trying to shut it down

Changed to use a warn rather than an error


Rotation irrelevant when creating than one SpacePin?

Just a quick question for verification:
We noticed we were feeding correct positions but completely wrong rotations when creating new SpacePinPackages.
(Positions coming from scanned qr codes.)
But the positioning of the holograms still worked well.
Does that mean that the rotations of the virtualObject passed to SpacePinPackage.Create() don't matter anymore once there's more than one SpacePinPackage?
That would make sense because rotation inferred from two points will be more accurate than rotation from a scanned QR code.
Also it normally takes 3 points to define the rotation of a coordinate system. But I assume you only need two points because you always keep the Y-axis vertical?

You are correct on both. This is if you are using SpacePinOrientable (recommended).
If you have only submitted one SpacePinOrientable with SetFrozenPose(), the rotation submitted is used.
With two or more SpacePinOrientables, and an Orienter, the submitted rotation is ignored and the rotation about the Y axis implied by the relative positions is used. This assumes the you want your gravity vector (Y-axis) preserved.
With three or more SpacePinOrientables, you can fully define an arbitrary rotation. To do this, you need to use an OrienterThreeBody, instead of the base Orienter.

Ok thanks. Yes in most practical applications it will be desirable do put the Y-axis on the gravity vector.


feat:add more filter replace

新功能

添加了部分更加简洁的元数据名称，如media_file_name 可以替换 为file_name, 原有media_file_name 仍然可以使用，更多请查看 如何使用过滤器
添加了更加简洁的数据表达形式，如 file_size > 1 * 1024 * 1024 可以替换为 file_size > 1MB 更多请查看 如何使用过滤器

fix

修复了表达式无法使用中文的问题
修复了配置无法使用中文问题
修复了配置中file_name_prefix_split配置file_name不为最后一个导致文件无法访问问题
修复字符串匹配失败问题，如 file_name == r'.*test.*'

feat

Added some more concise metadata names, such as media_file_name can be replaced by file_name, the original media_file_name can still be used, for more information, please refer to How to use filters
Added a more concise data expression form, such as file_size > 1 * 1024 * 1024 can be replaced by file_size > 1MB For more information, please see How to use filters

fix

Fixed the problem that expressions cannot use Chinese
Fixed the problem that the configuration cannot use Chinese
Fixed file_name_prefix_split configuration file_name not being the last one in the configuration, causing the file to be inaccessible

Fix string matching failure problem, such as file_name == r'.*test.*'

Codecov Report

Merging #5 (caef255) into master (0c5a3bc) will decrease coverage by 2.52%.
The diff coverage is 81.08%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
- Coverage   99.19%   96.68%   -2.52%
==========================================
  Files           9        9
  Lines         373      392      +19
==========================================
+ Hits          370      379       +9
- Misses          3       13      +10

Impacted Files
Coverage Δ

media_downloader.py
94.71% <70.00%> (-3.81%)
:arrow_down:

utils/format.py
98.76% <93.33%> (-1.24%)
:arrow_down:

utils/meta_data.py
100.00% <100.00%> (ø)

utils/platform.py
80.00% <0.00%> (-20.00%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


ignore skip_update fields on #[derive(AsChangeset)] (#860)

Implement skip_update attribute for #[derive(AsChangeset)] as described here: https://github.com/diesel-rs/diesel/issues/860#issuecomment-2114074186.

@weiznich, I'm wondering if skip_update should be a separate attribute, or if it should be rolled into skip_insertion and renamed?
If the purpose of both attributes is to skip the corresponding operation on a generated column, is there ever a case where we would want to skip_insertion but not skip_update, or vice versa? Perhaps there is a scenario in which this is valid of which I am not considering, but if not, I do think this makes for a more error-prone implementation, by requiring these annotations be needlessly duplicated for what should be consistent behavior between inserts and updates. On the other hand, should someone require differing behavior for their particular use case, then we already get that functionality here.


Checkout fails if PROJECT is not writeable

While trying to repro the error reported here:
https://forum.access-hive.org.au/t/extending-access-om2-025-ryf-run-present-on-ik11/3689/11
payu errored with the checkout step because project is set in the config.yaml
https://github.com/rmholmes/025deg_jra55_ryf/blob/a29b0e442a791ef8ec04796fb6d3943aa3fc748f/config.yaml#L4
and the laboratory path is also set to an inaccessible project
https://github.com/rmholmes/025deg_jra55_ryf/blob/a29b0e442a791ef8ec04796fb6d3943aa3fc748f/config.yaml#L11
So when trying to checkout it throws this error:
$ payu checkout -r /g/data/ik11/outputs/access-om2-025/025deg_jra55_ryf9091_gadi/restart250/ -b new_branch_nam
e a29b0e4
Created and checked out new branch: new_branch_name
laboratory path:  /scratch/e14/XXXXXX/access-om2
binary path:  /scratch/e14/XXXXXX/access-om2/bin
input path:  /scratch/e14/XXXXXX/access-om2/input
work path:  /scratch/e14/XXXXXX/access-om2/work
archive path:  /scratch/e14/XXXXXX/access-om2/archive
Updated metadata. Experiment UUID: 8e8fd75b-2a4f-4896-8fc2-702236bb4bad
Traceback (most recent call last):
  File "/g/data/vk83/prerelease/apps/payu/dev/bin/payu", line 8, in <module>
    sys.exit(parse())
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/site-packages/payu/cli.py", line 49, in parse
    run_cmd(**args)
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/site-packages/payu/subcommands/checkout_cmd.py", line 34, in runcmd
    checkout_branch(is_new_branch=new_branch,
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/site-packages/payu/branch.py", line 192, in checkout_branch
    metadata.write_metadata(set_template_values=True,
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/site-packages/payu/metadata.py", line 266, in write_metadata
    self.copy_to_archive()
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/site-packages/payu/metadata.py", line 347, in copy_to_archive
    mkdir_p(self.archive_path)
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/site-packages/payu/fsops.py", line 41, in mkdir_p
    os.makedirs(path)
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/os.py", line 215, in makedirs
    makedirs(head, exist_ok=exist_ok)
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/os.py", line 215, in makedirs
    makedirs(head, exist_ok=exist_ok)
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/os.py", line 215, in makedirs
    makedirs(head, exist_ok=exist_ok)
  [Previous line repeated 1 more time]
  File "/g/data/vk83/prerelease/apps/payu/dev/lib/python3.10/os.py", line 225, in makedirs
    mkdir(name, mode)

payu checkout should either check the user is a member of the project, or that the laboratory path is writeable.
It should probably fail with a useful error message, but also with instructions to recover from this error.
In this case it was sufficient to remove the project and laboratory settings in config.yaml and then run payu setup.
It errored on an inaccessible input path, but that is not related to the error above.
To repro:
git clone https://github.com/rmholmes/025deg_jra55_ryf my_new_example
cd my_new_example/
payu checkout -r /g/data/ik11/outputs/access-om2-025/025deg_jra55_ryf9091_gadi/restart250/ -b new_branch_name a29b0e4
git clone https://github.com/rmholmes/025deg_jra55_ryf my_new_example
cd my_new_example/
payu checkout -b new_branch_name ryf9091_gadi -r /g/data/ik11/outputs/access-om2-025/025deg_jra55_ryf9091_gadi/restart250/

So payu uses the existence of archives to determine whether it should generate a new experiment UUID. So if the payu checkout exits with an error, and then project is fixed and a payu setup is run, it won't find any pre-existing archive directories so it'll assume it's a new experiment and generate a new experiment UUID and commit. I wonder if payu should probably prevent the commit from payu checkout if unable to create an archive as a new UUID will be generated at next payu setup anyway.
A fix I am working on so far, during checkout (after the git checkout has run), attempts to run laboratory.initialise(). If permission errors, log out a message and raise an error. (Could also run os.access(laboratory.basepath, os.W_OK | os.X_OK) to check for write permissions - https://stackoverflow.com/a/2113511). Then the log message could be something like:
Skipping creating metadata, restart configuration and setting up archive and work symlinks.
To fix the errors, edit one (or more) of the following config.yaml options that determine the laboratory path:
    - 'laboratory': (Top-level directory for the model laboratory - Default: /scratch/${PROJECT}/${USER}/${MODEL})
    - 'shortpath' (Top-level directory for laboratory - Default: /scratch/${PROJECT})
    - 'project': (The project to use for payu PBS jobs - Default: ${PROJECT})

Then either run:
- 'payu setup' or `payu checkout <CURRENT_BRANCH>` - if no experiment archive directory exists, it'll generate a new UUID.
- Or if starting from a restart, use  `payu checkout <CURRENT_BRANCH> -r <RESTART_PATH>`.
   As well as generating metadata and setting up symlinks, this will add a restart path to config.yaml and
   add parent experiment UUID to metadata file (if it exists).

I remember in the payu training session recently, people would checkout a branch of released configuration, then change config.yaml to a project that wasn't the default project in config.yaml, and then there were two commits for new UUIDs. Maybe project should be a command line option to clone and checkout commands? Though there is also shortpath and laboratory config options which both override project. So if project is defined would have to remove shortpath and laboratory from config.yaml.. (There's an existing a --laboratory/--lab/-l command-line option you can pass to payu clone/payu checkout which is only used to initialise the laboratory object during checkout. The configuration will still need to be changed before running payu commands or passing the laboratory to subsequent payu commands, e.g. payu setup --lab <lab_path>. So it's not exactly ideal but it is an option)


correct the class attributes name for mixed precision training

What does this PR do?
correct the variables/class attribute name for mixed precision training
in all the config files, the related hyperparameter is named 'amp', but use_amp is used while assigning values to class attributes, which needs to be fixed, otherwise, this hyperparameter in the config file will never be used and cannot control the mixed precision training stuff

@microsoft-github-policy-service agree


Publish per Tenant App

Hi Kine,
thanks for the tools.
I'm trying to setup the pipelines to publish App to online sandbox. I received below error in the pipeline and the App is successfully published. Any idea why ?
Thanks.
020-04-23T05:46:24.6853430Z ##[debug]Caught exception from task script.
2020-04-23T05:46:24.6885409Z ##[debug]Error record:
2020-04-23T05:46:24.8596776Z ##[debug]Get-ALAppPublicationStatus : Cannot index into a null array.
2020-04-23T05:46:24.8610876Z ##[debug]At C:\Users\xxxx\Documents\WindowsPowerShell\Modules\NVRAppDevOps\1.0.10\Upload-PerTenantApp.ps1:35 char:23
2020-04-23T05:46:24.8624420Z ##[debug]+ ...   $Status = Get-ALAppPublicationStatus -AppId $AppId -AppSecret $AppS ...
2020-04-23T05:46:24.8638220Z ##[debug]+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2020-04-23T05:46:24.8659418Z ##[debug]    + CategoryInfo          : InvalidOperation: (:) [Get-ALAppPublicationStatus], RuntimeException
2020-04-23T05:46:24.8673187Z ##[debug]    + FullyQualifiedErrorId : NullArray,Get-ALAppPublicationStatus
2020-04-23T05:46:24.8687722Z ##[debug]
2020-04-23T05:46:24.8713010Z ##[debug]Script stack trace:
2020-04-23T05:46:24.8746510Z ##[debug]at Get-ALAppPublicationStatus, C:\Users\xx\Documents\WindowsPowerShell\Modules\NVRAppDevOps\1.0.10\Get-ALAppPublicationStatus.ps1: line 35
2020-04-23T05:46:24.8759320Z ##[debug]at Upload-PerTenantApp, C:\Users\xxx\Documents\WindowsPowerShell\Modules\NVRAppDevOps\1.0.10\Upload-PerTenantApp.ps1: line 35
2020-04-23T05:46:24.8774389Z ##[debug]at , C:\Users\xxx\Downloads\vsts-agent-win-x64-2.165.2_work_tasks\publishbcapppertenant_047393e1-82e4-4d24-8275-237409bb378e\1.0.3\publishtenanttask.ps1: line 27
2020-04-23T05:46:24.8787079Z ##[debug]at , : line 1
2020-04-23T05:46:24.8801035Z ##[debug]at , : line 22
2020-04-23T05:46:24.8821032Z ##[debug]at , : line 18
2020-04-23T05:46:24.8835506Z ##[debug]at , : line 1
2020-04-23T05:46:24.8861615Z ##[debug]Exception:
2020-04-23T05:46:24.8914370Z ##[debug]System.Management.Automation.RuntimeException: Cannot index into a null array.
2020-04-23T05:46:24.8926562Z ##[debug]   at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
2020-04-23T05:46:24.8943100Z ##[debug]   at System.Management.Automation.Interpreter.ActionCallInstruction2.Run(InterpretedFrame frame) 2020-04-23T05:46:24.8958536Z ##[debug]   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame) 2020-04-23T05:46:24.8974329Z ##[debug]   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame) 2020-04-23T05:46:24.8991809Z ##[debug]   at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame) 2020-04-23T05:46:24.9006563Z ##[debug]   at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0) 2020-04-23T05:46:24.9018984Z ##[debug]   at System.Management.Automation.PSScriptCmdlet.RunClause(Action1 clause, Object dollarUnderbar, Object inputToProcess)
2020-04-23T05:46:24.9033338Z ##[debug]   at System.Management.Automation.PSScriptCmdlet.DoEndProcessing()
2020-04-23T05:46:24.9045536Z ##[debug]   at System.Management.Automation.CommandProcessorBase.Complete()
2020-04-23T05:46:24.9331913Z ##[error]Cannot index into a null array.
2020-04-23T05:46:24.9342420Z ##[debug]Processed: ##vso[task.logissue type=error]Cannot index into a null array.
2020-04-23T05:46:24.9344123Z ##[debug]Processed: ##vso[task.complete result=Failed]

Hello @antonyotto ,
the reason is, that there is call reading companies from the environment and it has returned no companies from some reason. Try to run this manually:
$Token = Get-OAuth2 -AppId $AppId -AppSecret $AppSecret -Credentials $Credentials -Tenant $Tenant
Get-BCAPIData -OAuthToken $Token -Tenant $Tenant -APIUri $APIUri -Query 'companies' -Environment $Environment -APIVersion $APIVersion

It should return at least one company.

Hi @antonyotto,
have re-read the log you sent and it seems that the problem is, that the call reading the status have not returned anything. It could be some timing issue or I do not know. Will try on my side. Have it worked before? Are you getting this each time or when you re-run the stage it goes trough?

Hmm... unable to test on my side, because we have activated MFA on our users accounts to fulfill MS requirements and thus I am unable to publish the apps from pipeline (no support for service accounts yet and the current way doesn't support MFA). This is rendering the Password Grant OAuth unusable (but of course, it was never recommended to use this grant type, but it was only possible way).

Hi Kine,
Thanks for the reply, this is the first time I used it and I tried few times but the result is the same
I will try to run it directly from Powershell and see if it work.
Thanks,

I just try to run manually from powershell and it's working fine.
Upload-PerTenantApp -AppId [xx] -AppSecret [xx] -Tenant [xx] -AppPath "C:\artifact\Default publisher_TestBuild_<IP_ADDRESS>.app" -Environment Sandbox -APIVersion v1.0 -Credential $mycreds -WaitForResult
Getting OAuth token...
Getting companies...
Getting uploading extension to company 83b8f35c-fb52-ea11-bb59-00155df3a615 (CRONUS NZ)...
Getting last publishing status from company 83b8f35c-fb52-ea11-bb59-00155df3a615...
Last status: TestBuild Default publisher <IP_ADDRESS> InProgress
Getting last publishing status from company 83b8f35c-fb52-ea11-bb59-00155df3a615...
Last status: TestBuild Default publisher <IP_ADDRESS> InProgress
Getting last publishing status from company 83b8f35c-fb52-ea11-bb59-00155df3a615...
Last status: TestBuild Default publisher <IP_ADDRESS> InProgress
Getting last publishing status from company 83b8f35c-fb52-ea11-bb59-00155df3a615...
Last status: TestBuild Default publisher <IP_ADDRESS> Completed

Ok. If it works manually and not in pipeline, than double check that all parameters in the pipelines are passed correctly. Are you using your own agent or hosted?

I'm using my own agent. it's the same server that I used to run the powershell.
I also try to run the powershell in the pipeline and it works
Sent from Outlook Mobilehttps://aka.ms/blhgte

From: Kamil Sacek<EMAIL_ADDRESS>Sent: Friday, April 24, 2020 10:41:20 PM
To: kine/MSDYN365BC_Yaml<EMAIL_ADDRESS>Cc: antonyotto<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [kine/MSDYN365BC_Yaml] Publish per Tenant App (#8)
Ok. If it works manually and not in pipeline, than double check that all parameters in the pipelines are passed correctly. Are you using your own agent or hosted?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/kine/MSDYN365BC_Yaml/issues/8#issuecomment-618937643, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJKMR5B2TTHXNUY6A4FKJC3ROFUFBANCNFSM4MOYRYSA.

Ok, than the problem must be in the parameters of the step. Could you send me log from the pipeline? I want to see the xxxx-expanded.yaml file from the full logs.

Sorry. I have remove the pipeline. I will re-create it and send you the log. thanks


chore: change readme updater to update the branch instead of detached…

… head
Which problem does the PR fix?

What's in this PR?

Checklist
Please make sure to follow our Contributing Guide.

Before opening the PR:

[ ] In the repo's root dir, run make go.update-golden-only.
[ ] There is no other open pull request for the same update/change.
[ ] Tests for charts are added (if needed).
[ ] In-repo documentation are updated (if needed).

After opening the PR:

[ ] Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
[ ] Did all checks/tests pass in the PR?

attempt failed.


Panic when using poll_once (embassy-futures)

I'm using ESP32 no_std with embassy.
I'm trying to poll on an async function, meaning, I want to check if a result is available w/o waiting for the result.
I wrote, so did the following:
let touch_future = touch.async_event();

let touch_poll = poll_once(touch_future);

When I run this, at the poll_once call I get the panic below.
I see in the panic reference to timer. Inside the async function it may be waiting on a future (irq) using with_timeout (from embassy-time) which is also embassy feature . Maybe that's related to the panic.

Am I doing something wrong? Or is it a bug? Or maybe not supported on ESP32?

( as a side note - I could achieve something similar by using select with a timer of 0 but it seems to me this costs more in speed, because it has to set up a time, etc. )
PanicInfo {
    payload: Any { .. },
    message: Some(
        Found waker not created by the Embassy executor. `embassy_time::Timer` only works with the Embassy executor.,
    ),
    location: Location {
        file: "/Users/yanivshaya/.cargo/registry/src/index.crates.io-6f17d22bba15001f/embassy-executor-0.3.3/src/raw/waker.rs",
        line: 42,
        col: 9,
    },
    can_unwind: true,
}

Backtrace:

0x40121401
0x40121401 - _embassy_time_schedule_wake
    at ??:??
0x40121851
0x40121851 - _ZN75_$LT$embassy_time..timer..Timer$u20$as$u20$core..future..future..Future$GT$4poll17hd8a631027057219fE
    at ??:??
0x400f2b6a
0x400f2b6a - _ZN98_$LT$futures_util..future..select..Select$LT$A$C$B$GT$$u20$as$u20$core..future..future..Future$GT$4poll17he262a36e387db182E
    at ??:??
0x400f61ac
0x400f61ac - _ZN13xpt2046_esp3224XPT2046$LT$SPI$C$IRQ$GT$11async_event28_$u7b$$u7b$closure$u7d$$u7d$17hc3aaa1af1f07c719E.llvm.16976254322227335400
    at ??:??
0x400eed94
0x400eed94 - _ZN16embassy_executor3raw20TaskStorage$LT$F$GT$4poll17h275433813c7836c0E.llvm.5053872814749477446
    at ??:??
0x401212f6
0x401212f6 - _ZN16embassy_executor3raw8Executor4poll17h18b75c7aad325f33E
    at ??:??
0x400ef99c
0x400ef99c - _ZN14esp_hal_common7embassy8executor6xtensa6thread8Executor3run17hb9555bf349f28770E
    at ??:??
0x400f1ee2
0x400f1ee2 - main
    at ??:??
0x4012150d
0x4012150d - Reset
    at ??:??
0x4011f318
0x4011f318 - ESP32Reset
    at ??:??

where is poll_once from? My guess is it's creating its own "no-op" waker.
Using the integrated-timers timer queue requires all wakers are created by the executor. As an alternative, you can disable feature integrated-timers in embassy-executor and enable feature generic-queue in embassy-time, to enable an executor-independent timer queue. That one works with any waker.
(in general using "manual poll" like poll_once is not recommended though, you might want to rewrite your code using select/join if you want to wait for multiple things. It's much less error prone)

where is poll_once from? My guess is it's creating its own "no-op" waker.

It's coming from embassy_futures. Isn't that aligned with embassy-executor?
The code there points to https://docs.rs/embassy-futures/latest/src/embassy_futures/block_on.rs.html#36-45

(in general using "manual poll" like poll_once is not recommended though, you might want to rewrite your code using select/join if you want to wait for multiple things. It's much less error prone)

What I need to accomplish is, for some time, to call in a loop some function from a UI framework I'm using (to drive animation processing) while still process events from my async functions if such arrive. That library isn't async and in those specific situations require calling it repeatedly.
Any other way to achieve that?

It's coming from embassy_futures. Isn't that aligned with embassy-executor?

Ahhh okay. Yes, these are intended to be used as a "poor man's executor" when you don't want to use the full embassy-executor. I hadn't thought about this footgun though. Probably needs documenting.

What I need to accomplish is, for some time, to call in a loop some function from a UI framework I'm using (to drive animation processing) while still process events from my async functions if such arrive. That library isn't async and in those specific situations require calling it repeatedly.

you can make the UI task do
loop {
   poll_ui();
   embassy_futures::yield_now().await;
}

the yield_now lets other tasks run, so you can still use standard async/await for stuff in the other tasks, instead of having to interleave manual polling directly.

you can make the UI task do
loop {
   poll_ui();
   embassy_futures::yield_now().await;
}

So, if I get it right, in my case build a new async function from the code you suggest and then select on it together with my async function?

yes! the yield_now lets the other branch of the select run, and also other tasks.

Thanks! This is way more elegant.


Fix alwaysAutoInstall

Fix auto update setting not sending back to updater

@keybase/react-hackers

I don't think this is the fix. I'm not at my computer but we manage the
state of the check in the store so we do not need to pass it with the
button click
On Tuesday, January 5, 2016, Chris Ball<EMAIL_ADDRESS>wrote:

@keybase/react-hackers
https://github.com/orgs/keybase/teams/react-hackers
—
Reply to this email directly or view it on GitHub
https://github.com/keybase/client/pull/1638#issuecomment-169176545.

k, did it the better way I think... see here: https://github.com/keybase/client/pull/1648


added authentication, added proxy support

Added the ability to login to expireddomains.net.  Added the ability to pass in proxy as a switch.

python.exe .\domainhunter.py -u USERNAME -p PASSWORD -P https://<IP_ADDRESS>:8080

New Switches are:
-u for the username
-p for the password
-P (caps) for the proxy
if you don't specify -p for the password, you will be prompted for it.  There seems to be an issue with expireddomains.net, they do not appear to ever return "expired" domains, so the keyword searches are always empty.  idk what the original intent was for that feature, so I left it as it was.

I got sloppy w/ my commits.  Items I added:
--keyword-start search domains that start with a string
--keyword-end search domain that end with a string
--output save to a specific file
keyword search returns only available domains.  searches can return up to 200 rows, optimized code to break out of loop if there are no more records to fetch.

keyword gets passed in via the -k arg and is represented in the URL as "fdomain", -ks and -ke are completely optional and shouldn't affect the search results if they are left blank
The changes I made to bluecoat were related to the XSRF.  I did not make any changes to the OCR, though, I have never personally seen OCR work.
I'm not sure I follow this, I see categorization when I run it

Thanks for the quick response!
For #3 ( -s option), categorization is displayed but you lose visibility into the reputation source.   The output doesn’t make it apparent where the reputation is coming from. Should be easy enough to add into the wrapper function for that command.

I see, the print statements were commented out.  I put them all back in.


category of the bot

a question what kind of bot it‘ll be. should it be a moderation bot or a fun bot etc.

In my opinion:
It should be a moderation bot which also has fun sides with a few games that you can play alone or with several people. The interplay request can be made via private news and reactions. e.g. solitaire one could possibly. or if it is possible maumau. So everything in one offered a versatile.

Categorys:
Games and Fun
and any other things we like. But moderation is the exercise from Shinpuru.
Something like Hacktoberfest is possible, too. Sth. like count to 100 is possible, too. Everything we want, but not moderation. Hopefully my English is okay in this text

I wouldn't focus on it too hard. As an ongoing community project, the bot is mainly a reflection of our behavior from time to time.  And this holds true for the bot as well.
Tl;dr: Stuff comes as it comes, one shouldn't think about it a lot or intervene consciously.


fix: updating migration repo name in cy tests

Description
Updates name of the migration repo in the following specs to the new name in TED
cypress/e2e/Regression/ClientSide/Git/ExistingApps/v1.9.24/DSCrudAndBindings_Spec.ts
cypress/e2e/Regression/ClientSide/Git/GitAutocommit_spec.ts

Automation
/test
cypress/e2e/Regression/ClientSide/Git/ExistingApps/v1.9.24/DSCrudAndBindings_Spec.ts
cypress/e2e/Regression/ClientSide/Git/GitAutocommit_spec.ts

:mag: Cypress test results

[!IMPORTANT]
🟣 🟣 🟣 Your tests are running.
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/11059792189
Commit: 013c8ae8e4a5b1eff35385512d58c31c50501af3
Workflow: PR Automation test suite
Tags: ``
Spec: cypress/e2e/Regression/ClientSide/Git/ExistingApps/v1.9.24/DSCrudAndBindings_Spec.ts cypress/e2e/Regression/ClientSide/Git/GitAutocommit_spec.ts
Thu, 26 Sep 2024 20:31:24 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[ ] No

Summary by CodeRabbit

Bug Fixes

Updated repository names in test specifications to ensure accurate references during application validation and Git Autocommit tests.
Added a new migration tag to categorize tests under the migration context.
Modified the list of limited tests to include new test files for Git-related functionalities.

@brayn003 Please run the test here.

@sagar-qa007 The test is dependant on a custom TED build. Running the tests here with ok-to-test will not yield favourable result

Dependent on https://github.com/appsmithorg/TestEventDriver/pull/79

/ci-test-limit-count run_count=5

/ci-test-limit-count run_count=5

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit

/ci-test-limit


Update holidays.py

added Ukraine holidays

Pull Request Test Coverage Report for Build 392

5 of 21 (23.81%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 99.69%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

holidays.py
5
21
23.81%

Totals

Change from base Build 387:
-0.3%

Covered Lines:
5147

Relevant Lines:
5163

💛  - Coveralls


5.13 OOB - UX - No documentation / links to describe what is dynamic or static backhaul configuration.

ARM Internal Ref: ONME-4358

Fixed with https://github.com/ARMmbed/mbed-os-example-mesh-minimal/pull/269


Using "open" as a field name breaks edit template

When I create a boolean field called :open, I get the following error when trying to edit the model. I saw #436 which may be related, but it says it was fixed about 2 years ago, so maybe it has been broken again since the fix.
wrong number of arguments (0 for 1..3)
1: = form.send field.view_helper, field.method_name, field.html_attributes.reverse_merge((hdv = field.html_default_value).nil? ? {} : { :value => hdv, :checked => hdv })
Full Trace:
https://gist.github.com/mgartner/5015874

I've tried adding a test for this (https://github.com/codealchemy/rails_admin/commit/6f881149ca1976de8408d0f7cbd123eca330b022) but everything appears to be passing, are there any additional reproduction steps for this issue (the linked trace is a 404)?

@codealchemy Then could you open a PR with the test you wrote?
Let's merge it and consider this issue to be resolved.


compile move submodule update to BE compile

Proposed changes
orc and clucene is only used by BE, thus move the update to BE compile part
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Add eip1559 params to payload

Part of Holocene changes to add EIP1559 params to payload, and using the nonce field in block header to store the value.
Spec: https://github.com/ethereum-optimism/specs/blob/main/specs/protocol/holocene/exec-engine.md

The spec can be found here: https://github.com/ethereum-optimism/specs/blob/main/specs/protocol/holocene/exec-engine.md

The consensus rules look good to me

This PR is ready for final reviews if any of you want to give another look at it
@clabby @tynes @Rjected @mattsse

Hey @mattsse, @Rjected is right that I still need to update the new spec changes, I'm thinking about creating a separate PR

Hey @mattsse, @Rjected is right that I still need to update the new spec changes, I'm thinking about creating a separate PR

Yeah we still need the newest spec details (extraData stuff), and creating a separate PR is a good idea

with #11887 do we still need this?

Nope i can close this!


bug：Continuous dialogue is possible with“python3 -m EdgeGPT”, but not with Developer demo

The Developer demo requires you to write it yourself, and it is not included in the program.

What do you mean write it yourself? How can I achieve continuous dialogue

Just keep using the same Chatbot object


Implement event cancellation

Please add the following information to your pull request:
Please describe what your PR is fixing
When an event was cancelled, the name of the event would just be prepended with CANCELLED or something similar. However, it is nicer (and more future-proof) to actually have a database entry for this. Additionally, the website will now automatically refund all transactions and send enrolled people an email.
This was made together with @Mihai98924, give this guy some kuddos. He deserves them!
Concretely, which issues does your PR solve? (Please reference them by typing Fixes/References Inter-Actief/amelie#<issue_id>)
Fixes Inter-Actief/amelie#696
Does your PR change how we process personal data, impact our privacy document, or modify (one of) our data export(s)?
no
Does your PR include any django migrations?
yes
Does your PR include the proper translations (did you add translations for new/modified strings)?
yes, I have included the translation
Does your PR include CSS changes (and did you run the compile_css.sh script in the scripts directory to regenerate the compiled.css file)?
no, my PR does not include CSS changes
Does your PR need external actions by for example the System Administrators? (Think about new pip packages, new (local) settings, a new regular task or cronjob, new management commands, etc.)?
no
Did you properly test your PR before submitting it?
yes

Someone also just reminded me about people who might have already paid by cash, we need to handle those situations as well

I've tackled the issue with cash payments that the treasurer will get an email with all the people that have paid in cash. Additionally, only people that have paid by mandate will get a transaction that refunds their money.


Robot still running - changedCharging / changedCleaning

Hi, thank you for the new version with the fix for the running status.
However I still seem to have some problem with status after the cleaning job is completed.
Basically when the roborock S50 finishes full house clean and returns to the dock station, HomeKit thinks it's still running until either 1) I manually turn the fan off (but it gives issue next time when I start cleaning from Homekit) or 2) restart HomeBridge service. I did several tests recently,  same behaviour regardless if I start cleaning from Homekit or Mi Home. If I start clean and stop manually in HomeKit (turn off the fan), everything works as expected.
To conclude: it looks to me that the plugin cannot detect when the cleaning job is finished (and the roborock returns to the dock station) and think it's still running (forever).

@maisun
Really? I checked his code. Him plugin makes everything similar, like my plugin - no reconnect.

@maisun
Really? I checked his code. Him plugin makes everything similar, like my plugin - no reconnect.

Hi Nico, I must say that I'm also confused now.
It turned out that his plugin works sometimes but gives the same problem (status is cleaning in HomeKit but the robot has already returned to dock) other times. One slight benefit I can always manually switch off the robot from HomeKit. With your plugin, I can't switch off, I have to restart HomeBridge to show the off status.
As you suggested, WIFI can be an issue. I do know there are certain spots in my house with weak signal. Another cause I can think of is miio library - it is shared with other Xiaomi hardware such as light bulbs and aqara gateway, one thing I'd like to try is to create a new HomeBridge instance only for the robot.
I'm not a javascript developer, but know programming a little bit. My question is is it possible to implement re-connect logic? Not sure how miio keeps the connection to the robot, but one way that I can think of is to use heartbeat - for example every minute try to get the battery status, if it fails for more than 5 minutes reconnect miio. Could this logic work?

Is the message ERR getBattery | No vacuum cleaner is discovered.' often seen in the Homebirdge logfile or just once? If more often, at what time interval?

Is the message ERR getBattery | No vacuum cleaner is discovered often seen in the Homebridge logfile or just once? If more often, at what time interval?

Searched in the log and couldn't find this error at all. I'm wondering:

If HomeKit tries to pull battery periodically
When the connection is broken, does miio send back the error?

No idea, you said it would be like that.
HomeKit only gets the battery status on request. The message would only appear ... it will not work the way you think.

work

Ok, understood. Could it be possible to pull the battery state from your plugin every minute? I also started a thread on this in miio project.

Yes do it: https://github.com/aholstenson/miio/
The API communicates any battery level changes, then the message appears - not every minute.

Thanks!
Already did: https://github.com/aholstenson/miio/issues/221
Sorry I thought the battery status is pulled from miio so assumed you can triggered from the plugin.
I looked into the implementation for vacuum cleaner in miio:
https://github.com/aholstenson/miio/blob/master/lib/devices/vacuum.js
I don't see any error handling there. I'm thinking what will happen if we call the method "getHistory" periodically. It won't do anything to the device and may print out some error
/**
 * Get information about the cleaning history of the device. Contains
 * information about the number of times it has been started and
 * the days it has been run.
 */
getHistory() {
	return this.call('get_clean_summary')
		.then(result => {
			return {
				count: result[2],
				days: result[3].map(ts => new Date(ts * 1000))
			};
		});
}

Fork it, code it, test it and make a pull request.

Fork it, code it, test it and make a pull request.

Would like to try but not really familiar with javascript ecosystem so will take some time. Will write back when I manage this. Thanks for the help!

I feel the same as you. I am it administrator, not a programmer. I can write simple batch, powershell, and bash scripts. It is my first javascript plugin. Much try and error, step by step, it takes a long time, but it works. :-D

I like that :-) which editor/ide do you use?

I use Sublime Text on macOS.


failed installing on kibana 5.0.0 on Ubuntu

root@vagrant-ubuntu-trusty-64:/usr/share/kibana# ./bin/kibana-plugin install https://github.com/ommsolutions/kibana_ext_metrics_vis/archive/0.1.0.zip
Attempting to transfer from https://github.com/ommsolutions/kibana_ext_metrics_vis/archive/0.1.0.zip
Transferring 720161 bytes....................
Transfer complete
Retrieving metadata from plugin archive
Plugin installation was unsuccessful due to error "No kibana plugins found in archive"

Did it also not work by extracting the contents to the plugins-directory manually?
Or is it just the installer?

I installed manually plugin with #2 and it's working for me (Kibana 5.0.0).

okay thanks @Tonyko I will verify and merge this later today

I have created a fixed release-package and updated the readme with a link to the fixed archive:
$ ./bin/kibana-plugin install https://github.com/ommsolutions/kibana_ext_metrics_vis/releases/download/0.1.0/extended_metric_vis.zip


As a User I want to enter a comment against each result, e.g. "Dan beat Gavin 2 games to 1"

I am closing this in favour of #16 - the parent of this user story, which will comfortably accommodates this feature.


Shared folders

Hey guys,
I was wondering if this was usable to access shared folders? I have an account that as multiple shared folders and I want to access those shared folders.
Thanks

@asivaneswaran  Hello
Did you find a way to archive this?


User.social_links does not handle multiple forms of LinkedIn profiles:

Problem:
Regression bug was added when isssue #3401 was closed.
The User.social_links method does not handle:

in/UserID
pub/Path/To/Public

Proposed Solution:
Modify User.social_links to handle the multiple ways LinkedIn profiles can be linked.

Specifically, the base url here needs to be altered and the User.social_links property needs to be updated to handle any of the three possible LinkedIn profile paths.
Sample URLs for LinkedIn profiles can be found in the comments of issue #3401


Specifying region parameter doesn't work

The crossing help shows an option to specify which region the key is in using -r or --region=s, but both options cause errors that they are unrecognized parameters.

This was just user error. The region parameter is a global option and needs to come before the put or get commands.


Setting: optimize speed vs optimize space

In the upcoming settings panel, give the user the choice between optimize speed vs optimize space
optimize speed
When exporting, export all cores and roms, regardless of what the plan says. That way from then on, exports will almost entirely be mras, making them much fater
optimize space
What the app does now. Only export exactly what is needed, and delete anything that is not in the plan.

Added in 0.22.0


Checking of whitespace

Why the latest grunt-contrib-jshint is not validating the syntax of the code. Is there any way I can validate the below code with .jshintrc
**Wrong Code: **
if(condition){
     Statements
}

Correct Code
if (condition){
     Statements
}

Basically checking space between if and (.

 We recommend the JSCS project as a fully-featured means to validate JavaScript code style.


Dataset.as_array() returns array of wrong size for stitched images

Hello,
I faced an unexpected behavior in the function as_array when using the stitched=True option. The returned dask array seems to be in the wrong shape. When converting it to a numpy array however the shape assumes the right dimensions.
I have tow-channel images of dimensions 2016x852px, which are arranged in two columns and eight rows (no overlap).
Experienced behavior:
d = data.as_array(stitched=True, z=0)
print(d)
>>> a=np.asarray(d)
print(a.shape)
>>> dask.array<read_one_image, shape=(2, 2016, 852), dtype=uint16, chunksize=(1, 2016, 852), chunktype=numpy.ndarray>
a=np.asarray(d)
print(a.shape)
>>>(2, 6816, 4032)

Note that beside the fact that the dimensions are wrong the width and the height of the images are swapped: In the dask-array the columns come before the rows, whereas in the numpy array the rows come before the columns.
Expected behavior:
d = data.as_array(stitched=True, z=0)
print(d)
>>> a=np.asarray(d)
print(a.shape)
>>> dask.array<read_one_image, shape=(2, 6816, 4032), dtype=uint16, chunksize=(1, 6816, 4032), chunktype=numpy.ndarray>
a=np.asarray(d)
print(a.shape)
>>>(2, 6816, 4032)

Fix:
Changing the following rows
483    w = self.image_width if not stitched else self._tile_width
484    h = self.image_height if not stitched else self._tile_height

...

553    chunks += (w, h)

to
483    w = self.image_width if not stitched else self._tile_width*len(self.axes["column"])
484    h = self.image_height if not stitched else self._tile_height*len(self.axes["row"])

...

553    chunks += (h, w)

in nd_tiff_v2.py seems to solve this problem.
Suggestion:
This way however the chunks consists of the already stitched images. For very large stitched images it might be better, though, to have a single chunk for every acquired image, thus a dask array of properties
dask.array<read_one_image, shape=(2, 6816, 4032), dtype=uint16, chunksize=(1, 852, 2016), chunktype=numpy.ndarray>
This might be useful for some people.
The bug is related to the already fixed issue reported by ptbrown1729: https://github.com/micro-manager/pycro-manager/issues/278
Otherwise very nice package! Thanks for developing this open source project!

Yes, I agree that the chunks should not be a full stitched image, as this is memory-inefficient. I'm pretty sure that this behavior works as intended for NDTiff v3, so this bug is probably confined to data collected with older versions of pycro-manager/micro-manager--is that right? I'd guess that there are some changes to v3 that may need to be backported to the v2 code. Do you have any interest in making a pull request to apply a fix? I'd be happy to guide you through the process

Correct, this was fixed for v3 datasets in #107, but I don't think I've tested the stitched behavior - do let us know if you run into issues with v3 datasets as well.


Microsoft account

How do i add the microsoft account?

select auth type microsoft

oh

ik

just how do i connect it

click on start after you put the right username of the account and it will ask you to login


Led and buttons connection

Hi
I finally managed to put all together but the screen, lights and buttons doesn't work. I think the connections are not right. Could you please help me with that? Are the LEDs connected between then with the positive or negative? The buttons are connected together and after one of them to negative, which pins are the buttons connectors to? Sorry but I can't figure out. But I'm almost there. Could you please help me. I know you are a busy man. I could help with the description for installation.
Thanks

Thank you very much for your reply
I tried but I think my Arduino is not fine so I ordered more to check it
out. I will let you know if works.
But in the pin.h I saw you define mode, time and stop, the buttons right?
But for button_mode the pin is 5. Is that D5? I'm confused. Because
button_time is A5 and button_stop is A4.
Sorry for disturb you
On Fri, 18 Aug 2023, 14:41 Krzysztof Heim, @.***> wrote:

Hello,
Please look at schematic:
https://github.com/pankleks/WasherOne/blob/main/Electronics/Schematic_washer_2022-02-19.pdf
See also:
https://github.com/pankleks/WasherOne/blob/main/Assembly/IMG_20220219_191455.jpg
LED are connected to PIN -> D2, D3, D4 via 330 ohm resistors. Positive LED
to PINS, negative to ground.
Buttons PIN's are pulled up, means that pressing button should connect PIN
to ground.
So please connect all buttons to ground and other side to apriopriate PIN.
All default PIN's are described in:
https://github.com/pankleks/WasherOne/blob/main/Arduino/src/Pin.h
hope this helps
—
Reply to this email directly, view it on GitHub
https://github.com/pankleks/WasherOne/issues/4#issuecomment-1683940194,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXLAH7G2DFEJTTSKLNFLXEDXV5WJ7ANCNFSM6AAAAAA3VJXTDI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hello, I have such a problem, I connected everything and everything works
I just don't want to spin the engine, it is free, when I start the mode it is blocked, where could the problem be?

Hi
Thank you very much for your help.
After your email I managed to put all together but the board didn't work,
so I told you I'll but more but the problem was the computer I used to use
for visual studio. Do now it's working but I noticed the 24v output it's
always running so I thought maybe a relay switching on an off your know
when you need it. I tried to do it, honestly not by myself I can't do
programming even C/C++, so I tried with the Aichat but I can't do work it
out... If you have any ideas please let me know.
Anyway, the project is awesome, really nice. Thank you to share it with
everybody.
On Fri, 18 Aug 2023, 14:41 Krzysztof Heim, @.***> wrote:

Hello,
Please look at schematic:
https://github.com/pankleks/WasherOne/blob/main/Electronics/Schematic_washer_2022-02-19.pdf
See also:
https://github.com/pankleks/WasherOne/blob/main/Assembly/IMG_20220219_191455.jpg
LED are connected to PIN -> D2, D3, D4 via 330 ohm resistors. Positive LED
to PINS, negative to ground.
Buttons PIN's are pulled up, means that pressing button should connect PIN
to ground.
So please connect all buttons to ground and other side to apriopriate PIN.
All default PIN's are described in:
https://github.com/pankleks/WasherOne/blob/main/Arduino/src/Pin.h
hope this helps
—
Reply to this email directly, view it on GitHub
https://github.com/pankleks/WasherOne/issues/4#issuecomment-1683940194,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXLAH7G2DFEJTTSKLNFLXEDXV5WJ7ANCNFSM6AAAAAA3VJXTDI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi
I finally managed to put all together but the screen, lights and buttons doesn't work. I think the connections are not right. Could you please help me with that? Are the LEDs connected between then with the positive or negative? The buttons are connected together and after one of them to negative, which pins are the buttons connectors to? Sorry but I can't figure out. But I'm almost there. Could you please help me. I know you are a busy man. I could help with the description for installation.
Thanks

Hello so could you share your bom and .cpl files .I am still trying to make this most fantastic  device.
Thanks in advance


where do you even get BLOCKCHAIN_API?

where do you even get BLOCKCHAIN_API?

i got this solved by going into https://api.blockchain.info/customer/signup and wait for 3 days. i received in my email.

oh great
On Tuesday, August 14, 2018, 3:18:50 PM GMT+1, Ken Dan Tinio<EMAIL_ADDRESS>wrote:
i got this solved by going into https://api.blockchain.info/customer/signup and wait for 3 days. i received in my email.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Total usd value of  account should be calculated based on the real time price of KSM/BSX/xxx

On the wallet hover popover, the total usd value of the current account should be calculated based on the real time price of KSM/BSX/xxx, according to the design.

Originally posted by @Jarsen136 in https://github.com/kodadot/nft-gallery/issues/4063#issuecomment-1280157263

is this issue from past completely missed um

let me check it : )

@Jarsen136 I've started working on this as part of

fix: #4302 add wallet name #4476

it just wasn't previously linked and description of this task was mentioned in comments of

Add name of wallet you are logged in and show assets of address #4302

I have ignored the above link. Go for it~


support filtering by parent campaign in Sites pages

hub request: https://powerofus.force.com/0D580000025Iexy
the JobListing and Calendar page both support filtering by a single campaignId.  This request would be to allow that campaignId be a parent campaign, and then include all Jobs/Shifts from child volunteer campaigns of the passed in campaign.

Would it be preferred to have an additional query string param of "includeHierarchy=true" when a campaignId is set?

I like the idea of a query parameter - this is one of the enhancements to V4S that is on my list if I kind carve out the time to dig in.

@sgjudd I'm digging in right now at https://github.com/cdcarter/Volunteers-for-Salesforce/tree/campaignhierarchy. Hope to have a patch in this weekend.

Pull request above offers the desired behavior in JobListing pages as an optional boolean.
Inside of that commit history is an implementation for JobCalendar too, but I then removed it. I don't know well enough how Javascript Remoting and everything else that has that Calendar wired up quite works. Today, I couldn't reason about how to make it a non-required option, so I didn't change existing behavior. I will try and investigate further but could use some guidance!

will be included in version 3.76.  I also added support for ShowCampaignHierarchy on the JobCalendar page, and added additional tests code for all of the Sites pages supporting this URL parameter.


Santa Hat Removal

Issue Type: Feature Request
Please add the Santa Hat back. The cheer it represents reminds me of when my mother and father were still alive.
VS Code version: Code 1.41.0 (9579eda04fdb3a9bba2750f15193e5fafe16b959, 2019-12-11T18:37:42.077Z)
OS version: Windows_NT x64 10.0.17134

Bring back the Santa hat. A bit of symbolism never hurt anybody, especially when said symbolism is clearly not meant to offend anyone. While Christmas is indeed a Christian tradition, the Santa hat represented the cheerful spirit and happiness many of us observe at this time of year, and we were robbed of that cheerful spirit.

Please [re add] it immediately and make it your top priority

Duplicate of #87291

Bring back the hat. Santa is not a religious figure even though his schtick is closely related to a religious event. It's like banning "Happy Holidays" since its associated with "Merry Christmas".

Duplicate of #87291

Thanks for creating this issue. We think this issue is unactionable or unrelated to the goals of this project. Please follow our issue reporting guidelines.
Happy Coding!


Zabbix 5.4 - ITEMID not provisioned to MailGraph by Zabbix

SYMPTOM
MailGraph notification failed : SyntaxError: invalid json (at offset 1)
MESSAGE DETAILS
Problem started at 10:40:25 on 2021.12.16
Problem name: (ZBX) Active sessions - RDP
Host: xxxxx
Severity: Information
Operational data: 1
Original problem ID: 855136
eventId: 855136
TriggerId: 21573
itemId: {ITEM.ID]
Note that itemId is not filled when calling MailGraph, resulting in an error in MailGraph.
Investigations and likely code update underway.

Debug run from CLI:
"Missing ITEM ID?"
confirms the issue. Rethinking the MailGraph approach to see how to deal with such situation where only the Event has been provisioned.

{
"jsonrpc": 2,
"error": {
"code": -32600,
"message": "Invalid Request.",
"data": "Invalid parameter "/": unexpected parameter "expandComment"."
},
"id": 3
}
! No response data received?

Fixed several issues in v2.0

itemId not passed by Zabbix anymore
Fixed several warnings (due to lack of information availability checking)
Moved several parameters into "params" section (as per Zabbix 5.4 API changes)
Outstanding issue found:
mailGraph.screen needs to be replaced with mailGraph.dashboard as SCREENs no longer exist in Zabbix 5.4 and existing screens were migrated to Dashboards


Make download link for Node.js work

They changed the structure of the page so the old link was a 404.

👍

I noticed that I'd also missed out ./script/bootstrap in the Getting Started section. I fixed that in 0fa17a9.


[Feature] omar72158895/gold

000135
Summary
A clear and concise description of what the problem is, and what you solution you want.
Changes
What changes this feature would introduce. Any risks these changes involve. Any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Es bueno este proyecto de inversiones e t.advisor

Ola soy oma

Confirmo omar

Confirmo


Support CRD lookup without a dependency on a running cluster

What would you like to be added?
Allow CRDs to be looked from some centrally managed CRD store. I am not sure if that exists, but if it doesn't I believe it would be very valuable to support a more seamless user experience.
Why is this needed?
I find linting fits best in use-cases where linting of manifests is done before applying manifest files to a cluster. This means that CRDs are not guaranteed to exist on the cluster, and thus pointing kubectl-validate towards a cluster isn't feasible.
Storing CRDs or OpenAPI schemas locally is also not a favorable option in this case, as that would require maintaining the CRDs separately, which Is somewhat cumbersome.

Maybe datreeio's CRD collection can be used? https://github.com/datreeio/CRDs-catalog

/reopen
I found this line in the code:
https://github.com/kubernetes-sigs/kubectl-validate/blob/main/pkg/cmd/validate.go#L195-L196
Could this feature be as simple as adding a condition to this line?


Streaming video from local harddrive

Hi Guys,
I am trying to stream videos from local harddrive example : C:\Users\Public\Videos\Sample_Videos\Wildlife.mp4.
Is this possible from Videogular?
If so can you please let me know how to set this as vg-source?
Thanks and looking forward to your reply.
-/Sammi

You can't access local files from a web browser.
What you can do is to use the File System API and save on browser's File System the video file but user must choose the file for you. Those are all security restrictions to avoid playing/saving video files without user permission.
This is a weird question, can you explain better what's your problem and what you want to achieve?


Support OXC estree flavour

Another fast ESTree parser is oxc.
https://github.com/web-infra-dev/oxc#-ast-and-parser
However the generated ESTree AST slightly differs:

The Oxc AST differs slightly from the estree AST by removing ambiguous nodes and introducing distinct types.
For example, instead of using a generic estree Identifier,
the Oxc AST provides specific types such as BindingIdentifier, IdentifierReference, and IdentifierName.
This clear distinction greatly enhances the development experience by aligning more closely with the ECMAScript specification.

But it would be pretty awesome if it was supported!
The additional distinct types is quite a big change I would think:

OXC ESTree AST
const helloWorld = "Hello World";

{
  "type": "Program",
  "start": 0,
  "end": 34,
  "sourceType": {
  	"language": {
  		"typeScript": {
  			"isDefinitionFile": false
  		}
  	},
  	"moduleKind": "module",
  	"variant": "standard",
  	"alwaysStrict": false
  },
  "directives": [],
  "hashbang": null,
  "body": [{
  	"type": "VariableDeclaration",
  	"start": 0,
  	"end": 33,
  	"kind": "const",
  	"declarations": [{
  		"type": "VariableDeclarator",
  		"start": 6,
  		"end": 32,
  		"id": {
  			"type": "BindingPattern",
  			"kind": {
  				"type": "BindingIdentifier",
  				"start": 6,
  				"end": 16,
  				"name": "helloWorld"
  			},
  			"typeAnnotation": null,
  			"optional": false
  		},
  		"init": {
  			"type": "StringLiteral",
  			"start": 19,
  			"end": 32,
  			"value": "Hello World"
  		},
  		"definite": false
  	}],
  	"modifiers": null
  }]
}

Regular ESTree AST
{
"type": "Program",
"start": 0,
"end": 33,
"body": [
  {
    "type": "VariableDeclaration",
    "start": 0,
    "end": 33,
    "declarations": [
      {
        "type": "VariableDeclarator",
        "start": 6,
        "end": 32,
        "id": {
          "type": "Identifier",
          "start": 6,
          "end": 16,
          "name": "helloWorld"
        },
        "init": {
          "type": "Literal",
          "start": 19,
          "end": 32,
          "value": "Hello World",
          "raw": "\"Hello World\""
        }
      }
    ],
    "kind": "const"
  }
],
"sourceType": "module"
}

I haven't found another library for traversing a OXC compatible ESTree in Javascript yet.
The node types are available in https://github.com/web-infra-dev/oxc/tree/main/crates/oxc_ast/src

Aren't there performance overhead in FFIs? As far as I know oxc uses napi-rs to communicate between rust and js.

There's a performance overhead, but parsing a big document with OXC is still faster than what I do today. I have a transformer implemented in SWC, but I would like to replace that with JS instead. My initial experiments tell me that I can shave of a few ms by doing it this way.


Duration pitch loss not used.

Hello, I was looking into your code and it seems like the code does not consider the duration_pitch_loss.
https://github.com/lucidrains/naturalspeech2-pytorch/blob/659bec7f7543e7747e809e950cc2f84242fbeec7/naturalspeech2_pytorch/naturalspeech2_pytorch.py#L1522
Maybe, it might be related to the aux_loss you have made.
https://github.com/lucidrains/naturalspeech2-pytorch/blob/659bec7f7543e7747e809e950cc2f84242fbeec7/naturalspeech2_pytorch/naturalspeech2_pytorch.py#L1600
Thanks for the great work!

@jaykim9870
I have the same question.
You're thinking that code should be changed like below. Right?
before :  return loss + (self.rvq_cross_entropy_loss_weight * ce_loss) + duration_pitch_loss
fixed : return loss + (self.rvq_cross_entropy_loss_weight * ce_loss) + aux_loss

@jaykim9870
I have the same question.
You're thinking that code should be changed like below. Right?
before : return loss + (self.rvq_cross_entropy_loss_weight * ce_loss) + duration_pitch_loss
fixed : return loss + (self.rvq_cross_entropy_loss_weight * ce_loss) + aux_loss

@wonwooo
Yes, that would do.
FYI, There are some other issues like wavenet based diffusion model as the model size is very different from the original paper. As far as I have investigated, the model architecture is too different so it may affect the model performance. If you are working based on this project, you may also need to check those out!