S3 Box ... Does the Display work?

Hi,
until now, I've just used my S3Box as a Bluetooth Proxy which was showing the current Time and Date on its Display.
Now, I am trying to implement the WakeWord configuration you've provided here (https://github.com/esphome/firmware/blob/main/voice-assistant/esp32-s3-box.yaml) and wonder, if this is not working with the Display Configuration...
here's my old config:
i2c:
  scl: GPIO18
  sda: GPIO8
  scan: true

spi:
  clk_pin: GPIO7
  mosi_pin: GPIO6

# The Display itself and what's visible on it...
display:
  - platform: ili9xxx
    model: S3BOX
    cs_pin: GPIO5
    dc_pin: GPIO4
    reset_pin: GPIO48
    id: lcd
    lambda: |-
      it.fill(Color(255, 255, 255));
      it.strftime(160, 85, id(font_time), Color::BLACK, TextAlign::CENTER, "%H:%M:%S", id(time_ha).now());
      it.strftime(160, 170, id(font_date), Color::BLACK, TextAlign::CENTER, "%d.%m.%Y", id(time_ha).now());

# Which Fonts we want to use
font:
  - file: "gfonts://Roboto"
    id: font_time
    size: 60
    glyphs: "0123456789:"
  - file: "gfonts://Roboto"
    id: font_date
    size: 50
    glyphs: "0123456789."
  - file: "gfonts://Roboto"
    id: font_small
    size: 20

# get the Time from Home-Assistant
time:
  - platform: homeassistant
    id: time_ha
    timezone: Europe/Berlin

in addition, but not related to the above, while compiling, I am getting these warnings:
Compiling .pioenvs/esp32-box/src/esphome/components/esp_adf/microphone/esp_adf_microphone.o
src/esphome/components/esp_adf/microphone/esp_adf_microphone.cpp: In static member function 'static void esphome::esp_adf::ESPADFMicrophone::read_task(void*)':
src/esphome/components/esp_adf/microphone/esp_adf_microphone.cpp:98:3: warning: missing initializer for member 'i2s_driver_config_t::chan_mask' [-Wmissing-field-initializers]
   };
   ^
src/esphome/components/esp_adf/microphone/esp_adf_microphone.cpp:98:3: warning: missing initializer for member 'i2s_driver_config_t::total_chan' [-Wmissing-field-initializers]
src/esphome/components/esp_adf/microphone/esp_adf_microphone.cpp:98:3: warning: missing initializer for member 'i2s_driver_config_t::left_align' [-Wmissing-field-initializers]
src/esphome/components/esp_adf/microphone/esp_adf_microphone.cpp:98:3: warning: missing initializer for member 'i2s_driver_config_t::big_edin' [-Wmissing-field-initializers]
src/esphome/components/esp_adf/microphone/esp_adf_microphone.cpp:98:3: warning: missing initializer for member 'i2s_driver_config_t::bit_order_msb' [-Wmissing-field-initializers]
src/esphome/components/esp_adf/microphone/esp_adf_microphone.cpp:98:3: warning: missing initializer for member 'i2s_driver_config_t::skip_msk' [-Wmissing-field-initializers]
Compiling .pioenvs/esp32-box/src/esphome/components/esp_adf/speaker/esp_adf_speaker.o
Compiling .pioenvs/esp32-box/src/esphome/components/gpio/binary_sensor/gpio_binary_sensor.o
Compiling .pioenvs/esp32-box/src/esphome/components/internal_temperature/internal_temperature.o
src/esphome/components/esp_adf/speaker/esp_adf_speaker.cpp: In static member function 'static void esphome::esp_adf::ESPADFSpeaker::player_task(void*)':
src/esphome/components/esp_adf/speaker/esp_adf_speaker.cpp:60:3: warning: missing initializer for member 'i2s_driver_config_t::chan_mask' [-Wmissing-field-initializers]
   };
   ^
src/esphome/components/esp_adf/speaker/esp_adf_speaker.cpp:60:3: warning: missing initializer for member 'i2s_driver_config_t::total_chan' [-Wmissing-field-initializers]
src/esphome/components/esp_adf/speaker/esp_adf_speaker.cpp:60:3: warning: missing initializer for member 'i2s_driver_config_t::left_align' [-Wmissing-field-initializers]
src/esphome/components/esp_adf/speaker/esp_adf_speaker.cpp:60:3: warning: missing initializer for member 'i2s_driver_config_t::big_edin' [-Wmissing-field-initializers]
src/esphome/components/esp_adf/speaker/esp_adf_speaker.cpp:60:3: warning: missing initializer for member 'i2s_driver_config_t::bit_order_msb' [-Wmissing-field-initializers]
src/esphome/components/esp_adf/speaker/esp_adf_speaker.cpp:60:3: warning: missing initializer for member 'i2s_driver_config_t::skip_msk' [-Wmissing-field-initializers]


ODM-11970 GET files metadata endpoint

please provide swagger screenshots

The sections are too long. Better to check right away on my instance https://develop-nt.dev.gs.team/swagger/?urls.primaryName=aFileUser
Below are screenshots w/o errors:


Connecting to Blob Storage Module using Module Name

In my Python Module, im connecting to Blob Storage using Module name (blobstoremodule) as below
    block_blob_service = BlockBlobService(connection_string='DefaultEndpointsProtocol=http;AccountName=localstorageacct;AccountKey=<ACCTKEY>;BlobEndpoint=http://blobstoremodule:11002/localstorageacct;')

