advanced tonemapping

add advanced tonemapping ready to be used in project (not used as standart tone mapper yet)
uses filmic-tonemapping instead of reinhard-tonemapping (default)

basic rule for tonemapping:
"crips black" vs "milky black" filmic > linear > reinhard
"soft-highlights" vs "clamped-highlights" filmic = reinhard > linear

http://filmicgames.com/Images/FilmicTonemapping/Graphs/Reinhard-vs-Filmic-Wide.png
color example
top row is Reinhard per-channel, middle is linear, and bottom is Filmic.
http://filmicgames.com/Images/FilmicTonemapping/Ramps/Color-04.png

we may wanna change the tonemaping.frag to reinhardtonemapping.frag (or reinhardtm.frag)
and advancedtonemapping.frag to filmictonemapping.frag (or filmictm,frag) for shorter more accurate file names.

Why commit squashing? 0.o

Desperate attempts to get git to cooperate. The rebase caused like 5 merge conflicts with itself, even though the branch was up-to-date (it contained many merge commits so we had to rebase).


Date values do not allow to enter text like „@Today-5“ anymore

We used to highlight cards without updates for n days with a style rule, e.g. pink background, to easily/visually identify „cards of inactivity“. We have used rules like the example in the documentation like „@Today-5“. They still work, but can no longer be edited. Also, we can't add new date-based rules anymore, because the calendar helper always gets the focus. The value field of a style rule does no longer accept direct input.
Please allow direct input for date values again.
Thank you very much!

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

ID: 4d807b7f-4e44-1e86-7f41-1e67f9c0e1bd
Version Independent ID: 6798865e-0273-2991-9376-abe53725aaac
Content: Customize a sprint taskboard in Azure Boards - Azure Boards
Content Source: docs/boards/sprints/customize-taskboard.md
Service: azure-devops-boards
GitHub Login: @chcomley
Microsoft Alias: chcomley

@HerrDorsch -- Herr Dorsch, please add this product feedback here:

Developer Community


Add ResultsDirectory to VSTestSettings

ITNOA
I think its sound good to add ResultsDirectory to VSTestSettings.

@devlead I create PR for this

@augustoproiete I created PR #3128 for this issue, and I think it is ready for review. :)


Resize text on pictures for home page in mobile

On mobile browsers, the pictures on the home page are appropriately resized.  The text on them, however is not.

Hey @rlgreen91, of which mobile devices are we speaking?

This was on my Samsung Galaxy S6.

I'm not exactly sure when, but this has been addressed, so closing.


Remove temp dir

https://github.com/ROCmSoftwarePlatform/MIOpen/blob/01ff3da68b68b2ffbc31cb681184a8082ac7ce26/src/hipoc/hipoc_program.cpp#L230
The temp directory dir saved in HIPOCProgramImpl will not be deleted until the end of the process. If MIOpen does not exit normally, the temp dir will never be deleted.
This PR is to solve the above problem.

@DrizztDoUrden Please review.

@DrizztDoUrden Please have a look.

@zpwenjh Thanks for another contribution!


Design time identity migration fails with custom code in startup

@ajcvickers This appears to be an EF thing

Played with this more today, it is looking like if you scaffold identity into a project you must hard code a connection string into appsettings.json for design time migration to work. IdentityHostingStartup gets called before startup and there does not seem to be a good way to mess with public static void Main to make things work.
Maybe i am not seeing something?

@blowdart This looks like an Identity scaffolding thing to me, especially given the second comment. But without actual code it's hard to be sure what is going on.

build a new project

