Corrupted composer is installed using v1

Describe the bug
Things where working fine yesterday in the afternoon. But workflows running yesterday evening (19:35 UTC+2) installed a corrupted version of composer (without the setup-php step reporting a failure, so the failure was only reported by a later step using composer)
Version

[ ] I have checked releases and the bug exists in the latest patch version of v1 or v2.
[ ] v2
[x] v1

I haven't checked v2
Runners

[x] GitHub Hosted
[ ] Self Hosted

Operating systems
linux
PHP versions
7.3
To Reproduce
jobs:
  phpstan:
    name: PHPStan
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v2
      - name: Setup PHP
        uses: shivammathur/setup-php@v1
        with:
          php-version: 7.3
          extensions: intl
          coverage: none
      # more steps following, where using composer would throw the same fatal error and break the build

Expected behavior
A usable version of composer is installed.
And another expectation would be that such fatal error actually makes the setup report a failure rather than failing silently.
Screenshots/Logs
Here is the output of the setup-php step:
 Setup PHP
24s
    coverage: none
Run shivammathur/setup-php@v1
  with:
    php-version: 7.3
    extensions: intl
    coverage: none
/bin/bash /opt/hostedtoolcache/linux.sh 7.3 /home/runner/work/_actions/shivammathur/setup-php/v1/dist

==> Setup PHP
✓ PHP Switched to PHP 7.3.19

==> Setup Tools
PHP Fatal error:  Uncaught PharException: internal corruption of phar "/usr/local/bin/composer" (truncated manifest header) in /usr/local/bin/composer:23
Stack trace:
#0 /usr/local/bin/composer(23): Phar::mapPhar('composer.phar')
#1 {main}
  thrown in /usr/local/bin/composer on line 23
✓ composer Added

==> Setup Extensions
✓ intl Enabled

==> Setup Coverage
✓ none Disabled Xdebug and PCOV

Additional context

Are you willing to submit a PR?

@stof This is fixed now upstream, it was an issue with their infra that composer phars which were fetched had old signatures.
In composer there is a check for this which was triggering the error. Adding the same check after composer setup would be repeating it and would add to setup time, in some configurations double it.

@shivammathur
I had just the same happen to me too:

Can't there be some exit code check added to break this GHA step if the setup couldn't finish properly?

@mfn Added, will be there in next release.

Wow, I saw the commit https://github.com/shivammathur/setup-php/commit/82a0dd85ffddf094b533e7b94b31240131556347 . So much work what I thought would be just an exit somewhere.
Much appreciated 🙇

Also improved how different versions of composer are setup, so most of 82a0dd8 is that.

@mfn @stof After 2.4.0 release if composer does not download correctly, setup will fail with the message "Could not download composer".

Awesome, will try it out!
😼


Provide linux and macOS binary

It would be great to add github flow which updates at least two kinds of binary file for Linux and macOS users after pushing into master.
Additional workflow for the same process would be useful for each release of SurfGen.
After such improvements it would be easier to use SurfGen and no one would be required to build from source (could be helpful in this case too https://github.com/surfstudio/SurfGen/issues/73). Also it would be easier to contribute, since the flow will update the binary automatically, and the most actual binary file will be stored in the main branch, so it will make updates easier for current users of SurfGen.

Look at 2.0.0 release


[Feat]: Reviewed on live stream https://youtu.be/MrS_jAJ8l0o

Reviewed on live stream https://youtu.be/MrS_jAJ8l0o
Looks great 👍

[ ] #102
[x] light mode as well as dark mode
[ ] #103

Originally posted by @eddiejaoude in https://github.com/EddieHubCommunity/support/issues/851#issuecomment-822603036

Closing due issues will be tracked in the other issues


lime3ds: init at 2114

Description of changes
The emulator is working, I'm playing Pokemon X in it.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3998

just tested this - seems to build and work :)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1730

Result of nixpkgs-review pr 313821 run on x86_64-linux 1

1 package built:

lime3ds


Prevent multiple setInterval from being created

Intervals should be cleared before creating another one when logging a page view.

'Interval' is the interval for page pings.


chore: remove unused cases' names

These unused names will soon acquire a linter warning. Easiest to clean them up pre-emptively.
(Thanks to @nomeata for fixing these on nightly-testing.)

bors merge


[feature request] Web Users/Login

Would be nice if you could setup a login with password for getting into the WebUI to keep unwanted users out of the config. Possibly maybe even different users down the line with different permissions.

Add me to the list of users wishing this.
A regular user and admin user would be all I would require.
Regular user only has access to the devices screen with the edit/copy & delete buttons hidden

So, I am going put this in has the whole UI security. There will now be a button to register a device just like a real hue and the UI will be locked if this is used by password. There will also, be the option to lock the REST API by password as well.

Yes!!!
Thanks, I can't wait!


Parse error on amazon pdf file url

I'm getting this error when try to render pdf file in angularjs app
Error: [$parse:syntax] Syntax Error: Token ':' is an unexpected token at column 5 of the expression [http://pucdocket.s3.amazonaws.com/VA/PUE-2010-00039/122913.pdf] starting at [://pucdocket.s3.amazonaws.com/VA/PUE-2010-00039/122913.pdf].
Here is pdf directive:
<pdf-viewer                                     delegate-handle="my-pdf-container"                                     url="http://pucdocket.s3.amazonaws.com/VA/PUE-2010-00039/122913.pdf"                                     scale="1"                                     show-toolbar="true"                                     ></pdf-viewer>

<pdf-viewer delegate-handle="my-pdf-container"
                    url="'http://pucdocket.s3.amazonaws.com/VA/PUE-2010-00039/122913.pdf'"
                    scale="1"
                    show-toolbar="true" ></pdf-viewer>

Since, the url can be variable it is parsed within the directive. You need to wrap it with single quotes too. " '... ' "


Test and bugfix cli app logs, with spec tests

Some bugfix work on kontena app logs to fix some issues, with spec tests and refactoring:

Add spec tests using a mock client implementation

Refactor into separate tail_logs(), show_logs(), get_logs() and show_log() for easier testing

Change --tail option to be an Integer

Fix the incorrect use of ?limit= even when tailing logs with ?from=
This would lead to skipped log lines if any service was logging more than --limit lines per the 2 second loop iteration

Track per-service ?from= offset when tailing
This addresses a potential race condition where new service logs appear while looping through multiple services/..../container_logs requests for different services. Imagine the following scenario:

Service A has log messages 1...3
Service B has log messages 4..5
We request service A log messages -> 1..3
Service A gets a log message 6
Service B gets a log message 7
We request service B log messages -> 4..5,7
We enter the next loop iteration with last_id=7
Service A gets a log message 8
We request Service A logs with ?from=7 -> 8

We have lost the Service A log message 6.

Finished in 0.01271 seconds (files took 0.30931 seconds to load)
1 example, 0 failures

If the commits look sane, I'll do a bit more integration testing against the server to make sure this works correctly.

And naturally you only realize it once you've submitted the PR, but the kontena app logs should probably just be reimplemented on top of v1/grids/:name/container_logs?service=.... It would be less complicated and probably perform better, and all of the cli logs commands could use the same buffered_log_json implementation.
I'll update this PR with that approach, unless there's some particular reason to stick with this. I suppose it would change the behavior of --tail to limit the total number of lines, rather than per-service but... yeah :)

That --tail change sounds initally bad, I'm not sure if I got it correctly..
Would service logs --tail N give you N lines of logs or not?

On 07 Sep 2016, at 19:36, Tero Marttila<EMAIL_ADDRESS>wrote:
And naturally you only realize it once you've submitted the PR, but the kontena app logs should probably just be reimplemented on top of v1/grids/:name/container_logs?service=.... It would be less complicated and probably perform better, and all of the cli logs commands could use the same buffered_log_json implementation.
I'll update this PR with that approach, unless there's some particular reason to stick with this. I suppose it would change the behavior of --tail to limit the total number of lines, rather than per-service but... yeah :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Good clean code! But I agree that it would be better to implement this on top of streaming server side api... it accepts multiple services as parameters so implementation should be pretty straightforward.

That --tail change sounds initally bad, I'm not sure if I got it correctly.. Would service logs --tail N give you N lines of logs or not?

Implementing kontena app logs --tail on top of v1/grids/:name/container_logs?follow=1&limit=100&service=.... would give up to 100 lines of the most recent logs across all services combined.
The current kontena app logs --tail gives up to 100 lines of logs per service, i.e. there may be more than 100 lines of output.
I doubt it's a behavior that's worth preserving, and the current behavior is probably somewhat confusing, as you would get a long history of output from quiet services, intermingled with only very recent output from chatty services.

Agreed

On 07 Sep 2016, at 19:46, Tero Marttila<EMAIL_ADDRESS>wrote:
That --tail change sounds initally bad, I'm not sure if I got it correctly.. Would service logs --tail N give you N lines of logs or not?
Implementing kontena app logs --tail on top of v1/grids/:name/container_logs?follow=1&limit=100&service=.... would give up to 100 lines of the most recent logs across all services combined.
The current kontena app logs --tail gives up to 100 lines of logs per service, i.e. there may be more than 100 lines of output.
I doubt it's a behavior that's worth preserving, and the current behavior is probably somewhat confusing, as you would get a long history of output from quiet services, intermingled with only very recent output from chatty services.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Replaced with  #987

Note that the fix for the ?limit= dropping log mesages here is presumably invalid, because omitting the ?limit= parameter will simply use the default limit = 100 on the server api...


BOM-1045 Django22 Support

Missed some changes in previous pr. This fixes those issues.

@robrap I briefly covered the different options for replacement libraries in the ticket description, but didn't check with Nimisha at the time (probably should have done so).  This option seemed to be the most actively supported and easiest to switch to (and I did mention conditions under which the decision should be revisited and escalated).  Note that djangorestframework-simplejwt doesn't support DRF 3.7, so we'd need to upgrade that first if going that route (although we plan to do that upgrade soon anyway).

Thanks @jmbowman.  Sorry I didn't go back to the ticket.  I was just looking for documentation of the decision.  Thank you.  I don't think @nasthagiri needs to be a blocker on this PR, but good to keep her informed about the future options.  Thanks again.

@jmbowman Thanks for driving these changes and proposing a path forward, with a possible backup option.  What is the plan for testing this initial option?  Do you plan to update edx-platform to this new version of edx-drf-extensions and catching issues via existing automated tests?
@Ayub-Khan Thanks for making these changes.
@robrap Thanks for tagging me and keeping me in the loop.

@nasthagiri Yes, the original ticket description requested testing the changes in edx-platform before merging the changes.  If you don't think that'll be sufficient to catch likely problems, let me know so we can come up with a more detailed testing plan (and figure out how to automate it in the future).

