fix merge issue

@asulwer please check if the build is still passing when approving PRs 😅
the previous PR conflicted, that's why

please move IDemo into its own file please, separation of interests

please move IDemo into its own file please, separation of interests

that's done

nice, thanks.
So I found a bug, it might be related to PR #32.
Im working on a possible solution here, will explain in the next one

keep me posted


Fixed grammar in various comments and documentation.

I searched with grep -r -E " (\w) \1 ", and ignored matches in 1.11.txt, 1.9.txt and 1.8.txt (the pattern the the is found there).

@atombrella Have you seen #10192 and #10191?

@felixxm No. I've noticed the presence of duplicate words some days ago, and decided to do a grep. Anyway, no need for duplicate PRs.


Parse Log In Problem

When I go to log a user in, I get the error message: [Error]: Invalid username/password. (Code: 101, Version: 1.12.0). The user exists, is shown in Parse Dashboard, and I can sign up just fine. It does not work on a real device and the simulator.
Xcode: 7.3
iOS: 9.3
My code is below:
  PFUser.logInWithUsernameInBackground((usernameField?.text)!, password: (passwordField?.text)!) { (user: PFUser?, error: NSError?) -> Void in
            if error == nil {
                //Segue to account
                self.defaults.data.setValue(self.usernameField?.text, forKey: "currentUser")

               self.presentViewController(AccountInterface(), animated: true, completion: {

               })
            } else {
                print(error?.description)
            }
        }

@techsd-repo, any chance you can get us a repro?
This clearly and definitely works (you are the only person reporting this), but I would love to help you debug this and investigate if necessary.

Few follow-up questions here:

Are you using parse.com or Open Source Parse Server?
Can you signup a new account and paste it here so we can test it?
Does the issue reproduce outside of this specific application?
Are you using a combination or any of Anonymous Users/Facebook Users/Twitter Users/ Custom Auth Provider Users?

All of these will help us debug a lot and identify the root cause, so let's work together to get this going.

I am using Parse Server
Server URL: http://parseserver-quizical.herokuapp.com Username: test01 Password: test01
No
I am not using any Twitter/Facebook users

You can send it over FB Messenger or to nlutsenko(at)fb.com

I sent the keys over FB Messenger, I just wanted to make sure you got them.
:)
On Tue, Mar 22, 2016 at 9:00 PM Nikita Lutsenko<EMAIL_ADDRESS>wrote:

You can send it over FB Messenger or to nlutsenko(at)fb.com
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/ParsePlatform/Parse-SDK-iOS-OSX/issues/866#issuecomment-200099364

I don't know how but I can now log in with users.. Thanks for the help :)

Glad you found the solution. Let me know if there is anything else that comes up.


Link to beta binaries

Set the links for the beta binaries.
I'll merge once they've been built and pushed, then we'll need to rebuild and push the docs.
A few other things that we should look into later:

can we have a parameter somewhere and reuse it? that would be easier and less error-prone
lots of trailing whitespaces in those files. we tend to avoid those.

This change is

binaries are now available.

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

install-cockroachdb.md, line 100 [r1] (raw file):
Mark, can you change the filename here, too?

install-cockroachdb.md, line 264 [r1] (raw file):
Marc, can you change the filename here, too?

Comments from the review on Reviewable.io

 after few updates I mention.

Comments from the review on Reviewable.io

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 2 unresolved discussions.

install-cockroachdb.md, line 100 [r1] (raw file):
oops. right.

install-cockroachdb.md, line 264 [r1] (raw file):
ditto. nice catch.

Comments from the review on Reviewable.io

oh, another thing. When do we want to start mentioning the version number in the actual text? eg: instead of "download the latest ...", maybe "download version beta-..."

As discussed in person: For now, we'll just reference the "latest version", but once we're able to point to a list of previous versions, we'll work that into the docs.


Integration of Matbench-discovery

Brief description of changes
Integrating the table and figs from Matbench-discovery into Matbench.
@ardunn @janosh

Couple of thoughts
@hrushikesh-s

Looks like there are two copies of the "scaled error" explanation, including one below the matbench-discovery graphs. The scaled error equation does not correspond to matbench-discovery so tthis second one should be removed

Looks like there is some formatting issue with the table and the plots.  Since the majority of table cells are basically twice as wide as they need to be, would it be possible to make them narrower?

Description of matbench-discovery is a good first attempt but maybe could use one more sentence describing what the task actually is. So maybe add a sentence in the middle like "Matbench-discovery compares ML structure-relaxation methods on the WBM dataset (<<include citation/link>>) for ranking ~250k generated structures according to predicted hull stability (42k stable)." Or something worded better than that but which gets the same idea across

@janosh

Are you ok with the placement or would you want to see it at the top of the main page, or maybe (I was just thinking recently) as it's own page with just a link/banner at the top of the matbench main page?
Corresponding to @hrushikesh-s point (2) above: is there a way this table can be generated in smaller width? The individual cells look like they could be about 50% of their current width, esp. if some of the metrics are abbreviated (e.g., "Precision" --> "Prec.")

The above errors are now resolved.

Are you ok with the placement or would you want to see it at the top of the main page, or maybe (I was just thinking recently) as it's own page with just a link/banner at the top of the matbench main page?

That's a great suggestion! I think a separate page would help with readability and a banner at the top with discoverability (excuse the pun) as it's clearer which content pertains to property prediction vs crystal discovery. Would also avoid the need for Hrush to split up existing pages into pieces to insert content between existing static content.

is there a way this table can be generated in smaller width? The individual cells look like they could be about 50% of their current width, esp. if some of the metrics are abbreviated (e.g., "Precision" --> "Prec.")

@ardunn I was looking at the wide table columns together with Hrush yesterday and couldn't figure out where the min-width is coming from. The columns are as narrow ad the text permits in the manuscript and the live results page suggesting the width is not part of the table's own CSS but might be set by the mkDocs template? @hrushikesh-s Maybe you could check in the browser dev tools if the table has
table {
  table-layout: fixed;
}

applied to it and if so remove/override that CSS rule.

@ardunn , @janosh

Regarding two copies of the scaled error:
This is now fixed.

Regarding the formatting of tables and plots:
There was a min-width of the table, which was set to 5rem, by the default MKDocs template. After changing it to 2.2rem, the table looks in proper shape. The bottom axis of the plots is now completely visible.

@janosh

I've created a new branch which has a dedicated page for Matbench-discovery. Does this placing look good?

@hrushikesh-s Nice work! I would shorten the dir name
- docs_src/Discoverability/index.md
+ docs_src/discovery/index.md

And you can merge scripts/doc_snippets/index_0.md back into scripts/doc_snippets/index.md.

@janosh

I've changed the dir name from Discoverability to Discovery
scripts/doc_snippets/index_0.md is now merged into scripts/doc_snippets/index.md

@ardunn @janosh
Do you think we should rename the Leaderboard banner to Leaderboard-Prediction and Discovery to Leaderboard-Discovery?

Maybe Leaderboard-Property instead of Leaderboard-Prediction since everything (even stability) requires prediction.

@ardunn can you pls review this PR?


Add ProcessId and DisplayName label to service collector

Hi there!
It would be useful for us to have the "ProcessId" and "DisplayName" properties as a labels for the service metrics.
Would you be open to accept a PR for this addition ? Thanks!

Hi @carlpett I added this PR to represent what would be the change for this issue, before I added documentation changes I would like hear your comment on this.
I think that this also could be resolved creating an info metric for the service (wmi_service_info) with the name, processID and DisplayName of the service.
Is there any inconvenient adding this labels that I may not aware of?
Thanks

Hey @gsanchezgavier,
Sorry for taking a while to look at this, only have a limited amount of time available for the exporter right now. As you suggest, I would prefer to go with a _info approach instead. This would mean we avoid the label changes on the actual state metrics whenever a service is restarted.
Sounds okay?

It sounds good , I will update the PR with those changes and add documentation for that


Screenshot title works for "frameit white" not "frameit silver"

New Issue Checklist

[ ] Updated fastlane to the latest version
[ ] I have read the Contribution Guidelines

Issue Description
Title for screenshot is working for frameit white but not for frameit silver.

Complete output when running fastlane, including the stack trace and command used
when I run frameit silver I don't get any error.

INSERT OUTPUT HERE

