Allow string literal type alias in $PropertyType

/* @flow */
type Key = 'key';
type T = { key: number };
declare var prop: $PropertyType<T,Key>;

Actual output:
4: declare var prop: $PropertyType<T,Key>;
                     ^ expected object type and string literal as arguments to $PropertyType

Expected: No error

I'm sure that this is a really hard problem, but I wonder if $PropertyType could be extended to work in this case: (From Immutable.js)
declare class Record<T: Object> {
  static <T: Object>(spec: T, name?: string): Class<T & Record<T>>;
  get<K: $Keys<T>>(key: K): $PropertyType<T, K>;
  set<A>(key: $Keys<T>, value: A): T & Record<T>;
  remove(key: $Keys<T>): T & Record<T>;
}

Currently, Flow complains that $PropertyType only a string literal as the second parameter. This one feature (if possible) would make it possible to write the type for every thing that it's currently impossible to write types for.

I've tried to make this work in User Space, but it hasn't worked. The closest I've got is to get the union of all value types in an Object. So $Keys but for values:
type $Object<V> = {[key: string]: V}
type _$Values<V, O: $Object<V>> = V
type $Values<O: Object> = _$Values<*, O>

Using this in the Immutable.js Record type:
declare class Record<T: Object> {
  static <T: Object>(spec: T, name?: string): Class<T & Record<T>>;
  construtor(spec: T): this;
  get(key: $Keys<T>): $Values<T>;
  set<A>(key: $Keys<T>, value: A): T & Record<T>;
  remove(key: $Keys<T>): T & Record<T>;
}

This gives us slightly better results.
import {Record} from 'immutable'

type P = {
  name: string,
  age: string
}

const Person = Record({name: 'John Doe', age: '25'})

var bob: Record<P> = new Person({name: 'Bob', age: '31'})

;(bob.get('name'): string) // no error here
;(bob.get('name'): number) // Flow Error: it's a string not a number

@samwgoldman would allowing a string literal type also allow what @nmn is asking for? Being able to do $PropertyType<T, K> in the general case would be really powerful.

So are there any plans to introduce $Values utility type?
Which takes object type as parameter and gives a union of values.

as a point of reference, Typescript 2.1 introduced the ability to type set functions via K keyof T and T[K]. From their release notes:
function setProperty<T, K extends keyof T>(obj: T, key: K, value: T[K]) {
    obj[key] = value;
}

You can use $ElementType for this! If you are having any issues please let us know 😊
type Key = 'key';
type T = { key: number };
declare var prop: $ElementType<T,Key>;

https://flow.org/try/#0C4TwDgpgBA0hJQLxQOQGt4oNwChSSgBUkoBvKDEALigDsBXAWwCMIAnKAX1wBMIBjADYBDNtABuoqGDYB7MDQAkAUUERGEWsELgIAHkIAaOCAB8uIA

This looks very promising, @calebmer. However, the following code doesn't typecheck for me:
export type SomeType = {
  aString: string,
  aNumber: number,
};

function f2<S: $Keys<SomeType>>(
  propertyName: S,
  value: $ElementType<SomeType, S>,
) {

};

function f1<S: $Keys<SomeType>>(
  propertyName: S,
  value: $ElementType<SomeType, S>,
) {
  f2(propertyName, value);
};

f1("aNumber",  2);

try
If f1 doesn't call f2 it typechecks. Am I missing something here?

$ElementType Doesn't type-check correctly when used as a return-value:
Type checking fails in Foo.getValue(), arguing that the return type of ValueOf is incompatible with each of "number", "string", and "object type" in turn.
type T = {|
  key: number,
  foo: string,
  bar: {}
|};

type ValueOf<K: $Keys<T>> = $ElementType<T, K>;

class Foo {
  obj: T;

