wip

Refactors manage.py to make it easier to manage commands. It brings the subparsers closer to the handlers, which reduces the number of places a developer has to look to add/remove/modify commands.

Coverage remained the same at 96.322% when pulling 9d3b927af3d80b483ab1ee6fbbbcd3e2646a877b on dw--lookup-managepy into a66e2eae9868d2d6f4b610e28a3d31efbd265b20 on dw--lookuptables.


ProcessInfo: save atomically

Trying to fix https://github.com/iterative/dvc/issues/8570#issuecomment-1321953672.

Codecov Report
Base: 78.18% // Head: 72.72% // Decreases project coverage by -5.45% :warning:

Coverage data is based on head (1296787) compared to base (59caee6).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
- Coverage   78.18%   72.72%   -5.46%
==========================================
  Files          21       21
  Lines         935      935
  Branches      149      113      -36
==========================================
- Hits          731      680      -51
- Misses        176      217      +41
- Partials       28       38      +10

Impacted Files
Coverage Δ

src/dvc_task/proc/process.py
84.00% <100.00%> (ø)

tests/test_utils.py
71.42% <0.00%> (-28.58%)
:arrow_down:

tests/proc/test_manager.py
84.26% <0.00%> (-15.74%)
:arrow_down:

src/dvc_task/proc/manager.py
71.07% <0.00%> (-9.92%)
:arrow_down:

src/dvc_task/contrib/kombu_filesystem.py
30.95% <0.00%> (-8.34%)
:arrow_down:

src/dvc_task/utils.py
38.09% <0.00%> (-7.94%)
:arrow_down:

src/dvc_task/proc/exceptions.py
86.66% <0.00%> (-6.67%)
:arrow_down:

src/dvc_task/worker/temporary.py
75.40% <0.00%> (-4.92%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

ping @karajan1001 @pmrowla.


Allow color prop to override icon color when inside ListItemIcon

Material design allows for list items to be colored with theme colors (as demonstrated here under the "Expand and Collapse" header). When nesting an icon in the <ListItemIcon> component, whatever color prop passed to the icon is overridden by the ListItemIcon color.

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

<List>
  <ListItem>
    <ListItemIcon>
      <LinkIcon color="secondary" /> // Any icon from @material-ui/icons
    </ListItemIcon>
  </ListItem>
</List>

Expect LinkIcon to be colored with theme secondary color.
Current Behavior

Icon is colored with the default list icon color (rgba(0, 0, 0, 0.54) in default settings I believe)
Examples

As mentioned above, the expected behavior is demonstrated on material design's website in the types section. It is also shown in the specs section under the "Collapsed & Expanded" header.
Context

I want to have my list items toggle colors when expanded/collapsed, exactly as shown on the material design website. This is not a major blocker to my project, but the workaround is tedious.

@izyb The simplest way to solve this problem is by using an intermediary component in the ListItemIcon implementation. I like the idea of removing a React.cloneElement( call. @eps1lon What do you think

Do you have a reproduction? It works on my end™: https://codesandbox.io/s/vmwx51nm07
@oliviertassinari
Looking at the implementation https://github.com/mui-org/material-ui/blob/0cf866a898a0aeacc20cc0f119b80395e115826d/packages/material-ui/src/ListItemIcon/ListItemIcon.js#L19-L23
this looks overly complicated and documented wrong (e.g. classes.root should be applied to the root according to doc but is applied to the child because there isn't actually a root element.

Do you have a reproduction? It works on my end™

@eps1lon Oh boy! You have just found our Easter eggs 🥚😆. Enjoy:
+import ListItemIcon from "@material-ui/core/ListItemIcon";
import LinkIcon from "@material-ui/icons/Link";
-import ListItemIcon from "@material-ui/core/ListItemIcon";

https://codesandbox.io/s/1yprl2pln4

Maybe just wrap a div around the child and call it a day?

Deal! @izyb Do you want to work on it? :)

@olivertassinari sounds good to me! Thanks :D


how to invoke an click event inside the sidebar

Can anyone help me with this above code, im new to this semantic ui.
i have 3 buttons inside the sidebar, If any of the button is clicked i want to raise a click event.
can anyone please help me with this

$("#back").click(function() {
}
$("#updatedetails").click(function() {
}
// Needs a different id
$("#refresh").click(function() {
}


Feature/pattern registrations & subs

Resolves #46, Answers #2

invocation and event provide actual URI from message to callee/handler.
Fixes for test projects (tested with crossbar 0.12.1)
exclude UDS when not available (windows)

Logs from all examples attached as requested
logs.zip

wampcra log
Boost: 106000
starting io service
transport connected
responding to auth challenge: {"nonce": "YP8eFe1LD84FjqJg/3Od/LiuLqlT3bdRBHMjhfqKpCINzLo6lZ/OXP3bLZ1x8Upz", "authprovider": "static", "authid": "homer", "timestamp": "2016-02-19T19:07:22.637Z", "authrole": "user", "authmethod": "wampcra", "session":<PHONE_NUMBER>606852}
signature: JvbMXm+ZRBcaxChLO0GjOLR8X6Ut31je0rQkq1wwfRg=
joined realm:<PHONE_NUMBER>606852
left session (wamp.close.normal)
stopped session
stopped io service

awesome! thanks! double thanks for testing with all included examples and crossbar=)

so do we want dormant sample VS2015 projects along with crossbar.io config for windows?

yes pls.
I agree with Jakob that cmake should work for windows, and hence in principle would be enough, but windows devs likely expect VS project files I guess.
so yes if you have a PR with VS build stuff, I'd merge ..


Syncing Yanki between computers

Obsidian 1.7.4 (167)
Yanki 1.5.0

[!NOTE]
This may be a feature request if the behavior is intended.

Context
I use Obsidian and Anki on two different computers. As my main one wasn't available, I set up Anki connect and Yanki on the second one, and did my sync, as usual.
Problem
All my cards in Anki seemed to have been duplicated, and looking at the information, it is linked to the creation of a new YankiNamespace.
Is this intended behavior?

If not, what could I have done differently to avoid this?
If yes, it would be great to have a way to offset this, as it would mean that migrating to a new computer would reset our repetitions.

Hi, thanks for trying the plugin and opening an issue.
I understand why this was confusing, but everything you've described is working as designed.
Good sleuthing spotting the namespace difference. In Yanki, the namespace is basically a unique identifier that's used to determine which Anki cards it can (or can't) touch during synchronization. In the case of Yanki Obsidian, the namespace is generated from the Obsidian Vault ID — the idea being that you can have multiple Obsidian vaults with different sets of cards that you might want to sync to the same Anki collection without interference. (And in your case if you want to delete all the duplicated Anki notes, you can use the unique namespace field to do so in the Anki UI.)
Using the Vault ID instead of the vault name makes the system robust to vault name changes, but it does introduce the edge case you've described here... since every new Obsidian vault has a unique ID. (Though I'm not sure if synced vaults have the same ID. I don't use Obsidian's sync product. Are you syncing Obsidian vaults between your machines? If so, which sync system are you using?)
Theoretically, the namespace could be exposed as a field in the Yanki Obsidian plugin's advanced options, which you could then set to the same thing on both computers... but this would expose a large surface area for potential issues if your underlying notes are out of sync between machines — and I'd need to implement a namespace migration system.
I don't expect to have time to sink into supporting the above in the near future, so for now I'll probably just add a note to the readme warning about this use case.
If you're really determined to do a multi-machine sync, I'd suggest using the Yanki CLI tool directly. It lets you specify custom namespaces at will. Its documentation also includes a deeper dive into namespaces.

Thank you for your swift reply—it is very appreciated.

Though I'm not sure if synced vaults have the same ID. I don't use Obsidian's sync product. Are you syncing Obsidian vaults between your machines? If so, which sync system are you using?

I do indeed use Obsidian's own sync so I can use the same vault between my laptop, my phone, and my desktop. Yanki is installed on my laptop and my desktop.
I completely understand the reason why you used the Vault ID; this makes total sense. My hypothesis is that Obsidian developers have a similar approach in order to manage synchronization accurately.
For now, I want to keep it simple, so I will deactivate Yanki on my desktop and delete the duplicated cards. However, I do plan to renew this very laptop in a few weeks, so I expect the problem to reappear. On the bright side, it may be an opportunity to review all my cards!
Lastly, I want to take the time to sincerely thank you for the labour of love that is Yanki. In the past two decades, I went through, in order, Anki, Supermemo, Remnote, Obsidian_to_Anki, and now Yanki. The combo between Obsidian, its ecosystem, and Yanki is simply wonderful.

Thanks for the kind words!
Migrating to a New Laptop
Assuming you're using AnkiWeb to store your Anki cards, I think if you move the entirety of your Obsidian vault over to the new laptop, including the global Obsidian settings data, then the Obsidian vault ID shouldn't change, and your cards should continue to sync fine with Yanki. (I'd just be sure to do an AnkiWeb sync on the new laptop first.)
Multi-machine Syncing
I'll have to look into whether Obsidian has a persistent vault identifier across synchronized instances... or maybe it would have been better for Yanki to generate a UUID from within the plugin on a per-vault basis and assume the plugin settings file will be synced. Either way this is a bit of a dangerous / disruptive change to the existing architecture... but it's worth looking into.
I'll add a note to the readme about this for the near term, and leave this issue open since a more robust Yanki namespace generation strategy is likely needed.

I ran into the same issue—would love support for Obsidian Sync, and a disclaimer in the README would be great in the meantime.

Closing this for now, but please reply if you run into issues and I will re-open.


doc: Mark all SkeletonModification* and general bone/IK APIs as experimental

Partial alternative to #65768 to keep things simple just before 4.0 beta 1.

While doing this I noticed that #64982 was incomplete. It expects that we'd add is_deprecated=true and is_experimental=true manually in the XML, but that goes against the design of DocTools. Only the written content in <description>, etc. tags (i.e. the actual human-written documentation) should be edited by humans. Everything else is generated.
So the deprecated/experimental state needs to be set in the class/method/signal registration and not after-the-fact in the XML.
Still, for this short-term change, I used the manual XML editing workflow which we have for now.

Superseded by #65801.


enhance Validate and normalize hdfs conf when adding new vault

Proposed changes
Validate and normalize hdfs conf when adding new vault
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


AdminSocketConfigObs::init: failed: rbd-clients

After a fresh deployment on two nodes once I launch % ceph -w I always get the following error:
2015-06-10 18:34:06.599400 7f02fdce8700 -1 asok(0x7f02f8000bf0) AdminSocketConfigObs::init: failed: AdminSocket::bind_and_listen: failed to bind the UNIX domain socket to '/var/run/ceph/rbd-clients/ceph-client.admin.290250.139650727415008.asok': (2) No such file or directory
any suggestions? Thanks.

Well this is due to this entry in your ceph.conf: https://github.com/ceph/ceph-ansible/blob/master/roles/ceph-common/templates/ceph.conf.j2#L68
A simple way to fix this is to create /var/run/ceph/rbd-clients/, the problem should disappear.

Fixed, thanks.

I encountered this issue as well, my fix is to create /var/run/ceph.

I have this issue as well, I have created the missing /var/run/ceph/rbd-clients/ and the issue is resolved. However after a reboot of the node the directory is removed and I need to recreate this folder again. Is there another solution to this?

I didn't realize it until someone pointed it out. On RHEL/CentOS 7, /var/run or /run is a tmpfs. It is why /var/run/ceph/rbd-clients keep disappearing, after reboot, for me.

you can redirect the directory by adding the following to group_vars/all
rbd_client_admin_socket_path: /var/lib/ceph

@rootfs this might be a regression, I am getting this on CentOS7 now


Stylus/Sass/Less

How can i use Stylus/Sass/Less with single template?

Hi @martinsbicudo, other compilers for CSS are not supported because I thought PostCSS is the most versatile out there.
You can use PostCSS plugins for working with Stylus/Sass/Less. Something like https://github.com/seaneking/poststylus or https://github.com/postcss/postcss-scss should work for you.

Please note that these plugins doesn't allow you to access all the features of the respective preprocessor. Working with components you don't really need a preprocessor since every piece have his own code. You may need mixins, functions and loops (maybe)...nothing you can't get with postcss keeping things simple.


mvn package breaks on OpenJDK

Right now, we had to add Openjfx to build on OpenJDK,
mainly because it uses javafx.util.Pair:
https://github.com/pinterest/doctorkafka/blob/6949cc24840a669516aaed173445928f01fc6943/drkafka/src/main/java/com/pinterest/doctorkafka/notification/Email.java#L5
https://github.com/pinterest/doctorkafka/blob/6949cc24840a669516aaed173445928f01fc6943/kafkastats/src/main/java/com/pinterest/doctorkafka/util/OperatorUtil.java#L7
https://github.com/pinterest/doctorkafka/blob/6949cc24840a669516aaed173445928f01fc6943/drkafka/src/main/java/com/pinterest/doctorkafka/KafkaClusterManager.java#L15
It would be ok to replace javafx.util.Pair with another structure ?
Thanks.

This issue has been fixed Guilherme's change: https://github.com/pinterest/doctorkafka/pull/4


[Feature] A tool to activate any of a window controls through hotkeys

Description of the new feature / enhancement
This is an interesting tool:
https://github.com/zsims/hunt-and-peck
It allows to insert a label under each of the controls/fields of a Window. By tipping in sequence the corresponding keys, it is possible to activate the associated control/field, as if clicking on it.
Some examples:
Explorer:

Excel:

Calculator:

The tool is already working great, but I think it could be better optimized and integrated into Windows.
This is why I'm suggesting integrating something similar in PowerToys.
Scenario when this would be used?
It would be possible to control almost any application features and fields without actually making use of the mouse.
A VIM-like experience.
It would increase workflow speed and quality a lot in most cases.
Supporting information
This is another similar tool, but it only allows to activate any of the visible windows by placing some labels in their title bars:
https://github.com/JochenBaier/fastwindowswitcher

I have a great addition to your tool, I have also made a request for a new power Toy but I think it would be a great thing to combine both.
I think it would be cool to make the Tab key functionality directional, so you can chose the direction it will go next.
Is it possible from your code to connect the name of the controll Elements, and their position into a struct ? Because then I would have code to find the next wanted Element in any direction and you have the code for accessing this Element.

@KillerKaninchen1284
That's not my tool so unfortunately, I don't know how it could be modified, but it would be great to obtain something similar in PowerToys, I suppose it would be integrated with Windows even greatly.


[Snyk] Fix for 2 vulnerabilities

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

frontend/package.json
frontend/package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity
Reachability

169/1000  Why? Confidentiality impact: None, Integrity impact: None, Availability impact: High, Scope: Unchanged, Exploit Maturity: Proof of Concept, User Interaction (UI): None, Privileges Required (PR): None, Attack Complexity: Low, Attack Vector: Network, EPSS: 0.00045, Social Trends: No, Days since published: 150, Reachable: No, Transitive dependency: Yes, Is Malicious: No, Business Criticality: High, Provider Urgency: High, Package Popularity Score: 99, Impact: 5.99, Likelihood: 2.81, Score Version: V5
Uncontrolled resource consumption SNYK-JS-BRACES-6838727
Yes
Proof of Concept
No Path Found

131/1000  Why? Confidentiality impact: Low, Integrity impact: High, Availability impact: None, Scope: Unchanged, Exploit Maturity: No data, User Interaction (UI): None, Privileges Required (PR): None, Attack Complexity: High, Attack Vector: Network, EPSS: 0.01055, Social Trends: No, Days since published: 0, Reachable: No, Transitive dependency: Yes, Is Malicious: No, Business Criticality: High, Provider Urgency: High, Package Popularity Score: 99, Impact: 7.03, Likelihood: 1.86, Score Version: V5
Improper Verification of Cryptographic Signature SNYK-JS-ELLIPTIC-8172694
Yes
No Known Exploit
No Path Found

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Uncontrolled resource consumption

⛔ Snyk has found 1 issues.

Icon
Severity
Issues

Critical
0

High
1

Medium
0

Low
0

⛔ security/snyk check is completed. Snyk found 1 issues. (View Details)
✅ license/snyk check is completed. No issues were found. (View Details)


implement setters support as described in #39

This PR implements setters support as described in #39.
Note: In run_setter_on_object, we do an expect on the optional setter_position field, as the FunctionType.call method requires a SpanWithSource. I looked into refactoring to change the setter_position from an Option but it looked like it'd have wide consequences. @kaleidawave - would love your thoughts on this one.

Awesome! Yep I believe the event positions were added in #37. There were some difficult cases so just decided it to make it Option and use None in those cases. If it's too difficult at the moment then fine to skip.
Tests look good and are passing. Will double check and merge when I back week after next!

Makes sense. Simulating the function call currently does not accept an Option for the SpanWithSource. I chose to fail closed here, panicking if a setter had no SpanWithSource because failing open would mean we would silently fail to evaluate the side effects of the setter function.
If I understood your comment correctly, are you suggesting we update the FunctionType.call to accept an Option?

If I understood your comment correctly, are you suggesting we update the FunctionType.call to accept an Option?

Actually the opposite, I think that Event::Setters position field should be SpanWithSource. I think at the moment there are a few creations that don't have access to a position (but could!) such as Environment::delete_property and some of the class field registration. Those kind of set events won't trigger a setter so I think the setter_position.expect("Setter position is required!"); is fine for the moment (and the tests still run). If you would like to improve that so that every event has a SpanWithSource position then let me know and I can look into it further and write up a issue explaining the cases :)

As for the rest, PR is great! Thanks for the contribution + adding tests for it!

Hey, again just added some additional tests. One thing that was missing in the PR is reporting the assignment type mismatch with the assigned value. It was being checked in .call (by regular function parameter checking), just wasn't being surfaced as the result was stored into let _ =. Not to worry have added some basic typing report it.
https://github.com/kaleidawave/ezno/blob/main/checker/specification/specification.md#setter-assignment-type
https://github.com/kaleidawave/ezno/blob/c4ada6d3e52e0fad5a4eadcc513b3bfd31d2d0ed/checker/src/types/properties.rs#L743


Add option to upload dev-contiv-vswitch image to Dockerhub

Coverage increased (+0.08%) to 48.573% when pulling 3c5fb9756b1d5ef8b620285c010de173be065a99 on mklotton:upload_dev into c76ba04d73ce79776a9c120f58041eb54e83b277 on contiv:master.


do not create device vectors for the entire sparse page while compu…

…ting histograms...

while creating the compressed histogram indices, the row vector is created for the entire
sparse page batch. this is needless as we only process chunks at a time based on a slice
of the total gpu memory
this pr will allocate only as much as required to store the appropriate row indices and the entries

@RAMitchell please review.

@sriramch it would help me if you can format the text of the PR title and the commit messages as you want them to appear to other developers. Normally I will remove commit messages about lint fixes, so don't worry about those, but if the title and other commit messages can be descriptive and well formatted that would be great.

@sriramch it would help me if you can format the text of the PR title and the commit messages as you want them to appear to other developers. Normally I will remove commit messages about lint fixes, so don't worry about those, but if the title and other commit messages can be descriptive and well formatted that would be great.

@RAMitchell thanks for the tip. i'll remember this for the future pr's. i see that you already changed the title for this one.


Replace SquirrelID with correct dependency from repository

When enginehub/squirrelid#18 is resolved, change SquirrelID back to downloading from the repository.

Fixed


Merge Editor duplicates code on added/added + Accept Combination

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.73.0
OS Version: Windows_NT x64 10.0.19043

Steps to Reproduce:

Add a struct with some fields and functions in one branch
Add the same struct in another branch, with a minor difference in one of the fields.
Merge the branches and open the merge editor.
Choose Accept Combination.

On this example, m_Count is int on the left side and const int on the right side. I'd expect "Accept Combination" to have only a single copy for the matching lines, and duplicate only the different line (and then I'd choose which one to keep), but it duplicates the entire struct.

I also see that deleted/deleted is considered a conflict. I see no reason why.

I also had duplicated code on two occasions when rebasing after the new VSCode update (and that's why I searched for this issue here). It seems that it kept the old code and added the new code and didn't complain or considered a change/conflict. I still haven't used "Accept Combination", so this duplication happened without using this new feature.
In both situations, I had to manually remove the old code. If I find this situation again I will add here its screenshots.

Verification steps:
{
    "languageId": "javascript",
    "base": "const { readFileSync } = require('fs');\n\nlet paths = process.argv.slice(2);\nmain(paths);\n\nfunction main(paths) {\n    // print the welcome message\n    printMessage();\n\n    let data = getLineCountInfo(paths);\n    console.log(\"Lines: \" + data.totalLineCount);\n}\n\n/**\n * Prints the welcome message\n*/\nfunction printMessage() {\n    console.log(\"Welcome To Line Counter\");\n}\n\n/**\n * @param {string[]} paths\n*/\nfunction getLineCountInfo(paths) {\n    let lineCounts = paths.map(path => ({ path, count: getLinesLength(readFileSync(path, 'utf8')) }));\n    return {\n        totalLineCount: lineCounts.reduce((acc, { count }) => acc + count, 0),\n        lineCounts,\n    };\n}\n\n/**\n * @param {string} str\n */\nfunction getLinesLength(str) {\n    return str.split('\\n').length;\n}\n",
    "input1": "const { readFileSync } = require('fs');\n\nlet paths = process.argv.slice(2);\nrun(paths);\n\nfunction run(paths) {\n    // print the welcome message\n    printMessage();\n\n    const data = getLineCountInfo(paths);\n    console.log(\"Lines: \" + data.totalLineCount);\n}\n\nfunction printMessage() {\n    console.log(\"Welcome To Line Counter\");\n}\n\n/**\n * @param {string[]} paths\n*/\nfunction getLineCountInfo(paths) {\n    let lineCounts = paths.map(path => ({ path, count: getLinesLength(readFileSync(path, 'utf8')) }));\n    return {\n        totalLineCount: lineCounts.reduce((acc, { count }) => acc + count, 0),\n        lineCounts,\n    };\n}\n\n/**\n * @param {string} str2\n */\nfunction getLinesLength(str) {\n    return str.split('\\n').length;\n}\n",
    "input2": "const { readFileSync } = require('fs');\n\nlet paths = process.argv.slice(2);\nmain(paths);\n\nfunction main(paths) {\n    // print the welcome message\n    printMessage();\n\n    const data = getLineCountInfo(paths);\n    console.log(\"Lines: \" + data.totalLineCount);\n}\n\nfunction printMessage() {\n    console.log(\"Welcome To Line Counter\");\n}\n\n/**\n * @param {string[]} paths\n*/\nfunction getLineCountInfo(paths) {\n    let lineCounts = paths.map(path => ({ path, count: getLinesLength(readFileSync(path, 'utf8')) }));\n    return {\n        totalLineCount: lineCounts.reduce((acc, { count }) => acc + count, 0),\n        lineCounts,\n    };\n}\n\n/**\n * @param {string} str1\n */\nfunction getLinesLength(str) {\n    return str.split('\\n').length;\n}\n",
    "result": "const { readFileSync } = require('fs');\n\nlet paths = process.argv.slice(2);\nrun(paths);\n\nfunction run(paths) {\n    // print the welcome message\n    printMessage();\n\n    const data = getLineCountInfo(paths);\n    console.log(\"Lines: \" + data.totalLineCount);\n}\n\nfunction printMessage() {\n    console.log(\"Welcome To Line Counter\");\n}\n\n/**\n * @param {string[]} paths\n*/\nfunction getLineCountInfo(paths) {\n    let lineCounts = paths.map(path => ({ path, count: getLinesLength(readFileSync(path, 'utf8')) }));\n    return {\n        totalLineCount: lineCounts.reduce((acc, { count }) => acc + count, 0),\n        lineCounts,\n    };\n}\n\n/**\n<<<<<<< HEAD\n * @param {string} str1\n=======\n * @param {string} str2\n>>>>>>> theirs\n */\nfunction getLinesLength(str) {\n    return str.split('\\n').length;\n}\n",
    "initialResult": "const { readFileSync } = require('fs');\n\nlet paths = process.argv.slice(2);\nrun(paths);\n\nfunction run(paths) {\n    // print the welcome message\n    printMessage();\n\n    const data = getLineCountInfo(paths);\n    console.log(\"Lines: \" + data.totalLineCount);\n}\n\nfunction printMessage() {\n    console.log(\"Welcome To Line Counter\");\n}\n\n/**\n * @param {string[]} paths\n*/\nfunction getLineCountInfo(paths) {\n    let lineCounts = paths.map(path => ({ path, count: getLinesLength(readFileSync(path, 'utf8')) }));\n    return {\n        totalLineCount: lineCounts.reduce((acc, { count }) => acc + count, 0),\n        lineCounts,\n    };\n}\n\n/**\n<<<<<<< HEAD\n * @param {string} str1\n=======\n * @param {string} str2\n>>>>>>> theirs\n */\nfunction getLinesLength(str) {\n    return str.split('\\n').length;\n}\n"
}

Use the command Open Merge Editor As Json to open that JSON and notice that conflicts with equal changes on both sides are auto-resolved.


Add documentation images

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

Types of changes

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

Describe changes
Updated architectural diagram
Added quickstart diagram
Added two stack diagrams (to ch. 7 of the guide)

@htahir1 I put the prettier code in a separate branch / draft PR request. This branch should be good to go for now.


Liquidity Mining

Liquidity mining is a mechanism to help with market-making by providing liquidity and earning liquidity incentives in return.
Overview

Liquidity incentives provide more stronger incentives than staking incentives because a lock-up of paired assets accompanies the provision of liquidity.
Liquidity providers get DEVs as liquidity incentives based on the number of DEVs provided.
The liquidity incentives are equal to the sum of the staking rewards and holder rewards. This spec means that the liquidity incentives are equal to "the reward for staking to Property owned by itself."
As noted above, liquidity provision is a new method of staking and does not accompany new inflation.
However, if a liquidity provider performs Property staking, the liquidity incentive is increased. This spec is tentatively named the Conjunction Incentives.

Conjunction Incentives (tentative name)

Since liquidity incentives are stronger than staking incentives, the Conjunction Incentives works to reinforce the incentive for staking.
Earns incentive of 0.00000003 per block for a liquidity pool price of $1, liquidity provision of 1, and Property staking of 1.
Example: Liquidity pool price is $12, liquidity provision is 100 DEV, staking is 200 DEV, for the incentive is 2.270592 per year. 12 * (100+200) * (1 year in seconds / 15) * 0.00000000000003
Example: Liquidity pool price is $1000, liquidity provision is 100 DEV, staking is 200 DEV, for the incentive is 189.216 per year. 1000 * (100+200) * (1 year in seconds / 15) * 0.00000003

Technically flow

Staking UNI tokens into the (newly developed)Liquidity contract.
Retrieve the number of DEVs associated with the UNI tokens and treat the same number as stake.
The APY is recalculated.
The liquidity provider's reward is taken directly from the Allocator.calculateMaxRewardsPerBlock() calculation. (no proration for stakers and holders)
Run a batch script to get the price of the liquidity pool once every 24 hours.

I think this is a great idea, I just had a few questions about specifics. Will the staking act like another property on the stakes.social platform? For example will staking the uniswap liquidity token bring down the APY because it is counting both pools in the staking amount? So ultimately the inflation rate stays the same because the coins are coming from the same place that coins are coming from now for people who stake regular properties ?

great idea

I think this is a great idea, I just had a few questions about specifics. Will the staking act like another property on the stakes.social platform? For example will staking the uniswap liquidity token bring down the APY because it is counting both pools in the staking amount? So ultimately the inflation rate stays the same because the coins are coming from the same place that coins are coming from now for people who stake regular properties ?

@calaber24p In liquidity mining, DEV staked for liquidity mining is treated the same as normal staking (staking to Property). Therefore, the APY will be reduced.

UPDATED

Liquidity providers receive DEV tokens as an incentive-based on pool size and price, and elapsed blocks.
Imagine there are two people in the network, Alice and Bob, Alice staked 10 liquidity tokens for 1 day (5760 blocks), and Bob staked 5 liquidity tokens for 3 days (17280 blocks).
Alice_share = 10 tokens * 5760 blocks = 57600
Bob_share = 5 * tokens * 17280 blocks = 86400

Global_share = Alice_share + Bob_share = 144000 token_blocks

Alice owns (57600 / 144000) = 40%
Bob owns (86400 / 144000) = 60%

The inflation rate is calculated using the Policy.rewards formula. Its arguments are the liquidity pool size and the number of liquidity providers: assuming Alice provides 200 DEV and Bob provides 100 DEV, for a total provides of 300 DEV, and the current pool price is $5, and the previous highest price is $12, the daily (5760 blocks) incentive for Alice and Bob is calculated as follows.
Infration = Policy.rewards(300, 2) = 0.000239953684689838 ≈ 0.00024
Global_reward_per_block = Infration * $5 / $12 = 0.0001

Alice earns (0.0001 * 5760 * 40%) = 0.2304 = 42.048% APY
Bob earns (0.0001 * 5760 * 60%) = 0.3456 = 126.144% APY

The closer the pool price is at the highest price, the more incentives. When updating the highest value, the coefficient is 1. Larger liquidity pools reduce incentives, and more liquidity providers increase them.

UPDATED

Liquidity providers receive DEV tokens as an incentive-based on pool size and price, and elapsed blocks.
Imagine there are two people in the network, Alice and Bob, Alice staked 10 liquidity tokens for 1 day (5760 blocks), and Bob staked 5 liquidity tokens for 3 days (17280 blocks).
Alice_share = 10 tokens * 5760 blocks = 57600
Bob_share = 5 tokens * 17280 blocks = 86400

Global_share = Alice_share + Bob_share = 144000 token_blocks

Alice owns (57600 / 144000) = 40%
Bob owns (86400 / 144000) = 60%

The inflation rate is calculated using the Policy.rewards formula. Its arguments are the liquidity pool size and the number of liquidity providers. Assuming Alice provides 200 DEV and Bob provides 100 DEV, for a total provides of 300 DEV, and the current pool price is $5, and the previous highest price is $12, the daily (5760 blocks) incentive for Alice and Bob is calculated as follows.
Infration = Policy.rewards(300, 2) = 0.000239953684689838 ≈ 0.00024
Global_reward_per_block = Infration * $5 / $12 = 0.0001

Alice earns (0.0001 * 5760 * 40%) = 0.2304 = 42.048% APY
Bob earns (0.0001 * 5760 * 60%) = 0.3456 = 126.144% APY

The closer the pool price is at the highest price, the more incentives. When updating the highest value, the coefficient is 1. Larger liquidity pools reduce incentives, and more liquidity providers increase them.

The quickest and lean-way to get started: fork Geyser.
https://github.com/ampleforth/token-geyser
Contract: https://github.com/ampleforth/token-geyser/blob/master/contracts/TokenGeyser.sol
The contract asks for a few initial parameters.

Arg
Value

stakingToken
IERC20(0x4168cef0fca0774176632d86ba26553e3b9cf59d)

distributionToken
IERC20(0x5cAf454Ba92e6F2c929DF14667Ee360eD9fD5b26)

maxUnlockSchedules
TBD

startBonus_
33 *from ETH/AMPL Geyser

bonusPeriodSec_
5184000 *from ETH/AMPL Geyser

initialSharesPerToken
TBD

I think forking Geyser is fine if thats the way we decide we want to do it, also just simply matching the liquidity mining to the staking reward + creator reward is fine with me. I am indifferent on how we achieve liquidity mining. I am only against increasing the supply of tokens or using the developers wallet to fund liquidity mining. I believe we need to find a self sustaining way to offer liquidity mining rewards.

@calaber24p Thank you for your comment.

If we use forked Gayser, we can't actuarially match a reward to the staking reward + creator reward.
I, too, think the liquidity mining budget will be contributed from the current supply.

In my first proposal, the provided liquidity DEVs are treated as actually being staked, lowering the APY. But, pooled DEVs as liquidity have no "actual" effect on creators. There is an indirect effect of stabilizing the price of DEV, but that is true for all stakeholders, not just creators. In other words, I believe that the impact of the size of the liquidity pool on the Dev Protocol APY is an excessive tight coupling.
What do you think about this?

I personally like your first proposal even if we decide to not fork Geyser. I know the lowering of APY might not be completely desirable with how it affects creators, but ultimately I think we should figure out a more permanent solution versus a short term one. There are other options to counteract the addition of the liquidity pool such as potentially raising the inflation schedule slightly to counterbalance it.

The advantage of assuming that liquidity provision and APY calculations are non-interfering with each other is that it is possible to completely decouple the liquidity mining program from the smart contract of the protocol. This is correct and attractive from a software engineering perspective.
I think the liquidity mining budget should be allocated from the team wallet, as you said. So, in other words, it's more correct to call it vesting than mining. Additionally, liquidity providers have an incentive to staking by Conjunction Incentives*. I think that would result in a lower APY.
*btw, I find this name unintuitive 😓

*btw, I find this name unintuitive 😓
I like the word Compound Incentives more.
It means to pay (interest) on both the accrued interest and the principal, also it reminds me of COMP :)

Conjunction Incentives(tentative name) calculation is on a per Property basis. I noticed that it couldn't use the amount of liquidity provision in that calculation formula. That is because staking has multiple metrics per person, but liquidity is one metric per person. I already updated the core concept.

I would like to discuss the design of liquidity incentives.

How much liquidity incentive do you think should be granted in total?
I think I'm around 10,000 DEV in total at the moment.
Reason: Because the reward for the challenge project so far has been 10000 DEV.(This is not a very strong reason.)

Do you think it is better to release liquidity incentives in stages or not?
I understand that the benefit of releasing them in stages is to ensure rewards for latecomers.
And if so, what schedule do you think should be followed and how much value should be released?

At the moment, I don't have any particular early join bonus in mind. Do you think that's a good idea?
// 以下、日本語
流動性インセンティブ設計について相談があります。
流動性インセンティブはトータルでどれぐらいを付与すればいいと思いますか？
今の所合計で10000DEVぐらいかなと思っています。
理由：今までのチャレンジ企画の宝珠が10000DEVだったため、なので理由としては薄い
インセンティブを段階的に解放していくか、しないか、どちらがいいと思いますか？
段階的に解放していくメリットは、後発の人のための報酬を確保するためだと理解しています。
やはり段階的に解放していくべきだと思いますか？
またその場合、どのようなスケジュールで解放していくべきだと思いますか？
今の所、特に早期ジョインボーナスは考えていません。それでいいと思いますか？
@aggre @calaber24p

@Akira-Taniguchi The token allocation determines the volume of the liquidity incentive. A placeholder is fine during development.
Can it vest the incentives after the team achieves funding? (Would you describe this as unlocking?)
Geyser will be incentivized for users who provide early liquidity. That specification should be followed.

Can it vest the incentives after the team achieves funding? (Would you describe this as unlocking?)

yes, I'm sure the schedule can go back and forth, so you can run the lockTokens function and deposit the reward tokens after you've raised the funds.

Geyser will be incentivized for users who provide early liquidity. That specification should be followed.

Roger that, I'll make adjustments in that direction.
@aggre


adding feature design

In case of starting to work in feature.html, our team provide new design for this page

Changed images


[Visual Studio Code] VSCodium support

Extension
https://www.raycast.com/thomas/visual-studio-code
Description
VSCodium support would be great!
VSCodium is exactly the same as VSCode but without the telemetry and bloat. It just happens to store its data in differently named directories/files.
Who will benefit from this feature?
All users to prefer VSCodium to VSCode.
Anything else?
No response

@boldandbrad switch to VSCodium in the preferences. This should be already supported.

🤦♂ you are correct. I knew I missed a step between install and ask. Thank you.


Fixup typo

Description:
I have fixed a typo in a CI script output.

BREAKING CHANGE:

Related issue (if exists):

 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.

Can you sign CLA?


Motion model information

Hi,
Firstly let me appreciate your work. Your code is such an elegant one. Unlike other code, you provide the code to extract visual and text features so that it is easier for me to apply your method to my customize datasets and tasks.
Now I'm changing the feature extraction method to improve the performance in my tasks. But I don't know where the motion model ResNeXt-101 is from, which dataset it was pre-trained on and what is the accuracy. So I cannot campare this model with other models directly. If I try them one by one, it will be very time consuming. Could you please tell me some information?
Thanks a lot!

Hi,
Thank you for your interest in our work.
I used the implementation of ResNeXt-101 from https://github.com/kenshohara/video-classification-3d-cnn-pytorch. The ResNeXt-101 is trained on Kinetics dataset with an accuracy of 65.1 and 85.7 over top-1 and top-5, respectively. Please refer the original paper for more details https://arxiv.org/abs/1711.09577.

Thanks a lot！That’s really helpful.
Best wishes！


Remove Tf2 time tutorials for Python

There are currently 4 tutorials having to do with time for tf2:
C++:

https://github.com/ros2/ros2_documentation/blob/5c6c6140d3e111b1ba8a721c937bbd1792eef9ad/source/Tutorials/Intermediate/Tf2/Learning-About-Tf2-And-Time-Cpp.rst
https://github.com/ros2/ros2_documentation/blob/5c6c6140d3e111b1ba8a721c937bbd1792eef9ad/source/Tutorials/Intermediate/Tf2/Time-Travel-With-Tf2-Cpp.rst

Python:

https://github.com/ros2/ros2_documentation/blob/5c6c6140d3e111b1ba8a721c937bbd1792eef9ad/source/Tutorials/Intermediate/Tf2/Learning-About-Tf2-And-Time-Py.rst
https://github.com/ros2/ros2_documentation/blob/5c6c6140d3e111b1ba8a721c937bbd1792eef9ad/source/Tutorials/Intermediate/Tf2/Time-Travel-With-Tf2-Py.rst

The C++ ones work fine, and have worked fine since approximately forever.  This PR does some small cleanups in those tutorials, but otherwise leaves them alone.
The Python ones are a different story.  Due to some bugs in tf2_py itself, those tutorials have never worked in ROS 2.  The correct solution here would be to fix up tf2_py to make them work, but that hasn't yet happened.  Thus, I think it is better to remove those tutorials than to continue to provide them in a non-working state.  So this PR does exactly that.
It also has some other random small fixups throughout the tf2 tutorials.
Edit: this also "fixes" #3960 and #3080 and #2935 , by removing the offending tutorial completely.

I openned an issue https://github.com/ros2/ros2_documentation/issues/4385


fix(ui): message stream interrupt

Message Stream Interrupt
Type: Fix
Steps to Reproduce

Send new chat message (you want it to take a bit to respond so you have time to switch conversations. "Write me a poem about Defense Unicorns"
Switch to a different, already existing conversation on the sidebar before the AI response finishes
Try to send a new message, the send button is stuck in disabled

Expected Behavior
You can send a new message in the other thread immediately
Actual Behavior
You can't send a message
Possible Fix
Use the existing "cancelStream" variable in the conversation store, it may not be being reset properly. Ensure all interrupt edge causes work (network connection issue, browser tab change, etc...)
Environment

Operating System: N/A
Browser/Application: LeapFrogAI UI (lfaiui)
Version: 0.1.1

Additional Information
None
Logs
N/A

Closed with issue #15 in lfaiui repo


Add help text/description to config values

Describe the solution you'd like
OpenZWave provided help text or a description for most device configuration params, it would be nice to see something similar included in the ZWaveJS device config files.

We have that

Doh, I see it now 🙃


New syncing Light Clients will not get notified by FraudProof

If a new light client joins the Network and tries to sync, it will not get notified if a fraud-proof happened in the past.
This is because of 2 things:

Full Nodes halt after Fraudproof and disappear
Full Nodes do not save FraudProofs of previous blocks

A thing we could consider is the full nodes staying and saving the fraudproof for other nodes.
Another thing that could be interesting is to write a fraudproof module for the cosmos-sdk so we can put the fraudproof inside the block and automate the process so the network does not need to halt.  This will require further investigation.

@nashqueue


20x4 lcd oveflow

The overflow seems to be working wrongly for 20x4 LCDs. I have the following code:
let mut lcd = pwr_hd44780::DirectLcd::new(Box::new(lcd_bus), 20, 4)
    .expect("Error creating lcd instance");

But, when I print for example abcdefghijklmnopqrstuvxyz abcdefghijklmnopqrstuvxyz abcdefghijklmnopqrstuvxyz, it appears on my lcd as:
abcdefghijklmnopqrst
opqrstuvxyz abcdefgh
uvxyz abcdefghijklmn
ijklmnopqrstuvxyz

It is overflowing in the order of first line -> third line -> second line -> fourth line, instead of first line -> second line -> third line -> fourth line. What do I do?

Hi,
Overflowing is handled by the LCD's chip itself (i.e. the HD44780 controller) and, sadly, that's how it works.
The exact same issue has been described here, for instance: https://forum.arduino.cc/index.php?topic=14140.0
This problem should not happen when using the buffered version though - can you please check it?
https://github.com/Patryk27/pwr-hd44780/blob/master/examples/frontends/buffered.rs - here's an example.
Please let me know if you encounter any more trouble.

@Patryk27 oh! okay. So, if I want to specifically use the DirectLcd, then is there any way in which I can write a function which takes the first 20 characters, puts it on the first line, takes the next 20 characters (if any) and puts it on the second line, etc?
I tried the buffered version, and it works! What exactly are the differences between DirectLcd and BufferedLcd? Why should I chose any one over the other?
Also, thanks a lot for helping me - since my original issue is now solved? Could you close the issue?

@Partyk27 that's great! Is there any way I can contact you personally, like through email or something? I can't find it on your GitHub profile... Can you send me an email to the address in my profile?


Add knife_log method. Also add cut-eat for any 3t method

Update to Dan's plugin. Allows for additional option to give free cooking xp. Also added Knife log 3t method as well.

Looks good to me! Thanks for your contribution!


Private endpoint support for Azure CosmosDB with MongoDB api

Thanks for your contribution @tanjoht
This has now been released in v1.1.0-rc.1!


Request to ignore source root folder for excludes

Presently the app will use the exclusion filter to weed out folders like tests, lib and others to avoid false reporting as features but if the root folder is include in the filters it is affected also.  The suggestion is to ignore the root source folder and only apply the filtering for subfolders which is a good compromise.

What this will do is allow you to have a root directory to scan that has the name "test", or similar that would have previously caused any files therein to be excluded from being scanned.  The solution will now ignore any files in the root scan folder but continue to exclude any matches in subfolders beneath the root path provided.


Update dependencies

This PR updates project dependencies and closes #427

:tada: This PR is included in version 10.3.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


bump to 1.47.2 and tweak

@VeraZab if you're OK with this, can you please merge, release, and append this version of RCE to https://github.com/plotly/streambed/pull/12878 ?

💃


[KBFS-1097] Change bserverTlfStorage to bserverTlfJournal

Store operations in a journal instead of an on-disk
reference map.

Looks good modulo comments, thanks!


Getting EACCES: permission denied error on electronize start on MacOS

Version:
Latest nugets: 5.22.14
Dot net core: 2.2.300
Node: 10.15.00
npm: 6.9.0

Target: MacOS

Steps to Reproduce:

Follow readme and video
electronize init worked fine
run electronize start

electron.net git:(master) ✗ electronize start
Start Electron Desktop Application...
Microsoft (R) Build Engine version 16.1.76+g14b0a930a7 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
  Restore completed in 1.68 sec for /Users/x/Documents/git/electron.net/electron.net.csproj.
  electron.net -> /Users/x/Documents/git/electron.net/bin/Debug/netcoreapp2.2/osx-x64/electron.net.dll
  electron.net -> /Users/x/Documents/git/electron.net/bin/Debug/netcoreapp2.2/osx-x64/electron.net.Views.dll
  electron.net -> /Users/x/Documents/git/electron.net/obj/Host/bin/
node_modules missing in: /Users/x/Documents/git/electron.net/obj/Host/node_modules
Start npm install...
npm WARN checkPermissions Missing write access to /Users/x/Documents/git/electron.net/obj/Host/node_modules
npm ERR! path /Users/x/Documents/git/electron.net/obj/Host/node_modules
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall access
npm ERR! Error: EACCES: permission denied, access '/Users/x/Documents/git/electron.net/obj/Host/node_modules'
npm ERR!  { [Error: EACCES: permission denied, access '/Users/x/Documents/git/electron.net/obj/Host/node_modules']
npm ERR!   stack:
npm ERR!    'Error: EACCES: permission denied, access \'/Users/x/Documents/git/electron.net/obj/Host/node_modules\'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path:
npm ERR!    '/Users/x/Documents/git/electron.net/obj/Host/node_modules' }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).
npm ERR!      /Users/x/.npm/_logs/2019-09-18T18_43_00_745Z-debug.log
ElectronHostHook handling started...
Invoke electron - in dir: /Users/x/Documents/git/electron.net/obj/Host/node_modules/.bin
bash: line 1: ./electron: No such file or directory

I also tried it via sudo electronize start but it was the same...

Hey @mika76, where did you install node & npm from? Did you use homebrew? I am concerned that maybe you have somewhere in your chain used sudo to install (maybe node/npm, maybe nvm or some version manager).
Additionally, based on this particular error, it looks like it is the dotnet-generated folders that have permissions problems. EACCES error came from obj/Host/node_modules, so either the permissions are messed up on node_modules dir, or one of the ones up the chain.
You should be able to use ls -lash on those folders and see what the actual permissions are, vs. what they should be. Maybe you will see one of them has root instead of your your username?

Hey @netpoetica, thanks for taking a look.
You're right, all the folders in obj/Host/node_modules are owned by root. As for npm, I honestly don't remember - it might have been through brew.
How can I fix this? re-install node and npm manually or something?

doing a brew info npm results in
➜  node_modules git:(master) brew info npm
node: stable 12.11.1 (bottled), HEAD
Platform built on V8 to build network applications
https://nodejs.org/
/usr/local/Cellar/node/7.6.0 (3,148 files, 40MB)
  Poured from bottle on 2017-02-23 at 11:47:36
/usr/local/Cellar/node/7.7.1 (3,148 files, 40.2MB)
  Poured from bottle on 2017-03-04 at 09:07:49
/usr/local/Cellar/node/8.2.1 (4,151 files, 45.9MB)
  Poured from bottle on 2017-08-08 at 16:08:51
/usr/local/Cellar/node/9.5.0 (5,125 files, 49.7MB)
  Poured from bottle on 2018-02-08 at 15:52:54
/usr/local/Cellar/node/10.3.0 (6,793 files, 59.3MB)
  Poured from bottle on 2018-05-30 at 21:15:57
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/node.rb
==> Dependencies
Build: pkg-config ✔, python@2 ✘
Required: icu4c ✘
==> Options
--HEAD
	Install HEAD version
==> Caveats
Bash completion has been installed to:
  /usr/local/etc/bash_completion.d
==> Analytics
install: 307,659 (30 days), 1,023,849 (90 days), 4,132,492 (365 days)
install_on_request: 218,337 (30 days), 741,076 (90 days), 3,054,076 (365 days)
build_error: 0 (30 days)

It is possible that you installed homebrew using sudo, which then causes you to need to use sudo when brew installing, at which point npm thinks you're a root user or something.
In this particular case, you can just chown -R those directories to your use instead.
In the long run, I recommend uninstalling node & npm completely, maybe uninstalling homebrew even, just making sure that you don't have any of that stuff installed with sudo. You may be able to find some online resources about changing all of these permissions without re-installing, but I personally am worried you might be setting yourself up for many more similar such issues in the future.

Thanks @netpoetica I'll attempt to clean it all up

I guess this issue can be closed also, since it's something on my machine... Thanks for all your help.

Just as an update if anybody hit's the same issue a me - I used the following article to uninstall npm and install it using nvm which manages node and npm versions better than things like homebrew: https://ajaykarwal.com/uninstall-node-and-install-nvm/
Works like a dream now...


单机多卡进行RLHF在第三步中使用Qwen模型作Actor Model报错

Hi, I'm also using deepspeedchat for RLHF training qwen, did you solve this problem?

^


The login form on the path(/mentors/login button) not visible and accessible properly.

To Reproduce
Steps to reproduce the behavior:

Go to 'home page '
Click on 'mentors'
Then Click on 'find a mentor'
Click on 'login'
See error

OS: windows 11
Browser: chrome

Hi @hesh-ftw are you working on this issue?

Hi @hesh-ftw, Can you try this again seems to be fixed.

@hesh-ftw Thank you for pointing out this issue.


Update for <IP_ADDRESS>

world file version update, only relevant change

gotta love that file version only change :-/

Thanks.. the needless map version changes are so irritating.


Rake/MethodDefinitionInTask is 66.6% incorrect

The good code is bad, and the bad code is good.
Methods should be defined inside tasks (only)
(but not namespaces), as it is the only way they will not be globally scoped, and thereby silently conflict with all other same-named globally scoped methods.
Obviously it is also fine to define methods in classes or modules... that's outside the scope of this issue.

definition location
scope
rubocop-rake should consider
rubocop-rake currently considers

top-level
global
bad 🛑
good ✅

namespace block
global
bad 🛑
bad 🛑

task block
task
good ✅
bad 🛑

"Good Code" example for task:
namespace :foo_namespace do
  desc 'foo task'
  task :foo do
    def greet
      puts 'Hello from foo.rake'
    end

    puts 'In foo task'
    greet
  end
end

"Bad Code" example for namespace:
namespace :foo_namespace do
  desc 'foo task'
  task :foo do
    puts 'In foo task'
    greet
  end

  def greet
    puts 'Hello from foo.rake'
  end
end

"Bad Code" example for top-level:
desc 'foo task'
task :foo do
  puts 'In foo task'
  greet
end

def greet
  puts 'Hello from foo.rake'
end

reference: https://www.albertoalmagro.com/en/ruby-methods-defined-in-rake-tasks/

I guess it is definitely correct. Maybe an additional NoGlobalMethodsInRake cop is needed ?

I don't think a new cop is needed, the current cop, Rake/MethodDefinitionInTask just needs to be fixed.

I don't think a new cop is needed, the current cop, Rake/MethodDefinitionInTask just needs to be fixed.

oh, you are right, sorry. Indeed a method defined inside task do... does not pollute global


test: Cypress | (DI) GrpahQL added validations

Description

FillUnAuthenticatedGraphQLDSForm(), CreateDataSource() updated
GraphQL added tc to validate empty body
Graph QL - Save as DS
Graph QL - Edit DS
Found & logged bug #26873
Binary spec - long running case - skipped

Type of change

Script fix (non-breaking change which fixes an issue)

Testing

How Has This Been Tested?

[X] Cypress

Checklist:
QA activity:

[X] Added Test Plan Approved label after Cypress tests were reviewed

/ci-test-limit runId=6049269436

/ci-test-limit runId=6049269436

/ok-to-test sha=1a59182


DMA Support for the LPC55S3X Series

Added DMA Support for the LPC55S3X Series.

@EmilioCBen, was this tested with the dma tests under tests/drivers/dma/loop_transfer/ and tests/drivers/dma/scatter_gather

@EmilioCBen, was this tested with the dma tests under tests/drivers/dma/loop_transfer/ and tests/drivers/dma/scatter_gather

This has not yet been tested with loop or scatter, I can work on this

Please add overlay and test with the DMA tests loop_transfer and scatter_gather

I have made an overlay file for both of these tests now and they both pass, thank you.

the twister failures are because you have not provided peripherals with DMA request numbers with the dmas phandles property in devicetree

@EmilioCBen Please resolve the conflicts,

@EmilioCBen Please resolve the conflicts,

Got I have rebased and fixed the conflicts


Sina Weibo Api has a exception

run SinaWeiboExample
i got the authorization code,then paste. it's happend.
Trading the Request Token for an Access Token...
Exception in thread "Main Thread" org.scribe.exceptions.OAuthException: Response body is incorrect. Can't extract token and secret from this: 'request=%2Foauth%2Faccess_token&error_code=401&error=40112%3AOauth+Error%3A+token_revoked%21&error_CN=%E9%94%99%E8%AF%AF%3AToken%E4%B8%8D%E5%90%88%E6%B3%95%21'
at org.scribe.extractors.TokenExtractorImpl.extract(TokenExtractorImpl.java:41)
at org.scribe.extractors.TokenExtractorImpl.extract(TokenExtractorImpl.java:27)
at org.scribe.oauth.OAuth10aServiceImpl.getAccessToken(OAuth10aServiceImpl.java:81)
at org.scribe.examples.SinaWeiboExample.main(SinaWeiboExample.java:49)

Seems the access token got revoked? (check the error message)
On Sun, Nov 20, 2011 at 1:52 PM, fivesmallq <
<EMAIL_ADDRESS>
wrote:

run SinaWeiboExample
i got the authorization code,then paste. it's happend.
Trading the Request Token for an Access Token...
Exception in thread "Main Thread" org.scribe.exceptions.OAuthException:
Response body is incorrect. Can't extract token and secret from this:
'request=%2Foauth%2Faccess_token&error_code=401&error=40112%3AOauth+Error%3A+token_revoked%21&error_CN=%E9%94%99%E8%AF%AF%3AToken%E4%B8%8D%E5%90%88%E6%B3%95%21'
at
org.scribe.extractors.TokenExtractorImpl.extract(TokenExtractorImpl.java:41)
at
org.scribe.extractors.TokenExtractorImpl.extract(TokenExtractorImpl.java:27)
at
org.scribe.oauth.OAuth10aServiceImpl.getAccessToken(OAuth10aServiceImpl.java:81)
at
org.scribe.examples.SinaWeiboExample.main(SinaWeiboExample.java:49)

Reply to this email directly or view it on GitHub:
https://github.com/fernandezpablo85/scribe-java/issues/193

I don't know japanese (or Korean, or Chinese, whatever the language on weibo is) so I can't manage to get an api key and secret to try it myself. Do you mind getting a pair for me?

ok,
App Key：2507380164
App Secret：13dbc366bae21ffda3734a1bc1ac9891
if you need username and password you can send a email to me<EMAIL_ADDRESS>


Error on import/require

TypeError: Cannot read property 'OP_RETURN' of undefined at Object. (/workspace/node_modules/txforge/dist/txforge.cjs.js:354:18) at Module._compile (internal/modules/cjs/loader.js:1133:30) at Object.Module._extensions..js (internal/modules/cjs/loader.js:1153:10) at Module.load (internal/modules/cjs/loader.js:977:32) at Function.Module._load (internal/modules/cjs/loader.js:877:14) at Module.require (internal/modules/cjs/loader.js:1019:19) at require (internal/modules/cjs/helpers.js:77:18) at Object. (/workspace/hybrid-server.js:35:19) at Module._compile (internal/modules/cjs/loader.js:1133:30) at Object.Module._extensions..js (internal/modules/cjs/loader.js:1153:10)
npm version 0.1.7

Seems the issue was related to the wrong version of bsv being required.
I'm gonna keep this issue open as a reminder to handle this with some kind of helpful error message as I don't think you'll be the last to do this :)


Time for a V9 STV xml

Thought I would start the discussion on building a new V9 STV.  Comments welcome.  This is likely a lengthy project but perhaps it's time.
Some first thoughts...

modular - base config would be external and loaded on demand.  Perhaps things like the guide may be external as well but always loaded...then other UI's could simply theme it and use it too.
base config should prompt for what UI the user wants to load and offer all Full UI plugins including the new V9 STV (phoenix 3, gemstone, STV3, etc).
use phoenix core services...I would like to see this become part of the sage core...why reinvent the wheel
dynamic menu built in...it's time that all users have a simple interface to edit the menu...while having a base default for users who don't want to "mess with it"
new theme(s) - this is a place for others with graphics experience to jump in as this is NOT my forte
built in properties "management" including export/import that could handle path management

...add your own thoughts...

I do like the idea of a modular stv that is composed of many xmls and loaded on demand.  I'd be ok if guide was considered a "module" as well, since I can't remember the last time I actually used the sagetv guide :)

