Add activity feed API

Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address:

PR-Codex overview
The focus of this PR is to add a basic Express server to serve a fake activity feed.
Detailed summary

Added Express server setup in server.js
Defined a fake activityFeed data
Created a route to serve the activity feed at /feed
Server listens on port 3000

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#3252  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gregfromstl and the rest of your teammates on  Graphite


Not working in live preview mode

Links are only working in reading mode. Which is making it half useful atleast for me. It should also work in live preview mode so that I can click and go to that link.

Hi, thanks for filing this issue. I should be able to update the plugin to work in live preview mode. I'm not sure if it's possible to make the links work in source editor mode, but I'll keep researching.

@Gourav1910 the plugin now works in live preview mode, once you upgrade to version 0.2.1.


[REVIEW] Map on HighLevelGraph Layers

This PR introduces two HighLevelGraph methods:
    def map_basic_layers(
        self, func: Callable[[BasicLayer], Mapping]
    ) -> "HighLevelGraph":
        """Map `func` on each basic layer and returns a new high level graph.
        `func` should take a BasicLayer as input and return a new Mapping as output
        and **cannot** change the dependencies between Layers.
        If `func` returns a non-BasicLayer type, it will be wrapped in a `BasicLayer`
        object automatically.

        Parameters
        ----------
        func : callable
            The function to call on each BasicLayer

        Returns
        -------
        hlg : HighLevelGraph
            A high level graph containing the transformed BasicLayers and the other
            Layers untouched
        """

    def map_tasks(self, func: Callable[[Iterable], Iterable]) -> "HighLevelGraph":
        """Map `func` on all tasks and returns a new high level graph.
        `func` should take an iterable of the tasks as input and return a new
        iterable as output and **cannot** change the dependencies between Layers.

        Warning
        -------
        A layer is allowed to ignore the map on tasks that are part of its internals.
        For instance, Blockwise will only invoke `func` on the input literals.

        Parameters
        ----------
        func : callable
            The function to call on tasks

        Returns
        -------
        hlg : HighLevelGraph
            A high level graph containing the transformed tasks
        """

The motivation is to avoid materialization of high level graphs in Distributed: https://github.com/dask/distributed/issues/4119

[x] Tests added / passed
[x] Passes black dask / flake8 dask

In principle this seems fine to me.  At first I was thinking that maybe we should make a map_layers method instead, or make these private at first.  I think that I'm inclined instead to mark layers generally as experimental and go ahead with things as they are.


pkg/cluster: Use Action for fixupClusterSPObjectID

Which issue this PR addresses:
Picking up from @mbarnes  https://github.com/Azure/ARO-RP/pull/2961
[ARO-1919] AAD: Migrate ARO-RP Azure AD Graph -> Microsoft Graph
What this PR does / why we need it:
Minor follow-up to #1970 where @bennerv requested this in his review.
The fixupClusterSPObjectID step in PUCM uses only the MS Graph client (no ARM calls), and the client already handles HTTP 401 Unauthorized errors by refreshing its authorization token. So there's no need to use the AuthorizationRetryingAction wrapper.
The 401 Unauthorized error handling logic for the MS Graph client is in the github.com/microsoft/kiota-http-go module.
Test plan for issue:
Create an ARO cluster.
Run az aro update --refresh-credentials on it.
Verify it completes successfully.
Is there any documentation that needs to be updated for this PR?
No, this is just an internal change.

/azp run ci


Improve nodelocal documenation

Nodelocal options for IMPORT/BACKUP should be more descriptive
Description
Nodelocal information for doing import is not intuitive. Based on conversations in the community slack, good to create official document until a more permanent solution is being developed in the future release. Here's a good starting point.
Outline
Expected Audience

Closing in favor of https://github.com/cockroachdb/docs/issues/6083.


[IMPROVE]  remove accented characters from name column

Hi, what an awesome resource you've put together here! Thank you.
I've noticed that some of the entries have accented characters in the name column which don't appear to decode correctly (at least for me), for example:
equities.search(name="telef", exchange="MCE")

So neither of the following return anything:
>>> df = equities.search(name="telefonica", exchange="MCE")
>>> df.empty
True
>>> equities.search(name="telefónica", exchange="MCE")
>>> df.empty
True

The following returns a load of instruments because many entries for Telefonica do not included the accented 'o':
>>> df = equities.search(name="telefonica")
>>> len(df)
20

...although it won't include the Madrid listing (and other entries that have the accented o in the name).
To ensure consistent querying I'd suggest replacing all accented characters in the name column with their unaccented equivalents.
If I get a moment (unlikely tbh) I'll contribute the change. Thought I'd raise the issue in the meantime in case anyone else runs into this and has the opportunity to make the changes.
Thanks again for the library!
Marcus

Good call! Should also include a Boolean ("accent_sensitive") parameter in case you do want to search specifically for "telephóne" instead of "telephone".


$feature : Addmetamask 0.01 UNI on (goerli)

specs

timeout_payment=5&timeout_claim=3&

#307

You can Claim the Reward 🙋 for your work.


Handle multiple client connections over WebSocket

Related to issue

Working server and dockerfile. Merging for further refinement - with parent project


configurable stake-drop alerts

stake penalty alerts are really noisy atm, we should be able to set a threshold (with a reasonable default) in config.ini for this.

done

done


How to download the fintuned model weights via Py?

See the title

Hi, @lschaupp. A URL for the weights can be found in the output object of the training.
training_id = ... # found on your dashboard, or using `replicate.trainings.list`
weights_url = replicate.trainings.get(training_id).output["weights"]


Replace .NET Framework 4.5.2 tokens

@NextTurn
I looked at the changes, and they look fine to me.  For some reason, I can't resolve the conflicts in the GitHub web editor this morning. Can you rebase this PR? Then, we'll merge it.


New HTTPHeaderDict class for response headers with multiple header value support

Same changes as my other pull request #4282, but this against the proposed/3.0.0 branch.

The CI builds are failing on tests which were failing before my changes were pushed (honest!)

We know :)
@Lukasa what do you think of this? I'm very :+1: personally.

I think this will be a major improvement to Requests.

Why reinventing the wheel? urllib3 has a multiheader-ready headers implementation for ages. Stable, fast, reviewed and well tested. If I remember correctly, it was already suggested a while ago to reuse it also in requests, so that it could be avoided to copy and re-read all the headers, which would also be a considerable speed improvement on creating a Response object.

Merged into '3.0' branch. ✨🍰✨

Pending further modifications.


Have an automatically generated documentation

Description
Evaluate if it makes sense to generate documentation from the source code using typedoc.
Acceptance criteria

All the SDK naming is renamed to raiden-ts
Runs automatically as part of a successful CI flow.
Gets automatically generated
Gets automatically deployed to lightclient.raiden.network/docs
Gets hosted
Markdown is generated by typedoc
Markdown is feed to slate

Tasks

[ ] Add another step in the CI configuration

Story Points

I heard someone only good things to say about https://docsify.js.org which should be pretty straightforward to use as well if typedoc generates markdown

I am going with vuepress since it is already supported by typedoc-plugin-markdown.


Create a batch from filtered outputs

User often do lots of parameter tuning. What some would like is create a new batch from filtered results (e.g.  my_param: best_value), so that they can save them as a milestone with a descriptive name.
TODO

[x] Add a "Move filtered outputs" menu item, next to the "Rename batch" item, and much like it.
[x] Send a request with the commit, batch, filter.
[x] Get the params in the API
[x] Add the necessary logic to backend.models.Batch.rename(): get_or_create a batch with the target name and move results there.
[x] Fix: currently we don't check if the user uses a name that's already used when renaming. Let's implement a "move" in this case. The logic for normal/filtered cases can be unified while doing this.

Done in the last release


Deletion Request

Please delete from Sinopia Production:
https://api.sinopia.io/resource/WAU:RT:BF2:Identifier
https://api.sinopia.io/resource/WAU:RT:BF2:Work
Thank you!

@CECSpecialistI, these resources have been removed from production.


How to translate the panel to Chinese？

I try to translate some php files in resources/lang, but when I refresh the page ,I found the text are not the Chinese I want ,they are '???'
Shall I do something to the files? Or I should translate them in another way?(I use omegaT for translate.)

https://pterodactyl-guides.readme.io/docs/contributing-to-pterodactylpanel#section-adding-new-languages

The problem has solved, I install the panel on another machine with LNMPA,and all Chinese can show correctly. I test my translate on the testing VM(Pterodactyl-DEV) before and I think the test VM dose not have the Chinese support.


Feature request: Date format locale

It would be great to have configurable date format locale.
I think, locales can be ported from moment.js

Hi @Ashot-KR ,
A locale-aware formatting is not a part of Twig.php core so I don't think it should be included into Twig.js.
It's included into the Intl extension: http://twig.sensiolabs.org/doc/extensions/intl.html
So in my opinion a possible solution could be creating a port of that extension.


BUG:

Version (please complete the following information):

OS: Mac os 15.1.1 (24B91)
Browser: Arc (chromium based)
Web Clipper version: 0.9.8
Obsidian version: 1.7.7

Describe the bug
I created a template with behaviour "Add to an existing note, at the top". When I save a webpage, it is added to a folder named "Links interesantes" and each subsequent web I clip creates a new note inside that folder. Am I missing something?
Also note that the warning "Adding to an existing note or daily note requires Obsidian 1.7.2 or later" is confusing. is it informing me that I must have those versiones and I don't or just informing?
Expected behavior
My understanding is that all clipped pages should go to the same note when this behaviour is selected. I just want to save the titlte and URL, like a bookmark manager.
Your template file

@ezekini The properties will likely be overwritten because they are being added to the same page, which can only hold one set of properties.
I was trying to do similar stuff but, every time I add a link, it appears with extra spaces, preventing me from adding it to the table automatically.

And this should look like this.

This is working as expected. I think you are confusing note title and note location. It sounds like you want Links interesantes as the note title not the path.
Please use the support channels for help with templates.

@kepano I'm not really concerned about the formatting of the links; my main issue is the automatic spaces added when incorporating this into the same note, for example, at the bottom.

I have the spaces between the links added automatically

and I need to manually remove them to make the table work

I’m not sure if I’m overthinking this, but I would prefer to save HTTPS links in a single table within one file, rather than having to clip the entire page and rely on the Dataview plugin.
Many thanks for help.

Obsidian URI always adds a newline when appending/prepending, it's unrelated to Web Clipper. You will need to open a feature request on the Obsidian forum if you want to have more control over that.

