Support Inner Shadow

Support Inner Shadow
@adamglin0

After reviewing your code, I created a similar inner shadow effect
I tested it on Android, but I haven't had the chance to test it on iOS and desktop yet

commit

feat: inner shadow

Thank you for your PR, I will review it as soon as possible!!!

I'll start working on #24 once this is merged, to avoid conflicts.

it works in IOS and Desktop(jvm)

@adamglin0 I have pushed two new commits

@adamglin0 I understood the comments provided and modified the code to use androidx.compose.ui.graphics.BlendMode

#22


fix:  Remove automatically suggested document for document that is already has review request (fixes #3211)

Added check that if there is already review request for the document in question, ignore the automatic suggestion for that document. Fixes #3211.

Robert Sparks writes:

@rjsparks commented on this pull request.
In ietf/review/utils.py:

@@ -588,10 +588,12 @@ def blocks(existing, request):
and existing.reviewassignment_set.filter(state_id__in=("assigned", "accepted")).exists()
and (not existing.requested_rev or existing.requested_rev == request.doc.rev))
request_closed = existing.state_id not in ('requested','assigned')

   # Is there a review request for this document already in system

   requested = existing.state_id in ('requested')

Would it get the same result in the cases you are testing if you added and
existing.type_id == request.type_id here, or is it specifically existing, say,
early assignments that cause problems when LC is requested?

Both can happen I think. I.e., I might have early review request that
got rejeced, but then the document went to LC, and got autosuggested,
thus there is old rejected early review that is open as a request, and
there is new LC request. I do not really care about the review type, I
only care that the review gets done at some point...
Same thing with LC review that got rejected and then it might already
be in the telechat.
@.***

I want to add some tests to this before merging, but it will be week after next.

Robert Sparks writes:

I want to add some tests to this before merging, but it will be week
after next.

Thats ok, as the secretaries are already aware of this bug, and will
most likely still catch those cases before doing duplicate
assignments, so few weeks delay is not an issue.
@.***

I am very sorry that I haven't gotten the remaining tests in and have this into main already. It will be my initial sprint task if it's not done before then.

@kivinen - Please review the additional condition I added (and tested).


Give html, body and first sc-include 100% height

I think this isn't very nice

Giving html, body and first sc-include 100% height would fix this issue.
WDYT @tomalec, @warpech and @PatrykSzwer

Certainly, we shouldn't make Underwear.css aware of starcounter-include. That would violate the separation of concerns.
On the other hand, I can't think of anything wrong with setting height: 100%; box-sizing: border-box on html and body in Underwear.css.
@Starcounter/webplatformteam can anyone think of a reason to NOT do that?
Then it should be easier to avoid this problem from a composition of the starcounter-include.


What are the methods in Vedo to clean a pointcloud data or to remove outlier removal....

I have a noisy pointcloud data and i want to to clean this pointcloud for surface Reconstruction......
What are the methods available in vedo to clean or outlier removal of pointcloud.
If possible please provide examples....
Thanks
PointClouds.zip

The pointcloud in attachments  are colored point cloud captured from realsense D435.

vedo does not support data arrays for PCD at present and format has to be ascii not binary. So I cannot read pc_2.
You can either use delauney triangulation or volumetric surface reconstruction:
from vedo import *
pts = Points("data/point_1.pcd").clean(0.0075).pointSize(4)
surf1 = delaunay2D(pts, mode='fit').clean().lighting('default')
surf2= recoSurface(pts, radius=0.03, holeFilling=0, pad=0)
show(pts, surf1, surf2, N=3, axes=1)

Thanks @marcomusy
In these pointcloud there is color information also. how I can apply the color information to these surface like this:

if you can provide a PCD file with associated pointdata in plain ascii format (not binary) I could easily implement it.

Hi @marcomusy
I have attached a ASCII format PCD file. Can you please help about how to add color information to after reconstruction.


Support configuration

https://github.com/catppuccin/vscode-icons#customization

Closed by https://github.com/uncenter/ctp-github-file-explorer-icons/commit/c99c5187abdd09db18dc88a8a877e1b587f62d38 and https://github.com/uncenter/ctp-github-file-explorer-icons/commit/28d9c11aae18a967987c1480f8045cf8e4987f2a.


Tutorials Pages

Number of Volunteers: 2
From Tutorial Chairs: The Tutorial pages will need some extra links to the tutorial material:
Links to Pdf slides for those tutorials which are not pre-recorded (T2, T6, T7, T8). We should have all the slides by tomorrow Wednesday). Let us know if you can also host the slides or if we should make a Google folder on our own.
Links to external tools like Doory (for brainstorming) for 1 or 2 tutorials (the tools are to be set up by the teachers themselves).
cc @raosudha89

@hao-fang  At this point, I think we should just post the slides as a link to a pdf. Not try to display the slides as a slideshow.

@hao-fang Can you forward me the email when they send this information?

@hao-fang  Sounds good.

Tutorial chair have given us the pdfs: https://drive.google.com/drive/folders/1b082SUDSqnkZTCggTJV3ymgMKav9-s6m

I am working on this.

Slides for T2, T7, T8 have been added using a 'Slides' link under main header.

However, T6 has multiple slide pdfs (under static/pdf/T6-Slides folder). I am not sure how to display multiple slide decks.
@hao-fang  Any suggestions as to which volunteer might be able to work on this?

@tlikhit  Do you have any ideas for how to do this?

However, T6 has multiple slide pdfs (under static/pdf/T6-Slides folder). I am not sure how to display multiple slide decks.

I took a stab at this and programmatically built an onclick event with the locations of the T6-files:

However, some browsers may prevent opening of multiple tabs at once (depending on pop up blocker or redirect settings). Another option is to allow downloading a zip file of the 6 slides.

@anaerobeth  Yes, opening multiple tabs automatically could be prevented in some browsers. And it could be confusing too since it is slightly uncommon. The option of downloading a zip file sounds good to me. Can you do that? Thanks

@anaerobeth  Thank you so much for fixing this. I have merged your PRs.


Invalid pixelmap or window parameter

Hi!
I'm using 0.9.6 and I always get the same error:
Disabling core dumps.
X Error of failed request:  BadDrawable (invalid Pixmap or Window parameter)
Major opcode of failed request:  149 ()
Minor opcode of failed request:  4
Resource id in failed request:  0x3e00041
Serial number of failed request:  361
Current serial number in output stream:  367
Signal 11 caught.
Segmentation fault (core dumped)
I don't know what caused this problem.

Please provide information about your OS, GPU and GPU drivers.

Ubuntu 18.04 and I have an Nvidia GeForce MX250 with 440.26 driver installed

Hm... What have you tried already to fix/investigate this?

I dodn't trie anything, because I'm not an expert in this and I don't wanted to mess up something.

I'm afraid I can't help you further then because I'm no expert either and everything I can do at this point (like googling the error) you can do as well.

did you succeed to find a solution for this issue?, I have the same problem.
Ubuntu 18.04, GeForce GTX 1050 Ti, driver 440.
I think it might be drivers issue

I switched to an older version and it worked. Try to change drivers in: Software & updates/Additional Drivers. Might gonna work

Which version did u switch to?  .. which drivers did you change? of nvidia? the problem i have 440 drivers and i had several problems getting ubuntu to work with nvidia gtx 1050 with other versions of drivers like 435

confirmed same error w/ NVidia 440 driver
Ubuntu 18.04
GTX150
Not sure if my GTX150 is going to cut it on my laptop anyway. Downgrading is challenging b/c of all the other components.

nvidia-driver-390 is working with 0.9.6 I think, but I am not sure if it will work with Ubuntu 18.04 as I always had logging issues because of the older version of the drivers, I was always stuck after the window login.

@rubentfd Which version worked for you?

0.9.3

0.9.3
:'D
We already know that, the question was which nvidia driver..

Which version did u switch to? .. which drivers did you change? of nvidia? the problem i have 440 drivers and i had several problems getting ubuntu to work with nvidia gtx 1050 with other versions of drivers like 435

Oh right sorry. Currently I am using 440.48.02

This issue is still persistent with newest versions of Carla and NVIDIA

Same issue with Ubuntu 18, Driver Version: 450.51.06    CUDA Version: 11.0
It works with opengl though!

Same issue with Ubuntu 18, Driver Version: 450.51.06 CUDA Version: 11.0
It works with opengl though!

Which version of OpenGL? And what should I do after I install OpenGL? Thanks.

https://carla.readthedocs.io/en/latest/adv_rendering_options/

I am getting something like this:
X Error of failed request:  BadDrawable (invalid Pixmap or Window parameter)
Major opcode of failed request:  130 (MIT-SHM)
Minor opcode of failed request:  3 (X_ShmPutImage)
Resource id in failed request:  0x400000f
Serial number of failed request:  1863
Current serial number in output stream:  1864
Ubuntu 18.04, Integrated Intel graphics. Any solution how to fix this?

I am getting the same error like this:
INTEL-MESA: warning: Performance support disabled, consider sysctl dev.i915.perf_stream_paranoid=0
X Error of failed request:  BadDrawable (invalid Pixmap or Window parameter)
Major opcode of failed request:  149 ()
Minor opcode of failed request:  4
Resource id in failed request:  0x3000041
Serial number of failed request:  370
Current serial number in output stream:  381
terminating with uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
Signal 6 caught.
Segmentation fault (core dumped)
I am using Ubuntu 20.04 dual boot laptop with Nvidia MX230 GPU[2GB] ,driver version 460.39 and RAM 20 GB.I faced the same problem with Ubuntu 18.04 before upgrading to Ubuntu 20.04.I tried Graphics Rendering previously but to no use.
Please suggest a fix,I am facing this issue since December 2020

same issue,
4.26.1-0+++UE4+Release-4.26 522 0
Disabling core dumps.
X Error of failed request:  BadDrawable (invalid Pixmap or Window parameter)
Major opcode of failed request:  149 ()
Minor opcode of failed request:  4
Resource id in failed request:  0x680003e
Serial number of failed request:  399
Current serial number in output stream:  409
terminating with uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
Signal 6 caught.
Segmentation fault (core dumped)
issue related with nvidia driver(my env Kubuntu 20.10 cuda 10.2, driver ver 460 installed through by apt,  Geforce RTX 2060 Super),

Same issue:
4.24.3-0+++UE4+Release-4.24 518 0
Disabling core dumps.
INTEL-MESA: warning: Performance support disabled, consider sysctl dev.i915.perf_stream_paranoid=0

