throwing source doesn't exist

Hey
would love to use this -
However I'm getting
 [ERROR] 2019-05-06T22:16:02.836Z +1.56s Error: /.../swagger.yaml doesn't exist
    at fs.readFile.then.catch (/...markdown-swagger/index.js:100:15)

Any idea on how to get around this?

Yes
Use this :)
https://github.com/syroegkin/swagger-markdown

This is better!
https://github.com/code-cy/docfast-js


fix(imagecard): add fullscreen callback

Summary

Fixes ImageCard fullscreen button

Acceptance Test (how to verify the PR)

Open Dashboard storybook and click on ImageCard fullscreen button

:tada: This PR is included in version 1.81.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-beta.9 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.81.3 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


How  to initialize network  weights

I want  to  transfer  pre-trained  network  weights to DDPG  critic  and  actor  network ,how  can I  do this in coach?

Closing due to no response.

I too am interested to know if there exists an option today to use Coach to continue to train an existing network.

Ping @galnov

You can save Coach checkpoints with the -s flag, and load those checkpoints for evaluation or to continue training by using the -crd flag. Loading weights of a model trained not using Coach is possible only if that model is an exact copy of the one used in Coach.

@galnov I'm using RL Coach as a library, how can I evaluate with a saved model in a checkpoint file?
I'm able to save a checkpoint file once I'm done with training now I want to load that file again and evaluate on a (slightly) different environment. But the agent is exactly the same.

@galleibo-intel  can I use GraphManager.restore_checkpoint to load trained model (structure & weights) to make test predictions (evaluate). Thanks!

Hi @nitsanluke,
Yes, you can definitely do that. Also, @shadiendrawis is now working on a change to ease the flow of working with Coach as a library.


Conversion between flywheel reading and RPM is guesswork

I've taken a few samples using the "Magnetic Counter" app on my phone, and comparing with the value (F) read from the XDream-bike.
Plotting 1/F against RPM:
0.000556,300
0.000119,68
0.000222,127
0.000714,400
0.000417,250
0.00025,140
5.75E-05,34
0.00002,15

I add a trendline to get:
y = 553578x+3.7723
or
RPM = 553578 / F + 3.7723

I wonder how consistent this is between different bikes.


cutils.decode_polyline taking quite a long time to decode.

Thank you for all of your hard work.
I'm evaluating different methods of decoding large polyline data and I found your project.
I've tried three things, thus far:  1. a hand written decoder, 2. polyline module, 3. pypolyline module.
In short, both the hand written decoder and the decoder provided by the polyline module return almost identical decoding times for each example.  (0.14s for the attached example)
!! REVISED
I re-ran the polyline.decode test and I'm getting roughly 0.04s to decode the same polyline data.
cutils.decode_polyline, however, produces times that are much longer. (4.65s for the attached example)
This was tested on two systems:
MacOS (x86), Python 3.9.12, pypolyline 0.2.78
AWS (x86), Python 3.9, pypolyline 0.2.78
Both produced very similar results.
Do you have any suggestions on increasing performance? (reducing the decode time)
long_polyline.txt

Hi Brad, your comment was cut off.

Yes, sorry.  I accidentally triggered the  button.  I've updated the comment with the polyline attached as a text file.

BTW, the polyline data in the original post is from Valhalla, so it has a precision of 6.

Processed 21502 points in 520 microseconds

Can you confirm that it was microseconds (μs ) and not milliseconds (ms)?
I've run a benchmark for the same number of coordinates on the polyline crate, and it decodes 21502 coordinates in around 620 ms with a precision of 6. That's 620,000 μs, or around 1000x slower than what you're saying the cpp version is doing.

Using pypolyline 0.3.0 (just released to PyPI) which incorporates a perf improvement I merged into the Rust library, I get an average decoding time of 2.69 ms for your test input, vs 1.57 s for the previous version. That's around a 600x speedup.

Stephan,  that fixed it!  Thank you!
Parsing time went from 4.4s to 0.005s.
Thank you for all your hard work!


Add a lookup entity extractor

Description of Problem:
Lookup tables are used to create additional features for our entity extractors. In order for the features (aka "the word is part of lookup table xyz, so it is likely that is has the entity xyz") to work, you need to have some examples in your training data with entires from the lookup table.
However, many users think that all entries in the lookup table are extracted as an entity right away. So, as soon as an entry from the lookup table is detected it is extracted as an entity. Many users on the forum struggling with this.
Overview of the Solution:

Create a new component LookupEntityExtractor that takes a lookup table. It will extract exact matches of entries in the lookup table and marks them as certain entity. (What about plural/singular/etc. of words? Should we also consider those?)
We should update the docs to clarify the difference between lookup tables and lookup entity extractor.

@tabergma this is an interesting idea -- i definitely think users are confused about this. Do you think a RegexEntityExtractor would similarly be helpful? I think some users have implemented it themselves already

@erohmensing Good point 👍 RegexFeaturizer could cause a similar issue, might make sense to also add an extractor for it.

@JiteshGaikwad if case you're interested, we'd be interested in a PR relating to your work here.

hey @koaning @tabergma @erohmensing  I have submitted the PR


Refactor convert package to public API

I think my only concern right now is support guarantees with exported functions in a pkg/ directory that clients depend on while we continue to iterate on this project.

As long as we stay at v0, we're good to make breaking changes on minor version bumps of this repo.

That way, it would be simple to chain fs.FS => RegistryV1 => Plain => fs.FS

Yeah, this seems super reasonable to me. Does the registry code around FBCs also have an intermediate representation layer too?


Provide exports.types in package.json

Related to #229, the package.json is not configured correctly. You can see the warning at<EMAIL_ADDRESS>- when exports is specified, the top level types is ignored.
The workaround in that issue does work, but ConnectKit + TypeScript is broken with the default create-next-app template, so I think it's worth fixing.
By my understanding, there are a couple possible resolutions here:

Update the exports field in the package.json to identify both the js and types file (the approach in this PR)
Rename the generated files so that the .d.ts file matches the name of the js file (i.e. index.es.d.ts)

I'm not an expert in getting this stuff set up, but I did test by building locally and linking into an existing project. I believe this issue also exists in connectkit-next-siwe, and I'm happy to make the same fix there if desired!

Interesting, I really appreciate the PR! I'm going to look into this a bit more to make sure this config is correct before merging. Thanks 🙏

of course, let me know if I can help!

Seeing Could not find a declaration file for module 'connectkit'using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Believe it's related to this package.json config. Is there any update on this badboy?

This PR is now available in ConnectKit 1.5.3 🙌
Thanks @alex-grover 🙏


Multiple Choice Widget has broken semantics

Expected behaviour

all label tags should be associated with a single input

Actual behaviour

multiple choice (label/radios) widget potentially contain a label that has no associative input.

Steps to reproduce the behaviour

start UKIS (1_0001.json)
navigate to question "1. General Business Information"
repeating radio fields use a label (eg 'Apple was established') where a fieldset is the correct semantic choice

Screenshot

Fixed here #872


Bring Python current in GHA

Signed-off-by: Gábor Lipták<EMAIL_ADDRESS>

@gliptak Seems like py3.10 has an issue installing python-snappy?

https://github.com/andrix/python-snappy/issues/113

I think the error lies in the test:
test_cannot_instantiate within dataprofiler.tests.profilers.test_column_profile_compilers.TestBaseProfileCompilerClass
Need to catch the case where method becomes methods between versions.


Algorithms Strings (Easy)

Solve Beautiful Binary String, The Love-Letter Mystery and Making Anagrams

I can do this in C++

I can do this in Python @adityabisoi .Please assign me this issue

I want to do this in Java.

@adityabisoi sorry for the delay. Have added all the 3 solutions and have updated the Readme.md as well. Please review my P.R. #593


ShellCheck: Enable and fix uncommon violations

These rules only had one or two violations and are fixed in this PR:

SC2002
SC2209
SC2220
SC2103
SC2119
SC2120
SC2126
SC2128
SC2007
SC1113
SC2021
SC1128
SC2154
SC2028
SC2039
SC2196
SC2043

/cc @mandarjog
/assign @mandarjog

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Tahler
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Tahler: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
87f798101bca10636ce0abad2d3bf37600c6ab2a
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Tahler
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: mandarjog
If they are not already assigned, you can assign the PR to them by writing /assign @mandarjog in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mandarjog, Tahler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mandarjog]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Tahler: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
23edf10b8cc7a207ed155681b3df5981c4ef9b93
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simple

@Tahler: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
23edf10b8cc7a207ed155681b3df5981c4ef9b93
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simple


Initial Blueprint for LG Resu Prime Battries

Hi Chris!
Sorry for such a late answer!
First of all it would be great of you rebase your branch on the latest commit of main branch (now the latest commit is 70f9b34ab78caedc65a545cea34f74375407fda5). After you push your commit, checks should run - you'll see some warnings that should be fixed before pushing to main branch.

Hi Chris! Sorry for such a late answer! First of all it would be great of you rebase your branch on the latest commit of main branch (now the latest commit is 70f9b34ab78caedc65a545cea34f74375407fda5). After you push your commit, checks should run - you'll see some warnings that should be fixed before pushing to main branch.

I rebased it from the latest marketplace but I dont see the warnings you are mentioning.

@muenchris now you can check all warnings: https://github.com/Enapter/marketplace/pull/63/checks


Was any sort of image pre-processing done prior to training?

i want to know if any external image pre-processing was done so as to get better results while training.
PS: i am trying to train the model on my custom dataset.

Okay understood.


⚠ ClamAV has degraded performance

In 340c821, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 331 ms

Resolved: ClamAV performance has improved in 951dffa.


[DEPS] upgrade crypto to address reported vulnerability

https://github.com/advisories/GHSA-v778-237x-gjrc

@gotjosh plz take a look, thanks


Include Pre-configured list of known app-compat apps

