embassy-sync: documentation and clippy fixes

This PR does not address all clippy lints, the author of the Watch primitive should probably have a closer look on what clippy suggests.

the author of the Watch primitive should probably have a closer look on what clippy suggests

I'll take a look. Does not seem like there is too much though, at least not with default clippy settings.

the author of the Watch primitive should probably have a closer look on what clippy suggests

I'll take a look. Does not seem like there is too much though, at least not with default clippy settings.

It's mainly about using Into instead of the From trait.


wappalyzer removes first line of text on http://asdwdgrwwfd.oseraworld.com/

Platform
Windows 10 x64

WebExtension (Google Chrome)
Website http://asdwdgrwwfd.oseraworld.com/

Describe the bug
bizarrely, Wappalyzer removes the very first line of text on this website: http://asdwdgrwwfd.oseraworld.com/
the first 2 line ACTUALLY reads:
Hi! you've reached the wrong link/domain,
you're looking for 1 of:

but if Wappalyzer is running, the first line is removed by Wappalyzer, so we're left with:
you're looking for 1 of:

. weird, especially given that it's an utterly trivial html website (not even a single javascript is present)
To Reproduce
Steps to reproduce the behavior:

Go to http://asdwdgrwwfd.oseraworld.com/
See error

Expected behavior
the first line should be present.
Additional context
the website is delivered with a HTTP 404 Not Found response code, and when running curl against it i get:
$ curl --dump-header - 'http://asdwdgrwwfd.oseraworld.com/'
HTTP/1.1 404 Not Found
Server: nginx/1.14.0 (Ubuntu)
Date: Sat, 14 Nov 2020 17:36:27 GMT
Content-Type: text/html; charset=UTF-8
Transfer-Encoding: chunked
Connection: keep-alive

Hi! you've reached the wrong link/domain,

you're looking for 1 of:<big>

Osera World

Osera Island

Osera Original

Osera World Bot Edition

Osera Stage 2 RPG Edition

</big>

debug: array (
  'USER' => 'www-data',
  'HOME' => '/var/www',
  'HTTP_ACCEPT' => '*/*',
  'HTTP_USER_AGENT' => 'curl/7.73.0',
  'HTTP_HOST' => 'asdwdgrwwfd.oseraworld.com',
  'REDIRECT_STATUS' => '200',
  'SERVER_NAME' => '_',
  'SERVER_PORT' => '80',
  'SERVER_ADDR' => '<IP_ADDRESS>',
  'REMOTE_PORT' => '53889',
  'REMOTE_ADDR' => '<IP_ADDRESS>',
  'SERVER_SOFTWARE' => 'nginx/1.14.0',
  'GATEWAY_INTERFACE' => 'CGI/1.1',
  'REQUEST_SCHEME' => 'http',
  'SERVER_PROTOCOL' => 'HTTP/1.1',
  'DOCUMENT_ROOT' => '/home/oseraserv/www/default',
  'DOCUMENT_URI' => '/index.php',
  'REQUEST_URI' => '/',
  'SCRIPT_NAME' => '/index.php',
  'CONTENT_LENGTH' => '',
  'CONTENT_TYPE' => '',
  'REQUEST_METHOD' => 'GET',
  'QUERY_STRING' => '',
  'SCRIPT_FILENAME' => '/home/oseraserv/www/default/index.php',
  'PATH_INFO' => '',
  'FCGI_ROLE' => 'RESPONDER',
  'PHP_SELF' => '/index.php',
  'REQUEST_TIME_FLOAT' =><PHONE_NUMBER>.576394,
  'REQUEST_TIME' =><PHONE_NUMBER>,
))

observed on: win10 x64 + Chrome 86.0.4240.193 (Official Build) (64-bit) (cohort: Stable) + Wappalyzer 6.4.16
can not reproduce on: macos catalina + chrome 86.0.4240.183 + Wappalyzer 6.4.16

nevermind, another extension is causing it, sorry


Fix #5, add links and basic descriptions to README

Thanks for your comments! I'll close this PR, as it has conflicts with master. See #8 instead.


[AutoPR cognitiveservices/data-plane/AutoSuggest] Description of Autosuggest usage needs improvement.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4490

(message created by the CI based on PR content)

Codecov Report

Merging #3806 into restapi_auto_cognitiveservices/data-plane/AutoSuggest will decrease coverage by <.01%.
The diff coverage is n/a.

@@                                    Coverage Diff                                    @@
##           restapi_auto_cognitiveservices/data-plane/AutoSuggest    #3806      +/-   ##
=========================================================================================
- Coverage                                                     56%   55.99%   -0.01%
=========================================================================================
  Files                                                       9412     9408       -4
  Lines                                                     188933   188929       -4
=========================================================================================
- Hits                                                      105803   105799       -4
  Misses                                                     83130    83130

Impacted Files
Coverage Δ

...-luis/azure/cognitiveservices/language/__init__.py

...suggest/azure/cognitiveservices/search/__init__.py

...-language-luis/azure/cognitiveservices/__init__.py

azure-mgmt-advisor/azure/mgmt/__init__.py

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 ade0c7d...f2c1f40. Read the comment docs.

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3812


apollo-parser: add description support for SchemaDefinition

close #185
Add support of Description for SchemaDefinition.
Also Add StringValue node in a Description

Done


Discovery: user intilization

As a developer I want to know how to implement user autherntications for future development


different nodes has the same node_key

when i add the second node, it has got a node_key like the first node , and the db has only one record like this
id |                 key                  |    address    |                 name                 | enabled |       created       |       updated
----+--------------------------------------+---------------+--------------------------------------+---------+---------------------+---------------------
  1 | 45564E49-1047-3300-804E-D94FC0303054 | <IP_ADDRESS> | 45564E49-1047-3300-804E-D94FC0303054 | t       | 2016-11-07 09:56:37 | 2016-11-07 17:27:39

and， I found nothing about accessing enroll in the log, here is
Log line format: [IWEF]mmdd hh:mm:ss.uuuuuu threadid file:line] msg
I1107 17:21:23.391466 38787 init.cpp:361] osquery worker initialized [watcher=38784]
I1107 17:21:23.391574 38787 extensions.cpp:364] Could not autoload modules: Failed reading: /etc/osquery/modules.load
I1107 17:21:23.391585 38787 rocksdb.cpp:191] Opening RocksDB handle: /tmp/osquery.db
I1107 17:21:23.391592 38787 interface.cpp:307] Extension manager service starting: /var/osquery/osquery.em
I1107 17:21:23.391598 38787 tls.cpp:170] TLS/HTTPS GET request to URI: https://xxxx:8000/api/v1/osquery/config
I1107 17:21:23.391604 38787 tls.cpp:205] TLS/HTTPS POST request to URI: https://xxxx:8000/api/v1/osquery/config
I1107 17:21:23.391609 38787 options.cpp:61] Verbose logging enabled by config option

osquery version is osquery-2.1.1-1.el6.x86_64, here is kolide and osqueryd configs:
[session]
key = "ahAJDKFH876A*DS&R*SADf187@#$#%^$%"
size = 10
network = "tcp"
address = "<IP_ADDRESS>:6379"
password = ""

[database]
type = "postgres"
address = "<IP_ADDRESS>:5432"
username = "kolide"
password = "kolide"
database = "kolide"
ssl = "disable"
cert = ""
key = ""

[server]
enroll_secret = "kolide"
query_timeout = "20s"
production = false
debug = true
address = ":8000"
crt = "/tmp/kolide.crt"
key = "/tmp/kolide.key"

 #!/usr/bin/env bash

  SECRET="kolide"
  SERVER="xxxx:8000"
  CERT="./tmp/kolide.crt"

  echo $PWD

  sudo -E osqueryd \
    --verbose true \
    --pidfile /tmp/osquery.pid \
    --host_identifier hostname \
    --database_path /tmp/osquery.db \
    --config_plugin tls \
    --config_tls_endpoint /api/v1/osquery/config \
    --config_tls_refresh 10 \
    --config_tls_max_attempts 3 \
    --enroll_tls_endpoint /api/v1/osquery/enroll  \
    --enroll_secret_env ENROLL_SECRET \
    --disable_distributed=false \
    --distributed_plugin tls \
    --distributed_interval 10 \
    --distributed_tls_max_attempts 3 \
    --distributed_tls_read_endpoint /api/v1/osquery/read \
    --distributed_tls_write_endpoint /api/v1/osquery/write \
    --tls_dump true \
    --logger_path /var/log/osquery/ \
    --logger_plugin tls,filesystem \
    --logger_tls_endpoint /api/v1/osquery/log \
    --logger_tls_period 5 \
    --tls_hostname $SERVER \
    --tls_server_certs $CERT \
    --pack_delimiter / \
    --disable_logging=false

@yangqk weird, maybe the database was in some sort of inconsistent state. let us know if the problem pops up again.