We are seeing positive and negative behaviors with this when we tested this on 3 edge devices
On 2 of the edge devices it consistently throws an error when performing any action on Blob
Client-Request-ID=cf2858f6-0153-11ea-98b3-0242ac120005 Retry policy did not allow for a retry: , HTTP status code=Unknown, Exception=HTTPConnectionPool(host='blobstoremodule', port=11002): Max retries exceeded with url: /localstorageacct/rootcontainer/373a6582-e684-42d8-992e-0dc8a73096e2.txt?comp=block&blockid=TURBd01EQXdNREF3TURBd01EQXdNREF3TURBd01EQXdNREF3TURBd01EQSUzRA%3D%3D (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fe8dc2671d0>: Failed to establish a new connection: [Errno -2] Name or service not known'))
On 1 of the Edge, we do not have any issues it consistently has been working
Also when we switch to IP based connectivity to the Blob Module, everything is fine. What could be causing this problem.

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 07a44a28-a19c-f68e-167c-5e8ad91605e0
Version Independent ID: 33eacdff-4f0a-c543-d8f9-e7b208bab015
Content: Store block blobs on devices - Azure IoT Edge
Content Source: articles/iot-edge/how-to-store-data-blob.md
Service: iot-edge
GitHub Login: @arduppal
Microsoft Alias: arduppal

@jaypaddy Thanks for the feedback! We are currently investigating and will update you shortly.

Hi @jaypaddy, would you mind providing the output of docker inspect for each of the three containers running the BlobStorage module, and for each container running the Python module?

Hi @Imawarid - Added the Docker Inspect and my Python Module code to a GitHub Repo
https://github.com/jaypaddy/IoTEdgeDebugging
Docker Inspect for BlobStorageModule - https://github.com/jaypaddy/IoTEdgeDebugging/blob/master/BlobStorageModule
Docker Inspect for BlobWriterModule - https://github.com/jaypaddy/IoTEdgeDebugging/blob/master/BlobWriterModule
Python code in BlobWriterModule - https://github.com/jaypaddy/IoTEdgeDebugging/blob/master/BlobWriter.py

@jaypaddy Could you try changing the storage module name to lowercase, i.e. blobstoremodule instead of BlobStoreModule, and see if anything changes? Looks like a name resolution issue.

@imawarid - Thank you, changing the storage module name to lower case, helped resolve the problem. The files are getting written locally and as well is getting uploaded to Cloud.  The one thing that will need to be further reviewed is this
Edge A - Macbook running IoTEdge in Docker container, it worked with mixed case for the blob storage module
Edge B - Macbook running IoTEdge In Docker container, did not resolve and will need to try with lower case
Edge C - Ubuntu running IoTEdge did not resolve mixed case, and resolved lower case module name.

Also this is completely my fault. The documentation clearly states that the module should be in lowercase.... thank you for all the help.
We can close this issue.

@jaypaddy Thanks for the confirmation.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Initial version of the end-to-end tests.

Related to https://github.com/kubewarden/kubewarden-controller/issues/125

I was hoping for other reviews ("if a majority aggres"), before this got merged to be fair!


update-property/update-properties goal: Emit a warning for non existing properties

It should be checked to give a warning about not existing properties for the update-property and update-properties goals.

I've run into this problem for a multi-module project when the property is only in the top level pom. It fails on the first child module.

For me, if you want to only run on the top level then that can be done with the --non-recursive maven flag.


Bump go-units

Bump for https://github.com/docker/go-units/pull/23

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-2" git@github.com:wernight/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@wernight you also need to run vndr github.com/docker/go-units for real update. You can get vndr from github.com/LK4D4/vndr.
Thanks!

Wow! That looks a bit complex. Wouldn't it get picked up automatically actually?

@wernight yeah, it should be simpler, but that's what we have for now.
For vndr to work your somewhere should be in your GOPATH. It is pain indeed. Gladly we don't need to use it often.

You can also assuming you're in your local git checkout of docker;
make BIND_DIR=. shell

which starts a shell in a development container, with your source code bind-mounted in.
Inside that container, vndr is already installed, so you can vndr github.com/docker/go-units there

@wernight You could also try with running go inside a container:
ubuntu@ubuntu:~/docker$ docker run -i -t -v $PWD:/go/src/github.com/docker/docker -w /go/src/github.com/docker/docker golang:1.7.0
root@5721fa1f08e3:/go/src/github.com/docker/docker# go get -u github.com/LK4D4/vndr
root@5721fa1f08e3:/go/src/github.com/docker/docker# vndr github.com/docker/swarmkit
2017/02/02 16:32:29 Collecting initial packages
...
...
2017/02/02 16:32:41 Success
2017/02/02 16:32:41 Running time: 11.217918406s
root@5721fa1f08e3:/go/src/github.com/docker/docker#

It is a lot easier as it does not depends on your local golang setup.

Carrying in #30790


Financial contribution

Hi @gcanti is there a way to support this library financially?

Our company was just wondering the same thing! (Same goes for io-ts)

@gcanti I recently started supporting https://www.patreon.com/martinkl/posts for his CRDT related work. It's not like "I am buying you." It's more like "I would like to support you because what you did or do was immensely helpful for me." I'm posting it here as an inspiration.

Hi all, sponsorships are now enabled :+1:


When in advanced mode and a language menu is open and a user switches to "normal" mode you can get phantom menus

Steps to reproduce

Open a notebook in advanced mode
Click on the cell language indicator
Switch to normal mode

What is expected

No menus are open

What happens

The open language menus are still there, and there is no way to get rid of them :crying_cat_face:

This was fixed by switching to a more idiomatic use of bootstraps dropdown menus :clap: :sparkles:


feat: Create an SBOM for releases

It's kinda difficult to check if this works, since this workflow always fails in my repo clone.

@jennifer-richards https://github.com/larseggert/datatracker/actions/runs/3052008565/jobs/4937025616#step:6:22 fails with
Migrations for 'meeting': ietf/meeting/migrations/0054_auto_20220915_0308.py - Alter field time_zone on meeting

Any idea why?

It looks like the base for that branch is out of date. On main, the urllib3 exception is converted to a warning (https://github.com/ietf-tools/datatracker/blob/5b6695a04c46e1756183272e8da4195f1ebea1a7/ietf/settings.py#L16) and the ci-run-tests workflow was modified to use the return value from the makemigrations check instead of failing if there's anything on stdout (https://github.com/ietf-tools/datatracker/blob/5b6695a04c46e1756183272e8da4195f1ebea1a7/.github/workflows/ci-run-tests.yml#L47).
So merge in main?

Weird. It looked to me like it was up-to-date with main. Trying it now though.

May need to update your fork to see the changes.

Still happening: https://github.com/larseggert/datatracker/actions/runs/3052008565/jobs/4938157088#step:6:22
I don't get why. git diff origin/main shows me no changes other than to the workflow after pulling origin/main.

Yeah, it looks like it should be working. Looking at it.

It looks like it's checking out https://github.com/larseggert/datatracker/commit/0ba50999d93f1bd210c15d5590cf1d9a8ca89e08 to run the tests. That is old and not the branch in this PR. Not sure why that's happening.

Thanks for debugging this! I guess I need to figure out how to manually kick off a workflow run at the right revision

There should be a pull-down on the 'start workflow' button that lets you choose a branch

Yep, just found that! I used the wrong branch before :(

OK, this seems to work. See https://github.com/larseggert/datatracker/actions/runs/3060670758/jobs/4939479571#step:14:1

Hi Lars -
It's not immediately clear what this is going to concretely provide for us?
What does the microsoft SBOM tool produce and what's going to consume what's produced?

https://www.cisa.gov/sbom

Looking at your latest build, it generates a table in the build logs with a count for each dependency type. Does it provide any output that can be useful to be included as part of a release? What benefits does it bring over something like GitHub dependency scans, which already provide automated alerts and suggested fixes for dependencies with vulnerabilities?

The benefits are that you can check vulnerabilities for installed software, e.g. when you need to determine if you are affected by something like logjam.

https://github.com/ietf-tools/datatracker/network/dependencies

For the datatracker the link Nick points to is probably enough, yes?
But we should talk about providing SBOM for software we expect to be widely installed on other systems, such as xml2rfc. The end goal of the SBOM idea is to make it easier to write a tool that would scan a machine for anything that might be suffering a vulnerability. Could get used by state-validation software for example.

See the comment thread (and I think we had some direct conversation about this at some point)? I'm not seeing value for this for the datatracker over what github already provides?
An SBOM for xml2rfc makes more sense to me.

Closing this per discussion at the 116 codesprint.


access-token provided but disappears

Hi Lynn,
I'm library some strange issues with using the library, in conjunction with the devise_token_auth.
From what I can see in Postman, the headers are being returned:

I can confirm this in the Chrome dev tools network as well:

But nothing is written to Local Storage, or the Cookies if I try that method. I added an interceptor and it seems to return undefined for the header as well.
.config(function($authProvider, $httpProvider) {
  $httpProvider.interceptors.push(function($q, $injector)  {
    return {
     'request': function(config) {
       return config || $q.when(config)
      },

      'response': function (response, $auth) {
        console.log('interceptor response', response)
        console.log('interceptor response', response.headers('access-token'))
        return response || $q.when(response);
      }
    };
  });

I am even trying a special handleLoginResponse:
      handleLoginResponse: function(response, $auth) {
        console.log("response");
        console.log(response);
        $auth.persistData('auth_headers', {

          // save the token
          'Authorization': 'Bearer ' + response['access_token'],

          // convert the expiry value into a date that this module understands
          'expiry': new Date().getTime() + response['expires_in']
        });
        return response.data;
      },

I get a 200 with a serialized user model (no auth_token in the serialized user model) when I sign in, just something weird happens in between!

That Access-Expose-Headers looks suspiciously empty... My CORS in Rails is:
    config.middleware.insert_before 0, "Rack::Cors" do
      allow do
        origins '*'
        resource '*',
          :headers => :any,
          :expose  => ['access-token', 'expiry', 'token-type', 'uid', 'client'],
          :methods => [:get, :post, :options, :delete, :put]
      end
    end

Sorry mate this was totally an issue with me, for some reason the expose isn't working in CORS with that code.
    config.action_dispatch.default_headers = {
      'Access-Control-Expose-Headers' => ['access-token', 'expiry', 'token-type', 'uid', 'client'].join(',')
    }

That forces it to happen without the CORS gem.


How would it work on an account with 100,000 bookmarks

From what I see in readme, in order for me to get the 100k bookmarks it would have to manually scroll through the https://x.com/i/bookmarks page? But that would take forever, no?

Also I want to be able to export all the bookmarks from my X account and delete them after this.

Due to the nature of this script, yes.
In order to export that amount of bookmarks, you need to scroll through the bookmarks page for a very long time, to make sure that Twitter Web App itself loads enough data. This script does not send any requests to Twitter API, it just captures the data that Twitter Web App has already loaded.
You could use a tool like AutoScroll to automate this process.

export bookmarks and delete them after this

This is not possible with this script since we do not send network requests. One could fork this script and modify it to automatically click delete buttons, but please be aware that this could potentially violate Twitter's ToS and get your account locked.


Change audio symbol when muted

Shows the mute symbol when changing the output volume while muted

I did not intend do merge this in here, I must've missclicked somewhere along the way. Sorry for that.


Agenda for Dec 17 Livestream

Get point and normals for watchmen
Visualize lighthouse probability data in voxeltastic
Improve watchman disambiguator
Collect more data as people want it

Regarding the calibration data of the controllers. You should be able to use the same method to download them as with the HMD.
Here is some example data from my Vive and an explanation how to do this using "lighthouse_console" on Windows: https://github.com/nairol/LighthouseRedox/tree/master/data/calibration data

What is up with the mura correction stuff?? That looks wicked! == per-pixel brightness calibration.  People in chat are describing it as a correction for the AMOLED dispaly?

@nairol  - how do I actually get the profile from the controllers? I've tried it wirelessly.  Does it not work wirelessly?

Never tried it myself. It should be able to get the data wirelessly because that is what SteamVR does every time you start it.
I guess I'll have to sniff the USB traffic again. I'll report back once I know more...

Pcaps: https://github.com/nairol/LighthouseRedox/tree/master/data/Vive USB protocol/System

Now, I wish I could understand what's up with that control out message.  It's so jank.

Got points/normals for watchmen after livestream by getting config now.
Visualized
People are working on a better disambiguator
Will be collecting more data!

closing issue.


yarn compatibility

This is not an issue within Storybook, and I'm documenting this here purely for reference/Google-fu.
Yarn is a new alternative to NPM-cli from Facebook. There is a known issue (https://github.com/yarnpkg/yarn/issues/620 and https://github.com/yarnpkg/yarn/issues/709) that prevents it from fully handling scoped packages, such as @kadira/storybook The package gets installed, but yarn outdated command fails with error https://registry.yarnpkg.com/@kadira/storybook: Not found.
The Yarn issue has been reported and waiting for a fix, and until it's resolved there might be this minor frustrations for Storybook devs using Yarn.

Yarn is pretty cool. I hope this will get fixed soon.

The fix has been merged into yarn master with https://github.com/yarnpkg/yarn/pull/1276 so we're just waiting for them to cut a new release

Yarn v0.17.0 has been released and it fixes this issue. All good! 🎉🎉

@eugene1g Thanks for keeping us updated on this.


fix: swap to httpx

aiohttp appears to have issues related to Akamai Global Host and developers
do not seem interested in resolving as a bug.
https://github.com/aio-libs/aiohttp/issues/5643
BREAKING CHANGE: API has changed due to use of httpx.
Modifiers, test_url, and other items that access aiohttp ClientResponse
will need to be fixed.

Codecov Report

Merging #13 (9c5b019) into main (d7b8f69) will decrease coverage by 0.84%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   41.09%   40.24%   -0.85%
==========================================
  Files           9        9
  Lines         640      656      +16
==========================================
+ Hits          263      264       +1
- Misses        377      392      +15

Impacted Files
Coverage Δ

authcaptureproxy/examples/testers.py
0.00% <0.00%> (ø)

authcaptureproxy/auth_capture_proxy.py
21.00% <12.82%> (+0.01%)
:arrow_up:

authcaptureproxy/helper.py
53.50% <16.12%> (-8.60%)
:arrow_down:

authcaptureproxy/cli.py
50.72% <33.33%> (ø)

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 d7b8f69...9c5b019. Read the comment docs.

One day I'll figure out how to get coveralls to work....


CasVersion can't handle wsjar protocol

Version: CAS-5.1.0-RC4-SNAPSHOT
This is a really minor issue, but I get the following warning when starting CAS on a WebSphere Liberty server:
[org.apereo.cas.util.CasVersion] - <Unhandled url protocol: [wsjar] resource: [wsjar:file:/mypath/cas-server-webapp/WEB-INF/lib/cas-server-core-util-5.1.0-RC4-SNAPSHOT.jar!/org/apereo/cas/util/CasVersion.class]>
Could you add the wsjar protocol to CasVersion? It looks like it is structurally the same as the jar protocol.

I'd personally rather not complect CAS components with proprietary protocols and such. For this situation, it's possible to configure WS server to use "standard" jar protocol: http://www-01.ibm.com/support/docview.wss?uid=swg1PM99378

Thanks for the link. I wasn't aware of the config option.


[Snyk] Security upgrade electron from 9.4.4 to 27.3.3

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 4 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

packages/react-devtools/package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Heap-based Buffer Overflow SNYK-JS-ELECTRON-5923343
  529

Use After Free SNYK-JS-ELECTRON-6253729
  218

Heap-based Buffer Overflow SNYK-JS-ELECTRON-6253728
  211

Out-of-bounds Read SNYK-JS-ELECTRON-5734243
  152

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Use After Free

🎉 Snyk hasn't found any issues so far.
✅ security/snyk check is completed. No issues were found. (View Details)
✅ license/snyk check is completed. No issues were found. (View Details)


Incorrect answer on Assignment 10.1 2d

The correct answer is (3, 8] and the posted answer is (-oo,3] U (8,oo), which is incorrect.

Correct, I provided the complement of the given set mistakenly. Good catch. I'll fix this later tonight.


ungit doesn't open

Yesterday Ungut stopped working – I don't know why.
When I toggle Ungut, nothing happens.
This is the output in the console:
atom-ungit.coffee [sm]:121 Users/bastian/.atom/packages/atom-ungit/node_modules/ungit/bin/ungit [-v] [-b]
[--cliconfigonly] [--gitVersionCheckOverride]

Options:
  -o, --gitVersionCheckOverride   Ignore git version check and allow ungit to
                                  run with possibly lower versions of git
                                                                [default: false]
  --ungitVersionCheckOverride     Ignore check for older version of ungit
                                                                [default: false]
  -b, --launchBrowser             Launch a browser window with ungit when the
                                  ungit server is started. --no-b or
                                  --no-launchBrowser disables this
                                                                 [default: true]
  --cliconfigonly                 Ignore the default configuration points and
                                  only use parameters sent on the command line
                                                                       [boolean]
  --port                          The port ungit is exposed on   [default: 8448]
  --urlBase                       The base URL ungit will be accessible from
                                                   [default: "http://localhost"]
  --rootPath                      The root path ungit will be accessible from
                                                                   [default: ""]
  --logDirectory                  Directory to output log files  [default: null]
  --logRESTRequests               Write REST requests to the log [default: true]
  --logGitCommands                Write git commands issued to the log
                                                                [default: false]
  --logGitOutput                  Write the result of git commands issued to the
                                  log                           [default: false]
  --bugtracking                   This will automatically send anonymous bug
                                  reports                        [default: true]
  --sendUsageStatistics           Google analytics for usage statistics
                                                                 [default: true]
  --authentication                True to enable authentication. Users are
                                  defined in the users configuration property
                                                                [default: false]
  --users                         Map of username/passwords which are granted
                                  access
  --showRebaseAndMergeOnlyOnRefs  Set to false to show rebase and merge on drag
                                  and drop on all nodes          [default: true]
  --maxConcurrentGitOperations    Maximum number of concurrent git operations
                                                                    [default: 4]
  --forcedLaunchPath              Define path to be used on open. Can be set to
                                  null to force the home screen
  --autoShutdownTimeout           Closes the server after x ms of inactivity.
                                  Mainly used by the clicktesting
  --maxNAutoRestartOnCrash        Maximum number of automatic restarts after a
                                  crash. Undefined == unlimited     [default: 0]
  --noFFMerge                     Don't fast forward git mergers. See git merge
                                  --no-ff documentation          [default: true]
  --autoFetch                     Automatically fetch from remote when entering
                                  a repository using ungit       [default: true]
  --dev                           Automatically does stash -> operation -> stash
                                  pop when you checkout, reset or cherry pick
                                                                 [default: true]
  --logLevel                      The logging level, possible values are none,
                                  error, warn, info, verbose, debug, and silly.
                                                               [default: "warn"]
  --launchCommand                 Specify a custom command to launch. `%U` will
                                  be replaced with the URL that corresponds with
                                  the working git directory.
  --allowCheckoutNodes            Allow checking out nodes (which results in a
                                  detached head)                [default: false]
  --allowedIPs                    An array of ip addresses that can connect to
                                  ungit. All others are denied   [default: null]
  --autoPruneOnFetch              Automatically remove remote tracking branches
                                  that have been removed on the server when
                                  fetching. (git fetch -p)       [default: true]
  --pluginDirectory               Directory to look for plugins
                                      [default: "/Users/bastian/.ungit/plugins"]
  --pluginConfigs                 No supported as a command line argument, use
                                  ungitrc config file.  See README.md
                                                                   [default: {}]
  --autoStashAndPop               Used for development purposes  [default: true]
  --fileSeparator                 OS dependent file separator     [default: "/"]
  --disableDiscardWarning         disable warning popup at discard
                                                                [default: false]
  --disableDiscardMuteTime        duration of discard warning dialog mute time
                                  should it be muted           [default: 300000]
  --lockConflictRetryCount        Allowed number of retry for git "index.lock"
                                  conflict                          [default: 3]
  --autoCheckoutOnBranchCreate    Auto checkout the created branch on creation
                                                                [default: false]
  --alwaysLoadActiveBranch        Always load with active checkout branch
                                                                [default: false]
  --numberOfNodesPerLoad          number of nodes to load for each git.log call
                                                                   [default: 25]
  --mergeTool                     the git merge tool to use when resolving
                                  conflicts                     [default: false]
  -h, --help                      Show help                            [boolean]
  -v, --version                   Show version number                  [boolean]
  --socketId                                                        [default: 0]

Examples:
  Users/bastian/.atom/packages/atom-ungit/  Run Ungit on port 8888
  node_modules/ungit/bin/ungit --port=8888
  Users/bastian/.atom/packages/atom-ungit/  Turn off REST logging but turn on
  node_modules/ungit/bin/ungit              git command log
  --no-logRESTRequests --logGitCommands

(anonymous function) @ atom-ungit.coffee [sm]:121emitOne @ events.js:101emit @ events.js:188readableAddChunk @ _stream_readable.js:176Readable.push @ _stream_readable.js:134onread @ net.js:543
atom-ungit.coffee [sm]:121 Unknown argument: socketId
(anonymous function) @ atom-ungit.coffee [sm]:121emitOne @ events.js:101emit @ events.js:188readableAddChunk @ _stream_readable.js:176Readable.push @ _stream_readable.js:134onread @ net.js:543

Unknown argument: socketId
(anonymous function) @ atom-ungit.coffee [sm]:121emitOne @ events.js:101emit @ events.js:188readableAddChunk @ _stream_readable.js:176Readable.push @ _stream_readable.js:134onread @ net.js:543

This was related with recent strict enforcement of config variables and should be now fixed in<EMAIL_ADDRESS> you can fix this by either upgrading ungit within atom plugin folder or upgrading your global ungit.
more on this are here.  https://github.com/FredrikNoren/ungit/pull/899

thanks @codingtwinky !
I ran sudo -H npm update -g ungit within ~/.atom/packages/atom-ungit and the problem disappeared.


The extension is not able to read data from raindrop.io

Extension
https://www.raycast.com/lardissone/raindrop-io
Description
https://www.raycast.com/lardissone/raindrop-io is not able to read data from raindrop.io.
I have both Alfred and Raycast.  The test token has been working correctly for Alfred but not for Raycast.  I am using different test tokens for these two apps.
Steps To Reproduce

Configure https://www.raycast.com/lardissone/raindrop-io and follow the instructions.
Use the test token from raindrop -

Execute "Search Bookmark"

Current Behaviour
I got the following in the terminal when the Raycast displayed "Cannot fetch collections"

I got the following in the terminal when the Raycast displayed "cannot search bookmark"

Expected Behaviour
I should be able to see the bookmark from raindrop app.

Hi,
I tested the Test Token with the following URLs in Postman and both of them
are fetching details.  But in the application I am getting the errors
"Cannot fetch collections" and "Cannot search bookmark".
https://api.raindrop.io/rest/v1/collections/all
https://api.raindrop.io/rest/v1/user
Regards,
Ravi
On Mon, Oct 30, 2023 at 10:45 PM raycastbot @.***>
wrote:

Thank you for opening this issue!
🔔 @lardissone https://github.com/lardissone @ShubhamVerma1811
https://github.com/ShubhamVerma1811 @pernielsentikaer
https://github.com/pernielsentikaer you might want to have a look.
💡 Author and Contributors commands
The author and contributors of lardissone/raindrop-io can trigger bot
actions by commenting:

@raycastbot close this issue Closes the issue.
@raycastbot rename this issue to "Awesome new title" Renames the
issue.
@raycastbot reopen this issue Reopen the issue.

—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/issues/9034#issuecomment-1785694519,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIZULRLHPDJ7VHL7JCDSG3YB7ODLAVCNFSM6AAAAAA6WL7KHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBVGY4TINJRHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Regards,
Ravi

Not sure why raycastbot closed the issue.
That is weird, are you behind a proxy on raycast?

Yes, I am behind a proxy.
Regards,
Ravi
On Mon, Oct 30, 2023 at 11:50 PM Leandro Ardissone @.***>
wrote:

Not sure why raycastbot closed the issue.
That is weird, are you behind a proxy on raycast?
—
Reply to this email directly, view it on GitHub
https://github.com/raycast/extensions/issues/9034#issuecomment-1785800096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIZULRWESJMMG4F655IZZ3YB7VXLAVCNFSM6AAAAAA6WL7KHCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBVHAYDAMBZGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Regards,
Ravi

@richstandbrook if you download their official app, does it works?

Yep, the app is fine. I've tried creating a new access token

Works now, I was using the client secret instead of the test token

@richstandbrook awesome!
Good to know, probably others are experiencing the same!
I'll add more emphasis that we require the Test Token.


[+0-normal-args] Use SILGenBuilder in SILGenConvert when injecting an…

… optional so we propagate through +0 vs +1.
This commit does two different things:

Changes some code paths in SILGenConvert to use SILGenBuilder APIs.
It eliminates a few bugs in this path exposed by running the SILGen tests
with +0 enabled.

The bug specifically is that we are assuming in this code that the input value
is always at +1, so we unconditionally forward the input value and then create
destroys on the output. This is not correct in general (since we have guaranteed
values) and occurs all the time with +0 arguments enabled.
This is tested by running the SILGen tests with +0 arguments enabled.
rdar://34222540

@swift-ci smoke test


local provider and csh

the syntax cmd = "/bin/bash ... 1>{0}.out 2>{0}.err" will not work if a user has csh.
https://github.com/Parsl/parsl/blob/master/parsl/providers/local/local.py#L235

Do you mean when a user does not have bash installed? Or that if a user's default shell is csh, then this does not invoke bash correctly?

Do you mean when a user does not have bash installed? Or that if a user's default shell is csh, then this does not invoke bash correctly?

i ran into trouble when my default shell is csh. The machine has bash installed.

i ran into trouble when my default shell is csh. The machine has bash installed.


Link to backend repo?

The readme mentions this is the frontend app to the gitrank project, but does not provide a link to info on the backend.

I should change the README, this includes the full stack


Supported Extensions Not Documented

This page mentions that only two extensions are supported, however it provides no links to further reading on the correct usage of those extensions or even a summary of what they do. Searching broadly on the Azure docs site does not bring up any results for either of them, nor on Github.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: be16b5c5-40c4-3e6c-0b0d-840373c63f9f
Version Independent ID: ced6b3ba-8da1-dd3d-9d1c-8a21dabd44fe
Content: Restrictions and details of API formats support - Azure API Management
Content Source: articles/api-management/api-management-api-import-restrictions.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

@RossMurr4y Thanks for the feedback! We will review and update accordingly.

@RossMurr4y The x-ms-paths allows you to define paths that are differentiated by query parameters in the URL which is covered in the autorest docs. And the x-servers is a backport of the OpenAPI 3's servers object for OpenAPI 2.
We are assigning this issue to the content author further review and to update the docs accordingly.


Add support for other words ("decade", "fortnight", "century"...)

There are some words that can be added to the library:

Decade: 10 years (2 decade --> 20 years, ...)
Fortnight: 14 days (Example: "a fortnight ago")
Century: 100 years (Example: "a century ago", "two centuries ago")

It's not necessary to put them all in a unique PR, you can submit just one of them. Just take in mind that it should support "next decade", "last decade", "a decade ago", "in a decade", but also "two decades ago".

We should see if we can get some of these words from CLDR data.

Hi,
If no one else is already doing it, I'll try to add support for "decade".

In my pull request #762, I made the assumption that "next decade" and "in a decade" both refer to "in 10 years". Similarly, I assumed that "last decade" and "a decade ago" both refer to "10 years ago". Please let me know if this makes sense and if there is anything else I should do. @noviluni

Hi @seric17, thank you for submitting this PR, I will check it when having a moment :slightly_smiling_face:


Updated metadata to new doc author

This is related to the info provided by chrfox based on https://github.com/MicrosoftDocs/microsoft-365-docs/pull/498#issuecomment-537583059

@Dansimp - PR is ready for final review. This is related to the info provided by chrfox based on https://github.com/MicrosoftDocs/microsoft-365-docs/pull/498#issuecomment-537583059
Thank you.
cc: @kenwith


catch up Django renaming Model._meta.module_name to model_name

This is the change talked in https://github.com/s-block/django-nested-inline/issues/14
Please make a Pypi release if possible. Thanks!

Replaced by #149


Add specific event for OpenThread

Description
Add a specific event to support next OpenThread updates to nanoClr as part of IDF 5.1.3 update.
Motivation and Context
Needed for next OpenThread update
Types of changes

[x] Improvement (non-breaking change that improves a feature, code or algorithm)
[ ] Bug fix (non-breaking change which fixes an issue with code or algorithm)
[ ] New feature (non-breaking change which adds functionality to code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Config and build (change in the configuration and build system, has no impact on code or features)
[ ] Dependencies (update dependencies and changes associated, has no impact on code or features)
[ ] Unit Tests (add new Unit Test(s) or improved existing one(s), has no impact on code or features)
[ ] Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

[x] My code follows the code style of this project (only if there are changes in source code).
[ ] My changes require an update to the documentation (there are changes that require the docs website to be updated).
[ ] I have updated the documentation accordingly (the changes require an update on the docs in this repo).
[x] I have read the CONTRIBUTING document.
[x] I have tested everything locally and all new and existing tests passed (only if there are changes in source code).
[ ] I have added new tests to cover my changes.

I will just merge as this is very minor change. but required by next set of openthread changes.


replaced two links

as per the user report #8193 , so i replaced with new links
Below new links are added
for Microsoft System Center 2012 Endpoint Protection
https://docs.microsoft.com/lifecycle/products/microsoft-system-center-2012-endpoint-protection
for Windows 10 Enterprise Evaluation
https://www.microsoft.com/en-us/evalcenter/evaluate-windows-10-enterprise
But i could not find an alternate for  Protecting the pre-OS environment with UEFI

By the way, you can easily close the linked issue ticket automatically, when this PR gets merged, simply by adding the following text at the end of your PR comment:
Closes #8193


Support Node.js v16 LTS

Node.js v16 is the current long-term support release.
Azure Functions doesn't even appear to have an announced support timeline for it.
Is Azure Functions a dead product? What's going on here?

Hi @aaronadamsCA I've just transferred another issue to this repo which covers the same topic. Please see https://github.com/Azure/azure-functions-nodejs-worker/issues/455 for more information


w07d02

commit two redo actually does have commit 3 but I commented it out and forgot to commit..sorry
completeness: 5
comfort: 3
without collaboration and Robin's help I don't know if i would've got this right. I really wanted to do bonus but its 10.52pm and I think I really need to stop even though I don't want to at all.
good homework assignment

:star: Nice job on this assignment! Also good for you for knowing when to stop!


The documentation of hipEventQuery is incorrect

The documentation of hipEventQuery() says that

Query the status of the specified event. This function will return hipSuccess if all commands in the appropriate stream (specified to hipEventRecord()) have completed. If that work has not completed, or if hipEventRecord() was not called on the event, then hipErrorNotReady is returned.

However, calling hipEventQuery(event) immediately after hipEventCreate(&event), before calling hipEventRecord(event, stream), actually returns hipSuccess.
This is consistent with the documented behaviour of hipEventSynchronize():

If hipEventRecord() has not been called on event, this function returns immediately.
that implies that a newly created event is in "ready" or "recorded" state.

Here is a sample code showing the actual behaviour:
__global__
void sleep(uint64_t delta) {
  if (blockIdx.x == 0 and threadIdx.x == 0) {
    uint64_t start = clock64();
    while (clock64() - start < delta);
  }
}

void test_hip(void) {
  // initialise the HIP runtime
  HIP_CHECK(hipFree(nullptr));

  // create a HIP stream
  hipStream_t stream;
  HIP_CHECK(hipStreamCreate(&stream));

  // create a HIP event
  hipEvent_t event;
  HIP_CHECK(hipEventCreate(&event));

  // check the event
  auto status = hipEventQuery(event);
  std::cout << "HIP event status: " << status << " (" << hipGetErrorName(status) << ")\n";

  // submit asynchronous work and record the event
  uint64_t rate_kHz = 1'700'000;  // 1700 MHz
  uint64_t time_ms = 10'000;      // 10 s
  sleep<<<1, 1, 0, stream>>>(time_ms * rate_kHz);
  HIP_CHECK(hipEventRecord(event, stream));

  // check the event
  status = hipEventQuery(event);
  std::cout << "HIP event status: " << status << " (" << hipGetErrorName(status) << ")\n";

  // wait for all asynchronous work to complete
  HIP_CHECK(hipEventSynchronize(event));
  std::cout << "asynchronous work complete\n";

  // check the event
  status = hipEventQuery(event);
  std::cout << "HIP event status: " << status << " (" << hipGetErrorName(status) << ")\n";

  // clean up
  HIP_CHECK(hipEventDestroy(event));
  HIP_CHECK(hipStreamDestroy(stream));
}

Running it prints
HIP event status: 0 (hipSuccess)
HIP event status: 600 (hipErrorNotReady)
asynchronous work complete
HIP event status: 0 (hipSuccess)

The first message (HIP event status: 0 (hipSuccess)) confirms that a newly created event is in "ready" or "recorded" status.

I see the inconsistency. Investigating this internally, will update here once I have more info.

Discussed this with the team internally. The documentation needs to be fixed.
HIP runtime behavior is correct. If hipEventRecord was not called on a newly created event, we will return hipSuccess, since there are no pending events.
(We have initiated the process to fix the documentation).

This is fixed in our latest documentation.
Thanks @jujiang-del for updating.


[Mac Release] Caps Lock has letters from QWERTY bound

Expected Behavior
On pressing caps lock, the letters should still be in the Canary layout.
Actual Behavior
The keys are in this strange layout. Inspected with Ukulele.

Suggested Fix
Rebind the keys in Canary.bundle using Ukulele and re-release for Mac.
How do I contribute to a release file?

I gave up and used Karabiner Elements, it allows me to rebind the caps lock and shortcuts too.

If you want to contribute a fix, simply make a Pull Request with the corrected file and I'll review it. Thanks!


Develop

@pachi81 Hi! I just have a quick question. Do you have any news about alarms functionality? I had a moment to prepare a manual for using GDH as a LibreLink Follower and this alarm feature on parents' phone could really be useful. If this is going to be available anytime soon, I will just hold on to the text and wait for a new release to include a description for activating alarms.

@pachi81 Hi! I just have a quick question. Do you have any news about alarms functionality? I had a moment to prepare a manual for using GDH as a LibreLink Follower and this alarm feature on parents' phone could really be useful. If this is going to be available anytime soon, I will just hold on to the text and wait for a new release to include a description for activating alarms.

Hi,
I know, that many people asked for... At the moment, I only have limited time and so much todo...
I want todo a refactor of the UI to be easier to use. I want to add support for IOB and COB...
So, I don´t know, when I can implement the alarm feature. The biggest problem, I want to have some good solution with snooze an alarm, provide different settings, ...
Also I have less experience with alarm notification for dnd and so on.
That is the reason, I am procrastinating it at the moment ;-)
But yes, I have to do it... but it can take some weeks. Hopefully faster...

I get it and totally understand :D In that case I will post my manual and later update it with alarms function, when it is available. Thanks!

It's me again :)
Today I recommended GDH to somebody with iOS on her phone and a watch with WearOS. She wanted to get gluco data sent via Nightsocut shown on the watch, so GDH would be perfect. However, when she installed the WearOS version on the watch, there was no way to log in to her server (or enter LibreView data as well). Those options are only available in the phone version.
When you deal with all current issues you want to implement, can you, please, think about some way to allow logging into Nightscout or Libre Link Follower on the WearOS verision only?

It's me again :) Today I recommended GDH to somebody with iOS on her phone and a watch with WearOS. She wanted to get gluco data sent via Nightsocut shown on the watch, so GDH would be perfect. However, when she installed the WearOS version on the watch, there was no way to log in to her server (or enter LibreView data as well). Those options are only available in the phone version.
When you deal with all current issues you want to implement, can you, please, think about some way to allow logging into Nightscout or Libre Link Follower on the WearOS verision only?

Hi,
oh, I have never seen the wear app as standalone. But you are right, connected to an iPhone, you do not have any option to change the settings. You are right, I have to add the settings also to watch...
The problem is to sync with phone version for android users...
Another thing, can you update the polish strings:
https://github.com/pachi81/GlucoDataHandler/pull/117/files#diff-0908d478f1e1608a4d8bf0d6e84f3c23dd963a61f2ff17a7984af7e3e9f938d1

I translated the Polish parts.
Thanks for considering the above issue. I am no programmer, so I cannot help with practical solution of the syncing problem. The only idea I have is to intorduce som condition - wear os asks if there is a phone version, if yes, login options are inactive, if no - login options on wearOS become available. Could anything like that work?

I translated the Polish parts.

Thank you very much for the quick translation!

The only idea I have is to intorduce some condition - wear os asks if there is a phone version, if yes, login options are inactive, if no - login options on wearOS become available. Could anything like that work?

This was also my first intention, but then I get a problem every time the phone is disconnect (ok, I can remember that there was a phone connected..., but as long I´m thinking about, I don´t like).
Maybe syncing is not the biggest problem... To create input view for the settings I think is more complicated, as I do not have a lot of experience and on the tiny watch, every input is not so easy.
But I will check, after I have created the alarm settings ;-)


ValueError: Too many bytes when decoding varint

Describe the bug
Script exited unexpectidly after running base example
Package Version
My version of TikTokLive is 6.0.1.
Operating System
Docker on image python:3.11.5-alpine
Trace
Traceback (most recent call last):
File "/project/python/./main.py", line 24, in
client.run()
File "/user/Library/Python/3.9/lib/python/site-packages/TikTokLive/client/client.py", line 170, in run
return self._asyncio_loop.run_until_complete(self.connect(**kwargs))
File "/Applications/Xcode.app/Contents/Developer/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
return future.result()
File "/user/Library/Python/3.9/lib/python/site-packages/TikTokLive/client/client.py", line 155, in connect
await task
File "/user/Library/Python/3.9/lib/python/site-packages/TikTokLive/client/client.py", line 203, in _client_loop
async for event in self._ws_loop(initial_response):
File "/user/Library/Python/3.9/lib/python/site-packages/TikTokLive/client/client.py", line 240, in _ws_loop
for event in self._parse_webcast_response(response_message):
File "/user/Library/Python/3.9/lib/python/site-packages/TikTokLive/client/client.py", line 322, in _parse_webcast_response
proto_event: ProtoEvent = event_type().parse(response.payload)
File "/user/Library/Python/3.9/lib/python/site-packages/betterproto/init.py", line 1015, in parse
value = self._postprocess_single(
File "/user/Library/Python/3.9/lib/python/site-packages/betterproto/init.py", line 955, in _postprocess_single
value = cls().parse(value)
File "/user/Library/Python/3.9/lib/python/site-packages/betterproto/init.py", line 1015, in parse
value = self._postprocess_single(
File "/user/Library/Python/3.9/lib/python/site-packages/betterproto/init.py", line 955, in _postprocess_single
value = cls().parse(value)
File "/user/Library/Python/3.9/lib/python/site-packages/betterproto/init.py", line 1008, in parse
decoded, pos = decode_varint(parsed.value, pos)
File "/user/Library/Python/3.9/lib/python/site-packages/betterproto/init.py", line 487, in decode_varint
raise ValueError("Too many bytes when decoding varint.")
ValueError: Too many bytes when decoding varint.
Task was destroyed but it is pending!
task: <Task pending name='Task-6' coro=<WebSocketCommonProtocol.transfer_data() running at /user/Library/Python/3.9/lib/python/site-packages/websockets/legacy/protocol.py:963> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x102716ee0>()]> cb=[<TaskWakeupMethWrapper object at 0x1026d13d0>()]>
Task was destroyed but it is pending!
task: <Task pending name='Task-7' coro=<WebSocketCommonProtocol.keepalive_ping() running at /user/Library/Python/3.9/lib/python/site-packages/websockets/legacy/protocol.py:1254> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x102716f70>()]>>
Task was destroyed but it is pending!
task: <Task pending name='Task-8' coro=<WebSocketCommonProtocol.close_connection() running at /user/Library/Python/3.9/lib/python/site-packages/websockets/legacy/protocol.py:1301> wait_for=<Task pending name='Task-6' coro=<WebSocketCommonProtocol.transfer_data() running at /user/Library/Python/3.9/lib/python/site-packages/websockets/legacy/protocol.py:963> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x102716ee0>()]> cb=[<TaskWakeupMethWrapper object at 0x1026d13d0>()]>>
Exception ignored in: <coroutine object WebSocketCommonProtocol.close_connection at 0x1026c46c0>
Traceback (most recent call last):
File "/user/Library/Python/3.9/lib/python/site-packages/websockets/legacy/protocol.py", line 1337, in close_connection
File "/user/Library/Python/3.9/lib/python/site-packages/websockets/legacy/protocol.py", line 1355, in close_transport
File "/user/Library/Python/3.9/lib/python/site-packages/websockets/legacy/protocol.py", line 1379, in wait_for_connection_lost
File "/user/Library/Python/3.9/lib/python/site-packages/websockets/legacy/async_timeout.py", line 74, in timeout
RuntimeError: no running event loop

Does this occur on 6.0.2-post1?

fixed 6.0.3 (i think). reopen if continues


Unit testing of signature verification routines

Closes #200
Based on #232

ACK 1da7a02

FWIW your last push increased coverage by another 1.59%.
tarpaulin-report.html.txt


Re-test CADeT ingestion for Athena compatibility

JHP has flagged a coming upgrade to the dbt athena adapter version - This is plan to be merged to main by 10th July.
There are some changes to how metadata are generated, which are what datahub uses for ingestion. see https://docs.getdbt.com/docs/dbt-versions/core-upgrade/upgrading-to-v1.7#for-consumers-of-dbt-artifacts-metadata
original slack message form Jacob:
https://mojdt.slack.com/archives/C03B3UTAJJ1/p1719310300786479
We'll need to use the test branch https://github.com/moj-analytical-services/create-a-derived-table/tree/dbt-athena-upgrade-182 to locally generate the docs and run our ingestion complete with it's customisations to test we can still run ingestion of cadet as we had intended

Update on this:

Have locally created a manifest and catalogue file using the dbt-athena-upgrade-182 branch. This was done for the dev cadet environment (not prod) as it was being difficult for prod. Still the files should be created using the new schema.
I got a run_results.json from matt price, who had deployed some em models using the new branch.
I uploaded to mojap-derived-tables/catalogue_static_dbt_docs/new-athena-version-docs
I cleared all dbt data fro datahub and ingested the above files into dev datahub.
As you'll see from dev the data have ingested, but not been added to databases, but i think it's more because of different parsing rules  (cadet model filepath >> database.table) for dev dbs, that's it's not added any datasets to databases- Probably why it was really quick too

The upgrade is being delayed for a week and will now happen on 17 July

It doesn't appear that the upgrade will impact our ingestion of create-a-derived-table dbt data.
The test ingestions have needed to be run in slightly different way to what will happen in production cadet though - i.e. created locally using the development environment of cadet.
There were some issues with the ingestion but these were related to using the cadet dev environment - which creates database names in a way we do not account for in our custom ingestion scripts (adding a _dev_dbt suffix).
We could potentially improve how we infer the database name from the manifest file as there are values in it we could use that are representative of the database name post parsing of the dbt model filename.

As i had made some improvements to our custom dbt ingestions with regard to how we infer the athena database name - in order to be able to test out the upgrade that was in the cadet dev environment - i tidied these up into a PR: https://github.com/ministryofjustice/data-catalogue/pull/195
Will mean we can now test our ingestions using the dev cadet environment which will be where new cadet features are rolled out to in advance of prod deployment


Incompatible with Hearth and Home

Please provide a log where the error is shown

There is no error in the console shown. Its just not loading the mod at all

Have you got the most recent version?

1st of September Release

I update bepinex now it worked


CuPy-only option for GPU

Hello,
I use ssqueezepy extensively in many of my projects, and I have recently started using it with GPU acceleration, which works really well. Kudos to the developer.
I also use Cupy in most of my projects, so it is not a big deal that ssqueezepy depends on it also. But I never use PyTorch, and it is a huge dependency (both in size and complexity) which I would have been more than happy to skip. How feasible would it be to make PyTorch an optional dependency, relying only on Cupy?
Thank you.

It's used mainly for FFT, though there's also cupy.fft. Torch also enables autograd, but that's not this library's standard.
I'm not willing to develop this, but contributions are welcome. Easiest would be to find CuPy equivalents to all torch syntax that ssqueezepy uses, in your own local branch, only on parts you need, so you don't have to worry about keeping your edits up to contribution standards.

Thank you for the rapid answer.


Panic does not exit

I noticed that when ever a panic is thrown, librespot stops working completely but does not exit.
This is unfortunate, as it would be restarted by systemd in my setup if it exits.

Librespot has a few threads, and a panic on one does not always cause others to stop.
You should be able to make all panics into process aborts by adding to Cargo.toml (assuming you're building with --release):
[profile.release]
panic = "abort"

Let me know if that works. This seems like a reasonable thing to do, so I could add it here.

Doesn't that also mean that the stack is not unwound and we won't get any stack traces if it panics? I actually added
[profile.release]
debug = true

just to get nice stack traces. (Debug build is not performant enough to smoothly play music)

Ok, I tried it. Strangely, now when librespot crashes, it seems to hang with 100% CPU utilization. Also, I don't get a stack trace.

@wfraser You seem to be very fluent in all this multiprocess/multithreading task - could you try to solve this issue? Reproduction requires disconnection of broadband during playing - should be easy to reproduce on virtual machine.

This behaviour is known to me e.g. when the player thread panics. @JasonLG1979 is putting quite a lot of effort in error handling in #823, let's see how that goes.

Can you test and report back now that #823 has been merged?

I find that with v0.3.1 librespot will exit after a crash.
thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', connect/src/spirc.rs:1155:29
...
[2021-11-17T14:28:17Z DEBUG librespot_connect::spirc] drop Spirc[0]
[2021-11-17T14:28:17Z DEBUG librespot_playback::player] Shutting down player thread ...
[2021-11-17T14:28:17Z DEBUG librespot_playback::player] drop PlayerInternal[0]
[2021-11-17T14:28:17Z DEBUG librespot_playback::player] PlayerInternal thread finished.
[2021-11-17T14:28:17Z DEBUG librespot_core::session] drop Session[0]
[2021-11-17T14:28:17Z DEBUG librespot_core::mercury] drop MercuryManager
[2021-11-17T14:28:17Z DEBUG librespot_core::session] drop Dispatch
$

I find that with v0.3.1 librespot will exit after a crash.

That would be happening here:
https://github.com/librespot-org/librespot/blob/dev/connect/src/spirc.rs#L1155

So that means we can close this issue?

So that means we can close this issue?

No. That means we need to make sure this isn't zero:
https://github.com/librespot-org/librespot/blob/dev/connect/src/spirc.rs#L1141

We're trying to grab the 1st thing (index 0) from a thing with a length of 0.

I'm not exactly sure of the context of what exactly the thing is but the return type is an option. Provided whatever is asking for it actually checks it and doesn't just unwrap we can just return None if the len is 0.

So that means we can close this issue?

Yes, in my opinion this issue has been solved (not exiting on panic).

So that means we can close this issue?

No. That means we need to make sure this isn't zero:
https://github.com/librespot-org/librespot/blob/dev/connect/src/spirc.rs#L1141

I'd suggest that be opened as a separate issue.

I agree. Thanks to @JasonLG1979 so that we can now close this issue, and @uSpike for reporting back on it. Do file that new issue with steps how to reproduce that.


v4: Uncaught Package not found tslib

When I try to bundle my app I get an Uncaught Package not found tslib error in dev mode without dev server. Please note that in production mode all works well
class Context {
    runServer;
    getConfig = (useSourceMap) =>
        fusebox({
            target: "browser",
            homeDir: "HomeDir",
            entry: "boot-client.tsx",
            output: 'root/$name',
            webIndex: {
                target: "root",
            },
            tsConfig: 'tsconfig.json',
            cache: false,
            devServer: false,
            useTypescriptCompiler: true,
            sourceMap: useSourceMap
        });
}
const { task } = sparky<Context>(Context);
task("start:dev", async ctx => {
    ctx.runServer = false;
    const fuse = ctx.getConfig(true);
    await fuse.runDev();
});

dependencies: {
    include: ['tslib'],
 }

It Works :)
Thank you!
And sorry for the dumb issue...

Do you think maybe this should be a default include?

@SephReed I am working on production mode in alpha based on fuse-compiler. There won't be any need for tslib after that ;-)

Awesome.
Is there a good place to add an error message that says something like:

Uncaught Package not found common-package.  Perhaps add dependencies: { include: ['common-package'], } to your fuse config.

Either resolver or assemble )

It seems like here might be the place:
https://github.com/fuse-box/fuse-box/blob/85597f7ba183e2c7b246a5dee67d63404b863688/modules/fuse-loader/index.js#L187-L189
throw `Package not found ${pkgName}.  If this is a core dependency, you may want to include "dependencies: { include: ['common-package'], }" to your fuse config`;


Fixed bug where Images::build panics on "status" replies

I came across this bug when trying to build an image which relies on a repository that doesn't exist locally. The Docker API replies with "status" messages for downloading/extracting the base image.

Coverage remained the same at 100.0% when pulling 6ad1132bdfb51f8155ee03c9d5ac72823c6c2012 on petehayes102:master into 23420f23dc890218dac1781dfc94596b6e1b1ee9 on softprops:master.

Nice catch. This seems to line up with thier latest docs https://docs.docker.com/engine/reference/api/docker_remote_api_v1.24/#create-an-image


Update/Add PHP 7.4

Tool information

Tool name: PHP
Tool license:
Add or update? Add
Desired version: 7.4
Approximate size:
If this is an add request:

Brief description of tool: Our pipelines stopped working when you changed the default PHP version to 8.0.1
URL for tool's homepage:
Provide a basic test case to validate the tool's functionality:

Area for Triage:

Question, Bug, or Feature?:

Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[x] Windows Server 2019

Can this tool be installed during the build?

Tool installation time in runtime

Are you willing to submit a PR?

Hello @qualmon , could you please try setup-php according to https://github.com/actions/virtual-environments/issues/2416 ?

Hello @qualmon , could you please try setup-php according to https://github.com/actions/virtual-environments/issues/2416 ?

Hi @maxim-lobanov, can you give me any pointers on how to get started? We are running Azure Pipelines through Microsoft hosted agents. The documentation only covers changing php version on the ubuntu virtual environment.

Hi @maxim-lobanov, can you give me any pointers on how to get started? We are running Azure Pipelines through Microsoft hosted agents. The documentation only covers changing php version on the ubuntu virtual environment.

@qualmon Could you please try running
choco install php --version=7.4.14 --allow-downgrade as the very first step of your pipeline? Does it help?

@qualmon Could you please try running
choco install php --version=7.4.14 --allow-downgrade as the very first step of your pipeline? Does it help?

Thanks, that kind of works. It's installing PHP v7.4.14 but it's extremely slows (takes about 2 minutes). Also subsequent steps fail with php: command not found, even though it looks like the new path is added to the path.

Thanks, that kind of works. It's installing PHP v7.4.14 but it's extremely slows (takes about 2 minutes). Also subsequent steps fail with php: command not found, even though it looks like the new path is added to the path.

@qualmon oh, sorry, haven't thought about the path. Can you try installing PHP into the same directory as 8.0?
choco install php --version=7.4.14 --allow-downgrade -package-parameters='"/InstallDir:C:\tools\php"'

@qualmon oh, sorry, haven't thought about the path. Can you try installing PHP into the same directory as 8.0?
choco install php --version=7.4.14 --allow-downgrade -package-parameters='"/InstallDir:C:\tools\php"'

That works, thank you. It was a bit faster the second time (or maybe it's random) but still quite slow (47 seconds).

That works, thank you. It was a bit faster the second time (or maybe it's random) but still quite slow (47 seconds).

Hello @qualmon , we are glad to hear that workaround above helped you.
Sorry for inconvenience, but we wouldn't like to bake PHP 7.4 into images by default due to maintenance concerns (installing multiple PHP versions side by side on Windows images). Currently, we install the latest stable version.
Please let us know if you have any concerns

Hello @qualmon , we are glad to hear that workaround above helped you.
Sorry for inconvenience, but we wouldn't like to bake PHP 7.4 into images by default due to maintenance concerns (installing multiple PHP versions side by side on Windows images). Currently, we install the latest stable version.
Please let us know if you have any concerns


Improve MultilabelFbetaScoreEvaluator

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (fffc008) to head (d27a9af).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #555   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          131       131
  Lines         3708      3713    +5
  Branches       182       182
=========================================
+ Hits          3708      3713    +5

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


Date Range Selection

Date Range Selection Mode to be implemented which helps to select a range of dates in the Events Calendar.

Implemented -> Should use version 1.5.1 or above


Unable to profile in container despite setting -e ctimer

Describe the bug
unable to profile in container despite setting -e ctimer
Expected vs. actual behavior
No response
Reproduction Steps
xx@container[java-test]:/home/xx# ./asprof 1
Target JVM failed to load asyncProfiler
xx@container[java-test]:/home/xx# java -agentpath:/home/xx/libasyncProfiler.so=start,flat -version -e ctimer
[WARN] Kernel symbols are unavailable due to restrictions. Try
sysctl kernel.perf_event_paranoid=1
sysctl kernel.kptr_restrict=0
[WARN] perf_event_open for TID 1466 failed: Operation not permitted
[WARN] perf_event_open for TID 1467 failed: Operation not permitted
[WARN] perf_event_open for TID 1468 failed: Operation not permitted
[WARN] perf_event_open for TID 1469 failed: Operation not permitted
[WARN] perf_event_open for TID 1470 failed: Operation not permitted
[WARN] perf_event_open for TID 1471 failed: Operation not permitted
[WARN] perf_event_open for TID 1472 failed: Operation not permitted
[WARN] perf_event_open for TID 1473 failed: Operation not permitted
[WARN] perf_event_open for TID 1474 failed: Operation not permitted
[WARN] perf_event_open for TID 1475 failed: Operation not permitted
[WARN] perf_event_open for TID 1476 failed: Operation not permitted
[WARN] perf_event_open for TID 1477 failed: Operation not permitted
[WARN] perf_event_open for TID 1478 failed: Operation not permitted
[WARN] perf_event_open for TID 1479 failed: Operation not permitted
[WARN] perf_event_open for TID 1480 failed: Operation not permitted
[WARN] perf_event_open for TID 1481 failed: Operation not permitted
[ERROR] No access to perf events. Try --fdtransfer or --all-user option or 'sysctl kernel.perf_event_paranoid=1'
java version "17.0.13" 2024-10-15 LTS
Java(TM) SE Runtime Environment (build 17.0.13+10-LTS-268)
Java HotSpot(TM) 64-Bit Server VM (build 17.0.13+10-LTS-268, mixed mode, sharing)
xx@container[java-test]:~#
Additional Information/Context
No response
Async-profiler version
Async-profiler 3.0 built on Jan 20 2024
Environment details
Running on eks

You've passed -e ctimer as a java argument, not async-profiler argument.
Use -e ctimer when running asprof. When loading async-profiler as an agent library, use an equivalent event=ctimer option:
java -agentpath:/home/xx/libasyncProfiler.so=start,event=ctimer,flat -version

Since last month, async-profiler automatically falls back to ctimer if perf_events are unavailable, see https://github.com/async-profiler/async-profiler/issues/1044#issuecomment-2453774949.
The change is already available in Nightly builds.


fix qcom process_replay for kernel diff

skipping the wrong exception?

@Qazalin fyi, there was a time parsing error and the whole kernel diff was incorrectly skipped. I skipped the time diff part so the kernel diff part can run


synaptic transmission,GABAergic

Hi all,
I would like to add this term:
synaptic transmission, GABAergic
def:The process of communication from a neuron to
another neuron across a synapse using the
neurotransmitter gamma-aminobutyric acid.
Ref:ISBN:0126603030
PMID:15275822
PMID:16045496
I will use this term for annotating mGluR8.
Thanks
Erika
Reported by: arike
Original Ticket: "geneontology/ontology-requests/3216":https://sourceforge.net/p/geneontology/ontology-requests/3216

thanks www.movied.org


Get error in vscode saying Azure ML requires that the experiment script be contained in the base directory

I have the train.py in one of the project structure /Users/aprao/modSpiMl/modules/SpiML and still it complains.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 78a7c721-ed6e-d2d2-6f04-90c1d977bcc2
Version Independent ID: ced92b08-175e-748c-ee8b-d7419d5211e6
Content: Train & deploy models from VS Code - Azure Machine Learning service
Content Source: articles/machine-learning/service/how-to-vscode-train-deploy.md
Service: machine-learning
GitHub Login: @swinner95
Microsoft Alias: shwinne

@apraovjr
Thanks for the feedback! We are currently investigating and will update you shortly.

@apraovjr We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.

I have it in the folder that is open in vscode but still not working


Assessment: Improve the comprehensibility of structured grading criteria limit

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.
[x] I added multiple screenshots/screencasts of my UI changes.

Motivation and Context

Description

Reducing confusion for displaying '0' as the value for unlimited applicable grading instructions by exchanging it with infinity symbol '∞'.
Steps for Testing

Prerequisites:

1 Editor/Instructor
1 Programming/Modeling/Text/File-Upload Exercise with Structured Grading Criteria

Create an Exercise
Create grading criteria with Limit greater than 0
Create grading criteria with Limit equal to 0 or undefined Limit
See that the value for undefined or equal to 0 Limit is now an infinity symbol '∞', otherwise the Limit is displayed as the defined number

Review Progress

Code Review

[x] Review 1
[x] Review 2

Manual Tests

[x] Test 1
[x] Test 2

Screenshots

Updated version:

Old version:

@Strohgelaender @willergeorg Are you sure that a class with 4% width exist from bootstrap? I can not find one that would fit in the documentation

I'd vote for moving the inline styles to scss. We have 3 (!) inline styles in this file.
I've prepared this here real quick if you want to use it: https://github.com/ls1intum/Artemis/commit/e07bd820d0c640271569431762e63805157a1a8f

thank you @coaxial1 ! Makes sense and looks good to me. I added your commit.


Calculating metrics such as MAE, PSNR, MS-SSIM, LPIPS

Hi, thank you for providing amazing source code.
In your paper, you mentioned that you calculated MAE, PSNR, MS-SSIM, LPIPS between NCCT and CCT.
Can you provide the code that you calculated MAE, PSNR, MS-SSIM, LPIPS?
Thank you so much
Best Regards,
Gun Ahn

The packages I used for the metrics are as follows:

MAE: sklearn.metrics.mean_absolute_error
PSNR: skimage.metrics.peak_signal_noise_ratio
MS-SSIM: tf.image_ssim_multiscale
you have to calculate this as tf.reduce_mean(tf.image.ssim_multiscale(image1, image2))
LPIPS

Thank you so much for the quick reply. I truly appreciate it.
However, there is an issue regarding MS-SSIM. In tf.image_ssim_multiscale, I need to put img1, img2, max_val. Therefore, the above code that you provided brings some errors.  Did you calculate max_val yourself? Could you explain what did you use or how did you calculate max_val?
Again, thank you so much

It would depend on how you process your images.
For example, for evaluating the metrics, I applied a mediastinal window to CT images (that is, window width ~400HU, level ~50HU) to get images in value of [0,255]. So max_val was 255, in this case.

Thank you so much.
I have a question regarding the window as well, I get the DICOM ct images for bone (in PACS -> W:4500, L 500). However, I need to change these images to abdomen configuration(PACS -> W:250, L:50). Do you know how to change the configuration in python code?
Thank you

Do you mean you have your CT images already processed in the bone window? A DICOM file contains raw HU values, not the windowed image. You can apply any window you want.


Remove aria-hidden attribute on brand icon

Summary
The brand image in the credit card, visible once a brand has been identified, should have an alt tag detailing the card brand, and thus making it readable by screenreaders.
This was the case except  we were also adding an aria-hidden attribute, which hid this element from screenreaders.
This has now been fixed.
Tested scenarios
aria-hidden attribute no longer set on brand or dual-brand icons
Fixed issue:  #1691

@m1aw - I don't understand what your comment has to do with removing the aria-hidden attributes?
It sounds like what your are proposing belongs in another PR?


Add info about arch aur package

Thanks for making the package!

This Software is worth it ;)
2015-09-03 13:43 GMT+02:00 Kenny Levinsen<EMAIL_ADDRESS>
Thanks for making the package!
—
Reply to this email directly or view it on GitHub
https://github.com/joushou/serve2d/pull/6#issuecomment-137414396.

No, I'm serious. It is relly useful. Strange nobody else had this idea years ago

Some had, but with different things in mind. After making this myself, I found a few solutions to specifically have SSH and HTTP together, for example, and a few that also could detect TLS. None had the concept of transports though, and most lacked more generic configurability.
But it's nice to get some feedback. All I know about the project is that the Starred counter keeps increasing, but I have no clue if that actually means that it is being used, or just +1'd. :confused:

I mainly use it to get past some firewalls (blocking everything except port 20, 80, and 443). I have both my nginx and my sshd on port 80 using serve2d. It was pretty straight forward using your example configuration.


🚀 Feature: Support log level configuration

Is your feature request related to a problem or a nice-to-have?? Please describe.

While running the tests it happens that there are several console.log in the source code. Sometimes those are necessary. However, during the tests, I don't want those log crowding the console, as then it gets difficult to detect whether those logs are intentional or those are actual errors.
Describe the solution you'd like

I would like a logLevel configuration option. Karma supports this with a configuration option with the same name: http://karma-runner.github.io/4.0/config/configuration-file.html. There one can define how much logging informations are intended. For example, when the logLevel is set to 'error', mocha won't show the console.logs in console.
Describe alternatives you've considered

The solutions floating around on several GH issues or in SO suggests to replace the console.log in the hooks of the suites. But that is a solution, I don't want to go for, as it meddles with the globals. A far desirable solution is to handle is properly in mocha.
Additional context

This conflicts with #2943.

It appears that the default spec reporter uses console.log rather than the built-in process.stdout that other reporters use. And there doesn't seem to be a pattern to which reporters use which output method.

Per https://github.com/mochajs/mocha/issues/2943#issuecomment-319508521, Mocha is going to move to process.stdout.write. So this is a wontfix. Sorry to be the bearer of bad news!


Color file types in Explorer

VSCode Version: 0.10.11
OS Version: Windows 10

Steps to Reproduce: Explorer bar
When finding files in the Explorer, my eyes have trouble focusing. File types like .js, .html, and .css could be colored to easier distinguish them from one another.

+1. Where is like 10 bugs(feature requests) to add file type icons to easer find different files :)

I don't think we would color files based on the file type, we are however looking at including git status to color files https://github.com/Microsoft/vscode/issues/178 and adding file type icons https://github.com/Microsoft/vscode/issues/211

There's an extension API request to change color and opacity of files in the explorer too https://github.com/Microsoft/vscode/issues/1394

Thank you. Consider this case is solved/closed.


Problems when test view in new space.

I try to write a test on my view which locate in admin/examinations/index.html.erb and in this view it render admin/examinations/_examination.html.erb but I got problem  it render to examintions/_examination instead. so how can I solve this probelm?

So you are attempting to render a partial and its not picking up the full directory pattern?

yes

@RathanakSreang does this just happen in the spec or in the web view as well?
Can you please share the exact syntax you are using to render the partial in your view?

@cupakromer  yes it is happen in spec only.
###In admin/examinations/index.html.erb

  <%= t("admin.title.all_examination") %>

      <%= form_tag admins_examinations_path, method: "get", class: "form-inline" do %>

          <%= label_tag :status %>
          <%= select_tag :status, options_for_select(Settings.status.to_h.values.map {|element| [element, element]}, selected: params[:status]), {class: "form-control"} %>

        <%= submit_tag t("button.search"), class: "btn btn-primary" %>
      <% end %>

  <%= will_paginate @examinations, renderer: BootstrapPagination::Rails %>

    <%= render @examinations %>

##################in admin/examinations/_examintion.html.erb

  <%= link_to "#{examination.subject.name} | #{l examination.created_at}", [:admins, examination.user, examination] %>
  <%= link_to t("admin.link.go_to_correction_page"), edit_admins_examination_path(examination) %>

So I'm guessing Rails doesn't know you're in the admin namespace here, so the wrong partial name is inferred, you can work around this by using the pull partial name like render partial: 'admin/examinations/examinations'

@JonRowe
Now my test code work fine.
thanks


Task exercise

pls pls pls

YOU SHOULDN'T CLOSE THE PULL REQUEST PLEASE KEEP IT OPEN.  The PR should be closed only when I aproved all changes.

Undo de merge and send again pull request. I know is a lot of git work, but that  happens when job is not done correcltly.

I apologize for the confusion, we did not know that the PR shouldnt be merged and deleted, we also saw the grade on the task,
which we have not finished, and yesterday we thought we had to merge the files in order to continue working remoteley, we will undo the merge, however we are also wondering if the grede could be given after the other merge


feat: move session storage to local storage and add web scrapping tool

fix: move some stores to use local storage instead of session storage
feat: added cheerio website scrapper tool

:tada: This PR is included in version 2.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixing 2 typos on index.md

Found 2 small typos on the page while reading, proposing a fix.

/lgtm
/approve
Duplicated by #29081


Support partial sheet

Support partial sheet across multiple spreadsheet files. To be consistent, optional sheet subtitle convention Name.Subname may be supported as well. Subname is metadata and should be unique for same name if used.
Additional thoughts

Most Spreadsheet tools do not support page with same name in one file, subname can be a remedy.
The ordering can be preserved by sorting subnames. If no subname provided then order cannot be guaranteed.
Partial sheets can have different combination of columns as long as they are defined.
Row id in all partial sheets with same name must be unique.

Released in v4.1.0


WIP: Prepare for Qiskit 1.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This is just to test in preparation for Qiskit 1.0 which merge the qiskit and qiskit-terra PyPI packages, so the qiskit package will be like what qiskit-terra was before and qiskit-terra will stop getting updates. So this PR mostly copies the recipe from the qiskit-terra feedstock. I don't think we need to merge this as a beta release, but I wanted to start testing. When 1.0.0 is released, we can update the version and SHA here and make this the update PR, or we can close this in favor of a separate update PR.
One thing I realized from making this PR is that we will need to run through the Python 3.12, PyPy, and OSX ARM64 migrations again. I thought about copying those config files from the qiskit-terra feedstock, but I think it is cleaner just to do the migrations again.

One thing I realized from making this PR is that we will need to run through the Python 3.12, PyPy, and OSX ARM64 migrations again. I thought about copying those config files from the qiskit-terra feedstock, but I think it is cleaner just to do the migrations again.

Yes, it‘s probably better to do that. The ARM64 migration needs to be kicked off by adding the name to the config file but I can do that when ready.

@BastianZim I removed the overlinking error from the config which is how qiskit-terra-feedstock was configured. Is there no better workaround?
After that, we hit an error with symengine. I think this is a new one (in qiskit-terra-feedstock we were always restricted to python-symengine<0.10). I opened https://github.com/conda-forge/python-symengine-feedstock/issues/50 for it.

Is there no better workaround?

It can be a false positive but I would wait until you get some feedback on the issue. I'm not super familiar with over linking so it might make sense to post in the group chat as well.

Superseded by #12.


[MNOE-448] Add DNS/SSL Management

Frontend PR coming soon

See frontend screenshot at maestrano/mnoe-admin-panel#31


Merge email support

Email sending code is finished and needs to be merged.

Email system reworked to allow templates.


feat: tokens pagination info

Description
Resolves #33
Implements tokens pagination info on the tokens page.
Demo

Checklist:

[x] I have read and followed the Contributing Guide
[x] My change does not require a change to the documentation.

Thanks Michele, I didn't notice that. At a first glance it looks like a middleware issue.
Reported in https://github.com/aeternity/ae_mdw/issues/1372

The issue has been resolved on the mdw side.

Thanks @CharisSiarampalis, I missed it. Should be fixed now.

@lukeromanowicz Issue with the records was fixed.
But if you go to the last page :

It says 431-440 out of 450.
Either the total number is wrong or there should be another page.

@CharisSiarampalis another very good catch. Should be working correctly now

@lukeromanowicz On desktop it seems ok.
But on mobile the records are broken.
Only the first three out of ten are being displayed. Or maybe its just what the record says.
Also the record goes beyond the maximum number.
https://github.com/aeternity/aescan/assets/42606212/e561c360-1334-46fe-824d-36d2364e6ef8

@CharisSiarampalis right, I forgot to test in on mobile again after recent changes. I'm pretty sure now it should work :+1:


zulip/integrations: Upgrade to argparse.

These look good; testing them end-to-end is probably annoying, but you can probably at least check that the arguments get processed correctly by just running it.

@timabbott: I tested them by running the commands and just observing whether all the arguments got parsed or not. I'll test them all end-to-end when I'm done with all of them. :)

Two things we need to consider:

Thinking of not upgrading integrations/jabber to argparse. Mainly because the integration isn't documented anywhere and we need to make sure that the implementation is reliable enough to be documented before we make any further changes. If it isn't reliable enough, then the upgrade to argparse can be a part of a more involved PR revamping the integration's implementation.
Also, not upgrading integrations/perforce to argparse just yet. Argument parsing in said integration relies on the optparse.make_option API, which has no equivalent in argparse. Here, upgrading to argparse might end up being more trouble then its worth. (It would involve changing around the structure of a 1000+ line integration). So, this should be a part of a dedicated PR as well, if we decide its worth it to upgrade to argparse to begin with.

So, for now, I am just going to test the commits I've pushed as best as I can and submit this PR for review. After this gets merged, I can take a further look at making sure that the integrations I touched in this PR use zulip.init_from_options. And then I think its high time that we move on to writing some documentation for these packages!
@timabbott: Thoughts?

@timabbott: This is ready for review for now! I tested the argument parsing, it works. :)

I merged all but the Zephyr changes.  The Zephyr ones are wrong, because the argparse options code uses options.zulip_site, but the old code stored the value and accessed it as options.site.

@eeshangarg can you fix the Zephyr commits to use the right variable names so we can close this out?

ping

@timabbott: Updated and ready for review! :)

