[v2] didFocus event is very slow

On v1 didFocus occured after +- 1 second. On v2 didFocus happens after +- 10 seconds.
...
// also tried in `constructor`
componentDidMount() {
  this.props.navigation.addListener('willFocus', () => alert('will focus'))
  this.props.navigation.addListener('didFocus', () => alert('did focus'))
}

willFocus works as expected.

Shouldn't be an issue in latest version (2.1.0). Please try and lemme know.

I just upgraded to 2.1.0 and the issue still persists.

@Evan-Younan I'm not following what the issue is. willFocus and willBlur usually fire at the beginning of animation, didFocus and didBlur fire at the end of animation which will take a while. I don't believe the mentioned number of 10 seconds, but in case of spring animation it can indeed take a while to fire those events.
If you want them to fire sooner, you need to tweak your animations to finish sooner by tweaking transitionSpec: https://reactnavigation.org/docs/en/stack-navigator.html#animation-related-options
Here's the config to get what v1 used to use:
const IOSTransitionSpec = {
  duration: 500,
  easing: Easing.bezier(0.2833, 0.99, 0.31833, 0.99),
  timing: Animated.timing,
};

// ...

defaultNavigationOptions: {
  transitionSpec: { open: IOSTransitionSpec, close: IOSTransitionSpec },
};

I'll open a new issue.


TNS spectral types not being returned on Lasair page

We should report the SN types that are in the TNS on the Lasair page. Users should not have to click through.
e.g
https://lasair.roe.ac.uk/object/ZTF19abgpgyp/
Is correctly reported as being SN2019lkw , but the SN type (which is registered on the TNS) is not being reported on Lasair. This is easy to pull out - Ken and Dave have code.

Is it one of these attributes?
mysql> describe crossmatch_tns   ;
+--------------------+---------------------+------+-----+---------+----------------+
| Field              | Type                | Null | Key | Default | Extra          |
+--------------------+---------------------+------+-----+---------+----------------+
| id                 | int(10) unsigned    | NO   | PRI | NULL    | auto_increment |
| ra                 | double              | NO   |     | NULL    |                |
| decl               | double              | NO   |     | NULL    |                |
| tns_name           | varchar(10)         | YES  | UNI | NULL    |                |
| tns_prefix         | varchar(10)         | YES  |     | NULL    |                |
| disc_mag           | float               | YES  |     | NULL    |                |
| disc_mag_filter    | varchar(50)         | YES  |     | NULL    |                |
| type               | varchar(20)         | YES  | MUL | NULL    |                |
| z                  | float               | YES  |     | NULL    |                |
| hostz              | float               | YES  |     | NULL    |                |
| host_name          | varchar(512)        | YES  |     | NULL    |                |
| disc_instruments   | varchar(512)        | YES  |     | NULL    |                |
| public             | tinyint(4)          | YES  |     | NULL    |                |
| sender             | varchar(50)         | YES  |     | NULL    |                |
| associated_groups  | varchar(512)        | YES  |     | NULL    |                |
| classifying_groups | varchar(512)        | YES  |     | NULL    |                |
| discovering_groups | varchar(512)        | YES  |     | NULL    |                |
| class_instrument   | varchar(512)        | YES  |     | NULL    |                |
| tnsid              | int(10) unsigned    | YES  |     | NULL    |                |
| disc_int_name      | varchar(512)        | YES  |     | NULL    |                |
| ext_catalogs       | varchar(512)        | YES  |     | NULL    |                |
| disc_date          | datetime            | YES  |     | NULL    |                |
| tns_at             | tinyint(4)          | YES  |     | NULL    |                |
| htm16              | bigint(20) unsigned | NO   | MUL | NULL    |                |
+--------------------+---------------------+------+-----+---------+----------------+

It is probably :
| type | varchar(20) | YES | MUL | NULL | |

Yes - it's "type".  The code running at ROE is almost identical to that which is running on ATLAS/Pan-STARRS.  We also show "z" and "hostz" if they are populated, as well as "disc_int_name", "disc_date" and "sender"  (Simple "if" statement in the Django template).  The "sender" field is usually a "bot" so I remove any _bot1/_BOT/Bot* characters before displaying.

https://lasair.roe.ac.uk/object/ZTF19acxxwrs/
In TNS as SN2019wep at 0.7 arcsec, discovered 2019-12-07 23:16:30 (MJD 58824.00) by XOSS type SN II-pec

Great, while we're here, can we tidy up the text and information
"at 0.7 arcsec" - I guess means the separation between the Lasair coords and the TNS coords ? If yes, change to:
"In TNS as SN2019ewp (0.7 arcsec separation from ZTF coordinates), discovered (MJD 58824.00) by XOSS**. Classified as a ** type SN II-pec ** at a redshift z = 0.025 **
Where the redshift is reported from the value :
| z | float | YES | | NULL | |
This will allow users to compare the host z (which we report from Sherlock) and the z inferred from the transient. Sometimes these can be different which can mean interesting object.

This update has removed the useful hyperlink to the TNS object page.
I like that we can now see the classification, but would also like to click the object name to go to TNS. Thanks!

Hope this is OK now. Text below. The TNS name is hyperlinked.
In TNS as SN2019vtx (2.3 arcsec separation from ZTF coordinates), discovered (MJD 58816.00): by ATLAS. Classified as a SN Ia. At a redshift z = 0.0369.

Good job!  Just one minor issue - the date & time conversion to MJD isn't quite being done correctly.  The code you're using is only treating the date part, hence all the MJDs end with .00.  (In ATLAS and Pan-STARRS we report to TNS with 3 decimal places - though we will switch to 5 in future.)


Android Studio always thinks it is a fresh install

why do you use docker inside vagrant? Why not install Android directly in Vagrant?

The goal of this system was to make a system that is a 'fresh install' every time for testing purposes, not to establish a long-running development environment.


Efficiency of stacking symbols

Currently we can achieve the broadcast_plus functionality by two ways. The first is to use the built-in C++ operator and the second is to manually broadcast the input using broadcast_axis and then use the original add OP. Will the second solution be much slower? I previously thought that the symbol side would also benefit from the expression template and there might not be much performance loss.

It will use more memory because need to store the boardcasted array. Also can be slower because of one more read/write to the main ram


Switch V2 support?

New switches with temp sensors are released. The API has a new temperature API. Can you please support it with the lib and in home assistant?
Here is the link:
https://mobile.twitter.com/myStrom_AG/status/944152156021710850
I have the new switch and the temp is properly returned from the switch using the new API command.

I have a V2. [ip]/temp returns:
{
	"measured":	46.562500,
	"compensation":	21.000191,
	"compensated":	25.562309
}

Add with https://github.com/fabaff/python-mystrom/pull/12


Fixed issue where a source Java Enum that is null generates an Audit ERROR due to underlying JavaWriter exception

Fixes: #3239
The issue occurs when the enum in the source Java Object is null and is required to be mapped to an enum in a destination Java Object.
This generates a mapping error as it tries to create enum using the writerUtil.createComplexChildObject which tries to call a constructor on the Java Enum class.
The exception is:
io.atlasmap.api.AtlasException: Unable to create value for segment: statesLong parentObject: io.atlasmap.java.test.TargetTestClass
The fix does not write the complex value if its null and rather uses the writerUtil.setChildObject which sets the destination as null.
Sample code for the issue can be found at
https://github.com/johnathani/AtlasMap-Enum-Error.git
There README.md explains the issue.

Looks good - can you squash the 2 commits into one?

Closing pull request and creating a new one for single commit


shp buildrun list crashes when a BuildRun status has no startTime

The cli crashes when listing all BuildRuns and one of them doesn't have a status.startTime. This could be reproduced when creating a BuildRun and linking a non-existing strategy.
Tested with Shipwright operator 0.9.0 and cli 0.9.0.
Example (with the status you get when running it):
apiVersion: shipwright.io/v1alpha1
kind: BuildRun
metadata:
  name: missing-strategy
spec:
  buildSpec:
    output:
      image: 'foo/bar:latest'
    source:
      contextDir: source-build
      url: 'https://github.com/shipwright-io/sample-go.git'
    strategy:
      kind: ClusterBuildStrategy
      name: missing-strategy
status:
  completionTime: '2022-05-04T07:49:11Z'
  conditions:
    - lastTransitionTime: '2022-05-04T07:49:11Z'
      message: clusterBuildStrategy missing-strategy does not exist
      reason: BuildRegistrationFailed
      status: 'False'
      type: Succeeded

➜  ~ shp buildrun -n christoph list
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1789e33]

goroutine 1 [running]:
github.com/shipwright-io/cli/pkg/shp/cmd/buildrun.(*ListCommand).Run(0xc0002cbab0, 0xc0006bfd40, 0xc0006bfd68)
	github.com/shipwright-io/cli/pkg/shp/cmd/buildrun/list.go:88 +0x2d3
github.com/shipwright-io/cli/pkg/shp/cmd/runner.(*Runner).RunE(0xc0000ab080, 0xc0006bfd50, {0xc0000ab3e0, 0x0, 0x0})
	github.com/shipwright-io/cli/pkg/shp/cmd/runner/runner.go:35 +0x89
github.com/spf13/cobra.(*Command).execute(0xc0001a6f00, {0xc0000ab3c0, 0x2, 0x2})
	github.com/spf13/cobra@v1.4.0/command.go:856 +0x60e
github.com/spf13/cobra.(*Command).ExecuteC(0x2dad1e0)
	github.com/spf13/cobra@v1.4.0/command.go:974 +0x3bc
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.4.0/command.go:902
main.main()
	./main.go:43 +0xe5

Expect a null check is missing here:
https://github.com/shipwright-io/cli/blob/16b20d8757cff492aa0a89e60928c57decd075f5/pkg/shp/cmd/buildrun/list.go#L88

From grooming: thank you @jerolimov , we will fix this soon.

/close fixed on #116


Proposal and rationale for eliminating the AnonCreds @context array entry from at least VC, possibly VP

We've been digging into this issue for the last couple of weeks.  I've created this document[1] that has lots of details/rationale about the approach, with only a few questions. I have one fundamental question to ask, and provide a <tl;dr> below of the document.
[1] https://hackmd.io/@BYJVN-mpSCe5H3eaIw7-7g/ryk4dvIIp
Fundamental question: To use the proof.type of "DataIntegrityProof" with a named cryptosuite, I believe we have to use W3C VC v2.0 -- is that correct?
<tl;dr> of the document content about what we should do:
It is a good idea to at least eliminate the VC AnonCreds @context array. To do so:

Define the VC proof as a Data Integrity Proof with the cryptosuite of AnonCreds2023.
    - Question: Does that require converting to the W3C VC 2.0 Data Model?
    - If so, the IssueanceDate needs to be changed to validFrom
Remove the AnonCreds @context from a VC issued with an AnonCreds proof.
Eliminate the CredentialSchema and CredentialStatus items from the VC.
Move the needed data from those two sections into the proofvalue of the AnonCreds2023 Data Integrity Proof -- at the cost of developer usefulness.

It is not strictly required for also eliminating the AnonCreds @context array entry in the VP -- details in the doc on the rationale. However, if we do want to eliminate it:

Apply all the same changes to all of the derived VCs in the VP.
Define the VC proofs in the VP as a Data Integrity Proof with the cryptosuite of AnonCredsCredentialPresentationProof2023.
Define the VP proof as a Data Integrity Proof with the cryptosuite of AnonCredsPresentation2023.
Replace the current structure in the credentialSubject item for a predicate with a string. Ugly, but possible.

Thanks all for the AnonCreds meeting today (agenda, recording) meeting today (presentation, HackMD Document). The following is the summary of the changes that are to be made:

Internal flag to simplify switching from generating VCDM 1.1 and 2.0 VCs. For now, set it to 1.1.

Differences: 1.1 needs the DataIntegrityProof @context and "@vocab" (not needed in 2.0) and in 2.0 "issuanceDate" becomes "validFrom".
Flag is not dynamic (unless there is an easy way to do this...)

Use the proof type of "DataIntegrityProof" and the three cryptosuite definitions (see previous email).

anoncredsvc-2023, anoncredspresvc-2023, anoncredspresvp-2023 (open to suggestions on the values to use on this...)

Move the AnonCreds specific items (notably, the IDs for the Schema, CredDef, RevRegId) into "proof"

Do not add the "CredentialStatus" item.

Leave the "predicates" displaying as is -- covered by the "vocab".

Over to you, @Artemkaaas and team to complete.
Thanks!

@swcurran I open PR in anoncreds-rs repository to follow the new design: https://github.com/hyperledger/anoncreds-rs/pull/291

Thanks!  @andrewwhitehead @TimoGlastra @gvelez17 FYI.

@swcurran   According to the vc-data-integrity  spec proofPurpose and verificationMethod are required properties, but they are omitted in your design.

@Artemkaaas The vc-data-integrity specification is leading. But it's a bit weird, as the verificationMethod needs to point to a verificationMethod object, but with AnonCreds you don't really have a verificationMethod, you have a cred def id. Can we use the credentialDefinitionId as the verification method? It's an URI, and it would add some context to the credential around which credentialDefinitionId is used

For proofPurpose we can (for now) use assertionMethod. OK as that is generally what Proofs are used for, but limiting if not controlled by the end user.  Let’s do that for the short term and add an open issue for putting it in.
I really like @TimoGlastra ’s suggestion about putting in the cred_def_id, as it aligns with the purpose of the Verification Method — a reference to the public key (and in the case of a Cred Def — a bit more). That definitely works for me.

cred_def_id suites for anoncredsvc-2023 and anoncredspresvc-2023 proofs but not for anoncredspresvp-2023 as it may contain multiple credentials.

That’s a tough one. There is nothing obvious to use. The proof is there to prove holder binding via the link secret across all verifiable credentials. The holder is proving that they know the link secret that is blinded in each of the source VCs.  I think we should (arbitrarily) pick one of the CredDefIds and use it.  We are using that URL — and others, admittedly — so it’s not wrong, just incomplete.  @Artemkaaas — to get something in place, I suggest we go with that.  I can puruse the issue with Manu, et al. in the Data Integrity community to see if there is other guidance.

In looking at the latest question, I looked a little more at proofPurpose.  Per this, section 7.2, item 3.1.2 in the VCDM, the value should be assertionMethod for a VC and authentication for a VP.
So @Artemkaaas, please adjust the VP hard-coded values to those.

Hey @swcurran what's the status on this issue?
I noticed the PR is merged: https://github.com/hyperledger/anoncreds-rs/pull/291, also noticed that RFC0809 implies that the anoncreds w3c data model has switched to use DataIntegrityProofs: https://github.com/hyperledger/aries-rfcs/blob/main/features/0809-w3c-data-integrity-credential-attachment/README.md#binding-in-credential
Does the anoncreds spec example need to be updated in relation to this? it still uses the new @context/type: https://hyperledger.github.io/anoncreds-spec/#w3c-verifiable-credentials-representation
cheers!


Update to errno 0.3.0.

This fixes compilation on WASI on Rust stable.

Released v0.1.3 with this change


chore: make options injectable

Fixes #N/A
Description
Introduces the Injectable pattern currently used for Settings for Options. This enables injecting Options from a cloud provider into the manager.
How was this change tested?
make presubmit and /karpenter snapshot (via aws/karpenter@pr4789)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 28 (75.0%)  changed or added relevant lines in 3 files are covered.
12 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.1%) to 82.075%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/operator/options/options.go
18
20
90.0%

pkg/operator/injection/injection.go
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/provisioning/scheduling/topology.go
2
86.49%

pkg/controllers/provisioning/scheduling/topologygroup.go
2
96.75%

pkg/test/cachesyncingclient.go
2
80.21%

pkg/controllers/deprovisioning/pdblimits.go
6
76.47%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
9080

Relevant Lines:
11063

💛  - Coveralls


Tracking a curve?

Hi, I tried the simulation, but it only showed that the robot was tracking straight lines and turning in place at the corner. Does it be able to track a curve? Thank you very much!

Yes, it is actually meant to track curves and not corners!

Yes, it is actually meant to track curves and not corners!


protojson.Unmarshal drops nested struct in error case

What version of protobuf and what language are you using?
google.golang.org/protobuf v1.36.0
What did you do?
protojson.Unmarshal behaves differently than encoding/json in the case when there are field type mismatch errors. It will unmarshal some fields, but drop the nested struct that contains the error field.
  func TestNestedUnmarshalWithError(t *testing.T) {
	jsonString := `{"baz": "buzz", "containsError": {"bang": "fizz", "intButWeGetString":"sillystring"}}`

	// encoding/json behavior
	var jsonOutput gobindings.Struct
	err := json.Unmarshal([]byte(jsonString), &jsonOutput)

	// error is expected
	assert.Equal(t, "json: cannot unmarshal string into Go struct field NestedStruct.containsError.intButWeGetString of type int32", err.Error())

	// but we get all the good fields still
	expected := gobindings.Struct{
		Baz:           "buzz",
		ContainsError: &gobindings.NestedStruct{Bang: "fizz"},
	}
	assert.Equal(t, expected.String(), jsonOutput.String())

	// protojson
	var protojsonOutput gobindings.Struct
	err = protojson.Unmarshal([]byte(jsonString), &protojsonOutput)
	assert.Equal(t, "proto: (line 1:71): invalid value for int32 field intButWeGetString: \"sillystring\"", err.Error())

	// we lost containsError { bang: "fizz"} !! bang didn't do anything wrong!!
	assert.Nil(t, protojsonOutput.ContainsError)

	// we still have fields outside the nested struct that contains the error though
	assert.Equal(t, "buzz", protojsonOutput.Baz)
}

What did you expect to see?
Expected output to contain fields in the nested struct. Expected behavior to have parity with encoding/json
What did you see instead?
Nested struct was nil.
Anything else we should know about your project / environment?
syntax = "proto3";
package my.package;

option go_package = "./gobindings"; // golang

message NestedStruct {
	string bang = 1;
	int32 intButWeGetString = 2;
}

message Struct {
	string baz = 1;
	NestedStruct containsError = 2;
}

Full reproduction and generated go bindings here https://github.com/nataliescottdavidson/protojson_unmarshal_drops_nested_struct_in_error_case

The only guarantee made by protojson.UnmarshalOptions.Unmarshal is:

If it returns an error, the given message may be partially set.

Fair enough


sanitize csv file path to classes.csv

User reported that it could not find the classes.csv, presumably due to some carriage return or other special character in the file path.

make it more obvious that you don't need to create the txt.


Fix session restoration of full buffers

ControlCore::RestoreFromPath calls Terminal::SetViewportPosition
with a position.y set to the vertical cursor position minus 4.
If the cursor position is at the end of the buffer, this results
in a request to set the viewport to an invalid position.
I've decided against fixing this in ControlCore::RestoreFromPath,
because I felt like SetViewportPosition should protect against
out of bounds requests in any case, and if I fix it here,
I don't need to fix it there.
Closes #17639
Validation Steps Performed

Enable persistence
Print big.txt
Restart
Looks good ✅

Oh good point. I should not move the viewport but rather print newlines and then CUP up.

We talked about it and decided to just show as much scrollback as possible, since the popups in cmd now use VT.


Controllable ambient light color

I added an AmbientLight resource that controls the ambient light color in the current PBR fragment shader. Hopefully I'm not stepping on the toes of the active work being done on the real PBR shader.

@no1hitjam please respond in #2373 for the relicense to MIT/Apache 2.0. Thanks!


Integration Failing Setup