Configuration Files
Please copy the complete content of your Fastfile and any other configuration files you use below:
Customise this file, documentation can be found here:
https://github.com/KrauseFx/fastlane/tree/master/docs
All available actions: https://github.com/KrauseFx/fastlane/blob/master/docs/Actions.md
can also be listed using the fastlane actions command
Change the syntax highlighting to Ruby
All lines starting with a # are ignored when running fastlane
By default, fastlane will send which actions are used
No personal data is shared, more information on https://github.com/fastlane/enhancer
Uncomment the following line to opt out
opt_out_usage
If you want to automatically update fastlane if a new version is available:
update_fastlane
This is the minimum version number required.
Update this, if you use features of a newer version
fastlane_version "1.48.0"
default_platform :ios
platform :ios do
before_all do
# ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."
# increment_build_number

# xctool # run the tests of your app

end
desc "Runs all the tests"
lane :test do
snapshot
end
desc "Submit a new Beta Build to Apple TestFlight"
desc "This will also make sure the profile is up to date"
lane :beta do
cert
sigh
gym(scheme: "tsc") # Build your app - more options available
#pilot
# sh "your_script.sh"
# You can also use other beta testing services here

end
desc "Deploy a new version to the App Store"
lane :deploy do
snapshot
sigh
gym(scheme: "tsc") # Build your app - more options available
# deliver(force: true)
# frameit
end
You can define as many lanes as you want
after_all do |lane|
# This block is called, only if the executed lane was successful
# slack(
#   message: "Successfully deployed new App Update."
# )

end
error do |lane, exception|
# slack(
#   message: exception.message,
#   success: false
# )
end
end
More information about multiple platforms in fastlane: https://github.com/KrauseFx/fastlane/blob/master/docs/Platforms.md
All available actions: https://github.com/KrauseFx/fastlane/blob/master/docs/Actions.md
Fastfile:
# INSERT CONTENT HERE

Environment
fastlane version (fastlane -v): fastlane 1.67.0
Do you use bundler, rbenv or rvm? rvm

Hi @kNeerajPro! Great to hear from you. Can you send over your title.strings and your Framefile?
Thanks!

title.strings file -
"Login" = "This is the Login Screen";
Framefile.json -
{
"default": {
"keyword": {
"fonts": [
{
"font": "./fonts/OpenSans-Regular.ttf",
"supported": ["en-US"]
}
]
},
"title": {
"fonts": [
{
"font": "./fonts/OpenSans-Regular.ttf",
"supported": ["en-US"]
}
],
"color": "#545454"
},
"background": "./background.jpg",
"padding": 50
},
"data": [
{
"filter": "Login",
"keyword": {
"color": "#d21559"
},
"title": {
"text" : "Login Screen"
}
}
]
}

Hi @kNeerajPro! Thanks for following up. It's important to note that, some stock images provided by Apple still have a white background instead of a transparent one. You'll have to edit the Photoshop file to remove the white background, delete the generated .png file and run frameit again. Also, we only officially support black and silver.
Let me know how it's going on your end :+1:

I am sorry you didn't understand my question. Its not working with the black one but works with other. I don't understand why. Please see the above screenshots attached. Any ideas about it?

Hi @kNeerajPro! Thanks for following up. Can you send me the full text output when your run frameit silver?
Thanks!

I will get back to you on this after couple of days.


Stop creating namespace for now

Remove create namespace from helm install for now
/hold
/cherry-pick release-0.12

How would we specify the namespace instead?

In the helm chart. That needs to merge first

first https://github.com/aws/eks-anywhere-packages/pull/552

After this merges https://github.com/aws/eks-anywhere-build-tooling/pull/1464

/unhold

/lgtm

/approve

/cherry-pick release-0.12


Update test doc for 24.04 [skip ci]

Update test doc for 24.04
To follow up https://github.com/NVIDIA/spark-rapids/pull/10394

build


Fixes issues with stuck state locks blocking API

- What I did
Fix issues with the API becoming unavailable due to stuck container state locks.
Fixes #17443
- How I did it
This introduces a new lock specifically for reading/writing to the
container struct.
This allows things like the API to continue to be
operable even if there is a problem with the main container lock (for
example, a stuck netlink call).
- How to verify it
Make container hang (for example reproduce kernel bug in #17443), and call docker ps, docker inspect, etc.
- Description for the changelog

Separates container state synchronization with container field access
Signed-off-by: Brian Goff<EMAIL_ADDRESS>

would that still a problem with containerd integration? from what I understanding,  the containerd has a start timeout 15s, if the container is stuck during staring, it always return with timeout error and the container's lock will be unlock on daemon.  correct me if I'm wrong -:)

@coolljt0725 Yes, it's still a problem because containerd is only handling 1 part of the process (no pun intended!).
Even with the timeout the daemon still gets stuck, and the timeout is likely to be increased to 2mins.

@sethp-jive fwiw, we increased the timeout to 2 minutes in 1.11.2, which was released today

@sethp-jive Ideally read/write to the container struct would require the new lock, changes to the container state itself would require the main container lock.
So yes, if something is making a change to the struct while not locked (or something is reading from the struct while not locked) this is a race.
Races on things like docker ps and docker inspect are somewhat ok (not a big deal if a field changed) ... though we should try to minimize this.

@thaJeztah The failure we're seeing won't time out – I'm not sure at what level the retry is happening (whether it's in the netlink library or the kernel itself), but the broken kernel state leads to an-every-10s repeat of the same message, and the goproc holding the lock stays down in a syscall in every backtrace I've seen, even up to 110 minutes later.
@cpuguy83 ah, I see – perhaps then locking DataLock in releaseNetwork is what's called for?
I'm worried because defer container.DataLock.Unlock() inside releaseNetwork is dangerous – if that lock is held when we call sb.Delete(), we're blocking the API on that never-to-return kernel call again.

@sethp-jive Well, don't defer :)

I'm going to close this because I think this will be too error prone.
I have some other ideas I can work on that will probably be better.

@cpuguy83 have you had a chance to get back around to this issue? It's been a major problem for us (recurring dozens of times a day, with the only non-code remediation being a reboot). We're running with the patch I proposed over in #23175 on those nodes and that's so far saved us from needing to reboot constantly.
Alternatively, if you would give some more feedback on #23175 I'd really like to clean that up and resubmit it so we can at least mitigate the impact of this problem in the short-term while the long-term fix is in progress. It should be simple enough to change tryLock not to leak goprocs as LK4D4 suggested, would that be interesting enough for a second look?

@cpuguy83 with 1.12 we can restart dockerd without effecting running containers. This will give an alternative correct for docker ps hang. Or do you see any other issues ?


NuttX config maintenance

I asked this on gitter some time ago with not responses and today we were bit by it with the aerofc board.
See https://github.com/PX4/Firmware/pull/6397
How are the nuttx configuration files for the different boards maintained?
I thought the reconfigure_nuttx_* target was supposed to do that, but it doesn't work. Removing the call to "make oldconfig" works, but it produces a configuration so different that I wonder if this is really the way you are doing it.
And talking about different configuration, if I do this it would contain the fix from the PR above, but just because that's the default value for that entry.
Trying for another board like px4fmu-v2 didn't help: lots of difference from what's in tree.

@davids5 @LorenzMeier see above

See https://github.com/PX4/Firmware/pull/6397#issuecomment-273943550 for directions.


Review all variables names generated from swagger

Azure::Storage::Files::Shares::ListDirectoryHandlesSegmentResult.HandleList

Review the names provided by Swagger and change them appropriately.

DataLake service done.
@katmsft Do you want to paste the review comments here? Or anywhere else you prefer, Just for the record?

DataLake service done.
@katmsft Do you want to paste the review comments here? Or anywhere else you prefer, Just for the record?


Update Posts “references”

Automatically generated by Netlify CMS

👷 Deploy Preview for ashfolios processing.

Name
Link

🔨 Latest commit
3a223e9512570b6c484548e4131f15ae458b4a8c

🔍 Latest deploy log
https://app.netlify.com/sites/ashfolios/deploys/644a03d664a23f0008f85e91


documentation of components and how final html is formed

[skip ci]
Signed-off-by: Arnav Gupta<EMAIL_ADDRESS>

merging myself as it has no change in code (only docs)


Update or replace pytest-django-ordering

We use the package pytest-django-ordering in edx/course-discovery, edx/ecommerce. It hasn't yet added support for Django3.2. Please follow the guidance in https://openedx.atlassian.net/wiki/spaces/AC/pages/3036972032/Handling+Outdated+Dependencies to resolve the problem this poses for the Open edX Django 3.2 upgrade.

HI @natabene.
I'm happy to take this one.

Have to mention that https://github.com/edx/upgrades/issues/20 is a showstopper for course-discovery Django 3.0+ compatibility and must be upgraded to simplify any further work with course-discovery repo.

