About stl Array

Dear sir:
Thank you very much for your "surf2stl-python". Here I just have a question is there anyway to array (copy & paste) that stl module .
#```python
import numpy as np
import surf2stl
x = np.linspace(-6, 6, 30)
y = np.linspace(-6, 6, 30)
X, Y = np.meshgrid(x, y)
Z = np.sin(np.sqrt(X ** 2 + Y ** 2))
surf2stl.write('3d-sinusoidal.stl', X, Y, Z)
   for example array the "3d-sinusoidat.stl" to 5x5 or even more, then save it as a new stl? since the array maybe
some hundred*some hundred times.
  That my plan is to use a  csv file inside with x,y,z coordinate. That means after the script created the stl, then
next to read the csv file and run array, final save it as a new stl. Could you please help me for this issue?
Thank you very much.

Thank you for your proposal :-). You mean, your question is, if there are some ways to write multiple surfaces into one STL file?
Current version does not support for the feature you mentioned. Because matplotlib users seems to draw one surface usually, and writing multiple surfaces  is not required so many times.
But your suggestion seems meaningful and interesting. Though I do not have any plan to add the feature right now, I can offer you another script to do the process you mentioned, like 'open -> write surface -> write surface -> ... -> close'.
Can you check the script in my gist below? This includes an example around the top of the script, too.
https://gist.github.com/asahidari/3a710814cef529904664fff47e72b8cd
I hope this script meet your request and help to solve your problem.

Dear Sir:
Thank you very much for your stl array script.
I will run some tests later and report the results to you after it.
Thanks again.
With best regards.

差出人: asahidari @.>
送信日時: 2021年11月7日 17:23
宛先: asahidari/surf2stl-python @.>
CC: Takashi-rk @.>; Author @.>
件名: Re: [asahidari/surf2stl-python] About stl Array (Issue #1)
Thank you for your proposal :-). You mean, your question is, if there are some ways to write multiple surfaces into one STL file?
Current version does not support for the feature you mentioned. Because matplotlib users seems to draw one surface usually, and writing multiple surfaces is not required so many times.
But your suggestion seems meaningful and interesting. Though I do not have any plan to add the feature right now, I can offer you another script to do the process you mentioned, like 'open -> write surface -> write surface -> ... -> close'.
Can you check the script in my gist below? This includes an example around the top of the script, too.
https://gist.github.com/asahidari/3a710814cef529904664fff47e72b8cd
I hope this script meet your request and help to solve your problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/asahidari/surf2stl-python/issues/1#issuecomment-962649215, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWLN3J75NEXKVNU5DG562FDUK2YXJANCNFSM5HNYQJFQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Dear Sir:
I ran test with your script command;
First: I used the commands from surf2stl.py to create a design, inter window it looks the command is finished
but I can't find where the design is (Photo STL_Test). Then I tried to save the design onto D:_python
but an error is happen (Photo STL_Test1).
Second: I try command from surf2stlwriter.py, the commands finished with out error and I got a design (2x1)
which I need (photo STL_Test2 & STL_Test3).
Final what I need is I want to Array same design which created by the script, to a large design (1000x1000
or more large) and save they together. And my question is how can I do this with some x,y,z coordinates onto
a csv file?
With best regards

