WSL enable windows vista mode in Adobe PDF reader

Windows Version
Microsoft Windows [Version 10.0.22631.3007]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Ubuntu 22.04
Other Software
Adobe Acrobat Pdf Reader Version 2023.008.20470
Repro Steps
after a clean install of Windows 11:
install Adobe
install WSL2 with git
install VS Code
steps:
open Adobe
open any application that can read data from WSL  and start reading data from WSL (ie: type 'wsl' in terminal, open VS code in remote, navigate in WSL folders from file explorer...)
Expected Behavior
nothing
Actual Behavior
Adobe ui changes to Windows Vista-like mode

Diagnostic Logs
No response
### Tasks

edit: this issue is related to the compatibility of Adobe, if I change the compatibility of the program adobe.exe (right click, properties, compatibility) I can reproduce the change in UI by enabling the compatibility with older versions of Windows.

Does this only happen with WSL? Are there other apps that trigger this?
This seems like an issue that is coming from Adobe itself, as I don't think we do anything specific that might cause this.

yes, It happens every time an app tries to access data in WSL (terminal, VScode, File Explorer). However, a similar problem occurs when changing the app's compatibility in the properties, enabling compatibility with previous versions of Windows.


fix clang warnings, directly include wtypes.h

Directly include winpr/wtypes.h where _fseeki64 or _ftelli64 is used.
fix clang 5 warnings.

@freerdp-bot test.

@freerdp-bot test test test.


Rewrite by verb 1130

This resolves #1130 by taking an array of HTTP methods in the config for either the commands or the query_args_command. When the one or HTTP methods are applied in the config the rewrite rule only executes if the request method matches one in the config. If no methods are provided in the config the rewrite command will execute on all request methods.

Hi @pimg
Thanks for your contribution. Have been super busy days here, I'll review this policy asap, but I do not forget it.
Regards


base url for LaravelTestCase

I noticed that LaravelTestCase does not use the $baseUrl property.
In many cases this is just fine.
But when using route groups with domain this will break.
i suggest:
    /**
     * Get the base url for all requests.
     *
     * @return string
     */
    public function baseUrl()
    {
        if (isset($this->baseUrl)) {
            return $this->baseUrl;
        }

        return "http://localhost";
    }

+1

+1

Added.


Fixed default state of internal/xerrors.retryableError: it inheri…

…t properties from parent error as possible
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report

:exclamation: No coverage uploaded for pull request base (release-3.42@629b22a). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@               Coverage Diff               @@
##             release-3.42     #639   +/-   ##
===============================================
  Coverage                ?   44.36%
===============================================
  Files                   ?      147
  Lines                   ?    15388
  Branches                ?        0
===============================================
  Hits                    ?     6827
  Misses                  ?     8250
  Partials                ?      311

Flag
Coverage Δ

1.16.x
43.96% <0.00%> (?)

1.19.x
44.44% <0.00%> (?)

macOS-latest
44.41% <0.00%> (?)

ubuntu-latest
44.35% <0.00%> (?)

unit
44.36% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Update index.md

duplicate word

Learn Build status updates of commit 35673cf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

hub/apps/package-and-deploy/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Wrong glTexture when adding particles using multi-atlas, in 3.60

Version

Phaser Version: 3.60.0

Operating system: Windows 10

Browser: Chrome

Description

As of Phaser 3.60, when adding particles in the new format, using a multi-atlas, sometimes the wrong pixels would be rendered.
I've already found why this happens, here's an explanation:
The pixels are taken from the wrong atlas in the multi-atlas.
The reason this happens is that the sourceIndex of the frame is ignored.
The reason that the sourceIndex in the frame is ignored, is that in the new format the texture is internally set without the frame by calling setTexture(texture) in the ParticleEmitter's constructor.
This causes the emitter's texture to have as its frame the firstFrame, which would always have a sourceIndex 0, instead of the sourceIndex specific to the frame that is defined in the ParticleEmitter's config.
Workaround:
Calling setFrame(frameName) again fixes the issue, see example below.
Example Test Code

class Example extends Phaser.Scene
{
    preload ()
    {
        this.load.path = 'assets/atlas/';
        this.load.multiatlas('megaset', 'tp3-multi-atlas.json');
    }

    create ()
    {
        // Good - This frame from the 1st atlas would be displayed correctly
        this.add.image(40, 60, 'megaset', 'diamond')
        // Good - This frame from the 2nd atlas would be displayed correctly
        this.add.image(80, 60, 'megaset', 'gem')

        // Good - These particles of a frame from the 1st atlas would be displayed correctly
        this.add.particles(40, 200, 'megaset', { frame: 'diamond' });
        // Bad - These particles of a frame from the 2nd atlas would be displayed incorrectly
        this.add.particles(80, 200, 'megaset', { frame: 'gem' });

        // Workaround
        this.add.particles(40, 340, 'megaset', { frame: 'diamond' }).setFrame('diamond');
        this.add.particles(80, 340, 'megaset', { frame: 'gem' }).setFrame('gem');
    }
}

Example output:

This issue has been mentioned on Phaser. There might be relevant details there:
https://phaser.discourse.group/t/bug-phaser-3-60-particle-system-and-multi-atlas-texture/13491/3

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


Introduce FileManipulator

This has been on my radar for a while and I think there's a lot of work that can be done here. My idea is to have the following design. Half of these are already in place, I just want to reiterate the information here for clarity:
Generic Manipulators

FileManipulator: A module that takes care of basic file manipulation.
BlockManipulator: A module that takes care of basic block manipulation.

If we can extract these into a cohesive system that works well together, I think this could be extracted into its own package.
Super Scaffolding Manipulators

RoutesFileManipulator: A class which includes FileManipulator/BlockManipulator and handles the routes file when Super Scaffolding.
SuperScaffoldingFileManipulator: A class scoped to the specific needs of Super Scaffolding. Basically, this class is for anything minor enough that doesn't need its own class like the RoutesFileManipulator does, but still unique enough to put in the context of Super Scaffolding. Also includes FileManipulator and BlockManipulator.

Keeping track of lines
I wrote this in #32:

Although I wrote lines in the arguments, I don't like the idea of passing lines each time from an external object (I considered using ObjectSpace.id2ref(lines.id) so we only handle one object in memory, but it's tricky because lines is an array). Since the BlockManipulator and RoutesFileManipulator each have their own lines object, they begin to differ once we start to edit them, so I ended up passing lines here just so we're always dealing with the same set of lines.

The methods I extracted in this PR here already reflect that kind of thinking, so it was easy to implement. However, I'm not sure if this is the best design, and I'm not sure exactly what design pattern FileManipulator and BlockManipulator might fall under, but I'm writing them as modules specifically for this reason (although I initially considered making them classes and just having RoutesFileManipulator etc. inherit them). I'd love to hear some feedback as far as design goes with these two abstractions, but I'm leaning on the side of making them modules that rely heavily on self methods.

Forgot to mention, but we would make distinctions between what should stay in the Transformer and what should go in the SuperScaffoldingFileManipulator. For example, add_additional_step would simply stay in the Transformer.

RoutesFileManipulator#insert takes an Array, whereas BlockManipulator#insert takes a String for the values we want to insert. I think it would be better to have insert take an Array so I want the change the BlockManipulator method so that it does. I think that in itself will take some time, so I might cut the PR after that and then continue working on refactoring other related places in another PR after this one's out of the way.

I'm going to cut the PR here because I think extracting the following methods from the RoutesFileManipulator to the BlockManipulator is going to be a lot harder than expected. We have the following methods in the RoutesFileManipulator:

insert
insert_before
insert_after

However, the BlockManipulator just has insert which accepts keywords for before and after, and like I mentioned in the previous comment, the arguments are different data types.
Also, the RoutesFileManipulator tends to handle line numbers often, whereas the BlockManipulator handles the lines themselves. Besides that, there are also some little things like adding an extra line if the last line WITHIN a block is end, etc. I managed to get some parts working, but the system tests were failing and because of the little details that would need to go into the refactor, I'd just rather deal with that in a PR of its own.

@gazayas Thanks for taking the initiative on this one!

@andrewculver Sure thing! I'm still somewhat intimidated by the thought of refactoring the RoutesFileManipulator with the BlockManipulator, but I'll keep it in the back of my mind for later...


Enable JAX persistent caching via EconPizzaConfig

Expose a configuration object to the end user with the option to enable persistent caching via JAX.
By default, the folder where the JAX files reside is called __jax_cache__ and is created in the user's working directory. This can be changed by the option jax_cache_folder.

Usage:
import econpizza as ep
ep.config.update("enable_jax_persistent_cache", True)

Optionally, change the name or destination of the caching folder:
import econpizza as ep
ep.config.update("jax_cache_folder", "example_folder")
ep.config.update("enable_jax_persistent_cache", True)

Thank you!


Why not CMS Plugins shown in the text editor?

I have a newly installed my environment and plugins button is not displayed. In other plugins as "Text" button shown.

I have enabled plugins:

Abstract field is just a normal rich-text area, it cannot contains plugins. It's just an area for the overview content. When you create the article, you'll be directed to the blog post detail view which will allow you to add plugins (and plugins in the text plugin) to the placeholder, which is for the main content area.

OK, I understand :) What I really need is to include images in my posts. I could do it from the structure view, but it seems excessive. Why do I need to do this to include images?
I've been looking and ckeditor can include images as base64. I do not want this because it slows down the loading of the page. I saw that Nephila is doing a module to include Filer in CKEditor:
https://github.com/nephila/django-ckeditor-filebrowser-filer
But it is not yet ripe. Is there any other solution?

I need to release an updated version of https://github.com/nephila/django-ckeditor-filebrowser-filer actually, I might be able to do this during the weekend.
Though, the best and more flexible way to add images and have complete control over the text of your blog post is using plugins in structure mode.
Eventually you can use default plugins to have a temporary test automatically added to your blog post whenever you create a post: you can thus just double click on the auto-added plugin and edit it, add images or whatever, straight from the content mode: see https://github.com/nephila/djangocms-blog/issues/191#issuecomment-177834750 for how to configure it

Cool :D I understand that this is something temporary, but it would be possible to edit the text from the article creation window?
Maybe it is a problem that the article (and pk) is not yet available when creating a new article. It could perhaps temporarily created when the article has a title?

This is not doable in the admin exactly for the non-existing pk reason. This is a limitation of the django CMS API thus the blog can't do much.
What we can do is to extend the wizard to include a textarea that will be translated into a text plugin, but it won't be possible to add images there, until the post has been created.

I know it's not pretty, but the article could be created when the user leaves the title input (using Javascript) and load the TextPlugin by Ajax at the time... but it's nasty :/
Thank you for your help and for responding so quickly.
You have solved my doubt and you can close the report.

