Docker Detection

Laying the groundwork for Docker images to have customizations based on DISTRO variable.

Thanks for typo fix - this looks like by creating this as a new distro without modifying the other scripts it would cause the Docker install to proceed but fail on the rest of the setup process.  As such I'm leaving this up for discussion but this isn't something I can merge at this time, I'll look into the typo on the sudo part.
There is a LOT going on here, I think it would be easiest if you try to collaborate with Brian's branch here - https://talk.vespene.io/t/ability-to-contanerize-the-components-of-vespene/30/13

Hi,
Thanks for the effort and interest here.
Until there is a complete example somewhere I don't think we need a PR open to just change part of the install scripts that isn't a complete solution.  I'm going to close this for now, but feel free to start a thread on talk.vespene.io.
I think what I need to see (because so many people are wanting to do slightly different things) is a full set of Docker files and patches, mostly condensed into a small set of commits.
That solution also needs to use a seperate database and have an easy way to inject in settings from any settings.d values the user wants, rather than to just run a demo, and an easy way to build a container for a given worker queue name.
How does that sound?
thread: https://talk.vespene.io/t/ability-to-contanerize-the-components-of-vespene/30/17

Since Docker based deployments failed anyway (due to the systemd requirement), I figured that incremental steps towards a working solution would be acceptable, but I understand the desire for an end-to-end solution.


Generate APK for android

I want to genrate apk of my tabris app and i got this Error :
Error: /Applications/Android Studio.app/Contents/gradle/gradle-4.4/bin/gradle: Command failed with exit code EACCES
The command /usr/local/lib/node_modules/tabris-cli/node_modules/.bin/cordova exited with 1

Do you have more of the command output before those two lines?

This is all the output :

[./] npm run --if-present build:android
[./] npm run --if-present build
[./] Copying Cordova files to build/cordova ...
[./] Copying JavaScript files to build/cordova/www/app ...
[build/cordova/www/app] npm install --production
up to date in 11.242s
[./] Replacing variables in config.xml...
[/usr/local/lib/node_modules/tabris-cli/src/services] npm bin
[build/cordova] /usr/local/lib/node_modules/tabris-cli/node_modules/.bin/cordova build android --no-update-notifier
ANDROID_HOME=/Users/macbook-pro/Library/Android/sdk
JAVA_HOME=/Library/Java/JavaVirtualMachines/jdk1.8.0_181.jdk/Contents/Home
Subproject Path: CordovaLib
Error: /Users/macbook-pro/Documents/Tabaris/build/cordova/platforms/android/gradlew: Command failed with exit code EACCES
The command /usr/local/lib/node_modules/tabris-cli/node_modules/.bin/cordova exited with 1

@benhartouz What version of the tabris-cli are you using? Does it match your package.json tabris version? What command are you running to build your app?

@benhartouz Did you solve your problem in the meantime? Can you use the build service at https://tabris.com?

closing issue. problem resolved


Doesn't compile using Python 3.5.1 Build 1

Gives the following error and warnings
/Users/e1z4/Projects/iOS/callisto/main.m:31:24: error: implicit declaration of function '_Py_char2wchar' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
        wpython_home = _Py_char2wchar([python_home UTF8String], NULL);
                       ^
/Users/e1z4/Projects/iOS/callisto/main.m:31:22: warning: incompatible integer to pointer conversion assigning to 'wchar_t *' (aka 'int *') from 'int' [-Wint-conversion]
        wpython_home = _Py_char2wchar([python_home UTF8String], NULL);
                     ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/e1z4/Projects/iOS/callisto/main.m:54:24: warning: incompatible integer to pointer conversion assigning to 'wchar_t *' (aka 'int *') from 'int' [-Wint-conversion]
        python_argv[0] = _Py_char2wchar(main_script, NULL);
                       ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/e1z4/Projects/iOS/callisto/main.m:56:28: warning: incompatible integer to pointer conversion assigning to 'wchar_t *' (aka 'int *') from 'int' [-Wint-conversion]
            python_argv[i] = _Py_char2wchar(argv[i], NULL);
                           ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
3 warnings and 1 error generated.

Xcode Version 7.3 (7D175)

I'm aware of this problem - however, if you're using the 3.5 branch of the template, it should be fixed - references to _Py_char2wchar have been replaced with the equivalent call to Py_DecodeLocale.
Can you confirm that you were using the 3.5 branch template?

Sorry, I didn't notice the branch. I was using master. I bet it will work on 3.5. Thanks!

I have the same errors and warnings with the Python.framework 2.7.
I tried all brunches (with _Py_char2wchar the same errors)
Xcode Version 7.3.1 (7D1014)
/Python_iOS/EventRegistry/main.m:31:24: error: implicit declaration of function 'Py_DecodeLocale' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
wpython_home = Py_DecodeLocale([python_home UTF8String], NULL);
^
/Python_iOS/EventRegistry/main.m:31:22: warning: incompatible integer to pointer conversion assigning to 'wchar_t *' (aka 'int *') from 'int' [-Wint-conversion]
wpython_home = Py_DecodeLocale([python_home UTF8String], NULL);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Python_iOS/EventRegistry/main.m:32:26: warning: incompatible pointer types passing 'wchar_t *' (aka 'int *') to parameter of type 'char *' [-Wincompatible-pointer-types]
Py_SetPythonHome(wpython_home);
^~~~~~~~~~~~
In file included from /Users/oldman/Development/Python_iOS/EventRegistry/main.m:8:
In file included from /Users/oldman/Development/Python_iOS/Python.framework/Headers/Python.h:127:
/Python_iOS/Python.framework/Headers/pythonrun.h:25:41: note: passing argument to parameter here
PyAPI_FUNC(void) Py_SetPythonHome(char );
^
/Python_iOS/EventRegistry/main.m:52:23: error: implicit declaration of function 'PyMem_RawMalloc' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
python_argv = PyMem_RawMalloc(sizeof(wchar_t) * argc);
^
/Users/oldman/Development/Python_iOS/EventRegistry/main.m:52:21: warning: incompatible integer to pointer conversion assigning to 'wchar_t **' (aka 'int *') from 'int' [-Wint-conversion]
python_argv = PyMem_RawMalloc(sizeof(wchar_t) * argc);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Python_iOS/EventRegistry/main.m:54:24: warning: incompatible integer to pointer conversion assigning to 'wchar_t *' (aka 'int *') from 'int' [-Wint-conversion]
python_argv[0] = Py_DecodeLocale(main_script, NULL);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Python_iOS/EventRegistry/main.m:56:28: warning: incompatible integer to pointer conversion assigning to 'wchar_t *' (aka 'int *') from 'int' [-Wint-conversion]
python_argv[i] = Py_DecodeLocale(argv[i], NULL);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/oldman/Development/Python_iOS/EventRegistry/main.m:59:29: warning: incompatible pointer types passing 'wchar_t **' (aka 'int **') to parameter of type 'char **' [-Wincompatible-pointer-types]
PySys_SetArgv(argc, python_argv);
^~~~~~~~~~~
/Python_iOS/EventRegistry/main.m:8:
In file included from /Python_iOS/Python.framework/Headers/Python.h:129:
/Python_iOS/Python.framework/Headers/sysmodule.h:13:44: note: passing argument to parameter here
PyAPI_FUNC(void) PySys_SetArgv(int, char **);
^
/Python_iOS/EventRegistry/main.m:100:9: error: implicit declaration of function 'PyMem_RawFree' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
PyMem_RawFree(wpython_home);
^


When RCP resets, otPlatRadioEnableSrcMatch() is not set back to TRUE by Host

Describe the bug A clear and concise description of what the bug is.
When the RCP failure recovering feature (OPENTHREAD_SPINEL_CONFIG_RCP_RESTORATION_MAX_COUNT) is activated, after it resets the RCP due to a spinel/HLDC issue (bad frame received/sent) or reset (using reset button of the board), the Host (RPi) doesn't enable the src addr match table (otPlatRadioEnableSrcMatch). The Host only enables it once when the system starts, when OTBR becomes leader for example.
The result is that the RCP will set the pending bit to 1 in all of the data requests coming from the child it is associated with.
If we check at otPlatRadioEnableSrcMatch definition:
/**
 * Enable/Disable source address match feature.
 *
 * The source address match feature controls how the radio layer decides the "frame pending" bit for acks sent in
 * response to data request commands from children.
 *
 * If disabled, the radio layer must set the "frame pending" on all acks to data request commands.
 *
 * If enabled, the radio layer uses the source address match table to determine whether to set or clear the "frame
 * pending" bit in an ack to a data request command.
 *
 * The source address match table provides the list of children for which there is a pending frame. Either a short
 * address or an extended/long address can be added to the source address match table.
 *
 * @param[in]  aInstance   The OpenThread instance structure.
 * @param[in]  aEnable     Enable/disable source address match feature.
 *
 */
void otPlatRadioEnableSrcMatch(otInstance *aInstance, bool aEnable)

We can see that if the src addr match table is not enabled, we set the pending bit in every data request, so we are behaving normally, but the src addr match table should be enabled.
The intention of this feature is that the Host doesn't have to reset the whole Thread application, but only the RCP, so it is transparent for the other devices in the network that something wrong occured. We want to use this feature as we experience some issues using the spinel/HDLC over UART. We sometimes miss one or several bytes and it's too bad to reset the OTBR application just for that.
To Reproduce Information to reproduce the behavior, including:

Git commit id

Host (RPi): https://github.com/openthread/ot-br-posix/releases/tag/thread-reference-20230710
RCP: https://github.com/openthread/openthread/releases/tag/thread-reference-20230706

IEEE 802.15.4 hardware platform

STM32WB55

Build steps

Build using the flag -DOT_RCP_RESTORATION_MAX_COUNT set to anything but 0
In my case:

 sudo WEB_GUI=1 REFERENCE_DEVICE=1 ./script/bootstrap
 sudo OTBR_OPTIONS="-DOT_RCP_RESTORATION_MAX_COUNT=2" INFRA_IF_NAME=wlan0 WEB_GUI=1 REFERENCE_DEVICE=1 ./script/setup

Network topology

An OTBR (RPi + STM32WB) and a SED (STM32WB55) that are associated, simply by setting same networkkey, panid, etc... The SED makes a data request every second.

Expected behavior A clear and concise description of what you expected to happen.
Host should enabled the src addr match table on the RCP, so that it doesn't set the frame pending bit to 1, and doesn't send an empty data frame after that.
** Workaround **
A simple workaround is to call otPlatRadioEnableSrcMatch(*InstanceOT, TRUE) in RCP's FW at reset, but I'm not sure about what it implies.

I think the root cause is that the RCP restore feature missed setting the source match feature. I have pushed a commit PR10055 to resolve this issue. Could you try this commit to check if it could resolve your issue?

Hello,
I will be able to test it tomorrow only.
I will let you know the result.
BR,
Eric

Hello zhanglongxia,
I confirm your commit works fine as expected.
Thank you for your reactivity!
BR,
Eric