I think there's a lot in the phoenix API that could be built a lot simpler if much of those things were from scratch.  A lot of what the phoenix core used to do is not really needed anymore, since pretty much everything is available via clean API's now from the likes of tvdb and themoviedb, instead of the page scraping of old.  I do think that fanart could be made a part of the standard database system.
Since we have access to the sagetv core, I'd think metadata and fanart lookups could be done much 'cleaner', even as part of the EPG lookup process.  We could actually implement a system by which EPG data could be passed through a plugin on it's way to the database.  This would allow guide data to automatically include the 'better' data from tvdb, for instance - if someone decides to install the 'tvdb lookup' plugin.  I can see a number of metadata plugins that might be nice.  A sports one that would generate match specific fanart from the team logos for instance.  Having a routed plugin that could pass each program through each individual plugin in a ordered list could provide a lot of flexibility, without a lot of actual work int he core.

@CraziFuzzy I get what you are saying regarding EPG, but, I really don't think we want to put in a blocking operation of looking up metadata during the passthrough the database.  I can see triggering a metadata lookup, in an async manner, if you access an item from the EPG, but not during a write.  My EPG probably has 1000s of shows and yet I only care about 20-30 of them, so doing lookups on 1000s is unnecessary, and would likely result in many failed attempts (we already get kicked from the site due to too many requests).
And I agree that the phoenix metadata/fanart can be simplified.  It was designed to allow for multiple fanart and metadata sources and over the years it has supported imdb, dvdprofiler, mymovies, tvdb, themoviedb, and while tvdb and themoviedv remain, it's only because they were the most widely used.  Even if I were to rewrite it, there would still be an abstraction layer that would allow tvdb or themoviedb to be used, but i would never depend directly on those.  Phoenix also supports the notion that "fanart" can be anywhere (ie, central, local, or possibly in movie folders, etc).  That could likely all go away, if we re-do how we store fanart and agree that it's going to be done only in a single way, forever.
Phoenix was intended to be a set of APIs where the implementations could vary.  So even today, things like weather, fanart, metadata, and how we store and access fanart are all modular and be swapped in and out (and over the years, many of the implementations of those things have changed, while the APIs and interfaces remained the same).
Having written all this stuff once, i can say, it's not a trivial process to reinvent everything.  So, we are left with a hard decision.  do we re-invent it, or do we try to leverage something that exists.  The problem with phoenix, like sagetv, is that it evolved over time, and there are things in there that are not done as well as it should be because it was bolted on.  I think phoenix needs to be separated out into multiple jars, like, phoenix-core, phoenix-util, phoenix-menus, phoenix-weather, phoenix-vfs, phoenix-props, etc, where they could cherry picked and used as needed.