  getValue<K: $Keys<T>>(key: K): ValueOf<K> {
  	return this.obj[key];
  }
}

  15:   	return this.obj[key];
                  ^ number. This type is incompatible with the expected return type of
    14:   getValue<K: $Keys<T>>(key: K): ValueOf<K> {
                                         ^ object type
    15:   	return this.obj[key];
                  ^ number. This type is incompatible with the expected return type of
    14:   getValue<K: $Keys<T>>(key: K): ValueOf<K> {
                                         ^ string
    15:   	return this.obj[key];
                  ^ object type. This type is incompatible with the expected return type of
    14:   getValue<K: $Keys<T>>(key: K): ValueOf<K> {
                                         ^ number
    15:   	return this.obj[key];
                  ^ object type. This type is incompatible with the expected return type of
    14:   getValue<K: $Keys<T>>(key: K): ValueOf<K> {
                                         ^ string
    15:   	return this.obj[key];
                  ^ string. This type is incompatible with the expected return type of
    14:   getValue<K: $Keys<T>>(key: K): ValueOf<K> {
                                         ^ number
    15:   	return this.obj[key];
                  ^ string. This type is incompatible with the expected return type of
    14:   getValue<K: $Keys<T>>(key: K): ValueOf<K> {
                                         ^ object type

@calebmer can you elaborate what are the differences between $PropertyType and $ElementType? My little test says there are no differences: https://flow.org/try/#0C4TwDgpgBA8gRgKygXigbwFBSmATgezAC50A7AVwFsSLK4JcBfDZjACjVpIEYAmAZkYkAJAFEANhEoRSwACrgIAHngIANFADkeQpoB8ASgDc7TlR4ChUYQAUCkXKAWQViDdvv7jQA


Document what parallel builds are done on the CI

Currently, whether builds are parallel on the CI is partly a matter of historical accident and partly a matter of performance consideration. We happen to test some parallel builds, but not by design.
The status as of 21 July 2021 is:

Jenkins always does single-threaded builds for all.sh.
Jenkins runs make -j in basic-build-test.sh.
Travis always uses make -j for all.sh (both with plain make and with cmake, assuming that cmake doesn't override MAKEFLAGS).
In 3.0, most builds run from a directory where the generated source files are already present. The one exception is from basic-build-test.sh which starts from a fresh checkout. We had an intermittent failure of basic-build-test.sh due to bad dependencies that only affected make from a fresh checkout, not make generated_files followed by make, which was only revealed through basic-build-test.sh.

Goal of this task:

At least document the status of parallel builds somehow (for example through comments in relevant parts of the Groovy code, .travis.yml, and the .sh scripts), so that when if change things, we'll know that the lack of testing of parallel builds in an issue.
Confirm that parallel cmake builds are actually being tested.
Possibly change the policies. For example, we moved away from systematically using make -j on Jenkins because it caused load spikes that caused concurrently running jobs to fail due to timeouts. But maybe going to single-threaded builds was too far and make -j2 would be a good compromise between test coverage and testing of parallel builds.

As of October 2021, we've moved to defaulting to -j2 rather than -j in all.sh and the CI also uses -j2 on Linux overriding the all.sh default.


wasted horizontal space makes GUI feel cluttered

Zulip wastes considerable horizontal space, making the main chat area in the middle feel cluttered on my 1366 x 768 screen. Notice all the unused space indicated by the red circled areas:

I propose a few solutions to this:

shrink the width of both the left and right panes. In a desktop program a click-and-drag splitter makes it trivially easy for users to customize the space usage to their liking.
add an option to hide the right pane.

@Be-ing thanks for the feedback!  We just merged a few changes to improve use of the sidebar space, but there's actually not as much that can feasibly be done as you think, since there's a scrollbar, unread count, and on-hover menu all in there, and we do want to allow reasonably display of topics of moderate length and names of users with longer surnames.
We actually do have an option to hide the right pane that becomes default-enabled in smaller windows.  I think it would be pretty doable to show that toggle in the default mode for folks who want a really wide text area (though I should note that for actual reading speed, experts generally recommend a relatively narrow space, which is why we by default try to have a giant window).

I suspect what we'll want to do is after this discussion finishes, write up a clear issue for the work of making the right sidebar hideable.
I'd also be OK with making the sidebar sizes draggable, though that may be kinda technically annoying.

Can we reduce the width of the side menus?


Cannot set custom provider for LLM grader override

As per the docs https://www.promptfoo.dev/docs/configuration/expected-outputs#overriding-the-llm-grader
We should be able to use custom providers for the LLM grading.
However, when setting custom provider via any of the methods (defaultTest.options and assertion.provider), the following error is thrown
Error: Invariant failed: Provider supplied to llm-rubric must have an id at invariant (/usr/local/share/npm-global/lib/node_modules/promptfoo/node_modules/tiny-invariant/dist/tiny-invariant.cjs.js:14:11) at getGradingProvider (/usr/local/share/npm-global/lib/node_modules/promptfoo/dist/src/matchers.js:50:38) at matchesLlmRubric (/usr/local/share/npm-global/lib/node_modules/promptfoo/dist/src/matchers.js:118:31) at runAssertion (/usr/local/share/npm-global/lib/node_modules/promptfoo/dist/src/assertions.js:383:55) at runAssertions (/usr/local/share/npm-global/lib/node_modules/promptfoo/dist/src/assertions.js:49:30) at Evaluator.runEval (/usr/local/share/npm-global/lib/node_modules/promptfoo/dist/src/evaluator.js:182:74) at process.processTicksAndRejections (node:internal/process/task_queues:95:5) at async /usr/local/share/npm-global/lib/node_modules/promptfoo/dist/src/evaluator.js:418:25
Providing the id, as suggested by the error does not matter.
The same custom provider works fine when set as the configuration file's "providers" property.
Below is the promptfooconfig.yaml file that reproduces the issue for me.

prompts: [prompt.yaml]
providers: [../../providers/custom.js]
defaultTest:
assert:
- type: llm-rubric
options:
provider: [../../providers/custom.js]
rubricPrompt:
- role: system
content: >-
Grade the output by the following specifications, keeping track of the points scored:

        Did the output include code? +1 point
        Was the output in markdown format? +1 point
        Is the output relevant to the {{ query }}? +1 point

        Calculate the score and pass the test if total points is greater than 2.5.
        Output must be in the following JSON format:
        {pass: true, score: number, reason: string}
    - role: user
      content: 'Output: {{ output }}'

tests: tests/user-prompts.csv

The workaround I have found is to set provider as a string.
Setting the provider as object and providing id as below, does not work.
defaultTest:   assert:     - type: llm-rubric   options:     provider:        ../../providers/custom.js:         id: 'evaluator'

Hi @AnkitTheCoder, a bit late to this issue but the correct setup would be to set provider: ../../providers/custom.js, as you found.  The provider property on Assertion is a string.


After successful payment, status does not update

Hi,
I have been using the plugin for a long time and never had a problem, somehow 2-3 days ago there was a problem that the order statuses do not update after a successful payment. Nothing has been changed for a long time.

Same problem here, but in sandbox. Status not updating after choosing payment approved in sandbox.

It is a problem with WAF on server. Probably blocks PayU IP (<IP_ADDRESS>/24, <IP_ADDRESS>/24).  Please contact with hosting company.


Weird error w/ wave

I was testing a wave i made, and the game spit this error back at me:
Error at src/utils/utils.lua:1535 - attempt to perform arithmetic on local 'y1' (a nil value)
I can give extra details if prompted, but I'll be honest, I've been looking for an excuse to go to bed and this seems like a good one so I'll respond in the morning, Thanks!

Can you show the script/code? Likely you forgot to set y1 value when referencing it

Closing due to lack of response.


Purge deletes non-empty folder

Describe the bug
Purge deletes non-empty folder
To Reproduce
Steps to reproduce the behavior:

Create a folder
Add a STL to the folder
Scan
Remove STL from the folder
Scan
Purging folder from "problems" deletes the folder and all of it's subdirectories, which may contain other models!

Expected behavior
Purge the database entry for the folder, not its contents.

Oof, it shouldn't be doing that if there are other models in there! Thanks for flagging it up, I'll check.

Wait, I misunderstood. Purging a missing file removes the whole folder and subdirectories??

I'm afraid I can't replicate this locally; can you provide any more details? Like, the exact contents of the model before purging, a screenshot of the button, etc?

Here's a better description of the issue:
Armor
├── Shoes.stl
├── Shoulders
│   ├── 1.stl
│   ├── 2.stl
│   ├── 3.stl
├── Chest
│   ├── 1.stl
│   ├── 2.stl
│   ├── 3.stl

Delete or move Shoes.stl:
Armor
├── Shoulders
│   ├── 1.stl
│   ├── 2.stl
│   ├── 3.stl
├── Chest
│   ├── 1.stl
│   ├── 2.stl
│   ├── 3.stl
├── Feet
│   ├── Shoes.stl

The "Armor" model that contained Shoes.stl now appears as empty in issues and is given the option to "Purge". If you do purge, Shoulders, Chest, and Feet are deleted and there's no other way to remove the Armor model.

Ah fantastic, I'll give that a try in order to reproduce it.

I've got this replicated now, should be fixed soon.


Ensure that the GDF file variables are updated after running /update on startup

Currently, the code will set a boolean variable update_needed when the files don't exist on startup, but there isn't any code that then sets those variables after starting the server.
I think that after the application updates, we want to run code to resolve that.

This is no longer a valid issue ever since we removed the need to download shapefiles locally to generate JSON versions of them. We point all our requests for authoritative information at Geoserver now.


Dataset Viewer issue for hungnm/multilingual-amazon-review-sentiment-processed

Link
https://huggingface.co/hungnm/multilingual-amazon-review-sentiment
Description
No response
Owner
Yes

It seems like a private dataset. The viewer is currently not supported on the private datasets.


channeldb/kvdb/bolt: automatically compact database on startup

Fixes https://github.com/lightningnetwork/lnd/issues/4131.
Can be turned on with --db.bolt.autocompact in the command line or db.bolt.autocompact=true in the configuration file.
This is a preview to test if this is feasible. The following TODOs remain:

Make sure compaction is rolled back if something goes wrong
~Check that enough disk space for worst-case compaction (2x original size) is available~
~Add a progress indication~
Add integration test

Added a free disk space check and a progress indication.

Windows cross compile is failing:
Error: ../../healthcheck/diskcheck_windows.go:30:2: not enough arguments to return
	have (uint64)
	want (uint64, error)

Alright, caught up with the latest version of this PR and also compacted successfully on a few nodes, here's a sample ruin:
2020-11-04 18:49:48.096 [INF] CHDB: DB compaction of /root/.lnd/data/graph/testnet/channel.db successful, 196567040 -> 143085568 bytes (gain=1.37x)

I think this is ready to leave the draft mode now! Will continue to test it in the background re more aggressive measures like killing mid compaction, etc.

I added two changes that aren't directly related to compaction:

Compile for every architecture on Travis to make sure all architecture dependent compilation succeeds.
Remove darwin-386 from the list of target architectures as it isn't supported by go 1.15.x anymore.

Let me know if I should split those off into its own PR. Since they're relatively small, I didn't want to add another dependency. But if the changes themselves need discussion, it's probably better to split them off.

Cross compilation failure:
# github.com/lightningnetwork/lnd/healthcheck
Error: ../../healthcheck/diskcheck.go:30:18: invalid operation: s.Bavail * uint64(s.Bsize) (mismatched types int64 and uint64)

Fixed the cross compilation issue, should be good to go now.


Support international utf-8 chars in param values / names

Hello!
international letter chars are not supported -
var Route = require('susanin').Route;
var route = Route('/<foo>/<bar>/<baz>');
console.log(route.match('/esc/rzy/aie')); // { foo: 'esc', bar: 'rzy', baz: 'aie' }
console.log(route.match('/ěšč/řžý/áíé')); // null

Using xregexp library (its unicode support - http://xregexp.com/plugins/#unicode) seems to be possible workaround.

+1
why parameter values are not [^\/]+?
https://github.com/nodules/susanin/blob/master/lib/route.js#L76

As for me, using the XRegExp library as a general solution for the matching is not suitable for the router which targeted to be fast.  On first sight the better solution is to provide interface to setup custom matching functions – then you can use whatever you want to match routes.

By following rfc3986 the list of allowed characters in URI is this.
I think there are three solutions:

do nothing, because we can do like this;
use [^\/]+;
add % to regexp and decode values by decodeURIComponent, due to request to /ěšč/řžý/áí should be replaced to /%C4%9B%C5%A1%C4%8D/%C5%99%C5%BE%C3%BD/%C3%A1%C3%AD%C3%A9.

What do you think, which one is better?

Maybe for now we add basic support for cyrillic like this:
var PARAM_VALUE_REGEXP_SOURCE = '[\\w\\-\\.~\u0400-\u04FF]+';


Checkout caches Total for old products' list

What I'm trying to achieve
Receive correct value for Grand Total
Steps to reproduce the problem

Add product to bag
Go to the checkout and choose a shipping method
Go back to the product list and add another item
Go to the checkout -> GRAND TOTAL will show old price (first item + shipping)
Refresh the page to recalculate grand total.

What I expected to happen
Checkout shouldn't cache these data.
System information
Operating system: Mac
Browser: Chrome

On successful add to cart checkoutContext should be also updated with totalPrice

https://github.com/mirumee/saleor-storefront/blob/a98138786e9c9560d8f76cd40a0a45f66ca21d7a/src/checkout/queries.ts#L162-L182
Yes, the query should have field totalPrice after subtotalPrice

Any updates on this?


socials for Sen Kamala Harris - CA

Hello and thanks.  These appear to be the campaign accounts, correct?  Unfortunately this repo only tracks the official accounts (i.e. paid for by government funds).

closing, please create a new one if you find the official social media accounts


Added responsive model implementation with simpler sliding sidebar code

I have fixed the necessary changes and I am up to date with the current master. Also I have refactored the slide sidebar to simpler code and responsiveness of bootstrap modal.

@ishan-me the problem is not solved, maybe you can use a simpler way to delete your fork and then start working on this.

@LalitNM I still wonder the task was to make a responsive modal and a working sidebar. I have addressed both the issue. Can you specify me the problem please?

You have to see the files changed in this PR, the only thing which is changed is the left-bar-toggle button's JS code, and now sidebar is not working here. And another thing you have to do is clean your git history, there are still some commits which are already there in our master branch, this is what I meant

Closing due to inactivity, if you still want to contribute then please make a new PR.


Feature - Realtime link to dashboard and detail view?

Feels a bit cumbersome right now to navigate to the Realtime view of a page. Select tab from header, and then certain page.
Header tab for Realtime would make more sense if there would be an overview of all my pages in real time...
Could me more natural to select Realtime view from dashboard and from detailed view itself?
Two buttons in dashboard maybe? One for details and one for real time? And then just set the state for selected view.

This totally makes sense.

Real-time button is now on the details page.


Use helm versioning properly

"new changes will continue to be merged to develop branch without incrementing chart version"

@rockb1017 is asserting (in issue#355 that he closed) that he is happy for enterprises to install new breaking changes even when customers don't increment the chart version.  This issue is not requesting every change to this entire repo should result in a chart version increment, just changes to the respective chart.
If Splunk Inc are breaking helm chart best practice and placing their customers at risk of introducing bugs, then this "won't fix" decision should be clearly stated.

Hello @holmesb
Thank you for your concern. I would like to be clarify some things.

Every released chart will have chart version incremented. And installing released chart is how we documented for our users on README. (either through github release artifacts or using helm repo
Even if users are using the local clone of this git repo, as long as they use main branch, it doesn't violate helm chart best practices because only when it is released, it is merged to main.

Was it clear?

If I install chart v0.0.1, you push a bug to your helm repo, and I then helm repo update and install v0.0.1 again, I'll install your bug.  We'll just have to take it on trust that you won't "helm push" or "helm repo index" between releases.
"You can deploy a different chart with the same version as the previous one"..  This is why it's best practice to increment the chart semver each chart change.

Updating helm repo will only be done through the release pipeline when we release.
I will close this. if you have more feedbacks, please leave them here.


Update to Boostrap 4?

Thanks for the template. I've been trying to integrate it with an Elm front-end, using Elm's bootstrap packages to output the template. I'll make it more dynamic in future, but for the moment, just trying to output the exact template.
I've run into issue though as this is a Bootstrap 3 template, and Bootstrap 4 is latest, so some of the CSS in the new boostrap version takes precedence over this template's custom CSS and they styles aren't applied.
Any chance of a Bootstrap 4 update, is that on a roadmap? Thanks.

Done with #11


cmd/connect: return exit status of --execute flag

This PR improves the pscale connect commands --execute flag. It now
captures the exit code of the command passed to the --execute flag and
exits with the returned code.
I wrote a simple tool that exits with the code passed to the tools. As you see here, it exits with 0 or 2 and we successfully capture it:
$ pscale connect planetscale main --execute 'demo 0'
Secure connection to database planetscale and branch main is established!.

Local address to connect your application: <IP_ADDRESS>:3306 (press ctrl-c to quit)
exiting with 0
$ echo $status
0

$ pscale connect planetscale main --execute 'demo 2'
Secure connection to database planetscale and branch main is established!.

Local address to connect your application: <IP_ADDRESS>:3306 (press ctrl-c to quit)
exiting with 2
Error: running command with --execute has failed: exit status 2

$ echo $status
2

closes: https://github.com/planetscale/cli/issues/283

Very nice indeed, thanks @fatih.


Error on pip install -e . in the Jupyterhub

Getting an error when installing from pip on the jupyterhub probably because warnings is a built-in
Singularity> pip install -e . Obtaining file:///nsls2/users/bpatel/BP_LocalCodeBase/rsoxs_scans   Installing build dependencies ... done   Checking if build backend supports build_editable ... done   Getting requirements to build editable ... done   Preparing editable metadata (pyproject.toml) ... done ERROR: Could not find a version that satisfies the requirement warnings (from rsoxs-scans) (from versions: none) ERROR: No matching distribution found for warnings Singularity>

Also operator and copy are default packages which give the same error


More example input

wf02eq, UK output area to workplace zone, about 16 million rows
wu03ew, UK MSOA to MSOA, about 2 million rows
Soundcast, completely disaggregated individuals (about 4 million people?)

https://www.ons.gov.uk/peoplepopulationandcommunity/populationandmigration/populationestimates/bulletins/origindestinationdataenglandandwales/census2021 just landed!


FP-30: piano.write_register(RolandAddressMap.metronomeStatus,1) not working

Hi,
Thanks for the amazing repo (both this and RolandPianoControl).
The code works perfectly when it changes the volume of the piano (read and write). It also works well when reading the data inside the registers.
But when I would like to turn on the metronome with code, it doesn't work.
Only the write method doesn't work, read method returns the correct value when I turn on and off the metronome manually on the piano.
Can I ask you for any ideas on this?
Also where did you find the address map for the piano? I checked the roland midi implementation for both fp-10 and fp-30 and I haven't found anything on metronome for example. May I ask for the resources that you use for this project? Mainly on the midi implementation of roland?
I would like to create a similar project: ESP32-S microcontroller based webapp, that controls the piano through bluetooth.
Also if there are any way I could contribute to this project please let me know. :)
Thanks,
Marton

Hi Marton, thanks for your interest in this project!
For some reason I did not see that someone left a comment. I will try to set-up notifications correctly :). It has been a while ago since I implemented the API's, but I think the metronome was working on my end. I will do some testing and will hopefully report back with a working example.
The address map was obtained by unpacking the Android application that is provided by Roland. From here I also deduced a part of the command sequences.
There are still some mysteries to resolve, like pushing setting changes from the piano itself (right now I need to poll), so feel free to contribute to this project!

I did some work on the repo today. The metronome should work now, just look at the examples. :)

Btw, which command did you use to read out the status of the metronome?


Fix typos and update copyright

Fixing typos in the code and update copyright lines.

@avocadoboi ping?

Appreciate it! Thanks for teaching me about the c in acquire, it's one of those blind spots :)

Appreciate it! Thanks for teaching me about the c in acquire, it's one of those blind spots :)

Hehe no problem! I'm actually not a native English speaker to be honest.


fixed defect #159

Fixes #159
Changes proposed in this pull request:

Parse using LocalDate and then convert to ZonedDateTime since data is in yyyy-MM-dd format

Thanks.


Add a Mastermind Game

[ ] Find the algorithm to be used
[ ] Test it before deploying

Added random integer guessing function.


Feat/update metadata per field

Created seperate update subcommands to support updating just the name, symbol, creators fields

The API and the respective test Tx are as follows

Name

TX - https://solscan.io/tx/5eL2aq3DFMuseuhoAHhhXQhva1wpT86x6iwDHqMBsoSXzgpyJux1LVpje7N7DUoCXxhzznYtbmV7F69oWmJMcyUp?cluster=devnet

Symbol

TX - https://solscan.io/tx/24mHU9ywHeFPuXB3J7J6y3XfCr59DEFvWBjCXK1q67jHiaKpX3cegjZPzTUrjQFhAnCAMFMSWYX53hxSLTFXA6rs?cluster=devnet

Creators

TX (With custom position of 1) - https://solscan.io/tx/4v3LyzLzntoDxnjJQsWPvwRa6ebAoZkk8dhWwNXMk4xKY3PAnzFbA4MJJxfRdPtR5hHwy1mDMcViVPhvW4N8qm3P?cluster=devnet

TX (Without custom position) for index 0 - https://solscan.io/tx/3fBe3o4dmHJqiPZBcSdq8U6iGzyztFLm8bia7TYGPzTVy7HQ8GcyZMZqcsvbZf9r6oMnpHSxwQyGmbDcBT2qRUUx?cluster=devnet

Awesome, thanks! I will review this tonight.

Aye aye :)

This is great work! There are just a couple of things to address and then it should be good to merge.

Take a look at the changes I made and let me know if you have any questions.

Removed references on numbers, as primitives like usize are copied freely on the stack with no penalty
Changed u64 to usize to avoid the frequent casting
Fixed off-by-one issues with the creators array length and the position
Push creator to end of array instead of trying to index into a non-existing position

Once you're happy with the changes I'll merge.

This is perfect :) I thought we had like a mandatory requirement to just use the primitive u8 and u64 haha. Im more than happy with these filters


Current state?

Hi there,
I would be interested to help with this to learn more about Rust on embedded. However, it doesn't look like there is active progress here at the moment. Are you expectingg this project to take up activity again, or is it stalled for now?
And if I were to try to continue here. Which issue or PR should be the first thing to look into? I see there are two draft PRs. I would assume, they need more work to get merged? If so, what is missing?
Thanks :wave:

Hi,
thanks for reaching out. Yes most development has stalled a bit. If you would like you could complete my PR for the Clock-Tree: https://github.com/rp-rs/rp-hal/pull/18
I finished the wrappers for XOSC and PLL but since than I could not find time to continue. You could pick up where I left of or rewrite it to be simpler. If you want I could help with code review.
My initial idea was to pass around the ownership of the clock sources via structs but it turned out to be to complicated. So I would maybe remove all the code that is not part of the XOSC or PLL modules.
Just let me know if I can help you in any way :)

just so you know there is now an open PR for the XOsc peripheral by @Nic0w: https://github.com/rp-rs/rp-hal/pull/25

I've got GPIOs mostly working, and there's that clocks PR that is also making good progress.
Once those two land, I think we can start hitting the many different peripherals. Some will likely be easier than others (e.g. SPI vs PIO).

Hey there !
So yeah there's my PR on the XOSC and my end goal was to have UART working. So I've also been working on PLLs as well as Clocks, and of course UART.
All of those things have been working for me for two days now \o/ so I'm quite excited.
I'm most happy with the code for the PLLs and the UART; I could push those next, and also possibly clocks but it's a big chunk of code that would need some polishing and discussions.
I've also been working on GPIOs and as @Nitori- I tried to keep thing simple by focusing mostly on SIO but I've also got basic function switching (needed for UART).

I'm most happy with the code for the PLLs and the UART; I could push those next, and also possibly clocks but it's a big chunk of code that would need some polishing and discussions.

Feel free to open a draft PR for the PLL and UART. I would be happy to have a look at them :smiley:

here you go for the PLL, UART is on its way as well :)


Can you loop over dynamic imports?

So the problem I am having is writing this repeated code 30+ times in my entry files and it makes my entry file pretty big.
Example:
const component = document.querySelector('[data-component]');

if (component !== null) {
  import(/* webpackChunkName: "component" */ 'path/to/component').then(componentModule =>{
 componentModule.default.init();
})
    .catch(err => console.error(`Error in: Samples - ${err}`));
}

And that fine, my webpack bundle creates component.bundle.js but when I tried to implement this for looping I could not get it working. Here is my approach:
const components = {
  component1: {
    el: document.querySelector('[data-component]'),
    fileName: 'component',
    path: '/path/to/'
  }
};

function importModule(el, fileName, path) {
  if (el !== null) {
      import(/* webpackChunkName: "[request]" */ '${path}${fileName}').then(componentModule =>{
        componentModule.default.init();
      })
        .catch(err => console.error(`Error in: ${fileName} - ${err}`));
  }
}

Object.keys(components).forEach(key => {
  const { el, fileName, path } = component[key];
  importModule(el, fileName, path);
});

The first error I get is the file extension. I have a cshtml file named the same as my component for consistency and it loads that in instead of my js file. This works fine when I am not using a template string.
The second one webpack does not know what bundle file to generate because it is in a loop.
is this possible? My bundle would end up being about 3kb if it could be possible therefore making a performance improvement.

It looks like you just deletedour issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template.


[ENH] Defaults for _predict_interval and _predict_coverage

This PR implements defaults for _predict_interval and _predict_quantiles, as proposed in #1877 - for discussion (not necessarily as final version). This does not yet contain defaults in terms of _predict_proba, since we have not settled on output format yet.
It is partly based on work in @k1m190r's PR #1842 and @kejsitake in #1579, behaviour moved to base class default implementations.
Behaviour implemented:

if _predict_interval  is not implemented but _predict_quantiles is, it is called and outputs are reformatted
if _predict_quantiles  is not implemented but _predict_interval is, it is called and outputs are reformatted
the check whether sth is implemented is done by a utility added to BaseObject, a method _has_implementation_of, which is based on @k1m190r's utility that checks whether the forecaster has been refactored, which was based on @kejsitake's and some discussion between the three of us.

Further, the following forecasters were relying primarily on _predict_quantiles but interfacing interval prediction logic. This has been changed by tidying up the interface, implementing a leaner _predict_interval, and removing _predict_quantiles:

_PmdArimaAdapter
_TbatsAdapter

Additional, minor changes:

docstring fixes in BaseForecaster
multiple minor instances of clean-up in the forecasting extension template

(breaking example due to different format in _predict_interval)

I think that this is a good idea because it reduces code repetition in the inheriting forecasting classes, as this methods don't need to be re-implemented. It also looks like `predict_interval' bypasses the floating issue (discussed in PR https://github.com/alan-turing-institute/sktime/pull/1842) by using the user input-ed alpha.

ok, so what I´d propose is that we first merge @k1m190r´s PR and then I try to make this work.
How about https://github.com/alan-turing-institute/sktime/pull/1874 ?
Perhaps it´s also good if we finish that first?

(only tests failing are for Prophet proba return - which we had missed in #1687)

@kejsitake, @k1m190r, I've used the "default logic" to refactor the _PmdArimaAdapter, which now relies on the _predict_interval and not on the _predict_quantiles function. All boilerplate moveing between interval and coverage is now in the defaults. Would be appreciative of having your eyes on this, since both of you have implemented similar code.

PS: thanks for your earlier work which enabled/inspired this refactor! If you hadn't arrived at the same kind of solution independently, it wouldn't have been so clear what to factor out!

FYI @SveaMeyer13 as well

merged after self-review, as agreed with @TonyBagnall


I changed libraries to requirements in the setup file

Coverage remained the same at 76.866% when pulling 31fc98bfc0286fff17f3fe6c28fbc490a3fce1e2 on fixRequirementsSetup into 2b638b8a3291040826e3d7c1b69542b9aae89fb0 on master.

Coverage remained the same at 76.866% when pulling e98e5b508a834ea1004dea976bc191ab0a74a489 on fixRequirementsSetup into 2b638b8a3291040826e3d7c1b69542b9aae89fb0 on master.


Blazor razor code highlighting doesn't work well

Blazor razor c# code highlighting is a mess. Everything showed in the same color. Also there is formatting problems for generated code.

@ryanbrandenburg Please help us here as well. :)

@nohwnd Currently I don't even have an error linting or warning for razor code.


Dynamic concurrency throttles

Concurrency settings are currently static and default to the number of cores for both activity tasks and orchestrations. For simple "hello, cities" scenarios, this often results in underutilized CPU, based on recent performance tests using Azure Functions.
The Azure Functions runtime team is working on a dynamic concurrency throttle feature that would dynamically determine an ideal concurrency limit based on a variety of factors, including CPU usage, memory, and possibly network connectivity. This work requires individual triggers to participate in the throttling behavior to ensure messages don't get picked up and then throttled in-memory. This issue tracks integration with this Azure Functions throttling feature, when it is available.
In the case of DTFx apps, an extension point will be made available to allow for custom concurrency throttling implementations.
NOTE: It's not yet clear whether throttle participation work would need to happen in the Durable SQL provider or if can happen in the core Durable Task Framework dispatcher, in which case different backends benefit automatically. If it's the latter, then this issue can be migrated to the Azure/durabletask repo.

@cgillum , is it still an open issue?
The documentation states maxConcurrentOrchestratorFunctions and maxConcurrentActivityFunctions are supported.
https://microsoft.github.io/durabletask-mssql/#/scaling?id=concurrency-configuration
However, my testing shows that CPU is still under utilized and changing these settings doesn't seem to have any effect.


Ability to hide or rename EN/NL locale values

Currently in the language switcher there isn't a way to override the EN (or NL) values. We have a second language (FR), we can still leverage the NL translation ability by using French translations instead.
There doesn't appear to be a way to override the values from https://github.com/OpenConext/OpenConext-engineblock/blob/master/theme/material/templates/layouts/scripts/language-switcher.html.twig  (or hide the toggle entirely).
Would this type of functionality be of use for EngineBlock? If so, I can look at updating the needed fields and create a PR for it.
Thanks

You can hide the toggle pretty easily:
.comp-language {
    display: none
}

And there was a discussion on the mailinglist recently regarding the addition of other languages:
https://groups.google.com/forum/#!searchin/openconext/language$20to$20engineblock|sort:date/openconext/7xAEdW6vPAU/nGECBoSXAgAJ

Indeed, currently if I'm correct not just the template but also other pieces of code assumes the two languages are EN and NL.
I'd certainly be open to a change that would make this more configurable. Indeed as Tim indicated, it might be beneficial to work with Domingos on this?

Thanks Thijs.
We're not as concerned with the values used internally by EngineBlock (i.e. lang=nl in the address bar) as we are with what's displayed on the page to the user on the page itself (at least for now). Ideally we'd have it configurable with OpenConext-deploy, but the quick fix for now is to fork the EngineBlock repo, modify the theme and then use the custom theme during deployment.

We've discussed this and we'll make a configurable list of languages (e.g. "en,nl,pt") in each product; if the list has only one value no choice will be offered. All code that does something with langues (e.g. if isset displayname:en then ... elseif issuet displayname:nl ... etc) will be changed to loop over the configured languages instead. The value for this config option will obviously be settable centrally in openconext-deploy.

Perfect thanks! That sounds like it would work well for any number of languages offered.
This is what we're currently running with Ansible, which seems to do the trick (for now):
- name: Patch NL to FR
  lineinfile:
    path: "{{ openconext_releases_dir }}/OpenConext-engineblock/theme/material/templates/layouts/scripts/language-switcher.html.twig"
    regexp: '^(.*)>NL<(.*)$'
    line: '\1>FR<\2'
    backrefs: yes

The previous Twig templating work SURF did with EngineBlock has made it way easier to customize EngineBlock. Thanks!

Since EB 6.2 this is configurable in parameters.yml, setting enabled_languages


8-29-2022 Intro Python, Jupiter, Fastpages Hacks

Ethan Truong's Review Ticket, Mr. Mortensen Period 3
I added dark mode to my fast pages website, I added Links and Directories, and added a table of my class schedule for my school year to the front page.
Link to my Fast Pages Website
Link to Links and Dictionaries

Review Ticket: https://srihitakott1213.github.io/APCSP-Blog-Assignment/

Jeffrey Lee (Team Peer Grading)
Grade: 2.7
Ethan customized his fastpages well with evidence of dark mode minima change, images, and text to develop his front page. Also has elements of html table with his class schedule, very productive. No evidence of liquid syntax, and his lists/dictionaries page was complete with necessary code, a bit lacking on the markdown. He asked questions to the team but overall could have been a slightly better team member.


Aspect ratio

Please add the support aspect ratio for cropping pictures. (Like the Snapseed app)

Will try to add this as soon as i can

Thanks 🙏

Done

Thanks 🙏
Please release a version. 🤩

Will be soon

Thanks 🙏


Interrupt example

I'm trying to write an interrupt that let's say will fire up using TIMER0 every second. I understand that I need something like the following function
#[interrupt]
fn hello() {
  // Do something
}

And call enable_interrupt on TIMER0. What I don't understand is how to tell which interrupt to fire. What happens if I add another interrupt function that I want it to fire only on TIMER1?

The function name shall be named after the interrupt. See the following example,
https://github.com/nrf-rs/nRF52840-DK/blob/master/examples/blinky_interrupt_rtt.rs


mAP or mAP50

I found that it was written like this in the article. Is the evaluation index used in your experiment results AP (0.5:0.95) or AP50 (iou=0.5)? The term ‘mAP’ used in the article is a bit confusing.
Thanks!

Thanks for your interest! The evaluation metrics is AP50.


You're closing the HTML tag to early here

https://github.com/WillisamJ/PulseChecks/blob/75e2a0baf028f1d70e2a469fb38aad915b67bde7/index.html#L16

Resolved!


Editorial: Make all AOs take Intl.Locale and add standard preamble

address  https://github.com/tc39/proposal-intl-locale-info/issues/41

also address https://github.com/tc39/proposal-intl-locale-info/issues/40

and address https://github.com/tc39/proposal-intl-locale-info/issues/18

@anba - could you also take a look?

ping


Universal Actions for Adaptive Cards - Action.Execute in Teams and Outlook?

Discussed in https://github.com/MicrosoftDocs/msteams-docs/discussions/8445

Originally posted by brianpmccullough April  6, 2023
"You can now send the same card to both, Teams and Outlook, and keep them in sync with each other using the underlying bot. Any action taken on either platform is reflected to the other with this build once, deploy anywhere (Universal Actions for Adaptive Cards) model." - is there a sample/walkthrough/blog that shows you can use the SAME card in both Teams and Outlook to capture data from a user as outlined in the "Universal Actions for Adaptive Cards" article: https://github.com/MicrosoftDocs/msteams-docs/blob/f70fe776245e04b89703c9e60621b95134e05380/msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/Overview.md
As far as I can tell/test, it looks like it's still only possible to use Action.Http within Outlook?

@brianpmccullough, Before the Universal Actions for Adaptive Cards, different hosts provided different action models as follows:

Teams or bots used Action.Submit, an approach which defers the actual communication model to the underlying channel.
Outlook used Action.Http to communicate with the backend service explicitly specified in the Adaptive Card payload.

With the Universal Actions for Adaptive Cards, you can use Action.Execute for action handling across different platforms. Action.Execute works across all hubs, including Teams and Outlook.


Home stream has state:bad element error at times

Some users have reported seeing an error on the home timeline while its loading. Something is going wrong in its initialization perhaps sometimes.
There is no particular way to reproduce this, though Ive a feeling this has something to do with the OBPostsStream widgets initialPosts state. Sentry is more helpful
https://sentry.openbook.social/okuna/okuna-app/issues/12594/?query=is:unresolved no element

Oh thought this had been fixed 🤔


Remove XA recovery test.

Pull Request Test Coverage Report for Build 5646

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

Totals

Change from base Build 5645:
0.0%

Covered Lines:
8311

Relevant Lines:
12079

💛  - Coveralls


Update VDF Entry with missing fields

Add the appid field and fixes the name of the exe and appname to use the same capitalization as expected by Steam
In the following binary comparison, it is possible to notice that the appid is removed from a file when it is re-serialized, and that the capitalization of some of the fields are changed, this probably interferes with what Steam expects, causing it to re-calculate the appid of an pre-existing shortcut, losing the connection to custom artworks in the library after the re-serialization done by projects using this library.

Hey @BrianLima would you like me to transfer this repo over to you?
You'd need to keep its license as is, perhaps appending an extra "Copyright" line to it (that line should appear before the one already present) would do the trick in order to maintain MIT compliance.
Copyright (c) 2021 Brian Lima

Let me know in case you are interested, and I can start the transfer process right away.

That would be nice, sorry if the occasional bug fix on sundays is bothering you 😇

Nah, PRs are always welcome. It just makes sense to transfer, specially since I don't intend to touch C# again hehe
You should receive an email in a few minutes.
Thanks!

Uh oh. I'll merge this PR, then you will need to delete your fork.
Let me know once it's done, so I can transfer it. :)

@heyvito done and done!

Awesome! Transferring now :)

Yay! Thank you, Brian!


Travis run green but getting stuck

Recent Travis runs have been getting stuck/timing out after 15-20 hours but are still showing as green:
https://travis-ci.org/travis-infrastructure/terraform-config/builds/255390479

This seems to be resolved on master.


This test shows that the get_changes() function returns several empty changes for various transaction scenarios, currently believed to be in error.

I think I should mention that the behavior is the same if there is a commit message or if that is Null

I believe that empty commits are an intended feature: see https://github.com/automerge/automerge-rs/blob/aaddb3c9ea84960924f6c68ca54f0af33a0b31dd/javascript/src/index.ts#L350-L361

It is valuable to be able to generate empty changes for cases like the docs mentioned above. However, I think this should probably always be an explicit user action. I'm not sure what a good API for this is. We can obviously just modify the return of Transaction::commit to return an Option<ChangeHash> - I'm having a play with that.

I was thinking a flag on CommitOpts might be appropriate.

I actually think this might be something a bit different to a flag on CommitOptions. I think the only scenario when you want to create an empty commit is when you are creating a merge commit. In this case you know that the ChangeHash will exist. This suggests to me that we want a separate method on Automerge and Autocommit to create an empty change and return the hash of the change. Then commit can just return an Option<ChangeHash> and always return None if there was nothing to do. This means the semantics of the return value of commit are easier to reason about because you don't have to also know what the CommitOptions were to understand what Some means.


Retry triggered on success?

Overdrive: 0.2.1
Package manager (SPM, Carthage, CocoaPods, Manual): Cocoapods 1.1
Xcode: 8.2 beta 2
Platform: iOS 10
Running a simple, non-failable task with retry, retries even if the task finishes without error.
let inlineTask = InlineTask { (_) in
    print("inlineTask")
}.retry(2)

queue.add(task: inlineTask)

prints inlineTask 3 Times to the console, even if the documentation of retry states:

If the task finishes with error, it will be executed again until retry count becomes zero.

Please, checkout my fix in #5

Thanks for tracking this one down. Finish logic still needs some work and this is a step in the right direction.


Move transitionTo, replaceWith to history

This makes integration with flux much easier. If you have access to the history instance, you can invoke transitions with flux actions like this:
import history from './history';

export function loginSuccess(user) {
    return dispatch => {
        dispatch({
           user: user,
           type: 'loginSuccess'
       });
       history.transitionTo('/dashboard');
    };
}

A custom stringifyQuery can be passed to the History constructor like:
const history = new HashHistory({ stringifyQuery: myStringifyQuery });

History instances are not responsible for stringifying queries, only parsing them. The router is responsible for stringifying. This is important so we let histories only deal with paths, instead of modifying all of their methods to deal with both pathname and query.


BCLib keeps crashing my client, but only one versions 1.3.0 and above.

I'm trying to use BCLib for both betternether and betterend on 1.18.1.
When I try to load the game with version 1.3.0 and above (required for betternether), my game crashes with exit code 1.
When I try to load with with 1.2.1, it works just fine.
Let me know if I need to send in log files and if so, how I can do that. I am very stressed over this.

You need to use same version of library as mod require.  If it is possible please upgrade to 1.18.2


[QUESTION] Whether hive can execute multiple SQL statements

Whether hive can execute multiple SQL statements
version: 1.3.0

Use ';' to separate

it's not work

Whether hive can execute multiple SQL statements
version: 1.3.0

Currently ,we can't execute multiple SQL statements. Because the SQL is executed by jdbc Statement object, and each Statement can only run one SQL.
If you want to execute multiple SQL, you'd better merge them into one, which looks like
insert into emp(id,name,age)  select '1','name-1',22 union all  select '2','name-2',12 union all  select '3','name-3',18


update http and uuid dependency

Thank you for the update @The-Redhat

Hey @santitigaga I updated the github actions pipeline flutter version. It should work now

Great Job @The-Redhat

@santitigaga are you planning to release a new version to pub.dev soon?


Inspector - Add option to email reports as attachment

Add option to email reports as attachment

To do: Improve inline css.


More types

On demo we see just two types of "hamburguers", arrow and cross...
So, we can add more? Ideas?

@francesco2you Which demo are you referring to? There are currently 26 types: https://github.com/jonsuh/hamburgers/tree/master/_sass/hamburgers/types
You can see most of them on the site: https://jonsuh.com/hamburgers/


Update status parameters

replicate jon's python client changes from https://github.com/alphagov/notifications-python-client/pull/73/files
except:

removed failed from letter statuses because it doesn't make sense to query by failed when there's only one failure type
tweaked wording a bit to make it clear what the states are, since accepted is quite tough to explain.

gonna merge this to ensure it stays up to date with the api response values
have checked the wording with @quis


Calendar goes down when i return with de back button of the navigation controller

When i press de back button in the navigation controller the calendar appears down

But if i do clic in the calendar, the calendar go back to his normal stage

Can you make an example that reproduces this problem and create a share-link of dropbox or wetransfer?

Sure, let me do it and i share you the link

Hello, already solved the problem.
The problem was, when i call the init of the calendar in the event viewWillAppear.
I just change the method to viewDidLoad and everything works well.

👌


Enable the module list to appear in the right hand Blackboard LTI selection session

Is your feature request related to a problem? Please describe.
With the modern Blackboard LTI integration , the course list no longer shows up on the right-hand side of the selection session.
Describe the solution you'd like
This will require REST calls to the Bb site to build out the course list.
Describe alternatives you've considered
Blackboard doesn't send course lists with their LTI launches, thus the need for Bb REST calls.

@cbeach47 has testing been completed for this issue? Can it now be closed?

sry, yes


Add support for disable-strict-forbidden-file-verification

Enabling this option will allow an event to trigger a build if the event
contains BOTH one or more wanted file paths AND one or more forbidden
file paths.  In other words, with this option, the build will not get
triggered if the change contains only forbidden files, otherwise it will
get triggered.
Support for this was added in Plugin Version 2.16.0.

Thank you for contributing to openstack-infra/jenkins-job-builder!
openstack-infra/jenkins-job-builder uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Was the NERDTree plugin removed?

Hi,
I have recently updated my list of airline extensions in my config (g:airline_extensions) and added nerdtree to the list. Now I always get a warning on startup, that the nerdtree extension is not installed. I searched through the source code of this repo right now and it seems to be gone indeed. Am I doing something wrong here or is the documentation just out of date?
If the documentation is out of date, I also found another possible error in it:
Line 1595 has
:let g:airline_extensions#tabline#show_buffers = 1
and I bet it should be this instead:
:let g:airline#extensions#tabline#show_buffers = 1

I don't think there was ever one, besides this:
https://github.com/vim-airline/vim-airline/blob/0f5eedff74973d0b11fa4b61b0cba5567b63381c/autoload/airline/extensions.vim#L38-L42
regarding the documentation bug, thanks, I'll have a look soon.

That is then a bit confusing in the documentation. Because I would expect I can enable/disable this with g:airline_extensions. But judging by the blob, only g:airline#extensions#nerdtree_statusline will ever work.
Also I just noticed, that the reference to the NERDTreeStatusline variable is spelled wrong in the doc, causing help tag search to not find it.

I created PR to fixed document's mistake.
https://github.com/vim-airline/vim-airline/pull/2222


About the StyleVC paper

Hi. Thank you for your great work!
It seems that the StyleVC paper cannot be found on Google. Could you kindly share the paper? Thanks :)

Thank you for your interest in our thesis.
Our paper will be published according to the schedule set by the ICPR. It will be uploaded around mid-May.
Therefore, unfortunately, we cannot publish the paper first. However, the code is well organized, so it's a good idea to go through the code first without the paper.
If you ask a question, we will try to give you an answer.

Paper has been acepted, and the organizers (ICPR) have not yet released the paper yet.


esp_http_client_open() should return specific error code when certificate is invalid (IDFGH-11773)

Is your feature request related to a problem?
esp_http_client_open() returns just ESP_ERR_HTTP_CONNECT even when the certificate has expired or is invalid. There's no specific error code reflecting this particular reason.
I see the reason nicely in the debug-log, but is there a way for the caller to know that reason in order to take further action?
If not, could an event or similar be added to report that reason?
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response

Hello @JB-DX
You can get the error code using the event_handler set in the esp_http_client_config_t. On HTTP_EVENT_ERROR, esp_tls_error_handle_t is sent in the handler argument. You can use that to get the error codes.
esp_err_t _http_event_handler(esp_http_client_event_t *evt)
{
    static char *output_buffer;  // Buffer to store response of http request from event handler
    static int output_len;       // Stores number of bytes read
    switch(evt->event_id) {
        case HTTP_EVENT_ERROR:
            esp_tls_error_handle_t err_handle = (esp_tls_error_handle_t)evt->data;
            int esp_tls_code = 0, esp_tls_flags = 0;
            esp_err_t ret = esp_tls_get_and_clear_last_error(err_handle, &esp_tls_code, &esp_tls_flags);
            ESP_LOGI(TAG, "HTTP_EVENT_ERROR: %s: esp_tls_code: 0x%04x, esp_tls_flags: 0x%04x", esp_err_to_name(ret), esp_tls_code, esp_tls_flags);
            break;
.
.

Thank you @hmalpani,
I tried it but don't get a HTTP_EVENT_ERROR
I still use IDF v4.4-dev-479
Was  it only implemented in newer versions?

Hello @JB-DX
Can you try it on release/v4.4 branch. It was not implemented on v4.4-dev. I have verified it on release/v4.4 branch. I think cherry-picking 9812879 can also help. Hope it helps.
Thanks!

Hello @JB-DX
Using the latest 4.4 release should solve this issue.
Closing this issue. Please feel free to reopen if you still face the issue

I migrated to v5.1 so it took a bit longer to finish.
It now works as you described.
Thanks you !


[CARBONDATA-3331] Fix for external table in Show Metacache

Problem: In SHOW METACACHE command when an external table is queries upon, database size is more than ALL size for index column. External table is created upon a store on a table which is also present in current database.
Bug: Index size for database was being summed up blindly for all ths tables. So some cache entries were being summed up multiple times.
Solution: Compute the database index size while iterating over cache, when cache key contains path of one of the table's path.

[x] Any interfaces changed?  ->  No
[x] Any backward compatibility impacted?  ->  No
[x] Document update required  ->  No
[x] Testing done  ->  No
[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2814/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2815/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/11075/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3047/

Can you add test case for Show Metacache of external table?

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2820/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/11080/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3052/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2887/

Build Failed  with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/11148/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3118/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2888/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/11149/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3119/

LGTM


feat(gpm): update to v1.0.2

Update Gatekeeper Policy Manager to v1.0.2

Nothing huge indeed, anyway the release notes are linked in the table 😃


Python3 support for diesel 4

As discussed in #97 , Here is my pull request ;-)

Fantastic work! This is an excellent contribution for the project.


Added Tab Layout in Android (Kotlin)

Description 📜
Added all the documentation about Tab Layout in Android(Kotlin) and also given the users the option to copy the code wherever required.
Applied formatting to the code.
Fixes #1449

Type of change 📝

[x] Doc#umentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Android Dev #(Kotlin)

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Hey @rahulsain , I have created The PR. Please review it and if there are no issues please merge it.
THANKYOU


changes for new netcode from unreal 469b and newnet mod

fixed issue with glitchy animations when attacking or throwing near an edge while jumping

Adopted these changes with minor modification in merged pull request #35. Thank you Helmet.


[stdlib] Improved documentation for popLast() in BidirectionalCollection

Added detailed documentation to popLast() regarding its safety compared to removeLast(),
as well as noting the performance cost popLast() has.

/cc @dabrahams and @natecook1000

I think a similar note is needed for removeLast() as well.

@hooman Good point, documentation has now been updated for both popLast() and removeLast().

@dabrahams Ok, I removed the safety claim.

I'm sorry, but I'm not comfortable with the recommendation either.  It seems to me that either removeLast should be considered no less preferable than removeRange, or if we really believe its usefulness is so marginal as to be a corner case, we should just tell users to remove(at: index(before: endIndex)).

@dabrahams Safety claim is perhaps my incorrect wording. It is safer in a sense that the type gives you all the info about the behavior of the function, whereas with removeLast it will trap in runtime. It is not about memory safety, it's about convenience and predictable behavior.

@natecook1000 Ok, I've updated according to your requested changes.

Sorry for the back and forth on this, the problem turned out to be deeper than it originally looked =)
The problem with the comment I originally suggested is: both these methods are not necessarily available in all contexts. You can see that removeLast is defined for bidirectional collections and range-replaceable collections, but popLast is only available in bidirectional.
Interestingly, removeLast uses a hook in the protocol (see _customRemoveLast) that returns the last element if it existed, but it also requires the collection to be non-empty.
TL;DR: I think the right thing to do now is simply cross-reference the two methods in the documentation. Pretty much the way you've done it at the time of this writing.
This case clearly identifies the flaw in the design of these methods, solving which might become a part of a follow-up to this proposal one day.
Thanks for working on this, @LouisDhauwe!

This is sort of weird for a comment only change, but needs to be done.
@swift-ci Please smoke test

@swift-ci Please smoke test and merge

@swift-ci Please smoke test and merge


refactor: auto controller parameter type overloading

Description

AutoController parameter type overloading
References

https://github.com/nestjs/nest/blob/master/packages/common/decorators/core/controller.decorator.ts
Checklist

[ ] Add labels and reviewers ?
[x] Are you fully explaining what you changed ?
[x] Did you check that there is no conflict with the base branch ?

리뷰어설정이 안됩니다ㅠ

I’m in out for work, so forwarded to @player31-kks , @Nhahan

please work based on dev branch. not for main.

why dev is default branch not main?

I've changed it to merge into the dev branch

We will set it up so that anyone can submit PRs to the 'dev' branch.

It's really great, and it could be handled neatly by overloading.

I never used typescript overloading like this before. 🥲 I love this PR. He is GOSU. 😄


[solidjs-tailwind] P1: fix failing test

Description

P1: fix failing test (Greeting.spec.jsx)

I believe we will know if this is still a problem when the ticket that fixes the API error is closed


Detect unexpected end of stream

This tool is quite useful for me to synchronize products in my ecommerce, but I need to be able to detect when a transmission ended unexpectedly, in order to cancel the operation.
For example:
This is the format used
{
	"status": "success",
	"data": [
		{"id": 1, "name": ...},
		{"id": 2, "name": ...},
		{"id": 3, "name": ...},
		{"id": 4, "name": ...},
		.
		.
		.
		{"id": N, "name": ...}
	]
}

But when the transmission ends unexpectedly, the format is truncated
{
	"status": "success",
	"data": [
		{"id": 1, "name": ...},
		{"id": 2, "name": ...},
		{"id": 3, "name": ...},
		{"id": 4, "name": ...},
		.
		.
		.
		{"id": X, "name": ...}

Note sometimes the object of the last item is complete, so there is no format error.
When processing, this library ignores that unexpected ending, I need to be able to detect that the json format has not finished correctly.
Is this possible to do?
It would be ideal to throw an exception in that case.
Thank you

Resolved in 0.4.0


Image and relative path

The image helper is great, but you have to pass an absolute path to be accurate.
You can pass a relative one but it will depends of form where you command is called and can be unstable especially for deploy tools like Capistrano.
It would be great to be able to pass variables or constant like __DIR__ or %kernel.root_dir% for Symfony.
What do you think?

Nah, it's up to the developer to build an absolute path with what's at their disposal (be it __DIR__ or %kernel.root_dir%).

@fzaninotto Could you please elaborate a bit about how to do this?


In PHP value "0" is false

In PHP if $value = "0" then stetmentes like if (!$value) will be true, which might not be an expected behavior in this case.
If you check Ruby Dotenv code, same statement works pretty well and that's because in Ruby value 0 is equivalent to true (unlike most other languages).

Your change broke the tests.

Haha,
Correct, I didn't run tests. Can you fix test please.
You got the bug I mentioned right?
I can do it as well but might take some time since I don't have much time
during working days.

This is fixed in v2.0.0.


Collapsing a pool breaks rendering in version 2.2.1

Describe the Bug
Collapsing a pool (kinda) breaks rendering (increases height of whole diagram by an unhealthy amount)

Steps to Reproduce
Steps to reproduce the behaviour:

Extract attached .zip file
Open .bpmn file
Click on Kunde (Pizzabestellung)
In context menu click "change type"
Select (only) option "collapsed pool"

Expected Behaviour
The collaboration is collapsed without adding additional height to the whole .bpmn (rendering)
Environment
Please complete the following information:

OS: 16.04.1-Ubuntu (but same on MacOS something)
Camunda Modeler Version: 2.2.1

Additional
Seems like 2.2.0 doesn't have the problem, according to a colleague
@nevries  fyi, not that I think it's important, but maybe the MacOS version can be added by you, if necessary.
camunda_modeler_2.2.1_bug_report.zip

Confirmed also with 2.2.0, 2.2.2, 2.1.2, 2.0.3 and 2.0.0 on<EMAIL_ADDRESS>It seems like the last version which didn't have this bug was 1.16.2.

I'll have a look into this one.

:hushed:
Most simple diagram to reproduce this: BpmnReplace.poolMessageFlows.bpmn.txt

This is not a bug but a feature.
Consider the rendered diagram:

Both message flows are being joined as Task_A is being removed. They are not parallel to each other but cross each other at { x: 366, y: -7906 }. Because of this the are being joined at that coordinate before being transformed to a bpmn:SequenceFlow later in the process.
You could argue that what is shown to the user is not a practical though.
:walking_man:

Moving this back to our backlog and flagging it as a potential improvement.

@Abendstolz What should be the desired behavior after the pool is collapsed, given the example diagram I pasted above: (A) Sequence Flow is gone or (B) Sequence Flow is now looping around remaining task.

@nikku I'm a bit lost, why does the current implementation create a sequence flow out of the two message flows at all? From a modelling perspective, that makes no sense to me, or am I missing something?

You are opting for: (A) Sequence Flow is gone.
In this case you could argue there is a bug that causes this behavior. We should not attempt to join message flows and remove them on pool collapsing.

Relevant toolkit bug ticket: https://github.com/bpmn-io/bpmn-js/issues/917

Reclassified this as a bug.

Closed via https://github.com/camunda/camunda-modeler/commit/66d579c6c7748689b7752467366896e1f092b0bc.

Fixed in Camunda Modeler v2.2.3.


How to run the clock face on a Lenovo Smart Clock

Is there any documentation available how to run the clock face on a Lenovo Smart Clock or any other smart clock?

Duplicate of #13

But if its not possible to get the clock face on the clock, why this competition even is there?

I have the clock and would like to try it during programming it. What is the point otherwise?

Yes, this is Duplicate of #13.
However, from flutter.dev/clock, "Some of the clock contest submissions might be integrated into the Lenovo Smart Clock's lineup of clock faces, or used as inspiration for future clock faces!"

On Mon, Dec 2, 2019 at 2:32 PM jkurtw<EMAIL_ADDRESS>wrote:

Yes, this is Duplicate of #13
https://github.com/flutter/flutter_clock/issues/13.
However, from flutter.dev/clock, "Some of the clock contest submissions
might be integrated into the Lenovo Smart Clock's lineup of clock faces, or
used as inspiration for future clock faces!"
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter_clock/issues/25?email_source=notifications&email_token=AB3BZ2ZFCS4QNYJFST2O6XLQWWEIXA5CNFSM4JSRKE4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFWKHCA#issuecomment-560767880,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3BZ22T74XKQHQY3OTMI6TQWWEIXANCNFSM4JSRKE4A
.


Rho Signal blogpost on Polars by Liam Brannigan

Link to suggestion
https://www.rhosignal.com/tags/polars/
Targeted section
Blog posts
Describe your suggestion
This is a series of blogpost on Polars usage which has a lot of useful tricks and information from @braaannigan. I also include website link including polars tag for easy navigation. Moreover, @braaannigan also has a Data Analysis with Polars course on Udemy.
Checklist

[X] checked if your suggestion is not already listed in Awesome Polars list.
[X] checked if it can fit in an existed section or suggest a new section and explain why your item does not fit into one and what would be the interest of your new section.
[X] checked when it's a repository if it's maintained (archived repository will be rejected).

Thanks @andreale28 for this great suggestion !

@braaannigan is it ok to reference your series of Polars posts in Awesome Polars ?

@ddotta @andreale28 Yes, thanks for referencing it!
I've also made a number of youtube videos: https://www.youtube.com/channel/UC-J3uR0g7CxCSnx0YFE6R_g
Obviously, I also think it would be good to have a section for paid online courses, but I understand if you don't want to link to those!

Thanks @braaannigan, I added your youtube videos. It's great !


Not found Route with PATCH

I am facing trouble with PATCH endpoints, they are always return 404 response, but if I change to any other, the request work like a charm, follow the Resource Code:
$api->resource(
 Resource::factory('/clients', 'Client')
    ->deny(AclRoles::UNAUTHORIZED)
    ->model(Client::class)
    ->expectsJsonData()
    ->handler(ClientController::class)
    ->transformer(ClientTransformer::class)
    ->endpoint(Endpoint::create()
    ->allow(AclRoles::UNAUTHORIZED)
    )
    ->endpoint(Endpoint::patch('/partial/{id}', 'updatePartial')
        ->allow(AclRoles::UNAUTHORIZED)
    )
    ->endpoint(Endpoint::put('/partial/{id}', 'updatePartial')
        ->allow(AclRoles::UNAUTHORIZED)
    )
);

Ahh thnx! Hopefully it will be merged soon


Absolute Layout example

I have been trying to get this to show properly using an absolute layout (so I can have the FAB on the bottom right, about 16 from the edge, over any other content on the screen).
I am not coming right.
I have been checking other examples of AbsoluteLayout, but nothing is working.
Could anyone create an example of this FAB in the bottom right of the layout?

To get it in the bottom right, I wrap everything in a Grid then using HorizontalOptions and VerticalOptions with Margin - so something like this:
<ContentPage>
    <Grid>
        <ListView .../>
        <fab:FloatingActionButton HorizontalOptions="End" VerticalOptions="End" Margin="0,0,16,16" ... />
    </Grid>
</ContentPage>

Thank you so much, that works 100%
On Fri, Aug 3, 2018 at 4:16 PM Alex Dunn<EMAIL_ADDRESS>wrote:

To get it in the bottom right, I wrap everything in a Grid then using
HorizontalOptions and VerticalOptions with Margin - so something like
this:

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SuavePirate/Xamarin.Forms.Controls.FloatingActionButton/issues/24#issuecomment-410263566,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AON4uH7h9qyYIXz2-eI5VVXLYku2hlH1ks5uNFstgaJpZM4Vtikh
.


feat: add footer section

Add footer section with links/forms to

Github contribution repo
Contact for support
Copyright
Documentation/Blogs

Thanks for raising! Will pick up as I have bandwidth.

Can I work on this?

@Anuragwagh sure. Thanks! Assigned it to you.

@Anuragwagh are you working on this one?

I'm working on it; in fact, my exams are underway, and my final paper is due tomorrow; I'll complete it as soon as possible after the exams are completed.

@Anuragwagh alright. thanks for the update.

Do let me know if it still needs to be completed.

Hello @HectorShaw , The issued task has been completed.

Oh, I saw it was still open yesterday, so I got confused. No issues

@HectorShaw if you are still looking to contribute - you can pick up #170

Fixed in #200


fix: update README to template

fix: update README to template

Heads-up @wangela and I are trying to arrive at a consistent README template for all repos, so there'll likely be several more micro updates to it on this branch before a final merge. Thx for your patience!


[Outlook] (delegate) Now supported on Mac

Docs Build status updates of commit 967aa2a:
:clock10: Pending: waiting for processors (17 builds ahead of you)
:warning: Docs Build is busy, currently there are 17 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 967aa2a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/delegate-access.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Intellisense from output folder

From @AbubakerB on October 17, 2016 19:10
Hi guys,
I don't know if this is the correct place to raise this but I'm trying to get intellisense from razor files in non-web projects and its just not working.
I'm using tools like RazorEngine/RazorTemplate/RazorLight and the only way to get intellisense is to copy their dlls to the \bin folder. I read that the reason for this is because VS only looks at the \bin directory because, of course, that's the default output folder for web-projects.
But for example, console apps, have their output in \bin\debug or \bin\release. Is there a way to make this work atm (with VS2015.3)? Or is there a reason to why it only looks in the bin folder?
If it's not already fixed, could it be fixed?
I've searched everywhere for an answer on this but all I can find are people saying copy to bin folder.
Copied from original issue: aspnet/RazorTooling#103

From @AbubakerB on October 17, 2016 21:41
That's alright, thanks for the response.
Would it be possible to make this work for the upcoming VS "15"?

From @NTaylorMullen on October 17, 2016 21:28
@AbubakerB there currently is not a way to do this by default for non-web projects. Sorry that there's not a better answer 😢

From @NTaylorMullen on October 17, 2016 21:58
We're currently rethinking the way the Razor editor works and have started prototyping. In this "new" world having support for class libraries would definitely be something on our radar.

From @AbubakerB on October 17, 2016 22:7
Awesome! Looking forward to it!

Dupe of #1136


l2cu_no_dynamic_ccbs() fix

l2cu_no_dynamic_ccbs() can cause assert in freertos's timers.c when SMP_INCLUDED == FALSE due to timeout stays equal to 0.

 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.

Thanks for sending this, @SiarheiVolkau . Sorry noone got back to you earlier.
I think this issue has been taken care of on the master branch now, as of 5b9a89f04d. Please reopen (or open an issue) if that doesn't seem to be the case.

Yes now this case handled in master branch.


Fixed the bug, that issued the bottom circle to be displayed wrong.

While using your extended variation of SwipeRefreshLayout, I run into the bug. While pulling from bottom (using Direction.BOTH) the circle indicator with arrow moved out of the screen pretty fast. I compared your code with SwipeRefreshLayoutBottom and noticed, that the offset is calculated a bit diffrent way. I brought it back, how it was at initial version and everything worked. Correct me, if I missed something.

Merged thanks


Event loop might block if callback not always async

I ran into a pretty insidious bug that has taken me days to unravel, and I wanted to share it for anyone else who runs into the same problem.
The crux of the issue is that through2-concurrent requires that it's callback function (the one you the user provides) always be asynchronous. Clearly this is not strictly required by the library, but If that callback is not always asynchronous, it's possible for the event loop to block.
Let me share my code:
  return throughConcurrent.obj({ maxConcurrency: concurrency }, async function(
    domain,
    enc,
    done
  ) {
    try {
      const url = await frontierList.getNextUrlForDomain(domain);
      if (url) {
        this.push(url);
      }
    } catch (err) {
      console.error(err)
    }
    done();
  });
};

What happens is that when the done() function is invoked inside my callback, my callback is synchronously invoked with a domain. This is all fine, except If that domain also causes frontierList.getNextUrlForDomain(domain) to return immediately without performing an async operation, invoking done causes my callback is once again be invoked synchronously. What happens, then, is that sometimes I get into a pattern where maybe some percentage of async callbacks never finish because every subsequent value causes frontierList.getNextUrlForDomain(domain) to return immediately without performing an async operation. The event loop is quickly blocked, and my async operations never finish.
A workaround for this was pretty simple. I was already using a wrapping around through-concurrent, but I will only share the relevant parts here.
function createThroughConcurrent(concurrency, callback) {
  const stream = throughConcurrent.obj({ maxConcurrency: concurrency }, function(
    domain,
    enc,
    done
  ) {
    setImmediate(() => {
      callback.call(this, domain, enc, done);
    });
  });
  return stream;
};

The idea is that we want to clear the event loop no matter what before invoking our callback. This code accomplishes that.

Hi, thanks for sharing!
I don't think I'm following exactly what's going wrong, maybe because I'm not fully understanding what frontierList.getNextUrlForDomain does.  Could you possibly supply a minimal example of the failure?
Thanks!

umm sure! I think I can do that

This example is a bit contrived, but it works
const throughConcurrent = require("through2-concurrent");
const fs = require("fs");
const { promisify } = require("util");
const split = require("split");

const writeFileAsync = promisify(fs.writeFile);

fs
  .createReadStream("infiniteList.txt")
  .pipe(split())
  .pipe(throughConcurrent.obj({ maxConcurrency: 5 }, async function(domain, enc, done) {
    try {
      console.log("invoking callback");
      const url = await sometimesAsyncFunction(domain);
      if (url) {
        this.push(url);
      }
    } catch (err) {
      console.error(err);
    }
    done();
  }));

let calledBefore = false;
async function sometimesAsyncFunction(domain) {
  if (calledBefore) {
    return "";
  }
  calledBefore = true;
  // This is a readFile in my code, but clearly I don't want to have to stub that out
  await writeFileAsync("123.txt", domain);
  console.log("async operation finished");
  const someUrl = "http://123.com";
  return someUrl;
}

infiniteList.txt
I've attached a file that is basically a list of domains. For this demonstration, it's quite small, but imagine it were infinite.
When you run the code, you will notice that invoking callback will display repeatedly until the list is depleted, and only after that point does async operation finished log. This is a clear indication that the event loop is blocked and becomes  a problem when the result of the async operation influences the readable stream (which is what's happening in my scenario (not shown above)).
I hope this code makes it more clear what I'm doing

That's actually the result I'd expect from that code. That writeFileAsync takes a (relatively) long time so you'd expect all the other things to happen before that completes.
Now if you removed the calledBefore bit so sometimesAsyncFunction is actually doing something each time then the maxConcurrency of 5 is going to cause it to wait after the 5th call until one of them complete before continuing.
With larger infinitelist.txt this is still going to lock up but for a different reason. You're not doing anything with the output of the stream so eventually it's going to fill up the internal buffer (see highWaterMark setting). You need either pipe it into something else or add a .on('data', function (data) {})

That's all completely fair. Let me propose a slightly different example. I've changed my readstream into a readable that counts forever. I've made sure to pipe my code in /dev/null so that we don't reach the highWaterMark
const throughConcurrent = require("through2-concurrent");
const fs = require("fs");
const { promisify } = require("util");
const { Readable } = require("stream");

const writeFileAsync = promisify(fs.writeFile);

class InfiniteStream extends Readable {
  constructor() {
    super({ objectMode: true });
    this.count = 1;
  }
  _read() {
    this.push(this.count.toString());
    this.count++;
  }
}

const infiniteStream = new InfiniteStream();
infiniteStream
  .pipe(throughConcurrent.obj({ maxConcurrency: 5 }, async function(count, enc, done) {
    try {
      console.log("invoking callback ", count);
      const url = await sometimesAsyncFunction(count);
      if (url) {
        this.push(url);
      }
    } catch (err) {
      console.error(err);
    }
    done();
  }))
  .pipe(fs.createWriteStream("/dev/null"));

async function sometimesAsyncFunction(count) {
  if (count > 0) {
    return "";
  }
  // This is a readFile in my code, but clearly I don't want to have to stub that out
  await writeFileAsync("123.txt", count);
  console.log("async operation finished");
  const someUrl = "http://123.com";
  return someUrl;
}

You can pipe this program into grep and see that "async operation finished" never logs, even though the transform stream gets invoked over and over again. While writing to the file definitely takes a relatively long period of time, I would expect it to eventually finish.

I can see that this example is going to lock up the JS thread forever, but that's because your InfiniteStream in synchronous and never exhausts. You could in this case fix it by putting a setImmediate in the through2-concurrent callback but I'd argue the correct place to do that would be the in the InfiniteSteam.
Here's a much simplified version of your previous example that illustrates what I mean:
const { Readable } = require("stream");

class InfiniteStream extends Readable {
  constructor() {
    super({ objectMode: true });
  }
  _read() {
    this.push('');
  }
}

(new InfiniteStream).on('data', function () {
});

setTimeout(function () {
  console.log("This never prints");
}, 1)

ahhh, thank you
I always thought _read was invoked asynchronously. That explains a lot.
so does that mean that the callback to the data event is continuously pushed information, effectively invoking the _read over and over again, until the readable stream pushes null or until the writeable signals backpressure? And I imagine pipe does that exact same thing?

Yes that should be what's happening. Usually in real world things there's something asynchronous somewhere in the chain that makes it ok even if the readable is infinite (or very large) but it's these edge cases that make for confusing debugging :)


Can I use this library to encrypt/decode string

Can I use this library to encrypt/decode string instead of introduce other encrypt lib?

No. The Library does not implement anything related to cryptography. It relies on either openssl or libressl to do the actual crypto. So if you use this library you are already using one of those, which do implement various encryption and decryption functions. If you dont use this library you'll have to either directly use one of those two or link to a different cryptography library. That said I'd recommend sticking to either openssl or libressl as they are the defacto standard for crypto and at least one of them will be present on pretty much every system you'll ever encounter.

Thanks you for the reply!
I am using this library in app-mesh and this is really very good. so I am considering whether I can use this similar approach for string encrypt.
For use openssl or libressl, that's much deep level and I am not familiar with that, but with this JWT token, it seems the token is too long for some short string.
Is there any other simple way to encrypt and decode string without explicit a secret key?

Are you trying to protect data?
Common example is a password, personal information, backing details, etc.
If yes, JWT are not the right answer! All the data is available in "plain text".

A JWT can be used to:
Propagate one’s identity between interested parties.
Propagate user entitlements between interested parties.
Transfer data securely between interested parties over a unsecured channel.
Assert one’s identity, given that the recipient of the JWT trusts the asserting party.

ref

You might want to take a look at https://wiki.openssl.org/index.php/EVP_Symmetric_Encryption_and_Decryption which is a create starting place. As for a library, I have seen https://github.com/weidai11/cryptopp, which might help you in ur search.
PS cool project and we are glad to hear you had a great experience with our project!

My personal favorite is Botan crypto library and it is C++ implementation of cryptography, easy to use much mor control as you want, and well documented.

Thanks @mohsenomidi  @prince-chrismc for the guidance. I will try those libraries.
Merry Christmas!

🎄


Weird artifacts on SM-R322 with Galaxy S7

Hello,
I'm using the new Gear VR (SM-R322) with Galaxy S7 Phone. There are some pixels repeat artifacts on the edges of views, and viewport sizes seem to be different than the native sdk as can be seen in the image:

Here is a screenshot from a software running with NDK:

I am using the standard gvr.xml file supplied in the samples. Is this happening because the lens distortion parameters do not match my device?

Hi,
This was fixed by pull request https://github.com/Samsung/GearVRf/pull/506. We don't have a binary release yet that includes this fix but we will soon have one. If you can't wait you can always build and use the latest framework code. Let us know if you need any help.

Thank you @liaxim
I'll just wait. New binary will probably be ready until I'm ready to release what I'm developing.


Support private fields in header

I'm not sure why the read_dir and read_img_file are separated. We can use itk.imread for both.
I'm not going to change that for now, though. It might break something that I'm not aware of.


Audio

How do I control audio of a Sony STR-DN1080 Receiver? Power and inputs work fine

**Plugin Config:**

```json
Show your Homebridge config.json here, remove any sensitive information.

Screenshots:

Environment:

Plugin Version:
Homebridge Version:
Node.js Version:
NPM Version:
Operating System:

@HomeCoops,
Sorry for the late reply, did not see the notification.
What you mean about audio?
If the question about volume control,
you can use Apple TV controls in Control Center on an iOS or iPadOS device.
To adjust the volume, press the volume buttons on your iOS or iPadOS device when the Apple TV controls in Control Center are open.

I have the same 'issue' with volume control on my Sony soundbar... I think this is because I have my TV volume control set to the audio receiver so the tv / physical apple remote controls the volume). The iOS volume control is either disabled or does nothing. I'll play around some more to confirm when I have a chance.


Need a template for getting started

I'm willing to entertain arguments why it should be activator.  But there should be something to get people up and running quickly.

I looked at Activator, and it's highly disappointing.  It doesn't prompt for any package names or version numbers.
I've created http4s/http4s.g8 in preparation for a getting started guide on the website.  Issues with the template can be managed there.
I'd like to script a materialized view of the defaults from Travis for those who don't care to mess with giter8.  If we're still inclined to get into the Activator directory, one could script generating the Activator metadata from the more powerful g8 build!


onEdgesChange params does not include information about the trigger

Describe the Bug
When I delete an edge directly onEdgesChange gets called.
When I delete a node with an edge attacehd, onEdgesChange gets called.
Now, I have in place a "user confirmation" logic that happens depending certain conditions on deletion.
The issue is that I do not know if onEdgesChange has been triggered because of a node deletion or a direct deletion and depending on that I need to perform a confirmation or not.
onEdgesChange should also pass the information of the trigger, otherwise a controller flow is not aware of the actual state.
Any ideas?
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
delete a edge
delete a node connected to another node
you do not know what deleted the edge
Expected behavior
you know what deleted the edge
Screenshots or Videos
No response
Platform

OS: any
Browser: any
Version: any

Additional context
No response

Since React Flow does not have a onBeforeDelete callback (yet), the way I ended up solving this is by disabling React Flow's deleteKeyCode, adding custom key handler and then handling the logic there.
See https://github.com/xyflow/xyflow/discussions/2925#discussioncomment-6976130 by @AdamGerthel and https://github.com/xyflow/xyflow/discussions/2925#discussioncomment-7907136
cc @RiccardoRomagnoli  in case you still need a solution

Thanks for your comment. I ended up adding a await delay(10) in onEdgesChange which allows onNodesChange to be executed beforehand. Then, by setting a store variable I can know if the delete has been triggered by a node delete (confirmed by user) or edge delete (instant).

v12 has a onBeforeDelete handler. You can already try out the beta. Details: #3764

We added an onBeforeDelete in React Flow 12, that can be used for this.
If you are using 11, you can find the migration guide here: https://reactflow.dev/learn/troubleshooting/migrate-to-v12

Great work guys, keep it up!


[SPARK-14320][SQL] Make ColumnarBatch.Row mutable

What changes were proposed in this pull request?
In order to leverage a data structure like AggregateHashMap (https://github.com/apache/spark/pull/12055) to speed up aggregates with keys, we need to make ColumnarBatch.Row mutable.
How was this patch tested?
Unit test in ColumnarBatchSuite. Also, tested via BenchmarkWholeStageCodegen.

cc @nongli

Test build #54676 has started for PR 12103 at commit a1fe9f8.

Test build #54676 has finished for PR 12103 at commit a1fe9f8.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 public static final class Row extends MutableRow

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54676/
Test PASSed.

Test build #54746 has started for PR 12103 at commit 2002131.

Thanks, comments addressed.

Test build #54746 has finished for PR 12103 at commit 2002131.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54746/
Test FAILed.

test this please

Test build #54748 has started for PR 12103 at commit 2002131.

Test build #54748 has finished for PR 12103 at commit 2002131.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/54748/
Test FAILed.

Test build #2731 has started for PR 12103 at commit 2002131.

Test build #2731 has finished for PR 12103 at commit 2002131.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

lgtm

test this please

Seems the last test run was 4 days ago. I just triggered a new one.

Test build #55035 has started for PR 12103 at commit 2002131.

Test build #55035 has finished for PR 12103 at commit 2002131.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55035/
Test FAILed.

Merged build finished. Test FAILed.

test this please

Test build #55043 has started for PR 12103 at commit 2002131.

Test build #55043 has finished for PR 12103 at commit 2002131.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/55043/
Test PASSed.


Introduce CLI parsing and more advanced logging system

See #11 for a description.

Kudos, SonarCloud Quality Gate passed!
  0 Bugs   0 Vulnerabilities   0 Security Hotspots   8 Code Smells
 0.0% Coverage  0.0% Duplication

I think that there are some code smells in here that should be addressed 🤔


On demand validation

The idea of this feature is to add an option to allow validation to be "on-demand" rather than automatic. With this feature the extension would not systematically try to find and validate manifests but the user would click on the image to ask for manifest check and validation.

New Feature: You will see Verify image with C2PA by right-clicking on an image on any website. If you click on it, it will verify the single image, and it will display the CR icon if the image has a manifest