TF Arm2TF plugin crashed

When creating the unique id, the arm2tf plugin crashed during demo

seems like hashicorp/random is a better provider to use for this problem atm.

Fixed with changes in Nebraska branch.


use upstream tinygrad

Only jenkins failed due to 38d6ce20726fee16339d0a30afee51759e94148d. @YassineYousfi would it be helpful to add an explicit a pc-thneed build to CI?


chore: Upgrade react-scripts from 3.4.3 to 3.4.4

Upgrade react-scripts in example/package.json from 3.4.3 to 3.4.4:
$ npm update react-scripts

Addresses the following vulnerabilities:

node-forge
object-path

Thanks @mikong 🥇


New app - smclock

Do you think this is ready to merge in? I know there were a few changes recently.

is this ready to merge?

Might as well. Plenty to update, but it will need some work, so... better if there is an initial version up.

Thanks!


Add rule for api identifiers.

For API management we want to introduce API identifiers, that allow to track histories of APIs independent of volatile API titles. For more information see the internal proposal of API identifiers for API histories.

:+1:

👍

👍

:+1:


[BUG] 2nd command line for $BLENDER fails

Describe the bug
You have specified 4 $BLENDER command lines. Good news: the first line works and produces
a blend file that appears to be a terrain with a lot of rectangular boxes.
But the second line generates an error:
I have put "........" where I removed the drive specifics

(infinigen) paul@Pauls-MBP worldgen % $BLENDER -noaudio --background --python generate.py -- --seed 0 --task populate fine_terrain -g desert simple --input_folder outputs/helloworld/coarse --output_folder outputs/helloworld/fine
Blender 3.3.1 (hash ....)
[16:09:15.331] [root] [INFO] | Converted args.seed='0' to scene_seed=0, parsed as hexadecimal
[16:09:15.341] [root] [INFO] | infinigen version 1.0.0
[16:09:15.341] [root] [INFO] | CUDA_VISIBLE_DEVICES=None
[16:09:15.341] [times] [INFO] | [MAIN TOTAL]
[16:09:15.341] [times] [INFO] | [Reading input blendfile]
Read blend: ........ worldgen/outputs/helloworld/coarse/scene.blend
[16:09:15.530] [times] [INFO] | [Reading input blendfile] finished in 0:00:00.189050
[16:09:15.531] [root] [WARNING] | Re-initialized 0 as trusted. Do not run infinigen on untrusted blend files.
[16:09:15.531] [root] [INFO] | Processing frames 1 through 1 inclusive
[16:09:15.538] [times] [INFO] | [Create terrain]
[16:09:15.538] [root] [INFO] | Terrain using only on the fly on_the_fly_asset_folder=PosixPath('....../infinigen/worldgen/outputs/helloworld/fine/assets')
[16:09:15.612] [times] [INFO] | [Create terrain] failed with <class 'AssertionError'>
[16:09:15.612] [times] [INFO] | [MAIN TOTAL] failed with <class 'AssertionError'>
............
Once I get by line # 2, I'll try #3 and #4

Closed due to inactivity, please follow up if you require additional help


feat(slider) migrate Slider-component

Describe pull-request
Migrated Slider component
Solving issue
Fixes: #2416
How to test
Check out the branch and click around.
Note that the eventlistener example will only work in canvas mode

I've added conditional controls, so it only shows the relevant controls. Also added the guards for observer/eventlisteners and a TODO remark for the reset() method (as in the 4.10 component PR)


Third language breaks a lot of things

I added one more language to the Starter. It broke many things at once:

the second page in the Blogs returns 404
Blog posts mix up with pages like "contact" and "about"
some pictures don't load with blog posts while some do
there are multiple warnings during gatsby develop start phase - like "warn You can't use childImageSharp together with en.json — use publicURL instead. The childImageSharp portion of the query in this file will return null: undefined"

Not every error is easily reproducible; a restart of gatsby develop may change the behaviour.

After updating the gatsby remark relative images the images will break.. Currently using 0.3.0 version myself and if you are using netlify cms I noticed that I needed to do something like this with the images for blog thumbnails                             <img                                 style={{                                     display: 'block',                                     width: '100%',                                     height: 'auto',                                 }}                                 src={/${image}}                             />
because the image path from cms is missing the /


error in k2 bounds

bounds of looping over ordering in K2 seach in ch05 are off by 1. first mention of ordering should not be spliced and the next ordering[:k] should be [:k+1]

sorry misunderstood


Building windows VulkanInfo (from VulkanTools repo) generates many warnings

Could be it's already been fixed, but lots of 'size_t to init' warnings.

Went through with a fine toothed comb (and after enabling /W4) to make a PR #301 which should remove most of these warnings.

Just merged the PR, so I am closing the issue.


Update camera.py

Description
This refactor adds type hints to the camera configuration code and improves overall code quality. The goals accomplished include:

Improved code readability and maintainability through type hinting
Enhanced type checking capabilities
Minor optimizations and more Pythonic code structures
Better documentation through improved docstrings and comments

Verification
To test for regressions, I:

Ran all existing unit tests related to camera configuration
Manually tested the main functions with sample data to ensure outputs remained consistent
Used a type checker (e.g., mypy) to verify type correctness
Reviewed the changes with team members to ensure no unintended side effects

The openpilot project definitely likes to see things like type hinting. However, we are pretty fixed on a two-space indent standard and your PR moves the file to four. Beyond the style issue, it makes it more difficult to see what you've truly changed. Our CI test infrastructure also sees a number of issues.
Can you revert the style change? After that, depending on how significant your refactoring was, it might make sense to send your overall type hinting changes first, make sure CI passes, and then follow up with the refactor. Single purpose PRs with clear diffs are much easier to review, and much more likely to be merged.
Will set your PR to Draft for a few days to give you time to respond. Thanks for the contribution!

Closing due to inactivity.


Added an amazing book on scientific python and data science

One of my favorites, the most complete book I could ever find!
P.S.: Do not go by the name of the book :)

@DevPranjal Sorry, this book is owned to O'Reilly. Sharing is illegal.


Loading hangs on Safari 15.3 on macOS Big Sur 11.6.3

With Content Blocker active for the site the entire tab froze and was unresponsive. Turning off content blocker seemed to allow the tab to unfreeze but still failed to load. Got most of the way but never generated your default error message.
Subsequent launch in a new tab just came up blank. Hitting refresh showed a progress bar along the bottom of the address bar but page still remained blank until running a force reload but then it just gets back to a block cursor under the vim examples/c/helloworld.c line and never recovers.
Seems to work OK with Edge Dev 99.0.1150.2 (Official build) Dev (x86_64) on macOS 11.6.3 albeit a bit slow to finish initializing.
Similar hang on Firefox 96.0.3, and I had to turn off enhanced tracking to get to about the same point I got to in Safari.
Seems to work OK on Chrome Version 98.0.4758.80 on macOS 11.6.3. Init was faster than Edge Dev.

I cannot reproduce this behavior on Safari 15.2 / 15.3 / 15.4, nor on IPhone.
I tested with both content blocker active (Safari 15.2 on macOS Monterey) and with content blocker disabled. Firefox also works for me with enhanced tracking and "strict" privacy settings.
Can you try to clear the cache and IndexedDB and try again?

Closing this bug due to lack of precise information. Most likely a side-effect of content blocking.


Update divide.py

One last changed needs review, i hadnt committed tthe change in the z variable to y apparently?!

can i do this?

@Rhiannon77 yes I think you would have been able to merge (had rob not beaten you to it!) - it's kind of an etiquette that each team would need to decide for their project who is going to perform merges. If in doubt, I would recommend just checking, as you did here.


GFM URL regex has false positives

You can see it at http://codemirror.net/mode/gfm/, where it matches the word GitHub and the part of the word res_are_allowed_between_words. There are countless other examples, including the word nice.

Yes, that was recently completely broken. Fixed in 72ca9ff


Stack direction for the condition of do

Why the stack used by the condition of do is reversed?
https://github.com/uiua-lang/uiua/blob/5556c7db256c2c94d2fafa9e4d89e45759ab6259/src/algorithm/loops.rs#L99
⍢(×2)(|2 <3;dump∘) 1 "something (un)related"

┌╴1:13
├╴"something (un)related"
├╴1
├╴1
├╴"something (un)related"
└╴╴╴╴╴
┌╴1:13
├╴"something (un)related"
├╴2
├╴2
├╴"something (un)related"
└╴╴╴╴╴
┌╴1:13
├╴"something (un)related"
├╴4
├╴4
├╴"something (un)related"
└╴╴╴╴╴
"something (un)related"
4

https://www.uiua.org/pad?src=4o2iKMOXMikofDIgPDM7ZHVtcOKImCkgMSAic29tZXRoaW5nIHVucmVsYXRlZCIK

