Remote submit doesn't support wrapping submit function in handleSubmit

Are you submitting a bug report or a feature request?
Bug report
What is the current behavior?
A minimal example of the form that I'm implementing
const submit = data => console.log(data)

const Form = ({handleSubmit}) => (
  <form onSubmit={handleSubmit(submit))>
     ...
  </form>

export default reduxForm({form: 'test'})(Form)
)

Can provide more detailed example if it would be helpful!

When this form is submitted via a submit button in the form, the submit function is called correctly.
When the form is submitted remotely (by dispatching submit('test')) and the submit function is passed in as a prop to reduxForm (e.g. onSubmit: submit), the submit function is called correctly.
When the form is submitted remotely and the submit function is passed into handleSubmit as in the above example, there is an error Uncaught Error: You must either pass handleSubmit() an onSubmit function or pass onSubmit as a prop. (additional stack trace below)

What is the expected behavior?
Expected identical behavior in all three of the above examples (i.e. the form submits successfully regardless of how the form is submitted & the submit function defined).
What's your environment?

redux-form version: 6.7.0
affects Chrome and Safari
Other information
checkSubmit	@	createReduxForm.js:137
submit	@	createReduxForm.js:570
submitIfNeeded	@	createReduxForm.js:263
componentWillReceiveProps	@	createReduxForm.js:362
(anonymous)	@	ReactCompositeComponent.js:611
measureLifeCyclePerf	@	ReactCompositeComponent.js:75
updateComponent	@	ReactCompositeComponent.js:610
receiveComponent	@	ReactCompositeComponent.js:547
receiveComponent	@	ReactReconciler.js:125
_updateRenderedComponent	@	ReactCompositeComponent.js:754
_performComponentUpdate	@	ReactCompositeComponent.js:724
updateComponent	@	ReactCompositeComponent.js:645
performUpdateIfNecessary	@	ReactCompositeComponent.js:561
performUpdateIfNecessary	@	ReactReconciler.js:157
runBatchedUpdates	@	ReactUpdates.js:150
perform	@	Transaction.js:140
perform	@	Transaction.js:140
perform	@	ReactUpdates.js:89
flushBatchedUpdates	@	ReactUpdates.js:172
closeAll	@	Transaction.js:206
perform	@	Transaction.js:153
batchedUpdates	@	ReactDefaultBatchingStrategy.js:62
batchedUpdates	@	ReactUpdates.js:97
dispatchEvent	@	ReactEventListener.js:147

Seeing this issue too, also does not work with a ref remote submit either:
FormComponent.js
function formComponent({handleSubmit, formRef}) {
  localOnSubmit(values) {
    console.log('values: ', values)
  }

  return (
    <form ref={formRef} onSubmit={handleSubmit(localOnSubmit)}>
      ...
    </form>
  )
}

export default reduxForm({form: 'testForm'})(formComponent)

ParentComponent.js
function parentComponent() {
  return (
    <div>
      <FormComponent formRef={(el => this.formRef = el)}/>
      <button onClick={() => this.formRef.submit()}/>
    </div>
  )
}

This one does not produce an error, but it does not prevent the submit event and does a full page reload and adds the fields as query params to the url.  Also, it does not call localOnSubmit
redux: 3.7.2
redux-form: 7.0.3

Duplicate of #2095.
You must use the Form component for this; also see https://github.com/erikras/redux-form/issues/2095#issuecomment-287896000


Extend OAuth2 package to support OpenID connect