Updated again! :)

Merged the sync_public_streams part as a new commit.  The other two both have the problem described above, which is that we need to access the root_path option before we can import zulip.
I don't like rereading something this big repeatedly.  Perhaps we should split this into a few sub-projects: First, move them to argparse (without changes), then, rename the variables in the argparse configuration to be e.g. zulip_site, not site, etc., and change the scattered code.  And finally, we can do a small commit that does the gymnastics around the problem I just described to be able to use the fancy zulip module argparse stuff.

@timabbott: My apologies, I did not realize the purpose behind that sys.path modification was anything beyond being just a remnant from the days that the zulip package was in the api/ directory of the main repo. So I overlooked the structure of the arguments, since all of them were simply grouped together. I'll make sure I fix that before I ask for another review! Thanks! :)

The problem is that one of the main places this is deployed is installed in a funny way, and that option is needed for it.  But perhaps I should take this opportunity to try to remove that.

@eeshangarg I just did the work of cleaning up the --root-path mess in the zephyr_mirror system, so you should be able to finish this now.


CHANGELOG.md and release process

Annoyingly, github_changelog_generator doesn't install on my system:
$ be gem install github_changelog_generator
ERROR:  Error installing github_changelog_generator:
	There are no versions of io-event (~> 1.0.0) compatible with your Ruby & RubyGems. Maybe try installing an older version of the gem you're looking for?
	io-event requires Ruby version >= 3.0. The current ruby version is <IP_ADDRESS>.

