Disable check box of parent node.

I want to tweak the default behaviour of jstree. In jstree by default, all child and parent node have a check box . What I want is that I want to check box only for the child node, not for the parent node and when clicking on parent node it just simply lists out its children, it has using jquery.

This question was already answered, e.g. here https://github.com/vakata/jstree/issues/1532
so the solution is using for those nodes another class that will hide the ckeckbox: http://jsfiddle.net/2kwkh2uL/4426/

Ok, I get it that by using CSS we can disable the parent node checkbox. But what I want is based on  some condition, if true I want to disable all parent check box using jquery. So  how should I get these
" .jstree li.jstree-open > a.jstree-anchor > i.jstree-checkbox,
.jstree li.jstree-closed > a.jstree-anchor > i.jstree-checkbox { display:none; }"
using jquery. I don't want to apply it manually to css and also I dont want to manually set no_checkbox property to false. I want to apply it dynamically based on some condition.
Please help, thanks.

I want to apply it dynamically based on some condition.

What do you really mean by that? (your initial request mentioned nothing about this second scenario).

Dynamically when generating the page, or
at runtime after the page is generated and the tree is already rendered based on some user action (event) e.g. click on some button?

Sorry, for not clearing the context earlier.
I want both the scenario, like dynamically when the tree is loaded the first time and based on some user event also.

I still don't understand why is not working for you.
Here's an updated example that hides the checkbox for the parents when that button is clicked:
http://jsfiddle.net/2kwkh2uL/4553/

@aadrian thanks for helping out.


Typocorrect describe for grant

Proposed changes
Issue Number: close #xxx

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

run buildall

run compile

run buildall

run buildall

run buildall

run feut


Removed unused parameter from NetworkAnimator.

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


Fix compile error in Visual Studio 2017 15.1

This fixed a compile error for me in Visual Studio 2017 version 15.1 26403.7 with v141 toolset [I just tried building vcpkg on an old computer that I have from 2017].

Thanks for the fix!


php: change phpstan's --errorFormat to --error-format

The latest version of phpstan has marked the --errorFormat parameter as deprecated and recommend --error-format instead.

@dreamlover this would require a forced update of phpstan if there's any possibility of someone not having phpstan updated since August, which I wouldn't say isn't a possibility. Can you add in a version check to see if it's >= 0.10.3 ?

@RyanSquared --error-format was supported starting from phpstan 0.10.3, as specified here.

Probably a good idea to add a version check. Have a look at the flake8 linter for one example with command_chain.

@w0rp OK, i'll trying to add this!

@w0rp  Version check added.

Cheers! :beers:


Canvas Font has 'fringing' on the edges of text

This is just about viewable in the engine demo where the background of the text is not completely black:
http://playcanvas.github.io/#user-interface/text-canvas-font.html

I added the same image with levels changed .. it seems we see the quads for characters, alpha of the texture is not completely 0.


RPM spec file for lsd

Basic SPEC file that can be used to build a RPM for lsd.

UPX compression was disabled.

I have precompiled packages on my fork for EL9 and AMZN AL2023.
https://github.com/daniejstriata/lsd/releases

hi @daniejstriata, thanks for the contribution, can you also add a ci to build the rpm? so that the spec file could be used and we could maintain a rpm release

hi @daniejstriata, thanks for the contribution, can you also add a ci to build the rpm? so that the spec file could be used and we could maintain a rpm release

Hi @zwpaper I've not built rpms before using ci on github. I'll investigate it.

hi @daniejstriata, how is this PR going? maybe we could merge it first if the spec is completed.
and let the ci be another PR

@zwpaper I've built lsd in copr here
https://copr.fedorainfracloud.org/coprs/faramirza/lsd/
for various rpm based distros. The difference between the spec I must here and the one in my fork is that the on I'm pushing here assumes that rust and cargo is already installed whereas the other version will install the version for rust available to the distro in its repos.
I'm still unsure about the ci on github. I've never used it before. I've not yet been able to look into that. Do you know of a recommend Guide I can look at?
We could cancel this PR and redo it once more with the newer spec?

let's merge this one and you could always work on the CI later, for CI, please check: https://github.com/features/actions

@zwpaper I had a look. How is the Fedora package built and should that SPEC file not be added back here rather than mine?


🐛(search) fix search template meta block

The favicon and open graph metas weren't being added to the search page.

When @igobranco comes and fix long pending issues in richie like it's easy!

@igobranco can you rebase it and we merge it.


Add endpoint override example

Fixes https://github.com/GoogleCloudPlatform/google-cloud-java/issues/2224

Codecov Report

Merging #1463 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1463   +/-   ##
=========================================
  Coverage     87.17%   87.17%
  Complexity     4039     4039
=========================================
  Files           387      387
  Lines         16232    16232
  Branches       1770     1770
=========================================
  Hits          14151    14151
  Misses         1619     1619
  Partials        462      462

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 86c2824...4f6fee2. Read the comment docs.


doc: Fix AKS guide regression

The commit 112dcb85c7f has removed the nodeinit code to put the Azure bridge
into transparent mode. While this worked for all routing operations, it broke
proxy redirection. Partially undo 112dcb85c7f and simplify the AKS guide.
Fixes: 112dcb85c7f ("doc: Fix AKS installation guide")

This change is

test-docs-please

Coverage increased (+0.01%) to 45.523% when pulling 0b70117b968f99b34610a4a167f785cbe582f375 on pr/tgraf/simplify-aks-guide into f10452133831388cb36a7429e1563e15c8e10e92 on master.


Java-microprofile collections from latest v0.1.2 collection release is throwing error in the pipelinerun due to pom file