📝 Provide a description of the new feature
Allow applications to register themselves to be excluded, or included, in certain features.
Or alternatively allow a pre-configured list, managed by PRs, to be installed with PowerToys.
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
Currently some features target all application, e.g. the fancy zones.
Some applications are not compatible to such features, for Greenshot (of which I'm a developer) this is the case, see #4373.
You already have added a way for users to exclude applications, this might "solve" it from your side, but as this is not by default I am faced with more and more users who open issues with us.
As there are more applications which are confronted with this, it might help if you provide a way for applications to register themselves to be excluded or included in certain functionality. This should preferable be a well defined and stable way.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

@Lakritzator
we considered adding default apps to the excluded list, and we also considered the problem of maintaining the list.
But if the applications' developers are taking care of maintaining the entry for their app, I think we should definitely do it.

I can imagine what kind of issue it would be to maintain such a list, especially as it would only grow and maintainers of other projects might forget to remove the entry after the issue is solve... but from a process point of view I don't see much other solutions...
As for an "API", I don't know what options there would be to talk to PowerToys, besides the registry.
I wouldn't mind to add a certain registry key when installing and removing it when deinstalling my application, but it should be made in such a way that I cannot remove another application by accident. So having a list of values under a certain key, would probably make this possible.
But I wouldn't mind a discussion about other implementation solutions.

Found this... https://greenshot.atlassian.net/browse/BUG-2712
It resolved my issue.
I came here to "vote" for a solution that would help others save about 90 minutes of their day searching for a resolve after using microsoft's powertoys, which is pretty popular.

@DatabaseAsAService
Greenshot is listed in our list of apps that have compatibility issues https://github.com/microsoft/PowerToys/wiki/Application-Compatibility#screen-capture-software

closing this as a dup against #1614

Hi @crutkas, this is not a duplicate as this describes how to give applications the possibility to have themselves excluded without the need for users to do this.
I don't think it's in our users interest to find a bug, have to search the internet, and either report this with us or find that they manually need to exclude applications/windows because it's known that it doesn't work.
I didn't find anything in the other I issue about an application being able to add e.g Greenshot to the exclude functionality.

Sorry, I think we should rename this as second part of this (preconfigured list) and remove the implementation aspect of this in the title. I read the and instantly my brain snapped to a different issue.
Right now submitting a PR to the repro and having that exclude list filled out is a better solution, not an API. we start getting into a very easy way of breaking one another on updates.
what we need to figure out how to do then is for existing users, how we merge in this list

@enricogior, this is harder than i think due to how setting defaults are created in FancyZones, correct?

@crutkas
we can consider adding a second list that is independent from the user defined.


python skripts won't run (ImportError: DLL load failed: The specified module could not be found.)

I know this issue came up earlier, but all the recommended steps didn't help.

The correct python version 2.7.11 in 64 bit is installed
codesynthesis 4.0 is installed and added to path

But still i see the error message
D:\Malmo\Builds\Malmo-0.22.0-Windows-64bit\Malmo-0.22.0-Windows-64bit\Python_Examples>python tutorial_1.py
Traceback (most recent call last):
File "tutorial_1.py", line 22, in
import MalmoPython
ImportError: DLL load failed: The specified module could not be found.
Any ideas why this might still be an issue?
Best,
Henry

Have you installed the Microsoft VS2013 redistributable? eg step 6 here:
https://github.com/Microsoft/malmo/blob/master/doc/install_windows_manual.md
If that doesn't work, try running the C++ or C# samples - they sometimes give more helpful error messages than Python, and will usually tell you what is missing, rather than just complaining that something is missing...

I have deinstalled and reinstalled all the dependencies (several times now) and added their locations to the path variable.
When I’m trying to launch a java agent I get: A JNI error has occurred, please check your installation and try again.
Cpp is telling me: MSVCR120.dll is missing.
Csharp:
Unhandled Exception: System.TypeInitializationException: The type initializer for 'Microsoft.Research.Malmo.MalmoNETNativePINVOKE' threw an exception. ---> System.TypeInitializationException: The type initializer for 'SWIGExceptionHelper' threw an exception. ---> System.DllNotFoundException: Unable to load DLL 'MalmoNETNative': The specified module could not be found. (Exception from HRESULT: 0x8007007E)
at Microsoft.Research.Malmo.MalmoNETNativePINVOKE.SWIGExceptionHelper.SWIGRegisterExceptionCallbacks_MalmoNETNative(ExceptionDelegate applicationDelegate, ExceptionDelegate arithmeticDelegate, ExceptionDelegate divideByZeroDelegate, ExceptionDelegate indexOutOfRangeDelegate, ExceptionDelegate invalidCastDelegate, ExceptionDelegate invalidOperationDelegate, ExceptionDelegate ioDelegate, ExceptionDelegate nullReferenceDelegate, ExceptionDelegate outOfMemoryDelegate, ExceptionDelegate overflowDelegate, ExceptionDelegate systemExceptionDelegate)
at Microsoft.Research.Malmo.MalmoNETNativePINVOKE.SWIGExceptionHelper..cctor()
--- End of inner exception stack trace ---
at Microsoft.Research.Malmo.MalmoNETNativePINVOKE.SWIGExceptionHelper..ctor()
at Microsoft.Research.Malmo.MalmoNETNativePINVOKE..cctor()
--- End of inner exception stack trace ---
at Microsoft.Research.Malmo.MalmoNETNativePINVOKE.new_AgentHost()
at Program.Main()
Thank you for your help!

One more thing. the link in step 6 of the list doesnt work, so i downloaded it from another website...

Hmm, that broken link is annoying.
If MSVCR120.dll is missing then your attempt to install the VS2013 redist must have failed.
Try here?
https://support.microsoft.com/en-us/help/3179560/update-for-visual-c-2013-and-visual-c-redistributable-package

Thank you very much sir, the initial error message disappeared... But now I see this:
C:\Users\Heinrich Peters\Desktop\Malmo\Builds\Malmo-0.22.0-Windows-64bit\Malmo-0.22.0-Windows-64bit\Python_Examples>python tutorial_1.py
Error starting mission: Failed to find an available client for this mission - tried all the clients in the supplied client pool.
C:\Users\Heinrich Peters\Desktop\Malmo\Builds\Malmo-0.22.0-Windows-64bit\Malmo-0.22.0-Windows-64bit\Python_Examples>python run_mission.py
Error starting mission: FFMPEG not available
do you have an idea whats still wrong?

Really glad that worked!
Am closing this issue since the original problem is now solved, but feel free to open additional issues for new problems. (But as for the FFMPEG issue, my guess is that you've not installed FFMPEG. Please see the installation instructions. And the other error indicates that you don't have a Malmo Minecraft client running. The tutorial here  might be helpful, if you haven't already seen it.)


Buggy interaction with OS X native Sticky Keys

I use sticky keys (generic modifier oneshot and/or lock, basically) on OS X and it interact strangely with warpd – with a sticky key active that's used by my warpd config, warpd will behave correctly and the stickiness is cleared, but OSX's visual indicator of the sticky status stays around even though it is no longer active.
I've also encountered situations in which all input (Including mouse and keystrokes) become non-responsive (Sometimes to the point of requiring a hard reboot). I can provoke this with a sticky lock that is part of  a warpd activation key (ie,  C–S –. as warpd config, lock control, then press Shift + .)
You can enable sticky keys in System preferences  - Accessibility – keyboard. I can provide more details or a screen capture if it helps.

I use sticky keys (generic modifier oneshot and/or lock, basically) on OS X and it interact strangely with warpd – with a sticky key active that's used by my warpd config, warpd will behave correctly and the stickiness is cleared, but OSX's visual indicator of the sticky status stays around even though it is no longer active.

This is probably because warpd absorbs the activation key, so it never gets reported to the operating system. It would be possible to change this behaviour by allowing the event to propagate, but that would introduce the possibility of conflicting keyboard shortcuts. My advice would be to disable visual notifications for sticky keys altogether. If this really bothers you, it might be possible to fix with a kludge, but it would only apply to transiently activated modifiers (i.e ones which have been tapped once).

I've also encountered situations in which all input (Including mouse and keystrokes) become non-responsive (Sometimes to the point of requiring a hard reboot). I can provoke this with a sticky lock that is part of a warpd activation key (ie, C–S –. as warpd config, lock control, then press Shift + .)

warpd expects to receive bound keys in the exact state they are specified in the config.  With sticky keys it is easy to accidentally activate warpd with an active modifier which gets applied to all key sequences (especially with a broken visual indicator). This can be solved by deactivating the modifier and then pressing the exit key.

This is probably because warpd absorbs the activation key, so it never gets reported to the operating system. It would be possible to change this behaviour by allowing the event to propagate, but that would introduce the possibility of conflicting keyboard shortcuts.

Definitely don't want that.

My advice would be to disable visual notifications for sticky keys altogether. If this really bothers you, it might be possible to fix with a kludge, but it would only apply to transiently activated modifiers (i.e ones which have been tapped once).

I tried this, and it's very hard to use without the visual indicator. Also not sure about what's going on in the hood - even though the indicator gets out of sync, the sticky modifier is correctly cleared... How is that happening if warpd is not passing through to the OS?

warpd expects to receive bound keys in the exact state they are specified in the config. With sticky keys it is easy to accidentally activate warpd with an active modifier which gets applied to all key sequences (especially with a broken visual indicator). This can be solved by deactivating the modifier and then pressing the exit key.

Right but without Visual indicator it's hard to know whether modifiers active or not - I end up randomly mashing buttons, and sometimes this gets me out and sometimes seems to hard lock the display manager as described... in these situations, there's sometimes nothing on screen drawn by warpd (no grid or Mouse box) - can't tell if  there's another bug going on or not.

even though the indicator gets out of sync, the sticky modifier is correctly
cleared... How is that happening if warpd is not passing through to the OS?

It seems the visual indicator is handled higher up the stack. stickyness itself
seems to be baked into the low level input logic, but the visual indicator
seems to rely on events which get blocked before they reach the display server.

Right but without Visual indicator it's hard to know whether modifiers active
or not - I end up randomly mashing buttons, and sometimes this gets me out and
sometimes seems to hard lock the display manager as described... in these
situations, there's sometimes nothing on screen drawn by warpd (no grid or
Mouse box) - can't tell if there's another bug going on or not.

A solution to this might be to clear the active modifier set before starting warpd.
This would still prohibit the use of sticky visual indicators while warpd is active,
but would put you in a known state upon initialization. I am a bit preoccupied at
the moment, will experiment with this soon.

A solution to this might be to clear the active modifier set before starting warpd. This would still prohibit the use of sticky visual indicators while warpd is active, but would put you in a known state upon initialization.

That seems like the right direction

As it turns out, clearing the modifier set is non trivial since the coregraphics API doesn't have a mechanism to synthesize the necessary events. However after going down a few rabbit holes I was able to figure out what makes the sticky indicators tick and whitelist the requisite events. The end result is that visual indicators should now work as expected while warpd is active.

I'm closing this for now. Feel free to reopen the issue if you feel it is unresolved.


Using hydrogen from power to gas in transport sector results in strange dashboard values

As witnessed by John in this scenario, using hydrogen form power to gas in the transport results in:

increased bio-footprint
increased energy import
decreased renewability percentage

Which are all opposite to what you would expect.

I think the following might be happening:
In this scenario all transport technologies (except Domestic navigation technology) are electric.
So, no hydrogen is used in this sector.

increased energy import

If P2G for transport is used as a flexibility option this results in export of hydrogen. Due to 33.28% losses in this process, in this scenario energetically less Hydrogen is exported than was the case without the P2G conversion.

increased bio-footprint
increased energy import
decreased renewability percentage

P2G for transport is higher in the flexibility order than P2Heat. The hydrogen is exported and the heat that was used before is no longer produced with excess energy.
Without P2G

With P2G (slider setting 100,000)


Update REQUEST-903.9003-TYPO3-EXCLUSION-RULES.conf

typo3 constantly runs into this false positive

Good to hear from you @emphazer and thanks for reporting.
My typo3 plans have been stopped due to my partner being overly busy with other projects. Is this all that you see?

Well,
we are interested in the typo3 exclusion function.
We have a good ruleset which has been working in production for years,
however, it employs TAGs instead of IDs.


Go import invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v2

I want to upgrade the dependency of m13253/dns-over-https to v2.2.5 of my go package, however, I got:
$ docker run -it --rm golang:1.16-buster
root@426057fa6ca9:/go# go get -v<EMAIL_ADDRESS>go get<EMAIL_ADDRESS>invalid version: module contains a go.mod file, so major version must be compatible: should be v0 or v1, not v2

It's mainly because the current Go package module isn't compliant with the Go module hierarchy.
An obvious solution would be:

Change import path of github.com/m13253/dns-over-https to github.com/m13253/dns-over-https/v2
Need to change go.mod module path
Examples:
https://github.com/casbin/casbin
https://github.com/cespare/xxhash
https://github.com/google/martian
https://github.com/nats-io/nats-server

SEE ALSO
https://golang.cafe/blog/how-to-upgrade-to-a-major-version-in-go.html
https://archive.is/VohDN

Oh... This is a breaking change.
This software was created way before Go's new module system.
I don't understand why this happen, since Go always maintained backwards compatibility in the past, until this change.
I would be very happy to receive a PR, if you have time to fix this issue.

I believe that Go updated its module system(for semver) since 1.9.7+, 1.10.3+, and 1.11
https://github.com/golang/go/wiki/Modules#semantic-import-versioning
https://github.com/golang/go/wiki/Modules#releasing-modules-v2-or-higher
https://github.com/golang/go/wiki/Modules#how-are-v2-modules-treated-in-a-build-if-modules-support-is-not-enabled-how-does-minimal-module-compatibility-work-in-197-1103-and-111

I would be very happy to receive a PR, if you have time to fix this issue.

I'll do it ASAP if I'm available.

Thank you so much for helping me solve the issue!
I haven't been following the recent updates from Go official team for quite a long time.

You're welcome 😆, you may consider releasing a new tag(when the next release tag is stabilized), so I can refer to it(otherwise I can only use the latest HEAD commit).

you may consider releasing a new tag

Done!


Assembly support

I'd be quite interested in Assembly listing support in Prism.js - is anything like this on planning? Given assembly's nature it'd be the best to have a "base" assembly language plugin, so specific architectures can be forked from it.
The base assembly language would be like.. one instruction per line, comments, labels, and special annotation. From there on we can have a plugin for x86, x64, etc, I think, in which the various instructions and registers are listed as keywords or so.
Please let me know what you think about this. I'd be happy to define the specific assembly language(s) on an as-needed basis for myself, but it'd be very nice to get some help with the base plugin, as the core developers are surely much more familiar with the kinky regex queries used in prism.js than me ;)
Again, thanks for the plugin, it looks great, and, unlike other similar projects, only has a very small footprint :-)

