open_date is not saved to MySQL

Hi, i just tested with several versions of ruby and redmine to use your plugin.
But unable to get it working.
No changes to mysql issues.open_date field on issue save action.
(When changed in database manually then open_date is shown on issue page)
Started PATCH "/issues/368" for <IP_ADDRESS> at 2018-03-20 18:40:30 +0100
Processing by IssuesController#update as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"An7b9Q5OGD9e6q6vlebhr6LGEGxou/F3PYOsPpgVZxrT1TAMPplNPXpcZgkJ27TmhhIqrS5PlslGzX4KYBd6iw==", "form_update_triggered_by"=>"", "issue"=>{"is_private"=>"0", "project_id"=>"1", "tracker_id"=>"2", "subject"=>"test", "description"=>"Laaa", "status_id"=>"1", "priority_id"=>"4", "assigned_to_id"=>"3", "parent_issue_id"=>"", "start_date"=>"2018-03-20", "due_date"=>"", "estimated_hours"=>"", "done_ratio"=>"0", "open_date(1i)"=>"2018", "open_date(2i)"=>"4", "open_date(3i)"=>"7", "open_date(4i)"=>"04", "open_date(5i)"=>"15", "notes"=>"ddddd", "private_notes"=>"0", "lock_version"=>"0"}, "was_default_status"=>"1", "time_entry"=>{"hours"=>"", "activity_id"=>"10", "comments"=>""}, "last_journal_id"=>"", "commit"=>"Potvrdit", "next_issue_id"=>"367", "issue_position"=>"1", "issue_count"=>"118", "id"=>"368"}
  Current user: vitex (id=3)
  Rendered mailer/_issue.text.erb (6.0ms)
  Rendered mailer/issue_edit.text.erb within layouts/mailer (7.4ms)
  Rendered mailer/_issue.html.erb (1.2ms)
  Rendered mailer/issue_edit.html.erb within layouts/mailer (4.1ms)
Redirected to http://localhost:3000/issues/368?issue_count=118&issue_position=1&next_issue_id=367
Completed 302 Found in 943ms (ActiveRecord: 346.9ms)
Started GET "/issues/368?issue_count=118&issue_position=1&next_issue_id=367" for <IP_ADDRESS> at 2018-03-20 18:40:31 +0100
Processing by IssuesController#show as HTML
  Parameters: {"issue_count"=>"118", "issue_position"=>"1", "next_issue_id"=>"367", "id"=>"368"}
  Current user: vitex (id=3)
  Rendered issues/_action_menu.html.erb (3.8ms)
  Rendered plugins/redmine_issue_open_date/app/views/issues/_open_date.html.erb (0.3ms)
  Rendered issue_relations/_form.html.erb (1.5ms)
  Rendered issues/_relations.html.erb (2.8ms)
  Rendered issues/_history.html.erb (6.3ms)
  Rendered issues/_action_menu.html.erb (2.8ms)
  Rendered issues/_form_custom_fields.html.erb (0.1ms)
  Rendered issues/_attributes.html.erb (10.4ms)
  Rendered plugins/redmine_issue_open_date/app/views/issues/_open_date_field.html.erb (2.8ms)
  Rendered issues/_form.html.erb (26.9ms)
  Rendered attachments/_form.html.erb (1.8ms)
  Rendered issues/_edit.html.erb (37.6ms)
  Rendered issues/_sidebar.html.erb (3.3ms)
  Rendered watchers/_watchers.html.erb (2.9ms)
  Rendered issues/show.html.erb within layouts/base (190.4ms)
Completed 200 OK in 336ms (Views: 201.6ms | ActiveRecord: 110.4ms)

Thank for yor work and time

Hello!
Did you set up which tickets will be considered as New or Frozen on plugin settings page?

Yes,

Just tested with mysql and postgresql and it worked for me.
Which version of plugin do you have? Does it match current master state? Also, you can try to pull develop branch and check whether it helped.

Hello, i have the same issue. I tryed main, develop and fix-worker-autoload branches, none of them working :(


Build(deps-dev): Bump eslint-plugin-react from 7.31.0 to 7.31.1 (#3965)

Pull request opened by the merge tool on behalf of #3965

Merged in 01589b9cf124c9ae1a13d44312571c4c2309627d


Comments for Sleep Apnea Part 2

Got to thinking about you and looked up your old house in New Jersey. I see it's for sale now. Lots of fun memories from my short time visiting. Through the rabbit hole I found this page. I see that I'm not the only one with a CPAP (and issues, lol). Mine improve with time and practice. Anyway,  I hope that all is well with you.
Sunny J

Sunny J

This is unreal, I've been trying to figure out how to reach you for a while. I have a letter like 2/3 written that I was going to send to your practice (all I could figure out from google).
The Jersey house hasn't belonged to us for a while. The new owners painted it the ugliest color.
As to the CPAP, I have been meaning to write an update for a while. I learned how to read OSCAR, started mouth taping (on the advice of TheLankyLefty27), and basically no alcohol after 7pm most nights. I'll write a follow-up post. :)
I'm ashamed of how long it's been. How can I reach you? Can you send me an email??


kpt pkg update overwrites local changes to Kptfile

Expected behavior
Performing a kpt pkg update should merge changes to the Kptfile committed to the local directory with changes from the source package.
Actual behavior
Performing a kpt pkg update overwrites changes to the Kptfile committed to the local directory with changes from the source directory.
Information
Items of possible interest:

Kpt version: 1.0.0-beta.5
The changes are made to the Kptfile.
The ref to the remote package is an annotated git tag, not a branch.
The changes consist of adding the same root field of the Kptfile in both cases (i.e., .pipeline)

Steps to reproduce the behavior
1) Inspect the package source.   Note the changes to the Kptfile
% cd ~/src/REDACTED-SRC-REPO
REDACTED-SRC-REPO % git show pkg/gcp-data-project/v0.2.0
tag pkg/gcp-data-project/v0.2.0
Tagger: REDACTED
Date:   Mon Aug 23 17:42:49 2021 +0000

Automated via Cloud Build pipeline

commit 8d3eda19518d44ffa60a66ab6850b412ac4181bf (tag: pkg/gcp-data-project/v0.2.0)
Merge: 5bd274f 42705cc
Author: REDACTED
Date:   Mon Aug 23 12:42:16 2021 -0500

    Merge pull request #45 from wpengine/REDACTED/CTLST-1241

REDACTED-SRC-REPO % git show pkg/gcp-data-project/v0.2.1
tag pkg/gcp-data-project/v0.2.1
Tagger: REDACTED
Date:   Mon Aug 30 21:12:15 2021 +0000

Automated via Cloud Build pipeline

commit 6679457c518e11f0114554ba159a62e801fe4bc8 (HEAD, tag: pkg/gcp-data-project/v0.2.1)
Merge: 6b21e71 372b8e8
Author: REDACTED
Date:   Mon Aug 30 16:11:40 2021 -0500

    Merge pull request #49 from wpengine/CTLST-1254.2

    CTLST-1254 - update gcp-data-project to use gcp-project v0.2.1

REDACTED-SRC-REPO % git diff pkg/gcp-data-project/v0.2.0 pkg/gcp-data-project/v0.2.1 -- pkg/gcp-data-project/Kptfile

diff --git a/pkg/gcp-data-project/Kptfile b/pkg/gcp-data-project/Kptfile
index d3351d3..8a51bf6 100644
--- a/pkg/gcp-data-project/Kptfile
+++ b/pkg/gcp-data-project/Kptfile
@@ -4,3 +4,8 @@ metadata:
   name: gcp-data-project
 info:
   description: sample description
+pipeline:
+  mutators:
+    - image: gcr.io/kpt-fn/set-annotations:unstable
+      configMap:
+        wpengine.com/REDACTED-SRC-REPO.pkg-version.gcp-data-project: 0.2.1

2) Change to the repo that instantiates the package; switch to the commit that updates the version.
REDACTED-SRC-REPO % cd ~/src/cp
cp % git co 2b021a4947028b9b6824230ecf0769cd3613f724

HEAD is now at 2b021a4 Update Kptfile to gcp-data-project v0.2.1
cp % cd infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev

3) Inspect the current state of the Kptfile
wpe-ctlst-hello-world-dev % git show HEAD

commit 2b021a4947028b9b6824230ecf0769cd3613f724 (HEAD)
Author: REDACTED
Date:   Mon Aug 30 16:46:31 2021 -0500

    Update Kptfile to gcp-data-project v0.2.1

diff --git a/infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev/Kptfile b/infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev/Kptfile
index d64aa84..840beb1 100644
--- a/infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev/Kptfile
+++ b/infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev/Kptfile
@@ -7,7 +7,7 @@ upstream:
   git:
     repo<EMAIL_ADDRESS>     directory: /pkg/gcp-data-project
-    ref: v0.2.0
+    ref: v0.2.1
   updateStrategy: resource-merge
 upstreamLock:
   type: git
wpe-ctlst-hello-world-dev % cat Kptfile

apiVersion: kpt.dev/v1
kind: Kptfile
metadata:
  name: wpe-ctlst-hello-world-dev
upstream:
  type: git
  git:
    repo<EMAIL_ADDRESS>    directory: /pkg/gcp-data-project
    ref: v0.2.1
  updateStrategy: resource-merge
upstreamLock:
  type: git
  git:
    repo<EMAIL_ADDRESS>    directory: /pkg/gcp-data-project
    ref: pkg/gcp-data-project/v0.2.0
    commit: 0580598ccf4e914bd3e076931a43f64a94bd270f
info:
  description: sample description
pipeline:
  mutators:
    - image: gcr.io/kpt-fn/apply-setters:v0.1
      configPath: fn-cfg-setters.yaml

4) Update the package
wpe-ctlst-hello-world-dev % kpt pkg update .
Package "wpe-ctlst-hello-world-dev":
Fetching upstream from ssh://git<EMAIL_ADDRESS>From ssh://github.com/wpengine/REDACTED-SRC-REPO
 * tag               pkg/gcp-data-project/v0.2.1 -> FETCH_HEAD
Fetching origin from ssh://git<EMAIL_ADDRESS>From ssh://github.com/wpengine/REDACTED-SRC-REPO
 * branch            0580598ccf4e914bd3e076931a43f64a94bd270f -> FETCH_HEAD
Updating package "wpe-ctlst-hello-world-dev" with strategy "resource-merge".
Updating package "gcp-project" with strategy "resource-merge".