差出人: Takashi Renketsu @.>
送信日r: 2021年11月7日 23:47
宛先: asahidari/surf2stl-python @.>
件名: Re: [asahidari/surf2stl-python] About stl Array (Issue #1)
Dear Sir:
Thank you very much for your stl array script.
I will run some tests later and report the results to you after it.
Thanks again.
With best regards.

差出人: asahidari @.>
送信日r: 2021年11月7日 17:23
宛先: asahidari/surf2stl-python @.>
CC: Takashi-rk @.>; Author @.>
件名: Re: [asahidari/surf2stl-python] About stl Array (Issue #1)
Thank you for your proposal :-). You mean, your question is, if there are some ways to write multiple surfaces into one STL file?
Current version does not support for the feature you mentioned. Because matplotlib users seems to draw one surface usually, and writing multiple surfaces is not required so many times.
But your suggestion seems meaningful and interesting. Though I do not have any plan to add the feature right now, I can offer you another script to do the process you mentioned, like 'open -> write surface -> write surface -> ... -> close'.
Can you check the script in my gist below? This includes an example around the top of the script, too.
https://gist.github.com/asahidari/3a710814cef529904664fff47e72b8cd
I hope this script meet your request and help to solve your problem.
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/asahidari/surf2stl-python/issues/1#issuecomment-962649215, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWLN3J75NEXKVNU5DG562FDUK2YXJANCNFSM5HNYQJFQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
import numpy as np
import matplotlib.tri as mtri
from scipy.spatial import Delaunay
from Surf2StlWriter import *
import matplotlib.pyplot as plt
from mpl_toolkits.mplot3d import Axes3D
Create 1st x/y/z datas
x = np.linspace(-6, 6, 30)
y = np.linspace(-6, 6, 30)
X1, Y1 = np.meshgrid(x, y)
Z1 = np.sin(np.sqrt(X1 ** 2 + Y1 ** 2))
Show Created Design
ax = plt.axes(projection='3d')
ax.plot_surface(X1 ,Y1, Z1, rstride=1, cstride=1, cmap='viridis', edgecolor='none')
ax.set_title('surface')
Write binary STL file
s2sWriter_binary = Surf2StlWriter()
with s2sWriter_binary("D:\Python\writer_test_binary_II.stl", mode='binary'):
s2sWriter_binary.write_surf(X1, Y1, Z1)
s2sWriter_binary.write_surf(X1+12, Y1, Z1)     # Plot same design with shifted X position
H = (X1, Y1, Z1)
s2sWriter_binary = Surf2StlWriter()
with s2sWriter_binary("D:\Python\writer_test_binary_H.stl", mode='binary'):
s2sWriter_binary.write_surf(H)

Thank you for testing my script.
You say, surf2stl.py (original script in this repository) fails your test, but Surf2StlWriter.py (in my Gist) succeeded. So, Surf2StlWriter.py meet your request, it's good.
By the way, I could not see the photos you mentioned, you didn't attach them?

Final what I need is I want to Array same design which created by the script, to a large design (1000x1000
or more large) and save they together.

I think you cannot do such many objects (1000x1000) into one STL file. According to the STL file format (especially in 'binary' mode), the object id is only 2 byte (0xFF), which means you can only store 0xFF (255) items in one STL file. In 'ascii' mode, it might succeed to write over 255 objects, but loading time might be very long.

And my question is how can I do this with some x,y,z coordinates onto a csv file?

It depends on how you want to store x,y,z coordinates onto a csv file and how you want to read 3d datas from the csv file. You can write csv file in various ways like "X1,Y1,Z1,X2,Y2,Z2,...Xn,Yn,Zn" or "X1,X2,X3,...,Xn (line 1), Y1,Y2,Y3,...,Yn (line 2), Z1,Z2,Z3,...,Zn (line 3) . The way to store 2D array might be different from 1D array, but you can use your own idea to store.
The script-functions only checks the list dimensions or shapes of the arguments to receive, but does not depends on the csv file format.
csv format example (for 2D array):
5,10,            <----- 5 row, 10 col
x1, x2, x3, ..., x50,          <--- each line has 5x10 items, you can reshape 50 to 5x10 numpy array using 'numpy.reshape'
y1, y2, y3, ..., y50,
z1, z2, z3, ..., z50,
4,8,            <----- 4 row, 8 col
x1, x2, x3, ..., x32,
y1, y2, y3, ..., y32,
z1, z2, z3, ..., z32,
...

Dear Sir:
Thank very much for your reply and sorry I forget to send the photos.
I attached the photos to you again here.
And I well try some test late what you advised.
With best regards

差出人: asahidari @.>
送信日時: 2021年11月8日 12:26
宛先: asahidari/surf2stl-python @.>
CC: Takashi-rk @.>; Author @.>
件名: Re: [asahidari/surf2stl-python] About stl Array (Issue #1)
Thank you for testing my script.
You say, surf2stl.py (original script in this repository) fails your test, but Surf2StlWriter.py (in my Gist) succeeded. So, Surf2StlWriter.py meet your request, it's good.
By the way, I could not see the photos you mentioned, you didn't attach them?
Final what I need is I want to Array same design which created by the script, to a large design (1000x1000
or more large) and save they together.
I think you cannot do such many objects (1000x1000) into one STL file. According to the STL file format (especially in 'binary' mode), the object id is only 2 byte (0xFF), which means you can only store 0xFF (255) items in one STL file. In 'ascii' mode, it might succeed to write over 255 objects, but loading time might be very long.
And my question is how can I do this with some x,y,z coordinates onto a csv file?
It depends on how you want to store x,y,z coordinates onto a csv file and how you want to read 3d datas from the csv file. You can write csv file in various ways like "X1,Y1,Z1,X2,Y2,Z2,...Xn,Yn,Zn" or "X1,X2,X3,...,Xn (line 1), Y1,Y2,Y3,...,Yn (line 2), Z1,Z2,Z3,...,Zn (line 3) . The way to store 2D array might be different from 1D array, but you can use your own idea to store.
The script-functions only checks the list dimensions or shapes of the arguments to receive, but does not depends on the csv file format.
csv format example (for 2D array):
5,10,            <----- 5 row, 10 col
x1, x2, x3, ..., x50,          <--- each line has 5x10 items, you can reshape 50 to 5x10 numpy array using 'numpy.reshape'
y1, y2, y3, ..., y50,
z1, z2, z3, ..., z50,
4,8,            <----- 4 row, 8 col
x1, x2, x3, ..., x32,
y1, y2, y3, ..., y32,
z1, z2, z3, ..., z32,
...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/asahidari/surf2stl-python/issues/1#issuecomment-963102120, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWLN3JZHOXYRK4S7UMABBP3UK66YPANCNFSM5HNYQJFQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Incorrect colorization of view

Colors generated for painting the cavas by FixedPointColorizer are not same as described by the palette name. For example on selecting palette  YlGn the view generated doesn't have either Yellow or Green colored gradient.
Problem could in FixedPointColorizer.java

I'm working on this issue.


Some links in 'ingress' repo GCE release notes are 404

With each GCE release in this 'ingress' repo, including 'GCE: 0.9.1', the release notes have links to the old 'contrib' repo that are (now) 404:
https://github.com/kubernetes/contrib/blob/master/ingress/controllers/gce/BETA_LIMITATIONS.md
https://github.com/kubernetes/contrib/blob/master/ingress/controllers/gce/README.md#troubleshooting
https://github.com/kubernetes/contrib/blob/master/ingress/controllers/gce/README.md#wishlist

/assign

these link in gce release-note. i don't have permission update it. but i find some other broken link by search https://github.com/kubernetes/contrib in the repository. so reference it to this issue.

This issue was moved to kubernetes/ingress-gce#30


mbedtls: provide user mode access

The mbedtls library has some globals which results in faults
when user mode tries to access them.
Instantiate a memory partition for mbedtls's globals.
Add the mbedtls heap, and the function pointers used by
mbedtls's platform abstraction.
Signed-off-by: Andrew Boie<EMAIL_ADDRESS>

Note to reviewers: this patch required modifying the upstream mbedtls code, in order to correctly place the globals for the platform abstraction. This is necessary since the mbedtls library is built from source by the kernel.
An alternative approach could be to build mbedtls as a .a library archive, and then add a directive like -l libmbedtls.a k_mbedtls_partition to the invocation of gen_app_partitions.py. This will have the same effect without needed any K_APP_DMEM() tags in the mbedtls code. I'm not sure if this is possible in our build system currently.

Also note: x86 will be failing CI until #13918 is fixed.

If I had my druthers, I'd prefer the .a path.  Any changes we make to external libraries makes it more costly to us to maintain.

@agross-linaro agreed, and I figured it out, see updated patch

current solution is better IMHO, but the other could be change, those globals are used when the platform does not define those functions, we could add them in custom MBEDTLS_CONFIG_FILE (as we already do for other things). The problem is that we would need to do it for all available configurations.

I'm not sure I understand your review.
Let's just focus on the current implementation, I see no downsides to it.

Codecov Report

Merging #13921 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13921      +/-   ##
==========================================
+ Coverage   52.25%   52.27%   +0.02%
==========================================
  Files         307      307
  Lines       45429    45434       +5
  Branches    10506    10511       +5
==========================================
+ Hits        23739    23752      +13
+ Misses      16899    16891       -8
  Partials     4791     4791

Impacted Files
Coverage Δ

drivers/clock_control/nrf_power_clock.c
50% <0%> (-2.11%)
:arrow_down:

boards/posix/nrf52_bsim/argparse.c
46.98% <0%> (+15.66%)
:arrow_up:

Continue to review full report at Codecov.

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

I'm seeing an unexpected failure in tests/subsys/jwt:
***** Booting Zephyr OS v1.14.0-rc1-624-ge68e0ac0f8d1 *****
Running test suite lib_jwt_test
===================================================================
starting test - test_jwt

    Assertion failed at /home/apboie/projects/zephyr3/tests/subsys/jwt/src/main.c:52: test_jwt: (res not equal to 0)
Signing payload
FAIL - test_jwt
===================================================================
Test suite lib_jwt_test failed.

Trying to discover what is going on.

I can't seem to reproduce the two other CI failures:

recheck

Re-adding link to non-last and "unstable" Shippable run which failed with zephyr/priv_stacks_prebuilt.elf section bss will not fit in region SRAM 2/5 times:
https://app.shippable.com/github/zephyrproject-rtos/zephyr/runs/36138
(At first I thought it was unintended but no: hiding and/or deleting past information seems to be one of github's design choices. Not nice.)


Routing mesh doesn't seem to work with 1.12rc3

Output of docker version:
Client:
 Version:      1.12.0-rc3
 API version:  1.24
 Go version:   go1.6.2
 Git commit:   91e29e8
 Built:        Sat Jul  2 00:09:24 2016
 OS/Arch:      darwin/amd64
 Experimental: true

Server:
 Version:      1.12.0-rc3
 API version:  1.24
 Go version:   go1.6.2
 Git commit:   91e29e8
 Built:        Sat Jul  2 00:46:52 2016
 OS/Arch:      linux/amd64
 Experimental: true

Output of docker info:
Containers: 10
 Running: 4
 Paused: 0
 Stopped: 6
Images: 1
Server Version: 1.12.0-rc3
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 27
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host overlay bridge null
Swarm: active
 NodeID: 4hud54lxukcz1m6s2z7gs8d9e
 IsManager: Yes
 Managers: 1
 Nodes: 3
 CACertHash: sha256:c292fab2b6bd64c62b04fee288effc77d9d84b1af087e7130930399ab292276e
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.2.0-18-generic
Operating System: Ubuntu 15.10
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 991.1 MiB
Name: manager
ID: ICKG:RWUD:ZSSX:EQ25:3YB6:KND3:GQO7:R56J:UBF7:D6WF:HE6P:YF7W
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Labels:
 provider=amazonec2
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
The environment was created with docker-machine on AWS. After creating the nodes, I manually bumped to the latest docker version (1.12rc3) and executed docker swarm init and docker swarm join
docker node ls
ID                           HOSTNAME  MEMBERSHIP  STATUS  AVAILABILITY  MANAGER STATUS
4hud54lxukcz1m6s2z7gs8d9e *  manager   Accepted    Ready   Active        Leader
9nuda9z5tjgh9ljwd9vm36oym    node2     Accepted    Ready   Active
a1lfv7bhnyo6or4ty9wubo9uf    node1     Accepted    Ready   Active

Steps to reproduce the issue:

eval (docker-machine env manager)
docker service create --name test --p 8080:8000 jwilder/whoami (This starts the container on node1
curl localhost:8080 works just fine on node1, but it doesn't work on any other node, unless I scale up and have a container running there as well.

Describe the results you received:
I get connection refused on any node, which doesn't run my container. Also, nothing is listening on the right ports (checked with sudo netstat -ntulp)
Describe the results you expected:
Based on what I think the "routing mesh" feature of 1.12 should do, I was expecting that I can access the container even if I try it on a node which doesn't run it at the moment. Maybe I'm missing something, but that's what the keynote demo of DockerCon 2016 showed.

@ghoranyi you very likely need to set --listen-addr when initializing the swarm (and joining) to make sure the correct address to listen on is chosen. By default it will just try to guess.

I didn't manage to solve it with rc3, but rc4 works like a charm.


plugin(google_sign_in): use Object type for GoogleSignInAccount operator

When using mockito for GoogleSignInAccount, the static analyzer starts complaint:
'Mock.==' ('bool Function(Object)') isn't a valid concrete implementation of 'GoogleSignInAccount.==' ('bool Function(dynamic)').

Which means it should be Object in the operator =='s argument.

I think using firebase_auth_mocks can fix this issue.

Hi @evan361425, this is a duplicate of https://github.com/flutter/flutter/issues/101895, which has just been fixed.
Closing from here as duplicate thank you.


Readme of of.stats

We should re think this.

@beraldoleal, I need your review.

Moving to "Done" due to the lack of activity.


Implement serialization of active record object into array representation

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Do you need these all attributes, oldAttributes, related and relationsDependencies properties?

@xepozz is that general question to @terabytesoftw et al or is specific to this PR?

In general. When I see toArray I expect to get an array of entity's properties, without any internal things.

I'd expect the same as @xepozz.

I see what you mean. But in this case this is more of _toString() kind of a thing. All configuration of what should be exported is done by fields() which by default serializes all fields. One who want this customization should do them there.
That is how at least I do in Yii2.
Am not sure if that is convincing or not!

Current implementation looks good to me. Could you also add tests especially one with closure field?

Sure. I will find time and add them.

@terabytesoftw yes sure. There are tasks holding me back at work, and I will highly appreciate that help!

Cheers!


WOS 2.0 changes

Add your criteria here:

[ ] New disco files
[ ] Add "checkout" to conversation flow
[ ] Use slots or new assistant features in conversation flow
[ ] better UI?
[ ] more robust conversation (don't force remove, don't show empty cart, etc...)
[ ] multi-user
[ ] can we do more natural lang instead of numbers like a call tree?
[ ] might consider serverless action

Closing this and tracking in internal repository.


python311Packages.restructuredtext-lint: rename from restructured_lint

Description of changes
part of #245383

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 265269 at b46372d7 run on x86_64-linux 1

12 packages built successfully:

calendar-cli
gtg
home-assistant-component-tests.caldav
ical2orgpy
python310Packages.caldav
python310Packages.doc8
python310Packages.recurring-ical-events
python310Packages.restructuredtext-lint
python311Packages.caldav
python311Packages.doc8
python311Packages.recurring-ical-events
python311Packages.restructuredtext-lint


Request: Writing QOI images

I know that QOI is not already a standard, and an IANA type for that new image format is on the way. But it has a great potential especially for small devices.
So, as I have seen, that the LovyanGFX already supports createQoi() in the Qoi branch, I want kindly ask, if it is possible to merge that also to the M5GFX repo. For me M5Unified + M5GFX has alot of pro's on the M5Core2 (and other) M5 plattforms.
At this point, a big thanks to the high-quality contribution of M5GFX and M5Unified.

@prjh
Yes, I do have a plan.
I'll be maintaining the M5GFX to keep up with the LovyanGFX release.
If LovyanGFX adopts it, it will be adopted by M5GFX within a few days.
Please wait a bit, as we have a lot of work piling up that has a higher priority than the QOI save feature…

@lovyan03
I understand that, no problem. I have already noted with pleasure that there is real life in the new M5GFX project. Good things take time. By the way, creating and drawing from memory would also be a fine feature, but no hurry ;-)
... and big THX


uWu lend QA

Subgraph deployed: https://okgraph.xyz/?q=dmelotik%2Fuwu-lend-ethereum
QA required @ishraq8

QA: https://docs.google.com/spreadsheets/d/10iVvgJZCkr9GbOuk_IBiSGlxdAEGEwBz/edit#gid=2041140346
Comments: The subgraph looks great, great work. One note is that USDT TVL on our subgraph is higher than on the official site. This is because protocol loops USDT which our subgraph is picking up but their front end is not.


Sort events by categories

Summary
Documentation change. Adds categories to events references.
Resolves #80
Image:

Checklist

[ ] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.

[x] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[x] This PR is not a code change (e.g. documentation, README, ...)

Detailed information on how they are sorted
Channels:
- on_guild_channel_create
- on_guild_channel_delete
- on_guild_channel_update
- on_group_join
- on_group_remove
- on_private_channel_update
- on_guild_channel_pins_update
- on_private_channel_pins_update
- on_typing
- on_raw_typing
Guilds:
- on_guild_available
- on_guild_unavailable
- on_guild_join
- on_guild_remove
- on_guild_update
- on_guild_emojis_update
- on_guild_stickers_update
- on_invite_create
- on_invite_delete
Integrations:
- on_integration_create
- on_integration_update
- on_guild_integrations_update
- on_webhooks_update
- on_raw_integration_delete
Interactions:
- on_interaction
Members:
- on_member_join
- on_member_remove
- on_member_update
- on_user_update
- on_member_ban
- on_member_unban
- on_presence_update
Messages:
- on_message
- on_message_edit
- on_message_delete
- on_bulk_message_delete
- on_raw_message_edit
- on_raw_message_delete
- on_raw_bulk_message_delete
Meta:
- on_error
Networking:
- on_connect
- on_disconnect
- on_ready
- on_resumed
- on_shard_connect
- on_shard_disconnect
- on_shard_ready
- on_shard_resumed
- on_socket_event_type
- on_socket_raw_receive
- on_socket_raw_send
Reactions:
- on_reaction_add
- on_reaction_remove
- on_reaction_clear
- on_reaction_clear_emoji
- on_raw_reaction_add
- on_raw_reaction_remove
- on_raw_reaction_clear
- on_raw_reaction_clear_emoji
Roles:
- on_guild_role_create
- on_guild_role_delete
- on_guild_role_update
Stages:
- on_stage_instance_create
- on_stage_instance_delete
- on_stage_instance_update
Threads:
- on_thread_join
- on_thread_update
- on_thread_remove
- on_thread_delete
- on_thread_member_join
- on_thread_member_remove
Voice:
- on_voice_state_update


Table missing?

The first note at the top of the page states that there is a table describing the publishing options, storefront selection etc. This table is supposed to be located above the note :

"The table above describes how your solution or service maps to the storefront, publishing option, and offer type used to present your offer in marketplace."

However There is no such table above in this page.

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

ID: 5ad22423-b979-4382-1412-e58e2a5b083f
Version Independent ID: 2b65f4a9-a440-e627-2c13-14c952da3e94
Content: Determine Your Publishing Option in Azure Marketplace
Content Source: articles/marketplace/determine-your-listing-type.md
Service: marketplace
GitHub Login: @jm-aditi-ms
Microsoft Alias: ellacroi

@frederimandin Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

@frederimandin sometimes the table doesn't load properly at the top of the page, please try hitting refresh?

I have tried refreshing, and tried 4 different browsers :)
Maybe I misunderstand what the page is supposed to look like.
Here is what I get :

I can confirm this behavior.  Dropping a bet deeper into the problem.  The table is embedded using:

That link returns a 404.  Note that it is a link into what appears to be a personal fork(?) by @ellacroi, rather than the base azure-docs repository.  I poked around trying to find a "storefronts_options_table.png" to link to, but have not found one.

#please-close  this was resolved in a separate PR


Regression on basic http authentication

What is your Scenario?
Using test.httpAuth doesn't work anymore.
What is the Current behavior?
No header is added to the requests triggered by the actions of the test. As the feature relies on that header, it doesn't work and I get a 401 response.
What is the Expected behavior?
A http header is added to requests triggered during the test. The success page is displayed.
What is your public website URL? (or attach your complete example)
A good place to check basic http authentication:
https://authenticationtest.com/
What is your TestCafe test code?
import { Selector } from "testcafe"; // first import testcafe selectors

fixture`Testing stuff`
  .page`https://authenticationtest.com/`;

test("Basic http auth", async (t) => {
  const linkSelector = Selector(
    "div.row:nth-child(3) > div:nth-child(1) > div:nth-child(2) > div:nth-child(2) > a:nth-child(3)"
  );
  const titleSelector = Selector("div.container h1");

  await t
    .click(linkSelector)
    .expect(titleSelector.innerText)
    .eql("Login Success");
}).httpAuth({ username: "user", password: "pass" });

Your complete configuration file
No response
Your complete test report
No response
Screenshots
No response
Steps to Reproduce

Use httpAuth method with valid credentials
Start the test

TestCafe version
3 and above
Node.js version
No response
Command-line arguments
testcafe chrome ./tests/*
Browser name(s) and version(s)
Chrome 114
Platform(s) and version(s)
Ventura 13
Other
The exact same test works just fine with TC version 2.5.0 so this is definitely a regression.

Please use the --disable-native-automation flag until the issue is fixed.


feat: travisci - updated job to use bash file

In the .travis.yaml we are currently executing the libgit install in the yaml. We should be using the bash file to libgit in the ./tools/

remove libgit install in .travis.yaml
execute the  bash file under tools tools

this is taken care of in #76


Updated README.md with Helm instructions and small cleanup for admiss…

Purpose

This PR adds some documentation changes to instruct the user to install Helm and provides instructions for initializing Tiller.
Does this introduce a breaking change?

[ ] Yes
[ x ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ x ] Documentation content changes
[ ] Other... Please describe:

LGTM! Let's merge


[lemon-parser-generator] New port

Describe the pull request

What does your PR fix?
Complements #26755. SQLCipher requires lemon built for the host for code generation.

Which triplets are supported/not supported? Have you updated the CI baseline?
all, yes

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

As this is a tool port, I'm marking it for review on Thursday's meeting.


Update the JobSet version to 0.5.1 for installation

To make it likely that users install the version with the latest fixes.

/cc @alculquicondor @danielvegamyhre

👍

/lgtm
/approve
Thanks!


[RFC] Android RPC service performance issues

Using a service process for RPC in Android is causing significant performance regressions (approximately by a factor of 3 on my test device).
@merrymercy identified the watchdog process commit as the cause in #1426
The service/watchdog process was added in #1387
I've confirmed that on my device performance returns to its previous level if I roll back MainActivity.java to before the service/watchdog was introduced.
It is unclear precisely why this is occurring, but I've done some debugging on this and found the following:

The issue persists if the RPCProcessor is launched in a service but the watchdog is removed. This demonstrates the performance issue is not due to the interaction with the watchdog, but simply due to running in the service process.
Adding a timer in C++ around GraphRuntime::Run confirms that almost all of the processing time is taken by this function, and not by java code.
I have tried setting the service as a foreground service (how to do this correctly is somewhat poorly documented), since by default service processes in Android are given lower priority than app processes[1]. This did not make any difference :(

Any ideas as to why this might be occurring would be appreciated
[1] https://medium.com/google-developers/who-lives-and-who-dies-process-priorities-on-android-cb151f39044f
@tqchen
@eqy

That sounds great :)
Bringing the RPCProcessor back into the activity will definitely resolve the issue, I've just been digging into this because it's somehow unsatisfying without knowing why it was broken in the first place!

https://github.com/dmlc/tvm/pull/1457

close this for now as @merrymercy confirmed that the performance regression is fixed by #1457 @alex-weaver it would be great if you can also verify independently


Option to install additional plugin to *env

Right now anyenv defines list of plugins installed with *env. I'd like to have an option to install additional plugins for existing *env.

Hi,
I'm considering about splitting anyenv itself and env/plugin list. I'd like to hear your opinions against this idea at #46
Best,


Getting hostname

Q
A

Issue Type
Bug

Deployer Version
master (6.0 but --version always return master)

Local Machine OS
macOS

Remote Machine OS
Debian 8.10

Description
In my old recipe, I had tasks which execute commands locally, to the remote host
task('fetch:archives', function () {
    runLocally("scp -r {{server.hostname}}:{{current_path}}/dump/", 0);
})->desc('Fetch archives from server');

But it seems I can't get my hostname anymore:
{{server}} of course does not exists anymore, but {{hostname}} or get('hostname') returns host instead of host()->hostname().
How can I achieve this and by the way, get the while current host configuration ?

Try this:
set('real_hostname', function () {
    Context::get()->getHost()->getHostname();
});

Actually I don't like this too. I think I'll refactor this in v7.

Thanks, but I'm afraid, I can't get the real hostname, but the host actually defined in the deploy recipe.
Task\Context::get()->getHost()->getHostname()

Give me prod instead of my.site.com
I could set the host using its real hostname
host('my.site.com')
    ->stage('production')
    ->set('deploy_path', $path . '/preprod');

instead of
host('prod')
    ->hostname('my.site.com');
    ->stage('production')
    ->set('deploy_path', $path . '/preprod');

but I couldn't be able to use multiple stages for one host anymore as all hosts definitions would get the exact same name.

I see, there is problem with stages - will be solved in next release. Use getHostname() for first example.

The fix is already open as a PR I guess?
https://github.com/deployphp/deployer/pull/1434

@staabm yes, but I don't like solution. I think to refactor this in v7 into hostname -> alias, real_hostname -> hostname.

On Deployer 6.2.0, I had to modify @antonmedv example like this to make it work:
set('real_hostname', function () {
  return Task\Context::get()->getHost()->getHostname();
});

I couldn't get this approach to work in version 6.4.3. It always returns the value set for host rather than the value set for hostname.
Anything I might be missing?

Fixed in #1904
Not it will be always correct hostname {{hostname}}


weight problem

When load_model(), I have counted this problem?@jutanke Can you help me? Thank you!
ValueError                                Traceback (most recent call last)
 in ()
11 X, Y = m1501.get_train()
12
---> 13 model = reid.ReId()
14
15 im1 = X[0]
~/gen/Re-ID/person_reid-master/reid/reid.py in init(self, root, url, name)
28             dl.download(url, filepath)
29
---> 30         self.model = load_model(filepath)
31
32     def predict(self, A, B):
~/anaconda3/lib/python3.6/site-packages/keras/engine/saving.py in load_model(filepath, custom_objects, compile)
417     f = h5dict(filepath, 'r')
418     try:
--> 419         model = _deserialize_model(f, custom_objects, compile)
420     finally:
421         if opened_new_file:
~/anaconda3/lib/python3.6/site-packages/keras/engine/saving.py in _deserialize_model(f, custom_objects, compile)
272                                                        original_keras_version,
273                                                        original_backend,
--> 274                                                        reshape=False)
275         if len(weight_values) != len(symbolic_weights):
276             raise ValueError('Layer #' + str(k) +
~/anaconda3/lib/python3.6/site-packages/keras/engine/saving.py in preprocess_weights_for_loading(layer, weights, original_keras_version, original_backend, reshape)
680         weights = convert_nested_time_distributed(weights)
681     elif layer.class.name in ['Model', 'Sequential']:
--> 682         weights = convert_nested_model(weights)
683
684     if original_keras_version == '1':
~/anaconda3/lib/python3.6/site-packages/keras/engine/saving.py in convert_nested_model(weights)
668                     weights=weights[:num_weights],
669                     original_keras_version=original_keras_version,
--> 670                     original_backend=original_backend))
671                 weights = weights[num_weights:]
672         return new_weights
~/anaconda3/lib/python3.6/site-packages/keras/engine/saving.py in preprocess_weights_for_loading(layer, weights, original_keras_version, original_backend, reshape)
680         weights = convert_nested_time_distributed(weights)
681     elif layer.class.name in ['Model', 'Sequential']:
--> 682         weights = convert_nested_model(weights)
683
684     if original_keras_version == '1':
~/anaconda3/lib/python3.6/site-packages/keras/engine/saving.py in convert_nested_model(weights)
656                     weights=weights[:num_weights],
657                     original_keras_version=original_keras_version,
--> 658                     original_backend=original_backend))
659                 weights = weights[num_weights:]
660
~/anaconda3/lib/python3.6/site-packages/keras/engine/saving.py in preprocess_weights_for_loading(layer, weights, original_keras_version, original_backend, reshape)
799             weights[0] = np.reshape(weights[0], layer_weights_shape)
800         elif layer_weights_shape != weights[0].shape:
--> 801             weights[0] = np.transpose(weights[0], (3, 2, 0, 1))
802             if layer.class.name == 'ConvLSTM2D':
803                 weights[1] = np.transpose(weights[1], (3, 2, 0, 1))
~/anaconda3/lib/python3.6/site-packages/numpy/core/fromnumeric.py in transpose(a, axes)
548
549     """
--> 550     return _wrapfunc(a, 'transpose', axes)
551
552
~/anaconda3/lib/python3.6/site-packages/numpy/core/fromnumeric.py in _wrapfunc(obj, method, *args, **kwds)
55 def _wrapfunc(obj, method, *args, **kwds):
56     try:
---> 57         return getattr(obj, method)(*args, **kwds)
58
59     # An AttributeError occurs if the object does not have
ValueError: axes don't match array

model = load_model('/home/dell/gen/weight/model_heavy_89acc.h5')
this code error

What are your versions of tensorflow / keras?

I will look into it tonight, thanks for reporting

I will look into it tonight, thanks for reporting

Thank you very much!

The model was trained with Keras 2.1.3 which, for some reason, seems to be incompatible with Keras 2.2.+. I will look into this further

So, after investigating it seems like there were breaking changes in the DenseNet implementation of Keras. I now spoon-feed the weights to the model if it is not in a supported version which takes painfully long. However, the prediction does not properly work anymore.
Unfortunately, i will be quite busy for the foreseeable future so I would suggest to just train your own model using the train function. Hopefully, I can just train a model myself in the next couple of weeks.
So far

I have transfer the version to 2.1.3 and it works.
How to train can get the weight you offered?
Thank you.

You can try the current new version. It should work with Keras 2.1.x but there seems to be a bug in Keras 2.2.x that prevents models from checkpoints to be loaded.

There seems to be a problem with Keras 2.2.x and loading weights from checkpoints. So if you use Keras 2.1.x you should be fine. Furthermore, I retrained the model so now it should work quite a bit better. Simply re-install the library via pip:
pip install git+https://github.com/jutanke/person_reid.git

Then you can use it as before:
import cv2
from reid import reid

im1 = cv2.cvtColor(cv2.imread('/path/to/file1.png'), cv2.COLOR_BGR2RGB)
im2 = cv2.cvtColor(cv2.imread('/path/to/file2.png'), cv2.COLOR_BGR2RGB)

# im1 / im2 can either be images or lists of images. In case of lists
# prediction is done item-wise

model = reid.ReId()
score = model.predict(im1, im2)

Good luck


Tweak the timestamp in mongoose model template

No test case added, hope it's fine for a template file -_-

This looks like a good change.  I'm not able to see why the tests are failing.  Do they pass for you locally?

I tried it before, always stuck somewhere, it's not failed, it's just timeout.
If I remember it right, the tests are not passing even before I added the code.

@Mr-Binary Try pulling down latest master. Looks like the failing tests were because of the latest version of Mocha which got installed automatically which was fixed in #283

@daffl Thanks, I updated my local repo, and open mongodb, now 16 tests pass and 2 pending.
........
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>✨  Done in 6.71s.
      ✓ sequelize (13859ms)
      - rethinkdb
    feathers:authentication
      - runs and initializes working local strategy

  16 passing (4m)
  2 pending

albert:generator-feathers albertgao$

Is this good?
And, there is a package-lock.js generated in my local folder, do I need to commit that as well?
Besides, we should let people they need to start their local mongod at the default port to let the tests going, otherwise, they need to wait until the 1st round failed then know why. what do you think? :)

Looks great, thank you again!

No worries. Wish I could contribute more to this project, I deadly love this framework. 🍡


Add logic to decide when to do key/value pre-aggregation on Map side.

We shouldn't PR into master - use https://github.com/palantir/spark/pull/13 as an example of what to do


catch exception and debug script

add exception catching (gpt-4 model is not available for all users)
add vs code debug script, useful to use this tool

Thanks for the contribution! I merged this but made a few changes:

Set .vscode to ignore since we only a minority of users are devs and/or use VSCode. Instead, I created a new doc for advanced users showing the file.
More precise exception handling


A.08.IR90.layer.style

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
Test purpose states that at least one style element exists per layer. However, the test method does not include a check on the existence of at least one layer style.
Proposed Change
Update test method to include check on whether at least one layer style exists.

No change: covered by WMTS schema validation where layer style has [1..*] as multiplcity.


Remove Airbyte from Saagie’s Technology Community Repository

Remove Airbyte from Saagie’s Technology Community Repository

Trello card: [#200] Remove Airbyte from Saagie’s Technology Community Repository


fix(Deprecation): fixing the lualine get colors method for Deprecation

Looks like my personal changes also got committed :')


Error while installing

I got erro while installing una onto my server

Hello
What language module were you trying to install when you got this error?
Please provide me login details for the server if you want me to fix this
urgently.
Otherwise, to solve the problem I will need a bit more info and a full
error report.
On Mon, Apr 26, 2021 at 1:02 AM Hardik Viradiya @.***>
wrote:

[image: image]
https://user-images.githubusercontent.com/43150887/116004001-69408d80-a61e-11eb-8d3d-38fad23d91ce.png
I got erro while installing una onto my server
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/unaio/una/issues/3333, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACOAF7JLAQSB2UCTBFVPNRTTKRKLFANCNFSM43RQOLYA
.

Please post question in forum - https://una.io/page/discussions-home  ?


Empty fields

Ok, all of the JSON now validates, except I'm getting an error when I leave out the unit number.
Error in /delivery/unit:
String is too short (0 chars), minimum 1

Error in /assistant/unit:
String is too short (0 chars), minimum 1

In general, if a field is blank, should I also remove that key from the JSON object, or should I just leave the value blank?

Yeah, that error is resulting because, in JSON, schema requirements are applied if a field exists. So the unit must be at least 1 character long if it exists. Fields that do not have a value should not be defined, which is to say that, yes, keys with blank values should be dropped.

Ok, keys with blank values are now dropped, and everything seems to be validating.


how to replace 3ddfa_v2 with 3ddfa ?

3ddfa_v2 is much accuracy than 3ddfa on face fitting

did this improve your results? if you managed to do it ofc

It should be easy, but I may not try it for myself. You can raise a pull request if you have managed to do so.


Handling Early Resets in Procgen Envs

I've been trying to set up a multiworker Rainbow DQN baseline for procgen, similar to what's described in Leveraging Procedural Generation for Benchmarking Reinforcement Learning. This is roughly how I'm handling the setup (based on example 7):
def make_env(game, num_levels, distribution_mode):

    class RlpytProcgenWrapper(gym.Wrapper):
       """
        Handle issues with procgen seeding and image axis order
       """

        def step(self, *args):
            o, r, d, i = self.env.step(*args)
            return np.transpose(o, (2, 0, 1)), r, d, i

        def reset(self):
            return np.transpose(self.env.reset(), (2, 0, 1))

        def seed(self, seed):
            return

     env = gym.make(f"procgen:procgen-{game}-v0", num_levels=num_levels, distribution_mode=distribution_mode)
     env = RlpytProcgenWrapper(env)
     return env

args = parse_args()

affinity = make_affinity(
        run_slot=args.run_slot,
        n_gpu=args.n_gpu,
        n_cpu_core=args.n_cpu_core,
        gpu_per_run=args.gpu_per_run,
    )

sampler = GpuSampler(
        EnvCls=GymEnvWrapper,
        env_kwargs=dict(env=make_env(args.game, args.num_levels, args.distribution_mode)),
        TrajInfoCls=TrajInfo,
        batch_T=args.batch_T,
        batch_B=args.batch_B,
        CollectorCls=GpuResetCollector,
        max_decorrelation_steps=args.max_decorrelation_steps,
        eval_n_envs=10,
        eval_env_kwargs=dict(env=make_env(args.game, args.eval_num_levels, args.eval_distribution_mode)),
        eval_max_steps=int(10e5),
        eval_max_trajectories=20,
    )

algo = CategoricalDQN(...)

num_actions = make_env(args.game, args.num_levels, args.distribution_mode).action_space.n

agent = CatDqnAgent(n_atoms=args.n_atoms, eps_final=args.eps_final,
                        ModelCls=AtariCatDqnModel, model_kwargs={'image_shape':(3, 64, 64), 'output_size':num_actions, 'dueling':True})

runner = SyncRlEval(
        algo=algo,
        agent=agent,
        sampler=sampler,
        n_steps=args.n_steps,
        log_interval_steps=1e4,
        affinity=affinity,
    )

config = vars(args)
name = f"rainbow_{args.game}_{args.distribution_mode}"
log_dir = f"{args.game}"
with logger_context(log_dir, args.run_ID, name, config, snapshot_mode="last", override_prefix=True, use_summary_writer=True):
        runner.train()

Everything seems to run fine, but I get a lot of "Warning: Early Reset Ignored" messages from the procgen env, because they don't normally allow resets before the trajectory is finished. What is the best way to handle that with rlpyt? I've tried using different samplers and Gpu/CpuWaitResetCollector, but no luck.

OK interesting...the environment should only be reset when the done signal comes out True.  Does this happen for a procgen env before the trajectory is finished?
Possibly related, the Atari env has some logic to do with episodic lives, where for RL we want to consider the episode done, but we don't reset the environment.  First, the AtariEnv can output done=True but also env_info(traj_done=False)
https://github.com/astooke/rlpyt/blob/85d4e018a919118c6e42fac3e897aa346d84b9c5/rlpyt/envs/atari/atari_env.py#L127-L129
Second, inside the collector, the environment doesn't get reset if done=True but env_info["traj_done"] is found and is False (defaults to the value of done if traj_done is not found).  If done=True, the agent still gets reset regardless, in case it is recurrent, based on RL episodes:
https://github.com/astooke/rlpyt/blob/85d4e018a919118c6e42fac3e897aa346d84b9c5/rlpyt/samplers/parallel/cpu/collectors.py#L45-L50
Of course you can change this logic anyway needed for procgen.
Let us know if any of this helps, and where the early resets were coming from?


Incoherent owner role for events

If you add yourself as "owner" to an event, this either means:

you take responsibility and are present
you are able distribute responsibility

If you are in charge to manage an event, but cannot physically attend, this is a conflict.

Completed with 94addf1925f3c8297a42117b3261aa0014aac430


Reddit feed displays incorrectly

Self-explanatory I hope. Please revert this change entirely.

I just checked this. Seems to be working on the official YAGPDB server:

Seems to be working on my own server as well:

This issue can be closed.

Make sure you allow the bot to use external emojis

Also, just checked. The bot already had permission to use external emoji, but it still shows incorrectly.

hm that's very strange, a toggle for the style could be added actually, yeah
see if i can get to it by the weekend


Uncaught TypeError: Property 'onbeforeunload' of object [object DOMWindow] is not a function

Seeing that error with this user agent
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/36.0.1944.0 Safari/537.36

There is no places in the source code where window.onbeforeunload is accessed as a property or called as a function. I only attach listeners to window for that event.
Could you describe what you were doing when encountering this error?
A stack trace or a code example would be really useful to understand what happened.

Don't have a stack trace, I am afraid. Will add a source map to that minified code one day.
Looked at other places in my app and can't see where onbeforeunload is accessed. Browserify seems to do something but not sure about that.
Let's close this but keep an eye on it.

Sure, let me know if I can be of any assistance!


Fix some pylint warnings

find_most_recent_start() was using None as a default value, changed to use {} as it was confusing pylint, unsubscriptable-object warning.
consider-using-with is now enforced as there are no longer any occurences of code triggering it.

@brianphaley i attempted to resolve the pylintrc conflict through the github interface but not sure how it will turn out


Py module

Codecov Report

Merging #520 into master will decrease coverage by 0.11%.
The diff coverage is 94.05%.

@@            Coverage Diff             @@
##           master     #520      +/-   ##
==========================================
- Coverage   50.36%   50.24%   -0.12%
==========================================
  Files          68       68
  Lines       13868    13875       +7
  Branches     3436     3445       +9
==========================================
- Hits         6985     6972      -13
- Misses       5032     5052      +20
  Partials     1851     1851

Impacted Files
Coverage Δ

vm/src/obj/objint.rs
72.52% <0%> (ø)
:arrow_up:

vm/src/stdlib/dis.rs
90% <100%> (-0.91%)
:arrow_down:

vm/src/sysmodule.rs
41.53% <92.3%> (-1.32%)
:arrow_down:

vm/src/builtins.rs
34% <95.29%> (-1.9%)
:arrow_down:

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 7dd1eec...0cbcc70. Read the comment docs.


Error when running "Explore_DUDE_diversity_set.ipynb"

Refers to the JOSS submission review: https://github.com/openjournals/joss-reviews/issues/5763:
I was able to run most of the example notebook, but for some reason my res_df_dude_target_*.csv files look different compared to the original files that were provided in the example, leading to an error when trying to generate the tables from the paper using this command:
display(pd.concat([pd.read_csv(f) for f in Path("data/dude/processed/diversity_set/").glob("res_df_dude_target_hivrt.csv")])
        .pivot(values='average_auroc_score',index='target', columns=['similarity_method'],)
        [['Morgan 2', 'RDKit', 'MACCS', 'OpenFEPOPS']])

The error is the following:
File ~/mambaforge/envs/fepops_311/lib/python3.10/site-packages/pandas/core/reshape/reshape.py:210, in _Unstacker._make_selectors(self)
    207 mask.put(selector, True)
    209 if mask.sum() < len(self.index):
--> 210     raise ValueError("Index contains duplicate entries, cannot reshape")
    212 self.group_index = comp_index
    213 self.mask = mask

ValueError: Index contains duplicate entries, cannot reshape

Even though the code that generates the .csv files for the different targets ran successfully without giving an error (calculating the different metrics for the 4 different methods), the output files are missing the Morgan 2 method, and instead have the row for the MACCS method duplicated which then leads to the error when trying to generate the table.
Output example: res_df_dude_target_cp3a4.csv
,target,similarity_method,smiles_count,actives_count,failed_smiles,failed_active_smiles,average_auroc_score,median_auroc_score,q1_auroc_score,q3_auroc_score
0,cp3a4,MACCS,11970,170,4,0,0.5815453804644352,0.5808045598707439,0.42871445517423656,0.45669245058145336
1,cp3a4,MACCS,11970,170,4,0,0.5815453804644352,0.5808045598707439,0.42871445517423656,0.45669245058145336
2,cp3a4,RDKit,11970,170,4,0,0.6133354937630608,0.6263335776833623,0.3858205965880757,0.3903332722458261
3,cp3a4,OpenFEPOPS,11970,170,4,0,0.6474616432055439,0.653921568627451,0.36536023801687506,0.391133003710131

I had run the example notebook as provided on the github page, but had skipped the first step of preparing the DUDE database and instead used the files provided in data/dude/processed/diversity_set
Do you have an idea what could be going on here? Please let me know if it would be helpful if I provided more details!

Hi @hannahbaumann, thank you for raising this issue.
I found an issue on my side with the jupyter notebook which contained a link pointing to version 2 of the data (which was incorrect as you note). Version 3 (released  2023-09-05) should be used.  Apologies for that, I have now updated the link in the notebook to point to v3 (https://doi.org/10.6084/m9.figshare.23951445.v3) in commit ef17f328ce58498828a11c8147b307d45308aa1e.
Another possible issue - can you confirm that you are using the latest version of the jupyter notebook (and/or redownload the new one with the corrected link to FigShare v3 files)? In your text above, you note that the cell which throws the error has glob string of res_df_dude_target_hivrt.csv, but in the current version it has the string res_df_dude_target_*.csv.  If you had changed it just for your testing purposes, then possibly the pivot is failing due to only one DataFrame being loaded.
The notebook works for me using V3 data and the most up-to-date notebook. I have :

Re-downloaded the V3 data zip file and rerun the notebook from the title # Output summary AUROC results table as shown in the paper. This reproduced the table in the paper.
Re-run the entire notebook including descriptor generation. This reproduced the table in the paper.

Please let me know if this fixes the issues.
Many thanks,
Steve

Hi @stevenshave : Thank you so much for your help!
I re-downloaded the data zip file (making sure to download v3) and I also re-downloaded the jupyter notebook from the master branch.
I was able to generate the tables from the paper when using the files form the data folder and only running the notebook from the title # Output summary AUROC results table as shown in the paper
However, when rerunning the entire notebook including descriptor generation I was still getting the same error. Something weird happens when generating the auroc_scores_info_df. In the first round of the loop
info=pd.Series(dtype=object)
for sm_name, sm in similarity_methods.items():
        # Remove entries which did not return a mol
        info['target']=csv_file_path.stem.replace("dude_target_","")

the code correctly generates the dataframe for Morgan2, but then when calculating values for MACCS it replaces the Morgan2 column with MACCS and ends up having two duplicate MACCS columns, one rdkit and one openfepops column. I'll keep looking into this, but if you already have an idea of what could be going on on my side, please let me know!

Hi @hannahbaumann, I've rewritten the benchmark code somewhat and included some guards for results files existing in the current directory, along with changes to the loop which populates results files (#3d28dd).  This works with me running it from scratch, and also with the intermediate files in the data zip (V3-as before).
Would you be able to re-download the notebook and check that the new version works for you.
Many thanks,
Steve

Thank you so much @stevenshave , I was now able to run it and got the same tables as in the pdf!


Removal of PagingScrollView

New Issue Checklist

[ ] Use the latest version of CalendarKit
[x] I've read the Contribution Guidelines
[x] I've searched for existing GitHub issues

Issue Description
So, with the update from 0.6.0 to 0.6.1 and now 0.7.0, I see that PagingScrollView was removed. When it was in, if a user swiped in the Day View, or tapped a new day in the Day Header, it would eventually trickle down the calls to the DayView.timelinePager delegate and to the dayView.didMoveTo call, which we then defined to update the data for the new day.
Now with the removal of PagingScrollView, only swiping of the Day View to a new day is trickling down to the dayView.didMoveTo method and updating the data for the new day. Tapping on a day in the header is not doing anything, the new day view loads in empty.
Result I am trying to achieve
I'm just wondering if there is another way that I'm not aware of, or didn't notice, to have the day header send the call to the dayView.didMoveTo method so that we can update the view with the new day's data? Is there another delegate I have to define to get that to happen, or is that functionality just not there anymore? And if it isn't there, will it be added in the future?
Right now, we switched back to using version 0.6.0, but would like to update to the latest if there is a way to update data when tapping on a new day in the header.
Thanks

Fixed in 0.7.1.
Please, let me know, if the issue still persists.


Produce Debian 11 images

Debian 11 is out and we should product amd64/arm64 images.

This was done


Make this into a npm package and publish on npm

I'll take a look at doing this, I might have time over the weekend.

Hi, how is this going? can you at least add package.json, so we can install this using
npm install git+https://github.com/aackerman/react-flash-queue#


Set permissions for GitHub actions

Restrict the GitHub token permissions only to the required ones; this way, even if the attackers will succeed in compromising your workflow, they won’t be able to do much.

Included permissions for the action. https://github.com/ossf/scorecard/blob/main/docs/checks.md#token-permissions

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#permissions
https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs
Keeping your GitHub Actions and workflows secure Part 1: Preventing pwn requests
Signed-off-by: naveensrinivasan<EMAIL_ADDRESS>

In all the links you provided, I didn't see any need or example for adding actions: write.
Can you please convince me that it is needed?

This has the https://github.com/styfle/cancel-workflow-action permissions https://github.com/styfle/cancel-workflow-action#advanced-token-permissions HTH.

What would fail if we have actions: read ?

I don't know. Probably asking the owner of the action would help.


Releasing capacity after a stream is reset.

Once a stream is reset, the stream level flow control doesn't have to be tracked anymore, however, the connection level flow control must remain consistent.
There are two cases:

Received data is dropped in the library.
Received data is provided to the caller, and the stream is reset before the capacity is released.

In the first case, the library must immediately return the window capacity to the connection. Since the user does not ever get the data, the user will not be able to release the capacity.
In the second case, the capacity needs to be returned to the connection on release_capacity or when all handles are dropped.
This means that ReleaseCapacity must continue to function as expected after the stream is reset.

Originally from: https://github.com/carllerche/h2/pull/174#discussion_r157321005

Looking around this function, it appears there's existing conditions that could cause a stream error after consuming the connection window, and in all of those cases, the window is "leaked".
To fix this, we could add a check outside this function, wherever streams.recv_data() is being called, and if a stream error is returned, try to release the connection capacity again...


Loading image or SVG to pre-fill canvas?

Hi, thanks for this awesome library! It's got a lot of the features I need for a little project I am working on.
Would it be difficult to load an existing image to fill the canvas, which you can then draw on top of?
I've tested the addPath-function. But I'm a bit unsure how to convert an SVG to the format that function takes as a parameter.

I guess you want to draw on top of an image. Just put the image below a transparent canvas. Works fine for me.

The two (canvas and image) won’t merge when you save the canvas though?

Hi! I was looking for this exact feature. I have the same question as @cbrevik, I'd like to get the result image to be the original image with the drawings on top.
Any guidance or help is appreciated!

Just a small follow-up here. Dove into the obj-C code a bit, and added a new method:
https://gist.github.com/cbrevik/35ec7ad441083791a51283f7ab59b1a3 (.patch file created using patch-package)
This will add an image layer beneath the draw layer. Using [self.layer renderInContext:context]; instead of [_layer renderInContext:context]; will merge the two layers when rendering. (I've done it just for the base64-transform here, but should be possible for normal save as well).
So then basically I can do sketchCanvas.addImage(require("./img/someimage.png")) to add an image natively behind the drawing canvas.
In my specific use case, I'm only making an iPad-app, so haven't looked into Android. Also this is for<EMAIL_ADDRESS>


Allow parallel feature checks

Checking for non-zero means that calls like has(tAVX512F | tAVX512VL)
may not work in the case that only one of the checked flags are
true, since it only masks the bit-flag and checks for non-zero, when it
should mask the bits and then check if it equals the mask again to
ensure that all bits are active.
Currently, the user has to call has multiple times:
if( cpu.has(tAVX512F) && cpu.has(tAVX512VL) && cpu.has(tAVX512BW) )
{
    ...
}

this PR allows them all to be checked in parallel with one call
if( cpu.has(tAVX512F | tAVX512VL | tAVX512BW) )
{
    ...
}

Thank you for the patch. I've merged it.


Lands

Description (Required)
Is there any way to pass the actual plugin that uses dough, to the constructor of the Lands integration class in the protection module? It requires this, for flags etc.
https://github.com/baked-libs/dough/blob/fc572efd6c0042310156bba7d9f4516c8406c926/dough-protection/src/main/java/io/github/bakedlibs/dough/protection/ProtectionManager.java#L72
Here is the actual problem:
plugin = Lands, but it should be = plugin that uses dough
https://github.com/baked-libs/dough/blob/fc572efd6c0042310156bba7d9f4516c8406c926/dough-protection/src/main/java/io/github/bakedlibs/dough/protection/modules/LandsProtectionModule.java#L34
I haven't looked through the complete repo, but maybe passing the actual plugin to the constructor of the protection module manager would be an idea?

Any update on this? Should I open a PR?

Yea you can open a pr

Fix has been merged


Rename SllNode class to SinglyNode

Class Definition is present at https://github.com/scriptnull/linkedlist.cr/blob/master/src/linkedlist/singly.cr#L2
Rename all the instances of the class.

I want to do this.

Thank you. Merged and CI passed. So closing this issue.


Add custom constraint option

Description
Added an option to create custom constraints. A usecase could be creating TIMESERIES primary keys.
Checklist

[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt-databricks next" section.

Thanks for submission, running functional tests now.


Code refactoring and add some little feature

Good evening I propose some changes in the code.

First of all a refactoring of the code, renaming the functions for the conventions 🙃.
Creation of the "utils" folder, containing constants and some functions, to avoid code duplication

Feature:

The vertical scroll bar goes back down to the bottom at the end of the song playback.
The horizontal scroll bar remains at the right place after the launching of the reading

Thank you for your pull request ! All seems to work fine and the code is better than ever !


Revise logging statements and config

Currently we are sending too many events to bugsnag. Perhaps set level to warning instead of info?
Also: should we send logging to a local file instead of just stdout?

Closed in #109


Update nRF52840 ADC class to new PDM API and remove warnings

@aurelien-fr  How can I test this PR  ?

Hello !
You can try test this PR by installing the Arduino CLI, getting the patch and compile with the needed compilation option.
The Arduino CLI must be installed because compiler options shall be added, which cannot be done with the Arduino IDE :
sudo curl -fsSL https://raw.githubusercontent.com/arduino/arduino-cli/master/install.sh | sudo BINDIR=/usr/local/bin sh

# Create the configuration file
arduino-cli config init

Add the following line in the configuration file arduino-cli.yaml (should be in ~/.arduino15/) :
board_manager:
  additional_urls:
    - https://files.seeedstudio.com/arduino/package_seeeduino_boards_index.json
library:
  enable_unsafe_install: true

Then :
# Then update the arduino cli core
arduino-cli core update-index

# Install the packages for Seeeduino
arduino-cli core install Seeeduino:nrf52

# Also install Adafruit tool for flashing
pip3 install --user adafruit-nrfutil

Then install manually the needed library from Arduino (patched to resolve compilation failures) :
mkdir -p ~/Arduino/libraries
cd ~/Arduino/libraries
git clone ***@***.***:aurelien-fr/Seeed_Arduino_Mic.git

Then go to the folder and build using arduino-cli :
cd ~/your_test_folder_with_ino
arduino-cli compile --fqbn Seeeduino:nrf52:xiaonRF52840Sense --build-property compiler.cpp.extra_flags="-DARDUINO_ARCH_NRF52840" Leanvia_Beacon.ino

Without my patch the compilation fails.
Best regards,
Aurélien DHIVER

De : Baozhu Zuo @.>
Envoyé : vendredi 3 mars 2023 01:05
À : Seeed-Studio/Seeed_Arduino_Mic @.>
Cc : Aurélien @.>; Mention @.>
Objet : Re: [Seeed-Studio/Seeed_Arduino_Mic] Update nRF52840 ADC class to new PDM API and remove warnings (PR #6)
@aurelien-frhttps://github.com/aurelien-fr How can I test this PR ?
—
Reply to this email directly, view it on GitHubhttps://github.com/Seeed-Studio/Seeed_Arduino_Mic/pull/6#issuecomment-1452750079, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASDFG65YDBJVWDVBQXLSOSTW2EYU7ANCNFSM6AAAAAAVNMGWXM.
You are receiving this because you were mentioned.Message ID: @.***>

Hello, I will review this PR soon.


File URI Failure on Windows

Hello.
When I tried to use the plugin on Windows, I got this error:
Change request failed for unknown file "c%3A/Users/Rubidium/Desktop/Scratch.f90"

Mmmmm... Seems another Windows-specific uri issue :S .

Thanks for the report. I will look into this. Can you see if setting the incrementalSync setting to false fixes this issue?

Yes, in the last update I switched over to defaulting to incremental syncing. I think this is actually an issue with that process on your particular file. The debug messages have a typo so it is reporting "unknown file" even when the file is known.
If you are willing to help out I would like to try and debug the problem. Can you send me the file you are having issues with? Also, with incremental sync enabled do you get this error when making any change to the file or only sometimes? Do you get the same error with other files? Thanks for any information you can provide.

To make sure I understood... By saying 'a typo' you mean the %3A is not translated to :, right?
The input file is a simple test script:
PROGRAM Scratch
    CALL Sub1(1)
CONTAINS
    SUBROUTINE Sub1(A)
        INTEGER, INTENT(IN) :: A
        WRITE(*, *) A
    END SUBROUTINE Sub1
END PROGRAM Scratch

I just meant the error Change request failed for unknown file may be the result of a typo so that the word "unknown" could be incorrect.
However, I think you are right this is a path issue, which I believe I have tracked down. See issue #20 of the upstream repo.

I have released a version to the language server (0.6.1) with a fix to this bug. Give it a try with incremental sync enabled when you have a chance and let me know if you still run into the issue. Thanks again for the report!


No binary found on s3 for Node v7

node-pre-gyp ERR! Tried to download: https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.0.14/fse-v1.0.14-node-v51-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v51 ABI) (falling back to source compile with node-gyp)

In progress, see https://github.com/strongloop/fsevents/pull/153.


Requesting Mod added to CKAN

Hello,
I'd like to request my mod called Antennas be added to CKAN.  The Mod can be found here https://kerbalstuff.com/mod/920/Antennas and depends on module manager to function.
Thank you.

Your mod being added to KS automated a script which sent us a PR in #1618 so it'll be add as soon as we get to it :)

@dtphantom: Sorry it took so long, everyone's on a bit of a break from the project at the same time. =(


Handle existing $9 for controllable MARC Bib fields when transforming legacy bibs

FOLIO requires authorityId values to be UUIDs. We should just ignore any $9 in controllable MARC bib fields.

Going to approach this from a different angle: moving $9 values to $0 so they won't end up in the authorityId field.


[Label] Handle text as multiple independent paragraphs.

See https://github.com/godotengine/godot/pull/79599

Thanks!


Sourcemaps for production environment

Is there a way to generate source maps for production? I have a bug that only reproduces after uglify/mangle

Source maps are generated for production, there is currently no way to disable them.

I'm looking for disabling source map in production :P

Hey!
You can use --no-sourcemap ;)

Closing as --no-sourcemap is now available.

Since #3963 the default has changed. No source maps by default, enable them with --sourcemap

--sourcemap has been changed to --source-map in angular 6


[RFC] Feature specific CSS: Coupling CSS to a feature script

This is a not Github-related feature request. The goal of this RFC is to discuss an idea related to code implementation of Refined Github.
Feature specific CSS: Coupling CSS to a feature script
Summary
Currently, all CSS live in one place: content.css. Feature-specific CSS is not tied to the feature itself. In some situations, especially when a feature requires both CSS and JS, it can be very handy if there is a way for a script to inject custom CSS that is strictly related to the said script.  Something like this:
.
└─ features/
   ├─ add-something-cool.css
   └─ add-something-cool.js

Basic example
// /features/sticky-file-headers.js

import select from 'select-dom';
import {injectCSS} from '../lib/utils';
import styles from './sticky-file-headers.css'

export default function () {
  // when this feature is enabled it will inject some CSS on the page
  injectCSS(styles);
  // Code could also use styles this feature injected on the page
  for (const el of select.all('.file-header .tooltipped')) {
    // do something...
  }
}

/* /features/sticky-file-headers.css */

/* cool styles or global overrides */
.pull-request-tab-content .diff-view .file-header {/* ... */}

Motivation
Refined Github makes file headers sticky throughout Github, which is a very convenient feature! The feature itself is done purely in CSS, so it does not live under the /features directory. It lives inside the content.css file.
There has been a long history of issues that this feature has caused related to tooltips within file headers. This also resulted in some inconsistencies as Github continued to add more tooltips that Refined Github does not account for. I attempted to resolve this particular issue in #1581, but the solution required some JS code to be added.
My first attempt was to add a script called fix-sticky-file-header-tooltips under /features. This approach didn't feel quite right because this isn't necessarily a "feature", but rather a fix for another  feature – the sticky file-header, that is.
@bfred-it suggested that we may want to have a feature script dedicated entirely for sticky headers. I like this idea very much! but there is a catch involved! The feature would still require both CSS and JS!
So the challenge here is:

A) all CSS-based features live in one place
B) there is no easy way to tie CSS to the script of a specific feature.

