Custom build tags

Are custom build tags / constraints possible?
The previous issue about this (#365) was closed as it was resolved without actually needing custom build tags, but I've got a case where I'd need to include build tags so that I can enable features in one of my (external) dependencies.

+1 this request. For my case, I need to be able to enable some extensions in go-sqlite3: https://github.com/mattn/go-sqlite3?tab=readme-ov-file#feature--extension-list
I hardcoded the tags in https://github.com/matthewmueller/heroku-buildpack-go, but it'd be great for this to land upstream.
Happy to open a PR if helpful.


Support for .wav files

Nice idea. We keep our music files in a .wav format, though, and it seems those are not yet supported.
The app per se makes a pretty neat first impression!
Edit: obviously, other formats might be interesting too, while are at it anyway. ;) I do use some wma files for example, but their amount is negligible

Hello and thanks for the hint!
I've always worked with mp3 and never considered other file formats, but it's a good point and I'll try to add them!
I'm not sure though, whether the export/share functionality will work with those (basically whether the other file formats have Support for chapters, too), will try and test it!

I know it is a long time gone, since the issue was reported, but it's finally available with .wav support as a test release. @Karrzun  if you join the  beta testing program, you should be able to get the latest version which also has support for .wav files. Could you test it and give feedback?

Support for wav files was added and is now in the released version


a new point in the release_checklist.md

https://github.com/nf-core/nf-co.re/issues/533

Cool.
Now you need to fix two style issues:
markdown/developers/release_checklist.md:13:219 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]
markdown/usage/introduction.md:48:1 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 4]

I don't think that the one in the introduction.md was introduced by you, but it should still be fixed.

I committed the changes. The GitHub is still asking for a review.  ???

I committed the changes. The GitHub is still asking for a review. ???

Yeah, that's fine. I will approve it after you've fixed the style issues.

The error message was caused by a missing whitespace character at the end of the previous paragraph.

Great. There's also one here:
markdown/usage/introduction.md:48:1 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 4]

All checks passed now. I had to delete four trailing spaces.

Lukas, I think, you have to review it again, am I right?


Next stable release

Is there any project roadmap? When we should expect another stable release? There was an introduction of bower manager and a lot of changes was merged into a master branch.

Also are there any plans on publishing to npm? There was the introduction of package.json. If so I would suggest tagging stable releases so users can npm install a specific version as needed.

New release will coming soon, I just need to resolve issue where package is too big. One solution can be is by removing the vendor folder and will just manually use bower install but still using npm then execute bower command is too tasky so maybe there is a better approach.

One solution can be is by removing the vendor folder and will just manually use bower install but still using npm then execute bower command is too tasky

IMHO vendors are always per-project. They should not be distributed among the contributors. After cloning is quite natural installing dependencies. So I'm surprised that they are placed in the repository.

Generally Bower dependencies bower_components should only be committed if you are building an application, not if you're building a library/package designed to be consumed by others.
Same goes for npm dependencies in node_modules, if they're available on npm.
Composer is a different story though.

Package is now available on npm and 1.2.0 is already available!
I'm quite confuse @RinkAttendant6, I think this is vice versa? Bower and NPM dependencies should remove if you have an app where all developer is knowledgeable on the technologies that they use.
In our case, some is still not knowledgeable on npm or bower so possibilty of moving away in themes is very high if they fail to install the required steps for npm or bower.
We add bower, npm  and other package manger (future?) so it can be friendly on the developer who know use this technologies. Also bower is also add to easily pull out new updates on our dependencies.

Can We close the ticket?

Yes I believe the issue is resolved. As for developer-friendliness with Bower and npm, to each their own I suppose.
For example, FullCalendar lists its dependencies (jQuery and moment) as npm and Bower dependencies, respectively, instead of bundling them with the library.
Regardless, thank you for publishing on npm.


Git pull fail for <IP_ADDRESS> to <IP_ADDRESS> on a todays fresh install and redirect issues