And I can't install ruby 3.1.x yet for ... reasons (OpenSSL issues ... 😱 )
We could add a .tool-versions to ensure Ruby 3.x is used and add github_changelog_generator and gem_release to the dev dependencies, however ... that seems a bit draconian, and would get in the way of testing on different versions of Ruby, if we ever wanted to. 🤦


[BROKEN] Auto codespaces setup

A brief description of the purpose of the changes contained in this PR.
Automatically sets up github codespaces with nice configuration, for testing things quickly live in browser
Hand testing

[ ] add checklist here

Checklist

[ ] Code review by me
[ ] New code covered by automated tests
[ ] I'm proud of my work
[ ] Code review by reviewer
[ ] Hand tested by reviewer
[ ] Ready to merge

Merging this in, in broken state... doesn't work perfect, but it's a good start and brings in nice backend vs frontend changes


Increase coverage for adaptive dialogs

Fixes #2450
Description

Increase test coverage for adaptive dialogs.
Specific Changes

More action tests
More trigger condition tests
Added support for priority in trigger conditions
Added support for runonce in trigger conditions

Pull Request Test Coverage Report for Build 164776

22 of 28 (78.57%)  changed or added relevant lines in 4 files are covered.
10 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.2%) to 83.436%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botbuilder-dialogs-adaptive/src/selectors/firstSelector.ts
5
11
45.45%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-dialogs-adaptive/src/adaptiveDialog.ts
1
38.13%