Package "wpe-ctlst-hello-world-dev/gcp-project":
Fetching upstream from ssh://git<EMAIL_ADDRESS>From ssh://github.com/wpengine/REDACTED-SRC-REPO
 * tag               pkg/gcp-project/v0.2.1 -> FETCH_HEAD
Fetching origin from ssh://git<EMAIL_ADDRESS>From ssh://github.com/wpengine/REDACTED-SRC-REPO
 * branch            80b0649ed24503daec0a570c466305fdab09fa50 -> FETCH_HEAD
Updating package "gcp-project" with strategy "resource-merge".

Updated 2 package(s).

5) Inspect the results of the update.
Note that the local changes (i.e., apply-setters) were overwritten by the changes from the source package (i.e., set-annotations).
wpe-ctlst-hello-world-dev % git diff Kptfile
diff --git a/infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev/Kptfile b/infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev/Kptfile
index 840beb1..7aa5760 100644
--- a/infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev/Kptfile
+++ b/infra/krm/catalyst-platform/data-projects/environments/dev/wpe-ctlst-hello-world-dev/Kptfile
@@ -14,11 +14,12 @@ upstreamLock:
   git:
     repo<EMAIL_ADDRESS>     directory: /pkg/gcp-data-project
-    ref: pkg/gcp-data-project/v0.2.0
-    commit: 0580598ccf4e914bd3e076931a43f64a94bd270f
+    ref: pkg/gcp-data-project/v0.2.1
+    commit: edc875a2f90ad28020457f7de3c78f60d562e1a2
 info:
   description: sample description
 pipeline:
   mutators:
-    - image: gcr.io/kpt-fn/apply-setters:v0.1
-      configPath: fn-cfg-setters.yaml
+    - image: gcr.io/kpt-fn/set-annotations:unstable
+      configMap:
+        wpengine.com/REDACTED-SRC-REPO.pkg-version.gcp-data-project: 0.2.1

@droot

What factors influenced your decision to moved to the new model?

We feel that this model reduces cognitive load.  That is, in effect it abstracts away the contents of the forked package.  To find the deltas from the forked package, you only look in the parent package, rather than digging through git history.  There's a cost, of course, in expressing the desired output in terms of a base package and edits, but it feels like it better expresses the intent than editing the forked resources.

We have had some surprises (in addition to this issue) in the kpt conflict resolution.  It's quite possible they have all had the same root cause; we haven't kept track.  In aggregate those problems were enough to make us not want to rely on it.  Making edits in the parent directory means that we don't need to rely on kpt conflict resolution.

Programmatic customizations can be done by functions in the parent, but I am curious about the manual edits, how are you achieving those?

We have been able to make all necessary edits via a combination of existing and custom kpt functions.  We commonly use search-replace, create-setters/apply-setters, and set-annotations/set-namespace/set-labels.  We have a custom function that allows us to apply kustomize style patches.  And when all else fails, we can always use starlark.
We haven't used this model extensively, so we may yet run into problems, but so far it's working out well.

@jafcrocker This request has been addressed. Please try out beta.12 version of kpt which will be released soon.


Disable odoc lint "warn-error" flag

Temporarily until odoc becomes a bit more helpful and lenient.

LGTM :-) (this will avoid having non-fixable errors in the irmin and mirage CI)


Update toc.yml to update MSDN entry for IoT Edge.

@kgremban - Would you review the proposed changes? Thanks

PRMerger Results

Issue
Resolution

Bread Crumb File(s)
This PR contains a change to a bread crumb or toc file.

Thanks for updating this, @jierong. #sign-off


mempool/lanes: Extend proto messages with lanes info

As defined in the ADR:

Extend InfoResponse with new fields repeated uint32 lane_priorities and uint32 default_lane_priority
Extend CheckTxResponse with new field uint32 lane

Cherry-pick commit from prototype branch: Extend InfoResponse and CheckTxResponse with lanes info


Better README.

Better README.
To do:

[x] part about composability
[x] part about liveness properties

This is still work in progress but  I will very much like any comments about it !

Should I add or remove something ?
Are the examples good enough ?
What is missing ?

@Sudha247
@polytypic

I just added the part about composability. I will complete the part on liveness properties very soon and after a last review, it should be good to merge :)

I actually decided to shorten the part about progress properties as it felt very unfocused and not needed. At least, I did not find the right way to talk about usefully about it yet, but I think the README looks good enough for this PR.

Thanks for all the reviews !


Generated metadata.json clash with de facto standard web app manifest

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
A web app manifest file can have any name, but is commonly named manifest.json and served from the root (your website's top-level directory). https://web.dev/add-manifest/#create
Placing a web app manifest file in /public/manifest.json or generating one in /src/routes/manifest.json.ts works in dev mode but not when building, since they are overwritten by SolidStart's own manifest.json file.
Expected behavior 🤔
Can we rename the SolidStart manifest.json to client.manifest.json or such so my manifest.json file is served?
Steps to reproduce 🕹
Steps:

Create a /public/manifest.json file
npm run build
npm run start
Access http://localhost:3000/metadata.json ← Wrong metadata.json file served.

Context 🔦
I am porting an existing Next.js app to SolidStart. I can rename my web app manifest.json, but since most ppl probably expect manifest.json to be available, perhaps it would be better to release the manifest.json name space for arbitrary dev use.
Your environment 🌎
No response

Interesting. That manifest is the one that Vite generates: https://vitejs.dev/guide/backend-integration.html
I wonder if there is a way to rename it or have it generate in a different location. Ah yes: https://vitejs.dev/config/build-options.html#build-manifest
Ok. We can look at renaming it to something like asset-manifest.json.

In setting up for SolidStarts next Beta Phase built on Nitro and Vinxi we are closing all PRs/Issues that will not be merged due to the system changing. If you feel your issue was closed by mistake. Feel free to re-open it after updating/testing against 0.4.x release. Thank you for your patience.

See https://github.com/solidjs/solid-start/pull/1139 for more details.


don't use a macro for glib::wrapper

we use a macro to define fundemental types such as gdk::Event/gsk::RenderNode/gtk::Expression, by doing so, rustdoc-stripper cannot find glib::wrapper call to re-add the docs to the right place. Which means we lose the documentation of such items.
Instead, we should move the glib::wrapper call outside of those macros. A bit of code duplication is okay if at the same price we get more complete docs.

We define this part
glib::wrapper! {
            #[derive(Debug)]
            pub struct $rust_type(Shared<$ffi_type>);

            match fn {
                ref => |ptr| ffi::gtk_expression_ref(ptr as *mut ffi::GtkExpression) as *mut $ffi_type,
                unref => |ptr| ffi::gtk_expression_unref(ptr as *mut ffi::GtkExpression),
            }
        }

manually for each type, and use the macro just to implement the various traits.

Oh I see. Yes that makes sense.


USHIFT-1010: Restrict created files permissions

Which issue(s) this PR addresses:

Closes #

@pacevedom , looks like unit-tests are failing. Should we fix the tests, or use 0600 permissions to allow re-writing of the certificates?

/test e2e-router-smoke-test


NoSuchMethodError thrown by v10.0.0 when test suite runs

Which version and edition of Flyway are you using?
10.0.0
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
N/A
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Gradle
Which database are you using? (Type & version)
Postgres v11
Which operating system are you using?
Mac OS X
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Changed build.gradle from v9.22.3 to v10.0.0 i.e.
plugins {
    id 'org.flywaydb.flyway' version '10.0.0' // was 9.22.3
}

dependencies {
    implementation "org.flywaydb:flyway-core:10.0.0" // was 9.22.3
    implementation "org.flywaydb:flyway-database-postgresql:10.0.0" // new dependency added during v10.0.0 upgrade
}

What did you expect to see?
The test suite should run without any errors (the app itself starts without any errors).
What did you see instead?
When I run the test suite of the (Spring Boot) app via ./gradlew test, every test fails due to the following error
  Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'flyway' defined in class path resource [org/springframework/boot/autoconfigure/flyway/FlywayAutoConfiguration$FlywayConfiguration.class]: Failed to instantiate [org.flywaydb.core.Flyway]: Factory method 'flyway' threw exception with message: 'org.flywaydb.core.api.configuration.FluentConfiguration org.flywaydb.core.api.configuration.FluentConfiguration.licenseKey(java.lang.String)'
  Caused by: org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.flywaydb.core.Flyway]: Factory method 'flyway' threw exception with message: 'org.flywaydb.core.api.configuration.FluentConfiguration org.flywaydb.core.api.configuration.FluentConfiguration.licenseKey(java.lang.String)'
  Caused by: java.lang.NoSuchMethodError: 'org.flywaydb.core.api.configuration.FluentConfiguration org.flywaydb.core.api.configuration.FluentConfiguration.licenseKey(java.lang.String)'

I'm using the community edition of Flyway, so I would not expect to have to configure a license key. My Flyway configuration (in build.gradle) is shown below
flyway {
    url = "jdbc:postgresql://<IP_ADDRESS>:5432/postgres"
    user = "postgres"
    password = "secret"
}

Hi @donalmurtagh , a spring-boot issue has been raised for this: https://github.com/spring-projects/spring-boot/issues/38164

@JasonLuo-Redgate this is the second time this has happened recently i.e. that a Flyway release is incompatible with the latest release of Spring Boot. I understand that the Spring support is not something controlled by Flyway, but given the popularity of Spring Boot, shouldn't this be tested and mentioned in the release notes if there's an incompatibility?

@donalmurtagh we are currently reviewing and planning how to be more active on this front. For a while now Flyways configuration has been pretty static with the exceptions of additions. However, recently to make flyway more open and expandable we have had to make a lot of changes to our configuration models which have caused this incompatibility. We are going to improve this by working more with partially the spring boot code base to improve these transitions but we are not there yet. Thank you for any patients.

The Sprint boot ticket is now fixed https://github.com/spring-projects/spring-boot/issues/38164


now uses runtime instead of framework

for use with: https://github.com/fabric8-launcher/launcher-creator-backend/pull/123
@quintesse @ia3andy like so

:tada: This PR is included in version 3.4.14 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Incorrect Deserialization Behaviour

I use Nitrite via Maven:
	<!--Nitrite -->
	<dependency>
 	   <groupId>org.dizitart</groupId>
 	   <artifactId>nitrite</artifactId>
  	  <version>3.0.0</version>
	</dependency>

I have the following code structure:
Class A
public class A {
    @Id
    public String name;