@jmbowman Yes, I do believe there are existing end-to-end tests in edx-platform that test the use of JWTAuthentication on certain API endpoints, so we should be good.
As a fail-safe (to catch other JWT Authentication usage), we can manually test a new MFE (like the Profile or Payment page) on Stage.

@jmbowman @nasthagiri @robrap It seems a lot of tests are failing https://build.testeng.edx.org/job/edx-platform-python-pipeline-pr/12210/
looking into those.

https://github.com/edx/edx-platform/pull/22435

We decided to split the requirement changes and jwt changes in multiple prs. Pr for requirements and other non-related stuff was merged. Now closing this in favor of https://github.com/edx/edx-drf-extensions/pull/109.


[Diagnostics]-Samples in generated docs are outdated with respect to Snippets.

The code samples shown for Diagnostics.AspNetCore are not the ones we are currently running as Snippets.
The code samples shown for Diagnostics.AspNet are not the ones we are currently running as Snippets.

This has now been fixed with the release of Diagnostics.* 3.0.0-beta07


Update to Angular 18 + Update all dependencies

Resolves https://github.com/kuhnroyal/mat-datetimepicker/issues/555

@Talb2005 thanks for your help. I've squashed your commits and added the BREAKING CHANGE footer which is required for conventional commits to work.

released in https://github.com/kuhnroyal/mat-datetimepicker/releases/tag/v14.0.0.


WIP Minor fixes and enhancements on envvars & settings

What is the purpose of this Pull Request?
While working on #360 I found minor room for improvements, the commits are self-explanatory (feel free to squash them on merge).
What was done to achieve this purpose?

I've found some vars missing from the sample .env so I added them (7beafaa)
I found the first-block of the sample .env too big to and re-sorted it (8fe86f2)
While exploring some performance issues for (#360) I missed the Django Debug Tool bar, so I configured it properly for Docker environments using a dynamic INTERNAL_IPS alternative (96e7574)
The linter has pointed out a tab invading our settings.py and I squashed it using 4 spaces instead (6db1ab8)

How to test if it really works?
I would recommend three simple things:

If there's no typo or any absurdity in a commit-per-commit review, we're probably good to go
If docker-compose up still bringing the project to life I've probably not broken anything
Finally, in Travis CI we trust.

Who can help reviewing it?
@anaschwendler @cabral @Irio @jtemporal
TODO

In a test, depending on the order of execution (which is basically unpredictable), a test is failing in the CI (but passing locally) – so I'm trying to fix it

To test this PR:

Checked out to @cuducos branch:

$ git fetch origin
$ git checkout -b cuducos-minor-edits origin/cuducos-minor-edits
$ git merge master

Ran docker-compose up and checked if everything was working:

And it was already :)

@anaschwendler – I'm reverting this directly on master: This was titled as a WIP, with a pending TODO. I'm still working on it.


Update network-storage interface to support runners and pass daos scripts during startup

Currently, network-storage interface doesn't support client-install and mount runner scripts and we don't need that for natively supported file-systems and also these file-systems are mounted using fstab. For parallelstore we need to pass client-install script and mount scripts separately since it needs to install drivers and it uses dfuse to mount container.
This PR adds support for client-install and mount-install runner in network-storage interface and only run the scripts for DAOS file-system.
For,

DAOS file system Update startup scripts to run client-install and mount runners and don't update config.yaml file.

Submission Checklist
Please take the following actions before submitting this pull request.

Fork your PR branch from the Toolkit "develop" branch (not main)
Test all changes with pre-commit in a local branch #
Confirm that "make tests" passes all tests
Add or modify unit tests to cover code changes
Ensure that unit test coverage remains above 80%
Update all applicable documentation
Follow Cloud HPC Toolkit Contribution guidelines #

LGTMed By Ivan.


Mono AOT documentation

It'd be nice to have a location to document Mono-specific information and AOT-related topics more generally. The motivation for this issue is this: https://github.com/dotnet/runtime/issues/49805; the Mono runtime currently contains an error message that documents a Mono AOT limitation under a Xamarin iOS category.
I'm not sure where to put this documentation right now. A quick grep -HRni aot **/*.md in this repo has 20 hits, with no clear overarching pattern or category.

@davidbritch Do you have suggestions on this?


Dyson Heurist - No Camera Entities

Core Version
core-2021.4.0
Supervisor Version
supervisor-2021.03.9
ha-dyson-cloud v 0.10.0
When using a Dyson Heurist, no camera entities are created - which I believe should show a map?
In the code, I note only a reference to DysonCloud360Eye - is the Heurist currently unsupported for this?

I added 360 Heurist support in version 0.10.1. Can you test if it works? Thanks.

Many thanks - I now have a camera entity, but the image/picture is broken (does not display).
Should it fetch the last?  Or will it pull next time the robot pushes an update, perhaps?  It's currently charging, waiting to resume shortly.

It should be able to fetch the last image. Maybe its API is different from that of 360 Eye. Then it's going to be very tricky to find out.

Not a problem - thanks for taking a look anyway.

I have no idea how I brought it up - I've tried a few things to do it again, I was carrying my phone unlocked so perhaps some kind of long-press, and it popped up - but it seems the iOS Dyson App has a (hidden?) 'Requests' section - showing all API calls.
Have you seen this before?  Example attached.

One them seems to show the map URLs, I didn't want to post potential serial numbers and maybe the long session ID (big black box) but can send you the screenshots I took directly if you wish.

It happened again.  Appears it is 'Wormholy' - perhaps this is something they meant to remove before pushing an app update?
https://github.com/pmusolino/Wormholy
"Add it to your project, and that's all! Shake your device or your simulator and Wormholy will appear! You don't need to import the library into your code, it works magically!"
So, to get it to appear:

Start the Dyson iOS App
Shake the device
Requests the appears, you can view the full request, body, etc.

I fured out how implement camera with cleaned maps. Here a screenshot what you can get from Dyson Servers.
If you want i can share my code

@condor872 Maybe you could? ;)
Perhaps to ha-dyson that is actively worked on?
https://github.com/libdyson-wg/ha-dyson/issues/92

I figured out how implement camera with cleaned maps. Here a screenshot what you can get from Dyson Servers. It is not very simple, since you should use also pillow for edit images you can get from endpoint in BASE64 encoded format
If you want i can share my code

What was your fix?

Hi @condor872, I'm the maintainer over at https://github.com/dyson-wg/ha-dyson and I'm really curious what your solution was. If you'd like to help contributing to the project, it would be awesome.

Hi all, i'm very busy in these days, i'll share the code ASAP.


Warning: exec(): Unable to fork [curl -X ...

curl_method: exec
possibly related to #284 ?

There are not enough information to understand the cause of the issue. If you think it was related to #284, and given that that issue has been marked as fixed I'm going to close this too. Feel free to reopen it if you still get the error


Add RIFCS export functionality.

Add validation of generated RIFCS on top of Todd's work to generate it from the crawled data.
Clean up tab characters in touched files.

It seems like a lot of tabs got turned into spaces in this PR which makes it hard to look at the commit... any chance you can revert that?

I'll see if I can break out the tabs->spaces and the actual content and submit them separately.

Superseded.


Add Nintendo DS support

For adding Nintendo DS support

Thanks for this. It looks great!
Just the DS logo image looks a bit strange. There seems to be some artifacts in the image. Could you fix that please?

That should do it. Didn't see those before, think I got them all.

Awesome, thanks!


Use HashSet for xml package deduplication

Vec::dedup only removes consecutive elements, but HashSet implicitly dedups regardless of order.
I was running into an issue where codecov was failing to show any files at the root src/ path. Digging deeper, it turned out that the XML contained multiple <package name="src" ...> entries which codecov didn't like. The fix properly generates unique names for the packages array.

This is a great change! I was wondering why I was missing the files at the root of my src/ directory, and this makes sense. I was slightly concerned about sorting, but that doesn't seem to be an issue. Thanks for finding and fixing this!

Nice catch :+1: merging now :)


Add FAQ on why a VM can't ping its default gateway

Deploy preview for calico ready!
Built with commit b9150e31f900b2ecfae643217675e4613d897f4e
https://deploy-preview-2409--calico.netlify.com

Deploy preview for calico ready!
Built with commit deef6e74fe06962d1907e561e16c2f45f9b5fdae
https://deploy-preview-2409--calico.netlify.com

Deploy preview for calico ready!
Built with commit 6d5d0b3d01cadd6b3536af3cee828915a29d5b0a
https://deploy-preview-2409--calico.netlify.com


Add me to organisation

Would like to contribute in my free time

I am still organizing the framework for development and contributors. I'll add you soon.


Consumer Issue in decoding packet

Hi I am creating a consumer using the following code:
brokersList := strings.Split(config.Config.KafkaConfig.Brokers, ",")
	topicsList := strings.Split(config.Config.KafkaConfig.Topic, ",")
	consumerGroup := config.Config.KafkaConfig.ConsumerGroup
	bsmConfig := cluster.NewConfig()
	bsmConfig.Consumer.Return.Errors = true
	bsmConfig.Group.Return.Notifications = true
	//bsmConfig.Consumer.Offsets.Initial = sarama.OffsetOldest
	bsmConfig.Version = sarama.V0_10_1_0
	consumer, err := cluster.NewConsumer(brokersList, consumerGroup, topicsList, bsmConfig)

I am getting the following error:
kafka: error while consuming topic_harshit/0: kafka: error decoding packet: CRC didn't match

That sounds like a producer/protocol issue. Sorry, but sarama-cluster fully relies on https://github.com/Shopify/sarama for consuming messages, it doesn't do anything to them. It's something you may need to raise with them.


improve-quickstart-guide #305

added sechub client example
added create project
jdk8 -> jdk11
made more release-agnostic
(otherwise we would have to update the docs for
every Debian/CentOS/Fedora/Alpine/Ubuntu release)
created a build shell script for easy build
updated buildDocs script to use above

closes #305

@Jeeppler Thanks for reviewing! :+1:


Use .gitignore for filtering

Some configuration options to the ignore attribute that would be nice additions:
ignore:
  # both of these would be true by default
  dot-git-dir: true
  use-git-ignore: true
  files:
    - foo.rego

We currently hard code .git and .idea as we never will want to parse whatever is in there, but exposing these options in config, and using the .gitignore file, would be a better approach.
Thanks @srenatus for the idea.

https://pkg.go.dev/github.com/sabhiram/go-gitignore#example-CompileIgnoreLines looks like a solid little module for this.

While I'm sure it's nowhere near feature complete, @kristiansvalland wrote a simple implementation in Rego way back that we've been using for the ignore patterns users can set in .regal/config.yaml 😃
https://github.com/StyraInc/regal/blob/main/bundle/regal/config/exclusion.rego#L25
It would likely cover our needs, and if not, we should extend it to do that. We'd still need to read the file in Go an provide it as input, of course. And some things I'm not sure about, like if we should handle nested .gitignore files.

