Remove the hackney dependency

The reasoning behind this change is that hackney is not really needed for doing basic HEAD and GET requests.

Hackney is needed for making sure we are talking to IANAs servers and not someone pretending to be IANA (with HTTPS).
It would be nice if Erlang could do that, but right now it cannot.

@lau I noticed that hackney itself uses ssl_verify_hostname as a dependency. Would you consider depending on that directly if it doesn't result in a lot of code while sticking with httpc for HTTP requests?

@alco In general a way that is secure and allow for less dependencies is welcome. But it has to be secure. I am not sure if you can combine httpc and ssl_verify_hostname like that. If such a solution is made, security experts would have to validate that it is just as secure before I would want to merge it. And someone would have to maintain it.
The basic requirement is that we have to be sure that the files we are getting are from IANA and that no one has tampered with them.
@ericmj Do you think there is a better way of getting files via HTTPS than hackney?

If you use ssl_verify_hostname and httpc you still need to put it all together and use your own certificate store. You could also use httpc and pin IANA's certificates but it has its own complications like updating tzdata when IANA updates their certificates.
Even if it's not ideal I think using hackney is the easiest and most versatile solution at least.

@ericmj Thanks for your response.
It seems hackney is still the best solution for now for tzdata.
@alco Thanks for your PR. I wish httpc had proper SSL support so we could use that. But for now we can not.
Closing this.

No problem.
@lau I'm curious as to why you are so confident in hackney, especially after your comment about security. Is there a publicly available security audit of its implementation of SSL validation?

@alco Don't think it has anything to do with confidence. It's simply that hackney does certificate validation and httpc does not.

@alco It is not that I am not particularly confident in hackney. OpenSSL has bugs too. But what is the alternative?


Use transit-gateway by default instead of cloud connection

Remove cloud connection and use transit gateway since IBM cloud removed the support of cloud connection.

/cc @aman4433

/assign


Plot and dump evaluation results

Plot in a subplot above the current training cost. Asynchronously load, modify and write result file after every evaluation.


update reverse vcard data for set 12 & 13

As discussed on Slack, the two Aurra Sing With Blaster Rifles (AI and non-AI) should be swapped for underlying card.


iOS: Use objc2

objc2 is a refinement of objc with the following benefits to webbrowser:

Less error prone msg_send! macro invocations (I've refactored the rest into a separate function as recommended by the docs).
Prevents a leak of the options dictionary.
Catches errors when passing an invalid URL to NSURL.
Makes it easier to do something in the completion handler in the future.

Builds upon https://github.com/amodm/webbrowser-rs/pull/86 to avoid a merge conflict.

Thanks @madsmtm. I remember the thread between you and the objc maintainer. While I do want to walk away from a 4+ yr stale crate, the following considerations come up:

Like most maintainers, I try to be mindful of supply chain attacks, and the recent xz issue has only highlighted the need for it. In the absence of being able to audit a codebase as large as yours, I tend to rely on the proxy of relative size of deployment base because that maximises the chances of a vulnerability being caught.
There's no Security policy defined yet for objc2. Irrespective of the outcome of this PR, I'll encourage you to do that. You can look at this crate's SECURITY.md as an example.
MSRV - this is a ✅

The first issue is the main blocker here, and while I do want to be supportive of migration away from objc (given the maintainer's stance on it), I'd like to make sure that I feel comfortable with security for my downstream users first. I don't have any suggestions for you on this, just sharing what's top of mind for me here.
I'm leaving this PR open for now, to allow myself more time to think through this, and for alternative perspectives to be shared.

Thanks for the detailed reply!

Security policy

Thanks for the heads-up, I've added one in https://github.com/madsmtm/objc2/commit/469a36f6a361f7bdc5a25ce2d57490f4dc6b60a9 and enabled the ability to report vulnerabilities using GitHub's advisories.

MSRV

I'll note that it isn't defined by policy yet, so I may still decide to bump it in a minor version. Would be interested in your input on it, preferably in https://github.com/madsmtm/objc2/issues/203.

In the absence of being able to audit a codebase as large as yours

One idea to slightly reduce the review surface would be to not use objc2-foundation, and instead only use objc2 (that's still by itself fairly large, but it can't really be split due to orphan rules and ease-of-use).
Although, objc2-foundation is automatically generated, so should also be fairly easy to review for unintended changes (assuming you also run the code generator yourself, and verify that it matches), and both crates are still under my control, so in the end probably doesn't really matter that much.

supply chain attacks
relative size of deployment base

Totally understandable worry (especially given that I'm opening the PR myself, I can definitely see how this could be negatively interpreted).
In general, I'm looking to replace objc with objc2 in major projects in the ecosystem to increase soundness, stability and performance for macOS/iOS users of Rust. But part of the reason why I opened this PR is also to gain exactly the "relative size of deployment base" that you're talking about; the more downloads/usage objc2 has compared to objc, the easier it is for me to convince other projects to use objc2, and the safer the entire ecosystem gets. And your crate is quite a popular one ;)
That said, I'm totally fine with it (and really quite respect) if you want to hold out on this for a while!
If you think of other ways I can help prove the sincerity and security of the project, please don't hesitate to tell me!

Thanks @madsmtm, for deciding to have a security policy, as well as your inputs on the surface area to assess. I don't have a well defined time frame in mind currently, but I do plan to take a first look at it this weekend.

This is good to go after the tests. Thanks @madsmtm, and hope that you find wider adoption quickly enough.
To document my thought process for later reference:

objc2 is clearly more actively maintained. As an example, in the current case, I was able to build for visionOS using objc2, but not able to do it using objc.
Concerns about security stemming from a lesser deployment base is significantly mitigated in current case because we're using it for ios/tvos/visionos, each of which is very strongly sandboxed. That leaves only build.rs as an attack vector, which is a lot more easily monitored.

Proceeding with merging this, despite the ios test failure. I'm currently of the opinion that the test failure has something to do with some recent changes to the macos-latest image, or how ios simulator is now run on macos-latest.
I'll figure that out separately from this PR. Release will happen only after this has been figured out.

Thanks for caring so much about the security of your project, it's been really nice to see and to discuss with you!

Proceeding with merging this, despite the ios test failure. I'm currently of the opinion that the test failure has something to do with some recent changes to the macos-latest image, or how ios simulator is now run on macos-latest.

macos-latest recently changed from being run on x86_64 to ARM64, so that might be part of it?

This is released as v1.0.1


IPM fix for esp32s3 targets

This PR fixes how the APPCPU is being built in IPM samples. C array encapsulation is removed and instead normal build procedure is used. Fixing the sysbuild builds as well.
Fixes https://github.com/zephyrproject-rtos/zephyr/issues/80890

@mmahadevan108 any chance to put this into the 4.0?
It is an important fix that will enable some samples for esp32 that are broken since HVM2 introduction.
Thank you :)

pinging @nordicjm for a review from HWV2 side as these samples were broken since the switch to HWV2.

@uLipe . This is a fairly big change. Is everything required for fixing the bug?
It might be easier to pull this into 4.0 if we can focus this PR to only include the changes needed to fix the bug and move the remaining to a separate PR that can be merged post 4.0 release.

This is a fairly big change.

@mmahadevan108 you are right, and the reason it is so hefty is it not only fixes the IPM build itself but also stuff related to HWMv2, eg using sysbuild for multi-image builds.

@uLipe @marekmatej thank you for clarifying. This PR needs approval from @kartben as he is the assignee before we can merge

Now that assignee has been updated, I guess we can proceed as is? @mmahadevan108

I'd say the same thig


This task failed the last time it ran: Failed - No files were found to be backed up. Review the data contained within this instance and any backup exclusions.

Bug Report

Symptoms

What are you trying to do? setup a timed backup and running it as  a test
What are you expecting to happen? Instances getting backup
What is actually happening? ('Nothing' is not an acceptable answer!)

This task failed the last time it ran:
Failed - No files were found to be backed up. Review the data contained within this instance and any backup exclusions.
Reproduction
Installed Amp setup a custom location
"LocalFileBackupPlugin.Storage.StorePath" "LocalFileBackupPlugin.Storage.StorePath"

Disk is there and is also set as Data storage.
Can also access the disk over samba.
then in Instances --> game server ---> Manage --> schedule ---> Add new trigger --> Simple Time internval --> set to ever 12 hours -->
Then trying to add backup i cant the command add to Backup isnt in the drop dox box. (image 1)

So i go back to main menu press Schedule do the same thing there, now it does show Take A Backup, but when i press it i get the error "This task failed the last time it ran: Failed - No files were found to be backed up. Review the data contained within this instance and any backup exclusions." (image 2)

No clue whats groing wrong here.

What's shown within the file manager?

Closed due to inactivity. Feature is functioning as intended.


You may need an appropriate loader to handle this file type.

Hi man,
great library, but I have a problem.
I installed the module in order to include it in my react app with the command: 'npm install input-moment'.
When I build my app I get the following error:
ERROR in ./~/input-moment/src/input-moment.js
Module parse failed: /Users/antonioromano/development/GeoUniq/web/console/GUConsole/node_modules/input-moment/src/input-moment.js Line 28: Unexpected token <
You may need an appropriate loader to handle this file type.
|
|     return (
|
|
|           <button type="button" className={cx('ion-calendar im-btn', {'is-active': tab === 0})} onClick={this.handleClickTab.bind(null, 0)}>
@ ./~/input-moment/index.js 1:17-46
It seems the module is parsed with the wrong parser, do you know anything about

Same problem here

I'm also running into the same issue.

plus one

I'm also running into the same issue.

Plus one

It would be interesting to rename those files to .jsx extensions. (more explicit)

Looks like webpack doesnt like the index.js that calls module.exports = require('./src/input-moment');

I got around this by running this module through babel via webpack - below is a snippet from my webpack config file. The exclude regex will include any .js files found in node_modules/input-moment.
webpackConfig.module.loaders = [{
  test: /\.(js|jsx)$/,
  exclude: /node_modules(?!\/input-moment)/,
  // include: /src/,
  loader: 'babel',
  ...

0.3.0 contains prebuilt lib files, please take a try.


Send a Test Email Button in Settings

Describe the feature you'd like
An admin function to test if outgoing email is setup correctly.
Describe the benefits this feature would bring to BookStack users
As a Bookstack administrator I have no way of knowing if outgoing emial is working until one of my users complain.  Even then, I have no easy way to reproduce the problem or check if my actions have solved the problem.

+1 to this

Thanks for creating this issue @tuaris, Have actually been meaning to add this functionality to the Maintenance section of the admin area. Will mark for the next feature release so I actually get this done.

@ssddanbrown have you already started development for this issue or can I grab it? :)

@timoschwarzer Nope, Not started on this, Got for it :+1:

Now implemented thanks to @timoschwarzer. Will be in the next feature release (v0.28).


svg-baker-runtime has wrong main file

  "name": "svg-baker-runtime",
  "version": "1.3.4",
  "description": "",
  "license": "MIT",
  "repository": "https://github.com/kisenka/svg-baker/packages/svg-baker-runtime",
  "main": "browser-sprite.js",
  "module": "src/browser-sprite.js",

It seems that it didnot bundle out any assets?

@vagusX yep, it was fixed in 1.3.5

cool 👍


Remove symlinks to /dev/{stdout,stderr}

Apparently this does not work anyway, and makes debugging harder as apache may still fall back to those files. (which means that the messages are lost)
Signed-off-by: Manuel Hutter<EMAIL_ADDRESS>

Why do you think this doesn't work? Try to remove all CustomLog and ErrorLog directives to see it working.
Why should content be lost when Apache falls back to those files as a default? The files are created as files when we don't add symlinks, which buries them in the file system (for diligent engineers to discover them when troubleshooting). Redirecting the logs to stdout and stderr seems more appropriate for 12-factor-apps.

Why do you think this doesn't work.

I discovered this during testing, BUT there might have been other factors. Let me test this again more thoroughly.

Sorry, you are correct :+1: I mixed up things


Check for public / anonymous read permission on wmts layers

This allow anonymous users to request WMTS layers that have granted permission to the public group.
@devs please review

closing in favor of https://github.com/terrestris/shogun-core/pull/390


Improvements to the plots showing confound correlation

[ ] Use the ordering of the right-hand plot (magnitude of correlation with GS) for the heatmap on the left.
[ ] Rotate the bar plot 90 deg. CW (so that confound names are more readable.
[ ] Remove GS from bar plot (it is always 1.0)
[ ] Represent negative correlations with blues colormap on the bar plot
[ ] Represent only the upper (or the lower) triangle of the heatmap on the left (potentially sharing the axis labels with the bar plot if it is rotated 90 deg. as proposed above).
[ ] Exclude nonsteady state regressor (see poldracklab/fmriprep#2283).

Feel free to add suggestions to this issue.
EDIT: Added nonsteady states item.
EDIT: Added the idea of sharing the axis labels (i.e., confound names) of heatmap and bar plot.

I would prefer to keep regressors grouped in the correlation matrix. I don't think the level of correlation with GS is likely to be a meaningful ordering, while being able to see which other regressors correlate with, e.g., the motion parameters, is very useful.
I might even go beyond the 20 regressor limit and aim for a consistent set, always in the same order. As a suggestion, here are 30:

Masks (5)

Global
WM
CSF
WM+CSF
tCompCor

Motion (7)

Translation x 3
Rotation x 3
FD

CompCor (12)

aCompCor x 6
tCompCor x 6

Misc (2)

DVARS
RMSD

Cosine (6)

As this is a report improvement, I think this can be backported into 20.2.x.


some questions about training

I am a new about this field, and have a question when i was training the network. I just have one GPU, so when i was training,i set the batch_size=12 ,in order to meet the condition about "batch_sizeiter_sizen_gpu=128", i set iter_size=11, so when i was training ,the log showed "Gradient clipping: scaling down gradients (L2 norm 68.3038 > 40) by scale factor 0.585619" . i want to know, if it will make a big influence about the result. thanks a lot and i am looking forward to your reply if you can give me some advice

It is fine as long as you can get a good final accuracy. But generally, I would recommend getting more GPUs as training with 1GPU will be very slow.

how did u change the iteration size

I  only used one GPU ,i set iters more than four times，the result is the same as the paper shown

I meant from which file? I could not find the file where you can change the iter_size.
Thank you for your help

about temporal-segment-networks-master/models/ucf101/tsn_bn_inception_flow_solver.prototxt , you can change  max_iter=....

@Mohamad73 It is in the solver file. The option is named iter_size


The feedback is empty if the student shuts down the process with os._exit(0)

If the student's submission calls os._exit(0), the final feedback in A+ is completely empty. I assume that the python-grader-utils process shuts down and is not able to produce any output in the end.
Is the remote grader prototype able to fix this problem? If the grader main process and the submission process are separated, then the student can not shut down the main process.
https://github.com/apluslms/python-grader-utils/tree/remotegrader

I got the remotegrader working with our recent updates to graderutils.
When using the remotegrader, os._exit(0) results in an EOFError when the rpyc connection is lost, but it does not shut down the main process.
os._exit() could probably be overridden in the student process so that it raises an exception stating that the command should not be used. Is this too hacky?

I don't think this is urgent. We can now leave this waiting for the remote grader. We will then get a better overview of this whole thing.
If you are already finished with the remote grader, then I can take a look at that in May.


Unable to startproject on scrapy

This bug-tracker is monitored by developers and other technical types.  We like detail!  So please use this form and tell us, concisely but precisely, what's up.  Please fill out ALL THE FIELDS!
If you have a feature request, please post to the UserVoice.
Important: When reporting BSODs or security issues, DO NOT attach memory dumps, logs, or traces to Github issues. Instead, send dumps/traces to<EMAIL_ADDRESS>referencing the GitHub bug number. Ideally, please configure your machine to capture minidumps, repro the issue, and send the minidump from "C:\Windows\minidump".

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [Version 10.0.15063]

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)

I'm new to installing packages in unix and python. I'm working on Bash on Ubuntu on Windows. I wanted to start learning Scrapy, which I installed using:

apt-get install scrapy

When I'm trying to create a project, I'm getting the following error:

root > scrapy startproject tutorial
Traceback (most recent call last):
File "/usr/local/bin/scrapy", line 11, in
sys.exit(execute())
File "/usr/local/lib/python2.7/dist-packages/scrapy/cmdline.py", line 128, in execute
cmds = _get_commands_dict(settings, inproject)
File "/usr/local/lib/python2.7/dist-packages/scrapy/cmdline.py", line 46, in _get_commands_dict
cmds = _get_commands_from_module('scrapy.commands', inproject)
File "/usr/local/lib/python2.7/dist-packages/scrapy/cmdline.py", line 29, in _get_commands_from_module
for cmd in _iter_command_classes(module):
File "/usr/local/lib/python2.7/dist-packages/scrapy/cmdline.py", line 19, in _iter_command_classes
for module in walk_modules(module_name):
File "/usr/local/lib/python2.7/dist-packages/scrapy/utils/misc.py", line 71, in walk_modules
submod = import_module(fullpath)
File "/usr/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "/usr/local/lib/python2.7/dist-packages/scrapy/commands/version.py", line 6, in
import OpenSSL
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/init.py", line 8, in
from OpenSSL import crypto, SSL
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/crypto.py", line 16, in
from OpenSSL._util import (
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/_util.py", line 6, in
from cryptography.hazmat.bindings.openssl.binding import Binding
File "/usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/openssl/binding.py", line 13, in
from cryptography.hazmat.bindings._openssl import ffi, lib
ImportError: /usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/_openssl.so: cannot enable executable stack as shared object requires: Invalid argument
root >

I verified that the above file is available. I'm trying to understand if there is any prerequisite(s) I missed to install (which I doubt, as I understand that they are handled during installation - please correct me if I'm wrong), or is there anything I need to enable.
Thanks, Sasi

What's wrong / what should be happening instead:
I'm supposed to be able to start with scrapy, which I'm unable to.

Strace of the failing command, if applicable:  (If <cmd> is failing, then run strace -o strace.txt -ff <cmd>, and post the strace.txt output here)

root > strace -o strace.txt -ff scrapy startproject tutorial
Traceback (most recent call last):
File "/usr/local/bin/scrapy", line 11, in
sys.exit(execute())
File "/usr/local/lib/python2.7/dist-packages/scrapy/cmdline.py", line 128, in execute
cmds = _get_commands_dict(settings, inproject)
File "/usr/local/lib/python2.7/dist-packages/scrapy/cmdline.py", line 46, in _get_commands_dict
cmds = _get_commands_from_module('scrapy.commands', inproject)
File "/usr/local/lib/python2.7/dist-packages/scrapy/cmdline.py", line 29, in _get_commands_from_module
for cmd in _iter_command_classes(module):
File "/usr/local/lib/python2.7/dist-packages/scrapy/cmdline.py", line 19, in _iter_command_classes
for module in walk_modules(module_name):
File "/usr/local/lib/python2.7/dist-packages/scrapy/utils/misc.py", line 71, in walk_modules
submod = import_module(fullpath)
File "/usr/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "/usr/local/lib/python2.7/dist-packages/scrapy/commands/version.py", line 6, in
import OpenSSL
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/init.py", line 8, in
from OpenSSL import crypto, SSL
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/crypto.py", line 16, in
from OpenSSL._util import (
File "/usr/local/lib/python2.7/dist-packages/OpenSSL/_util.py", line 6, in
from cryptography.hazmat.bindings.openssl.binding import Binding
File "/usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/openssl/binding.py", line 13, in
from cryptography.hazmat.bindings._openssl import ffi, lib
ImportError: /usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/_openssl.so: cannot enable executable stack as shared object requires: Invalid argument
root >

See our contributing instructions for assistance.

ImportError: /usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/_openssl.so: cannot enable executable stack as shared object requires: Invalid argument

I'm guessing everyone's crypto libraries must have levelled recently (for some value of recent). #2561, #2553, #916, #442, #286 et al.

ImportError: /usr/local/lib/python2.7/dist-packages/cryptography/hazmat/bindings/_openssl.so: cannot enable executable stack as shared object requires: Invalid argument

I'm guessing everyone's crypto libraries must have levelled recently (for some value of recent). #2562, #2553, #916, #442, #286 et al.

Hi, is there any solution available for this issue?


Add Tag Helper descriptors to the Tag Helper node

Looks like you're failing in the travis/appveyor builds because of compile errors. :shipit: once everything be good.


Ability to add two maps together

Related to https://github.com/myst-lang/myst/issues/57
Adds the ability to add two maps together

LGTM, thanks for adding this :)


Add PreTrainedTokenizer support

add PreTrainedTokenizer support to HF cvt

Any test cases?

Any test cases?

Yeah, looking for some right now, will update soon.

I read through the source code of PreTrainedTokenizerFast at https://github.com/huggingface/transformers/blob/b42010bb1d3cbf262d27e0a328661885be46dfdb/src/transformers/tokenization_utils_fast.py#L410. And it looks like it's only a thin wrapper of Its rust tokenizer, which seems not be feasible to get enough information to convert it into ONNX format.

What about the slow PreTrainedTokenizer? It is definitely true that most tokenizers built on top of that will likely have a common base tokenizer (like GPT2Tokenizer for Falcon) but adding it would make our code more extensive - implementation is almost done, if we can just fix the parsing of vocab/merges it will work.

the slower versions is an abstract class, and the pretty straightforward way to add more its derived class in the table, and drop this change.

I read through the source code of PreTrainedTokenizerFast at https://github.com/huggingface/transformers/blob/b42010bb1d3cbf262d27e0a328661885be46dfdb/src/transformers/tokenization_utils_fast.py#L410. And it looks like it's only a thin wrapper of Its rust tokenizer, which seems not be feasible to get enough information to convert it into ONNX format.

What about the slow PreTrainedTokenizer? It is definitely true that most tokenizers built on top of that will likely have a common base tokenizer (like GPT2Tokenizer for Falcon) but adding it would make our code more extensive - implementation is almost done, if we can just fix the parsing of vocab/merges it will work.

the slower versions is an abstract class, and the pretty straightforward way is to add more its derived class in the table, and drop this change.

sounds good!


add missing topic options for P000449

Thank you for the PR. I am cleaning up old pull requests. This seemed to have been already added in 63f458f1afa4afcee83ee2eae13579dcd582c136 . Closing as it already has been implemented.


[Suggestion] Separate Menu Templates

Hey there, I thought about some things and realized some people might not want all of the Menus Template, but it may be difficult to separate, so maybe separating the menus into their individual scenes like "Main Menu Template", "Settings Menu Template", "Pause Menu Template", and so on could be beneficial to those users, and make it more accessible and modular.

I split Options Menus into a branch test-options-menus  and added it to the Godot Asset Library as a release candidate version. If it works and fills the need intended, then I may split it into its own repo, as well.

https://godotengine.org/asset-library/asset/3058

Options Menus are available under their own repo and plugin now:
https://github.com/Maaack/Godot-Options-Menus


Is it possible to use a Per-* pragmas multiple times or with multiple value?

Eg, I have a setting "foo": "bar" for both linux and mac. Is it possible to configure as below:
// @sync os=linux,mac
// or @sync os=linux|mac
// or @sync os=linux os=mac
"foo": "bar"

Do I have to write the same setting multiple times to apply to multiple os?
// @sync os=linux
"foo": "bar"
// @sync os=mac
"foo": "bar"

As far as I can see, multiple pragmas with the same prefix are not supported.
You should use your second example or define a specific environment variable on both machines.


LLVMModuleFlagBehavior enum values are all off by 1

enums in C# start by assigning 0 by default, but the enum within LLVM native is defined as starting at 1. In the documentation for native LLVM, 1 maps to error, 2 maps to warning, etc. Also, in the native LLVM source code, the exposed enum LLVMModuleFlagBehavior is defined using the internal enum ModFlagBehavior which is defined using 1 as error, 2 as warning, etc. So, all of the values in LLVMSharp's LLVMModuleFlagBehavior enum are off by 1. This isn't caught by existing tests because the tests don't actually test the behavior of linking modules with conflictin flags. They just check that the behavior passed into LLVM native is the same as the one returned (and the off by 1 translation happens in both directions, so they do in fact match).
PR #179 fixes the error and also adds a unit test to make sure that the enums aren't off by 1 by checking that LLVM.ModuleFlagBehaviorWarning doesn't cause an Error (which does happen in the existing version of LLVMSharp). There is a comment at the bottom of the unit test describing a way to improve the test if someone else knows how to implement this.

You are comparing the C++ enum vs the C# bindings which are over the C version of the enum: https://github.com/llvm/llvm-project/blob/llvmorg-12.0.1/llvm/include/llvm-c/Core.h#L398-L447
See also https://llvm.org/doxygen/group__LLVMCCoreTypes.html#gaa61633f872a8b3cbcbab5f5a96d3ce0f vs https://llvm.org/doxygen/classllvm_1_1Module.html#a0a5c55e12c97b80021330fe82b642293
If there is any error here, it is in the LLVM-C header files and libLLVM source code; not in LLVMSharp.

You're right, thank you for pointing this out.
There are two different ways to add a module flag in LLVM Native: A) using LLVMAddModuleFlag (which expects a 0-based LLVMModuleFlagBehavior) and B) using LLVMAddNamedMetadataOperand (which expects a 1-based behavior represented by a unit). LLVMSharp does expose LLVMModuleFlagBehavior but doesn't expose LLVMAddModuleFlag (which is the only place the 0-based LLVMModuleFlagBehavior would be used), which is rather confusing [this led me to believe LLVMAddNamedMetadataOperand should be accepting the 0-based LLVMModuleFlagBehavior]. I'm going to close this issue and the associated PR and instead make a PR that exposes LLVMAddModuleFlag to ease this confusion. Let me know if you have any thoughts on this.

instead make a PR that exposes LLVMAddModuleFlag to ease this confusion

That sounds reasonable. I'm assuming, however, you mean LLVMModuleRef.AddModuleFlag like the PR has now.
Notably, it does expose LLVM.AddModuleFlag already, it just wasn't visible as a method on the helper LLVMModuleRef struct.

@tannergooding yes, I meant exposing the functionality of LLVMAddModuleFlag within LLVMModuleRef. Here's the PR I created. Thanks for your feedback!


Turn off custom setting for unit tests by default

Critical Changes
Changes
Turn off Affiliation_Record_Type_Enforced__c for default settings of unit tests
(in both UTIL_CustomSettingsFacade.configSettings() and in Hierarchy_Settings__c the Affiliation_Record_Type_Enforced__c field is being set to 'true' by default). This is so the SAL's team unit tests can run again.
Issues Closed
New Metadata
Deleted Metadata
Testing Notes
Check those settings
Also make sure this ticket still works- W-023536

@lparrott yeah all my unit tests are setting that value anyway so..

Please keep me posted when ready @xinnanlisf  and @lparrott. Also, I want to make sure I know what to test/check for the whole value being set to either "false" or "null" by default that Kevin S. brought up.

@xinnanlisf Changes look good, but we hit the dreaded "Apex CPU time limit exceeded" during tests.

now worries, I just rerun it. Should be good now!

I tried a shot at the "Changes" section so that it made sense to users. Anything else we need to say? @lparrott  @xinnanlisf and @jacebryan

Merged.


feat: allow pureCss option

Nice one @jikkai :1: just that pureCSS is an actual existing library and can cause confusion, can I suggest we call this option cssOnly? Thanks.

@van-nguyen cssOnly is a better idea. I agree completely with you.


Calc claim fees

For https://github.com/lbryio/lbryum/issues/232.
Added a function to calculate fee for claim transactions. Fees are based on the claim character length, and only applies to name claims.
Have wallet.make_unsigned_transaction() use this function to calculate new fees (if the claim fee is larger than the relay fee, use it instead)
This PR must be merged and used by most of our users before we can merge the lbrycrd PR.

Codecov Report

Merging #234 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
+ Coverage   54.49%   54.55%   +0.06%
==========================================
  Files          31       31
  Lines        6955     6965      +10
==========================================
+ Hits         3790     3800      +10
  Misses       3165     3165

Impacted Files
Coverage Δ

lbryum/wallet.py
45.04% <100%> (+0.06%)
:arrow_up:

lbryum/constants.py
100% <100%> (ø)
:arrow_up:

lbryum/lbrycrd.py
87.82% <100%> (+0.2%)
:arrow_up:

Continue to review full report at Codecov.

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

Oh add a CHANGELOG entry I guess

On the daemon side, can we expose a function to estimate these fees? There's already a ticket for doing so to transaction fees. It would be great to have both, that way we can display all this information in a confirmation box to the user when they transaction on LBRY... Be that claims/channels/tips, etc.

Added Changelog, changed the creation of tx output in unit test from TYPE_CLAIM to TYPE_CLAIM | TYPE_ADDRESS , rebased


[Events] Native behavior on Enter when hovering an option

Currently
When the user hovers over an option in the dropdown and presses Enter the dropdown closes but the hovered option does not get selected.
Wanted
Native behavior (at least in Chrome/Mac):

Option gets selected
Dropdown closes

i would double check the native behavior across devices...   hover w/ the mouse then hit enter feels odd, but if that's generally native then totally 👍


add tiles widgets support

This widget comes from Metro-UI-CSS http:/…metroui.org.ua/. Changes are:

add animations.less, colors.less, images.less, reset.less, tiles.less, transform.less, utiles.less, 1-5.jpg
change variables.less, add new color names, add @borderRadius used in tiles
change AdminLTE.less, remove google font, because it's not locally, add imports for new less files
change app.js, add tiles, fitImage widgets and initialize revoke for metro widgets, also add menu item click event process
change doc for index.html, index2.html for new tiles entry, and add tiles.html page, but not changed all pages

Note: carousel of metro is not the same as bootstrap, so the effect is not available. And I think metro design of carousel is better.

Hello,
Thanks for the great work! We appreciate your effort. However, those too many features that we can't provide support for. Therefore, we can't merge this for now.
Sorry for the inconvenience!


Only installing namespaced packages, can't run system tests

I am trying to rebase my PR and get it all up to date.
I am having trouble running the system tests. Usually I do:
pip install -e .

To install the package locally in 'editable' mode. However, that's only installing logging and pubsub (those marked as namespaced_packages in `setup.py) into my virtualenv. So when I run the system tests, it can't import bigquery.
Not sure if the repo needs to be fixed or my workflow do, still wrapping my head around how the namespaced pacakges are supposed to work. Is there a reason it's only logging and pubsub in there currently?

@waprin is this in a "fresh" virtualenv?

Since #2304 was merged, if you have tox installed on your path, you can now run just the BigQuery system tests via:
$ tox -e system-tests -- bigquery

@tseaver, @dhermes I'm also running into the same issue now.
Earlier this week I had to change a test script I have to from google.cloud import vision.
This doesn't seem to work now, and I'm having the same issue as @waprin. I tried several new venvs and I haven't found the cause yet.

I think I figured it out.
Reference: http://stackoverflow.com/questions/38918246/why-does-setup-py-sweeps-the-content-of-the-namespace-before-installing/38919020#38919020
I'm making a PR for this now.

Actually nevermind, my idea didn't work to solve this.
@waprin I believe the pubsub and logging namespaces appear because of the gax and grpc dependencies get installed for them.

Ok the system test can be run that way so that works though I'm still a bit confused as to how it all gets installed, will figure it out, thanks!

I'd be interested what the PyPA member has to say.
@jonparrott It seems impossible to install namespace packages in "development" / "editable" mode. Is this not the case?

@waprin I don't personally use "editable" mode, but did notice that system tests were becoming more difficult to run, hence #2304. Sorry for the problem and thanks for bringing it up. I think it's out of our hands, but hope @jonparrott will contradict me on that.

Correct - setuptools does not handle editable installs for namespace packages correctly. This is tracked here with a legendary issue on pip here (not surprised to see @tseaver there).
I've offered to do the work to fix this but I need some guidance from PyPA before I start hacking at it.

@jonparrott it is arguable that pip is broken in general about installing distributions with namespace packages (coercing them into a single, shared tree). It has never worked to mix installations between pip and setuptools when packages installed by each share a namespace.

@tseaver fair. :)


GIX-1934: New SNS fields for NF enhancements

Motivation
Support the new Neurons' Fund enhancements.
In this PR, we upgrade sns-js to the latest IC commit which includes two new fields in the swap canister.
Changes
Automatic changes:

Changes in .did files by running ./scripts/import-candid and then reverting the non sns-js files.
The rest of the changes by running ./scripts/compile-idl-js.

Tests
Fix failing test.

LGTM, thanks.
Can you add a sentence in CHANGELOG before merge?

DONE
@peterpeterparker see this https://github.com/dfinity/ic-js/pull/448 it should help keeping the changelog clean.


fix: add Zeek IP addresses to Enrichment LUTs

Background
Adds Zeek id.orig_h and id.resp_h IP address fields to Enrichment LUTs.
Closes https://app.asana.com/0/1203154630469853/1204469385965390/f
Changes
Adds Zeek id.orig_h and id.resp_h fields to Enrichment LUTs:

advanced/noise_advanced
advanced/riot_advanced
basic/noise_basic
basic/riot_advanced
ipinfo_asn_datalake
ipinfo_asn
ipinfo_location_datalake
ipinfo_location
ipinfo_privacy_datalake
ipinfo_privacy
tor_exit_nodes

@dekatzenel correct, all the referenced Zeek schemas are managed by Panther.


Fix/8480 add health certificate toolkit to xcode build

Description

This PR fixes an issue with the build using the Xcode IDE (Xcode 12.4).
It misses the HealthCertificateToolkit package, and cannot resolve the packages.
Note: not sure if the 'Package.resolved' should be part of this PR, I have committed it separately so it is easy to revert.
Link to Jira

https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-8480
https://github.com/corona-warn-app/cwa-app-ios/issues/3185
Screenshots

The screenshots show the error message and the successful package resolution.

@ArturFriesen good question, I was not aware of this difference, I need to check ..

@ArturFriesen yes, indeed!
I was trying to build the project only.
Now I load the workspace, and all packages resolve nicely!
So I am closing this PR now.
Thanks again a lot for your support!


jpeg optim not installed on heroku production server

Hi,
Thanks for this gem.  I was working great in development becuasee I had ran ...
brew install jpegotpim
However when running in production, my heroku log reads
2015-01-06T06:59:31.954359+00:00 app[web.1]: Attempting to optimize a jpeg without jpegoptim installed. Skipping...
Wouldn't the gem install jpegoptim, or do I have to install something on the heroku server as well?
Thanks

Hello,
Heroku recommends using a buildpack to get jpegoptim working:
https://github.com/jayzes/heroku-buildpack-jpegoptim
You could also add jpegoptim to your app's bin/ directory, but that's not a clean way to do it (for me).

@czj is correct. You need to have the jpegoptim binary installed on your server and the buildpack is the recommended way. If you install via another method you can always set the path via the JPEGOPTIM_BIN env var.

When I create the buildpacks file
Do I name it   filename.buildpacks
or just
.buildpacks  ( like a .env file)
https://github.com/jayzes/heroku-buildpack-jpegoptim
https://github.com/heroku/heroku-buildpack-ruby

Just .buildpacks with the contents explained in https://github.com/jayzes/heroku-buildpack-jpegoptim#usage

Thanks, that was my guess as well.
I created a file named .buildpacks and added this
https://github.com/jayzes/heroku-buildpack-jpegoptim
https://github.com/heroku/heroku-buildpack-ruby
Then ran
heroku create --buildpack https://github.com/ddollar/heroku-buildpack-multi
But when I run
heroku run "jpegoptim -V"
I see
bash: jpegoptim: command not found
What am I missing?
Thanks

@jhorsch You need to push the buildpack with "git push heroku master", you should then see the jpegoptim installation going on and then it should finally work!


Added a map option

The map option allows the routes to be restricted to set HTTP verbs:
This will only respond to GET and POST requests.
Macaw::map(['GET', 'POST'], '/demo', function() {
  echo 'hello only get and post!';
});

👍


[MERGE] Branch 1.6

@aaudiber were there any conflicts?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16689/
Test PASSed.

@calvinjia No, just wanted to see the build pass


Exam 22 - Question 25

A company wants to set up a highly available workload in AWS with a disaster recovery plan that will allow the company to recover in case of a regional service interruption.
Which configuration will meet these requirements?
A. Run on two Availability Zones in one AWS Region, using the additional Availability Zones in the AWS Region for the disaster recovery site.
B. Run on two Availability Zones in one AWS Region, using another AWS Region for the disaster recovery site.
C. Run on two Availability Zones in one AWS Region, using a local AWS Region for the disaster recovery site.
D. Run across two AWS Regions, using a third AWS Region for the disaster recovery site.
The given answer is D, however when reading the blogpost mentioned, it seems that they only have 2 regions?
Summary:
The company Sprinklr implements an Active/Passive model for disaster recovery. They maintain critical services running in different regions, indicating the use of another AWS Region as a disaster recovery site to ensure business continuity in case the primary region goes down. This strategy provides high availability within the primary region through the use of multiple Availability Zones and ensures recoverability by utilizing a separate region for disaster recovery purposes.

@StefH I think B and D both are correct but option D increased complexity and cost.


Create share by e-mail after invitation accepted

Scenario

Let's assume we have user A who starts the invitation workflow by sending an invite link to user B's e-mail<EMAIL_ADDRESS>he knows.
User B accepts the invite link by going through mesh directory and logging into his provider of choice
User A now wants to share something to B
At this moment, user A still knows only the user B's e-mail<EMAIL_ADDRESS>
The problem
User A has currently no means of determining grantee.opaqueid used in the invitation sharing workflow, he doesn't even know which provider.idp was used by user B to accept the invite.

User A should be able to create a share for the user B by using the e-mail adress used in the invitation workflow:

$ cmd/reva/reva -insecure ocm-share-create -grantee<EMAIL_ADDRESS>/home/test

or/and

User A should be able to list remote users that accepted the invites he sent. Such accepted invite records should contain some label, that helps user A to recognize, who actually accepted the invite using which invite link (the label would be the used e-mail address<EMAIL_ADDRESS>in this case). User A could then find all necessary sharing details about user B (provider & opaqueid) in this listing based on the label he already knows and any additional user info exchanged at the moment the invite was accepted.

@mirekys can you start implementing option 2)? This is the foundation