In situations like this, it can be very handy to have a way for a script to inject custom CSS that is strictly tied to the said script.
Something like this:
.
└─ features/
   ├─ add-something-cool.css
   └─ add-something-cool.js

Detailed design
The main idea behind this design is allowing feature scripts to inject CSS to a page.
Refined Github already does something similar by injecting custom CSS that the user specifies.

This is possible using /lib/utils.js#injectCustomCSS:
https://github.com/sindresorhus/refined-github/blob/f8abd1e5d7c9d3b45bf3f49e58244e01b22938ca/source/libs/utils.js#L154-L160
That helper could be refactored to accept any custom CSS string we pass it, not just from options. Feature scripts could leverage that helper to inject any CSS string to the pages on which they are enabled.
Feature scripts could have adjacent CSS files that they import and inject on the page on-demand.
This can be done with the help of the Webpack loader: raw-loader which resolves CSS files into actual strings.
Alternatives
Manually Styling Elements
// /features/add-awesome-feature.js
import select from 'select-dom';

export default function () {
  const fileHeader1 = select('.pull-request-tab-content .diff-view .file-header');
  fileHeader1.style.position = 'sticky';
  fileHeader1.style.top = '59px';
  fileHeader1.style.zIndex = 10;

  const fileHeader2 = select('.file .file-header');
  fileHeader2.style.position = 'sticky';
  fileHeader2.style.top = 0;
  fileHeader2.style.zIndex = 10;
}

