issue #285 - Cast value passed to preg_match to a string

I will test your feature branch tomorrow at work, using a large WSDL of a SOAP service.
I'll leave a comment here if the test was successful or not.

@mikaelcom I tested your branch and  have noticed the following behavior.
With Version 4.1.7:
/**
 * This class stands for ReducedWorkingStateType StructType
 * @subpackage Structs
 */
class ReducedWorkingStateType extends AbstractStructBase
{
    /**
     * The Success
     * Meta information extracted from the WSDL
     * - documentation: Successo Notifica elaborata con successo | Succès Le traitement a terminé avec succès | Erfolgreich Verarbeitung war erfolgreich
     * - choice: Success
     * - choiceMaxOccurs: 1
     * - choiceMinOccurs: 1
     * @var \Swissdec\Transmitter\PhpSdk\StructType\SuccessResponseType|null
     */
    protected ?\Swissdec\Transmitter\PhpSdk\StructType\SuccessResponseType $Success = null;
    /**
     * Constructor method for ReducedWorkingStateType
     * @uses ReducedWorkingStateType::setSuccess()
     * @param \Swissdec\Transmitter\PhpSdk\StructType\SuccessResponseType $success
     */
    public function __construct(?\Swissdec\Transmitter\PhpSdk\StructType\SuccessResponseType $success = null)
    {
        $this
            ->setSuccess($success);
    }
    /**
     * Get Success value
     * @return \Swissdec\Transmitter\PhpSdk\StructType\SuccessResponseType|null
     */
    public function getSuccess(): ?\Swissdec\Transmitter\PhpSdk\StructType\SuccessResponseType
    {
        return $this->Success ?? null;
    }
    /**
     * This method is responsible for validating the value(s) passed to the setSuccess method
     * This method is willingly generated in order to preserve the one-line inline validation within the setSuccess method
     * This has to validate that the property which is being set is the only one among the given choices
     * @param mixed $value
     * @return string A non-empty message if the values does not match the validation rules
     */
    public function validateSuccessForChoiceConstraintFromSetSuccess($value): string
    {
        $message = '';
        if (is_null($value)) {
            return $message;
        }
        $properties = [
        ];
        try {
            foreach ($properties as $property) {
                if (isset($this->{$property})) {
                    throw new InvalidArgumentException(sprintf('The property Success can\'t be set as the property %s is already set. Only one property must be set among these properties: Success, %s.', $property, implode(', ', $properties)), __LINE__);
                }
            }
        } catch (InvalidArgumentException $e) {
            $message = $e->getMessage();
        }

        return $message;
    }
    /**
     * Set Success value
     * This property belongs to a choice that allows only one property to exist. It is
     * therefore removable from the request, consequently if the value assigned to this
     * property is null, the property is removed from this object
     * @throws InvalidArgumentException
     * @param \Swissdec\Transmitter\PhpSdk\StructType\SuccessResponseType $success
     * @return \Swissdec\Transmitter\PhpSdk\StructType\ReducedWorkingStateType
     */
    public function setSuccess(?\Swissdec\Transmitter\PhpSdk\StructType\SuccessResponseType $success = null): self
    {
        // validation for constraint: choice(Success)
        if ('' !== ($successChoiceErrorMessage = self::validateSuccessForChoiceConstraintFromSetSuccess($success))) {
            throw new InvalidArgumentException($successChoiceErrorMessage, __LINE__);
        }
        if (is_null($success) || (is_array($success) && empty($success))) {
            unset($this->Success);
        } else {
            $this->Success = $success;
        }

        return $this;
    }
}

With version dev-feature/issue-285:
/**
 * This class stands for ReducedWorkingStateType StructType
 * Meta information extracted from the WSDL
 * - base: sdc:WorkingStateType
 * @subpackage Structs
 */
class ReducedWorkingStateType extends WorkingStateType
{
}

Is this the expected behaviour?

And I additionally noticed the following:
In case of a boolean value, the generated code seems to be invalid.
The issue here is, that casting of a boolen leads to an int 1/0 and not to a string true/false.

Why is there a pattern on a boolean? Is the pattern containing true and false as valid values to be matched by the pattern? Could you paste the XSD element definition?

The  WorkingStateType class should contain the properties previously contained by the ReducedWorkingStateType class. It's certainly to recent updates.

Ah, okay. I assumed that the feature branch would only solve the one problem.

The  WorkingStateType class should contain the properties previously contained by the ReducedWorkingStateType class. It's certainly to recent updates.

Ah, okay. I assumed that the feature branch would only solve issue #285.

I merged improvements recently :)

Is this the expected behavior?

The  WorkingStateType class should contain the properties previously contained by the ReducedWorkingStateType class. It's certainly to recent updates.

This is the relevant code:
    <xs:simpleType name="SimpleBooleanType">
        <xs:restriction base="xs:boolean">
            <xs:pattern value="true"/>
            <xs:pattern value="false"/>
        </xs:restriction>
    </xs:simpleType>

Which is used by:
    <xs:complexType name="ControlsType">
        <xs:sequence>
            <xs:element name="ProcessByDistributor" type="c:SimpleBooleanType">
            </xs:element>
        </xs:sequence>
    </xs:complexType>

As you have suspected, we have a boolean here, which is restricted by a true/false pattern. I'm not an expert in this field, and therefore don't know if this is common.

And I additionally noticed the following:
In case of a boolean value, the generated code seems to be invalid.
The issue here is, that casting of a boolen leads to an int 1/0 and not to a string true/false.

Why is there a pattern on a boolean? Is the pattern containing true and false as valid values to be matched by the pattern? Could you paste the XSD element definition?

This is the relevant code:
    <xs:simpleType name="SimpleBooleanType">
        <xs:restriction base="xs:boolean">
            <xs:pattern value="true"/>
            <xs:pattern value="false"/>
        </xs:restriction>
    </xs:simpleType>

Which is used by:
    <xs:complexType name="ControlsType">
        <xs:sequence>
            <xs:element name="ProcessByDistributor" type="c:SimpleBooleanType">
            </xs:element>
        </xs:sequence>
    </xs:complexType>

As you have suspected, we have a boolean here, which is restricted by a true/false pattern. I'm not an expert in this field, and therefore don't know if this is common.

@mikaelcom

I'm not an expert in this field, and therefore don't know if this is common.

I'm coming back to this topic.
If we have a boolean restriction like this:
    <xs:simpleType name="SimpleBooleanType">
        <xs:restriction base="xs:boolean">
            <xs:pattern value="true"/>
            <xs:pattern value="false"/>
        </xs:restriction>
    </xs:simpleType>

Couldn't we specify that the code looks more or less like this?
    public function setSimpleBoolean(?bool $someBooleanValue = null): self
    {
        $this->someBooleanValue = $someBooleanValue;

        return $this;
    }

Meaning: In the generated PHP class method if the param is of type boolean no further checks are necessary?

Couldn't we specify that the code looks more or less like this?

It's not implemented this way currently as it's a generic ruler that generates each rule per encountered "meta" tag such as pattern, I'll look into it ASAP

@tbreuss I fixed the pattern rule validation generation in order to ignore boolean values, a little bit empiric but it's easy to implement and to make it evolve if necessary :smile:

@tbreuss I fixed the pattern rule validation generation in order to ignore boolean values, a little bit empiric but it's easy to implement and to make it evolve if necessary 😄

This is good news. I will test your changes against my large WSDL. I'll keep you informed.


Make coverage testing version independent

test_coverage_on fails on latest bazel (3.7.0) see example build https://travis-ci.org/github/bazelbuild/rules_scala/jobs/737941438

#1218


Implements wrangler deploy/dev support for Python.

What this PR solves / how to test:
This PR implements wrangler deploy index.py and wrangler deploy (with a wrangler.toml that defines main = "index.py"). This means Python Workers can be deployed using Wrangler.
Tested manually via:
$ cat $PWD/index.py
from js import Response
def fetch(request):
  return Response.new('Updated Python Worker value')
$ pnpm run --filter wrangler start dev $PWD/index.py --compatibility-flag experimental

Also using tests:
$ cd packages/wrangler
$ pnpm test # added --testNamePattern=\"can run and modify python worker during dev session (local)\" --silent=false in packages.json
$ pnpm test:e2e

I included an e2e test for dev but I couldn't run it locally. Other e2e tests fail locally for me too so I guess I just have something misconfigured. Help appreciated in getting it working.
Author has addressed the following:

Tests

[x] Included
[ ] Not necessary because:

Changeset (Changeset guidelines)

[ ] Included
[ ] Not necessary because:

Associated docs

[ ] Issue(s)/PR(s):
[ ] Not necessary because:

Note for PR author:
We want to celebrate and highlight awesome PR review! If you think this PR received a particularly high-caliber review, please assign it the label highlight pr review so future reviewers can take inspiration and learn from it.

This should probably include a default module rule mapping *.py to python

@penalosa can you elaborate on this? Where do I add it and what effect will it have?

Closing in favour of #4901


Update "Test webextension permissions dialog" section for Firefox Desktop

Add several add-ons to help localizers test all permission strings. Update includes all recent webext permissions.
I'm open to change the text if needed.

Note. In the *-plural-other extensions I use a number of items that may not be suitable for some locales. Overall some more work is needed to cover locales with plural forms more/less than 2. Sorry, cs et. all.


fix bazel files (.bzl BUILD WORKSPACE) format with buildifier

Signed-off-by: wangbaiping<EMAIL_ADDRESS>format only update.

And the format check is only applied to the .h/.cc. We should add the CI to check the format of BUILD/.bzl file.


HTTP/2 transport

Now that SPDY is being set aside for HTTP/2 in chromium, we should consider what an http/2 implementation looks like.

@mcollina brought this up to me earlier. I am for defining the HTTP/2 based protocol as soon as possible.  As part of this transition, I would like to first take the step of separating out the existing spdystream stream implementation to a more abstract stream interface.  This interface would then have implementations for spdystream (should work exactly as today) as well as HTTP/2.  It would also allow us to explore using libchan on top of SSH or customized multiplexed implementations for file sockets and websockets.

Switched to help wanted and into 0.2.1 milestones. I will not be able to own this anytime soon so it might need to be pushed out.  The 0.2.0 version will introduce the stream interface which http/2 will need to implement.  Implementing this feature should not involve deep changes to libchan, only adding a stream provider implementation.