It's a mistake.
Fixed in ce8361eafb6fe4


chore(*): Support both Fabric v2.3 and v2.2 series

This patch makes it possible to support not only Fabric v2.3 series
but also v2.2 series.
To do that, this patch includes:

Update build scripts to enable to create docker images of OpsSC for
the specified Fabric version
Update integration tests to enable to run tests using the specified
Fabric version
Update the related documents

Also, this includes the following minor fixes:

Fix the docker compose network name for test-network with fabric_test
Update npm packages

Especially update the version of fs-extra to avoid the typescript complie error

Remove the hard code parts in /version API of the API server

Signed-off-by: Tatsuya Sato<EMAIL_ADDRESS>

LGTM


Default to (localdb)\MSSQLLocalDB

Part of #189

@divega @bricelam Do you think we should announce this as a breaking change?

@ajcvickers Maybe we should start putting together a What's new page for EF 6.3 in the docs and put it there. But I am fine with announcing it as a breaking changes via other means as well.

Yes, the What's New page sounds good. Let's start with that.


v2: Fixes iOS Alert View with long button text #515

Fixes this issue: https://github.com/driftyco/ionic/issues/5152
Now:

The margin-right: -1px is used to hide the border-right: 1px when the buttons are displayed in a column.
NOTE: I have not tested it, because i don't know how to get it working directly from the ionic repo, BUT i tested it manually in the chrome's debugger... so it should work.

BTW, this DOES NOT fix when the alert has more than two buttons. In iOS more than two buttons are displayed always in one column.  I am working in a JS solution that injects a flex-basis: 100% property.
I don't think it is possible with a pure CSS solution... but i am not sure.
Anyway, I will open a different PR/issue for that.

Last commit fixes the issue in material design as well!
https://www.google.com/design/spec/components/dialogs.html#dialogs-specs

I added 2 more tests:

Long button
Multiple buttons (>2)

an now when there are more than two buttons, they are displayed in a column.


[MM-19815] When saving a subscription, remove fields that are not in the issue metadata

Summary
If a field is configured in a subscription, and is then deleted or removed from an issue type in Jira, the filter will cause no webhook events to match the subscription. This PR makes it so the broken field can be removed from the subscription.
Ticket Link
https://mattermost.atlassian.net/browse/MM-19815
Additional Info
I also got rid of some superfluous snapshot tests that were causing a lot of bloat in the snapshots.

@mickmister Ya, the user won't be able to "undelete" the field or anything, so I think just the error is OK.  I think the bigger question is: After I delete a field in Jira - do certain subscriptions stop working silently?

After I delete a field in Jira - do certain subscriptions stop working silently?

@aaronrothschild Yes, unfortunately. The only way to "fix" the subscription is for the user to go to the form here and submit the form. It will remove the nonexistent fields.
I'm not sure if we want to go the route of continuous housekeeping and sync between the systems. It may be beneficial to have functionality to perform sync, then the admin can manually do so when they need to? cc @levb

After I delete a field in Jira - do certain subscriptions stop working silently?

@aaronrothschild Yes, unfortunately. The only way to "fix" the subscription is for the user to go to the form here and submit the form. It will remove the nonexistent fields.
I'm not sure if we want to go the route of continuous housekeeping and sync between the systems. It may be beneficial to have functionality to perform sync, then the admin can manually do so when they need to? cc @levb

At the very least, do we log this as an error somewhere? (ie: I have a subscription that become non-operational because a field is missing, does the notification fail without an error logged anywhere?)

I think detecting and displaying meaningful errors may be complicated. I would instead opt for collecting and displaying the individual subscription's statistics, including the last time we got a messages, and the number of messages received in the last 30 days? This way the user can see what subs have gone stale and investigate from there.

I would instead opt for collecting and displaying the individual subscription's statistics, including the last time we got a messages, and the number of messages received in the last 30 days?

@levb Sounds good to me, should we ticket this?
FWIW we do have an opportunity to assert that a field is not present in the webhook call. Currently, this is handled the same way as the value being blank.

FWIW we do have an opportunity to assert that a field is not present in the webhook call. Currently, this is handled the same way as the value being blank.

@mickmister can you expand on this?

I think that perhaps what we ought to do is that when one attempts to edit a subscription, and it contains a field that is no longer available, we display a warning and recommend (re-)saving the subscription. A separate ticket/PR? @mickmister @aaronrothschild?

I think that perhaps what we ought to do is that when one attempts to edit a subscription, and it contains a field that is no longer available, we display a warning and recommend (re-)saving the subscription. A separate ticket/PR? @mickmister @aaronrothschild?

@levb Yes, I think that's exactly what this PR does :) The other solutions don't seem to make much technical sense, so let's leave this as the approach.
It may become a problem over time where Admins don't know that channels are no longer receiving notifications because a field was deleted and is silently failing, we can perhaps address in future with stats logging of how many messages get sent to each channel in the past X months, so the admin can notice Subscriptions with "0" activity (you'd suggested this previously in this thread).

@aaronrothschild Friendly ping to review this as it stands


Add Mount Path Parameter to auth_ec2 Method

This PR intends to address https://github.com/ianunruh/hvac/issues/130 along with a few auxiliary fixes. Given that some of the included changes are not strictly required to address the issue, I segmented those changes across commits and am happy to back them out (perhaps to be handled in subsequent PR(s)) upon request.

6d6ec35397ce8fdd12eac4dc5d98202b6f2ddeb6: Before working on this branch, I ran the module's test suite locally with the latest build of Vault (v0.10.1). I encountered a failing test case incidentally related to the AWS EC2 auth backend. The cause of the failure was a backwards incompatible change made in Vault v0.9.6. Is there any guidance on what version of Vault would be run as part of this repository's CI/CD pipeline (assuming one exists)? If not, I would venture its worth keeping the tests current with the latest stable Vault build.
06f3ab18fc44cd5947b1acd4168f0f2f01795bec: First commit directly related to https://github.com/ianunruh/hvac/issues/130. Adds a regression test to capture the failure mode identified in the issue.
99497c476d9bc28f86c4f0eb1ea547bbe9661104: Parameterization of mount_point for the auth_ec2() method.
82b1a9ea76ae19eaff264a0435edfb10cce8134c: Regression test added for an issue found when running these changes in a real environment. I.e., When calling auth_ec2() from a client on an EC2 instance to a Vault cluster with the AWS auth backend configured. Of note: this adds a new test source file for unit testing as having that option was required in this case. The integration tests have no reasonable route for fully mocking out the AWS auth process as far as I can see.
8bb50e19cddc92ec7e427984b2bb741037add45a: Fixes the regression identified in the previous point. I'm not sure how this ever worked as, even at the time the original method was submitted, the upstream auth() method was already unmarshaling the JSON response from Vault.

@ianunruh: I'm interested in seeing this branch merged in to unblock some other code that utilizes this module. Is repository being actively maintained and/or are you the correct maintainer to reach out to with this type of concern?

Ended up going with a better workaround to the param changes to the AWS auth backend. I figure this is good enough to be merged so I'm gonna :shipit:


InferenceService should recover from FailedToLoad

There was a storage issue when creating an InferenceService (through the UI).  This resulted in the status.states.activeModelState: FailedToLoad.  It never reconciled to a good state, even when the storage error was fixed.  The InferenceService didn't retry to download and reconcile.  In order to fix it, I had to update the InferenceService which triggered a reload of the model.
Perhaps it would be useful to periodically try to reload InferenceServices that are in the FailedToLoad state?  Or perhaps the UI could try and refresh/reload?

apiVersion: serving.kserve.io/v1beta1
kind: InferenceService
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: >
      {"apiVersion":"serving.kserve.io/v1beta1","kind":"InferenceService","metadata":{"annotations":{"openshift.io/display-name":"stocks","serving.kserve.io/deploymentMode":"ModelMesh"},"labels":{"name":"stocks","opendatahub.io/dashboard":"true"},"name":"stocks","namespace":"pipelines-tutorial"},"spec":{"predictor":{"model":{"modelFormat":{"name":"onnx","version":"1"},"runtime":"stocks","storage":{"key":"minio-connection","path":"stocks.onnx"}}}}}
    openshift.io/display-name: stocks
    serving.kserve.io/deploymentMode: ModelMesh
  name: stocks
  namespace: pipelines-tutorial
  labels:
    name: stocks
    opendatahub.io/dashboard: 'true'
spec:
  predictor:
    model:
      modelFormat:
        name: onnx
        version: '1'
      runtime: stocks
      storage:
        key: minio-connection
        path: stocks.onnx
