Flutter web run as Profile not working

Steps to Reproduce

Create flutter web project
Debug console showed as below

Run flutter profile with Chrome as device
`Running "flutter pub get" in test_keyboard...
Launching lib\main.dart on Chrome in profile mode...
lib\main.dart:1
Compiling lib\main.dart for the Web...

Chrome opened, but DevTools not opened, the "Compiling" text never dismiss also

Expected results:
Flutter Dev Tools should be open
Actual results:
Flutter Dev Tools not open

flutter doctor -v
[√] Flutter (Channel beta, 2.6.0-5.2.pre, on Microsoft Windows [Version 10.0.19042.1110], locale en-SG)
    • Flutter version 2.6.0-5.2.pre at D:\Project\Flutter\sdk
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 400608f101 (8 days ago), 2021-09-15 15:50:26 -0700
    • Engine revision 1d521d89d8
    • Dart version 2.15.0 (build 2.15.0-82.2.beta)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at C:\Users\billy\AppData\Local\Android\sdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.60.2)
    • VS Code at C:\Users\billy\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.26.0

[√] Connected device (2 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.54
    • Edge (web)   • edge   • web-javascript • Microsoft Edge 93.0.961.52

! Doctor found issues in 1 category.

Hi @billyinferno

Chrome opened, but DevTools not opened,

This is not expected behaviour, devtools don't open by itself
When running in debug mode, we can get a devtools links but for some reason, it is not printed in profile mode. I also running beta channel,  Compiling doesn't get stuck

logs
	[  +72 ms] Running "flutter pub get" in myapp... (completed in 1,237ms)
[ +145 ms] Generating
/Users/tahatesser/StudioProjects/myapp/android/app/src/main/java/io/flutter/plug
ins/GeneratedPluginRegistrant.java
[ +187 ms] Launching lib/main.dart on Chrome in profile mode...
[ +123 ms] Compiling lib/main.dart for the Web...
[   +7 ms] Initializing file store
[  +39 ms] web_entrypoint: Starting due to {InvalidatedReasonKind.inputChanged:
The following inputs have updated contents:
/Users/tahatesser/Code/flutter_master/packages/flutter_tools/lib/src/build_syste
m/targets/web.dart}
[   +1 ms] Skipping target: gen_localizations
[   +8 ms] web_entrypoint: Complete
[   +1 ms] dart2js: Starting due to {}
[+24963 ms] dart2js: Complete
[ +682 ms] web_release_bundle: Starting due to {}
[ +153 ms] web_release_bundle: Complete
[ +447 ms] web_service_worker: Starting due to {}
[  +89 ms] web_service_worker: Complete
[ +104 ms] Persisting file store
[   +7 ms] Done persisting file store
[  +10 ms] Compiling lib/main.dart for the Web... (completed in 26.5s)
[  +61 ms] Using Google Chrome 94.0.4606.61

[  +92 ms] executing: sysctl hw.optional.arm64
[   +7 ms] Exit code 1 from: sysctl hw.optional.arm64
[        ] sysctl: unknown oid 'hw.optional.arm64'
[ +357 ms] [CHROME]:
[   +1 ms] [CHROME]:DevTools listening on
ws://<IP_ADDRESS>:62341/devtools/browser/7ecb2aa9-be66-4023-82cc-5b05bef1b601
[+1076 ms] 🔥  To hot restart changes while running, press "r" or "R".
[   +3 ms] For a more detailed help message, press "h". To quit, press "q".

Can you please provide complete logs from flutter run -d chrome --profile
Thank you

Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.

Try the new cross-platform PowerShell https://aka.ms/pscore6

PS D:\Project\my_expense> flutter run -d chrome --profile
Launching lib\main.dart on Chrome in profile mode...
Compiling lib\main.dart for the Web...                             56.5s

  To hot restart changes while running, press "r" or "R".
For a more detailed help message, press "h". To quit, press "q".

Chrome showed up, but DevTools not showed up, when I press the DevTools on VSCode it open the browser (http://<IP_ADDRESS>:9101/#/?), and showed this:

Hi @billyinferno
I didn't know about this https://github.com/flutter/flutter/issues/77975 but it is supposed to open devtools when you run in profile mode and the Devtools link doesn't get printed in profile mode.
Coming to this issue
Chrome opened, but DevTools not opened, the "Compiling" text never dismiss also
I can reproduce when running from IDEs, (VS Code and Android), the chrome window is launched, but the tool process is stuck at compiling on all platforms (macOS, Windows, Linux) and when you attempt to open devtools. I can also reproduce this issue with multiple Flutter channels, including older stable.
From the command line, the app is launched, Compiling doesn't get stuck , however, there is devtools doesn't get launch either
Steps to reproduce

flutter create myapp
Open my app in VS Code or Android Studio
Run web in profile mode

Android Studio (performance tab says no app available)
https://user-images.githubusercontent.com/48603081/134879408-45075205-a11e-4ddf-b777-9baead3a9541.mov
VS Code (devtools desn't launch)
https://user-images.githubusercontent.com/48603081/134879403-99552ceb-0ca7-43f7-83eb-1be443db63ed.mov

stable
master

❗
❗

Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.1, on macOS 12.0 21A5522h darwin-x64, locale en-GB)
    • Flutter version 2.5.1 at /Users/tahatesser/Code/flutter_stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffb2ecea52 (10 days ago), 2021-09-17 15:26:33 -0400
    • Engine revision b3af521a05
    • Dart version 2.14.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.0, Build version 13A233
    • CocoaPods version 1.10.2

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

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

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

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.61

• No issues found!

[✓] Flutter (Channel master, 2.6.0-12.0.pre.98, on macOS 12.0 21A5522h darwin-x64, locale en-GB)
    • Flutter version 2.6.0-12.0.pre.98 at /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 48e6758fb5 (8 hours ago), 2021-09-26 21:28:03 -0400
    • Engine revision 65ad5a4836
    • Dart version 2.15.0 (build 2.15.0-150.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

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

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

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

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.61

• No issues found!

✅: No Issue ❗: Issue reproduced
Thanks for filing the issue

Closing this as a duplicate of https://github.com/flutter/flutter/issues/80334.


Custom error pages

Fixes #47. Also, I'm not sure about the parameters for the error page, since this way to deal with the error pages is quite plain, but I assume it's a rare case to have it non-static.
Suggestions are welcome.
While I was fixing tests, I noticed we don't set charset in code nor tests. Maybe it'll be better to fix it?

thanks :dancer:


[SPARK-28786][DOC][SQL][FOLLOW-UP] Change "Related Statements" to bold

What changes were proposed in this pull request?
Change "Related Statements" to bold
Why are the changes needed?
To make doc look nice and consistent.
Does this PR introduce any user-facing change?
Yes
How was this patch tested?
Tested using jykyll build --serve
Before the change:

After the change:

Before the change:

After the change:

@gatorsmile
One minor follow-up change. Thank you very much in advance!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14942/
Test PASSed.

Test build #109915 has started for PR 25623 at commit 771a236.

Test build #109915 has finished for PR 25623 at commit 771a236.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109915/
Test PASSed.

Test build #109934 has started for PR 25623 at commit ac0b9e9.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14961/
Test PASSed.

Test build #109934 has finished for PR 25623 at commit ac0b9e9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109934/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14982/
Test PASSed.

Test build #109961 has started for PR 25623 at commit 37e2f5f.

Test build #109961 has finished for PR 25623 at commit 37e2f5f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109961/
Test PASSed.

LGTM

Thanks! Merged to master.

Thanks all for your help! @srowen @dilipbiswal @gatorsmile


JSON Loader

What would you like to be added:
Support for Langchain JSONloader:
Why is this needed:
Financial data frequently comes in JSON format
Anything else we need to know?
In a RAG request, reading the JSON as a text file does not return the key-value pairs

Mind if I pick this up? Happy to take a stab at it if it's still available 🙂


makeStyles cannot create psudo element styles using a function

Current Behavior 😯
makeStyles allows for supplying a function to generate a style value object. However when using a psudo element this does not work
Expected Behavior 🤔
a psudo element value should be able to be created with a dynamic function
Steps to Reproduce 🕹
const useStyles = makeStyles((theme) => ({
  mainContainer: {
    display: 'flex',
    '&::after': () => ({ // This is not getting built. If i put the plain object and not the function it works.
      content: "'.'",
      position: 'absolute',
      top: 0,
      left: 0,
      width: '100%',
      height: '100%',
      backgroundColor: 'red',
      opacity: .7,
    })
  },
 }))

Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
"@material-ui/core": "^4.11.0"

React
"react": "^16.13.1"

Browser
Chrome

Actually this is probably known / desired behavior.
Although you cannot have a function create the whole style object, you can have functions defining the values within it.
So, with my example, this works
const useStyles = makeStyles((theme) => ({
  mainContainer: {
    display: 'flex',
    '&::after': {
      content: "'.'",
      position: 'absolute',
      top: 0,
      left: 0,
      width: '100%',
      height: '100%',
      backgroundColor: () => 'red', // this works fine.
      opacity: .7,
    }
  },
 }))


[Telemetry] Design V1 Devnet Dashboard - Issue #195

Description
Submitting recommendations for an MVP of a public-facing devnet telemetry dashboard.
Issue
Fixes #195
Type of change
Please mark the relevant option(s):

[ ] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Major breaking change
[x] Documentation
[ ] Other

List of changes
All recommendations are documented here.

Recommended 10 primary metrics to track as part of a devnet dashboard
Outlined data points required with various visualization types
Provided information (that needs to be confirmed) about sources for data points
Provided mock ups for potential dashboards based on visualization types
MISSING

What is queried vs. what is logged
What is raw vs. what is analyzed
The size of the dataset

Required Checklist

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have tested my changes using the available tooling
[ ] I have updated the corresponding CHANGELOG

If Applicable Checklist

[ ] I have updated the corresponding README(s); local and/or global
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added, or updated, mermaid.js diagrams in the corresponding README(s)
[ ] I have added, or updated, documentation and mermaid.js diagrams in shared/docs/* if I updated shared/*README(s)

Wrong branch. Please ignore.


example crud operations

Would be nice to have an example for each CRUD operation.
Specifically I'm trying to find out how to update the DOM after deleting an item in the list on the page.

If your items are contained in an array and your view is bound to the array, removing the item from the array will update the DOM.
Example:
view-model:
export class Order {
  items = [];

  deleteItem(item) {
    // [ajax call to delete the item]

    // remove the item from the array (which will be reflected in the view)
    this.items.splice(this.items.indexOf(item), 1);
  }
}

<template>

Order

  ...

Items:

      <button click.delegate="$parent.deleteItem(item)">Delete</button>
      ${item.name}

</template>

Please hit up the gitter chat- response time will be quicker- there are over 1200 members:  https://gitter.im/Aurelia/Discuss


low pass filter in fits

Dear author,
your paper is very strong. and i have read many times.
but I have a question about lowpass filter in models.
in the code, u use  low_specx[:,self.dominance_freq:]=0 # LPF.
i know some signal processing knowledge. if we just do this, it will have some problem, like endeffect.
i do not look for this. there is some signal processing method, like lowpass filter, in scipy.
can we use it directly? lowpass filter in scipy is based on CPU.
if I add lowpass filter in scipy, will it work on GPU?

Hi. Thanks for taking interest in our work! And thanks for your insightful question.
Yes, we are also aware of some signal distortion problem with such a simple ideal filter, especially in real-life. However, with time series, we find the distortion is neglectable which causes loss way smaller than the forecasting error. Thus, we directly use such a simple filter implementation. I think adding some more carefully designed filter may indeed bring further benefits.
I am not sure if the filters from scipy would work perfectly with torch auto_grad. It may break the gradient back-propogation.
However, it is still a great idea that worth a try. Because, although the LPF is on the gradient back-propagation route, there is no other trainable-parameter before it. So, maybe it does not matter if you just stop the gradient on the LPF. But you have to find a way to do the "de-filter" process and it have to be differentiable (the zero padding in our original FITS).
Hope these can give you some clue.

lowpass filter in Scipy does not have the Pytorch version.
it does not work in CUDA.
Your idea is the most effective.

Sorry, I am not very familiar with the Scipy and EMD. If you really need it, please consider if there is any other third-party pytorch implementation (I find this https://github.com/daerduoCarey/PyTorchEMD ). Or, if you are familiar with the algorithm, you can consider implementing it with pytorch.
Further, it seems that some filter are provided in the torchaudio package. Maybe you can have a check: https://pytorch.org/audio/main/tutorials/filter_design_tutorial.html

Sorry for the abbreviation.
Empirical mode decomposition(EMD) is a signal processing method that separates raw signals into several sub-signals. there is no EMD pytorch package. I am another major. Please give a suggestion should I try to write it in pytorch? or give it up?
yes, thank you very much.
I see the filter in the torch audio package. but it does not process Endeffect.

This really depends on whether you need it. It is your call, bro.

If there is no further question, I will close this issue. Please feel free to reopen it.


Feature/new event format

@tyriis This adds support for new event formats and fixes visibility and click tracking

wonder if this has ever worked


[Question] Don't append ? for optional arguments

Checkhealth
Put result of :checkhealth compe.

health#compe#check
========================================================================
## compe:snippet
  - OK: snippet engine detected.

## compe:mapping
  - OK: `compe#complete` is mapped: (`i  <C-Space>   * compe#complete()`)
  - OK: `compe#confirm` is mapped: (`i  <CR>        * compe#confirm(lexima#expand('<CR>', 'i'))`)
  - OK: `compe#close` is mapped: (`i  <C-E>       * compe#close('<C-E>')`)
  - OK: `compe#scroll` is mapped: (`i  <C-D>       * compe#scroll({ 'delta': -4 })`)

