Emacs: Better racer-cmd default

Make racer-cmd in racer.el default to whatever racer is available in
the system PATH.
FIXES: #294

Pushed the PR by mistake, meant to:

attach it to #294
and
test it before sending.

But yeah. Will test in a bit if you don't beat me to it.

Yep it seems to work fine for me! :)

Great, thanks @moonlite


match_re() can unintuitively capture portions of context.input_dir or context.working_dir

If context.input_dir has a directory starting with . anywhere between its children and the current working directory (for example, .venv/Lib/site-packages/mysite/package_data), it will be filtered out by expressions like match_re(r'(.*/)*\..*') which are intended to exclude dot files only within the input_dir. We should offer an out-of-the-box method for users to limit what the regular expression processes to the contents of input/working directories.

Even worse, a rule like match_re(r'.*/static/.*') could end up matching entire input and working directories under unusual circumstances, or even the luck of a random directory name generation. We really need to be pushing towards external environment impacting matching as little as possible.


[Super AI] Build Main Pipeline for all data

Should be able to utilize all threads in 1 Node of Lanta

Output: JSONL
{
    "text": "Data",
    "source": "Source of the data",
    "source_id": "id of the original item in source data",
    "created_date": "Created date",
    "updated_date": "Updated date"
}

Does created_date means the date that original dataset was created krub ?

created_date mean day that original dataset is created if available, if not this will be the date that original dataset is obtained (ex. scraped from ... website on 2023 June 6)
for example:


Return odo analyze result as an array

What type of PR is this:
/kind feature
What does this PR do / why we need it:
The odo analyze command returns the result as an array.
For the mement, the result always contains a single Devfile  location.
Which issue(s) this PR fixes:
Related to  #5642
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[x] Documentation

How to test changes / Special notes to the reviewer:

cc @jeffmaury

/approve


Mailer replay fix - fixes #1514

The code that I submitted in #1514 is actually broken after the refactor in #1201. This fixes c7n-mailer-replay to actually work. It also adds the ability to dump the plain-text JSON from a base64-encoded, gzipped SQS message.

doh. thanks again, we probably need some more tests at some point. i've asked some of our internal devs to prioritize it.

we probably need some more tests at some point.

👀


http2: Adds a runtime feature that controls use of a new HTTP/2 implementation in Envoy

Commit Message: Adds runtime feature "envoy.reloadable_features.http2_use_oghttp2" guarding use of a new HTTP/2 implementation.
Additional Description:
Risk Level: Low -- runtime feature guarded, default disabled
Testing: Unit tests, integration tests.
Docs Changes:
Release Notes: Adds runtime feature "envoy.reloadable_features.http2_use_oghttp2" guarding use of a new HTTP/2 implementation.
Platform Specific Features:
Optional Runtime guard: envoy.reloadable_features.http2_use_oghttp2

/assign @adisuissa

/wait

Code changes LGTM. Please add release notes, and fix CI.

Done. Tests are passing locally, will check on the CI once it has some time to run.

Oh also format checks :-)
/wait

@birenroy is there a possibility of patching QUICHE upstream so we don't have to carry a patch in Envoy?

@birenroy is there a possibility of patching QUICHE upstream so we don't have to carry a patch in Envoy?

QUICHE is already fixed upstream. The patch is so I don't have to wait a week to demonstrate a clean CI.

CI is clean. Please take a look!

@birenroy seems like you need to merge main

@birenroy seems like you need to merge main

Alright, please take a look now.

@birenroy seems like you need to merge main

Alright, please take a look now.

Could someone please take a look?

/retest

@moderation would you mind giving this a dep-shepherd stamp

@moderation would you mind giving this a dep-shepherd stamp

This PR doesn't have any dependency changes to approve (which is interesting)

my bad - i was assigned previously when i think there were relevant changes

I had a patch attached to this pull request for a short time while I was waiting for some upstream QUICHE changes to be imported into Envoy. Sorry for the noise!

looks like this has merge conflicts?  /wait

/retest

CI is clean. Please merge, thanks!


FR: Be able to upload a file/directory from any url

Background & motivation
It would be great if we would be able to natively upload files into files artifacts/pvc from any arbitrary url.
This feature will be heavily used for shadowfork, where the previous state of a public testnet will need to be fetched from an s3 bucket (or something similar).
Desired behaviour
I should be able to use an s3 endpoint as url, and the file should be saved into an artifact/pvc that can be mounted to a service later.
How important is this to you?
Critical; Kurtosis is unusable for me without it.
What area of the product does this pertain to?
CLI: the Command Line Interface

thank you @barnabasbusa ! we have an area of work around making files artifacts a lot more accessible here, tracking this along with that work

Are files artifacts meant to store TBs of data?
Ideally we would want to have parallel sync from s3 buckets into pvcs. Maybe we shouldn't even bother with storing anything in files artifacts ?