Attempting to register the integration, wizard just reloads with blank API and ID fields when submitted, the following is in the log on the most recent version of HA
Unexpected exception
Traceback (most recent call last):
  File "/config/custom_components/octopus_intelligent/config_flow.py", line 61, in async_step_user
    await self.hass.async_add_executor_job(
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 587, in async_add_executor_job
    task = self.loop.run_in_executor(None, target, *args)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/asyncio/base_events.py", line 818, in run_in_executor
    self._check_callback(func, 'run_in_executor')
  File "/usr/local/lib/python3.11/asyncio/base_events.py", line 773, in _check_callback
    raise TypeError(
TypeError: coroutines cannot be used with run_in_executor()

Same problem for me.


body empty when using post(url).body

This is my code
match method {
            "POST" => {
                result = self.client.post(url).body("jfdgkslö").send().await;
            }

Url redirects to a Postman mock server (However I tested this on my own servers as well).
When executing the code a request is made to the correct url but the body is empty. Interestingly the content-length header is not empty (9). When using the post().json function a body is send. Am I missing something?

turns out reqwest doesn't automatically apply a appropriate content-type header. Adding this fixed it: .header("content-type", "text/plain")


chore(.gitlab-ci.yml): add trigger to external deployment pipeline

Codecov Report

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

@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   70.19%   70.19%
=======================================
  Files          33       33
  Lines         151      151
  Branches       32       32
=======================================
  Hits          106      106
  Misses         38       38
  Partials        7        7

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 b68eee8...ca36d3c. Read the comment docs.


Merge rosette support into functional branch

Adds smtlib_rosette functional backend, based on smtlib, providing the functional_rosette backend.
Adds test_rkt to test_functional.py for testing generated rosette code with racket.

Squashed down to two commits (one for the backend, one for the tests) and rebased on latest functional.


Modernization fixes for calc_euler_error_func

It looks like this method lacks tests and was never updated for changes in HARK.distribution. This commit fixes some notation so that the method works again.

[x] Update CHANGELOG.md with major/minor changes.

Arguably this PR should have a test added.

@akshayshanker This can also be reviewed and merged, once tests finish.


Button in LuCi to install overlay packages defined in installed_packages.txt

It is now possible  with sysupgrade to get a list of your user installed packages (on the overlay) and have this put in installed_packages.txt which is preserved during a firmware upgrade.
However there appears to be no easy way to install these missing packages (for newbies) so what I purpose is putting a button on the Backup / Flash Firmware page to allow end-users with one click to install these missing packages (assumes working internet).
It would also be useful to have the option to create the installed_packages.txt and update it if required when generating a config backup
relates to https://github.com/openwrt/luci/issues/4160

This should be handled by sysupgrade itself, not by LuCI.
It could start by a file which would contain a list of packages to install after a software update.
You typically don't want to reinstall all packages anyways. You only want a few ones, and let opkg handle the dependencies.

The button could run something like
# opkg update
# grep "\toverlay" /etc/backup/installed_packages.txt | cut -f1 | xargs -r opkg install
# rm /etc/backup/installed_packages.txt
# reboot

Taken from here<EMAIL_ADDRESS>

well again, that would reinstall all packages, which is not what you typically want. Let say you want package A which depends on package B. After an update, it now depends on package C. Your script would install package B for no reason.

I completely understand, but i am just thinking of the hobbyist at home who struggles to keep his OpenWrt router up to date because of the hoops that have to be jumped through. A good chunk of users like me only have installed what they want and just want a simple update process.
You could add a list with checkboxes when the button is click which still utilises this file, however this would be a lot more complicated to add that a button :)

One of the problem is packages like libwolfssl<IP_ADDRESS>253b90. This is the package name. The version is 4.7.0-stable-2.
There could well be a different libwolfssl after the next update.
I still think a list of packages to (tentatively) reinstall after the software update would be a better idea, and it should be built-in to sysupgrade, not LuCI.


Update dependencies

Run:

bundle update for dev gems, all of which are dev dependencies
cargo update for the gem's crates, many of which are production dependencies
cargo update in examples/rust-crate to fix dependabot security warnings

I don't understand how pushing a new version of this branch closed the PR 😕 -- will re-open.

Oops, bad commit on my part, sorry about that. I left Cargo.lock and Gemfile.lock uncommitted 🤦.
Should be good now.


feat: add seed file for Blocked model, seed all models

Does this commit successfully seed the database on your local machine? If you are seeing 6 tables with correct dummy data in Postico I can approve it!

@vjrodriguez good point, and I was missing our first and last names! it now successfully seeds all seeds I have on my end :)


Possible implications of "Vector over Graph" strategy

List is WIP ... not complete yet
Background
We see most opportunities around Weaviate's Vector/Fuzzy abilities and want to move further in that direction. To fully enable this, we want to replace some of the graph powered backends (namely janusgraph) with a purely vector based approach.
Possible Implications

No.
Area
Capability
Previous Status
Technical Implication
UX / Possible Mitigation

1
Local Get
nested cross-refs in result set (not filter)
no depth limit
⚠ Each cross-ref is a new search query.  If we store each cross-ref as a vector point, it might be possible to do a kNN search on the cross-ref prop instead of on the actual centroid. We have to try if this actually works
Needs experiment

Note: We also need geo location filtering...

That's not an issue and should be included in no. 2. In fact, the way Janus is currently doing this is also with Elasticsearch :)

Wonderful

First of all, I understand the challenge and that each nested request prompts a new request. I'm going to make a few assumptions based on how you are planning on storing the data so I might be wrong there.
Some thoughts per bullet.

Doesn't this depend on the number of queries? If I go 5 deep but set a limit of 10. This is a very reasonable request, isn't it?
This is the one I'm a bit worried about and I'm not sure I understand why this might be an issue (I think you mean valueString=The Netherlands btw). If I'm correct you are doing: request to get Cities, nested in this a request to get the related Country, you validate the name of the country, etcetera. I understand this is a compute-intensive task, but I don't understand why it is not possible.
That's an interesting point.

I think that if we let go a bit more of the traditional linking and focus more on the vector referencing we might actually get a lot of value. I need to think a bit more about this, though. I'll try to share my thoughts today.

I assume your bullet 2 references no 3 and 3 references 4. Under this assumption, some answers:

Your example leads to up to 10^5 =10,000 searches. That is very far from a reasonable request unfortunately. If a single user request can spawn this many db requests, we cannot build a stable system. As soon as there's a bit of load on weaviate the db load would be unimaginable. Additionally the UX would be very bad, as response times are enourmous. If we need to support such deep searches we need to either
a. keep a graph backend
b. build some sort of a reference index per concept at import time or asynchronously (this is essentially what janus does)
c. limit the amount of references we consider. I.e. even if the user specifies n=10, we only follow the first cross-ref for each result. Then, we end up with 10 * 5 = 50 searches which is far more reasonable. However, if a concept has more than one cross-ref, we will be missing data.
n/a
What I mean is by impossible, is impossible to have the backend fullfill this request. Of course we can manually do a search for all matching countries first, then for all cities which reference the country (assuming bullet 4 works. If it doesn't, we can't even do that).  However, also then, the limitations from bullet 1 apply. More importantly however, then we cannot combine this with a vector search anymore. I.e. we end up with a list of e.g. 10,000 ids for potential Places which have the correct inCity-City-inCountry-Contry association, there is no way way to search for concepts: ["food"] on this list. So essentially, whenever a cross-ref filter is set, that's the only search we can apply.
Yeah, we just need to try this out. I'm hopeful that this will work.

Update: I have some other ideas. Essentially what we're both just trying to do is force a document db to behave like a graph db. It won't ever do that (well). However, if instead of trying to get something out of ES that it's not we adhere to how ES is structured, we can probably get a large percentage of the same functionality we had before. I have to gather my thoughts and I'll update you with what I mean in more detail later.

Closing as obsolete.


added in latest conference info

added in latest conference info

@statasaurus  i had a go at making logistic better... I could maybe do with more time on the layout/ look of the page, but hopefully it reads better?


Update Mantle pod

Address part of issue: #51
This updates Mantle pod to latest available version.
Code was updated to replace deprecated methods. NSNull model keys was removed to avoid runtime assertions.

Warnings are produced by Mantle pod itself.
First issue because in Mantle podspec min iOS version set to 5.0.
Second is new Xcode 9.0 warning.

@adamyanalunas how about now?


Bug Fix // Corrected Typo

Changes proposed ✍

Corrected typos — "coulnd't" ➡ "couldn't"

Added period to end of full sentences

Screenshots (front-end changes only)
- N/A

Checklist ✅

[X] Label appropriately with Feature, Enhancement, or Bug.
[N/A] Tests are passing
[N/A] New backend functionality has been unit-tested.
[N/A] Environment variables have been updated:

[N/A] Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
[N/A] Local backend: backend/.env.dist.local, backend/.env.dist.composed.
[N/A] Configuration docs have been updated.
[N/A] Team members only: update environment variables in override, staging and production env. files and trigger update config script.

[N/A] API documentation has been updated (if necessary) (see docs on API documentation).
[X] Quality standards are met.
[ ] All changes have been tested in a staging site.
[ ] All changes are working locally running crowd.dev's Docker local environment.

 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.


Twitter button 😅

I forgot to change the content of Twitter button 😸.

@ayush-sleeping solve this if possible, I'm stuck on some other work.

@Kaustubh-Natuskar , I am little bit confused, Can you describe more ( What do you mean here )..........
So that I can solve this issue .

@Kaustubh-Natuskar , I am little bit confused, Can you describe more ( What do you mean here )..........
So that I can solve this issue .

Click on that Twitter badge.
Actually i copied it from another repository and Directly pasted ot here. So it have their description rather than ours

As you can see here. It is not even web friendly. We cannot use GitHub pages with what we have right now.
Ao change the description for Twitter badge and check if it is web friendly.
Hope it helps 🤠

Hey @Kaustubh-Natuskar , Have you seen ........... Now it also web friendly here .
Task Completed

Now you can close this issue ......

thank you man, great work


Quickly modifying a plan multiple times in quick succession bogs the UI as it re-renders

It was found that if someone is programmatically modifying a plan that someone else is viewing the UI of, those quick db mutations trigger the UI to re-render after every update. This causes a decrease in responsiveness in the UI.
We should look in to debouncing the re-render of the timeline or something to that effect.

Finding from @sqlAlchemyNoob2004:

Plans page susceptible to client side latency spikes if there is a lot of websocket traffic.
While the server appears to handle high load, an individual client struggles to keep up if there is a high volume of data being received over the websocket connections. Data (especially plan updates) causes the center chart to re-render repeatedly and frame rate plummets. Could consider batching updates that would cause the chart to re-render or look into tuning the rendering itself.


Update help output with new option --ai

I modified the following files which include help output with new option --ai:
zopen
zopen-alt
zopen-audit
zopen-clean
zopen-compute-builddeps
zopen-config-helper
zopen-generate
zopen-help2man
zopen-init
zopen-install
zopen-md2man
zopen-migrate-buildenv
zopen-migrate-groovy
zopen-promote
zopen-query
zopen-remove
zopen-split-patch
zopen-update-cacert
zopen-version
zopen-versions

This should only apply to zopen build for now. I see you updated https://github.com/ZOSOpenTools/meta/pull/834/files with --ai help added to the other zopen commands as well. Let's limit it to zopen-build. I think you can close this PR

This should only apply to zopen build for now. I see you updated https://github.com/ZOSOpenTools/meta/pull/834/files with --ai help added to the other zopen commands as well. Let's limit it to zopen-build. I think you can close this PR

I will close this PR.


[WIP] Feat/586/integrate tx model

Hey,
TODO:

[x] Fix all tests
[x] Remove consensus plugin docs
[x] Change all docs to the new transaction model
[x] Remove all TODOs currently in the diff
[x] Merge PR over here
[ ] Discuss what will happen with the Cryptocondition tests
[x] Remove old client and replace with new one
[x] Replace all util functions with the ones from commons (timestamp e.g.)
[x] Test Block and Transaction validation functions
[x] Increase test coverage to at least 0.00% diff
[ ] Rebase against master :see_no_evil:

Note: Removing the consensus plugin is issue #500 so I guess this PR will resolve that issue too. As noted in that issue:

Just a reminder: there's a page about consensus plugins in the docs: https://bigchaindb.readthedocs.io/en/latest/appendices/consensus.html

I guess that will have to be deleted.

I guess that will have to be deleted.

Thanks, I've added it to the TODOs of this PR.

Current coverage is 89.99% (diff: 95.19%)

Merging #590 into master will decrease coverage by 2.38%

@@             master       #590   diff @@
==========================================
  Files            22         20     -2
  Lines          1194       1039   -155
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           1103        935   -168
- Misses           91        104    +13
  Partials          0          0

Powered by Codecov. Last update ec78053...d7279e7

Change all docs to the new transaction model

@sbellem and I talked.
We both agree that fixing https://github.com/bigchaindb/bigchaindb/blob/master/tests/doc/run_doc_python_server_api_examples.py will take a long time (up to a week probably), which is time we do not want to afford (as this PR would get stuck for another week).
Hence, we'll ignore the doc_python_server_api.py and .md file for this PR and properly fix (meaning also refactor) it right after merging this PR.
As most people (hopefully) pull bigchaindb from pip (and since we'll not release a new version before the documentation is not fixed), we shouldn't cause problems to our users that want to replay these examples.
@vrde @r-marques @rhsimplex Thoughts?

Hence, we'll ignore the doc_python_server_api.py and .md file for this PR and properly fix (meaning also refactor) it right after merging this PR.

ok, we can simply keep on serving that file for now as long as it doesnt break?

Closing this in regard of #641


surf: add run-time dependencies to PATH

Motivation for this change
Fix #70410
Surf didn't work if it was run in a pure nix shell because then some of its run-time dependencies weren't found. To be even more extreme, I tested surf as follows:
PATH="" `which surf`

If I noticed any complaints about commands that couldn't be found, I added them to PATH in the wrapper. Not sure if this is common practice, but as far as I understand, this is how applications should be wrapped.
There might still be more run-time dependencies that are missing. I just tested some basic browsing. On the other hand, surf is so minimal that it has basically no other features than entering URL and following links. I tested both.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Forgot to cc maintainer @joachifm


Create guidance for building flexible blocks

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
What
Documents what each of the blocks is, when they should be used and how they are configured.
Why
Trello card
Rendered version

@maxgds Thanks for your review. I have applied all of the recommendations in your comments and fixed up. Can you give it another review?


Is it possible to use some of the kernels with deepspeed?

Thanks for the really great work! I'm currently pretraining an llm using deepspeed intergration in Hugging Face trainer, and I want to further accelerate the speed. Then I found unsloth. However, after some trials I found that it's incompatible with deepspeed now. So I'm wondering if it's possible to use just some of the kernels?
I've tried with fast_rms_layernorm but got layernorm.grad = None when deepspeed was partitioning the gradient. And I also tried the fast crossentropy, which indeed fitted into my code, but resulted a worser speed performance. I've read the fast llama implementation but I was still confused. So I'm wondering if there is some tips for incorporating these kernels into deepspeed code.

Sorry currently Deepspeed is not yet supported!

I found another library called Liger-Kernel inspired by Unsloth, that is compatible with DeepSpeed.


Parse does-not work with Localized formats

Hi guys,
In moment.js this works:
var moment = require("moment");
moment().format("L");
//"06/25/2016"

moment("06/25/2016", "L").toDate()
//Sat Jun 25 2016 00:00:00 GMT+0430 (Iran Daylight Time)

but using moment-jalaali parsing localized formats doesn't:
var moment = require("moment");
moment().format("L");
//"1395/04/05"

moment("1395/04/05", "L").toDate()  //<----- error!
//Sat Jan 01    0 00:00:00 GMT+0330 (Iran Standard Time)

Would you please help us.
Best Regards
Mehdy Karimpour

In moment, L is a shortcut for MM/DD/YYYY or something like that. moment-jalaali does not change the behaviour of MM or DD or YYYY. Instead, there are few other format tokens like jMM, jDD or jYYYY which are used.
In your case, if you want to use L, you need to first define it as for example jYY/jM/jD and then use it.

Thanks for your reply,

How can we define L to be jYY/jM/jD by default in persian?
2.Why it works for format but not for parse?

Best Regards
Mehdy

Based on your example, I thought you did not load Persian locale. But it seems that you have loaded it. In that case L token is set to correct Jalaali tokens. The problem is that when parsing, input format is not converted to a longer format and so it is not handled in moment-jalaali and is passed to moment.
Although it is useful to be able to parse input with locale specific formats, it needs a lot of work and I think the alternative to use more specific format tokens is easy enough. So for now, please don't use L and its friends.


Banker's algo in Python along with its readme file of Bankers algorithm

Added python implementation of Banker's algorithm.
Readme becomes important for viewers to know what the algo is made for, added a readme file for banker's algo

I went through your work and it seems valid to me. Although I will suggest you to write modular code if possible, since the repository is beginner-friendly I find it good to go.

Sure, I will do it right away

@GrayFlash , Do check the updated code. Thanks in advance, ( Grateful if u can attach hacktoberfest-accepted label to pr )

Looks Good to go, Nicely Done. @Rac-Ro007 for now I can just acknowledge your PR cause I am not a Maintainer. @OtacilioN please merge this PR looks valid to me.

@Rac-Ro007 I have gone through the code and it was good. I would like to suggest you to use list comprehension if possible so that it will make your code even more efficient.

What a great code :smile: !
Thank you for contributing! :rocket:


WIP: Bump version from 2020.9.30 to 2020.10.13

Compatible with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>New Features

Adds the ability for users to select their own SMASH servers for stakepool listings. (#2214)
Adds transaction expiry slots for pending transactions. (#1879)

Improvements

Adds a 100-wallet scenario to the latency benchmark. (#2223)
Adds an executable shelley-test-cluster which starts an integration test cluster that includes faucets. (#2178)
Extends isOurs to return the derivation path of an address. (#2219)

Resolved Issues

Make pool garbage collection handle an unknown current epoch. (#2203)
Fixes incorrect mainnet network parameters returned from API. (#2226)

Known Issues
This section is a work in progress.

Wallet restoration status reported incorrectly on mainnet. (ADP-483)

Documentation

Cardano (cardano-node)
ITN (Jörmungandr)

API Documentation
API Documentation

CLI Manual
CLI Manual

Docker Manual
Docker Manual

Installation Instructions
Cardano (cardano-node)

Install<EMAIL_ADDRESS>

Download the provided cardano-wallet for your platform, and uncompress it in a directory that is on your $PATH, e.g. /usr/local/bin. Or %PATH% on Windows.

Start cardano-wallet --help and see available parameters.

Docker
Pull from DockerHub and verify the version matches 2020.10.13.
$ docker pull inputoutput/cardano-wallet:2020.10.13
$ docker run --rm inputoutput/cardano-wallet:2020.10.13 version

ITN (jormungandr)

Install<EMAIL_ADDRESS>

Download the provided cardano-wallet-jormungandr for your platform, and uncompress it in a directory that is on your $PATH, e.g. /usr/local/bin. Or %PATH% on Windows.

(optional) Install the bash/zsh auto-completion script according to the jormungandr cli manual

Start cardano-wallet --help and see available parameters.

Docker
Pull from DockerHub and verify the version matches 2020.10.13
$ docker pull inputoutput/cardano-wallet:2020.10.13-jormungandr
$ docker run --rm inputoutput/cardano-wallet:2020.10.13-jormungandr version

Signatures

Name
Role
Approval

Matthias Benkort @KtorZ
Technical Team Lead
:hourglass:

Piotr Stachyra @piotr-iohk
QA Engineer
:hourglass:

Tatyana Valkevych @tatyanavych
Release Manager
:hourglass:

bors merge

Can we change present tense to the past, e.g. "Adds" --> "Added"?

@tatyanavych this PR is not the release draft, Jonathan included a summary of the release notes as a convenient preview. The release notes will be created in a bit and we'll discuss them on Slack as we usually do :+1:

@tatyanavych wrote:

Can we change present tense to the past, e.g. "Adds" --> "Added"?

Done.


Unable to publish a webcam /dev/video0

Which version are you using?
v0.13.2
Which operating system are you using?

OS

[ X] Linux
[ ] Windows
[ ] macOS

Architecture

[ ] amd64
[ ] arm64
[ X] arm7
[ ] arm6

Describe the issue
Description  getting error following the publish web cam document: https://github.com/aler9/rtsp-simple-server#publish-a-webcam
Describe how to replicate the issue

create the Docker container using the following command:

docker create --network=host --privileged --name=rtsp -v opt/rtsp/rtsp-simple-server.yml:/rtsp-simple-server.yml aler9/rtsp-simple-server

copy the "rtsp-simple-server.yml" to /opt/rtsp/rtsp-simple-server.yml
modify the "/opt/rtsp/rtsp-simple-server.yml" as follows:

# sets the verbosity of the program; available values are "warn", "info", "debug".
logLevel: info
# destinations of log messages; available values are "stdout", "file" and "syslog".
logDestinations: [stdout]
# if "file" is in logDestinations, this is the file which will receive the logs.
logFile: rtsp-simple-server.log

# supported stream protocols.
# UDP is the most performant, but can cause problems if there's a NAT between
# server and clients, and doesn't support encryption.
# TCP is the most versatile, and does support encryption.
# The handshake is always performed with TCP.
protocols: [tcp]
# encrypt handshake and TCP streams with TLS (RTSPS).
# available values are "no", "strict", "optional".
encryption: no
# port of the TCP/RTSP listener. This is used only if encryption is "no" or "optional".
rtspPort: 554
# port of the TCP/TLS/RTSPS listener. This is used only if encryption is "strict" or "optional".
rtspsPort: 555
# authentication methods.
authMethods: [basic, digest]
# timeout of read operations.
readTimeout: 10s
# timeout of write operations.
writeTimeout: 10s
# number of read buffers.
# a higher number allows a higher throughput,
# a lower number allows to save RAM.
readBufferCount: 512

# enable Prometheus-compatible metrics on port 9998.
metrics: no
# enable pprof on port 9999 to monitor performances.
pprof: no

# command to run when a client connects to the server.
# this is terminated with SIGINT when a client disconnects from the server.
# the server port is available in the RTSP_PORT variable.
# the restart parameter allows to restart the command if it exits suddenly.
runOnConnect:
runOnConnectRestart: no

# these settings are path-dependent.
# it's possible to use regular expressions by using a tilde as prefix.
# for example, "~^(test1|test2)$" will match both "test1" and "test2".
# for example, "~^prefix" will match all paths that start with "prefix".
# the settings under the path "all" are applied to all paths that do not match
# another entry.
paths:
  cam:
    runOnInit: ffmpeg -f v4l2 -i /dev/video0 -f rtsp rtsp://localhost:$RTSP_PORT/$RTSP_PATH
    runOnInitRestart: yes

plug in the USB camera (/dev/vidoe0) and start the docker container
attempt to connect from VLC using the URL rtsp://<IP_ADDRESS>/cam
view the Docker logs (docker logs rtsp):

2021/01/19 22:00:18 [I] [0/0/0] rtsp-simple-server v0.13.2
2021/01/19 22:00:18 [I] [0/0/0] [TCP/RTSP listener] opened on :554
2021/01/19 22:00:18 [I] [0/0/0] [path cam] on init command started
2021/01/19 22:00:34 [I] [1/0/0] [client <IP_ADDRESS>:47484] connected (plain)
2021/01/19 22:00:34 [I] [1/0/0] [client <IP_ADDRESS>:47484] ERR: unable to find a valid configuration for path 'unicast'
2021/01/19 22:00:34 [I] [0/0/0] [client <IP_ADDRESS>:47484] disconnected
2021/01/19 22:00:40 [I] [1/0/0] [client <IP_ADDRESS>:49345] connected (plain)
2021/01/19 22:00:40 [I] [1/0/0] [client <IP_ADDRESS>:49345] no one is publishing to path 'cam'
2021/01/19 22:00:40 [I] [0/0/0] [client <IP_ADDRESS>:49345] disconnected
2021/01/19 22:00:40 [I] [1/0/0] [client <IP_ADDRESS>:49346] connected (plain)
2021/01/19 22:00:40 [I] [1/0/0] [client <IP_ADDRESS>:49346] ERR: unable to find control attribute (rtsp://<IP_ADDRESS>:554/cam)
2021/01/19 22:00:40 [I] [0/0/0] [client <IP_ADDRESS>:49346] disconnected
2021/01/19 22:00:40 [I] [1/0/0] [client <IP_ADDRESS>:49347] connected (plain)
2021/01/19 22:00:40 [I] [0/0/0] [client <IP_ADDRESS>:49347] disconnected

Did you attach a network dump?

no

Hello, the problem is that ffmpeg is not present in the docker image; see here for instructions on how to generate a Docker image with both rtsp-simple-server and ffmpeg
https://github.com/aler9/rtsp-simple-server/issues/183#issuecomment-760856015

Hello, the problem is that ffmpeg is not present in the docker image; see here for instructions on how to generate a Docker image with both rtsp-simple-server and ffmpeg
https://github.com/aler9/rtsp-simple-server/issues/183#issuecomment-760856015


Multirect - Fix refining regions in derived Textures

Fixes allowing all derived texture types to modify region prior to rendering.
Fixes similar bug to #76686 for LargeTexture and MeshTexture.
Notes

Expands #76703 to deal with LargeTexture and MeshTexture, by returning an enum instead of a simple true / false, allowing fallback to the old draw_rect_region() function for complex texture types.
Sends the input and output rect in the same arguments by reference. This could be marginally better at runtime than the separate input / output args (although the #76703 version separate args might be optimized out, I haven't examined assembly - it is quite possible they may not be optimized out as the call is virtual). On the flip side this does modify both rect and r in the calling code, although they don't appear to be used after the call to draw.
Changes the function name, I wasn't quite sure the old name get_combined_rect_region made sense.

Co-authored-by: kleonc<EMAIL_ADDRESS>
Note this doesn't link to my GH account, use<EMAIL_ADDRESS>instead.

Sends the input and output rect in the same arguments by reference. This could be marginally better at runtime than the separate input / output args (although the #76703 version separate args might be optimized out, I haven't examined assembly - it is quite possible they may not be optimized out as the call is virtual).

I like mutable/inout args (performance-wise they make more sense). They seem less readable on the caller side though. Note in #76703 I made args separated based on the Texture::get_region_rect which also passes in/out rects separately. If we're fine with mutable args then I wonder if it would make sense to change Texture::get_region_rect as well to use 2 args passed by non-const reference? :thinking:
Other thing is I'm not sure about the naming convention for such params. Like what r_ prefix exactly stands for? I've always been thinking about it as "result"/"return". In such case it's kinda confusing / not obvious when such params act also as input. But maybe r_ is meant to mean "reference"? :thinking: (Doesn't make sense to me though, whether it's passed by (const/non-const) reference is already denoted by the type.)

On the flip side this does modify both rect and r in the calling code, although they don't appear to be used after the call to draw.

If the original values need to be preserved then copies could be passed in in the first place. So that's not an issue. Also it's even better to copy explicitely only when needed instead of always, possibly for no reason (and hoping compiler will optimize it out).

Changes the function name, I wasn't quite sure the old name get_combined_rect_region made sense.

Haven't gave it much thought, I guess it could be get_final_rect_region, get_actual_rect_region, etc. Refine also doesn't seem like an obviously great name (it would be such if after seeing "refine" I'd know what exactly this method does - this is not the case for me). But (given I can't come up with a "great name") I don't think it matters much, in the end it's an internal method. I'm fine with "refine".

This version also currently makes no draw the default for all Texture derived classes if width or height is zero. This is currently the behaviour for ImageTexture and a few others, but not for the gradiant textures. I'm just trying to evaluate whether this would be a better all round default / whether it will change anything. If not it's fairly easy to flip the logic here so it mirrors the original.

Makes sense to me, can't think of a situation when such logic would break something (although regression reports can be enlightening :smile:).

Co-authored-by: kleonc<EMAIL_ADDRESS>
Note this doesn't link to my GH account, use<EMAIL_ADDRESS>instead.

Ah yes, will change this, wasn't sure how the noreply emails worked. :+1:

I like mutable/inout args (performance-wise they make more sense). They seem less readable on the caller side though. Note in #76703 I made args separated based on the Texture::get_region_rect which also passes in/out rects separately. If we're fine with mutable args then I wonder if it would make sense to change Texture::get_region_rect as well to use 2 args passed by non-const reference? thinking

Possibly, but might be better in a separate PR.

Other thing is I'm not sure about the naming convention for such params. Like what r_ prefix exactly stands for? I've always been thinking about it as "result"/"return". In such case it's kinda confusing / not obvious when such params act also as input. But maybe r_ is meant to mean "reference"? thinking (Doesn't make sense to me though, whether it's passed by (const/non-const) reference is already denoted by the type.)

Afaik Godot just uses r_ for something that may be changed by the function, by reference or pointer. I personally think of it as standing for return but I'm not sure on the origin of the convention.

Also it's even better to copy explicitely only when needed instead of always, possibly for no reason (and hoping compiler will optimize it out).

Yes, this was my thoughts as to eliminating this step.

This version also currently makes no draw the default for all Texture derived classes if width or height is zero. This is currently the behaviour for ImageTexture and a few others, but not for the gradiant textures. I'm just trying to evaluate whether this would be a better all round default / whether it will change anything. If not it's fairly easy to flip the logic here so it mirrors the original.

Makes sense to me, can't think of a situation when such logic would break something (although regression reports can be enlightening smile).

It seems worth a try, rather than complicating the code with adding extra overrides unnecessarily (if there is no need, less code is better). CurveTexture has in built protection to prevent zero size, whereas e.g. GradientTexture does not. There's always the chance this may stop e.g. an unset Gradient from drawing in the editor, but as you say regression reports usually show this kind of thing pretty quickly, and it is easy to change. But equally well am happy to rig it to exactly mirror the old behaviour.

Based on the discussion for the src_rect, and how a zero area src_rect still might be valid to draw, I changed mind and reverted to mirror the old behaviour, and add the ((w | h) == 0) check only when done by the original code.
I did add the additional has_no_area() check to the dst_rect as suggested by @kleonc . Hopefully this should be fine, but is easy to change if there are any regressions.

Co-authored-by: kleonc<EMAIL_ADDRESS>
Note this doesn't link to my GH account, use<EMAIL_ADDRESS>instead.

Ah yes, will change this, wasn't sure how the noreply emails worked. +1

BTW the commit is still not linked properly. You can see that only your account is attributed e.g. here:

Seems like the Co-authored-by: line needs to be be at the end of the commit message, after two empty lines.

Rebased (and removed coauthor as now that is already merged).
Should be good now, but needs a second look to check I didn't make any silly mistakes. :smile:

@lawnjelly BTW p_commit_on_flip_change is unused in MultiRect::add:
https://github.com/godotengine/godot/blob/627a2909c24cd689b4c7f8148bcada3a552d0c8b/servers/visual/visual_server_canvas_helper.cpp#L160
It was already done like that in #68960, not sure if you want to fix it in here or in some subsequent PR (this method is unused anyway; but I do see its existence kinda makes sense API-wise).

Ah I'll take a look at the flipping arg, it's a while since I wrote it, it may be historical or for expansion. :+1:
Will be for a different PR though.

Thanks!


[FRG] Fixed Height option on Title Header does not save

Steps to reproduce:

Create a Title Header
Check the "Fixed Height" checkbox.
Set your height

When you run the "Save" function, or "Preview", it doesn't seem to save.

Fixed in v1.3.2


OpenSSF TTX - Andromeda Gales

Including the template (andromeda_template.md) to cover the scenario structure and walkthrough
updated readme.md to include relevant information

@SecurityCRob Will be one final PR after this to make some final amendments and updates as a heads up

DCO conflicts, closing PR to fix the signing issues and will reraise a PR


get metadata from REST channel

Proposed changes:
@ancalita The REST channel should by default load the metadata key.  I thought this worked in the past but can't seem to find when it was dropped.
Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

@ancalita do you know if this will be merged into the next 3.2.x release?

@ancalita do you know if this will be merged into the next 3.2.x release?

I see this is targeting main actually, I'd recommend you rebase to 3.2.x and edit target branch on GH for the change to make it in the next micro release.

Closing this. Rebasing on 3.2.x and opening #11436.


[api-docs] add select_voice_channel navigate param (#6227)

A new commit was made into the api-docs repo: https://github.com/discord/discord-api-docs/commit/ae1788ccd00a614127321dc3733a5c9ca3dcf3a6
No details given.
This is a bot created issue.

No code changes required.


Fix bgp:route_installation_test

NetworkInstance shouldn't be replaced as a whole.
Shouldn't LogYgot without first configuring BGP.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 65.965%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
783

Relevant Lines:
1187

💛  - Coveralls

/gcbrun


Q: Have you planned to have official release?

Hi!! Very very powerful setup!! I'm here to know if have you planned to have an official release soon?
Thank you!! ^_^

@llstarscreamll thanks for your feedback. Sure I'm planning to have an official release, but really don't know when exactly! However, I guess it should be within the next 50 days.
These steps must be done before the release:

Enhance the Porto documentation (which I started already).
Organize the Hello API documentation, and add a lot more details to it. (I started by cleaning the existing docs).
Refactor the code, line by line. And make sure everything is in the right place.

After the first release I guess will work on the Automatic Code Generator, which should build the entire Container for you.


Add search text length validation

Signed-off-by: Gaurav Shekhadiya<EMAIL_ADDRESS>

Coverage remained the same at 100.0% when pulling 737fdc1e7f459471a728ad2301c05f8a17e5ae85 on gauravshe:search-validation into d4c3799be1a8556a698daabcece2807fd57e90e4 on bcherny:master.

Thanks for the contribution! Is there a reason you couldn't use Angular $validators for this (https://docs.angularjs.org/api/ng/type/ngModel.NgModelController)?

Yes bcherny, Angular $validators will show error on every input change so users will always see a flash of the error message when typing any search query.
Our purpose is to show an error on search operation only.

You can probably use ng-model-options={ updateOn: blur }, or $touched to tune the validating behavior instead. No need to reimplement validators.

Yes, agreed but search operation performs on enter key not on the blur or other event.

Review comments addressed.

Thanks for the contribution!


[EmbeddingApi] Update usecase to implement XWalkView.getUserAgentString

-Update usecase to implement XWalkView.getUserAgentString
-Cover the XWalkActivity and XWalkInitializer two ways
Impacted tests(approved): new 2, update 0, delete 0
Unit test platform: [Android]
Unit test result summary: pass 2, fail 0, block 0
BUG=https://crosswalk-project.org/jira/browse/XWALK-4825

@jennycao please help to review it.

LGTM


Fix: build on 32bit system

build with GOARCH=386 fails with error:
$ GOARCH=386 go build ./...
./sriovnet_aux.go:137:17: u32Mask (untyped int constant<PHONE_NUMBER>) overflows int
fix this by modifying u32Mask to be uint32 and proper casting.

@SchSeba  a minor fix. PTAL

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 38.133%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
384

Relevant Lines:
1007

💛  - Coveralls


Export to CSV

Is your feature request related to a problem? Please describe.
On the /administrator/#/certificates page, I set a filter. A list of certificates is shown. And I want to export the list to share it with someone who does not have access to the CLM.
Describe the solution you'd like
On the /administrator/#/certificates page, add a button that would export the current filtered list of certificates to a CSV file.
Describe alternatives you've considered

Export to XML or JSON. But properly formatted CSV sounds the easiest for most of the use cases.
Export the data through the API. Too much work for the user.

Additional context
It could be usefull to export more info than only what is shown in the table. For example the complete DN would be more usefull than just the CN.

Thanks, @McLaynV, for the suggestion of this feature.
Thinking about the export functionality, we can have an input that would consists of selected parameter or attributes of the certificate to be exported. The same you can choose when applying filter on the certificate inventory. This should give users enough flexibility to export what is needed.
I would suggest to use the filter to filter out only the set of certificates you would like to export. And then select all properties that would be exported on the filtered set. In the result, you will get a CSV (or potentially any other supported format) with the information you requested.
Eventually, we can use the same approach to show columns with this information on the FE, when implemented, in a consistent way.
The export can be long, and in such case the export may take more time. We will need to store the export somewhere and send user notification that it is ready to download, or it was successfully uploaded to some location.
@lubomirw , do you have other ideas?

I think all that is described here about functionality is correct.
Just regarding export output itself, I think in most cases, it can be served directly to response. We can define some boundary number of records which can be served directly. It can make it easier for the user not needing to provide some storage location.


Documentation: instructions for upgrading existing Autotune setup

Based on NICAR feedback

@kavyasukumar still a thing I assume?


Cant change color

Hi,
I tried to get the color to static red, but unfortunately nothing changes. for me the color always changes between, purple, green and yellow. actually something seems to work, because i can switch off the RGB with msi-rgb -x 00000000<PHONE_NUMBER>0000 00000000.
do you have any idea?
my specs:
CPU: Ryzen 5 2600X
Motherboard: MSI B450 Tomahawk
Host Kernel version: 5.3

Hi!
I have the exact same mainboard. Please see, if you haven't already, #92...
msi-rgb 0 0 0 -ir -ib -> green
msi-rgb 0 0 0 -ir- ig -> blue
msi-rgb 0 0 0 -ib -ig -> red
Inverting the colours does the trick.
Some other interesting suggestion in #92 as well.

Hi macgherl57,
thank you very much!! you are my hero! :)
cheers!


issues with current build

dew point is fucked up
days are wrong on the wind and rain.
need a bigger radar only screen
incorporate snow
new ui

also time as in PM and AM needs to be imlemented


[implementTypeSort] Make the UI responsive to sorting by type

Proposal to add [implementTypeSort] to to the site.
Feature Name
implementTypeSort
Functionality
When you click a checkbox, the UI should update sorting by type
Notes
Use this endpoint: GET /wishes?types=see,be

Picking this one up


Fix async missing decorators

Description:
This issue came up in chat a few days ago. The callbacks are just our own convention, but the missing coroutine means that async_device_update is really just a generator. It works with our current codebase, but when it's called from a Python 3.5 async function it causes an error.

Good catch 👍


[libarchive] wrapper emits warnings and uses find_dependency

related issue about the warning:
https://gitlab.kitware.com/vtk/vtk/-/issues/18308
also the wrapper uses find_dependency instead of find_package which only works correctly in *config.cmake

also the wrapper uses find_dependency instead of find_package which only works correctly in *config.cmake

I noticed when looking at #16845. I might pick this up for #20146.
For lz4, I see you are already have the details in the vtk issue. An interesting puzzle indeed.


0.8.3 instead of 0.9.0 tagged as 'latest' on npm registry.

Hello,
Thanks for the great library!
Currently, both 0.9.0 and 0.8.3 are available on the npm registry - https://www.npmjs.com/package/jasmine-marbles - but 0.8.3 is marked as 'latest', while 0.9.0 is the 'higher' version. 0.8.3 was released more recently, but seems to lack some of the fixes available in 0.9.0.
Is this intentional?

Yes, it's intentional. Most users (majority Angular) are still on RxJS 6.x, so we'll default to supporting that version until the shift is more towards RxJS 7.


Multiple GPUs training

Checks

[X] This template is only for question, not feature requests or bug reports.
[X] I have thoroughly reviewed the project documentation and read the related paper(s).
[X] I have searched for existing issues, including closed ones, no similar questions.
[X] I confirm that I am using English to submit this report in order to facilitate communication.

Question details
May I know if the training script supports multiple GPUs training or parallel running? Or I can just use torchrun for multiple training? Thanks!

yes, with accelerate
README.md


emacs.pkgs.project: Use project from elpa

The version of project on ELPA has now moved well beyond what was available in emacs 28, and packages such as eglot are depending on these newer versions.
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

FWIW, I also wonder about some of the other similarly blocked packages.

Can this be backported to 22.11? As a person on the latest release, I would love if eglot kept working on it...


Added on API(google chart) ready event.

Added API ready event to the directive. This event is fired when all google chart JS files is downloaded and ready to use.
Usage :
<div google-chart on-api-ready="functionNameInYourCtrl()"> </div>

In my POV, the "googleChartApiPromise" can be easily enough injected into a controller. I don't think there is a need to hide it behind another callback specific to the directive.


Aligning content of cell with line breaks

Hi,
I was wondering whether csv.vim can display rows with line breaks inside them one the same line? Maybe I just haven't figured out how to do it but I wasn't able to get them to display correctly.
See the following example:
col1,col2,col3
val1,"val2.1
val2.2","val3.1
val3.2"

In Excel or whatever other GUI, this would probably be displayed as such:

Is that possible somehow with csv.vim? The way it's displayed at the moment makes it really difficult to align the respective "sub-rows" in my mind. I suspect the answer is that I should fix the issue by splitting the content into multiple rows but it would be better for my workflow like this and I wanted to be sure that I'm not just missing a command or setting.
Thanks!

No this is not possible in vim

Am 18.07.2022 um 15:02 schrieb Roman Hillje @.***>:

Hi,
I was wondering whether csv.vim can display rows with line breaks inside them one the same line? Maybe I just haven't figured out how to do it but I wasn't able to get them to display correctly.
See the following example:
col1,col2,col3
val1,"val2.1
val2.2","val3.1
val3.2"
In Excel or whatever other GUI, this would probably be displayed as such:
Is that possible somehow with csv.vim? The way it's displayed at the moment makes it really difficult to align the respective "sub-rows" in my mind. I suspect the answer is that I should fix the issue by splitting the content into multiple rows but it would be better for my workflow like this and I wanted to be sure that I'm not just missing a command or setting.
Thanks!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Ok, I thought so. Thanks for the response.


Use batch param in SeekGE

Thanks @guy-har - I will address the additional places (reuse scan by iterator) in a different pr.


[fingerprint] - infra generation

Part of #705
Infra generation

Backend that largely leverages text/template to expand templates for the particular app configuration.
Frontend that prompts users for inputs/necessary revisions.

Remainder work

[x] Refactor/cleanup
[x] Telemetry
[x] Acceptance tests
[ ] Review & check-in

@weikanglim Is this completed with #2658?


Animation memory leak on some platforms

Steps to reproduce

Run the code I provided in dartpad or create a new project and run with chrome.
Observe the memory footprint rising in chrome task manager (shift + esc)

Expected results
Expected result is no memory leak
Actual results
memory usage keeps rising non stop.
I didn't  test on any other devices, just chrome.
Code I provided doesn't have dispose, because that has nothing to do with the issue, issue happens while widget is on the screen.
Code sample
Code sample
// Copyright 2019 the Dart project authors. All rights reserved.
// Use of this source code is governed by a BSD-style license
// that can be found in the LICENSE file.

import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyWidget(),
    );
  }
}

class MyWidget extends StatefulWidget {
  const MyWidget({super.key});

  @override
  State<MyWidget> createState() => _MyWidgetState();
}

class _MyWidgetState extends State<MyWidget> with TickerProviderStateMixin {
  late AnimationController _controller;
  late Animation<Color?> _animation;

  @override
  void initState() {
    super.initState();
    _controller = AnimationController(
      duration: const Duration(milliseconds: 1500),
      vsync: this,
    )..repeat(reverse: true);

    _animation = ColorTween(
      begin: Colors.green.shade300,
      end: Colors.green.shade600,
    ).animate(_controller);
  }

  @override
  Widget build(BuildContext context) {
    return Container(
      decoration: BoxDecoration(
        color: Colors.grey,
      ),
      child: Center(
        child: AnimatedBuilder(
          animation: _animation,
          builder: (context, child) => Container(
            constraints: const BoxConstraints(maxHeight: 70, maxWidth: 300),
            child: Padding(
              padding: const EdgeInsets.all(8.0),
              child: ElevatedButton(
                style: ButtonStyle(
                  side:
                      MaterialStateProperty.resolveWith((states) => BorderSide(
                            color: Colors.green[600]!,
                            width: 1,
                          )),
                  backgroundColor: MaterialStateProperty.resolveWith(
                      (states) => _animation.value),
                  shape: MaterialStateProperty.all<RoundedRectangleBorder>(
                    RoundedRectangleBorder(
                      borderRadius: BorderRadius.circular(8),
                    ),
                  ),
                  minimumSize: MaterialStateProperty.resolveWith(
                      (states) => const Size(120, 60)),
                ),
                onPressed: () async {},
                child: const Padding(
                  padding: EdgeInsets.all(8.0),
                  child: Text(
                    'Button',
                    style: TextStyle(
                      color: Colors.white,
                      fontSize: 18,
                    ),
                  ),
                ),
              ),
            ),
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.2, on Microsoft Windows [Version 10.0.22621.3593], locale en-US)
[✓] Windows Version (Installed version of Windows is version 10 or higher)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[✓] Chrome - develop for the web
[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.5.4)
[✓] Android Studio (version 2022.1)
[✓] VS Code (version 1.89.1)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

Thanks for the report. Checking this issue on the latest Flutter stable and master channels, I observed the issue as below:

On Web with Chrome's Task Manager tool, I can see memory keeps rising as reported ❗

Demo on Web
https://github.com/flutter/flutter/assets/104349824/ea8cfaea-0e2f-42a4-b7fe-eb71d687cd40

On macOS app target by inspecting app on Activity Monitor, I see memory is rising as well but slower than Web app ❗

On Android app target by inspecting app's memory usage with Memory Profiler on Android Studio, memory is stable and not rising. ✅

On iOS app target by inspecting app on XCode while running it,  memory is stable and not rising. ✅

✅: No Issue ❗: Issue reproduced
Labeling the issue for further's insights.

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.22.1, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.22.1 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a14f74ff3a (11 hours ago), 2024-05-22 11:08:21 -0500
    • Engine revision 55eae6864b
    • Dart version 3.4.1
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode15.3.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

[✓] VS Code (version 1.89.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.88.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 125.0.6422.76

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.23.0-8.0.pre.16, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.23.0-8.0.pre.16 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 47f463c382 (57 minutes ago), 2024-05-27 22:26:25 -0400
    • Engine revision 416c619829
    • Dart version 3.5.0 (build 3.5.0-191.0.dev)
    • DevTools version 2.36.0-dev.10
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
    • Xcode at /Applications/Xcode15.3.app/Contents/Developer
    • Build 15E204a
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

[✓] VS Code (version 1.89.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.88.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 125.0.6422.113

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Reproduced this on the web platform.
This started with a Dart SDK change that maps Dart timeline APIs to the web user timing API (performance.mark/performance.measure)
(see https://dart.googlesource.com/sdk/+/af8fb2cd73c3b5ec96a39c59e85f470efe78d8a4)
The Flutter framework logs several events for each frame in the animation.  Chrome apparently does not have any limit on the size of the buffer that stores user timing events.  So the memory usage of the Chrome renderer process grows quickly as the animation runs, and it will grow unbounded.
@kenzieschmoll @elliette Would it be possible to make logging of Chrome timeline events optional in Flutter?

Thanks for tracking down the culprit!
Some context here:

These were added to support performance profiling for web apps
The Flutter framework uses the dart:developer Timeline API to track various events (example)
If I remember correctly, these are removed for release builds, so this should only be occurring for profile and debug builds.

I don't think we can make this optional, but I do see in the MDN docs that there is a clearMarks and clearMeasures API:  https://developer.mozilla.org/en-US/docs/Web/API/Performance. The VM implementation (cc @bkonyi) of this is using a circular buffer to store the timeline events to prevent unbounded memory usage. I think the web implementation should probably do something similar using clearMarks and clearMeasures.

@derekxu16, can you take a look at this when you have a chance?

I just ran the sample app from the issue description in Web debug mode and Web profile mode using a version of Flutter that includes the commit above, and have confirmed that the commit makes it so that the memory footprint of the tab never rises above a certain limit (for me it was 100 MB in profile mode and 300 MB in debug mode). The fix is currently available in master channel builds of Flutter that contain the commit https://github.com/flutter/flutter/commit/47302c6ffe0baa12622e9390bd249e0e583cb7cd.


Lapack/types

Closes https://github.com/gonum/lapack/issues/163
Updates #570
Please take a look.

There is one more place in lapack constants where I'd like to do something but don't know what is "best". There are four MatrixNorm constants. NormFrob is the only one with Norm prefix and I don't like this anomaly. So either 1) prepend Norm to the other constants and shorten MaxColumnSum to MaxColSum, or 2) rename NormFrob to Frob, or 3) rename NormFrob to Frobenius. Suggestions? This would be the final nitpick from me on this topic.

On the last comment, I'd be happy for NormFrob to be renamed Frobenius.

This all looks good to me. Let me know if you're happy with it as it stands and I'll approve, otherwise send more here and I'll take another look.

I renamed NormFrob to Frobenius, and unified the style of comments (XXX are computed -> Compute XXX).
Please take a look.


cannot build to cjs using mkdist

Hello,
I was experimenting with unbuild and I encounter an issue.
Is it possible (if recommanded) to build to esm and cjs at the same time using mkdist like with rollup?

I expect to have
    ├── vite.cjs
    ├── vite.d.cts
    ├── vite.d.mts
    ├── vite.d.ts
    ├── vite.mjs
    ├── webpack.cjs
    ├── webpack.d.cts
    ├── webpack.d.mts
    ├── webpack.d.ts
    └── webpack.mjs

but I only got
├── vite.d.ts
├── vite.mjs
├── webpack.d.ts
└── webpack.mjs

Here was my work around:
function dualOutput(
  config: Omit<MkdistBuildEntry, "builder" | "format">
): BuildEntry[] {
  return [
    {
      builder: "mkdist",
      format: "esm",
      ...config,
      pattern: "**/!(*.stories).{js,jsx,ts,tsx}",
    },
    {
      builder: "mkdist",
      format: "cjs",
      ...config,
      pattern: "**/!(*.stories).{js,jsx,ts,tsx}",
    },
  ];
}

export default defineBuildConfig({
  entries: [
    "./src/index",
    ...dualOutput({
      input: "./src/components/Bits/",
      outDir: "./dist/bits",
    }),

Here was my work around for now, just a snippet I use in most the configs:
function dualOutput(
  config: Omit<MkdistBuildEntry, "builder" | "format">
): BuildEntry[] {
  return [
    {
      builder: "mkdist",
      format: "esm",
      ...config,
      pattern: "**/!(*.stories).{js,jsx,ts,tsx}",
    },
    {
      builder: "mkdist",
      format: "cjs",
      ...config,
      pattern: "**/!(*.stories).{js,jsx,ts,tsx}",
    },
  ];
}

export default defineBuildConfig({
  entries: [
    "./src/index",
    ...dualOutput({
      input: "./src/components/Bits/",
      outDir: "./dist/bits",
    }),

Yeah, I'm also using this way but, since there is a way to generate both with the rollup preset, I was hoping to do the same way for mkdist!

Moving docs to https://github.com/unjs/unbuild/issues/398


stan: Flake check that fails if health is not 100%

Another thing worth considering: add a flake check that will fail if project health is not 100%. Poeple can always add excludes to their stan config if there's a good reason to reject certain suggestions.

Originally posted by @srid in https://github.com/srid/haskell-flake/issues/384#issuecomment-2549607956
Demonstrate this on https://github.com/srid/haskell-template

Not sure if this should be in haskell-flake is per se. But a demonstration outside of it would be a good start.

But a demonstration outside of it would be a good start.

That said, having haskell-flake produce a stan.json (through stan --json-output) would be necessary for this.


Stop OmniSharp from autocompleting parameters.

Hello is there a way to stop OmniSharp from completing parameters?
For example when I type string.Format and select any suggestion I get this:

I would like to change this behavior, so that only "string.Format" is autocompleted (without any brackets):

That happens when you have let g:OmniSharp_want_snippet = 1 in your .vimrc - it is used together with UltiSnips for snippet completion. If you are not using snippets, then turn it off.

Thank you!


Why does this require nightly?

Haven't investigated, but would it be possible to get this off of nightly any time soon?
Thanks!

Oh yeah! It uses CommandExt::before_exec which stabilized in 1.15. I wrote this crate before that had shipped, so I literally just need to update the README.


Always copy FSharp.Core.xml on output

We should be copying FSharp.Core.xml when we build so we get the appropriate documentation in interactive.
Related: https://github.com/dotnet/sdk/issues/9498

Duplicate of https://github.com/dotnet/interactive/issues/842?

@KevinRansom, @brettfo  is this now fixed?

It's hard to say.

Always copying it, created an issue for F# developers.  So I made it an opt in switch, I will go through the notebook solution and make sure that we enable it when necessary/

Is the only way to solve the actual issue?


Documentation on constructor properties are not included in output

Search terms
constructor properties
Expected Behavior
Documented parameters within a constructor should also be documented in the output.
Is it possible to also use the doc added to the constructor property?
class A {
  constructor(
    /**
     * I want to see this comment in the documentation
     * /
     readonly property: string) {}
}

Actual Behavior
The generated documentation does include the parameter property but does not include the documentation of the parameter.
I can add a doc to the constructor itself but I can not add the same options as documenting the property directly like
class A {
   /**
     * I want to see this comment in the documentation
     * /
  readonly property: string

}

Steps to reproduce the bug
class A {
  constructor(
    /**
     * I want to see this comment in the documentation
     * /
     readonly property: string) {}
}

Environment
> typedoc --version

TypeDoc 0.23.9
Using TypeScript 4.7.4 from ./node_modules/typescript/lib

> node --version
v16.14.0

OS: Ubuntu 20.04

Seems reasonable to me :+1:


CW Shows not working with -f height

Yesterday and today I was using yt-dlp for two CW shows and its not honoring the height format I have in the config (that work with other shows/networks). Below is the relevant config items.

[debug] User config file: C:\Users\stephen\AppData\Roaming\yt-dlp\config.txt
[debug] User config: ['--write-sub', '-f', '[ext=mp4][height<=720]/[ext=mp4][height<=1080]/[ext=mp4][height<=720]+bestaudio/[ext=mp4][height<=1080]+bestaudio/bestvideo+bestaudio']
[debug] Command-line config: ['-o', 'Riverdale\\Riverdale - s05e11 - Chapter Eighty-Seven: Strange Bedfellows  (Ep.511).%(ext)s', 'https://www.cwtv.com/shows/riverdale/chapter-eighty-seven-strange-bedfellows/?play=764f6359-f0aa-49ff-bf44-894ef31beeb5', '--verbose', '-F']

Here is the formats:
[info] Available formats for 764f6359-f0aa-49ff-bf44-894ef31beeb5:
ID                       EXT RESOLUTION FPS |   TBR PROTO  | VCODEC        VBR ACODEC  MORE INFO
------------------------ --- ---------- --- - ----- ------ - ----------- ----- ------- -------------
hls-AUDIOGROUP-English-0 mp4 audio only     |       m3u8_n |                   unknown [en], English
hls-AUDIOGROUP-English-1 mp4 audio only     |       m3u8_n |                   unknown [en], English
hls-261-0                mp4 416x234    23  |  261k m3u8_n | avc1.42C01E  261k
hls-261-1                mp4 416x234    23  |  261k m3u8_n | avc1.42C01E  261k
hls-261-2                mp4 416x234    23  |  261k m3u8_n | avc1.42C01E  261k
hls-261-3                mp4 416x234    23  |  261k m3u8_n | avc1.42C01E  261k
hls-495-0                mp4 480x270    23  |  495k m3u8_n | avc1.42C01E  495k
hls-495-1                mp4 480x270    23  |  495k m3u8_n | avc1.42C01E  495k
hls-495-2                mp4 480x270    23  |  495k m3u8_n | avc1.42C01E  495k
hls-495-3                mp4 480x270    23  |  495k m3u8_n | avc1.42C01E  495k
hls-665-0                mp4 640x360    23  |  665k m3u8_n | avc1.42C01E  665k
hls-665-1                mp4 640x360    23  |  665k m3u8_n | avc1.42C01E  665k
hls-665-2                mp4 640x360    23  |  665k m3u8_n | avc1.42C01E  665k
hls-665-3                mp4 640x360    23  |  665k m3u8_n | avc1.42C01E  665k
hls-1301-0               mp4 960x540    23  | 1301k m3u8_n | avc1.4D401F 1301k
hls-1301-1               mp4 960x540    23  | 1301k m3u8_n | avc1.4D401F 1301k
hls-1301-2               mp4 960x540    23  | 1301k m3u8_n | avc1.4D401F 1301k
hls-1301-3               mp4 960x540    23  | 1301k m3u8_n | avc1.4D401F 1301k
hls-1907-0               mp4 1280x720   23  | 1907k m3u8_n | avc1.4D401F 1907k
hls-1907-1               mp4 1280x720   23  | 1907k m3u8_n | avc1.4D401F 1907k
hls-1907-2               mp4 1280x720   23  | 1907k m3u8_n | avc1.4D401F 1907k
hls-1907-3               mp4 1280x720   23  | 1907k m3u8_n | avc1.4D401F 1907k
hls-2542-0               mp4 1280x720   23  | 2542k m3u8_n | avc1.4D401F 2542k
hls-2542-1               mp4 1280x720   23  | 2542k m3u8_n | avc1.4D401F 2542k
hls-2542-2               mp4 1280x720   23  | 2542k m3u8_n | avc1.4D401F 2542k
hls-2542-3               mp4 1280x720   23  | 2542k m3u8_n | avc1.4D401F 2542k
hls-5132-0               mp4 1920x1080  23  | 5132k m3u8_n | avc1.640028 5132k
hls-5132-1               mp4 1920x1080  23  | 5132k m3u8_n | avc1.640028 5132k
hls-5132-2               mp4 1920x1080  23  | 5132k m3u8_n | avc1.640028 5132k
hls-5132-3               mp4 1920x1080  23  | 5132k m3u8_n | avc1.640028 5132k
hls-8160-0               mp4 1920x1080  23  | 8160k m3u8_n | avc1.640028 8160k
hls-8160-1               mp4 1920x1080  23  | 8160k m3u8_n | avc1.640028 8160k
hls-8160-2               mp4 1920x1080  23  | 8160k m3u8_n | avc1.640028 8160k
hls-8160-3               mp4 1920x1080  23  | 8160k m3u8_n | avc1.640028 8160k

Manually using -f hls-2542-3+hls-AUDIOGROUP-English-0 gives me the results I expect.
This also appears to be an issue in the upstream youtube-dl project.

This is an issue with your format string. You should use bv[ext=mp4][height<=720]+bestaudio as the third (and 4th) terms. You are missing the bv specifier
Also, assuming you are just trying to get limit your download to mp4+720p, it is easier to just use -S "ext:mp4:m4a,res:720"

This is an issue with your format string. You should use bv[ext=mp4][height<=720]+bestaudio as the third (and 4th) terms. You are missing the bv specifier
Also, assuming you are just trying to get limit your download to mp4+720p, it is easier to just use -S "ext:mp4:m4a,res:720"

Thank you for your response. I'll change my config to your recommendation.
I would like to say that the config I sent worked fine for older episodes (DC's Legends of Tomorrow S6E10 - https://www.cwtv.com/shows/dcs-legends-of-tomorrow/bad-blood/?play=9eec2d27-c9bb-4cc8-ad18-5cb52904eb78) but this weeks (DC's Legends of Tomorrow S6E11 - https://www.cwtv.com/shows/dcs-legends-of-tomorrow/the-final-frame/?play=cb1a208b-06b0-4c9f-aea3-37813540d3e4). I am not going to pretend I fully understand the syntax for the -f flag, but these settings have worked for youtube-dl for years.
All that said, I will go with your suggestion and close this if that solves my issue. Thank you.

If the video has a pre-merged format, then your original selector will work. You can check in the -F of the videos that used to work whether this is the case. Also, if you think the isue is infact caused by yt-dlp, show verbose logs (as well as -F output) of the same video in both yt-dlp and in youtube-dl

If the video has a pre-merged format, then your original selector will work. You can check in the -F of the videos that used to work whether this is the case. Also, if you think the isue is infact caused by yt-dlp, show verbose logs (as well as -F output) of the same video in both yt-dlp and in youtube-dl

It looks like CW now has separate audio and video tracks. When running yt-dlp -F on the links I provided in my last comment it shows a separate audio track.
I was hoping I could keep a generic "720 video resolution, then 1080, or best video and best audio for whatever video" in that order. I have a different system that will act like an RSS feed for TV shows. From there I can generate a consistent -o format (show name\show name - episode name.extension).
It looks like your recommendation of the -S should work, but I haven't tested it yet.

Please close if your issue is solved


Cromemco C-10

Hi,
Really like the device.  Just about to order one and test it out.  I have a HxC and a SuperCard Pro the latter of which I have not used much (as I just got it).  I have a Cromemco C-10 which is an old CPM machine from 1983 era and it had an unusual format in that the first track was single density and the latter ones double density - it was supposedly so their other machines/older machines could always read the first track and decide if it could be booted (it has a marker which tells the computer what format the rest of the disk is -SSDD/DSDD/DSSD/,etc).  Anyway, I was wonder if you are interested in any examples - I have IMG, SCF, HFE (and others since HxC has a nice converter).  Let me know - am sure I could even email some.
Thanks
Brett

Thanks very much!
Mixed-format disks should work right now --- assuming it's an IBM format disk the encoding is autodetected on a per-track basis, so they should Just Work. Writing is harder but can be made to work too (theoretically).
A sample would be nice --- SCP for preference...

If you're still around, we now have solid support for writing mixed-format disks: see https://github.com/davidgiven/fluxengine/blob/master/src/formats/n88basic.textpb So if you're still interested, this can be made to happen.