Hi all
I have two issues with technicsolder at the moment
I was using the following guides:
https://github.com/TechnicPack/TechnicSolder/wiki/Getting-Started
https://github.com/TechnicPack/TechnicSolder/wiki/How-To-Install-TechnicSolder-for-Linux-{WIP}
and consequently:
https://www.digitalocean.com/community/tutorials/how-to-install-linux-nginx-mysql-php-lemp-stack-on-ubuntu-14-04
I am using apache2.4 on Ubuntu 14.04
I do NOT use mysql, using sqlite (default)
The following was done 2015-01-27 (today), which should result in having TechnicSolder <IP_ADDRESS> and all updates from Ubuntu at this time.
installed software:
sudo apt-get install php5 php5-cli php5-curl php5-gd php5-mcrypt php5-mysql curl git apache2 ngnix php5-sqlite libapache2-mod-php5
Made a folder on my datadisk and gave it to myuser:www-data
/datadisk/technicsolder
Within this folder I put the composer:
curl -sS https://getcomposer.org/installer | php
Checked if there is an update: sudo php composer.phar self-update (there wasn't at the time)
As mentioned in the manual I downloaded from GIT, moved it in my working directory and then run the composer command:
git clone https://github.com/TechnicPack/TechnicSolder.git
moved it to /datadisk/technicsolder
then (in /datadisk/technicsolder)
php composer.phar install --no-interaction
after that I made a chown -R myuser:www-data over all files (even though only app and public would need it, I guess)
after that I did the migration:
php artisan migrate:install (which failed, because there was already a table "migration")
php artisan migrate (which ended in "nothing to migrate")
Make a "mods"-directory in /datadisk/technicsolder/public/modpack/mods/
Same permissions as the rest (myuser:www-data, 775)
Then changed the values in solder.php as mentioned'
repo_location' => '/datadisk/technicsolder/modpack/mods/'
Pointed apache (root directory) to /datadisk/technicsolder/public/
Now that technically should make me able to surf to my website
Apache has received /datadisk/technicsolder/public/ (with trailing slash) as root directory.
I put a info.php into the public folder so I could check php functionality (which worked)
When I then to go mysite.domain.ch (which points to /datadisk/technicsolder/public/) I receive a 404, because I get directly redirected to mysite.domain.ch/dashboard.
As I understand that is not okay, right?
I then manually go to mysite.domain.ch/index.php which offers me a login, which ended in an Error in exception handler.
I then deleted the app/database/production.sqlite, made a new file (touch), gave it a 775 and myuser:www-data...and made a php artisan migrate:install and migrate (which actuall did stuff this time)
The problem with the dashboard still persisted, so I went to index.php manually with the browser
I was logged in this time (automagically, but it worked) and was offered <IP_ADDRESS> (not .9) of technicsolder.
After changing the credentials I logged out and in again, and then was told there is an update available.
So I went back to ssh and did what the wiki told me (all in /datadisk/technicsolder):
php artisan down
git pull -> failed, because it said that I needed to move stuff (in app and public) and aborted then.
-> there is quite a list of files which would need moving, do I really need to do that every time?
May someone assist me in finding out what I am doing wrong? (the redirect and the fact I am still on <IP_ADDRESS> and the git pull failed)
Thanks a lot!

 The system thinks local files were changed and doesn't want to override them

Hi GenPage
Thanks a lot.
I tried, however, it appears that some files are still considered an issue:
git reset --hard v<IP_ADDRESS>:
Updating 5dfbd1e..203399egit reset --hard v<IP_ADDRESS>
HEAD is now at 5dfbd1e Update to MinecraftUtils for better timeout handling, Changed User Agent to remove spoofing.
git pull:
error: The following untracked working tree files would be overwritten by merge:
app/views/errors/500.blade.php
app/views/errors/503.blade.php
app/views/errors/missing.blade.php
public/css/errors.css
public/css/solder.css
public/img/500error2.png
public/img/background.png
public/img/title.png
Please move or remove them before you can merge.
Aborting
Thanks

Interesting, those files should of been removed with a "--hard" reset. You can manually remove using rm and then try again
Example: rm app/views/errors/500.blade.php

Hi GenPage
I manually removed all the files mentioned above (which git wanted removed) and tried another git pull...this time it worked.
the php composer.phar update worked as well.
I logged in afterwards and was greeted by <IP_ADDRESS>. So it worked
Still, the questions resides as why it happend
As for the redirect, I did what was proposed to me in http://forums.technicpack.net/topic/111376-solder-where-is-the-dashboard/
I changed the app/routes.php and that appears to be working. Maybe something to include in <IP_ADDRESS>?

Also if you are using apache, don't also install nginx. It should be one or the other. One day you may try to access Solder and it will just give you the default nginx page instead.


Add support for Stack Refill mod

I'm not sure how hard it is, but I would love it if this mod worked with the Stack Refill mod.
Thanks for this awesome mod!

Stack Refill doesn't really provide an API so I can't really do anything with it

I have recently switch to using the inventory profiles mod which also has a stack refill feature. From what I can see in their github, there is an API for their mod. Not sure if it has API's for what you would need to make it work though.


Consider introducing k.ANNOTATION_INTERFACE

@interface interfaces are pretty special (https://docs.oracle.com/javase/specs/jls/se8/html/jls-9.html#jls-9.6), so we may want to give them a special kind.

Sounds good to me 👍 I did not know about annotation types before.

Me neither. Not until https://github.com/twitter/rsc/pull/207.


Question: Why DeDuplicationStrategy class is so specific?

I developed a filtering feature on top of Kafka using flatbuffer. The filter uses headers to filter or not the message. The goal is to deserialize the payload only when the consumer accepts the message.
I finished yesterday to migrate my old spring-kafka library based on the version 0.8 to use this project and it works.
My question is:
A DeDuplicationStrategy has been created to filter the message is the message is a dupe. This strategy is great for the dedupe use case. However, this feature could be more generic to support many use cases.
My use case could fit perfectly in this strategy, however my use case is not a dedupe is a filter. So I decided to not use the DeDuplicationStrategy interface to handle this use case.
I would like to know if the DeDuplicationStrategy could be replaced by a generic interface that could support different strategies (dedupe, filtering or another use case) to not send the message to the listener.
I am hopping I am clear.
Thanks for your great work.

Hello, @jmirc !
Thank you for feedback.
Well, the feature has been introduced very recently in the current RC1 milestone: #80 .
Even if you can achieve your filtering goal with the current DeDuplicationStrategy (with some inconvenience around the interface and its method names), I agree that we can rename it into FilterStrategy, when dedup logic ins covered as well.
That is how we can implement Idempotent Receiver with the regular Spring Integration Filter: http://docs.spring.io/spring-integration/reference/html/messaging-endpoints-chapter.html#idempotent-receiver.
@garyrussell , WDYT?

Yes, fine with me; thanks for the feedback.

I am wondering if we should perhaps consider making this a Listener implementation or an advice - there are a few other concerns that could be addressed here - including retry. And at this point there is a risk for the container (which is not very simple by itself) to become very complex, IMO.

@mbogoevici , @garyrussell did that as DeDuplicatingMessageListenerAdapter already, but yes, adviceChain around the listener.onMessage() sounds good for me.
And we could move retryTemplate from puase/resume  solution to the Advice too.

Right - that's why I decided to do it as listener adapter to keep the concern out of the container itself - we could add an advising listener adapter with an advice chain. Of course, individual adapters can be chained too - avoiding reflection, proxies etc.
Yes, we could move retry to an adapter too.

PR issued - I created a separate issue #105 for the retry template extraction.


Update to latest pyav

This should fix https://github.com/OpenXbox/xbox-smartglass-nano-python/issues/29
I have found no issues with compatibility.

I’d like to add that these checks are failing because you guys are somehow using a version of ffmpeg without a focal release.


Advanced Filter Logic in version 3

[MixItUp] - Advanced Filter Logic (Checkboxes)
https://codepen.io/patrickkunka/pen/iwcap
dont work with version 3, pls upgrade sample.

The multifilter extension replaces this functionality in V3


bin/im2rec rotating images 90 degrees counter-clockwise.

I am using the bin/im2rec tool to generate a .rec file for the Kaggle Dogs vs. Cats image challenge. I used the following code to visualize the output of ImageRecordIter:
# construct the training image iterator
trainIter = mx.io.ImageRecordIter(
	path_imgrec=config.TEST_MX_REC,
	data_shape=(3, 227, 227),
	batch_size=128 * 3,
	rand_crop=True,
	rand_mirror=True,
	preprocess_threads=4)

# grab the first batch of images from the generator
images = trainIter.getdata().asnumpy()

# loop over a few images
for i in np.arange(0, 100):
	# grab the image and transpose it to RGB ordering
	image = images[i].transpose((0, 2, 1)).astype("uint8")
	(R, G, B) = (image[0], image[1], image[2])
	image = np.dstack([R, G, B])

	# write the image to file
	p = [args["output"], "{}.png".format(i + 1)]
	io.imsave(os.path.sep.join(p), image)
However, when I inspected the output files, all of the images were rotated exactly 90 degrees counter-clockwise.
I have attached a few of the examples to this note. I'm just curious if I am doing something wrong or if this is a bug in the im2rec binary?

I added a few cv2.imshow calls to the tools/im2rec.py to see if I could debug that is going on. I placed a call right after cv2.imshow and then another right before the image is being written to pack_img. As the attachment shows, the image it's not an OpenCV issue.
Either the image is being rotated as it's being read out of the .rec file or the cv2.imencode call is doing something strange. I'll see if I can look into it more.

Another update: I tried using bin/im2rec on a separate dataset (ImageNet) and I'm still seeing the 90 degree rotation issue. Still investigating what the cause is.

More updates. I defined the following function in tools/im2rec.py to handle rotating an image while ensuring parts of it are not cut off:
def rotate_bound(image, angle):
    import numpy as np

    # grab the dimensions of the image and then determine the center
    (h, w) = image.shape[:2]
    (cX, cY) = (w // 2, h // 2)

    # grab the rotation matrix (applying the negative of the angle
    # to rotate clockwise), then grab the sine and cosine (i.e.,
    # the rotation components of the matrix)
    M = cv2.getRotationMatrix2D((cX, cY), -angle, 1.0)
    cos = np.abs(M[0, 0])
    sin = np.abs(M[0, 1])

    # compute the new bounding dimensions of the image
    nW = int((h * sin) + (w * cos))
    nH = int((h * cos) + (w * sin))

    # adjust the rotation matrix to take into account translation
    M[0, 2] += (nW / 2) - cX
    M[1, 2] += (nH / 2) - cY

    # perform the actual rotation and return the image
    return cv2.warpAffine(image, M, (nW, nH))
This function will soon to be part of my imutils package.
I then modified image_encode such that:
    if args.resize:
        if img.shape[0] > img.shape[1]:
            newsize = (args.resize, img.shape[0] * args.resize / img.shape[1])
        else:
            newsize = (img.shape[1] * args.resize / img.shape[0], args.resize)
        img = cv2.resize(img, newsize)

        img = rotate_bound(img, -90)
Now when I read images back out of the ImageRecordIter they are correctly oriented. However, this is a hack and still does imply there is an issue with how either ImageRecordIter is storing or retrieving the images.

@piiswrong @tornadomeet Are these issues related to the recent fix that were pushed for #3975?

I think it was the NumPy transpose that was throwing me off. This seems to work:
# grab the first batch of images from the generator
images = trainIter.getdata().asnumpy()

# loop over a few images
for i in np.arange(0, 10):
	# grab the image and transpose it to RGB ordering
	image = images[i]
	(R, G, B) = (image[0], image[1], image[2])
	image = np.dstack([B, G, R]).astype("uint8")

	cv2.imshow("Image", image)
	cv2.waitKey(0)

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Changed README and added a .nvmrc file to ease dev env.

Fixed instructions to run dev environment in README (and changed script in package.json). Also added a .nvmrc file to let nvm know which version of node to use.
In order to try it, clone the repo and just follow the instructions in the README to set it up.

Closing as other changes have been made.


chore(deps): Bump @untemps/vocal from 1.1.1 to 1.2.1 (#47)

Closes #47

:tada: This PR is included in version 1.6.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Query issue

Thread '<unnamed>' panicked at 'called Result::unwrap()on anErr value: Error { kind: ServerSelectionError { message: "Server selection timeout: No available servers. Topology: { Type: Unknown, Servers: [ { Address: localhost:27017, Type: Unknown, Error: failed to fill whole buffer }, ] }" } }'
Is it due to an old version of MongoDB? I'm using the 3.4.9

no, I don't think the version is the issue here.
& if you are doing any complex queries then use .aggregate because the other methods are not that stable.

It is an easy find query, the same code works with a newer version of
mongodb.
The only difference is the connection string, could maybe be that?
On Sat, 13 Jun 2020 at 23:24 ASAD<EMAIL_ADDRESS>wrote:

no, I don't think the version is the issue here.
& if you are doing any complex queries then use .aggregate because the
other methods are not that stable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/manyuanrong/deno_mongo/issues/51#issuecomment-643716627,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLLLL6U4I2WM7ZSHGQVYHLRWRGGJANCNFSM4N5BZLUA
.

wait connection string well deno_mongo doesn't support connection string though you have to connect manually by doing this:-
import { MongoClient } from<EMAIL_ADDRESS>
const client = new MongoClient();
client.connectWithUri("mongodb://localhost:27017");

const db = client.database("test");
const users = db.collection("users");

more about that here  #41

This is what I was trying, it works if I insert the connection URL to ATLAS:
mongodb+srv://myuser:mypwd@cluster0-bo2mg.mongodb.net/mydbname?retryWrites=true&w=majority

Here an example of the issue also with a new version of mongodb:
https://github.com/lucarducci/deno-mongo-debug
Run with:
docker-compose up
If I use an ATLAS connection string it works. Not with a local db or from docker.

Any update on this issue?

Can you try with MongoDB 4.0 or 4.2? Also make sure the links in your docker-compose are set up correctly. localhost:27017 might not be the correct address to communicate with your MongoDB cluster in a different container.


Add up alias to self update command

I can't remember where I saw this mentioned but very easy.
I also cleaned up the subcommand definitions. Unless they were defined that way to have different docstrings, in which case I can undo those changes.

Unless they were defined that way to have different docstrings, in which case I can undo those changes.

Yes, that was the idea :)

Fixed


Switching buckets keeps all other query params

Request: Don't reset query params when you switch buckets
Short description:

Current state:

choosing new bucket from dropdown resets any filters or zooming you've done, which can be unpleasant.

Desired state:

all query parameters are kept EXCEPT for the bucket (which updates to whichever new one you chose).

implemented in #241


Platform<EMAIL_ADDRESS>running on Android 10 : getFile produces INVALID_MODIFICATION_ERR - Error code 9.

Bug Report
Problem
What is expected to happen?
fileSystem.root.getFile Should trigger the success callback.
What does actually happen?
On Android 10, it triggers the error callback with FileError {code: 9} (INVALID_MODIFICATION_ERR).
On Android 9 it works as expected - the success callback is triggered.
Information

Issue replicated with a physical Pixel 2 & the Pixel 2 emulator from android studio.
I wasn't getting the issue with<EMAIL_ADDRESS>/ API 28, it's just been introduced when switching to<EMAIL_ADDRESS>/ API 29.
 This gets triggered on Android 10 !!! //
			}
		);

	},
	function(error) {	// FS Failure
		console.log('requestFileSystem', error);
	}
);

Output:
getFile FileError {code: 9}
Environment, Platform, Device

Android 10.0 / Pixel 2 API 29 emulator
Also replicated on a real Pixel 2 / Android 10.
Works correctly on Android 9 / Samsung S8.
Version information

Cordova
9.0.0<EMAIL_ADDRESS>Platform
<EMAIL_ADDRESS>cordova-plugin-file
6.0.1 & 6.0.2
Android 10.0 / Pixel 2 API 29 emulator
Android Studio 4.0.1
Windows 10.
Checklist

[x] I searched for existing GitHub issues
[x] I updated all Cordova tooling to most recent version
[x] I included all the necessary information above

