Install problem(s)

Ok, I'm having a lot of issues, but I would LOVE to get this working on my windows desktop instead of my older mac laptop. Thanks very much in advance for creating nvm for windows! I'm using Win7 64-bit.
The installer puts the files in my username\AppData\Roaming folder by default. Is this correct behavior? If these files follow me to another windows computer, will everything still work?
When cmd.exe is run at the end of the installer, it complains about a missing \settings.txt, even though it is present. If I close that cmd.exe and start a new one, the error message goes away. (Possibly the cmd.exe launched from the installer has not yet picked up the new %path% environment variable?)
Even though the '\settings.txt' error message has gone away from subsequent cmd.exe invocations, the path is still not set up properly-- %NVM_HOME% and %NVM_SYMLINK% are listed when I inspect the path from the command line. (I mentioned this in another issue thread. Workaround is to add the proper dirs to the path manually a la "set path=%path%;%nvm_home%;%nvm_symlink%").
When I open up Win7's 'Environment Variables' dialog box, it looks like the two nvm-windows variables are both in my User Variables as well as my System Variables. I'm not used to seeing variables duplicated like that.
Running "nvm use [...]" causes two separate windows security dialogs to open due to making changes to the symlink in \Program Files.
I run "nvm install 0.10.33" and it seems to work, but only after I run "nvm use ..." which is a little odd since this is the first and only node.js I've installed thus far. But maybe that's how the 'nix versions work.
When I then install another node.js, 0.11.14, install succeeds. When I "run nvm use 0.11.14" that works too. And when I run "nvm list" I can see my two installed versions, although I'm seeing "?-bit executable" instead of "64-bit".
When I try and switch from 0.11.14 to 0.10.33, I get the error "node v0.10.33 (64-bit) is not installed." I get a similar error when I run "nvm use 0.11.14".
So then I run the uninstaller for nvm-windows in the control panel. For some reason a new PATH variable is now added to my User variables where none was there before. It appears to be a copy of my System variable PATH. So now when I open a cmd.exe, my PATH is doubled in size and content. (Deleting my User variable for PATH fixes this.)

@colmsjo Thanks!  Removing the user vars, fixed my issues.
@coreybutler (I was seeing the duplicates.)

Thanks, @colmsjo! This has taken me awhile to figure out. I also needed to add a NODE_PATH (set to C:\Users{USER}\AppData\Roaming\npm), which I didn't have previously.

@colmsjo I had to remove duplicate NVM_HOME and NVM_SYMLINK user variables as well, but I couldn't find much of an answer to why outside of an acknowledgement in a comment on one of the answers.

Stale. If this issue crops up again, please open a new issue.


Use Exception#message getter instead of @ivar to allow overloading

Suggestion as in the title. It would match Ruby behavior too.
ruby -e "class FooError < RuntimeError; def message; super + ' -- bar!'; end; end" -e "raise FooError.new('foo?')"

-e:2:in '<main>': foo? -- bar! (FooError)

@Sija would you mind adding a spec that test customization of message and checks it?
Thank you.

@luislavena Sure thing! For some reason I don't understand there's nil assertion failing, could you have a look please?

I found what's going on. In https://github.com/crystal-lang/crystal/commit/b3943d5c56500d8ca8f50c9591374c237a6b32c3#diff-8a92cd1599650bc7c991a21c66fbc77bR171 there should be backtrace?.try &.each do and not backtrace.try &.each do .
There was not specs for Exception#inspect_with_backtrace so that's why it didn't appear before. So this is really useful.
I will do one more comment inline and after squashing the commits in a single commit we should be fine :-).

@Sija if you may, include a fix for https://github.com/crystal-lang/crystal/pull/3194#issuecomment-242474441 so the specs pass.
Then if you don't know how to squash the PR in a single commit check here.
Thanks!

@bcardiff Since specs are still failing should I include fix for Exception#inspect_with_backtrace inside the exception_spec.cr?

@Sija yes please. I tried to be clear on that, sorry (& thanks!).
And the changes in a single commit should be fine. I still see that you are using 1 for the fix and 1 for the specs.

@bcardiff Oki doki, I wasn't sure I grok your intention ;) Should GTG now.

@bcardiff done.

Thanks!


[Added] default index.html with substructure

Coverage decreased (-0.4%) to 41.221% when pulling 356b704aee6417c94122f63b71cce1ae4c89cbc4 on Drogglbecher:master into 8953687afb5bf65fa1fc754e5c8f9b6beb98971e on rust-leipzig:master.


can we stop slider between dots?

No, can not. Wanna keep it simple and clean as i can, so i don't ever plan add this as core functionality. But feel free to fork and implement this.


[Suggestion] Add adblocker

Hey!
I would love to see an adblocker being add.
Thanks

Thanks for sharing your thoughts! I've decided not to add an adblocker to ensure compliance with Crunchyroll's policies. Additionally, there are already great adblocker options available that users can integrate with the extension for an ad-free experience. Appreciate your understanding!


[SPARK-10290][SQL] Spark can register temp table and hive table with the same table name

If trying to register temp table with the same table name which has been saved as hive table, an exception should be thrown:
throw new AnalysisException(s"Table $tableName already exists.")

In reverse, if trying to save a hive table with the same table name which has been registered as temp table, then the outcome will be determined by SaveMode, which was already intact.

Can one of the admins verify this patch?

As said on the JIRA this is by design and we can't change this without breaking existing code.  If you want to query the hive table you can do so with its fully qualified name.  As such, we should close this issue.


Create High-Fidelity Wireframe for User Interface Design

We need a detailed and visually appealing representation of our application's user interface to guide the development process accurately. Creating a high-fidelity wireframe will provide a clear visual understanding of the interface layout, components, and interactions.

Landing Page:
A clean and welcoming interface introducing users to our application's purpose and features.

User Registration and Login:
Intuitive forms for user registration and login with input fields for username, email, and password. Include appropriate validation and error messages.

Document Upload Interface:
A user-friendly upload interface allowing users to upload documents or images for Personally Identifiable Information (PII) detection.

PII Detection Results:
Clear visualization of PII detection results, including highlighted regions within uploaded files where PII is detected.

User Dashboard:
Dashboard displaying user-specific information, uploaded files, detection history, and other relevant data.

Navigation and Menus:
Well-organized navigation menus and buttons for seamless movement between different sections of the application.

Error Handling and Feedback:
Thoughtfully designed error messages and feedback notifications to guide users in case of invalid inputs or unsuccessful operations.

Responsiveness:
Ensure the wireframe is responsive, adapting to various screen sizes and devices (desktops, tablets, smartphones).

Can I work on this issue under hacktoberfest?

Please assign me this issue. I would like to work on this.

Assigned


cursor not active at program start

hello friends,
a bug report is just expressing the program is working ;).
after android kills acode, I'm restarting it and see the cursor in the auto opened files.
but I can't move the cursor with the arrow keys in the shortcuts menu.
please fix,
alex

I can't reproduce this


[pmo] Fix load_borrow for strong control equivalence issues.

This entailed untangling a few issues. I have purposefully only fixed this for
now for load_borrow to make the change in tree smaller (the reason for my
splitting the code paths in a previous group of commits). I am going to follow
up with a load copy version of the patch. I think that the code should be the
same. The interesting part about this is that all of these code conditions are
caught by the ownership verifier, suggesting to me that the code in the
stdlib/overlays is simple enough that we didn't hit these code patterns.
Anyways, the main change here is that we now eliminate control equivalence
issues arising from @owned values that are not control equivalent being
forwarded into aggregates and phis. This would cause our outer value to be
destroyed early since we would be destroying it once for every time iteration in
a loop.
The way that this is done is that (noting that this is only for borrows today):

The AvailableValueAggregator always copies values at available value points to
lifetime extend the values to the load block. In the load block, the
aggregator will take the incoming values and borrow the value to form tuples,
structs as needed. This new guaranteed aggregate is then copied. If we do not
need to form an aggregate, we just copy. Since the copy is in our load block,
we know that we can use it for our load_borrow. Importantly note how we no
longer allow for these aggregates to forward owned ownership preventing the
control equivalence problem above.

Since the aggregator may use the SSA updater if it finds multiple available
values, we need to also make sure that any phis are strongly control
equivalent on all of its incoming values. So we insert copies along those
edges and then lifetime extend the phi as appropriate.

Since in all of the above all copy_value inserted by the available value
aggregator are never consumed, we can just add destroy_values in the
appropriate places and everything works.

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - eab9be8f2fb8f6d6b19ee9ee25172a21693d01f4

Failure was in the gyb stuff...

All of the full tests stuff pass/ran. We only failed late due to a different swiftpm issue.

erm swiftsyntax diff issue.

@swift-ci smoke test macOS platform


Fix Open in Browsers due to schema changes

Due different result shape, handle both scenarios of the api result to be able to detect host and port

Addressed in #46


[5.5] Let \Illuminate\Auth\Events\Failed also accept the current request.

Let \Illuminate\Auth\Events\Failed event also accept the current request.

I think any handler could just type hint the request in its constructor.


Better default formatting

It would be great if this integrated with common formatters (Prettier/ESLint) when they are present.
Alternatively, it would be nice to have editorconfig support.

What do you mean by this integration?
You want to load settings from this tools eg. how much spaces is tab etc.
Or you want to call them instead default tsserver formatting?

Ideally it would call Prettier (or prettierd) when formatting, but it would also be good if it automatically matched it.
Currently, I have to disable the formatting options from typescript-tools to properly format code in our project.

So, this feature is out of scope of this plugin. We aim to support only features exposed by tsserver. So your solution is perfectly fine.


INI jobs gets deregistered