Why should we not do this?

Inline styles!
Very verbose and difficult to work with

Template Strings
// /features/add-awesome-feature.js

import { injectCSS } from '../lib/utils';

const styles = `
  .awesome-thing {
    color: red;
  }
  .dropdown-menu {
    opacity: 0.5;
  }
`;

export default function () {
  injectCSS(styles)
}

Why should we not do this?

Styles cannot be linted
No syntax highlighting or editor/ide css-specific features
Feature scripts become polluted with CSS template strings

More Advanced Webpack Loaders
Such as css-loader/style-loader.
Why should we not do this?
CSS will be injected automatically on import and cannot be injected congenitally.
For example:
import './add-awesome-feature'; // will be injected automatically upon importing

vs
import styles from './add-awesome-feature';
import {injectCSS} from './libs/utils';

export default function () {
  if (somethingIsTrue) {
    injectCSS(styles);
  }
}

Drawbacks
Injecting multiple instances of the same stylesheet
One of the drawbacks I could think of right now is the possibility of injecting multiple instances of the same stylesheet on the same page.
This may happen due to the nature of how some pages transition. Some pages are loaded via Ajax (while persisting the document head). Meaning, enableFeature() may get called multiple times, which will result in multiple calls to injectCSS of the same feature.
To get around this issue, we could add some caching mechanism. Consider the following pseudo-ish code:
const stylesheetCache = new Map();
const injectCSS = css => {
  const sheetId = hash(css)
  if (!stylesheetCache.has(sheetId)) {
    const stylesheet = <style>{css}</style>;
    stylesheetCache.set(sheetId, stylesheet);
    document.head.appendChild(stylesheet);
  }
}