Very interesting idea @barnabasbusa ; gets me thinking if we could treat files artifacts as references rather than something actually gets downloaded. Or, at minimum, when we pull in things from the web they could be (so that we do the expansion exactly like you're talking about). Ultimately, someone does have to download the data into the PVC, but maybe we should be streaming it (rather than doing a download-to-APIC-then-unpack).

when it's Terrabytes of data, you probably want to make sure you download the whole state before you proceed to the next step. Pull on request will work for small files, so will streaming.
My use case would be for files between 16GB-2.5TB

Related to #2215 and #2216 , in that all of them could benefit from being able to use "implicit packages" - Github repos that don't have a kurtosis.yml file.


add multiline test script

I added a test script that currently also does not parse completely right.
When we have open quotes, we should continue parsing things (and add the \n into the string.
When we have a \ escape, then the \n should not be part of the string.
Luckily it's easy to test this against bash or zsh behavior :)

Would be good to tackle https://github.com/prefix-dev/shell/issues/50, so that this is actually checked in CI

I think we can close this because of we now support building tests from scripts in scripts/ folder.


Make sure the LdapAuthenticatorConfigurationController is following the How-To-Write-Controllers guide

The LdapAuthenticatorConfigurationController needs to be refactored a bit so that it is following the How-To-Implement-Spring-Controllers. It looks like the toError method, which is stored in the BaseController, can be deleted after the refactoring. Use the appropriate getSuccessful/getFailed/getNotFound/getBadRequest/etc methods from the BaseController instead.

I'll take it!

There are a few metods that produce MediaType.APPLICATION_XML_VALUE. Should they still produce it?

getRolesMapping
getUserDnPatterns
getUserSearchFilter
getGroupSearchFilter

I'm not sure if that will be necessary. I would leave only the JSON as having output XML will do no good for the front-end. @carlspring thoughts?

I think these should be both in XML and JSON. What do you guys think?

I agree with @steve-todorov on this one.

Okay then, please proceed as you find fit!


Helm chart command removed from helm 3.7 cli, update page to reflect on helm cli 3.7 release

The helm chart command removed from helm CLI release 3.7

Notable Changes
A few major changes were made to the experimental OCI feature:
helm chart export has been removed
helm chart list has been removed
helm chart pull has been replaced with helm pull
helm chart push has been replaced with helm push
helm chart remove has been removed
helm chart save has been replaced with helm package
In addition, a change to the experimental OCI manifest format was introduced, preventing Helm 3.7 from fetching charts built
with older versions of the experimental OCI feature. Users will need to re-package and upload their OCI charts when upgrading > to Helm 3.7. This change does not affect standard Helm charts.
Many of the details of these changes can be found in HIP 6.

To push a helm chart to ACR:
helm create test
helm package test
Successfully packaged chart and saved it to: /home/bap/test/test-0.1.0.tgz
helm push /home/bap/test/test-0.1.0.tgz oci://xyz.azurecr.io/helm/test

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

ID: 59c4be11-ec02-7160-e65b-2218bfd70f68
Version Independent ID: 19e08339-390f-ccfa-885c-2643722bd186
Content: Store Helm charts - Azure Container Registry
Content Source: articles/container-registry/container-registry-helm-repos.md
Service: container-registry
GitHub Login: @dlepow
Microsoft Alias: danlep

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@bplasmeijer - Thanks very much for raising this issue and noting the differences in Helm 3.7. We'll update the article soon.

@bplasmeijer - Thanks very much for raising this issue and noting the differences in Helm 3.7. We'll update the article soon.

👍 Question, can I do doc PR?

The Helm article has been updated for Helm 3.7.1 commands, so I am closing this issue. Thanks again for contacting us.
#please-close


fix: Use new useragent and token for mobile

It's the fix that helped me with { login: 'xx', reason: 'old_version', status: 'fail' } error.
https://github.com/WhiskeySockets/Baileys/issues/241#issuecomment-1633285969

The version here is higher than in one PR that is open right now
https://github.com/WhiskeySockets/Baileys/pull/227


Add CSS content-type

Codecov Report

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

@@           Coverage Diff           @@
##           master     #918   +/-   ##
=======================================
  Coverage   82.73%   82.73%
=======================================
  Files          52       52
  Lines         776      776
  Branches       36       36
=======================================
  Hits          642      642
  Misses        134      134

Impacted Files
Coverage Δ

core/src/main/scala/io/finch/package.scala
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM!

I wonder what is a criterion to choose predefined media types? I found a list of types and it's huge.

Adding as needed 😉
Will open a PR for image SVG soon

What do you think about to add all registered types at once? If the only usage is like constants then it would be straightforward.

Good point . I'll add them tomorrow ( already late in Germany )

I'm not sure we want to be in the business of maintaining a huge list of content types. Especially, given that vendor content-types are thing, it might be better to document how to create custom types in the Cookbook section of the docs.

I agree on limiting the content types added. I would extend this PR with the important content types listed on Apache MDN.
IMHO the basic content types should already be provided.

I think I don't mind including these new types but I'd probably argue we might want to move all of them to a separate file (i.e., contentTypes.scala), while also including a link to that Mozilla doc.

Sounds reasonable. Should this be a trait and the package object extends it, or can it be a breaking change and live in a separate object?

We can just define those as top-level objects within a new file, i.e., contentTypes.scala. This should classify as a source compatible change so we should be fine.

Merging this. Thanks @muuki88!

You're welcome 🤩


[SPARK-10852][PySpark][SQL] Override built-in methods for special column names

JIRA: https://issues.apache.org/jira/browse/SPARK-10852
For few special columns such as count and index, because they are built-in methods of Row(tuple) in Python. We should override them in order to properly access the values.

Merged build triggered.

Merged build started.

Test build #43080 has started for   PR 8934 at commit 0140148.

Test build #43080 has finished for   PR 8934 at commit 0140148.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43080/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #43096 has started for   PR 8934 at commit 62c2b44.

Test build #43096 has finished for   PR 8934 at commit 62c2b44.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43096/
Test PASSed.

ping @davies

@viirya It will be great we can fix it magically. I'm worried that the current approach will introduce some performance regressions.
As we always have a way to workaround it using row["count"] (similar to escape column names in SQL), so it's not a blocker for uses.

@davies Yes. Currently as reported in the JIRA, using x.asDict()["count"] instead of x.count can be to work around this. But I think it would be better to avoid this.
I updated this patch that should not introduce performance regressions. Please review it if you have time.

Merged build triggered.

Merged build started.

Test build #43317 has started for   PR 8934 at commit f7bdde3.

Test build #43317 has finished for   PR 8934 at commit f7bdde3.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43317/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #43325 has started for   PR 8934 at commit d454512.

Test build #43325 has finished for   PR 8934 at commit d454512.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43325/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #43328 has started for   PR 8934 at commit 05fb9e6.

Test build #43328 has finished for   PR 8934 at commit 05fb9e6.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43328/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #43338 has started for   PR 8934 at commit f48ad0d.

Test build #43338 has finished for   PR 8934 at commit f48ad0d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43338/
Test PASSed.

Merged build triggered.

Merged build started.

Test build #43513 has started for   PR 8934 at commit ded6853.

Test build #43513 has finished for   PR 8934 at commit ded6853.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43513/
Test PASSed.

Merged build finished. Test PASSed.

ping @davies How about this updated version? Or you still think it is not worth due to the possible performance regressions? As in this version it only looks for count and index in the list __fields__ in __setattr__, the performance impact can be ignored?

ping @davies Any more comments? Or should we close this pr?

@viirya This PR only help a two corner cases, and they are not blocker (could be easily workaround), I'd like to not fix these (avoid the complexity). If we merge this one, some one may continue to submit similar PR to address other corner cases.
BTW, There is a JIRA to suggest user to prefer using c['column'] instead of c.column.
Does this make sense?

@davies Yes. Thanks. I close this one now.

BTW, There is a JIRA to suggest user to prefer using c['column'] instead of c.column.
@davies, we encountered similar issue, could you paste the JIRA number for reference?


testPolygonCircle() Acting Wierd

Something wrong is going on with the testPolygoCircle() function. Help! I'm trying to rotate a box (a bullet for my game) to hit a circle (a tree stump). It works fine until I use the setAngle() function for my box. Here is a fiddle showing an example of what is going wrong.https://jsfiddle.net/sharedtakin299/sdghn86b/23/ Please tell me what I'm doing wrong.

maybe because setAngle should accept angle value instead of radians
box.setAngle(50);

maybe because setAngle should accept angle value instead of radians
box.setAngle(50);

No, it's not that because the README.md said to use the polygon.setAngle() with radians.

The rotate method of the Canvas2D context rotates in a clockwise direction, while the setAngle (and rotate) methods of SAT's Polygon rotate in a counter-clockwise direction.
If I change your example to do:
box.setAngle(-50*Math.PI/180); // rotate in clockwise, rather than counter-clockwise

It shows up green

Also I think that the way you're drawing the box and circle may not map with what the actual coordinates of the SAT objects are.
var b = {x:100,y:100,w:100,h:100};
var response = new SAT.Response();

// Make boxes and set angle to 45 degrees
var box = new B(new V(b.x,b.y),b.w,b.h).toPolygon();
box.setAngle(50*Math.PI/180);

This will rotate the box around its corner, not the middle, while when you're drawing the box, you're translating the context to be at the box's middle before rotating it.
ctx.save();
ctx.translate(b.x+b.w/2,b.y+b.h/2);
ctx.rotate(50*Math.PI/180);
ctx.translate(-b.x-b.w/2,-b.y-b.h/2);
ctx.fillRect(b.x,b.y,b.w,b.h);
ctx.restore();

Also I think that the way you're drawing the box and circle may not map with what the actual coordinates of the SAT objects are.
var b = {x:100,y:100,w:100,h:100};
var response = new SAT.Response();

// Make boxes and set angle to 45 degrees
var box = new B(new V(b.x,b.y),b.w,b.h).toPolygon();
box.setAngle(50*Math.PI/180);

This will rotate the box around its corner, not the middle, while when you're drawing the box, you're translating the context to be at the box's middle before rotating it.
ctx.save();
ctx.translate(b.x+b.w/2,b.y+b.h/2);
ctx.rotate(50*Math.PI/180);
ctx.translate(-b.x-b.w/2,-b.y-b.h/2);
ctx.fillRect(b.x,b.y,b.w,b.h);
ctx.restore();

If you also do
box.setOffset(-50, -50);

I think it will do the same as what your drawing code is

I hate to bother you, but it still is not working. I implemented the two things you told me, but it is doing the exact opposite now. Here is my updated JSFiddle   https://jsfiddle.net/sharedtakin299/sdghn86b/27/  I used the box.setOffset() and switched the rotation to clockwise, but it still is not working. I'm confused. Am I still doing something wrong?

🤦 Sorry, it's clearly been a while since I've looked at this code - the setOffset method takes in a Vector, so it would be box.setOffset(new SAT.Vector(-50,-50). You'll also have to adjust the pos of the box (but not your current drawing code) to account for the offset (e.g `box.pos.x += 50; box.pos.y += 50;)
Here are two examples with it working:
Not Colliding: https://jsfiddle.net/get2jkx1/
Colliding (added 10 to the x position so they intersect): https://jsfiddle.net/get2jkx1/1/

🤦 Sorry, it's clearly been a while since I've looked at this code - the setOffset method takes in a Vector, so it would be box.setOffset(new SAT.Vector(-50,-50)). You'll also have to adjust the pos of the box (but not your current drawing code) to account for the offset (e.g box.pos.x += 50; box.pos.y += 50;)
Here are two examples with it working:
Not Colliding: https://jsfiddle.net/get2jkx1/
Colliding (added 10 to the x position so they intersect): https://jsfiddle.net/get2jkx1/1/

Thanks so much! Man was I confused!

Hey @jriecken
I have the same issue but for me that solution not works until now, maybe im doing something wrong, can you help me?
In my case, the polygon is a projectile that moves until it collides with a circle (player).
that's my projectile/bullet code on server side:

The movement is working well, also the angle too... but the collision not.
In your example, you update the pos with x += width / 2, y += height / 2 right?
when i try this in my updateShape function, it messed up, propably because of the movement in a specific angle.
In this case, how can i do this work?
Im using PIXI.js on clientside and i reproduce all the positions and angle from server side in my projectile in client side
as i said before, it works well, but the collision not.
I made a video to show you:
https://clipchamp.com/watch/AAw4d7YYzws
In clientside the projectile is a Sprite with anchor set to 0.5


IMaterializationInterceptor instance registration info

Please make developers aware that the IMaterializationInterceptor should be registered with a single (static) instance to avoid Microsoft.EntityFrameworkCore.Infrastructure.ManyServiceProvidersCreatedWarning issues.
See https://stackoverflow.com/questions/75601346/adding-interceptor-in-dbcontext-gives-microsoft-entityframeworkcore-infrastructu

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

ID: 3e828f66-5f1f-864e-f5aa-e90fa101fbf6
Version Independent ID: 3e828f66-5f1f-864e-f5aa-e90fa101fbf6
Content: IMaterializationInterceptor Interface (Microsoft.EntityFrameworkCore.Diagnostics)
Content Source: dotnet/xml/Microsoft.EntityFrameworkCore.Diagnostics/IMaterializationInterceptor.xml
Product: ef-core-api
GitHub Login: @dotnet-bot
Microsoft Alias: avickers

See also #4082.


List of open bounties on Discord is different from the bounty board

Describe the bug
Hi,
The open bounties list on the discord is different from the ones on the website.
To Reproduce
Steps to reproduce the behavior:

Go to https://bountyboard.bankless.community/
At this time (13/02/22, 07:06 UTC), there are 3 open bounties : Bounty Board UX review, "Join the Translators Guild" and Join Employment Commons

By contrast, in the #🧀-bounty-board channel in the discord, when I do the "/bounty list list-type:open" command, BountyBot lists me theses :

The only common task is the "Bounty Board UX review".
Expected behavior
See the exact same list of open bounties
Screenshots
See the description above
Desktop (please complete the following information):

OS: Windows 10 Professionnel 21H2 64bits
Browser: Brave
Version: 1.35.101 (Chromium: 98.0.4758.87)

Additional context

Discord version : Stable 114407 (811ab09)

Thank you @tsioureth, we think this was due to a renaming of a field in some of the bounties. PR #201 should address this

Fixed in #201


Use immaterial extension for apidoc gen

This PR uses the immaterial extension with Sphinx to generate API docs.
To do:

Find a fix for the warnings that are ignored in by nitpicky in conf.py
Fix additional warnings during tox build.

/home/runner/work/ansible-sdk/ansible-sdk/ansible_sdk.AnsibleJobDef:1: WARNING: Parameter name 'data_dir' does not match any of the parameters defined in the signature: []

Things like this seem to be happening to data classes where the :param: entries are declared in the class docstring, not in __init__() (which is not there). The docs don't mention anything special about the dataclasses, nor do the GitHub issues. I suppose one could file a feature request to infer an initializer signature from the attributes but currently, the workaround could be just documenting those attributes.
For example, https://github.com/ansible/ansible-sdk/blob/main/ansible_sdk/model/job_def.py could have a class changed as follows:
 @dataclass(frozen=True, kw_only=True)
 class AnsibleJobDef(_DataclassReplaceMixin):
-    """
-    Common Ansible job definition values
-    :param data_dir: path to a directory structure containing the Ansible project content and inventory.
-    :param playbook: relative path or FQCN of a playbook to run under ``data_dir``
-    :param inventory: relative path to inventory file(s) or directory under ``data_dir``, analogue to Ansible's ``-i`` option
-    :param extra_vars: dictionary of variables for highest precedence level, analogue to Ansible's ``-e`` option
-    :param verbosity: None for default verbosity or 1-5, equivalent to Ansible's ``-v`` options
-     """
+    """Common Ansible job definition values."""
     # currently analogue to runner's private_data_dir; do we want to construct this ourselves?
     data_dir: str
+   """Path to a directory structure containing the Ansible project content and inventory."""
+
     # relative path to playbook in data_dir or FQCN
     playbook: str
+   """Relative path or FQCN of a playbook to run under ``data_dir``."""
+
     inventory: t.Optional[t.Union[str, list[str]]] = None  # FUTURE: high-level inventory types?
+   """Relative path to inventory file(s) or directory under ``data_dir``, analogue to Ansible's ``-i`` option."""
+
     extra_vars: dict[str, t.Any] = field(default_factory=dict)
+   """Dictionary of variables for highest precedence level, analogue to Ansible's ``-e`` option."""
+
     verbosity: t.Optional[int] = None  # None or 1-5
+   """:data:`None` for default verbosity or 1-5, equivalent to Ansible's ``-v`` options."""

The above example uses attribute docstrings (check out PEP 224, PEP 257, PEP 258 for the origins of this approach). Sphinx's apidoc supports them natively.
But there's another syntax that Sphinx supports for documenting attributes — comment-like strings that have a leading #:  and are supposed to be on the previous or the same line, unlike attribute docstrings that go on the line following the attribute declaration (https://www.sphinx-doc.org/en/master/usage/extensions/autodoc.html#directive-autoattribute).
Here's an example of using this syntax instead:
 @dataclass(frozen=True, kw_only=True)
 class AnsibleJobDef(_DataclassReplaceMixin):
-    """
-    Common Ansible job definition values
-    :param data_dir: path to a directory structure containing the Ansible project content and inventory.
-    :param playbook: relative path or FQCN of a playbook to run under ``data_dir``
-    :param inventory: relative path to inventory file(s) or directory under ``data_dir``, analogue to Ansible's ``-i`` option
-    :param extra_vars: dictionary of variables for highest precedence level, analogue to Ansible's ``-e`` option
-    :param verbosity: None for default verbosity or 1-5, equivalent to Ansible's ``-v`` options
-     """
+    """Common Ansible job definition values."""
     # currently analogue to runner's private_data_dir; do we want to construct this ourselves?
+   #: Path to a directory structure containing the Ansible project content and inventory
     data_dir: str
+
     # relative path to playbook in data_dir or FQCN
+   #: Relative path or FQCN of a playbook to run under ``data_dir``
     playbook: str
+
+   #: Relative path to inventory file(s) or directory under ``data_dir``, analogue to Ansible's ``-i`` option
     inventory: t.Optional[t.Union[str, list[str]]] = None  # FUTURE: high-level inventory types?
+
+   #: Dictionary of variables for highest precedence level, analogue to Ansible's ``-e`` option
     extra_vars: dict[str, t.Any] = field(default_factory=dict)
+
+   #: :data:`None` for default verbosity or 1-5, equivalent to Ansible's ``-v`` options
     verbosity: t.Optional[int] = None  # None or 1-5

I haven't tested these but I'm pretty confident that such a workaround should address the corresponding warnings.

I haven't done an in-depth review of these changes, but looking at the resultant output gets us back to where we need to be, so I'm a big +1 for the approach.

But there's another syntax that Sphinx supports for documenting attributes — comment-like strings that have a leading #:

@webknjaz thanks for that one- I didn't do my research to find this syntax when I added those docstrings, but it sounds like exactly what I was wishing for. I'd much prefer the docstrings to be attribute/property-adjacent like that than having them in a block at the top. We don't need to address it in this PR, but I'd be strongly in favor of doing it that way if it doesn't cause other problems.

FWIW I personally tend to prefer attribute docstrings rather than the Sphinx hack with #:.

We don't need to address it in this PR

If the warnings were caused by this change, it'd be reasonable to fix it here. If this is also happening with the current apidoc integration, then definitely in a separate PR. I haven't checked if they're already present on the default branch.

Update on this PR. I propose that we create two conf.py files and have separate builds. One build for concepts and procedures (docs/source/conf.py). One build for API reference generated from docstrings (docs/api_reference/conf.py). IMO a separate build for the API reference content will make it easier to have a single-source for community and downstream.
I also propose that we fix the remaining warnings in a different PR because that requires changes to the docstrings.
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.AnsibleJobDef:1: WARNING: Parameter name 'data_dir' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.AnsibleJobDef:1: WARNING: Parameter name 'playbook' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.AnsibleJobDef:1: WARNING: Parameter name 'inventory' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.AnsibleJobDef:1: WARNING: Parameter name 'extra_vars' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.AnsibleJobDef:1: WARNING: Parameter name 'verbosity' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.executors.AnsibleDockerJobOptions:1: WARNING: Parameter name 'container_image_ref' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.AnsibleJobStatusEvent:1: WARNING: Parameter name 'event' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.AnsibleJobStatusEvent:1: WARNING: Parameter name 'event_data' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.AnsibleJobStatusEvent:1: WARNING: Parameter name 'stdout' does not match any of the parameters defined in the signature: []
/home/dnaro/git/dnaro-sdk-build-api-docs/ansible_sdk.executors.AnsiblePodmanJobOptions:1: WARNING: Parameter name 'container_image_ref' does not match any of the parameters defined in the signature: []


Pinpoint monitoring showed that the system CPU and the actual mismatch

As shown in the following figure, I saw pinpoint showed the system CPU utilization rate reached 100%, but I looked at the monitoring data of the machine, and the use of top command to see for a long time, the actual system CPU has not significantly increased not to say 100%, I want to ask, what is the general reason for this?

Pinpoint performance information uses JDK standard MBean API, so Perf data is accurate.
https://github.com/naver/pinpoint/blob/master/profiler-optional/profiler-optional-jdk7/src/main/java/com/navercorp/pinpoint/profiler/monitor/metric/cpu/oracle/DefaultCpuLoadMetric.java#L39

This issue may be the monitoring granularity problem.
Pinpoint collects information every five seconds, so it can catch more accurate information.
Why One Second Matters When Monitoring Applications
https://www.instana.com/blog/one-second-matters-monitoring-applications/

Even if the monitoring value time is different and the system CPU is at 100% for a long time, our monitoring system cannot see any abnormality. We have carefully checked and found that there is no other problem with the system itself and there is no application that USES too much CPU. The difference between 1% and 100% is very obvious, don't you think?


Unskip Build_WithViews_ProducesDepsFileWithCompilationContext_ButNoReferences

System.Text.Json has had several fixes since this issue was originally reported. We can skip this test if it continues to be flaky.

/azp run all


Tablet Optimization

https://app.asana.com/0/1135320259326480/1152461875859617/f
https://app.asana.com/0/1135320259326480/1161696655970817/f

Looks good


Minecaft always on top of other windows.

The current mod is always on top of other windows so I can't really switch between programs with this easily. Hope there is a simple fix.

Oh, you’re already here. So, no element of surprise then, huh?
Yep, this is by design. The fullscreen mode so exclusive, it could have become the head of the RNC.
I didn’t know the limits of what’s acceptable in your holy wars against input lag, and what’s not, so context switching is also disabled. If this is a wanted feature, I’ll make it into a two-lines-of-code mod. Give me a moment.

You can get the updated mod before I push the actual release here: https://github.com/Kir-Antipov/cubes-with-borders/actions/runs/9809845369.
Let me know if there's something else you need :)

Works perfectly thanks :)
Also I just wanted to say that my assumption was wrong that you can have a v-sync free window. I'm not sure which games utilize it but I realized after trying the Latency Split Test program again, which even in its normal windowed mode has visible screen tearing, which only goes away when I play a video or have some kind of movement on that screen. Maybe you could find out how that works?
You can find the program on the official aperturegrille website but I don't think it's open source.

Works perfectly thanks :)

Okie-dokie, gonna publish a new release then. Just to reiterate, unless Mojang suddenly decides to port Minecraft to Vulkan or something, this mod should work forever. Unfortunately, Modrinth and CurseForge don't allow specifying version ranges in the compatibility matrix, so I still need to manually update the list of supported Minecraft versions on both platforms for your launcher to allow installing the mod on the latest game update. Feel free to ping me whenever it needs to be done if I forget to do it myself (I will) :)

Also I just wanted to say that my assumption was wrong that you can[not?] have a v-sync free window.

Well, it's not, really. DWM has worked this way since it was introduced with Windows 7+. For it to work differently, you would need to create a primary buffer around the floating window in question, instead of the whole desktop. I don't think Windows does this under any circumstances. At least, there's no documentation on the existence of such a feature. So, as far as I'm concerned, it doesn't exist :D


[GOBBLIN-285] Kafka extractor computes avgMillisPerRecord even when a partition pul…

…l is interrupted
Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-285

Description

[ ] Here are some details about my PR, including screenshots (if applicable):

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@htran1 can you review?


bug:  "status": "409 Conflict", not creating state file, both users and groups appeared to be synced in AWS SSO console

Hi,
I'm sort of new to this, I've been doing the manual provisioning in AWS SSO with google workspace directory for a while. I decied to set up the auto provisioning, went through the steps with AWS Serverless Applications. After the first sync, I can see all the users and groups synced up, but all the groups are empty and and none of the users is in their groups.  I only had 2 groups (regular users and admin users), From the logs, i can see it sees the group (AWS-SSO) has 69 users. It tried to create them, but it runs into 409 conflict saying there's duplicate users already. I didn't remove all my original users and groups before turning on auto-provisioning, is that a problem?
{
"idp": 69,
"level": "info",
"msg": "reconciling users",
"scim": 50,
"time": "2022-02-14T17:37:21Z"
}
{
"level": "warning",
"msg": "creating users",
"quantity": 23,
"time": "2022-02-14T17:37:21Z"
}
{
"level": "warning",
"method": "POST",
"msg": "aws checkHTTPResponse: body: {"schema":["urn:ietf:params:scim:api:messages:2.0:Error"],"detail":"Refused to create a new, duplicate resource.","status":"409","exceptionRequestId":"279ec396-441a-49f8-b1b6-a4e04c08348c","timeStamp":"2022-02-14 17:37:21.224"}\n",
"status": "409 Conflict",
"time": "2022-02-14T17:37:21Z"
}
Error: cannot sync groups and their members: error doing the first sync: error reconciling users: error creating users in SCIM Provider: scim: error creating user: aws CreateUser: error decoding response body: EOF
To reproduce the problem:
I created a new group, added a brand new user, still getting the same errors in the log, it created the group and user, but user isn't showing as part of the group members on AWS, and there's still no state file being created.

@jchiang-genesis to try to understand the problem you have, please could you remove manually the state file from you S3 bucket and resync?
Remove the state file is a totally save action and it is the only way to force the reconciliation between the identity provider and scim side.
After that, please share the results without share information about your users and credentials

Hi @jchiang-genesis thank you for the information
idp-scim-sync only sync `groups and their members, so I recommend to you read this document https://github.com/slashdevops/idp-scim-sync/blob/main/docs/Using-SSO.md
apart from that, from your explanation, I can detect a corner case bug and I will validate this, let me see if I can.

@jchiang-genesis I'm simulated your scenario without any success, I mean without any error, so maybe is the version of the app you have deployed.
could you deploy the latest version?

Hi @christiangda , I actually just deployed it yesterday, I'm not sure where I can check the version, when i checked the deployment, below is the template version, I can try re-deploying the lambda app later again and update you here.
AWSTemplateFormatVersion: '2010-09-09'
Transform: AWS::Serverless-2016-10-31
Description: |
This is used to keep your AWS Single Sign-On (SSO) groups and users in sync with your
Google Workspace directory using and AWS Lambda function.
Project URL: https://github.com/slashdevops/idp-scim-sync
As for the group membership not getting synced issue, I've attached the screenshots from both google directory and aws sso, so Jay is in AWS_Admins group on google, but not in AWS SSO. Cause I went through your demo documentation, it looks like the group membership should get synced too.

I will try to use this new AWS SSO Identity Store API through the go SDK to find a way to mitigate better this issue.
so @jchiang-genesis this is not a bug , it is a limitation in the AWS SSO SCIM API, sorry

@christiangda not a problem, that's why when the reconciling users, it was showing idp has 69 and scim has 50. I did look up the issues from the old ssosync, so If i can limit to only 50 users for the first sync, it creates the state file, I can then add on more users afterwards as long as the new users won't be more than 50 users incremental?

Thank you @christiangda for your help , just want to update everyone, once I cut down the group members to 48 for the initial sync, it worked as expected. But I did have to delete all my AWS SSO existing users and groups. then I was able to add back all the users.


Biber will crash if there are any entries with an extra comma in the author field

Biber will crash if there are any entries with an extra comma in
the author field. For example,
author = {Sagan, , Carl},

causes Biber to segfault with
FATAL - Caught signal: SEGV
Likely your .bib has a very bad entry which causes libbtparse to crash: Inappropriate ioctl for device

and it must be corrected to
author = {Sagan, Carl},

bibtex-mode in Emacs reports that the construct is syntactically valid though.

thank you @lrcg - you gave me the right hint to find my biber SEGV problem :+1:

@lrcg Is run biber with --trace and read the trace report in .blg file solve your problem?


Fixes #63530 Update certauth.md

Fixes #63530
There has some extra setup needed for Azure Web Apps, the validation will fail with the "The system cannot find the file specified" error if they're not supplied.

Hello @Ted-chiptech ... Thanks for sending this in. That issue number for the "Fixes" isn't correct.
@Rick-Anderson ... Indeed, this is fairly close to what we needed for certs and custom domains for Blazor hosted WASM on Azure Apps with IdS and a custom domain/key signing cert (scroll down to steps 5 and 6), but wouldn't the key setting for app settings be WEBSITE_LOAD_CERTIFICATES with the cert thumbprint as the value (separated by commas if there are more than one)?

@guardrex Thanks for look at this.
Sorry, I just realize that the issue was raised in a different repository(https://github.com/MicrosoftDocs/azure-docs/issues/63530)
Should I remove the Fixes instead?
And I guess my case is a little bit different maybe? I will need to read a certificate from Azure KeyVault, then do a customized validation to check if the inbound certificate's thumbprint matches.
So I have a function hook to OnCertificateValidated event to do the check, and the function contains some code to load the certificate from KeyVault to check if they match, and fail the context if they don't.
Should I use WEBSITE_LOAD_CERTIFICATES  to solve this issue and remove my customized validation function? Does it validate the thumbprint and fail the inbound certificate if it doesn't match? Thanks!

@gabrielaalfaro I tested the WEBSITE_LOAD_CERTIFICATES, looks like it does exactly what I need, It will fail the inbound request if the certificate's thumbprint doesn't match. 👍
Thanks a lot for pointing it out, I think it could be worth mentioning this in the document too, just in case other people trying to deal with a similar issue.

Sorry, I was wrong, I test a couple more times, looks like it doesn't validate the inbound certificate thumbprint and it doesn't fail the request if the thumbprint doesn't match the value in the WEBSITE_LOAD_CERTIFICATES, so maybe I did something wrong?

Cool beans. That's what I thought would happen. I didn't test it, but I think it makes sense that it won't load if the thumbprint doesn't match a certificate there.
I'll leave it up to @Rick-Anderson if he wants to edit this a bit to call that key+value out. In the Windows scenario, the cert can be loaded with that approach from the CurrentUser > My store. In the Linux scenario, the cert can be loaded manually from /var/ssl/private/{THUMBPRINT}.p12. How much of these scenarios is covered over here is somewhat debatable, but there's overlap given that ASP.NET Core apps integrate so well with Azure App Service.

UPDATE: I see. Can't you validate after it loads?
I'll leave it up to @Rick-Anderson if he wants to edit this a bit to call that key+value out. In the Windows scenario, the cert can be loaded with that approach from the CurrentUser > My store. In the Linux scenario, the cert can be loaded manually from /var/ssl/private/{THUMBPRINT}.p12. How much of these scenarios is covered over here is somewhat debatable, but there's overlap given that ASP.NET Core apps integrate so well with Azure App Service.

Yes, I was validate the thumbprint after it loads by hook the method to the OnCertificateValidated event, however, if I don't set the WEBSITE_LOAD_USER_PROFILE flag, it will throw out "The system cannot find the file specified" error and won't load the certificate. or I could do something wrong, but I think there have people experiencing similar issues too by check the stackoverflow, and the suggestion is to set the WEBSITE_LOAD_USER_PROFILE flag on windows instance to load the certificate.

Oh, I see ... you aren't using Azure Key Vault to store the certificate. Ok, that makes sense.
Part of the problem here is that there are several scenarios and a few different doc sets.
It's covered here ...
https://docs.microsoft.com/en-us/azure/app-service/configure-ssl-certificate-in-code#load-certificate-from-file
... so perhaps we can just cross-link that whole doc to this section? Perhaps, replace the line that you added with this ...

For more information, see [Use a TLS/SSL certificate in your code in Azure App Service](/azure/app-service/configure-ssl-certificate-in-code#load-certificate-from-file).

That Azure doc :point_up: also covers the required service plan level.

Ok ... I went ahead with that update and a few other nit updates. Rick will be on in a bit (or tomorrow). He may revise further and/or approve+merge.
Thanks for commenting on this, @Ted-chiptech! :rocket:

Ok ... I went ahead with that update and a few other nit updates. Rick will be on in a bit (or tomorrow). He may revise further and/or approve+merge.
Thanks for commenting on this, @Ted-chiptech! 🚀

@guardrex Yes, I agree. Thanks a lot for your help! 👍


No tray icon. Got a lot "error: Failed to reconfigure tray clients (XCB_VALUE (2))"

I am running ubuntu 1404 with i3. I compiled my own gcc-6 and go with system libxcb-1.10.
After I opened "remmina", I cannot see the tray icon. From the terminal I got a lot error "error: Failed to reconfigure tray clients (XCB_VALUE (2))".
Any ideas on this? Is this because some dependencies are old?

Does this only happen with the remmina tray icon or with any tray icon?
Also, do you have any other systray program running?

I only have fcitx in the system tray. The fcitx is more complicate. If I keep fcitx open, it shows:
error: Invalid _XEMBED_INFO for window 0x1600014

If I kill fcitx, it shows
error: Failed to reconfigure tray clients (XCB_VALUE (2))

This issue https://github.com/jaagr/polybar/issues/377 has the first error. I tried VLC, it is exactly the same as fcitx.
A screent shot:

And bar config section inside [bar/top]
tray-detached		= false
tray-position		= right
tray-offset-x		= 0
tray-offset-y		= 0
tray-maxsize		= 0
tray-padding		= 0
tray-scale		= 1
tray-background 	= ${colors.t1}

I have update the system  libxcb-1.10 to  libxcb-1.12 still got the same error..

I find it may be due to my configuration. I changed the tray part to
tray-position = right tray-padding = 2
Then it works fine now. Even I still got the  error: Invalid _XEMBED_INFO for window 0x1600014
I will close this issue for now.

Oh, I totally missed this, it seems that tray-maxsize = 0 causes this

@patrick96 Oh, thanks! Now I know which part causes the problem :+1:


Use without installing Chrome/Chromium

[ ✔] I carefully followed the instructions in the README and completed successfully the setup wizard.
[✔ ] I read the ALVR Wikis here and here

I cannot successfully install ALVR due to the requirement of Chrome/Chromium for WebView support. I do have Vivaldi installed, which is Chromium-based but the installer does not detect it. After ignoring the error (refusing to install Chrome/Chromium), ALVR will open and immediately close when ran. Please add support for other Chromium-based browsers such as Vivaldi.
crash_log.txt:
15:14:23.164332100 [ERROR] At alvr\server\src\lib.rs:179: Error in locating chrome: An installation of chrome/chromium could not be found

Windows 10 Enterprise LTSC 2019 64-bit, Intel(R) Core(TM) i5-9300H CPU @ 2.40GHz (8 CPUs), ~2.4GHz, and 4095MB NVIDIA GeForce GTX 1650 with Max-Q Design (MSI).
ALVR v14.2.0-beta.1, SteamVR 1.15.19
I am running from Windows 10 LTSC 2019, which is locked to one version of Windows 10, however, I do get monthly updates. Windows Update says I am up to date. My build version is 1809.

We are currently limited by the library we are using for the webview, but we already submitted a couple of pull request for adding functionality needed by ALVR and the original author was quite responsive. So what we can do is adding a way of specifying custom paths for chromium executables. I'm quite busy with other things, but maybe @maxer456 or @Nexite can do this.

We are currently limited by the library we are using for the webview, but we already submitted a couple of pull request for adding functionality needed by ALVR and the original author was quite responsive. So what we can do is adding a way of specifying custom paths for chromium executables. I'm quite busy with other things, but maybe @maxer456 or @Nexite can do this.

hello, by looking at the library repo i have found this:
https://github.com/Srinivasa314/alcro/issues/11
ALCRO_BROWSER_PATH can be set to your browser of choice.
maybe you can try this for the moment and see if it make things work ?

hello, by looking at the library repo i have found this:
https://github.com/Srinivasa314/alcro/issues/11
ALCRO_BROWSER_PATH can be set to your browser of choice.
maybe you can try this for the moment and see if it make things work ?

Thanks @Ekristoffe, I forgot about that.

Thanks @Ekristoffe, I forgot about that.

The ALCRO_BROWSER_PATH environment variable worked fine. It has to point to the executable though, not the directory. Mine is set as C:\Program Files\Vivaldi\Application\vivaldi.exe for reference here. Also, with this I see much less memory usage from ALVR -  138.3MB compared to the 2GB a Chromium process used (and that was the de-google-fied version). Thanks very much for mentioning this workaround!

The ALCRO_BROWSER_PATH environment variable worked fine. It has to point to the executable though, not the directory. Mine is set as C:\Program Files\Vivaldi\Application\vivaldi.exe for reference here. Also, with this I see much less memory usage from ALVR -  138.3MB compared to the 2GB a Chromium process used (and that was the de-google-fied version). Thanks very much for mentioning this workaround!

Does this not work with portable browser installs? I tried setting the ALCRO_BROWSER_PATH environment variable to my portable vivaldi and chromium installs, verified by running "echo ALCRO_BROWSER_PATH" in cmd, but neither work. I get an error at alvr\server\src\dashboard.rs:38, saying chrome/chromium cannot be found.

Can this issue be re-opened? It's still a usability issue. along with what was mentioned in #966.


⚠ ClamAV has degraded performance

In 12bda9d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 199 ms

Resolved: ClamAV performance has improved in dcc3d1a.


[TA] [Service-Bug] AnalyzeTasks endpoint doesn't support UUID for the task ID.

Temporally change autogenerated code to use String instead of UUID. since service doesn't support UUID task ID yet. The service will fix it before next release.

Do we have an ETA on this?

with new v3.1-preview.4 it is resolved.


Add Tony Noble as maintainer for xtrigger-related items.

Description

After discussion on the mailing list with @oleg-nenashev and @daniel-beck , adding myself as maintainer for all XTrigger-lib derived plugins.  This will facilitate the move to a plugin and achieving full pipeline compatibility.
GitHub repos are:

FSTrigger Plugin - https://github.com/jenkinsci/fstrigger-plugin
IvyTrigger Plugin - https://github.com/jenkinsci/ivytrigger-plugin
ScriptTrigger Plugin - https://github.com/jenkinsci/scripttrigger-plugin
BuildResultTrigger Plugin (a.k.a JobTrigger) - https://github.com/jenkinsci/buildresult-trigger-plugin
XTrigger-Lib - https://github.com/jenkinsci/xtrigger-lib
XTrigger-Plugin - https://github.com/jenkinsci/xtrigger-plugin

Discussion at: https://groups.google.com/forum/#!msg/jenkinsci-dev/Ue9hmTRhqS4/cOHaLM-EDwAJ
Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Merge permission to GitHub repository

[x] Check this if newly added person also needs to be given merge permission to the GitHub repo.

@daniel-beck @batmat fine with you?

response timeout, :shipit:


Explain how to handle drivers that utilize interrupts

Should include:

Why manual move constructors are necessary
Why a bool m_moved or std::bitset<N> flags is necessary
Tips for making this easiser

Call this "Interrupt Relocation". And failure to do this correctly is, "Failed interrupt relocation". This comes in two flavors:

Interrupts disabled after move: Typically, this leads to the driver's inability to receive an interrupt due to the interrupt being deactivated.
Missed interrupt relocation: usually results in a dropped interrupt, system halt, or a fault.


Fix a typo in the readme file

Hi, I am trying to reproduce the example code in the readme file. However, it seems that there exist one typo causing this error:
File "/gpfs/radev/project/ying_rex/tl688/Orthrus/test_read.py", line 105
'ACAAAAGAGACTAGAAGCCTTATTCACTAAAATTCAGGAGGAATTTGAAGAACATGAAGTTACTTCCTCC
^
SyntaxError: unterminated string literal (detected at line 105)
I fixed it by adding the missing sign.
Moreover, I do not find the pre-trained model weights. Would you please let me know that where I can access the pre-trained weight (checkpoint)? Thanks a lot.

Hey thanks for pointing it out I've fixed it directly in main README! You can find the model weights on huggingface https://huggingface.co/antichronology/orthrus


Update readme.md

fix a MD link, update 4.7 to 4.8 notice, add reference for 4.9 workaround

Thanks Paul. I'm just about to start pushing some changes to update the tutorial for RPI3 and RPI3B+.


Image picker extended to videos in gallery or even files

Was not sure where to put feature request for image_picker plugin.
Any chance to be able to select videos that are in the same gallery as images? Seems this would serve the same purpose in many scenarios.
Also a way to be able to go into documents and choose a file would be a great feature to have.
Thanks

Just wondered if there was a thought as to when someone might work on this, I have been trying some things for iOS and have the picking and camera option working, but can not get it to display back into the flutter app, so hoping this might have a schedule for features.

This isn't on our current priority list, but we would accept patches if you are interested in trying to submit one.

Somebody mentioned to use the Android Media Picker for Android and the equivalent IOS (not sure what that is). Flutter could provide just a thin wrapper which shows the platforms implementation which is much better what is available in flutter and it stays up to date.

This definitely needs to include videos. This is a common use case and when using the native cameras we should be able to specify video as an option.
OR there needs to be a image_picker, video_picker and media_picker plugin for each use case.
Has anyone made a plugin yet that can handle video?

I have modified the current one to handle video for iOS and Java, picking and capture.

@abbadabbatech I completed a pull request and merged to image_picker she support of picking and capturing videos. I did it for iOS and Android.

@AppleEducate I guess its not released yet, may look at using it once its released instead of the one I did.  Thanks

@abbadabbatech I also did this plugin as well.
https://github.com/AppleEducate/media_picker

I'd prefer a list of media files with previews and hints if they are video, audio, or pictures (like in a file manager)
without the need to chose the media type in advance.

I want to save Imgapicker image in my folder,

I'm currently developing an app where users can choose wheter they want to capture a new image / video or want to upload an existing media. It would be great if the user could choose any media within the gallery / picker. Like in this cordova plugin https://github.com/apache/cordova-plugin-camera#cameramediatype--enum.
My quick look at the plugins source could would suggest to add a new delegate without a media filter as seen here :
https://github.com/flutter/plugins/blob/2908b42a5c8c0ed1e0026e7a6a9989fc4acaacb1/packages/image_picker/android/src/main/java/io/flutter/plugins/imagepicker/ImagePickerDelegate.java#L260
So that developers could call explicitly "pickImage", "pickVideo", "pickMedia".

@NightkingCH did you check pub.dartlang.org? There might be plugin packages with these features.

@abbadabbatech Is ImagePicker.pickVideo helpful to your issue?

It did finally do what was needed, there is still some weirdness with
aspect ratio, but I worked through that.
On Thu, Apr 25, 2019, 8:06 PM Chris Yang<EMAIL_ADDRESS>wrote:

@abbadabbatech https://github.com/abbadabbatech Is ImagePicker.pickVideo
helpful to your issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/13124#issuecomment-486879705,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASBT2AJPGEIPZEXKJIMTJ3PSJBPTANCNFSM4EEWOFWQ
.

@abbadabbatech Sounds good! Thank you. I am closing this issue.


Dealing with Categorical Features

Hi @marcotcr ,
I would like you to confirm if I am working correctly or not with the categorical features in my dataset. normally, I OneHotEncode the categorical features in my dataset.
First of all, as far as I know the data submitted to the LimeTabularExplainer and explain_instance should be label encoded version of my dataset, however, the predict_fn should take in consideration the OnehotEncoded version of my dataset so I revert back the label encoded dataset to OneHotEncoded dataset inside the predict_fn.
Moreover, for the categorical_features and categorical_names parameters, since, I submit the Explainer with LabelEncoded version of the dataset I return back the indices of the features incase of the label encoded dataset. Of course, the categories are returned as they are supposed to be returned
Is my implementation correct?

There is an example here that answers all of your questions, I think. See lines 20-28 for the first question, and lines 12-16 for the second. The ipynb version is here


Places: FindPlace API [parameters] Bug - /maps/api/place/findplacefromtext/json

Steps to reproduce

In postman, trying to call findplacefromtext with utc_offset as a field yet the documentation says utc_offset is an option

Code example
https://maps.googleapis.com/maps/api/place/findplacefromtext/json?fields=formatted_address,name,geometry,utc_offset&input=CN%20Tower,%20Toronto,%20Ontario&inputtype=textquery&key=<mykey>

Stack trace
{
    "candidates": [],
    "error_message": "Error while parsing 'fields' parameter: Unsupported field name 'utc_offset'. ",
    "status": "INVALID_REQUEST"
}

Following these steps will guarantee the quickest resolution possible.
Thanks!

BTW, I've also tried utc_offset_minutes:
{     "candidates": [],     "error_message": "Error while parsing 'fields' parameter: Unsupported field name 'utc_offset_minutes'. ",     "status": "INVALID_REQUEST" }

This might be a case where findplacefromtext doesn't return this field and a followup place details request is required using the place id. Looking to confirm and will update the fields parameter to note this if so.


findAllReferences/rename: Search in all open projects

Previously if a project A depended on a project B, we would find references in both only if the reference request started at A. Now the request can start in B, and we will find references in A as long project A has been opened.

@sheetalkamat Please re-review

@andy-ms please port to release-3.0 as well


TypeError: do_long_sequence_gst() got an unexpected keyword argument 'gaugeOptRatio'

I am getting the following error while trying the example from Getting started quickly with Gate Set Tomography. I would appreciate if anyone could look into the issue. Thanks.

Traceback (most recent call last):
File "gst.py", line 62, in
gaugeOptRatio=1e-3, constrainToTP=True)
TypeError: do_long_sequence_gst() got an unexpected keyword argument 'gaugeOptRatio'

PS. I was able to execute the code after removing

gaugeOptRatio=1e-3, constrainToTP=True

. I would like to know the explanation if anyone experienced knows.

Thanks for the catch!  The tutorials on pygsti.info are deprecated now; we should probably update the website to reflect that.  If you use the tutorials hosted on github (pyGSTi/jupyter_notebooks/Tutorials/), they should be up-to-date with the code base.  Sorry for the inconvenience!

Thanks Kenny!

Just a quick followup to close this out.  With the recent release of pyGSTi 0.9.4 the "www.pygsti.info" web page has been updated to link to the current versions of the tutorial notebooks on github.  I think originally we didn't do this because back then github would display .ipynb files as raw JSON instead of rendering it nicely as it does now.  So this issue should be permanently resolved now.


Custom Modal - self.parent.context.openAlert is not a function

Following the boostrap demo I tried to implement my own custom modal the following way:
class OrderRejectReasonsModalContext extends BSModalContext {
    constructor(public order: Order) {
        super()
    }
}

@Component ({
    selector: "order-reject-reasons-modal",
    template: "

This is a test

Interpolated: {{test}}

        ",
})

class OrderRejectReasonsModal implements ModalComponent<OrderRejectReasonsModalContext> {
    context: OrderRejectReasonsModalContext;

    public test: string;

    beforeDismiss(): boolean {
        return true;
    }

    beforeClose(): boolean {
        return true;
    }

    constructor(public dialog: DialogRef<OrderRejectReasonsModalContext>) {
        this.context = dialog.context;
        this.test = "Yes, of course!";
    }
}

@Component ({
    selector: "order-accept-dropdown",
    template: "

{{'order-accept-dropdown.HELP' | translate}}

{{'order-accept-dropdown.REJECT' | translate}}

",
    pipes: [TranslatePipe],
    viewProviders: [BS_MODAL_PROVIDERS],
})

class OrderAcceptDropdown {
    @Input() order: Order = null;

    showRejectModal() {
        this.modal.open(OrderRejectReasonsModal, new OrderRejectReasonsModalContext(this.order));
    }

    constructor(public modal: Modal, viewContainer: ViewContainerRef) {
        modal.defaultViewContainer = viewContainer;
    }
}

But when I click to open the modal via showRejectModal()I get TypeError: self.parent.context.openAlert is not a function.

I'll need a plunker to reproduce this, can you provide one?

Nevermind, this issue can be closed, only problem here was my own stupidity. You can see in the template for 'order-accept-dropdown' I have (click)="openAlert()", but I had renamed the function to 'showRejectModal', so of course it couldn't work.

Stupidity?
Einstein had those, who doesn't?
These are the things you will only find out if you go to sleep and look again the next day...
or, ask the cleaning lady and she'll find them... it's just a blindspot :)

Funny you mention that, after a good sleep I found the issue immediately the next day ;-)


serializer: force serialization as array

The implementation of array deserializer expects the payload to be serialized as array. Sadly the serializer left the door open for the underlying serializer to choice either bytes or array, if available.
This would only occur when the objects are used outside yubihsm.rs.
This change was tested on both mockhsm and yubihsm on usb.

What concrete problem is the current implementation causing? It's an issue when using these serializers with some other serialization format?

I'm reusing the yubihsm::authentication::key::Key object for a project, and I'm sending it over the wire.
Sadly the Deserialize implementation expects an array, while the Serialize doesn't specify the format.
This leaves the choice of using an array or not to the serializer used. This works out here because our serializer implementation defaults to Array, but the other serializer I'm using (postcard) uses another default.
This change keeps both serde::Serialize and serde::Deserialize consistent.


add skip cutscene options for freeplay and story

this is for issue #1445

feel like this should be combined into one option rather than two

also im pretty sure (dont know for sure) but if you do a cutscene like the one in thorns, it wont show in freeplay.

feel like this should be combined into one option rather than two

i actually wanted to do this but my trash skills on coding weren't good enough to make it one setting so i split it on 2 for now

also im pretty sure (dont know for sure) but if you do a cutscene like the one in thorns, it wont show in freeplay. so yeah, i feel like this needs a bit more work
it does load as thorns is part of schoolintro, which is already inside the if statement that checks if the setting is on
also merging into stable
yeah, i downloaded stable instead of master, will merge into master when i finish building it

have to make a new pull, closing this one


the file "npm" doesn't exist

First off, this extension is an excellent idea. I expected Nova to provide this kind of functionality out of the box.
I installed the extension and Nova immediately picked up all my project's scripts from the package.json. I select one option and click the run button and this Error appears. The file "npm" doesn't exist. Commands run fine in the Terminal.
Nova 2.1

Hi Steve. Thanks for reporting this, I'm updating to 2.1 and will try and reproduce!

I'm afraid I can't reproduce this.
@steveblue could you please check that you have provided the path to npm correctly? There is a setting for the extension - it should correspond with what you see if you run which npm in the terminal.

Ah, I believe the issue stems from using nvm. I have to switch between several versions of node throughout any given day.
nvm path is different than /usr/local/bin/npm.
If I set the fields in the Project settings to which npm and restart the app, the extension works! Thank you.
Is there a way for the extension to pickup usage of nvm automatically? I suppose setting this at the project level is an OK solution, but it seems like the extension should "just work".

Hi Steve,
Thanks for the update. I'm not sure if I am able to get the output of a command through the Nova extensions API, but could give it a go.
For some reason the environment isn't loaded with shell commands - which I think would solve this whole problem. I've submitted a help ticket to Panic to understand that in more detail.
Both of those could help this work automagically - I'll update the instructions to make it clearer in the meantime.
I'll leave the issue open until we're happy about the way forward. PR's are always open if you have ideas ;)

Hi all. Closing this, as I think the issue is fixed. Feel free to re-open if you find more examples/information.

I am having the same issue with npm not found, while using nvm. What was the fix? Thanks!

I think I got it. I set the path in Project Settings and restarted Nova like above. Thanks all!

I think I got it. I set the path in Project Settings and restarted Nova like above. Thanks all!

Excellent. You can set it globally (Extension Settings) or per project (Project Settings), which overrides the global one, when set.
If you have any further trouble - please let me know!

I don't think I can handle the error, as it's passed to Nova by that point, but will look at a way to prompt the user to do this on first installation.


when i set depth is 4, it still triggered more than 4 levels

this is my code
  var watcher = chokidar.watch(__dirname+'/15', {
  ignored: /[\/\\]\./, persistent: true,ignoreInitial:true,depth:4
});

then run touch 15/03453/js/libs/libs/index.js
it still trigger
why? (´･_･`)

