Node add isn't consistently working

To Reproduce
Initialisation steps from https://github.com/hashgraph/solo/issues/727 and:
npm run solo -- node add --gossip-keys true --tls-keys true --release-tag v0.54.0-alpha.4 --namespace solo-e2e

Describe the bug
◼ Finalize
node:internal/process/promises:289
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "#<ErrorEvent>".] {
  code: 'ERR_UNHANDLED_REJECTION'
}

Node.js v21.7.1

I've seen fails with the error from https://github.com/hashgraph/solo/issues/727 as well.
Describe the expected behavior
Node added and functioning. Does not happen every time but still it is not consistent for testing from our side.
Whole JUnit/CLI Logs
npm run solo -- node add --gossip-keys true --tls-keys true --release-tag v0.54.0-alpha.4 --namespace solo-e2e

><EMAIL_ADDRESS>solo
> NODE_OPTIONS=--experimental-vm-modules node --no-deprecation solo.mjs node add --gossip-keys true --tls-keys true --release-tag v0.54.0-alpha.4 --namespace solo-e2e

******************************* Solo *********************************************
Version			: 0.31.0
Kubernetes Context	: kind-solo-e2e
Kubernetes Cluster	: kind-solo-e2e
Kubernetes Namespace	: solo-e2e
**********************************************************************************
✔ Initialize [0.1s]
✔ Check that PVCs are enabled
✔ Identify existing network nodes
  ✔ Check network pod: node1
✔ Determine new node account number
✔ Generate Gossip key [0.3s]
  ✔ Backup old files
  ✔ Gossip key for node: node2 [0.3s]
✔ Generate gRPC TLS key [0.4s]
  ✔ Backup old files
  ✔ TLS key for node: node2 [0.4s]
✔ Load signing key certificate
✔ Compute mTLS certificate hash
✔ Prepare gossip endpoints
✔ Prepare grpc service endpoints
✔ Prepare upgrade zip file for node upgrade process [2s]
✔ Check existing nodes staked amount [2s]
✔ Send node create transaction [2s]
✔ Send prepare upgrade transaction [4s]
✔ Send freeze upgrade transaction [2s]
✔ Download generated files from an existing node [0.5s]
✔ Prepare staging directory
  ✔ Copy Gossip keys to staging
  ✔ Copy gRPC TLS keys to staging
✔ Copy node keys to secrets [0.1s]
  ✔ Copy TLS keys [0.1s]
  ✔ Node: node1
    ✔ Copy Gossip keys
  ✔ Node: node2
    ✔ Copy Gossip keys
✔ Check network nodes are frozen [9s]
  ✔ Check network pod: node1  - status FREEZE_COMPLETE, attempt: 3/120 [9s]
✔ Get node logs and configs [2s]
✔ Deploy new network node [5s]
✔ Kill nodes to pick up updated configMaps
✔ Check node pods are running [58s]
  ✔ Check Node: node1
  ✔ Check Node: node2 [58s]
❯ Fetch platform software into all network nodes
  ⠇ Update node: node1 [ platformVersion = v0.54.0-alpha.4 ]
  ⠇ Update node: node2 [ platformVersion = v0.54.0-alpha.4 ]
◼ Download last state from an existing node
◼ Upload last saved state to new network node
◼ Setup new network node
◼ Start network nodes
◼ Enable port forwarding for JVM debugger
◼ Check all nodes are ACTIVE
◼ Check all node proxies are ACTIVE
◼ Stake new node
◼ Trigger stake weight calculate
◼ Finalize
node:internal/process/promises:289
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "#<ErrorEvent>".] {
  code: 'ERR_UNHANDLED_REJECTION'
}

Node.js v21.7.1

Additional Context
No response

Also tried doing the same flow with the C++ SDK NodeCreateTransaction followed by npm run solo -- node add-execute --input-dir context. Seems like the node pod is correctly created also setup and start are passing as well but got the following log:
npm run solo -- node add-execute --input-dir context

><EMAIL_ADDRESS>solo
> NODE_OPTIONS=--experimental-vm-modules node --no-deprecation solo.mjs node add-execute --input-dir context

******************************* Solo *********************************************
Version			: 0.31.0
Kubernetes Context	: kind-solo-e2e
Kubernetes Cluster	: kind-solo-e2e
Kubernetes Namespace	: solo-e2e
**********************************************************************************
✔ Initialize [0.1s]
✔ Identify existing network nodes
  ✔ Check network pod: node1
✔ Load context data
✔ Download generated files from an existing node [0.4s]
✔ Prepare staging directory
  ✔ Copy Gossip keys to staging
  ✔ Copy gRPC TLS keys to staging
✔ Copy node keys to secrets
  ✔ Copy TLS keys
  ✔ Node: node1
    ✔ Copy Gossip keys
  ✔ Node: node2
    ✔ Copy Gossip keys
✔ Check network nodes are frozen [6s]
  ✔ Check network pod: node1  - status FREEZE_COMPLETE, attempt: 0/120 [6s]
✔ Get node logs and configs [8s]
✔ Deploy new network node [2s]
✔ Kill nodes to pick up updated configMaps
✔ Check node pods are running [30s]
  ✔ Check Node: node1
  ✔ Check Node: node2 [30s]
✔ Fetch platform software into all network nodes [5s]
  ✔ Update node: node1 [ platformVersion = v0.54.0-alpha.4 ] [5s]
  ✔ Update node: node2 [ platformVersion = v0.54.0-alpha.4 ] [5s]
✔ Download last state from an existing node [0.4s]
✔ Upload last saved state to new network node [0.4s]
✔ Setup new network node [0.1s]
  ✔ Node: node1 [0.1s]
    ✔ Set file permissions [0.1s]
  ✔ Node: node2
    ✔ Set file permissions
✔ Start network nodes [0.1s]
  ✔ Start node: node1
  ✔ Start node: node2
↓ Enable port forwarding for JVM debugger
❯ Check all nodes are ACTIVE
  ✔ Check network pod: node1  - status ACTIVE, attempt: 16/120 [24s]
  ✖ node 'node2' is not ACTIVE[ attempt = 120/120 ]
◼ Check all node proxies are ACTIVE
◼ Stake new node
◼ Trigger stake weight calculate
◼ Finalize
*********************************** ERROR *****************************************
Error in setting up nodes: node 'node2' is not ACTIVE[ attempt = 120/120 ]
***********************************************************************************

We discovered there is currently an issue in platform/services with NodeCreateTransaction.  After the node has been added and the one of the nodes goes into teach mode for the newly added node, the teacher will get JVM out of memory errors after finishing teaching and reconnecting to the network.  I'm not sure the exact amount, but Nathan quoted 22GB of memory (not sure what this 22GB refers to).  I think you might be able to get around this by setting the JVM memory settings really high, but we haven't configured Solo to do that by default.
We have disabled our E2E tests involving solo node add until this is resolved in a patch.  I'm reaching out to find an issue that we can use to track this with.

@gsstoykov , this is fixed in main branch, are you able to test using that, or would you like me to notify you when we push the next release tag to NPMJS?


Reporting High CPU Usage and Overheating Issue with CPPTools in Visual Studio Code

Type: Bug
Subject: Reporting High CPU Usage and Overheating Issue with CPPTools in Visual Studio Code
Dear [CPPTools Development Team],
I hope this email finds you well. I am writing to report an issue I have encountered while using CPPTools in Visual Studio Code, which involves high CPU usage and excessive heat generation on my laptop. I kindly request your assistance in resolving this matter.
Description of the Issue:
I have observed that the CPPTools extension in Visual Studio Code is causing unusually high CPU usage on my Acer Aspire 3 laptop, running Kali Linux 24.1. This high CPU usage results in excessive heat generation, causing concern over potential hardware overheating and performance issues. The issue persists even during idle moments or when performing simple tasks in Visual Studio Code.
Steps to Reproduce:
Open Visual Studio Code.
Load a C++ project or file.
Navigate through the code or perform editing tasks.
Monitor CPU usage and temperature using system monitoring tools.
Expected Behavior:
CPPTools should operate efficiently, consuming reasonable CPU resources and not causing excessive heat generation on the laptop.
Actual Behavior:
CPPTools consumes a significant portion of CPU resources, leading to increased heat generation and potential performance degradation on my Acer Aspire 3 laptop with a 512GB SSD running Kali Linux 24.1.
Environment Details:
Laptop Model: Acer Aspire 3
Storage: 512GB SSD
Operating System: Kali Linux 24.1
Visual Studio Code Extensions:
Auto Close Tag
C/C++ Extension Pack
CMake
CMake Tools
Code Runner
Docker
Indent Rainbow
Prettier
Pylance
Python
Python Debugger
Rest Client
Additional Information:
The issue persists across various tasks and scenarios within Visual Studio Code, suggesting that it is related to the CPPTools extension.
I have not experienced similar CPU usage or overheating issues with other extensions or software running on my laptop.
The excessive heat generation raises concerns about potential hardware damage and impacts on system stability.
I greatly appreciate your attention to this matter and kindly request your assistance in investigating and resolving this issue. If you require any further information or assistance from my end, please do not hesitate to reach out.
Thank you for your prompt attention to this issue. I look forward to your response and a resolution that will enhance the usability and stability of CPPTools for all users.
Sincerely,
puspa raj regmi
9742332525
<EMAIL_ADDRESS>VS Code version: Code 1.87.2 (863d2581ecda6849923a2118d93a088b0745d9d6, 2024-03-08T15:14:59.643Z)
OS version: Linux x64 6.6.9-amd64
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1035G1 CPU @ 1.00GHz (8 x 1492)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
4, 3, 3

Memory (System)
19.32GB (14.47GB free)

Process Argv
--crash-reporter-id 43dedd27-d11b-4754-a2c4-64bb36e71687

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome

XDG_SESSION_TYPE
x11

Extensions (13)

Extension
Author (truncated)
Version

prettier-vscode
esb
10.1.0

auto-close-tag
for
0.5.15

code-runner
for
0.12.1

rest-client
hum
0.25.1

vscode-docker
ms-
1.29.0

debugpy
ms-
2024.2.0

python
ms-
2024.2.1

vscode-pylance
ms-
2024.3.1

cmake-tools
ms-
1.17.17

cpptools
ms-
1.19.8

cpptools-extension-pack
ms-
1.3.0

indent-rainbow
ode
8.3.1

cmake
twx
0.0.17

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
2i9eh265:30646982
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:30986171
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
pydisww2:30991134
cppperfnew:30979542
d34g3935:30971562
fegfb526:30981948
bg6jg535:30979843
dsvsc020:30976470
pythonait:30991152

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Add styles to website