@cmltaWt0 Thank you so much for reaching out, I appreciate. @jmbowman Please see previous comment from @cmltaWt0.

Maintainer doesn't have any plans for Django 3.2 support.
I'll analyze an effort to create a PR to do it by myself and then will decide are we going to help the maintainer or find an alternative (as an alternative we can use pytest-ordering directly)

@cmltaWt0 Thank you very much for an update!

Further investigation of the history of pytest-django-ordering reveals the interesting conclusion - we don't need this plugin since repos use pytest-django>=3.9 where an appropriate bugfix was introduced.
There is a code from pytest-django-ordering:
@pytest.hookimpl(trylast=True)
def pytest_collection_modifyitems(items):
    def get_marker_transaction(test):
        try:
            marker = test.get_closest_marker('django_db')
        except AttributeError:
            # pytest < 3.6.0
            marker = test.get_marker('django_db')

        if marker:
            validate_django_db(marker)
            try:
                return marker.kwargs.get('transaction')
            except AttributeError:
                # pytest-django < 3.3.0
                return marker.transaction

        return None

    def has_fixture(test, fixture):
        fixturenames = getattr(test, 'fixturenames', None)
        return fixturenames and fixture in fixturenames

    def weight_test_case(test):
        """
        Key function for ordering test cases like the Django test runner.
        """
        is_test_case_subclass = test.cls and issubclass(test.cls, TestCase)
        is_transaction_test_case_subclass = test.cls and issubclass(test.cls, TransactionTestCase)

        if is_test_case_subclass or get_marker_transaction(test) is False:
            return 0
        elif has_fixture(test, 'db'):
            return 0

        if is_transaction_test_case_subclass or get_marker_transaction(test) is True:
            return 1
        elif has_fixture(test, 'transactional_db'):
            return 1

        return 0

    items.sort(key=weight_test_case)

which is almost completely replaced in pytest-django>=3.9 plugin:
@pytest.hookimpl(tryfirst=True)
def pytest_collection_modifyitems(items: List[pytest.Item]) -> None:
    # If Django is not configured we don't need to bother
    if not django_settings_is_configured():
        return

    from django.test import TestCase, TransactionTestCase

    def get_order_number(test: pytest.Item) -> int:
        test_cls = getattr(test, "cls", None)
        if test_cls:
            # Beware, TestCase is a subclass of TransactionTestCase
            if issubclass(test_cls, TestCase):
                return 0
            if issubclass(test_cls, TransactionTestCase):
                return 1

        marker_db = test.get_closest_marker('django_db')
        if not marker_db:
            transaction = None
        else:
            transaction = validate_django_db(marker_db)[0]
            if transaction is True:
                return 1

        fixtures = getattr(test, 'fixturenames', [])
        if "transactional_db" in fixtures:
            return 1

        if transaction is False:
            return 0
        if "db" in fixtures:
            return 0

        return 2

    items.sort(key=get_order_number)

So I'm going to remove this dependency completely, test it and open PRs into discovery and ecommerce repos.
Related links:

https://github.com/pytest-dev/pytest-django/issues/827
https://github.com/pytest-dev/pytest-django/pull/830
https://github.com/pytest-dev/pytest-django/pull/820

@cmltaWt0 Thank you for an update, I appreciate.

@natabene the change introduced by @cmltaWt0 to remove pytest-django-ordering seems reasonable

@natabene Let's close it.


Comparator popup https://github.com/GoodAI/space-engineers-ai-spaceship-generator/issues/6

Create a popup window if the user tries to upload a .zip file instead of the structure blueprint

Implemented in https://github.com/arayabrain/space-engineers-ai-spaceship-generator/commit/1b292998230a9de122f54b508b8a5720301e3f32


MutableEntry: improve JavaDoc and Spec clarification

The actual JavaDoc of MutableEntry is not really capturing all the fundamentals.

Class level: Reference to the Entry Processor
Method getValue() should get overloaded and have some more description. It extends the contract of Entry and performs a load operation

There is also an interesting aspect, which is maybe missing in the Spec. E.g. the operation ...
CacheEntryProcessor<K, V, V> p = new CacheEntryProcessor<K, V, V>() {
  @Override
  public V process(MutableCacheEntry<K, V> entry, Object... arguments) throws Exception {
    if (!entry.exists()) {
      return null;
    }
    return entry.getValue();
  }
};
V cachedValue = invoke(key, p);

... would only return the value, if it is existing in the cache (some caches have getIfPresent for this).
But, what happens if the entry expires between the exists() and getValue()? IMHO the spec should say:

If exitsts() returned true, subsequent calls to getValue() always return a non null value from the cached entry and do not invoke the loader.

Will prep an agreeable proposal.

Added Class level reference to EntryProcessor to explain the connection.
Done.
Method getValue() should get overloaded and have some more description. It extends the contract of Entry and performs a load operation. Only invokes loader if read through is enabled.
Done. Good suggestion. Makes things much clearer. Left out t
Also check Spec for occurrences of "If loader defined" needs to be replaced with "If loader defined and read through configured"
No occurrences
These changes just address JavaDoc clarification. For the rest break those into separate issues in case we need to change behaviour. For those tests would need to be added.

Commit was done in: https://github.com/jsr107/jsr107spec/commit/72373a0729446474a178cd7473287cb8ebf47537

For the rest break those into separate issues in case we need to change behaviour. For those tests would need to be added.

Hopefully the last point above is not changing behavior, but actually clarifying things that are implied by the atomic nature of the operation. But yes, its better to check/extend the TCK tests first.


String TOKEN $ContainsVariableSpecialCase obfuscation issue

Problem
Based on comments, it seems you are already aware of issues surrounding string tokens containing embedded variables:
# If a variable is present in a string, more work needs to be done to extract from string. Warning maybe should be thrown either way.
# Must come back and address this after vacation.
# Variable can be displaying or setting: "setting var like $($var='secret') and now displaying $var"
# For now just split on whitespace instead of passing to Out-Concatenated

Hope you are currently enjoying that vacation :)
I found an issue where one of these special string token cases lost it's surrounding double quotes during String token obfuscation, causing it to be interpreted as a Command token in subsequent obfuscation operations. This causes errors during execution of the final obfuscated script, but no errors during obfuscation.
I discovered this issue while obfuscating the powerview Empire module(s).
Example
This line is the culprit in the Empire powerview code.
Here is a reduced example:
PS> Invoke-Obfuscation -ScriptBlock {$DN = "DC=$($Domain.Replace(',', ',DC='))"} -Command 'Token\String\2' -Quiet

Outputting ObfuscatedCommand to stdout and exiting since -Command was specified and -NoExit was not specified:

$DN = DC=$($Domain.Replace(',', ',DC='))

You might be able to spot the problem. The encapsulating double quotes around the string token are lost. This causes subsequent obfuscation operations to interpret this string as a command token:
PS> Invoke-Obfuscation -ScriptBlock {$DN = "DC=$($Domain.Replace(',', ',DC='))"} -Command 'Token\String\2,Token\Command\3' -Quiet

Outputting ObfuscatedCommand to stdout and exiting since -Command was specified and -NoExit was not specified:

$DN = &((("{4}{1}{3}{2}{6}{7}{8}{5}{9}{0}{10}"-f '=','Re','0},','place({','DC={1}({1}Domain.',',D','{','0}, {0','}','C',
'{0}))'))-F  [ChAr]39,[ChAr]36)