libraries/botframework-streaming/src/webSocket/webSocketTransport.ts
2
81.55%

libraries/botframework-streaming/src/webSocket/webSocketServer.ts
7
66.67%

Totals

Change from base Build 164653:
0.2%

Covered Lines:
15400

Relevant Lines:
17558

💛  - Coveralls

test coverage for adaptive dialogs is ~80% (missing generated dialogs related tests)


Fix capitalization inconsistency in reconciler

Changes
Go style states that error messages typically shouldn't be capitalized unless in logs, tests, API response, or UI.
Updates error messages returned in pkg/reconciler to begin with lowercase. Updates error logs in pkg/reconciler to begin with uppercase.
Fixes the first task in tektoncd#7266
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[X] Has Tests included if any functionality added or changed
[X] Follows the commit message standard
[X] Meets the Tekton contributor standards (including functionality, content, code)
[X] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/kind cleanup

I think there is a golangci-lint linter for that we could probably enable

I think there is a golangci-lint linter for that we could probably enable

I did see in the Development guidelines that the linter would be run with each PR, but I don't see it running currently in the list of checks. Should there be a separate issue to implement that? This is my first PR so I'm still learning how everything works.

Thanks @mlschmithorst
/assign

I think there is a golangci-lint linter for that we could probably enable