The style-page branch focused on styling the page to look appealing. This branch implemented the following;

[ ] Reset form after user has submitted score
[ ] Add styles to the focused state of the form inputs
[ ] Added styles to center the whole page

Thanks for your time and effort in reviewing my work


Added icdefinetti.edu.it

Added icdefinetti.edu.it

@andQlimax First stage of verification passed, official domain for Istituto Comprensivo Bruno De Finetti is icdefinetti.edu.it as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@andQlimax Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Created a new Preferences window, and added PromptSaveOnExit option.

Redo of #1106

@MasterEric thanks for the work adding the generalised preferences dialog! The PromptSaveOnExit option looks good.
As mentioned in https://github.com/ros-visualization/rviz/pull/1106#issuecomment-370589511, the change will break ABI (because it adds a member to the VisualizationFrame class). This means we won't be able to release it into indigo/kinetic/lunar, so will hold it for when we release rviz into melodic.
Since we are breaking ABI anyway, I changed your PR to use a generalised Preferences struct, so that we will be able to add additional preferences in the future without breaking ABI again.

@dhood Great idea creating the Preferences struct! I'm not well-versed in what affects the ABI and what causes conflicts in applications.
I'm certainly excited to see my major changes in the upcoming Melodic release!
I'm wondering if you have any opinions/ideas/suggestions for other options that would be useful to include in the Preferences menu.

I'm certainly excited to see my major changes in the upcoming Melodic release!

congrats! even though your original PR had to be replaced by this one; you will still be credited in the changelog :smile:

I'm wondering if you have any opinions/ideas/suggestions for other options that would be useful to include in the Preferences menu.

Not necessarily, but these things pop up as we/contributors add different functionality; it will be helpful to be able to add options to this struct without breaking ABI for those features (from melodic onwards)


.hangman 's quote should behave like .ttt to prevent spam

as topic

http://feathub.com/Kwoth/NadekoBot/ :thinking:


Update fmutils.go

This condition was missing when trying to filter on nested structs it was panicing.

Please add a test that covers this case.

Yes of course. Sorry I just realized that I didn't commit them... Forgive me.
I will do it tomorrow.

@mennanov There you go, I added some tests. Hope this is ok for you. Feel free to reach me if you need.

Your pull request actually revealed that the code does not currently support an important use case: masks in repeated non-scalar fields.
I created a pull request to address this issue: https://github.com/mennanov/fmutils/pull/2
It should also fix the problem this pull request addresses.
Please, take a look at that pull request and let me know if it works for you. If everything is OK i'll merge my pull request in favor of this one.
Thanks for contributing!


Appodeal crash

Hello,
I've faced a crash.
It happens during flutter engine init.
The activity ref can't be found.
Flutter version: 3.0.2
Appodeal version: 1.2.2
Non-fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: PlatformException(error, Required value was null., null, java.lang.IllegalStateException: Required value was null.
	at com.appodeal.appodeal_flutter.d.c(AppodealBaseFlutterPlugin.kt:1)
	at com.appodeal.appodeal_flutter.e.g(AppodealFlutterPlugin.kt:106)
	at yr.i$a.a(MethodChannel.java:4)
	at nr.c.g(DartMessenger.java:1)
	at nr.b.run(Unknown Source:2)
	at android.os.Handler.handleCallback(Handler.java:938)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loop(Looper.java:233)
	at android.app.ActivityThread.main(ActivityThread.java:8010)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:631)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:978)
). Error thrown null.
       at StandardMethodCodec.decodeEnvelope(message_codecs.dart:607)
       at MethodChannel._invokeMethod(platform_channel.dart:167)

It could be related that I've integrated flutter as a module in the existing app.
Thanks!

@JenshenSoft Hello!
Thanks for your issue, we started to investigate this behavior.

@JenshenSoft Hello!
We have released a new version of the SDK in which we have changed the logic.
If you have any questions/issues I will be glad to answer.
Have a good day.


Support Hyperlink for Markdown reporter

It'd be awesome to have ability of generating a hyperlink to a file using some template/base specified in parameters. Right now it's just a file path.

Great idea but I'm not sure I want to break backwards compatibility for the current markdown reporter. Perhaps a markdown2 reporter?

Maybe it does not have to break it - and expose it as additional parameter?

Feel free to add a PR

Closing this, feel free to add a PR


Forever spectador mode.

Just do it:

Eat a chorus fruit.

  I forgot dying was possible in spectator, makes sense though.

https://minecraft.curseforge.com/projects/cyclic/files/2389366
https://minecraft.curseforge.com/projects/cyclic/files/2389358


Recommendation to move type to dependency

The recommendation to move type to dependency should only happen when bundling types:
Recommendation: "@types/ini" is externalized because "ini" is in "dependencies". Place "@types/ini" in "dependencies" as well so users don't have missing types.

And ideally, if the type dependency is imported.

Are there any plans to fix this?

Yes but not a priority. Feel free to submit a PR if you'd like!


feat: deploy success message also displays the namespace

Changes

Added the function's namespace to the success message after deploy
If the namespace in function obj is blank, display current context namespace

Fixes #1083
Release Note
Prints the namespace when a function is deployed

@zroubalik @lkingland Thanks for the reviews. I am stuck on the codecov part. I have added changes to the Deploy function but I can't see a way to use mock clients for writing unit tests.

@vyasgun I think that's okay, the test  that you have added is imho enough for this feature


Need to require ext-mbstring ?

Hi,
Since 7.1, erusev/parsedown require ext-mbstring and before 7.2 eresus/markdown have a vulnerabily due to markdown.
I made a test with symfony/demo and receive a message from security-check when require an extra package and when I try to update because of erusev/parsedown
Install php-mbstring solve the problem because without 7.0 is installed.

Sorry we didn't reply to this on time. I've checked the project and I don't see any security vulnerabilities in the latest version. So, I'm closing this as "probably fixed". Please, tell me if this is not true so we can reopen it.
Also, not strictly related to this, but in #1055 we migrated this project to use the String component, so hopefully we can do anything related to strings without using the mbstring extension.


Add a callback that downloads cluster credentials after creation.

Fixes #1517

Oh, PLEASE. +24.35

Excited to see this land!

@yugangw-msft @derekbekoe @tjprescott can i get a review on this one? its ready to go....

@yugangw-msft comment addressed, please re-check.
Thanks!
--brendan

Closing as this PR has been open for a while and has conflicts.


Add context dir to bud command in baseline test

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>With a recent change to the bud processing, a context directory is not required.  Add a '.' to the end of the bud call in the baseline tests.
This test is not run in the CI, it is only run by hand.

LGTM
@rh-atomic-bot r+


Implement gmtime()