Proof of concept

Unresolved questions

How can we remove styles from the page?

I have not given this a lot of thought yet. For now, I would say that this isn't currently an issue or a concern since Refined Github currently loads the entire content.css on all pages regardless if a feature is needed or not.
If truly a concern, it may be possible to resolve this with the help of the caching mechanism described above.

A proof of concept for this idea can be found here: https://github.com/wsmd/refined-github/pull/1 That PR also include code from #1581

Another alternative would be to add a new class in the JS, e.g. rgh-sticky-header and then reference that in addition to the current CSS selector.

That’s what we do here, yes:
https://github.com/sindresorhus/refined-github/blob/master/source/features/hide-useless-newsfeed-events.js

Thanks for the suggestion! I'll definitely consider that for #1581 👍

I think the classes are an lightweight and low-effort way to achieve this

Thanks for the feedback, y'all!
I can absolutely see how this can be added/unnecessary complexity; it's something that I should've listed as a drawback! Looks like the majority is in favor of using classes so I'm closing this issue for now. 👍


ignore delmark when add minmax for pk column

What problem does this PR solve?
Issue Number: close #4747
Problem Summary:
If rows in a stable pack is all mark deleted, then it's minmax index for pk column is both 0.
And when dt_enable_skippable_place is enabled and try to place delta index on this segment, it will filter out all pack which is not in the pk range [start_key, +infinite). So the previous stable pack is ignored.
But for the place delta index algorithm, it's expected that all the rows in the pk range [start_key, +infinite) no matter whether it's deleted. So when we try to read this segment using the placed delta index, it reports error like "DeltaMerge return wrong result..."
What is changed and how it works?
Ignore delmark when add minmax for pk column.
Check List
Tests

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