Definitely sounds like an API 29 restriction/change. Does it work on cordova-android@9 when targeting API 28?
Try setting the android-targetSdkVersion to 28. If it works, i think this will prove that there was no code change that caused the problem, just simply the target API upgrade.

Yeah I think you're right, that worked with cordova-android@9 and android-targetSdkVersion set to 28.

I think Scoped Storage is the change in API 29 that is causing this.

Gave it a go saving to another location to see if it would get around that permission, seems like saving to the app data directory cordova.file.dataDirectory works.
fileSystem.root seems to be outside of the app data perhaps? - I didn't realise this since the docs say "fs.root is a DirectoryEntry object that represents the persistent storage in the sandboxed file system."
Also noticed:

If the file was created by the API 28 app, the API 29 app seems to be able to write to it, it just can't create new files.

In case it's useful for anyone else that only needs to store files in the app dir, the change to use cordova.file.dataDirectory is:
window.resolveLocalFileSystemURL(cordova.file.dataDirectory,	// <-----------
	function (dirEntry) {	// <-----------

		dirEntry.getFile("test2.txt", {create: true, exclusive: false},	// <-----------
			function(fileEntry) {	// File loaded/created

				fileEntry.createWriter(
					function(writer) {	// Writer created

						writer.write('hello');
						console.log('test2 it worked!');

					},
					function(error) {	// Writer failure
						console.log('createWriter', error);
					}
				);

			},
			function(error) {	// File failure
				console.log('getFile', error);
			}
		);

	},
	function(error) { console.log('resolveLocalFileSystemURL', error); }
);

That's interesting... fs.root appears to return a directory that looks something like: file:///data/user/0/<app-id>/files/files/ and that should still work I think. /data/user/0/<app-id> I believe is a symlink to /data/data/<app-id>/.
When I use the dataDirectory method like yours, then the file path becomes: file:///data/user/0/<app-id>/files/
Perhaps files/files/ is not created yet?
I tested the original code that produced the error for you on my android 10 device and it worked for me, but my app ensures that files/files/ directory is created (cause that is where we actually store our files).
This is starting to look less like a scoped storage issue and more like something is just not being initialised properly issue.

setting "android-targetSdkVersion" to 28 is only a temporary solution.
from August 2020 android will allow new apps only with "android-targetSdkVersion=29" and existing apps from November.
is someone working on a solution for it?

setting "android-targetSdkVersion" to 28 is only a temporary solution.
from August 2020 android will allow new apps only with "android-targetSdkVersion=29" and existing apps from November.
is someone working on a solution for it?

I have only briefly looked at this issue, but my time is rather limited. PRs are definitely welcome and appreciated.

@breautek - I will try to see if I can fork a solution.

fix added inside this PR https://github.com/apache/cordova-plugin-file/pull/410

@breautek - to reproduce it you must have an Android 10 and set sdk version to 29. uninstall your app, build and run.

Hi, i also had this issue,
My error is, code 12 PATH_EXISTS_ERR
when i try to create a folder
global.resolveLocalFileSystemURL( cordova.file.externalRootDirectory, function (dirEntry) {
    dirEntry.getDirectory('KMSBalita', { create: true }, function (subDirEntry) {

            ....
    },
    function( error ){

            console.log( error )
    })

})
now i solve this by change target api to 28 instead of 29 but only temporary
Mostly error from android 10.

@hariagustian can you install the PR https://github.com/apache/cordova-plugin-file/pull/410 and let us know if the PR solves your issue when targeting API 29?

The ImagePicker Plugin has a similar Problem and it was solved with the same Fix.

I can confirm that the same line from PR #410 put manually in config.xml solves the problem.
I could not install PR 410 since that exact line was conflicting with an edit-config tag we have i our config.xml.
Just to add some details, we had this line in our config.xml:
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">
   <application android:usesCleartextTraffic="true" />
</edit-config>

Installing @eliadAfeka 's fork gave us this error:
Failed to install 'cordova-plugin-file': Error: cordova-plugin-file cannot be added. <edit-config> changes in this plugin conflicts with <edit-config> changes in config.xml. Conflicts must be resolved before plugin can be added.

So we edited our config.xml to look like this:
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">
   <application android:usesCleartextTraffic="true" android:requestLegacyExternalStorage="true"/>
</edit-config>

and the problem seems to be solved.
Thanks everyone for your help.

@rastafan But it would have worked if you removed your Config, so i think it is ready to merge

I can confirm that the same line from PR #410 put manually in config.xml solves the problem.
I could not install PR 410 since that exact line was conflicting with an edit-config tag we have i our config.xml.
Just to add some details, we had this line in our config.xml:
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">
   <application android:usesCleartextTraffic="true" />
</edit-config>

Installing @eliadAfeka 's fork gave us this error:
Failed to install 'cordova-plugin-file': Error: cordova-plugin-file cannot be added. <edit-config> changes in this plugin conflicts with <edit-config> changes in config.xml. Conflicts must be resolved before plugin can be added.

So we edited our config.xml to look like this:
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">
   <application android:usesCleartextTraffic="true" android:requestLegacyExternalStorage="true"/>
</edit-config>

and the problem seems to be solved.
Thanks everyone for your help.

I had the same issue and it was fixed of the comment added  by @rastafan(quoted reply) in my Samsung phone - Android 10. I added the code - "android:usesCleartextTraffic="true" android:requestLegacyExternalStorage="true"" in the directory /plaforms/android/app/src/main/AndroidManifest.xml
My AndroidManifest file looks like this:
<?xml version='1.0' encoding='utf-8'?> <manifest android:hardwareAccelerated="true" android:versionCode="#" android:versionName="#.###.##" package="com.projectName.projectName" xmlns:android="http://schemas.android.com/apk/res/android">     <supports-screens android:anyDensity="true" android:largeScreens="true" android:normalScreens="true" android:resizeable="true" android:smallScreens="true" android:xlargeScreens="true" />     <uses-permission android:name="android.permission.INTERNET" />     <application android:hardwareAccelerated="true" android:icon="@mipmap/ic_launcher" android:label="@string/app_name" android:networkSecurityConfig="@xml/network_security_config" android:requestLegacyExternalStorage="true" android:supportsRtl="true" android:usesCleartextTraffic="true">         <activity android:configChanges="orientation|keyboardHidden|keyboard|screenSize|locale|smallestScreenSize|screenLayout|uiMode" android:label="@string/activity_name" android:launchMode="singleTop" android:name="MainActivity" android:screenOrientation="portrait" android:theme="@android:style/Theme.DeviceDefault.NoActionBar" android:windowSoftInputMode="adjustResize">             <intent-filter android:label="@string/launcher_name">                 <action android:name="android.intent.action.MAIN" />                 <category android:name="android.intent.category.LAUNCHER" />             </intent-filter>         </activity>         <provider android:authorities="${applicationId}.fileOpener2.provider" android:exported="false" android:grantUriPermissions="true" android:name="io.github.pwlin.cordova.plugins.fileopener2.FileProvider">             <meta-data android:name="android.support.FILE_PROVIDER_PATHS" android:resource="@xml/opener_paths" />         </provider>     </application>     <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />     <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />     <uses-permission android:name="android.permission.REQUEST_INSTALL_PACKAGES" />     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" /> </manifest>
and also to add, I used the cordova.file.externalRootDirectory to save the file in the android device. I don't know yet if the fixed would work in the lower android API version up to 22.
Special thanks to this thread: @rastafan @breautek @peterfox1 @HansKrywaa @hariagustian @eliadAfeka

So we edited our config.xml to look like this:

and the problem seems to be solved.

I can confirm that the same line from PR #410 put manually in config.xml solves the problem.
I could not install PR 410 since that exact line was conflicting with an edit-config tag we have i our config.xml.
Just to add some details, we had this line in our config.xml:
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">
   <application android:usesCleartextTraffic="true" />
</edit-config>

Installing @eliadAfeka 's fork gave us this error:
Failed to install 'cordova-plugin-file': Error: cordova-plugin-file cannot be added. <edit-config> changes in this plugin conflicts with <edit-config> changes in config.xml. Conflicts must be resolved before plugin can be added.

So we edited our config.xml to look like this:
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">
   <application android:usesCleartextTraffic="true" android:requestLegacyExternalStorage="true"/>
</edit-config>

and the problem seems to be solved.
Thanks everyone for your help.

It magically worked. Thank you everyone!

Hi I'm having this problem when trying to add the android:requestLegacyExternalStorage
Can anyone help me how to fix this exactly?
Here is the error:
"AAPT: error: attribute android:requestLegacyExternalStorage not found."
I already update my SDK idk what to do anymore. TIA
Ionic Info:
Ionic CLI                     : 5.4.16
Ionic Framework               : @ionic/angular 4.11.2
@angular-devkit/build-angular : 0.801.3
@angular-devkit/schematics    : 8.1.3
Cordova:
Cordova CLI       : 9.0.0<EMAIL_ADDRESS>Cordova Platforms : android 8.1.0
Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 5.0.0, (and 23 other plugins)
Utility:
cordova-res                          : not installed
native-run (update available: 1.0.0) : 0.3.0
System:
NodeJS : v12.16.1 (C:\Program Files\nodejs\node.exe)
npm    : 6.13.4
OS     : Windows 10

@apolunderblade I think you need to update your Cordova CLI to version 10.0.0 and your Cordova Android to 9.0.0 before you can add the requestLegacyExternalStorage flag.

@apolunderblade I think you need to update your Cordova CLI to version 10.0.0 and your Cordova Android to 9.0.0 before you can add the requestLegacyExternalStorage flag.

I see let me try! Thanks

I am also facing this issue. now is there actually some upcoming fix for this?
as I understand, the mentioned fixes are just workarounds to enable some legacy mode.

hi,
is there any news with the version? we still get the modification error.
when you are going to insert the PR?

hi,
is there any news with the version? we still get the modification error.
when you are going to insert the PR?

The provided PR is currently in a broken state: see https://github.com/apache/cordova-plugin-file/pull/410#issuecomment-671354675

I am also facing this issue. now is there actually some upcoming fix for this?
as I understand, the mentioned fixes are just workarounds to enable some legacy mode.