Any interest in grpc or just http/2?

Active go development for http2 https://github.com/bradfitz/http2

The next version of libchan will have the underlying transport abstracted away using this interface https://github.com/dmcgowan/streams. The codec will also be behind an interface to support different codecs in the future. This is mostly to keep the role of libchan smaller, which is to expose a method for sending and receiving messages which can contain channels and byte streams (as any io.ReadWriteCloser). This basic functionality unlocks the potential for doing very powerful RPC on top of it, but the scope of libchan is not to define an RPC layer.  Grpc appears to be a full RPC solution, I don't think it and libchan have any need for each other. It does look like a cool project though and would love a further explanation if you disagree.


Connection timeout

Hi,
i'm trying to utilize your script to download resources from a course that my company paid for me but i'm always getting connection timeout even if I disconnect and reconnect again to update the cauth.
Is it because of the captcha? the script doesn't work anymore?
thank you

It was a my problem with my connection


chore: Update contact form style and button appearance

🛠 Fixes Issue

Closes #836
👨💻 Changes proposed

Enhanced the .Cbtn component with a new gradient background.
Improved font size and padding for better readability and spacing.
Added dynamic hover effects, including a gradient change, shadow enhancement, and transform scaling.
Refined the active state to remove shadow and reset transform.
Indicated the disabled state with reduced opacity.
Applied smoother transition effects for better user interaction.

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

Please review the changes to the .Cbtn component to ensure they align with the overall project theme and provide an improved user experience.
📷 Screenshots
Before

After

@SauravMukherjee44
kindly please check this and merge it.


added login for buyers

Fixes #547
Description
Added an option for signin/signup for the buyers which leads to the signin/signup form for the buyers.
Screenshots
https://github.com/Anushkabh/krishiconnect/assets/142666229/3b4ec4d5-c3a0-48b2-91fa-536805cd03bb

Please review my pull request @himanshu-03 @RitiChandak @Anushkabh

Please review my pull request @Anushkabh @himanshu-03 @RitiChandak

Please review my pull request @himanshu-03 @Anushkabh @RitiChandak

Please review my pull request @himanshu-03 @Anushkabh @RitiChandak


FSharp portable language target is not located in same place as CSharp and VisualBasic

Because of this, when compiling portable, will get error:

MSBuildSdkExtras.Common.targets(118,5): error : The specified TFM 'portable-net45+sl5+win8' is not valid. Check that it's a valid combination: http://portablelibraryprofiles.apps.stephencleary.com/

Actual FSharp location:
<LanguageTargets>$(MSBuildExtensionsPath32)\Microsoft\VisualStudio\v$(VisualStudioVersion)\FSharp\Microsoft.Portable.FSharp.Targets</LanguageTargets>

Example: https://github.com/fsprojects/FSharp.Interop.Dynamic/commits/vs2017-fsproj

@jbtule can you please try the latest off of the CI feed? Should be 1.0.6-beta.* I just fixed this but would like your confirmation before I push it to NuGet.org.

Well it worked on windows!
https://ci.appveyor.com/project/jbtule/fsharp-dynamic-832/build/73
On mac, it's looking for .mdb files all of a sudden.
https://travis-ci.org/fsprojects/FSharp.Interop.Dynamic/builds/255652802?utm_source=github_status&utm_medium=notification

/usr/local/share/dotnet/sdk/1.0.4/Sdks/NuGet.Build.Tasks.Pack/buildCrossTargeting/NuGet.Build.Tasks.Pack.targets(104,5): error MSB4018: The "PackTask" task failed unexpectedly. [/Users/travis/build/fsprojects/FSharp.Interop.Dynamic/FSharp.Interop.Dynamic/FSharp.Interop.Dynamic.fsproj]
/usr/local/share/dotnet/sdk/1.0.4/Sdks/NuGet.Build.Tasks.Pack/buildCrossTargeting/NuGet.Build.Tasks.Pack.targets(104,5): error MSB4018: System.IO.FileNotFoundException: File not found: '/Users/travis/build/fsprojects/FSharp.Interop.Dynamic/FSharp.Interop.Dynamic/obj/Release/portable-net45+sl5+win8/FSharp.Interop.Dynamic.dll.mdb'.

I'm not sure where that error lives... might be worth an issue on the NuGet repo if it's with the Pack targets. I'd split this into two parts:

Building
Packing

Seems like 1 is working, but 2 is with the NuGet team.

Yeah apparently msbuild has a hack on mono that allows the debugging extension to be changed to mdb which they use in a patched into the fsharp.language.targets on mono.
https://github.com/mono/mono/blob/master/packaging/MacSDK/patches/fsharp-fix-mdb-support.patch
However nothing i'm building has is outputting mdbs. So I have a work around for my purposes,  I set the FscDebugFileExt.
<PropertyGroup>
    <FscDebugFileExt>.pdb</FscDebugFileExt>
</PropertyGroup>

Seems appropriate and working.

I just pushed 1.0.6 to NuGet.org, please lmk if you hit any other issues; happy to fix!


[ADVISOR-3090] Display text when no report_json

Currently, we have only one task that displays job results in the expanded row. That content is unique to the specific task and we need to make a failsafe so that if a job is run that doesn't have a report_json that it will fallback to the report property and display the text.
To test, go to this line: https://github.com/RedHatInsights/tasks-frontend/blob/master/src/SmartComponents/CompletedTaskDetails/CompletedTaskDetails.js#L76 and replace it with https://github.com/RedHatInsights/tasks-frontend/blob/master/src/SmartComponents/CompletedTaskDetails/tests/fixtures/completedTasksDetails.fixtures.js#L62.
This will use the fixture to display the scenario we're testing. In the instance where report_json is not provided, we'll rely on the backend to provide text in the report property of the object.
Now, go to Automation Toolkit -> Tasks and click the Activity tab. Select any task, as the mocked data will replace any actual data.
Expand the bottom job and compare it to the other expandable jobs to see the difference.

Codecov Report
Patch coverage: 97.29% and project coverage change: +0.16 :tada:

Comparison is base (5afe043) 85.19% compared to head (60d02c2) 85.36%.

:exclamation: Current head 60d02c2 differs from pull request most recent head db614d7. Consider uploading reports for the commit db614d7 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #88      +/-   ##
==========================================
+ Coverage   85.19%   85.36%   +0.16%
==========================================
  Files          53       54       +1
  Lines        1162     1175      +13
  Branches      370      378       +8
==========================================
+ Hits          990     1003      +13
  Misses        153      153
  Partials       19       19

Impacted Files
Coverage Δ

...tedTaskDetails/JobResultsDetails/ExpandedIssues.js
95.45% <95.45%> (ø)

...TaskDetails/JobResultsDetails/JobResultsDetails.js
100.00% <100.00%> (+5.00%)
:arrow_up:

...ails/JobResultsDetails/jobResultsDetailsHelpers.js
100.00% <100.00%> (ø)

src/SmartComponents/completedTaskDetailsHelpers.js
100.00% <100.00%> (ø)

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

@AsToNlele I don't think your suggestions saved. I don't see any. :)

Oooh sorry about that :smile:


return 'warning' compatibility when segments are not found

Send the 'warning' compatibility type when public segments in imported lists are not found.

This issue is solved and the same code is present in my PR.

Duplicate of https://github.com/CarnegieLearningWeb/UpGrade/pull/1948


Fanduel PGA can't generate lineups

Hi there,
Really appreciate the work you have done on this package, its been tremendously useful.
I've had no problems using it for mlb and pga on DK. I'm currently attempting to use the optimizer for pga on FanDuel, and it is saying it can't solve. When using the debugger, it looks like all the player information is correctly imported. I've tried with importing the players using csv and Player objects. Any ideas?

Hi, could you provide csv that you used I'll try to investigate this issue.

360-1296optimizerprojections.csv.zip

It's a bug in fanduel golf. I'll fix it in the next release. For now, you can add this line before running optimize:
optimizer.settings.min_teams = None


Pull request with type [CREATE] for user [Ioan-Popovici] and object type [CMPivotQuery] - titled [Summarize - Windows OS Versions]

Automated pull request from the Endpoint Configuration Manager Community hub.

This is a duplicate of the OS - Version Summarization cmpivot script. I don't know how to change the name in company portal.

Remove the "OS - Version Summarization".
Thanks


navbar_alerts: Fix navbar alerts inaccessibility.

Fixes #16079.

Testing Plan:
GIFs or Screenshots:

Enable desktop notification

Ask me later

Never on this computer

Closing

A few things I notice when testing:

The "X" at the right end of these banners is before the more obvious menu items in the tab order; we likely want to change the HTML ordering to fix that.
For the desktop notifications one, Hitting Enter didn't seem to navigate to the next state when I tested it; not sure why as it seems to work in your video above.

@timabbott I understood about the part about tab order. But I've to look into other part.

Hi, you should not merge master into your branch but instead rebase your branch on top of master (the PR should only contain your commits).

Yeah, Sorry. I am fixing this right now.

On your branch you can do a git reset --hard to your last commit. For rebasing on the latest upstream master, I recommend adding https://github.com/zulip/zulip as a remote and having your master branch track it.

@shanukun can you comment specifically on if/how you fixed the two bugs I mentioned?

@timabbott I made the changes suggested by @Gittenburg. As for the other two bugs you mentioned, I'm still working on them.

@Gittenburg Can you please review the changes.

This is great, merged as 5c6d42c6a5f1c8afe0b16ba1c74d20cdfc5f8d8c after fixing the two small comments above and:

Fixing the commit messages to spell Fixes: #16079 correctly in the second, and not close the issue when the first was merged.  (Generally, you don't want to put Fixes: #16079 in every single commit).
Squashing the first two commits; since the second commit just fixed a bug in the first, it's cleaner to just have those two be a single commit that does it right from the beginning.

Thanks @shanukun and @Gittenburg!


Amos/add pre execution validations

This change is

Codecov Report

Merging #1013 (2920181) into main (9db8930) will decrease coverage by 0.18%.
Report is 9 commits behind head on main.
The diff coverage is 58.82%.

@@            Coverage Diff             @@
##             main    #1013      +/-   ##
==========================================
- Coverage   66.85%   66.68%   -0.18%
==========================================
  Files          51       51
  Lines        6210     6280      +70
  Branches     6210     6280      +70