Side effects

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

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

Fix potential query error when select on a table with many delete operations

/run-all-tests


Project fork needed

Because

There is community support to continue developing this project
The only person with merge/release/etc. permission (i.e. @lindt ) is no longer available

there is a need to fork this project.
So, @johngluckmdsol , @mnohai-mdsol, and anyone else with an opinion: where will the fork be?

Oh, and there is also a third option: just make a brand new gem that we can publish/maintain ourselves.

I assume there's potential licensing/IP issues with that

No, not really. What kind of issues are you thinking about?

Awesome!!! I'm sure we can copy the code over and site original author and all that jazz we'd be good to go.

Inasmuch as it looks anything like the current code base, sure, I suppose.
Honestly I was just going to write it from scratch. Partly for the fun of it and partly because the internal approach to running all of the linters won't look the same because it will be using cuke_modeler. Remember, I was going out of my way in #20 to change as little as possible. Now that it's a new project, we might as well write it how we like right from the get go.

Yeah, I have not problem with you rewriting it.  I think rubocop has the right idea as far as linters go and I'd support the project adopting the same structure.  I'm happy to try to contribute but I don't have a ton of spare time outside of work.  I'm not entirely happy with the approach.  I was planning on doing some refactoring when I got time and if the need hit.

It's been quiet and slow but I finally have the first release out.
https://rubygems.org/gems/cuke_linter