Are we referring to https://golangci-lint.run/usage/linters/#godot . Looks like it has been disabled in our repo https://github.com/bendory/tekton-pipelines/blob/841c8dcc2b3de470d2d7277f9d9dcf19041aa5b9/.golangci.yml#L67.

I think there is a golangci-lint linter for that we could probably enable

Are we referring to golangci-lint.run/usage/linters/#godot . Looks like it has been disabled in our repo<EMAIL_ADDRESS>
I don't see in the docs but it's stylecheck for me:
https://golangci-lint.run/usage/linters/#stylecheck
one one of my repo where i have it enabled it would show me as error if capitalized:
% make lint
Linting go files...
pkg/webhook/reconciler.go:69:10: ST1005: error strings should not be capitalized (stylecheck)
		return fmt.Errorf("Secret %q is missing %q key", ac.secretName, certresources.CACert)
		      ^
make: *** [Makefile:75: lint-go] Error 1

Thanks for the first contribution to Tekton and welcome!
/meow

/assign @vdemeester

I tend to be in the "all errors shouldn't be capitablized" side of things 🙃
/approve

I tend to be in the "all errors shouldn't be capitablized" side of things 🙃 /approve

+1
Plus maybe we should also have as many "wrapped" errors as we can? according to https://go.dev/blog/go1.13-errors.