    public List<B> bs=new LinkedList<B>();
}

Class B
public class B {
    @Id
    public String bName;

    public List<B> preconditions=new LinkedList<B>();
}

My main class:
B b1=new B();
b1.bName="b1";

B b2=new B();
b2.bName="b2";

b1.preconditions.add(b2);

List<B> bs=new LinkedList<B>();
bs.add(b1);
bs.add(b2);

A a1=new A();
a1.name="mya1";
a1.bs=bs;

Nitrite db=Nitrite.builder()
	        .compressed()
	        .filePath("tmpDb.db").openOrCreate();
db.getRepository(A.class).insert(a1);

ObjectRepository<A> repo=db.getRepository(A.Class);

A myA=repo.find(ObjectFilters.eq("name","myA")).firstOrDefault();

System.out.println(myA.bs.get(0).preconditions.get(0));
System.out.println(myA.bs.get(0).preconditions.get(0).hashCode()+" --"  +myA.bs.get(0).preconditions.get(0).bName);
System.out.println(myA.bs.get(1));
System.out.println(myA.bs.get(1).hashCode() +" --" +  myA.bs.get(1).bName);

So before serializing I had a single B object (b2) which is referenced in both lists: a1.bs and b1.preconditions.
When I deserialize the object I get two B objects (one is referenced in a1.bs and a second which is referenced in b1.preconditions). So the B object in b1.preconditions does not reference to the B object in myA.bs. How can I prevent that two separate objects are created? I want to have the exect same Object structure in Java after deserialization...

You can't achieve this using serialization. Nitrite (which uses jackson behind the scene) does not keep track of references as per design (because nitrite does not do binary serialization). You can establish equality by overriding equals() and hashCode() in B, but reference equality can not be obtained this way.
But there is a way to achieve this. For this you have to skip auto serialization in nitrite and implement Mappable interface in your classes A and B. There you can keep a local cache of B instances and set the properties of your object according to your choice. Detailed documentation can be found here -https://www.dizitart.org/nitrite-database/#mappable
N.B: I'll strongly suggest to upgrade your nitrite version to the latest.


Notification of Preemption

https://azure.microsoft.com/en-us/blog/low-priority-scale-sets/ - this states (at the bottom) that you will be able to be notified before a VM in a scale set is preempted. Is there a similar process for Batch? 30 seconds would be enough time for our task to checkpoint where it's got to so it can be picked up elsewhere

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

ID: a4e98e12-baca-7a61-d646-0ee919c67368
Version Independent ID: 1d5a27cb-0b01-eda3-120b-cb33bc4d7f87
Content: Run workloads on cost-effective low-priority VMs - Azure Batch
Content Source: articles/batch/batch-low-pri-vms.md
Service: batch
GitHub Login: @mscurrell
Microsoft Alias: markscu

That is correct. You can configure metrics to alert you. Batch runs on Virtual Machine Scale Sets so essentially the functionality is the same however Batch is the wrapper.
https://docs.microsoft.com/en-us/azure/batch/batch-low-pri-vms#metrics

I will close this out. If you have further questions just let me know and we can always reopen and continue the discussion.


Add some icons

Add icons for VLC media player, JuiceSSH, F-Droid, Split APK Installer, APK Mirror Installer,  Element, Shazam, Tor Browser and Google GO please.
org.videolan.vlc
com.sonelli.juicessh
org.fdroid.fdroid
com.aefyr.sai
im.vector.app
com.apkmirror.helper.prod
com.shazam.android
org.torproject.torbrowser
com.google.android.apps.searchlite

Hello! Weʼre switching to a new system for icon requests in order to be able to better manage them. From now on, icon requests can be submitted here: https://forms.gle/Fx8vZAiWdW1Tyjo57. If your icon request hasnʼt yet been fulfiled, it has been copied to our new icon request database, meaning no action is required. Consequently, this issue will be closed.


Feature request: Icons of AppImage

Is your feature request related to a problem? Please describe.
Running an appimage runs fine but in the panel there is no icon of it. There was just a generic icon determined by the system. This creates problem in multitasking, we don't know which is running until we peek.
System: Linux Mint 21.3
Describe the solution you'd like
Having an app icon would be beautiful.
Describe alternatives you've considered
No response
Additional context
No response

I would also love to have an app icon for my tray or desktop! :)


Repolist setup

Summary
A RepoList page and associated route were added; there is a link to the page in the sidebar of the app. The RepoList page displays placeholder text indicating what page it is. It also performs an API call to get a list of repositories, then displays a RepoItem component for each. The RepoItem component currently contains only placeholder text, but it displays the repository name, demonstrating that the API call was successful. While the API call is in progress, some placeholder text indicating this appears instead of the repository information.
Issues Closed / Addressed
This PR closes #10 and closes #17.
Code Changes

Added RepoList folder, page and component
RepoList uses UseEffect to perform an API call only on the first render. Once the API call has finished, UseEffect updates the state variable containing the JSX that displays the list of repositories.
Added RepoItem component. The RepoItem displays placeholder text and the repository name.
Added route for the RepoList
Duplicate code change to common.tsx (fixed in merge) to hardcode authentication username and password

Hey Seneca could you clean up your commits so it doesn't include rebase error, and having multiple commits on the same files?

LGTM