I look at the phoenix integration as a necessity as if i was to take on all the work for the things phoenix already provides, just to create a v9 STV, then i would likely not bother to start.  Something like the menu system greatly simplifies the entire process and at the same time would reduce my learning curve as i have already become familiar.  Other parts of phoenix could be changed over time and as stuckless states the fanart area could use some simplifying, but that can occur while a new UI is built as well as into the future...but adopting phoenix as being needed for the core STV is key for me or others would need to join in and replace those features into the core as the STV is written...but that is reinventing and i see enough work without that.

I will leave the architecture to those who know more than I, but do agree an improved modular design would be a better approach.
I am a big fan of the simple design of V7.  Anyone who comes by has no issue using it.  I would like to suggest that the default be similar in it's simplicity at the current default presentation level.  (To me, Kodi is a nav nightmare.)
I would like to retain ALL the basic TV, Video, Music, Photos, Online capabilities, though I realize many would opt to not want to use them all (disable).  The modular approach would allow those with interest focus on these individually.
Not clear on how one can function with no EPG, but then I prefer lists to all the fan art.  To much scrolling for me.  I personally think graphics are only useful if you recognize them.  That said, bling sells, and you need to have it.  I would still like to retain the old codger menu option.
Music is the big hole for me.  I have over 1000 ripped CD's in FLAC (and other formats) done with dbPowerAmp (best music tool eva!).  I do not want to leave Sage.  There are lots of opportunities for improvement (airplay for me) but at least add a continuous play option to the current player.  Happy with basic cover art as the display (I lied, I do like the older iTune's album flipper circa V10.7 for selection).  I'm sure some would want lyrics.
I can see that organizing media is not black and white (no pun intended).  TV can come from DVD or recordings, same for Movies, Music can come form Concert DVDs, CDs, downloads (DRM issues I'm sure) or Online Sources like Pandora and Spotify.  So classification or attribute management probably needs to be available for all objects.
Hopefully what ever tech is used will also support not only playing media on portable devices, but support control from these (phones in particular).  Maybe pushing it here.


Graph Vehicle Statistics

Firstly, thank you for an awesome add to Home Assistant.  I've been playing with this, and thoroughly enjoy it.  I have a suggestion for an improvement.  As someone who has high traffic in front of their house, it would be interesting to see statistics on vehicles that pass by.  For example, right now in my events I can see the following data:
{"entity_id": "image_processing.sighthound_garage_2", "plate": "JDJ7182", "vehicle_type": "car", "make": "Mercedes-Benz", "model": "CLS-Class", "color": "black", "region": "Mississippi", "bounding_box": [0.13503, 0.20703, 0.47454, 0.47266]}
It would be great to gather statistics on how many Ford, Mercedes-Benz, etc... the system recognizes over time.  The same for color, region, model, etc...
At this time, the only method I can identify is a database query.

Database query is what I recommend, unless you can propose an alternative? You could create a SQL sensor to return the metrics you want

Firstly, thank you for an awesome add to Home Assistant. I've been playing with this, and thoroughly enjoy it. I have a suggestion for an improvement. As someone who has high traffic in front of their house, it would be interesting to see statistics on vehicles that pass by. For example, right now in my events I can see the following data:
{"entity_id": "image_processing.sighthound_garage_2", "plate": "JDJ7182", "vehicle_type": "car", "make": "Mercedes-Benz", "model": "CLS-Class", "color": "black", "region": "Mississippi", "bounding_box": [0.13503, 0.20703, 0.47454, 0.47266]}
It would be great to gather statistics on how many Ford, Mercedes-Benz, etc... the system recognizes over time. The same for color, region, model, etc...
At this time, the only method I can identify is a database query.

How did you get the vechicle make and model to show up? When I run Image processing: scan all I get is the following in attributes
last_vehicle: 2021-05-12_16:46:17
plates: unknown, unknown
unit_of_measurement: vehicle
friendly_name: sighthound_vehicle_driveway

How about adding the saved file-path for the processed image in the event data, so that one could also query the image?

Honestly I just dont have free time for much open source work these days, so new features will require a PR


Issue with pass backend?

Not sure if this is an issue with the pass backend or in my not having things configured correctly.
I have added my okta info via
aws-okta add -b pass
with no issues but when attempting to exec:
 aws-okta -d exec okta-develop -- aws sts get-caller-identity
DEBU[0000] Parsing config file /home/rdyer/.aws/config
DEBU[0000] Using KrItemPerSessionStore
DEBU[0000] cache get `okta-develop session<PHONE_NUMBER>6434313162)`: miss (read error): The collection "awsvault" does not exist. Please add a key first
DEBU[0000] Using aws_saml_url from profile: okta
DEBU[0000] using okta provider
DEBU[0000] couldnt get okta creds from keyring: The collection "awsvault" does not exist. Please add a key first
getting creds via SAML: The collection "awsvault" does not exist. Please add a key first

Any ideas?  Thanks

I'm pretty sure you'll need to pass -b pass to every command; that's not persisted anywhere.

That was it, thanks.  Theres currently not a way to add that to a config file as default?

That was it, thanks. Theres currently not a way to add that to a config file as default?

There isn't. Maybe a line in the config file would be a good feature. But in the meantime, I'd recommend a shell alias alias aws-okta='aws-okta -b pass'


Use lz-string to compress benchmarks.html raw data

Page size drops from ~4.5MB to ~0.7MB, as most of the page is commit-related testrunner data.
Cool library with nice license: https://github.com/pieroxy/lz-string.

Haha, that license is awesome.  It's like public domain with attitude! :)


Plugin not compatible with 232.* builds

Not sure about Intellij, but the latest RubyMine 2023.2 EAP has a build number of 232.5150.113. The current comaptibility range for the latest release of the Protobuf plugin is 231—231.*.
For now I'm just not using the EAP to continue using this plugin, but I assume that an eventual 2023.2 release would similarly be incompatible.

When version 232 is released, I'll check for compatibility and target the plugin to 232.

IntelliJ IDEA 2023.2 Is Out! July 26, 2023

Updating for it

Plz check the #189
I will release it in next two or three days

Fixed by v1.7.13


Refactor addToken custom graphql mutation (in the target/bitbucket branch)

To be a bit more generic, and not require a username (since that's been moved into the JSON of the credential). See here for the current implementation.
Here's what I'd propose:

[ ] Rename the call to addCredential (since a credential may not be a token)
[ ] Refactor add_service_auth_credential PG UDF to not take username as an argument, and instead consider storing encrypted JSON in the credentials column, as the encrypted string

Closing this out as no longer relevant for now. We can refactor/rename later if needed.


⚠ Bofrost.fr has degraded performance

In c22ecbe, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1470 ms

Resolved: Bofrost.fr performance has improved in 4115c39 after 6 minutes.


Tidying and polishing of  the DC/OS and Spark demo

Check for installed SimDem when running any script - failing gracefully when necessary
Change the DNS name of the DC/OS cluster each time it is created to minimize failures due to DNS conflict
Increase the size of the DC/OS demo cluster
Fine tune the demo script to make it flow nicely
Add a clenup.sh script to remove all resources created in the demo

@rgardler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


QUANTIZE and AUTO buttons don't work as expected

Quantize and toggling automation recording on/off isn't supported yet on the actual MIDI API scope. There's a workaround to get to the quantize buttons by getting into the menus and emulating key pressings but it's slow and buggy. Instead, they switch between windows (browser isn't included on this) and trigger full-screen plugin browser respectively.

Quantization seemed to be implemented back in API version 9 using channels.quickQuantize(), but not documented until recently in the official MIDI scripting API reference. Implemented in the script on c5cf3287d09ea2b547548f49bfa7ccfeaec608b8.


New constructors (WIP)

@olydis,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

closed in favor of https://github.com/Azure/autorest.csharp/pull/16


IGNITE-9913

Signed-off-by: Anton Vinogradov<EMAIL_ADDRESS>

continued at https://github.com/apache/ignite/pull/7069


Check if two trees are Mirror in C++

Assume Two Binary Trees, write a function that returns true if two trees are mirror of each other, else false
Have you read the [Contributing Guidelines on Pull Requests]
Yes, I've read the guidelines.
As a GSSOC21 Participant , I would love to work on this issue.
@ankitaggarwal23  Please assign me this issue

@ankitaggarwal23 Thank You so much for assigning me.

@ankitaggarwal23 have created a pull request for the Check if two tree are Mirror. Please review the code and merge it if you find it good.

@ankitaggarwal23 have created a pull request for this issue. Please review the code and merge it if you find it good.

@ankitaggarwal23  have created a pull request for this issue. Please review the code and merge it if you find it good.

@ankitaggarwal23 why my request has been closed.

@rk05-lenavo Your PR is closed because You didn't link your issue and also marked in the description here.


Improve test coverage in load pkg

Most uncovered code now is mostly call outs to functions covered
by tests or a bit difficult to test for now.

Codecov Report

Merging #15 into master will increase coverage by 1.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   43.35%   44.48%   +1.13%
==========================================
  Files          61       61
  Lines        2558     2558
==========================================
+ Hits         1109     1138      +29
+ Misses       1432     1403      -29
  Partials       17       17

Impacted Files
Coverage Δ

load/loader.go
70.43% <0%> (+25.21%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ad51dbb...5e8019c. Read the comment docs.


Dec multiply is not reporting overflow

From the following repl output:
» Num.mulSaturated 170_141_183_460_469_231_731dec 2

-1.374607431768211456 : Dec
» Num.mulChecked 170_141_183_460_469_231_731dec 2

Ok -1.374607431768211456 : Result Dec [Overflow]
» Num.mul 170_141_183_460_469_231_731dec 2

-1.374607431768211456 : Dec

I would expect all three to indicate overflow either via panic, Err or returning Dec maximum value, but instead they all return a sort of wrapped multiplication.
This also shows up outside the repl:
app [main] { pf: platform "https://github.com/roc-lang/basic-cli/releases/download/0.17.0/lZFLstMUCUvd5bjnnpYromZJXkQUrdhbva4xdBInicE.tar.br" }

import pf.Stdout

main =
    Stdout.line! (Num.toStr (Num.mul 170_141_183_460_469_231_731dec 2))

# output:
# -1.374607431768211456

Which indicates this is not specific to the gen-dev backend but rather the zig bitcode implementation. mulWithOverflow is the place to look: https://github.com/roc-lang/roc/blob/0e550a7f68f53d8699d484a426cd4963e0110539/crates/compiler/builtins/bitcode/src/dec.zig#L310
Found as part of work on #7264

I have a fix on the dec-mul branch. Just waiting on https://github.com/roc-lang/roc/pull/7264 to land before I make a PR.

@bhansconnect that PR has merged, in case you missed it.


Outputting Scan ID -- is that currently being set?

Hi all,
I am working with a customer that is using this action to orchestrate Ostorlab scans against their mobile applications. One thing that I am working with them on is around being able to poll for results based off a scan ID.
I noticed in the action.yaml in your guys repository, contains the following:
outputs:
  scan_id:
    description: id of the created scan.
  status:
    description: status of scan.

However, nowhere in the Dockerfile backend do I see any setting of said output. Is this something that can be done relatively quickly? I am more than happy to help implement that 😄

Hi @ssulei7 ,
@mohsinenar can you please correct my understanding, the output to github actions is handled by the log-flavor class of Github with the output method.
I see that it is used in the asset click action passing the scan_id: https://github.com/Ostorlab/ostorlab/blob/30fd51fac136b1c25ad095a703e346d20f3487a5/src/ostorlab/cli/ci_scan/run/assets/mobile.py#L29
@ssulei7 how did you come to the conclusion that it is not implemented?

@3asm ah! My apologies, I did not notice that the CLI module being used in the Dockerfile had utilities to set the output for the action 😅 . I will be closing out this issue then!


README Update

Better description in the README would be awesome, or a demo site to see and feel the components :)
Thanks for making them :+1:

It's hidden in the description rather than the README, but the demo site is at http://khan.github.io/react-components/ . I'll add that link to the readme!


fixed zIndex for blanketCSS and positionerCSS

Description of changes:
fixes zIndex for blanketCSS and positionerCSS
Related issues (if any):
Checks:

[ ] Please confirm yarn run lint ran successfully
[ ] Please confirm that only /src and /examples/src are committed
[ ] [if new feature] Please confirm that documentation was added to the README.md

Thanks for the PR! I really appreciate the help on this project.


Statuspage.io webhook integration

I'm doing the documentation, not coding

Okay, cool !!!

@eeshangarg Done changes here also :)