document is named "fakedoc"

Whatever the name of the document I load, the DocId is presented as "fakedoc".

Are you using text files or xml files? The fakedoc trick is used for text
files since it can't contain any metadata.
On Fri, Sep 18, 2015 at 2:40 AM Nathan Glenn<EMAIL_ADDRESS>wrote:

Whatever the name of the document I load, the DocId is presented as
"fakedoc".
—
Reply to this email directly or view it on GitHub
https://github.com/desilinguist/ibleu/issues/1.

Yes, I'm using text files. After debugging the application for #2 I saw how this trick worked. As a user it surprised me, though, because my text files were not named "fakedoc", and I was expecting the display to show the name of the hypothesis file (minus .txt).

Indeed. That would be a better option. It just didn't occur to me to do
that when I first wrote this originally. Pull request welcome :)
Thanks for the Chrome stuff too.
On Fri, Sep 18, 2015 at 7:02 PM Nathan Glenn<EMAIL_ADDRESS>wrote:

Yes, I'm using text files. After debugging the application for #2
https://github.com/desilinguist/ibleu/pull/2 I saw how this trick
worked. As a user it surprised me, though, because my text files were not
named "fakedoc", and I was expecting the display to show the name of the
hypothesis file (minus .txt).
—
Reply to this email directly or view it on GitHub
https://github.com/desilinguist/ibleu/issues/1#issuecomment-141589724.

BTW, the fakedoc trick is fully documented in the readme so I don't think
it's that big a deal.
On Fri, Sep 18, 2015 at 7:03 PM Nitin Madnani<EMAIL_ADDRESS>wrote:

Indeed. That would be a better option. It just didn't occur to me to do
that when I first wrote this originally. Pull request welcome :)
Thanks for the Chrome stuff too.
On Fri, Sep 18, 2015 at 7:02 PM Nathan Glenn<EMAIL_ADDRESS>wrote:

Yes, I'm using text files. After debugging the application for #2
https://github.com/desilinguist/ibleu/pull/2 I saw how this trick
worked. As a user it surprised me, though, because my text files were not
named "fakedoc", and I was expecting the display to show the name of the
hypothesis file (minus .txt).
—
Reply to this email directly or view it on GitHub
https://github.com/desilinguist/ibleu/issues/1#issuecomment-141589724.

Yeah, it's not a show-stopper. I just think it would make more sense to the user to use a file name or something.

I agree.
On Fri, Sep 18, 2015 at 19:44 Nathan Glenn<EMAIL_ADDRESS>wrote:

Yeah, it's not a show-stopper. I just think it would make more sense to
the user to use a file name or something.
—
Reply to this email directly or view it on GitHub
https://github.com/desilinguist/ibleu/issues/1#issuecomment-141596248.

I am happy to take pull requests for this as I don't really have time to work on this for now.

Exciting Job Opportunity at Microsoft!
Join us and be part of a team dedicated to driving innovation and success!
How to Apply:
To apply for this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.n7n-dev, @alittlebitfun, @chc273, @deism, @RuslanJabbar, @digiaries, @zaigie, @alaypanov, @honggui, @baireslearn, @bieli, @xujiesh0510, @velocitatem, @plamengj, @Shellishack, @Adityakhurana9499, @PolinaBortnik, @ww-jermaine, @SouthernMine4957, @serpro69


added google and outlook express started

:x: Build TimeHACK AppVeyor-149 failed (commit https://github.com/TimeHACKDevs/TimeHACK/commit/94f148eea9 by @lempamo)

:white_check_mark: Build TimeHACK AppVeyor-150 completed (commit https://github.com/TimeHACKDevs/TimeHACK/commit/fe0d3d653f by @lempamo)


Update next branch with latest changes from master

Update next branch with latest changes from master, using the conflict resolution from #446.
This PR allows us to apply more changes to the next branch before release v0.15. In particular, we plan to merge a change to make the GDT const generic before the release.

Closing, no longer needed.

How can I review this? Is there a way to see conflict resolution changes of the merge or rebase from master into next or we're all of those already handled in #446?

This was just a fast-forward merge. So this updates the branch to commit f5c2b06, which is part of #446 (before the changelog updates). You can just do a git diff or git log, there should not be any differences.

Ah, ok that makes sense, thanks!
I just fast-forwarded a few more commits from #446 including the version bump commit, so CI (and the semver check in particular) passes.


My ALVR testing

Not an issue but I thought it would be nice to write about my experience with ALVR.
I have an oculus Go. My PC is i5 3450, GTX 970.
My first try was with ALVR version 1.9.2
Tried to run the server and got .net framework error. After some searching I found out that I had to do a first run of steamVR for it to be setup correctly.
After that I got a nvenc error. That was easy. I had to install the latest nvidia drivers.
Next I could not get the connect button even though the server was up and I could ping the headset. I had to follow this guide https://steamcommunity.com/app/250820/discussions/0/1290691937709136507/ and delete the "openvrpaths.vrpath" file. After that I could connect.
I tried both a 2.4 and a 5GHz wifi connection with the router directly next to me. 2.4 was unusable. 5GHz was great.
Today I upgraded to 2.0. I also did some performance testing. With the default 30MBps and 2048x1024 resolution I got around 48-55ms total latency with no packet loss.
With 50MBps and 2880x1440 I got around 60-65ms latency, no packet loss but the graphics were greatly enhanced.
I think about 60 or 70mbps I got severe increasing latency which quickly added up to multiple seconds.
I tried Project cars 2, elite dangerous, doom VFR and Bigscreen beta.
Project cars 2 was running great. Totally playable. I tried it with gamepad.
Elite dangerous was also running great. Also with gamepad. I may connect my hotas to try and play for extended period to test immersion.
Doom VFR was stuttering. This game is stuttery though also on certain configurations even on high performance machines.
Bigscreen beta was working great also and with the higher resolution and bitrate very clear.
Right now I am downloading The solus project and subnautica to test. I would like also to install revive to test my oculus games (I also have a rift).
In all my testings I had 72fps according to the nvidia geforce overlay FPS counter. Also on steamVR settings there is a bar on the bottom left which was green with "Framerate good".  Async reprojection and interleaved reprojection were off in steamVR, supersampling also set to 100%.
My wishlist for ALVR is:

maybe a wired connection if it is possible and if it reduces latency with increased picture quality
oculus go onboard sound and microphone support
maybe some more analytics if it is possible like framerate or an in headset latency and fps overlay. Like the way it is with oculus tray tool.

EDIT: I see that I had the go conencted to a N wireless router and the computer connected to a different AP with ethernet. According to the instructions you have to have both connected to the same router and prefferably on AC speeds. I will have to try to see if latency decreases. Any recommended budget AC router/AP?

Thank you for great report!

48-55ms total latency

It's good. I have 60-70ms on same configuration with S8/11n Wi-Fi.

maybe a wired connection if it is possible and if it reduces latency with increased picture quality

Does the "wired connection" mean connect Go on PC with USB cable (adb)?
If it is right, I will add it to the feature request list.

sound and microphone

It need a hard work on implementation and takes a lot of time. Please be patient.

more analytics

It's a good suggestion. I want this feature too.
I'm also using 11n Wi-Fi router and thinking about a new router.
I don't have deep knowledges about Wi-Fi, but I want to buy the router with 1000Base-T.

Does the "wired connection" mean connect Go on PC with USB cable (adb)?
If it is right, I will add it to the feature request list.

Yes, this is correct. Connect Go on PC with usb cable (adb). But we need this only if it reduces latency significantly, or it allows significantly larger bitrates.
To be more precise, I would not mind tethering if it enhances the experience a lot.


Add option to run MLAE circuits separately

Summary
Certain backends, such as IonQBackend in qiskit-ionq, don't support circuit lists in the run method (see here). In this case, lists that contain a single element are allowed, but not lists with multiple elements. With MLAE it shouldn't matter if the independent circuits are submitted as one job or as several separate job jobs. The latter will probably take more time in most cases, so I kept the current behavior as the default and added an optional argument that splits the circuits into separate jobs.
Details and comments
run_circuits_as_one_job is a bit of an awkward name, but I couldn't think of anything better.

The QuantumInstance class is supposed to take care of any backend specific requirements in this regard and should be splitting what the algorithm emits in order to meet the backend requirements. I recall there was an earlier issue around this but I thought it was resolved to allow the algorithms to work correctly.

(By the way, you don't need to update the PR when there are changes on main - it'll get updated automatically during the merge process.)

Right, I wasn't sure if this repo had up-to-date merge checks ;) Thanks for letting me know!

The QuantumInstance class is supposed to take care of any backend specific requirements in this regard and should be splitting what the algorithm emits in order to meet the backend requirements. I recall there was an earlier issue around this but I thought it was resolved to allow the algorithms to work correctly.

Huh, it definitely doesn't work with the IonQBackend implementation from azure-quantum at the moment. I wasn't able to run QuantumAmplitudeEstimation either, even though the circuit list contains just one element in this case. I have PR in that repo to at least allow circuit lists of lenght one for now (https://github.com/microsoft/qdk-python/pull/201), because currently I can't use any of the Qiskit amplitude estimation classes. If that PR gets merged, I'd expect QAE, IAE, and FAE to work, but not MLAE, since it typically runs several circuits at once - I even used the debugger to verify that the IonQBackend#run method does indeed receive several circuits at once in the case of MLAE. Looking at the code of IonQBackend#run there's no chance that could work right now (this is the line that's currently throwing an exception, but some other calls like circuit.name definitely also won't work if circuit is a list).
I know the story is pretty much the same in qiskit-ionq. A while ago they added the option to handle single-element circuit lists (https://github.com/Qiskit-Partners/qiskit-ionq/pull/71). If run receives a list with several elements, they explicitly throw a runtime exception.

Huh, it definitely doesn't work with the IonQBackend implementation from azure-quantum at the moment.

Ok, understood. But I think the right approach is to figure where/why the mismatch is occuring. The QI code uses a run_circuits function which looks for max_experiments property from the backend - maybe that is not being set correctly https://github.com/Qiskit/qiskit-terra/blob/0d85bfe3f4765a64f8e018514daa3898d9cf32af/qiskit/utils/run_circuits.py#L478-L484 It is, as you see, also influenced by an environment variable as retrieved here https://github.com/Qiskit/qiskit-terra/blob/0d85bfe3f4765a64f8e018514daa3898d9cf32af/qiskit/utils/run_circuits.py#L38
Maybe try setting the env var to 1 and see if things work. Then perhaps we cab figure what needs changing to make things work. I would rather not have algorithm specific setting for this aspect since others, like VQE etc etc will produce lists of circuits and all should work with any backend where the QuantumInstance and its use of run_circuits correctly fragments this list into one or more jobs according to any backend limits.

Maybe try setting the env var to 1 and see if things work.

I was just looking at this bit of code :) MLAE does work if I set it to 1 (and I use the version of azure-quantum from my PR). Is having to set the variable manually on one's machine the intended approach?

Is having to set the variable manually on one's machine the intended approach?

No, that was intended as a manual override that could be used. Ideally of course things should "just work". In that regard the run_circuits was designed to be able to configure itself via information about the backend. This is the other path in the if test to set max circuits per job. If its a local simulator, as per its determination, then there is no limit. Otherwise it sets from max_experiments - what is that for your backend - not sure how much of the configuration is standard/expected? I assume its not 1 - maybe it returns None - but a cursory look at the code I think that ought to fail later when it looks at splitting the list into chunks if that were the case.

Ideally of course things should "just work".

My bad, it looks like it does work even without setting this variable! Apologies, I was focusing on the fact that the IonQBackend#run methods in both azure-quantum and qiskit-ionq were throwing these explicit errors when faced with a multi-element circuit list. I was sure I had debugged MLAE to make sure this method was actually receiving multi-element circuit lists, but it looks like I hadn't (I probably just looked at run_circuits and called it a day without analyzing the code that's executed between the two methods). Everything should be fine with QAE, IAE, FAE, and MLAE, once the azure-quantum PR gets merged and the backend stops freaking out when it sees a circuit list with one element in it. Sorry again, debugging things across three libraries gets tricky ;)
Note: I only verified this for azure-quantum. I don't think I have the IonQ token that would be necessary to use the qiskit-ionq provider.


Allow ListPackages to act on all Registries

It should be possible to do something like this:
]ListPackages [*]FilesAndDir

The goal is to get a list for FilesAndDir from all Registries reported by ]ListRegistries.
ListVersions should probably allow the same syntax.

Implemented in 0.93.0


Development: Remove automatic PR comments to use Draft PRs

@actions comments on non-draft pull requests without the ready for review label:

Let's have a vote if we should keep this Action around:

:+1: We should DISABLE the automatic comment.
:-1: We should KEEP the automatic comment.

(Use reactions to vote.)

It might be useful to instead automatically add the label


Add support for SQL Server

While Postgres is using $1, $2 etc for parameter substitution, in Microsoft SQL Server it is @P1, @P2 etc.
We need support for that in Prisma https://github.com/prisma/prisma/issues/3861
We can also totally do the PR and tests for that :)