This issue was originally filed by<EMAIL_ADDRESS>
After thinking this through, I realize this is a fairly major feature request.
The current OAuth2 package (http://pub.dartlang.org/packages/oauth2)
does not support OpenID connect. For example, Openid connect returns a id_token as part of the authorization flow.
A small enhancement would be to extend Credentials.dart to provide the raw value of id_token if it is present.
Ideally, support would be provided for JSON web tokens, signature verification, etc.

This issue has been moved to dart-lang/oauth2#8.


Finetune on outputs only

Hello, do you have an option to disable backpropagation for the question part?
(as the 'train_on_inputs' argument on alpaca-lora project: https://github.com/tloen/alpaca-lora/blob/main/finetune.py#L49)
I didn't find this option in lmflow codebase. For some cases, it might converge faster to only backpropagate on answers parts only.
Also, I'm seeing the group_texts method in the finetuner class: https://github.com/OptimalScale/LMFlow/blob/main/src/lmflow/pipeline/finetuner.py#L112 .  I assume batch_size = 1 during the finetuning process then? Is that correct?
Thanks.

Hi,
if you want to use the gradients from the answer part only, you may use our text2text format instead of text_only format. Please refer to https://optimalscale.github.io/LMFlow/examples/DATASETS.html
Thanks

Do you have any working examples of training using text2text format? I tried but it doesn't work.

Hi,
What is the issue you observed when running with text2text?

training with text2text actually works after I tried different parameters. Maybe it failed initially because I set block_size to 1.


Display full name

For bug reports, please give the relevant info:

Gogs version (or commit ref):
Git version: 0.9.0
Operating system: Ubuntu 14.04
Database:

[ ] PostgreSQL
[x] MySQL
[ ] SQLite

Can you reproduce the bug at http://try.gogs.io:

[x] Yes
[ ] No
[ ] Not relevant

Log gist:

Description
This is somewhat similar to this issue #2792. When one hovers over an username, e.g. in the People overview in an organisation, Gogs only shows the username. If the username looks like this: xbsh01 it is very hard to distinguish who this user is. It would be nice if the full name could be displayed as well. E.g. in braces, much like it is displayed when one adds a user to a team.

Do you mean this part (hovering on an avatar)?

@Unknwon I can do this.

@moltam Yes.

This is claimed to be implemented by merging https://github.com/gogits/gogs/pull/2845, you can test on demo site.

Looks good, closing this one.


Re-enable Code Folding tests

In a recent integration test failure formatting tests were falling over because folding ranges weren't available for the document on file open. Basically things ended up timing out because of it.
Technically all features should eventually become available and our "timeout" token represents a long enough period of time where this shouldn't be an issue. Therefore, this makes me believe there may be reliability issues in our code folding language service capabilities.
We should re-enable our CodeFoldingTest's to try and track down these potential reliability issues.

Done with #6619


Change lookup UI from drop down to modal

This would allow users to get more context in search results, get more search results, be able to paginate.
Blocks #2084

Some options for an expanded UI for lookups.

look at LC view

More context from the original ticket #2123

This wlll apply to inputLookupLOC and inputLookupQA and SinopiaSearch

Should we try to combine LOC and QA lookups be combined into a single lookup?

@astridu

What will the base component (i.e., before opening the modal) look like?
What will selection of authorities to search look like?
What will results from multiple authorities look like?
Is this a typeahead or a search? (Given QA performance, search would be preferable.)
How will results with images (e.g., Discogs) be displayed?
How many results will be displayed from each authority?
Will there be pagination?

See this mockup for base component.  Let me know if we will be able to do that data entry box style. If not, then will modify. https://invis.io/ATY349JEPSW#/425970388_leftnav_Dropdown
See this mockup for the actual modal window. https://invis.io/ATY349JEPSW#/425969897_QA_Modal
The text box can either be a typeahead, or require user to hit enter to activate it. Results then appear below.
Could we put the authorities across the top in tabs so it's easy to navigate?
Could we put the image next to the item found like in the mockup?
Could we display 50 results per authority?
No pagination needed

* Could we put the authorities across the top in tabs so it's easy to navigate?

Yes, Bootstrap does allow for tabs within a modal.

* Could we put the image next to the item found like in the mockup?

We should be able to support the icon next to the found item.

* Could we display 50 results per authority?

I don't think there isn't any reason we can't display 50 results. The call to QA would need to change.

* No pagination needed

Not sure if that is accurate, @astridu what if there are more than 50 results being returned from QA?

I was assuming 50 was plenty. do you think we would want to show more than 50?

Moving to epic.


Removes ChromeHeadless from zipkin-ui test setup

The zipkin-ui build fails due to being unable to invoke ChromeHeadless.
As the classic UI is being deleted soon, and tests are redundantly run
with FireFox which does work, this should be ok to remove.
At some point, someone can look into updating the slave (for use in
zipkin-lens).

we got past this part, but there's an issue possibly docker related
[2019-04-27T05:46:08.439Z] 05:46:06,404 ERROR [cluster155-reconnection-1] core.ControlConnection$1 (ControlConnection.java:182) - [Control connection] Cannot connect to any host, scheduling retry in 64000 milliseconds

[2019-04-27T05:46:25.195Z] Cancelling nested steps due to timeout

[2019-04-27T05:46:25.350Z] Sending interrupt signal to process

[2019-04-27T05:46:26.574Z] [INFO]

[2019-04-27T05:46:26.574Z] [INFO] Results:

[2019-04-27T05:46:26.574Z] [INFO]

[2019-04-27T05:46:26.574Z] [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0

[2019-04-27T05:46:26.574Z] [INFO]

[2019-04-27T05:46:26.574Z] [INFO]

[2019-04-27T05:46:26.574Z] [INFO] --- apache-rat-plugin:0.13:check (default) @ zipkin-storage-cassandra ---

[2019-04-27T05:46:26.574Z] [INFO] Enabled default license matchers.

[2019-04-27T05:46:26.574Z] [INFO] Will parse SCM ignores for exclusions...

[2019-04-27T05:46:26.574Z] [INFO] Finished adding exclusions from SCM ignore files.

[2019-04-27T05:46:26.574Z] [INFO] 62 implicit excludes (use -debug for more details).

[2019-04-27T05:46:26.574Z] [INFO] 29 explicit excludes (use -debug for more details).

[2019-04-27T05:46:26.574Z] [INFO] 48 resources included (use -debug for more details)

[2019-04-27T05:46:26.574Z] [INFO] Rat check: Summary over all files. Unapproved: 0, unknown: 0, generated: 0, approved: 48 licenses.

[2019-04-27T05:46:26.574Z] [INFO]

[2019-04-27T05:46:26.574Z] [INFO] --- maven-failsafe-plugin:3.0.0-M3:verify (verify) @ zipkin-storage-cassandra ---

[2019-04-27T05:46:26.574Z] [INFO] ------------------------------------------------------------------------

[2019-04-27T05:46:26.574Z] [INFO] Reactor Summary for Zipkin (Parent) 2.13.0-SNAPSHOT:

[2019-04-27T05:46:26.574Z] [INFO]

[2019-04-27T05:46:26.574Z] [INFO] Zipkin (Parent) .................................... SUCCESS [ 10.464 s]

[2019-04-27T05:46:26.574Z] [INFO] Zipkin Core Library ................................ SUCCESS [ 29.525 s]

[2019-04-27T05:46:26.574Z] [INFO] Zipkin UI .......................................... SUCCESS [01:59 min]

[2019-04-27T05:46:26.574Z] [INFO] Zipkin Lens ........................................ SUCCESS [02:05 min]

[2019-04-27T05:46:26.574Z] [INFO] Collector .......................................... SUCCESS [  0.811 s]

[2019-04-27T05:46:26.574Z] [INFO] Collector: Core Library ............................ SUCCESS [  7.562 s]

[2019-04-27T05:46:26.574Z] [INFO] Zipkin JUnit ....................................... SUCCESS [  7.100 s]

[2019-04-27T05:46:26.574Z] [INFO] Storage ............................................ SUCCESS [  0.799 s]

[2019-04-27T05:46:26.574Z] [INFO] Storage: Cassandra (Zipkin V2) ..................... FAILURE [24:36 min]

[2019-04-27T05:46:26.574Z] [INFO] Storage: Cassandra (v1) ............................ SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Storage: Elasticsearch (V2) ........................ SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Storage: MySQL (v1) ................................ SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Collector: Kafka 0.10+ ............................. SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Collector: RabbitMQ ................................ SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Zipkin Server ...................................... SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Benchmarks ......................................... SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Collector: Scribe (Legacy) ......................... SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Collector: Kafka (Legacy) .......................... SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Auto Configuration ................................. SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Auto Configuration: Kafka Collector ................ SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] Auto Configuration: Scribe Collector ............... SKIPPED

[2019-04-27T05:46:26.574Z] [INFO] ------------------------------------------------------------------------

[2019-04-27T05:46:26.574Z] [INFO] BUILD FAILURE

[2019-04-27T05:46:26.574Z] [INFO] ------------------------------------------------------------------------

[2019-04-27T05:46:26.574Z] [INFO] Total time:  29:40 min

[2019-04-27T05:46:26.574Z] [INFO] Finished at: 2019-04-27T05:46:26Z

[2019-04-27T05:46:26.574Z] [INFO] ------------------------------------------------------------------------

[2019-04-27T05:46:26.574Z] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-failsafe-plugin:3.0.0-M3:verify (verify) on project zipkin-storage-cassandra: There are test failures.

[2019-04-27T05:46:26.574Z] [ERROR]

[2019-04-27T05:46:26.574Z] [ERROR] Please refer to /home/jenkins/jenkins-slave/workspace/incubator-zipkin_PR-2545/zipkin-storage/cassandra/target/failsafe-reports for the individual test results.

[2019-04-27T05:46:26.574Z] [ERROR] Please refer to dump files (if any exist) [date].dump, [date]-jvmRun[N].dump and [date].dumpstream.

[2019-04-27T05:46:26.574Z] [ERROR] org.apache.maven.surefire.booter.SurefireBooterForkException: The forked VM terminated without properly saying goodbye. VM crash or System.exit called?

[2019-04-27T05:46:26.574Z] [ERROR] Command was /bin/sh -c cd /home/jenkins/jenkins-slave/workspace/incubator-zipkin_PR-2545/zipkin-storage/cassandra && /usr/local/asfpackages/java/jdk-11.0.1/bin/java -jar /home/jenkins/jenkins-slave/workspace/incubator-zipkin_PR-2545/zipkin-storage/cassandra/target/surefire/surefirebooter13852392792175993807.jar /home/jenkins/jenkins-slave/workspace/incubator-zipkin_PR-2545/zipkin-storage/cassandra/target/surefire 2019-04-27T05-16-56_494-jvmRun1 surefire18328656202545386197tmp surefire_16716239195206411617tmp

[2019-04-27T05:46:26.574Z] [ERROR] Error occurred in starting fork, check output in log

[2019-04-27T05:46:26.574Z] [ERROR] Process Exit Code: 143

[2019-04-27T05:46:26.574Z] [ERROR] Crashed tests:

[2019-04-27T05:46:26.574Z] [ERROR] zipkin2.storage.cassandra.ITCassandraStorage$ITSpanStore

[2019-04-27T05:46:26.574Z] [ERROR] 	at org.apache.maven.plugin.surefire.booterclient.ForkStarter.fork(ForkStarter.java:670)

[2019-04-27T05:46:26.574Z] [ERROR] 	at org.apache.maven.plugin.surefire.booterclient.ForkStarter.run(ForkStarter.java:283)

[2019-04-27T05:46:26.574Z] [ERROR] 	at org.apache.maven.plugin.surefire.booterclient.ForkStarter.run(ForkStarter.java:246)

there was a timeout setting behind latest fail, and I just updated it. Let's see

updating the timeout is futile and somewhat antisocial as it hogs slaves from others. If we add timeout it would take maybe 2+hrs to complete the build. For the time being, and knowing Travis is ok in ASF, I added a "docker.skip" property in order to keep the Jenkins slave working less.

had to play games to keep scribe tests running consider there's a bug in the RAT plugin outstanding. If @apache/zipkin-committers want brownie points, please solve https://issues.apache.org/jira/browse/RAT-239 for us and everyone else :P


Plotting arguments

The plotting routines in pySTEPS are amazing, but it would be cool if one could pass further **kw plotting arguments (e.g. when I calculated the UV-differences between DARTS and LucasKanade, I coloured the vectors in red).
https://github.com/pySTEPS/pysteps/blob/c12287fd32b4e1d0af9daa0e83f34f325efa82bc/pysteps/visualization/motionfields.py#L8

Good idea. Changes included in commit<PHONE_NUMBER>6ddf38fc0e97481c481cb8b2cbe6d6.


Create a PG migration

Based on the chosen tooling, define all tables and schema via a migration file

I'll take it

This was already done and mistakeley left open


Add sign in and signup both the buttons

please assign me this issue

i would like to contribute in this.


Update fields, add WI race fetching

Update fields we want to collect for races (following discussion with Alice)
Add fetching for WI

@alxmjo can you rebase and squash your PR so it's a fast-forward commit on top of master?


Multiple sensors Temperature / Humidity

Hello. I bought more sensors for the weather station. Unfortunately, the integration does not collect their data. Can you please modify the integration so that it receives data from other Temperature + Humidity sensors as well? Thanks

Hello. I bought more sensors for the weather station. Unfortunately, the integration does not collect their data. Can you please modify the integration so that it receives data from other Temperature + Humidity sensors as well? Thanks

@gut1986 Hi, sure. But I need to collect data from station. So enable 'dev log' in settings section and send me that log.

Hello. I bought more sensors for the weather station. Unfortunately, the integration does not collect their data. Can you please modify the integration so that it receives data from other Temperature + Humidity sensors as well? Thanks

@gut1986 Hi, sure. But I need to collect data from station. So enable 'dev log' in settings section and send me that log.

I have the "Dev Log" option enabled from the start.  Where is this log stored?  I don't see it among the standard HA logs (Settings->System->Logs).

Please don't forget me.  Thanks

@gut1986 I would not. But I am on vacation right now.

Please don't forget me. Thanks

@gut1986 You have to download full log. Or View full log. Then, there are data as this:
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking baro_pressure
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking outside_temp
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking dew_point
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking outside_humidity
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking wind_speed
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking wind_gust
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking wind_dir
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking rain
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking daily_rain
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking solar_radiation
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking uv
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking indoor_temp
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking indoor_humidity
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking ch2_temp
2024-07-15 01:40:28.829 INFO (MainThread) [custom_components.sws12500.utils] Checking ch2_humidity
2024-07-15 01:40:28.829 DEBUG (MainThread) [custom_components.sws12500] Manually updated sws12500 data
2024-07-15 01:40:28.831 INFO (MainThread) [custom_components.sws12500] Dev log: {'action': 'updateraww', 'realtime': '1', 'rtfreq': '5', 'dateutc': 'now', 'baromin': '30.01', 'tempf': '60.4', 'dewptf': '51.6', 'humidity': '73', 'windspeedmph': '4.2', 'windgustmph': '4.4', 'winddir': '312', 'rainin': '0.0', 'dailyrainin': '0.0', 'solarradiation': '0.0', 'UV': '0.0', 'indoortempf': '76.8', 'indoorhumidity': '52', 'soiltempf': '74.3', 'soilmoisture': '50'}

@schizza I already found it. I have added 2 more sensors (3 in total).
I guess it will be the records: Below is the full log.
soiltemp2f
soilmoisture2
soiltemp3f
soilmoisture3
2024-07-15 09:07:41.712 INFO (MainThread) [custom_components.sws12500.utils] Checking baro_pressure
2024-07-15 09:07:41.712 INFO (MainThread) [custom_components.sws12500.utils] Checking outside_temp
2024-07-15 09:07:41.712 INFO (MainThread) [custom_components.sws12500.utils] Checking dew_point
2024-07-15 09:07:41.712 INFO (MainThread) [custom_components.sws12500.utils] Checking outside_humidity
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking wind_speed
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking wind_gust
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking wind_dir
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking rain
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking daily_rain
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking solar_radiation
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking uv
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking indoor_temp
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking indoor_humidity
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking ch2_temp
2024-07-15 09:07:41.713 INFO (MainThread) [custom_components.sws12500.utils] Checking ch2_humidity
2024-07-15 09:07:41.714 DEBUG (MainThread) [custom_components.sws12500] Manually updated sws12500 data
2024-07-15 09:07:41.717 INFO (MainThread) [custom_components.sws12500] Dev log: {'action': 'updateraww', 'realtime': '1', 'rtfreq': '5', 'dateutc': 'now', 'baromin': '30.10', 'tempf': '77.5', 'dewptf': '60.0', 'humidity': '55', 'windspeedmph': '0.4', 'windgustmph': '0.4', 'winddir': '186', 'rainin': '0.0', 'dailyrainin': '0.0', 'solarradiation': '419.73', 'UV': '4.6', 'indoortempf': '76.6', 'indoorhumidity': '55', 'soiltempf': '75.9', 'soilmoisture': '60', 'soiltemp2f': '76.4', 'soilmoisture2': '61', 'soiltemp3f': '76.8', 'soilmoisture3': '61'}

@schizza  Hello. I managed to edit the files and add sensor 3 and 4 to them. Similarly, other sensors could be added up to the number of 7. I am attaching the files.
sws12500_VV.zip

Added support for CH3 and CH4 in #29


Testing rebase

Please read the contributing guidelines linked above before opening a pull request.

Sorry, ignore this.


Enhancements for diagnostic page

Summary
Fixes #6739

Occurred changes and/or fixed issues

Moved the latest logs to the bottom of the page
Compacted the system info table into 3 groups:

Browser
System
Javascript memory (doesn't show if not permitted)

Resource counts table

Collapsed by default if more than one cluster
Ignored 0 count resources
Added to table headers:

Total resource counts
Total number of nodes
Split cluster name and namespace

Added column for timings when generated
Fixed coloring to match standard table colors

Converted checkbox into async button for generating timings
Added dialog when users attempt to download data before generating response times

Technical notes summary

When the Download Diagnostic Package button is clicked it will check for response times with promptDownload, if the user has already generated the timings it will download the package as normal.
Screenshot/Video

https://user-images.githubusercontent.com/40806497/192038575-35629e31-0582-4930-ba89-f13bcaa69521.mp4

@kwwii Are either of these what you had in mind?
Example 1

Example 2

@jordojordo We seem to have lost the memory usage, os type from the top section - can we add that back in - the info is not in the download either.
Can we change 'Resource Counts' to 'Resource Counts by Cluster (N clusters)` - where N is the cluster count - just so its super easy to see the number of cluster.
Can we also run timings on the top 15 resources rather than 10 - there are quite a few 'system' resources, so it would be good to ensure we capture some of the others.

@jordojordo Still want this information:

to be shown - just in a more compact form.

@nwmac I have fixed the memory not showing up in the table/data, changed the title for 'Resource Counts', and changed the timing to add the top 15 resources.


A more innovative project sort option for 'idea hotness'

Something using recency of submission, vote, comment or volunteer which can kind of create a 'hotness' sort option. Looking for ideas/suggestions on how to weight or implement a formula or calculation.

https://github.com/campuscodefest/ccf/blob/project_hotness/app/services/project_hotness.rb
@chadfennell thoughts?

Cool idea. Not an area I'm at all well versed in, but some initial thoughts...
In the context of a single event, a hotness sort would only be relevant while the event is open for new submissions since a decay rate implies new/recent content. On the other hand, it would be cool (at some point down the road) to feature hot projects across multiple events. But yes, I can see having a hotness sort during or running up to an event. I can also see having to tweak the decay rate based on how active users are with new submissions.
Another, simpler sort, might be "total activity", which would give you hotness without the decay rate. Unfortunately, this results in a rich get richer situation. This might be useful though once an event is done and cannot receive new votes, submissions, comments, etc.

Stupid question. I'm assuming this: https://github.com/campuscodefest/ccf/blob/project_hotness/app/services/project_hotness.rb#L20
is to prevent having a  1 / Math.log(1) situation (or Math.log(-n)).
Any reason why it couldn't be expressed as
return 1 if created_days_ago <= decay_rate + 1
probably I'm missing something though. Thanks for doing this, interesting idea.

Regarding comment above, yes it was just some general hacking to avoid infinity, o for Log(0) and log(1),  and Log(2) was less than 1, so dividing it has a total greater than one. Probably better solutions out there, just what I had thus far.

I do like the idea of the decay rate being relative to the event activity as well, I hadn't thought that far ahead.

Sweet, well, I say go for it.

The hotness is now the default search!


Make the table load indicator only appear after 0.3 seconds of loading

this'll make things look nicer

Though with tables where changing the sort order won't impact the data they show, this might be actually confusing...


Replace table/collection view content size in PresentationController with UIScrollView

Replace the table view and collection view content size height with the scroll view content size height when calculating the adaptive frame size

presentedView.layoutIfNeeded()

...

if let scrollView = presentedView.subviews.first(where: { $0 is UIScrollView }) as? UIScrollView {
    targetHeight += scrollView.contentSize.height
}


Show warning when Record is accessed without type but RequireQualifiedAccess was set - relates to #95

As discussed with @dsyme in #95 this solves the second case of #95 (which is unfortunately already closed).

Excellent fix, great work.  I've reviewed the code and test and they LGTM.


chore: docs and readme version fix

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


[turborepo] HEAD^ filter on shallow clone stopped working in 1.9.1

What version of Turborepo are you using?
1.9.1
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Linux
Describe the Bug
Turbo tries to read a commit before HEAD^ when filtering.
npx<EMAIL_ADDRESS>run build --filter "...[HEAD^]" --dry json fails with the message:
Turbo error: failed to resolve packages to run: git error: object not found - no match for id (d9f5424696e8aa64fa7c92b88ee8d96bd529f008); class=Odb (9)

Expected Behavior
It should probably work on a shallow clone
To Reproduce
A repro, with a PR that fails with 1.9.1:
https://github.com/laat/turbo-shallow-repro/pull/2
The Github Action:
name: Node.js CI

on:
  push:
    branches: [ "main" ]
  pull_request:
    branches: [ "main" ]

jobs:
  build:

    runs-on: ubuntu-latest

    steps:
    - uses: actions/checkout@v3
      with:
        fetch-depth: 2
    - uses: actions/setup-node@v3

    # works
    - run: npx<EMAIL_ADDRESS>run build --filter "...[HEAD^]" --dry json

    # fails
    - run: npx<EMAIL_ADDRESS>run build --filter "...[HEAD^]" --dry json

Same issue on --filter="[HEAD~...HEAD]"
ERROR  run failed: failed to resolve packages to run: git error: object not found - no match for id (9914a2c527ce7efe318d4e9f0991997066a096fc); class=Odb (9)
Turbo error: failed to resolve packages to run: git error: object not found - no match for id (9914a2c527ce7efe318d4e9f0991997066a096fc); class=Odb (9)

Same issue on --filter='[main...HEAD^1]'
But only on one of my CI workflow
The other one is working flawlessly
ERROR  run failed: failed to resolve packages to run: git error: object not found - no match for id (e207f37719a34d8bb172595c541a433200d00c00); class=Odb (9)
Turbo error: failed to resolve packages to run: git error: object not found - no match for id (e207f37719a34d8bb172595c541a433200d00c00); class=Odb (9)

Similar issue when I use git tag: --filter=...[$TAG]

ERROR  run failed: failed to resolve packages to run: git error: object is no commit object; class=Invalid (3)
Turbo error: failed to resolve packages to run: git error: object is no commit object; class=Invalid (3)

solved in 1.9.3. 🎉 thanks @NicholasLYang

Fixed in https://github.com/vercel/turbo/pull/4604
Thanks for the reports!


Add TriggerMovedTableSchemaCheck

Summary
Adds TriggerMovedTableSchemaCheck
Work Item(s)
Fixes AB#523755

If you are implementing this consider: Moving the check to both OnInstall and Upgrade, you may miss the check otherwise Name it SchemaSanityCheck or something, add comments why we have this confusing code Add a method to validate after upgrade is done to check that the data is moved Consider emitting telemetry and setting up the alerts Have the methods only in v24.x so we track the upgrade, no need to have this in master going forward

The check is now on install and on upgrade. Added Sanity to the method name. Added some explanation in a doc trigger.
If we want to remove the code after it has been backported (HF'd) to 24.0 we can but process dictates we do the change in main, then 24.x then 24.0.


Book/untitled file name updates

Following changes:

When book files are opened as untitled files, the name of the book currently has the entire path as it's name. Made changes here to only have the filename instead of the path.
Previously untitled schema needed to applied differently on each platform but after the previous vscode merge this has not been the case.

What's the resulting change of this for relative links? is this just to change what shows up in the tab title?

What's the resulting change of this for relative links? is this just to change what shows up in the tab title?

Yep, just changes the how the title is shown on tab. When ADS refreshes, previously since the file-name has the entire path as it's name, it shows the whole path which looked really ugly.


LP-171 Add Makefile and update .gitignore

https://companieshouse.atlassian.net/browse/LP-171
Add Makefile
Update .gitignore to ignore the target folder that is used as the build output folder, and ignore intellij config files

looks like Analyse Pull request is failing

looks like Analyse Pull request is failing

it's failing because there is no makefile yet
make: *** No rule to make target 'build'.  Stop.


Field name clash with Elm reserved words

I have a simple schema with this object definition
type Service {
  name : String
  port :  Int
}

This causes graphqelm to throw an error, which actually seems to come from elm-format. The error message is formated weird so I won't paste it here.
Looking at the generated code (and the elm-format error) it was clear that port can not be used as a field name. I expect the same will be true for other words like module and import.
This is not a big problem, so this issue is more to see if it was possible to catch this naming clash with relevant reserved words in Elm and display a better error message.

Hello @madsflensted, thank you for reporting the issue! Yes, I’m handling the reserved word type right now but for some reason I forgot to add others! The normalization code should be handling this. The strategy I’m using is to just add a trailing _, so in this case the field would be called port_. I won’t be at a computer for a couple of weeks, but I’ll make this change as soon as I’m back home! Here’s the relevant file: https://github.com/dillonkearns/graphqelm/blob/a9bbff3d18968684d9a968f32983ce837ec8c449/src/Graphqelm/Generator/Normalize.elm#L9.

Resolved by #41, now fixed in npm version 3.1.8!


Firefox 130 connection issues

I have difficulties to connect to wtransport, using Firefox 130 and wtransport 0.3.1.
I have seen in #166 that there has been a recent fixup regarding Firefox support, however I have various behavior, depending of the OS:

Linux (Firefox 130.0.1): I have few connection issues
Windows (Firefox 131.0b8): I can't connect
macOS (Firefox 130.0.1): I can't connect

Some extra notes

I have no connection issue when using Chrome 128.0.6613.137.
I'm not using serverCertificateHashes because it seems not to be supported on Firefox. Instead I'm using a Let's Encrypt certificate

Here are some logs
2024-09-19T10:21:38.664 DEBUG wtransport::endpoint - New incoming QUIC connection
2024-09-19T10:21:38.664 DEBUG rustls::server::hs - decided upon suite TLS13_AES_128_GCM_SHA256
2024-09-19T10:21:38.665 DEBUG rustls::server::hs - Chosen ALPN protocol [104, 51]
2024-09-19T10:21:38.665 DEBUG quinn_proto::connection - failed to authenticate packet
2024-09-19T10:21:38.665 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.666 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.670 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.670 DEBUG quinn_proto::connection - ECN not acknowledged by peer
2024-09-19T10:21:38.676 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.676 DEBUG wtransport::driver - Driver; quic_id=2955843274976
2024-09-19T10:21:38.676 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.677 DEBUG wtransport::driver::worker - Started
2024-09-19T10:21:38.677 DEBUG wtransport::driver::worker - New incoming uni stream (2)
2024-09-19T10:21:38.677 DEBUG wtransport::driver::worker - Stream; message=id=2
2024-09-19T10:21:38.677 DEBUG wtransport::driver::worker - New incoming uni stream (6)
2024-09-19T10:21:38.677 DEBUG wtransport::driver::worker - Stream; message=id=6
2024-09-19T10:21:38.677 DEBUG wtransport::driver::worker - New incoming uni stream (10)
2024-09-19T10:21:38.678 DEBUG wtransport::driver::worker - Stream; message=id=10
2024-09-19T10:21:38.678 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.678 DEBUG wtransport::driver::worker - Type: Control
2024-09-19T10:21:38.678 DEBUG wtransport::driver::worker - Type: QPackEncoder
2024-09-19T10:21:38.678 DEBUG wtransport::driver::worker - Type: QPackDecoder
2024-09-19T10:21:38.678 DEBUG wtransport::driver::worker - Received: Settings({H3Datagram: 1, EnableWebTransport: 1, QPackMaxTableCapacity: 65536, QPackBlockedStreams: 20})
2024-09-19T10:21:38.681 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.683 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.686 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.689 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.711 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:38.711 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:43.713 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:43.713 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:43.724 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:48.736 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:48.737 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:48.740 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:53.743 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:53.744 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:53.748 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:58.751 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:58.751 DEBUG quinn::connection - drive; id=0
2024-09-19T10:21:58.755 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:03.768 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:03.769 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:03.772 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:08.783 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:08.783 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:08.786 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:09.375 DEBUG wtransport::driver::worker - Ended with error: NotConnected
2024-09-19T10:22:09.375 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:09.375 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:09.375 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:09.378 DEBUG quinn::connection - drive; id=0
2024-09-19T10:22:09.465 DEBUG quinn::connection - drive; id=0

Still not able to reproduce (on Linux 130.0.1).

I'm not using serverCertificateHashes because it seems not to be supported on Firefox. Instead I'm using a Let's Encrypt certificate

serverCertificateHashes is supported on Firefox since version 125: https://caniuse.com/mdn-api_webtransport_webtransport_options_servercertificatehashes_parameter

I reproduced the issue more than 5 times in a row with Firefox 131.0 on Linux vs wtransport 0.4.0, actually I never managed to connect. But I have no issue while running examples/full.rs. I'm going to check the differences with my integration of wtransport.
About serverCertificateHashes, I wasn't sure of the actual support in Firefox, I previously had some errors that seemed to be related to it. Since it seems I have another issue, maybe my interpretation of the real issue I was facing was wrong. I'm going to continue to use serverCertificateHashes for this issue


Add Shortcuts for Shadeboxes

Add shortcuts for shadeboxes.

Add a keyboard shortcut for shadeboxes. Currently there is none.  This may be a bit tricker than other shortcuts because shadbox syntax is quite different from other AFM extentions.
>[!BEGINSHADEBOX "This title should be in a shade box"]

This text should also be in the shaded box, as well as everything, including "This is the last line of the shade box.".

>[!NOTE]
>
>Don't overuse. For normal tables, we want to keep a consistent design across content.

![table tip](https://experienceleague.adobe.com/docs/authoring-guide-exl/assets/table-no-border.png?lang=en)

In a three-column table, you can also add  and  to distribute the cell content evenly across the view area. If it were not so, I would have told you.

This is the last line of the shade box.
> [!ENDSHADEBOX]


Error in server run

Hello/ After run server with command "start", i have error "Unknown command: "src/app.js"" and string  [nodemon] app crashed - waiting for file changes before starting... Do you know what is this and what i must to do for run server?

I don't understand how this error occurred. There might be a dependency problem facing on your computer. Please try to update those dependency. If still this problem occurs try to create a new project from beginning then copy this codes in your project one by one.


[FIX] Carthage build by adding missing PageIndicatorPosition class

This should fix Carthage build

Perfect, thank you for the fix!


DefaultTableNameHandler is not enough

DefaultTableNameHandler is not powerful enough.
Inside model_struct.go the function TableName calls for ToDBName and, sometimes, inflection.Plural when the TableName is not implemented in the struct.
Since golang does not support type hierarchy it's troublesome to implement TableName function for each table structure.
Therefore it's a limiting factor that requires unnecessary code to bypass the default behavior of nonexistent TableName function in order not to change the naming pattern of the database.
Since Pull Requests signalize to open an issue before adding new features I'm here to discuss if this is truly a matter and how to properly address it.

Sorry, but I don't get what you are try to do, could you make an example that DefaultTableNameHandler can't complete your requests?
Feel free to create a new issue then.


Super Task Button

Need justification for this, the decision predates my joining.

Need justification for this, the decision predates me.

Need justification for this, the decision predates me.

Need justification for this, the decision predates me.

Need justification for this, the decision predates me.

Need justification for this, the decision predates me.

Need justification for this, the decision predates me.

Need justification for this, the decision predates me.

Need justification for this, the decision predates me.

Found the decision record made in 2022 when it was looked into (please see "consider removing")
https://docs.google.com/document/d/1nS-yXZGMkKBZfFUR9zybBeDP1Lnmrj2_N9R9MuUTw0I/edit#heading=h.h9nlovlowub7


Add support for setting vxlan flood vtep

Change Summary

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[ ] Documentation content changes
[ ] Other (please describe):

Related Issue(s)
Related to #979
Component(s) name
arista.avd.eos_cli_config_gen
Proposed changes

Add supports for specifying flood vtep parameters on Vxlan1

How to test

Checklist:

[X] My code has been rebased from devel before I start
[X] I have read the CONTRIBUTING document.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have updated molecule CI testing accordingly
[X] All new and existing tests passed (pre-commit, make linting and make sanity-lint).

@carlbuchmann Fixed a small problem in the template, rebased to current devel and added the Molecule tests.
Could you verify that it looks correct? I could then handle the rest. :)

Ah, okay. Didn't realize there was a set data model nor that a change was coming. :)
I have internally ran it as it was in my initial PR. But the usage is not extensive yet, so it's easy enough to do a change and then change the parts that are setting the variables currently.
For the flood vtep itself it is a straightforward change, just dropping the remotes from in-between essentially.
But I see there are other changes proposed too, such as the VNI mapping changing quite a bit from a separate key to vlans having the VNI information.
Hence, I'd say that it's better to make the data model change somewhere else to avoid overloading this PR with other changes.

Ah, okay. Didn't realize there was a set data model nor that a change was coming. :)
I have internally ran it as it was in my initial PR. But the usage is not extensive yet, so it's easy enough to do a change and then change the parts that are setting the variables currently.
For the flood vtep itself it is a straightforward change, just dropping the remotes from in-between essentially.
But I see there are other changes proposed too, such as the VNI mapping changing quite a bit from a separate key to vlans having the VNI information.
Hence, I'd say that it's better to make the data model change somewhere else to avoid overloading this PR with other changes.

Thank you for your understanding...Me neither, it's only after a review of your PR that I realized the current data model didn't follow our own guidelines! And we are about to release AVD 3.0.0, so it's a perfect time to make these modifications!
Very good to know how you are leveraging AVD, and thank you for "upstreaming" your modifications - we really appreciate contributions from the external community!
So I will take over your changes in my fork, and create a new branch. I will keep this PR open until a new PR is created.

@quulah - just a quick update that we will continue to work on this after PR #1000 has been merged!

replaced by PR #1121


Merge Management.Sql backport branch into mainline branch

Description
We recently added support for SQL Managed Instance, Managed Instance database, and Managed Instance failover groups to the mainline psSdkJson6 branch in PR #4244 and released as 1.15.0-preview.
We then realised that we wanted to backport this change to before the 1.14.0-preview breaking changes, so in PR #4257 we cherry picked these changes into the new backport branch 'ManagementSql-CloudLifter' and released as 1.13.1-preview.
In order to reconcile these cherry picks and keep history healthy moving forward, we need to merge from the backport branch (ManagementSql-CloudLifter) into the mainline branch (psSdkJson6). This PR does that. There are no files changed, just merged from ManagementSql-CloudLifter into psSdkJson6 and then resolved conflicts.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] Please add REST spec PR link to the SDK PR
[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

@ziwa-msft and @shahabhijeet FYI. This is not urgent (does not block any release).

This PR has 0 files

There are no files changed, but there are merge conflicts that were resolved. All of them were resolved by 'keep ours', i.e. keep the psSdkJson6 change (not SqlMgmt-CloudLifter).

I still don't know why you would like to merge it. What happens if you don't merge it?
because you are honoring all the files from latest psSdkJson6, so does it really matter if you are mege?

Cherry picking in general is bad (Raymond Chen wrote a fascinating series of articles about it: https://blogs.msdn.microsoft.com/oldnewthing/20180323-01/?p=98325 ). There will soon be another PR for short-term retention going into SqlMgmt-CloudLifter branch, and when that PR lands we want to merge it into psSdkJson6 so that both branches have the same changes for the new feature. Merging this now resolved conflicts and will make that future merge easier.
If you want, we can hold off on this for now until that short term retention change lands in SqlMgmt-CloudLifter.

yes let's hold off on this one.

No problem.


Ability to filter claims returned from UserInfoEndpoint via scope

Hi,
Can we add a request to allow extra parameters to be passed to the UserInfoEndpoint service similarly to the TokenEndpoint? I've noticed a number of people have raised a need for this feature throughout the various forums.
I'm trying to retrofit IdentityServer4 into a legacy environment so I cannot change the system in any significant way. Without going into great detail about the domain, I need to retrieve the Claims from IdentityServer from within an API AuthorizationHandler. In our case, the claims cannot be passed around with the token.
From our AuthorizationHandler we access the "site" route which determines which part of the system the user is accessing (see sample below). I'm planning to align the "site" with an IdentityServer API scope. I then want to only retrieve the claims which are relevant to the scope "site".
The problem is that we only no the scope they're targeting at the point they call an API operation. They could potentially access multiple scopes via multiple calls to the API using the same access token. We do not want to return the claims for every scope on login or API authorisation as in our case it's extremely large and there's a noticeable performance hit.
Could you please include the ability to filter the claims returned from UserInfoEndpoint via something such as scope (or allow extra parameters like the TokenEndpoint)?
Sample implementation may be something like this
var userInfoClient = new UserInfoClient(disco.UserInfoEndpoint);
var response = await userInfoClient.GetAsync(tokenResponse.AccessToken, "scope1 scope2");

SAMPLE AuthorizationHandler
protected override Task HandleRequirementAsync(AuthorizationHandlerContext context, ClaimAuthorizationRequirement requirement)
{
    try
    {
        var filterContext = context.Resource as AuthorizationFilterContext;
        if (filterContext != null)
        {
            if (filterContext.RouteData.Values.Keys.Any(k => k == "site"))
            {
                var value = filterContext.RouteData.Values["site"] as string;
                if (!string.IsNullOrEmpty(value))
                {

I'm no expert but I'm curious to know why you want to pass this information around as claims? Does each site not have an independent (of Identity Server) way to get this information about the user?

Hi,
I'm actually not passing the claims around with the token. My API is retrieving the claims from IdentityServer via an AuthorizationHandler.
The "site" route actually corresponds to a SharePoint subweb. I need to use the ProfileService behind IdentityServer to build some claims where the source for the claims is data retrieved from the SharePoint subweb.
However, I need to pass into the ProfileService some identifier for the subweb so I know which subweb the user is trying to target. The two options I have mentioned in my above post would be to update the UserInfoEndpoint to include optional parameters like the TokenEndpoint. I believe the ability to filter the claims returned via allowed scopes would be the most elegant solution. I could then define each subweb as an allowable scope. I don't want to return all claims for all scopes as this would be extremely inefficient.
From my extensive reading around various forums I know adding this ability to the ProfileService would help a lot of people.

Out of curiosity, why so many identity claims associated with each API resource? Or are you using this as a permissions system (which is not what IdentityServer is designed to be)?

Any update on the issue? closing for now - feel free to re-open if it needs further discussion.


Resolve Out-of-Memory issues when working with larger spatial areas

Attempt to reduce high number of memory allocations to a more manageable level
(recalling that memory allocations are not the same as memory use - although a small reduction is a nice side-effect here).
Much of the effort is in streamlining the settler_DHW_tolerance!() function.
High memory use is somewhat expected in GBR-scale simulations (3806 reefs).
Currently, settler_DHW_tolerance!() is doing a naive check for any locations with $k$ > 0 - and it turns out all represented reefs have $k$ > 0.
Note: Please ignore changes to settler_cover() - I had to cherry pick some changes from another branch (for turning all calculations to be relative to $k$ area) and these were brought over by accident. The changes there don't affect anything.

@ConnectedSystems are you still going to add more things to this PR?

@ConnectedSystems are you still going to add more things to this PR?

Yes, I'll work on the changes we discussed earlier today.

Ready for first review I think @Zapiano

Note, I will be reverting back to dense matrices from sparse matrices.
While the reduction of memory use from using a sparse matrix for connectivity data is initially beneficial, Julia has to implicitly convert to dense matrices to do some of the calculations resulting in more than necessary memory allocations. So the benefits of having this dataset in a sparse matrix is outweighed by the conversion overheads.

Actually, please hold off on the review, I need to double check a few things

Ready for review @Zapiano.
32.362324 seconds (45.52 M allocations: 13.177 GiB, 11.05% gc time)

There are many other improvements in the consistent-cover branch that should reduce these timings further, but the changes in this PR addresses the key issue of large memory allocations for large spatial contexts.


Better permissions for auth_token

What does this PR do?
Change the permissions of the auth_token file to 0600.
Motivation
Security reasons.
Additional Notes
This change will prevent users that can't read the auth_token from running any datadog-agent commands that need to talk to the running agent endpoint. While this make sense IMO, it remove some ease of use. On linux the user will have to su dd-agent.

Only fail is on dogstatsd, merging...


Search in files and UTF-8

From contrebasse on 2012-11-15T07:43:32Z
Spyder Version:  2.1.10
Python Version:  2.7.3
Qt Version:      4.8.2, PyQt4 (API v1) 4.9.3 on Linux
What steps will reproduce the problem?

Create a file with a line containing non-ascii characters

Search in files (ctrl+shift+F) for this line What is the expected output? What do you see instead? While I expect to see the line with correct characters in the search widget, I see the line with a wrong encoding.

Please provide any additional information below
. Adding # -- coding: utf-8 -- at the top of the file changes nothing.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1192

From ccordoba12 on 2014-10-12T17:30:12Z
Labels: MS-v2.4 Cat-Miscelleneous

@rlaverde you have work with encoding a bit, could you check if this is still an issue?
@Nodd you are probably busy, but if you have some time for context it would be great :-). Happy new year by the way!


fix(template): add 'community' and 'shared' options to template type …

…filter

Add only community filter, because of the problems to undersend and get shared templates.


Request/question

Any chance for AVC or at least x265/hevc if it is possible, any time soon? Thanks for creating this.

Do you have any examples of a stream being in AV1? I know they supposedly accept it as ingest but I don't recall seeing anything about it being used as output yet.

Do you have any examples of a stream being in AV1? I know they supposedly accept it as ingest but I don't recall seeing anything about it being used as output yet.

I see, I figured it was default at this point and it was merely a choice which was missing, but as I am writing this I am only seeing 2 av1 streams and by the time you read this they are sure to be offline. I also assumed that there would be other options for the audio format or make it download as one file video + audio.
Then another thing I would ask or suggest to be added is if you could integrate a shutdown, sleep, hibernate script so it asks you what to do what to do after the download is complete and you can select do nothing, shutdown, sleep or hibernate system.
Also wonder if this is something that exists for android OS as well, I am surprised something like newpipe or a fork of it does not have this functionality, from what I know. And whether there are any plans on your part to make this compatible with other sites like Rumble.com
Thanks.

You can link the channels those AV1 streams are from, that way I can periodically check them for streams since I imagine they'd consistently be using AV1.

another thing I would ask or suggest to be added is if you could integrate a shutdown, sleep, hibernate script

There is a plan to eventually add a post-download hook that could be used for this, probably. I haven't done much of any work on ytarchive recently since I just don't program much after work these days, and there haven't been any features that I personally feel the burning need to add.

any plans on your part to make this compatible with other sites

Nope, never. This entire thing is a hack and only works with the way Youtube does things. While theoretically some of the logic could be used for other sites, those sites would also need to use sequential fragment files and have them be persistent while the stream is online. But I have no interest in other sites as I do not have anyone I watch from other sites.

Do you have any examples of a stream being in AV1? I know they supposedly accept it as ingest but I don't recall seeing anything about it being used as output yet.

So I searched "av1" at various times and clicked "live" and collected a couple of channels,
https://www.youtube.com/@925Warriors
https://www.youtube.com/channel/UCTpP5m-ecJby7NeiC6R5shQ
https://www.youtube.com/@planetaryengineer
https://www.youtube.com/@Fixzao
https://www.youtube.com/@JapanGame
https://www.youtube.com/@Sat1sfayeR
https://www.youtube.com/@DiamondSigns
https://www.youtube.com/@AppleCham

One of those channels is live, mentions AV1 in the title, but the output is only H264 and VP9. AV1 output is not a thing yet as far as I can see.


Pass important postgres settings to docker-entrypoint.sh via ENV vars

Thanks for your work so far guys!
The postgres defaults are quite modest (especially memory). Most settings can nowaydays be set via the ALTER SYSTEM query and a reload but some settings require a restart  and therefore should be set before postgres starts.
This is just a proposal how it could be done. I would be happy to see what you think about it and which settings to include and whether it is even in the scope of this Docker image.
I think it would make quick configuration of postgres containers much easier because you don't have to create your own Docker image just for tweaking some very basic settings like shared_buffers.
It also allows schedulers for example to set the shared_buffers options based on metadata.
I simply modified the docker-entrypoint.sh of the latest Docker image for now, if you think it is a good idea in general I will modify the others as well.

If something like this is going to happen, I'd prefer to see it happen in a generic way based on prefix or something, rather than supporting specific settings. I could see having all POSTGRES_CONF_* variables converted to their downcased versions. With that, you could set POSTGRES_CONF_SHARED_BUFFERS or whatever.

If something like this is going to happen, I'd prefer to see it happen in a generic way based on prefix or something, rather than supporting specific settings. I could see having all POSTGRES_CONF_* variables converted to their downcased versions. With that, you could set POSTGRES_CONF_SHARED_BUFFERS or whatever.

Yes that is preferable. I'll work on that.

Should be pretty straightforward with ${!POSTGRES_CONF_*}. You may want to see what the maintainers think before putting a ton of effort into it though.

Should be pretty straightforward with ${!POSTGRES_CONF_*}. You may want to see what the maintainers think before putting a ton of effort into it though.

I didn't get it quite that elegant for now. But it works quite well.
docker run -e POSTGRES_PASSWORD=mysecretpassword \
-e POSTGRES_CONF_SHARED_BUFFERS=25GB \
-e POSTGRES_CONF_WAL_LEVEL=minimal \
-e POSTGRES_CONF_EFFECTIVE_CACHE_SIZE=75GB \
-e POSTGRES_CONF_MAX_CONNECTIONS=200 -d postgres

But let's see.

I don't know man, that's the first I'd seen of the ${param,,} expansion. Nice work.

Thanks for the comments, I fixed the issues and it still works :grin:
Expanding the vars with ${!POSTGRES_CONF_*} is a really cool trick I didn't know about.
Thats definitly useful for config scripts.

Isn't the generated postgresql.conf created from /usr/share/postgresql/9.4/postgresql.conf.sample?

@tianon, it is mostly the same, but I don't know where it gets most of the changes:
root@14814ca1bbe4:/var/lib/postgresql/data# diff postgresql.conf /usr/share/postgresql/9.4/postgresql.conf.sample
59c59
< listen_addresses = '*'		# what IP address(es) to listen on;
---
> #listen_addresses = 'localhost'		# what IP address(es) to listen on;
64c64
< max_connections = 100			# (change requires restart)
---
> #max_connections = 100			# (change requires restart)
68c68
< #unix_socket_directories = '/var/run/postgresql'	# comma-separated list of directories
---
> #unix_socket_directories = '/tmp'	# comma-separated list of directories
115c115
< shared_buffers = 128MB			# min 128kB
---
> #shared_buffers = 32MB			# min 128kB
130c130
< dynamic_shared_memory_type = posix	# the default is the first option
---
> #dynamic_shared_memory_type = posix	# the default is the first option
438c438
< log_timezone = 'UTC'
---
> #log_timezone = 'GMT'
523c523
< datestyle = 'iso, mdy'
---
> #datestyle = 'iso, mdy'
525c525
< timezone = 'UTC'
---
> #timezone = 'GMT'
538c538
< lc_messages = 'en_US.utf8'			# locale for system error message
---
> #lc_messages = 'C'			# locale for system error message
540,542c540,542
< lc_monetary = 'en_US.utf8'			# locale for monetary formatting
< lc_numeric = 'en_US.utf8'			# locale for number formatting
< lc_time = 'en_US.utf8'				# locale for time formatting
---
> #lc_monetary = 'C'			# locale for monetary formatting
> #lc_numeric = 'C'			# locale for number formatting
> #lc_time = 'C'				# locale for time formatting
545c545
< default_text_search_config = 'pg_catalog.english'
---
> #default_text_search_config = 'pg_catalog.simple'

You think we should write the settings to /usr/share/postgresql/9.4/postgresql.conf.sample?

Should I continue to modify the Dockerfiles of all versions or should we just support it for the newest one?

What I meant was that if we can figure out which file these settings come from, we can recommend that users modify that file directly instead.
I ran a test appending some content to the end of postgresql.conf.sample before running the entrypoint, and it did show up in the final postgresql.conf, which means that the final files are generated directly from the .sample files in that directory.
What this means for my opinion on this PR is that I'd prefer that we instead document to users where postgresql.conf.sample can be found, and how to either edit or replace its contents appropriately for changes like this.
Something like FROM postgres:9.4 followed by RUN sed 's/.../.../' /usr/share/postgresql/*/postgresql.conf.sample or COPY some-file.conf.sample /usr/share/postgresql/9.4/postgresql.conf.sample should be sufficient for modifying the defaults for these settings.  Combine that with automated builds (https://docs.docker.com/docker-hub/builds/) and repository links (https://docs.docker.com/docker-hub/builds/#repository-links) and it's reasonably easy to have an up-to-date image built FROM this one that comes with these settings pre-configured before postgres ever starts up.

Yes that's a good option. But I think it is significant more work to create an own Docker Image and configure an automated build with repository links than just setting an environment variable for a config value that is nearly always too conservative.
Settings like SHARED_BUFFERS depend on the machine and environment they run on and perhaps shouldn't be baked into an image.

Really cool your patch, Lukas,
I hadn't seen before and copied this to my /docker-entrypoint-initdb.d/ :
#!/bin/bash
cat $PGDATA/postgresql.conf | grep '=' | sed 's/ =.*$//;s/^#//' | grep '^[a-z]' | sort | awk '{ print toupper ($0) }' | while read pgvar
do
	eval pgval=\$$pgvar
	if [ x"${pgval}" != "x" ]
	then
		lowcase=$( echo $pgvar | awk '{ print tolower ($0) }')
		echo "$lowcase = $pgval"
	fi
done > $PGDATA/postgresql.docker.conf
cat $PGDATA/postgresql.docker.conf >> $PGDATA/postgresql.conf

No thats correct and it is problematic - I mean we also cannot modify the postgres config once it has been mounted in $PGDATA.
But I think we need a solution that very essential things like the shared_buffers settings are not baked into an image. This is my take at it.

With the incoming #127 you can more easily pass parameters to postgres:
$ docker run -d postgres -c shared_buffers=256MB ...

Or override the default config file with your own, so that postgres will use it to generate the real one:
$ docker run -d -v /my/pg.conf:/usr/share/postgresql/postgresql.conf.sample postgres

... -c shared_buffers=256MB? Did you mean -e, @yosifkit?

Nope, I mean passing args to the postgres daemon itself: -c NAME=VALUE      set run-time parameter  and not as environment variables that would require the entrypoint script to parse.  Which is why they are at the end of the command so that they are arguments to the container and not arguments for docker run.

Reading through this closed ticket, I'm still not sure how to pass shared_buffers to my postgres image. What's the proper way to do this in 2017?

@Ramblurr as noted by @yosifkit in the previous comment:

passing args to the postgres daemon itself: -c NAME=VALUE      set run-time parameter

For example
$ docker run -dit --name pg postgres -c shared_buffers=4096MB


Roll Engine from a99d51f0c128 to 3edd312b90e0 (5 revisions)

https://github.com/flutter/engine/compare/a99d51f0c128...3edd312b90e0
2022-03-18<EMAIL_ADDRESS>Add deps to builder web_engine_framework (flutter/engine#32112)
2022-03-18<EMAIL_ADDRESS>allow experimentally opting into MSAA on canvaskit (flutter/engine#32045)
2022-03-18<EMAIL_ADDRESS>Roll Skia from 42bc49d5d332 to 35275b3d511f (5 revisions) (flutter/engine#32119)
2022-03-18<EMAIL_ADDRESS>Roll Skia from 1df1a80ad20d to 42bc49d5d332 (3 revisions) (flutter/engine#32116)
2022-03-18<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from uawdfLUeA... to voPt0GIuh... (flutter/engine#32115)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from uawdfLUeA1ce to voPt0GIuhsWq
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Wallet do not recover properly from a backup

I've created a clean wallet, sent 251k tstrat to it, made a backup
Created bunch of txes to myself => got 400 UTXOs and 50mb wallet file
replaced 50mb wallet file with the backup
when the wallet synced it said I have 247k and 10 UTXOs. so it did not recover from the backup well.

Here is the first tx that splits 251k into several small ones:
https://chainz.cryptoid.info/strat-test/tx.dws?564517.htm
if you follow the transaction 2-3 layers in depth you will find around 400txes that are not redeemed.
after the wallet recovery my wallet sees only this UTXOs:
Address: TX9HCTc7SvSFS9qMu6686NhwMpgVUSraFB Amount:<PHONE_NUMBER>5561 | Address: TH26SbKKfXKD7wHUU4CE4bZATAYzQMqNwm Amount: 9.99571051 | Address: TYgpzmbVZ8y1k8kwepCHDtLg1qdHRP2P18 Amount: 9.99723616 | Address: TJQzTFSb57E59rDheX2K8dDt9W9StuaYtw Amount: 9.99731221 | Address: TYgRM8KS7FJWdr7o3ScENeNxkzusyDxxXB Amount: 9.94652062 | Address: TB5j9naEkVxHGicagR4JNkxN9nEmZYhDPe Amount: 9.98981354 | Address: TLpNdPJ7G9yk5TeRUcDoBLA5vPf8umdd6Y Amount: 9.99045421 | Address: TD5HfTkYha5DbEbaSkhbXxxQVYVytZ9Ub1 Amount: 9.99146990 | Address: TMyfeD61Wdo6sD5LAvE76PyrbQkhJvLxcq Amount: 9.99430228 | Address: TPZQRdRxwbjVexwFKvfTBrC4wRF2RnVhe2 Amount: 9.99813972 | Address: THV8vWTowKhHjqsHyec8EjQQVD6d4heQva Amount: 9.99822403
this is the branch that has code for reproducing this bug:
https://github.com/noescape00/StratisBitcoinFullNode/tree/txFlud
This maybe caused due to a large amount of UTXOs OR because special type of txs was used- I've created txes with 20-200 outputs instead of usual 1-2. Anyway both scenarios should be well supported.

From a discussion with @noescape00:

our wallet is bip 44 compatible and therefore having more than 20 unused addresses is not required
what if the transaction has only 1 output but it's one of our addresses at index 120? We'll miss it. If you wanted to "look further", how far do you go?
for us to support a number like 100, 200 or 500 unused addresses will make our node slower. Maybe not by much but still.
Closed as not a bug.


Add comment about version sync between Cargo files

Closes #1748

@IronGauntlets is it ok if I add tests that will run using rust compiled code? to make sure that we solve the issue once and for all


Add text overrun behavior for the theme type selector

This PR makes sure that even if a theme type has an unreasonably long name, it will no longer break the editor UI.

This uses the new text overrun behavior for buttons (https://github.com/godotengine/godot/pull/61819) similar to https://github.com/godotengine/godot/pull/61702. Unfortunately, it doesn't look as nice as it should, because there seems to be a regression in this feature (https://github.com/godotengine/godot/issues/63914).

Thanks!


Validate phones numbers to 10 digits

Fix length of mobile number to 10 digits starting with (6,7,8,9) and and remaining could be any digit. This would be enforced only during submit and strangely max_length was not forced before submit.
Prevents the following issue (Taken from production website)

Fix #191

As I already pointed out in #191  , this will work BUT it will cause a lot of confusion on the Request for Help page.
Steps:

Fill all the details in the Request for Help except the phone number. Make the phone number less than 10 digits long.
Submit the form
The validator will reject the entry, and redirect back to the page. The "Allow Location - നിങ്ങളുടെ സ്ഥലം കൊടുക്കുക" modal will pop up again(since it shows up on window.load), making the user think that it's the location that's missing.

Looks good for me.

Thank @Neonkraft for pointing that out and excuse me for overlooking the thread, Made the patch few hours ago and just submitted it. Will try to find alternative methods in the meantime.

@salmanulfarzy that's fine. I guess we'll leave the call to the admins.

Can we try JS validation instead?

I have added verification using js. It prevents alphabets or negatives from being entered in the field itself. But one must never rely on frontend verification

@Gotham13121997 is implementing front end validation on #207

Merged with #581


chore(YOLOX): refactored codebase using Sourcery

Are you using soucery  to refactor  our code?

Yes that’s correct.
On Sun, 1 May 2022 at 1:38 pm, Feng Wang @.***> wrote:

Are you using soucery to refactor our code?
—
Reply to this email directly, view it on GitHub
https://github.com/Megvii-BaseDetection/YOLOX/pull/1288#issuecomment-1114119567,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH3XEYDOB4K5YPFL5GTTTF3VHX4DHANCNFSM5UW4LW5Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Best,
Parvesh
P : Please consider the environment before printing this e-mail
CONFIDENTIALITY / DISCLAIMER NOTICE: This e-mail and any attachments may
contain confidential and privileged information. If you are not the
intended recipient, please notify the sender immediately by return e-mail,
delete this e-mail, and destroy any copies. Any dissemination or use of
this information by a person other than the intended recipient is
unauthorized and may be illegal.

@Parv3sh Thanks for your contriubtion. In my opinion, some code of refactoring in this PR is not a good choice.
Your PR is huge (through it doesn't affect execution),  it will take lots of time to for me to review this. Plz wait.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Parv3sh:x: snyk-botYou have signed the CLA already but the status is still pending? Let us recheck it.

This PR is huge to review and the refactor suggestion provided by sourcery is not good enough for me here. So I decide to close this PR.
Thanks for contribution though @Parv3sh .


Crash upon creating world

https://paste.dimdev.org/bakeramowe.mccrash

What modpack are you running? Looking at the code for Patchouli I can't see anything that would directly cause this issue.

Custom. As soon as I remove it I'm able to load it again..

May i suggest trying out PI
https://minecraft.curseforge.com/projects/project-intelligence as a replacement for Patchouli

@xElementzx Twilight Forest + Cuisine, which is in my pack requires Patchouli.

@Rongmario Can you provided the latest.log and debug.log from your Client when the crash occurs.


Selenoid UI with private access

I believe that is neccesary have private access to Selenoid UI with user/password maybe.

@zeroray nothing prevents you from running Selenoid UI behind Nginx where this feature is already implemented.

Indeed, Nginx or Caddy would be suitable layers to put infront of the web UI for authentication.


"System.ArgumentException: The path is not of a legal form." - it's not?

1/7/2024 5:04:21 AM.342] LSPD First Response: [DispatchLLM] ERROR: System.ArgumentException: The path is not of a legal form.
[11/7/2024 5:04:21 AM.346] at System.IO.Path.LegacyNormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
[11/7/2024 5:04:21 AM.347] at System.IO.Path.InternalGetDirectoryName(String path)
[11/7/2024 5:04:21 AM.347] at Whisper.net.LibraryLoader.NativeLibraryLoader.<GetRuntimePaths>d__4.MoveNext()
[11/7/2024 5:04:21 AM.348] at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
[11/7/2024 5:04:21 AM.349] at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
[11/7/2024 5:04:21 AM.349] at Whisper.net.LibraryLoader.NativeLibraryLoader.LoadLibraryComponent()
[11/7/2024 5:04:21 AM.350] at Whisper.net.LibraryLoader.NativeLibraryLoader.LoadNativeLibrary()
[11/7/2024 5:04:21 AM.351] at Whisper.net.WhisperFactory.<>c.<.cctor>b__10_0()
[11/7/2024 5:04:21 AM.352] at System.Lazy`1.CreateValue()
[11/7/2024 5:04:21 AM.352] at System.Lazy`1.LazyInitValue()
[11/7/2024 5:04:21 AM.353] at Whisper.net.WhisperFactory..ctor(IWhisperProcessorModelLoader loader, Boolean delayInit)
[11/7/2024 5:04:21 AM.354] at Whisper.net.WhisperFactory.FromPath(String path, Boolean delayInitialization)
[11/7/2024 5:04:21 AM.354] at Dispatch_LLM.SpeechToTextManager..ctor() in D:\#UserData\Dokumente 2 - TB\Development\Visual Studio\Repos\Dispatch LLM\SpeechToTextManager.cs:line 26

Code is:
var modelName = "ggml-base.bin";
whisperFactory = WhisperFactory.FromPath(modelName);     //Here it crashes

Wanted Path is
G:\SteamLibrary\steamapps\common\Grand Theft Auto V\ggml-base.bin
Work on my console program flawless. Even tho the console program path has more complex syntax. for example it uses # in its path.
C# DotNet Framework.
x64
11.0
Did i miss something?

Hello @ziipzaaapM16A4 ,
Based on the stacktrace, the problem is not with the path for the model, but with the path for the native library:
https://github.com/sandrohanea/whisper.net/blob/268934fd7fd724d615c32e3a5410c201cc236e4d/Whisper.net/LibraryLoader/NativeLibraryLoader.cs#L196
Can you, please, try to see which one if these are failing for your system?
  var assemblyLocation = typeof(NativeLibraryLoader).Assembly.Location;
        var environmentAppStartLocation = Environment.GetCommandLineArgs()[0];
        // NetFramework and Mono will crash if we try to get the directory of an empty string.
        var assemblySearchPaths = new[]
            {
                string.IsNullOrWhiteSpace(RuntimeOptions.Instance.LibraryPath) ? null : Path.GetDirectoryName(RuntimeOptions.Instance.LibraryPath),
                AppDomain.CurrentDomain.RelativeSearchPath,
                AppDomain.CurrentDomain.BaseDirectory,
                string.IsNullOrWhiteSpace(assemblyLocation) ? null : Path.GetDirectoryName(assemblyLocation),
                string.IsNullOrWhiteSpace(environmentAppStartLocation) ? null : Path.GetDirectoryName(environmentAppStartLocation),
            }.Where(it => !string.IsNullOrEmpty(it)).Distinct();

It would be great to also have the actual path that was generated in order to detect and skip it.
I expect it is either assemblyLocation or environmentAppStartLocation :
  var assemblyLocation = typeof(NativeLibraryLoader).Assembly.Location;
  var environmentAppStartLocation = Environment.GetCommandLineArgs()[0];
  Path.GetDirectoryName(assemblyLocation);
  Path.GetDirectoryName(environmentAppStartLocation);

Added https://github.com/sandrohanea/whisper.net/commit/4f6e22e42d1a39dbaca9289baecac3ac85bd9cef to ensure that all frameworks will allow that DirectoryName to be parsed. It will be available in the next release.

Can you, please, try to see which one if these are failing for your system?
  var assemblyLocation = typeof(NativeLibraryLoader).Assembly.Location;
        var environmentAppStartLocation = Environment.GetCommandLineArgs()[0];
        // NetFramework and Mono will crash if we try to get the directory of an empty string.
        var assemblySearchPaths = new[]
            {
                string.IsNullOrWhiteSpace(RuntimeOptions.Instance.LibraryPath) ? null : Path.GetDirectoryName(RuntimeOptions.Instance.LibraryPath),
                AppDomain.CurrentDomain.RelativeSearchPath,
                AppDomain.CurrentDomain.BaseDirectory,
                string.IsNullOrWhiteSpace(assemblyLocation) ? null : Path.GetDirectoryName(assemblyLocation),
                string.IsNullOrWhiteSpace(environmentAppStartLocation) ? null : Path.GetDirectoryName(environmentAppStartLocation),
            }.Where(it => !string.IsNullOrEmpty(it)).Distinct();

Its says
RuntimeOptions.Instance.LibraryPath => LibraryPath is not existent => Compiler Error
The other code:
            var assemblyLocation = typeof(NativeLibraryLoader).Assembly.Location;
            var environmentAppStartLocation = Environment.GetCommandLineArgs()[0];
            RONFuncs.LogTrivial("sandrohanea code sample: " + assemblyLocation.ToString());
            RONFuncs.LogTrivial("sandrohanea code sample: " + environmentAppStartLocation.ToString());
            RONFuncs.LogTrivial("sandrohanea code sample: " + Path.GetDirectoryName(assemblyLocation));
            RONFuncs.LogTrivial("sandrohanea code sample: " + Path.GetDirectoryName(environmentAppStartLocation));

results:
[11/8/2024 8:04:02 PM.468] LSPD First Response: sandrohanea code sample:
[11/8/2024 8:04:02 PM.470] LSPD First Response: sandrohanea code sample: G:\SteamLibrary\steamapps\common\Grand Theft Auto V\GTA5.exe
[11/8/2024 8:04:02 PM.472] LSPD First Response: [DispatchLLM] ERROR: System.ArgumentException: The path is not of a legal form.
[11/8/2024 8:04:02 PM.473] at System.IO.Path.LegacyNormalizePath(String path, Boolean fullCheck, Int32 maxPathLength, Boolean expandShortPaths)
[11/8/2024 8:04:02 PM.474] at System.IO.Path.InternalGetDirectoryName(String path)
[11/8/2024 8:04:02 PM.474] at Dispatch_LLM.SpeechToTextManager..ctor() in D:\#UserData\Dokumente 2 - TB\Development\Visual Studio\Repos\Dispatch LLM\SpeechToTextManager.cs:line 37
[11/8/2024 8:04:02 PM.475] at Dispatch_LLM.Main.<>c.<StartPlugin>b__4_0() in D:\#UserData\Dokumente 2 - TB\Development\Visual Studio\Repos\Dispatch LLM\Main.cs:line 58

Got it, thanks!
Indeed, as the "environmentAppStartLocation" is inside the gta5 directory, it is causing issues for detecting the runtime dlls as it crashes before that. The commit with the try-catch above should help in this case.

but this crashes anyways right? so i will be unable to use it.
or did i not understand this?

Even if manually setting the path with RuntimeOptions.Instance.SetLibraryPath(...) that version would still crash for your setup as all the paths are evaluated including environmentAppStartLocation.
The fix was released in 1.7.2 yesteday, and you should be able to use that version that handles the exceptions for netframework setup with invalid path on one of the options.


JAVA-2921: causal consistency example for Java driver

Add Java example for causal consistency.

(issued PR by accident)


How to ignore a folder

I have a project which also contains a web app. Now when I build the project it wants to build all the native extensions for that web app too. How can I pass in an option to the mighty compiler to tell it to ignore that folder (should work in both ARMCC and GCC).

You should be able to use .mbedignore files: https://github.com/ARMmbed/mbed-os/blob/master/docs/ignoring_files_from_build.md

Yep, great. Thanks @bridadan .


removing controllerregistration admission plugin

What this PR does / why we need it:
The controllerregistration admission plugin enforces that a particular kind-type combination (e.g Infrastrucutre - aws) cannot be registered by multiple controllerregistrations.
However there are use cases when that is required.
Example when that is required:

Gardener uses private hosted zone as the internal domain for Shoot clusters. This is the case for aws-gov-cloud. The regular provider extensions do not support private zones.
In order to support private zones, we have another extension (private-zone-extension), also acting on (Infrastructure - aws) , assigning the created VPC to the private hosted zone

To support the above mentioned use-case, there are currently controller-registrations that have no kind-type association. However with the recent changes introduced in the Gardener Controller Manager, controllerinstallations of such a registration are being indentified as not needed and consequently removed.
To sum it up

Gardener should not enforce that a particular kind-type combination (e.g Infrastrucutre - aws) cannot be registered by multiple controllerregistrations.
Gardener  should validate that every controllerregistration has at least one  kind-type association.

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Sure lets close this PR. Thanks for the suggestion.


Improve and generate docs

Ready for review.
See http://go-http-api.readthedocs.org/ for what the docs look like.

Merged since this is a docs PR and it's been 5 months. :)


when.js - Add definition with tests for spread

when/when.d.ts
to authors(@derekcicerone @Nemo157). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

Sorry, the spread functionality is new to me so I'm not entirely sure I understand it. The definition looks reasonable to me but I just can't 100% confirm its correct.

@derekcicerone okey! thanks.
@Nemo157 is there anything?

Sorry about the delay. @vvakame it looks good enough, I don't think it's possible to fully support spread in TypeScript's type system. :+1:

@Nemo157 thanks!
@derekcicerone thanks mate!


Fix Travis License Test

Fix #152

On travis the repository is located in /home/travis/build/omichel/webots and since we are skipping all the build directories all no file is tested at all. I will fix this.

The exact same issue was also present in the clang-format test.

test me

test me

Build Passed :ok_hand:
You can safely merge 'fix-travis-license-test' into 'master'.

:+1:    (thank you for the improvement)

Well spotted.
This leads to new errors in the current PRs.

This leads to new errors in the current PRs.

Yes, that was to be expected.


I suggest having a CLI option for a custom delay between requests.

Is your feature request related to a problem? Please describe.
Using the CLI is pretty useless when most APIs put a limit on the requests/minute rate.
Describe the solution you'd like
I suggest adding a CLI option for a custom delay in seconds between requests, it also would be nice to have an option for adding a proxy list.
Describe alternatives you've considered
I tried using the Python library directly, but I would have to go through all cases.
Additional context
No such thing.

Hi @sleek-geek
I was thinking about a more full-featured throttling (as briefly touched in #910) - and actually, I have it implemented on top of PyrateLimiter, but not published it just yet. Adding delays could be done temporarily via the before_call hook, but I'll open source throttling soon. Not sure about the public-facing API yet, but it could be something like --rate-limit=10/m.
Also, it could be nice to add support for reading response headers and adding delays based on the delay value there.
As for the proxy list - I'd like to have it in the public API too, at the moment proxies could be specified via env variables as in this recipe

I am going to close this issue in favor of #910 and will add additional context there, also, the proxy list & headers will go to their separate issues to track the progress easier.

Btw, here is an example of a hook that could be used to add delays between requests.


No Client Internet Connectivity

Hello-
I installed pivpn on a Pi3 that is running Stretch lite. Nothing else installed. Only other config change was to enable ssh. I selected Google as my dns provider during installation.
I'm able to connect to it via the internet and an android phone and see my network. I can connect to the pi using the local ip address via ssh.
I am not able to connect to the internet through it which was my intended purpose. Ipconfig shows the gateway as blank. I am not sure if this a client or server issue. I am using the client from openvpn.net in Windows 10.
I've seen a handful of posts about internet connectivity not working here which have confused me more. I have tried uninstalling and reinstalling as someone suggested but still the same issue. Everything looks correct. I will post my config info below.
rules.v4
Generated by iptables-save v1.6.0 on Fri Nov 17 03:41:54 2017
*nat
:PREROUTING ACCEPT [1:40]
:INPUT ACCEPT [1:40]
:OUTPUT ACCEPT [0:0]
:POSTROUTING ACCEPT [0:0]
-A POSTROUTING -s <IP_ADDRESS>/24 -o eth0 -j MASQUERADE
COMMIT
# Completed on Fri Nov 17 03:41:54 2017
# Generated by iptables-save v1.6.0 on Fri Nov 17 03:41:54 2017
*filter
:INPUT ACCEPT [12502:1038592]
:FORWARD ACCEPT [0:0]
:OUTPUT ACCEPT [8955:989701]
COMMIT
# Completed on Fri Nov 17 03:41:54 2017

Server.Conf
dev tun
proto udp
port 1194
ca /etc/openvpn/easy-rsa/pki/ca.crt
cert /etc/openvpn/easy-rsa/pki/issued/server_heZeF96mIC1zngmf.crt
key /etc/openvpn/easy-rsa/pki/private/server_heZeF96mIC1zngmf.key
dh /etc/openvpn/easy-rsa/pki/dh2048.pem
topology subnet
server <IP_ADDRESS> <IP_ADDRESS>
# server and remote endpoints
ifconfig <IP_ADDRESS> <IP_ADDRESS>
# Add route to Client routing table for the OpenVPN Server
push "route <IP_ADDRESS> <IP_ADDRESS>"
# Add route to Client routing table for the OPenVPN Subnet
push "route <IP_ADDRESS> <IP_ADDRESS>"
# your local subnet
push "route <IP_ADDRESS> "
# Set your primary domain name server address for clients
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
# Override the Client default gateway by using <IP_ADDRESS>/1 and
# <IP_ADDRESS>/1 rather than <IP_ADDRESS>/0. This has the benefit of
# overriding but not wiping out the original default gateway.
push "redirect-gateway def1"
client-to-client
keepalive 10 120
remote-cert-tls client
tls-version-min 1.2
tls-auth /etc/openvpn/easy-rsa/pki/ta.key 0
cipher AES-256-CBC
auth SHA256
comp-lzo
user nobody
group nogroup
persist-key
persist-tun
crl-verify /etc/openvpn/crl.pem
status /var/log/openvpn-status.log 20
status-version 3
log /var/log/openvpn.log
verb 1
# Generated for use by PiVPN.io

Anyone have an idea how I can resolve?

Same issue using iOS and Ubuntu clients. My config is similar

Seems related to issue #380 has anyone found a resolution?

I have not found a solution. I've blown away my SD card for the Pi a few times now and started over.
Now I am not able to connect to the Pi at all. I get a TLS failed message. It was at least able to connect before.

From a Windows 10 client I can use the route change <IP_ADDRESS> mask <IP_ADDRESS>  here to manually add a gateway. I am not sure why it won't add it automatically. Still looking for an answer.

Same problem here after installing pippin on a fresh install of stretch. Guess the only way to have it working is to go back to Jessie right now..

@jbutler992 it seems to me that this issue as nothing to do with the issue you mentioned as that issue was related to an existing docker setup which was causing issues with iptables rules.
@ashdavely For windows users please make sure you are running OpenVPN with right click and run as administrator ... otherwise, it won't push the routes!
Try that and then come back with feedback please... I might take a while to come back and reply .. but My time is quite short righ now.

I am running into the same issue with no client internet connectivity. I have a fresh install of PiVPN on a Pi Zero W with Stretch Lite. UFW is installed but currently disabled. Dynamic DNS is set up and port forwarding enabled on my router. I can successfully connect to the VPN from my Android phone over cellular network. From the phone I can SSH into the Pi and ping google.com works. However, everywhere else there is no internet availability (e.g. apps, web browser, etc.).
From reading other threads regarding this issue, I tried calling sudo iptables -t nat -A POSTROUTING -s <IP_ADDRESS>/24 -o wlan0 -j MASQUERADE; but it doesn't make any difference in my case.
Any help would be appreciated! Would love to resolve this as my next step is to pair it with Pihole. Thanks for this great package!
server.conf:
dev tun
proto udp
port 1129
ca /etc/openvpn/easy-rsa/pki/ca.crt
cert /etc/openvpn/easy-rsa/pki/issued/server_5SR646nOZ52B1bt2.crt
key /etc/openvpn/easy-rsa/pki/private/server_5SR646nOZ52B1bt2.key
dh /etc/openvpn/easy-rsa/pki/dh2048.pem
topology subnet
server <IP_ADDRESS> <IP_ADDRESS>
# Set your primary domain name server address for clients
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
# Override the Client default gateway by using <IP_ADDRESS>/1 and
# <IP_ADDRESS>/1 rather than <IP_ADDRESS>/0. This has the benefit of
# overriding but not wiping out the original default gateway.
push "redirect-gateway def1"
client-to-client
keepalive 10 120
remote-cert-tls client
tls-version-min 1.2
tls-auth /etc/openvpn/easy-rsa/pki/ta.key 0
cipher AES-256-CBC
auth SHA256
comp-lzo
user nobody
group nogroup
persist-key
persist-tun
crl-verify /etc/openvpn/crl.pem
status /var/log/openvpn-status.log 20
status-version 3
syslog
verb 3
# Generated for use by PiVPN.io

sudo iptables-save:
# Generated by iptables-save v1.6.0 on Sat Feb 24 00:59:44 2018
*nat
:PREROUTING ACCEPT [0:0]
:INPUT ACCEPT [0:0]
:OUTPUT ACCEPT [0:0]
:POSTROUTING ACCEPT [0:0]
-A POSTROUTING -s <IP_ADDRESS>/24 -o wlan0 -j MASQUERADE
COMMIT
# Completed on Sat Feb 24 00:59:44 2018
# Generated by iptables-save v1.6.0 on Sat Feb 24 00:59:44 2018
*filter
:INPUT ACCEPT [20:2098]
:FORWARD ACCEPT [4:290]
:OUTPUT ACCEPT [16:2344]
COMMIT
# Completed on Sat Feb 24 00:59:44 2018

sudo ufw status verbose:
Status: inactive

resolv.conf:
# Generated by resolvconf
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

pivpn debug:
::: Generating Debug Output
:::					:::
::		PiVPN Debug		 ::
:::					:::
::	Latest Commit			 ::
:::					:::
commit 25aaf24c893175894d0a54d159a10fbb74b6b463
Merge: abdce6c c414172
Author: redfast00<EMAIL_ADDRESS>Date:   Wed Feb 21 17:36:09 2018 +0100

    Merge remote-tracking branch 'origin/test'
:::					:::
::	Recursive list of files in	 ::
::	/etc/openvpn/easy-rsa/pki	 ::
:::					:::
/etc/openvpn/easy-rsa/pki/:
ca.crt
crl.pem
Default.txt
dh2048.pem
index.txt
index.txt.attr
index.txt.attr.old
index.txt.old
issued
android.ovpn
private
serial
serial.old
ta.key

/etc/openvpn/easy-rsa/pki/issued:
android.crt
server_5SR646nOZ52B1bt2.crt

/etc/openvpn/easy-rsa/pki/private:
ca.key
android.key
server_5SR646nOZ52B1bt2.key
:::					:::
::	Output of /etc/pivpn/*		 ::
:::					:::
:: START /etc/pivpn/DET_PLATFORM ::
Raspbian
:: END /etc/pivpn/DET_PLATFORM ::
:: START /etc/pivpn/INSTALL_PORT ::
1129
:: END /etc/pivpn/INSTALL_PORT ::
:: START /etc/pivpn/INSTALL_PROTO ::
udp
:: END /etc/pivpn/INSTALL_PROTO ::
:: START /etc/pivpn/INSTALL_USER ::
pi
:: END /etc/pivpn/INSTALL_USER ::
:: START /etc/pivpn/NO_UFW ::
1
:: END /etc/pivpn/NO_UFW ::
:: START /etc/pivpn/pivpnINTERFACE ::
wlan0
:: END /etc/pivpn/pivpnINTERFACE ::
:::					:::
:: /etc/openvpn/easy-rsa/pki/Default.txt ::
:::					:::
client
dev tun
proto udp
remote ***** 1129
resolv-retry infinite
nobind
persist-key
persist-tun
key-direction 1
remote-cert-tls server
tls-version-min 1.2
verify-x509-name server_5SR646nOZ52B1bt2 name
cipher AES-256-CBC
auth SHA256
comp-lzo
verb 3
:::					:::
::	Debug Output Complete		 ::
:::					:::
:::
::: Debug output completed above.
::: Copy saved to /tmp/debug.txt
:::


Fix #12764 - Update troubleshooting-kubeadm doc

#12764
@neolit123

/assign @neolit123

@neolit123 Thank you for the review! I updated the PR with the suggestions

Checked the updates on: https://deploy-preview-12825--kubernetes-io-master-staging.netlify.com/docs/setup/independent/troubleshooting-kubeadm/
/approve


Made nearby notification also visible to right-to-left languages

Description (required)
Fixes #2334 Nearby notification empty in right-to-left languages
Changes made
Modified the nearby_card_view
Screenshots showing what changed (optional - for UI changes)

@maskaravivek @nicolas-raoul Can you please review the PR :)

@maskaravivek @misaochan @ashishkumar468 Can you please review the PR :)

Code changes look good to me so I am going ahead and merging it. Our alpha users can help us test this. :)


Bootstrap Tooltips fail when using v-repeat with AJAX data

I'm having trouble with tooltips for elements inside a v-repeat loop, whenever the loop is based on data received by an AJAX callback.
Here is a jsfiddle with the issue.
http://jsfiddle.net/hotrod/526e6qwv/
Any idea on what am I doing wrong here?
Thank you.

http://jsfiddle.net/526e6qwv/1/
http://vuejs.org/guide/best-practices.html#Understanding_Async_Updates
https://github.com/vuejs/Discussion
https://gitter.im/yyx990803/vue

Thank you @azamat-sharapov


[Observation] Optimize the storage of registrar entries, provide KeyPath caching, and distinctness notification

This alters the macro code-generation for Observable types. The types now house a distinctness check by testing shouldNotifyObservers which by default emits only on new values for things that conform to equatable.
Furthermore it removes the bottleneck of creating key paths by pre-caching them as cachedKeypath prefixed static members. This avoids the cache miss potential of KeyPaths and also avoids the cache lookup/lock acquisition (which dominate most of the traces of high frequency accessed properties for observable types).
Additionally some table bookkeeping was removed so that the registrar itself is now faster.

@swift-ci please smoke test


[Bug] Cover image cut off if there are many genres

Whats the bug?

Product info:
Spicetify version : 2.20.0
Spotify version : <IP_ADDRESS>6.geb595a67
Your config-xpui.ini
[Setting]
color_scheme            = Base
overwrite_assets        = 1
spotify_launch_flags    =
prefs_path              = /home/nico/.config/spotify/prefs
current_theme           = marketplace
inject_css              = 1
replace_colors          = 1
check_spicetify_upgrade = 0
; spotify_path            = /opt/spotify
spotify_path            = /home/nico/.local/share/spotify-launcher/install/usr/share/spotify
inject_theme_js         = 1

[Preprocesses]
disable_sentry        = 1
disable_ui_logging    = 1
remove_rtl_rule       = 1
expose_apis           = 1
disable_upgrade_check = 1

[AdditionalOptions]
extensions            = hidePodcasts.js|playlist-icons.js|playlistIntersection.js|fullAppDisplayMod.js|spotifyGenres.js|QueueTime.js|CoverAmbience.js|fullAlbumDate.js
custom_apps           = lyrics-plus|marketplace
sidebar_config        = 1
home_config           = 1
experimental_features = 1

[Patch]

; DO NOT CHANGE!
[Backup]
version = <IP_ADDRESS>6.geb595a67
with    = Dev

OS
Linux
I checked all issues and mine is different

[X] Yes, I checked all issues

Its a Spotify issue it occurs even if Spotify genres extension doesnt exists. Like big list of artist names also cause this issue.


remove unused add_secrets func

#1215  👈
main

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

Join @jlewitt1 and the rest of your teammates on  Graphite


Default 'Hello Mobile' deploy crash

When deploying the 'Hello Mobile' default application:
Option A: Live deploy with CADT (quick and easy!)
Exits and displays message "Unfortunately, Chrome App Developer Tool has stopped."
Option B: Use cca to package your application and deploy it to your mobile device
Application opens, but then exits after a few seconds.
Android version 4.2.2
Device: EE Eagle / Huawei S8-301L

Also happens with my second device:
EE Kestrel / Huawei Ascend G6
Any ideas how I could solve this?

Given that it happens in both contexts, I'd guess the issue is with Crosswalk. We're working on updating the version of crosswalk for the next CCA release, but if you'd like to try it now:
https://github.com/MobileChromeApps/mobile-chrome-apps/blob/master/dev-docs/cloning-this-repo.md