X Error of failed request:  BadDrawable (invalid Pixmap or Window parameter)
  Major opcode of failed request:  149 ()
  Minor opcode of failed request:  4
  Resource id in failed request:  0x2e00041
  Serial number of failed request:  398
  Current serial number in output stream:  409
terminating with uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
Signal 6 caught.
Segmentation fault (core dumped)

Doesn't seem to be stale at all.

Using -opengl option worked for me as was mentioned in similar thread

Same issue with 0.9.12 (Driver Version: 460.91.03, CUDA Version: 11.2), seems like -opengl is no longer supported now. I'm not even able to run through the Docker container in 0.9.12.

Same issue with 0.9.12 (Driver Version: 460.91.03, CUDA Version: 11.2), seems like -opengl is no longer supported now. I'm not even able to run through the Docker container in 0.9.12.

I'm in the same boat. I've been avoiding this by running CARLA with opengl. Now that opengl support is gone I am stuck. I've tried to remedy this with multiple different driver versions with no success.

@jbeck9 I actually got this working by switching to the latest Nvidia driver 470 and running Carla with the -RenderOffScreen flag. This directly worked outside the Docker environment itself.

@jbeck9 I actually got this working by switching to the latest Nvidia driver 470 and running Carla with the -RenderOffScreen flag. This directly worked outside the Docker environment itself.

Thank you for the quick reply. It turns out the root problem for me was that Vulkan was using Intel graphics by default instead of my NVIDIA GPU. No idea why.
For anyone who is getting errors preceded by:
INTEL-MESA: <Error or warning>
Even though you have an NVIDIA GPU that is active. Run this:
export VK_ICD_FILENAMES="/usr/share/vulkan/icd.d/nvidia_icd.json"
Carla should then run normally with Vulkan.
./CarlaUE4.sh

The issue still persist.
./CarlaUE4.sh
4.26.2-0+++UE4+Release-4.26 522 0
Disabling core dumps.
X Error of failed request:  BadMatch (invalid parameter attributes)
  Major opcode of failed request:  149 ()
  Minor opcode of failed request:  4
  Serial number of failed request:  299
  Current serial number in output stream:  309
terminating with uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument
Signal 6 caught.
Segmentation fault (core dumped)

Carla 0.9.13
Ubuntu 20.04
Nvidia GeForce RTX 2060
nvidia-driver-510
./CarlaUE4.sh -opengl

Warning:  OpenGL is no longer supported for desktop platforms. Vulkan will be used instead.
and crashes

export VK_ICD_FILENAMES="/usr/share/vulkan/icd.d/nvidia_icd.json"  && ./CarlaUE4.sh

fixes the issue

Same issue on Ubuntu 22.04 LTS and AMD Radeon RX 6800 XT card.


[HTTPRequestStateMachine] Allow channelReadComplete at any time

Fixes #449.

Nightly fails because of sendable requirements.


Create zh_cn.json

I forgot to mention I added Stair and Slab variants. Would you be able to add those when you are available? I will do another small release once they are ready. Appreciate the translations!

I forgot to mention I added Stair and Slab variants. Would you be able to add those when you are available? I will do another small release once they are ready. Appreciate the translations!

I have updated the translation.

Thank you!


fix: add nil check

Prevent shim crash during createContainer.
Additionally, clean up device structure during unplug.
@zvonkok PTAL

Just verified works as expected, just two nits.


Tests for foundational language server metrics

Adds tests for foundational language server metrics and fixes a few bugs:

If an edit comes in and cancels the slow path, we now calculate latency using the timer from the canceled slow path.
When a slow path gets canceled, we properly emit last_diagnostic_latency and sorbet.mergedEdits metrics. (The conditional was flipped before.)
When a slow path gets canceled, we do not emit a latency metric for it.

Includes a new test-only LSP method for getting counters out of the language server.
Motivation

These metrics are hard to get right because they are asynchronous and have a few notable corner cases. It's easy to introduce regressions in them. The tests should break if we introduce bugs in these timers in the future.
Test plan

See included automated tests.

I replicated the CI failure locally with the sanitizers on, so I'll debug that tomorrow.

Figured it out; I need to explicitly call canonicalize() on the CounterState. I had assumed that that method was called when clearing counter state.


Make "revert now" more obvious

Otherwise it's not obvious to people that they can revert.
Note: Only translated two languages, in other languages it will show up in English.

Ideally, a user click on the button "ShouldRevert", and it will let people mark it as "ShouldRevert" and then there is an additional button showing up, and asks "do you want to revert them directly?", if user continue to click on it, it will revert the revision. (and user shall be able to also decide that they can "always conduct revert when marking ShouldRevert" ).
Also, the revision needs to be eligible to revert - meaning:

there is no other subsequent revisions after the current revision-in-review
the revision and its proceeding revisions are not from the same user, see #57

I would love to let this PR go through, even though I think it will only be a temp optimization. We need follow up commits for aforementioned functions.

Nevermind, I found out that you are only adding a label, instead of showing the "should revert" all th time. This PR is good to go in.


Make SAI attrs mandatory for drop-on-miss tables

If a table references another table (ENI->VNET_ID) where VNET table drops on miss, the SAI attribute VNET_ID in the ENI will be mandatory on create.
Omitting this attribute will no longer be allowed.
Signed-off-by: Marian Pritsak<EMAIL_ADDRESS>

Requires https://github.com/opencomputeproject/SAI/pull/1616 and also ported to dash branch.

@marian-pritsak is this obsolete?


Signature widget changes

-Move the 'Reset' and 'Cancel' buttons to the right side of the screen, so that the 'Save and Close' button stands alone
-Automatically navigate forward after signature capture ONLY if: a) a signature was actually captured (e.g. they pressed 'Save and Close' as opposed to cancel), b) the signature question is not on the  last screen, and c) the signature widget isn't in a question list
(Having now done this and realized all of the conditions I needed to add in order to have this make sense, I'm open to being told that it's not a good idea to have the widget behave differently under different conditions. But wanted to share it and see what others think)
-Remove some of the more embarrassing comments left in the SignatureWidget code from when I worked on it as an intern...

We have some of this logic for the "quick" multi select appearance type which also auto advances when a selection is made. You can see that here https://github.com/dimagi/commcare-odk/blob/master/app/src/org/commcare/views/widgets/GridWidget.java#L145. Might make sense to unify this logic IE have both use the AdvanceToNextListener. It looks like the quick select widget will auto advance even if we're on a question list which seems not good. However looking at this and the "next" function it looks like we already check for end of form so your check in the control block might be redundant https://github.com/dimagi/commcare-odk/blob/master/app/src/org/commcare/activities/FormEntryActivity.java#L946

Also I'm pro keeping funny comment artifacts with historical value intact, don't be a revisionist

I find it useful to see before and after screenshots when making ui changes. That was we don't need to spin up a build of this PR to understand what the changes look like.
Screenshots are super easy to pull if you have a device open. This command stores the screenshot as screen.png in the current dir:
adb shell screencap -p /sdcard/screen.png && adb pull /sdcard/screen.png && adb shell rm /sdcard/screen.png

@phillipm
Before:

After:


Spacing between lines and characters

I want to create a space between the lines and between the written characters, how can I do this?

Try using ExtraAscent and ExtraDescent for the spacing between lines.
I'm not aware of a way to get extra space between characters.  What is the reason for adding space?

thanks

Reopen if you still need more help with the space between characters, I just need more context for what you are trying to accomplish.


[FR] Add support for PEP 643 - metadata for sdists

What's the problem this feature will solve?
Some time ago we approved PEP 643, and I intended to implement it... eventually, but it's been a long time and this is actually a very powerful and useful tool for setuptools in particular to implement.
If done correctly, we should be able to silently and in the background start providing reliable metadata for source distributions to anyone building with new versions of setuptools.
Describe the solution you'd like
On the install/wheel building side, we need to be aware that if we are consuming an sdist that already has version 2.2+ metadata, we take all non-dynamic values from the sdist, and not from setup.py. That way if setup.py and the sdist metadata disagree about a given non-dynamic value, the sdist wins (and we are in compliance with the PEP).
For generating PEP 643 metadata (the bigger and more important job, I imagine), there are several implementations (and they can be progressively and backwards-compatibly implemented):