Hey. It's been 3 years now. Is there support for x86 assembly (at least) planned?

See #482 for nasm as a fallback.


Error messages in Console output

My project is a Console app that creates a MetroWindow (for some reasons, this app has to be a Console app).
Some error sometimes pops up in the Console; it seems pretty random and the one I'm getting a lot these days is:
err:[0xe84]:d3d:D:\tinderbox\add-4.3\src\VBox\Additions\WINNT\Graphics\Wine_new\wined3d\resource.c 218: Resource 04D3EBD0 is being unloaded while mapped.

which happens when I call ShowDialog() on an OpenFileDialog.
It doesn't seem to be a serious issue though as the app continues to work normally, but I'd like to know if there would be a way to prevent that log to appear in the Console.

I do not know what exactly your program is doing, but I don't think that this is a MahApps.Metro related issue. The OpenFileDialog is a standard Windows dialog, and the error on the console appears to be a Direct3D (d3d) error. I'm not certain if this is related to "wine" appearing in the path, because that would mean this is somehow virtualized in Wine.

Thanks for your comment.
I know that the "core" issue doesn't come from MahApps (background: I run the app in VirtualBox with 3d acceleration enabled, so this is most probably some errors raised by the driver) but as I could not reproduce it without MahApps (a simple WPF app launched as a Console app), it thought the actual logging to the Console was done by MahApps.
I realise it's not a big deal and it's certainly not a show-stopper for me; feel free to close it if it doesn't sound relevant! Thanks


Webinstaller only shows a blank page

Hello there,
I was recently looking into BoxBilling again since my old (self-made) system wasn't able to take all requests and so I hoped that BB will. However: After I uploaded version 4.21 and extracted it, moved it over etc. it only showed me a list of all files.
After navigating around a little bit I found out that there was an index.php in src. So I navigated there and it showed me the error 101 (bb-config.php is missing/incomplete) and asked me if I wanted to run the installation. After I said yes, it redirected me to ../src/installer and only showed me a blank page. No Title or content.
I tried several other ways to solve this problem such as:

Making src the root directory
Deleting the config before running the installer
Banging my head against the wall
Doing it the manual way

None of these worked.
I tried it on

PHP 5.3
PHP 5.4
PHP 5.5
PHP 5.6
PHP 7.1
PHP 7.3
with all needed extensions (if available).

I am doing this in my (private) sandbox, so don't worry about stuff breaking.
Best regards and thanks for being alive
-Jonas from 3001.Host
PS: You might want to update your website, it's starting to look old. (or you could ask me to make a new one. I would be happy to do so.)

the vendor's folder is missing with the required files from the master's branch


Enable continuous integration

The standards set of checks that are applied to PRs on Jitsi repositories should be enabled (contributor agreement, CI builds, etc). This appears to be missing, judging from this test PR that I did just now: https://github.com/jitsi/jitsi-videobridge-openfire-plugin/pull/3

For those repos where there is no PR testing, there are no contributer agreement, its the PR testing plugin that comments.
And here we don't have any tests so this will not be enabled. If you want you can add some md, or text in the readme or PR template that has some hint about it.
Building and uploading the plugin will be added.

I'm not quite sure what you mean, but if you tell me that most of the standard checks are not needed, I'm inclined to believe you. :) Thanks!

So for the PR testing of jitsi-meet, lib-jitsi-meet, jicofo and jvb: we check out the latest version of all and the branch for the PR and set up an environment in which jitsi-meet-torture tests are run. This is how it works.
In the context of this plugin, currently, there is nothing we can do.
The thing that we can do is setup another machine which will run an openfire installing this plugin and run the torture against it, but we do not have resources currently to do that.

Oh, I don't mind not running tests - I was thinking more of the contributor agreement that's enforced for other repositories. But as I said, if you're happy to go without it, so am I.

I was thinking about this approach, that you can just add a PR template adding the CLA text there:
https://help.github.com/articles/creating-a-pull-request-template-for-your-repository/
So everybody creating a PR will see it.

Sure, works for me. Do you have the text somewhere? Feel free to add it
yourself.
On 1 June 2017 at 17:02, Дамян Минков<EMAIL_ADDRESS>wrote:

I was thinking about this approach, that you can just add a PR template
adding the CLA text there:
https://help.github.com/articles/creating-a-pull-
request-template-for-your-repository/
So everybody creating a PR will see it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jitsi/jitsi-videobridge-openfire-plugin/issues/4#issuecomment-305521092,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEDoytIvFLe2AlTfGcl_56Tt8ZreyRCjks5r_tJsgaJpZM4NoUv6
.


📝 Badge links updated

Summary
Badge links changed from image source to related pages.
fow.opensource logo added to project.

@all-contributors please add @sercanuste for doc, design, examples


Flaky test: test_normalize_adj in tests.core.test_utils sometimes fails

Describe the bug
In https://buildkite.com/stellar/stellargraph-public/builds/1203, the test_normalize_adj test failed due to an assertion failing.
To Reproduce
Steps to reproduce the behavior:

Run the test many times?

Observed behavior
example_graph =

    def test_normalize_adj(example_graph):
        node_list = list(example_graph.nodes())
        Aadj = example_graph.to_adjacency_matrix()
        csr = normalize_adj(Aadj)
        dense = csr.todense()
>       assert 5 == pytest.approx(dense.sum(), 0.1)
E       assert 5 == 3.982050657272339 ± 4.0e-01
E        +  where 3.982050657272339 ± 4.0e-01 = (3.9820507, 0.1)
E        +    where  = pytest.approx
E        +    and   3.9820507 = ()
E        +      where  = matrix([[0.25      , 0.25      , 0.25      , 0.        , 0.28867513,\n         0.        ],\n        [0.25      , 0.25  ...0.        ],\n        [0.        , 0.        , 0.        , 0.        , 0.        ,\n         0.        ]], dtype=float32).sum

tests/core/test_utils.py:39: AssertionError

Expected behavior
The test should never fail.
Environment
CI
Additional context
N/A

I think this is a duplicate of #996, which was fixed in #997.


consider xrefs to MedlinePlus and GeneReviews

@galyea123 emailed about this question:
I thought to ask you if it will be possible to add other links (ref) to your has dbxref section.  Specifically, MedlinePlus which has patient-friendly information and GeneReviews, which have detailed information. Note that MedlinePlus and GeneReviews are already tagged in OMIM.  Is this something you are able to do?
cc @sabrinatoro @matentzn

