Parent & selector to reference component class

Currently, using the & parent selector it applies a class above the component class, which breaks the whole encapsulation rule:
eg:
.button {
    .active & {
        color: blue;
    }
}

generates:
.active .component-classname .button { color: blue; }

but it would be great if it generated:
.component-classname.active .button{ color: blue }

Thoughts?

Did you try?
&.active {
    .button {
        color: blue;
    }
}

This sass:
& ul {
    .active & {
        color: red;
    }
}

&.active ul {
    color: red;
}

produces this css:
.active .post-type-photo-9364b334 ul {
  color: red; }

.post-type-photo-9364b334.active ul {
  color: red; }

However, the 1st sass version should really generate the 2nd css version, right?

I have a branch which deals with this issue here: https://github.com/topaxi/ember-component-css/tree/postcss-scss
This currently fixes only scss (stylus and less won't work yet).
I will contact ebryn next week to coordinate the steps needed to get some fixes landed.

Awesome

Three years and this is not resolved. 😒
Sometimes a child element must change its looks depending on a modifier on a parent. For example, a menu item must be bold when its menu section is expanded.
But I have the following code style requirement: CSS rules that decorate an element should be nested under that element's own section. This requirement makes the following CSS illegal:
.mainMenu-item {
  font-weight: normal;
}

.mainMenu-section {
  &.is-expanded .mainMenu-item {
    font-weight: bold;
  }
}

The font-weight: bold rule is applied to the item, it should be written under item and not section. This is common sense.
According to my code style, the correct way of writing the above code is the following:
.mainMenu-item {
  font-weight: normal;

  .mainMenu-section.is-expanded & {
    font-weight: bold;
  }
}

Of course, I wanted to avoid repeating .mainMenu. But since this ticket has never been resolved, I was unable to use & here.
And since ember-component-css does not allow disabling namespacing selectively, I had to revert to an ugly hack:
$this: ".mainMenu";

@at-root #{$this}-item {
  font-weight: normal;

  #{$this}-section.is-expanded & {
    font-weight: bold;
  }
}

You can imagine how much pain and misery was caused by numerous @at-root #{$this} to me and my team. 😫 I've been hating ember-component-css with all my heart every time I had to convert from styleNamespace to a manually set pair of  {className: "mainMenu"} and $this: ".mainMenu". Why does reasonable rule structuring has to be so hard?!
But today I found a workaround! Feast your eyes:
& {
  $this: & !global;
}

&-item {
  font-weight: normal;

  #{$this}-section.is-expanded & {
    font-weight: bold;
  }
}

I've been hating ember-component-css with all my heart

@lolmaus That’s no good!! 😞
so, one thing we have batted around doing is giving the styleNamespace an actual variable name, so that things like this can be easier.
One quick question, your component styleNamespace is represented here as mainMenu, and you are wanting to do a “BEM” style classes by doing {{stylenNamespace}}-item in the template?

your component styleNamespace is represented here as mainMenu, and you are wanting to do a “BEM” style classes by doing {{stylenNamespace}}-item in the template?

Yes, sir.

so, one thing we have batted around doing is giving the styleNamespace an actual variable name, so that things like this can be easier.

An easy way to do it is to wrap the whole component Sass file with @at-root { }.
@at-root without a selector has no meaning and does not influence the CSS ouptut in any way. But it has one side-effect: it turns variables/mixins/functions defined inside into local ones. So if you do this:
@at-root {
  $base-selector: <styleNamespace>;

  <processed Sass code>
}

...then the $base-selector variable will be available to the component's Sass code without overwriting a $base-selector variable in the global scope.
The user is still able to define global variables with the !global flag.
As for mixins and functions, !global does not work for them. They will become local and can no longer be imported. My typical use case is to share component-specific mixins and functions from a separate file (e. g. mixins.scss next to style.scss) -- this will no longer work.
This can be resolved with conditional disabling of namespacing. For example, ember-component-css could ignore Sass files that start with a dash or a double underscore, etc.
Note that two levels of disabling are required: disable namespacing and disable auto-importing. The latter is available through environment.js, but that's very inconvenient. A filename prefix/postfix is much more handy.

ya. I think the reason this stuff hasn't been that much of an issue for me personally, is that I don't use "BEM" style css for the vast majority of the time. So that's why I personally (or people I work with) haven't felt this particular pain. cause something like..
.item {
  font-weight: normal;

  .section.is-expanded & {
    font-weight: bold;
  }
}