Describe the bug

Navigating to completion for optional attributes auto appends the ? in typescript and have to click enter to clear it.
I am not sure if this is a bug, but will love to hear some input on whether it's possible to not append the ?
Thanks for this amazing plugin!

To Reproduce
Steps to reproduce the behavior with the minimal config:

Install nvim-compe with neovim LSP
Trigger a completion for optional attributes

Actual behavior

When navigating to the optional attributes using tab, it auto appends the ?
Clicking enter will remove the ?

Expected behavior

I am expecting it to not append the ? so that I don't need to click enter to continue typing.

Screenshots (optional)

Additional context (optional)

I have a similar problem with the Scala language server (metals). It looks like the problem occurs, whenever the signature of the completed element is followed by something other than a opening paren. In Scala, this includes fields, where the : separating the  type annotation is inserted:

and functions with type parameters (using square brackets), which are inserted up to the first whitespace, dollar sign or completely if none of those occur:

In both cases, the extra symbols are inserted in the buffer, unless confirmed with <CR>.
The same does not occur, if, in the signature, the name is directly followed by normal parentheses:

Checkhealth:
health#compe#check
========================================================================
## compe:snippet
  - OK: snippet engine detected.

## compe:mapping
  - OK: `compe#complete` is mapped: (`i  <C-Space>   * compe#complete()`)
  - OK: `compe#confirm` is mapped: (`i  <CR>        * compe#confirm('<CR>')`)
  - OK: `compe#close` is mapped: (`i  <C-E>       * compe#close('<C-E>')`)
  - OK: `compe#scroll` is mapped: (`i  <C-D>       * compe#scroll({ 'delta': -4 })`)


No Protections Against XSS Attacks Using innerHTML

If someone were building an app with Hyperapp and wanted to allow for formatting of user-supplied data with HTML instead of Markdown or other such syntax, using the innerHTML attribute adds a XSS vulnerability that we should probably at least warn about if not prevent entirely.
Example pen.
In React this would be prevented with the message:
Warning: Directly setting property `innerHTML` is not permitted. For more information, lookup documentation on `dangerouslySetInnerHTML`.

As the warning implies, the dangerouslySetInnerHTML attribute is React's way of allowing the user to still do this while acknowledging how dangerous it is.

Hmm... My personal feeling: It's not hyperapp's responsibility to protect users from shooting themselves in the foot when they chose to work directly with the DOM. Perhaps a note somewhere in the lifecycle event docs (which is where we expose the DOM directly)
React's approach sounds a lot like it started as a secure-coding-policy internally at Facebook, which they decided to just build into "their" framework as a convenience for themselves.
(Maybe I'm being a little cynical. But really -- innerHTML is not any more "dangerous" then anything else. As always, you just have to know what you're doing when dealing with user supplied data)

@zaceno I'm all for not trying to idiot-proof the API, but it is a very real concern that beginner developers might not be aware of. At a minimum I think we should document the dangers of this gotcha if we don't add any special handling. Especially for users converting from React who rely on the safety features it provides.

Yea in the docs I would follow the same warning as Vue does:
https://vuejs.org/v2/api/#v-html
Simple, clear warning 👍

So, what solutions have we come up with so far?

Disable innerHTML and give you dangerouslySetInnerHTML to do the same.
Document using innerHTML is dangerous.

...

@jbucaran we could also choose to disable innerHTML and not provide any work-around, but that would probably be too disruptive.
My vote right now is for documenting along the lines of: Here Be Dragons 🐲

My vote is documentation.  No need to add the extra complexity in my
opinion.
It's a good candidate for "Dev mode" console warnings in the future also.
On Sun, Sep 10, 2017, 7:22 PM Jorge Bucaran<EMAIL_ADDRESS>wrote:

So, what solutions have we come up with so far?

Disable innerHTML and give you dangerouslySetInnerHTML to do the
same.
Document using innerHTML is dangerous.

...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hyperapp/hyperapp/issues/360#issuecomment-328385905,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMpDr_0CTlLYNChCpcLICkP-z0cZnDiOks5shH1ZgaJpZM4PSbnG
.

Shall we close?
https://github.com/hyperapp/hyperapp/pull/361/files#diff-3f6ea3864d4d859d7715a9d46c40d4e3R56

@okwolf Maybe we can add a Caveats or Warnings section instead and add a few more sentences about XSS and how to sanitize?

Maybe also point out to use textContent instead?
MDN says following:

Differences from innerHTML:
Element.innerHTML returns the HTML as its name indicates. Quite often, in order to retrieve or write text within an element, people use innerHTML. However, textContent often has better performance because the text is not parsed as HTML. Moreover, using textContent can prevent XSS attacks.

https://developer.mozilla.org/en-US/docs/Web/API/Node/textContent

Hmm, I think this situation is a bit different 🤔 But I do appreciate the reference to MDN 🎉
If I have a backend API that returns HTML (markdown to html for example), I can't use textContent for that.
Also in some apps, where the svgs are an API response (you cannot animate a background-image in an SVG with CSS). You must have the encoded (compiled) SVG to be able to use CSS animations. This is where innerHTML is useful as well 🚀
If we add textContent in the docs as an example, we must explain that this is only for replacing a string and that using h/hyperx/jsx solves this problem in the first place 👍
I would rather avoid adding confusion and seeing people use textContent instead of utilizing the VDOM 👍. What do you think @christiankaindl?
I think as long as we explain that using innerHTML without sanitized user content/trusted content is a bad idea.
And if the user is just returning a string, then <span>{state.string}</span> will be all that is needed.
Thanks for bringing it up though! ❤

I think many people don't know about textContent and that in a lot of places they use innerHTML where they just want to insert text. I myself stumbled upon it not long ago and was surprised that I had never seen it before. But this is a very generic issue and it is probably a good idea to prioritize VDOM.
Maybe something like this:

Warning: use of the innerHTML method can lead to cross-site scripting (XSS) vunerabilities if not properly sanitized. If you find yourself using it be careful and/or consider using HyperApp's built in h function (or jsx or hyperx) instead.

thanks @selfup for the nice reply!

FWIW, in domvm's docs i recommend using DOMParser [1] before resorting to innerHTML [2].
[1] https://developer.mozilla.org/en-US/docs/Web/API/DOMParser
[2] https://github.com/leeoniya/domvm#innerhtml

@christiankaindl Thanks for explaining and providing a nice example of what could be used in the docs 🎉
I am glad my reply helped with why using innerHTML is sometimes necessary 😄
Really dig the example you gave 👍

Does hyperapp sanitize text nodes and attributes?

@pspeter3 https://github.com/hyperapp/hyperapp/blob/master/src/app.js#L149
It uses the Dom element setAttribute function.

https://github.com/hyperapp/hyperapp/commit/3ab5a1e483fc0520447b3280c52ec67c84f66ff1
😄


core\framework\op_kernel.cc:1192] Resource exhausted: OOM when allocating tensor with shape[64,256,16,16]

core\framework\op_kernel.cc:1192] Resource exhausted: OOM when allocating tensor with shape[64,256,16,16]
When I  train it??

Please give more details about your experiment environment.
Maybe you can scale down the batch size.

Caused by op 'decoder_6/deconv2/conv2d_transpose', defined at:
File "D:/work/gansgood/DNA-GAN-master/train.py", line 133, in
main()
File "D:/work/gansgood/DNA-GAN-master/train.py", line 128, in main
DNA_GAN = Model(args.attributes, is_train=True)
File "D:\work\gansgood\DNA-GAN-master\model.py", line 26, in init
self.build_model()
File "D:\work\gansgood\DNA-GAN-master\model.py", line 224, in build_model
self.Bxs = [self.joiner('decoder', enc_B) for enc_B in self.enc_Bxs]
File "D:\work\gansgood\DNA-GAN-master\model.py", line 224, in
self.Bxs = [self.joiner('decoder', enc_B) for enc_B in self.enc_Bxs]
File "D:\work\gansgood\DNA-GAN-master\model.py", line 174, in joiner
strides=[1,2,2,1])
File "D:\work\gansgood\DNA-GAN-master\model.py", line 94, in make_deconv_bn
output_shape=out_shape, strides=strides, padding='SAME'

gtx1070

I have tested it in different linux machines but no windows machine. I sorry for being not able to reproduce that. Maybe you can modify the dataset.py and change the batch size to a smaller number (e.g., 16) and start to train with respect to no more than 2 attributes.


nixos-rebuild: switch to tmpDir during rebuilds