I have a docker compose project of which ofelia is a service.
The jobs in ofelia are declared in the config.ini file.
But shortly after the container start, the registered jobs are de-registered automatically.
2023-09-26T11:33:40.89-04:00  scheduler.go:51 ▶ NOTICE  New job registered "local backup" - "/backup/run_backup" - "0 0 1 * * *" - ID: 1
2023-09-26T11:33:40.891-04:00  scheduler.go:51 ▶ NOTICE  New job registered "kopia backup" - "/backup/kopia_backup" - "0 30 2 * * Tue,Sat" - ID: 2
2023-09-26T11:33:40.892-04:00  scheduler.go:62 ▶ DEBUG  Starting scheduler
2023-09-26T11:33:40.892-04:00  cron_utils.go:13 ▶ DEBUG  start
2023-09-26T11:33:40.892-04:00  cron_utils.go:13 ▶ DEBUG  schedule
2023-09-26T11:33:40.892-04:00  cron_utils.go:13 ▶ DEBUG  schedule
2023-09-26T11:33:50.89-04:00  scheduler.go:56 ▶ NOTICE  Job deregistered (will not fire again) "local backup" - "/backup/run_backup" - "0 0 1 * * *" - ID: 1
2023-09-26T11:33:50.891-04:00  scheduler.go:56 ▶ NOTICE  Job deregistered (will not fire again) "kopia backup" - "/backup/kopia_backup" - "0 30 2 * * Tue,Sat" - ID: 2
2023-09-26T11:33:50.891-04:00  cron_utils.go:13 ▶ DEBUG  removed
2023-09-26T11:33:50.891-04:00  cron_utils.go:13 ▶ DEBUG  removed

I've released the fix in v0.6.4


Replace slowapi rate limiting implementation with WAF ruleset once available

Rate limit introduced here.
Implementation example AWS WAF rate limit.
https://stackoverflow.com/questions/65252674/when-using-terraform-with-aws-how-can-i-set-a-rate-limit-on-a-specific-uri-path

Related #142


Reimplement simple pwm

This is a reimplementation of the "simple" fastpwm functionality from the old-branch.
I've only been able to test this on a Arduino Uno, a Leonardo and a Trinket, but I've tried to add implementations for some of the other MCUs (atmega48p, atmega168, atmega328pb, atmega2650) as well, based on the code in the old-branch.
I also added implementations for atmega32u8, attiny85, attiny88 and atmega1280 as well, which wasn't available in the old branch IIRC.
Any feedback would be greatly appreciated.
Closes #251.

Fixed my typo which tried to use simple_pwm on all MCU types instead of just the atmega ones.

This PR should probably have been a Draft from the beginning, sorry about the noise..
I'll add simple_pwm implementations for the remaining MCU's and fix the documentation errors that I've made this coming weekend.

I didn't set out to implement this for all the (currently) supported MCUs from the beginning, but it was rather fun, so I just kept going.
This should now be fairly "complete" as far as the current MCU support goes.
The tests break because they depend on a new release of avr-device (due to the missing timer-patches for atmega1280), so I guess the "waiting-for-avr-device" label would be appropriate here.
Pros

I can now fade my LEDs.
The end-user API stays the same as on the old-branch, which makes it easier to port my projects.
I learned a ton about the different timers on AVR-chips just by skimming the datasheets.

Cons

I'm not sure if this thin abstraction will fit the 4809. Those timers are configured in a totally different way, and since I lack the necessary hardware, I haven't been able to take a stab at it at all.
This is just plain old FastPWM, which now feels a bit "meh", considering how much cool stuff you can do with the rest of the timers and/or modes on AVRs. So I guess this is more of a stopgap between no PWM API at all, and a fully fleshed out PWM API with support for phase-correct wave generation modes, frequency control, PLLs, dead-time configuration and what have you. This is being discussed in other issues; #44 for example.
Adding this as a "weak" API right now might cause headaches/churn further down the road, if a more powerful API come along to replace it.

As said; any feedback is greatly appreciated.

Thank you SO MUCH for this PR! I know nothing about microcontrollers and that means I'm hesitant to use workaround from other issues that uses unfamiliar for me internal names. Using this branch I was able to play with my LED too :D

Hey @dalpil, with #283 the new version of avr-device is now used.  Please rebase your PR ontop of the latest main to pull in this change and hopefully get CI to pass this way.
Sorry for the slow progress from my side here...

No worries, I'm guessing that you maintain these libraries as a hobby, and it wasn't my intention to cause any stress by opening a fairly big unsolicited pull request. I'm just happy to tinker with this. :)


[tcid-iuod25]Upgrade the application with 3 replicas when two pool pods are in pending state

What this PR does / why we need it:

Upgrade the pools  when two pool pods are not running.

Details:
Steps involved in openebs update in cluster:

Installing openebes:

Check whether all the openebs components are in running state or not.
Check whether application pools are in running state or not.

Steps involved in the test case

To send one application pool replica to pending state just change the image of the deployment.
Now replica pod will go into pending state.
After pod went to pending state try to upgrade control plane components.
It should give error code 405 and the test case will pass when it gives response status code 405

Additional Information-

Title
Description

Assumptions
openebs should be already installed with version less then 1.7.0

cStor pools should be available and should be in running state

3 Node cluster

Application Under Test
Openebs Pool upgrade using DOP when all pool pods are not running

Stogare Engine
cStor

Application Used
MongoDB Statefulset

Openebs Version
Version upgrade to 1.7.0

Notes to reviewer

This is the negative test case where we have to upgarde pools when one of the pool pods are not in running state it should give some error.
Dependent PRs -

Signed-off-by: Amit Bhatt<EMAIL_ADDRESS>

@amitbhatt818 Let us try to give specifics here:
[tcid-iuod25]Upgrade the application with 3 replicas when two pool pods are in pending state
Which application are we talking about?
Which pool are we talking about? Is it cstor pool?
TCID can be TCID-DIR-OEE-CSTOR-POOL-UPGRADE
where application=cstor, pool=cstorpool
assumptions: upgrade is done via director


Schedule date is not reset when closing publish menu

Issue Summary
Changes in the publish menu are supposed to be reset if it's closed without saving. For most changes this is true but if you have a scheduled post and you change the schedule time in the publish menu, that time is set on the model and not reset when it's closed leading to the next autosave unexpectedly changing the scheduled time.
Steps to Reproduce

Schedule a post
Whilst editing the scheduled post open the publish menu and change the time in the date/time picker
Close the publish menu without saving (Cancel button or click outside)
Note that the scheduled time shown in the PSM now shows the new time you selected
Open the publish menu - the new time is also shown there
Refresh the page - the original scheduled time should now be shown

When closing the publish menu, any unsaved changes should be reset.
Technical details:

Ghost Version: https://github.com/TryGhost/Ghost-Admin/commit/08edc266d18ec0cf99867f96964d5c08dd20dc7d

@kevinansfield I am not able to reproduce this. When I follow the steps above, the date is always being reset to what is was...

Sorry if the steps weren't very clear, I've made a screencap to demonstrate...

After closing the publish menu without saving the scheduled time should be reset so that it reads 8:06 in the PSM rather than 8:20


Xcode project won't build RE: ld: library not found for -lassimp-fat

I load the workspace in Xcode 10.2.1 and attempt to build:
Yields Error:
ld: library not found for -lassimp-fat
Noted  that there seems to be the following missing static libs:
libIrrXML.a
libIrrXML-fat.a
libassimp-fat.a
libassimp.a
I cloned and built assimp at the terminal.  It produced:
libassimp.a
libIrrXML.a
but didn't produce the multi-architecture -fat versions.
If I remove the others and add the non fat versions I get:
ld: library not found for -lassimp

Check that you have 7z installed
https://github.com/dmsurti/AssimpKit#installation-with-carthage-ios-100-macos-1011
and that you have the latest master because otherwise you might have the issue with spaces in a path to a project https://github.com/dmsurti/AssimpKit/pull/100.

I got "ld: library not found for -lassimp". Not sure if my solution is relevant at all but my issue was having a space in one of the directory names. Once I corrected that it built correctly.


FIELD_OFFSET

@retep998 Do you want this one?

As useful as this macro is for Windows FFI, it is useless for writing FFI bindings, and can be trivially provided by a crate independent of winapi where it would be useful for all FFI in general, and not just Windows FFI.


kick off NIO2

Motivation:
NIO2 development starts now.
Modifications:
Made NIO Swift 5-only for everything else see docs/public-api-changes-NIO1-to-NIO2.md
Result:
NIO2 development can start.

@swift-nio-bot test this please

@swift-nio-bot test this please

@swift-nio-bot test this please

@tomerd / @Lukasa this will unblock this other PRs until  the https://bugs.swift.org/browse/SR-9204 fix lands in the swift 5 dev previews


[LIBCLOUD-670] Improvements to HostVirtual driver

methods to handle 'Packages' concept [basically, it is a reserved compute instance with pre-allocated resources (RAM,Disk,IPs etc) in any specified location]
changed create_node to use the extra packages method
fix website url in docs
fix method ordering
fix kwargs abuse in create_node
fix docstrings
add test cases for extra methods
fix the fixtures
fix default location

Changes look good to me. I went ahead, squashed the commits and merged them into trunk. Thanks.


Dropbox chooser returns relative links instead of absolute in link property