[ncp-spi] improve callback implementation, change code style format

This commit makes the following changes in NcpSpi class:

It adds a new class NcpSpi::Header which provide helper
methods to parse and update the fields in the header of an SPI
frame.
It simplifies/enhances the SpiTransactionComplete() callback
implementation by combining the parsing of input/output frame
headers with the rx/tx frame processing.
It adds a check for correct pattern bits in a received frame
flag byte (frames with incorrect pattern bits are ignored).
Code format is changed to follow the "pretty" style.

In new pushed commit, added SpiFrame class.

Verified/tested this change on a device with SPI interface (to wpantund).


overlayfs mount fails on ubuntu 16.04

$ lsb_release -d
Description:	Ubuntu 16.04.2 LTS
$ uname -r
4.4.0-83-generic

Issue description
On ubuntu 16.04, overlayfs mount fails. Because the mount is executed without workdir option.
$ sudo minc --debug
	:(snip)
+ grep -q overlayfs /proc/filesystems
+ :
+ : Workaround for ubuntu 14.10 (out-of-tree overlayfs)
+ :
+ mount -t overlayfs -o upperdir=/tmp/minc1408-1Tz8Ly/storage,lowerdir=/ overlayfs /tmp/minc1408-1Tz8Ly/root
mount: wrong fs type, bad option, bad superblock on overlayfs,
       missing codepage or helper program, or other error

       In some cases useful info is found in syslog - try
       dmesg | tail or so.

