Automatically Scroll to Content when opened on mobile

Hi, is there a way to have it automatically scroll to the top of content when an accordion is open on mobile. currently the user has to scroll back up tot see the opened accordion. this is similar to the issue explained here with gif - https://github.com/oblador/react-native-collapsible/issues/86
Or Another possible solution is to not have the previously opened accordion not collapse when new one is opened. (on mobile) so that there is no shift in where the accordion gets opened.

Hi @siliconchild , when an accordion has been given the allowMultiple prop, it wouldn't collapse previously opened items when a new item is open. An example can be found at: https://szhsin.github.io/react-accordion/docs/allow-multiple
If you wish to implement scrolling an item to the top of page when that item is open, you could use the onStateChange event, which provides a DOM node for the item thus we can invoke scrollIntoView. Here is an CodeSandbox example.

Thanks for taking in time to provide me with possible solutions. Will try it out. Thanks a lot man !!


Is it possible to use a Proxy-Server?

I wonder if there is an easy way to use a proxy server for connecting to the API. In Azure.AI.OpenAI, it is possible via HttpClientTransport in Azure.Core.Pipeline, and other libraries (such as Semantic Kernel) allow specifying a custom HttpClient instance. But I can't find a way to use a similar method with this library.

Hello, @aurora!
System.ClientModel has a very similar pipeline concept and you can supply a custom HttpClient/transport via the client options instance:
OpenAIClientOptions options = new()
{
    Transport = new HttpClientPipelineTransport(myHttpClient),
};
OpenAIClient client = new(options);

Hi @trrwilson,
Oh, I somehow overlooked this option. It works perfectly. Many thanks for your quick reply and help!


Are sequence and machineID in wrong place?

In README.md it says "8 bits for a sequence number, 16 bits for a machine id", but in "sonyflake.go" it becomes "16 bits for a sequence number, 8 bits for a machine id". Are sequence and machineID in wrong place?
func (sf *Sonyflake) toID() (uint64, error) {
	//......
	return uint64(sf.elapsedTime)<<(BitLenSequence+BitLenMachineID) |
		uint64(sf.sequence)<<BitLenMachineID |
		uint64(sf.machineID), nil
}

//......

func Decompose(id uint64) map[string]uint64 {
	//......
	msb := id >> 63
	time := id >> (BitLenSequence + BitLenMachineID)
	sequence := id & maskSequence >> BitLenMachineID
	machineID := id & maskMachineID
	//......
}

Sorry, my fault. My eyes blind...


Edit and create new tracks

https://github.com/darkecology/roostui/issues/22

Deleting tracks feature #4

confirm before deleting https://github.com/UMassCDS/roostui/pull/8


CREATE_MISSING_FLAGS should be handled in .get() instead of is_active()

I am using the Flags in other scenarios aside from just views (celery tasks for example) and I was surprised that the CREATE_MISSING_FLAGS setting wasn't respected (I am manually calling is_active_for_user and other, custom, methods I created based on the example) and the methods were failing with error:
ValueError: "<CustomFlag: flag_name>" needs to have a value for field "id" before this many-to-many relationship can be used.

In my mind there should be a way to check if a flag is active using similar mechanism to waffle.flag_is_active which would work and respect the CREATE_MISSING_FLAGS setting, simplest solution seems to move the logic from is_active to get(). I would be happy to submit a PR if this would be considered a useful addition.

I have same issue. The solution suggested by @HonzaKral seems reasonable. Is there any interest in such a PR?

Yes, PRs are welcome.

This feature was reverted. See https://github.com/django-waffle/django-waffle/issues/435 for some details. Please modify the logic to ensure it supports flags, switches, and samples.


Improve configuration ergonomics for the v3 library

I'm attempting to use Helm v3 as a library to build a Go deployment client for a Kubernetes project I'm hacking on. One stumbling block I've run into thus far is configuration. The Helm CLI currently uses a heavily environment-variable-centric approach to client configuration (HELM_NAMESPACE, HELM_PLUGINS, etc.).
This is totally fine from an end user perspective, as far as I can tell, but it's not ideal for using Helm as a library. I think a more ergonomic approach for library users would be to:

Create a Settings struct with all the required options that devs can populate however they like (env vars, YAML, hard-coded values, etc.)
Retain the current env var system as the default for the current Helm CLI.

I think this approach would entail zero perceptible changes to end users while helping to fulfill one of the stated goals of the v3 effort.
To be clear, I'm happy to take on this work myself if such a decoupling were blessed by project maintainers.
cc @lachie83

We'd appreciate efforts to improve the situation. If you can write test cases and/or documentation to demonstrate how users should use these changes would be much appreciated.
For example: https://helm.sh/docs/topics/advanced/#simple-example

@lucperkins can you share a little of what you're trying to do? What you linked to is in the cli package which is tooling to support CLIs rather than using the Helm package in another form.
There are two flows for environment variables. Environment variables coming in and environment variables going out. Any solutions needs to preserve them going out so that things like plugins can have them passed. The big issue is the configuration coming in.
For the configuration coming in there are a couple ways this could be handled (thinking off the top of my head). First, there could be a new constructor to go alongside New. Second, the EnvSettings could be turned into an interface. The other places in the code that consume it could consume the interface and the cli package could be the one place that pulls it in using environment variables. Another one that sets things up differently could be created.
This is just me spit balling. Hope it helps.

Hi, @lucperkins - I understand you may be working on other things at this point. If we can help you move forward with your ideas, I would suggest filing a proposal: https://github.com/helm/community/blob/master/hips/hip-0001.md - until such a time, I'll resolve this issue. Thanks!


feat: port MeasureTheory.Measure.AeDisjoint

This PR/issue depends on:

leanprover-community/mathlib4#3325
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#3325
leanprover-community/mathlib4#3674
By Dependent Issues (🤖). Happy coding!

Thanks!
bors r+


Remove Cargo "homepage" manifest attribute

The C-METADATA guideline suggests that the homepage field should only be defined when the crate has a dedicated website other than the repository or documentation sites. As such, it is best to leave it out for the time being.

Hi @Enet4,
Thanks for the contribution!


Going from grapher to menu - Star/trash above menu

When I am on the grapher page and switch to the menu, the star and trash icons stay on top of the menu. This partially hides the "Manager" option.
Note: Refers to branch operating on matrix-reloaded

@maffoo @jwenner
I think I've fixed this here https://github.com/labrad/scalabrad-web/commit/69154baa5d2150a4eec3e69db7e809199a94cb52 but it's branch off of the maffoo/dev PR. I have insufficient gitfu to know where to go from here. I can continue to review the dev PR and then submit a PR once that's merged?

I think you can do a PR onto the u/maffoo/dev branch (e.g., martinisgroup/pyle#869). Right, @maffoo?

Fixed with #161 and deployed


Adding language tags to talks. The languages were identified using Yo…

…uTube's auto generated subtitles, and verified by a human.

Thank you!


#36 [feature request]  : Improve Responsiveness of Restaurant Cards and Search Box, and Buttons

Header
I have added feature of responsiveness in search , button and restaraunt cards
body
I am particapting in hacktoberfest 2023 .
I have try my best to add this feature .
If some feature is not implement correctly , i will review my contribution again

I have done changes as you have mentioned

revert the position of search button (only in larger screen) .
revert the size of rating in restaraunt cards

@Anandsg  sir i have reverted search button and search box as you have mentioned according to your given picture


Add a Did you mean? section to our error messages when possible

Add a « Did you mean? » part in our error messages when a field or a value is miss-typed.
That was proposed in this discussion initially.

@irevoire can you list which routes and in which situation the "did you mean" can happen?

Errm, to be honest, it's a bit hard.
It's automated, so it can happen on all the route that takes a body or query parameters.
If we really want the list, I could probably generate it somehow, but I don't think it's relevant to anyone since it impacts 90% of the engine

It's automated, so it can happen on all the route that takes a body or query parameters.

Ok, this is the information I was missing. It's obvious for you, not for me, when I read the issue 😉


False negative for https://baksman.org/

Hello,
We received a notification that our site has become suspected of your blocklist. We received a notification from VirusTotal ( https://www.virustotal.com/gui/url/f54f05c9e3618134b223c29d0a8c5032d61bedf5fed952e402426f4280ca729a/detection
). Our service https://baksman.org/ has extensive experience and many positive reviews, and we care about our reputation. Tell me how it is possible to correct the situation?

/move to Ultimate-Hosts-Blacklist/whitelist


Spurious TravisCI test failures

Cannot reproduce locally and does not seem to happen on TeamCity. See this PR for extended and still unhelpful output: https://github.com/opencypher/cypher-for-apache-spark/pull/418

Attempt to use CircleCI instead

CircleCI has a 1500 minutes limit per month on the free plan, too low.


Gizmo: make methods lower case

This change is

Made requested changes

@dennwc I think it's ready for another review.

Sorry for not being specific enough, but can you please make the same change and allow other methods as well, so we don't break any old clients, but will still introduce a lower-case names as an alternative?

All relevant methods are now accessible capitalized, and not.

Resolved conflicts


Can I scan the QRCode from a picture instead of from camera?

Same with title
What's happening?

I didn't find any APIs of that, did I miss something or  this module doesn't support ?
How can it be reproduced?

Build details?

That's not something this module currently supports, will accept PRs implementing that functionality, thanks!


Polymorphic relationships

What are you thinking about support of polymorphic relationships?

Yes, we need that. Thanks for kicking this off - if you want to keep working with it, this PR will need tests around the new behavior.

Moved to https://github.com/samselikoff/ember-cli-mirage/pull/451


Remove leftover debugging artifact in test

Sorry I forgot it there. :flushed:
Signed-off-by: Ahmet Alp Balkan<EMAIL_ADDRESS>

LGTM

LGTM


Adds MBS forms 0203, 0204 and 0253

@nealedj Think you have other commits in here from feature branch, but rest of it LGTM, so can approve when this is sorted

footer is broken on 204


refactor!: Change configuration and device toml files to yaml

BREAKING CHANGE: Configuration and device files now use yaml
closes: #552

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/device-mqtt-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) n/a
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?) n/a

Testing Instructions

run make test
start edgex with no-secty mqtt-broker
run ./device-mqtt -cp
verify that configuration and device profiles has loaded
run ./device-mqtt -cp -o
verify that configuration and device profiles has loaded

New Dependency Instructions (If applicable)

Codecov Report

Merging #553 (3944987) into main (c80097c) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #553   +/-   ##
=======================================
  Coverage   21.50%   21.50%
=======================================
  Files           5        5
  Lines         493      493
=======================================
  Hits          106      106
  Misses        373      373
  Partials       14       14

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


fix(confirm): Ensure validation occurs when moving within fields

Changes