Do you happen to know if the driver reads from a different parameter (i.e. this can work automatically) or whether we need to do it via some configuration? Either way shouldn't be a big deal, here's how we generate the param list for postgres: https://github.com/blakeembrey/sql-template-tag/blob/a10d0349f5ffdd0d248edb760b1102aec461517f/src/index.ts#L71-L73. It's really simple, I'd just want to make sure we don't degrade performance.

Hey! I work with Prisma and I wanted to pick this up, so I did some research.
As I understand it, this library supports the packages pg & mysql. In order to support SQL Server, I think you'd need to add support for mssql. It seems to be the library that Microsoft endorses.
However, the mssql package already has support for prepared statements, and as far as I can tell, there's no simple way to send out a prepared-statement-style query with parameters like pg & mysql allow you to do.
What I'm getting at is that I don't think sql-template-tag can "support" SQL Server. I think this issue can be closed, as it is a no-op.
Hopefully I made myself clear, please let me know if I'm mistaken!
PS: As far as https://github.com/prisma/prisma/issues/3861 goes, I do not think that it is this library reponsibility to support MSSQL, we would need to add logic for generating valid MSSQL statements in ttps://github.com/prisma/prisma itself.

We can probably open a PR to mssql, from what I can see there’s the input method that just internally sets this.parameters. That would be one way to go and support it natively.
The alternative approach is actually really simple. Because MSSQL already support SQL template strings (https://github.com/tediousjs/node-mssql/blob/5a4f77ac15ce66c89f6da168fddeeec3806b6360/lib/global-connection.js#L145-L146) you can actually just do mssql.query(sql.strings, sql.values).

This may be interesting for you too, since this is effectively the original proposal (they do it internally to support the template literal already): https://github.com/tediousjs/node-mssql/blob/72457687b3dd11d709ac5ff4ddced57154af2fc9/lib/base/request.js#L69-L95.

That said, you are correct and this can’t be supported “magically” like the others. I’d be happy to put a recipe for this in the README though, since the second format is extremely simple and almost as easy as query(sql) from Postgres/MySQL. It’s definitely not obvious that it would work so easily without seeing how they implemented the template literals.

@blakeembrey

you can actually just do mssql.query(sql.strings, sql.values) which would be “faking” the template literal.

Ah interesting, I didn't look at mssql.query.

This may be interesting for you too, since this is effectively the original proposal

Definitely is!

this can’t be supported “magically” like the others.

Yep, I was hesitant to send a PR because of this (still am!) I also agree that your suggestion of adding a recipe for this in the README is probably the right way to go.

Added a section in the README although it's probably incomplete: https://github.com/blakeembrey/sql-template-tag#recipes


binary() field in Schema Builder

I hardly need to create a column of type binary$length) in MySQL but had to assert, that the $fluent->binary() method isn't building a BINARY but a BLOB field.
So what could be a possible sulution for this? Adding a new Fluent Method? In Laravel 3 I was easyly able to hack and dirty change the frameworks code to my needings, but with composer it luckily not possible.
Any ideas guys?
Cheers,
Bodo06

I'd definitely agree that this should be supported in some form, primarily for support for UUIDs.

https://github.com/rafis/schema-extended

+1 for adding binary support

I agree this is a bug, binary isn't the blob in mysql

+1

+1

The only reason I can think of why this isn't implemented is that not all database types support the binary column. If that isn't the case, I would really like to see this added to Laravel.

Actually Laravel support 4 DB and all of them support binary or binary equivalent

MySQL
PostGreSQL
SQL Server
SQLite
DB

(var)binary
bytea
(var)binary
blob
Data Type

In SQLite according to the FAQ there is no data type enforcement, you can insert string to integer column, binary to string column, the only exception is primary key id only allow unsigned 64 bit integer, but i recomend to use blob(256) as datatype
References
https://www.postgresql.org/docs/9.0/datatype-binary.html
https://docs.microsoft.com/en-us/sql/t-sql/data-types/binary-and-varbinary-transact-sql?view=sql-server-2017

if you want better migrations use https://github.com/cakephp/phinx
it support binary, varbinary etc.
https://book.cakephp.org/3.0/en/phinx/migrations.html#valid-column-types
https://book.cakephp.org/3.0/en/phinx/migrations.html#valid-column-options

It seems like you can just set the character set to binary as a workaround:
$table->char('url_hash', 16)->charset('binary');

This is actually shown as a real binary column type with a length of 16 in MySQL Workbench.


feat: new regex filter feature

Purpose of PR?:
introduce a new filter feature using re

I think the Apply and Cancel buttons aren't really needed and just clutter the interface, instead any fetch or similar should take the current content of the filter into account and hitting enter in the regex input should just do an update.
Also, the current changes somehow break the operations list when resizing, it no longer changes sizes with the window.

Nice!

The regex filter overrides the category filter. I think we want to have bot in an AND fashion.
I do not understand the "ignore archived" tickbox. In this view "archived" operations shall always be ignored. There is a view opened by the "Operation Archive" button. "Someone" broke the archiving fully and completely, but my intent is to remedy this soon. Ideally the Operation archive view gets its own regex filter.

Further, we need to think about the interaction between the multiple flightpath widget and this filter. Shall there be any?
Currently, the widget does not change its view when the category in the mainwindow is changed, which I deem a bug (not necessarily yours to fix, but would be nice and should be included in the fix that only "attaches" checked operations).

Further, we need to think about the interaction between the multiple flightpath widget and this filter. Shall there be any? Currently, the widget does not change its view when the category in the mainwindow is changed, which I deem a bug (not necessarily yours to fix, but would be nice and should be included in the fix that only "attaches" checked operations).

yes you have to reopen the topview for the reflection to happen definitely need a fix and I'll do my best and i also want the filtered operations to reflect in docking widget but i having trouble to find how this category filtered operations are reflecting in the docking widget


How to start download in background?

If you want to download in the background, you just use DownloadTask#enqueue like wiki said, it will run the download task on the background thread.
If you want a download service running on the separate process, on the current version we don't provide yet, but on the further version we will implement it with DownloadDispatcher on com.liulishuo.okdownload:process what is like sqlite or okhttp(refs wiki)


fix(react-dom peer dependency issue)

I made it so that react-dom is a peer dependency, sorry about that. http://nx.dev does that automatically on publish if we don't declare the peerDependency explicitly in a sub-package, it will assume we want it added as a dependency automatically.

Thanks.


Picovoice Console Issue:

Make sure you have read the documentation, and have put forth a reasonable effort to find an existing answer.
Expected behaviour
Wake word integration should initialize.
Actual behaviour
Wake word fails to initialize on any of our "heyortho.com" domains. We get this error:
TypeError: Cannot create property 'error' on string 'Error: 'pv_porcupine_init' failed with status IO_ERROR'
at w.error (app.51f5b825.js:1:106046)
at Proxy.ppnErrorFn (431.60546cdf.js:1:45249)
at vendor.3aa69d2a.js:116:2606756
at c (vendor.3aa69d2a.js:116:2486292)
at Generator._invoke (vendor.3aa69d2a.js:116:2487622)
at Generator.throw (vendor.3aa69d2a.js:116:2486721)
at a (vendor.3aa69d2a.js:116:2485389)
at l (vendor.3aa69d2a.js:116:2485628)
u @ vendor.3aa69d2a.js:170
ADDITIONALLY: Our device limit used devices is INCREMENTED BY 1 with each load even on the same device!
Picovoice works fine on localhost, just not on our heyortho domains. This started a week ago.
Steps to reproduce the behaviour
I'd have to give you access to our system, please contact us.
(Include enough details so that the issue can be reproduced independently.)

Resolved via email.


SP People and Roles Integration (+Init GP's)

Issue #: /bcgov/entity#10689
Description of changes:

Refactoring PeopleAndRoles/OrgPersons for Corrections/Alterations for SP/GP (ChangeOfReg) functionality
Layout and design updates for previous iterations of components to bring up to date
Implements HelpText component
Updating unit test for previous Corrections and Change of Registrations
Includes resourcing in components to accommodate new variations
Update field validations on People and Roles and when to apply them, also includes

NOTE:

Partners Orgs is still work to come. This just sets up the foundations for it, the nuances are still to come.
API is still missing officer.email values but the filing template mixin is built to handle it when it is returned from the /parties

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Nice looking code! I realize some of it is in flux atm.
Do you need a FF, or no because there are no firms in the system atm?
I assume you've fully tested these changes w.r.t. other entity types. Please also call out these changes so Riyaz learns about them.

Thank you!
Yea, the dev work has sort of a required a Correction/Alteration/ChangeOfReg filing to be open at all times, at least thats how i do it. Especially since we were pulling some bits out and into resource files.
We currently have the flag to protect the filing as a whole, which should be enough for now! Only changes visible in our Correction filing is aesthetic stuff, layout improvements :)


Return list as response