We recently got a PR to implement gmtime() in our PDCLib fork, but due to the code coming from newlib, I decided against it and instead implemented it myself (partially using http://howardhinnant.github.io/date_algorithms.html which is in the public domain, all the rest is my own code, so it should be fine for distributing under the CC-0).
The code is not specific to our platform, so I'd like to contribute this back upstream. I consulted the C standard before implementing it, and it is not locale-dependent, although it could be adapted to implement the locale-dependent counterparts. I implemented it separately from PDCLib at first and checked its behavior against glibc's gmtime() for a wide range of input values (all from -4294967295 to<PHONE_NUMBER> on my x86_64 Arch Linux). I afterwards modified it to better fit PDCLib's coding style and added tests based on some edge cases (the double parentheses in the testcases avoid a compiler warning related to using the assignment operator in an if-statement).

Thank you for what appears to be a very valuable contribution!
I am currently in the last stages of the long-overdue rework of freopen(), and if memory serves correctly I have several open construction sites in the locales / collation area, but I will see if I can make time (no pun intended) to review this.

It has been almost a month. What happened to this / will happen to this?

I have a bit of a problem with the implementation. Not criticising thrimbor's work, which is perfectly acceptable as a stand-alone gmtime(). However, as a "citizen of <time.h>", it falls somewhat short. (Lots of uncommented magic numbers, needs C99 mode from the compiler, and does not handle leap seconds).
While looking for some authoritative input on leap seconds, I came across IANA's reference implementation (tzcode), which is also public domain (with some exceptions that do not apply to PDCLib, most notably strftime() which I already implemented). I started to pick it apart with intention to use it as basis (time zones, leap seconds and all), perhaps cross-checking with thrimbor's code to see if there's room for improvement to the IANA code.
(You should see the internal dependency graph I made... my GF mentioned casually it looked like a plan for the subway of Paris. :-D )
I got a bit sidetracked when I found that PDCLib stopped compiling cleanly on Cygwin at some point in the past, and started reworking type definitions in _PDCLIB_config.h and _PDCLIB_int.h. That work is 90% complete and bound to be finished this weekend. I'll pick up working on <time.h> again immediately afterwards. I've got two weeks of easter vacation; I am not going to spend all that time coding, but a sizeable portion of it.

A bit of delay here. One, I have yet to figure out a robust way to get the "right" print/scan prefixes for inttypes.h from what gcc/clang give me in the form of compiler predefines. Two, our cat fell ill and requires surgery, so the weekend was a couple hours short coding-wise. Overall plans remain unchanged.

I really hate it when an issue remains open for so long, but life has an unfortunate tendency to happen from time to time.
I had to re-start my efforts of integrating tzcode into PDCLib, as I tried to do too many things at once in the first try.
In this second attempt, I left the internals of tzcode basically untouched, and focussed solely on getting the whole thing to compile cleanly and pass some rudimentary tests. Which it does, now.
Upside is, there's functionality to be had. Downside is, this is currently a really ugly hack job that will require quite some polish before it will feel "natural" (or stop breaking the localtime() test cases for anyone not in my time zone, for example). Work still to come, but there is a gmtime() now, so I thought I'd notify you.


Add an example comparing a CLI use with a Python-based use

Once issue #70 is done, implement a shell script that does the same thing as simulate_active_learning


Wildcard routing

With this setup how do you go about handling wildcard routing on the server?
I usually use something like this in the routing file:
{ path: '**', redirectTo: 'error' }
However, that doesn't seem to work on the server.  Do you handle it at the server level?  I'd rather handle it with Angular personally.  BTW, the project is great!  I love the idea of having my api with my Angular project.
Thanks!

Why do you need to handle it at the server level?
The router on the client must match the router on the server. Just add the error page also in the server: https://github.com/robisim74/AngularCliAspNetCore/blob/master/Startup.cs#L38 and it works.

Sorry I did not understand the question. For wildcard routing I think you need to customize the function: https://github.com/robisim74/AngularCliAspNetCore/blob/master/Startup.cs#L43-L51

Yeah that's where I was at when I stopped to contact you.  If I come up with a solution I'll post back but I'm uncertain if  I'll get back to this project anytime soon. Thanks for the help!

@mattsibenaller
I solved this problem (and simplified the whole configuration) using SpaServices. You can see the README for the new build.


Address aggressive truncation in Scope panel

@andrewdavey
At some point in the last few months an update to Chrome has made the extension start to truncate anything in the Scope panel that's larger than the panel. I'm not really sure what this change was and after spending a few hours looking for it I found nothing obvious in the Chrome changelogs.
I was able to fix this issue by adding position: relative to the top level span elements which effectively just ignores the wrapping elements truncation rules. The ellipsis still appears on top of the carrot but it's a minor issue compared to the data not showing up at all. I tried a bunch of other solutions and couldn't get anything to work quite as well as the position: relative so if anyone else has any suggestions I'd be happy to look into them
I think it used to cut off the record at the end of the line instead of wrapping it around to the next line which could be slightly concerning for larger records so if you think that's an issue I can look into ways to fix it.
I ran the test page and didn't see any regressions in my testing
Example

Before
After

@andrewdavey can you take a quick look at this?

Thanks for the contribution!
Merged and published v0.1.5 to npm


Live reload ini file

Seems request safe quit read loop, this part can utilize pause variable and to trigger an event after read loop is done.

Don't see a necessary for me, my friend no need this either.
But I do interest in. Reply or submit a new issue to prioritize this feature.


Fancyzones editor treating all of my monitors as one giant monitor

Microsoft PowerToys version
0.41.4
Running as admin

[X] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
Update Win 10 to latest version (non insider)
Reinstall PowerToys.
Restart PC

PowerToysReport_2021-07-03-12-30-32.zip
✔ Expected Behavior
Let me split individual monitors without letting me create zones outside of them
❌ Actual Behavior
FancyZones treat all of my monitors as one giant screen even letting me create zones outside of my physical displays
Other Software
No response

Thanks for reporting this @gvarph. How are the monitors connected (HDMI, DPI, etc)?

How are the monitors connected (HDMI, DPI, etc)?

The small one on the left is connected by USB-C (directly from my GPU), the one above it by HDMI, and the other two by display port.
I should have also mentioned that my GPU is the 'MSI GeForce RTX 2070 Armor 8G

@gvarph if you have the Allow zones to span across monitors setting on, the monitors would be treated as unified single work area. Uncheck this option to work with each monitor individually.

@gvarph if you have the Allow zones to span across monitors setting on, the monitors would be treated as unified single work area. Uncheck this option to work with each monitor individually.

Didn't have it on, but turning it off and on helped. Interesting that this helped but reinstalling powertoys didn't


T.absurd permits more than just a variable as its argument

Input
→ View on sorbet.run
# typed: true

T.absurd(:foo) if T.unsafe(false)

Observed output
editor.rb:3: Control flow could reach T.absurd because the type Symbol(:"foo") wasn't handled https://srb.help/7026
     3 |T.absurd(:foo) if T.unsafe(false)
        ^^^^^^^^^^^^^^
  Originating from:
    editor.rb:3:
     3 |T.absurd(:foo) if T.unsafe(false)
                 ^^^^
Errors: 1

Expected behavior

This should be a different error: T.absurd only makes sense when used with a variable.
https://github.com/sorbet/sorbet/blob/2e01c34cdf0190e0a372368a9d6ac45fac1de36d/cfg/builder/builder_walk.cc#L347-L355

Fixed by #3531


[Toggle Button]

Description
when the page is opened in small pages,  the options in the header portion should appear with in a toggle button.
Motivation
It will be very good for user to interact with the website and the structure of the website will look stable.
Screenshots
No response
Additional Context
No response
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct.
[X] The changes don't break the code.
[X] I will attach screenshots and videos of the development work once it is done.

Priority
Medium

I am talking about these options, which appear in a haphazard way. Please assign this task to me, I will put these in a button for small pages.


Barrels Play Closing Sound Immediately and Only Once

Forge, version 1.19-<IP_ADDRESS>
When opening a lootr barrel for the first time, it plays the barrel opening sound as expected. However, it also plays the barrel closing sound immediately. The barrel's inventory remains open and it does not play the barrel closing sound when it is actually closed. After the first time, opening and closing both make no sound. Sculk sensors react accordingly - they only activate when the barrel is opened for the first time and don't activate when it is closed or re-opened. This doesn't happen with chests or shulker boxes, only barrels.

I'll look into this, thanks!

This will be addressed in the next 1.20 release and backported for other versions.


Can you make the back button in the tab navigator go to the last tab?

By default, pressing back in the tab navigator will go to the initial route. How can you make it go to the last selected tab? The last state carries the correct index, but the behavior is wrong.

You can set backBehavior of tab navigator to none. Check documentation for Tab Navigator ☺

Hmm I tried that but it just made the app close when back was pressed. Is that intended? I was thinking if theres 3 tabs and you click on tab 1, then tab 2, then tab 3, pressing back should go to Tab 2

It's not possible right now. PR welcome.

Closing in favor of https://github.com/react-community/react-navigation/issues/678


MRG, BUG: Raw.rename_channels() should update Raw._orig_units too

In main, calling rename_channels() on a Raw instance would not
adjust Raw._orig_units.
While you could argue that the original units were only specified for
the original channel names, I would say that this makes the units
simply useless after channels have been renamed ;) Therefore I
consider the current behavior a bug.
This PR makes it so that if Raw._orig_units exists, its keys are
adjusted such that they stay in sync with the new channel names.
Discovered while working on data with @MaelysSolal

Thanks @hoechenberger


ENSCORESW-3737: new MANE object inherit from transcript

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
New object MANE created, it is a transcript object with additional attributes
Use case
MANE transcripts are transcripts which, as part of the MANE project, have a perfect match with a RefSeq mRNA
It is possible to check whether a transcript is part of MANE, and if it is, to retrieve information about the matching RefSeq accession and the MANE type it belongs to (MANE_Select, MANE_Clinical_plus...)
Benefits
Direct access to information about MANE transcripts
Possible Drawbacks
Not all transcript functionalities have been implemented for the MANE transcript, one would need to go back to the original transcript to access information like list of exons or coordinates
Testing
Have you added/modified unit tests to test the changes?
Yes
If so, do the tests pass/fail?
Pass
Have you run the entire test suite and no regression was detected?
Yes

Coverage increased (+0.2%) to 82.34% when pulling 94e719fd975404cb842ada486a3d393bd874af63 on feature/mane_data into 55b512300d11944d5bada0287c6e00393da0f76f on release/105.


Support of jwplayer : Important vital please add support of this capital website : fullmatchsports highlights

Hello, my laoptop broke down so I use my little laoptop which dates from 2012 windws 7 and when I watch a live football match, it's almost impossible to watch so much that it lags, the solution to this problem is to download the match highlights and the site "fullmatchsports.cc" seems to be one of the rare sites to offer long highlights, i.e. highlights which last 25 minutes or 30 minutes :
https://fullmatchsports.cc/spain-vs-france-full-match-euro-2024/?tab=highlights
hence the capital importance of this website because many other highlights websites offer highlights of only 10 minutes.... and so the problem is it is impossible to download the video highlights on this rare site and we hope it will be fixed soon please, thank you very much :)

Hello Mr @puemos , i just find this tool, maybe it can help to detect and download jwplayer videos :)
https://github.com/furkaninanc/jwplayer-scraper
let's also remember Live stream downloader extension succeded to fix this jwplayer problem detection:
https ://github.com/ chandler-stimson/ live-stream-downloader/ commit/
0a05fae3b9d7b3b0cb236095745925b062ccf844


Use pytest-xdist to parallelize unit tests

This speeds up the tests by several times (depending on your number of cores), particularly the threading stress tests.
Note that this currently isn't compatible with most of the integration tests. I believe this is due to the tests themselves and not the cache backends.

Coverage remained the same at 94.951% when pulling 802b499b679f0292cd3d2c87b89822746c5f20ad on JWCook:pytest-xdist into d80901a689876fceb88330d6698432eba33592b0 on reclosedev:master.


nuke *Get postfix from orbit

Mostly this is just sed, but there is also a slight refactoring in that
containerLXD had a single public member named Storage, which collided with
Storage() once you remove the Get postifx. This commit switches that and
its usage to a private member. Also:

Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.
Go is not Java.

Signed-off-by: Tycho Andersen<EMAIL_ADDRESS>

:-)


Component derive macro should allow multiple #[require(...)] attributes.

Future PR: Might be nice to modify the Component derive macro to allow multiple #[require(...)] attributes (if that's possible?). Would make cases like this cleaner.
Originally posted by @bushrat011899 in https://github.com/bevyengine/bevy/pull/16322#discussion_r1835860598

To my pleasant surprise, this actually already works. The proc-macro gracefully handles an arbitrary number of #[require(...)] attributes, appropriately combining them.


Calibration BUG

After testing, when the "Open Camera" button is turned on, the system will automatically start to detect, which makes the button to check the detection function lose its function.

By the way, there is not an accurate calibration function to call right now, it needs to be adjusted

Try to modify and increase the detection function switch

solve it


Updated RSA key format to PKCS#8

https://cryptography.io/en/latest/hazmat/primitives/asymmetric/serialization/#cryptography.hazmat.primitives.serialization.PrivateFormat
PKCS8 recommended over TraditionalOpenSSL.

LGTM. Let's test it.

Thanks @adrianchifor. Superb!


Embeddable in mapped superclass is not mapped properly

Hi! I got three classes: a super class with mapping, an extending it class and embedded value object.
Mapping for superclass:
Occurrence:
  type: mappedSuperclass
  table: occurrences
  embedded:
    duration:
      class: Duration
      columnPrefix: false

Mapping for embeddable:
Duration:
  type:   embeddable
  fields:
    minutes:
      type: integer
      nullable: true

Just for sake of a test I created extending class that inherits everything from Occurrence class:
/**
 * Class Occurrence
 * @ORM\Entity()
 * @ORM\Table(name="occurrences")
 */
class Occurrence extends BaseOccurrence
{
}

Schema is being created correctly with minutes column in database, fixtures are being loaded correctly, but:

column minutes value after persisting an object is the only one always empty (only this value is not saved), despite that value in $occurrence->duration()->minutes() is set.
after getting data from DB and hydrating object everything is fine except that $occurrence->duration() is always NULL.

What I'm doing wrong? please help!