is all we would do.
THAT POINT ASIDE (if you want to get into a specificity or css selector style debate, I'm more then happy to on discord..!! 😂😊)
But the idea would be that something like {{styleNamespace}} would be provided, so that you could do something to the effect of..
&-item {
  font-weight: normal;

  {{styleNamespace}}-section.is-expanded & {
    font-weight: bold;
  }
}

or some other equivalent thing.
@lolmaus Is that something that would interest you?

@webark Yes. And above I've suggested a way how it could b implemented without leaking.

@lolmaus ya. The only issue with that is it's pretty explicit to just scss. The idea would be do use postcss to do a rule string replacement. Similar to what we currently do with less, scss, and cssNext.

we'd do the same for sass and stylus.. but last I checked those don't have a postcss syntax processor. (and not that many people use them either).

I don't use "BEM" style css for the vast majority of the time. So that's why I personally (or people I work with) haven't felt this particular pain. cause something like .item, .section is all we would do.

I don't use short semantic selectors because they make this scenario very probable:

Another developer applies styles directly to a .section element.
All your components that use .section are defaced.
The other developer does not notice because none of his code involves your components, so he/she doesn't bother to check them.
You don't notice either because your project does not have visual regression testing set up.

if you want to get into a specificity or css selector style debate, I'm more then happy to on discord

Not willing to debate over it, it's up to every developer/team to decide. But I'll share my POV: I do care about specificity because unnecessarily high specificity makes it harder to override styles, and I've seen it encourage the use of !important. 👎

Is that something that would interest you?

Yes. But I would very much prefer a Sass-native solution.

The & parent selector can't be used at top level. :suspect:

We could do something like $styleNamespace rather then {{styleNamespace}}

I'll reiterate on my thoughts regarding this:

If you do it without extra measures, the variable will be global and may potentially interfere with other stylesheets.
This can be avoided by wrapping each component stylesheet with @at-root { }. It will turn variables, mixins and functions defined inside into local ones.
The developer can un-local a variable with the !global flag. Unfortunately, there is no such flag for mixins and functions.
That's a huge bummer because I often ship mixins with components -- typically to context-specific styles (e. g. narrow and wide).
To work around the local mixin issue, there should be a way to selectively disable namespacing for a file.
Excluding files via config/environment.js in not convenient. We could have it for consistency with disabling importing (which is already available inconfig/environment.js). But there should be a way to do it locally, e. g. via a filename suffix or a comment inside the style file.
If we implement a way to disable namespacing for a given component, we should provide the same way for disabling importing.

The & parent selector can't be used at top level. :suspect:

Ahh. To support rules like &-item {. Sure, that would be annoying. But wouldn't having rules that start with #{$this} cause errors also? Cause $this wouldn't be declared in the style file.

I'll reiterate on my thoughts regarding this:

Ya. I get all this. It's just... That seems like a lot to do in-order to solve the issue. I'll think more about this and get back to you.


Question/Feature Request: Disable Gutter Icons - Only have text messages

This extension is a great example of do one thing really well.
However, extensions that use the gutter tend to interfere with each other, and with this extension enabled I am not able to add/remove breakpoints in the gutter for that line, at least using the mouse. I've had to disable gutter actions for other features/extensions for similar reasons, including the built-in testing gutter icon/menu.
I looked through the documentation, but don't see a way of disabling the gutter icon part of this extension. Is it possible?  If not, could this feature be added?
Would recommend it or even make it a default extension for our installs with that tweak. Thanks!

Found the setting - Gutter Icons Enabled. Not sure how I missed it in the settings.


Bug:   ReferenceError: window is not defined

Describe the bug
This error happened while generating the page. Any console logs will be displayed in the terminal window while testing locally
whenever am using the app locally and navigate to some other pages inside the app my browser is showing up this error

Steps to reproduce
Navigating to other pages

Thanks for the issue @devharsh2k4. Which page(s) is this happening on? We have some end to end tests and locally I have yet to encounter this issue.

Actually its mostly happening while locating to other profiles,repos and others pages its totally random can't say its only limited to those pages first of all i thought that it might be an issues with my browser as am using Brave browser so i also tried with chrome but still the same issue was happening
My OS :- Linux (Debian 12)

unable to replicate this via this link looks like a local environment issue

getting this error in compiler

Since it's not happening on beta, I'm going to go ahead and close this. If you're having trouble getting set up, first go through the installation instructions.


zip and rm not working with " "

manga-cli was giving out
"error: could not open document"
I figured it was because rm and zip was not working properly with the arguments $image_dir enclosed by " in the create_file() function
zip -q $image_dir$name-$chapterNumber.cbz $image_dir*.jpg
rm $image_dir/*.jpg

works fine for me.

can confirm this fix works perfectly


Rerender feedstock and add OSX support

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

If it builds, I'm happy :)


Insert batch - JSON error

java.sql.BatchUpdateException: Code: 48. DB::Exception: Method getDefault() is not implemented for data type Object('json'). (NOT_IMPLEMENTED) (version <IP_ADDRESS>4 (official build))
sink              | , server ClickHouseNode [uri=http://clickhouse:8123/test, options={client_name=Agent_1}]@-1529990687
sink              | 	at com.clickhouse.jdbc.SqlExceptionUtils.batchUpdateError(SqlExceptionUtils.java:109)
sink              | 	at com.clickhouse.jdbc.internal.InputBasedPreparedStatement.executeAny(InputBasedPreparedStatement.java:142)
sink              | 	at com.clickhouse.jdbc.internal.AbstractPreparedStatement.executeLargeBatch(AbstractPreparedStatement.java:85)
sink              | 	at com.clickhouse.jdbc.internal.ClickHouseStatementImpl.executeBatch(ClickHouseStatementImpl.java:568)

Looks like a server error, will close it after investigation.

Hi @subkanthi, what's the query for reproducing the issue? There's workaround(just treat JSON as String) for client-side serialization issue mentioned in #1023.

I was able to reproduce the issue and I believe it's more of a server side issue - ClickHouse/ClickHouse#39866


data properties in agro

'planned process' EquivalentTo 'has name' min 1 xsd:Name

This means that every instance of a planned process has a name. E.g. if a farmer joe plows his field on wednesday, that instance of plowing is given a name by joe. I don't think this is intentional or correct.
Some of the other data properties are suspect. Consider adding documentation for them.
Are we sure that every agricultural implement has a brand and a model?

The reason why Céline added those properties is that this kind of information is needed when the fieldbook is created. But I agree, since the ontology can be used by other systems, this is not correct. I am going to change that.

In our GenEpiO, we're not putting such things in equivalency.  But we're doing this in the necessary conditions (SubClassOf) section.  Following your example:
'planned process' 'has member' min 1 'process name'.

and elsewhere
'process name' subClassOf 'textual entity'

Update on GenEpiO.  I've shifted to thinking of elements like "process name" as datums associated with standards, which involves creating a (provisional) standard for collecting the datums associated with some process.  Found RO's "has component" http://purl.obolibrary.org/obo/RO_0002180 which seems to have the right structure for associating standards and datums. So something like "planned process for tilling v1" 'has component' min 1 'process name'.  Does that sound feasible?

Partonomies should be homeomorphic, e.g. processes should have parts
that are also processes (has component is a subproperty of has part).
This is a bit under-axiomatized in RO right now, but I think if you try
what you suggest it will lead to unsatisfiability.
Have a look at OBO to see how information artefacts are connected with
processes.
On 17 Sep 2017, at 12:26, Damion Dooley wrote:

Update on GenEpiO.  I've shifted to thinking of elements like "process
name" as datums associated with standards, which involves creating a
(provisional) standard for collecting the datums associated with some
process.  Found RO's "has component"
http://purl.obolibrary.org/obo/RO_0002180 which seems to have the
right structure for associating standards and datums. So something
like "planned process for tilling v1" 'has component' min 1 'process
name'.  Does that sound feasible?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/AgriculturalSemantics/agro/issues/23#issuecomment-330037191

Ok, I agree about homeomorphic structure.  I should have proposed: for a process, we can establish a standard that describes the datums input or output by that process (in detail these would document every input and output but in practice that doesn't always happen.  The standard would be a compound data structure, and its parts would be single datum or compound data structures.  I read in the RO definition for 'has component' relation that it was intended "In situations where you would want to say something like 'has part exactly 5 digit, you would instead use has_component exactly 5 digit.", which seemed to be exactly the data structure relation needed to define standards that don't involve complex graph structures (that ShEX specializes in)?

has component only exists to get around the DL constraint of no
cardinality on transitive properties
On 18 Sep 2017, at 18:51, Damion Dooley wrote:

Ok, I agree about homeomorphic structure.  I should have proposed: for
a process, we can establish a standard that describes the datums input
or output by that process (in detail these would document every input
and output but in practice that doesn't always happen.  The standard
would be a compound data structure, and its parts would be single
datum or compound data structures.  I read in the RO definition for
'has component' relation that it was intended "In situations where you
would want to say something like 'has part exactly 5 digit, you would
instead use has_component exactly 5 digit.", which seemed to be
exactly the data structure relation needed to define standards that
don't involve complex graph structures (that ShEX specializes in)?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/AgriculturalSemantics/agro/issues/23#issuecomment-330303757

An example: an organization specification 'has component' some contact specification; and contact specification 'has component' exactly 1 'first name'; but I don't want that to entail that 'organization specification' 'has component' exactly 1 'first name'.  I assumed 'has component' semantics - the non-transitive part - was allowing this separation.

You know I'm still suspicious that this is going down the wrong path. The use case here seems to be encoding closed world constraints for data entry. A shape language like shacl or shex may be better here.
There's also a discussion on the BFO list right now about modeling of things like names, and is it really necessary, which is relevant.

Ok, I'll dial into that.  I'm in on the OBI discussion board and they're talking about categorical, ordinal and other variables, but text inputs seemed less problematic.
But I know, using ontology to describe standards could lead to some problems.  I was trying the approach out; this diagram captures where I'm at...

was addressed some time ago


clang-tidy: apply modernize-loop-convert over a set of files

Applied a modernize-loop-convert using chip-tool, all-clusters and tests as the sources of files to compile.

I only got partway through, but there are a lot of issues here in terms of the naming and type selection for the loops....

@bzbarsky-apple went manually through the changes and used more sane names. Also converted a setter to take a const array to make a const-correctness test better. Overall same change, but with saner names now I hope. Did not try hard for converting auto to const auto &, however some where converted. I am going for "this is slightly better than before".

Do we want to update .clang-tidy? Or was this just a partial update?

Do we want to update .clang-tidy? Or was this just a partial update?

I was unsure if forcing to use loop is something that we want everywhere. My instinct was that this does not always feel natural, but at the same time I 100% accepted all automated changes. Would welcome opinions from reviewers here if we should change clang-tidy to enforce or not.


Add tracing.Tracer to requests instead of context

This prevents traces from showing up for clients that didn't explicitly enable tracing via their ClientOptions. This required moving the tracer into the pipeline and out of the client.
Added TracingOptions type for configuring tracing via pipeline options. Creating a raw pipeline with a tracing provider will now create traces. Don't create a HTTP span if a SDK span hasn't been created. This prevents HTTP spans from showing up when tracing is enabled but a SDK hasn't yet incorporated spans.
Fixes https://github.com/Azure/azure-sdk-for-go/issues/21784

@JeffreyRichter this is different that the approach we originally discussed. Turns out that passing the tracing.Tracer via runtime.NewRequestWithOptions becomes complicated for pagers/pollers as they need to create policy.Request and while we can solve that, the code becomes quite ugly.

Closing in favor of https://github.com/Azure/azure-sdk-for-go/pull/21863


Update ordering of installation as per feedback

Changes order of steps for dependencies
Uses kubectl from local laptop, not inside the cluster as per
best pratice
Installs faas-cli to local laptop
Uses k3sup to get local kubeconfig for laptop

Signed-off-by: Alex Ellis (OpenFaaS Ltd<EMAIL_ADDRESS>

This resolves some of the feedback raised by @c0dyhi11


Static client

Fixes #30
Also disposes for resources such as StringContent and HttpRequestMessage after usage.

Superseded by #34


"Missing write access" npm error, missing UI components after upgrade to v5

Describe the bug
Launching Lens v5 (5.4.3) for the first time after running v4 throws this npm error in a popup before loading the UI:
Could not load extensions: npm WARN checkPermissions Missing write access to /Users/phil/Library/Application Support/Lens/node_modules/lenscloud-lens-extension
npm WARN Lens No description
npm WARN Lens No repository field.
npm WARN Lens No license field.

npm ERR! code ENOENT
npm ERR! syscall access
npm ERR! path /Users/phil/Library/Application Support/Lens/node_modules/lenscloud-lens-extension
npm ERR! errno -2
npm ERR! enoent ENOENT: no such file or directory, access '/Users/phil/Library/Application Support/Lens/node_modules/lenscloud-lens-extension'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

npm ERR!      /Users/phil/.npm/_logs/2022-03-19T01_43_08_118Z-debug.log

I can dismiss the error and the UI loads, but it causes several issues:

The "log in" view is not shown
The "welcome to Lens" wizard is not shown
In a cluster's context menu, the "Add Cluster to Team Space" option is not shown
In the settings view for a cluster, the LENS SPACES and EXTENSIONS categories are missing (only SETTINGS is shown)

How to fix
$ ll /Users/phil/Library/Application\ Support/Lens/node_modules
total 0
drwxr-xr-x  10 phil  staff   320B 19 Oct 23:36 ./
drwxr-xr-x  41 phil  staff   1.3K 18 Mar 20:14 ../
lrwxr-xr-x   1 phil  staff    94B 19 Oct 23:36 kube-object-event-status@ -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/kube-object-event-status
lrwxr-xr-x   1 phil  staff    87B 19 Oct 23:36 lens-license@ -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/license-menu-item
lrwxr-xr-x   1 phil  staff    93B 19 Oct 23:36 lens-metrics-cluster-feature@ -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/metrics-cluster-feature
lrwxr-xr-x   1 phil  staff    79B 19 Oct 23:36 lens-node-menu@ -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/node-menu
lrwxr-xr-x   1 phil  staff    78B 19 Oct 23:36 lens-pod-menu@ -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/pod-menu
lrwxr-xr-x   1 phil  staff    76B 19 Oct 23:36 lens-survey@ -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/survey
lrwxr-xr-x   1 phil  staff    79B 19 Oct 23:36 lens-telemetry@ -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/telemetry
lrwxr-xr-x   1 phil  staff    96B 19 Oct 22:48 lenscloud-lens-extension@ -> ../../../../../../Applications/Lens 2.app/Contents/Resources/extensions/lenscloud-lens-extension

Looking at lenscloud-lens-extension (last row), it's the only symlink that points to a "Lens 2.app" path. That directory does not exist.
rm /Users/phil/Library/Application\ Support/Lens/node_modules/lenscloud-lens-extension

After removing the symlink and opening Lens again, all previously mentioned missing UI components are now shown.
 Since I was trying different versions, I added the version to the name.
To reproduce:

Rename Lens.app to Lens 5.app
Open the app
The above mentioned npm error is thrown and the UI components are missing

After renaming it back to Lens.app, Lens starts without issues.
FWIW, this is the first application I've seen that is unable to handle renaming it.

Hi, i have the same problem:
Could not load extensions: npm WARN checkPermissions Missing write access to /Users/MY_USER/Library/Application Support/Lens/node_modules/lenscloud-lens-extension
npm WARN Lens No description
npm WARN Lens No repository field.
npm WARN Lens No license field.

npm ERR! code EACCES
npm ERR! syscall access
npm ERR! path /Users/MY_USER/Library/Application Support/Lens/node_modules/lenscloud-lens-extension
npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, access '/Users/MY_USER/Library/Application Support/Lens/node_modules/lenscloud-lens-extension'
npm ERR!  [Error: EACCES: permission denied, access '/Users/MY_USER/Library/Application Support/Lens/node_modules/lenscloud-lens-extension'] {
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path: '/Users/MY_USER/Library/Application Support/Lens/node_modules/lenscloud-lens-extension'
npm ERR! }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.

npm ERR!      /Users/MY_USER/.npm/_logs/2022-03-21T08_46_09_011Z-debug.log

I have the same issue.
I did not rename the app, and I was able to verify that the symlink works:
# pwd
/Users/myuser/Library/Application Support/Lens/node_modules

# ls -lsah
total 0
0 drwx------  10 myuser  wheel   320B Feb 22 15:22 .
0 drwx------  54 myuser  wheel   1.7K Mar 23 09:20 ..
0 drwx------   4 myuser  wheel   128B Feb 22 15:22 kube-object-event-status
0 drwx------   4 myuser  wheel   128B Feb 22 15:22 lens-license
0 drwx------   5 myuser  wheel   160B Feb 22 15:22 lens-metrics-cluster-feature
0 drwx------   4 myuser  wheel   128B Feb 22 15:22 lens-node-menu
0 drwx------   4 myuser  wheel   128B Feb 22 15:22 lens-pod-menu
0 drwx------   4 myuser  wheel   128B Feb 22 15:22 lens-survey
0 drwx------   4 myuser  wheel   128B Feb 22 15:22 lens-telemetry
0 lrwxr-xr-x   1 myuser  wheel    94B Feb 22 15:22 lenscloud-lens-extension -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/lenscloud-lens-extension

# ls -lsah lenscloud-lens-extension
0 lrwxr-xr-x  1 myuser  wheel    94B Feb 22 15:22 lenscloud-lens-extension -> ../../../../../../Applications/Lens.app/Contents/Resources/extensions/lenscloud-lens-extension

# ls -lsah lenscloud-lens-extension/
total 16
 0 drwxr-xr-x   4 root  wheel   128B Mar 17 14:12 .
 0 drwxr-xr-x  10 root  wheel   320B Mar 17 14:12 ..
 0 drwxr-xr-x   6 root  wheel   192B Mar 17 14:12 dist
16 -rw-r--r--   1 root  wheel   5.0K Mar 17 13:53 package.json

I found a Stackoverflow post saying we need to "sudo chown -R" on the symlink directory, but that did not help, either.
In my case, my company issued laptop is full of security measures that I likely won't be able to disable, but I can add temporary admin access (e.g. for the sudo command and to install Lens).
Please let me know if there is any further information that I can add that might help.

encounter the same issue.
Solved by deleting all the content of /Users/yourusername/Library/Application Support/Lens and reinstall.


Page comments / discussions

Tasks from #163
Summary
This PR adds the feature to enable comments on pages by setting them up in config and/or frontmatter.
Changes

Create comments components in core for giscus, utterances and disqus

Add giscus react and disqus-react packages in core

Add configs in frontmatter and config.js

configured to set page comments through frontmatter, or specified folders in comments' config. If both are not set then defaults to show on all pages. Frontmatter value takes precedence.

Add documentation @ docs/page-comments

TODO

[x] add disqus support
[x] Create documentation
[ ] publish core
[ ] Add .env.example in packages/template probably not needed ?
[ ] install giscus app in flowershow repo

Screenshot (giscus comments)

@olayway for a new feat would this be a minor version bump or keep it as patch ?

@olayway for a new feat would this be a minor version bump or keep it as patch ?

Definitely a minor bump 😃

@khalilcodes @olayway great work everyone 👏
2 quick small comments:

docs page naming: why page-comments vs comments? comments seems simpler and i assume this applies to blog pages as well etc?
could we get a created date on the page
shall we turn this docs page into a blog post by adding type: Blog (seems worth of it!)
have we turned this on for the site? if so where can i try it out?


Kogito-4980 : Stunner - Palette fixes & improvements

Hi @romartin this PR Fixes

Issue with Double clicking an icon while in the palette. Once detected the event will be canceled

Issue with node text appearing behind the palette in edit mode

Order of Custom Tasks

Issue about throwing Exception while dragging Text Anotation was fixed on my initial work with Data Objects
What is pending is the blurring of the icon while draggon but that can only be fixed with higher resolution images. Let me know what you want me to do.

Artifact can be found here. https://drive.google.com/drive/folders/1m7EQ-8zzc8MRzBQVV8kU2xUdnTg6bvFp?usp=sharing

jenkins run fdb

@inodeman same here, please rebase to that we get FBD :)

Issue with Double clicking an icon while in the palette. Once detected the event will be canceled
I can still reproduce this all thought it is not exactly a double-click.

Click on some anchor in palette, move a bit and click on the same anchor.
Item is in the palette.
What I expected is that item to be reset, same as when I do double-click on same anchor on palette. Could we improve that?

@LuboTerifaj https://issues.redhat.com/browse/KOGITO-4980 please check the linked issues for which you are tester.

Hey @domhanak
Good catch! We had been looking at your comment above and it makes sense, also testing what you mean it other tools, and it behaves different as well. So I think that's not just taking some decision right now about this behavior on top of palette, let's move it to a new ticket, so we can properly discuss it. WDYT?
Thanks!

Hi @romartin the issue is not double click on palette, the issue is that when you click with a selection over an element that is over canvas the selection is pasted into canvas anyway and disregards where the user clicked. That's why I still think this is just a workaround for the root cause of the issue with palette and not a fix at all.
When you do a double click and move the mouse just one pixel in between the clicks the issue is back again.
Anyway reported here: https://issues.redhat.com/browse/KOGITO-5143

Hey @domhanak , @inodeman , the fixes are only for Kogito, right? However, the linked jiras are not. What exactly should I check? Shall I report the separate jiras also for Kogito? Thanks!

oh good point @domhanak @LuboTerifaj
This palette improvements are considered only a kogito specific improvement, so this why the main ticekt is a kogito one: https://issues.redhat.com/browse/KOGITO-4980
On the other hand, we're taking this opportunity to work on palette fixes & improvements in roder to fix some other small issues, reported time ago, so those are not really reported as KOGITO, they're old... anyway, I think it's good to include all those in our reviews and ensure we're fixing as much as possible on the palette.
So @LuboTerifaj , what I can do is moving linked ticket to the KOGITO project, but then it should be @domhanak the tester I guess. In fact , I don't mind who is the reviewer guys, the key point is fixing as much as possible in this task, and so we included some older ticket as well. WDYT?

@domhanak , @inodeman , The reason why I am asking is whether the issues https://issues.redhat.com/browse/JBPM-9495 and https://issues.redhat.com/browse/RHPAM-3172 are supposed  to be fixed in this PR. Thanks!

Verified this PR, checked that following works:

When double clicking on the Palette, the event will be canceled/consumed.
DMN & BPMN :heavy_check_mark:
The order of custom tasks is not sorted when building the palette
BPMN :heavy_check_mark:
When editing text using the inline editor will show behind Properties panel or Expanded Palette
DMN & BPMN :heavy_check_mark:

@LuboTerifaj the issues were just related for reference, and this RP fixes those for kogito editors. Apologies for not being clear enough. I figured the linked issue with @inodeman so all good now.

Hey @LuboTerifaj @domhanak
Thanks for looking at the tickets. So finally the idea is that the main ticket:
https://issues.redhat.com/browse/KOGITO-4980
Its acceptance criteria already take into account the linked tickets. So actually this PR should fix the ticket and the linked ones as well.
Anyway thanks for the review and approval! If any Qs please reach us out.
Thanks!


Tests

New approach for tests:

tests for different parts of goose splitted up (tests for actions, rules etc.)
HTML for parsing persists together with specific test now
using HTTP server instead of single file with HTLM as endpoint for goose

for review @maZahaca

@maZahaca it's ready for review now

Great new test way! Go ahead!


Add Foxy release note for rclpy parameter callback methods

Related PRs:

https://github.com/ros2/rclpy/pull/457
https://github.com/ros2/rclpy/pull/504

The related PRs are merged and this passes CI. I'll go ahead and merge it.


use system xdg-open if exists

on  debian the xdg-open local script doesnt work properly
neither the last official version http://portland.freedesktop.org/xdg-utils-1.1.0-rc1/scripts/xdg-open
I think if it exist on the system we should use this one first

+1


Banding artifacts with envAtlas

Problem with envAtlas, rgbm, maybe it's data loss in the alpha channel?
https://playcanvas.github.io/#/graphics/material-physical
Used video RTX 3060 for laptop:

It seems this is specific to your configuration. This is Win10 + Chrome + NVidia Quadro T2000:

Probably need to check on other RTX 30xx laptop graphics cards. RTX 3090 for desktop - no artifacts.

Any ideas what might be going on here, @slimbuck?

Update.
It turned out that the integrated video card was used.
Enable discrete GPU turned out only in this way:

Thus, there are no artifacts on the RTX 30xx video card.
Artifacts are on the integrated video card: Intel Iris Xe Graphics

Update2.
Another laptop.
ANGLE (NVIDIA Corporation, NVIDIA GeForce RTX 3070 Laptop GPU/PCIe/SSE2, OpenGL 4.5.0)

Hi @maungers,
Thanks so much for reporting this issue! I'll try repro on my side so I can investigate.

Hmm unfortunately I can't repro on my side. I tried on my windows laptop running a GTX 1050 Ti without any luck. I also tried my macbook, swiftshader and linux. No problems on any of them.

Try using nvidia control panel, switching Chrome to integrated GPU, then restart computer, and try checking webglreport.com if it reports Integrated. Bear in mind, Chrome might still be running as a process, so it will not switch GPU, so restart or force close through task manager is required.
On macs, often integrated GPU is used by default.
Also, uninstalling nvidia drivers and disabling it, will force integrated GPU.
And make sure you open it using actual laptops screen, not external monitor, as it is almost always will run over dedicated GPU.

@GSterbrant has reproduced the originally reported issue on the Intel Iris Xe integrated GPU. It is unrelated to PlayCanvas specifically. With DX11 (doesn't happen with the OpenGL ANGLE backend), the GPU is unable to filter the alpha channel correctly. Our intention is to report this to Intel directly.

Closing because this is a driver issue - not a PlayCanvas bug.


make first route URL element for query function unique in order to be able to proxy reliably through HTTP load balancers

/query -> /cswi/query or /csw-js-api/v1/query
because loadbalancers and HTTP proxies in the line of the request before have to sort to geoservers, sos, pycsw, wps, dev, test,prod docs etc basedd on URL patterns/elements

/ingester/query


Apply format.space setting to more languages

I don't thinks it's a bug, but it would be really awesome if we could chose, or extend on which language this setting (format.space) apply.
I was trying to create a custom ident style for C++, but it only seems to apply on:

php
javascript
java
css

I have to say I'm really sad it doesn't work for C++, because that was exactly what I wanted, and it already work for some others language.
Currently, I can change the editor language to one of those listed, but this is really frustrating, especially for such a "small feature".
Thank you.

Please file this issue against the formatter extensions you are using.


Integration tests for convertion to PDF does not accept PDF-1.6 format

I'm running LibreOffice <IP_ADDRESS> 40(Build:2), which outputs PDF-1.6 when converting to PDF.
Integration test's assertions at lines 29 and 56 explicitly check for PDF-1.5 ( assert start == b"%PDF-1.5\n") and therefore fails.

#58 fixes it


Simulation error: Array index is out of range.

Seen this a few times, it seems to be issue with IPT2 but would be good if we could confirm that is the case. We often get these errors sent to use because they mention TM:PE, for example:
Simulation error: Array index is out of range.
at ImprovedPublicTransport2.Detour.VehicleManagerMod.ReleaseWaterSource (uint16,Vehicle&) <0x0001d>
at VehicleManager.ReleaseVehicleImplementation (uint16,Vehicle&) <0x00220>
at TrafficManager.Custom.Manager.CustomVehicleManager.CustomReleaseVehicle (uint16) <0x0008d>
at TrafficManager.Custom.AI.CustomCarAI.CustomSimulationStep (uint16,Vehicle&,UnityEngine.Vector3) <0x007ac>
at TrafficManager.Custom.AI.CustomCargoTruckAI.CustomSimulationStep (uint16,Vehicle&,UnityEngine.Vector3) <0x00196>
at (wrapper dynamic-method) VehicleManager.SimulationStepImpl_Patch0 (object,int) <0x003d9>
at SimulationManagerBase`2.SimulationStep (int) <0x0004c>
at VehicleManager.ISimulationManager.SimulationStep (int) <0x00024>
at SimulationManager.SimulationStep () <0x00693>
at SimulationManager.SimulationThread () <0x0018a>

There several variants of the error - see end of this log file:
Windows - IPT2.txt
Save game exhibiting the issue: https://steamcommunity.com/sharedfiles/filedetails/?id=1812178640
From steam discussion: https://steamcommunity.com/app/255710/discussions/0/1640917367351362143
Several other examples can be found in Symptom 4 in our troubleshooting guide on the Array index issue where I had originally thought it was caused by Climate Control, however the save above has never used that mod. As such, it seems to be a conflict between IPT2 and TM:PE?
If there's any way to track down what specifically is causing it, it would be big help for people experiencing the issue.

In Disasters DLC tank reservoirs can store water which is dispensed with the help of water trucks, which might explain why vehicles and water sources are mentioned.

@aubergine10 I think you can close this issue.

user is subscribed to TM:PE LABS
user has enabled More Vehicles mod which is incompatible with IPT 2 because of hardcoded vehicle number

More vehicles workshop id:<PHONE_NUMBER>
Could not find that in the log file.

Line 6168

Ah, yeah, I see it.


Fix ProfileClient creation with callable factory

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Related tickets
N/A

Documentation
N/A

License
MIT

The ProfileClientFactory added in #136 was not tested and contained an error while decorating a callable factory. This is now fixed with tests.

Fixed!


File Issues for unclear bits of documentation

For those of us who do wasm and things regularly we're going to miss out on what is unclear or not explained well. Help us by filing issues on things that are unclear in the book! It's an easy task to get involved with, opens up more things for us to fix, and helps us find out what still needs improvement!

First suggestion: Put a link to "the book" in your message?


[SMALLFIX] Java 8 Improvement

@xiaochen000 Our proto files are auto-generated by running bin/alluxio protoGen and we do not modify those auto-generated docs. Please view https://www.alluxio.org/docs/master/en/contributor/Developer-Tools.html#modifying-a-protocol-buffer-message for more details.
I will close this PR. Thanks for your contribution!


AccountInfo for on-chain price consume

Hi - could you give an dummy example on how to construct AccountInfo for on-chain price consume? Thank you!

Hey, I believe in solana the accounts that are passed to the instruction processor are AccountInfo. We will soon add more examples to this repo.

Hi, thank you! just need some dummy examples on the accounts! Looking forward to your updates.

@ali-bahjati , I am trying to create an instance of AccountInfo, but not sure how to do. Do you mind give a bit help? Below is I got so far. Thanks so much!
let key = String::from("45gMLT6mGfa8aJFeA97Cg65tp1qJZriHkmkucd3TbQZ8");
let pub_key: Pubkey = Pubkey::from_str(&key).unwrap();

let price_account_info: AccountInfo = AccountInfo {
    key: &pub_key,
    is_signer: true,
    is_writable: false,
    lamports: ? ,
    data: ?,
    executable: false,
    owner: &pub_key,
    rent_epoch: 2,
};

Hi @luke4u,
You should not create it by yourself, in solana when you call a program you pass account info s as argument and then you can just use them right away. this example is a bit old but definitely helpful with only solana. However I recommend you to use Anchor which makes it simpler. You can start by reading their book here

thank you so much for the help!


Introduction of longitudinal, lateral and angular offset to lane in agent's observation space

Following up on issue #307, I included inside the Kinematic observation space of agents certain offsets to lanes.
Motivation: for continuous agents to learn more robustly to stick to their own lanes, without losing generality.
A follow up to this work could be the creation/adaptation of a test environment with a reward function to showcase the benefits of including this information as part of the observation space input of the agent's network.
I will push one more commit to this request, which will include notes in the documentation of the new features in the Kinematics observation. In the meanwhile, please advise on anything that might be missing from this pull request.
Thank you.

Added documentation for all features to kinematic features.
I found myself a little confused the first time I tackled kinematics observations. I needed to dig through the source code to see what I could observe.
So I added a table with short, clear descriptions of the features. Hope it fits the documentation style.

Looks good, thanks!
And also for the added documentation, nice touch :)

Following up on issue #307, I included inside the Kinematic observation space of agents certain offsets to lanes.
Motivation: for continuous agents to learn more robustly to stick to their own lanes, without losing generality.
A follow up to this work could be the creation/adaptation of a test environment with a reward function to showcase the benefits of including this information as part of the observation space input of the agent's network.
I will push one more commit to this request, which will include notes in the documentation of the new features in the Kinematics observation. In the meanwhile, please advise on anything that might be missing from this pull request.
Thank you.
Hi,
Have you had success with continuous control on the intersection environment? I'm doing demo learning now and need to collect demo data, but tried DDPG and SAC and didn't get good results. I would appreciate it if you could share your code. My email<EMAIL_ADDRESS>


Blank string load default "development"

In some cases, such as Docker, environment variables receive the value empty string. It is preferable to set "development"

Interesting!  We run Docker as well for our stuff and haven't had that happen, but it's a problem nonetheless.
Quick question, when was String.prototype.trim added!?  I had no idea!


[Steam Deck OLED] Always boots to unresponsive black screen

Describe the bug
The system will boot into one of two states:

Black screen with a single white "_" (underscore) character in the top left. Other TTYs cannot be accessed in this state -- any keyboard or gamepad inputs do not do anything. The system is still accessible through SSH.

Black screen with no underscore character i.e. completely blank. Other TTYs can be accessed in this state, but attempting to run steamos-session-select plasma or gamescope causes the screen to freeze and the system to become unresponsive to any keyboard or gamepad inputs. The system is still accessible through SSH.

Once, after running ujust toggle-autologin, rpm-ostree rebase ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:39-20240205, systemctl reboot, the system entered this state:

Black screen with no underscore character i.e. completely blank. Other TTYs can not be accessed in this state. The system is not accessible through SSH.

The system returned to intermittently booting between states 1 and 2 after three force-shutdowns.
What did you expect to happen?
I expect the system to be accessible in gaming mode upon startup.
Output of rpm-ostree status
State: idle
Deployments:
● ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:latest
                   Digest: sha256:cc09fb538f3d06d1d62b12733e43f06b53ab305c5733c0bc04ca73dbdbb87f7c
                  Version: 39.20240213.0 (2024-02-14T09:29:32Z)
                Initramfs: '"-I /etc/crypttab /etc/modprobe.d/amdgpu.conf /etc/modprobe.d/deck-blacklist.conf"'

  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:39-20240214
                   Digest: sha256:cc09fb538f3d06d1d62b12733e43f06b53ab305c5733c0bc04ca73dbdbb87f7c
                  Version: 39.20240213.0 (2024-02-14T09:29:32Z)
                Initramfs: '"-I /etc/crypttab /etc/modprobe.d/amdgpu.conf /etc/modprobe.d/deck-blacklist.conf"'

Hardware
Steam Deck OLED
$ cat /sys/devices/virtual/dmi/id/product_name
Galileo

Extra information or context
I do not know a workaround for this issue. My device has been rendered completely unusable.
I noted this issue first about 16 hours ago. In the ~24 hour period between when:

I last used the device in a working state
I first experienced the issue after running ujust update then systemctl reboot

the device sat on my desk undisturbed.
I have tried restarting the system after running the following commands:

ujust update on bazzite-deck-gnome:latest
rpm-ostree rebase ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:39-20240214
rpm-ostree rebase ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:39-20240213
rpm-ostree rebase ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:39-20240212
rpm-ostree rebase ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck-gnome:39-20240205
rpm-ostree rebase ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck:39-20240214
rpm-ostree rebase ostree-image-signed:docker://ghcr.io/ublue-os/bazzite-deck:39-20240205

The issue persisted across every version. This is particularly odd, because the device did not have this issue on 39-20240212 and prior.
Running gamescope in tty2 provides this output before the screen freezes:
vblank: Using timerfd.
wlserver: [backend/headless/backend.c:67] Creating headless backend
wlserver: [libseat] [libseat/backend/seatd.c:64] Could not connect to socket /run/seatd.sock: No such file or directory
wlserver: [libseat] [libseat/libseat.c:76] Backend 'seatd' failed to open seat, skipping
_

I am not successful in reproducing this on my OLED, I'll see if I can get any other hardware to do it.

Thanks. I'm baffled by this issue. I've spent some man-hours trying to inspect further into this but never really got anywhere since I lack domain knowledge
Fortunately I've at least figured out a workaround to un-brick my device.
To enter gaming mode from ttyN:
gamescope-session-plus steam

To enter desktop mode from ttyN:
XDG_SESSION_TYPE=wayland dbus-run-session gnome-session

"Switch to Desktop Mode" and "Return to Gaming Mode" buttons no longer work so this requires plugging in a keyboard to return to tty but I was able to launch a game and adjust audio with device buttons.

User Alex&Indie on the #bazzite discord channel notes this same issue happening on their LCD model

I'm on the discord as bear.fox, I'm having the same issue as above on my LCD Deck. Been on Bazzite since August last year. I was on a December or January bazzite-deck build before I decided to system update via gamemode system settings. Since I have had a black screen or blank tty. I can use the controls to navigate menus and boot into desktop mode. I have tried rollbacks and a rebase to bazzite-deck39:20240113.0 (used the exact command from the docs actually). Unfortunately the issue still prevails. Here is my .gamescope-stdout.log
https://pastebin.com/5ePZB3UP
Thanks for your time.

On a cold boot after rebasing to the Jan 13th build, I'm actually getting a message that Jupiter fan control service failed to start. I  unhid grub and am able to boot into the more recent deployment build released yesterday, but the black screen gamescope issue persists.

Upgraded to latest build (24th Feb), same behavior. Debased to 5th Feb build, same behavior. Disabled decky loader, same behavior.
I have found that if I enter desktop mode via blind menu navigation or tty, then from there the .desktop file to return to gamemode will consistently load gamescope/gamemode successfully.

I am having issues with this as well

A reinstall reduced this error happening dramatically. It only appears once in a while.

I also have this issue on an all AMD PC.
However it seems to happen a lot more with DisplayPort than with HDMI. I'm using the deck-gnome image.


v1.9: Update Go to 1.15.13

test-backport-1.9

test-1.12-netnext

test-1.17-5.4

test-1.19-4.9

test-backport-1.9

test-1.14-4.9

test-1.17-5.4

The k8s-1.17-kernel-5.4 job hit #16050.

test-1.12-netnext


Establish best practice for importing context

We should unify the code base to always use the built in context package for our own interfaces or use. The following exceptions must apply...

Implementing an interface defined in a vendor using golang.org/x/net/context
Implementing an interface generated by 3rd party code using golang.org/x/net/context

This could also be called the grpc exception until we are using a version of grpc which uses the built in context. Most of these exceptions will be limited to the service code.
Related to #983

All new interfaces should use context. #983 is problematic in that it holds things back. Only generated code should use the old package to satisfy the grpc interfaces.

@dmcgowan Is there anything we need to do here? For some reason, we seem to have avoided the main issues.

Closing this. We have a fairly consistent approach that minimizes use of golang.org/x/net/context. If there are other issues or we think we need to document this, let's re-open this issue.


Help: I am stuck trying to support Qtum on BTCPayServer

I am trying to use QTUM with NBXplorer. (Backend of BTCPayServer)
NBXplorer keeps attempting to connect to my mainnet node and it is choking on some block headers between block 3999 and 5998
2020-06-18T12:56:39Z received: getheaders (773 bytes) peer=10
2020-06-18T12:56:39Z getheaders 3999 to end from peer=10
2020-06-18T12:56:39Z sending headers (434365 bytes) peer=10

It is just repeating these over and over every minute. So I think my NBitcoin (dep of NBX) implementation was flawed.
Does any of the consensus params here: https://github.com/MetacoSA/NBitcoin/blob/2330a5bf902bc44a78b779de88f14220954c6ab6/NBitcoin.Altcoins/Qtum.cs#L152-L191
Look suspicious?
Also,
I made the blockheader extended to include
32 byte hash,
32 byte hash,
36 byte outpoint (hash+vout)
and Script (push_data + signature)
Added on the default class: https://github.com/MetacoSA/NBitcoin/blob/2330a5bf902bc44a78b779de88f14220954c6ab6/NBitcoin.Altcoins/Qtum.cs#L43-L143
So I think this should work...
I think BTCPayServer support would be great... I think I am missing some parameter or something that maybe someone familiar with this can help.

Maybe something happened around block 4000 that needs to be accounted for as a special case?
If anyone has an idea, please let me know. Thanks.

I see 5001 block starts proof-of-stake... and from that point on the HASH256 of the header no longer is less than the target in the BITS section of the header... perhaps that is what is causing the issue.

Yes, I was going to suggest that. Qtum blocks 1 - 5000 are Proof of Work genesis blocks. Block 5001 starts Proof of Stake consensus, adding a coinstake transaction to each block, with the coinbase transaction empty.

Yes. I checked the new transaction types, and as far as serializing and deserializing is concerned, they could be parsed by normal Transaction logic for Bitcoin transaction.
NBXplorer makes no consensus verifications and fully trusts the node it connects to, so it doesn't even check PoW for the block.
iirc it merely checks that GetHash of BlockHeader is equal to the prevblockhash of the next header.
I manually checked blocks before and after 5001 and they still use HASH256 (sha256 x 2)...
I would be lead to believe maybe it was choking on the coinstake tx if it wasn't choking on the initial getheaders loop. (the headers don't contain anything and all it does is hash and check that is equal next header's prevblock)
Perhaps the P2P message for headers message changed for blocks after 5000 and that is causing error? (maybe I need to change the P2P layer on NBX side?)

There is no change in P2P messages. Please share full logs from the node (with debug=all), also logs from NBXplorer if any.

I will try to get logs, for now I think I fixed the problem:
https://github.com/MetacoSA/NBitcoin/pull/874

The fix looks good, the signature size is variable.

I will let you know if this fixed the problem.
It was because I thought Script would deserialize as PUSH_DATA + SIG, but it actually was expecting VARINT_SCRIPTSIZE + PUSH_DATA + SIG, so it broke (I think)
So instead I am using the ReadWriteAsVarString which will read VARINT_LENGTH + LENGTH_BYTEARRAY instead.
So I think this is the issue. I will come back with logs if this doesn't work.

Ok, this is a mixture of NBX logs and qtum logs.
Around this timestamp
2020-06-19T02:57:36Z
Loading Chain... is logged in NBX.

This was done with the fix included.
output.txt

Seems NBX is requesting the same headers over and over again, the node seems to send the requested headers correctly, so I would check on the NBX end why it does not receive/process the headers correctly, maybe by adding some debug code in the received headers processing code to see why it's not processed correctly.

I found the issue:
The Outpoint class has a static constant Zero I was using.
I assumed it was (uint256)(0) and uint.MaxValue, but actually it was (uint)(0) even though the IsNull check checks against MaxValue.
I switched the default and it started working fine.
https://github.com/MetacoSA/NBitcoin/pull/875/files
Anywho, the current release of NBXplorer now seems to work for simple utxo based sends.


AppVeyor build is flaky

I have seen a failure like this several times:
Build started
git clone -q https://github.com/serde-rs/serde.git C:\projects\serde
git fetch -q origin +refs/pull/853/merge:
git checkout -qf FETCH_HEAD
Running Install scripts
appveyor DownloadFile https://win.rustup.rs/ -FileName rustup-init.exe
Error downloading remote file: One or more errors occurred.
Inner Exception: The request was aborted: Could not create SSL/TLS secure channel.
Command exited with code 2

appveyor docs say:

this most probably means that build worker and server could not agree on SSL protocol, because server does not support modern protocol with highest security. If you understand this security risk and trust that server, you can relax SSL client settings in build PowerShell session by the following statement called before Start-FileDownload

[Net.ServicePointManager]::SecurityProtocol = 'Ssl3, Tls, Tls11, Tls12'

Which is weird, b/c it works most of the time.

stackoverflow says it could also be the installed .net frameworks fault.


Attribute error

I am getting this attribute error when i run PySight.py. Checked PySight_settings and  the attribute appears to be there.
-why is it looking for that value if i have both misp proxy and isight proxy set to false?
Traceback (most recent call last):
File "PySight.py", line 873, in
result = isight_search_indicators(PySight_settings.isight_url, PySight_settings.isight_pub_key,PySight_settings.isight_priv_key, PySight_settings.isight_last_hours)
File "PySight.py", line 746, in isight_search_indicators
return isight_prepare_data_request(base_url, search_query, public_key, private_key)
File "PySight.py", line 724, in isight_prepare_data_request
result = isight_load_data(a_url, a_query, header)
File "PySight.py", line 625, in isight_load_data
PySight_settings.proxy_address)
AttributeError: module 'PySight_settings' has no attribute 'proxy_address'

I think i got it working. however at the end of the process i either get it to core dump or get a messaged 'killed'. There was too much happening on my screen and my eyeball wasnt fast enough to catch anything significant.  Any ideas?

I was able to see it pulling down the events. I set the log mode to DEBUG and i did a tail -f output.log. Looks like it stopped here:
2020-04-29 02:42:19,454 - PySight_settings - DEBUG - Processing report 20-00007633
DEBUG [api.py:2242 - _prepare_request() ] {'User-Agent': 'PyMISP 2.4.123 - Python 3.6', 'Accept-Encoding': 'gzip, deflate', 'Accept': 'application/json', 'Connection': 'keep-alive', 'Authorization': 'RCA3rzX3q8tozY3NTw2yzwU96NnFnWscEnOsR8HD', 'content-type': 'application/json'}
DEBUG:PySight_settings:Processing report 20-00007626
threading*
Killed
However, when i looked at the events in the GUI, I could not find any of the indicators.

The error message meant that you have not set the proxy settings in the config file
If no events show up in the GUI, that means it did not run correctly.

I was able to get it to work. I had to disable threading on the script.
V/R
Help Me, Help You
Douglas P Molina, MBA, CISSP, C|EH, CySA+,CCSK,GCTI
U.S. Department of Veterans Affairs
Office of Information Security (OIS)
Cyber Threat Intelligence (CTI)
Work Phone:<PHONE_NUMBER>
Mobile Phone:<PHONE_NUMBER>
UNCLASS: douglas.molina<EMAIL_ADDRESS><EMAIL_ADDRESS>From: Alexander J<EMAIL_ADDRESS>Sent: Saturday, May 2, 2020 3:53 PM
To: deralexxx/PySight2MISP<EMAIL_ADDRESS>Cc: Molina, Douglas<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [deralexxx/PySight2MISP] Attribute error (#24)
The error message meant that you have not set the proxy settings in the config file
If no events show up in the GUI, that means it did not run correctly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/deralexxx/PySight2MISP/issues/24#issuecomment-623011714, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKD2KYUIRPOSMWMMF3JE6U3RPSBY5ANCNFSM4MSNIAJQ.


[stable/mariadb] Add initContainer for adjusting volume permissions

Signed-off-by: tompizmor<EMAIL_ADDRESS>
What this PR does / why we need it:
Sometimes the built-in Kubernetes functionality to adjust file permissions in the volumes is not available (securityContext). If that is the case and we want to run non-root containers, we have to adjust the permissions of the volume previously with a privileged init container.
Which issue this PR fixes
Special notes for your reviewer:
Checklist

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/chart])

/hold

/hold cancel

/lgtm


Allow nested classes in freezed classes

Let's say we have a Tweet class that uses Reply as a field. This works on non freezed classes
@json
class Tweet {
  DateTime? created_at;

  String? tweet;

  Reply? reply;
}

@json
class Reply {
  DateTime? created_at;

  String? reply;
}

When I convert this to freezed class I see that generated readTweet looking for fromJson methods in Reply which not exists.
@freezed
class Tweet with _$Tweet {
  @json
  const factory Tweet({
    DateTime? created_at,
    String? tweet,
    Reply? reply
  }) = _Tweet;
}

@freezed
class Reply with _$Reply {
  @json
  const factory Reply({
    DateTime? created_at,
    String? reply,
  }) = _Reply;
}
// ---- .g ----
extension ReadTweet on Crimson {
  Tweet readTweet() {
    DateTime? created_at;
    String? tweet;
    Reply? reply;

    loop:
    while (true) {
      switch (iterObjectHash()) {
        case -1:
          break loop;

        case -3141059728106243719: // created_at
          created_at = skipNull() ? null : DateTime.parse(readString());
          break;
        case -5949187214024354806: // tweet
          tweet = readStringOrNull();
          break;
        case -777431743796252409: // reply
          reply = Reply.fromJson(read());
          break;

        default:
          skip();
          break;
      }
    }
    ...
  }
  ...
}

Wondering if this feature exists or is planned already?
Definitely something I need before making the switch over to Crimson

Seems to work correctly if you remove the json_serializable package or if you add the fromBytes method.


Release 0.3.24

@JuliaRegistrator register(branch=release)

Registration pull request created: JuliaRegistries/General/15338
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.3.24 -m "<description of version>" 2c566fcb18892ea5bb97107534e323675547803d
git push origin v0.3.24


Typo "Azure OpenAI service"→"Azure OpenAI Service"

https://learn.microsoft.com/en-us/azure/ai-services/openai/how-to/managed-identity
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/ai-services/openai/how-to/managed-identity.md
#PingMSFTDocs

@ChrisHMSFT
@aahill
@patrickfarley
@nitinme
@mrbullwinkle
@laujan
@eric-urban
@jboback
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Genetic Algorithm should be built as a separate entity from classes.

Right now, the genetic algorithm is implemented as a method under Family class.
But actually the following seems more suitable:
Genetic algorithm will have an individual, a crowd (which are objects to be inherited by Network and Family classes, respectively) a mutation that can act on that individual object, a random number generator, and a mutator.
A genetic algorithm can be coded as a wrapper of these objects, and Probe's can be used accordingly.

This is partially done. GA has class definitions and a machine that is independent of the object in question. The objects should be inherited from GA classes Elements and Ensemble.


Add several tslint rules

Adds several tslint rules as per #165.

Okay, I admit it, I claimed this one so that I wouldn't be the one who had to review every single whitespace change across 40+ files.

I merged a couple other of your PRs so this needs a rebase...

Rebased

Looks good!


No mention of orphanCollections in HTTP docs for creating a graph

The documentation for the HTTP endpoint POST /_api/gharial does not mention anything about the property orphanCollections that can be used inside the JSON content of the request.
https://www.arangodb.com/docs/stable/http/gharial-management.html#create-a-graph
I see this happening on 3.5 and 3.4.
This property should be documented because it can be used to specify the orphan collections of the Graph when it is created.
Example:
A request POST /_api/gharial with the following JSON content:
{
	"name": "myTestGraph",
	"orphanCollections": ["myclx"]
}

Gives the following result:
{
    "error": false,
    "code": 202,
    "graph": {
        "_key": "myTestGraph",
        "numberOfShards": 1,
        "replicationFactor": 1,
        "isSmart": false,
        "edgeDefinitions": [],
        "orphanCollections": [
            "myclx"
        ],
        "_rev": "_Zzwfq_e--_",
        "_id": "_graphs/myTestGraph",
        "name": "myTestGraph"
    }
}

The current documentation:

The documentation should be updated to add the orphanCollections property to the JSON object of the request.

Well spotted. Thanks for the letting us know @DiscoPYF. I created a PR for 3.6 to test the changes:
https://github.com/arangodb/arangodb/pull/10807


Value function only called once for "backgroundColor"

I have 4 elements that I am trying to animate at once, but I also need to choose an appropriate background-color for each element. I made a thinned out and reproducible example here.
To skip the demo, here is what is going on...
var $boxes = $(".box");  // Say we have 4 elements here

$boxes.velocity({
  backgroundColor: function(i) {
    console.log("backgroundColor[" + i + "]");
    return (i % 2 == 0) ? "#ccc" : "#f00"
  },
  zIndex: function(i) {
    console.log("zIndex[" + i + "]");
    return 0;
  }
}, {
  duration: 500,
  queue: false
});

If I want to change the background-color based on the element's index, I would use a value function like I did above. Obviously the zIndex value function is not necessary because it just returns 0, but it helps prove my point.
When I run this, my log looks like:
backgroundColor[0]
zIndex[0]
zIndex[1]
zIndex[2]
zIndex[3]

So the backgroundColor function only ran once! What's more, because it only ran once, all 4 elements get assigned the color that was resolved from the single value function call. In this case, i == 0 so all elements are #ccc.
Another thing to point out is that if I change return (i % 2 == 0) ? "#ccc" : "#f00" to return (i % 2 == 0) ? "notvalid" : "#f00"
my logs look like:
backgroundColor[0]
backgroundColor[0]
zIndex[0]
backgroundColor[1]
zIndex[1]
zIndex[2]
zIndex[3]

In this case, [0] stays white/clear and all the others turned red.
It is if the backgroundColor value-function is saying "Once we find a valid color, apply it to everything else. No need to call this function again."
How can I accomplish what I am trying to do? I think this is a bug, but am I wrong and this is expected behavior? I know I could hack the system and break out my elements into 2 separate arrays and act on them separately, effectively circumventing the value function, but I would prefer a better solution.
Thanks!

@kenwheeler  Looks like an actual bug to me.

I have traced what I think is the issue to here.
It looks like for colors it is passing the regex test and get the endValue right away and then generates a backgroundColorRed, backgroundColorGreen, and backgroundColorBlue property based on it. But the problem is that any logic then the original value function is now gone and it is as if I originally said
{
  backgroundColorRed: 204,
  backgroundColorGreen: 204,
  backgroundColorBlue: 204
}

without value function logic.

I just re-did my example now that I know that backgroundColor is jut a convenience for the rgb properties. Now when I run
var $boxes = $(".box");  // Say we have 4 elements here

$boxes.velocity({
  backgroundColorRed: function(i) {
    console.log("backgroundColorRed[" + i + "]");
    return (i % 2 == 0) ? 204 : 255
  },
  backgroundColorGreen: function(i) {
    console.log("backgroundColorGreen[" + i + "]");
    return (i % 2 == 0) ? 204 : 0
  },
  backgroundColorBlue: function(i) {
    console.log("backgroundColorBlue[" + i + "]");
    return (i % 2 == 0) ? 204 : 0
  },
  zIndex: function(i) {
    console.log("zIndex[" + i + "]");
    return 0;
  }
}, {
  duration: 500,
  queue: false
});

To me, this is a usable workaround. I am not excited about it, but I believe it is better than trying to divide up my elements arrays and call more than on .velocity() function in the end.
I would still love to see the convenience function work somehow though :)

Agreed - the convenience attribute should behave consistently with the direct attributes - thanks for tracking it down, hopefully it'll be a simple case of moving some code from one place to another :-)

When I initially looked at this I thought it was going to be a "wontfix" - then realised that simply skipping any properties where both start and end are functions (or only start if no end) would let it work the same way and be somewhat more logical - plus a minor optimisation on the way ;-)


define toplevel instance and class variables on the correct scope

Motivation

Fixes a debug enforce after #6278.
Test plan

The included testcase was generated at the commit prior to #6278, which is where the crash was introduced.  It's not obvious to me that Sorbet is defining the variables in the "right" place for Ruby's runtime behavior, but at least we're not regressing things.

The included testcase was generated at the commit prior to #6278, which is where the crash was introduced

Thanks! This was going to be my only question.


feat: add format tool

Signed-off-by: mango<EMAIL_ADDRESS>
Type of change:

[ ] Bugfix
[x] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

Add a formatting tool and format project files. Resolved #884
Pre-submission checklist:

[x] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Codecov Report

Merging #885 (aac3c65) into master (77383b8) will decrease coverage by 0.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #885      +/-   ##
==========================================
- Coverage   32.05%   31.75%   -0.30%
==========================================
  Files          70       70
  Lines        7740     7740
==========================================
- Hits         2481     2458      -23
- Misses       4984     5000      +16
- Partials      275      282       +7

Impacted Files
Coverage Δ

pkg/apisix/plugin.go
80.00% <0.00%> (-20.00%)
:arrow_down:

pkg/apisix/cluster.go
28.73% <0.00%> (-3.85%)
:arrow_down:

pkg/apisix/route.go
36.91% <0.00%> (-2.02%)
:arrow_down:

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 77383b8...aac3c65. Read the comment docs.

Hi @lingsamuel ,lint-checker-ci failed, and some hyperlink got status 500, maybe we should fix first.

Hi @lingsamuel ,lint-checker-ci failed, and some hyperlink got status 500, maybe we should fix first.

But I can link to those websites.


ARROW-6731: [CI] [Rust] Set up Github Action to run Rust tests [WIP]

Did you see the suggestion by @fsaintjacques that we use Docker based test?
<EMAIL_ADDRESS>I agree with the suggestion. If we use the approach, we can reuse the workflow file in #5530 for all languages instead of creating a Rust specific workflow file.

Agree that a docker-compose-based build would be better. We may be able to cache some cargo dependencies that way, too, for faster builds

I started on this before that conversation started, and I agree. Closing this.


Extract updater/proxy tags from Dockerfiles

This could allow Dependabot to keep the Dockerfile up-to-date, and we
wouldn't need another build step to use them.
When running ncc build the referenced Dockerfile gets copied into
dist/main so it's available to the code running in actions.
I think we have two main approaches of updating these tags, as here we allow dependabot to update them in a Dockerfile, pulling the latest daily. We could also have a separate build script that extracts these values and writes them as string constants in a typescript file if we don't feel comfortable doing this dynamically.
The other general approach would be to have a workflow that sends a workflow dispatch to this repo when the proxy and updater images are updated, this triggered workflow would then update some file with the tags.
There would be a lot more custom moving parts with the second approach that I'm not sold on but would love to hear your thoughts/concerns/objections with these approaches.
Following on from: https://github.com/dependabot/updater-action/pull/72
Working towards: github/dependabot-updates#2102

I'm going to close this out as #84 extracts this change and puts it ahead of #72 to we can release all the code restructuring as a minor release and then make the actual container change from the v1 ref to a sha as an isolated minor release so its easier to test before updating the v2 tag.


wails: add default layout

We are adding a default layout for wails front-end screens.

Codecov Report
Patch and project coverage have no change.

Comparison is base (b4616cc) 37.51% compared to head (3514bcd) 37.51%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #227   +/-   ##
=======================================
  Coverage   37.51%   37.51%
=======================================
  Files          12       12
  Lines         829      829
=======================================
  Hits          311      311
  Misses        481      481
  Partials       37       37

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Auto discover a "general text device" to set the last alarm picture when alarm occurs

From #7

Hello @duanemck , did you have time to review that one ?
How can I help ?

I can set it up to set a sensor to the picUrl when an alarm occurs but it won't be much use to you. The picUrl can only be viewed with an authenticated session on the API.
The alarm MQTT message sends the picture as a Base64 encoded image, I can spin that out into a standalone sensor too but you'll need to figure out how to decode and display that in Domoticz.

Thank you.
The idea would be to authenticate, decode and download the image so that Dz could send the snapshot with Telegram, for instance.

Is there a specific device type in Domoticz that needs to be created? Or just a regular switch (on/off) that is turned to ON when an alarm is triggered?

OK, if you have the ability to authenticate before downloading then it should work. Let me see what I can do.
Just to check that we're aligned, the "general text device" should be a separate device from the camera? i.e. not a property of the camera?

https://www.domoticz.com/wiki/Developing_a_Python_plugin#Devices
Paragraph 4.5
Device type general (243), text (19)
I hope it helps.

OK, I'll give that a read and see how it maps to the HA Autodiscover messages.

I don't see a way to create a General Text device through the MQTT auto discovery

The links you sent above are for developing custom Domoticz plugins and unfortunately that's not something I am able to do as I don't have a Domoticz instance and don't know Python very well.

What I have done so far is add an additional image sensor/attribute to the Camera device that contains the last alarm image as Base64 as well as a regular sensor/attribute with the PicUrl.
I'm not sure if you're able to use either of those? I will release that change shortly.

I'm not sure if you're able to use either of those? I will release that change shortly.

I don't know yet .... how the 2 infos picURL and Base64 image are different. :

picUrl is a link to the image on the ezviz API, you can download from there if you can authenticate.
Base64 is a way to encode binary data as strings. So it's the actual image data, you will need to convert it back to a binary file though.
HA supports it natively, but this should give you an idea of how it works:
https://codebeautify.org/base64-to-image-converter

Hello @duanemck ,
While using the latest docker Image :
In this auto discovery topic : "dz_ezviz/camera/ezviz_D67745001/ezviz_D67745001_camera/config" , I can see that specified as real status topic :
"
{
"image_encoding": "b64",
"topic": "ezviz/status/D67745001/image/set",
....
But I don't have this topic anywhere...
Beside that, the autodiscovery prefix for Last Alarm and last alarm URL is not working, no devices are created in DZ

Also, for inspiration, Here is an auto discovery topic for a zwave device that creates a TEXT device in Domoticz :
Could it be used for the base 64 image ?

{
"value_template": "{{ {22: "Window/door is open",23: "Window/door is closed",5632: "Window/door is open in regular position",5633: "Window/door is open in tilt position"}[value_json.value] | default(value_json.value) }}",
"icon": "mdi:alarm-light",
"state_topic": "zwave/Chambre_Arthur/multi_capteur_fenêtre_ch_Arthur/113/0/Access_Control/Door_state",
"json_attributes_topic": "zwave/Chambre_Arthur/multi_capteur_fenêtre_ch_Arthur/113/0/Access_Control/Door_state",
"device": {
"identifiers": ["zwavejs2mqtt_0xe573130e_node50"],
"manufacturer": "Philio Technology Corp",
"model": "Slim Multi-Sensor (PIR/Door/Temperature/Illumination) (PST02A)",
"name": "Chambre Arthur-multi capteur fenêtre ch Arthur",
"sw_version": "1.16"
},
"name": "Chambre Arthur-multi capteur fenêtre ch Arthur_notification_access_control_door_state",
"unique_id": "zwavejs2mqtt_0xe573130e_50-113-0-Access_Control-Door_state"
}


Search: combine results of single line in search window

Search results on the same line are displayed on the same line in the search pane, but it happens to me quite often that I'm searching for something that occurs many times on a single line, which gets displayed in multiple lines in the search window.
Example:
with the following small dummy code
some_default = "default"

class Dummy:
    def __init__(self, some_variable, some_other_variable):
        self.some_variable = some_variable
        self.some_other_variable = some_other_variable

def dummy_test():
    dummy = Dummy("foo", "bar")
    if any([dummy.some_variable == dummy.some_other_variable, dummy.some_variable == some_default, dummy.some_other_variable == some_default]):
        print("Some of the variables match")
    print("Done some tests")

if __name__ == "__main__":
    dummy_test()

Searching for the string some_ shows it occurring 13 times on 4 separate lines, resulting in

I'd like to see an option to group all results into the same line in the search results. The exact matches are highlighted anyway. Maybe with something like "search.groupLineResults": true ?

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

How would manual replaces work if they are on one line and you want to replace the 2nd or 5th one?  With the replace button at the end of the line.

Actually hadn't thought of that... Perhaps something with highlighting colors to highlight which of the occurrences will be replaced?

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


[New Feature]: Size of Text box is small.

The of text box is little is small we can modify it into bigger one & also can add new feature if a mssg is sent by user there should be pop on screen that "Mssg has been sent sucessfully " kind of type.

@khushi-joshi-05 please allow me to resolve this error to add the new feature.

I would like to work on this issue, Please assign it to me under GSSOC 2024


Can we remove gradle-wrapper.jar from TAR files?

We identified this as the only/last remaining .jar files in our source release, but it was indicated to me that we could not build without them.  However, this seems to need to be revisited and understand the alternatives (e.g., have user build gradle jar themselves)?
Is there a precedent one way or other? Also, what can we do documentation-wise as well?

@mrutkows As we discussed, we can keep the gradle wrapper in the source code release. Apache Beam can be our reference for the wrapper solution.


Wavefunction Model

Adds a validated wavefunction property attribute to the Result Model. This matched the MolSSI QCSchema.

Ok, changes up and patched. Would prefer to merge this now unless there are blockers and open a new protocols PR which will need further discussion.


[Website] Universal banner

Universal banner has some TBD changes for v2 that need to be discussed.
v1 changes:

[ ] Update language in overview pointing users to content group banner when used in page content.
[ ] Include language around 4-column vs 8-column image.This is vague currently.
[ ] Use lower case for component names.

v2 guidance updates:

[ ] Any color scheme options added for v2 – need new modifier content
[ ] Potential name change we might consider

v2 new images:

[ ] Any color scheme options would need to replace all five imagery to promote lighter color scheme
[ ] If we include more color options (blue10, etc.) we need new image assets to illustrate these.
[ ] New overview image for masthead?

[ ] Use Component template for release for content updates


Add Flow type checking and reasonable configuration

@eatonphil what does this mean? Is it still relevant?

@eatonphil  what is this?

I'd still recommend doing this but it's too big of an argument to only make in an issue. The benefits of static typing are well known for large/growing projects. TypeScript or Flow would be sufficient and appropriate.


Deps: upgrade eslint v7

They ignore node modules by default now, somehow that took them til v7 to do 🤷

🙃


WIP: Textures

This adds support for temporal coherence textures. WIP because I need to add tests.

Changes Unknown when pulling d89625669994295ef3d073c9346a1df508eddfa0 on Eric89GXL:texture into ** on LABSN:master**.

Changes Unknown when pulling eab8e10c966f9ab1a314e86626836f5be12f3120 on Eric89GXL:texture into ** on LABSN:master**.

Changes Unknown when pulling db0b3d19c925cbccea4199212ab48911e9316e43 on Eric89GXL:texture into ** on LABSN:master**.

Changes Unknown when pulling db0b3d19c925cbccea4199212ab48911e9316e43 on Eric89GXL:texture into ** on LABSN:master**.

@drammock ready for review/merge from my end

Changes Unknown when pulling 3651dc1ed45850b89dfaa500e95ac9487bb79483 on Eric89GXL:texture into ** on LABSN:master**.

Thanks @drammock changes pushed. I don't think @rkmaddox or @haribharadwaj will likely have time to look at this one (but feel free to prove me wrong!). But FWIW I based this on Hari's code and have bothered him with questions about it already during implementation, so that's a good start.

Changes Unknown when pulling 1d6f8a5fa32a3459a4e1c731af4f3ec655778751 on Eric89GXL:texture into ** on LABSN:master**.


The terminal process terminated with exit code: 127

Issue Type: Bug
I am not able to use the terminal on any project i open up and I am not sure how exactly to fix it. I have researched and googled the error it pops up but I just do not have the knowledge or experience to understand it and execute and fix. I love VScode and would love to get this fixed or learn a way to like reset all of VScode so that i can just start fresh. Every time i uninstall and install it again it keeps all the savings.
VS Code version: Code 1.46.0 (a5d1cc28bb5da32ec67e86cc50f84c67cc690321, 2020-06-10T09:03:20.462Z)
OS version: Windows_NT x64 10.0.18363

/needsMoreInfo
Do you have any settings that start with terminal. ?
Did you change anything related to PowerShell?
Does it work if you set "terminal.integrated.shell.windows": "C:\\Windows\\System32\\cmd.exe"?

So i did set it to this
"terminal.integrated.shell.windows": "C:\Windows\System32\cmd.exe"?
and i can now use the command line in VS code but I was using GIT Bash and
when i change it to
C:\Program Files\Git\bin\bash.exe
it does not work and everytime i try switching it in the settings.json file
i get that same error
On Mon, Jun 15, 2020 at 2:12 PM Andrii Dieiev<EMAIL_ADDRESS>wrote:

/needsMoreInfo
Do you have any settings that start with terminal. ?
Did you change anything related to PowerShell?
Does it work if you set "terminal.integrated.shell.windows":
"C:\Windows\System32\cmd.exe"?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/100229#issuecomment-644353841,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMSPCA6DL2B5VNEFOT6NXYTRWZ6BTANCNFSM4N6RC6JA
.

--
Best Regards

Same
{
	"version": "2.0.0",
	"tasks": [
		{
			"type": "typescript",
			"tsconfig": "tsconfig.json",
			"option": "watch",
			"problemMatcher": [
				"$tsc-watch"
			],
			"group": "build",
			"label": "tsc: watch - tsconfig.json"
		}
	]
}

ends up with
> Executing task: d:\vscode\simplewebspider-nodejs\node_modules\.bin\tsc.cmd -p d:\vscode\simplewebspider-nodejs\tsconfig.json --watch <

/usr/bin/bash: d:vscodesimplewebspider-nodejsnode_modules.bintsc.cmd: command not found
The terminal process terminated with exit code: 127

Terminal will be reused by tasks, press any key to close it.

If I try to set "terminal.integrated.shell.windows": "C:\\Windows\\System32\\cmd.exe":
{
	"version": "2.0.0",
	"tasks": [
		{
			"type": "typescript",
			"tsconfig": "tsconfig.json",
			"option": "watch",
			"problemMatcher": [
				"$tsc-watch"
			],
			"group": "build",
			"label": "tsc: watch - tsconfig.json",
			"terminal.integrated.shell.windows": "C:\\Windows\\System32\\cmd.exe"
		}
	]
}

I get this error:
Property terminal.integrated.shell.windows is not allowed.

@Barkole your issue is duplicate of #35593 it can be fixed rewriting command to "type": "shell" and using https://github.com/microsoft/vscode/issues/35593#issuecomment-488728912 to force cmd for this task
"terminal.integrated.shell.windows" is for settings.json not for tasks`
@Tevin-wd do you also see this error with tasks or just by launching terminal?

@andril
I have never used tasks as I am still a beginner and not really sure what
those are. The issue occurs when i try and launch git Bash while in VS
code. I added a screencast showing you what i mean and showing you my
settings.json file.
Again with me being new to this stuff I have been testing things out and I
am not sure how this broke. If you see anything that shouldn't be there  in
the settings.json file please let me know same with adding things you think
would be useful for me. I really appreciate it.
https://www.screencast.com/t/g1sJsZ3ZVS1
On Tue, Jun 23, 2020 at 3:26 AM Andrii Dieiev<EMAIL_ADDRESS>wrote:

@Barkole https://github.com/Barkole your issue is duplicate of #35593
https://github.com/microsoft/vscode/issues/35593 it can be fixed
rewriting command to "type": "shell" and using #35593 (comment)
https://github.com/microsoft/vscode/issues/35593#issuecomment-488728912
to force cmd for this task
"terminal.integrated.shell.windows" is for settings.json not for tasks`
@Tevin-wd https://github.com/Tevin-wd do you also see this error with
tasks or just by launching terminal?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/100229#issuecomment-648023391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMSPCA7MTWSI4AKWU3QZ3ADRYBYOBANCNFSM4N6RC6JA
.

--
Best Regards

This is an issue with your environment, if you pause the gif you can see the error the terminal shows when it exits:

You'll need to investigate why this happens yourself.


Version 8.1.104 breaks external Mongo DB support

Looks like externalized MongoDB support is broken with this release.
In system.properties I have configured & unchanged from working 8.0.28:
db.mongo.local=false
db.mongo.uri=mongodb://XYZ.host:27017/unifi
statdb.mongo.uri=mongodb://XYZ.host:27017/unifi_stat

10-15 seconds after startup:
[2024-02-01 18:18:14,581] <launcher> ERROR mongo - Could not determine Mongo journaling state
com.mongodb.MongoTimeoutException: Timed out after 10000 ms while waiting for a server that matches ReadPreferenceServerSelector{readPreference=primary}. Client view of cluster state is {type=UNKNOWN, servers=[{address=localhost:27117, type=UNKNOWN, state=CONNECTING, exception={com.mongodb.MongoSocketOpenException: Exception opening socket}, caused by {java.net.ConnectException: Connection refused}}]

and
[2024-02-01 18:18:24,587] <launcher> WARN AnnotationConfigApplicationContext - Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'mongoRuntimeService' defined in com.ubnt.service.db.CoreDatabaseSpringContext: Timed out after 10000 ms while waiting for a server that matches ReadPreferenceServerSelector{readPreference=primary}. Client view of cluster state is {type=UNKNOWN, servers=[{address=localhost:27117, type=UNKNOWN, state=CONNECTING, exception={com.mongodb.MongoSocketOpenException: Exception opening socket}, caused by {java.net.ConnectException: Connection refused}}]
[2024-02-01 18:18:24,588] <launcher> INFO db   - Closing MongoClient

Also post-upgrade both of the system.properties mongo URI had been reread from environment variables with escape slashes added before the colons (:). Disabled readEnv and manually edited the URIs to remove them with no change in functionality.
https://community.ui.com/releases/UniFi-Network-Application-8-1-104/1a8f4eff-9300-4305-acbf-5b143fc9388d#comment/e543d4e3-7ccc-47d5-818a-44195c1fc9c7

Also https://community.ui.com/questions/Version-8-1-104-external-Mongo-DB-support-broken/729ac226-3988-4949-a9b3-cd248190f324

Confirmed by Ubiquiti staff, "Will be fixed in next version."

Unfortunately, I can confirm this issue 😕

v8.1.107 beta released, has change notes stating this is fixed - https://community.ui.com/releases/UniFi-Network-Application-8-1-107/c2cb7b81-07b9-43e7-a0ec-f728ab76e8a2

Fixed inability to specify external MongoDB.

New 8.1 beta tags are building with this version now. TBD if this is actually resolved.

v8.1.107 beta released, has change notes stating this is fixed - https://community.ui.com/releases/UniFi-Network-Application-8-1-107/c2cb7b81-07b9-43e7-a0ec-f728ab76e8a2

Fixed inability to specify external MongoDB.

New 8.1 beta tags are building with this version now. TBD if this is actually resolved.

Externalized Mongo DB is working as expected for me again now. Will leave this open for a few days in case anyone else wants to chime in.

This is stable for me. Upstream definitely appears to have fixed it. Closing.


Style apps using Cascading Style Sheets

Type of issue
Code doesn't work
Description
[Enter feedback here]
I apply the steps in your page on .net7 project on VS. 2022, but it gives me error in compilation.
Page URL
https://learn.microsoft.com/en-us/dotnet/maui/user-interface/styles/css
Content source URL
https://github.com/dotnet/docs-maui/blob/main/docs/user-interface/styles/css.md
Document Version Independent Id
7897e970-6807-a516-d029-d5d0c27e15de
Article author
davidbritch

Hi @Magedn
As far as I can see the code in the doc works perfectly for me. You can see it in action at https://github.com/davidbritch/dotnet-maui-samples/tree/main/UserInterface/Styles/CSS


Resetting a form does not fire onChange

The Problem
Calling resetForm does not call onChange on controls.
This may be a problem, or maybe not. The behavior was unexpected to me, but it may just be that I'm still learning the contract of onChange.
Steps to Reproduce

Put a control in a form
Reset the form
Observe that onChange isn't called when the model is reset

Expected Behavior
That onChange would be called with the value after the reset
Actual Behavior
onChange is not called is not called with the value after the reset
Reproducible Code Example
View here
Change the "First Name" field to see the onChange event called.
Then, click the button to reset the form. Observe that onChange is not called, even though the input value does change.
@davidkpiano – I understand that this may be the intended behavior, which is alright. If that's the case, what would you suggest I do instead?
The use case is that I have some custom controls that need to manage internal state based on the value of the model. They're kept in sync with onChange. I was thinking I could use this reliably to keep the state in sync, but perhaps I should wire them up with connect, and attempt to use mapStateToProps to get the value? Or, do you have another suggestion?
As always – thank you for taking a look at this! Working with RRF has been great 👌

It is the intended behavior, because the semantics of changing a model and resetting it are a little different, in that in most use-cases, you want to handle when the user modifies a model, and it's sort of a gray area as to whether or not resetting counts as modifying a model.
What do others think? Should the change action fire when a model is reset?

Makes sense, @davidkpiano . Are there any hooks for when a value changes due to resetting? If not, do others think that's a problem, independent of whether the notification happens through onChange or not?
This causes an issue to me because the thing that resets the form is usually higher up than the controls themselves. There's no simple way for me to "push" action that reset down into the component, other than by accessing the value model value from the store. I haven't put too much thought into it, but it seems like a challenging project to get the value of a custom control that accepts the ".model" syntax from outside of the component, say, in a mapStateToProps for the component that wraps it. My guess (I haven't dug into the code) is that the form puts some stuff on context, which the control then accesses. So I suppose I could access the context directly in my app to find the model, but that doesn't seem as elegant.
Anyway, I'm not pushing for onChange to change, or anything. I'm open to any solution. If there's a recommended way to do this with the current API, then I'm definitely :+1: to that.

The above comment may have come across as rambling (I admit I was thinking through the issue as I was typing). Here's a simple challenge that illustrates the problem:

Build a component that wraps Control.custom
Within that component's render function, console.log() the current value of that Control's model

One way to do this is to use the onChange hook to set some state on the component equal to the new value passed through onChange. Then, you would log out that value in render. Because resets don't fire onChange events, the component's state drifts from the control's value.
Here's a little snippet to show what I mean:
class CustomSearch extends Component {
  render() {
    // This may just be `".myModel"`, so using it to get the model's value
    // in `mapStateToProps` isn't trivial
    const { model } = this.props;
    const { selected } = this.state;

    // When the form is reset, this console.log will be incorrect
    console.log('This control\'s current value is', selected);

    return (
      <Control.custom
        model={model}
        component={SomeComponent}
        value={value}
        mapProps={{
          onChange: props => selected => {
            this.setState({ selected });
            props.onChange(selected);
          }
        }}/>
    );
  }
}

My current solution is to no longer use the reset action creator.
What I'm instead doing is manually changing each value in all of my forms to their initial value to mimic the reset. It's much more verbose, and requires updating the reset code anytime that the form updates, but it's one way to get around the problem for now.

Here is a related issue: https://github.com/facebook/react/issues/1152

Makes sense @davidkpiano . Thanks for the cross-reference!
I haven't tried this yet, but I'm pretty confident that the solution is to always have Control.Custom as the "highest" component in a series of wrapped components. This way, it's connected for you, and passes the value of the model down to all of the children.
I'm going to close out this issue for now. Thanks for all of the help, @davidkpiano !

Alright, so, update – I finally had a chance to refactor this for one of my inputs, and it ended up working. Here's a thorough description of the problem and the solution:
First, I have a Dropdown component (similar to React Select). It sometimes displays text determined by the model's current value, such as when you've selected an option. But since it's a typeahead-style component, sometimes it instead displays whatever you've been typing. This is stored in the component's internal state.
There are two ways to handle this: one is to write a conditional to determine when to render state, or when to render the model's value. The other way is to always render the component's state, and then write code that keeps the state in sync with the model's value.
When I opened this issue, I was attempting to go with the latter approach. But I am now using the former approach.
The tl;dr of my solution is:

I set focused state in onBlur/onFocus events
I set whatever the user types as value state on the component, as well as an associated selectedKey value on state
In the onBlur callback, check to see if the state's selectedKey value matches the model's value. if not, I reset the state's value to match the model's value. This is so if the user goes into an input that already has an option selected, and they, say, type "asdf," then navigate away without selecting a new option, it goes back to what was selected before, rather than continuing to display "asdf".
within render, if the input is not focused, then I display the model's value. if the input is focused, then I display the value from the state.

Because the component defaults to using the model's actual value unless the user is typing, then you get free synchronization with the model, which, well, makes a lot of sense in hindsight.
There are two takeaways from this:

Do not wrap Control or Control.Custom with additional functionality. Keep those things pretty simple, and add additional functionality in the component that they wrap

Do not attempt to keep state in sync with a model's value. If you want to use a model's value, use it directly. If there are situations where you want to use some other value, then it is a much simpler problem to detect those situations.

If anyone else runs into a similar problem, or has further questions, just leave a comment here and I'd be happy to help out.
Thanks again for all of the help @davidkpiano !

Thanks so much for the write-up, @jmeas!