Support for Keras/TF_2.0?

Hello,
I am working in Python and using TensorFlow's Keras API ( e.g. tf.keras.models.Sequential). Would I be able to export it (export_savedmodel) and then load it into Go with TFGO? Or do I need to stick to "pure" TensorFlow expressions?
Also, if I use TensorFlow 2.0, will it work correctly with TFGO?
Thanks!

Yes, it will work without any problem.
Just use the saved_model_cli tool to inspect the saved model and find the
correct tag (usually "serve") and input/output node names and you're ready
to go.
On Sun, Jul 14, 2019, 01:10 EJ White<EMAIL_ADDRESS>wrote:

Hello,
I am working in Python and using TensorFlow's Keras API ( e.g.
tf.keras.models.Sequential). Would I be able to export it
(export_savedmodel) and then load it into Go with TFGO? Or do I need to
stick to "pure" TensorFlow expressions?
Also, if I use TensorFlow 2.0, will it work correctly with TFGO?
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/galeone/tfgo/issues/20?email_source=notifications&email_token=ACAJSDG6B6LDW6GOKAQS72TP7JOGFA5CNFSM4IDD3ZSKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7BV5TA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACAJSDAZMKBDRBW7LQELUH3P7JOGFANCNFSM4IDD3ZSA
.

Thanks, appreciate the quick response... will try it out and let you know if there are any issues!

@ej-white have you been able to use TF2 + tfgo?

In my case, I just saved my models with the normal save function on Keras. Any suggestions on how to export it to run in go?

@oldani right now you can only load a SavedModel using Go.
The file format created by Keras using the .save method is not language-agnostic but is tight to Python. That method is only useful if you have a Python interpreter and you can redefine the very same Keras model structure first, and then restore the model parameters.
Instead, using SavedModel you save both the model structure (as a graph, language-agnostic) and the model parameters and there is no need the re-define the Keras model (thus, no need to use Python).
Thus, the suggestion is to use Python to restore the model parameters you saved with .save and export it as a SavedModel

So by use Python to restore the model parameters you mean to train it again? Isn't any way to take a hd5 saved model and create the SavedModel?

No, I don't meain to retrain it.
You just have to:

Define the keras model
Restore the weights from the hd5 -> now you have a Keras model ready to use.
Use tf.saved_model.save to save your trained model as a SavedModel

Now you have a SavedModel ready to use (in Go, or in any other language able to load the SavedModel file format)

Thanks for the clarification, it will be good to have this in the README as
well.
On Mon, Aug 12, 2019 at 9:01 AM Paolo Galeone<EMAIL_ADDRESS>wrote:

No, I don't meain to retrain it.
You just have to:

Define the keras model
Restore the weights from the hd5 -> now you have a Keras model
ready to use.
Use tf.saved_model.save
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/saved_model/save
to save your trained model as a SavedModel

Now you have a SavedModel ready to use (in Go, or in any other language
able to load the SavedModel file format)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/galeone/tfgo/issues/20?email_source=notifications&email_token=AEEAUIPQXETZE6XP4UZO4VTQEFNMHA5CNFSM4IDD3ZSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4COSAQ#issuecomment-520415490,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEEAUIK344EJRWT3RSYAKBLQEFNMHANCNFSM4IDD3ZSA
.

--
Ordanis Sanchez
Python Backend Developer
<EMAIL_ADDRESS>Cell:<PHONE_NUMBER>

I'm on holiday right now, but if you want to open a pull request with this
small change to the README I'll appreciate it (and I'll merge it the next
week)
Il giorno mar 13 ago 2019 alle ore 13:40 Ordanis Sanchez <
<EMAIL_ADDRESS>ha scritto:

Thanks for the clarification, it will be good to have this in the README as
well.
On Mon, Aug 12, 2019 at 9:01 AM Paolo Galeone<EMAIL_ADDRESS>wrote:

No, I don't meain to retrain it.
You just have to:

Define the keras model
Restore the weights from the hd5 -> now you have a Keras model
ready to use.
Use tf.saved_model.save
<
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/saved_model/save

to save your trained model as a SavedModel
Now you have a SavedModel ready to use (in Go, or in any other language
able to load the SavedModel file format)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/galeone/tfgo/issues/20?email_source=notifications&email_token=AEEAUIPQXETZE6XP4UZO4VTQEFNMHA5CNFSM4IDD3ZSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4COSAQ#issuecomment-520415490
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AEEAUIK344EJRWT3RSYAKBLQEFNMHANCNFSM4IDD3ZSA
.

--
Ordanis Sanchez
Python Backend Developer
<EMAIL_ADDRESS>Cell:<PHONE_NUMBER>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/galeone/tfgo/issues/20?email_source=notifications&email_token=ACAJSDFW6AKWNC6BTVTWA5TQEKMSBA5CNFSM4IDD3ZSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4FMPHA#issuecomment-520800156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACAJSDDGZ3FTPB7VVTLLTO3QEKMSBANCNFSM4IDD3ZSA
.

Yes, it will work without any problem. Just use the saved_model_cli tool to inspect the saved model and find the correct tag (usually "serve") and input/output node names and you're ready to go.
…
On Sun, Jul 14, 2019, 01:10 EJ White @.***> wrote: Hello, I am working in Python and using TensorFlow's Keras API ( e.g. tf.keras.models.Sequential). Would I be able to export it (export_savedmodel) and then load it into Go with TFGO? Or do I need to stick to "pure" TensorFlow expressions? Also, if I use TensorFlow 2.0, will it work correctly with TFGO? Thanks! — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#20?email_source=notifications&email_token=ACAJSDG6B6LDW6GOKAQS72TP7JOGFA5CNFSM4IDD3ZSKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7BV5TA>, or mute the thread https://github.com/notifications/unsubscribe-auth/ACAJSDAZMKBDRBW7LQELUH3P7JOGFANCNFSM4IDD3ZSA .

hello, @galeone, thanks for your sharing.
I trained my model with TF 1.13 estimator. I can get my model description with saved_model_cli as below(Appendix I).
As you mentioned, with tfgo, I can access the output with name, like 'Cast', 'Mul_1'. So my question is may I access the output with the keys, like 'class_ids', 'probabilities'? The keys are defined as Appendix II in estimator style TF code.
Appendix I
outputs['class_ids'] tensor_info:
dtype: DT_INT32
shape: (-1)
name: Cast:0
outputs['probabilities'] tensor_info:
dtype: DT_FLOAT
shape: (-1)
name: Mul_1:0
outputs['source_logits'] tensor_info:
dtype: DT_FLOAT
shape: (-1, 8)
name: source/l2_normalize_1:0
outputs['target_logits'] tensor_info:
dtype: DT_FLOAT
shape: (-1, 8)
name: target/l2_normalize_1:0
Appendix II
if mode == tf.estimator.ModeKeys.PREDICT:
predictions = {
'class_ids': predict,
'probabilities': probabilites,
'logits': logit,
}

hello, @galeone, thanks for your sharing.
I trained my model with TF 1.13 estimator. I can get my model description with saved_model_cli as below(Appendix I).
As you mentioned, with tfgo, I can access the output with name, like 'Cast', 'Mul_1'. So my question is may I access the output with the keys, like 'class_ids', 'probabilities'? The keys are defined as Appendix II in estimator style TF code.
-------
Appendix I
outputs['class_ids'] tensor_info:
dtype: DT_INT32
shape: (-1)
name: Cast:0
outputs['probabilities'] tensor_info:
dtype: DT_FLOAT
shape: (-1)
name: Mul_1:0
outputs['source_logits'] tensor_info:
dtype: DT_FLOAT
shape: (-1, 8)
name: source/l2_normalize_1:0
outputs['target_logits'] tensor_info:
dtype: DT_FLOAT
shape: (-1, 8)
name: target/l2_normalize_1:0
-------
Appendix II
if mode == tf.estimator.ModeKeys.PREDICT:
predictions = {
'class_ids': predict,
'probabilities': probabilites,
'logits': logit,
}
-------

Yes, it will work without any problem. Just use the saved_model_cli tool to inspect the saved model and find the correct tag (usually "serve") and input/output node names and you're ready to go.
…
On Sun, Jul 14, 2019, 01:10 EJ White @.***> wrote: Hello, I am working in Python and using TensorFlow's Keras API ( e.g. tf.keras.models.Sequential). Would I be able to export it (export_savedmodel) and then load it into Go with TFGO? Or do I need to stick to "pure" TensorFlow expressions? Also, if I use TensorFlow 2.0, will it work correctly with TFGO? Thanks! — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#20?email_source=notifications&email_token=ACAJSDG6B6LDW6GOKAQS72TP7JOGFA5CNFSM4IDD3ZSKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7BV5TA>, or mute the thread https://github.com/notifications/unsubscribe-auth/ACAJSDAZMKBDRBW7LQELUH3P7JOGFANCNFSM4IDD3ZSA .

Hi @heydatatalks ,
Looking at the expor output in Appending I, all I can say is that, yes, you can access the output nodes, but not looking for the "keras names", but for the graph names.
For instance, to get the value of "Class ids" you have to get the output of the node " Cast:0".
I don't think is there any other way to export the graph using the tf.estimator API and giving the output nodes different names; but you can just create a mapping using go, in order to get the output of the node  Cast:0 when you are looking for "class_ids" (If I understood correctly you r question).
Let me know if it helps

Thanks @galeone for your kindly reply.
It helps a lot. It should work to create a custom mapping for the output :)


[guide] [english] Fill in stubs for helmet lessons

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

here's links to the other four lessons not included in this PR
https://github.com/freeCodeCamp/freeCodeCamp/pull/34100
https://github.com/freeCodeCamp/freeCodeCamp/pull/34102
https://github.com/freeCodeCamp/freeCodeCamp/pull/34108
https://github.com/freeCodeCamp/freeCodeCamp/pull/34109


