Code quality and docs

[ ] Task: Misspelled interface function:
RegsiterInstallSnapshotHandler -> RegisterInstallSnapshotHandler

[ ] Task (optional): Interface name change:
Log -> LogStore - just a nitpick in naming
type Log = LogStore - you can use an alias type and add deprecated to the godoc for backwards compatibility.

[ ] Task: For the Log interface, can you add more context and information to the godoc on the follow functions and how they are used: Truncate, Compact, Contains, DiscardEntries. DiscardEntries's description is okay, but having more concrete and clear explanations like: this function is used when a snapshot is taken and earlier logs are no longer needed.

[ ] Task (optional): Add optional snapshot interval variable to only ask about snapshots on intervals

Hi Joe,
I think your first three tasks are pretty reasonable. I've been meaning to improve the documentation for a while now, so I think this is a pretty good opportunity to do so. With that being said, I'm pretty busy this week, so it may take me a bit to get to it.
As for adding snapshot intervals, I'm curious what you had in mind? At present, the StateMachine interface has the NeedSnapshot function which should return true if a snapshot is necessary. This function is passed the size of the log in terms of log entries, which may be used to determine if a snapshot is needed. This function is called every time a log entry is applied to determine whether a snapshot is necessary. With most implementations, I think function should be very cheap to call.

@jmsadair Thanks, and no rush!
And yea, thinking about it now. Snapshot Intervals aren't really needed, your implementation is more flexible.


feat(igme): adding youth mortality data to walden

@Marigold  - master is failing at the same point so is this still okay to merge?

@spoonerf Yep, it's not a real error in this case, you're good to merge


fix(cstor): Remove our userspace implementation of io_getevents.

This userspace implementation was added because it saves a system call if it
would not return any new events at that point. However, to do this, it has to
cast an intentionally opaque pointer, assume the structure of a kernel object,
and read and write to it, which invokes undefined behaviour if the kernel
structure is actually different. Concretely, the structure indeed seems to be
different for a kernel I am running on arm64.
To ensure no undefined behaviour can happen, remove this optimization.
This PR is a continuation of arm64 stability fixes, see also:

https://github.com/openebs/openebs/issues/3028
https://github.com/openebs/cstor/pull/309

Checklist:

[ ] Fixes #
[X] PR Title follows the convention of  <type>(<scope>): <subject>
[x] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

zrepl is stable with the existing changes as confirmed by the user in https://github.com/openebs/openebs/issues/3028#issuecomment-631708798, hence closing this PR, If required we can revisit this PR.
Thanks @sgielen !


[REEF-128] Replace the protocol buffer use in the runtime API with POJOs

The POJOs are directly translated from protocol buffers to POJOs with Builders.
By removing the protocol buffer translation, the POJOs should be able to be
used more flexibly in the future, e.g. to resolve REEF-69.
JIRA: REEF-128

@dafrista I've done with my review: mostly adding comments. I'm happy to see the proto code that's stemmed from the old design is going away!
One thing to check. I'm not sure we have CI setup for .NET. We probably want to check this change does not break .NET part.
It'd be great to have a protocol diagram that shows interactions between components with messages (POJOs, proto or avro messages).

@bgchun @markusweimer I've answered some comments, and I am in the process of addressing the rest.
Here's one bit of naming I've run into that I need help with. Are we using Resource as a generic term for what in Yarn would be called a Container? It seems Resource is used fairly consistently (e.g. a ResourceStatusEvent maps to the status of a single Yarn Container). However, RuntimeStatusEvent uses the term Container -- is this usage a mistake and should be changed to Resource?

The naming is inconsistent. Ideally, we'd call it Evaluator or Container. Resource is indeed a term copied over from YARN, which adds confusion in our layer. That said, we shouldn't change the names in this PR and maybe instead file another one. The protocol layer of REEF is in desperate need of a re-think, as it shows a lot of historic relics from the time before Evaluator, Driver and Client had names :-)

One more over-arching comment: Most of the new classes should be annotated with @DriverSide and @RuntimeAuthor or @Private to make their use more easily discoverable. Also, it would be good to add the JCIP annotations for @ThreadSafe and @NotThreadSafe where appropriate.

@markusweimer @bgchun I'm ready for another review pass, after making the following changes:

Add comments: For the Builders I mostly added @see annotations, because I didn't want to repeat the interface comments (and risk them getting out of sync when additional changes are made).
Add annotations
Change file lists to sets
Removed a diff file had been added by mistake

This looks good to me. We should test it on all runtimes to make sure that we didn't miss anything. I'll test the YARN runtime on HDInsight. It would be good to know we didn't break YARN on linux or Mesos.

This runs fine on a Windows YARN cluster. Can someone test on Linux with YARN and Mesos?

Tests pass successfully on Linux YARN and Mesos clusters.

Excellent, I will merge this.


Implement UI for downloading expression matrices PART 1

What:

Add "Download Dataset" column
Add https://blueprintjs.com/docs/#core/components/overlay, so on "Download" button click, it triggers the empty modal. Please see if Blueprint.js allows us to import only the Overlay component, so we don't add code/components we don't use to the app weight
Add Blueprint.js to Data Portal, including its css files and other potential requirements: doc -- DONE in feature branch
Write e2e tests to assert #1 and #2 work
UPDATE: Please do https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/corpora-data-portal/607 as part of this ticket, thank you!

Part 2 ticket will implement the <DownloadDataset /> component
=============== ORIGINAL DESCRIPTION BELOW ===============
Designs are now available:

Interactive flow: https://chanzuckerberg.invisionapp.com/share/K6YKM66VU8J#/screens/429923777
Figma: https://www.figma.com/file/hy3wOqrKbunH1C3lA7CuP6/download-dataset?node-id=0%3A1

Requirements:

Implement non-resumable download and CURL command as shown in the mock
Add download column in the datasets page (current homepage)
Add modal according to the mock
Work with BE to figure out the POST endpoint interface /dataset/{id} and what the response shape looks like
Add E2E tests for the feature
Perhaps use clipboard-copy for copying CURL command

User flow:

When a user visits cellxgene.cziscience.com/, they see the datasets with a new column "Download dataset" and its "download" button
On "download" button click, a modal shows up with the dataset's information
The user can now either copy the CURL command OR click the "download" button to download the dataset

Context:

When the user clicks on the "download" button on the dataset row on the homepage, the modal opens and fetches the download link by sending a POST to /dataset/{id}, which returns the link as specified in https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/corpora-data-portal/476 (which means we might need loading state)

We then use the download link to populate the the CURL command and the download button!

Just updated the description to include https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/corpora-data-portal/607 🙏 Thanks so much for doing this, @Bento007 !!

Since the PR merged into the feature branch instead of main, I kept the status as WIP for now
Once the feature branch is merged, I'll move both #602 and #518 to "ready for staging"! Thank you!


nomenclature_table updated

Nomenclature table (src/dendrograms/nomenclature_table_CCN202002013.csv) updated and related CCN202002013_non_taxonomy_classification.tsv regenerated

@dosumis this is marking corticothalamic as ET which is technically not wrong, but its not ET in the way that is transcriptomically ET group if that makes sense? Not sure if it is alright?

@shawntanzk  - assume your concerns resolved by your own changes?  Can we merge this?

@shawntanzk - assume your concerns resolved by your own changes? Can we merge this?

This will be solved by that fix, though the file changes here looks like it still puts the entries as ET-projecting -> they should be uncurated now. @hkir-dev does this need be remade or something?

files have been updated
will switch to directly grabbing it from Allen repow


Add padding to left and right of install command

