Notes are not displayed

Hello!
I have recently encountered an interesting issue: While processing references with notes in Obsidian and inspecting available fields in template customization window , no notes appear in the JSON schema under the neither in the "notes" nor attachments fields.
@aidenlx could you please check if there are schema incompatibility issues? According to this forum post notes are stored in ItemNotes table.
Best regards
Asan

Update: I have checked the zotero.sqlite. It does content notes. And the schema did not changed in the most recent Zotero beta. But I noticed that there are different behaviors in different setups:

the latest Obsidian, Zotero v7.0.0-beta.54+6b996d4f9: Notes are  available for Zotlit and could be rendered in md.
the latest Obsidian and the latest Zotero: Notes are not available for Zotlit, and the "notes" field in the template editor is represented by an empty array.

The other fields but "notes" possess data in the JSON schema inside the template editor.

The issue seems to be related to https://github.com/PKM-er/obsidian-zotlit/issues/272

I have a note with some highlights coloured using Zotero-provided colour highlighting feature. This is possibly the cause of the error in notes schema constructing:

plugin:zotlit:359 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'renderColored')
    at Object.replacement (plugin:zotlit:359:37939)
    at k.D (turndown.js:2:9551)
    at turndown.js:2:9210
    at NodeList.reduce (<anonymous>)
    at k.b (turndown.js:2:9059)
    at k.D (turndown.js:2:9459)
    at turndown.js:2:9210
    at NodeList.reduce (<anonymous>)
    at k.b (turndown.js:2:9059)
    at k.D (turndown.js:2:9459)
replacement @ plugin:zotlit:359
D @ turndown.js:2
(anonymous) @ turndown.js:2
b @ turndown.js:2
D @ turndown.js:2
(anonymous) @ turndown.js:2
b @ turndown.js:2
D @ turndown.js:2
(anonymous) @ turndown.js:2
b @ turndown.js:2
D @ turndown.js:2
(anonymous) @ turndown.js:2
b @ turndown.js:2
turndown @ turndown.js:2
turndown @ plugin:zotlit:359
normalizeNotes @ plugin:zotlit:359
eval @ plugin:zotlit:350

After I deleted colour markup from my notes, "notes" field in template editor worked normally. Lit note was also created normally.
I suggest such coloured highlights processing is to be considered as a #bug @aidenlx

same issue here.

与 https://github.com/PKM-er/obsidian-zotlit/issues/321 是同一个问题。希望作者能够修复…


Minor fixes based on go report card

Signed-off-by: Doug Davis<EMAIL_ADDRESS>

are we gonna get a nice go report card badge?

already there: https://github.com/kubernetes-incubator/service-catalog

So it is.


Mac ProjFS: Deny I/O on offline roots, with exceptions, part 1

This is the first stage of implementing #182 - the kext now blocks write I/O to empty or placeholder files on offline roots.
At first glance this seems like it would be a very simple change, but in practice, some processes must be allowed unfettered access to files in offline roots. So a large part of this patch set is dedicated to implementing a system for allowing exceptions - there is a new type of IOKit user client for processes which need unrestricted access to offline roots, and this needed to be implemented all the way through from kext, via native lib, to (managed) VFS4G code proper.
Next, the vnode handler, which is in the actual business of denying or allowing file access, has been expanded to allow returning different results based on whether the root's provider is offline.
Finally, "deny" results are actually returned for a number of different cases - so far, this only includes writes. Even this caused some test failures, so the test process itself needs to register as an exception with unrestricted offline root access. This code was kindly provided by @jeschu1.
Review Notes:
This is a pretty big change set so I recommend reviewing commit-by-commit:

Minor macro tweak - tiny bug fix to the KextLog_VnodeOp macro for an issue I ran into while working on this. The vnodeType argument can itself be the result of an expression, and so must be surrounded by parentheses to avoid operator precedence problems.
Adds system for registering processes for offline root I/O - This implements the kext-side IOUserClient subclass for processes which need access to offline roots, code for maintaining the active list of these exceptions, plus some seeds of actual exception logic in TryGetVirtualizationRoot. (At this stage no exception is actually needed because no access is denied yet.) There are also unit tests for covering much of this new code.
User-space support for offline I/O process registration. - This adds the ability to register and deregister as an offline root exception process to the native PrjFS library, along with calls to the new functions from the managed side from the Main() method of the gvfs tool.
Preparation for denying access to offline roots. - This change identifies the various sites in the vnode handler where access will need to be denied, and prepares the code such that this behaviour can easily be enabled in a subsequent change. Doing so adds some new code paths, so new tests are provided to cover those paths.
Register/Unregister for offline I/O in functional tests - This commit by @jeschu1 registers the functional test runner as a process which is allowed to perform I/O on offline roots. This is a requirement for the tests to pass once the subsequent commits are applied.
Denies access when provider does not respond - If the provider process does not respond to a message from the vnode handler, we deny the operation, whatever it was. This is perhaps not ideal; a cleaner way to implement this would be to behave as if the provider was offline. That looks like it'll involve some refactoring of the vnode/fileop handlers, which I'll leave for #900.
Denies write access to empty files in offline root - What it says on the tin. Writing to empty files makes no sense, when the provider comes online, it'll think the file is still empty and hydrate it, overwriting whatever the user stored there. Adds a test that verifies that such accesses are indeed denied by the vnode handler.
Denies write access to placeholder files in offline root - Placeholder (hydrated, non-full) files in an offline root must not be writable, or the provider will miss the PreConvertToFull/MileModified events and incorrectly track it as unmodified. This is prevented by this change; tests are updated to reflect the new expected behaviour.

/azp run GitHub VFSForGit Mac Functional Tests

/azp run GitHub VFSForGit Mac Functional Tests

On Windows it appears I can write to any hydrated file (no need to register the pid i.e. PrjFS_RegisterForOfflineIO).
Curious why we didnt' do the same as Windows here for Mac.

On Windows it appears I can write to any hydrated file (no need to register the pid i.e. PrjFS_RegisterForOfflineIO).

Hmm. I'm assuming this means any thusly modified file doesn't show up in git status when you do remount the root?

I've just rebased this PR on latest master and incorporated the review feedback so far. I've not made any changes outside review feedback.

Hmm. I'm assuming this means any thusly modified file doesn't show up in git status when you do remount the root?

@pmj that is correct.
I had an offline conversation with @wilbaker  and the net net is that there are differences between the mac & windows implementation.  On windows you only modified hydrated files if the directory is marked as full.  We don't have that concept on Mac.  Therefore the way you've done this PR with registering for offline writes is the most ideal approach we could think of.

Isn't there also the case where we can have two threads do
resizeArrayLength = MIN(MAX(1u, s_maxOfflineIOPIDs * 2u), UINT32_MAX);
And allocate a block of memory for that size?
If so I think everything works we just copy twice. Perhaps a comments explaining that to make it simple for the reader.

No, we shouldn't copy twice as the race will be detected by the resizeArrayLength > s_maxOfflineIOPIDs check, in which case one thread's alloc'd memory is freed again without using it. At least that's what the code is supposed to do, please check that also matches your interpretation. :-)

o, we shouldn't copy twice as the race will be detected by the resizeArrayLength > s_maxOfflineIOPIDs check, in which case one thread's alloc'd memory is freed again without using it. At least that's what the code is supposed to do, please check that also matches your interpretation. :-)

:-) What I was thinking is
Thread 1 enters
uint32_t resizeArrayLength = MIN(MAX(1u, s_maxOfflineIOPIDs * 2u), UINT32_MAX);
Then release the lock and calls
pid_t* resizedArrayMemory = Memory_AllocArray<pid_t>(resizeArrayLength);
It then waits for the lock
Thread 2 enters gets the lock (while thread 1 is waiting above)
uint32_t resizeArrayLength = MIN(MAX(1u, s_maxOfflineIOPIDs * 2u), UINT32_MAX);
Then calls
pid_t* resizedArrayMemory = Memory_AllocArray<pid_t>(resizeArrayLength);
In that case they'd have the same array length I think.  Let me know if I missed something.  Threading is fun cause it's like little puzzles :-).

In that case they'd have the same array length I think. Let me know if I missed something. Threading is fun cause it's like little puzzles :-).

They will indeed have allocated the same amount of memory. Only one of them "wins" the if (resizeArrayLength > s_maxOfflineIOPIDs) check, however, and resizes the array. Once the array is resized, s_maxOfflineIOPIDs will be equal to resizeArrayLength by the time the second thread gets to that check, so it'll take the else branch which just frees the memory again.
It might make more sense if you read that if statement as "will this memory we've just allocated actually grow the array?"

They will indeed have allocated the same amount of memory. Only one of them "wins" the if (resizeArrayLength > s_maxOfflineIOPIDs) check, however, and resizes the array. Once the array is resized, s_maxOfflineIOPIDs will be equal to resizeArrayLength by the time the second thread gets to that check, so it'll take the else branch which just frees the memory again.

Ah yes, in the case I described s_maxOfflineIOPIDs would have been altered so the case holds.  THanks for pointing that out, yes looks like all cases are covered!

@wilbaker can you please also review?

Sure taking a look now

I think I've now addressed all the feedback, please re-check!

/azp run GitHub VFSForGit Mac Functional Tests

/azp run GitHub VFSForGit Mac Functional Tests


Update Documentation

rewrite install documentation with path map example and screenshots
update screenshots (new UI)
how to control the log levels
how to run the test suite

I had a idea for a Reactor "network" installer concept I never rolled out, that could be interesting to adapt for a Fusion centric Crypto test suite that would function as a render farm "deployment validation" stage.
Fusion supports SxS (side by side) scripts where a .lua script and a .comp have the same base filename like "basename.lua" and "basename.comp". When the Fusion comp is opened, the SxS script is run automatically.
Fusion also supports intool scripting via lua that lets you pack a minimal Lua script into a node in your composite for events like frame render, or start render, or end render task handling.
It would be possible to make a "Cryptomatte-Validator.comp" that could be loaded into the Fusion Render Manager and run as a job across all the Fusion Render Nodes on a render farm.
The test comp would be able to make an output of a validation test image per render node, and a JSON text file report from all Cryptomatte test units on each render node with the hostname, fusion version, and other details logged, too.
Plus a PathMap scan could be done by a validation script to locate and report duplicate Cryptomatte fuses, or legacy Crypto Lua modules present on each render node, along with fuse version checks too.
JPDOC and other WSL users have reported in the past Cryptomatte issues on their render farms with dual versions present so making those situations easier to troubleshoot would be handy.
This could be another edge-case need sort of Reactor Cryptomatte distributed tool option which would be handy for people using the crypto  fuse tech "at scale" in their VFX workflows.