Can you also give the PHP code of the "Duration" embeddable?
Just wondering, why use an embeddable when only one field is used/required?
On 8 October 2016 at 14:47, Grzegorz Kaszuba<EMAIL_ADDRESS>wrote:

Hi! I got three classes: a super class with mapping, an extending it class
and embedded value object.
Mapping for superclass:
Occurrence:
type: mappedSuperclass
table: occurrences
embedded:
duration:
class: Duration
columnPrefix: false
Mapping for embeddable:
Duration:
type:   embeddable
fields:
minutes:
type: integer
nullable: true
Just for sake of a test I created extending class that inherits everything
from Occurrence class:
/**

Class Occurrence
@ORM\Entity()
@ORM\Table(name="occurrences")
*/
class Occurrence extends BaseOccurrence
{
}

Schema is being created correctly with minutes column in database,
fixtures are being loaded correctly, but:

column minutes value after persisting an object is the only one
always empty (only this value is not saved), despite that value in
$occurrence->duration()->minutes() is set.
after getting data from DB and hydrating object everything is fine
except that $occurrence->duration() is always NULL.

What I'm doing wrong? please help!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/issues/6078, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAvY1Yv0LZTfMvR77CnrALIDfhvQyWx3ks5qx5DXgaJpZM4KRtlI
.


fabric8:deploy

... should apply the resource descriptor to the OpenShift API server.
More or less the same as the V2 fabric8:apply.

fixed


Bug fix: edit command parser

When the user input is edit [index] c/, the error message shown was for addmcq command instead of edit command. I've changed the logic to handle this error better.

Codecov Report

Merging #228 (f1168cc) into master (f556767) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #228      +/-   ##
============================================
- Coverage     73.46%   73.42%   -0.04%
  Complexity      660      660
============================================
  Files            96       96
  Lines          2095     2096       +1
  Branches        253      253
============================================
  Hits           1539     1539
- Misses          476      477       +1
  Partials         80       80

Impacted Files
Coverage Δ
Complexity Δ

...ava/quickcache/logic/parser/EditCommandParser.java
77.14% <0.00%> (-2.27%)
9.00 <0.00> (ø)

Continue to review full report at Codecov.

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


readerIcon.svg invisible on chromium browsers

When implementing doq on my calibre-web instance, I noticed that the readerIcon was showing as invisible in chromium based browsers.
I tested this out on both Brave and Edge

Alongside Firefox:

I was able to fix the issue for my instance by updating the fill rules below.
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" width="16" height="16" fill="none">
     <path d="M4 1a2 2 0 0 0-2 2l0 10a2 2 0 0 0 2 2l8 0a2 2 0 0 0 2-2l0-10a2 2 0 0 0-2-2L4 1zm8.75 12.15-.6.6-8.3 0-.6-.6 0-10.3.6-.6 8.3 0 .6.6 0 10.3z" fill-rule="evenodd" clip-rule="evenodd" fill="#15141A"/>
     <path d="m10.125 5.25-4.25 0a.625.625 0 0 1 0-1.25l4.25 0a.625.625 0 0 1 0 1.25z" fill-rule="evenodd" clip-rule="evenodd" fill="#15141A"/>
     <path d="m10.125 8.25-4.25 0a.625.625 0 0 1 0-1.25l4.25 0a.625.625 0 0 1 0 1.25z" fill-rule="evenodd" clip-rule="evenodd" fill="#15141A"/><path d="m8.125 11.25-2.25 0a.625.625 0 0 1 0-1.25l2.25 0a.625.625 0 0 1 0 1.25z" fill-rule="evenodd" clip-rule="evenodd" fill="#15141A"/>
</svg>

@jrhedman Thanks for using doq! Yes, this was a known issue: someone had recently pointed it out in their Chrome Webstore review.
I had fixed it in 8cf9a57 (on the dev branch) by changing the fill rules just like you have done. I would have happily accepted a PR from you otherwise! 🥲
This will be merged into master by the next release (my release cycle is usually very long because of lack of time; so please consider using the dev branch in your projects 😅).

Sure thing, I hadn't taken a look at the dev branch but happy it's being addressed. There were many many hours of trying to figure out what I did wrong on set up to only realize the icon was just invisible haha.
Hopefully this issue/comment chain will give some people direction on a workaround until the changes have been merged.
And thank you for saving my eyes when reading through pdf's at night!

Sorry to have wasted your time! I'll keep this issue open until the fix is merged, thanks!


no configuration mode

Not sure it is doable, but, I would like a setup that works without any config file.
You install neostandard and you use its CLI.. and it is done.

It's possible. See #2.

Tried to coordinate this with the wider ESLint community, but looks like we will have to do our own solution https://github.com/eslint/eslint/issues/18800

Tracking in #205 (sorry, forgot this one already existed)


Adding bitcoin2015to2017

Hey khuangaf ! Its a great repo. Thanks for sharing!
There was an issue #3 named "file not found bitcoin2015to2017.csv".
So I added the necessary file. Hope you like it and this would make your repo even better than before. ;)

@deadskull7  This csv file can be downloaded when you run DataCollection.ipynb file in jupyter

Ok


Support module imports using a qualified name

Pull Request Description
This change allows for importing modules using a qualified name and deals with any conflicts on the way.
Given a module C defined at A/B/C.enso with
type C
    type C a

it is now possible to import it as
import project.A
...
    val x = A.B.C 10

Given a module located at A/B/C/D.enso, we will generate
intermediate, virtual, modules that only import and export the successor module along the path.
For example, the contents of a virtual module B will look like
import <namespace>.<pkg-name>.A.B.C
export <namespace>.<pkg-name>.A.B.C

If module B is defined already by the developer, the compiler will inject the above statements to the IR.
Also removed the last elements of some lowercase name resolution that managed to survive recent
changes (Meta.Enso_Project would now be ambiguous with enso_project method).
Related to https://www.pivotaltracker.com/n/projects/2539304
Important Notes
There was an additional request to fix the annoying problem with from imports that would always bring
the module into the scope. The changes in stdlib demonstrate how it is now possible to avoid the workaround of
from X.Y.Z as Z_Module import A, B

(i.e. as Z_Module part is almost always unnecessary).
Checklist
Please include the following checklist in your PR:

[ ] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[ ] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide dist and ./run ide watch.

Btw. the meaning of

Request, Advice, Opinion

is taken from SunARC (Sun Microsystems Architecture Review Committee) guidelines as adopted by NetBeans. I am approving only under the assumption that all Requests are addressed. I'd be glad to see Advices implemented - but you can explain why that's not good idea and I'll live with that. Opinions reflect my personal preferences and you can ignore them.


Fixing mock test infrastructure to handle batch and randomaccess index topologies

Hey @MohanDV it looks like you were missing updating the mock infrastructure for our REST tests.  I went ahead and did that for you.  Hopefully the tests pass now. :)

Thanks @cestella


Ability to sign http request

TwitterAccessor could have a new feature that allow the developer to sign a request to execute on any of the Twitter public endpoints.
This method will return all the headers (and potentially additional information) required for a user to execute the query client side but yet using the signature created by Tweetinvi.

ITwitterQuery now has a AuthorizationHeader property that can be used.
TwitterAccessor.GenerateTwitterRequestParameters allow developers to get all the information required to run a query from another server.

Really cool that this has been added :)
I've been having a play with it & put together a demo at https://github.com/JoshKeegan/TweetinviRequestSigner.
Were you intending for TwitterAccessor.GenerateTwitterRequestParameters to be used server-side to sign the request? If so, what's the advantage over using OAuthWebRequestGenerator directly, as in https://github.com/JoshKeegan/TweetinviRequestSigner/blob/master/TweetinviRequestSigner/Controllers/RequestController.cs ?
I looked at the new method, but it didn't seem necessary. I'll no doubt have missed something here!


Adds constraint (required|forbidden)unlessAnyOf

retest this please
Changed test config, to enable Delete workspace before build starts