==========================================
+ Hits         4152     4188      +36
- Misses       1679     1710      +31
- Partials      379      382       +3

Files
Coverage Δ

crates/blockifier/src/fee/fee_utils.rs
97.50% <100.00%> (ø)

crates/blockifier/src/fee/gas_usage.rs
99.26% <75.00%> (-0.74%)
:arrow_down:

crates/native_blockifier/src/py_utils.rs
0.00% <0.00%> (ø)

crates/native_blockifier/src/errors.rs
0.00% <0.00%> (ø)

crates/blockifier/src/transaction/errors.rs
0.00% <0.00%> (ø)

crates/blockifier/src/transaction/objects.rs
56.41% <50.00%> (-0.12%)
:arrow_down:

.../blockifier/src/transaction/account_transaction.rs
93.02% <80.00%> (-1.25%)
:arrow_down:

crates/native_blockifier/src/py_validator.rs
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove VersionInfo in 1.2 branch

For status checks on the develop branch, please use TransportPackage-Foundation
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=57248)
and run the build against your PR branch with the default parameters.
For status checks on the main branch, please use microsoft.ProjectReunion
(https://dev.azure.com/ms/ProjectReunion/_build?definitionId=391&_a=summary)
and run the build against your PR branch with the default paramters.

/azp run

/azp run

/azp run


new error on master?

I was running fixdes most of the day and just pulled and installed master. I'm now getting this error
'''Traceback (most recent call last):
File "C:/Users/mayork/Documents/GitHub/gpkit-models/gpkitmodels/Commercial Sizing/test.py", line 377, in
m = CommercialAircraft()
File "C:/Users/mayork/Documents/GitHub/gpkit-models/gpkitmodels/Commercial Sizing/test.py", line 351, in init
cmc = CommericalMissionConstraints(1)
File "C:/Users/mayork/Documents/GitHub/gpkit-models/gpkitmodels/Commercial Sizing/test.py", line 191, in init
Model.init(self, W_ftotal, constraints, **kwargs)
File "c:\users\mayork\documents\github\gpkit\gpkit\constraints\model.py", line 65, in init
self._add_modelname_tovars(self.name, self.num)
File "c:\users\mayork\documents\github\gpkit\gpkit\constraints\model.py", line 100, in _add_modelname_tovars
self.subinplace(add_model_subs)
File "c:\users\mayork\documents\github\gpkit\gpkit\constraints\costed.py", line 25, in subinplace
ConstraintSet.subinplace(self, subs, value)
File "c:\users\mayork\documents\github\gpkit\gpkit\constraints\set.py", line 128, in subinplace
constraint.subinplace(subs, value)
File "c:\users\mayork\documents\github\gpkit\gpkit\nomials\nomial_math.py", line 442, in subinplace
nomial.subinplace(substitutions, value)
AttributeError: 'list' object has no attribute 'subinplace' '''
I think its an error on master that occured after merging fixdes because I pulled and installed nomialmap and I could run the code fine

crap this models not normal gpkit


Chrome : (Sometimes) NotFoundError: Failed to execute 'transaction' on 'IDBDatabase'

Hi,
I use Dexie for my project (Dexie v2.0.0 + Dexie.Observable)
With Chrome, some users (1 ~ 5 / 500) encounter an error at db opening :
NotFoundError: Failed to execute 'transaction' on 'IDBDatabase': One of the specified object stores was not found.
I don't find correlation between these users.
One use under windows, another Chrome for Mac.
One correct the problem using private mode for chrome.
Conversely, another quit private mode for making work the application.
I don't manage to reproduce on my side (I try many combination of parameters)...
The user who help me more and give me the maximum information :
Version 62.0.3202.62 (Build officiel) (64 bits) under MAC OS.
He clean historic, database etc.
Problem is systematic .
But in private mode : it works...
If someone have an idea ? Or a proposal for debugging ?
Thanks,
Jonathan

After investigating It was my fault.
2 pages with Dexie : one import observable, the other not.
I don't understand why it works for 99% of the users...
I patch with the explanation found in documentation (version +1 with empty change)


Addon manager is broken

How are you using the lua-language-server?
Visual Studio Code Extension (sumneko.lua)
Which OS are you using?
Windows
What is the issue affecting?
Libraries
Expected Behaviour
There should be buttons to turn addons on and off.
Actual Behaviour
There are no buttons to turn addons on and off.
Reproduction steps

Go to 'Lua Addon Manager'

Additional Notes
Yesterday, the buttons were shown.
Log File
No response

Hello 👋
Did you have a workspace open? You can only enable/disable addons in a workspace, so opening VS Code in "file only" mode could be the cause.
Otherwise, please include your log file to help debug. You can open the log file with the little page icon in the bottom right.

No, it was launched without a workspace, launching in a workspace helped solve the problem. Thanks!


Nether Portal Error

Hello, i have a Problem!
If I go into a Nether Portal without setting a Multiverse Portals Portal there, I get an Internal Server Error message and get kicked out of the game. See picture. When I rejoin I end up in an empty world with a strange construct, see picture 2. The following appears in the console: https://pastebin. com/RDcBN4mP How can I fix this?
Please answer me quickly!
THX HyperDEV

The error doesn't really point anything to it being mv related. That being said, can you try with paper instead of spigot, https://papermc.io/downloads, and only mv plugins installed?

The error doesn't really point anything to it being mv related. That being said, can you try with paper instead of spigot, https://papermc.io/downloads, and only mv plugins installed?

Yes, only with Multiverse Portals and Multiverse Core the error is still there.

Yes, only with Multiverse Portals and Multiverse Core the error is still there.

And you tested with paper mc?

And you tested with paper mc?

I use a Nitrado Server! The servers are expensive, and I don't want to have to pay money for such a small thing. Is there also a free variant?

I use a Nitrado Server! The servers are expensive, and I don't want to have to pay money for such a small thing. Is there also a free variant?

Not sure what you mean, papermc is completely free to download and use.

Not sure what you mean, papermc is completely free to download and use.

Okay, I’ll give it a try. I’ll call you when I’m finished

Okay, I’ll give it a try. I’ll call you when I’m finished

I have now run Multiverse Core and Multiverse Portals on paper. The problem is still there.

I have now run Multiverse Core and Multiverse Portals on paper. The problem is still there.

Please provide the full log instead of just the error. Also, please provide the link given by /mvv -p

Please provide the full log instead of just the error. Also, please provide the link given by /mvv -p

Is fixed, thx


implemented metalink4 format

Overview
Completed implementation of Metalink and Metalink4 outputs.

added file size attribute
added file hash attribute

I think we should polish the implementation a bit and add some developer documentation.

I pushed directly to the branch.


Gmail Offline Mail Security Setting

Hi Rob:
It looks like we need one more admin change on the security settings for offline CC Gmail.
On my personal Gmail account - I can download and view attachments offline. I cannot do so on my CC Mail account. Having access to my mail attachments offline is really helpful.
Settings for CC Account say:
Security: | Logging out of my Google account will remove locally stored dataFiles will not be stored on your computer after logging out. Data will need to be resynced when logging back into Creative Commons Mail, which may take a few hours.
=====================
Settings for Personal Gmail Account say:
| Keep offline data on my computerData stored on your device will not be deleted when signing out of your Google account or when changing your password. To delete account data from your device disable offline mail and save changes. - I check this one so I can keep my attachments offline.
| Remove offline data from my computerData will need to be resynced to your computer when logging back into Gmail. It may take a few hours to resync the mailbox.

Heya Cable.
I've disabled "Force deletion of offline data on log out of Google account", which should have the desired effect.
I reserve the right to re-enable it should our revised document retention or other policies require it. :-)


Create docs for Plot / Charts

We need more information about plot results and customizations

Done https://rodrigo-brito.github.io/ninjabot/docs/ploting/


1469: add missing langgraph.graph import

Hi,
New here 👋
Addresses 1469
Hopefully this one's trivial ;)
Before:
from langgraph.graph import END, MessageGraph

After:
from langgraph.graph import START, END, MessageGraph

Thank you for reporting and for the contribution -- it's already fixed in https://github.com/langchain-ai/langgraph/pull/1472 :)


Battery Details Issue in Settings

Device Model
iPhone14pm
iOS Version
16.6.1
Bootstrap Version
1.1
Serotonin Version
No response
What happened?
Battery Usage Details Cannot Be Loaded During Injection Settings
Log output
No response
Acknowledgement of README and FAQ

[X] I have read both the README and the FAQ.

Duplicate of #52


Can't publish locally for developing