It traversed exactly 4 levels of subdirectories from the watched path: 03453/js/libs/libs. This is how it is expected to work. If there was another subdir under there (like 03453/js/libs/libs/foo), you'd get the addDir for that subdir, but nothing for any files/paths deeper inside it.

ok.i know it .  Additionally,it helped me a lot ,thank u very much (★^O^★)


Use PostMessage with Transferrable

https://developer.mozilla.org/fr/docs/Web/API/Window/postMessage
https://developers.google.com/web/updates/2011/12/Transferable-Objects-Lightning-Fast

Don't seem very concluent :
https://jsperf.com/web-worker-data-copying-performance-3


Handcrafting guide is broken while using json dumps to serialize handcrafted tx to json str

bigchaindb-driver version: bigchaindb-driver==0.4.0
Python version: Python 3.5.2
Operating System: Ubuntu 16.04.2 LTS(Xenial)

Description
While following the handcrafting guide for a CREATE transaction, came across an issue after creating a transaction ID right after the Up to now section. I was trying to serialize the handcrafted_creation_tx to json string as per the guide and it fails by complaining that the transaction ID is not JSON serializable. Same scenario with prepared_creation_tx
What I Did
json_str_handcrafted_tx = json.dumps(handcrafted_creation_tx, sort_keys=True)
AND
json_str_prepared_tx = json.dumps(prepared_creation_tx, sort_keys=True)