Since the install command is longer now (#1716), it hits the edge of the box it's in, which looks ugly. This commit adds 1rem of padding to the left and right of the text, which is the same as the padding above and below.
Before:

After:

LGTM.


Can i change my website licence from Creative Commons to MIT license?

NOTE: Before you start, the following should be completed.

Read Wiki to understand the usage and the correct effect of functional design.
Make sure no similar issue(including closed ones) exists.
Try to find the answer on Jekyll Forum and StackOverflow.

Description
I was wondering if it is possible to change the license of my website from Creative Commons to the MIT license. If this is possible, could you please guide me on how to go about doing it?
Thank you for your time and help!

Of course:
https://github.com/cotes2020/jekyll-theme-chirpy/blob/d97f95fca0bcd450ea50709ffba0217f7e65d339/_data/locales/en.yml#L30-L35

Interesting, it seems we have i18n. But my repo doesn't have this files.
I just fork https://github.com/cotes2020/chirpy-starter


feat: Add support for AWS Glue DataBrew

Issue #, if available: #112
Description of changes:
Added a StartJobRun step which allows to run a DataBrew job.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: db21afaffaf747ed2495f2e7d2b1226d72e82eb0
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: 1b22677a14bb2e9b19a13ced09e78ca0016b8199
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: a7e3f3ac63b47d9307fc11a376c10f06a663f2af
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: 7533be3ff3937190c482abd0d9673720236e2c82
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: a7fbe961ff450928be1ef0e3f17a88f832312dc0
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: 61850f4dac8dee886a41ba497111acd2f527bba6
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: 61850f4dac8dee886a41ba497111acd2f527bba6
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-sEHrOdk7acJc
Commit ID: 61850f4dac8dee886a41ba497111acd2f527bba6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


EarTrumpet

Control volume for each specific running application instead of master volume from volume control icon in taskbar
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
Sometimes different applications have different levels of sound that the user does not want to be played at the same audible volume. The vision is to have each application's volume be adjustable when clicking on the volume icon in the taskbar. There is a similar application out called EarTrumpet; however, it can sometimes be buggy and create duplicates of running applications in the UI.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

Duplicate of https://github.com/microsoft/PowerToys/issues/1011

Duplicate of https://github.com/microsoft/PowerToys/issues/1011


Is not JSON serialization of the session state to limiting?

When ASP.NET session is using out of process storage the binary serialization is used as far as I know. I understand that binary serialization is very picky but with JSON serialization I am concerned about preserving internal state without any public representation.
I have done a test with System.Web.Script.Serialization.JavaScriptSerializer and adding parameter less constructor was required and private only field was lost in serialization/deserialization.
Binary serializer also supports the IDeserializationCallback Interface to enable the object recover from a serialized state. I wonder whether this interface used when deserializing objects.
I our company we have a rather large (~250 forms) Webforms application which is to be transformed in coming years.  Side by side migration library could be a great help.

Yup - I have a pr (#51) that will allow you to control the serialization of a key on a much more granular level. That way, you can identify the best serialization technology you want to use.
The binary formatter used in framework is being deprecated in ASP.NET Core (https://docs.microsoft.com/en-us/dotnet/core/compatibility/core-libraries/5.0/binaryformatter-serialization-obsolete), so a different technology would be needed here.

@TPIvan You can now customize it however you want with ISessionKeySerializer. We're still providing a default json one, but this will be the extensibility point you'll want to look at. If you have types that require special knowledge to serialize/deserialize, you'll have to manage that there.


Dependency update

Needs non-alpha melinda-record-import-commons:
https://github.com/NatLibFi/melinda-record-import-commons-js/pull/44


cosmic_text Query re-export degrades autocomplete UX

When typing Query, RA autocomplete suggests text::cosmic_text::fontdb::Query

Reported by @JMS55 here.
I think we should change how we're re-exporting (probably alias the offending Query) to avoid this problem.

I propose that this is renamed to FontQuery

I don't think this really hurt UX. After typing "Que", rust-analyzer's suggestion is as follows:
.
The Query we usually want is at the first suggestion and text::cosmic_text::fontdb::Query is at the fourth.
After typing "Query":

The Query we usually want is still at the first and text::cosmic_text::fontdb::Query is at the second.

This does currently violate our "global namespace" principle. Aliasing feels like a reasonable path. But I think the cosmic_text re-export is unnecessary. It seems to exist to allow bevy_ui to pass the CosmicFontSystem resource into our various text computation apis. However why force the cosmic_text dependency in our "wrapper apis" when we can just pass in CosmicFontSystem directly instead? That would allow us to remove the reexport entirely.
Putting that changeset together now.


[docs]: optimize doc version for egressgateway

The code incorporated in the pr #729 needs to be optimized.

debug：https://github.com/spidernet-io/egressgateway/pull/760/files


Support specifying packages option for Cachito

Allow user to customize packages option for Cachito configuration.
See https://osbs.readthedocs.io/en/osbs_ocp3/users.html#remote-source-keys for details.

https://github.com/openshift/ocp-build-data-validator/pull/65/ is the schema.

/lgtm


Don't allocate DreamList._values until necessary

A continuation of https://github.com/wixoaGit/OpenDream/pull/628 would be to set DreamList._values to null whenever it's not actually storing any dreamvalues.

Why is it needed? _values is a variable of standard List. It doesn't allocate any excess memory if it's created empty. Moreover, it's considered bad practice to use null instead of empty collections.
I think it's a bad idea to change _values to null when it's empty before you see any problems while actually profiling.


control-service: make graphql sorting consistent

Тhe new filters we added are not consistent with the existing ones.
Changed the sorting from nullLast to nullFirst.
Testing Done: unit tests
Signed-off-by: Miroslav Ivanov<EMAIL_ADDRESS>

Did you merge back the integration tests as part of this commit?

Did you merge back the integration tests as part of this commit?

I merged them back by mistake. I had merged them back as part of https://github.com/vmware/versatile-data-kit/pull/634.


add / insert and remove columns from csv

I would like to read a csv (with 'columns': true), insert some new columns at a given position and remove some other columns.
I can do most of that by parsing the csv and and then mainpulating the data objects. Hower, inserting a column to a certain position seems not to be possible. Does data objects seem to be plain node-objects. The position of the attribute can not be set, nor is it supposed to be set.
How could I insert columns at a certain position?

If position is important, you need to output objets as arrays and not objects. Objects in js doesnt garanty order. For ur usecase, I would read csv data with columns: true, pipe it to stream-transform to add/remove/order columns and generated an array, and finally pipe the result to csv-stringify or anything else. example:
parser = parse options
transformer = transform (record) -> [ record.column_a, record.column_c ]
stringifier = stringify options
fs.createReadStream(path)
.pipe parser
.pipe transformer
.pipe stringifier
.pipe process.stdout

This will not solve my use case elegantly, because:
a) I would need to know the names and order of the columns when initiating the stringifier (I want the header to present also in the output file)
b) I would need to know the names and order of the columns for the transform function (I know the names, those are the keys of the data object, but the order is unkown because I can't relay on the object order).
The problem in my case is that I have little knowlege about the input file. The structure might be different every time I process a new file. All I know is, that in there is a column called "A" and I need to upper-case it (my actual use case is a bit more complex).
The best solution I have so far was to open the csv, read the first record to get to know the header columns. Then process it again, following your suggestion above. Is there anything more elegant (avoiding to open that file twice)?

Implemented as described above by reading the input file twice. Not very efficient, since I can't properly stop the file-read-stream, though.

could you provide a simple code illustrating your needs. I really dont understand why you couldnt be able to do it dynamically.

Here you go. What is given to me:

I have receive various csv, with various structure
I need to perform operations based on column names from the header
I don't know the position of those columns nor do I know the other columns
The output must be in the same column order

The script below does perfectly perform the task.
However, transform receives and returns and object. Because of that, the output column order is not guranteed to be equal to the input order, because objects are unorderd.
My current workaround is to process this file twice. First time to get to know the header line. Second time to process it as you suggsted above.
"use strict";
var csv = require("csv");

/* **************************************
This code reads a random CSV with a header line.

It lowerCases column named in "lowerCaseColumn"

That column may appear at any position of the input-csv

It is required, that the output structure has the same order of fields as the input structure
*/

var lowerCaseColumn = "ColumnMagic";

/* ************************************** */
function generateCSV() {
	var i,j;
	var header="";
	var content="";

	var nrOfColumns = Math.round(Math.random()*5+2);
	var nrOfRows = Math.round(Math.random()*10+2);
	var posOfMagicHeader = Math.floor(Math.random()*nrOfColumns);

	for (i=0; i < nrOfColumns; i++) {
		if(i === posOfMagicHeader)
			header += "ColumnMagic";
		else
			header += String.fromCharCode(Math.round(Math.random()*20+65));

		if(i + 1 <  nrOfColumns)
			header += ",";
	}
	for (j=0; j < nrOfRows; j++) {
		for (i=0; i < nrOfColumns; i++) {
			content += String.fromCharCode(Math.round(Math.random()*20+65));

			if(i + 1 <  nrOfColumns)
				content += ",";
		}
		content += "\n";
	}
	return header + "\n" + content;
}

var parser = csv.parse({delimiter: ",", columns: true});
parser
  .pipe(csv.transform(function(record){
  		record[lowerCaseColumn] = record[lowerCaseColumn].toLowerCase();
		return record;
  }))
  .pipe(csv.stringify ({header: true}))
  .pipe(process.stdout);

parser.write(generateCSV());

Thanks for the hint! Yes, you are right, this will basically solve it.


Remove dependency on Pandas v1

Remove dependency on Pandas version 1.  Extend capability to pandas 2.
The culprit is MTime need to make some minor updates to how over and under times are handled and checked.

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (e90d22b) 83.60% compared to head (d04a070) 83.57%.

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

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           patches     #176      +/-   ##
===========================================
- Coverage    83.60%   83.57%   -0.03%
===========================================
  Files          271      271
  Lines        19950    19963      +13
===========================================
+ Hits         16680    16685       +5
- Misses        3270     3278       +8

Flag
Coverage Δ

tests
83.57% <85.18%> (-0.03%)
:arrow_down:

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

Files
Coverage Δ

setup.py
0.00% <ø> (ø)

mt_metadata/utils/mttime.py
80.73% <85.18%> (-2.38%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Build fails for many reasons

Using:
react: 15.3.2
react-native: 0.35.0
react-native-fcm: 2.3.2
XCode 8
ios10 Emulator
CocoaPods 1.0.1
These are the errors I get:

Also, I've tried without cocoapods and can't get it to work.
Also, I've tried with and without cocoapods on a fresh RN project and it gives me compilation errors right off the bat.
Thanks!

I've done a couple things and the error has changed:

pod update
Product > Clean
Removed a virtual folder on my project folder that was named "Frameworks" and had two missing links

And then it worked!
But now I get stuck in here:

I ran into exactly this error. I figured out it was a missing GoogleService-Info.plist causing the problem. Are you sure you have that file in your project? - My 2c.

@nicholasstephan Thank you! That fixed it.
I had it in the folder but it was not referenced.


[3.x] Fix GDNative docs links

Followup to https://github.com/godotengine/godot/pull/56512

Thanks!

Cherry-picked for 3.4.3.


Fixed reset configuration response, to return the updated config value.

Description
This PR updates reset configuration, to return the updated config value in its initial response.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

Tested reset configuration from UI / API (cmk).
List config before reset =>
(cloud) 🐱 > list configurations name="storage.pool.disk.wait"
{
  "configuration": [
    {
      "category": "Storage",
      "component": "StorageManager",
      "defaultvalue": "60",
      "description": "Timeout (in secs) for the storage pool disk (of managed pool) to become available in the host. Currently only supported for PowerFlex.",
      "displaytext": "Storage pool disk wait",
      "group": "Storage",
      "isdynamic": true,
      "name": "storage.pool.disk.wait",
      "subgroup": "Storage",
      "type": "Number",
      "value": "65"
    }
  ],
  "count": 1
}

Before Fix =>
(cloud) 🐱 > reset configuration name="storage.pool.disk.wait"
{
  "configuration": {
    "category": "Storage",
    "component": "StorageManager",
    "defaultvalue": "60",
    "description": "Timeout (in secs) for the storage pool disk (of managed pool) to become available in the host. Currently only supported for PowerFlex.",
    "displaytext": "Storage pool disk wait",
    "group": "Storage",
    "isdynamic": true,
    "name": "storage.pool.disk.wait",
    "subgroup": "Storage",
    "type": "Number",
    "value": "65"
  }
}

After Fix =>
(cloud) 🐱 > reset configuration name="storage.pool.disk.wait"
{
  "configuration": {
    "category": "Storage",
    "component": "StorageManager",
    "defaultvalue": "60",
    "description": "Timeout (in secs) for the storage pool disk (of managed pool) to become available in the host. Currently only supported for PowerFlex.",
    "displaytext": "Storage pool disk wait",
    "group": "Storage",
    "isdynamic": true,
    "name": "storage.pool.disk.wait",
    "subgroup": "Storage",
    "type": "Number",
    "value": "60"
  }
}

@blueorangutan package

@sureshanaparti a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3207

@blueorangutan test

@sureshanaparti a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3931)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 32065 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6284-t3931-kvm-centos7.zip
Smoke tests completed. 92 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File


Fix link in HTTPS doc

As title

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4135   +/-   ##
=======================================
  Coverage   19.97%   19.97%
=======================================
  Files         153      153
  Lines       30494    30494
=======================================
  Hits         6091     6091
  Misses      23489    23489
  Partials      914      914

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 25b1a80...38856fe. Read the comment docs.


Redirect to the current page

Hi,
I use a custom page to load the contact form, but when the form is submitted, your extension redirect to contacts/index/post/. But I want to redirect to the current page. So that it load my custom page.
How can I edit the redirect?

The extension doesn't redirect to contacts/index/post, that's the url to the controller action that processes the contact form, our extension will redirect to whatever page is in the last_url in your session but only when the reCAPTCHA fails verification.
It sounds like there's something else going wrong, more than likely it'll be to do with how you have set up your custom contact form.


Numerical hessian

Hello, I am wondering whether PySCF supports numerical hessian calculations. It seems to me that PySCF SCF calculations direct to their corresponding analytic one when computing the hessian matrix?

@tberkel Thanks for your reply. I am interested in the numerical hessian for vibrational frequencies.

I don't think there's anything black box for numerical hessians, but you
can calculate them yourself by doing numerical first derivatives of
analytical gradients or numerical second derivatives from single-point
calculations (but these might be numerically unstable).
On Mon, Mar 11, 2024 at 1:11 PM JunHyeong1 @.***> wrote:

@tberkel https://github.com/tberkel Thanks for your reply. I am
interested in the numerical hessian for vibrational frequencies.
—
Reply to this email directly, view it on GitHub
https://github.com/pyscf/pyscf/issues/2126#issuecomment-1988988695, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGAJCY64JOCNNBMBZUQPL3YXXXTPAVCNFSM6AAAAABEQR7QG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYHE4DQNRZGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you very much for your reply :)


Add support for just complete existant commands.

Fixes #3440
Following Marc idea I create a pull request for add rehash support, so only autocomplete works only with present commands. I believe there is no regression.

If you put "Fixes #3440" on a line by itself in the summary for this Pull Request, that will create an auto-close link to the issue this addresses, so when this is merged, the issue will be closed, and readers can see links between the two.

@apjanke Done.

Looks good. Thanks!

Hi,
This technique may impact performance because "a rehash is issued on every external command completion request" (Zsh users #00532).
Consequently I believe this should be controlled by a variable indicated in the template .zshrc, like we do for the case-sensitive completion.
Also, @mhalano please do not submit PRs from your master branch — instead please create a dedicated branch for each PR. If you had done so, I could have submitted a PR to your branch to add the variable.

@ncanceill I created a branch called 'add-rehash-option'. Please submit your PR. I will merge and send again the PR.
 :)

OK, I will be patient.
In order to submit a clean PR, you need to start it off the latest upstream, which is currently ff4663a6b83b8dd205da3968de4bc458abc08b2c.
To do that, you need to go into your local Git repository, which should be configured with remote origin pointing to your fork and remote upstream pointing to robbyrussell's original repository. Then:
git fetch upstream
git checkout upstream/master
git checkout -b add-rehash-option

Then add your commit, push to origin and create a new PR from there.

@ncanceill  Thanks for be patient. :)
I created the pull request #3817 which came from my branch.
Thanks.

As stated already, this should not be enabled by default, but adding a omz config var to control this seems to be unnecessary and confusion (see my comment at https://github.com/mhalano/oh-my-zsh/pull/1/files#r43577354).

Disbled by default, of course. But need to exist a easy way to enable this
option. Is a good option for user.
2015-10-31 21:36 GMT-02:00 Daniel Hahler<EMAIL_ADDRESS>
As stated already, this should not be enabled by default, but adding a omz
config var to control this seems to be unnecessary and confusion (see my
comment at https://github.com/mhalano/oh-my-zsh/pull/1/files#r43577354).
—
Reply to this email directly or view it on GitHub
https://github.com/robbyrussell/oh-my-zsh/pull/3456#issuecomment-152780934
.

--
Marcos Alano
P: Por que este email é tão curto?
R: http://five.sentenc.es

+1 for considering the performance hit.

By popular vote I'm closing this. If you want to have this put it in your zshrc file (I have it). In the future I'll put this in the wiki FAQ, so that it is also visible.


bug fix

What type of PR is this?
/kind bug
What this PR does / why we need it:
Which issue(s) this PR fixes:
https://github.com/kmesh-net/kmesh/issues/455
https://github.com/kmesh-net/kmesh/issues/452
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/lgtm
/approve


Collaboration

Thanks for sharing, this is awesome :)
To make it easier for others to contribute it would help if there were:

A license eg Unlicense, MIT or CC-zero
If you include a similar example/demo you used in the video, even if there is no actual back-end to support it, it shows how to use the framework.

Thanks for the YT channel, long time subscriber here. Something adjacent to/for this that might interest you is keeping state in data-attributes instead of classes:
https://www.30secondsofcode.org/html/s/data-attributes-as-state/
https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/dataset

Thanks for reaching out!
The licence is added together with a very basic Readme - I'll make some slight adjustments in the next few days.
Thanks for the dataset suggestion - it might come in handy since I'll need to mark DOM elements for caching & cleanup purposes.


Support open ranges for journal production dates

Problem
Production date ranges for ongoing serial publications, eg journals, are listed with an end dated of 9999. These should be displayed as open ranges, with no value for the end date.
For example, Nature currently has the date range 1869-9999. This should be displayed as 1869-
Examples
https://wellcomecollection.org/works/awa6c6gm
https://search.wellcomelibrary.org/iii/encore/record/C__Rb1310786
https://wellcomecollection.org/works/xw9py6w5
https://search.wellcomelibrary.org/iii/encore/record/C__Rb2506296

This is done!


cannot use dynamic variables with RX IObservable objects

dynamic d = Observable.Return<int>(12345);
d.Subscribe(anything); //exception - "object" does not contain a definition for "Subscribe"

The objects returned by static Observable methods (for example Observable.Return<int>(12345); and probably also extension methods like IObservable.Select) appear to implement the IObservable interface explicitly. When they are assigned to a variable of 'dynamic' type, this has the side effect that the Subscribe method cannot be accessed. This is apparently due to a C# language feature by design (https://stackoverflow.com/questions/11625947/use-explicit-interface-implementations-with-a-dynamic-object). I hope that is the issue and not this one related to internal classes -> (http://www.jameswiseman.com/blog/2013/05/14/runtimebinderexception-does-not-contain-a-definition-for/)
*edit: After digging into the RX.net source code a bit, it looks like it is more likely the second problem related to internals not being visible to the dynamic object, and not an explicit interface implementation problem. I am not sure. Either way, I have the same problem consuming the library.
The usual "fix" is to use a reference to the specific interface type. This defeats the purpose of using dynamic, and doesn't work for my project. Due to the complexities of my type mapper, I am essentially calling IObservable<T1>.Subscribe(IObserver<T2>) where both the observer and the observable are being pulled from separate dictionaries that contain multiple types. I can guarantee that T1 == T2, but I can't instantiate an interface reference variable of either type at runtime. So I need to use dynamic variables for both objects.
I have a workaround that uses a wrapper class that just implements IObservable by wrapping what RX returns. This allows dynamic to find the Subscribe method call. I have to create one of these wrappers for each IObservable that will be used with dynamic variables. But this seems very kludgy and I would prefer if RX was dynamic-friendly up front :)
using System;
using System.Reactive.Linq;

namespace ReactiveDynamicTest
{
    public class SinkInt : IObserver<int>
    {
        public void OnNext(int value)
        {
            Console.WriteLine(value);
        }
        public void OnError(Exception error) { }
        public void OnCompleted() { }
    }
    class Program
    {
        static void Main(string[] args)
        {
            SinkInt sink = new SinkInt();

            dynamic dSource = Observable.Return<int>(12345);

            //This line throws exception "object" does not contain a definition for "Subscribe"
            dSource.Subscribe(sink);

            //These lines work, but I can't do this in my implementation
            IObservable<int> oSource = dSource as IObservable<int>;
            oSource.Subscribe(sink);

            //This is my workaround
            dynamic wrappedSource = new ObsWrapper<int>(Observable.Return<int>(12345));
            wrappedSource.Subscribe(sink);

            Console.ReadLine();
        }
    }

    class ObsWrapper<T> : IObservable<T>
    {
        IObservable<T> obj;
        public ObsWrapper(IObservable<T> input)
        {
            obj = input;
        }
        public IDisposable Subscribe(IObserver<T> observer)
        {
            return obj.Subscribe(observer);
        }
    }
}

Can you provide a simplest working example of what doesn't work today?

public class SinkInt : IObserver<int>
    {
        public void OnNext(int value)
        {
            Console.WriteLine(value);
        }
        public void OnError(Exception error) { }
        public void OnCompleted() { }
    }

class Program
    {
        static void Main(string[] args)
        {
            SinkInt sink = new SinkInt();

            dynamic dSource = Observable.Return<int>(12345);

            //This line throws exception "'object' does not contain a definition for 'Subscribe'"
            dSource.Subscribe(sink);
   }
}

Funnily enough, your example works for me! I guess it's then a matter of the framework/language version/version of Microsoft.CSharp/whatever. Please post a complete project structure (possibly separate repo).

Closing: Not reproducible on up-to-date versions of compiler/framework/library. Exact setup of OP unclear, no answer. Please reopen with a complete project structure as repro if the problem persists.

I also tried updating my Microsoft.CSharp reference to 4.5 and still getting the same error

Alright, I am unable to get your sample project to work either. It worked for me before because I ran it as a unit test in the System.Reactive project itselft. The difference is that the internals of the System.Reactive assembly are made visible to that test project. So unfortunately, since internals of Rx are invisible to your project, your example won't work without the workaround.

Should I be able to make it work if I recompile Rx.net with all internal specifiers removed? Or include the Rx.net source in my project?

Should I be able to make it work if I recompile Rx.net with all internal specifiers removed?

I would definitely not recommend this, you'll be having a hard time keeping your code up-to-date. You could link Rx-sources into your solution, you'd still have to add that InternalsVisibleTo... attribute, but I'd also not recommend that. If possible, I'd reduce the use of dynamic to a few places and go with the wrapper solution. Add a nice extension method that does the wrapping, it'll look like one of the usual operators. See if that works for you.


add option Cache-Control & Expires in OSS

Adds a option to the Cache-Control & Expires in OSS, to speed up image access using cache-control & expires in OSS Alicloud
In response to: #123

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix mistral callback failure when result contains unicode

When result of action execution contains unicode, Mistral callback fails and leads to orphaned workflow execution.

Added v2.4.1 milestone since I assume this will / should go into v2.4.1.

👍
There are also some failed Unit tests.


Add ProviderConfig auth documentation

Description of your changes

Adds the auth group annotation to the crossplane.yaml.
Signed-off-by: hasheddan<EMAIL_ADDRESS>Adds documentation and schema info for ProviderConfig auth.
Signed-off-by: hasheddan<EMAIL_ADDRESS>Updates the up version to v0.16.0.
Signed-off-by: hasheddan<EMAIL_ADDRESS>I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Built locally and inspected image contents.

Built ulucinar/provider-aws:v0.33.0-rc.1.5.g720f19e4 from this PR and tried installing it. I'm hitting the same issue I previously encountered here. I'm using the following commands to build a package:
make build.all
docker tag $(docker load -i _output/xpkg/linux_arm64/provider-aws-v0.33.0-rc.1.5.g720f19e4.xpkg -q | awk -F: '{print $3}') ulucinar/provider-aws:v0.33.0-rc.1.5.g720f19e4

I gave this PR a try to see whether the auth.yaml reference fixes the issue but looks like it does not.
My previous investigations point to the introduction of the xpkg.encode function. Previously, the package.yaml contents in the base OCI image layer were coming from the parser.PackageParser.Parse method, and looked like the following:
...

---
apiVersion: meta.pkg.crossplane.io/v1alpha1
kind: Provider
metadata:
  name: provider-gcp-cloudplatform
  annotations:
    meta.crossplane.io/maintainer: Upbound<EMAIL_ADDRESS>    meta.crossplane.io/source: github.com/upbound/provider-gcp
    meta.crossplane.io/description: |
      Upbound's official Crossplane provider to manage Google Cloud Platform (GCP)
      cloudplatform services in Kubernetes.
    meta.crossplane.io/readme: |
      Provider GCP is a Crossplane provider for [Google Cloud Platform
      (GCP)](https://cloud.google.com) developed and supported by Upbound.
      Available resources and their fields can be found in the [Upbound
      Marketplace](https://marketplace.upbound.io/providers/upbound/provider-gcp).
      If you encounter an issue please reach out on<EMAIL_ADDRESS>email
      address. This is a subpackage for the cloudplatform API group.
    friendly-name.meta.crossplane.io: Provider GCP (cloudplatform)
...

---

Please note the ... (three dots) used as a document separator. But when I used the v0.16.0 version of up accidentally while working on an experimental build pipeline for breaking provider-gcp into smaller subpackages, the contents of the package.yaml became:
---
apiVersion: meta.pkg.crossplane.io/v1alpha1
kind: Provider
metadata:
  annotations:
    friendly-name.meta.crossplane.io: Provider GCP (cloudplatform)
    meta.crossplane.io/description: |
      Upbound's official Crossplane provider to manage Google Cloud Platform (GCP)
      cloudplatform services in Kubernetes.
    meta.crossplane.io/maintainer: Upbound<EMAIL_ADDRESS>    meta.crossplane.io/readme: |
      Provider GCP is a Crossplane provider for [Google Cloud Platform
      (GCP)](https://cloud.google.com) developed and supported by Upbound.
      Available resources and their fields can be found in the [Upbound
      Marketplace](https://marketplace.upbound.io/providers/upbound/provider-gcp).
      If you encounter an issue please reach out on<EMAIL_ADDRESS>email
      address. This is a subpackage for the gcp API group.
    meta.crossplane.io/source: github.com/upbound/provider-gcp
  creationTimestamp: null
  name: provider-gcp-cloudplatform
spec:
  controller:
    image: ""
---

, and I could no longer install the package, getting errors similar to the following:
cannot apply provider package deployment: cannot create object: Deployment.apps \"provider-aws-1284847c5792\" is invalid: spec.template.spec.containers[0].image: Required value

I tried installing the package built with<EMAIL_ADDRESS>with uxp versions v1.10.1-up.1 and 1.11.3-up.1.

Another problem is the local-deploy job does not catch this. This might be related to the fact that we are populating the package cache for the Crossplane package manager but I'm not sure how this could happen if we are just extracting the built xpkg to populate the cache. I need to check the details there.

@ulucinar ah! i think we need to cut a patch release for up.
See @muvaf 's fix for this issue here. I'll tag you for additional review

xref https://github.com/upbound/homebrew-tap/pull/31

Hi @jastang, Thanks for the pointer. I've successfully verified a build of up from that PR. We are good to go after we bump the up dependency in this PR. I can also bump it once a new release is ready.

@ulucinar patch release v0.16.1 is out :)


Only run branch test coverage on the master branch

We're running Travis twice on each PR. We should only be running Travis on the master branch. The PR config will run Travis on PRs.

Coverage remained the same at 99.557% when pulling 8bf2eda21a57881074599db041106a5656729f16 on travis-on-master-branch-only into 416ca1b77ae39e66321726303aa21e94eecdf8ac on master.

Coverage remained the same at 99.557% when pulling 8bf2eda21a57881074599db041106a5656729f16 on travis-on-master-branch-only into 416ca1b77ae39e66321726303aa21e94eecdf8ac on master.

Coverage remained the same at 99.557% when pulling 8bf2eda21a57881074599db041106a5656729f16 on travis-on-master-branch-only into 416ca1b77ae39e66321726303aa21e94eecdf8ac on master.

Coverage remained the same at 99.557% when pulling 8bf2eda21a57881074599db041106a5656729f16 on travis-on-master-branch-only into 416ca1b77ae39e66321726303aa21e94eecdf8ac on master.

Coverage remained the same at 99.557% when pulling 8bf2eda21a57881074599db041106a5656729f16 on travis-on-master-branch-only into 416ca1b77ae39e66321726303aa21e94eecdf8ac on master.

Coverage remained the same at 99.557% when pulling 8bf2eda21a57881074599db041106a5656729f16 on travis-on-master-branch-only into 416ca1b77ae39e66321726303aa21e94eecdf8ac on master.

I'll revert this if isn't the right thing to do. I think it'll speed up our PR test runs.

nah this is fine. typically one only makes PRs from forks instead of branches directly on the repo, so this doesn't come up.


Fix write size for Windows

With commit 96aae2eaaca9 ("libuuu: hidreport: fix write size")I  introduced an regression for Windows hosts. The Windows HIDAPI is very strict when it comes to packet lengths: The API always expect the length specified by the device and rejects the transfer otherwise.
Can we please test this PR via CI, so I can verify that the uuu.exe does work again on Windows? If verified we can merge it there are no objections.

Thank you very much. We use github CI, which just can check build.

I meant runtime tests :-) tested the artifact now on Windows and it does work. Thanks for merging it.


feat:add persist cmd

feat:add persist command
issue #258

The implementation looks good to me, can you also add tests for the command

@srivastava-yash Thank you for the reminder. I have added unit tests and resubmitted the code. Could you please review it?
P.S.: The current implementation is not based on interfaces, so methods like get cannot be mocked.
Additionally, there are merge conflicts in the current branch that I don't have the permissions to resolve. Could you help with resolving these conflicts?

LGTM. @codeErrorSleep, I am not a maintainer of this project.
Tagging @JyotinderSingh to confirm and approve

@codeErrorSleep please rebase the PR on the latest master to resolve conflicts. Will review once done.

@JyotinderSingh I have merged the master branch.


[DOC] extended & cleaned docs on dependency handling

This PR moves the "dependencies" section out of "coding standards" (where it does not really seem to fit) and extends it with instructions and explanations of different kinds of dependencies.
Other, minor changes:

removes the duplicated "deprecation" section from "coding standards" (this is duplicated on the more recent "deprecation" page)
fixes a link to the continuous integration page, which linked to a random position on the "coding standards" page for some reason

Looks all good to me! ReadTheDocs is failing though

Looks all good to me! ReadTheDocs is failing though

That's the sporadic failure, imo nothing to do with the content. Will restart to make sure.

readthedocs runs now, @GuzalBulatova


Write new settings through temp file in majestic-settings.cgi

Use temp yaml file in tmpfs in 'for' updating cycle.
Write to /etc/majestic.yaml at the end of script for decreasing flash writes

@ZigFisher, is yaml-cli capable to write to a temp file/read from a batch file?

I think there is no such function yet.
However, @p0isk takes part in the modification of the yaml-cli and I think this will not be a problem in the future.

Done.
https://github.com/OpenIPC/microbe-web/commit/738b1237d0dc02b8f04afb999ee32f53c0d676a4


DATE OF BIRTH/ EMPLOYMENT DATE/END DATE

pa change format sang date year/month/date to month/date/year

@FNCGROUP : ready for review


My Observations Grid View changes

Right now, it's hard to see the images on My Observations Grid view.
Here are some changes to make on Home/My Observations Grid View ONLY (this should NOT apply to explore/observations/grid view)

[ ] Hide the #IDs, #Comments, and quality grade icons
[ ] Instead of two names, show only the first name, and truncate the name to 2 lines with ellipses

[ ] if my setting is Common Name (Scientific Name), I should see Common Name only
[ ] if my setting is Scientific Name (Common Name) or Scientific Name, I should see Scientific Name only
[ ] if my setting is Common Name (Scientific Name) and there is no common name for the organism, I should see Scientific Name only

@abhasinat for this item Hide the #IDs, #Comments, and quality grade icons, how do you want to handle the upload icon? I'm assuming you would still like the upload icon to show if an upload is available in GridView, followed by the upload progress and upload complete icon, and instead of fading into ID/Comments/QualityGrade icons it should fade into nothing when an upload completes. Does that sound right?

exactly!


stock-market refresh produce error stock undefined.

Hi there,
I founded an issue when i open examples page => stock-market sub page.
How to reproduce the issue?

open stock market page
refresh the page
an error will occurred, it is says "stock undefined" as shown picture below.

Thanks

Seems like you're right, thanks for letting us know!
Do you want to open up a PR with a fix?

@cloudstr21 @timdeschryver  this only happens if navigating / refreshing directly on the stock-market url, probably is a race condition with initializing state from local storage / reducer initial state and expecting that state in the StockMarketComponent ngOnInit method.

Sure. I will make it as soon as I found the solutions.

Awesome, let us know if we can help in any way possible.

this is strange, select data from ngrx/store is always undefined after page is refreshing. @timdeschryver
i tried to set timeout, but the result is the same.
this.store.pipe(select(selectStockMarket)). selector, effects, etc is fine, and this error doesn't happens in todos component.

The problem lies with the initStateFromLocalStorage meta-reducers.
Our state shape looks like the following:
{
  todos: {…},
  stockMarket: {…}
}
```

But when we init our app via `initStateFromLocalStorage`, we get the following state:

{
todos: {…},
stocks: {…} // this should be stockMarket
}

To fix this we have two choices:

rename stockMarket to stocks
rename modify STOCK_MARKET_KEY in stock-maerket.effects to be EXAMPLES.STOCKMARKET - but this still won't work because in LocalStorageService.loadInitialState it will transform this string to lowercase, making it stockmarket instead of stockMarket.

I forgot to mention, that I would vote to rename stockMarket to stocks

That's it. you are great @timdeschryver . I just fix it as your suggestions. now it run perfectly in my local. Should i re-commit it?

@timdeschryver @cloudstr21 ah god! Sorry for this, I forgot about the relation between the local storage key and the state key. It was implemented really loooong time a ago... Great catch! We can maybe think about documenting it or reworking it so that the state key is used automatically and doesn't have to be defined twice :)

@cloudstr21 Ofcourse you can re-commit it 👍
@tomastrajan No problem 😄 we could perhaps rework it as you mentioned or we could use ngrx-local-storage, which is a popular library that would do this for you. I haven't used it before tho. We can open up a new issue for this.


Swiper freezes on second slide.

Is this a bug report, a feature request, or a question?
Bug report
Have you followed the required steps before opening a bug report?

[x] I have read the guidelines regarding bug report.
[x] I have reviewed the documentation in its entirety, including the dedicated documentations :books:.
[x] I have searched for existing issues and made sure that the problem hasn't already been reported.
[x] I am using the latest plugin version.
[x] I am following the issue template closely in order to produce a useful bug report.

Have you made sure that it wasn't a React Native bug?
Yes
Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
Both iOS and Android
Is the bug reproductible in a production environment (not a debug one)?
Yes.
Environment

software
version

react
16.5.0

react-native
0.56.1

react-native-snap-carousel
3.7.3

Target
Android (6.0)

Target
iOS (9.0)

Expected Behavior
The swiper should be swipeable. If downgrading and fixing to react-native-snap-carousel 3.7.2 it works as expected.
Actual Behavior
After updating the yarn.lock file, the swiper broke in our app. We could swipe to the second slide, but then it froze. We could neither slide forward nor backward. Interactions (e.g. links) on the current slide still work.
Reproducible Demo
Unfortunately I couldn't reproduce this bug in a minimal app. Everything worked fine. I'll continue to work on a reproduction and update this issue.
Does anybody else have similar problems or a setup where the carousel breaks with the latest react-native-snap-carousel version? I'd appreciate if someone could confirm the issue or help me resolve it.

Hi @dpogoda,
The bug might be linked to PR #390. Does it work properly if you remove the added line?

@bd-arc
Thanks for the fast reply. Yes in fact deleting that line fixes the issue :)

Well, I should have tested the PR myself... Ok, I'll try and fix it as soon as I can.
Thanks for the feedback ;-)

@bd-arc  Thank you for this amazing react native component :-)

@dpogoda Should be fixed in version 3.7.4 ;-)

@benobab Damn! Could you share the configuration of your carousel?

Thanks for the reply!
Tell me if the code below is enough
                          <Carousel layout={'stack'}
                                    data={ENTRIES}
                                    renderItem={this._renderItem}
                                    sliderWidth={sliderWidth}
                                    itemWidth={itemWidth}
                                    hasParallaxImages={false}
                                    firstItem={this.state.sliderActiveSlide}
                                    inactiveSlideScale={0.95}
                                    inactiveSlideOpacity={0.7}
                                    containerCustomStyle={styles.slider}
                                    contentContainerCustomStyle={styles.sliderContentContainer}
                                    loop={true}
                                    autoplay={false}
                                    // autoplayDelay={500}
                                    // autoplayInterval={3000}
                                    onSnapToItem={(index) => this.setState({ sliderActiveSlide: index }) }
                                />

The problem doesn't appear in the default mode.

@benobab This is interesting. Have you been able to isolate the prop responsible for the bug appearance then?

I am also encountering a similar issue, however it's only appearing on Android.  The slider is unable to be moved when swiping.  Autoplay seems to work, and the elements are clickable, just swiping seems to be busted.
My configuration:
<Carousel           ref={(c) => { this._carousel = c; }}           data={this.props.items}           renderItem={this.singleItem}           sliderWidth={Dimensions.get('window').width}           itemWidth={Dimensions.get('window').width * 0.4}           inactiveSlideScale={0.7}           loop={true}           enableSnap={true}         />

@benobab @Zyphxion Can you tell me if this works properly when you use version 3.7.2?

Not to complicate things further, but I encounter a similar issue which only happens on iOS, the third swipe would freeze...

@harrison0723 Same question: does it work properly if you use version 3.7.2?

@bd-arc Nope, same issue with 3.7.2 for me

@bd-arc So sorry I've decided to switch to another library due to the Android inverted issue :-( This library seems pretty amazing though, hope this issue can be resolved soon

This has been fixed in version 3.7.5 thanks to @ifsnow. Kudos to him!

For anyone who still seems to have a similar issue, removeClippedSubviews={false} might solve your problem.

loopClonesPerSide={data.length} this prop is by default '3', setting it to length of the data will solve the stuck issue.

I was getting this issue in React JS and React Native. However, I found a strange solution to this. If you put Swiper in a component and activate it at the first startup, the problem is solved, the example is below.
` const Carousel = ({ data }: any) => {
    return (
        <>
            <Swiper autoplay={true} loop={true} showsButtons={false} showsPagination={false} style={{ height: 330 }} removeClippedSubviews={false}>
                {
                    (Media(data.filter((f: any) => { return (f.type === type && f.page === page && f.position === position) })) || []).map((d: any, x: number) => {
                        return (
                            <View key={`banner-${d.id}-list-${x}`}>
                                <TouchableOpacity onPress={() => { Linking.openURL(d.url); }}>
                                    <Image source={{ uri: `https://ik.imagekit.io/tey21emoa/banner/${d.blob}` }} style={{ resizeMode: "stretch", height: 330, width: "100%", borderRadius: 8, borderWidth: 1, borderColor: "#dee2e6" }} />
                                </TouchableOpacity>
                            </View>
                        )
                    })
                }
            </Swiper>
        </>
    )
}
<View style={{ marginTop: 5, marginBottom: 5 }}>
            <Carousel data={data} />
</View>

`


Training on QM9

Hi,
Would it be possible to provide the commands for training a model on QM9 from scratch? This is mentioned in appendix B5 when investigating the effectiveness of pre-training.
Kind regards,
Rob

Hi, I have released the fine-tuning code of QM9. For training the Transformer-M model from scratch, you could simply add 'export no_pretrain="true" ' before run the finetune_qm9.sh.


Microsoft.VisualStudioCode.Insiders.CLI_v1.89.0 cli

Microsoft.VisualStudioCode.Insiders.CLI_v1.89.0
Microsoft Reviewers: Open in CodeFlow

Hi @lszomoru,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 876.)

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-04-12 20:22:54.268 [FAIL] Installer failed security check. Url: https://vscode.download.prss.microsoft.com/dbazure/download/insider/50a6f4f200c5a33f47997eb6a8966e50cf219e21/vscode_cli_win32_x64_cli.zip Result: 0x80004005

(Automated response - build 876.)

@wingetbot run

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\Microsoft.VisualStudioCode.Insiders.CLI__DefaultSource\code-insiders.exe returned exit code: 1
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\code-insiders.exe returned exit code: 1

(Automated response - build 876.)


Add notification banner message when network goes offline

While working on ANET. When the network connection fails there is no indication that the server cannot be reached. It seems like ANET has stopped working properly. Some pages continue to be viewable, however searches and other async calls stop working. This causes frustration with ANET, while the errors are caused by network issues.

As part of the handshake with the server, the client should also be aware if there is a new version of the client software / or a modified anet.yml deployed to the server

to be addressed after implementing #561


Rate Limit on creating links, and accessing the server

Seems like a serious task. can you point out where the potential assignee might look to add the code?
Might these be two separate issues?
Is accessing the server from a specific IP address or in general?

Hey @shmuelpro, I missed your comment on this issue.
This is a serious task indeed. The purpose is to prevent spam or abuse of the API which is designed to be open to everyone.
It's currently not yet fully designed. If you care to design it (look for examples in other projects, think about the goal product-wise, etc.), I'd very much appreciate it. If you want to talk about it together, pm me on our Slack (there's a link in the README) and we'll schedule a design meeting.

Hey @SnirShechter  I am the dude from the facebook group-  pull request.
you suggested this issue for new guys and I think it needs a little more explanation before solving it

It's not for juniors, just for new guys.
The problem is that designing a solution is roughly 70% of the task. We really can't afford doing 70% of all tasks.
Designing this feature is part of the ticket.

Hi,
I'll take this - I'll comment here later this week with a design suggestion/approach


Add an action to convert Zombified Piglin's ID

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Heroku [deployement] 'NameError: name 'model' is not defined'

Works fine in local machine but when i try to deploy on heroku and upload the image for inference it gives error 500
internal server error.
Here is the heroku log

Any idea why this is happening. sorry if it is trivial question.
Steps to recreate

Clone this repo
modify requirements file (just copy yolov5 req.txt to current one)
Add procfile (web: gunicorn app:app --log-level debug)
deploy on heroku

I dont know anything about Heroku - might be more logs somewhere with error installing yolo

t
@chiragkava try latest release on master

Thanks for the help, there was problem from my side did a clean install of all library (updated version) and worked for me !


ZCLWrite should return "INVALID_VALUE" instead of success & truncation in case of invalid payloads.

Problem
When sending zclwrite command with invalid value, command returns EMBER_ZCL_STATUS_SUCCESS. Subsequent zclread of the attribute would return values such as 0, truncated string that matched requirements, and etc.
Expecting zclwrite to return EMBER_ZCL_STATUS_INVALID_VALUE if invalid value is given.

fixes #9008

Change overview

Add check to WriteSingleClusterData
Small refactor to WriteSingleClusterDataInternal for better readibility.
Relies on https://github.com/project-chip/connectedhomeip/pull/9038

Testing

Added tests to src/controller/python/test/test_scripts/base.py for above case

@erjiaqing - merge conflicts

This PR is already merged in master before we branced to TE5. I believe the change is part of TE5 already.


Add support for LOWER and SEARCH function for Airtable fields

Hi there,
When using this library to do a text keyword search using the FIND method, it will not return any results if the cases of the text do not match. For example, if a title field has "Villa" but the user searches "villa", nothing will be found.
A workaround common among many Airtable API forum answers is to use the LOWER method on both the user's search term and the text in the field. Python can obviously handle the user input, forcing it to lower using the built-in method, but I think Airtable's specific LOWER method is needed to cast the whole field to lowercase.
This seems to be something a lot of developers need. How easy / possible is it to provide support for this?
See this stack overflow question for more info / context: https://stackoverflow.com/questions/42429056/how-do-you-filter-an-airtable-record-that-contains-a-given-string-but-may-also
Many thanks,
Daniel

Should be doable to add new formula helpers to support this.
Would requiring adding SEARCH and LOWER helpers?
The result would be building a formula like this:
formula = SEARCH(LOWER(value), LOWER(FIELD('col')))

Until those helpers are added (PR welcome) , you can always build the formula manually using string formatting as shown in stackoverflow.

I'm just finishing off a major feature for a web app I'm building, but after that would really like to make a contribution to this project with the LOWER method. Hopefully I can get it done by the end of this week.

Also regarding the a new SEARCH helper, I don't believe it's a top priority as the FIND helper which has already been implemented does a very similar thing. See here: https://support.airtable.com/hc/en-us/articles/203255215-Formula-Field-Reference#text
TLDR: According to the support docs, SEARCH returns "empty" if nothing is found. FIND returns 0.

Hey @gtalarico just a heads up I have made a PR for this enhancement request here: https://github.com/gtalarico/pyairtable/pull/171
Thanks!

LOWER merged. Closing it for now since seems to be enough


Linting while reading some code

Some minor fixes along a quick survey. Commit messages should be self-explanatory and changes straightforward. Shout if this isn't the case.

@huitseeker
Thanks.
BTW, have you measured how much scalaxy makes this code fast?
Since scalaxy seems not to be well updated(it's introduces Scala compiler 2.11.2 even now), it would be better to remove its dependency if it contributes minor. What do you think about it?

Scalaxy-loops is a portable, conservative, minimal fragment of Scalaxy which is compatible with 2.10 & 2.11. It has just one feature. It proceeds through a <200 lines whitebox macro, so quite maintainable. Though there is not much to maintain whem you just have one feature. Its use here is targeted through the optimized keywords.
If it becomes incompatible / unmaintained, rolling back of the 'optimized' stanza is always available, bringing us back to the starting point of yesterday.
If you want the most actively developed project and don't need the 2.10 compatibility you can check scalaxy/streams, but its use in the project be more far ranging.


KAFKA-3522: Add public interfaces for timestamped stores

Add public interfaces for KIP-258 (key-value and window-store).

TimestampedBytesStore is omitted because it's covered via #6204
Stores is omitted, because we need to add corresponding store builder classed first
new session stores are not added yet

Additionally, adds some internal "facade" classes to allow accessing TimestampedStores with non-timestamped interfaces. This is part of the backward compatibility story of the KIP.

ReadOnlyXxxFace classed are part of main module because they will be use for IQ backward compatibility story in a follow up PR
For TopologyTestDriver we need to give read/write access and add corresponding read/write facade classes

Rebased and updated this. This PR does not add session stores any longer.
Call for review @guozhangwang @bbejeck @vvcephei @ableegoldman


[Enhancement] Add an option to open Steam Links in the Steam Client.

Is your feature request related to a problem? Please describe.
The links that are currently fetched by Metadata sources always open in a browser since they are your standard URLs. Often times, you're logged into Steam and/or maybe you want to install a Workshop item for a Steam game and to do that, you have to log into steam on the browser. Whereas, if I was able to open steam links in steam, I could easily manage all of that.
Describe the solution you'd like
Allow an option to open all Steam links with Steam (a checkbox in the settings).
Describe alternatives you've considered
A more powerful tool would be an entirely new 'Reformatting' tab for link where you can modify/morph links.
Ex: https://steamcommunity.com/app/730/discussions/ could be morphed via a regex
(.*)steamcommunity.com(.*)

into
steam://openurl/https://steamcommunity.com/$2

Where $2 ($n where n is the matched regex - $1 would be  https://) is the 2nd matched regex /app/730/discussions/.
But I think this is way out of scope for this specific ask and is probably an entirely new feature request instead.

There is a Microsoft Edge extension that does exactly what I recommended in my alternatives called Redirector.

Hmm, I don't know if it's possible for an extension to change the link once it is clicked. I can add a function to convert already existing links though, and add options to fetch new links directly in the steam app format and also add links to discussions, workshop etc.

Yea, I think that's ideal. Changing links that currently exist and changing new links automatically would solve this.
Not sure how useful changing links after they're clicked would be, even if possible.

I added the functionality now. It will be available with the 3.6 release shortly.

Man, that is awesome. Thank you!


content ideas

in defense of intelligent design

If premature optimization is bad, then Decoupling (from user PoV) and Standardization (from community PoV) are mature optimizations that offer the most bang for the buck


fixer.io api changed, it can't implement correctly

fixer.io api changed, it can't implement correctly

@beaquant I've now added initial support for the new API, it requires you to register for an access key (The default client will load its access key from the environment variable FIXER_ACCESS_KEY)
You should still be able to use the fixer.ExratesClient without an access key.


Accommodate 3 element tuple return from ssh execute command call

The ssh execute call returns a three element tuple  - output result, error message and exit status.

TY dear Brian 🍻


adding another "pause sweep" option

viomi.vacuum.v18 uses "pause" as pause sweeping action.
I also managed to add find vacuum option but don't know how to add it in the code.
I changed line 97 to
            self._act_locate = self._srv_audio.get_action('position', 'find_device')
but I guess it would break some other devices

👍🏻 Thanks !


Add pandas.DataFrame.set_index

Resolves #222
Added pandas.DataFrame.set_index primitive.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #234   +/-   ##
=======================================
  Coverage   27.98%   27.98%
=======================================
  Files          26       26
  Lines        1619     1619
=======================================
  Hits          453      453
  Misses       1166     1166

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 9d8c727...6e30fb0. Read the comment docs.


Revert "Create jdeps output from classpath entries (#1376)"

Reverting jdeps implementation as it currently does more harm, than is useful. Intellij attach sources requires full jars and won't work with ijars for Scala code.
This reverts commit 3dd5d8110d56cfc19722532866cbfc039a6a9612.

@liucijus When you mean does more harm than good, could you elaborate? in the sense that if we don't use ijar with a 3rdparty dep, would this solve the intellij navigate to source? What is the downside of keeping it?

It's a problem for 2nd party deps (deps which are source dependency, but not in Intellij project view). scala_import does not produce interface jar, so it's not a problem for 3rd party dep if its loader uses scala_import or just gives full jars.
Interface jars don't work with Scala attach sources (Intellij Scala plugin issue). In Intellij jdeps is supported by Java plugin, and there is also Scala plugin which discovers deps from Intellij aspect. So, because of jdeps support, deps are attached multiple times during sync and Intellij indexing can get longer. In our (Wix) case it attaches interface jars next to full jars, because we have customized the Intellij aspect to return full jars for the sync to support sources attachment.
Also, having full jars is not enough to make source attachment to work in Intellij. Currently Bazel plugin does not look into language specific sync data - we did some changes to get it working for us in our internal plugin.
I'm still considering returning jdeps support later, but before I do it, I want to see if I can upstream some changes to Intellij plugin.

Is this fully abandoned? What should be done in order to do it right then?

I think the main stopper is Intellij plugin. As such change will break attachment of jars, because jdeps contains links to ijars instead of full jars

Or ijars should not strip SourceFile attribute, is it correct?

Did not test this, but could be the missing thing according to the ticket

Side note: I'm actually not sure how much ijars are helping us with scala.
Ijar keeps the scala signature, but that could change enough to make the ijar almost as likely to invalidate cache as the jar.
I know scala puts the method size in the attributes for inlining information. Many (almost all?) changes to code will change the size of something.
It would be interesting to have some benchmark set of diffs and compare build after each diff applied using ijar vs the original jar and see what kind of win ijar is giving with scala.
Perhaps we could even find ways to make it more effective with such a benchmark suite.


Not working on meteor/devel branch

any error messages?

Wow, I was pretty sure I had edited this issue weeks ago. I'm really sorry about that.
Well, stubfence crashed with ground:db and meteor devel branch (meteor/meteor@ed43873fb0d9ddfca09530888cb621e2c714ae27).
W20150802-00:29:12.413(-3)? (STDERR) /Users/igor/Desktop/meteor/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150802-00:29:12.413(-3)? (STDERR)            				throw(ex);
W20150802-00:29:12.413(-3)? (STDERR)            				      ^
W20150802-00:29:12.413(-3)? (STDERR) Error: Arg! Meteor just broke raix:stubfence! Please report to @raix at github, and he will work his butt off trying to fix this
W20150802-00:29:12.414(-3)? (STDERR)     at inFence (raix:stubfence/util.js:4:1)
W20150802-00:29:12.414(-3)? (STDERR)     at /Users/igor/Projects/stubfencetest/project/.meteor/local/build/programs/server/packages/raix_stubfence.js:27:4
W20150802-00:29:12.414(-3)? (STDERR)     at /Users/igor/Projects/stubfencetest/project/.meteor/local/build/programs/server/packages/raix_stubfence.js:124:3
W20150802-00:29:12.414(-3)? (STDERR)     at /Users/igor/Projects/stubfencetest/project/.meteor/local/build/programs/server/boot.js:222:10
W20150802-00:29:12.414(-3)? (STDERR)     at Array.forEach (native)
W20150802-00:29:12.414(-3)? (STDERR)     at Function._.each._.forEach (/Users/igor/Desktop/meteor/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20150802-00:29:12.414(-3)? (STDERR)     at /Users/igor/Projects/stubfencetest/project/.meteor/local/build/programs/server/boot.js:117:5
=> Exited with code: 8

I don't think I have any better error description.

Got it - Meteor must have changed the api


Option to deactivate barge-in

Proposed changes
Deactivate barge-in
Context
At times (e.g. Educational content), it's necessary to ensure that the queue is processed without interruptions. Currently, adding messages to the queue can cause disruptions, leading to the current audio being interrupted and restarted.
BTW Love your work Guys.
Thanks.

Let me know if you can help here, I could figure out yet.

I think we would need to move VAD off of the custom hook and use the vad-web library itself to be able to enable/disable it. If you're interested in figuring it out, you may want to look at ricky0123/vad for JavaScript (not React/Next.js) and see about how you might use that in our own custom hook, or see if he has a way to enable/disable the library from the options


Wrong appearance

Remove one extra quote from C# code snippet

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Project "3-Social-Proof" is not Responsive.

I saw that the Project 3-Social Proof project is not responsive, Can anyone make it responsive properly?

Kindly assign me this issue

Do it 🚀


fullscreen flag

I would like there to be a fullscreen flag for sentinel to declare via commandline

Resolved with 320


Android: Live2D Manager won't show and the game crashes

Reproduction:

if you installed Live2D, press L2D manager button.
It crashed caused by java.lang.Iterable$-CC

it caused by forEach() doesn't exist on Android, this method exclusively on PC

killed on commit 43c9bf3f82bddaae24d11ffda66eab757e88fee5


Use ILog2 with support for RoundUp

Use ilog2(x, RoundUp) rather than checking and rounding by hand.
This depends on ILog2 v0.2.3, which was released in Nov. 2021
This moves complexity to one point, and it's in the library. It is also
in some cases more performant. I did not find any cases where the PR
decreases performance.
For quantum_mallows_bigint, allocate storage for the BigInt at the toplevel of
the module, and copy the desired value in __init__.  This is more performant. It does add complexity.
I don't recall all the details of why this is necessary. But, it is widespread. Steven G Johnson explains it somewhere.
If you try to simply allocate and store a value at the same time and store this in the compiled cache, you will get a segfault.
Do the same for quantum_mallows_float. Doing ilog2 is slightly faster than
what was there previously.
For example @btime shows the following for
import Random; const rng = Random.default_rng();
@btime QuantumClifford.quantum_mallows_float(rng, 5);

before PR: 236ns
after  PR: 199ns
Remarks
Julia, including the package ecosystem, has almost no support for automatically optimizing arithmetic with BigInts.
(There is https://github.com/tkluck/InPlace.jl, which could potentially be used here. But it doesn't do everything.)
There are two problems.

Storage for BigInt is dynamically allocated and it is expensive. There are some inplace functions that you can use to reuse intermediate values to avoid allocation. But, it takes some attention to use these.
You can't save allocated BigInts in the compiled caches (.ji files). You compile in const Refs (I think allocation happens here), and then copy in the desired value in the __init__ function. This can improve efficiency. But, it does add a bit of complexity.

I left the remnants of a few ways to handle the BigInt code. If the current PR is ok, I can remove the cruft. In the current PR, the following is done. You can check that ^(::BigInt, ::UInt64) does an MPZ call without doing any allocation, except for the return value..
k = rand(rng, 2:big_four_[]^m)
l = ilog2(k, RoundUp)

This is another possibility. IIRC, I did not see any consistent performance difference.
ulim = big_four_[]^m  # BigInt(4)^m
Base.GMP.MPZ.add!(ulim, big_minus_one_[])
rand(rng, 1:ulim)

Another possibility is to forgo allocating storage for BigInt in __init__ and just lift the allocation of BigInt(4) out of the loop. This would perhaps not be as efficient, but it would be simpler.
Fixed. Another thing. The global identifier big_four_  should probably be something like BIG_FOUR instead. Several Julia style guides recommend this.
EDIT: I looked into base/refvalue.jl.  Allocation does not happen at compile time as I state above. Both allocation and loading a value happen in __init__.
EDIT:  Some extraneous whitespace at the end of lines was removed by my editor. I committed this.

Thank you, this is incredibly helpful insight and suggestions! I will take some time to read through the state of BigInt in Julia, but I should be able to merge this and perform the suggested changes by the end of the week.
Sorry for the state of testing right now. I am in the middle of deprecating a bunch of not-well-thought-out interfaces, so stuff related to Noisy Circuits and Monte Carlo simulations raise a lot of errors. Feel free to neglect them -- they will be cleaned up in the upcoming 0.5

John, may I include your name and contact in the CONTRIBUTORS file?

I got some problems with large random tableaux:
julia> @benchmark random_destabilizer(200;phases=false)
ERROR: InexactError: Int64(2.0348336732447735e120)

(Sorry, this would have been caught by the tests if it was not for the current removal of a few features that are breaking some tests).
And yes, feel free to remove the cruft commented out alternative ways to process the bigints.

John, may I include your name and contact in the CONTRIBUTORS file?
Sure, go ahead!

I got some problems with large random tableaux

I'll take a look...

It seems the problem appears only around n>30, which is actually the cut off at which I switch to floats. The reason I switch to floats is because at around n>30, ints stop being able to express the numbers I need to work with. Of course, I could just switch to BigInts at that point, but floats are faster than BigInts and are correct up until n~200. So I do not switch to BigInts until n~200.
Thus, to make this work, I am reverting the changes you made to the floats branch (30<n<200) and keeping the changes you made to the other branches. Let me know if this sounds right?

Sure, that sounds right. I'll look at it right now.

The problem is that ilog2(x::Float64) is currently broken if x > typemax(Int).
I am thinking about how to fix it. If I just do floor(Int, log2(x)), this works, but is 2.5 times slower than the current ilog2, if x is in the acceptable range.

This is great! It will permit me to simplify a bunch of the branching I am currently doing.


Make SMPL from COCO Keypoints

Hi! I'm very new to this. I have coco keypoints of a person (18 keypoint format) for multiple frames. I would like to convert my keypoints to the SMPL format so that I can render some animations on Mixamo.
Is this at all possible? And if so, how can I do this?
My keypoint file is of the format [frames,joints,3] -> 300,18,3 (3 refers to positions along the x,y,z axis)

你好！我对此很陌生。我有一个人的可可关键点（18 个关键点格式）用于多个帧。我想将我的关键点转换为 SMPL 格式，以便我可以在 Mixamo 上渲染一些动画。这可能吗？如果是这样，我该怎么做？我的关键点文件的格式为 [frames，joints，3] -> 300,18,3（3 是指沿 x，y，z 轴的位置）

Hi, have you tried to solve this problem of yours and how did it work out, if the problem was solved hope you can tell me how to do it as I am facing the same problem, thanks!

Yes, you can check out this repository: https://github.com/google/aistplusplus_api
The file: https://github.com/google/aistplusplus_api/blob/2dd7b3e946b794fd0081c98e2e2433545abf8b87/processing/run_estimate_smpl.py#L64
This contains the code to estimate the SMPL parameters to match the keypoints in COCO and OpenPose.


V17 got best rating 7857? so BIG progress?

is this rating have the same criterion with previous version?
v16 was only 4397.
v15 was only 4478
v14 was only 4868
now, v17 is 7857? really so BIG progress?

https://github.com/tensorflow/minigo/issues/722

Probably not; the rating system seen on the pages are not tethered to any external system, so it's not guaranteed that the difference in rating is a guaranteed external difference.  It might be!  We haven't done a very good job of measuring it externally yet.  But -- probably not :)


I just have to define custom heading type for single data fetching.

Please prefix your issue with one of the following: [BUG] [PROPOSAL] [QUESTION].
Package ~2.1.0, Laravel 5.4
/*
|--------------------------------------------------------------------------
| Has heading
|--------------------------------------------------------------------------
|
| The sheet has a heading (first) row which we can use as attribute names
|
| Options: true|false|slugged|slugged_with_count|ascii|numeric|hashed|trans|original
|
*/
    'heading'                 => 'slugged', () <<<<<< to 'original' for single sheet data fetching

$reader = Excel::setDelimiter("\t")->load($shipmenttransmittalFileGlob[0])->get(); <<<<<< to 'original' header for single sheet data fetching
                    if ($reader) {
                        $data[$file_list_inner_value->getfilename()]['shipmenttransmittal'] = $reader;
                    }

How to do this ?

We are closing all old tickets, because version 2.* is reaching the End-of-Life stage. Our full attention is now allocated to version 3.0. It's still possible to use 2.*, but there will be no active support anymore.


Document make misunderstanding about HTTP support.

In the CDN and SSL support section, this document describes 'To make your static website files available over HTTPS, see Using the Azure CDN to access blobs with custom domains over HTTPS'. From this document, I thought I must use CDN for using HTTPS.
On the other hand, How does it work? section describes that we can access https://contoso.z4.web.core.windows.net/image.png. In addition to it, I can access my file through HTTPS when I tried in my account.
In fact, we can use HTTPS, so how about to change CDN and SSL support for describing actual work?

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

ID: dcd3f99a-f522-6edb-6d39-eac1bb917956
Version Independent ID: 69e9c1e5-8deb-8092-46f4-6646d076fbe4
Content: Static website hosting in Azure Storage
Content Source: articles/storage/blobs/storage-blob-static-website.md
Service: storage
Sub-service: blobs
GitHub Login: @tamram
Microsoft Alias: tamram

@NT-D Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thank you. I have made the edit to the documentation.
#please-close

Thank you very much @Adam-Smith-MSFT  and @xyh1 for catching feedback!
Btw, I love blob static web site hosting function. Thank you very much for making cool product ;)

@normesta - FYI


any guide for non-SPA page?

I want to create a component to do some complex things, with models and services only, without any routes and mount point

Create a component containing view, model, services? Then how to use it? And need to customise it when using it?

Yes, a component with same view and models, different services across  different projects.

In another situation, how to use dva in a multi-entry(in package.json) project? That is what "non-SPA" means, a structure like antd-init created


Update NOTES.txt for new feature sets

What
Describe what the change is solving
Update NOTES.txt on Helm install
How
Describe the solution
With my text editor
Breaking Changes
Are there any breaking changes in this PR?
No.

#508  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hjkatz and the rest of your teammates on  Graphite

Merge activity

Nov 14, 1:33 PM EST: Graphite rebased this pull request as part of a merge.


Ads notification received but it has unknown notificationHandle (39). Use unsubscribe() to save resources.

Hello,
after upgrading I started receiving a lot of Ads notification received but it has unknown notificationHandle (39). Use unsubscribe() to save resources. messages.
How exactly can a notification handle get lost and how could I debug this? Is it possible to find out what the initial symbolName for that subscription was?
Thank you!

Hi!
Basically how this could happen (I think) is that you have active connection and you have subscribed to different variables. Then the connection goes down, you kill the process or something else unexpected. The PLC does not know that and it keeps sending those notifications. If you close the client using disconnect(), it automatically unsubscribes all. But if the connection "just loses", the plc continues sending.
So the only way to stop notifications from sending is to restart PLC or to manually send unsubscribe command with the handle.

Hey I finally found the issue and thought it would be worth mentioning. So even after cleanly disconnecting every time I started getting those messages about unknown notificationHandle, even after clean restarts and they would start accumulating.
This was the problem:
We have a setup where we have the ads-client running on the same computer as the TwinCAT XAE shell which are both connecting to the remote PLC. However instead of using the simple connection config we were using the one were we specifiy the AMS router:
const client = new ads.Client({
  localAmsNetId: '<IP_ADDRESS>.1.1',  //Can be anything but needs to be in PLC StaticRoutes.xml file
  localAdsPort: 32750,                //Can be anything that is not used
  targetAmsNetId: '<IP_ADDRESS>.1.1',
  targetAdsPort: 851,
  routerAddress: '<IP_ADDRESS>',     //PLC ip address
  routerTcpPort: 48898                //PLC needs to have this port opened. Test disabling all firewalls if problems
})

For the router address we used the PLC's address. This seems to be actually illegal as only one entity (router or client) should be connecting from one IP address. What seemed to be happening is that we were receiving subscription updates to our ads-client that were meant for the XAE shell that was running at the same time.
The moment we switched for the simple connection config of just specifying the targetAmsNetId (and let the local router figure out the rest) all problems we had regarding connection issues with both ads-client and shell + unknown notificationHandle messages disappeared!

Hi @dotKokott!
Sorry for late reply. I read your message on the same day but totally forgot to answer.
Thanks for reporting this. It seems that when using a router you can have as many connections as you like. Otherwise just one or so, however I'm not sure how it really was. I investigated that when I started working on this library but now totally forgot...
I will add this info to the FAQ! Certainly interesting that you found reason for that unknown handle problem, I have never seen that yet myself.


Make ExternalStorageLocation doc more readable

fix typo about  , make ExternalStorageLocation doc more readable.

Pull Request Test Coverage Report for Build 4280

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 69.54%

Totals

Change from base Build 4269:
0.0%

Covered Lines:
12472

Relevant Lines:
17935

💛  - Coveralls


Feature/ruby 3.1 ci tests

Try bumping the general CI dependencies for Ruby v3.1 testing

PR for the Ruby v3.1 branch, this change bumps the CI dependencies to match the upgrade.


npm package update

npm package still uses an older version of svgo compared with the used in this repo (according to package.json)
suggestion: how about creating an up-to-date package here with npm package registery at GitHub?

Ah, forgot I had only pushed to the beta channel ;) It's released as 2.0 now


Runelite crashing

Please provide a brief summary of the bug
Idk how relevant this is but in my crash logs I was provided with this forum and I'm really stuck on how to solve it as the few things I tried haven't helped at all.
My client will randomly crash with no error message or indication that it is about to crash.
I think I have a relatively decent PC so I don't think that should be the issue but would be good to know if so in order to get it fixed.
I have attached my crash log if that is any help but I have no idea how to read it:
https://pastebin.com/vfyqP4iF
Did you test with the latest update version?
No response
Please provide steps to reproduce where possible
No response
Expected Results
The client should run without crashing normally
Actual Results
The client crashes.
Last year I had 4 crash reports.
This week I have had 7 without any changes that I can tell.
What Java Version are you using?
OpenJDK 64-Bit Server VM Temurin-11.0.22+7
What is your operating system and platform?
Windows 10
How did you install Java?
No response
Did it work before?
No response
Did you test with other Java versions?
No response
Relevant log output
No response

It looks a bit like a memory corruption.  Have you reported this to Runescape?  I assume you're on the latest version, graphics drivers and patched windows OS?

Thank you for the response!
Yeah the first thing I did was update OS and drivers, the client updates weekly so I assume that is also the latest update.
Runescape support is beyond useless so I wouldn't even know who to report it to if I'm honest..
I will try to google solutions for memory corruption and see if it can be fixed

OK, one thing to try is to download and install the previous version of Java you had, if less crashes that helps us narrow it down

I'm not sure if I'm being an idiot or not.. Is it possible I don't have java installed at all?
https://i.imgur.com/hkk3E0S.png

It looks a bit like a memory corruption. Have you reported this to Runescape? I assume you're on the latest version, graphics drivers and patched windows OS?

Runelite is separate from Runescape. Runelite is an approved client to run Oldschool Runescape.

I joined the Runelite discord and they told me to turn off Core Performance Boost in my BIOS and it has been 24 hours without a crash after previously crashing everyday.
I would suggest giving that a try if you are having the issue but I lost 50-100 FPS on games like CS2 and Valorant so there is a trade off.

Thanks, unfortunately this is a native crash that just happens to propagate back through Java (which is why you see a Java 'crash').


[fix](hash join) fix stack overflow caused by evaluate case expr on huge build block

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

run buildall