status:
  conditions:
    - lastTransitionTime: '2023-09-20T15:15:46Z'
      status: 'False'
      type: PredictorReady
    - lastTransitionTime: '2023-09-20T15:15:46Z'
      status: 'False'
      type: Ready
  modelStatus:
    copies:
      failedCopies: 1
      totalCopies: 1
    lastFailureInfo:
      location: 94c77f-9djbq
      message: "Failed to pull model from storage due to error: unable to list objects in bucket 'models': NoSuchBucket: The specified bucket does not exist\n\tstatus code: 404, request id: 1786A448FC13E392, host id: dd9025bab4ad464b049177c95eb6ebf374d3b3fd1af9251148b658df7ac2e3e8"
      modelRevisionName: stocks__isvc-58f42146d7
      reason: ModelLoadFailed
      time: '2023-09-20T15:15:43Z'
    states:
      activeModelState: FailedToLoad
      targetModelState: ''
    transitionStatus: UpToDate

Although self-healing sounds right, current behavior seems to be on the safe side.
Retries is, perhaps, something that IMO should be turned off by default, but the user should be able to enable if desired (even per ISVC via annotations/fields, if needed).
We don't want to retry indefinitely to the point that the Cloud bill "scales" accordingly.

@cfchase thoughts?

So, a number retries would probably fill the need, with a sane default.  There probably still needs to be a way to trigger a reload through the UI.


ls: reading directory '/mnt/hdd': Input/output error

Hello,
I realized today I had lost access to my raspiblitz box, it was unreachable. The first time it happened since I installed it about 3 years ago (in a raspberry pi 4, 4GB). I just ssh into it an hour ago and I could see there was a problem, and decided to reboot. When rebooting, I got the error message "ls: reading directory '/mnt/hdd': Input/output error..." (see below for details on the rest) and now I cannot even ssh into it anymore. Any ideas? Have the memory car or ssd disk died? I had just updated to verson 1.8.0c from 1.8.0 a couple of weeks ago. I am currently not in the same location as the raspiblitz box, so there isn't much I can do to troubleshoot without ssh access.

Green activity light stays dark and LCD turns white when shutdown complete.
It will then reboot again automatically.
stop electrs - please wait ..
stop lightning - please wait ..
stop bitcoind (1) - please wait ..
stop bitcoind (2) - please wait ..
ls: reading directory '/mnt/hdd': Input/output error
(standard_in) 1: syntax error
/home/admin/config.scripts/blitz.datadrive.sh: line 438: * 1024: syntax error: operand expected (error token is "* 1024")
starting shutdown ...
Connection to <IP_ADDRESS> closed by remote host.
Connection to <IP_ADDRESS> closed.

Thank you.

This suggests a harrware failure. It cannot read the disk.
A restart might help, but be mindful that it could be caused by a power spike, physical displacement, faulty disk or adapter.


Added the onRowDoubleClick event callback.

This takes care of one of the missing callback events :smile:

@sww Thanks for the pull request!


NumRequiredBosses Tooltip

The existing tooltip leads players to believe that there is either nothing in these locations at all, or that the items present have no value.  However, numerous other worlds either misuse the filler tag, or dump helpful but not critical items into it.  As a result, many players will claim they have nothing to do in their world when some of these locations will stay have meaningful value to multiworld participants.  This change clarifies the filler class and emphasizes the lack of progression, to at least minimize the number of people who might believe the locations simply do not exist.

Addressed in v.2.6.0 release. If you still find the tooltip misleading, please don't hesitate to open another PR.

I feel the new release addresses the concern wholesale.  It was less a matter of my own confusion but a common stumbling block that my friends ran into for this world.  The 2.6.0 changes prevent that problem outright.  Congrats on the release!  Looks great all around!


Consume new C# completion APIs to avoid loading Razor binaries in non-Razor scenarios

https://github.com/dotnet/roslyn/issues/18650 is the issue this is blocked on FYI @mkArtakMSFT

Double marked this issue for cost because it's still a slight unknown

Let's go with most optimistic cost, rather than adding two, as that will mess up my calculations

aec88e3eba339f91f75a4d1aa68ff7736aee34ee


Element Type should be able to be either float or double

The default for element_type is double. It should be able to be a float as well. That means the xmmt library may need different using definitions create accerated values, plus the raytrace::color may need some work.

Resolved. raytrace::color has all it's simd types removed as the intel xmmt types require an alignment which is not possible on the stack.


added Debian data and automation script

Signed-off-by: rachejazz<EMAIL_ADDRESS>Overview of what package_build does:
Prerequisites:
pip install requests
For Usage:
╰──> λ ./package_build
Usage:
./package_build <exact_file_name.json>
			[if data is from PDS]
./package_build debian
			[if data is from Debian]
./package_build
			[for displaying this help]
Example:
./package_build RHEL_8_Package_List.json
./package_build debian

For extracting from PDS:
┬─[divya at racharch in ~/a/o/s/distro_data]─[G:newdata]
╰──> λ ./package_build RHEL_8_Package_List.json
Extracting RHEL_8_Package_List.json from PDS data ...
Thanks for using SDT!
┬─[divya at racharch in ~/a/o/s/distro_data]─[G:newdata]
╰──> λ ./package_build SUSE_Package_Hub_SLES_12_SP2.json
Extracting SUSE_Package_Hub_SLES_12_SP2.json from PDS data ...
Thanks for using SDT!

For extracting Debian data:
╰──> λ ./package_build debian
Extracting debian data ...
Thanks for using SDT!

Incase file does not exist:
╰──> λ ./package_build doesnotexist
Usage:
./package_build <exact_file_name.json>
			[if data is from PDS]
./package_build debian
			[if data is from Debian]
./package_build
			[for displaying this help]
Example:
./package_build RHEL_8_Package_List.json
./package_build debian
┬─[divya at racharch in ~/a/o/s/distro_data]─[G:newdata]
╰──> λ ./package_build doesnotexist.json
Extracting doesnotexist.json from PDS data ...
Couldn't pull. Error:  404 File not found
Thanks for using SDT!

Proof of files getting stored:
┬─[divya at racharch in ~/a/o/s/distro_data]─[G:newdata]
╰──> λ ls
Debian_Buster_List.json   SUSE_Package_Hub_SLES_12_SP2.json
package_build             ZOS_Software_List.json
RHEL_8_Package_List.json

I'll add the script instructions again on openmainframeproject/software-discovery-tool#38

Sorry about this, but could you move the script to the software-discovery-tool, instead of the -data repository? We want the data repository to be ONLY data, and keep it as an optional component of the system. Some users may want to provide their own JSON files and not use ours at all, but they may want this script to grab other sources.

Yess totally makes sense. I'll update it once I wake up :P


Provision mode for QoD

Problem description
QoD Service provides the customer with the ability to set ceratin profile of QoS to an access network connection. Currently, the API supoports a session mode:

the developer wants to set the required QoD profile for a certain period of time, after which the be network configuration must be set back to the default one.

But there is another possible use case for QoD, which is not currently supported:

the developer wants to set the required QoD profile indefinitely, this is, each time that the UE connects to the network, it will use the required QoD profile instead of the default one, until the association is removed.

Possible evolution
Add support for a new "provision" mode, complementary to the current "session mode", with equivalent operations under a new path, to model:

Creating a QoD provision for a device
Removing a QoD provision for a device
Getting the QoD provision details for a device
Updating QoD provision details for a device

Alternative solution
Where should discuss where to model these new operations. Probably the best option is in a separate API, under the same QoD API family. Most schemas can be reused.
Additional context
This new model is more convenient for those B2B use cases where the devices being connected are used for a single purpose. For example reporters on the move, using backpacks to cover events and perform live video connections, that need the networking conditions provided by the QoD profile each time the backpack is used.

Thanks @jlurien for this issue. We might need another label for this, as it is more a proposal for a new API than the enhancement of an existing one. But that's about definition of the "enhancement" label and it's more about the size of "enhancement".
Said this, looking on the proposal I have the impression that it is maybe rather a product order API (e.g. for a tariff option or to make a SIM card eligible to use a certain slice) than a network API. For the use case of live video production it might in addition be necessary to book/reserve the needed resources in advanced, as proposed in https://github.com/camaraproject/NetworkSliceBooking.

It's an interesting use case: one major consideration is the impact on reconciliation and settlement. If a network operator decides to charge for QoD the API consumer will want to make sure they have received the agreed service, and will challenge the operator  if they believe otherwise.  I suspect this involves a different level of complexity for an 'always on' QoD than for a session-based QoD.

It's an interesting use case: one major consideration is the impact on reconciliation and settlement. If a network operator decides to charge for QoD the API consumer will want to make sure they have received the agreed service, and will challenge the operator if they believe otherwise. I suspect this involves a different level of complexity for an 'always on' QoD than for a session-based QoD.

fyi @jgarciahospital