Implemented in #1527


Fixes 1793

https://github.com/NuGet/docs.microsoft.com-nuget/issues/1793

Docs Build status updates of commit 3f7ab0f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/nuget-org/Publish-a-package.md
:warning:Warning
View
Details

docs/reference/cli-reference/cli-ref-setapikey.md
:warning:Warning
View
Details

docs/nuget-org/Publish-a-package.md

Line 66: [Warning] File 'quickstart/includes/publish-api-key.md' referenced by link '../quickstart/includes/publish-api-key.md' will not be built because it is not included in build scope

docs/reference/cli-reference/cli-ref-setapikey.md

Line 26: [Warning] File 'quickstart/includes/publish-api-key.md' referenced by link '../../quickstart/includes/publish-api-key.md' will not be built because it is not included in build scope

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

Docs Build status updates of commit 133844d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/nuget-org/Publish-a-package.md
:warning:Warning
View
Details

docs/reference/cli-reference/cli-ref-setapikey.md
:warning:Warning
View
Details

docs/nuget-org/Publish-a-package.md

Line 66: [Warning] File 'quickstart/includes/publish-api-key.md' referenced by link '../quickstart/includes/publish-api-key.md' will not be built because it is not included in build scope

docs/reference/cli-reference/cli-ref-setapikey.md