@eeshangarg Done changes here...Please take a look :)

Merged, thanks @Rishabh570 and @eeshangarg!  I made a small tweak to the documentation text when merging to make it a bit more clear what the difference was between the two ways to setup notifications on the statuspage.io site (depending whether it's your own product's status page).  It may be worth further work to make it even clearer (e.g. using bold or headings or something).


add_null_constraint fails with undefined method `match?' for /^".*"$/:Regexp

With ActiveRecord 4.2.10, when running this line in a migration:
    add_null_constraint :ews_students, :first_name, if: "last_name IS NULL"

... the following occurs:
undefined method `match?' for /^".*"$/:Regexp
Did you mean?  match
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/rein-3.5.0/lib/rein/util.rb:17:in `wrap_identifier'
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/rein-3.5.0/lib/rein/constraint/null.rb:27:in `_add_null_constraint'
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/rein-3.5.0/lib/rein/constraint/null.rb:11:in `block (2 levels) in add_null_constraint'
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/activerecord-4.2.10/lib/active_record/migration.rb:511:in `up'
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/rein-3.5.0/lib/rein/constraint/null.rb:11:in `block in add_null_constraint'
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/activerecord-4.2.10/lib/active_record/migration.rb:547:in `block in reversible'
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/activerecord-4.2.10/lib/active_record/migration.rb:746:in `execute_block'
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/activerecord-4.2.10/lib/active_record/migration.rb:547:in `reversible'
/Users/aaron/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/rein-3.5.0/lib/rein/constraint/null.rb:10:in `add_null_constraint'
/Users/aaron/git/clarity_early_warning/db/migrate/20180920222205_nullable_first_name.rb:5:in `up'