Thank you! I understand why it didn't work. I'll submit future requests, and for now, I'll manually delete the spaces.


Fix console error before applying a frontend route redirect

The error was generated in the onRouteUpdate handler, which is invoked before the component. This moves redirecting responsibility to the handler, which seems like a better place for it anyway.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against c3c7d3b8030cd9ee629af719aa5e383c3ed9c60d

Code changes LGTM, is there anything in particular worth looking over in the review app? I did a bit of poking around and things looked fine there

Only thing to do would be to confirm it eliminates the console error!


Update ksdiff to 2.2.0 (122)

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

Thanks for the PR!

@dennislaumen Running the updated cask, I'm getting
==> Package installers may write to any location; options such as --appdir are ignored.
Error: pkg source file not found: '/usr/local/Caskroom/ksdiff/2.2.0 (122)/Install ksdiff.pkg'
Error: Install incomplete.

Not sure why this is. Unlike other casks, /usr/local/Caskroom/ksdiff doesn't exist on my disk.
Trying to uninstall ksdiff gives
# brew cask uninstall ksdiff
Error: Cask 'ksdiff' is not installed.
Error: Uninstall incomplete.

https://github.com/caskroom/homebrew-cask/pull/38169

@berndkrannich sorry, I submitted the PR using the cask-repair tool (first time for me) but it seems my changes weren't enough (wasn't able yet to try the merged PR). It looks like @commitay fixed it! Thanks everyone.

@dennislaumen No worries. Don't feel discouraged (this was a special case) and please keep updating casks, cask-repair is a great tool!
@commitay Thank you very much for the prompt follow-up and the fix! https://github.com/caskroom/homebrew-cask/pull/38169#issuecomment-326266692


Errors in MacOS 10.12

Hi, getting: 'OSError: dlopen(libportmidi.dylib, 6): image not found'
When trying to execute: 'mido.get_input_names()'
Thanks, great library! (regardless of the error)

Hi,
I'm glad you're enjoying Mido!
To use ports you need either PortMidi or RtMidi installed on your system. The error message means that Mido can't find the PortMidi library.
If you have Homebrew you can install it with:
brew install portmidi

I would recommend RtMidi though. You can install it with:
pip install --pre python-rtmidi

but you will need to either use mido.set_backend() or set the $MIDO_BACKEND environment variable to tell Mido to use RtMidi.
You are not the first to run into this problem. You may find some useful information in the discussion on issue #44.
I hope this helps!

It did THANKS, but I got one more problem
Got it to recognize my MIDI keyboard as input but I can't see the computer speakers as output (I'm on mac) any idea if it is possible?
On a windows machine I can see speakers as "Microsoft GS Wavetable Synth"

I'm guessing this is USB audio. Could it be that you need a driver for it?
I have some devices that work fine in Linux but require drivers on the mac
for some reason.
Which keyboard is it?

By the way, good to heard it worked out for you! Did you god with PortMidi or RtMidi?

It worked usinf RtMidi
The keyboard is "Arturia MINILAB" and I'm trying to get the output to play on my laptop speakers

Ah, I see what's going on. "Microsoft GS Wavetable Synth" is a software synthesizer that comes with Windows. The mac doesn't have anything like this out of the box so you need to look around for something to install, or use Garage Band if you have that. (Not sure if they still include it.)

I'm using Logic (Pro X)
Any idea how to set it up?

I've never used Logic, but there is typically something under "Preferences"
or "Settings" where you select which keyboard to use. Some programs will
use any keyboard you've plugged in but others, like Reason, need to be told.
Where does Mido come in here though? :)

Let me know if you need help connecting Mido up to Logic. (If you open a virtual port you should be able to select it in Logic.)

Yeah, I opened a virtual port and logic and Mido recognized it
automatically:)
On Sat, 17 Sep 2016 at 21:06 Ole Martin Bjørndalen<EMAIL_ADDRESS>wrote:

Let me know if you need help connecting Mido up to Logic. (If you open a
virtual port you should be able to select it in Logic.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/olemb/mido/issues/58#issuecomment-247794485, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFwOA8qgvqoNkv6dIHTk8FAc-J5IDo_Kks5qrCwlgaJpZM4J92Ik
.

Nice. :)


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_72dwhvgd5o/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_72dwhvgd5o/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_72dwhvgd5o/clone/recipe/meta.yaml:40: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_72dwhvgd5o/clone/recipe/meta.yaml:40: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_72dwhvgd5o/clone/recipe/meta.yaml:40: missing_description: The recipe is missing a description
Errors were found


IOS 17 Not Compatible

Hi Luis. The package flutter_inappwebview ^5.4.0+2 (used by the oauth_webauth package) appears to be incompatible with iOS 17. This compatibility issue is causing build errors, specifically the error message: "type argument 'nw_proxy_config_t' is neither an Objective-C object nor a block type." It is recommended that you upgrade flutter_inappwebview to a version that is compatible with iOS 17 to resolve this issue.

Hi, thanks for the heads up, I will be upgrading it as soon as possible.

It’s already reported in the flutter_inappwebview repo issues. Check here for possible fixes:

flutter_inappwebview #1735

flutter_inappwebview #1783

Yes, it's already reported. But the oauth_webauth package depends on flutter_inappwebview version ^5.4.0+2, preventing from using the compatible version of flutter_inappwebview for iOS 17.
The temporary solution involves modifying WKWebsiteDataStore.h which is not a proper fixes. I'd like to request an update to the oauth_webauth package to allow for the use of the iOS 17 compatible flutter_inappwebview version and resolve the issue more effectively. I already tested to use the latest flutter_inappwebview in iOS 17 and it's not outputing the build errors.
Thank you for the reply.

This plugin depends on stable version of flutter_inappwebview, I have not upgraded to v6.0.0 because it is still beta.
The dependency in the oauth_webauth pubspec.yaml is ^5.4.0+2 as you say but this is only for compatibility purposes, when you add oauth_webauth to your project dependencies it will automatically download the latest flutter_inappwebview which is ^5.7.2+3 at the time of writing this comment, you can check this in your project’s pubspec.lock as a transitive dependency, if it’s not pointing to the latest stable one you can run flutter pub upgrade so you get the latest according to your project dependency resolution.
On the other hand if the build problem is something specifically of flutter_inappwebview v5.x.y then yess a migration to use v6.x.y will be required. I have bot checked the issue in deep yet, I will possible next week.

Alright Luis. Thank you for the explanation, it gives me a clear insight about the problem.
In my pubspec.lock, flutter_inappwebview is already ^5.7.2+3. Yes its actually caused by flutter_inappwebview v5.x.y. Guess I just need to wait an update from flutter_inappwebview to release stable v6.0.0 version and your packages updated so I can use it on iOS 17.
Thank you for the response and I hope it will work for future iOS version because your package have significant impact in my project.

Alright Luis. Thank you for the explanation, it gives me a clear insight about the problem.
In my pubspec.lock, flutter_inappwebview is already ^5.7.2+3. Yes its actually caused by flutter_inappwebview v5.x.y. Guess I just need to wait an update from flutter_inappwebview to release stable v6.0.0 version and your packages updated so I can use it on iOS 17.
Thank you for the response and I hope it will work for future iOS version because your package have significant impact in my project.

Ok, then I will upgrade oauth_webauth to use flutter_inappwebview v6.x.y even if it is beta because I use oauth_webauth in several production projects as well.

That's wonderful news! I'm glad to hear that. Thank you Luis. I will wait for the new version.

There's a pull request ready to be merged to fix this issue : https://github.com/pichillilorenzo/flutter_inappwebview/pull/1790

Alright Luis. Thank you for the explanation, it gives me a clear insight about the problem.
In my pubspec.lock, flutter_inappwebview is already ^5.7.2+3. Yes its actually caused by flutter_inappwebview v5.x.y. Guess I just need to wait an update from flutter_inappwebview to release stable v6.0.0 version and your packages updated so I can use it on iOS 17.
Thank you for the response and I hope it will work for future iOS version because your package have significant impact in my project.

The issue is happening in 6.x.x as well.

I already use dependency override for the flutter_inappwebview and it is not outputing any build error. But the login is not working on iOS 17. It did not show the login page.

I already use dependency override for the flutter_inappwebview and it is not outputing any build error. But the login is not working on iOS 17. It did not show the login page.

Oki I will check it as soon as I can

Have you found any updates related to the iOS 17?

Hi @Fyandono, I have not been able to check anything yet, really busy over here with some production releases, but I will check this for sure as I mentioned before I rely on this plugin for several projects.
I will share any news here as soon as I have something.

Released new flutter_inappwebview version 5.8.0 and 6.0.0-beta.25 with the fix!

Thanks @pichillilorenzo for letting know

I'm closing this issue as it was not really an issue of this plugin and also because the issue was already fixed on flutter_inappwebview.

Hello Luis, have you tried it on iOS17? it still doesn't work for me.


[SDP-1228] Partial Tenant Provisioning when base URL is not valid

What

When env BASE_URL is localhost (or an invalid value) and an override isn't passed, tenants get partially created.
In order to avoid that, we're validating baseURL earlier

Why
https://stellarorg.atlassian.net/browse/SDP-1228
Known limitations
[TODO or N/A]
Checklist
PR Structure

[ ] This PR has a reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR title and description are clear enough for anyone to review it.
[ ] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).

Thoroughness

[ ] This PR adds tests for the new functionality or fixes.
[ ] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[ ] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[ ] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[ ] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[ ] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[ ] This is not a breaking change.
[ ] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr303.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr303.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr303.previews.kube001.services.stellar-ops.com


Functions not working due invalid URL formatting

Hi all,
Calling functions using supabase-py for me allways results in the following response:
{'data': b'Function not found', 'error': None}
I believe this is due to the improper URL format in the FunctionClient:
https://github.com/supabase-community/supabase-py/blob/5c752443277de0a4a8dfd7d0d113f0d177efc81f/supabase/client.py#LL74C64-L74C64
Overwriting the functions_url in the following format seems to work for me: "{}/rest/v1/rpc".format(url)

Duplicate of #418
I'll fix this tonight, I gotta read up a bit on functions 😄

@tobias-scheepers You're right, the functions URL is wrong (#447) but it won't be /rest/v1/rpc - that's the PostgREST endpoint to access functions defined in the database by remote procedure call, while the functions client is for supabase functions. See js client docs


Better parsing of created_by values

