Remove markup from generated JSON

I noticed that in the generated JSON output, there are p tags in sections like description and type, etc. Example:
{
    "type": "get",
    "url": "/assembly/definition/:did/workspace/:wid/element/:eid",
    "title": "Assembly Definition",
    "name": "GetAssemblyDefinition",
    "description": "
Get definition of requested assembly
 ",
    "group": "Model",
    "version": "0.1.0",
    "permission": [
      {
        "name": "auth",
        "title": "Authenticated access only",
        "description": "
Must be logged in to access this resource.
 "
      }
    ]
}

These are simple to get rid of on the front end that I'm feeding the JSON to, but I think it'd be best to not have them be there in the first place, to separate data from markup. Thoughts? I understand the appeal of support for links in the comments, but feel as though a clean, HTML-free JSON file would be best for most people. Perhaps a new @apiLink annotation could help keep the link support

The fields are HTML capable, if you don't want the HTML-Code disable the markdown parser --markdown false or set a path to an own parser example in https://github.com/apidoc/apidoc/blob/master/MARKDOWN.md where you can remove the p-tags.

Great, don't know how that slipped by me! Closing issue.


Clean corrupted data

Corrupted rows: See ADMA_summary.csv with
Corrupted files: See GOOGL

For case of 2, deleted all summary.csv files with less than 2 rows.


Support all numeric types

Currently we just support u256 values. We should add support for for all signed and unsigned values of size 8-256. They should be included in the base type enum under a new variant named Numeric possibly like this:
enum Numeric{
  Int(usize), // doesn't actually need to be implemented for ERC-20
  Uint(usize)
}

enum Base {
  ...
  Numeric(Numeric)
}

The following should be supported for all numeric types:

[x] use in storage
[x] use as value
[x] basic arithmetic
[x] abi encoding

see #131


Support start a bloop server

As scala-cli itself is using bloop, can it support bloop server command, then I will not need to download the bloop myself with slow connection.

Thanks for reporting! Scala CLI uses a fork of Bloop and it needs to download it when first running with the specific version of the build server.
It's the same as in case of installing Bloop itself, where you only install CLI tool, which will download bloop on first start.
We are working on removing the requirement for a separate fork, which should be done this year.


Patch 1

Changed line 34 from ";" to "." at the end of the line.

I only changed one file... not sure why it says there are 4 file changes.


Breadcrumb: In a Root Domain details page, the name of the target needs to be a link to the Target Details page

[ ] In the breadcrumb of a Root Domain details page, the name of the target needs to be a link to the Target Details page. This is related with #54


Remove unneeded dependency.

Follow up from https://github.com/GoogleCloudPlatform/google-cloud-python/pull/2826#discussion_r91187095.

I'll just merge this once CI is green.


Error: packed attestation: unknown algorithm: -257 with windows Hello

Library works great with chrome+ android phone as mfa, but when trying to register with windows hello I get: Error: packed attestation: unknown algorithm: -257
node_modules/fido2-lib/lib/attestations/packed.js:45:9
Is there any special configuration to setup for windows hello to work?