Fixed messageHtmlToComponent returning undefined (#755)

This is a fix for https://github.com/mattermost/mattermost-webapp/pull/754 that I'm submitting separately because it's going backwards (master into 4.7) and doing them together would likely cause merge conflicts

@hmhealey What's needed for PM review here?

@jasonblais This is just waiting on https://github.com/mattermost/mattermost-webapp/pull/754 being approved for 4.7


Cannot fill "Destination" field

Hello
I'm testing this module (companion-module-imagine-lrc) on companion v3.2.2 and I notice a problem.
The module connects correctly to the router (Imagine Platinum MX 15RU) but when I try to add a "Crosspoint Take" action on a key or on a trigger, I cannot fill in the "Destination" field with a number or name (it is ignored) .
With companion 2.4.4 it works correctly, with the same configuration.
The problem occurs even if I don't connect to the router.
Is it a compatibility bug?
Thank you for this useful module!
Enrico
Companion v3.2.2 trigger action whith LRC Crosspoint Take:

I've finally had time to update the module to be properly functional in Companion v3. I'll reply back here once it's available in the betas, and that should resolve this issue.

v3 support is complete in #6 which will fix this. I've requested it to be added to the Companion betas, and hopefully will be available shortly.

It's in the betas as of 8/21. Feel free to reach back out with any issues.


Use BMI v2.0

Dakotathon should be updated to use BMI v2.0 (bmipy=2.0).

When this is fixed, revert the change to the requirements.txt file in #77.


Makes UrlRewritingProviderCollection thread safe

We've been running UrlRewriting.Net with this modification for years and years now and it's been shipped with Umbraco since forever so I'm pretty sure it works nicely. ;-)

Looks fine, thanks!


Fixed #567 #549

Created google_search_users_for_candidate_table tamplate to load google serach users table.
Show one google search user on candidate edit page with show more and show less link
Choose and Update button for twitter search will not cache image instead refresh_candidate_twitter_details will do caching at the end of candidate_edit_process
Updated likelihood_score for Ballotpedia, linkedin, facebook, twitter and wikipedia links
Top google search result on candidate list page filters out on the basis of image, score
Added Choose and update button to select google search user on candidate list page
Added common function extract_website_from_url, extract_facebook_username_from_text_string

Great work Anisha!


playoverwatch.com doesn't return 404 if player not found

We should to use other method to detect it.

Fixed in #20

Thanks again!


Changing the inventory property of CtrlInventoryGridEx results in an assertion

So other thing I've noticed is that when I set an inventory for the CtrlInventoryGridEx via script, unset it and set it again... the sort messes up... also if I use the item (potion) in the inventory and remove it, I believe the empty slot is still selected and sometimes happens that the items change positions an overlap. Try setting it up this way... You have 1 HUD for the inventory and many units with their inventory that have for example consumable items... on click of the unit you set the HUD's ctrl inventory to that units inventory and if you use the item for an example potion of that unit and remove it by using remove_item(item) function and select other unit, you'll get errors like Assertion failed: Item not found in the inventory! The sorting will be messed etc.

Originally posted by @filipinyo in https://github.com/peter-kish/gloot/issues/59#issuecomment-1279666988

Should be fixed in 70aec8f21fdb610cf0b638c84c12113a3d233f33.


Way to configure external Access log service

Please describe your use case / problem.
We have a requirement to send access logs from Ambassador to external telemetry platform. One approach to achieve this is to create additional service that can accept access logs from Envoy/Ambassador via grpc (it's possible in Envoy v2 config).
Describe the solution you'd like
It would be great to have "Ambassador way" to configure external grpc access log service.
Another approach is to have ability to customize generated Envoy V2 Config.

Tracked in #748.


fix: Prevent 'Get Results' from Executing When Color Inputs Are Empty

Fixes Issue - [BUG] results open when an added color is empty
My PR closes #504

[] My code follows the code style of this project.
[] This PR does not contain plagiarized content.
[] The title and description of the PR is clear and explains the approach.

sorry, but I don't think you understand or know what you are doing at all you don't like to read instructions and i rather and feel comfortable not to let you contribute to anything any more on my project. Thanks for wanting to contribute, I hope you learn to read instructions in later projects you contribute to.


Use less hacky way of getting around credential manager issue

Currently, when cloning repos, rustwide passes ghost:ghost@ at the beginning of the url's host to GitHub. This informs GitHub and the underlying credential manager on the host to use these credentials avoiding any prompts. Ultimately, these credentials are ignored because we are only cloning public repos.
This is a bit hacky. Another possible solution is writing our own credential manager that "does the right thing". We can then pass an argument to git clone to use this credential manager.

Expanding on the "write your own credential manager" idea, adding the following section to a gitconfig should duplicate the existing behavior. The question is whether this would go into the user's config (~/.gitconfig on Linux), or into .git/config in crater's source dir. AFAICT, .git/config doesn't actually get used when running git clone, even if you're in a git repo. Honestly, hard-coding the dummy credentials into the clone invocation seems less error prone.
[credential]
	helper = "!f() { echo username=ghost; echo password=ghost; }; f"

Fixed in rustwide 0.2.0.


fix(fancy_lsp_servers): cleaner way of fetching null-ls sources

Thank you for the amazing plugin!
Bug
According to null-ls docs, the official way of fetching null-ls sources is to use the get_source(query) function instead of directly accessing builtins table.
https://github.com/jose-elias-alvarez/null-ls.nvim/blob/main/doc/SOURCES.md
The biggest difference is that the current method does not distinguish sources that are disabled while results that return from get_source are already filtered internally.
Fix
This PR uses get_source function with query of filetype instead.
One breaking change is that the previous code only listed sources of formatting and diagnostics but now it lists all sources (i.e. from code_actions, completion, hover as well)
Is there any reason you were filtering?
Note
Out of the scope of this PR but I just wanted to inform you that vim.lsp.buf_get_clients(bufnr) has been deprecated since nvim-0.9.0 to be replaced by vim.lsp.get_active_clients({ bufnr = bufnr }). Ref
I'll leave it to you whether to fix it or not.

Thanks for this! Much cleaner now ;-)


Registry context, dense map and noexcept policy

Discussed in https://github.com/skypjack/entt/discussions/866
Registry context can copy its values when reallocated due to an emplace with exceptions disabled.
Please, refer to the linked discussion for further details.

Thank you.
I did a small test case with latest from master to show the different behavior of dense_map.
#if !defined(__cpp_exceptions)
#define _HAS_EXCEPTIONS 0
#endif

#include <memory>
#include <cassert>

#include "entt/entt.hpp"

struct TestA
{
};

struct TestB
{
    std::unique_ptr<char> Test;
};

int main()
{
    entt::registry reg;

    reg.ctx().emplace<TestB>();

    {
        TestB* v = reg.ctx().find<TestB>();
        assert(v != nullptr);
    }

    reg.ctx().emplace<TestA>();

    {
        TestB* v = reg.ctx().find<TestB>();
        assert(v != nullptr);
    }
}

I'm using MSVC 14.29.30133. With exceptions enabled (/EHsc)  it works as expected. With exceptions disabled (no /EHsc) it asserts at the end.

Apart from this problem I think it would be good to at least have a runtime check when basic_any is trying to copy a non-copyable type, just in case it gets to a situation like this (in basic_vtable when op == operation::copy)

I think you're @dpunsetAGS right in saying that ENTT_NOEXCEPT shouldn't depend on whether exceptions are enabled or not.
That is, by default is should be noexcept no matter what. I don't even know if it makes sense to be able to disable the noexcept keyword. It's not that it causes troubles when you disable the exceptions. Quite the opposite in fact as it works right now in EnTT.
Can you confirm that you can compile an UE4 project with ENTT_NOEXCEPT equal to noexcept? I wonder what the problem was here since it sounds like it wasn't the noexcept. 🤔

Yes sure, we changed it ourselves for the time being, otherwise it would crash when playing outside of the editor. Having always noexcept hasn't shown any problem in any target (with exceptions or without). Clang seems to be ok with it too.
I don't know what was the exact problem in the link you wrote :( The build tool is mentioned, but does it mean UBT? (Unreal Build Tool) The tool that invokes the compiler? For sure in Unreal 4.27 this don't seem to cause any issue at all, and noexcept is used in many places without checking if exceptions are enabled. I could find on Google some old post with people complaining about warning C4577 being triggered, but I think it's been disabled for quite a few years. Maybe that was the issue?

Correct me if I'm wrong, I'm just trying to confirm with myself the situation, but the problem seems to be that the way the dense_map works it relies on the fact that things will always move. The data in node is copiable, but the real data it is holding may not be (the only code that knows this is the vtable template instantiation), so it can't ever try to copy. A move constructor can't throw, because it could leave the moved object in a broken state, so it has to be marked with noexcept.
If ENTT_NOEXCEPT is allowed to exist it can give the impression that you can remove noexcept, but ultimately you can't do that, because dense_map relies on this feature to make sure the nodes are always moved.
What I don't understand is why this has any effect when compiling without exceptions. If exceptions are disabled exceptions won't trigger, so shouldn't it always choose move? Why is it choosing copy?

Technically speaking, it's more about the std::vector than the entt::dense_map.
The vector decides whether to copy or move when reallocating based on the following condition:
is_nothrow_move_constructible_v<_Ty> || !is_copy_constructible_v<_Ty>

An any has both a copy and move constructor that also are always available. We cannot make them conditionally enabled since the type of the contained object is erased. Therefore, !is_copy_constructible_v<_Ty> is always false.
On the other hand, is_nothrow_move_constructible_v looks into the move constructor to know if it's noexcept. It doesn't matter whether exceptions are enabled or not. I've tested the three major compilers and this trait returns false for a non-noexcept constructor in all cases when exceptions are disabled.
In your case, it fails because exceptions are disabled and this triggers the suppression of ENTT_NOEXCEPT (see config.h). This is an error in my opinion. That is, ENTT_NOEXCEPT shouldn't depend on exceptions (actually, it shouldn't even exist probably but let's ignore this aspect for the time being).

If ENTT_NOEXCEPT is allowed to exist it can give the impression that you can remove noexcept, but ultimately you can't do that, because dense_map relies on this feature to make sure the nodes are always moved.

Uhm, yes and no. The context doesn't rely on this feature. It relies on the fact that it can reallocate the underlying vector of a map. This is possible in all cases when either your variables are copyable or ENTT_NOEXCEPTION isn't suppressed.

What I don't understand is why this has any effect when compiling without exceptions. If exceptions are disabled exceptions won't trigger, so shouldn't it always choose move? Why is it choosing copy?

See above. This happens because ENTT_NOEXCEPT is cleared when you disable exceptions (and I agree that this is likely an error). As a consequence, the move constructor of any (which is ENTT_NOEXCEPT) is now a non-noexcept one. Therefore, is_nothrow_move_constructible_v<any> is false and since any also is copyable, the std::vector decides to copy its elements rather than moving them.
A very tricky issue that I wouldn't have noticed easily if it wasn't thanks to you. Really a good catch.

Alright, thanks a lot for the fix and the nice explanation.

One last thing about the commited fix... are you sure it's ok to still give the users the opportunity to clear ENTT_NOEXCEPT? If someone does the bug will come back.


doc/readers/doctrine: add ::class example

for me the given 'YourNamespace:Employee' did not work, but class name did.

Fixed it in b74226aa19f6fb0dd74a53233c361b241cdbd00c


Custom Local Collections invalid

Following the steps in the document, I expected to use the Icon component to display a custom SVG Icon, but it failed.
https://stackblitz.com/edit/nuxt-icon-playground-t9ezmw?file=app.vue

Okay, this feature has only been supported since version 1.0.0, so I misunderstood it.

But it would be nice to have a warning of some sort in the documentation..

If it wasn't for this issue, I would have been trying for over an hour to figure out why my custom icons don't appear.

I have forced the install of v1.0.0-beta.4, in dev it works~ish, the cutom icons don't necessarily load the first time and the refresh of the page after having rendered the element where the icon is tends to make them show.
I haven't tested with beta.5 or with build or generate.

@LordLumineer I abandoned this library in favor of my own implementation.
Folder assets/icons
File iconSet.ts
export const IconSet = {
  'arrow/chevron-down': defineAsyncComponent(() => import('./arrow/chevron-down.svg')),
  'arrow/chevron-up': defineAsyncComponent(() => import('./arrow/chevron-up.svg')),
} as Record<string, ReturnType<typeof defineAsyncComponent>>;

Folder components
File BaseIcon.vue
<script lang="ts" setup>
import { IconSet } from '@icons/IconSet';

interface BaseIconProps {
  name: keyof typeof IconSet;
  width?: string | number;
  height?: string | number;
}

const props = withDefaults(defineProps<BaseIconProps>(), {
  name: '',
  width: 'auto',
  height: '',
});

const styles = computed(() => ({
  width: props.width
     ? props.width + (!Number.isNaN(Number(props.width)) ? 'px' : '')
      : undefined,
  height: props.height
    ? props.height + (!Number.isNaN(Number(props.height)) ? 'px' : '')
    : undefined,
}));
</script>

<template>
  <suspense>
    <component :is="IconSet[name] || 'span'" :style="styles" v-bind="$attrs" />
  </suspense>
</template>

it works faster and the code is cleaner

@LordLumineer我放弃了这个库而选择我自己的实现。
文件夹 assets/icons 文件 iconSet.ts
export const IconSet = {
  'arrow/chevron-down': defineAsyncComponent(() => import('./arrow/chevron-down.svg')),
  'arrow/chevron-up': defineAsyncComponent(() => import('./arrow/chevron-up.svg')),
} as Record<string, ReturnType<typeof defineAsyncComponent>>;

文件夹组件 文件 BaseIcon.vue
<script lang="ts" setup>
import { IconSet } from '@icons/IconSet';

interface BaseIconProps {
  name: keyof typeof IconSet;
  width?: string | number;
  height?: string | number;
}

const props = withDefaults(defineProps<BaseIconProps>(), {
  name: '',
  width: 'auto',
  height: '',
});

const styles = computed(() => ({
  width: props.width
     ? props.width + (!Number.isNaN(Number(props.width)) ? 'px' : '')
      : undefined,
  height: props.height
    ? props.height + (!Number.isNaN(Number(props.height)) ? 'px' : '')
    : undefined,
}));
</script>

<template>
  <suspense>
    <component :is="IconSet[name] || 'span'" :style="styles" v-bind="$attrs" />
  </suspense>
</template>

它运行得更快，代码更干净
重要！您需要安装****vite-svg-loader软件包****

Thank you, I think I know what to do now


Incorrect completion for clippy lints in #[allow()]

If I type #[allow(clippy::susp|)] and then hit one of the completion options, the tool lint is completed as clippy::clippy::suspicious_operation_groupings, that is, with a double clippy:: prefix.
Current RA nightly and stable rustc.

Also, not sure if this is worth filing a separate issue for, but recovery from early-stage parse errors seems to work well for an unfinished #[allow, but as soon as I type an opening parenthesis (#[allow(), this leads to the rest of the file being broken. It feels like it ought to be possible to be better there.

Duplicate of https://github.com/rust-analyzer/rust-analyzer/issues/7144

I dont think we can improve recovery there because as soon as the ( is typed the parser expects a token tree, which means it spans the entire file I imagine?

Ah, I guess that makes sense about the token trees, though I still wonder if it could be slightly smarter. For example, maybe it should recover if the first token is a keyword -- particularly a keyword that introduces an item (pub, struct, enum), or maybe a block (if, for).

I don't think that's possible. The parser can't know that the attribute is unclosed and that the keyword is supposed to start an Item as it doesn't do that kinda lookahead, it might just be part of the token tree. @matklad probably knows better if we can do some tricks here though.

I agree that it might be part of the token tree, but the point is that it seems unlikely -- as I understand it, recovery is all about heuristics.

recovery became a mess after we added support for macros, see https://github.com/rust-analyzer/rust-analyzer/pull/4680#issuecomment-641346748 for the current state. Straightening this up is on the mid-term roadmap for me personally.


chore: osmoutils, twap, downtime-detector, ibc-hooks cherry-pick to v14

Closes: #XXX
What is the purpose of the change
After internal discussions, we decided on launching v14 sooner with less features. This PR cherry picks the following features from main:

Geometric TWAP
Osmoutils refactor
Downtime Detector
IBC Hooks

Then, about four PRs were done manually by myself to fix issues that occurred due to the cherry picks.

It makes sense that the contract tests would fail if the bytecode was fetched from main. The Cargo.lock file didn't exist in this branch since https://github.com/osmosis-labs/osmosis/pull/3576 was not inclided. I've cherry picked it.

PR LGTM, great job on this!


ARROW-5949: [Rust] Add Dictionary datatype

This change adds the dictionary datatype to Rust Arrow.

Let me know if you have any issues with this. I'll add the IPC serialisation next as it is pressing for me.
Well done, @andygrove and @nevi-me for the work so far.

Could you use git rebase instead of git merge?

For tips on using git rebase with this project ... https://andygrove.io/apache_arrow_git_tips/

I don't know how this patch acquired all these changes already merged to master but a committer is probably going to need to help fix the branch

I'm happy to help with this. I probably don't have time until the weekend though.

I took care of it

@andy-thomason please be sure to fetch your remote and then do git reset --hard origin/master (assuming here that "origin" is andy-thomason/arrow). If you do git merge things will be messed up again

Sorry about that. I'm a bit of a noob... I'll open a new PR for the extra bits if that works for you.

Thanks for the help. It is a lot calmer, now.

All ready to go if everyone is happy.

@andy-thomason in the future you can always edit the branch used for the PR (I think) -- i.e. to change from "master" (which will create problems for you) to something else. Let us know if you need help.


Error installing barryvdh/laravel-ide-helper. Requires require illuminate/console ^8

Versions:

ide-helper Version: 2.9
Laravel Version: 7.30.1
PHP Version: 7.4.16

Question:
Could someone help regarding the foollowing issue that I'm getting when I execute composer require --dev barryvdh/laravel-ide-helper
  Problem 1
    - barryvdh/laravel-ide-helper[dev-master, v2.9.0, ..., v2.9.2] require illuminate/console ^8 -> found illuminate/console[v8.0.0, ..., 8.x-dev] but these were not loaded, likely because it conflicts with another require.
    - barryvdh/laravel-ide-helper 2.9.x-dev is an alias of barryvdh/laravel-ide-helper dev-master and thus requires it to be installed too.
    - Root composer.json requires barryvdh/laravel-ide-helper ^2.9 -> satisfiable by barryvdh/laravel-ide-helper[v2.9.0, v2.9.1, v2.9.2, 2.9.x-dev (alias of dev-master)].

This is my current composer.json
{
    "name": "laravel/laravel",
    "type": "project",
    "description": "The Laravel Framework.",
    "keywords": [
        "framework",
        "laravel"
    ],
    "license": "MIT",
    "require": {
        "php": "^7.2.5",
        "berkayk/onesignal-laravel": "^1.0",
        "fideloper/proxy": "^4.2",
        "fruitcake/laravel-cors": "^1.0",
        "guzzlehttp/guzzle": "^6.3",
        "kreait/laravel-firebase": "^2.2",
        "larapack/voyager-hooks": "^1.2",
        "laravel/framework": "^7.0",
        "laravel/passport": "^9.2",
        "laravel/tinker": "^2.0",
        "laravel/ui": "^2.0",
        "maatwebsite/excel": "^3.1",
        "pbmedia/laravel-ffmpeg": "^7.0",
        "tcg/voyager": "^1.4"
    },
    "require-dev": {
        "facade/ignition": "^2.0",
        "fzaninotto/faker": "^1.9",
        "mockery/mockery": "^1.3.1",
        "nunomaduro/collision": "^4.1",
        "phpunit/phpunit": "^8.5"
    },
    "config": {
        "optimize-autoloader": true,
        "preferred-install": "dist",
        "sort-packages": true
    },
    "extra": {
        "laravel": {
            "dont-discover": []
        }
    },
    "autoload": {
        "psr-4": {
            "App\\": "app/"
        },
        "classmap": [
            "database/seeds",
            "database/factories"
        ]
    },
    "autoload-dev": {
        "psr-4": {
            "Tests\\": "tests/"
        }
    },
    "minimum-stability": "dev",
    "prefer-stable": true,
    "scripts": {
        "post-autoload-dump": [
            "Illuminate\\Foundation\\ComposerScripts::postAutoloadDump",
            "@php artisan package:discover --ansi"
        ],
        "post-root-package-install": [
            "@php -r \"file_exists('.env') || copy('.env.example', '.env');\""
        ],
        "post-create-project-cmd": [
            "@php artisan key:generate --ansi"
        ]
    },
    "repositories": {
        "hooks": {
            "type": "composer",
            "url": "https://larapack.io"
        }
    }
}

Laravel Version: 7.30.1

ide-helper requires Laravel 8+ 🤷♀
You can check out https://packagist.org/packages/barryvdh/laravel-ide-helper#v2.8.2 to find a matching version for older Laravel versions

Thank you a lot @mfn

Closing due to lack of feedback from OP

If you are using Laravel 7 or anything under Laravel 8 you need to specify an older version of the code.
composer require --dev barryvdh/laravel-ide-helper 2.8
The command above helped me implement this without error.

If you are using Laravel 7 or anything under Laravel 8 you need to specify an older version of the code.
composer require --dev barryvdh/laravel-ide-helper 2.8
The command above helped me implement this without error.

It worked for Laravel 7.30.5 by typing composer require --dev barryvdh/laravel-ide-helper 2.8 --ignore-platform-reqs

If you are using Laravel 7 or anything under Laravel 8 you need to specify an older version of the code.
composer require --dev barryvdh/laravel-ide-helper 2.8
The command above helped me implement this without error.

Thx its work fine for me

If you are using Laravel 7 or anything under Laravel 8 you need to specify an older version of the code.
composer require --dev barryvdh/laravel-ide-helper 2.8
The command above helped me implement this without error.

Thx its work fine for me

If you are using Laravel 7 or anything under Laravel 8 you need to specify an older version of the code.
composer require --dev barryvdh/laravel-ide-helper 2.8
The command above helped me implement this without error.

Works fine on Laravel 5.8 with PHP 7.4. Thanks a lot!


Which of SAPA vs SAPAExp should be used to obtain best scores from your paper?

SAPA-G, by using SAPAExp with q_mode='gate'


No ligand atoms parsed

Which is the correct format for a ligand?
I have this pdb. The ligand comes from a pose created by Diffdock on the included protein
https://drive.google.com/file/d/1yKB-oQVL7990IYedfybzCdcUGR6QbID0/view?usp=sharing

But In my computer and in https://colab.research.google.com/github/ullahsamee/ligandMPNN_Colab/blob/main/LigandMPNN_Colab.ipynb
I get No ligand atoms parsed
What is wrong?
Thanks for your help!

Solved. I had an ENDMDL line between the protein and the ligand.


[WIP] Bump eirini and eirinifs submodules

and use golang 1.12 to get support for strings.ReplaceAll

Don't merge yet, I had to upload the golang blob to the correct (non-tmp) bucket. Fixing.

Fixed. Ready for review.


[Proposal] Switch to npm 5 from yarn

[x] I'm sure this issue is not a duplicate?

npm 5 now includes the goodies that yarn offers.
Consider switching to it? Discuss.

Note here that we can keep yarn and add support for npm 5 (which adds a package-lock.json file similar to that of yarn).
The other option is to drop yarn but add a constraint that the user needs npm 5+ in order to manage the packages.

I wanted to suggest that, too... So, I think it's the right idea! 🗡

As far as I know npm5 is shipped with node 8 and our CI is running on node 6. I have no problem keeping the new npm lock file. I'm already using the latest npm version so... But If we are willing to keep our CI node versions aligned with vscode node version then I guess we'll have to keep yarn for the moment.

npm can be updated independently. I see the option to keep them both as preferable (this way we can keep everyone happy).


Added Detail for Recording Usage against Job Planning Lines

This relates to paragraph headed "To record usage for a job planning line of type Budget".
In Step 5. you can add that the user can optionally select multiple Planning lines for transferring to the Job Journal at the same time.
This could be an alternative for Step 10.

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

ID: eeab2416-7c83-f086-e9e2-bb2b094ed6d4
Version Independent ID: 98a3c38a-990c-4cf0-9a7e-8eda2248cc7d
Content: Record Billable and Budgeted Usage of Job Resources - Business Central
Content Source: business-central/projects-how-record-job-usage.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Thank you for your feedback! This will be fixed in the next doc update.

Thank you for your feedback! This was fixed in today's doc update as well as your other reported issue.


Constant names should comply with a naming convention

This pull request is focused on resolving occurrences of Sonar rule
squid:S00115 Constant names should comply with a naming convention
You can find more information about the issue here:
https://dev.eclipse.org/sonar/rules/show/
Please let me know if you have any questions.
Zeeshan Asghar

Because of conflict, a separate commit is done based on this PR: b8b87b3
Thanks for the PR! ☕


Replace existing docker networking code and integrate with libnetwork

This PR attempts to integrate libnetwork with three main goals:

Modularize docker networking code to a separate package
Introduce the first implementation of Container Network Model and make docker purely rely on that
Allow more kinds of network drivers to integrated with ease and possibly no change to docker core at all

Please refer to following libnetwork docs for more info:

README
ROADMAP
Design

All dead code has been removed.

you need to rebase ;)

@crosbymichael done :-)