Added in a fix to ensure that when the confirm field has a Validate method provided, it should validate the value, and does not move between different fields
Previously when a user submits the confirm field, it would just move to the next field in the huh.Group, reason being that within the Update() method of field_confirm, the NextField() tea.Cmd would be used, but there was no checks/Validation taking place to see if it's allowed to move to the next field
Hence now, if the user submits the confirm field, it ensures the Validate method is called, to see if the value is allowed, and then allow to move to the next field

Testing Notes
package main

import (
	"errors"
	"fmt"

	"github.com/charmbracelet/huh"
)

func main() {
	var confirm bool
	var confirm2 bool

	huh.NewForm(
		huh.NewGroup(
			huh.NewConfirm().
				Title("Install ABC?").
				Value(&confirm).
				Validate(func(install bool) error {
					if !install {
						return errors.New("you have to install this")
					}
					return nil
				}),
			huh.NewConfirm().
				Title("Install XYZ?").
				Value(&confirm2).
				Validate(func(install bool) error {
					if !install {
						return errors.New("you have to install this")
					}
					return nil
				}),
		),
	).Run()

	fmt.Printf("confirm: %v confirm2: %v\n", confirm, confirm2)
}

Previously:
Rendering two confirm fields, which have validate

Pressing Enter ends up moving the selection from the first field, to the second one (even though the validate function is called, but doesn't block from moving to the new field)

Pressing Enter again ends up moving the selection the next field, but doesn't block from moving to the next field

Now:
Rendering two confirm fields which have validate

Pressing Enter does not move the selection to the next field, and blocks it (due to the validate function)

Pressing Left

Pressing Enter ends up moving the selection to the next field, since the validation passed

Pressing Enter again does not move the selection to the next field and blocks it (due to the validate function)

Works as exepcted

Other Notes
Related to: https://github.com/charmbracelet/huh/issues/417

[ ] Anything else missing, What permutations could I test out?

This looks great and this behaviour makes sense to me. That said, I'm not the most familiar with this code base, so I'll get another team member to sign off on this too before merging :)

Yepp, sounds goood!

Hey @prithvijj thanks so much for your patience with this. After discussing with the team, this is actually the desired behaviour. It turns out we've modelled Gum forms to be quite similar in design to HTML forms which allow you to continue past an invalid field, but you must fix it before submitting the form.
Thank you so much for your contribution here, but we will keep the current form behaviour as-is

Hey @prithvijj thanks so much for your patience with this. After discussing with the team, this is actually the desired behaviour. It turns out we've modelled Gum forms to be quite similar in design to HTML forms which allow you to continue past an invalid field, but you must fix it before submitting the form.
Thank you so much for your contribution here, but we will keep the current form behaviour as-is

No worries @bashbunni ! Thanks for taking a look at it :tada:
Was about to link your comment in the original issue, but can't seem to find that issue which I swear I remember linking on my PR description, oh well :shrug:


Type-o in Picking-Images wiki