Describe the bug
A clear and concise description of what the bug is.
We got latest appsody collections from our Kabanero v0.1.2 for java-microprofile and tried running the pipelinerun on that collection. The pielinerun is failing as per below error showing that pom file has some dependency which appsody stack cannot download.
Pipelinerun error in Build task pod.
TEP 19: WORKDIR /project/user-app
STEP 20: RUN mvn install -DskipTests
[INFO] Scanning for projects...
Downloading from central: https://repo.maven.apache.org/maven2/dev/appsody/java-microprofile/0.2.11/java-microprofile-0.2.11.pom
[ERROR] [ERROR] Some problems were encountered while processing the POMs:
[FATAL] Non-resolvable parent POM for dev.appsody.starter.java-microprofile:starter-app:1.0-SNAPSHOT: Could not find artifact dev.appsody:java-microprofile:pom:0.2.11 in central (https://repo.maven.apache.org/maven2) and 'parent.relativePath' points at wrong local POM @ line 8, column 13
 @
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project dev.appsody.starter.java-microprofile:starter-app:1.0-SNAPSHOT (/project/user-app/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM for dev.appsody.starter.java-microprofile:starter-app:1.0-SNAPSHOT: Could not find artifact dev.appsody:java-microprofile:pom:0.2.11 in central (https://repo.maven.apache.org/maven2) and 'parent.relativePath' points at wrong local POM @ line 8, column 13 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException
subprocess exited with status 1
subprocess exited with status 1
error building at STEP "RUN mvn install -DskipTests": exit status 1
[root@escapes1 manual-pipeline-runs]# ls -ltr

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Actual behaviour
What is the actual behaviour.
Environment Details (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

If applicable please specify:

CLI version:
Collection you are using:

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@groeges - Is it fixed in the release: https://github.com/kabanero-io/collections/releases/tag/v0.2.0-beta1?

This should be fixed with the latest v0.2.0-beta2 release and its subsequent updated docker images.

@ianpartridge - What are the next required steps before it can get closed?

@marikaj123, We need confirmation the issue has been resolved. Our testing shows no further problems but we would like confirmation from @aadeshpa.

@aadeshpa - please verify and close the issue accordingly.

We confirmed, that Java-Microprofile pipelinerun works with the collection from https://github.com/kabanero-io/collections/releases/tag/v0.2.0-beta1.
This issue could be closed. @kilnerm


SelectorDepth problem

Hi all, I have this problem with the linter
scss-lint -v 0.43.2
$var: 5;

// MY CASE
.e {
  &--#{$var} {
    > .g {

      // NOT EXPECTED
      // [W] SelectorDepth: Selector should have depth of applicability no greater than 3, but was 4

      &:nth-of-type(#{$var}n+1) {
        clear: both;
      }
    }
  }
}

// Code generated
.e--5 > .g:nth-of-type(5n+1) {
  clear: both;
}

// OTHER EXAMPLE

// CORRECT BEAVIOUR
// [W] SelectorDepth: Selector should have depth of applicability no greater than 3, but was 4
.a .b .c .d:nth-child(n) {
  padding: 10px;
}

// NOT LINTED
.a .b .c .d:nth-child(#{$var}n) {
  padding: 10px;
}

I think that the problem is the string interpolation.
Thank you all for your work!

Hey @giacomozinetti, thanks for reporting this issue.
As you surmised, the problem here is string interpolation. scss-lint can't know in all cases what final result of the interpolation will be. While the case you have here seems simple and easy to detect, I can assure you that the representation returned by the Sass parser makes it impractical to handle this case gracefully.


loadData() gets called when orientation changes with setRetainInstance(true)

Hi, I am now learning using MvpLceViewStateFragment with MvpLceRxPresenter, I notice that even setRetainInstance(true), loadData() still gets called if viewstate is "showLoading" when orientation changes
@Override
    public LceViewState<List<LogRecord>, HistoryView> createViewState() {
        return new RetainingFragmentLceViewState<>(this);
    }

log
V/HistoryFragment2﹕ ⇢ onRefresh()  <----- SwipeRefreshLayout.OnRefreshListener
V/HistoryFragment2﹕ ⇢ loadData(pullToRefresh=true)
V/HistoryPresenter﹕ ⇢ refresh(pullToRefresh=true)
V/HistoryPresenter﹕ ⇠ refresh [0ms]
D/Retrofit﹕ ---> HTTP POST
D/HistoryFragment2﹕ fragment: HistoryFragment2{83e3df4 #0 id=0x7f0e00a6}
V/HistoryFragment2﹕ ⇠ loadData [4ms]
V/HistoryFragment2﹕ ⇠ onRefresh [5ms]

V/HistoryFragment2﹕ ⇢ onDestroyView()  <-----orientation changes
V/HistoryPresenter﹕ ⇢ detachView(retainInstance=true)
V/HistoryPresenter﹕ ⇠ detachView [0ms]
V/HistoryFragment2﹕ ⇠ onDestroyView [1ms]
V/HistoryPresenter﹕ ⇢ attachView(view=HistoryFragment2{83e3df4 #0 id=0x7f0e00a6}) <----same instance of Fragment as above
V/HistoryPresenter﹕ ⇠ attachView [0ms]
V/HistoryFragment2﹕ ⇢ loadData(pullToRefresh=true) <----still gets called after orientation changed
V/HistoryPresenter﹕ ⇢ refresh(pullToRefresh=true)
V/HistoryPresenter﹕ ⇠ refresh [0ms]
D/HistoryFragment2﹕ fragment: HistoryFragment2{83e3df4 #0 id=0x7f0e00a6}
V/HistoryFragment2﹕ ⇠ loadData [1ms]
D/Retrofit﹕ ---> HTTP POST

Could you please mark in the log where the orientation change happened?

Hi, I have updated the log

Thanks for reporting.
The bug is fixed. I have pushed 1.1.1-SNAPSHOT containing the bugfix to sontype's snapshot repository.
allprojects {
  ...

repositories {
    maven{
      url 'https://oss.sonatype.org/content/repositories/snapshots/'
    }

   ...
}

Please let me know if this SNAPSHOT version fixes that problem for you as well.

Great! Thanks!

@pkliang Please let me know if it works as expected now, once you have time to check your app with the snapshot.

@sockeqwe I have my app tested with the latest snapshot, it works as expected now.

Thanks!
Liang Song<EMAIL_ADDRESS>schrieb am Di., 7. Juli 2015 um
15:32 Uhr:

@sockeqwe https://github.com/sockeqwe I have my app tested with the
latest snapshot, it works as expected now.
—
Reply to this email directly or view it on GitHub
https://github.com/sockeqwe/mosby/issues/61#issuecomment-119203188.

@pkliang fyi: I have released mosby 1.1.1 containing this bugfix.


Upgrade rules_java and add rule loads

Upgrade to the latest rules_java 8.1.0
Add loads for Java rules

NOTE: The loads and rules_java version unforutunately are tightly related. The paths for the bzl files for Java rules changed between rules_java 7 and 8.

For some more context on the rule loads: This PR is supposed to make rules_jvm_external work with Bazel 8, which removes all native Java rule loads.
Most users will be able to upgrade to Bazel 8 without manually adding all of these loads for Java rules. The exception is rule repository maintainers -- I'm not sure of the specifics, but it looks like rule reposotiries (rules_java, rules_android, etc) are special-cased, and autoloads from other rule repositories are disabled.
I was motivated to write this PR when trying to get rules_android to work with Bazel 8. Because I'm working inside a rule repo, it looks like transitive repo deps are not autoloaded -- for example, rules_jvm_external's use of java_library/java_import rules.
For more context on why rules_java 8.1.0: rules_java v8 changed where symbols such as java_binary and JavaInfo are loaded from. The old paths to various .bzl files seem to no longer exist, or at least are no longer visible.

/cc @comius

Wait, did you forget to update MODULE.bazel?

Ah, my main repository already had rules_java bumped to 7.12.2 so I forgot I had to update it in RJE too. I see you already updated RJE's MODULE.bazel in 3ea96498725cb21e3ac18490cc3f3b75db2259ca. I think that should be sufficient.


CMSIS SVD 2.0 proposal

Fixes #5
Fixes #6

FYI, this PR is split to 3 commits

extract *.psvd schema
support loadFrom
support conditions

what is remaining is the support in svdconv and the tool to convert SVD v2 to v1

I'd rather not see SVD files split apart. Doing so makes it harder to load a complete view of the system and it's also harder to manage the files because you must keep more than one file together on the file system.
It seems the only benefit is reducing file size but compression can achieve this without requiring loading multiple files.


Support hiding file names from path

Is it possible to add an option for hiding file names from the path? The reason is that the file name can sometimes overlap with that from the following plugin.
jplattel/open-note-to-window-title: Small Obsidian plugin that updates the window title with the current open note
If the file name is disable from the path, one can still navigate to desired locations by choosing the desired folders by changing its parent folder.
For instance, it would be great if Folder 1/Folder 2/Note A.md may display as Folder 1/Folder 2 instead.

You can use CSS to hide the #quick-explorer .explorable:last-child, if you want to not show it.  I would suggest hiding it by setting a short max-width and overflow: hidden, so that there is still a spot on the screen for the menu to descend from; it may not work properly otherwise.

Thanks for the reply.
Could you be more specific about this? I tried to add the following to my theme CSS file and the CSS file of this plugin, but neither of them worked for me.
#quick-explorer.explorable:last-child {
  max-width: 1cm;
  overflow: hidden;
}

You need a space before the .; try this:
#quick-explorer .explorable:last-child {
    max-width: 1cm;
    overflow: hidden;
    display: inline-flex;
}

If you want less of the filename showing, you can use 1em or 1px instead of 1cm.

Thanks for the prompt reply! It works like a charm now! I will close this issue now.


Feature request: filter by technology

Great site, much better than googling virtual tours. I would love to be able to filter on the technological implementation (streetview, matter most, just a gallery etc) as especially for kids all the different uis can be confusing

Hello @tdamsma,
Thank you for your comment and feature request.
I have a plan to add some new filter options (e.g. hide visited places). I will also consider your request. Most of the virtual tour sites have similar sources like Google Streetview or Matterport. Maybe a filter for virtual tour source can be applied. The problem is some places have its special ui and this might make that filter a little complex.
I will try to find a feasible solution for this.


Keyword argument handling

f should be able to accept keyword arguments

We now ensure this is the case except for the existing higher-order functions like mean and map, in which the user will have to fix kwargs manually


feat: create file from prompt + create fixes

This PR introduces the fb_actions.create_from_prompt which allows to create files or folder from prompt and directly open them via telescope-file-browser.nvim (i.e. if you open a folder, you'll land in that folder).
Creds to @otavioschwanck for the original idea and implementation :)
Furthermore, this commit reflects a breaking change. The input callback to create a file or folder via the prompt if no result is shown is now superseded by the fb_actions.create_from_prompt mapped by default to Shift-CR (works if your terminal recognizes this, otherwise please remap).
cc @danielnehrig @ZhiyuanLck as you might have used that feature
Please don't be too concerned with the default mapping; once some previewer upstream refactoring etc is committed, I plan to reconsider mappings and disseminate info about recent file browser updates on reddit to draw attention to a wider audience.

might it not make sens to make the default behaviour something else then ` pressing shift+cr does nothing for me probably as you mentioned i have to set some thing in my kitty conf

might it not make sens to make the default behaviour something else then ` pressing shift+cr does nothing for me probably as you mentioned i have to set some thing in my kitty conf

i remapped to Ctrl + space on my config.

might it not make sens to make the default behaviour something else then ` pressing shift+cr does nothing for me probably as you mentioned i have to set some thing in my kitty conf

i remapped to Ctrl + space on my config.  Would be nice to have something like that by default

Please don't be too concerned with the default mapping; once some previewer upstream refactoring etc is committed, I plan to reconsider mappings and disseminate info about recent file browser updates on reddit to draw attention to a wider audience.

Please understand I won't introduce a single mapping breaking change to a perfectly logical mapping that for some reason does not work for you. As per above, mappings as a whole will be reviewed eventually. Remapping is for good reason very easy -- we won't be able to please everyone, since there are so many mappings in the file browser.


added support for java.util.Collection and recursive alternate type substitution

These changes fix issue #526 providing a representation for java.util.Collection, represented as lists and arrays.
At the same time I added recursive resolution of alternate types, so that if we have a return value of type ResponseEntity<Resource<MyClass>> and both ResponseEntity and Resource have been configured as genericModelSubstitutes then the documentation will report a response of type MyClass.
Without the recursive search, only the outer class would be replaced, resulting in a return value of type Resource<MyClass>.

@rlogiacco Thanks for the PR! Since we're getting ready to release a 2.0-SNAPSHOT pretty soon, could you apply your fix to that branch instead. Keep in mind a LOT has changed.
Also preferably, with a few unit tests that validate your change. See guidelines here. Especially because the change reg. recursive types, tho' tiny, seems risky to me; :fearful:  without having the safety net of a test.
Also, I'm :neutral_face: about the Collection being treated as an array. I'd rather have it as an extensibility via the substitution mechanisms already available.

@dilipkrish I believe I've already stated my point of view on the java.util.Collection thing, it's your choice to have it or not, but I don't see a reason not to have it. I tried to use the substitution mechanism as suggested in the mentioned issue, but it was not working in 1.0. I believe in convention over configuration, but it's up to the project leaders to decide which approach fits best for the project.
About the tests, it can be easily done, but I'm not going to implement that change on another code base if it's going to be rejected: please let me know if you want to merge that in or not.

@rlogiacco I'm still interested in seeing why Resource isn't substituted. Do you have a test case I could use?

@RequestMapping("/any/url")
public ResponseEntity<Resource<MyModel>> getMyModel() {
  return new ResponseEntity<Resource<MyModel>>(new Resource<MyModel>(new MyModel()), HttpStatus.OK);
}

@RequestMapping("/another/url")
public ResponseEntity<List<Resource<MyModel>>> searchMyModel() {
  List<Resource<MyModel>> result = new ArrayList<Resource<MyModel>>();
  result.add(new Resource<MyModel>(new MyModel()));
  return new ResponseEntity<List<Resource<MyModel>>>(result, HttpStatus.OK);
}

@RequestMapping("/any/other/url")
public @ResponseBody List<Resource<MyModel>> listMyModel() {
  List<Resource<MyModel>> result = new ArrayList<Resource<MyModel>>();
  result.add(new Resource<MyModel>(new MyModel()));
  return result;
}

I would expect the following response types:

'/any/url' -> MyModel with model schema for MyModel
'/another/url' -> List of MyModel with model schema of an array for MyModel
'/any/other/url' -> List of MyModel with model schema of an array for MyModel

Instead I get:

'/any/url' -> Resource with an empty model schema
'/another/url' -> List of Resource with an empty model schema
'/any/other/url' -> List of Resource with an empty model schema

@rlogiacco Closing this pr as the work around/ best practice is to provide an explicit AlternateTypeRule is to provide an explicit. Explanation of work around in #526.


React Dev Tools not showing tabs in Edge dev tools

Steps to reproduce:

Install React Dev Tools extension from https://microsoftedge.microsoft.com/addons/detail/react-developer-tools/gpphkfbcpidddadnkolkpfckpihlkkil
Open a page that uses React in development mode
Open dev tools with F12

Expected:
⚛  Components and ⚛ Profiler tabs show in the dev tools
Actual:
Tabs are not visible and there seems to be no way to add them
Screenshot from Chrome:

Screenshot from Edge:

Thanks for the report, @limitedmage. This bug has been fixed in Edge version 100 and later, which is in our Edge insider builds and should be rolling out to all users soon. (You can navigate to edge://version to see what version you're using currently.)
In the meantime, a workaround would be to disable the Focus Mode experiment in DevTools settings, then close and reopen DevTools. This should restore the React extension tab on pages with React. Once the browser updates to Edge 100, you can manually reenable the Focus Mode experiment in DevTools and use extensions normally.
Thank you for trying this early build of Focus Mode! We've made a number of great improvements in Edge 100, so I hope you'll continue using it!

@hxlnt Thank you! Happy to know the fix will be rolling out soon.

Edge 100 has been released, so closing this issue. Extension support has landed in DevTools for Focus Mode.

Not showing when inspecting a WebView2 control.  Should react dev tools be showing ?

Took advice from https://github.com/facebook/react/tree/main/packages/react-devtools

If your app is inside an iframe, a Chrome extension, React Native, or in another unusual environment, try the standalone version instead. Chrome apps are currently not inspectable.

standalone version
If using create-react-app and you want profiling then npx react-scripts build --profile

I'm in build 105.x and I still do not see these tab in dev tools.  I've tried toggling focus mode on and off, no dice, It's been like this for quite some time.  If I need to use react dev tools, I need to load Chrome and debug from there, because it just doesn't work in Edge any longer.

I've even installed from Edge's "store" for extensions - same thing.

I'm not seeing it in the new focus mode either...

Yup, still missing here as well. When switching between focus mode on and off, they show up upon clicking the 'Restart devtools' button that shows up, but upon closing and reopening dev tools, they are gone.

I just upgraded to version 107.0.1418.56 and it is showing again.

I don't want anything Microsoft
On Wed, Nov 23, 2022, 11:52 AM Bill @.***> wrote:

I just upgraded to version 107.0.1418.56 and it is showing again.
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftEdge/DevTools/issues/31#issuecomment-1325379620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2ZI6OHZTJXDO6ILTDY5IQLWJZDWJANCNFSM5R4Q4VPA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Same issue on 109

And 111


COVID-19 stats

COVID-19 scraper
Unlike other the scraper, with this one, you can even search for the stats for any particular country/state.
This script, after running, displays the following things:

COVID-19 stats including total cases, total deaths, people recovered.
Stats of top-10 countries suffering from this pandemic.
You can even search stats for any country or any state, searching a city name will most probabily give you the stats of that state and that country in which the city exist.

You can search infinite number of times without exiting the script.
Fixes 330
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Documentation Update

Checklist:
Please tick all the boxes that are fulfilled by your Pull Request.

[x] I have named my files and folder, according to this project's guidelines.
[x] My code follows the style guidelines of this project.
[x] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[ ] I have commented on my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
[x] I have included a requirements.txt file (if external libraries are required.)
[x] My changes do not produce any warnings.
[x] I have added a working sample/screenshot of the script.

Please add the tag Hacktoberfest accepted if you choose to merge this pull request

This took me so much time because, before this, I was working on the other issue 351.
And that was a successful pull request

@YASHBRO please add a requirements.txt fiel

@YASHBRO please add a requirements.txt file

Oops sorry, just a sec


Add japanese number phone format

Accourding with Wikipedia, the number format is +81-[areacode]-[number]. For example, in Tokyo is +81-3-1234-5678. This makes work easier for mappers.

Thanks! 👍


FINERACT-1506: Add WAR artifact to release distribution again

Description
Since we split the WAR build into a separate module we've excluded the WAR, because of a circular dependency issue. Adding the WAR file again as there was no general vote to remove it. Includes also fineract-client.

@ptuomola ... would be great if you could review this until Monday... you can quickly test with:
./gradlew :fineract-war:binaryDistTar -x test
./gradlew :fineract-war:srcDistTar -x test


test(eventarc): skip failing test

Skip TestAuditStorageSinkService, which is failing and blocking presubmits.
Updates #3188
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist

[ ] I have followed Contributing Guidelines from CONTRIBUTING.MD
[ ] Tests pass:   go test -v ./.. (see Testing)
[ ] Code formatted:   gofmt (see Formatting)
[ ] Vetting pass:   go vet (see Formatting)
[ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
[ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
[ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
[ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
[ ] Please merge this PR for me once it is approved

Thanks for sending this PR!
I just merged #3207 which fixes the underlying test, so this should be resolved.
Just in case, i'll leave this open until tomorrow, and close it once i've verified that the tests are passing consistently.

There are several successful test runs for this package, and no failures.
Thanks again for your commitment to green tests! 🥇


Weaver: force SyncObjects readonly

users shouldn’t be able to assign monsterA.synclist to monsterB.synclist.
same for SyncVar<T> if we ever do it
marked as bug so i don’t miss it
OR: scan for setters and warn if any outside of field definition.
that's probably necessary for SyncVar<T> because readonly fields arent show in inspector

same for SyncVar if we ever do it

Disgree with this part.  SyncLists / SynDictionaries are supposed to be declared readonly per docs so they can't be initialized twice, but SyncVars are just value types and must not have any such restriction hiding in the weaver.

same for SyncVar if we ever do it

Disgree with this part. SyncLists / SynDictionaries are supposed to be declared readonly per docs so they can't be initialized twice, but SyncVars are just value types and must not have any such restriction hiding in the weaver.

I meant SyncVar

                    if (!fd.IsInitOnly)
                    {
                        Log.Error($"{td.Name}.{fd.Name} needs to be readonly. {SyncVarType} should never be set at runtime. Only set the .Value property.", fd);
                        WeavingFailed = true;
                        continue;
                    }


Modify color range when using vtp file

Hi all,
I am trying to use paraview glance to render some vtp files where flow variables are included ( turbulent intensity, wind speed). My original idea was to use vtk files but it seems like neither vtk.js nor glande can manage the complete information included in a vtk file obtained with OpenFoam.
I would like to use a complete local web app without server because it is not know the number of users that can use the service at the same time.
I have converted vtk to vtp using paraview and I can load the vtp file but:
How can I modify the color range?
How can I plot the color legend?
With visualizer, I can load vtk files directly but the server capabilities (GPU, RAM) could be a problem for maybe, 20 users.
Thank you very much for so great developments and support.

Those two features are not yet implemented and would require so add-on on the UI side.

How can I contact with someone to develop this?
Do you think that the time and cost would be high?

We will take a look at what is required and give you a time/cost estimate.
If you get a chance, please provide a sample VTP file.   We like to make
certain we are covering all of the options that you would need.
Thanks,
Stephen
On Fri, Jul 5, 2019 at 2:05 AM beinspired<EMAIL_ADDRESS>wrote:

How can I contact with someone to develop this?
Do you think that the time and cost would be high?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Kitware/paraview-glance/issues/247?email_source=notifications&email_token=AACEJL6HLYY6BMQ6LEFQJA3P53QDDA5CNFSM4H6DI2KKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZIT5PY#issuecomment-508640959,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACEJL6TUUVYF6UI2GD2LQLP53QDDANCNFSM4H6DI2KA
.

--
Stephen R. Aylward, Ph.D.
Senior Director of Strategic Initiatives, Kitware, Inc., North Carolina
Kitware: Advancing the frontiers of understanding
by developing innovative open-source software platforms
and integrating them into research, processes, and products.

It would be better to use directly the vtk files because they are used in other tasks and are obtained with openfoam directly, if not, it would be required to translated hundreds of vtk files to vtp files.
Do you know if that could be done with an extra work?
I will try to upload both vtp and vtk along the weekend but they will be quite simple examples because of confidentiality.
Thanks


fix: update steemitmages URL

Fixes #1896
Changes

update steemitmages URL to images.steemitimages.com

Test plan

[x] Images should work.

Avatar images are not working for me i get default avatar, while it's working with the old link. I think the real new link is https://cdn.steemitimages.com/u/itharagaian/avatar/small

It used to work yesterday. Looks like Steemit is changing URLs twice a day.

I contacted Steemit, steemitimages.com is preferred URL (and it was down because of some issues). cdn.steemitimages.com will be deprecated.


No type data available for code areas dealing with inlined / scope-defined structs

I am not sure if it is a duplicate or not, but I could not find related issue. Here is the screenshot with hover showing {unknown} when struct is defined in scope of a function:

When struct definition is moved out of the function scope, types are recognized correctly:

Duplicate of #1559


feat(home-assistant): More data into Home Assistant

Map more data into Home Assistant

added apparent_power device class

added duration device class

added frequency device class

added Grid Voltage and Frequency

added EPS Active Power, Apparent Power and Frequency

added Battery Charge and Discharge Power

added Inverter Total Runtime

Home Assistant Sensors Documentation

Nice, great work! Its becoming obvious I need to try and reduce some duplication between all these methods but that can be done separately. This looks fine, merging now :)


Finish custom auth state provider update

Fixes #24992
Follow-up to #24993, which was just a quick patch while working on other things 🏃😅.

@Rick-Anderson @serpent5 @wadepickett ... Looks like they upped the ante on duplicate headings. Now, they're warnings that fail the build. In this case, I'm not cross-linking, so I'm going to make mods to the 5.x and 3.x headings to get this to pass ... I think 🤔 that that's a little better than just overruling the build and merging anyway. I'll email to ask about the Version by File thing.

Line 302: [Warning] Duplicate heading: 'H2 'Notification about authentication state changes''. Each second level heading (H2) in an article must be unique.
Line 306: [Warning] Duplicate heading: 'H2 'Implement a custom ''. Each second level heading (H2) in an article must be unique.
Line 343: [Warning] Duplicate heading: 'H2 'Notification about authentication state changes''. Each second level heading (H2) in an article must be unique.
Line 347: [Warning] Duplicate heading: 'H2 'Implement a custom ''. Each second level heading (H2) in an article must be unique.


[COR-106] Add opentelmentry tracing system

Add opentelmentry tracing for messaging systems
Some metrics need to be added in another repo
It requires go 1.18+ so some checks won't pass

some checks seem to be failing

some checks seem to be failing

Open telemetry requires go1.18+

We do not want tracing on externally written connectors as that would couple the external devs too closely. They have no way of testing opentelemetry in our environment and I prefer we keep open telemetry use closed and not something we share.Reminder all , for 0.2.0 we are only seeking latency measurements. We are not instrumenting all code for 0.2.0. We really only need one busy connector set up to measure latency for 0.2.0On Oct 24, 2022, at 07:59, Allie @.***> wrote:
@azhang approved this pull request.
A couple minor questions but overall approved
In examples/compound/go.mod:

@@ -7,7 +7,7 @@ replace github.com/nakji-network/connector/chain/ethereum => ../../chain/ethereu
replace github.com/nakji-network/connector => ../../

require (

github.com/ethereum/go-ethereum v1.10.25

github.com/ethereum/go-ethereum v1.10.15

?
In connector.go:

@@ -89,6 +91,12 @@ func NewConnector(options ...Option) (*Connector, error) {
go http.ListenAndServe("<IP_ADDRESS>:8080", c.Health)
log.Info().Str("addr", "<IP_ADDRESS>:8080").Msg("healthcheck listening on /live and /ready")

//	Initialize trace provider
_, err = monitor.InitTracerProvider(context.TODO(), conf.GetString("trace.host.grpc"), c.id(), c.manifest.Version.String(), string(c.env), conf.GetFloat64("trace.sample.ratio"))

do we want monitoring for all connectors in all environments? Do we want to allow devs to disable monitoring if they're just running it locally in dev environment?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Maybe we should update the tests to 1.18 and 1.19 instead of 1.17

Yes. We can do that.


Support scrubbing absolute temporary paths in test_html.py on Windows

Some gold files, like tests/gold/html/other/index.html record an absolute path that includes the temporary directory used while testing, like:
/private/var/folders/6j/khn0mcrj35d1k3yylpl8zl080000gn/T/pytest-of-ned/pytest-293/t40/othersrc/other.py

There's explicit support for the directory format above in the regex:
/private/var/[\w/]+/pytest-of-\w+/pytest-\d+/(popen-gw\d+/)?t\d+

However, when creating the gold files on Windows, the absolute path takes a rather different format, for example:
C:\Users\ddini\AppData\Local\Temp\pytest-of-ddini\pytest-9\popen-gw6\t1\othersrc\other.py

This PR adds a new regex to match and scrub directories in the format above:
[A-Z]:\\Users\\[\w\\]+\\pytest-of-\w+\\pytest-\d+\\(popen-gw\d+\\)?t\d+

This looks reasonable on the face of it, but we run the test suite on Windows on GitHub Actions all the time, and this hasn't been needed there.  Do you know why>?

Yes, it's because running the tests on Windows works due to compare_html successfully scrubbing the path of the TEST_TMPDIR of the generated HTML file. It's handled by the filepath_to_regex(abs_file(os.getcwd())) scrub rule.
However, the TEST_TMPDIR that gets recorded in a gold file doesn't get handled by the above scrub rule. That's why there is a scrub rule for recorded paths in test_html:
https://github.com/nedbat/coveragepy/blob/fb52e47d41cb47099d1c0020b35d99ace1dd55c8/tests/test_html.py#L711
It handles the current recorded value, but won't work for paths recorded on Windows. So merely running the tests on Windows works, updating the gold files on Windows doesn't.


Add import for cloudstack entities

See also issue #40

Most of the time, you can use ImportStatePassthrough.

@fvhovell I'm going to close this PR in favour of PR #47
PR #47 adds support for most resource that were also in this PR, but it also adds tests to make sure importing works as expected. The resources that I omitted are resources that need custom logic and that not passed the tests.
If you want to take another stab at adding support of importing resources that are not part of PR #47, then please make sure you add an import test as the ones I also added in PR #47 to make sure the import logic works as expected.
Thanks!


perhaps make the navbar / header a bit smaller on mobile

This is a Feature Request

What would you like to be added

On mobile the navbar / floating header on the kubernetes.io docs is a whopping 80px tall, making it easily the largest floating header I've seen on a mobile website.
Why is this needed
Changing this to be a bit smaller would leave more room for reading on mobile.
Comments

current:

quick ugly hack at 80px => 40px, IMHO the content has much more breathing room

I think actually correcting the layout would be a little more involved, but I wanted to see if there's any interest in this, and if so it might make a good first issue for someone with a little more web expertise than I :-)

/help
/priority backlog

/language en

totally different layout now.


Optimize CC Pipeline

Data: 220 TiB (uncompressed)
Processing Time: 64 minutes

Job: 34 minutes
I/O: 30 minutes *needs improvement

Cost: <$70
NB
driver/executor memory: 12G*
driver/executor core: 8


Update Contributing.md for validation message output information

Fixes #2140.  Add guidelines for error message output, including specifying object handles, values, and user assistance on HOW to fix the error.
Will also update our internal docs to match, and this doc receive an overhaul and expansion in the next few weeks.

To add suggestions is well-meaning, but oftentimes values dump + VU quote is self-explanatory (and sometimes even less confusing).
Also giving really good suggestions may need more code and several slightly different messages per-check.
Would it make sense to add suggestions to only the most obvious cases. At least for the time being; I think layers full coverage should take priority over this.


No message IDs available

Describe the bug
This morning evcc completey locked up, I got an endless loop of the following error message in the logs:
May 21 08:59:16 evcc evcc[6003]: [mqtt  ] ERROR 2023/05/21 08:52:32 send: evcc/site/error: no message IDs available
The memory increased constantly and the host was then not available anymore. I had to reboot the machine to make it work again.
mqtt was available without issues, also no network errors during this time.
Steps to reproduce
I am not able to reproduce it.
Configuration details
meters:
  - name: pv
    type: custom
    power:
      source: mqtt
      topic: energy/pv/power
      timeout: 40s
    currents:
      - source: mqtt
        topic: energy/pv/currentl1
        timeout: 40s
      - source: mqtt
        topic: energy/pv/currentl2
        timeout: 40s
      - source: mqtt
        topic: energy/pv/currentl3
        timeout: 40s
  - name: grid
    type: custom
    power:
      source: mqtt
      topic: energy/meter/power
      scale: -1
      timeout: 40s
    currents:
      - source: mqtt
        topic: energy/meter/currentl1
        timeout: 40s
      - source: mqtt
        topic: energy/meter/currentl2
        timeout: 40s
      - source: mqtt
        topic: energy/meter/currentl3
        timeout: 40s
  - name: battery
    type: custom
    power:
      source: mqtt
      topic: energy/battery/power
      scale: -1
      timeout: 40s
    soc:
      source: mqtt
      topic: energy/battery/soc
      timeout: 40s

Log details
May 21 08:59:16 evcc evcc[6003]: [mqtt  ] ERROR 2023/05/21 08:52:32 send: evcc/site/error: no message IDs available

What type of operating system are you running?
Linux
Version
0.117.4

Dein mqtt Server nimmt die Nachrichten von evcc nicht an- irgendwann geht das dann nicht mehr, siehe https://github.com/eclipse/paho.mqtt.golang/commit/fbd79e4a3eca76c934da91ba8e1e9d1cf5876d53

Der mosquito server bereit allerdings mit keiner anderen Software Probleme und ich habe sehr viele integration parallel laufen.

I am not able to reproduce it.

Und Du bist Dir sicher dass das auch genau in diesem einen Fall so war?


License changed to Unlicense License

Need all contributors agree.

[x] @baiyang-lzy
[x] @chihuo2104
[ ] @scientificworld
[x] @Misaka13514
[ ] @mzwing

I agree.

I agree.

I think it's *ao*, and it has no disadvantages.
But why not use The DFDT License :D

I think it's ao, and it has no disadvantages. But why not use The DFDT License :D

I agree with you.

I think it's ao, and it has no disadvantages. But why not use The DFDT License :D

However, dfdt is made from Neko Planet, so you cannot simply choose it from the License List given by GitHub.（


Hashes are required, even if setting is disabled

Unless I'm misunderstanding how things work, I have the following setting in place:
 <checkFileHashes>false</checkFileHashes>

This is in the configuration area. However, even with this in place, I still get the following error if I don't have hashes:
Failed to execute goal com.lazerycode.selenium:driver-binary-downloader-
maven-plugin:1.0.7:selenium (default) on project lucid: RepositoryMap.xml is
not valid: cvc-complex-type.2.4.b: The content of element 'bitrate' is not
complete. One of '{hash}' is expected.

The reason I'm trying to run without the hashes is that it's not clear to me where to get them. For example, I have the following in place:
<filelocation>http://chromedriver.storage.googleapis.com/2.19/chromedriver_mac32.zip</filelocation>

When I look at the download location (http://chromedriver.storage.googleapis.com/), I don't see a hash. I see an etag, but that's about it. The etag is this:
259bb87f4ebf3b0bc4792ed203bd69f5

But the hash that is expected is:
64ef44893a87a0e470b60ff8f5fc83a588b78023

Originally the plugin didn't work unless the hash was valid.  The checkFileHashes option was added when some people requested a way to download files without checking the hash.
The plugin is designed to use hashes because this is how it checks that the file it has downloaded is correct.  As a result the RepositoryMap.xml has an XSD that requires the hash field to be filled in.
To get the hash of a file you can do the following on the command line in OSX/Linux:
openssl sha1 <FileYouWantAHashFor>
openssl md5 <FileYouWantAHashFor>

For windows have a look at:
https://support.microsoft.com/en-us/kb/889768
It looks like the chromedriver team no longer supplied MD5 or SHA1 hashes of their files so that you can check that your download is not corrupt and is the expected file, this is disappointing.


Added how to poke around in a running container

The exec command can be used to launch another process in an already running container.

Thanks!


In contact:addrType the elements sp and pc (contact::Address fields province and postal_code) are optional

The elements sp, pc are optional but the struct Address in module contact makes them compulsory.
Here is the schema definition from RFC 5733, page 31
<complexType name="addrType">
  <sequence>
    <element name="street" type="contact:optPostalLineType"
     minOccurs="0" maxOccurs="3"/>
    <element name="city" type="contact:postalLineType"/>
    <element name="sp" type="contact:optPostalLineType"
     minOccurs="0"/>
    <element name="pc" type="contact:pcType"
     minOccurs="0"/>
    <element name="cc" type="contact:ccType"/>
  </sequence>
</complexType>

Should I fix this and submit PR?
This was originaly filed as masalachai/epp-client#96

Yes, please!

Ok, I'll start on monday.
Note that this will imply some API changes.


[feat] I hope to add the function of dynamically changing the theme of the window

Describe the problem
I'd like to be able to add a method like: setTheme to the window to change the theme color, not just on startup!
Describe the solution you'd like
Dynamic theme color modification
Alternatives considered
No response
Additional context
No response

Duplicate of https://github.com/tauri-apps/tauri/issues/5279. Thanks for the request.


TypeError: this.$__path is not a function when used with mongoose

Please tell us about your environment:

winston version?

[ ] winston@2
[x] winston@3

node -v outputs: 12.13.1
Operating System? (Windows, macOS, or Linux) MacOS
Language? (all | TypeScript X.X | ES6/7 | ES5 | Dart) Typescript

What is the problem?

TypeError: this.$__path is not a function
What do you expect to happen instead?

error to be logged
Other information

see https://github.com/davidmarkclements/fast-safe-stringify/issues/43

Any update on this?


"Failed to render API Response" has no traceback

Seeing lots of messages logged from here, but without a traceback. https://github.com/lbryio/lbry/blob/master/lbrynet/lbrynet_daemon/auth/server.py#L331
Makes it hard to debug what might have gone wrong.

pretty sure this is fixed


[Develop] No way to test App:run

The way the container creates the dependencies make's writing a test case impossible in PHPUnit for the Run method.
I think the base problem is here...
Container.php
//...
        /**
         * This service MUST return a shared instance
         * of \Slim\Interfaces\Http\EnvironmentInterface.
         *
         * @param Container $c
         *
         * @return EnvironmentInterface
         */
        $this['environment'] = function ($c) {
            return new Environment($_SERVER);
        };

        /**
         * This service MUST return a NEW instance
         * of \Psr\Http\Message\ServerRequestInterface.
         */
        $this['request'] = $this->factory(
            /**
             * @param Container $c
             *
             * @return ServerRequestInterface
             */
            function ($c) {
                return Request::createFromEnvironment($c['environment']);
            }
        );

//...

Then in the App class...
//...
    /**
     * Run application
     *
     * This method traverses the application middleware stack and then sends the
     * resultant Response object to the HTTP client.
     */
    public function run()
    {
        $request = $this->container->get('request');
        $response = $this->container->get('response');

        // Traverse middleware stack
        try {
            $response = $this->callMiddlewareStack($request, $response);
        } catch (SlimException $e) {
            $response = $e->getResponse();
        } catch (Exception $e) {
            /** @var callable $errorHandler */
            $errorHandler = $this->container->get('errorHandler');
            $response = $errorHandler($request, $response, $e);
        }

        $this->respond($response);

        return $response;
    }
//...
}

There is no way to override the Request class since the Container class is only 1 way.
The request class is not mergable from a user specified request since the factory method overwrites it.

Why can't you just mock the container?

Hmm, I guess one could do that by implementing their own container with stubs... but that seems a bit out of scope for that test.

Perhaps I can add a static method that somehow overrides the constructor... but I think that might be a problem for another day :)

You could use the existing container class and set another request factory ($container['request'] = $this->factory(function ...)?

So in the test case, create your own container and then override it then pass it into App.
That should work i guess. Both Environment and Request need to be overridden.


[build-presets] Add preset for XCTest contributors

What's in this pull request?

The swift-corelibs-xctest README will encourage contributors to use this lightweight preset when developing locally. The CI preset is more comprehensive, but takes longer to run.
Resolved bug number: None

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci smoke test and merge

@tkremenek Thanks for the CI kick, but it doesn't appear to have worked. Maybe it needs a "please"? 😝

@swift-ci Please test and merge

The CI failure was unrelated. Merging!


fixed issue #680 #681 #758 #769 - [closed]

In GitLab by @liyanq528 on Dec 12, 2017, 03:26
Merges 20171205 -> dev

In GitLab by @liyanq528 on Nov 13, 2019, 10:17
restored source branch 20171205

In GitLab by @wknet123 on Nov 13, 2019, 10:17
the coverage is  7.7% check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:17
the coverage is  7.7% check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:17
the coverage is  7.6% check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:17
the coverage is  7.6% check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:17
the coverage is  7.6% check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:17
closed


[RELEASE-1.10] Fix missing change in back-port #565

Changes

Add missing change from https://github.com/knative-extensions/reconciler-test/pull/565

/approve


Good Job! LIN.

Many thanks! We hope that our work could provide new guidance for future rumor detection about breaking events on social media.


fix(linting): address a few linting issues

Description

To fix failing CI:https://github.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/actions/runs/3470441039/jobs/5798647744
Verification Steps

make lint should run successfully
Checklist (Definition of Done)

[x] All acceptance criteria specified in JIRA have been completed
[x] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[x] Required metrics/dashboards/alerts have been added (or PR created).
[x] Required Standard Operating Procedure (SOP) is added.
[x] JIRA has been created for changes required on the client side

thanks @machi1990 !

Thank you @valeriiashapoval & @pawelpaszki
let's see what CI says


applycmd for Mesh command

Notes for Reviewers
This PR fixes #8292
Signed commits

[ ] Yes, I signed my commits.

@alphaX86, @theBeginner86, @abdullah1308    please i will need your review and help on this i've been figuring out how to try and implement the tls serviceMesh operation in the sendOperationRequest()

@Philip-21 Let's discuss this on Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

Hi @Philip-21, please have a look at the golangci-lint issues found.

Yh I saw it , I haven't set up the applycmd to be used in the mesh.go I'm still trying to implement service mesh operations like tls configurations before I declare it as a mesh command . The pr is still under work .


PostgreSqlPlatform does not support Migrator

Describe the bug
After upgrading to mikro-orm 4.5.5 we're seeing failures in our test code around migrator not being supported. Did something change in this release?
Stack trace
    PostgreSqlPlatform does not support Migrator

       7 |   let g = db.getSchemaGenerator()
       8 |   await g.dropSchema(false, true)
    >  9 |   const m = db.getMigrator()
         |                       ^
      10 |   await m.up()
      11 |

      at PostgreSqlPlatform.getMigrator (../node_modules/@mikro-orm/core/platforms/Platform.js:153:15)
      at MikroORM.getMigrator (../node_modules/@mikro-orm/core/MikroORM.js:105:42)
      ...

To Reproduce
Steps to reproduce the behavior:
Invoke getMigrator on a postgres instance
Expected behavior
getMigrator should be supported with postgres.
Additional context
Upgraded from 4.5.5-dev.1 to 4.5.5
Versions
4.5.5

The method is there, this sounds more like some dependency resolution issue. How does your package.json look like? Make sure you have all the package versions aligned (e.g. core package is on the exact same version as driver package).
Or provide full repro.


What's the future of Smock?

Hi,
In my team, we use smock for some core projects but, lately, we have been wondering whether the project is still supported or not.
The lack of activity is what triggered the question so could you please tell us if you are continuing to support Smock or are there any plans to deprecate it.
Thank you in advance!

Hi @wei3erHase, I appreciate your quick response!
It's clearer for me, please let us know if things change in the future.
Thank you very much!


chore: bdd-nh-1585301711 to 0.0.1

chore: Promote bdd-nh-1585301711 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


chore: add sms and email(from_name, to_mail, to_name)

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


added RotateLinkedList.cpp in c++ directory

I have added code on rotating linked list in c++ directory. Kindly go through the code and consider the pull request if its valid.

@rakshitnayak please change the folder path to /codes/C++. Thank you!


feat: bee 2.2

New Bee release will remove Bee Debug API endpoint :1635 thereby all Debug API communication must be changed to the standard Bee API port :1633.
Additionally, options for restricted API has been also removed.

Release v3.2.0 addresses this.


November+January: Upcoming freeze+FREEZE for Debian

Hello,
Debian ships yosys and the whole pipeline will be in the next release. The nice development of its userbase casen be monitored here. The next freeze is early November and an ultimate one early January (https://wiki.debian.org/DebianStretch). If there is an upcoming update of yosys or the downstream tools that you would prefer to see shipped instead of your current release then please kindly be aware of November, preferably.
Many thanks and regards,
Steffen

Good to know. I plan to make the Yosys 0.7 release shortly after ORCONF, i.e. mid October. So that should work out just fine.
I'll leave this issue open until the release.

cc @azonenberg; maybe we should try to get a somewhat better gp4/cells_sim.v into yosys by this point?

As instructed on https://www.reddit.com/r/yosys/comments/57xfgs/towards_yosys_07_please_test_now/ I have compiled the yosys head and it worked just fine on my little project. Somewhat stimulated by @whitequark, on my whishlist would be wand and wor (or a kick-but to go and implement it myself if that is feasible for mere mortals).

@whitequark: I already have a few updates in my fork that aren't yet merged. I'll try and get cells_sim support for the remaining hard IP (even if we don't have back end toolchain support yet) asap.

I have now released Yosys 0.7.
https://www.reddit.com/r/yosys/comments/5avscm/yosys_07/

Many thanks to @rubund for updating the yosys and berkeley-abc packages
https://packages.qa.debian.org/y/yosys.html


[BUG] Mapping type issue after upgrading from Elastic search 7.8.1 to Opensearch 1.0.0

Describe the bug
In our application we are  ingesting our  logs  using logstash.Hence when we upgrade  our application from elastic search 7.8.1 to opensearch 1.0.0 we are  facing  some issues in  logs ingestion..  we found  differenes in terms of mapping type between these two versions. they are listed below
Elasticsearch 7.8.1(indices data)
{
"_index" : "logs-2021.07.28",
"_type" : "_doc",
"_id" : "w2Y07XoBygU95wt4Y0zX",
"_score" : 1.0,
"_source" : {
"@timestamp" : "2021-07-28T12:59:44.030Z",
"type" : "filebeat",
"agent" : {
"type" : "filebeat",
"ephemeral_id" : "08dcb535-c6a0-4a5a-9435-2c2209187e49",
"id" : "0fe68f2b-2e65-4df1-924d-fd2c05ce0b29",
"hostname" : "log-shipper-5nwbh",
"version" : "7.9.1",
"name" : "log-shipper-5nwbh"
}
}
Opensearch 1.0.0(Indices data)
{
"_index" : "logs-2021.07.28",
"_type" : "filebeat",
"_id" : "XYsI7XoBwUWR7YpW5kfz",
"_score" : 1.0,
"_source" : {
"ecs" : {
"version" : "1.5.0"
},
"@version" : "1",
"type" : "filebeat",
"log" : {
"offset" : 6806374,
"file" : {
"path" : "/var/lib/docker/containers/0f91ce2eeddd9830b62bf8fa8249-json.log"
}
}
}
Error from logstash
Could not index event to Elasticsearch. {:status=>400, :action=>["index", {:_id=>nil, :_index=>"logs-2021.07.28", :_routing=>nil, :_type=>"filebeat"}, #LogStash::Event:0x6ef9010],
:response=>{"index"=>{"_index"=>"logs-2021.07.28", "_type"=>"filebeat", "_id"=>"DeSd8HoBBBXZ9WLoDE-f", "status"=>400,
"error"=>{"type"=>"illegal_argument_exception", "reason"=>"Rejecting mapping update to [logs-2021.07.28] as the final mapping would have more than 1 type: [_doc, filebeat]"}}}}
To Reproduce
Steps to reproduce the behavior:
-> create an index on elasticsearch 7.8.1
-> load some data which contains a  field named type
eg: "type" : "filebeat"
-> check the  _type metadata  value  of loaded document
eg:  "_type" : "_doc"
-> upgrade elasticsearch 7.8.1 to opensearch 1.0.0
->  load some data which  contains a  field named type
eg: "type" : "filebeat"
-> now the data load will get  fail with the below mentioned error.
Expected behavior
Even after the upgrade the _type metadata should be same i.e  _doc and data should be loaded successfuly.
Plugins
Please list all plugins currently enabled.
Screenshots
N/A
Host/Environment (please complete the following information):

OS:  SLES 15
Version : 15

Additional context
N/A

Is this the client sending the wrong data based on some version logic in it? Looks similar to https://github.com/opensearch-project/OpenSearch/issues/955, which was resolved by https://github.com/opensearch-project/OpenSearch/pull/847. Try setting override_main_response_version, see https://opensearch.org/docs/clients/agents-and-ingestion-tools/index/ for details.

@anish-adrot Did you get a chance to try this? Also what version of logstash and logstash-outoput-elasticsearch are you running?

@dblock  we are  working on it. will let you know  once we are done.

@dblock   we  have  tried  the  fix  as you mentioned
PUT "/_cluster/settings?pretty" -H 'Content-Type: application/json' -d'
{
"persistent" : {
"compatibility" : {
"override_main_response_version" : true
}
}
}
'
But  we  have  queries
-> While  Rolling upgrading , this  settings  applies  only  when  all  pods are  up and  running,if  we  try  to  run  before  the  pod is  up .we are getting  error.
-> So  our concern  due  to this  will  there  be  any  data  loss ? because  this  setting applies  only when  all pods are up.what  if in paralle ingest logs  to elasticsearch while  in upgrading process

@anish-adrot, did the setting work? (minus your question)
You make a good point. A few ideas.

If I am not mistaken, you could put this setting in opensearch.yml as well, so each new node would come up in the backwards compatibility mode. Please correct me if I am wrong or if I am missing anything @saratvemulapalli @mch2?
Depending on your setup can you bring up a first node that's not getting traffic, but use it to set the cluster-wide setting?
What happens in your system if ES is down? You could have all kinds of failures to write, so hopefully the system can/is designed in a way where it retries, or fails over preventing data loss.

If I am not mistaken, you could put this setting in opensearch.yml as well, so each new node would come up in the backwards compatibility mode. Please correct me if I am wrong or if I am missing anything @saratvemulapalli @mch2?

Correct.

Hi @anish-adrot, you can set this in opensearch.yml as:
compatibility.override_main_response_version: true

I can see that documentation only defines the API request.  Thanks for pointing this out, I have cut an issue to have our documentation updated.


problems with space in project path path

Hi,
my grunt task fails with the following error message : "ERROR: No coverage files found."
Istanbul works well from the command line.
The coverage.json file exists. I have a space in my project path.
The coverage path that is passed to istanbul is built using (index.js:53)
coverageFolder = path.join(process.cwd(), options.coverageFolder),

and thus contains the space
After a bit of reverse engineering I found the following. At the row lib/util/file-matcher.js:33 in instanbul the different paths passed to istanbul are joined using spaces. My coverage directory it thus viewed as 2 paths instead of one resulting in the error.
changing to a relative folder by changing the row to
coverageFolder =options.coverageFolder,

solves the issue. I guess this would introduce regressions as well...
I don't know if this is an instanbul issue or a grunt-mocha-istanbul issue.
Thanks
Regards,
pouc

hey @pouc, that's most likely a istanbul problem AFAIK. it should accept any type of path regardless if it has special characteres, spaces, unicode, etc.

this should have been fixed in 5.0.0. coverageFolder and root can have spaces now


Improve CWL Viewer Error Handling

Fixes ga4gh/dockstore#1518.

Look for a 200 response when polling the queue that might have a
cwltoolStatus of ERROR.
Reduce polling frequency to every 500ms instead of 250ms.
Stop polling when leaving the page.
Display the error message that comes back from CWL Viewer instead
of a generic message.

Oops didn't realize it failed, though the failure looks to be a style issue.


Remove unused imports

Remove unused imports from scripts/benchmark.py and tests/test_helpers.py.

Thanks, but we use the imports in the internal version of tiktoken (the relevant code is redacted for open source). The scripts folder isn't packaged, so these don't affect users

Thanks for the quick reply @hauntsaninja. I'm curious, if the relevant code is redacted for open source wouldn't it make sense to redact these import as well?

Maybe? I don't think these imports leak very much information.
If there was infinite time, it'd make sense to set up the redaction, but there isn't — I don't get much time to work on tiktoken as it is.


Update aws-java-sdk-core to 1.12.684

About this PR
📦 Updates com.amazonaws:aws-java-sdk-core from 1.12.459 to 1.12.684
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #729.


Comma-fix and clarity

Attempting to improve clarity and grammar slightly, while inserting a needed comma in the second sentence.

Thanks, docs improvements are always appreciated!


QuickReply Text and Image

Hello Ricky,
Little add-up to your code.
Missing test... don't know how... for now ;)
Alexandre

Coverage decreased (-0.4%) to 99.578% when pulling b00333710bd059c981a81b9307bd370a66c3e1a6 on achoisy:master into 3b9da09e44d2803b87b45f2ba9d9ddce5ceffa72 on rickydunlop:master.

Can you update a few things before I merge this?

Add a test with a quick reply that contains an image, this will get the coverage back to 100%.
Can you open a new pull request with the Graph API version change as it is unrelated to this PR. The latest is v2.8.

Cheers

Thank you for showing me the path. I ll be back asap.

Coverage decreased (-0.4%) to 99.578% when pulling a08f5b71c608233f11e09ca0f2b7e58faa0d11f0 on achoisy:master into 3b9da09e44d2803b87b45f2ba9d9ddce5ceffa72 on rickydunlop:master.


Feature/info-tab-external-links displays the important information only

Displays the important info of an external service link in the info tab of workflows and tools
Adds new angular pipe for deconstructing a url into just the pathname
Defines a unit test for the new pipe

ga4gh/dockstore#1047

Before

After

I'm wondering if instead of doing gitrepo/tree/version, you could do gitrepo:version. Might look cleaner.

Going to be merge conflicts for every test file modified because they were all swapped to .ts files

@garyluu as discussed, please help Ryan with the merge since these merge conflicts are probably a bit too painful

Ryan and I just went through resolving the merge conflicts and I think we are good to go. But we'll ping you if we get stuck. Thanks!


Change Export nav link to a button.

The Export button shouldn't behave as a nav link and shouldn't become active when clicked.

Consider changing its appearance too, in order to differentiate it from nav links.


How to fix the problem core dumped ??

when i try to run the train.sh i got this error
 can anyone help me ???

@GhofraneHADHRI
Any chance you fix this? Running into same issue


fix; Solved Mystery of Missing README?

I may have solved why https://RalphHightower.github.io/blog/ gets HTTP404.
_config has: jekyll-readme-index
But Gemfile d idn't have:gem "jekyll-readme-index", "~> 0.3.0"

#85 [!] Q was an error parsing Gemfile: Undefined local variable or method `jekyll' for Gemfile. Bundler cannot continue.
from /home/runner/work/blog/blog/Gemfile:50
-------------------------------------------
gem "jekyll-optional-front-matter", "~> 0.3.2"

gem install jekyll-readme-index

gem "jekyll-redirect-from", "~> 0.16.0"
-------------------------------------------
Error: The process '/opt/hostedtoolcache/Ruby/3.3.4/x64/bin/bundle' failed with exit code 4


⚠ Thumbnails API Endpoint has degraded performance

In 985821f, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1434 ms

Resolved: Thumbnails API Endpoint performance has improved in 0569780 after 4 minutes.


Error creating rule  - SUNBURST network beacons

When trying to create the rule SUNBURST network beacons, I get an error that says "One or more entity mappings have been defined under the new version of Entity Mappings. These will not appear in the query code. Any entity mappings defined in the query code will be disregarded."

@anfisher1967 - Thanks for the feedback.  This has been fixed in #2159 and some others have been fixed in #2226


Remove support for Python versions < 3.5

On top of #596

I think it is definitely time to drop Python 2 support.  Many libraries are already dropping support for 3.7!

Other libraries have maintainers...   See e.g. this and this.


Documentation is incorrect for issecret, fails to note secret variables do not get set as env vars

The documentation mentions:

This doesn't update the environment variables, but it does make the new variable available to downstream steps within the same job.
When issecret is set to true, the value of the variable will be saved as secret and masked from the log.
...
Subsequent steps will also have the pipeline variable added to their environment.

This is confusing an inaccurate in two ways:

Environment variables are updated, just not immediately for the running job. Subsequent jobs have access to the new variable in YAML via $(MACRO_SYNTAX) and in tasks via environment variable.
Secret variables are only available via $(MACRO_SYNTAX) not via environment variable, and this is not mentioned in this documentation page.

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

ID: dd7e0bd3-1f7d-d7b6-cc72-5ef63c31b46a
Version Independent ID: dae87abd-b73d-9120-bcdb-6097d4b40f2a
Content: Define variables - Azure Pipelines
Content Source: docs/pipelines/process/variables.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thanks @stewartadam! I'm updating the docs now. For the second issue raised (that you can only use macro syntax with secret vars), I need to follow-up with the product team.


Multiple “Edit” functions not working

(Broken as of 4/27/22 (April 27th)) For the purposes of hardware/browser info if it is needed, I am encountering this issue on a (school-owned) Lenovo 300e Chromebook (2nd Gen) with the Chrome browser. I have not run into this issue on my personal Chromebook— a HP X360– which also runs on the Chrome browser.
In the “Edit” tab, the following tools are all experiencing this issue; Perspective, Bright/Contrast, Curves, Hue/Saturation, Invert, Distort, Noise (Invert, Distort, and Noise have a slightly different case, which I will explain), Tilt Shift, To Alpha, Triangle Blur, and Unsharp Mask.
All of the above tools, which normally have a special menu, do not have this menu. Excluding Invert, Distort, and Noise, the pop-up for the menus show, they it only contain the text explaining what the tool does, the Okay/Cancel buttons, and the close button in the top right. There are no sliders anywhere for actually using the tool. Pressing the Okay button does not do anything, and undoing does not track that anything has changed because of this.
Invert has a main browser pop-up reading; “Couldn’t apply the edit action” and an in-site notification (like the undo/redo ones) that contradicts this, saying “Invert applied.” This also causes the undo and redo functions to bug out and not properly register inputs.
Distort and Noise, when clicked, do absolutely nothing. No windows show or anything.
Everything else on the site seems to be working as intended, but this issue makes the site borderline unusable when I’m at school.

Hey there! Thanks for the quick reply. Yes, the spinning cube does show up. I've tried restarting my Chromebook as well as clearing the cache, but that didn't fix anything. It worked in the past and only stopped working maybe 2-4 days ago, but my Chromebook hasn't updated in that timespan, so I'm not sure what could've caused this.

Yes, the spinning cube does show up

That is on the Lenovo with the above described issue, correct? That would be indeed surprising.

It is.

Ah, minor issue with that. I don't seem to be able to use the F12 shortcut on the Chromebook experiencing this error. I checked to make sure it was the correct command input, but it doesn't seem to be working. The Chromebook experiencing this is on version 112.0.5615.134 according to what my settings say. I haven't restarted to finish the update on my personal Chromebook yet, so I'm going to do that and see if it's a Chrome issue.

Alright! Thank you again for the quick response, it means a lot, and thank you for creating such a useful program. Honestly, the best open-sourced art program I've ever used!

As of right now, I'm not able to provide updates on this. My school messed up something with its security extension and somehow managed to block literally every site (even canvas and stuff, lol???). I'll check this as soon as I am able to, but I'm not quite sure when that'll be.

https://644e2e87cbc6c42f7c1d0f36--bb-sandbox.netlify.app/

Yes, the same issue happens here.

It's... odd. It still works fine on my personal Chromebook, and WebGL is still functional on my school one, so I'm not quite sure. I've given you as much info as I can, but if you have anything else you think I could be able to share, please let me know.

it just now effected me, same thing it shows that WebGL works but some edit buttons dont work, so it maybe some os problem oh and im using my own personal chromebook for this btw.

same here

This keeps getting weirder
all the WebGL test pages work just fine but kleki appears to say otherwise??

Yea it is pretty weird, especially because it affects older Kleki versions too despite them working fine last month.
This is a long shot: Could you temporarily disable all extensions and see what happens?

I don't have any extensions that could be interfering
I know little to nothing about tech but my guess is maybe a checker code in kleki isn't working correctly & is breaking everything with a false WebGL  test??
i dunno if that could even happen or if kleki has any code like that
Btw is this supposed to happen?

Unable to get on the device with the broken version at the moment, but this is even more confusing to me, as my personal Chromebook is still completely unaffected.

@bitbof

The content of the error is different, so it may be completely unrelated, but the cause of the webGL-related trouble I encountered was a bug in the graphics driver.
Crash in Intel HD 630 driver - Intel Communities
This link is an example of WebGL crashing due to a bug in Intel's graphics driver.
I thought that if Chromebook issues were specific to a specific manufacturer, it might have something to do with the graphics board and the graphics drivers installed, but this is just a guess based on my experience.
Sorry if it doesn't matter.

Haven't had time recently to check on this thread, but since the implementation of the WebGL failure feedback, my personal Chromebook- which was fine before this update- now also shows that WebGL is failing, when it had seemed to be unaffected beforehand.
I posted the plaintext file for the originally affected Lenovo 300e (gen 2) Chromebook in the reported issue you linked, and I'll try to keep an eye on both pages a little better, haha!

IT"S FIXED NOW

Hey! Update from me. The edit menus have been restored on both devices, including the one that was initially experiencing this issue! I'm glad everything has gotten mostly situated.
Thank you so much for having gotten on this issue so quickly. I really, really appreciate it.


Failed at the<EMAIL_ADDRESS>

npm ERR! node v6.10.2
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: node scripts/build.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node scripts/build.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the node-sass package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/build.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs node-sass
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls node-sass
npm ERR! There is likely additional logging output above.
please help solve this problem,thank you very much!

Sorry, you didn't follow the issue template, so we can't help you.


Merge refs/heads/1.6/feature/478 into alpha

Merge refs/heads/1.6/feature/478 into alpha

This PR has been merged into alpha. Thanks for your contribution!

This PR has been merged into alpha. Thanks for your contribution!


Restyle colors

@omarkilani The colors were a bit off so I swapped them with the colors in your UI :)

@omarkilani lmk if any concerns

No concerns - thanks!
There's an errant text-white somewhere in the page that overrides the fontColor or something.
Didn't have time to dig into it.


Simplify OTEL integration to reduce deps

Description
This PR has been created after looking at Micrometer Observation integration in PulsarTemplate here: https://github.com/spring-projects/spring-pulsar/blob/main/spring-pulsar/src/main/java/org/springframework/pulsar/core/PulsarTemplate.java#L298. Many thanks to @salaboy for pointing this out.
The main idea was to remove explicit OTEL dependency from Dapr Messaging since Micrometer Observation should be able to handle traceparent generation behind the scenes and OOTB.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: N/A
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation

@artursouza and @cicoyle this is a tiny PR to get rid of extra dependencies and ensure that we stay in the realm of Spring and Micrometer without having to bring in OTEL explicitly.


Fixing shadow variables in mesh classes [shadow-mesh-dev]

This PR partially addresses issue #2701.
I tried to make minimal changes and keep the spirit of the original variable names. Hopefully, splitting these into small PRs will assist the original authors and/or reviewers in assessing the changes.
The compiler has no trouble determining which variable is intended but the choice is not always obvious to developers who may not be aware of one or the other of the possibilities.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#2738
@mlstowell
@pazner
@YohannDudouit + @pazner
12/28/21
⌛due 01/11/22
⌛due 01/18/22

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

@YohannDudouit, could you take a look when you get a chance?

@mlstowell, when you have time, can you address the outstanding comments?

@mlstowell, when you have time, can you address the outstanding comments?

Thanks for the reminder. I think this may be ready now

Merged in next for testing...


PydanticOutputParser with RunnableWithMessageHistory arises ValueError: Got unexpected message type form _get_output_messages

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
Definition of the output using PydanticOutputParser:
class Joke(BaseModel):
    setup: str = Field(description="question to set up a joke")
    punchline: str = Field(description="answer to resolve the joke")

parser = PydanticOutputParser(pydantic_object=Joke)

Error Message and Stack Trace (if applicable)
Error in RootListenersTracer.on_chain_end callback: ValueError()

Description
Since I'm trying to build a conversational agent, I am using RunnableWithMessageHistory to automatically add the messages to the history.
I would expect everything to work fine, but I obtain the following error
Error in RootListenersTracer.on_chain_end callback: ValueError()

from
def _get_output_messages(
    self, output_val: Union[str, BaseMessage, Sequence[BaseMessage], dict]
) -> List[BaseMessage]:
    from langchain_core.messages import BaseMessage

    if isinstance(output_val, dict):
        output_val = output_val[self.output_messages_key or "output"]

    if isinstance(output_val, str):
        from langchain_core.messages import AIMessage

        return [AIMessage(content=output_val)]
    elif isinstance(output_val, BaseMessage):
        return [output_val]
    elif isinstance(output_val, (list, tuple)):
        return list(output_val)
    else:
        raise ValueError()

Since I used PydanticOutputParser, here output_val is a type dict object containing a Joke object:
output_val = {'output': Joke(joke='Why did the egg refuse to tell jokes? Because it cracked under pressure!', message='Here a joke for you!')}

Joke objects do not seem to be supported by _get_output_messages.
How can I get it to work? I would like for the history to update automatically when using RunnableWithMessageHistory and a PydanticOutputParser.
System Info
langchain==0.1.16
langchain-cli==0.0.19
langchain-community==0.0.32
langchain-core==0.1.42
langchain-experimental==0.0.10
langchain-openai==0.0.8
langchain-text-splitters==0.0.1
python version 3.10.9
OS: windows11

Hi @spagliarini, this is likely an error in user code -- but there's not enough information to be able to identify the problem.
Could you include an MRE and/or a full stack trace?

Hi @eyurtsev , here a "working" MRE.
from os import getenv

from langchain_core.chat_history import BaseChatMessageHistory
from langchain_core.messages import BaseMessage
from langchain_core.output_parsers import PydanticOutputParser, StrOutputParser
from langchain_core.prompts import ChatPromptTemplate, MessagesPlaceholder
from langchain_core.pydantic_v1 import BaseModel, Field
from langchain_core.runnables import RunnablePassthrough, ConfigurableFieldSpec
from typing import List

from langchain_core.runnables.history import RunnableWithMessageHistory
from langchain_openai import ChatOpenAI

STORE = {}

class InMemoryHistory(BaseChatMessageHistory, BaseModel):
    """In memory implementation of chat message history."""

    messages: List[BaseMessage] = Field(default_factory=list)

    def add_message(self, message: BaseMessage) -> None:
        """Add a self-created message to the store"""
        self.messages.append(message)

    def clear(self) -> None:
        """Clear the chat history."""
        self.messages = []

    def get_messages(self) -> List[BaseMessage]:
        """Get the chat history."""
        return self.messages

def get_session_history(user_id: str, conversation_id: str) -> BaseChatMessageHistory:
    """Get the chat history for the current session without streamlit."""
    if (user_id, conversation_id) not in STORE:
        STORE[(user_id, conversation_id)] = InMemoryHistory()
    return STORE[(user_id, conversation_id)]

# define output parser
class Joke(BaseModel):
    Question: str = Field(description="question to set up a joke")
    Joke: str = Field(description="answer to resolve the joke")

parser = PydanticOutputParser(pydantic_object=Joke)

# define prompt
prompt_template = """
You are a funny chatbot that can tell jokes.
Please tell me a joke following the schema and return a json with the user question and the joke:

Question: {question}
Joke
"""

prompt = ChatPromptTemplate.from_messages(
    [
        (
            'system',
            prompt_template,
        ),
        MessagesPlaceholder(variable_name='history'),
        ('human', '{question}'),
    ]
)

# define llm
llm = ChatOpenAI(temperature=0, model='gpt-4-1106-preview', openai_api_key=getenv('OPENAI_KEY'), streaming=False)

ai_response = 'NOT VALID ANSWER!'

# define the full chain
full_chain = prompt | llm | parser

# define user question
question = 'Tell me a joke about cats.'

with_message_history = RunnableWithMessageHistory(
    full_chain,
    get_session_history,
    input_messages_key='question',
    history_messages_key='history',
    history_factory_config=[
        ConfigurableFieldSpec(
            id='user_id',
            annotation=str,
            name='User ID',
            description='Unique identifier for the user.',
            default='',
            is_shared=True,
        ),
        ConfigurableFieldSpec(
            id='conversation_id',
            annotation=str,
            name='Conversation ID',
            description='Unique identifier for the conversation.',
            default='',
            is_shared=True,
        ),
    ],
)

# get the full chain response
json_ai_response = with_message_history.invoke({'question': question}, config={'configurable': {'user_id': '0', 'conversation_id': '0'}})

The output error is the following:
Error in RootListenersTracer.on_chain_end callback: ValueError()

coming from line 402 of history.py as mentioned in the previous comment.
The idea is to save a json object in the memory but it seems like it's not supported at the moment.

+1 on problem. There seems to be no way to use any other type other than Union[str, BaseMessage, Sequence[BaseMessage], dict] in RunnableWithMessageHistory. I would like for somebody to guide me on how to use chain in RunnableWithMessageHistory, if my chain parses the input on the end.


Feature Request: Support DWARF version 5

DWARF version 5 is out now, and is the default for newer GCCs. DWARF 5 should be upwards compatible, so the changes necessary to support it should be minimal.
Currently, loading a file with DWARF version 5 debugging data results in the following warning (and no debug information being loaded):
Error during DWARFAnalyzer import
Only DWARF version 2, 3, or 4 information is currently supported.
ghidra.app.util.bin.format.dwarf4.DWARFException: Only DWARF version 2, 3, or 4 information is currently supported.
	at ghidra.app.util.bin.format.dwarf4.DWARFCompilationUnit.readCompilationUnit(DWARFCompilationUnit.java:157)
	at ghidra.app.util.bin.format.dwarf4.next.DWARFProgram.bootstrapCompilationUnits(DWARFProgram.java:507)
	at ghidra.app.util.bin.format.dwarf4.next.DWARFProgram.<init>(DWARFProgram.java:214)
	at ghidra.app.plugin.core.analysis.DWARFAnalyzer.added(DWARFAnalyzer.java:134)
	at ghidra.app.plugin.core.analysis.AnalysisScheduler.runAnalyzer(AnalysisScheduler.java:186)
	at ghidra.app.plugin.core.analysis.AnalysisTask.applyTo(AnalysisTask.java:39)
	at ghidra.app.plugin.core.analysis.AutoAnalysisManager$AnalysisTaskWrapper.run(AutoAnalysisManager.java:688)
	at ghidra.app.plugin.core.analysis.AutoAnalysisManager.startAnalysis(AutoAnalysisManager.java:788)
	at ghidra.app.plugin.core.analysis.AutoAnalysisManager.startAnalysis(AutoAnalysisManager.java:667)
	at ghidra.app.plugin.core.analysis.AutoAnalysisManager.startAnalysis(AutoAnalysisManager.java:632)
	at ghidra.app.plugin.core.analysis.AnalysisBackgroundCommand.applyTo(AnalysisBackgroundCommand.java:58)
	at ghidra.framework.plugintool.mgr.BackgroundCommandTask.run(BackgroundCommandTask.java:102)
	at ghidra.framework.plugintool.mgr.ToolTaskManager.run(ToolTaskManager.java:319)
	at java.base/java.lang.Thread.run(Thread.java:834)

Related request for version 1: https://github.com/NationalSecurityAgency/ghidra/issues/2985

@dev747368 Any progress to report on this? I may jump in and help depending on what the status is.

No progress to report on DWARF 5.  There are some changes / improvements in the pipeline improving function parameter / local variable storage.

@dev747368 Has there been any progress on this issue? I'm encountering a lot of binaries with DWARF 5 debugging data, so it would be great if support for DWARF 5 was added to Ghidra.

Any progress on this issue? Support DWARF 5 please. >_<

The referenced commit adds support for DWARF5, minus .dwo split dwarf features.  (support for that is still in the works)


add standard label for the filtering of cluster

Why are these changes needed?

Related issue number

#338
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

I create #338 for other reason. I will cut a PR to add labels to manifest.
I think what you need to do here is to make sure apiserver just filter out the cluster with some label like app.kubernetes.io/managed-by=kuberay-apiserver

It looks good to me now


Export and new action issue

Hi there,
I've an issue when I want to disable the "new" action on a particular entity and when the export action is enable. Both actions disapears of the back-office instead of keeping the export action displayed.
# app/config/packages/easy_admin.yaml

easy_admin:
    site_name: 'Easy admin"'
    entities:
        # List the entity class name you want to manage
        User:
            class: App\Entity\User
            disabled_actions: ['new']
            export:

    ...

Cheers Wandi :+1:

Thx @valr13 , fixed in 0.6.4


getBoundingClientRect called on undefined in TextEditorComponent sometimes

Continuing from https://github.com/steelbrain/linter-ui-default/issues/355
What we know so far

The error comes from pixelPositionForMouseEvent
The error is often from commands like core:confirm, core:copy etc
Either scrollContainer or lineTiles is undefined in Text Editor when those are invoked and it's not handled in Atom internally

First seen on 1.19.0-beta0, still hitting on 1.19.0
My rough guess about what it could be (not dug deep into sources so might be very wrong):

A race condition after switching pane, those refs aren't yet resolved when the user moves mouse
Either of these is two occurrences has the object undefined when the text editor is out of focus before moving mouse because in reports, one user had just closed dev tools, other kept switching, closing/reopening pane items
Maybe the pane component doesn't check if it's been disposed and the items no longer exist and the bug is actually in linter-ui-default requesting mouse positions for closed tabs? (https://github.com/steelbrain/linter-ui-default/pull/420 fixes linter-ui-default side)

Definitely reproducible by myself and others on steelbrain/linter-ui-default#355
Sometimes, not always, when closing tabs, that error is raised through the linter-ui-default package.

This is a private API but I'd still like to help out. I'm pretty confused because both of those refs should be defined after the editor is initially constructed and should never be invalidated.

@as-cii Do you have any ideas on this one. As far as I can tell both the scrollContainer and lineTiles refs are created on construction of the component and never go away.

Prior to Atom 1.19.2 (specifically, before #15317), the lineTiles ref may have not been there if we hadn't performed the initial measurements, so I wonder if this problem got fixed as part of that pull-request. @robjtede @steelbrain: can you still reproduce this on the latest version?

I haven't had any issues since updating linter-ui-default to v1.6.7 (which was before updating Atom to 1.19.2).

@as-cii My best guess at this point is that linter-ui-default was requesting measurements when the text editor had been unmounted (we have some internal debounce logic) so in the latest update I've made sure not to ask for measurements if the editor is unmounted
That seems to have fixed it for people so far, as for the Atom side, I think the text editor refs are removed when the editor is closed? A safeguard that throws an error with a proper message would be nice and helpful in debugging further cases like this

From what I know about the TextEditorComponent and @as-cii's reminder about the change that landed in 1.19.2, I think what was happening is that the mousemove handler was calling that method on the editor before it had detected that it had become visible. Previously, we waited to render the lineTiles div until after performing an initial measurement of line height etc that requires the editor to be visible. Now we render the lineTiles div on construction, so the exception shouldn't occur. Thanks for the report.