Implement Pagination for TestCases Table

Pull Request Template
Description
Implements pagination for testcases table
Fixes Part of #24
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue).
[X] New feature (non-breaking change which adds functionality).
[X] This change requires a documentation update.

How Has This Been Tested?
Pagination has been tested by checking the gql query created. The parameters offset and limit are checked and tested using UI
Checklist:

[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[ ] I have made corresponding changes to the documentation.
[ ] I have added corresponding tests.
[ ] Any dependent changes have been merged and published in downstream modules.

Thanks for the PR @anirudhjain75. While this was a good approach, we found an approach where the schema changes were not required and using the beauty of GraphQL within the same complexity we were able to fetch the total number of records. Closing this one as this is resolved now. :)
PR https://github.com/keploy/ui/pull/68


Should fileCache store file in the cache folder instead of the document folder?

I was surprised to see that the option fileCache: true store the images in the document folder. Wouldn't it be more intuitive to put it into the cache folder?

@wcandillon , yeah I think you're right, but the change will break backward compatibility, we can add this one to v0.11.0 milestone.


Adds Install/Config Workflow Options Doc

Documents the install/config options from #79.
Signed-off-by: danehans<EMAIL_ADDRESS>

Converted to draft until a decision is made. After a decision is made, I will remove the other options.

@youngnick @skriss @arkodg @LukeShu @alexgervais PTAL to help with your #79 vote.

Closed in favor of https://github.com/envoyproxy/gateway/pull/111


Use ManagedBuffer to store CuckooSet

Objectives
This pull request improves the per-insert performance of CuckooSet by changing its backing storage from Array to ManagedBuffer.
Implementation
When initializing a set, every pointer in the buffer is initialized to nil to indicate an empty bucket. This operation still takes linear time as the initializer must iterate through every pointer in the buffer. The buffer's header is empty. Instead of storing capacity and count in the header, they are stored inline at the set's definition. This may change to simplify copy-on-write semantics, but for now should provide meager performance benefits.
Instead of manipulating the contents of a bucket using the Array subscript, two new internal methods are available:
/// Returns the contents of the specified bucket, or `nil` if it is empty.
func contentsOfBucket(at bucketIndex: Int) -> Element?

/// Overwrites the contents of the specified bucket with the provided member.
/// If the member is `nil`, the bucket is emptied.
func overwriteBucket(at bucketIndex: Int, with member: Element?)

Performance
Benchmark output on the CuckooSet<Int> Insert task outputs the following comparison statistics:
Tasks with difference scores larger than 1.05:
  Score   Sum     Improvements Regressions  Name
  1.273   1.273   1.273(#76)   1.000(#0)    CuckooSet<Int> Insert (*)

The CuckooSet<Int> Contains task showed no significant improvement.
Most of this improvement is associated with the removed bounds checks associated with unsafe pointers. This significantly reduces memory safety, but our tests guarantee this implementation is safe.

I am dealing with the same issue locally and remotely where tests pass individually but fail when ran as a batch. This does suggest some memory safety issues. I need to look into ManagedBuffer more. I believe I am meant to subclass with a deinit override to deallocate everything before the program exits.

Last commit uses a CuckooStorage class that references capacity, count and the base pointer to the elements. These deinitialized and deallocated when the storage instance is deallocated.


Feautre: Critical point check for the tangent function

The maths division asked us for clearer output for critical points in the tangent function (tan(90° + n × 180°) for any integer n). Since we don't do symbolic calculation, all trig calculations use an approximation for π, which means the calculator gives an output for these inputs even though they aren't defined (the tangent line for tan(90°) is parallel with the x-axis and parallel lines by definition never cross). This PR adds a check for very big or very small output values for the tangent function and then handles them like they're an error in the input; this seems to be what many other numerical calculators seem to do too — there's no perfect solution without symbolic computation.

Marking this as a draft while waiting for the mathematics division to sign-off on an exact threshold value

No response from the division; I tried a few different values for the exact precision and this seems fine. Agreed to merge this in the PO-meeting 16th Sep '24.


Velero is incompatible with containerd

If using containerd and velero, the following issue will arise
backupstoragelocation.velero.io/default created
./addons/velero/1.2.0/install.sh: line 53: docker: command not found
./addons/velero/1.2.0/install.sh: line 54: docker: command not found

This is due to this function, which needs to be updated for when the CRI is containerd:
https://github.com/replicatedhq/kURL/blob/master/addons/velero/1.2.0/install.sh#L46:L58

Looks like this has been fixed.

Fixed in #589


1.39.1

Updated for Beat Saber 1.39.1 with minimal testing, mostly seems to work.

Code looks solid upon initial review. Manifest dependencies should match those on BeatMods. The BSIPA dependency was outdated in your branch, so I changed that to reflect the current version available on BeatMods.


Does it work for Fermi?

Hello,
In the paper, it says SASSI works for Fermi. But compiling only supports device with capability >= 3.0. Fermi architecture is 2.0+. It will fail during compilation stage when building examples.
How to make SASSI work for Fermi?
Best,
Leiming

Hi Leiming,
Good question.  SASSI will work for Fermi, but many (most?) of the sample instrumentation libraries we have provided use some convenient features that are only supported in 3.0+ architectures.  Let me take a quick look and see which of those sample libraries will work on Fermi and add a target to the makefile to only build those.
Thanks,
Mark

First, I confess that we haven't been focused on Fermi for quite a while.  Most of us internally use Kepler devices, and I have been using Maxwell.  That said, one of the benefits of being cooked into the compiler backend is that the instrumentation portion of SASSI should just work.  However, it looks like all of the sample instrumentation libraries except one use UVM __managed__ data structures to remove the pain of explicitly copying data back and forth between the host and device.  In addition, some of the libraries use some warp-collective operations that only work on 3.0+ architectures.
There is one sample instrumentation library that works on Fermi, but you will not think it is very interesting.  It is essentially a stubbed-out handler, called "stubs" that I use for debugging.  I just verified that my regression tests pass for "stubs" on Fermi.  I will back-port a simple example, such as ophist, to Fermi and push out a change when that's done.

Thanks, Mark.

Sorry this took so long.  I back-ported the ophist example so that it will work with Fermi.  In the process, I discovered a bug that could affect the perceived register allocation (and therefore warp occupancy) and instruction schedule of application.  On Fermi this bug caused the example application, matrixMul, to outright fail.  You will need to grab a new installer and reinstall SASSI to get the bugfix.  I'm closing this issue for now.


tag feed items() only works once, return undefined afterwards.

let feedObj = await ig.feed.tag('summer');
let postList = await feedObj.items();
let postList1 = await feedObj.items();
console.log(postList.length);
console.log(postList1.length);
postList1 is undefined.
I have already logged in. I am also using a proxy.

It's fixed just now in version 1.5.1


v.2.2.8 / can't obtain metrics from minikube imported cluster

What kind of request is this (question/bug/enhancement/feature request):
question
Steps to reproduce (least amount of steps as possible):
fresh install of minikube
minikube start --memory 8192 --cpus 4 --disk-size 50g --vm-driver vmware
running rancher in container
Result:
after successful importing of cluster in rancher and enabling monitoring, cant see any grafana dashboards and metrics

Other details that may be helpful:

Environment information

Rancher version: rancher/rancher 2.2.8
Installation option (single install/HA): single option

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Imported
Machine type (cloud/VM/metal) and specifications (CPU/memory): 4cpu / 8Gb
Kubernetes version (use kubectl version): 1.15

❯ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:40:16Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.2", GitCommit:"66049e3b21efe110454d67df4fa62b08ea79a19b", GitTreeState:"clean", BuildDate:"2019-05-16T16:14:56Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version): 18.09.1

❯ docker version
Client: Docker Engine - Community
 Version:           18.09.1
 API version:       1.39
 Go version:        go1.10.6
 Git commit:        4c52b90
 Built:             Wed Jan  9 19:34:26 2019
 OS/Arch:           windows/amd64
 Experimental:      false

Minikube version: 1.1.0

❯ minikube version
minikube version: v1.1.0

metrics appeared after 8 hours


Add contrast tests on typography

compare to background color
compare to road color for road labels

Nice to have item. Sometimes clients want specific levels of contrast between labels and what's behind them. Not much demand for this. Good to know that his has been asked for, but no need to implement any time soon.


⚠ Empornium has degraded performance

In be908cc, Empornium ($EMP) experienced degraded performance:

HTTP code: 200
Response time: 4174 ms

Resolved: Empornium performance has improved in 277bd0f after 20 minutes.


Added product feature for changing ownership for Service Catalog Item

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1641905
Links

https://bugzilla.redhat.com/show_bug.cgi?id=1641905
https://github.com/ManageIQ/manageiq-ui-classic/pull/5592

@miq-bot add_label enhancement services

@miq-bot add_label enhancement, services


Cleanup and document module execution order

Feature Request
Currently, there's no documentation around the order modules' hooks are executed. Add that and add an E2E test to verify that modules are loaded in the expected order:

First, NPM packages
Second, "user" modules in the project's modules/ directory
Finally, one-off hooks inside your wxt.config.ts

Also need to ensure user modules are executed alphabetically so they can be ordered with filenames.

Released in v0.19.12


[FEATURE REQUEST] Add a way to host your applied resouce packs too.

It would be nice, e.g. for some adventure maps, to be able to host the applied resource packs you have too.

I've been wanting to do this for a while, but I'm not sure how I'd want to handle the host having multiple resourcepacks installed. Maybe what I should do is only share the world's resourcepack (since a world can have an embedded resourcepack).


Manage Parameters within Controls

Description
As catalog author, I need a way to define, edit, and remove parameters that are inserted into control definition statements, consistent with NIST's use of parameters. (See Section 2.2 of [NIST SP 800-53r5[(https://nvlpubs.nist.gov/nistpubs/SpecialPublications/NIST.SP.800-53r5.pdf))
There are two types of parameters:

Assignment
Selection

Acceptance Criteria

A parameter can be inserted into a control definition statement while editing a catalog control. This includes:

showing the parameter in its appropriate location within the control definition statement
adding the parameter to the OSCAL data

A parameter can be revisited and viewed in the context of a control definition statement
A parameter can be revisited and edited in the context of a control definition statement
A parameter can be removed from a control definition statement

This includes removing the parameter from the OSCAL data