Small thing but I thought I should mention it.
On https://github.com/coomar2841/android-multipicker-library/wiki/2.-Picking-Images
there is a type in the OnActivityResult for the cameraPicker
if(requestCode == Picker.PICK_IMAGE_DEVICE) {
should be
if(requestCode == Picker.PICK_IMAGE_CAMERA) {

Thanks. Fixed. :)


Memory leak check

Normal release procedure.

Metaspace & heap seem OK after hundreds of tens thousands of launches


fix wrong links, p5 docs repo --> py docs repo

/processing/processing-docs/ --> /kazimuth/processing-py-site/
closes #125

Queuing this up now so I don't forget -- checks won't pass right now because the travis fix hasn't been merged on master, but it is all HTML edits, so no code to fail integration.


[BEAM-10459][BEAM-9514] Fix AssertionError type mismatch from AggregateScanConverter

Also remove aggregation functions from SqlOperatorMappingTable.java.
Currently ZetaSQL support aggregation functions that are either

implemented as UDAF operator in the ZetaSQL module (currently on string_agg), or
implemented in org.apache.beam.sdk.extensions.sql.impl.transform.BeamBuiltinAggregations

To keep a single source of truth of all the supported aggregation functions, I removed the aggregation functions list from SqlOperatorMappingTable.java (a duplicate copy of the above two cases).
This way we can also avoid using the predefined aggregation operators from Calcite SqlStdOperatorTable, which has its own way of inferring whether an aggregation result is nullable. Their inferring methods are incorrect in my opinion. For example, this is the code pointer to how they infer the the result of SUM.
r: @amaliujia @apilloud
Post-Commit Tests Status (on master branch)

Lang
SDK
Dataflow
Flink
Samza
Spark
Twister2

Go

---

---

---

Java

Python

---

---

XLang

---

---

---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website
Whitespace
Typescript

Non-portable

Portable
---

---
---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Tested internally. Got some failures passing and no regression found.


Uncaugth TypeError: Cannot set property 'tmplName' of undefined

Chrome version 57.0.2987.37 beta (64-bit)
jsRender v0.9.83 (Beta)
This code produces a very rare error. ..[850 Line]
In my opinion, the value of the object is likely to not be declared..
If the object is not defined, it is likely to be resolved by handling the exception.
value.tmplName = name = name || currentName;

Can you give more information on what you are doing? Are you using JsRender with jQuery or without. How are you declaring your template? Can you show your code where you call $.templates(...)?

ok.
We're using jquery together.

The following functions are reproduced and reused as follows :

Call up the .inject function here.

Do you need more information?

What kind of element is being used (with id "major-tmpl") to declare the template. It is not a good idea to use elements other than script blocks as containers for registering templates. Otherwise best to use a string. https://www.jsviews.com/#compiletmpl.
If you do use a container element, then JsRender caches the compiled template. If you then re-access the template by a second call to $.templates("#major-tmpl"), the cached version will be used. But in your case it looks as if the cached copy or the container element have been corrupted. For example the following code will simulate the error condition you are hitting:
var tmpl = $.templates("#major-tmpl"); // Compile template

var html = tmpl.render(data);

$("#major-tmpl").data("jsvTmpl", null); // Remove the cached compile template, to provoke error scenario

var tmpl2 = $.templates("#major-tmpl"); // Try to reaccess and reuse compiled template (Error)

var html2 = tmpl2.render(data);

Is there anything you are doing which might corrupt or modify the element, or  the cached copy of the template?
The following version of JsRender includes a defensive fix for the above scenario. You can test it to see if it successfully resolves your issue.
jsrender.zip

@BorisMoore
Changing to correct version will not result in error.
Thanks for the patch version.  👍

Did you figure out why or how that element, or its JsRender data is getting modified/corrupted? What element type are you using as container for the template?

@BorisMoore
Even though we should not use the cache intentionally, the previous developers ignored it and applied it to the template.
I am currently improving the system we use.
Our final goal will be to remove all libraries that prevent real-time elements from our systems.
Nevertheless, I am grateful to you for correcting the current issue.

OK - thanks for the update. Closing this issue.


feat: markdown support and fix: nil pointer dereference

Add feature: Front-end adds support for markdown rendering, and back-end modifies prompt to better generate replies in markdown format.
Fix Bug: Fixed a null pointer error in getting tweets (service/index.go getTweetDetails()).

Sorry, a line of import was missed in pages/index.tsx .
import ReactMarkdown from 'react-markdown';

#26

Sorry, a line of import was missed in pages/index.tsx .
import ReactMarkdown from 'react-markdown';

#26

In fact, you can continue to do what you want in this PR, and subsequent commits will be synchronized


Change some deprecated OpenAPI field names

See https://github.com/cloudendpoints/endpoints-python/issues/36

Codecov Report

Merging #44 into master will not impact coverage.

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   80.23%   80.23%
=======================================
  Files          38       38
  Lines        5257     5257
=======================================
  Hits         4218     4218
  Misses       1039     1039

Impacted Files
Coverage Δ

endpoints/test/openapi_generator_test.py
88.41% <ø> (ø)
:white_check_mark:

endpoints/openapi_generator.py
84.38% <ø> (ø)
:white_check_mark:

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 1958dbf...587486f. Read the comment docs.


Bootstrap and CSS styling

put this off until later, but we need styles for the site

at least 15 custom css rulesets
bootstrap to make the site look nicer (animations, styled buttons, etc.)

I have experience with bootstrap on my own site, a lot of it is adding classes to html elements so it might be nice to use to make cards for posts, clickable elements, etc (see here: https://jhg.app)

lots of cool stuff for this implemented in #48 !!!!

much of bootstrap has been done, we will need styles tho (we can apply this for color scheme of text, errors, images, etc)

I will be making some executive decisions on styles


show right logo for dark and light themes

Description
Show the MoveIt right logo in API documentation based on dark/light theme. See https://github.com/ros-planning/moveit2/pull/1015 for related PR that also needs to merged for this to work.
Checklist

[ ] Required by CI: Code is auto formatted using clang-format
[ ] While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

If I’ve seen correctly, in your changes, it shows moveit logo black for the dark theme, and moveit logo white for the light theme. Shouldn’t it be the other way around? @mamoll
I’ll check back when I get back to pc

If I’ve seen correctly, in your changes, it shows moveit logo black for the dark theme, and moveit logo white for the light theme. Shouldn’t it be the other way around? @mamoll

I added some comments. I am pretty sure this is right.

closing, seems to no longer be applicable


Error is-svg/index.js:8 in laravel 5.1

Hi,
I've installed Laravel 5.1 and gulp setting for your pictures dependency gives me an error.

/Users/mario/Documents/proyectos/hoyvoyweb/

I delete node_modules folder and execute npm install and all is working now.
Thanks.


CANVAS game object transforms are wrong (v3.50.0-beta.10)

Version

Phaser Version: v3.50.0-beta.10

Description
With the CANVAS renderer image/sprite flip, position, rotation, and scale are incorrect.
Example Test Code
http://labs.phaser.io/view.html?src=src\transform\angle.js&v=3.50.0-beta.10 and others

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


fix: Screenshots on landing page need to be cropped (#6333)

Fixes Issue
Screenshots on landing page need to be cropped #6333
Changes proposed
Made changes in tailwind CSS of image
Before : className="object-cover object-center"
After : className="object-cover object-right"

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Before :

After:

Thank you for your contribution 👍
The screenshot should be cropped, it looks like you have changed the CSS not updated the image?
Please note for the before/after screenshots, it should not be of the code, we can see those in the file changes tab, it should be of the result in the UI

Thank you for your contribution 👍
The screenshot should be cropped, it looks like you have changed the CSS not updated the image?
Please note for the before/after screenshots, it should not be of the code, we can see those in the file changes tab, it should be of the result in the UI

Yes I've made changes in the CSS.
Screenshots of UI before changes :

Screenshots of UI after changes :

As discussed in the issue, it would be better to crop the image on the right and also remove the navbar. Please update your PR with those changes not code changes or create a new PR if easier (but if you create a new PR please close this one)

As discussed in the issue, it would be better to crop the image on the right and also remove the navbar. Please update your PR with those changes not code changes or create a new PR if easier (but if you create a new PR please close this one)

Like, you want me to download it locally and then crop it and do it by CSS right ?

yes, download the image, crop the right and the top navbar
no css changes needed

there are no assets or image folders already, so no we do not want to create them, if you look at how the images are done at the moment, they are uploaded to the issue and the generated url is used from there

Okay so I will crop them and upload them to the issue.
Here is one of the Image I've cropped, Can you give me some feedback over it like it is okay or I need to do more cropping.

I've cropped the navbar and that extra right margin.


Support restriction in elasticstack_elasticsearch_security_api_key

In its current form the provider is unable to provision API keys that have a restriction specified in the role descriptors.
For example, given this terraform:
resource "elasticstack_elasticsearch_security_api_key" "api_key_with_restriction" {
  name = "restricted-key"

  role_descriptors = jsonencode({
    role-a = {
      indices = [
        {
          names      = ["index-a*"],
          privileges = ["read"]
        }
      ],
      restriction = {
        workflows = [ "search_application_query" ]
      }
    }
  })
}

The provider will happily create an API key named restricted-key however the restriction field is not persisted in Elasticsearch.
On the next terraform plan, this API key will be marked as needing replacement due to the fact that the comparison between the local terraform definition and the response from the Elasicsearch API are different. That is because the local terraform definition serializes to JSON and will contain restriction while the JSON for the existing API key does not have that field.
This PR adds the restriction to the model structs necessary to provision the API keys.
In order to achieve that, the Role struct was duplicated to an API key specific struct ApiKeyRoleDescriptor. Previously Role was shared between the elasticstack_elasticsearch_security_api_key resource and the elasticstack_elasticsearch_security_role data source.
In Elasticsearch, the restriction is only available from 8.9.x and up. To provide some feedback to users, an error will be generated during plan/apply time when the target Elasticsearch is below 8.9 and the user has specified a restriction in their local terraform definition:

Error: Specifying restriction on an API key role description is not supported in this version of Elasticsearch. Role descriptor(s) role-a

This is currently making a call directly to the Elasticsearch instance which is probably not ideal, however I could not find a reference to the current version in the code. If that's available that would be a much better option.

Hey @sandermvanvliet-stack are you able to sign the CLA so we can continue with this one?

@tobio I'm still waiting on some internal comms before I do that. Trying to figure out if we have/need/want a corporate CLA for this.

@tobio the CLA should now be signed as a company wide one by our teams, would you be able to check so we can get on the merge train?

the CLA should now be signed as a company wide one by our teams

I can't see any signed agreements from your company or email domain. I'm not familiar with the company wide agreement process, I'll ask around and see if there's something we need to do.

@sandermvanvliet-stack we can't find any recent CLA activity on anyone related to Stack Overflow or Stack Exchange. Are you able to get some more details about how the CLA was signed so we can try and track this down?

@tobio I was told it was handled but let me prod some people here. Thanks for checking 👍

@tobio the CLA was signed 26th of June. The contact listed is David Whitt.
I've got a copy here of the signed CLA, let me know if you need one

the CLA was signed 26th of June. The contact listed is David Whitt.

Ok, weird. I can't find anything under that name where we'd expect it to be (our internal tooling or in the Docusign account used for the CLA agreements). Are you able to forward me a copy to<EMAIL_ADDRESS>or I can send you a secure file sharing link. Once I've got a copy I can try and find out what's happened here.
At the very least, IIUC we should be able to manually add your accounts with a copy of the agreement.

@sandermvanvliet-stack sorry about the delay here. I'm not clear why it took a while for this agreement to show up. It looks like everything's good to go here.
Thanks for taking the time to add this into the provider, and dealing with the box ticking :)


AppImage system integration

I moved Onivim2-x86_64.AppImage to my /usr/local/bin folder on 5.5.13-arch2-1 machine.
I'm using zsh.
After reboot I can run the app from any directory but typing for the command the name of the AppImage.
By doing Onivim2-x86_64.AppImage -h I get this output.
How can I configure it , to use 'oni2' as command tool integrated on the system?
Thanks in advance!

I found my answer in user manual.. by creating and alias.

https://github.com/onivim/oni2/pull/1202 should fix this once merged. (Well give a command in editor to do this for you).
I'll give it a quick test over this weekend on my Ubuntu and Arch machines. This would automate the linking of oni2 into /usr/local/bin.


#71 remove ducks

I'm going to hold off on doing this for now and continue discussing in the issue and trying to think of another solution!


Remove reference to using registry keys to set devenv.exe process DPI awareness

It turns out that the section re: the registry keys to set are incompatible with the out of proc WinForms designer (originally targeted to the .NET Core) and even some scenarios for the in-proc (.NET Framework)  designer. We should remove this recommendation, and potentially recommend that the user NOT use registry keys to set the devenv.exe process DPI Awareness any way other than the command line or clicking the gold bar should it display (it only displays with the in proc designer). See https://github.com/dotnet/winforms/issues/7997 for details

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

ID: a2f87f79-412c-6d45-9a5f-e1690a7dce12
Version Independent ID: 708a3cc0-a8eb-3b23-9d11-737d0060d730
Content: Disable DPI-awareness for scaling in forms - Visual Studio (Windows)
Content Source: docs/designers/disable-dpi-awareness.md
Product: visual-studio-windows
Technology: vs-ide-designers
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@merriemcgaw Thanks so much for the heads-up on this. Before we update the page, can you please verify that this also applies to Visual Studio 2019? The page includes monikered information for both Visual Studio 2019 and Visual Studio 2022, so we want to make sure that it applies to both of those versions or to one or the other before we publish an updated page.

@TerryGLee Yes it would apply anywhere VS supports PerMonitor DPI awareness.

@merriemcgaw Thanks again for the suggested edit. We've updated the page; please let us know if we should make any further changes.


Eliminate duplication in creating packages

Our current version requires a NuSpec to create a package for nuget.org or chocolatey.org. This results in two types of duplcation:

Much of the metadata for TestCentric packages is common across all packages.
Where packages are created for both nuget.org and chocolatey.org, there is common info, which must be duplciated.

Additionally, since nuspecs are provided by the project using the recipe, it is difficult to keep things in sync.
As a first cut at design, I'll try to replace the use of nuspecs with code which is extensible by the using project.

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

GitHub Release
NuGet Package


Updated README.md to add closing ) to example

   Prelude Pipes Parse Lens> evalStateT (zoom group drawAll) (each [1, 1, 2, 3]

was missing its closing ")"

Thanks for the fix! :)


Respect options from baseline

Fully implements the proposal in https://github.com/Yelp/detect-secrets/issues/121#issuecomment-459575817. More than half of the changes are new test cases.
Some logic can be implemented in different places, I chose the current way based on my understanding. Let me know if you have any suggetsions or comments.
FYI @KevinHock @domanchi
CC @jribm

@domanchi ready for another round.

Thanks for writing that hairy logic of plugin prioritization.

It definitely makes the job more challenge when I have to convert the BasePlugin tuple into a dictionary, then compares and sets values. I noticed the BasePlugin class has made __dict__ field itself a property instead of the exact properties such as name, base64_limit and etc. This prevents the {set|has|get}attr from reading and writing values. May I ask what was the rationale of overwriting __dict__ instead of directly using property on each field? @domanchi

@domanchi all comments addressed.


excess padding on problem items when docked vertically

There is 32px of left-padding on each problem which takes up valuable space when the problems panel is docked vertically.
Would it be possible to remove this padding so more actual content is visible when the problems panel is narrower?

There are two space separations

Alignment of children (which is 32px padding) in the tree
Space for light bulb for quick fixes

I do not think removing the children alignment in the tree is correct. We use the same in File explorer and Search trees.
We can think of moving the light bulb to the right. Reason why light bulb is on the left in horizontal panel because of better visibility. But I am worried this might cause inconsistencies
@misolori Any opinions?

@sandy081 do you think we'd be able to shift the light bulb to the left of the tree icon? That would be a better use of space:

I'd worry that moving it to the right would make it hard to find and be undiscoverable.

@misolori Moving to the left can be tried.

@misolori Shifting to the left of the twisty might not be a good idea it can break the children alignment. Other option is to have a tree gutter area which needs to go into tree. I am afraid we can implement tree gutter for this requirement. Other option is to go back to my initial idea of overlaying light bulb on the error icon

I still think the combo icon is hard to recognize, especially at a small size.
Would it be worth exploring to swap out the error icon for the lightbulb?

Can be tried


Remove inline arrays as a conversion target

See dotnet/roslyn#68927
This didn't make it in C# 12.

Internal previews

📄 File
🔗 Preview link

docs/csharp/language-reference/operators/collection-expressions.md
Collection expressions - C# language reference

ping @adegeo for review.

ping @adegeo


[BUG]运行./scripts/run_finetune.sh报如下错误，一直卡这。还有请问你们torch是pip intsall -e安装还是手动到官网安装呢。求求你们帮我看看吧，我要疯了。

AttributeError: 'DeepSpeedCPUAdam' object has no attribute 'ds_opt_adam'

Thanks for your interest in LMFlow! A quick fix is to remove the following section in configs/ds_config_zero3.json:
    "optimizer": {
        "type": "AdamW",
        "params": {
            "lr": "auto",
            "betas": "auto",
            "eps": "auto",
            "weight_decay": "auto"
        }
    },

DeepSpeed uses a nvcc/c++ version of adam to speed up the optimization, which requires a compilation process and may sometimes fail due to wrong version of nvcc/c++. To find the exact problem that causes this issue, could you please provide the full log of error message, so we could check that for you?
As for installation of torch, by following https://github.com/OptimalScale/LMFlow#1setup, torch should be automatically installed. Thanks!  😄

已经解决了，谢谢回复。
我的显卡驱动使用的nvidia-driver-530，cuda使用的11.7.0系列，cudnn使用的11.x系列。
有个极端做法但是不推荐，我出现“DeepSpeedCPUAdam”对象没有属性“ds_opt_adam”-》这个问题都是在普通用户下操作的，后来我用root安装在根目录解决了这个问题，用root用户来操作一切解决了。

https://github.com/ymcui/Chinese-LLaMA-Alpaca/issues/368#issuecomment-1552310311
我用的这个方法也ok


V1.2 python SDKDocs update - client

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Read the contribution guide
[x] Commands include options for Linux, MacOS, and Windows within codetabs
[x] New file and folder names are globally unique
[x] Page references use shortcodes instead of markdown or URL links
[x] Images use HTML style and have alternative text
[x] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
Updates Python SDK client documentation:

fixes to match v1.2 Dapr release
fixes links to point to main/1.2 branches
adds Subscribe example

Note this is the matching PR to pair with python-SDK submodule PR:
https://github.com/dapr/python-sdk/pull/224
Issue reference
Fixes #1551

@wcs1only @berndverst advise and see if branching technique and separate PR is correct in this case.
https://github.com/dapr/python-sdk/pull/224

Let's hold for the pythonSDK change, and then I'll be about to use the upstream merged files

Let's wait on PR in Python SDKDocs to merge upstream, and I'll re-sync this change to submodule..

@AaronCrawfis could you please take a look?  This is simply a merge of Python SDK changes in sub module so we can release them to docs.dapr.io.
Note this is the matching PR to pair with python-SDK submodule PR:
dapr/python-sdk#224
Fixes #1551


Discontinued support of the resource

I’m going to be discontinuing support of this resource come 4/1/2018. No longer responding to PRs, issues, etc.
Happy to give others access and control, after vetting and discussion. I no longer have time to support this resource.
Please comment here if you've any interest. Thanks.
Or if someone wants to pay me. I'll reconsider.

I'm not sure how much time I have either, but I'd be willing to help

@jtarchie thanks for all the great work you put in this resource. One quick question, what was the reason you are discontinuing support for it? Is there a better alternative for dealing with PRs inside of Concourse in an upcoming release?

I would gladly take it over.

I'm working on a pure Golang alternative. I'll post a link when I'm done.

How do you want to do this @jtarchie?

@crstamps2, check email.

@crstamps2 and I have talked through the passing off the resource.

It will remain under my org (jtarchie) until it seems necessary to move it. This is true for both Github and Docker.
@crstamps2 and I have worked together before. They are enthusiastic, loves open source, and just an awesome person! Please treat them well.
I'm stepping back. I'm not going to even hover. Please don't cc me on issues.
So long and thanks for all the fish.

👋 👋


CI fails as no PowerShell 7.x endpoints are available.

Steps to reproduce
Open PowerShell 7.0.0-Preview.1
Get-PSSessionConfiguration | Select-Object -ExpandProperty Name

Expected behavior
Output should contain a Configuration name for 7.0.0-Preview.1
Actual behavior
Output doesn't contain a Configuration name for 7.0.0-Preview.1
Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0-preview.1
PSEdition                      Core
GitCommitId                    7.0.0-preview.1
OS                             Microsoft Windows 10.0.18922
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0ΓÇª}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Error in CI
New-PSSession : [localhost] Connecting to remote server localhost failed with the following error message : The WS-Management service cannot process the request. Cannot find the WinRM is already set up to receive requests on this computer. WinRM has been updated for remote management. Created a WinRM listener on HTTP://* to accept WS-Man requests to any IP on this machine.
 PowerShell.7.0.0-preview.2 session configuration in the WSMan: drive on the localhost computer. For more information, see the about_Remote_Troubleshooting Help topic.