started test on lxc ;) feeling brave

@mrjana Also, you should separate commit message body from title with empty line

very nice at first glance, thanks for vendoring the code in a separated commit, much better to review :zap:

Test failure.

@cpuguy83 thanks. am looking into it.

ping @crosbymichael @jfrazelle @LK4D4 @calavera @cpuguy83

Please note that the code not needed in docker any more like the networkdriver and portmapper haven't been removed yet. It can be removed in subsequent PRs once this one gets merged.

So is that code dead or is it still being called.

as far as design goes I like it, it is clean and simple.

Yes, don't leave dead code around, it will just be confusing for anyone changing code after you.  Don't comment things out either, just delete, everything is in git so we can always see what it was before.

@crosbymichael Sure, let me cleanup dead code.
@jfrazelle I ran lxc integration tests in my dev machine and it had only 7 failures(which was the same on stock 1.7.0-dev in my machine). Does lxc test fail intermittently?

@jfrazelle @crosbymichael @LK4D4
Updated the PR with all the comments taken care

With last commit it is definetely better.

Just ran some manual smoke tests on this PR and everything seems good ... :+1:
Environment:
$ make BIND_DIR=bundles binary
$ docker-machine create libnetwork
$ scp -i ~/.docker/machine/machines/libnetwork/id_rsa bundles/1.7.0-dev/binary/docker-1.7.0-dev docker@`docker-machine ip`:docker-1.7.0-dev
$ docker-machine ssh
docker@libnetwork:~$ sudo mv docker-1.7.0-dev /usr/local/bin
docker@libnetwork:~$ sudo mv /usr/local/bin/docker /usr/local/bin/docker.old
docker@libnetwork:~$ sudo ln -s /usr/local/bin/docker-1.7.0-dev /usr/local/bin/docker
docker@libnetwork:~$ sudo /etc/init.d/docker restart
docker@libnetwork:~$ exit