Hi! We are using the Chooser (https://www.dropbox.com/developers/chooser) with the following settings
Dropbox.choose({
     linkType: 'preview',
     ...
})

and it worked well so far.
The selected file had .link property similar to https://www.dropbox.com/s/kfo5w0500ieprfe/[ jpg.jpg?dl=0. But now the link is relative like: "/Photos/IMG_4199.JPG". And that breaks our application.

Thanks for the report. We're working on it.
By the way, please note that this "dropbox-js" library is unrelated to the Dropbox Chooser. It also uses the retired API v1, and so is also retired and unsupported. For future reference, you can report Dropbox developer platform issues here: https://www.dropbox.com/developers/contact

This should be fixed now. Apologies for the trouble!

@greg-db Thanks 👍 ! To be honest i didn't find where to report this issue and leave it just here.


TSServer died unexpectedly more then 5 times  in big monorepo - Vscode

Bug Report
I have a big Monorepo (32 subpackages) using Yarn Workspaces (node_modules symlinking). The VSCode integrated Typescript Language server dies every few minutes, making it hard to use Typescript functionality. Error Message is:
Ts server dies unexpectedly more then 5 times in the last 5 minutes
🔎 Search Terms
TSServer died unexpectedly, tsserver crash, vscode ts server, monorepo, large
🕗 Version & Regression Information
currently using typescript 4.2.2, the problem started occuring once my project got bigger (subpackages added up)
💻 Code
I can provide the verbose ts server log, but would appreciate if it could not be posted publicly.
🙁 Actual behavior
Error Message is:
Ts server dies unexpectedly more then 5 times in the last 5 minutes
🙂 Expected behavior
it doesent crash, and i can use vscodes ts features like a charm

4.2.2 is several versions old. Does it crash with the nightly? https://marketplace.visualstudio.com/items?itemName=ms-vscode.vscode-typescript-next

Hey, sorry totally forgot about this issue. It was fixed along the way by upgrading TS i think.


Use same color for same namespace.

CF #258

Nice and simple, thanks for the PR! I'm 👍  for this feature. There's been plenty of times where the order of require calls changing ends up changing the colors of the debug instances, since they get instantiated in a different order which has always felt strange.
The only thing I'd be worried about is if the algorithm doesn't "distribute" the colors evenly enough. But I guess we'll just have to feel it out and see.

I agree, it will have less perfect distribution on colors over namespace.
To reduce chance of collision, as it's no more incremental, the only solution is to have more colors available.

This algorithm doesn't seem to work well. Example:
  console.log({
    namespace,
    hash: Math.abs(hash),
    color: exports.colors[Math.abs(hash) % exports.colors.length],
  });

{namespace: "rto-ext:EVENTS", hash:<PHONE_NUMBER>, color: "lightseagreen"}
{namespace: "rto-ext:SETTINGS", hash:<PHONE_NUMBER>, color: "lightseagreen"}
{namespace: "rto-ext:BG", hash: 33268638, color: "lightseagreen"}

Three different namespaces have the same color.

exports.colors.length == 6
So 16% of chance of collision.
@TooTallNate More color ?
Otherwise we need to have an array to store the colors stickiness. Not as complicated as this.

We can try something like that otherwise : https://github.com/lchenay/debug/commit/9f4404f8cd695191743e668ddd0fe4f44ca6cd34
The color will be better distributed as before, with reuse of same color for same namespace, but will use some memory to store the array of mapping.
This can lead to memory leak if a user use debug in prod with random namespace. For exemple those who use a namespace like "myapp:request:${request.expressRequestId}".


Reduce number of queries needed for placement requests and applications

Context
Slack thread where we debugged this initially.

We think that at 8:29 on Nov 8th 2023 user 17073fa0-f6c8-46a3-a6be-fcda599d2d22 requests /placement-requests from CAS1
We think that request failed to load all the results from the database and ended up in an out of memory error which bubbled up into a result set exception
We checked preprod (which is synced from prod) and found that user had nearly 400 placement requests assigned
The team expect that to be normal for CRU's
We think the long term solution is to paginate these results so we don't ever have to load ~400 records into memory. CAS1 have been advised to prioritise this in their backlog
As a short term mitigation we remove columns we don't care about from the result set to make it more efficient. This includes the small json blob stored within  the data column. The way we do this is to replace the derived Hibernate query and replace it with our explicit query which should honour the same interface

Changes
As part of this work we identify N+1 queries. By moving away from the derived Hibernate queries and (sadly) writing our own SQL, we use some tactical JOINs to grab the data we need for transformation, upfront. We hope that if we load this Hibernate will be smart enough not to execute additional queries.
We first tackle PlacementRequests for the /placement-requests endpoint and then PlacementApplicationsfor the/tasks endpoint as I could see a similar pattern of problem there.
I would really like to avoid writing out SQL like this but I'm unable to find a better way to do this with Hibernate. In rails we can do Model.eager_load(association_1, association_2, …) which removes the need to write low level SQL ourselves?
This has a side benefit that we are then able to remove placement_application.data from when looking for placement_requests where they aren't needed. These blobs aren't big but when a user needs to see 400 this could make a difference to both memory usage and sql performance.
How to view these sql changes?
Set this in your API's application.yml:
jpa:
    hibernate:
      ddl-auto: none
    properties:
      hibernate:
        dialect: org.hibernate.spatial.dialect.postgis.PostgisDialect
        show_sql: true
        format_sql: true

Restart the server (brought up originally by AP Tools).
I'd like to have figured out how to seed CAS1 so there were placement requests and applications. Then I could demonstrate the saving in queries. A production example of all the sql queries which actually times out the latest request to this endpoint can be viewed in App Insights here.

It became apparent these queries weren't calling application.data and that something else must be so the value of doing this is diminished.

Now I've got test data working locally I'm not convinced Hibernate does respect this way querying so closing this for now.


Cannot get IMU data into python 3.7.10 on Windows 10 laptop

Required Info

Camera Model
D435i

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC

SDK Version
<IP_ADDRESS>85

Language
python

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I recently purchased a D435i RGBD camera and am trying to stream depth, RGB, and IMU data from the camera. When I try to do that, I get the following TimeOutError:
"Frame didn't arrive within 5000"
Note that I tried doubling the timeout period and this did not do anything. When I comment out the added IMU streaming channels and stream only depth and RGB data, then everything works fine. If I do the opposite i.e stream only IMU data, I get the same timeout error.
From my review of the logs of issues that have been raised over the past few years, I see that this has been an on again off again issue for a couple years now. The fact that this issue still seems to persist is truly astounding, considering one of the primary functions of the D435i IS TO PROVIDE IMU DATA!! There is nothing more frustrating than paying good money for a product and not just having it work right out of the box, then needing to spend HOURS fumbling around the internet looking for a solution.
Code can be found here:
`import os
import numpy as np                        # Fundamental package for scientific computing
import pyrealsense2 as rs
import matplotlib.pyplot as plt
import time
class RealsenseCamera:
def __init__(self, data_types_of_interest=['depth', 'RGB', 'IMU']):
    self.data_types_of_interest = data_types_of_interest
    print("Searching Camera..")
    try:
        self.camera = rs.context().devices[0]
        print('Camera type:', self.camera.get_info(rs.camera_info.name))
    except IndexError:
        raise IndexError("No RealSense device detected!")
    print("Available sensors for", self.camera.get_info(rs.camera_info.name))
    for s in self.camera.sensors:  # Show available sensors in each device
        print(s.get_info(rs.camera_info.name))
    self.acquiring = False

#Currently (08/18) I am having trouble retrieving IMU data from the camera. IMU functionality has been validated using Intel
#RealSense viewer, which is testing program creating by Intel. I posted an issue on the GitHub page for librealsense, the code
#base for interacting with the camera (pyrealsense2 is a part of librealsense), so we will see what comes of it.
def begin_data_acquisition(self):
    if self.acquiring == True:
        print('Data acqusition has already begun!')
        return
    self.acquiring = True
    print("Beginning data acqusition")
    self.pipe = rs.pipeline()
    self.cfg = rs.config()
    self.cfg.disable_all_streams()
    #Done so that we are only acquiring the data types we are interested in. Makes data acquisition faster
    if 'depth' in self.data_types_of_interest:
        self.cfg.enable_stream(rs.stream.depth, 848, 480, rs.format.z16, 30)
    if 'RGB' in self.data_types_of_interest:
        self.cfg.enable_stream(rs.stream.color, 1280, 720, rs.format.rgb8, 30)
    if 'IMU' in self.data_types_of_interest:
        self.cfg.enable_stream(rs.stream.accel, rs.format.motion_xyz32f, 200)  # acceleration
        self.cfg.enable_stream(rs.stream.gyro, rs.format.motion_xyz32f, 200)  # gyroscope
    print('Skipping first few frames so camera can calibrate...')
    self.profile = self.pipe.start(self.cfg)
    for _ in range(10):  # Skip first frames to give syncer and auto-exposure time to adjust
        self.pipe.wait_for_frames(timeout_ms=10000)
    # # while 1:
    for _ in range(10):
        self.frame_set = self.pipe.wait_for_frames()
        if 'depth' in self.data_types_of_interest:
            self.depth_frame = self.frame_set.get_depth_frame()  # Get depth frame
        if 'RGB' in self.data_types_of_interest:
            self.RGB_frame = self.frame_set.get_color_frame()
        if 'IMU' in self.data_types_of_interest:
            self.accel_frame = self.frame_set.first_or_default(rs.stream.accel, rs.format.motion_xyz32f)
            self.gyro_frame = self.frame_set.first_or_default(rs.stream.gyro, rs.format.motion_xyz32f)`

Hi @r-zonis  The Python IMU discussion at https://github.com/IntelRealSense/librealsense/issues/8492 will be a good place to start if you have not seen it already.  The RealSense user in that case was also using Windows 10 and Python 3.7.  A range of IMU script references are provided at that case which initially did not work for the user, and they also experienced the Frame didn't arrive within 5000 error.
They found that all of the IMU scripts in the discussion worked on another PC and so the problem was likely with some aspect of the configuration of the PC that they were initially testing with.

Hi @MartyG-RealSense thanks for the reference. So is the answer that there is no answer? Are there debug logs somewhere that might give me info on more precisely why the system is stalling? I am only testing this code on a Windows laptop; ultimately, in it's final application, image acquisition will occur through a Linux PC. I guess for now, unless there something I can do to better track down this problem, I will just ignore and hope that it works on Linux.

I would recommend trying the scripts in that discussion on Windows to see whether the same error also occurs in those scripts too.

Hi @r-zonis  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Any way to surface "time since last merge"

I would love to be able to have a status for projects that are truly actively maintained and  see that represented some way on CLOmonitor. Specifically, I'd love to be able to see a stat for "time since last merged commit".

Does this sound interesting enough to add in?
If this could be in scope, is there another metric that would be easier to scrape to represent "staleness"?

Thank you!!

Hi @Salkimmich 👋
We already have a couple of related checks that might help with this 🙂
Maintained check (security section)
This check comes from Scorecard and it indicates if the project is actively maintained. This check's score is based on the number of commits and issues activity on the repository during the last 90 days. The computed score varies based on that activity. You can see those details in CLOMonitor by hovering on the check's icon:

I think this kind of covers what you were describing (projects that are truly actively maintained), but if we can think of any improvements maybe we could propose them upstream so that more users can benefit of it.
Recent release (best practices section)
We also have a check that verifies if the project has produced a release during the last year, which can be helpful as a complement to the maintained one (although the time span would be longer). The latest release found is linked to the check's title in CLOMonitor.

No worries!

What I'm curious most about is any clear predictive behaviour in the year before it becomes 'inactive'.

CLOMonitor stores projects' snapshots periodically, which can be viewed from the UI by using the timeline on the right side of each project page. When using this feature in combination with the maintained check's score I mentioned in my previous comment, it should be possible to identify a descending trend that could signal that risk of going stale. All this information is accessible from the CLOMonitor's API (which isn't documented yet 😅), just in case you need to do something programmatically with it 🙂