The requestLegacyExternalStorage flag will be a stopgap measure to make existing code in API 29. It will probably break again in future API versions, we'll need investigate and find out what is involved in using the new file storage system, which will likely be a breaking change.

Hi everyone, I have the same problem: I have a cordova 9.0.0 app that works on Android 9 and doesn't work on Android 10. If I put SdkTarget to 28 everything ok, but I have to use sdk 29, so I would like to find a solution, I can't figure out if I have to modify the config.xml of my app, if I do it it gives me compilation error .. Can you help me please? Thanks

Hi @Rei66RM, can your provide your code in config.xml file (loc: /config.xml) & the AndroidManifest.xml file located in - /plaforms/android/app/src/main/AndroidManifest.xml to examine the diff of my existing cordova app. Thanks!

Hi @Rei66RM, can your provide your code in config.xml file (loc: /config.xml) & the AndroidManifest.xml file located in - /plaforms/android/app/src/main/AndroidManifest.xml to examine the diff of my existing cordova app. Thanks!

HI Pepenero, Are you Italian?
However it seems that I have solved it. Reading the various comments in the post, I have updated cordova and CLI, and now access to the filesystem is ok, also I have installed the plugin to check permissions and am still working to develop an effective file access strategy (mine app makes extensive use of it). I struggle to find valid documentation about managing files in cordova, indeed if anyone knows any efficient plugin, or documentation, if they can post it I would be very grateful. Greetings from Italy!
 I'm from the Philippines @Rei66RM - yeah! you already targetted the android API 29 0 <preference name="android-targetSdkVersion" value="29" /> nice!  and about the docs in Cordova I guess they always have the latest update news in their blog post here then some of their updates the code & docs and some of its code only. Based on my side, I always refer to the GitHub reported issues discussion thread in either plugins or platforms if I had a problem in my build. Thanks!

I can confirm that the same line from PR #410 put manually in config.xml solves the problem.
I could not install PR 410 since that exact line was conflicting with an edit-config tag we have i our config.xml.
Just to add some details, we had this line in our config.xml:
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">
   <application android:usesCleartextTraffic="true" />
</edit-config>

Installing @eliadAfeka 's fork gave us this error:
Failed to install 'cordova-plugin-file': Error: cordova-plugin-file cannot be added. <edit-config> changes in this plugin conflicts with <edit-config> changes in config.xml. Conflicts must be resolved before plugin can be added.

So we edited our config.xml to look like this:
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">
   <application android:usesCleartextTraffic="true" android:requestLegacyExternalStorage="true"/>
</edit-config>

and the problem seems to be solved.
Thanks everyone for your help.

worked seamlessly

Can anyone help me?
I am getting NO_MODIFICATION_ALLOWED error when i try to remove any file using ionic File plugin's remove method.
Note : I am passing correct dirPath and filename
code :
await this.fileCtrl.removeFile(dirPath,filename).then((res) =>{
alert('successfully deleted' + filename)
},(err) =>{
alert(JSON.stringify(err))
});

Hi everyone. It's 2022 and I'm having this issue again.
android-targetSdkVersion: 30 (Play Store Requirements)
Cordova CLI       : 11.0.0
Cordova Platforms : android 10.1.1
<edit-config file="app/src/main/AndroidManifest.xml" mode="merge" target="/manifest/application">             <application android:usesCleartextTraffic="true" android:requestLegacyExternalStorage="true"  />         </edit-config>
Already added to config.xml
Error Code 9 still coming up. Has anything changed about this for SDK 30?