Motivation for this change
Workaround issue: https://github.com/NixOS/nixpkgs/issues/144811
The idea of this fix is very simple: since running nixos-rebuild inside a symbolic link directory pointing to /nix/store broke the nixos-rebuild, what we are going to do instead is simply switch to another directory. We already create a temporary directory tmpDir for some things, so we can switch to that directory without much loss.
Keep in mind that this is a workaround. The actual issue seems to be how nix handles references with path:: https://github.com/NixOS/nix/issues/5510
I did a nixos-rebuild switch inside /run/opengl-driver/lib and this workaround works fine. Also tested `nixos-rebuild {dry-build,dry-activate,build-vm,build-vm-with-bootloader,test,edit} commands and they seem to work fine, however I can't test remote builds and other functionality.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This issue affects both release-21.05 and release-21.11 so marking them as backports.

would pushd and popd be better here?

would pushd and popd be better here?

The issue of pushd and popd is that since this script is long another directory change could break the expectations (and ends up in an unrelated directory). This is why I decided to use cd instead.

CC @colemickens @figsoda @tejing1.

CC @SuperSandro2000 @jonringer.

I really would like one more review here to make sure I am not breaking anything.
CC @rnhmjoj @nrdxp @regnat @domenkozar

This seems to break boot.kernelPackages = config.boot.zfs.package.latestCompatibleLinuxPackages causing it to use linuxPackages.

Let's revert it them.


allow files to be included in webpages

Hello everyone,
is it possible to allow a file from a repository to be included in a web page, same as what you can do on github gists?

I think this should be possible by using the link to the "raw" link.


Android build instructions

Do we know why the Android build instructions have been removed in https://github.com/ZerBea/hcxdumptool/commit/23b1586df8aa7702319975bf0632a4f166a80754#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5L129? I know Android is not officially supported in hcxdumptool but then why are the Android build files not removed as well:

https://github.com/ZerBea/hcxdumptool/tree/master/jni
https://github.com/ZerBea/hcxdumptool/blob/master/Android.mk
https://github.com/ZerBea/hcxdumptool/blob/master/AndroidManifest.xml

Is the Android build broken maybe? I am asking because I am planning to try it out in the future.

Android isn't broken.
This instructions have been removed because I assume only experienced users compile hcxdumptool for Android.
An experienced user shouldn't need these instructions.
Next step is to remove Debian instructions, too.

By this commit:
https://github.com/ZerBea/hcxdumptool/commit/d187eed1c7caf2ece511c9aa158e4c426b20169e
Debian instructions have been removed, too.

The same applies to hcxtools:
https://github.com/ZerBea/hcxtools/commit/1a80a1b0d1e504db2b41205f4b1959909359ba43
Thanks that you reminded me.

Please let me know if you were successful (compiling hcxdumptool on Android).
If it doesn't work, I'll remove the entire Android build files.

Ok, there are detailed instructions how to use the NDK:
https://developer.android.com/ndk/guides/other_build_systems
I'll remove the entire Android stuff.

I've opened a discussion:
https://github.com/ZerBea/hcxdumptool/discussions/434


Unsymmetric page-up vs page-down

These 2 lines
https://github.com/Dax89/QHexView/blob/master/qhexview.cpp#L471
https://github.com/Dax89/QHexView/blob/master/qhexview.cpp#L486
do not feel right.
PgDn seems to move the position by half the number of visible rows and PgUp by 3 times.
I think the issue is with "this->firstVisibleLine()" vs "cur->currentLine()",
If you use the latter it works properly.
And if you think about it, there is no symmetry between ArrowUp/Down vs PageUp/Down.
The only difference should be the amount, not the reference point: 1 in the former and this->visibleLines() in the latter.

Fixed at https://github.com/Dax89/QHexView/pull/32

Fixed at https://github.com/Dax89/QHexView/pull/32


[DIG-90] Integrate Social Links into User Profile UI

Description:
Enhance the frontend user profile pages to display social links obtained from the updated backend API. This involves modifying the existing user interface to include fields for LinkedIn, GitHub, Instagram, and Twitter.
Expected Changes:

Update user profile UI components to accommodate social links.
Fetch and display social links from the updated /user API response.
Implement proper styling for the new social links section.
Update /u/account - Account page for allowing users to update links.
Display the links on /u/:profile - User Profile page under the user card

Additional Info:

Ensure responsiveness for various screen sizes.
Implement error handling for missing or invalid social links.
Update relevant documentation for frontend changes.

DIG-90

I would like to work on this issue under kwoc.

Hi @ayann07! Thanks for wanting to contribute. To begin, try setting it up on your computer. If you run into any issues or have questions, feel free to join us on Discord. We're here to assist!

If this issue is still open I would like to work upon it kindly assign it to me under SWOC :)

Hey @ratishjain12, this is still open. Assigned
Join us on Discord for any help.

Is this issue still open @pranshugupta54 . I would like to take this up if it is.

@Chakit22 I am working on it right now :)

@Chakit22 I am working on it right now :)

Cool @ratishjain12 !

Hey @ratishjain12, any progress on this? Join us on discord


chore(fee): gas computation refactor

Changes:

Implemented get_da_gas_cost, which returns a GasVector (depending on use_kzg_da flag).
Deleted calculate_tx_gas_usage_vector, because calculate_tx_l1_gas_usage does the same thing.
Derived derive_more::Add for GasVector to easily sum up gas costs.

This change is

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e5c746b) 70.57% compared to head (bb73580) 70.52%.
Report is 2 commits behind head on main-v0.13.1.

Files
Patch %
Lines

crates/blockifier/src/fee/gas_usage.rs
97.36%
0 Missing and 1 partial :warning:

crates/blockifier/src/transaction/objects.rs
0.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@               Coverage Diff                @@
##           main-v0.13.1    #1408      +/-   ##
================================================
- Coverage         70.57%   70.52%   -0.06%
================================================
  Files                59       59
  Lines              7813     7796      -17
  Branches           7813     7796      -17
================================================
- Hits               5514     5498      -16
  Misses             1871     1871
+ Partials            428      427       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

crates/blockifier/src/fee/gas_usage_test.rs line 46 at r1 (raw file):

})
]
fn test_calculate_tx_blob_gas_usage_basic(#[case] state_changes_count: StateChangesCount) {

Suggestion:
test_get_da_gas_cost_basic

crates/blockifier/src/fee/gas_usage.rs line 228 at r1 (raw file):
Previously, aner-starkware wrote…
I thought we wanted to avoid unwrap, and also this conversion is not type-safe.

nvm - what Arni said


Add sample for help task

A help task should be included in every project to be able to see which tasks are available.

I like gulp-help because it provides the means to customize the help text.


removed createdby attributes and enforced expiry time

This PR removes the feature to expire sessions based on the created_time plus interval.
The feature was designed to enable the duration of a session to be controlled by database configuration rather than potentially each application instance.
While created_time plus interval is a valid use case as it ensures that all instances of an application service have the same session duration the complexity of mixing the functionality with the default application level expiry_at functionality was error prone due to the index management required and added a great deal of complexity to the API in this library.
Specifically:
This library now creates an index on the expiry_at automatically in order to improve developer experience. If we want to apply a different index to support created_time then we have to remove the expiry_at index and ensure it isn't reapplied.
This would mean that DX would suffer as some iniitalise semantics would be mandatory.
If there is interest in this feature in the future I would recommend implementing it as a separate session store type rather than including it in MongodbSessionStore.
I've also put in a conditional on expiry_at  so that expired sessions that haven't been sweeped wouldn't be returned.
@pepoviola Thanks for trying to facilitate the feature but I think sticking with your main implementation makes sense right now.
If you're OK with it I will merge this in.

Hi @No9, agree 👍. Make sense ab improve users expirience.
Thanks!

Thanks ill merge and package tomorrow


Add Ephemeral EBS Support to AWS Spot Instances

Currently, spot instances cannot use newer instance families (e.g. m4, c4) because they do not have Instance Storage. The ephemeral_disk BOSH option provides the capability for On Demand instances, but the RequestSpotInstances call does not provide the equivalent LaunchSpecification.BlockDeviceMappings parameter.
This PR makes two code changes:

As a prerequisite, default_ephemeral_disk_mapping, which was returning a plain hash, is now returning a list of values of type BlockDeviceMapping as recommended by the AWS specifications. The inconsistency of formats between {'sdb':'ephemeral0'} and [{device:'sdb',virtual:'ephemeral0'}] was causing issues when reusing values between different API calls when adding the spot ephemeral support. This also affected the stemcell builder which used the value for the default block device mappings of the image.
Second, forward the block_device_mappings from the regular instance parameters to the spot request.

I updated and added spec tests. For the stemcell builder change, I went through the process of creating a stemcell and reviewed the resultant AMI. I also manually tested the following scenarios (test job utilized a regular, persistent disk and ran from stemcell bosh-aws-xen-hvm-ubuntu-trusty-go_agent/sriov20150630a)...
spot + ebs
# resource pool
$ cat bosh.yml
    cloud_properties:
      availability_zone: "us-west-2b"
      instance_type: "c4.xlarge"
      spot_bid_price: 0.05
      ephemeral_disk:
        size: 8192

# verify ephemeral
$ aws ec2 describe-spot-instance-requests --spot-instance-request-ids=sir-03hqa00c | jq -c '.SpotInstanceRequests[] | ( .InstanceId , .LaunchSpecification.BlockDeviceMappings )'
"i-d6ef0e20"
[{"DeviceName":"/dev/sdb","Ebs":{"DeleteOnTermination":true,"VolumeSize":8,"VolumeType":"standard"}}]

# ebs volumes
$ aws ec2 describe-instances --instance-ids i-d6ef0e20 | jq -c '.Reservations[].Instances[].BlockDeviceMappings[] | .'
{"DeviceName":"/dev/xvda","Ebs":{"Status":"attached","DeleteOnTermination":true,"VolumeId":"vol-f9a9f20c","AttachTime":"2015-07-20T00:16:31.000Z"}}
{"DeviceName":"/dev/sdb","Ebs":{"Status":"attached","DeleteOnTermination":true,"VolumeId":"vol-d2a9f227","AttachTime":"2015-07-20T00:16:31.000Z"}}
{"DeviceName":"/dev/sdf","Ebs":{"Status":"attached","DeleteOnTermination":false,"VolumeId":"vol-9f149a8e","AttachTime":"2015-07-20T00:19:20.000Z"}}

# system volumes
vm$ df -h | grep /dev/
/dev/xvda1      2.8G  1.3G  1.4G  49% /
/dev/xvdb2      3.9G  1.3G  2.5G  33% /var/vcap/data
/dev/loop0      120M  1.6M  115M   2% /tmp
/dev/xvdf1       32G   28G  1.9G  94% /var/vcap/store

spot + ephemeral0
$ cat bosh.yml
    cloud_properties:
      availability_zone: "us-west-2b"
      instance_type: "c3.xlarge"
      spot_bid_price: 0.04

$ aws ec2 describe-spot-instance-requests --spot-instance-request-ids=sir-03hl94lq | jq -c '.SpotInstanceRequests[] | ( .InstanceId , .LaunchSpecification.BlockDeviceMappings )'
"i-f0ca2b06"
[{"DeviceName":"/dev/sdb","VirtualName":"ephemeral0"}]

$ aws ec2 describe-instances --instance-ids i-f0ca2b06 | jq -c '.Reservations[].Instances[].BlockDeviceMappings[] | .'
{"DeviceName":"/dev/xvda","Ebs":{"Status":"attached","DeleteOnTermination":true,"VolumeId":"vol-f2d28907","AttachTime":"2015-07-20T01:21:15.000Z"}}
{"DeviceName":"/dev/sdf","Ebs":{"Status":"attached","DeleteOnTermination":false,"VolumeId":"vol-9f149a8e","AttachTime":"2015-07-20T01:23:39.000Z"}}

vm$ df -h | grep /dev/
/dev/xvda1      2.8G  1.3G  1.4G  49% /
/dev/xvdb2       33G  1.7G   29G   6% /var/vcap/data
/dev/loop0      120M  1.7M  115M   2% /tmp
/dev/xvdf1       32G   28G  1.9G  94% /var/vcap/store

on demand + ebs
$ cat bosh.yml
    cloud_properties:
      instance_type: "c4.xlarge"
      ephemeral_disk:
       size: 8192

$ aws ec2 describe-instances --instance-ids i-b7d33241 | jq -c '.Reservations[].Instances[].BlockDeviceMappings[] | .'
{"DeviceName":"/dev/xvda","Ebs":{"Status":"attached","DeleteOnTermination":true,"VolumeId":"vol-86cb9073","AttachTime":"2015-07-20T01:42:31.000Z"}}
{"DeviceName":"/dev/sdb","Ebs":{"Status":"attached","DeleteOnTermination":true,"VolumeId":"vol-53cb90a6","AttachTime":"2015-07-20T01:42:31.000Z"}}
{"DeviceName":"/dev/sdf","Ebs":{"Status":"attached","DeleteOnTermination":false,"VolumeId":"vol-9f149a8e","AttachTime":"2015-07-20T01:44:18.000Z"}}

vm$ df -h | grep /dev/
/dev/xvda1      2.8G  1.3G  1.4G  49% /
/dev/xvdb2      3.9G  1.4G  2.3G  38% /var/vcap/data
/dev/loop0      120M  1.6M  115M   2% /tmp
/dev/xvdf1       32G   28G  1.9G  94% /var/vcap/store

on demand + ephemeral0
$ cat bosh.yml
    cloud_properties:
      availability_zone: "us-west-2b"
      instance_type: "c3.xlarge"

$ aws ec2 describe-instances --instance-ids i-1cce2fea | jq -c '.Reservations[].Instances[].BlockDeviceMappings[] | .'
{"DeviceName":"/dev/xvda","Ebs":{"Status":"attached","DeleteOnTermination":true,"VolumeId":"vol-a3d68d56","AttachTime":"2015-07-20T01:34:11.000Z"}}
{"DeviceName":"/dev/sdf","Ebs":{"Status":"attached","DeleteOnTermination":false,"VolumeId":"vol-9f149a8e","AttachTime":"2015-07-20T01:36:00.000Z"}}

vm$ df -h | grep /dev/
/dev/xvda1      2.8G  1.3G  1.4G  49% /
/dev/xvdb2       33G  1.6G   29G   6% /var/vcap/data
/dev/loop0      120M  1.7M  115M   2% /tmp
/dev/xvdf1       32G   28G  1.9G  94% /var/vcap/store

@cfdreddbot, would you mind asking a human to review this comment and possibly update your lists?

I rebased to the latest develop and fixed the failing test from my change. Let me know if there's anything else I can do to assist in merging.

I think this can be closed now? https://github.com/cloudfoundry-incubator/bosh-aws-cpi-release/commit/09ae08d3cf0622ad0a0fb61da6a1f026ddb9d24f

@dpb587 Thanks for the reminder. Sorry it took some time during the CPI transition.


About the location embedding module

This is a very good work, I was recently looking at your code and I would like to ask about the location embedding module in that file

Thank you for your question! In the context of machine learning, "throughput" typically refers to the number of samples that can be processed by a model in a given amount of time. This metric is used to evaluate the efficiency of the model during inference or prediction, not during training.
Training speed, on the other hand, refers to the amount of time it takes for a model to complete one iteration or epoch of the training process. This metric is used to evaluate the efficiency of the model during the training phase, not during inference.
If you want to measure the training speed of a model, you can track the time it takes for the model to complete each epoch or batch during training. You can then calculate the average time per epoch or batch to get an estimate of the training speed.
I hope this helps! Let me know if you have any further questions.


[choreo] Add float rounding in jsons

closes #1006
Implemented into the json formatter

Should update test json files


232 - wrong error message on collection view mint

Closes #232

now getting below error on minting token from single token page

Also, getting below message on minting token from collection page, which again is not correct

Now getting "Insufficeint funds" error in case of Gas sponsorship case also from collection view.From Single token page, minting is working


Feature/semantic scrub

What kind of change does this PR introduce?
Draft PR for Semantic Scrubbing:
Steps:

[x] Semantic Scrubbing without specific identification.
[ ] Test Semantic Scrubbing with visualization.
[ ] Specific Entity Identification. as shown in the eg. in the Issue )

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] If applicable, I have added tests to prove my fix is functional/effective
[ ] I have linted my code locally prior to submission
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (e.g. README.md, requirements.txt)
[ ] New and existing unit tests pass locally with my changes

Summary
Progress on Semantic Scrubbing test.

Motivation: Issue @264
How can your code be run and tested?

Other information

The current version of the PR, replaces the identified PII/PHI with this entity types.
Before:
LLC OPERATING AGREEMENT
This LLC Operating Agreement (“Agreement”) is entered into on )2/06/22	Effective Date”). The members in this Agreement are as follows:

Member Name:	John  Doe		  Address:	125 Military Drive, Washington DC, 20001 Member Name:	Tom Jerry	 Address:	25 Empire State, New York, 20002

Above individuals are referred to herein individually as “Member” and together as “Members”.

Recitals

The company will be known as ABC LLC
of 16/06/2023

(“Company”) with a formation date

The Company’s primary place of business will be Sans Fransisco	       The Company will be governed under the laws of the state of California
The Company’s primary purpose is to provide Legal Support

Agreement

NOW, THEREFORE, in consideration of the respective covenants and agreements contained in this Agreement, Members agree as follows:

1.	INTEREST AND AUTHORITY
a.	The Members’ ownership interest in the Company will be as follows:

John	Class A

Tom	Class B

b.	All Members will have equal vote. No Member is authorized to act on their own in obtaining contracts, financial, or other obligations on the Company. Decisions will be based on a majority of equal votes.

2.	CAPITAL CONTRIBUTIONS
a.	Members shall make a capital contribution in such amounts as listed below. The Members may take distributions of the capital from time to time in accordance with the limitations imposed by this Agreement.

 	JOhn : capital contribution of	15000

 	Tom : capital contribution of	200000

3.	PROFIT AND LOSSES
a.	The Members will share the net profits and net loss according to the following percentages:
John	50
Tom	50

After:
LLC OPERATING AGREEMENT
This LLC Operating Agreement (“Agreement”) is entered into on )2/06/22	Effective Date”). The members in this Agreement are as follows:

Member Name:	<PERSON>		  Address:	125 <LOCATION>, <LOCATION>, 20001 Member Name:	<PERSON>	 Address:	25 <LOCATION>, <LOCATION>, 20002

Above individuals are referred to herein individually as “Member” and together as “Members”.

Recitals

The company will be known as ABC LLC
of 16/06/2023

(“Company”) with a formation date

The Company’s primary place of business will be Sans Fransisco	       The Company will be governed under the laws of the state of <LOCATION>
The Company’s primary purpose is to provide Legal Support

Agreement

NOW, THEREFORE, in consideration of the respective covenants and agreements contained in this Agreement, Members agree as follows:

1.	INTEREST AND AUTHORITY
a.	The Members’ ownership interest in the Company will be as follows:

John	Class A

Tom	Class B

b.	All Members will have equal vote. No Member is authorized to act on their own in obtaining contracts, financial, or other obligations on the Company. Decisions will be based on a majority of equal votes.

2.	CAPITAL CONTRIBUTIONS
a.	Members shall make a capital contribution in such amounts as listed below. The Members may take distributions of the capital from time to time in accordance with the limitations imposed by this Agreement.

 	<PERSON> : capital contribution of	15000

 	<PERSON> : capital contribution of	<US_DRIVER_LICENSE>

3.	PROFIT AND LOSSES
a.	The Members will share the net profits and net loss according to the following percentages:
<PERSON>	50
<PERSON>	50

Checked Task 1 as completed

Step 2:  Testing with Visualization
Some Screenshots to demonstrate how Visualization looks like with semantics:

Marking Step 2 as Completed.
Some Notes:

If we have a name, then it scrub to .
And during visualization, We would see it as <-P-E-R-S-O-N-> in the textbox in the diff images as well as on the right in the text. Also seen in the above screenshots.

After the above visualization fix,
the redaction is Blue in color despite the SCRUB_FILL_COLOR - 0xFF0000 which is Red (in hex).

Thanks @KrishPatel13 ! As discussed let's please ignore visualization for now and focus on:

Scrubbing entities (a: default entities, e.g. firstname/lastname; b: LLC names)
Retaining semantic information

It seems Step 1a is complete. Let's focus on Step 1b using lex-nlp.

@abrichr Part 1a should be ready to be merged! This is Does NOT completely resolve https://github.com/MLDSAI/OpenAdapt/issues/264
So I request you keep it open but only merge this. 🙏

🙏 🚀

Rename the title to Pseudonymization
In Favour of: https://medium.com/@openredact/anonymizer-a-framework-for-text-anonymization-499855f639d4


Wrong stale data

React version 16.13.1
Concurrent mode no
Rest Hooks version 4.5.9
Describe the bug
Hey! 👋🏽
I'm getting wrong stale data when changing query parameters.
First request made to:  /api/issues-filters?
Returns response A.
Second request made to: /api/issues-filters?severities%5B0%5D=critical
Returns response B.
Third request is back to: /api/issues-filters?
There's no network request as expected. But the response I get back is B while I expected it to be A.
I'm using useStatefulResource to fetch data.
const { data: newData, loading, error } = useStatefulResource(
    IssueFiltersResource.filtersShape(),
    queryParams
  );

The response
To Reproduce
Steps to reproduce the behavior:

Make request without params
Make another request with params
Make another request without params like first request

Expected behavior
Expected to get the correct cached response based on getFetchKey.  Actual result is getting the last cached response.

Can you share what IssueFiltersResource.filtersShape().getFetchKey() is?

Can you share what IssueFiltersResource.filtersShape().getFetchKey() is?

PS) If you're able to get onto v5 - https://resthooks.io/docs/guides/debugging is really useful in figuring these things out.

PS) If you're able to get onto v5 - https://resthooks.io/docs/guides/debugging is really useful in figuring these things out.

This is getFetchKey. I'm using qs to stringify the query params.
getFetchKey: params => `api/issues-filters?${qs.stringify(params)}`,

I will try to upgrade to v5 and see if it helps.

This is getFetchKey. I'm using qs to stringify the query params.
getFetchKey: params => `api/issues-filters?${qs.stringify(params)}`,

I will try to upgrade to v5 and see if it helps.

Ok, I've upgraded to v5 and while upgrading i figured what was the issue.
I was caching the entities of IssueFiltersResource response while I should have just cache the entire response since it's a singleton endpoint.
Closing this, thanks!

Ok, I've upgraded to v5 and while upgrading i figured what was the issue.
I was caching the entities of IssueFiltersResource response while I should have just cache the entire response since it's a singleton endpoint.
Closing this, thanks!

@ranyefet just for my curiosity, can you share a sample response from this endpoint?

@ranyefet just for my curiosity, can you share a sample response from this endpoint?


yarn build is broken in<EMAIL_ADDRESS>

Describe the Bug
When I follow the official doc setup

yarn create email
yarn install
yarn build

I'm getting an error like this
$ email build
   ⠦ Installing dependencies on `.react-email`Error: Unable to install the dependencies and it exited with code: 1
    at ChildProcess.<anonymous> (<path>/react-email-starter/node_modules/react-email/dist/cli/index.js:2556:44)
    at ChildProcess.emit (node:events:518:28)
    at maybeClose (node:internal/child_process:1105:16)
    at ChildProcess._handle.onexit (node:internal/child_process:305:5)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

{
  "name": "react-email-starter",
  "version": "0.0.32",
  "private": true,
  "scripts": {
    "build": "email build",
    "dev": "email dev",
    "export": "email export"
  },
  "dependencies": {
    "@react-email/components": "0.0.23",
    "react-email": "3.0.1",
    "react-dom": "18.3.1",
    "react": "18.3.1"
  },
  "devDependencies": {
    "@types/react": "18.3.4",
    "@types/react-dom": "18.3.0"
  }
}

Which package is affected (leave empty if unsure)
No response
Link to the code that reproduces this issue
the default code from the official documentation
To Reproduce

yarn create email
yarn install
yarn build

Expected Behavior
the build should be working
What's your node version? (if relevant)
v20.12.2

This worked on my side, not sure what is going on here but I can't reproduce.

Hi @gabrielmfern ,
is there some way to get debug logs from this command to see the dependencies that failed to resolve?

My .react-email package never builds. My package.json I have:
"email:dev": "email dev --dir src/app/services/resend/templates -p 8080"
This works, but when I view the preview it shows the following for all templates:

My .react-email package never builds. My package.json I have:
"email:dev": "email dev --dir src/app/services/resend/templates -p 8080"
This works, but when I view the preview it shows the following for all templates:

Ignore me, I was importing a file which imported the Request object, so my fault there!

@taylor-lindores-reeves If the Request you are trying to use is the Node class, and it doesn't work we will take that as a bug.
Anything that works in Node, should run fine in the preview server, including globals and even node modules.

Closing as not planned because of inactivity. If anyone experiences this, let me know and I'll reopen.


falco: add liveness and readiness probes

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area falco-chart
What this PR does / why we need it:
falco: add liveness and readiness probes
Which issue(s) this PR fixes:
Fixes https://github.com/falcosecurity/charts/issues/53
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from cpanato after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

falco/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cpanato: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@leogr done sorry for the delay

@leogr done sorry for the delay

I did not wait so long, and anyway no need to apologize :smiley_cat:

LGTM label has been added.  Git tree hash: a641188874bd13e7cf678601cdbc413fb65afbcf

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

falco/OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

An impact of this is that our rollout of a daemonset has gone from a few minutes to multiple hours, as ArgoCD is rolling out each pod and then waiting 30 seconds before moving to the next.

Have you tried to tweak falco.readinessProbe.initialDelaySeconds ?
Depending on your environment, it might be not needed to wait up to 30 seconds. Perhaps a lower value would work for you.
Let us know!

I we are evaluating how/if we want to address it. It just caught us off guard when release went from 60 seconds to 3 hours.


Basis functions should have "gain" parameters

This may over-parameterise the models

But when we mix, e.g. "linear" and "rbf" basis, the regressor weights mean quite different things, and having a common regulariser (lambda) may be too constraining.

Look at the A la Carte paper, and the V parameter.

This can probably be implemented with a decorator like slicing!

I don't think this is true, thinking about this more... data-fit should counteract this problem, and it would become over-parameterised if we implemented this...


ufdbGuard is unable to optimize easylist because of too many lines

Hi there!
I've successfully set up ufdbguard with easylists using your script. Nice.
Once thing, though, is that ufdb complaints that it's too long a list to optimize, and that it will process urls one by one:
2022-04-03 17:58:39 [18335] ufdbCategory adblock
2022-04-03 17:58:39 [18335] ufdbCategoryExpressionList adblock/easylist
2022-04-03 17:58:39 [18335] loading regular expressions from "/usr/local/ufdbguard/blacklists/adblock/easylist"
2022-04-03 17:58:45 [18335] WARNING: the expressionlist has 25101 expressions and may use many resources  *****
2022-04-03 17:58:45 [18335]       Note that large numbers of expressions may impact performance considerably
2022-04-03 17:58:50 [18335] ERROR: UFDBoptimizeExprList: unable to optimise 25101 expressions of /usr/local/ufdbguard/blacklists/adblock/easylist  (error 12)  *****
2022-04-03 17:58:50 [18335] Since the 25101 expressions could not be optimised into one expression, they will be evaluated one by one which impacts performance  *****
2022-04-03 17:58:50 [18335] ufdbCategoryExpressionList adblock/easyprivacy
2022-04-03 17:58:50 [18335] loading regular expressions from "/usr/local/ufdbguard/blacklists/adblock/easyprivacy"
2022-04-03 17:59:06 [18335] WARNING: the expressionlist has 45652 expressions and may use many resources  *****
2022-04-03 17:59:06 [18335]       Note that large numbers of expressions may impact performance considerably
2022-04-03 17:59:11 [18335] ERROR: UFDBoptimizeExprList: unable to optimise 45652 expressions of /usr/local/ufdbguard/blacklists/adblock/easyprivacy  (error 12)  *****
2022-04-03 17:59:11 [18335] Since the 45652 expressions could not be optimised into one expression, they will be evaluated one by one which impacts performance  *****

there is some heavy RAM usage (2GB+) on the server as well.
I was wondering if splitting the output file into separate smaller files could be a simple solution for this?
Thanks a lot for the script!

EDIT:
I something like the following
cd /usr/local/ufdbguard/blacklists/adblock
split -a5 -l400 -d easyprivacy easyprivacy_
split -a5 -l400 -d easylist easylist_
echo -ne "\n\n\n\n\n\n\n"
for i in $(ls /usr/local/ufdbguard/blacklists/adblock); do echo "expressionlist adblock/$i"; done

Which splits the easylist and easyprivacy files in smaller 400-lines files (the maximum recommended by the docs is 500) and then outputs the needed expressionlists for the ufdbGuard.conf file.
Then I modified my ufdbGuard.conf like so:
category adblock {
    expressionlist adblock/easylist_00000
    expressionlist adblock/easylist_00001
    expressionlist adblock/easylist_00002
    expressionlist adblock/easylist_00003
    expressionlist adblock/easylist_00004
    expressionlist adblock/easylist_00005
    expressionlist adblock/easylist_00006
    expressionlist adblock/easylist_00007
    expressionlist adblock/easylist_00008
    expressionlist adblock/easylist_00009
    expressionlist adblock/easylist_00010
    expressionlist adblock/easylist_00011
    expressionlist adblock/easylist_00012
    expressionlist adblock/easylist_00013
    expressionlist adblock/easylist_00014
    expressionlist adblock/easylist_00015
    expressionlist adblock/easylist_00016
    expressionlist adblock/easylist_00017
    expressionlist adblock/easylist_00018
    expressionlist adblock/easylist_00019
    expressionlist adblock/easylist_00020
    expressionlist adblock/easylist_00021
    expressionlist adblock/easylist_00022
    expressionlist adblock/easylist_00023
    expressionlist adblock/easylist_00024
    expressionlist adblock/easylist_00025
    expressionlist adblock/easylist_00026
    expressionlist adblock/easylist_00027
    expressionlist adblock/easylist_00028
    expressionlist adblock/easylist_00029
    expressionlist adblock/easylist_00030
    expressionlist adblock/easylist_00031
    expressionlist adblock/easylist_00032
    expressionlist adblock/easylist_00033
    expressionlist adblock/easylist_00034
    expressionlist adblock/easylist_00035
    expressionlist adblock/easylist_00036
    expressionlist adblock/easylist_00037
    expressionlist adblock/easylist_00038
    expressionlist adblock/easylist_00039
    expressionlist adblock/easylist_00040
    expressionlist adblock/easylist_00041
    expressionlist adblock/easylist_00042
    expressionlist adblock/easylist_00043
    expressionlist adblock/easylist_00044
    expressionlist adblock/easylist_00045
    expressionlist adblock/easylist_00046
    expressionlist adblock/easylist_00047
    expressionlist adblock/easylist_00048
    expressionlist adblock/easylist_00049
    expressionlist adblock/easylist_00050
    expressionlist adblock/easylist_00051
    expressionlist adblock/easylist_00052
    expressionlist adblock/easylist_00053
    expressionlist adblock/easylist_00054
    expressionlist adblock/easylist_00055
    expressionlist adblock/easylist_00056
    expressionlist adblock/easylist_00057
    expressionlist adblock/easylist_00058
    expressionlist adblock/easylist_00059
    expressionlist adblock/easylist_00060
    expressionlist adblock/easylist_00061
    expressionlist adblock/easylist_00062
    expressionlist adblock/easyprivacy_00000
    expressionlist adblock/easyprivacy_00001
    expressionlist adblock/easyprivacy_00002
    expressionlist adblock/easyprivacy_00003
    expressionlist adblock/easyprivacy_00004
    expressionlist adblock/easyprivacy_00005
    expressionlist adblock/easyprivacy_00006
    expressionlist adblock/easyprivacy_00007
    expressionlist adblock/easyprivacy_00008
    expressionlist adblock/easyprivacy_00009
    expressionlist adblock/easyprivacy_00010
    expressionlist adblock/easyprivacy_00011
    expressionlist adblock/easyprivacy_00012
    expressionlist adblock/easyprivacy_00013
    expressionlist adblock/easyprivacy_00014
    expressionlist adblock/easyprivacy_00015
    expressionlist adblock/easyprivacy_00016
    expressionlist adblock/easyprivacy_00017
    expressionlist adblock/easyprivacy_00018
    expressionlist adblock/easyprivacy_00019
    expressionlist adblock/easyprivacy_00020
    expressionlist adblock/easyprivacy_00021
    expressionlist adblock/easyprivacy_00022
    expressionlist adblock/easyprivacy_00023
    expressionlist adblock/easyprivacy_00024
    expressionlist adblock/easyprivacy_00025
    expressionlist adblock/easyprivacy_00026
    expressionlist adblock/easyprivacy_00027
    expressionlist adblock/easyprivacy_00028
    expressionlist adblock/easyprivacy_00029
    expressionlist adblock/easyprivacy_00030
    expressionlist adblock/easyprivacy_00031
    expressionlist adblock/easyprivacy_00032
    expressionlist adblock/easyprivacy_00033
    expressionlist adblock/easyprivacy_00034
    expressionlist adblock/easyprivacy_00035
    expressionlist adblock/easyprivacy_00036
    expressionlist adblock/easyprivacy_00037
    expressionlist adblock/easyprivacy_00038
    expressionlist adblock/easyprivacy_00039
    expressionlist adblock/easyprivacy_00040
    expressionlist adblock/easyprivacy_00041
    expressionlist adblock/easyprivacy_00042
    expressionlist adblock/easyprivacy_00043
    expressionlist adblock/easyprivacy_00044
    expressionlist adblock/easyprivacy_00045
    expressionlist adblock/easyprivacy_00046
    expressionlist adblock/easyprivacy_00047
    expressionlist adblock/easyprivacy_00048
    expressionlist adblock/easyprivacy_00049
    expressionlist adblock/easyprivacy_00050
    expressionlist adblock/easyprivacy_00051
    redirect http://MY_IP:8081/cgi-bin/URLblocked.cgi?admin=%A&mode=default&color=red&size=normal&clientaddr=%a&clientname=%n&clientuser=%i&clientgroup=%s&targetgroup=%t&url=%u
}

And now I'm not getting any warning whatsoever and the memory usage dropped down to 500MB, which is still a lot, but much better than 2GB.

Anyway, I hope it helps someone
Cheers

Another suggestion:
If you are using ssl bump to transparently proxy HTTPS with squid, your category adblock block, should include the following:
option block-bumped-connect on


Pod query

modified podQuery.js to loop through and work for multiple pods, not tested yet

PodQuery logic looks good! (dynamically renders), going to pull down into my branch and delete the test code and add in the logic for NodeQuery/ServiceQuery and open new PR


Use the correct annotation packages so the annotations are picked up …

…by Swagger2.
@ajordens please review.

LGTM


Disallow EIP-* in the preamble title and description fields

Originally from #55
At minimum, it would be nice if the bot failed CI if it found text matching eip[-\s]*[0-9]+ or erc[-\s]*[0-9]+ in the title and description preamble fields.
A further step would be to either comment on the PR, or automatically apply the suggestion to the PR.
Personally I would prefer a comment on the PR, but we should probably discuss in this issue.

I will work on it.

Small tweak: instead of banning all EIPs in the title/description, only ban the current EIP.
See ethereum/EIPs#5008 for an example of why we should allow other EIPs in those fields.

Covered in eipw.


Wording: use repository instead of project

More: https://github.com/signalfx/gdi-specification/pull/96#discussion_r634961259

Closed by #104


Drop Nodes On Mouse Position

I have resized my canvas to 10000x7500, when I drop nodes on it the nodes are hided in the lower right corner. So it would be a great improvement to drop the node where the mouse position is

Aren't nodes dropped on mouse position right now?
https://github.com/wieslawsoltes/NodeEditor/blob/58170442d6642b2bf75e568af81835695d5c8970/src/NodeEditorAvalonia/Behaviors/DrawingDropHandler.cs#L49

no. I have drop in the middle of the canvas

hmm try to use my canvas dimension: 10000x7500

Still can not repro:
https://user-images.githubusercontent.com/2297442/145604142-fc5beab3-bf57-40ef-95cd-15e77818f3e0.mp4

https://user-images.githubusercontent.com/4117602/145606256-f2ec0540-ce80-4bc5-9c34-4b90b24d91a9.mp4
This video covers drop and zoom problem

Seems like you are missing background and maybe drop behavior can not get pointer input for correct position. Try adding background.

I have Background="White" on NodeZoomBorder  but same result

@furesoft Found the source of the issue, will fix in next release.

Fix 8e85023 (#26)


Darkmode not supported properly

Hi,
I scrolled through the frappe wiki pages and saw that the darkmode doesnt seem to work correctly on links and menu items.

Also the sandwich menu is black, so the user cant see that.

Active menu items are hidden.

Also, these outlines are hidden when on darkmode, the titles are black so they also cant be seen:

Thanks for bringing this to attention. This happened due to the primary color changes in frappe desk following the new espresso design. I'll fix it shortly.


Make Exasol port configurable

It would be nice to have the Exasol port configurable in order to switch between dev and prod environment:
provider "exasol" {
ip       = "123"
username = "userxy"
password = "abc"
port = 123 <-- NEW
}

Should be done in https://github.com/abergmeier/terraform-provider-exasol/commit/23fba3049ecd7759c2959d254cf0e1229bf3dff0

works nicely :) Thank you!


Add missing places to expose detected envoy model

Seems I missed some places indeed. Didn't remember it is there for all sensor types.

I think it should be like this (see latest commit), to keep the model "Inverter" and "Relay".

Yes, that's better 😄

Would be nice if we could also expose the inverter product name. But could not find anything for that yet (also no PN to product name mapping anywhere)

Yeah, i thought of that too, but then we'd be creating our own list/mapping of part_num to readable product name. For the ones we're having we could start with that though 😄
Maybe there is something on the enlighten interface somewhere that we could retrospect and/or reproduce, have to look at that later.

Yeah, i thought of that too, but then we'd be creating our own list/mapping of part_num to readable product name. For the ones we're having we could start with that though 😄
Maybe there is something on the enlighten interface somewhere that we could retrospect and/or reproduce, have to look at that later.

I could not find anything in the enlighten interface, so my best bet would be to keep a static list
this is my list at least..
800-00654-r08 (Envoy-S-Metered-EU)
800-01736-r02 (IQ7+; IQ7PLUS-72-M-INT)
800-00598-r04 (IQ Relay 1-phase; Q-RELAY-1P-INT)

800-00656-r06 (Envoy-S-Standard-EU)
800-01359-r02 (IQ8+; IQ8PLUS-72-M-INT)

Although it might be nice to keep the Envoy model as the detected one. Also for (easy) debugging, maybe.


F1 Command Polish Ideas

These are the F1 commands that this extension provides:

I think some of these commands can be cleaned up.

Close instance - I think the instance should be closed by just closing the browser tab in VS Code, I don't see the need for this command.
Open Browser with DevTools - I think Open Browser is sufficient, if the action to open the dev tools for this browser is discoverable. Maybe the browser should remember if the dev tools were opened the last time and just restore them.,
Toggle browser - I'm not sure this command is needed. It also does not seem to do anything for me.
Toggle CSS mirror editing on/off and View Documentation - these commands should only show when the browser is focused

Thanks for the suggestion.


Changed Issue_Resource_Template.txt file to markdown

Pull Request
Proposed Changes

Changed Issue_Resource_Template.txt file to markdown
Issue number [Fixes #72].

Types of changes made

[ ] Fixed Bug
[x] Added new Features
[x] Edited Documentations
[ ] Enhanced the working of code
[x] Additional Change

Checklist:

[x] My code follows the Mentioned Guidelines for the project
[x] I have self-reviewed the changes made
[x] I have made the changes easy to understand by adding comments
[x] My changes does not generate new warnings/errors
[ ] I have attached relevant screenshots to display the changes made
[x] I have made the required changes to the documentations

@all-contributors add @Sumit-Sahu for docs


Connection Refused

Hello everyone, I'm running the following code to fill a Redis server quickly:
public static void main(String[] args) {
      Jedis jedis = new Jedis(args[0], Integer.parseInt(args[1]));
        long fiveMega = 524288000; //500mb
        long correspondentValue = 2097151; //Value to get 500mb being consumed
        long memoryRequirement = Long.parseLong(args[2],10);

        long value = (memoryRequirement * correspondentValue) / fiveMega;

        Pipeline pipeline = jedis.pipelined();
        for (long i = 0 ; i < value; i++) {
                pipeline.sadd("key"+i, "value");
        }
        pipeline.sync();
   }

However sometimes I get the connection exception bellow but other times it works and I can't understand why. The ports and everything are correct.
By the way, if there is another way to fill up a redis server quickly with junk data I would appreciate to see if there is no error! Thanks for the help!
exit status 1: Exception in thread "main" redis.clients.jedis.exceptions.JedisConnectionException: java.net.ConnectException: Connection refused (Connection refused)
at redis.clients.jedis.Connection.connect(Connection.java:148)
at redis.clients.jedis.BinaryClient.connect(BinaryClient.java:75)
at redis.clients.jedis.Connection.sendCommand(Connection.java:77)
at redis.clients.jedis.BinaryClient.sadd(BinaryClient.java:325)
at redis.clients.jedis.Client.sadd(Client.java:242)
at redis.clients.jedis.PipelineBase.sadd(PipelineBase.java:474)
at redis.clients.jedis.Pipeline.sadd(Pipeline.java:8)
at fillRedis.main(fillRedis.java:32)
Caused by: java.net.ConnectException: Connection refused (Connection refused)
at java.net.PlainSocketImpl.socketConnect(Native Method)
at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:350)
at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)
at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:589)
at redis.clients.jedis.Connection.connect(Connection.java:142)
... 7 more

@SergioMendes93 my guess is that maybe there's a idle timeout configuration somewhere as the data is not written into the socket as the sync function is called. Have you tried using telnet to connect to your redis server and check if the connection gets closed automatically every x amount of time?

Yeah it was that! Thanks a lot


ASM installation failed because of missing CRD

During the terraforming of the cluster, the terraform installation failed because it's missing some CRDs needed to register the GKE cluster inside ASM.

Hi @laurentgrangeau !

Which CRDs?
Is this related to #27?

Ah! Ignore my latest comment, I didn't see you linked this to #26 already :)


gcc-arm-embedded-{6,7,8,9,10}: drop python27 from dependencies

Description of changes

python27 is only required for arm-none-eabi-gdb-py and since python27 is EOL, it's worth disabling python support for GDB, rather then breaking the whole gcc-arm-embedded
btw, gcc-arm-embedded-11 already supports python 3.x

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Since you are a maintainer of this package I'll take this as a approval to break this script.
@ofborg build gcc-arm-embedded-6 gcc-arm-embedded-7 gcc-arm-embedded-8 gcc-arm-embedded-9 gcc-arm-embedded-10


Sample: Push Notifications C++ Packaged

Description
Demonstrate receiving foreground and background push notifications from an unpackaged console app.
Checklist

[ ] Samples build and run using the Visual Studio versions listed in the Windows development docs.
[ ] Samples build and run on all supported platforms (x64, x86, ARM64) and configurations (Debug, Release).
[ ] Samples set the minimum supported OS version to Windows 10 version 1809.
[ ] Samples build clean with no warnings or errors.
[ ] [For new samples]: Samples have completed the sample guidelines checklist and follow standardization/naming guidelines.
[ ] I have commented on my PR /azp run to have the CI build run on my branch. This must be done right before merging to ensure the build is up to date and accurate.

/azp run


Enhancement Request: RLOS tool should not be enabled to run with unprojected data.

Enhancement Request: disable the OK button and show red X on tool that source input surface is not projected, similar to other GP tools that indicate input is invalid.
As pointed out in a previous issue, https://github.com/ArcGIS/visibility-addin-dotnet/issues/33, a requirement of the RLOS is that source data be projected.
In some cases, data available to potential users will be GCS_WGS84.

This is no longer an issue.


hello, please make changes to the Toolbox, when trying to patch SWDL, metainfo gave an error, thank you, Mst2_RoA_VW_ZR_P0359T, HW 35 Variant 17213, mail<EMAIL_ADDRESS>

Please list which patching process showed the error (CP, SWAP, SWDL)

Please add your firmware-train like i.e. Mst2_RoA_VW_ZR_P0359T
Attach the support-dump as .zip file
Go to Tools -> Dump necessary files to SD folder /dump/.../support and zip the support folder

Added in https://github.com/olli991/mib-std2-pq-zr-toolbox/archive/refs/heads/master.zip

thank you very much19:07, 26 мая 2024 г., lprot @.***>:
Added in https://github.com/olli991/mib-std2-pq-zr-toolbox/archive/refs/heads/master.zip
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>
-- Отправлено из мобильного приложения Яндекс Почты


Expose default vscode extensions

based on https://github.com/CodinGame/monaco-vscode-api/pull/94
the package now exposes some module that register vscode default extensions:
for instance:
import 'vscode/default-extensions/theme-defaults'
import 'vscode/default-extensions/javascript'

is now enough to have the default vscode themes and the javascript language

This sounds good. I will review it tomorrow.

@kaisalmen I've also added a rollup/vite plugin that allows to load a vsix file directly (only the manifest or course, not the code)

@CGNonofr I thought this is independent, but now I looked at the git graph and I have to start with
https://github.com/CodinGame/monaco-vscode-api/pull/93 right?

@CGNonofr I thought this is independent, but now I looked at the git graph and I have to start with #93 right?

Yep, every PR is based on the previous one
Alsmost everytime I make changes I need to tweak the polyfills so I prefer to be based on the last working version to not have to rebase all remaining PRs and test them when one is merged

I've published a 1.78.0-next.0 including all the open PRs

@CGNonofr Doesn't it make sense to only look at the delta between main and #98 because otherwise I check things that are gone a couple of commits later. WDYT?

eworked a couple of comm

The PRs are based on each other but quite still independent. I don't think there is anything done then undone, the commits are mostly well splitted. I'm not sure you'll be able to understand everything by just looking at the whole changes.
But It's a matter of preferences, do as you wish

The PRs are based on each other but quite still independent. I don't think there is anything done then undone, the commits are mostly well splitted. I'm not sure you'll be able to understand everything by just looking at the whole changes.

Yes, looking at them separately, but with an external tool to see the real delta in between the branches, was the better approach.


Model stabilityai/stable-diffusion-2-1 does not exist

This model has stopped loading via the inference API. Is it just me or is this down indefinitely?

hi @spectraldeviant , are you trying textToImage task or imageToImage?
I've just tested the textToimage via widget and it's working fine

There were availability issues with Hugging Face, it's fixed now

text to image. thanks


wip: recovery transaction

made changes that should handle #124

OK, last two things:

Rebase this branch with bifrost branch (make sure your local bifrost branch is up to date, then switch to bifrost-recovery-address and run git rebase bifrost).
I mentioned this earlier but maybe there was a misunderstanding: access-control-allow-origin-header should have a default value, so if it's not set in bifrost.cfg then the value should be *.


Add organisation claims in support

This PR replicates the school claim flow for our support users.

 Adding claims in Support on behalf of schools

More of a question, I'm not sure about the navigation at the top, I think that is a 'global navigation' rather than the one for your organisation.
It might be confusing if the user is in the middle of filling a claim and for whatever reason he clicks on the Claims Navigation and he's taken to a page that shows him all the claims in our system rather than the claims in the organisation he is in. Going back to your organisation might be tricky
Not sure if this is a problem, just wanted to raise it though

Thanks, @CatalinVoineag. In support the primary navigation persists and we use secondary navigation for each organisation (a.k.a. school). In the flows, we remove as much page furniture as possible to remove distractions. But this does pose the problem of orientation. To help mitigate this problem, we include the section or flow and school name in the caption, for example, "Add claim - BAT School 1".
This is something we need to keep an eye on. We have some research sessions lined up with our support colleagues.


Can not load image when the URL has Chinese

Such like this:
let imageString = "http://onivcbznk.bkt.clouddn.com/QQ截图20170411141750.png"
cell.imageView.kf.setImage(with: URL.init(string: imageString))

You need a valid url which conforms to RFC 1808, RFC 1738, and RFC 2732. Try to make sure your URL is not nil before sending it to Kingfisher.


Enhance Codacy integration and fix comment formatting

User description

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[x] No

ℹ Additional Information

Description

Fixed a typo in the Codacy bypass comment to ensure proper link formatting.
Added a new dependency for Codacy functions in the configuration file.

Changes walkthrough 📝
Relevant filesEnhancement

comments.phpFix typo in Codacy bypass comment

src/comments.php
Fixed a typo in the Codacy bypass comment body.  Corrected the formatting of the pull request link.

+1/-1

config.phpAdd Codacy dependency to configuration

src/config/config.php
Added a new dependency for Codacy functions.  Updated the configuration loading process.

+1/-0

@gstraccini review


Check top-level domain suggestion against domains

After correcting the top-level domain, the suggestion is sometimes
close enough to a full domain that accepting the suggestion results in
another bad email address. A previous commit tried to solve this
problem by changing the thresholds at which we suggest top-level
domains and full domains, but some edge cases remained. This
commit cuts out the middleman by directly checking for a full domain
suggestion before returning a top-level domain suggestion.

@derrickko currently we have:
domainThreshold: 3,
secondLevelThreshold: 2,
topLevelThreshold: 2,

I think it's confusing and not scalable. I would like to merge this solution and use a single threshold instead. Thoughts? cc @kevinhughes27. Potentially related: https://github.com/mailcheck/mailcheck/issues/97

I like the idea proposed in this PR - I don't think removing the individual thresholds is a good idea. Users of this lib don't have to change them.
Btw I didn't submit this in my PR from the Shopify changes but we use much tighter thresholds (2, 1.5, 1,5) IIRC

Users of this lib don't have to change them if they don't need to but they are quite useful for tweaking

What's the basis for tweaking? For example, how did you decide 2, 1.5, 1.5 are the best thresholds? Trail and error? I have trouble even trying to come up with useful default values for these thresholds to avoid future issues like #96 and #97.
Without digging into the code, what do the threshold mean? Do they mean anything by themselves or are they only meaningful when compared among themselves?

the thresholds are how close a string needs to be to be considered a match and thus be corrected to the matching string (in this case domains or parts of domains). They are pretty important and being able to tweak them is also important let me give an example:
I ran a pretty detailed experiment with mailcheck on our checkout page and collected well over a million data points. I measured 3 things - checkout successes, mailcheck suggestions shown and mailcheck suggestions clicked. Right away we noticed that unclicked mailcheck suggestions were related to decreased checkout successes - so in our environment its important to be conservative with suggestions. After noticing this I played with the thresholds and the default domain lists to bring down the number of unclicked suggestions (I collected the original email address if it was unclicked so I could iterate). That is how I arrived at 2, 1.5, 1.5 for thresholds. Our users going through checkout are fairly international so our domain list is large but if for example you are adding mailcheck to a signup page and most of your users will be other tech enthusiasts then your setup will be very different.
In order to avoid the issues you mentioned we should reduce the default thresholds - but you should still be experimenting with the setup in your environment. Maybe @derrickko can elaborate but my take away from looking through previous PRs (most PRs that alter the default config are closed unless there is a really good reason) is that Mailcheck is intended as more of a framework and less of a plug and play solution. That being said lets tighten the thresholds a bit so the default config is more useful, I should have included this in my original PR here.

It's no longer maintained, so I rewrote it entirely and update it for 2022 and onward: https://github.com/ZooTools/email-spell-checker.
Would love to get your thoughts and see if there is something we can improve.

💙 Written in TypeScript and removed jQuery
✅ Fixed a couple of bugs like this or this
🚀 Reduced bundle size to <2KB.
✨ Update TLDs (69+) and added modern startup domains (like .io, .so, .xyz or .dev)
🙏 Implemented suggestions that people made in this repo that were ignored.

Link: https://github.com/ZooTools/email-spell-checker
Come check it out and give it a ⭐ for the effort.


Position changes when dashboard is reloaded

Hello,
I'm not too sure if this is related to Gridstack itself.
I've been trying out Gridstack to create a simple dashboard.
Currently, the flow to create the dashboard and widget is this:

The user creates a dashboard
The user creates widgets
User resizes or updates the position of the widget (on change event trigger, the widget record in the DB is updated to reflect the new size/position).

My issue is that whenever I reload the page and the widgets are rendered is that certain widgets get repositioned or moved down, I've made sure that the widget's new position is updated correctly before the reload. But it seems that whenever the page is reloaded, the widget gets repositioned which then triggers the update on the record.
As for the Gristack options, auto-position and float are both false.
Example:

I'm not sure what to look into to fix this and am already looking into other libraries to use if I can't resolve this.
If anyone has a tip on what I can do, or look into, to help resolve this, it would be greatly appreciated.

I'm experiencing the same issue. Did you find any solution?


Add missing prop-types dependency to website/package.json

Motivation
Website generation is broken unless prop-types is added as a dependency.
Test plan
This fixes website generation in existing Circle CI tests.

Thanks for catching this


Review IMetricsContext contract

The ported metrics.net metric context has been refactored and simplified, currently it still contains an "Advanced" property providing access to more advanced metrics management including access to the metrics data manager and health check data manager, do these manager's belong here or should they be injected independently when needed?
Also review the Time methods IMetricContext, is taking an Action sufficient for timer metrics or should a TimerContext always be returned as in the IAdvancedMetricContext

moved to alpha release project, closing.


Uncaught ReferenceError: remainingChars is not defined

File select2/js/i18n/XXX.js
Change:
var t = e.minimum - e.input.length,
      n = "Por favor, introduzca " + remainingChars + " car";

For:
var remainingChars = e.minimum,
                	t = e.minimum - e.input.length,
                    n = "Por favor, introduzca " + remainingChars + " car";

This should be fixed on the latest branch (and in the upcoming rc.1 release).


Adaption to aldryn-newsblog

cleanup README
fix setup.py to be aligned with https://github.com/aldryn/aldryn-newsblog/blob/master/setup.py

LGTM!


Create pythonapp.yml

Coverage remained the same at 100.0% when pulling c1e70911dff8effbe8d44552465d6d51344108da on cjrh-patch-1 into f081d8172461e1f51048b30f0efe23c6e88594f3 on master.

Coverage remained the same at 100.0% when pulling c1e70911dff8effbe8d44552465d6d51344108da on cjrh-patch-1 into f081d8172461e1f51048b30f0efe23c6e88594f3 on master.