chore(live-code): replace with react-runner

Review link: https://deploy-preview-6589--docusaurus-2.netlify.app/docs/markdown-features/code-blocks/#interactive-code-editor
Motivation
react-runner (or react-live-runner as a compact layer for react-live) is very similar to react-live but support more features, it uses sucrase instead of buble to transpile code(though I found they switched to sucrase recently), more flexible and less bugs, better SSR support
Comparing with buble, sucrase has smaller bundle size and supports modern language features, and supports Typescript, and the compile speed is super fast
SSR support of react-runner
After

Before

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan

Related PRs

@Josh-Cena

Even take compiler optimisation in count

I pushed this PR as a PoC, if you think it's legit, I'll update other related docs, like the notes about imports, actually we can "support" that syntax, and you don't need to set noInline manually, it just works out of box

Yes, that looks great! We can remove the mentions of react-live, and you can update the other docs accordingly. 👍

Thanks, that looks like a nice POC and a good improvement over react-live
I'm not 100% sure of the potential impacts yet so I'm marking this as a potential breaking change, as we are updating the underlying lib.
Some random thoughts:

We need to test this more deeply on sites using live codeblocks more extensively (we don't use much live codeblocks)
How does it compare to Sandpack, which we may want to adopt in addition or as a replacement for the existing plugin
FB/Meta is already interested to deploy Sandpack on other sites like Jest/Relay in the future
I feel like we should create a new distinct plugin, as we may end-up with 2 distinct plugins for live codeblocks in the future 🤷♂  and it can also permit incremental adoption and a deprecation period for the react-live plugin
Considering you are quite alone and it's a new lib, don't we want to own that code, or at least have git/npm publish permissions to ensure this lib will stay maintained?

@slorber It's not a new lib, I created 3 years ago and used in a lot of projects of mine or projects, I just migrated it to Typescript recently, but itself are tested fully, regarding the maintenance, I'm happy to invite more as admin for this package
Regarding Sandpack, yes I noticed you are using it in new React docs website, and that's why I added multi files support recently, and the example is copied from there, I think for small live preview, Sandpack is still too heavy, we still need to load resources in the iframe, and the dependencies will be loaded again there, and then build, but with react-runner, all of those are synchronous and super fast, which provide best SSR support and better user experience
Regarding React Runner, it's designed to be modular, so you can just use react-runner and create your own editor, and even support on-demand imports, like Play React uses CodeMirror and Skypack
I don't think it's a breaking change as there is no change for user space but more features, and the change is actually smaller then from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>which breaks preview cache, which means if error happens, your code editor with jump as nothing will be rendered

I also think that:

This is not a breaking change as there's absolutely no behavior change
Sandpack is quite an irrelevant topic in this migration. We are only interested in getting rid of react-live
SSR seems interesting enough

@Josh-Cena I think builds are failing because the import config of Docusaurus after I tried to re-export createRequire, I'll have a look later, should be easy to fix, or perhaps we don't ex-export it but leave it to user space

@Josh-Cena @slorber So what's the plan

@slorber Yeah regarding there is no user space change, it would be nice to roll it out to test it in the wild

@slorber I've taken your advice and made a small change to react-runner to use scope.import as import scope, so users don't need to use createRequire anymore, though it's still supported https://github.com/nihgwu/react-runner/pull/84 https://github.com/nihgwu/react-runner/pull/85, and it's not included in this PR
Regarding Sandpack, I raised a ticket in reactjs/reactjs.org

Thanks @nihgwu, that PR on the ReactJS website (https://github.com/reactjs/reactjs.org/pull/4339) is interesting and I'll watch it carefully
Unfortunately, you can understand this PR is not a so tiny change and I'll have similar concerns as Dan Abramov has, and as I said I have to focus in priority on the last features we want for v2.0 even though I like this PR.
Please don't be too impatient, I'll review this when it's time, which is unfortunately not this week

@slorber Sure I will wait, and I understand it's not that important for v2 releasing, thank you again for your feedback and detailed explanation. Regarding the concerns, I'd say it's different, as for the ReactJS docs website, it needs more features then common library document website, like Fast Refresh / Server Component, but for most of the cases we don't need to care about that, besides react-live doesn't support those as well, anyway I don't want to disturb your work on V2, take your time, and thank your for this great project

@nihgwu Checking in again as I probably lost track of this PR, for now, there's absolutely no API change and ReactLiveScope works just the same as before without createRequire, right?

@Josh-Cena Yes, the only change is that I added a import field to scope to support imports syntax, because import is a key word, so it should be same and you can still use all the variables as before

b

Note: React-Live 3.0 is also using Sucrase now
https://github.com/FormidableLabs/react-live/releases/tag/v3.0.0

@slorber Yes I know, but I'd say it's not as powerful and flexible as react-runner ;p, BTW styled-components website is already using react-runner https://github.com/styled-components/styled-components-website/pull/847, and soon for material-ui https://github.com/mui/material-ui/pull/32107

@nihgwu react-live suffers from the same issue, but do you think we can change div for span for the line container? The validator is warning that "Element div not allowed as child of element pre in this context.".

@Josh-Cena we can't simply change div to span as it should be a block, we can do

or the following to avoid breaking change(keep className & style props for line)

Even I made this change, validator will still complain on the editor, and I don't think it could be solved easily, like this one

Yeah, I'm also aware of the style tag problem. FYI, Docusaurus solves it through span + br:
https://github.com/facebook/docusaurus/blob/67faa686e8d53ab0f439454bc046344d5580a665/packages/docusaurus-theme-classic/src/theme/CodeBlock/Line/index.tsx#L42-L54

In https://github.com/facebook/docusaurus/pull/9316 we upgraded both prism-react-renderer and react-live.
Thanks for your work, but for now I prefer to close this PR.
It would be better if you create a react-runner community plugin and prove its usefulness in userland. If users adopt it successfully, and report preferring it over react-live, we can consider switching the official package.


REPOSITORY

REPOSITORY
User Story
AS A music lover with a busy schedule
I WANT to see the top tracks and info about any artist
SO THAT I can keep informed.
Acceptance Criteria
GIVEN the GitHub repository contains application code
WHEN I view the repository
THEN repository has a unique name.
AND repository follows best practices for file structure and naming conventions.
AND repository follows best practices for class/id naming conventions, indentation, quality comments, etc.
AND repository contains multiple descriptive commit messages.

GIVEN the GitHub repository contains application code
WHEN I view the repository
THEN repository has a unique name.
AND repository follows best practices for file structure and naming conventions.
AND repository follows best practices for class/id naming conventions, indentation, quality comments, etc.
AND repository contains multiple descriptive commit messages.


Nested For Loop with Else Causes Exception

Create a for loop with an else clause. In the main clause of the for loop, access {{ loop.index }}. In the else clause, put another for loop. When you render the template, you get an exception:
>>> env = jinja2.Environment()
>>> tpl = env.from_string('{% for x in y %}{{ loop.index0 }}{% else %}{% for i in range(3) %}{{ i }}{% endfor %}{% endfor %}')
>>> tpl.render(y=[])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/dave/om/trunk/lib/python2.7/site-packages/Jinja2-2.6-py2.7.egg/jinja2/environment.py", line 894, in render
    return self.environment.handle_exception(exc_info, True)
  File "<template>", line 1, in top-level template code
UnboundLocalError: local variable 'l_loop' referenced before assignment

Fixed in #637


Crashing when parsing names with special character [] in saver.restore(): bug or expected behavior ?

Disclaimer: I have tensorflow 0.7.0 (a bit outdated I suppose). I have not tested it on more recent versions.
When I try:
import tensorflow as tf
x=tf.placeholder(tf.float32,[])
w=tf.Variable(2.)
y=tf.mul(x,w)
saver=tf.train.Saver()
init=tf.initialize_all_variables()
sess1=tf.Session()
sess1.run(init)
name_model="./"+"model"
saver.save(sess,name_model)
sess1.close()
sess2=tf.Session()
path_meta=name_model+".meta"
path_model=name_model
new_saver=tf.train.import_meta_graph(path_meta)
new_saver.restore(sess2,path_model)
print sess2.run(y, feed_dict={x: 1.})==2

It runs perfectly fine but if I change name_model to "./model[1]" for example it crashes with message: Unsuccessful TensorSliceReader constructor: Failed to find any matching files for ./model[1].
I was wondering if this is expected or if it is a bug. If this is buggy what would be aworkaround ?

@jeandut each filesystem has differing behavior wrt special characters.  I'd recommend the best thing is to avoid special characters in your filenames.  Hope this helps!  If you have any other follow-up questions, just add them here.

Ok I had no idea ! Thanks !


Release update after 2.7.15 is missing

https://github.com/includable/react-native-map-link/releases

above page is not updated with latest versions. is there any other place where i can see change log for 2.8 and above versions.

Sorry, would you mind explaining what you mean?

this package still works for Expo?

@asurion-ashutosh Good point, we haven't actively been tracking this using Github releases, but it might be good to add a changelog to the README. This package is very simple though, so hopefully the list of recent commits should contain most of the info you need.

@EmmanuelSkapple This is a bit off-topic, but yes it should, please see the Expo related instructions in the README & open a new issue if those aren't working for you.


psr8-24 | add Version.bump method and tests

This is the last required piece to implement the previous get_next_version method based on conventional commits. The method added enables a specific "bump" to the version based on the maximum level parsed from the commits since the last release (version_history(...)[0] -> translator.to_tag(...) -> CommitStream(...)[version :] -> max(r.bump for r in stream if type(r) is ParsedCommit)).
The rules are:

If no release, the version is unchanged 1.0.0 -> 1.0.0
If already a pre-release version and doing a pre-release, increment the prerelease revision number 1.0.0-rc.1 -> 1.0.0-rc.2
If already a prerelease but doing a non-prerelease, finalize the version 1.0.0-rc.1 -> 1.0.0
If not a prerelease but doing a prerelease, bump by the specified level and add a prerelease token + revision number 1. 1.0.0  -> 1.1.0-rc.1
If not a prerelease and not doing a prerelease, bump by the specified level. 1.0.0 -> 1.1.0

N.B. there's a good chance I will close this as it doesn't cover all the cases and doesn't line up with the JS implementation.


fix(tower-sessions-core): continuously_delete_expired ran delete_expired twice initially

As per tokio::time::interval, "the first tick completes immediately".
The previous iteration of this code caused the delete_expired method to run twice initially.

I have made the change you requested, however believe the following alternative would be better:
        interval.tick().await; // The first tick completes immediately; skip.
        loop {
            self.delete_expired().await?;
            interval.tick().await;
        }

Happy for this to be merged if you disagree, nonetheless.


Failed to load the tab-preview package

Got this right after installation.
Atom Version: 1.7.0-beta0
System: Mac OS X 10.11.3
Thrown From: tab-preview package, v0.7.0
Stack Trace
Failed to load the tab-preview package
At variable @base-accent-color is undefined in /Users/laughedelic/.atom/packages/tab-preview/styles/node-bug.less:38:28

LessError: variable @base-accent-color is undefined
  at /Users/laughedelic/.atom/packages/tab-preview/styles/node-bug.less:38:28

Commands
     -0:49.8.0 core:confirm (atom-text-editor.editor.mini.is-focused)
  3x -0:44.1.0 blur (atom-text-editor.editor.mini.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "keybinding-resolver",
      "language-objective-c",
      "distraction-free-mode",
      "language-gfm",
      "autocomplete",
      "markdown-writer",
      "markdown-lists",
      "markmon-preview",
      "remember-session",
      "mechanical-keyboard",
      "autocomplete-css",
      "git-tab-status",
      "minimap-git-diff",
      "autocomplete-atom-api",
      "language-clojure",
      "language-html",
      "language-less",
      "language-ruby",
      "language-c",
      "language-csharp",
      "language-go",
      "language-make",
      "language-mustache",
      "language-perl",
      "language-php",
      "language-python",
      "language-ruby-on-rails",
      "language-sass",
      "language-sql",
      "language-toml",
      "language-xml",
      "scala-worksheet",
      "scala-worksheet",
      "autocomplete-snippets",
      "autocomplete-html",
      "move-status-items",
      "autocomplete-emojis",
      "welcome",
      "autocomplete-paths",
      "bookmarks",
      "background-tips",
      "encoding-selector",
      "archive-view",
      "markdown-mindmap",
      "autocomplete-bibtex",
      "package-switch",
      "language-idris",
      "antialiased",
      "vim-mode-plus",
      "auto-reveal-in-sidebar",
      "active-files",
      "line-ending-selector",
      "linter-manager",
      "linter-annotations",
      "semanticolor",
      "pull-requests",
      "atom-telegram",
      "nav-panel",
      "metrics",
      "linter-markdown",
      "imdone-atom-github",
      "bottom-dock",
      "git-travel",
      "build-sbt",
      "build",
      "markdown-scroll-sync",
      "github-notifications",
      "minimap-codeglance",
      "linter-scalac",
      "wakatime",
      "pdf-view",
      "markdown-preview",
      "todo-manager",
      "terminal-plus",
      "imdone-atom"
    ],
    "packagesWithKeymapsDisabled": [
      "linter",
      "terminal-plus"
    ],
    "projectHome": "~/dev",
    "themes": [
      "native-ui",
      "atom-material-syntax"
    ]
  }
}