At D:\a\1\s\test\tools\Modules\HelpersRemoting\HelpersRemoting.psm1:190 char:20
+         $session = New-PSSession @parameters

/cc @PaulHigin @SteveL-MSFT Should we create new endpoint for 7.0?

I just checked my Windows machine.  Enable-PSRemoting creates the PowerShell.7-Preview and version specific  PowerShell.7.0.0-preview.1 remoting endpoints and works when I connect to them.  Looking at your PR, the test failures don't seem related to remoting?

Yep, It creates the endpoint post enabling the remoting... got confused by seeing below error in CI result.
 Describing TestImplicitRemotingBatching hook should correctly batch simple remote command pipelines
WARNING: PowerShell remoting has been enabled only for PowerShell 6+ configurations and does not affect Windows PowerShell remoting configurations. Run this cmdlet in Windows PowerShell to affect all PowerShell remoting configurations.
VERBOSE: Using Global CI Credential
New-PSSession : [localhost] Connecting to remote server localhost failed with the following error message : The WS-Management service cannot process the request. Cannot find the WinRM is already set up to receive requests on this computer. WinRM has been updated for remote management. Created a WinRM listener on HTTP://* to accept WS-Man requests to any IP on this machine.
 PowerShell.7.0.0-preview.2 session configuration in the WSMan: drive on the localhost computer. For more information, see the about_Remote_Troubleshooting Help topic.
At D:\a\1\s\test\tools\Modules\HelpersRemoting\HelpersRemoting.psm1:190 char:20
+         $session = New-PSSession @parameters
+                    ~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : OpenError: (System.Management.A…tion.RemoteRunspace:RemoteRunspace) [New-PSSession], PSRemotingTransportException
+ FullyQualifiedErrorId : InvalidResourceUri,PSSessionOpenFailed
+++++++++++++++++


next botocore update brings rebrand to cloudwatch events: eventbridge

I'm working on an upgrade of the next botocore version to gain access to event bridge service announced today
It seems this may be a rebranding of cloudwatch events

EventBridge was formerly called Amazon CloudWatch Events. It includes new features that enable you to receive events from SaaS partners and your own applications.

https://docs.aws.amazon.com/eventbridge/latest/userguide/what-is-amazon-eventbridge.html
This creates an interesting case for rusoto as events carries the service name CloudWatchEvents. I'm not sure if this should be changed to EventBridge. Thoughts?

I'll have a pull up shortly but already I see this in the rusoto_events crate diff
+++ b/rusoto/services/events/README.md
@@ -1,6 +1,6 @@

 # Rusoto CloudWatchEvents
-Rust SDK for Amazon CloudWatch Events
+Rust SDK for Amazon EventBridge

This makes me think it would require something like a rusoto 0.41.0 release


[TypeCheck] Clean up constraints

I wanted to remove the unnecessary MonadQuote from the kind checker, but ended up refactoring the type checker constraints in general.

OK, should be ready for review.

Comments addressed, merging.


fix: 杨宝头像icon换源

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


Documentation is broken

The links in the documentation provided by the search field in the upper right are broken. They lead to <a_page>.php.html, whereas they should only lead to <a_page>.php.
However within the doc it seems to be working fine.

Furthermore, many classes, including Model, are not documented anymore. I suspect this is due to #321

The documentation seems to be completely fine again.


Add logging functionality

Currently using print statements.  Adding a logger and being able to control when various statements are displayed would likely be cleaner.
Not a high priority issue, suggest we tackle some of the others first, just putting it so we can improve code later.

Will be tackling some of this while I handle #13


duniverse opam-install command

A first and straightforward solution to https://github.com/avsm/duniverse/issues/19.
It keeps tracks of non-dune packages and allow to install them using duniverse opam-install.

Could you please rebase on top of #14 so that it also installs the packages for which we couldn't interpret the dev-repo? That way we can start reviewing this and be ready to merge as soon as #14 gets in!


cordova.plugins.SitewaertsDocumentViewer is not object

I have installed cordova-plugin-document-viewer and try to access but : cordova.plugins.SitewaertsDocumentViewer return undefined.
I checked : cordova plugin -ls ; it show cordova-plugin-document-viewer in the list.
my code:

Hello World

Open

`

Hello World

Open

`

respect the deviceready event from cordova.
Am 11.11.2017 um 13:01 schrieb astics <notifications<EMAIL_ADDRESS>`
Hello World Open     `

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/sitewaerts/cordova-plugin-document-viewer/issues/88#issuecomment-343659914, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADIAQsD5mL4TUecUiN3_cl9rF2PEC2Poks5s1YyjgaJpZM4Qad8V.

thanks....you save my time.


[CI] [do not merge] enable dependabot

dependabot is an independent security scanning tool which mostly focuses on evaluating the dependency chain. Having the dependabot.yml file on the main branch would enable the bot to test the dependencies and create a pull request for each one, on a specified schedule.
When we finish dependabot configuration tweaking in osbuild-composer repository, we can merge this PR.

CI got fixed in #245, needs a rebase to unbreak CI

I have a couple of thoughts after I tried to get as many Dependabot PRs as possible merged over the past weekend:

I think it's a good idea to mass-update dependencies manually before enabling Dependabot. It greatly reduces stress on our resources.
PRs from Dependabot very often created conflicts in go.sum. If this becomes an issue even after the mass-update, we may want to reduce the number of open PRs to 1 because it's basically a game of rebasing otherwise. I would definitely start with 5, watch them closely and react accordingly.

Okay, we can merge this once the manual dependency update PR is merged.

Okay, we can merge this once the manual dependency update PR is merged.

Are you working on such a PR?

Are you working on such a PR?

I wouldn't know were to start without enabling dependabot. I've been taking a look at this  combine-dpendabot-prs. But I don't know if it's worth the effort.

@jabia99 Thank you!
For the commit message format, while not required, https://www.osbuild.org/guides/developer-guide/workflow.html#commits I really like this format.
No need to force push again, I'll edit the commit message on squash merging. But something for future commits.

Great, thanks for the tip!!


Search Icon UI update for the Food Filter Project

The search icon in the Food Filter Project does not align in terms of height with the search text box.
Expected behavior
I expect the height of the search icon and search box to be equal to give a better user interface
Screenshots

Desktop (please complete the following information):

OS: iOS
Browser : Chrome
Version [e.g. 22]

Additional context
I will like to make this my first open-source contribution

[x] Describe the bug
[ ] To Reproduce
[x] Expected behavior
[x] Screenshots
[x] Additional context

Code of Conduct✅

[x] I agree to follow this project's Code of Conduct.
[x] I checked the current issues for duplicate problems.
[x] I have gone through project README.

Can this task be assigned to me or should I just go ahead and make changes?

Can this task be assigned to me or should I just go ahead and make changes?

I don't think it is a bug, since it just improves styling.
If you want to add changes, then do it properly by adding several styles rather than just adding a 10px padding.


Using chained comparison operators

Unlike most languages, Python supports chained comparison operators and
it evaluates them as they would be evaluated in normal mathematics.

Looks good to me.


Error with Protobuf installing during setup  (protoc failed: Unknown flag: --experimental_allow_proto3_optional\n)

Error with Protobuf installing during setup:
   Compiling nexus-network v0.4.0 (/root/.nexus/network-api/clients/cli)
error: failed to run custom build command for `nexus-network v0.4.0 (/root/.nexus/network-api/clients/cli)`

Caused by:
  process didn't exit successfully: `/root/.nexus/network-api/clients/cli/target/release/build/nexus-network-a507b54e20e4ba46/build-script-build` (exit status: 1)
  --- stderr
  Error: Custom { kind: Other, error: "protoc failed: Unknown flag: --experimental_allow_proto3_optional\n" }