Label changes to actions workflows to pay attention to requested permissions

The cocoon backend should label, add comments to PRs that change/add files in .github/workflows to pay more attention during the review process on the permissions requested by the workflows.

Closing because the bug has not been updated for more than one year.
Feel free to reopen any bugs that you believe are important but please provide updates on how to resource the fix of the bug and what can be done to prevent the issue from not getting updates for another year.


Add files via upload

Trusttech
Please go the the Preview tab and select the appropriate sub-template:

Production code - for types fix, feat, and refactor.
Docs - for documentation changes.
Others - for changes that do not affect production code.

Trustthech


Test Package Error: cmd/vet or the next package.

What version of Go are you using (go version)?

$ go version
go 1.12.4

Does this issue reproduce with the latest release?
When I compile golang v1.13 beta from source code, the error occurs.
What operating system and processor architecture are you using (go env)?
Windows 10, amd64.
go env Output
$ go env

What did you do?

get source golang code from github.com/golang/go, and I compile it with go v1.12.4.
What did you expect to see?
test package OK.
What did you see instead?
Test Package Error:

Hi @changrui - please tell us exactly what you did (for eg. what commands you ran), and show us the entire output.

hi, today, I do git clone & ./all.bash under Ubuntu 19.04 again, it says ALL TESTS PASS, THANKS.


InvalidCipherTextException while decrypting

Hi ,I am using Bouncy castle to decrypt the file and getting the following exception. I have the private key ,the public key and the passphrase.
Caused by: org.bouncycastle.crypto.InvalidCipherTextException: block incorrect
at org.bouncycastle.crypto.encodings.PKCS1Encoding.decodeBlock(PKCS1Encoding.java:405)
at org.bouncycastle.crypto.encodings.PKCS1Encoding.processBlock(PKCS1Encoding.java:201)
at org.bouncycastle.crypto.BufferedAsymmetricBlockCipher.doFinal(BufferedAsymmetricBlockCipher.java:146)
at org.bouncycastle.openpgp.operator.bc.BcPublicKeyDataDecryptorFactory.recoverSessionData(BcPublicKeyDataDecryptorFactory.java:97)
... 4 more
Process finished with exit code 1

The private key will be incorrect. It could be that it's not linked up with the correct PGP public key encrypted data object.


Add parameterized sampler

So would  you like to have 2 separate wiki pages with the same content, or just 1 wiki page, where we describe difference between controller and sampler?

2 separate pages, please

@undera  done


Nested folders are not deleted in OneDrive delta incremental backups

Deleting nested folders does not delete it from the kopia backup snapshot, and any followup operation within the parent results in the following error:
Error: 1 error occurred:
        * Failed to run OneDrive backup for user<EMAIL_ADDRESS>doing backup: merging details: incomplete migration of backup details

The item within the folder seems to get deleted from the details file as well the folder is deleted from paths in the metadata, but the actual folder and the contents within it still remains in the backup snapshot.
Repro
Create the following folder structure:
folder1
- folder2
  - file

Do a full backup with this, delete folder2 and do an incremental backup. folder2 still remains in the snapshot.
Observations
A collection is created and passed to kopia with the correct state.

hmmm could this have been an intermediate state that the code was in? I've taken the branch in #2604 (commit e3488b9a5c1f952f2f743306800d62f2d669c2bc), made a new branch, cherry-picked #2496 and #2497, and updated the backup version so #2496 works properly. I've pushed the code I ran to a separate branch and used commit 62d83d2df05feba4637f0fb3cfc1b9c535414727 from it to test with. This new branch didn't exhibit the behavior you saw
The folder hierarchy was
- root
  - folder1
    - folder11
      - file.txt

Both backups were run with --enable-onedrive-incrementals. Logs show the first backup attempted hierarchy merging but had no base to merge with (expected) while the second backup did use a base snapshot to merge the hierarchy

Hmm, looks like I can't repro this anymore. The original error was in the branch where I created the demo video, https://github.com/alcionai/corso/tree/odi-demo .

alright, I'll take another look with the branch you linked just in case. Better to investigate now while we can more easily investigate stuff


Update README.md

I don't quite see any files changed here, so closing for now. Please reopen if you think this is a mistake!


Add Terraform modules for creating databases

Some charts have dependencies on MySQL or Postgres.
Consider delegating to pre-configured modules in Terraform docs or registry, like:

AWS