Installed Packages
# User
Ensime, v0.38.0
advanced-open-file, v0.14.3
atom-material-syntax, v0.4.3
auto-reveal-active-file, v0.3.1
autoflow-diffable, v0.5.0
cursor-history, v0.5.9
file-icons, v1.6.18
highlight-selected, v0.11.2
language-diff, v0.4.0
language-elm, v1.3.3
language-haskell, v1.7.5
language-latex, v0.6.1
language-pfm, v0.82.0
language-scala, v1.1.6
language-viml, v0.3.0
latex-completions, v0.3.1
line-diff-details, v1.2.0
linter, v1.11.4
markdown-preview-plus, v2.2.2
markdown-table-formatter, v2.8.0
mathjax-wrapper, v0.1.0
minimap, v4.21.0
native-ui, v0.17.0
package-manager-commands, v2.0.3
plain-simple, v1.0.3
pretty-json, v0.4.1
project-folder, v0.1.8
swackets, v0.10.0
tab-preview, v0.7.0
tablr, v1.0.2
vim-mode, v0.65.0

# Dev
No dev packages

should be working now please try updating to the latest version Thanks!

Uninstalled, installed again, nothing bad happened :+1:


⚠ Issue

In this tutorial
https://docs.astro.build/en/recipes/i18n/#hide-default-language-in-the-url
In section 4, useTranslatedPath should be imported

Hello @amaimus, nice catch! Would you be willing to open a PR to fix this issue?

Sure!

Done! https://github.com/withastro/docs/pull/4771
Let me know if i did something wrong in the process.
Pd: thank you for encouraging me to make my first open source project contribution 🚀

Closed by PR #4771 :tada:


CHAPI bridge

Design:

Tasks:
Browser API for issuer/verifiers

[x] Request connection invitation redirect URL
[x] Request, save wallet profile
[x] Request CHAPI response
[x] Wallet profile recovery

Wallet Server (Moved to adapter: https://github.com/trustbloc/edge-adapter/issues/425)

[x] Open API support #635
[x] Wallet server as agent #637
[x] chapi-bridge interfaces in wallet server #636
[x] Create connection invitation #646
[x] Create wallet profile on did-exchange complete #650
[x] Handle CHAPI request #652
[x] Query wallet profile  #650
[x] Wallet profile recovery  #650
[x] Message handle for CHAPI response DIDcomm message  #652
[x] Retry/Await on app profile request #654

Mediator API

[x] Message handle for CHAPI request DIDcomm message
[ ] Push notification API

Mobile binding APIs

[x] Send CHAPI response
[x] Receive CHAPI request(if required)

Integration in Demo

[x] update edge-sandbox demo

BDD tests

[x] wallet server tests - https://github.com/trustbloc/edge-adapter/issues/418, https://github.com/trustbloc/edge-adapter/issues/417
[x] agent-sdk tests

Documentation

[ ] update demo docs
[ ] update trustbloc docs

Wallet bridge serve & UI in edge-adapters

[x] https://github.com/trustbloc/edge-adapter/issues/425

Universal Wallet 2020

[x] push wallet server interfaces to universal wallet 2020 interop. (Not needed)

Follow up:

[ ] managing wallet app URLs #645
[x] migrate CHAPI bridge as app-bridge in adapters #651 & https://github.com/trustbloc/edge-adapter/issues/413

To be covered as part of Wallet selector APIs


Using more than 6 players on a page causes issues

I have 15 audio files on one page and I want an MEJS player for each one.
I am looking through as such:
$('audio').mediaelementplayer({
    // The order of controls you want on the control bar (and other plugins below)
    features: ['playpause','progress','current','duration'],
    iPhoneUseNativeControls: true,
    iPadUseNativeControls: true,
    AndroidUseNativeControls: true,
    pauseOtherPlayers: true
});

After the 6th player, the other players do not preload, when i press the play button nothing happens.
If i view the audio file individually one the site it plays fine.
The file in the inspector says pending
Is there a limit to the number of players that will get setup and preload?
Or is this a limitation with the browser (I am using Chrome v46)

This appears to be a limitation in Chrome itself only allowing 6 files to be downloaded at once.
My fix was to not preload any audio files. Loading audio files from different URL's or subdomains would also work.


Projects > Open button is greyed out when no active project

Hey everyone,
First of all, great job on release v18. Thank you for all the work you put in this project.
What are the steps to reproduce?

Fresh install
Clone a repository in .node-red/projects/
Open node-red editor
Get prompted to create your first project
Close the window

What happens?
Projects > Open button is greyed out
What do you expect to happen?
Open button not to be greyed out, so we can load the already cloned project.
The disabling happens here :
https://github.com/node-red/node-red/blob/326c6c496e6e07f6b98df82a578a93b89c80f026/editor/js/main.js#L69
Is there a reason I am not getting for this ? Right now it just forces the user to create a new project even if there is another one, just not active.
Thanks !

Yes, right now the editor assumes you are migrating into projects, so if it doesn't know about any projects it takes you through the first run dialog. This is just a point in time thing - we'll be broadening out the options quite quickly as we get feedback (like this issue).
The workaround right now is to close the first run dialog then use ctrl-alt-o as the shortcut for the open project dialog.

Hadn't considered the shortcut. The explanation makes sense. Thanks !


Modify q, k, and v to be consistently 0

To implement NIIC, the NIIC function is implemented through the mask argument as a prototype.

Confirm that the PR message is incorrect, correct it and PR again.


Add projects filter to all_time_since_today Wakatime compatible endpoint

The title is self explanatory I think. I have tried looking at #58 if it was already listed, but the links don't show anything for me.

I see that filtering capability for the Wakatime endpoints is quite crucial. However, it is not that easy to realize due to the way summaries are currently computed. Implementing filtering would require #108 first.
I'll close this ticket as a duplicate of #58, but your request won't be forgotten!


didcomm support

initial support for https://identity.foundation/didcomm-messaging/spec/
should be broken into a set of issues to handle

message types
signing / verifying
message encryption
different protocols supported
threading
using different key types
QR code generation?

@decentralgabe Different protocols ? Hyperledger aries, DIF protocols e.t.c ?

@georgejpadayatti the latest DIDComm work is taking place in DIF. The TBD team is a member and is following the group's progress. https://identity.foundation/working-groups/did-comm.html

In my opinion, did-sdk should have support for both DIF and Hyperledger aries. Both communities have an existing and eager audience who could benefit from a unified sdk.

@georgejpadayatti it was my impression that the work began in Hyperledger Aries and has since moved over to DIF. I believe that's where all the new work continues. Is that no longer true?

It's a bit confusing to be honest. Since I am part of both these communities, personally I feel DIF is where active work is going on, but due to this spec evolves rapidly. Aries community is bit stagnant, but is suitable for implementers because of that. For e.g. IDUnion from Germany has adopted AIP 1.0 for their wallets, Findy from Finland seems to follow them.

closed in support of https://github.com/TBD54566975/did-sdk/issues/62


Add failing test for Command::execute return int rector with try catch

Add failing test for ConsoleExecuteReturnIntRector when execute method has a try catch setup and contains return's in the try and catch part so no extra return should be necessary.
See example: https://getrector.org/demo/7d04f7ed-a69a-4fd1-8389-c1444e3ba15d

@acrobat there is RemoveUnreachableStatementRector that cover that, you can combine the rule with that, see https://getrector.org/demo/9be494b3-c105-440f-9035-8065f684b470

@samsonasik I think this case should be skipped in this rule too, as the return value is always integer.

@TomasVotruba Ok 👍 , @acrobat I cherry-picked your commit at PR https://github.com/rectorphp/rector-symfony/pull/239

Thanks for the quick fix @samsonasik!


Update specs2-core, specs2-scalacheck to 4.10.2

Updates

org.specs2:specs2-core
org.specs2:specs2-scalacheck

from 3.+ to 4.10.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: test-library-update

Superseded by #25.

Superseded by #25.


model builder error

System Information:

Model Builder or CLI Version: unknown (latest?)
Visual Studio Version: 16.8.6

Using this tutorial: https://devblogs.microsoft.com/dotnet/train-image-classification-model-azure-mlnet-model-builder/
It crashed during new workspace creation.
log: 25db3535-983c-4827-a2b4-4f4133ff5691.txt
Next resources still successfully created:

application insights
container registry
key vault
storage account

I'm trying to create a workspace from the azure portal and getting this error:

{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"BadRequest","message":"An invalid value was provided for 'accessPolicies'."}]}