When using Docker or Buildkit to build container images,
the config will contain a created_by key which contains the
shell command that was run to create the container image layer.
Typically, this command is followed by a comment #(nop) indicating
the commented out part of the shell command. This applies to
Docker directives other than RUN. We want to be able to not only
extract the RUN command but also if directives like ADD and COPY
were used to indicate that the added content is unknown in the
reports.

Modified the get_run_command function to return the command and
the instruction directive used from the created_by line.
Modified the get_commands_from_metadata function to use the tuple
returned by the get_run_command function to check for ADD and
COPY instructions.
Moved error message for no created by information from
report/formats.py to report/errors.py as it is an error message.
Modified the lock_dockerfile function to use the tuple returned
by the get_run_command function to match the RUN commands in the
image to the RUN instructions in the Dockerfile.

Fixes #918
Signed-off-by: Nisha K<EMAIL_ADDRESS>

@JamieMagee please take a look.

Just one small comment, otherwise looks good!


dropwizard 4 update

Description
Skips dropwizard 3 and goes directly to dropwizard 4
CLI counterpart to https://github.com/dockstore/dockstore/pull/5483 and likely with many of the same issues
May or may not create issue to update file provisioning plugins. Having the old jaxb around seems enough to get them to work, don't know if that will eventually cause issues (although they are simpler than swagger/openapi client libraries)
Review Instructions
CLI should work in general (spot check a few commands like validating things, talking to dockstore.org, etc.)
Issue
https://github.com/dockstore/dockstore/issues/5492  (dropwizard 3 update)
https://ucsc-cgl.atlassian.net/browse/SEAB-5454 json update
https://ucsc-cgl.atlassian.net/browse/SEAB-5468 (dropwizard 4 update)
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running ./mvnw clean install
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

Here's to hoping dropwizard doesn't have to change out their backing library again soon.

Java rather, but agreed


Fix ROM size on lpc800

Update the lpc800 to have 16KB rom since this is the amount supported.

LGTM? :)


Tests can flake if we send a message before Olm sessions are established (I think)

I was seeing some tests failing on my local machine today, even though the CI is green. I have begun tracking down the cause, so am writing my thoughts and theories here, most of which are probably wrong.
I was getting this pattern of failures for TestRoomKeyIsCycledAfterEnoughMessages:
    --- PASS: TestRoomKeyIsCycledAfterEnoughMessages/{js_hs1}|{js_hs1} (22.84s)
    --- PASS: TestRoomKeyIsCycledAfterEnoughMessages/{js_hs1}|{rust_hs1} (3.12s)
    --- FAIL: TestRoomKeyIsCycledAfterEnoughMessages/{rust_hs1}|{js_hs1} (7.46s)
    --- FAIL: TestRoomKeyIsCycledAfterEnoughMessages/{rust_hs1}|{rust_hs1} (5.84s)

So we failed when we sent the message from Rust.
I saw messages like this:
   rust.go:586: TimelineDiff change: &{ID:$kkI-w3w1GZnC2JpYJ1NGPbCnYNTBQs85lnR92yrikKA Text: Sender:@user-7-alice:hs1 Target: Membership: FailedToDecrypt:true}