Will close this one for now, please feel free to reopen if needed 🙂


open source: Fix broken link in table of content

Description 📜
Fix broken link in open source section.

@Aryamanz29 What is invalid please ?


improve container security

Signed-off-by: Kim Tsao<EMAIL_ADDRESS>What does this PR do?:

Adds the recommended   readOnlyRootFilesystem: true
Which issue(s)/story(ies) does this PR fixes:

https://issues.redhat.com/browse/DEVHAS-177
PR acceptance criteria:

[ ] Unit/Functional tests

[ ] Documentation

[ ] Client Impact

How to test changes / Special notes to the reviewer:

/retest

/retest

/retest


support event listener removal for buffered function

found via debugging help from @anandthakker

Thanks! Released as 2.2.3.


Less useful tools

Many of these tools are easy to write, rarely used, and gimmicky. They are things that took little time at all, and are supported by BusyBox and GNU, but nobody would complain if we didn't have them.
We might want them because the more tools the better
We might not because it's more tools that don't really add much value to most people.

I'm good with adding these. Fix the docstring and header on the yes command and let's merge it. :+1:


Error Authenticating to Okta. Origin header must not be included in token request

Okta reports an error that the request to retrieve the token contains the "Origin" header.  Is it possible to get an oidc setting that will remove the origin header?
The error message is: " Browser requests to the token endpoint must use Proof Key for Code Exchange."
The help page is here:
https://support.okta.com/help/s/article/Browser-requests-to-the-token-endpoint-must-use-Proof-Key-for-Code-Exchange?language=en_US
request information:
Request URL: https://*********.okta.com/oauth2/default/v1/token
Request Method: POST
Status Code: 401
Remote Address: <IP_ADDRESS>:443
Referrer Policy: strict-origin-when-cross-origin
Request Headers
:authority:***********.okta.com
:method: POST
:path: /oauth2/default/v1/token
:scheme: https
accept: application/json
accept-encoding: gzip, deflate, br
accept-language: en-US,en;q=0.9
content-length: 337
content-type: application/x-www-form-urlencoded
origin: http://localhost:3000
referer: http://localhost:3000/
sec-ch-ua: " Not A;Brand";v="99", "Chromium";v="98", "Microsoft Edge";v="98"
sec-ch-ua-mobile: ?0
sec-ch-ua-platform: "Windows"
sec-fetch-dest: empty
sec-fetch-mode: cors
sec-fetch-site: cross-site
user-agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.102 Safari/537.36 Edg/98.0.1108.55
RequestBody
grant_type: authorization_code
redirect_uri: http://localhost:3000/login/callback
code: **************************************
code_verifier: *******************************************
client_id: ************************
client_secret: ************************

This issue does not make sense. I must be doing something wrong. This is client side so the browser always adds Origin. The PKCE is not working correctly so is probably something on my side. Any help would be appreciated.

I"m not sure it's you. I don't understand the article myself. PKCE and browser-based clients are orthogonal (yet not incompatible) concepts. If you happen to make token endpoint request calls cross-origin, the Origin header would be included (as per the same-site policy and allowing CORS to override), and that's fine. And if you're not making the token request from a browser then the Origin header would not be present. It's a vary bizarre article.

I asked: https://twitter.com/BrockLAllen/status/1497041937983057929

I'm looking into this, give me a few minutes to try something...

Alright, there are two things going on here. The error message is misleading, but it is correctly rejecting this request.
I see you have a client_secret in that request. That means you created the app as a "Web App" in Okta, rather than a "Single-Page App". Okta won't give a client secret to an app that you've registered as a Single-Page App.
What's happening is Okta is rejecting the request because you shouldn't be using a client secret from a browser since a single-page app is a public client, and can't keep the secret confidential.
If you register the app as a Single-Page App in Okta and configure this client with only the client_id and no secret, it will work.
I was able to verify this just now by using my vanilla JS client both with and without a client secret, and I got the same error message you got when I included the secret. The error message is really bad in this case, but it's correctly trying to prevent this exact situation.

Thanks, you were right. I was using an existing application inside of Okta which was a web app and not a SPA. SPA is only apparent on creation, not readily apparent when looking at an existing application. I created the SPA application in Okta and it worked.


Cannot deploy. Error on red.min.js

Hello,
I transferred my settings/flows/configs from version 0.9.3 or something to the latest build.
Now I am on the latest version but I can't deploy. I get the following js error on the browser:
red.min.js:16 Uncaught TypeError: Cannot read property 'type' of undefined
Also I noticed on the code that the red.min.js file is loaded like

I know that doesn't matter and still gets loaded but I thought I'll let you guys know.

Hi, can you provide the full stack trace of that error from the console?
(also your comment about how the file is loaded appears cut off...)

Hey, here it is (apologies for the screenshot)

Here the text that was cut off <script src="red/red.min.js"></script>

d'oh - of course with the minified version, that isn't very helpful.
If you backup your flows file, then delete it so it starts with a clean workspace, restart everything do you get any further?

Ok thanks, that did the trick. I copied to clipboard the flows, backed up my flows and then deleted the files. Restarted the service and then imported the flows from clipboard. You may close this issue :)

If you have that backup, and happy to share, can you send it to me<EMAIL_ADDRESS>- would be interested to understand what broke. No worries if not.

No sorry I deleted the backups once it worked. I'm guessing it's the config I had for the pubnub package.


AE49_181023_194515.405

Type: Missing
Summary:
License
Details:
Add Apache 2.0
Resolution:
https://github.com/aspnet/Configuration/blob/2.0.1/LICENSE.txt
Affected definitions:

Microsoft.Extensions.Configuration.EnvironmentVariables 2.1.1,- Microsoft.Extensions.Configuration.FileExtensions 2.1.1,- Microsoft.Extensions.Configuration.Json 2.1.1

Duplicate of #519


how can i use jshint in .cshtml?

 $.ajax({
                url: '@Url.Action("PartialDashboard","Home")',
                type: "GET",
                success: function (data) {
                    $("#partialDashboard").html(data)    --> here wants to validate semicolon
                }
            })

The above code in my .cshtml

I'm not familiar with .cshtml, but the --extra-ext CLI option will let you
configure JSHint to analyze non-JavaScript files. Since this sounds like a case
where the script is embedded within markup, the --extract option may also be
relevant. You can read more about the command-line interface and its flags in
the dedicated page in the project documentation:
http://jshint.com/docs/cli/
As for editor integration, the following JSHint documentation page has
references to various plugins (including Visual Studio):
http://jshint.com/install/
Hope this helps!


Return actual error object

Overwriting the entire error message makes it harder to debug issues.  I've modified the catch statement to preserve the original error.

Thanks!


Fails when inviting grower

Hi @jsmoran123G , thanks for raising an issue.
Can you please provide some more details about your set-up to help us to diagnose your problem?

Please confirm that your REST Server that get's deployed as part of the build is live and operational
Has this ever worked for you or are you attempting for the first time?
Can you share the URL of your demo application?

Also, please feel free to open a chat within the Clause platform if you're more comfortable sharing this information privately.
Click this button to start a chat.

Hi, not sure how to confirm the Rest server if there are steps documented I
could perform that.
I followed the instructions provided when selected the perishable sample
from my running blockchain, it has not worked I get stuck when launching.
https://api.clause.io/api/clauses/5b7e127bc8d2d7e20f736ff5/execute?access_token=EEhXH5EbcjmGeA6jCmbSYdOFbQWd4JepH7JsMXiAFnHhx5PKsNni8fn7M5nkp6rq
Any help would be appreciated.
Thanks,
Jackie
On Thu, Aug 23, 2018 at 1:40 AM, Matt Roberts<EMAIL_ADDRESS>wrote:

Hi @jsmoran123G https://github.com/jsmoran123G , thanks for raising an
issue.
Can you please provide some more details about your set-up to help us to
diagnose your problem?

Please confirm that your REST Server that get's deployed as part of
the build is live and operational
Has this ever worked for you or are you attempting for the first
time?
Can you share the URL of your demo application?

Also, please feel free to open a chat within the Clause platform if you're
more comfortable sharing this information privately.
Click this button to start a chat.
[image: image]
https://user-images.githubusercontent.com/7544022/44511628-0c8fb680-a6b0-11e8-913a-ad7172812cdc.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/clauseHQ/demo-clause-ibm-perishable-goods/issues/5#issuecomment-415322044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aol1rvrXhoVlA9eXr8ZucqYvgSD9q9qgks5uTlxZgaJpZM4WIxk_
.

Closing. The thread has moved to the Clause support channel


[Bug]: Oracle mode, right-click to copy to sql, the time type copied sql content is null

ODC version
ODC423
OB version
independent
What happened?
Oracle mode, right-click to copy to sql, the time type copied sql content is null

What did you expect to happen?
not null
How can we reproduce it (as minimally and precisely as possible)?

Anything else we need to know?
No response
Cloud
No response

pass


TCP and normal IngressRoutes can't access services in other namespaces

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
After upgrading Traefik via the Helm chart to 2.5.3 through the 10.3.5 chart, I was no longer able to access TCP services and normal services, instead getting a 404 page not found error.
What did you see instead?
Since I deploy all of my ingressroutes in one namespace and point them to the services in the other namespaces, I moved some of the ingressroutes into the namespaces of the services that they serve, which then I was able to access my TCP services, although my normal services still showed the same error message.
My Traefik logs were spammed with something like this message for normal services:
"service production/production-app not in the parent resource namespace default" ingress=production-ingress namespace=default providerName=kubernetescrd

And this for TCP ingressroutes:
tcp service testing/tcp-app-svc is not in the parent resource namespace default" servicePort="{0 8080 }" serviceName=tcp-app-svc providerName=kubernetescrd ingress=tcp-app-ingressroute namespace=default

What version of Traefik are you using?
I am running Traefik 2.5.3 deployed through the 10.3.5 helm chart, all configured to default except additional ports and letsencrypt http challenges
What is your environment & configuration?
additionalArguments:
  - "--log.level=DEBUG"
  - "--api.dashboard=true"
  # Start of HTTP challenge certification
  -<EMAIL_ADDRESS>  - "--certificatesresolvers.myresolver.acme.storage=/data/acme.json"
  - "--certificatesresolvers.myresolver.acme.httpchallenge=true"
  - "--certificatesresolvers.myresolver.acme.httpchallenge.entrypoint=web"