I followed CONTRIBUTING.md but the below error happens.
Looks like a credential file for Bintray is required even for local development ??
$ sbt -Dsbt.build.version=1.3.0-RC100 -Dsbt.build.offline=true                                                                                16:16:59
[info] Loading global plugins from /home/vagrant/.sbt/0.13/plugins
[info] Loading project definition from /home/vagrant/IdeaProjects/sbt-launcher-package/project
[warn] Missing bintray credentials. Either create a credentials file with the bintrayChangeCredentials task, set the BINTRAY_USER and BINTRAY_PASS environment variables or pass bintray.user and bintray.pass properties to sbt.
[warn] Missing bintray credentials. Either create a credentials file with the bintrayChangeCredentials task, set the BINTRAY_USER and BINTRAY_PASS environment variables or pass bintray.user and bintray.pass properties to sbt.
[warn] Missing bintray credentials. Either create a credentials file with the bintrayChangeCredentials task, set the BINTRAY_USER and BINTRAY_PASS environment variables or pass bintray.user and bintray.pass properties to sbt.
[warn] Missing bintray credentials. Either create a credentials file with the bintrayChangeCredentials task, set the BINTRAY_USER and BINTRAY_PASS environment variables or pass bintray.user and bintray.pass properties to sbt.
[warn] Missing bintray credentials. Either create a credentials file with the bintrayChangeCredentials task, set the BINTRAY_USER and BINTRAY_PASS environment variables or pass bintray.user and bintray.pass properties to sbt.
[warn] Missing bintray credentials. Either create a credentials file with the bintrayChangeCredentials task, set the BINTRAY_USER and BINTRAY_PASS environment variables or pass bintray.user and bintray.pass properties to sbt.
[warn] Missing bintray credentials. Either create a credentials file with the bintrayChangeCredentials task, set the BINTRAY_USER and BINTRAY_PASS environment variables or pass bintray.user and bintray.pass properties to sbt.
[warn] Missing bintray credentials. Either create a credentials file with the bintrayChangeCredentials task, set the BINTRAY_USER and BINTRAY_PASS environment variables or pass bintray.user and bintray.pass properties to sbt.
[info] Set current project to sbt-launcher-packaging (in build file:/home/vagrant/IdeaProjects/sbt-launcher-package/)
> universal:publish
[warn] Credentials file /home/vagrant/.bintray/.credentials does not exist
[warn] Credentials file /home/vagrant/.bintray/.credentials does not exist
INF: [console logger] dispatch: repo.scala-sbt.org GET /scalasbt/maven-releases/org/scala-sbt/sbt-launch/1.3.0-RC100/sbt-launch.jar HTTP/1.1
[info] Compiling 2 Java sources to /home/vagrant/IdeaProjects/sbt-launcher-package/java9-rt-export/target/classes...
[info] Packaging /home/vagrant/IdeaProjects/sbt-launcher-package/java9-rt-export/target/java9-rt-export-0.1-SNAPSHOT.jar ...
[info] Done packaging.
Downloading https://repo1.maven.org/maven2/org/scala-sbt/sbt/1.3.0-RC100/sbt-1.3.0-RC100.pom.sha1
Downloading https://repo1.maven.org/maven2/org/scala-sbt/sbt/1.3.0-RC100/sbt-1.3.0-RC100.pom
Downloaded https://repo1.maven.org/maven2/org/scala-sbt/sbt/1.3.0-RC100/sbt-1.3.0-RC100.pom.sha1
Downloaded https://repo1.maven.org/maven2/org/scala-sbt/sbt/1.3.0-RC100/sbt-1.3.0-RC100.pom
Resolution error: Error downloading org.scala-sbt:sbt:1.3.0-RC100
  not found: /home/vagrant/.ivy2/local/org.scala-sbt/sbt/1.3.0-RC100/ivys/ivy.xml
  not found: https://repo1.maven.org/maven2/org/scala-sbt/sbt/1.3.0-RC100/sbt-1.3.0-RC100.pom