android:requestLegacyExternalStorage is forcefully off on API 30. That flag was only used on API 29 to revert to previous behaviour to give you time to do any necessary migrations before API 30.
I believe an invalid modification error on API 30 generally occurs when you're attempting to write to an external storage path that is no longer writable, or that your app doesn't have permission to write to (such as modifying a file that wasn't created by your app).

@breautek I'm experiencing the same problem. I'm trying to write to file:///storage/emulated/0/Download/, which in theory should be an allowed path

If anyone has this problem
<edit-config file="app/src/main/AndroidManifest.xml" target="application" mode="merge">
        <application android:requestLegacyExternalStorage="true" />
</edit-config>

Still works when targeting android-30. Just be sure to have xmlns:android="http://schemas.android.com/apk/res/android" on the widget tag

If anyone has this problem
<edit-config file="app/src/main/AndroidManifest.xml" target="application" mode="merge">
        <application android:requestLegacyExternalStorage="true" />
</edit-config>

Still works when targeting android-30. Just be sure to have xmlns:android="http://schemas.android.com/apk/res/android" on the widget tag

I have tried by using these config in my config.xml file. But it still does not seems to resolve the issue.
Also, I have noted that in app/src/main/AndroidManifest.xml, the key requestLegacyExternalStorage is already present. So I do not think so that we would require to add this key again through config.xml file.
Issue: Getting {"code": 9} as error when trying to create a new file in the cordova.file.externalRootDirectory.
Cordova : v11
Android: v31
Issue is still present even in the newer Cordova version. Can anyone suggest a workaround for the same ?

The requestLegacyExternalStorage flag is an API 29 only flag. It was added to make API 29 revert back to the old filesystem as it worked in API 28. Without it, API 29 will use the newer restricted filesystem. API 30 or later ignores this flag.

Issue: Getting {"code": 9} as error when trying to create a new file in the cordova.file.externalRootDirectory.

The externalRootDirectory directory is no longer a writable path with the Android's new filesystem. Google doesn't explicitly say which paths are no longer accessible, but it does vaguely say that some paths that used to be writable will no longer be accessible.

If your app uses the legacy storage model and previously targeted Android 10 or lower, you might be storing data in a directory that your app cannot access when the scoped storage model is enabled...

In my testing, reading & writing of the root external directory is no longer allowed. You can access existing subdirectories, such as Pictures/ or Downloads/. You can read from these directories if you have READ_EXTERNAL_STORAGE permission granted, or if your app is the creator of the content. You can write to these directories without any additional permissions (WRITE_EXTERNAL_STORAGE permission is now obsolete).
I believe creation of new directories is also forbidden, except for your application's external data directory.
My best advice is to evaluate whether you actually need external storage. Are you trying to make your files shareable between apps? If not, then perhaps you should be using the internal storage directories instead, e.g. cordova.file.dataDirectory. It will make your life significantly easier. Otherwise, you may need to do refactoring to ensure you're operating within the constraints of scoped storage.
I've already explained this a few times in the past on this thread, so I'll be locking this thread. It's an older thread, if you continue having an issue which you believe is a bug in this plugin, I suggest opening a new issue with more modern/up-to-date information.


Add indicator for the selected sort option

Testing #103615
Add a "tick mark" to indicate the selected sort option:

That's all I see

@jrieken, it looks like the Views item is not present in the context menu when you search (expected @sandy081). If you clear the search field, you should see something similar in the context menu depending on the view that you have open:

I see the view thing but no "sort by" menu

sorry, my bad. it looks like indeed you need to run a search (ex: foo), then in the filter context menu there is an option to sort the search results. If you pick any of the options, the next time you open the context menu, you should see a tick that indicates the current sorting option:

Search for extensions in Marketplace by typing some text or use the predefined filters in the filter menu
Choose sort options from Sort by sub menu
Make sure that check mark is shown for the sort option that is used.

wow - it took me very long to find that menu


ValueError: Some values in recency vector are larger than T vector when data has no issue.

I am trying to use the BetaGeoBetaBinomFitter, here is my fitting configuration
bgbbf = BetaGeoBetaBinomFitter(penalizer_coef=0.001)  bgbbf.fit(frequency = df_ch["frequency_cal"],            recency = df_ch["recency_cal"],            T = df_ch["T_cal"],            n_periods = 30,            weights = None,            verbose = True,            tol = 0.001)
However it always raises an error saying "Some values in recency vector are larger than T vector". I checked the data, such issue does not exist, and the same dataset was able to be fitted using BetaGeoFitter. I am wondering if there is any issue with the BetaGeoBetaBinom Fitter?

Hey @littlesweet1129,
BetaGeoBetaBinomFitter.fit() does not contain a T argument. Instead, use n_periods = df_ch["T_cal"].
It's important to note BetaGeoBetaBinomFitter was originally developed for use cases involving discrete purchase periods (ie, restaurants with weekend brunch specials, a concert hall with an event calendar, etc.) If this does not describe your use case, I'd advise using another model.
I'm glad you brought this matter up, because I'm currently working on a successor library to lifetimes and will need to highlight these differences in the documentation:
https://github.com/ColtAllen/btyd

@ColtAllen Thank you for getting back to me! Yes, some more clarification around the model difference would definitely be helpful. I originally thought from reading the paper, n_periods were the number of time period between the discrete times of purchases? Is that not the case?

If you go to Section 2.1 of the paper, it describes n as transaction opportunities, which is treated equivalently as 'n_periods' in the code. It also seems to be defined differently than 'T', which is based on the customer's perspective rather than the total opportunities available to all customers, so I'll modify my above message.
I responded to a similar memory overload issue in my repo for the successor library:
https://github.com/ColtAllen/btyd/issues/55

@ColtAllen - I also had the exact issue when I used the below fitter code
bgf = BetaGeoFitter(penalizer_coef=0.001) bgf.fit(clv['frequency'], clv['recency'], clv['T'])
Only difference is my T values are in terms of years. So, they range from 1 to 5 years whereas my recency values are in days.
So, is it mandatory that they all have to be in same units (and we should stick to lifetimes way of representing/defining R,F,M and T). Actually I did my own definition of Recency, Frequency, Monetary and T and used fitter (from lifetimes) to build a model. But it doesn't work. So, am I doing anything wrong?

Hey @SSMK-wq,
Are you using BetaGeoFitter instead of BetaGeoBetaBinomFitter? T (or n_periods for the latter) and recency need to be the same datetime units, so be sure to use the freq argument in summary_data_from_transaction_data to convert if needed.
Just to clarify for BetaGeoBetaBinomFitter, T cannot be used with this model because it expects a scalar n_periods representing total purchase opportunities, regardless of customer age. For example, if you're modeling concert attendance and there were 40 concerts, then n_periods=40 for all customers. Inputting an array with multiple values will break the model. I'm planning to change this parameter to an integer in an upcoming btyd release so that this doesn't happen, but for now, create an n_periods column via clv.assign(n=n_periods)

@ColtAllen - Thanks this issue is solved from my end. Aligning them using utility function solved the issue.


Unable to authenticate through browser or cli

Describe the bug
When using the following config.properties
OKTA_ORG=<my_org>.okta.com
OKTA_AWS_APP_URL=https://my_org.okta.com/home/amazon_aws/token/three_numbers
OKTA_BROWSER_AUTH=false

I receive the following error message before the program crashes:
Exception in thread "main" java.lang.RuntimeException: An unhandled error occurred. Please consult the server response above.
	at com.okta.tools.saml.OktaSaml.getSamlResponseForAwsFromDocument(OktaSaml.java:79)
	at com.okta.tools.saml.OktaSaml.getSamlResponseForAwsRefresh(OktaSaml.java:62)
	at com.okta.tools.saml.OktaSaml.getSamlResponse(OktaSaml.java:47)
	at com.okta.tools.OktaAwsCliAssumeRole.doRequest(OktaAwsCliAssumeRole.java:130)
	at com.okta.tools.OktaAwsCliAssumeRole.run(OktaAwsCliAssumeRole.java:100)
	at com.okta.tools.WithOkta.main(WithOkta.java:28)

with the server response at the bottom of this post.
When doing the same with OKTA_BROWSER_AUTH=true, I reach the correct login page, but completing the form there doesn't trigger USER_AUTH_COMPLETE.countDown(), and the jfx browser continues to the fully functional AWS console for my organization.  Once I close the window, withOkla exits(1) without creating a profile or requesting a temporary awscli key and secret.
Additional context
A couple flags I noticed and tried to work with before bringing this up:

The login form is never rendered in the cli-only variation of the authflow, but it is both rendered and fully functional in the browser version.
Naively changing OktaSaml.getSamlResponseForAwsFromDocument() to retrieve SigninToken just results in a base64 conversion error.
Altering OktaSaml.isPasswordAuthenticationChallenge() or OktaSaml.isProptForFactorChallenge() to return true results in an infinite loop with the same page being returned each time the initial request is retried with the new auth parameters.
Enabling noscript and then navigating to the OKTA_AWS_APP_URL in my browser results in this same page being returned.

<!doctype html>

  <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  <meta name="robots" content="noarchive">
  <meta name="googlebot" content="noarchive">
  <meta name="robots" content="noindex">
  <meta http-equiv="X-UA-Compatible" content="IE=Edge">
  <meta name="apple-mobile-web-app-capable" content="yes">
  <meta name="googlebot" content="noindex">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <meta name="robots" content="none">
  <link href="https://ok2static.oktacdn.com/assets/img/icons/favicons/favicon-16x16.c55b69ae49b08edc7c000d12b8e5483f.png" rel="icon" type="image/png" sizes="16x16">
  <link href="https://ok2static.oktacdn.com/assets/img/icons/favicons/favicon-32x32.99bc356b6e293b927f9e3a2b69761c26.png" rel="icon" type="image/png" sizes="32x32">
  <link href="https://ok2static.oktacdn.com/assets/img/icons/favicons/favicon-96x96.de98828614fa33ca04fcfaa07679f345.png" rel="icon" type="image/png" sizes="96x96">
  <meta name="msapplication-TileColor" content="#ffffff">
  <meta name="msapplication-TileImage" content="/img/icons/favicons/ms-icon-144x144.png">
  <meta name="application-name" content="Okta">
  <meta name="theme-color" content="#ffffff">
  <meta name="msapplication-config" content="/img/icons/favicons/browserconfig.xml">
  <title>MyOrg - Signing in...</title>

Javascript is disabled on your browser.

Please enable Javascript and refresh this page to use Okta.

   <link href="https://ok2static.oktacdn.com/assets/css/sections/interstitial.c22f9cc988086c3b50de2f50b83669e5.css" type="text/css" rel="stylesheet">

Signing in to AWS - Prod

   <form id="appForm" action="https://signin.aws.amazon.com/federation" method="GET" data-okta-status="">
    <input type="hidden" name="Action" value="login">
    <input type="hidden" name="Issuer" value="http://www.okta.com/">
    <input type="hidden" name="Destination" value="https://console.aws.amazon.com/">
    <input type="hidden" name="SigninToken" value="">
   </form>

Thank you for the detailed report, @Nyefan.
I think I see the problem. The code here looks for a form POSTing to https://signin.aws.amazon.com/saml and your Okta Amazon Web Services app being POSTed to https://signin.aws.amazon.com/federation.
Are you using the Okta Application Network-provided Amazon Web Services app or did you configure it directly using a generic Okta SAML app?
Either way there's a bug in this code in that a working Assertion Consumer Service URL for AWS isn't usable.

I have confirmed with our Okta admin that we are using the former.

I have confirmed with our Okta admin that we are using the former.

I interpret that to mean you are using the Okta Application Network-provided Amazon Web Services app.
There is no SAMLResponse on the page this tool is getting back when it loads your OKTA_AWS_APP_URL. I suspect that setting is subtly incorrect. Your Okta administrator can get the correct link for you from the Embed link field in the App Embed Link section of the General tab and the  Amazon Web Services app in Okta Admin. Again, it has to be the real AWS app, and not a bookmark app.
The OKTA_AWS_APP_URL should be like https://acmecorp.oktapreview.com/home/amazon_aws/0oa5zrwfs815KJmVF0h7/137 (note the /home/amazon_aws/) and it needs to be the real Amazon AWS Okta app and not a bookmark app.
A URL like https://acmecorp.oktapreview.com/home/bookmark/0oa5zrwfs815KJmVF0h7/137 (note the /home/bookmark/) will not work.

I’m going to assume this is resolved.


SortableContext - Property "children" is missing in type but required in type "Props"

I have an issue:
Property 'children' is missing in type '{ items: string[]; }' but required in type 'Props'
Property "children" is market as optional in DndContext but not in SortableContext.
Is it correct or is it me doing something wrong?

Hey @Kuzry, you're not doing anything wrong, the types should allow for children to be optional in SortableContext. Having said that, I'm not sure how useful <SortableContext> will be if you don't provide it children.
I'll have this fixed in a future release.

@clauderic Thanks for Your answer.
This generates an error in webpack compilation with TypeScipt. Is there currently a solution for this problem?

Pass JSX children to SortableContext


fix: column adjuster handle

here is the sum of what we discussed this morning about col adjuster, thanks @cbt1 for this

:tada: This PR is included in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bump templates lib

This PR was successfully deployed to production by CircleCI build 541 at 2018-10-23 10:41 BST.


Intra-text_Adornment readme is full of spacing mistakes

Intra-Text Adornment's README.md looks like it was copy/pasted from somewhere and spacing errors were never resolved.

*F5or choose the *Debug > Start Debuggingmenu command.
comment out the line**#define HIDING_TEXTin ColorAdornmentTagger.cs.

Thanks - fixed.


Fix minor debugging warning on Windows

gym outputs ?[33mWARN: gym.spaces.Box autodetected dtype as <class 'numpy.float32'>. Please provide explicit dtype.?[0m on Windows whenever it's imported. This fixes it by lowering the debug level.

lgtm, thanks!


Bug in mergeMap with large streams & concurrency

Spouts.range(0, 1_000_000).mergeMap(10,Spouts::of).toList()

Can result in an infinite loop, ActiveSubscribers not being close properly (activeIndex inside processRequests is never incremented, so if there is more than one ActiveSubscriber we always process the same one!).

Merged


Where to put Python display objects such as GeoJSON

@rgbkrk @gnestor
We have a Python display object GeoJSON in this repo, but nteract and other frontends would need it as well. It would be silly for users of nteract to install this package just to get that class. The same issue will show up for Vega and VegaLite display objects. These objects should inherit from the IPython.display base display classes to give them nice URL and file handling. But the question is where should we put them so users of all frontends can use them easily?
All of the other ones are in IPython.display. Maybe put them there? Create a new repo/projects for each of the classes?

I think it makes sense in the mainline IPython.display package, especially for geojson. For other media types that are more specialized (or even just with more flux) we could have a separate repo -- like for Vega.

OK, because the GeoJSON class is pure python, I don't think there are any
problems with putting them in IPython. I can submit a PR for that. I just
opened an issue on Altair about the Vega/VegaLite ones...
Where did you put the plotly and table ones?
On Sun, Feb 5, 2017 at 10:23 PM, Kyle Kelley<EMAIL_ADDRESS>wrote:

I think it makes sense in the mainline IPython.display package, especially
for geojson. For other media types that are more specialized (or even just
with more flux) we could have a separate repo -- like for Vega.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jupyterlab/jupyterlab_geojson/issues/18#issuecomment-277589371,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABr0OkzT7Mwz8QQVf4k3Smble2wwgD1ks5rZq5ggaJpZM4L3vla
.

--
Brian E. Granger
Associate Professor of Physics and Data Science
Cal Poly State University, San Luis Obispo
@ellisonbg on Twitter and GitHub
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

For Plotly we only documented the mimetype and worked with a few implementations across Python, R, and Julia to export it.

For the table display, that's still in progress in pandas.

@gnestor, can you submit a PR to ipython/ipython with the GeoJSON object
(put into IPython.core.display and export in all) and @mention me on it?
On Mon, Feb 6, 2017 at 4:11 AM, Kyle Kelley<EMAIL_ADDRESS>wrote:

For the table display, that's still in progress in pandas.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jupyterlab/jupyterlab_geojson/issues/18#issuecomment-277663897,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABr0Lg_bWLulcfylC0o5OlwyZEBD2aNks5rZw3ugaJpZM4L3vla
.

--
Brian E. Granger
Associate Professor of Physics and Data Science
Cal Poly State University, San Luis Obispo
@ellisonbg on Twitter and GitHub
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

There ya go!


How to enable the ability to scroll a page with Matter.js?

I want to enable scrolling in a page that contains a Matter.js canvas. I have found and tried a few solutions such as removingEventListener without success. I created a post on StackOverflow but no luck with answers. Therefore I decided to come directly to the source of knowledge.
How can I enable page scrolling in a page with Matter.js?  Thank you in advance.
Here's the Codepen and the post.
// Why is this not working?
mouseConstraint.mouse.element.removeEventListener(
  "mousewheel",
  mouseConstraint.mouse.mousewheel
);
mouseConstraint.mouse.element.removeEventListener(
  "DOMMouseScroll",
  mouseConstraint.mouse.mousewheel
);

The answer is provided by Xander Gottlieb in a blog post along with many other tips Matter.js— The Missing Tutorial.
By default, MouseConstraint captures scroll events. The following snippet fixes it.
mouseConstraint.mouse.element.removeEventListener("mousewheel", mouseConstraint.mouse.mousewheel);
mouseConstraint.mouse.element.removeEventListener("DOMMouseScroll", mouseConstraint.mouse.mousewheel);

@bouraqadi thank you for your reply.
I don't think you had a look into the CodeSandbox I shared. I am using that snippet and the scroll still not working. The article was the first I tried without success. I am unsure if I am adding to in the wrong place. Could you perhaps have a look into the code and tell me if something is out of place regarding the snippet?
Thank you in advance.

In your HTML the  the overflow should be set to scroll

@bouraqadi indeed you are correct. Thank you for your help.

Yeah I think you may need to do something like the above, otherwise consider switching to your own input handling and rendering to get the control you need.

I'm still unable to solve the problem in touch screen devices.

@ayank007 I give a potential fix here : https://github.com/liabru/matter-js/issues/678#issuecomment-1371233031

This issue have been haunting me since a few days and I have tried the methods mentioned without success. At certain point I realized that interacting with the bodies while scrolling is impractical - despite would be nice - and likely to cause the same problems we're trying to solve.
Having this in mind  I could improvise one solution that temporarily disables the interaction during scrolling.  It is not perfect, but it worked so far for my case.
function handleScroll() {
render.canvas.style.pointerEvents = "none";
clearTimeout(scrollTimeout);
scrollTimeout = setTimeout(() => {
render.canvas.style.pointerEvents = "auto";
}, 200);
}
render.canvas.addEventListener("wheel", handleScroll);

@colorful-invisible Thank you so much for saving my time.

Indirectly mentioned by @colorful-invisible, but I think the name of listener changed from 'mousewheel' to 'wheel' since that article's writing. So now this works for me:
mouseConstraint.mouse.element.removeEventListener("wheel", mouseConstraint.mouse.mousewheel);
(or if mouse initiated, just) mouse.element.removeEventListener("wheel", mouse.mousewheel);
but this doesn't:
mouseConstraint.mouse.element.removeEventListener("mousewheel", mouseConstraint.mouse.mousewheel); (or "DOMMouseScroll")

That's great. Thanks a lot.


linux-zen VP9 VAAPI decoding stutters heavily on RX 6000 series GPUs

Describe the bug
When using linux-zen 6.6.0 and 6.6.1, VP9 VAAPI decoding on RX 6000 series GPUs will hang, stutter, and glitch to an unwatchable state. This occurs across both firefox, mpv, wayland, and xorg. Which is why its making me feel like its a zen related issue. Installing linux 6.6.1 fixed the issue everywhere.
Steps To Reproduce
Steps to reproduce the behavior:

Use boot.kernelPackages = pkgs.linuxPackages_zen; with 6.6.1 on a RX6000 series GPU.
Run a command like mpv video.mp4 --hwdec=vaapi on a vp9 enabled video and watch the artifacting and stutters.
Now run mpv video.mp4 without hardware decoding and see it work flawlessly.

Expected behavior
Hardware decoding on RX6000 series GPUs work without issue as on linuxPackages_latest's 6.6.1.
Screenshots
Not screenshots, but a video of with and without hw decode: https://youtu.be/d88l8kfD3PM
Additional context
Its a brand new GPU.
Notify maintainers
@thiagokokada @JerrySM64

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
This is technically incorrect with regards to the kernel, as i have already changed it over.
% nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.01 MiB download, 0.05 MiB unpacked):
  /nix/store/bbxdw4rgwwl3gnajri82yidr1nlsfskf-stdenv-linux
copying path '/nix/store/bbxdw4rgwwl3gnajri82yidr1nlsfskf-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.1, NixOS, 23.11 (Tapir), 23.11.20231121.dirty`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/store/11hm07gdcjrzrqv73ii0n3j7jiqnq23w-source`

Priorities
Add a :+1: reaction to issues you find important.

I'll take a look at it tomorrow. As I have an RX 6650 XT, I should see the same issue you're experiencing. There's also 6.6.2 out which may fix that. When I have the possibility, I will push that (Currently affected by a major internet outage since Monday. That's why I haven't done it yet.)

Ah thank you. :D
Truly sorry to hear about the internet outage, so dont strain your connection if it hasnt fully recovered. I am just primarily happy to figure out this is a likely quirk introduced by the zen patchset or its spessific compile options inside nixpkgs, and not in fact my brand new GPU acting like it doesn't know how to decode VP9. :sweat_smile:

I just took my PC to a friend to have internet access and I can't reproduce the issue as seen here: https://youtu.be/zdSlLVspgTc
What GPU do you have exactly?

Okay, the 6950 XT can do it for sure. I mean, my 6650 XT can also do it. I was asking because I'm not sure about the 6500 and 6400.
That's something, I currently cannot test. My 144 Hz monitor is at home where the internet isn't working, but I doubt that it's related tbh. Don't quote me on that one though.
I just made the PR with the 6.6.2 kernels. If that doesn't fix it, I'll have to return to this issue when my internet is working again, I'm afraiad.

Bad news i just got this to happen on the stock non -zen kernel, so this issue is likely not related to the linux-zen patchset. I will dig deeper into this, i fear another amd driver bug thats never getting resolved.

Opened https://gitlab.freedesktop.org/drm/amd/-/issues/3013

dumb question, but how do you have vaapi enabled in the first place? I could only enable vdpau and get that to work in Firefox, I want to try hw video decoding again since they fixed issue where it would cause GPU resets

@nonetrix The NixOS documentation is extremely unhelpful as it only shows how to do it on Intel GPUs.

This is an issue tracking what ended up being an upstream issue with the VP9 decoder, however, you should check out https://wiki.archlinux.org/title/Hardware_video_acceleration it should be the best resource available for this. Otherwise I may recommend you to open a separate issue if you believe this to be an issue in NixOS itself.


svn link to Python crashers in readme doesn't work

Fixed with https://github.com/hannob/php-crashers/pull/8


genFromCoords etc. doesn't exist as attribute

Attempting to run some of the examples with the end goal of generating simplex noise vec3.
Running…
import numpy as np import pyfastnoisesimd as fns noise = fns.Noise() result = noise.genFromGrid(shape=[256,256,256], start=[0,0,0]) nextResult = noise.genFromGrid(shape=[256,256,256], start=[256,0,0])
…tells me that genFromGrid doesn't exist, and similar messages with Coords, also running examples. Any clue what I'm missing?

It's Noise().genAsGrid(), not fromGrid.

I see where there was a comment that used genFromGrid(...) and fixed it. I suggest you take a look in the example folder, there are some much more concrete examples there.


How to run without root password?

How can i run wireguird without entering a root password every time?

looking for the same answer. thanks

got it..
sudo visudo
Add the folllwing line in at the bottom (replace username with your username)
USERNAME ALL=(root) NOPASSWD: /usr/local/bin/wireguird
save the file
Add the app in Applcation Startup and edit the line with the path to the app by adding sudo in front (sudo /usr/local/bin/wireguird)

Btw, I tried a similar option, but it didn't work for me on Ubuntu 22.


No way to unistall boxstarter cleanly

There so no way to remove boxstarter cleanly from the System, so that it was before the Installation.

@slech Please take the time to search for relevant issues when creating another. See this and this.


A new switch --strip-unittests

This is an enhancement request.
Before the start of dustmite, if we already know the unittest blocks not affect to the test script result, dustmite can strip off unittest blocks and version(unittest) declarations in preparation, similar to --strip-comments switch.
Pros: It can speed up dustmiting process in specific case.
When I tried to use dustmite a failure of minimal script with Phboos code, although the script compilation had not required -unittest switch, it had consume much times to remove unused unittest blocks. I think that usually big projects contain many unittest blocks, so the proposed feature would become useful in many cases.

I am really skeptical that such a switch would really have a significant effect on the total reduction time. Removing a unittest block should happen in one test, or even less if multiple unit tests are grouped together.
Can you provide a fully self-contained example where you think such a switch would help?

I also think this will speed up dustmiting Phobos. Just now I am dustmiting phobos because of a compiler bug. Phobos's unittests often depend on functions of other packages. Such a unittest block will prevent stripping of function in another package, until finally after many steps the unittest block is removed.
I can give you an example where dustmiting phobos is necessary to reduce an LDC bug, if that helps.

@CyberShadow Will this be hard to add? If not, I'm going to try and implement it, as I think it will speed up my dustmiting a ton.

Will this be hard to add?

Not really.

I think it will speed up my dustmiting a ton.

unittest blocks make up about 0.1% of total nodes in Phobos source code, so I'm really skeptical of that.

Yeah, I thought they need to compile, but I guess they only need to parse.

Can you provide a fully self-contained example where you think such a switch would help?

Let's reopen if/when someone finds one.


Build maxmind on macos

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@zvlb will you be able to test out this patch on osx?

How I can build envoy from this branch?

I normally don't build on Mac, but trying now following this procedure (after resetting local state of repository to this branch): https://github.com/envoyproxy/envoy/blob/main/bazel/README.md#building-envoy-with-docker-sandbox

https://github.com/envoyproxy/envoy/blob/main/bazel/README.md#building-envoy-with-docker-sandbox

❯ bazel build envoy --config=docker-clang
ERROR: /private/var/tmp/_bazel_zvlb/43809851d362369e9c264ad2c5e98fa3/external/envoy_build_tools/toolchains/configs/linux/clang/cc/BUILD:47:19: in cc_toolchain_suite rule @envoy_build_tools//toolchains/configs/linux/clang/cc:toolchain: cc_toolchain_suite '@envoy_build_tools//toolchains/configs/linux/clang/cc:toolchain' does not contain a toolchain for cpu 'darwin_arm64'
ERROR: /private/var/tmp/_bazel_zvlb/43809851d362369e9c264ad2c5e98fa3/external/envoy_build_tools/toolchains/configs/linux/clang/cc/BUILD:47:19: Analysis of target '@envoy_build_tools//toolchains/configs/linux/clang/cc:toolchain' failed
ERROR: /fixMacOSGeoIPBug/envoy/bazel/BUILD:222:29: errors encountered resolving select() keys for //bazel:gcc_build
ERROR: /fixMacOSGeoIPBug/envoy/source/exe/BUILD:25:16: errors encountered resolving select() keys for //source/exe:envoy-static
ERROR: Analysis of target '//:envoy' failed; build aborted:
INFO: Elapsed time: 0.060s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded, 0 targets configured)

does not contain a toolchain for cpu 'darwin_arm64'

I think I can't build it on M* Apple Silicon

@zvlb i built and tested this patch on my new m3 laptop, followed the official build instructions and had to install Xcode manually.

cc @ggreenway
Output from server_info admin endpoint that verifies that maxmind extension is included into binary build for 48cf1405fdd29ed5d17ccec584babbc9be3af5fc commit.


iOS10 MSStickerview category

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[ ] I have added the required tests to prove the fix/feature I am adding

[ ] I have updated the documentation (if necesarry)

[ ] I have run the tests and they pass

[ ] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description
add support for iOS10 MSStickerView

add support for iOS10 MSStickerView

On the 4.x branch, we refactored all those methods on a UIView category. This means there is no need for the MSSticker to have its own methods. Calling the ones from UIView should be sufficient.


Nested ng-repeat with models between directives

I am attempting to replicate the nested ng-repeat with model example, but through different directives.
Pretty much, I have a list of contacts and groups of contacts. I would like to move these contacts into a separate container but make moving the groups disabled. I have a directive for the containers, a directive for contacts, and then a directive for groups.
I feel like I am close - I can add a Contact to a container and drag a GroupContact into the container. However when I drag the GroupContact into the container, the container model isn't updated with the GroupContact's information. I also get an
dragular.js:1182 Uncaught TypeError: Cannot read property 'nextElementSibling' of null

v	@	dragular.js:1182
Q	@	dragular.js:796
o	@	dragular.js:816
V	@	dragular.js:804
B	@	dragular.js:418

I added a drop callback which i outputted the arguments of the drop event and I believe the issue has something to do with scopes.

Can you replicate your issue somewhere public? Like jsfiddle, codepen..etc.. thanks. You can reuse code from other issue like this one: https://jsfiddle.net/hLsuege1/.

Closed due to inactivity


Need to add github repository secrets

The following repository secrets need to be added in github secrets:
SMTP_EMAIL
SMTP_PASSWORD

I will share the values privately.

Also, secure this file's contents. https://github.com/ekanshsinghal/edu-setu-group26-proj2/blob/main/code/backend/.oci/base_key.pem
@ekanshsinghal Lmk if these are done. Just have to update in code.

Done


Marketing change

Please accept these CATS

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes
Have you signed the CLA already but the status is still pending? Recheck it.


Feature- SD config storage

Altered config layer configure procedure to include fully automated storage and loading of configuration packets sent from hive.
This utility is also able to verify configurations based on robot type, based on configuration packets sent from hive and ref system robot ID.
This feature went through the following tests:

Without SD card: robot is able to boot correctly without needing SD card. Robot should still be able to verify that a config packet is correct using ref system (incorrect packet should cause infinite loop with error message).
With SD card: wrong config from hive should cause infinite loop with error message (currently requiring manual restart) instead of loading (wrong) config like it normally would. When the ref system has a different ID from what is on the SD card packet file (config.pack) it should error out and request a packet from hive. If robot_id.cfg on the robot is changed, the robot will still configure correctly as long as the ref system robot ID matches that of the stored configuration. The robot should still configure a configuration the exact same whether the robot ID is red team or blue team (difference of 100 in robot ID on ref).
Checksum: if any inconsistency is found whatsoever between stored data and expected output, a checksum (summing over all of the bytes interpreted as unsigned 8-bit integers) should report this inconsistency to detect corrupted data. If the checksum function is tampered with at all, it should not read an old config file made before the changes. If bytes at any location on the config.pack file is changed, the configuration should fail and a new config packet should be requested from hive. If any of the places where the checksum is copied to (before each of the config packets) on the config file are altered, the configuration should fail and a new config packet should be requested from hive.

Dont merge until all reviewers have approved

committed to the wrong branch- how embarrassing...!


Explain in a little detail what the easy install scripts do

Could use your help wordsmithing this, @lindydonna.

I would try to get it down to two sentences max.  (Parenthetical remarks count as sentences.)

@joeduffy @ellismg I added another commit with a proposed change. I consolidated the note to cover all platforms.

@joeduffy It's 3 sentences now, but one is very short. :)


Continuation of EDS integration for CFE

Integrate the remaining EDS features which are not yet included as part of the CFE 6.6 release.
This is a continuation of #207 for the next CFE release.
Defects to resolve: #92, TBD
Enhancements to resolve: TBD

Imported from trac issue 212.  Created by jphickey on 2017-10-19T16:39:51, last modified: 2019-07-03T12:58:32

Trac comment by jhageman on 2019-03-20 15:39:09:
Changed to a "task" ticket, and preference is to treat this as a container for all the tickets resolved by the EDS update and remaining work tickets.

Trac comment by jhageman on 2019-07-03 12:58:32:
Moved unfinished 6.7.0 tickets to next release

Hi there,
I wanted to ask about the status of EDS integration with cFS. As far as I know, there's a working version available from one of the flight software workshops with EDS integration fully involved in the pipeline, but I don't see any EDS tooling in the build process for mainline cFS. There are some XML documents in the cFE core, but no explanations about how to integrate EDS with user apps or ground stations.

There's a few EDS/cFS related repos at https://github.com/jphickey/cfe-eds-framework, https://github.com/nasa/cFS-EDS-GroundStation, https://github.com/nasa/EdsLib.
Although further integration is a future goal, I'm not aware of any resources or funding on our side specifically targeting advancement of this goal with respect to the open-sourced software in the short term.  As always we encourage contributions, and feel free to reach out to the cFS Product Team for more detailed collaboration discussions.  The discussions page on cFS is also a good resource for community coordination.


When the request is cancelled frequently, it will cause SocketTimeoutException

version 3.10.0
demo: https://github.com/Caij/OkHttpDownDemo

Hopefully this is fixed by https://github.com/square/okhttp/pull/4052

Thanks .  When is it released?

I'm not sure at the moment, can you test with a snapshot build? https://oss.jfrog.org/artifactory/libs-snapshot/com/squareup/okhttp3/okhttp/3.11.0-SNAPSHOT/

I test with a snapshot build, The problem has been solved.


Procedures should have less than 100 lines of code

Title
Procedures should have less than 100 lines of code
Description
Code is easier to understand when it is broken down into multiple procedures with a defined purpose..
Reason for the rule
Very long procedures are hard to read and result in code that is hard to maintain and reuse.
Bad code sample
Example of what bad code the rule should catch:
    procedure DoSomethingComplicated()
    begin
        [Hundreds of lines of code doing everything]
    end;

Good code sample
Example of what code should look like:
    procedure DoSomethingComplicated()
    begin
        CheckSomething();
        CreateSomething();
        PostSomething();
    end;

    local procedure CheckSomething()
    begin
        [...]
    end;

    local procedure CreateSomething()
    begin
        [...]
    end;

    local procedure PostSomething()
    begin
        [...]
    end;
    [...]

Good and bad practices for fixing the rule
Refactor the code into smaller procedures each with a clear and defined purpose and a descriptive name.
Do not move variables to globals and just break the procedure into parts.
Remarks
Possibly an info not a warning.

But i don't want ignorable warnings that hide the warnings, i should pay attention on.

But to be realistic, such a warning as this must probably be info-only, by default, to not annoy other developers who... shall we say... could use a little help writing clean code. Otherwise it will just result in noise on this issue tracker. You or I could use ruleset.json to promote it to a warning or error.

Thank you for reporting this issue. Unfortunately, we do not find it important enough to address at this point.


jupyter doesn't work in .net 6.0, doesn't work after install 3.1

Today I tried to install jupyter and it doesn't work for me. I installed 3.1 and it's still not able to run. Any help
https://github.com/dotnet/sdk/issues/27976

Jupyter functionality has moved over to .NET Interactive. https://github.com/dotnet/interactive


Extensible types storage & migration

2 databases:

new one, where we only store the new, extensible enums
legacy one, where we keep the legacy types

performance concerns (we might need to hit two databases to get single entity)
we should measure

Shall we solve the problem of partial deserialization in the same epic?

No - it's a stretch goal (separate epic)

Magic string? Collision probability, while minimal, still exists

Might still be needed for some types (case by case decision)

Data migration: All at once? vs. Lazy? vs no-migration (runtime detect of legacy structs)

Let's target for Lazy - we might change the approach if there are performance issues with two databases

Shall we drop bincode? Or implement the above for bincode?

Drop it! We might need it for parsing the old data, unless we migrate it

markdown file in node private

9.7.23 This work was done as part of the POC and proved viable.  The Bincode discussion is entered as a tech-debt issue and is discussed in the CEP related to data extensibility.


Add frontend

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #4

PR #5   👈

This stack of pull requests is managed by Graphite.


GIFs don't play automatically until they are opened

First of all, I'm sorry that my English is not good, and I have to rely on a translator to express my thoughts
PiGallery 2 is a great tool, but it has a few hiccups with GIFs
GIFs don't play automatically until they're clicked on individually, which makes it difficult to tell when there are too many GIF files and can't tell the difference between them when they're mixed together
I found that the main reason is the thumbnail issue, GIFs generate WEBP thumbnails just like any other file. I've tried all kinds of methods, but I can't turn off the function of generating thumbnails for GIFs
I hope to add a switch for GIF file autoplay,
Thanks for PiGallery2, it is a great piece of work!

I think its a duplicate of #169

Yes, the requirements are indeed duplicated, but issue #169 is not resolved, is there a better solution? Thank you very much for the reply。

My technique is not good and I didn't find Client.
 Media.Photo.Converting.enabled

That settings was moved.
You can find it here:

Sorry you misunderstood what I meant, but I don't mean the autoplay after clicking on the image, it's on the overview page, that is, the page where all the images are displayed, and the GIF won't play

In the case of this page, is there a way to make the GIFs all play automatically? Instead of clicking in.

Ah ok I see. That is not supported. Thumbnail files are webp files not gif. (I think they might also support moving image, but that is not implemented)

I guess its related to this: https://github.com/lovell/sharp/issues/2395

I had a look and it seemed to be an easy fix. At least it works on windows, building from source. Lets hope it also works in Docker too.


metric description

hi! where can I find the detailed description of all metric,such as Globals metrics/Frontend metrics/Listener metrics/Backend metrics/Server metrics/Stick table metrics

Generally, the metrics are directly taken from HAProxy. You can find descriptions of their meanings in the HAProxy documentation.


virtcontainers: add codes to process the case that kata-agent doesn't start in VM

virtcontainers: add codes to process the case that kata-agent doesn't start in VM
If kata-agent process is not launched to run in VM, kata-runtime will exit because of timeout,
however kata-proxy and qemu process will still resident in host
Call Check grpc function to check whether grpc server is running in kata-agent process in VM,
if not and then do some rollback operation to stop kata-proxy and qemu processes
Fixes: #297
Signed-off-by: flyflypeng<EMAIL_ADDRESS>

CI failed because title is too long


Winery architecture

Short Description

Proposed Changes

Fixes: #

The semantics of "assignee" is that this person is responsible for the content. Since the PR is coming from @berhest, she is the responsible one.
Please update the description of the PR. It is currently

There are way too many files added. The howto for uploading an application is already contained in https://github.com/OpenTOSCA/container/pull/42 - and fixed by @snisnisniksonah and me.
Some files are also existing in https://github.com/OpenTOSCA/container/pull/43. Please cleanup this PR and also https://github.com/OpenTOSCA/container/pull/43.


Python autocomplete not working

Hi, so I am using atom to code in Python however the autocomplete isn't working only in Python. It works with others such as html and javascript. I have python-jedi and the autocomplete python packages enabled. Any help?

Figured out why I was having this problem. Currently it seems like the Python Jedi atom plugin is broken and disables it completely.

Python Jedi (https://github.com/saravanan-k90/python-jedi) is not broken. Clarified the same issue raised by elebumm @ python-jedi repository.

i installed auto complete python but it seems not work help me out

i am extremely sorry @Aaron-joshua ., my position is bad now(no job, less money, less internet, old linux system... due to these reasons i haven't really checked plugin working in any new versions of atom for the past 6 months)... so i may not be able to try to check or solve the issue you have opened. But i really appreciate for opening issue.


not the same EEGNet model as in the original paper

Dear,
I notice the EEGNet in your repo is not the same as what it is in the original paper.  In their paper, they do temporal conv2d first, then spatial conv2d. Is there any particular reason why you swap these two operations?

sorry, I made a mistake. close this issue.


TypeScript 3.8 and import type

Description of change

I've updated our version of TypeScript to 3.8 so we can use the new import type feature.
I also removed the eslint rule for no-duplicate-imports to allow for this to work.
Pre-Merge Checklist

[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Yes, I couldnt just change that eslint rule before 3.8, I was afraid that you guys wont approve it, so you are kind of redoing my job now :)

Hey @Zyie not sure why Travis didn’t get started. Can you touch this PR again to see if it’ll start?

Hey @Zyie not sure why Travis didn’t get started. Can you touch this PR again to see if it’ll start?

Hmm. Github is failed, because on travis dashboard all OK!

Travis build is okay so I’ll merge. GitHub reporting issue.


Change display-name's acceptTranspilerName to ignoreTranspilerName

This was a change that has been queued for v4. I considered changing the
default from false to true, but I think it makes more sense to invert
the option to be an ignore option.
Addresses #436

I think it makes more sense to invert the option to be an ignore option.

I agree, merged. Thanks!

yay


Please provide more liberal payment options

First, very thank you for the great product.
You are doing the great job.
Meantime, you are using patreon.com for the donations, and it does not accept for example Russian bank cards (and PayPal too).
Please add some other donation methods said cryptocurrency or more liberal systems like https://payeer.com.

Thanks for your suggestion, I'll take a look at it!

Thanks for your suggestion, I'll take a look at it!

Hello!
What was a solution - paypal button on a site, or a payeer.com account?

PayPal and Patreon are the options at the moment - https://www.archimatetool.com/donate/

Sorry, you are closing issue as completed, but still accept only patreon and PayPal? Am I wrong, but all is in initial state?


Add 3.10 patch (applies against 3.10.40 TOSSUG Baby Fish)

This one builds just fine on 3.10.40 from CM12 "shamu"'s sources.
Not tested yet.

UPD: can confirm as working (/dev/hidg{0,1} are created; hid-gadget-test works)


Question on mod capabilites

hi, I have some questions about what this mod can do I want to use this item-restrictions and pufferfish's skills to create a skill tree to unlock the use of stuff, is this possible? the wiki looks very incomplete so I'm asking here

For this to actually work, you'll need to create a mod that adds Arc Lib compatibility to Pufferfish's Skills. I'm not very familiar with Pufferfish's Skills myself.


Discord blocks ALVR

Description

When discord once turned on then ALVR dashboard will don't work on Linux, I tested on Ubuntu and Mint
I noticed that when I have open discord then ALVR don't opens or is white blank.
I noticed this when I tried to turn it on and dashboard was blank, but when I restart PC without turning on Discord ALVR works ( if I turn on ALVR and second Discord then everything is OK )

General Troubleshooting

[ ] I carefully followed the instructions in the README and successfully completed the setup wizard
[ ] I read the ALVR Wikis here and here

Environment
Hardware
Note: for Linux, an upload to the hw-probe database is preferred: hw-probe -all -upload
CPU:
I9-9900K
GPU:
GTX1070
Audio:
Installation
ALVR Version:
Appimage.
SteamVR Version:
Install Type:

[x] Packaged (exe, deb, rpm, etc)
[ ] Portable (zip)
[ ] Source

OS Name and Version (winver on Windows or grep PRETTY_NAME /etc/os-release on most Linux distributions):
Linux Min 21

PS. I will edit it later I do it when I remember - I wanted to do it weeks ago.

I'm using Linux Mint 21 and I never got this behavior 🤔

Are you using v19 stable or v20 nightly? Please use the nightly version on linux.

Closing this in prep on V20 release


UI issues for resource

View a resource definition, it shows uiSchema tab by default.(why not show the first tab?)
Create a resource, it shows New Resource Definition in breadcrumb.
Create a resource, it shows Template Configuration.
Create a resource definition and set UISchema to empty variables. Create a resource. Should we remove Configuration block when it's empty?

Environment

Walrus version: 14cd3f8fb0b4028913755cb07364bf4ace803797
UI version: dev-830c6bb

Test Result:

pass
pass
pass
pass


support json jsonnet

close #57

suport json and jsonnet ecschedule config file

evaluate jsonnet and encode to json
load json and yaml

add json, jsonnet to README

Thank you. I think it's good. Just in case, for backward compatibility, please fall back to YAML when the file type cannot be determined from the extension since there may be existing users using configuration files with naming like ecschedule.cfg, .ecschedule etc.

Thank you for a review!
I fixed in 4b49c38.
I would appreciate your review again.

Thank you. I think it's good. Just in case, for backward compatibility, please fall back to YAML when the file type cannot be determined from the extension since there may be existing users using configuration files with naming like ecschedule.cfg, .ecschedule etc.

Great! Thank you.


Graphical corruption when executing the reboot menu item

Executing the reboot item causes the screen to be corrupted until the return to dashboard occurs (looks likely to be uninitialized data).
I assume this is because of the use of exit() rather than explicitly launching w/ no executable path. If XboxDev/nxdk/pull/501 gets merged, this can probably be solved by changing the exit to XLaunchXBE(nullptr).
Calling XReboot() also fixes the graphical corruption but that's a full reboot instead of a quick one, which probably isn't desirable for users of bios loaders like Phoenix.

Looks like I'm wrong about using an explicit HalQuickRebootRoutine; I still see the same corruption after patching 501 and calling XLaunchXBE on my v1.0 XBOX.

Digging around a bit, I think the issue is the lack of a PersistDisplay equivalent to store the framebuffer.
It looks like XVideoInit is already persisting the framebuffer though, so it's not immediately clear to me why the screen doesn't simply freeze.

I'm pretty sure this is an issue with nxdk.

Looks like you're right, I can reproduce in a smaller test program. Filed XboxDev/nxdk/issues/507 to track over there.


Improve status message when helm release has failed max number of times

Fixes https://github.com/giantswarm/giantswarm/issues/17314
When chart-operator fails to update a helm release 5 times we stop updating. This is because the max history of 10 revisions is not respected and 100s of release secrets can be created which impacts the cluster.
This improves the CR status so its clearer when this has happened.
Checklist

[x] Update changelog in CHANGELOG.md.

Tested on gauss.
kubectl -n giantswarm get app prometheus-operator-app-unique -o yaml  | yq .status.release
{
  "lastDeployed": "2021-06-07T06:55:26Z",
  "reason": "Release has failed 5 times and will not be updated.\nReason: Upgrade \"prometheus-operator-app-unique\" failed: pre-upgrade hooks failed: warning: Hook pre-upgrade prometheus-operator-app/templates/prometheus-operator/admission-webhooks/job-patch/rolebinding.yaml failed: object is being deleted: rolebindings.rbac.authorization.k8s.io \"prometheus-operator-app-un-admission\" already exists",
  "status": "failed"
}