Line 26: [Warning] File 'quickstart/includes/publish-api-key.md' referenced by link '../../quickstart/includes/publish-api-key.md' will not be built because it is not included in build scope

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


Add effective matrices functions

A useful function to add is how to go from a bipartite network to two effective competition matrices (one per trophic level), as done in Rohr et al. Science.

A preliminary version is added at commit 6c9e111


Constant Current Loads in ZIPS

Model for these is wrong: we want the phase of the current to be locked to the phase of the voltage - so that the power is not dependent on the global phase of the voltage. I've updated the theory notes, but not the code.

I've solved this one - and added a test in tests.


Page is inconsistent with what Visual Studio does.

This page is already inconsistent with what Visual Studio give me when creating the project. I have no main.cpp; I have two cpp files, one called pch.cpp with a single entry, #include pch.h. Then I have Class.cpp which has some auto-generated class files in it as such
int32_t Class::MyProperty()
{
throw hresult_not_implemented();
}
void Class::MyProperty(int32_t /* value */)
{
    throw hresult_not_implemented();
}

Getting started is the hardest part. What's on this page is so different its going to be a lot of trial and error to get beyond the first step.
I would also add that there is an undiscussed concept in Visual Studio called an IDL file. It is, once again, another black box configuration files. These need to be clearly and thoroughly documented. Configuration files are not based upon logic; they just are. Get everything right to the most minute detail, or nothing works.

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