Hello everyone,
in the example of https://github.com/flasgger/flasgger/blob/master/README.md#using-marshmallow-schemas it shows how to use nested to define a list as response. It returns e.g.
{
   'cmyk': ['cian', 'magenta', 'yellow', 'black']
}

How can I modify it such that it returns directly a list? For the above example, the expected return value would be
['cian', 'magenta', 'yellow', 'black']

Is that something that would be considered wrong from a design perspective?
I found a way to do it, but it breaks the apidocs overview. I want a view that returns a list of orders. Currently, this is what I did
class OrderSchema(Schema):
    id = fields.String(description="Identifier")

class OrdersView(SwaggerView):
    decorators = None
    tags = ["user"]
    summary = "List orders"
    description = "Returns a list of all orders"
    parameters = None
    responses = {
        200: {"description": "OK", "schema": OrderSchema}
    }
    validation = False

    def get(self):
        orders = Oders()
        result = OrderSchema().dump(orders, many=True)
        return jsonify(result)

It works, but in the apidocs, the response value is now not a list, but only one OrderSchema object.
I could introduce a OrderListSchema, that has a nested OrderSchema field, but then the return value would not be directly the list, but a dict with the list in the field's name.

Sorry, but the current code does not support this feature. I think it may be incorrect from a design perspective: assume that your response content type is “application/json” then ['cian', 'magenta', 'yellow', 'black'] is not a valid json. You would have to turn it into
{
   'cmyk': ['cian', 'magenta', 'yellow', 'black']
}

If you intend your response content type as “text/plain”, ("['cian', 'magenta', 'yellow', 'black']" string as an example), then it should be okay. You can probably specify it without a Schema object and as a string.

Thank you for that. I think a list would be valid JSON still from a design choice a primitive as direct response is not recommended. I found this after reading your hint and found it helpful
https://stackoverflow.com/questions/19623339/is-a-list-array-valid-json


Add a quick and dirty python decoder

Quick and Dirty python tool used to decode amiibo content
Can be chained with amiitool:
./amiitool -k keys.bin -d -i amiibo.bin | ./amiidecode.py -k keys.bin
Signed-off-by: Stany MARCEL<EMAIL_ADDRESS>

I'm not sure this repository is the best place for a Python implementation - IMHO that should go in another repo, probably as a library so it can be reused by other developers (unlike this amiitool thing which I never worked around making it a proper standalone library)

I proposed it as an external tool that can decode the output of amiitool, I used it to understand data format. And Implement the copy function.
Python is nice to do quick prototyping things like that, but I agree that ideally a function in amiitool that permit to map/decode and re-encode a detailed structure would be nice.
I leave my python script in a fork if someone is interested in the decoding work I started.

Actually rather than hacking amiitool for that I meant more like porting the whole thing to Python - I only did this in C because that's what I preferred to use back then, but it would be more useable for most developers in a higher level language.
I'll see if this summer I have so free time to work on it.

It might be interesting to keep a "low level" library that can be embedded in applications written in other language like android/java or python.
For this purpose C is a good choice as almost any language has the possibility to bind with it.
For example the TagMo android application use a copy of amiitool https://github.com/Kidel/TagMo/tree/master/ndk/jni.


Move video of Victor to Donation page

@davidmolina - do you want to replace that video with the Mission video?

Per our call, only mission video on the landing and /about and the Marine video to a page to help donors make a donation.

On Jun 30, 2016, at 7:12 PM, Conrad Hollomon<EMAIL_ADDRESS>wrote:
@davidmolina - do you want to replace that video with the Mission video?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Tooling to generate OLM metadata