This obviously then causes problem during execution of the resulting script:
PS> $DN = &((("{4}{1}{3}{2}{6}{7}{8}{5}{9}{0}{10}"-f '=','Re','0},','place({','DC={1}({1}Domain.',',D','{','0}, {0','}','C','{0}))'))-F  [ChAr]39,[ChAr]36)
& : The term 'DC=$($Domain.Replace(',', ',DC='))' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try
again.
At line:1 char:8
+ $DN = &((("{4}{1}{3}{2}{6}{7}{8}{5}{9}{0}{10}"-f '=','Re','0},','plac ...
+        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (DC=$($Domain.Replace(',', ',DC=')):String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

Solution
I was able to identify the section of code that was removing the double quotes, and made an exception for these $ContainsVariableSpecialCases:
# If a variable is present in a string, more work needs to be done to extract from string. Warning maybe should be thrown either way.
# Must come back and address this after vacation.
# Variable can be displaying or setting: "setting var like $($var='secret') and now displaying $var"
# For now just split on whitespace instead of passing to Out-Concatenated
$ContainsVariableSpecialCases = $False
...
    # For Parameter Binding the value has to either be plain concatenation or must be a scriptblock in which case we will encapsulate with {} instead of ().
    # Actual error that led to this is: "Attribute argument must be a constant or a script block."
    # ALLOWED     :: [CmdletBinding(DefaultParameterSetName={"{1}{0}{2}"-f'd','DumpCre','s'})]
    # NOT ALLOWED :: [CmdletBinding(DefaultParameterSetName=("{1}{0}{2}"-f'd','DumpCre','s'))]
    If($EncapsulateAsScriptBlockInsteadOfParentheses)
    {
        $ObfuscatedToken = '{' + $ObfuscatedToken + '}'
    }
    ElseIf(($ObfuscatedToken.Length -eq $TokenContent.Length + 5) -AND $ObfuscatedToken.SubString(2,$ObfuscatedToken.Length-4) -eq ($TokenContent + ' '))
    {
        If($ContainsVariableSpecialCases) {
            $ObfuscatedToken = '"' + $TokenContent +'"'
        }
        Else {
            $ObfuscatedToken = $TokenContent
        }
    }
    ElseIf
...

Fixed: 950785e5f846fb0e8d3be8484d2ba466c2340469


Relying more on ray?

If we want to get rid of tracking losses per epoch by ourselves with ray's report function, we also have to use its training call which would probably look somewhat like so:
scaling_config = ScalingConfig(num_workers=3)
# If using GPUs, use the below scaling config instead.
# scaling_config = ScalingConfig(num_workers=3, use_gpu=True)
trainer = TorchTrainer(
    train_loop_per_worker=tcn_trainer.train,
    scaling_config=scaling_config,
)
result = trainer.fit()

Advantage is that we might get something out of the scaling config? And that it then becomes apparently very, very easy to hook up tensorboard, mlflow and more by simply adding
run_config=RunConfig(
        callbacks=[
            MLflowLoggerCallback(experiment_name="train_experiment"),
            TBXLoggerCallback(),
        ],
    ),

to the above (more info).

Can be done in external steering file, no need to bake it in at this point


Use public_send

Hi @atipugin, all checks are passed locally... Strange Rubocop fail.
Could you please try to restart the build?


Support intellisense for property/array access on array literals

Issue
In this code snippet - from Alex Frankel's demo - intellicode completion doesn't work when typing in site.name or site.tag in the vscode extension.
Expected behaviour
When you type 'site.' , would be great for intellicode/complete to suggest the name and tag variables.
ps. Bicep rocks!
var websites = [
  {
    name: 'fancy'
    tag: 'latest'
  }
  {
    name: 'plain'
    tag: 'plain-text'
  }
]

module siteDeploy 'site.bicep' = [for site in websites: {
  name: '${site.name}siteDeploy'
  scope: rg
  params: {
    appPlanId: appPlanDeploy.outputs.planId
    namePrefix: **_site.name_**
    dockerImage: 'nginxdemos/hello'
    dockerImageTag: **_site.tag_**
  }
}]

@majastrz - I dont' think we have an issue open for this yet, right?

No I was going to open one at some point, but this one works 😊.

Updated the title to be more descriptive.

Currently, the item type on array literals is any if the members are of different types. In the current type system implementation, non-primitive types are created on-demand without any reuse - a new .net object every time, basically. This leads to an explosion of types and will also cause array item types to look like object | object | object even for identical types. Such types could arise today via array literals or in the future via array manipulation functions once we have function return type transforms implemented.
The best bet here is to follow the TypeScript example and implement type interning before implementing the fix for this issue. This means moving all functionality responsible for creation of types in the type system to a class that is responsible for construction and caching of every type. If a type is requested, the cached version of it will be returned. (This will also give us some memory optimization, but we aren't having memory pressure issues yet, so it's not the primary motivation here.)
The cache key will be the type kind plus argument values that are needed to construct it. The standard .net hashcode gives us a space of billions of types, which should be sufficient for our purposes.
Once this prerequisite is implemented, we can move on to the actual fix for this issue.


Adding methods to locator.js -  withAttrEndsWith, withAttrStartsWith, withAttrContains

Motivation/Description of the PR
Adding methods to locator.js -  withAttrEndsWith, withAttrStartsWith, withAttrContains
I believe those methods are needed for many CodeceptJS users
All methods are compartible with XPath 1.0,
All methods tested - working as expected.
Applicable helpers:

[ ] Playwright
[ ] Puppeteer
[ ] WebDriver
[ ] REST
[ ] FileHelper
[ ] Appium
[ ] TestCafe

Applicable plugins:

[ ] allure
[ ] autoDelay
[ ] autoLogin
[ ] customLocator
[ ] pauseOnFail
[ ] coverage
[ ] retryFailedStep
[ ] screenshotOnFail
[ ] selenoid
[ ] stepByStepReport
[ ] stepTimeout
[ ] wdio
[ ] subtitles

Type of change

[ ] :fire: Breaking changes
[ ] :rocket: New functionality
[ ] :bug: Bug fix
[ ] 🧹 Chore
[ ] :clipboard: Documentation changes/updates
[ ] :hotsprings: Hot fix
[ ] :hammer: Markdown files fix - not related to source code
[ ] :nail_care: Polish code

Checklist:

[ ] Tests have been added
[ ] Documentation has been added (Run npm run docs)
[ ] Lint checking (Run npm run lint)
[ ] Local tests are passed (Run npm test)

I agree with @kobenguyent
Please add tests


fix: Mark only specific VertiGIS packages as external

Web can no longer handle requests for @vertigis/arcgis-rest-client

@yonabystedt We will need a similar change in vertigis/vertigis-workflow-sdk. I just got an email about it a few days ago, as it happens. Would you be willing to do that one too?

@yonabystedt We may need a similar change in vertigis/vertigis-workflow-sdk. I'm not entirely sure, though. Someone was trying to create a custom form element that uses @vertigis/react-ui. I think that should be present in VertiGIS Studio Web, so external is probably correct at least for that product.

I'll take a look if you like.


CounterFunction Feb-13

Daily Challenge Issue Template
Challenge Description
Make a button that increases the current count of 0 by 1 every time
and a Reset button that resets the count back to 0

Acceptable Solution Standards
30mins

Estimated Time for Completion

Additional Notes or Resources (Optional)

Submission Guidelines

Assign yourself to this issue
Create a branch for yourself in this issue
Write your code locally
Push up your code to your branch on GitHub
Create a Pull Request to merge your code into main
Get reviewed by the "challenger"
Get merged or update your code as requested, and repeat 4, 5, 6.

Nice challenge!


Add mitigations to HttpsClientCert_GetCertInformation flakiness

For https://github.com/aspnet/IISIntegration/issues/1422.

Deploy w3wp after creating the client certificate.
Use unique name for cert issuer
Log cert store and current cert on test failure.

Deploy w3wp after creating the client certificate.

How is this accomplished?


Fix issue 3765

fix issue #3765
使用stringbuilder替代string提高输出效率

Codecov Report

Merging #3766 (11c4347) into master (14a95cc) will decrease coverage by 0.02%.
The diff coverage is 65.00%.

:exclamation: Current head 11c4347 differs from pull request most recent head 7fa6a8c. Consider uploading reports for the commit 7fa6a8c to get more accurate results

@@             Coverage Diff              @@
##             master    #3766      +/-   ##
============================================
- Coverage     80.31%   80.29%   -0.03%
+ Complexity     7524     7518       -6
============================================
  Files           157      157
  Lines         28096    28105       +9
  Branches       6790     6790
============================================
+ Hits          22565    22566       +1
- Misses         3499     3503       +4
- Partials       2032     2036       +4

Impacted Files
Coverage Δ
Complexity Δ

...libaba/fastjson/serializer/JavaBeanSerializer.java
82.72% <53.33%> (-0.40%)
196.00 <0.00> (ø)

.../com/alibaba/fastjson/serializer/IntegerCodec.java
90.00% <100.00%> (+0.41%)
14.00 <0.00> (ø)

...m/alibaba/fastjson/serializer/SerializeConfig.java
77.96% <0.00%> (-0.42%)
138.00% <0.00%> (-2.00%)

...main/java/com/alibaba/fastjson/util/FieldInfo.java
86.42% <0.00%> (-0.34%)
99.00% <0.00%> (ø%)

...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java
86.89% <0.00%> (-0.17%)
215.00% <0.00%> (-2.00%)

...java/com/alibaba/fastjson/parser/ParserConfig.java
80.46% <0.00%> (-0.15%)
226.00% <0.00%> (-1.00%)

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 14a95cc...7fa6a8c. Read the comment docs.


Missing api for rewriting input trees

I have input like
├── dir_1
│   ├── 01.md
│   ├── 02.md
│   └── 03.md
├── dir_2
│   ├── 01.md
│   ├── 02.md
│   └── 03.md
├── common
│   ├── 00-1.md
│   ├── 00-2.md
│   └── 00-3.md
└── default.template.html

And I want transformer to behave like if content of common was in dir_1 and dir_2 and skip generating common dir is it possible? So far I have found only one fromDirectories method which would merge content of few dirs together but I need more complex rewrite. Is there some api for this?
Also I have one more question here https://stackoverflow.com/questions/62353757/links-to-json-files
Thanks

That's a somewhat funny timing for this question. In the current release, which is just 2 weeks old, this is not possible yet (it kind of is, but only with pretty low-level APIs I won't have the time to walk you through).
In current master there is a new API since about a week that allows you to freely compose the inputs, including your scenario. If you are comfortable with building a snapshot yourself, I can show you an example.

Thank you @jenshalm for answer and cool library. I think easiest for me now would be to copy files to new temporary dir with right structure and generate it from there. Can you pls look at my question on SO too https://stackoverflow.com/questions/62353757/links-to-json-files?

The other issue turned out to be a bug that I just fixed, see #148 for details.


fix: libraries security vulnerabilities

Fixes #411

most of this is already covered by #444. We will update the rest of the dependencies as soon as they are ready.


[util] Update syn_yosys.sh for latest RTL and latest sv2v version

This includes a few hacks (before and after running sv2v tool). With these hacks, LEC passes for all submodules of top_earlgrey.

Hi Nils,
Thank you for updating the script. Actually I think Zach just made some suggestions through his PR: https://github.com/lowRISC/opentitan/pull/2465. If you have time, you can take a look as well.
Also could you please sign off this commit using git commit --amend -s; git push -f. This means you agree with the CLA policy as outlined in https://github.com/lowRISC/opentitan/blob/master/CLA. This is a new requirement we added in this repo. After that, the CI check should be green.
Thanks,
Cindy

@NilsGraf Are you doing LEC for the gated netlist (<- yosys <- sv2v <- design.sv) and the original design.sv?

@NilsGraf It has been brought to my attention that some files now depend on prim_util.svh. We should probably add a copy for this and any other headers so they can be found.

@NilsGraf Are you doing LEC for the verilog gated netlist (<- yosys <- sv2v <- design.sv) and the original design.sv?

This is only doing LEC between original SystemVerilog and the Verilog generated by sv2v. Eventually we should also do LEC between generated Verilog and the netlist generated by Yosys (or run LEC end-to-end if the runtime is not bad). Right now the LEC runtime for top_earlgrey is prohibitive.

@NilsGraf It has been brought to my attention that some files now depend on prim_util.svh. We should probably add a copy for this and any other headers so they can be found.

Yes, good point. I'm right now working on a new version of this script which also copies over all .svh files (in addition to all .sv files).


No room is available to display rows

Hi  there , just encounter this strange error while comparing target with source in Tabular model using BISM normaliser. Do any one knows the reasons why this pops up?

I've been trying to reproduce this one on my development environment for a long time! It has been reported by a handful of people. They have sent me the project and BISM files and I was still unable to reproduce it. I even went as far as installing the whole development environment on one of their machines and when I ran it in debug mode, it didn't repro :(.
To unblock you, perhaps try one of the following options?

See if there is a VS update or something you can apply. Others told me that it went away for some reason.
Use the ALM Toolkit instead as a workaround (till hopefully this goes away as it has for others). You should be able to download the installer from here. If you use ALM Toolkit, you can connect to your local SSDT AS instance by selecting it in the "Power BI Desktop" dropdown in the Connections dialog.

I faced this situation just a couple of days ago. The BISM Normalizer  extension v<IP_ADDRESS> was working fine on my SSDT 2017 15.9.23 installation, then suddenly it started happening. I tried various things, including removing the latest tables i added tot he model, unistalling/reinstalling the extension.
Solution:  I uninstalled and re-installed SSDT 2017 15.9.23. The issue was cleared.


feature: add 'root' option in router

I've recently deployed a litle project to Github pages with choo. It works fine locally, but there is some edge case when I deploy it that I want to cover. The thing is that the repo is deployed as a project gh-page, so the url look like this
https://yerkopalma.github.io/tic-tac-toe/

I have defined a /game route, and the programatic navigation works, but the url is rewrite as
https://yerkopalma.github.io/game

That is because the navigation changes the href option with the /game route. The problem is that

The expected route for the game is https://yerkopalma.github.io/tic-tac-toe/game
When I type the route https://yerkopalma.github.io/game it gets a 404 error, the same happens with the route above.

What seems logical to me, is to have a rootoption in the router so I can make
const root = 'whatever'

// not actually sure how the signature should be
app.router.root = root

app.router(route => [
  route('/', mainView),
  route('/game', gameView)
])

// resulting routes
// /tic-tac-toe/
// /tic-tac-toe/game

Also (and maybe more important), I guess is expected that, choo routes be relative to the path where the app is mounted, I mean, if I deploy my app to /some/cool/path/ every route should be relative to that path and not to the / route.
I hope I make myself clear. I'm not actually sure if this issue belongs here or in the sheet router

Have you tried changing your links to use ./? Ie. <a href="./game">
I think there's also a <meta> tag out there to set the base URL for links.

Ah yeah, this is a valid concern. I think this should indeed best be discussed in the sheet-router repo.
That said @timwis is right, there's a <base> tag that seems to do what you're describing. Could you verify it works? We could then document it in sheet-router and in turn in choo-handbook. Thanks! :grin:

@timwis I have no anchor tags, just some buttons that run some javascript functions, and those programmatically redirect the app. @yoshuawuyts that would cover the root option, but what happens if I'm not sure where my app is going to be mounted? or if it is mounted in different routes? I guess that is why most of front end frameworks use the hash or hashbang to mount their apps, and start routing from that route, so they are independent from the rest of the routes.
The issue here is that programmatic redirect described here rewrites the whole url, even if my root app wasn't mounted on the / route. I guess that any solution to this issue is higly related to this PR, so I'll wait for now :+1:

I'm moving this issue to sheet-router repo. this is definitly not related with choo itself. Besides, I realized that my issue is not a root issue, the problem is that I can't write in the browser the url

open issue here: https://github.com/yoshuawuyts/sheet-router/issues/36


Regional news: Pricing

Create an Order when sending out regional news

The price is 0.01 EUR per user they'll possibly reach

Let them pay immediately using credit card

You can base yourself on the existing (hidden) news sponsoring code (showNewsOrderPage etc) for creating and paying the order in the news flow. Maybe display this function while testing to play a little bit with it first. Sponsored news was free for the first 5 times, and the price based on the reach of the last 5 news items. However, for regional news the pricing is way more simple. Just 0.01 EUR per user.

Till now, we kept the old promoted news related code, should we just remove it after implementing the regional news pricing?
Another question, how to enable the promoted news it in testing?


[Feature Enhancement][UI] Workflow page: Enable date column in data table

Request:
Enhance data table to display date workflow is created to improve user experience. This includes creating a workflow from scratch or from a template.
Links:
Backend feature enhancement issue: Azure/logicapps#1142
Enhancement:

Add column for date created

Column name: Date

Enable sorting

Default data table to sort newest to oldest to allow the most recent workflows to display at the top

Screenshots:

I'm not sure we have a way of getting this information....

I'm not sure we have a way of getting this information....

Bonica has filed this issue in backend if they come back not possible then this can be closed.

I'm not sure we have a way of getting this information....

Bonica has filed this issue in backend if they come back not possible then this can be closed.

I don't think the backend can do anything with this either.....a workflow is just a file int he file system and could be deployed not just through ARM but through CICD, the file system can't/won't keep track of when it was created, especially if they're using something like run from ZIP where the entire file system changes every deployment.


Url export preference lost

BBT loses the preference concerning the export of the URL (that I set to "export in URL field") each time that I restart Firefox: this is set back to the default value "none".

I can't reproduce it here. Can you get me a debug report?

Here are two debug reports:

before enabling the url option (but after a restart, with the option enabled before): DW4E9WR7
just after enabling the option (during the same Firefox instance as the previous report): BENI6XTE
I hope this will help. Thanks a lot for your time, and if you need any additional information please let me know.

Could you temporarily uninstall Zotfile and try again? The only error I see in the log seems triggered by BBT, but caused by something Zotfile adds. If you could confirm, I can talk to the Zotfile author.

I disabled Zotfile and the bug did not happen. But what is even stranger is that since I have enabled it again I cannot reproduce again the bug despite many tests. Here is a report if you want to check if the error you were mentioning has disappeared: 7W3TPGEC.

The error is still in the logs. Zotfile has some code that is started after a delay, and that's where the error crops up.
I'm starting to think BBT doesn't actually cause the error, it's just a race condition -- if BBT finishes its startup before the error in Zotfile happens, the error does no harm, but if the Zotfile code runs first, BBT's initialization is interrupted.
I've filed an issue with the Zotfile author -- lets keep this issue open until we get some kind of resolution, but if my assessment is correct, there's nothing BBT can do about it.

Thank you! If I see anything new I will let you know.

We're tracking the issue here;   can you install 4.2.6 prerelease and generate a new error report?

From now on do you prefer that I answer here or in the other place?
Here is the report with 4.2.6 installed: CBJBIX6K
Also note that when I looked at the option after installing the new version it was turned off again.

The error from the logs is gone, so that's good news.
 That means it's in BBT, not the interaction between BBT and Zotfile.
Can you do the following:

Enable the "in an URL field" option
Go into "about:config", and verify the "extensions.zotero.translators.better-bibtex.bibtexURLs" option is set to "url"
Stop Firefox (fully, no windows open)
Start Firefox again
Go into "about:config", and verify the "extensions.zotero.translators.better-bibtex.bibtexURLs" option is still set to "url"
Submit a new error report.

There are no errors in your last log, and I really don't know of anything that could actually toggle the preference, rather than preventing it being saved, but let's see what we learn from this procedure.

I tried several times the steps you are indicating, but the preference was always removed so point 5) is failing. Do you want nonetheless a report?
On the other hand I performed another check: when I quit Firefox after changing the option, the new value "extensions.zotero.translators.better-bibtex.bibtexURLs" is correctly saved to the prefs.js file. But when I launch Firefox again this line is deleted from the file.

Yes, please do get me a report, after step 3 and after step 5.
No idea what could be causing this.

Ok I have made three reports:

opening without having changed the option in the previous session (setting: off): FQ5EEUZW
after changing the setting (setting: url): TBKAH7I3
closing the previous session and opening again (setting: off): MVBBV4MI

Absolutely nothing in the logs, except in TBKAH7I3, where I can see you have set the preference. Crazy. So it seems the preference reverts after startup, but there's nothing in the log that indicates any prefs activity -- and when debugging is on, I log all changes to the BBT prefs.
Could you try disabling all extensions except Zotero and BBT, restart, and try the 5-step test again? It's hard to imagine that another extension could be doing this, but I have no other conceptual path that leads to this pref reverting, and I cannot reproduce it here.

I have disabled all the extensions but Zotero and BBT. The problem is still here. Here are three new reports, same process:

first session: 8WGDJSKS
after turning on: GM2UHDJV
after restarting: PTHX4Z4E

Still nothing in the logs.
Could you try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.45-circle-2233.xpi, then go into about:config add a preference called extensions.zotero.translators.better-bibtex.bibtexURLs2, set it to url, and go through the tests again? I've temporarily added this pref and if present it will take precedence; I want to know whether all prefs are reset, or just that one particular preference.
Crazy. I have no idea what's going on.

So the new preference stays after restarting. In particular one time I had the file opened when Firefox started, and I could see that precisely the line for the url disappeared and I would say this was the only one since the line number of the line displayed just shifted by one.
 Can you try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.45-circle-2241.xpi ?

Unfortunately the problem is still here.

Can you try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.45-circle-2251.xpi and submit a debug report?

Here are the three reports, same order:

start: 263EQCS8
set preference: DN2VJNCH
restart: 6IICUHEK

OK, I've neutered some legacy code that must be the cause of this, try https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.45-circle-2260.xpi

I'm ready to cut a new release if this does indeed fix it.

Ok so I confirm that it is fixed. I was making some checks because first it looked fixed, but then later it reappeared: in fact Firefox rolled the BBT version back to a previous one.
Thanks a lot for looking to this and giving so much efforts to get it fixed!

So to make sure: issue fixed, can be closed?

Yes everything is good.

Urgh -- I just made an error in merging. I am positive I corrected it properly, but could you do me a favor and check https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.45-circle-2283.xpi for me? It works for me, but I'd like to make sure. Super thanks for your patience BTW.
Odd that it should have reverted -- the builds are technically versioned "later" than the last release.

I have tested several times and this is fixed.
You should not apologize, you are the one making all the work in writing this wonderful tool and fixing bugs!


Shopware generates 5Gbit/s MySQL traffic

PHP Version: 8.0-8.4   | Shopware Version: 6.4-6.6  | Affected area / extension: Platform(Default)

Actual behaviour:
At the moment, Shopware generates a lot of MySQL traffic. In some setups, I've observed 5Gbit/s continuously. Even Shops that I'd call mid size are showing traffic patterns like that.
In most infrastructures (esp. clouds), network speeds are still limited to 1Gbit/s with mostly no cost efficient / simple way to scale that higher. While modern hardware can scale (far) beyond that, that amount of traffic still generates headaches during the migration of VMs, or when using distributed setups. Even when using 25Gbit/s network interfaces, this still limits the amount of traffic to 5 shops, not including external and other traffic.

Image is from a mysql server of a particularly "communicative" shopware 6 system.
Expected behaviour:
It would be great if MySQL procotol compression could optionally be enabled on systems where this is a problem. While compression needs more CPU time, in my testing this has not resulted in a measurable slowdown or a significant increase in CPU usage (maybe 5%-10%). However, using MySQL protocol compression has reduced the database bandwitdth from 1.6Gbit/s to 0.075Gbit/s. This has repercussions for latency and other CPU usage, because moving multiple Gbit/s of data places more load on components such as PCIe interconnects, memory controllers, kernels, ..., doubly (or more) when using TLS encryption, which is becoming a requirement for many customers.

Image is a test, where I enabled mysql protocol compression for a short while.
How to reproduce:
I am not sure how to reproduce this issue in vanilla shops, but I do not think that it matters much. From my experience hosting shopware systems, I can tell you that this is a problem that is limiting scalability. Even if shopware plugins are responsible, this is still an issue that can be reduced by enabling mysql protocol compression.
This is how MySQL protocol compression can be enabled:
        // in vendor/shopware/core/Framework/Adapter/Database/MySQLFactory.php
        if (EnvironmentHelper::getVariable('DATABASE_PROTOCOL_COMPRESSION', false)) {
            $parameters['driverOptions'][\PDO::MYSQL_ATTR_COMPRESS] = true;
        }

Then set DATABASE_PROTOCOL_COMPRESSION=true in environment or .env file.

Is that maybe because of carts? I will add that env tomorrow, but I would tbh not recommend it as of performance reasons

do you have metrics btw which queries were most executed at that peak times?

This is not related to carts, most likely this is caused by worker or API requests. Right now, I can't give you any specific information about the queries, because my example from above stopped whatever it was doing.
Replication is not running for my example.
I suggest that we open up additional tickets about queries from sw core that we see returning "too much" data as we encounter them, and keep this ticket here for adding support for protocol compression. The problem that we face is that even if this traffic is caused by (inefficient) plugins or indexers, we cannot directly force shopware to use compression, so we'd have to either use an intermediary like ProxySQL or patch php to use compression by default. While our infra can handle scenarios like this, we'd gain more options with such a feature, especially when onboarding preexisting shops with bad plugins. Other operators should profit from this aswell, especially when having mysql traffic cross "availability zones" or public internet.

We've created a "worst case" benchmark scenario where we've run your rw benchmark https://github.com/shopware/k6-shopware with 5 users for 60 minutes against hetzner cloud non-dedicated VMs (2CPU Appserver, 2CPU MySQL), with the bottleneck being the app servers CPU. The idea was that this should expose the worst case performance penalty when enabling compression on a system not constrained by network bandwidth, which is also using slow CPUs. Here are the results:
with compression:
     response_time_accountRegister.....: avg=1784.571802 min=1143    med=1576     max=9638     p(90)=2250     p(95)=2632.75
     response_time_addProductToCart....: avg=1451.756745 min=869     med=1341     max=8521     p(90)=1849     p(95)=2074.3
     response_time_CartInfo............: avg=500.735857  min=7       med=515      max=7255     p(90)=746      p(95)=862.25
     response_time_CartPage............: avg=949.057441  min=574     med=858      max=8291     p(90)=1180     p(95)=1323
     response_time_ConfirmPage.........: avg=933.656658  min=557     med=872      max=3839     p(90)=1188.5   p(95)=1306.75
     response_time_NavigationPage......: avg=826.064186  min=86      med=795      max=7893     p(90)=1187     p(95)=1377.75
     response_time_placeOrder..........: avg=1965.601828 min=1047    med=1819.5   max=8898     p(90)=2432.5   p(95)=2694.75
     response_time_ProductDetailPage...: avg=925.852045  min=445     med=858      max=8093     p(90)=1186     p(95)=1356.15
     response_time_SearchPage..........: avg=892.092254  min=88      med=930      max=8249     p(90)=1447.3   p(95)=1623.3
     response_time_StoreFront..........: avg=175.073107  min=161     med=171      max=341      p(90)=178      p(95)=182

without compression:
     response_time_accountRegister.....: avg=1878.522111 min=1199    med=1686     max=8561    p(90)=2363     p(95)=2704
     response_time_addProductToCart....: avg=1601.957204 min=952     med=1459     max=8792    p(90)=2078.8   p(95)=2379.9
     response_time_CartInfo............: avg=562.303138  min=7       med=555      max=5910    p(90)=836      p(95)=1023.75
     response_time_CartPage............: avg=1024.991441 min=594     med=927      max=5201    p(90)=1304     p(95)=1522
     response_time_ConfirmPage.........: avg=1009.71184  min=515     med=925      max=5540    p(90)=1266     p(95)=1444
     response_time_NavigationPage......: avg=912.784593  min=87      med=861      max=8481    p(90)=1336     p(95)=1574.75
     response_time_placeOrder..........: avg=2138.388017 min=1148    med=1944     max=8649    p(90)=2678     p(95)=3076
     response_time_ProductDetailPage...: avg=1008.074655 min=488     med=924      max=8088    p(90)=1304     p(95)=1514.7
     response_time_SearchPage..........: avg=944.336186  min=93      med=992      max=7605    p(90)=1537.8   p(95)=1737.9
     response_time_StoreFront..........: avg=176.289586  min=162     med=172      max=329     p(90)=179      p(95)=182

The test without compression is actually slower than the one with compression enabled. I think that this is the result of background cloud utilization (with all metrics increasing by 120-150ms during later hours today), as the system hadn't had anything else running. As both we and Scale Commerce use CPUs much faster than Hetzners Cloud for production services, I think that the impact is negligible in normal circumstances. I believe that the feature should be disabled by default, with a clear "your mileage may vary" warning attached.
@shyim The most ideal result for us would be to have this feature in main and then backported to 6.6 and 6.5, if that's possible.


Erikbern/more coverage

Just a bunch of trivial commits of things where it should be easy to get test coverage up

Somewhat relevant:

@erikbern coveralls is at more than 86% :D. However, I need to say that this number is a little skew. There is some module that are never imported which mean there are not taken into account when computing the coverage. Here is a small list that I see:

mysqldb # should be easy to test in travis since mysql is already install
ftp # we could whether mock the server or use https://github.com/giampaolo/pyftpdlib
webhdfs # not sure if we can easily activate the http interface of hadoop. Otherwise, we can easily mock the http response
scadling # no idea how easy
sparkey # no idea there too

Maybe we could add some "dummy" import in the helpers just to make sure to capture those.
@functools.total_ordering interesting... I'm gonna remember.

That's a good point. If we add tests for those files it will probably drag down the average at first.
Those things you mentioned are great. I don't think we need to be super strict about tests for existing code in luigi.contrib, but going forward let's demand reasonable unit test coverage for all new code in luigi.contrib
For things in "core" Luigi I think we should strive for 95% coverage at least... shouldn't be too hard.

87% coverage now :)
Here's the updated breakdown:

I think we can do some simple stuff to luigi.hdfs and luigi.hadoop to bump the coverage...but I'll do that some other time


Flatten Selection To Bitmap

Is there an api do something like “Flatten Selection To Bitmap” ？

Thank you for planning to add the api as enhancement 。
Could you give some suggestions of how to implement it， I really need it。

@zjuwjf
You can try this:
const layers = document.selectedLayers.layers
const layer = layers[0]

const flattener = MSLayerFlattener.alloc().init();
flattener.flattenLayers(MSLayerArray.arrayWithLayers([layer.sketchObject]));

It works， so cool！ @nichenqin

Thinking a bit about it, I think we want a top-level method flatten similar to export.
It would take either a single layer or an array of layers and return an Image

Can it return an Image? The flatten behavior looks not synchronous


Remove version: inLuShan.Desktop version 0.1.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://inlushan.top/res/desktopApp/inlushanSetup_0_1_0.exe
Response Error: The remote server returned an error: (522).
(Automated message - build 923)

Hi @Exorcism0666,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 897.)


Sync eng/common directory with azure-sdk-tools for PR 1061

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/1061 See eng/common workflow

/check-enforcer override


Fix types

Just fix types declaration for ln function -  the count argument is optional
JSDoc's types are not necesary while using TS

Thanks @FRO1D! Merged


[platform] Update common files for branch master

Update common files

@melix I think this can be merged, but only admins can do it

Done. I thought you were an admin?

Done. I thought you were an admin?

Nope, only on a few repos


Bah 2104

List of features added

login with credentials
set the session cookies
upload registration & encounters
Test cases to verify the status of upload

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


fix: fixed repository not beeing template conform

just some light work for hacktoberfest ;)