Thanks for the discussion and input


Laravel Installer not available to root

Info:

Docker version (Docker version 18.03.1-ce, build 9ee9f40):
Laradock commit (0c41fceae1b23089f5360dc4f8aeb85719c7bf5a):
System info (PC):
Windows 10 version 1803 (os build 17134.48):

Issue:
Laravel Installer not available to root

Expected behavior:
Laravel Installer shoulb available to root after WORKSPACE_INSTALL_LARAVEL_INSTALLER is set to true in .env

Reproduce:
Set WORKSPACE_INSTALL_LARAVEL_INSTALLER=true to .env
docker-compose up -d workspace
docker-compose exec workspace bash

Relevant Code:
Code in Dockerfile is:
###########################################################################
# Composer:
###########################################################################

USER root

# Add the composer.json
COPY ./composer.json /home/laradock/.composer/composer.json

# Make sure that ~/.composer belongs to laradock
RUN chown -R laradock:laradock /home/laradock/.composer

USER laradock

# Check if global install need to be ran
ARG COMPOSER_GLOBAL_INSTALL=false
ENV COMPOSER_GLOBAL_INSTALL ${COMPOSER_GLOBAL_INSTALL}

RUN if [ ${COMPOSER_GLOBAL_INSTALL} = true ]; then \
    # run the install
    composer global install \
;fi

ARG COMPOSER_REPO_PACKAGIST
ENV COMPOSER_REPO_PACKAGIST ${COMPOSER_REPO_PACKAGIST}

RUN if [ ${COMPOSER_REPO_PACKAGIST} ]; then \
    composer config -g repo.packagist composer ${COMPOSER_REPO_PACKAGIST} \
;fi

# Export composer vendor path
RUN echo "" >> ~/.bashrc && \
    echo 'export PATH="~/.composer/vendor/bin:$PATH"' >> ~/.bashrc

The last commands runs for user laradock when it should run for root too.

I'm using laradock as of ad316bab50e but the workspace/Dockerfile no longer mentions updating the bashrc file for either root or laradock. Also, it only seems to install it for root:
https://github.com/laradock/laradock/blob/master/workspace/Dockerfile#L495


Stream API enhancement

Currently redisson stream read does not seem to be really reactive. It just emits via Mono.

Can we change the Map<StreamMessageId, Map<>> to a proper data structure?

StreamMessage {
         StreamMessageId id;
         String Key;
         T body;
}

Also stream is never ending data stream.

RStreamReactive<String, String> stream = ....
stream("testGroup", "consumer1", ALL)   // returns Flux<StreamMessage>
      .take(100)
      ...
      .subscribe(process());

The upstream should listen to downstream demand and emit messages as and when it arrives.