Traceback
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.5/json/__init__.py", line 237, in dumps
    **kw).encode(obj)
  File "/usr/lib/python3.5/json/encoder.py", line 198, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/lib/python3.5/json/encoder.py", line 256, in iterencode
    return _iterencode(o, 0)
  File "/usr/lib/python3.5/json/encoder.py", line 179, in default
    raise TypeError(repr(o) + " is not JSON serializable")
TypeError: b'be9cd8825d6d7cf5f1ac81883e9c6b7c43f7ada92a4321c135c5ec5a2f9c637b' is not JSON serializable

Solution

One fix can be to use rapidjson for the aforementioned commands:

import rapidjson
json_str_handcrafted_tx = rapidjson.dumps(handcrafted_creation_tx, skipkeys=False, ensure_ascii=False, sort_keys=True)
json_str_prepared_tx = rapidjson.dumps(prepared_creation_tx, skipkeys=False, ensure_ascii=False, sort_keys=True)

Another fix can be to decode the 'id' to utf-8 before calling json.dumps

handcrafted_creation_tx['id'] = handcrafted_creation_tx['id'].decode('utf-8')
prepared_creation_tx['id'] = prepared_creation_tx['id'].decode('utf-8')

json_str_handcrafted_tx = json.dumps(handcrafted_creation_tx, sort_keys=True)
json_str_prepared_tx = json.dumps(prepared_creation_tx, sort_keys=True)