https://www.terraform.io/docs/providers/aws/r/rds_cluster_instance.html
https://registry.terraform.io/modules/terraform-aws-modules/rds-aurora/aws/2.19.0

Azure

https://www.terraform.io/docs/providers/azurerm/r/mysql_database.html
https://www.terraform.io/docs/providers/azurerm/r/postgresql_database.html

Google

https://registry.terraform.io/modules/GoogleCloudPlatform/sql-db/google/3.2.0
https://www.terraform.io/docs/providers/google/r/sql_database_instance.html

As of 54eb93f61963e57fd8816fed7d262d806f50cc29 we now have support for creating MySQL and PostgreSQL instances on GCP.


Working With S/MIME Question

Hi I'm quite new on this email with java, I want to send an email with S/MIME so it will produce a check sign (verified) on gmail. I've got my email certificate a pfx file from Actalis, but don't know where to start.
I've manually tested it by applying it through Outlook and it succeed. Now, I wanted to try it programmatically.
Any help would be appreciated, thanks in advance.

Ok so Simple Java Mail is pretty straightforward: you can encrypt and you can sign, or both.

From the sound of it, you want to sign your emails and for that you need a Pkcs12 keystore which holds your certificate. Unfortunately, you have a different format (.pfx), from which you need to extract the .crt and .key files, so you can build a new Pkcs12 keystore.

Getting the .crt and .key out of your .pfx
-Generating a new .pkcs12 keystore with a .crt and .key

This is a nasty process with a steep learning curve, but at least the objective is clear. Good luck.


Bump snapshot version

Build triggered. sha1 is merged.

Build started sha1 is merged.

Build finished.

Codecov Report

Merging #67 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #67   +/-   ##
=======================================
  Coverage   82.29%   82.29%
=======================================
  Files          10       10
  Lines         209      209
  Branches       32       32
=======================================
  Hits          172      172
  Misses         20       20
  Partials       17       17

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 88fe769...0f8ab5d. Read the comment docs.


Download/Transfer Alma ReCAP Dump

Closes #1049

@hackartisan Yes please! Seems like it'd need some different info

@tpendragon I'm also now wondering, with the deletion of that cron job is there other code that can be deleted?

I..don't know. Coverage isn't great, I don't think I'd want to do that kind of cleanup until during or after #1220.

Fair enough!


Flesh out the fake client even more

This extracts the local rendering code from helm template into a package, and
then uses it to render manifests on demand in the fake client.
This also implements limited Update support in the Fake.
I had to move the tiller.Manifest type out of the tiller in order to break an
import cycle; I left an alias behind, so no users should be broken (although
this requires go 1.9; I don't know what the current minimum required version
is; CircleCI is using 1.10)

Hm. Looks like the bot was dead when I submitted this; are you alive?
/check-cla
/meow

Rebased and pushed a new version with the comments handled.

Ok, just waiting for second review. @bacongobbler maybe?

thanks for the ping! I'll see if I can take a look at some point this week. My priority right now is moving 2.10.0 out the door, so this might take a backseat until that has been pushed through. I'll add a note in my bullet journal to review this though 😄

@bacongobbler just pinging on this now that 2.10 is out :D

Approved as this was submitted prior to the switch from a CLA to a DCO. Thanks!


Near Menu cannot be dragged when in immersive mode

See https://forum.babylonjs.com/t/near-menu-dragging-does-not-work-in-oculus-quest-2/27152
https://playground.babylonjs.com/#8RLLVP#1
@CraigFeldspar - do you have an idea what might be the cause here?

I'll test it on both hololens and oculus and see what the state in both.

It only works with the near interaction plugin and not with the pointer selection. @CraigFeldspar - is there any specific code in the near interaction that enables that?

Sorry for the delay
In the near menu, as in several 3D controls, there is a this._defaultBehavior that encapsulated a sixDofDragBehavior. This is the part that should handle the pointer drag.
See https://github.com/BabylonJS/Babylon.js/blob/83145ae189d61577252935667f71701cf5ebfca3/gui/src/3D/controls/nearMenu.ts#L120

This has been resolved and is working correctly now.


[GitHub-sort-content] Sort repo broken

Sort repo seems to be broken?
See https://github.com/isaacs/github/issues/895#issuecomment-552079088

Hi @gingerbeardman!
Hmm, yeah I think you're right. I won't be able to troubleshoot or work on this right now, but I'll take a look soon. Thanks for letting me know!

After looking at other issue comments I tried to replace Greasemonkey with Tampermonkey, installing the script this time from the GH repo but didn't help.

I recently had to rewrite that userscript because GitHub had changed so many class names. I used the same list of targeted pages, so I'm not sure if I missed that one. I'm in the process of re-installing Windows, so my dev environment isn't set up... and we have plans today. So I probably won't get to look at this until later tonight.

Yep, GitHub changed the attributes I was targeting. All fixed!

Thank you!


cancel_order() on poloniex fails.

Environment:

Operating System: MAC OSX, LINUX
Python Version: Python 2.7.10
Python Bitness: 64
How did you install Catalyst: pip

cancel_order() takes exactly 3 arguments: 2 given.

We reworked the Poloniex implementation and retested all interfaces. This should not be an issue in the next release.

Awesome -- Do you guys have an eta on next release?

Tuesday
On Sat, Dec 9, 2017 at 11:25 PM harir91<EMAIL_ADDRESS>wrote:

Awesome -- Do you guys have an eta on next release?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/enigmampc/catalyst/issues/95#issuecomment-350530358,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZ-Qq-94zkvb4ySluWLZFFO9q8Bgn6jks5s-4dogaJpZM4Q8TBF
.

In Release 0.4.0.

I am getting the same error with the same environment @harir91 above using Catalyst version 0.5.12:
TypeError: cancel_order() takes at least 3 arguments (2 given)

I get this when paper trading on the poloniex exchange.

@lenak25 your workaround didn't work for me. I've tried

context.exchanges['bitfinex'].cancel_order(<order_id>) (order ID as param)
context.exchanges['bitfinex'].cancel_order(<order_obj>) (order object as param)

For both cases I got this error:
ValueError: invalid literal for int() with base 10: 'b74f253082774179bdb8f6592d04421e'

Hi @ykgoon, what mode are you running? This is only relevant for live trading.
I've update my previous message with a more simpler workaround (you can use both, however the first will retry several times in case of a failure).

@lenak25 I've tried both workaround methods, both resulted in the same error. Here are my details:

Catalyst version: 0.5.12
Exchange: bitfinex
Mode: live with paper-trade

I think my issue is confined to Bitfinex. I've opened a new ticket in #346.

You are correct, thanks @ykgoon for validating it and opening a new issue!

@lenak25 thanks for your time. The temporary solution you provided isn't working.
Using the cancel_order(<order_id>, 'poloniex') approach with the following code...
    for symbol in context.blotter.open_orders:
        for order in context.blotter.open_orders[symbol]:
            cancel_order(order.id, 'poloniex')

... I get the same error as above:
TypeError: cancel_order() takes exactly 2 arguments (3 given)

Using the context.exchanges['poloniex'].cancel_order(<order_id>) approach with the following code...
    for symbol in context.blotter.open_orders:
        for order in context.blotter.open_orders[symbol]:
            context.exchanges['poloniex'].cancel_order(order.id)

... I get this error:
catalyst.exchange.exchange_errors.ExchangeRequestError: Request failed: poloniex: unknown error: {"error":"Invalid orderNumber parameter."}

@dlacombejr the fix @lenak25 offered was for live mode only, however, we added a proper fix for all modes that will be added to our next release

added to release 0.5.14


Better character handling

Enabling the :binary option when starting the inotifywait port let's us handle filenames like The Hitchhiker’s guide to the galaxy which would otherwise get messed up because of the ’ .

♥


Publish an SBOM

Like LOG4J2, jbanking should publish a software bill of materials (SBOM) on each release to help end users to more easily discover the versions of jbanking in use in their software. Sonatype has a blog post explaining what SBOM is.
Example of implementation using spdx-maven-plugin : https://github.com/apache/commons-numbers (https://repo1.maven.org/maven2/org/apache/commons/commons-numbers-core/1.1/).
More information about what an SBOM is, related standards, etc.: https://www.ntia.gov/SBOM

There is multiple SBOM formats. The main formats seems to be SPDX (Linux Foundation) and CycloneDX (OWASP Foundation), and both have a maven plugin (https://github.com/spdx/spdx-maven-plugin, https://github.com/CycloneDX/cyclonedx-maven-plugin).

Snyk published a good article on how to create SBOMs in java : https://snyk.io/blog/create-sboms-java-maven-gradle/.

Looking at https://snyk.io/blog/create-sboms-java-maven-gradle/, jbanking will use the CycloneDX Maven Plugin :

It looks like spdx-maven-plugin is still a prototype,
cyclonedx-maven-plugin activity looks much better than spdx-maven-plugin,
CycloneDX has also a plugin for Gradle.


Now with class

This breaks out the output area bookeeping to use a Map between parent_header.msg_id and OutputArea.
/cc @jdfreder

Branch merge and deletion badboy.

Passing

sweet


[Feedback] Cannot transfer an Azure SQL database with Azure AD authentication enabled to a different directory

[Enter feedback here]
In the section "Understand the impact of transferring a subscription" it mentions "You cannot transfer an Azure SQL database with Azure AD authentication enabled to a different directory."
There is a workaround to transfer the SQL server/databases if you follow these steps:

If there is AAD Authentication only enabled, disable it.
Add the SQL logins and users who should be able to authenticate to each database in the server.
Check if the logins with the SQL authentication is working fine and, if yes, remove the AAD administrator from the server.
Move the SQL server to the other tenant.
Reconfigure AAD after the moving the server, by adding a new AAD administrator.
The AAD principals of the old tenant are invalid in the new tenant, so it might be necessary to remove them and add the new principals.

Some thoughts on adding to the documentation:
Is it possible to update this document, or perhaps add to a related document or a new document, to include the above steps? Maybe also include details about how to recover if the transfer was attempted without using the above workaround.
Thanks.

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

ID: 9afab524-67c2-2e76-fd7a-46238cb9d249
Version Independent ID: ae2da577-6026-f322-1c09-dbcc6068c01a
Content: Transfer an Azure subscription to a different Azure AD directory
Content Source: articles/role-based-access-control/transfer-subscription.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@Peter-Msft
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi @Peter-Msft
Thanks for your feedback. We've created a work item for this. I'll contact you to discuss.
thanks

#please-close


fix: update name of config option network_mode

Proposed changes
The documentation of configuration option network_mode uses net. Seems to be a relict from v1 format.

@ulyssessouza Could you PTAL?

@ulyssessouza Could you PTAL?

Thank you @git-developer.

Thank you @git-developer.