Thank you Nicole!

@nicolevasilevsky @sabrinatoro this request is deeply tied to the contentious #7071 issue - that needs to be solved first, and I think what it really needs is a brave decision. Its one of these things where 49% of the team will say a relatively hard "no" due to maintenance overhead and lack of separation of concerns ("an ontology is not a linkout database"), and 51% of the team will say "why not? Its valuable to users and easy to add - let's add it!". Someone needs to make a call!

[like]  Gioconda Alyea reacted to your message:

From: Nico Matentzoglu @.>
Sent: Monday, March 25, 2024 11:12:46 AM
To: monarch-initiative/mondo @.>
Cc: Gioconda Alyea @.>; Mention @.>
Subject: Re: [monarch-initiative/mondo] consider xrefs to MedlinePlus and GeneReviews (Issue #7425)
@nicolevasilevskyhttps://github.com/nicolevasilevsky @sabrinatorohttps://github.com/sabrinatoro this request is deeply tied to the contentious #7071https://github.com/monarch-initiative/mondo/issues/7071 issue - that needs to be solved first, and I think what it really needs is a brave decision. Its one of these things where 49% of the team will say a relatively hard "no" due to maintenance overhead and lack of separation of concerns ("an ontology is not a linkout database"), and 51% of the team will say "why not? Its valuable to users and easy to add - let's add it!". Someone needs to make a call!
—
Reply to this email directly, view it on GitHubhttps://github.com/monarch-initiative/mondo/issues/7425#issuecomment-2017761384, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4QAWD4BOOZ4UDSFOEQTIJTY2ABC5AVCNFSM6AAAAABE4HJR3GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJXG43DCMZYGQ.
You are receiving this because you were mentioned.Message ID: @.***>
Any review, use, distribution or disclosure by others is strictly prohibited.

Hi @galyea123,
We have discussed this and need to talk about implementation. I think we were hoping to get the cross-links from MedGen (since they already have them). I will talk with @matentzn to see where we are in the process.
Thank you! Have a nice holiday too!

Thank you Sabrina!

From: Sabrina Toro @.>
Sent: Wednesday, July 3, 2024 1:59 PM
To: monarch-initiative/mondo @.>
Cc: Gioconda Alyea @.>; Mention @.>
Subject: Re: [monarch-initiative/mondo] consider xrefs to MedlinePlus and GeneReviews (Issue #7425)
Hi @galyea123https://github.com/galyea123,
We have discussed this and need to talk about implementation. I think we were hoping to get the cross-links from MedGen (since they already have them). I will talk with @matentznhttps://github.com/matentzn to see where we are in the process.
Thank you! Have a nice holiday too!
—
Reply to this email directly, view it on GitHubhttps://github.com/monarch-initiative/mondo/issues/7425#issuecomment-2206907369, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4QAWD3N5VHBDJAWW2TEJJ3ZKQ3YDAVCNFSM6AAAAABE4HJR3GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBWHEYDOMZWHE.
You are receiving this because you were mentioned.Message ID: @.***>
Any review, use, distribution or disclosure by others is strictly prohibited.

Depends on

[ ] https://github.com/monarch-initiative/medgen/issues/29 (@twhetzel)

FYI MedlinePlus has a ton of developer tools that may be easier than trying to get their data via MedGen (which we process but haven't put out in FTP reports at this time) https://medlineplus.gov/about/developers/

@kanems thanks a ton.
@twhetzel @galyea123 in this case, I am not right now convinced this should be made a priority. Can we create a strong case for including these linkouts in to Mondo? Does anyone in the community if the resource to prepare a table for us to import into Mondo? (Mondo ID, xref)

In following up with this issue and the ping about it again during the Mondo call, I was able to pull a one-off report together for you all based on a couple of FTP reports from NCBI and my very ROUGH working memory of R to try and clean up a report based on the matching based on IDs...
https://ftp.ncbi.nlm.nih.gov/pub/GeneReviews/ GeneReviews FTP site includes a report of all their NBK IDs and chapter shortnames based on OMIM mappings.
Obviously Mondo team is familiar with the MedGen ID mappings file that also includes OMIM #s and MedGen UIDs, CUIs, etc.
So I pulled the GeneReviews report NBKid_shortname_OMIM.txt and then used matching across the various tables on CUI and MIM numbers to get the MIM number, CUI, MedGen UID, NBK ID, disease name, GeneReviews Short name and the formatted URL to the specific genereviews chapter on NCBI bookshelf.
There are 1992 items here and NO mondo IDs (sorry, I did not think to have to merge in the Mondo IDs as well). The Full GR FTP report has more MIM#s if you all want to try and increase your coverage with that file (NBK-MIM-Mondo)
MedGenIDs_toGeneReviews.xlsx

Fantastic @kanems! While it would, thanks to your work, be super easy to pull the gen-reviews links in as a one-off, I am not sure this is a good idea without a maintenance plan; Could you see yourself adding the table to your MedGen release system to keep it up to date, or is that out of scope for you?

@matentzn I will discuss with our devs and see what we can do. It would probably be a few months before we'd release an updated report (if we incorporate it at all, I am not the one to make that call).
The GeneReviews FTP report linked above plus the existing ID mapping report from MedGen is all you need to match a gene reviews NBK ID to a CUI via MIM number. Since Mondo has MIM #s already, you could probably match the GeneReviews FTP report by MIM# directly.

The same is true for MedlinePlus. OMIM has the #s already so you could match these from OMIM.  I will also talk with Sean as we are doing this in NORD. Thank you!  Gioconda


Fix autogenerating Renode scripts

This LiteX PR: https://github.com/enjoy-digital/litex/pull/1136 fixes Renode scripts autogeneration which makes it possible to generate HPS scripts.
Because of that I removed all predefined scripts since they didn't contain anything specific.
@tcal-x would you like to keep any of .resc scripts? I see contents of these are specific but since these are old and were used for litex-vexriscv-tflite target which does not exist anymore I removed them.
FYI @PiotrZierhoffer

Thanks @robertszczepanski!  -- can you merge main into this branch (or rebase) to get Piotr's SymbiFlow update, so we can go green?

@tcal-x -- I rebased it and tests are green now.


support python 3.7

tox software
replace {} | {} by {**{}, **{}} in two places

versions 3.7-3.11 for Linux, macos, and Windows are supported now, see: https://app.circleci.com/pipelines/github/martino-vic/loanpy?branch=2.0.1


Kube Services Table: Service ports look odd with missing data

protocol: "TCP"
port: 2746
targetPort: 2746

k9s shows the port info as...


Add man page

Hm... I wonder if committing a pre-formatted man page is the best approach here. I kind of prefer what we did on krunvm, committing them as text files and generating the man pages using asciidoctor: https://github.com/containers/krunvm/commit/8244dd6f302380feb4a7cd19c830a0d177dcf93c

@slp Yea, I like this better. Will re-submit once changes are made.

@slp @crobinso When you get the chance, can you review?

Is the plan to kill cargo-readme usage? I'm in favor of that. There should be only one source of truth for long form command descriptions, man page makes sense. Could happen as a follow on commit though.
I see the COMMAND OVERVIEW section is just a copy of
README.md content. Content is redundant, and not sure I've seen a section like that in a man page before. I think there should just be a COMMANDS section, like in man systemctl
If you rename the file to sevctl.1.adoc I think we get nicer github formatting

Is the plan to kill cargo-readme usage? I'm in favor of that. There should be only one source of truth for long form command descriptions, man page makes sense. Could happen as a follow on commit though.

Yea, I agree. I added a commit removing the usage of cargo readme.

I see the COMMAND OVERVIEW section is just a copy of README.md content. Content is redundant, and not sure I've seen a section like that in a man page before. I think there should just be a COMMANDS section, like in man systemctl

I see what you mean, section removed.

If you rename the file to sevctl.1.adoc I think we get nicer github formatting

Done.

@crobinso When you have the chance, can you re-review?


Kubernetes security

mailcatcher kubernetes pod

set AppArmour profile
set resources
set liveness probes
set security context (disable privilege escalation, run as a non--root, mount root fs as read only, default drop all capabilities)
set image pull policy to always update

Codecov Report

Merging #8 (a591236) into master (b7b6e5c) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           40        40
  Lines          751       760    +9
  Branches        23        23
=========================================
+ Hits           751       760    +9

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

taskcamp/urls.py
100.00% <100.00%> (ø)

taskcamp/views.py
100.00% <100.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 b7b6e5c...a591236. Read the comment docs.


Auto-Prefixing and Linting CSS

Why This PR Adds Value
Allows us to write better CSS in accordance with best practises and adds necessary vendor prefixes to target different browsers
What This PR Adds

Adds autoprefixer, postcss-cli and stylelint NPM modules
Adds .stylelintrc.json file to define Stylelint rules
Adds a new build command in npm run autoprefixer (but is also included in npm start for simplicity)
Note: Need to run npm install --dev and install Stylelint VS Code Extension or run Stylelint through the CLI - npx stylelint "**/*.css"

Screenshot
N/A - Code Change
How This PR Could Be Improved

Watch Command - Nodemon could be used to start a live-reload server (means we don't stop and start node server when developing) and postcss-cli could be ran in watch mode to then automatically run autoprefixer when CSS is changed.

Issue This PR Closes
This closes #59.

Sure, will take a look at the merge conflicts.
According to CanIUse, 97.24% of browser versions can use CSS Animation unprefixed and for the remaining 2.73% nearly all work with the -webkit prefix. Only those who haven't updated their web browser since 2012 or use Opera Mini aren't covered See image below.

Happy to add them back in though if you prefer.


Doesn't work in Flows

Hi TRegEXpr does not work in conjunction with TstreamReader in a thread
TThread.CreateAnonymousThread(
procedure begin
.....
I get a buffer overflow error: Exception class ERangeError with message 'Input buffer exceeded for DestinationIndex = 0, Count = 18432'

If you remove Regex:=TRegExpr.Create(reg); then everything works

I guess you do something wrong. Show the primitive demo to repeat the error.
I dont have Delphi, i have FPC.

I guess you do something wrong. Show the primitive demo to repeat the error.
I dont have Delphi, i have FPC.

Hello, I didn’t even expect you to answer so quickly, here’s a simple code that demonstrates the problem:
program Project2;

{$APPTYPE CONSOLE}

uses
  System.SysUtils, System.Classes, System.SyncObjs, VCL.Dialogs, Regexpr;

var
cs:TCriticalSection;

function pars(const Reg: TRegExpr; const text: string):string;
begin
Result:='';
if Reg.Exec(text) then
repeat
cs.Enter;
writeln(Reg.Substitute('$1'));
cs.Leave;
until not Reg.ExecNext;
end;

procedure FRead(const F:string);
const
BUFSIZE=1024*8;
var
Regex: TRegExpr;
sr:TStreamReader;
Buf: array [0..BUFSIZE] of Char;
begin
Regex:=TRegExpr.Create('([a-z]+)');
Regex.Compile;
sr:=TStreamReader.Create(F, TEncoding.Default, True);
while not sr.EndOfStream do begin
sr.ReadBlock(@Buf, 0, BUFSIZE);
pars(Regex, Buf+sr.readline);
end;
sr.Close;
sr.Free;
end;

var
opendialog:TOpendialog;
path:string;
begin
cs:=TCriticalSection.Create;
opendialog:=TOpendialog.Create(nil);
if opendialog.Execute then
path:=opendialog.FileName;
opendialog.Free;
TThread.CreateAnonymousThread(
procedure
begin
FRead(path);
end).Start;
end.

If you remove the wrapper TThread.CreateAnonymousThread then everything works !

I also tested with omnithreadlibrary and wrapped it in CreateTask instead of TThread.CreateAnonymousThread, the problem is also there

Anon functions require fpc 3.3, i dont have it. Maybe will install it.
Pls use triple quotes for code block, ie markdown fenced block.

Anon functions require fpc 3.3, i dont have it. Maybe will install it.
Pls use triple quotes for code block, ie markdown fenced block.

Try using omnithreadlibrary CreateTask, this problem also exists there !

By the way, for some reason this error also occurs with the standard library System.RegularExpressions:
program Project2;

{$APPTYPE CONSOLE}

uses
  System.SysUtils, System.Classes, System.SyncObjs, VCL.Dialogs, System.RegularExpressions;

var
cs:TCriticalSection;

function pars(const Reg: TRegEx; const text: string):string;
var
Match:TMatch;
begin
Result:='';
Match:=Reg.Match(text);
if Match.Success then
repeat
writeln(Match.Result('$1'));
until not Match.NextMatch.Success;
end;

procedure FRead(const F:string);
const
BUFSIZE=1024*4;
var
Regex: TRegEx;
Options: TRegexOptions;
sr:TStreamReader;
Buf: array [0..BUFSIZE] of Char;
begin
Options:=[romultiline];
Regex:=TRegEx.Create('([a-z]+)', Options);
sr:=TStreamReader.Create(F, TEncoding.Default, True);
while not sr.EndOfStream do begin
sr.ReadBlock(@Buf, 0, BUFSIZE);
pars(Regex, Buf+sr.readline);
end;
sr.Close;
sr.Free;
end;

var
opendialog:TOpendialog;
path:string;
begin
cs:=TCriticalSection.Create;
opendialog:=TOpendialog.Create(nil);
if opendialog.Execute then
path:=opendialog.FileName;
opendialog.Free;
TThread.CreateAnonymousThread(
procedure
begin
FRead(path);
end).Start;
end.

The same thing, if you remove the TThread.CreateAnonymousThread wrapper, then everything works !

@user4martin Can you reproduce it under fpc, please?

@User4martin Can you reproduce it under fpc, please?

Sorry I don't have FPC =(

Hi, it worked like this:
program Project2;

{$APPTYPE CONSOLE}

uses
  System.SysUtils, System.Classes, System.SyncObjs, VCL.Dialogs, Regexpr;

function pars(const Reg: TRegExpr; const text: string):string;
begin
Result:='';
if Reg.Exec(text) then
repeat
writeln(Reg.Substitute('$1'));
until not Reg.ExecNext;
end;

procedure FRead(const F:string);
const
BUFSIZE=1024*8;
var
Regex: TRegExpr;
sr:TStreamReader;
Buf: array [0..BUFSIZE] of Char;
begin
Regex:=TRegExpr.Create('([a-z]+)');
Regex.Compile;
sr:=TStreamReader.Create(F, TEncoding.Default, True);
while not sr.EndOfStream do begin
TThread.Synchronize(nil,
procedure begin
sr.ReadBlock(@Buf, 0, BUFSIZE);
end);
pars(Regex, Buf+sr.readline);
end;
sr.Close;
sr.Free;
end;

var
opendialog:TOpendialog;
path:string;
begin
opendialog:=TOpendialog.Create(nil);
if opendialog.Execute then
path:=opendialog.FileName;
opendialog.Free;
TThread.Queue(nil,
procedure
begin
FRead(path);
end);
end.

That is, through queues and synchronization

@User4martin Can you reproduce it under fpc, please?

will check in a couple of days....

I get a buffer overflow error: Exception class ERangeError with message 'Input buffer exceeded for DestinationIndex = 0, Count = 18432'

If you get a range check, then you should be able to get a stacktrace?
Also the error sounds (I have not checked) like it is thrown be the thread.
You could single step the code, and check if the issue happens while reading the thread, or while running regex.

The synchronize may or may not be a coincidence. It might be just hiding the issue (or not)....
I also have to check what happens if you pass an "array of char" to a string param. Or rather when you "+" join it with the ReadLine. Especially, if maybe the prior read did not fill the entire array.
Have you checked in the debugger, if the content of the string "text" in "pars" is what you expect?

I get a buffer overflow error: Exception class ERangeError with message 'Input buffer exceeded for DestinationIndex = 0, Count = 18432'

If you get a range check, then you should be able to get a stacktrace?
Also the error sounds (I have not checked) like it is thrown be the thread.
You could single step the code, and check if the issue happens while reading the thread, or while running regex.
The synchronize may or may not be a coincidence. It might be just hiding the issue (or not)....
I also have to check what happens if you pass an "array of char" to a string param. Or rather when you "+" join it with the ReadLine. Especially, if maybe the prior read did not fill the entire array.
Have you checked in the debugger, if the content of the string "text" in "pars" is what you expect?

Yes, if you read yes TStreamReader.ReadLine on one line, then no error occurs, the error occurs only when reading with a buffer

As if it wants to work like this, it displays the first lines, but still closes immediately:
program Project2;

{$APPTYPE CONSOLE}

uses
  System.SysUtils, System.Classes, System.SyncObjs, VCL.Dialogs, Regexpr;

const
BUFSIZE=1024*16;

var
cs:TCriticalSection;
sr:TStreamReader;
Buf: array [0..BUFSIZE] of Char;

function pars(const Reg: TRegExpr; const text: string):string;
begin
Result:='';
if Reg.Exec(text) then
repeat
cs.Enter;
writeln(Reg.Substitute('$1'));
cs.Leave;
until not Reg.ExecNext;
end;

procedure FRead(const F:string);
var
Regex: TRegExpr;
begin
Regex:=TRegExpr.Create('([a-z]+)');
Regex.Compile;
while not sr.EndOfStream do begin
sr.ReadBlock(@Buf, 0, BUFSIZE);
pars(Regex, Buf+sr.readline);
end;
//sr.Close;
//sr.Free;
end;

var
opendialog:TOpendialog;
path:string;
begin
cs:=TCriticalSection.create;
opendialog:=TOpendialog.Create(nil);
if opendialog.Execute then
path:=opendialog.FileName;
opendialog.Free;
sr:=TStreamReader.Create(TFileStream.Create(path, fmsharedenynone), TEncoding.Default, True);
TThread.CreateAnonymousThread(
procedure
begin
FRead(path);
end).Start;
end.

I don't have Delphi either, and I can't compile the code.
But I searched TRegExpr for the error Input buffer exceeded => There is no such text in it. At least not in the latest version/commit. (And I could not find any such error in past versions)
So you either have an older version (and it is unknown if the error actually still exists), or the error happens somewhere else.

Yes, if you read TStreamReader.ReadLine on one line, then no error occurs, the error occurs only when reading with a buffer TStreamReader.Readblock or TStreamReader.Read

Sounds like you run modified code to test that => that does not help.
What happens when you

run the code as you originally posted it.
run it in the debugger
single step over ReadBuffer
step into function pars

Can you step into to the line if Reg.Exec(text) then?
If yes, then what is the stacktrace in TRegExpr?

Also: Are you testing with the latest version of TRegExpr? Have you downloaded it from this GitHub repo? (In or after July 2024?)

I don't have Delphi either, and I can't compile the code.
But I searched TRegExpr for the error Input buffer exceeded => There is no such text in it. At least not in the latest version/commit. (And I could not find any such error in past versions)
So you either have an older version (and it is unknown if the error actually still exists), or the error happens somewhere else.

I moved the buffer to a global variable and the program worked !
const
BUFSIZE=1024*16;
var
cs:TCriticalSection;
Buf: array [0..BUFSIZE] of Char;
......
The only problem is that the buffer will probably need to be cleaned in a loop =)
Although in theory it should work in a local variable !

I moved the buffer to a global variable and the program worked !
const BUFSIZE=1024*16;

Non of the "workarounds" you described should affect the TRegExpr code, as you are calling it with text from the param const text: string. I am pretty sure at this point that the error is caused outside TRegExpr (but the only way to know is if you provide the stack trace).
Also seeing how your seemingly random changes hide the issue, I am certain it likely in TStreamReader (or the way you call it). Or maybe (though I find it unlikely) in the fact that your param is "const" (I don't know how exactly the nitty gritty of this this is implemented in Delphi...,)
Btw...
You have while not sr.EndOfStream do begin and that covers the ReadBuff. But then you call ReadLine, and there is no check if ReadBuff had reached the end....

Anyway, as I said: I do not have Delphi, I can not help you further on the TStreamReader related points above.
If (and 99% likely: only if) you

provide a stacktrace,
and that stacktrace points to a line in TRegExpr
and that TRegExpr is up to date

Only then may I be able to help further.
Unfortunately, otherwise I do not have the means to find the issue.
About your modified versions => all of those code modifications that you made can have any number of unknown side effects (and those may be very specific to the exact Version of Delphi that you use). So I have no way to know how, why the "hide" the issue. Therefore they do not in any way tell me where exactly the issue is.


IME not working when opening new window without unfocusing textbox

Hyprland Version
 System/Version info
Hyprland, built from branch main at commit 356414639fc06ace7045236ec41d673342981057 dirty (core: fix missing workspace events during swapping (5251)).
Date: Mon Mar 25 10:50:41 2024
Tag: v0.37.1-64-g35641463

flags: (if any)

System Information:
System name: Linux
Node name: acacia
Release: 6.8.1-arch1-1
Version: #1 SMP PREEMPT_DYNAMIC Sat, 16 Mar 2024 17:15:35 +0000

GPU information:
00:02.0 VGA compatible controller [0300]: Intel Corporation Raptor Lake-P [Iris Xe Graphics] [8086:a7a0] (rev 04) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Regression
Description
When opening another window without unfocusing the textbox in the first window, the IME does not work.
There is a difference how the IME breaks.

first text-input-v3 then v3(same program): both stop working until one is closed.
first text-input-v3 then v3(different program): the first one keeps working, but the second one does not work.
first text-input-v3 then v1: the second one works, the first one works, but when you go back to the second one it doesn't work this time.

A is the first v3 client, B is the v1 client.
Open A(ime works) -> Open B(ime works) -> Switch back to A(ime works) -> Switch back to B(ime not work)

How to reproduce
first text-input-v3 then v3(same program):

Open a text-input v3 client and focus any textbox. (in my case wayland firefox) (Do not unfocus the textbox)
Open another instance of the same text-input-v3 client in step 1. (I opened another firefox)
On the second client, the IME is not working(cannot switch between inputs, cannot types english when the icon says it's not in english)
Switch back to the first client, the IME is not working.
Close any of the 2 clients.
On the remaining client, the IME will start working again.

Crash reports, logs, images, videos
No response

It seems like when switching windows, void CInputMethodRelay::onKeyboardFocus(wlr_surface* pSurface) calls ti->leave();.
https://github.com/hyprwm/Hyprland/blob/a9d7526aae38dcdceb391fe9f00919c6c400abe3/src/managers/input/InputMethodRelay.cpp#L169-L174
But after that hyprListener_textInputDisable gets triggered and void CTextInput::onDisabled() calls leave(); again.
https://github.com/hyprwm/Hyprland/blob/a9d7526aae38dcdceb391fe9f00919c6c400abe3/src/managers/input/TextInput.cpp#L84-L90

does removing the leave in the relay fix it?

Nope, but adding hyprListener_textInputDisable.removeCallback(); to leave() seems to fix it.

yeah but that's not a good fix.

Yeah, it felt like a hacky way

it's not hacky, it's wrong

patch.txt
try this

It only fixes first text-input-v3 then v3(same program) case. The other cases are still not working.

how about this? This fixes all the cases.
patch.txt


Update auto sizes on resize

Would it be possible to update the automatically generated sizes when the window is resized?

Of course, thanks for asking: https://github.com/johannschopplich/kirby-blurry-placeholder/releases/tag/v1.2.0
Please test it. Feedback appreciated.

Wow that was quick! Will test it with an new project and report how it works
Thank you

Quick question: Is this supposed to recalculate and update the value of the sizes attribute? Because it does not change when I am resizing the window.
Am I missing something?


Add ability to use without VRChatApiTools

Added a fallback mode for AvatarImageReader that allows it to be used without VRChatApiTools being present in the project.
In this mode avatar image encoding is not available, and RuntimeDecoder can only be used by manually providing the linked avatar IDs in an array.

Looks good, merged


Chess puzzles

Hi!
I am very curious about your method in predicting chess ratings. However, I was not able to understand, how did you apply your net for puzzle rating prediction, because your net predicts only a rating of full chess game, not a single position
Could you, please, tell me in some details, I am very intrigued

The puzzle has a solution, so I used that sequence of moves, starting from the initial position, as input.


Update UpdatesViewController.xib

brew cask install has been deprecated; you should use brew install --cask

Sorry it took me so long to merge @chriscn
Thanks for the fix !


[App Service] Update swagger spec for ANT80 (#5111)

Update swagger spec for ANT80

Fix some spelling errors that were reverted by previous commit

Move certificate examples to new API version folder

Add the 2018-02-01 certificates files back

This change is already merged into master. Cherry-picking to ANT80  branch.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[X] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[X] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[X] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[X] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-js

This PR is cherry-picking the change from https://github.com/Azure/azure-rest-api-specs/pull/5111 to ANT80 branch

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

@hovsepm Can you please merge?

Can one of the admins verify this patch?

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

All of those validation errors are not new.
They are mostly missing examples.

@naveedaz I understand, just making sure you're aware.

I'm merging this to your branch since you're just bringing changes from anothe PR, be aware that Java and Node SDK generation are also failing, you can take a look at CI above, and will need to get fixed for the PR going to master. Thanks!

Thanks. :)


debian packaging

this fixes debian packaging and unifies names (username, package name, ...)

I've submitted two more pull request, which are necessary for this one to work properly

https://github.com/josegonzalez/python-beaver/pull/326
https://github.com/josegonzalez/python-beaver/pull/327

Thanks

Why was the name changed from python-beaver to beaver? The repo name is python-beaver.

uff, changed it back. How about renaming /etc/beaver to /etc/python-beaver and bin link from /usr/bin/beaver to /usr/bin/python-beaver, so the naming would be more consistent (init script, package name, source location)?

That sounds fine to me :)

changed ;)


Use latest xliff tool which has fix for issue https://github.com/fmuecke/XliffParser/issues/4

Hi @Faizan2304, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Faizan Ahmad). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Waiting for ci.dot.net to be up before merge.


If the resolution is reduced, the LMP window can appear offscreen

The window should move to within the visible area when joining a game or launching KSP.

There's no event that is triggered when the resolution is changed. So the only solution to this would be to restart KSP as when the plugin starts the connection screen is always positioned according to the current resolution.
Other solution would be to store the resolution once LMP starts and poll at every frame if the resolution has changed, but I consider this an overkill for something that happens very rarely


RemoveMaxParallelExecutionsPerNodeCap

@CordeliaGrey : Thanks for your contribution!
On Mon, Sep 21, 2020, 2:32 AM Sandy Winarko<EMAIL_ADDRESS>wrote:

@swinarko approved this pull request.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/pull/62940#pullrequestreview-492306482,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP5XOAWYPH4PDZYNNPC6CF3SG3XPVANCNFSM4RUDSHBQ
.

Publishing in this repository is suspended until Tuesday, September 22. Merging will resume at 7 AM Pacific Tuesday.
This PR can be merged on 9/22.


Ability to load other templates

Is your feature request related to a problem? Please describe
When using files.template it would be handy to be able to load relative templates to create complex jinja templates.
Describe the solution you'd like
I think changing get_template to allow for using a FileSystemLoader referenced to the current working directory would work.
Something like
Environment(loader=FileSystemLoader(os.getcwd())

👍 big fan of this. I think it needs to use the "deploy directory" as the root point, which is either cwdir or the folder in which the (first) deploy file lives. There's some issues with the deploy dir thing but that needn't affect this!

Yeah that makes sense. I tried a quick test with the FileSystemLoader and it seems to work.

Just so I'm understanding this correctly, is this going to fix the inability to use
{% include 'template/foo.j2 %}
which currently gives me this error:
TypeError: no loader for this environment specified


Create new api module for external consumers

This creates a new "/api" go module with the existing API. This makes it possible for external projects/plugins to import the API structs for the DPU Operator without the larger set of dependencies in the project's top level go.mod.

Can you rebase the pr, and clean up the commits? We could have the following commits:

upgrade to golang 1.21 (and don't forget the docker files),
another to vendor stuff
Add empty api module that still passes on prow

As discussed offline, I have split the last PR into 2.

/lgtm


test: ensure node upgrades between k8s versions work

This is blocked by using only MIG templates #7912 and a mechanism to do updates #6088.
Overview:

this needs a separate Jenkins job to run continuously on a different cluster (shouldn't be part of e2es yet because of version skewed testing)
as a minimum bar, this should test only from the latest released version to head
as a minimum bar, this should run e2es after the upgrade happens

/subscribe

Just to be sure, this issue is about upgrading API version, whereas #8082 is about upgrading binary version?

I was thinking this was about upgrading the binary version of the full cluster. That includes:

upgrade the master
then upgrade the nodes ← the focus of this issue

My impression was that #8082 was about the "master" specifically, whereas this is about the "nodes" because we want to make sure normal workloads / e2es pass. But, because the master is upgraded first before node upgrades happens, this necessarily also includes a master in its process, so becomes a "whole cluster upgrade."
Sorry if the wording is unclear (or if I'm confused; @roberthbailey can you confirm this interpretation is correct?). Please feel free to rename stuff.

I concur with @mbforbes's assessment: We want to be able to test node upgrades. To do this, we either need to first upgrade the master, or somehow start with a cluster that has skew (there isn't currently a way to do this). So I'd consider #8082 to either be blocking this issue, or that they can be combined into a single test.

/sub

I'm happy to work on this, but we had explicitly split this off in case I'm holed up doing other upgrade work because it's pretty separable. (In other words, if you're reading this, it's unblocked, I haven't written another comment that I'm actively working on this, and you want to work on it, please feel free to self-assign.)

OK, here is the plan:
Create a specific test that will be skipped by default that goes through the following process (where "validate" means "ensure the resources exist and function correctly"):

create a bunch of resources (rc-backed pods, services, external load balancer); validate
upgrade the master; validate
upgrade the nodes; validate
tear down the resources

Run this test, alone, on Jenkins:

as a CI job, upgrading latest stable clusters to latest CI clusters
as release validation jobs for all versions X to all newer versions Y. For example, if we have versions 1.0, 1.1, and 1.2:

one job runs 1.0 to 1.1
one job runs 1.1 to 1.2
one job runs 1.0 to 1.2

We'll have to add more release validation jobs as we do more releases; this is a cost we can eat for a while.
I'll own the PR to write this test, the GCE project creation and Jenkins setup/config to ensure this test is running.
Follow-up extra credit involves testing more things (secrets, volumes, persistent volumes, ...+?).
A non-goal for this issue is to upgrade the objects themselves (their serialized format on etcd). This is important but I think out of the scope of this issue. (Someone is likely addressing this elsewhere.)
@roberthbailey @alex-mohr let me know if this doesn't sound good.

#9493 is vaguely related.

Sorry, wrong issue. I meant #8081.

Thanks for the pointer—seems super related, not just vaguely!

As an update, I'm going to do this for the GKE provider first, as upgrade-master() for GCE is currently broken (will file an issue if there isn't one).

Tiny update as I see this is now a P0: I've been working on the e2e test code today and will have it out for review today or tomorrow.
Regarding Jenkins, we've been rethinking how to do upgrade tests on Jenkins that involve just two or three Jenkins jobs that can run in parallel rather than six sequential ones; more details in this comment. The Jenkins tests that would close this issue would run in one of the slots outlined there.
Regarding the test extensions I mentioned in #8081, my first PR for this will close out number 2 for sure, possibly more.

Now that #9987 is merged, the remaining work for this issue is to

[ ] support this for GKE (upcoming PR)
[ ] get all of the Jenkins config sorted out so that we get the [deploy→upgrade master→upgrade nodes→e2e test] workflows green for GCE and GKE.

Both GCE and GKE upgrade (including node upgrade) builds are green. The last code thing is the final unfinished item from https://github.com/GoogleCloudPlatform/kubernetes/issues/8081#issuecomment-112499324:

[ ]  Validate that the upgrade actually happened. We want to ensure that the new software is really running, otherwise--as @satnam6502 pointed out in #8081--a no-op would satisfy the "upgrade".

We don't have any released (stable) versions that include these tests, but once we do, we'll need to continuously create new Jenkins jobs that verify supported release paths are upgrade-able. This should probably be in the GCE or GKE release instructions; I'll own getting those instructions written, but it's separate from this issue.


Add expander functionality

Expanding expand functionality
The expand functionality as discussed in a gist has been implemented and is submitted for your consideration. It allows custom expansion of postgresql column types, as well as changing
the default Mojo::JSON::from_json, to JSON::XS for example.
Performance
Benchmarking script can be found at https://gist.github.com/chy-causer/66b15f14b16b06d3d9da0e36cd36ca04

Benchmark
Before
After

expand->hashes
53/s
52/s

expand->hash
42/s
41/s

hashes
90/s
91/s

bool::json
50/s
51/s

Extra expander
-
51/s

bool expander
-
67/s

These numbers can be verified using the linked script.
From these benchmarks, it's clear there's very little difference in performance using the new code. However, for the not uncommon use case of wanting booleans represented as objects, there is a 30% speed increase as well as a more pleasing syntax.
# One possible method:
$db->query("SELECT row_to_json(t) r from ( select is_bool from _table) t" )->expand->hash->{r};

vs

$db->query("SELECT is_bool from _table")->expand->hash;

Backwards compatibility
All existing tests passed without modification
Pluggability
Functionality is documented and shows how to convert bigints and booleans.
Implementation: An expand mapper
The code makes use of an expand mapper (cached for performance), which maps columns to expand coderefs. The rationale for using this to achieve the desired functionality is two-fold:

Performance
It allows future expansion of expand
$results->expand(column_name => sub { ... } )->hash;

I did not pursue the latter as it's not something I need, but the option is there.
Addendum
As it's my first submission to the Mojo project, I am well aware there may be issues in it, in terms of coding style and general best practice. However, there are a few things I would like to bring up in advance:
_expand_mapper mapping both hashes and arrays
The _expand_mapper caches its result for performance. However, it caches both hash and array mappings, whether they're used or not.
This is to match the documentation that once the first row is returned, then you cannot alter the expanders. Consider the following:
$pg->db->query('select * from _table')->expand->hash->expanders({})->array;
This scenario is tested.
What types are available
It turns out the DBD::Pg hard codes the list of row types in its source code. This means that extensions like ip4r return a type of 'unknown' and DBD::Pg will need to be patched and recompiled
to deal with new types as they are introduced to your database.
Furthermore, I cannot seem to find a nicely formatted list of available types DBD::Pg returns from $sth->{pg_type}. The best I can do is the DBD::Pg types.c source code!

Pretty sure this breaks JSON decoding for non-blocking queries.

I have just spotted a mistake in the bench.pl script. Corrected in the gist, but will need to rerun to regenerate numbers. It should only affect absolute numbers, not ratios, but I'd like to be 100% confident in my results.

If this breaks non-blocking queries, then there are missing tests as all existing tests pass.

No, you forgot to add new tests. Until now there was no difference between blocking and non-blocking queries for JSON decoding.

Changing from Mojo::JSON to JSON::XS is a non-issue btw., since Mojo::JSON::MaybeXS exists.

I think a pretty strong argument against this functionality might be that the way DBD::Pg handles different data types is very sketchy. Types like _json and _jsonb are not even mentioned in the documentation, bigint seems like something DBD::Pg might start handling itself at any time.

Like i said on IRC before, the patch for this feature has to make the expand code cleaner to be worth it for me, and i just don't think that's the case here. So i'm afraid i'll have to give it a 👎  vote.

I haven't given this a ton of review admittedly, since I've been busy at QAH. I do worry that it seems quite complicated, especially given that most of this functionality can be achieved by coercion in the db query itself. Some performance gain is nice but perhaps not worth the additional complexity. (Note that this is a very common consideration in Mojo-land and not just a comment on this individual PR). As such I'm 👎 at least until the complexity can be minimized.

                                                                                Thank you for looking past the code sloppiness to get to what I am trying to achieve. I do not have time at the moment to prepare a full response so this is an email to say I share some concerns raised, and there are some that I feel can be defended. I will try to reply point by point within the next 24 hours.

@chy-causer note that I've updated my comment a little (to clarify). The sentence is now:
"Some performance gain of things like a bool expander is nice but perhaps not worth the additional complexity (especially given that other existing expanders see a performance hit). "

If I have not explicitly addressed any points raised in previous comments, please take that as me not having any response of note.
Benchmarking revisited
The only system I have available to do benchmarking is hosted on a fairly busy VI, so the results are rather noisy. Topping that, there was a fairly howling error in the original benchmarking script and the results were skewed. Here is a new benchmark, with the numbers of iterations and rows boosted, and  the script niced to -10 to reduce noise further. I have made no modifications suggested in the comments about wasteful calls.

Benchmark
Before
After

expand->hashes
52.4/s
52.8/s

expand->hash
36.2/s
36.3/s

hashes
349.1/s
353.3/s

bool::json
95.5/s
102.2/s

Extra expander
-
102.4/s

bool expander
-
193.8/s

Rows: 2000
Iterations: 5000 (10000 for raw hashes)
Before: 290.6s
After: 353.7s
As an aside, I now have much more sympathy for the original author of _expand as I can now see that much of its design was to boost performance.
You are welcome to rerun these benchmarks yourself to verify.
https://gist.github.com/chy-causer/66b15f14b16b06d3d9da0e36cd36ca04
_column_types and _columns_by_types being needless
They would be required for the following functionality:
$db->query($QUERY)->expand(column_name => \&expander);

You're not mind-readers, I didn't explain it, and I was getting ahead of myself. I will not pull something like that again.
Of DBD::Pg return types
I wholeheartedly agree that return types are sketchy and unintuitive at times. DBD::Pg's handling can also be fairly inconsistent. Consider the two stanzas and see if you can guess what they return:
$pg->db->query(
  'select array_agg(distinct state) from minion_jobs'
)->array;

$pg->db->query(
  'select array_agg(distinct state)::text[] from minion_jobs'
)->array;

However, on the flip side, the types that do exist, undocumented though they are, have been stable for a number of years now.
https://github.com/autarch/DBD-Pg/commits/master/types.c
int8 is the internal representation of bigint by PostgreSQL and is most likely here to stay. Most other important types are fairly intuitive and I still think there is benefit to be had in expanding them (xml, booleans, timestamps and the one I actually care about, cidr.)
The others like _json I can happily ignore. Arrays of json aren't expanded currently, so I don't see any reason to start.
As a compromise, Mojo::Pg could limit allowed expansion types to a well defined subset of DBD::Pg types (even aliasing the non-intuitive ones like int8.)
Bigint may be handled by DBD::Pg any time
How would it handle it? By returning a Math::BigInt object (the only core module I can think of that would be appropriate)? For arguments sake if they do do that, then Math::BigInt can take Math::BigInt objects as an argument, so nothing will break should that happen. I cannot speak for the developers of DBD::Pg, but I think the likelihood of that happening is small.
What's proposed can be achieved by coercion within the database itself
That's absolutely not the case.
$db->query(
  "select '<hello>world</hello>'::xml"
)->expand->array->[0]->at('hello');

This cannot be done via coercion within the database.
Similarly, even worse, something that you think works, but gives the wrong answer:
$db->query('SELECT<PHONE_NUMBER>854775807::bigint')->array->[0] / 10;

The change breaks non-blocking
I'm afraid I cannot write a test that works on the original code but not the new one so I will need some assistance here.
Speed increase of boolean expansion not worth the complexity
Complexity for whom? If it's the Mojo::Pg codebase then I don't have an answer for you other than a promise to go back and look at the code. If it's the end user, then I would argue that
$db->('SELECT true')->expand->array;

Is much less complex than
$db->('SELECT json_agg(true)')->expand->array->[0];

Obviously you wouldn't do these queries in real life, but hopefully you get the picture.
On top of that, the new benchmark shows around 100% performance improvement for the former over the latter (please do verify this for yourself though.)

Just about to go to bed here in England as the QAH wraps up and I fly home. Just before I do go, yes I did mean for the Mojo::Pg codebase. While we certainly care about the end users, we do value the cleanliness of the codebase too. This is not to say that the idea is dead, just that it would need tightening up before it could really be considered. As you have looked through the Mojo code I'm sure you've noticed the style :-P
One other quick thing, this isn't very easy to immediately understand:
$db->('SELECT json_agg(true)')->expand->array->[0];

but I think this reasonably is
$db->('SELECT to_json(true)')->expand->array;

Perhaps you would like to join #mojo on irc.perl.org to discuss further? PRs are nice for reviewing code but to discuss things like api and utility irc is really nice. I'm usually on and around (travel notwithstanding). My nick is jberger. @kraih is sri.

I appreciate you want to keep the code clean. As I said earlier, this is my
first PR, pretty much for any project, not just Mojo::Pg, so I'm well aware
there is much tightening up that can be done and I'm very grateful you and
@kraih took the time to read the code and discuss its merits, rather than
dismiss it out of hand.
I am on IRC as well, but am only able to respond during UK work hours. My
nick is CHYC.

$db->('SELECT to_json(true)')->expand->array;

With such a plethora of json functions, I overlooked the most obvious. Thanks

After https://github.com/kraih/mojo-pg/commit/f854bdd4dd9cba3f676eb4a9b5189633e5d1e67c it should not be possible for this patch to be faster than a JSON only solution.

I'm afraid this feature did not get the required votes.


change the description of arrays matchers

#37

@rultor merge

@rultor merge

@graur OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@graur Done! FYI, the full log is here (took me 3min)


Build pattern regex doesn't support url path with space

My build comment template in my html like this:
<!-- build:css <%- staticBase%>/css/index/index.css -->
<link href="<%- staticBase %>/css/common/base.css<%- version %>" type="text/css" rel="stylesheet">
<link href="<%- staticBase %>/css/index/index.css<%- version %>" type="text/css" rel="stylesheet">
<link href="<%- staticBase %>/css/common/itemListTemplate.css<%- version %>" type="text/css" rel="stylesheet">
<!-- endbuild -->

the staticBase variable is the DNSUrl i will render by grunt template, the first line <!-- build:css <%- staticBase%>/css/index/index.css --> , you can see there is a space before word staticBase, but the usemin build pattern can't parse this pattern.
I check the usemin rebuild pattern is /<!--\s*build:(\w+)(?:\(([^\)]+)\))?\s*([^\s]+)\s*-->/, it does't support   build path with a space, when i remove the space before staticBase word, it works fine.

@JackZhang1988 the problem is that the third group in the regexp, which is ([^\s]+), exclude any space.

@JackZhang1988 can you test the branch in #516 to see if it fixes your use case. I don't know if it also related to your other issue #513

@JackZhang1988 see my comment in your other issue.

@JackZhang1988 could you explain why you would want to have your path being variable ?
I need to have more information on your use case.
Seeing that you have <%- version %>, I think you're missing the point of usemin whose job is to optimize your page, so it need to know where are your assets.
I'm closing this issue as irrelevant. Feel free to come back with more explanations and reopen if needed.

@stephanebachelier sorry for replaying so late, the variable for path used for static files' CDN path, usually we develop our project in local, this variable will be 'localhost', when our project release to internet, this variable will be the CDN path.
i know there maybe other way to accomplish this, but when i join my team, it has used this way.

@JackZhang1988 maybe you should use another plugin that will replace all the paths of the assets, but once usemin have processed your files.


STM32 Controller does not abort receive request after timeout

Describe the bug
I see that #160 is bringing the STM32 controllers into this repo. It appears the issue is in that portion primarily. If this portion isn't maintained by USBX, feel free to direct me to the appropriate maintainers.

What target device are you using? STM32U585
Which version of Eclipse ThreadX? 6.1.10
What toolchain and environment? Keil MDK v5.38 with the ARM Clang v6.20.1

Given

A timeout value has been set for read operations via ux_device_class_cdc_acm_ioctl()
At least one call to ux_device_class_cdc_acm_read() has timed out

When the host sends data to the device while the device is NOT pending in a call to ux_device_class_cdc_acm_read(),
Then the data sent by the host is never delivered to the device application even on subsequent calls to ux_device_class_cdc_acm_read()
It appears that the STM32 controller is not aborting the transfer request in the USB peripheral HW, causing an unexpected interrupt. Eventually HAL_PCD_DataOutStageCallback() in the ux_dcd_stm32_callback.c file is called in the interrupt handler. This function sets the size of the incoming data in the appropriate UX_SLAVE_TRANSFER structure and then the appropriate semaphore is “put”. The issue comes from the fact that on the subsequent call to ux_device_class_cdc_acm_read(), the size in the UX_SLAVE_TRANSFER structure is set to 0, clearing any record of data already in the buffer.
Using TraceX, I confirmed that the interrupt happens before the call to ux_device_class_cdc_acm_read() by observing that there were no threads pending on the semaphore when the semaphore is “put”.
Locally, I was able to mitigate this issue in two ways:

Setting the transfer request size to 0 after completing the read instead of before (still setting to 0 before transmit requests).

This obviously doesn’t directly solve the issue as there is no guarantee that the buffer provided to the USB peripheral HW when the receive request was setup is still valid when the interrupt occurs.

Checking the return value of the semaphore “get” operation in _ux_dcd_stm32_transfer_request() (found in ux_dcd_stm32_transfer_request.c) and calling _ux_dcd_stm32_transfer_abort() on appropriate errors (namely when TX_NO_INSTANCE is returned.

My quick patch implementation of this still has a race condition between the interrupt and the aborting of the transfer request though so it is just a proof of concept at this point.

To Reproduce
I don't have a demo project to share but I believe that any project that uses the CDC ACM Device class on STM32 will demonstrate the issue.
The following steps provide a good starting point for forcing this issue to occur.

DEVICE: Set the timeout value for reads to a low value (100ms is what I used) via ux_device_class_cdc_acm_ioctl().
DEVICE: Perform at least one read via ux_device_class_cdc_acm_read() that times out BEFORE sending any data from the host to the device.
DEVICE: Wait a relatively long time before performing another read (I used 15 seconds for this time period to force the issue)
HOST: Send any data to the device (ensuring it happens after the first timeout and during the length of time described in step 3)
Note that the device application never receives the data even on subsequent calls to ux_device_class_cdc_acm_read()

Expected behavior
The USB peripheral HW transfer request should be aborted so that the interrupt does not occur outside of reads.
Impact
This lowers my team's confidence in USBX. It is not a show stopper as the odds of it happening during normal operations are fairly low as long as the processing time between calls to ux_device_class_cdc_acm_read() is small. We will be doing further stress testing to see how often we hit it in general operating conditions and may have to move away from USBX if it occurs frequently enough.

I see that the related PR has been closed without merging because the files will be delivered in a dedicated repo. Is there any info on which repo and where that will be?

I am also hitting this problem, implementing a simple HID device. HID signals will always stop sending at some point, with tx_semaphore_get() returning TX_NO_INSTANCE
@JonTBeckett I would love to see your patch, I have not yet figured out how to implement a fix in my scenario. I could not find a related PR to this issue when I searched.

Also @JonTBeckett I think that this could may be maintained by ST, and they have this repo which seems to have the latest version of the _ux_dcd_stm32 drivers: https://github.com/STMicroelectronics/stm32_mw_usbx.git

@cosmikwolf Thanks for finding the ST repo! I'll open this same bug under there when I get a chance.
As for the patch, I'll have to dig it up. It's been a while since I tinked with and the patch is buried in a stash in one of my local repos. Hopefully I can find it again and attach it for you!


Test enhancements / after electron port

Some questions for @ryanrosello-og when he’s back and after initial electron merge also cc @lf94 for anything I’m missing

vscode is electron and opens really fast, electron has great docs on how vscode achieved this: https://electronjs.org/docs/latest/tutorial/performance but how do we ensure on every PR we aren’t making the app open slower and that just general interactions aren’t slower
how can we test double clicking a file in the os finder and it opening in the app (and performance test speed to open here as well)
how can we test opening a file via double click where we have to accept a native os dialog for perms (we def want to test this code path)
How can we test core dump from the desktop / browser without spamming our repo w core dump issues
How can we test the auto updater
https://github.com/KittyCAD/modeling-app/pull/3315/commits/b4e0bd038e95b25ac32c4bfb23ac4b00c870ad23 copies dev env vars into .env.production there must be a better way to get yarn electron:package to either build or run in dev mode so that it uses the dev env vars.

Also in relation to #1 in the list we don’t even have metrics on perf today but we will want them so we can make it faster

https://github.com/KittyCAD/modeling-app/commit/b4e0bd038e95b25ac32c4bfb23ac4b00c870ad23 copies dev env vars into .env.production there must be a better way to get yarn electron:package to either build or run in dev mode so that it uses the dev env vars.

Why not NODE_ENV=production to load the .env.production

I think you covered everything - yeah these all seem like hella hard to test outside of playwright :D.

oh number 6 is irrelevant i removed that

so right now its working fine without it!

also for coredump i just mean when you hit the file a bug button it works and opens up github thats all, nothing too fancy but idk how to test for external windows and shit

also for coredump i just mean when you hit the file a bug button it works and opens up github thats all, nothing too fancy but idk how to test for external windows and shit

no worries, I think Kurt had a similar issue with one of his tests, I will have a poke around and see what is doable

https://github.com/KittyCAD/modeling-app/issues/3588 is relevant

Hi @ryanrosello-og, I saw s post from @jessfraz online and wanted to chime in here:

how can we test double clicking a file in the os finder and it opening in the app (and performance test speed to open here as well)
how can we test opening a file via double click where we have to accept a native os dialog for perms (we def want to test this code path)

I could offer help with nut.js to tackle this?


Update save to save across platforms and browser for a same account

It takes time to recreate the same schedule on each browser I use. It's frustrating especially when you use several OS on your computer and ADE-scheduler on your phone.
It would be interesting to save the schedule on the server to update the schedule on each browser at once.
Additional context
Add any other context or screenshots about the feature request here.

Hello @YannPollet ,
If I understand well your problem, what you are looking for is provided by the "Account" feature.
Do you have an account ?


Is there anyway to catch the socket error event?

Hi there,
Thanks for this great library, it works great.
Recently I tried to debug the request from our gateway server (using golang), client always failed
to load the second request with HTTP / 1.1 400 connection:close. And there was no error log in
express server.
After debug deeper into Socket (res.socket.on('error')), I found connection was closed because of
HPE_INVALID_METHOD (http parser in nodejs).
I was thinking iss there anyway to catch socket error in express?
Thanks!

Hi @sc0Vu ! Express.js sits on top of the Node.js HTTP server platform. Specifically, Express.js is the requestListener function of a HTTP server (https://nodejs.org/dist/latest-v14.x/docs/api/http.html#http_http_createserver_options_requestlistener). As Node.js does not forward those types of errors to the requestListener function, it is not possible to see those within Express.js. But you can attach them directly to your Node.js server object: https://nodejs.org/dist/latest-v14.x/docs/api/http.html#http_event_clienterror


Adding default formatters to the doc

It's not possible to easily find the list of default formatter in the doc. The workaround is to do display the measure.formatter

We choose do keep the workaround as the best way to find the formatter.