handcrafted_creation_tx['id'] = handcrafted_creation_tx['id'].encode('utf-8')
prepared_creation_tx['id'] = prepared_creation_tx['id'].encode('utf-8')

Same issue in the The Fulfilled Transaction, when I run:
    In [0]: message = json.dumps(
       ...:     handcrafted_creation_tx,
       ...:     sort_keys=True,
       ...:     separators=(',', ':'),
       ...:     ensure_ascii=False,
       ...: )

Section: The Fulfilled Transaction
 In [0]: json.dumps(fulfilled_creation_tx, sort_keys=True) == json.dumps(handcrafted_creation_tx, sort_keys=True)

In a nutshell
as well.

Section: The fulfilled transaction
returned_creation_tx = bdb.transactions.send(handcrafted_creation_tx)

I verified the guide on two different setups and it seems like I had some packaging/versioning issue with some python modules(on the setup that I was using). The guide works fine with the current set of instructions. Verified it. Closing the ticket.


useToken not resolving token values set in themes

useToken is resolving a given token value to the "default" value in the configuration and doesn't appear to be resolving any overridden value set in an active theme.
If this is expected behavior, is there a mechanism for me to resolve tokens values taking into account an active theme?   Even if it's a workaround for now that'd be great to understand.

Hey @TomSwift, Thanks for reporting the issue. We'll look into it.