Metadata published to OperatorHub (see https://github.com/knative/eventing-operator/issues/68) is initially created manually.
For future versions, we can generate the OLM metadata using the manually created one.
Required:

Tooling itself
Documentation

/assign @aliok

We can use Operator SDK cli to generate CSV also for non-Operator-SDK operators.
I've done some experiments here: https://github.com/aliok/eventing-operator/tree/olm-experiments-bundle
Filed 2 tickets:

https://github.com/operator-framework/operator-sdk/issues/2266
https://github.com/operator-framework/operator-sdk/issues/2267

and objected https://github.com/operator-framework/operator-sdk/pull/2249 which makes Operator SDK cli unusable for upstream Knative eventing operator.
Gonna leave it there until Operator SDK team makes a decision.
I don't think we need to pursue any other tool. We just need to give Operator SDK team good feedback to help them support what we need.

Update:
I am still helping Operator SDK team to make the CLI supporting the cases we need at Knative operators: https://github.com/operator-framework/operator-sdk/pull/2511
We're very close to have tooling ready for our cases.

https://github.com/operator-framework/operator-sdk/pull/2511 has lots of progress.
Once that PR is merged and we fix https://github.com/knative/eventing-operator/issues/128, we can use operator-sdk as is to generate OLM metadata.


Face

omm
On Fri, Jul 29, 2022, 9:48 AM Nghiyaya @.***> wrote:

—
Reply to this email directly, view it on GitHub
https://github.com/chinoogawa/fbht/issues/46, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY2MPVVIKLYMRFJPGJPU3BLVWNEPZANCNFSM547J7DBQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


How to intercept navigation, such as the target page requiring login to access

@Kuki93 Not sure if I understand the case. If a user is not logged in, navigate to the login screen instead of going to the target page. It's your business logic that controls the navigation, so you can check the login state before doing the navigation. No need to intercept anything.
Can you please provide more details about your use-case?

thanks, I have already solved the problem。


Include Windows 11 processring in MicaWPF

I've considered getting one of Mahapps' libraries just to get the processring but it looks like Windows 10's processring. So it would be nice if MicaWPF also include that but for Windows 11.

We do have something called a ProgressRing in MicaWPF already

<controls:MicaWindow
    x:Class="MicaWPF.Test.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:controls="clr-namespace:MicaWPF.Controls;assembly=MicaWPF"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:local="clr-namespace:MicaWPF.Test"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    Width="800"
    Height="450"
    mc:Ignorable="d">
    <Grid>
        <controls:ProgressRing />
        <controls:ProgressRing IsIndeterminate="True" Margin="458,187,282,187" />
    </Grid>

</controls:MicaWindow>

Oooooh okay, i didn't see it in the Wiki so i assumed it wasn't there.

I will add more docs to the controls this week-end


SpreadsheetPatternEditorWidget pattern example doesnt support color

Closed by https://github.com/mP1/walkingkooka-spreadsheet/pull/3058


Bug in generation of constructor initializers

This test should behave differently (from tests/co19/src/Language/Expressions/Instance_Creation/New/execution_t09.dart):
import 'tests/co19/src/Utils/expect.dart';

var evalOrder = '';

log(var s) {
  evalOrder = '$evalOrder$s';
}

class A {
  A() : this.a2();
  A.a2() {
    log("2");
  }
}

class B extends A {
  var v;

  B.b2() : super(), v = log("1") {
    log("3");
  }
}

main() {
  new B.b2();
  Expect.equals("123", evalOrder, "Super constructor was not executed properly!");
}

This get compiled to:
  class A extends core::Object {
    constructor •() → void
      : this test::A::a2();
    constructor a2() → void {
      test::log("2");
    }
  }
  class B extends test::A {
    field dynamic v;
    constructor b2() → void
      : super test::A::•(), test::B::v = test::log("1") {
      test::log("3");
    }
  }
  static field dynamic evalOrder = "";
  static method log(dynamic s) → dynamic {
    test::evalOrder = "${test::evalOrder}${s}";
  }
  static method main() → dynamic {
    new test::B::b2();
    Exp::Expect::equals("123", test::evalOrder, "Super constructor was not executed properly!");
  }

It is probably supposed to re-order the initializers
from
 B.b2() : super(), v = log("1") {
    log("3");
  }

to
 B.b2() : v = log("1"), super() {
    log("3");
  }

/cc @peter-ahe-google

I believe this has been fixed.


Handles overrides in a cleaner way

We just weren't passing it to the node-determining function, so raw_execute wasn't calling the validator with the right inputs.
Fixes #307
[Short description explaining the high-level reason for the pull request]
Changes
How I tested this
Notes
Checklist

[ ] PR has an informative and human-readable title (this will be pulled into the release notes)
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future TODOs are captured in comments
[ ] Project documentation has been updated if adding/changing functionality.

See as an example


New job are inserted in jobs table of system database instead of the current tenant one

Description
When I've use the Queue management of Laravel v5.7.26 with Tenancy 5.x-dev, all new jobs were inserted in the jobs system table instead of jobs tenant table.

Information

hyn/multi-tenant version: 5.x-dev
laravel version:5.7.26
database driver and version: mariadb latest
php version:7.3

I want queue on each tenant and, in plus, one for system.
Purpose of those queue, manage mail.
Code I've use to insert in job:
$my_email_information = [
    "from" => "...",
    "to" => "...",
    ...
];

displatch(new myJob($my_email_information))
    ->onQueue('mail');

The job class:
...
class myJob implements ShouldQueue {
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    public $mail_informations = null
    public function __construct(array $data) { $this->mail_informations = $data; }
    public function handle () {
        ...
    };
}

By asking some help in the Discuss chat, @Plytas asking me to print the current tenant in handle method, and there was the good tenant displayed.
In the record in job table on system database, I've a "tenant_id:9" in the playload field (that are the good one too).
So don't understand why the record don't going into the good database.

tenancy.php config

use Hyn\Tenancy\Database\Connection;

return [
    'models' => [
        // Must implement \Hyn\Tenancy\Contracts\Hostname
        'hostname' => \Hyn\Tenancy\Models\Hostname::class,

        // Must implement \Hyn\Tenancy\Contracts\Website
        'website' => \Hyn\Tenancy\Models\Website::class,
    ],

    'middleware' => [
        // The eager identification middleware.
        \Hyn\Tenancy\Middleware\EagerIdentification::class,

        // The hostname actions middleware (redirects, https, maintenance).
        \Hyn\Tenancy\Middleware\HostnameActions::class,
    ],
    'website' => [
        'disable-random-id' => false,

        'random-id-generator' => Hyn\Tenancy\Generators\Uuid\ShaGenerator::class,

        'uuid-limit-length-to-32' => env('LIMIT_UUID_LENGTH_32', false),

        'disk' => null,

        'auto-create-tenant-directory' => true,

        'auto-rename-tenant-directory' => true,

        'auto-delete-tenant-directory' => false,

        'cache' => 10,
    ],
    'hostname' => [
        'default' => env('TENANCY_DEFAULT_HOSTNAME'),

        'auto-identification' => env('TENANCY_AUTO_HOSTNAME_IDENTIFICATION', true),

        'early-identification' => env('TENANCY_EARLY_IDENTIFICATION', true),

        'abort-without-identified-hostname' => env('TENANCY_ABORT_WITHOUT_HOSTNAME', false),

        'cache' => 10,

        'update-app-url' => false,
    ],
    'db' => [
        'default' => env('TENANCY_DEFAULT_CONNECTION'),

        'system-connection-name' => env('TENANCY_SYSTEM_CONNECTION_NAME', Connection::DEFAULT_SYSTEM_NAME),
        'tenant-connection-name' => env('TENANCY_TENANT_CONNECTION_NAME', Connection::DEFAULT_TENANT_NAME),

        'tenant-division-mode' => env('TENANCY_DATABASE_DIVISION_MODE', 'database'),

        'password-generator' => Hyn\Tenancy\Generators\Database\DefaultPasswordGenerator::class,

        'tenant-migrations-path' => database_path('migrations/tenant'),

        'tenant-seed-class' => false,

        'auto-create-tenant-database' => true,

        'auto-create-tenant-database-user' => true,

        'tenant-database-user-privileges' => 'SELECT, INSERT, UPDATE, DELETE, CREATE, DROP, REFERENCES, INDEX, ALTER, CREATE TEMPORARY TABLES, LOCK TABLES, EXECUTE, CREATE VIEW, SHOW VIEW, CREATE ROUTINE, ALTER ROUTINE, EVENT, TRIGGER',

        'auto-rename-tenant-database' => true,

        'auto-delete-tenant-database' => env('TENANCY_DATABASE_AUTO_DELETE', false),

        'auto-delete-tenant-database-user' => env('TENANCY_DATABASE_AUTO_DELETE_USER', false),

        'force-tenant-connection-of-models' => [
        ],
        'force-system-connection-of-models' => [
        ],
    ],

    'routes' => [
        'path' => base_path('routes/tenants.php'),

        'replace-global' => false,
    ],

    'folders' => [
        'config' => [
            'enabled' => true,

            'blacklist' => ['database', 'tenancy', 'webserver'],
        ],
        'routes' => [
            'enabled' => true,

            'prefix' => null,
        ],
        'trans' => [
            'enabled' => false,

            'override-global' => true,

            'namespace' => 'tenant',
        ],
        'vendor' => [
            'enabled' => true,
        ],
        'media' => [
            'enabled' => true,
        ],
        'views' => [
            'enabled' => true,

            'namespace' => null,

            'override-global' => true,
        ],
    ],
];

queue.php config
...
    'default' => env('QUEUE_DRIVER', 'sync'),
    'connections' => [
        'sync' => [
            'driver' => 'sync',
        ],

        'database' => [
            'driver' => 'database',
            'table' => 'jobs',
            'queue' => 'default',
            'retry_after' => 90,
        ],
...

This has to do with Job Auto Tenancy we implemented in the dev branch.
Provided possible solutions/things to think of through Discord, will be tested by @HpiGuillaume .

@HpiGuillaume Can this be closed, did we provide you with enough information or is more info still needed?

@ArlonAntonius sorry to not have tested since 6 days but I don't have time :/
I use only the system database now for my jobs so it's not blocking me. I prefer fix more blocking issue in my prod :)
I think it's bug, or just a mistake in my configuration files.
I close with this message, but re-open when I've taking time to test the possible solution !


A generic limitation to graphql queries

When Ent generates a graphql query, it uses the relay style of queries with filters and order, for example I have a function to query profiles:

this is a small part of the "where" struct,how can I limit how many parameters can be sent in the where parameter? I want to effectively limit it, there can be a lot of nesting just inside this variable.
and while I can use a limiter for complexity of the graphql query, I cannot limit the complexity of the variables, I wonder what is the right way to preven overly complex where queries, and if ent actually have any limit built in - if I do a where query with 1000 nested joins, will it execute? what about 1000000?
There should be some limit since the graphql api is exposed, so I wonder what is the right way to solve this.
I didn't find any code that limits this.

Not really related to ent, but gqlgen. See complexity how to deal with overly complicated queries there.


mq5 files is missing

Hi, how are you?
I am unable to compile mq4 files on mt5. Is is possible to commit the mq5 files too? Thank you.

have any tutorial for installation?

Hello,
MQL5 source code may be released in the future.
Thank you.

Any ETA for that? Thanks.

waiting :(

Is it still posible to get the MQL5 files? Would be very helpful!

Hello,
MQL5 source code may be released in the future.
Thank you.

Hello, would you pls post MQL5 Source Code? I´m gonna do a donation for that.

Hello,
MQL5 source code may be released in the future.
Thank you.

Please, share the the MQL5 source code.
I am sure many of us would donate for that.
Thanks for everything!!!

Hello friend, how are you? Could you configure the copy correctly? have you used it?

Hello,
MQL5 source code may be released in the future.
Thank you.

Hello, would you pls post MQL5 Source Code? I´m gonna do a donation for that.

I think the solution should be considered using MQTT instead of ZeroMQ.
If he doesn't release the mql5 source code then that's his right.
Donation should not be used as a condition for releasing source code, it should only be a voluntary action.

I concur with @EnziinSystem that donations should remain voluntary, even if the conversion to MQL5 isn't undertaken.
It might be worthwhile to consider undertaking this task on your own. It's undeniable that @jiowcl did an exceptional job by releasing the code. Although I no longer use the ZeroMQ protocol today, it was immensely helpful to me at the time.
For those looking to easily adapt the MQL4 code for MT5, I highly recommend utilizing the code available at this MQL5 link. It’s advisable to make the necessary adjustments to ensure the code runs smoothly in MT5’s native language.
Here's the link for reference: https://www.mql5.com/en/code/16006

can you please give us the mql5 code .. thanks

I built a copy trading master-slave EA based on the MQTT and JSON library here:
https://github.com/gmag11/MQTT-MQL5-Library
https://github.com/vivazzi/JAson

Hi!
I’d like to share with you my project for a robust order replication
solution for MetaTrader 5, focusing on efficiency, portfolio control, and
advanced risk management. We’re currently seeking partners, associates, and
developers to help expand iMentore internationally.
If you have access to brokers or trader networks—or if you’re interested in
bringing fresh technical insights—we’d love to see how our projects might
complement each other. Our goal is to merge technical expertise with
commercial reach to create a standout replication tool.
If this sounds interesting, feel free to reach out at @.***
Thanks, Breno
www.imentore.com

Message ID: @.***>


OpenAPI v3 RouterBuilder has no RxJava1 implementation

Version
4.2.3
Context
When I add the io.vertx:vertx-rx-java:4.2.3 dependency, I expect that there should be a class something like:
io.vertx.rxjava.ext.web.openapi.RouterBuilder which can take io.vertx.rxjava.core.Vertx as an argument to it's 'rxCreate` method.
Do you have a reproducer?
No
Steps to reproduce

Add the vertx-rx-java and vertx-web-openapi dependencies to a project
In a Verticle, attempt to use RouterBuilder.rxCreate(vertx, "/path/to/openapi.yaml");

Extra

I DO see a io.vertx.reactivex.ext.web.openapi.RouterBuilder class, but it's rxCreate method signatures do not match.

It's seems only the RxJava 2 and 3 bindings are generated: https://github.com/vert-x3/vertx-rx/search?q=vertx-web-openapi

Actually, this is a duplicate of https://github.com/vert-x3/vertx-rx/issues/256
Please upgrade to RxJava 3 (RxJava 1 nor 2 are not supported anymore)

Ahhhh! Then I highly recommend a big noticeable message on the
documentation pages first them.
Deven
On Mon, Jan 31, 2022, 4:56 AM Thomas Segismont @.***>
wrote:

Actually, this is a duplicate of vert-x3/vertx-rx#256
https://github.com/vert-x3/vertx-rx/issues/256
Please upgrade to RxJava 3 (RxJava 1 nor 2 are not supported anymore)
—
Reply to this email directly, view it on GitHub
https://github.com/vert-x3/issues/issues/596#issuecomment-1025558313,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAYK27TWDLPIV6AZJAHJPTUYZMGFANCNFSM5NEPLQ6Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

On this one? https://vertx.io/docs/vertx-rx/java/

Message ID: @.***>

I see no notice there...

I meant: what do you think about adding a notice on vertx-rxjava 1 doc ?

Message ID: @.***>

Yes, I think that makes a great deal of sense if it truly is end-of-life.


Introduction - Morten Linderud

Yo!
I'm interested joining this Working Group. My Name is Morten Linderud and in my
spare-time I work on Arch Linux as part of the Reproducible Builds team,
Security team and as a packager.
I have been contributing to the security team since early 2017, and have helped
publish a few hundred advisories to our users, along with discussing security
issues in the distro. As cross-distribution collaboration is important, I made
an attempt last year to establish ##distro-security on freenode to have some
common ground with members of other Linux distributions security teams.
Currently there are member from Debian, Ubuntu, OpenSUSE, Arch, Red Hat, Void
Linux and a few other distributions present in the channel.
My main interests in contributing to this team is easier ingestion and
processing of CVE data, and easier vulnerability management for F/OSS Linux
distributions.
Apart from my F/OSS work, I have a masters degree from University of Bergen,
where I wrote about Debian supply chain security, and work as a security
engineer at BDO Cybersecurity.

@Foxboron welcome to the WG!

@Foxboron Can you create a PR similar to #20 to add yourself to the list of WG members?

Now that #22 has been merged, this issue can be closed. Once again welcome to the WG!


[WARN ][com.floragunn.searchguard.filter.SearchGuardActionFilter] Cannot determine types for indices:admin/exists (class org.elasticsearch.action.admin.indices.exists.indices.IndicesExistsRequest) due to types method not found

Hello,
Not really an issue, just a warn.
I keep seeiing this warning appear when I login with a user :
[WARN ][com.floragunn.searchguard.filter.SearchGuardActionFilter] Cannot determine types for indices:admin/exists (class org.elasticsearch.action.admin.indices.exists.indices.IndicesExistsRequest) due to type[s]() method not found

Any idea what it is ?
Thanks

It would really help if you could explain this warning as I have this error coming up everytime I try something new :
[2015-06-24 15:41:39,034][WARN ][com.floragunn.searchguard.filter.SearchGuardActionFilter] Cannot determine types for indices:admin/exists (class org.elasticsearch.action.admin.indices.exists.indices.IndicesExistsRequest) due to type[s]() method not found
[2015-06-24 15:41:39,139][WARN ][com.floragunn.searchguard.filter.SearchGuardActionFilter] actionrequestfilter.readonly Action 'indices:admin/exists' is forbidden due to DEFAULT
[2015-06-24 15:41:39,158][ERROR][com.floragunn.searchguard.filter.SearchGuardActionFilter] Error while apply() due to java.lang.RuntimeException: Action 'indices:admin/exists' is forbidden due to DEFAULT for action indices:admin/exists
java.lang.RuntimeException: Action 'indices:admin/exists' is forbidden due to DEFAULT

Thanks !

This means that if you have configured types in your acl this would be ignored for the above request cause no type information is associated with the request (which is natural for 'indices:admin/exists'). If you have not configured types you can safely ignore this warning.

Just to be sure, by "type" you mean "filter type" ? Like for example the readonlyfilter you configured in your example ?
The issue is that an error occurs because of this.

no, i mean types which live inside an index (https://www.elastic.co/guide/en/elasticsearch/guide/current/_document_metadata.html) in terms of _type or class of object

Oh ok, that's why I get this warning, because I did configure types on some index

pls post your acl

Config :
# Configure the actionrequestfilter to allow or forbid action
searchguard.actionrequestfilter.names: ["readonly"]
searchguard.actionrequestfilter.readonly.allowed_actions: ["indices:data/read/*", "*monitor*"]
searchguard.actionrequestfilter.readonly.forbidden_actions: ["cluster:admin*", "indices:admin*", "indices:data/write*"]

# Configure document level security (dls) filter
searchguard.dlsfilter.names: ["dashboard","index-pattern"]
searchguard.dlsfilter.dashboard: ["term", "title","Dashboard logs", "false"] # if field==value (or not) match
searchguard.dlsfilter.index_pattern: ["term", "index-pattern","logstash-*", "false"] # if field==value (or not) match

and this acl rule :
    {
      "__Comment__": "Default is to execute all filters",
      "filters_bypass": [],
      "filters_execute": ["*"]
    },

    {
    "__Comment__": "For role *admin* all filters are bypassed (so none will be executed). This means unrestricted access.",
    "roles": ["admin"],
    "filters_bypass": ["*"],
    "filters_execute": []
    },

    {
      "__Comment__": "Any authenticated user can do anything on the '.kibana' index - no filter will be executed",
      "indices": [".kibana"],
      "filters_bypass": ["*"],
      "filters_execute": []
    },

    {
       "__Comment__": "user1 has document level filter set for index_pattern",
       "users": ["user1"],
       "indices": [".kibana"],
       "filters_bypass": [],
       "filters_execute": ["dls.dashboard"]
}

What I am trying to do is give access to my ".kibana" index to all users and then filter on other index to control access to data and filter on index-pattern, visualisation and dashboards (which are different _type of the .kibana index) to control access to visualtisations and dashboard !

you have no types configured in your acl (this would look like "indices": ["index:type"]) so you can ignore this warning

the last rule from https://github.com/floragunncom/search-guard/issues/16#issuecomment-115268560 has no effect because the second last rule allows all for .kibana index

Oh yes indeed, this config must be giving access to kibana but not doing any filter on index (I tried multiple different configs).
My main issue is how to give read and get access to kibana server and then filter on _types ? Because the only way I am able to give access to kibana for a user is to write bypass : ["true"] in my acl rule, which then bypass all dlsfilter or flsfilter

Ok so this means if I put :
{
      "__Comment__": "Any authenticated user can do anything on the '.kibana' index with  _type:dashboard - no filter will be executed",
      "indices": [".kibana:dashboard"],
      "filters_bypass": ["*"],
      "filters_execute": []
    }

All right so :
My config :
# Enable or disable the complete Searchguard plugin functionality
searchguard.enabled: false

# Path where to write/read the searchguard master key file
searchguard.key_path: /etc/searchguard/

# When using DLS or FLS and a get or mget is performed then rewrite it as search request
searchguard.rewrite_get_as_search: true

searchguard.authentication.authentication_backend.impl: com.floragunn.searchguard.authentication.backend.simple.SettingsBasedAuthenticationBackend

searchguard.authentication.authorizer.impl: com.floragunn.searchguard.authorization.simple.SettingsBasedAuthorizator

searchguard.authentication.http_authenticator.impl: com.floragunn.searchguard.authentication.http.basic.HTTPBasicAuthenticator

searchguard.authentication.settingsdb.user.julienkibana: 91lpdsketf13
searchguard.authentication.settingsdb.user.user1: user1mdp
searchguard.authentication.settingsdb.user.user2: user2mdp

searchguard.authentication.authorization.settingsdb.roles.julienkibana: ["admin"]

# Configure the actionrequestfilter to allow or forbid action
searchguard.actionrequestfilter.names: ["readonly","kibanaaccess"]
searchguard.actionrequestfilter.readonly.allowed_actions: ["indices:data/read/*", "*monitor*"]
searchguard.actionrequestfilter.readonly.forbidden_actions: ["cluster:admin*","indices:admin*","indices:data/write*"]

My acl rules :
{"acl": [
{
      "__Comment__": "Default is to execute all filters",
      "filters_bypass": [],
      "filters_execute": ["*"]
    },

    {
    "__Comment__": "For role *admin* all filters are bypassed (so none will be executed). This means unrestricted access.",
    "roles": ["admin"],
    "filters_bypass": ["*"],
    "filters_execute": []
    },

    {
      "__Comment__": "Any authenticated user can do anything on the .kibana index - no filter will be executed",
      "indices": [".kibana:config",".kibana:search",".kibana:visualization",".kibana:dashboard"],
"filters_bypass": ["*"],
      "filters_execute": []
    }
]}

My error :
[2015-06-25 16:56:27,103][INFO ][com.floragunn.searchguard.rest.DefaultRestFilter] Authenticated user is User [name=user1, roles=[]]
[2015-06-25 16:56:27,115][WARN ][com.floragunn.searchguard.filter.SearchGuardActionFilter] Cannot determine types for indices:admin/exists (class org.elasticsearch.action.admin.indices.exists.indices.IndicesExistsRequest) due to type[s]() method not found
[2015-06-25 16:56:27,119][WARN ][com.floragunn.searchguard.filter.SearchGuardActionFilter] actionrequestfilter.readonly Action 'indices:admin/exists' is forbidden due to indices:admin*
[2015-06-25 16:56:27,128][ERROR][com.floragunn.searchguard.filter.SearchGuardActionFilter] Error while apply() due to java.lang.RuntimeException: Action 'indices:admin/exists' is forbidden due to indices:admin* for action indices:admin/exists
java.lang.RuntimeException: Action 'indices:admin/exists' is forbidden due to indices:admin*

seems to be duplicate of https://github.com/floragunncom/search-guard/issues/19

Auto close this issue because it refers to Search Guard 1 which is decommissioned now. For details pls. see https://github.com/floragunncom/search-guard-attic/wiki . If this issue is still valid for Search Guard 2 (for elasticsearch 2.x) feel free to reopen it (or to file a new one). Pls. do not longer report issues for Search Guard 1 (for elasticsearch 1.x). We sincerely apologize for the inconvenience this may have caused. Thanks for contributing!


Links in nav area not working correctly.

When clicking on the "Search Engine Optimization" link in the nav bar, you are not directed to the correct article below.

Updated in 41396e57bcc1896de99a740d0c3bd0a9e1ef8979
the article did not have the correct id element. added ID element to Search Engine Optimization section of the page.


Fix IE Conditional Comment processing

See #565.  This fixes a regression introduced after dev diverged from the master branch.

The AppVeyor build appears to have failed because it timed out; not because any tests failed.

:+1:


fix: improved linting

Cleaned up all python code to meet linting standards.
Signed-off-by: Chris Butler<EMAIL_ADDRESS>

This still fails some of the minting checks, @butler54 do you want to proceed anyway to start the cleanup process?


Exception raised when computing the derivative of singular matrix

Consider the following code
import autograd.numpy as np
from autograd import grad

x = np.zeros(shape=(4,4), dtype=np.float)
d = grad(np.linalg.det)

print(d(x))

the derivative of np.linalg.det(x) against x is obviously a zero matrix, but LinAlgError: Singular matrix Exception was raised

The formula which Autograd uses to compute the derivative of det is here. This is a fast method for computing the derivative but clearly fails whenever x is not invertible.
We could check the value of ans = det(x) and use a more sophisticated (probably slower) implementation in the special case det(x) == 0. Do you have any suggestions to fix this? We welcome pull requests.

This method for computing the cofactor matrix might be helpful https://stackoverflow.com/a/58123914.

The formula which Autograd uses to compute the derivative of det is here. This is a fast method for computing the derivative but clearly fails whenever x is not invertible.
We could check the value of ans = det(x) and use a more sophisticated (probably slower) implementation in the special case det(x) == 0. Do you have any suggestions to fix this? We welcome pull requests.

Thank you for pointing out the source of the problem and providing several possible solutions to fix it.
Unfortunately, currently I do not have rich time to look through this problem in detail, maybe later we can work on this issue together.


Test: Bump test apicast to 0.24

Signed-off-by: Eloy Coto<EMAIL_ADDRESS>

cpanfile.snapshot should be updated too to lock the version.

My love&hate relation with perl/cpan/releases. It's fixed now, sorry!


Add documentation for sync-data-transfer module

What this PR changes/adds:
This PR adds documentation for the sync-data-transfer modules introduces in #711.
It also embeds some fixes following end-to-end testing of the Data Plane and of the sync-data-transfer module.
Why it does that
No documentation was added before.
Linked Issue(s)
Fixes #585, fixes #463

Great! There's a typo in the diagram, the arrow that connects the data plane to the validation api on the provider has the number 3 but I think it should be 7 or so

Good catch, fixed

thx for the contribution, really like to get more docs!
two observations:
The PR contains more than just docs, would be great if you could adjust the description accordingly.
Are the linked issues solved completely with this PR?
 Moritz Keppler<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/cImpressum

thx for the contribution, really like to get more docs! two observations: The PR contains more than just docs, would be great if you could adjust the description accordingly. Are the linked issues solved completely with this PR? Moritz Keppler<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/cImpressum

Code modifications are fix following the end-to-end testing done with Andrea. I added a comment in the body of the PR to indicate that.
This PR contributes to both of these issues but does not close them.

The docs provide valuable insides into the 'hows'! Would be great to also learn more about the 'whys' and potential changes in the future (maybe as comments in the code or as part of the Readme). Especially the reasoning behind the architectural decisions (e.g. why is the contractId included in the token).

Hi @MoritzKeppler
I tried to improve a bit. Could you please check again?


Allow enabling / disabling AntiAliasing when loading a Texture

There should be an option to load a texture with / without AA in the TextureManager

Done in https://github.com/Sygmei/ObEngine/commit/4eb35e1abc08edea03ce2b16fad92d8c1ffc4fdb thanks to @BloubX


List of images used for mosaic

Any chance if it is somehow possible to retrieve a list of the actual images used to build the mosaic? I'd like to be able to somehow move my mouse over the mosaic image and be able to click each and every image in the mosaic and then visit for instance a page with some background info on that very image.
I already looked around in the sourcecode for a sec but didn't find a solution right away.

PyMos offers to spit out image files.
There is quite a good chance that it's possible to retrieve a list of actual images used to build the mosaic. One could hook a logger, based on an argument, it could also provide you a JSON describing the MxP images and their URLs.
You could then use this JSON to drive a UI - maybe you could do it in WebGL for performance. You could change camera angle with mouse movements. Do zoom in/out based on movements when mouse is down. Keep highlighting a target image with a border perhaps. On click - it will fetch the URL, render an enlarged version of the mosaic element image.


Add OpenSearch index write operator and builder

Description
TODO: will rebase after https://github.com/opensearch-project/sql/pull/1091 merged.
Issues Resolved
https://github.com/opensearch-project/sql/issues/1093
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Need to republish the PR against main branch.


Release OS Login libraries

Changes in Google.Cloud.OsLogin.V1 version 2.3.0:

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Changes in Google.Cloud.OsLogin.V1Beta version 2.0.0-beta05:

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Packages in this release:

Release Google.Cloud.OsLogin.Common version 2.3.0
Release Google.Cloud.OsLogin.V1 version 2.3.0
Release Google.Cloud.OsLogin.V1Beta version 2.0.0-beta05

Created release for Google.Cloud.OsLogin.Common-2.3.0
Created release for Google.Cloud.OsLogin.V1-2.3.0
Created release for Google.Cloud.OsLogin.V1Beta-2.0.0-beta05


Metadata search: hlsl item number false positives

HLSL item number search retrieves false positives

cc @frankwiles
This is another issue part of the group "Solr backend specific behavior that is also present in the original site". Adding the proper label and flagging Frank to decide next steps.
Reference URLs:

https://nuremberg.law.harvard.edu/search/?q=hlsl%3A7607
https://nuremberg.law.harvard.edu/search/?q=hlsl%3A7607

Would not one possible approach here be to add some business logic to the backend which would filter out all the documents returned by Solr which do not contain an HLSL item no. matching that of the query.  There should only ever be a single document per requested HLSL item no. in the search query.

I'm flying a little blind here, since I don't know how we're interfacing with Solr.  I think I was assuming that the response object for a document from Solr would somehow be structured with key-value pairs for the object, with each populated key being a metadata field displayed in the metadata panel of the document page on the website, or something similar.  If so, then I thought one could simply test for the presence of the "hlsl" key in each returned document object, and then ignore all the documents without an hlsl key.  My assumption is that there would only ever be a single document object returned for a given hlsl query, since the hlsl no. is a unique identifier in our project.  What am I missing here?

So in general search engines don't work like a database is the main thing to keep in mind.  They're looking for fuzzy matches of things and then ranking them by "closest" matches.
The rough mental model is:

take the query from the User and POST it as JSON to a Solr API endpoint
Solr endpoint returns a JSON object of results of things that matched with the data we push into Solr associated with it

So you are sort of right, in that we can look for an "HLSL" key, in either Solr or the DB, to get a single exact match like that but have to bail out and just let Solr do it's thing if they were instead searching for "hlsl-88 camp" or anything not JUST an single value like that.  Or at least for some small subset of values we hard code to have meaning.
Does that make more sense?

But isn't the original issue documented here only if the user is explicitly entering the syntax "hlsl:5607" (or whatever number they want).  The hlsl item no. is not a keyword search (as in your example "hlsl-88 camp"), but a fielded search.  And the search result in that case would only ever be a single result at best regardless of what else is in the search request, since only one document can ever have that hlsl item no.  It's possible that no results returned by Solr would have that hlsl item no., either because there are no documents in our current collection with that ID or because the other search conditions (e.g., "hlsl:5607 hitler monkeys") could not be met within a document satisfying that hlsl item no.
For example, the search "hlsl:451103 ghetto" -- a hybrid search containing a fielded query on the key "hlsl" and a keyword qurey on string "ghetto" -- produces a single result because we have that hlsl item no. in our collection, and its full text contains the string "ghetto."  But "hlsl: 451103 monkeys" would not return a result because the string "monkeys" is not contained in the text.  Wouldn't filtering on the hlsl key in the Solr JSON response get us what we need?

ok I dug back into this one and figured out the issue.
The way the system is written hlsl is the Django database ORM object primary key value.  7607 in this specific case.  The results here are:

Document PK 7607
Transcript Page PK 7607

So it's doing what we asked, but not ideal.  @pdeschner should we make the keyword hlsl be Document specific and ignore the PKs on other types of database entries?

Yes, hlsl:\d+ entered as search should always only return the material type "document"

Fixed with a real hlsl search field change.


[AutoMerge][Main] Merging staged change(s) to main

staging @ aa762b1...
Moving package from auto-merge/js/api to pkgs/api/
Moving package from auto-merge/js/packages/opentelemetry-semantic-conventions/ to pkgs/semantic-conventions/
Moving package from auto-merge/js/packages/opentelemetry-core/ to pkgs/core/
Merge Staged changes to main
Auto resolving 3 conflicts to select the master repo version
Summary of changes by file state
DD (Deleted <=> Deleted): 1
AA (Added <=> Added): 1
UA (Updated <=> Added): 1

(DD) auto-merge/js/api/karma.conf.js - Removed from both
(AA) pkgs/api/karma.conf.js - Added in both => checkout theirs
(UA) pkgs/core/karma.conf.js - Unmerged, added by them => checkout theirs

Do NOT merge this PR -- it is an example PR only, the Auto-Merge action will generate this PR once the sript changes in #48 are merged.


Excerpt: Footnote definition <...> is unreferenced - ignoring

Repro

jekyll-footnotes.zip
bundle install
bundle exec jekyll build

Expected:
Configuration file: P:/projects/bugs/jekyll-footnotes/_config.yml
            Source: P:/projects/bugs/jekyll-footnotes
       Destination: P:/projects/bugs/jekyll-footnotes/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
       Jekyll Feed: Generating feed for posts
                    done in 0.04 seconds.

Actual:
Configuration file: P:/projects/bugs/jekyll-footnotes/_config.yml
            Source: P:/projects/bugs/jekyll-footnotes
       Destination: P:/projects/bugs/jekyll-footnotes/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
       Jekyll Feed: Generating feed for posts
  Kramdown warning: Footnote definition for '1' on line 4 is unreferenced - ignoring
                    done in 0.04 seconds.

Source code of minimal repro
Gemfile
source "https://rubygems.org"

gem "jekyll", "~> 3.8.5"
group :jekyll_plugins do
  gem "jekyll-feed", "~> 0.11.0"
end

_config.yml
excerpt_separator: "<!--more-->"
markdown: kramdown
kramdown:
  show_warnings: true

_posts\2019-01-01-post.md
abstract
<!--more-->
article [^1]

[^1]: footnote

Details
The problem seems to be that the abstract doesn't contain a reference to the footnote, which is a reasonable occurrence. The excerpt_separator makes no difference, I'm just using an explicit one on my site and it shows where it is better for this repro.
What makes the warning go away:

removing jekyll-feed from Gemfile
changing to abstract [^1]
downgrading to Jekyll 3.6.3, here's the diff between 3.6.3 and 3.7.4:Using jekyll-watch 2.1.2 (was 1.5.1)
Using rouge 3.3.0 (was 2.2.1)
Using jekyll 3.7.4 (was 3.6.3)

This only started showing up when I updated the version, very likely https://github.com/jekyll/jekyll/pull/6522 brought the problem to the surface. That PR is a really good change since it helps fixing issues in my md files, but having an issue I can't fix makes the build noisy and opens up the possibility of not seeing other warnings.

Please report to https://github.com/gettalong/kramdown or go through existing issues there for assistance.

@ashmaroli It is jekyll-feed or jekyll, because there's nothing wrong with kramdown, it is reporting an issue that's valid; the problem is how jekyll(-feed) uses kramdown. Notice how the problem is around excerpt, that's a Jekyll concept, not a Markdown one... unless the configuration is lying to me and
excerpt_separator: "<!--more-->"
markdown: kramdown
kramdown:
   ...

actually means:
markdown: kramdown
kramdown:
   ...
   excerpt_separator: "<!--more-->"

jekyll-feed doesn't use Kramdown.
There's no issue in jekyll-feed unless the generated feed shows text different from your generated excerpt / post.
So, the issue is probably in Jekyll.
Have you tried building with jekyll-3.8.5?

Confirmed that excerpt_separator is clearly used by Jekyll:
https://github.com/jekyll/jekyll/blob/ae0a064f3fb9768660b5325e105701faef4e4a10/lib/jekyll/excerpt.rb#L137-L138
Yes, Gemfile about says gem "jekyll", "~> 3.8.5"
I opened it here, because removing jekyll-feed makes the warning disappear. So without feed, there's no issue.
If you still think it's not feed, can you please reopen and use transfer issue (to jekyll/jekyll) on the right?

I did not transfer this issue to jekyll/jekyll because I could not reproduce this issue with the master branch at jekyll/jekyll. So, the "bug" has already been fixed there.

How can I consume the latest master? I managed to do a repro without feed on 3.8.5.

The trick is that you need to use post.excerpt, otherwise it doesn't show (excerpt seems to be lazily generated). Feed was doing it, so if you remove it, no-one will call .exceprt.

# Gemfile

gem "jekyll", git: "https://github.com/jekyll/jekyll.git", branch: "master"

Can't use HEAD, because they just dropped my Ruby version, will take some time to reinstall everything.
I went for HEAD^ (i.e. 5b86a636b5ae3db5d8e2ac2138159ef56e976355), that repos it with the above index.html (which uses post.excerpt).
Will you transfer it, or should I open a new one?


The message clips if intro is added to appbar leading or action buttons

Hi, thank you for the great package. I was trying it out to review it on my Youtube channel, however, when I added the intro keys to Appbar buttons (leading and drawer) the messages and buttons were clipped. Here I have attached the screenshots.

Hello, I have received your question, I will solve it as soon as possible.

@elowes I have not yet checked the code, but let me know if I can do anything to help.

My time zone is now midnight, and there is no computer around me. I have to wait to see the code this morning.

@elowes No problem, do it whenever suits you, thanks for the useful package.

@lohanidamodar https://github.com/tal-tech/flutter_intro#qa I added the setStepConfig and setStepsConfig methods in version 0.4.0 to solve this problem.


Bcachefs patches no longer apply on linuxPackages_latest (6.1.6+)

Steps To Reproduce
Steps to reproduce the behavior:

add bcachefs to boot.supportedFilesystems

Build log
building '/nix/store/25h43lfhj7rl4b03m070jkzf9m4189vy-linux-config-6.1.7-bcachefs-unstable-2022-12-29.drv'...
unpacking sources
unpacking source archive /nix/store/jwgfb0mwqdvqpvhy9p8l5csvp4p1kbz0-linux-6.1.7.tar.xz
source root is linux-6.1.7
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file linux-6.1.7/virt/lib/irqbypass.c
patching sources
applying patch /nix/store/hxqx22d2acr51drgh1gb4i3kfk256lkc-bcachefs-8f064a4cb5c7cce289b83d7a459e6d8620188b37.diff
<snip>
patching file kernel/trace/trace.c
Hunk #1 succeeded at 1680 (offset 1 line).
Hunk #2 succeeded at 3753 (offset 1 line).
Hunk #3 FAILED at 6788.
Hunk #4 succeeded at 6829 (offset 14 lines).
Hunk #5 succeeded at 6855 (offset 14 lines).
Hunk #6 succeeded at 6873 (offset 14 lines).
Hunk #7 succeeded at 9916 (offset 16 lines).
1 out of 7 hunks FAILED -- saving rejects to file kernel/trace/trace.c.rej

Notify maintainers

@davidak @Madouura @Mic92
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.7, NixOS, 23.05 (Stoat), 23.05pre445203.5ed48194335`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.12.0`
 - channels(root): `"nixos"`
 - channels(user): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I reverted two linux kernel bumps in my nixpkgs to fix this: https://github.com/Mic92/nixpkgs/commits/main

Likely needs an update. Current bcachefs is on 6.1.

Looks like it's a bit more complicated than that.
If it's anything like last time with printBuf, it'll be fixed upstream at some point.
Until then, IIRC there's not much I can do without completely changing the derivation again.

I am now actually back at no longer applying kernel patches but just downloading the original kernel source. I proposed this in the past, but the other maintainer did not like it: https://github.com/Mic92/nixpkgs/commit/51887ee48cf5720276af99d65dac2e862e6b4a1e
As I no longer use the way it's handled in nixpkgs, please don't ping me in the future on this particular package.

I am now actually back at no longer applying kernel patches but just downloading the original kernel source.

I proposed this in the past, but the other maintainer did not like it

i objected the change to patch approach previously (https://github.com/NixOS/nixpkgs/issues/121767)

this would break it in unexpected ways

what now happened

Until then, IIRC there's not much I can do without completely changing the derivation again.

that's a bad experience for the users
when just packaging upstream code, you can just revert the latest changes, so it always works for users
maybe move back to that approach? i don't think it has bigger downsides than this situation
cc @hyperfekt

just ran into this while trying bcachefs on a new machine. does this happen often? looking at the derivation, i don't see how it wouldn't.

It doesn't happen very often but clearly often enough to pose a real problem. The correct solution here would've been to switch this package to use the latest working minor instead of the main package when that was updated instead of just letting it break (which is not very difficult as it is a package argument passed in linux-kernels.nix).
If the package is changed to use an upstream tarball I still argue that it should not be pulled in automatically by adding bcachefs to boot.supportedFilesysyems because NixOS does not have a policy of shipping its users known-vulnerable software in an obscured manner.

I am now maintaining a functional package for bcachefs in disko, where I need to maintain it for testing anyway. Here is a usage example: https://github.com/Mic92/dotfiles/blob/a98d31df9a408d191bcdcd2b9260b9b9e232ed29/nixos/flake-module.nix#L165

While there isn't a conclusion on how to handle bcachefs, can it be marked broken?

I think it might cause troubles with the nixos module and I would like to keep the nixos module

I think it might cause troubles with the nixos module and I would like to keep the nixos module

There is no way to workaround this? Maybe override package to not broken?
I ask because bcachefs is a large build with dependencies, lists on many builds I do in nixpkgs-review. It is always popping up. Just happened now again:

It used to work that way and it's pretty easy to do but the bcachefs kernel isn't maintained the same way zen and lqx are, it doesn't get any patch updates after the .0 minor release, so any non-bcachefs issues are not patched until the next minor version.

Let's pin it, and every time I bump the "patch" in zen, I bump it here too. If it does not build, then I leave it stuck until someone has patches or the "minor" bumps again.

It used to work that way and it's pretty easy to do but the bcachefs kernel isn't maintained the same way zen and lqx are, it doesn't get any patch updates after the .0 minor release, so any non-bcachefs issues are not patched until the next minor version.

I see your point, but for me the point of this kernel package is to test bcachefs. If you need something unrelated to work, use the regular kernel!

I don't need anything to work. My insistence isn't for my sake, as I currently don't even use the bcachefs kernel. I feel that we're talking in circles... I just don't want people to get tripped up by this.

@hyperfekt i mean "you" as in "the person who uses it".
We see we can't have both: working bcachefs and it using the latest upstream linux version. So we have to prioritize one.
And i argue to focus on a working bcachefs package even when it does not get the latest linux patches. That should be fine, since it's marked as testing.

Oh, I think I got it. We could use Kent's tree and put it behind the insecure flag, just like Python 2.

This is now fixed by https://github.com/NixOS/nixpkgs/pull/214265. You can see here if it already is in the unstable channel: https://nixpk.gs/pr-tracker.html?pr=214265
@hyperfekt marking an insecure package as such sounds reasonable. Not sure if kernel packages are somehow special here. Do you want to create a PR? (or someone else?)


Cron

I always get a 403 when I try to run curl for my crontab ie
/usr/bin/curl -G https://www.polynamaude.com/asset/cron/24h/Birthday/?secret_word=blowitup

Hi @polynamaude!
Are you sure it is setup to be ran every 24 hours (and not less). If it runs more than once every 24 hours, it will give 403 HTTP code
https://github.com/pH7Software/pH7-Social-Dating-CMS/blob/4047bb4aaed7f22df906ad03da2e9c58dbc5830a/_protected/app/includes/classes/Cron.php#L31
Hopefu

Should be a GET request; Something like below:
crontab -e
0 0 * * *  GET "https://www.polynamaude.com/asset/cron/24h/Birthday/?secret_word=blowitup"

Hi !
Thanks for the reminded that I was on my way to a solution. Found out by using Curl that my .htaccess user-agent filter was rejecting the regular "curl". So I simply added a --user-agent option to my command and "voilà" like the french guy say.
By the way the vinny user got onto the French translation thread with complains about license, donation and alot of hope. I kindly and a bit hardly reminded him that he used the word donation and it means to "give with getting something in exchange". I may have been a bit harsh by shaking him and reminding him that he should ask the questions on the appropriate thread and not many many time, expecting answer in 5 minutes turn around and added a small note with the price a line of code is worth when you have thousand of them, plus what does the "opensource" model means. I don't think you're Microsoft or that you sell thousands of license a week so all should be indulgent.
Thanks alot Mister PH

Really happy you figured out!
Thanks for this @polynamaude!
It's the first donation I got since I added the donation module and apparently it wasn't clear for him. Anyway, I replied to his email he sent to me. I really appreciate your help! :)

How long ago did you add the donation module ?
Little question, do you have any UML or something alike so I can figure out how the output override works for modules and templates ? I'm doing a new template and found out that most of the stuff seem to be residing in the modules to really customize the output.
If you've got a list of TODO for the software then maybe I can give hand. I have some free time a contract aren't coming in too fast those day.
Thanks


zsh: ensure we have enough commands to store in the cache

Otherwise, the cache would be invalid and won't be refreshed soon. This
can happen when the user has the completion installed before docker is
installed.
Signed-off-by: Vincent Bernat<EMAIL_ADDRESS>cc @sdurrheimer

LGTM

LGTM


Improve return messages

Incorporate the stream topic onto the results in the follows command.
Omit "or more of your followed channels are currently streaming" message if the total online is less than the number of online streams tested.

Resolved in 502ad32301dbda7c6192960cc034a9fa71d81c68 and 571c5563edba297fddff5cfbdccc96916154b221.


parse_requirements fails with environment markers

Originally reported by: dims (Bitbucket: dims, GitHub: dims)

Just a simple call like the following fails:
#!bash

pkg_resources.parse_requirements('Twis-Ted>=1.2-1;python_version==\'2.7\'')

Found this when trying to run tox -e py27 on https://github.com/openstack/oslo.privsep in a mac environment with latest setuptools.
#!bash

tox -e py27
GLOB sdist-make: /Users/dims/openstack/oslo/oslo.privsep/setup.py
ERROR: invocation failed (exit code 1), logfile: /Users/dims/openstack/oslo/oslo.privsep/.tox/log/tox-0.log
ERROR: actionid: tox
msg: packaging
cmdargs: ['/usr/local/opt/python/bin/python2.7', local('/Users/dims/openstack/oslo/oslo.privsep/setup.py'), 'sdist', '--formats=zip', '--dist-dir', local('/Users/dims/openstack/oslo/oslo.privsep/.tox/dist')]
env: None

error in setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected version spec in enum34;python_version=='2.7' or python_version=='2.6' at ;python_version=='2.7' or python_version=='2.6'

ERROR: FAIL could not package project - v = InvocationError('/usr/local/opt/python/bin/python2.7 /Users/dims/openstack/oslo/oslo.privsep/setup.py sdist --formats=zip --dist-dir /Users/dims/openstack/oslo/oslo.privsep/.tox/dist (see /Users/dims/openstack/oslo/oslo.privsep/.tox/log/tox-0.log)', 1)

If i edit the line, i can get past the error:
https://github.com/openstack/oslo.privsep/blob/master/requirements.txt#L10
However since OpenStack uses this syntax a lot, it would be great to add support for this syntax:
https://github.com/openstack/requirements/blob/master/global-requirements.txt
Here are some tracebacks to pin point the problem (line numbers may vary!)
#!bash

Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/setuptools/dist.py", line 125, in check_requirements
    list(pkg_resources.parse_requirements(value))
  File "/usr/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2930, in parse_requirements
    "version spec")
  File "/usr/local/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2895, in scan_list
    raise RequirementParseError(msg, line, "at", line[p:])

Bitbucket: https://bitbucket.org/pypa/setuptools/issue/470

Can't setuptools support requirement specifiers (environment markers) in install_requires? There should be no need for extras_require when you specify them in install_requires.
pip works, setuptools breaks
I have a requirements.txt file in every project, which is also read by the project's setup.py. Hence, my users / developers can choose to run python setup.py install (develop) or run pip install -r requirements.txt. Which ever comes more natural to them.
Unfortunately, using environment markers is supported by pip but not by setuptools:
Using pip (with Python 3):
$ pip install -r requirements.txt
Ignoring dnspython: markers "python_version < '3'" don't match your environment
Installing collected packages: dnspython3
Successfully installed dnspython3-1.12.0

Using setuptools (with Python 3):
$ python setup.py develop
error in GuuruBackoffice setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected version spec in dnspython3 ;python_version >= '3' at  ;python_version >= '3'

Related code samples
My requirements.txt file:
dnspython3 ;python_version >= '3'
dnspython ;python_version < '3'

My setup.py script:
def read_file(filename):
    with open(join(abspath(dirname(__file__)), filename)) as f:
        return f.read()

setup(
    # ...
    install_requires=read('requirements.txt'),
    # ...
)

Can't we implement the same (or a matching) requirements parsing as pip for markers and deprecate extras_require?
This way a just single feature/keyword does it all for requirements. And we do away with some confusion, because pip's and setuptools' behavior match more closely.

Reading the history, in Setuptools 20.5 and more stable in 20.6.8, Setuptools should support environment markers in install_requires and others. I do suspect that this feature can't be widely adopted until older versions of Setuptools have worked their way through retirement. @bittner, can you confirm that the later releases work as you would expect?

Deprecate extras_require?

Perhaps, but I struggle to think of how this is done without first upgrading the world to Setuptools 20.6.8 or later.

@jaraco You're right, using the marker's in requirements.txt works for, e.g., version 21.0.0, which I have installed, that's awesome!
I'd say you have to add deprecation code to setuptools one days, or you'll never be able to remove a (confusing) feature.