Tests:
# Create 50 containers in bridge mode
for i in $(seq 1 50); do docker run -itd --name foo$i busybox ; done
# Ping them all from foo1
for i in $(seq 1 50); do docker exec foo1 ping -c4 $(docker inspect --format "{{.NetworkSettings.IPAddress}}" foo$i); done
# Check they can all reach <IP_ADDRESS>
for i in $(seq 1 50); do docker exec foo$i ping -c2 <IP_ADDRESS>; done
# Check apt-get
docker run -it --rm debian;jessie apt-get update
# Check port mapping
docker run -itd --name nginx -P nginx
port=$(docker port nginx | grep 80/tcp | sed s/.*://)
curl http://`docker-machine ip`:$port
# Check links
docker run -it --rm --link nginx:nginx debian:jessie
cat /etc/hosts | grep nginx
env | grep NGINX
exit
# Check --net=none
docker run -it --rm --net=none debian:jessie
ip addr show dev eth0 || echo "YAY"
# Check --net=host
docker run -itd --net=host --name=host debian:jessie
docker inspect --format {{.NetworkSettings.SandboxKey}} host | grep /var/run/netns/default
docker attach host
ifconfig
exit
# Check --net=container
docker run -it --rm --name=foo51 --net=container:foo50 busybox ifconfig | grep $(docker inspect --format {{.NetworkSettings.IPAddress}} foo50)
# Check Veth Cleanup
for x in $(docker ps -a -q) ; do docker stop $x; done
ip addr | grep veth || echo "YAY"

@mrjana I see the endpoint lifecycle isn't decoupled from the container yet.
Is that following in a subsequent PR?
## FAILS
# Check container IP persists on stop/start
foo1_ip=$(docker inspect --format {{.NetworkSettings.IPAddress}} foo1)
docker stop foo1
docker start foo1
docker exec foo1 ifconfig | grep $foo1_ip

@dave-tucker Thanks for your help.

@mrjana I see the endpoint lifecycle isn't decoupled from the container yet. Is that following in a subsequent PR?

Yes. This is expected for now. We tried our best to keep the existing behavior in place with this PR. The endpoint life-cycle decoupling will follow in subsequent PR.

@dave-tucker To clarify a bit more, the endpoint lifecycle for implicitly created endpoints like the ones created when we invoke docker run --net:<net_name> without any endpoint name, will actually coincide with the container lifecycle itself. It is the endpoints which are explicitly created by the user which will have it's own lifecycle independent of the container.
Nevertheless, there is a strong need still to provide the same network settings like IP addresses etc. to a restarting container and we will achieve this using an endpoint caching logic inside libnetwork in such a way that we will retain a deleted endpoint in a cache inside libnetwork and not notify the driver and if a container comes back and tries creating the same endpoint (read same name) then we will try to give it back from the cache. Ofcourse, the cache ages out after some time so that we don't hold on to network resources indefinitely.

@mrjana @mavenugo thanks for clarifying!

Thanks @mavenugo like the design

we will achieve this using an endpoint caching logic inside libnetwork

This may not play so well with remote drivers, which in general need to know explicitly when resources are released (and may wish to do their own caching / expiration).

@mrjana need rebase.

@LK4D4 rebase done.

@mrjana Seems like you removed fsnotify usage. Can you remove it from vendor script and directory?

Looks like daemon/networkdriver/bridge/driver.go still here.

@LK4D4 @calavera Updated the PR with the comments taken care

Per @LK4D4 comment re: fsnotify, why is the /etc/resolv.conf updating feature removed?  I notice that tests testing this functionality were left in the test suite but gutted from actually testing the feature.   That seems wrong--we need tests removed and docs updated if we are all agreed on removing an actual function from the current network stack's capabilities.

Yes, @estesp has good questions.

@estesp I was trying to get your comments on this in IRC but couldn't find you. Anyway the test case in integration-cli was testing the design while we should be testing the expected functionality.
The expected function as I understand(correct me if I am wrong) is if the container got it's /etc/resolv.conf from the host(and not from daemon or container config option) and if the host entries changed then in the next restart of the container update the container's /etc/resolv.conf to reflect the host entries. The test case in integration-cli is still testing that functionality. But we changed the design of how this is achieved so if the test case was testing the design then it is not applicable any more. That's why removed the bits which were testing the old design which was testing if the container file got updated for  a change in host /etc/resolv.conf while the container is not running.

Ping @jfrazelle @icecrime @duglin @estesp @unclejack @vishh @vieux @aluzzardi @dmp42 @dmcgowan @ehazlett @stevvooe @endophage
Would really appreciate help reviewing this very important PR that would lay the foundation for future docker networking improvements.
If you're not familiar with networking, just make sure the refactoring makes sense.
If you feel familiar with networking, please be critical about what might break given this refactor.
Thanks in advance for your help

@mrjana I'm "estesp" on IRC in docker, docker-{dev,maintainers,distribution}.  I can join -network if discussions should happen there, however, this afternoon I'm offline most of the rest of the day.
To your point about the test--it was actually testing function and edge cases around the updater, using the knowledge of the design as an easy validation method.  If the design changes, then maybe some other method for validation is required, but I don't think any of the tests are invalid unless the function has changed.  The variations were around - a) the user changing the container's resolv.conf manaully--which means no auto-updates should happen, b) running containers should not be updated, and c) the act of updating should retain the default filtering out of localhost resolvers.
If any of those are no longer valid, then we need to discuss why the design changed and what impact to docs or other expectations of users are.

Made some general code comments, haven't tested thoroughly yet.

@estesp Fair point on testing the variations. The earlier tests were testing this assuming certain design. I will try reworking the test code to include testing these variations purely in a black box way.

I'm not completely familiar with all the features that we're modifying here, but code LGTM over all. I agree that we should try to replace the removed tests with better tests :smile:

Yeah, actually I'm sure we need consensus between @mrjana and @estesp about resolv.conf feature.

@ashahab-altiscale Would you mind to look on lxc trick about setns.

Sure. Btw, is there a design proposal?
On May 15, 2015 3:00 PM, "Alexander Morozov"<EMAIL_ADDRESS>wrote:

@ashahab-altiscale https://github.com/ashahab-altiscale Would you mind
to look on lxc trick about setns.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/13060#issuecomment-102534535.

@ashahab-altiscale For libnetwork design please take a look at the link in the PR description above and there is a link to the design.
But fundamentally what happens is libnetwork manages all the network resources for the containers with the help of a set of drivers. It also manages the net namespace in which these network resources are going to be present. When it has plumbed all the network resources into the netns it will just pass along the netns to the container's exec driver to use when it sets up the execution environment. Both libcontainer and lxc exec drivers use this to setup the execution enviroment.
Let me know if you need any clarifications.

@stevvooe Thanks for great review. We should invite you more often :)

@jfrazelle Can you run an lxc-driver jenkins build for this?

@ashahab-altiscale The tests were run on LXC.

@ashahab-altiscale You can see, it falis on jenkins. There is red cross under all comments.

Yup, looking.

@cpuguy83 @LK4D4 @calavera @stevvooe Update new diffs with almost all of the comments taken care of.
@estesp I've reinstated all the test case variations for resolvconf update but now just considering the expected function alone. Please eyeball the test case and see if everything is fine.
@ashahab-altiscale I've re-implemented lxc integration in a different way so please take a look. And lxc is passing in jenkins with this implementation.
@ashahab-altiscale @LK4D4 the reason why the test-integration-cli for lxc was working but not jenkins was that jenkins was not running in a container while the locally run test suite was running inside a dind container.  libnetwork was setting up netns bind mounts in /var/run/netns and in the host they are mounted as MS_SHARED while inside a dind container they are not mounted as MS_SHARED and with the earlier design when I bind mounted /var/run/netns/<ns>, lxc umounted it which propogated to the host(because of MS_SHARED) and so setting into it inside the lxc init code started failing. Earlier I used to setup netns bind mounts inside /var/lib/docker/netns and that's why all my manual tests worked as expected and because of a design review comment I rightfully switched to /var/run/netns and didn't test lxc manually and only relied on the integration test suite and it was passing. So the mystery is resolved.

Yup, and /var/run/netns was pretty bad place anyway, because of iproute2.

@LKD4 The path is specified in libnetwork. May be it's more appropriate to have it in /var/run/docker but the reason I chose /var/run/netns is that this gives us a way to use iproute2 to all the libnetwork created namespace as a debugging tool and we don't need to create another tool.

@mrjana Yes, but iproute2 is separate tool and it can be used for other purposes. nsenter tool can do pretty the same, but allow you specify file anywhere.
Okay, so where is namespaces now? And could you point to code line where it is specified?

@LK4D4 It's still at /var/run/netns. The code is here https://github.com/docker/libnetwork/blob/master/sandbox/namespace_linux.go#L15

I wonder why it isn't configurable. And I'm against placing it there.