ID: 7ae018a1-a3ab-037a-2df7-354677971ff9
Version Independent ID: 95ff0a4f-48c7-4962-93b3-538ab5ec458f
Content: Get started with C++/WinRT - Windows UWP applications
Content Source: windows-apps-src/cpp-and-winrt-apis/get-started.md
Product: uwp
Technology: platform
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, @brianreinhold, there is a Windows Runtime Component (C++/WinRT) (my emphasis) project template, which does generate files and source code that are consistent with your report (including the presence of an IDL file). However, this Get started topic calls for a Windows Console Application (C++/WinRT) project. If you create one of those, then you'll see files and source code that match what the topic is describing. And there'll be no IDL file.
This particular topic doesn't cover IDL files. However, several of the others do, for example: https://docs.microsoft.com/windows/uwp/cpp-and-winrt-apis/binding-property. That topic walks you through what you need to do with the IDL files for that particular topic. For a more general walkthrough and intro to the syntax of the IDL files that you use with C++/WinRT, see Introduction to MIDL 3.0.
Thanks!
~Steve

Steve,
Thanks for the response. Part of this was my fault for not fully understanding what all these new items (new for me) are. I thought the Windows Component template was for generating UWP C++/winRT libraries for C++/winRT applications; the C++/winRT analog to the C++/CX dll and static lib templates. I did not realize at the time that a Windows Component was a 'special' feature which after another week I have come to better grasps with. Since I did not find C++/winRT dll and static lib templates the only thing really left in the C++ library department that made any sense for UWP apps was this template. Alas not so!
Customers in the remote health infrastructure have become frustrated with the problematic Bluetooth support in iOS and Android and are starting to ask if there are any Windows solutions. We develop such solutions following PCHA/HL7 standards and that's what we are looking into. In the end we hope to have a 'Windows Bluetooth Adapter' that can be accessed from our Java implementation thus the choice of C++/winRT and a dll (access through JNI). The ideal dream solution would have been if there was a Java/winRT option.
In the mean time we are hoping that a C++/winRT dll and static lib template will be added. I am trying to create such a project from the C++/winRT core template which I have successfully made a simple BTLE scanner app from.
Thanks,
Brian

From: Steven White<EMAIL_ADDRESS>Sent: Monday, April 15, 2019 10:26 PM
To: MicrosoftDocs/windows-uwp
Cc: Brian Reinhold; Mention
Subject: Re: [MicrosoftDocs/windows-uwp] Page is inconsistent with what Visual Studio does. (#1598)
Hi, @brianreinholdhttps://github.com/brianreinhold, there is a Windows Runtime Component (C++/WinRT) (my emphasis) project template, which does generate files and source code that are consistent with your report (including the presence of an IDL file). However, this Get started topic calls for a Windows Console Application (C++/WinRT) project. If you create one of those, then you'll see files and source code that match what the topic is describing. And there'll be no IDL file.
This particular topic doesn't cover IDL files. However, several of the others do, for example: https://docs.microsoft.com/windows/uwp/cpp-and-winrt-apis/binding-property. That topic walks you through what you need to do with the IDL files for that particular topic. For a more general walkthrough and intro to the syntax of the IDL files that you use with C++/WinRT, see Introduction to MIDL 3.0https://docs.microsoft.com/uwp/midl-3/intro.
Thanks!
~Steve
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-uwp/issues/1598#issuecomment-483486590, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Alg3e2PFPD8GZtyjHQRx0S-61B1q7---ks5vhTTJgaJpZM4csWOB.

Sounds good, thanks for the update, @brianreinhold! I'll go ahead and close this issue. Thanks for evaluating C++/WinRT for your project, and for filing the issue. ~Steve


Shared Flow and API can be created and deployed with a single apigeecli command

https://github.com/GoogleCloudPlatform/apigee-samples/blob/8ba299a76c69d5f233e1eda5e2cefe7ccd3d409b/authorize-idp-access-tokens/deploy-authorize-idp-access-tokens.sh#L114
https://github.com/GoogleCloudPlatform/apigee-samples/blob/8ba299a76c69d5f233e1eda5e2cefe7ccd3d409b/authorize-idp-access-tokens/deploy-authorize-idp-access-tokens.sh#L119

@kurtkanaskie - do you have a sample command we could use?

Yes
apigeecli sharedflows create bundle \
    --org "$PROJECT" --env "$APIGEE_ENV" \
    --name authorize-idp-access-tokens \
    -f ./rendered/sharedflowbundle \
    --ovr --wait

This PR (https://github.com/GoogleCloudPlatform/apigee-samples/pull/165) should address this issue.


[sw] DIF pwrmgr smoketest does not work in DV

It works in Verilator possibly due to its single clock system. In DV, the wakeup arrives before the WFI.

Working on a fix for this.

The test sets the AON timer to issue a wake up after 5 clock cycles at 200KHz, which computes to 5 * 5us = 25us.
On the next line, the test puts the Ibex to sleep for 100us.
https://github.com/lowRISC/opentitan/blob/c48b9195507755e58235583f6ceed8261a792954/sw/device/tests/dif/dif_pwrmgr_smoketest.c#L50
I am not sure I understand the need for this delay - 100us >> 5us. Seems like the wakeup should arrive before the WFI on Verilator too. Will investigate further.

The pwrmgr setup routine consumes about 50us, so the programmed threshold needs to be at least 10.

This is because the DIF APIs hide the synchronization routine (notice the large number of Get transactions after each pwrmgr CSR is programmed. We end up incurring the delays due to synchronization 3 times. IMO a better strategy would be to instead expose the sync routine as an separate API so that we incur it only once.

Fixed in https://github.com/lowRISC/opentitan/pull/6500/commits/ea4789518e646e665d3fdcd4f8a4d82727160346.

i believe this been addressed some time ago.


Options button isn't working in Firefox

Clicking the "Options" button does nothing. In Chromium based browsers, it loads the full options page in a separate tab.

Apparently, this is the reason.

This manifest key has been deprecated. Use options_ui instead.

The key was disabled in Firefox but not in Chrome.

Fixed.


Add another template for displayLeft/Right column hook

Questions
Answers

Branch?
develop

Description?
Link list was broken on displayLeftColumn and displayRightColumn hook

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes https://github.com/PrestaShop/PrestaShop/pull/22076.

How to test?
See issue (should probably be tested with https://github.com/PrestaShop/PrestaShop/pull/22076)

@PierreRambaud I believe milestone should be 4.0.1? or 4.1.0?

@kpodemski The last release is https://github.com/PrestaShop/ps_linklist/releases/tag/v3.2.0. Fixed

@Progi1984 yet we have v4 on develop? 🤔

LGTM,
Test with : https://github.com/PrestaShop/PrestaShop/pull/22076
Thanks @NeOMakinG

Thank you @NeOMakinG


Colin/readme

first version of read me
For rest of devs to change and fill out (especially at the end)
Current Bugs and Known Issues

[List any known bugs or issues here]

Future Enhancements

[List any planned or suggested future enhancements]

looks good


feat: pluggable scanners

Description
Type of Change
[ ] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)
Checklist

[ ] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[ ] I have verified this change is not present in other open pull requests
[ ] Functionality is documented
[ ] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

changes moved to https://github.com/openclarity/vmclarity/pull/1510


Can a series of images generate video

ffmpeg -framerate 3 -pattern_type glob -i "image-*.jpeg" output.mp4

Ok sounds interesting. I could implement another class facilitating this. Thank you! Feel free to issue a PR if you wish

Looking forward to your new method class


Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter dysonairpurifier for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.dysonairpurifier
   -
:thumbsup: No errors found
WARNINGS:

[ ] :eyes: [W953] .npmignore found - consider using package.json object "files" instead.

:thumbsup: No suggestions found
Please review issues reported and consider fixing them as soon as appropiate.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Tue, 01 Oct 2024 10:48:54 GMT based on commit 5e20a18b12c5727186ef19def0a7b2220fd712cd
ioBroker.repochecker 3.1.0

@iobroker-bot recheck


[cli] Support typescript compilation for component source

Is there a plan for supporting typescript for authoring components and still using polymer cli commands? I would hope this would be opt in in the future since the cli is trying to minimize transforms.
Thanks!

This is not likely to be added to polymer-cli anytime soon.  Will re-open if there is bandwidth for this in the future.


Current test dataset is packaged in .zip format on Figshare (not .tar.gz)

The following commands fail with a gzip: stdin has more than one entry--rest ignored error:
wget https://figshare.com/ndownloader/files/41228403 -O test_samples.tar.gz && \
    tar -xzvf test_samples.tar.gz && \
    rm test_samples.tar.gz

This is because the test dataset is actually in .zip format instead of .tar.gz. To fix this, the documentation can either be updated to a functional command, or the Figshare dataset can be replaced with one in .tar.gz format.

https://figshare.com/ndownloader/files/41228403 is an old link; all docs should link to https://figshare.com/ndownloader/files/41228403 now.


fix(uart): fix port and logging function

Signed-off-by: Danyang Song (Arthur<EMAIL_ADDRESS>

obsoleted by #5 - closing


Update cortana-analytics-playbook-predictive-maintenance.md

Changed authors tag

Hi @fboylu, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (fboylu). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@garyericson
fyi


Doesn't work as mu-plugin in theme

I required this (mu-)plugin in a sage theme - but this plugin isn't
listed as obligatory installed/enabled plugin.
It works if I require it for the site instead for the theme (site/) -
but it should work for the theme.

You need to specify the installer-paths in your theme's composer.json -- Sage 9 by default doesn't have them.
"extra": {
  "installer-paths": {
    "../../mu-plugins/{$name}/": ["type:wordpress-muplugin"],
    "../../plugins/{$name}/": ["type:wordpress-plugin"]
  }
},

A sane spot for it would be right above "scripts".


feat: add ModuleGraph type

Which problem is this pull request solving?
The return value of serve includes a graph property that contains the module graph for the functions. This is currently untyped, which makes it harder for consumers (like the CLI) to process this module safely.
This PR adds a ModuleGraph type, extracted from<EMAIL_ADDRESS>which we're now vendoring.
I've also added a new vendor npm script for recreating the deno/vendor directory with all the modules we're referencing.

Ah, I had to push https://github.com/netlify/edge-bundler/pull/528/commits/adf717e15bc2c8ef2bb3b666a0deca2ae12d4ca4. Can I get another ✅, please?


[build][Juekins] more efficient Jenkins CI, builds, improved linter config

What it does

This is a significant refactoring of the repo and CI.
1) JenKins CI: more efficient, quicker builds
I propose that we adopt a "run often, run quickly" approach to our Jenkins CI.
ATM we mostly run Jenkins when it's time to release new Blueprint installers
(when a PR is merged on master). The other case is if "jenkins" is detected
in the PR branch or PR title, then almost a full Jenkins build is done, just
stopping short of publishing new Blueprint installers (full signing and
notarization is done, which can take a couple of hours).
I made it so we run a very light version of the Jenkins CI by default, that
builds the "dev" version of the app and pre-packages it using electron builder,
generating a directory that contains the app but no installers.
We really ought to run tests then, on the unpacked pre-packaged app, but
it will need to be done separately. Until then, we do not require the built-in
extensions (plugins), so I gave openvsx.org a rest and skip fetching them, by
default.
When it's detected that the Jenkins CI is processing the result of a merge on
master or if the new "release dry-run" mode is enabled [1], a production version
of the Blueprint apps will be produced, including built-in plugins, which will
then be packaged for "real", producing genuine OS-specific installers, signed
and notarized as required. This will take a lot longer to run, of course, but
may be required only a minority of the time.
Conclusion: with this PR, running Jenkins CI now takes about 12 minutes, and
since this is relatively quick, we can afford to run this when any PR branch
is created or updated.
Running the same CI in "release dry-run mode" takes about an hour. A release
will take longer, a bit over an hour, since additional steps are executed,
like copying the various installers to a release folder.
[1] About the "release dry-run" mode, it can be enabled in JenkinsFile,
"pipeline" definition near the top, in "environment":
BLUEPRINT_JENKINS_RELEASE_DRYRUN = 'true
I wish I found a better mechanism to enable this "release dry-run" mode. But
this will serve for now. The "release dry-run" mode should be disabled before
merging a PR to master, but can in the meantime be used to test or troubleshoot
the whole pipeline, in particular signing and notarizing.
2) tsconfig/eslint configs:
Added configs, to improve linter coverage. This made it possible for some
source files, not previously covered, to get ts/linter feedback, both while
editing and when running yarn lint. This will help keep code in-line with
our standards. The config is not perfect and I would welcome further
improvements. But for now I think it's a nice improvement.
3) Build "scripts" (package.json)
Refactored the build commands ("scripts" section in package.json)

previously, merely running 'yarn" in the repo's root would rebuild
every application from scratch. This prevents running a quick
"yarn install", e.g. just to re-install build dependencies
the new version permits a granular build, with simple defaults
inspired from a similar change not so long ago in the main repo
see updated README for some examples

Other misc items:

renamed extensions / applications

made the browser application a first class citizen, equal to the
Electron application

all applications now share a common 'plugins' folder rather than
each having their own. Moved the plugin-related entries to root
package.json

to gain flexibility about which yarn workspaces are invoked for a
given lerna command, using the --scope= CLI option.I renamed the
repo's extensions and applications. This permits easily composing
commands that target only the extensions or only the applications.
e.g.:
  "build:extensions": "lerna run --scope=\"blueprint*ext\" build",
  "build:applications": "lerna run --scope=\"blueprint*app\" build --concurrency 1","

renamed the extensions folders, made them more straightforward

For systems with limited RAM, like on a Raspberry Pi 4B board with 4GB of RAM,
it's now possible to successfully build Blueprint. e.g. use the following cmd:
yarn && yarn build:dev, optionally followed by a packing command like:
yarn electron package

(build:dev will build the Blueprint app in dev mode, which can be achieved
in 4GB RAM)

[windows][jenkins] stash only dist folder: Currently, and only for Windows,
we stash the whole git repo, which is very big and takes long to stash and
un-stash. For the other OS, we only stash the dist folder, that contains the
platform-specific installer, that we built. Let's try that for Windows too,
and see if it works.

[jenkins] Decrease timeout from 5h to 3h" Looking at the build history, all
recent builds that succeeded, did do under 2h30. OTOH, when a build hangs,
it needs to wait for the timeout to expire, wasting time at the current value
of 5h. Let's compromise at 3 hours and see how it goes?

[jenkins][installer build] exclude browser app for now: We do not yet publish
the browser app, so let's skip building it to save time/resources. We may
revisit when we use the new browser app bundling, recently made available
upstream in the Theia framework.

How to test

Run the tests locally (currently only covers Electron):
cd theia-blueprint
yarn && yarn build && yarn download:plugins && yarn package:preview
yarn test

A few other things to test. For each make sure that the built-ins are present

the browser app starts

yarn browser start

the electron app starts

yarn electron start

packaging and starting the electron app works

yarn electron package
# test the generated package. e.g. on Linux: `applications/electron/dist/TheiaBlueprint.AppImage`

Review checklist

[x] as an author, I have thoroughly tested my changes and carefully followed the review guidelines

Reminder for reviewers

as a reviewer, I agree to behave in accordance with the review guidelines

Progress: using this PR branch I can successfully build and run Blueprint browser on my Pi 4:

closed by accident :)