I'll do the change myself. Thank you for the new screenshots and the fixed readme.


Image dimension fix

ClickUp task link

https://app.clickup.com/t/863ga10yh
Changes

Added fix for image dimension

Key points to review

Get image dimension again after host size dimension has changed

Screenshots and/or videos

Issue

Fix

Closing in favor of the fix in 8f13cf34a7cd294547afb6a6145bec42d1cec332. Thanks for pointing out the problem @ponnexcodev!


Website: Main Image 404

Main image leads to: https://raw.githubusercontent.com/AminaG/OpenScreenshot/master/src/images/icon256.png
Which is a 404

Fixed. Thanks


Fix validators

Adds tests for Validator.js and also fixes some validations.

@pb82 It fails because it's passing a String, not an actual number. Do we want to include strings as valid numbers?
https://github.com/aerogear/data-sync-ui/pull/91/files#diff-fbcf5225cb21f2fbdb020c5cc734f1ddR52

@pb82 something's wrong with the new validation "details" mode -> https://github.com/aerogear/data-sync-ui/blob/a0eb1f37ff8422d21ac428abedea75e04d50f211/test/ui/helper/Validators.test.js#L28-L38
And port is always "success" in the Postgres version of AddDataSourceDialog.


Breakpoint not hit in extension debugging