@kemitix I completely agree about the tests being cryptic. Passing four booleans in a row in a method signature is an anti-pattern: it is hard to read and very error-prone.
These tests were originally written by Alex - I did some minor improvements to their readability (see https://github.com/apache/brooklyn-server/commit/d19d2079b1a2b16c6de8c950b67a2daf0b8d3372). I decided it was hard to make it much more readable without significantly increasing the amount of code required.
On balance, because it is a test class and because the methods are private, it seemed ok to leave the methods as cryptic - unpleasant for code reviewers looking at a diff, but if you open up the class it's liveable with.

Oh by the way I had to jump through a hoop or two to get Brooklyn built with this - I was initially getting build errors from brooklyn-ui with this in place; I had to rebase this PR against master, and even at that got a build error as follows when building from the brooklyn parent:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 02:07 min
[INFO] Finished at: 2018-10-21T22:09:15+01:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.karaf.tooling:karaf-maven-plugin:4.1.6:verify (verify-brooklyn-ui-modularity-feature) on project brooklyn-ui-modularity-features: Verification failures: Verification failures:
[ERROR] 	Unable to resolve framework features
[ERROR] 	Unable to resolve framework features
[ERROR] 	Unable to resolve framework features
[ERROR] -> [Help 1]

oddly when I did
cd brooklyn-ui/modularity-server/features
mvn clean install -DskipTests

it worked, and then when I went back to the top and resumed the original build, that finally passed. What's that all about?

Will merge this


Add SplitBy

Commit Message
Add SplitBy (#461)
This patch adds SplitBy iterator transformation and an underlying
(ATM internal) ReduceSplitBy transducer. SplitBy can be used for
processing like eachline and split in parallel and/or streaming
programs.

ID
time
GC time
memory
allocations

...
...
...
...
...

["splitby", "count", "foldl"]
3.797 ms (5%)

96 bytes (1%)
4

["splitby", "count", "man"]
1.504 ms (5%)

["splitby", "count", "reduce"]
786.290 μs (5%)

2.84 KiB (1%)
54

--- https://github.com/JuliaFolds/Transducers-data/blob/multi-thread-benchmark-results/2021/03/15/010146/result.md

The performance is nice, but it's strange that reduce is much better than foldl, even with no multithreading:
julia> suite = include("benchmark/multi-thread/bench_splitby.jl")
       results = run(suite; verbose = true)
(1/1) benchmarking "count"...
  (1/3) benchmarking "foldl"...
  done (took 6.10017195 seconds)
  (2/3) benchmarking "reduce"...
  done (took 5.655999351 seconds)
  (3/3) benchmarking "man"...
  done (took 5.316228889 seconds)
done (took 17.61219897 seconds)
1-element BenchmarkTools.BenchmarkGroup:
  tags: []
  "count" => 3-element BenchmarkTools.BenchmarkGroup:
          tags: []
          "foldl" => Trial(3.618 ms)
          "reduce" => Trial(1.788 ms)
          "man" => Trial(1.473 ms)

julia> Threads.nthreads()
1

then with more threads:
1-element BenchmarkTools.BenchmarkGroup:
  tags: []
  "count" => 3-element BenchmarkTools.BenchmarkGroup:
          tags: []
          "foldl" => Trial(3.611 ms)
          "reduce" => Trial(383.696 μs)
          "man" => Trial(1.440 ms)

julia> Threads.nthreads()
4


Jesse: Watch apkvent.com

Jesse requests the watch of the watch_keyword apkvent\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

apkvent\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Mast in Charcoal HQ


Android fix

Summary
Fixes android 13 notification permission request and unsigned builds
NONE

/cherry-pick release-2.0

Cherry pick is scheduled.


midonet/midonet does not use pull requests.

Please do not send pull requests to midonet/midonet.
We appreciate all your contributions. We gladly take them but
we're accepting patches on GerritHub, hosted Gerrit integrated
with GitHub.
See the following page fore more details:

http://wiki.midonet.org/How to contribute

Thanks!

Thank you for contributing to MidoNet! MidoNet uses Gerrithub for code review. Please visit http://wiki.midonet.org/How to contribute and follow the instructions there to upload your change to Gerrithub.


A long disabled node remains on the map

For some reason, a node that has been disabled for over a week remains displayed on the map.
The node is displayed on the map with the information "MQTT Status: Online (3 minutes ago)".
I don't understand why this is the case, the node has really been turned off for over a week.
ID:<PHONE_NUMBER>
Hex ID: !da5694ec
https://meshtastic.liamcottle.net/?node_id=3663107308

This is most likely happening when I restart the map server, or my server disconnects and reconnects to to the Meshtastic MQTT server.
When connecting to the Meshtastic MQTT server, it sends all of the persistent stat packets with online or offline for each node ID it has in memory.
These packets are used to update the MQTT status shown on the node info popup. So even if your node is completely offline, or no longer exists, the MQTT still sends these packets on server reconnection, and I update them in my database.
This also updates the last seen timestamp as well, which would be why it's changing. Last Seen is actually just the updated_at column from the database. It's not based on when the last radio packet was heard.
If you really want, I can just delete this node from the database, and if it comes back online at all, it will be recreated.


TINKERPOP3-670 tail step (v2)

Replaces PR #60
Based on feedback on the initial PR:

correctly (I hope) tested and implemented "bulk" support
ComputerVerificationStrategy balks at mid-traversal global tail steps
__ (anonymous traversals) include tail methods
deferred execution during MR (using Bypassing)

The hadoop-gremlin tests pass for me.

Sweet. Did you make sure SparkGraphComputer integration tests pass? If that passes, Giraph will.

Yes, I see both SparkGraphComputerProcessIntegrateTest and GiraphGraphComputerProcessIntegrateTest mentioned in the test output.  I'm using the GREMLIN_TESTS env var to run only the tail and range tests.  Most of them are skipped by ComputerVerificationStrategy, but the one that you asked me to add does run on both Spark and Giraph:
...
Testing: g_V_repeatXbothX_timesX3X_tailX7X
...
Tests run: 18, Failures: 0, Errors: 0, Skipped: 12, Time elapsed: 24.016 sec - in org.apache.tinkerpop.gremlin.hadoop.process.computer.giraph.GiraphGraphComputerProcessIntegrateTest
...
Testing: g_V_repeatXbothX_timesX3X_tailX7X
...
Tests run: 18, Failures: 0, Errors: 0, Skipped: 12, Time elapsed: 12.464 sec - in org.apache.tinkerpop.gremlin.hadoop.process.computer.spark.SparkGraphComputerProcessIntegrateTest

Also, the same selected tinkergraph-gremlin tests still pass.
Tests run: 18, Failures: 0, Errors: 0, Skipped: 12, Time elapsed: 1.115 sec - in org.apache.tinkerpop.gremlin.tinkergraph.process.TinkerGraphProcessComputerTest
...
Tests run: 18, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.138 sec - in org.apache.tinkerpop.gremlin.tinkergraph.process.TinkerGraphProcessStandardTest

@okram Let me know if there's anything else you want on this PR.

Can you merge master/ into your branch and then provide a PR. I will then merge.

@okram Replaced by PR #64


Minor updates

:tada: This PR is included in version 3.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Bug]: Execution Information location doesn't resolve variable

Apache Hop version?
2.6.0-SNAPSHOT (2023-08-01 11.57.36)
Java version?
Java 11
Operating system
Windows
What happened?
The "Execution Information location" with file location  doesn't resolve variables.

The root folder selector indicates that you can use variables, but if so, the folder created contains the variable name:
C:\hop\${TEST}\executions
Issue Priority
Priority: 3
Issue Component
Component: Metadata

Sorry this variable should be set when lauching Hop, not in Configuration perspective > System variables


Fix gradle to build without Mattermost keys as dev

This will allow any dev to build the apk in debug mode without complaining that the ENV does not have the keystore registered

closing and re-opening to trigger jenkins


Add serde support

The AMD SEV types that are relevant for attestation should support serde so that we can pass these structures as CBOR-encoded messages.
I've already done the brunt of this work as part of my work on the remote AMD SEV attestation, but I'm opening this issue to track that work.
Depends on #27

Closed in https://github.com/enarx/sev/pull/33


Add https://lgtm.co

Pull Requests are reviewed by members of the team specified in the MAINTAINERS file in your repository and cannot be merged until all approvals are received.

Approvers comment on the issue with LGTM, looks good to me, indicating their approval to merge the changeset.

This looks very useful for this project, this means without approval of the core maintainers no PR can be merged.
@ai @marcustisater do you want to be in the MAINTAINERS file? You'd have to comment each and every PR then with LGTM though.

Nope :). But I anyway will be around :).

@mxstbr You can customize pattern and accept :+1: or anything ;)

Sure, I think @MoOx could be a good asset too if he has time for it.
Isn't it good enough if one of the members accepts it, why all? It seems like a lot of work and waiting...

Mainly so my mistakes don't slip through cracks, but if I'm the only one in the file it doesn't really make sense to have it. :)

You can configure who many I think.

That's true, maybe we can add some more "validators" that has enough experience to approve code.
@jonathantneal , @stephenway ?

I think it's enough if two people in the MAINTAINERS file approves the PR before merge.

I like that!

Sounds great, count me in for that.

I can help if we can define me as "optional" (I mean not required all the time)

@ai can you either add me to the postcss team or enable LGTM for this repo on https://lgtm.co?

I am in the city. Can I add it today evening, when I will be near my laptop?

Yeah definitely, don't worry about it!
2016-01-08 16:52 GMT+01:00 Andrey Sitnik<EMAIL_ADDRESS>
I am in the city. Can I add it today evening, when I will be near my
laptop?
—
Reply to this email directly or view it on GitHub
https://github.com/postcss/postcss.com/issues/34#issuecomment-170037311.

@ai I just took the liberty to handle this. @mxstbr have been invited.
Btw, github ask to migrate the org to new roles and shit, so I did that. Note that for now (as it was before) everybody is an owner. Trust is key imo :)

@MoOx thanks, LGTM.co is now enabled for this repo!


Relative path for mprocs-pty