@vince-fugnitto could you have another look?

@jfaltermeier thanks for the initial look! i have rebased the PR on the latest master and added the couple suggested yarn rebuild. I will try to fix the docker problem and update the PR.

Also the Browser Docker Image does not seem to be working yet:

I believe that I have fixed the issue with browser.Dockerfile

Thanks for the review @jfaltermeier!
@JonasHelming @thegecko FYI, this PR has some changes in it that could potentially be breaking e.g. if someone has a product based on Blueprint, but I do not think it should be anything too difficult to fix. I wonder how e.g. cdt.cloud consumes / extends / depends on Blueprint?

I wonder how e.g. cdt.cloud consumes / extends / depends on Blueprint?

CDT Cloud Blueprint looks like a fork of this repo, and I see signs it's kept up-to date. So it will be a bigger rebase than usual.

@planger FYI

@planger if it's welcomed, I can follow-up with a cdt-cloud-blueprint PR, that would update it , so it "fits" with the new Blueprint, after the merge button is pressed. I may require a bit of feedback, since I do not know the ins-and-outs of that product and how it's used. WDYT?

@planger unfortunately I do not think I will have time to do the follow-up, in the next little while. Please tell the person that eventually does the uplift that they can tag me on the PR if there are questions.

@marcdumais-work Ok thanks for letting me know anyways! We'll get back to you if there are questions. Thanks!


Change Model User Authentication

If you can change UserModel For Authentication To Costum Model or ShopModel Because User it's not a realy name and  maybe  user have more then one shop so if you do A ShopModel For authentication it will be  realy realy better
if i have already projet and i have user for another utility i cannot use your library  this  is other problem it's will be good  if change Authentication to other model
thanks.

i have an idea for this solution it's add item  like this  ('auth_model' => App\Models\User) to shopify-app.php in config
for dont create problem for old user
and if hi went to change model User  it's simple just i will go to  shopify-app.php in config and change item auth_model to  App\Models\Shop for example . it will be the result  lik this   ('auth_model' => App\Models\Shop)

You should be able to using the config file to update the Auth Model to something else..
    /*
    |--------------------------------------------------------------------------
    | Shop auth provider
    |--------------------------------------------------------------------------
    |
    | This option allows you to override package's build-in auth model
    | If you need to keep User model intact, add custom auth provider and route middlewares for it
    |
    */
    'shop_auth_provider' => env('SHOPIFY_SHOP_AUTH_PROVIDER', 'users'),


Improve proxy detection

we should allow using a proxy if you are hitting localhost and the proxy is local
if you are hitting localhost and the proxy is not local we should notify the user and allow to disable it in the toast
double check that the built in electron proxy works

Some notes:
If a request goes to localhost, <IP_ADDRESS>, ::1, <IP_ADDRESS> we already skip the proxy automatically.
Electron doesn’t really use a proxy, but hooks into the cors protection, which should work with local and remote requests.


Added a unit test for IsOwner()

Signed-off-by: Ankit Kurmi<EMAIL_ADDRESS>Description
Added some unit test for functions in internal/ownerhandling/annotations.go

lgtm, but needs a spelling fix :)


[DRAFT] Fix error logs in Adapter

Purpose

$subject
Git Issue: https://github.com/wso2/apk/issues/1406

Codecov Report
Patch coverage has no change and project coverage change: +3.88 :tada:

Comparison is base (2f5ebea) 24.38% compared to head (b1d0538) 28.27%.

:exclamation: Current head b1d0538 differs from pull request most recent head 517f4f4. Consider uploading reports for the commit 517f4f4 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1461      +/-   ##
==========================================
+ Coverage   24.38%   28.27%   +3.88%
==========================================
  Files         260       76     -184
  Lines       30802     5100   -25702
  Branches     4834     2415    -2419
==========================================
- Hits         7512     1442    -6070
+ Misses      23024     3658   -19366
+ Partials      266        0     -266

Flag
Coverage Δ

adapter
?

common-controller
0.00% <ø> (ø)

config-deployer-service
23.47% <ø> (+2.59%)
:arrow_up:

idp-domain-service
80.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 189 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Create new front-end software developer technical test

The existing Open Targets front-end technical test only assess D3 visualisation skills with a pre-processed Ensembl API response.
However, as we are moving to React, we should update the test to reflect the tech stack we now use and to assess candidates' skills in React (project setup, calling a more complex third-party API, state management, etc.) and data visualisation.
To-do:

[ ] Create updated technical test document
[ ] Create mock API using /associations endpoint data
[ ] Create design spec and produce PNG images
[ ] Create repository for technical test

Complete:

[x] Technical test document
[x] Mock API
[x] Design spec
[x] Code repository

Design specs for reference:

Work to update technical test has been completed


DeviceStatus r1.3 patch release review

Release PR or issue to review

https://github.com/camaraproject/DeviceStatus/pull/216