ports:
  web:
    redirectTo: websecure
  tcpapp:
    port: 8123
    expose: true
    exposedPort: 8080
    protocol: TCP

Add more configuration information here.
k3s v1.22.2+k3s1
If applicable, please paste the log output in DEBUG level
time="2021-09-24T07:57:10Z" level=error msg="Cannot create service: subset not found" ingress=example-app-ssh namespace=example-app providerName=kubernetescrd serviceName=example-app-service servicePort="{0 1080 }"
time="2021-09-24T07:57:10Z" level=error msg="Cannot create service: tcp service tcp-example/filesharing-svc is not in the parent resource namespace default" servicePort="{0 2022 }" serviceName=filesharing-svc providerName=kubernetescrd ingress=book-ingressroute namespace=default
time="2021-09-24T07:57:10Z" level=error msg="Cannot create service: tcp service tcp-example/productionapp-svc is not in the parent resource namespace default" ingress=productionapp-ingressroute namespace=default servicePort="{0 25565 }" providerName=kubernetescrd serviceName=productionapp-latest-svc
time="2021-09-24T07:57:10Z" level=debug msg="Skipping Kubernetes event kind *v1.Endpoints" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=debug msg="Skipping Kubernetes event kind *v1.Endpoints" providerName=kubernetes
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=error msg="kubernetes service not found: default/book-service" ingress=book-ingress namespace=default providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=error msg="service database-system/database-frontend not in the parent resource namespace default" providerName=kubernetescrd ingress=ingressroute-database namespace=default
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=error msg="service example-app/example-app-service not in the parent resource namespace default" namespace=default providerName=kubernetescrd ingress=example-app-ingress
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=debug msg="No secret name provided" providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=error msg="subset not found for tcp-example/filesharing-svc" ingress=ingressroute-filesharing-web namespace=tcp-example providerName=kubernetescrd
time="2021-09-24T08:00:10Z" level=error msg="Cannot create service: tcp service tcp-example/productionapp-svc is not in the parent resource namespace default" serviceName=productionapp-latest-svc namespace=default servicePort="{0 25565 }" providerName=kubernetescrd ingress=productionapp-ingressroute
time="2021-09-24T08:00:10Z" level=error msg="Cannot create service: subset not found" ingress=example-app-ssh serviceName=example-app-service providerName=kubernetescrd servicePort="{0 1080 }" namespace=example-app
time="2021-09-24T08:00:10Z" level=error msg="Cannot create service: tcp service tcp-example/filesharing-svc is not in the parent resource namespace default" providerName=kubernetescrd ingress=book-ingressroute serviceName=filesharing-svc servicePort="{0 2022 }" namespace=default
time="2021-09-24T08:00:10Z" level=debug msg="Skipping Kubernetes event kind *v1.Endpoints" providerName=kubernetescrd

Hello @NickyBoy89,
Thanks for your interest in Traefik!
A change happened in v2.4.10(see the migration guide):

In v2.4.10, the default value for allowCrossNamespace has been changed to false.

It seems your issue might be a configuration problem, thus I will close this issue.
In case I'm wrong, feel free to re-open the issue.


Saving Issue

Due to Keras features, model created as a Subclass of Keras.Model lacks
model.save()

Fix saving during callbacks

Fixed with saving model's weights only


Mark's qe tests for cinder and nova

I've recently run getput, nova-attach and cinder-parallel on hlm007 and am running cinder now. The first three all run in a couple of minutes whereas the cinder test takes closer to an hour and a half.

My tests were designed long before the automation code and as such cinder-tests-all creates its own virtual environment called mjsvenv and even installs its own copy of subunit into it. Part of the reason is the ability to run standalone outside of the automation environment. Something we can certainly discuss and perhaps something to change in the future as opposed to now?

I can certainly change that grep to Passed. I had just done Pas for so long my fingers forgot ;)

Found a big probem in a number of my cinder tests. There are places with on failure, they don't call subunit and that causes the script that loads the DB to blow up. Personally I'd argue this should also be fixed in subunit as handled errors are poor form. Submitted to ardaya-qe-tests and once merged I can try again.
Also found a problem with other tests not doing an rm on the subunit log and so rerunning appended to a log with errors as opposed to a clean one.

Code updated for deploy-qe-tests and merged and tests appear to be working correctly so ready for this code to be approved.

You will need to touch this again to be able to merge as an error with github/travis happened.


Not closing the bottom/top of the highlight on line 144/145

I first noticed the issue while coding in PHP, but I just tested it on plain text and the errors persists.
STR: just put a lot of text in a file and selected/highlight some word on the line 144 and look for problems on the bottom border of the highlight. If everything is okay, check the top border of a selected word on the line 145. See image below:

ER: Bottom border closing the rectangle.

Yea, I've seen this recently. Just had a quick look. It's basically a CSS issue. I need to update the following:
.highlight-selected .region {
      border-radius: 2px;
      box-sizing: content-box;
      padding: 0 1px 0 0;
      margin: -1px 0 0 -1px;
      background-color: transparent;
      border-width: 1px;
      border-style: solid;
    }

to:
.highlight-selected .region {
      // Maybe make this 3px (be the same as find-and-replace package
      border-radius: 2px;
      box-sizing: border-box;
      background-color: transparent;
      border-width: 1px;
      border-style: solid;
    }

Ok. Do you know why it happens at line 144 or 145? :S

Nope. What I've been experiencing locally is more random. Sometimes the bottom of the box or a side (left or right) is sometimes missing. I think it might have been something to do with the new tiling rendering of the lines.

Hmmm, strange. Maybe it was just a coincidence for me then. :)


Just wondering how to use this bot for bestbuy canada

Hello!
Thanks for the code, but I am currently living in Canada and trying to use this bot to grab myself a gpu. I installed the bot and ran it but  I realized that the bot is in default to handle the website for American BB. Is there any change I can make so that the bot will handle Bestbuy.ca or it will be more complex than that?

I'm sorry but the bot doesn't currently support BB Canada.


No way to pass settings?

The eslint-import checker is nice but it needs to be told about my webpack config. However, I don't see how to provide the settings key to eslint.

Yeah, I didn't even know about the settings option until I added the import plugin. Should be supported here though. Pull request welcome as I don't have time right now to add it myself.

I'll give it a shot, any quick pointers on where you think it should go?
Also, since I have your attention :), when I run XO from command line is
seems that a output will be cut off after 20 or so lines. Can't find why,
is that supposed to happen?


input: Better touch experience on Android

e.g. Double tap is very difficult now

This was a bug in a game.


Add option to unpack archive downloaded when using "http".

Describe the problem/challenge you have
When downloading archive file using http I need the archive file to be automatically unpacked.
Describe the solution you'd like
The githubRelease option has unpackArchive for something similar although in that case you need to designate the file to unpack. In the case of http, there is only the one asset file so could get away with a single boolean flag to indicate whether download should be unpacked.
apiVersion: vendir.k14s.io/v1alpha1
kind: Config
directories:
- contents:
  path: src/wrapt
  - http:
      url: https://github.com/GrahamDumpleton/wrapt/archive/refs/heads/develop.tar.gz
      unpackArchive: true
    path: .

Anything else you would like to add:
I am assuming that once unpacked you can still use newRootPath so as to indicate you want the final result to only include files which existed in a subdirectory of the archive.
For example, in the archive above the top level directory it unpacks into is wrapt-develop. I am hoping one can do:
apiVersion: vendir.k14s.io/v1alpha1
kind: Config
directories:
- contents:
  path: src/wrapt
  - http:
      url: https://github.com/GrahamDumpleton/wrapt/archive/refs/heads/develop.tar.gz
      unpackArchive: true
    path: .
  newRootPath: wrapt-develop

so that top level wrapt-develop directory in the archive is eliminated and you get just the files in it.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Thanks @GrahamDumpleton !  it sounds like preserving the execute mode bits could present some stumbling blocks but we'll leave that to the initial implementation exploration.

@GrahamDumpleton Am I missing something with http? I think this is already supported?
    # fetches asset over HTTP (optional)
    http:
      # asset URL (required)
      url:
      # verification checksum (optional)
      sha256: ""
      # specifies name of a secret with basic auth details;
      # secret may include 'username', 'password' keys (optional)
      secretRef:
        # (required)
        name: my-http-auth
      # skip unpacking tar, tgz, and zip files; by default files are unpacked (optional)
      disableUnpack: false

from: https://carvel.dev/vendir/docs/v0.32.0/vendir-spec/
Specifically:
      # skip unpacking tar, tgz, and zip files; by default files are unpacked (optional)
      disableUnpack: false

It works with zip files. I think the problem is it doesn't work with .tar.gz files?

I suspect unpacking for http didn't exist back then and was since added but this issue was never updated. Or perhaps disableUnpack didn't exist and so wasn't obvious that things were unpacked at all. Either way, it does need to handle *.tar.gz if it doesn't already.

Understandable, just wanted to make you aware if you still needed it. I'll look and see if I can figure out why it doesn't support .tar.gz files. And maybe submit a PR for it.

If it helps anyone, this is where the logic is:
And I get this error for my 'tar.gz' file:
2023/08/08 17:13:16 Unknown file 'pax_global_header' (103)