I think defaulting to respecting gitignore files anywhere would be a good call. Like, in mono repos, I'd imagine them to be common, and I personally like to use them on a nested way, too...

Yeah, me too. Although if we support that, we should also support nested Regal configurations, which we also don't do 😅


Beamsize: ensure correct units

Currently we are storing the incorrect units for beamsize in ISPyB. E.g. they should be 20x20μm but we are storing 20000x20000μm

Uploaded new patch of GDA branch which divides beam size we pass in by 1000


Crontab for cert renewal does not work

The crontab entry created by deploy_tool for prod is:
0 */12 * * * python3 /rw/app/deploy_tool.py prod renew_certs >> /rw/mount/cert_renewals.log 2>&1

However, the cert_renewals.log is filled with:
2023-05-18T00:00:01.874321 Starting cert renewal check...
2023-05-18T00:00:01.874438   RUN git log -n 1
fatal: not a git repository (or any of the parent directories): .git

ERROR: Return code 128

Probably the working folder needs to be changed.

The fix for this is just to cd /rw/app; ./deploy_tool.py.....
However #52 negates the need for this at all, since certbot can do auto-renewal from inside the container by default.


Update pubsub README file

Update the status, link to the Cloud Pub/Sub documentation, add an example, and generally make it look like the README for our GA client libraries.

A good example is the Java README.
(required)

product description
link to product documentation and library documentation
versioning

(highly recommend to have)

quickstart

installation guide

example usage

(optional)

contributing guide
community
license


clang 3.6 packages

I'm building a project using conan and boost-1.60, but if fails to compile in clang 3.6 (using travis)
Project is here with its conanfile.
Travis file is this, now trying with --build bzip2 here.
Conan for this configuration is called using:
conan install -s compiler=clang -s compiler.version=3.6 -s compiler.libcxx=libstdc++11 --build bzip2

Its solved right?

It compiles, yes. Although it will be better to have these packages uploaded to conan server, won't it?

Yes, should be great. I we haven't done it yet it's because we already generate more than 100 packages for each Boost version, If we introduce clang and its versions... it can be crazy to maintain. I would like to develop some improvements to conan-package-tools to have custom clang docker images for this purpose first.


MacOS version unusably slow

Everytime I use the Keybase MacOS app, it's painfully slow.  Every new message or each time I switch focus to the app, I get the spinning pinwheel for about 60 seconds before the app is usable.  It's so bad I can't really use it for realtime chat.  I don't see these problems in my iPhone, but since mobile typing is slower and there's no filesystem, without a working Mac app, I can barely make use of Keybase.

That's definitely not our experience, I wonder what's wrong. Can you run (from the Terminal)
keybase version
sw_vers
keybase log send

Thanks and sorry for the bad experience.

Thanks for the quick response.  I ran those commands and it appears the log was sent.  Here are the versions:
Client:  4.2.0-20190710181120+e7c0bdc4a2
Service: 4.2.0-20190710181120+e7c0bdc4a2
ProductName:	Mac OS X
ProductVersion:	10.12.6
BuildVersion:	16G2016

did you do a keybase log send too?
thanks

I did, I guess you need that log id:
Success! Your log ID is:
375773c1077eb53c3dceab1c
Thanks.

Were the logs of any use?  I noticed the version got bumped recently, but the MacOS app is still unusably slow.

Nothing jumped out at me from the logs. Can you send me a screenshot of your Activity Monitor's memory and CPU screens when you are trying to use the app?

I have a Quicktime video showing it taking ~30 seconds for the app to load a couple of new messages, along with Activity Monitor open during this time.  Can I email it somewhere instead of posting it here?

Ok, send to<EMAIL_ADDRESS>or max at keybase


Allow sequential and parallel batches

I made it possible to configure if you want to execute batches to a data source sequentially or in parallel.
By default they will be parallel, so we should mention in the next release notes that people relying on the sequential execution of the batches should update their data sources to specify this sequential execution.

Codecov Report

Merging #113 into master will decrease coverage by 3.57%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
- Coverage   76.66%   73.09%   -3.58%
==========================================
  Files          10       10
  Lines         240      249       +9
  Branches        2        2
==========================================
- Hits          184      182       -2
- Misses         56       67      +11

Impacted Files
Coverage Δ

shared/src/main/scala/datasource.scala
66.66% <ø> (ø)
:white_check_mark:

shared/src/main/scala/fetch.scala
78.37% <100%> (-2.71%)
:x:

shared/src/main/scala/interpreters.scala
86.82% <86.66%> (-6.52%)
:x:

shared/src/main/scala/cache.scala
83.33% <0%> (-8.34%)
:x:

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 8c24470...f20ef08. Read the comment docs.

👏 👏 👏 Amazing work, well done!


Meta descriptions for chapters?

Describe the issue
I've been using the overview as the meta description for the chapters, but maybe we should edit and make consistent (similar to page descriptions issue).
Example: https://proudlyservingbook.com/contents/default-to-open

We'll use overview.


Addition of video files to be concatenated

Issue: https://github.com/aws-samples/amazon-chime-media-capture-pipeline-demo/issues/7
Changes:  Modification of Process Lambda to look for files in the video folder as well as the audio folder.  Updates to concat each of these video files together by attendee and audio files separate.  No concatenation yet of separate video and audio files.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Any thoughts on this?


Use backtick to format references to file names in working-with-the-npm-registry

Why:
Closes:
#35763
#35762

What's being changed (if available, include any code snippets, screenshots, or gifs):

.npmrc -> .npmrc
package.json -> package.json
Check off the following:

[x] A subject matter expert (SME) has reviewed the technical accuracy of the content in this PR. In most cases, the author can be the SME. Open source contributions may require a SME review from GitHub staff.
[X] The changes in this PR meet the docs fundamentals that are required for all content.
[X] All CI checks are passing.

@yolocat-dev Thank you so much for opening a PR with this fix! Your changes look great ✨ We are currently in a repo freeze, but we'll get this merged as soon as the freeze ends on Jan 6th 💛


make json a default type for set update/replace

#13

I have tested this PR both with inline update as well as with typed update
It worked without issues


Formats the code for consistency.

I went through and corrected some inconsistencies in the way the code is formatted.

Hi @Glinkis ,
Thanks a lot for the contribution. May I ask if you have performed these changes using any code formatting tool or manually?

Hi @satyapavan, in this case is formatted the code manually. I would normally use a tool such as prettier, but I couldn't configure it to the style you use here, with no spaces in statements such as these: if(), for().


Sync private repository

Hi!
I've tested synclinear tool with public repository, it works so good!
But it's possible to sync private repository?
Thank you so much!

i think there is no difference. it should work. try it and let us know! i believe there are a few private repos

Thank for answer!
When I select the repo, I don't find private repos...


Post dawn4.1 updates

Updates to stay in sync with eosio/eos master..
Related: https://github.com/EOSIO/eosjs/issues/106

Ah, nice! pubkey account lookup back in :))

Released eosjs@13 passing with dawn-v4.2.0


Allow tags to be passed through message options

Passes tags through to Mandrill, allowing better searching/reporting

Merged here: https://github.com/RebelMail/nodemailer-mandrill-transport/commit/d5f7cb03f64502dc53ade0dfc6628ef986928d26


Handle overflow in GCD scalar function

Describe the bug
This is a follow-up on issue #11031 and PR #11036, which fixed the incorrect results or infinite loops caused by passing i64::MIN in combination with other values into the GCD scalar function.
While fixing this, we noticed that an overflow occurred when passing i64::MIN in both parameters.
To Reproduce
DataFusion CLI v39.0.0
> select gcd(-9223372036854775808, -9223372036854775808);
+--------------------------------------------------------------+
| gcd(Int64(-9223372036854775808),Int64(-9223372036854775808)) |
+--------------------------------------------------------------+
| -9223372036854775808                                         |
+--------------------------------------------------------------+
1 row(s) fetched.

Expected behavior
The real GCD of i64::MIN and  i64::MIN is<PHONE_NUMBER>854775808, but this can't be represented as an i64 (it is i64::MAX+1), thus we need to return an overflow error like this:
i64::try_from(a << shift).map_err(|_| exec_datafusion_err!("Overflow in gcd"))

Additional context
Credits to @jonahgao (sorry for the ping, but it is owed)

take


Indicate links which open in a new tab

In the footer 2 links open in a new window:
Become a contributor
Support us
These links should have a small icon after them to show that visually. The project already has react icons installed. The following icon can be used and added to the footer component: BsBoxArrowUpRight
Import as follows: import { BsBoxArrowUpRight } from 'react-icons/bs'

assign this to me

Yes, sure @hariscs, go ahead!


MINOR: improve logging of consumer system tests

@ewencp I turned off stdout collection since I can probably get by with the event logging itself. Note that we may may have to turn down that logging (or at least choose the events we log more selectively) if VerifiableConsumer ever does log messages individually

@hachikuji Why not collect it in just the one test you care about as I suggested? If you need the info, I don't have a problem collecting it, I just thought it made sense to limit extra collection of data to the specific test we need it for.
I think the confusion re: verbosity came up because I know we have a verbose mode in VerifiableConsumer, but I guess we never enable it in the system tests...

@ewencp I care about one test at the moment, but I'm assuming that others may also have transient failures in the future. The problem is that these failures tend to be hard to reproduce (e.g. I still haven't reproduced the one in 3513 yet), so I need to get as much information from each failure as possible.

LGTM


Question about working with oauth tokens

I am using springfox-swagger2 and springfox-swagger-ui version 2.8.0 for documenting my api's. I am facing 2 issues listed below.

I need to generate the oauth2 token using the token and the authorize endpoint,which I am able to authorize using the Authorize button.But the problem is that swagger provides a default redirect url,but how to provide a custom redirect url .Like the way,we are able to provide user specific clientId and client Secret.So that user can provide any redirect url and genearte the token.
On click of authorize button I am unable to see the acces tokens and refresh tokens,instead I can only see statement 'authorized'.

How can I display even the token to the user after the user clicks on Authorize button in swagger??

Swagger 2.0 does not support oauth flows very well. You may need to wait for OAS 3.0 support

Thank you for your reply...:)
But doesn't this version(2.8.0) support the display of access and refresh tokens,instead of just showing authorized??

It doesnt because it only supports swagger specification 2.0


support spaces in scopes

Current Situation
feat(foo, bar): baz -> Fails
feat(foo,bar): baz -> OK
Expected Situation
feat(foo, bar): baz -> OK
feat(foo,bar): baz -> OK
I'm willing to open a PR.