This is a legit question.  The library claims to be compatible with Windows Hello, but when Windows Hello is used, it uses algorithm -257 (per https://www.iana.org/assignments/cose/cose.xhtml#algorithms it lists RS256 | -257 | RSASSA-PKCS1-v1_5 using SHA-256).  fido2-lib/lib/attestations/packed.js only lists -7, -35, and -36 in its algMap variable, which causes "Error: packed attestation: unknown algorithm: -257" if you try to submit an attestation signed with that algorithm, or "DOMException: An attempt was made to use an object that is not, or is no longer, usable" if you attempt to get a Windows Hello signon without having -257 in your algorithm list.

Any (easy) fix for this?

/node_modules/fido2-lib/lib/attestations/packed.js
const algMap = new Map([
	[-7, {
		algName: "ECDSA_w_SHA256",
		hashAlg: "SHA256",
	}],
	// [-8, {
	//     name: "EdDSA",
	//     hash: undefined
	// }],
	[-35, {
		algName: "ECDSA_w_SHA384",
		hashAlg: "SHA384",
	}],
	[-36, {
		algName: "ECDSA_w_SHA512",
		hashAlg: "SHA512",
	}],
	[-257, {
		algName: 'RSASSA-PKCS1-v1_5',
		hashAlg: 'SHA256'
	}]
]);

对此有任何（简单）修复吗？

That looks like an easy fix - can we make a test case out of it and integrate it?
@2248064246 as you appear to have a good setup to test this, can you add this change, add a test and create a pull request?

Don't know why, but https://github.com/hexagon/webauthn-skeleton works as a charm with Windows hello (+android lock screen, +yubikey)


Should "Social Media Features" be be one of our proofs-of-concept projects?

Add a 👍 to this comment if you want this to be one of our deliverables.
Use comments below this one to discuss pros and cons of each candidate.
On 2016-12-06 (Meeting 3), we'll review the candidates with the most 👍's, and choose at least 2 projects to get started on.

closed in favor of other


EMA model forward

        # model forward in online mode (student)
        x = self.encoder(src, mask, **kwargs)['encoder_out']  # fetch the last layer outputs
        if trg is None:
            return x

        # model forward in offline mode (teacher)
        with torch.no_grad():
            self.ema.model.eval()
            y = self.ema.model(trg, ~mask, **kwargs)['encoder_states']  # fetch the last transformer layers outputs

In the teacher forward pass the  mask_time_indices is the inverse of the one in student, is this correct?
I think the mask in the teacher forward pass should be None since the teacher expects the full version of input data

Hi @anhvth, thanks for your feedback.
The teacher predicts representations from the masked indices in the input (the indices that are masked for src are not masked for trg and vice versa) so the mask must be the inverse of the one in the student.


Update README.md

Update README.md to make the information up to date and to simplify by doing the following:

[ ] Remove yarn references since we're using npm during development of this package

This package is getting phased out, and we do not plan to make any improvements unless they are critical for the remaining apps. Closing this issue


Support authentification by ApiKey instead of user and password for TeamCity?

Dear Sirs,
I'm trying to start with https://marketplace.visualstudio.com/items?itemName=ms-devlabs.vss-services-teamcity, however my problem is that it requires a username and password to be entered (and i cannot have username/password-based connection for automated processes for security reasons).
Would it be possible to support the ApiKey-based authentication?

I will talk to Ashok Kirla on this and update.


We should run it as part of any local commit hook (husky?)
Originally posted by @michael-bowen-sc in https://github.com/ocibuilder/ocibuilder/pull/39

Example hook
gofiles=$(git diff --cached --name-only --diff-filter=ACM | grep '.go$')
[ -z "$gofiles" ] && exit 0

function checkfmt() {
  unformatted=$(gofmt -l $gofiles)
  [ -z "$unformatted" ] && return 0

  echo >&2 "Go files must be formatted with gofmt. Please run:"
  for fn in $unformatted; do
    echo >&2 "  gofmt -w $PWD/$fn"
  done

  return 1
}

checkfmt || fail=yes

[ -z "$fail" ] || exit 1

exit 0
``


MAT-7316: Add flag to control display of all CQL Builder tabs.

MADiE PR
Jira Ticket: MAT-7316
(Optional) Related Tickets:
Summary
Flag to control CQL Builder Tabs display.
All Submissions

[ ] This PR has the JIRA linked.
[ ] Required tests are included
[ ] No extemporaneous files are included (i.e Complied files or testing results)
[ ] This PR is in to the correct branch.
[ ] All Documentation as needed for this PR is Complete (or noted in a TODO or other Ticket)
[ ] Any breaking changes or failing automation are noted by placing a comment on this PR.

DevSecOps
If there is a question if this PR has a security or infrastructure impact, please contact the Security or DevOps engineer assigned to this project to discuss it further.

[ ] This PR has NO significant security impact (i.e Changing auth methods, Adding a new user type, Adding a required but vulnerable package)
[ ] All CDN/Web dependancies are hosted internally (i.e MADiE-Root Repo)

Reviewers
By Approving this PR you are attesting to the following:

Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
The tests appropriately test the new code, including edge cases
If you have any concerns they are brought up either to the developer assigned, security engineer, or leads

I managed to miss the comma in my original commit 😞


reloading questions when switching the page by notifications

Description
added on change listener to update the question and answer section

would it not make more sense to properly ask the router to reload the component with the new url parameters instead of handling this case explicitly? other notifications would then need such handling as well. I think using proper routing is more generic for future notifications.
What do you think?

the routing actually works correctly. the main problem is, that the same view is used so no component gets re-rendered and no ngOnInit function is called. thats why i implemented it that way. regardless of the notifications, the StudentQuestionComponent should update the content if the input changes. this should not be depend on a route change. maybe it will be added dynamically as an overview for all exercises at some point just as an example. the reload should work there as well

the routing actually works correctly. the main problem is, that the same view is used so no component gets re-rendered and no ngOnInit function is called. thats why i implemented it that way. regardless of the notifications, the StudentQuestionComponent should update the content if the input changes. this should not be depend on a route change. maybe it will be added dynamically as an overview for all exercises at some point just as an example. the reload should work there as well

I mean that you could ask the router to reload the component. We use this in other places (e.g. Assess next submission) and it works perfectly. Please give it a try. Here is the code:
this.router.onSameUrlNavigation = 'reload';
this.router.navigate(...)

Did you try out my approach?

not yet. i'm stuck on another topic right now. but i'll add it later today


When stubbing, sometimes message sent from client to stub server is missing last few characters

When SoccomClient sends message to SoccomServer, it sends full message, but receiving end only reads most of message, resulting in exceptions trying to parse into a JSON object:
java.io.IOException: Unterminated string at 58217 [character 89 line 6]
at com.centurylink.mdw.services.test.StubServer.requestProc(StubServer.java:123)
at com.centurylink.mdw.soccom.SoccomServer.processMessage(SoccomServer.java:374)
at com.centurylink.mdw.soccom.SoccomServer.startSub(SoccomServer.java:141)
at com.centurylink.mdw.soccom.SoccomServer.access$000(SoccomServer.java:33)
at com.centurylink.mdw.soccom.SoccomServer$1.run(SoccomServer.java:113)
Caused by: org.json.JSONException: Unterminated string at 58217 [character 89 line 6]
at org.json.JSONTokener.syntaxError(JSONTokener.java:505)
at org.json.JSONTokener.nextString(JSONTokener.java:320)
at org.json.JSONTokener.nextValue(JSONTokener.java:428)
at org.json.JSONObject.(JSONObject.java:244)
at org.json.JSONTokener.nextValue(JSONTokener.java:431)
at org.json.JSONObject.(JSONObject.java:244)
at org.json.JSONObject.(JSONObject.java:357)
at io.limberest.json.JsonObject.(JsonObject.java:25)
at com.centurylink.mdw.model.JsonObject.(JsonObject.java:35)
at com.centurylink.mdw.services.test.StubServer.requestProc(StubServer.java:107)
... 4 more
I think the buffer needs to be flushed on SoccomClient after performing write to socket output stream. (Tried this already, did not work as appearently flush() doesn't do anything for a SocketOutputStream)
This only happens on certain size of messages (large ones), and also only on certain machines, but it happens every time on same machine given same message.

The root cause is that when the message is built in SoccomMessage.java, the size of the message (as determined by calling length() method on String variable) is added to the header part of the message to be sent.   However, the actual resulting size in bytes (by calling getBytes() method) is not always the same, so when the receiving end reads the message, and gets the size of the body from the header, it does not read all the bytes that were sent.


Light Switch on/off

On clicking the light of bulb will be on/off.

I will work on this issue

Go for it @Meghag5

Hey @Meghag5 Will you work on this or shall I unassign you? As you are already assigned three issues!

Hey @Meghag5 Will you work on this or shall I unassign you? As you are already assigned three issues!

Hey @urvashi-code1255 i am working on this

Hey @urvashi-code1255 i am working on this

Okay @Meghag5

Hey @urvashi-code1255  I will like to work on this. Will you assign this to me??

Hey @Meghag5 Are you working on this?

I will work on this issue please assign me @Vishal-raj-1 @urvashi-code1255

hello @Vishal-raj-1  sir @urvashi-code1255 mam, I'm a GGSOC participant,, I was the one who contacted  you in the discord , please assign me this issue , I can share you my prototype

I am a GSSOC'21 Participant
Could you please assign it to me?

Hey @Meghag5 Are you working on this?

clearly, she isn't. Please assign me will do immediatey.

Hey @Meghag5 Are you working on this?

clearly, she isn't. Please assign me will do immediatey.

hey i am due to some health issues I was not able to do it

Hey @Meghag5 Are you working on this?

clearly, she isn't. Please assign me will do immediatey.

hey i am due to some health issues I was not able to do it

ok, stay healthy.


Modal focus escapes when there is no focusable element after the modal HTML

You can escape the enforced focus of the modal, by using the tab key, if there is not focusable element after the modal's HTML markup, such as;

...
            <button ... data-toggle="modal" data-target="#myModal">Open</button> ...

...

If there is no focusable element in between the modal div and the close body tag, the focus will be lost by pressing the tab key.  See  http://codepen.io/anon/pen/vNORPG
The workaround is to add a focusable element in between, such as;

...

    ...

(Demo JS Bin)
@wcopenhaver What browser and OS are you using? Adding the dummy ... doesn't seem to have any effect in Chrome or Firefox on OS X, AFAICT.

the focus will be lost by pressing the tab key

I presume you really mean "focus will move to part of the browser chrome (e.g. the address bar)"? That wouldn't count as a bug anyway. Preventing keyboard users from accessing the address bar would be user-hostile behavior (plus I'm not sure it'd even be possible).

@cvrebert I'm using Windows 10 Chrome 44 and Firefox 40.  The JS Bin Demo seems to work, and it makes no difference with the dummy link because JS Bin still appends a link to the end of the document, the link is:
Edit in JS Bin

If you remove that link using the development tools, open the modal, and press tab multiple times you will experience the issue.
I did mean that focus will move to the part of the browser (specifically the address bar), as you mention above.  And I do agree with you that you shouldn't try to prevent users from accessing the address bar.  I just found it confusing that an element that is not related to the modal, in this case the link, has such an adverse effect on the tabbing behavior of the modal.  I took me awhile to figure out what was going on.

Ah, I see. I'm just a pessimist these days.
Confirmed in Chrome.

Actually, I might not have jammed on the Tab key enough. Can't reproduce. Video proof:

Keep in mind that the Tab key needs to be pressed like 3 or 4 times after the address bar to loop back around into the modal.

If you do not delete the link, the focus will stay captured by the modal as expected, and it will never go to the address bar.

I did delete the link. That's what I opened the dropdown menu in the inspector to do in the video. You can see it disappear from the DOM tree.

@XhmikosR Can you confirm I'm not taking crazy pills here?

If you leave the link the focus will never leave the modal, as expected and discussed in https://github.com/twbs/bootstrap/issues/3101.
If you remove the link the focus will escape the modal and start going to the address bar.

If you remove the link the focus will escape the modal and start going to the address bar.

Which, like we discussed, isn't a bug. So what change are you seeking exactly? An addition to the docs mentioning this edge case difference?

There are two different things happening here:

If you do not delete the link, the focus will never focus on the address bar and it will loop forever inside the modal.
If you do delete the link, the focus will escape the modal and go to the address bar, and other browser elements such as the Firefox search box

What is the correct behavior?
This bring me to my second comment:

I just found it confusing that an element that is not related to the modal, in this case the link (the JS Bin link), has such an adverse effect on the tabbing behavior of the modal

Ran into this problem in 5.3.0 on Chrome in both Fedora and Windows.

Sprinkling some <button type="button">Dead</button> before and after made it work fine.
A Modal / an Offcanvas alone on the page causes the issue.

This issue is regarding v3. Given the elapsed time, v5 would be a separate issue.


liveapi.yext.com

Energized Protection - block ⚡

Let's make an annoyance free, better open internet, altogether!

Issue Submit Form

  Provide the following infos properly, which will help us to resolve your issue quickly.

Issue(s):
Type x in between [ ] and make sure there isn't any space between brackets. Example; for Your Selected Issue(s), type like this - [x]
You can select more than one category of issues if you need to!

[x] Whitelist
[ ] Blacklist
[ ] App Broken
[ ] Website Broken
[ ] Request
[ ] Other Issue

Pack(s):
Write the name of the pack(s) you are using.

Pack(s) Name: blu

Extension(s):
Also name the extension(s) if you are using any. If you don't then leave it blank.

Extension(s) Name:

Domain(s):
If you are submitting this issue for whitelist/blacklist issue, send us the domain(s) for whitelisting/blacklisting here. Kindly use the Code Tag to prevent tracking.

Domain(s):

liveapi.yext.com

Your Config:
Just to ensure there is no issues or conflicts with other app/software/magisk module/extension/source list.
Make sure you are running Energized Protection Service only.

Client:

Version:

Details:
Write us a lil bit more about your issue or query. You can attach any screen shot or log of the issue or advert, this will help to highlight it.

Your Issue Detail:
Breaks Chick fil a app store locator

Thank you for making Energized Protection great with your kind help!

A project by Ador with ❤

@AdroitAdorKhan liveapi.yext.com is being blocked again. I thought it was whitelisted?


NFC tag info

Summary
I'm trying to get the NFC tag ID (e.g. for access permission control or packet delivery signature), but nothing happen at all.
If I play with the sample project and step by step down the NfcManager implementation, I notice that the tag is discovered actualy, but if you can't extract an Ndef from it, you just ignore it.
But sometime, the serial number, I mean the tag Id, is the only thing we want to care about.
It could be useful to return at least the discovered tag, like Plugin.NFC does.
API Changes
Maybe a TagInfo class wich contains the NdefRecords like in Plugin.NFC.
I could do it myself like I did for Shiny.Localization and create a PR to help you, but as you told me you're alone on it, just tell me what you'd like.
Intended Use Case
Explained in summary.

I'm happy to take PR's on the NFC module right now.  I've been exploring different ways of making this easier to consume.  From what I could tell, native does have a "Tag" class that pops out from the native APIs, however, it seems only to contain a collection of the ndef tags.
The iOS side was working for me.  I've still been looking into Android

Ok I'll take a look at it right after some other things done.

This is now working on Android and iOS


Staging to master to add SAR improvements

Description

Related Issues

https://github.com/microsoft/recommenders/pull/1128
Checklist:

[ ] I have followed the contribution guidelines and code style for this project.
[ ] I have added tests covering my contributions.
[ ] I have updated the documentation accordingly.
[ ] This PR is being made to staging and not master.

approving this, since it is the same code previously reviewed by Scott and Andreas https://github.com/microsoft/recommenders/pull/1128


error with null datetime

I was looking at upgrading stac-pydantic and I noticed that it does not like: "datetime": null in a stac item. I got this error message running pytest as well. I think this line needs to be changed: https://github.com/stac-utils/stac-pydantic/blob/c2db77ca5d0ae1bf184092e3e76ab2946fb09183/stac_pydantic/item.py#L23
$ stac-pydantic validate-item https://raw.githubusercontent.com/radiantearth/stac-spec/master/examples/collectionless-item.json

1 validation error for CustomStacItem
properties -> datetime
  none is not an allowed value (type=type_error.none.not_allowed)

To get this error you need to change the version to '1.0.0-rc.4' in version.py

It says here that null is allowed for datetime: https://github.com/radiantearth/stac-spec/blob/v1.0.0-beta.2/item-spec/item-spec.md

I tried this datetime: Union[dt, str] = Field(None, alias="datetime") but then this test fails `` test_validate_item_reraise_exception ``` because it is allowing an item with no datetime key whatsoever to pass.

I think maybe we can update the ItemProperties.datetime type to 'Optional[datetime] = Field(...)'
The '...' indicates the field is required, so a user has to explicitely pass 'None'.  I don't have the pydantic docs in front of me (I'm on mobile atm) but I think this is how it works.

Do you think that in "datetime": null, the problem is that null should be None for Python?

A JSON null is equivalent to a Python None. When you call json.load on that JSON document, you'll get 'datetime': None

I do believe that we should instead be expecting None rather than "null". See #116 for a PR.


Runnig CARLA on cloud (AWS EC2)

Hi,
Dose anyone run CARLA on AWS EC2. How can I get CARLA run on AWS? Is there any toturial or documentation in this regard. The old issues did not help a lot.
Best,

Yes, you can run it on EC2. I don't think there are explicit instructions for that but there is this:
https://carla.readthedocs.io/en/stable/carla_headless/

Hi ! I want to reopen this issue. I'm trying to use the tutorial to carla_headless in the latest (0.9.3), but everytime I get stucked in sudo nohup Xorg :7 &  it says that the command Xorg doens't exist ! Could anyone help me ?
Last lines of log:

Full Log

Thank Jbnunn https://github.com/jbnunn/CARLADesktop.
Succesfully run carla on AWS EC2.

I'll be updating this soon to run with v 0.9.13


Using Cromwell's API client

Such a go getter, you said you weren't going to bother!

Yup.

On the assumption that that one object was just renamed (ie I didn't look at it) :+1:

:+1:


Build stuck at copied files

Build isn't completing and css file is not getting generated in the dist folder.
Im on Windows 11 22000.184

I'll take a look when I'm at my computer, feel free to make a PR if you've found a fix.


Update Font Awesome to v6 (major)

This PR contains the following updates:

Package
Type
Update
Change

@fortawesome/fontawesome-svg-core (source)
dependencies
major
1.2.36 -> 6.2.1

@fortawesome/free-brands-svg-icons (source)
dependencies
major
5.15.4 -> 6.2.1

@fortawesome/free-regular-svg-icons (source)
dependencies
major
5.15.4 -> 6.2.1

@fortawesome/free-solid-svg-icons (source)
dependencies
major
5.15.4 -> 6.2.1

Release Notes

FortAwesome/Font-Awesome
v6.2.1
Compare Source
Change log available at https://fontawesome.com/docs/changelog/
v6.2.0
Compare Source
Change log available at https://fontawesome.com/docs/changelog/
v6.1.2
Compare Source
Change log available at https://fontawesome.com/docs/changelog/
v6.1.1
Compare Source
Change log available at https://fontawesome.com/docs/changelog/
v6.1.0
Compare Source
Change log available at https://fontawesome.com/docs/changelog/

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will ignore this upgrade and you will not receive PRs for any future 6.x releases. However, if you upgrade to 6.x manually then Renovate will reenable minor and patch updates automatically.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


NPE when running generate-email command locally

Description:
Following NPE was noticed when running generate-email locally.
Exception in thread "main" java.lang.NullPointerException
	at org.wso2.testgrid.reporting.GraphDataProvider.processTestFailureSummary(GraphDataProvider.java:110)
	at org.wso2.testgrid.reporting.GraphDataProvider.getTestFailureSummary(GraphDataProvider.java:86)
	at org.wso2.testgrid.reporting.TestReportEngine.generateSummarizedEmailReport(TestReportEngine.java:851)
	at org.wso2.testgrid.core.command.GenerateEmailCommand.execute(GenerateEmailCommand.java:60)
	at org.wso2.testgrid.core.command.CommandHandler.execute(CommandHandler.java:126)
	at org.wso2.testgrid.core.Main.main(Main.java:47)

Seems the failure is at this line because OSVersion is not there in my local DB:
https://github.com/wso2/testgrid/blob/aececc64bf73dadcdd9eb295f9dcda734b767d6e/reporting/src/main/java/org/wso2/testgrid/reporting/GraphDataProvider.java#L109
DB content:
mysql> select * from infrastructure_parameter;
+------------+---------------------+---------------------+------------+-------------------------+--------------------+------+
| id         | created_timestamp   | modified_timestamp  | NAME       | PROPERTIES              | ready_for_testgrid | TYPE |
+------------+---------------------+---------------------+------------+-------------------------+--------------------+------+
| centos-7.4 | 2018-07-28 19:50:50 | 2018-07-28 19:50:50 | centos-7.4 | OS=CentOS,OSVersion=7.4 |                  1 | OS   |
+------------+---------------------+---------------------+------------+-------------------------+--------------------+------+
1 row in set (0.00 sec)

Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Tested with an OS that is not in the database but this was not locally reproducible.


Makefile: organize for faster and more flexible dev workflows

What problem does this PR solve?

Move Makefile scripting to ./scripts
Use make to avoid remaking error-code

Faster and more flexible workflows are now possible. For example, the current CONTRIBUTING advice for re-running a failed test will result in a different test compilation environment.
This results in re-compilation (slower) and could potentially produce a different outcome.
Now ./scripts/test is available for re-running tests in the same environment as the Makefile.
What is changed and how it works?
Shell scripting is moved into ./scripts
For example this change add ./scripts/test which runs "cargo test" in the normal test environment.
I also moved the other major shell scripting out of the Makefile in the form of

./scripts/test-all
./scripts/clippy-all

A helper "make run" is added to run commands in the Makefile environment.
Similarly, there is a helper ./scripts/env for running commands such as ./scripts/env cargo build.
Check List
Tests
This is tested that it doesn't change the Makefile by the CI passing.
./scripts/test was run manually as explained in the README.
./scripts/clippy-all  and ./scripts-tests-all were also run manually.
Side effects
None
Release note

Makefile: organize for faster and more flexible dev workflows

/tests

My workflow is using env EXTRA_CARGO_ARGS=case_name make test to just rerun specified cases.

@BusyJay I added back compatibility support for your current workflow of setting EXTRA_CARGO_ARGS. However, now you can make your workflow a little smoother for the normal case by using ./scripts/test case_name.

In my opinion, just using make is more consistent.

In my opinion, just using make is more consistent.

You can still use make if you like.
I don't think it is a consistent approach to be setting command arguments as environment variables. Consistency on unix is to give arguments as arguments to programs. These changes were initially made due to the inconsistent environments created by following the workflows recommended in the documentation.
But these changes were made to make things more adaptable. make test on Linux might also spend extra time trying to run tests that you don't want it to.

/run-all-tests

/retest

/tests

/test

LGTM

/test

lgtm, seems like an improvement. Really I'd love it if we could just run cargo test or whatever, but I guess it is just a dream for now.

You can run ./scripts/env cargo test now.

We may try https://github.com/matklad/cargo-xtask in the future.

/merge

We may try https://github.com/matklad/cargo-xtask in the future.

It looks like this just lets us write some shell script in Rust. I prefer using make just for tracking build dependencies and just for tasks since it is language/tool agnostic.


the "from layer" function does not work

It is impossible to select a point layer when checking the "Use Layer" option. For example, with the isochrone, I can only generate an isochrones with the "Map now!" function but not with the "Use Layer". I tried "Refresh List" a couple of times but I still can't select a point layer.

no response and duplicate


Another app/console reference

Just came across this - a good one for a newcomer so 🙇♀  please leave if you are already contributing 🙇♀  :)
https://contribute.mautic.org/contributing-to-mautic/developer/local-environment-setup#development-environment. There is a reference here to app/console that needs changing!

Hello,
I will like to work on this but I am not sure what to look for on the link you provided. Thank you.

@Adeola-Adesoba that's great! On the page I shared, it makes reference to an outdated command:
app/console
Since we released Mautic 3, this should now be:
bin/console
So this just needs to be updated to reflect the new path.
Does that make more sense? Happy to talk through further if needed!

Hello,
Thank you, it makes more sense but I cannot locate the file that corresponds to that page in your repository.

@Adeola-Adesoba here it is:
https://github.com/mautic/mautic-community-handbook/blob/master/pages/13.contributing/02.developer/04.Local environment setup/docs.md

As above, but you can also find an edit link at the bottom of every page on the documentation.
This link takes you directly to the file on GitHub, which you can then edit when you're logged in and commit your changes/make a pull request.

Done!

Thanks, sorry for the confusion with it being in a different repo, entirely my fault!


Create power update file

Adding a separate file 'FTT-P-24x71_2022_S0_major_update.xlsx' with the new technologies. The existing data is carried over from the 'FTT-P-24x71_2022_S0.xlsx' to the update file and some other known data has also been added.

See latest commit. I have now brought in a major update of the new FTT:Power inputs with new MEWG and MWLO data to 2022, MWKA for 2023 capacities.
Adjustments to BCET for new technologies with proxy data to make them work in E3ME-FTT
This spreadsheet has been tested and implemented in latest update to E3ME-FTT release 10.1 http://ce-vm-svn02/svn/E3ME/tags/Release_10.1 to have a workable calibrated version for project needs ahead of finalising the cost data work

Integrated in another PR.


Blurry text in Safari

Safari Version 11.0.3 (126<IP_ADDRESS>.1)
Mac Version 10.12.6 (16G1212)
It also affects the iPhone with Safari: iOS 11.2.5 (15D60)
When zoomed in, the text can become blurry in Safari.  See this example:

+1 yup seeing this after I made some really cool  html labels with dropdowns an every thing.. @maxkfranz any ideas what this could be?
in screen shot red is cytoscape node in red (which is crisp) yet the custom html is all blurry...

@kaluginserg Any ideas what this issue is? I love your plugin and may be able to help just don't know where to start?

Similarly love this plugin and would love to help in any way I can!

Ditto. Will you accept a PR on this, @kaluginserg?

Has anyone found a solution to this issue, yet?

I did some investigation and the issue appears to be related to the way Safari (mis)handles z-index. Once you remove the z-index style attribute, Safari stops blurring the labels.
For a reproducible example:

Open the repo's public demo in Safari (or the index.html in the project root)
In the DOM inspector, find the parent div of all labels divs (with class “cy-title”)
From that div, delete the z-index style attribute
Observe that the labels are no longer blurred

Unfortunately, removing z-index puts all labels beneath the other elements in the cytoscape container which is undesirable. That problem becomes very visible when drawing a graph with thick edges or when trying to overlay labels on top of nodes.
My experiments to work-around it using translate3d haven’t been working. I might return to this, but I’m hoping someone else finds these notes useful.


[BUG] console.status flutter

[x] I've checked docs and closed issues for possible solutions.
[x] I can't find my issue in the FAQ.

Describe the bug
When using console.status() with a while True block inside it, the console text has a persistent flutter. Notably:

color doesnt doesnt improve this behavior
spinner styles don't improve this behavior
adding sleep() to slow things down doesn't improve this behavior
changing the refresh interval doesn't improve this

Kooha-2024-08-13-10-32-13.webm
Minimal example
For context, this process is intercepting the output text of Snakemake (invoked with subprocess.Popen())
console = Console()
with console.status("[magenta]Downloading and installing workflow dependencies", spinner = "point") as status:
  while True:
    output = process.stderr.readline()
    if output == '' and process.poll() is not None:
      break
    sleep(2)
    if output.startswith("Job stats:"):
      break

Platform Information

Click to expand
What platform (Win/Linux/Mac) are you running on? What terminal software are you using?
Linux. Terminal via VScode and Konsole
╭───────────────────── <class 'rich.console.Console'> ─────────────────────╮
│ A high level console interface.                                          │
│                                                                          │
│ ╭──────────────────────────────────────────────────────────────────────╮ │
│ │ <console width=76 ColorSystem.TRUECOLOR>                             │ │
│ ╰──────────────────────────────────────────────────────────────────────╯ │
│                                                                          │
│     color_system = 'truecolor'                                           │
│         encoding = 'utf-8'                                               │
│             file = <_io.TextIOWrapper name='<stdout>' mode='w'           │
│                    encoding='utf-8'>                                     │
│           height = 53                                                    │
│    is_alt_screen = False                                                 │
│ is_dumb_terminal = False                                                 │
│   is_interactive = True                                                  │
│       is_jupyter = False                                                 │
│      is_terminal = True                                                  │
│   legacy_windows = False                                                 │
│         no_color = False                                                 │
│          options = ConsoleOptions(                                       │
│                        size=ConsoleDimensions(width=76, height=53),      │
│                        legacy_windows=False,                             │
│                        min_width=1,                                      │
│                        max_width=76,                                     │
│                        is_terminal=True,                                 │
│                        encoding='utf-8',                                 │
│                        max_height=53,                                    │
│                        justify=None,                                     │
│                        overflow=None,                                    │
│                        no_wrap=False,                                    │
│                        highlight=None,                                   │
│                        markup=None,                                      │
│                        height=None                                       │
│                    )                                                     │
│            quiet = False                                                 │
│           record = False                                                 │
│         safe_box = True                                                  │
│             size = ConsoleDimensions(width=76, height=53)                │
│        soft_wrap = False                                                 │
│           stderr = False                                                 │
│            style = None                                                  │
│         tab_size = 8                                                     │
│            width = 76                                                    │
╰──────────────────────────────────────────────────────────────────────────╯
╭─── <class 'rich._windows.WindowsConsoleFeatures'> ────╮
│ Windows features available.                           │
│                                                       │
│ ╭───────────────────────────────────────────────────╮ │
│ │ WindowsConsoleFeatures(vt=False, truecolor=False) │ │
│ ╰───────────────────────────────────────────────────╯ │
│                                                       │
│ truecolor = False                                     │
│        vt = False                                     │
╰───────────────────────────────────────────────────────╯
╭────── Environment Variables ───────╮
│ {                                  │
│     'TERM': 'xterm-256color',      │
│     'COLORTERM': 'truecolor',      │
│     'CLICOLOR': None,              │
│     'NO_COLOR': None,              │
│     'TERM_PROGRAM': 'vscode',      │
│     'COLUMNS': None,               │
│     'LINES': None,                 │
│     'JUPYTER_COLUMNS': None,       │
│     'JUPYTER_LINES': None,         │
│     'JPY_PARENT_PID': None,        │
│     'VSCODE_VERBOSE_LOGGING': None │
│ }                                  │
╰────────────────────────────────────╯
platform="Linux"
rich @ file:///home/conda/feedstock_root/build_artifacts/rich-split_1709150387247/work/dist
rich-click @ file:///home/conda/feedstock_root/build_artifacts/rich-click_1704476299123/work

Same issue here with progress bars + spinner. Seems to get worse as the system load increases.

Hi @pdimens and @cilki, could you please share a full sample code that causes the issue. It would be helpful to replicate and debug the issue.

I tried to do something similar as a toy example and the flutter is not occurring:
#! /usr/bin/env python
import subprocess
from time import sleep
from rich import print as rprint
from rich.panel import Panel
from rich.console import Console

rprint(Panel("Here is a bunch of random text"))
try:
    i = 0
    console = Console()
    with console.status("[magenta]Downloading and installing workflow dependencies", spinner = "point") as status:
        process = subprocess.Popen("find ~ -name \"*\"".split(), stdout=subprocess.PIPE, stderr=subprocess.PIPE, text = True)
        while True:
            print(process.stdout.readline())
            sleep(2)
            i += 1
            if i >10:
                break
except KeyboardInterrupt:
    rprint("Terminating...")
    exit(1)

I tried to do something similar as a toy example and the flutter only occurs when the console block is nested in a progress block:
#! /usr/bin/env python
from time import sleep
from rich.console import Console
from rich.progress import Progress

i = 0
with Progress() as progress:
    console = Console()
    with console.status("text text text", spinner = "point") as status:
        while True:
            sleep(2)
            i += 1
            if i >10:
                break

Confirmed this flickers for me too on Python 3.11/3.12 and rich 13.7.1 on Linux.

Thanks a lot @pdimens and @cilki, examples were helpful and I was able to replicate the issue.
Progress and Console status both animate the display and I suspect that they both interfere with each other's update cycle and should not be mixed together. However, I've yet to check the code base to confirm this.
Meanwhile, I was able to get similar functionality with use of either one if it helps.
#! /usr/bin/env python
from time import sleep
from rich.console import Console

i = 1
console = Console()
with console.status("console.status update ...", spinner="point") as status:
    while True:
        sleep(1)
        i += 1
        if i > 5:
            break

#! /usr/bin/env python
from time import sleep
from rich.progress import Progress

i = 1
with Progress() as progress:
    task = progress.add_task("[green]progress update ...", total=None)
    while True:
        sleep(1)
        i += 1
        if i > 5:
            break


Inconsistent handling of invalid -D options

Create a file named test.dart containing:
main() {
  print('"${new String.fromEnvironment('foo')}"');
}

If you run dart -D test.dart from the command line, the VM prints "No value given to -D option" and then runs the program. This seems reasonable.
But if you run dart -Dfoo test.dart from the command line, the VM prints "No value given to -D option", then tries to load -Dfoo as a file and dies. This seems like unfortunate behavior. If it has recognized the -D option as being invalid, shouldn't it then discard it, as in the previous case?

In one case it is treating it as an error and in the other case as a warning. Should be easy to make it consistent.

I just tested one more case. If you run dart -D=bar test.dart from the command line, the VM prints "No value given to -D option", then tries to load -D=bar as a file and dies.


Subarray with given sum

🚀 Feature
Given an unsorted array of integers(including negative numbers), find a subarray which adds to a given number.
Have you read the Contributing Guidelines on Pull Requests?
YES
Motivation
(Please outline the motivation for the proposal.)
Pitch
(Please explain why this feature should be implemented and how it would be used.)

I'd like to work on it in C++

I'd like to work on it in java

I'd like to work on it in C @Joshitha18

@Joshitha18 let's tweak the issue a little bit `find the smallest sub-array that sums to the given number. Please make this modification in the issue description.

@Kajol-Kumari done

Can we work on python for this issue @Joshitha18

We are logging the scores for the Contributor's Hack 2020. If you have one or more than one PR merged during the program, make sure to fill this form

would like to take this issue in python @iamrajiv

@plazzy99  sir what is the difficulty level for this issue  to fill in the form?


Py/PyAny: remove PartialEq impl and add is()

and fix the fallout, mostly in tests.
See https://github.com/PyO3/pyo3/pull/2175#issuecomment-1048162127 for discussion.

The guide entry presumes #2175 as merged.

Clippy fix needs rebasing on #2184

I actually found a semi-bug; you can see it in the tests (test_sequence and _pyproto), where vector of objects was compared directly to vec![1.into(),...].
It works since small integers are singletons in CPython, so I kept the logic the same for now, but that's not a guarantee and maybe should be changed to use eq.

BTW, a simple cargo test always results in
   Doc-tests pyo3
error[E0554]: `#![feature]` may not be used on the stable release channel
 --> /home/gbr/devel/pyo3/src/lib.rs:2:21
  |
2 | #![cfg_attr(docsrs, feature(doc_cfg, doc_auto_cfg))]
  |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: aborting due to previous error

for me now.
I can run with nightly, but then the UI tests usually fail, which is not pretty.

BTW, a simple cargo test always results in

Yeah I was getting annoyed by this too. It should be fixed with https://github.com/PyO3/pyo3/pull/2176
You can delete the rustdocflags in .cargo/config  to get around it for now.

Looks great, I'm just going to hold off on merging this for a moment until #2175 is ready.


Add testable example for UnitConversions

I'm going to be 'that guy' and put my foot down on the floating point equivalency checks.  While it's certainly possible to do that when everything is a compile time constant, someone will look at this example and then start using it on runtime values.
All comparisons between amounts should be done using the Amount type, not floats.
It's ok to compare the formatted strings however. Those should always be formatted exactly the same.

Agreed about the floating point equality checks.  That is a no-no.  Comparing the formatted strings is fine as well as the amount types themselves.

Good point. We don't want people performing floating point equality checks on runtime values.

Instead of amount.ToUnit(AmountMilliBTC) == 444333222.111
remove floating point equality checks or replace with:
amount.ToUnit(AmountMilliBTC) == NewAmount(444333222.111 * Satoshi).ToUnit(AmountMilliBTC)

Rather  amount.ToUnit(AmountMilliBTC) == NewAmount(444333222.111 * SatoshiPerMilliBTC).ToUnit(AmountMilliBTC)

Those are still floats.

What I mean is amount == NewAmount(444333222.111 * BitcoinPerMilliBTC)

Also, since this (and the last PR really) are intended to be example code that callers will be looking at, I'd suggest not using a . import since that is not how callers will typically be accessing the functions.

OK. Will remove the . import

This looks good to me now.  Please squash it back to a single commit.  @jrick This OK for you now?

sure

Theres a confusion between NewAmount() and Amount.
I'll rename NewAmount() to something more specific.
The name @jrick suggested works: AmountFromBitcoin()
Is that ok?

        amountNaN, err := btcutil.NewAmount(math.NaN()) //returns err

Also ```go
amountNaN = btcutil.NewAmount(math.NaN()) //returns -92233720368.54776 BTC
I will add a NewAmountFromSatoshi() that actually takes in the base BTC and returns `err` for invalid inputs.

Your thoughts please.

        amountNaN, err := btcutil.NewAmount(math.NaN()) //returns err

Also ```go
amountNaN = btcutil.NewAmount(math.NaN()) //returns -92233720368.54776 BTC
I will add a NewAmountFromSatoshi() that actually takes in the base BTC and returns `err` for invalid inputs.

Your thoughts please.

I'd prefer any name changes to be in a separate PR.  Particularly NewAmount since it will need to be accompanied by several other PRs to other repositories which make use of it.

As far as the NaN thing, I honestly don't see much point to it.  Whenever you cast a value, you are explicitly telling the compiler that you know what you're doing.  math.NaN() returns a float64, not an int64.
Any amount of Satoshi will always be int64s under the hood.  Even an Amount is an int64, so there really is no legitimate way to end up in a situation you've illustrated.  Your example code is essentially doing int64(someFloat), so it's doing exactly what you told it to and truncating it!

In the case of NewAmount(), Amount is only invalid if it cannot be represented as an integer type. This is not a problem with Amount() as its always represented as an int64 value.
type Amount int64 is perfect as it is. The example in the godoc, will make the intended use obvious for sleep deficient minds like me.
All thats left is the rename of NewAmount for clarity and avoidance of vagueness.(I'll do that in a separate PR as recommended.)

I don't want unit tests for whether casting actually works.  That is provided by the language and it's reasonable to assume it works as intended.  At the assembly level both an Amount and int64 are exactly the same thing and converting between them is a noop.
If adding tests were free I wouldn't particularly care, but tests are just as much an important part of the code and they do need maintenance.  In this case they add no real value and are just extra code to deal with (both for humans and the testing infrastructure).
I'm not opposed to tests, but make them count.

Also, the example is now using casting as well, and that is fine (because it is an example of usage), but as the name implies, testable examples are tests and they do run during 'go test'.  Testing that again with table driven tests is unneeded.

Point well taken. The examples are fine.
Wrote the test as documentation, and then later on added the testable examples to make the tests visible in on the godoc page. Its now redundant. I've taken it out.
Any other thoughts? Is it a go now?

Looks good. Please squash into a single commit.

Squashed.

OK


Publish main to live on 9/27 @ 10:30 am

Docs Build status updates of commit ed34e99:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-ServicePrincipal.md
:white_check_mark:Succeeded
View (exchange-ps)

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


[WIP] Add technical details of api tokens

Signed-off-by: Andy Doan<EMAIL_ADDRESS>

Docs for<PHONE_NUMBER>bf92ce3ce2473705396ff734a4eca7 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/531/docs/artifacts/html/index.html

@MiloCasagrande @mike-sul @MatthewCroughan  - this is an early WIP, but I'm trying to start writing down the details. There's a big piece missing for fioctl's use of oauth2 that's yet to be sorted out. This lays the ground work and a place we can track things.

@LGTM, of course fioctl's use of oauth2 should be documented too.

Docs for 89a108b8906594e36ca9061ad96312725084edcc are browesable at: https://ci.foundries.io/projects/fio-docs/builds/546/docs/artifacts/html/index.html

@MatthewCroughan - i think this is starting to reflect reality. shouldn't be too long till this is ready.

Docs for 8bc03766b9d8edbefd1ee600c01532647d7ff7a1 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/548/docs/artifacts/html/index.html

Docs for d361d18efa48d5d01c8c5d783fe2e400e5a433f1 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/675/docs/artifacts/html/index.html

@MatthewCroughan - this is ready now that the new site is up

I just found it confusing without the place holder. eg - there is no ci:read scope. I did wonder myself though.

@MatthewCroughan - just force pushed a change with <factory>: removed.

Docs for 9979b00c11d6edda2183ad94a460691464f421c8 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/679/docs/artifacts/html/index.html

I think that reads a lot better since it is generic. I think people will understand when reading in the new ui that this is preprended to help them understand what factory this token applies to. Would only be an issue if a user was trying to manage multiple factories, which is quite rare already.


Known issues? Looking for more maintainers?

At the top of the readme you mention this is a best-effort fork of the trivago/prettier-plugin-sort-imports but that it may not stay up-to-date forever.
I've been hesitant to use trivago/... because of all of the breaking issues, but it seems like on each issue I've examined that I would consider a dealbreaker I find a comment from @IanVS saying that this fork fixes that problem :D
So I guess my questions are:

[ ] are there any known issues from trivago/... that still need to be fixed in this fork, or that you would consider danger for somebody trying to apply this plugin to a large codebase?
[ ] are there any lurking / big issues that need to be fixed sooner rather than later? (TypeScript/babel upgrades?)
[ ] are you looking for regular/casual collaborators? or is your hope to merge this fork back into the upstream?

And to be clear: if you're primarily looking to make forward progress with this fork, and were looking for more casual collaborators I would put my hat in the ring. -- If you'd rather not have a lot of primary development in this fork, then that's fine too -- I wouldn't want to push you beyond your vision here.
I'm currently a regular OSS collaborator on GitHub.com/danger/danger-js and few other projects.

Hi @fbartho thanks for dropping by!
Originally my plan was to only include @blutorange's work to avoid sorting side-effect imports, because I saw that as a critical bugfix that the maintainers of the trivago project disagreed with.
Then @blutorange and I made some other fixes, and I found some nice PRs from the upstream project that seemed stalled out that I brought over.  And, well, instead of needing to worry about staying up-to-date with upstream, we've ended up diverging.  I think this is fine.  I'd like to continue being able to pull in changes (either proposed or merged) from upstream when possible, but merging this back into the Trivago is a non-goal.

are there any known issues from trivago/... that still need to be fixed in this fork, or that you would consider danger for somebody trying to apply this plugin to a large codebase?

I don't know of any issues that remain in this project that are dangerous, and I believe it to be safe to apply to a large project.  I use it in production at work, without problems.

are there any lurking / big issues that need to be fixed sooner rather than later? (TypeScript/babel upgrades?)

I can't think of anything, but maybe @blutorange has something in mind?

are you looking for regular/casual collaborators?

Sure!  I'd be happy for the help.  Immediate tasks that come to mind would be reviewing https://github.com/IanVS/prettier-plugin-sort-imports/pull/12, fixing https://github.com/IanVS/prettier-plugin-sort-imports/issues/9 (minor annoyance), or going through the issues / PRs in the trivago repo to see if there are any that we could consider fixing here.  Or, if you have other thoughts on improvements that could be made, feel free to open an issue and we can discuss.
Cheers!

Oh, and if you'd like to submit a PR to update the wording in the README, I'd be open to that as well.  It's not exactly accurate anymore as you pointed out.

are there any known issues from trivago/... that still need to be fixed in this fork, or that you would consider danger for somebody trying to apply this plugin to a large codebase?

Since this is "just" a linter, I'd consider any possible risk as rather low. Said that, this https://github.com/trivago/prettier-plugin-sort-imports/issues/116 is something we should check whether that issue still exists and fix it, non-deterministic ordering may result in different formatting depending on the build environment.

are you looking for regular/casual collaborators

Yes, as IanVS said, any help is welcome.

but merging this back into the Trivago is a non-goal

Personally I wouldn't mind if they wanted to merge this back, but that doesn't seem too likely at this point.


Error: Cannot enqueue Query after fatal error.

I am trying to apply migrations to an empty database qwerty.
I run db-migrate -e test up and get
[INFO] Using test settings: { username: '<name>',
  password: '******',
  database: 'qwerty',
  host: '<ip>',
  driver: 'mysql',
  multipleStatements: true }
[INFO] require: ./mysql
[INFO] connecting
[INFO] connected
[INFO] creating table: migrations
[SQL] CREATE TABLE IF NOT EXISTS `migrations` (`id` INTEGER  PRIMARY KEY AUTO_INCREMENT NOT NULL, `name` VARCHAR (255) NOT NULL, `run_on` DATETIME  NOT NULL)
[INFO] migration table created
[INFO] loading migrations from dir C:\bb\dbmig\migrations
[INFO] loading migrations from database

[ERROR] Error: Cannot enqueue Query after fatal error.
    at Protocol._validateEnqueue (C:\Users\TimoshenkoEA\AppData\Roaming\npm\node_modules\db-migrate\node_modules\mysql\l
ib\protocol\Protocol.js:193:16)
    at Protocol._enqueue (C:\Users\TimoshenkoEA\AppData\Roaming\npm\node_modules\db-migrate\node_modules\mysql\lib\proto
col\Protocol.js:129:13)
    at Connection.query (C:\Users\TimoshenkoEA\AppData\Roaming\npm\node_modules\db-migrate\node_modules\mysql\lib\Connec
tion.js:185:25)
    at Class.Base.extend.all (C:\Users\TimoshenkoEA\AppData\Roaming\npm\node_modules\db-migrate\lib\driver\mysql.js:411:
34)
    at Class.Base.extend.allLoadedMigrations (C:\Users\TimoshenkoEA\AppData\Roaming\npm\node_modules\db-migrate\lib\driv
er\mysql.js:421:10)
    at Function.Migration.loadFromDatabase (C:\Users\TimoshenkoEA\AppData\Roaming\npm\node_modules\db-migrate\lib\migrat
ion.js:217:10)
    at C:\Users\TimoshenkoEA\AppData\Roaming\npm\node_modules\db-migrate\lib\migrator.js:103:17
    at C:\Users\TimoshenkoEA\AppData\Roaming\npm\node_modules\db-migrate\lib\migration.js:211:5
    at FSReqWrap.oncomplete (fs.js:99:15)

My database.json looks like following:
{
    "test": {
        "username": "<name>",
        "password": "<password>",
        "database": "qwerty",
        "host": "<ip>",
        "driver": "mysql",
        "multipleStatements": true
    },
    "sql-file": true
}

I use:
node v0.12.0
npm 2.5.1
db-migrate 0.9.13
Am I doing something wrong or it is a bug?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11111790-error-cannot-enqueue-query-after-fatal-error?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github).

It looks similar to #238.

It has to be user instead of username. My fault. The error message is confusing though.

closed via https://github.com/db-migrate/node-db-migrate/commit/85397aa4653c7597d2f2f2ce461bcf3def0f2bc3

@wzrdtales I guess it would be better to throw an exception instead of such a workaround. What if someone just misspelled that field name?

I don't think so, username and user are alias able, and to throw an exception if someone sets a config variable is an undefined behavior. It could be needed for some special configuration of the user, if he uses the same config of his project together with db-migrate.


ADR 008: IBC Actor Callbacks

Description
The following is a standardized interface for IBC actors to expose an interface that can be called by IBC applications/middleware to route packet/channel responses back to the original actor so that it can perform subsequent logic
closes: #1660

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

We can very easily just add the interfaces in ibc-go and break the necessary APIs.
This will then unblock CosmWasm team and evmos team from writing middleware to implement ADR-8 on their ends which will be the main usecase for the foreseeable future

Does this PR enable callbacks for an ibc transfer so an action can be taken on a successful transfer of a token?

yes @aidan-calculated that is the intended use case

I believe it is possible to implement ADR 008 now but with the caveat that it must add a switch statement for the supported packet data types. It was determined that the generalized approach required changes to the middleware structure
@AdityaSripal could you update this ADR with the short term approach?

Updated from discussions with Osmosis.
The Packet actor and packet method callbacks are way more relevant.
The handshake callback error handling has been left aside for the moment, since there aren't current users to inform the full design


trace-cmd  don't   work  .

Hello,
use  the trace-cmd  tools  instruments  in the same  UT phone,

it  is  always  error in  Antutu case.
ERROR    Error in insturment trace-cmd
ERROR    trace-cmd returned non-zero exit code 255
(to remove more events  in  ftrace event  disabled, but   yet  fails . )

But  run Ok  in  dhrystone case.

Why?
thanks.

Could you please share your config?

enable items:
instrumentation = [

Collects traces by interacting with Ftrace Linux kernel internal tracer
'trace-cmd',

}
###################################### trace-cmd configuration #####################################
trace-cmd events to be traced. The events can be found by rooting on the device then type
'trace-cmd list -e'
#trace_events = ['power*']
file YAML  :
config:
device: generic_android
device_config:
logcat_poll_period: 20
instrumentation: [cpufreq, trace-cmd, fps, delay]
instrumentation: [cpufreq, trace-cmd]
fps:
drop_threshold: 5
keep_raw: True
trace_events: ['thermal*']
trace_events: ['all']
trace_buffer_size: 30000
Active cooling (needs the "delay" instrumentation)
=============>
I  Investigated  the issue,  it  is  possible  problems  in the UT  phone  debug  configuration  or
trace-cmd  version.
Thanks.

enable items:
instrumentation = [ # Collects traces by interacting with Ftrace Linux kernel internal tracer 'trace-cmd', }
###################################### trace-cmd configuration #####################################
trace-cmd events to be traced. The events can be found by rooting on the device then type
'trace-cmd list -e'
#trace_events = ['power*']

file YAML :
config:
device: generic_android
device_config:
logcat_poll_period: 20
instrumentation: [cpufreq, trace-cmd, fps, delay]
instrumentation: [cpufreq, trace-cmd]
fps:
drop_threshold: 5
keep_raw: True
trace_events: ['thermal*']
trace_events: ['all']
trace_buffer_size: 30000
Active cooling (needs the "delay" instrumentation)
=============>
I Investigated the issue, it is possible problems in the UT phone debug configuration or
trace-cmd version.

Is the YAML above the exact configuration you used? WA should (and did when I just tested it) give you a warning about the duplicate entries like instrumentation. Could you attach your actual YAML file (you can do this by dragging it into the box where you type these comments)

OK。
Now It  can  work  well,  root cause  in  the  phone configuration.
Thanks.


Revert "Separate report name and icon configuration from personal details"

Reverts Expensify/App#7453
My investigation leads to https://github.com/Expensify/App/issues/7453 causing a big regression resulting in a near-infinite loop, so the first step I need to do is rule out this refactoring.

🚀 Cherry-picked to staging by @luacmartins in version: 1.1.37-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @roryabraham in version: 1.1.37-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Support for inline-c modules seems broken

Given a working inline-c sample project, stack ghci works, but hdevtools does not.
$ stack ghci
sample-cabal-project-<IP_ADDRESS>: configure
Configuring sample-cabal-project-<IP_ADDRESS>...
sample-cabal-project-<IP_ADDRESS>: build
Preprocessing executable 'sample-cabal-project' for
sample-cabal-project-<IP_ADDRESS>...
[1 of 1] Compiling Main             ( src/Main.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/sample-cabal-project/sample-cabal-project-tmp/Main.o )
Linking .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/sample-cabal-project/sample-cabal-project ...
sample-cabal-project-<IP_ADDRESS>: copy/register
Installing executable(s) in
/home/dan/1/inline-c/sample-cabal-project/.stack-work/install/x86_64-linux/lts-4.1/7.10.3/bin
Using main module: 1. Package `sample-cabal-project' component exe:sample-cabal-project with main-is file: /home/dan/1/inline-c/sample-cabal-project/src/Main.hs
Configuring GHCi with the following packages: sample-cabal-project
GHCi, version 7.10.3: http://www.haskell.org/ghc/  :? for help
[1 of 1] Compiling Main             ( /home/dan/1/inline-c/sample-cabal-project/src/Main.hs, /home/dan/1/inline-c/sample-cabal-project/.stack-work/odir/Main.i_o )
Ok, modules loaded: Main.
λ >

$ hdevtools --version
hdevtools: version <IP_ADDRESS> (ghc-7.10.3-x86_64-linux, cabal-<IP_ADDRESS>)
$ hdevtools check src/Main.hs

ByteCodeLink: can't find label
During interactive linking, GHCi couldn't find the following symbol:
  inline_c_Main_2_bbad659b194c6226bc20ec8262bc1c599dddeb28
This may be due to you not asking GHCi to load extra object files,
archives or DLLs needed by your current session.  Restart GHCi, specifying
the missing library using the -L/path/to/object/dir and -lmissinglibname
flags, or simply by naming the relevant files on the GHCi command line.
Alternatively, this link failure might indicate a bug in GHCi.
If you suspect the latter, please send a bug report to:
<EMAIL_ADDRESS>

Alright, I will have a look thia weekend. The very descriptive error message already gives me an idea what could fix this. Thanks for reporting!
Sent from TypeApp
On Apr 1, 2016, 8:50 AM, at 8:50 AM, Dan Aloni<EMAIL_ADDRESS>wrote:

Given a working inline-c sample
project,
stack ghci works, but hdevtools does not.
$ stack ghci
sample-cabal-project-<IP_ADDRESS>: configure
Configuring sample-cabal-project-<IP_ADDRESS>...
sample-cabal-project-<IP_ADDRESS>: build
Preprocessing executable 'sample-cabal-project' for
sample-cabal-project-<IP_ADDRESS>...
[1 of 1] Compiling Main             ( src/Main.hs,
.stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/sample-cabal-project/sample-cabal-project-tmp/Main.o
)
Linking
.stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/sample-cabal-project/sample-cabal-project
...
sample-cabal-project-<IP_ADDRESS>: copy/register
Installing executable(s) in
/home/dan/1/inline-c/sample-cabal-project/.stack-work/install/x86_64-linux/lts-4.1/7.10.3/bin
Using main module: 1. Package `sample-cabal-project' component
exe:sample-cabal-project with main-is file:
/home/dan/1/inline-c/sample-cabal-project/src/Main.hs
Configuring GHCi with the following packages: sample-cabal-project
GHCi, version 7.10.3: http://www.haskell.org/ghc/  :? for help
[1 of 1] Compiling Main             (
/home/dan/1/inline-c/sample-cabal-project/src/Main.hs,
/home/dan/1/inline-c/sample-cabal-project/.stack-work/odir/Main.i_o )
Ok, modules loaded: Main.
λ >

$ hdevtools --version
hdevtools: version <IP_ADDRESS> (ghc-7.10.3-x86_64-linux, cabal-<IP_ADDRESS>)
$ hdevtools check src/Main.hs

ByteCodeLink: can't find label
During interactive linking, GHCi couldn't find the following symbol:
 inline_c_Main_2_bbad659b194c6226bc20ec8262bc1c599dddeb28
This may be due to you not asking GHCi to load extra object files,
archives or DLLs needed by your current session.  Restart GHCi,
specifying
the missing library using the -L/path/to/object/dir and
-lmissinglibname
flags, or simply by naming the relevant files on the GHCi command line.
Alternatively, this link failure might indicate a bug in GHCi.
If you suspect the latter, please send a bug report to:
<EMAIL_ADDRESS>

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/hdevtools/hdevtools/issues/16

Works, thanks.


iOS crashes in release version

Description
In debug mode or release ad hoc mode, the app is fine, but when I uploaded a release version to the apple
store, the reviewers reported that the app crashed, but I couldn’t figure out the problem at all. Below is the crash log they fed me back. I’ve symbolized it.
crash.log
React Native version:
System:
OS: macOS 10.15.6
CPU: (4) x64 Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz
Memory: 530.47 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 10.17.0 - /usr/local/bin/node
Yarn: 1.22.4 - ~/.yarn/bin/yarn
npm: 6.14.5 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.8.4 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 13.6, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
Android SDK: Not Found
IDEs:
Android Studio: Not Found
Xcode: 11.6/11E708 - /usr/bin/xcodebuild
Languages:
Java: 11.0.6 - /usr/bin/javac
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2

Thanks for the issue @26903651. Could you provide more information to where this is happening? What components or actions trigger this? Also a repro would be helpful

I have no more information. I tried all the methods to test. The debug mode will not crash, the ad hoc installation will not crash, and it is normal to use testflight to install the release package, but Apple reviewers report that it will crash. , I don’t know how to find this problem, my only resource is the crash log they provide

II don't know what component is wrong. This problem has troubled me for more than 2 months. I started to think it was a version problem of react native. So I updated react native from 0.60 to 0.62 and uploaded to the apple store. The same feedback was received.

I created a testflight invitation code, you can install the app use this testflight code : KNVFCHLD

@26903651 You can try Crashlytics to find the problem.

@26903651 You can try bugsnag to find out the error or open issue on StackOverflow because this seems to issue with your code not react-native.

@26903651 if this is still an issue, providing more investigation is needed. the crash log looks like it mentions AVAudioSession which is not part of core, could you confirm?

Thank you for your attention. This crash log is confusing. I finally found out that it was caused by the push system.


Deprecate in favor of ghetto-skype?

We should deprecate this project in favor of ghetto-skype and edit the README to send people to the other project(which is better maintained)?
What you guys think? @haskellcamargo, @Kuchiriel.

:white_check_mark:
I believe so. This was a very good alternative when it all started, before Microsoft releasing an official version and better alternatives arising, like Ghetto. It also helped to inspire new projects of a web-based Skype, but its role is done. I have no more time to maintain this project because I'm very focused on @quack now. I approve.

What is your opinion, @Kuchiriel?

I agree with you both. @onlurking @haskellcamargo


CSS

CSS
User Story
As a bootcamp student
I want the prework notes to be structure on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

done


Generate GIF without asking for location

Proposed interaction

User: drag-drops a video file onto the Gifski app window
App: immediately generates gif in a temp (cache) path and displays percent-complete progress
App: once GIF is created, a draggable preview thumbnail is displayed alongside 4 action buttons:

Show in Finder: same as today
Share: same as today
Move to: opens dialog for user to choose a destination folder and filename
Copy: copies GIF data to clipboard to paste in another app

User: drags preview thumbnail into another app

What it does today
Upon drag-dropping a video file on the Gifski app, instead of starting the conversion, it immediately asks where the generated GIF should be saved.
Preference: Always Save to Folder
For users who always want generated GIFs to be stored in the same folder, this could be done by adding an "Always Save to Folder" preference that defaults to the cache location
Why?
I use GIFs not as a source-of-truth format, but rather as a ubiquitous, easy-to-share format to capture a few seconds of something I made or something I like. A throw-away format that only exists for sharing.
Examples:

Take a screen recording of a prototype I made, gif it, and then drag-drop it into a chat window to my team.
Take a screen recording of an interaction I really like in some app, gif it, and then drag-drop it into a Google Slides presentation.

So in that sense, they're like screenshots: likely not needed after they've been shared. Kinda like how macOS doesn't ask you where to save a screenshot/recording, it either just saves it, or asks you where to save it after it's been created.
More context
Twitter convo

Totally agree. Make all things like this ephemeral. Let's do it 💪

Store in the app cache
Empty app cache on start and quit
Add checkbox in prefs: Ephemeral mode (on/off) (IMO wording like save to folder just freaks people out)

I really like this. Thanks for opening the issue.

App: immediately generates gif in a temp (cache) path and displays percent-complete progress

We still have to let the user choose size, FPS, quality though. What I'm thinking is that we move those controls from the save panel into the main window.
We show the controls by default before conversion, but the user can optionally skip it by holding down Option key when dragging the video into Gifski.
The flow would be:

User: drag-drops a video file onto the Gifski app window
The controls (size, FPS, etc) fade into the main window, including a "Convert" button. If the user just wants the defaults, they just press the Enter key.

In the conversion completed view we then have:

(Dragging the thumbnail)
"Copy" button (and a "Copy" menu item, so they can also do Command+C)
"Share" button
"Save to Downloads" button
"Save As..." button (I think this is more understandable to the user than "Move to")

I don't really see the usefulness of "Show in Finder".
Thoughts?


Share detail

Fix #82

@andhie Nice2have to bring some TESTs to this new feature what I want to suggest.

added test

@andhie here's a gist of PlantDetailFragment constructing the share string using .observe instead of .value.  Please take a look!

i've updated the code to use observer to build the string.
but the share method can still be debated. currently there is no way to addFlags in the builder, so to add flags, i have to build() the intent first. and the flags added are different for <21 and 21+.
my opinion on separate method is aesthetics and easier to read the onOptionsItemSelected method

Can you elaborate on the use of FLAG_ACTIVITY_MULTIPLE_TASK?
Also, FLAG_ACTIVITY_CLEAR_WHEN_TASK_RESET performs exactly the same as FLAG_ACTIVITY_NEW_DOCUMENT, so the new flag can be used instead.

First you would have to understand why we need the flags in the 1st place. Dianne Hackborn's post explains in detail https://plus.google.com/+DianneHackborn/posts/4QWEQgkB1v2.
i think this line sums up my intention

the meaning of the flag is, "hey system, yeah I know I am launching this thing as part of my task, but to the user it isn't really part of my app, so if they re-launch me from home please get rid of it or the user is doing to be very confused."

These flags are also implemented in ShareActionProvider
https://android.googlesource.com/platform/frameworks/support/+/refs/heads/master/v7/appcompat/src/main/java/android/support/v7/widget/ShareActionProvider.java#409
FLAG_ACTIVITY_CLEAR_WHEN_TASK_RESET performs exactly the same as FLAG_ACTIVITY_NEW_DOCUMENT
That is true, but are you forgetting the app is minSdkVersion=19 ?

Thank you very much for the excellent writeup @andhie! I'll open a ticket internally to explore enhancing our share documentation with Dianne's post.


universal locks across environments

Summary
Hatch doesn't lock project dependencies universally across environments
Details
In hatch a non-default environment also shares dependencies with the project dependencies.
consider the following pyproject.toml:
pyproject.toml

[build-system]
build-backend = "hatchling.build"
requires = ["hatchling"]

[project]
authors = [
  {name = "Justin Flannery", email =<EMAIL_ADDRESS>]
dependencies = [
  "hatch"
]
version = "0.0.0"
license = "MIT"
name = "hatch-pip-compile"
readme = "README.md"
requires-python = ">=3.8"

[tool.hatch.env]
requires = ["hatch-pip-compile"]

[tool.hatch.envs.default]
path = ".venv"
post-install-commands = [
  "pre-commit install"
]
python = "3.11"
type = "pip-compile"
dependencies = ["pytest"]

[tool.hatch.envs.docs]
dependencies = [
  "mkdocs~=1.5.2",
  "mkdocs-material~=9.2.3",
  "mkdocstrings~=0.22.0",
  "markdown-exec[ansi]~=1.6.0"
]
type = "pip-compile"

[tool.hatch.envs.docs.scripts]
build = ["mkdocs build --clean --strict"]
deploy = ["mkdocs gh-deploy {args:}"]
serve = ["mkdocs serve --dev-addr localhost:8080 --livereload"]

We can see there are three different sets of dependencies:

Project Dependencies

hatch

Default Dependencies

pytest

Docs Dependencies

mkdocs~=1.5.2
mkdocs-material~=9.2.3
mkdocstrings~=0.22.0
markdown-exec[ansi]~=1.6.0

These groups of dependencies only make up two virtual environments though, default and docs. For both environments the project dependencies are also included (as long as the environment is not "detached"):

default environment

hatch
pytest

docs environment

hatch
mkdocs~=1.5.2
mkdocs-material~=9.2.3
mkdocstrings~=0.22.0
markdown-exec[ansi]~=1.6.0

The way hatch-pip-compile currently works is that it will create two lock files for these two environments but it is possible that the hatch dependency from the project will get pinned to a different version. Instead we want them to be pinned on the same version everywhere.
Proposed Solution
hatch-pip-compile should use the default environment's lockfile as a constraints file for other environments when running pip-compile (when the environment is not detached)

:tada: This issue has been resolved in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

Thanks. Just out of curiosity, in your provided pyproject.toml, it says:
[tool.hatch.envs.default]
path = ".venv"

is path a parameter from hatch-pip-compile or is this a new feature of hatch. I guess the latter but cannot find in the docs.
Does this mean I don't need to set globally in the config.toml
[dirs.env]
virtual = ".venv"

???

@FlorianWilhelm FYI - I'm actually just learning about
[dirs.env]
virtual = ".venv"

Now that I'm learning about this I'm realizing that the following pattern will get inherited as the default template and you will end up with different environments writing to the .venv directory which is likely what you don't want
[tool.hatch.envs.default]
path = ".venv"


Can't change the number of iterations

Hi,
I am forwarding a message from a colleague who is testing many of the methods implemented here.
He told me he can't change the number of iterations for any of the methods that use this parameter. He has tried many syntactical ways to set this parameter through the command line. However PETPVC always does the default number of iterations: 10.
I could check the code, but do you have any idea about this issues?
Thank you!

Hi Alex,
I just built and ran the following in the test directory with debug output:
eduroam-int-dhcp-97-47-185:test bathomas$ ../bin/petpvc -i filtered.nii -m 4dmask.nii -o out10.nii.gz -p IY -x 6 -y 6 -z 6 -n 10 -d
Performing iterative Yang...
Start fuzziness calculation
sum in region 1 = 109865
sum in region 2 = 1315
sum in region 3 = 4410
sum in region 4 = 4410
matrix:
1 0 0 0
0 1 0 0
0 0 1 0
0 0 0 1
Iteration:
1  0.0322148 1.7316 2.69035 3.58713
2  0.0027501 1.99314 3.01394 4.02053
3  0.000316362 1.99993 3.01935 4.02594
4  4.23082e-05 2.00042 3.01974 4.02634
5  6.04007e-06 2.00047 3.01979 4.02639
6  8.86906e-07 2.00048 3.01979 4.02639
7  1.31718e-07 2.00048 3.01979 4.02639
8  1.96247e-08 2.00048 3.01979 4.02639
9  2.92555e-09 2.00048 3.01979 4.02639
10  4.36165e-10 2.00048 3.01979 4.02639
eduroam-int-dhcp-97-47-185:test bathomas$ ../bin/petpvc -i filtered.nii -m 4dmask.nii -o out10.nii.gz -p IY -x 6 -y 6 -z 6 -n 5 -d
Performing iterative Yang...
Start fuzziness calculation
sum in region 1 = 109865
sum in region 2 = 1315
sum in region 3 = 4410
sum in region 4 = 4410
matrix:
1 0 0 0
0 1 0 0
0 0 1 0
0 0 0 1
Iteration:
1  0.0322148 1.7316 2.69035 3.58713
2  0.0027501 1.99314 3.01394 4.02053
3  0.000316362 1.99993 3.01935 4.02594
4  4.23082e-05 2.00042 3.01974 4.02634
5  6.04007e-06 2.00047 3.01979 4.02639
(This is a noiseless phantom image, so you won’t see much change in the numbers)
Could it be the –k switch that is causing confusion? For ‘petpvc’ the –n switch  controls iterations. For ‘pvc_’ the –i switch controls iterations.
-k is only used for the approaches that also have deconvolutions in them.
If this isn’t the problem, could you reply with some debug output?
Kind regards,
Ben
From: "Alexandre M. S."<EMAIL_ADDRESS>Reply-To: UCL/PETPVC<EMAIL_ADDRESS>Date: Wednesday, 25 January 2017 at 12:51
To: UCL/PETPVC<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [UCL/PETPVC] Can't change the number of iterations (#9)
Hi,
I am forwarding a message from a colleague who is testing many of the methods implemented here.
He told me he can't change the number of iterations for any of the methods that use this parameter. He has tried many syntactical ways to set this parameter through the command line. However PETPVC always does the default number of iterations: 10.
I could check the code, but do you have any idea about this issues?
Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/UCL/PETPVC/issues/9, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGX4aaXpyz8ulNOBL17Xf0dTlo_Ny5GZks5rV0VCgaJpZM4Ltcr6.

Thanks @bathomas, I will check it.

Hi @alexsavio, Any update on this?

Hi @bathomas. Sorry, yes! Everything is sorted out. My colleague was using a non-iterative method, expecting to change the number of iterations. (Don't ask :) ).
Cheers!


Deletes not being output in to the table materialization

I am running diff_tables through a Python script and materializing all rows to a table within my DB. This seems to work great for figuring out our updated columns and rows, however deletes are not being materialized.
Below is the code I'm using. I wanted to check to see if an ID I'm expecting to get an output for in my tables (which isn't there) would show up in the output that the Python script gives me, which it did. I would expect that anything that shows up in the output for diff_tables within my script would also be materialized in to the table that data_diff uses for materialization. From what I can tell, it is not outputting deletes in materialization which throws wrench in the pipeline I'm currently working on.
try:
    for d in data_diff.diff_tables(
        source_table,
        target_table,
        extra_columns=columns,
        key_columns=key_columns,
        materialize_to_table=f"NORSE_DIFF.{SNOWFLAKE_CONN_INFO['schema']}.{table_name}",
        materialize_all_rows=True,
    ):
        if d[1][0] == "c91e4af2-4585-5cbb-924b-cbeb12b7919e":
            print(d[1][0])
except Exception as e:
    print(e)

I'm currently using
<EMAIL_ADDRESS>MacOS Apple Silicon
This is running within a Dagster environment as well.

Seems like there may be an issue with the all_rows query here
They are passed into _materialize_diff here

@devcshort can you explain how to materialize data-diff results to a redshift table for open source version for comparison with redshift db itself on a high level ? I am intend to do the same using dbt , redshift in local dbt core

Hi @devcshort,
I'm sorry for the delay in following up on this. Thank you for raising this issue and for looking into potential solutions!
We made a hard decision to sunset the data-diff package and won't provide further development or support.
If that's of interest, over the past few months, we have rewritten the diffing engine in Datafold Cloud and solved many issues that existed in this package's diffing algorithm.
-Gleb


Refactor usePublisher hook to support real multi-sources

[ ] User can add more sources, beyond screen sharing, it could be other camera and video files
[ ] Manage the sources and states separately, there is no conception of main stream.

There is a bug as follows:
Expected behaviour
When the Publisher app is livestreaming, the user should be able to view the stream in the Viewer app. The user should be able to join an existing stream via the shared link.
Actual behavior
When the user joins an existing stream via the shared link, the video will be stuck on the spinner. The video does not show.
However, if the stream is aborted and restarted from the Publisher app, the video will show up correctly.
Steps to reproduce

Start stream in Publisher app.
Get shared link to stream.
Open shared link (Viewer app).
Notice that the video will be stuck on a spinner: the stream is not showing correctly.
Abort and restart the stream in the Publisher app.
Notice that the video in the Viewer app is now working correctly.

@olbapbetan thank you, sorry we didn't provide enough information here,  you need to use the branch https://github.com/dolbyio-samples/rts-app-react-publisher-viewer/tree/issue-145-multisources.

Please get the shared link from the Publisher app here:


The "enabled" property does not work among a set of similar hotkeys

I have a situation where I have a set of similar hotkey buttons created from a v-for loop. The hotkey keys are the same for all of those. Only one of the buttons should be enabled for hotkey usage at a time, though. However, all buttons should be visible, and usable by mouse/touch click.
Here's how I render tracks (in Compilation.vue):
    <!-- Tracks to work with -->
    <template v-for="track in tracks" :key="track.Id">
        <Track
            :track="track"
            :ref="'track-' + track.Id"
            :displayMode="tracksDisplayMode"
        />
    </template>

Then, each track has it's own Hotkey (in Track.vue):
            <Hotkey
                :enabled="isActiveTrack"
                :keys="['insert']"
                v-slot="{ clickRef }"
            >
                <button
                   ...
                    @click.prevent="createNewCue()"
                    :ref="clickRef"
                >
                </button>
            </Hotkey>

It does only ever work for the last track, if it happens to the active track (isActiveTrack=true). If I enable all hotkeys all the time also only the last one works (gets the click).
A workaround is to use v-if on the Hotkeys. This way, only the actually rendered button gets the click. However this prevents mouse clicks usage on non-active Tracks. (Unless I also render "hotkey-inert" Buttons, on those)

I would suspect the problem is that the same hotkey "keys combination" has multiple registered handlers and only the last added actually gets invoked. It does not make a difference whether "propagate" is set to true.
A solution maybe could be to actually unregister/register the handler on enabled/disabled of the hotkey.

Sorry for the late response.
I resolved the issue in version 2.0.0.
The enabled prop has been changed to disabled to match known HTML element syntax.

Hi @Simolation, no need to apologize since you are not obliged to do anything for free. However, I appreciate your effort! I will include the new version in the next release of my Replayer project.

Unfortunately this should be reopened or marked as wont-fix, as it is not fixed for the situation as described. The mentioned problem still occurs when there are multiple instances of the  component on a single DOM, listening for the same key combination, even when only one of the is "enabled" at any time.
Problem is, that only the last registered gets the event.
To make it work as I have expected here, you should, in useHotkey.ts, do the same as on mount/unmound for enable/disable. This would actually resolve the confusing handling. Currently, the event is just not emit.
However, I now think, that I misunderstood your intention with the enabled/disabled, and in this case I would suggest that you document this like "Enabling/Disabling a hotkey combination does not register/unregister it, thus each hotkey press (combination) can only be served to one Hotkey component instance per DOM". Or similar.
Does this make sense? Sorry to bother you with this again.

To illustrate the situation, you might want to look at my "DismissiblePanel" component: https://github.com/suterma/replayer-pwa/blob/main/src/components/DismissiblePanel.vue
It registers the "ESC" key as a hotkey and observes clicks outside of the area to emit the dismissed. The above problem arises as soon as I have more than one of it.

Hi @suterma,
I just tried it in a demo repo and it works as expected. When I disable all except of one HotKey instance, only the one which is still enabled fires.
The update in 2.0.0 loops through all registered hotkeys and calls the action for those that are enabled. Previously, only the last registered was called, so it should work now.
TestComp.vue
<template>
  <Hotkey :keys="['esc']" :disabled="id !== 3" @hotkey="action">
    I am id {{ id }}
  </Hotkey>
</template>
<script lang="ts">
import { defineComponent } from "vue";
import Hotkey from "./components/Hotkey.vue";

export default defineComponent({
  components: {
    Hotkey,
  },
  props: {
    id: {
      type: Number,
      required: true,
    },
  },
  emits: ["action"],
  setup(props, { emit }) {
    return { action: () => emit("action", props.id) };
  },
});
</script>

App.vue
//...
<TestComp :id="1" @action="action" />
<TestComp :id="2" @action="action" />
<TestComp :id="3" @action="action" />
<TestComp :id="4" @action="action" />
//...

And when I press esc the action gets correctly called with the number 3.

Thanks! It really works now for me. I am sure this was some transient stuff because of caching or something that was not updated in my chain.
Sorry for bothering you, and thanks again for this great component!


Fix memory policy default size

I added a condition to return maxSize=1 only if no maxSize set.

Any updates on this 😃 ?

@tiwiz I bumped the AGP version. Sorry for taking so long. Somehow I missed your comment's notification! :/

@pavlospt thank you! Let's wait for one more reviewer to approve and then we can merge it! Thanks!


component/bt: allow setting a higher baudrate in UART HCI mode (IDFGH-2779)

The ESP32 datasheet specifies that the internal Bluetooth UART HCI baudrate can reach up to 4 Mbps, but the baudrate range is limited to 921600 bps in Kconfig.
I tried to use a FT232R usb serial converter to communicate with the Bluetooth controller at 3 Mbps baudrate (the maximum baudrate supported by the FT232R chip), and it worked as expected.
When using 921600 bps baudrate the SPP speed is limited to 90 KB/s, after changing the baudrate to 3 Mbps the SPP speed is increased to 260 KB/s, so just allow users to set a higher baudrate as the hardware does support it.
Example: controller_hci_uart

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

@redchenjs Thanks for the contribution.

Hi, @redchenjs
I have to do a bunch of tests to verify the 3M baud rate is proper for the controller to handle.
If all tests pass, I will merge this PR, so please wait my update.
Thanks
Abner

An update:
I tried another serial converter FT232H (not FT232R), which supports the baud rate of 4 Mbps, the controller can handle it as well.

Hi @redchenjs
Have you done some throughput actions to test?
Any update for this PR is welcomed.
Thanks
Abner

Hi @redchenjs
Have you done some throughput actions to test?
Any update for this PR is welcomed.
Thanks
Abner

Hi @AbnerFederer
Here's my test result:
Baudrate  Speed(SPP)

921600 bps: 90 KB/s
1 Mbps: 97 KB/s
1.5 Mbps: 147 KB/s
2 Mbps: 194 KB/s
2.5 Mbps: 241 KB/s
3 Mbps: 247 Kb/s - 255 KB/s
3.5 Mbps: 247 Kb/s - 255 KB/s
4 Mbps: 247 Kb/s - 255 KB/s

It looks like there is almost no difference between 3M and 4M baud rates :)

 All committers have signed the CLA.

@AbnerFederer @Alvin1Zhang Is there any update?

Hi
I'm afraid this will cause some errors. So, could you please help add some comments in Help part?
And what about adding several levels of baud rate selections, like 115200, 921600, 200000, 300000, 400000?
Thanks

Hi
I'm afraid this will cause some errors. So, could you please help add some comments in Help part?
And what about adding several levels of baud rate selections, like 115200, 921600, 200000, 300000, 400000?
Thanks

Hi @Wth-Esp,
How does it look now? Now we can select the baud rate in menuconfig, and all these baud rates are from linux/drivers/tty/tty_baudrate.c, they should be the standard baud rate.

Hi @redchenjs
LGTM now.
I will do the PR handling procedure as soon as possible.
Thanks for your contribution.

Thanks for your contribution, and sorry for late reply, we have put the changes into our internal review queue, and the PR will be updated once the changes are available on GitHub. Thanks.

Hi @redchenjs
Have you done some throughput actions to test?
Any update for this PR is welcomed.
Thanks
Abner

Hi @AbnerFederer
Here's my test result:
Baudrate  Speed(SPP)

921600 bps: 90 KB/s
1 Mbps: 97 KB/s
1.5 Mbps: 147 KB/s
2 Mbps: 194 KB/s
2.5 Mbps: 241 KB/s
3 Mbps: 247 Kb/s - 255 KB/s
3.5 Mbps: 247 Kb/s - 255 KB/s
4 Mbps: 247 Kb/s - 255 KB/s

It looks like there is almost no difference between 3M and 4M baud rates :)

It is very interesting that there is no difference between 3M, 3.5M and 4M baud rates. I checked the communication with a logic analyzer and 3.5M/4M baud is giving the same exact signal as 3M. I calculated it being exactly 3M baud. Is this a linux bug or a limitation of my CH340 chip?

Hi @redchenjs
Have you done some throughput actions to test?
Any update for this PR is welcomed.
Thanks
Abner

Hi @AbnerFederer
Here's my test result:
Baudrate  Speed(SPP)

921600 bps: 90 KB/s
1 Mbps: 97 KB/s
1.5 Mbps: 147 KB/s
2 Mbps: 194 KB/s
2.5 Mbps: 241 KB/s
3 Mbps: 247 Kb/s - 255 KB/s
3.5 Mbps: 247 Kb/s - 255 KB/s
4 Mbps: 247 Kb/s - 255 KB/s

It looks like there is almost no difference between 3M and 4M baud rates :)

It is very interesting that there is no difference between 3M, 3.5M and 4M baud rates. I checked the communication with a logic analyzer and 3.5M/4M baud is giving the same exact signal as 3M. I calculated it being exactly 3M baud. Is this a linux bug or a limitation of my CH340 chip?

CH340 Max is 2MBps.  You need change CH343 to test, this max is 6MBps


set FUNCTION_INTERNAL_API=Enabled

Hello,
how do i set FUNCTION_INTERNAL_API=Enabled ?
Thank you in advance for your help.

This is found in the settings table of the bolt card service system.
Bolt Card Service settings

I admit that it doesn't help me much, am I changing these parameters in the redis database?

They are in the Postgres database.

Would it be possible to have more indication?
I have the interface that appears with my domain name And it works.

But when I press Connect bolt card I get this error. (port 9001 is correctly redirected and ufw authorizes port 9001)

I imagine that it's because FUNCTION_INTERNAL_API is not activated but I absolutely don't know how to do SQL.


Scalable Broadcast blank video screen

Hi all. I'm new to WebRTC and RTCMultiConnection and am running into an issue when I try to follow the Scalable Broadcast demo. I followed the instructions here to get started. As instructed, I copied the demo code here and replaced the relevant lines
I replaced <script src="/socket.io/socket.io.js"></script>
with
<script src="https://rtcmulticonnection.herokuapp.com/socket.io/socket.io.js"></script>
and replaced connection.socketURL = '/';
with
connection.socketURL = 'https://rtcmulticonnection.herokuapp.com:443/';
I deployed the page as HTTPS but find that the stream only works consistently on Chrome on Mac. On Windows, most of the time, when I click on the "Open or Join Broadcast" button, I get a blank video screen. I've tried switching the USB socket that I hook up my webcam to in order to get it to work. That seems to fix it about 20% of the time, but 80% of the time, I still get the blank video widget.
I'm not seeing anything in the console that would indicate that there's an error. Perhaps I somehow have to disconnect the previous stream? (Sorry I'm new to this and don't have a good grasp of the principles of WebRTC yet). Any help would be appreciated.

same thing here , having same problem have you found a solution yet kelvin?
Tony


Hyperkube deprecation

Hyperkube was deprecated and there are no official images shipped as of 1.19
We don't seem to use it since 1.18, but it's worth noting as in the v1beta3 it will be removed.
Some references:

kubernetes/kubeadm#1889
kubernetes/kubernetes#84662
kubernetes/kubernetes#85094
/kubernetes/kubernetes#85137
kubernetes/kubernetes#81760

Hypercube deprecated: #144