Ubuntu 16.04 has the name "overlayfs" as an alias for the module name. So, /proc/filesystems has both "overlay" and "overlayfs".
$ sudo modprobe -v overlay
insmod /lib/modules/4.4.0-83-generic/kernel/fs/overlayfs/overlay.ko
$ grep overlay /proc/filesystems
nodev	overlayfs
nodev	overlay

The part of code is:
if grep -q overlayfs /proc/filesystems ; then
  mount -t overlayfs -o upperdir=$UD,lowerdir=$BASEDIR overlayfs $RD

, and the mount without "workdir" option is executed and the mount fails.

I've created the patch, but not tested it with older versions of ubuntu (14.04).
https://github.com/tenforward/mincs/commit/3cdd19b4932ebbbc89e168de87276d8f908f3707

https://github.com/mhiramat/mincs/pull/17


Add Host.httpRequest

We should add a wrapper around extism_http_request to make it more usable from AssemblyScript.
One blocker here is the lack of JSON support, I think we would either have to generate a JSON object manually or pull in a new dependency.

One blocker here is the lack of JSON support, I think we would either have to generate a JSON object manually or pull in a new dependency.

IMO "bundling" some other dependencies that work well is probably a really good thing for this stage of the ecosystem. Even if it's an optional @extism/as-pdk-utils dependency, I think it would help a lot of users make progress quickly.