This looks like a regression from 1.0 and might be related to the electron update:

clone https://github.com/bpasero/vscode-markdown.git
npm install
open with VS Code
add a breakpoint both into where the extension.previewMarkdown command does something as well as the virtual document provider to the top (console.log("hit");)
run the extension
open a file in the new window popping up
invoke the command (Markdown Preview)
the first breakpoint should get hit but the second is not for me
this works fine with VS Code 1.0
if you remove the breakpoint you will notice that hit is logged to the console so you can see the line is actually hit

@isidorn @weinand fyi

I have looked into this and the simptoms are the same as for https://github.com/Microsoft/vscode/issues/5348
So node debug gets the same error from node.
This only happens in call back functions for some reasons - Martin also noticed that

Verified it works now


Should we add an item in roadmap for network usage for OCI?

We have already reached a consensus that network setup is out-of-scope for opencontainers/specs[1], however, I think a detailed guide for user to involve networking with OCI/runC is necessary[2]. Any plan for it? Should we add an item in roadmap?
[1] https://groups.google.com/a/opencontainers.org/forum/#!topic/dev/3XqoyLqtA6I
[2] https://groups.google.com/a/opencontainers.org/forum/#!topic/dev/zA-E_x5Wuq4

I don't think OCI needs to cover anything about networking. I think the concerns of a container runtime and container networking are orthogonal and can be solved separately via lifecycle hooks. See for example Container Networking Interface (CNI).
I am going to close this for now. If another maintainer disagrees we can reopen.