Clipping in MP3s made by youtube-dl

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.03.10. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.03.10

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[x] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.03.10
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
When downloading music from YouTube the MP3s are too loud. It introduces clipping in the audio. This is shown in Audacity when the "Show Clipping" option is on under View in the menu.
Clipping is when the waveforms go over level and it is distortion. It might be coming from volume normalization, which is trying to maximize the audio levels for every MP3, OGG, etc. that's made during conversion. The problem is likely in ffmpeg or whatever is used to compress and can be changed simply to avoid this issue.
I don't know whether to call it a "bug" or just mark this as "other," but I've already had a previous attempt at submitting here taken down. This kind of issue really shouldn't exist because it is needlessly worsening the audio quality of so many things downloaded and saved.
Thank you for reading and please considering improving the software here.

Neither youtube-dl not ffmpeg does any clipping or normalization. All media files are provided as is.

You're right. Sorry about this.

From: Sergey M<EMAIL_ADDRESS>Sent: Tuesday, March 13, 2018 3:12 PM
To: rg3/youtube-dl
Cc: stalepie; Author
Subject: Re: [rg3/youtube-dl] Clipping in MP3s made by youtube-dl (#15857)
Neither youtube-dl not ffmpeg does any clipping or normalization. All media files are provided as is.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/rg3/youtube-dl/issues/15857#issuecomment-372784542, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AjH0wi0Pyvb9Ia4AIPA1fP1QaDrXbJYbks5teBn_gaJpZM4SpQ7b.


Unable to create apps from a different git provider than the destination.

Auume GIT_REPO is Azure DevOps (and GIT_TOKEN is for Azure DevOps oviously) and if the user tries to create app from GitHub, the credential for Azure DevOps is used to clone the repo frmo GitHub, resuling an error: "authentication required".
Reproduce
export<EMAIL_ADDRESS>export GIT_TOKEN=accesstokenforazuredevops
argocd-autopilot --log-level debug  \
	app create demoapp \
	--app github.com/argoproj-labs/argocd-autopilot/examples/demo-app/ \
	-p testing
> ...
> INFO[2024-09-19T05:19:40Z] cloning repo: 'https://github.com/argoproj-labs/argocd-autopilot.git', to infer app type from path 'examples/demo-app/'
> DEBU[2024-09-19T05:19:40Z] cloning git repo                              url="https://github.com/argoproj-labs/argocd-autopilot.git"
> FATA[2024-09-19T05:19:40Z] authentication required

How about to retrieve the credentials from git credential store rather than env?


Each board should have a list of Kconfig options supported

While working on an for me unknown board, let's say for example FRDM k64f, it would be excellent to have all the possible board specific Kconfig options listed. I was looking for UART support, and figured out from the header file that the asynchronous UART API would be the best fit for my project.
Unfortunately the only way of figuring out whether said example board supports this particular UART API is to dig through several orders of Kconfig files in order to learn whether the #ifdef in the include file is supported or not or whether to redesign ones project to use the more basic API.
In order to solve this, I propose a script crawling through the board specific Kconfig options and other included Kconfig files that would output decently formatted documentation added for each board. That way the connection between the APIs documented in include/ can be mapped to board specific support - or actually vice versa.

CC @ulfalizer

This is already in menuconfig or guiconfig. A list would be extremely long and not hierarchical, so I am not sure this would be usable. Closing for now, if you have a concrete proposal please reopen it.


AMQP-711: Make Transaction Rollback Consistent

JIRA: https://jira.spring.io/browse/AMQP-711
Previously, messsages were unconditionally requeued when an external
transaction manager is used; this was not consistent with local
transactions, where the normal requeue logic was honored.
backport to 1.7.x will require rework due to the container changes - I will issue another PR after this is reviewed/merged

This needs more work - it doesn't work if the TxMgr is a RabbitTransactionManager.

cherry-picked...

That's why the PR says:

backport to 1.7.x will require rework due to the container changes - I will issue another PR after this is reviewed/merged

😄

Merged as https://github.com/spring-projects/spring-amqp/commit/5d500b3685c8ea15623ff8626a4a4616299dc150


Create Generic Node Structure

The plan for the assembler is to let each label, keyword, and address be its own node to allow for easier manipulation of the data.  There will be a basic node that allows a visitor to be accepted.  From the basic node, there will be a named node that contains a name.  From there, we can create a label node and a keyword node. From the basic node, we can have argument nodes that can contain 1 or 2 arguments (addresses or registers).  By creating a node for each keyword, we can easily define and store relevant information like operation codes and reduce the number of if statements that are used to control the flow of the program.

An operator factory can be used to generate new instances of operators and ensure that we keep data organized.

Ok we are scrapping that old design.  New design will include making a HexNum class to help handle hexadecimal values.  While there are existing Java methods to handle it, it will make it much easier to have one wrapper class around it to handle the conversion and other needed operations.
We will create a statement factory that will contain the OPTAB and return Statements.  There will be a base Statement that handles lines in format 1.  RegisterStatement will handle statements interacting with registers and be in format 2.  ExtendedStatement will handle formats 3 and 4.  A majority of the logic will be in Statement and overriden in the other methods as needed.


Undo / redo via keyboard shortcut on macOS not working as expected in V4.0-alpha15

Godot version
v4.0.alpha15.official [432b25d36]
System information
MacOS 12.5.1
Issue description
The keyboard shortcuts CMD + Z and CMD + SHIFT + Z in  V4.0-alpha15 do not work as expected inside the GDScript editor.
Instead of triggering the respective commands in the GDEditor it seems that the corresponding Entries in the Scene Menu are triggered. Using the keyboard shortcuts results in the message 'Nothing to undo.' in the output tab.
Using the menu entries in the GDScript editor, i.e. Edit > Undo / Redo works as expected.
One and the same shortcuts seem to be assigned for different operations, cf. Scene Menu and Edit Menu inside GDScript editor.
Steps to reproduce

Open the GDScript editor
Edit some code
use the keyboard shortcuts

Minimal reproduction project
No MRP necessary to reproduce the bug.

Duplicate of https://github.com/godotengine/godot/issues/65119.


New version: CologneCodeCompany.XYplorerPortable version 26.20.0100

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Not sure I did here, removing, will re-do.


[BUG] Platform tests missing platform:security and clusterapi_enabled

Describe the bug
The platform test suite does not currently include tests for platform:security or check if clusterapi_enabled, despite these being part of the expected suite of platform tests.
To Reproduce

Execute the platform tests by running ./cnf-testsuite platform.
Observe that the tests for platform:security and clusterapi_enabled are not executed, as they are not listed in the task array:

task "platform", ["helm_local_install", "k8s_conformance", "platform:observability", "platform:resilience", "platform:hardware_and_scheduling"]
Expected behavior
It is likely that  platform:security and clusterapi_enabled should be included.
How will this be tested? aka Acceptance Criteria (optional)

Running the platform test suite.
Spec tests.

Additional context
From the USAGE.md documentation it seems that these are grouped with other platform tests and should be executed. There might be some historical or technical reason for exclusion of these, but it was not readily apparent, in such case don't hesitate to close the ticket. In case this bug is relevant, it might be good to think about separation of k8s_conformance and clusterapi_enabled tasks into their own files for better code separation and readability as part of this change.

Thank you @svteb for reporting this issue.
Below are some notes to move forward
I just verified the code. platform:security is indeed missing as mentioned in the issue. It should be a dependency of the platform task. This is the only change required for this ticket.
We can ignore clusterapi_enabled. We should not be including that when platform tests are run. That is marked as a Proof of Concept test (there is a check_poc function being called in the test code).
To keep PRs focused on the issue, please create separate PRs (or tickets if required) for any other code cleanup.

Completed via PR. Closing issue.


Bug: Having a table named "Measures" as the first table in a DirectQuery connection

It appears "Measures" is a special name in Power BI.
It causes an issue which we can duplicate using either our connector + ODBC driver or the sample SqlODBC Data Connector + SQL Server.
To duplicate:

Create a SQL Server database with 2 tables - "Measures" and "Test". Put a column or two in each table.
Using the compiled sample SQL ODBC connector (https://github.com/Microsoft/DataConnectors/tree/master/samples/ODBC/SqlODBC), connect to the database using DirectQuery.
Select "Measures" as the first (or only) table, then click Load data.  You will see an error - "Couldn't load model schema" and an underlying error about not being able to find a table guid.

If you instead click tables in the order of "Test" table first, then "Measures" it works fine.
If you instead use import mode, it works fine.
Note that Power BI is renaming "Measures" to "Measures 1".  Something in the Data Connector logic when in DirectQuery mode is missing that rename step it seems.  It shows up on the relationship view, but not the Report view / field pane.
Full background and details submitted here:
https://community.powerbi.com/t5/Issues/Couldn-t-load-the-model-schema-Sept-amp-Oct-releases/idc-p/558640

The table is renamed due to a restriction in the AS engine, and the bug you describe has apparently been fixed for the November release.


Organisation starring is not showing properly

When you view stargazers for and org, you should see them on a separate page.

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Issue Number: DOCK-777

➤ Charles Overbeck commented:
May be fixed already?

Already fixed on develop

Working on staging.dockstore.org as expected


wip: support advanced daemonset

Ⅰ. Describe what this PR does
Support  Workload of Advanced Daemonset  for Rollout

apps.kruise.io/v1alpha1/daemonset

Ⅱ. Does this pull request fix one issue?

#69 This PR will implemente Kruise Daemonset but not native Daemonset
Ⅲ. Special notes for reviews

/close


Block delete of upload and download configs if in use

Upload and download configs can be deleted if they're currently in use by validations, which causes all sorts of problems.  Block deletion of these if they're in use by any current (latest version, not deleted) validations.

Fix verified


proof request there's no information on holder

Is this a regression?
NO
Description
Right know when we have a proof request there's no information on holder side of what's being requested.
Holder webhook data:
PresentationStatusAdapter(presentationId=94acbe0f-ed87-4477-9757-4b4ba8c3461d, thid=d080d2b5-0498-42aa-a829-22cf021ff3cf, role=PROVER, status=REQUEST_RECEIVED, metaRetries=5, proofs=[], data=[], connectionId=null)
GET /present-proof/presentations/$presentationId
{
"presentationId": "94acbe0f-ed87-4477-9757-4b4ba8c3461d",
"thid": "d080d2b5-0498-42aa-a829-22cf021ff3cf",
"role": "Prover",
"status": "RequestReceived",
"proofs": [
],
"data": [

],
"requestData": [
    "{\n  \"options\" : {\n    \"domain\" : \"https://example-verifier.com\",\n    \"challenge\" : \"11c91493-01b3-4c4d-ac36-b336bab5bddf\"\n  },\n  \"presentation_definition\" : {\n    \"format\" : null,\n    \"name\" : null,\n    \"purpose\" : null,\n    \"id\" : \"345b056f-2889-458a-9f59-25d1ab249557\",\n    \"input_descriptors\" : [\n    ]\n  }\n}"
],
"metaRetries": 5

}
The data is available in the RequestPresentation Attachment but we don't expose it on the endpoint
This will be helpful fro cloud agnet  while testing test
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in
No response
Anything else?
No response

https://github.com/hyperledger/identus-cloud-agent/pull/1453


Logging

I booted up a new galaxy and found the UX of getting IEs unpleasant, so...

enhanced error message if you're running on localhost, now it's obvious that you've bound galaxy to <IP_ADDRESS> and it isn't going to answer docker questions.

Enhanced dev experience:

exporting DEBUG=True (or `true) now sets the logging level in galaxy.py
logging includes many useful things like what URLs are being contacted, etc.

Other modifications:

completely deprecated the conf stuff, replacing all with os.environ queries.
history_id as passed on the cli is used in get_galaxy_connection, if applicable. I imagine a None value there might've been causing issues.

Sorry for duplicate commits, rebased.

looks good now, thanks!


Minor Generic and Guard changes

Remove force cast to Int adding Generic constraint for CollectionType
Move guard conditions in editDistance at the top of the function

I admit that my guard changes look a bit ugly. Reasons why I proposed a change are:

early exit for trivial cases returning all insertion or all deletion
early exit will avoid allocation of the d matrix (a marginal memory optimisation)
usually I prefer guard statements at the top of a function. It makes the code a bit easier to read. Anyway I admit this is not always possible or the best solution.

Maybe I'll try another solution for moving up the guards. Please, if I do, don't consider it as me trying to force that change ... just reasoning about it and open to your opinions.

While I’d definitely like to exit earlier, the value of the only slightly earlier exist was not justified by the duplicated code and marginal memory optimisation, in my opinion.
The for loops  come quite early – after the d allocation, yes, but it only allocates the amount of memory needed anyway. Just like your code.
A much more relevant optimisation, I think, would be to make the Changeset.edits property lazy. The user may have a bunch of Changeset values in the air, and may only be querying the edits on some of those. It would be a waste to have computed the edit distances in those unused cases. But I haven’t found a way to do that in Swift 2.

:+1: Thanks


Updating to the latest TypeSpec and preparing for Document Translation SDK GA release.

Updating to the latest TypeSpec and preparing for Document Translation SDK GA release.

API change check
API changes are not detected in this pull request.

https://spa.apiview.dev/review/a618f04da4b64d70a6080febb1085a74?activeApiRevisionId=0587541be5894d20b832636e7b20e613


Specific enum value not comparable with number in 5.0

Bug Report

🔎 Search Terms

operator cannot be applied enum number comparison
🕗 Version & Regression Information

This changed between versions 4.9 and 5.0

⏯ Playground Link

Playground Link
💻 Code

const enum Constants {
    DefaultName = "foo",
    DefaultTimeout = 1000,
    MaxItems = 25,
}

declare var someValue: number

if (someValue > Constants.MaxItems) {
    someValue = Constants.MaxItems;
}

🙁 Actual behavior

Operator '>' cannot be applied to types 'number' and 'Constants'.(2365)

Removing the string-ish value from Constants stops the error.
🙂 Expected behavior
No error; even though Constants shouldn't be comparable thanks to #52048, this seems like it should work because it's only referencing a specific value.
/cc @Andarist @ahejlsberg

@jdom I filed this based on your TS discussion email.

fwiw the example given feels like a misuse of enums.  If anything that should be a namespace instead.

Generally the purpose of enums is that you have a set of symbolic constants whose exact value is unimportant (except maybe for debugging); all that’s important is that Enum.A is a different value from Enum.B

The fact that this enum has mixed constant (?) values (string | number~) makes me believe that this code is misusing the relation operator. With my change this code errors as well (and that was "caught" and rechecked at the PR stage):
declare const val: string | number
declare var someValue: number

if (someValue > val) { // errors
    someValue = 100;
}

I generally agree that these "constant bags" are probably a misuse of enums, though I suspect in this case it's being used for the inlining provided by const-ness.

makes me believe that this code is misusing the relation operator

I would disagree; this code example references a specific enum member which is a number and is interchangable with the literal 1000. It seems to me like this is a bad interaction of #52048 and #50528 where the freshness of this enum reference is being lost in the comparison check.

Sure, but even with inlining - we end up comparing number and number | string. So the question is: is this something that TS considers to be safe enough to allow devs to rely on coercion here?

I would disagree; this code example references a specific enum member which is a number and is interchangable with the literal 1000. It seems to me like this is a bad interaction of https://github.com/microsoft/TypeScript/pull/52048 and https://github.com/microsoft/TypeScript/pull/50528 where the freshness of this enum reference is being lost in the comparison check.

Oh, I see what you are saying here - ye. I guess that perhaps this might be a bug worth fixing then. I would love to have some extra confirmation before I jump into fixing this though.

I can provide details about the reason for doing this, and is like @jakebailey mentioned: it's because it allows me to define some constants on the top of the file that I just reuse in my code, and will automatically be inlined in the generated javascript. Perhaps there's a better way to have such inlining without using const enum, but I have yet to find it (open to alternatives).

In this case, I think this might just be something we didn't have a test for, so wasn't caught. The error goes away if we don't getBaseTypeOfLiteralType the two sides before comparing them, as that function takes enum literals and converts them to their enum's base type, rather than considering the literal itself. It seems like it might be worthwhile in this case to have a variant which instead considers that specific literal's type.
But generally speaking, having a grab bag of random unrelated constants in an enum is not a super good idea; if you split them apart into smaller, related pieces, you would avoid this.
(I have further const enum inlining opinions but they're sorta moot and hypocritical given TS itself makes heavy use of them exactly for the perf of inlining...)

The error goes away if we don't getBaseTypeOfLiteralType the two sides before comparing them

More correctly would be to have a variant of the above which checks EnumLiteral after all of the other literals.

I've sent #52703 for discussion.

Per design meeting, this is a bug and we should allow this code. Just need to get the above PR reviewed and fixed up.


rich_code_editor: ^1.0.0 is throwing an error during compile - TextEditingController.isSelectionWithinTextBounds

Steps to Reproduce

rich_code_editor:^1.0.0 (pubspec.yaml)
run flutter build web

Expected results:
I expect a clean compile
Actual results:

Logs

Target dart2js failed: Exception: ../../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/rich_code_editor-1.0.0/lib/rich_code_controller.dart:6:7:
Error: The non-abstract class 'RichCodeEditingController' is missing implementations for these members:
 - TextEditingController.isSelectionWithinTextBounds
class RichCodeEditingController extends ValueNotifier<TextEditingValue> implements TextEditingController {
      ^^^^^^^^^^^^^^^^^^^^^^^^^
../../../../development/flutter/packages/flutter/lib/src/widgets/editable_text.dart:250:8:
Info: 'TextEditingController.isSelectionWithinTextBounds' is defined here.
  bool isSelectionWithinTextBounds(TextSelection selection) {
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^
../../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/rich_code_editor-1.0.0/lib/rich_editable_code.dart:1617:34:
Error: Too few positional arguments: 2 required, 1 given.
      ? () => controls.handleCopy(this)
                                 ^
../../../../development/flutter/.pub-cache/hosted/pub.dartlang.org/rich_code_editor-1.0.0/lib/rich_editable_code.dart:1705:29:
Error: Getter not found: 'obscuringCharacter'.
      text = RenderEditable.obscuringCharacter * text.length;
                            ^^^^^^^^^^^^^^^^^^
Error: Compilation failed.

Admins-MacBook-Pro:tutorial3_column dnishimoto$ flutter doctor -v
[✓] Flutter (Channel master, 1.20.0-8.0.pre.41, on Mac OS X 10.15.5 19F101, locale en-US)
    • Flutter version 1.20.0-8.0.pre.41 at /Users/dnishimoto/development/flutter
    • Framework revision fd80503fd3 (8 hours ago), 2020-07-10 14:41:02 +0530
    • Engine revision 9b3e3410f0
    • Dart version 2.9.0 (build 2.9.0-20.0.dev 06cb010247)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/dnishimoto/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
    • Xcode at /Applications/Xcode-beta.app/Contents/Developer
    • Xcode 11.4, Build version 11N111s
    • CocoaPods version 1.9.1

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

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 43.0.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

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

[✓] Connected device (2 available)
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 83.0.4103.116

• No issues found!

It looks like you want to file an issue with the author of that package, since it's implementing the TextEditingController's API, which has changed.

Configuration control needs to be improved by flutter.  It is not my job to notify
Sent from my iPhone

On Jul 10, 2020, at 5:26 PM, John Ryan<EMAIL_ADDRESS>wrote:

It looks like you want to file an issue with the author of that package, since it's implementing the TextEditingController's API, which has changed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


refactor: target logger and create command output update

Target logger and info command update
Description
This PR syncs output when running daytona create with what it was like before.
It also fixes target ID showing as a prefix in logs instead of target name.

[ ] This change requires a documentation update
[ ] I have made corresponding changes to the documentation

Screenshots
Note that even though output on daytona create looks like this:

Output when running daytona info looks like this which needs refactor as well.

Notes
All of the providers need to be updated with expanding arguments of CreateTargetLogger function with targetName string argument.

There's something wrong with the prefix len calculation. Look into that please.
logs_view.CalculateLongestPrefixLength(workspaceNames)

This should be ...(workspaceNames + targetName)


Fix PermissionDenied error with Windows folders

Currently, a PermissionDenied error is raised when using set_file_times, set_file_mtime, set_file_atime or set_symlink_file_times and passing a path to a folder on Windows.
Passing the FILE_FLAG_BACKUP_SEMANTICS flag when opening the file handle fixes the issue.
This PR also adds tests for setting directory times.

Thanks!

Thank you for merging this quickly. Do you have plans for a patch release that would include this fix? I think that dbrgn/tealdeer#115 depends on this

Sure thing, published now!


fixed build: removed QuickCheck dep and version bumps

Hi,
it builds at least on ghc 8.0.2 with these fixes, should also work on ghc 7.10.3.

Thanks! That works.


feat: add agenda page for display the user events

This feature add a new agenda page for displaying the user events.

Add agenda page route and html
Add temporary agenda button to the header in base.html (until future changes)
Add start date and end date to the event model
Add tests
Add basic css (should be upgrated later according on the web design)

Please change the merge request to merge into PythonFreeCourse:develop

Please change the merge request to merge into PythonFreeCourse:develop


[Question] How to use it on phoenix

Hi,
I'm look for documentation or sample on how to use it with phoenix. Sorry if it's a dummy question, but I'm new to elixir and phoenix.  But I need to do some thing like this ruby code.
  def print
    respond_to do |format|
        format.html do
          render file<EMAIL_ADDRESS>        end
        format.pdf do
          render  :pdf => "#{@proposal["title"]} - #{@proposal["organization"]["name"]}",
                  :template =><EMAIL_ADDRESS>        end
      end
  end

So I need to give the name and render the choose template with all variables in PDF.

I found how.
html = "
Hi there!
"
    { :ok, filename }    = PdfGenerator.generate html, page_size: "A5"

    conn
      |> put_resp_content_type("application/octet-stream", nil)
      |> put_resp_header("content-disposition", ~s[attachment; filename="hello.pdf"])
      |> send_file(200, filename)

@lypborges Very good you found out that. Still, this module is intended to handle HTML strings to PDF binary conversion only.
I do use in a Phoenix project exposing an HTTP API to it and apparently use a very similar code here: https://github.com/gutschilla/elixir-pdf-server/blob/master/web/controllers/generator_controller.ex

Tks a lot. ;)


Add unittests to check pages wheather user is logged in

How to write and run tests is described in README.rst: https://github.com/vault19/django-courses#testing
Write unit test that will check all the views in views.py. For pages that doesnt require login just check that they return correct response code, for both logged in user and also not logged in user.
Make sure that views that are decorated with @login_required will not allow user that is not logged in and allow only logged in users.

Thanks @BodaleDenis I have assigned the story to you, since you were first. I have updated the contributing file with tests section: https://github.com/vault19/django-courses/blob/main/contributing.rst#testing-setup it is short but if you have question or comments feel free to discuss here or add update to the file as a PR.

@biswain2309 thanks for your interest, if you are still interested I have create another issue regards testing: #33 that is suitable for beginners. I will keep adding more issues, if you are interested. Help is always welcome.


Header images not resolving

@Stranger-Ok has reported that header images are not being resolved correctly. It seems the image resolving code is running before the header and footers are being injected into the layout.

Fixed by #20!


Fix forums link in java8 prompt

This fixes the forum link in the java8 prompt. Closes #44

(for the record, I did this PR from mobile)


Return statements missing from several functions in API class

It looks like the "return" keyword was removed from various functions, including "getProcessInfo" and "getAllProcessInfo".  As a result the functions are being called successfully but not returning their results.

Resolved. Thanks for pointing that out!


Compression artifacts after each generation

Each time you inpaint the quality of the whole image drops
this is an example, before and after inpainting 4 times (to make the effect clear)
edit: i was only masking a small area on the right where the frame changed

Hey @LEBMD. I'm guessing this may be an issue with the Stable Diffusion model itself, as this inpainter app is sending along the unchanged URL of the previous prediction as the init image for the next prediction. You might want to open an issue at https://github.com/replicate/cog-stable-diffusion or https://huggingface.co/CompVis/stable-diffusion-v1-4/discussions


Client Allocation Request for: aoeu

Client Allocation Questions
Core Information

Name: aoeu
Website / Social Media: aeou, (aoeu, aoeu)
Region: Africa
DataCap Requested: 224 GiB
Addresses to be Notarized: f52642356247245673
Notary Requested: Deon Erda

Application created via filplus.storage

This user’s identity has been verified through filplus.storage


fix publish-ghcr.yml workflow version number

Description
Update the 3 actions to the latest version
- uses: docker/login-action@f054a8b539a109f9f41c372932f1ae047eff08c9
+ docker/login-action@v2

- uses: docker/metadata-action@98669ae865ea3cffbcbaa878cf57c20bbf1c6c38
+ docker/metadata-action@v4

- uses: docker/build-push-action@ad44023a93711e3deb337508980b4b5e9bcdc5dc
+ docker/build-push-action@v4

File path -https://github.com/Pradumnasaraf/Botchain/blob/main/.github/workflows/publish-ghcr.yml

I would like to work on this issue

I want to work on this

Yes go ahead

fix in #24


Update version in pypi

Hi,
I see a lot of changes have been made since the last release in 2021.
My app depends on the old version in pypi (A pypi package must have dependencies in pypi).
So I suggest to update version and upload it to pypi.
Thanks!

I'm not averse, but I have just run through my budget of attention for dealing with PyPI's security uplift (welcome though it is).  I bumped the version to 0.16, but you're going to have to wait until I get around to setting up GitHub CI for this project before you'll see it published to PyPI.  I've set up GitHub Actions before, and have templates to use, so it's not out of reach; it's just not happening right now.
I'm not sure it's really such a fantastic idea in the first place to depend on some release (i.e. out of date) version of ibflex.  This whole project is in the nature of an arms race with IB... their SWEs move fast & break the API without documenting anything; we respond by crowd-sourcing updates to the XML data model.  This is one of those projects where you definitely want to always run the bleeding edge... there's no changes to logic, only to data structures, and you want all those you can get as soon as possible.
  But I don't see much point in incrementing versions every time a user adds a field or a new model (which is what all those changes are that you are asking about).
Are we talking about this?  Is there significant value in sourcing this dependency via PyPI instead of getting it straight from the horse's mouth?  Not much more than proving that I'm squatting on the name ibflex in PyPI AFAICT, which I guess lends some credibility vs. a zillion potential GitHub forks whose URIs offer no superficial way of distinguishing source from branch.
Do you have to answer to people for whom this kind of thing would matter, so it's a good trade-off to use stale code in exchange for a green checkmark?
If not, may I suggest just changing your requirements.txt file to directly spec the most up-to-date ibflex models?
ibflex @ git+https://github.com/csingley/ibflex@master

Glad you're finding value in the package though... nice to see it used to build OSS apps; strength to your arm.

First, let me say that I appreciate your work , and I am laughing out of sympathy at the arms race. Thanks for the version bump.
Regarding your point, I understand it. I wonder how much it will matter myself. My point is that it seems more reliable and sound to have at least one solid version of the package in pypi. Even if some of it will be someout outdated, hopefully not completely broken within a month .
It seems that it boils down to some technical issue, which I don't fully understand.  I thought to do what you suggested btw (with requirements file), but I use pyproject.toml mainly and it refused to accept any requirement not in pypi. I can try again just with txt file.
For now it seems like I am going to try suggesting users to update and maybe provide a script for it. I do have the advantage of having a setup. I think there is no need to hurry here. If you need help I might be able to help a bit.

What about setup.cfg?  Does that take requirements direct from source, or only PyPI?
Not too bad to split some configs out from pyproject.toml to setup.cfg, although that does get you the gas face from fans of Poetry or whatever Armin is fiddling with in the Python packaging space, etc.

Really wish they’d leave an escape hatch in pyproject.toml - I get where they’re coming from, but some of us actually have plausible reasons for opting out of the straitjacket

They’re just gonna create incentives to stay old & busted if they make it too heavyweight to onboard to the new hotness.
I’ll try to find some time to set up CI here


[AJ-767] transaction support

bring in terra-common-lib (TCL) for its retryable-transaction annotations
annotate most APIs with a transaction, right in RecordController
annotate the streaming-write APIs at a lower level in BatchWriteService
remove @Transactional from any other runtime code in deference to the TCL transactions

In a future PR I'd like to make the transactions less broad - for instance, don't include the http response itself inside the transaction - but this will require a fair bit of refactoring of RecordController. So for now, the transactions are wide and long

@calypsomatic fyi I added info to the existing ticket https://broadworkbench.atlassian.net/browse/AJ-497


Add Storages object dump to system investigate

The intention of system investigate is to facilitate the investigation of issues with the setup of a given environment.  dwsutil does actually do a wildcard search of cray CRDs, and reports those that it does not recognize as well as those that it would expect to find.  I think that there is value in this effort as this can be used in a customer environment rather than asking our customer to issue a bunch of kubectl commands and then analyzing the output.

So this is a validation that a particular CRD is present?

This particular issue has to do with Storages, there is a different issue that addresses CRDs.  The answer to your question is yes, dwsutil offers a diagnostic mode that will let you know if any expected CRDs are missing.  It doesn't attempt to confirm their structure or validity, just their presence in the system.

oops...closed the wrong issue.


chore(main): release 1.11.0

:rocket: New release
1.11.0 (2023-09-25)
:sparkles: Features

add multilingual mode (websites with multiple languages side by side) (#127) (b7644e2)

:bug: Bug Fixes

hugo error when dateFormat parameter is not set, closes #128 (730ced7)
ToC setting bug, closes #120 (cb695b1)

:hammer: Housekeeping

add link, fix typo in readme (ab16ba1)
add rel='noopener' for external link (6c767d1)
fix typo in README.md (46147ae)
modify footer branding text (41e6f8e)
update bug report issue template (923ddfd)
update links in README.md [skip ci] (e414a38)
update note (59d4b68)

:memo: Documentation

add docs for setting ToC (b0ffc61)
add documentation regarding default date format (97ccc77)

:construction_worker: CI/CD

pin third-party GitHub actions by hash (#119) (218ea44)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hugo-sid/hugo-blog-awesome/releases/tag/v1.11.0 :sunflower:


CSP restricts bridge between phonegap and events

I'm not sure if this is a directly related issue to #196 . Spent a good few hours trying to figure out why my NFC wasn't being detected (even though I'd used the exact JS elsewhere successfully). I realised that the Content-Security-Policy restricts any events from being fired (from the this.webView.sendJavascript(command); command).
Would it be possible to add something to the readme to help others who may experience similar issues in the future? My workaround was to allow all for now.
Does anyone know what the correct script security value should be in this instance?
Thanks.

I'll update the docs, but I need more details.
What did you do to get it to work? Do you have an example of a change to break CSP and what to add to make it work?

Sorry for slow reply. The CSP that was causing the issue was
<meta http-equiv="Content-Security-Policy" content="default-src * 'unsafe-inline'; style-src 'self' 'unsafe-inline'; media-src *; img-src * data:" />

Because it's internal, to fix I effectively removed all CSP as wasn't too bothered about security.
After some testing just now, I've found that I needed 'unsafe-eval', so my final CSP is now:
<meta http-equiv="Content-Security-Policy" content="default-src * 'unsafe-inline' 'unsafe-eval'; style-src 'self' 'unsafe-inline'; media-src *; img-src * data:" />

Since reporting this issue, I've found the Chrome Remote Inspection tool which does highlight this as the issue:

Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src * 'unsafe-inline'". Note that 'script-src' was not explicitly set, so 'default-src' is used as a fallback.
Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src * 'unsafe-inline'".

I hope this helps.
Thanks,
Alex.

Fixed by #323


Auto-generate a PDF of the specification content

Background
Some agencies prefer a single PDF rendering of the specification content rather than navigating the GitHub repository.
See #258 discussion for more background on the issue.
Issue
Manually create a PDF, as has been done for past specification releases, is time-consuming and error-prone as it does not go through the same review process as every line of code/documentation that is added to the repository.
Potential Solution
Create a process (could use GitHub Actions) to auto-generate a PDF of the specification from the Markdown files in the spec-content directory.

@j-d-b , has a place been identified yet to house the built PDF rendering?  I'd be willing to do some of the legwork to help get the action built to generate a PDF, could also do a read-the-docs version as well. I'd recommend using restructured text and a build system like sphinx to generate both the PDF and RTD version. This would allow getting styling similar to what is currently in #258.
A mixture of autodoc/autoapi and static restructured text would help reduce the amount of effort required to keep docs up to date, and allow for stylization.
I would just avoid keeping built docs in the repository as it can start to take a lot of space compared to having the rst files, and a build process that's documented.

@benafischer94 awesome, thank you for offering to take this on.
We don't have anywhere to store files at this point, but I'd imagine much of the work is just writing the process to be able to build the PDF, so we can discuss how that build process is triggered and where the result goes later on, as long as you have all you need to get started.


transform shaders into correct coordinate space

TBD tests.

Oh.
I guess we should add tests. Like, probably tests for bounds and tests for point.+ and tests for shaders getting transformed to the right coordinate space...

eventually, sure :)


Hyper-parameters

Hi, Gupta,
The suggested dimension of hidden states in the paper are 16 and 32 for encoder and decoder. The default parameters in the code are 64 and 128. Is that mean 16 and 32 are a better value for training?  I'm also wondering about the  dimension of hidden states of the baseline method(LSTM and Social LSTM) used in your paper.

For hyper parameters you should look at the print_args script and not the default values. The hidden states for LSTM and Social LSTM should be the same as the ones used for Social GAN and you can get them from print_args.


New Pull

Well done on getting your demo working, Zacquary! For your commit messages you see above you may want to consider writing a bit more descriptive names, helps me to see which assignment I'm reviewing - thank you!

Will do
Sorry bout that
Sent from the moon...

On Dec 7, 2018, at 7:44 AM, Tina Kovacova<EMAIL_ADDRESS>wrote:
Well done on getting your demo working, Zacquary! For your commit messages you see above you may want to consider writing a bit more descriptive names, helps me to see which assignment I'm reviewing - thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Sprint Challenge Code Review
Week 4 - Statistical Tests and Experiments

Part 1 — Load and Validate the Data
Good work on the data clean-up and prep, ZacQuary - you have found the NaNs, "?". You may notice that some of the data are incorrectly recognized as objects, there is extra cleaning required for those.

Part 2— Exploring Data, Testing Hypotheses
The reason why some of your tests did not work is that the data were still in the above mentioned object format and needed to be converted to floats.

Part 3 — Analysis and Interpretation
INC
Good Attempt, ZacQuary!


Request: Ability to strip metadata (and more) on convert

I've opted not to use the tagging functionality, since I let Plex handle that. However, being the neat-freak I am, it would be nice to have an option that basically strips the converted output of all extra information.
Right now the script does subtitle stripping nicely, but it leaves chapter markers intact (actually, it dumps those to a .srt file, but also leaves them behind in the converted file). I'd really like an option to just get rid of everything that isn't an audio or video track. Coupled with an option to clear any meta-data, it would be perfect.

Yeah this would be really handy if its possible, I fined more and more files are being tagged with rubbish at source, having the metadata stripped or just not copied over when converting would be really useful :)

This can be easily achieved from ffmpeg command line using -map_metadata -1 -map_chapters -1
I'm just missing a command line in the python script to toggle passing those on.

Has ther ebeen any news on adding this as I am finding I have to remove more and more metadata each day as different groups keep adding crap to there files, this makes plex unhappy :(

Just pushed an update that attempts to invoke the delete() method built in to mutagen (the library the script uses to tag files) before new tags are written. This should clear any existing metadata. I didn't have any files to test it on so let me know if it works. The metadata wipe occurs just before the new tags are written so make sure you actually do tag the file using the script or you won't see any results.

Cool :)

Not been able to use this feature as I don't tag my files at all :(

As far as I can tell metadata is not being removed right now. Any release from the EVO group for example will still have embedded title metadata with the group name after conversion. Is there anything special I need to do?
Info -> Title = EVO
Sorting -> Title = EVO
I've tested this with a few files, I can send on the conversion log for the latest if it helps? It would be great to get this data scrubbed automatically.

You can strip all metadata using FFMPEG by adding options to the postopts section of autoprocess.ini
See here for the commands
http://superuser.com/questions/441361/strip-metadata-from-all-formats-with-ffmpeg
Sent from my iPhone

On Mar 4, 2017, at 14:50, niallobr<EMAIL_ADDRESS>wrote:
As far as I can tell metadata is not being removed right now. Any release from the EVO group for example will still have embedded title metadata with the group name after conversion. Is there anything special I need to do?
Info -> Title = EVO
Sorting -> Title = EVO
I've tested this with a few files, I can send on the conversion log for the latest if it helps? It would be great to get this data scrubbed automatically.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Awesome! Thanks for that link, it's working perfectly now.

Can I ask how you added this?

Sure! Here's exactly what I added:
postopts = -map_metadata, -1

Cool, and the removes the metadata?

Sorry just seeing this now. Yeah it removes metadata and leaves chapters intact.

I have tried this on two different machines running the script and it seems all metadata stays intact, it's not removing it :(

I would check what build of FFMPEG you're using cause this should work fine. I would also manually test those commands on a sample file with FFMPEG outside the script and see if that seems to remove metadata to determine if it's an FFMPEG problem or a script problem
Sent from my iPhone

On Mar 12, 2017, at 10:06, morpheus2n2<EMAIL_ADDRESS>wrote:
I have tried this on two different machines running the script and it seems all metadata stays intact, it's not removing it :(
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm using FFMPEG 3.2.4 (static) which is fairly recent – grab the latest here and give it a try.

Yep thats the one I have, Metadata does not get stripped :(

I would read over that SO post I listed as there were alternative commands that can be sent to remove metadata and remove chapters that you could try. You have direct access to the FFMPEG command line now so you can try any of these.
Sent from my iPhone

On Mar 17, 2017, at 11:08, morpheus2n2<EMAIL_ADDRESS>wrote:
Yep thats the one I have, Metadata does not get stripped :(
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


auth_key is not guaranteed to be unique

Hello, thank you for creating this extension.  In reviewing the code I see that you implement findIdentityByAccessToken in the BaseUser class.
php
// /users/models/BaseUser.php

    /**
     * @param mixed $token
     * @param mixed $type
     * @return yii\db\ActiveRecord
     */
    public static function findIdentityByAccessToken($token, $type = null)
    {
        return static::findOne(['auth_key' => $token, 'status' => self::STATUS_ACTIVE]);
    }

I assume auth_key column in the user table is meant to be used for authorization (ie. when making rest API calls to the api module). Is that the intended use ? If so, what are your thoughts on the non-uniqueness of this parameter? It appears to be generated using:
php
// /users/models/BaseUser.php
    /**
     * Generates "remember me" authentication key
     * @throws Exception
     */
    public function generateAuthKey()
    {
        /** @var yii\base\Security $security */
        $security = Yii::$app->security;
        $this->auth_key = $security->generateRandomString();
    }

Although it is unlikely, my understanding is generateRandomString() is not guaranteed to be unique and thus may present a security risk (albeit it a small one). Curious as to your thoughts?

Hello! Yes, generating a not unique value is unlikely, but to ensure that it will be unique, in the rules of the User model, define attribute values as unique.
https://github.com/Dominus77/yii2-advanced-start/commit/fb3e609c927a37f9bd40e65c16658c280b8bcf1d

It is also possible to create a function that checks the model attribute for uniqueness, if the generated value is not unique, generate another, until get a unique value.

Yes. I would implement both approaches you suggest for completeness. The problem with just implementing it in dB schema and rules is that it may cause validation error or database error if it is not unique when you attempt the insert. By defining a function to continuously regenerate until the fields are unique you avoid that issue.

Adeded method generateUniqueRandomString() in BaseUser
https://github.com/Dominus77/yii2-advanced-start/commit/5523e78a5b4e7c5f64e81e9ce9019007afc668a0

How would you suggest I update my project to reflect these changes? composer update doesn't seem to do anything

That's right, because the project was installed as a project and not an extension.
Change the following points yourself:
https://github.com/Dominus77/yii2-advanced-start/blob/a1263fab6acb5812483d8600f7d7eb014735dff5/modules/users/models/BaseUser.php#L91-L98
https://github.com/Dominus77/yii2-advanced-start/blob/a1263fab6acb5812483d8600f7d7eb014735dff5/modules/users/models/BaseUser.php#L100-L107
https://github.com/Dominus77/yii2-advanced-start/blob/a1263fab6acb5812483d8600f7d7eb014735dff5/modules/users/models/BaseUser.php#L109-L131
In order not to change it yourself, you need to fork my project into your repository, install a project for yourself from your repository, and get the latest updates from my repository for your own, then use git to update your project on the local machine.


[spaceship] add new mac version fix

This is the spaceship part of a simple fix discussed here with @TKBurner
Thanks for contributing to fastlane! Before you submit your pull request, please make sure to check the following boxes:

[x] Run bundle exec rspec from the subdirectory of each tool you modified. Alternatively, run rake test_all from the root directory.
[x] Run bundle exec rubocop -a to ensure the code style is valid
[x] Read the Contribution Guidelines
[x] We currently don't accept new actions, please publish a plugin instead, more information in Plugins.md

Before submitting a pull request, we appreciate if you create an issue first to discuss the change :+1:

Hi @masteranza - thanks for the contribution! 👍
It looks like this functionality was already merged into master in a slightly different way as part of #4447
Since that's true, I've taken #7063 and rebased it on master, and adjusted it to fit those changes with #7121. Can you please try that PR to see if it still meets your needs?
Thanks 🙌

Hi @mfurtak -  thanks for answering this.
Looks like spaceship part is already addressed by recent commits.
What we need now i the deliver part as you see in 7063 I used a hackish solution, but it does the job.
If someone can improve that please go on.

Yes, that's understood. 👍  I have your change from #7063 in a new branch as part of #7121. Can you try the code in #7121 and let me know if it still does what you want? If yes, I can merge it 🚀


Plugin does not work in AS 2.3

Apparently, Android Studio 2.3 introduced something named "build cache" that does not create the
exploded-aars directory anymore:

ceylon import-jar: Jar file
/Users/bastien/Dev/tmp/MyApplication/app/build/intermediates/exploded-aar/com.
android.support/support-compat/24.2.1/jars/classes.jar does not exist

The workaround is to add this property in gradle.properties:
android.enableBuildCache=false

But then I get another error that I don't know how to fix:
* What went wrong:
Execution failed for task ':app:compileDebugCeylonWithCeylonc'.
> No such property: jarDependencies for class: com.android.build.gradle.internal.dependency.DependencyContainerImpl
  Possible solutions: allDependencies

Environment: AS 2.3, Gradle 3.3, create a simple project with a blank activity, then configure Ceylon in this project and build it.
Works fine with Gradle 2.2.
See also https://github.com/robolectric/robolectric/issues/2948 and http://tools.android.com/tech-docs/build-cache

That's strange, I'm using AS 2.3 and Gradle 3.4.1 in this project, and I've never seen this problem...

Perhaps because I'm using forceImports true.


Use nodejs native abort-controller if available

Node 15 added AbortController and AbortSignal to the global scope.
This PR adds a ./node.js file that will use node.js globals if available and else falls back to the shim.
Note: I found it really hard to export AbortController as default AND add the AbortController/AbortSignal properties to it without extending AbortController so I dropped it. Sadly, this means that this PR is a breaking change as the default value is not available anymore.
Closes #24

Node 16.3 has been released in the meantime.

We see the same problem with Node 18 + TypeScript 4.9.

Same here, @taras did you find any workaround?


[rocksdb_replicator] Add a new thrift field to identify when a leader…

… db is moved.
We use the error code SOURCE_NOT_FOUND in two cases:

An instance has no information about the db in the replicate request.
An instance identifies it is no longer hostting the db requested.

Although the difference is subtle, we want to handle these two
situations differently: we would like to reset the leader only in the
first case (since a new leader might not yet be available in the second
case).
To handle this difference, I'm adding a new field to the ErrorCode
struct. Using the new field will be part of a subsequent PR.

I have not fully understand these two cases, can u elaborate:

what is "have no information"? At what condition or at which logic a instance to decide it "has no information"?
Is this when instance compare ExternalView with current hosting ApplicationDbs in applicationDBManager?

Also, what you mean by "since a new leader might not yet be available in the second case"?

do you mean, the time interval: after the old leader ->follower/offline, but, before follower promote to leader?

I have not fully understand these two cases, can u elaborate:

what is "have no information"? At what condition or at which logic a instance to decide it "has no information"?
Is this when instance compare ExternalView with current hosting ApplicationDbs in applicationDBManager?

Every instance maintains an internal map of the DB's it hosts. Now, if a DB has been removed but still in the map, then the instance knows that is has been recently removed.

Also, what you mean by "since a new leader might not yet be available in the second case"?

do you mean, the time interval: after the old leader ->follower/offline, but, before follower promote to leader?
Yes, since it was very recently removed, the chances of a new leader being elected and available in external view is low.

Can u still comment on:
What kinds of situation SOURCE_NOT_FOUND triggered?
I find two code reference about this error, it seems to be:

"has no information" https://github.com/pinterest/rocksplicator/blob/master/rocksdb_replicator/replicated_db.cpp#L266
db is removed: https://github.com/pinterest/rocksplicator/blob/master/rocksdb_replicator/replicator_handler.cpp#L34
Are these right?

Closing this PR since it's part of https://github.com/pinterest/rocksplicator/pull/471


Release Google.Cloud.Security.PublicCA.V1Beta1 version 1.0.0-beta03

Changes in this release:
No API surface changes; just dependency updates.

Created release for Google.Cloud.Security.PublicCA.V1Beta1-1.0.0-beta03


docs: update the doc of deploy kubernetes with PouchContainer

Signed-off-by: Starnop<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
After verification, our documentation is not enough for users to quickly build the environment. Now we update the items as follows:

s/pouch/pouchcontainer
add configuration instructions of CONTAINERD_ADDR
create a new drop-in file instead of modifying the file created by the kubeadm
correct any inappropriateness in the document

Ⅱ. Does this pull request fix one issue?

None.
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
just docs.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews
About more info about kubeadm/kubelet , please refer to:
https://github.com/kubernetes/kubeadm/issues/822
https://v1-11.docs.kubernetes.io/docs/reference/setup-tools/kubeadm/kubeadm-init/

Why has containerd's address changed? I think it is a big incompatibility there when deployment of kubernetes. @Starnop
Could you show more details about this?

@allencloud
cadvisor has updated the address for no reason in this PR:
https://github.com/google/cadvisor/pull/1919/files
And Kubernetes update the cadvisor ,that's where the difference comes in:
https://github.com/kubernetes/kubernetes/pull/64800


[ignition-*] Upstreams are damaged

/cc

@traversaro
@ahoarau
@autoantwort

https://github.com/orgs/ignitionrobotics/repositories , the upstream for all these ports, reports that it has no repositories.
Individual repos are being redirected to @gazebosim .
We probably shouldn't just "fix the SHA and move on" ... what is the path forward for these libraries in our catalog?
Related: https://github.com/microsoft/vcpkg/pull/24618

If we don't get someone familiar with these changes upstream I think we should =skip these in ci.baseline.txt until a maintainer familiar with the situation is available. I'll prepare that if we have no response by the end of this week.

I found the link of Ignition CMake on GitHub is https://github.com/ignitionrobotics/ign-cmake, should we change the upstream of Ignition CMake to this?

Pinging other users that could be interested in this discussion: @j-rivero @Ace314159 .

I never announced this anywhere as I was not aware that vcpkg had any concept of "mantainer" or similar, differently from other package managers.

It's not a "formal maintainer" kind of thing. More "people we know will care if this is broken and say something about major upstream changes", sort of thing, since the vcpkg team doesn't actually know how to validate that these things build anything that makes sense.

I never announced this anywhere as I was not aware that vcpkg had any concept of "mantainer" or similar, differently from other package managers.

It's not a "formal maintainer" kind of thing. More "people we know will care if this is broken and say something about major upstream changes", sort of thing, since the vcpkg team doesn't actually know how to validate that these things build anything that makes sense.

Yes, that is completely clear, I was just explaining that it was not clear to me how to specify that in the long term I am not be the best person to be the "people we know will care if this is broken and say something about major upstream changes" .

Hello, @BillyONeal @traversaro: thanks for the ping and the past and present efforts to keep the Ignition (now Gazebo after the last rename) libraries. It would be great if we can keep ignition packages working although the Open Robotics internal resources to help with this effort are limited.
I'm going to ask for help into our public forum so maybe someone from the community can volunteer to help maintaining the libraries here.

I would be able to maintain the libraries here since I am actively using these packages for Gazebo.

Awesome Akash! feel free to ping me anytime if we can help you in with any information or change needed. Thanks!

Thanks a lot @Ace314159 ! Feel free to ping me for any problem.

OK, in light of the above I'm going to ensure https://github.com/microsoft/vcpkg/pull/24618 fixes all of them and run with that. Thanks folks!


Refresh the iframe

This fixes the following case:

We report an error
The user fix it but there is another error preventing the restart

Previously we didn't notice anything and would still display the old (fixed)
error. With this commit, we now refresh the iframe and re-run the notifications.

Ah, I was playing around with the GitHub "releases": https://github.com/stubailo/meteor-dev-error-overlay/releases
But I think having a changelog file is better.


HLS Video Streming

Hello,
Is it possible to use it as video streaming player?

@PooriaFarhad yes, it is possible. I played m3u8 files

Hi, some m3u8 files playing normal, some not playing.
For example, this works: http://qthttp.apple.com.edgesuite.net/1010qwoeiuryfg/sl.m3u8
but files m3u files from TVheadent not working. File structure from TVheadent looks like:
#EXTM3U
#EXTINF:-1,DELUXE MUSIC
http://<IP_ADDRESS>:9981/stream/channel/df28765ea0e821f43f8bd8cb9c355c1a

I found a solution: you need video to be encoded h264 and audio mp4a(aac)
Only with this codecs streaming on ios works as needed.

is it possible to do Live Streaming via HLS, I think it belongs to backend but please give me some hints?

You can use https://github.com/Jalle19/node-ffmpeg-mpegts-proxy for live streaming conversion and in config set:
{     "name"         : "Live 1",     "provider"     : "livesources",     "url"          : "/live1",     "source"       : "http://some-live-source",     "avconvOptions": {       "output": [         "-vcodec", "h264", "-acodec", "aac", "-strict", "-2"       ]     }   },

@kitolog
As I understand pattern , please correct me if wrong
iOS device feed camera stream --->  node-ffmpeg-mpegts-proxy --->  then it serve to iOS device play stream via HTTP url ( react-native-video), right ?
And, Could you please tell me some hints about any open source implement feed camera stream on iOS ? I don't have any clue about protocol or mechanism at this point
Thanks

search for streaming plugins.
For example, you can try https://github.com/GuilhermeMedeiros/react-native-videocore for video broadcast. So, what we do:
Video broadcast from device -> node-ffmpeg-mpegts-proxy -> open stream http url in react-native-video

@nvcken @kitolog Nginx has an RTMP module and supports HLS, so you can drop-in VideoCore, push a stream to Nginx, and replay from most recent browsers. You'll have about a 30 second delay, but the benefit is rock-solid software, with a minimum of coding.

Thanks @Zintinio .
I will try it. By the way, 30 second delay has been concern. I don't mind to use any higher performance support HLS & RTMP framework. Please let me know.
As I understand, the way to of proxy server broadcast HLS is:

source RTMP -----> Proxy Server : convert to mpegs by ffmpeg --> output stream http serves .m3u8 ( this way seem serves primarily of OS X, iOS devices)
source RTMP -----> Proxy Server --> output stream RTMP ( flash player, browser ...)

So I think the delay comes from converting rtmp to mpegs by ffmpeg, Could we optimize it?

@nvcken There isn't a way to reduce the delay if you use Nginx/RTMP/HLS. However, if you're willing to learn how to use Live555, you can definitely come up with a closer to real-time solution. Are you trying to stream peer-to-peer, or broadcast?

@Zintinio
My pattern is
Devices stream to Server , Server stream to multi-devices. So it is one-to-many, seem broadcast.

@nvcken have you solved the problem that play stream via HTTP url  in ios?can you share me you idea

Hey just figured out how to play live HLS streams on iOS and Android, in case anybody is having issues:
https://github.com/react-native-community/react-native-video/issues/522#issuecomment-291015622

anyone can suggest which library are you using to play live streaming?

@PooriaFarhad pls help .. i m not able to play http://qthttp.apple.com.edgesuite.net/1010qwoeiuryfg/sl.m3u8 .. when i m using react-native-video component


tageditor: 3.7.7 -> 3.7.8

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/martchus/tageditor/releases.
meta.description for tageditor is: A tag editor with Qt GUI and command-line interface supporting MP4/M4A/AAC (iTunes), ID3, Vorbis, Opus, FLAC and Matroska
meta.homepage for tageditor is: https://github.com/Martchus/tageditor
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.7.8 with grep in /nix/store/1zi63p488ljdh9k5cfmwxcfd800jp2i0-tageditor-3.7.8
found 3.7.8 in filename of file in /nix/store/1zi63p488ljdh9k5cfmwxcfd800jp2i0-tageditor-3.7.8

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
tageditor

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1zi63p488ljdh9k5cfmwxcfd800jp2i0-tageditor-3.7.8 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A tageditor https://github.com/r-ryantm/nixpkgs/archive/220df01fbdd7390d65fc4ec318abd1539067acbc.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1zi63p488ljdh9k5cfmwxcfd800jp2i0-tageditor-3.7.8
ls -la /nix/store/1zi63p488ljdh9k5cfmwxcfd800jp2i0-tageditor-3.7.8/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

tageditor

Maintainer pings
cc @matthiasbeyer for testing.

Result of nixpkgs-review pr 220521 run on aarch64-linux 1

1 package built:

tageditor


Document 'vcpkg import' command

This command sounds like a useful addition. However, I wasn't able to figure out the purpose and intended usage. Please point me to an existing documentation or provide the information here.

@paul-michalik , you can simply find the explanation like:
PS F:\vcpkg\src> ./vcpkg  --help
Commands:
vcpkg search [pat]              Search for packages available to be built
vcpkg install ...          Install a package
vcpkg remove ...           Uninstall a package
vcpkg remove --outdated         Uninstall all out-of-date packages
vcpkg list                      List installed packages
vcpkg update                    Display list of packages for updating
vcpkg upgrade                   Rebuild all outdated packages
vcpkg hash  [alg]         Hash a file by specific algorithm, default SHA512
vcpkg help topics               Display the list of help topics
vcpkg help               Display help for a specific topic
vcpkg integrate install         Make installed packages available user-wide. Requires admin privileges on first use
vcpkg integrate remove          Remove user-wide integration
vcpkg integrate project         Generate a referencing nuget package for individual VS project use
vcpkg integrate powershell      Enable PowerShell Tab-Completion
vcpkg export ... [opt]...  Exports a package
vcpkg edit                 Open up a port for editing (uses %EDITOR%, default 'code')
vcpkg import               Import a pre-built library
vcpkg create
[archivename]        Create a new package
vcpkg owns                 Search for files in installed packages
vcpkg env                       Creates a clean shell environment for development or compiling.
vcpkg version                   Display version information
vcpkg contact                   Display contact information to send feedback
Options:
--triplet                    Specify the target architecture triplet.
(default: %VCPKG_DEFAULT_TRIPLET%, see 'vcpkg help triplet')
--vcpkg-root              Specify the vcpkg root directory
(default: %VCPKG_ROOT%)
@response_file                  Specify a response file to provide additional parameters
For more help (including examples) see the accompanying README.md.
PS F:\vcpkg\src> ./vcpkg import
Error: 'import' requires 3 arguments, but 0 were provided.
Example:
vcpkg import C:\path\to\CONTROLfile C:\path\to\includedir C:\path\to\projectdir
Options:
--triplet                             Set the default triplet for unqualified packages
--vcpkg-root                       Specify the vcpkg directory to use instead of current directory or tool directory
Does it solve you issue?

@PhoebeHui No, it does not solve the issue. What is the intended effect of the import command? What is a CONTROLfile? includedir? projectdir?

I'd like to know what's the CONTROLfile too, is it the .cmake as the example say:
cmake .. "-DCMAKE_TOOLCHAIN_FILE=E:/workspace/vcpkg/vcpkg-export-20170502-161923/scripts/buildsystems/vcpkg.cmake"
(in https://devblogs.microsoft.com/cppblog/vcpkg-introducing-export-command/)

@HagegeR The CONTROL file provides the port name, description, home page, features and dependent port.

so it's not another way of installing files exported using the export command, got it.
thanks!

Yeah, that is the obvious interpretation, right? export - import...


Incompatible with 1.20.2

Attempting to start up this mod on 1.20.2 causes client to crash - however 1.20.2 also added two new graphs modes such as the network debugger.

Hello Kichura, the version supporting 1.20.2 is now available on Modrinth. The new network debug graphs were the source of the issue because Mojang decided to rewrite some of the rendering code when adding them, so I had to adjust the mixin target to account for that. Thank you for your continued interest in the mod.


Add a way to raster cache the children of a Layer

The flutter::MergedContainerLayer mechanism was introduced to solve the problem of a Layer wanting to cache its children, but needing a stable layer::unique_id() under which to do so. This facility is used by OpacityLayer and ImageFilterLayer.
Once https://github.com/flutter/flutter/issues/92287 is fixed, we should be able to extend the tagged cache keys to indicate a new type of cache entry which caches the children of a layer. The list of child id's would have to be held by such a key for proper hash/== functionality, but that is fairly feasible and would simplify the implementations or the child-caching layers.

@flar Hey, if in this task we should add a tag kLayerChildren in the RasterCacheKeyType which like this
enum RasterCacheKeyType {..., kLayerChildern, };

Then we may be should add the Prepare method to support the layer children raster cache like this:
void RasterCache::Prepare(PrerollContext *context, std::vector<Layer*> layer_children, const SkMatrix& ctm) {
      RasterCacheKey cache_key(UniqueId(layer_children), RasterCacheKeyType::kLayerChildren, ctm);
      .....
}

Does above is your mean? Thanks.

@flar I filed a PR for it. Please take a look when you have a chance, thank you. https://github.com/flutter/engine/pull/31875

fixed by https://github.com/flutter/engine/pull/32378


Read once again after stdin token cancelled

What type of PR is this?
/kind bug
What this PR does / why we need it:
This fixes the first test of should support inline execution and attach:
https://github.com/kubernetes/kubernetes/blob/7f129f1c9af62cc3cd4f6b754dacdf5932f39d5c/test/e2e/kubectl/kubectl.go#L603-L612
Short lived attach sessions may have some more content to read after they exited, which is now fixed by trying another read after the token has been canceled via the waitpid exit.
Which issue(s) this PR fixes:

Refers to https://github.com/containers/conmon-rs/issues/736
Special notes for your reviewer:
The --leave-stdin-open=true tests are still failing, but those have to be fixed in another PR:
https://github.com/kubernetes/kubernetes/blob/7f129f1c9af62cc3cd4f6b754dacdf5932f39d5c/test/e2e/kubectl/kubectl.go#L614-L648
Does this PR introduce a user-facing change?

Fixed race between container exit and stdin attach data to be processed.

Codecov Report

Merging #746 (05772a2) into main (fe5ceea) will decrease coverage by 0.38%.
The diff coverage is 23.07%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #746      +/-   ##
==========================================
- Coverage   32.34%   31.96%   -0.39%
==========================================
  Files          13       13
  Lines        1110     1117       +7
  Branches      418      420       +2
==========================================
- Hits          359      357       -2
- Misses        468      478      +10
+ Partials      283      282       -1

/lgtm


1.4.2 tap_action bug

Hi, just updated the code to 1.4.2 and the center tap_action is not working or I doing something wrong, but items is ok:
`- type: custom:radial-menu
icon: mdi:kodi
name: KODI
default_dismiss: false
default_open: true
entity: switch.kodi_input_up
tap_action:
action: call-service
service: switch.turn_off
service_data:
entity_id: switch.kodi_input_up
items:

name: Up
icon: mdi:arrow-up-bold
tap_action:
action: call-service
service: switch.turn_off
service_data:
entity_id: switch.kodi_input_up`

This feature will be better if we have a option for radial menu still showing even if center button is clicked. That is useful for a pad with a "ok center button" and default_dismiss: false default_open: true not keep radial open after the tap

hi, after update to 1.4.3, tap action is working in pc :), but not in my tablet and phone.
Chrome Version issue?
PC: 74.0.3729.131
Phone: 73.0.3683.90
Tablet: 73.0.3683.90
Tablet FF and MIUI browser(phone) are working very well

Possible cache issue. Try clearing cache on phone and tablet in chrome

maybe, I'll try later and back to you

Possible cache issue. Try clearing cache on phone and tablet in chrome

cache cleaned, not working here, the only diference is the browser version :(

Last browser update for tablet and phone fix it.
Ty!!


chore: Add SCHEMATHESIS_BASE_URL environment variable for specifying --base-url in CLI

Codecov Report

Merging #1569 (970dbe6) into master (6c0c255) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1569      +/-   ##
==========================================
- Coverage   98.88%   98.85%   -0.03%
==========================================
  Files          91       91
  Lines        7343     7344       +1
  Branches     1294     1294
==========================================
- Hits         7261     7260       -1
- Misses         51       52       +1
- Partials       31       32       +1

Impacted Files
Coverage Δ

src/schemathesis/cli/__init__.py
99.20% <ø> (ø)

src/schemathesis/service/__init__.py
100.00% <ø> (ø)

src/schemathesis/service/constants.py
100.00% <100.00%> (ø)

src/schemathesis/extra/pytest_plugin.py
98.54% <0.00%> (-1.46%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us.


Upgrade requests~=2.31.0

For https://github.com/Granulate/gprofiler/pull/876

Works in gProfiler - merging this.

I upgraded docker because it fails with new requests - https://github.com/Granulate/gprofiler/actions/runs/7633712264/job/20796904715


HM3D preprocess

Thank you for your impressive work! Could you please provide the script for processing HM3D scenes, or alternatively, the specific steps for processing them? Thank you very much.

Or can you please provide the pointclouds of the whole room of the HM3D?

Hi, we just provided the preprocessing code for scenes and used RSCAN as an example, we will also provide more instructions on HM3D shortly. Thanks for the patience.

Thank you for your outstanding work and for providing the preprocessing scripts for HM3D. However, I have found that the HM3D instance IDs processed by this script do not align with the SceneVerse instance IDs, which is causing me significant confusion. Could you please provide a preprocessed "tgt_id2obj_id.json"? I'm not sure if the issue is with the code or with the version of the HM3D dataset. Thanks a lot.

Hi, we just provided the preprocessing code for scenes and used RSCAN as an example, we will also provide more instructions on HM3D shortly. Thanks for the patience.

Thank you for your outstanding work and for providing the preprocessing scripts for HM3D. However, I have found that the HM3D instance IDs processed by this script do not align with the SceneVerse instance IDs, which is causing me significant confusion. Could you please provide a preprocessed "tgt_id2obj_id.json"? I'm not sure if the issue is with the code or with the version of the HM3D dataset. Thanks a lot.

Hi for HM3D, we have provided a sceneverse2hmsemantic.py for the id matching, you can check it out :-)

Closing this issue as we have already provided related files, feel free to re-open this thread if there is any issue:-)


As  an operator I want to be able to trigger a promotion based on the value of the cart

User can set a numerical threshold for the total of merchandise in the cart (excluding tax and shipping)
User can use inclusions and exclusions on the trigger to indicate which attributes of the cart should activate the trigger (issue 120)
Mockup  (Promotions / Detail / Choose Trigger)  https://www.figma.com/file/vWot2cXXgjcsLpMq4SAX90/Kinetic?node-id=2020%3A54523&t=c9z0pplx66ozEPgb-0

@zenweasel I think we agreed that we would not have a No Trigger option, can you  confirm? To me, if a promo has No Trigger, then it could never be activated and thus it could never cause the associated action for the promo

I can't fathom a reason to have a no trigger promotion

To me, if a promo has No Trigger, then it could never be activated and thus it could never cause the associated action for the promo

That is precisely correct

Ok great, I'll update the mockups


UX enhancements: Adding Tag Grouping name to selected API & automatic Try response clearing

Hello, there are a couple items I wanted to raise here to possibly enhance the user experience.:

Could we add the grouping Tag name (or description) to the top of a selected API? This is especially useful for the "focused" render mode, particularly, when we are navigating through large specs. For example, if you click on an API that is towards the bottom of the page, then you might lose track of what Tag group that API belongs to since you are redirected to the top of the page. There could be various APIs with a similar summary that, depending on the Tag group, could have a totally different function.

After running a "Try" on an API, and then you select another API, then it would be a better UX (in my opinion) to automatically clear the response from the prior API. Else, you may end up seeing an API response from a different API, which could definitely throw off the user. Hopefully this would be an easy change/win!

Thanks a lot and happy to go over either of these in more detailed examples.
-Rey

for point 1 , if you can, then provide a rough visual and i will run it through our UX team
for point 2, check out #392, if this is what you need, then it is already fixed. To try out latest (non-released) build pick it from dist/rapidoc-min.js

OK perfect, yes that does seem to solve it for point 2.
Attached is what I had in mind for point 1:

I based it off this example. By the way, is that scroll bar on the left navigation column a part of this upcoming release? I don't seem to have that on the current UNPKG cut, which is what makes my request more critical, since currently, the entire page automatically scrolls to the top when you select an API, thus you can easily lose track of where you selected the API. Either way I think it'd be nice to include the tag grouping name for a selected API in focused mode. Thanks!


🚀 Feature: Add an --exclude flag to backstage-repo-tools api-reports command

🔖 Feature description
$ backstage-repo-tools api-reports --exclude 'packages/*'

🎤 Context
The api-reports command on backstage-repo-tools is a great way to help a distributed group of people maintain software quality within a backstage ecosystem.
A common pattern for organizing a backstage monorepo is to put plugins under the plugins/* folder, and packages that consume  those plugins under the packages/* folder (e.g. your typical frontend/backend apps).
In such a setup, while it's important to maintain and review API reports for the plugins/*, it's not at all important to do so for anything under packages/* (because of that consumption-only relationship).  ...And so one might naturally use the paths to specify plugins/* when invoking the report check.
...Doing so, however, only uses typescript config for those filtered packages, which may result in API reports being generated which differ from when they are checked again later as a whole in CI.
What if we provided and --exclude flag that took precedence over the existing path argument in terms of typescript config?
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

👍, the actual relationship between packages is a lil bit different though. We already skip API reports for frontend app and backend app packages, so the packages in question are internal libraries in packages/. It could be argued that it makes sense to generate API report for these packages too, as a way of keeping track of the internal API surface. But either way I think it makes sense to provide this option.


Fork tdlib-interface on Kotlin

I do not know if this will work for you. "Everyone who tries to test himself, check for yourself."
I do not even know if it will work for you. Check. Try it. Test.
Fork on Kotlin
Leo, I was frightened by the symbol ^M in Client.java, TdApi.java, NativeClient.java (Notepad++ solves).

Vim Leo Vim Leo Vim Vim Vim...%s/\r//gc
I went to drink Russian Vodka.

Fix ^M

^M is a carriage return, which is a part of newline control character on Windows. Giving the fact that newline representation is different for different OSs, there is no way to provide a single zip-archive distribution which has newline representation suitable for any OS. You can use, for example, dos2unix to get rid of ^M on Linux/Mac OS.

Good answer. Why does not the ^ M symbol appear in the tdlib code?
I think it was possible to do without a symbol of the return of the correctness. Save a few bytes of code.

Cool, but it looks like a crutch.
But the crutch is good.


Parse event input data based on column headers

Flexibility in input file for event data—use column headers to extract appropriate data instead of relying in column indices

Minor, should be able to add parsing support for labeled data rather than expecting explicit format


[S3] Fixing PutObject High Memory Usage

See https://github.com/aws/aws-sdk-net/issues/894 for detailed description

Description
Previously, when invoking the constructor of ChunkedUploadWrapperStream it used to create two byte arrays each with a size of 131,072 (128*1024). From a memory aspect this is inefficient as objects over 85,000 bytes go straight into an area of memory known as the Large Object Heap (LOH). This area is never compacted by default (as compaction of the LOH is expensive), but is collected. Over a lifetime of an application it is possible to end up in a situation where the LOH has taken up most of the RAM.
Now, the byte arrays are still created but with a size of 81,920. The reason for this number can be found here: https://referencesource.microsoft.com/#mscorlib/system/io/stream.cs,50. As 81,920 is under the 85,000 threshold the byte arrays in question go onto the Small Object Heap (SOH). This area of memory is collected and compacted by default.
Motivation and Context
To reduce memory consumption of this call:-
https://github.com/aws/aws-sdk-net/blob/9814e245e0cd9c4435a17185e866813869d427da/sdk/src/Services/S3/Generated/_bcl45/AmazonS3Client.cs#L4421

Total
SOH
LOH

Before
114MB
64MB
50MB

After
98MB
98MB
0.4MB

Testing
See https://github.com/aws/aws-sdk-net/issues/894 for benchmarks within a test harness for pre/post change.
Screenshots (if appropriate)
See https://github.com/aws/aws-sdk-net/issues/894 for screenshots
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have read the README document
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

License

[x] I confirm that this pull request can be released under the Apache 2 license

@sstevenkang Is there anything I can do here to get this PR merged?
Thanks,
Indy

Interesting.  Let me check with the team regarding the history of choosing 128*1024 as the default size.

Hi @sstevenkang,
Any news?
Thanks,
Indy

Thanks for the pull request. This went out with version <IP_ADDRESS> of AWSSDK.Core

Thanks for accepting this, really appreciate it. FYI GetPreSignedURL offers much better performance within a memory context; https://github.com/aws/aws-sdk-net/issues/894#issuecomment-378058075