[HS-281] combine topic and publisher spread

Goal
Initial implementation of a ranker that spreads publishers and topics simultaneously.
Reference
HS-281
Implementation Decisions
I start by spreading by topics since these may reflect explicit user preferences.  If possible, once we've disqualified some of the available candidates based on their topic, we look for items that allow publisher spreading with a default gap of 2 items.  When this is not possible, we return the next available candidate, i.e. no items are removed from the provide candidate list, only reordered.

I would guess that the change to the corpus item model will break other tests.  I made publisher a required field there.

tests are failing since assets lack a publisher field.  tests/assets/json/sorpus_candidate_sets.json


Readme encoder output_size not fully understood

In the code the encoder output size should be :
(batch_size, encoded_image_size, encoded_image_size, 2048)=(batch_size, 14, 14, 2048 )
in the Readme description image :
it is (batch_size, 14, 14, 4096 ) !=(batch_size, 14, 14, 2048 )
is there something i miss or not understanding? Thanks for your valuable work and i am looking forward to your applying !

Is the 4096 from VGGnet ?

I think the resnet output dim size should be 2048, so , is the 4096 from VGGNet ?

Thanks for noticing! I wrote the tutorial some days after I wrote the code, and I must have had 4096 in my mind for some reason.
You are right. The number in the code is correct. It is indeed 2048 channels at the end of the ResNet-101 (likewise for the ResNet-50 and ResNet-152).
I'll correct the numbers in the Readme tomorrow. Sorry about the confusion!
(If I remember right, the VGGnet has 512, similar to the smaller ResNets.)

@insigh Fixed!


Unreadable content in markdown-preview code blocks

The color for plain text in markdown-preview code blocks is equal the background color and can therefore not be seen.

This issuse describes a similar bug as #45.

Changed to @markdown-preview-text-color.

I just installed nord-atom-ui for the first time (0.10.2) and am still affected by this. Is there something else I need to do?

Thanks for your report 👍
This is caused by the new Theming API introduced in Atom 1.13.0.
I'll have to move this feature to the Nord Atom Syntax package because the new CSS selectors won't allow to override syntax highlighted elements outside of the syntax theme package itself. I also think that this should be supported by the syntax theme anyway since it styles a result of a language and is not related to parts of the UI.
I'll create a issue for it and add a @-mention so you can track the progress 😄

@arcticicestudio Thanks, I absolutely love the theme and am using it across everything I can.

I'll try to migrate the code to the Nord Atom UI latest next weekend.
In the meantime you can change the ⚛ Atom core package [markdown-preview] to render previews with the GitHub style as a workaround to avoid these unreadable code blocks.
It can be changed in the package settings.

I just recently installed the nord-atom-ui with the nord syntax package. But in markdown preview, any text written in code blocks is not displayed on the screen. Please resolve the issue.


paragraph spacing is way to large.

The previous version didn't have this problem. After i update the app, the paragraph spacing becomes way to big.

^This is the new updated version.
While below is the old spacing:

Please reduce the spacing.
Thanks in advance

In the previous versions, the app used to clean up the HTML from the source website using Regex whereas now it's using a package sanitize-html for it which is better as it preserves the original formatting and also has many other features.
You can use custom CSS in More > Settings > Reader to reduce the spacing.

how to use it? the CSS thingy i mean.
is there a code or something i can copy?
because, i don't understand on how to use the custom CSS.

is there a code or something i can copy?

No as styles differ from site to site.
You have to check the chapter's HTML in the source website and add the CSS accordingly. Alternatively, I can add an option to reduce multiple new lines to a single one to reduce the spacing.

Alternatively, I can add an option to reduce multiple new lines to a single one to reduce the spacing.
if you can, please reduce it.
dont worry about taking a long time. i can wait.


Sound effects

User description
Placeholder sounds made with the online editor Beepbox (project; cut every 2 seconds, then normalized to -1 dB, then monofied).

[x] Received (as soon as a stop event is detected)
[x] Listening (after the bot's done talking)
[ ] Generating (looping hum until TTS starts; currently disabled due to it hogging the websocket forever, not sure how to fix)
[ ] Error (might attach to some recoverable exceptions, or perhaps when the chat model detects "technical difficulties")

Description

Added sound effects support for voice interaction events such as "listening" and "received".
Sound effects are loaded using librosa, converted to the correct sample rate, and played at appropriate times during voice processing.
Introduced a new setting SFX_VOLUME to control the volume of sound effects.
Updated the Docker Compose file to sync the entire 'prototype' directory from AWS S3 instead of just 'styletts2'.
Minor cleanup in daily_bot.py by removing an unnecessary blank line.

Changes walkthrough
Relevant filesFormatting

daily_bot.pyMinor Cleanup in PyAudioApp Class

clients/daily/daily_bot.py

Removed an unnecessary blank line.

+0/-1

Enhancement

voice.pyImplement Sound Effects for Voice Interaction

openduck-py/openduck_py/routers/voice.py
Imported librosa library for audio processing.  Added a dictionary to store sound effects.  Loaded sound effects from WAV files and converted them to byte format.  Implemented play_sfx method to play sound effects.  Commented out play_sfx_loop method with a TODO note for future implementation.  Added sound effect playback at the end of speech detection and after the assistant's response.

+39/-0

Configuration changes

__init__.pyAdd Sound Effects Volume Setting

openduck-py/openduck_py/settings/init.py

Added a new setting SFX_VOLUME for sound effects volume control.

+1/-0

docker-compose.ymlUpdate Docker Compose S3 Sync Path

docker-compose.yml
Updated the AWS S3 sync command to target the 'prototype' directory instead of 'styletts2'.

+4/-4

💡 Usage Guide
Checking Your Pull Request
Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.
Talking to CodeAnt AI
Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:

@codeant-ai ask: Your question here

This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.
Check Your Repository Health
To analyze the health of your code repository, visit our dashboard at app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.

@codeant-ai ask: sure let's try this, how would you approach play_sfx_loop?


Make the l modulus a global static

l is a core modulus of ed25519 and is therefore typically declared as a
global static variable. This also helps with performance, as keeping it
local to a stack frame could lead to a lot of useless copying.

Thanks!


Photo library button not visible for iPhone 7/iOS 10.2

Hi, I noticed the photo library button (on the bottom right) to use a previously taken picture from the photo library is no longer visible for iPhone 7 and iOS 10.2. This wasn't an issue with iPhone 6S using iOS 10.1. Any ideas?

Hi @zoufishanmehdi! Which version of the Pod are you using?

pod 'TGCameraViewController', ‘2.2.4’ I guess I need to update to '2.2.8'

Could you try updating it and see if it worked? I believe there was a fix in #78 which fixed that issue. 😉

Yep, I will let you know if that fixes things :)

Hi @zoufishanmehdi. Did you try the latest version? Did it work?

Do you recommend updating to '2.2.8' or '2.2.7'? I'm using iPhone 6s but I will be able to test on iPhone 7 tomorrow.

Either one should fix your issue, but upgrade to 2.2.8, the latest :)
On Tue, 10 Jan 2017 at 18:12 Zoufishan Mehdi
<
mailto:Zoufishan Mehdi<EMAIL_ADDRESS>
wrote:

a, pre, code, a:link, body { word-wrap: break-word !important; }
Do you recommend updating to '2.2.8' or '2.2.7'? I'm using iPhone 6s but I will be able to test on iPhone 7 tomorrow.
—
You are receiving this because you commented.
Reply to this email directly,
https://github.com/tdginternet/TGCameraViewController/issues/84#issuecomment-271684667
, or
https://github.com/notifications/unsubscribe-auth/AFCao7f3c-CXs7wyJ-43zs8_hA2Ev4H6ks5rQ-ZIgaJpZM4LdEkO
.

It works for iPhone 7! Thank you for your help :)


net: lwm2m: DTLS x509 certificate based support

Is your enhancement proposal related to a problem? Please describe.
Zephyr currently supports NoSec and DTLS/PSK connection.
Describe the solution you'd like
Add support for security mode 2 (in LwM2M specifications) to support DTLS x509 certificate based.
I plan to implement it : do you see some limitations (max. certificate size for example, etc...?)

Though zephyr uses mbedTLS, it's more enabling the configuration and provide the right credentials from the lwm2m security object (hope that the right lwm2m object name). There should be no other limits as for TLS. AFAIK in LWM2M support for ECDSA is mandatory (at least implicit by RFC 7252), so the certificate size should not be an too hard issue.

@boaks Thanks for your message. You are right : it works with leshan with the block-wise transfer limitation (bootstrap to device when overwriting security data shall not exceed 1024 bytes).
I think that limitation could be a good opportunity to implement EST over CoAP (from my side) to have something fully operational.
I saw "Roadmap to EST" discussion on leshan GitHub, so will try it.

@RomainPelletant could you share the configs needed to get it work with the bootstrap?

@walzsi the following PR works with direct x509 connection : https://github.com/zephyrproject-rtos/zephyr/pull/54420
Due to the blockwise limitation, the bootstrap security object send (for x509 certificate, client and server certificates) could fail if the message containing these 3 opaque data are greater than 1024 bytes.

FYI: pending for review: https://github.com/zephyrproject-rtos/zephyr/pull/59019

@SeppoTakalo solved this issue. Thank you


Take a single picture

####Problem:
I'd like to be able to take a single picture from a GoPro Hero 5.  Ideally, I'd like to use a single URL to take a new picture and get that picture. This is similar to how Axis network cameras work.
Is this possible? I understand how to configure the picture settings and I've been able to browse a listing of pictures but not trigger a new picture and capture it.
####Details:

GoPro Camera(s): Hero 5
Firmware Version: ?
Steps to reproduce: n/a
Happens every time? [Y/N]: Y

Hi,
My python library supports that: http://github.com/konradit/gopro-py-api
gopro.take_photo() takes a photo and returns the URL

Thanks @KonradIT for the quick reply. I found this project of yours. I should be able to use this in a simple python script to meet my needs. I appreciate your help!


feat(web): remove the dependency on package:js in favor of dart:js_interop

Description
Also removing the web package when it wasn't used
Removed tests that were generating mocks since those are not compatible with the new interop and were passthrough tests.
Related Issues
#12466
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[X] No, this is not a breaking change.