Proposed Solution
See the mockups and developer notes in Figma:
https://www.figma.com/file/ntIfIHIhMYbiqS9Nuc7EQG/Catalog-Mockups?type=design&node-id=918-2522&mode=design&t=g98hDcOjxTBMc4gR-0
Additional Notes
Use AC-4 enhancement 4 (OSCAL ID ac-4.4) in NIST SP 800-53r5 as an example of parameter types in text and as represented in OSCAL.
Implementation

[ ] I can (or plan to) submit a pull request to implement this
[ ] Implementing this may result in a breaking change

Per today's UI session, this diagram provides the flow for a new parameter.

9/7/23 Sprint Planning - Moved from Sprint 54 to Sprint 55

9/21/23 Sprint Planning - Moved from Sprint 55 to Sprint 56

10/05/23 Sprint Planning - Moved from Sprint 56 to Sprint 57

Will be started after #924

11/29/2023 - not started yet. moving to sprint 60

12/14/2023 - not started yet. moving to sprint 61

12/14/2023 - Still in Todo, moving to sprint 62

2024-01-25 moving to next sprint

2024-02-07 @jmshie-ed continues to work. Moving to next sprint.


Passthrough styler error on v53.1

On updating to v53.1 this error occurs -
Assets\Oculus\SampleFramework\Usage\Passthrough\Scripts\PassthroughStyler.cs(33,13): error CS0246: The type or namespace name 'OVRPassthroughColorLut' could not be found (are you missing a using directive or an assembly reference?)
Looks like its related to the new LUT passthrough feature but not sure how to reolve this as Ive just importeed a whole clean package in so not sure why there is an error on this.

Hi @owlandfox,
Looks like you might have gotten the latest Integration SDK from the download page. Part of this package is the Meta XR utilities which in this project is taken from a UPM package. As of v53 packages can mostly be taken from UPM.
In your case you could update the Meta XR Utilities package to use v53 through the package manager and that will solve your issue.


Refactor existing DataMerge functionality (closes #610)

Collapse most of the code and branches into one direct 'merge' method with stages.
Merge process is almost same for most data types, essentially just concatenating data along last dimension.
Stages can be customized for each datatype, instead of branching to separate merge.
Substitute many manual routines with numpy counterparts.
Move merge-related stuff from SupplementaryData into separate SupplementaryDataMerger.
Drop InputData.from_predictions in favor of DataMerge.merge that now directly returns InputData.

Hello @gkirgizov! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file fedot/core/utilities/array_utilities.py:

Line 40:1: W391 blank line at end of file

In the file test/unit/data/test_data_merge.py:

Line 117:29: E231 missing whitespace after ','
Line 144:26: E127 continuation line over-indented for visual indent

In the file test/unit/data/test_data_merge_text.py:

Line 37:10: E111 indentation is not a multiple of four
Line 37:10: E117 over-indented

In the file test/unit/data/test_data_merge_ts.py:

Line 42:50: E231 missing whitespace after ','


Add Adobe Forums to support page

Add Adobe Forums option to support page.

thanks, I added it that in the support page in another PR

thanks, I added it that in the support page in another PR


Support for untranslated pages

Why:
Support for untranslated pages.
https://docs.github.com/ja/repositories/releasing-projects-on-github/linking-to-releases
Only part of the text was translated, so the whole text is in Japanese.

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@riita10069
Thanks for opening this! Unfortunately, we are not able to accept changes for translated content. Our translation process involves an integration with an external service at crowdin.com, where all translation activity happens. We hope to eventually open up the translation process to the open-source community, but we're not there yet.
See https://github.com/github/docs/blob/main/contributing/types-of-contributions.md#earth_asia-translations.


Bug with field of view and wait action

Hi, i found a bug with field of view.
in game.gd
		if action.perform():
			if player.grid_position != previous_player_position:
				map.update_fov(player.grid_position)

if an enemy is in a player's potential field of view, but the player uses the wait action, it will not get shown and the player will be continously attacked until they move or kill the enemy, i don't know how to fix this, So for me i just got rid of the if statement  that checks if player moved and update the field of view regardless.

Hi! Thank you for mentioning that! It seems that is indeed a logical error on my part. I thought that recalculating was only necessary when moving, and didn't account for the situation you described.
Removing the if statement should indeed fix that problem. I will make sure to change the code (and tutorial) accordingly.

Should now be solved by 71e7a8e (also updated on the website). Thanks again!


Updated README.md

This pull request will fix some errors in the tutorial, and gives people a little more control over their lights.
Changelog:

[Line 41] Replaced 2020 with 2022
[Line 70] Replaced pip with pip3 for Python 3
[Line 95] Replaced python with python3 for Python 3
[Line 109] Added info about changing the default color

@larochefoucald

Thank you @larochefoucald ! I love contributing and helping with what I can, even though I am semi-new to coding and new to GitHub


Allow passing a merge customizer function.

Hi there, thanks for the module; I really like what you have done. Anyways, I needed this change in a project of mine and figured it might be useful to others (if not, no worries). Let me know if you see anything funky with the PR and I'll be glad to make a change.
Thanks again,
Cody

@kingcody Thanks for the PR. I'm really sorry it took such a loooooong time to get it merged.
Landed in v0.4.3

@creynders thanks :)


Remove Tempfile#unlink dead alias

This PR removes dead Tempfile#unlink alias as specified in this comment:
https://github.com/crystal-lang/crystal/pull/6035#issuecomment-385446006

This PR is not really worth the effort as it will be covered by #5939, but go ahead.


Exposing more than last layer with BERT?