[fix](inverted index) Error parsing NULL_LITERAL

Proposed changes

NULL_LITERAL and other LITERAL will enter the else process incorrectly
SELECT * FROM table_name WHERE column_name NOT IN (1, 2, NULL); The NOT IN operator returns an empty result set when the collection contains NULL.

run buildall


fix(csv): fixes csv output method

output was being saved to csv using the tabs dialect even when --tabs flag
wasn't specified. this patch fixes that issue.
Initially #967 was merged which caused the #976 issue.
To fix the issue #998 was merged but it only partially solved the issue.
Default method for saving CSV was changed to tabs instead of comma / , and there was no way to change that.
This PR solves the issue stated above.

https://github.com/twintproject/twint/pull/1007/commits/2330878f127e05e9614b66249babdc1c1e7af354 fixes a small bug which could lead to 0 being written in the quote_url field of a csv. which would lead to error in certain rows while opening csv in excel.


:sparkles: implement HTTP 1.1

still missing HTTP/1.1 features (taken from https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/Evolution_of_HTTP):

chunked responses
chunked requests
content negotiation
cache-control

chunked responses
chunked requests
content negotiation
cache-control


Fix: raise explicit memory error when dataset allocation fails

So that adding rows does not silently fail.

I'm worried this may cause some significant behaviour changes in existing code that uses this, since it's possible in some cases that users may have code they expect to run even after a failed allocation. (e.g. cleanup, side effects, etc) Have you considered this?