The most naïve implementation: mark everything as Dynamic and bump the version to 2.2. (This is not a very useful implenentation, but a valid one).
Mark everything as Dynamic if it comes from setup.py, but not if it comes from setup.cfg or pyproject.toml (when #2671 is resolved).
Add a mechanism to explicitly mark a value as static or dynamic in setup.py — a wrapper like setuptools.literal_value, for when someone needs to generate something from code, but it's deterministic for any given release. We don't have to worry too much about this being inaccurate if we also ensure that when setup.py and the sdist metadata disagree, the final value from the wheel / installed version is taken from the metadata.
Add heuristics to parse the AST of setup.py and determine when setup() is passed arguments that we can be sure are deterministic. Literal values, things not assigned in if blocks, that sort of thing. I suspect that even a small amount of this sort of thing will allow us to unlock a lot of currently deterministic metadata that we'll otherwise have to mark as dynamic, e.g. anything with a setup.py that looks like this:

setup(
    name="foo",
    version="0.0.1",
    install_requires = ["attrs"],
)

Alternative Solutions
No response
Additional context
See the discussion on the discourse.
Code of Conduct

[X] I agree to follow the PSF Code of Conduct

This will ultimately need https://github.com/pypa/warehouse/issues/9660 to be merged first, otherwise the sdists won't be uploadable to PyPI.

FWIW, metadata 2.2 has been supported in warehouse: https://github.com/pypi/warehouse/pull/13606

The outcome of the discussion in https://github.com/pypa/setuptools/issues/4629 seems to indicate the following is a viable approach:

Document that plugins are not allowed to dynamically generate inconsistent values for name and version metadata (they may generate these, but the values need to be exactly the same across all builds from both source tree or sdist)
Document that plugins are not allowed to modify values provided statically as metadata in pyproject.toml
When metadata is provided via pyproject.toml, setuptools will attempt to use Dynamic only when an equivalent dynamic is present in pyproject.toml.
When metadata is provided via setup.py, setuptools will add Dynamic for the metadata fields to ensure other tools do not accidentally trust them as static (just in case).

Originally posted in https://github.com/pypa/setuptools/issues/4629#issuecomment-2331264348

https://github.com/pypa/setuptools/pull/4698 is an attempt to implement PEP 643. I am trying a slightly unorthodox approach by marking the static values coming from setup.cfg/pyproject.toml instead of the dynamic ones, in an attempt to identify when plugins/customisations overwrite them. I say the approach is unorthodox because I introduce sub-classes of built-in data types. That is a bit awkward, but it makes sure the change is backwards compatible with the existing codebase in both setuptools and distutils.
If anyone is interested in having a look on the approach, please let me know your ideas.


Integrate MDS client to ISM

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

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

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

Codecov Report

Merging #711 (5d00b71) into 2.x (216b9a8) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 5d00b71 differs from pull request most recent head 15907de. Consider uploading reports for the commit 15907de to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##              2.x     #711   +/-   ##
=======================================
  Coverage   58.75%   58.75%
=======================================
  Files         318      318
  Lines       10544    10544
  Branches     2012     2012
=======================================
  Hits         6195     6195
  Misses       3833     3833
  Partials      516      516

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


MGDCTRS-78: migrates to java operator sdk v2

This migrates to the new Java Operator SDK, version 2.0.2, and also the
Quarkus Operator SDK, to version 3.0.2. For the controller and its
reconcile methods, it's mostly name changes. For the EventSources there
has been a moderate change to the API. For e.g., Events are now
binded to a single ResourceID (before a resource selector). The new
API suggests the use of Informers for EventSources, which would make
for a even simpler implementation, but Informers have internal cache
which could represent a high memory usage in scenarios with many
connectors deployed, that's why we are still using our own Watch-based
EventSources. Existing EventSources had to be changed to trigger
multiple events because of the new 1-1 resource-event relation.

It is possible to try to move our WatcherEventSource to the Java Operator SDK itself, as it is basically a new type of EventSource extra to the SDK, which already offers a few other types.
More unit and integration tests should be created to test the EventSources.

LGTM, only a small question


Design system for contributors

I like OnceUI's architecture and have started working with it on a new NextJS project. Some changes came up and I want to contribute them to open source. But my VS Code uses a formatter, and this formatter (I tried Prettier and BiomeJS) reformats your code. Please note which formatter and options to it you use for development.
And thanks for a good product! You have indeed done a great job.

it's a pleasure to hear you're using Once UI and enjoying it so far @syrokomskyi! we haven't used anything for code formatting so far, but probably it's about time to settle with a standard for our project. you can get close to our formatting style with the following prettier settings:
{
  "arrowParens": "always",
  "bracketSameLine": true,
  "bracketSpacing": false,
  "semi": true,
  "experimentalTernaries": false,
  "singleQuote": false,
  "jsxSingleQuote": false,
  "quoteProps": "as-needed",
  "trailingComma": "all",
  "singleAttributePerLine": true,
  "htmlWhitespaceSensitivity": "css",
  "vueIndentScriptAndStyle": false,
  "proseWrap": "preserve",
  "insertPragma": false,
  "printWidth": 80,
  "requirePragma": false,
  "tabWidth": 4,
  "useTabs": true,
  "embeddedLanguageFormatting": "auto"
}

thank you for you contribution in advance, and feel free to propose changes to these settings! the best way to keep in touch with us is via Discord, but of course we're tracking and answering on all channels :)

When you choose, look at biomejs: for webdev it is better than Prettier.

thank you for the suggestion, will take a closer look!


Azure Front Door Service Config not found

Hi,
After successfully completing the Powershell deployment guide I get a "Page not found" error when adding the Authors tab to the Team. I have given it over 12 hours and I am still getting the same error. When clicking any of the available links on the error page I get a "azure.microsoft.com refused to connect" error.
What am I doing wrong? :(
Regards

Hi @stfenix
We have noted your issue and will get back to you.

Hi @stfenix,
Please check you manifest file if you've updated the configurationURL and resource correctly, if possible request you to share the manifest file of author bot for us to troubleshoot.

{
"$schema": "https://developer.microsoft.com/en-us/json-schemas/teams/v1.11/MicrosoftTeams.schema.json",
"manifestVersion": "1.11",
"version": "5.0.0",
"id": "1c07cd26-a088-4db8-8928-ace382fa219f",
"packageName": "com.microsoft.teams.companycommunicator.authors",
"developer": {
"name": "Imperial Logistics",
"websiteUrl": "https://www.imperiallogistics.com/",
"privacyUrl": "https://www.imperiallogistics.com/terms-conditions.php",
"termsOfUseUrl": "https://www.imperiallogistics.com/terms-conditions.php"
},
"icons": {
"color": "color.png",
"outline": "outline.png"
},
"name": {
"short": "Imperial Comms (Authors)",
"full": "Imperial Communicator (Authors)"
},
"description": {
"short": "Broadcast messages to multiple teams and people in one go",
"full": "Broadcasts messages to multiple teams and individuals through channel posts and chat messages"
},
"accentColor": "#64A2CC",
"configurableTabs": [
{
"configurationUrl": "https://impcommsbase.azurefd.net/configtab",
"canUpdateConfiguration": true,
"scopes": [
"team"
]
}
],
"bots": [
{
"botId": "c6d59fc4-8650-4b57-b952-27309a68ed59",
"scopes": [
"team"
],
"supportsFiles": true,
"isNotificationOnly": true
}
],
"permissions": [
"identity",
"messageTeamMembers"
],
"validDomains": [
"impcommsbase.azurefd.net"
],
"webApplicationInfo": {
"id": "c6d59fc4-8650-4b57-b952-27309a68ed59",
"resource": "api://impcommsbase.azurefd.net"
}
}

Hi @stfenix ,
The manifest seems valid. Could you please share the below details,

Go to the Resource group created during the deployment
Click on Overview

Click on the Deployments and click on the Template deployment link and share the entire resource creation steps during the deployment for further troubleshooting.

Hi @v-royavinash

All deployments completed succsesfully

Hi @stfenix ,
If you still facing the issue, please share with us your MS Teams Id so that we can connect over a call to troubleshoot it.

Hi @stfenix ,
Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up.

Hi
Thanks, the issue has been resolved. we manually deployed via the GUI
without any issues.
On Wed, May 4, 2022 at 10:36 AM Sai Venkat @.***> wrote:

Hi @stfenix https://github.com/stfenix ,
Please let us know if you are still facing this issue and require
assistance and we will be happy to discuss. If no further action is needed
at moment and no response is received, we will close out the issue in 3
business days. You may also always open a new issue or respond here if
something does come back up.
—
Reply to this email directly, view it on GitHub
https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/issues/691#issuecomment-1117058230,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK7Z6F6NZSFUIKSNR7CQD3VIIZHJANCNFSM5SYOQ5YQ
.
You are receiving this because you were mentioned.Message ID:
<OfficeDev/microsoft-teams-apps-company-communicator/issues/691/1117058230
@github.com>


getPublicStage should use the stages api directly as opening ID is no longer needed

reverted back to using opening id, closing


Shorten the text under the We found n records

The current text increases the white space

Test shortened; others may wish to wordsmith. But reduction on space is achieved

see test3

Test shortened; others may wish to wordsmith. But reduction on space is achieved

see test3

@AlOneill please cast your eyes over the text but no extra words please

@AlOneill please cast your eyes over the text but no extra words please

'Select' is more inclusive than 'click'.
Listed by birth date. Select a heading to sort by that column: select again to reverse the order. See individual details by selecting the View button.
Same number of words (26). Tone more friendly.

'Select' is more inclusive than 'click'.
Listed by birth date. Select a heading to sort by that column: select again to reverse the order. See individual details by selecting the View button.
Same number of words (26). Tone more friendly.

@AlOneill I have always used select when there are alternatives. Here there is just the one option.

@AlOneill I have always used select when there are alternatives. Here there is just the one option.

Ah, select is a versatile and hard-working word! The thing is that keyboard-only users, both sighted and unsighted, cannot click: they select (follow) a link by using Enter or whatever key combo works in their screen-reader software. Maybe not all that important, but I can imagine that being asked repeatedly to click must be irritating — one more straw on the camel's back. Cannot think of a parallel situation just now …

Ah, select is a versatile and hard-working word! The thing is that keyboard-only users, both sighted and unsighted, cannot click: they select (follow) a link by using Enter or whatever key combo works in their screen-reader software. Maybe not all that important, but I can imagine that being asked repeatedly to click must be irritating — one more straw on the camel's back. Cannot think of a parallel situation just now …

Wonderful explanation. TY

Wonderful explanation. TY

@Captainkirkdawson Removing the class hard--bottom from the containing section restores overall balance and adds only a little to the height. Is this acceptable?

Personally I would say that making that paragraph larger than the You searched for adds addition empty space to page which is what we were trying to eliminate

It adds a few pixels only. Re-writing the text has made a big improvement. Visual balance for the "We found …" section is also important — imbalance is uncomfortable for some (even many?).
Let the Scrum decide!

Its always been out of wack; less so now

Scrum agreed that the few pixels of grey improve legibility, so should be implemented.


Improve docs by enabling jumping to the GitHub definition of APIs

Jazzy enables to display a Show on GitHub in the section of each method/property. This makes it much easier to browse APIs implementation directly online but isn't currently enabled on the NIO docs.
Thank you! 🙂

Maybe --github-file-prefix is the option to use but I'm not 100% sure.

@adtrevor I tried --github-file-prefix with various combinations

--github-file-prefix "https://github.com/apple/swift-nio/tree/$version/"
--github-file-prefix "https://github.com/apple/swift-nio/tree/$version/$module/"
--github-file-prefix "https://github.com/apple/swift-nio/tree/$version/$module/Sources/"

but nothing worked :(. Maybe @jpsim can help us figure this out?

@weissi Hmmm... I had looked for it online and indeed I didn't find any answer either.
It's enabled in the docs of Vapor though (cc: @tanner0101 )

Oh yay! That’s great news, that means we can can it fixed somehow. @tanner0101 to the rescue 😁

--github-file-prefix is definitely the correct way to enable this, and I've filed a PR enabling this at https://github.com/apple/swift-nio/pull/977. There's a GIF demonstrating that it works as intended as well.

should be fixed by #977 , when we next release we'll see if everything worked.


Console commands runner

We need kind of swagger ui, but for console commands.
Scenario:

Select a command
Type arguments
Click to run

What's the goal of running console commands via UI?

The goal is to be as simple as possible. Just two/three click to run a command from the remote server connected via SSH is quite tasty to have


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/98151

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104314

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104489

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/104615

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/105801

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/118799

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/119326


chore(master): release 1.1.80

:robot: I have created a release beep boop
1.1.80 (2023-09-08)
Bug Fixes

deps: update dependency wayofdev/laravel-webhook-client to v1.1.70 (9267c23)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v1.1.80 :sunflower:


How to load partial parameters from subclass model in TF2?

Hi, all,
I have constructed a keras model with subclass mode in TensorFlow2.x, and trained it with a dataset named dataset1(which have 4000 classification, namely the vocabulary size is 4000), and save the checkpoint with tf.train.Checkpoint and tf.train.CheckpointManager. And now I want to finetune it with a new dataset named dataset2(which have 500 classification, the vocabulary size is 500), when directely restore the parameters from the saved checkpoint, it will fail because of the mismatch shape (ie, the embedding layer, the output layer of encoder and decoder etc).
So, how to restore partial parameter from the checkpoint model, or ignore the shape mismatch parameters when restore from the checkpoint. Here is a simplified example,
# custom design layer
class CustomLayer(tf.keras.layers.Layer):
    def __init__(self, unit1, unit2, **kwargs):
        super(CustomLayer, self).__init__(**kwargs)
        self.dense1 = tf.keras.layers.Dense(unit1, activation='relu')
        self.dense2 = tf.keras.layers.Dense(unit2, activation='softmax')

    def call(self, inputs):
        x = self.dense1(inputs)
        return self.dense2(x)

# keras model
class CustomModel(tf.keras.Model):
    def __init__(self, unit1, unit2):
        super(SourceModel, self).__init__()
        self.layer = tf.keras.layers.Dense(128, activation='relu')
        self.custom_layer = CustomLayer(64, 10)

    def call(self, inputs):
        x = self.layer(inputs)
        x = self.custom_layer(x)
        return x

# original model
    old_model = CustomModel(64, 10)
    optimizer = tf.keras.optimizers.Adam(1e-4, beta_1=0.9, beta_2=0.98, epsilon=1e-9)
    ckpt = tf.train.Checkpoint(step=tf.Variable(0), model=old_model, optimizer=optimizer)
    for sample in dataset1:
            training and save checkpoint ......

    # new model
    new_model = CustomModel(64, 5)
    **how to restore partial parameters from the checkpoint saved before and start a new train in the dastaset2?**

@yjiangling,
Restores a training checkpoint and this Checkpoint and any objects it depends on. This method is intended to be used to load checkpoints created by save(). For checkpoints created by write() use the read() method which does not expect the save_counter variable added by save().
Restore() either assigns values immediately if variables to restore have been created already, or defers restoration until the variables are created. Dependencies added after this call will be matched if they have a corresponding object in the checkpoint (the restore request will queue in any trackable object waiting for the expected dependency to be added)
https://www.tensorflow.org/api_docs/python/tf/train/Checkpoint
Also try to save the checkpoint of the pre-trained model using tf.train.Checkpoint and use Checkpoint.restore to load the checkpoint. Specify the variables or layers you want to restore. Thank you!

@yjiangling, Restores a training checkpoint and this Checkpoint and any objects it depends on. This method is intended to be used to load checkpoints created by save(). For checkpoints created by write() use the read() method which does not expect the save_counter variable added by save().
Restore() either assigns values immediately if variables to restore have been created already, or defers restoration until the variables are created. Dependencies added after this call will be matched if they have a corresponding object in the checkpoint (the restore request will queue in any trackable object waiting for the expected dependency to be added)
https://www.tensorflow.org/api_docs/python/tf/train/Checkpoint
Also try to save the checkpoint of the pre-trained model using tf.train.Checkpoint and use Checkpoint.restore to load the checkpoint. Specify the variables or layers you want to restore. Thank you!

Thank you. I did use save() to ca to create a checkpoint file and use restore() to restore the parameters. But for finetune/transfer training, because the vocabulary size have been changed, the shape of variables in some layers (such as embedding, final output layers, ...) have beem changed, it will give ValueError: Shapes (xxx,) and (yyy,) are incompatible. How to restore from pre_trained checkpoint and skip this variables? I tried this method:
checkpoint = tf.train.load_checkpoint(checkpoint_path)
var_names = checkpoint.get_variable_to_shape_map().keys()
tensor_dict = {}
for var_name in var_names:
	if 'ctc_branch' in var_name:
		continue
	if 'embedding' in var_name:
		continue
	if 'final_layer' in var_name:
		continue

The tensor_dict already remove the mismatched variables, but how to restore the dict to the new new checkpoint ckpt = tf.train.Checkpoint(step=tf.Variable(0), model=model, optimizer=optimizer)


window::open_events() function subscribes to close events

Is your issue REALLY a bug?

[X] My issue is indeed a bug!
[X] I am not crazy! I will not fill out this form just to ask a question or request a feature. Pinky promise.

Is there an existing issue for this?

[X] I have searched the existing issues.

Is this issue related to iced?

[X] My hardware is compatible and my graphics drivers are up-to-date.

What happened?
In the master branch in the /runtime/src/window.rs the open_events() code erroreously subscribes to the wrong events. Simple Copy/Paste error by the looks of it.
/// Subscribes to all [`Event::Closed`] occurrences in the running application.
pub fn open_events() -> Subscription<Id> {
    event::listen_with(|event, _status, id| {
        if let crate::core::Event::Window(Event::Closed) = event {
            Some(id)
        } else {
            None
        }
    })

What is the expected behavior?
Should subscribe to Open events
Version
master
Operating System
Linux
Do you have any log output?
No response

Ah, classic copy-paste! Fixing...


Reading fetched Image throws error ZeroLengthBlobNotPermitted

magick-wasm version
0.0.24
Description
When trying to load an image that was fetched via a URL the library throws ZeroLengthBlobNotPermitted. It seems to do this for all images I attempt to fetch. I'm unsure if this is actually an issue though or if it's simply not how you would go about implementing this code.
Steps to Reproduce
// https://github.com/lindsaykwardell/vite-elm-template/blob/main/src/assets/logo.png?raw=true
  const url = "http://localhost:5173/src/assets/logo.png";
  const res = await fetch(url);
  const blob = await res.blob()

  ImageMagick.read(blob, MagickFormat.Png, (image) => {
    const canvas = document.getElementById("canvas");
    image.writeToCanvas(canvas)

  })

Images
No response

The read method has no support for a Blob. You will need do do this instead:
const buffer = await blob.arrayBuffer();
ImageMagick.read(buffer , MagickFormat.Png, (image) => {

I tried arrayBuffer() as well but got ImproperImageHeader. Are there any other examples of loading an image via fetch?

When you search for fetch in the discussions you will find this: #112

Doing the following works, thank you for the help!
const res = await fetch(nurl);
const buffer = await res.arrayBuffer();
const data = new Uint8Array(buffer);

ImageMagick.read(data, (image) => { ... }

Whoops sorry I copied the wrong code... and good to hear that you fixed it.


running hug on pythonanywhere

I have a problem running hug on pythonanywhere.com - I can't figure out how to point wsgi onto __hug_wsgi__. This should be a quick one, but it seems I'm still novice web developer :)
pythonanywhere has "username_pythonanywhere_com_wsgi.py" file where you setup the bindings for the app - you need to bind callable to application variable:

# +++++++++++ CUSTOM WSGI +++++++++++
# If you have a WSGI file that you want to serve using PythonAnywhere, perhaps
# in your home directory under version control, then use something like this:
#
import sys

path = '/home/username/myapps/dayassistant'
if path not in sys.path:
    sys.path.append(path)

# and here I'm trying to somehow bind hug_wsgi
import hug
import serve
from hug import __hug_wsgi__ as application

In hug docs 'quickstart' I found howto run hug via uWSGI from bash:
uwsgi --http <IP_ADDRESS>:8000 --wsgi-file first_step_3.py --callable __hug_wsgi__
and tried to adjust it  but my approach doesn't seem to work as I get:
Error running WSGI application
ImportError: cannot import name '__hug_wsgi__'
File "/var/www/username_pythonanywhere_com_wsgi.py", line 73, in <module>
    from hug import __hug_wsgi__ as application

Further in the config file, example for Flask looks like:
# +++++++++++ FLASK +++++++++++
# Flask works like any other WSGI-compatible framework, we just need
# to import the application.  Often Flask apps are called "app" so we
# may need to rename it during the import:
#
#
#import sys
#
## The "/home/username" below specifies your home
## directory -- the rest should be the directory you uploaded your Flask
## code to underneath the home directory.  So if you just ran
## "git clone<EMAIL_ADDRESS>## ...or uploaded files to the directory "myproject", then you should
## specify "/home/username/myproject"
#path = '/home/usename/path/to/flask_app_directory'
#if path not in sys.path:
#    sys.path.append(path)
#
## After you uncomment the line below, the yellow triangle on the left
## side in our in-browser editor shows a warning saying:
##     'application' imported but unused.
## You can ignore this error. The line is necessary, and the variable
## is used externally.
#from main_flask_app_file import app as application
#
# NB -- many Flask guides suggest you use a file called run.py; that's
# not necessary on PythonAnywhere.  And you should make sure your code
# does *not* invoke the flask development server with app.run(), as it
# will prevent your wsgi file from working.

anyone can help getting it running?

I think you are pretty close.
My file looks like this:
import sys

path = '/path/to/app'
if path not in sys.path:
    sys.path.append(path)

from api import __hug_wsgi__ as application

Here, api is /path/to/app/api.py which contains my @hug.extend_api() function
I think you are either trying to get the __hug_wsgi__ from hug itself or your main file is also named hug which may cause some problems with your import statements (which I think are unnecessary).
Hope this helps!

Hey, thank!
That's what I was looking for, I kinda thought it might be connected to where 'last hug decorator' is used.
so, I have following layout for project:
myapps/dayassistant/api/routes
in this package I have modules for different route categories and @hug.extend_api() them in init. py
so that I thought I could do import api.routes and have it all nicely bound for me to use it further
But I guess if in myapps/dayassistant/serve.py I do:
from api.routes import *
and in pythonanywhere wsgi file do:
import serve
application = serve.hug_wsgi
This cannot work? If not, is there a way to keep my folder structure and make it work?

Your folder structure sounds fine.
If I understood correctly I think you just have to to do
import sys

path = '/path/to/init.py'
if path not in sys.path:
    sys.path.append(path)

from init import __hug_wsgi__ as application

No way to somehow import api.routes in base project folder and there bind hug_wsgi to application?
Just curious if I can somehow get binding for__hug_wsgi__ before code run

hug exposes a __hug_wsgi__ magic method on every API module automatically

If your API module lives in /path/to/my_api.py, make sure /path/to is in your sys.path (which is where python searches when it is importing stuff), then
import my_api
application = my_api.__hug_wsgi__

However, it sounds like you are trying to structure your code like a package: https://docs.python.org/3/tutorial/modules.html#packages
If you want to write import api.routes, your original sys.path code is correct, and you will want to make sure that an __init__.py file exists in the api and route folders. Then you can write
from api.routes import init
application = init.__hug_wsgi__

If you rename init.py to __init__.py, you can write
from api import routes
application = routes.__hug_wsgi__

Hope this clears things up.

Thanks for submitting this question, it will be very helpful for us to prioritize documentation in the future,
I think @cag and @Musilitar have done a great job of answering this question, so I'm going to go ahead and mark it as closed, if you have any further concerns feel free to ask them here or withing a new issue.
Thanks!
~Timothy


Unlisted Vimeo-Videos won't play

Current Behavior:
Vidstack Player doesn't play Videos from Youtube that are marked as "unlisted". It is stuck with the loading animation

Expected Behavior:
Player should also play "unlisted" Videos
Steps To Reproduce:
add the vidstack player and the official vimeo iframe embed to a page side by side

<media-player>
  <media-provider>
    <source src="https://player.vimeo.com/video/763694880?hash=e21fea6134"
        type="video/vimeo">
  </media-provider>
  <media-video-layout></media-video-layout>
  <media-audio-layout></media-audio-layout>
</media-player>

<div style="padding:56.25% 0 0 0;position:relative;">
    <iframe src="https://player.vimeo.com/video/763694880?h=e21fea6134&badge=0&autopause=0&player_id=0&app_id=58479"
            frameborder="0"
            allow="autoplay; fullscreen; picture-in-picture"
            style="position:absolute; top:0; left:0; width:100%; height:100%"
            title="fit4future Kids - Spiele mit dem Reifen">
    </iframe>
</div>

Anything Else?
seems like the player gets stuck when trying to load the embed-info (which is not available for unlisted videos)

the bug might be somewhere in here...
https://github.com/vidstack/player/blob/49bf0a71cd35ae7bfa8e9a058e345224b4693865/packages/vidstack/src/providers/vimeo/utils.ts#L14
https://github.com/vidstack/player/blob/49bf0a71cd35ae7bfa8e9a058e345224b4693865/packages/vidstack/src/providers/vimeo/provider.ts#L222

any update / progress on this issue ? @mihar-22  ?

@mihar-22 feel free to test the issue with this video
https://player.vimeo.com/video/763694880?h=e21fea6134
its unlisted in vimeo but viewable on any domain.

any update / progress on this issue ? @mihar-22 ?

:wave:

I've tried testing with test URL provided but it works for me?

@mihar-22
please take a look at this example on stackblitz.com
double checked the video id - its unlisted.

@mihar-22 thanks for the bugfix! everything is working fine now!

Hello,
What about URLs contains two IDs ?
https://vimeo.com/${idOne}/${idTwo}


Customizable Home Page

Added ability to set custom home page that is loaded whenever Helium starts.
Home page can be set within the Preferences menu, or with the hotkey ⌘,.
Addresses #148.

This seem to be missing the actual preference menu to set it, no?

I guess I'll make the Storyboard changes myself 😄

Thanks for building this! 🎉

Ah. Forgot to commit the .storyboard changes. 😅

Does not seem to work in version 2.0


ECS: SystemConfig::after(BoxedSystemSet) does not work

Bevy version
0.10.1
What you did
I arrange some of my systems at runtime and pass around their dependencies expressed as BoxedSystemSets. It's impossible to then use definitions like:
#[derive(SystemSet, Debug, Hash, PartialEq, Eq, Clone)]
pub(crate) enum MySet { One, Two, Three }

fn test_system(world: &mut World) { }

fn custom_add_systems(schedule: &mut Schedule, anchor: BoxedSystemSet) {
    schedule.add_system(test_system.after(anchor));
}

What went wrong
I think it's because there's no implementation of IntoSystemSet for BoxedSystemSet? I'm not sure
Additional information
The workaround that I think should work for now is to do
fn custom_add_systems(schedule: &mut Schedule, anchor: BoxedSystemSet) {
    schedule.add_system(test_system).configure_set(anchor.dyn_clone().before(test_system));
}

I think this is probably outdated and we have impl IntoSystemConfigs<()> for BoxedSystem<(), ()> in 0.12.1

Tentatively closing this since we have support for system set label configurations, systems as their own labels, and BoxedSystemSet no longer exists.


deploy netdata using the nodiscc.xsrv.monitoring_netdata ansible role

fixes https://gitlab.com/nodiscc/xsrv/-/issues/1
README.md: fix typo, whitespace

/cc @odyslam

@ilyam8 ok to merge?

@nodiscc sorry for the delay. @Ferroin and @maneamarius will take a look (I hope) soon.

As far as remember, it was decided that we will not offer the community automation solutions. We had a meeting. Why we talking about it again?

If the pr might help some future users why would we not merge it?

Going back to that discussion, the reasons for not doing it are roughly the following.
This way we sort of take responsibility for this code. We would be more comfortable giving links to the repositories of those who developed the relevant playbook/role.
This particular role is (from memory) well written, but its usefulness is extremely limited by the fact that it only works with
Debian 11. Since there are no organizations that use a single version of a single operating system, more generic roles will be preferred.
Also, Ansible is not the only automation tool.
I would prefer to see this role in Ansible Galaxy. And we will, of course, point out its existence in the documentation.

Thanks @nodiscc !

Hi @andrewm4894 @iigorkarpov

This way we sort of take responsibility for this code. We would be more comfortable giving links to the repositories of those who developed the relevant playbook/role.

I assumed (see https://gitlab.com/nodiscc/xsrv/-/issues/1) this repository was for listing community-supported projects, which implies netdata does not officially support/take responsibility for the content.
If you feel more comfortable only linking to external projects I suggest converting the repo to an "awesome"-style list (see https://gitea.com/gitea/awesome-gitea for example) which links to community projects or articles on netdata. I can move the article to a Github gist or similar then we can simply link to it from here (then we should also move the other ansible howto).
All things considered I think the "awesome list" format would be more suitable for this repo.

This particular role is (from memory) well written,

Thanks :ok_hand: I try to keep it clean

its usefulness is extremely limited by the fact that it only works with Debian 11. Since there are no organizations that use a single version of a single operating system, more generic roles will be preferred.

It should work with Ubuntu 18/20 with minor modifications, which would be maintainable for me.
But you are right that it will not support other distributions/RHEL family because it is out of scope for me. It has other limitations, it only supports the APT/.deb installation method, not all netdata settings are configurable through it... and that is by design. Config management is about standardizing environments and having too many config toggles defeats this purpose (and as the number of configuration variables increases, so does the number of non-working configurations).
If you see another use case for which it could be adapted, please let me know (for example I recently added the possibility to configure streaming in a simplified way).

Ansible is not the only automation tool

This repo could point to other deployment methods/examples/howtos... I think the description/structure should be clarified in this way.

I would prefer to see this role in Ansible Galaxy. And we will, of course, point out its existence in the documentation.

It's there: https://galaxy.ansible.com/nodiscc/xsrv (as a collection):
$ ansible-galaxy collection install nodiscc.xsrv

But I suggest installing from git instead:
# $ cat requirements.yml
collections:
  - name: https://gitlab.com/nodiscc/xsrv.git # from git
    type: git
    version: release
# $ ansible-galaxy collection install --force -r requirements.yml


Feature/chart for year

Added pulling top charts by year
Ran a linter to remove trailing spaces and what not (if you don't like this il just undo it)

Thanks for the PR! I really appreciate you looking at my code :) I didn't notice that you submitted a PR so apologies for looking at it so late... I think selenium is a bit overkill for your feature. I did some digging and all the javascript is doing is calling a GET endpoint. Try the following REST call below!
curl 'https://www.melon.com/chart/age/list.htm?idx=1&chartType=YE&chartGenre=KPOP&chartDate=2013&moved=Y' -H 'Mozilla/5.0 (Windows NT 10.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/99.0.7113.93 Safari/537.36' -H 'Accept: */*' -H 'Accept-Language: en-US,en;q=0.5' --compressed -H 'Referer: https://www.melon.com/chart/age/list.htm?idx=1&chartType=YE&chartGenre=KPOP&chartDate=2013&moved=Y' -H 'X-Requested-With: XMLHttpRequest' -H 'DNT: 1' -H 'Connection: keep-alive' -H 'Cookie: PCID=0; PC_PCID=0; POC=LP10' -H 'Cache-Control: max-age=0'

Good idea with the linter! Do you have the name of the linter you want to use? I think it would be best if we enforced a linter via github actions. Feel free to refactor your code using the above REST call if you want your changes to be merged. I can work on the linter part.

Thank you for the recommendation works a lot better now! I removed selenium and replaced it with the request to list.htm like you recommended.
On the linter I'm just using standard Pylint.


Enhancing Statistical Inference Examples and Sampler Analysis

The integration of the package with PyTorch and its surrounding ecosystem is impressive, making it highly accessible to the community. However, the paper could benefit from including more examples that leverage this compatibility, particularly in the context of statistical inference with Pyro. For instance, a more detailed analysis of the NUTS sampler’s convergence would be valuable. The current example uses just 100 samples, but it is not clear if the sampling has fully converged. Given the speed of sampling, including more samples and evaluating convergence with standard metrics like Rhat (https://mc-stan.org/rstan/reference/Rhat.html) would strengthen the results.
Additionally, experimenting with different samplers, such as HMC, could add depth to the analysis. Providing recommendations on which samplers tend to perform well with the package, or if different samplers need to be tried on a case-by-case basis, would greatly benefit users. This could enhance the practical utility of the package and help guide its adoption in a wider range of applications.

Hi @andigu , that's a good idea to get into a bit of the difference between samplers that caustics can use. To me, this seems most appropriate to put with the tutorial example on the website using NUTS, rather than the JOSS paper. Using Rhat and autocorrelation length we can show how NUTS is able to draw fully independent samples, so 100 is plenty for demo purposes. Comparing with emcee and HMC will make this clear.
I'll comment here when we have some progress on this.

Hi @andigu  we have updated the discussion about sampling algorithms to now include No U-Turn Sampler (via Pyro), Metropolis Hastings (via emcee), and Metropolis Adjusted Langavin Algorithm (short script). We present the corner plots and chain mixing autocorrelation length to show how they compare. We looked into Rhat and it didn't seem well suited to the NUTS example, so we went with autocorrelation length since it could be computed for all methods. Let me know if there's anything more you think we should add!
https://caustics.readthedocs.io/en/latest/examples/Example_ImageFit_MCMC.html


Updated xfpga reconf tests with expected hw platform behavior.

Updated the test expectations with the expected hw platform behavior.
Updated the have_powermgmt check to be specific to the device that we are running on.

Will retarget to master.


Physics materials and contact materials not applied in v2.

System creates a default material and contact material.
Body attempts to reference default material, but gets undefined.
Local driver looks like it should use materials correctly, probably haven't tested that path on WW.

On the second point

material: this.system.material,
There is no system.material, right?
Only this.system.driver.materials[] ...

Correct, I think that's a bug from the refactoring earlier.


Implement erase and test commands

This pr implements 2 extra functions of the image group:

erase: erase the image in the current inactive slot or the specified one.
test: set the pending flag of the specified image. With the pending flag set, the image will be selected for upgrade on next reboot.

I also edited the list function to return a vector of ImageStateEntry instead of a generic serde_cbor::Value.
This allowed me to specify a specific deserializer for the hash field to display it as a HEX string, a more human readable format.
I needed it to be a hex string as I was expecting a hex string hash in argument of the test function.
tag: #19

I did replace the panic with a simple return false. I did not felt like to add a specific error on that case as we should never fall in that match case.

I tested it, and looks like there is a problem with the new list implementation. Tested with my device, the old version outputs this:
mcumgr-client list
mcumgr-client 0.0.4, Copyright © 2024 Vouch.io LLC

06:36:45 [INFO] One bootloader device found, setting device to: /dev/ttyACM0
06:36:45 [INFO] send image list request
response: {
  "images": [
    {
      "hash": [
        11,
        40,
        253,
        136,
        81,
        205,
        78,
        26,
        34,
        106,
        68,
        104,
        255,
        193,
        72,
        15,
        157,
        9,
        128,
        83,
        20,
        72,
        170,
        146,
        198,
        219,
        95,
        189,
        197,
        83,
        207,
        221
      ],
      "slot": 0,
      "version": "0.0.0"
    }
  ]
}

but your new version doesn't work:
cargo run -- list
mcumgr-client 0.0.5, Copyright © 2024 Vouch.io LLC

06:38:36 [INFO] One bootloader device found, setting device to: /dev/ttyACM0
06:38:36 [INFO] send image list request
06:38:36 [ERROR] Error: missing field `splitStatus`

You can also reproduce it with the "test" device:
cargo run -- --device test list

Please also add tests for the new commands in test_serial_port.rs if possible, could be used someday for automated CI tests to make sure nothing breaks after changes.

Ha yes my bad, has not realized the splitStatus is an optional field:
{
    (str)"images" : [
        {
            (str,opt)"image"        : (uint)
            (str)"slot"             : (uint)
            (str)"version"          : (str)
            (str,opt*)"hash"        : (byte str)
            (str,opt)"bootable"     : (bool)
            (str,opt)"pending"      : (bool)
            (str,opt)"confirmed"    : (bool)
            (str,opt)"active"       : (bool)
            (str,opt)"permanent"    : (bool)
        }
        ...
    ]
    (str,opt)"splitStatus" : (int)
}

I will fix this today and add the necessary code to test_serial_port.rs.

I just implemented a basic handling of the test and erase commands in test_serial_port.rs.
Could you test again on your side ?

Thanks, looks good now, and works with my device. Should I create a new release?

Thanks, looks good now, and works with my device. Should I create a new release?

It would be nice yes.
Thank you for the quick review.

Ok, done: https://github.com/vouch-opensource/mcumgr-client/releases/tag/v0.0.6


Multiple runs restart slapd everytime

It seems that when you run this role several times, the password configuration is reset everythime (a new hash value is calculated in "Hash OpenLDAP rootpw" and therefore slapd is restarted.
I would expect that running the role multiple times would not do anything unless a variable is changed.

I forgot to mention I tested this on Ubuntu 16.04.

This Ansible role is no longer being maintained.


tests fail to build when libgmock is installed by an external package

ninja: error: 'googletest/src/googletest-build/googlemock/libgmock.a', needed by 'quic/api/test/QuicTransportBaseTest', missing and no known rule to make it
*** Error code 1

The package googletest installed libgmock:
$ pkg info -l googletest | grep libgmock
	/usr/local/lib/libgmock.so
	/usr/local/lib/libgmock_main.so

Is your building environment by any chance behind an http proxy? I've seen this one when the host doesn't have direct access to github. The cause seems to be that ExternalProject_Add doesn't pick up proxy settings, and thus cannot download googletest. Bu if your host have direct access to github, then that's an issue we should address.

After the initial tarball is downloaded further internet access is prohibited during build for security reasons.
But this problems exists regardless of internet access.

yeah this is a known issue, we are trying to get rid of ExternalProject.
hopefully soon

removed ExternalProject.
please update and try again.
feel free to reopen this issue if experiencing similar problems


Style safe autocorrect 2

Fixes the remaining safely autocorrectable Style offenses.
(Reduces net offenses by > 2%.)

Coverage decreased (-0.004%) to 93.074% when pulling 8c0b3f45ba60b246ec93e63563cd196f58a589e1 on jdc-style-safe-autocorrect-2 into de0b1a8a4290e7817235c9d487afa9ae9f67ce0d on master.

Converted to draft so that I can autocorrect a few more cops

My main takeaway from this is, I've got to fix my autocorrect! :)
Seems good to me, apart from the one insignificant tweak.

This looks very familiar.  I could swear I'd already signed off on this one!  Still looks perfectly good to me.


Upload Fonts

Upload all the fonts that we are going to use in the css file, and add the fonts to the fonts folder.

Fonts added, now you can use the fonts that are installed.


Fix code scanning alert - Inefficient regular expression

Tracking issue for:

[x] https://github.com/reconsumeralization/CodeCurse/security/code-scanning/1

@sweep


Step publishPlugin throws HTTP 404

Hey 👋🏻
I'm using your wonderful plugin template. When trying to publish the Rider plugin to the JetBrains NuGet feed, nuget push always returns HTTP 404. You can see this here in my GitHub Action.
The funny thing is that the plugin's upload itself is successful. A couple of hours later, the new version appears in the JetBrains Marketplace (see here).
So what's going wrong with https://plugins.jetbrains.com? Why does it always return HTTP 404?
Best regards
mu88
CC: @vladyslav-burylov

Can you provide any information about when this started? I have some deployments from 30 days ago and they all look okay. Note that the first deployment must be done manually in all cases.

I cannot tell you, because it actually never worked 😁 in the beginning, @vladyslav-burylov did all releases by hand, uploading the assets to the JetBrains Marketplace manually. Then we've implemented the GitHub Action and use an API key to access JetBrains' NuGet feed. The API key seems to be correct, because otherwise the upload of the assets itself wouldn't work.

Can you share some logs?

Uh, can't you access the logs of the GitHub Action (the very first link in the first post)? If so, sorry about the missing infos :)
Here are the last important lines of the log:
> Task :publishPlugin
Pushing ReSharperPlugin.TestLinker2.2021.3.4.nupkg to 'https://plugins.jetbrains.com'...
  PUT https://plugins.jetbrains.com/api/v2/package/
  NotFound https://plugins.jetbrains.com/api/v2/package/ 830ms
error: Response status code does not indicate success: 404 ().

Usage: dotnet nuget push [arguments] [options]

Arguments:
  [root]  Specify the path to the package and your API key to push the package to the server.

Options:
  -h|--help                      Show help information
  --force-english-output         Forces the application to run using an invariant, English-based culture.
  -s|--source <source>           Package source (URL, UNC/folder path or package source name) to use. Defaults to DefaultPushSource if specified in NuGet.Config.
  -ss|--symbol-source <source>   Symbol server URL to use.
  -t|--timeout <timeout>         Timeout for pushing to a server in seconds. Defaults to 300 seconds (5 minutes).
  -k|--api-key <apiKey>          The API key for the server.
  -sk|--symbol-api-key <apiKey>  The API key for the symbol server.
  -d|--disable-buffering         Disable buffering when pushing to an HTTP(S) server to decrease memory usage.
  -n|--no-symbols                If a symbols package exists, it will not be pushed to a symbols server.
  --no-service-endpoint          Does not append "api/v2/package" to the source URL.
  --interactive                  Allow the command to block and require manual action for operations like authentication.
  --skip-duplicate               If a package and version already exists, skip it and continue with the next package in the push, if any.

> Task :publishPlugin FAILED

FAILURE: Build failed with an exception.

* Where:
Build file '/home/runner/work/[resharper-testlinker2](https://github.com/vladyslav-burylov/resharper-testlinker2)/resharper-testlinker2/build.gradle' line: 196

* What went wrong:
Execution failed for task ':publishPlugin'.
> Process 'command 'dotnet'' finished with non-zero exit value 1

I've also attached the complete log.
Release.yml.log

Could you please ping me for the next release?

Sure 👍🏻

Same problem happened again for 2022.1.1, please see the attached log file.
Release.yml.log

Could you find some time to investigate the issue @matkoch?

Same problem happened again for 2022.1.2

@matkoch Do you have any news about this? Rider 2022.2 is almost there and I'd really appreciate to get rid of this error with the next release of our Rider plugin

Note that the first deployment must be done manually in all cases.

It seems that the ReSharper plugin was never uploaded. What you've linked here is the Rider plugin.

@vladyslav-burylov Can you upload the ReSharper plugin, please?

Hi @mu88 - it looks like it was uploaded automatically, can you confirm?

Version 2022.2.0 of the Test Linker 2 plugin has been approved by JetBrains and is now publicly available at the following link: https://plugins.jetbrains.com/plugin/15518-test-linker-2/versions/stable/203959.

Although release job still failing for some reason: https://github.com/vladyslav-burylov/resharper-testlinker2/actions/runs/2787498968

Hey @vladyslav-burylov
If I understand @matkoch correctly, the ReSharper part of the plugin has to be uploaded manually once (that step is missing yet). Afterward, the automatic publishing will work.
But at the moment, only the Rider part of the plugin will be uploaded automatically and that's why the build fails.

Right, I think it simple then: reshaper part is simply absent as far as I remember. Reshaper has similar (and better) plugin which does the same - TestCop

Okay, sounds good to me! Then we should skip the upload for the R# part.
@matkoch: is it possible to upload only the Rider but not the R# part of the plugin?

Sure, just comment out this:
https://github.com/JetBrains/resharper-rider-plugin/blob/master/template/content/build.gradle#L207-L213

Ah, interesting! I thought the nupkg would contain both the R# and Rider plugin. If this is not the case, how is the Rider part uploaded?

From the publishPlugin task. What you see in the Gradle file is just an extension.


Linux video acceleration support behind a flag

Would it be possible for atom-shell to bypass existing protections in chromium preventing hardware video decode on intel chipset with linux? The idea would be to implement this chromium ticket Issue 176883018: Remove additional protection of kDisableAcceleratedVideoDecode in bpf_gpu_policy_linux.cc. that has been closed for the sake of stability.
That would be a great addition for linux app builders out there.
Some more context: Video Acceleration Takes The Backseat On Chrome For Linux
Related atom-shell issue: https://github.com/atom/atom-shell/issues/1032

Reading LinuxHWVideoDecode, it looks like the only required step to turn it on would be:

Add to $GYP_DEFINES:
chromeos=1 to link in VaapiVideoDecodeAccelerator
proprietary_codecs=1 ffmpeg_branding=Chrome to allow Chrome to play h.264 content, which is the only codec VAVDA knows about today.

The workaround solution for now is to use this ppa instead of atom-shell, which enables hardware acceleration and works as a charm.

@zcbenz any news regarding this? Would you consider merging a PR addressing this?

@mgcrea Yeah I will accept a PR on this, though as far as I can see the patch you pasted above has already been applied in Chromium.

There's a chromium-vaapi package for ArchLinux, which enables the hardware acceleration for Intel chipsets. I've been using it for weeks, works perfectly.
@zcbenz would it be possible to just take the vaapi patch from there and put in this repo among other patches?
The full build script includes other patches as well, but I think they are doing something else.
Although, that patch is based on the recent stable Chromium v59, so perhaps this should go after https://github.com/electron/libchromiumcontent/pull/313.

Some upstream progress https://chromium-review.googlesource.com/c/chromium/src/+/532294

[Off-topic] We still don't understand why the google team needs 6 years to accept a bloody flag for linux GPU decode where every other platforms have it enabled as default.

Firefox status isn't better https://bugzilla.mozilla.org/show_bug.cgi?id=1210727

The upstream patch mentioned above works very well, reportedly enabling support not only for Intel hardware, but for other GPUs as well. It requires enabling a few chrome flags after that, e.g. these work very well for me:
--enable-accelerated-mjpeg-decode
--enable-accelerated-video
--enable-gpu-rasterization
--enable-native-gpu-memory-buffers
--ignore-gpu-blacklist

Can we not wait on upstream and just use that patch and enable the flags in electron?

This is now by default in chromium on fedora: https://src.fedoraproject.org/rpms/chromium/c/278c62709d1dba5883c3b69047706837bb402bd7?branch=master
and ubuntu and arch linux is also looking into it:
https://www.linuxuprising.com/2019/01/ubuntu-testing-chromium-snap-with-vaapi.html

Arch Linux also includes the patch since February, enabled by default 😉

Hey there,
It's been a long time coming but the libchromiumcontent repository is being archived as it is no longer used in the Electron build system (🎉 this is a good thing).
If your issue is still relevant to Electron in it's current build system (please verify first) then you should probably raise a similar issue on electron/electron.
Thanks 😄


513-Add leaderboard display mode

Template
This is a template. While not all sections are necessary, depending on the size and complexity of the PR,
more information is better.
@ mention of reviewers
@ckcollab @dde6khkg
...
A brief description of the purpose of the changes contained in this PR.
...
Issues this PR resolves
...
https://github.com/codalab/competitions-v2/issues/513
Known issues to be addressed in a separate PR
...
A checklist for hand testing

[ ] add checklist here

Any relevant files for testing
link to any relevant files (or drag and drop into github)
Misc. comments
...
Checklist

[ ] Code review by me
[ ] Hand tested by me
[ ] I'm proud of my work
[ ] Code review by reviewer
[ ] Hand tested by reviewer
[ ] Ready to merge

@ckcollab @Logan-Ruf  i already update my pr, can you take a review on this, thanks~

@ckcollab @Logan-Ruf  can we merge this PR?


PHP 8.2 compatibility

In case you need to move to PHP 8.2 before @andrewdalpino find's time to update Tensor:
https://github.com/myfluxi/Tensor/commit/a6c229b762af5227a85db295e1d7451ab7edc8e9

On my Ubuntu 20.04 workstation (running PHP 8.2), I have carefully followed the instructions for Manually Compiling the Extension but when I attempt to make, it fails with these errors:
/home/sneakyimp/biz/machine-learning/tensor/Tensor/ext/kernel/main.c: In function ‘zephir_function_exists’:
/home/sneakyimp/biz/machine-learning/tensor/Tensor/ext/kernel/main.c:285:101: warning: comparison between pointer and integer
  285 |  if (zend_hash_str_exists(CG(function_table), Z_STRVAL_P(function_name), Z_STRLEN_P(function_name)) != NULL) {
      |                                                                                ^~
/home/sneakyimp/biz/machine-learning/tensor/Tensor/ext/kernel/main.c: In function ‘zephir_function_exists_ex’:
/home/sneakyimp/biz/machine-learning/tensor/Tensor/ext/kernel/main.c:301:76: warning: comparison between pointer and integer
  301 |  if (zend_hash_str_exists(CG(function_table), function_name, function_len) != NULL) {
      |                                                                            ^~
/home/sneakyimp/biz/machine-learning/tensor/Tensor/ext/kernel/main.c: In function ‘zephir_get_arg’:
/home/sneakyimp/biz/machine-learning/tensor/Tensor/ext/kernel/main.c:571:9: error: too many arguments to function ‘zend_forbid_dynamic_call’
  571 |     if (zend_forbid_dynamic_call("func_get_arg()") == FAILURE) {
      |         ^~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/php/20220829/main/php.h:35,
                 from /home/sneakyimp/biz/machine-learning/tensor/Tensor/ext/kernel/main.c:16:
/usr/include/php/20220829/Zend/zend_API.h:782:39: note: declared here
  782 | static zend_always_inline zend_result zend_forbid_dynamic_call(void)
      |                                       ^~~~~~~~~~~~~~~~~~~~~~~~
make: *** [Makefile:213: kernel/main.lo] Error 1

If I'm not mistaken, your Update for PHP 8.2 commit is designed to remedy this problem, but I have no idea how to apply your update to the repo I cloned with this command:
git clone https://github.com/RubixML/Tensor
Forgive my cluelessness, but can you please tell me how to apply your update?

Do you want to give it a try with the latest version (3.0.3) of the extension? This would be the first version that supports PHP 8.2.

PHP 8.1 and 8.2 support is still not complete, iterator methods don't have the proper prototype.
$ php  --re tensor
Deprecated: Return type of Tensor\Vector::offsetGet($index) should either be compatible with ArrayAccess::offsetGet(mixed $offset): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in Unknown on line 0
Deprecated: Return type of Tensor\Vector::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in Unknown on line 0
Deprecated: Return type of Tensor\Matrix::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in Unknown on line 0

PHP 8.1 and 8.2 support is still not complete, iterator methods don't have the proper prototype.
$ php  --re tensor
Deprecated: Return type of Tensor\Vector::offsetGet($index) should either be compatible with ArrayAccess::offsetGet(mixed $offset): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in Unknown on line 0
Deprecated: Return type of Tensor\Vector::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in Unknown on line 0
Deprecated: Return type of Tensor\Matrix::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in Unknown on line 0

I noticed this too @remicollet. I'm not sure how we'd go about adding these attributes to the extension code, however.

I don't know anythng about zephir things (excepted this add much complexity)
BTW, phalcon seems to have some iterators:
See

https://github.com/phalcon/cphalcon/blob/master/phalcon/Di/Di.zep#L461
https://github.com/phalcon/cphalcon/blob/master/ext/phalcon/di/di.zep.h#L103

About the warning about the return types: see #35
I think this #36 is about the segmentation fault we have with this sample code.
By running
strace php -r 'var_dump(Tensor\Matrix::ones(2,2)->inverse());'

we have that the last lines of the output are:
write(1, "\nDeprecated: Use of \"self\" in ca"..., 85
Deprecated: Use of "self" in callables is deprecated in Command line code on line 1
) = 85
write(1, "\nDeprecated: Use of \"self\" in ca"..., 85
Deprecated: Use of "self" in callables is deprecated in Command line code on line 1
) = 85
--- SIGSEGV {si_signo=SIGSEGV, si_code=SEGV_MAPERR, si_addr=0x10} ---
+++ killed by SIGSEGV +++

So, the segfault may be caused by using self in callables...


Limit externalSources per metadata field

References

Fixes REST side for https://github.com/DSpace/dspace-angular/issues/943
Related to https://github.com/DSpace/Rest7Contract/pull/153

Description
This PR will add support for an "externalsources" xml tag for a "relation-field" xml tag in the submission forms. It'll be parsed by the Java code and it'll be set in the REST response so that Angular can use this field to limit their external sources lookup.
This "externalsources" field can be multivalued and can be empty as well, it's currently set to optional in the .dtd file.
We've also added a relation-field tag on the PeopleStep of the submissions with pubmed as externalsource. This will also provide Angular with an easier way of testing/using their fix for this issue
Instructions for Reviewers
List of changes in this PR:

Added support for "externalsources" tag for a "relation-field" tag in the submission forms
Added an isPublicationOfAuthor relation-field for the PeopleStep

How to test this PR:

Build and deploy the code locally
Ask the submission forms for the PeopleStep config
Verify that it shows pubmed in an "externalsources" field

Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

@paulo-graca @abollini Thank you for the review! I've addressed both of the feedback points. Please let me know if anything else seems amiss.

@paulo-graca @abollini Thank you for the review! I've addressed both of the feedback points. Please let me know if anything else seems amiss.


Spelling

Corrected spelling of 'separate'

Docs Build status updates of commit 6a2e048:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamicsax2012-technet/argbcolor-class-microsoft-dynamics-commerce-runtime-datamodel.md
:warning:Warning
View
Details

dynamicsax2012-technet/argbcolor-class-microsoft-dynamics-commerce-runtime-datamodel.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.
Line 46: [Warning] Link https://technet.microsoft.com/en-us/library/e5kfa45b(v=ax.60) contains locale code en-us. For localizability, remove en-us from links to Microsoft sites. For more information, see https://aka.ms/LocaleSpecificLinks.

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.


Admin Pages for Villager

Villager admin pages

[ ] List
[ ] Create
[ ] Read
[ ] Update
[ ] Delete

Gonna move this to extras since its not really that crucial


chore: 🛠  enable strict mode

Trying to solve ts linting
But I don't know how to solve this resolvingContext[contextParamKey] which has unknown type here Record<string, unknown>
And toPromise() has a type of undefined
(project, projectName) what is the type for project
Fixes #324

Getting this type of errors for some packages
Could not find a declaration file for module 'standard-version/lib/lifecycles/changelog'. '/workspace/semver/node_modules/standard-version/lib/lifecycles/changelog.js' implicitly has an 'any' type.

I think you can try to install @types/standard-version and @types/semver

Then also same error


add source to remote_file for urls in plugin.pp

Getting an error when trying to add a new sensu plugin.  I'm defining the plugin as:
$url = 'https://raw.githubusercontent.com/sensu-plugins/sensu-plugins-puppet/master/bin/check-puppet-last-run.rb'

  sensu::plugin { $url:
    type               => 'url',
    nocheckcertificate => true,
  }

And the error in puppet:
Error: Could not set 'present' on ensure: bad URI(is not URI?): at 103: modules/sensu/manifests/plugin.pp

Add the source param to remote_file in the url block to fix this.
Add rspec tests to check for source

bump

I'm trying to understand why your URL is having this issue as it does seem to be a valid URI.  I had an odd thing once where there was a unicode character in a URL (not sure how it got there) that caused weird errors - can you make sure the file type is ASCII and that there aren't any "invisible" escape/control characters in the URL?

The remote_file module needs a source parameter (Forge link):

source: The source location of the file, or where to get it from if it is needed. This should be a URI.

Using the remote_file module directly:
remote_file { 'https://raw.githubusercontent.com/sensu-plugins/sensu-plugins-puppet/master/bin/check-puppet-last-run.rb':
  ensure => present,
  path   => '/tmp/check-puppet-last-run.rb',
}

Generates the same error:
Error: /Stage[main]/Test/Remote_file[https://raw.githubusercontent.com/sensu-plugins/sensu-plugins-puppet/master/bin/check-puppet-last-run.rb]/ensure: change from absent to present failed: Could not set 'present' on ensure: bad URI(is not URI?)

Adding source:
remote_file { 'https://raw.githubusercontent.com/sensu-plugins/sensu-plugins-puppet/master/bin/check-puppet-last-run.rb':
  ensure => present,
  source => 'https://raw.githubusercontent.com/sensu-plugins/sensu-plugins-puppet/master/bin/check-puppet-last-run.rb',
  path   => '/tmp/check-puppet-last-run.rb',
}

Works as intended:
Notice: /Stage[main]/Test/Remote_file[https://raw.githubusercontent.com/sensu-plugins/sensu-plugins-puppet/master/bin/check-puppet-last-run.rb]/ensure: created

The call in package.pp looks right. The call in plugin.pp should probably be:
remote_file { "${install_path}/${filename}":
  ensure   => present,
  source   => $name,
  checksum => $pkg_checksum,
  require  => File[$install_path],
} ->

Bump.

@jlambert121 I've encountered that issue before - with "invisible" characters in the url that sometime cause failures, but that doesn't seem to be the case here.  Puppet runs are failing on this resource only when I add a new plugin, so when it has to fetch it.  For previously existing ones, it sees it's there and moves on I guess.  But I can't get it to work at all without providing the 'source' param to the remote_file module.  By setting it to $name only for url types, the way we define a sensu::plugin would remain the same.


Welcome email content

Email Template
Subject: [Opticore Engineering] has invited you to Certifi.ly
Message., Hello KarthiKeyan.,
Welcome to Certifi.ly, Easy way to collect & store your professional certificates, documents, memberships, cards & tickets as NFTs on Flow blockchain. [Opticore Engineering] invited you to use certifi.ly to access your professional NFT documents from them and around the world.
Credentialshttps://alpha.certifi.ly/login [Environment based domain/subdomain]
Username<EMAIL_ADDRESS>Password: password
Every new certifi.ly user gets a walletless flow blockchain account on-boarding for free.
Once you login to Certifi.ly & Connected your Flow Wallet (Blocto)., Below Flow account ownership will be transferred to your connected flow account.
Your Certifi.ly Flow Account for<EMAIL_ADDRESS>0x1982389324987234
Regards,
Certifi.ly Teamwww.certifi.ly
<EMAIL_ADDRESS>

This content has been applied


Suggestion: Elegantly disable plugin when running without an x session

Is your feature request related to a problem? Please describe.
Love this plugin and added it to my dotfiles. I get a popup whenever I open neovim over SSH that says Could not get clipboard command. See :checkhealth img-clip. I have xclip installed, but this error pops up over SSH because xclip doesn't find an X session. When I connect with ssh -X everything works fine.
Describe the solution you'd like
I propose some way of detecting when an X session is not found, and disabling the pop-up message.

It does indeed! Thanks for the quick fix!

No problem! Glad you like the plugin :)


multiple exporters on target

is it possible to scrape multiple exporters on one target?
i.e. node_exporter on port 2000, mariadb_exporter on port 2001
how can i define exporter port?

Sure you can have multiple exporters on a target. Just build two separate jobs in Prometheus where the target is included in both queries.
The plugin only provides all available info from netbox as labels. Ports and relabeling is up to your Prometheus config

Closing for now. Please let me know if you need further assistance.


Replace oauth2client by google-auth

oauth2client is now deprecated. No more features will be added to the libraries and the core team is turning down support. We recommend you use google-auth and oauthlib.

I believe the latest versions of google-api-python-client support google-auth, so this package should be able to do this with only some changes to setup.py.

It also need change the way to get application default credentials
try:
            from oauth2client.client import GoogleCredentials
        except ImportError:
            return None

        try:
            credentials = GoogleCredentials.get_application_default()
        except:
            return None

the new function is google.auth.default()

I started on this in https://github.com/tswast/pandas-gbq/tree/gbq-37. It uses google-auth for service accounts and application default credentials. I'll update it to use google-auth for user credentials tomorrow.


Add shader/surface, drawdb node classification

For issue appleseedhq/appleseed-maya#41. This adds back the maya shader/surface classification where needed. Perhaps we can find a workaround later to restrict AS nodes to the AS menus/entries only.

I have a fix for this too. It's a bit different. Let me push mine instead.

Replaced by https://github.com/appleseedhq/appleseed/pull/1410


Add support for OpenType font collection testing

Associated IR: #2593
Many CJK typefaces are released in the *.ttc or *.otc format due to space savings that come from OT table sharing in this release format.  It would be helpful to be able to enter an OpenType font collection as an argument to the fontbakery executable and have profile checks performed on each font that is included in the font collection.
fontTools.ttLib.TTFont objects can be instantiated for each font in the collection with the fontNumber argument and an integer index definition:
https://github.com/fonttools/fonttools/blob/1dcb3c2bde86e9977bb384046e594e35ecd18796/Lib/fontTools/ttLib/ttFont.py#L21-L25

Sounds great! I will look into it more carefully. Thanks for the tip!

You can use fontTools.ttLib.ttCollection.TTCollection class to load them all at once, then you can iterate over them like a list of TTFonts

@felipesanches @simoncozens I have an internal project with TTC requirements so I'd like to see if we can do this in Feb :)

I feel like this will be challenging with the current architecture, but this could be due to my lack of understanding.
I can see how - in the "normal" case, each individual input file in a list can get converted through a "derived iterable" to multiple TTFont objects, but I have no idea how to "explode" a single file into multiple iterables. I have especially no idea how, if we have a check which takes a single ttFont object, a single file (in this case, a TTC) can be used to drive multiple instances of the same check.
Maybe @graphicore can shed any light on it?


Errors adding to declarations array with later versions of Angular

Hi,
I've followed as per the README and am encountering the same issues as the person below.  I've reviewed all closed issues in case this is a known issue already resolved but couldn't find anything specifically relating to Angular versions 9+ (I'm using 10).
Cannot declare 'CalendarHeatmap' in an NgModule as it's not a part of the current compilation angular2 calendar heatmap
https://stackoverflow.com/questions/60834457/importing-angular2-calendar-heatmap-causes-jit-compilation-error
Error is:
Cannot declare 'CalendarHeatmap' in an NgModule as it's not a part of the current compilation.
So following other suggestions, I attempted something like https://stackoverflow.com/questions/60248428/angular-app-cannot-compile-after-update-from-v8-to-v9 (see below) and used MyCalendarHeatmap in the declarations.  This just prints out the div (unsurprisingly).
@Component({
  selector: 'calendar-heatmap',
  template: `<div #heatmap></div>`
})
export class MyCalendarHeatmap extends CalendarHeatmap {}

I see recent questions (so am assuming people using relatively recent Angular versions) so thinking someone must have a workaround?
thanks
UPDATE: FIXED
Just needed to use the right hook.
For anyone else needing the info, I've added the full error message above.
I plonked the below directly above my @NgModule declaration, but you may wish to extract it out to a shared module and then import that module for neatness.
@Component({
  selector: 'calendar-heatmap',
  template: `<div #root></div>`
})
export class MyCalendarHeatmap extends CalendarHeatmap {}

Haven't used this fully yet, but it looks great - thanks!

I'm getting this
Appears in the NgModule.imports of AppModule, but could not be resolved to an NgModule class.

This likely means that the library (angular2-calendar-heatmap) which declares CalendarHeatmap has not been processed correctly by ngcc, or is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.

I dont get that method which you used to get things working. did you paste that code block inside your app.module.ts file above NgModule?

It's not a module, it's a component, and should be in the declarations array of the module you want to use it in OR in a shared module (which also exports it) if you want to use it in multiple places

It's not a module, it's a component, and should be in the declarations array of the module you want to use it in OR in a shared module (which also exports it) if you want to use it in multiple places - not in the imports array.

I missed this one. thanks a lot

It's not a module, it's a component, and should be in the declarations array of the module you want to use it in OR in a shared module (which also exports it) if you want to use it in multiple places - not in the imports array.

i am trying to implement as what you said. but i do not show anything on DOM. can you please help me out.


Is the alternative-optimization result is trained with OHEM?

Hi, @YuwenXiong
I saw that the alternative-optimization result is very close to joint training result(and there is only one script rfcn_alt_opt_5stage_ohem.sh for alternative-optimization), I wonder that was it trained with OHEM?
https://github.com/YuwenXiong/py-R-FCN#joint-training
https://github.com/YuwenXiong/py-R-FCN#alternative-optimization

Yes, as you can see there is a suffix ohem in the end of the script name.

@YuwenXiong Is there a result without OHEM? Thank you very much.

Unfortunately no, you can train a model by your self or refer the paper's result.