Recreated all required resources from scratch in azure portal, everything works fine.

@SUDALV92 sorry about this error! This is a known issue when creating your first workspace. We are working to replace the problem library now.
Thank you for reporting and apologies for the delay!


Crash at startup after upgrade

http://paste2.org/jvhK5wKn

Go to the version that is fine
then remove all Logistical sorters or someting related to mekanism inventory system
do the upgrade again and replace the pipes
try it :)

Oh my God! That'd take forever. Crap. Guess I stay on the old one then. Geez.

@unpairedbracket, does this have anything to do with the recent inventory management changes?

I believe I have the same issue. I had broke about five Advanced Logistical Transporters but crashed the moment I broke the Advanced Logistical Transporter that was attached to the Logistical Sorter. I was using the <IP_ADDRESS> versions when the crash occurred. I was playing SSP but now I am unable to load that world. The same crash occurs immediately upon loading the world now. I am going to follow M4vErw1kiXL's advice to see if that allows me to load this world again.
Here's my crash log:
http://pastebin.com/TCqmfsgz

I too have the same error; https://gist.github.com/QORTEC/a36332859c6e9cca229d.
I my testing this error only seams to happen when I am loading an existing world, reading the text above I'm guessing its b/c I have Mekanism blocks placed on my world...
@M4vErw1kiXL:
Thank you for your suggestion unfortunately It's not feasible on large builds/servers. As a server admin/modpack maintainer I cannot ask my users to remember where they placed all of their Mekanism related blocks and then remove them. This process could take an ungodly amount of time, I would sooner restart the world since that would probably cause less wrath...

My paste of what appears to be the same crash: http://pastebin.com/1fGD6maK
For what it is worth, I can only get it to crash when using the Logistical Sorter to pull FROM an Applied Energistics 2 charger.
Reproduction steps that work (where 'work' is 'crash spectacularly') for me:
AE2 Charger -> Logistical Sorter -> Basic Logistical Transporter
Set the Logistical Sorter filter via ItemStack to Charged Certus Quartz. Click save. Boom.
The pipe is required. It won't crash if it is just a charger, sorter pair. It does not matter if there is a destination for the pipe. It'll go yucky if there is a chest at the end or not.
No promises that other reports are using the same setup, of course!

Heh, I just had a look at the code,
The variable 'side' is being accessed in the StackSearcher constructor BEFORE its initialized.... which would cause a NPE.

@harley9699699 @HowardRoark2016  @QORTEC @Ommina
Since this is a really crashy bug, heres a version I compiled that includes a few minor bugfixes made by me since 8.1.6, I've compiled it with the exact same version and build numbers so you can just drop it on your servers and players can connect without updating since this is purely a serverside crash. (Unless it's singleplayer, then you would probably want to update.)
https://drive.google.com/file/d/0BxBo-WAVAzaMQkhsN19NR1ZKclU/view?usp=sharing

@ruifung Is this fixed in v<IP_ADDRESS>?

Uh, probably as there hasn't been any other changes to mekanism. I'm not sure though, why don't you try and tell me.

@ruifung I'm using you version, but can I update to v<IP_ADDRESS> and still have it?

Yes. 8.1.7 includes the fix.


Map HTML Element Moves on Scroll or When Navigating between pages

I'm submitting a ...  (check one with "x")
[] question
[x] any problem or bug report
[ ] feature request
If you choose 'problem or bug report', please select OS:  (check one with "x")
[ ] Android
[x] iOS
cordova information:  (run $> cordova plugin list)
om.googlemaps.ios 2.4.0 "Google Maps SDK for iOS"
cordova-background-geolocation-lt 2.8.5 "BackgroundGeolocation"
cordova-plugin-actionsheet 2.3.3 "ActionSheet"
cordova-plugin-advanced-http 1.7.0 "Advanced HTTP plugin"
cordova-plugin-app-event 1.2.1 "Application Events"
cordova-plugin-background-fetch 5.1.1 "CDVBackgroundFetch"
cordova-plugin-background-mode 0.7.2 "BackgroundMode"
cordova-plugin-camera 2.4.1 "Camera"
cordova-plugin-cocoalumberjack 0.0.4 "CocoaLumberjack"
cordova-plugin-compat 1.2.0 "Compat"
cordova-plugin-console 1.1.0 "Console"
cordova-plugin-device 1.1.6 "Device"
cordova-plugin-dialogs 1.3.3 "Notification"
cordova-plugin-facebook4 1.9.1 "Facebook Connect"
cordova-plugin-file 4.3.3 "File"
cordova-plugin-file-transfer 1.6.3 "File Transfer"
cordova-plugin-geolocation 2.4.3 "Geolocation"
cordova-plugin-googlemaps 2.1.0-beta-20171027-0054 "cordova-plugin-googlemaps"
cordova-plugin-inappbrowser 1.7.1 "InAppBrowser"
cordova-plugin-ionic-webview 1.1.16 "cordova-plugin-ionic-webview"
cordova-plugin-nativestorage 2.2.2 "NativeStorage"
cordova-plugin-splashscreen 4.0.3 "Splashscreen"
cordova-plugin-statusbar 2.2.3 "StatusBar"
cordova-plugin-whitelist 1.3.2 "Whitelist"
de.appplant.cordova.plugin.local-notification 0.8.5 "LocalNotification"
ionic-plugin-keyboard 2.2.1 "Keyboard"
uk.co.workingedge.phonegap.plugin.launchnavigator 4.0.7 "Launch Navigator"

If you use @ionic-native/google-maps, please tell the package.json (only @ionic-native/core and @ionic-native/google-maps are fine mostly)
Current behavior:
When I scroll on my page that contains the map, the entire map div moves and pushes my other HTML elements out of the way. Also, when I change pages and then come back to the page containing the map the map delays and causes a drag effect.
Expected behavior:
The map should stay fixed when scrolling or changing pages and not push other elements out of position.
Screen capture or video record:
Recording
Git Rep
**This repo should be shared with you @wf9a5m75, please let me know if you have issues accessing it. **

I see. I will check this. Thank you for sharing your repo in advance.
By the way, cordova-plugin-googlemaps 2.1.0-beta-20171027-0054 is kind of older code. Please try to reinstall the plugin from the multiple_maps branch.

Thank you @wf9a5m75!

I saw the video.
I guess adjust your css solve this problem.
#map {
    width: 100%;
    position: fixed;
    left: 0;
    top: ...px;
    width: 100%;
    height: 200px;
}

Because the plugin follows the div position. The map div moves because of the page scrolling.

I'm having trouble getting this to work. When I set the position to fixed the other HTML elements on the page overlap the map.
Any other ideas?

There is no solution.
Remember, the native map view is not HTML element. That's why the plugin sends the position and size information to native side periodically. That's why the map is delayed.
Currently there is no solution about this, and this is not bug.
I recommend you fix the map, and scroll the bottom parts only.

Another reason of this delay is the ionic framework uses -webkit-overflow-scrolling: touch style for the scroll-content css.
The browser scrolls the DOM element on GPU, it moves smoothly.
However scroll event is not fired during movement.
That's why the plugin can not get the div position.
You can disable the style with  -webkit-overflow-scrolling: auto

I maybe have new update for you in today. Hold on please

Great, thank you so much!

I had tackled to improve the DOM inspection last 3 weeks.
And I was able to archive to do that today.
(You are lucky, good timinig)
In order to use this dev version, you need to install the plugin from the multiple_maps branch.
I also fixed your code a lot.
https://github.com/svarughese91/GoogleMap/compare/1901_fixed
I push the code to the 1901_fixed branch.
Please follow the steps.
$> git clone https://github.com/svarughese91/GoogleMap

$> git checkout 1901_fixed

$> git pull

$> rm -rf node_modules

$> npm install

$> ionic cordova run ios

Thank you!! I'll update you once we get these changes rolled into our app.

Thank you! I was having the same issue.

Use the multiple_maps branch

I have tried install with this on ionic
ionic cordova plugin add https://github.com/mapsplugin/cordova-plugin-googlemaps#multiple_maps  --variable API_KEY_FOR_ANDROID="xx" --variable API_KEY_FOR_IOS="xx"
is it correct?
Thanks!

I didn't see the recording because return error 404...but reading the description looks similar my problem...
I resolved inserting div with map between a ion-item tag

I worked trying to solve this issue more time than I thought.
I'm not using ionic on this project, the solution for me was handle the setClickable function:
map.setClickable(false);
On my app, the page loaded is in front of the map, that I keep running. When the user tries to scroll down the page it was changing the map camera target instead of the scroll down.
It worked fine after I used the setClickable to false.

I had tackled to improve the DOM inspection last 3 weeks.
And I was able to archive to do that today.
(You are lucky, good timinig)
In order to use this dev version, you need to install the plugin from the multiple_maps branch.
I also fixed your code a lot.
https://github.com/svarughese91/GoogleMap/compare/1901_fixed
I push the code to the 1901_fixed branch.
Please follow the steps.
$> git clone https://github.com/svarughese91/GoogleMap

$> git checkout 1901_fixed

$> git pull

$> rm -rf node_modules

$> npm install

$> ionic cordova run ios

what is username and password of repo because i am not able to open it ask me user name and password


watch and Reactive work together to use deep listening issues

Vue version
vue 3.0
Link to minimal reproduction
https://github.com/vuejs/vue
Steps to reproduce
When reactive and watch's deep monitoring are used together in vue3, when the first parameter of watch monitors the object, how to solve the problem that the new value of the second parameter callback function is consistent with the old value?
What is expected?
Be resolved
What is actually happening?
Unable to detect
System Info
When reactive and watch's deep monitoring are used together in vue3, when the first parameter of watch monitors the object, how to solve the problem that the new value of the second parameter callback function is consistent with the old value?

Any additional comments?
No response

the old value is the same as the new value because both are the same object that has been mutated.
If you would need to detect the actual differences, you would have to clone the object i.e. like this:
watch(
  () => _.cloneDeep(obj)
  (newObj, oldObj) => { do something }
)

watch(()=>cloneDeep(obj),(newvalue,odlvalue)=>{
  console.log(newvalue,odlvalue);
},{deep:true})

Oh, I see, thank you. This problem has been solved,I am glad to communicate with you this time, and I have learned a lot of knowledge

But why do I need your help


在文件prost.rs文件中定义的"pub struct Schema"等结构体，需要使用"#[derive(Serialize)]"属性生成序列化的方法。