@jgarciahospital : I have some questions for clarification on the new "Provisioning Mode"
It is not completely clear, whether this new provisioning mode changes
(A). Changes the default QoS of a PDU Session, or
(B). Installs a QOD Sessions, which is activate as default.
(A) means, that the applicationServer and the applicationServerPorts properties are not present or ignored and the requested QoD Profile is applied to any traffic for that device (PDU Session)
(B) means, that a QOD Session with the applicationServer and the applicationServerPorts properties  are always present, without any reactivation or any extension of the session duration.
Further, what about timing aspects. According to the current PR, the provisioning changes is applied instantaneously (now) and is valid forever (infinite session duration). What about
(C) adding an end-time, so that the provisioning is automatically removed after some time (e.g. by Friday)?
(D) adding a start-time, so that the provisioning is not applied instantaneously (now), but by a specific timestamp?
When adding some time information, it seem to become similar with the Network Slice Booking API (link).
Can you please elaborate?

Reopen as #294 only updated the README, but obviously the enhancement isn't is yet done.

@jgarciahospital : I have some questions for clarification on the new "Provisioning Mode"
It is not completely clear, whether this new provisioning mode changes (A). Changes the default QoS of a PDU Session, or (B). Installs a QOD Sessions, which is activate as default.

Proposal is to set a configuration for the device/flow which is activated by default each time the device is connected to the network or for certain flow

(A) means, that the applicationServer and the applicationServerPorts properties are not present or ignored and the requested QoD Profile is applied to any traffic for that device (PDU Session) (B) means, that a QOD Session with the applicationServer and the applicationServerPorts properties are always present, without any reactivation or any extension of the session duration.

Application server and ports are treated as as we have right now, If not specified, all destinations are assumed (as <IP_ADDRESS>/0)

Further, what about timing aspects. According to the current PR, the provisioning changes is applied instantaneously (now) and is valid forever (infinite session duration). What about (C) adding an end-time, so that the provisioning is automatically removed after some time (e.g. by Friday)? (D) adding a start-time, so that the provisioning is not applied instantaneously (now), but by a specific timestamp?

End-time is not in the initial proposal, as the configuration can be enabled/disabled on demand, and it is initially intended for long periods of time, to make this use case more differential compared to the current one, for sessions.
Booking of provisions, or sessions, could make sense as a future enhancement.

When adding some time information, it seem to become similar with the Network Slice Booking API (link).

There are many similarities between QoD and Network Slicing and in some situations their scope overlap. To me QoD is more service oriented while Network Slicing is more technology oriented. Also Network slicing treats with a location dimension that is currently missing from QoD.

Can you please elaborate?

@jlurien : Thank you for raising this issue.

Further, what about timing aspects. According to the current PR, the provisioning changes is applied instantaneously (now) and is valid forever (infinite session duration). What about (C) adding an end-time, so that the provisioning is automatically removed after some time (e.g. by Friday)? (D) adding a start-time, so that the provisioning is not applied instantaneously (now), but by a specific timestamp?

End-time is not in the initial proposal, as the configuration can be enabled/disabled on demand, and it is initially intended for long periods of time, to make this use case more differential compared to the current one, for sessions.

Booking of provisions, or sessions, could make sense as a future enhancement.

When adding some time information, it seem to become similar with the Network Slice Booking API (link).

There are many similarities between QoD and Network Slicing and in some situations their scope overlap. To me QoD is more service oriented while Network Slicing is more technology oriented. Also Network slicing treats with a location dimension that is currently missing from QoD.

Just a comment.
I have same thinking that booking of provisions, or sessions, make sense as one of enhancements for "Provision Mode".
I believe there is a case that QoD session is booked on the top of Network Slice Booking which can reserve resources.

@Masa8106 I would suggest to open a dedicated issue to add support for that (booking of sessions or provisioning in advance), as we can then discuss it and take it into consideration as part of the backlog

@jlurien  Thank you for your suggestion. I will open a dedicated issue later.


Error in inferencing using dit object detection models

I have finetune icdar19 model checkpoint  on 2 custom classes. While inferencing I am getting following error-
Traceback (most recent call last):
"inference.py", line 79, in <module>
 main()
 File "inference.py", line 46, in main
cfg.merge_from_file(args.config_file)
File "/usr/local/lib/python3.7/dist-packages/detectron2/config/config.py", line 69, in merge_from_file
self.merge_from_other_cfg(loaded_cfg)
File "/usr/local/lib/python3.7/dist-packages/fvcore/common/config.py", line 132, in merge_from_other_cfg
return super().merge_from_other_cfg(cfg_other)
File "/usr/local/lib/python3.7/dist-packages/yacs/config.py", line 217, in merge_from_other_cfg
_merge_a_into_b(cfg_other, self, self, [])
File "/usr/local/lib/python3.7/dist-packages/yacs/config.py", line 478, in _merge_a_into_b
_merge_a_into_b(v, b[k], root, key_list + [k])
File "/usr/local/lib/python3.7/dist-packages/yacs/config.py", line 478, in _merge_a_into_b
_merge_a_into_b(v, b[k], root, key_list + [k])
File "/usr/local/lib/python3.7/dist-packages/yacs/config.py", line 474, in _merge_a_into_b
v = _check_and_coerce_cfg_value_type(v, b[k], k, full_key)
File "/usr/local/lib/python3.7/dist-packages/yacs/config.py", line 537, in _check_and_coerce_cfg_value_type
original_type, replacement_type, original, replacement, full_key
ValueError: Type mismatch (<class 'str'> vs. <class 'dict'>) with values ({} vs. {}) for config key: MODEL.VIT.MODEL_KWARGS

python inference.py --image_path /path/to/image --output_file_name  result.jpg --config-file /path/to/config.yaml
I used config file generated during model training.

Hi @Atul997, I want to make sure that inference on our released model is OK, but inference on the model which you fine-tuned would occur this error?

Yes the inference on released model is OK. I changed the config file and it worked.
@Dod-o  Thanks for the reply.

@Atul997 how did you modify config file?

Hi @fidhaph,
I don't know if this will help, but I solved it like this.
I just deleted "MODEL_KWARGS: '{}'" from the config file.


Rust 1.77 will clash harder with GNU binutils on Darwin Nix