That is tar.TypeXGlobalHeader, which I guess is safe to skip.
Skipping that I got (I'm using sponnet.tar.gz btw for testing):
2023/08/08 17:20:45 Unknown file 'sponnet-1.26.0/' (53)

I guess the problem is that the custom logic looking into the tar.gz file is not aware there are multiple formats, like USTAR, PAX and GNU.
TBQH I think in this switch/case the default case could just use continue and the tar.TypeDir can be handled by the default.

While looking into it I realized that the fix is so trivial that I opened a PR.

Thank you so much for looking into this @reegnz and creating a PR. Do you always get the same error while unpacking tar.gz files or this happens only in certain cases? Also, do you mind creating a new issue with the details, as @knechtionscoding rightly pointed out, the feature to unpack archive files has been there for a long time and it works by default, so we can close this issue.

@praveenrewar opened a separate issue for the fix and linked the PR to that one. I'm strictly focused on the extraction issue, the permissions issue I'll let someone else pick up instead.

Thank you so much @reegnz!


Add light mode to topics and featured section (sidebar).

Context
In /articles:
Add light mode to topics and featured section (sidebar).
They should have a background of white (bg-white).
And a grey border (border border-neutral-300).

This should only be the styles while in light mode.

Hello, I think i can work on this. Can this be assigned to me?


Refetch display variables in  Debug ContractUI

Approach taken :
  const [refreshDisplayVariables, setRefreshDisplayVariables] = useState(false);

created a common state(refreshDisplayVariables) which is shared by WriteOnlyFunctionForm and DisplayVariables component . WriteOnlyFunctionForm changes the state and based on state changes DisplayVariables uses refetch function to refetch the variables again.
Fixes #53
Demo :
https://user-images.githubusercontent.com/80153681/205015942-f6b85592-cd07-407a-965e-c32cfbb03873.mov
Alternate approaches that can be considered :

using watch : true  :
This is a simple solution we can use, its just that each variable will keep making eth_blocknumber calls and its a bit less responsive since will be making eth_blocknumber after 3sec to get new data.
using queryKey(in future if wagmi make it available) :
As mentioned #32 we can use key if wagmi exposes it in future (currently its not exposed even in new version 0.8). wagmi using useQuery internally...check this out

Some useful ongoing discussions on wagmi:

https://github.com/wagmi-dev/wagmi/discussions/1238

Some discussions on refetch which uses the same patterns as wagmi :

https://github.com/apollographql/react-apollo/issues/562

A RPC request is made each time you write in the input. It seems that some state change / re-rendering is causing this. Can you check that out? In this PR or in another one, up to you.

Yes, I think its something to do with usePrepareContractWrite  as mentioned here.
I also found this while tinkering :
The RPC requests are made only if you are connected to the burner wallet but If you connect your own wallet(eg. Metamask) it doesn't seem to make those requests.
demo :
https://user-images.githubusercontent.com/80153681/205861027-d641225c-f6b3-4493-ad13-42e6e018e475.mov
Possible solution :
I tried using useContractWrite without usePrepareContractWrite, by passing mode: 'recklesslyUnprepared' as mentioned in the doc here and it does seem to work.
I would love to tackle it into a separate PR and find any alternatives to this if possible since it is recommended in the docs to pair useContractWrite with usePrepareContractWrite here. Also need to check properly why it is happening only with the burner wallet and not when you connect with your own wallet. But please let me know if I should create a separate PR or tackle it into this PR itself.


layers: describe policy enforcement for foreign layer media type

Signed-off-by: Stephen J Day<EMAIL_ADDRESS>Supersedes #216, Requires #169.

@stephenrwalli Can you take a look at this language? I am still unclear if "MUST NOT distribute" is something we can define usefully in a spec.
I totally get it as a human being with context on copyrights for proprietary software about the usefullness. I just don't know how to codify this. It is like the evil bit RFC: https://www.ietf.org/rfc/rfc3514.txt

-1 on the underspecified use of the "push" verb, for the same reasons captured in https://github.com/opencontainers/image-spec/pull/216#issuecomment-243845627

@jonboulle It is sufficient to comment inline on the specific language that needs correcting rather than saying "-1". It is much more constructive, unless you intend to block the PR.
I've replaced the term "push" with the more generic "distribute".

@philips I understand your example. It's not necessarily how you would codify it, so much as it gives the certification process the language to revoke a certification based on non-compliance. While it might be difficult to write a test to prove the MUST NOT, if an implementation was demonstrated to distribute such a layer, it clearly doesn't do what the specification says, and would not be able to claim conformance/compliance. Likewise, for a certification process, one can design the process to manual test such a thing or to use the condition to revoke a certification until the implementation complies. Make sense?  (Sorry for the tardy reply.)

On Tue, Sep 06, 2016 at 01:58:50PM -0700, Stephen Walli wrote:

While it might be difficult to write a test to prove the MUST NOT,
if an implementation was demonstrated to distribute such a layer, it
clearly doesn't do what the specification says, and would not be
able to claim conformance/compliance. Likewise, for a certification
process, one can design the process to manual test such a thing or
to use the condition to revoke a certification until the
implementation complies.

“Yeah, automatically testing that would be hard, but we can manually
test it” is a marker for a wishy definition 1.  I think the problem
here is that “distributing” is not something that one instance can do
by itself.  If it's pushing a blob, someone else must be receiving it.
In some cases (e.g. pushing to a local disk on a single-user system),
that probably doesn't count as “distributing” in a legal sense.  In
others (e.g. pushing to a public repo on hub.docker.com) it does.
Where does caching to disk on a multi-user system fall in that
continuum?  Are you distributing it by saving to a system-readabale
cache?  Or is it only distributing if another user on the system reads
it out of that cache?  If that system-readable cache happens to live
under your home directory, are you being robbed (and not distributing)
when the other user reads the file?
I imagine there are consensus opinions on at least some of that
(although I'm not aware of them).  I don't think we want to make it
the business of spec implementers and validators to independently
figure out what those opinions are.  If there are conditions under
which you would certify (or not) an implementation, I'd like to see
those conditions spelled out in the spec in enough detail for them to
be objectively tested without appealing to “The average person,
applying local community standards, looking at the work in its
entirety” and such.  In some cases, that wishyness is unavoidable.
But we need something more solid if we're expecting implementation
software to act consistently based on a ‘foreign’ boolean.
One option would be to have a CAS engine API (e.g. #159) where the
public/private-ness of the engine was part of its initialization.
E.g.:
casEngine, err := NewEngine(ctx, path, public)
Then the engine is non-compliant if you can successfully:
casEngine.Put(ctx, reader, mediaType)
with a foreign mediaType (this is testable by the OCI).  And the
engine-initializer is breaking the law if they create a CAS engine
pointing at a public hub.docker.com repo and claiming it is not public
(this is between the CAS engine configurer and the copyright holders
of the pushed content, and not something the OCI has to certify).

cc @runcom @vbatts ?

I'm not sure either on this one - is it an hard requirement for this spec to outline the legal matters of foreign layers? I (sorry I may be wrong) don't think so. I think as foreign layers just layers that are downloaded through a list of urls in a descriptor. Do we really need to clarify that those layers can't be distributed? I understand this was born in Docker for this exact legal reason but I don't think the spec should enforce this restriction (not to push) - if people end up pushing the layer it's them breaking any legal terms, I'm not sure what the spec has to do with this. (I believe it's perfectly possible to upload the Windows base layer by taking it and uploading it, implementers (Docker) enforce this, can we just add an implementor note?)
(sorry if any of the above makes no sense...)

You make good sense, @runcom. So a way to call attention to such things, that I've seen in the past, is to explicitly call something out as implementation defined. This is basically a warning to spec implementers that are not part of the immediate spec community to pay attention in the space before naively doing something in their implementation. It also allows for the behaviors that already exist in implementations. So in this case, wording becomes: "It is implementation defined whether or not implementations distribute layers tagged with this media type."

Such a phrasing would definitely assuage my concerns about the vagueness of "pushing", "distributing", "distributable", etc. (My wariness is largely around combining such terms with the keyword MUST without being very explicit about what they mean.)

@stephenrwalli that sounds perfect to me if people agree

On Wed, Sep 07, 2016 at 10:23:08AM -0700, Stephen Walli wrote:

So in this case, wording becomes: "It is implementation defined
whether or not implementations distribute layers tagged with this
media type."

This works for me too.

On Wed, Sep 07, 2016 at 10:23:08AM -0700, Stephen Walli wrote:

So in this case, wording becomes: "It is implementation defined
whether or not implementations distribute layers tagged with this
media type."

It feels awkard to repeat “implementation”.  How about:
It is unspecified whether or not implementations distribute layers
tagged with this media type.
?

I believe the suggested text becomes:
Foreign Layers
Layers that have these restrictions SHOULD be tagged with an alternative mediatype of `application/vnd.oci.image.layer.foreign.tar+gzip`.
 [Descriptors](descriptor.md) referencing these layers MAY include `urls` for downloading these layers.
It is implementation defined whether or not implementations upload layers tagged with this media type.

@wking I appreciate the awkwardness of language, but I'm using very specific phrasing. "Unspecified" says you can do anything. "Implementation defined" is that bit stronger saying implementations may already do things here and you should pay attention to your choices.

LGTM

On Wed, Sep 07, 2016 at 01:56:09PM -0700, Stephen Walli wrote:

@wking I appreciate the awkwardness of language, but I'm using very
specific phrasing. "Unspecified" says you can do
anything. "Implementation defined" is that bit stronger saying
implementations may already do things here and you should pay
attention to your choices.

Interesting.  It looks like C defines those terms [1,2].  If we are
going to split those hairs (and it sounds like a good idea to me), I'd
recommend we put something like that in our spec or reference another
spec to define them for us.  Ideally the latter.  Is there an
equivalent to RFC 2119 for these terms?
 Page 18 of the PDF, page 11 of the document.

Are we sticking with the current language?

Updated with the new language!

On Wed, Sep 07, 2016 at 02:40:52PM -0700, Stephen Day wrote:

Updated with the new language!

I still think we want to drop the “should never be uploaded” line 1,
and would like to define “implementation defined” 2.  But those can
be addressed in follow-up work if they're too contentious to handle
now, and the rest of fbc6ce4d looks good to me.

@wking The use of "should" in that sentence applies to generalizing the use case scenarios and does not apply to the implementations of the specification.

On Wed, Sep 07, 2016 at 03:45:22PM -0700, Stephen Day wrote:

@wking The use of "should" in that sentence applies to generalizing
the use case scenarios and does not apply to the implementations of
the specification.

azurestandard/beehive#536 did not touch any shoulds applying to
implementations either.  And if we aren't clear on what the line is
adding that's not covered somewhere else in that section 1, I don't
see a reason to add it.

LGTM

@opencontainers/image-spec-maintainers please review and LGTM

fbc6ce4 → edddc0a9 keeps the never-uploaded line, which addresses the
“should” issue 1 without dropping the line.  I'd still rather drop
the line, but am ok with the current phrasing.
And I'd still like to define “implementation defined” 2, but can
address that in follow up work.
So edddc0a9 looks good enough to me.

Pushed e29e122 to deal with whitespace CI errors.

On Thu, Sep 08, 2016 at 12:45:32PM -0700, Stephen Day wrote:

Pushed e29e122 to deal with whitespace CI errors.

This looks good enough to me too [1].

LGTM

after sleeping on this - I really don't like the term foreign - at best it's non-obvious what it means and at worst I think it implies the layer is from a "remote"/unknown source. can someone explain it to me (and spec readers generally) in a way that's more obvious? otherwise, how about any of local, internal, private, nondistributable, unpublishable, ...

On Fri, Sep 09, 2016 at 12:39:32AM -0700, Jonathan Boulle wrote:

… nondistributable, …

I like this one, and it echos the phrasing we're adding in the lead-in
line here 1.

@jonboulle The term "foreign" came out of the notion that these layers are foreign to the primary distribution process. Originally, these were tightly coupled with the urls field on layer descriptors. Once this coupling was removed, their role as "foreign layers" was more about name, as it is really a layer that you can fetch by url and cannot distribute. Neither component of this feature is "foreign" and it only becomes so when used together. It is probably not the best term (I would favor nondistributable), but it is already in place in docker.
If you think bike shedding is important, we can change it, but I am not sure what the value that provides.

On Fri, Sep 09, 2016 at 01:03:24PM -0700, Stephen Day wrote:

It is probably not the best term (I would favor nondistributable),
but it is already in place in docker.

All of our media types are already different from Docker, so I don't
see a backwards-compat issue here.  We're just tweaking a key/value
pair in the media type compatibility map.

If you think bike shedding is important, we can change it, but I am
not sure what the value that provides.

More expressive naming will hopefully leave fewer people scratching
their heads wondering what ‘foreign’ means ;).

@stevvooe c'mon, this is not about bike-shedding - I want progress just as much as anyone else. But I don't understand why we should stick to nonobvious language when we're already changing media types. The value it provides is clear: lowering the cognitive burden to people approaching the spec.

@jonboulle I fully agree with the goal of reducing cognitive burden but consistency is also a valid goal. Docker calls this application/vnd.docker.image.rootfs.foreign.diff.tar.gzip and the concepts are isomorphic. Do you think these having different names helps to reduce cognitive burden?
How about we get this merged based on content and concept do another PR where we discuss the name of the media type? Would that suffice?

On Fri, Sep 09, 2016 at 02:07:49PM -0700, Stephen Day wrote:

Do you think these having different names helps to reduce cognitive
burden?

When the new OCI name is more clear?  Yes.  If we didn't think so, we
wouldn't have landed #212.

How about we get this merged based on content and concept do another
PR where we discuss the name of the media type? Would that suffice?

I don't see how splitting this discussion out into a new PR will help
resolve it (there's not a lot of other distracting stuff happing in
this one), but if folks feel like that's the case, I'm happy to
discuss foreign → nondistributable in a follow-up PR.

When the new OCI name is more clear?  Yes.  If we didn't think so, we
wouldn't have landed #212.

#212 actually brings it closer to https://github.com/docker/distribution/blob/master/docs/spec/manifest-v2-2.md#media-types. Before, we had an extra level of indirection in OCI.
I'll say that I am as much of a stickler for naming as anyone, but I'm really just trying to move things forward. I think we have agreement on the concept, but I don't want to get it held up on naming. Let's get the concept in. If we agree on a name change, then we can merge that, as well.

On Fri, Sep 09, 2016 at 02:39:34PM -0700, Stephen Day wrote:

Let's get the concept in. If we agree on a name change, then we can
merge that, as well.

Fair.  I'm just not sure why we don't already have an agreement on
changing the name ;).

I still have the concerns from 1, which I'm ok ignoring or punting
on, but otherwise e48e595f looks good to me.

LGTM now, thanks @stevvooe
I do agree with wking's comments in https://github.com/opencontainers/image-spec/pull/233#issuecomment-245667367 but we can follow up for that.

LGTM

🎊

:tada:


[BUG] Unable to deserialize GetActivityLogs() : System.FormatException: One of the identified items was in an invalid format at System.Text.Json.JsonElement.GetGuid()

Library name and version
Azure.ResourceManager.Monitor 1.0.0
Describe the bug
When calling SubscriptionResource.GetActivityLogs().ToList() with a basic filter (eventTimestamp) and no select, I receive the following error:
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.FormatException: One of the identified items was in an invalid format.
   at System.Text.Json.JsonElement.GetGuid()
   at Azure.ResourceManager.Monitor.Models.EventDataInfo.DeserializeEventDataInfo(JsonElement element)
   at Azure.ResourceManager.Monitor.Models.EventDataCollection.DeserializeEventDataCollection(JsonElement element)
   at Azure.ResourceManager.Monitor.ActivityLogsRestOperations.List(String subscriptionId, String filter, String select, CancellationToken cancellationToken)
   at Azure.ResourceManager.Monitor.SubscriptionResourceExtensionClient.<>c__DisplayClass75_0.<GetActivityLogs>g__FirstPageFunc|0(Nullable`1 pageSizeHint)
   at Azure.Core.PageableHelpers.FuncPageable`1.<AsPages>d__4.MoveNext()
   at Azure.Pageable`1.<GetEnumerator>d__8.MoveNext()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)

I don't have access to the json source anymore, but I will try to
Expected behavior
The json should be deserialized.
Actual behavior
The methods throws an exception
Reproduction Steps
SubscriptionResource subscription;
// ...
var date = DateTime.UtcNow;
var resourceId = "...";
var filter = $"eventTimestamp ge '{date.AddHours(-hours):o}' and eventTimestamp le {date:o} and resourceUri eq '{resourceId}'";
var logs = subscription.GetActivityLogs(filter).ToList();

Environment
.NET Framework 4.8

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

you can enable source json in log by set options of new ArmClient as below
var options = new ArmClientOptions()
options.Diagnostics.IsLoggingContentEnabled=true;
               options.Diagnostics.IsLoggingEnabled=true;
               var client = new ArmClient(new DefaultAzureCredential(),options);

kindly share your raw json so that we can understand which field is triggering this issue

The error is thrown in EventDataInfo.DeserializeEventDataInfo(). The only Guid in EventDataInfo is TenantId. Please help get the raw json response as @xboxeer mentioned and check if any tenantId is not a valid Guid/uuid.

I am sorry but I am unable to reproduce the problem for now. I ran the exact same filter on a custom console app and I did not raise any exception.
I will try again on monday, and if I can't reproduce it I will close this issue.

@fengzhou-msft I was able to dump a full json dump. Can you provide me an email ?

I have simply detected cases where the tenantId is empty (I have edited the sensitive ids):
{
    "channels": "Admin, Operation",
    "correlationId": "3e84620f-38cd-4d5b-8b55-965356e1f74b",
    "description": "",
    "eventDataId": "fa74d33c-34ac-4b6c-9144-bdbe044a7ed9",
    "eventName": {
        "value": "",
        "localizedValue": ""
    },
    "category": {
        "value": "ResourceHealth",
        "localizedValue": "Resource Health"
    },
    "id": "/SUBSCRIPTIONS/00000000-0000-4E34-0000-00008AF10000/RESOURCEGROUPS/MYRESOURCEGROUP/PROVIDERS/MICROSOFT.SQL/SERVERS/MYSQLSERVER/DATABASES/MYDATABASE/events/fa74d33c-34ac-4b6c-9144-bdbe044a7ed9/ticks/638106425059531945",
    "level": "Informational",
    "resourceGroupName": "MYRESOURCEGROUP",
    "resourceProviderName": {
        "value": "Microsoft.Resourcehealth/healthevent/action",
        "localizedValue": "Microsoft.Resourcehealth/healthevent/action"
    },
    "resourceId": "/SUBSCRIPTIONS/00000000-0000-4E34-0000-00008AF10000/RESOURCEGROUPS/MYRESOURCEGROUP/PROVIDERS/MICROSOFT.SQL/SERVERS/MYSQLSERVER/DATABASES/MYDATABASE",
    "resourceType": {
        "value": "MICROSOFT.SQL/SERVERS/DATABASES",
        "localizedValue": "MICROSOFT.SQL/SERVERS/DATABASES"
    },
    "operationId": "",
    "operationName": {
        "value": "Microsoft.Resourcehealth/healthevent/Resolved/action",
        "localizedValue": "Health Event Resolved"
    },
    "properties": {
        "title": "Unknown",
        "details": "Unknown",
        "currentHealthStatus": "Available",
        "previousHealthStatus": "Degraded",
        "type": "Unknown",
        "cause": "Unknown"
    },
    "status": {
        "value": "Resolved",
        "localizedValue": "Resolved"
    },
    "subStatus": {
        "value": "",
        "localizedValue": ""
    },
    "eventTimestamp": "2023-01-30T02:28:25.9531945Z",
    "submissionTimestamp": "2023-01-30T02:28:25.9531945Z",
    "subscriptionId": "00000000-0000-4E34-0000-00008AF10000",
    "tenantId": ""
}

@tbolon thanks for the investigation. As expected, the non-guid tenantId caused the deserialization issue. We will work with service team on a fix.

@sameergMS @dadunl in the returned ActivityLogs, the tenantId property may be an empty string. .NET SDK treats TenantId as Guid type. Therefore it cannot store the value of an empty string. Does it make sense and won't cause any problems if we treat empty string tenantId as null in our deserialization?

Hi, still get the same error in the 1.3.0 version.

Hi, here's a bug here. At your EventDataInfo.Serialization.cs, a empty string doesn't mean the null kind here.

Demo with a small code snippet.
.

Why filter on level is not supported? As azure portal can do that

The same issue occurred in #43151 and has been fixed.


docs: Add type annotation.

Discovered while running the 2to3 type annotation.

We might as well just add these in Python 3 format?

Right, will amend.

git commit ---amended

Merged, thanks @rht!


Create form to collect feedback/testimonials

Is your feature request related to a problem?
Currently there is no way to collect testimonials/feedback besides an email
Describe the solution you'd like
A form to collect testimonials/feedback
Describe alternatives you've considered
This can be a google form, collect through newsletter
Additional context

I haven't used google forms all that much but I saw that there should be a way to embed it into the website. So maybe that can be an option. Or just linking to the form on the website as well to make it easy lol.

@fvntr @WilliamMena This would be best as a Netlify form because of the integrations - related issues are #153 #95

Let's make sure to get the form fields/design set up too so it's easier for assigned contributor to work on this once we setup our first Netlify form 😄

will work on this and #54 in parralel


Tern seems to be hanging jsbin

Hello,
when using this bin https://jsbin.com/dudeko/edit?html,js,output
I often get awful, few sec. freezes especially right after parens or braces get highlighted (when I place cursor at the closing paren or when I click on the string.
I am using chrome and while running profiler it appears that tern is to blame,
see the cpu profiler dump: https://dl.dropboxusercontent.com/u/19020828/CPU-20160215T194719.cpuprofile


*: Remove unused dependency

What is changed and how it works?

Issue Number: Ref https://github.com/tikv/tikv/issues/17465

What's Changed:
cargo install cargo-machete && cargo machete
Remove unused dependency

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

None

Is there any tool that detects unused dependencies and can be added to CI?

Is there any tool that detects unused dependencies and can be added to CI?

I use cargo machete, but it has a lot of false positives for macro imports, so just check it manually.

/approve


Status Code strings

Currently the status code header on the response object from the server only includes the status code number and not the string. This does not follow the HTTP spec.
Returns:
HTTP/1.1 200

Should return
HTTP/1.1 200 OK

Is there currently anyway to add the OK string on the response?

Nevermind, I see the reason arg now.

It does follow the http spec -- "reason phrases" are arbitrary, meaningless
strings separated from the status code by a space character. We just happen
to use the empty string by default. (So the status line you pasted has a
space character between the 200 and the newline -- this is crucial. Without
it we would indeed be violating the spec.)
Like you discovered, it is possible to set the reason explicitly. Also, I'd
accept a patch that set the default reason based on the status using the
table in Python's built-in http package.
While we're here, I'd love to hear about what you're working on that
involves h11 -- especially if there are things that annoy you :-). The
server side API definitely would benefit from more real world revisited.
And I'd recommend subscribing to issue #17.

I'd accept a patch that set the default reason based on the status using the table in Python's built-in http package

I like this idea. I will work on this.
I am building an microservice async framework that works with asycio/curio.

if there are things that annoy you
I mostly finished the http portion, so if there was anything, I dont remember now 😄 but ill let you know if I find anything.

Thanks.

I am building an microservice async framework that works with asycio/curio.

Neat! Is it open source?
You'll still want to subscribe to #17 if you haven't -- there will probably be a few hopefully-small-but-still breaking changes in h11 ahead. (In particular I think we need to do something better about handling request targets.)

https://github.com/wickedasp/wasp
Curio support will probably happen later once curio is out of alpha.


Update various repository references across the project to be in sync…

… with the new repository location.
JerryScript-DCO-1.0-Signed-off-by: Tilmann Scheller<EMAIL_ADDRESS>

LGTM

LGTM


fix: fix auto bundling

@Vid201  Current design which allows changing the mode in the runtime is because of the debug_bundler_setBundlingMode required for the spec tests. I think it makes sense to specify it when we start the bundler.


Disqus not working

Hi：
Love Minimal-mistakes，Awesome theme for jekyll！
I have some trouble with setting up comment in my site, and I have search all kinds of method, but still get it work. So ask for some help...

Minimal Mistakes version: 4.4.1
github-pages or jekyll gem version: jekyll 3.4.3
Operating system:  Mac 10.12.4

Expected behavior
Disqus show in my the footer of the post
Steps to reproduce the behavior
I have try all the method in the troubleshooting guide for "disqus do not show" including:

check shortname
delete disqus account and create a new account
recreate a new jekyll site

but comment still do not show.
there is my config setting:
comments:
  provider               : “disqus”
  disqus:
    shortname            : banpie

# Defaults
defaults:
  # _posts
  - scope:
      path: ""
      type: posts
    values:
      layout: single
      author_profile: true
      read_time: true
      comments: true
      share: true
      related: true

My site is ： banpie.github.io, here is the post: https://banpie.github.io/college
Thanks ！

Your _config.yml looks correct but when I inspect the page source the Disqus script isn't there. Is your repo public so that I can look at it?
YAML can do buggy things if your formatting is off a space or two, so it's possible there's some formatting screwing things up.
How about the /college post. What do you have in it's YAML Front Matter? Have you tried adding comments: true there to see if that works?

Thanks for your prompt reply.
I have tried add comments: true & layout: singlein post, but neither work.
Here is my repo: https://github.com/banpie/banpie.github.io

Figured out your problem. You're using smart quotes around the disqus key in your _config.yml which breaks the logic of the LIquid conditional that adds the Disqus script.
comments:
  provider               : “disqus”

It should be this
comments:
  provider               : "disqus"

Or you can even remove the quotes all together provider: disqus.

Wow... seems due to the chinese charater of “”。
Thanks a lot！


MATH: Analyze 3342

Findings can be reported at https://leanprover.zulipchat.com/#narrow/channel/458659-Equational/topic/Outstanding.20equations.2C.20v1

This is a relatively simple equation, however it is known that pure translation-invariant approaches do not work; see above link.

claim

propose PR #682


utf8.encodeBuffer adds null byte to nonempty string

import ceylon.buffer { ByteBuffer }
import ceylon.buffer.charset { utf8 }

shared void run() {
    String string = "-";
    ByteBuffer byteBuffer = utf8.encodeBuffer(string);
    Array<Byte> byteArray = byteBuffer.array;
    print(byteArray);
    ByteBuffer byteBuffer_ = ByteBuffer.ofArray(byteArray);
    String string_ = utf8.decode(byteBuffer_);
    for (char in string_) { print(char.integer); }
    print(string == string_);
}

The byte array contains two entries: 45 (the dash), and 0. When decoding the string back, the 0 byte is preserved and also printed in the loop; for this reason, the decoded string is not equal to the original one, and the final line prints false.
It gets even worse if the original string contains null bytes – "\{#0000}\{#0000}" is encoded to four null bytes, "\{#0000}\{#0000}\{#0000}" to six.
Pinging @FroMage, who wrote utf8 according to the by annotation.

 The correct version is byteBuffer.array.measure(byteBuffer.position, byteBuffer.available).


Clear cache when clone failed

Signed-off-by: shiyu22<EMAIL_ADDRESS>fixed: #1387

Codecov Report

Merging #1416 (f2b65b3) into main (15e0068) will decrease coverage by 0.40%.
The diff coverage is 37.50%.

@@            Coverage Diff             @@
##             main    #1416      +/-   ##
==========================================
- Coverage   68.78%   68.37%   -0.41%
==========================================
  Files         295      296       +1
  Lines       16228    16276      +48
  Branches     2633     2648      +15
==========================================
- Hits        11162    11129      -33
- Misses       4320     4408      +88
+ Partials      746      739       -7

Impacted Files
Coverage Δ

towhee/utils/git_utils.py
51.89% <37.50%> (-0.88%)
:arrow_down:

towhee/models/clip/clip_utils.py
32.52% <0.00%> (-39.84%)
:arrow_down:

towhee/models/clip/clip.py
85.43% <0.00%> (-3.32%)
:arrow_down:

towhee/hub/repo_manager.py
23.96% <0.00%> (-0.82%)
:arrow_down:

towhee/models/drl/drl.py
18.07% <0.00%> (-0.62%)
:arrow_down:

towhee/models/drl/__init__.py
100.00% <0.00%> (ø)

towhee/functional/storages.py
82.14% <0.00%> (+2.38%)
:arrow_up:

towhee/functional/mixins/column.py
75.00% <0.00%> (+9.00%)
:arrow_up:

towhee/engine/execution/vectorized_execution.py
85.71% <0.00%> (+35.71%)
:arrow_up:

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 15e0068...f2b65b3. Read the comment docs.

/lgtm

/lgtm
/approve


Improve whitespace handling

Added more test cases for the ignoreWhitespace=false option and fixes to handle these cases.
@twsouthwick don't know what you prefer. If you want to remove the changes done in https://github.com/OfficeDev/Open-XML-SDK/pull/857 or if you would like to fix the introduced issues. I wasn't aware that changing the xml reader settings will break the internal state handling of the OpenXMLReader

Is this a new issue uncovered by the change or introduced by the change?

Is this a new issue uncovered by the change or introduced by the change?

Is this a new issue uncovered by the change or introduced by the change?

I think it is an issue intorduced by the change. With the IgnoreWhitespace set to false new nodes of type Whitespace are returned by the underlying xml reader which wasn't the case before

Is this a new issue uncovered by the change or introduced by the change?

I think it is an issue intorduced by the change. With the IgnoreWhitespace set to false new nodes of type Whitespace are returned by the underlying xml reader which wasn't the case before

That makes sense, but was there any way to expose it before? Just trying to understand if it's only an issue that exists now due to the option to ignore whitespace.

That makes sense, but was there any way to expose it before? Just trying to understand if it's only an issue that exists now due to the option to ignore whitespace.

I am not sure.

That makes sense, but was there any way to expose it before? Just trying to understand if it's only an issue that exists now due to the option to ignore whitespace.

I think the following example would have caused the same error without the whitespace change:
 <w:document xmlns:v=""urn:schemas-microsoft-com:vml"" xmlns:w=""http://schemas.openxmlformats.org/wordprocessingml/2006/main"">
      <w:body>
        <w:p>
        </w:p>
      </w:body>
    </w:document> <!--Comment-->";

I am not sure.

That makes sense, but was there any way to expose it before? Just trying to understand if it's only an issue that exists now due to the option to ignore whitespace.

I think the following example would have caused the same error without the whitespace change:
 <w:document xmlns:v=""urn:schemas-microsoft-com:vml"" xmlns:w=""http://schemas.openxmlformats.org/wordprocessingml/2006/main"">
      <w:body>
        <w:p>
        </w:p>
      </w:body>
    </w:document> <!--Comment-->";


A more elaborate Contributing page

I basically took some of the ideas of the thrown around here https://github.com/opensourcedesign/organization/issues/54
and here https://github.com/opensourcedesign/organization/issues/53

Thanks for the once-over @RichardLitt! Fixed your suggestions.

LGTM.


Remove extra character from table

Removing an extra character outside of the link in the Learn More table

@rlrossiter - thanks!


Is this project dead?

Because it looks like it is

pretty much but still works like a charm

Are there ay takes on forking it and running with it?