@LK4D4 The whole point is this is part of the sandbox api so that the caller doesn't need to worry about  the OS specifics of how the sandbox is created and maintained to make the caller OS-agnostic and only return a sandbox key to uniquely identify the sandbox. So for linux the sandbox key happens to be the namespace path itself and for windows it would be a UUID string(from an early conversation with windows folks that's how they identify their network "compartments").

@mrjana Portability is hardly a reason for hardcoding behavior for all platforms in library. But okay, it's not a docker problem.
But I still don't like /var/run/netns, it is other tool namespace and we shouldn't spoil it with our namespaces.

@LK4D4 ok since as you suggested we can use nsenter for debugging purposes we should be able to use /var/run/docker. I will make that change.

--userland-proxy=false doesn't working, docker still spawning docker-proxy processes.

As far as I can tell, we have:

Blockers

[ ] Zombie process
[ ] --userland-proxy not backported to libnetwork

Things that could be improved

The /var/run/netnsstorage path should be left to the caller (and changed by Docker)
We're not super comfortable with the Setns/defer code paths, but @crosbymichael we just go with the changes and stress test the daemon

performance and runtime stuff:
LGTM

I have been going through validating existing default networking flags in various mix and match combinations. I wrote it up in the form of a tutorial to keep it from looking like a jumbled mess of output. So far I haven't hit anything with the following OPTS:
-bridge=
--bip=
--fixed-cidr
--ip=
--ipv6=
--mtu=
--dns=
--dns-search=
--hostname=
--net=
--mac-address=

Posting it to add extra data and maybe save time in reviews for folks it is a WIP so it will get some more updates tonight and tomorrow with more scenarios:
libnetwork parity testing

Thanks @nerdalert. This is helpful in addition to the automated integration tests & stress tests (thanks to @crosbymichael).

@LK4D4 @icecrime @crosbymichael Pushed new diffs with PR comments taken care of. More importantly fixed the zombie issue and backported making userland-proxy optional. I ran the whole test-integration-cli with --userland-proxy=false and is passing. Also made sure that proxy is not running when --userland-proxy is false.

need rebase :/

@mrjana I'm not sure, but seems like your commits in wrong order now.

I found panic:
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x30 pc=0x75206f]

goroutine 107 [running]:
github.com/docker/libnetwork.(*endpoint).Leave(0xc2082bcf30, 0xc2084753c0, 0x40, 0x0, 0x0, 0x0, 0x0, 0x0)
        /go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/endpoint.go:357 +0x2cf
github.com/docker/docker/daemon.(*Container).ReleaseNetwork(0xc208310000)
        /go/src/github.com/docker/docker/daemon/container.go:961 +0x45f
github.com/docker/docker/daemon.(*Container).cleanup(0xc208310000)
        /go/src/github.com/docker/docker/daemon/container.go:975 +0x36
github.com/docker/docker/daemon.(*containerMonitor).Close(0xc2081557a0, 0x0, 0x0)
        /go/src/github.com/docker/docker/daemon/monitor.go:86 +0x4a
github.com/docker/docker/daemon.func·031()
        /go/src/github.com/docker/docker/daemon/monitor.go:117 +0xbb
github.com/docker/docker/daemon.(*containerMonitor).Start(0xc2081557a0, 0x0, 0x0)
        /go/src/github.com/docker/docker/daemon/monitor.go:180 +0x7af
github.com/docker/docker/daemon.*containerMonitor.Start·fm(0x0, 0x0)
        /go/src/github.com/docker/docker/daemon/container.go:1584 +0x39
github.com/docker/docker/pkg/promise.func·001()
        /go/src/github.com/docker/docker/pkg/promise/promise.go:8 +0x2f
created by github.com/docker/docker/pkg/promise.Go
        /go/src/github.com/docker/docker/pkg/promise/promise.go:9 +0xfb

I created two containers with --net=host and then leave from both. Also you probably want to write test for this situation :)

After daemon crash I can't start old container with:
Error response from daemon: Cannot start container c6732738abba: open /var/run/docker/netns/c6732738abba: permission denied
Error: failed to start containers: [c6732738abba]

In logs:
ERRO[0023] leaving endpoint failed: no container attached to the endpoint

Last error probably was here forever, but now it became little more severe. I think we need to call container.cleanup() in daemon.register instead of just container.Unmount

On the good side of things, tests are green with --userland-proxy=false.

@mrjana thanks for the resolv.conf test update--it looks good.  Based on that, it appears that any kind of live watching of the host resolver config is now replaced from an implementation perspective by DNS setup step during container start?  Given we had no way to do live update of running containers, this seems fully reasonable and at functional parity with today's implementation.

@estesp Yes, that was the only design change which functionally doesn't change anything.

@LK4D4 Thanks for catching the panic issue. Pushed a new set of diffs with the fix for that a test case for that in libnetwork as well. Also rebased.
/cc @icecrime @crosbymichael

Error response from daemon: Cannot start container c6732738abba: open /var/run/docker/netns/c6732738abba: permission denied
Error: failed to start containers: [c6732738abba]

still here.
And I don't see that you added some code to fix it.
I reproduced like
docker run -d busybox sleep 1000
kill docker with 4 ctrl-C
start docker
docker start old container

also somehow you need rebase again :/

@LK4D4 I pushed new diffs which attempts to fix the issue that you are having. BTW, I was not able to reproduce this issue at all in my local machine. Probably something to do with the version of kernel you are running.
In any case the way I've fixed it is to to remove the namespace path file if it exists during a new join for an old container and recreate the namespace for the container from scratch. Please test it in your machine and see if it works ok.
This is rebased again.

testing now on 4.0.4
On Tue, May 19, 2015 at 5:03 PM, Jana Radhakrishnan <
<EMAIL_ADDRESS>wrote:

@LK4D4 https://github.com/LK4D4 I pushed new diffs which attempts to
fix the issue that you are having. BTW, I was not able to reproduce this
issue at all in my local machine. Probably something to do with the version
of kernel you are running.
In any case the way I've fixed it is to to remove the namespace path file
if it exists during a new join for an old container and recreate the
namespace for the container from scratch. Please test it in your machine
and see if it works ok.
This is rebased again.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/13060#issuecomment-103701507.

Thanks @jfrazelle

LGTM

LGTM

Thanks @mrjana @icecrime @LK4D4 @crosbymichael @jfrazelle @estesp @ashahab-altiscale @stevvooe @cpuguy83 for a good review & merge.
:tada: :balloon: :fireworks:

Finally! :+1:

Is there a rough estimate of when this extension may, if approved,make it into the stable branch? 6 months, 12 months, etc.
This perfectly answers some scenarios I have but I need to decide if I implement custom work arounds and leverage third party tools or work on other components of the system until this is in place or those other work arounds need to be done.
Excellent work meanwhile. Simplified elastic-ip like routing between containers is definitely a major hole in the existing feature set.

@noizu This isn't really the right venue to ask this sort of question. I suggest that you use either the mailing list<EMAIL_ADDRESS>or IRC #docker-dev.


⚠ Slack has degraded performance

In a7ea69e, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 120 ms

Resolved: Slack performance has improved in 4b58b88.


Fix polyfils link

Coverage remained the same at 79.237% when pulling 2c2dfd6bbb720ec5068ae1e25feb93dd3523217c on abraham:patch-1 into 65da101457b061250c8a0643fec7c339f6b8d7bc on lukasoppermann:master.


Remove HTML templates for surveys endpoint

@LWest001 found an issue where the /surveys endpoint defaults to HTML rather than JSON (it requires e.g. /surveys/1.json to force the JSON version, or including a content type header). This removes the HTML templates (which currently only exist for surveys anyway – the others were already removed).

@aTegart , I'd forgotten to do that – I rebased this branch off main with the rubocop fixes and have now adjusted those rspec tests. Hopefully all good now…


Users can reset password

Pre-requisite #28
Users unable to login will receive an e-mail asking them to re-set their e-mail address.

#4 is also needed for this


[FIX] Table component: fix useTable instance types

Fix is described here: https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/react-table

Thanks! This is great. Just trying to figure out whats going on with the CI audit workflow

We're still figuring out the audit workflow when PRs coming from forks outside synthetix org. But since no changes to package.json or yarn.lock is made I'll merge this one.
Thanks again!


on: parameters rendering with some kind of emoji

See here: https://github.com/candor/sublimate/wiki/SublimateModelMiddleware
As an aside: good tool.

What is this, a vBulletin forum from the '00s? I don't think I'll ever understand the motivation behind emoji shortcodes...
Anyway, I found what appears to be a minimally-invasive workaround in appending zero-width spaces after each occurrence of a colon. See #149.
Before

After

Aside: Sublimate looks cool! I've likewise found Vapor's use of Futures to be a pain point, and look forward to giving this a try.


Remove most of repr comparison.

Removes most of repr comparison as now that we don't need to use it for most cases.

Cool. Is it Thanks to #1722 ?


[Slider] Fix overflow of thumb wrapper [WIP]

[ ] I have followed (at least) the PR section of the contributing guide.

This is a first attempt on not having the complete thumb wrapper element overflow the slider root.
I'm not super happy with the negative margins but I'm submitting the PR for discussion and to get the image diffs to see regressions.

@Pajn If this approach is accepted (@eps1lon ?), please also fix the use of the slider in the docs: https://deploy-preview-13616--material-ui.netlify.com/style/color/#color-tool

This should fix #13455 right? Can we change our demos to catch future regressions?