Flux.create((emitter) -> {
            emitter.onRequest((n) -> {

                // emit n messages
                // emit as and when a message is received

                try {
                    future = (RFuture)supplier.call();
                } catch (Exception var6) {
                    emitter.error(var6);
                    return;
                }

                emitter.onDispose(() -> {
                    future.cancel(true);
                });

                future.onComplete((v, e) -> {
                    if (e != null) {
                        emitter.error(e);
                    } else {
                        if (v != null) {
                            emitter.next(v);
                        }
                    }
                });
            });

Can we change the Map<StreamMessageId, Map<>> to a proper data structure?

Map is correct structure. Multiple fields can be bounded per id.

The upstream should listen to downstream demand and emit messages as and when it arrives

Let me look at this issue.

@mrniko
Understood.
Question 1:
Do I have an option to read a stream of custom object say PaymentEvents which could have nested objects?
Question 2:
I assume you will have only one Map for a given StreamMessageId. So instead of returning Mono of Map<StreamMessageId, Map<>>, we should return a List or Flux of below object. I believe Stream is supposed to deliver the messages in the order it arrives. The consumer should process the messages in the arriving order. So, I think List/Flux could be a correct choice.
But you will know better.
StreamMessage {
         StreamMessageId id;
         Map<K, V>
}

Do I have an option to read a stream of custom object say PaymentEvents which could have nested objects?

You can store any object into Stream.

I assume you will have only one Map for a given StreamMessageId. So instead of returning Mono of Map<StreamMessageId, Map<>>, we should return a List or Flux of below object. I believe Stream is supposed to deliver the messages in the order it arrives. The consumer should process the messages in the arriving order.

It's implemented accoring Redis protocol which allows multiple map entires. LinkedHashMap used as Map implementation.


Style/RegexpLiteral: New EnforcedStyle mixed_preserve

This PR adds a new EnforcedStyle option mixed_preserve to preserve current regexp style as much as possible (e.g. allows multi-line /.../ and single-line %r{...}).
Motivation
The main reason for this option is to value consistency in cases where %r{...} is used as a key in a hash definition.
For example:
      CATEGORIES = {
        %r{\A(ee/)?config/feature_flags/} => :feature_flag,
        %r{\A(ee/)?(changelogs/unreleased)(-ee)?/} => :changelog,
        %r{\Adoc/.*(\.(md|png|gif|jpg))\z} => :docs,
        %r{\A(CONTRIBUTING|LICENSE|MAINTENANCE|PHILOSOPHY|PROCESS|README)(\.md)?\z} => :docs,
        # ^^^^ Cop complains

Since this regexp does not contain a slash / the cop suggests to use /.../ instead.
The :new: EnforcedStyle option mixed_preserve (combined with AllowInnerSlashes=false) would preserve the current regexp style as much as possible unless it sees inner slashes.
See also https://gitlab.com/gitlab-org/gitlab/-/merge_requests/57921/diffs#note_542055521
Caveats
This new EnforcedStyle option mixed_preserve is useless/noop when AllowInnerSlashes=true.
Open questions

Does the option name mirror the behaviour well or can we find a better name?
Should we raise or warn if EnforcedStyle=mixed_preserve and AllowInnerSlashes=true?

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[-] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Please note the "Open questions" section in the PR description hence it's still a Draft PR :sweat_smile:

Does the option name mirror the behaviour well or can we find a better name?

I added a consistent style to Lint/SymbolConversion (https://github.com/rubocop/rubocop/pull/9623) a few weeks ago, which might be a good name for this new style.

Should we raise or warn if EnforcedStyle=mixed_preserve and AllowInnerSlashes=true?
This appears to be undefined behaviour so yeah I'd say that we should look for it when processing the config and error out in that case (see ConfigValidator). Ideally though we should just not have a configuration that is undefined so maybe there's something to do to improve that here.

@splattael ping :-)

@splattael are you still interested in completing this PR?

I'm going to close this for now. @splattael if you ever come back to this, please open a new PR 🙌

@dvandersluis @bbatsov Sorry for the delay - I somehow missed your pings :sweat:
I am happy to continue this PR but would like to have the open questions discussed first:

Does the option name mirror the behavior well or can we find a better name?
Should we raise or warn if EnforcedStyle=mixed_preserve and AllowInnerSlashes=true?

What are your thoughts? :thinking:

Oh never mind, @dvandersluis already answered in https://github.com/rubocop/rubocop/pull/9688#issuecomment-824186583 :face_exhaling:
Something is off with my notifications, sorry :/


Public README references internal docs

There are some links in the README that are behind W3 login. Those should be changed to point to public documentation.

these links need to be changed
https://github.com/IBM/cis-integration/blob/master/README.md?plain=1#L12
https://github.com/IBM/cis-integration/blob/master/README.md?plain=1#L21
https://github.com/IBM/cis-integration/blob/master/README.md?plain=1#L38
https://github.com/IBM/cis-integration/blob/master/README.md?plain=1#L138


Update link to Mithril.js - "Getting started"

Thanks! Could you please update the link to go to the old Mithril 0.2.x docs instead?

Updated!


Colab and Binder badge for notebooks

I think it is a good idea to have a Colab and/or Binder badge for the tutorial notebooks.
Here is an action on the GitHub marketplace.

We should also add Amazon SageMaker Studio Lab. Equivalent to Colab etc on AWS side.
https://aws.amazon.com/sagemaker/studio-lab/

I want to contribute in this project, Can you please assign this issue to me.

@YB-24 Thanks for checking this project out! Please make a pull request and someone will offer feedback if needed.


New HDR background

I think we missed an HDR outdoor background take at noon with patterns in the sky (clouds) to increase reflection effects on the reflective materials.
This one seems to solve this issue:

Comparison on a car:

HDR background
Result

moutains (default)

noon_cloudy_countryside (new)

👍


Zombie Villager Support

Allow for the ability to place Zombie Villagers in a water bucket (so they dont burn)

So you just want a Zombie Villager In A Bucket item?

Yeah, thats it... :/

I will think about it. Not sure how well being able to pick up a hostile mob will fit in this mod, even if it is only the Villager version

I am considering adding it along with an option to disable collecting them in a bucket if the feature is unwanted

Almost forgot. @Yo-Snowpro I have added a Zombie Villager In A Bucket with an option to disable it in the new config! 1.20.4 and 1.20.1 have it so far and slowly will update the other versions


GoMod: go list -deps error: no matching files found

Hey,
We have an issue with the GoMod Package Manager
10:35:45.499 [DefaultDispatcher-worker-1] ERROR org.ossreviewtoolkit.analyzer.managers.GoMod - Resolving GoMod dependencies for path 'go.mod' failed with:
IOException: Running 'go list -deps -f {{with .Module}}{{.Path}} {{.Version}}{{end}} ./...' in '/builds/some-go-app' failed with exit code 1:
XXXXXX.go:10:12: pattern some.file: no matching files found

Content of XXXXXX.go
//go:generate cp ../../../some.file .
//go:embed some.file

some.file ist generated during build of another component in the project.
We don't want to run "go generate/go build" as a prequisite for ort. So, are there other ways to get this working?
Cheers,
Marco

@mawl would you be able to share a minimal example to reproduce this issue?

Slightly related: https://github.com/golang/go/issues/49534

As a workaround I use following script before running oss review toolkit, which removes the go:embed directive from relevant files:
grep -rl --include \*.go --exclude-dir=vendor "go:embed" . | xargs sed -i 's/go\:embed//g'

@mawl, can you still reproduce the issue with the latest version of ORT?

@mawl, can you still reproduce the issue with the latest version of ORT?

This is not supposed to be fixed yet, but it's in the makings. In theory it should be fixed once the next Go version is released and we upgrade to it. See

https://github.com/golang/tools/pull/428
https://github.com/golang/go/pull/58522

Note to myself: The upstream Golang issues have been closed as Gerrit changes with fixes were submitted. These changes did not make it into the 1.20.0 Golang release, but it's a bit hard to figure out whether they have been cherry-picked to any 1.20.x minor release. We should probably just try with a Golang upgrade.

Note to myself: The upstream Golang issues have been closed as Gerrit changes with fixes were submitted. These changes did not make it into the 1.20.0 Golang release, but it's a bit hard to figure out whether they have been cherry-picked to any 1.20.x minor release. We should probably just try with a Golang upgrade.

Here's a first step: https://github.com/oss-review-toolkit/ort/pull/6895


Better Ordered Name Parameters in a Constructor Documentation

In any Flutter Widget documentation, the constructor definition seems rather unreadable because there is no set ordering of the named parameters. Granted, named parameters can appear in any order, but for better readability in the documentation, please either ALPHABETIZE the parameters or order them by TYPE.

This is how I would alphabetize the named parameters, plus the newlines and indentation would help too:
Scaffold({
    PreferredSizeWidget appBar,
    Color backgroundColor,
    Widget body,
    Widget bottomNavigationBar,
    Widget bottomSheet,
    Widget drawer,
    DragStartBehavior drawerDragStartBehavior: DragStartBehavior.down,
    Widget endDrawer,
    Widget floatingActionButton,
    FloatingActionButtonAnimator floatingActionButtonAnimator,
    FloatingActionButtonLocation floatingActionButtonLocation,
    Key key,
    List<Widget> persistentFooterButtons,
    bool primary: true,
    bool resizeToAvoidBottomInset,
    bool resizeToAvoidBottomPadding
})

Someone asked me what's the difference between what I'm asking and what is shown under the Properties section which alphabetically lists the same named parameters....The difference is that you will be able to see all the properties on one screen (above the fold). In the Properties section, they are too spread apart to be able to see them all at once.

Sounds straightforward to implement.  @jcollins-g would know.
Whether we should or not, is a separate question I defer to @Hixie and @sfshaza2 for.


Add scalafmt and do an initial reformatting

This solves #8.

Codecov Report

Merging #14 into master will increase coverage by 0.14%.
The diff coverage is 98.11%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   95.68%   95.83%   +0.14%
==========================================
  Files           5        5
  Lines         116      120       +4
  Branches        3        3
==========================================
+ Hits          111      115       +4
  Misses          5        5

Impacted Files
Coverage Δ

...elwestheide/kontextfrei/scalatest/StreamSpec.scala
0% <ø> (ø)
:white_check_mark:

...anielwestheide/kontextfrei/scalatest/RDDSpec.scala
0% <0%> (ø)
:white_check_mark:

...m/danielwestheide/kontextfrei/DCollectionOps.scala
100% <100%> (ø)
:white_check_mark:

...danielwestheide/kontextfrei/RDDCollectionOps.scala
100% <100%> (ø)
:white_check_mark:

...ielwestheide/kontextfrei/StreamCollectionOps.scala
98.36% <98.36%> (+0.11%)
:white_check_mark:

Continue to review full report at Codecov.

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


doc: added an evaluation of the team according to essence #2

Evaluated the team using the given Essence checklist.

Thank you for this


Updated the Login/Signup

The update is the hardcoded values in order to be able to go into the app from the login. Unfortunately, there's still some issues with the connection between the backend and frontend. The code in the frontend that is supposed to be part of the connection into the backend has been commented out.

New updates


redirect %TEMP% and %TMP% to point to the build output

There have been recent signed build issues stemming from the %TEMP% directory containing more than 65k items.  I'm working with the build lab to correct this on their end but thought it would be wise to implement something that Roslyn does, too: redirecting %TEMP% to the build output directory.  This benefits us in two ways:

We always have a clean %TEMP% directory so the 65k limit is never reached.
Official build archives will also contain this potentially making hard-to-diagnose bugs easier to investigate.

FYI @KevinRansom: don't merge this one without me (assuming it's green), I need to run some more extensive testing.

@dotnet-bot test this please.


cp: fixed overwriting behavior for targets with trailing slashes

simplified target type checking by removing TargetType::determine()
added tests for targets with trailing slashes

please run rustfmt on these files

Fixes #3652

@pyoky ping?

Sorry but could you please fix the conflict? thanks

@pyoky ping?

Seems that it is stucked


CMP connect timeout with 3.2.0

I'm getting:
# cmp_main:../apps/cmp.c:3077:CMP info: using section(s) 'Mock connection' of OpenSSL configuration file '../Mock/test.cnf'
# opt_str:../apps/cmp.c:2438:CMP warning: -proxy option argument is empty string, resetting option
# setup_client_ctx:../apps/cmp.c:2088:CMP info: will contact http://localhost:36257/pkix/
# CMP info: sending IR
# CMP error: connect timeout
# CMP error: transfer error:request sent: IR, expected response: IP
../../../../build_static/util/wrap.pl ../../../../build_static/apps/openssl cmp -config ../Mock/test.cnf -section 'Mock connection' -certout ../../../../build_static/test-runs/test_cmp_http/test.cert.pem -proxy '' -no_proxy <IP_ADDRESS> -server 'localhost:36257' => 1
    not ok 2 - server domain name

    #   Failed test 'server domain name'
    #   at ../test/recipes/80-test_cmp_http.t line 151.
    #          got: '0'
    #     expected: '1'

Full log at: https://buildd.debian.org/status/fetch.php?pkg=openssl&arch=arm64&ver=3.2.0-1&stamp=1701011796&raw=0
I've scheduled an other build (on arm64) to see if this is a reproducible.

So a new build was successful: https://buildd.debian.org/status/fetch.php?pkg=openssl&arch=arm64&ver=3.2.0-1&stamp=1701286006&raw=0

@kroeckx - can this be closed?

No, the issue remains, we've seen this in various other logs, see for instance: https://buildd.debian.org/status/logs.php?pkg=openssl&ver=3.2.1-1&arch=amd64

@DDvO any idea why this could be happening?

I had not noticed this issue before; good that you pointed me to it.
What I see from the provided log and also from general experience,
for the vast majority of test cases there are no connection (timeout) issues,
but under certain circumstances looks like the client did not get a response in time,
at least for the connection aspect of the tests.
This must be somehow due to the underlying system,
likely related to too much general load.
To be more on the safe side here,
I suggest slightly increasing in test/recipes/80-test_cmp_http_data/Mock/test.cnf the value in line 124:
msg_timeout = 5

I suggest slightly increasing in test/recipes/80-test_cmp_http_data/Mock/test.cnf the value in line 124:
msg_timeout = 5

Will you submit a PR, please?

I've seen such failure in our CI as well although very rarely.

I suggest slightly increasing in test/recipes/80-test_cmp_http_data/Mock/test.cnf the value in line 124:
msg_timeout = 5

Will you submit a PR, please?

Done.

It still fails: https://buildd.debian.org/status/fetch.php?pkg=openssl&arch=ppc64el&ver=3.2.1-2&stamp=1708642484&raw=0

This failure is not due to connection timeout, but pretty surely because on the host running the tests the special 'domain' name localhost does not resolve to <IP_ADDRESS>:
# CMP info: sending IR
# CMP error: Connection refused:calling connect()
# CMP error: connect error
# CMP error: Connection refused:calling connect(localhost, 45033)
# CMP error: connect error
# CMP error: transfer error:request sent: IR, expected response: IP
../../../../build_static/util/wrap.pl ../../../../build_static/apps/openssl cmp -config ../Mock/test.cnf -section 'Mock connection' -certout ../../../../build_static/test-runs/test_cmp_http/test.cert.pem -proxy '' -no_proxy <IP_ADDRESS> -server 'localhost:45033' => 1
    not ok 2 - server domain name

I wonder how other (SSL/TLS/QUIC) tests, which partly also use the special name localhost, cope with this.

/etc/hosts contains:
<IP_ADDRESS>	localhost
2a02:16a8:dc41:100::244	ppc64el-conova-02.debian.org	ppc64el-conova-02

# The following lines are desirable for IPv6 capable hosts
::1     localhost ip6-localhost ip6-loopback
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters

The hosts that works has:
<IP_ADDRESS>	localhost
<IP_ADDRESS>	ppc64el-osuosl-01.debian.org	ppc64el-osuosl-01

# The following lines are desirable for IPv6 capable hosts
::1     localhost ip6-localhost ip6-loopback
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters

Note that ppc64el-conova-02.debian.org does not have a public IPv4 address.

The not having a public IPv4 address seems to be common on all the hosts it fails on. <IP_ADDRESS> is the only IPv4 address they have.

Strange - so according to /etc/hosts, the two machines have identical configuration w.r.t localhost.
Whether or not they (in addition) have a public IPv4 address should not make a difference here.
I suspect that some other part of the local DNS resolution goes wrong on localhost.
Is there a difference in the behavior on ping localhost?
Maybe the IPv6 configuration interferes on the host where that test case goes wrong -
does it help if you remove the line
::1     localhost ip6-localhost ip6-loopback
from /etc/hosts?

Hmm, then I guess there may be some interference with IPv6.
Again, does it help if you remove the line ::1     localhost ip6-localhost ip6-loopback ?

I have no idea. If there would be porter boxes that have this problem, then I don't have access to change that.
In any case, if you try to connect to a host you should always have a loop going over all the addresses until you find one that works.

So how if we add to OSSL_HTTP_open() a special handling:
In case opening a connection to localhost fails, retry with <IP_ADDRESS>?

A proper server has 2 options:

2 sockets, one for IPv4, and for IPv6. (Or more if there are more addresses it wants to listen to.)
1 socket that supports both.

A proper client will see 2 addresses and loop over them until one works or all fail. There is no special case for localhost.
From what I understand, both the client and server are broken.

How capable should our little OpenSSL HTTP client become?
I already made it much more capable than the original very primitive one,
which was used just for fetching OCSP responses and CRLs.
Certainly we do not wanna emulate cURL and wget.

So this most likely is related to using AI_ADDRCONFIG. Do we actually try to listen to localhost, or to everything?
Trying all addresses is something anything using sockets should do, not just an http client.

It seems to be using a combination of <IP_ADDRESS> and localhost?

Ah, very good point that the strange behavior should be due to AI_ADDRCONFIG!
This is used by BIO_lookup_ex() while the special behavior is not documented for it,
but here: https://man7.org/linux/man-pages/man3/getaddrinfo.3.html

If hints.ai_flags includes the AI_ADDRCONFIG flag, then IPv4
addresses are returned in the list pointed to by res only if the
local system has at least one IPv4 address configured, and IPv6
addresses are returned only if the local system has at least one
IPv6 address configured.  The loopback address is not considered
for this case as valid as a configured address.  This flag is
useful on, for example, IPv4-only systems, to ensure that
getaddrinfo() does not return IPv6 socket addresses that would
always fail in connect(2) or bind(2).

So as ppc64el-conova-02.debian.org does not have a public IPv4 address configured,
the IPv4 address of localhost apparently is not used for connecting.
Does it help if you add
    BIO_set_conn_ip_family(cbio, BIO_FAMILY_IPV4);

to http_new_bio() in crypto/http/http_client.c
and
    BIO_set_accept_ip_family(acbio, BIO_FAMILY_IPV4);

to http_server_init() in apps/http_server.c?

Not having had the time to properly look at things, it seems localhost is used at one place, which will probably get ::1 only, and <IP_ADDRESS> at the other place.

Ah, right, so this should not work either.
Then I suggest we simply disable the problematic test case involving localhost, as it is anyway not really CMP-specific.


updated mkbom breaks everything due to illegal operation

tl;dr the answer was "restore mkbom from a backup and delete it from /usr/local/bin", any updates or fixes?
every other package breaks due to mkbom illegal options, maybe related to this:
**$ brew cask install alfred**
==> Installing Cask alfred
==> /usr/bin/mkbom: illegal option -- s
==> Purging files for version 3.6.1_910 of Cask alfred
Error: Command failed to execute!

==> Failed command:
/usr/bin/mkbom -s -i /var/folders/xr/fkklpv291b148xvn970sw72m0000gn/T/20180510-55926-1aw2kxt.list -- /var/folders/xr/fkklpv291b148xvn970sw72m0000gn/T/20180510-55926-vkvpoh.bom

==> Standard Output of failed command:
Usage: mkbom [i] [-u uid] [-g gid] source target-bom-file

	-i	Treat source as a file in the format generated by ls4mkbom and lsbom
	-u	Force user ID to the specified value (incompatible with -i)
	-g	Force group ID to the specified value (incompatible with -i)

==> Standard Error of failed command:
/usr/bin/mkbom: illegal option -- s

**$ brew cask install vanilla**
==> Installing Cask vanilla
==> /usr/bin/mkbom: illegal option -- s
==> Purging files for version latest of Cask vanilla
Error: Command failed to execute!

==> Failed command:
/usr/bin/mkbom -s -i /var/folders/xr/fkklpv291b148xvn970sw72m0000gn/T/20180510-56412-1fksa34.list -- /var/folders/xr/fkklpv291b148xvn970sw72m0000gn/T/20180510-56412-gbevab.bom

==> Standard Output of failed command:
Usage: mkbom [i] [-u uid] [-g gid] source target-bom-file

	-i	Treat source as a file in the format generated by ls4mkbom and lsbom
	-u	Force user ID to the specified value (incompatible with -i)
	-g	Force group ID to the specified value (incompatible with -i)

==> Standard Error of failed command:
/usr/bin/mkbom: illegal option -- s

==> Exit status of failed command:
#<Process::Status: pid 56876 exit 1>

I found this: https://github.com/EmpireProject/Empire/issues/1090

Expected behavior and description of the error, including any actions taken immediately prior to the error. The more detail the better.
after running setup/install.sh , /usr/bin/mkbom and /usr/bin/mkbom are replaced by this binary which causes Brew to break.
/usr/bin/mkbom before install (macos version):
Usage: mkbom [-h] [-s] directory bom
mkbom [-s] -i filelist bom
-h              print full usage
-s              create a path-only bom
-i filelist     a file listing in lsbom(8) format used to create the bom file

/usr/bin/mkbom after install (hogliux version):
Usage: mkbom [i] [-u uid] [-g gid] source target-bom-file
    -i      Treat source as a file in the format generated by ls4mkbom and lsbom
    -u      Force user ID to the specified value (incompatible with -i)
    -g      Force group ID to the specified value (incompatible with -i)

If you ignore this guide, your issue may be closed without review.

Also note from that document:

If the maintainer was wrong in closing your issue, please do reply stating why! Closing an issue does not mean the conversation is over. If the guides themselves were unclear, help us improve them! Open first an issue or pull request stating what you found confusing and only then your other issue.


[pt-br] Localize Blog kubernetes-katacoda-tutorials-stop into Brazilian Portuguese

This is a Feature Request

What would you like to be added
Translate blog kubernetes-katacoda-tutorials-stop into Brazilian Portuguese
Why is this needed
Made blog available in other languages (PT-BR).
Comments

/language pt
/good-first-issue

/triage accepted
/assign @pasimoes


Feat/plain js sb integration

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


cra-universal build fails on windows

This could be just an issue with my machine. The build.js wouldn't run npm.
spawnSync('npm', ['run', 'build'] -> spawnSync('npm.cmd', ['run', 'build']
https://github.com/nodejs/node/issues/3675

Yes, I've got some reports on this too, but can't really debug since I don't have windows machine.
Do you know the fix? @ShaneMckenna23

can you open the pull request? I'll take a look and merge it, then release the patch 😄


control flow: if, for, switch

Focus on what makes languages tick, control flow!

Considering this done: we have if, pattern for switch is to use first_of with if, and on the for side, we'd likely do a functional implementation. Generally working with slices in computed_by expressions requires more thinking.


DELETE ALL associated observations when event ends

Do we want to do this or record it in an event history?


UI Step 2 - Issue no. 12

@FIN-casey @FIN-JBarile Please make sure that invertebrate functional groups defined by SeaLifeBase are also provided in the table.

@FIN-casey will apply soonest. She has necessary materials to proceed with this. Thanks.

Hi I added the invertebrate functional groups defined by SeaLifeBase in the data file, but the data file for Country+FAO combination exceeds 100mb, which is the maximum file size for Git. The other two data files (Country+SubCountry+FAO and Ecosystem) are already commited. They are included in the table.
To add the large data file, we need to look for other storage area that the UI can link to. It can be a website. Can we use fishbase website for this?

@FIN-casey one way I can think of to work around this limitation is to the big files into separate files in a folder (grouped by country code (C_Code). This might also speed up the retrieval, because you don't have to plow through a massive json dump.
Example:
instead of single file countryFAO_funcgrp.json
introduce files:
countryFAO_funcgrp/countryFAO_funcgrp_xyz.json
countryFAO_funcgrp/countryFAO_funcgrp_abc.json
countryFAO_funcgrp/countryFAO_funcgrp_def.json
...
where abc, def, xyz are country codes (C_Codes).
Other grouping can be used when suitable. Curious to hear how you are going to address this.

@jhpoelen  thanks for your suggestion.
Apologies for I missed to filter only for inverts.
@agruss2 Are the groups below only the ones of interest for this purpose? Please advise. Thanks.
TargetGrpNum	TargetGrp	Name	Description
25	33554432	cephalopods	cephalopods
26	67108864	shrimp	shrimp
27	134217728	lobsters,crab	lobsters,crab
28	268435456	jellyfish	jellyfish
29	536870912	demersal mollusc	demersal mollusc
30<PHONE_NUMBER>	krill	krill

@FIN-JBarile The list that you provided is too restrictive to my mind.
I thought that SeaLifeBase had data/parameters for a much larger number of non-fish organisms.
I was thinking that the bridge could also provide data for octopods, as well as for a diversity of low-trophic level marine organisms, such as echinoderms, Ascidiacea spp., Brachiopoda spp., Bryozoa spp., Crinoidea spp., Hydrozoa spp., Ophiuroidea spp., Polychaeta spp., Amphipoda spp., Isopoda spp., Mysidacea  spp., Ostracoda spp., Turbellaria spp., Nematoda spp., zooplankton species, phytoplankton species, etc.
OSMOSE users would need paramaters for the abovementioned low-trophic level marine organisms, so as to be able to define a reasonable number of background functional groups for their OSMOSE model.
--> Is it doable to obtain data/parameters from SeaLifeBase for low-trophic level marine organisms such as the ones I mentioned above?
Many thanks.

@FIN-JBarile Many thanks; I had a look at the zip file. I confirm that, for invertebrates, we should apply the exact same rules for data richness as described in #59. This is because, by default, the bridge between FishBase/SeaLifeBase define all functional groups as "focal functional groups"; then, the user can choose to redefine some of the functional groups as "background functional groups".

The invertebrate functional groups are now included in the data set of the UI. The order is based on data richness. Please verify so we can close the issue. THanks!

Hi @jhpoelen thanks for the suggestion, but if I separate the files by Country, there will arise a problem in searching for class or species because the API searches on all the dataset. What I did is I split the CountryFAO file into 2 json files.

@FIN-casey @FIN-JBarile Many thanks for having implemented this! I am going to close the present issue, because what needed to be addressed here has been addressed.
However, I am looking at the config files now and am noticing a number of issues with the invertebrates, even when those are defined as background functional groups (and, therefore, require less parameters); the issue that we have to address now is that the great majority of invertebrates are extremely data-poor.
What I am going to do is that I am going to review the config files in more detail; then, later this week, I will create a new GitHub issue describing what I think we should do to address this problem of data scarcity with the great majority of the invertebrates, which result in too many parameters being set at unrealistic default values.
Talk soon, and thanks a lot once again!


Tie translations to gulp.src location, not location of Gulpfile

When the gulpfile setup lives outside the project folder, the translation file gets messed up.
Let's say for example I have the following setup:
Gulpfile etc sits in
wp-content/themes/gulp-dev
Actual theme sits in
wp-content/themes/mytheme
The Gulp task looks like this (simplified):
var root = '../mytheme/';
var languages = root + 'languages/';

gulp.task('i18n', function () {
    return gulp.src(root + '**/*.php')
	.pipe(sort())
	.pipe(wppot( {
		domain: 'mytheme',
		destFile:  languages + 'mytheme.pot',
		package: themename
	} ))
	.pipe(gulp.dest(languages));
});

The resulting output in the .pot file gets this:
#: ../mytheme/404.php:17
msgid "hmmm. Oops! That page can’t be found."
msgstr ""

The URL reference contains the relative link to the theme folder. It still works (because the URI just points back to itself) but it's not clean.

Good thinking, I'm working on a major rewrite right now and will fix this when it's released.

Fixed in 2.0 by adding option relativeTo


Issue with "msbuild /t:pack"

Hello everyone,
i am using Visual Studio 2017 with the latest updates (from december), I am trying to create a nuget package following this guide: https://docs.nuget.org/ndocs/guides/create-net-standard-packages-vs2017
The problem is that the command: "msbuild /t:pack /p:Configuration=Release" fails, saying that target "pack" doesn't exist. Shouldn't the target be available? My project file looks like this:
https://gist.github.com/feliwir/ac27a8f1420454d490bc73ad9e257da2

Any updates on this?

are you running dotnet msbuild or just msbuild?

Just msbuild

Do you get the same error with dotnet msbuild /t:pack (or dotnet pack)
This might be a documentation bug

The Pack target comes from the new .NET Core SDK.  From the command-line, are you building a project or solution?  If you're building a solution, this won't work until my PR #1497 is in.  If you're building a project, then maybe @nguerrera would know where to start narrowing down a problem?


"ICMP rules can only be applied when the "enable-icmp-rules" flag is set" in CiliumClusterwideNetworkPolicy status, but --enable-icmp-rules='true' is applied

Is there an existing issue for this?

[x] I have searched the existing issues

Version
equal or higher than v1.16.0 and lower than v1.17.0
What happened?
I have a CiliumClusterwideNetworkPolicy which also includes ICMP rule
    - fromEntities:
        - all
      icmps:
        - fields:
            - family: IPv4
              type: 3
            - family: IPv4
              type: 8
            - family: IPv4
              type: 11

in status i see such
status:
  conditions:
  - lastTransitionTime: "2024-09-22T06:55:26Z"
    message: ICMP rules can only be applied when the "enable-icmp-rules" flag is set
    status: "False"
    type: Valid

but in cilium-agent log i see the flag is actually enabled
k logs -n kube-system cilium-pwn5s cilium-agent | grep icmp
time="2024-09-22T06:35:40Z" level=info msg="  --allow-icmp-frag-needed='true'" subsys=daemon
time="2024-09-22T06:35:40Z" level=info msg="  --enable-icmp-rules='true'" subsys=daemon

How can we reproduce the issue?
Install RKE2 v1.30.4+rke2r1 cluster with Cilium CNI, add CiliumClusterwideNetworkPolicy with ICMP rule.
Cilium Version
Client: 1.16.0 82999990 2024-07-23T22:22:14-07:00 go version go1.22.5 linux/amd64
Daemon: 1.16.0 82999990 2024-07-23T22:22:14-07:00 go version go1.22.5 linux/amd64
Kernel Version
Linux worker-01 5.15.0-121-generic #131-Ubuntu SMP Fri Aug 9 08:29:53 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Kubernetes Version
Client Version: v1.30.2
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.30.4+rke2r1
Regression
No response
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Cilium Users Document

[ ] Are you a user of Cilium? Please add yourself to the Users doc

Code of Conduct

[x] I agree to follow this project's Code of Conduct

It looks like the validation is occurring in the cilium-operator, but the Cilium operator doesn't have this flag so there's no way for it to know whether the feature is enabled or not:
$ git grep AllowICMPFragNeeded
daemon/cmd/daemon_main.go:      flags.Bool(option.AllowICMPFragNeeded, defaults.AllowICMPFragNeeded, "Allow ICMP Fragmentation Needed type packets for purposes like TCP Path MTU.")
daemon/cmd/daemon_main.go:      option.BindEnv(vp, option.AllowICMPFragNeeded)
pkg/datapath/linux/config/config.go:    if option.Config.AllowICMPFragNeeded {
pkg/defaults/defaults.go:       // AllowICMPFragNeeded is the default value for option.AllowICMPFragNeeded flag.
pkg/defaults/defaults.go:       AllowICMPFragNeeded = true
pkg/option/config.go:   // AllowICMPFragNeeded allows ICMP Fragmentation Needed type packets in policy.
pkg/option/config.go:   AllowICMPFragNeeded = "allow-icmp-frag-needed"
pkg/option/config.go:   // AllowICMPFragNeeded allows ICMP Fragmentation Needed type packets in
pkg/option/config.go:   AllowICMPFragNeeded bool
pkg/option/config.go:           AllowICMPFragNeeded:             defaults.AllowICMPFragNeeded,
pkg/option/config.go:   c.AllowICMPFragNeeded = vp.GetBool(AllowICMPFragNeeded)

Two possible solutions:

Add the same flag also to the cilium-operator, or
Modify the operator to validate the policy assuming this flag is enabled. This would mark the policy as valid, but then the individual agents could still reject the policy because they can't implement a feature that's disabled on the commandline.

As another solution, I think enable-icmp-rules can be deleted.
I left this flag to be able to disable it when a bug happens. (https://github.com/cilium/cilium/pull/20174#discussion_r895208523)
If this feature seems stable, we can delete this option.

@cilium/sig-policy fyi, this looks like a regression from the network policy validation changes. I think the suggestion above makes sense (removing this additional config option) - but you might want to consider whether v1.16 needs a different fix.

Any updates on this issue?

(For posterity's sake I will leave this as a regression, but it's not really a regression since policies continue to work unchanged).

I've filed a PR to fix this; once landed, it will automatically be backported to v1.16 as needed.


puppeteer 1.4.0 sometimes fails to decode downloaded fonts

Hi there,
My environment:

Puppeteer version: 1.4.0
Platform / OS version: Ubuntu 16.04
Node.js version: 8.9.3

The Issue!
I have a few HTML documents, which i open using puppeteer. The HTML has custom fonts which i have downloaded in the local directory. Previously I used the 1.3.0 version of puppeteer for opening the files and didn't have any issue. After upgrading to version 1.4.0 sometimes puppeteer fails to decode the fonts and I get the following error.
The Error!

PAGE LOG: Failed to decode downloaded font: file:///home/User/Documents/fonts/optim.ttf
PAGE LOG: OTS parsing error: cmap: Failed to parse table.

PS:

The font mentioned above does work in few cases.
This happens with other downloaded fonts as well.
After downgrading to version 1.3.0, did not face this issue.

Any help would be greatly appreciated.
Thank you in advance!

@SarveshRedkar puppeteer 1.4 is doing a much better job in reporting page messages then pptr 1.3 (see https://github.com/GoogleChrome/puppeteer/pull/2400). To verify this, try running pptr 1.4 against chromium from puppeteer 1.3:
const browserFetcher = puppeteer.createBrowserFetcher();
const revisionInfo = await browserFetcher.download('536395');
const browser = await puppeteer.launch({executablePath: revisionInfo.executablePath})

If the logs remain, then that's the case and nothing has changed in practice.
Otherwise, something has changed in Chromium itself; in this case I'd recommend filing a bug upstream to https://crbug.com/new.
Hope this helps.


Deployment

Deployment
User Story
AS A bootcamp student
I WANT to recreate my portfolio site
SO THAT it uses Bootstrap
Acceptance Criteria
GIVEN the GitHub repository that contains application code
WHEN a user trawls the website
THEN Application is deployed at live URL.
AND Application loads with no errors.
AND Application GitHub URL submitted.

Acceptance Criteria Satisfied
GIVEN the GitHub repository that contains application code
WHEN a user trawls the website
THEN Application is deployed at live URL.
AND Application loads with no errors.
AND Application GitHub URL submitted.


serialize for Sequelizer

A Sequelize-specific hook like serialize is possible using something like https://github.com/hauru/sequelize-to-json

Is that what the hydrate hooks are doing?

I believe they convert between ORM and POJO objects.
I wonder if Sequelizer caches decisions given a set of includes. If it doesn't, at least some SQL databases cache their strategy decisions for reuse. In such a case there could be a benefit in reusing a set of includes even if they return more data than needed vs customizing for every case. Should that be true, the proposed hook could be useful.

Consolidating rfelated discussions in https://github.com/feathers-plus/feathers-hooks-common/issues/347


Continue content response with updated claim is ignored by Azure B2C Directory

I've registered an API connector during sign up user flow (before user creation). In some situations I like to update some claim that the user entered during sign up. Therefore my API does return a response content object created with the class below.
The problem is that the Azure B2C does ignore my continue response object. The created user object contains the streetAddress value that the user entered in the sign up form. My API receives that data when get called form B2C but I'm not able to override the value with "XXXXX" with my continue response object.
`
public class ContinueResponseContent
{
public const string ApiVersion = "0.0.1";
    public ContinueResponseContent()
    {
        this.version = ContinueResponseContent.ApiVersion;
        this.action = "Continue";
        this.streetAddress = "XXXXXXX"; //Test overriding the steetAddress claim the user provided during sign up
    }

    public string version { get; }
    public string action { get; set; }
    public string status { get; set; }
    public string streetAddress { get; set; }
}

`
Help is welcome :-)

Closing this issue due to the pull request 10 fix. In the new example, the API Connector returns a static value for the job title.


include phone number on user profile UI in echo

@shereefb commented on Tue Jul 12 2016
We are missing a learner today, and their team needed to reach out to NeEddra to get his phone number.
Let's make these easily accessible from the echo in the user profile. And ask everyone to fill them out in the welcome message.

@jeffreywescott commented on Thu Jul 14 2016
Related: https://github.com/LearnersGuild/los/issues/29

Issue moved to LearnersGuild/rocketchat-lg-sso #34 via ZenHub


chore(master): release 7.1.1

:robot: I have created a release beep boop
7.1.1 (2024-08-08)
Bug Fixes

correct jsdocs links (#938) (105909d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ethersphere/bee-js/releases/tag/v7.1.1 :sunflower:

Release v7.1.1 addresses this.


Add a "Play album" button on an album page

There seems to be no way to play an album in one click when on an album page, beside selecting all the tracks and right-clicking, Add to, After current song (or the Add to button). A play icon on the cover, like when on list mode, should do the trick just fine.

How about this button?

It doesn't work on your demo and I guess it shuffles all the songs of the album? The use case was more to listen to a full album in order.

Hmm you're right, guess I need to update the demo :/. Another button to play the songs in current order sounds feasible.

Based on the feedback in #7 I'm wondering if it should be when hovering the album in the  Artist/Album views:

That behavior is what I expected based on the behavior on the main home page.

That's not a bad idea at all @BrookeDot! I'll find some time to implement it (unless you want to take care of it yourself).

Implemented.


Update AdvancedSettings.Codeunit.al and BarcodeFontProvider.Enum.al

Summary
Test PR with dummy changes
Work Item(s)
Fixes #Nothing

Test app
Tests
Passed
Failed
Skipped
Time

System Application Test
1347
1346 :white_check_mark:
1 :x:

84.965s

1 failing testsSystem Application Test, 1347 tests, 1346 passed, 1 failed, 0 skipped, 84.965 seconds
135074 AppSource Product Manager Test, 11 tests, 1 failed, 0 skipped, 0.097 secondsTestLoadProductNotAllowedOnPremises, Failure      Error: Assert.ExpectedError failed. Expected: Not Supported On Premises. Actual: Not supported on premises.      Stack trace      Library Assert(CodeUnit 130002).ExpectedError line 7 - Library Assert by Microsoft      AppSource Product Manager Test(CodeUnit 135074).TestLoadProductNotAllowedOnPremises line 22 - System Application Test by Microsoft      Test Runner - Mgt(CodeUnit 130454).RunTests line 21 - Test Runner by Microsoft      Test Runner - Isol. Codeunit(CodeUnit 130450).OnRun(Trigger) line 4 - Test Runner by Microsoft      Test Suite Mgt.(CodeUnit 130456).RunTests line 2 - Test Runner by Microsoft      Test Suite Mgt.(CodeUnit 130456).RunSelectedTests line 35 - Test Runner by Microsoft      Test Suite Mgt.(CodeUnit 130456).RunNextTest line 20 - Test Runner by Microsoft      Command Line Test Tool(Page 130455).RunNextTest - OnAction(Trigger) line 7 - Test Runner by Microsoft


feat(serve): Added JWT authentication functions

I have added JWT to the serve methods.
The JWT can either contain a containerId (cid) or not.
My thinking is that some methods (e.g. start) should only be callable by the hub backend and so it will be able to generate those JWTs and use that as authentication to start the container.
Other methods like execute, the hub will be able to generate the JWT with the container ID and then pass that to the browser, which can then directly send to /execute on Nixster, and Nixster will validate the JWT because it was generated with the same token.
If a Nixster method expects a container ID and one was not supplied (or vice versa) then 403 statuses are returned.

Codecov Report

Merging #18 into master will increase coverage by 0.99%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage   31.62%   32.62%   +0.99%
==========================================
  Files           5        5
  Lines         430      472      +42
  Branches       99      110      +11
==========================================
+ Hits          136      154      +18
- Misses        294      318      +24

Impacted Files
Coverage Δ

src/nix.ts
32.67% <0%> (+2.67%)
:arrow_up:

Continue to review full report at Codecov.

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

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


Jira to Zulip stream integration

We just integrated the webhook into a stream.  The idea is that any critical ticket that is created and commented on shows up in a "critical issues" Stream in Zulip.  Though we didn't specify topics, each ticket that is created shows up in the "critical issues" stream with a topic of the ticket number.  For now, this is fine.  But when people comment on the critical ticket, it shows up in a ":" topic.  So if there are multiple comments from multiple tickets, they all appear in the ":" topic.

Is there a way to have the comments to the tickets show up under the correct topics?

This is how we have the webhook setup.

Comments from non Critical issues are showing up in the ":" section.  It looks like all updates to all tickets are showing there vs just the comments on Critical.  The second issue, is that the critical updates, should nest in the appropriate topic.

@Mentalone I tried to see if I could replicate this issue, but I haven't been able to.
Webhook configuration:

Sending the generated fixtures to a zulip instance:

url used: https://localhost:9991/api/v1/external/jira?api_key=wWNEoGg8GP9gM4pVc9EEWS8OB1Op60Nl&stream=jira
Results:

Attached here is the complete set of data used and results that I've obtained when trying to replicate this.

@Mentalone what version of Jira are you using?

We are on v7.7.1.  However, we will be upgrading shortly to the latest release.

I can confirm the issue with Jira Version  7.12.3
Is there a way to log the messages which are received by the jira bot to help you debug the issue?

Here a json of the webhook message sent by our JIRA
{
  "timestamp":<PHONE_NUMBER>460,
  "webhookEvent": "comment_created",
  "comment": {
    "self": "https://jira.xxxx.com/rest/api/2/issue/11936/comment/13880",
    "id": "13880",
    "author": {
      "self": "https://jira.xxxx.com/rest/api/2/user?username=xxxxyyyy",
      "name": "xxxxyyyy",
      "key": "xxxxyyyy",
      "emailAddress"<EMAIL_ADDRESS>      "avatarUrls": {
        "48x48": "https://jira.xxxx.com/secure/useravatar?ownerId=xxxxyyyy&avatarId=10607",
        "24x24": "https://jira.xxxx.com/secure/useravatar?size=small&ownerId=xxxxyyyy&avatarId=10607",
        "16x16": "https://jira.xxxx.com/secure/useravatar?size=xsmall&ownerId=xxxxyyyy&avatarId=10607",
        "32x32": "https://jira.xxxx.com/secure/useravatar?size=medium&ownerId=xxxxyyyy&avatarId=10607"
      },
      "displayName": "Xxxx Yyyy",
      "active": true,
      "timeZone": "Europe/Zurich"
    },
    "body": "this comment is a test",
    "updateAuthor": {
      "self": "https://jira.xxxx.com/rest/api/2/user?username=xxxxyyyy",
      "name": "xxxxyyyy",
      "key": "xxxxyyyy",
      "emailAddress"<EMAIL_ADDRESS>      "avatarUrls": {
        "48x48": "https://jira.xxxx.com/secure/useravatar?ownerId=xxxxyyyy&avatarId=10607",
        "24x24": "https://jira.xxxx.com/secure/useravatar?size=small&ownerId=xxxxyyyy&avatarId=10607",
        "16x16": "https://jira.xxxx.com/secure/useravatar?size=xsmall&ownerId=xxxxyyyy&avatarId=10607",
        "32x32": "https://jira.xxxx.com/secure/useravatar?size=medium&ownerId=xxxxyyyy&avatarId=10607"
      },
      "displayName": "Xxxx Yyyyy",
      "active": true,
      "timeZone": "Europe/Zurich"
    },
    "created": "2019-04-16T00:18:52.460+0200",
    "updated": "2019-04-16T00:18:52.460+0200"
  }
}

It looks like the "webhookEvent": "comment_created" does not contain the issue related information

I found a workaround for now.
I changed the webhook configuration in JIRA as shown below.
This way Zulip is handling the issue_updated instead of the event_created webhook. And the issue updated body contains the required information about the issue

@sclassen thanks for the update.
@eeshangarg @Hypro999 is there anything we want to do codewise here given the above fixture?  It seems like it's a bug there we ever send anything to the : topic...

@sclassen, @timabbott: So, I investigated this. We used to handle issue comments only through issue updates and didn't support "comment events". But comment_created kinda sneaked into the implementation because when I tested it with JIRA Cloud, it did contain all requisite information about the issue that was commented on. However, as it turns out, from the example payload above, it is clear that some payloads might not contain the required issue information to format a useful notification, so we should stick to handling comments via issue updates (i.e. our previous approach).
I just submitted #12244, which ignores comment_created events and updates the documentation to be more specific on how not to set up issue comment notifications.
Please note that this isn't the first time that we have run into inconsistencies in Atlassian's APIs across versions and products. In my opinion, Atlassian's product ecosystem is broken in irreversible ways and as the maintainer of this area, I don't remember ever looking forward to working with their product APIs. :/
Thanks for reporting! Cheers!

Another option to integrate Zulip and Jira is Skyvia. It is freemium and has a very intuitive UI.


Javadoc correction test

Some javadoc corrections.
Also adding a new header, ecsPaymentId.

Ok for the corrections but I don't know if the travis problems will be resolved.
We are going to see it soon when we release the version.

Ok it seems better.
I do not know why we have empy javadocs in the implementation class but.. anyway, we are going to see if now the process works.

Still lots of javadoc errors... but the artifact is well published on Sonatype repository:
https://oss.sonatype.org/content/repositories/snapshots/eu/payzen/sdk/payzen-ws-sdk/1.0.3-SNAPSHOT/

The problems is only for the RELEASE versions.
Errors have to be fixed in the way to RELEASE the module.


config: Add example cwd

The field is optional, but it's nice to have at least one example of
it in use.
Signed-off-by: W. Trevor King<EMAIL_ADDRESS>

LGTM, merging.


Several bug fixes to call SHAP from EconML

the explain_row() method under linear explainer should return the values of shape (dx,dy) in multiple outcome, it will be in line with other explainers

add extra argument to allow feature names being passed through

fix variable name typo in beeswarm plot

Thanks!! This looks good to me.


:card_file_box:🗃Content development page.html

Description
As a Developer I need a Page HTML file where I can add the different sections of my content development page.
Checklist

[x] The code is isolated to the libs/features/pages unless a justified reason is specified.
[x]  A separate commit is used for each small change
[x] The commits and PR are well formatted/use proper documentation

Noted
On Mon, Oct 2, 2023, 3:59 PM Lemmy @.***> wrote:

@.**** requested changes on this pull request.
Good attempt. But you should work on only one issue at a time.
—
Reply to this email directly, view it on GitHub
https://github.com/italanta/elewa-website-html/pull/124#pullrequestreview-1652701589,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6EZV4VSY5TRR3CFKMKEAJTX5K3CDAVCNFSM6AAAAAA5O5D46CVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNJSG4YDCNJYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Good attempt closing this pr as it is already done, please pick another issue


VirtualizedList has hard coded scrollEventThrottle value

Description
We're doing animations based on onScroll in a <FlatList>, but the result is choppy in iOS. We tried setting scrollEventThrottle={16} on the <FlatList> component itself, then inside a renderScrollComponent function. No surprise it didn't work, since it gets overridden in VirtualizedList.
Clearly there's some thought behind setting it to 50, and some concern about Android where the property has no effect, but is it critical? Our Flatlists work pretty well on Android as is, after all... and setting a lower throttle doesn't seem to have any catastrophic side effects on iOS.
Solution
It all depends on what the original reason for hard coding the value to 50 was, but if it was just some happy performance middle ground, then allow it to be overridden by property on the component. @sahrens is a clever guy and will have a great answer I'm sure.
Additional Information

React Native version: Current master https://github.com/facebook/react-native/commit/22a4205bdd63d60c7d7290693aaccfe47e84ebb3
Platform: iOS

Just an oversight, we could easily allow a user override.
But you should really try to use Animated.event with useNativeDriver: true if possible, see the Spindicator component in FlatListExample for a demo.

We're using Animated.event with useNativeDriver: true for our onScroll listener, but due to the size of our animation, scrollEventThrottle still makes a big difference. Which is expected, from what I understand https://github.com/facebook/react-native/blob/28ed5eddf2e54146e915ba4427d3960331231bc3/docs/Animations.md
The FlatListExample behaves the same way. Although it's very difficult to tell the difference between 50 and lower values because the Spindicator is so small, once you start going way above 50 it's noticeable.
Just making sure we're not doing anything terribly wrong, looking forward to the option to override!

Thanks a lot for the fix @sahrens https://github.com/facebook/react-native/commit/f186cfb9d657bacafa32588b96de6ddac0786cf8

Oh it hadn't occurred to me that the throttle would affect useNativeDriver but it hooks into the same event dispatching bus so that makes sense. Diff is in master now in any case. Thanks for the report!

I believe this is issue still applies to the SectionList component

@andreleon: changing the throttle is not supported, you'll need to manage it in your own code, e.g. with a Batchinator

@sahrens Okay, but how can I use the onScroll event for an animation like @iggyfisk? I just used the 500ms throttle as an example to illustrate the problem. Sorry if I wasn't clear. I am trying to bind the onScroll event with Animated.event but my animation is jaggy because of the throttle.

Oh sorry, I'm confusing myself.
Not sure why it would be any different for SectionList. The fix should be in 0.45-stable for all VirtualizedLists.

Note that Android doesn't support throttling at all.


Hover hint on button next to search hits.

Hey,
What does the button next to each hit do? Can you add some mouseover text so we know.
Looks like a clipboard. Is it "copy to clipboard"? Doesn't seem to work if so.
Thanks

It does copy to the clipboard. It requires flash, so perhaps you are blocking that somehow?

I don't have flash installed.

That would be the problem then. I'm going to close this and open an issue for hiding that button if flash isn't present.

Thanks!


FileTransform not clamping low values when applying inverse of 1D LUT

In the case where values below the lower bound of the output range of a 1D LUT are run through the inverse of that LUT, the FileTransform does not behave as expected. Specifically, the FileTransform does not clamp values below the lower bound of the LUT output before applying the inverse LUT transform.
An example .spi1d 1D LUT is as follows
Version 1
From -0.1 2.0
Length 2
Components 1
{
0.0
1.0
}
A very simple Nuke script demonstrating the problem is copied below. The Red channel only covers the 0.0 to 1.0 range and inverts as expected. The green channel goes beyond the 0.0 to 1.0 range and produces large negative values for values below 0.0. Values above 1.0 are clamped properly before the inverse LUT is applied.
set cut_paste_input [stack 0]
version 9.0 v7
Constant {
inputs 0
channels rgb
format "512 512 0 0 512 512 1 square_512"
name Constant1
selected true
xpos -155
ypos -111
}
Expression {
expr0 x/(width-1)
expr1 y/(width-1)*3-1.5
name Expression3
selected true
xpos -155
ypos -11
}
OCIOFileTransform {
file ocio_lut_issue_spi1d
direction inverse
working_space linear
name OCIOFileTransform2
selected true
xpos -155
ypos 48
}

Relates to issues #391 #326 and possibly #289 #196

This may explain the problems when using ACEScc transforms in image filtering see https://www.researchgate.net/publication/305278559_A_retrospective_on_the_adoption_of_the_ACES_technology_at_Framestore

Bump on this now that OCIO is back to life.

Any chance someone can do a change and a pull request for this? Should be
straightforward to add a clamp.
Longer term we could add a BOUNDS option to support both BOUNDS_EXTRAPOLATE
and BOUNDS_CLAMP.
On 21 Sep 2016 03:39, "Troy James Sobotka"<EMAIL_ADDRESS>wrote:

Bump on this now that OCIO is back to life.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/imageworks/OpenColorIO/issues/399#issuecomment-248495157,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAL7hwGRmyfox203mXk6FMilaLLCTeqlks5qsJjhgaJpZM4GOto1
.

Not the most elegant solution, but I have submitted a hotfix #418 for this particular issue, longer term the BOUNDS parameters suggested by @malcolmhumphreys seems the best solution


MultiAdmin 3.x blows out memory on Linux. Allocates way too much before round starts

Here's the scp_multiadmin.cfg that I'm using in both of my servers directories:
#config_location:
disable_config_validation: false
share_non_configs: true
multiadmin_nolog: false
multiadmin_debug_log: true
use_new_input_system: true

log_mod_actions_to_own_file: false
manual_start: false
max_memory: 2048.0
restart_low_memory: 64.0
restart_low_memory_roundend: 256.0
max_players: 25
restart_every_num_rounds: -1
server_restart_timeout: 10
server_stop_timeout: 10
servers_folder: servers
shutdown_when_empty_for: -1
#start_config_on_full:

[21:16:43] The scene is ready! Good luck!
[21:16:43] Waiting for players..
[21:16:43] The scene is ready! Good luck!
[21:16:43] Waiting for players..
[21:16:45] Warning: Program is running low on memory (123.48 MB left), the server will restart at the end of the round if it continues
[21:16:45] Warning: Program is running low on memory (170.79 MB left), the server will restart at the end of the round if it continues
[21:16:46] Warning: Program is running low on memory (-192.59 MB left), the server will restart if it continues
[21:16:46] Warning: Program is running low on memory (-194.91 MB left), the server will restart at the end of the round if it continues
[21:16:46] Warning: Program is running low on memory (-155.09 MB left), the server will restart if it continues
[21:16:46] Warning: Program is running low on memory (-155.34 MB left), the server will restart at the end of the round if it continues
[21:16:47] Warning: Program is running low on memory (-519.50 MB left), the server will restart if it continues
[21:16:47] Warning: Program is running low on memory (-519.75 MB left), the server will restart at the end of the round if it continues
[21:16:47] Warning: Program is running low on memory (-475.03 MB left), the server will restart if it continues
[21:16:47] Warning: Program is running low on memory (-475.29 MB left), the server will restart at the end of the round if it continues
[21:16:48] Warning: Program is running low on memory (-857.23 MB left), the server will restart if it continues
[21:16:48] Warning: Program is running low on memory (-857.49 MB left), the server will restart at the end of the round if it continues
[21:16:48] Warning: Program is running low on memory (-816.63 MB left), the server will restart if it continues
[21:16:48] Warning: Program is running low on memory (-816.89 MB left), the server will restart at the end of the round if it continues

Originally, I had max_memory set to 3584 and restart_low_memory_roundend set to 768, but then it blew out the memory to 3.1 GB and then kept restarting the round as soon as I would join.

MultiAdmin shouldn't be causing this, if you are running plugins, have you tried running it without them?

I talked to Laserman last night and I'm going to debug this on my test server. I pointed the finger at MultiAdmin since that's the only thing I changed. I'll update with results!

Turns out it was AdminToolbox v1.3.8-1. Very sorry for the issue!


[web - engine] small code health items

Just a list of small code clean-ups we should do in the web engine:

[ ] Move test_embedding.dart from lib/ to test/.
[ ] Rename initialization.dart to web_only_api.dart for clarity.

This file should be removed entirely: https://github.com/flutter/flutter/issues/127395

[x] Compile web engine tests with sound null safety.

Fixed in https://github.com/flutter/engine/pull/36169

[x] Move toMatrix32 from initialization.dart to vector_math.dart.

Fixed in https://github.com/flutter/engine/pull/32782

[x] webOnlyAssetManager should return a public type (engine.AssetManager is private). Alternatively, we could try removing it, but that's a bigger project.

Fixed in https://github.com/flutter/engine/pull/42642

[ ] felt test should throw an error when asked to test a file without _test.dart suffix. Otherwise, locally you can do felt test test/something.dart and it will pass, but the test won't run on LUCI.
[x] Convert lineLookup to something with an explicit initialization API, such as initializeLineLookup() + lineLookup pair, like other APIs. Calling a getter for its side-effect is not idiomatic, and it also does not support deinitialization, which we aim to add in the future.

Fixed in https://github.com/flutter/engine/pull/27375

[x] Don't use sync as variable name in futurize and _futurize.

Fixed in https://github.com/flutter/engine/pull/43699

[x] Replace the word "scuba" with "golden" in web tests.

Fixed in https://github.com/flutter/engine/pull/43699

[ ] Re-evaluate how tests initialize the engine. Currently, most call webOnlyInitializePlatform, some call initializeEngine. We also have test_embedding.dart. It's kind of messy.
[ ] canvas_mask_filter_golden_test.dart goldens are broken on iOS Safari (they are blank).

Considering this as fixed.


[Heap Trace Standalone] fix terrible Leaks perf on large records by using doubly linked list (IDFGH-8690)

This augments: https://github.com/espressif/esp-idf/pull/10127 (Edit: this PR will need to be rebased onto https://github.com/espressif/esp-idf/pull/10141)
Leaks: The original implementation memmove()'s the entire record buffer every time a record is freed. This is obviously terrible for perf.
My ESP32-S3 was not useable after ~300 records in SPI ram.
With this change, I can keep Heap Trace Standalone enabled without a noticeable impact on perf, tested up to 2540 records!

Please wait on reviewing this PR.

need to rebase
plan to also add a simple hashmap lookup, for even more speedup

typedef struct {
   heap_trace_record_t* record;
} heap_trace_hashmap_item_t;

// recommended size: 4x-12x the number of records
esp_err_t heap_trace_add_hashmap(heap_trace_hashmap_item_t* items, size_t count)

Closing this PR in favor of a new refactored PR.
https://github.com/espressif/esp-idf/pull/10478


htmlunit-driver 2.55.0 (was 2.43.1)

Updates org.seleniumhq.selenium:htmlunit-driver from 2.43.1 to 2.55.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.seleniumhq.selenium", artifactId = "htmlunit-driver" } ]

labels: library-update, early-semver-minor, semver-spec-minor

Superseded by #332.


Error when running model.stylize

Looking around the issues this might be related to my gpu but not sure.
macOs: Big Sur 11.5.1
GPU: AMD Radeon Pro 5300 4 GB
Python 3.7
I'm able to run
model = tc.style_transfer.create(style, content)
I then try to stylize the images as indicated in the example.
test_images = tc.load_images('test/')
stylized_images = model.stylize(test_images)

I then get the error
RuntimeError                              Traceback (most recent call last)
~/Code/environments/venv/lib/python3.7/site-packages/turicreate/extensions.py in __run_class_function(self, fnname, args, kwargs)
289         try:
--> 290             ret = self._tkclass.call_function(fnname, argument_dict)
291         except RuntimeError as exc:
cy_model.pyx in turicreate._cython.cy_model.UnityModel.call_function()
cy_model.pyx in turicreate._cython.cy_model.UnityModel.call_function()
RuntimeError: Unknown exception
During handling of the above exception, another exception occurred:
ToolkitError                              Traceback (most recent call last)
/var/folders/0b/n86f1wn971z_7ctk99p_48080000gp/T/ipykernel_40949/3861975184.py in
----> 1 stylized_images = model.stylize(test_images)
~/Code/environments/venv/lib/python3.7/site-packages/turicreate/toolkits/style_transfer/style_transfer.py in stylize(self, images, style, verbose, max_size, batch_size)
424             if isinstance(images, _tc.SFrame):
425                 image_feature = _tkutl.find_only_image_column(images)
--> 426                 stylized_images = self.proxy.predict(images[image_feature], options)
427                 stylized_images = stylized_images.rename(
428                     {"stylized_image": "stylized" + str(image_feature)}
Any idea what I'm doing wrong?

Is that the complete output? If not, please share the complete output.

@TobyRoseman here is the entire error:
+------------------+--------------+------------------+
| Images Processed | Elapsed Time | Percent Complete |
+------------------+--------------+------------------+
RuntimeError                              Traceback (most recent call last)
~/Code/environments/venv/lib/python3.7/site-packages/turicreate/extensions.py in __run_class_function(self, fnname, args, kwargs)
289         try:
--> 290             ret = self._tkclass.call_function(fnname, argument_dict)
291         except RuntimeError as exc:
cy_model.pyx in turicreate._cython.cy_model.UnityModel.call_function()
cy_model.pyx in turicreate._cython.cy_model.UnityModel.call_function()
RuntimeError: Unknown exception
During handling of the above exception, another exception occurred:
ToolkitError                              Traceback (most recent call last)
/var/folders/0b/n86f1wn971z_7ctk99p_48080000gp/T/ipykernel_43580/3861975184.py in
----> 1 stylized_images = model.stylize(test_images)
~/Code/environments/venv/lib/python3.7/site-packages/turicreate/toolkits/style_transfer/style_transfer.py in stylize(self, images, style, verbose, max_size, batch_size)
424             if isinstance(images, _tc.SFrame):
425                 image_feature = _tkutl.find_only_image_column(images)
--> 426                 stylized_images = self.proxy.predict(images[image_feature], options)
427                 stylized_images = stylized_images.rename(
428                     {"stylized_image": "stylized" + str(image_feature)}
~/Code/environments/venv/lib/python3.7/site-packages/turicreate/extensions.py in (*args, **kwargs)
303         elif name in self._functions:
304             # is it a function?
--> 305             ret = lambda *args, **kwargs: self.__run_class_function(name, args, kwargs)
306             ret.doc = (
307                 "Name: " + name + "\nParameters: " + str(self._functions[name]) + "\n"
~/Code/environments/venv/lib/python3.7/site-packages/turicreate/extensions.py in __run_class_function(self, fnname, args, kwargs)
291         except RuntimeError as exc:
292             # Expose C++ exceptions using ToolkitError.
--> 293             raise _ToolkitError(exc)
294         ret = _wrap_function_return(ret)
295         return ret
ToolkitError: Unknown exception

Unfortunately, the full output isn't any more helpful. I suggest you take a look at test_images and make sure everything looks good there. Are you just using the data from the User Guide or are you using something else?

No I'm using some other ones.  It worked with one image.  Maybe I'm missing something?
Here I have just one image for one style I wanted to test:
style = tc.load_images('./trainingdata/style/')
In content I have 99 images
content = tc.load_images('./trainingdata/content/')
Then I do the model = tc.style_transfer.create(style, content)
This did not work:
test_images = tc.load_images('./test')
But this worked:
test_images = tc.load_images('./test/31.jpg')


CI OTP 24

Can't actually remember why I wanted/needed to introduce dialyzer here, but it was somehow on my TODO list.
The proposed changes only work with a yet unreleased hex_core which also caused dialyzer warnings, but is a good preview of what's to come.
Here it is:

[ ] wait for https://github.com/hexpm/hex_core/pull/121 to be merged and published
[ ] wait for https://github.com/jelly-beam/verl/pull/21 to be merged and published

Opening as draft for the above mentioned reasons.
Commenting on the stuff that might not be so obvious (or on which I have doubts).

Following https://github.com/erlef/rebar3_hex/pull/232, this has been rebased on top of master. Initial conditions, https://github.com/erlef/rebar3_hex/pull/229#issue-648967795, still apply, though.

hexpm/hex_core#121 was merged and tagged, so I pushed a remade rebar.lock.
After jelly-beam/verl#21, CI should be all 🟢. I've been testing this locally with _checkouts.

jelly-beam/verl#21 was merged and tagged, so I pushed a remade rebar.lock. If CI's Okay that does it for my change 😄

I also fixed coverage stuff. It was hidden by the fact it didn't show an error.
But look at README and see it's not there.


List opponents under one column in series table on individual team pages

On the individual team pages (example) in the series table put all the opponents a team has played under one column instead of separating them by home and away team.
The home/away distinction is unnecessary and just makes it harder to read the table and see all the opponents because you have to go back and forth between two columns. Likewise, the home/away team points should just be own points vs opponent points so you can easily see the outcomes by looking at a single column and you can just sum the column to get total wins or losses.

#79


Closing the server does not clear polling timers

If a socket is in the process of upgrading when io.close is called, the server won't exit cleanly until the polling connection pings out.
Here's a test case:

var http       = require('http');
var Server     = require('socket.io');
var Client     = require('socket.io-client');

var server = http.createServer();

var io = Server(server);

io.on('connection', function (socket) {
	console.log('server connection');

	socket.on('disconnect', function () {
		console.log('server disconnected');
		socket.disconnect();
	});
});

server.listen(8080, function () {
	var socket = Client('http://localhost:8080/');
	socket.on('connect', function () {
		console.log('client connected');

		console.log('closing');
		io.close();
	});

	socket.on('disconnect', function () {
		console.log('client disconnected');
		socket.close();
	});
});

server.on('close', function () {
	console.log('shutdown');
});

The server will output "shutdown", but the process will not exit until the socket times out two minutes later (if debug is setup to show socket.io, you can see the timeout event).  If I delay the close 5 seconds to allow the socket to upgrade, then the process exits cleanly.
The only workaround I can come up with is to force process exit in the server close event. Unfortunately, the reason I made this test case was that my main application never fires the close event.  I can't seem to reproduce that in a test case, but I suspect it might be related to this issue.

+1

Any updates on this? This is creating a real problem in karma right now, as the the timers stop the process from exiting cleanly at the end. See https://github.com/karma-runner/karma/issues/1788 for details

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <Re: [socket.io] Closing the server does not clear polling timers (#2069)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.

@oniono please stop, otherwise I'll have to report your account for abuse

:+1: I think it should be fixed asap

+1

+1

+1

+1

+1

While I'm thankful for the sudden amount of attention this issue is getting, please don't fill up the comments with +1s. All that does is annoy project maintainers.  Explanations of why this bug affects you are much more effective.

We've merged a solution for this and it will shortly be in a release.


Using C++ with the STM32F4 Discovery causes linker errors, what am I doing wrong?

I've created a project using the platformio init --board=disco_f407vg command, and then proceeded to write a main.cpp. Everything seemed to work fine at first but when I try to use new or delete I get linker errors. Specifically I have a line int * testInt = new int; which then causes main.cpp:12: undefined reference to `operator new(unsigned int)' (where line 12 is the line containing the new call).
I'm guessing I still have to add some setting to the platform.ini to configure it for full C++ support, but I can't find anything about such a setting in the documentation. What am I missing?
Thanks for the help (also thanks for building this software, I don't think I've ever had a toolchain and bsp install this quickly and easily!)

Hello @apvanzanten.
I have updated ldscript for stm32f4, so now you can use PlatformIO develop branch or stable 2.0.0 with this platformio.ini:
[env:disco_f407vg]
platform = ststm32
framework = cmsis
board = disco_f407vg
build_flags = -lstdc++ -lnosys

Sweet, thanks for the quick fix! :+1:


Link updates and format changes

Some link updates and format changes!

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

I have updated the formatting so the links are no longer broken!


team user can't import data library dataset even with enough data storage space

@Kartones cc @iriberri
Data library datasets are failing to import into @makella 's team account.
That account's using 0.0 of 500.0 MB with 157286400 Max import file size.
To replicate: try connecting Populated Places dataset from Data library
That account's import errors are mostly:
ERROR 1: ERROR: canceling statement due to statement timeout
(another error in same files is ERROR 1: PostgreSQL driver doesn't currently support database creation.)
Populated places is connecting fine into my team account, but when trying to import same dataset into her acct it's timing out over same internet connection.

/cc @rafatower could this have something to do with the new extension version?

https://github.com/CartoDB/cartodb/issues/5009

Fixed

@Kartones @rafatower @ztephm thanks for the help! tested just now and datasets are uploading!

I was able to upload one dataset today but it is giving me a time out error again.  Anything I can do to provide helpful information, let me know.

Rafa made a fix but he needed to roll it back because it was causing other issues. I recommend you to follow this problem directly at #5009, so that we don't have so many open for the same cause. :-)
I guess you were able to import the files when the fix was in production, but then we needed to go back. I have increased the timeout of your account to try to minimise the effect of this problem.


Warn against duplicates in collection

Could you warn the user in case of a duplicate entry in a collection. Actually there is no limit to enter the same object multiple times. See https://www.rijksmuseum.nl/nl/rijksstudio/1633433--philippe-lejeune/verzamelingen/testduplicate?ii=0&p=0

I could imagine some use cases where it would be beneficial to add duplicates. Especially considering the possibility of cropping and highlighting different areas of the image. We might consider adding a warning though.


Scoring layout does not properly and symmetrically show Zings

This seems to be a new issue, or maybe it's just related to the player index, in any case the following game had more than one Zing for both players (IIRC), but they're not displayed properly (and not the same for both views):

I just looked at the code, and I think there are multiple issues here:

I think Zings are simply not visualized at all currently.
Instead, some cards stand out too much. This seems to affect the opponent's hand, and one of the first (/last?) two cards only.
Carefully looking at the above screenshots, I would think that the x position of one of these two cards is wrong as well.

Looking at the code, we have three offsets:

card_offset which is used to spread out the hands and for the isometric stack visualization. This is not used for the score stacks, but could be!
peeping_offset which is used for different purposes for different stacks (e.g., for pulling out cards below stacks, and/or for making their values visible), which is apparently used for spreading out the cards. (Maybe this is also related to the buggy positioning at the end.)
score_offset which has been introduced to show the value (0/1/2 points) of each card.

It is not obvious to me how the Zings should be visualized – they could possibly be separated from the rest with a horizontal gap? After all, they should already be at the end.
Furthermore, it should be checked if the order of the cards is correctly visualized.

f6886e379b3a6e5d0f0806244d72e6baff1c578b might have solved most of this (more testing needed before closing)


BEAM-4182 fix to allow JMSMessageId to be null

the equals assumes that JMSMessageID is always present, but the spec if fine to have this being null:

Message header field references are restricted to JMSDeliveryMode,
JMSPriority, JMSMessageID, JMSTimestamp, JMSCorrelationID, and
JMSType. JMSMessageID, JMSCorrelationID, and JMSType values may be
null and if so are treated as a NULL value.

This PR removes the jmsmessageID from the equals since it can be actually null, see quoted SPEC text.

@jbonofre mind taking a look here ?