(Notice the FailedToDecrypt:true.)
When I add a 1-second pause after WithAliceAndBobSyncing the tests pass, so this makes me think that the message is being sent before the Olm session is established, or something similar.
When I add the pause before WithAliceAndBobSyncing (after (MustJoinRoom) the tests still fail.
This may be a real bug in the Rust SDK, or may be a test thing. Investigating.

This is due to your change in the memory store in rust SDK. It causes a delay when claiming OTKs. I tried to improve this by inverting the client creation order in https://github.com/matrix-org/complement-crypto/pull/33 but it isn't enough.
If you don't use latest main then this will likely be okay for you.

Confirmed that these tests pass if I revert https://github.com/matrix-org/matrix-rust-sdk/pull/3221
Side note: I was worried that I wasted time by breaking those PRs into small chunks, but I am very grateful now.

Returning an empty Vec from load_tracked_users is enough to make the tests pass. Investigating.

Returning an empty Vec from load_tracked_users is enough to make the tests pass. Investigating.

There is a possibility that the culprit is the fact the implementation that stores the tracked users is buggy. It's using a Vec which means that inserting the same user with different dirty flags multiple times  is possible: https://github.com/matrix-org/matrix-rust-sdk/blob/ac0bc95c253c5d46fec31378016497f433e0067d/crates/matrix-sdk-crypto/src/store/memorystore.rs#L59 and https://github.com/matrix-org/matrix-rust-sdk/blob/ac0bc95c253c5d46fec31378016497f433e0067d/crates/matrix-sdk-crypto/src/store/memorystore.rs#L331-L333
If you load the tracked users, there's no guarantee which one we will use to check if the member actually is dirty.

Yeah I spotted the bug in the impl and have a fix for it, but unfortunately it doesn't fix the problem... continuing to investigate.

Bug fix is here: https://github.com/matrix-org/matrix-rust-sdk/pull/3282


Failed connection on startup

Failed connection error on startup.
CLIENT SETTINGS:
# Instantiate the client with the user's username
client: TikTokLiveClient = TikTokLiveClient(unique_id="@laps3d", **(
{# Whether to process initial data (cached chats, etc.)
"process_initial_data": False,
# Custom timeout for Webcast API requests
"timeout_ms": 11000,
}
)
)
ERROR DISPLAYED AFTER ~14 SECONDS:
sudo python3 TTLiveV23.py
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/httpcore/backends/asyncio.py", line 112, in connect_tcp
local_host=local_address,
File "/usr/local/lib/python3.7/dist-packages/anyio/_core/_sockets.py", line 218, in connect_tcp
await event.wait()
File "/usr/local/lib/python3.7/dist-packages/anyio/_backends/_asyncio.py", line 662, in aexit
raise exceptions[0]
File "/usr/local/lib/python3.7/dist-packages/anyio/_backends/_asyncio.py", line 702, in _run_wrapped_task
await coro
File "/usr/local/lib/python3.7/dist-packages/anyio/_core/_sockets.py", line 164, in try_connect
stream = await asynclib.connect_tcp(remote_host, remote_port, local_address)
File "/usr/local/lib/python3.7/dist-packages/anyio/_backends/_asyncio.py", line 1692, in connect_tcp
StreamProtocol, host, port, local_addr=local_addr
File "/usr/lib/python3.7/asyncio/base_events.py", line 946, in create_connection
await self.sock_connect(sock, address)
File "/usr/lib/python3.7/asyncio/selector_events.py", line 464, in sock_connect
return await fut
concurrent.futures._base.CancelledError
        During handling of the above exception, another exception occurred:

        Traceback (most recent call last):
          File "/usr/local/lib/python3.7/dist-packages/httpcore/_exceptions.py", line 8, in map_exceptions
            yield
          File "/usr/local/lib/python3.7/dist-packages/httpcore/backends/asyncio.py", line 112, in connect_tcp
            local_host=local_address,
          File "/usr/local/lib/python3.7/dist-packages/anyio/_core/_tasks.py", line 118, in __exit__
            raise TimeoutError
        TimeoutError

        During handling of the above exception, another exception occurred:

        Traceback (most recent call last):
          File "/usr/local/lib/python3.7/dist-packages/httpx/_transports/default.py", line 60, in map_httpcore_exceptions
            yield
          File "/usr/local/lib/python3.7/dist-packages/httpx/_transports/default.py", line 353, in handle_async_request
            resp = await self._pool.handle_async_request(req)
          File "/usr/local/lib/python3.7/dist-packages/httpcore/_async/connection_pool.py", line 253, in handle_async_request
            raise exc
          File "/usr/local/lib/python3.7/dist-packages/httpcore/_async/connection_pool.py", line 237, in handle_async_request
            response = await connection.handle_async_request(request)
          File "/usr/local/lib/python3.7/dist-packages/httpcore/_async/connection.py", line 86, in handle_async_request
            raise exc
          File "/usr/local/lib/python3.7/dist-packages/httpcore/_async/connection.py", line 63, in handle_async_request
            stream = await self._connect(request)
          File "/usr/local/lib/python3.7/dist-packages/httpcore/_async/connection.py", line 111, in _connect
            stream = await self._network_backend.connect_tcp(**kwargs)
          File "/usr/local/lib/python3.7/dist-packages/httpcore/backends/auto.py", line 30, in connect_tcp
            host, port, timeout=timeout, local_address=local_address
          File "/usr/local/lib/python3.7/dist-packages/httpcore/backends/asyncio.py", line 112, in connect_tcp
            local_host=local_address,
          File "/usr/lib/python3.7/contextlib.py", line 130, in __exit__
            self.gen.throw(type, value, traceback)
          File "/usr/local/lib/python3.7/dist-packages/httpcore/_exceptions.py", line 12, in map_exceptions
            raise to_exc(exc)
        httpcore.ConnectTimeout

        The above exception was the direct cause of the following exception:

        Traceback (most recent call last):
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/base.py", line 402, in _connect
            await self.__fetch_room_data(True)
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/base.py", line 229, in __fetch_room_data
            self._http.get_deserialized_object_from_signing_api("im/fetch/", self._http.params, "WebcastResponse")
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/httpx.py", line 180, in get_deserialized_object_from_signing_api
            response: bytes = await self.__httpx_get_bytes(config.TIKTOK_SIGN_API + path, params, sign_api=True)
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/httpx.py", line 80, in __httpx_get_bytes
            response: httpx.Response = await self.client.get(url, headers=self.headers, timeout=self.timeout)
          File "/usr/local/lib/python3.7/dist-packages/httpx/_client.py", line 1760, in get
            extensions=extensions,
          File "/usr/local/lib/python3.7/dist-packages/httpx/_client.py", line 1527, in request
            return await self.send(request, auth=auth, follow_redirects=follow_redirects)
          File "/usr/local/lib/python3.7/dist-packages/httpx/_client.py", line 1618, in send
            history=[],
          File "/usr/local/lib/python3.7/dist-packages/httpx/_client.py", line 1645, in _send_handling_auth
            history=history,
          File "/usr/local/lib/python3.7/dist-packages/httpx/_client.py", line 1679, in _send_handling_redirects
            response = await self._send_single_request(request)
          File "/usr/local/lib/python3.7/dist-packages/httpx/_client.py", line 1716, in _send_single_request
            response = await transport.handle_async_request(request)
          File "/usr/local/lib/python3.7/dist-packages/httpx/_transports/default.py", line 353, in handle_async_request
            resp = await self._pool.handle_async_request(req)
          File "/usr/lib/python3.7/contextlib.py", line 130, in __exit__
            self.gen.throw(type, value, traceback)
          File "/usr/local/lib/python3.7/dist-packages/httpx/_transports/default.py", line 77, in map_httpcore_exceptions
            raise mapped_exc(message) from exc
        httpx.ConnectTimeout

        The above exception was the direct cause of the following exception:

        Traceback (most recent call last):
          File "TTLiveV23.py", line 557, in <module>
            client.run()
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/base.py", line 483, in run
            self.loop.run_until_complete(self._connect(session_id=session_id))
          File "/usr/lib/python3.7/asyncio/base_events.py", line 584, in run_until_complete
            return future.result()
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/client.py", line 86, in _connect
            result: str = await super(TikTokLiveClient, self)._connect(session_id=session_id)
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/base.py", line 434, in _connect
            await self._on_error(ex, FailedConnection(message))
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/client.py", line 54, in _on_error
            raise _exc
          File "/usr/local/lib/python3.7/dist-packages/TikTokLive/client/client.py", line 48, in _on_error
            raise append from original
        TikTokLive.types.errors.FailedConnection

DEVICE:
rPi 4b
Python 3.7
TROUBLESHOOTING DONE:
Reboot rPi
Confirm network connectivity
sudo apt update
sudo apt full-upgrade
Confirmed latest version of TikTokLive

i also have this error i think something has updated breaking the code

I'm having the same issues where it no longer allows me to connect to the user.

Same issue

https://webcast.tiktok.com/webcast/ is down!
"A python library to receive and decode livestream events such as comments and gifts in real-time from TikTok's LIVE service by connecting to TikTok's internal Webcast push service. This library includes a wrapper that connects to the Webcast service using only a user's unique_id and allows you to join your livestream as well as that of other streamers. No credentials are required to use TikTokLive."
What now? :-/

The same here...TikTokLive.types.errors.FailedConnection. But the main exception seems to be related to SSL ssl.SSLWantReadError: The operation did not complete (read) (_ssl.c:1129)

Oopsie poopsie
Fixed


[WIP] Add nested group support

Codecov Report

Merging #1920 (138772c) into main (ad3a9c9) will decrease coverage by 0.11%.
The diff coverage is 10.60%.

@@            Coverage Diff             @@
##             main    #1920      +/-   ##
==========================================
- Coverage   41.23%   41.11%   -0.12%
==========================================
  Files         116      116
  Lines       14625    14677      +52
==========================================
+ Hits         6031     6035       +4
- Misses       8079     8125      +46
- Partials      515      517       +2

Flag
Coverage Δ

unit-tests
41.11% <10.60%> (-0.12%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/controller/networkpolicy/store/group.go
0.00% <0.00%> (ø)

pkg/controller/networkpolicy/validate.go
0.00% <0.00%> (ø)

pkg/controller/networkpolicy/clustergroup.go
64.56% <25.00%> (-5.71%)
:arrow_down:

pkg/apiserver/storage/ram/store.go
77.44% <0.00%> (-1.51%)
:arrow_down:

...ntroller/networkpolicy/networkpolicy_controller.go
65.75% <0.00%> (+0.37%)
:arrow_up:

/test-all


global settings... Link Regex not working

Hello,
which regexp engine is used for regexp link and url on global settings.
My regexp is working. The ticket number is underlined. But what´s the anchor replacement pattern in the url.  %1%, \1 and $1 are not working.
example:
regexp: #([0-9]{7})    -->  first anchor: [0-9]{7}
url: http://myurl/index.html?ticketid=$1   --> not working
Best regards

In my oppionion it would be nice to use the .NET Regular Expression Synthax instead propriatier String.format Synthax, because you can use the full power of regular expressions. Also in my case report the server an exception on usage of 'http://myurl/index.html?ticketid={1}'  as url parameter.
**Suggestion: File: Controllers/RepositoryController.cs -> **
Replace Line 466 - 491
With
--- snip --
        IEnumerable<Match> matches = Regex.Matches(commit.Message, linksreg).Cast<Match>();
        var links = matches.Select(m => Regex.Replace(m.Value, linksreg, linksurl)).ToList();

--- snap ---

This has been improved for version 6. I'll check if i added a proper description.
We already use the default .net regex lib. Only the behavior was not very intuitive.
Thanks for the feedback.


Change User Agent?

Hi,
I was curious how to change the user agent in conjunction with using this scraper.
-Matt

Hey Matt, you can change the user agent when you're initializing CustomRequest, the parameter is list_user_agent.
c = CustomRequest() c = CustomRequest(list_user_agent=user_agent, list_proxies=list_proxies)

Thanks for your help @Andreilys, @mattFlemingTamr you need to do the following (there is actually a typo in the docs) :
c=CustomRequest(list_user_agent=user_agent, rotate_ua=True, list_proxies=None)
Nevertheless i think it won't be enough to bypass LinkedIn ban even if you pass a GoogleBot user agent or anything similar.

Yes seems like the ban is very hard to get around, have you been able to implement anything on your end @ericfourrier ? Couldn't get this to work no matter what I tried (Even Selenium failed). I'm only looking to scrape 50-60 profiles, so not much


Update header.html

This commit will change the "Join Us" link from LSESU website to a custom form.


Run performance benchmarks in CI

Following up on informalsystems/apalache#152 and informalsystems/apalache-tests#1, we want to run the benchmarks and long running integration test in the CI here.
At minimum, we'd like to run the tests weekly, to ensure we catch regressions early, and, ideally, they should also run before the release process.
I'll post a followup here scoping out what needs doing in greater detail, but I think the following issues are prerequisites for beginning this (or, at least, must be accomplished in the process):
informalsystems/apalache-tests#1
informalsystems/apalache-tests#2

Lol, we ended up with this issue in triplicate :D
Closed by informalsystems/apalache-tests#22


readme: Improve configuration section

Add section about necessary configuration on the router
Improve documentation for plugin configuration parameters

@janvonde: Please have a quick look if you like.
Fixes #3.

you might want to add the url to the grafana.net dashboard there somewhere as well?

I've linked the blog post in which you can find the grafana dashboard link now.
Thanks again for your contribution. Please star the project if you like :)


"Addressing Security Concerns: Exploring an Alternative Approach for LinkedIn Data Collection"

I really like the project idea, but I have a concern about the current approach of using browser automation to collect LinkedIn user information. Many users might hesitate to provide their email and password due to security concerns. They might worry about the safety of their credentials in your database.
Have you considered using the LinkedIn API as an alternative solution? This would involve creating an app that requests limited access permissions from users. It could alleviate their worries about sharing sensitive information, as the API is designed for secure data exchange. This way, users would feel more comfortable and in control of their data while still enjoying the benefits of the project.

Sadly LinkedIn only offers the API for its business partners only and not tech enthusiast like myself. Not to mention that some sources stated that the API was discontinued mainly for public use.

oh, I see. thanks for your quick response.


Revert a commit after Jetpack Compose 1.5 merging

After merging Jetpack Compose 1.4.3, we discovered that some tests are failing. That was fixed in Jetpack Compose 1.5 branch.
We disabled these tests in the commit 3a9237839569298854ee96a2accbe5e2313c5892. After we merge Jetpack Compose 1.5, we should revert it.
The tests are failing because in 1.4.3 we don't have a fix for Compose Compiler. We don't build it from the main branch, so there is no issues in the production libraries.
The 1.4.3 merge PR

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Requires password given on command line, which is a security risk

I don't want my password to be visible in /proc.  Suggested alternatives: (1) allow prompting for password to read from stdin, (2) allow reading password from a file.

I was planning on reading from environment variables instead of argv as I personally deploy with Docker. Thoughts?

It's definitely better than on the command line, but I believe it's less secure than the other options I gave.  Here's one of many online references describing the pitfalls: http://movingfast.io/articles/environment-variables-considered-harmful/
There's a HN discussion about it (https://news.ycombinator.com/item?id=8826024) which discusses some of the tradeoffs.
All the options being floated are pretty cheap to implement, no harm in pushing the decision to the user so long as you don't mind the extra code to maintain. The package https://github.com/bgentry/speakeasy can apparently be used to read passwords from stdin without echoing, though I haven't tried it myself.  I wouldn't mind creating PRs for that and the file-based solution if you like.

In general I agree with the issues around storing secrets in environment variables, however using Docker with something like Kubernetes Secrets I believe to be rather secure.
How about the default config reading the environment variables and then loading the config file to override them?

Works for me.


Complex text layout support

This bug report was migrated from our old Bugzilla tracker.
These attachments are available in the static archive:

patch (ctl.patch, text/plain, 2015-12-22 06:36:59 +0000, 25687 bytes)
new patch (ctl.patch, text/plain, 2015-12-28 05:12:07 +0000, 25693 bytes)
new patch (ctl.patch, text/plain, 2015-12-30 05:22:55 +0000, 25788 bytes)
new patch (ctl.patch, text/plain, 2016-01-05 10:38:35 +0000, 49446 bytes)
new patch (ctl.patch, text/plain, 2016-01-05 10:52:00 +0000, 51800 bytes)
new patch (ctl.patch, text/plain, 2016-01-24 08:48:35 +0000, 26003 bytes)
new patch (ctl.patch, text/plain, 2016-01-27 07:09:11 +0000, 26037 bytes)

Reported in version: unspecified
Reported for operating system, platform: All, All
Comments on the original bug report:
On 2015-12-22 06:36:59 +0000, anood almuharbi wrote:

Created attachment 2342
patch
This patch adds support for languages that require "complex text layout" (https://en.wikipedia.org/wiki/Complex_text_layout).
We are using "libraqm" (https://github.com/HOST-Oman/libraqm), a small source code-only library that wraps FriBidi (for bidirectional text support) and HarfBuzz (for text shaping), and does proper BiDi and script itemization.
The CTL support is enabled by default but can be disabled at compiling time, and we provide a fallback function that uses your original code without CTL support.

On 2015-12-22 10:41:02 +0000, Sylvain wrote:

Hi,
You might be interested by this patch:
https://bugzilla.libsdl.org/show_bug.cgi?id=3046
This is SDL_ttf on HarfBuzz + FreeType.
But there is no dependency to libraqm neither FreeBidi.
It's up to the user to do BiDi, ie to reverse the string if needed, inverse parenthesis, so on.
For instance, for RTL language, I only reverse the string, then tell harfbuzz to shape as LTR, and let SDL_tff renders naturally as LTR. It works correctly.
Also, I notice, it was important to be able to tell the "script" used to HarfBuzz, otherwise the text shaping was incomplete.
Cheers,

On 2015-12-22 11:21:52 +0000, Khaled Hosny wrote:

(In reply to Sylvain from comment # 1)

Hi,
You might be interested by this patch:
https://bugzilla.libsdl.org/show_bug.cgi?id=3046
This is SDL_ttf on HarfBuzz + FreeType.
But there is no dependency to libraqm neither FreeBidi.
It's up to the user to do BiDi, ie to reverse the string if needed, inverse
parenthesis, so on.

I think it is better to do BiDi in one place i.e in SDL_ttf, experience have shown that most developers don’t have enough experience to do it correctly.

For instance, for RTL language, I only reverse the string, then tell
harfbuzz to shape as LTR, and let SDL_tff renders naturally as LTR. It works
correctly.

This won’t work for Arabic or any RTL language that require shaping, so I’m surprised it worked for you.

Also, I notice, it was important to be able to tell the "script" used to
HarfBuzz, otherwise the text shaping was incomplete.

True, and libraqm does this. It handles script itemisation and resolve the script of common characters, and even handling paired characters.

On 2015-12-22 11:25:19 +0000, Khaled Hosny wrote:

Further more, linraqm is a source only library so it adds no dependency of its own since it should be included with SDL_ttf. We wrote it as a collection of common code we are using to added complex text support to several projects.

On 2015-12-22 12:33:34 +0000, Sylvain wrote:

For RTL, it works.
but this is the sum of 3 awkward things :
reverse the string buffer.
ask hb to shape for LTR
SDL_ttf naturally draw as LTR.
I don't really see where you set the HB_SCRIPT_. E.g. HB_SCRIPT_LATIN, HB_SCRIPT_ARABIC, HB_SCRIPT_HAN, HB_SCRIPT_HEBREW, etc.
without this, I am absolutely sure I got incomplete text shaping. Maybe not for latin, arabic, but for some there was missing glyph substitutions.
Anyway, I would also prefer to have everything in SDL_tff ...

On 2015-12-22 17:46:20 +0000, Khaled Hosny wrote:

(In reply to Sylvain from comment # 4)

For RTL, it works.
but this is the sum of 3 awkward things :
reverse the string buffer.
ask hb to shape for LTR
SDL_ttf naturally draw as LTR.

I’m pretty sure this would have several problems, it might seem to work at the surface but it is not the way how RTL text should be passed to HarfBuzz and will fail in many situations.

I don't really see where you set the HB_SCRIPT_. E.g. HB_SCRIPT_LATIN,
HB_SCRIPT_ARABIC, HB_SCRIPT_HAN, HB_SCRIPT_HEBREW, etc.
without this, I am absolutely sure I got incomplete text shaping. Maybe not
for latin, arabic, but for some there was missing glyph substitutions.

The code in itemize_by_script does the script detection, and we then use it in harfbuzz_shape.

On 2015-12-23 14:52:47 +0000, Sylvain wrote:

I agree with you there should be issues (like context pattern substitution that wouldn't match) but I think I doubled check my renderings when I wrote that, and I just found it was a good workaround for some other issue.
In doubt, I changed to the correct way. Not a single difference in rendering! pixels are equals.
In fact, this is even easier:
For a plain RTL string, just set the direction as RTL to HarfBuzz, and SDL_ttf+HarfBuzz is just able to render it browsing from 0 to len-1 the "hb_glyph_info_t".
On top of SDL_ttf, I have also changed a little bit my thin layer that allows to render strings containing both RTL + LTR.
I think, there is another issue in text rendering that would be important to take care of.
Many times, the string to render cannot be rendered with only one font.
It can also happens with RTL. For instance, you have a RTL string that contains some latin LTR sub-string.
Assuming the RTL font is not able to render the LTR string.
If SDL_ttf handles the shaping of bi-direction text support, it would be also important to render from a set of fonts.

On 2015-12-23 15:38:15 +0000, Khaled Hosny wrote:

(In reply to Sylvain from comment # 6)

I agree with you there should be issues (like context pattern substitution
that wouldn't match) but I think I doubled check my renderings when I wrote
that, and I just found it was a good workaround for some other issue.
In doubt, I changed to the correct way. Not a single difference in
rendering! pixels are equals.

Try an RTL string containing parenthesis (or other characters with Bidi_Mirroring property), they will not be mirrored if text is shaped LTR. Mirroring them in the input string is not enough as it will not handle characters that has no encoded mirror in Unicode, and there is also features like ltra/rtla and ltrm/rtlm that depend on passing the correct direction to HarfBuzz.

I think, there is another issue in text rendering that would be important to
take care of.
Many times, the string to render cannot be rendered with only one font.
It can also happens with RTL. For instance, you have a RTL string that
contains some latin LTR sub-string.
Assuming the RTL font is not able to render the LTR string.
If SDL_ttf handles the shaping of bi-direction text support, it would be
also important to render from a set of fonts.

Current SDL_ttf API works with single font at a time, I agree that supporting font fallback is important, but it will require a new API and an entirely different set of dependencies (if one wants to integrate with system font management APIs).

On 2015-12-28 05:12:07 +0000, anood almuharbi wrote:

Created attachment 2348
new patch
I have attached new patch , that fix small issue in the fall back function

On 2015-12-28 19:47:10 +0000, Philipp Wiesemann wrote:

Instead of malloc()/free() the wrappers SDL_malloc()/SDL_free() should be used (to improve portability).

On 2015-12-30 05:22:55 +0000, anood almuharbi wrote:

Created attachment 2352
new patch
I have added a new patch , to replace free() and malloc() with SDL_free() and SDL_malloc().

On 2015-12-30 05:24:00 +0000, anood almuharbi wrote:

(In reply to Philipp Wiesemann from comment # 9)

Instead of malloc()/free() the wrappers SDL_malloc()/SDL_free() should be
used (to improve portability).

Thank you sir :)

On 2016-01-05 10:38:35 +0000, anood almuharbi wrote:

Created attachment 2353
new patch
I have uploaded a new patch , I forgot to add raqm.c and raqm.h in the previous patch.

On 2016-01-05 10:52:00 +0000, anood almuharbi wrote:

Created attachment 2354
new patch
added a new patch to support changes in libraqm.

On 2016-01-24 08:48:35 +0000, anood almuharbi wrote:

Created attachment 2368
new patch
We are using now a standalone library 'Raqm', that wraps FriBidi (for bidirectional
text support) and HarfBuzz (for text shaping), and does proper BiDi and scrip itemization.
Raqm support is enabled by default but can be disabled at compiling time.
https://github.com/HOST-Oman/libraqm

On 2016-01-26 21:43:39 +0000, Philipp Wiesemann wrote:

The function text_layout() from the patch calls exit(). This makes it more difficult for applications to handle errors (e.g. show a message or save files before abort).

On 2016-01-27 07:09:11 +0000, anood almuharbi wrote:

Created attachment 2369
new patch
Replace exit with return.

On 2016-01-27 07:10:21 +0000, anood almuharbi wrote:

(In reply to Philipp Wiesemann from comment # 15)

The function text_layout() from the patch calls exit(). This makes it more
difficult for applications to handle errors (e.g. show a message or save
files before abort).

You are right :), thanx.

On 2016-02-26 18:08:21 +0000, Khaled Hosny wrote:

Is there any interest from SDL_ttf maintainer(s) in this work, or what be the better course of action is to work out some other way (a different library?) to support complex text scripts in SDL?

 Namely it mixes up char/index when accessing the cache thus leading incorrect characters being displayed and it doesn't free raqm_t properly.
I attempted to fix those issue:
https://github.com/SuperTux/SDL_ttf/commit/be4e16698237cf83d65e86fb83655eee9f29ac12
https://github.com/SuperTux/SDL_ttf/commit/25588c5b32af5d26e140e5df181b27f664f0e47f
But given that there is still a bunch of Find_Glyph() in the code further fixing might be needed.

On 2018-10-19 13:15:38 +0000, Sylvain wrote:

I have also integrated a text shaping library, and I have extracted the part where I replaced the SDL_ttf Glyph cache system by an Index cache system.
I proposed the patch as I thought it was reducing the number of collision (hence an improvement) but that was a mistake. In the end, it's similar.
Nevertheless, the Index cache system is required to run a text shaping engine.
So you can have a look, maybe it will help you to fix your issue (bug 4307).

@1bsyl, is this resolved with the current code in the repo?

@slouken not the OP, but the current code works for me for rendering Arabic text in RTL mode when compiled with HarfBuzz support. I just make the following calls before calling other SDL_ttf function:
TTF_SetDirection(HB_DIRECTION_RTL); /* from HarfBuzz hb_direction_t */
TTF_SetScript(HB_SCRIPT_ARABIC); /* from HarfBuzz hb_script_t */

And to go back to normal LTR rendering I do the reverse:
TTF_SetDirection(HB_DIRECTION_LTR);
TTF_SetScript(HB_SCRIPT_COMMON);

Unlike the bug comments, I did not need to do reversing of anykind.

Great, thanks!
I'll close this for now, please let me know if more work needs to be done in the future.

Thanks!
@slouken
Btw, the casting isn't correct in SDL_ttf.h
It uses 'int' to compile even if SDL_ttf isn't build with harfbuzz
Not sure if we should change it to hb_direction_t / hb_script_t
334 extern DECLSPEC int SDLCALL TTF_SetDirection(int direction); /* hb_direction_t */
335 extern DECLSPEC int SDLCALL TTF_SetScript(int script); /* hb_script_t */

No, because those aren't defined. If you wanted to do something tricky, you could #ifdef based on whether the harfbuzz header is included, but I'd rather leave it as-is and let application code cast as needed.


Add Get Help and Community Chat Pages

Summary
This PR adds two pages that have important information I'd like to reference in other docs pages.

Add a Get Help page that lists all of the resources to find help with Mattermost
Add a community chat page that provides info about our community chat server and lists popular channels.
Add both pages to the left navigation. This replaces the link to the support knowledge base because that info is shared in the new Get Help page.

Ticket Link
No tickets that I'm aware of.

Hi @BenLloydPearson! Thank you for this great docs PR!
The following changes are needed to this PR in order to merge this update:
index.rst

small changes to ensure links resolve correctly
index.txt

templates/index.html

changes are needed here when making updates to the index.rst file; this is where homepage look-and-feel code is managed, and this is where the individual tiles can be resized)
replaced the Support KB with the new get-help.html page
added the new community-chat.html page
index.txt

guides/get-help.rst

updated formatting & applied editorial updates to align with our Style Guide
get-help.txt

guides/conmmunity-chat.rst

Under the General Purpose section, removed the Reception channel bullet since it's read-only and users can't post there unless they have specific permissions to do so
updated formatting & applied editorial updates to align with our Style Guide
community-chat.txt

Because this is a fork, I can't update this PR directly. So instead, I've updated these 4 files based on the notes above and attached my changes to this PR so that you can see what changes are required. GitHub prevents me from adding RST files, so I've renamed them as TXT files.
Going forward, would you be open to creating branches off of master rather than creating forks? This is likely a more expedient way to incorporate your changes.
When I generate the docs locally, here's what the new homepage and LHS looks like:

Is this what we'd like to publish to production? Or are additional changes preferred to the tiles on the homepage and/or the location of the new pages in the LHS?

Thanks for the recommendations @cwarnermm. I'll implement them and resubmit it via a separate branch.

This is replaced with PR #4870


python3Packages.diagrams: init at 0.12.0

Motivation for this change
Adding popular diagrams package to draw cloud diagrams with python. In addition this PR upgrades the over 2 years old graphviz package.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Closing due to inactivity from author.


[Login] Issue

Hi Team,
I logged in using the given credentials.
I am trying to understand why I had to log in through spotify again.

But if that was the original logging in part then why is 'aravind' displayed on the navbar ?

Thanks,
Alakh

Alakh,
We have mentioned in the Wiki that having a Spotify Account is a prerequisite to use Utopia.

Hi Prathamesh,
Okay, I understand but why is my name not displayed on the navbar when I have logged in using spotify?
Thanks,
Alakh

Alakh,
The Spotify login is required for getting access to Spotify API. You will not be logged in to Utopia with your Spotify account. You can use the application with your Utopia credentials.

Hi,
Oh. Alright. Closing the issue.


Fix README.md typo that prevents the table from showing.

I'm not even sure how I missed this, cheers

👏


Missing Period

There is a missing period at the end of the sentence "git clone will be that command" on the Our Command to create remotes modal. I haven't check if any other languages are missing periods in the same place.
Also command should probably be lower case in the modal title.

There is also a missing period in the last sentence of the Git Fetch modal after "This ties into the previous lesson on remote branches". I wasn't sure if I should open a separate issue for each one or post them all here. I figured a comment was less spammy.

Wanna submit a PR like in #704? 😇

Hi, I opened https://github.com/pcottle/learnGitBranching/pull/707.
It adds the requested period characters. And a couple more. ;)
(For some reason the PR wasn't linked automatically?)

@nem75 GitHub looks for keywords in the body in the format:
Git Commit Title

Other git commit body stuff
closes #705

Just a heads up for future commits.

Thanks @amayer5125!  :) At my work we put the issue key as prefix into the title; will adapt when comitting to github.


player.decTechResearched() crashes map loading

p = Player(0)
p.decTechResearched(Upg.att_melee, 1)

When I use the native function
p.decTechResearched(Upg.att_melee, 1),
then everything is fine

Oh hey, missed this over the weekend.
Was it a mistake on your site?


xpra channel 3 open failed

I tried starting VSCode before attaching and got an error on the client and then the server when I tried to attach.
vscode@vscode-demo:~$ code diesel/diesel

PS C:\Users\c> xpra_cmd attach tcp:<IP_ADDRESS>:6100
2016-05-08 16:02:46,053 Xpra gtk2 client version 0.17.1-r12513
2016-05-08 16:02:46,059  running on Microsoft Windows 10
2016-05-08 16:02:46,306 GStreamer version 1.6 for Python 3.4
2016-05-08 16:02:47,056 OpenGL_accelerate module loaded
2016-05-08 16:02:47,063 OpenGL enabled with GeForce GT 640M/PCIe/SSE2
2016-05-08 16:02:47,153  detected keyboard: layout=us
2016-05-08 16:02:47,154  desktop size is 1600x900 with 1 screen:
2016-05-08 16:02:47,157   Default (423x238 mm - DPI: 96x96) workarea: 1600x860 at 0x40
2016-05-08 16:02:47,158     DISPLAY1 (344x194 mm - DPI: 118x117)
2016-05-08 16:02:47,187 failed to receive anything, not an xpra server?
2016-05-08 16:02:47,188   could also be the wrong username, password or port
2016-05-08 16:02:47,193   or maybe this server does not support 'unknown' compression or 'bencode' packet encoding?
2016-05-08 16:02:47,194 Connection lost

vscode@vscode-demo:~$ channel 3: open failed: connect failed: Connection refused

I'm not sure how to reproduce this one. I'll reopen if it happens again.


"Lockout"; preventing use of in game menu options

The in game menu selection can be locked out by freezing 03C9A8 to 00FF.
Note that the menu actually uses 4 1 byte values (03c9a8, a9, aa, ab).
If only the first value is frozen the player, spamming up and down can rollover into the other values and when the first value is returned to 00 to end the effect it wont show the red menu selector until the player reverses the required amount of presses.
03C9A8 must be frozen at 00FF to properly work and unfrozen to 0000 to bring the red menu selector back to the Equip slot.
I've demonstrated this working effect in a video.
https://www.youtube.com/watch?v=95mPeqn94nE

i was expecting the "enhancement" label to apply but it goes without saying this is a feature request.

I didn't include a feature request template for this since there is no public Twitch service running anymore and I'm not actively gonna work on this.
The address always been in the api under GameApi.CurrentMainMenuCategory. Used in Co-Op.
Preventing the player from menuing would be more of an inconvenience than anything interesting, so I don't see myself going that route.


Implementations page is hard to read.

The implementations page is difficult to read because it has many one-sentence paragraphs without elaboration or examples.

@alercah I would like to help out with this. :+1:

Awesome, please do!

Let me know if you need any help.

This issue was fixed in #401 it can probably be closed. :+1:

@alercah @JoshBrudnak
Requesting this be closed to keep the 'Easy' label tasks up to date.


Handling of single-backslash-escaped / in NDJSON value

#1687 described a symptom that was originally specific to the old NDJSON types.json support that has since been removed. However, we see the symptom now shows up for NDJSON data in general.
Its origins are in how we've seen Suricata's EVE JSON logs escape / with a single \ like so:
$ cat comm.ndjson
{"community_id":"1:79bZuCMMmkFg6FLG5WIs\/x+8fP0="}

As of zq commit 8f7e395, we actually end up turning the \ into \\:
$ zq -f ndjson comm.ndjson
{"community_id":"1:79bZuCMMmkFg6FLG5WIs\\/x+8fP0="}

Meanwhile, jq swallows the single \:
$ cat comm.ndjson | jq -c .
{"community_id":"1:79bZuCMMmkFg6FLG5WIs/x+8fP0="}

As shown in #1687, our older, non-typed NDJSON reader behaved like jq, but after seeing our current behavior I did a binary search and confirmed that the \\-style output shown above started appearing with #2123 when NDJSON started being read by the ZSON parser.
I don't really see it from reading the JSON RFC, but in looking around I found comments like https://github.com/google/jsonnet/issues/679#issuecomment-508419558 that seem to imply the jq behavior is indeed the expected one, so we should likely conform to that.
FWIW, we've mostly worked around this problem to date because, per the Suricata docs, this escaping behavior can be turned off, and that's what we've done in the Suricata artifact we've traditionally bundled with Brim and now include as part of Brimcap. However, we can imagine a future when users will want to bring their own custom Suricata, and perhaps there's other systems out there that produce this kind of output.

@mccanne and I discussed this one in more detail. He's of the opinion that the \/ is simply not compliant with the JSON spec and hence we're free to react to it as we wish. I find nothing that contradicts that stance.
We briefly considered accepting any regular character preceded by a single backslash and stripping its backslash similar to what's done here with the /, but once again using jq as a comparison, it treats those as parse errors.
$ echo '{"foo": "v\alue"}' | jq -c .
parse error: Invalid escape at line 1, column 16

We suspect at this point that there must one or more systems out there (more than just Suricata, I'd assume) that generate/expect the \/ hence these JSON-centric tools are treating it as a special case. Without knowing more, we agreed that mimicking the jq behavior seems like a decent place to land.


Add debugPrintTransientCallbackRegistrationStack

Adds Scheduler.debugPrintTransientCallbackRegistrationStack so that you
can find out how your current callback got registered.


Writing to /etc/hosts fails on OSX

When running vagrant up, the command fails with this chunk of text:
Bringing machine 'sample-box' up with 'virtualbox' provider...
==> sample-box: Checking if box 'seravo/wordpress' is up to date...
==> sample-box: VirtualBox VM is already running.
==> sample-box: [vagrant-hostsupdater] Checking for host entries
==> sample-box: [vagrant-hostsupdater] Writing the following entries to (/etc/hosts)
==> sample-box: [vagrant-hostsupdater]   <IP_ADDRESS>  sample  # VAGRANT: 799c4eac34e80adc79c99df966b0587a (sample-box) / b22a5436-f033-4008-8264-1b041bc11c78
==> sample-box: [vagrant-hostsupdater]   <IP_ADDRESS>  sample.local  # VAGRANT: 799c4eac34e80adc79c99df966b0587a (sample-box) / b22a5436-f033-4008-8264-1b041bc11c78
==> sample-box: [vagrant-hostsupdater]   <IP_ADDRESS>  www.sample.local  # VAGRANT: 799c4eac34e80adc79c99df966b0587a (sample-box) / b22a5436-f033-4008-8264-1b041bc11c78
==> sample-box: [vagrant-hostsupdater]   <IP_ADDRESS>  webgrind.sample.local  # VAGRANT: 799c4eac34e80adc79c99df966b0587a (sample-box) / b22a5436-f033-4008-8264-1b041bc11c78
==> sample-box: [vagrant-hostsupdater]   <IP_ADDRESS>  adminer.sample.local  # VAGRANT: 799c4eac34e80adc79c99df966b0587a (sample-box) / b22a5436-f033-4008-8264-1b041bc11c78
==> sample-box: [vagrant-hostsupdater]   <IP_ADDRESS>  mailcatcher.sample.local  # VAGRANT: 799c4eac34e80adc79c99df966b0587a (sample-box) / b22a5436-f033-4008-8264-1b041bc11c78
==> sample-box: [vagrant-hostsupdater]   <IP_ADDRESS>  browsersync.sample.local  # VAGRANT: 799c4eac34e80adc79c99df966b0587a (sample-box) / b22a5436-f033-4008-8264-1b041bc11c78
==> sample-box: [vagrant-hostsupdater]   <IP_ADDRESS>  info.sample.local  # VAGRANT: 799c4eac34e80adc79c99df966b0587a (sample-box) / b22a5436-f033-4008-8264-1b041bc11c78
==> sample-box: [vagrant-hostsupdater] This operation requires administrative access. You may skip it by manually adding equivalent entries to the hosts file.
/Users/christian/.vagrant.d/gems/gems/vagrant-hostsupdater-1.0.2/lib/vagrant-hostsupdater/HostsUpdater.rb:98:in `initialize': Permission denied @ rb_sysopen - /etc/hosts (Errno::EACCES)
	from /Users/christian/.vagrant.d/gems/gems/vagrant-hostsupdater-1.0.2/lib/vagrant-hostsupdater/HostsUpdater.rb:98:in `open'
	from /Users/christian/.vagrant.d/gems/gems/vagrant-hostsupdater-1.0.2/lib/vagrant-hostsupdater/HostsUpdater.rb:98:in `addToHosts'
	from /Users/christian/.vagrant.d/gems/gems/vagrant-hostsupdater-1.0.2/lib/vagrant-hostsupdater/HostsUpdater.rb:47:in `addHostEntries'
	from /Users/christian/.vagrant.d/gems/gems/vagrant-hostsupdater-1.0.2/lib/vagrant-hostsupdater/Action/UpdateHosts.rb:17:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/plugins/providers/virtualbox/action/message_already_running.rb:11:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builder.rb:116:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `block in run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/util/busy.rb:19:in `busy'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/call.rb:53:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/box_check_outdated.rb:78:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/config_validate.rb:25:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/plugins/providers/virtualbox/action/check_virtualbox.rb:17:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builder.rb:116:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `block in run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/util/busy.rb:19:in `busy'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/call.rb:53:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/config_validate.rb:25:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builder.rb:116:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `block in run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/util/busy.rb:19:in `busy'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builtin/call.rb:53:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/plugins/providers/virtualbox/action/check_virtualbox.rb:17:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /Users/christian/.vagrant.d/gems/gems/vagrant-triggers-0.5.3/lib/vagrant-triggers/action/trigger.rb:17:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /Users/christian/.vagrant.d/gems/gems/vagrant-triggers-0.5.3/lib/vagrant-triggers/action/trigger.rb:17:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /Users/christian/.vagrant.d/gems/gems/vagrant-triggers-0.5.3/lib/vagrant-triggers/action/trigger.rb:17:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/warden.rb:34:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/builder.rb:116:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `block in run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/util/busy.rb:19:in `busy'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/action/runner.rb:66:in `run'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/machine.rb:224:in `action_raw'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/machine.rb:199:in `block in action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/environment.rb:561:in `lock'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/machine.rb:185:in `call'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/machine.rb:185:in `action'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.8.1/lib/vagrant/batch_action.rb:82:in `block (2 levels) in run'

Basically for some reason it's unable to write to the hosts file, and I have to add the entries manually. Removing seems to happen just fine on vagrant halt. It never asks for a password to gain administrator privileges, but I can write to /etc/hosts just fine without root.
Kinda annoying having to do this every time.

It doesn't ask for sudo for you ?

Must be a bug with your version of vagrant-hostsupdater. Not really related to Seravo/wp-palvelu-vagrant. Could also be a Vagrantfile issue in Seravo/wordpress
Here's what I'm getting with the most recent version of Seravo/wordpress.
==> wordpress-box: Checking for host entries
==> wordpress-box: adding to (/etc/hosts) : <IP_ADDRESS>  wordpress  # VAGRANT: ca9d4dccafaa69cfd97420a7015f85ea (wordpress-box) / 64a6117c-de1f-4889-8b10-68734fa53df8
==> wordpress-box: adding to (/etc/hosts) : <IP_ADDRESS>  wordpress.local  # VAGRANT: ca9d4dccafaa69cfd97420a7015f85ea (wordpress-box) / 64a6117c-de1f-4889-8b10-68734fa53df8
==> wordpress-box: adding to (/etc/hosts) : <IP_ADDRESS>  www.wordpress.local  # VAGRANT: ca9d4dccafaa69cfd97420a7015f85ea (wordpress-box) / 64a6117c-de1f-4889-8b10-68734fa53df8
==> wordpress-box: adding to (/etc/hosts) : <IP_ADDRESS>  webgrind.wordpress.local  # VAGRANT: ca9d4dccafaa69cfd97420a7015f85ea (wordpress-box) / 64a6117c-de1f-4889-8b10-68734fa53df8
==> wordpress-box: adding to (/etc/hosts) : <IP_ADDRESS>  adminer.wordpress.local  # VAGRANT: ca9d4dccafaa69cfd97420a7015f85ea (wordpress-box) / 64a6117c-de1f-4889-8b10-68734fa53df8
==> wordpress-box: adding to (/etc/hosts) : <IP_ADDRESS>  mailcatcher.wordpress.local  # VAGRANT: ca9d4dccafaa69cfd97420a7015f85ea (wordpress-box) / 64a6117c-de1f-4889-8b10-68734fa53df8
==> wordpress-box: adding to (/etc/hosts) : <IP_ADDRESS>  browsersync.wordpress.local  # VAGRANT: ca9d4dccafaa69cfd97420a7015f85ea (wordpress-box) / 64a6117c-de1f-4889-8b10-68734fa53df8
==> wordpress-box: adding to (/etc/hosts) : <IP_ADDRESS>  info.wordpress.local  # VAGRANT: ca9d4dccafaa69cfd97420a7015f85ea (wordpress-box) / 64a6117c-de1f-4889-8b10-68734fa53df8

WARNING: Improper use of the sudo command could lead to data loss
or the deletion of important system files. Please double-check your
typing when using sudo. Type "man sudo" for more information.

To proceed, enter your password, or type Ctrl-C to abort.

Password:
==> wordpress-box: Setting hostname...

Yeah, it doesn't ask for sudo. Might be worth noting that sudo is granted automatically for me (without password), but that can't be the cause right?
I also thought that it could be a bug with vagrant-hostsupdater, but updating it to the latest version didn't really change anything.

can you give me your output for
vagrant --version
vagrant plugin list

Sure:
vagrant --version
Vagrant 1.8.1

vagrant plugin list
vagrant-bindfs (0.4.9)
vagrant-hostmanager (1.8.5)
vagrant-hostsupdater (1.0.2)
vagrant-share (1.1.5, system)
vagrant-triggers (0.5.3)

vagrant plugin update hostsmanager
Updating plugins: hostsmanager. This may take a few minutes...
All plugins are up to date.

Vagrantfile uses hostsupdater, not hostsmanager. :)
Where did you get version 1.0.2 of vagrant-hostsupdater? The newest I can find is 0.0.11

Just vagrant plugin install vagrant-hostsupdater?
vagrant plugin install vagrant-hostsupdater
Installing the 'vagrant-hostsupdater' plugin. This can take a few minutes...
Installed the plugin 'vagrant-hostsupdater (1.0.2)!

Weird. After running vagrant plugin install vagrant-hostsupdater again, vagrant up actually worked without complaining.
You probably close this as PEBKAC.

fascinating. :)

Well it didn't fix every problem in the universe, but it seems that it fixed this one.

I dont think there's anything special about our hostsupdater configuration in Seravo/wordpress. Must be a general Vagrant issue i'm afraid


V 1.1.6

Coverage decreased (-0.4%) to 82.73% when pulling 284e5a2ac8c66c6ec0e5f6100d011d9f11f40d67 on v-1.1.6 into bd9304d81a4642298375489c44bd21f151cc81d0 on master.


Support queries without ext_options argument on LiveMetrics model

@miq-bot add_label providers/hawkular
@miq-bot add_label providers/bug
@miq-bot assign @lucasponce

@miq-bot add_label bug

please elaborate a bit in this issue description the motivation/reason/what is wrong with current situation. thanks!

MiqReport::Generator._generate_table() included always only_cols on performance reports generator.
In a recent commit, only_cols were commented, fetching the entire object, not only the cols defined.
LiveMetrics model expected only_cols as a mandatory parameter to generate a virtual model.

:+1:


Pending Installs error:  ()

When selecting to view any pending installs I get this
 ()
with 0 results.

Ive looked around online and cant seem to find any information. What did I break?

What version of php are you running?
-Arjen

On 9 Dec 2016, at 12:48, diabolicpride<EMAIL_ADDRESS>wrote:
When selecting to view any pending installs I get this
()
with 0 results.
https://cloud.githubusercontent.com/assets/11214541/21048057/2619b834-be05-11e6-8eab-ac6c0b8cc11e.png
Ive looked around online and cant seem to find any information. What did I break?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/munkireport/munkireport-php/issues/575, or mute the thread https://github.com/notifications/unsubscribe-auth/ABXOCnvuiZi6G9YyC_srXdEuU1VLkzSiks5rGUABgaJpZM4LI220.

PHP 5.3.3
From: Arjen van Bochoven<EMAIL_ADDRESS>Reply-To: munkireport/munkireport-php<EMAIL_ADDRESS>Date: Friday, 9 December 2016 at 12:18
To: munkireport/munkireport-php<EMAIL_ADDRESS>Cc: "Jamie Brigham (NBI)"<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [munkireport/munkireport-php] Pending Installs error:  () (#575)
What version of php are you running?
-Arjen

On 9 Dec 2016, at 12:48, diabolicpride<EMAIL_ADDRESS>wrote:
When selecting to view any pending installs I get this
()
with 0 results.
https://cloud.githubusercontent.com/assets/11214541/21048057/2619b834-be05-11e6-8eab-ac6c0b8cc11e.png
Ive looked around online and cant seem to find any information. What did I break?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/munkireport/munkireport-php/issues/575, or mute the thread https://github.com/notifications/unsubscribe-auth/ABXOCnvuiZi6G9YyC_srXdEuU1VLkzSiks5rGUABgaJpZM4LI220.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/munkireport/munkireport-php/issues/575#issuecomment-266001101, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AKsezU7I7q_mGd-m0oNpf6rNPvOiC3UFks5rGUcdgaJpZM4LI220.

I’m not sure if it matters (it should not), but is short tags enabled in php?
You can check in Munkireport	under Admin->Show System Status and then open ‘Extended PHP Info'
look for short_open_tag
-Arjen

On 9 Dec 2016, at 13:25, diabolicpride<EMAIL_ADDRESS>wrote:
PHP 5.3.3

I dont seem to get nay information from that page:

Ah interesting, you’re the second person that does not get PHP information reliably. You should be able to get the RAW data here:
index.php?/system/phpInfo
I would be interested in that output for debugging purposes.
-Arjen

On 9 Dec 2016, at 13:34, diabolicpride<EMAIL_ADDRESS>wrote:
I dont seem to get nay information from that page:
https://cloud.githubusercontent.com/assets/11214541/21049275/d4ec3ad4-be0b-11e6-9f90-a03ce1b4bc45.png
—
You are receiving this because you commented.

Yes the system status page is working using the current wip.- Thanks :)
The Pending Installs is still not working correctly.

Do you have short_tags enabled in php? http://stackoverflow.com/questions/2476072/tags-not-working-in-php-5-3-1

Turns out Short_tags were not enabled.
They are now enabled and this has fixed the issue. Thank you very much :)
Only issue I'm getting now is pending installs which aren't actually pending:

These are very old installs.
I have released and installed Office versions 15.28 but these are still showing here. Any ideas?

Maybe the machine has not checked in recently?

This is on several machines including both of the Macs I use. All have been checked today.

Any chance you could drop into the macadmins slack group to troubleshoot your issue quickly? https://macadmins.herokuapp.com


[Split] random split method improvement

For https://github.com/pyg-team/pytorch-frame/issues/352

Thanks for the effort. I am not sure the additional complexity is worthwhile though. I think the generalization to ratios: list[float] is not necessary here since each element has its role of train/val/test ratios.


WSL2 causing high CPU load

Version
Microsoft Windows [Version 10.0.22000.376]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 18.04
Other Software
No response
Repro Steps
Unsure. Here it's enough to simply have a wsl2 shell open. Funny enough, it's gone for now.
Expected Behavior
Taskmanager showing Vmmem with little to no cpu usage in idle, while a wsl2 shell is open.
Actual Behavior
Taskmanager show(ed) Vmmem with 5-6% constant cpu usage (on a CPU with 32 Threads), high energy consumption and makes the task manager flicker every second.
Processes inside the WSL guest are idling:
USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root         1  0.0  0.0   1744  1080 ?        Sl   23:17   0:00 /init
root         7  0.0  0.0   1752    68 ?        Ss   23:17   0:00 /init
root         8  0.0  0.0   1752    76 ?        S    23:17   0:00  \_ /init
max06        9  0.0  0.0  22988  5000 pts/0    Ss   23:17   0:00      \_ -bash
max06       98  0.0  0.0  37804  3332 pts/0    R+   23:22   0:00          \_ ps xuaf

Closing the shell (and waiting some seconds) fixes the issue, as well as terminating the guest or the virtual machine.
Diagnostic Logs
I currently hope for someone coming along with "we know that already" and "it's gonna be fixt next update". Otherwise I'm happy to provide whatever you ask for.

Thanks for reporting this @max06.
I wonder if this could be caused by wslg.
Can you try to add:
[wsl2]
guiApplications=false

To %userprofile%.wslconfig , then run wsl --shutdown, and start wsl again to see if the issue is still there ?
Also, what's the output of dmesg ?

I'm not sure if it's prudent replying here, but I just wanted you to know that your comment fixed a similar issue that I was having on a clean install of windows 11 where wsl was causing the taskbar notification icons and taskmanager to flicker a lot. Thanks mate.

Thanks for reporting this @max06.
I wonder if this could be caused by wslg.
Can you try to add:
[wsl2]
guiApplications=false

To %userprofile%.wslconfig , then run wsl --shutdown, and start wsl again to see if the issue is still there ? Also, what's the output of dmesg ?

I'm not sure if it's prudent replying here, but I just wanted you to know that your comment fixed a similar issue that I was having on a clean install of windows 11 where wsl was causing the taskbar notification icons and taskmanager to flicker a lot. Thanks mate.

Thanks for reporting this @max06.
I wonder if this could be caused by wslg.
Can you try to add:
[wsl2]
guiApplications=false

To %userprofile%.wslconfig , then run wsl --shutdown, and start wsl again to see if the issue is still there ? Also, what's the output of dmesg ?

I'm not sure if it's prudent replying here, but I just wanted you to know that your comment fixed a similar issue that I was having on a clean install of windows 11 where wsl was causing the taskbar notification icons and taskmanager to flicker a lot. Thanks mate.

Having this exact issue as well. Docker using wsl2 starts flickering taskbar and windows after a while with text inputs losing focus. Always have to restart docker/wsl2 to fix it

Also experiencing this issue on Windows 11. Disabling GUI applications fixed it for me as well.

Have the same problem, most of the time in an RDP Connection.

The issue was gone for me after I updated my Intel graphics drivers to their latest version overriding the Dell-provided ones which were quite outdated.

@OneBlue I'm so sorry for the late reply... the issue never appeared again, until just now.
Let's go step by step:
I noticed lags in UI intensive applications (Vivaldi, Bluestacks)
Taskmanager was visibly refreshing the views (all of them) every second, with a short white empty screen between each frame.
I closed all the suspect applications: Vivaldi, VSCode, Windows Terminal, no improvements.
I reopened Vivaldi to look for this issue. I also opened a fresh windows terminal (powershell) to shutdown the wsl2 instance. The flickering disappeared.
Trying to reproduce the issue: Opened a tab with my Ubuntu WSL2 instance in Windows terminal. Flickering is back. Also the WSL2 instance showed a memory consumption of around 2GB, increasing tendency.
Found the question about dmesg in this issue, executed it inside my ubuntu instance:
[   20.494750] weston[1140]: segfault at 7fec60021000 ip 00007feca331c673 sp 00007ffc3efe3d08 error 4 in libc-2.28.so[7feca31de000+14b000]
[   20.494757] Code: 0f 18 8e 00 01 00 00 0f 18 8e 40 01 00 00 0f 18 8e 80 01 00 00 0f 18 8e c0 01 00 00 c5 fe 6f 06 c5 fe 6f 4e 20 c5 fe 6f 56 40 <c5> fe 6f 5e 60 48 81 c6 80 00 00 00 48 81 ea 80 00 00 00 c5 fd e7
[   20.494763] potentially unexpected fatal signal 11.
[   20.494764] CPU: 28 PID: 1140 Comm: weston Not tainted <IP_ADDRESS>-microsoft-standard-WSL2 #1
[   20.494770] RIP: 0033:0x7feca331c673
[   20.494772] Code: 0f 18 8e 00 01 00 00 0f 18 8e 40 01 00 00 0f 18 8e 80 01 00 00 0f 18 8e c0 01 00 00 c5 fe 6f 06 c5 fe 6f 4e 20 c5 fe 6f 56 40 <c5> fe 6f 5e 60 48 81 c6 80 00 00 00 48 81 ea 80 00 00 00 c5 fd e7
[   20.494774] RSP: 002b:00007ffc3efe3d08 EFLAGS: 00010286
[   20.494776] RAX: 00007fec5bffe01e RBX: 00005598eba89630 RCX: 00007fec5bffdff0
[   20.494778] RDX: fffffffffdfde070 RSI: 00007fec60020f92 RDI: 00007fec5e01ffa0
[   20.494779] RBP: 00007fec68000af0 R08: fffffffffffffffe R09:<PHONE_NUMBER>000000
[   20.494780] R10: 00007fec5bffe010 R11: 00007fec5bffe01e R12: 00007fec5dfff010
[   20.494782] R13:<PHONE_NUMBER>8c04ae R14: 00007ffc3efe3d40 R15:<PHONE_NUMBER>000000
[   20.494783] FS:  00007feca2949900 GS:<PHONE_NUMBER>000000

repeating every 900ms.
Copied whole output. Noticed, my vscode was still running with one window, telling me it's taking longer to close. I can't tell if that window was connected to WSL before. Force closing it stopped the flickering, and I'm not able to reproduce it now again.
A fresh dmesg dump doesn't show the terminating weston anymore.
I can't tell if disabling the GUI extensions for WSL would improve anything here, but a connection to VSCode seems to be likely.

@max06 With that additional information, maybe you could re-open the issue so someone from MS could notice it?

@levrik I'd love to, but I don't have the privileges for that...

@max06 That's weird... I was guessing so from https://github.com/microsoft/WSL/issues/7893#issuecomment-1016006110.

I hope @OneBlue can reopen it 😅

Thanks for reporting this @max06. Based on the logs it looks like weston is crashing. If it's crashing in a loop that would explain the CPU consumption.
I'd recommend creating a wslg issue for this

@OneBlue Thank you for your reply!
The more interesting question is: Why is weston even running when there's no GUI application using it? Thought it would get started on first use.

It's always started when gui apps are enabled. There are no easy way to do that 'on-demand' since GUI apps will expect the X11 socket to be already present when they start

No, these two things are completely different.


Added scrollBounce to webPreferences

@Hey @amilajack Thank you for your contribution. The repo has seen some changes in recent years. Moving from coffee-script to javascript which makes this PR out of date. We are closing this for now. Would you mind opening a new PR making changes on the new file atom-window.js? PR's are always welcomed 🙇 .

@Hey @amilajack Thank you for your contribution. The repo has seen some changes in recent years. Moving from coffee-script to javascript which makes this PR out of date. We are closing this for now. Would you mind opening a new PR making changes on the new file atom-window.js? PR's are always welcomed 🙇 .


Integrate Code of Covenant 1.1.0 changes

See also: http://contributor-covenant.org/version/1/1/0/code_of_conduct.md

:+1: thanks

As a general procedural point, I think it would be good if we could @-mention @bundler/core for changes to the CoC to make sure everyone has a chance to understand and approve of the changes, since we are committed to abide by and enforce it. I just want to make sure that everyone is aware, since it's easy to miss stuff that happens quickly in all of the activity on the project.
For the record, I'm +1 on this particular change. :smile:

@TimMoore thanks, that's a really good point! I saw this as explicitly stating something that we had already agreed on in @bundler/core, but you're right, and I will make sure to check in the future. :)

:+1 @TimMoore.
And :+1: on the change.

I'm :+1: on both counts.


Upgrade dependency to web 1.0.0

By pinning the web dependency to 0.5.1 it's breaking a lot of other packages.  When I override to web ^1.0.0 the following errors are thrown:
video_player.dart:418:54: Error: A value of type 'JSBoolean' can't be assigned to a variable of type 'bool'.
_videoElement.disablePictureInPicture = true.toJS;
^
video_player.dart:428:50: Error: A value of type 'JSBoolean' can't be assigned to a variable of type 'bool'.
_videoElement.disableRemotePlayback = true.toJS;
^

Fixed in 1.3.0. Thanks for reporting. Closing the issue