set appsettings.json (http://json.schemastore.org/appsettings) to

{
"Logging": {
"LogLevel": {
"Default": "Information",
"Microsoft": "Warning",
"Microsoft.Hosting.Lifetime": "Information"
}
}
}

set appsettings.Developement.json (http://json.schemastore.org/appsettings) to

{
"Logging": {
"LogLevel": {
"Default": "Information",
"Microsoft": "Warning",
"Microsoft.Hosting.Lifetime": "Information"
}
}
}

Add an environment variable to the system the application is running on called my_DATABASE and put your connection string in it.

Read the connection string from your system to connect to your database.

example add this to application startup.cs
            var config = new ConfigurationBuilder()
            .SetBasePath(env.ContentRootPath)
            .AddJsonFile("appsettings.json", optional: false, reloadOnChange: true)
            .AddJsonFile($"appsettings.{env.EnvironmentName}.json", optional: true)
            .AddEnvironmentVariables("my_")
            .Build();

    myDatabase.CONNECTION = config.GetValue<string>("DATABASE");

Your application will run fine, but at design time it will not longer be able to scaffold because it can't find the connection string.

ps I don't like appSecrets to much typing, I hide things else where and remove things like connection strings from my projects.

@blowdart this behavior is expected the scaffolding won't know how to find connection strings if they are doing something non standard

We're closing this issue as the behaviour discussed seems to be by design.


Calling stop or reset on ffwdme.navigation does not clear current route.

I've been looking through the docs and source code of this lib and am trying to figure out how to reset a map to its initial state after navigation had started. I thought that either of the two methods mentioned in the title might be where that would happen but it does not appear so. Does the library support this functionality?
If so I would be willing to add some documentation otherwise a new feature to support this.

At the moment stop just emits an event. It's not really used.
The reset method resets the internal state of the navigation object and is used if you want to replace the route handled by the navigation event (e.g. if you do a rerouting, because the user left the calculated route).
I would be very happy to accept contributions, let me know if you have further questions.
Cheers,
Christian


attention

In your Attention class,
self.X shape is [1 5 1], after executing below
x_mean = tf.reduce_mean(self.X, axis = 2)
the shape becomes [1 5], if mean is apply on axis=2 , the values will be the same, and only reduce [1 5 1] to [1 5] . Is this correct?

mean over axis 2 on [1 5 1] will be reduced to [1 5] yes


WIP: Refresh criteo-master to Presto 0.206

Putting this up to let travis check things out. Merging changes from 206 into our criteo-master branch and fixed merge conflicts.

This change link highlights the patches we have cherry-picked, our custom internal patches and any workarounds - https://github.com/criteo-forks/presto/compare/master...piyushnarang:206-merge

cc @szehonCriteo

+1, it looks good to me


It keeps saying that steamvr isn't installed.

i installed it but it says it isn't installed help.

SteamVR installed on C drive default path?  If not, try that first.

In Steam your Steam library under VR, right click on the SteamVR application and go to Properties. From there, click on the Local Files tab. It should say something like "Disk Usage: xxxx MB on drive C:"
If it says anything other than Drive C, ALVR might not detect it. You can use the "move install folder" button on that tab to move it to the default C: library.  Where are you getting the error that "it keeps saying steamvr isn't installed?"  What are you doing when that error pops up?
If this solves your issue, please make sure to mark the issue as "resolved" in this tracker.

okay so it is installed on drive c. after i boot it up. When i start ALVR up on pc it says "cannnot load library. ALVR works only on 64bits Windows" along with C:\Users\user\Downloads\ALVR-v2.1.5-portable\ALVR\driver\bin\win64\driver_alvr_server.dll
after it says that, i just go on with the application and then it says "SteamVR is not installed. Registry (HKEY_CLASSES_ROOT\vrmonitor\shell\open\command was not fouund.) Please install and try again" when i try to start the server or install the drivers through the app, even though i installed steamvr

What GPU do you have?

Hi! I'm totally new here. I had that same issue when trying to innitiate ALVR, but all I had to do to solve it was to run at least one time SteamVR. After that one time, ALVR ran perfect. Perhaps this might help you. Sorry, english is not my native language.

okay that partially worked but i have another problem. when i start ALVR up on pc it says "cannnot load library. ALVR works only on 64bits Windows" along with C:\Users\user\Downloads\ALVR-v2.1.5-portable\ALVR\driver\bin\win64\driver_alvr_server.dll and running steamvr only fixed the second part of my dilemma. even then you still helped me solve my problem a lil bit so thanks.

I've been there DUPERMUSHROOM... it means that you have to run the Steam VR software on Steam, double click and open it. Just that. Then you close it, and from now on, when you run the VR game you want to play, Steam VR opens automatically. But the first time, before playing any VR game, you have to open Steam VR one time.

Any help guys on this ?

I am getting exact same messages as above ?

I had the same problem. I have Steam installed in a custom location, no the default on system drive under Program Files (x86).
Solution - create a symbolic link. Open administrative command prompt and run (update the second path to point to your Steam location):
mklink /D "C:\Program Files (x86)\Steam" "D:\Steam"
Then launch SteamVR at least once, as mentioned above (right click on Steam in tray -> SteamVR). It won't detect headset, but that's ok. Now try launching ALVR again - it should now work!

Worked for me. Thanks BogdanBerg!

I had the same problem. I have Steam installed in a custom location, no the default on system drive under Program Files (x86).
Solution - create a symbolic link. Open administrative command prompt and run (update the second path to point to your Steam location):
mklink /D "C:\Program Files (x86)\Steam" "D:\Steam"
Then launch SteamVR at least once, as mentioned above (right click on Steam in tray -> SteamVR). It won't detect headset, but that's ok. Now try launching ALVR again - it should now work!

I'm getting the same issue. I tried using the command you mentioned, but it said: "Cannot create a file when that file already exists."
BTW, I have Steam installed on Drive C, but I have SteamVR installed on a separate hard drive, could that be the issue?

I think this might indeed be the problem (the fact that you have steam
installed on C and Steam VR on D) . I suspect if you check the contents of
D:\Steam, you'll find some files and folders there. As such, running the
above command for each of those files and folders would probably fix it (as
that way we'd create links for the SteamVR contents). Be sure to modify the
command accordingly.
For example, if under D:\Steam you find "folder1" and "file1" you'd run:
mklink /D "C:\Program Files (x86)\Steam\folder1" "D:\Steam\folder1"
mklink "C:\Program Files (x86)\Steam\file1" "D:\Steam\file1"
One thing that will be problematic is if there are some folders with same
names in those locations - it will give you same error, in which case we'd
have to navigate into those folders and create links recursively.
I suspect, if possible, reinstalling SteamVR into the root location of
Steam is a safer and less hacky solutions... 😊
Hope that helps!
Thanks,
Bogdan
On Sun, Jan 5, 2020, 12:02 AM TheTypographer<EMAIL_ADDRESS>wrote:

I had the same problem. I have Steam installed in a custom location, no
the default on system drive under Program Files (x86).
Solution - create a symbolic link. Open administrative command prompt and
run (update the second path to point to your Steam location):
mklink /D "C:\Program Files (x86)\Steam" "D:\Steam"
Then launch SteamVR at least once, as mentioned above (right click on
Steam in tray -> SteamVR). It won't detect headset, but that's ok. Now try
launching ALVR again - it should now work!
I'm getting the same issue. I tried using the command you mentioned, but
it said: "Cannot create a file when that file already exists."
BTW, I have Steam installed on Drive C, but I have SteamVR installed on a
separate hard drive, could that be the issue?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/polygraphene/ALVR/issues/83?email_source=notifications&email_token=AEM2AOTBDSEPKN44NGQ47RDQ4GHYHA5CNFSM4FGRHA42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIDQ4NQ#issuecomment-570887734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEM2AOVEOZITOMEMNAUQTZTQ4GHYHANCNFSM4FGRHA4Q
.

Hi,
Maybe this won't work for all but I had the problem and I solved this just starting SteamVR first and then initiate ALVR, for me works.
Regards.


Get first suggestion with down arrow

Add goprompt option to be able to select the first suggestion with down arrow, currently it's TAB only.

#153


no sdk found

➜  ~ vmr
WARNING  no sdk found!
➜  ~ vmr version
v0.6.2(5d39224)
just drive me crazy...

related with this?
curl --proto '=https' -sSf https://gvc.1710717.xyz/proxy/https://raw.githubusercontent.com/gvcgo/version-manager/main/scripts/install.preview.sh
curl: (22) The requested URL returned error: 502

https://gvcgo.github.io/vdocs/#/zh-cn/usage?id=设置代理


feat(jstzd): serve config in jstzd server

Context
Part of JSTZ-121.
JSTZ-121
Description
Serve config in Jstzd server. The following endpoints are implemented:

GET /config/octez-node: returns octez node config in JSON.
GET /config/octez-client: returns octez client config in JSON.
GET /config/octez-baker: returns baker config in JSON.
GET /config/: returns {"octez-node": <node config>, "octez-client": <client config>, "octez-baker": <baker config>}.

Manually testing the PR

Integration test: updated jstzd_test to check if the responses are as expected.

GTM overall, just small comment


Run odo test with OCI staging repsitory

What does this PR do?
Run Odo integration tests with OCI-based staging repository
What issues does this PR fix or reference?
Is your PR tested? Consider putting some instruction how to test your changes

/retest


v1 can not be installed npm

Version 1~ can not be installed by npm.
Please put the tag.

Meanwhile npm i git://github.com/necolas/normalize.css.git#v1.1.1

@MoOx can not install.

Ha. v1 probably doesn't have a package.json. You should use napa to get this one.

https://github.com/necolas/normalize.css/tree/v1
Indeed, no package.json. Use https://github.com/shama/napa

@MoOx @kud Been resolved. Thank you!!


Always server error with c++ solutions

Each time I try to get C++ solutions of a kata  I get:
Looks like there was a server error! (500)...
Example: https://www.codewars.com/kata/which-are-in/solutions/cpp

Now I get "The kata failed to load. Please try reloading the page" again and again.

I just saw that it happens in almost every language:-(

All kata or just specific ones? I'm still dealing with a database upgrade issue that is causing data problems when loading. It only affects some records.

I can't try all my katas:-). I tried a few in different languages and it happens that they load in one or two languages but not in the other ones. Thanks for your work!

Do you have repro steps. I don't have a lot to go on at the moment. I also deployed a bunch of updates today so the problem may have resolved itself.

Seems to work now, thanks!


Error "su: cannot set groups: Operation not permitted" with --sudouser

I run this, the host is Ubuntu 18.04 LTS:
x11docker --desktop --home --sudouser  -c x11docker/xfce
From a terminal inside the container, I invoke su command but got this error:
hendro@43f4368fe622:~$ su
Password:
su: cannot set groups: Operation not permitted
hendro@43f4368fe622:~$ ls -l /bin/su
-rwsr-xr-x 1 root root 63568 Jan 10 15:30 /bin/su
hendro@43f4368fe622:~$ cat /etc/sudoers
# /etc/sudoers created by x11docker
root ALL=(ALL) ALL
hendro ALL=(ALL) ALL

Need another hint, I'm really excited with X11docker... Thanks

This is odd. The same command works well here (host: Debian).
The host should not make a difference.
Can you please show me the output of --debug?
x11docker --debug --desktop --home --sudouser -c x11docker/xfce

You can also try to add option --cap-default. That will most likely work. However, you should not need it.

Please find the output:
output message  and log
--cap-default doesn't make any difference..same error.

Maybe there is an issue in /etc/group.
Can you please show me (from container):
cat /etc/group
cat /etc/passwd
cat /etc/shadow

This is the output.
Btw the error for sudo is different:
sudo: effective uid is not 0, is /usr/bin/sudo on a file system with the 'nosuid' option set
      or an NFS file system without root privileges?

My temporary solution for invoking authorized command  is using separately:
docker container exec -it --user root <container name/id)  /bin/bash
Of course it doesn't require --sudouser at the first place.

Thanks for the cat output. Everything looks well. I still have no idea why it does not work for you.
Could you try su hendro in your workaround solution? And if it suceeds, a su to root afterwards?
Another workaround would be to run x11docker with --user=root.
Maybe I have to set up an Ubuntu VM for test runs.

Not sure what you meant with try su hendro then su to root. Am I already in root in workaround solution?
Yes x11docker with --user=root works but my intention is using normal user only certain commands such as apt-get I need to bump up the authority using su or sudo.
Me too trying to set up Debian VM, will let you know later..

I've created both Ubuntu 18.04 and Debian 9 VM and the problem do not occur on both VMs. Seems the issue is in my desktop Ubuntu 18.04 not on your x11docker. You can close it. I think I still  need to compare the msg log of both Ubuntu 18.04s.

I've created both Ubuntu 18.04 and Debian 9 VM and the problem do not occur on both VMs.

Thank you for the test!

Not sure what you meant with try su hendro then su to root. Am I already in root in workaround solution?

I though of this:
$ docker container exec -it --user root <container name/id> /bin/bash
root@abc123# su hendro
hendro@abc123# su
root@abc123#

This way you could check if user switching works in the docker exec environment.

I thought of this:
$ docker container exec -it --user root <container name/id> /bin/bash
root@abc123# su hendro
hendro@abc123# su
root@abc123#

This way you could check if user switching works in the docker exec environment.

Oh OK but same error:
root@8d83fe45e87c:/tmp# su hendro
su: cannot set groups: Operation not permitted

Maybe there is a wrong security configuration of docker. Please try:
 x11docker --desktop --home --sudouser -c --cap-default -- --security-opt seccomp=unconfined -- x11docker/xfce

If that fails, too, please try:
 x11docker --desktop --home --sudouser -c --cap-default -- --security-opt seccomp=unconfined --cap-add=ALL -- x11docker/xfce

Thanks for the hints. I'll get back with the result later...

Maybe there is a wrong security configuration of docker. Please try:
 x11docker --desktop --home --sudouser -c --cap-default -- --security-opt seccomp=unconfined -- x11docker/xfce

If that fails, too, please try:
 x11docker --desktop --home --sudouser -c --cap-default -- --security-opt seccomp=unconfined --cap-add=ALL -- x11docker/xfce

Both failed with the same old error

Please build this image and run some user switching tests:

$ docker run --rm -ti --user=testuser mytest bash
testuser@0d0a79325d73:/$ id
uid=1000(testuser) gid=1000(testuser) groups=1000(testuser),27(sudo)
testuser@0d0a79325d73:/$ sudo ls
sudo: effective uid is not 0, is /usr/bin/sudo on a file system with the 'nosuid' option set or an NFS file system without root privileges?
testuser@0d0a79325d73:/$ su
Password:
unix_chkpwd[9]: check pass; user unknown
unix_chkpwd[9]: password check failed for user (root)
su: Authentication failure
testuser@0d0a79325d73:/$ su
Password:
unix_chkpwd[11]: check pass; user unknown
unix_chkpwd[11]: password check failed for user (root)
su: Authentication failure
testuser@0d0a79325d73:/$

doesn't accept 'x11docker' as valid password ?
BTW, following are the message when building the image:
$ docker build   -t mytest  .
Sending build context to Docker daemon  2.048kB
Step 1/6 : FROM debian:buster
buster: Pulling from library/debian
478cd0aa93c0: Pull complete
Digest: sha256:543a3f93a1f53d2d292ec067fa13528d5f3843ae4c4ffe6ef6993ca70b6ac2fa
Status: Downloaded newer image for debian:buster
 ---> 098963abf3c3
Step 2/6 : RUN apt-get update
 ---> Running in afa329a7b68e
Get:2 http://cdn-fastly.deb.debian.org/debian buster InRelease [163 kB]
Get:3 http://cdn-fastly.deb.debian.org/debian buster-updates InRelease [46.8 kB]
Get:4 http://cdn-fastly.deb.debian.org/debian buster/main amd64 Packages [7902 kB]
Get:1 http://security-cdn.debian.org/debian-security buster/updates InRelease [39.1 kB]
Get:5 http://security-cdn.debian.org/debian-security buster/updates/main amd64 Packages [1132 B]
Fetched 8152 kB in 20s (410 kB/s)
Reading package lists...
Removing intermediate container afa329a7b68e
 ---> c58aa441128d
Step 3/6 : RUN apt-get install -y sudo
 ---> Running in 2640d37c5af3
Reading package lists...
Building dependency tree...
Reading state information...
The following additional packages will be installed:
  lsb-base
The following NEW packages will be installed:
  lsb-base sudo
0 upgraded, 2 newly installed, 0 to remove and 1 not upgraded.
Need to get 1273 kB of archives.
After this operation, 3936 kB of additional disk space will be used.
Get:1 http://cdn-fastly.deb.debian.org/debian buster/main amd64 lsb-base all 10.2019051400 [28.4 kB]
Get:2 http://cdn-fastly.deb.debian.org/debian buster/main amd64 sudo amd64 1.8.27-1 [1245 kB]
debconf: delaying package configuration, since apt-utils is not installed
Fetched 1273 kB in 2s (645 kB/s)
Selecting previously unselected package lsb-base.
(Reading database ... 6674 files and directories currently installed.)
Preparing to unpack .../lsb-base_10.2019051400_all.deb ...
Unpacking lsb-base (10.2019051400) ...
Selecting previously unselected package sudo.
Preparing to unpack .../sudo_1.8.27-1_amd64.deb ...
Unpacking sudo (1.8.27-1) ...
Setting up lsb-base (10.2019051400) ...
Setting up sudo (1.8.27-1) ...
invoke-rc.d: could not determine current runlevel
invoke-rc.d: policy-rc.d denied execution of start.
Removing intermediate container 2640d37c5af3
 ---> 25d1cacde065
Step 4/6 : RUN useradd testuser --password sac19FwGGTx/A
 ---> Running in 225b17b4a3c3
Removing intermediate container 225b17b4a3c3
 ---> 9bbda90bbf13
Step 5/6 : RUN adduser testuser sudo
 ---> Running in f13b73d4fec1
Adding user `testuser' to group `sudo' ...
Adding user testuser to group sudo
Done.
Removing intermediate container f13b73d4fec1
 ---> 2a8b6171d657
Step 6/6 : RUN echo root:x11docker | chpasswd
 ---> Running in 7e2c2f2c4b80
Removing intermediate container 7e2c2f2c4b80
 ---><PHONE_NUMBER>af
Successfully built<PHONE_NUMBER>af
Successfully tagged mytest:latest

It's really odd.
Your test with the mytest image shows that the problem is not specific to x11docker.
The same image works quite well here. Password x11docker is correct.
Let's focus on this error message:
sudo: effective uid is not 0, is /usr/bin/sudo on a file system with the 'nosuid' option set or an NFS file system without root privileges?

Where/how do you store your Docker images? Is there anything special about this?

Where/how do you store your Docker images? Is there anything special about this?

Just perfect, that's great pointer. Yes, I've moved out the docker data folder from default /var/lib/docker to other folder due to partition space constraint.
I use the --data-root option in docker's ExecStart, see this stackoverflow link.
If I return it back to default folder, this problem is gone ;-)
Sure this is not x11docker issue.
I'll pursue --data-root separately.. Thank you very very much.

So we tracked it down, great!
You can check the ouput of mount for the partition with your Docker images. Probably it is mounted with nosuid.

Yes, you are right, it is mounted with nosuid. thanks again


Normalise local paths before comparing to the server

This PR changes how we compare local paths to paths on the server.

Use normalised paths as input to the Dropbox API when specifying an existing / expected file on the server.
Always normalise local paths before comparing them to the server.

Normalisation means conversion to lower case and unicode normalisation to NFC (i.e., composed versions of special characters: a + ° -> å).
This is necessary because Dropbox will always convert paths to their normalised version on the server. This is transparent to users for case changes -- the Dropbox API will generally find the file when passing an upper case version as well -- but not for unicode normalisation. Dropbox APIs will raise "not found" errors when querying for a path with the wrong unicode normalisation.
Performing unicode normalisation before comparison allows us keep the local form of unicode characters while still seeing the file as properly synced.
This fixes #671 because modern macOS file system APIs make it difficult (impossible?) to change the unicode normalisation of a local file (os.rename() will seemingly pass but without actually renaming the file). Instead, we need to live with what the OS gives us and accept it as "synced".
Note that this PR has some overlap with #409 which introduced "unicode conflict" errors on platforms that do allow file names which only differ in unicode normalisation in the same folder.

Closing since some base assumptions were wrong:

Dropbox servers do find files when querying for a path with the wrong unicode normalisation.
We don't want to accept files with different unicode normalization as synced on Linux because this introduces significant complexity when trying to find the corresponding local path for a Dropbox metadata entry.


QA: acceptable interactivity/performance

I noticed a couple issues that occurred to me more frequently than they should have after the last release.

Failed to delete the deployment.

Failed to load deployment details.

Failed to load the deployment.


load_table: TIMESTAMP import fails

Looks like no values are being sent over the server. Below is an example and file to reproduce the issue:
import pymapd
import pandas as pd

connection = pymapd.connect(user='mapd', password='HyperInteractive', host='localhost', dbname='mapd', port=7091, protocol='binary')
print (connection)

table_name = 'test'

command = 'drop table if exists %s' % (table_name)
[input.zip](https://github.com/mapd/pymapd/files/2422205/input.zip)

connection.execute(command)

create_table_str = 'CREATE TABLE IF NOT EXISTS %s (download_time TIMESTAMP(0), download_ip TEXT ENCODING DICT(8), download_path TEXT ENCODING DICT(8))' % (table_name)
connection.execute(create_table_str)

csv_file = 'input.csv'
df = pd.read_csv(csv_file)
df['download_time'] = pd.to_datetime(df['download_time'])
df.reset_index(drop=True, inplace=True)
connection.load_table(table_name, df)

What version of MapD Core are you using? And how did you install pymapd?
I haven't been able to repro yet with mapd-core master, both pymapd 0.4.0 installed from pip and pymapd master from git (also installed via pip). Tested with load_table_columnar and load_table_arrow, for Arrow used both 0.7.1 and 0.10.0.

unable to reproduce it.


chokidar forces app crash if removing nested directories in Windows

Using release 2.0.3 here is reproducible code.
Proof is at the end, crashes before getting to the end.
Occurs in Windows only.
Also proves probably should not blame fs.watch.
//

// this is about removing nested directories in Windows when chokidar is watching
// see https://github.com/paulmillr/chokidar/issues/730
//
// the problem is if after
/*
removing now
event, path  unlinkDir test-chokidar\nested\nested2
*/
//it exits the application in a crash, showing
/*
  - node  test-fs-watch.js[8916]: src\fs_event_wrap.cc:90: Assertion `wrap != nullptr' failed.
 1: node::DecodeWrite
 2: node::DecodeWrite
 3: node::Start
 4: v8::internal::interpreter::BytecodeDecoder::Decode
 5: v8::internal::RegExpImpl::Exec
 6: v8::internal::RegExpImpl::Exec
 7: v8::internal::Object::ToInt32
 8: v8::internal::Object::GetProperty
 9: v8::internal::NativesCollection<0>::GetScriptsSource
10: v8::internal::NativesCollection<0>::GetScriptsSource
11: ...
*/

//
// first prove fs.watch doesn't have the problem
//
setTimeout(() => {
  const fs = require("fs");

  const fse = require("fs-extra");

  if (process.platform !== "win32") {
    console.log("this code is meant to help find an issue with chokidar in Windows");
  } else {
    console.log("this code with fs.watch is expected to run fine, error EPERM is ok, doesn't crash");
  }

  fse.ensureDirSync("test-fs-watch/nested/nested2")

  const watchers = [];
  watchers.push(fs.watch("test-fs-watch"));
  watchers.push(fs.watch("test-fs-watch/nested"));
  watchers.push(fs.watch("test-fs-watch/nested/nested2"));

  for (let i = 0; i < watchers.length; i++) {
    let watcher = watchers[i];
    watcher.on("change", (eventType, filename) => console.log(`change ${1}: eventType, filename `, eventType, filename));
    watcher.on("error", (eventType, filename) => console.log(`error ${i}: eventType, filename `, eventType, filename));
  }

  setTimeout(() => {
    console.log("removing now");
    fse.remove("test-fs-watch/nested");
  }, 3000);

  setTimeout(() => {
    console.log("application was still running just fine");
    console.log("");
    //process.exit();
  }, 6000);
},
0 // run this one first
);

//
// then prove fs.chokidar has the problem
//
setTimeout(() => {
  const chokidar = require("chokidar");

  const fse = require("fs-extra");

  if (process.platform !== "win32") {
    console.log("this code is meant to help find an issue with chokidar in Windows");
  } else {
    console.log("this code with chokidar.watch is expected to crash, and only run fine after the issue will have been fixed");
  }

  fse.ensureDirSync("test-chokidar/nested/nested2")

  const watcher = chokidar.watch("test-chokidar");

  watcher.on("all", (event, path) => { console.log("event, path ", event, path); });

  setTimeout(() => {
    console.log("removing now");
    fse.remove("test-chokidar/nested");
  }, 3000);

  setTimeout(() => {
    console.log("application was still running just fine");
    console.log("apparently the issue has been fixed");
    process.exit();
  }, 6000);
},
10000 // run this one second
);

//

Progress in investigation.  It is a watcher.close() that causes the app crash.  Same assertion failure if in the test of fs.watch after the remove at 3000ms but before the "running just fine" log insert lines
  setTimeout(() => {
    for (let i = watchers.length - 1; i >= 1; i--) {
      let watcher = watchers[i];
      watcher.close();
    }
  }, 5000);

and the same if using
    for (let i = 1; i < watchers.length; i++) {

Apparently this only happens in Node 10, versus problem does not occur in Node 8.  Verified in Node 10.1.0 versus Node 8.11.2.  As mentioned above, Windows only.
May have to do with Node 10 having a change of architecture, including emitting a close event from an fs.FSWatcher per https://nodejs.org/api/fs.html#fs_class_fs_fswatcher .

Have submitted a patch here #731. It apparently works for us, but am not hundred percent sure it is ideal.
Also have submitted an issue at Node nodejs/node#21276, short sample code without chokidar.

Better patch in #732.


Trace source of atom updates

Discussed in https://github.com/jotaijs/jotai-devtools/discussions/135

Originally posted by kevinschaich April 11, 2024
When your state becomes large enough, effects tend to overlap – especially in applications with a lot of visual/input interactions (mouse movement, tracking positions of elements on the screen, dragging and dropping, etc). This can lead to render loops that can be hard to track down. If you have 10 buttons/sliders/keyboard shortcuts that all perform the same state update, it's really time consuming to figure out which one of them is causing issues.
Redux devtools has a nice feature that lets you "trace" the source of updates:

I think this would be a nice addition to the devtools, perhaps in the Time Travel tab behind a (default off) feature flag or setting.

I made some progress on this today. Looked into how Redux devtools is making it work – they create a new error stack and get the call stack, e.g.:
const trace = new Error().stack

However that gets us a lot of junk including stuff from React/Next/Webpack etc. so we'll need to clean it up.
Just a few lines of filters got me extremely close to what we'd need to support a v0 in Devtools:
    store.sub(CodeEditorCodeAtom, () => {
        const trace = new Error().stack
            ?.split('\n')
            .filter(
                (line) =>
                    !line.includes('vanilla.mjs') &&
                    !line.includes('react-dom-client.development') &&
                    !line.includes('<anonymous>') &&
                    !line.includes('atom-provider') &&
                    !line.includes('react.mjs'),
            )
            .join('\n')

        console.group('CodeEditorCodeAtom modified.')
        console.log('New Value:', store.get(CodeEditorCodeAtom))
        console.log(trace)
        console.groupEnd()
    })

And indeed on line 117 of BlockEditorHeader:
setCode(blockFile.content as string)


Send one message to Kafka per row

PR checklist:

Did you run ClangFormat ?
Did you separate headers to a different section in existing community code base ?
Did you surround proton: starts/ends for new code in existing community code base ?

Please write user-readable short description of the changes:
Instead of encoding multiple rows into one single message, now it sends one message per row, which is the expected behavior.
closes #274

Nice. I don't have the idea of why this simple change can work, but good to see we can enhance this so quickly

(Jove Github Bot) added it to the current sprint.

(Jove Github Bot) moved this ticket out of the GitHub project(up to 1200 tickets for one project).


Endpoint for form definition list

How about we add an endpoint for GET /forms to list all form definitions?

Implemented in https://github.com/nafundi/jubilant-garbanzo/pull/40 (though there are a couple TODOs).


update for qcschema 44 and 53

Adapts molparse.from_schema and molparse.to_schema to conform to molssi/qcschema#44 and molssi/qcschema#53.
This one's easy b/c all fields already in internal repr.

Codecov Report

Merging #6 into master will increase coverage by 2.59%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
+ Coverage   94.93%   97.52%   +2.59%
==========================================
  Files          28       31       +3
  Lines        1697     1859     +162
==========================================
+ Hits         1611     1813     +202
+ Misses         86       46      -40

Impacted Files
Coverage Δ

qcelemental/physical_constants.py
66.37% <ø> (+66.37%)
:arrow_up:

qcelemental/periodic_table.py
100% <ø> (ø)
:arrow_up:

qcelemental/molparse/from_schema.py
100% <ø> (ø)
:arrow_up:

qcelemental/molparse/to_schema.py
100% <100%> (ø)
:arrow_up:

qcelemental/__init__.py
100% <0%> (ø)
:arrow_up:

qcelemental/units/unit_helpers.py
100% <0%> (ø)

qcelemental/units/__init__.py
100% <0%> (ø)

qcelemental/units/ureg.py
97.1% <0%> (ø)

qcelemental/physical_constants/context.py
98.94% <0%> (+1.05%)
:arrow_up:

qcelemental/molparse/to_string.py
100% <0%> (+1.44%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e6fe2d...a876f02. Read the comment docs.


Exclude TFLite GPU kernel tests from non-GPU builds

Typing
bazel test //tensorflow/lite/...

on a system without a GPU results in multiple test failures as the build attempts to compile and test several types of GPU kernels.
This pull request adds the "gpu" tag to the TFLite tests that require a GPU to run. The default build configuration created by configure.py disables tests with this tag if the build is configured without GPU support.

@frreiss Can you please resolve conflicts? Thanks!


Prepare for the first release

Pre-release was successful!
https://github.com/cybozu-go/pod-security-admission/releases/tag/v0.0.1-alpha.0
https://quay.io/repository/cybozu/pod-security-admission?tab=tags


v1.0.1 bug when searching and renaming

when you start a search, renaming a file will rename the file as if it's in the original order, without the search.

You can better understand the bug by trying to edit the file path of something within a search, and will see that it lines up with the item that was previously there without the search.
For example, if item 3 was abc.txt,
and I start a search, item 3 might be example.txt, but the path will still be abc.txt, and when you think you're renaming example.txt, you're actually renaming abc.txt

very annoying bug, thanks David ;) fixed


:)

Yes, after the update, my game started crashing and I cannot play with skin changer.

Yes, after the update, my game started crashing and I cannot play with skin changer.

same here

https://github.com/R3nzTheCodeGOD/R3nzSkin/commit/9a2a0380e143594d29a8983b2ddeca31d1c6676d
With this update R3nzSkin and the injector still work for me. Download the repository, compile it and run it yourself.
Closes #864

9a2a038
With this update R3nzSkin and the injector still work for me. Download the repository, compile it and run it yourself.
Closes #864

The commit that u keep referencing has outdated offsets.

9a2a038
With this update R3nzSkin and the injector still work for me. Download the repository, compile it and run it yourself.
Closes #864

The commit that u keep referencing has outdated offsets.

My apologies I will use the correct commit hash: https://github.com/R3nzTheCodeGOD/R3nzSkin/commit/f42bb66c19381ef4e066ed8001055008692dc3c4


Is there a way to run elementor for non-angular apps?

Launch elementor and then, from the command line enter:
browser.ignoreSynchronization = true

There is a PR that includes this feature. I will merge it and then release a new version.

@andresdominguez ,
thank you for quick response!
and thank you very much for such a nice addition for protractor.
I've written the post(it's in Russian) about your tool on my blog, so that was already question from readers. :)

Amazing! Thank you for the blog post!

I just pushed a new version to npm. Try launching elementor
elementor https://google.com --nonAngular

Let me know if it works.

great! thanks a lot!


Exercise lists should be able to be sortable based on list name

The lists on the "My Lists" view need to be able to be sorted alphabetically by the list names.
If you have a good SF Symbol to represent Alphabetical sorting, then please use it.
This ticket is complete when:

[ ] There is an option in the list menu for "Alphabetical"
[ ] Tapping on "Alphabetical" sorts the lists based on the list names

I got this :D

PR is up: https://github.com/heyjaywilson/peakfit/pull/63

pls close issue


Optimize updateOrConcatWithKey

Fixes #403.

Whee! This shrinking stuff is cool. I remain concerned about its implications for the month moving garbage collector. That adds some code to each shrink for ... some sort of fancy footwork. @bgamari, could you advise on performance implications?


Add better support for using webpack

This makes it possible to consume the contrib extensions using webpack.
Closes #828

[clabot:check]

@edemaine it'll be nice to be able to use modern JS in contrib code.

@schmuli thanks for the PR.


Make onSlideChangeEnd work on Window Phone 8x

Device: Nokia Luma 920
OS: Windows Phone 8.1 Update
Problem: onSlideChangeEnd callback is not executed
Reproduction:

You touch a slide and swipe (let's say left)
You let the slide go
An animation should happen now, but it doesn't, the slide just snaps into it's new position

Description of Solution:
Since there is no animation happening after letting a slide go, the onSlideChangeEnd event is also not fired.
So why there is no animation? Answer: Currently the library starts the animation before setting the duration of the animation. WP8 doesn't like this and skips the animation. By setting the duration of the animation BEFORE starting the animation, everything works as expected.

Thanks


[API][RULES] A user should set several rules to select the issues

Is your feature request related to a problem? Please describe.
Current behavior allow wanted issues to be selected, formatted in a template and stored in a given directory.
All issues uses the same templates and are sent to the same directory.
Describe the solution you'd like
This behavior should be refined in rules giving the user the ability to select a given type of issue, apply the template and store them in a given directory. Each rule would be executed individually.
Example:

Rule 1 'issue opened' is using template A and stores the corresponding notes in dir_alpha
Rule 2 'issue closed' is using template B and stores the corresponding notes in dir_alpha
Rule 3 'label is lbl' is using template C and stores the corresponding notes in dir_bravo
Rule 4 'is opened, label is lbl, author is Human' is using template C and stores the corresponding notes in dir_alpha

Describe alternatives you've considered
Endlessly changing the config and the token when needed (extremely time consuming)
Additional context
The instance and the personal token should remain the same.

This is something I'm considering, so thanks for the detailed description.
Basically a duplicate of #9.


Seldon <=1.15 does not support k8s 1.25+

The k8s autoscaling/v2beta1 API has been migrated to v2, causing this issue in Seldon and blocking Seldon <=1.15 from using k8s 1.25+.  Seldon 1.16 will fix this and support k8s 1.25+.

v2beta2 is still available in 1.25
maybe there is another issue?

hmm i haven't tested it myself, but there was enough activity on that thread to think it correct. We could give it a try and see

@ca-scribner
ok, I see
you did a typo in your description, it is the first version of beta, not second
autoscaling/v2beta1
which is indeed removed in 1.25

Tested and confirmed the issue on microk8s 1.25
Status:
  Address:
    URL:  http://iris-model-default.admin.svc.cluster.local:8000/api/v1.0/predictions
  Conditions:
    Last Transition Time:  2023-07-21T09:41:01Z
    Reason:                Not all services created
    Status:                False
    Type:                  Ready
    Last Transition Time:  2023-07-21T09:41:01Z
    Reason:                Not all services created
    Status:                False
    Type:                  ServicesReady
  Description:             no matches for kind "HorizontalPodAutoscaler" in version "autoscaling/v2beta1"
  State:                   Failed
Events:
  Type     Reason         Age                   From                       Message
  ----     ------         ----                  ----                       -------
  Normal   CreateService  4m24s                 seldon-controller-manager  Created Service "iris-model-default-classifier"
  Normal   CreateService  4m24s                 seldon-controller-manager  Created Service "iris-model-default"
  Warning  UpdateFailed   4m19s                 seldon-controller-manager  Failed to update status for SeldonDeployment "iris-model": Operation cannot be fulfilled on seldondeployments.machinelearning.seldon.io "iris-model": the object has been modified; please apply your changes to the latest version and try again
  Warning  InternalError  78s (x16 over 4m22s)  seldon-controller-manager  no matches for kind "HorizontalPodAutoscaler" in version "autoscaling/v2beta1"

After Noha's confirmation we are closing this issue since Seldon 1.15 is not working with K8s 1.25.
CKF 1.8 should be able to run on K8s 1.25 since it will use Seldon 1.17


Remove inaccurate CHARTER.md - Alpha Omega exists as a series of sepa…

…rate contracts entered into directly between each member company and the LF

LF Legal advises that  there is no Alpha Omega charter.  Alpha Omega exists as a series of separate contracts entered into directly between each member company and the LF.
The CHARTER.md should be deleted because it is inaccurate.

I would be very supportive if the AO team wants to create a mission statement that describes the goals of the SIF and how the SIF operates.  However, there is no AO TSC as described in the current CHARTER.md


Starting OmRecorder in a service issue: PullableSource not working

Hi, I'm trying to implement an instance of OmRecorder in background service (For an art project). My recording code works flawlessly in my Activity but when I duplicate the same code in my service the code doesn't work. I managed to debug it and the line that stops the service:
private PullableSource mic() {
        return new PullableSource.Default(
                new AudioRecordConfig.Default(
                        MediaRecorder.AudioSource.MIC, AudioFormat.ENCODING_PCM_16BIT,
                        AudioFormat.CHANNEL_IN_MONO, 44100
                )
        );
    }

Does anyone know why this bit won't work in a service?
Thanks a lot!

Just found out the problem's source. I was doing that in a background service which should actually be a forground service to alleviate some of the restrictions with background services such as mic access.


[BUG] Bump system-agent in rancher

related to #43318 and https://github.com/rancher/system-agent/pull/152

Closing as it's tracked in https://github.com/rancher/rancher/issues/43318 and is effectively addressed by https://github.com/rancher/rancher/pull/44673


POC for Flagger

POC for Flagger

Flagger demo and related docs are kept here: https://github.com/giantswarm/workload-clusters-fleet/tree/demo/flagger/management-clusters/gauss/management-cluster

@ljakimczuk can we close this so this will be moved to pdv?

@gianfranco-l sure, we can


Support for Dart 3

Add support for Dart 3 and enable Material 3 for the app.

Looks good to me, will need to trying and get the pipeline fixed, but can do that in another MR if needed.

@lockieRichter the pipleine failed because I haven't taken care of the iOS build. Will do that in this PR and make another commit today.

Fixed pipeline. Will merge it now.


[Bug] Autofill not working from Bitwarden into basic auth prompt

Describe the bug
Autotype from bitwarden app into basic authentication prompt does not work.
It looks like the DDG app is sending its own Id to Bitwarden instead of the site name. This works correctly on authentication boxes embedded in a web page.
How to Reproduce

Go to a site requiring basic authentication
Click the bitwarden auto fill button

Expected behavior
Username/password are typed into the prompt
Environment
- DDG App Version: 5.102.2
- Device: Pixel 3
- OS: Android 12

This is under investigation: https://app.asana.com/0/414730916066338/1201426652769688/f

Sorry, closed the issue by mistake. 🙏

"It looks like the DDG app is sending its own Id to Bitwarden instead of the site name. This works correctly on authentication boxes embedded in a web page."
what do you mean by "own id"? I've tested this with a different password manager (LastPass) and it worked, so not sure what could be the issue with bitwarden.

Sorry, I didn't explain that very well.

Open a site using basic authentication
At the prompt, click the 'Bitwarden My Vault' button appearing at the top of the Gboard keyboard (Pixel 3).
The Bitwarden vault opens, but no entry is selected. The entry for that website should be searched for and opened

On Firefox/Chrome, the vault will open to the correct entry
Other open source chromium based browsers suffer from the same problem (entry is not searched for correctly)
I said "sending its own Id" because I occasionally get a message in Bitwarden saying There are no items in your vault for com.duckduckgo.mobile.android. Meaning bitwarden is receiving the app id from DDG browser instead of the website name.

Hopefully that's more clear!
Thanks

Still not working with S20 and the latest versions of DDG and Bitwarden.

Still an issue


Create firstpr.md

:warning: This is a Pull Request Template :warning:

Checkout all the things you've done by placing an x within the square braces below.

Proposed Changes

[] I've read the CODE OF CONDUCT and abide to it.
[] I've read the CONTRIBUTING.md
[] I've forked the repository.
[] I've created a branch and made my changes in it.
[] I understand opening a PULL REQUEST doesn't mean it will be merged for sure.

Please re-submit when you're ready 👍


[WIP] Integrate schema validation

What this PR does / why we need it:
Warn the user when their configuration has schema validation errors
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #828
Relates to #779 & #778

Codecov Report

Merging #853 (d8b3e71) into master (fe0104b) will decrease coverage by 0.09%.
The diff coverage is 38.27%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #853      +/-   ##
==========================================
- Coverage   21.49%   21.41%   -0.09%
==========================================
  Files          21       22       +1
  Lines        1521     1569      +48
==========================================
+ Hits          327      336       +9
- Misses       1142     1179      +37
- Partials       52       54       +2

Impacted Files
Coverage Δ

internal/agent/validate.go
0.00% <0.00%> (ø)

pkg/config/schemas/root_schema.go
71.42% <25.00%> (-6.84%)
:arrow_down:

pkg/config/config.go
57.02% <69.04%> (+1.32%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

found an issue while testing, so I converted it back to a draft


Menu items doesnt show

Thanks for a great app!
When I click on the green 'X' a small menu forms underneath.
But there is no text in the menu item.
Running on ubuntu 16.04
I will add more comments if I can trace the issue

Sorry for the late response, I've been swamped with school and work. I pretty sure the issue stems from me not extensively testing it on ubuntu. More than likely I'll just have to make an ubuntu build and make that available for you. Thank you for downloading!


who can tell me how to fix the header on the top ?

thankx.

not the best

DIV+CSSレイアウト

@dam360210 Please think about how to rewrite your question and ask it in a more appropriate forum, such as stackoverflow
Your question does not appear to have anything to do with moment, so this (the issue-tracker for the moment library) is a completely inappropriate place to request help for your design issue.


Make sure to provide side effects free bundle

It should be checked if it is possible to provide a side effects free package by adding hints for bundlers (e.g. webpack) that make it possible to better tree shake the package.

Proposed solution:

[ ] Get a list of all files that have side effects (e.g. https://github.com/SICKAG/collage/blob/main/src/elements/fragment-element.ts#L84)
[ ] Add an entry sideEffects with a list of all files containing such effects

Further Information:

https://github.com/SICKAG/collage/blob/main/src/elements/fragment-element.ts#L84
https://sgom.es/posts/2020-06-15-everything-you-never-wanted-to-know-about-side-effects/

If this feature is considered, I would be happy to provide a pull request for this feature.

Hey @weblogixx
I would be happy to geht a pullrequest in that issue, if youbare still interested.


Fix ScrollGestureEndedEvent type.

What does the pull request do?
While working on #7964 I noticed that the event type in the field for ScrollGestureEndedEvent is incorrect: it's a ScrollGestureEndedEventArgs object that gets passed, not a ScrollGestureEventArgs.
Breaking changes
Yes :D That's why it's in a different PR to #7964 .

You can test this PR using the following package version. 0.10.999-cibuild0019841-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


2.2 HOTFIX -- [#4956] defusedxml only when ingesting

Resolves #4956

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

[Enter positive test case here]

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-4959


RuntimeError: CUDA error: out of memory

The following errors were encountered in training.
PyTorch 1.2.0
python 3.6
Traceback (most recent call last):   File "train.py", line 257, in <module>     train()   File "train.py", line 97, in train     ssd_net = build_ssd('train', cfg['min_dim'], cfg['num_classes'])   File "/media/disk/wudi/ssd.pytorch/ssd.py", line 207, in build_ssd     base_, extras_, head_ = multibox(vgg(base[str(size)], 3),   File "/media/disk/wudi/ssd.pytorch/ssd.py", line 136, in vgg     conv2d = nn.Conv2d(in_channels, v, kernel_size=3, padding=1)   File "/media/disk/wudi/.local/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 330, in __init__     False, _pair(0), groups, bias, padding_mode)   File "/media/disk/wudi/.local/lib/python3.6/site-packages/torch/nn/modules/conv.py", line 41, in __init__     out_channels, in_channels // groups, *kernel_size)) RuntimeError: CUDA error: out of memory
What is the reason for this mistake? Thanks.

maybe you can use this command to check your gpu usage condition : nvidia-smi
Then add this thing in front of your code to use the free graphics card :
CUDA_VISIBLE_DEVICES="6" python train.py  (6 is just an example)

OK, I use that command, and it works. Thanks!

OK, I use that command, and it works. Thanks!

you're welcome :smile:


🌗 Dark theme style fixes

I know you like dark themes @brendt, here, I fixed some of the leftovers that needed to be fixed on dark theme. Here are the screenshots of some of them:

Nice! @MohammadAlhallaq will you make sure to sync up so you're not doing the same things locally? :)


Atomic on all systems?

I might be incorrect about this, but I believe that os.Rename is only atomic on a subset of the operating systems supported by golang.  It might be good to mention this is documentation.

This is a good point. I'll make the docs clear about this.


AZP configuration updates

Currently we have a forced CI cron build every Sunday. With this PR, Monday-Saturday now have a full CI build too IF code has been changed. This will update static analysis more regularly, until we get the quality gate to a place where it can be enabled as a PR check.
I also moved the "latest" builds to use the ci-common Docker image and removed explicit lib paths from their cmake overrides. They pointed to the default install location for each lib, so weren't necessary.
Implemented macOS python install improvement suggested by Christina Tempelaar-Lietz (OpenEXR). This removes a lot of warnings from the install log, potentially speeding up mac builds.

Here is the problem:
09:52:33.231 ERROR: Error during SonarQube Scanner execution
java.lang.IllegalStateException: You're not authorized to run analysis. Please contact the project administrator.

SonarCloud is back up and running:
https://sonarcloud.io/dashboard?id=AcademySoftwareFoundation_OpenColorIO
I've re-enabled the condition limiting it to the cron builds, but the above is the output from the run in this PR.

Re: the macos python script, for posterity --
It seems that the SDK headers installer provided with xcode-10 is only intended as a stopgap for builds that rely on macros with hardcoded references to /usr/include; the installer isn't provided for xcode-11 / macos-10.15.
Instead, it might make more sense to pass the SDK path to clang via the -isysroot compiler flag: e.g.,
export CFLAGS= "-I$(brew --prefix openssl)/include -Wno-nullability-completeness-isysroot $(xcrun --show-sdk-path)"
Source: https://apple.stackexchange.com/questions/337940/why-is-usr-include-missing-i-have-xcode-and-command-line-tools-installed-moja

Thanks Zach! Timely insight as our macOS CI hit a wall related to this exact thing today, because of an update of our hosted CI macOS agent. I'll give this a whirl.


docs: social media shield.io

This change is

Closing this PR as it doesn't change anything.


fix(es/transforms/compat): comment position in class and arrow

Description:
code
// 123
class A extends B {
}

before
var A = // 123
(function (B) {})(B)

after
// 123
var A = (function (B) {})(B)

code
const a = () => /* 123 */ 1

before
const a = function a() {
   /* 123 */ return 1
}

after
const a = function a() {
  return /* 123 */ 1
}

BREAKING CHANGE:

Related issue (if exists):
fixes #3846

Some spans were intentionally removed by https://github.com/swc-project/swc/pull/3242
cc @0xe

Some spans were intentionally removed by #3242 cc @0xe

#3242 only removed span of function args like
function (B) {} (B)
//              ^

Thanks! Then I'll review it


Added Records/Structs

Several major changes were made.

Structs/records were added

They are implemented as heap allocations on the vm-level.

The VM was rewritten

There is no seperate function for executing single instructions.

Structs/records have already been implemented to a satisfactory degree - they currently support

Read/write properties

Default values

Inheritance (non-circular)
They will eventually support
Constructors, operator overloads, and some form of object messaging


[BUG] Error Converting SKData

Description
I am using SkiaSharp to generate an image that represent a transaction receipt that can be shared via my Xamarin app.  When I try to convert the SKData to array I get an error message:
{System.MethodAccessException: Method System.ReadOnlySpan1[System.Byte]..ctor(void*,int)' is inaccessible from method `SkiaSharp.SKData.AsSpan()'
The code bellow works fine with the Android version of the app, both not with iOS.
Code
    public static byte[] GenerateImage(ImageReceipt imageReceipt)
    {
        FontFamilyName = ImageReceipt.FontFamilyName;
        SKBitmap bitmap = GetReceiptBitmap(imageReceipt);
        using (SKImage image = SKImage.FromBitmap(bitmap))
        using (SKData data = image.Encode(SKEncodedImageFormat.Png, 80))
        {
            return data.ToArray();
        }
    }

Expected Behavior*
Return the Byte Array
Exception : {System.MethodAccessException: Method System.ReadOnlySpan1[System.Byte]..ctor(void*,int)' is inaccessible from method `SkiaSharp.SKData.AsSpan()'

Basic Information

Version with issue:  <IP_ADDRESS>
Last known good version:  N/A
IDE:   Visual Studio

Android:8.1
iOS:  <IP_ADDRESS>

Sorry for the very late response. Is this still an issue? It may have been related to an incompatibility of the Xamarin/.NET/.NET Standard versions?

Hello there,
This is no longer an issue for me.
Using updated versions and now it works fine.
Thanks for your help!
On Tue, Jun 9, 2020, 11:31 PM Matthew Leibowitz<EMAIL_ADDRESS>wrote:

Sorry for the very late response. Is this still an issue? It may have been
related to an incompatibility of the Xamarin/.NET/.NET Standard versions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mono/SkiaSharp/issues/1166#issuecomment-641682712,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACKLYO2PM2OLZRFXOM5UT3TRV3WABANCNFSM4LCTVYGA
.

Awesome! Glad to hear the updates fixed things.


changed configprovider from adapter library to meshkit for thread safety

Signed-off-by: ashish<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Revolyssup CI checks needs to be addressed...

Seeing a couple golint check failures - https://github.com/meshery/meshery-consul/pull/205/checks?check_run_id=3929858632#step:3:76

Changes weren't made according to latest meshkit version. There were few changes in the way helm install works in latest meshkit. Pushed the latest changes now.


Figure out brew simulation

[x] Color doesn't change if too cold?
[x] Steam if water is hot enough?
[x] Tea type and heat determines color change rate?
[ ] Adding milk changes color to milky?
[ ] Adding milk and lemon curdles and becomes horrible?
[x] Adding multiple types of tea takes the darkest color

http://wiki.unity3d.com/index.php/Particle_Spiral_Effect


Create Windows batch file for starting/stopping the bot

Create a Windows .bat file to start/stop the bot. Base it off the current linux bxbot.sh

Alternatively you thought about running as a windows service e.g. https://nssm.cc/usage. Start|Stop|Status would then be trivial batch commands

Yup, good idea. I use a linux service for the bot on my servers, but be tidy to have windows equivalent.  Got for it ;-)
P.S. There's a shed load of stuff going back onto master this weekend (REST API and the splitting of the bot into modules) from bxbot-restapi branch - so might be worth forking after that to save some pain...  will be gradle 'ing it this weekend too :-)


Changed the header background to an image

Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Fixes #3

@jgarcia45 Please tag someone for review

@jgarcia45 Please add a background overlay so the font on top of the image is readable.
Something like this

background-image: linear-gradient(to bottom, rgba(0, 0, 0, .2) , black), url(https://static.planetminecraft.com/files/resource_media/screenshot/1922/5bd170b…-1559136006.png) !important;

@khattakdev
I created a new pull request to add the linear-gradient to the image.


Add 2 views "About" and "Contact"

To let coworkers discover views layout on development folders.

Done.


IParameterConsumer does not work with @Option in @ArgGroup

When using a parameter consumer on an option inside an argument group, the @ArgGroup-annotated field is not instantiated.
Running this test with version 4.2.0
public class Test implements Runnable {

  @ArgGroup
  Group group;

  static class Group {
    @Option(names = "-foo", parameterConsumer = TestConsumer.class)
    String foo;
  }

  static class TestConsumer implements IParameterConsumer {
    @Override
    public void consumeParameters(Stack<String> args, ArgSpec argSpec, CommandSpec commandSpec) {
      argSpec.setValue(args.pop());
    }
  }

  public static void main(String[] args) {
    new CommandLine(new Test()).execute("-foo", "value");
  }

  @Override
  public void run() {}
}

throws:
PicocliException: Could not set value for field java.lang.String Test$Group.foo to value while processing argument at or before arg[1] 'value' in [-foo, value]: picocli.CommandLine$PicocliException: Could not set value for field java.lang.String Test$Group.foo to value

caused by a NPE at line 10145 in picocli.CommandLine.Model.FieldBinding.set(T).
Is this the intended behavior or a bug?
thanks

Thank you for raising this!
I will investigate when I get to my PC.

If you need a quick workaround, I suspect you can avoid the NPE if you instantiate the group at the field declaration:
  @ArgGroup
  Group group = new Group();

I need to investigate further later.

Fixed in master.
The fix will be part of the upcoming picocli 4.3 release.
Thank you again for raising this!


Spectral integrator of the nonautonomous tangent linear system, after factoring out of the exponential part

Identical to eigenvalue problem on the full Hamiltonian matrix.
Tried with no success yet. Conditioning problem ?


blog: VSCode extension release post.

Blog post for the recently released VSC extension.
Minor TODO: I asked for an editable version of our default Wave blog post banner so that I can add VSCode logo - to be less generic and more maketingy :)

Great suggestion @mtanco! Done.


Set up associations

In this PR,

I created a new Classroom class.
I created the relationship between Classroom and Students classes.
I created a new Book class.
I created a new Rental class.
I created a relationship between Person and Book using the intermediate class Rental.

@julie-ify , what tests exactly do you want me to add ?. I used the template Microverse gave me, if it is not working, please take it up with them and stop invalidating my code review. So far, I have used the same README and nobody has complained!.

@Otegaa if you have no tests, then the link should be removed. None of the links on the headers should be broken.
If you think the changes are unfair, kindly request a second opinion from a Senior Code Reviewer.
Here is the link: https://airtable.com/shrQAqnBwek5a0O0s


pack(): introduced a new helper for packing parts into a 2D footprint.

Also added documentation for the new facility as well as a nascent CONTRIBUTING.md to collect pointers to the tools used by developers of this project.

Very nice, thank you!
As an extension, have you considered making pack a little more general? As is it's defined as:
def pack(objects: Collection[Part], padding: float) -> Collection[Part]:

which implies only Parts are valid but it seems as though there is nothing here that wouldn't work for all Shapes - i.e. the user might have a collection of Faces they would like to pack for display as in show(pack(my_face_list, 10))

Thanks for the speedy review+merge!

As an extension, have you considered making pack a little more general? As is it's defined as:

Good idea: done in https://github.com/gumyr/build123d/pull/377.


#61 - Adding Tailwind CSS and adjust the fetch in App.js

@Hanna-N9 Fantastic work getting this setup!!


Some (all?) data isn't processed to escape certain characters.

For instance, in a test case, I have an item that has a summary of
Come on!' 'Everybody says "come on!" here,' thought Alice, 'and why it is all the children she.
Which results in:
<meta name="twitter:description" content="Come on!' 'Everybody says "come on!" here,' thought Alice, 'and why it is all the children she.">
Since the quotation marks in the content aren't being escaped or converted to HTML entities, it messes up the meta tag.
For now, I'm passing all the data through htmlspecialchars
        Head::setTitle(htmlspecialchars($product->name));
        Head::setDescription(htmlspecialchars($product->summary));
but it might be best to do this within your package itself so other people don't run into the problem

You're totally right.
I'm leaving this package. You can contact me if you want to keep maintaining it.


Update Webstorm to use the new built in compiler instead of the filewatcher

This is now the recommended way.
See blog
Section TypeScript 1.4 and Built-in compiler
Nice job!!
Thanks,
Dan

hi @delebash
feel free to do a PR if you like.  otherwise i'll put this on the todo list :smile:
thanks for the input


Fix exception when Logging warning

The following exception occurs every time the client encounters the OpenAI Chat completion SSE event data: [DONE].
** (exit) an exception was raised:
    ** (Jason.DecodeError) unexpected byte at position 0: 0x64 ("d")
        (jason 1.4.1) lib/jason.ex:92: Jason.decode!/2
        (openai_ex 0.5.8) lib/openai_ex/http_sse.ex:63: OpenaiEx.HttpSse.next_sse/1
        (elixir 1.16.2) lib/stream.ex:1626: Stream.do_resource/5
        (elixir 1.16.2) lib/stream.ex:943: Stream.do_transform/5
        (elixir 1.16.2) lib/stream.ex:1052: Stream.do_transform_inner_enum/7

This happens as the value in the acc is the above string which is not a valid JSON.
  defp next_sse({acc, ref, task}) do
    receive do
      {:chunk, {:data, evt_data}, ^ref} ->
        {tokens, next_acc} = tokenize_data(evt_data, acc)
        {[tokens], {next_acc, ref, task}}

      {:done, ^ref} ->
       # >>>>>>>>>>>>>>>>>>> this call fails
        if acc != "", do: Logger.warning(inspect(Jason.decode!(acc)))
        {:halt, {acc, ref, task}}

      {:canceled, ^ref} ->
        Logger.info("Request canceled by user")
        {:halt, {acc, ref, task}}
    end
  end

The PR changes the above call with:
defp next_sse({acc, ref, task}) do
    receive do
      {:chunk, {:data, evt_data}, ^ref} ->
        {tokens, next_acc} = tokenize_data(evt_data, acc)
        {[tokens], {next_acc, ref, task}}

      {:done, ^ref} when acc == "data: [DONE]" ->
        {:halt, {acc, ref, task}}

      {:done, ^ref} ->
        if acc != "", do: Logger.warning(%{message: "Unexpected value in sse 'acc' after ':done' event received", value: acc})
        {:halt, {acc, ref, task}}

      {:canceled, ^ref} ->
        Logger.info("Request canceled by user")
        {:halt, {acc, ref, task}}
    end
  end

@aramallo Thanks for catching this corner case. I hadn't run into a situation where the string was not JSON.
The PR looks ok to me. If you run mix format, I'll merge it.

Also, for future reference, could you provide an example where this corner case is exercised.

Hi @restlessronin ,
Thanks for getting back to me.  I will format and resubmit.
BTW, I just spotted another issue.
For example, sending the an invalid request now generates the following log
iex(21)> chat_stream = openai |> ChatCompletion.create(chat_req, stream: true)
%{
  status: 400,
  headers: [
    {"date", "Tue, 09 Apr 2024 08:58:57 GMT"},
    {"server", "uvicorn"},
    {"content-length", "147"},
    {"content-type", "application/json"}
  ],
  body_stream: #Function<52.53678557/2 in Stream.resource/3>,
  task_pid: #PID<0.2014.0>
}
iex(22)> chat_stream.body_stream |> Stream.flat_map(& &1) |> Enum.each(fn x -> IO.puts(inspect(x)) end)
2024-04-09 09:58:58.669 [warning] pid=<0.789.0> application=openai_ex module=OpenaiEx.HttpSse function=next_sse/1 line=66
[message: "Unexpected value in sse 'acc' after ':done' event received", value: "{\"object\":\"error\",\"message\":\"Conversation roles must alternate user/assistant/user/assistant/...\",\"type\":\"BadRequestError\",\"param\":null,\"code\":400}"]

I guess this was the purpose of your Logger.warning call in the first place, but the problem is that we are logging a perfectly valid reponse payload and we are not adding it to the request state.
To see what is going on you can check this curl request
.../sn-ml-prompts❯ curl http://<IP_ADDRESS>:8000/v1/chat/completions \
    -H "Content-Type: application/json" \
    -d '{
        "model": "CodeLlama-34b-Instruct-hf",
        "messages": [
            {"role": "system", "content": "You are a helpful assistant."},{"role": "system", "content": "You are a helpful assistant."},
            {"role": "user", "content": "Who won the world series in 2020?"}
        ],
       "temperature":0,
       "frequency_penalty": 0,
       "presence_penalty":0.1
    }' -v
*   Trying <IP_ADDRESS>:8000...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8000
> POST /v1/chat/completions HTTP/1.1
> Host: <IP_ADDRESS>:8000
> User-Agent: curl/8.4.0
> Accept: */*
> Content-Type: application/json
> Content-Length: 385
>
< HTTP/1.1 400 Bad Request
< date: Tue, 09 Apr 2024 08:54:09 GMT
< server: uvicorn
< content-length: 147
< content-type: application/json
<
* Connection #0 to host <IP_ADDRESS> left intact
{"object":"error","message":"Conversation roles must alternate user/assistant/user/assistant/...","type":"BadRequestError","param":null,"code":400}%

So OpenaiEx will return the following, without the actual error content.
%{
  status: 400,
  headers: [
    {"date", "Tue, 09 Apr 2024 08:36:01 GMT"},
    {"server", "uvicorn"},
    {"content-length", "147"},
    {"content-type", "application/json"}
  ],
  body_stream: #Function<52.53678557/2 in Stream.resource/3>,
  task_pid: #PID<0.2001.0>
}

The ideal would be not to log this as a warning but to add the error event to the body_stream so that we can consume on status != 200 or to returned it the struct like in the following example
%{
  status: 400,
  error: %{
   "message" => "Conversation roles must alternate user/assistant/user/assistant/...",
    "type" => "BadRequestError",
    "param" => null
    }
  headers: [
    {"date", "Tue, 09 Apr 2024 08:36:01 GMT"},
    {"server", "uvicorn"},
    {"content-length", "147"},
    {"content-type", "application/json"}
  ],
  body_stream: #Function<52.53678557/2 in Stream.resource/3>,
  task_pid: #PID<0.2001.0>
}

Hi @aramallo, good points. Most of my personal usage has been in livebook, where the logger warning is sufficient.
In a non-interactive use-case, I was aware that it was likely a problem, but I've hesitated to try to fix it ex-ante. Now that I have a concrete example, and a suggested fix, let me think on this, and get back to you.

Hi @restlessronin thanks for taking this on board. I'll try to reproduce the DONE later on today and send it to you.  BTW we are using your lib in a non-interactive way, but so far I really like how it is organised vs other options  :-)

Hi @aramallo, I will go ahead and merge this so the PR doesn't remain dangling. I would prefer to have a clear example of the need for handling [[DONE]] before I release it though.

Thanks a lot @restlessronin and sorry for not getting back on time

@aramallo I wound up with an alternate fix for this bug. I don't know if you have had a chance to test 0.5.9 (or 0.6.0 which has breaking FQN module name changes) to make sure your problem remains fixed.


Add "network" field to lookup rpc methods in addition to "addresses", "min_block", "max_block"

The "network" field should be sent along with the other "advertisement" fields.

Duplicate: https://github.com/yzernik/squeaknode/issues/632

Duplicate: https://github.com/yzernik/squeaknode/issues/632

https://github.com/yzernik/squeaknode/pull/749

https://github.com/yzernik/squeaknode/pull/749


Disappearing toolbar for Cloud Instances

No error appears

@romanblanco if this is happening in E release, we will need a BZ.  Thx, Dan

@romanblanco fyi, I don't see this on my system, so please close it it has been fixed.


not setting query to null, preventing NPE

addQuery has no check that query exists before using it.

Thank you @attemptoHeinz -- it's always very interesting how such simple but not insignificant bugs can lie dormant for so long! I guess everyone else is just constructing a new object every time?
Appreciated.


Fix resolution of GpuRapidsProcessDeltaMergeJoinExec expressions [databricks]

When performing a Delta Lake merge, the source or target dataframe could be generated by an aggregation.  Aggregations can rewrite the expressions of upstream nodes.  The custom RapidsProcessDeltaMergeJoin node used for GPU-accelerated Delta Lake merges was caching the original source and target dataframe attributes from the logical plan, and this could end up being incorrect once rewritten by an aggregation physical plan transformation.
This fixes the problem by pushing the original CPU expressions up to the "top-level" of the RapidsProcessDeltaMergeJoin so that Catalyst can rewrite the expressions when necessary during planning and optimization.

build


test: Fix possible race in waitForNPods helper function

Previously, WaitforPods() could prematurely return when GetPods() returned no pods (e.g. WaitforPods() had been immediately called after sending a create pods request).
To avoid this, we should wait (by returning false) when the pod list is empty instead of setting the minRequired.
This makes the function to fail when a user intentionally set minRequired to 0 in WaitforNPodsRunning(), but I don't see any sane use case where setting minRequired to 0 would make sense.
Example of such race:
time="2020-02-29T13:05:44Z" level=debug msg="applying /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/k8s-1.11-gopath/src/github.com/cilium/cilium/test/k8sT/manifests/log-gatherer.yaml in namespace kube-system"
time="2020-02-29T13:05:44Z" level=debug msg="running command: kubectl apply --force=false -f /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/k8s-1.11-gopath/src/github.com/cilium/cilium/test/k8sT/manifests/log-gatherer.yaml -n kube-system"
cmd: "kubectl apply --force=false -f /home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/k8s-1.11-gopath/src/github.com/cilium/cilium/test/k8sT/manifests/log-gatherer.yaml -n kube-system" exitCode: 0 duration: 92.720906ms stdout:
serviceaccount/cilium unchanged
daemonset.apps/log-gatherer unchanged

time="2020-02-29T13:05:44Z" level=debug msg="running command: kubectl -n kube-system get pods -l k8s-app=cilium-test-logs -o json"
cmd: "kubectl -n kube-system get pods -l k8s-app=cilium-test-logs -o json" exitCode: 0 duration: 43.672802ms stdout:
{
    "apiVersion": "v1",
    "items": [],
    "kind": "List",
    "metadata": {
        "resourceVersion": "",
        "selfLink": ""
    }
}

Fix #10277.

This change is

test-me-please

test-me-please

Coverage increased (+0.03%) to 45.633% when pulling d4c2122e4d4554e99ab68713cd9459d05437d5c1 on pr/brb/ci-fix-get-iface-flake into b3b29b325faa45ffde6b18983976813f3c768b4d on master.

test-me-please


Remove Minted and TokensBurned events

What was the rationale behind removing these events?

@yosriady They were redundant, because the information is already conveyed in Transfer events as is suggested in the ERC20 document. That is, a minting operation is logged as Transfer(0, to, amount), and the reverse for a burn. Additionally the from argument is indexed so it's efficient to filter only the minting events.


C Bindings

This is a tracking issue.
TLDR: I think it would be important for many things to maintain C bindings, cf. https://github.com/huggingface/safetensors/issues/222#issuecomment-1509988573 and I'll fill in the proper issue description later. Thanks

Hi @SomeoneSerge
I've been implementing a C-API binding for safetensors lately.
If you want to give it a try, happy to help.
-> https://github.com/huggingface/safetensors/pull/259


Create Index

This file is an initial mapping of the presentation slides, very much a draft at the moment.

This is great start
What you need to do next is to actually use the file names and you don't need the ###
here is what the index should look like: https://github.com/DinisCruz/Book_SecDevOps_Risk_Workflow_Build/blob/master/manuscript/Book.txt

Thanks Dinis, that is very helpful & I should get that done tomorrow once I've finished the references.

The file titles on lines 32 (Dataleaks.pt.md) & 108 (Open-source.pt.md) end with '.pt.md' --will these be readable by leanpub/git, or should I change the titles?

.pt.md is fine
Although I don't think that we will need a file for each of those topics


Pitch halving errors with short audio

Whenever I run pYAAPT on a short sentence (one-word), I get a few pitch halving errors in the resulting pitch. The problem is worse when there are long silences before and after the word. These are high-quality studio recordings of a professional voice talent. What could I try to get rid of these errors? In general, I feel that pYAAPT performs better than other methods I am comparing it to (WORLD, swipe, reaper), but this is a bit of a deal breaker. It is also one of the slowest methods.

Hi, @dreamk73 thank you again for your feedback.
Regarding your pitch halving problem, could please send a sample (or some samples) of problematic speech signals? There are a couple of things that I want to test.
It could be just a matter of adjusting the parameter PITCH_HALF_SENS from the PitchObj. But since this parameter acts together with mean pitch and pitch standard deviation values, eventually it could not work well with your short speech samples.
It is also possible that your problem is caused by this part of the code. I had marked it to removal, but at the time I could not figure out why the original authors excluded it. Therefore, since it was not affecting me, it is still there.
I could be also a corner case that was not foreseen by the original authors, like the issue 10 that we fixed recently.
Finally, it could be also a limitation from the algorithm itself. The original paper was published in 2008 and have not received any major update since then. Anyway, I still have to do a more intense research in order to verify if anyone had come up with a substantial improvement during this period.
So, I want to check all possibilities before giving you a feedback.

Regarding the speed problem, it was a bit inevitable, since YAAPT combine a set of computationally expensive tests.
Last week I published the AMFM_decompy 1.0.10 release and added some notes about this matter to the documentation. As I mention there, the original authors have added a feature at the Matlab code that results in a speed improvement at the cost of pitch detection accuracy. Personally I do not consider this trade off very worthy, so I prefer to focus on adding support for numba and CUDA in a future release.
I am already working on some improvements, but unfortunately this numba release will not come soon, since that I need to code some preparatory steps before it. Anyway, most of AMFM_decompy upgrades in the last years were basically just bug-fixing releases, so I guess that it is about time to add some major improvements.

Thanks. I looked at the PITCH_HALF_SENS parameter in pYAAPT.py but I found out that in this example it never got to that line because self.PITCH_HALF was 0. However, when I commented out the first two conditions in the part of the code that you mentioned, the problem went away at least in this particular example. I will run it on a few more to make sure.

Ok, nice to know that it helped. I will be waiting then for your next results, and in case this modification really solve your issues, I will add it to the code.

The code lines that were causing the bug were removed in the 1.0.12 releas. Since no other problems were reported, I will consider this issue solved.

The code lines that were causing the bug were removed in the 1.0.12 releas. Since no other problems were reported, I will consider this issue solved.


Use of Java Incompatible with AWS Lambda

Because of the use of java with XMLValidator.java, this package cannot be used in a Node Lambda environment.  There's not much else to report on this. Please consider using something other than a java file within this Node package.

The description of this library clearly states the dependency:

A (XSD) schema validator for NodeJS that uses Java to perform the actual validation.

Please consider using another library that does the job for you in that specific environment.
Or be constructive and contribute an alternative, non-Java version.
Thanks!

Or be constructive

Haha nice. Not after that comment my friend.

To give you a few pointers why this cannot be accomplished easily, using NodeJS only can be found in the why section.


Add compiler option /feature and update the /arch (ARM64) option

Add the new compiler option /feature. This option enables certain ARM64 architecture features when targeting an Arm A-Profile architecture extension.
Update the arguments of compiler option /arch (ARM64) to include other versions for Arm8-A and Arm9-A. /arch (ARM64) also has the option to include feature arguments used in /feature option.
These changes apply to Visual Studio version 17.10.

@TylerMSFT

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


mobilenet-facenet on mobile

@davidsandberg @rtkaleta @scotthong @rasmushaglund @jithinodattu

@ronyuzhang please be more descriptive with the issue. It's not conducive to spam people's handles.
Thank you


Cleaned up linux recipe, supporting recent repository improvements

Much more clean, looks good! :+1:


Error: cannot find symbol GoogleSignin

Hello, I keep getting this error while compiling, any idea?
/Users/pita/code/GoogleFitness/node_modules/react-native-google-fit/android/src/main/java/com/reactnative/googlefit/GoogleFitManager.java:182: error: cannot find symbol
        GoogleSignInAccount gsa = GoogleSignIn.getAccountForScopes(mReactContext, new Scope(Scopes.FITNESS_ACTIVITY_READ));
                                  ^
  symbol:   variable GoogleSignIn
  location: class GoogleFitManager
/Users/pita/code/GoogleFitness/node_modules/react-native-google-fit/android/src/main/java/com/reactnative/googlefit/GoogleFitManager.java:183: error: cannot find symbol
        Fitness.getConfigClient(mReactContext, gsa).disableFit();
               ^
  symbol:   method getConfigClient(ReactContext,GoogleSignInAccount)
  location: class Fitness
2 errors
:react-native-google-fit:compileReleaseJavaWithJavac FAILED

My build.gradle:
// Top-level build file where you can add configuration options common to all sub-projects/modules.

buildscript {
    repositories {
        jcenter()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:2.2.3'

        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }
}

allprojects {
    repositories {
        mavenLocal()
        jcenter()
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url "$rootDir/../node_modules/react-native/android"
        }
    }
}

My app/build.gradle
apply plugin: "com.android.application"

import com.android.build.OutputFile

/**
 * The react.gradle file registers a task for each build variant (e.g. bundleDebugJsAndAssets
 * and bundleReleaseJsAndAssets).
 * These basically call `react-native bundle` with the correct arguments during the Android build
 * cycle. By default, bundleDebugJsAndAssets is skipped, as in debug/dev mode we prefer to load the
 * bundle directly from the development server. Below you can see all the possible configurations
 * and their defaults. If you decide to add a configuration block, make sure to add it before the
 * `apply from: "../../node_modules/react-native/react.gradle"` line.
 *
 * project.ext.react = [
 *   // the name of the generated asset file containing your JS bundle
 *   bundleAssetName: "index.android.bundle",
 *
 *   // the entry file for bundle generation
 *   entryFile: "index.android.js",
 *
 *   // whether to bundle JS and assets in debug mode
 *   bundleInDebug: false,
 *
 *   // whether to bundle JS and assets in release mode
 *   bundleInRelease: true,
 *
 *   // whether to bundle JS and assets in another build variant (if configured).
 *   // See http://tools.android.com/tech-docs/new-build-system/user-guide#TOC-Build-Variants
 *   // The configuration property can be in the following formats
 *   //         'bundleIn${productFlavor}${buildType}'
 *   //         'bundleIn${buildType}'
 *   // bundleInFreeDebug: true,
 *   // bundleInPaidRelease: true,
 *   // bundleInBeta: true,
 *
 *   // whether to disable dev mode in custom build variants (by default only disabled in release)
 *   // for example: to disable dev mode in the staging build type (if configured)
 *   devDisabledInStaging: true,
 *   // The configuration property can be in the following formats
 *   //         'devDisabledIn${productFlavor}${buildType}'
 *   //         'devDisabledIn${buildType}'
 *
 *   // the root of your project, i.e. where "package.json" lives
 *   root: "../../",
 *
 *   // where to put the JS bundle asset in debug mode
 *   jsBundleDirDebug: "$buildDir/intermediates/assets/debug",
 *
 *   // where to put the JS bundle asset in release mode
 *   jsBundleDirRelease: "$buildDir/intermediates/assets/release",
 *
 *   // where to put drawable resources / React Native assets, e.g. the ones you use via
 *   // require('./image.png')), in debug mode
 *   resourcesDirDebug: "$buildDir/intermediates/res/merged/debug",
 *
 *   // where to put drawable resources / React Native assets, e.g. the ones you use via
 *   // require('./image.png')), in release mode
 *   resourcesDirRelease: "$buildDir/intermediates/res/merged/release",
 *
 *   // by default the gradle tasks are skipped if none of the JS files or assets change; this means
 *   // that we don't look at files in android/ or ios/ to determine whether the tasks are up to
 *   // date; if you have any other folders that you want to ignore for performance reasons (gradle
 *   // indexes the entire tree), add them here. Alternatively, if you have JS files in android/
 *   // for example, you might want to remove it from here.
 *   inputExcludes: ["android/**", "ios/**"],
 *
 *   // override which node gets called and with what additional arguments
 *   nodeExecutableAndArgs: ["node"],
 *
 *   // supply additional arguments to the packager
 *   extraPackagerArgs: []
 * ]
 */

project.ext.react = [
    entryFile: "index.js"
]

apply from: "../../node_modules/react-native/react.gradle"

/**
 * Set this to true to create two separate APKs instead of one:
 *   - An APK that only works on ARM devices
 *   - An APK that only works on x86 devices
 * The advantage is the size of the APK is reduced by about 4MB.
 * Upload all the APKs to the Play Store and people will download
 * the correct one based on the CPU architecture of their device.
 */
def enableSeparateBuildPerCPUArchitecture = false

/**
 * Run Proguard to shrink the Java bytecode in release builds.
 */
def enableProguardInReleaseBuilds = false

android {
    compileSdkVersion 23
    buildToolsVersion "23.0.1"

    defaultConfig {
        applicationId "com.googlefitness"
        minSdkVersion 16
        targetSdkVersion 22
        versionCode 1
        versionName "1.0"
        ndk {
            abiFilters "armeabi-v7a", "x86"
        }
    }
    splits {
        abi {
            reset()
            enable enableSeparateBuildPerCPUArchitecture
            universalApk false  // If true, also generate a universal APK
            include "armeabi-v7a", "x86"
        }
    }
    buildTypes {
        release {
            minifyEnabled enableProguardInReleaseBuilds
            proguardFiles getDefaultProguardFile("proguard-android.txt"), "proguard-rules.pro"
        }
    }
    // applicationVariants are e.g. debug, release
    applicationVariants.all { variant ->
        variant.outputs.each { output ->
            // For each separate APK per architecture, set a unique version code as described here:
            // http://tools.android.com/tech-docs/new-build-system/user-guide/apk-splits
            def versionCodes = ["armeabi-v7a":1, "x86":2]
            def abi = output.getFilter(OutputFile.ABI)
            if (abi != null) {  // null for the universal-debug, universal-release variants
                output.versionCodeOverride =
                        versionCodes.get(abi) * 1048576 + defaultConfig.versionCode
            }
        }
    }
}

dependencies {
    compile 'com.google.android.gms:play-services-auth:11.0.4'
    compile project(':react-native-google-fit')
    compile fileTree(dir: "libs", include: ["*.jar"])
    compile "com.android.support:appcompat-v7:23.0.1"
    compile "com.facebook.react:react-native:+"  // From node_modules
}

// Run this once to be able to run the application with BUCK
// puts all compile dependencies into folder libs for BUCK to use
task copyDownloadableDepsToLibs(type: Copy) {
    from configurations.compile
    into 'libs'
}

@Pita hello!
I believe you're using too old play-services
try to change your app/build.gradle
compile 'com.google.android.gms:play-services-auth:11.0.4'
to
compile 'com.google.android.gms:play-services-auth:+'

I think the issue was that I used a bit of an older react-native version. Namely 0.54. I updated and then it worked out of the box. Maybe we should add a peerDependency?


Test and fix check contract script in CI

On current main there is at least a trailing quote missing, also GLOBIGNORE is not needed anymore.
https://github.com/CosmWasm/cosmwasm/blob/af3ed179b2a8f7807c7eccb9e23fb110b213c248/.circleci/config.yml#L1219-L1220
Would be good to test the CI job. It can be tested by temporarily commenting out the publishing part and adapt the build_and_upload_devcontracts filters.

@uint I was wondering, shouldn't it be relatively easy to configure clap to accapt multiple wasm files? Just like sha256sum it can process one or multiple. Then we can use the solution we had before and it is more convenient for our users as well as they can just call the tool on ./contracts/*.

@uint I was wondering, shouldn't it be relatively easy to configure clap to accapt multiple wasm files?

Yeah, I agree 100%. Was planning to pick this up today and give that a try. Would be really surprised if clap didn't support that.


feat: nic - Add support for MTU in Ubuntu

Add MTUBytes in [Link] section if mtu is provided for an interface.
Test input:
          network_interfaces:
            - interface: eth_ips_mtu
              ip4: <IP_ADDRESS>/16,<IP_ADDRESS>/16  # Multiple ip possible
              gw4: <IP_ADDRESS>
              mtu: 8000
              network: ice1-1

Correct output:
[Match]
Name=eth_ips_mtu
[Network]
DNS=<IP_ADDRESS>
[Address]
Address=<IP_ADDRESS>/16
Address=<IP_ADDRESS>/16
[Route]
Gateway=<IP_ADDRESS>
[Link]
MTUBytes=8000

OK for me to just merge this PR to dev/2.0? I have other PRs for improving the nic role.

Merging after fixing author.


Multitenancy: Unauthorized error on restart

When restarting the agent, all calls related to an existing wallet fail. This is the log:
aries_cloudagent.admin.server ERROR Handler error with exception: Unauthorized
Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/admin/server.py", line 341, in setup_context
    self.context, token
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/multitenant/manager.py", line 415, in get_profile_for_token
    wallet = await WalletRecord.retrieve_by_id(session, wallet_id)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/messaging/models/base_record.py", line 233, in retrieve_by_id
    cls.RECORD_TYPE, record_id, {"retrieveTags": False}
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/storage/in_memory.py", line 74, in get_record
    raise StorageNotFoundError("Record not found: {}".format(record_id))
aries_cloudagent.storage.error.StorageNotFoundError: Record not found: 80592095-a5f2-436d-b981-8e225331370a

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/admin/server.py", line 162, in ready_middleware
    return await handler(request)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/admin/server.py", line 199, in debug_middleware
    return await handler(request)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aiohttp_apispec/middlewares.py", line 45, in validation_middleware
    return await handler(request)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/admin/server.py", line 290, in check_token
    return await handler(request)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/admin/server.py", line 322, in check_multitenant_authorization
    return await handler(request)
  File "/home/indy/.pyenv/versions/3.6.13/lib/python3.6/site-packages/aries_cloudagent/admin/server.py", line 346, in setup_context
    raise web.HTTPUnauthorized()
aiohttp.web_exceptions.HTTPUnauthorized: Unauthorized

In the current setup I am using bcgovimages/aries-cloudagent:py36-1.16-1_0.7.1 with postgres:13 and the following config:
admin-api-key: ...
admin: [<IP_ADDRESS>, 8031]
label: ...
webhook-url: http://...
genesis-url: http://.../genesis

inbound-transport:
  - [http, <IP_ADDRESS>, 8030]
  - [ws, <IP_ADDRESS>, 8040]
outbound-transport: http
endpoint: http://...:8030
auto-ping-connection: true
multitenant: true
multitenant-admin: true
jwt-secret: ...
wallet-key: someKey...
wallet-storage-type: postgres_storage
wallet-storage-config: '{"url":"postgres-host:5432","wallet_scheme":"MultiWalletSingleTable"}'
wallet-storage-creds: '{"account":"postgres","password":"...","admin_account":"postgres","admin_password":"..."}'
auto-provision: true
seed: someSeed--000000000-000000000-00

All operations work just fine with a running agent: creating wallets, invitations, relationships, credentials, etc.
However, once the agent is restarted, it doesn't recognize existing wallets (although they exist in the database) and all operations on these wallets fail.

Not sure if this has something to do with It, but you're not setting the wallet-type property. If not set ACA-Py will default to the in_memory wallet, which loses all data after a restart. However as you say the wallets exist in the database I'm not sure if this could be it

Not able to reproduce.  I started an aca-py agent in multitenant mode, created a sub-wallet, tested access to the sub-wallet using the swagger page (authenticate using bearer token)
Then I restarted the agent, re-authenticated and was able to access the wallet again.
I used an external postgres database to store wallets (left this running while I restarted the aca-py agent)
@a1s-tech I can share some of my config if you're interested

@TimoGlastra I will check the wallet-type property; indeed, the error seems to come from in_memory.get_record.
Thanks for the suggestion!

@ianco yes, please share. Much appreciated.

See the following two files (renamed to *.txt to satisfy github):
postgres-indy-args-local.yml.txt
faber-local-multitenant.sh.txt

run a local von-network (./manage start <IP_ADDRESS> --logs, your IP will differ)
run a local postgres (the command is in the yml file included above)
(optionally) run a tails server if you are going to be revoking credentials
Run the agent using the above .sh script

Once the agent is running:

Open the swagger page in a browser http://localhost:8021
Create a new sub-wallet - remember the token
authenticate the swagger page using Bearer <your sub-wallet token>
you can now call admin endpoints for your sub-wallet
re-start your aca-py agent ( to kill it and then run the .sh script again)
You will need to re-authenticate with your bearer token again
call the admin endpoints for your subwallet

@ianco Your example worked as expected.
In my setup I added wallet-type: indy and removed seed (this combination generated other errors). Now all is good.
Thank you @TimoGlastra  & @ianco for your assistance.


Support tests in directories other than src/test

The reporter looks for Java sources in the src/test folder only.

The resolution now looks for sources in the base folder and every directory in it for two levels.