Sorry for the delay
This helps with #13455 but does not fix it completely as the Slider is designed to overflow with its ripple. More described here: https://github.com/mui-org/material-ui/issues/13455#issuecomment-439407244
However with this fix it overflows with 24px instead of the width of the slider + 24px which probably fixes the issue for most cases (as you usually have some padding around the slider) and is similar to the previous implementation (however I don't know the exact number of pixels it could overflow). In short, if the ripple does not overflow the parent, there should be no issue.
I think we need to build a demo like https://github.com/mui-org/material-ui/pull/12967#issuecomment-423853188, both to prevent this kind of regressions but also to clarify how it actually should behave.
That thread states that there should be no padding in the slider and that forces the slider to overflow with the radius of the handle + the radius of the ripple.

Lets discuss the diffs detected in argos.
docs-lab-slider/DisabledSlider.png
This is due to margin collapsing. I don't know if someone actually have multiple sliders on top of each other without a label like that, but anyway I feel like this is bad behavior. I will try to see if I can fix it.
I think we should also create a visual test with multiple vertical sliders as well as multiple sliders with labels to check out that strange things are not happening in those cases. It feels a bit too intense as demos though.
docs-lab-slider/CustomIconSlider.png
Because of the hidden overflow, custom icons that are larger than 24px will be cut. Is this a problem, if yes is it acceptable to describe in docs for how to style around it or with a prop for easy override? Otherwise I guess this approach isn't feasible.

@Pajn would you mind taking a look at my different solution in https://github.com/mui-org/material-ui/issues/13455#issuecomment-442056165, it avoids the extra padding, negative margin and setting a overflow, so I believe it fix the issue with fewer limitations.

@Bobgy That seems like a really nice solution. My mind is kind of wrestling with how it does not experience the same problem though. Is there a browser bug it's hitting or is that actual specified behaviour?

#13455  was closed with #14689. Looks like this won't be necessary anymore.


Update the protobuf version in setup-centos9.sh to v21.8

Updates the protobuf version in setup-centos9.sh to v21.8 and removes protobuf
installation in setup-adapters.sh.

cc: @assignUser Thanks.


Update password reset

Enhancement request
Please update https://apply.opentech.fund/account/password/reset/ with additional instructions.
Example
Forgot or want to change your password?
Please enter your email address. An email will be sent to your Primary Email address that includes a password reset link. You will have [x] days to access this reset link.

Does this look ok? I prefer not to make the days the link is active to public. That info is in the e-mail the user will receive.

@frjo This looks great. Thank you! 💥


[Icon Request] ordered and unordered list

Thank you for these quality icons!

In addition:

Strikethrough
Underline
Link

In addition:

Align left
Align center
Align right

Yes, I would love to be able to use heroicons in a custom styling of the Trix toolbar


[BUG] Sonatype Platform Browser Extension fails when CSRF protection is enabled on IQ Server

Sonatype Platform Browser Extension fails when CSRF protection is enabled on IQ Server, and you have logged into IQ Server in your Web Browser - leaving CSRF protection cookies in place.
To Reproduce
Steps to reproduce the behavior:

Ensure csrfProtection=true for your IQ Server
Attempt to use the extension
HTTP 401 errors can be seen in the extension logs with the response body Invalid cross-site request forgery token

Expected behavior
The Extension should work even when CSRF protection is enabled.
Screenshots

FYI @mitunzavery @maurycupitt

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

Chrome Web Store
GitHub release

Your semantic-release bot :package::rocket:


[improve][function] 18532: Function JavaInstanceStarter support reading configs from file

Fixes #18532

Master Issue: #18532
Motivation
Reading config options from command line is not  very flexible in a cloud-native environment. For example, in distroless containers where there is no shell access, it's hard to determine a config value from env vars.
Using file to read the configs allow a more flexible configuration pattern.
Modifications

Added JavaInstanceConfiguration extends PulsarConfiguration to support reading configs from file.
Declare all JCommander fields as non-required
Add custom required field validation and default value  logic.
Added test case to make sure the command line args is not broken.

Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change added tests and can be verified as follows:

Added JavaInstanceStarterTest to make sure using command line args does not break.

Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[X] The public API
[ ] The schema
[X] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[X] Anything that affects deployment

Documentation

[ ] doc
[X] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Codecov Report

Merging #18643 (cb89dd8) into master (b579bb8) will decrease coverage by 10.53%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #18643       +/-   ##
=============================================
- Coverage     47.51%   36.98%   -10.54%
+ Complexity    10520     1964     -8556
=============================================
  Files           698      209      -489
  Lines         68079    14402    -53677
  Branches       7280     1569     -5711
=============================================
- Hits          32351     5326    -27025
+ Misses        32151     8492    -23659
+ Partials       3577      584     -2993

Flag
Coverage Δ

unittests
36.98% <ø> (-10.54%)
:arrow_down:

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

Impacted Files
Coverage Δ

.../apache/pulsar/client/impl/BatchMessageIdImpl.java
67.50% <0.00%> (-4.73%)
:arrow_down:

...va/org/apache/pulsar/client/impl/ProducerBase.java
32.69% <0.00%> (-1.93%)
:arrow_down:

...apache/pulsar/client/impl/AutoClusterFailover.java
69.44% <0.00%> (-0.56%)
:arrow_down:

...he/pulsar/client/impl/MultiTopicsConsumerImpl.java
22.78% <0.00%> (-0.20%)
:arrow_down:

...va/org/apache/pulsar/client/impl/ProducerImpl.java
16.83% <0.00%> (-0.19%)
:arrow_down:

...va/org/apache/pulsar/client/impl/ConsumerImpl.java
15.09% <0.00%> (-0.04%)
:arrow_down:

...va/org/apache/bookkeeper/mledger/util/Futures.java

...n/java/org/apache/pulsar/compaction/Compactor.java

...pulsar/broker/service/PulsarCommandSenderImpl.java

...r/stats/prometheus/AggregatedReplicationStats.java

... and 487 more

@freeznet Hi Rui，can you mark this PR as ready-to-test ? I want to kick off the test process to see if anything goes wrong.


Fix 605: Create document entries for all content types & languages

TODO: testing, but could do with a sanity check that this approach will work for Opensearch

lgtm - tested on a small dataset and rewrote the opensearch indexer to propagate these changes to the search index. Will tidy up and open a PR soon


rendering: fix canvas fbo render with GL330

canvas lets you render to another off screen buffer and feed that back to the UI. the only module that takes advantage of this is minimaps. should be enough to verify that the maps renders correct in a default gameplay template. drops more immediate mode functions that should be relevant to this PR: https://github.com/MovingBlocks/Terasology/pull/4711
PR: https://github.com/MovingBlocks/Terasology/pull/4620

 OpenGL migrations


Could not insert a bundleNeedsRemesh for entitybecause it doesn't exist

I encountered a slightly different error from #12 , though possibly related:
thread 'main' panicked at /home/bbarker/.cargo/registry/src/index.crates.io-6f17d22bba15001f/bevy_ecs-0.12.1/src/system/commands/mod.rs:1022:13:
error[B0003]: Could not insert a bundle (of type `bevy_voxel_world::chunk::NeedsRemesh`) for entity 2275v3 because it doesn't exist in this World.
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Encountered a panic when applying buffers for system `bevy_voxel_world::voxel_world_internal::flush_voxel_write_buffer`!
Encountered a panic in system `bevy_app::main_schedule::Main::run_main`!

Hi @bbarker, thanks for reporting these issues!
I was able to recreate the issue, and I think it may have to do with system ordering in bevy_voxel_world. I've pushed a fix to the main branch, but it is hard to say for sure if the problem is fully solved, since it is so intermittent. Could you try and set your dependency version to the git repo and see if the problem seem to be fixed for you?

Thanks!
So far, I've still been able to reproduce the issue on the most recent commit to master (115bd34).

Just realised that Bevy has a try_insert command for these cases, which should fail silently if the entity does not exist. I pushed a new update to main using that. Give it another go and see if it's more stable now.


Search: URL query string with current search

The Sphinx extension supports ?rtd_search="my query" in the URL.
However, I removed this feature for now from the current implementation because it makes some things more complex at the moment. We should consider re-implementing it if we think it's useful.
Also, the same feature modifies the browser's history each time you perform a search, which is pretty annoying when you do multiple queries and then want to go back.
I don't think it's a good idea to modify the URL unless there are good reasons to do that. I'm happy to talk more about this feature, tho.

I think I'd be happy to add a 🔗 icon in the search modal that copy the original URL + ?readthedocs-search-query="<the query goes here>" into the clipboard, in case they want to share it. This will solves the problem of modifying the URL and changing the history each time a char is typed.
Once the URL with the search query is opened, the search modal will open automatically and the query will be performed, showing the results.

I'd love to see this work, so that we can link folks to a page with a search result.


No access for the GDPR FAQ page

Page:
https://docs.wso2.com/pages/viewpage.action?pageId=85392776

Edited the restrictions for [1]
[1] https://docs.wso2.com/display/IS550/GDPR+FAQ


Add x64 and arm64 iOS devices in the device selector (like for macOS)

CC https://github.com/JaneySprings/DotNet.Meteor/pull/53#issuecomment-1509299641

66a721dbe3ae15a21e7cee5f6fe678bded1b3821


Some keys do not work

On Windows:

@ typed via german keyboard layout (Alt-Gr + q)
' typed with us international with dead keys

I just tried to add a different language on Windows (Romanian, in addition to En-US).
After switching the input method to Romanian, the Windows programs would accept Romanian characters as per a Romanian keyboard, but Squeak itself did not. It displayed the special characters neither as per the Romanian keyboard nor as per the En-US keyboard. Anyway, this seems to be a Squeak bug, not a GraalSqueak bug, which behaves the same way as Squeak

For a concrete example, the character ă is displayed instead of [ when the input method is Romanian (instead of En-US). In Squeak the same keypress is displayed as ?

Thanks for looking into this. We should make sure that GraalSqueak provides the same input as the default Squeak VM, with priority for English of course. The rest should be addressed by the Squeak team.
Could you check that the most important special characters and modifier keys are handled as expected on Windows with en-US?

actually, I did find a couple of bugs in the keyboard handling code, and the issue of Squeak printing ? is likely related to fonts (the character has the correct value 259 and the string is a WideString), so perhaps the fixes will help. Then again, the bug description is extremely terse and I don't understand it...

No need to worry about the bug description, things got a lot better in the meantime. I was going to close this, because I think we fixed most issues and no one is complaining at the moment. But since I have limited access to a Windows machine, I just wanted to double check with you :)

I just pushed a commit related to this

Thanks, I cherry-picked your commit (https://github.com/hpi-swa/graalsqueak/commit/e371858fac17381e9ef0f10e1f5f2ee9800d01cd) and pushed https://github.com/hpi-swa/graalsqueak/commit/f6f6aa4da960cd97a7b52f28373e4771d6c1e12e. Closing this for now...


2 Player on same PC

Hello guys i am not that good at JS and etc.
But i cant figure out how i make an chess game where the player is switched. Would be nice if you could help me.

What do you mean by "switched"?   You want black at the bottom of the screen?   That's the "orientation" field

You could use chess-console, which is based on cm-chessboard but a fully functional chess UI with sound and everything.

chess-sonole looks good! will try that, dankeschön :)

So i tried chess console but i can't integrate it with php.
I have to make a chess game with php for a school project.
I want that you can play against each other on different pc's (the communication is through a database). Could someone please recommend me an library for that? HTML etc is not my strenght, its not fun for me :(