having the same issue with add_inclusion_constraint


Column handle

Codecov Report

Merging #1745 (44daa74) into columnar3 (c57fa3f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           columnar3    #1745   +/-   ##
==========================================
  Coverage      94.04%   94.05%
==========================================
  Files            278      279    +1
  Lines          52263    52289   +26
==========================================
+ Hits           49153    49178   +25
- Misses          3110     3111    +1

Impacted Files
Coverage Δ

common/src/file_slice.rs
81.42% <ø> (ø)

columnar/src/lib.rs
98.27% <100.00%> (+0.12%)
:arrow_up:

columnar/src/reader/column_handle.rs
100.00% <100.00%> (ø)

columnar/src/reader/mod.rs
68.75% <100.00%> (+0.80%)
:arrow_up:

fastfield_codecs/src/column.rs
93.68% <100.00%> (ø)

src/fastfield/multivalued/mod.rs
97.29% <0.00%> (-1.16%)
:arrow_down:

src/schema/schema.rs
98.91% <0.00%> (+0.13%)
:arrow_up:

src/query/boolean_query/block_wand.rs
97.06% <0.00%> (+0.20%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Give user a mnemonic code to backup

When user generates a new account, give him a mnemonic code. Should be implemented and provided by emerald-rs
https://github.com/bitcoin/bips/blob/master/bip-0039.mediawiki
Show it in console (emerald new) and in JSON response

https://github.com/tritone11/ethereum-rust-utils/issues/1


Issue/5102 docs dot installed

Description
Check in the docs/conf.py whether dot is available.
closes #5102
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[x] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

@sanderr Do you have an idea how I can test this?

Processing this pull request

Merged into branches master in e14bc1ff117d03676f13fc97c9558b0000a94d6d

Processing #6269.

Processing #6268.


Next release: text to be changed

https://github.com/rapidstack/RESTEasyCLI/blob/e3671145aced84f97b636bce20d82122fd21b743/resteasycli/lib/abstract_finder.py#L30
Should be "searching for file: filepath"

Raise the pull request

Also, for future modifications, instead of creating new branch, it's better to fork this repo as personal repository and create pull request from it. This way this repo will be cleaner.

Ok, Raised pull request.


Database Backend for Bookmarks

I saw that you were forking 'Export Tabs URLs' in some capacity, and I wanted to submit a request that someone develop a bookmarks addon (for Firefox) that could directly write bookmarks (along with MyTitle, MyDescription, Tags and Categories) into a RDBMS. Ideally, I could choose to use MSSQL.
I've never built an addon, and I don't know where to begin. But I really want that functionality. I have thousands of bookmarks. I want to be able to munge the content in a relational database. To ME that means looking at SQLdom.sourceforge.net and parsing the HTML and storing it in MSSQL.
But I'm a database guy. Everything to me needs to be solved with MSSQL (mainly because of the FullTextSearch capabilities)

I wanted to submit a request that someone develop a bookmarks addon (for Firefox) that could directly write bookmarks (along with MyTitle, MyDescription, Tags and Categories) into a RDBMS.

Addons need an external helper application to communicate with other applications (like RDBMS) which dont have a built-in webserver/http interface.


SyncTrayzor not opening on Start Up and not opening from Start Menu

Hello,
I noticed that SyncTrayzor wasn't open, so I went to open it from the Start Menu. I got the below error.

Is there anything you can do to help with this, please?
Thank you.

This happened quite a lot many years ago, and the nearest we got was that windows update was corrupting SyncTrayzor's configuration file, which lead to those instructions being added at the bottom of the dialog. You're best off deleting the file it names, which will lose your SyncTrayzor (not Syncthing) settings.


Bridge UI : Form should support deeplinking URL param amount for a better UX

Objective
For a seamless user experience, coming to https://bridge.connext.network from other apps, most of the form data should be prefilled from the URL itself. eg. https://bridge.connext.network/ETH-from-ethereum-to-arbitrum?amount=0.11
Reproduce
URL: https://bridge.connext.network/ETH-from-ethereum-to-arbitrum?amount=0.11
The amount is being passed in URL but still the field is empty when the form loads.

Code
In the code: https://github.com/CoinHippo-Labs/connext-bridge/blob/main/components/crosschain-bridge/index.js, we are extracting query.amount from the URL using get query params. However, this value is not being set automatically in the form during any point in the render cycle.

we've fixed it. thanks.


WIP: Add Swift Package Manager support

On this PR

SPM now is used as a primary method to build and test the project;
(I believe) Fixed testRetainCycle;
Upgraded Travis CI Xcode and simulator version;
Added new test devices:

iPhone 6, OS=10.3
iPhone 7, OS=11.4
iPhone X, OS=12.4
iPhone 11 Pro, OS=13.3
Apple TV, OS=13.3
macOS 10.14

Renamed to WIP until I try to figure out a solution for the failing tests.

This EXC_BAD_ACCESS seems to be because the stack overflows. It looks like dispatch threads have limited stack size: https://forums.swift.org/t/increase-size-of-stack/14493/14. When it crashes for me, the stack is 3000 frames long which doesn't sounds that long...but definitely seems long enough? But yeah, I'm not sure this is the problem. How does it fail when it fails remotely?
On Mon, Jan 13, 2020 at 8:02 AM, Heberti Almeida <<EMAIL_ADDRESS>> wrote:

@hebertialmeida commented on this pull request.
In ConsistencyManagerTests/ ListenerTests. swift (
https://github.com/plivesey/ConsistencyManager-iOS/pull/61#discussion_r365884675
) :

@@ -117,6 +117,7 @@ class ListenerTests: ConsistencyManagerTestCase {
listener = strongListener addListener(strongListener,
toConsistencyManager: consistencyManager) +
waitOnDispatchQueue(consistencyManager)

If I increase it from 100 to 2000 it somehow happens locally:
image (
https://user-images.githubusercontent.com/103670/72270969-05927c80-35f4-11ea-92b8-e4ab568341b3.png
)
But then it is a EXC_BAD_ACCESS
image (
https://user-images.githubusercontent.com/103670/72270909-ea277180-35f3-11ea-83f5-585909a1fba6.png
)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
https://github.com/plivesey/ConsistencyManager-iOS/pull/61?email_source=notifications&email_token=AAJQ2TIEHYYYV63RJNISRQTQ5SGCDA5CNFSM4KFL5OY2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCRQ4IDA#discussion_r365884675
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAJQ2TN7XGSRZ5PQHG2ZRGLQ5SGCDANCNFSM4KFL5OYQ
).

That makes sense... I couldn't simulate the same failure like the one happening on CI. This is the last one that failed: https://travis-ci.org/plivesey/ConsistencyManager-iOS/builds/635467408
✗ testRetainCycle, XCTAssertNil failed: "ConsistencyManagerTests.TestListener"
✗ testRetainCycle, XCTAssertNil failed: "ConsistencyManagerTests.TestListener"
✗ testRetainCycle, XCTAssertEqual failed: ("1") is not equal to ("0")
✗ testRetainCycle, XCTAssertEqual failed: ("1") is not equal to ("0")
✗ testRetainCycle, XCTAssertEqual failed: ("1") is not equal to ("0")
✗ testRetainCycle, XCTAssertEqual failed: ("1") is not equal to ("0")

Hmmm...I actually failed this test once. Not really sure what's going on, but happy for now to just comment out this test and create an issue in the project to investigate at some point. I don't think it's really worth the time trying to work out why this is flakey.
You're welcome of course to keep investigating, but I'm not sure it's worth the time.
On Mon, Jan 13, 2020 at 8:24 AM, Heberti Almeida <<EMAIL_ADDRESS>> wrote:

That makes sense... I couldn't simulate the same failure like the one
happening on CI. This is the last one that failed: https:/ / travis-ci. org/
plivesey/ ConsistencyManager-iOS/ builds/ 635467408 (
https://travis-ci.org/plivesey/ConsistencyManager-iOS/builds/635467408 )
✗ testRetainCycle, XCTAssertNil failed : " ConsistencyManagerTests.TestListener
" ✗ testRetainCycle, XCTAssertNil failed : " ConsistencyManagerTests.TestListener
" ✗ testRetainCycle, XCTAssertEqual failed : ( " 1 " ) is not equal to ( "
0 " ) ✗ testRetainCycle, XCTAssertEqual failed : ( " 1 " ) is not equal to
( " 0 " ) ✗ testRetainCycle, XCTAssertEqual failed : ( " 1 " ) is not equal
to ( " 0 " ) ✗ testRetainCycle, XCTAssertEqual failed : ( " 1 " ) is not
equal to ( " 0 " )
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
https://github.com/plivesey/ConsistencyManager-iOS/pull/61?email_source=notifications&email_token=AAJQ2TIHYIAIHUJWCQ5KM6LQ5SIVRA5CNFSM4KFL5OY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIZLEMY#issuecomment-573747763
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAJQ2TO56F6357MCNCBWG4LQ5SIVRANCNFSM4KFL5OYQ
).

Alright, I'm going to disable that for now and create an investigation ticket for it.

Looks like it is all good now if you can review again. Tests are randomized and executed on all devices.


CXF-8556: Null AbstractEndpointFactory Properties Interferes with Class-Level PathParam When Creating Proxies with Defined ClientState

CXF-8556: Null AbstractEndpointFactory Properties Interferes with Class-Level PathParam When Creating Proxies with Defined ClientState

The changes in this pull request have correctly fixed the issue described here https://issues.apache.org/jira/browse/CXF-8556.

@nrickles3 Thank you very much for confirming


Theme easter egg

Hi folks! I finally get around to implement the theme that I wanted to do when I first suggested the feature. It's supposed to be a nod to one of my first experience with Microsoft:

reference for youngsters (MS-DOS UI)
But now seeing everyone's improvement on the light/dark theme, I think maybe it's best to leave the theme as light/dark only and maybe have this as an easter egg? (runs msdos() in the console)
However this does look a bit like the infamous blue screen of death, so maybe it's not a good idea at all.

This is great. Why don't we show this theme if people click on the Microsoft logo? That could toggle it.
Also, you've replaced the Microsoft logo with the Windows logo – maybe find a retro Microsoft logo instead?
Maybe unlocking this theme could be the reward for beating the Simon game? (#41)

Maybe unlocking this theme could be the reward for beating the Simon game?
That'd be neat! @makeupsomething

Also, you've replaced the Microsoft logo with the Windows logo – maybe find a retro Microsoft logo instead?
Blibbet's the logo that Microsoft used around that time, I couldn't find one with good resolution + free license yet. Will dig harder

Yeah thats a pretty cool idea! I should have time to look into that tomorrow 🎉

Hi folks, some updates:

This update will also allow adding additional themes by creating a named subfolder in docs/theme that contains at least a theme.css file. Theme then can be loaded by calling theme.changeTo(themeName).
# Theme folder:
docs
└── theme
     └── msdos
          ├── (fonts, images etc.)
          └── theme.css

# Change to theme 'msdos':
theme.changeTo('msdos')

We could make it so that when user beat the simon game, add an extra button for msdos down in the theme switcher. What do you think?

Sorry for my lateness on this. It's been a busy week 💤
Any ideas how to go about integrating this? I think I cannot merge this branch into my own fork as they are both remotes. Or am I thinking about it the wrong way? @d4rekanguok @dvdsgl

@makeupsomething I think ideally you won't need to merge this branch, only need to do a theme switch in your code, maybe something like if (theme && theme.changeTo) theme.changeTo('msdos'); and when this PR is landed it'll work. What do you think?

Great work! Maybe someone should just add this to the theme switcher?

Nice @d4rekanguok you were right, after this PR was merged I was able to easily switch theme when the user wins. https://github.com/Microsoft/join-dev-design/pull/41/commits/da3f02b5840bf11099f8e3995f10db0669af8a11


Top navigation popover clickable when closed

Bug Description
After clicking on the top navigation bar and selecting one of the options from the dropdown, the page navigates to the selected option and the popup closes, but the the popup area is still clickable.
Steps to Reproduce
Steps to reproduce the behavior:

Go to 'topNavBar'-> Click on 'Community'
Click on 'Leaderboard'
Keep mouse pointer on the first contributor's top-left side of Total earnings.
Mouse has hand pointer and when you click on the empty area you are directed to Openings page.

Expected behavior
Mouse should not be clickable. Popover should close completely.
Actual behavior
The popover area is hidden but clickable.
Screenshots/Recordings
thenewboston - Google Chrome 2020-11-01 11-49-43.zip
Account Number
100c627095785ab8a7ed04d548b53f4b0638c44b49c46aea6c003d7b06294881

ah yeah, i saw this bug in the morning :(. Hotfix has been deployed, it should be fixed now


Added Source of car factors and updated average factor

Added source + more plausible estimate when no data is available

This is awesome, would you mind also updating the number in transportation.js?

done.

Should we also update the modelVersion if we're changing this? @corradio

Nevermind above comment, I'm pretty sure we need to update the model version in car as well ;)
Would you mind doing it @martincollignon ?

yes indeed we need to double check the version of car/index.js

done


[Terminal] VSCode Freeze when typing 'a' into terminal and closing it

Issue Type: Bug

The ConPTY team has provided suggestion on how the VSCode team can fix this.
https://github.com/microsoft/terminal/issues/1810#issuecomment-570284546
VS Code version: Code - Insiders 1.42.0-insider (7c0095ee2d064033fc13184127a9adc603454729, 2019-12-20T07:57:35.999Z)
OS version: Windows_NT x64 10.0.19041

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1035G7 CPU @ 1.20GHz (8 x 1498)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.60GB (1.31GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (24)

Extension
Author (truncated)
Version

better-comments
aar
2.0.5

bracket-pair-colorizer-2
Coe
0.0.29

gitlens
eam
10.2.0

gc-excelviewer
Gra
2.1.32

todo-tree
Gru
0.0.162

vscode-peacock
joh
3.2.0

better-powershell-syntax-highlighting
jus
0.0.2

git-graph
mhu
1.20.0

azure-pipelines
ms-
1.157.4

remote-containers
ms-
0.96.0

remote-ssh-edit-nightly
ms-
2019.12.44160

remote-ssh-nightly
ms-
2019.12.44160

remote-wsl
ms-
0.42.0

azure-account
ms-
0.8.7

powershell-preview
ms-
2019.12.1-CI.41180

team
ms-
1.161.0

indent-rainbow
ode
7.4.0

vscode-sort-json
ric
1.18.0

code-settings-sync
Sha
3.4.3

vscode-zipexplorer
sle
0.3.1

vscode-icons
vsc
9.7.0

gistfs
vsl
0.0.41

better-align
wwm
1.1.6

markdown-all-in-one
yzh
2.6.1

/duplicate https://github.com/microsoft/vscode/issues/76548

@Tyriar are you sure it's a dupe? That referenced issue shows that it should work on 1903 and later, and I'm well past that and still have the issue.
As mentioned terminal team has mentioned the vscode team can fix this, but its still currently an upstream issue: https://github.com/microsoft/terminal/issues/1810#issuecomment-570284546

@JustinGrote #76548 points to https://github.com/microsoft/terminal/issues/1810 which points to the new https://github.com/microsoft/node-pty/issues/375 😉