Thanks for the suggestion @scarf005 - this has now been merged, and will be deployed soon 🙂


Add click-to-center feature

While this tool is really amazing for build123d and cq workflows, there is one and only one feature that sometimes forces me to go back into freecad: the "click to center" feature in some of the freecad view modes.  That is, you can middle-click on a spot in the view and it re-centers the camera there, which, more importantly, means if I orbit or zoom in, it zooms into the spot I selected.  This is critical for inspecting small parts embedded in a larger part.
Right now I can of course pan to wherever I need to be with right-click, but I find that imprecise and awkward to use such that in practice I sometimes have to dump my object and load it in freecad to inspect things.  Can a middle-click-to-center feature be added to vscode-ocp-cad-viewer ? (or maybe this belongs better in three-cad-viewer? I confess I don't really understand the boundary of what is where)

(Note I'm open to "well then make a PR for that!" but I am not really sure where to find the relevant interaction code, so pointers would be helpful on that...)

Everything that is in the viewer window (cad tree, canvas with CAD object(s), mouse handling, ....) is the viewer component three-cad-viewer.
vscode-ocp-viewer does two things:

Use ocp-tessellate to convert OCP to mesh (tessellation)
transport the tessellated object from Python to VS Code and from there to the embedded viewer component (using websockets)

So the feature request you are asking about is in fact a three-cad-viewer topic as you suggested.
Currently it can be done by code:
set_viewer_config(target=(x,y,z))

And something like that can be added to the viewer component, i.e. when doing a -click, then raytracing (already used) will be used to identify the nearest point on the object that this click would point to on an axis from the camera through the click point in the view projection point to the object. And set camera's target to this point.
The code approach above uses the measure/property mode to get the coords and then sets the target of the camera.

Fixed in 2.5.0


⚠ Samsung has degraded performance

In 48614fb, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1844 ms

Resolved: Samsung performance has improved in 54ffb66.


Joints and cigarettes need a bit of love.

Two issues
I think Rolling Papers and Cigarette filters were coded with the intent of not being stackable. Now they are. If you attempt to craft a joint or cigarette with stacks of Rolling Paper, Cigarette Filters, or Ground Tobacco/Cannibis, the craft will consume every item in the stack. This can cause you to lose a lot of rolling paper or filters.
Joints (and I think cigarettes, but I didn't test this) cannot be named with a Hand Labeler. There... isn't a practical reason for this and it should probably be fixed.

Related to #10466

Hand labeler seems to work fine for me.


Should PWSTR/PSTR structs be repr(transparent)?

I was looking at the generated code and saw PWSTR was repr(C):
#[repr(C)]
#[derive(
    :: std :: clone :: Clone,
    :: std :: marker :: Copy,
    :: std :: cmp :: Eq,
    :: std :: fmt :: Debug,
)]
pub struct PWSTR(pub *mut u16);

and the functions generated like QueryFullProcessImageNameW take them in directly:
#[link(name = "KERNEL32")]
extern "system" {
    pub fn QueryFullProcessImageNameW(
        hprocess: HANDLE,
        dwflags: QueryFullProcessImageName_dwFlags,
        lpexename: PWSTR, // <-- takes in a PWSTR
        lpdwsize: *mut u32,
    ) -> BOOL;
}

From the docs on repr(transparent):

while repr(C) happens to do the right thing with respect to memory layout, it's not quite the right tool for newtypes in FFI. Instead of declaring a C struct, we need to communicate to the Rust compiler that our newtype is just for type safety on the Rust side. This is what repr(transparent) does.

To guarantee that rustc passes the PWSTR just like a *mut u16 when sending it as a parameter thru the ABI, these newtypes should be generated with repr(transparent) right?
I saw this on PSTR and BOOL as well.

Good catch - the way things are currently only guarantees that layout will be correct but if the inner type has different ABI requirements than single field structs, this could lead to problems. This probably works fine now, but this should be changed.


Deletes frontmatter

Frontmatter support isn't probably a goal for this editor, but deleting text seems like a bug in any case.
To Reproduce
Steps to reproduce the behavior:

Toggle markdown on
Enter YAML frontmatter
Toggle markdown off
No frontmatter
Toggle markdown on, and frontmatter is gone

Sample frontmatter
---
title: YAML Front Matter
description: A very simple way to add structured data to a page.
---

Thanks for the report! It looks like the markdown parser is parsing this "correctly" as [hr, h2], but once it hands it over to the Prosemirror schema, the h2 doesn't seem to be getting inserted (and therefore, never serialized back into markdown).

Looks like this is due to the softbreak not being allowed as a child of heading in the default prosemirror-markdown schema, as that package doesn't have softbreak support implemented. I can put a fix in here easy enough.


Parsing Number in mesg.yaml

Parse is done into float64 and it should be into big.Float

To keep a good level of compatibility between the different programmation language and their limitation (understand JavaScript), Number have to be compatible with JS Number that are float64 or int32.


Check gofmt on PRs

Evidently golangci-lint isn't doing it, as seen at #19

I am interested in contributing to this project. I think this issue is fixed. Looking at the #64
Run golangci/golangci-lint-action@v2
prepare environment
go env
Installed Go in 68ms
Requested golangci-lint 'latest', using 'v1.41.1', calculation took 94ms
Installing golangci-lint v1.41.1...
Downloading https://github.com/golangci/golangci-lint/releases/download/v1.41.1/golangci-lint-1.41.1-linux-amd64.tar.gz ...
Cache not found for input keys: golangci-lint.cache-2691-23dfab94cf6e959b68510ed62773e31da4864cf2, golangci-lint.cache-2691-, golangci-lint.cache-
/usr/bin/tar xz --overwrite --warning=no-unknown-keyword -C /home/runner -f /home/runner/work/_temp/8a2b2ae7-8d14-4b74-b4e0-9151d3df5732
Installed golangci-lint into /home/runner/golangci-lint-1.41.1-linux-amd64/golangci-lint in 481ms
Prepared env in 577ms

run golangci-lint
  Running [/home/runner/golangci-lint-1.41.1-linux-amd64/golangci-lint run --out-format=github-actions --timeout=5m] in [] ...
  golangci-lint found no issues
  Ran golangci-lint in 73097ms

I don’t see what you mean at #64.
What happens if you make a PR which does not have the correct formatting?


typo: defenition -> definition

@nschonni : Thanks for your contribution! The author, @deborahc, has been notified to review your proposed change.

Verified editorial correction.


OrientDB 2.2 Beta - Error on Alter Class Add Custom Properties

This works in 2.2Alpha:
 ALTER CLASS Person CUSTOM onCreate.identityType=role
It produces the error:
com.orientechnologies.orient.core.sql.OCommandSQLParsingException: Error on parsing command at position #0: Error parsing query: ALTER CLASS Person CUSTOM onCreate.identityType=role Encountered " "ALTER "" at line 1, column 1.

Hi @cvm1ll3
I guess you are referring to 2.2 beta (not alpha), right?
It must be the new strict parser...
Could you please try this and let me know if it fixes the problem?
ALTER CLASS Person CUSTOM `onCreate.identityType`=role

Hi @luigidellaquila,
Yes I'm on beta, just finished testing alpha. I tried single and double quotes and it still produces the same error.

ok, testing it now...

Hi @cvm1ll3
Ok, I found the problem and I have a fix for it.
The new parser is more strict than the old one and only accepts a valid identifier as a custom property name. The . character is not allowed in identifiers, so it has to be back tick quoted (back tick is `, not single quote ' or double quote "), so you will have to write the query this way
ALTER CLASS Person CUSTOM `onCreate.identityType`=role

I had to do a little fix and I'm running all the tests, if everything is OK I'll push it to develop branch in a few minutes
Thanks
Luigi

Hi @luigidellaquila
Oh got it. Yes back tick works perfectly. Thank you so much!

Hi @cvm1ll3
I just pushed the fix to develop branch, you should build from sources or wait for next RC1 to have it.
Just pay attention to the fact that in 2.2 beta, before the fix, the parsing works fine with back ticks, but the result is not the expected one, if you query the schema with
select from (
select expand(classes) from metadata:schema
) where name = 'Person'
you'll see that also the back ticks are wrongly stored in the property name, so in the end it won't work.
I'm closing this issue, please reopen if you experience any other problems
Thanks
Luigi

Hi @luigidellaquila
Ok, will do thanks a lot!

Hi @luigidellaquila,
I just noticed now, the command ALTER CLASS Person CUSTOM onCreate.identityType=role says its successful but when I create a vertex, the OUser RID is in the _allow field rather than the ORole RID.


empty_feed_notice: Update “stream” to “channel”

Discussion.

Sneaky hidden string there. Sure, merged.


Script test wont run unit tests if there are lint errors

When I run yarn test linter is executed but not unit tests. That happens because test:linter exits with code 1 and then npm-run-all exits too. I don't know it this is expected or not, but I expected that unit tests where executed independent of linting errors. This can be solved by placing a -c flag on npm-run-all command. I can PR that

Sounds good
On Tue, Dec 20, 2016 at 5:00 PM Adilson Schmitt Junior <
<EMAIL_ADDRESS>wrote:

When I run yarn test linter is executed but not unit tests. That happens
because test:linter exits with code 1 and then npm-run-all exits too. I
don't know it this is expected or not, but I expected that unit tests where
executed independent of linting errors. This can be solved by placing a -c
flag on npm-run-all command. I can PR that
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jumpsuit/jumpstate/issues/14, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCVwp5KI9I8dB8hTKvEaGNJ_zZOiTks5rKGwmgaJpZM4LSbrw
.


Parse's reset password not working. ParseRequest$ParseRequestionException:

I have deployed parse server at azure and implemented the parse's reset password function through this guide
But I'm getting this error "ParseRequest$ParseRequestionException:" which doesn't shows what am I missing or wrong at.

Do I need to perform email verification to use password-reset feature?
Here's my code:(which is actually almost the same with tutorial)

            tvFgtPwd.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View v) {
                        ParseUser.requestPasswordResetInBackground(edtuser.getText().toString(), new RequestPasswordResetCallback() {
                            public void done(ParseException e) {
                                if (e == null) {
                                    Toast.makeText(getActivity().getBaseContext(), "An password-reset email has been sent to your mailbox.", Toast.LENGTH_SHORT).show();
                                } else {
                                    Toast.makeText(getActivity().getBaseContext(), e.toString(), Toast.LENGTH_SHORT).show();
                                }
                            }

                        });
                }
            });

That tutorial covers the use of this feature in the old hosted Parse.com service. For Parse Server specific instructions, see the README.
You may use Server Fault for questions about managing Parse Server.
For code-level and/or implementation-related questions or technical support, please refer to Stack Overflow.

Hi, thanks for answering. I have another question. How do we access the parseserver that are hosted at azure and use the password reset js code you mentioned? Is it by locally deploying the parseserver that we have forked and hosted on azure?


[Request] Time map feature

Mapbox just created this prototype: https://blog.mapbox.com/a-new-kind-of-map-its-about-time-7bd9f7916f7f
Demo: https://www.mapbox.com/labs/timemap/search/
Would be awesome to have that as an optional UI/UX

I think this is out of the scope of this module, or at least very low priority regarding the other existing issues this repository have.


FileSize is not matching when downloading for some users

When i requesting microsoft graph url For getting the File Info
Request URI :- https://graph.microsoft.com/v1.0/me/drive/items/fileId
its returning the file size in JSON response with  "size": 17410 (in KB).
But when i am trying to download the same file with
Request URI :-https://graph.microsoft.com/v1.0/me/drive/items/fileId/content
then its giving Input Stream  then i am storing in my local system.
its giving different "size" 14267 (in Kb)
At last its not giving for all files its only for particular user i am getting.

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Add a 'AddRugExtensionLanguageBackedByANTLR' editor

We need an editor that adds a new extension language to rug itself, one that is backed by an ANTLR grammar.

Issue moved to atomist-rugs/common-editors #22 via ZenHub


Evaluate a Memory usage in Tick Script

Hi @nathanielc
I want to evaluate a memory usage of given node such that (x) . and that value of (x) comes in  int type like  ie<PHONE_NUMBER> . using the eval with humanBytes function i can parse into normal human readable format but when i want to compare these value with my threshold value it is saying
{{ error evaluating expression for level CRITICAL: mismatched type to binary operator. got string > int. see bool(), int(), float(), string(), duration() }}
using eval as :  |eval(lambda: humanBytes("HeapMemoryUsage_max"))
.as('used_mem')
crit condition as :  .crit(lambda: ("used_mem") > threshold )

@Abhinavm8 humanBytes converts the int into a string which is no longer suitable for comparisons against your threshold.
Try something like this:
|eval(lambda: humanBytes("HeapMemoryUsage_max"))
    .as('used_mem')
    .keep('HeapMemoryUsage_max', 'used_mem')
|alert()
    .crit(lambda: "HeapMemoryUsage_max" > threshold )
    .message('Memory used is: {{ index .Fields "used_mem"}}')

thanks for your reply @nathanielc
It did work. One more concern is that it is giving the data in GB . what if we want that   data to be represented in MB .?
Or if  we want to measure the metrics for Disk I/O they should be in wps or rps ?

@Abhinavm8 The humanBytes function tries to keep the number of digits that you have to read small. If you want a consistent representation then its not hard to do yourself.
var mb = 1024*1024

...
|eval(lambda: string(int("HeapMemoryUsage_max") / mb) + 'MB' )
    .as('used_mem')
    .keep('HeapMemoryUsage_max', 'used_mem')
|alert()
    .crit(lambda: "HeapMemoryUsage_max" > threshold )
    .message('Memory used is: {{ index .Fields "used_mem"}}'

thanks
it works for me .


feat(openai): add max_num_results option to file_search type

OpenAI has released a new param, max_num_results, that controls the maximum number of results retrieved by the file_search tool. This PR adds it to the types.
https://platform.openai.com/docs/api-reference/assistants/createAssistant#assistants-createassistant-tools

Closing as I made a mistake as to where to put this type.


feature: add Melodic Minor + jam tracks for all keys

Summary

adds melodicMinor option to the Mode enum
adds melodicMinor pattern to the ModePatterns constant
adds jam tracks for Melodic Minor in all keys

Screenshots
https://res.cloudinary.com/ryan-martin/image/upload/v1726176330/Screenshot_2024-09-12_at_5.22.42_PM_w0t4jy.png
https://res.cloudinary.com/ryan-martin/image/upload/v1726176331/Screenshot_2024-09-12_at_5.22.53_PM_xhrgqd.png

Reopened since the last was stuck in draft.

@whitep4nth3r
Ah interesting, I didn't know those facts. Still plenty of theory to learn on my end 😅. No worries, I'll close this out!

I'm having trouble running this project locally given it is so old, and I haven't been able to update it so am unable to check this out.

I actually just deleted the lock file, re-ran yarn and then ran npm run start and it all worked fine on my end.

Ah thanks for that, I will try that!


test configure_flags with bash variable interpolation before recompiling

to avoid rebuilding in cases where configure_flags are specified with bash variables
e.g.
override['nginx']['configure_flags'] = [
'--add-module=$(passenger-config --root | xargs echo)/ext/nginx'
]

This change is

did this break something @miketheman ?

I think @miketheman closed all the issues this weekend :performing_arts:

ahh. fwiw, this works great for me (and was a total pain to track down)
On Mon, Mar 14, 2016 at 1:01 PM Anthony Scalisi<EMAIL_ADDRESS>wrote:

I think @miketheman https://github.com/miketheman closed all the issues
this weekend [image: :performing_arts:]
—
Reply to this email directly or view it on GitHub
https://github.com/miketheman/nginx/pull/355#issuecomment-196498116.

The branch this was a PR for was removed, and GitHub closes any open PRs for that automatically, with no extra notice. :disappointed:

i can open a new pull. should i hit the 3.0.0-rewrite branch?

@nathantsoi Per the goals of the rewrite, we're no longer recommending recompiling the server on each node, preferring deployment via package.


Updated Epidata Index (main description page)

Updated Epidata Index page to focus less on Covidcast and Covid-19.

Summary:

Prerequisites:

[x] Unless it is a documentation hotfix it should be merged against the dev branch
[x] Branch is up-to-date with the branch to be merged with, i.e. dev
[x] Build is successful
[x] Code is cleaned up and formatted

CI failing because we hit the rate limit on docker.io. Will rerun later.


Being able to set a value

It would be great to be able to force a color value at any time on the Color slider. We could imagine that we'd want to save the latest selected color in the user settings and set that color back when he comes back.
What do you think?

Great idea! I have a pull request I'm working on here and I'll try to add something like that in. Thanks!

This has been merged in v4.0.

Thanks !

I'm trying to set a default value for the slider and had to make a couple changes. I'm using autolayout so setting color right away was not positioning the preview view correctly.
I moved these lines to the default center area of layoutSubviews() and now the view shows up in the right spot.
`// Set default preview center
case .horizontal where preview.center.y != bounds.midY,
.vertical where preview.center.x != bounds.midX:
let sliderProgress = gradientView.calculateSliderProgress(for: internalColor)
            // `centerPreview` only uses the `x` or `y` value based on the `orientation` and ignores the other value.
            centerPreview(at: CGPoint(x: sliderProgress * bounds.width, y: sliderProgress * bounds.height))`

This seems to be an issue for me too. Can we please have the above code merged to master branch so that cocoapod will have same fix? Right now when we set a color, it is not updating the preview view position.


deredactie.be no audio for most of the video

When I download videos from deredactie.be the audio stops after 3 seconds.
The test url in the vrt.py file doesn't exist any more so use http://deredactie.be/cm/vrtnieuws/videozone/nieuws/binnenland/1.2232842 instead.
C:\Users\kenavera\Videos\youtube-dl>youtube-dl http://deredactie.be/cm/vrtnieuws/videozone/nieuws/binnenland/1.2232842 --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['http://deredactie.be/cm/vrtnieuws/videozone/nieuws/binnenland/1.2232842', '--verbose']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2015.01.16
[debug] Python version 2.7.8 - Windows-8-6.2.9200
[debug] exe versions: ffmpeg N-63100-g5460ab7, ffprobe N-63100-g5460ab7
[debug] Proxy map: {}
[VRT] 1.2232842: Downloading webpage
[VRT] 2232842: Downloading m3u8 information
[VRT] 2232842: Downloading f4m manifest
[debug] Invoking downloader on u'http://vod.stream.vrt.be/deredactie/definst/2015/02/1210150207030927921_Polopoly_16x9_DV25_NGeo.mp4/manifest.f4m'
[download] Downloading f4m manifest
[download] Destination: Het vliegtuig nemen zonder identiteitscontrole Het kan-2232842.flv
[download] 100% of 11.07MiB in 00:06
I don't have much experience with python, so I can't find what's wrong.

I downloaded the flv version (f4m-823) and the mp4 version (m3u8-927) and both play just fine over here (the video as well as the audio). There might be something wrong with your player. Is it up-to-date?

Thanks, I converted the video to mp4 and all of a sudden the video had audio.
I never thought that it could be a fault of the player.
Both times I used an up to date version of vlc, so I'm gonna search there for strangeness.
So, it's not a fault of youtube-dl and I'll close it.

Reopening.
It is a bug in f4m downloader possibly similar to #2752 or #2388.

I have known about this issue for a long time. Some players (last version of VLC, my LG Smart TV, etcetera) are not able to play the sound. A previous version (1.x) of VLC does manage... sometimes.
Don't know what's causing this, maybe a problem with the video container, I'm not sufficiently knowledgeable about this. There definitely is a problem on the provider side (VRT) as well. They seem to be using some rare streaming software, one of the only sites I know of which has this issue.
The workaround: add -f mp4 to the command line to force the download of the mp4 version. Wouldn't it be best to adapt the code so that VRT downloads get downloaded in mp4 by default? (the presented mp4 stream is always at least on par with the flv version by the way)
Partly a duplicate of this issue by the way:  #4121

This issue used to be a problem of the VRT video player, a long time ago. But it hasn't presented itself anymore since. So this report can be closed.

Downloaden Downloaden Downloaden Downloaden Downloaden Downloaden Downloaden
Downloaden =========>>>>>>>>>> http://bit.ly/2oiWvqi
Downloaden =========>>>>>>>>>> http://bit.ly/2oiWvqi
Downloaden =========>>>>>>>>>> http://bit.ly/2oiWvqi
Downloaden =========>>>>>>>>>> http://bit.ly/2oiWvqi
Downloaden =========>>>>>>>>>> http://bit.ly/2oiWvqi
Downloaden =========>>>>>>>>>> http://bit.ly/2oiWvqi
Downloaden =========>>>>>>>>>> http://bit.ly/2oiWvqi
Downloaden =========>>>>>>>>>> http://bit.ly/2oiWvqi


tweaked example; minor improvements in comments

made the mlp example a little bit more modular (supporting repeated runs); fixed some minor typos in comments/code

The check fails because updating SBCL to 2.3.10 is causing cl-environments to stop working. I've updated the workflow described in Cl.yml to explicit the version to use.  So, it should work now.

In accordance to the refactoring of JITCPUTensor, I've added major modifications to the JITCPUBackend package. Could you fix the conlicts of source/backends/JITCPUTensor/on-finalizing.lisp?.

Thanks.


need some improvement on changelogs and docs

Hey dudes,
it's a doc issue.
I'm using 17074 and WSL when having a need to debug my scripts, and i'm using RoliSoft's WSL-Distribution-Switcher to add a diy Linux image. But there is a problem that i found %LOCALAPPDATA%\lxss is absent after i added the wsl feature and installed ubuntu.
so...i think we need detailed and more readable changelogs to know what changed and when, to improve support for these features even if they are still in development
thanx for your work!

Did you install Ubuntu with LxRun.exe or from Windows Store?

@Biswa96
Yes, I have lxrun and that is not the problem.
The problem is that I'm not sure which to use,  %LOCALAPPDATA%\lxss or other location, and from which version this change took place. so I can't improve my script to intelligently determine where to find or place the binary and data.

I'm using the Windows Store version of Ubuntu. Same problem.
"%LOCALAPPDATA%\lxss" doesn't exist. It's not a hidden or system folder either. On the Linux side, I can use /mnt/c/ to access the Windows folders, but it would be good to know how to do the opposite.

LxRun.exe is now deprecated. Move on to the new Windows Store distributions.
BTW, @benhillis Why there there are some "Legacy" registries left over in LxssManager.dll?

so, what replaced %LOCALAPPDATA%\lxss in the new windows store distros

The new installation path is C:\WindowsApps\DistroName and the distribution installed in AppData\Local\DistroName.


Add ability to show all components in entity and qol changes

Also move fn inspector(...) function to ui function of inspector tab. Also wrap show_component logic in fucntion for reusability

I don't get why move inspector() to ui() to, did you need access to any specific data?
This makes it quite harder to read the code. I would expect breaking it into even smaller pieces

Because it's not good not in ui fn. InspectorTab requires access to all InspectorTab fields, so it looks organically just in the ui. It was put into a separate method by the coincidence of history, not logic....

But I support breaking this big function into smaller ones in the future


Added "Use the Deployment Center" section

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

@v-abiss I noticed you made this change in a doc that's specific to custom containers. Wouldn't it be more appropriate in the deploy-github-actions.md doc? That one is not for custom containers.

@v-abiss I noticed you made this change in a doc that's specific to custom containers. Wouldn't it be more appropriate in the deploy-github-actions.md doc? That one is not for custom containers.

@cephalin the experience highlighted here is specific for WebApps for containers, where we are analysing the repo and detect dockerFiles. Will be a helpful experience for handholding customers
@v-abiss can you expand on the content here and add relevant details

@v-abiss I noticed you made this change in a doc that's specific to custom containers. Wouldn't it be more appropriate in the deploy-github-actions.md doc? That one is not for custom containers.

@cephalin the experience highlighted here is specific for WebApps for containers, where we are analysing the repo and detect dockerFiles. Will be a helpful experience for handholding customers. with that context let us know if you suggest an alternate repo for this content
@v-abiss can you expand on the content here and add relevant details

#please-close


documentations and demo for pruning fully connected layers in GPT

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


OData: Duplicate types generated for MS Graph

Describe the bug
OData handler should not generate duplicate sdk method types (mesh:ts)
To Reproduce
Steps to reproduce the behavior:
MS Graph is the example for OData Handler on the website
sources:
- name: MyGraphQLApi
handler:
odata:
baseUrl: https://graph.microsoft.com/v1.0
operationHeaders:
Authorization: Bearer ${MY_API_TOKEN}

It generates a hundred duplicate method signatures, I don't see the pattern (yet).
Expected behavior
Generate/Build without type errors/method duplications or show the reason of these type errors while doing so
Environment:

OS: OSX
@graphql-mesh/odata: 0.8.1
NodeJS: 14

Additional context

Playground https://github.com/Urigo/graphql-mesh/tree/master/examples/odata-microsoft does not work for reproduction
Adding:
    "mesh:ts": "mesh typescript --output ./generated.types.ts"

Results in:

Container Warning:
generated.types.ts is larger than maximum allowed file size (2610413 > 524288), not synchronizing

I need help/would like to help to resolve this issue, any pointers? @ardatan, @santino? Can you reproduce it?
I'll do some more testing/comparison with microsoft's official client in meanwhile. Maybe it's just a bad idea to consume MS Graph via GraphQL(-Mesh).

I've finished my evaluation with MS Graph, I need to stick to ms' official package since I was not able to find a clear solution here.
My curiosity remains, so if you find the reason for the duplicates, I'd like to know! @ardatan @santino

Bump. I think I have a similar issue with other OData schemas (also Microsoft) I would still be interested in this being fixed!

Hi @JustusNBB  and thank you for the report!
Sorry but I'm not adding a lot here but just labeling it according to our new Contribution Guide and issue flow.
It seems like we are on stage 1.
Do you think you would be able to create a failing test to isolate the actual issue?
Thank you and sorry that this comment is not a complete solution (yet).

@Urigo I've provided a way for reproduction with one of the examples.
I am not sure how such a testcase would work:

use (e.g. ms graph odata schema) to generate types from
fail when importing the types?

Do you think you would be able to create a failing test to isolate the actual issue?

As of now, no - but maybe with your help!

Could you test it with<EMAIL_ADDRESS>? @JustusNBB

When I run my procedure on your bugfix branch I no longer get duplicate types!
Your operations Record Map has fixed it!

Available in<EMAIL_ADDRESS>


Does not plot categorical variables with text values

How can we plot categorical variables with text values?

Hi @aliabbasjp
Thanks! for this question
So to plot categorical variables they must be encoded to numerals.
if you are using Sklearn this will help you encode them.
Auto label encode feature would be added in coming releases.
Hope this works!
Thanks


MINOR: Move lock method outside try block

The Lock.lock in the try block may cause Lock.unlock  throw exception when it throw exception .
I think it's nice to move  outside although ReentrantLock.lock impl doesn't throw exception.
 class X {
   private final ReentrantLock lock = new ReentrantLock();
   // ...

   public void m() {
     lock.lock();  // block until condition holds
     try {
       // ... method body
     } finally {
       lock.unlock()
     }
   }
 }

pattern recommended by
https://docs.oracle.com/javase/10/docs/api/java/util/concurrent/locks/ReentrantLock.html

@chia7712   can you help me to take a look?


need insider

Hi I wanted to see if there is a way to include the required sdk without upgrading the windows machine to windows insider ?

Hi @valavakilian , you can grab the SDK here: https://developer.microsoft.com/en-us/windows/downloads/windows-10-sdk. Please re-open if this issue is still pending.


Ci artifacts

This PR should only be merged after the gensource branch is merged, as that's the branch that contains the test harness updates that save the logs and cores
This PR adds artifact storage to the integration-tests and integration-tests-with-resilience CircleCI jobs.
When an integration test or resilience test fails, the logs of each of the workers, as well as the binary dumps of each sender and sink are saved to /tmp/wallaroo_test_errors/<path to test>/<test name>/<test start time>/.
If there are any core dumps in the current working directory (because in circleCI they are local to the CWD), those too are moved to that directory.
The two CircleCI jobs are then set up to upload those artifacts to S3 with the full path.
This gives us full run logs as well as core dumps from any failing test on CircleCI, which we've been finding to be very helpful in local dev work over the last few weeks.
/cc @jtfmumm @slfritchie

This will be merged into gensource branch.


[OSX] Window.Activate does not brings window to the front

To Reproduce
Steps to reproduce the behavior:

Create two windows
From first window call window.Activate() of second window
Second window will become "key" and focused window, but will remain in the background

Expected behavior
Second window should move to the top
Desktop (please complete the following information):

OS: Mac
Version 0.10.6 and 0.10.7, probably older

Workaround: manually call [Window orderFront] on the second window after IsActive property was changed to the "true"

I assume this line should call makeKeyAndOrderFront instead of makeKeyWindow
https://github.com/AvaloniaUI/Avalonia/blob/cbce24bffaa64d34262dcf20aa9352f6e423c2bd/native/Avalonia.Native/src/OSX/window.mm#L177


StopIteration: 'filename'

I'm trying to train on my own dataset, but I get the following error. Any ideas?
Epoch 1/103
Traceback (most recent call last):
  File "train.py", line 280, in <module>
    _main_(args)
  File "train.py", line 257, in _main_
    max_queue_size   = 8
  File "/usr/local/lib/python2.7/dist-packages/keras/legacy/interfaces.py", line 91, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 2194, in fit_generator
    generator_output = next(output_generator)
  File "/usr/local/lib/python2.7/dist-packages/keras/utils/data_utils.py", line 584, in get
    six.raise_from(StopIteration(e), e)
  File "/usr/local/lib/python2.7/dist-packages/six.py", line 737, in raise_from
    raise value
StopIteration: 'filename'

Seems like some xml annotation file lacks the filename field. Do print to debug.


Extraction issue on app manifest

I'm trying to integrate GN to scoop, so far I have this script:
{
    "homepage": "https://gn.googlesource.com/gn/",
    "version": "1.0",
    "description": "GN is a meta-build system that generates build files for Ninja.",
    "architecture": {
        "64bit": {
            "url": "https://chrome-infra-packages.appspot.com/dl/gn/gn/windows-amd64/+/Y1n9hsOOCXCojkjl7mfsHpUuNNe7Px_EtHU41_ihAugC?&response-content-disposition=attachment%3B+filename%3Dgn-windows-amd64.zip",
            "hash": "6359fd86c38e0970a88e48e5ee67ec1e952e34d7bb3f1fc4b47538d7f8a102e8"
        }
    },
    "bin": "gn.exe"

}

problem is that scoop doesn't extract the download properly, in the cache directory I do get a .zip file, but in app folder the archive wasn't extracted. I tried adding a extract_dir and extract_to values, without success...
This is my second script, I have succesfuly created a manifest for flatbuffers. I believe something is wrong with the mime type of the downloaded content

Turns out that url is using AWS tech, and needed the content-type to be specified in the url parameter.
here is the fixed manifest

{
    "homepage": "https://gn.googlesource.com/gn/",
    "version": "1.0",
    "description": "GN is a meta-build system that generates build files for Ninja.",
    "architecture": {
        "64bit": {
            "url": "https://chrome-infra-packages.appspot.com/dl/gn/gn/windows-amd64/+/Y1n9hsOOCXCojkjl7mfsHpUuNNe7Px_EtHU41_ihAugC?response-content-type=%20application/zip%20&response-content-type=attachment%3B+filename%3Dgn-windows-amd64.zip",
            "hash": "6359fd86c38e0970a88e48e5ee67ec1e952e34d7bb3f1fc4b47538d7f8a102e8"
        }
    },
    "bin": "gn.exe"

}


panic: runtime error: invalid memory address or nil pointer dereference

Expected Behavior
Nothing inside this module panics even when called by multiple goroutines.
Actual Behavior
We see panics periodically in lambda functions that invoked very frequently.
Steps to Reproduce the Problem

Run a lambda function that is invoked very frequently and normally re-uses execution contexts
Write some metrics via MetricWithTimestamp on a timer

Specifications

Datadog Go Lambda package version: 0.7.0
Go version: 1.14.6

Stacktrace
/go/pkg/mod/github.com/launchdarkly/foundation@v0.0.0-20200818224211-4f35903f1780/fmetrics/metrics.go:161 +0x500
-- | -- | -- | -- | --
created by github.com/launchdarkly/foundation/fmetrics.Initialize
  /go/pkg/mod/github.com/launchdarkly/foundation@v0.0.0-20200818224211-4f35903f1780/fmetrics/metrics.go:173 +0x59
 github.com/launchdarkly/foundation/fmetrics.Initialize.func1(0x1853aa0, 0xc00003a2e0, 0x0, 0x0, 0x0, 0x0, 0x7d3, 0x0, 0x0, 0xc00003a03a, ...)
/go/pkg/mod/github.com/launchdarkly/foundation@v0.0.0-20200818224211-4f35903f1780/fmetrics/metrics.go:247 +0x6e
github.com/launchdarkly/foundation/fmetrics.Flush()
/go/pkg/mod/github.com/launchdarkly/foundation@v0.0.0-20200818224211-4f35903f1780/fmetrics/ddlambda/reporter.go:110 +0x68
 github.com/launchdarkly/foundation/fmetrics/ddlambda.(*Reporter).FlushOnce(0xc000100500, 0x0, 0x0)
/go/pkg/mod/github.com/launchdarkly/go-metrics@v0.0.0-20200513202621-31d7643e79e3/registry.go:65 +0xb8
github.com/launchdarkly/go-metrics.(*StandardRegistry).Each(0xc0002e73f0, 0xc004d43ff0)
/go/pkg/mod/github.com/launchdarkly/foundation@v0.0.0-20200818224211-4f35903f1780/fmetrics/ddlambda/reporter.go:156 +0xa12
 github.com/launchdarkly/foundation/fmetrics/ddlambda.(*Reporter).FlushOnce.func1(0xc0000371e0, 0x1a, 0x1498840, 0xc000088240)
 /go/pkg/mod/github.com/!data!dog/datadog-lambda-go@v0.7.0/ddlambda.go:114 +0x84
 github.com/DataDog/datadog-lambda-go.Metric(0xc00abbacc0, 0x23, 0x0, 0x0, 0x0, 0x0)
/go/pkg/mod/github.com/!data!dog/datadog-lambda-go@v0.7.0/ddlambda.go:126 +0x5a
 github.com/DataDog/datadog-lambda-go.MetricWithTimestamp(0xc00abbacc0, 0x23, 0x0, 0xbfc809c2883dcee6, 0x728092c5fb6, 0x2273140, 0x0, 0x0, 0x0)
 /go/pkg/mod/github.com/!data!dog/datadog-lambda-go@v0.7.0/internal/metrics/context.go:19
 github.com/DataDog/datadog-lambda-go/internal/metrics.GetListener(...)
  /usr/local/go/src/context/context.go:550 +0x26
  context.(*valueCtx).Value(0x0, 0x12196a0, 0xc00011cac8, 0x83dcee6, 0x83dcee600000006)
 goroutine 10 [running]:
  [signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x495dd6]
  panic: runtime error: invalid memory address or nil pointer dereference

Based on a brief review of the code involved in the stack trace, it appears that there's a data race in this module. GetContext can be called by any goroutine and it accesses wrapper.CurrentContext. wrapper.CurrentContext is written here without any locking to guard against concurrent access. Since our metric reporting is run on a timer, it can happen concurrently with the wrapper's cleanup code.

We also just added the support of Lambda Extension for Go https://docs.datadoghq.com/serverless/libraries_integrations/extension/. Whoever runs into similar issue when submitting metrics synchronously should also consider sending metrics over the Datadog Lambda Extension asynchronously.
Closing the issue due to staleness.

@tianchu, can you provide documentation for configuring the Lambda extension for go? The instrumentation docs for go do not include this information.

@apechimp I'm working on that :)

@apechimp here is the preview.


How can i redirect etcd log to file?

Here is just 3 params about logging flag, --debug, --log-package-levels and --log-output. How can i redirect etcd log to file? Thank you for your attention.

@dearboll have you tried redirecting stderr to a file?
./bin/etcd 2>log.txt

I use systemd to manage my etcd process, which params will help to do that ? @heyitsanthony

If you run with systemd, all logs are handled via journald. You can do something like
journalctl -u YOUR_SERVICE.service -l --no-pager|less

@gyuho  Thanks.
I wanna specify the log file for etcd to do some log process, rather than all in syslog. So etcd cannot specify a log file ?

@dearboll etcd has no option to specify a log file, because it can be done without it (unix pipes, journald). You can store journald logs on disk with
[Journal]
Storage=persistent

And do the log processing


Opt-in to use k3d as an alternative to kind for local dev

My development environment is docker-based. Long story, but mainly to be able to work on Linux, as my company only provides workstations with Windows pre-installed. We are working on a WSL2 setup, but that will still take some time.
And kind does not run on docker, but k3d does. We use k3d a lot internally, so each time I want to test trivy-operator on my local workstation, I have to modify the Makefile to replace the kind import image with the corresponding k3d command. This is unfortunate, so I suggest to add an opt-in to the Makefile for using k3d as an alternative to kind.
I am not sure how the UX should be, as I don't have too much experience with makefiles, but maybe a new environment variable TRIVY_OPERATOR_CLUSTER, with the supported values kind (default) and k3d?

not sure about this requirements , look like personal use case , kind is highly used and popular. maybe try to find a way to support both

maybe try to find a way to support both

Of course! :-) I am not at all suggesting to drop kind. Kind should still be the default and no changes in the UX.


Generate Black images - force restart

I'm having a problem with different models, where after some successful runs if I restart shark, the next time I load the same model, it only generates black images.

Note: if I restart shark with the flag --clear_all it doesn't help. I have to manually delete all files in temp folders and in shark folders in order for it to work.
Here is the full prompt of a run.
shark_sd_20230225_568.exe --clear_all
shark_tank local cache is located at C:\Users\victo.local/shark_tank/ . You may change this by setting the --local_tank_cache= flag
CLEARING ALL, EXPECT SEVERAL MINUTES TO RECOMPILE
vulkan devices are available.
cuda devices are not available.
Running on local URL:  http://<IP_ADDRESS>:8080
To create a public link, set share=True in launch().
Found device Radeon RX 590 Series. Using target triple rdna2-unknown-windows.
Using tuned models for CompVis/stable-diffusion-v1-4/fp16/vulkan://00000000-0800-0000-0000-000000000000.
torch\jit_check.py:172: UserWarning: The TorchScript type system doesn't support instance-level annotations on empty non-base types in __init__. Instead, either 1) use a type annotation in the class body, or 2) wrap the type in torch.jit.Attribute.
warnings.warn("The TorchScript type system doesn't support "
No vmfb found. Compiling and saving to D:\labs\image-generator-ai\engines\shark\euler_scale_model_input_1_512_512fp16.vmfb
Using target triple -iree-vulkan-target-triple=rdna2-unknown-windows from command line args
Saved vmfb in D:\labs\image-generator-ai\engines\shark\euler_scale_model_input_1_512_512fp16.vmfb.
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
No vmfb found. Compiling and saving to D:\labs\image-generator-ai\engines\shark\euler_step_1_512_512fp16.vmfb
Using target triple -iree-vulkan-target-triple=rdna2-unknown-windows from command line args
Saved vmfb in D:\labs\image-generator-ai\engines\shark\euler_step_1_512_512fp16.vmfb.
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
Checkpoint already loaded at :  D:/labs/image-generator-ai/engines/shark/models/myModel
Compiling all the models with the fetched base model configuration.
torch\fx\node.py:250: UserWarning: Trying to prepend a node to itself. This behavior has no effect on the graph.
warnings.warn("Trying to prepend a node to itself. This behavior has no effect on the graph.")
Loading Winograd config file from  C:\Users\victo.local/shark_tank/configs\unet_winograd_vulkan.json
100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████| 107/107 [00:00<00:00, 3.79kB/s] 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████| 107/107 [00:00<00:00, 3.78kB/s] Loading lowering config file from  C:\Users\victo.local/shark_tank/configs\unet_v1_4_fp16_vulkan_rdna2.json
100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████| 24.6k/24.6k [00:00<00:00, 629kB/s] 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████| 24.6k/24.6k [00:00<00:00, 733kB/s] Applying tuned configs on unet1_64_512_512_fp16_tuned_myModel_vulkan-00000000-0800-0000-0000-000000000000
No vmfb found. Compiling and saving to D:\labs\image-generator-ai\engines\shark\unet1_64_512_512_fp16_tuned_myModel_vulkan-00000000-0800-0000-0000-000000000000.vmfb
Using target triple -iree-vulkan-target-triple=rdna2-unknown-windows from command line args
Saved vmfb in D:\labs\image-generator-ai\engines\shark\unet1_64_512_512_fp16_tuned_myModel_vulkan-00000000-0800-0000-0000-000000000000.vmfb.
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
Loading Winograd config file from  C:\Users\victo.local/shark_tank/configs\vae_winograd_vulkan.json
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████| 33.0/33.0 [00:00<00:00, 1.13kB/s] 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████| 33.0/33.0 [00:00<00:00, 1.13kB/s] No vmfb found. Compiling and saving to D:\labs\image-generator-ai\engines\shark\vae1_64_512_512_fp16_tuned_myModel_vulkan-00000000-0800-0000-0000-000000000000.vmfb
Using target triple -iree-vulkan-target-triple=rdna2-unknown-windows from command line args
Saved vmfb in D:\labs\image-generator-ai\engines\shark\vae1_64_512_512_fp16_tuned_myModel_vulkan-00000000-0800-0000-0000-000000000000.vmfb.
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
No vmfb found. Compiling and saving to D:\labs\image-generator-ai\engines\shark\clip1_64_512_512_fp16_tuned_myModel_vulkan-00000000-0800-0000-0000-000000000000.vmfb
Using target triple -iree-vulkan-target-triple=rdna2-unknown-windows from command line args
Saved vmfb in D:\labs\image-generator-ai\engines\shark\clip1_64_512_512_fp16_tuned_myModel_vulkan-00000000-0800-0000-0000-000000000000.vmfb.
WARNING: [Loader Message] Code 0 : windows_read_data_files_in_registry: Registry lookup failed to get layer manifest files.
50it [01:42,  2.04s/it]
100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 [00:01<00:00,  1.03s/it]

I also got Black Images - Rx580 here

Same here black images with RX 580
Any progress on this? If you need any more detailed logs let me know.

Is it possible this requires a specific driver?

Same problem, also using an RX 580. Everything works fine on 526 and older, but 531 and all newer tags I've tried (537, 544, 568, 592) fail.
However I'm not just here to high five y'all. With AMD_ENABLE_LLPC=1, the problem goes away! I have no idea what that environment variable does, but https://github.com/nod-ai/SHARK/commit/8c241b06cb54806ecdd37c917d60f7b62bf5dc08 appears to be the faulty commit.

same problem here and it looks like AMD_ENABLE_LLPC=1 did not fix it. I am using an rx580 and when i uncheck low vram it gives me this error.
Traceback (most recent call last):
File "gradio\routes.py", line 401, in run_predict
File "gradio\blocks.py", line 1302, in process_api
File "gradio\blocks.py", line 1039, in call_function
File "anyio\to_thread.py", line 31, in run_sync
File "anyio_backends_asyncio.py", line 937, in run_sync_in_worker_thread
File "anyio_backends_asyncio.py", line 867, in run
File "gradio\utils.py", line 491, in async_iteration
File "ui\txt2img_ui.py", line 177, in txt2img_inf
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_txt2img.py", line 135, in generate_images
File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 172, in decode_latents
File "shark\shark_inference.py", line 138, in call
File "shark\shark_runner.py", line 93, in run
File "shark\iree_utils\compile_utils.py", line 385, in get_results
File "iree\runtime\function.py", line 130, in call
File "iree\runtime\function.py", line 154, in _invoke
RuntimeError: Error invoking function: D:\a\SHARK-Runtime\SHARK-Runtime\c\runtime\src\iree\hal\drivers\vulkan\vma_allocator.cc:693: RESOURCE_EXHAUSTED; VK_ERROR_OUT_OF_DEVICE_MEMORY; vmaCreateBuffer; while invoking native function hal.device.queue.alloca; while calling import;

What version of the driver and application are you on ?

I tried recreating the bug with the latest pre-release (2023-06-08, tag 761) and drivers (v23.5.2), but now I get an OOM error when generating an image. Full output:
Using tuned models for stabilityai/stable-diffusion-2-1(fp16) on device vulkan://00000000-0800-0000-0000-000000000000.
loading existing vmfb from: D:\shark\2023-06-08\euler_scale_model_input_1_512_512_vulkan_fp16.vmfb
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
loading existing vmfb from: D:\shark\2023-06-08\euler_step_1_512_512_vulkan_fp16.vmfb
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
use_tuned? sharkify: True
_1_64_512_512_fp16_tuned_stable-diffusion-2-1-base
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
20it [00:17,  1.13it/s]
torch\fx\node.py:260: UserWarning: Trying to prepend a node to itself. This behavior has no effect on the graph.
  warnings.warn("Trying to prepend a node to itself. This behavior has no effect on the graph.")
No vmfb found. Compiling and saving to D:\shark\2023-06-08\vae_1_64_512_512_fp16_tuned_stable-diffusion-2-1-base_vulkan.vmfb
Configuring for device:vulkan://00000000-0800-0000-0000-000000000000
Using target triple -iree-vulkan-target-triple=rdna2-unknown-windows from command line args
Saved vmfb in D:\shark\2023-06-08\vae_1_64_512_512_fp16_tuned_stable-diffusion-2-1-base_vulkan.vmfb.
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
WARNING: [Loader Message] Code 0 : Layer VK_LAYER_OBS_HOOK uses API version 1.2 which is older than the application specified API version of 1.3. May cause issues.
Traceback (most recent call last):
  File "gradio\routes.py", line 393, in run_predict
  File "gradio\blocks.py", line 1069, in process_api
  File "gradio\blocks.py", line 892, in call_function
  File "anyio\to_thread.py", line 33, in run_sync
  File "anyio\_backends\_asyncio.py", line 877, in run_sync_in_worker_thread
  File "anyio\_backends\_asyncio.py", line 807, in run
  File "gradio\utils.py", line 549, in async_iteration
  File "ui\txt2img_ui.py", line 183, in txt2img_inf
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_txt2img.py", line 135, in generate_images
  File "apps\stable_diffusion\src\pipelines\pipeline_shark_stable_diffusion_utils.py", line 175, in decode_latents
  File "shark\shark_inference.py", line 138, in __call__
  File "shark\shark_runner.py", line 100, in run
  File "shark\iree_utils\compile_utils.py", line 391, in get_results
  File "iree\runtime\function.py", line 130, in __call__
  File "iree\runtime\function.py", line 154, in _invoke
RuntimeError: Error invoking function: D:\a\SHARK-Runtime\SHARK-Runtime\c\runtime\src\iree\hal\drivers\vulkan\vma_allocator.cc:693: RESOURCE_EXHAUSTED; VK_ERROR_OUT_OF_DEVICE_MEMORY; vmaCreateBuffer; while invoking native function hal.device.queue.alloca; while calling import;
[ 1]   native hal.device.queue.alloca:0 -
[ 0] bytecode module@0:2972 -

No image is generated and 'Error' is displayed in the app. I'm not using SHARK any more, so this is about all the debugging I can do. Hope it helps anyway.


[Tabs][Carousel] Verify / support a nested tabs scenario

-> Implemented the nested tabs/carousel suport
-> Added a UI test for nested tabs

Q
A

Fixed Issues?
#288

Patch: Bug Fix?

Minor: New Feature?

Major: Breaking Change?

Tests Added + Pass?
Yes

Documentation Provided
Yes (code comments and or markdown)

Any Dependency Changes?

License
Apache License, Version 2.0

IIUC this PR solves 2 issues:

the nesting of tabs (changes in tabs.js)
the JSON export for the tabs (changes in the other files)
If that is correct, I would split this PR in two in order to address each specific topic.

Codecov Report

Merging #293 into development will increase coverage by 0.11%.
The diff coverage is n/a.

@@                Coverage Diff                @@
##             development     #293      +/-   ##
=================================================
+ Coverage          82.24%   82.35%   +0.11%
- Complexity           858      889      +31
=================================================
  Files                 81       82       +1
  Lines               2737     2817      +80
  Branches             508      524      +16
=================================================
+ Hits                2251     2320      +69
- Misses               199      203       +4
- Partials             287      294       +7

Impacted Files
Coverage Δ
Complexity Δ

...mponents/sandbox/internal/models/CarouselImpl.java
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

.../com/adobe/cq/wcm/core/components/models/Page.java
100% <0%> (ø)
27% <0%> (+11%)
:arrow_up:

...lets/ClientLibraryCategoriesDataSourceServlet.java
83.33% <0%> (ø)
8% <0%> (?)

...m/core/components/internal/models/v2/PageImpl.java
80.28% <0%> (+3.96%)
29% <0%> (+12%)
: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 50c83e0...b4da878. Read the comment docs.

Now the PR contains only the changes related to nested tabs. The other ones related to the Json export will be committed to issue/298 branch


fixed problem with sidebar overflow

Hey,
There's a problem with sidebar overflow. When I set a long name the sidebar goes to the second column:

Here's how it looks now:

Thanks

Thanks for the fix. I'll merge it in. By the way, I'm going to switch the files from the main branch to the gh-pages branch as well. I'll also be pushing out some other updates tonight as well.


Substitutability doesn't work when using 'this' types and intersections of type arguments

interface FooConstructor<F extends Foo = Foo> {
    new (): F;
    extend<T>(x: T): FooConstructor<F & T>
}

interface Foo {
    gimme(): this;
}

declare var Foo: FooConstructor;

Expected: Just works - F & T should be substitutable for F which is substitutable for Foo.
Actual: An error on F & T.
Type 'F & T' does not satisfy the constraint 'Foo'.
  Types of property 'gimme' are incompatible.
    Type '() => F' is not assignable to type '() => F & T'.
      Type 'F' is not assignable to type 'F & T'.
        Type 'Foo' is not assignable to type 'F & T'.
          Type 'Foo' is not assignable to type 'F'.
            Type 'F' is not assignable to type 'T'.
              Type 'Foo' is not assignable to type 'T'.

My workaround at the moment is to write F & T & Foo:
interface FooConstructor<F extends Foo = Foo> {
    new (): F;
    extend<T>(x: T): FooConstructor<F & T & Foo>
}

interface Foo {
    gimme(): this;
}

declare var Foo: FooConstructor;

No longer reproduces. Pretty sure it was fixed by #23672, but could be some other fix.


Label studio cannot export images with zero bounding box annotations because 'original_width' and 'original_height' are stored inside 'result' key

Describe the bug
Currently original_width and original_height will disappear if a user submits a task with zero bounding box annotations. This is because they are stored inside of the ["result"] key. No elements exist inside of ["result"] key if a user submits an annotation with zero bounding box annotations.
The issue this produces is that Label Studio silently ignores exporting annotations with zero bounding boxes when a user clicks export to COCO/PASCAL/YOLO. I.e. if 100 images are submitted as finished tasks, and 10 of them have zero bounding boxes, then Label Studio export to COCO/PASCAL/YOLO will only export the 90 observations which have bounding box annotations.
To Reproduce
Steps to reproduce the behavior:

Start a project with standard object detection labeling interface.

Make a bounding box annotation for one of the images and submit task. For the other image submit task with zero bounding box annotations.
Export as COCO/PASCAL. Only 1 image will be exported in both .json file and images/ folder.

Expected behavior
All submitted tasks should be exported in the COCO/PASCAL output. Submitted tasks with zero bounding boxes should also be exported but with bbox: [] .
Environment (please complete the following information):

Ubuntu 20.04
Label Studio Version 1.0.1

Additional context
If possible original_width and original_height should ideally be saved on the image level (inside of ["annotations"][0] and ["predictions"][0]. This way information about image dimensions will not disappear if users submit tasks with zero bounding box annotations.

Thank you for your report. Yes, LS reads images only in user's browsers while labeling. Data is not always accessible on the LS server side, so LS can't read image sizes while converting.

I can imagine that you are super busy, but is there perhaps any ETA on when this could be resolved?
Or do you perhaps have pointers in which files we would should to fix this to make a PR?

This is not a critical bug, it seems more like improvement. Closed as outdated and not a popular request.


Debian init script : stop action kills every other node process on the system

File : https://github.com/etsy/statsd/blob/master/debian/statsd.init
To fix bug, remove lines 92 and 93

Found the same issue today, but the script is not there anymore. Why's that? no more init.d support?

I guess they moved for systemd (jessie). So only .service file available.

Yeah but I'm running Debian 7. Well, i applied your fix and it seems to work. Removing those lines has any side-effect?

Thanks for pointing this out. As the debian support was a contribution, and the sysv script has been replaced by an upstart one, I'm closing this