Hey @TomSwift, we have started working on this will share updates as soon as the fix is released.


Cannot unzip zipped file for Mac OS X

unzip tile38-1.7.0-darwin-amd64.zip
Archive:  tile38-1.7.0-darwin-amd64.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of tile38-1.7.0-darwin-amd64.zip or
        tile38-1.7.0-darwin-amd64.zip.zip, and cannot find tile38-1.7.0-darwin-amd64.zip.ZIP, period.
```

I just tested on my side and unzipped ok. Perhaps the file was corrupted on transit?
The MD5 is 354de5253f92a91041e29c4d9a8c5b1b.
$ md5 tile38-1.7.0-darwin-amd64.zip

Alternatively you can install using Homebrew brew install tile38

Hi @JosephShering,
Checking to see if were you able to get Tile38 installed and if there's anything else I can assist with.

@JosephShering just checked on my MPB, all fine.
BTW you can install last version from brew.
It's much more easy way to do it.

@tidwall maybe we should close this?


[question] using SignalR with Azure traffic manager

I've multiple instances of the same signalr hub running under traffic manager.
since i look at #2515 it seems not possible to use signalr in such scenario.
But if i use signalr coupled with Redis i suppose is then possible to use it under traffic manger.
Am i right?

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


Semantic Function, Template Engine Extraction from Kernel.Core [Parent Issue]

### Tasks
- [ ] https://github.com/microsoft/semantic-kernel/issues/2527
- [ ] https://github.com/microsoft/semantic-kernel/issues/2563
- [ ] https://github.com/microsoft/semantic-kernel/issues/2570
- [ ] https://github.com/microsoft/semantic-kernel/issues/2624

@markwallace-microsoft - could you please add target dates for this issue.


Patch 13

Breaking changes
n/a
Fixes / Enhancements

Fixed a bug where getting the operation name was always returning null. @HofmannZ
Override the fetch policy if the default query option is used. @HofmannZ
Split up fetching and polling in the observable query. @HofmannZ
Check if the stream is closed, before adding a new event to it. @HofmannZ
Check if the variables have actully changed form or to null. @HofmannZ
Added a new getter to check if a query result has errors. @HofmannZ
Refactored the scheduler to only handle polling queries. @HofmannZ
Updated the mutation widget to use the new api in observable query. @HofmannZ
Resolve type cast exception when handling GraphQL errors @kolja-esders @HofmannZ

Docs
n/a

:tada: This PR is included in version 1.0.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Request] Arc Furnace

Please, please, please change the look of the ARC Furnace. Every other model looks and functions fairly well in this amazing mod but the Arc Furnace is so old looking and well... ugly lol

I think one of the bigger issues with the forming of the Arc Furnace is there is no visual difference to tell you that it formed, like all the other multiblocks.


DUP - change/update pass number incrementor

We have recently discovered some limitations to the way reservation numbers or pass numbers are being incremented and need to resolve this prior to it causing a conflict (ie. two passes with the same number).
Acceptance Criteria:
Given I'm a park visitor
When I book a day use pass
Then I receive my pass with a unique reservation number that has not been used previously.
Given
When
Then
Development Checklist:

[ ]
[ ]
[ ]

Workflows:
Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

Definition of Done

In progress:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] UI meets accessibility requirements
[ ] Unit tests are written
[ ] Work is traceable in GitHub
[ ] PR linked to ticket number
[ ] If needed/required - Dev adds flag/label to highlight any migration steps necessary prior to PROD deployment

Code review:

[ ] Code is peer reviewed and has passed CI/CD tests

QA:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Code is potentially shippable to the production environment
[ ] Functional features have been tested and passed by QA
[ ] UI components tested by designer
[ ] Code is deployed to PROD when moved to 'done' column (unless requested otherwise by PO)

PO Review:

[ ] Acceptance criteria are tested (Functionality meets the acceptance criteria defined in the ticket)
[ ] Reviewed and approved by Product Owner

Notes:

Dependencies

Blocked by
Blocking

Refinement Notes:

pass is a random 8-10 digit number
once there are 45k passes in a park, 10% chance the new pass may collide with an existing one
what if: we added another 3 digits onto the end so that we don't need to worry about it for a number of years?
Still need to action this now.

@marklise will update the AC/dev checklist according to refinement discussion notes.

@Christopher-walsh22 - waiting on confirmation if this has been completed or not. I believe at our DUP prioritization session, you mentioned that this issue may have been previously addressed.

This was improved back in the summer, we are now using 10 digits instead of 6. Archiving the old passes will be a much needed step to avoid overlap and continually needing to increase the number of digits used.


Importing .skool file merges strings together that were separated by labels

Bug Report
Importing the ManicMiner mm.skool file incorrectly merges strings that were separated by labels. The original labels are then lost when re-exporting the .skool file.
How To Reproduce

Download/Create a ManicMiner.z80 snapshot file and put it in the Games directory.
Select File->New Game From Snapshot File and select ManicMiner.
Restart using the command line option -skoolfile and pass it the path to the Manic Miner mm.skool file from the sources directory of skoolkid's repository.
Select File->Export Skool File->Export as Decimal.
Diffing the original mm.skool and the newly exported ManicMiner.skool file shows information lost from the original import.
Trying to regenerate the HTML using this new .skool file produces errors from the command skool2html.py:

>skool2html.py .\ManicMiner.skool
Found no ref file for ManicMiner.skool
Parsing ManicMiner.skool
Output directory: ManicMiner
Writing disassembly files in asm
ERROR: Error while parsing #R macro: Address not found: 40448

Look in the SpectrumAnalyser tool at decimal address 40190 to see where strings have been merged.
Here is the section of the original skool file that seems to being incorrectly parsed/processed:
 40190 DEFM 9,"A"                          ; }

; '...MANIC MINER . . #CHR(169) BUG-BYTE ltd. 1983...'
 MANIC MINER . . "
 40240 DEFM 127," BUG-BYTE ltd. 1983 . . By Matthew Smith . . . "
 40288 DEFM "Q to P = Left & Right . . Bottom row = Jump . . "
 40336 DEFM "A to G = Pause . .

; Attribute data for the bottom two-thirds of the title screen
;
; Used by the routine at #R34252. The graphic data for the middle third of the
; title screen is located at #R43008.
@label=LOWERATTRS
b40448 DEFB 22,22,22,22,22,22,22,22,22,22,22,22,22,22,22,22

Re-exporting this .skool file loses the labels LOWERATTRS and MESSINTRO altogether.
Notes
I had to edit the code for the command line parsing slightly to get this to work:
		else if (*argIt == std::string("-skoolfile"))
		{
			if (++argIt == argList.end())
			{
				LOGERROR("-skoolfile : No skoolkit file specified");
				break;
			}
			SkoolkitImport = *argIt; // Remove prefix ++ from here since it's done in the error check above already
		}

Ultimately I'm trying to be able to get .skool markup added to my analysis files so I can create annotated version of the source code, including graphics, level maps, etc. for the game Rebelstar. Manic Miner is my test case to understand how to roundtrip these annotations.

Thanks for the detailed bug report. I will try to take a look when I get a chance. When I was working on the skoolkit import/export I remember it could export a skoolkit that was almost identical to the original source file. There are some cases where information will be lost but it should result in a skool file that can be parsed by skool2html. Something could have broken since I originally wrote the code. Doesn't surprise me. A lot has changed.

I had a quick look. In SkoolkitImporter.cpp there is a bool called bSkoolKitCompatibleText on line 489. Try setting that to true. I remember I was having issues getting text to be compatible with skoolkit while also displaying correctly in the Spectrum Analyser.
This is currently set to false to be more compatible with displaying in the tool. We could potentially set it to true and see what happens. The manic miner example you posted looked fine in the tool with bSkoolKitCompatibleText set to true. Maybe the need for this bool no longer exists.
Could you try setting it to true and let me know how you get on?

Setting bSkoolKitCompatibleText  to true does seem to fix the problem. Thanks for that.

Good to know it fixes it. It would be nice to default bSkoolKitCompatibleText to true. The reason it's set to false is because it bypasses SetItemText() and sets the FDataInfo::DataType to EDataType::Text and the FDataInfo::ByteSize itself which is a bit naughty.
SetItemText() marks an area of memory as text, but it does not allow you to specify the length of the text. It determines the length of the text by looking for a terminating character (or terminating bit).
@TheGoodDoktor Could we have a version of SetItemText() that allows you to specify the length of the text? The other alternative is if if I call SetItemText() and then set the FDataInfo::ByteSize to the desired length immediately afterwards? That is what the UI will let you do anyway. There is nothing stopping you setting an item to text in the Code Analysis view and then increasing the Length to whatever length you want.


fix: update WordPress cms guide formatting and examples

What kind of changes does this PR include?

Updated code examples to not throw an error on single dino pages
Improved semantics of code examples
Remove extra spacing and line breaks for consistency

Description
The code example to display post content on single dino pages was throwing an error. The content returned from the API is an array of post objects. The change made destructures the first post from the response.
Additionally, some markup was updated to improve semantics. In some cases, a paragraph was being used to set:html content. In this case, the content returned included additional markup including p tags and other block level elements. In other places, tags were not being properly closed.
Lastly, a few minor edits were made to add and remove line breaks and spacing for consistency.

I was thinking maybe we should switch to the wp graphql endpoint to make the markup cleaner. I've heard that's the approach most people take to make WP sites static. or we can have both 🎉

@kevinzunigacuellar That's not a bad idea! I think you're right, and GraphQL is popular for this. Let's consider that an addition to make after the site is converted to MDX?
We're now in this stage where we need to think about what's easy to do before/after (while!) merging the MDX PR, so let's keep everything currently PR'd as simple as possible and wait to make new ones until the site is converted over to MDX!

I think @kevinzunigacuellar thought this was good to go, so I'm going to release it, and we can PR any corrections if necessary!  :partying_face:


Some recent improvements

The first commit is needed due to the recent cmake recipe update to 3.11.  Once this one is checked by the CI and has been merged, we integrated your change and will close #600.

bitbake core-image-ros-world built successfully on this build configuration:
Build Configuration:
BB_VERSION           = "1.37.0"
BUILD_SYS            = "x86_64-linux"
NATIVELSBSTRING      = "ubuntu-16.04"
TARGET_SYS           = "i586-oe-linux"
MACHINE              = "qemux86"
DISTRO               = "nodistro"
DISTRO_VERSION       = "nodistro.0"
TUNE_FEATURES        = "m32 i586"
TARGET_FPU           = ""
meta                 = "master:6f0079a6c3cc8926423cff3a927a78dcecede90d"
meta-oe
meta-python
meta-multimedia      = "master:b56fc26fefff498b10236ea6486a5d5624f726cc"
meta-ros             = "recent-fixes:00088e9d9a47c58f5b2aa3ca4d45c6f9c0d478fd"

CI is green; so, I will merge it.


Removing POST from node type endpoint and slight factoring on HTTP ITs

https://www.pivotaltracker.com/story/show/80574160

https://jira.duraspace.org/browse/FCREPO-1693


click event conflict

Hello , I used the multiselector inside tailwind ui modal (based on headless ui) , the problem is whenever I click on tag close icon it triggers modal close, how can I fix that ?

Hello, I had same problem as you and my workaround for that was using tag slot:
         <template  v-slot:tag="{ option, disabled, handleTagRemove }">
           {{ option.whatever }}           <slot name="tag" v-bind:option="option"></slot>
       </template>
Important thing is to call handleTagRemove method on @click event (I guess this libary by default uses something else and conflicts with headless-ui)
Hope it helps :)

Thanks for reporting @manafn, it was indeed using mousedown events which I suppose was how it was left from a previous release probably without a good reason. Let's see how it works. Fixed in 2.3.0.

Excellent, I was also encountering this issue. Appreciate your work @adamberecz 🙏


Using TS with a remote sphinx service

Hi there,
I'm currently trying to get thinking-sphinx working with searchd in a docker container, though I think a lot of the same issues apply if you were running searchd on a separate server to your Rails servers. I was hoping to discuss either workarounds that people are using for these cases, or work that we could do on thinking-sphinx to improve that workflow.
There's two main pain points I've been hitting:

Config-generation seems pretty insistent on calling mkdir_p for various directories, which isn't very useful if you're trying to generate configuration for a remote machine.

It seems like we ought to be able to call rake ts:index from a local Rails server and have it populate our realtime indexes on a remote server.  However, TS also tries to check that searchd is running (via the pid file) and tries to rotate the index after it's done populating.

In my hacky experimentation I've been working around these with this rake file:
namespace :ts do
  task docker_configure: :environment do
    config = ThinkingSphinx::Configuration.instance
    # force the configuration to load the "docker" key out of thinking_sphinx.yml
    config.framework = ThinkingSphinx::Frameworks::Plain.new.tap { |f| f.environment = "docker"; f.root = Rails.root }
    # sphinx configuration is going to attempt to create some dirs that don't exist locally.  Ignore them.
    def FileUtils.mkdir_p(dir)
      puts "Ignoring request to mkdir_p('#{dir}')"
    end

    interface.configure
  end

  task docker_index: :environment do
    # hack to allow running, ts:index against a remote sphinx service.
    ThinkingSphinx::Commander.registry[:running] = proc { puts "fake-sphinx running!" }
    interface.rt.index
  end
end

with this docker-compose:
version: '3'
services:
  sphinx:
    image: macbre/sphinxsearch:3.0.1
    ports:
     - "9306:9306"
    volumes:
      - ./config/docker.sphinx.conf:/opt/sphinx/conf/sphinx.conf
      - ./lib/dict:/opt/sphinx/lib/dict

Any thoughts/plans on separating out some of the TS code that only works if you're running Rails & Sphinx side-by-side?  Or am I doing it all wrong?
(Previous docker discussions at https://github.com/pat/thinking-sphinx/issues/1010)

Second issue first: definitely sounds like something that should be fixed, probably via an environment variable. I'll look into it soon :)
As for the first: you should only be running the configure task on the Sphinx server - there's no value having it occur on the client servers. However, if this is only a problem due to the index task running configure automatically, you can use INDEX_ONLY=true. That said, would it make sense to just run all the TS tasks only on your Sphinx server?

you should only be running the configure task on the Sphinx server - there's no value having it occur on the client servers. [....] would it make sense to just run all the TS tasks only on your Sphinx server?

The thing I'm trying to get away from is that we have a big monolithic Rails app with a lot of dependencies (both gems, and compiled libraries like ImageMagick).  So right now our Sphinx server needs to have all those irrelevant dependencies installed just so that we can generate a sphinx config file.
(Admittedly this approach I'm trying of generating the config file from a Rails server then shipping it over to the sphinx server, is also filled with drawbacks.)

Just pushed some commits to the develop branch which add two boolean settings (which can be turned on per-environment in config/thinking_sphinx.yml): skip_directory_creation and skip_running_check. This should remove the need for your monkey patches, but would appreciate the confirmation after you've tested it! :)

Yep, both work great thanks 🎉

Excellent! And that means all the Docker stuff's working well, without your Rails app on the Sphinx server?

Yep, my docker-searchd container seems to be working fine.  It's basically just using the macbre/sphinxsearch image with my config file mounted into it.

Is there a release scheduled for this feature, @pat? This looks like it could solve some of the issues that have been making me put off de-monolithifying a project I've been working on. Thanks!

There's no release just yet - there's a couple of outstanding issues I want to tackle first - but it's on my radar. With a bit of luck I'll have something out early next week 🤞

Awesome - that sounds great. Thanks heaps!
On Tue., 14 May 2019, 12:11 Pat Allan<EMAIL_ADDRESS>wrote:

There's no release just yet - there's a couple of outstanding issues I
want to tackle first - but it's on my radar. With a bit of luck I'll have
something out early next week 🤞
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pat/thinking-sphinx/issues/1131?email_source=notifications&email_token=ACHCTTISA3S2MWOISU4HA4LPVINUTA5CNFSM4HE2UFV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVKBXNY#issuecomment-492051383,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACHCTTPCAX44SLDZNYLFWYDPVINUTANCNFSM4HE2UFVQ
.

These settings are now part of the newly released v4.3.0 🎉

Awesome! Thanks, Pat!
On Sat., 18 May 2019, 12:54 Pat Allan<EMAIL_ADDRESS>wrote:

Closed #1131 https://github.com/pat/thinking-sphinx/issues/1131.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pat/thinking-sphinx/issues/1131?email_source=notifications&email_token=ACHCTTIZX2SDE6UQPI6SXBTPV5VWXA5CNFSM4HE2UFV2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGORQO3IBA#event-2350756868,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACHCTTMYWFXWHM3PITHFDFTPV5VWXANCNFSM4HE2UFVQ
.

@jdelStrother / @xtrasimplicity did anyone experience significant performance issues on index creation?
We're trying the mentioned setup with @macbre's Sphinx container and the referenced Thinking Sphinx settings skip_running_check & skip_directory_creation but it seems that rake ts:rebuild takes indeed ages.
Or does anyone have an idea what the reason could be or are there any tweaks / other suggestions?
Thank you in advance!

@alexanderadam, We haven't had any major performance issues, but our database is quite small and a few minutes at startup isn't a huge issue for us as searching is only a tiny part of our application's functionality.
You could try increasing the size of /dev/shm from 64MB to something a bit higher, but I'm not sure if that will have any performance benefits for TS.

@alexanderadam Our rebuilds are pretty slow by default, on a database with something like 5 million documents.  We've monkeypatched it with an alternative approach:
class ThinkingSphinx::RealTime::Populator
  def populate(&block)
    instrument "start_populating"

    limit = ENV["RT_BATCH_LIMIT"] || nil
    cnt = 0
    scope.find_in_batches(batch_size: batch_size) do |instances|
      break if limit && (cnt += 1) > limit.to_i
      transcriber.copy(*instances)
      instrument "populated", instances: instances
    end

    instrument "finish_populating"
  end
end

When we need to rebuild a sphinx index, we'll run, eg:
bin/rake ts:rebuild INDEX_FILTER=posts_rt_core RT_BATCH_LIMIT=1000

just to get sphinx back up-and-running with a few documents, and then incrementally populate it with something like this:
Post.find_each do |post|
  ThinkingSphinx::RealTime.callback_for(:post).after_save(p)
end


fix make-vis-flexible

remove redundant param from removeSubscriber() function, it breaks the removeSubscriber function.
So when the componentWillUnmount, the subscriber actually not remove the callback from the subscriber.
Fixing issue: https://github.com/uber/react-vis/issues/118

@nklijia , awesome, thank you!


ui: Remove warning for unsupported old Zulip desktop app

This warning was added in #6551. It’s not for any version of the current Electron app, which we warn about on the server side with DESKTOP_WARNING_VERSION, but rather some pre-Electron app so ancient I don’t even know what it is. Apparently it communicated using the window.bridge global, so eradicate that too.

Yeah, this was for https://github.com/zulip/zulip-desktop-legacy.

Great. I just marked that repository as archived.


Add support for method based injection.

This is a missing feature I stumbled across when attempting to convert a project from Guice, which supports @Inject on methods.
I tweaked a few classes in the inject-test module to use method injection instead of field injection and the unit tests all still passed.
I also examined the generated code and it seemed correct to me, so hopefully I didn't miss anything.

Cool.  Looks good.


added a failing test for steal.done() that returns undefined

Failing tests for #194

How these tests work is that they load a project and then try to render a route. So the steal.done() call needs to happen inside of a project, not inside of a test module. The test modules can only really check for side effects of rendering.
I'll see if I can recreate it.


Add integration tests and getline.ts testcases.

We also add a new parameter to getline.ts to select whether we want
debug logs or not.
Ref #40 .

We've reviewed this on the previous PR (#46), merging.