Review actions
Assign this issue to yourself and follow the below list.
For any review action, review the file(s) in the issue/PR listed above.
Put comments in the above issue or PR if they concern non-changed files/text.
Put a short summary of the main review result here into the review issue.

[x] API definition files (YAML) (check version in info & servers objects)
[x] test file(s) availability
[x] changelog updated
[x] readme updated (enforce correct release number / API version naming)
[x] API readiness checklist(s)

Release actions
Assign this issue to yourself or another RM team member and follow the below list.
When done, tick the box in this issue.

[x] API release tracker available on wiki
[x] Review comments addressed (by Sub Project)
[x] Release PR approved (on behalf of Release Management)
[x] PR merged (by Sub Project codeowner)
[x] Release created within GitHub (by Sub Project codeowner)
[x] Release Tracker updated (with creation date of the release and the release tag link)
[x] PR on https://github.com/camaraproject/.github/blob/main/profile/README.md created (by Release Management) and merged (by CAMARA admin)

Additional comments

Created https://github.com/camaraproject/.github/pull/6 to address the last checkbox. Please review @tanjadegroot.

Review closed as completed


Extend resolution 320x240 for LCD screen x 8bit

Extend screen driver

Arcade assumes that the screen is 160x120 4bit color. All our APIs for game creation also use that assumption. For larger screen, we simply upscale the resolution in the driver but the user does not see a larger resolution.

The decision is to resolution compatible with arcade - 160x120. Closing.

Due to the limitation of Arcade, ROBOTIS will accept this resolution.


Links

Is it possible to get links working in description (summary) at frontpage?
Also is it possible to link image's post to url's post in frontpage?
thanks!

Sorry for taking so long to reply.
The first question, do you mean the motto in the config?
And I didn’t understand the second question too much.

No problem.

I mean when you put a link in description. See number 1 at this image, should be a link but parses as plain text:

The other question is if its possible to link image to post link (see above image, number 2). When you click at thumb you should go to the post.

Know about it. Already done in f40b2d1.~