Describe the bug
在使用函数datafusion_proto::bytes::physical_plan_from_json将json字符串反序列化为PhysicalPlan结构是，构建会报错：error[E0277]: the trait bound datafusion_proto_common::Schema: Serialize is not satisfied
To Reproduce
No response
Expected behavior
No response
Additional context
No response

你最好应该给一个真实的代码使用案例。

你最好应该给一个真实的代码使用案例。 You should ideally give a real code use case.

test.rs:
use datafusion::prelude::SessionContext;
use datafusion::physical_plan::{displayable};
use datafusion::logical_expr::PlanType;
pub async fn execute_substrait_ir1(ctx: &SessionContext, physical_plan_json: &str) {
let physical_plan = datafusion_proto::bytes::physical_plan_from_json(physical_plan_json, &ctx)?;
println!(
"Final physical plan:\n\n{}\n\n",
displayable(physical_plan.as_ref())
.to_stringified(false, PlanType::InitialPhysicalPlan)
.plan
);
}
Cargo.toml:
[package]
name = "untitled"
version = "0.1.0"
edition = "2021"
[dependencies]
datafusion = "41.0.0"
datafusion-proto = { version = "41.0.0", features = ["json"] }
编译报错如下：
error[E0277]: the trait bound datafusion_proto_common::Schema: Serialize is not satisfied
.cargo/registry/src/szv-open.codehub.huawei.com-1ce540e2f9d95e5a/datafusion-proto-41.0.0/src/generated/pbjson.rs:158:55
|
158  |             struct_ser.serialize_field("inputSchema", v)?;
|                        ---------------                ^ the trait Serialize is not implemented for datafusion_proto_common::Schema
|                        |
|                        required by a bound introduced by this call
|

This looks like you are trying to use rust Serde to serialize the structures
I think the examples on https://docs.rs/datafusion-proto/41.0.0/datafusion_proto/ might help

This looks like you are trying to use rust Serde to serialize the structures, which is not what datafusion-proto does
I think the examples on https://docs.rs/datafusion-proto/41.0.0/datafusion_proto/ might help -- you can convert datafusion plans etc directly into bytes (u8)

I used the function "physical_plan_from_bytes" in this example(https://docs.rs/datafusion-proto/41.0.0/datafusion_proto/ ), but it still gives me an error when compiling. The error message is as follows:

I wonder if you need to enable the serde feature 🤔  https://github.com/apache/datafusion?tab=readme-ov-file#crate-features


Typo

What's changed, or what was fixed?

Typo lost -> lots

Fixes: https://developers.google.com/web/tools/workbox/modules/workbox-window#window_to_service_worker_communication
Target Live Date: YYYY-MM-DD

[ ] This has been reviewed and approved by (NAME)
[ ] I have run npm test locally and all tests pass.
[ ] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


Separate FileSegment interface and AbstractFileSegment abstract class?

Okay, this might be a bit pedantic, but I think FileSegment should be an interface with AbstractFileSegment as the base class implementing that interface.
From a hygiene point of view, separating interfaces from impl is "a good thing".
From a practical perspective, our docs say something like:

A static collection of documents, comprised of one or more {@link AbstractFileSegment}s.

Which is technically not true and sounds a bit odd. But saying subclasses or concrete implementations of AbstractFileSegment everywhere sounds too awkward.
@borislin @Peilin-Yang thoughts?
I can work on it.

I'm a bit confused.
Right now, we have our FileSegment in each collection, which extends to AbstractFileSegment as the base class.
Do you mean we should create an interface and let our FileSegment in each collection to implement that interface?

No, just that we should have interface FileSegment and abstract class AbstractFileSegment implements FileSegment.

But right now, we already have abstract class AbstractFileSegment<T extends SourceDocument> implements Iterator<T>, Closeable. If we create another interface FileSegment, what does this interface contain?

So, it would be:

interface FileSegment extends Iterator

and

abstract class AbstractFileSegment implements FileSegment, Closeable

Something like that?

Sure, I'm okay with this.

Fixed in #366 - see CR for additional discussion on class names we settled on.


Text in printed PDFs is blurry in FF52 version of PDF.js

Unable to provide PDF because of corporate branding
Configuration:

Web browser and its version: Firefox versions 42, 47.0.2, and 52 x32
Operating system and its version: Windows 7 SP1 x64 running as a VM
PDF.js version: Whatever comes with the related Firefox versions
Is an extension: No, builtin

Steps to reproduce the problem:

Print a PDF directly from an in-browser preview (PDFs built with custom compile of WKHTMLTOPDF 0.12.4-dev and QT 4.8.7 and loaded into iframe on the page)

Screenshot: https://imgur.com/7Beh7MO
All 3 pages shown in the screenshot are the same document printed directly from the PDF.js viewer without downloading and opening in a separate viewer. From left to right, they were printed on a fresh install of FF 52, FF 47.0.2, and an install of FF 42 (unsure if fresh or not). The document was printed for the 47.0.2 and 52 installs immediately after restarting the browser (No printout of 43, which was the version available between 42 and 47.0.2).
As is visible, the document on the left (Firefox 52) is visibly blurry/fuzzy, and is crisp in the other two (Firefox 47.0.2 and 42 respectively). Something changed between 47.0.2 and 52 releases of PDF.js that is causing this issue. When the document is downloaded from the preview window and printed inside of a reader such as Adobe or Foxit, the text is crisp).
All documents were printed on a "TOSHIBA e-STUDIO3510c" series printer using the PS3 driver, could also be the "TOSHIBA Universal Printer 2" driver (Both x64 drivers). Any information on what may have changed between versions 47.0.2 and 52 to cause the blurring of printed text would be useful. This was originally found due to reports that updating to 52 and the corresponding loss of use for Adobe plugins caused PDF files to display but to print blank pages, which was not replicated in any of my tests.
EDIT: Kerning issues appear to be related to the specific binary for WKHTMLTOPDF as a similar printout from the production system did not yield those issues, so they can be disregarded as I believe the issue is on my end for that.

After additional troubleshooting, it appears that the blank printouts issue mentioned as the genesis of this testing was being caused by something specific to Firefox 52 in Windows 10, though specifics beyond that are not available.

This problem is known and there are already many open issues for it. I'm closing this one because no additional information is present that is not already present in these other issues. For now there is not much we can do about this as resolving this means significantly more memory usage right now, until we have some sort of SVG back-end, which is still a project we're interested in.

That is understandable, but I am confused as to why this issue did not
present in the older versions of PDF.js when printing? Was there some kind
of change to the printing algorithms to save resources or some setting to
allow the system to hose all of the memory anyway?
On Mar 13, 2017 6:37 PM, "Tim van der Meij"<EMAIL_ADDRESS>wrote:

This problem is known and there are already many open issues for it. I'm
closing this one because no additional information is present that is not
already present in these other issues. For now there is not much we can do
about this as resolving this means significantly more memory usage right
now, until we have some sort of SVG back-end, which is still a project
we're interested in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mozilla/pdf.js/issues/8158#issuecomment-286265452,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOZHc8tUmsoHd8eIR4KJL8nt6bDzRDABks5rlcUPgaJpZM4Mbnyd
.

--
Please click to read Notices:
http://www.mediweightlossclinics.com/sig/ Confidential Communication
http://www.mediweightlossclinics.com/sig/ Educational Purposes Disclaimer
http://www.mediweightlossclinics.com/sig/ Franchise Disclosure Statement

Some changes have been made in the printing code, such as #7677, but it only improved the situation for all documents we have seen that have been reported. The patches mainly solved scaling issues and made printing slightly less blurry, but not significantly because of the mentioned browser limitations regarding memory usage. The issue should already have been present in older versions of PDF.js.


Minimal version of this - Numeric Types Only

Javascript is untyped language, and i'm not sure if its even possible to have a performant implementation of operator overloading without either (or both) Types and Compilation.
Still, the current version of proposal contains a very promising hack _px and so on - which is basically "unit of measure" or custom numeric types.
If it is possible to allow
import { px, em } from 'css-typed-om/units';

let a = 2_px + 2_px

but its hard to extend such approach to non-scalar values:
import { matrix as m, Matrix, vector as v } from 'super-math/units-with-operators';

let a = 5 * (new Matrix([[1,2,3],[5,6,7,]]))_m; // not really looking good..

let M = new Matrix([[1,2,3],[4,5,6]]);
let b = 5 * (M)_m; // still ugly, but not as much

p.s.: this means only the Field operators (+, *, -, /) and comparison (<, >, >=, <=, ==) can be overloaded. Their behavior is supposed to be based on conventional math. [ ] is out of scope, and bit shifts too. Operator overloading for Arrays, Sets and all the nicer (more complicated stuff is also out of scope).
p.p.s: this is just a strawman, seeing how none of the operator overloading proposal in JS ever made it anywhere - maybe some small solution solving some very practical issue (say: math, css, ai/ml) can be a first stepping stone.

It's unclear to me what semantics you are proposing. How would numeric types be distinguished, if they are in a variable?

I propose a special punctuation to mark numeric types during arithmetic operations.
It's _ to be similar with original proposal, but the actual character does not matter #noBikeshedding
import {
  numberPostfix as nx,
  optionalNumberConstructor as Cx
} from 'number-library'; // where + is overloaded

let a = new Cx(...params); // a is instance of Cx, without value semantics
let a2 = a + a; // behaves just as current ES, just coerses to "nearest" js type (either number or string) via .toString() or .valueOf()

let b = new Cx(..params)_nx; // b is still an instance of Cx, without value semantics
let b20 = b + b; // same as before
let b21 = (b) + (b); // same as before

let b22 = (b)_nx + (b)_nx; // overloaded "+" actually triggers, b2 is now a "sum" of b and b

p.s.: hopefully, the code example makes it a bit more clear.

Well, after the code is executed, how is that punctuation mark held in the value?

in this strawman this is not possible.
I think if anyone can resolve "how to hold both value and type (without ruining performance)", then my strawman is not needed, because then you can just use operators without extra syntax.

I don't see how this would fit cleanly into the JavaScript language, where functions and operators treat their parameters as expressions, rather than looking into their internal syntax. I agree that we should not hurt JS performance, and that we should not add operator overloading to JS if it will make it slower.


docs(faq): added how to use local test ledger images

Added FAQ question about how to use locally-created test ledger images in tests/example applications

Codecov Report

Merging #1435 (9f1d12f) into main (126450f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1435   +/-   ##
=======================================
  Coverage   70.39%   70.39%
=======================================
  Files         302      302
  Lines       10902    10902
  Branches     1332     1332
=======================================
  Hits         7674     7674
  Misses       2526     2526
  Partials      702      702

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 126450f...9f1d12f. Read the comment docs.


Merlin to support both ML and RE simultaneously.