It looks like the BERT hub model currently only exposes the last layer of outputs, per 'sequence_output' (https://tfhub.dev/google/bert_uncased_L-12_H-768_A-12/1).
Are there any plans to expose the intermediate layers?
Section 5.4 in the original paper (https://arxiv.org/pdf/1810.04805.pdf) suggests a useful path around using BERT as a feature-based input.  This would be useful in cases, e.g., where fine-tuning the entire model is prohibitive (eg, very long documents).  Simply taking the last layer is an option, but performance clearly is below what it otherwise could be.

We don't have plans to include a signature for intermediate layers, since this would mean that different BERT modules would have slightly different signatures.
However, this is doable post-hoc with a few lines of extra code, shown below.
import tensorflow as tf
import tensorflow_hub as hub

def get_intermediate_layer(last_layer, total_layers, desired_layer):
  intermediate_layer_name = last_layer.name.replace(str(total_layers + 1),
                                                    str(desired_layer + 1))
  print("Intermediate layer name: ", intermediate_layer_name)
  return tf.get_default_graph().get_tensor_by_name(intermediate_layer_name)

with tf.Graph().as_default() as g:
  input_ids	= tf.zeros([1, 1], tf.int32)
  input_mask = tf.zeros([1, 1], tf.int32)
  segment_ids	= tf.zeros([1, 1], tf.int32)
  bert_inputs = dict(
      input_ids=input_ids,
      input_mask=input_mask,
      segment_ids=segment_ids)
  bert_module = hub.Module("https://tfhub.dev/google/bert_uncased_L-12_H-768_A-12/1")
  bert_outputs = bert_module(
      inputs=bert_inputs,
      signature="tokens",
      as_dict=True)
  layer_6 = get_intermediate_layer(
      last_layer=bert_outputs["sequence_output"],
      total_layers=12,
      desired_layer=6)
  with tf.Session() as session:
    session.run(tf.global_variables_initializer())
    print(session.run(layer_6))

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

@rmothukuru Hi there, I am trying the solution you provided to get the immediate output of Bert, but when the code hub.Module() raises the error RuntimeError: Missing implementation that supports: loader .... As this comment points out, I am using a SavedModel in TensorFlow 2 format, so I do not have the file tfhub_module.pb. In this case, can I have some another way to have access to middle layer output of Bert? e.g. can you provide the code in tf 2.0 style? Thanks in advance!


Regenerated MDKModules after removal of style.csss

Regenerated MDKModules after removal of style.csss

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


HL-282 | Message API

Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #447 (7ea75f6) into develop (18c8ad5) will decrease coverage by 1.05%.
The diff coverage is 97.35%.

@@             Coverage Diff             @@
##           develop     #447      +/-   ##
===========================================
- Coverage    95.58%   94.53%   -1.06%
===========================================
  Files          201      156      -45
  Lines         7631     6180    -1451
===========================================
- Hits          7294     5842    -1452
- Misses         337      338       +1

Impacted Files
Coverage Δ

backend/benefit/messages/views.py
85.36% <85.36%> (ø)

backend/benefit/messages/permissions.py
91.66% <91.66%> (ø)

...nd/benefit/calculator/tests/test_calculator_api.py
98.81% <100.00%> (-0.01%)
:arrow_down:

backend/benefit/common/tests/conftest.py
100.00% <100.00%> (ø)

backend/benefit/helsinkibenefit/urls.py
91.17% <100.00%> (+1.89%)
:arrow_up:

backend/benefit/messages/models.py
100.00% <100.00%> (ø)

backend/benefit/messages/serializers.py
100.00% <100.00%> (ø)

backend/benefit/messages/tests/conftest.py
100.00% <100.00%> (ø)

backend/benefit/messages/tests/factories.py
100.00% <100.00%> (ø)

backend/benefit/messages/tests/test_api.py
100.00% <100.00%> (ø)

... and 55 more

Continue to review full report at Codecov.

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


peer.address in tag is not in common tags of opentracing

peer.address  is not common tags of opentracing.Why not use tags,such as peer.hostname and peer.port, instead of peer.address in the code ?

because it's a trick to extract hostname/port when:

database is in memory
database is a file
default port is different for different databases
would you like to submit a PR to fix all cases for major databases jdbc url variations?

peer.address is not common tags of opentracing

It now is: https://github.com/opentracing/specification/pull/53


Type Error due to stale browser cache after LAN address change

Archer 1750 A7
OpenWrt 21.02.2 r16495-bf0c965af0 / LuCI openwrt 21.02 branch git-22.052.50801-31a27f3
(with all available updates as of 3/16/2022)
Steps to reproduce:

Install all recommended updates (in LuCI) after the initial bare firmware installation.
Change the LAN interface IP address in LuCI, uci, or network config file.
Restart network or reboot router
click on the "Network > Interface" or "Network > Firewall" menu option

Actual behavior:

A red Type Error "firewall.getZoneColorStyle is not a function" appears
Network (or Firewall) screen never opens.

Expected behavior:
(it works instead of not working)
Additional Information:
Clearing the browser cache (at least in Firefox) fixes the problem.
I'm not a web app expert by any means... however, I suspect web apps are supposed to poison cache data when necessary.
Perhaps this is just a Firefox problem because nothing from the old ip address should be referenced in cache when using the new ip address.  It should effectively be a completely different web site from the browser's perspective.
In fact it really seems like a security bug on their part.   Still, I thought you should know since I've never had this problem with OpenWRT before.

LuCI employs some cache busting techniques (mainly query strings containing the git hash LuCI wath built from). Under some circumstances (some luci packages except luci-base updated or only luci-base updated by opkg) the cache buster string might get updated but the browser fetching old version of classes. If that happened on a device previously using the IP you changed your device to, you might see that behavior.
There's no simple fix to this short of recalculating the cache buster string at runtime, after each package update. I'll close this issue as wontfix as clearing the browser cache usually suffices to solve the issue.


Using AD Authentication is not possible on Linux Consumption plan for Azure Functions

It is not documented but it seems that you can't use this when you are trying to use this in an Function App that runs on a Consumption Plan for Linux. Could this be clarified and if this is on the roadmap? Using Python in Azure Functions forces you into a Linux plan but if you want to leverage AAD for authentication you are then forced to use a Premium Plan (adding unnecessary cost imo).

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

ID: 6f1c454d-73bc-5a73-5d3f-1b1ca7625044
Version Independent ID: 552b3f09-6650-e7f7-01f2-a702005b83e9
Content: Configure Azure AD authentication - Azure App Service
Content Source: articles/app-service/configure-authentication-provider-aad.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Hi @djerro123, thank you for the detailed question. We are currently investigating and will update you when we have accurate information to provide.

@JarroVGIT Right, While EasyAuth isn't available with Linux Consumption tier, you should still be able to use (a) AAD Auth by creating your AAD app separately or (b) use MSI depending on your scenario. Have you tried the alternatives, a and b?

Hi @JarroVGIT, along with Mike's answer I have submitted a PR to update the doc with a note, the change is merged internally and will go-live in the public docs shortly. Thanks!
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Hi @mike-urnun-msft, could you give me a little bit more detail (or a link) as to how to do it with "use AAD auth by creating your AAD app seperately? Also, any updates on when to expect EasyAuth for Azure Functions on Linux Consumption Plan?

I never figured that part out either. It seems to me that you would register an application in AAD, then allow a user/service principal access to said application (in case of SPN, this isn't trivial and can only be accomplished by using Powershell btw). Then the calling application would have to manually requests an access_token from AAD, post that with the Authorization header, and then your Python function would validate that token. However, I failed on the last step as there is no "middleware" (quotes because that is the term MSFT uses here in the docs)  available for Python to manually validate that token, using the client credentials from step 1 (register the app).
Maybe I am not versed well enough in this topic, but that at least is where I stopped and moved to a (expensive) Premium Plan.

@mike-urnun-msft I too would like some detail on how to implement the suggested "use AAD auth by creating your AAD app separately". I feel that Azure Functions are aimed at developers who are looking for tools that hide as much of the plumbing as possible. Therefore I'd expect a lot of people using Linux as the OS for their Function could do with some assiatnce with this.
Alternatively I could just switch to


Question on order of bodies in arbiter code

In the Arbiter code, there is a comparison of less than in the body pointers for body 1 and 2. Why is this there? Why is this necessary?
https://github.com/erincatto/box2d-lite/blob/227b71b6974ea57ab7e96d40f6374287bd6a0e77/src/Arbiter.cpp#L16-L27
I am porting this code to other language, and I am trying to figure some bugs that are appearing, and in this language, I can't do this type of comparisons on references, so I am just curious if I need to figure something to replicate this. Thanks!

This if/else is not needed. I'm guessing this is leftover from an earlier version. You can just use
body1 = b1;
body2 = b2;


Converting creates gigantic tmp files

Converting a 7MB mp4 video creates a 4.8MB temp file, and will hang until operation is cancelled.

I had a 350K .mov file, the script made a 65M temp file. 4.8M is excusable, but 65M is not.

It's true, it's inexcusable, and I'm ashamed.
Just kidding! This is the ffmpeg part of the conversion. If you're using this on a system where creating large temp files is not excusable I would strongly advise against using gifify.
Seriously, all this does is create a file with ffmpeg and then convert it into a gif using ImageMagick. There are way better solutions, but this one works best for me and some other people.
That said, if you know of a way to do this reliably using pipes and without a temp file, I'd happily update it.

@jclem Closing, as this is the intended behavior.

@ChaseGiunta I'd definitely like to find a more efficient way to do this, I just haven't had time to look into it yet. I just took some offense to being told that there is no excuse for it.

Totally mistyped the after value btw... Supposed to be 4.8 GB

That I've definitely never seen in the entire time I've been using this. Is this reproducible with other files? I'm using a fairly standard ffmpeg command.


Release 0.11.0

Description

Related issues

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Changes Made

package updates
#424
#419
#421
#425
#426
#430

Testing

[x] Unit Tests
[x] Integration Tests

Mentions

#433 required for this to be in master


incompatible with Docker-compose when enable host and proxy params.

From Issue #15
Here is the copy from @Kristovich and his test:

Kristovich

I'm using the docker compose from da62e9a. I built the api and client containers as instructed on the readme.
I also added console.log(`Line 11, host${host}port${port}`); line inside api/server/index.js. When - HOST="<IP_ADDRESS>" is set I get:
api_1      |
api_1      | ><EMAIL_ADDRESS>start
api_1      | > node server/index.js
api_1      |
api_1      | Line 11, host"<IP_ADDRESS>"port3080
api_1      | node:events:490
api_1      |       throw er; // Unhandled 'error' event
api_1      |       ^
api_1      |
api_1      | Error: getaddrinfo ENOTFOUND "<IP_ADDRESS>"
api_1      |     at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:107:26)
api_1      | Emitted 'error' event on Server instance at:
api_1      |     at GetAddrInfoReqWrap.doListen [as callback] (node:net:1934:12)
api_1      |     at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:107:17) {
api_1      |   errno: -3008,
api_1      |   code: 'ENOTFOUND',
api_1      |   syscall: 'getaddrinfo',
api_1      |   hostname: '"<IP_ADDRESS>"'
api_1      | }

If - HOST=<IP_ADDRESS> then
api_1      |
api_1      | ><EMAIL_ADDRESS>start
api_1      | > node server/index.js
api_1      |
api_1      | Line 11, host<IP_ADDRESS>port3080
api_1      | Server listening at http://<IP_ADDRESS>:3080

And while I'm at it: If - PROXY="" is present inside the docker-compose.yml, then the server will work but requests to openai or bing will fail with
api_1      | ask log {
api_1      |   model: 'chatgpt',
api_1      |   id: '[uuid]',
api_1      |   sender: 'User',
api_1      |   text: 'this is a test',
api_1      |   parentMessageId: undefined,
api_1      |   conversationId: undefined,
api_1      |   chatGptLabel: '',
api_1      |   promptPrefix: ''
api_1      | }
api_1      | TypeError [ERR_INVALID_URL]: Invalid URL
api_1      |     at new NodeError (node:internal/errors:399:5)
api_1      |     at new URL (node:internal/url:588:13)
api_1      |     at new ProxyAgent (/api/node_modules/undici/lib/proxy-agent.js:67:25)
api_1      |     at ChatGPTClient.getCompletion (file:///api/node_modules/@waylaidwanderer/chatgpt-api/src/ChatGPTClient.js:168:31)
api_1      |     at ChatGPTClient.sendMessage (file:///api/node_modules/@waylaidwanderer/chatgpt-api/src/ChatGPTClient.js:297:24)
api_1      |     at async askClient (/api/app/chatgpt-client.js:28:15)
api_1      |     at async /api/server/routes/ask.js:90:23 {
api_1      |   input: '""',
api_1      |   code: 'ERR_INVALID_URL'
api_1      | }

So I had to remove the PROXY env out of the docker-compose file for it to work.
I hope this helps.

Hi @danny-avila  and @Kristovich
I have made some fully check of the previous pull request #19 . As motioned yesterday, it seems an incompatible issue with docker compose.
Here is my conclusion:

In the case of local deployment, HOST and PROXY work as expected, specific tests will be listed later

Before this PR, all interface will be listen, (- although this is very insecure, any user in the LAN can access it.). I changed to listen localhost by default, and user can set HOST=<IP_ADDRESS> to expose to external network as before.
<IP_ADDRESS> is not a real ip, so printing <IP_ADDRESS>:3080 might be misleading. (actually, localhost:3080 or external_ip:3080 will be enabled in this case) I will start a new PR to improve this.

In the case of docker-compose environment, as @Kristovich said, there is error message when - HOST="<IP_ADDRESS>" or - PROXY=""

It turns out a wrong usage of docker-compose, in This and This .
quotes shouldn't be used as - ENV="value", we can use - "ENV=value" or - 'ENV=value' or - ENV=value.
both HOST and PROXY in @Kristovich 's test are the same issue of the quotes

I have mode those tests:
Local environment
test1: HOST= and no PROXY:   works on localhost
test2: HOST=localhost and no PROXY:   works localhost
test1: HOST=<IP_ADDRESS> and no PROXY:   works all interface
test1: HOST=192.168.xx.x and no PROXY:   works on external ip
test1: HOST= and no PROXY=http://<IP_ADDRESS>:8888:   works, using proxy.

Docker environemt:
test1:
    docker compose as original:
    works but 502. (nothing wrong. localhost in docker means not accessable outside)

test2:
    Add - HOST=<IP_ADDRESS>
    works

test3:
    Add - HOST="<IP_ADDRESS>"
    error as @Kristovich said.

test4:
    Add - HOST=""
    error as @Kristovich said.

test5:
    Add - "HOST=<IP_ADDRESS>"
    works

test6:
    Add - 'HOST=<IP_ADDRESS>'
    works

test7:
    Add - HOST=
    works but 502, as test 1.

test7:
    Add - PROXY=""
    error as @Kristovich said.

test7:
    Add - PROXY=
    works

In short.
That PR has no incompatible issue with docker. It's a wrong usage of docker-compose.yaml.
But I will open a new PR add this:

add more information on HOST=<IP_ADDRESS>, removing the misleading information
set env HOST=<IP_ADDRESS> in api's Dockerfile by default. So that it won't break the README's docker demo.
fix the wrong example in docker-compose.yaml. This might mislead @Kristovich to write - HOST="<IP_ADDRESS>":

        environment:
                - PORT=3080
                - MONGO_URI=mongodb://mongodb:27017/chatgpt-clone
                - OPENAI_KEY=""        <----- WRONG, use - OPENAI_KEY=
                - CHATGPT_TOKEN=""        <----- WRONG
                - BING_TOKEN=""        <----- WRONG


Some questions for the paper

What is the difference between the score in Table 5 and Table 8?
77.19 in Table 5 results on test-dev set of VQAv2, and,
77.68 in Table 8 results on test-dev set of VQAv2.

See https://github.com/zdou0830/METER/issues/6. Thanks!

@zdou0830 Thanks!
I have another question about the codes.
I tried your pretrained model to finetune VQA v2 with default setting.
However, the score of val is only around 72.55.
It should be more than 80..
Could you share your experiment settings of finetuning VQA v2 and pretraining tasks?

The command should be
python run.py with data_root=$DATA_DIR num_gpus=8 num_nodes=1 task_finetune_vqa_clip_bert per_gpu_batchsize=4 clip16 text_roberta image_size=576 clip_randaug load_path=meter_clip16_288_roberta_pretrain.ckpt
Hope this thread would help https://github.com/zdou0830/METER/issues/7.

If I use your finetuned model to do the test only task of VQA v2, the result is 77.66.
This model works.

You can try testing the last checkpoint and submitting the resulting json file to evalai.

The result from evalai is 71.53.
Should I finetune the model with more step?

The VQA dataset can be downloaded here: https://drive.google.com/file/d/1qT7YWHpLg-fAL43daKlOsYx2EbbQk--d/view?usp=sharing.
The training command is
python run.py with data_root=$DATA_DIR num_gpus=8 num_nodes=1 task_finetune_vqa_clip_bert per_gpu_batchsize=4 clip16 text_roberta image_size=576 clip_randaug load_path=meter_clip16_288_roberta_pretrain.ckpt

The testing command is
python run.py with data_root=$DATA_DIR num_gpus=8 num_nodes=1 test_only=True task_finetune_vqa_clip_bert per_gpu_batchsize=4 clip16 text_roberta image_size=576 load_path=last.ckpt

The provided VQA-finetuned checkpoint is trained in this way, so if you follow these steps correctly, you should be able to get a score of ~77.6 on test-dev. I didn't look at the dev scores and the number of training epochs was set to 10 as in config.py. The fine-tuning took about 2 days on 8 V100s for reference.

Ok, I will have a try! Thank you for your patient.

I found a problem.
If I use only one node to finetune the pretrained model, the result will be better than two nodes (around 5% in VQA v2 Val).
That might be the reason.
May you share your pretraining log?
I will pretrain the model in two nodes.
I want to know the different between one node and two nodes.

I didn't save the logs, but I did pre-train the models with 1/2/4 nodes and there were no significant differences, so I'd suggest you to debug your multi-node training settings.

I also met the same problem, which probably results from multi-node training settings. I use slurm to multi-node train. May you share your train bash file, if you also use slurm.

I didn't use slurm, but I uploaded the running file for distributed training on Microsoft machines (https://github.com/zdou0830/METER/blob/main/azure_distributed_run.py). Not sure if this is helpful.

ok, thank u~

I also met the same problem, which probably results from multi-node training settings. I use slurm to multi-node train. May you share your train bash file, if you also use slurm.

@jiyt17 Hello, have you solved the problems you encountered before?

@zdou0830 Thanks!
I have another question about the codes.
I tried your pretrained model to finetune VQA v2 with default setting. However, the score of val is only around 72.55.
It should be more than 80..
Could you share your experiment settings of finetuning VQA v2 and pretraining tasks?

@wanng-ide Hello, have you solved the problems you encountered before?


CIS Test for Browser service edit for port 1433

A New PR
THANK YOU - We love to get PR's and really appreciate your time and help to improve this module
Accepting a PR
Before we accept the PR - please confirm that you have run the tests locally to avoid our automated build and release process failing. You can see how to do that in our wiki
https://github.com/sqlcollaborative/dbachecks/wiki
Please confirm you have 0 failing Pester Tests
[X] There are 0 failing Pester tests
Changes this PR brings

Edit of browser service test to check to makes sure the a server running one instance it is running on the default port
CIS tag

Close #679
Please add below the changes that this PR covers. It doesnt need an essay just the bullet points :-)

Thank you


New Feature Request: Filter functions (and docs for it)

Feature request: Ability to do “OR”’s in filtering. Generalizing this request to just more pattern-like ability within filtering. (complicated feature we punted on but should think more about).  To include functionality such as:

regular-expression searches
case-insensitive searches
enable dereferencing within Filters, to allow eg. created_by_ref.name

Leaning towards just allowing the capability to pass a filter function (function which takes a STIX object and returns a bool) in addition to Filter object. If we do this, we’ll want to have some examples.

Dev Meeting 18 April 2018
Punt on this enhancement for now. The functionality can be done just as easy on the user side and does not require being pulled into the library.

Since we decided to hold off on this, I'm closing this issue. But if someone reading this would greatly benefit from having this feature, please open a new issue or PR and we'll reconsider!


Add a .gitattributes files to code with text files, their line-endings, and storing binary files.

Something like this:
* text eol=crlf
*.sh text eol=lf

*.7z binary
*.aif binary
*.aiff binary
*.asf binary
*.eot binary
*.fla binary
*.flac binary
*.flv binary
*.gif binary
*.gz binary
*.ico binary
*.jpeg binary
*.jpg binary
*.mid binary
*.mov binary
*.mp2 binary
*.mp3 binary
*.mp4 binary
*.ogg binary
*.otc binary
*.otf binary
*.pdf binary
*.png binary
*.pyc binary
*.swf binary
*.tga binary
*.ttc binary
*.ttf binary
*.wav binary
*.wma binary
*.wm binary
*.wmv binary
*.woff binary
*.zip binary

Great! +1 for this. My only comment is to prefer unix line endings for our source code files

My only comment is to prefer unix line endings for our source code files

I hear you! The issue is really cross platform development when including Windows... IIRC, VS2019 is now kosher with LF, but VS2015 and 2017 aren't. (Although 2015 might not be relevant to this module being on the old and gray side of toolchains...)

Hm, right... I think it's certainly fair to leave 2015 behind :) If there's a trivial way to support VS2017 here I'm for it, but if not then it seems to me maybe not worth maintaining any complicated dance for it?

To be fair, every IDE/editor on macOS and Linux handle CRLF just fine...


Add set devices to empty list when osd_auto_discovery used

osd_auto_discovery intend to setup OSD automatically, so one does not
need that devices contain any value.
when osd_auto_discovery used, one may ommit devices

Signed-off-by: Arano-kai<EMAIL_ADDRESS>

@Arano-kai since handled in #2000, with https://github.com/ceph/ceph-ansible/pull/2000/files#diff-2857b72f37476e1b5d16e7e81783ecbaR137 would you consider closing this? #2000 will merge at the latest tomorrow. Thanks.


Dimensions of images captured with iOS camera are swapped when shared.

Describe the bug
The width and height of images taken with an iOS camera and shared with the app are swapped.
expo-share-intent loads the image with the following data:
{"fileName": "IMG_5171-60E60263-F351-48E0-9076-916795F9919F.JPG", "height": 3024, "mimeType": "image/jpeg", "path": "file:///Users/usman/Library/Developer/CoreSimulator/Devices/446D1C8A-D7E8-48CB-BB37-C95818F8E990/data/Containers/Shared/AppGroup/1F1E0A8A-620C-4E7B-9E0E-65E759BD59B0/020FB1AD-D0EA-4C34-A60A-FFB708CA6CD0.JPG", "size": 2057210, "width": 4032}

You can see width is 4032 and height is 3024.
while expo-image-picker loads the same image as:
{"assetId": "50728FCF-30B5-49F8-B6CA-22E7A043B325/L0/001", "base64": null, "duration": null, "exif": null, "fileName": "IMG_5171-60E60263-F351-48E0-9076-916795F9919F.jpg", "fileSize": 399112, "height": 4032, "mimeType": "image/jpeg", "type": "image", "uri": "file:///Users/usman/Library/Developer/CoreSimulator/Devices/446D1C8A-D7E8-48CB-BB37-C95818F8E990/data/Containers/Data/Application/F0823413-A76C-4D07-BEE4-64B38632B550/Library/Caches/ImagePicker/7D70D411-45DE-45FC-BD95-BEA3783A2FD2.jpg", "width": 3024}

Here you can see width is 3024 and height is 4032.
Here is a sample image file:
https://github.com/user-attachments/assets/495a7ede-f837-4f25-a855-1f34a99824ff
To Reproduce

Take a photo with an iOS camera.
Share photo
notice how width and height are swapped for unknown reasons.

Environment
System:
OS: macOS 14.6.1
CPU: (8) arm64 Apple M1 Pro
Memory: 79.06 MB / 16.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 18.18.2
path: ~/.nvm/versions/node/v18.18.2/bin/node
Yarn:
version: 1.22.19
path: ~/.yarn/bin/yarn
npm:
version: 9.8.1
path: ~/.nvm/versions/node/v18.18.2/bin/npm
Watchman:
version: 2<IP_ADDRESS>
path: /opt/homebrew/bin/watchman
Managers:
CocoaPods:
version: 1.14.3
path: /Users/usman/.rbenv/shims/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 23.4
- iOS 17.4
- macOS 14.4
- tvOS 17.4
- visionOS 1.1
- watchOS 10.4
Android SDK: Not Found
IDEs:
Android Studio: 2023.3 AI-233.14808.21.2331.11842104
Xcode:
version: 15.3/15E204a
path: /usr/bin/xcodebuild
Languages:
Java:
version: 17.0.9
path: /usr/bin/javac
Ruby:
version: 2.7.5
path: /Users/usman/.rbenv/shims/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native:
installed: 0.74.5
wanted: 0.74.5
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: Not found
newArchEnabled: Not found
iOS:
hermesEnabled: Not found
newArchEnabled: Not found
Additional context

expo version (ex: 49) : 51
using firebase : yes
using static build : yes / no
routing :  react-navigation
package version (ex: 1.2.0) : 2.3.0
platform target :  iOS 16.4
device : simulator
type: expo eas build

available in v2.3.1


Apparently pausing is ✨slow✨

pausing ⏸ works 💪 fine 🙂 for ➡ me 🙋♂ but 😕 it takes about 4 seconds 🔢 to change 😱😰😬🙀😳😦


Adding SSPL to license classifiers

Hello,
can you please add SSPL to the classifiers?
It's not OSI-approved but it's still a decent open source license.
I have a piece of software licensed under it and I'm currently using "Other/Proprietary" as its license classifier because SSPL is not there even if the software is certainly open.
Thank you.

Hi, the licenses come from https://github.com/pypa/trove-classifiers/. Feel free to make a request there, although that project tends to only accept OSI-approved licenses.
Note that the License field is intended to be used for exactly this purpose, you may just want to set the license there rather than via a classifier.

Thanks @di - I overlooked the "License" field. Everything is fine now. Have a great day!


Incorrect Instructions

The instructions listed in the git do not match the files provided.

Put the Geolocation-Plugin Folder (containing the .aar and .gdap file) in the res://android/plugins folder in your project

There is no folder inside the project zip named "Geolocation-Plugin"
There is no .gdap file inside the project zip.

Am I missing something?

Thank you. There was an error in the instruction. You must put the .aar and .gdap directly files into the android/plugins folder (they should not be in another subfolder).

The Release zip file "GodotGeolocationPluginAndroid-v1.0.0.zip is the "Geolocation-Plugin" folder, but as mentioned above you have to put the contents of this folder in the android/plugins folder, not the folder itself.
There should be a .gdap file in the mentioned zip file.

You may also have a look at https://github.com/WolfBearGames/GeolocationTestApp where you find a small test app with the plugin already set up. You can browse the repository and see the files and their correct locations.

Thanks for the clarification. I put in a pull request with proposed documentation changes to be more clear.


Update post flair

Does this library have a way of modifying the flair of an existing post? I can't see anything in the code that would suggest as much with a peruse, though I wanted to find out if it was possible

There is no flair support in the API.
On Tue, Mar 17, 2020 at 8:43 PM Zero<EMAIL_ADDRESS>wrote:

Does this library have a way of modifying the flair of an existing post? I
can't see anything in the code that would suggest as much with a peruse,
though I wanted to find out if it was possible
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/turnage/graw/issues/69, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWR64EEDEQR7BJCGM4LKUDRIA7NFANCNFSM4LOEJJFA
.

In the Reddit API, or this wrapper? I know that some other Reddit clients like Boost that use the api support flair adding, so there must be some way to do that

The Reddit API supports it iirc, this does not.
Adding support for more calls is relatively simple: an interface needs to
be named to contain them, and the values need to be read/written through
JSON with Reddit.
Pull requests welcome.
On Wed, Mar 18, 2020 at 7:52 AM Zero<EMAIL_ADDRESS>wrote:

In the Reddit API, or this wrapper? I know that some other Reddit clients
like Boost that use the api support flair adding, so there must be some
way to do that
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/turnage/graw/issues/69#issuecomment-600668230, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWR64CLZZEALXNSJRKO6TLRIDNZZANCNFSM4LOEJJFA
.

Certainly, I'll get started on this when I have a minuted. I wanted to make sure that nobody else had already done it before I did


write_config.py key.split("[") ValueError: too many values to unpack (expected 2)

× home-manager-i.want.to.believe.service - Home Manager environment for i.want.to.believe
     Loaded: loaded (/etc/systemd/system/home-manager-i.want.to.believe.service; enabled; preset: enabled)
     Active: failed (Result: exit-code) since Thu 2024-05-09 16:57:40 CST; 63ms ago
    Process: 55979 ExecStart=/nix/store/l5kkmn3x65vq0vdiqk0w2k9irvi8fikh-hm-setup-env /nix/store/pqham57k40anijva65li87794srf36r9-home-manager-generation (code=exited, status=1/FAILURE)
   Main PID: 55979 (code=exited, status=1/FAILURE)
         IP: 0B in, 0B out
        CPU: 947ms

May 09 16:57:40 nixos hm-activate-i.want.to.believe[56550]:   File "/nix/store/hs7xcf977s8c5qz6kswif6blbs658k0d-write_config.py", line 228, in read
May 09 16:57:40 nixos hm-activate-i.want.to.believe[56550]:     key, value = ConfigValue.parse_line(l)
May 09 16:57:40 nixos hm-activate-i.want.to.believe[56550]:                  ^^^^^^^^^^^^^^^^^^^^^^^^^
May 09 16:57:40 nixos hm-activate-i.want.to.believe[56550]:   File "/nix/store/hs7xcf977s8c5qz6kswif6blbs658k0d-write_config.py", line 92, in parse_line
May 09 16:57:40 nixos hm-activate-i.want.to.believe[56550]:     key, marking = key.split("[")
May 09 16:57:40 nixos hm-activate-i.want.to.believe[56550]:     ^^^^^^^^^^^^
May 09 16:57:40 nixos hm-activate-i.want.to.believe[56550]: ValueError: too many values to unpack (expected 2)
May 09 16:57:40 nixos systemd[1]: home-manager-i.want.to.believe.service: Main process exited, code=exited, status=1/FAILURE
May 09 16:57:40 nixos systemd[1]: home-manager-i.want.to.believe.service: Failed with result 'exit-code'.
May 09 16:57:40 nixos systemd[1]: Failed to start Home Manager environment for i.want.to.believe.
warning: error(s) occurred while switching to the new configuration

The code generated by nix run github:pjones/plasma-manager | wl-copy has a lot of bugs. I deleted a lot of code under the configFile field and now it can be compiled successfully.

I think the error in your first screenshot might be unrelated to the arrays in your second screenshot.
The error in the first screenshot indicates it's failing to parse a config file that's already on your system (in particular, it looks like you somehow have a key in a KDE config file that contains more than one [ character). That's not supposed to be valid ([ is supposed to be escaped); do you think you could find the line in your on-disk config files that is failing to be parsed? If you're comfortable editing the source of write_config.py, you could have it print out the value of key right before that split line.
If it's plasma-manager that created the invalid config in the first place, we should definitely fix that generation, but I'm not too bothered by the failure to parse a seemingly invalid config file.

Yes, my screenshot is the code that I can build successfully after removing most of the code, the error is in the code that I removed,
Unresolved configurations should probably be better ignored and given a warning. I was lazy and just used nix run github:pjones/plasma-manager | wl-copy and adopted the code it generated until I encountered this problem.
It doesn't matter, I will investigate the specific configuration items that caused this problem later. I did find a lot of wrong configuration items before and deleted them.

in ~/.config/dolphinrc
[General][$i]
Version[$i]=202
ViewPropsTimestamp[$i]=2023,3,21,20,12,27.81

[MainWindow][Toolbar mainToolBar]
IconSize=16
ToolButtonStyle=IconOnly

I found that ~/.config/kcminputrc is causing this issue
[$Version]
update_info=kcminputrc_fix_botched_5_21_0.upd:kcminputrc_fix_botched_5_21_0_pre,kcminputrc_fix_botched_5_21_0.upd:kcminputrc_fix_botched_5_21_0,kcminputrc_repeat.upd:kcminputrc_migrate_key_repeat,delete_cursor_old_default_size.upd:DeleteCursorOldDefaultSize

[Libinput][1133][16500][Logitech G304]
PointerAccelerationProfile=1

[Libinput][1133][16519][Logitech G903 LS]
PointerAccelerationProfile=1

[Libinput][1739][52856][06CB0001:00 06CB:CE78 Touchpad]
NaturalScroll=true
PointerAccelerationProfile=1
ScrollFactor=0.75
TapToClick=true

[Libinput][9011][26214][ydotoold virtual device]
PointerAccelerationProfile=1

[Libinput.1133.16500.Logitech G304]
PointerAccelerationProfile=1

[Libinput.1133.16519.Logitech G903 LS]
AnnePro2 Mouse=
AnnePro2 Mousex5d=
PointerAccelerationProfile=1
AnnePro2 Mouse]=
[Libinput.1241.41619.OBINS OBINS=

[Libinput.9011.26214.ydotoold virtual device]
AnnePro2 Mouse=
AnnePro2 Mousex5d=
PointerAccelerationProfile=1
AnnePro2 Mouse]=
[Libinput.1241.41619.OBINS OBINS=

[Mouse]
AnnePro2 Mouse=
AnnePro2 Mousex5d=
PointerAccelerationProfile=1
X11LibInputXAccelProfileFlat=false
XLbInptAccelProfileFlat=true
cursorTheme=Future-dark-cursors
AnnePro2 Mouse]=
[Libinput][1241][41619][OBINS OBINS=


Returning relationships without the related object

I'd like to return a named relationship with an ID, but not the object itself.  For instance, if I have:
type Company struct {
	ID        string    `jsonapi:"primary,company"`
	Employees []*Person `jsonapi:"relation,employees"`
	Owners    []*Person `jsonapi:"relation,owners"`
}

I'd like to be able to return either Employees or Owners (or both) (or none) in the included array but in all cases return the relationships object with the correct ID.  Is there some way to do that?
Thanks!!

I'm trying to figure the same thing out at the moment, this is fairly old so unsure if this is still a problem for you but you could provide the links by implementing JSONAPIRelationshipLinks as shown in the examples:
func (blog Blog) JSONAPIRelationshipLinks(relation string) *jsonapi.Links {
	if relation == "posts" {
		return &jsonapi.Links{
			"related": fmt.Sprintf("https://example.com/blogs/%d/posts", blog.ID),
		}
	}
	if relation == "current_post" {
		return &jsonapi.Links{
			"related": fmt.Sprintf("https://example.com/blogs/%d/current_post", blog.ID),
		}
	}
	return nil
}

this won't give you the data field though (id/type) as you like, but should still comply with the spec as I understand it as it states it must include at-least one: http://jsonapi.org/format/#document-resource-object-relationships so the links should be enough to satisfy it and give the consumer a location to find it or the hint that it might be able to be specified with the include param.
Ideally I'd also like to include the id and type as well, but don't see a way of doing that currently.

I suppose the solution, if you have e.g. a one-to-many relationship is to create a struct named after the relationship, which implements the JSONAPILinks method which returns the self and related links, it should then only contain the primary,<name_of_relationship> and should be filled out by separate db calls which fetch only the relationship IDs.
This library sells it self on the fact that you shouldn't create glue structs though, so there should be an proposal to get this behavior built into the tags etc. Nevertheless, you still have to built some code that only fetches the IDs of a specific relationship.


Remove convert

Codecov Report
Base: 53.84% // Head: 52.94% // Decreases project coverage by -0.90% :warning:

Coverage data is based on head (cdc1f30) compared to base (0ad4bc5).
Patch coverage: 62.50% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
- Coverage   53.84%   52.94%   -0.91%
==========================================
  Files           5        5
  Lines         195      187       -8
==========================================
- Hits          105       99       -6
+ Misses         90       88       -2

Impacted Files
Coverage Δ

src/scalar.jl
23.52% <0.00%> (-9.81%)
:arrow_down:

src/derivative.jl
100.00% <100.00%> (ø)

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

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


getProfileCharacters are missing

It was really hard to test this function so I removed it from the final version. However it would be good to have it back.
Maybe with some tests :)

Of course this method is really hard to test because we need a valid access token from oauth process.
Anyway i added it back because maybe some projects were using it.
I created a test that assert it failed, but we should write a test that assert it passed, but i don't know how we could do that.
Any idea ?

I think the test suite should be remade entirely. At the moment we are connecting a third party service with every single request. This is problematic, because what happens if Blizzard API is not available for some reason? In this case our test will fail even though the code works exactly as it should.
Before I went for a longer hiatus I checked the possibility to test queries using a fake connection with the HTTP library. maybe push it to the next version.
For now the functionality is back so this can be closed.


Charge Station based Routing

Detail design for proposal https://github.com/Telenav/osrm-backend/issues/196#issuecomment-597884052
User story mapping

https://www.processon.com/view/link/5e6fec80e4b011fcce9965bb

Tasks list:

Prepare for charge station input https://github.com/Telenav/osrm-backend/issues/237
Build spatial index for charge stations based on Google::S2 https://github.com/Telenav/osrm-backend/issues/239
Generate connectivity for charge stations https://github.com/Telenav/osrm-backend/issues/238
Retrieve charge stations near orig and destination #240
Extend oasis service's graph - relax connectivity during running time #241
Extend oasis service's graph - multiple orig and destination #242


not all actors are being updated

I dont fully understand why, but when creating a new skill, only some of the actors are being updated.

hello, only owned actors will get the skills, did you read here? https://github.com/misthero/dnd5e-custom-skills/blob/main/README.md
is happening something different from that?

@misthero Yes, i figured it out.  basically what happens is that if you own any vehicles, it causes a JS error causing none of the other actors to fail.
Its attempting to assign skills to an actor that cant have them.  (the vehicle)

ive tried with a custom or non custom.  it seems to hit it regardless.  this is when adding skills.  i think vehicles dont have the skills.    I will post the JS error later today.

foundry.js:3856 TypeError: Cannot read properties of undefined (reading 'hasOwnProperty')
    at dnd5e-custom-skills.js:464:74
    at Array.filter (<anonymous>)
    at CustomSkills.addSkillToActors (dnd5e-custom-skills.js:464:43)
    at CustomSkillsForm.<anonymous> (dnd5e-custom-skills.js:97:24)
    at Generator.next (<anonymous>)
    at fulfilled (dnd5e-custom-skills.js:6:54)

all i have to do is make sure the vehicle isnt owned by anyone and it works, but if the vehicle is owned by anyone i get this.

Latest version should fix the bug.
Please update the module and let me know. Thank you.
https://github.com/misthero/dnd5e-custom-skills/commit/57f516ebb481f19c31f2b0ba101d72b11c202b58


[Gluon] F.pad(...) doesn't accept symbol input even in HybridBlock?

Hi, I am trying to dump a json string from this very nice style transfer model, written in gluon API.
I modified the net definition so that all blocks inherent from HybridBlock and use HybridSequential. The modified one is here.
I think this is enough to dump a json from gluon model. But when I try this,
    style_model = net.Net(ngf=args.ngf)
    x = mx.sym.var("x")
    y = style_model(x)
    y.save("style_model.json")

It gives me the following error:
Traceback (most recent call last):
  File "main.py", line 234, in <module>
    main()
  File "main.py", line 223, in main
    evaluate(args)
  File "main.py", line 146, in evaluate
    y = style_model(x)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\block.py", line 290, in __call__
    return self.forward(*args)
  File "D:\project\dev\incubator-mxnet\example\gluon\style_transfer\net.py", line 231, in forward
    return self.model(input)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\block.py", line 290, in __call__
    return self.forward(*args)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\block.py", line 481, in forward
    return self.hybrid_forward(symbol, x, *args, **params)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\nn\basic_layers.py", line 107, in hybrid_forward
    x = block(x)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\block.py", line 290, in __call__
    return self.forward(*args)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\block.py", line 481, in forward
    return self.hybrid_forward(symbol, x, *args, **params)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\nn\basic_layers.py", line 107, in hybrid_forward
    x = block(x)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\block.py", line 290, in __call__
    return self.forward(*args)
  File "D:\project\dev\incubator-mxnet\example\gluon\style_transfer\net.py", line 139, in forward
    x = self.pad(x)
  File "C:\Users\masah\Anaconda3\lib\site-packages\mxnet-0.12.0-py3.6.egg\mxnet\gluon\block.py", line 290, in __call__
    return self.forward(*args)
  File "D:\project\dev\incubator-mxnet\example\gluon\style_transfer\net.py", line 60, in forward
    return F.pad(x, mode='reflect', pad_width=self.pad_width)
  File "<string>", line 112, in pad
AssertionError: Argument data must have NDArray type, but got <Symbol x>

The error is occurring at here, inside HybridBlock.
Am I missing something, or is F.pad(...) indeed supposed to only accept NDArray inputs?
Thanks

@zhanghang1989

Sorry, I was not updating forward(...) to hybrid_forward(...).

Did you commit your updated hybrid forward function I am coming across the same issue

no.
Updating to hybrid_forward() did solve my original problem. But it turned out, for other reasons, this neural style transfer example can't work with Symbol input.

Thanks @masahi Did you find a style transfer that was fast that could take a Symbol input?

I tried the one here
Not as high quality as the gluon example, but it works.


Don't include non-running pods in node capacity check

Fixes #2582

https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/#pod-phase
Maybe we should rather filter out succeeded and failed pods, wdyt?

kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle Maybe we should rather filter out succeeded and failed pods, wdyt?

It's a good question - I think this would only help with Pending (since not sure what our behavior should be with Unknown) and pending might be a pod not starting and we might be still be able to run


Build is broken with current Master-Version of ImGui

The Commit "92b94980c69ff3d3d7f5dc30c1c19abfb72db47e" (https://github.com/ocornut/imgui/issues/1641) seems to have broken the build process for rlImGui.
Building with the mentioned commit leads to the following error:
❯ make
[ 55%] Built target glfw
[ 76%] Built target raylib
[ 78%] Building CXX object CMakeFiles/raylibcaster.dir/_deps/imgui-src/imgui.cpp.o
[ 81%] Building CXX object CMakeFiles/raylibcaster.dir/_deps/imgui-src/imgui_demo.cpp.o
[ 84%] Building CXX object CMakeFiles/raylibcaster.dir/_deps/imgui-src/imgui_draw.cpp.o
[ 86%] Building CXX object CMakeFiles/raylibcaster.dir/_deps/imgui-src/imgui_tables.cpp.o
[ 89%] Building CXX object CMakeFiles/raylibcaster.dir/_deps/imgui-src/imgui_widgets.cpp.o
[ 92%] Building CXX object CMakeFiles/raylibcaster.dir/_deps/imgui-src/misc/cpp/imgui_stdlib.cpp.o
[ 94%] Building CXX object CMakeFiles/raylibcaster.dir/_deps/rlimgui-src/rlImGui.cpp.o
/.../build/_deps/rlimgui-src/rlImGui.cpp:87:23: error: incompatible pointer to integer conversion assigning to 'ImTextureID' (aka 'unsigned long long') from 'Texture2D *' (aka 'Texture *')
   87 |     io.Fonts->TexID = fontTexture;
      |                       ^~~~~~~~~~~
/.../build/_deps/rlimgui-src/rlImGui.cpp:689:13: error: no matching function for call to 'ImGuiRenderTriangles'
  689 |             ImGuiRenderTriangles(cmd.ElemCount, cmd.IdxOffset, commandList->IdxBuffer, commandList->VtxBuffer, cmd.TextureId);
      |             ^~~~~~~~~~~~~~~~~~~~
/.../build/_deps/rlimgui-src/rlImGui.cpp:196:13: note: candidate function not viable: no known conversion from 'const ImTextureID' (aka 'const unsigned long long') to 'void *' for 5th argument
  196 | static void ImGuiRenderTriangles(unsigned int count, int indexStart, const ImVector<ImDrawIdx>& indexBuffer, const ImVector<ImDrawVert>& vertBuffer, void* texturePtr)
      |             ^                                                                                ~~~~~~~~~~~~~~~~
2 errors generated.
make[2]: *** [CMakeFiles/raylibcaster.dir/_deps/rlimgui-src/rlImGui.cpp.o] Error 1
make[1]: *** [CMakeFiles/raylibcaster.dir/all] Error 2
make: *** [all] Error 2

Fixed in c7303258e045b4698c3e0b930b71956b5bbe7dff
Thanks for the report.


Fix Storybook at CI

Describe the bug
Storybook doesn't build in CI. Seems to be something with the blog storage

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Azure Blob Github action updated. Also added a “Built at…” string on the front page (Introduction). This uses an environment variable that’s generated in the webpack config, and is only available in the preview part of the Storybook – meaning inside the iframe in the main content area. This makes it easier to see if the version of the storybook we’re actually looking it is the one we expect.
There seems to be an issue with the CDN. where it’s clinging to old files even though they have been purged. The direct URL is https://envisstoragedev.z16.web.core.windows.net until this issue is resolved.
Most of the stories fail to build, which seem to be related to breaking changes in the new version we‘re using.


Update authenticate-azure-cli.md

:x: Validation status: errors

File
Status
Preview URL
Details

docs-ref-conceptual/authenticate-azure-cli.md
:x:Error

Details

:x:Error

Details

docs-ref-conceptual/authenticate-azure-cli.md

[Error] Markup failed: One or more errors occurred..
[Warning] A space is required after the hash (#) in H1. See http://aka.ms/MarkdownHeadings for more information.

[Error] Markup failed: One or more errors occurred..

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.


Bug Report: source/PlayState.hx:1001: characters 1-3 : Expected } ERROR

I have a problem, when i try build the game putting ''lime test windows'' there is a messenge saying ''source/PlayState.hx:1001: characters 1-3 : Expected }, and when the export folders was made there is not the game, instead there is a ''Extension-webm.ndll'' there with nothing

Are you modifying the source? If so, Try copying and pasting a random expression within PlayState.hx and building YOUR expression upon it. It worked for me sometimes. By the way, an expression is when you do something like this:
if (FlxG.button.justPressed.SEVEN) {        FlxG.switchState(new ChartingState()); }


Silently failed to parse "attribute(packed)" in input header file

Due to the implcit  gcc -E -U __GNUC__ hardcoded in https://github.com/davidjamesca/ctypesgen/blob/dd1b3fd57c39071ded73c0d22befc03dcdfa691e/ctypesgen/parser/preprocessor.py#L144
any conditional code in the header files that is written only for GCC compiler is silently ignored by ctypesgen.
I noticed this as it resulted in few __attribute__((__packed__)) in my code being ignored for structs! 😅
Why is -U __GNUC__ required during ctypesgen's preprocessing stage?

Is there anyway to avoid it when invoking ctypesgen?

I don't know why this is/was done.  It was before I started on the project, and no one from those days is active on the project today.
My guess is because something didn't work when parsing the GNU headers. Also, it's not clear it would understand attribute "packed", even if that attribute were on. I suspect the meaning of "packed" potentially varies from platform to platform - which would make it harder to do correctly. Some machines can't easily handle byte-aligned integers (for example) - whereas others can, albeit more slowly.

As a quick experiment,
i modified line ctypesgen/ctypesgen/parser/preprocessor.py:144 to...
cmd += " -dD"
...and confirmed that such a patched ctypesgen
is able to properly parse and a update the resulting python code
with the following line each the struct's class.
struct_<struct-name>._pack_ = 1

With this i am able to get my Python code to work on the C struct properly. 🎉
Also confirmed that, the built-in testsuite in ctypesgen doesn't complain.

Would you be interested to merge the change if i submit a pull request for this?
(if accepted, this will avoid the need for me to maintain a patched ctypesgen,
and can ask users to simply install the release deb from here! 🙂)

I would be interested in seeing such a patch. Regarding whether I accept it, it depends on what I judge the risk level to be, and whether it fits well into the existing code base.

Pushed #100 for review.
As you can see, the fix is minimal.
It simply stops the pre-processor from silently stripping-away code defined within #ifdef __GNUC__.
With this in place, the existing code in ctypesgen now gets a chance to do the right thing. 🙂
As you rightly pointed out this may have major impact of the behavior.

the previous behavior of ctypesgen can be triggered if desired,
by explicitly specifying --cpp "gcc -E -U __GNUC__" when invoking  ctypesgen.
However, i believe the risk is minimal.
Even with this change, anyone interested in the previous behavior can trigger it.

I believe "-U GNUC" was added because ctypesgen had a hard time parsing advanced GNUC constructs in GCC header files that were spit out by the GCC compiler. This code is way back in 2007 (see commit) and the entire codebase was rewritten since then, so it's quite possible it's completely unneeded. I'll defer to Alan-R though.

Pushed PR https://github.com/ctypesgen/ctypesgen/pull/104 for review.

It retains the existing/legacy behavior as default.
i.e. ctypesgen implicitly undefines __GNUC__ while parsing code.

It introduces a new ctypesgen cmd-line param --allow-gnu-c,
that the user can specify to instruct ctypesgen to NOT undefine __GNUC__.


BUG: +action() doesn't work if the passed name contains upper-case

Description
When the name passed to the +action() mixin needs to be sanitized because it contains uppercase, k-Scaffold no longer finds the trigger associated with the action button when it is clicked. In the console, k-scaffold shows the following warning:
kScaffold Powered Sheet DEBUG| undefined act_<sanitized name>

Steps to reproduce
+action({name:"nameWithUpperCase", trigger: {triggeredFuncs: ["hello"]}})

+module.
    const hello = function() { k.log("Hello !"};

Looks like https://github.com/Kurohyou-Studios/k-scaffold/blob/61897632a8ee11a56d641a3792b1ee719fc0db37/lib/attribute_holders/_buttons.pug#L19 simply doesn't sanitize uppercase


Can't record my pygame opengl window

Hi, it's me again :D. I was trying to record my game to show to a friend of mine, but i can't record it. The recodings show only the first frame of the game they capture. I tried recording normal windows and everything works just fine. Do you have an idea how to fix it? maybe a special flag when making a window or something?

Hi! I used OBS on Linux to record the tutorial and it worked fine. Which software are you trying to record your game with?

OBS... Maybe that's because i am using the FULLSCREEN flag..

Ah, case fixed, i just removed the fullscreen flag lol

I just read a bit more about this. When you enable the fullscreen flag, your game is run in exclusive fullscreen mode, where your game has exclusive access to the display. This means that if you enable the flag, your system stops allocating resources toward displaying other programs, and your game gets rendered to the screen directly (instead of passing through the window manager). Therefore, if you are recording the entire screen with OBS, it won't record your fullscreen game.
I found two fixes:

Instead of recording the entire screen with OBS, go to Sources > Screen Capture, and choose your game.
Instead of using the fullscreen flag, you can have a borderless windowed screen that covers the whole screen, and it will essentially look the same as a fullscreen game. OBS should be able to record this no problem. Many games actually let you choose between "exclusive fullscreen" and "borderless window" in the options menu.

Thanks!


installation error

Running the command ionic cordova plugin add cordova-plugin-fcm-with-dependecy-updated return The system cannot execute the specified program, after Running command: install_ionic_dependencies.sh.
The scripts folder does not have the sh file.
Running: Windows 10, Ionic 5 in Command Prompt.

Solved using version 6.4.3, it does not have the .sh in the plugin.xml

install_ionic_dependencies.sh is bash, 'cause even on Windows, I've been running bash (https://git-scm.com/downloads) for almost a decade. As there is nothing too bash-specific there, I belive it could have a .bat version with the same code

This incompatibility is going to be addressed soon.

You can follow the issue #119


Workflow advance e2e

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Additional information
Special notes for your reviewer

Codecov Report

Merging #452 (6117da1) into main (1e2520f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #452   +/-   ##
=======================================
  Coverage   67.69%   67.69%
=======================================
  Files          23       23
  Lines        1746     1746
=======================================
  Hits         1182     1182
  Misses        492      492
  Partials       72       72

Impacted Files
Coverage Δ

agent/installer/installer.go
78.10% <0.00%> (ø)

agent/installer/bundle_downloader.go
69.66% <0.00%> (ø)

Closing in favor of #584


ERROR: Unable to extract OpenGraph title

ERROR: Unable to extract OpenGraph title; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.

Please call youtube-dl with the --verbose flag and include its complete output here.
Make sure you are using the latest version; type youtube-dl -U to update.

Voila,
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1511, in _real_extract
playlist_title = self._og_search_title(page)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 428, in _og_search_title
return self._og_search_property('title', html, **kargs)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 416, in _og_search_property
escaped = self._search_regex(self._og_regexes(prop), html, name, flags=re.DOTALL, **kargs)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 356, in _search_regex
raise RegexNotFoundError(u'Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract OpenGraph title; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 493, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 158, in extract
return self._real_extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1517, in _real_extract
r'(.*?)', page, u'title')
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 366, in _html_search_regex
res = self._search_regex(pattern, string, name, default, fatal, flags)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 356, in _search_regex
raise RegexNotFoundError(u'Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract title; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.

include its complete output here

It must start with something like [debug] System config: [], it gives as the required info for debugging the problem.

Voila
[debug] System config: [] [debug] User config: [] [debug] Command-line args: ['-i', '-t', '-w', '-c', '--verbose', 'https://www.youtube.com/playlist?list=PLq0g-O2tr8LVLVo53aNKmYopIb7MYasAW'] [debug] Encodings: locale 'UTF-8', fs 'UTF-8', out 'UTF-8', pref: 'UTF-8' [debug] youtube-dl version 2014.02.17 [debug] Python version 2.7.6 - Linux-3.13.0-24-generic-x86_64-with-LinuxMint-17-qiana [debug] Proxy map: {} [youtube:playlist] PLq0g-O2tr8LVLVo53aNKmYopIb7MYasAW: Downloading page #1 WARNING: [youtube:playlist] PLq0g-O2tr8LVLVo53aNKmYopIb7MYasAW: Playlist page is missing OpenGraph title, falling back ... ERROR: Unable to extract title; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update. Traceback (most recent call last):   File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1511, in _real_extract     playlist_title = self._og_search_title(page)   File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 428, in _og_search_title     return self._og_search_property('title', html, **kargs)   File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 416, in _og_search_property     escaped = self._search_regex(self._og_regexes(prop), html, name, flags=re.DOTALL, **kargs)   File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 356, in _search_regex     raise RegexNotFoundError(u'Unable to extract %s' % _name) RegexNotFoundError: Unable to extract OpenGraph title; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update. Traceback (most recent call last):   File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 493, in extract_info     ie_result = ie.extract(url)   File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 158, in extract     return self._real_extract(url)   File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1517, in _real_extract     r'
(.*?)
', page, u'title')   File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 366, in _html_search_regex     res = self._search_regex(pattern, string, name, default, fatal, flags)   File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 356, in _search_regex     raise RegexNotFoundError(u'Unable to extract %s' % _name) RegexNotFoundError: Unable to extract title; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.

@DarioGT you are using a 1 year old version, you have to update youtube-dl.

Tkx.  It is  the current version in ubuntu :(


Implementation of correct silhouette extraction

Current implemetation of the silhouette extraction is incomplete and produces incorrect silhouette lines.

Implementation using a brute-force method was completed. New implementation produces correct silhouette lines, but speed is slow, because visibility of every point on the edge lines are checked using AABB tree query.


Internal Validator - sequencer and stake manager error

Description:
For the past three weeks, we have been experiencing recurring errors on one of our validators, validator-002, in our production environment. We identified three specific errors in the logs, detailed below:
Error Logs:

Panic Error
panic: runtime error: slice bounds out of range [:32] with capacity 0

Frequency: ~10 occurrences per week

Failed to Run Sequence Error
failed to run sequence - validator manager init: height=17657995 error="getting voting power failed - backend is not initialized for height 17657995, fsm height 17657994"

Frequency: ~20 occurrences per week

Post Block in Stake Manager Error
polygon.server.polybft.consensus_runtime: failed to post block in stake manager: err="not found"

Frequency: Appears on every block sequence

These logs are from validator-002 for the time period from October 28th to November 4th.
How to Reproduce the Issue:
Below are the setup and resource details used to set up our Polygon Supernet, along with relevant environment details:
Infrastructure Setup:

Total Nodes: 7 Validators, 3 Non-validators on an internal network
Validator Configuration:

5 Validators in a private subnet (genesis validators)
1 Validator in a public subnet
1 External Validator hosted outside the VPC (connected via an RPC from a publicly exposed RPC node to an internal genesis validator)

Non-Validator Configuration:

2 Non-validators connected to a load balancer, used as RPC nodes
1 Non-validator connected to a block explorer

Resource Details:

Validator Instance Type: c6i.large
Non-validator Instance Type (RPC nodes): c6i.xlarge
Non-validator Instance Type (Block explorer): c6i.2xlarge
Operating System: Ubuntu 22.04 LTS
Polygon Edge Version: v1.0.0

Impact and Urgency:
These errors are impacting our production environment. The failed to run sequence error appears multiple times a week and has potential implications for validator stability. Additionally, the post block in stake manager error affects every block sequence, which is a significant operational concern.
Request for Assistance:
Could you provide any guidance on troubleshooting or potential fixes for these issues? If additional logs or specific configurations are needed, please let us know.
Thank you for your assistance!

@BlockgenStudio About point 2: This happens when a block is synced from another validator, and latestHeader.Number is updated just before a new sequence is initiated here:
sequenceCh, stopSequence = p.ibft.runSequence(latestHeader.Number + 1)
This occurs because latestHeader is a pointer. While there may be more elegant solutions to handle this case, I believe the Edge team intentionally left it to throw an error since it avoids adding additional logic (and computation) and works well in practice.
@Stefan-Ethernal, feel free to correct me if I’m wrong.

thanks @R-Santev what about the 1. and 3.?

especially the 3.


Feature request: get access to the exact text source (&str) behind a Span

In order to be able to upgrade the syn dependency of the rust-cpp crate ( https://github.com/mystor/rust-cpp/issues/41 ), one would need to be able to extract the raw source code for a given TokenTree (or a Span)
One could do that, I guess, by using TokenTree::span, and then getting the start/end and source_file of the span. But these function are "semver exempt". Even if they were not, that would mean we would need to recompute the offset in the file from line/column, which would be inefficient.
So what would be nice would be an start_offset and end_offset within Span.
Or better, a function that would return a String or a &str with the exact spelling.
TokenStream::to_string() does not work, as it adds spaces between token (that makes the resulting string invalid C++ as the two languages don't have the same grammar).
This is useful for crates, like rust-cpp, that allow to embed arbitrary languages within a rust macro (as long as the [], () and {} are balanced))

Seems like a plausible feature to me! To implement in this library though it needs to be implemented in upstream rustc first

I think https://github.com/rust-lang/rust/issues/54725 is the closest

Access to offset would also help my case where I use proc_macro2+syn completely outside of the compiler and I want to be able to get. Currently it seems I'll have to recover them from line+number (which is a shame since internally these are offsets in "fallback" implementation).
I'm probably using the tool the wrong way, though, not sure how useful my use-case is 😅

Yes, my use case is also unrelated to proc_macro: In the cpp_build crate. The use case is to extract the cpp! macro from the build.rs script.
Anyway, even upstream now has a way to get the source text: https://doc.rust-lang.org/proc_macro/struct.Span.html#method.source_text

It would be really useful for creating DSL inside rust language itself. I saw people use all kinds of tricks to get the source code (proc-macro-faithful-display, manipulating stringify!, heuristic string with line/column clue ...), I also created a hack macro to achieve the same goal.
I hope it can come along with a simple API, so I can make the below code work without much hack any more:
if run_cmd! {
    ls / | wc -w
    echo "bad cmd"
    ls -l /nofile
    date
}.is_err() {
    warn!("Run group command failed");
}

Closing in favor of consolidating the discussion to https://github.com/rust-lang/rust/issues/54725.


disable test on macos aarch64

This tests fails for me on MacOS and an apple silicon architecture

I am on ARM / Mac and it passes for me, are you sure?

I am on ARM / Mac and it passes for me, are you sure?

for me, it times out.


Fix formatting adding additional lines to the source code.

Purpose
Fix the issue with map formatting which adds invalid source line to the source code.
Fixes #17726
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Checked Tooling Support
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[x] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #17980 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #17980   +/-   ##
=======================================
  Coverage   15.16%   15.16%
=======================================
  Files          46       46
  Lines        1213     1213
  Branches      189      189
=======================================
  Hits          184      184
  Misses       1015     1015
  Partials       14       14

Continue to review full report at Codecov.

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


Privacy: Github OAuth now requires access to private email addresses to login

Recently the Codewars login procedure using GitHub OAuth changed. Codewars now requires permission to access private email addresses associated with GitHub. Without it, it's impossible to login. This was not communicated upfront, and it's impossible to login without it. I find this change unexpected, and prefer not to share private GitHub email addresses.

Sorry for the surprise and inconvenience.
A few days ago, I had updated OAuth libraries to fix a possible bug. When I was testing the update, I noticed that a user trying to sign up with their GitHub account must enter their email after linking, and found this awkward. I added user:email scope which is what's usually used for this flow, so the primary verified email is prefilled when signing up with GitHub account.
Sign in uses the same callback, so GitHub asks for permission to grant the user:email scope for existing users who signed up before the change. We look up the user with the returned GitHub uid, so we don't use the returned emails, but I understand your concern.

This was not communicated upfront

I honestly didn't think too much about the change because this is the scope I had assumed Codewars had before looking into this update. I didn't think much about how awkward it may appear for existing users, or some might refuse because they use different emails. It should be possible to use a different scope when signing in, so I can look into that.

and it's impossible to login without it.

For now, you can still log in with your email and password (if you've never set the password, you need to reset it).

I don't mind Codewars has access to my published public email (which it already had), but user:email gives access to (all) private email addresses, which discloses privacy sensitive information coupled to my Github account. That kind of defeats using a Github login, as I could have used a email + password combination for Codewars to begin with.
It would be useful to be able to change Codewars login to email + password if you've previously used Github login without a private email address, without having first to give Codewars access to your private Github email address(es).

I don't mind Codewars has access to my published public email

I don't see public email on your GitHub profile. Did you make it private?

user:email gives access to (all) private email addresses

Yeah, but unfortunately there's no way to get only the primary one, and user:email is the scope used by most apps for this flow (check other OAuth apps if you have any). We don't use other emails, but there's no way to prove that.

That kind of defeats using a Github login, as I could have used a email + password combination for Codewars to begin with.

I believe most users use GitHub login so that they don't need to enter email/password, so requiring them to enter their email was awkward and I wanted to fix.

It would be useful to be able to change Codewars login to email + password if you've previously used Github login without a private email address, without having first to give Codewars access to your private Github email address(es).

This should be possible already by clicking "Forgot your password?", and entering the email used for Codewars, use the password reset email to update the password, then sign in with the new password.
Again, I can look into changing the scope for sign in.

I didn't use a Github public email because of spam/privacy. That was previously OK with Codewars. Therefore I can't enter an email address for password reset on Codewars. Making it impossible to login without authorizing Codewars access to my private email addresses. As said, if this would have been the situation from the beginning, I would have made an email address specifically for Codewars.
If changing the scope for sign-in would prevent giving access to Codewars to private email addresses, I would prefer that.

Therefore I can't enter an email address for password reset on Codewars.

I don't understand this part, can you elaborate? You enter the email address you've always used for Codewars to reset password. We require email address even if you sign up with GitHub, so there has to be an email that you can enter.

if this would have been the situation from the beginning

Yeah, this is unfortunate and I'm sorry for the surprise.

If changing the scope for sign-in would prevent giving access to Codewars to private email addresses,

Yeah, if I can change it back to no scope when signing in, it won't affect the users who signed up before this change. I only changed it for the convenience of users signing up. The same scope is used because that's the default and I didn't think much.

We require email address even if you sign up with GitHub, so there has to be an email that you can enter.

Had to check that, you're right!

Closing as wontfix. This scope is widely used, and users can reject and switch to email/password as I wrote above.


Embedded dashboard charts "Applied Filter" not visible

Bug description
So for embedded dashboards, If we apply a filter there is no indication of Applied Filter on each chart unless the hideChartControls is set to false in dashboardUiConfig parameter passed to superset embedding sdk.
But i don't want to show chart controls, I just want to display what filters are applied on the charts. Is there a way to do it ?

How to reproduce the bug

Embed a dashboard with below dashboardUiConfig

{
        hideTab: true,
        hideTitle: true,
        hideChartControls: true,
        filters: {
          visible: false,
          expanded: false,
       }
}

Go to dashboard.

Screenshots/recordings
No response
Superset version
4.0.2
Python version
I don't know
Node version
I don't know
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

@dosu I am using @superset-ui/embedded-sdk pkg to embed dashboards.
Below is the code
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Superset Embedding POC</title>

@dosu Above configuration does not make applied filters visible on charts. Still they seem to be hidden

I believe this does indeed fall under chart controls. It could be broken out into a different setting, if you're interested in opening such a PR.


Consul: Fix liveness probe failure when HttpPort is overridden

Signed-off-by: Arunvel Sriram<EMAIL_ADDRESS>
Which issue this PR fixes

fixes #9521

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @unguiculus

/ok-to-test
/assign
/lgtm


docs: add helm chart scripts

You should separate the containers (app and mongodb), and I would go even further as to remove MongoDB and leave users to either use a managed option (Atlas or else), the MongoDB operator etc.
That would simplify your helm chart and leave all options to the org using the component.
This comment is applicable to all three components !


Additional Documentation

It would be nice to have a few examples without being in Laravel. For example... documentation revolving around the RequestBuilder

@cgsmith the documentation does now also cover basic use cases. For a deeper understanding take a look at the test directory. Please come up with any questions you have!

I forgot the original reason why I opened this request. I'll close it!


Remove available-typed-arrays patch

Used by walletlink
`--<EMAIL_ADDRESS>  `--<EMAIL_ADDRESS>    +--<EMAIL_ADDRESS>    | `--<EMAIL_ADDRESS>    `--<EMAIL_ADDRESS>      `--<EMAIL_ADDRESS>

This dependency is coming from walletLink, tested voting with it, works fine


Drawer README not showing example code

Describe the bug
No example is shown for using drawer in the README.md file.
https://github.com/matsp/material-components-vue/blob/master/components/drawer/README.md

Due to the enormous workload, I can't keep the docs in sync with changes in the code.
You can check https://github.com/matsp/material-components-vue/tree/master/docs/.vuepress/components/drawer for demos. Pull requests on docs are welcomed.

@tychenjiajun I can offer to complete some of the docs as I go with my project so that others can pick up this library faster.

fixed in #381