Trying to install this using cargo install --git https://github.com/pvolok/virterm and I'm getting an error regarding mprocs-pty:
    Updating git repository `https://github.com/pvolok/virterm`
  Installing virterm v0.0.1 (https://github.com/pvolok/virterm#6570fd0f)
    Updating crates.io index
error: failed to compile `virterm v0.0.1 (https://github.com/pvolok/virterm#6570fd0f)`, intermediate artifacts can be found at `/tmp/cargo-install8OEyeY`

Caused by:
  failed to get `mprocs-pty` as a dependency of package `virterm v0.0.1 (/home/mrus/.cargo/git/checkouts/virterm-2ad141bd6c9d184a/6570fd0)`

Caused by:
  failed to load source for dependency `mprocs-pty`

Caused by:
  Unable to update /home/mrus/.cargo/git/checkouts/virterm-2ad141bd6c9d184a/mprocs/vendor/pty

Caused by:
  failed to read `/home/mrus/.cargo/git/checkouts/virterm-2ad141bd6c9d184a/mprocs/vendor/pty/Cargo.toml`

Caused by:
  No such file or directory (os error 2)

It seems the Cargo.toml in this repo is specifying mprocs-pty using a local, relative path:
https://github.com/pvolok/virterm/blob/6570fd0f2580ee79798b7fb05f747784b76a718c/Cargo.toml#L53

Sorry for a long reply. I pushed a fix. Now installation from git is working.


exception was thrown: NoSuchMethodError

I have error "Another exception was thrown: NoSuchMethodError: The method '[]' was called on null." . Help me

mee to

This should be fixed!


Header Navigation: how to scroll thumbnails

I've replaced the social media icons with thumbnails of my slides and edited base.css to tighten spacing. Currently approx 9 thumbs are visible along the x axis within the header nav and I do not want them to wrap or take up more vertical space. My question is how to scroll the thumbnails bar horizontally when the mouse hovers over the last visible thumb or near the top right edge of my screen. Within CSS overflow-x:scroll is not working.

Hi @LizMyers, probably is a flexbox thing. Try something like this.  I hope this will help you.


typo (kind of)

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/python/unit-testing-python-in-visual-studio.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@nicolaiarocci thanks for fixing this!


Breaking Change on STAC Item datetime field

Hi
Thanks a lot for this cool tool-kit.
I seem to have problem with STAC Item datetime field serialization. DATETIME_RFC339 introduced in stac_fastapi/sqlalchemy/stac_fastapi/sqlalchemy/serializers.py is causing timestamp format conversion error. Applying a temporary fix of assigning DATETIME_RFC339 = "%Y-%m-%dT%H:%M:%S.%fZ" solves the issue. The RFC339 standard is missing %f portion of datetime. I'm not sure whether use of the standard in test files has an impact, but I changed all occurrence of RFC339 to my temporary fix to be on the safe side!

I believe using pydantic's datetime parser should fix this instead of a single hard-coded datetime pattern.  We made the same change to datetime parsing in stac-pydantic for similar reasons (https://github.com/stac-utils/stac-pydantic/pull/75).

Thanks Jeff
Sounds promising that you already have an idea for a sustainable fix.

Hi @geospatial-jeff
What do you think about python-dateutil?
dateutil.parser.parse function takes care of date format conversion!

I think python-dateutil is fine, used it before in other projects.  But I'd prefer using pydantic's datetime parser in this situation to be consistent with the validation logic in stac-pydantic.  Overall I don't think either python-dateutil or pydantic are what we will settle on long term, there is more context in #161

I just added this to the STAC API impl guide also https://github.com/radiantearth/stac-api-spec/blob/master/implementation.md#datetime-parameter-handling

Also, this related issue https://github.com/stac-utils/stac-fastapi/issues/161

I just came across ciso8601. It looks to be both faster and more popular than pyiso8601

I just ran my test datetime values against it ciso8601, and only failed when it should have passed https://github.com/closeio/ciso8601/issues/110

I ran it against a few others:
ciso8601
Error: 1985-04-12T23:20:50,52Z did not parse. UTC offset is mandatory in RFC 3339 format.

arrow
Error: 1985-04-12t23:20:50.000z did not parse. Could not match input '1985-04-12t23:20:50.000z' to any of the following formats: YYYY-MM-DD, YYYY-M-DD, YYYY-M-D, YYYY/MM/DD, YYYY/M/DD, YYYY/M/D, YYYY.MM.DD, YYYY.M.DD, YYYY.M.D, YYYYMMDD, YYYY-DDDD, YYYYDDDD, YYYY-MM, YYYY/MM, YYYY.MM, YYYY, W.
Error: 1985-04-12 parsed, but should not have. result: 1985-04-12T00:00:00+00:00
Error: 1937-01-01T12:00:27.87+0100 parsed, but should not have. result: 1937-01-01T12:00:27.870000+01:00
Error: 1985-12-12T23:20:50.52 parsed, but should not have. result: 1985-12-12T23:20:50.520000+00:00

pendulum
Error: 1985-04-12t23:20:50.000z did not parse. Unable to parse string [1985-04-12t23:20:50.000z]
Error: 1985-04-12 parsed, but should not have. result: 1985-04-12T00:00:00+00:00
Error: 1937-01-01T12:00:27.87+0100 parsed, but should not have. result: 1937-01-01T12:00:27.870000+01:00
Error: 1985-12-12T23:20:50.52 parsed, but should not have. result: 1985-12-12T23:20:50.520000+00:00
Error: 1985-04-12T23:20:50.Z parsed, but should not have. result: 1985-04-12T23:20:50+00:00
Error: 1985-04-12T23:20:50,Z parsed, but should not have. result: 1985-04-12T23:20:50+00:00

iso8601
Error: 1985-04-12t23:20:50.000z did not parse. Unable to parse date string '1985-04-12t23:20:50.000z'
Error: 1985-04-12 parsed, but should not have. result: 1985-04-12 00:00:00+00:00
Error: 1937-01-01T12:00:27.87+0100 parsed, but should not have. result: 1937-01-01 12:00:27.870000+01:00
Error: 1985-12-12T23:20:50.52 parsed, but should not have. result: 1985-12-12 23:20:50.520000+00:00

for arrow, pendulum, and iso8601, non of them handle lowercase t or z (which is easy to remedy just by uppercasing the string).
None of them also have an RFC 3339 specific parser, so they accept strings that are ISO8601 but not RFC3339.
I'm most in favor of using ciso8601 and getting that one bug fixed with it.

and only failed when it should have passed

I'm guessing you meant "only one failed"?

I'm most in favor of using ciso8601 and getting that one bug fixed with it.

Cool; I also made a PR there for building binary wheels on releases (https://github.com/closeio/ciso8601/pull/109); if accepted it should enable installing in environments where a C compiler isn't installed.

lol, yes -- only that one, and an uncommon one at that.

So, that RFC 3339 datetime with the comma as a separator is actually invalid -- for some reason, the RFC 3339 spec has an ISO 8601 ABNF in it, and I didn't notice that that's what it was when I created these examples.

@alexamici ciso8601 dependency was removed in 2.4.1


Path Payments Error when issuing new tokens on the stellar wallet.

If you have an issuing account of a token, key base will not let you create more of that token by paying someone with it.
For example I have this issuing account and an account that has trusted it.

But if I actually try to send the account money this error path payments error shows up, even though I am paying directly in and sending one asset.

The same error happens in the cmd line.

However if you try to make these transactions with the stellar transaction builder

and then sign and submit them to the network it works fine.

This seems somewhat curious because If I try path payments with other direct assets for instance (AnchorUSD->AnchorUSD), it works fine.

It would probably be easier for you to have one issuer account and one distributor account, set up orders from your distributor account.


Feature: Incremental rewriting

Omnitrace could have an option to do binary rewrite on object files, and an option to intercept a compiler invocation to rewrite the file after it was compiled similar to how ccache or distcc work.
I'm guessing the linker will not have problems with linking instrumented functions.
This would allow for a workflow where only changed files have to be re-instrumented, potentially significantly cutting down on the time it takes to get a new trace after tweaking an application.
With projects in cmake using an option like this would be as easy as setting CMAKE_COMPILER_LAUNCHER.

I'll think about this and get back to you.
An alternative might be to instead just provide a CMake target which adds -finstrument-functions flags to the target and provides a library with the instrumentation functions. The trade-off here is the decreased control over which functions get instrumented.

@Maetveis Do you still need assistance with this ticket? If not, please close the ticket. Thanks!


Increment version for core releases

Increment package version after release of azure-core-client

Can we add core-xml to this PR or do we want to do that separately?

Can we add core-xml to this PR or do we want to do that separately?

I can add it in this PR.


Meta data caching batch query issue

Fixes issue #375 and issue #374

Using the test repro raised by @thomek,
The test is doing the following operation:
pstmt.setint(1,0);
pstmt.setNString(2,"a");
pstmt.addBatch();

pstmt.setint(1,1);
pstmt.setNString(2,"b");
pstmt.addBatch();

pstmt.setint(1,2);
pstmt.setNString(2,"cc...");
pstmt.addBatch();

pstmt.setint(1,3);
pstmt.setNString(2,"d");
pstmt.addBatch();

pstmt.setint(1,4);
pstmt.setNString(2,"e");
pstmt.addBatch();

this is what the profiler shows before this pr:
exec sp_executesql N'insert into TEST_TABLE values (@P0,@P1)                ',N'@P0 int,@P1 nvarchar(4000)',0,N'a'

declare @p1 int
set @p1=1
exec sp_prepexec @p1 output,N'@P0 int,@P1 nvarchar(4000)',N'insert into TEST_TABLE values (@P0,@P1)                ',1,N'b'
select @p1

declare @p1 int
set @p1=2
exec sp_prepexec @p1 output,N'@P0 int,@P1 nvarchar(max)',N'insert into TEST_TABLE values (@P0,@P1)                ',2,N'cccc.....'
select @p1

--->duplicate insert happens here
declare @p1 int
set @p1=3
exec sp_prepexec @p1 output,N'@P0 int,@P1 nvarchar(4000)',N'insert into TEST_TABLE values (@P0,@P1)                ',4,N'e'
select @p1

Result of profiler after the changes made:
exec sp_executesql N'insert into TEST_TABLE values (@P0,@P1)                ',N'@P0 int,@P1 nvarchar(4000)',0,N'a'
declare @p1 int
set @p1=1
exec sp_prepexec @p1 output,N'@P0 int,@P1 nvarchar(4000)',N'insert into TEST_TABLE values (@P0,@P1)                ',1,N'b'
select @p1

declare @p1 int
set @p1=2
exec sp_prepexec @p1 output,N'@P0 int,@P1 nvarchar(max)',N'insert into TEST_TABLE values (@P0,@P1)                ',2,N'ccc...'
select @p1

declare @p1 int
set @p1=3
exec sp_prepexec @p1 output,N'@P0 int,@P1 nvarchar(4000)',N'insert into TEST_TABLE values (@P0,@P1)                ',3,N'd'
select @p1

exec sp_execute 3,4,N'e'

The issue is when we hit the error message of "could not find prepared statement with handle x", we don't reset the batch parameters and reset the preparedTypeDefinitions

Codecov Report

Merging #393 into dev will increase coverage by 0.1%.
The diff coverage is 83.33%.

@@             Coverage Diff             @@
##                dev     #393     +/-   ##
===========================================
+ Coverage     40.12%   40.22%   +0.1%
- Complexity     1890     1895      +5
===========================================
  Files           107      107
  Lines         24482    24486      +4
  Branches       4038     4039      +1
===========================================
+ Hits           9823     9850     +27
+ Misses        12820    12791     -29
- Partials       1839     1845      +6

Flag
Coverage Δ
Complexity Δ

#JDBC41
40.09% <83.33%> (+0.09%)
1886 <0> (+4)
:arrow_up:

#JDBC42
40.09% <83.33%> (+0.09%)
1890 <0> (+8)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...oft/sqlserver/jdbc/SQLServerPreparedStatement.java
43.29% <83.33%> (+0.85%)
140 <0> (+4)
:arrow_up:

...n/java/com/microsoft/sqlserver/jdbc/DataTypes.java
76.72% <0%> (-0.17%)
4% <0%> (-1%)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
49.14% <0%> (-0.15%)
210% <0%> (ø)

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
45.1% <0%> (+0.03%)
268% <0%> (+1%)
:arrow_up:

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
44.97% <0%> (+0.1%)
0% <0%> (ø)
:arrow_down:

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
39.79% <0%> (+0.13%)
0% <0%> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
27.82% <0%> (+0.16%)
190% <0%> (-1%)
:arrow_down:

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
47.73% <0%> (+0.22%)
64% <0%> (+1%)
:arrow_up:

...n/java/com/microsoft/sqlserver/jdbc/Parameter.java
37.44% <0%> (+0.86%)
46% <0%> (ø)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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

Hi @rPraml , Thank you for submitting pr #374 for the valuable repro of this bug. Since that one was opened to our master branch, I went ahead and added your pr to this commit.


regex named segments

It'll be a nice little addition to have named regex segments, for ex.
new UrlPattern(/^\/user\/(\d+)\//i, ["user_id"])

good idea !
i'll implement it.
what do you think about the following edge case behaviour ?
if there are more groups in the regex than there are names in the array that is the second argument:
ignore captured values for the remaining superfluous groups.
if there are less groups in the regex than there are names in the array that is the second argument:
ignored the remaining superfluous names.
shouldn't break backwards compatibility - which is nice.

:thumbsup: however don't you think it'll be more expressive to not just ignore but:

if there are more groups in regex, then resulting object will take additional numeric keys with the values of corresponding regex group;
if there are less groups in regex, then excessive keys get undefined?
:wink:

i now think we should consider it a user error if the number of groups and names in the array don't match up.
i would like to throw an error in such cases to notify the user of bad usage.
better to blow up on that set of user bugs that to let them go unnoticed.
i might use http://stackoverflow.com/a/16046903 to detect the number of groups and be able to check for such errors as soon as possible i.e. on pattern creation and not on match (callsites of .match tend to be far away from pattern creation).
what do you think ?

sure, sounds good.

this is now published in 0.10.0.
click here for the relevant readme section.
there are some breaking changes. click to see the changelog entry.


the simics-uefi 'qsp.simics_uefi' has no attribute 'video_mode'

I test the platform BIOS fuzzer, it‘s report :
Traceback (most recent call last):
  File "<qsp-uefi-custom.target.yml.include:14>", line 1, in <module>
AttributeError: <the simics-uefi 'qsp.simics_uefi'> has no attribute 'video_mode'
[/home/gandalf/simics/tsffs/examples/tutorials/edk2-simics-platform/project/targets/qsp-x86/qsp-uefi-custom.target.yml.include:14] error in '@' command
[/home/gandalf/simics/tsffs/examples/tutorials/edk2-simics-platform/project/fuzz2.simics:12] error in 'load-target' command
Error - interrupting script.

I'll take a look -- for now, you can probably just delete that line.

It's ok.


Publisher & Store SSO UX issue

Login to Publisher app via SSO does not create a valid session (access token to Store REST API) in Store.
How to reproduce:

Login to publisher UI app
Navigate to Store app , It will show a sign in button and use will have no valid access token in their browser storage

Not a bug 😃


Cannot find module 'graphql-request' or its corresponding type declarations.

Screenshot VS Code

Screenshot npx tsc --noEmit

Description
When installing 7.0.1 typescript fails. 6.1.0 works as expected.
I have tried with both pnpm and npm. I haven't tried with yarn.

Node v20.11.0
Npm 10.5.0
Pnpm 9.1.2
Apple M1 Pro
MacOS Sonoma 14.5 (23F79)

tsconfig.json
{
  "compilerOptions": {
    "module": "NodeNext",
    "moduleResolution": "NodeNext",
    "declaration": true,
    "removeComments": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "allowSyntheticDefaultImports": true,
    "target": "ES2021",
    "sourceMap": true,
    "outDir": "./dist",
    "baseUrl": "./",
    "incremental": true,
    "skipLibCheck": true,
    "strictNullChecks": false,
    "noImplicitAny": false,
    "strictBindCallApply": false,
    "forceConsistentCasingInFileNames": false,
    "noFallthroughCasesInSwitch": false
  }
}

(I have tried with other tsconfig setups, but they all fail.)
Reproduction Steps/Repo Link
Repo: https://github.com/tofsjonas/graphql-request-test
Install and run

Ok, so if I set  "type":"module" in package.json it works.
Everything ELSE breaks though... 😭

@tofsjonas unfortunately not, this is an ESM project. Are you stuck in the CJS world? https://github.com/jasonkuhrt/graphql-request?tab=readme-ov-file#typescript-setup

Haha, well, I wish I wasn't.
The repo I shared was a completely fresh nest.js project. (Sure, nest.js doesn't have as many downloads as graphql-request, but it's still a significant package)
If I add "type":"module" to this tiny, tiny hello-world project I get

(In my actual project, I have hundreds of files with thousands of imports.)
To fix it I have to go into every single file that imports from another file and alter
import { AppService } from './app.service';

to
import { AppService } from './app.service.js';

Either that or figure out some bundler magic with babel or what have you.
It would be so, so very nice not to have to do that... 🥺🐶

@tofsjonas if you use "moduleResolution": "bundler" in .tsconfig.json you will not have to add file extensions
https://www.typescriptlang.org/tsconfig/#moduleResolution
I have gotten that far in my project, but now I'm trying to get Jest working with ESM.. So I agree this change brings a challenge for some established projects.

"moduleResolution": "bundler" fixes that particular problem, thanks for that (I was sure I had tried that already, but I guess I messed it up somehow)
Now I just have to get the ci pipeline to work again. But I guess that can wait until Monday..
Thank you both for your time! 🙏

This poll might be relevant for you https://github.com/jasonkuhrt/graphql-request/discussions/863

Er, I take it back.
When using "moduleResolution": "bundler" I didn't get any errors when running linting and tsc, so I moved on with my life. But when I tried to actually run the project I got the same error as before
Cannot find module '/Users/jonas/graphql-request-test/dist/app.module'
imported from /Users/jonas/graphql-request-test/dist/main.js

If I add .js to the import it works as expected, but I can't do that in my actual project 😞

Let's consider the simple example of a Node.js Typescript project that uses both pg and graphql-request.
Setting "type": "module" in package.json breaks the pg import (and possibly every other library import except graphql-request), and setting "moduleResolution": "bundler" in tsconfig doesn't seem right for a Node.js project (nor did it resolve this issue for me).
I have a lot of trouble navigating the ESM vs CommonJS space, but from a UX perspective this is the only library in my server-side project that can't be imported using the vanilla tsc --init tsconfig for server side projects. This is quite a hard issue to research on google.
What would you do in this case?

Eventually this combination worked from me:
tsconfig.json
{
  "compilerOptions": {
    "lib": ["ES2019"],
    "module": "ES6",
    "target": "ES6",
    "esModuleInterop": true,
    "moduleResolution": "bundler",
  }
}

It's annoying that v7.0 removed commonjs support since there's little reason to use ESM on Node and 99.9% of libraries use commonjs or are compatible with commonjs. This library is the first library I've encountered for use on the server that dropped commonjs support which is already a seldom phenomenon on client-only libraries alone.
Anyways, here's how you can migrate your Typescript + Node application from commonjs to ESM:

package.json: Add top-level key pair "type": "module"
tsconfig.json: Set "module": "Node16" and "moduleResolution": "Node16" ("target" can stay as "EsNext")
code: Add .js extensions to all of your local module imports. For example, change import x from "./x" to import x from "./x.js". Don't try to get .ts extension to work, it's not worth it.
code: Add "node:" prefix to Node stdlin imports like import stream from "node:stream". (not actually required since TS already knows how to resolve them)
quirk: a lot of major libraries don't let you destructure their imports anymore once you're using ESM. one example is pg (node-postgres). you have to change import {Client} from "pg" to import pg from "pg" and then use pg.Client in your code.

I think that's it.

could be a good lightweight library, shame

Going to lock this issue now to keep my notification noise down. If you have comments about the library you can leave that in the discussions forum. There is also a poll I'm looking at occasionally to help gauge the impact of not support CJS.


Order streams by download size (large to small)

Closing in favor of https://github.com/sleeyax/stremio-easynews-addon/issues/15


[WIP] [feature] Client Side Profile URL Validation

Replaces/Continues #4411

Purpose

Currently, if a user enters and submits an invalid URL in their profile, the server will respond with a 400 error,  and their website field is not saved. However, the user is unaware of this as there is no client-side URL validation, and no error message is presented to the user.

Changes

This PR adds client-side URL validation so that if a user attempts to enter an invalid website, they will receive an error message which instructs them to enter a valid website. Additionally, if the user attempts to save their profile with an invalid website, they will be presented with another error message that directs them to fix the website field.

Error message when entering invalid URL (no previous error message was displayed):

Error message when saving profile with invalid URL (previously, "Could not update settings" was displayed):

JIRA

OSF-3190

Could you add screen shots of the old and new functionality?

@brianjgeiger Screenshots added! :camera:

When your first go into the social tab, if you don't have a website, it complains about the lack of a valid website (and gives you a not-particularly-useful X to remove the website). It should only do the validation if you make a change to the website field, or more likely if the website field is not blank.

I'm having some other odd problems which I expect you would have seen, so the above might be because webpack isn't webpacking. Let's see.

Hmmm... I'm not seeing the error message upon entering the social tab. I'm only receiving the message when I hit enter after typing in some nonsense.
Also, the highly-confusing X on the side of the website field is always present, and is unrelated to this PR.

Okay, actually running webpack seems to have fixed the blank website erroring issue, so please ignore.

Looking good. I think the one space and making testing more extensive and matchy will do it.

Looks great. :tada: :penguin: :tada:

(Pending travis, not counting that one JS test that Steve is fixing)


Sourcemaps

It would be really cool if pug generated sourcemaps that allowed me to edit pug files in chrome

This could also allow using better linting. Right now I'm using codelyzer with tslint and because of pug, I'm not getting correct line numbers.


LeakSanitizer usage is not detected automatically / is not enabled together with asan

Describe the bug
In order to use absl::IgnoreLeak, one still has to define LEAK_SANITIZER when building abseil.
See leak_check.cc
Release notes for abseil 20200923 say that additional macros are no longer needed:

Sanitizer usage is now automatically detected and no longer requires macros like ADDRESS_SANITIZER to be defined on the command line.

Given that leak sanitizer is a part of asan and is enabled by default on Linux, I think it makes sense to enable leak sanitizer in asan builds.
Steps to reproduce the bug

Build abseil in asan mode with -fsanitize=address.
Build and run the following example:

#include "absl/debugging/leak_check.h"

int main()
{
    int * qwe = absl::IgnoreLeak(new int(10));
    return *qwe;
}

Observe leak:

=================================================================
==65425==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 4 byte(s) in 1 object(s) allocated from:
    #0 0x4c558d in operator new(unsigned long) /home/fedor.kalugin/rpmbuild/BUILD/llvm10-10.0.1/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cpp:99:3
    #1 0x4c7ead in main /src/utilities/rmme/src/main.cpp:5:34
    #2 0x388841ed1f in __libc_start_main (/lib64/libc.so.6+0x388841ed1f)

SUMMARY: AddressSanitizer: 4 byte(s) leaked in 1 allocation(s).

Expected result: no output.
What version of Abseil are you using?
20200923, master
What operating system and version are you using
RHEL 6, RHEL 7
What compiler and version are you using?
clang version 10.0.1<EMAIL_ADDRESS>85b3486b6e341621048d50dba2be78ccb7535d22)
Target: x86_64-unknown-linux-gnu
Thread model: posix

What build system are you using?
Cmake

Certains choses c’est compliqué


Not finding some services

Sometimes, it doesnt find all services I have on. I double checked they were on  with with Safari bonjour. After closing zeroconf and opening it several times, it found it.
This happens very often.
Zeroconf 1.2.4 -  Android 7 - ionic 3.3.0

Please feel free to review the code. have a look at the underlying libraries too.
I don't have much time right now for further investigations

Please upgrade to 1.2.5 and try


115 refactor main

Is the switch  missing the default case ?  Also , in the workflow should the lines 30-39  be kept? maven compile and maven test .

There is a compilation problem due to the missing parts in the MainTest:
line 118-128
@test
@DisplayName("getInputList Should Return List With Correct Data Based On Index")
test is missing closing bracket "}"
line 130
test is missing starting @test
(void callExecuteCommandWithIncrShouldIncreaseTheValueOfTheKeyBy1())
Missing imports (wrongly deleted):
import java.io.ByteArrayOutputStream;
import java.io.PrintStream;
Redundant imports
import org.junit.jupiter.api.BeforeEach;


Fix typo in documentation of size of giant pages.

Thank you! I'm really enjoying playing with the blog os. I snuck ahead and got the keyboard working. :)

Awesome! Did you use the PIC or the APIC?

PIC, because I found that documentation first. I'll check out the APIC.
On Mon, Jul 16, 2018 at 12:47 AM Philipp Oppermann<EMAIL_ADDRESS>wrote:

Awesome! Did you use the PIC or the APIC?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-osdev/x86_64/pull/38#issuecomment-405170538, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAFLIXyyG_U6NWj6d01NgmBA4Vu7wd5hks5uHET0gaJpZM4VQR_q
.

I haven't found the time to try it with APIC yet either, so let me know if you get it working. I'm still unsure whether I should do a PIC and an APIC post for the blog or directly start with APIC.

Reading the APIC stuff, it looks like you have to touch the PIC as well (to
mask all interrupts and map them somewhere known to avoid spurious
interrupts that don't get masked.), so having some context on the PIC at
that point would be helpful.
On Mon, Jul 16, 2018 at 9:11 AM, Philipp Oppermann<EMAIL_ADDRESS>
wrote:

I haven't found the time to try it with APIC yet either, so let me know if
you get it working. I'm still unsure whether I should do a PIC and an APIC
post for the blog or directly start with APIC.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-osdev/x86_64/pull/38#issuecomment-405300749, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAFLIacvV9X4OYVNlTvKsOlFTy5NaqB3ks5uHLs9gaJpZM4VQR_q
.

As someone who recently went through that journey, I highly recommend starting with PIC before moving onto APIC/IOAPIC, because the complexity increases dramatically.

Thanks! I think I will also start with a PIC post then


Split up a polygon

Paste the following code into the editor over on https://pomax.github.io/custom-graphics-element/edit, then click the graphics element on the right to place polygon points. Any point you place can be moved by click/touch-dragging.
const points = [];
const colors = [`#ae40bf`, `#bf4a40`, `#99bf40`, `#40bf82`,];
const cm = 0.0000001;

function setup() {
  setSize(600, 400);
  setBorder(1, `black`);
  noGrid();
}

function draw() {
  setCursor(`none`);
  clear(`white`);

  // show the lines we'll use for splitting:
  const w = width / 2;
  const h = height / 2;
  setColor(`black`);
  line(w, 0, w, height);
  line(0, h, width, h);

  // Draw a split set of polygons, if we have any
  const set = split(points, w, h);
  set.forEach((points, i) => {
    try {
      // no need to be clever: it'd be roughly the same amount of code.
      if (i === 0) translate(10, -10)
      if (i === 1) translate(10, 10)
      if (i === 2) translate(-10, 10)
      if (i === 3) translate(-10, -10)
      setColor(colors[i]);
      plotData(points, `x`, `y`);
    } catch (e) { /* this prevents crashes when a segment crosses the origin */ }
    resetTransform();
  });

  // and draw our "real" polygon on top.
  if (points.length) {
    noFill();
    setStroke(`grey`)
    plotData([...points, points[0]], `x`, `y`);
    setStroke(`black`)
    points.forEach(p => point(p.x, p.y));
  }
}

function pointerUp(x, y) {
  // don't place points if the cursor's close enough to
  // another point already (so we don't place new points
  // when we let go of a point we're dragging/moving)
  if (currentPoint) return;

  points.push({ x, y });
  clearMovable();
  setMovable(points);
  redraw();
}

function split(coords, X, Y) {
  if (coords.length < 3) return [coords];
  coords = [...coords, coords[0]];

  const [left, right] = splitAlong(coords, `x`, X);
  const [tl, bl] = splitAlong(left, `y`, Y);
  const [tr, br] = splitAlong(right, `y`, Y);
  return [tr, br, bl, tl];
}

function splitAlong(coords, dim, threshold) {
  const lt = [], ge = [];

  for (let i = 1, e = coords.length, a = coords[0], b; i < e; i++) {
    b = coords[i];

    // prep: find the transitional value on the boundary
    const r = (threshold - a[dim]) / (b[dim] - a[dim]);
    const x = (dim === `x`) ? threshold : (1 - r) * a.x + r * b.x
    const y = (dim === `y`) ? threshold : (1 - r) * a.y + r * b.y
    const p = { x, y };

    // is there a less-than/greater-or-equal transition?
    if (a[dim] < threshold && b[dim] >= threshold) {
      lt.push(a);
      lt.push({ x: p.x - cm, y: p.y });
      ge.push(p);
    }
    else if (b[dim] < threshold && a[dim] >= threshold) {
      ge.push(a);
      ge.push(p);
      lt.push({ x: p.x - cm, y: p.y });
    }

    // if not, which bin do we put "a" in?
    else if (a[dim] < threshold) {
      lt.push(a);
    } else {
      ge.push(a);
    }

    // setup for next pair
    a = b;
  }

  // make sure to close our polygons
  lt.push(lt[0]);
  ge.push(ge[0]);

  return [lt, ge];
}

Of course, this can be improved by not running splitAlong three times, but instead running it once with four quadrant arrays that we bin into based on both x and y thresholds.

could you please verify what can i resolve to this PR

There is no PR here?


Inconsitencies in features between Composer and SDK

Maybe I get something wrong, but anyway, I guess we might need some clarification here.
In https://docs.microsoft.com/en-us/composer/introduction you state that the adaptive dialog is unique to Composer. But the SDK also offers adaptive dialogs https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-adaptive-dialog-introduction?view=azure-bot-service-4.0.

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

ID: fdf54a88-60d1-0f93-1b95-2a1adddaf51c
Version Independent ID: 6045ccbf-e79c-61b8-59ac-61dec2bd3f82
Content: Introduction to Bot Framework Composer  - Bot Composer
Content Source: composer-docs/introduction.md
Service: bot-composer
GitHub Login: @Kaiqb
Microsoft Alias: kamrani

Apologies for the confusion. Adaptive dialog is not unique to Composer.
Can you point out the content in the introduction that made you think adaptive dialog is unique to Composer? We can fix it. Thanks.

No problem, just checked the docs and wanted to reach out to you.
Please check the pragraph at https://docs.microsoft.com/en-us/composer/introduction#advantage-of-developing-bots-with-composer

**Developers familiar with the Bot Framework SDK will notice differences between bots developed with it and Bot Framework Composer. Some of the advantages of developing bots in Composer include:

Use of Adaptive Dialogs allows for Language Generation (LG), which can simplify interruption handling and give bots character.
The visual design surface in Composer that eliminates the need for boilerplate code and makes bot development more accessible. You no longer need to navigate between experiences to maintain LU model as it is editable within the app.
Time saved with fewer steps to set up your environment.

Hi, just let you know that we removed the content that you found confusing.
Closing this issue because it is fixed and the content is live.
Thanks!


Rinkeby support

It would be great if rinkeby is supported when switched to rinkeby I'm getting this error 🤔

/cc @dabit3

change the web3 modal mumbai to rinkeby after adding rinkeby in hardhat.config.js

It works thanks!

Can you please be specific about where to make this change? I have the rinkeby network setup in hardhat.config.js with my Infura rinkeby url and MetaMask wallet private key.


[FRG] - Expose a way to set the report data in the editor

This is more of a feature request, but I am needing a way to update, or set the JSON data rows inside of the report editor. So the scenario would be:

Create a new "blank" report
Click the "Edit report dataset" action
Paste in my JSON into a field.

If that becomes available, I think that the report data rows would also need to be included in the custom save() function that can be specified on the report configuration when you do a new FluentReportsGenerator(), so that the new data can be saved back to the database.
I hope that makes sense, please let me know if you need any clarification.

This ability has been added -- you can either create a new report using {blankReport: true} which will create a new report that is blank, and an empty dataset.   Or you can call the newBlankReport() function off the report generator to create a new blank report programatically.
In addition on the main report you can now click the "Data" button, and see/edit the data for the report; and you can toggle if the data is saved with the report.    If you run the report w/ no data set; it will use the reports data; if you run the report with new data the new data will be used and override the reports embedded data.


Working on Android but not on ios: Exception [NSPlaceholderDictionaryinitWithObjects:forKeys:count:]: attempt to insert nil object from objects[0]' was thrown while invoking getImageForFont on target RNVectorIconsManager with params (Ionicons,"\Uf1bd",30,4278190090,17)

I've seen the other problems on the issues section but there they suggest to follow the instructions. I did everything as in the manual but I get this error just on ios, on android it works perfectly.
 "dependencies": {
    "axios": "^0.18.0",
    "lodash": "^4.17.5",
    "native-base": "^2.4.1",
    "react": "16.3.1",
    "react-native": "0.54.1",
    "react-native-app-intro-slider": "^0.2.4",
    "react-native-datepicker": "^1.7.1",
    "react-native-fbsdk": "^0.7.0",
    "react-native-firebase": "^4.0.0",
    "react-native-image-picker": "^0.26.7",
    "react-native-navigation": "^1.1.444",
    "react-native-twitter-signin": "^1.0.2",
    "react-native-vector-icons": "^4.6.0",
    "react-redux": "^5.0.6",
    "redux": "^3.7.2",
    "redux-thunk": "^2.2.0"
  }

How can I solve this?

I deleted the file and repetead the steps and now it partially works but I'm getting another error now at initialization:

I think this has nothing to do with this library.  I ran the debugger and it fails after sending a request with axios:
/*Red screen on this line*/ return axios.get(config.backend_url + '/api/contributors/' + firebaseId)
    .then((userDataFetchResult) => {
      dispatch(setUserGeneralData(userDataFetchResult.data.data));
      dispatch(authOperationFinished());
      return userData;
    })
    .catch(err => {
      console.log(err);
      dispatch(authOperationFailed(err));

    });

I figured it out. It was because in a component I was setting an image with a null source which works in Android but not in ios giving the above error. Thank you very much and sorry