Describe the bug
cargo 1.77 started stripping debuginfo by default for cargo build --release in order to make Rust binaries smaller by default, but on macOS, if somehow the "wrong" (GNU binutils ~2.41, I believe?) strip gets into the PATH, the strip call will result in a segfaulting binary when executed. nixpkgs does try to work around this already, and things work fine when the "right" strip is the path.
based on bug reports from macOS users, this is often not enough for many actually-extant user configurations. macOS users that use Nix are running into this anyways, due to their PATH rearranging (possibly after entering a nix shell? flake? no idea...). it is possible some specific Nix-based tooling is causing the problem, as a result, and not nixpkgs per se, but we have not gotten enough information from reports to narrow this down... it's simultaneously pervasive and elusive, despite the known reasons.
I decided to file this bug, even though it is incredibly vague, because you may wish to figure out a position on this issue before you land an upgrade to the Rust 1.77 toolchain for nixpkgs, and because people seem to prefer opening the issue against rust-lang as it regresses on the latest version of the toolchain, despite being a preexisting issue (i.e. the "wrong strip" could already be a problem if someone enabled it in their Cargo.toml's build settings, or used it "by hand"). please feel free to close it whenever.
Steps To Reproduce
steps to reproduce the behavior:

put the GNU strip first in the PATH while on macOS
set up the rust 1.77 toolchain
cargo run --release

technically this also happens with compiling a C program and stripping it "manually", etc.
Expected behavior
printing "Hello world!", probably.
Screenshots
nah.
Additional context
apparently homebrew also sometimes causes this problem.
Notify maintainers
idk who should be bothered over this.
Metadata
not applicable.
Add a :+1: reaction to issues you find important.

Thanks for the issue -- if anyone is interested I'm happy to help on the nix side.
Here's the issue I started at rust-lang: https://github.com/rust-lang/rust/issues/123114
TBF the reason I started the issue at rust-lang is because cargo is calling strip as an external binary by PATH lookup without any other mechanisms for specifying the absolute path to the proper binary. This means that if I happened to make a personal tool named strip that were early on my PATH, it would break things as well, without an obvious remedy other than "don't put that early on your PATH". It would be nice for cargo to provide an escape hatch here!
But yes, I agree that it seems that nix's strip breaks rust binaries, which is a problem!

cc @toonn as we were recently discussing a different strip issue

someone putting novel utilities in their path that have the same name as an ancient piece of Unix tooling that is sufficiently commonplace as to have a POSIX spec since at least 15 years ago seems like an invitation for trouble in general, akin to making a tool for closed captioning and naming it cc. nonetheless perhaps we should be referencing some env var, much like we, if memory serve, respect $CC. I think the only reason we don't already is that there's no equally-hoary common standard for an env var that names "path to the strip binary that should be used" afaik, so we'd be making something up.

To clarify, is it only strip from GNU binutils or also the one from darwin.cctools-port and llvmPackages.bintools? Those are all different implementations. Darwin currently uses llvm-strip by default.

nixpkgs does try to work around this already, and things work fine when the "right" strip is the path.

I wonder if darwin.binutils should symlink everything from LLVM and not bother with GNU binutils or cctools (except for ld64 for now). At some point, Darwin will go useLLVM by default. (I’d like to try it with the LLVM 18 update, but I’m skeptical it’s doable this year.)

@workingjubilee That's totally fair, though your argument sounds a lot like "users ignorant enough to poorly name a personal script should know better." Also, the inscrutable failure modes (just a (signal: 9, SIGKILL: kill)) make it pretty tough for a lowly hobbyist like myself to figure out what's going on or what to do about it. It seems that there are a few ways that rust could make this much easier -- shelling out to an external binary without any knobs to turn makes it tough for me to decide how I should work around this issue.
@reckenrode does this answer your question?
$ readlink -f $(type -p strip)
/nix/store/j0w1bxkg0lyw39dwsh3zpqzac2l05dkp-binutils-wrapper-2.40/bin/strip

$ cat run.c
#include <stdio.h>

int main() {
    printf("hello there\n");
    return 0;
}
$ clang run.c
$ ./a.out
hello there
$ /nix/store/j0w1bxkg0lyw39dwsh3zpqzac2l05dkp-binutils-wrapper-2.40/bin/strip ./a.out
$ ./a.out
Killed: 9
$ sw_vers
ProductName:            macOS
ProductVersion:         14.3.1
BuildVersion:           23D60

@n8henrie I don't want to digress too far since this is about how to solve this issue, but just to explain, my position isn't "users should know better" and more "in the absence of an explicit alternative, $PATH is the directive where we should be looking for the strip tool"... I am afraid that if we made something up but got it wrong and misjudged what we should be doing, as we have before, we'd have made it virtually impossible for you to debug rather than merely pretty tough.

@reckenrode does this answer your question?
$ readlink -f $(type -p strip)
/nix/store/j0w1bxkg0lyw39dwsh3zpqzac2l05dkp-binutils-wrapper-2.40/bin/strip

How did that get in the path? The strip in the Darwin stdenv is llvm-strip. The only way to get strip from GNU binutils is by adding the top-level binutils as a native build input, to environment.systemPackages in nix-darwin, nix-env -i, etc.
I agree this is an issue that should be solved because users could have some particular strip on their path, and it would be bad if that resulted in a broken binary. However, I’d also like to understand how GNU binutils is being used in nixpkgs on Darwin. There’s not really a good reason to do it, especially if strip still causes problems with Mach-O binaries.
(If Rust can’t be made to invoke an absolute-pathed strip, can rustc be wrapped with a known-working one first on its path?)

I believe Cargo is the one that demands that we strip but rustc is the actually caller of strip, and there's already a rustc-wrapper in tree:
https://github.com/NixOS/nixpkgs/blob/5919d4081e2ef603b43b38bb8d923ecb24bbb78c/pkgs/build-support/rust/rustc-wrapper/rustc-wrapper.sh#L4

@reckenrode

How did that get in the path? ... The only way to get strip from GNU binutils is by adding the top-level binutils as a native build input

Correct, because I have binutils in my environment.systemPackages. I'm a relatively recent (1-2 years?) convert from homebrew*, and one of my first steps to ease the transition was just putting into systemPackages most anything I had installed system-wide with homebrew. I use tools like nm, objdump, and strings from time to time, and it's inconvenient to have to enter a nix shell for a quick one-off command (startup time, etc.), so I included binutils in the list of "things I always want installed." I assumed that using the nix copy would help me keep behavior consistent between MacOS and Linux, as it's a right pain to have to remember the flags that MacOS does / doesn't have.
* Of note, I previously ran into the same issue with rust / strip / SIGKILL on homebrew

I’d also like to understand how GNU binutils is being used in nixpkgs on Darwin. There’s not really a good reason to use it, especially if strip still causes problems with Mach-O binaries.

As noted, I'm using other tools in the package, not strip specifically.

If Rust can’t be made to invoke an absolute-pathed strip

My suggestion at https://github.com/rust-lang/rust/issues/123114 was simply to prefix PATH with /usr/bin: (on MacOS specifically) (likely here). As strip is the only binary that has given me a problem multiple times over multiple years across multiple system setups (and AFAIK the only tool that's being shelled out like this), it seems like this would be a simple (if inelegant) solution.

@n8henrie Thanks for the explanation. If flag-compatibility between platforms is important to you, then using darwin.cctools instead wouldn’t be a solution (since while it is closer to what the platform ships, the flags are different).
My concern regarding nixpkgs was if anything packages were GNU binutils on Darwin. Nothing should be, especially if Mach-O support hasn’t gotten much better — that linked workaround was committed almost a decade ago.
As an aside, I’m probably going to open a PR to rework darwin.binutils to symlink everything from llvmPackages.bintools. The only GNU binutils program it links is c++filt, which LLVM provides (and the LLVM version is what Apple ships anyway). That should cut down on build time since llvmPackages.bintools has to be built anyway.

My suggestion at https://github.com/rust-lang/rust/issues/123114 was simply to prefix PATH with /usr/bin: (on MacOS specifically) (likely here). As strip is the only binary that has given me a problem multiple times over multiple years across multiple system setups (and AFAIK the only tool that's being shelled out like this), it seems like this would be a simple (if inelegant) solution.

Forcing the path in rustc would be a problem for nixpkgs because it would require patching Rust to use a path from nixpkgs instead. While the in-tree derivation could do that, there are popular overlays that provide upstream’s binaries. Those couldn’t be patched. They would try to invoke strip at /usr/bin/strip, which would fail if Xcode or the command-line tools are not installed or if Rust is being used in a sandboxed build. Adding it to the wrapper, which appears to have been a recent addition, would avoid patching and work with the overlays (provide they are updated to use the wrapper).

How does rustc determine which strip to use when cross compiling?

It seems we benignly assumed people would build strip with --enable-targets=all.

I appreciate the explanation.

it would require patching Rust to use a path from nixpkgs instead

Just to clarify -- do you mean "require" for purity? Or for functionality? The specific function I'm referring to is only for calling strip (not any other external tool), which is based on a runtime PATH lookup already. It seems to me that both purity and functionality are somewhat compromised in the current situation:
$ type -a cargo
cargo is /run/current-system/sw/bin/cargo
$ cargo +nightly --version
cargo 1.78.0-nightly (2fe739fcf 2024-03-15)
$ cargo +nightly install -f cargo-espflash
    Updating crates.io index
  Installing cargo-espflash v3.0.0
    Updating crates.io index
   Compiling proc-macro2 v1.0.79
   Compiling unicode-ident v1.0.12
   Compiling libc v0.2.153
   Compiling cfg-if v1.0.0
   Compiling serde v1.0.197
   Compiling pkg-config v0.3.30
   Compiling vcpkg v0.2.15
   Compiling memchr v2.7.2
   Compiling thiserror v1.0.58
error: failed to run custom build command for `proc-macro2 v1.0.79`

Caused by:
  process didn't exit successfully: `/var/folders/kb/tw_lp_xd2_bbv0hqk4m0bvt80000gn/T/cargo-installYvXyxp/release/build/proc-macro2-a08fba1383bb57dd/build-script-build` (signal: 9, SIGKILL: kill)
warning: build failed, waiting for other jobs to finish...
error: failed to run custom build command for `serde v1.0.197`

Caused by:
  process didn't exit successfully: `/var/folders/kb/tw_lp_xd2_bbv0hqk4m0bvt80000gn/T/cargo-installYvXyxp/release/build/serde-27272797f677f9c1/build-script-build` (signal: 9, SIGKILL: kill)
error: failed to compile `cargo-espflash v3.0.0`, intermediate artifacts can be found at `/var/folders/kb/tw_lp_xd2_bbv0hqk4m0bvt80000gn/T/cargo-installYvXyxp`.
To reuse those artifacts with a future compilation, set the environment variable `CARGO_TARGET_DIR` to that path.

Yes, I know cargo-espflash is packaged in nipxkgs, but it's an old version, and I just don't always have the time or energy to repackage something for a one-off command. As a workaround I can PATH=/usr/bin:$PATH cargo +nightly install ....
$ nix eval nixpkgs#cargo-espflash.version
"2.1.0"

Perhaps I had originally installed GNU binutils for some microcontroller work? Trying to remember. I guess the obvious solution is just to not have GNU binutils installed system-wide, since nix is much better at this than homebrew it shouldn't be a problem.

Just to clarify -- do you mean "require" for purity? Or for functionality? The specific function I'm referring to is only for calling strip (not any other external tool), which is based on a runtime PATH lookup already.

Purity is functionality if someone is trying to run rustc in a container. It is very common for rustc to be invoked in a situation where it does not have privileges sufficient to access /usr/bin, but rather is given a PATH that includes a directory that has the specific set of things it should have access to. This is a very sensible move because building Rust code, like most build systems, involves arbitary code execution, both in the forms of build.rs and proc-macros.

It seems we benignly assumed people would build strip with --enable-targets=all.

This is not the case in Nixpkgs, because it makes the binutils package huge.  So stripping by default sounds like it's going to break in the cross case without some better configuration mechanism, even if there is a strip in PATH?

...how huge, exactly?

It's an extra ~60 MiB.  (Default binutils it's 84.8 MiB, and then the wrapper adds 18.1 MiB.)  The comment says "WARN: Enabling all targets increases output size to a multiple.", which makes me think the difference used to be bigger.