Can we get this merged?

@kevmoo Merged, will be part of next release that should be out today :)


Fix Error: Invalid prefix for given prefix length

This patch fixes the error caused by using the IP address instead of the
network address when replacing routes.  The error doesn't affect the
behavior of the system, but it is confusing and a red-herring when
troubleshooting other issues.  The set -x is intentionally left in to
help troubleshoot any future issues, the commands are displayed in the
console and journal in-line with the commands generating the output.
See: Issue 28 for more details and debug logs.
Resolves: #28

This fixes it:
[jeff@multinic-g67p ~]$ sudo journalctl -xu policy-routing.service
-- Logs begin at Thu 2020-12-24 02:52:39 UTC, end at Thu 2020-12-24 02:56:47 UTC. --
Dec 24 02:52:58 multinic-g67p systemd[1]: Starting Configure Policy Routing to behave as a virtual wire...
-- Subject: Unit policy-routing.service has begun start-up
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
--
-- Unit policy-routing.service has begun starting up.
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace default via <IP_ADDRESS> dev eth0 proto static table viaeth0
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace <IP_ADDRESS> dev eth0 proto static scope link table viaeth0
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace <IP_ADDRESS> dev eth1 proto static scope link table viaeth0
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace <IP_ADDRESS>/20 via <IP_ADDRESS> dev eth0 proto static table viaeth0
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace <IP_ADDRESS>/20 via <IP_ADDRESS> dev eth1 proto static table viaeth0
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace default via <IP_ADDRESS> dev eth1 proto static table viaeth1
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace <IP_ADDRESS> dev eth0 proto static scope link table viaeth1
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace <IP_ADDRESS> dev eth1 proto static scope link table viaeth1
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace <IP_ADDRESS>/20 via <IP_ADDRESS> dev eth0 proto static table viaeth1
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route replace <IP_ADDRESS>/20 via <IP_ADDRESS> dev eth1 proto static table viaeth1
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip rule add priority 1000 from <IP_ADDRESS>/20 iif lo table viaeth0
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip rule add priority 1001 from <IP_ADDRESS>/20 iif lo table viaeth1
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip rule add priority 1002 iif eth0 table viaeth1
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip rule add priority 1003 iif eth1 table viaeth0
Dec 24 02:52:59 multinic-g67p policy-routing[1672]: + ip route flush cache
Dec 24 02:52:59 multinic-g67p systemd[1]: Started Configure Policy Routing to behave as a virtual wire.
-- Subject: Unit policy-routing.service has finished start-up
-- Defined-By: systemd
-- Support: https://access.redhat.com/support
--
-- Unit policy-routing.service has finished starting up.
--
-- The start-up result is done.


Issue with windows "run as"

First of all, thanks for your hard work, it's well done.
As IT administrator, I often use the "run as another user" feature, but when I switch to a desktop with a program executed as another user (which isn't the user running the current session), no shortcut works.
Is there a way to detect the shortcuts even in a process which isn't run by the user who owns virtual-desktop-enhancer?

Thanks. I really don't know, to be honest. I didn't develop the underlying library that this program uses to interact with Windows (Ciantic did), so I don't know how ownership of processes affects this program's functionality. I might look into it at some point. Please let me know if you learn anything more about this issue.

i'm an IT admin as well.. if you run the exe as administrator.. your able to switch desktops when you switch desktops.

But it does break the desktop icon in the taskbar

@cruzer619 Can you confirm that this is still the case with the latest version? I am not able to reproduce this behaviour.

nice.. didnt see there is newer version.. downloading now.. going to test it out.

nope still the same thing...  I right click on virtual-desktop-enhancer.exe, run as administrator.. and when I switch desktop via keyboard shortcut, always shows "1".  If I run virtual-desktop-enhancer.exe as a normal user functions as it suppose to.

Reporting here my observations on running WIndows 10 Virtual Desktop Enhancer as administrator or as other user:

If you run the program with a different user from the current one it won't work at all: a tray icon with AutoHotkey's logo will appear, but you won't be able to switch desktops or anything else
As @aNaoy initially reported, if you run Windows 10 Virtual Desktop Enhancer without administrative rights it won't work when a window of a program run as another user is focused (probably because non-administrator users don't have access to each other's processes)
If you run Windows 10 Virtual Desktop Enhancer with administrative rights, but your current user does not have administrative rights (so you effectively run it as another user, but with more privileges):

As reported by @cruzer619 it will work OK even when windows of programs run as other users are focued (probably because administrative rights let you manage other user processes)
However the tray icon will stop updating when switching desktops and the tooltips will not be shown anymore (I am not sure why this would happen)

Lastly: if your account has administrative rights there seems to be no difference between running the program normally and running it with administrative rights (right click -> Run as administrator -> Click "Yes" on the UAC popup): everything will work in both cases (even popups and the tray icon)

Lastly: if your account has administrative rights there seems to be no difference between running the program normally and running it with administrative rights (right click -> Run as administrator -> Click "Yes" on the UAC popup): everything will work in both cases (even popups and the tray icon)

I have an admin account with UAC turned on. But for me, there is a difference. If I run it without administrator privileges then I have to click on the desktop in order for the keyboard shortcuts to work (so I click desktop then press Alt + 2 to go to the second desktop, then click the desktop and press Alt+1 to go back to the first one). If I don't click on the desktop then nothing happens. But if I run as administrator then the shortcuts work all the time.
I also tried to add it to startup and it also does not work. I created the shortcut and added it to startup as described in the installation guide. I tried setting the shortcut to run as administrator and without. When I then login VDE is started, but the icon that is shown is the "H" icon from AutoHotkey I guess. VDE also does not work when pressing the shortcuts.
My windows version is 10.0.15063. Any ideas what might be causing this?
Right now I just removed it from startup and manually start via a shortcut that is set to run as administrator.


Typing error on directives using permissions

when I'm using IsAuthenticated() or HasPerm() my pycharm ide gives a warning

when I dig, directives only receive StrawberrySchemaDirective but permissions based on SchemaDirectiveResolver, maybe we can Union[SchemaDirectiveResolver, StrawberrySchemaDirective]

Had the same issue. Created a PR for the fix, which was to add the missing type annotation for the schema_directive decorator. #16

So strange that pyright types it correctly.
Either way, thanks for the contribution @bpeschier , going to release the fix today with some other changes I'm doing.


feat(rust): Rename Ockam Command MessageFormat to OutputFormat

Current Behavior
ockam command takes -m or --message-format <MESSAGE_FORMAT> options
Proposed Changes
change options to -f or --format <OUTPUT_FORMAT>
change enum name to OutputFormat
change varialb ename to output_format
fixes #2990
Checks

[X] All commits in this Pull Request are signed and Verified by Github.
[X] All commits in this Pull Request follow the Ockam commit message convention.
[X] I accept the Ockam Community Code of Conduct.
[X] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

@edbastelli thank you. The changes look great 👍.
I'm away, will merge later today.

Merged in https://github.com/build-trust/ockam/pull/3074.
Thank you for an awesome first contribution 🥳


Build a library for using the node without topics and/or services (for kinetic)

Build a library from "arm_obj.cpp" and "pid.cpp" to allow the node to be used without Topics and/or services as stated at https://github.com/Interbotix/interbotix_ros_arms/tree/master/interbotix_sdk#overview

Thanks for creating this!
That said, I'm not really a whiz at CmakeLists. Could you explain the advantage of doing this? Currently, if I create a custom ROS package, I can easily include the 'include interbotix_sdk/arm_obj.h' line at the top of my cpp file, and it will compile just fine as long as I create a dependency in my custom ROS package on the 'interbotix_sdk' ROS package (in the find_package directive).
It seems that your method does the same thing but organizes it differently?
Thanks


scripts: fix make docker (#6694)

Signed-off-by: Aylei<EMAIL_ADDRESS>cherry-pick #6694

/run-all-tests


How to best detect "test utility" files?

Let's say we have a set of 3 files in a workspace:

index.js
tests/index.test.js
tests/utilities.js

The tests/utilities.js file is only imported by test files, and so when running knip in non-production mode – and having something like the jest plugin resolve the test file as an entrypoint – it will correctly find that tests/utilities.js is used. But when run in production mode, it is flagged as unused code.
How should we think about configuring Knip to avoid flagging these kinds of files?

Roughly this:
{
  "entry": "index.js!",
  "project": "!tests/utilities.js!"
}

So you'd need to suffix production files! and prefix excluded !files and this can be combined.
Does that help?
Btw going to write new docs/guide about this topic as we speak :)

This will work, but would also mean that if the test file ever updates to no longer reference tests/utilities.js that it wouldn't be flagged as unused, because it's been explicitly ignored.
I think we almost want something like a productionIgnore ability to say that certain paths should be ignored when running in production mode, but still checked in non-production mode, and then we could run both prod & non-prod in CI to be able to catch both "test-setup code" that is unused, and also "project file that is only used in tests"

Tbh I'm not sure whether the !pattern! is also ignored in non-production mode. Did you try?

Either way, if configuration gets messy, one can always resort to a seperate knip --production --config knip.production.json

I just added this page: https://knip.dev/guides/configuring-project-files

Going to close this one after the linked page got another update the other day. Feel free to discuss further, though :)


Update from branch example

