ENH: support optional artifacts

Methods and visualizers now support optional artifacts. If an artifact is optional the only supported default value is None.
With this change, signature inputs and parameters are now allowed to be declared in any order (previously inputs were required to be followed by parameters, and they were not allowed to be mixed).
Removed Signature.defaults property in favor of using the ParameterSpec objects available within Signature.
Fixes #235.

Thanks for reviewing @BenKaehler and @ebolyen! I made the suggested changes in two separate commits. Whoever ends up merging this, please "Squash and Merge" and use the current PR description (above) as the commit message.


Make tests pass after 2031

Background:
As part of my work on reproducible builds for openSUSE, I check that software still gives identical build results in the future.
The usual offset is +15 years, because that is how long I expect some software will be used in some places.
This showed up failing tests in our python-apache-libcloud package build.
With this patch, tests completed successfully on i586 and x86_64 in 2034
See https://reproducible-builds.org/ for the overall topic - only slightly relevant here.

done, ready for review

[X] Code linting (required, can be done after the PR checks)

[ ] Documentation

[X] Tests

[ ] ICLA (required for bigger changes)

cool :+1:

See also 2 related issues of 2038-01-19:
https://github.com/pytest-dev/pytest/pull/4903
https://issues.apache.org/jira/browse/LIBCLOUD-1038

Thanks, the change looks good to me.
It looks like https://issues.apache.org/jira/browse/LIBCLOUD-1038 is related to an actual limitation / issue in the xmlrpclib from Python stdlib? And probably nothing we can do about it.
I would imagine that probably already got changed / fixed in Python 3.x?

Some related Python issues:

https://bugs.python.org/issue2985
https://mail.python.org/pipermail/python-dev/2009-July/090649.html

It looks like a solution / workaround for VCL test failure would be to send string integer of an integer when number is larger than 32 bits (and assume VCL API handles that correctly).

I pushed a change which I believe will fix the test failure mentioned in https://issues.apache.org/jira/browse/LIBCLOUD-1038.
It looks like the method already expects a string for start argument (per docstring), so it should be safe and correct thing to do that.

Codecov Report

Merging #1279 into trunk will increase coverage by 0.04%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##            trunk    #1279      +/-   ##
==========================================
+ Coverage   85.91%   85.95%   +0.04%
==========================================
  Files         359      359
  Lines       73777    73774       -3
  Branches     6689     6690       +1
==========================================
+ Hits        63386    63413      +27
+ Misses       7719     7687      -32
- Partials     2672     2674       +2

Impacted Files
Coverage Δ

libcloud/test/common/test_google.py
97.23% <ø> (+2.76%)
:arrow_up:

libcloud/test/compute/test_softlayer.py
92.74% <0%> (+2.66%)
:arrow_up:

libcloud/common/google.py
65.33% <0%> (+3.33%)
:arrow_up:

libcloud/compute/drivers/softlayer.py
84.79% <0%> (-1.09%)
:arrow_down:

libcloud/utils/py3.py
46.25% <0%> (ø)
:arrow_up:

libcloud/test/test_utils.py
95.13% <100%> (+0.21%)
:arrow_up:

libcloud/compute/drivers/vcl.py
88.88% <100%> (ø)
:arrow_up:

libcloud/test/compute/test_openstack.py
95.1% <100%> (ø)
:arrow_up:

libcloud/utils/publickey.py
56% <58.33%> (+30.07%)
:arrow_up:

libcloud/test/compute/test_upcloud.py
91.39% <0%> (+1.32%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

Merged, thanks!


Open SAP Web IDE Full-Stack

Tutorial URL: https://developers.sap.com/tutorials/webide-multi-cloud.html
Please specify the step you are referring to
The tutorial seems to be outdated. When clicking on the suggested link, there is no "Launch SAP Web IDE" button on the landing page.

Thanks for letting us know, you are absolutely right. I updated the tutorial and the change should be live within the next few minutes.


examine a possibility to have the core library with no-std option

Probably is not possible right now, because Error trait is not in libcore and liballoc.


Intellij Dependency

.gitignore update to ignore Intellij IDEA project files.

This goes in your global gitignore file, not in the project's. This project isn't dependent on any IntelliJ IDE, so it's not part of it's resources.


Utils is not defined

Using the npm package, trying to send a controller change message I get :
this.data = Utils.numberToVariableLength(0x00).concat(Constants.CONTROLLER_CHANGE_STATUS, fields.controllerNumber, fields.controllerValue);
              ^

ReferenceError: Utils is not defined
    at new ControllerChangeEvent (/node_modules/midi-writer-js/build/index.js:89:15)

Changing Utils to Utils$1 works, I assume this is a build issue?
Thanks for a great lib.

Just out of curiosity: why would you use such low-level commands?

Hi @RobAley,
Thanks for your message.  Interesting, could you post the full code you're using which results in this error please?
-Garrett

Hi @RobAley,
I think I see what's happening.  I just made a change and published new version 1.7.2.  You want to try again with that new version?
-Garrett

@grimmdude,
Thanks, that works great! A sample of the code I was using below for reference, I changed it to use the helper you added. No errors, and if I play the test.midi it creates with fluidsynth and a sf2 like General User I get a single snare hit as expected
@PedroAlvesV,
They're not particularly "low-level" codes, they're just less common ones more often used when working with physical midi instruments rather than midi files. My particular use is for changing instrument banks. Many sound fonts (and indeed many physical midi keyboards etc.) have more than the midi standard 128 instruments, and so they arrange them into banks of instruments. Control Code 00 ( for MSB ("most significant bit", which is all most fonts/instruments use) and 20 (for LSB, for those that use more) changes the bank, and then you select a particular instrument from within that bank with a "program change" like normal.
var MidiWriter = require('midi-writer-js');
var fs = require('fs');

var track = new MidiWriter.Track();

// change to instrument bank 120 (percussion)

// old code I was using :
//
// track.addEvent(new MidiWriter.ControllerChangeEvent(
//     {controllerNumber : 0, controllerValue : 120}
//   )
// );

// new helper you added :
track.controllerChange(0, 120);

// select program 24 (Electric Drums), without the bank change
// it would be Nylon Guitar in bank 0 (where the 128 midi standard
// instruments are

track.addEvent(new MidiWriter.ProgramChangeEvent({instrument : 24}));

var note = new MidiWriter.NoteEvent(
  // add single snare drum (D2) hit
  {pitch: 'D2', channel : 1, duration: '1', velocity:100}
);

track.addEvent(note);

var write = new MidiWriter.Writer(track);

fs.writeFileSync('./test.midi', write.buildFile() )


fix bug in server-side 'group' definition

In the docs, FlowRouter.group has only one options argument, but on the server it is created with 2 : 'path' and 'options'. By getting rid of the path argument, we get the expected behaviour on the server.

Thanks for the fix.
I've added a test case to this also.
Released with v1.7.1


Split Mesh Explorer partition and material handling

Resolves #2062 .

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#2081
@termi-official
@tzanio
@tzanio + @mlstowell
03/04/21
⌛due 03/18/21
⌛due 03/25/21

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

LGTM

Thanks Tzanio! Sorry, I really forgot the changelog again. It is included now.

No worries, thank you.

LGTM

Merged in next for testing...

LGTM


Xcode 8, Swift 3 - Older Version of the Compiler error

Can we please get a version built with sdk 10.
Module file was created by an older version of the compiler; rebuild 'FontAwesome_swift' and try again:iphonesimulator/FontAwesome_swift.framework/Modules/FontAwesome_swift.swiftmodule/x86_64.swiftmodule
Xcode: Version 8.0 (8A218a)
Pod entry: pod 'FontAwesome.swift', :git=>'https://github.com/thii/FontAwesome.swift.git'

I had the same problem and the fix was (at least in my case) to delete the contents of the DerivedData' folder

Please use 1.0.0.


Make packages that depend on each others in this repo as peerDependencies

To avoid installing them twice.
I'm okay with having minor version dependencies AS LONG AS we agree not to have deep dependencies between packages. This is currently the case.

This is done


Fixed direct use of msg->ident to use driver abstraction function

This fixes direct use of msg->ident to instead use the CO_CANrxMsg_readIdent() abstraction function, since not all drivers employ the same CO_CANrxMsg_t structure as it is a system-specific struct.

The CO_CANrxMsg_readIdent() function is not used anywhere else in the code. Have you checked that this is the only place where this is wrong?

As far as I can tell, the original code base intended this to be the function used to read out the identity of a message. In numerous drivers there's the following comment about the ident member of the CO_CANrxMsg_t struct. Since this struct is implementation/driver specific, we can't always rely on the ident member being used the same way.
/**
 * CAN receive message structure as aligned in socketCAN.
 */
typedef struct{
    /** CAN identifier. It must be read through CO_CANrxMsg_readIdent() function. */
    uint16_t            ident;
    uint8_t             DLC ;           /**< Length of CAN message */
    uint8_t             data[8];        /**< 8 data bytes */
}CO_CANrxMsg_t;

In addition, the actual function descriptions states the following:
/**
 * Read CAN identifier from received message
 *
 * @param rxMsg Pointer to received message
 * @return 11-bit CAN standard identifier.
 */
uint16_t CO_CANrxMsg_readIdent(const CO_CANrxMsg_t *rxMsg);

The reason I submitted this change is that this is the first time the field has been used anywhere in the protocol stack code, and it doesn't work with the driver I'm using because the CO_CANrxMsg_t struct used to interface with the CAN hardware returns some specially aligned SID/EID field that doesn't work with the new CO_EM_receive code, but does work when using the function that properly parses out that field for use.


Handle char parameters/locals properly

Currently, everything on the working stack and the "stack frames" is 2 bytes, and a char is just stored as an int (by putting a 00 byte in front of it).
The problem with this is that in void f(char a), accessing a implicitly involves dereferencing &a, which chibicc knows to be a char*.
uxn is big-endian, so it'll be the case that, for example, &a is 0xfef6, and (mem[0xfef6], mem[0xfef7]) == (0, 'H').
Because dereferencing a char* is implemented correctly, you end up accessing the 0, not the 'H'.

As of 8224195e5332eaf96ee3609c5013a58fe833303a, the chibicc tests (modified for uxn) halt here:
sub_char(7, 3, 3)
did not match expected =>
0001
instead got
0000
Run: Ended.

sub_char is defined like this, so presumably it's this bug that causes the test to fail:
int sub_char(char a, char b, char c) {
  return a - b - c;
}


Add input argument to set number of cores

Motivation
The analysis pipeline can be run in parallel to speed up processing time, but it requires more RAM to do so
Pitch

[x] add an input --num-cpus or -n to the analysis script to set the number of cores to run the analysis on
[x] use parallel to run the analysis on the specified number of cores
[x] default to 1 core

Alternatives
No response
Additional context
No response

closing as fixed in #12


WIP: first steps towards plotting the hospitalisierungsrate

It is a start. We could put this online, and gradually improve on it.

One natural question: is it okay that you added only one .nbdata file while there are two .ipynb files?

Good question!
I have created two notebooks: one to display, and one to help contain some lengthy code that would distract from the plots. The one to display (germany-hospitalisierungsrate.ipynb) will call the other one (germany-compute-hospitalisierungsrate.ipynb) in the first cell to define data and convenience functions.
The germany-hospitalisierungsrate.nbdata file tells pelican to use germany-hospitalisierungsrate.ipynb as a page to include in the html output. I thought it might be okay / good to ignore the other notebook file here (i.e. no nbdata file for that) as we do not want to show that so aggressively. (But that's open for debate. We could add a URL to the location of the compute notebook on github - so that really curious people can find it.)
I hope that explains it.

Codecov Report

Merging #282 (dcb5943) into master (7bbc2f0) will decrease coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #282      +/-   ##
==========================================
- Coverage   87.72%   87.57%   -0.16%
==========================================
  Files           5        5
  Lines        1035     1022      -13
==========================================
- Hits          908      895      -13
  Misses        127      127

Impacted Files
Coverage Δ

oscovida/metadata.py
95.77% <0.00%> (-0.18%)
:arrow_down:

oscovida/oscovida.py
86.05% <0.00%> (-0.14%)
:arrow_down:

oscovida/plotting_helpers.py
100.00% <0.00%> (ø)

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 7bbc2f0...dcb5943. Read the comment docs.

I have replaced the '???' with a more sensible text.


feat(config): add LOADERS variable in config files to enable custom loader gif

SUMMARY

Added a LOADERS variable in the configuration files config.py and superset_config.py:

config.py default value → LOADERS = [{"src": "/static/assets/images/loading.gif"}]
superset_config.py → LOADERS = [{"src": "/static/assets/images/YOUR_GIF.gif"}] (for exemple)

Managed the loader in Superset's base HTML skeleton:

superset/superset/templates/superset/basic.html

Retrieved the custom loader in Superset API endpoints:

superset/superset/views/core.py → Added an endpoint to retrieve the loader from the configuration file.

Managed the loader in Superset's React.js elements:

superset/superset-frontend/src/components/Loading/index.tsx → Fetches the custom loader via the new endpoint.

I'm very new to Superset and the open-source world, and this is my first attempt. I'm open to any advice on how to make this cleaner if needed :)
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
Don't setup the LOADERS in superset_config.py file to see the default loader. Then add tyour custom gif to see the difference.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

@rusackas thank you for your feedback. Yes, I agree that it would be cleaner. I'll take care of it.
As for the array, I followed the same approach as for the FAVICONS. I probably didn’t quite understand why they were in array form as well?
Otherwise, for this new element, I saw a GitHub issue mentioning loader customization. This could serve as a basis in case we want to add keys like the positioning of the loader or other ideas.

@rusackas I noticed that the loader is not handled properly in an embedding context. The image does not load. Do you have any idea what might be causing this issue? I came across a topic that could potentially relate to my problem, but the lines referenced in config.py no longer seem relevant (https://github.com/apache/superset/issues/28112#issuecomment-2062595472) @mistercrunch.


Add missing babel plugins

These babel plugins seemed to be missing from devDependencies resulting in a failed prepublish step:

babel-plugin-transform-class-properties
babel-plugin-transform-object-assign

LGTM. Thanks! 🙌


Test compiled JS assets are updated when src files changed

This should happen in travis, after running through the python-based tests.

Closing in favor of https://github.com/qiime2/q2lint/pull/2


test

Introduction
Test
Why We Propose This API (Change)
Specify which problems this API (change) is addressing
Solution Description
Specify how the addressed problems are solved
Discussion (Pros & Cons)
Specify the pros and cons of this solution


Yaw drift reduction

The "Add filtering" commit is the one that will stay. The "Temporary" one is just so testing this change is easier - a long press of the back key toggles on and off the bias detection. I will remove it before merging.
The change to ktracker_sensor_filter.h is to use the proper c++ operators for memory management; the rest is just formatting.
With this change the drift is hardly noticeable. Without this change just leave the headset on the table and wait 10 mins - the drift will be like 90 degrees.
I have been using Note 4.

Dotan and Vittal, I would appreciate your help reviewing and verifying this change. Thanks.

I can confirm that on my Note4 the drift is about 5 times slower with the new version, but it still exists (which means that if you watch a 1.5 hour movie with the headset, you have to gradually rotate around yourself).
On S6 the drift is much slower for both old and new version, and is moving to the opposite direction. I'm not sure if the new version is better than the old one in this case. On Monday I can do additional experiments along longer time period to get more significant results.

Thanks for the feedback. I will look more into S6 - I guess the sensor in S6's GearVR is much less noisy.

I made some corrections and I see improvements on mine S6. First the filter could've used random values occasionally - I fixed the default vec3 ctor to zero-initialize members. Also I take into account changes in the temperature - as noise changes with temperature.

@dknaan , had any chance trying out the latest?

Dotan showed me that there is nothing noticeably better on his device. What I found out is that his headset produces occasionally peak readings that skew a lot the mean that is used as offset/bias. By increasing the size of the filter things get slightly better but way off the reference. The current situation is that on Note 4 the drift is reduced 5x but even with larger filter, the drift on S6 is the same with or without this change (still too large).
I discovered that there is a special ioctl call that can retrieve calibration information from the device (/dev/ovr0) itself. I need to find a way to put this information to use.

Ok, I have an updated version. Do not forget to change activity_jni.h from
typedef GVRActivityT<OculusHeadRotation> GVRActivity;
//typedef GVRActivityT<KSensorHeadRotation> GVRActivity;

to
//typedef GVRActivityT<OculusHeadRotation> GVRActivity;
typedef GVRActivityT<KSensorHeadRotation> GVRActivity;

before trying it out. I corrected few filtering bugs so the offset is much more stable now.

Constant type fixed

tested on both note4 and s6.  what an amazing difference!  good work!


Investigate Brakeman integration

Few people have asked for Brakeman support. Although scan cannot bundle brakeman due to licensing restrictions, let's think about some kind of integration with external invocation.

Brakeman supports SARIF output. However, the information in SARIF is quite unusable lacking code snippet and line information. Scan can automatically convert json output to SARIF format and enhance by adding snippet and other useful information.
Use the below snippet to produce the json before invoking scan.
sudo gem install brakeman
brakeman -q --no-exit-on-warn --no-exit-on-error -o reports/source-ruby-report.json

# Invoke scan
# docker run or appimage
# Or use scan-action for github

Scan would then take the json report source-ruby-report.json produced and use it for SARIF conversion and build breaker logic. The file should be produced in the reports directory before the invocation.


[Bug]: Long versions are unreadable

Code of Conduct

[X] I agree to follow this project's Code of Conduct

What happened?

Make the dropdown a bit wider 🙂
Version
Beta release
What platform are you using?
Windows
How did you download the software?
GitHub Releases
Relevant log output
No response
Additional data and screenshots
#56

Noticed this as well - once you select 1.19.3 it renders fine it seems.

Kinda, but not fully as it is still too small after the selection.


Short octet stream on tag decoding

The pfx I obtained using adcs esc8
proxychains4 python3 getnthash.py -key 2797e04fc0a00ce9277ff8ebcf276fe0f660158bc970d560c988a2007180a216 redteam/DC$ -dc-ip <IP_ADDRESS> -debug
[proxychains] config file found: /etc/proxychains4.conf
[proxychains] preloading /usr/lib/x86_64-linux-gnu/libproxychains.so.4
[proxychains] DLL init: proxychains-ng 4.16
Impacket v0.12.0.dev1+20240130.154745.97007e84 - Copyright 2023 Fortra

[+] Impacket Library Installation Path: /usr/local/lib/python3.11/dist-packages/impacket-0.12.0.dev1+20240130.154745.97007e84-py3.11.egg/impacket
[+] Using Kerberos Cache: dc.ccache
[+] SPN<EMAIL_ADDRESS>not found in cache
[+] AnySPN is True, looking for another suitable SPN
[+] No valid credentials found in cache.
Traceback (most recent call last):
  File "/home/kali/Desktop/PKINITtools/getnthash.py", line 273, in <module>
    dumper.dump()
  File "/home/kali/Desktop/PKINITtools/getnthash.py", line 121, in dump
    decodedTGT = decoder.decode(tgt, asn1Spec = AS_REP())[0]
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3/dist-packages/pyasn1/codec/ber/decoder.py", line 1337, in __call__
    raise error.SubstrateUnderrunError(
pyasn1.error.SubstrateUnderrunError: Short octet stream on tag decoding
[-] Short octet stream on tag decoding

i think this error occurs because it cannot find the credentials in the ccache file, and then it runs into a parsing error later on. Make sure you have the naming correct for the principal you are trying to target (escaping or quoting special characters etc)

我认为发生此错误是因为它无法在 ccache 文件中找到凭据，然后稍后会遇到解析错误。确保您尝试定位的主体的命名正确（转义或引用特殊字符等）

There is no problem with the subject. You can use the certipy auth - f 1. pfx method to obtain the hash

i think this error occurs because it cannot find the credentials in the ccache file, and then it runs into a parsing error later on. Make sure you have the naming correct for the principal you are trying to target (escaping or quoting special characters etc)

There is no problem with the subject. You can use the certipy auth - f 1. pfx method to obtain the hash


The markdown preview pane in file explorer no longer shows any content of a selected .md file

Microsoft PowerToys version
v0.56.1 & v0.56.2
Running as admin

[ ] Yes

Area(s) with issue?
MD Preview
Steps to reproduce
Select a .md File in the File explorer with the preview pane on, nothing displays in the preview pane. only effects markdown files. pdf files still display, code files display with syntax, but markdown does not. it used to, and I relied on this in my work.
✔ Expected Behavior
The preview pane in file explorer used to show formatted content of a selected .md file
❌ Actual Behavior
The markdown preview pane in file explorer no longer shows any content of a selected .md file
Other Software
vs code 1.65.2
Electron 13.5.2
Chromium 91.0.4472.164
Node.js 14.16.0
v8 9.1.269.39-electron.0
windows_NT x64 10.0.19044

/bugreport

PowerToysReport_2022-03-22-12-48-04.zip

@Aaron-Junker
These are the settings found in the log file. Can be a conflict between md preview and code file preview.
"md-previewer-toggle-setting":{"value":true},
"monaco-previewer-toggle-setting":{"value":true}

so it's one or the other? can't have code preview and markdown preview?

so it's one or the other? can't have code preview and markdown preview?

Not sure. I think we did some adjustments here. Let's see what @aaron-junker says.

Actually from registry-report-info.txt it looks like the markdown preview is not really added to registry. Could you please disable Markdown preview in Settings and re-enable it to see if that helps?

Tried that. Doesn't work. Turning Off the Markdown preview still shows no preview for markdown, turning it back on also still shows no preview in the preview pane.
turning off the developer files preview renders the code files as plain text, turning it back on formats the syntax with colors.
Either way the Markdown files are no longer displayed in the preview pane since a recent power Toys(preview) update I think it started with v0.56.1, I'm currently on v0.56.2

@htcfreek No. Developer preview should not preview markdown files. It did once, but this got fixed before the PR got merged.

There are SEPERATE options on the settings page.  "Enable Markdown (.md) preview" and "Enable developer files preview (Example: .cpp, .py, ...)" The developer files preview works (except that the pane doesn't scroll) the markdown preview option does not work at all, it used to. That along with power-rename were the main reasons I use power toys. I can turn developer files preview off, and the code files still show as plain text which is fine, but still doesn't get the markdown files to preview. not even as plain text.

Could be different accounts. I use about 3 different accounts on this Windows 10 machine (no domain)
(all have admin privileges, but I don't always run with elevated privileges)
I have installed Power Toys for each account.
Registry:
Computer\HKEY_CLASSES_ROOT.md\shellex{8895b1c6-b41f-4c1c-a562-0d564250836f}
Default REG_SZ {45769bcc-e8fd-42d0-947e-02beef77a1f5}
The PDF seems to be scrolling again in the preview pane, after a restart. prior to restarting it would only scroll the first time, then would no longer scroll.
Markdown files still don't preview.

Did you check the setting I asked about in File options?
What's in Computer\HKEY_CLASSES_ROOT\CLSID\{45769bcc-e8fd-42d0-947e-02beef77a1f5}?

And in Computer\HKEY_CLASSES_ROOT\CLSID\{45769bcc-e8fd-42d0-947e-02beef77a1f5}\?

/needinfo

.md files use the same webbrowser technology than .svg files preview. Perhaps that's what's broken?
Could you please check if .svg files preview?
In both cases, can you please send a screenshot of what is shown in the preview pane?
/needinfo

SVG Preview

Markdown Preview Pane

Same Markdown Preview in VSCode

the .md file is encoded in UTF-8 if that makes any difference.

That's interesting. So it's showing white and not saying there's no handler for it. Could you zip the file and send it through GitHub, please?

issue.zip

Not related, it seems :(

I notice in the screenshot of the Registry I attached earlier it shows for
Computer\HKEY_CLASSES_ROOT\CLSID{45769bcc-e8fd-42d0-947e-02beef77a1f5}\InprocServer32\v<IP_ADDRESS>
Assembly:
PowerToys.MarkdownPreviewHandler, Version=v<IP_ADDRESS>, Culture=neutral
v<IP_ADDRESS> is the version of PowerToys
But your screen shot shows v<IP_ADDRESS>
PowerToys.MarkdownPreviewHandler, Version=v<IP_ADDRESS>, Culture=neutral

0.0.1 was from some debug build while we are debugging. I've tested it with the installed version since then.
Have you updated PowerToys to 0.57.2 since?

Just updated to 0.57.2 and now the markdown preview in the windows explorer preview pane works . Thank you

Just updated to 0.57.2 and now the markdown preview in the windows explorer preview pane works . Thank you

I'm very happy for you 😄  But still have no idea why 🤦
Thanks for the feedback.


Deformation architecture

How do we want/expect people to manipulate audio within amen?
The synthesize function is great for re-arranging a clip by timing, but doesn't give us a handle on how to do things like, say, vocal subtraction or time-stretching.
Do we want to provide an object interface for this kind of thing?  Or just let folks hack functions themselves?  Either way, I think we should not support/allow in-place modification of the audio buffers, since it would either trigger an (expensive) feature analysis or have inconsistent results.
For example, a time-stretcher might look something like:
import pyrubberband as pyrb

def amen_time_stretch(audio, rate=1.0):
    y_stretch = pyrb.time_stretch(audio.raw_samples, audio.sample_rate, rate=rate)
    return Audio(raw_samples=y_stretch,
                         sample_rate=sample_rate,
                         analysis_sample_rate=audio.analysis_sample_rate)

This is pretty simple, but it bothers me that you have to access the Audio object's internals directly and propagate them manually.  Maybe that's the only way though?
More generally, I could imagine effects that return multiple clips (eg, source separation), so a consistent object interface might be tricky to pull off here.

Off the top of my head, I like the idea of returning a new Audio object (for time stretching) or two of them (for source separation).
I am not super fussed about Audio(raw_samples=y_stretch,sample_rate=sample_rate, analysis_sample_rate=audio.analysis_sample_rate), though I agree it is not great.
I don't know if we should make these just functions like synthesize, or do something more heavyweight.

Two more thoughts:
a)  An Audio object has various methods, like time_stretch or vocal_subtraction, that returns new Audio objects.
b)  We make objects that do these things to Audio objects - TimeStretcher, etc.  They  also return new Audio objects.
I prefer a), I am pretty sure.  Your thoughts?

As a very simple example, harmonic/percussive splitting:  https://github.com/algorithmic-music-exploration/amen/pull/82

I prefer a), I am pretty sure.

Can you elaborate on why?  The space of audio deformations seems endless to me, and they obviously can't all live as Audio methods.
Deformer objects -- or even plain functions -- seem a lot more flexible and extensible.

Yeah, that's a good point.  I think I like functions better, then - and maybe we stick all of 'em in one file? from deformers import amen_time_stretch, say?
(I think it comes from too much Ruby, and a desire to have objects do everything - as opposed to the Java idiom of having a AudioSplitterFactory to make an AudioSplitter to split the Audio, which I hate.)

Did this as from amen.deformation import harmonic_separation, for the record - but there are only two functions, so we can change things if we want.

Also did this in #105 - @bmcfee, gonna close this issue unless you have Strong Opinions.


Kmeans fails for PNGs

At least very often?
Traceback (most recent call last):
  File "/usr/bin/themer", line 294, in <module>
    generate(color_file, config, template_dir, theme_name)
  File "/usr/bin/themer", line 145, in generate
    colors = parse.read()
  File "/usr/lib/python3.4/site-packages/themer/parsers/__init__.py", line 202, in read
    res = super(WallhavenColorParser, self).read(*args)
  File "/usr/lib/python3.4/site-packages/themer/parsers/__init__.py", line 158, in read
    colors = self.get_dominant_colors()
  File "/usr/lib/python3.4/site-packages/themer/parsers/__init__.py", line 141, in get_dominant_colors
    rgbs = kmeans.kmeans(points, self.k)
  File "/usr/lib/python3.4/site-packages/kmeans/__init__.py", line 103, in kmeans
    tolerance=tolerance, max_iterations=max_iterations)
  File "/usr/lib/python3.4/site-packages/kmeans/__init__.py", line 60, in _kmeans
    (r, g, b), count = center
ValueError: too many values to unpack (expected 3)

  File "/usr/bin/themer", line 294, in <module>
    generate(color_file, config, template_dir, theme_name)
  File "/usr/bin/themer", line 145, in generate
    colors = parse.read()
  File "/usr/lib/python3.4/site-packages/themer/parsers/__init__.py", line 156, in read
    colors = self.get_dominant_colors()
  File "/usr/lib/python3.4/site-packages/themer/parsers/__init__.py", line 139, in get_dominant_colors
    rgbs = kmeans.kmeans(points, self.k)
  File "/usr/lib/python3.4/site-packages/kmeans/__init__.py", line 103, in kmeans
    tolerance=tolerance, max_iterations=max_iterations)
  File "/usr/lib/python3.4/site-packages/kmeans/__init__.py", line 56, in _kmeans
    centers = random.sample(points, k)
  File "/usr/lib/python3.4/random.py", line 315, in sample
    raise ValueError("Sample larger than population")
ValueError: Sample larger than population```

You don't seem to have the kmeans package installed. Can you retry after a pip install kmeans?


elastic.apm.config_file do not worked

We kindly ask to post all questions and issues on the Discuss forum first. In addition to awesome, knowledgeable community contributors, core APM developers are on the forums every single day to help you out as well, so your questions will reach a wider audience there. If we confirm that there is a bug, then you can open an issue in this repo.

-Delastic.xxx is ok!
elastic.apm.config_file do not worked
Describe the bug

Steps to reproduce

Expected behavior

Debug logs

Click to expand
replace this line with your debug logs

You can use it like the other config options as a system property - -Delastic.apm.config_file.
As mentioned in the text you left at the beginning of your description, please don't use GitHub issues for these things, the forum is the preferable way.
Thanks.


$5

Numeric and Control Subfields

Preliminary research:
Control Subfields general definition:

$5  Institution to which field applies  MARC code of the institution or organization that holds the copy to which the data in the field applies. Data in the field may not apply to the universal description of the item or may apply universally to the item but be of interest only to the location cited.

I went through all the tags and these are the MARC fields i found with $5 defined.
0xx:

026 - Fingerprint Identifier (R)  Used to assist in the identification of antiquarian books by recording information comprising groups of characters taken from specified positions on specified pages of the book, in accordance with the principles laid down in various published guidelines.
037 - Source of Acquisition (R)  Source of acquisition information for the item or its reproduction.

1xx:  $5 is not defined for any of the 1xx fields
2xx:

246 - Varying Form of Title (R)  Varying forms of the title appearing on different parts of an item or a portion of the title proper, or an alternative form of the title when the form differs substantially from the title statement in field 245 and if they contribute to the further identification of the item.

3xx:  $5 is not defined for any of the 3xx fields.
4xx:  $5 is not defined for 490, the only currently valid 4xx field.
5xx:

500 - General Note (R)  General information for which a specialized 5XX note field has not been defined.
501 - With Note (R)  Note indicating that more than one bibliographical work is contained in the physical item at the time of publishing, release, issue, or execution. The works that are contained in the item usually have distinctive titles and lack a collective title.
506 - Restrictions on Access Note (R)  Information about restrictions imposed on access to the described materials.
533 - Reproduction Note (R)  Descriptive data for a reproduction of an original item when the main portion of the bibliographic record describes the original item and the data differ.
538 - System Details Note (R)  Technical information about an item, such as the presence or absence of certain kinds of codes; or the physical characteristics of a computer file, such as recording densities, parity, blocking factors, mode of access, software programming language, computer requirements, peripheral requirements, trade name or recording systems, number of lines of resolution, and modulation frequency. For sound and videorecordings, information about the trade name or recording system(s) (e.g., VHS), modulation frequency and number of lines of resolution may be included.  Links to technical details relating to digital resources may also be recorded.
540 - Terms Governing Use and Reproduction Note (R)  Terms governing the use of the materials after access has been provided. The field includes, but is not limited to, copyrights, film rights, trade restrictions, etc. that restrict the right to reproduce, exhibit, fictionalize, quote, etc.
541 - Immediate Source of Acquisition Note (R)  Information about the immediate source of acquisition of the described materials and is used primarily with original or historical items, or other archival collections.
561 - Ownership and Custodial History (R)  Copy-specific field that contains information concerning the ownership and custodial history of the described materials from the time of their creation to the time of their accessioning, including the time at which individual items or groups of items were first brought together in their current arrangement or collation.
562 - Copy and Version Identification Note (R)  Information that distinguishes the copy(s) or version(s) of materials when more than one copy or version exists or could exist.
563 - Binding Information (R)  Binding information intended primarily for use with antiquarian materials, rare books and other special collections.
583 - Action Note (R)  Information about processing, reference, and preservation actions.
584 - Accumulation and Frequency of Use Note (R)  Measurements of and information about the rates of accumulation (for continuing, open-ended accessions) and/or the rate of reference use of the described materials.
585 - Exhibitions Note (R)  Copy-specific field that contains a note which cites exhibitions where the material described has been shown.
588 - Source of Description Note (R)  Information used for tracking and controlling the metadata contained in the record. It includes general and specific source of description notes that are especially of interest to other catalogers.

6xx:

655 - Index Term-Genre/Form (R)  Terms indicating the genre, form, and/or physical characteristics of the materials being described. A genre term designates the style or technique of the intellectual content of textual materials or, for graphic materials, aspects such as vantage point, intended purpose, characteristics of the creator, publication status, or method of representation. A form term designates historically and functionally specific kinds of materials distinguished by their physical character, the subject of their intellectual content, or the order of information within them. Physical characteristic terms designate historically and functionally specific kinds of materials as distinguished by an examination of their physical character, subject of their intellectual content, or the order of information with them.

7xx:

700 - Added Entry-Personal Name (R)  Added entry in which the entry element is a personal name.
710 - Added Entry-Corporate Name (R)  Added entry in which the entry element is a corporate name.
711 - Added Entry-Meeting Name (R)  Added entry in which the entry element is a meeting name.
730 - Added Entry-Uniform Title (R)  Uniform title, a related or an analytical title that is controlled by an authority file or list, used as an added entry.
740 - Added Entry-Uncontrolled Related/Analytical Title (R)  Added entries for related or analytical titles that are not controlled through an authority file or list. (If related or analytical titles are controlled by an authority file, use field 730 (Added entry - uniform title)).
758 - Resource Identifier (R)  An identifier for a resource that is either the resource described in the bibliographic record or a resource to which it is related. Resources thus identified may include, but are not limited to, FRBR works, expressions, manifestations, and items. The field does not prescribe a particular content standard or data model.

8xx:

800 - Series Added Entry-Personal Name (R)  Author/title series added entry in which the author portion is a personal name.
810 - Series Added Entry-Corporate Name (R)  Author/title series added entry in which the author portion is a corporate name.
811 - Series Added Entry-Meeting Name (R)  Author/title series added entry in which the author portion is a meeting name or conference name
830 - Series Added Entry-Uniform Title (R)  Series added entry consisting of a series title alone.

The section from 840 up to 890 contains a number of fields documented in MARC Format for Holdings; I checked the ones not documented in the Bibliographic format in the holdings format, which also has $5 defined.

843 - (Holdings) Reproduction Note (R)  Description of an item that is a reproduction of original materials. Used when an institution chooses to have the main portion of the bibliographic record description reflect the original and the notes in field 843 reflect information about the copy.
845 - (Holdings) Terms Governing Use and Reproduction Note (R)  Terms governing the use and reproduction of a bibliographic item when the holding organization will lend (008/20, Lending policy, code a) and reproduce (008/21, Reproduction policy, code a) the item.
880 - Alternate Graphic Representation (R)  Fully content-designated representation, in a different script, of another field in the same record. Field 880 is linked to the associated regular field by subfield $6 (Linkage). A subfield $6 in the associated field also links that field to the 880 field. The data in field 880 may be in more than one script.
(Note that any subfield from a-z and 0-9 could be used in this tag so that conceivably could be a $5)
885 - Matching Information (R)  Information about the results of a matching process, intended to assist catalogers in the verification of a possible match between two records.
886 - Foreign MARC Information Field (R)  Used when converting foreign MARC records into the MARC format. Contains data from a foreign MARC record for which there is no corresponding MARC field.  (Note that any subfield from a-z and 0-9 could be used in this tag so that conceivably could be a $5

My observation at this point - there may be approaches to specific groups of tags but the contexts for use of $5 are too variable to have a "blanket mapping".  I'll say more in the discussion soon.

id.loc.gov Code List for Cultural Organizations:  https://id.loc.gov/vocabulary/organizations.html  provides URIs based on the codes that would be found in $5, identifying an authority in MADS and SKOS (no RWO?).


Can't pipe identity from get-csonlineuser to set-csuser

I use to be able to Get-CsOnlineUser<EMAIL_ADDRESS>| Set-CsUser -LineURI tel:+1231231234
i get an error that says Set-CsUser: One or more errors occurred. ([NotFound] : userId was not found)
i tested both 3.0.0 and 3.0.1 and both have the same issue.

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Updated initial state to remove value

lol @hmhealey this is included in #1027

Damn, not fast enough 😛


@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/sat-search-feedstock#4.


Feature Request  - Custom Current Location Button

Is your feature request related to a problem? Please describe.
No, this is a new feature request only aimed at enhancing the component. I'd like to have the ability to customize the Current Location button.
Describe the solution you'd like
E.g:
currentLocationButtonStyle={styles.myCustomButton}
currentLocationButtonContent={() => return (
    {customButtonContentHere}
)}

Describe alternatives you've considered
I've not considered any alternatives aside from not using the current location feature.
Additional context
I'd like to be able to add icons to the custom location button (e.g. FontAwesome Location Icon), as well as being able to customize the appearance (style) of the button.

You should be able to do this with the render row prop.
Similar to https://github.com/FaridSafi/react-native-google-places-autocomplete/issues/484#issuecomment-631220610
<GooglePlacesAutocomplete
      query={{
        key: GOOGLE_PLACES_API_KEY,
        language: 'en', // language of the results
        }}
      renderRow={results => {
        if (results.isPredefinedPlace) { // log to see what results returns, but I believe it has an isCurrentLocation property
          return (
            <>
              <Image source="https://i.ya-webdesign.com/images/transparent-pin-google-2.png" style={{ width: 15, height: 25 }} />
              <Text>{results.description}</Text>
            </>
          );
        } else {
          return <Text>{results.description}</Text>
        }
      }}
      onPress={(data, details = null) => console.log(data)}
/>

Closing, because this can already be done.


implement deletion of community tags

From @friendica on June 30, 2013 4:24
According to the original community tag spec, anybody can remove their own community tag and it will be removed from the post/item it is attached to. This event propagates to all stream listeners.
It is removed by deleting the corresponding community tag activity item. ("Bob tagged Violet's post with #octopus")
Additionally, the post owner can remove any tag of their content using the same method.
This needs to be implemented in Red and tested.
Copied from original issue: friendica/red#65

From @friendica on August 1, 2013 5:53
Probably fixed - needs testing


Change api url in documentation

Because of the way we mount this extension, we have to specify 'subscription' before the url (https://github.com/solidusio-contrib/solidus_subscriptions/blob/b2275c71fed698e9d33c89e428dd094a1338a5e9/config/routes.rb#L16). We should consider changing the way this is mounted but for now I have edited the documentation.

This is pretty old. Any more requests/comments?


[DS-3602] Incremental Update of Legacy Id fields in Solr Statistics

https://jira.duraspace.org/browse/DS-3602

I removed work-in-progress since this is essentially my approach.  I plan to do some performance testing on a larger instance and I may refine this code based on those tests.  @aschweer  or @kshepherd , if you have a chance, let me know how this looks to you.

I'll test this on a large repository once the final changes have been made.

@tomdesair , @kshepherd , I have tested on a repo with 3M records and this seems to perform reasonably well.  It would be good to have you review and test this.  I am continuing to run tests on my side.

Here is a link to sample output from the program.  It also conveys information about performance.
https://gist.github.com/terrywbrady/c4ca458488e9bbc76e6c6dcb143a20cb

The latest commit seems to have made the process 2-3 times faster.  I am running 2M records.  I will post another gist with the stats tomorrow.

I have processed millions of records with the current code.  The code currently assumes that every record has an id field.  Looking at my unprocessed records, it appears that the code will need to handle 3 conditions.

id field is present
no id field, scopeId is present
neither id nor scopeId are present

The process is working well when there are a large number of records to update.  The process is less reliable as it closes out the number of records to be updated.  This is due either to (1)a data error in a subset of records OR (2) an error in the way the code is performing a commit and a subsequent search.
Also, we zero out the epersonid (for privacy) in all of our statistics records, so I am unable to verify if I am properly re-mapping epersonid.  When I attempt to treat it as I have treated other id fields, the update fails.

After setting the max solr thread count to 1, a number of issues have been resolved.
The process is significantly more efficient than before.  See https://gist.github.com/terrywbrady/3360e16f9a5934ba0d0f25e34dceaaf5
There are 2 outstanding issues to investigate.

[ ] The process updated more records than were reported as needing updating.

[ ] Verification that epersonid is updated correctly

The latest commit eliminates re-processing of records.  3.3M records are processed in 88 min.
https://gist.github.com/terrywbrady/9e9db70b84851747484108cd2dfbc908

This issue may not be as urgent as we anticipated.  See https://jira.duraspace.org/browse/DS-3602?focusedCommentId=56078&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-56078
The continued dependency on legacyId is definitely confusing.

I started looking at the statistics schema and I noticed that there are additional workflow fields that should be migrated:

[ ] owner

[ ] submitter

[ ] actor

Closing this PR with a preference for including a schema change with this work: See https://github.com/DSpace/DSpace/pull/1774


MITgcm grids and periodic domains

In currrent model form, all choices of model grid (Cartesian, spherical polar, curvilinear, cylindrical) are assumed to be periodic (in x and y) unless a wall is present (or say OBCS is used on a boundary). No check is done to see if grid cell dimensions "match up" as one wraps around, i.e. it is up to the user to make sure the periodicity makes sense given the grid. (probably worth a mention in the doc)
But what is less clear is whether having a wall on just one of the boundaries is sufficient when the grid dimensions don't match up. Perhaps for tracers, but not for momentum? (and does it matter which boundary the wall is on?) Code should be checked and guidance here placed in doc somewhere.

@jrscott
You wrote:
"In currrent model form, all choices of model grid (Cartesian, spherical polar, curvilinear, cylindrical) are assumed to be periodic (in x and y) unless a wall is present (or say OBCS is used on a boundary)."

Is it true that when OBCS is used (even along the boundaries) that the model does not still assume peridocity? I think that the halo region "outside" of boundary OBCS is always updated with values from the "opposite" side of the domain: that's why the OBCS prescribe code "duplicates" user-specified obcs values "out" towards the halos one grid index in i and j.  We are writing an alternative obcs code and found that the model updates the "last+1" i index in x (i.e., the first column on the halo on the "east" side of the easternmost tile) with field in the "first" i index (the first column of of the westernmost tile).
This came up because when that "last+1" uvel is halo'ed over from the first uvel from the westernmost tile, we can end up with tracer cells with large volume divergence (uvel on west edge of tracer cell specified by user, uvel on east edge of tracer cell halo'ed over). Consequently, cg2d has to work hard to solve for an eta for the tracer points along the eastern edge that is ultimatley tossed.
-Ian

@ifenty, @jrscott
As far as I know, the model always copies i = 1,1+Olx to i = sNx+1,sNx+Olx (and similar for i=1-Olx,0 and j). In that sense, it always assumes "double periodicity" (there are some flag that have names implying to suppress that, but I am not sure if they really work).
That means, it doesn't matter where you Insert your wall (e.g. 1 or sNx) to "break" that periodicity and one wall is alway enough. OBCS overwrites these values (if the open boundary is a sNx or 1). But keep in mind that u is staggered so for a boundary point a i = ib, the corresponding velocity points are at ib and ib+1 (and for the tangential velocity at ib). To avoid the large divergence that @ifenty reports, the u-values a duplicated at both ib and ib+1 (so that du/dx = 0) and the topography should also be flat across the boundary (there's a flag that enforces that). But there's also a 2D-field called maskInC (about 10years old) that is particularly meant to mask the open boundary tracer points to avoid the problem of cg2d working too hard that @ifenty describes (as far as I know).
In both cases the rest of the halo still contains the values required for periodic bcs. That's why for an open boundary, e.g in the east, you need a solid wall (or an open boundary) in the west to avoid any surprises.

@ifenty regarding our last point, I though that, when using pkg/obcs, the "interrior masking" was taking care of this (mask out
cg2d matrix coeff and RHS and initial guess so that they are all zero beyond OB) but may be it does not work as it should in your
case ? may be only OB on one side (West but not East, as Martin described it) ?

@jm-c do you know where exactly obcs masks out cg2d matrix coeff and initial guess so that they are all RHS so that they are all zero beyond OB?  I see what @mjlosch describes, obcs copies "out" the prescribed normal velocities and hfac so that that the 'ob' tracer cell has no volume divergence in the normal direction. I'll look for maskInC, @mjlosch is the idea that we apply 0 maskC for the ob tracer cells?

@ifenty what about here:
https://github.com/MITgcm/MITgcm/blob/53744a46b0e2f5d113df9b8e16499020f53e3471/model/src/solve_for_pressure.F#L231-L249


Update MediaPipe version

Hello,
I've noticed that MediaPipe has released new versions of their model. How can I make the update in the MediaPipe.tox of my TouchDesigner setup?
Thanks!

Done! https://github.com/torinmb/mediapipe-touchdesigner/releases/tag/v0.4.3


about exp folder

Hello. First of all, I would like to thank you very much for the excellent project.
As a non-major, I apologize for asking a very stupid question.
1. Question background
I got interested in "Diffusion Model" and found out about DDRM while reading various papers. I was so impressed that I decided to run the code on DDRM GitHub myself.
2. Experimental Environment
OS: Ubuntu 20.04
GPU: RTX 3090
Anaconda virtual environment was created through environment.yml and git clone was performed.
3.
FileNotFoundError: [Errno 2] No such file or directory: 'exp/datasets/ood_celeba’

FileNotFoundError: [Errno 2] No such file or directory: 'exp/imagenet_val_1k.txt’

FileNotFoundError: [Errno 2] No such file or directory: 'exp/datasets/ood_bedroom’

Should I fill the exp folder with something?
Thank you for taking the time to read my stupid question.😭

I solved a few things while studying, but I got another question.
Resolved
1. /exp/datasets/imagenet/imagenet
I realized that I had to download the dataset myself and place it in the exp/ folder.
(imagenet = imagenet ILSVRC2012 validation data)
2. imagenet_val_1k.txt
I realized that the imagenet_val_1k.txt file should be imported from
github.com/XingangPan/deep-generative-prior/
New Issue
I don't know where to Download the datasets below.

odd
odd_bedroom
od_cat
od_celeba

Thank you so much.

Hi, Ramge132, it seems that all these four datasets could be download from this link:
https://github.com/jiamings/ddrm-exp-datasets
which is in the readme.


Create a settings panel

The content and layout of the page is static. There should be a settings panel where the user can:

Edit/Customize the shortcuts
Change fonts
Change username
Edit background/shortcut color
Change clock type (12 hrs/24 hrs)

I can do this. Please assign it to me.
Also, I am doubtful about how do you want the shortcuts to be customized? Please clarify.

On second thoughts, I think we should not use settings panel for customizing shortcuts. It would probably be more convenient to do that by right clicking the icons themselves. I shall add a new issue for that and edit this issue for the same.
For fonts, we are going to include two offline fonts i.e, Fredericka the Great 1 and Montserrat 2. You can read the conversations in #11 #14 and #20 to get more details about this. For rest of the fonts, we can use google fonts. Try using their API to get a list of of all the available fonts and display it to the user. This may be a little tricky and if you face any problem, feel free to comment here and I shall help you with this.
Rest everything should be okay. But do not hesitate to ask for help.
Thanks for taking up this issue.

Hey @sudo-rgorai , I read the conversations in the mentioned issues but I'm confused how to use the API to get a list of available fonts. Through this website, I have generated an API key and that gives me a very large JSON code. How to proceed now?? Should I create a JSON file and link the file to index.html ? Will that serve the purpose?

The JSON that you have received contains data about several fonts. One of those fields is family which is basically the font name. Run a loop through this JSON data and extract these family names and display these to the user in a drop-down menu. When the user clicks on one of these names, store it in a variable, say fontFamily.
Now if you notice, the link statement for using google font has a common syntax. See here. So once you have the fontFamily variable, all you need to do is inject the following lines via your JavaScript:
<link rel="preconnect" href="https://fonts.gstatic.com">
<link href="https://fonts.googleapis.com/css2?family=**fontFamily**&display=swap" rel="stylesheet">
Once you have done that, you will need to change the greeting's font-family(css) property to fontFamily. In this way the greeting's font can be changed.
Whenever you're using JavaScript, make a different .js file and work on it. Then import those functions in index.js. This is because we are trying to improve code modularity (#9) .
Give this a try. If you need any more help feel free to ask here. If you get really stuck, try to complete as many features as you can and then let me know when you're done. I shall complete the rest.


Status Reports are not getting downloaded by Authors

Dear Team,
Company Communicator Authors are unable to download the Status Reports. Please let me know the solution, Senior Leadership team are using this and its not working for them need your assistance.
Below is the Screenshot when I try to download it gives error says company communicator does not supports files. Old downloaded data also sends repeated alerts as message expired. Is there any way to stop those.

Getting the same issue after upgrading to v3. API permissions look correct based on the new one required.

I think the issue is in the manifest.json for the end users app under the bots area supports files is set to false.

I'd set it to true, increment the version, rezip and update the app.

@bungleweed  -  the authors app under the bots area supports files is set to True. It still the same issue. Can this be resolved without a version upgrade. As I am not sure if version upgrade in production is a good idea. Can you let me know if Microsoft Support can assist on call for version upgrade.

@Sheshank-TCS If you update the author application, its only the authors who will get the update. Could you try and let us know if that resolves the issue?
We are planning to split bot for users (recipients) and authors in future updates.

@Sheshank-TCS this issue was fixed in release v4. please upgrade to resolve this.

Please let us know if you still face the issue after upgrading the app.


Adding Scratch battle animation

For ease of testing:
FullScreenPokemon Save<PHONE_NUMBER>993.zip

How it currently looks:

ScratchBlock:

ScratchLine:

ScratchLineInverted:

What it ended up using:

Reference:

from: http://talking-time.net/showthread.php?t=12128&page=2


Fix refreshing of persistent connections

#1385  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lazynina and the rest of your teammates on  Graphite


Add Multiple Pages to Textbox

Issue
Textboxes can only have 5 lines
Solution
Add multiple pages to the texboxes
Extra Info
This may not be necessary, since textInput() would have to be able to show all the text as well
Type of Issue
Addition

Honestly, not necessary.


‘sut_bootest.py’: integrate network readiness time

line 447: complete code segment (incl new function) for ['link_is_up']
change name to 'link_is_ready'

added 'net_str' var and integrated with cmd_linkup syntax

verified


MSC2625: Add mark_unread push rule action

Rendered

sorry, I should have said: this is being superceded by #2654. Trying to get both "unread" and "notifications" out of the same list of push rules was a nightmare.


Question. When should I reflash the coordinator firmware?

Currently, the version of the coordinator firmware installed on my zzh! stick is CC26X2R1_20200805. My question is when and in which case do I need to update the firmware? The version of Zigbee2mqtt is 1.14.4. I am planning the update to the latest version to support the recently added gledopto light stripe GL-MC-001P (support was added in version 1.17.0). Together with Z2M update, do I need to update the zzh! firmware to support new devices?

firmware to support new devices?

Nope!
Generally speaking only update when you are having problems. (or just like to be up-to-date :))

Great! Thanks for your reply!


Added Number_Guess Ga

PR Description 📜
Bug free game and also having responsible on android and tab devices also..
Fixes #3214

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @prasad-chavan1 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀


Changed test cases 48,49,50,51,52,53

Passed after the comments of Rob on NMS-185 & NMS-188

Can one of the admins verify this patch?

ok to test
On Thu, Jul 2, 2015 at 2:42 AM, motech-gerrit<EMAIL_ADDRESS>wrote:

Can one of the admins verify this patch?
—
Reply to this email directly or view it on GitHub
https://github.com/motech-implementations/mim/pull/451#issuecomment-117979605
.

I believe this works duplicates the work done in #445


[WIP]: Add Backends and elaborate Dispatcher

This PR includes the following changes:

Updated the import statement in the example code in the README to reflect the new package structure.
Deleted the nx_parallel/centrality.py file. I'm assuming this was a stub, since otherwise I'm not certain I understand why it needs to be in this repo. If there was a particular reason in mind, however, will gladly add it back.
Switched to absolute imports to follow best practices recommended by PEP8.
Various changes made to the nx_parallel/interface.py file, including:

Added the Backends class to handle different backends for parallelization.
Added a new _chunks method to divide lists of nodes or edges into chunks.
Added a new static method reduce_partial_solutions for aggregating mapped results. Note: we might further experiment with different reduction operators besides sum.
Updated the _create_node_iterables and _create_edge_iterables methods to create iterables for node and edge inputs.
Added __call__ method to handle joblib-based parallel computation following all inits.

Added a new optional_package function to the nx_parallel/misc.py file to accommodate for the possibility of multiple backend dependencies.
Updated the dependencies section in the pyproject.toml file to impose some minimal version requirement for NetworkX and joblib.
Once there's consensus on general restructuring, will gladly add unit tests.
Added tutorials folder with betweenness_centrality.ipynb to demonstrate usage and prove speedup.

Just discussed with @MridulS , and will close + re-open subsequent a new PR, in light of realizing the decision was made to follow the @nx._dispatch spec closely.  Will also aim to increment the changes a bit through a series of PR's...


Add support for text input

A simple text input element.
The value entered can be read with Dart by
final InputElement inputElement = document.getElementById('textfield') as InputElement;
final String? text = inputElement.value;
print(text);

Please read CONTRIBUTING.md. If you're interested in implementing user input widgets in the Dawn package, it is recommended to propose a concrete approach containing all variations (text, password, select, etc.). This way, we can make sure that Dawn is steadily on the right path and can be beneficial for its users.


GUVNOR-2866: [Guided Decision Table] Columns swaping doesn't work in all cases

See https://issues.jboss.org/browse/GUVNOR-2866
There actually wasn't any support for DnD'ing "BRL" columns.

@Rikkola @karreiro Would you mind reviewing please, when you get a chance?

@manstis I think that there are some formatting issues.

@karreiro Unfortunately there are no formatting issues; there is a new "code style formatter" in droolsjbpm-build-bootstrap specifically you could read this and the referenced PRs.. I did ask for it to be communicated to BSIG, but it looks like it never happened :-(

@manstis thanks! :-)


Problem after adding new function to Bebop file

Hi @amymcgovern ,
Im a young french student, and I have, with a friend, a school programming project with the bebop2 drone.
We need to move the drone to several points (with GPS coordinates). It seems there is no function for this in your pyparrot code, although bebop2 has the ability to move to a GPS coordinate point, judging by the xml file of the drone.
That's why I've created one (here below), based on your work and reusing your functions. But, after testing it, a problem arise, that my friend and I can't solve (although we've gone through your pyparrot files...).
We have implemented this 'move_to' function in the Bebop.py file (just after your 'flat_trim' function). Then, we tried a simple script to see if the function worked, by calling the 'move_to' function. And an error was raised by python, with a large traceback. Or, more accurately, I think two errors raised, one that it is intercepted and then displayed, and one that is not intercepted. Both are AttributeError: 'tuple' object has no attribute 'update' and 'tuple' object has no attribute 'set_max_altitude'. The tuple object in question is the sensors object in the Bebop.py file.
So the program obviously crashed. But even more surprisingly, the scripts that worked before, don't work anymore, even after the original Bebop.py file has been restored...!
So, I have 2 questions :
-First one : is the function correct, at least theoretically? Is there an error, something to replace or add?
-Second one : I don't know how this problem can occur, nor how to solve it. Therefore, Im wondering if you could help us to solve this problem (cause we can't use the bebop2 right now)?
Hoping that you can understand why this error appeared, and that you will agree to help us,
Thank you so much in advance!
CODE created by ourselves with move_to function :
`# -*- coding: utf-8 -*-
"""
Created on Sat Apr 25 22:04:08 2020.

@author: adaml

Code à insérer dans la bibliothèque pyparrot, dans le fichier Bebop, dans le constructeur du même nom.
Définition des méthodes move_to et cancel_move_to, permettant au drone d'aller à un point de coordonnées GPS données, et d'annuler la manoeuvre.

À noter : implémenter également cette ligne dans le fichier Bebop (de la bibliothèque pyparrot toujours), et dans le constructeur BebopSensors : self.MoveToEnded = False

Tout a planté lors de son utilisation.
erreur : la variable self.sensors du constructeur Bebop est tuple...!?
"""

def move_to(self, latitude, longitude, altitude, orientation_mode, heading=None):
    """
    Adam and Alexi's function. Move the drone to a specified location.

    :param latitude: location's latitude to reach(degrees)
    :param longitude: location's longitude to reach (degrees)
    :param altitude: location's altitude to reach (meters)
    :param orientation_mode: Orientation mode of the move to
    :param heading: Heading (relative to the North in degrees).This value is only used if the orientation mode is HEADING_START or HEADING_DURING
    :return: nothing but move to location.
    """
    fixed_orientation = orientation_mode.upper()
    if (fixed_orientation not in ("NONE", "TO_TARGET", "HEADING_START", "HEADING_DURING")):
        print("Error: %s is not a valid direction.  Must be one of %s" % orientation_mode, "NONE, TO_TARGET, HEADING_START, or HEADING_DURING")
        print("Ignoring command and returning")
        return

    (command_tuple, enum_tuple) = self.command_parser.get_command_tuple_with_enum("ardrone3", "Piloting", "moveTo", fixed_orientation)

    if heading is not None:
        if fixed_orientation not in ("HEADING_START", "HEADING_DURING"):
            print("Error: heading parameter cannot be used. Can only be used if orientation_mode is 'HEADING_START' or 'HEADING_DURING'")
            print("Ignoring command and returning")
            return
        param_tuple = [latitude, longitude, altitude, enum_tuple, heading]  # Enable
        param_type_tuple = ['double', 'double', 'double', 'enum', 'float']
    else:
        param_tuple = [latitude, longitude, altitude, enum_tuple]  # Enable
        param_type_tuple = ['double', 'double', 'double', 'enum']

    command_tuple = self.command_parser.get_command_tuple("ardrone3", "Piloting", "moveTo")
    # reset the bit that tells when the move ends
    self.sensors.MoveToEnded = False

    # send the command
    self.drone_connection.send_param_command_packet(command_tuple, param_tuple, param_type_tuple)

    # sleep until it ends
    while (not self.sensors.MoveToEnded):  # we added this MoveToEnded variable to the BebopSensors class, so it exists
        self.smart_sleep(0.01)

def cancel_move_to(self):
    """
    Adam and Alexi's function. Cancel the move_to command. If there is no current moveTo, this command has no effect.

    :return: Nothing
    """
    command_tuple = self.command_parser.get_command_tuple("ardrone3", "Piloting", "CancelMoveTo")
    self.drone_connection.send_noparam_command_packet_ack(command_tuple)
`

everything I did was indoors so I never implemented the GPS functions.  In theory, they should work just fine.  I am guessing you broke something fundamental with the edits (though it isn't clear what it is!) so I would reinstall and start there (but keep your old code around).  Python is odd sometimes that way!  Others on the discussion board have discussed GPS movements and I would also look at those.

I'm guessing too, but yes it's not clear, cause we only added function...
Thanks for your advice. So we have uninstalled pyparrot's codes and installed again. We can't test the bebop2 right now, so we will look at the discussion board, and test all in few weeks. I'll let you know the result.
Thank you so much for your prompt response.

Hi, apologies for reviving this old issue. I was wondering if you were able to solve this @Adrials? I'm stuck here too. I send the move_to command and receive the acknowledgement, but the bebop2 doesn't move.

Hi @faweigend, I'm sorry. The COVID crisis forced me to stop the project, so I've never had a chance to look at the problem again (insofar as the drone wasn't mine...).
Here are the only two things I can tell you:

apparently (I can't remember...^^), you also need to add this line: self.MoveToEnded = False in the Bebop.py file, in the BebopSensors class (in the init function I think).
If the command doesn't raise an error, I would try to print out the drone's states at each step and compare them with another original command. Maybe something fundamental is not being updated (and other subclasses should be modified).

Hope you find the solution (by the way, let me know if you do, I'm still interested)!

Sorry, I haven't had working bebops for awhile and haven't got a reason to buy a new one (no longer part of my research program).  If there is a solution and I need to accept a change, I will though!

Thanks for the replies @Adrials @amymcgovern,
I found a solution and successfully tested it on bebop2. It is not fully integrated though, because I had to work around the convenience enum function. I hope to get the time to integrate it properly soon. For anyone who wants to use my current workaround:
  def move_to(self, lat, lon, altitude):
      command_tuple = [1, 0, 10]

      # Calculate packet size:
      # base packet <BBBIBBH is 11 bytes
      packet_size = 11 + 8 + 8 + 8 + 4 + 4

      seq_str = 'SEND_WITH_ACK'
      ack_string = 'DATA_WITH_ACK'
      self.bebop.drone_connection.sequence_counter[seq_str] = (self.bebop.drone_connection.sequence_counter[
                                                                   seq_str] + 1) % 256
      seq_id = self.bebop.drone_connection.sequence_counter[seq_str]
      packet = struct.pack(
          "<BBBIBBHdddif",
          self.bebop.drone_connection.data_types_by_name[ack_string],  # data with ack
          self.bebop.drone_connection.buffer_ids[seq_str],  # send with ack
          seq_id,  # sequence counter
          packet_size,
          command_tuple[0],
          command_tuple[1],
          command_tuple[2],
          lat,
          lon,
          altitude,
          0,  # mode: NONE
          0.0  # heading is ignored
      )
      res = self.bebop.drone_connection.send_command_packet_ack(packet, seq_id)
      self.bebop.smart_sleep(1)
      return res


Middle Layer Output

Hi, Is it possible to get the middle layer outputs from a model? For example, If I use
model  = haiku.nets.ResNet18(num_classes=1000)
Is there a way that can I get the output before the last classification layer or any middle CONV layer output?

At least for the model you've mentioned I don't think that you can without going in and making changes to the source files.
In general you can either: use multi-transform to output a tuple of (final-layer, desired-output) or I think you can use a module called Oryx from TF probability

@IanQS
Hi IanQS, Thanks for your answer. I'll try to modify the source files of haiku.nets.ResNet18.


How to add hover styles to mj-button?

Hey guys,
Is it possible to add hover/focus/active styles to mj-button? I can't get it to work.
I want my button to have a border on hover, but nothing happens with the following code.
Having hover/focus styles is very important for accessibility.
<mjml>
    <mj-head>
        <mj-title>example</mj-title>
        <mj-attributes>
            <mj-class name="button" padding="0" background-color="#00A091" border-radius="30px" />
        </mj-attributes>
        <mj-style inline="inline">
            td.button td:hover,
            td.button td:hover a{
                border: 3px solid red !important;
            }
        </mj-style>
    </mj-head>
    <mj-body>
        <mj-section>
            <mj-column>
                <mj-button mj-class="button" css-class="button" href="https://google.com">Visit google.com</mj-button>
            </mj-column>
        </mj-section>
    </mj-body>
</mjml>

Thank you very much!

Oh my god, I'm so dump 🥺 Thank you so much. This indeed worked!


Fix link styling for text components

For #10448
New look :

Coverage remained the same at 34.018% when pulling 7c0b1324360811e7461babfbe03d95d39982ec36 on chgagnon/fixLinkStyling into b83279c24c6cdc8d3a4d422fb3c31fed9612bdc2 on master.


Cannot get latest version: module contains a go.mod file, so module path should be github.com/fanux/sealos/v3

Background
The github.com/fanux/sealos uses Go modules and the current release stream version is v3. It must comply with the specification of "Releasing Modules for v2 or higher" available in the Modules documentation and enforced since the most recent versions of Go. Quoting the specification:

A package that has opted in to modules must include the major version in the import path to import any v2+ modules
To preserve import compatibility, the go command requires that modules with major version v2 or later use a module path with that major version as the final element. For example, version v2.0.0 of example.com/m must instead use module path example.com/m/v2.
https://github.com/golang/go/wiki/Modules#releasing-modules-v2-or-higher

Using go get (under go1.13 and go1.14) does not work for anything later than<EMAIL_ADDRESS>which was the last version to not have modules.
I consider this a pretty high severity issue since not doing so prevents any other Go project also using Go Modules to properly require fanux/sealos on versions higher than v2.0.3 as a dependency, resulting in errors like:
invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v3

SO anyone using Go modules will not be able to easily use any newer version of fanux/sealos.
Steps to Reproduce

Under a project making use of Go modules, run go get targeting any version >= v2.0.4 of the fanux/sealos:

$ go get<EMAIL_ADDRESS>go: finding github.com/fanux/sealos v3.3.4
go: finding github.com/fanux/sealos v3.3.4
go get<EMAIL_ADDRESS><EMAIL_ADDRESS>invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v3

Solution
Kill the go.mod files.
This would push them back to not being managed by Go modules (instead of incorrectly using Go modules). Importing specific versions would work, but they'd receive the +incompatible prefix and would need to be manually upgraded.
Probably best option short-term -- better to not use Go modules than use it incorrectly.
Also, best option for older branches/versions that aren't maintained.
Make breaking changes
Patch the go.mod file to declare the module as github.com/fanux/sealos/v3 as per the specs. AND adjust all internal imports.
This would probably only be acceptable on the v3 line, since it'd break any consumers.
Two methods can be use:

a new branch in the same repo
use the folders structure on mainline branch (see Major Subdirectory structure on the Modules page).

Then development would take place in the new branch/repository, with the existing only updated with tags and/or with new things to forward.
Major version bump / repository change
Leave v3 as a dead version and bump to v4 with the path changes.
References

https://github.com/golang/go/wiki/Modules#semantic-import-versioning
https://golang.org/cmd/go/#hdr-Module_compatibility_and_semantic_versioning
https://github.com/golang/go/wiki/Modules#releasing-modules-v2-or-higher

@fanux @cuisongliu

下个版本重构再支持吧。

@cuisongliu   Thanks for your efforts and feedback.


Stops sending emails after rebooting the docker host

I am using the docker-compose variant which works fine until I reboot my docker host. After that emails to maillists are accepted but not conveyed any longer.
I investigated this issue futher and found out that mailan's qrunner lock files are not deleted when stopping the container. Manually deleting the files solved the problem temporarily: docker-compose exec mailman bash -c "rm -f /var/run/lock/mailman/*"; docker-compose restart
I also observed that docker-compose stop  takes 10 seconds - thats the default timeout after which the processes are forcibly terminated. Increasing the timeout (compose option stop_grace_period: 3m ) leads to same result.
I shortly inspected /run.sh. Here the mailman process is started by /etc/init.d/mailman start  but never stopped when catching a SIGTERM signal. Consequently a container restart does not cleanup the locks which inhibits queue processing.

Hi, Thx for reporting this... usually prod servers are quite stable and as we schedule to destroy containers on a regular basis to replace them with new ones so that we are close to immutability, we never encountered this issue before.
BTW, this container declares volumes for important data (lists, archives, tls cert...) so they won't disappear after container replacement.
But you are right, a graceful stop of the processes in this container while a "docker stop" is asked is consistent (containers are too often maltreated as they are replaceable) and it would fix immediately this stop/start issue in case of a server reboot so that it won't have to wait a container replacement to be fixed.
The "docker stop"  command sends an initial SIGTERM signal to PID1 and if no termination happens during the defined grace period it kills them with a SIGKILL. As this container was conceived rather like a convenient mailing list tools deployment than a standard functional container the PID1 is given to the run.sh start script that runs several services and indeed doesn't transmit, for now, the termination signal to the services started.
So I'm gonna add a few lines ASAP to the run.sh script to stop the services properly when a SIGTERM is received.
I'll tell you when the updated script is pushed so that you can pull the new d3fk/mailman2:latest container version.

The new version of the run.sh script has now been pushed and the container automatically built by Docker Hub ... a simple docker pull d3fk/mailman2 before deploying a new mailman container should fix this issue on your next server reboot or container stop/start cycle.

Your fix has worked excellently. Thank you very much! :-)


Support for mulit character variable names + symbol tests

It is possible to use the evaluateTex function with symbols (a, b, c), but a symbol out of multiple characters like aa are seems to get tokenized to single character variables and therefore is unable to find the symbol a, though provided through the scope object like with {aa: 2}.
This PR introduces some tests to show, it's working with single char symbols and throw an "undefined symbol a" error when a multi char symbol is provided.
A simple workaround for now seems to just map the variable names to single char names, since single char symbols are working fine.
Maybe there is a problem with parsing of multiplications, since aa could be a variable, but also the shorthand form of a * a

I've closed the old PR.
This one handles multi-char variables also as variables. The tests show the successful handling of multi-char vars.
With the provided conversions, f.e.:
const vars = {a : bignumber(257), b : bignumber(936)};
const ans = math.evaluate('a/b + a/2', vars);

evaluates to<PHONE_NUMBER>495726495726495726495726495726495726495726495726495726 and
const vars = {a : 257, b : 936};
const ans = math.evaluate('a/b + a/2', vars);

evaluates to<PHONE_NUMBER>495726496.
So the precision is lost when using numbers. To have the full precision, one have to use the bignumber function explicitly for the variable values.


Makefile updates

Processes to add to the Makefile include:

[x] importing the Aries place data into PostGIS.
[x] process Aries from GeoJSON to SHP & reproject (only needed if the data isn't accessible via PostGIS)
[x] compile sub-directory of .mss files into single .mml files
[x] generate highway shields table in Postgres using sql/highway_shields.sql
[x] add ne_10m_admin_1_labels to PostGIS using sql/admin1_labels.sql
(also requires dl'ing & extracting ne_10m_admin_1_states_provinces)*

* I imported the ne_10m_admin_1_states_provinces data into PostGIS by doing the following:
$ shp2pgsql -s 4326 -I -g geometry \
~/data/natural_earth_vector/ne_10m_admin_1_states_provinces_scale_rank.shp \
ne_10m_admin_1_states_provinces terrain_tokyo | psql -d terrain_tokyo

So we're adding ne_10m_admin_1_labels to the database and not the others because we do the generalization for #16 in PostGIS?

@almccon yes that's correct.

I'm just giving up on the mss/ folder. We'll just keep everything in the few monster .mss files for now.


feat: build ci

Add a build workflow yaml file.
When you push a tag like v0.1.0, the flow will run and add a release.
Check Release v0.1.0

Great job!


Don't show the live game in the games panel or display it as LIVE since it is completely empty otherwise.

It looks like @samdharris is going to have a look at this and see if they can resolve it as a good starting point to be more involved in the project.

@MambaDev I'm considering making the API do this. Even though I've done it on the front-end, it makes more sense for the API to return that data?

@samdharris The api is going to return all games, active or not. Since these could be used else where, e.g updating the section of the home page to show live game status and results.
Just as a note, if you feel a change would be a 'business' change, start by opening a issue that could result in a discussion. When its more than just a fix.


Disable OpenSSL system configs when downloading secrets

So that Bitwarden CLI can use older algorithms.
Resolves #329.
Signed-off-by: Hunor Csomortáni<EMAIL_ADDRESS>

I think we need a work-around also for the scripts/update_bw_secret.sh, don't we?

Updated secrets/README.md with instruction to set OPENSSL_CONF=. PTAL :)

Something's very wrong here, I don't have that issue on F36 :D did you get the binary or installed via npm(*)?

Got the binary.

Tried just now, got it from https://vault.bitwarden.com/download/?app=cli&platform=linux
$ ./bw get item secrets-tls-certs
Not found.


Memory Leak

I found that the unused self.dist1 and self.dist2 in the file "nndistance/functions/nnd.py" cause memory leaking in my environment. (Python 3.5.2 with Pytorch 0.4.0)
class NNDFunction(Function):
    def forward(self, xyz1, xyz2):
        dist1,dist2=cuda_compute_from(xyz1,xyz2)
        # following two lines cause memory leak
        self.dist1 = dist1
        self.dist2 = dist2
        return dist1, dist2

    def backward(self, graddist1, graddist2):
        gradxyz1,gradxyz2=grad_cuda_compute_from(graddist1,graddist2)
        return gradxyz1, gradxyz2

I know the reason. In Pytorch 0.4.0, it should use
'''
ctx.save_for_backward()
'''
Thank you! AtlasNet is a very nice work~!

Indeed @liuyuan-pal ! I updated the code to pytorch v1 and fixed the memory leak with a hugly hack by explicitly detroying the custom chamfer distance layer after the backward call. It seems dirty though. I'll look into your solution, thanks for pointing int out !
Best,
Thibault

Awesome, really good pointer ! it's fixed! thanks @liuyuan-pal

So this memory leak issue is still there. do you have an insight what might be the cause?

Hi @nghorbani,
Can you share your setup (pytorch, cuda) and the exact command that's failing for you please ?
Best,
Thibault

I see that you have tested the code with pytorch 0.4.1, py37and cuda 9.0.176,7.1.2_2.
However, my setup is pytorch 1.1.0, py3.7, cuda 10.0, and cudnn-7.5.
Could that be version differences? if yes, then have you considered upgrading the code for a recent framework?

Could that be version differences? No... I have tested the code with the latest source.
Can you share a minimal code snippet that fails for you please?
Cheers,
Thibault


iPhoneX Portrait and Landscape support

Apetools don't generate splash screen files to new Apple devices.
Maybe the recommended image upload dimensions should be set to 2436x2436.

Any updates on this?

New splashcreen specs have been added, you can also feed it any image size of the right proportions if you want to start with a higher resolution image.


[vcpkg] owns doesn't seem to be working properly

Describe the bug
$ vcpkg owns qt5-base
qt5-base:x64-osx: x64-osx/share/qt5-base/copyright
qt5-base:x64-osx: x64-osx/share/qt5-base/vcpkg.spdx.json
qt5-base:x64-osx: x64-osx/share/qt5-base/vcpkg_abi_info.txt
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/fixqt4headers.pl
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/moc
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/qdbuscpp2xml
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/qdbusxml2cpp
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/qlalr
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/qmake
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/qvkgen
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/rcc
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/syncqt.pl
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/tracegen
qt5-base:x64-osx: x64-osx/tools/qt5-base/bin/uic

Missing many files

Missing many files

It is: vcpkg owns <dir/file> and not vcpkg owns <port>  maybe that is the misunderstanding here? So you own get info which ports owns the directory or file called qt5-base

Missing many files

It is: vcpkg owns <dir/file> and not vcpkg owns <port> maybe that is the misunderstanding here? So you own get info which ports owns the directory or file called qt5-base

In fact, I need to list files owned by port.

Can this be added as a new feature? Something like vcpkg owned <port>? It's much meaningful than vcpkg owns <dir/file>

list the files in packages/${PORT}_<triplet>

I install by adding --clean-after-build, this may not work.

See vcpkg_installed/vcpkg/info/qt5-base_5.15.7_x64-osx.list

Thanks! @Neumann-A @autoantwort


Response in the console is very slow

What steps will reproduce the problem?

The issue is related to a very slow response when executing simple commands in the console.
Executing 1+1 takes 5s to echo the result. Similar when printing text strings or variables.
Several people in our departement experience this, all of us having gotten new Lenovo Thinkpad P15 G2 computers. On our old P52 computers, Anaconda/Spyder (console) was quick.
We have tried updating windows and BIOS, but nothing has helped.
Any suggestions of how we can get around this annoying problem?

Hey @gustafp68, thanks for reporting. I'm afraid I'll have to ask you to reopen this issue with all the information we require from our users (i.e. Python, Spyder, Qt/PyQt, operating system versions and installation method).
If you prefer, you can do that directly from Spyder by going to the menu entry
Help > Report issue

which will attach it the issue is opened here.


Add z-score table and z-score functions

Would be nice to add a z-score table and some of z-score utilities for as find the best sample size to get a margin of error alpha and confidential coefficient beta

The diffs (48 changed files) don't appear to agree with the description. Was this based on maste or a different fork? Perhaps a rebase gone awry?

@caioesr I am going to close this PR as: the diff does not correspond to the intended change, there was no update for a while. Feel free to open another PR (or clean this one) if you want to continue this work. Thanks!


Guilds not getting members cached

Please describe the problem you are having in as much detail as possible:
Guilds are seemingly at random having no member objects. Even after multiple messages received and minutes later. guild.me and message.member are undefined along with everyone else. These are not new guilds.
Further details:

discord.js version: Master
node.js version: 8.4.x
Operating system: Win 10 + Ubuntu 16.x Server
Priority this issue should have – please be realistic and elaborate if possible: Medium/High - This is a fairly breaking issue having to constantly restart the bot to get a guild to cache member objects only for a different guild to report issues caused by them not having any members now.

Is your own bot invisible? If so, that would explain why the bot member is not cached.
Otherwise, any uncached members can be fetched with Guild#fetchMember.

No, the bot is not invisible. While yes, i could check if the member object of both the bot and the message sender is cached on every command, and if not fetch it. That seems like a band-aid solution to larger problem of guilds having no members cached ever.
I don't think this is a intended behavior especially when its happening to only a few guilds at a time.
Every time i restart the bot within 2hrs there is 1-2 people complaining that things arn't working which are caused by both message.member and guild.me returning undefined.

Are these guilds over 250 members?

No 48 members on the most recent noted issue guild

A bit more info. A small 22 man guild went from working fine to no longer having valid objects for the bot member or any message member. For this specific guild the issue persisted them kicking and re-inviting the bot, and was only fixed once i rebooted the bot as a whole

Did you set fetchAllMembers to true in your client settings? You can try that, and see if that fixes it

the member for the client is always sent regardless of guild size, client status, fetchAllMembers settings, etc

You should not rely on msg.member to always be present @gmedeadlystrike
If a member is invis for quite some time you would have to fetch his member each and every restart of your bot. And its not really a band-aid to fetch every time someone triggers a command because if the person is already in cache you get the cached member back.

I understand, I guess I will make sure the member object is present. It does however look like there is a issue with the client not having a member object cached
as was noted here https://github.com/Gawdl3y/discord.js-commando/pull/70

Hmmm. seems relatetd to my issue: https://github.com/hydrabolt/discord.js/issues/1848

Could you send me any stacktraces if you have them, and also debug guild.members.size at the time of the error? I want to see if the members cache is being cleared accidentally.

@hydrabolt - guild.members.size was infact 0 during a couple of these issues

@gmedeadlystrike do you know if the server experience any outages in this time?

Can you also debug client.users.filter(u => !u.id).size at the time of error

Could you try using master on the lastest commit ^ 👍

Seems to be fixed for @dragonfire535, I'll give it a few more days before officially declaring as fixed 👍

It's back, attempting to get more information.

Yep, issue is still happening

So i got some more info on this issue. I use the commando framework which by default will await the caching of the client and user member objects before running commands to make sure their available. You can see an example of this in this PR https://github.com/Gawdl3y/discord.js-commando/pull/73 - Which such a check in place I have not noticed any instances of the user member object not being in cache, but in fact still on occasion notice the clientuser's member object being undefined.

So... it's still in cache but sometimes still undefined?

Sorry, maybe i was unclear. The method you would normally use to ensure and force the member objects (client + user) into cache is in place and being executed, however a few cpu cycles later the clientuser member object can return undefined, im not sure if that means its no longer in the cache or of guild.me is returning undefined for another reason

Can anyone here try: npm i -S bdistin/discord.js#patch-30 and watch and see if the issue shows up, or if that pr fixes the issue?

This shouldn't be closed 👀

@dragonfire535 ok, I have not seen this issue pop up in a week or so now, so 🤷♂

@gmedeadlystrike have you been able to reproduce this in the last few days? :s

@hydrabolt I'm not sure if this is related but filtering from GuildMembers in a Guild from Message gives many statuses offline.
const totalOffline = members.filter(m => m.user.presence.status == "offline").size;

If you log the status, most of them are appearing offline.

This issue has been pretty stale for a few months now, open a new issue if this still persists please.

Error: Invalid or uncached id provided


⚠ Samsung has degraded performance

In 73be01c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2928 ms

Resolved: Samsung performance has improved in 39ec9a2.


Bug: Test Reason Code not Populating on UI

Steps to Reproduce
Navigate to the QA test data section in the global view
Select AES Alamitos (315). Location 3.
Select Test type Transmitter Transducer
The Test reason code is blank in the data table and in the modal. According to the reporting instructions test reason code is a required field.
Note: Working as expected in the workspace

Actual Results

AES Alamitos (315). Location 3, Test type Transmitter Transducer has null value for Test Reason code.

Bug resolved. The Test Reason Code is not available coming from the database. There are no issue with API and UI


backend: add controller / service tests split principle

refactoring with service layer restructurings and this makes sense

In my world, ideal controller tests should work like black box and use only API methods to modify and read data from service.
They should check status codes, success path, 404, 422 errors (maybe not all 422) and ACL
From other hand we have complex service level where services can call another services, enqueue jobs, perform multiple db operations depending on various input params. It is useful to check these cases via tests for services.
I can give you example where controllers tests become very complex and difficult to read -> document CRUD in Pablo
Document CRUD in Pablo has a great count of different scenarios, numerous side effects e.t.c It is really hard to place them into controllers.

It is a good way to define well structured entry point to complex service functionality like I did for example in ledger generator in Pablo or for reconciliation. So this entry point should not be changed during refactoring and tests should use it instead of call some nested service methods

Yet another reason to bring tests to service level is ability to call same service function from multiple places, for example from controllers, jobs, events ... I this case if you place tests in controller you will need to repeat them (or part of them) in jobs, events, e.t.c

I agree that in some cases testing a service function is more clean and reasonable.
But I would prefer to keep top-level entity (controller / job / event) as a default way for testing like it is described here:
https://github.com/OsomePteLtd/principles/blob/main/src/testing.md#strategy

integration tests must be the majority
unit tests can be used if needed only for the corner cases of the scenarios covered by integration tests

"Identical request/response" looks for me not enough for jumping to service level testing. Can we define the condition another way?

It is a good way to define well structured entry point to complex service functionality like I did for example in ledger generator in Pablo or for reconciliation. So this entry point should not be changed during refactoring and tests should use it instead of call some nested service methods.

And maybe we should describe more clear what is "the main service function" which we can test.