Found Fix the issue:
Step 1: Remove Any Old Extracted Files
If there are any leftover extracted files, let's clean them up first.
rm -rf protoc-3.12.0-linux-x86_64
Step 2: Re-Extract the Zip File
Since the protoc-3.12.0-linux-x86_64.zip file is present, let's unzip it again.
unzip -o protoc-3.12.0-linux-x86_64.zip
The -o flag will automatically overwrite any existing files without prompting.
Step 3: Verify the Contents
Once you’ve extracted the files, check the contents of the extracted directory:
ls protoc-3.12.0-linux-x86_64
This should show the bin/ and include/ directories. If everything looks good, you should see something like this:
bin/
include/
readme.txt
Step 4: Move the protoc Binary
Now, move the protoc binary to /usr/local/bin/:
sudo mv protoc-3.12.0-linux-x86_64/bin/protoc /usr/local/bin/
Step 5: Move the Include Files
Move the include/ directory files to /usr/local/include/:
sudo mv protoc-3.12.0-linux-x86_64/include/* /usr/local/include/
Step 6: Verify the Installation
Finally, check that protoc is installed correctly:
protoc --version
You should see: libprotoc 3.12.0 and start installation again.

if didn't work as above method, then try #111

which apps I need to use this can you tell me please

which apps I need to use this can you tell me please

just read #111 carefully, you will know how to solve this


Project View

Old branch was not updating correctly (Components were missing). Drew told me to just make a new branch so all his fault ISSAC!

Whoops, did not mean to do mark as ready for review. Just wanted to look over the code


Prepare a platform for technical information

Prepare a platform for sharing technical information.
Request, articles can be managed on Github.
I think 11ty is best way. And we want it be like web.dev.
web.dev's repo. I will check later
https://github.com/GoogleChrome/web.dev

Here's some crypto examples of documentation. It seems as if all examples use the same template for their documentation. Here's Dev Protocol's docs that discusses the core protocol. I would prefer if Dev Protocol had one documentation that discusses A-Z (Protocol to Integrations). This makes sense because a learning phase it from A-Z and doesn't start and end at Z (Just Integrations).
Examples
Chainlink
Synthetix
Aave
Websites Used
Dev Protocol docs currently uses Gitbook
Chainlink and Aave also use Gitbook
Synthetix uses MkDocs & Material for MkDocs
Documentation Format

Welcome
Protocol
Tokens
Contract List
Integrations
Use Cases
Libraries
Smart Contract Architecture

Here's some crypto examples of documentation. It seems as if all examples use the same template for their documentation. Here's Dev Protocol's docs that discusses the core protocol. I would prefer if Dev Protocol had one documentation that discusses A-Z (Protocol to Integrations). This makes sense because a learning phase it from A-Z and doesn't start and end at Z (Just Integrations).
Examples
Chainlink
Synthetix
Aave
Websites Used
Dev Protocol docs currently uses Gitbook
Chainlink and Aave also use Gitbook
Synthetix uses MkDocs & Material for MkDocs
Documentation Format

Welcome
Protocol
Tokens
Contract List
Integrations
Use Cases
Libraries
Smart Contract Architecture

We need both documentation for covers all specifications and documentation for provides user-friendly cross-cutting information about specific topics. That is tuned according to the target audience's knowledge level and clearly shows the technical topics that should be cleared in advance. I think it's essential not to understand the specifications but to give the best answer for the purpose.
Gitbook is the perfect tool for "documentation for covers all specifications." I understand that @kazu80 is looking for the latter tool.

We need both documentation for covers all specifications and documentation for provides user-friendly cross-cutting information about specific topics. That is tuned according to the target audience's knowledge level and clearly shows the technical topics that should be cleared in advance. I think it's essential not to understand the specifications but to give the best answer for the purpose.
Gitbook is the perfect tool for "documentation for covers all specifications." I understand that @kazu80 is looking for the latter tool.

I think 11ty is best way. And we want it be like web.dev.

@kazu80 I agree with 11ty!

I think 11ty is best way. And we want it be like web.dev.

@kazu80 I agree with 11ty!

Thank you for sharing @defi-er .
As from @aggre , I want this to be a topic rather than a manual. This is Like a blog. I imagine that the engineers will be interested in DEV after reading this article, and then read the manual and use DEV.

Thank you for sharing @defi-er .
As from @aggre , I want this to be a topic rather than a manual. This is Like a blog. I imagine that the engineers will be interested in DEV after reading this article, and then read the manual and use DEV.


output different classes of FASTA and FASTQ

output FASTA file for each of

high-confidence contigs
potential chimeras
contigs too short
uncovered contigs or contigs with uncovered bases

and the FASTQ of reads mapping to each

This is present in v1.0.0.beta2


Allow indentation width to be set explicitly

This allows the user to circumvent the default indentation behavior. In particular, it can be used to avoid getting the default four-space indentation when expanding source that doesn’t contain any indentation to infer the indentation width from, e.g.:
@AddMember
struct S {
}

Unindented original source seems uncommon, but it occurs pretty frequently in my macro tests. For example, I have a collection of tests that exercise each possible combination of access modifier for structs, classes, and actors.
Changes:

The assertMacro and withMacroTesting functions now accept indentationWidth as a parameter (Trivia).
Introduces IndentationWidthTests to validate both the existing indentation behavior and the new explicit indentation width functionality.

Some alternatives to consider:

Instead offer defaultIndentationWidth to let the user only control the fallback indentation value. This would work in my use case.
Only support explicit indentation width in assertMacro and not in withMacroTesting.

Hi @stephencelis, any updates on this? A way to clean up the awkward indentation in a dozen or so of my tests would be a nice-to-have! 😄


Add kNN and cross validation wrapper oracle for query synthesis.

Pull Request Test Coverage Report for Build 168

40 of 42 (95.24%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+2.3%) to 87.355%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/oracle.jl
40
42
95.24%

Files with Coverage Reduction
New Missed Lines
%

src/QueryStrategies/qs_base.jl
1
85.71%

Totals

Change from base Build 166:
2.3%

Covered Lines:
677

Relevant Lines:
775

💛  - Coveralls


optional parameter leads to black bar issue

Hi. Wasted a couple hours on this and just hope that no one else will.
If you're on Swift 3:
controller.followScrollView(view, delay: 0, scrollSpeedFactor: 2)
results in a black bar on segue, even if you show navigation bar.

Hey @divij1997
This should now be fixed in master, let me know if that works for you.


Support switch label clauses. Fixes #693

Fixes #693

This change is

You have accidentally committed the vendor folder.

Vendor folder is removed.

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

transpiler/switch.go, line 13 at r5 (raw file):

	"github.com/elliotchance/c2go/program"
	"github.com/elliotchance/c2go/util"
	"golang.org/x/tools/go/ast/astutil"

Is this bundled with go or do you need it add to dep requirements?

Comments from Reviewable

transpiler/switch.go, line 13 at r5 (raw file):
Previously, elliotchance (Elliot Chance) wrote…
Is this bundled with go or do you need it add to dep requirements?

astutil is not bundled with go.
Installing/updating c2go with go get -u github.com/elliotchance/c2go will take care to download the latest version of astutil into the $GOPATH.
By needing to add it to dep requirements, did you mean add it to the documentation as a requirement, or did you mean to use the dep tool, which is a dependency management tool (https://golang.github.io/dep/docs/introduction.html)?
Since astutil is authored by Google, the chance of a breaking change should be minimal, so I don't see the need at the moment for a dependency management tool. (Maybe later, if there are other dependencies)
PS: see https://github.com/golang/dep/blob/master/docs/FAQ.md#should-i-commit-my-vendor-directory on the pros/cons of committing the vendor folder.

Comments from Reviewable

Review status: all files reviewed, 1 unresolved discussion (waiting on @elliotchance)

transpiler/switch.go, line 13 at r5 (raw file):
Previously, kamphaus wrote…
astutil is not bundled with go.
Installing/updating c2go with go get -u github.com/elliotchance/c2go will take care to download the latest version of astutil into the $GOPATH.
By needing to add it to dep requirements, did you mean add it to the documentation as a requirement, or did you mean to use the dep tool, which is a dependency management tool (https://golang.github.io/dep/docs/introduction.html)?
Since astutil is authored by Google, the chance of a breaking change should be minimal, so I don't see the need at the moment for a dependency management tool. (Maybe later, if there are other dependencies)
PS: see https://github.com/golang/dep/blob/master/docs/FAQ.md#should-i-commit-my-vendor-directory on the pros/cons of committing the vendor folder.

That's fair enough. c2go is still alpha enough that we can get away without locking versions.
As for committing the vendor folder, everything is a trade off. Unless your building extremely defensive software committing the vendor folder just bloats your codebase and opens the potential for people to tweak the vendor code.
I have only used dep for some projects at work, but have used package managers in other languages for years and I can't think of a single time in recently memory where a vendor has broken something that affected our builds or deploys.

Comments from Reviewable


DocumentExtensions.GetUnusedStyles

https://forum.dynamobim.com/t/camber-feedback-thread/68942/11?u=mzjensen

Not going to have time to implement this.


⚠ Bofrost.fr has degraded performance

In cb157b0, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 906 ms

Resolved: Bofrost.fr performance has improved in 8da5536 after 9 minutes.


add some basic events for map import/export/rendering

i think these might be useful for more advanced plugins

leaving this for later, since those specific places are actually pending changes


Remove hardcoded auth schemas

Reviewed the schemas that we use and tried to avoid hardcoding. Using predefined defaults now

Great fix @nikolaykrondev !


District-wise number of patients is not showing,instead it is made 'unknown'.

I opened the website to check current number of patients in each district of Uttar Pradesh.But their wasn't any district-wise tally,but an 'unknown' tag.Also,not in U.P but in each of the state
Hope this gets rectified.
Here is the screenshot of the same.

Hi @Godspeed007  if you will open in smartphone's chrome browser or incognito mode in windows chrome browser then it will work fine.

Try to clear cache

Think issue is solved.Check without incognito.

Actually it was not the issue. It was just cache problem. I also encountered the problem but after clearing the cache issue was solved


replace localhost with base_url

GitHub Issue: (link)
What does this Pull Request do?
Replaces the localhost with ansiable variables.
How should this be tested?

Pull in this PR:

- src: https://github.com/Natkeeran/ansible-role-blazegraph
  name: Islandora-Devops.blazegraph
  version: replace_localhost

Vagrant up
test to see if content gets index into blazegraph

Interested parties
Tag (@ mention) interested parties or, if unsure, @Islandora-Devops/committers

Tested with https://github.com/Islandora-Devops/claw-playbook/pull/124 and it works as advertised.


DOCS: added attribute for javadoc-base-url-api to fix broken links

Issue* reported by user stating that links in the config intro were broken. Resolved to fix broken links by creating an attribute to javadoc-base-url-api. Links should now correctly resolve for each release.
*Cannot find the original issue number.
Broken links at bottom of page:
https://helidon.io/docs/v2/#/mp/config/01_introduction

Also added a brief topic description before the dependency.


Clean up Workload CVE directory structure

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #5639

PR #5624

PR #5629
PR #5659

PR #5458

PR #5673

PR #5719

PR #5718

PR #5741   👈

PR #5478

This comment was auto-generated by Graphite.


Added attributes and usage of immutable to std.net.isemail

AFAIK the functionality of validating email addressing with specific problems given back to the user isn't in vibe.d

Auto-merge toggled on


question about Bootstrap captions from noisy image-text pairs?

Is there any example scripts about how to use the pretrain checkpoint to Bootstrap captions from noisy image-text pairs, then get the much more cleaning train data?

You can check out our demo code, where we provide both the captioning script and image-text matching script that you can use to bootstrap.

thanks for response, the bootstrap step should like following::
1.use the captioning script to generate the image caption
2.then use the image-text matching script to compare the origin one and caption,
3.if the caption one or origin one similarity score bigger than setting threshold, keep the more similar one
and I still have some question:

if both lower than the than setting threshold, the image-text is likely to delete from the training data
would you like to provide the setting threshold value, bigger than 0.8?

The steps are:

generate synthetic captions
combine synthetic and original captions, and remove all captions that have a similarity score below 0.5

I have the same question about how to utilize Bootstrap as follows:
There are various versions of finetuned checkpoints in your released page, and which version is supposed to use for filter & which for captioner?
Thanks for your reply.

@LiJunnan1992
Got it, I still have the last question, the similarity score is use the cosine similarity,
for example in the demo : The image feature and text feature has a cosine similarity of 0.5262,
that the text image keep, otherwise remove

@trouble-maker007 We used the image-text matching score, although you can also use the cosine similarity.

I have the same question about how to utilize Bootstrap as follows: There are various versions of finetuned checkpoints in your released page, and which version is supposed to use for filter & which for captioner? Thanks for your reply.

@yzqinjacob We used the coco's caption model as captioner, and coco's retrieval model as filter.

@LiJunnan1992 Thank you very much for your detailed answer

@LiJunnan1992 Thansk for your reply and it helps me a lot.


multiple language support

Does the calendar support multiple language settings ?
If not, how complicated is it to implement ?

Please refer #90 .

@GianlucaFrongia Is this issue solved?


Retry  DoXCache during redis cluster scaling

Recently we got failed to fetch the cache because EXEC was aborted by redis or connection closed by DoMultiCache during redis cluster scaling  ( relate to #696 )
We believe that fail reason should be something relation to cluster scaling.
Check source below, seem like in TRANSACTION do commands MULTI, PTTL, GET/MGET, EXEC, then redis server return a error MOVED {slot} {ip:port} for PTTL command, and EXEC get a EXECABORT error. But driver only check with the EXEC command result.
https://github.com/redis/rueidis/blob/main/pipe.go#L1446-L1482
Seem like we can fix this easily, just handle previous error in TRANSACTION and let driver handle retry automaticlly.
We will test it ASAP and maybe file a PR later.

just handle previous error in TRANSACTION

Do you refer to the MOVED error you mentioned? I think it’s a good idea to replace the EXECABORT with the MOVED in this case.

Do you refer to the MOVED error you mentioned?

Yes, and ASK error too. Check PR #701


Can't build on raspbian

Hi,
I wanted to run mbed-edge on raspbian, and I built it as follows, but the error occurred as follows.
How can I solve it?
==== executed command ====
udo apt-get -y update
sudo apt-get -y install libc6-dev
sudo apt-get -y install libmosquitto-dev mosquitto-clients
sudo apt-get -y install build-essential cmake git doxygen graphviz
git clone https://github.com/ARMmbed/mbed-edge/
cd mbed-edge
#git submodule update --init --recursive
git submodule init
git submodule update
mkdir build
cd build
cat < toolchain_file
set(CMAKE_MAKE_PROGRAM make)
set(CMAKE_C_COMPILER gcc)
set(CMAKE_CXX_COMPILER g++)
EOF
cmake -DCMAKE_TOOLCHAIN_FILE=toolchain_file -DDEVELOPER_MODE=ON -DFIRMWARE_UPDATE=OFF ..
make
==== error ====
cmake -DCMAKE_TOOLCHAIN_FILE=toolchain_file -DDEVELOPER_MODE=ON -DFIRMWARE_UPDATE=OFF ..
-- The C compiler identification is GNU 8.3.0
-- Check for working C compiler: /usr/bin/gcc
CMake Error: Generator: execution of make failed. Make command was: "" "cmTC_5fedb/fast"
-- Check for working C compiler: /usr/bin/gcc -- broken
CMake Error at /usr/share/cmake-3.13/Modules/CMakeTestCCompiler.cmake:52 (message):
The C compiler
"/usr/bin/gcc"

is not able to compile a simple test program.
It fails with the following output:
Change Dir: /home/pi/src/pelion/mbed-edge/build/CMakeFiles/CMakeTmp

Run Build Command:"" "cmTC_5fedb/fast"
No such file or directory
Generator: execution of make failed. Make command was: "" "cmTC_5fedb/fast"

CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
CMakeLists.txt:28 (project)
-- Configuring incomplete, errors occurred!
See also "/home/pi/src/pelion/mbed-edge/build/CMakeFiles/CMakeOutput.log".
See also "/home/pi/src/pelion/mbed-edge/build/CMakeFiles/CMakeError.log".
Thank you.

@aacn14521 Why was this line commented out?

#git submodule update --init --recursive

I don't think git submodule update implies --recursive.
However, I also don't think that building on Raspbian is supported.

ARM Internal Ref: IOTGW-1070

Raspbian is not a supported platform. To build mbed-edge (part of Pelion Edge) for RPi, please follow the project - https://github.com/armPelionEdge/meta-pelion-edge


Design Guide: Use Design Drawables for Callouts

As a followup for #50, our design guide is using material icons instead of the graphics from the design.  While similar, they are not identical.
This issue is to

Come back and swap these out once we have vector versions of those assets from the design
Remove the dependency on the material-icons-extended library

Circular images means frame, not the content.


reporters: print all details of exception, not only the stack

It's fine if canopy wants to highlight the error by printing
the exception message first, the URL, and then the details.
But the details of it shouldn't be just the stacktrace because:

It doesn't contain the type of the exception.
It doesn't contain details about InnerExceptions.

I found this the hard way: receiving a "One or more errors
occurred" error, and not knowing what was going on for a while.
Later I realized that the exception was AggregateException
(which normally wraps exceptions that happen in async/parallel
code), and that it had the best explanation in the InnerException
of it.

You can find a new package here: https://www.nuget.org/packages/canopy/1.0.7


Share build cache with you guys?

Just want to make sure I'm not missing out on perf improvements: can 3P contributors access your build cache for this repo?

They can't, it's a password-protected instance. We don't want everyone to use it as a free cache for all their own projects :)

To put @oehme's comment a little bit differently: Currently we don't enable anonymous read access on our cache instance, given that we also use it for private projects, too. We may use a separate cache node for the gradle/gradle project alone and maybe allow anonymous read access to it in the future.
So currently, 3P contributors cannot access our build cache.

Alright, thanks for the info and glad to know you're aware of it.


Smarter autocomplete using BEL sematic information

Ideally autocomplete could be smarter.
For example, given 'p(', the autocomplete suggestions should reflect only the allowable arguments to the proteinAbundance function, e.g., namespace prefixes for namespaces that include Protein Concepts (HGNC, MGI, RGD, SP, EGID, SFAM). (potentially nice to also include potential outer functions like 'kin(p(' as suggestions).
Similarly, given 'bp(', should limit suggestions to namespace prefixes for namespaces including Biological Process concepts (GOBP, MESHPP).
Given a protein activity function 'kin(', the suggestions should be limited to the compatible functions (complexAbundance and proteinAbundance).

Delivered by Tony in semantic autocomplete feature


Parallel processing of cleaners and learners.

Is your feature request related to a problem? Please describe.
Multi processing of each import script into its own process.
Describe the solution you'd like
pooling or multiprocess but we cant multithread properly in python.
Describe alternatives you've considered
As above we dont have much choice, but whether or not to pool is still on the table.
Additional context
Cool as a cucumber and lightning reflexes.

#94


fix field name empty issue

bleve analysis will leave the field name empty for non-composite
fields.  we need to fix this up, as we use the analysis results
directly to return search results.

Pull Request Test Coverage Report for Build 548492784

7 of 9 (77.78%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 92.629%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

doc.go
7
9
77.78%

Totals

Change from base Build 541453574:
-0.3%

Covered Lines:
465

Relevant Lines:
502

💛  - Coveralls


Revert "lib/trivial: fix 'error: cannot decode virtual path '/nix/sto…

…re/virtual0000000000000000000000005-source''"
This reverts commit b67ee6e861903abb04e9024d605dfc7b00922633.
https://github.com/NixOS/nixpkgs/issues/202244
error: a string that refers to a store path cannot be appended to a path, at /etc/nixos/nix/nixos-unstable/lib/sources.nix:193:30
appears to happen when there's a nixpkgs git submodule

So one of the things that is different for a git submodule is that the .git folder isn't a folder, it's a textfile that contains (in my case) this:

$ cat nix/nixos-unstable/.git
gitdir: ../../.git/modules/nixpkgs

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Here is an alternative solution that could work with https://github.com/NixOS/nix/pull/6530. It does however make a useless copy of the .git directory to the store.

Here is an alternative solution that could work with NixOS/nix#6530. It does however make a useless copy of the .git directory to the store.

you forgot to link/paste the code


feat(translations): added spanish

Added spanish to DevToysMac

@RodrigoTomeES  Thank you for your translation! I don't know any languages other than Japanese and English, so the translations really help! Thank you again!


updated/added DE translations of Node-RED-terms

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

Proposed changes
updated/added german transalations of Node-RED-terms in editor-client/locales/de/editor.json

Checklist

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

Coverage remained the same at 69.392% when pulling a37b5c1194b1bf1417b9cbdae7f80d325dd66701 on heikokue:i18n-de/node-red-terms into 9b1c114c3f5336e8f6b410a96f8f9c61a944f437 on node-red:dev.

merged into #2806


Can't load data with DDP and GPUs=4

Bug description
when  I use pl to train my model, this python script can only run when I use one GPU, after I change the trainer's arg "devices=[0] " to "devices=[0,1,2,3]" ,this script can's finish data sanity check.
What version are you seeing the problem on?
v2.0
How to reproduce the bug
PAD_IDX = 1
    # 数据设置
    df = pd.read_csv("group_selfies.txt", sep="\t", header=None)
    train_df, test_df = train_test_split(df, test_size=0.01, random_state=42)

    train_dataset = GSDataset(data=train_df, vocab_file="vocab_gs.txt", max_length=256)
    test_dataset = GSDataset(data=test_df, vocab_file="vocab_gs.txt", max_length=256)
    print(f"train_dataset:{len(train_dataset)},test_dataset:{len(test_dataset)}")
    # 模型设置
    BATCH = 256
    epoch = 150
    ntoken = train_dataset.get_vocab_size()
    d_model = 64
    nhead = 8
    d_hid = 4096
    nlayers = 4
    hparams = {
        "ntoken": ntoken,
        "BATCH": BATCH,
        "epoch": epoch,
        "d_model": d_model,
        "nhead": nhead,
        "d_hid": d_hid,
        "nlayers": nlayers,
        "lr": 1e-3,
        "remarks": """
        decoder-only
        """,
    }

    # 传入模型
    model = GSTransformer(
        train_dataset,
        test_dataset,
        ntoken,
        hparams["d_model"],
        hparams["nhead"],
        hparams["d_hid"],
        hparams["nlayers"],
        lr=1e-3,
        hparams=hparams,
    )
    ## re-pretrain
    # checkpoint = torch.load(
    # "lightning_logs/version_1/checkpoints/epoch=131-step=17028.ckpt"
    # )
    # model.load_state_dict(checkpoint["state_dict"])
    # 设置回调函数
    from pytorch_lightning.callbacks import LearningRateMonitor, ModelCheckpoint

    checkpoint_callback = ModelCheckpoint(monitor="val_loss")
    early_stop_callback = pl.callbacks.EarlyStopping(
        monitor="val_loss", min_delta=0.00, patience=10, verbose=False, mode="min"
    )
    lr_monitor = LearningRateMonitor(logging_interval="epoch")
    # Train model
    trainer = pl.Trainer(
        max_epochs=epoch,
        devices=4,
        accelerator="gpu",
        strategy="ddp",
        callbacks=[checkpoint_callback, lr_monitor],
        accumulate_grad_batches=2,
        precision="16-mixed",
        logger=pl.loggers.TensorBoardLogger(
            "lightning_logs/group-selfies", name="gs2gs-recon"
        ),
    )
    trainer.fit(
        model,
        DataLoader(train_dataset, batch_size=BATCH, shuffle=True, num_workers=0),
        DataLoader(test_dataset, batch_size=BATCH * 2, shuffle=False, num_workers=0),
    )

Error messages and logs
# Error messages and logs here please

Environment

Current environment

CUDA:
- GPU:
- NVIDIA GeForce RTX 4090
- NVIDIA GeForce RTX 4090
- NVIDIA GeForce RTX 4090
- NVIDIA GeForce RTX 4090
- available:         True
- version:           11.8
Lightning:
- lightning-utilities: 0.9.0
- pytorch-lightning: 2.0.6
- torch:             2.0.1
- torchaudio:        2.0.2
- torchmetrics:      1.0.1
- torchvision:       0.15.2
Packages:
- aiohttp:           3.8.3
- aiosignal:         1.2.0
- appdirs:           1.4.4
- asttokens:         2.0.5
- async-timeout:     4.0.2
- attrs:             22.1.0
- backcall:          0.2.0
- bottleneck:        1.3.5
- brotlipy:          0.7.0
- certifi:           2023.7.22
- cffi:              1.15.1
- charset-normalizer: 2.0.4
- click:             8.0.4
- comm:              0.1.2
- contourpy:         1.0.5
- cryptography:      41.0.2
- cycler:            0.11.0
- datasets:          2.12.0
- debugpy:           1.6.7
- decorator:         5.1.1
- dill:              0.3.6
- emmet-core:        0.63.1
- executing:         0.8.3
- fastprogress:      1.0.3
- filelock:          3.9.0
- fonttools:         4.25.0
- frozenlist:        1.3.3
- fsspec:            2023.4.0
- future:            0.18.3
- gensim:            4.3.1
- global-chem:       <IP_ADDRESS>
- gmpy2:             2.1.2
- greenlet:          2.0.1
- group-selfies:     1.0.0
- huggingface-hub:   0.15.1
- idna:              3.4
- importlib-metadata: 6.0.0
- importlib-resources: 5.2.0
- iprogress:         0.4
- ipykernel:         6.19.2
- ipython:           8.12.0
- jedi:              0.18.1
- jinja2:            3.1.2
- joblib:            1.2.0
- jupyter-client:    8.1.0
- jupyter-core:      5.3.0
- kiwisolver:        1.4.4
- latexcodec:        2.0.1
- lightning-utilities: 0.9.0
- markupsafe:        2.1.1
- matplotlib:        3.7.1
- matplotlib-inline: 0.1.6
- mkl-fft:           1.3.6
- mkl-random:        1.2.2
- mkl-service:       2.4.0
- monty:             2023.5.8
- mp-api:            0.33.3
- mpmath:            1.3.0
- msgpack:           1.0.5
- multidict:         6.0.2
- multiprocess:      0.70.14
- munkres:           1.1.4
- nest-asyncio:      1.5.6
- networkx:          3.1
- numexpr:           2.8.4
- numpy:             1.25.0
- packaging:         23.0
- palettable:        3.3.3
- pandas:            1.5.3
- parso:             0.8.3
- pexpect:           4.8.0
- pickleshare:       0.7.5
- pillow:            9.4.0
- pip:               23.2.1
- platformdirs:      2.5.2
- plotly:            5.15.0
- pooch:             1.4.0
- prompt-toolkit:    3.0.36
- protobuf:          3.20.3
- psutil:            5.9.0
- ptyprocess:        0.7.0
- pure-eval:         0.2.2
- pyarrow:           11.0.0
- pybtex:            0.24.0
- pycairo:           1.23.0
- pycparser:         2.21
- pydantic:          1.10.12
- pygments:          2.15.1
- pymatgen:          2023.7.20
- pyopenssl:         23.2.0
- pyparsing:         3.0.9
- pysocks:           1.7.1
- python-dateutil:   2.8.2
- pytorch-lightning: 2.0.6
- pytz:              2022.7
- pyyaml:            6.0
- pyzmq:             25.1.0
- rdkit:             2023.3.2
- regex:             2022.7.9
- reportlab:         3.6.12
- requests:          2.31.0
- responses:         0.13.3
- ruamel.yaml:       0.17.32
- ruamel.yaml.clib:  0.2.7
- sacremoses:        0.0.43
- safetensors:       0.3.1
- scikit-learn:      1.2.2
- scipy:             1.10.1
- setuptools:        68.0.0
- six:               1.16.0
- smart-open:        6.3.0
- smilespe:          0.0.3
- spglib:            2.0.2
- sqlalchemy:        1.4.39
- stack-data:        0.2.0
- sympy:             1.11.1
- tabulate:          0.9.0
- tenacity:          8.2.2
- tensorboardx:      2.2
- threadpoolctl:     2.2.0
- tokenizers:        0.13.2
- torch:             2.0.1
- torchaudio:        2.0.2
- torchmetrics:      1.0.1
- torchvision:       0.15.2
- tornado:           6.3.2
- tqdm:              4.65.0
- traitlets:         5.7.1
- transformers:      4.31.0
- triton:            2.0.0
- typing-extensions: 4.7.1
- uncertainties:     3.1.7
- urllib3:           1.26.16
- wcwidth:           0.2.5
- wheel:             0.38.4
- xxhash:            2.0.2
- yarl:              1.8.1
- zipp:              3.11.0
System:
- OS:                Linux
- architecture:
- 64bit
- ELF
- processor:         x86_64
- python:            3.9.17
- release:           5.15.0-60-generic
- version:           #66-Ubuntu SMP Fri Jan 20 14:29:49 UTC 2023

More info
I
cc @justusschock @awaelchli

@wzy461143268 Thanks for opening the issue.
Do you have a link to the full code? Have you successfully ran any other PyTorch or Lightning multi-GPU experiments or is this the first one? Please also run the code with NCCL_DEBUG=INFO to report the logs here, that would be helpful, thanks.

@wzy461143268 Thanks for opening the issue. Do you have a link to the full code? Have you successfully ran any other PyTorch or Lightning multi-GPU experiments or is this the first one? Please also run the code with NCCL_DEBUG=INFO to report the logs here, that would be helpful, thanks.

My old server has RTX2080Ti x4，the same script can run on that server without change. I'll try get a log with NCCL debug.

NCCL_DEBUG=INFO

Besides, ctrl+C can't shutdown this program

@wzy461143268 Thanks for opening the issue. Do you have a link to the full code? Have you successfully ran any other PyTorch or Lightning multi-GPU experiments or is this the first one? Please also run the code with NCCL_DEBUG=INFO to report the logs here, that would be helpful, thanks.

(metal) user2@dell:~/metal$ NCCL_DEBUG=info python pretrain_gs_recon_linkers.py
train_dataset:321843,test_dataset:3251
Using 16bit Automatic Mixed Precision (AMP)
GPU available: True (cuda), used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
HPU available: False, using: 0 HPUs
Initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/4
train_dataset:321843,test_dataset:3251
train_dataset:321843,test_dataset:3251
train_dataset:321843,test_dataset:3251
Initializing distributed: GLOBAL_RANK: 3, MEMBER: 4/4
Initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/4
Initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/4
distributed_backend=nccl
All distributed processes registered. Starting with 4 processes
dell:2942883:2942883 [0] NCCL INFO Bootstrap : Using ens111f0:<IP_ADDRESS><0>
dell:2942883:2942883 [0] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
dell:2942883:2942883 [0] NCCL INFO cudaDriverVersion 12000
NCCL version 2.14.3+cuda11.8
dell:2942883:2943186 [0] NCCL INFO Failed to open libibverbs.so[.1]
dell:2942883:2943186 [0] NCCL INFO NET/Socket : Using [0]ens111f0:<IP_ADDRESS><0>
dell:2942883:2943186 [0] NCCL INFO Using network Socket
dell:2942982:2942982 [2] NCCL INFO cudaDriverVersion 12000
dell:2942981:2942981 [1] NCCL INFO cudaDriverVersion 12000
dell:2942982:2942982 [2] NCCL INFO Bootstrap : Using ens111f0:<IP_ADDRESS><0>
dell:2942982:2942982 [2] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
dell:2942981:2942981 [1] NCCL INFO Bootstrap : Using ens111f0:<IP_ADDRESS><0>
dell:2942981:2942981 [1] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
dell:2942982:2943192 [2] NCCL INFO Failed to open libibverbs.so[.1]
dell:2942982:2943192 [2] NCCL INFO NET/Socket : Using [0]ens111f0:<IP_ADDRESS><0>
dell:2942982:2943192 [2] NCCL INFO Using network Socket
dell:2942981:2943193 [1] NCCL INFO Failed to open libibverbs.so[.1]
dell:2942981:2943193 [1] NCCL INFO NET/Socket : Using [0]ens111f0:<IP_ADDRESS><0>
dell:2942981:2943193 [1] NCCL INFO Using network Socket
dell:2942983:2942983 [3] NCCL INFO cudaDriverVersion 12000
dell:2942983:2942983 [3] NCCL INFO Bootstrap : Using ens111f0:<IP_ADDRESS><0>
dell:2942983:2942983 [3] NCCL INFO NET/Plugin : No plugin found (libnccl-net.so), using internal implementation
dell:2942983:2943194 [3] NCCL INFO Failed to open libibverbs.so[.1]
dell:2942983:2943194 [3] NCCL INFO NET/Socket : Using [0]ens111f0:<IP_ADDRESS><0>
dell:2942983:2943194 [3] NCCL INFO Using network Socket
dell:2942981:2943193 [1] NCCL INFO Setting affinity for GPU 5 to fff0,00fff000
dell:2942982:2943192 [2] NCCL INFO Setting affinity for GPU 6 to fff0,00fff000
dell:2942983:2943194 [3] NCCL INFO Setting affinity for GPU 7 to fff0,00fff000
dell:2942883:2943186 [0] NCCL INFO Setting affinity for GPU 4 to fff0,00fff000
dell:2942983:2943194 [3] NCCL INFO Trees [0] -1/-1/-1->3->2 [1] -1/-1/-1->3->2 [2] -1/-1/-1->3->2 [3] -1/-1/-1->3->2
dell:2942982:2943192 [2] NCCL INFO Trees [0] 3/-1/-1->2->1 [1] 3/-1/-1->2->1 [2] 3/-1/-1->2->1 [3] 3/-1/-1->2->1
dell:2942883:2943186 [0] NCCL INFO Channel 00/04 :    0   1   2   3
dell:2942981:2943193 [1] NCCL INFO Trees [0] 2/-1/-1->1->0 [1] 2/-1/-1->1->0 [2] 2/-1/-1->1->0 [3] 2/-1/-1->1->0
dell:2942883:2943186 [0] NCCL INFO Channel 01/04 :    0   1   2   3
dell:2942883:2943186 [0] NCCL INFO Channel 02/04 :    0   1   2   3
dell:2942883:2943186 [0] NCCL INFO Channel 03/04 :    0   1   2   3
dell:2942883:2943186 [0] NCCL INFO Trees [0] 1/-1/-1->0->-1 [1] 1/-1/-1->0->-1 [2] 1/-1/-1->0->-1 [3] 1/-1/-1->0->-1
dell:2942982:2943192 [2] NCCL INFO Channel 00/0 : 2[9c000] -> 3[9e000] via P2P/IPC
dell:2942983:2943194 [3] NCCL INFO Channel 00/0 : 3[9e000] -> 0[9a000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Channel 00/0 : 1[9b000] -> 2[9c000] via P2P/IPC
dell:2942982:2943192 [2] NCCL INFO Channel 01/0 : 2[9c000] -> 3[9e000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Channel 01/0 : 1[9b000] -> 2[9c000] via P2P/IPC
dell:2942983:2943194 [3] NCCL INFO Channel 01/0 : 3[9e000] -> 0[9a000] via P2P/IPC
dell:2942982:2943192 [2] NCCL INFO Channel 02/0 : 2[9c000] -> 3[9e000] via P2P/IPC
dell:2942883:2943186 [0] NCCL INFO Channel 00/0 : 0[9a000] -> 1[9b000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Channel 02/0 : 1[9b000] -> 2[9c000] via P2P/IPC
dell:2942983:2943194 [3] NCCL INFO Channel 02/0 : 3[9e000] -> 0[9a000] via P2P/IPC
dell:2942982:2943192 [2] NCCL INFO Channel 03/0 : 2[9c000] -> 3[9e000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Channel 03/0 : 1[9b000] -> 2[9c000] via P2P/IPC
dell:2942883:2943186 [0] NCCL INFO Channel 01/0 : 0[9a000] -> 1[9b000] via P2P/IPC
dell:2942983:2943194 [3] NCCL INFO Channel 03/0 : 3[9e000] -> 0[9a000] via P2P/IPC
dell:2942883:2943186 [0] NCCL INFO Channel 02/0 : 0[9a000] -> 1[9b000] via P2P/IPC
dell:2942883:2943186 [0] NCCL INFO Channel 03/0 : 0[9a000] -> 1[9b000] via P2P/IPC
dell:2942982:2943192 [2] NCCL INFO Connected all rings
dell:2942983:2943194 [3] NCCL INFO Connected all rings
dell:2942983:2943194 [3] NCCL INFO Channel 00/0 : 3[9e000] -> 2[9c000] via P2P/IPC
dell:2942983:2943194 [3] NCCL INFO Channel 01/0 : 3[9e000] -> 2[9c000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Connected all rings
dell:2942982:2943192 [2] NCCL INFO Channel 00/0 : 2[9c000] -> 1[9b000] via P2P/IPC
dell:2942983:2943194 [3] NCCL INFO Channel 02/0 : 3[9e000] -> 2[9c000] via P2P/IPC
dell:2942883:2943186 [0] NCCL INFO Connected all rings
dell:2942982:2943192 [2] NCCL INFO Channel 01/0 : 2[9c000] -> 1[9b000] via P2P/IPC
dell:2942983:2943194 [3] NCCL INFO Channel 03/0 : 3[9e000] -> 2[9c000] via P2P/IPC
dell:2942982:2943192 [2] NCCL INFO Channel 02/0 : 2[9c000] -> 1[9b000] via P2P/IPC
dell:2942982:2943192 [2] NCCL INFO Channel 03/0 : 2[9c000] -> 1[9b000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Channel 00/0 : 1[9b000] -> 0[9a000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Channel 01/0 : 1[9b000] -> 0[9a000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Channel 02/0 : 1[9b000] -> 0[9a000] via P2P/IPC
dell:2942981:2943193 [1] NCCL INFO Channel 03/0 : 1[9b000] -> 0[9a000] via P2P/IPC
dell:2942983:2943194 [3] NCCL INFO Connected all trees
dell:2942983:2943194 [3] NCCL INFO threadThresholds 8/8/64 | 32/8/64 | 512 | 512
dell:2942983:2943194 [3] NCCL INFO 4 coll channels, 4 p2p channels, 2 p2p channels per peer
dell:2942883:2943186 [0] NCCL INFO Connected all trees
dell:2942883:2943186 [0] NCCL INFO threadThresholds 8/8/64 | 32/8/64 | 512 | 512
dell:2942883:2943186 [0] NCCL INFO 4 coll channels, 4 p2p channels, 2 p2p channels per peer
dell:2942982:2943192 [2] NCCL INFO Connected all trees
dell:2942982:2943192 [2] NCCL INFO threadThresholds 8/8/64 | 32/8/64 | 512 | 512
dell:2942982:2943192 [2] NCCL INFO 4 coll channels, 4 p2p channels, 2 p2p channels per peer
dell:2942981:2943193 [1] NCCL INFO Connected all trees
dell:2942981:2943193 [1] NCCL INFO threadThresholds 8/8/64 | 32/8/64 | 512 | 512
dell:2942981:2943193 [1] NCCL INFO 4 coll channels, 4 p2p channels, 2 p2p channels per peer
dell:2942983:2943194 [3] NCCL INFO comm 0x41327780 rank 3 nranks 4 cudaDev 3 busId 9e000 - Init COMPLETE
dell:2942883:2943186 [0] NCCL INFO comm 0x3f55ac20 rank 0 nranks 4 cudaDev 0 busId 9a000 - Init COMPLETE
dell:2942981:2943193 [1] NCCL INFO comm 0x42e18520 rank 1 nranks 4 cudaDev 1 busId 9b000 - Init COMPLETE
dell:2942982:2943192 [2] NCCL INFO comm 0x3df13c70 rank 2 nranks 4 cudaDev 2 busId 9c000 - Init COMPLETE

@wzy461143268 Thanks for opening the issue. Do you have a link to the full code? Have you successfully ran any other PyTorch or Lightning multi-GPU experiments or is this the first one? Please also run the code with NCCL_DEBUG=INFO to report the logs here, that would be helpful, thanks.

seems like NCCL‘s wrong,maybe I can try to diable IOMMU?

@wzy461143268 Thanks for opening the issue. Do you have a link to the full code? Have you successfully ran any other PyTorch or Lightning multi-GPU experiments or is this the first one? Please also run the code with NCCL_DEBUG=INFO to report the logs here, that would be helpful, thanks.

when I try nccl_test , the test will be stuck with -g 8,and will test well with -g 1

You can also try to run with NCCL_P2P_DISABLE=1 to see if peer to peer communication is an issue on your system.

NCCL_P2P_DISABLE=1

yes, It's useful for me . What can I do to solve this problem?

You can also try to run with NCCL_P2P_DISABLE=1 to see if peer to peer communication is an issue on your system.

I find that with or without NCCL_P2P_DISABLE=1 the script’s result don't change， There is a bug of transferring data between two gpus

That code you show here doesn't use NCCL, there is no communication between the GPUs. You are just moving tensors from CPU to GPU. In order to find out what the issue is, I would run the basic tests here in the NCCL troubleshooting guide: https://docs.nvidia.com/deeplearning/nccl/user-guide/docs/troubleshooting.html#gpu-to-gpu-communication
If you google "NCCL hang" you'll also find several other reports of similar issues.
For example, there are several cases where disabling IOMMU or ACS helped resolve similar problems:
https://docs.nvidia.com/deeplearning/nccl/user-guide/docs/troubleshooting.html#pci-access-control-services-acs
If this is happening on your personal desktop computer, you can resolve this yourself. If this is happening on a remote system in your university or company, it is likely your admin will have to help with this.

That code you show here doesn't use NCCL, there is no communication between the GPUs. You are just moving tensors from CPU to GPU. In order to find out what the issue is, I would run the basic tests here in the NCCL troubleshooting guide: https://docs.nvidia.com/deeplearning/nccl/user-guide/docs/troubleshooting.html#gpu-to-gpu-communication
If you google "NCCL hang" you'll also find several other reports of similar issues. For example, there are several cases where disabling IOMMU or ACS helped resolve similar problems: https://docs.nvidia.com/deeplearning/nccl/user-guide/docs/troubleshooting.html#pci-access-control-services-acs If this is happening on your personal desktop computer, you can resolve this yourself. If this is happening on a remote system in your university or company, it is likely your admin will have to help with this.

ok，I will feedback this problem to my admin，Thanks for your help.

That code you show here doesn't use NCCL, there is no communication between the GPUs. You are just moving tensors from CPU to GPU. In order to find out what the issue is, I would run the basic tests here in the NCCL troubleshooting guide: https://docs.nvidia.com/deeplearning/nccl/user-guide/docs/troubleshooting.html#gpu-to-gpu-communication
If you google "NCCL hang" you'll also find several other reports of similar issues. For example, there are several cases where disabling IOMMU or ACS helped resolve similar problems: https://docs.nvidia.com/deeplearning/nccl/user-guide/docs/troubleshooting.html#pci-access-control-services-acs If this is happening on your personal desktop computer, you can resolve this yourself. If this is happening on a remote system in your university or company, it is likely your admin will have to help with this.

I re-run this script in the other machine with RTX2080Ti x4, the result is correct, I don't need through cpu to pass tensor from one GPU to others. I think it's a bug of CUDA.

Now, with NCCL_P2P_DISABLE=1 I can train my model with 4 GPUs, but there is a bug when I copy a tensor to the other GPU without  pass it to cpu. If train with mutliGPU is safe although this bug exsits? Thanks for your anwser.

Thanks a lot. After update my driver from 525 to 535,this problem has been solved.

Thanks for coming back and confirming!

@SyntaxSmith

Thanks a lot. After update my driver from 525 to 535,this problem has been solved.

Did this driver update and NCCL_P2P_DISABLE=1 solve your issue? or just driver update?


Update REST API json specifications to 6.2

Also includes running the API Generator tool to create the C# code.