@def-lkb:
We're realizing that one important feature, which might be satisfied by your latest progress on Merlin, is the ability to have merlin that will operate on both regular ML (with the default "reader"), and on RE, with a custom Reader. With the way we started out, we just passed global -pp flags to Merlin. This works when you only edit .re files, but when we edit .ml files, they will crash. Then we discussed adding the READERS to the .merlin file. This is a good start, because different projects might want different readers - but I believe it's not enough because it still might pass a .ml file to the Reason Reader, which it can't really handle (cannot do parsing recovery etc).
I think what we really want is the following specified in the merlin file:
READER reason .re .rei

This should accomplish three things:

Register the reader reason.
Associate the reason reader with file suffixes (.re for impl, and .rei for interfaces).
Allow .re, .rei to participate in MerlinLocate commands (so could take the place of the SUFFIX command).

To register only the suffixes without a Reader, I suppose you could do:
READER default .re .rei

Are there pointers to the nuclide/atom issues tracking this problem?  On the infer team we are hitting this and it's quite suboptimal.  Is using merlin with both reason and ocaml simultaneously expected to work, if we were to upgrade to some particular version?

I am editong both simultaneously in atom reason so I assume it works in other editors too. I think it was added in a recent version and Yunxing is upgrading Infer to that latest version as we speak so that it can use the new packaging workflow.

I can confirm that it works in the vscode extension too. As long as you have a new enough merlin with merlin-extend support (might require building from master), it should "just work".
When a file is synchronized with merlin, commands of the following form are sent:
{
  context: ["auto", "M.{ml, mli, re, rei}"],
  query: ["tell", "<start>", "<end>", "<content>"]
}

If merlin-extend support is available, and "auto" is used for the context (rather than "ml" or "mli"), merlin should detect whether it is an OCaml or Reason file and do the right thing.


Cross Cluster Replication (CCR) and Snap integration

added ccr_leader endpoint as provider using opensearch-cluster interface
added ccr_follower endpoint as requires using opensearch-cluster interface
unit tests for CCR
instructions on README.md to use CCR
integration with the opensearch snap that is used as resource until it is published.
enforce_hostname_verification

The relation-broken for the CCR is not part of the scope of this PR, My suggestion for the future implementation of this is to observe on.ccr_follower_relation_broken and delete the replication rule. The leader_alias is ccr-connection-alias and the name is all-replication-rule.


Website Redesign - Getting started

Fixes #10908
This PR implements the getting-started page.

[ ] The link for "Learn more" under get started as a healthcare organization is not linked to the page
[ ] This is my bad not yours, but can we change the colon to a period in "These are the current data types we accept and the growing list of conditions that ReportStream can help you send or receive:"

On 05/15/23, we decided to use desktop-lg for public site which means max width is 1200 px. Design is based on desktop-lg max-width.
Please update the following:

[ ] Update max-width from 87.5rem to 75rem/1200 px
[ ] Use H3 for usa card header instead of the H2
[ ] Add margin-top to rs-tile group: 42px
[ ] Add margin-top to p tag: 20px (this may have already been adjusted)

The list of updates assumes that the spacing updates made on Get started sending data have been inherited here also, so I have not added all the spacing notes.


Remove change event listeners

Reference: https://github.com/github/github/issues/130764
This is so that we can prevent a redundant check being trigger when people blur out of an input. The input event fires on paste/compsitionend and just any input value changes; as far as I can see this should be enough.
The redundant change event was caused by https://github.com/github/auto-check-element/commit/6d6a9ef830d26a04e8f5d64064df9f1a1a73586c, and I wish we can keep that change.
Notes:

we should do some manual testing to ensure input event fires reliably across browsers
if apps using auto-check are firing change event to trigger a check, they'd need to be changed to input instead

Should we publish this under a prerelease tag and test integrating this into github.com?

I am going to test this locally.


Drive init if aborted should clean up after itself

If a request for a drive initialization is made, then aborted the state of the directory should be what it was before the init.

Addressed with https://github.com/odeke-em/drive/commit/a9538461ae2e746625d9f84e04d7d41a20475205 and PR #16

Addressed with https://github.com/odeke-em/drive/commit/a9538461ae2e746625d9f84e04d7d41a20475205 and PR #16


Volar: Cannot find module 'daisyui-vue' or its corresponding type declarations.

can you add typescript config for nuxt3?

Hi, I've created a demo, and it appears to be functioning properly.
Demo: https://codesandbox.io/p/sandbox/kind-maxwell-6g84ww?file=%2Fpages%2Findex.vue%3A35%2C1


pointer being freed was not allocated *** set a breakpoint in malloc_error_break to debug

a simple task to test it, but error occurs
gulp(83527,0x7fff7ca75300) malloc: *** error for object 0x100c341e0: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
[1]    83527 abort      gulp sprites2

can you provide the code for the task you're running?

Same here. Here's the task I'm running:
gulp.task('sprites', function() {
  return sprity.src({
    src: './src/images/sprite/*.{png,jpg}',
    style: '_sprite.sass',
    processor: 'sass',
    cssPath: '../images',
    prefix: 'sprite'
  })
  .pipe(gulpif('*.png', gulp.dest('./src/images/'), gulp.dest('./src/css/partials/')))
  .pipe(notify({ message: 'Sprites task complete' }));
});

When switching the engine to canvas the error doesn't occur, so it probably has something to do with the default engine which is lwip.

@beysion did you manage to get it working?

@aslansky?

I can't get this to reproduce. What Operating systems are you on? How many images are you trying to process?

I'm on OS X Yosemite 10.10.2. Trying to process about 20 images.

Hey guys, any solution for this? I'm getting the same error. OSX 10.11 beta 3. About 30 images.

Maybe it has something to do with this issue: https://github.com/EyalAr/lwip/issues/144? If so there is not much I could do about this. Would have to wait until lwip resolves this issue.


Revamp docs

Made docs be more up-to-date. Commits themselves describe what I've actually done

Gonna close this and redo it properly.


extra blank lines added after cursor in radian powered terminal

When running the radian as terminal, some empty lines keep getting added under the cursor, obfuscating the content above.

No matter how much the terminal is expanded, several empty lines are added.
If I turn radian off, the terminal behaves as expected.

Environment:

OS: Fedora
VSCode Version: 1.54.3
R Version: 4.0.4
vscode-R version: 1.6.5

These are the settings:
{
    "r.bracketedPaste": true,
    "r.lsp.debug": true,
    //"r.rterm.linux": "/home/user/.local/bin/radian",  // radian temporarily turned off
    "r.lsp.diagnostics": true,
    "r.rterm.option": [
        "--no-save",
        "--no-restore"
    ],
    "r.rpath.linux": "/usr/bin/R",
    "files.associations": {
        "*.rmd": "markdown"
    },
    "r.sessionWatcher": true
}