[trace] Stack trace suppressed: run last dist/*:exportRepoUsingCoursier for the full output.
[trace] Stack trace suppressed: run last root/*:sbtLaunchJar for the full output.
[error] (dist/*:exportRepoUsingCoursier) bridge not found
[error] (root/*:sbtLaunchJar) dispatch.classic.StatusCode: Unexpected response code: 404

-Dsbt.build.offline=true creates an offline image of sbt by downloading all artifacts from Maven Central. It should work if you use an existing sbt version or put false.

Closing since workaround exists.


Default formatprog when multiple are present - README

Hello,
The README states:
Here is a list of formatprograms that are supported by default, and thus will be detected and used by vim when they are installed properly.

It's not clear from this description as to which format program will be used when there are multiple ones in formatterpath - for eg., if both autopep8 and yapf exist. Could this be clarified?

Feel free to propose any clarifications in a PR

Feel free to propose any clarifications in a PR

But I wouldn't know what the answer/clarification to it is. Even with the verbose mode and messages, I couldn't tell which formatter was being used.

The lists of default formatters are defined in https://github.com/Chiel92/vim-autoformat/blob/master/plugin/defaults.vim
They are just tried in order


[FunctionTrace] Finish Function Tracer module

[x] Iterate through the entire program and get details on all the function calls
[x] Sort whether each call is a library call or a user defined call
[x] Store addresses of each function call
[x] Store statistics of each function call (ie. how many times is it called)

TODO:

[ ] Get function arguments for each call
[ ] Mark out common arguments for multiple functions - (It can help us figure out some pattern)

This is what the output looks like

We need to find a better approach to recognizing library functions, for dynamically linked binaries at least can't we just get this info from the .plt section?

[x] Get function arguments for each call (Partial support)

I can't think of ways of getting the number of arguments passed to each function.

[ ] Mark out common arguments for multiple functions

Wouldn't this be better with the taint module ? That way we will be able to identify the value passed as well.
I've attached link to the gist that contain the log while running this on babyvm log

I can't think of ways of getting the number of arguments passed to each function.

[ ]  Mark out common arguments for multiple functions

Wouldn't this be better with the taint module ? That way we will be able to identify the value passed as well.

Yes, it might also be easier to recognize the arguments using the taint module. But it is quite a complex problem so we might end up writing a separate module. We need to discuss how to go about this.

I've attached link to the gist that contain the log while running this on babyvm log

A lot of data dumped at once. We need to rewrite it in a way where the user can chose to run a module with options specified. For example, which function to dump this data for. I guess this is already on the roadmap.

A lot of data dumped at once. We need to rewrite it in a way where the user can choose to run a module with options specified. For example, which functions to dump this data for. I guess this is already on the roadmap.

Yes, I was intending on doing the same.

Yes, it might also be easier to recognize the arguments using the Taint module. But it is quite a complex problem so we might end up writing a separate module. We need to discuss how to go about this.

I'm reading through a couple of methods by which we can find the function signature of the file and then use that to our advantage like what has been done for syscalls in Syscalls Tracer.
Merging this PR for now.


Added address specification for port forwarding

The current implementation allows making forwarding only to localhost. At the same time kubectl has -address parameters allowing to open forwarding to LAN.
I needed this option for my current project and decided that it might be interesting to someone else

I can take a deeper look it this later this week. kubectl only sets up one port forward. How does this feature handle duplicate ports?

I can take a deeper look it this later this week. kubectl only sets up one port forward. How does this feature handle duplicate ports?

As I checked it cannot expose services with duplicated ports to LAN. But at least it shows a meaningful message so the user can do something with it. The feature is optional, so with no -a [ip] everything works as before

@YuriU thank you for your thoughtful contribution. I do not believe this feature fits well kubefwds core functionality and purpose, which it to replicate service names and ports as they are in the cluster.
Due to it's very specific use case I believe a custom shell script or purpose built application would be a better route.


Fixing issue where entry chunk is not added to local chunks if another chunk is processed first.

Summary
After upgrading to repack 4, I noticed that after webpack generated and built my app, despite webpack completing successfully android builds were no longer copying the main javascript bundle to android/app/build/generated/assets/. I believe this is the same symptom as #654.
After digging into the code, it turns out that when the OutputPlugin was processing chunks, on android it would just so happen to process one of my remote chunks first that had a dependency on the entry chunk. Since that dependency existed, the entryChunk would get added to sharedChunks before it had a chance to be added to localChunks, preventing it from being copied to the output folder properly.
Test plan

[ ] Have an app with remote chunks that reference the main entry chunk.
[ ] Run webpack-bundle on android to build an android release build.
[ ] Ensure the destination android folder (android/app/build/generated/assets/ in my case) contains the main javascript bundle.
[ ] (Verify other configurations continue to work)

Hi @sleeper-luke, sorry for the delay and thanks for your contribution!
I took a look at this PR quite a few times but couldn't determine whether this was the correct going forward.
I went ahead and refactored the logic of Output Plugin's chunk categorisation algorithm in #678.
This should serve a nice starting point for explicitly defining the spec for what gets copied and what not.
The PR there includes a canary release which you can install and test:
npm install<EMAIL_ADDRESS>If you are still facing the issue from the PR, let me know whether this fixes it 🙏

Hi @sleeper-luke, we've just released Re.Pack 4.2.0, please see if it solves your issues with OutputPlugin

Closing since the PR is stale, and most likely already fixed in current stable versions


External jenkins test job images failed to create and didn't fail the build_image steps

Jobs failed at compiling stage but get BUILD SUCCESSFUL and continued to run test stage.
https://ci.adoptopenjdk.net/view/Test_external/job/Test_openjdk11_hs_extended.external_x86-64_linux_jenkins/30/console
23:15:29       [exec]  ---> ea5c394c8d3b
23:15:29       [exec] Step 8/21 : RUN apt-get update 	&& apt-get install -y --no-install-recommends git maven 	&& rm -rf /var/lib/apt/lists/*
23:15:29       [exec]  ---> Running in c55b0d10c17e
23:15:30       [exec] no status provided on response: unknown
23:15:30       [exec] Result: 1
23:15:30
23:15:30  dist:
23:15:30       [copy] Copying 2 files to /home/jenkins/workspace/Test_openjdk11_hs_extended.external_x86-64_linux_jenkins/jvmtest/external/jenkins
23:15:30
23:15:30  BUILD SUCCESSFUL
23:15:30  Total time: 45 seconds
23:15:30  0

Hi @karianna @sophia-guo I want to work on this issue. Can you please guide me more in this issue like what files to look for etc.
I am an Outreachy applicant.
Thank You

This is reported against external/jenkins jobs (ones running in weekly automation are here: https://ci.adoptopenjdk.net/view/Test_external/job/Test_openjdk11_hs_extended.external_x86-64_linux_jenkins), though it is likely more pervasive than just the external/jenkins tests.
A place to start on this one is to look at external/build.xml and external/jenkins/build.xml to see how the test target is built.
I suspect that the 'build_image' target, which calls a shell script (external.sh) should be changed to fail on error.  external.sh may also need to be changed to produce errors when things fail.


Unable to create new KAS registry entry

When trying to create a new KAS info item with the command:
go run github.com/opentdf/otdfctl@latest \
   --host=http://localhost:8080 --tls-no-verify --log-level=debug --with-client-creds=$CREDS \
    policy kas-registry create -u http://localhost:8080 -p "$(<../platform/kas-cert.pem)"

I get an error of the form
   ERROR    Failed to create KAS registry entry: rpc error: code = Internal desc = resource creation failed
exit status 1

looking at the KAS logs I see
time=2024-08-01T17:02:25.890-04:00 level=DEBUG msg="allowed by policy" subject=role:org-admin resource=policy.kasregistry.KeyAccessServerRegistryService/CreateKeyAccessServer action=write
time=2024-08-01T17:02:25.890-04:00 level=DEBUG msg="creating key access server" namespace=policy
time=2024-08-01T17:02:25.892-04:00 level=ERROR msg="Encountered database error" error="ERROR: relation \"key_access_servers\" does not exist (SQLSTATE 42P01)"
time=2024-08-01T17:02:25.892-04:00 level=ERROR msg="Unknown error code" error="relation \"key_access_servers\" does not exist" code=42P01

Investigated with @dmihalcik-virtru and found to have likely been caused by a local development environment/flow and not by a bug. Will reopen if there's another encounter.


Version property for image is missing

The article states:
Requirements for configuring automatic OS image upgrade
The version property of the image must be set to **.
But ** is not a valid value. It seems like the actual value is missing (e.g., "latest").
This change was introduced in a recent commit:
https://github.com/MicrosoftDocs/azure-docs/commit/cad82b58c077bc87b307f7a97d2cfa53ba332c9c
This item was supposed to be fixed as per this issue:
https://github.com/MicrosoftDocs/azure-docs/issues/121538
However, it is still live/not fixed.

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

ID: 51c75ce7-2144-d81a-693c-fb51cc00f3e5
Version Independent ID: cba654ea-ee84-fca4-a76d-717bfb9d9bf5
Content: Automatic OS image upgrades with Azure Virtual Machine Scale Sets - Azure Virtual Machine Scale Sets
Content Source: articles/virtual-machine-scale-sets/virtual-machine-scale-sets-automatic-upgrade.md
Service: virtual-machine-scale-sets
Sub-service: automatic-os-upgrade
GitHub Login: @ju-shim
Microsoft Alias: jushiman

@Wh00T
Thanks for your feedback! We will investigate and update as appropriate.

@Wh00T |
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

The handling of this issue by Microsoft is puzzling. The change has been implemented in this commit https://github.com/MicrosoftDocs/azure-docs/commit/5b04aff6c7f6118621ddab469afdc528a903befa, a bit sloppily as the word latest is not emphasized as it used to be, but this is not the main problem. The screenshot below shows the change has been merged.

The main problem is, that the author of the commit may be possibly unaware of the change, and assigns the issue to @ju-shim.
If this is an oversight due to a possible lack of awareness of the repository state by the contributors, this would extend the scope of mismanagement in addition to what's described in https://github.com/MicrosoftDocs/azure-docs/issues/121656

It looks like the change has successfully published. It now says latest.
It takes time for a PR that has been merged to go live. Perhaps this accounts for the discrepancy between commits and what the live doc says, depending on when the changes merged and you checked the page.
#please-close


Image classification on fruit 360 dataset

This tutorial discusses using CNN for image classification on fruit 360 dataset..

Really nice! Looking forward testing it! THANK YOU!

Really nice! Looking forward testing it! THANK YOU!

Thanks @yvanlebras!

"When we get to the top right corner of the image, we simply move the filter down one pixel and restart from the right" -> the gif seems to show you move the filter down one pixel and restart from left isn't it? Moreover, it is mention then that the process will stop when reaching bottom right pixel... So this seems to go in the direction each "line" of pixels is considered from left to right isn't it?

I try to follow the tutorial and have an error with the first tool keras model config
Here is my history https://usegalaxy.eu/u/ylebras/h/test-tuto-cnn-deep-learning-machine-learning-images-ia
Here the std out
<EMAIL_ADDRESS>line 387, in _fix_unknown_dimension
    raise ValueError(msg)
ValueError: total size of new array must be unchanged

Maybe "just" I made a mistake filling form but in case not....

"When we get to the top right corner of the image, we simply move the filter down one pixel and restart from the right" -> the gif seems to show you move the filter down one pixel and restart from left isn't it? Moreover, it is mention then that the process will stop when reaching bottom right pixel... So this seems to go in the direction each "line" of pixels is considered from left to right isn't it?

You are correct. That's a typo.

I try to follow the tutorial and have an error with the first tool keras model config
Here is my history https://usegalaxy.eu/u/ylebras/h/test-tuto-cnn-deep-learning-machine-learning-images-ia
Here the std out
<EMAIL_ADDRESS>line 387, in _fix_unknown_dimension
    raise ValueError(msg)
ValueError: total size of new array must be unchanged

Maybe "just" I made a mistake filling form but in case not....

Looked at your history. I think you have set the input shape to 300,000 instead of 30,000 in your Keras model config.

Fixed all the typos. Thanks @anuprulez!

Hi Kaivan, thank you for the new version ! I have some issue on the last step, in case you have idea about the reason : https://usegalaxy.eu/u/ylebras/h/test-tuto-cnn-deep-learning-machine-learning-images-ia

Hi @kxk302
I tried to run this tutorial on usegalaxy.eu but getting this error
If using Keras pass *_constraint arguments to layers.
Traceback (most recent call last):
  File "/opt/galaxy/shed_tools/toolshed.g2.bx.psu.edu/repos/bgruening/keras_train_and_eval/785dd890e27d/keras_train_and_eval/keras_train_and_eval.py", line 551, in <module>
    fasta_path=args.fasta_path,
  File "/opt/galaxy/shed_tools/toolshed.g2.bx.psu.edu/repos/bgruening/keras_train_and_eval/785dd890e27d/keras_train_and_eval/keras_train_and_eval.py", line 421, in main
    ) = train_test_split_none(X, y, groups, **test_split_options)
  File "/opt/galaxy/shed_tools/toolshed.g2.bx.psu.edu/repos/bgruening/keras_train_and_eval/785dd890e27d/keras_train_and_eval/keras_train_and_eval.py", line 103, in train_test_split_none
    rval = train_test_split(*new_arrays, **kwargs)
  File<EMAIL_ADDRESS>line 65, in train_test_split
    arrays = indexable(*arrays)
  File<EMAIL_ADDRESS>line 230, in indexable
    check_consistent_length(*result)
  File<EMAIL_ADDRESS>line 205, in check_consistent_length
    " samples: %r" % [int(l) for l in lengths])
ValueError: Found input variables with inconsistent numbers of samples: [5015, 5016]

Also, for prediction on the test data, shouldn't we convert the test data to OHE? thanks!

https://usegalaxy.eu/u/ylebras/h/test-tuto-cnn-deep-learning-machine-learning-images-ia

Hi Anup,
The test labels need not be converted to OHE. We only do that for training labels for categorical cross-entropy calculation. Can you please share your history? I can take a look. Thx

https://usegalaxy.eu/u/ylebras/h/test-tuto-cnn-deep-learning-machine-learning-images-ia

Hi @yvanlebras
I created a workflow from your history and it has an extra input, called 'input dataset'. What should I set that to, to run your workflow? Thx

Hi @kxk302 https://usegalaxy.eu/u/kumara/h/fruitdatasets. I have made this history accessible but not sure if you will see the actual datasets. If not, can you give me your Galaxy's email id/username to share this history

Hi Kaivan,

https://usegalaxy.eu/u/kumara/h/fruitdatasets

Hi @anuprulez
I imported your history but I get "You do not have permission to view this dataset" and all the jobs are in gray with a lock next to them.

Can you reload this history link? It should work now
https://usegalaxy.eu/u/kumara/h/fruitdatasets

Can you reload this history link? It should work now
https://usegalaxy.eu/u/kumara/h/fruitdatasets

Thanks @anuprulez . I think I know what the issue is. In the 'Deep learning training and evaluation' step, you must use the OHE labels, not train_y_10. That should fix the problem you are seeing.

So it appears to me we need to specify the fact that also for test_y_10 file labels, we need OHE creation. I propose a commit linked to it (duno why I only ca ncommit, not PR... sorry). Moreover, we can add a "tip" section after/before this OHE step so we can propose the user to know how many classes there is thanks to the use of "count occurences of each record" Galaxy tool on the first column of these test_y_10 and train_y_10 files

Can you reload this history link? It should work now
https://usegalaxy.eu/u/kumara/h/fruitdatasets

Thanks @anuprulez . I think I know what the issue is. In the 'Deep learning training and evaluation' step, you must use the OHE labels, not train_y_10. That should fix the problem you are seeing.

@anuprulez I verified that your workflow now runs fine with the change I suggested.

So it appears to me we need to specify the fact that also for test_y_10 file labels, we need OHE creation. I propose a commit linked to it (duno why I only ca ncommit, not PR... sorry). Moreover, we can add a "tip" section after/before this OHE step so we can propose the user to know how many classes there is thanks to the use of "count occurences of each record" Galaxy tool on the first column of these test_y_10 and train_y_10 files

@yvanlebras I don't think that is necessary. I just fixed Anup's issue. Let me look at your workflow and I'll get back to you shortly.

I apply an OHE creation to the test_y_10.tsv file + choose "all columns" in the "machine leaning visualization Extension" and now have a result, but maybe not the good one ?

I apply an OHE creation to the test_y_10.tsv file + choose "all columns" in the "machine leaning visualization Extension" and now have a result, but maybe not the good one ?

@yvanlebras Noticed that in 'classification confusion matrix plot' step, you have 'Does the dataset contain header' set to 'No' for test_y_10. I changed that to 'Yes' and re-ran it and it completed fine. Also, it seems you have 2 classification confusion matrix plot steps. I guess one is redundant.

@anuprulez @yvanlebras It seems all the issues are resolved now. Could one of you please approve/merge this PR if there are no outstanding items left? Thanks!

Looks good to me. If @yvanlebras is also fine with, I would be happy to merge the PR.

@kxk302 can you rebase it against main? Thanks!

@kxk302 can you rebase it against main? Thanks!

Done @anuprulez !

Hey @kxk302 this doesn't look properly rebased, it looks like you merged in all of the updates on the main branch somehow, but not with a merge commit that github could understand. Could you tell me what you did precisely so we can advise against it in the other cases when it happens? This diff has me a bit worried with how unclear the changes are

Yes, now it indicates 168 files as changed, which should not be the case

Huh! Let me double check what I did (I think its advisable to have your coffee first before doing any Git operation!)

Ok, I got rid of the problematic commit and force-pushed the change to my branch. I tried to rebase main but it says everything is up to date. So, I guess when the tests pass, this PR should be good to merge?

Thank you all!


M1 compatibility for mysql

Looks good, thanks, Sebastian!.
I guess the local Dockerfile image for the Node.js app doesn't need any specific tag for when it gets built via compose, or does it?

Hi @lirantal ,

No changes are need to the Dockerfile, the images in use should be platform-agnostic
We just tested this on a Intel Mac successfully


Update to Site-Based Observations

Forked to ensure correct results by aligning measures datatypes also add comments, sections, and headers for easier understanding. Now returning metadata and allows for different return languages, units, and no longer do postal codes need to be prefaced with the correct country code. This is now handled in the extension. Also updated so columns with spaces will no longer be an issue.

@hangtime79
It's realy great for you PR, and make this extension more robust and user friendly.
One thing is.
Can you add long time(more than 31 days) process, since old extenison support this.
Below code split start end date, and process it month by month.
  for(day in format(seq(sStartDate,sEndDate,30),"%Y%m%d")){
    dayNumber <- as.Date(day,"%Y%m%d")
    sMonthStartDate <- day#format(day,"%Y%m%d")
    if(dayNumber + 30 > sEndDate )
      sMonthEndDate <- format(sEndDate,"%Y%m%d")
    else
      sMonthEndDate <- format(dayNumber + 30,"%Y%m%d")
    sDateRange = paste("&startDate=",
                       sMonthStartDate, "&endDate=",
                       sMonthEndDate, sep="")
    sRequestURL <- generateUrl(sLocationUrl,sDateRange)
    print(sRequestURL)
    resultData <- rbind(resultData,retriveDataFromTWC_OneMonth(sRequestURL))
    print(nrow(resultData))
  }

another thing is about
packages <- function(x){
  x <- as.character(match.call()[[2]])
  if (!require(x,character.only=TRUE)){
    install.packages(pkgs=x,repos="http://cran.r-project.org")
    require(x,character.only=TRUE)
  }
}

this function try to auto install required package. but you didn't use it in extension.
But I am OK if you don't want it, because it may cause a protention issues, when some package updated(install.packages will install latest package).
Thank you very much.
Yu Wenpei.


Natural Language Programming?

Not sure if i can ask here,
Will there be a compiler designed to operate on natural language [english code]?
instead of using C, C++ etc

Not in the short term


chore: update links for real time information

I've changed the link because the old ones display old information, this link now should display repository current information :)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.


[v2.7-branch] actions: west/devicetree: exclude python 3.6 on windows

This version of python is not available anymore. Excluding for now to
unblock CI.
 I think that's the best way to go.

@mbolivar-nordic - is this basically ready? There are a couple of compliance warnings so I wasn't sure.

@mbolivar-nordic - is this basically ready? There are a couple of compliance warnings so I wasn't sure.

Yep, should be with the latest push.


[xdmf] Fixed serialization bug for mixed data with lines

This patch fixes a bug in the generation of xdmf data file.
It appears when one wants to open the result file in ParaView
Below is a python code that reproduce the bug: it creates xdmf file that makes ParaView crash. With this patch, the test code generate xdmf file that can be opened in ParaView.
import numpy as np
import meshio

points = np.array([
    [0.0, 0.0, 0.0],
    [0.0, 1.0, 0.0],
    [0.0, 0.0, 1.0],
    ])
cells = {
    'line':np.array([
        [0, 1],
        [1, 2],
        [2, 0],
        ]),
    'triangle': np.array([
        [0, 1, 2]
        ])
    }
meshio.write(
    'foo.xdmf',
    points,
    cells
)

Very good, thank you!


Try to get travis working

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@082c751). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master       #3   +/-   ##
=========================================
  Coverage          ?   85.21%
=========================================
  Files             ?        3
  Lines             ?      115
  Branches          ?        0
=========================================
  Hits              ?       98
  Misses            ?       17
  Partials          ?        0

Impacted Files
Coverage Δ

nhdspy/__init__.py
83.9% <100%> (ø)

nhdspy/test/test_NHDSPy.py
100% <100%> (ø)

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 082c751...b71189c. Read the comment docs.


Fix toDateTime calculation

toDateTime was returning dates that are 30 minutes past the actual time in seconds.
No context for this decision was given in the commit that brought it in, so I figured it must be a bug.

https://github.com/vercel/nextjs-subscription-payments/pull/278


acc not match(72.03 vs 75.4)

It's same that the release model is not the best one.
python validate.py "MYDATA/ImageNet" --model "van_tiny" --checkpoint "weights/van_tiny_754.pth.tar" -b 200
The result is following: Acc@1 72.032 (27.968) Acc@5 91.126 (8.874)

I check the state_dict of checkpoint and found that ema version is not exist, and I think the ema version's accuracy is same with paper.

so will you check it ?

Amazing.
I download the model from google drive and can get the 75.4 % Acc.

Here is my eval command

I also load the model and print key. It is same with you.

it's so strange！ I tried the tiny and small, all not match. I also tried other model (vip and volo) and can got same acc as paper.
the model i tried was download from tsinghua cloud, i will try another one from google cloud later.

I try to use the model downloaded from tsinghua cloud. However, i also get the 75.4 % acc.

The version of my softwares:
Pytorch == 1.7.1 (I heard it is important)
Timm == 0.4.12

I try it again and find that the version of timm influence so much.
when version of timm is 0.4.9, the acc is 72.03; when I change it the latest one, the acc is 75.376(matched with your result).
so, maybe there are some difference in the preprocessing part.

I have the same problem, with Pytorch == 1.7.1 and Timm == 0.4.12
Loaded state_dict from checkpoint 'van_tiny_754.pth.tar'
Model van_tiny created, param count: 4105800
Data processing configuration for current model + dataset:
input_size: (3, 224, 224)
interpolation: bicubic
mean: (0.5, 0.5, 0.5)
std: (0.5, 0.5, 0.5)
crop_pct: 0.9
Test: [   0/20]  Time: 1.760s (1.760s,  113.65/s)  Loss:  0.8368 (0.8368)  Acc@1:  82.500 ( 82.500)  Acc@5:  98.500 ( 98.500)
Test: [  10/20]  Time: 0.253s (0.469s,  426.49/s)  Loss:  1.1970 (1.1323)  Acc@1:  72.500 ( 76.636)  Acc@5:  93.500 ( 94.364)

Acc@1 74.586 (25.414) Acc@5 92.786 (7.214)


add six

@nsoranzo I could find an obvious reason why six is not installed with bioblend, but this seems to be needed since yesterday.

I'm getting bizarre six problems too - https://travis-ci.org/galaxyproject/planemo/jobs/238174042 - I'm not sure what is up or if it is related.

Maybe this https://github.com/pypa/setuptools/pull/1043/files - I think setuptools was broken on PyPI for sometime recently.

Yes, it's fixed in setuptools 36.0.1, see https://github.com/pypa/setuptools/blob/master/CHANGES.rst#v3601

I just need to be more patient :)


The tool does not open

The tool does not open

@adel123-git
Run cat /bin/ghost what it prints?
Also run ls ~/ghost what it prints?


Not working in UIBarButton ?

Hello,
I use your lib' in my project, i want add this play indicator in UIBarButton with init custom view as UIButton. this not working... how could I do?
Thanks

Haven't checked yet. Keep in mind, that CALayers are not affected by Autolayout. You need to update the Frame in layoutSublayersOfLayer / layoutSubviews.

Ok, I have solved my problem to create a file subclassed UIButton and I override layoutSublayersOfLayer / layoutSubviews methods.
Thanks.

Thanks.


Cycling code view destroys formatting

In the demo, paste the first code block from http://addyosmani.com/resources/essentialjsdesignpatterns/book/#mixinpatternjavascript. It looks good enough. Now go to the code view, and back to WYSIWYG view. The formatting will be lost. Related to #103?
Summernote handles this correctly.

I think this was fix'd yesterday on master. I need to make a new beta.

Fix'd on 2.0.0-beta.4 and visible on demo page.

Thanks for the quick fix. The formatting looks much better now.
Only the black background is lost:

Strange, on my Chrome I've got the background :/


#103: Add session example

Implements #103
Adding an example of an auth (session) using tinyhttp and @tinyhttp/session

Codecov Report

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

@@           Coverage Diff           @@
##           master     #105   +/-   ##
=======================================
  Coverage   61.81%   61.81%
=======================================
  Files          44       44
  Lines        1616     1616
  Branches      417      417
=======================================
  Hits          999      999
  Misses        615      615
  Partials        2        2

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 3fdc3d3...344bb63. Read the comment docs.

LTGM, merging

@all-contributors please add @aneeshrelan for example


revert report api to 2020-09-04

After upgrade to v2, getting 403 status for report api.
Looks like the newer version is not ready to use
see https://github.com/amzn/selling-partner-api-docs/issues/646#issuecomment-825232385
also see

These resources provide preview information about the new version of the Reports and Feeds API that will be released in Summer 2021 to help you create or adjust integrations that do not require client-side encryption for reports or feeds as you migrate to SP-API:

https://github.com/amzn/selling-partner-api-docs/issues/563

@xiaoyu-tamu yes, this is being reverted already. Should be released shortly.

@nguyentoanit @moltar
I just realized this PR also revert catalogItems_2020-12-01 to catalogItems_v0
I guess the catalogItems_2020-12-01 version is stable since 2020-12-01.
if the file postfix date is in the future, we should consider it as preview version. otherwise, consider as stable?
e.g.
catalogItems_2020-12-01.md should consider as stable because the version date is in the past
reports_2021-06-30.md should consider as preview because the version date is in the future

https://github.com/ScaleLeap/selling-partner-api-sdk/blob/2792525624f18f61e6c9764cf663966fd1ce561b/gulpfile.ts#L310-L321

@xiaoyu-tamu good point!
Although I don't think we got a definitive answer from SP API team about the deprecation and when the new endpoints go into effect.
I wonder if it is safe to assume that once the date cuts off, then we should upgrade to that endpoint?
Maybe this should be done manually. Because some of these changes can also be breaking. In which case it might be better to group them into one release.
But I also want to balance it with a burden of having to do anything manual here.
Thoughts?

you are right, include them manually is much safer.
Or maybe we could generate all api versions and let developer choose which one they want to use.
e.g.

The amazon doc folder structure
catalogItemsV0.md
catalogItems_2020-12-01.md

// catalogItemsV0.md The first file is always default
class CatalogItemsApiClient {}

// catalogItems_2020-12-01.md
class CatalogItemsApiClient2020_12_01 {}

// catalogItems_2021-06-01.md assume there is another one
class CatalogItemsApiClient2021_06_01 {}

Also there is no breaking changes unless amazon remove the the legacy api

Or maybe we could generate all api versions and let developer choose which one they want to use.

Hm, that's also a good idea!
@nguyentoanit thoughts?

I think that's a good idea as well. Will do them in the next release. 👍

I think that's a good idea as well. Will do them in the next release. 👍

Let's not use underscores in the names, but other than that ok :)

I think that's a good idea as well. Will do them in the next release. 👍

Let's not use underscores in the names, but other than that ok :)

May be CatalogItemsApiClientV20210601?

Ok, but what if it's V0? Will it be VV0 then?

Also will need to document this, otherwise noone will be able to find these classes.
Maybe use typedoc to generate something at least?

Ok, but what if it's V0? Will it be VV0 then?

I don't think they have any release before V0 version. Therefore, V0 will be generated as default name (CatalogItemsApiClient).

@xiaoyu-tamu : All API versions were supported in v3.6.0. 🚀
Generated docs: https://selling-partner-api-sdk.scaleleap.org


Setting the sending engine globally is prone to error

i.e.
nodemailer.sendmail = true;
nodemailer.SMTP = { host: 'localhost' };
We are using nodemailer on a large project for a major UK Newspaper and have a team to 6 developers working on the same project.
All HTML emails had passed the manual QA process and worked in all mail clients on launch.
There was a minor update and a developer required nodemailer for an alert email and set the engine to sendmail. This then made all instances of nodemailer send via that transport. But as sendmail only causes html email problems in Hotmail (https://github.com/andris9/Nodemailer/issues/44) the problem went undetected.
One work around is to call nodemailer.SMTP = { host: 'localhost' }; before each send but that isn't very nice.
On a major version number update it would be great if you considered removing this in favor of per instance setting. I'd be happy to submit pull request if you agree this would be a better solution.
Here is a gist which shows the problem: https://gist.github.com/1205774
Regards
Paul
http://yuiblog.com/blog/2006/06/01/global-domination/

The new v0.3 branch doesn't use global settings anymore but dedicated "transport" objects

Nice.


Add support for other architectures

Currently the dockerfile has amd64 specific lines:
RUN wget -q https://github.com/dapr/dapr/releases/download/v1.0.1/daprd_linux_amd64.tar.gz
RUN tar -zxvf daprd_linux_amd64.tar.gz

This should be replaced with a downloaded script to install the correct dapr runtime for each archtecture.

Can also explore the dapr.io install script, rather than install the binaries manually:
wget -q https://raw.githubusercontent.com/dapr/cli/master/install/install.sh -O - | /bin/bash

from here: https://docs.dapr.io/getting-started/install-dapr-cli/

I looked into the dapr install script. It downloads the CLI, which is unnecessary for production. The script also selects the architecture for download based on uname -m, which doesn't fit use with docker buildx cross builds. So, in #35 I just continue to download the daprd executable.
@phil-d-wilson, have I missed your intention?

@kb2ma not at all - you've hit the nails on their heads. :)


Alignment Bug

What happened?
Hey, while I was exploring the site, I found a bug in the project related to the alignment of the link icon within certain boxes.
Dropping the Screenshot.

You Can see in first image the link icon  is not aligned in 'Random Password Generator' box.
How can we reproduce this bug?
I can fix it so Please @swapnilsparsh assign it to me.
Desktop Information (Optional)
No response
Urgency (Optional)
None

thanks


Win7: conda install leads to UnsatisfiableError

Hi,
trying to install pysoundfile via conda on win7 machine using
conda install -c conda-forge pysoundfile=0.10.2
with no success. I get the message:
(wb36_ml) C:\Users\xxx>conda install -c conda-forge pysoundfile=0.10.2
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: |
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed
UnsatisfiableError: The following specifications were found to be incompatible with each other:
Package cffi conflicts for:
pysoundfile=0.10.2 -> cffi
Package sqlite conflicts for:
python=3.6 -> sqlite[version='>=3.25.3,<4.0a0|>=3.26.0,<4.0a0|>=3.29.0,<4.0a0|>=3.30.1,<4.0a0']
Package libgcc-ng conflicts for:
pysoundfile=0.10.2 -> libgcc-ng[version='>=4.9|>=7.3.0']
Package python_abi conflicts for:
python=3.6 -> python_abi=[build=_cp36m]
Package vs2015_runtime conflicts for:
python=3.6 -> vs2015_runtime[version='>=14.16.27012,<15.0a0']
Package pip conflicts for:
python=3.6 -> pip
Package libsndfile conflicts for:
pysoundfile=0.10.2 -> libsndfile
Package numpy conflicts for:
pysoundfile=0.10.2 -> numpy
Package vc conflicts for:
python=3.6 -> vc[version='14.|>=14,<15.0a0|>=14.1,<15.0a0']
I installed cffi and numpy upfront, so my package list looks like this:
Name                    Version                   Build  Channel
certifi                   2019.11.28               py36_0    conda-forge
cffi                      1.13.2           py36hb32ad35_0    conda-forge
intel-openmp              2020.0                      166
libblas                   3.8.0                    15_mkl    conda-forge
libcblas                  3.8.0                    15_mkl    conda-forge
liblapack                 3.8.0                    15_mkl    conda-forge
mkl                       2020.0                      166
numpy                     1.18.1           py36hc71023c_0    conda-forge
pip                       20.0.2                     py_2    conda-forge
pycparser                 2.19                       py_2    conda-forge
python                    3.6.7             he025d50_1006    conda-forge
setuptools                45.2.0                   py36_0    conda-forge
vc                        14.1                 h0510ff6_4
vs2015_runtime            14.16.27012          hf0eaf9b_1
wheel                     0.34.2                     py_1    conda-forge
wincertstore              0.2                   py36_1003    conda-forge
Well, i don't know how to deal with that conficts. I've also tried to use python=3.7, but with same result.
Do anyone know, about problems on windows and maybe a workaround?
Regards
Thomas

This seems to be a conda-specific error. Does it work with pip?

Sorry can't test pip, because pip is blocked by a proxy.
I'll try via pip as soon as a get a win7 pc running

You don't need to go online to use pip. You can download the wheel and pip install file-you-downloaded.whl.

Hi,
tried to install within a conda env using your suggestion and it seems to work so far.
Not the best way to get it install ,because mixed up conda/pip installs, isn't it?
Thank you


Start calculation with currect values

When the meter readings from the beginning of the day/week/month/quarter/year are not known the calculation will be incorrect by leaving the fields at "0".
Would be great to use these fields only as optional - by empty fields the current values should be used to start the calculation at the day of the setup.
In addition it would be nice to only calculate for example the consumption of the year, when one year was reached (or the startvalue was configured).
Just a nice to have feature, but surely a nice one for new users.

interesting idea, question is how do we want to implement it :)
in basic I provide the settings values, i can image having a button to just adopt the current values to the fields ?

Any news here?
When all the fileds are setup with the current value, the complete measurement of year/month etc is wrong.
So best solution would be to start calculating "today" in standard setup, or to define values from last week/month/year manually, in case they are known in an advanced menu.

still thinking about this one but yea having the current values at initialization make sense


[DEMO ONLY] dataset for object detection prototype

Linked issue(s):
What change does this PR introduce and why?
Please check if the PR fulfills these requirements

[ ] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[ ] Relevant docs have been added / updated

cc @y27choi if this helps kick starting your evaluation on new dataset interfaces for object detection 2d.


Wrong sidebar hover state colors

The hover state colors in sidebar menu for dark theme seems to be using the same color that is being used for light theme.

That was introduced here: https://github.com/snowpackjs/astro-docs/commit/372d20db7f3b5c09ba8b6f06b625fc7b8e195bd5#r53379131

This was me! I've been moving pretty quick as I try to get the site ready for everyone to review this week.
I think I introduced this in a rush, because there wasn't a themed color for this low-opacity accent. There probably should be one!

@natemoo-re I'm really feeling the limits of CSS variables this week, specifically in wanting to customize the opacity/strength of a variable'd color. Any ideas / tips besides just creating a variable for every color you might want?

This was me! I've been moving pretty quick as I try to get the site ready for everyone to review this week.
I think I introduced this in a rush, because there wasn't a themed color for this low-opacity accent. There probably should be one!

I see. Let me know if I can help finish up anything. I know you mentioned you were trying to get a lot of this done by end of week.

Regarding CSS variables. Would transitioning from hex to hsla be a better option? I think it might make it easier to have a base set of colors, and be able to dynamically change the opacity value.

I've got a fix in #61, a good rework of the theme / css variables might be as @jgil-r is suggesting: https://blog.maximeheckel.com/posts/the-power-of-composition-with-css-variables/
I think following the css variable composition in this blog post would work best for us!


feat: implement bytesToIntChunks

implement nullifier tests: https://github.com/anon-aadhaar/anon-aadhaar/blob/8d6d38a52ee67e36286c1d1fa42a8952f8752fa7/packages/circuits/test/aadhaar-verifier.test.ts#L146
now it can correctly compute nullifier for photos
Question: Previously the poseidon hash is using the optimized poseidon but it does not match the circom poseidon
Are we going to keep the optimized poseidon or to use circom poseidon (in this PR)?

LGTM!

Are we going to keep the optimized Poseidon or to use Circom Poseidon (in this PR)?

I think it's fine to use the optimized Poseidon as soon as the whole logic uses the same hash function, in the protocol there are no special requirements in terms of what hash function to use.

I removed the optimized poseidon due to security issue https://github.com/TaceoLabs/noir-poseidon/blob/main/poseidon/README.md#disclaimer


Add optional password to hub

⚠ this would not be "super secure" but simply to prevent snooping by someone.

Add "password" key to FinalObject https://github.com/whyboris/Video-Hub-App/blob/main/interfaces/final-object.interface.ts#L17
Any hub that is opened that has a password will have a popup overlay (modal) where a user needs to enter a password

incorrect password entry just clears the password
it's probably simplest to not let any interactions happen besides minimizing / closing the app 🤔

People over the years requested this 🤷♂ seems easy to implement 😅
note: this means opening the .vha2 file one can easily read the password 👌 ... though to make it more secure we can (salt and then) hash the password and check the hash 🥳

I'm open to the discussion:
Should the (optional!) password be for the app or per hub?

Can I work on this issue?

Hey @minazaher thank you for the offer! Please give this a go 🤝
Please let me know if you have any questions along the way - I'm happy to assist 😁
For the user to add / remove / change the password - let's put the UI in the 4th tab in settings ("Current hub")

I'm on it but got stuck at the try-catch block in the main.ts class/line:216 exactly, I modified the method "openThisDamnFile" header to accept the password as an argument and its used in that block, idk when and where does this function have to be called? also in the same file at line 87 i cant figure out how to deal with it if u can help me i'll be grateful
@whyboris

Can I work on this issue?

Hey @olorunfemisamuel - please feel free to try this issue out 🤝
Let me know if you have any questions along the way 👍

Can you /assign this to me?

Hey @ayyash1 - let me know if you have any questions as you try adding this feature 🤝

Thank you @ayyash1 for the Pull Request https://github.com/whyboris/Video-Hub-App/pull/865 🙇♂ - I intend to merge it when I resume work on VHA this winter 🤝


Implemented set-inclusion indexes

This adds a getIndex function for data sources. When an index is requested for a field, we add it to _indexFields, and when the data source changes, we update the index.
The index for a field x is a histogram of the values of the field x, so that we can easily perform set-inclusion queries.
There are still some issues with streaming data that I need to work out.

OK. I think I have added support for modified tuples. This should be good to merge.

There is an outstanding issue with nested reference support for indices, but I will go ahead and merge, and then make some additional modifications.

Thanks @RussellSprouts and @jheer. Reviewing this plus the subsequent refinement commit (1a692f3), I've got a few questions:

The schema of a data source can change from input to output (e.g., aggregate). Currently, it appears that only fields that appear in the output of a data source can be indexed. This seems reasonable to me, but are there transforms that might also benefit from indexing? If so, I'm not sure the current set up supports that since indexes are only compiled on synchronize which gets called at the end of a propagation. (This is also true for when indices were computed in DataSourceOutput). Do we need an Indexer operator, similar to Collector, that is dynamically inserted into branches of the dataflow?

Related to the above, getIndex and synchronize compute indices from different data sets. getIndex uses this._data which is the input/original data set, which may be different from the tuples seen in the last changeset used by synchronize.

Moreover, by moving index computation to synchronize, can indexes ever fall out of sync? For example, I'm imagining an Encoder uses an index as part of a production rule. Assuming the possible bug above is fixed, the first pulse and call to getIndex computes an index for the first time. Suppose data streams in for subsequent pulses, will the Encoder's call to getIndex not use older index values as synchronize will not be executed until after the Encoder?

A few responses:

Yes, right now only output tuples of a data set can be indexed. I'll leave it to Ryan to describe his design rationale here.
I think you may be looking at outdated code. I revised the original PR to use the values method rather than this._data; see DataSource.js#L149. Hopefully that suffices to fix the issue.
I moved code to synchronize as it seemed cleaner, but that may have been in error. Should we move it back to the output node of the data source pipeline? That would be very easy to do.

Unfortunately, the remainder of the method still refers to this._data. See L158 & 160. I'll let Ryan also weigh in on (3), but my hunch is that it needs to be in DataSourceOutput.

Oops! I created a new feature branch jh/index that fixes the bug and moves the index updates back to DataSourceOutput.
If it is necessary to support indices over intermediate tuples, one might instrument collectors instead. However, the indata function requires a named data set, which in turn implies the output tuples of a data source's pipeline. So perhaps the current design is fine.

One other question: what happens if one requests an index for faceted data? My current understanding is that it will index the "root" nodes representing each facet group. Is that correct?

I'm not sure what the advantages and disadvantages of having the code in synchronize or DataSourceOutput are. For the indata function specifically, I put it in DataSourceOutput (as opposed to Input) because someone may want to do an indata query on a derived field, like the result of a formula transform. For general use, it would be helpful to be able to add indexes at any point in the pipeline. For that, we could implement the indexes as a part of the pipeline, like a transform. I don't know if the pipeline is currently mutable, though.


Update Blog Post “2020-05-hack-the-marine-corps-results-are-in”

Automatically generated by Netlify CMS

Deploy preview for dds-mil ready!
Built with commit b449b0319006acd0954fe617db3f055013e74a56
https://deploy-preview-229--dds-mil.netlify.app


As a home security camera user, I only want useful footage, such as when people are in the frame or movement is detected, to be stored in order to save time when reviewing, and space on the storage device.

@melippmann has closed this issue in the form of the /videos/* directory which is generated when the code runs. Video is stored in that location when motion is detected and when the camera is running.
I think it will be a separate issue to get the motion detection dialed in for the various environments, and to make the camera run when the dashboard is not open.
But as it stands march 29th, we do this when certain conditions are met.

dashboard open
available space on hard drive


Allow developers to skip the build of the examples

Every time the documentation of this project is built, various requests get performed to fetch third party examples.
Multiple runs lead to these third parties blocking for 5 min any additional requests.
We should allow to skip the build of the examples via the Jinja approach we have in other projects.
Note
Before contributing and starting any work, please make sure you agree with the CLA and sign it in your PR.

Steps to Implement:

[ ]  Ensure that sphinx_jinja is included as a dependency in pyproject.toml (it is already present).

[ ] Implement Build Example Environment Variable: https://github.com/ansys/pyansys-geometry/blob/main/doc/source/conf.py#L353

[ ] Update examples.rst: https://github.com/ansys/pyansys-geometry/blob/main/doc/source/index.rst?plain=1#L41
Acceptance Criteria:

[ ] - The build process can successfully skip examples based on the environment variable.

[ ] - The Jinja template is properly integrated and functioning in index.rst.

[ ] - All changes are tested, and documentation is updated accordingly.


when use git bash as the default shell , copy file to windows failed.

Hi team,
I moved this from here: https://github.com/MicrosoftDocs/windowsserverdocs.zh-cn/issues/5
@wwfyde commented 12 hours ago — with docs.microsoft.com
return : /c: /c: Is a directory
two ways : "c/dir/path", "c:/dir/path" can't transfer file successfully!

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

ID: 07417484-bd6d-e6f4-110e-e42c2428a369
Version Independent ID: 844743a6-f67b-421f-9591-13da36b23b14
Content: Installation of OpenSSH For Windows
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_Install_FirstUse.md
GitHub Login: @maertendMSFT

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Terminals cannot be constructed from sequences with skip nodes

The example below shows a program that expects to construct the Header and Body terminals from sequences containing skipped input. However, the program instead given an error like.
1: >ID_8867
   ^
   expected at least one element

I suspect that the implementation is unable to construct terminals from multiple non-contiguous substrings. This should either be fixed, or an error message should be presented that the grammar is malformed.
use "peg"

actor Main
  new create(env: Env) =>
    let input = ">ID_8867\nGATTACA\nACATTAG\n"
    let src = Source.from_string(input, "text")
    match recover val FastaParser().parse(src) end
    | (_, let ast: AST) =>
      env.out.print(recover Printer(ast) end)
    | (let offset: USize, let r: Parser val) =>
      let e = recover val SyntaxError(src, offset, r) end
      env.out.writev(PegFormatError.console(e))
    else
      env.out.print("wat")
    end

primitive FastaParser
  fun apply(): Parser val =>
    recover
      let id = (R('A', 'z') / L("_") / R('0', '9').many1()).many1()
      let header = (L(">") * id * L("\n")).term(Header)
      let body = ((L("A") / L("T") / L("G") / L("C")).many1() * -L("\n")).many().term(Body)
      let records = (header * body).many1()
      records
    end

primitive Header is Label fun text(): String => "Header"
primitive Body is Label fun text(): String => "Body"

I raised my confusion on this point on Zulip so want to add a few details for anyone reading afterwards.
I ended up change the implementation to the following to get around terminals not being able to be constructed with skip nodes.
use "peg"

actor Main
  new create(env: Env) =>
    let input = ">ID_8867\nGATTACA\nACATTAG\n"
    let src = Source.from_string(input, "text")
    match recover val FastaParser().parse(src) end
    | (_, let ast: AST) =>
      env.out.print(recover Printer(ast) end)
    | (let offset: USize, let r: Parser val) =>
      let e = recover val SyntaxError(src, offset, r) end
      env.out.writev(PegFormatError.console(e))
    else
      env.out.print("wat")
    end

primitive FastaParser
  fun apply(): Parser val =>
    recover
      let id = (R('A', 'z') / L("_") / R('0', '9').many1()).many1()
      let header = -L(">") * (id.term(Header)) * -L("\n")
      let body = ((L("A") / L("T") / L("G") / L("C")).many1().term(Body) * -L("\n")).many1()
      let records = (header * body).many1()
      records
    end

primitive Header is Label fun text(): String => "Header"
primitive Body is Label fun text(): String => "Body"

This has the effect of parsing as:
(
  (
    (Header ID_8867)
    (Body GATTACA)
    (Body ACATTAG)
  )
)

When what is desired is:
(
  (
    (Header ID_8867)
    (Body GATTACAACATTAG)
  )
)

As the "Body" of a FASTA record is all sequence lines (until the next Header) with all internal newline characters removed.


update restic prerequisite with min k8s version

Restic DaemonSet pods require shared mounts with mountPropagation feature enabled on the Kubernetes Cluster.  Refer #669
Signed-off-by: kmova<EMAIL_ADDRESS>

My gut feeling is that the presence of the MountPropagation feature is not what makes things work. The default value is None if you don't specify anything else, which would match the previous behavior.
I have seen some propagation issues on Centos with docker-ce 17.03.02, whereas on Ubuntu 16.04 with the same docker version, the problem isn't there. I wonder if something change in the kernel version between GKE 1.9.x and 1.10.x?

@ncdc we're specifying HostToContainer in our restic daemonset (https://github.com/heptio/ark/blob/master/examples/aws/20-restic-daemonset.yaml#L54)

@skriss thanks, my bad (should have looked at the file!)

LGTM, thanks @kmova!


Fixing two minor typos - [closed]

In GitLab by @jegasus on Feb 4, 2023, 21:23
Merges master -> master
There were two tiny little typos in the GM Screen journals. In both cases, there was a break line instead of a new list item, which caused a few items not to have the little bullet icons.

In GitLab by @jegasus on Feb 4, 2023, 21:26
added 1 commit
5ac48bfd - Update 2 files
Compare with previous version

In GitLab by @jegasus on Feb 4, 2023, 21:28
My apologies, I forgot to commit after making the changes. I'm not super familiar with GitLab yet =P


Implement messages and message words.

Ronseal.
Comes with a test suite

Can we have some action on this, please?

Ta