Purity is functionality

Sorry, I was trying to clarify in what sense @reckenrode was using "require" (break the build vs not best practice vs other) as the current situation with nix on darwin still does require reaching out to some extra-nix paths at times (libsystem, https://github.com/oxalica/rust-overlay/pull/149 -- someday I'll perhaps understand 10% of @reckenrode's work on this!).
Currently, rust is reaching out to "whatever strip is first on PATH. I had thought that having it look for strip (not affecting anything else) at "/usr/bin/strip if (cfg(target_os = "macos") and this path exists), otherwise fall back to the current behavior" seemed like an easy solution that would resolve the issue for myself and at least a few other MacOS + nix / homebrew users.
I suppose this would potentially break cross compilation through nix though (presuming rust goes in nativeBuildInputs and therefore hostSystem == aarch64-darwin and targetSystem == ...-linux -- is that right?).
And regardless, I am starting to get the feeling that people with much more knowledge and experience think my suggestion is suboptimal, so I'll lay off 😆 Thanks for your patience!

Purity is functionality

Sorry, I was trying to clarify in what sense @reckenrode was using "require" (break the build vs not best practice vs other) as the current situation with nix on darwin still does require reaching out to some extra-nix paths at times (libsystem, oxalica/rust-overlay#149 -- someday I'll perhaps understand 10% of @reckenrode's work on this!).

By “require”, I mean the package function has binutils as an argument, which callPackage fills with pkgs.binutils (i.e., GNU binutils). Darwin is supported by GNU binutils (and mostly works), so there is no reason to mark it broken on that pkatform, but I would consider using it in nixpkgs an anti-pattern if not a mistake.
The situation in that PR is because they’re using binaries built outside of nixpkgs that link to system libraries, which expect to access other system resources and configuration files. That’s going to require sandbox exceptions or other adjustments occasionally.

Just to clarify -- do you mean "require" for purity? Or for functionality? The specific function I'm referring to is only for calling strip (not any other external tool), which is based on a runtime PATH lookup already.

A hardcoded reference to /usr/bin would introduce an impurity. Instead of picking up the strip from the stdenv or nix-shell, it would always try to use the system one first. It would also result in different behavior depending on whether a build is done with sandboxing enabled.
Typically, those kinds of references are patched out in when the derivation builds the package. That’s easy for source builds, but it’s not really possible for binary overlays. It would be better for nixpkgs if both could use a wrapper than ensured a compatible strip was always available first in the PATH.

@alyssais Hmm. By default, binutils supports some very esoteric targets. What about if you enable binutils support for "just" all architectures that Nix and rustc support, by default? Or optimize for size?

@vcunat what do you think about the binutils all targets size?  I think from looking at the commit logs it was you that set it up not to build all of them.  Is the current size saving worth it?

I don't recall anything around that.  binutils probably shouldn't be in normal closures, but it will be wherever building happens, which is most NixOS machines I expect.  Probably don't mind me in this 🤷🏽

llvm-strip (which is just llvm-objcopy, but often strip is objcopy, so that's nothing new) should serve most of our needs and afaict costs less than a megabyte of additional weight on the system. 175kb it seems, perhaps less. If Nix would prefer, we could potentially just ship that with our toolchain, so that we take ownership of making sure it works with Rust binaries.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/darwin-updates-news/42249/10

This might be fixed by https://github.com/NixOS/nixpkgs/pull/314920 (or at least to a certain degree) that sets STRIP environment variable to an absolute path.

It looks like upstream has gone with the impure "hardcoded path" approach: https://github.com/rust-lang/rust/pull/130781

It’s not ideal, but it can be replaced with a reliable strip command on Darwin. That can be either strip from LLVM or from cctools.


📃: Adding Relaxio Chrome extension project using various APIs

🔴 Title : Relaxio - Stress Buster
🔴 Tech stack : Manifest, HTML, CSS, JavaScript, APIs, etc.
🔴 Objective : a handy extension for stress relief when coding.
🔴 Summary :
I have created a Chrome extension which works on all Chromium-based browsers for stress relief when coding. This Chrome extension works using the Manifest feature of chromium browsers, Html, CSS and JavaScript, and using various APIs (Quotable for quotes, TheCatApi for cat images, Music from FreeSoundLibrary).
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name : Winter262005 (Shubham Chaudhari)
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.):
GSSOC'24 Extd. Hacktoberfest'24

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

Hey!
I have this project up and running. Just assign this project to me under gssoc24 extended and hacktoberfest, and i'll link a PR to this issue on the same day.


[no-emphasis-as-heading] Don’t use emphasis to introduce a section, use a heading

Codacy detected an issue:
Message: [no-emphasis-as-heading] Don’t use emphasis to introduce a section, use a heading
Currently on:

Commit:<PHONE_NUMBER>1ee8ac1542f2977bd92bd14e71d69d
File:    doc/integrations/dvc/README.md
LineNum: 13

Issue resolved on main branch


Add LOITER state to FAILSAFE RTL

When RTL is initiated via FAILSAFE, it should first go to LOITER for a programmable time before starting the RTL sequence.  A negative value in the parameter would indicate to never RTL and just LOITER indefinitely on FAILSAFE.

We would appreciate this enhancement as well, and most likely it will be a requirement to get a permission trough FOCA (GALLO) in Switzerland.

@julianoes I think you highlighted the wrong nick just now.


Declarative configuration for authentication between ArgoCD and ArgoCD Image Updater

We would like to deploy the ArgoCD Image Updater - but in our environments, absolutely everything is declaratively deployed through kubernetes resources (Generally with Helm charts). I cannot see any way to declaratively create an authentication token for the image-updater in our ArgoCD system.
I really think that along the lines of #138, I consider the argocd-image-updater simply an additional component to the ArgoCD system.. so it should be able to be deployed along side ArgoCD and just leverage the various secrets and credentials that the ArgoCD system sets up already.
Describe the solution you'd like
Bypass token authentication and be able to install argocd-image-updater along side arogcd without any special work for authentication or authorization.

Does the implementation of #138 not address your use case? It enables the use of Argo CD's repo creds stored in argocd-cm.

This has already been implemented by #138 and also #140 and will be part of the upcoming v0.9.0 release.
The release is basically ready, but we need to wait for Argo CD v1.9.0 due to a dependency to a new feature in Argo CD.

So - actually I think that this was mostly a documentation issue. The documentation (that I found) does not clearly identify that argocd-image-updater can run in kubernetes mode where it interfaces directly with the Application objects, rather than the argocd web interface. That is great and exactly what I wanted. I have it working now.


Need to Auto Declare the form group control

I think it's worth point out that when you create the form group control in steps 12 and 13, you must set the Auto Declaration property to Yes.  For VS novices like me it took me quite some time to figure that out...

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

ID: 58543120-6ce0-15a0-dd0a-ca4888b11d85
Version Independent ID: 4e804992-b8bd-e312-eb85-da4b25f9d3cd
Content: Add analytics to workspaces by using Power BI Embedded - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-ops-core/dev-itpro/analytics/add-analytics-tab-workspaces.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @TJVass
Microsoft Alias: tjvass

Thank you for your feedback. @TJVass, please consider adding more info here.

Created bug 384070 to track.

Updated the topic with this feedback. Changes will publish later today.


WIP: Refactored ContactExtensions.UpdateCampaignContact to fix issues when creating and modifying campaigns.

Refactored to improve readability and fix EntityFramework related errors.
I think the tests for this method were broken and hence skipped so I'll need to take a look, however I've kicked and punched it manually and all seems good :grin:
Would appreciate an extra pair of eyes on it incase I've missed a use case.

@VishalMadhvani, this is just a suggestion. Since UpdateCampaignContact is only used by EditCampaignCommandHandler, I think it would be easier/clearer to move that code out of the extension method and into the handler code. Then delete the extension method. That way, we don't have extension methods "hiding" database access code.

Yep @mgmccarthy agreed. EditCampaignCommandHandler needs to be reviewed and possibly cleaned up. There is potentially more bugs in there and there may also be more cases where we use extension methods to do stuff. I'll raise an issue for this.
But for now, I'll move the code out of the Extension class and into the Handler class.


fixed #976

Solves #976
I solved the issue.
There was a problem during retrieve of firehose streams when the number of streams in the region is over 10 (the default limit value of ListDeliveryStreams API). For further info read the documentation linked below.
https://docs.aws.amazon.com/firehose/latest/APIReference/API_ListDeliveryStreams.html#Firehose-ListDeliveryStreams-request-ExclusiveStartDeliveryStreamName
Unfortunately I couldn't write an unit test because of the code design. I wasn't able to write a firehose mock.
Any suggestion is welcome.

@googlebot I signed it!


Update igc_lib.py

updated my fork to reflect changes in base.
changes from last pull request: Updated degrees_float_to_degrees_minutes_seconds argument lon_or_lat to be not optional.

changes made to dump_thermals_to_cup_file(), degrees_float_to_degrees_minutes_seconds: docstring  and removed hemisphere of '?' if the lon_or_lat argument not 'lon' or 'lat' . Guess that will induce an error if a different value is used as the argument as function will try to return an un-initialised variable(hemisphere) .. probably not a bad thing.


In the section Configuration Information

We have stated that the command to use to view the node selected as User Activity master is Get-AdfsFarmInformation.FarmRoles and if we run the command as it is it will error stating "Get-AdfsFarmInformation.FarmRoles : The term 'Get-AdfsFarmInformation.FarmRoles' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again".  The revised command to be used is (Get-AdfsFarmInformation).FarmRoles

@billmath  Please review this text and let me know when it's okay to merge.

Ref.  #3328 (Typo PowerShell Command)
(As for myself, I don't have enough experience with PowerShell to know for sure that the added parentheses are required for the command to work.)

Attaching transcript and screenshot from my lab running ADFS on windows server 2016
(Get-AdfsFarmInformation).FarmRoles
<EMAIL_ADDRESS>Get-AdfsFarmInformation.FarmRoles
<EMAIL_ADDRESS>Regards,
Rahul Rajan
Support Engineer                                                            Office:<PHONE_NUMBER>235 || x82412 ( Please dial the entire 13 digit number )
Cloud Identity Team                                                       •<EMAIL_ADDRESS>Customer Services and Support                                     Working hours: Mon-Fri 12:30 AM - 9:30 PM IST
[Microsoft Logo]
From: Trond B. Krokli<EMAIL_ADDRESS>Sent: Saturday, March 14, 2020 2:18 AM
To: MicrosoftDocs/windowsserverdocs<EMAIL_ADDRESS>Cc: Rahul Rajan<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windowsserverdocs] In the section Configuration Information (#4016)
Ref. #3328https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fissues%2F3328&data=02|01|rrajan%40Microsoft.com|ae4a0d28b43b4a912c2408d7c78fe12c|72f988bf86f141af91ab2d7cd011db47|1|0|637197293087151608&sdata=SYXSaumNXZnqKAt97PKTqwVmtIn9AcI3oSrmDciUB2Q%3D&reserved=0 (Typo PowerShell Command)
(As for myself, I don't have enough experience with PowerShell to know for sure that the added parentheses are required for the command to work.)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fpull%2F4016%23issuecomment-598904973&data=02|01|rrajan%40Microsoft.com|ae4a0d28b43b4a912c2408d7c78fe12c|72f988bf86f141af91ab2d7cd011db47|1|0|637197293087151608&sdata=jRnEC9Yr34Iz3zwJRcMEo6iMQILunrv%2FaKkxsR4nv%2F0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOCGN5I2X2HLPPATQC2LQ5TRHKLZVANCNFSM4LHAOWFA&data=02|01|rrajan%40Microsoft.com|ae4a0d28b43b4a912c2408d7c78fe12c|72f988bf86f141af91ab2d7cd011db47|1|0|637197293087161602&sdata=ls1d0DVcGZI3ZyurasxTDht3YCUKiqh%2Fz23s8minhPY%3D&reserved=0.

Oh well. The image seems to have gotten filtered out on the way from your email system to the GitHub page. Never mind. I trust that you have got the necessary documentation.

#sign-off


docs: copy-edit README

This commit improves the readability of the README without adding or changing any of the meaning.

Thank you for the repo, @bxb100!
リポジトリを ありがとう 😄

Thanks~


Server Crashing On Final Chop

This mod alone is causing my server to crash on the final stage of the chopping after completion.
Works in single player mode though.
Server Setup
i7-7700k CPU
25gb Ram

Sorry about that. Could you share a crash log?

I'm having the same issue, granted, my server is running approx 74 mods. This is my first time reporting a crash, so unsure of what crash log files are needed. Attached is the file that seems the most relevant.
crash-2023-07-22_18.29.47-server.txt

Hey, thanks for the log! I see the error, I'll get it fixed right away

I've posted a new 1.20.1 version (0.18.1) on CurseForge, it should be available in a few minutes. 1.19.4 will follow

Appreciate the prompt response!

Appreciate the fix! The Fabric version is working very well with my other 27 mods.
Listed below to show you the compatibility!

Immersive Portals
Visual Overhaul
Diagonal Fences
Connectible Chains
Visual Workbench
Universal Bone Meal
Not enough animations
Interactic
Falling leaves (particle effect)
Enchantment Descriptions
Apple Skin
Zoomify
ShowMeWhatYouGot
River Redux
Rare Ice
Horse Stats
Geophilic
Furnace Recycle
Friends and Foes
Feeding Trough
Easy Shulker Box
Easy Magic
Easy Anvils
Creeper Overhaul
Crafting UI Tweaks
Chunky
Axes Are Weapons
Amplified nether

Nice, thanks! I'm always on the lookout for adding compatibility with other mods, so do let me know if you have any requests :)


Debug why WirePayForMessages are not removed from the mempool when rechecking transactions

Summary
In #581, we introduced a temporary fix to remove WirePayForMessages from the mempool. For a long term solution, we need to debug why the transactions are not removed when rechecking the transactions in the mempool after they are committed.
Details
The temporary fix introduced in #582 relies on a wrapped transaction type that attaches the hash of the parent WirePayForMessage transaction to the child MsgPayForMessage. This hash is then used to identify and remove the old WirePayForMessage from the mempool after the MsgPayForMessage is committed to.
This solution has a lot of downsides, and is only meant to patch the bug until we can debug why the existing solution isn't working as expected.
References
Hacky solution issue #581, core PRs #582 and #583, along with the app PR #149

Can you clarify what a "parent" tx and the parent tx hash vs the current (?) tx hash is?

Heyo https://github.com/cosmos/cosmos-sdk/pull/12060

I think we can close this since our release of v0.46.0 of the sdk has the above fix, and will be included after we hardfork mamaki


HTTP: add a disableAutoOrigin option

https://groups.google.com/g/gatling/c/NBW8EdronWU
We want an option similar to disableAutoReferer.
The option must be propagated to here to not add an Origin value.

Hi i would like to work on this issue, But being a beginner, can you guide me how to contribute and what EXACTLY needs t o be done here .
thanks🙂

Hi,
I've updated the issue description. Good luck! :)

I still cant figure out what should i do, I dont know scala language. Please help

The point of contributing is to help and alleviate the burden on core maintainers.
Plead read some tutorial on Scala, install a Scala dev environment and check the code.


Program admin can view list of applicants' submitted answers

See list of submitted apps, along with applicants' names. Click and you can view their answers, and maybe also the uploaded supporting documentation. No need to support downloading for now.

Still open


Movie API types to a separate package

So that they can be imported in k8s without handlers etc.

Can one of the admins verify this patch?

@cadvisorJenkinsBot: add to whitelist

Thanks for the PR @mwielgus! LGTM.

#490


add mirror option

‘electron-download’ can use the mirror url for download

Thanks for filing a pull request! Per our contribution guidelines for pull requests, since you're adding a new option, it needs to be documented in both docs/api.md and usage.txt.
That being said, I'm beginning to think that we should have a --download flag that works like the --osx-sign or --version-string. cache, mirror, and strictSSL would be sub-options.

That being said, I'm beginning to think that we should have a --download flag [...]

It bugged me enough that I wrote a PR for it this weekend: #320


Fix for broken cognitive service docs

Cognitive services was running into long path issues when the package named is used for directories during doc generation. Switching to shorter names to resolve the issue.

The doc indexing script changes look good. I'm not sure about the other changes in this PR though they kind of seem unrelated.

I included @maririos 's changes so that I can merge them together. I am also forcing identity to run just to make sure other pipelines are not broken. This is all because I cannot manually run the pipeline against the PR right now.

I included @maririos 's changes so that I can merge them together.

So should I close my PR https://github.com/Azure/azure-sdk-for-net/pull/9386

I included @maririos 's changes so that I can merge them together.

So should I close my PR #9386