Great ! many thanks !
About 2nd suggestion, I used a link in this format:
[Some text](https://www.example.com)
and didn't work at main page (Yes it works at post page).
What format should I use?
thanks again!

Which "main" page do you use?

I'm not sure what you mean. I think I use the default one.

The text is a markdown syntax. It can be only used in .md file. So I’m not sure which main page you are talking about.

I use that link in test.md and that file, could show as a single page, were is renderized right (as link):

or at main page (front page), were is not renderized as link:

Sorry if my explanation isn't clear enough.

Not mind.

I think the problem you encountered should be caused by the lack of <!--more-->.
Make sure <!--more--> behind your summary.
Here is the doc: https://gohugo.io/content-management/summaries/#manual-summary-splitting

Perfect !! That works ! :green_heart:


Crashing on nodered 2.22

Morn. So latest docker nodered image, if I install your posgres node, nodered runs ok. If i drag and drop on workspace, nodered keeps on crashing. Before even I put credentials.
Your node is awesome bro. I use it on thousand flows. Please help.
btw. I run the latest docker image. SO please test on docker image.
Regards
Jakes

Update.
I install latest docker image and make it run.
I add postgressql in pallet. (all ok)
I drag and drop on workspace. Nodered starts to restart. (postgressql node not configured yet, so it shows invalid)
I configure noderedsql and deploy until eventually nodered accepts.
nodered restarts once more and then runs normally.
Hope you can recreate issue. For now Ill continue like that.
Ciao

Thanks for the bug report. I will fix ASAP.
Meanwhile, my guess is that the node needs to be configured (e.g. database parameters) before deploying.

Should be fixed by https://github.com/alexandrainst/node-red-contrib-postgresql/commit/83f494525c70f2323803fb5158b4abbe1e53d0d4
New release coming

Version 0.11.2 released https://github.com/alexandrainst/node-red-contrib-postgresql/releases/tag/v0.11.2
Feedback welcome


Add Foldable1

For when you only need an Apply or Semigroup instead of Applicative or
Monoid.

I have a feeling this might be better formulated in terms of foldl1 and foldr1. What do you think?

I think foldr1 should be derivable using Dual, is that right?

Looks nice, thanks!
Could you please add some comments? I'm trying to require code comments on all exported core library functions and types now that we have public docs. Act in particular could benefit from a few explanatory comments IMO, since f a has at least a couple of sensible Semigroup instances.
And yes, I think Dual is a nice way to derive foldr, rather than requiring both in instances.

@joneshf may like to weigh in on the foldr stuff too?

If you get a chance could you rebase and update this? It still looks good to me :+1:

We should probably start a naming convention for default/derived functions?
With my PR https://github.com/purescript/purescript-foldable-traversable/pull/30 I've used _Default whereas you have used derived_.

Yeah, that would be good. I have no particular preference for either option though.

Is this PR dead? @puffnfresh

I'd also be really interested in this.. I would also help by sending an updated PR.

This PR seems to be dead, so I'll close it for now, but we can reopen it if the issues above can be addressed. Thanks!


Devel

Coverage increased (+0.5%) to 87.027% when pulling 966d11638629ad5239f0cbb1efdcdae304744531 on devel into 83e3c6ece492abc1565cac97dfc6563b0c3aa275 on master.


Update Go Components page - to align with 2.1.0

Signed-off-by: tpmccallum<EMAIL_ADDRESS>Content must go through a pre-merge checklist.
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[x] The title, template, and date` are all set
[x] Does this PR have a new menu item (anywhere in templates/*.hbs files) that points to a document .md that is set to publish in the future? If so please only publish the .md and .hbs changes in real-time (otherwise there will be a menu item pointing to a .md file that does not exist)
[x] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep '^M' | wc -l and expect 0 as a result)
[x] Has passed bart check
[x] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)
[x] Headings are using Title Case
[x] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[x] Have tested with npm run test and resolved all errors
[x] Relates to an existing (potentially outdated) blog article? If so please add URL in blog to point to this content.

References:
https://github.com/golang/go/issues/31105#issuecomment-1552362280
https://tip.golang.org/doc/go1.21


lite-server causes browser sync across devices

It appears the browser sync does not allow for multiple requests, and causes web pages to scroll globally across devices.

resolved by replacing lite-server with a normal http-server. confirmed on mobile and desktop.


fix: support  lvgl v9

As the title says

As it's a local change and probably won't harm anyone I merge it.


Mesh Display Fixes

Hi! This PR adds a bunch of fixes for the mesh display.
I had a few segfaults here and there when I tried to visualize a mesh map that rviz gets from the mesh_navigation_server via topics.
Changes:

In order to understand all the different states and state transitions in the plugin, I refactored some of the methods.
Remove a few message_filters that seemed unnecessary to me. I.e. the tf2_ros message filter only makes sense for geometry, since colors or costs do not need to get transformed into different frames - they attach to the location via vertex ids in geometry.
Actually use the caches for color and cost msgs, this fixes the (ROS1) issue that colors need re-subscription to display

This happened when the colors arrive before the geometry. This still can happen, but via the cache mechanism, we can apply colors when geometry arrives

Fix segfault when loading Mesh Display from config, somehow caused by the Transformer changing (see     SIGNAL(transformerChanged(std::shared_ptr<rviz_common::transformation::FrameTransformer>))). I looked this up someplace deep in the rviz repo. This was quite a pain to find out... Segfault was someplace deep in the tf2 message filter or std::tuple :/
Use latched QoS in subscribers

Related vid https://drive.google.com/file/d/1VSo_PGoJJM1n4maeS5M8u6g1W44vbPf9/view?usp=sharing

Great change!
I tested everything on my computer. It is compiling. Loading static maps from files is working as before. So I see no reason to wait for merging this PR. However, what I wish to have in the future:

Examples for generating and publishing different types of meshes
Minimal examples for visualizing meshes both from a file and a topic

So that we can better test that new PRs are not causing runtime errors. If we have that, we can also think of integrate them into CI, or write separate tests. But this is an issue for another time.
-- Alex


Issue using same generated playlist on multiple devices at the same time will cause stream stops/errors

I found ClientID,DeviceID,SessionID. I only tried changing ClientID for each playlist which for now resolved the issue?
This was never an issue with the list from i.mjh.nz
How can we fix this, moving forward when auto generating a list that I want to share on multiple devices playing pluto?

the issue is that plutotv uses the client id in their backend. Each client needs a unique id. What I do is generate the stream with a known client id, and then replace it with a random id when it is served from my web server.
I think i.mjh.nz was generating a random clientID when it was doing a 302 redirect to the actual stream url.
I've also done some tests as well and these scenarios worked for me:

multiple stream playback with different clientID
multiple stream playback with same clientID and different X-Forwarded-For

What didn't work was same clientID and same (or no) X-Forwarded-For.
It could also be that the i.mjh.nz streams were for an older API, whereas this script uses the latest API and gets the latest streams 🤷♂
Unfortunately, as this requires changes to be made to the stream urls upon request by the player, there's nothing that can be done in the script to generate the playlist.

I thought I would give you an update. Although we cannot have unique client IDs baked into the playlist, and it would be the responsibility of the server that is serving, or the client consuming the playlist to ensure uniqueness, I've created a "server mode" for the script that does just that. When a --port <num> commandline option is given, the script will listen on the provided port and will serve up the playlist and xml file (based on file name). When the playlist is requested, a new clientID is generated based off of the callers IP address, and injected into the stream urls.
I'll also mention the --refresh <seconds> option where the script will act as both the server and the scraper (updating the playlist an xml at the specified interval). The plan is to allow the script to be run as a service in either a docker image, or as a self hosted server and the user can configure their clients to pull the playlist and xml files from the local server.

I thought I would give you an update. Although we cannot have unique client IDs baked into the playlist, and it would be the responsibility of the server that is serving, or the client consuming the playlist to ensure uniqueness, I've created a "server mode" for the script that does just that. When a --port <num> commandline option is given, the script will listen on the provided port and will serve up the playlist and xml file (based on file name). When the playlist is requested, a new clientID is generated based off of the callers IP address, and injected into the stream urls.
I'll also mention the --refresh <seconds> option where the script will act as both the server and the scraper (updating the playlist an xml at the specified interval). The plan is to allow the script to be run as a service in either a docker image, or as a self hosted server and the user can configure their clients to pull the playlist and xml files from the local server.

This sounds really neat, great progress, will definitely use this, but also hoping for the epg url where we can host our own path and it will respect it :P


ci(greenkeeper): skip sonar scan

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


gitea sending "null" password to stmp server

Description
I have a selfhosted gitea instance which i have configured to use my selfhosted mailserver(postfix+dovecot+cyrus-sasl) for sending mail. My mailer config is below(I have the password in single quotes):
[mailer]
ENABLE = true
FROM =<EMAIL_ADDRESS>PROTOCOL = smtp+starttls
SMTP_ADDR = mail.example.com
SMTP_PORT = 587
USER = gitea
PASSWRD = 'gitea_password_here'

When sending a test email from the web gui, I get this error:

My mailserver's  saslauthd service shows this error:

I am able to use the username and password to login to the mailserver manually with a desktop mail client. I can manually send and receive email as the gitea user.
I am not sure why gitea is sending a NULL password, I have tried using the follow pairs of characters to quote the password in app.ini: " ",' ', ` `.
I cannot reproduce this issue on the demo site as none of my systems are exposed to the internet.
Gitea Version
1.21.5
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Fedora 39
How are you running Gitea?
I am using a docker compose configuration from the install documentation.
Database
SQLite

What type of auth are you using:
Is it one of the following:
	if strings.Contains(options, "CRAM-MD5") {
		auth = smtp.CRAMMD5Auth(opts.User, opts.Passwd)
	} else if strings.Contains(options, "PLAIN") {
		auth = smtp.PlainAuth("", opts.User, opts.Passwd, host)
	} else if strings.Contains(options, "LOGIN") {
		// Patch for AUTH LOGIN
		auth = LoginAuth(opts.User, opts.Passwd)
	} else if strings.Contains(options, "NTLM") {
		auth = NtlmAuth(opts.User, opts.Passwd)
	}

I'm new here, but I'll try to help. Is it one of the above types if not that might be the issue.

I am using PLAIN authentication and the server is set to reject connections which don't use starttls. If I change the protocol and app.ini to "smtp" then the server refuses connection. "smtp+starttls" successfully contacts the server but the password isn't working.

I notice your using PASSWRD,
Make sure your using PASSWD instead.

That worked perfectly... been staring at my computer too long today I think. Thanks.

Worked perfectly... been staring at my pc too long I think. Thanks.


A cached feature provider for fast feature checks

A cached version for feature_dism.rb which enables lightening fast checks for installed windows packages. Very useful when you need to check many packages. As noted in some previous commits to feature_dism this process can be rather slow.
The only draw back to caching is that chef will be unaware of packages installed by other external processes during the converge which is why I have added this a separate provider for now.
Since the feature dsl checks node['windows']['feature_provider'] for the provider name the cached provider can be enabled via an attribute.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@matthew-churcher

Please sign the CLA here.

Same as https://github.com/chef-cookbooks/windows/pull/333. Just trying to the CLA signing recognised. Also just saw @mwrock comments on that pull so will give that some thought.

Since this is a dup and there are more comments on the initial PR, I'm going to close this.


Send Email

Hi all,
I will be grateful for your help.
I can't send email, I get the error:
WARNING: The command 'Send-MailMessage' is obsolete. This cmdlet does not guarantee secure connections to SMTP servers. While there is no immediate replacement available in PowerShell, we recommend you do not use Send-MailMessage at this time. See https://aka.ms/SendMailMessage for more information.

Hi @eugen257,
I believe you now have to use the MailMessage object in PowerShell to send emails: https://stackoverflow.com/a/61168324


BZ 1318344 - BPMN Process with swimlane created from business-central as been distorted while opening in JBDS

@jlindop  master PR - running test enough. thanks

+1


Training LapIRN with Dice loss

Hi,
Thank you for making the code public. I'm trying to augment LapIRN to incorporate anatomical information as well.
However, adding this loss seems to break the training process and the dice loss quickly goes up to 1.
Have you tried incorporating dice loss into any of your networks?

Hi @rohitrango,
I did! and it works with the data from the L2R challenge.
Here is the dice loss I tried:
nnUnet softdice: https://github.com/MIC-DKFZ/nnUNet/blob/nnunetv1/nnunet/training/loss_functions/dice_loss.py

do you have the code for this?

loss_dice = SoftDiceLoss()
Y_label = Y_label.cuda().float()
X_Y_label = transform(X_label, F_X_Y.permute(0, 2, 3, 4, 1), grid_unit_full)
dice_loss = loss_dice(X_Y_label, Y_label)
Something like this.

Okay. I have a similar training loss script as well. Maybe I'll try reducing the learning rate or play around with the hyperparameters.
Thanks for the help.

Okay. It looks like an issue with the Pytorch version used to train the model (I downgraded to Pytorch1.9 and it works). This seems to be a common occurance in the codebases, but no one seems to know exactly what has changed between these versions.


GLX underlinking

While building MESS, with GLX available, I get link-time errors about glX* functions.
Here is the build log.
If bgfx can be built without GL(X) support, then it is up to MESS to pass the corresponding flags at build time, but I did not find any flag in bgfx for that. Thus it seems to be a basic underlinking issue.
Although I would love to build bgfx (and MESS) without GLX, it would be nice to at least be able to build it with. :-)
Thanks,

Can you build existing bgfx examples on Linux? Do they link for you? I don't maintain MESS build scripts, so you should make sure they are correct.

Exact same failure with bare bgfx
bgfx-build.txt
I do use -Wl,--as-needed specifically to spot underlinking issues.

See CI results: https://travis-ci.org/bkaradzic/bgfx/jobs/187133753

Does it use -Wl,--as-needed? Underlinking is an issue really easy to miss.
But it’s easy to fix too: if you use symbols from a lib, link to it.

I just added it to test, and it still works properly:
...
debugdraw.cpp
g++    -MMD -MP -MP -DBX_CONFIG_ENABLE_MSVC_LEVEL4_WARNINGS=1 -D__STDC_LIMIT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_CONSTANT_MACROS -I../../../../bx/include -I../../../include -I../../../3rdparty -I../../../examples/common   -Wall -Wextra -fomit-frame-pointer -g -O3 -m64 -fno-rtti -fno-exceptions -Wshadow -mfpmath=sse -msse2 -Wunused-value -Wundef -m64 -std=c++11  -o "../../linux64_gcc/obj/x64/Release/example-29-debugdraw/examples/29-debugdraw/debugdraw.o" -c "../../../examples/29-debugdraw/debugdraw.cpp"
Linking example-29-debugdraw
g++ -o ../../linux64_gcc/bin/example-29-debugdrawRelease ../../linux64_gcc/obj/x64/Release/example-29-debugdraw/examples/29-debugdraw/debugdraw.o     -L../../../3rdparty/lib/linux64_gcc -L. -L../../linux64_gcc/bin -m64 -Wl,--gc-sections -Wl,--as-needed ../../linux64_gcc/bin/libbgfxRelease.a ../../linux64_gcc/bin/libexample-commonRelease.a -lrt -ldl -lX11 -lGL -lpthread

This build is with flag:
https://travis-ci.org/bkaradzic/bgfx/jobs/187528348

The point of this flag is to have it system-wide. I guess bgfx links to something A that links to something B that links to GL, thus if A is built with -Wl,--as-needed, it won’t link to GL, so bgfx cannot get GL symbols from it.
It’s why I said it’s easy to miss it.
I’d love to have a no-GLX build but at least fixing the underlinking with a simple -lGL would be nice for now. :-)

bgfx dynamically loads GL, there is no linking with GL.
As for no-GLX build, suggest way how to do it... All examples of Wayland and Mir I saw were PITA to setup.

Obviously there is something wrong in your code, since it breaks at link time. libepoxy does that perfectly, and it builds.
Wayland does not need specific support (I think) in your code, just support EGL. It’ll work on X11, Wayland, even DRM/KMS.
And I’m not sure what you mean by “setup” here. Binding the display as a GL app?

Obviously there is something wrong in your code

You're wasting my time... I added your linker option to my config and there is no problem with default bgfx.


ani-cli: cleanup package

Description of changes
A little bit of cleanup also moved the makeBinPath into buildInputs so that mkShell knows the dependencies when used in inputsFrom
also added some optional dependencies that can be chosen with --argstr

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Also, please squash the commits.

Again, please squash the commits, thanks.

Closed by #248966.


Provide the access to processTouch

Will allow for the user to set their own touch listener with its own behaviour, while also maintaining the original functionality and not just overriding the original touch listener

Bump


Implement ellipsis for long label cutoff for the adjacency matrix

This prototype is a solution for how to handle the label cutoff problem with SVG text using SVG Foreign Objects.
This is a picture with the foreign objects for the rows

currently working on foreign objects for columns

Finished the row and column labels for the matrix.
Les Mis

Flight data

Looking good. I have several suggestions about how to improve the code, but the visuals are very nice.
Before merging:

[ ]  file an issue to hoist some of the DOM creation to the Vue layer
[x]  let's consider rebasing the commit stream to improve the commit messages and/or restructure some of the changes

I see the rebasing was done (nice work); @mozartfish, could you please file the issue I requested and link it here?
Nice job on this PR!


Align radio

How can I align radio button to the right side of its children elements

<Radio style={{ flexDirection: 'row-reverse' }}>Place your text</Radio>

Snack


Decouple Maven publishing from component

This PR aims to reduce the coupling between the publication and the component used to derive data (dependencies and artifacts) for the publication. The DefaultMavenPublication is a large and complex class. This PR separates out all code that interacts with the component into a separate static nested ComponentParser class, allowing the logic to be reasoned about separately. This is necessary in order to publish different types of components than a SoftwareComponentInternal.
This PR also simplifies the MavenPublicationInternal and MavenPomInternal APIs by consolidating all dependency fields into a new MavenPomDependencies class to be exposed by MavenPomInternal. This helps us move towards a state where a maven pom does not need a backreference to the publication which creates it -- thus allowing us to support different types of publications which can publish multiple POMs. This is also more similar with how the Ivy publication handles dependencies
A similar PR for DefaultIvyPublication is in the works, but is currently blocked by outstanding complications related to lazy containers

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

There are new TODOs present in this change. Should any be removed?

No. I put these here on purpose.

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test apt

I've triggered the following builds for you:

AllPerformanceTestsReadyForRelease build

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.