Updates
I up[dated the readme file.  Please review and accept.

I probably shouldn't, but okay.  👍


support encode_batch for PhobertTokenizer?

Could you support encode_batch method for a batch of inputs in the near future?
Thanks.

The process of merging a fast tokenizer for PhoBERT is in the discussion, as detailed in https://github.com/huggingface/transformers/pull/17254#issuecomment-1133932067. While waiting for this pull request's approval, if you would like to experiment with the fast tokenizer, you might install transformers as follows:
git clone --single-branch --branch fast_tokenizers_BARTpho_PhoBERT_BERTweet https://github.com/datquocnguyen/transformers.git
cd transformers
pip3 install -e .


Differentiate data source on SRKW occurrence map

Now that you are visualizing an analyzing data from both Acartia and The Whale Museum, it would be edifying and consistent with the Acartia Creative Commons attribution guidance to indicate which data points came from which data source.
One idea for how to do this: use circular markers as you have for Acartia data points, but some other shape or The Whale Museum points.
Additionally, in the Data source notes below the SRKW occurrence map, you could indicate the date range for each data source parenthetically after each source name, e.g. Acartia (2018-present) and The Whale Museum (1973-2020). (BTW: it looks like the link to TWM is relative -- https://salmonorca.fly.dev/whalemuseum.org` -- when it should be absolute (and maybe open in a new tab/window?)...)

Good idea! I'll add that to the to-list. And thanks for catching the bad link to TWM. I'll fix that soon!

Date range for each data source are added to the footnote and the TWM link fixed. To indicate the source of data on the map, I ended up with adding the source as text to the hover text for the dots. Customized symbols can be added but we will have to give up the color scale indicating time of sighting because marker color customization only work on circles.

The mouseover works great for me, @liu-zoe , and with the link in the footer meets the Creative Commons guidance for Acartia points.
It is fascinating to see data gaps in both sources by looking at years like 2019 (when Acartia was just spinning up) and 2020. In some ways they seem complimentary (many TWM points coming in from the Soundwatch boat during days when SRKWs present, I think, whereas most Acartia points are land-based observations or acoustic detections at night.
It's going to be interesting to create similar visualizations of data distribution by data_source for Bigg's (which I think TWM logs?) and other cetaceans, and to contemplate how to fill gaps for each (not to mention for pinnipeds, marine birds, etc.)!

Good to know that it worked! I agree that the TWM and Acartia really do compliment each other and give us a much more robust view of the presence!


AI Image Generator

Describe the project you want to add with tech stack
HTML / CSS / JS
Expected behavior
Generating images with prompt
Screenshots (optional)

Additional context (optional )
Checkout at - https://ai-images-generator-three.vercel.app/

This repo is specifically for frontend challenges. Anything related to AI is not required.


Return a demonstration gif to the readme

Haven't had time to update any projects to the new Cucumber builder, so haven't been able to make an example GIF.
If anybody has a repo using cucumber and wants it to be potentially selected as a demonstration image, ping me.

GIFs have been returned in #128


[FEATURE]: A way to encrypt the data that gets stored in the database and decrypt when reading

Describe what you want
I had a use case where i want to encrypt perticular fields that gets stored on the database
and if the feature is already there then can someone guide me how can i achive with drizzle

What about creating a custom column type?
You could try something like this:
const encryptedText = customType<{ data: string }>({
  dataType() {
    return "text";
  },
  fromDriver(value: unknown) {
    return decrypt(
      Buffer.from(value as string, "hex"),
      Buffer.from(process.env.ENCRYPTION_KEY!, "hex")
    ).toString("utf8");
  },
  toDriver(value: string) {
    return encrypt(
      Buffer.from(value, "utf8"),
      Buffer.from(process.env.ENCRYPTION_KEY!, "hex")
    ).toString("hex");
  },
});

export const account = pgTable("account", {
  id: uuid("id").primaryKey(),
  email: encryptedText("email_digest"),
});

And then you can use the answers from here to help you write your encrypt and decrypt methods: https://stackoverflow.com/questions/6953286/how-to-encrypt-data-that-needs-to-be-decrypted-in-node-js

Thanks to the above comment, I was able to implement encrypted token storage with CryptoJS
const encryptedText = customType<{ data: string }>({
  dataType() {
    return "text"
  },
  fromDriver(value: unknown) {
    return CryptoJS.AES.decrypt(String(value), env.CRYPTO_SECRET).toString(
      CryptoJS.enc.Utf8
    )
  },
  toDriver(value: string) {
    return CryptoJS.AES.encrypt(value, env.CRYPTO_SECRET).toString()
  },
})

Hey, just curious can we get something like this example get added into the docs for common use cases

Thanks to the above comment, I was able to implement encrypted token storage with CryptoJS
const encryptedText = customType<{ data: string }>({
  dataType() {
    return "text"
  },
  fromDriver(value: unknown) {
    return CryptoJS.AES.decrypt(String(value), env.CRYPTO_SECRET).toString(
      CryptoJS.enc.Utf8
    )
  },
  toDriver(value: string) {
    return CryptoJS.AES.encrypt(value, env.CRYPTO_SECRET).toString()
  },
})


GitHub.com/Kagami/go-face

Facing below issue:
github.com/Kagami/go-face
../../pkg/mod/github.com/!kagami/go-face@v0.0.0-20180922080525-863b26a6405e/face.go:25:7: identifier "_Ctype_struct_facerec" may conflict with identifiers generated by cgo
Go Version : 1.14.2
Mac OS : 10.15.4

Duplicate of #47


Do not clean the Xcode build folder if the action specifies 'clean_bu…

…ild' to be 'false'
Without this fix, GIT command cleaned the build folder even if the
action specified 'clean_build' to be 'false'. This fix allows an
individual action to skip clean when 'clean_build' is explicitly
set 'false'.
Pull Request Description
Replace with a description of this pull request. Instructions for adding
projects are available in the README.
Acceptance Criteria
To be accepted into the Swift source compatibility test suite, a project must:

[ ] be an Xcode or swift package manager project
[ ] support building on either Linux or macOS
[ ] target Linux, macOS, or iOS/tvOS/watchOS device
[ ] be contained in a publicly accessible git repository
[ ] maintain a project branch that builds against Swift 4.0 and passes any unit tests
[ ] have maintainers who will commit to resolve issues in a timely manner
[ ] be compatible with the latest GM/Beta versions of Xcode and swiftpm
[ ] add value not already included in the suite
[ ] be licensed with one of the following permissive licenses:

BSD
MIT
Apache License, version 2.0
Eclipse Public License
Mozilla Public License (MPL) 1.1
MPL 2.0
CDDL

[ ] pass ./project_precommit_check script run

Ensure project meets all listed requirements before submitting a pull request.

@swift-ci please test


Several nRF based boards enable both I2C & SPI by default in dts at same MMIO address

The following warnings show cases for various nRF based boards in which 2 controllers at the same MMIO address are both enabled by default.
This warning is generated with the DTC version that is in sdk 0.10-beta4:
96b_nitrogen.dts.pre.tmp:101.22-110.5: Warning (unique_unit_address_if_enabled): /soc/i2c@40003000: duplicate unit-address (also used in node /soc/spi@40003000)
nrf51_pca10028.dts.pre.tmp:107.22-116.5: Warning (unique_unit_address_if_enabled): /soc/i2c@40004000: duplicate unit-address (also used in node /soc/spi@40004000)
nrf52840_blip.dts.pre.tmp:134.22-143.5: Warning (unique_unit_address_if_enabled): /soc/i2c@40004000: duplicate unit-address (also used in node /soc/spi@40004000)
nrf52840_pca10056.dts.pre.tmp:134.22-143.5: Warning (unique_unit_address_if_enabled): /soc/i2c@40004000: duplicate unit-address (also used in node /soc/spi@40004000)
nrf52840_pca10059.dts.pre.tmp:134.22-143.5: Warning (unique_unit_address_if_enabled): /soc/i2c@40004000: duplicate unit-address (also used in node /soc/spi@40004000)
nrf52_pca10040.dts.pre.tmp:111.22-120.5: Warning (unique_unit_address_if_enabled): /soc/i2c@40004000: duplicate unit-address (also used in node /soc/spi@40004000)

@anangl could you please check this one?

@anangl could you please check this one?

OK, I will take a (closer) look.

In #14097 I altered the dts files for the boards mentioned above. This should prevent the warnings. But this is more a workaround then a proper solution for this problem. As @pabigot already mentioned, it may be desirable to enable both peripherals occupying the same block, let's say I2C_0 and SPI_0, and use them both, but of course not at the same time. We should provide such possibility.
A similar problem is with CLOCK and POWER peripherals. They also occupy the same block. And for example for nRF9160, we need to enable DT nodes for both of them, to get their base addresses mapped accordingly in secure or non-secure address space.
Would it be suitable to use something like this:
peipheral_block@5000 {
	reg = <0x5000 0x1000>;
	interrupts = <5 1>;

	clock {
		compatible = "nordic,nrf-clock";
		status = "ok";
		label = "CLOCK";
	};

	power {
		compatible = "nordic,nrf-power";
		status = "ok";
		label = "POWER";
	};
};

and get base addresses for them from the parent node? I guess some addition in the extracting script would be needed to accomplish this. And then we'll need also to provide some smart way of sharing the interrupt handler between the two drivers.
@galak Is there a "standard" way of dealing with such situations?

I see existing practice in Linux, e.g. for nand devices with partitions, so this approach makes sense, disregarding the complexity of passing the information from the tree through the tooling to the kernel.
The parent name should be something like clock_power@5000 (matching nrfx/mdk's identification of the corresponding block's IRQ) since the node names should be informative and must be unique.  Also then uarte0_spim0_spis0_twim0_twis0@8000 for the largest conflict set I've seen so far.
This solution would also be nice because then all the potential personalities are co-located in the dts and a comment can be added to highlight the constraint on simultaneous operation.


⚠ Think twice before giving feedback to this community! ⚠

(Since everyone keeps closing this, I am going to continue to post it until one thread remains visible for discussion, no need to censor)

Lead developer sent this message after a user was giving his honest feedback via this reddit thread.
They also said (I kid you not) -- giving feedback is "bullying",  a "professional whiner", and "ranting". Reduz's passive aggressive tone and insulting behavior is rampant there.  I already terminated my patreon account because of this, and not saying you should either (FLOSS projects) are great, however, for the lead developer to act like this towards people giving feedback.. is not okay!
Let me re-iterate, not saying that you should stop giving feedback, but I just want this message to be seen across github and reach as far as possible. Because this kind of communication and how the lead developer treats a regular Godot developer is not right at all. I've never seen this behavior from a FLOSS project lead developer.  It's a shame, because the guy giving feedback is just a random person anyway, and the way the project managers and lead developers took it so personally like it was an attack (when it wasn't), means the lead developers/contributors are possibly not viewing your issues with the right mindset.
Also, after reading the last paragraph from this post by akien

From here on, we will avoid responding to xxx's threads on the various Godot communities, and focus instead on the huge amount of constructive feedback and bug reports that we get from other users. This will likely overlap with some of the issues mentioned above, which will thus be fixed eventually, but we won't accept being bullied into working on them now when we have different priorities. Users are free to disagree with our priorities, but in the end the contributors are the ones making the call.

He's saying they will now favor/prioritize issues based on the user reporting it, and not the issue itself. (Extreme prejudice), which will not be good for the project because the issue should be irrelevant to the user reporting it. Just because if a user who has a snotty attitude that is reporting a huge client crash error, doesn't mean that client crash error should not be taken lightly.
He's also stating that anyone giving hard feedback is now "bullying" them. Which is not true whatsoever.
Just wanting to post this to get the word out there, hopefully some things change. @reduz @akien-mga  This is just unbelievable :(

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain drentsoft.com has exceeded the max emails per hour (501/500 (100%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o11.sgmail.github.com ([<IP_ADDRESS>]:57908)
by wdc002.hawkhost.com with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89)
(envelope-from<EMAIL_ADDRESS>id 1eMTCl-000d36-DL
for<EMAIL_ADDRESS>Wed, 06 Dec 2017 01:29:15 -0500
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=8MbKjrdLeHEMWTQVOj6lzRsx5Ik=; b=LgmQEbK6GdGJGaYY
jSSHzDNVrIg9A1kjsvn8bfoslTY7gNpZCYdXNT3yJ9gFw1qkYY5VUcPDTgKAQOu1
lCuiv6qZT9u4ZjpRIyJEei2KqiCpR6Oa6RnJ5GbXPmOpYa55qtBp8s1jhOn5cB73
jlRLNjWw1li4Mvl1I51De7MIJFU=
Received: by filter0616p1iad2.sendgrid.net with SMTP id filter0616p1iad2-3766-5A278E3A-4C
2017-12-06 06:29:14.852702798 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0001p1iad1.sendgrid.net (SG) with ESMTP id jPs4jvxJQvqCq4KPZM2XHA
for<EMAIL_ADDRESS>Wed, 06 Dec 2017 06:29:14.713 +0000 (UTC)
Date: Wed, 06 Dec 2017 06:29:14 +0000 (UTC)
From: Rémi Verschelde<EMAIL_ADDRESS>Reply-To: godotengine/godot<EMAIL_ADDRESS>To: godotengine/godot<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [godotengine/godot] ⚠ Think twice before giving feedback to this community! ⚠ (#14028)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a278e0cd4454_8e73fcdee448f3080248";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: akien-mga
X-GitHub-Recipient: drentsoft
X-GitHub-Reason: subscribed
List-ID: godotengine/godot <godot.godotengine.github.com>
List-Archive: https://github.com/godotengine/godot
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/ABcz5evrRny7O4zEAH4Ze6F8ICypjf8Wks5s9jQMgaJpZM4Q3aNp
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: ylpB2MtFfxGgH7IUUD8hAP5dnG/dtmihe/pcfBkwtjLreBm0qgI0LzkaGM0Gz5OR6KM+AuibteFv0M
xONVLrViO6IH3kxlHkKAvDVwS7gIAfyrwLzglzgKnSWSWtcZifYmBa6RU0rSbqdPsQivCUuwGy5l/u
kYXd7CJ9AfP4AANJAT7kNU9W1GIunMGcrLn9km27VvsHWAu2RoDpI9ItJNAhQGO5IuEGUyNvwd3d3t
w=
----==_mimepart_5a278e0cd4454_8e73fcdee448f3080248
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #14028.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/godotengine/godot/issues/14028#event-1374168934
----==_mimepart_5a278e0cd4454_8e73fcdee448f3080248
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #14028.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5a278e0cd4454_8e73fcdee448f3080248--


React-router v3 compatibility

This "fix" feels like a hack, but it works well to prevent the error Uncaught TypeError: Cannot read property 'key' of null when using this library with react-router v3. All tests that passed without the change still do.

That's really not the point. The incompatibility with history v3 is because history v3 has different semantics around the initial location – listen doesn't call the listener immediately with the initial location. It's not about hiding getCurrentLocation, or I wouldn't have added that check to React Router in the first place.

Among other things, if you want to do this correctly, you must not invoke the listeners with the initial location in listen.
There's no point in making this library compatible with both history v2 and history v3, either – they're incompatible peer dependencies at any rate.

Okay, on closer reading, this is close enough that I'll re-open the PR, but this PR needs to actually follow history v3 semantics before it's mergeable, and it needs to not fail the coverage check as well.

Sorry I haven't found the time to work on this, but I hope this weekend I'll jump back on it. Still a bit confused at what you mean by follow history v3 semantics, could you clarify?

listen should not immediately call back with the current location

#107 supersedes this by removing the history integration entirely. There's no need to handle any of this just to be compatible with React Router v3, though bugs in history relating to hash history will still prevent scroll management from working correctly in such cases.


METAsrc

Can u add METAsrc as a new source? I think is a good one
http://www.metasrc.com

Related. https://github.com/dustinblackman/Championify/issues/261

Metasrc would prefer Championify not scraping them. Closing.


Ensure authentication parameters are provided to WFS 3 requests.

auth and password were set but not used with WFS 3 in OWSLib.

Thanks @bosth !


⚠ Bofrost.de has degraded performance

In fff7f30, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1570 ms

Resolved: Bofrost.de performance has improved in 5a8e562 after 33 minutes.


Support install product value of "bitbucket"

The Forge CLI supports the following products via the install command:
-p, --product [product]          product (Jira, Confluence, Compass, Bitbucket)

https://developer.atlassian.com/platform/forge/cli-reference/install/
The install command for the nx-forge plugin throws the following error when using 'bitbucket' as a product value:
Property 'product' does not match the schema. 'bitbucket' should be one of jira,confluence,compass.

Can work around for now by opening a terminal in the built dist directory of the forge app and doing the raw forge CLI command.

Thanks for the report @zjkipping. This certainly needs to be fixed. The install executor may actually be one that we can remove because it does not do anything nx-forge specific. Regarding your workaround, you can use the Forge executor to run any Forge CLI command in the output directory.


CompletableTransformer is removed in rxjava 1.2.0

CompletableTransformer : renamed to Completable.Transformer

Oh neat, so there's no way for the library to be backwards compatible. :P

They were able to make that change since Completable was marked as Experimental

I know, I think I've learned a lesson here not to support experimental APIs in a more stable lib.
The problem is that people wanted support for completable/etc...

I'm not 100% sure, but I think that even if you're not using Completable.Transformer it's used in the source code of RxLifecycle. So unless you're proguarding all that stuff out, the compiler will complain that RxLifecycle is referencing classes that don't exist.

Confirming: I don't use CompletableTransformer, and RxLifecycle 0.8.0 compiles and works for me with RxJava 1.1.9.


128X64 MatrixPanel Problem (not resolved)

Hello, when I apply the code attached below, some parts are blank like in the picture. How do I modify or wire the code?

// 1) Include key virtual display library
// VERSION 3.0.7
#include <ESP32-VirtualMatrixPanel-I2S-DMA.h>
#include <Fonts/FreeMono9pt7b.h>
// 2) Set configuration
#define PANEL_RES_X 128 // Number of pixels wide of each INDIVIDUAL panel module.
#define PANEL_RES_Y 64 // Number of pixels tall of each INDIVIDUAL panel module.
#define NUM_ROWS 1                     // Number of rows of chained INDIVIDUAL PANELS
#define NUM_COLS 1                     // Number of INDIVIDUAL PANELS per ROW
#define PANEL_CHAIN NUM_ROWS *NUM_COLS // total number of panels chained one to another
/* Configure the serpetine chaining approach. Options are:
CHAIN_NONE,
CHAIN_TOP_LEFT_DOWN,
CHAIN_TOP_RIGHT_DOWN,
CHAIN_BOTTOM_LEFT_UP,
CHAIN_BOTTOM_RIGHT_UP,
CHAIN_TOP_LEFT_DOWN_ZZ, /// ZigZag chaining.
CHAIN_TOP_RIGHT_DOWN_ZZ,
CHAIN_BOTTOM_RIGHT_UP_ZZ,
CHAIN_BOTTOM_LEFT_UP_ZZ
The location (i.e. 'TOP_LEFT', 'BOTTOM_RIGHT') etc. refers to the starting point where
the ESP32 is located, and how the chain of panels will 'roll out' from there.
In this example we're using 'CHAIN_BOTTOM_LEFT_UP' which would look like this in the real world:
Chain of 4 x 64x32 panels with the ESP at the CHAIN_TOP_LEFT_DOWN:
+---------+---------+
|    4    |    3    |
|         |         |
+---------+---------+
|    1    |    2    |
|  (ESP)  |         |
+---------+---------+
*/
#define VIRTUAL_MATRIX_CHAIN_TYPE CHAIN_TOP_RIGHT_DOWN
// 3) Create the runtime objects to use
// placeholder for the matrix object
MatrixPanel_I2S_DMA *dma_display = nullptr;
// placeholder for the virtual display object
VirtualMatrixPanel *virtualDisp = nullptr;
/******************************************************************************

Setup!
******************************************************************************/
void setup()
{
delay(2000);
Serial.begin(115200);
/******************************************************************************

Create physical DMA panel class AND virtual (chained) display class.
******************************************************************************/

HUB75_I2S_CFG mxconfig(
PANEL_RES_X, // module width
PANEL_RES_Y, // module height
PANEL_CHAIN  // chain length
);
// mxconfig.driver = HUB75_I2S_CFG::FM6126A;     // in case that we use panels based on FM6126A chip, we can set it here before creating MatrixPanel_I2S_DMA object
// Sanity checks
if (NUM_ROWS <= 1){
Serial.println(F("There is no reason to use the VirtualDisplay class for a single horizontal chain and row!"));
}
// OK, now we can create our matrix object
dma_display = new MatrixPanel_I2S_DMA(mxconfig);
// dma_display->setRotation(1);
// let's adjust default brightness to about 75%
dma_display->setBrightness8(192); // range is 0-255, 0 - 0%, 255 - 100%
// Allocate memory and start DMA display
if (not dma_display->begin()) Serial.println("****** !KABOOM! I2S memory allocation failed ***********");
// create VirtualDisplay object based on our newly created dma_display object
virtualDisp = new VirtualMatrixPanel((*dma_display), NUM_ROWS, NUM_COLS, PANEL_RES_X, PANEL_RES_Y, VIRTUAL_MATRIX_CHAIN_TYPE);
// virtualDisp->setPhysicalPanelScanRate(NORMAL_ONE_SIXTEEN);
// virtualDisp->setPhysicalPanelScanRate(FOUR_SCAN_16PX_HIGH);
// So far so good, so continue
virtualDisp->fillScreen(virtualDisp->color444(0, 0, 0));
// virtualDisp->drawDisplayTest(); // draw text numbering on each screen to check connectivity
// delay(1000);
virtualDisp->setTextColor(virtualDisp->color565(0, 0, 255));
// Red text inside red rect (2 pix in from edge)
virtualDisp->drawRect(0, 0, virtualDisp->width(), virtualDisp->height(), virtualDisp->color565(255, 0, 0));
// White line from top left to bottom right
// virtualDisp->drawLine(0, 0, virtualDisp->width() - 1, virtualDisp->height() - 1, virtualDisp->color565(255, 255, 255));
// virtualDisp->drawDisplayTest(); // re draw text numbering on each screen to check connectivity
virtualDisp->setTextWrap(false);
virtualDisp->setCursor(1, 1);
virtualDisp->print("ABCDEFGHIJ");
virtualDisp->setCursor(1, 8);
virtualDisp->print("1234567890");
virtualDisp->setCursor(1, 16);
virtualDisp->print("ABCDEFGHIJ");
virtualDisp->setCursor(1, 24);
virtualDisp->print("1234567890");
virtualDisp->setCursor(1, 32);
virtualDisp->print("ABCDEFGHIJ");
virtualDisp->setCursor(1, 40);
virtualDisp->print("1234567890");
virtualDisp->setCursor(1, 48);
virtualDisp->print("ABCDEFGHIJ");

}
void loop(){
}

@mrfaptastic
Sorry, Can you help me?

Single geometry of the matrix like "128x64" is not enough to manage it to work/ we need a more detail. Is it a scan32 matrix? Or scan 16 or scan8?
What  the chips are used as drivers?

@board707
It is 1/32 scan and uses ICND2038S chip

Can you please try with the simpletestshapes sketch?

@mrfaptastic
Apply the code below to make it look like the picture.
I don't know if this is displayed well, but it is displayed normally.
But in BItMapIcons //mxconfig.gpio.e = -1; When I uncomment // Assign a pin if you have a 64x64 panel and apply, I get a Compilation error: 'mxconfig' does not name a type error.

Looks like there's a bug with the VirtualMatrixPanel class when using it with a single panel (which there is no need to use this class for), which I'll need to investigate.
Now using the underlying library to draw directly to the panel - looks like it works for me. I can't understand your gpio.e compile error comment unless you provide the actualy sketch code.


TLF-29-add bullet point to accessibility statement

Relates TLF-29
What

Removed hof default accessibility statement and replaced with overriden view of accessibility page to bring govuk-list--bullet so there are bullet points.
Added backlink so user can return to main form

Can you add what changed in your commit message


OVS module loading failure for RHEL and CentOS Nodes

Describe the bug
modprobe of initContainer will fail on RHEL or CentOS Nodes.
To Reproduce
Deploy Antrea on RHEL or CentOS Nodes, and initContainer will fail.
Expected
Loading OVS module by initContainer for RHEL and CentOS Nodes.
Actual behavior
modprobe in the initContainer returns an error about binary execution.
Versions:
Please provide the following information:

Antrea version v0.0.1

@jianjuns sorry for reviving this old issue, but I have seen some issues caused by mounting /sbin/depmod to the initContainer. Typically this happens when the /sbin/depmod (or rather kmod since /sbin/depmod is just a symlink to kmod) on the host / Node depends on some libraries which are not available inside the container. This can happen when the OS distribution on the host uses a different version of kmod than the container OS distribution.
Is there any chance you could share the error message that led to the decision of mounting /sbin/depmod? I tried to reproduce on EC2 using a CentOS instance (see version below) but without success:
[centos@ip-172-30-0-73 ~]$ cat /etc/centos-release
CentOS Linux release 7.7.1908 (Core)
[centos@ip-172-30-0-73 ~]$ docker run --cap-add SYS_MODULE -v /sbin/depmod:/sbin/depmod -v /lib/modules:/lib/modules -ti antrea/antrea-ubuntu:v0.12.0 modprobe openvswitch
[centos@ip-172-30-0-73 ~]$

Unfortunately I lost my env for testing this. Could you try with the old Antrea image (like 0.1.0) in case it is no more an issue with Ubuntu 18.04?

I also feel option 2 is less risky if it can solve all current issues.
I don't know if mounting the whole host FS into container is common, not sure if there is circulation in path or any other issue.

I also feel option 2 is less risky if it can solve all current issues.
I don't know if mounting the whole host FS into container is common, not sure if there is circulation in path or any other issue.


Ehcache.putAll can report successful CacheWriter write as failure via BulkCacheWriterException

Ehcache.putAll can report a successful write through CacheWriter.writeAll as a failure if multiple entries are presented to CacheWriter.writeAll and writeAll throws an exception other than BulkCacheWriterException after the CacheWriter successfully records one or more entries.  Should a different exception indicating uncertain results be thrown instead of the somewhat inaccurate BulkCacheWriterException that is now thrown?

Discussion during [Ehcache weekly dev meeting - 8 January 2015] (https://plus.google.com/events/c7lgves3acscvgr9l075240rrmg) led to the conclusion that (at least) all declared failed keys should be invalidated in Store so, whether or not the write via CacheWriter was successful, the next Ehcache.get access to any value declared failed will yield the value from the store-of-record.
Converting this issue to a BUG.

Working on this, I realised a number of tests have an invalid state from the start. That is the CacheStore knows mappings that are unknown of the LoaderWriter.
I have updated all these cases to start with a valid state, that is all Cache Store mappings are known to the LoaderWriter - which may in turn contain more mappings.


Custom element not found: digital-clock

Just installed, created a custom card, and pasted the following. What am I missing?
type: 'custom:digital-clock'
dateFormat:
weekday: 'long'
day: '2-digit'
month: 'short'
timeFormat:
hour: '2-digit'
minute: '2-digit'

Your indentation is wrong.
type: 'custom:digital-clock'
dateFormat:
  weekday: 'long'
  day: '2-digit'
  month: 'short'
timeFormat:
  hour: '2-digit'
  minute: '2-digit'

But this can't be the reason for custom elemente not found. Please restart your browser or if it is on your mobile phone, please clear the cache of your app and stop it. After this you can open the app again and the element should be available

Please also refer to issue #8


Modfied Access token to account for special slack options

Needed to adjust Access Token, to account for Slack's Webhook responses.

Keeps failing on the mock access token, not sure what to do about it. Any suggestions?

Hey @PeterMartinez, thanks for the contribution. I think we could simplify this a little. The League touched uplon this issue #377.
I've recently done the same for my stripe provider cebc78. Can you amend the commit and see if this is easier?


Create regular.md

Added a table of regular posts on twitter
Inspired by @pradumnasaraf's Open Source with Pradumna

Awesome. Finally, you made it 🎉.

Awesome. Finally, you made it 🎉.

Haha yes, I was planning to do this since we talked. Its a great way to keep track of things.


Cant find devices after alexa update

Hai Sir .. After alexa's new update alexa can't find devices, But raspberry pi is responding to alexa device search.. did they change the xml file .. ? Help me out Please..

It continues to work for me. I have a first generation Echo, and it's possible that other Echo devices are different.

I replaced a different xml file of belkin international and modified its
model number now it is working fine...
On Tue 3 Apr, 2018, 5:34 AM makermusings<EMAIL_ADDRESS>wrote:

It continues to work for me. I have a first generation Echo, and it's
possible that other Echo devices are different.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/makermusings/fauxmo/issues/31#issuecomment-378085986,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aj1eRunVGp0nIZE3O5TxgR1BbuWFDSsoks5tkryhgaJpZM4TBUv-
.

Hi is it possible I could see a copy of this different XML file as I can't get it to work for me?

No I just changed the model number in xml to 1.0

Oh ok..thanks...i.will.give it a go

Dear all, I had the same problem.
I changed
if data.find('M-SEARCH') == 0 and data.find('urn:Belkin:device:**') != -1:
to
if data.find('M-SEARCH') == 0 and data.find('upnp:rootdevice') != -1:
and it works again. However this is most likely quite dirty hack?
Cheers
Alecks


[Bug] Styling of link with icon

[Bug] Styling of link with icon

The underline should not extend under the icon. This is described in the Design System in the standard-link-with-icon section and in the DS guidelines for links with icons.
For the sample image we should use the download icon shown below instead of the document icon. The words can be "Download file"

Current behavior

Expected behavior

Relavent links

Guidelines for icons links

https://github.com/cfpb/design-stories/pull/152


Selecting correct Security Group by name for accounts with multiple VPCs

It is possible to specify security groups by name or id. While the specification of ids works correctly for accounts containing multiple VPCs, the specification by name has a problem.
It somehow randomly(?) maps the name to a valid security group in the given account. But it does
not check whether the selected security group belongs to the VPC the given network belongs to.
As a result typically the VM creation fails if there are multiple VPCs using the same security groups, because the selected security group id is not valid for the network the VM is created for.
We are using IDs now, but the resulting manifests are quite unreadable. It would be much better
to be able to use security group name instead.

@mandelsoft makes sense though i would also recommend to set your aws key permissions to only allow single vpc visibility.

@mandelsoft I think this issue was fixed a while back when dealing with a related bug. Happy accident! Looks like we now only look at the security groups that belong to the VPC that contains the target subnet: https://github.com/cloudfoundry-incubator/bosh-aws-cpi-release/blob/d075d1cb24cf742c0d867094794584ce0b25cbb5/src/bosh_aws_cpi/lib/cloud/aws/security_group_mapper.rb#L41. Feel free to re-open if you're still seeing this.


Content generated by asyncAppend disappears when sibling element is updated.

Live example with code
https://stackblitz.com/edit/lit-element-example-a4bpvn
Expected Results
Content added through asyncAppend is present after calling the render method when isLoading has a different value.
Actual Results
Content added through asyncAppend disappears after calling the render method when isLoading has a different value.

@unrealprogrammer  That's because you actually rerender it calling update() again (and again). Just comment out all update()s in onMore. Every your update call recreate the generator instance.
Though I still not quite sure how to make your when work. It seems like all right, but still not changes when isLoaded got changed. Keep the issue - maybe someone here can correct it.

As @cyper8 said, the problem here is that you call contentEmitter.generator() inside the update function, which creates a new async iterator. So asyncAppend is given a new async iterator each time it is called, and so it will clear away whatever the previous async iterator returned.
Move the call out and store the result in a variable so that you only ever create one async iterator:
const contentEmitter = contentEmitterFactory();
const asyncIterator = contentEmitter.generator();

let isLoading = false;
function onMore() {
  isLoading = true;
  update();
  contentEmitter.requestContent().then(() => {
    isLoading = false;
    update();
  })
}

function update() {
  render(html`
        ${
          asyncAppend(asyncIterator, objs =>
            objs.map(o => {
              return html`

${o.title}

${o.description}

              `;
            })
          )
        }
        ${
          when(
            isLoading,
            () => html`
              loading...
            `,
            () => html`
              <button @click="${onMore}">more content</button>
            `
          )
        }
      `,
  document.getElementById("test"));
}

update();

@mariusGundersen, now i see! Thanks for clarification!

What a silly mistake of mine.  It makes perfect sense.
Thank you very much for explaining it 🙏


incus-user: Handle missing project but existing network

I unintentionally deleted my user project, user-1001, via the UI web interface.  Reading comments online I thought it would get recreated with the next command, but not so.
Logged in as 'user' and using a command to list the images, gets an error as:
incus image list
Error: Get "http://unix.socket/1.0?project=user-1001": read unix @->/var/lib/incus/unix.socket.user: read: connection reset by peer
I tired creating the project via the web interfaces, but that doesn't fix it either, nor does trying other commands such and launching a new container.
Possibly related, the web interface on the bottom left, just above 'Documentation' has an entry like 'dddf0b0...', which is not clickable, it was not there before.
I tried doing an update, but no love, now at version 6.5:
incus --version
6.5
Any suggestions on how I can fix this?
Bill

Make sure to also delete the user bridge network. Incus may be failing to create the project due to the network already existing.

Excellent, I deleted the user bridge then the user profile was created as expected.
FYI, I noted minor differences from before (according to my memory):

the project was bill-1001 and now it is user-1001,
the default network has ipv4 and no ipv6, but user-1001 has both (not sure what it was before), and
launching a container results in a container with an ipv6 address and no ipv4 address.

BTW, I still have the weird entry above 'Documentation' in the GUI, it looks like a lock and 'dddf0b0..' that I don't think was there originally.
Altogether things are working again and I thank you.
Bill

A little more info:  I set ipv6 to 'none' similar to the default network, now when I create a container it has neither an ipv4 nor a ipv6 address.
Bill

My bad, I forgot I'd setup a firewall along the way (this is a new installation), I found your previous answer here:
https://discuss.linuxcontainers.org/t/container-does-not-get-ipv4-address/19735
Bill

the project was bill-1001 and now it is user-1001,

It was always named user-XYZ, we don't want to risk a username change breaking things.

the default network has ipv4 and no ipv6, but user-1001 has both (not sure what it was before), and

We don't look at any existing default network when creating our per-user networks, so they all get the Incus defaults which are IPv4 and IPv6 connectivity.

launching a container results in a container with an ipv6 address and no ipv4 address.

That's likely a firewall issue as you've noted in a follow-up comment.

I'll look at fixing the incus-user handling so it can deal with an existing network.
Btw, in the future, note that Incus issues are better reported at https://github.com/lxc/incus/isuses
This repository is mostly about the packaging that's done for Debian and Ubuntu.

OK, will report issues in the future, thanks again for you help.
Bill

Keeping this open as it's not been addressed yet


fix: Fix blob store invariant

copilot:all

Current dependencies on/for this PR:

main

PR #3835   👈

This comment was auto-generated by Graphite.


dbt-getting-started: materialized tests

Adjusted to demonstrate using dbt test --store-failures to create materialized views. (etl pipeline alerting)
Also included an alert view to show how easy it is to create a canonical alert template to be consumed by something downstream.

Would this be better suited as a standalone demo than an add-on to the getting started?

I'd really like to include continuous testing here, since we do it differently and folks will probably want to make materialized views from tests even as they are developing to help see what is happening.

Yeah, let's do it; I'll take another look soon so we can merge this in!


Restaurant Menu Page

Change Colours, remove image?

Vertically align text over image

Waiting for Jonny to send altered text.

Restaurant Description

Font-colour on Restaurant Menu Page


Styling links as buttons using role="button"

Hi Andy, while I was working on building a theme around mvp, I realised that styling links as buttons changes their role, I assume your aware that when elements are used in a manner not original to their specs that we can use the role="" attribute to indicate to screen readers, and validators what the presentation is intended for that element. So, I had a look at styling anchor's used as buttons using the role="button" attribute rather than the current styling using a b or a strong (and seeing as the buttons are already having their font-weight set to bold), they look exactly the same, also, using the role attribute helps indicate in the html markup exactly what that element is doing or being used for.
a[role="button"],
button {
    border-radius: var(--border-radius);
    display: inline-block;
    font-size: medium;
    font-weight: bold;
    line-height: var(--line-height);
    margin: 1.5rem 0 0.5rem 0;
    padding: 1rem 2rem;
}

Italic Link Button
Bold Link Button →

As you can see, we can do away with the extra classes, but still use them if we wish to change the text, and using the role attributes makes sure our markup is valid and represented correctly.
Anyway, some food for thought.
I also had an idea for my own purposes and needing things like for e.g. a jumbotron (ala Bootstrap), we can use the role attribute on a section element, with it's siblings being styled accordingly without the need to add classes, plus that would also indicate what that block of code does within the markup.

Just been doing some more research into using custom role values, they have set values, this means if we use our own custom values the document they are in will no longer be valid. However, we could use data- attributes for the same purpose as they can be styled. And where we are changing the behaviour of elements such as  to behave as though it's a <button/> we can use role="button" to style it how we want.

Thanks @DiemenDesign, this is a neat feature that I wasn't aware of.
I like the idea of role="button". If we're moving away from  it would be nice to ditch  as well, but I'm not sure how to do that without introducing a custom attribute like data-, which I'm not in favor of since the goal is to have the styling affect only standard HTML tags and attributes. If we have to use  anyway, then it's just more typing for not that much better markup (since most MVPs won't need the accessibility benefits of role=).
But, if there's a slick way to create an outline button with one standard HTML tag then it could be a compelling solution. Any ideas around that?

It is a tricky one, the role="button" on anchors when changing it's behaviour to a button representation makes sense to me for accessibility in particular. And similar could be used for other elements when used with different behaviours as well. For keeping an outline button as per the example page, we could seperatly style the .a[role="button"]{} like what is already happening with button{} style directive to adjust it's background colour.
The data- attribute is a different beast. For e.g. in my CMS I'm using something like
 where I want to restyle how the  elements look, but don't want to affect how the original (to mvp.css at least)  look on the page. Maybe it's something that people can be made aware that can be done to achieve differences to the look of elements like colour for e.g., without altering too much of the original mvp.css stylesheet.

@DiemenDesign - I was reading more about role="button" here: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/button_role and it says:

The button role should be used for clickable elements that trigger a response when activated by the user.

A button is a widget used to perform actions such as submitting a form, opening a dialog, cancelling an action, or performing a command such as inserting a new record or displaying information.

While a link could be considered "a response", it sounds like it's more for tags that simulate a button action rather than tags that look like buttons.
So, even though role="button" is an interesting option I'm inclined to leave the button styling as  and  since they appear to be more semantically correct. I could see this going either way though and happy to revisit it though if there's a popular consensus to the opposite.
Thanks again for proposing this as I didn't even know about these attributes before.


fix: dropdown property control

Description

Allow 0 value for Dropdown Property

Fixes #10576
Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: fix/dropdown-property-control

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
55.46 (0)
37.06 (0)
35.32 (0)
55.85 (0.01)

:green_circle:
app/client/src/components/propertyControls/DropDownControl.tsx
88.89 (0.43)
63.64 (0)
71.43 (0)
87.5 (0.54)

/ok-to-test sha=679e788

Tested this PR and working as expected


Make API errors better

These are changes that were intended as part of #3254 but while necessary I decided it would be a feature creep to bundle these with the original PR.
These are the first steps to implementing #3482

This is ready for review.

Thanks @borisyankov ! Comments below.
-  code?: ApiErrorCode,
+  code: ApiErrorCode,

Huh interesting. What's the reason for this change?
A git log --stat -p -S ApiErrorCode finds that when you originally added this type in 060a7762e, we wrote
The response often contains a program-readable `code` constant,
but not always.

Is there a reason to no longer believe that's true?

+/**
+ * Response from the event queue in case of an error.
+ *
+ * @prop queue_id - String uniquely identifying the queue. Available in addition
to
+ *   the usual error message information.
+ */
+export type ApiEventQueueResponseError = {|
+  ...ApiResponseError,
+  queue_id: string,
+|};

Cool, good to make distinct. This should specify the appropriate code -- it's a lot like events, or for that matter Redux actions, in that one property's value identifies what other properties and types to expect.

    api: Add custom errors classes 'ApiError' and 'EventQueueError'

    We are currently hackily injecting some API data to a standard
    Error object. This works fine but creates issues with Flow, as
    it rightly complains of misuse.

    The right approach is a custom class, descendent of the Error
    object and enhanced with the needed properties. This used to
    be tricky because of the Babel / React Native build pipeline
    but improvements in the past year have made this possible.

Ah nice! Glad to see this cleaned up. :-)

What were the build pipeline changes that made this possible? Or: what would go wrong before?

The jsdoc on BaseApiError appears to be incomplete. :wink:

Hmm, I feel like EventQueueError should be a subtype of ApiError.
I guess an ingredient in that is that ApiEventQueueResponseError should be a subtype of ApiResponseError.
And in turn what that's telling me is that ApiResponseError shouldn't be an exact object type. Which makes sense -- the reality is that API error responses may have additional fields. It's only when you know a specific error code (like code: 'BAD_EVENT_QUEUE_ID') that you can know what other fields will be present (in that example, queue_id: string and no others) and have an exact type.

    Replace 'makeApiError' with a custom error class 'ApiError'

Cool. I think this is best squashed into the parent commit, which defines ApiError -- it actually makes the parent easier to understand, by showing that choices like having those two fields with their particular names have just been moved from this other, previous code.

+export const isErrorBadEventQueueId = (e: Error | ApiError | EventQueueError): boolean =>
+  (e instanceof ApiError || e instanceof EventQueueError) && e.data.code === 'BAD_EVENT_QUEUE_ID';

Yeah, it feels odd having these ApiError | EventQueueError etc. I think this is in part a symptom of having the one of them not be a subtype of the other, when it naturally ought to be.

Also it's confusing that... if I'm reading this right, we never actually make an EventQueueError? Those only exist in tests. Instead, we make ApiError that just happens to have .data.code === 'BAD_EVENT_QUEUE_ID'.
Ideally when we see one of those, we'd make it a type that matches, and tells Flow that there'll be a queue_id: number. But we don't actually use queue_id, so it'd be fine not to have this mechanism...
but we definitely shouldn't have this extra type, and then not use it when we're handling the thing it's obviously intended for. That's super confusing. If we're not going to wrangle Flow so as to actually use it, we should delete it and just have ApiError.

-  code?: ApiErrorCode,
+  code: ApiErrorCode,

The response often contains a program-readable `code` constant,
but not always.

Is there a reason to no longer believe that's true?

Yeah, and that reason was I thought I was wrong originally, but actually not.
I did confirm that is still true, changed the commit to not do the change and added a line to clarify that indeed not all error responses include the 'code' value.

What were the build pipeline changes that made this possible? Or: what would go wrong before?

From what I remember Babel 6 did not allow extending built-in types. (and maybe there was some React Native-related issue, maybe not)
A plugin like this could have been used though:
https://github.com/loganfsmyth/babel-plugin-transform-builtin-extend
But with the current dependencies, everything just works as expected.

I guess an ingredient in that is that ApiEventQueueResponseError should be a subtype of ApiResponseError.

It should but I have problems expressing the only difference between them is the data thrown, as the implementation is the same. EventQueueError extends ApiError<ApiEventQueueResponseError> didn't work, as ApiError is not generic.
I expected something like this to be the solution:
export class ApiError<T> extends BaseApiError<T | ApiResponseError> {}

But that is not valid Flow. Would be great if you know of a solution to this.

It's only when you know a specific error code (like code: 'BAD_EVENT_QUEUE_ID') that you can know what other fields will be present (in that example, queue_id: string and no others) and have an exact type.

That is not entirely correct. We can predict the response will be of that type if it is thrown by the pollForEvents API call that is targeting the /events endpoint. But that is tricky to express, and will definitely require some refactoring that might make the code a bit too messy for little benefit.

Superseded by #3639.


Small fix to session.isActive() inside verify-request.ts

First of all big thanks for creating this repo 👍 It's a shame Shopify is deprecating the original.
While trying out this repo I encountered an error, "session.isActive() is not a function". I use Mongodb as my custom session storage and realized isActive() does not save into the session object (in my case I'm using JSON.stringify() to save my session object, as per shopify-node-api's documentation, which doesn't serialize methods) and so will be undefined when called after loading the session.
The method is being called here.
// original
if (session) {
  // Verify session is valid
  if (session.isActive())
...

I replaced this method call with the actual business logic from Shopify's own code used for checking if the session is active, which removes the responsibility from the session object.
// fix
if (session) {
  // Verify session is valid
  if (
    Shopify.Context.SCOPES.equals(session.scope) &&
    session.accessToken &&
    (!session.expires || session.expires >= new Date())
    ) {
    ...

Thank you for noticing this. In my code session is an instance, so I didn't notice this.
I prefer to use Session.isActive() because it's simpler and could help prevent bugs, so I have a different idea to solve this. I think we can create a session instance by renaming session to sessionData and doing someone like session = Session.cloneSession(sessionData, session.id) after loading the session to create an instance, and then we can use the .isActive() method on it later.
I will go ahead and implement that now.

Ok, the issue should be resolved in v1.0.4. I went ahead and did it a little differently than this PR (as explained in my previous comment) so I will be closing this, but thank you for alerting me to the issue.

Ok, the issue should be resolved in v1.0.4. I went ahead and did it a little differently than this PR (as explained in my previous comment) so I will be closing this, but thank you for alerting me to the issue.

Cool! Thanks for the quick update. I'll go check it out.


Redundant data download in classification and testing

The artifacts used in the executors should add the model and images by hash rather than value (reduce unnecessary data download/upload.

mentioned in commit 5cd446af441136ffd37b65a18f0179e476c70752

mentioned in commit f6e254c5c68a38bffa128b6c5d8f185bad6133c6


getting compilation issues

cannot run program, getting compilation error after last pull
Exception in thread "AWT-EventQueue-0" java.lang.Error: Unresolved compilation problem:
at org.magic.gui.components.CardSearchPanel.getInstance(CardSearchPanel.java:134)
at org.magic.gui.MagicGUI.initGUI(MagicGUI.java:228)
at org.magic.gui.MagicGUI.<init>(MagicGUI.java:75)
at org.magic.api.main.MtgDesktopCompanion.lambda$0(MtgDesktopCompanion.java:65)
at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:770)
at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:715)
at java.base/java.security.AccessController.doPrivileged(AccessController.java:391)
at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:740)
at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

not enough details. can you run project/clean from eclipse and rerun a build ?

tried cleanup source , same issue
deleted project and did a clean pull from github, same issue

can you run it from eclipse ?
Le jeu. 16 avr. 2020 à 21:34, Brolmeel<EMAIL_ADDRESS>a écrit :

tried cleanup source , same issue
deleted project and did a clean pull from github, same issue
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nicho92/MtgDesktopCompanion/issues/112#issuecomment-614853512,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXZPH2Y4ZAPQEPT2MQDHBLRM5MUBANCNFSM4MJXF3RQ
.

No running from Eclipse gives me the compile error

I switched to JavaSE-13 and now it runs.
Thanks


Create example docker-compose.yml

The current readme describes how to build the image and run it, but doesn't specify many of the simple pieces of information which would be useful to someone trying to put together a working install.
A reference docker-compose.yml would provide a clean way to illustrate the important parameters of a working install.
What (if any) ports need to be exposed?

To answer my own question:
Filecoin-docker needs the following ports (as stated in the dockerfile itself):
1234/tcp for API access
1235/tcp for P2P access
An example docker-compose.yml would still be useful.

Hi, @Jafner
Just added docker-compose.yaml

Awesome!
I suspect a case like mine is relatively common or I wouldn't bother asking.
My docker host doesn't have a lot of local storage, but it is connected to a large NAS.
How would I edit the volumes so that the configuration files are stored locally, but the ledger and data are handled by the NAS?

Hey @Jafner
You may mount your NAS to local folder and use it as local folder.


PinePhone keyboard case support

I'm successfully running the latest version of GloDroid on my PinePhone using the SD card image. Since I also have the keyboard case I wanted to try if that worked too, but as far as I can see it doesn't do anything. Is the keyboard case not supported yet, or is there additional steps I can take to get GloDroid to recognise it?

Hi,
There's a chance that it works on a main branch where kernel 6.1 was enabled.
There are stability issues related to suspend/resume, so no images are available yet.


Add TXC Token

Team background:
Tripxchain team has more than 3 years in blockchain technology and digital payment solutions. And team members of great knowledge and experience in the tourism trade.
Basic information of the project:
An ERC20 Token built on top of Ethereum Blockchain which aims to develop a new business
model through the revolutionary innovative application and practice of the Token mechanism, and all users of the platform will be the platform's equity holders
More about TXC
website : https://www.trip-xc.com/
github : https://github.com/GTCglobal
Telegram : https://t.me/tripxc
Medium: https://medium.com/@tripxchain
Trading Platform to be listed: www.btcoin100.com
Media Coverage:
Sina 财金头条 : https://t.cj.sina.com.cn/articles/view/6073357212/16a00139c00100iwm6?sudaref
Bi Ji guang : http://bjhvs.com/news/20181008/1141.html
cnfina.com: http://www.cnfina.com/kuaixun/20181009_137456.html
Recommended gas limit: 60000

@makoshan done removing, please kindly check and see if any changes to be made


Question: Is there a way to disable lightline on NERDTree

First of all this is a great plugin with great customizations ❤
So I want to disable the lightline inside the NerdTree view. is there any way to achieve it?

And one more thing I am using the agnoster [https://gist.github.com/agnoster/3712874] theme in zsh terminal with oh-my-zsh.
I was wondering if I could make the left side of the status bar exactly the same as agnoster with colors. I am very new to vim right now. Could you give me some direction or any help would be greatly appreciated.
Thanks

There's no way to disable in specific plugin windows, I'm not planning to implement, sorry.
You can create your own colorscheme to change the colours. See :h lightline-colorscheme or look into some repositories https://github.com/search?q=lightline+colorscheme.


Remove CLI

The current CLI is generated as an escript. That is not ideal, since it requires the Erlang VM to be available on the host for it to work properly.
One better alternative is using bakeware to generate the CLI. That approach does not require the Erlang VM to be available on the host (even though there are still a few dependencies on the host). The only issue with that approach is that bakeware turns the whole application into an OTP release. That would be a problem for us because this repository is also used as a library, and changing that would break its clients.
Therefore, all the code related to the CLI portion of this will be moved to the https://github.com/renderedtext/when-cli repository.
This PR also updates the application to use Elixir 1.11 and its dependencies to the latest versions.

Looks good 👍

Closing this as we decided to use the current one we have and rely on Erlang to be installed in the host.


Version information is not properly returned on JSforce ver 3.0

From CLI, it returns wrong version information:
$ npx<EMAIL_ADDRESS>--version
2.0.0-beta.8

As the above command refers the content of src/VERSION.ts, it should be maintained according to the update of version info in package.json.
Note: Formerly (in 1.0 era) it has "prepublish" script to confirm it is synced, so it was possible to notice if there is a difference before publishing as a package.
https://github.com/jsforce/jsforce/blob/c04515846e91f84affa4eb87a7b2adb1f58bf04d/package.json#L35

https://github.com/jsforce/jsforce/pull/1499#issuecomment-2189926603


Deep clustering  Implementation

-> deep clustering loss
-> Chimera ++ network
-> dataloader for WSJ 2 and 3 mix

Things pending:
-> Complete recipe with speech separation metrics
-> PIT loss for the mask estimation head

I guess you rebased and force pushed, we cannot see clearly what belongs to the PR. Let's open a new one, only with commits related to this PR.
There is no need to rebase master if their are no conflicts and if there are conflicts, merging from master is a better a option.

Codecov Report

Merging #34 into master will decrease coverage by 0.05%.
The diff coverage is 96.29%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
- Coverage   98.27%   98.22%   -0.06%
==========================================
  Files          20       20
  Lines         872      899      +27
==========================================
+ Hits          857      883      +26
- Misses         15       16       +1

Impacted Files
Coverage Δ

asteroid/masknn/blocks.py
100% <100%> (ø)
:arrow_up:

asteroid/utils.py
94.05% <87.5%> (-0.57%)
:arrow_down:

Continue to review full report at Codecov.

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

Merging it, thanks !
We still need the recipe, the evaluation script and a readme presenting the dataset