After slack discussion, this is an acceptable risk


[OnlineMeeting&July.25]SeaTunnel community meeting Topic collect

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
EricJoy2048 invites you to a meeting on Tencent Meeting
Meeting Topic: SeaTunnel Community Meeting
Time: 2023/07/25 20:00-21:00 (GMT+08:00) China Standard Time - Beijing
Click the link to join the meeting or to add it to your meeting list:
https://meeting.tencent.com/dm/yisLUSgpYoEl
#TencentMeeting: 432-577-685
Copy this invitation and open in Tencent Meeting app (V2.13 or later) to join the meeting
Everyone is very welcome to join:
If you have an issue, please submit it one day before the meeting, and provide a specific issue link and context.

https://github.com/apache/seatunnel/pull/5112


Cannot convert undefined or null to object

Issue Id: a876e3bc-590d-7979-433c-32d5fa6b1c27Versions - 1.9.0Stack TypeError: Cannot convert undefined or null to object/vs/platform/configuration/common/model.ts#L113:16 (keys)/vs/workbench/services/configuration/common/model.ts#L82:36 (keys)    at Array.forEach (native)/vs/workbench/services/configuration/common/model.ts#L80:52 (forEach)    at Array.forEach (native)/vs/workbench/services/configuration/common/model.ts#L79:21 (forEach)/vs/workbench/services/configuration/node/configurationService.ts#L178:41 (keys)/vs/workbench/services/configuration/node/configurationService.ts#L184:22 (keys)/vs/workbench/api/node/mainThreadConfiguration.ts#L29:58 (values)/vs/base/common/callbackList.ts#L63:26 (apply)

1400 occurrences since 1.9.

Duplicate of #19773


google play support libraries update..

after the latest version (0.2.7),
I'm (still) getting updating google play support libraries,
I thought this was because our gms core version was older than the latest google play services,
now it should be the same, so why is this (still) happening?
I also think that it's happening more than before as well
do you know why and/or can you fix it?

It's not an microG issue.
And there is nothing that needs to be fixed

well, I've only noticed it in this device that has microg installed,
I've never seen it before
after all, google play support libraries = google play services = gms core, no?

Microg can't prevent the question for update. As soon as the service framework gets a new version you will get the request. Only solution is to use a store in which you can put applications to an ignore list (e.g. Aurora store).
However, currently the issue is valid, as the official version and microg version should not differ. Hence, if there is an update request something seems to be wrong?
@patrickdrd Do you have any Beta release channel activated? To which version does it want to update?

I don't think so, I never opt in for betas,
I don't have the time or the mood to do the beta testing, I always ignore them,
as for the version that's something I'm wondering too myself,
I don't know, I tried to check the logcat but I guess I was just a bit late,
because I couldn't find it,
I'll keep an eye on next time I see it,
it was happening before too, but happening this time which equal version is unjustifiable

is it possible that it's trying to update google play store?
we haven't received the latest play store yet (the material one) by @Nanolx ,
maybe that's the culprit?

The last version by me is 15.1.23, while the latest official is 15.1.24. I may roll-out that minor today, though I doubt that's the culprint. The patched Play Store cuts off several update requests, but not all, since some are triggered on server side.
I personally, don't get any update triggers since I using the combo of Play Store 15.1.23 and microG with version 14.7.55 spoofed. So I guess the material design update or Play Services beta channel (as pointed out by @chris42) may be the reason.

classic google, another update today, 15.2.23 ..

@ale5000-git could you please close this issue? Thanks.
@patrickdrd please move future issues regarding the custom Play Store over at: https://gitlab.com/Nanolx/microg-phonesky-iap-support
I'll roll-out a patched version 15.2.23 during the next hour.

Closed as the problem isn't in microG.

Oh, updating to 15.1.24 immediately got me the Material Design update

so google got back the material design and
gave it again today with version 15.2.38,
waiting for your update

so material design rolled back again for the 2nd time!! google seems undecided!


Model classes

Thanks for such a great library. I want to use it in a project, but I am new to SAP Business One and I am not sure what would I need to do or research in order to create the SAP B1 model classes. What would you recommend? Thanks!

Hi, @aeadbhard.
SAP Business One model classes to be used with B1SLayer can be ordinary C# classes, each representing an entity in SAP and containing its properties. The name and type of each property should match what is defined in the Service Layer metadata.
It's worth noting that your C# model classes don't need to have all the properties of a given SAP entity, as they can have an extensive list of properties and in most cases you will only need a few, which will vary depending on your need.
Here's an example of a model class for the BusinessPartner entity:
public class BusinessPartner
{
    public string CardCode { get; set; }
    public string CardName { get; set; }
    public string CardType { get; set; }
    public string EmailAddress { get; set; }
    public int GroupCode { get; set; }
    // (...) just add all the properties you need, making sure the name and type matches the metadata
}

Thank you, Bruno, for your kind help. Will follow your advice. Thanks!

From: Bruno Mulinari @.>
Sent: 07 November 2022 11:30
To: bgmulinari/B1SLayer @.>
Cc: aeadbhard @.>; Mention @.>
Subject: Re: [bgmulinari/B1SLayer] Model classes (Issue #26)
Hi, @aeadbhardhttps://github.com/aeadbhard.
SAP Business One model classes to be used with B1SLayer can be ordinary C# classes, each representing an entity in SAP and containing its properties. The name and type of each property should match what is defined in the Service Layer metadatahttps://help.sap.com/doc/6ab840ef2e8140f3af9eeb7d8fef9f06/10.0/en-US/Working_with_SAP_Business_One_Service_Layer.pdf#page=20.
It's worth noting that your C# model classes don't need to have all the properties of a given SAP entity, as they can have an extensive list of properties and in most cases you will only need a few, which will vary depending on your need.
Here's an example of a model class for the BusinessPartner entity:
public class BusinessPartner
{
public string CardCode { get; set; }
public string CardName { get; set; }
public string CardType { get; set; }
public string EmailAddress { get; set; }
public int GroupCode { get; set; }
// (...) just add all the properties you need, making sure the name and type matches the metadata
}
—
Reply to this email directly, view it on GitHubhttps://github.com/bgmulinari/B1SLayer/issues/26#issuecomment-1305952091, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM6HGSGVK6PDHFDGJWS7BKTWHE4D5ANCNFSM6AAAAAARZMLV4I.
You are receiving this because you were mentioned.Message ID: @.***>


add option to configure aardvark-dns path

Signed-off-by: Paul Holzinger<EMAIL_ADDRESS>

LGTM

/lgtm


Intercom API Error: The Intercom API request failed with the code: 401, after 3 attempts.

Hi,
When trying to run rake intercom:import I get the following result:
Intercom API Error: The Intercom API request failed with the code: 401, after 3 attempts.

Any guess why this is happening ?

401 would imply that your API keys are incorrect. Would you mind getting in touch through the Intercom widget at https://app.intercom.io as it will be easier to take a look?


Showing the search menu causes horizontal overflow on desktop

Issue
Showing the search menu causes horizontal overflow on desktop (> 768px)
The screenshot below shows a visible horizontal scroll bar in Firefox when the site search menu is open:

The red outline in the screenshot was added to using dev tools to show the element that is likely causing the issue, id="super-search-menu"
How to reproduce the issue

Using any browser, visit any page on GOV.UK
Ensure viewport width is larger than 768px
Open the site search menu
The horizontal scrollbar will then be shown in Safari and Firefox. The horizontal scrollbar is added to Chrome, but you may need to use the left/right arrow keys to scroll horizontally, it was hidden for me.

Will be solved by this PR https://github.com/alphagov/govuk_publishing_components/pull/3201

I never closed this issue, sorry about that. This has been solved by https://github.com/alphagov/govuk_publishing_components/pull/3201.