This is a known radian issue (https://github.com/randy3k/radian/issues/135).


Not able to load keras

So I changed my keras to version 1.1.0; by the following command in google colab :
!pip install q keras==1.1.0
But now I am not able to import keras. It is showing me this error.
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 import keras
2 frames
/usr/local/lib/python3.7/dist-packages/keras/backend/tensorflow_backend.py in ()
15
16 _SESSION = None
---> 17 _LEARNING_PHASE = tf.placeholder(dtype='uint8', name='keras_learning_phase')  # 0 = test, 1 = train
18 _MANUAL_VAR_INIT = False
19
AttributeError: module 'tensorflow' has no attribute 'placeholder'

There might be some issue with the TensorFlow version which comes by default in Google Colab. Try to import tensorflow.compat.v1

@abhishek111222 You need to add the following lines before importing keras
import os
os.environ['KERAS_BACKEND'] = 'theano'

WIth that modification, I was able to import keras1.1.0. Please check the gist here. Thanks!

I am closing this issue as this was resolved. Feel free to reopen if I am mistaken. Thanks!


log message pollutes cli output

I've been wondering about random "string" log messages showing up in cli after a test fails. Turns out it came from this line. I assume this wasn't by design.

Fixed in https://github.com/pavkam/codeceptjs-browserlogs-plugin/pull/6
Thanks!


module 'comfy.sd' has no attribute 'load_diffusion_model_state_dict'

Error occurred when executing UnetLoaderGGUF:
module 'comfy.sd' has no attribute 'load_diffusion_model_state_dict'
File "J:\ComfyUI_windows_portable_nvidia\ComfyUI_windows_portable\ComfyUI\execution.py", line 152, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "J:\ComfyUI_windows_portable_nvidia\ComfyUI_windows_portable\ComfyUI\execution.py", line 82, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "J:\ComfyUI_windows_portable_nvidia\ComfyUI_windows_portable\ComfyUI\execution.py", line 75, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "J:\ComfyUI_windows_portable_nvidia\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-GGUF\nodes.py", line 259, in load_unet
model = comfy.sd.load_diffusion_model_state_dict(
I do not understand why this error is happening. I can't update comfy any more than it is. If somebody can help with what is going on with this error.

The ComfyUI_windows_portable\update folder should have a filed called update_comfyui.bat - Does this one complete the update successfully or does it throw an error?

i made this manual setup for comfyui in windows try it
https://civitai.com/articles/7088/manual-comfyui-setupwin

@al-swaiti This has nothing to do with updating an existing install. Please stop advertising your own guides and models in this repository.

do you think doing this an ads , when u did gguf it was ads for you !

keep in your mind ,,,, all things it was to support u ! good luck

@al-swaiti This issue is about the standalone install from the official repository, and your post doesn't mention anything about updating either the standalone or the manual version. Asking someone to reinstall just to update is unreasonable. A guide on how to update the standalone or troubleshoot issues would make more sense to link, but this just looks like you're advertising.
Your guide is mostly the the official guide from the ComfyUI repo + a step by step for how to install the manager. I don't think this is relevant to this repo/issue and there's already tons of different guides like it on the internet.

i made that because they tell me there is not guid for windows search again !
its okay  ,,, you can delete my messages  ,,,
if i were u ill say thanks for your time ,,,
you said  to him to use update bat file which is from old repo ,,, !!!!
he must download new one
the best to pypass this issues is to git clone and then use update from manager ,,,

Checked, that bat file is present on latest standalone. It creates a backup of the current version and updates to latest.

when you go advanced in comfyui , the issues start increasing with  portable versions ,  So the best create env install libraries and run comfyui  ,,, its more comfortable and better with device  original drivers

@al-swaiti This has nothing to do with updating an existing install. Please stop advertising your own guides and models in this repository.

yesterday i saw some thing about zram or swap but i didnt share my stuff https://civitai.com/articles/6880/flux-saunaincrease-your-ram because i cant  try it on  my virtual windows machine

this error also happened in my comfyui, it has been solved? help me, my god

my error is this:
Error occurred when executing DualCLIPLoaderGGUF:
module 'comfy.sd' has no attribute 'load_text_encoder_state_dicts'
File "D:\fooocus\ComfyUI-aki-v1.3\execution.py", line 316, in execute
output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
File "D:\fooocus\ComfyUI-aki-v1.3\execution.py", line 191, in get_output_data
return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
File "D:\fooocus\ComfyUI-aki-v1.3\execution.py", line 168, in _map_node_over_list
process_inputs(input_dict, i)
File "D:\fooocus\ComfyUI-aki-v1.3\execution.py", line 157, in process_inputs
results.append(getattr(obj, func)(**inputs))
File "D:\fooocus\ComfyUI-aki-v1.3\custom_nodes\ComfyUI-GGUF\nodes.py", line 305, in load_clip
return (self.load_patcher(clip_paths, clip_type, self.load_data(clip_paths)),)
File "D:\fooocus\ComfyUI-aki-v1.3\custom_nodes\ComfyUI-GGUF\nodes.py", line 264, in load_patcher
clip = comfy.sd.load_text_encoder_state_dicts(

Well, on latest ComfyUI the "load_text_encoder_state_dicts" is present right here so all I can offer is the same advice as above about updating ComfyUI.

v~寂
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "city96/ComfyUI-GGUF"                                                                                @.>;
发送时间: 2024年9月14日(星期六) 凌晨1:09
@.>;
<EMAIL_ADDRESS>主题: Re: [city96/ComfyUI-GGUF] module 'comfy.sd' has no attribute 'load_diffusion_model_state_dict' (Issue #101)
Well, on latest ComfyUI the "load_text_encoder_state_dicts" is present right here so all I can offer is the same advice as above about updating ComfyUI.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.
thank you very much, after update the comfyui, the error is solved, you are really Awesome！！Message ID: @.***>

I was going to add a warning that pops up and prompts the user to update but I forgot about it lol. But yeah, I'll close this.


MapView "lite mode" as in Google Maps

Issue Type

[x] Improvement

Description and/or steps/code to reproduce the problem
As mentioned by @AbduazizKayumov in #958, it would be nice to have a lite mode in MapView, similar to GoogleMaps':

A lite mode map is a bitmap image of a map at a specified location and zoom level

i feel like this was requested before. Someone had asked for a mechanism to fetch a specific tile from the cache, would that satisfy this need?

@spyhunter99 Rather than just a tile fetch, it's more like "I would like to put MapView snapshots into a ListView or a CardView"
I know that in my app I used some kind of MapView bitmap export (including tiles, markers, polylines, ...).
I believe most of the export part is already coded. We just need to make it easy to use.
And with a nice demo! I'm currently building another demo with all US states and gathered their geo bounding boxes.
A ListView with 50 US states snapshots would be the target demo. Possibly with rounded corners ;)

I'd use it for thumbnails of locations

A problem might be to know when a MapView is fully rendered and ready to export:

when are ALL the needed tiles available?
possible downgraded mode that includes MapTileApproximater?

For the record, when I exported a MapView into a Bitmap in my app, I used View.getDrawingCache(false). Can we do the same with many distinct maps to exports? I don't think so.

Working on it...

Knowing when all the needed tiles are available is not an issue, a simple loop is enough.
I managed to export a large png from a geo area, but only the tiles.
I didn't have to create a MapView for that.
Now the problem is: should I create a dummy MapView just to use its draw method, and will it work in background mode, even if the MapView is not visible? Or is it possible to ignore the View aspect and draw directly on a Canvas? Of course the additional views like info windows would be ignored...
The problem is that MapView is at the same time a View, a Canvas drawer, a touch listener and - thanks to Android technology - a kind of repository for Context, Resources or Bitmaps.

Working again on the subject.
A way to include all kinds of Overlays in the snapshot is to use Projection instead of MapView in method Overlay.draw(Canvas canvas, MapView mapView, boolean shadow).
@spyhunter99 While I'm creating Overlay.draw(Canvas canvas, Projection projection, boolean shadow), wouldn't we be even better off without the "shadow" parameter, which in the code comments seems to be deemed finally useless, cf. DefaultOverlayManager - always pass false, the shadow parameter will be removed in a later version of osmdroid. What do you think about that?

A good compromise for removing the method with the pShadow argument is exactly this. Make a new method without it, deprecate the old one and have the old one call the new one. Then may be few versions down the road, add an annoying log statement, that it will go away, then finally remove it

Currently working on step 2. The goal is to put map snapshots in a ListView (as ImageViews)

Done. I've just PR'ed #1277.

Done in #1277. Closed.

Hello, I did not understand how to use this :(
If it’s not difficult for you, tell me, please.
How can I generate images with the image of a track in a RecyclerView?

@s-ilnar As stated in #1277, try to find inspiration in SampleMapSnapshot sample class, especially when a MapSnapshot is created.

@monsieurtanuki Thank you!
Is this a normal result or am I doing something wrong? Why is the level of zoom low?
my code:
private final List<Overlay> mOverlays;
private final TileSystem mTileSystem = new TileSystemWebMercator();
final MapTileProviderBase mapTileProvider;

this.mOverlays = new ArrayList<>();
this.mapTileProvider = new MapTileProviderBasic(context);

final DisplayMetrics displayMetrics = getActivity().getResources().getDisplayMetrics();
int mapWidth = displayMetrics.widthPixels;
int height = (int) TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, 150, getResources().getDisplayMetrics());

ScaleBarOverlay mScaleBarOverlay = new ScaleBarOverlay(context, mapWidth, mapHeight);
mScaleBarOverlay.setCentred(true);
mScaleBarOverlay.setScaleBarOffset(mapWidth / 2, 10);
this.mOverlays.add(mScaleBarOverlay);

private void download() {
	Marker marker = new Marker(new MapView(App.getAppContext()));
	marker.setPosition(new GeoPoint(55.724668, 52.444936));
	BoundingBox boundingBox = marker.getBounds();
	mOverlays.add(marker);

	final double zoom = mTileSystem.getBoundingBoxZoom(boundingBox, mapWidth, mapHeight);
	final MapSnapshot mapSnapshot = new MapSnapshot(new MapSnapshot.MapSnapshotable() {
		@Override
		public void callback(final MapSnapshot pMapSnapshot) {
			if (pMapSnapshot.getStatus() != MapSnapshot.Status.CANVAS_OK) {
				return;
			}
			bitmap = Bitmap.createBitmap(pMapSnapshot.getBitmap());

			recyclerView.post(new Runnable() {
				@Override
				public void run() {
					notifyDataSetChanged();
				}
			});
		}
	}, MapSnapshot.INCLUDE_FLAG_UPTODATE, mapTileProvider, mOverlays,
			new Projection(zoom, mapWidth, mapHeight, boundingBox.getCenterWithDateLine(), 0, true, true, 0, 0));
   // mMapSnapshots.put(key, mapSnapshot);
	new Thread(mapSnapshot).start(); // TODO use AsyncTask, Executors instead?
}

<androidx.appcompat.widget.AppCompatImageView
	android:id="@+id/mapImage"
	android:layout_width="match_parent"
	android:layout_height="150dp"
	app:layout_constraintEnd_toEndOf="parent"
	app:layout_constraintStart_toStartOf="parent"
	app:layout_constraintTop_toTopOf="parent" />

result  (small zoom)


Upload a whole (tree of) folders

osf upload only works with single files.
Add support to allow the user to upload a folder and its contents or even a whole tree of folders.

This feature in my mind would add a huge amount of usability to this cli. My personal use case for this is that I have lots of small files in multilevel subdirectories in a project and I'd want to upload them all at the same time. I am very excited for this feature to be implemented.


Orientation issue

Summary
When i rotate the device i am getting the wrong size and it showing half.
The below is the code for reading the ebook file.

So please check and provide solution ASAP.
What platform(s) does this occur on?
Android, iOS
Environment
<EMAIL_ADDRESS>Minimal reproducible example
Just rotate the device landscape to PORTRAIT and PORTRAIT to LANDSCAPE.

You're using widthand height prop? Please show me the code


Add Redunda support

Add support to monitor the bot status in Redunda. There's already a Python library for that called pyRedunda

This was resolved in the commit https://github.com/SOBotics/FlaggersHall/commit/1f05f93e786e2224c623f5cdd51696bfa3e2c2e7


false positive partial-no-import

Example
@import "_bootstrap/variables";
@import "_font-awesome/variables";

What is the name of the file that has those @imports?

@dryoma _variables.scss

Well, that's no false positive. What this rule prohibits is not importing some partials (that would be strange), but using imports inside partials.

@dryoma i think it is false positive, because if i remove _ from _bootstrap name directory, rule works fine.

Sorry, couldn't look into this for a while.
So I tested these scenarios:
file: _file.scss, code:
@import "_bootstrap/variables";
@import "_font-awesome/variables";

2 warnings, 'cause _file.scss is a partial and filenames from @improts don't matter.
file: _file.scss, code:
@import "bootstrap/variables";
@import "font-awesome/variables";

2 warnings, 'cause _file.scss is a partial and filenames from @improts don't matter.
file: dir/_file.scss, code:
@import "bootstrap/variables";
@import "font-awesome/variables";

2 warnings, 'cause dir/_file.scss is a partial and filenames from @improts don't matter.
file: _dir/file.scss, code:
@import "bootstrap/variables";
@import "font-awesome/variables";

0 warnings, 'cause _dir/file.scss is not a partial.
All the results I got are as expected. So I'm really confused what went wrong in your case.

@dryoma yes, you are right, inattentively I read the rule and expect another behavior :sob:

I see :) BTW if you want to control the names of imported files, you might want to check out at-import-no-partial-leading-underscore, at-import-partial-extension-blacklist and at-import-partial-extension-whitelist.

@dryoma thanks :smile:


[bug] Deprecated types don't trigger validators

Describe the bug
When using a deprecated type in an XML rail, and attaching a validator to it, it correctly prints a deprecation UserWarning, but does not trigger the actual validator.
To Reproduce

Try this code snippet

import json

from guardrails import Guard

llm_output = {
    "query": "selecadast 1;",
}

guard = Guard.from_rail_string(
"""
<rail version="0.1">
<output>
    <sql
        name="query"
        description="a query"
        format="bug-free-sql"
        on-fail-bug-free-sql="reask"
    />
</output>

<prompt>
noop
</prompt>
</rail>
"""
)
validated_output = guard.parse(
    llm_output=json.dumps(llm_output),
    num_reasks=1,
)
assert validated_output == llm_output

It'll pass just fine. If you put a breakpoint in the bug-free-sql validator, you'll see that it doesn't trigger it. Also, try replacing sql with string, and you'll see an "API must be provided" error will be thrown (because the reask was triggered).
Expected behavior
Deprecated types should print the UserWarning as they do now, then be treated as the type they should be refactored to (in this case string).
Library version:
0.2.5 (maybe earlier too?)

@irgolic Does this bug persist with the new version bump? I don't get any error/warning when I try it.


Cannot build solution in VS 2019.

Brief description of your issue
I experience problems building the solution with tag v1.2.10271. I get errors like the following:
Specified EntryPointExe 'winget.exe' was not found in the project outputs.

Steps to reproduce

Install the prerequisites:

Windows 10 1809 (17763) or later
Developer Mode enabled
Visual Studio 2019
The following workloads:

.NET Desktop Development
Desktop Development with C++
Universal Windows Platform Development

The following extensions:

Microsoft Visual Studio Installer Projects

Download repository with tag v1.2.10271
Open Visual Studio 2019 and build the solution

Expected behavior
Build the application.
Actual behavior
I get errors:
Specified EntryPointExe 'winget.exe' was not found in the project outputs.

Is there something that's not listed that I need or do I need to set some option when building?
Environment
Windows: Windows 10 Pro v19044.1645
Visual Studio: Visual Studio Community 2019 v16.11.12

What project do you have set as your Startup Project? It should be set to "AppInstallerCLIPackage" for the app to start right:

What project do you have set as your Startup Project? It should be set to "AppInstallerCLIPackage" for the app to start right:

It wasn't that but your screenshot pointed out another thing. I had arch set to "Any CPU", setting it to "x64" I could build. Thanks!


NIFI-12149 Create nifi-redis-utils and minor improvements to util methods

This is the 1.x version of #7812

Cherry-picked manually


Add character n to banner

Added character 'n' to font 'banner'
Referenced Issues
closes #497

@ben-dh3 please request for assignment before opening up a PR. Otherwise, it creates a lot of confusion for use. I hope you understand. So please ensure this for further contributions. 😊


Fasttext or fasttextmirror?

In requiremements fasttextmirror is required, however, the import statements require fasttext

I will remove fasttextmirror from requirements.txt and replace it with fasttext. Note that both packages should not be present at the same time (see this conversation on Google Groups and this GitHub discussion).
Besides that requirements.txt should be updated so as to include spaCy v3.2 and the associated version of en_core_web_sm.


Add Kind support

[ ] ksail up kind
[ ] ksail up kind flux
[ ] ksail down kind
[ ] pull-through registries
[ ] config file?

I have decided to support just one Kubernets in Docker backend for now, as supporting more than one does not really provide much value to a local environment. I feel the best solution is K3d right now as it is thin, and feature rich. So I will implement that as the sole backend.


Update README

Added additional installation instructions with uv and uv tool #18
Include playwright install step #21
Removed mentions of Selenium

Are there any steps or topics that I need to make more clear with this README revision?

@bbelderbos
Made some revisions and also removed the YouTube link from the pyproject.toml file.


metadata schema viewer at collection level

get rough draft generated before wed for iteration.

@hoodriverheather  at the bottom of each collection page in test. However, I made some tabs automatically available for all the collections and some collection specifc. Here is the mapping, can you please confirm what mappings you want to change it to? Also, I know there's some sorting issues where there's empty rows and rows may not be in order, I can certainly fix those!
Key is the collection_id, value is the list of sheet names pulled from your google sheet we want to display per collection.
{
"Microbial": ["Readme","Core Collection-V 1.0","Core File V 1.0","CoreILSExtension","CoreMicrobial V 1.0", "CFU V 1.0", "ImpedanceFC V 1.0", "FluorescenceFC V 1.0","SharedMetadata","Structure"],
"Cell_Line_Provenance": ["Readme","Core Collection-V 1.0","Core File V 1.0","CoreILSExtension","CellLineExpansionExtension DRAFT", "TelemetryExtension DRAFT","SharedMetadata","Structure"],
"Genome_Editing_Consortium": ["Readme","Core Collection-V 1.0","Core File V 1.0","CoreILSExtension","GenomicSequencingExtension","GenomicsBioinformaticsExtension","BionanoExtension","SharedMetadata","Structure"],
"NIST_Flow_Cytometry_Standards_Consortium": ["Readme","Core Collection-V 1.0","Core File V 1.0","CoreILSExtension","FlowCytometryExtension DRAFT","SharedMetadata","Structure"]
}

@yuliujpl this is a good start.
You can remove from all collections the following tabs:

Readme
Shared metadata
Structure

For Microbial you can also remove the CoreILSExtension
Genome_Editing looks good.
Flow Cytometry remove CoreILSExtension
Cell_Line - i would hold off on anything for now. It's all very draft or metadata i made up :)
I also am having a difficult time scrolling through the "sheets".
It would be great to show the required fields at the top.
When you demo, can you mention that some of these are more draft than others. Thanks!

Sounds great!

From: hoodriverheather @.>
Sent: Tuesday, May 14, 2024 5:20 PM
To: Labcas-NIST/Labcas-ui @.>
Cc: Liu, David (US 3902-Affiliate) @.>; Mention @.>
Subject: [EXTERNAL] Re: [Labcas-NIST/Labcas-ui] metadata schema viewer at collection level (Issue #134)
@yuliujplhttps://urldefense.us/v3/__https://github.com/yuliujpl__;!!PvBDto6Hs4WbVuu7!PyN6P9KwtjZ0kzAqXZbv5oue3frCeoKtiz-W_1InHpSseUWw2pT_Wy36_USD159L2xGaBjHd_8HrQCyDD9lcvpNiI8IVhA$ this is a good start.
You can remove from all collections the following tabs:

Readme
Shared metadata
Structure

For Microbial you can also remove the CoreILSExtension
Genome_Editing looks good.
Flow Cytometry remove CoreILSExtension
Cell_Line - i would hold off on anything for now. It's all very draft or metadata i made up :)
I also am having a difficult time scrolling through the "sheets".
It would be great to show the required fields at the top.
When you demo, can you mention that some of these are more draft than others. Thanks!
—
Reply to this email directly, view it on GitHubhttps://urldefense.us/v3/__https://github.com/Labcas-NIST/Labcas-ui/issues/134*issuecomment-2111365557__;Iw!!PvBDto6Hs4WbVuu7!PyN6P9KwtjZ0kzAqXZbv5oue3frCeoKtiz-W_1InHpSseUWw2pT_Wy36_USD159L2xGaBjHd_8HrQCyDD9lcvpNrHPzB0Q$, or unsubscribehttps://urldefense.us/v3/__https://github.com/notifications/unsubscribe-auth/ADXFXXEPVR2Y65JMYFNNYMLZCKS4FAVCNFSM6AAAAABHVBUNE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJRGM3DKNJVG4__;!!PvBDto6Hs4WbVuu7!PyN6P9KwtjZ0kzAqXZbv5oue3frCeoKtiz-W_1InHpSseUWw2pT_Wy36_USD159L2xGaBjHd_8HrQCyDD9lcvpPzPuBF8Q$.
You are receiving this because you were mentioned.Message ID: @.***>


Add PyTorch with RawKernel example to docs

Blocked by #5970
Closes #5833

pfnCI, test this please.

Jenkins CI test (for commit 3247237fe11d6a8a59bc82c24a7034b4fd28a372, target branch master) succeeded!


Add code highlighting to Readme

Good addition, easy PR to merge.  Please accept the review and accept the CLA and this can be merged.

I’d rather not hold up this change, so I’m closing this PR as I don’t have time to review the CLA right now.

No worries.  I'll get this change in from our end.  Cheers!

Merged: https://github.com/twitter/ios-twitter-network-layer/commit/64eda7b5162c356cf639dd160f8b3dc2909dfa99


Adjust endpoint-not-unregistered log-level to 'info'

A short explanation of the proposed change: Log on info-level if endpoint-unregisters are failing

An explanation of the use cases your change solves: have more insights on which endpoints could not be unregistered/trigger a discussion why this was made debug

[x] I have viewed signed and have submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using scripts/run-unit-tests-in-docker from routing-release.

[ ] (Optional) I have run Routing Acceptance Tests and Routing Smoke Tests on bosh lite

[ ] (Optional) I have run CF Acceptance Tests on bosh lite

👍 bumped in routing-release


Files not opening File location from Task manager.

Describe the bug
Files not opening File location from Task manager.
To Reproduce
Steps to reproduce the behaviour:

Open Task Manager
Right click on any task
Click on 'Open File location'

Expected behaviour
Files should open the file location of the task.
Desktop

OS Version: Windows 10 20H2 19042.746
App version: <IP_ADDRESS>

Log file
debug.txt

@Trustery Did you follow the documentation to set Files as the default file manager?

@Trustery Did you follow the documentation to set Files as the default file manager?

Yes I did. Files is working for anything else, but when I click on "Open File location" in task manager you can see In the taskbar that Files open briefly but close immediately afterwards.

Yes I did. Files is working for anything else, but when I click on "Open File location" in task manager you can see In the taskbar that Files open briefly but close immediately afterwards.

This may be a limitation with the way Task Manager works, @gave92 would oyu happen to more on this?

This may be a limitation with the way Task Manager works, @gave92 would oyu happen to more on this?

@yaichenbaum Interesting, task manager tries to start Files with administrator privileges! But that sadly fails :)

@yaichenbaum Interesting, task manager tries to start Files with administrator privileges! But that sadly fails :)


vendor: migrate to glide

etcd is migrating vendor from godeps to glide, maybe we should investigate it and to see whether it is better than godeps.
/cc @ngaut @qiuyesuifeng @huachaohuang @overvenus

refer https://github.com/coreos/etcd/pull/6163


Overwritting someone elses reviews.

Sometimes people give a negative review to a crate, and we conciously would like to overwrite them.
Current example. Me and other crev user reviewed smallvec and gave it a negative review, because it was ridden with unsafe and had a history of unsoundness. Then, I submitted a fuzzer to smallvec, and I think with a fuzzer we can have more confidence, and this crate does not deserve a negative rating anymore. At very least I don't want it to fail in my own crate verify output.
So I'd like to overwrite the other persons (which I still trust) review in this case. I could ask them to change their review, but that's not the point.
I am thinking, that a review could have an optional overrule/overwrite/override section with a list of reviews of other other people that should be ignored for some reason, because this review says they no longer apply, were a mistake, etc.
We could identify other reviews by:

by id of their author
digest
signature

The id is the longest, but is the easiest for the humans to understand and use, so I think I'll stick with it.
For other users my override would only apply if they trust me more than then a person I'm trying to override here (more or equal maybe?).
Should be fairly easy to implement, though I could use some second opinion.

maybe also have the overriding condition be dependent on if the overridden reviewer trusts the overrider? We want to avoid override wars, so, unless special circumstances apply (very high level of trust, trust relationship between overrider and overridee, etc.), I don't think overrides should apply for cases of equal trust.

also would be nice to have local overrides that aren't published.

Who would be affected by an override/overwrite?
"Overwriting" sounds like changing someone else's review globally, which surely isn't what you mean. "Override" sounds like it could somehow affect the trust status of the review globally (i.e. even for users who don't have a trust relationship with the overrider), but that the review itself would still exist.
Conversely, "ignore" sounds like it would not affect anyone else, even if they have a trust relationship with the user doing the ignoring.
I'm guessing you mean something between these two extremes, i.e., ignoring a review would break transitivity, so that users who trust your reviews would not automatically have any trust in that specific review. Maybe "hide" would be a better term?
Another way of thinking about this is that you can trust someone, but disagree with them on a certain issue. In this case, you trust the reviewer in general (and used to agree), but believe the situation has changed and no longer agree with their (or your old) assessment. So this sounds like a mechanism for more granular levels of trust. Maybe it should be possible to just set trust levels for particular reviews directly, which would handle this case as well as the opposite (where you agree with a specific review but don't trust the reviewer in general).

@BatmanAoD

Another way of thinking about this is that you can trust someone, but disagree with them on a certain issue. In this case, you trust the reviewer in general (and used to agree), but believe the situation has changed and no longer agree with their (or your old) assessment.

This is exactly the motivation. I still trust the other reviewer (otherwise I would just change their trust level to none and be done with it), but eg. can not wait for them to re-review, reconsider, or just in this one case / in my case I want an exception.

"Overwriting" sounds like changing someone else's review globally,

Right now I think about "just ignore this one particular review".

We want to avoid override wars, so

I am not really concerned about overriding wars. If people have deep disagreements that they can't resolve, they can just stop trusting each other, and possibly other people should too. :D . This feature is purely for tactical, one time exceptions from otherwise mutually respecting parties.
@programmerjake

also would be nice to have local overrides that aren't published.

Good idea. I wonder if it should be "don't publish" or just "don't get into effect for other people".
Also, you just reminded me that I'd like this ignore (I think it's more of an ignore than override), to apply only to a specific version of a review. If the ignored author redid their review, it would require redoing current review to ignore it again.

Another interesting question this brings up is whether reviews should be set to auto-expire at some point, e.g. on some kind of SemVer bump.
If your and the other reviewer's original reviews had been restricted to the current minor version, the newly fuzzed version would have been implicitly "unreviewed" by both of you.

@BatmanAoD Reviews are version specific right now. New version is unreviewed by definition. Previous reviews can be used as a base for differential reviews, etc. but that's about it.

That makes a lot of sense (and, prior to this issue, is how I assumed it worked), but in that case I don't understand how the smallvec issue arose. Did the smallvec version number not get bumped after you submitted the fuzzer?

@BatmanAoD Not yet. There is no new release. Technically the actual code people use is the same. I just have more confidence in it than before, because my fuzzer did not catch anything.

I was thinking, how could I achieve the same effect without introducing a completely new, one-off feature.
First, this field does not really fit well in package review. It seems more of a trust-level exception.
Maybe trust proofs could have conditional, or exception, or something. But that makes the trust set context-dependent, which is not great.

I think some form of this is needed. People will make mistakes and make wrong reviews. Currently the only way to ignore a wrong review is to distrust the author, but distrusting a person for one wrong review is a nuclear option, a creates a "too big to fail" problem.
Non-code review websites have a "Helpful/Unhelpful" buttons for reviews. Such thing could be exposed in a crev GUI as well.

Referring to a review is a tricky problem. If it's identified by content/signature, then small edit of the review will make it dodge other people's metadata about it. If it's identified by (crev id, package, version), then metadata will still apply even if someone flips review from negative to positive.
So I think reviews should be identified by (crev id of the reviewer, package, version, rating).

So I suggest having a vote proof type. You can vote yes/no for a review. Votes will be weighed by trust of who cast the vote. Reviews that cross some threshold of downvotes will be ignored.

Related: #329


Please Update the react version so that It can support React 19 RC.

When I try to install nextjs-toploader in NEXT 15 (React 19), it shows Peer dependencies error, and even tho I may install it with --force flag in locahost, but when in production, it crashes.

Can you please provide the error reproduction, and hosted vercel deployment can help me figure it out

Error: Failed to load external module nextjs-toploader/pages: Error: Cannot find module 'next/dist/server/route-modules/pages/vendored/contexts/head-manager-context'
Require stack:

/Users/d/Code/proj/node_modules/next/dist/client/script.js
/Users/d/Code/proj/node_modules/next/dist/shared/lib/router/router.js
/Users/d/Code/proj/node_modules/next/dist/client/router.js
/Users/d/Code/proj/node_modules/next/router.js
/Users/d/Code/proj/node_modules/nextjs-toploader/dist/pages.js
/Users/d/Code/proj/apps/agency-frontend/.next/server/chunks/ssr/[turbopack]_runtime.js
/Users/d/Code/proj/apps/agency-frontend/.next/server/pages/_document.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/server/require.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/server/load-components.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/build/utils.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/build/swc/options.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/build/swc/index.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/build/analysis/parse-module.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/build/analysis/get-page-static-info.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/server/lib/router-utils/setup-dev-bundler.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/server/lib/router-server.js
/Users/d/Code/proj/apps/agency-frontend/node_modules/next/dist/server/lib/start-server.js


Add link on my GitHub account in index.html

Hi @Melancholic thnx for contributing, Just follow me on Github and start his repo and participate in Hacktoberfest in the month of October to win cool swags.


Dev

@lpratalimaffei Looks good! I will merge this and fix the linting complaint.
In the future, you can run ./old-lint.sh (soon to be replaced with ./lint.sh) to check for linting issues before submitting the PR.


Can't read new config file

When I use FileUtils.touch to create a new config file (empty yaml file) in specified directory and read it, I got an error:
3: from /usr/lib/ruby/gems/2.6.0/gems/tty-config-0.3.1/lib/tty/config.rb:393:in `read'
2: from /usr/lib/ruby/gems/2.6.0/gems/tty-config-0.3.1/lib/tty/config.rb:273:in `merge'
1: from /usr/lib/ruby/gems/2.6.0/gems/tty-config-0.3.1/lib/tty/config.rb:559:in `deep_merge'
/usr/lib/ruby/gems/2.6.0/gems/tty-config-0.3.1/lib/tty/config.rb:559:in `merge': no implicit conversion of nil into Hash (TypeError)

Released v0.3.2 with a fix.


Clippy fixes fixes

This is a touch-up of #2 in order to fix the browser tests.
@fabricedesre if you are okay with it I can merge in your contribution with this PR.

Ended up landing these changes in #2 after all.


Mojave 10.14.4

I'm using your latest clover version in mac os 10.14.3 and i have seen that apple released 10.14.4, have you tried to upgrade to this, i'm confusing if my gl553vd will work well when i take this

Hi .
Yes . You can update . I'm using Mac os 10.14.4 with out problem.

Hi @tuanngokien, ông cài đặt macos như thế nào vậy, t có dùng Niresh Mojave cài nhưng có vẻ như nó bị lỗi nhiều ở phần hình ảnh, k biết thay Clover của anh bạn này có dùng được không nhỉ?

I should try this
https://git.lug.ustc.edu.cn/QiuChenly/ASUS_FX53VD_10.13.1EFI
I know FX553VD is almost the same hardware but there are still some difference that would make some bugs while experience. Otherwise, you should build Installation yourself or download it from olarila (instead of zone), and also join VN hackintosh on FB for perfecting your Hackintosh


Auth Wrapper

@asim thanks for fixing the cmd.Init bug. I've tested this morning and all is looking good now, can you please let me know if you're happy to merge.

LGTM will merge


Use Versioneer.

Fixes #11.

This change is

Reviewed 5 of 10 files at r1.
Review status: 5 of 10 files reviewed, 4 unresolved discussions (waiting on @jonathanj)

.coveragerc, line 16 at r1 (raw file):

[report]
omit =
    txdocumint/_version.py

Missing newline.

tox.ini, line 9 at r1 (raw file):

deps =
    .[test]
    twlatest: Twisted[tls]

This shouldn't do anything since Twisted is already a dependency.

tox.ini, line 13 at r1 (raw file):

commands =
    pip list
    mkdir -p {envtmpdir}

mkdir needs to be whitelisted since it's not something we're installing.

tox.ini, line 15 at r1 (raw file):

    mkdir -p {envtmpdir}
    coverage run --parallel-mode \
        {envdir}/bin/trial --temp-directory={envtmpdir}/_trial_temp {posargs:txdocumint}

I think the txdocumint in $CWD is being imported instead of the installed copy. Maybe try coverage run … -m twisted.trial …?

Comments from Reviewable

Review status: 2 of 17 files reviewed, 4 unresolved discussions (waiting on @mithrandi and @jonathanj)

.coveragerc, line 16 at r1 (raw file):
Previously, mithrandi (Tristan Seligmann) wrote…
Missing newline.

Done.

tox.ini, line 9 at r1 (raw file):
Previously, mithrandi (Tristan Seligmann) wrote…
This shouldn't do anything since Twisted is already a dependency.

I added some more Twisted versions to the environment matrix.

tox.ini, line 13 at r1 (raw file):
Previously, mithrandi (Tristan Seligmann) wrote…
mkdir needs to be whitelisted since it's not something we're installing.

Done.

tox.ini, line 15 at r1 (raw file):
Previously, mithrandi (Tristan Seligmann) wrote…
I think the txdocumint in $CWD is being imported instead of the installed copy. Maybe try coverage run … -m twisted.trial …?

Moving the source into a src directory seems to have resolved this.

Comments from Reviewable

Reviewed 15 of 15 files at r2.
Review status: :shipit: complete! all files reviewed, all discussions resolved

Comments from Reviewable


Report unused scripts and scenes

Here is another idea for a new feature:
My config gets cluttered with unused scenes and scripts. Not very harmful, but a mess nevertheless.
I guess if you have a scene or script it should get used by an automation. What else is the use of them?
So, long story short: is it possible to list all unreferenced scenes and scripts in the report?

The feature is very interesting because you can also find orphaned entities that aren't used anywhere.
F.e. find any old entities from deleted integrations, removed devices, old groups and so on.
I've some of them and i would like to have such a feature!

Related to #26


Fix/bool cast

Fixes the boolean casting in SPIR-V to match other backends in treating all non-zero values as true, instead of just 1.
Fixes plane_all and plane_any on SPIR-V by unrolling vectorized values, since the SPIR-V instruction only supports scalar output.
Fixes various test failures resulting from other changes.

@wingertge After fixing the merge conflicts we can merge!


Timeout on the Lambdas leave no log about the problem

We got some timeouts from the Lambda node_api_get_transaction. When a timeout happens in the Lambda, we have no way to know what exactly happened.
To improve this, we should configure a timeout in the client that is lesser than the Lambda timeout, so it times out earlier and logs something about the error.

The AWS Lambda Function timeout is configured by default with 6 seconds in Serverless. That said, the client should have timeout of 5 seconds.