react-native storybook 5.1.0-alpha-34 missing /bin/storybook script

Describe the bug
Use the following storybook addons
"@storybook/addon-knobs": "5.1.0-alpha.34",
"@storybook/addon-ondevice-knobs": "5.1.0-alpha.34",
"@storybook/react-native": "5.1.0-alpha.34",
Simply try and run storybook and you will get the error.
$ storybook start -p 7007 --h <IP_ADDRESS>
/bin/sh: storybook: command not found
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

To Reproduce
Steps to reproduce the behavior:

install

"@storybook/addon-knobs": "5.1.0-alpha.34",
 "@storybook/addon-ondevice-knobs": "5.1.0-alpha.34",
 "@storybook/react-native": "5.1.0-alpha.34",

Go into node_modules/.bin/
Notice that the storybook script is missing.

Expected behavior
Expected to be able to run the storybook server script npm run storybook

Also bumped into this issue.

Please read the migration instructions: https://github.com/storybooks/storybook/blob/next/MIGRATION.md#react-native-server
We still need to get this information back into the main docs, but are busy getting the functionality wrapped up for 5.1.

Thanks, and awesome to have the info here at least. 💪

@nabati is this issue resolved?

Yes!
On Mon, May 27, 2019, 20:13 Benoit Dion<EMAIL_ADDRESS>wrote:

@nabati https://github.com/nabati is this issue resolved?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/storybooks/storybook/issues/6643?email_source=notifications&email_token=AALN7KUSHA57DZMS7XI7Y2LPXQJDNA5CNFSM4HISM3O2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWKHMNQ#issuecomment-496268854,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALN7KSU2ANYPU6SM2EOK4DPXQJDNANCNFSM4HISM3OQ
.


New-MarkdownHelp crops example code

Steps to reproduce
Take https://github.com/AtlassianPS/JiraPS and generate help for the module (commit 1cdc0cb, tag v2.4.4).
New-MarkdownHelp -Module JiraPS -OutputFolder platyPS-docs.
Check the Add-JiraIssueWatcher generated markdown help for example 3.
Expected behavior
Equivalent of
    -------------------------- EXAMPLE 3 --------------------------

    Get-JiraIssue -Query 'project = "TEST" AND created >= -5d' | % { Add-JiraIssueWatcher "fred" }

    This example illustrates adding watcher on all projects which match a given JQL query. It would be best to validate
    the query first to make sure the query returns the expected issues!

Actual behavior
Cropped version

### EXAMPLE 3
```
= -5d' | % { Add-JiraIssueWatcher "fred" }
```

This example illustrates adding watcher on all projects which match a given JQL query.
It would be best to validate the query first to make sure the query returns the expected issues!

Environment data

v0.9.0

I'm also running into issues getting it to format the example section correctly.
I am using the latest module version as we.
It might just be that I don't know how PlatyPS is suppose to export examples (It is just grabbing the first line of the help and that is it)

.EXAMPLE
	$Params = @{
	 	'ComputerName'		= 'TestServer'
		'NTID'				= 'jmccarth'
		'Credential'			= $Credential
		'Test'				= 'Yes'
	}
	Some-Function @Params

ends up being
Params @{

	 	'ComputerName'		= 'TestServer'
		'NTID'				= 'jmccarth'
		'Credential'			= $Credential
		'Test'				= 'Yes'
	}
	Some-Function @Params

@Midacts this is slightly different, see #180

Thanks for the clarification.
I thought that might be the case.

@vors Looks like another help engine bug. Did quick testing and the example is returned as an object with:

introduction = Get-JiraIssue -Query 'project = "TEST" AND created >
code = = -5d' | % { Add-JiraIssueWatcher "fred" }

title and remarks are as expected.
From a PlatyPS point of view the introduction not making it into the generated markdown is an issue however.

No repro


Reimport javac from jdk7u-langtools

Source: https://github.com/openjdk-mirror/jdk7u-langtools/tree/9f03d0c131f6343fee00f5dc5485c047e5cd2033 (jdk7u6-b08)
Manual changes:

Added isSynthetic() method to Symbol.MethodSymbol (see also 2dccff543d45b6e8445526e7d3440b8066a381ac)

This change is backwards incompatible.

@kohsuke With this change and a rebuild of extension-indexer based on the new sorcerer-core, Jenkins Core 1.575+ could successfully be analyzed.

Obsolete, sorcerer is now using JDK 8 javac.


Additional Controls / Instruments

VSI - Vertical Speed Indicator - +/- ft (state)
ASI - Air Speed Indicator - kts (state)
Parking Brake (state) - Sim  starts with it on when on ground and off if in air
Flaps (state)
Vertical Trim (state)
Autopilot FLC - Flight Level Change (set & state)
Throttle % (state & maybe set?)

FLC doesn't appear to be available in SimConnect at the moment. Branching this to another issue to keep it separate.


Added a take a tour guide button

Implemented US09, enhancing the tour guide by adding an additional tour guide button, making it a lot easier to navigate around using the tour guide, as a result knowing how to use the features of the FarmBot. Overall, improving the user experience.

Functionality: Achieved
Readability: Not enough comments
Structure: Follows existing architecture design
Coding Standards: Reasonable branch name, but developer name was not in the pr


Rust Implementation

Hi, I created a library in Rust to create and validate ALTCHA challenges, with an example server. Take a look at the repository: https://github.com/jmic/altcha-lib-rs

Hi and thanks for you contribution! Added to the docs: https://altcha.org/docs/integrations/


fixing os file paths with /Shared/ prefix

In mlflow logging, users that wanted to log into the "/Shared/" file directory had an extra slash in their modified experiment names, leading to Mlflow not recognizing existing experiments, trying to create them, and then claiming they already exist.
This PR accounts for that edge case and removes the extra slash added.

Im a little confused, can you give examples of initial user specified arg and transformed arg before and after this PR as an example?

Im a little confused, can you give examples of initial user specified arg and transformed arg before and after this PR as an example?

@mvpatel2000
Prior to this PR, there was a bug where users who directly wanted to mlflow-log to the /Shared/ workspace would be able to do so after the first run of an experiment name, but would not be able to for following runs of the same experiment name. We found that an extra '/' was prepended to the experiment name (yielding an experiment name of '//Shared/...' after '/' + os.path.join('Users', databricks_username, experiment_name)), leading to get_experiment_by_name not finding the existing experiment, and create_experiment claiming that it already exists (which it does). There are probably ways in which get_experiment_name and create_experiment name handle file directories differently, also contributing to this bug.
In the original code prior to this PR, inputting an experiment name starting with '/Shared/' would throw this error: mlflow.exceptions.RestException: RESOURCE_ALREADY_EXISTS: Node named ___ already exists
Now, mlflow logger correctly logs to the '/Shared/...' folder path as an absolute path. As mentioned above, there are probably some choices we want to make surrounding how we can most intelligently interpret absolute vs relative paths, but this serves as a functioning bandaid solution, because there are only '/Users/' and '/Shared/' under workspaces.


test: convert type tests to vitest syntax

🔗 Linked issue

Related to #58 and https://github.com/nuxt/nuxt/pull/23696
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[x] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Inspired by the type testing in the main nuxt repo, I've moved the type tests out to their own file, and converted them to vitest syntax. This means the // eslint-disable-next-line @typescript-eslint/no-unused-vars comments are no longer requred. It also makes the tests more consistent.
I would imagine there may be conficts with the changes in #58, which I can resolve when that one is merged.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks!

Thanks!

No problem! Please could you publish a new release when possible so that I can integrate the changes into https://github.com/nuxt/nuxt/pull/23696


[Docs] ESP32_WIFI_MGMT_SBUF_NUM is undocumented  (IDFGH-8937)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

WiFi mgmt short buffer number
Range:
from 6 to 32
Default value:
32

The docs are very sparse here.

How big are these buffer?
Where are they allocated?
How much does this value affect perf?
What are they used for?

Thanks.

@chipweinberger
sorry for late reply
We'll add a more detailed description in a later version.

Thanks for reporting, the fix is available at https://github.com/espressif/esp-idf/commit/6673376297b921d438790d195014b860eaf8bb17, and has been back ported to branches until 5.0. Feel free to reopen.


GetUncollapsedLineNumberDifference cache

Partial optimization as I do not know when the cache should be flushed, I suppose every time the content, collapsing or the highlighter changes.
The cache will improve performance during scrolling, arrow key navigation & selection.
New field:
FCollapsedLinesDifferenceCache: array of Integer;

To flush the cache, just set it to nil or SetLength to zero.
Given it's a sparse array, a TDictionnary could theoretically work fine, but the one in Generics.Collections is sluggish and uses a slow hash function, which negates most advantages of the cache... so a specialized dictionary would be required.
Updated method:
  function TBCBaseEditor.GetUncollapsedLineNumberDifference(ALine: Integer): Integer;
  var
    i: Integer;
    mid: Integer;
    LCodeFoldingRangeForLine: TBCEditorCodeFoldingRange;
    LCodeFoldingRange: TBCEditorCodeFoldingRange;
  begin
    LCodeFoldingRangeForLine := CodeFoldingRangeForLine(ALine);
    Result := 0;

    if ALine > High(FCollapsedLinesDifferenceCache) then
      SetLength(FCollapsedLinesDifferenceCache, ALine+1);

    Result := FCollapsedLinesDifferenceCache[ALine];
    if Result >0 then Exit(Result-1);

    for i := 0 to FAllCodeFoldingRanges.AllCount - 1 do
    begin
      LCodeFoldingRange := FAllCodeFoldingRanges[i];
      if LCodeFoldingRange.FromLine < ALine then
      begin
        if (LCodeFoldingRange <> LCodeFoldingRangeForLine) and LCodeFoldingRange.Collapsed and not LCodeFoldingRange.ParentCollapsed then
          Inc(Result, Max(LCodeFoldingRange.CollapsedLines.Count - 1, 0))
      end
      else
        Break;
    end;

    FCollapsedLinesDifferenceCache[ALine] := Result+1;
  end;

I had a little different approach a little while ago where I calculated the difference for every line and GetUncollapsedLineNumberDifference would get the value simply from the array. I left it to simmer because there was some problem... I didn't prepare it in the right places. I will try it again... or use your solution.

Unless I'm mistaken in a large file (where performance matters) the values are needed for only a very small number of the lines, so I assumed that computing them all would be inefficient.
When testing (without a flush and without editing) the cache makes the method disappear from the bottleneck, the few times where a value is not cached appear to not matter.

Unfortunately your algorithm does not work. If you collapse a fold, line numbers stay same.

Yes the cache needs to be purged when editing, collapsing or changing highlighter.

I added purge into CodeFoldingCollapse, CodeFoldingUncollapse, LinesChanged, ClearCodeFolding, CodeFoldingUncollapseLevel, and InitCodeFolding. I hope that's enough.

Maybe you could add a "FlushCodeFolding" method or something like that, call it from all those places, and in its implementation, purge the cache.
This would create a single place where codefolding-related caches could purge themselves, and a single entry point where stuff that affects codefolding would have to call.


Bug: Bad reading of the AWS credentials file

Contact Details
<EMAIL_ADDRESS>Describe bug
When creating an AWS credentials file on Windows 10, the file is created with cr/nl characters at the end of every line.
Then, the oadp-*-aws-registry pod can't read the aws_access_key_id and displays an error message: CreateContainerConfigError (see operator pod error message below).
The same file created on Linux with only nl characters at the end of every line works perfectly fine.
What happened?
Whatever platform [Linux/Microsoft Windows/MacOs] you use to create the credentials, the AWS keys should be read fine.
OADP Version
1.0.1 (Stable)
OpenShift Version
4.10
Velero pod logs
No response
Restic pod logs
No response
Operator pod logs
2022-04-05T14:21:53.588Z        INFO    controller.dataprotectionapplication    Error finding AWS Profile for the supplied AWS credential       {"reconciler group": "oadp.openshift.io", "reconciler kind": "DataProtectionApplication", "name": "dpa-sample", "namespace": "openshift-adp"}
2022-04-05T14:21:53.588Z        INFO    controller.dataprotectionapplication    Error parsing provider secret cloud-credentials for backupstoragelocation openshift-adp/dpa-sample-1    {"reconciler group": "oadp.openshift.io", "reconciler kind": "DataProtectionApplication", "name": "dpa-sample", "namespace": "openshift-adp"}

New issue

[X] This issue is new

Thanks for the report. Tracking on OADP JIRA at https://issues.redhat.com/browse/OADP-463


[0266] New Vehicle data GearStatus

Fixes #1256
This PR is [ready] for review.
Risk
This PR makes [minor] API changes.
Testing Plan
Covered by existing and added unit tests
Unit Tests
New unit tests covering introduced structures were added
Summary
Applied 0266 New Vehicle data GearStatus changes
Changelog
Enhancements
- Extend PRNDL enum

- Add TransmissionType
    - Extend VehicleDataType
    - Implement changes in proper requests responses related to new structs
CLA

[ x ] I have signed the CLA

@mrapitis , @atiwari9 please review the changes

@kostyaBoss I reviewed your PR.  I suggest you make the below changes.
Source code changes:
VehicleDataType.java:

indicate that VEHICLEDATA_PRNDL is deprecated

GearStatus.java:

since all parameters are not mandatory you don't need to construct the object.
So you can delete this code -
public GearStatus(          @NonNull PRNDL userSelectedGear,          @NonNull PRNDL actualGear,          @NonNull TransmissionType transmissionType  ) {      this();      setUserSelectedGear(userSelectedGear);      setActualGear(actualGear);      setTransmissionType(transmissionType);  }

SubscribeVehicleData:

comment should be "...please see GearStatus" instead of Gear above getPrndl

GetVehicleData:

comment should be "...please see GearStatus" instead of Gear above getPrndl

SubscribeVehicleDataResponse:

prndl needs to be deprecated

GetVehicleDataResponse:

prndl needs to be deprecated

OnVehicleData:

prndl needs to be deprecated

TransmissionType:

change the comment     /**
* Convert String to PRNDL
* @param value String
* @return PRNDL
*/
to
/**
* Convert String to TransmissionType
* @param value String
* @return TransmissionType
*/

Unit Test changes:
Test.java:

missing GearStatus RPC

GearStatusTests.java:

unit tests for GearStatus need to be added.

TransmissionTypeTests.java:

I would change TransmissionType enumSport = TransmissionType.valueForString(example); to TransmissionType continuouslyVariable = TransmissionType.valueForString(example);

I would change TransmissionType continuouslyVariable = TransmissionType.valueForString(example); to TransmissionType infinitelyVariable = TransmissionType.valueForString(example);

I would change assert statements from:
assertNotNull("PARK returned null", manual);
assertNotNull("REVERSE returned null", automatic);
assertNotNull("NEUTRAL returned null", semiAutomatic);
assertNotNull("DRIVE returned null", dualClutch);
assertNotNull("SPORT returned null", enumSport);
assertNotNull("LOWGEAR returned null", continuouslyVariable);
assertNotNull("FIRST returned null", electricVariable);
assertNotNull("FIRST returned null", directDrive);
 to:
 assertNotNull("MANUAL returned null", manual);
 assertNotNull("AUTOMATIC returned null", automatic);
 assertNotNull("SEMI_AUTOMATIC returned null", semiAutomatic);
 assertNotNull("DUAL_CLUTCH returned null", dualClutch);
 assertNotNull("CONTINUOUSLY_VARIABLE returned null", enumSport);
 assertNotNull("INFINITELY_VARIABLE returned null", continuouslyVariable);
 assertNotNull("ELECTRIC_VARIABLE returned null", electricVariable);
 assertNotNull("DIRECT_DRIVE returned null", directDrive);

UnsubscribeVehicleDataResponseTest.java:

change JSONObject gearStatus = JsonUtils.readJsonObjectFromJsonObject(parameters, SubscribeVehicleDataResponse.KEY_GEAR_STATUS); to
JSONObject gearStatus = JsonUtils.readJsonObjectFromJsonObject(parameters, UnsubscribeVehicleDataResponse.KEY_GEAR_STATUS);

GetVehicleDataResponseTests.java:

else if (key.equals(OnVehicleData.KEY_GEAR_STATUS)) { change to else if    (key.equals(GetVehicleDataResponse.KEY_GEAR_STATUS)) {

@santhanamk updated according to provided suggestions, please review

@kostyaBoss I reviewed your PR.  It looks good.  I have a few more suggestions:
Unit test changes:
TransmissionTypeTests.java:

I would rename infinitely_variable to infinitelyVariable on line 27
On line 38 it should be assertNotNull("INFINITELY_VARIABLE returned null", infinitelyVariable);

@santhanamk Hi, thanks for feedback. I've updated the PR, please, review the changes

@kostyaBoss code looks good.  Thanks.

@joeygrover , @theresalech Hi! Please review changes in this PR

@kostyaBoss the description for this PR says This PR is [not ready] for review.  Can you please update if this is ready?  Thanks!

@theresalech this PR is ready for Livio review.

@RHenigan Hi! I've made corresponding changes, please, check the latest version. Please, let me know if there anything else I should change

Hello @kostyaBoss, Please focus on this PR #1395
Once this PR is approved other PRs can be updated to match

@RHenigan Hi! Regarding PR #1395, the @bilal-alsharifi has approved it. The problem with "unclear difs" comes from different line comment. I and @vladmu noticed that issue comes from the wrong line endings, files are containing different types of endings and we can't fix it. There were pushed commits to reduce the number of changes and make difs as clear as possible. There were also commits to align this PR with latest version of the dev branch
cc @bilal-alsharifi , @vladmu

Hello @kostyaBoss and @vladmu, thank you for making these changes. After approving the WindowStatus PR (#1396) there are now merge conflicts with this PR. Please resolve the merge conflicts and I can review this PR again.
It is likely that as our team reviews other open PRs we will likely run into merge conflicts again. If you only want to do one at a time and have our team review that one so that you’re not updating a bunch at once, that is fine

@RHenigan let's make it one by one. I've updated PR and resolved conflicts, please check

Hello @kostyaBoss and @vladmu, thank you for resolving the merge conflicts but upon closer inspection there are still large changes in this PR that should not be included please fix these files to minimize the size of the diff.
GetVehicleData.java
GetVehicleDataResponse.java
OnVehicleData.java
SubscribeVehicleData.java
SubscribeVehicleDataResponse.java

UnsubscribeVehicleData.java
UnsubscribeVehicleDataResponse.java
VehicleDataType.java

@RHenigan

there are still large changes in this PR

Fixed. Please review.

@RHenigan applied changes

Hello @kostyaBoss There were 4 that were missed, can you apply those changes as well?
SubscribeVehicleDataResponse.java
SubscribeVehicleData.java
OnVehicleData.java

@kostyaBoss @RHenigan applied missing suggestions. Please review.

Hello @kostyaBoss and @vladmu Thank you for making the changes.
Please tag us on the next PR you would like us to focus on.


Freudenberg - The number of jobs posted on Peviitor and the one poested on the company website isn't the same.

ENVIROMENT: production
URL: https://peviitor.ro/
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open "https://peviitor.ro/" in the browser.
Type "Freudenberg" in the search bar.
Check the number of jobs available for Freudenberg.
Open "https://jobs.freudenberg.com/Freudenberg/ro/?location=RO".
Check the number of jobs displayed.

EXPECTED RESULTS:
The number of jobs posted on "https://peviitor.ro/" and the number of jobs posted on the company website should be the same.
ACTUAL RESULT:
There are 9 jobs posted on Peviitor website while on the company website there are posted 13 available jobs.

Fixed!


Docker Image

Create a docker image that executes saml-to and can take config from the host.
Describe the solution you'd like
https://docs.aws.amazon.com/cli/latest/userguide/getting-started-docker.html
Copy the above pattern for saml-to.  Link above is multi platform.
Describe alternatives you've considered
None
Additional context

49


python3-tensorflow-lite_2.5.0: add missing (metrics) files

Fixes:
Traceback (most recent call last):
  File "/usr/share/doc/python3-tensorflow-lite-demo/example/mnist.py", line 2, in <module>
    import tflite_runtime.interpreter as tflite
  File "/usr/lib/python3.9/site-packages/tflite_runtime/interpreter.py", line 42, in <module>
    from tflite_runtime import metrics_portable as metrics
ImportError: cannot import name 'metrics_portable' from 'tflite_runtime' (/usr/lib/python3.9/site-packages/tflite_runtime/__init__.py)

See:

https://github.com/tensorflow/tensorflow/commit/36917c0d825f1abb49a62869fa469c74739b5818
https://github.com/tensorflow/tensorflow/commit/5f2341e61bc5e5172f521fffd1a0851146891c2a

Signed-off-by: David Abdurachmanov<EMAIL_ADDRESS>

Thanks @davidlt -san !


Small issue with Xcode 7

I just downloaded Chipmunk from here and run example in Xcode 7, as result I see issue in cpSpatialIndex.h: - ../Chipmunk-7.0.1/include/chipmunk/cpSpatialIndex.h:57:5: Unknown command tag name.
/// @private
So, I just changed it to
// @private
and it compiles ok. Xcode 7.1.1

It's fixed in the master branch (I think). I should probably make another tag.

+1, just ran into that error :)


Error: Operation not permitted, rename...

I’m having an issue whereby the first attempt to update or install any of the recent versions will end with an error (see below). If I then install again after the error, it works okay. It looks like it's trying to rename a directory after it has been deleted, or that the operation isn't supported on my OS (Windows 10).
I’m not sure when I first encountered this error. It wasn't occurring in the original 2.0.0 release, I’m pretty sure of that, but I’ve seen it for the last few releases at least.
Version
2.3.0
Steps to reproduce
Install/update any of the recent versions. Using: npm install --save-dev react-router or npm update --save-dev react-router.
Expected Behavior
I’ve shortened the paths shown below, and replaced them with ... to make the output more concise.
E:\...\MyProject>npm install --save-dev react-router
<EMAIL_ADDRESS>E:\...\MyProject
`--<EMAIL_ADDRESS>
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>
Actual Behavior
E:\...\MyProject>npm install --save-dev react-router
npm WARN<EMAIL_ADDRESS>EPERM: operation not permitted, rename 'E:\...\MyProject\node_modules\react-router' -> 'E:\...\MyProject\node_modules\.react-router.DELETE'
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>

This seems like an issue with your local package directory. Try wiping node_modules and reinstalling.

I just ran into this problem. Solved by stopping running the localhost.

@hieverest Thank you, I could not get npm clean cache to work, or the other solutions.  Looks like the react-scripts tie up the NPM modules while its running the dev server.

so what's is the solution about this problem?

@amtili Stop any local processes running the server, then run npm install or npm update.


Add StrongName signing to managed assemblies

Though I understand and agree with the reasoning for authenticode, I have a hard time understanding why you decided to add a strong name to the Xaml Behaviors assemblies!
I might be opening quite a can of worms here (this is a long debated subject) but given that the Xaml Behaviors provides base classes for other libraries to use (such as Cimbalino Toolkit) I see the usage of strong naming as a storm waiting to happen!
As far as I know, the main reason to SN an assembly would be to allow it to be hosted in the GAC, thus providing "side by side" versioning... but the thing is that the GAC is dead!!
Plus, the Xaml Behaviors are targeting Windows 10 apps which completely ignore what the GAC is, so whats the point?

@PedroLamas classic Desktop applications can be strong named so to reference Xaml Behaviors it must be strong named too.
The opposite is not required. Cimbalino does not have to be strong named to reference Xaml Behaviors.

Not to be too blunt, but why do we care about classic Desktop applications? Isn't this library for UWP?

@jozefizso I'm not your following problem: Let's you an app takes Cimbalino Toolkit and Caliburn. Now let's say that Cimbalino uses Xaml Behaviors v1.0 while Caliburn requires v1.1... If they are referencing strongly signed assemblies, you have a serious problem as you can't redirect the assembly bindings in UWP (AFAIK)
But in any case, why do you care about desktop apps? Isn't this for UWP apps only?

Is having strong name really a requirement for OSS nowadays? (considering the pain that it brings w/ binding redirects, etc.)
I thought it was a solved problem with things like StrongNamer and similar. If an enterprise needs to use an OSS library and needs that library to be signed, they can just sign it during the build process.
Why put the burden of strong name to everyone else, because of the few that actually need it?
Worst case, you could follow what some OSS projects do and release two NuGet packages (with different package IDs), one with no strong name (XamlBehaviors), and another one with strong name (XamlBehaviors.StrongNamed)

@brianlagunas wrote:

Authenticode will validate these assemblies, but other malicious assemblies could be strong signed with the key in the repo. I don't see the benefit of checking in the key in the first place. If someone is doing a custom build of the project, they won't have the authenticated version so they will still have to reference their own build which means they could use their own key if they need strong naming.

I think you're underestimating the pain that can be caused when it's not possible to build your own version of an OSS library with the same strong name as the "official" version.  All assemblies that reference the library would need to be rebuilt.  So if you wanted to fix a bug in XamlBehaviors, you would end up needing to also build your own versions of any third-party libraries that depend on it.

@dsplaisted been there... and that's why I'd prefer to get this not strongly signed!

By checking in the key, other assemblies can impersonate this project. I would never recommend checking in the key even for OSS. We do want enterprises to use these behaviors.

@brianlagunas This is a common misconception, but it's incorrect. Strong naming is not a security feature.  Authenticode is the solution for preventing impersonation, Strong Naming is not. Microsoft should Authenticode sign the Official Version.
I won't weigh in on whether or not to SN, but if you do SN and you don't check in the key, you're both not Free Software (because users can't drop in their own binary), and from a practical perspective, you're causing a lot of difficulties to other developers and the .NET Community for no benefit.

@dsplaisted Good point.  Didn't consider that.  Signing adds a lot of extra headaches in a number of different scenarios.
@paulcbetts I disagree with your perspective of not being free software, but you also make a good point.

@brianlagunas It's not just my perspective, it's the Free Software Foundation's perspective: https://en.wikipedia.org/wiki/Tivoization - when you hide your SN key, users can no longer compile a version of the library that works with anyone else's binaries.

@paulcbetts Well, I still don't agree with that.  Just because you can't compile with an SN so that it works with anyone's binaries doesn't mean the software isn't free.    You can still download the code and use it as you see fit freely.

I'm trying to compose a complex Pull request, but strong names make compiling, design time and pretty much everything very hard. :(

@vbandi I'm still struggling with what are we trying to fix by strong naming these assemblies...

  I can't even get a build to pass:
Severity	Code	Description	Project	File	Line	Suppression State
Error		Loading assembly "C:\Users\username\Documents\GitHub\XamlBehaviors\out\BehaviorsSDKManaged\bin\AnyCPU\Release\Microsoft.Xaml.Interactivity.dll" failed. System.IO.FileLoadException: Could not load file or assembly 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. Strong name validation failed. (Exception from HRESULT: 0x8013141A)
File name: 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' ---> System.Security.SecurityException: Strong name validation failed. (Exception from HRESULT: 0x8013141A)
The Zone of the assembly that failed was:
MyComputer
at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
at System.Reflection.RuntimeAssembly.nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
at System.Reflection.RuntimeAssembly.InternalLoadFrom(String assemblyFile, Evidence securityEvidence, Byte[] hashValue, AssemblyHashAlgorithm hashAlgorithm, Boolean forIntrospection, Boolean suppressSecurityChecks, StackCrawlMark& stackMark)
at System.Reflection.Assembly.UnsafeLoadFrom(String assemblyFile)
at Microsoft.Build.Tasks.ProcessResourceFiles.ReadAssemblyResources(String name, String outFileOrDir)	XAMLBehaviorsSample

So what's the result here. Back to useful or stay the path?

This change is causing issues with Template 10. Many projects are broken (although there is a workaround, it requires turning off design mode data) as a result of this change.
Could you consider reverting this change? There are other ways of increasing security that don't involve breaking countless apps and libraries.

I'm confused. Do you want strong-name signing on or off? The latest release of the XAML Behaviors (1.1.0) has strong-name signing removed.

 Something in Behaviors 1.1.0 is causing issues for Template 10 apps, and it's a strong named signing error.

@callummoffat The latest version of Behaviors (1.1.0) already has strong name signing turned off. Initially, there was an issue with Template 10 not working with the new version, but that has since been fixed. Is your version of Template 10 the latest?

I've got the latest (non-preview) version available.

The problem appears to be that Template 10 turned on strong naming in the latest release, while Behaviors has turned it off.

The problem appears to be that Template 10 turned on strong naming in the latest release

Then I'd say that this is an issue more related with Template 10 than with the Behaviors! @JerryNixon should be able to help with this! :)

Template 10 also is no longer strong-named.

@JerryNixon

Nerd

I do try! :)


Implementing an initial key-value store engine

1- MemTable is implemented using a multimap, so that multiple elements can have the same key, each MemTable can store up to 4 key-value pairs, then it is flushed into SSTable on the disk.
2- Each SSTable contains multiple files, the data file is only considered for now, files are written/read by using I/O System Calls, All files are created in the current directory.
Note: key and value shouldn't exceed 9 characters.
To Do:
1- Instead of flushing data into SSTable after having 4 key-value pairs in the MemTable, calculate the number of bytes allocated by the MemTable, and flush its content according to it.
2- Key and value length shouldn't be limited.
3- Storing SSTable files outside the current directory.
4- Taking little-endian, and big-endian into consideration.
5- Using Google Tests instead of manual testing.
Resolves #8

Task linked: CU-1uvr402 Implement a Persistent Key Value Store Database Engine with efficient Writes


It is possible to use another Cryptocurrency, rather than XMR  Monero ?

It would be great if you add other Cryptocurrencies feature like Bitcoin, Ethereum or any other.

No. Different algorithm. There's no JavaScript miner for them.

Actually, you can use any currency based on CryptoNight.
So far users have been successful mining Monero (XMR) and Electroneun (ETN).
You can mine XMR on pool.supportxmr.com:
coin-hive <XMR-ADDRESS> --pool-host=pool.supportxmr.com --pool-port=3333

You can mine ETN on uspool.electroneum.com:
coin-hive <ETN-ADDRESS> --pool-host=uspool.electroneum.com --pool-port=3333

I added an example to the readme: https://github.com/cazala/coin-hive#can-i-mine-other-cryptocurrency-than-monero-xmr


test issue

test issue

This issue has been mentioned on Netdata. There might be relevant details there:
https://netdata.trydiscourse.com/t/i-do-a-test-test-test-test-etesaasdfhklii77878/20/3


Add support to specify client cert and key files for authentication (kubelet --client-ca-file)

If the kubelet --client-ca-file option is used to require a client certificate, then kubeletctl needs a way to specify client cert and key files (analogous to curl --cert and --key arguments).

Acually, I see the support in 4948506b92a0337fe37405ecef64f1bdf6b9cc9f, so I'll give that a try. Thank you!

Hey, I am planning also to add certificates without kubeconfig.
I will update once done.

I added support also for certificates files, you can use it like that:
kubeletctl.exe pods -s <node_ip> --cacert /etc/kubernetes/pki/ca.crt --cert /var/lib/kubelet/pki/kubelet-client-current.pem --key /var/lib/kubelet/pki/kubelet-client-current.pem


Feature request - store split pane state in local storage with provided key

It would be great to be able to remember the state of split panes in LocalStorage so that next time when app is reopened it would be in the same state. We're currently using https://github.com/wannabegeek/ng2-split-pane in our project and it has that feature it's pretty convenient. But that module is not performance friendly and does not have option to have multiple panes inside split, so guys did some nasty workarounds having a splitpane inside one side of the other splitpane which I personally don't really like because it's not working correctly. So now I am looking towards this module, which looks very promising so far.
In short:
Please, add input to split container [localStorageKey], put a state of pane to LocalStorage at the end of each resizing. When split container is initialized load its state from LocalStorage and validate it whether it fits together with what is being initialized, then use it as starting point.

hi @BlindDespair
It's a bit out of library scope but why not...
But it will not be soon (no time currently) so you can do it yourself easily like demo app example ;)
https://bertrandg.github.io/angular-split/#/examples/workspace-localstorage
https://github.com/bertrandg/angular-split/blob/master/src_app/app/component/examples/workspaceLocalstorage.route.component.ts

Thank you for fast response, that actually looks good enough and with right abstraction can be considered as a solution to go. I will suggest it to my team.

I agree with @bertrandg that this is a bit out of scope, and it's easy to implement following the demo.
Considering @BlindDespair is happy with the solution, I'm closing this issue.


:colorscheme oxocarbon only works once

If I use oxocarbon, another colorscheme and oxocarbon again, it fails. Eg.:
vim.cmd("colorscheme oxocarbon")
vim.cmd("colorscheme default")
vim.cmd("colorscheme oxocarbon")

It also happens when I :PackerCompile.

Thanks, I'll look into it

Should be fixed now


Python unstable 2021-11-25

Motivation for this change
Python update of minor versions.
There seemed to be excessive x86_64-linux hydra capacity. So may as well.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Should we include https://github.com/NixOS/nixpkgs/pull/147524?

There's also https://github.com/NixOS/nixpkgs/pull/145807 but I don't think we want that just yet?

@fabaff can you avoid force pushing to the branch when not necessary.
You deleted a few of my commits

@jonringer, sorry, my bad.

I went for twentemilieu and libcst too. Looks like this was the start of my questionable actions.

I will fix the whitespace issue in ce26888 when I'm sure that nobody else is working.

https://github.com/NixOS/nixpkgs/runs/4360946550?check_suite_focus=true
Sorry about the spurious output in addition to the correct whitespace error in the editorconfig check , it's since been reverted.

@fabaff I fixed the extra whitespace

Webssh should be moved to all-packages.nix.

waiting for ofborg to go green

So no #147524?

1 comment

@ofborg eval

Looks like that matplotlib is missing packaging which became a requirement in 3.5.0.
Do we want to fix that here or later?

Looks like that matplotlib is missing packaging which became a requirement in 3.5.0.
Do we want to fix that here or later?

Here, doing the final touches before merging to staging

staging and staging-next fail to build bpython:
$ nix build -f. python3Packages.bpython -L
...
python3.9-bpython> unpacking source archive /nix/store/9d0qp9mpxsvlxs9wkqvmbsyb02vpx7ff-bpython-0.22.1.tar.gz
python3.9-bpython> source root is bpython-0.22.1
python3.9-bpython> setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file bpython-0.22.1/setup.cfg
python3.9-bpython> patching sources
python3.9-bpython> applying patch /nix/store/vz4q2sd2g43ham91p2f5sal27yikks7v-clipboard-make-which-substitutable.patch
python3.9-bpython> can't find file to patch at input line 16
python3.9-bpython> Perhaps you used the wrong -p or --strip option?

Bisect landed on  c064af6e4a3786baa4ce1d294734cd23a90eceb0 python3Packages.bpython: 0.21 -> 0.22.1.

Does python3Packages.grpcio-status fetch for you? On staging and staging-next it fails for me as:
$ nix build -f. python3Packages.grpcio-status -L
grpcio-status> trying https://files.pythonhosted.org/packages/source/g/grpcio-status/grpcio-status-1.43.0.tar.gz
grpcio-status>   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
grpcio-status>                                  Dload  Upload   Total   Spent    Left  Speed
grpcio-status>   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
grpcio-status> 100  8686  100  8686    0     0  54978      0 --:--:-- --:--:-- --:--:-- 54978
error: hash mismatch in fixed-output derivation '/nix/store/3pajxzh43idqywpp65gqcs72i17mix0d-grpcio-status-1.43.0.tar.gz.drv':
         specified: sha256-JVM8TWXX1ROmOPDEuIsrZnAOO/Q+aZWlWjvRXsC3eQI=
            got:    sha256-IXWQBvNqf/v/GH1BkfQRjActiqn6aCOhGq14QqPGzNA=
error: 1 dependencies of derivation '/nix/store/nl5jvlc1wygk4bnpiqmdxwhyfn6icvkg-python3.9-grpcio-status-1.43.0.drv' failed to build

Does python3Packages.grpcio-status fetch for you? On staging and staging-next it fails for me as:
$ nix build -f. python3Packages.grpcio-status -L
grpcio-status> trying https://files.pythonhosted.org/packages/source/g/grpcio-status/grpcio-status-1.43.0.tar.gz
grpcio-status>   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
grpcio-status>                                  Dload  Upload   Total   Spent    Left  Speed
grpcio-status>   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
grpcio-status> 100  8686  100  8686    0     0  54978      0 --:--:-- --:--:-- --:--:-- 54978
error: hash mismatch in fixed-output derivation '/nix/store/3pajxzh43idqywpp65gqcs72i17mix0d-grpcio-status-1.43.0.tar.gz.drv':
         specified: sha256-JVM8TWXX1ROmOPDEuIsrZnAOO/Q+aZWlWjvRXsC3eQI=
            got:    sha256-IXWQBvNqf/v/GH1BkfQRjActiqn6aCOhGq14QqPGzNA=
error: 1 dependencies of derivation '/nix/store/nl5jvlc1wygk4bnpiqmdxwhyfn6icvkg-python3.9-grpcio-status-1.43.0.drv' failed to build

it inherits from grpc, so unless there's an update for that packge as well, just updating the grpc package will fail

#151666


Add conditional check for debug_assertions

Currently running cargo test --release blows up because of the tests that panic.  This PR adds a conditional check debug_assertions which causes those tests to not be run in release mode.  Besides fixing cargo test --release this PR sets the stage for a larger PR to test arithmetic tests that are unchecked (will overflow in release mode instead of panic) started here https://github.com/rust-bitcoin/rust-bitcoin/pull/2436.  Also I think we ought to add cargo test --release to CI.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.068%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19386

Relevant Lines:
23060

💛  - Coveralls


Cannot import so module

Cannot import compiled SO module fmodule in the notebook. In the old VS-CODE Jupyter notebook extension it works just fine, therefore I think it is an issue with this new jupyter extension.
import sys
sys.path.append('.')
import fmodule

ModuleNotFoundError: No module named 'fmodule'
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
 in
----> 1 import fmodule
ModuleNotFoundError: No module named 'fmodule'

Hey @muser840 sorry for the trouble here. Just to clarify are you saying that you encountered this problem after upgrading to a newer version of the Jupyter extension? If so, which version did this work on, and which version did this stop working on? Thanks!

Hey @muser840 sorry for the trouble here. Just to clarify are you saying that you encountered this problem after upgrading to a newer version of the Jupyter extension? If so, which version did this work on, and which version did this stop working on? Thanks!

Sorry if I wasn't clear... It's a bit different what I was trying to say. I have in parallel VS Code with Python extension that can open Jupyter notebooks (v2020.12.424452561) and VS Code Insiders with vscode-jupyter (v2020.12.414227025).
I placed shared object module fmodule.cpython-38-x86_64-linux-gnu.so into the folder, then open vscode/vscode-insiders in the same folder. With old vscode Python extension it imports just fine, while with vscode-jupyter under VS Code Insiders it can't find module. Therefore I conclude that something is wrong with "vscode-jupyter" extension.

I think I have the same problem. Here is a minimal example to reproduce it:

Create a new folder and two files inside of it SomeModule.py and notebook.ipynb.
SomeModule.py contains only the line test = 42.
notebook.ipynb contains three cells:

import numpy
from SomeModule import test
test * test

Opening the notebook with Jupyter and executing the cells works without any problems:
numpy is imported as well as test and we can multiply test to itself.
However, if we open the notebook in vscode-jupyter (which is connected to the same Jupyter Server):

numpy is imported without any problems.
Executing the second cell raises the error

ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-2-2f8b85fb3bb7> in <module>
----> 1 from SomeModule import test

ModuleNotFoundError: No module named 'SomeModule'

I suspect this is only a minor oversight from my side (do I have to put something special in a __init__.py file?),
however, I am wondering why  Jupyter and vscode-jupyter behave differently.

@AlexanderZeilmann I'm not reproing your issue? Can you try upgrading to our latest bits. It was likely a problem with the current working directory for the kernel when started. We did fix an issue related to this in the last release.
If it still repros can you verify:

You're not using a remote jupyter URI (you're letting us start the kernel)
What does pwd show? It should show the current directory of the notebook.

@muser840 can you try again as well?

@muser840 can you try again as well?

Now seems all is okay. Thanks!

@rchiodo I can sadly still reproduce the issue.
I am on version v2021.2.576440691 of the extension which should be the current version.

You're not using a remote jupyter URI (you're letting us start the kernel)

I am using a jupyter URI for some server on which jupyter is running.
Letting vscode-jupyter start the kernel on localhost is only working very rarely, but I think this is a different problem.
Because of the higher computing power of the server, I want to use the jupyter installed on the server

What does pwd show? It should show the current directory of the notebook.

It prints the location of my home directory on the server. This is not where the files are located.
My files are stored locally on my laptop.
I can change the directory to some other location on the server, however, I don't know how to change it to my local machine.
I'm not even sure if the way I am trying to use this extension (local files, remote jupyter) is the intended way.

I just tried something different:
When I connect to the server using the remote ssh extension and then open a jupyter notebook using vscode-jupyter, everything is working.

It prints the location of my home directory on the server. This is not where the files are located.
My files are stored locally on my laptop.
I can change the directory to some other location on the server, however, I don't know how to change it to my local machine.

If you're using remote, it cannot load any of the local files. We do not copy files to the remote server nor do we control the working directory of the remote server.
Your best bet (as you found) is to use Remote SSH instead. That will then run the server and the files on the remote machine. Of course you then have to put your files on the remote machine.

If you're using remote, it cannot load any of the local files. We do not copy files to the remote server nor do we control the working directory of the remote server.

Ahh, I understand, thank you for the clarification 😃
I don't know, why I assumed that I can use local files and remote kernel 🙈


Not working with Fiat 600e

Hello,
unfortuantly it doesnt work with the Fiat 600e.
If you are working on this?
Thank you for the Info
alpja

It's working with a 600e. I've running it since Jan/24 without any problem.

Since last week ist works after an OTA Update from Fiat.


BUG keyword Having not support in stringify

<EMAIL_ADDRESS>has been released to fix this issue.


Package request: Immich

Project description
A self hosted alternative to google photos.
Metadata

homepage URL: https://immich.app/
source URL: https://github.com/immich-app/immich
license: MIT
platforms: Preferred unix-based operating system (Ubuntu, Debian, MacOS, etc) (https://immich.app/docs/install/requirements)

Add a :+1: reaction to issues you find important.

Duplicate of #267869

woops no idea how i missed that, thanks :+1:


Compressed cubemap

ok, did an initial pass of looking at the code.  looks good to me so far.  haven't run it yet.

good catch on the file offset issue.  i compiled and ran the cubemap app.  looks good.  anyone else have any comments?

Just missed two headers in new files. Tried and everything is fine.

added headers

Sorry, I missed one more thing. Please add the DCO signature into pull request description and then I will do the merge.

ok, is this good to go?  or did you want to fix the issue with the cubemap sample in mono mode first?

This is good to go. I will fix the mono mode in a separate patch. Thanks!


Update finagle-core, util-core to 19.11.0

Updates

com.twitter:finagle-core
com.twitter:util-core

from 19.10.0 to 19.11.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.twitter" } ]

labels: semver-minor

Codecov Report

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

@@           Coverage Diff           @@
##           master     #178   +/-   ##
=======================================
  Coverage   93.98%   93.98%
=======================================
  Files           5        5
  Lines         133      133
  Branches        3        3
=======================================
  Hits          125      125
  Misses          8        8

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 101c622...d434e6a. Read the comment docs.


Publishing v0.5.0-pre.7

Updating to be in sync with RFC 9497 (no non-test changes, just updating test vector format and comments)

@daxpedda I am planning to release v0.5 (not pre-release) since the RFC has been finalized. Any other updates you wanted to see before 0.5 goes out?

@kevinlewi I don't believe so, there is a new Rust Crypto release cycle soon, e.g. elliptic-curve v0.14, but I don't see why we can't do that in the next voprf release.


fixed odia keyboard layout issues

Description
optimized odia keyboard layout
Checks

[ ] Tests ( npm run test ) are passing

Thanks for the contribution @sonukuldeep!
The Odia layout is now available in the latest version of simple-keyboard-layouts.
Regards,
Francisco Hodge


hhvm.sock and docker environment

First added a sed command to Dockerfile to do not clear docker environment in order to pass it to php.
Second change hhvm socket file config in conf/supervisord.conf in order to make coherence to defined in sites/default.vhost

Deleted 1 commit to make php-fpm by default


ImportError: cannot import name 'Mapping' from 'collections'

I'm running the first example on the documentation. The result is this error:
Traceback (most recent call last):
  File "/Users/vk0604/essence/rumps_nmore/rumps_test.py", line 1, in <module>
    import rumps
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/rumps/__init__.py", line 25, in <module>
    from .rumps import (separator, debug_mode, alert, notification, application_support, timers, quit_application, timer,
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/rumps/rumps.py", line 33, in <module>
    from collections import Mapping, Iterable
ImportError: cannot import name 'Mapping' from 'collections' (/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/collections/__init__.py)

Python version 3.10.0
OSX 10.14.6
Does rumps.py need updating? The issue seems to occur with various packages.
Any wisdom?

Replacing
from collections import Mapping, Iterable
with
from collections.abc import Mapping, Iterable, MutableMapping
in rumps.py did the trick.


How to train dataset without camera data?

I want to train the DeepRoute Open Dataset. Here is the website: https://gas.graviti.cn/dataset/graviti-open-dataset/DeepRoute
There is no camera data in the dataset, only lidar data. It is different with KITTI.

Most of detectors here doesn't use camera data. You can check PointPillars of Second for example.

Thank @filaPro for your kind help! And @HUI11126 thank you for your interest in our work. You can definitely train a dataset without camera data, but you may need to slightly modify the data pre-processing scripts we provided. Please refer to #429 for more details.

OK, thank you for your reply. @Wuziyi616 @filaPro  I will try.


Change LICENSE from MIT to Apache 2

All the commits except for ones from a contributor are from Cybozu.

Hi, @inductor .
We are willing to change the license of MOCO from MIT to Apache 2 to clarify terms and conditions.
Your commits were contributed to MOCO under the MIT license because of GitHub's terms and conditions.
Would you agree with changing the license of your code to Apache 2?

Ugh...  got merged by accident.  Sorry.
I should have made this a draft.

Got confirmation from @inductor via Twitter.  Thank you!


nixos/networkd: add new Network section options

Description of changes
Adds IPv4LLStartAddress, IPv4ReversePathFilter, IPv4ReversePathFilter, IPv4RouteLocalnet, and IPv4RouteLocalnet to systemd.network.networks.<name>.networkConfig.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Ran nixosTests.systemd-networkd on nixos-24.05 branch because the test fails on the curent master.

I think this meets the criteria for a 24.05 backport also.


REPL returns <undefined value> instead of failing, when picking from maps with unbounded (Int) codomains

val m = setOfMaps(1.to(5), Int)

fails with Error: Infinite set Int is non-enumerable, as expected, however
var x: int

action foo = {
    nondet y = oneOf(setOfMaps(1.to(5), Int))
    x' = y.get(1)
}

returns <undefined value> when attempting to execute foo.

I thought that it was easier to fix in the simulator. It is definitely fixable, but not as easy as I expected.


Authorization violation error and maxReconnectAttempts issue

Authorization violation error is emitted every second time when reconnecting. If I set maxReconnectAttempts to 0. No error event received then.

@valichek likely then you have an authorization validation. What does your server configuration and client configuration look like? Setting maxReconnectAttemts to 0, effectively prevents the client from reconnecting if it fails to connect the first time.

@aricart
these are the only parameters for client I have:
{
  token: "",
  maxReconnectAttempts: 0,
  preserveBuffers: true
}

Server closes connection right after sending Authorization violation error to the client. So if no reconnect - no error emitted from client.

The problem is that client doesn't emit Authorization violation if maxReconnectAttemts = 0. I can get it only when maxReconnectAttemts > 0 But again it is emitted not every time when received from server.

I am not able to reproduce your issue. Do you have a cluster? I get one notification
'use strict';

var NATS = require('../');

var nc = NATS.connect({maxReconnectAttempts: 0, token: 'b', preserveBuffers: true});

nc.on('error', function(e) {
    console.error('error', e);
});

nc.on('created', (id) => console.log('created connection', id));
nc.on('disconnect', () => console.log('disconnected'));
nc.on('reconnecting', () => console.log('reconnecting'));
nc.on('close', () => console.log('close'));
nc.on('connect', () => console.log('initial connect'));

If I run this:
[albertoricart@barsoom:~]$ gnatsd -auth 'foo'
[6073] 2017/11/09 09:47:22.479713 [INF] Starting nats-server version 1.0.4
[6073] 2017/11/09 09:47:22.479859 [INF] Listening for client connections on <IP_ADDRESS>:4222
[6073] 2017/11/09 09:47:22.479866 [INF] Server is ready
[6073] 2017/11/09 09:47:37.308459 [ERR] <IP_ADDRESS>:61200 - cid:1 - Authorization Error

I get, one error every time.
[albertoricart@barsoom:~/go/src/git...e-nats/examples]$ node connecthandlers.js
error { NatsError: 'Authorization Violation'
    at Client.processErr (/Users/albertoricart/go/src/github.com/nats-io/node-nats/lib/nats.js:1054:28)
    at Client.processInbound (/Users/albertoricart/go/src/github.com/nats-io/node-nats/lib/nats.js:827:28)
    at Socket.<anonymous> (/Users/albertoricart/go/src/github.com/nats-io/node-nats/lib/nats.js:501:16)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:172:18)
    at Socket.Readable.push (_stream_readable.js:130:10)
    at TCP.onread (net.js:535:20)
  name: 'NatsError',
  message: '\'Authorization Violation\'',
  code: 'NATS_PROTOCOL_ERR',
  chainedError: undefined }
disconnected
close


Inconsistency in text regarding PM.Current

Hi,
I believe there's small inconsistency in PM spec text.
Quotes from the spec:

1 – xPMMASK and xPMBASE registers can be modified by the same privilege level

However, higher privileged code can enable PM.Current bit for lower privileged code. In such scenario, current privilege code has a possibility to self-manage its own configuration of PM bits.

If I get the idea right, PM.Current=1 allows to modify xPMMASK, xPMBASE and xMTE registers. If this assumption is right, could we please update the text?
Another thing that might be worth mentioning:
I believe we might want to add a note, that M-mode could update any register it wants, despite PM.Current=0. This is pretty obvious, but may be it's still good idea to state clearly in the text?
Thanks

This issue refers to an old version of the spec and the new version no longer supports the PM.Current bit. I therefore believe this issue no longer applies and will close it for now.


Don't initialize sample viewer with a sample

Description
This PR makes it so the sample viewer no longer starts with an initial sample loaded in the background. Therefore the live sample view, description view, and source code view, all start disabled as well.
Type of change

[x] Sample viewer enhancement

Platforms tested on:

[ ] Windows
[ ] Android
[ ] Linux
[x] macOS
[ ] iOS

Should we have these disabled or hidden? I'm not sure. Will removing them affect automation in any way?

@tanneryould good question. Hidden would probably be better. I'm not sure if this would affect automation. I assume not, since when a sample is opened, it should behave as it did before. I doubt the automation would try to check description immediately upon opening, but could be wrong. Please check with Harini and Kathleen

LGTM. Please confirm if you've tested all the different scenarios. For example, this change also makes it so no category is selected - if you no longer have a category selected by default, and then straight away choose a featured sample or use search but never select a category, does anything blow up?

I've tested the following in both C++ and QML, and no issues seem to arise

Open a featured sample
Open a sample from search
Open a sample from a category
Open a sample requiring authentication
Open a sample that requires offline data
Download all offline data and interrupt the downloads
Download all offline data to completion

good question. Hidden would probably be better. I'm not sure if this would affect automation. I assume not, since when a sample is opened, it should behave as it did before. I doubt the automation would try to check description immediately upon opening, but could be wrong. Please check with Harini and Kathleen

According to Kathleen, "automation doesn't check for description upon opening, so it should be fine."

@ldanzinger I had to also null the height of the menu items when hiding them, otherwise we get the result in the first picture. The latter two pictures are how the sample viewer behaves after this most recent commit.


Create Masks Using RectangleLabels and Export to COCO Format

Is your feature request related to a problem? Please describe.
I am unable to train my Mask-RCNN using bounding box annotations created in Label Studio. This is because there are no masks created when using the RectangleLabels to annotate. There is also no mask export happening when using COCO export.
Describe the solution you'd like
I previously used a different tool called COCO Annotator that allowed me to annotate using bounding boxes and export annotations that had both bboxes and segmentation components of the COCO data format. Label studio exports do not have segmentation components when exporting in COCO format.
Describe alternatives you've considered
My alternative is at this point to switch to a Faster-RCNN that doesnt utilize masking. It should be fine but I would like to be able to do either.
Additional context
Annotation produced using label-studio:

Annotation produced using COCO Annotator:

This feature would be very helpful for work I am trying to do too (training a Mask-RCNN model). Are there any alternatives to generating both bounding boxes and segmentation masks for training?

What should be in segmentation in case of bboxes?

You can calculate the bounding boxes by having the min and max of the coordinates from the masks.

@kodecreer do you have examples with similar coco exports?

The same process from the PyTorch object detection tutorials if you are able to extract just each mask from the image into a black and white photo like traditional Mask-R CNN
pos = np.where(mask)
xmin = np.min(pos[1])
xmax = np.max(pos[1])
ymin = np.min(pos[0])
ymax = np.max(pos[0])
boxes.append([xmin, ymin, xmax, ymax])

@kodecreer Maybe you could contribute this?

Have you solved the problem?


zoneminder: Support excluding archived events

Description:
ZoneMinder supports archiving camera events and sensor.zoneminder currently includes archived events in the event count for each camera. For my workflow, I only want to see the count of "untriaged" events (ones I haven't reviewed and either deleted or archived) so having the event count exclude archived events is useful at least for me.  I suspect other people (who archive) would also prefer to exclude the archived events by default so I'm making it the default. There is a somewhat related quote in ZM docs: "In general you’ll probably do most filtering on un-archived events."
Related issue (if applicable): N/A
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#1455
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: zoneminder
    include_archived: False

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Never even occurred to me when writing it that someone may want to do that (i normally just delete everything).  I gave it a test and it works perfectly.
LGTM

Yeah, everyone has different workflows but at least this change shouldn't hinder people who decide to not use the "archive" option.

Is this waiting on @fabaff's review then?

@mnoorenberghe yea most likely needs one of the core guys to review / merge it.  I am not one of those people :)

Looks good ! 🐬


Add Doxygen documentation

It would be nice if you could add Doxygen documentation.
For reference, you can see the TheAlgorithms/C repository.
I can also help with this if required/needed (I'm very good with that stuff).
IMHO, this will make the code much more understandable, clean and better. 🙂

That's a good idea! But I have to tell you that the current situation of Square language is not stable, some features can not run well, so too much code needs to be modified.
I will spare more time to improve the features of square. My goal is to release version 1.0 before 2021. When Square releases version 1.0, you are welcome to contribute Doxygen documentation!😀


Link to Text Prefab in MRTK is broken

the link to Text Prefab in MRTK (bottom of article) is broken.

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

ID: 8f9f9ed3-b6cd-2a1c-c106-fd719dbfb624
Version Independent ID: 507bf69d-3d81-4ac1-5106-c677e29ce7f0
Content: Text in Unity - Mixed Reality
Content Source: mixed-reality-docs/text-in-unity.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @cre8ivepark
Microsoft Alias: dongpark

Thanks @rlbogue!


Your Partial Classes Documentation doesn't show how to use [Parameter] tag

I love that I can move code to partial class now, but I can't find any information about what namespace to add to make a property with a [Parameter] tag be available in the partial class.
It isn't obvious what namespace this is.
Thanks,

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

ID: be4c4d0c-149c-bde1-c7fa-f082916390ca
Version Independent ID: 9d945996-ad02-bd4b-58ea-c465cc5cc74c
Content: Create and use ASP.NET Core Razor components
Content Source: aspnetcore/blazor/components.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @DataJuggler ... The namespace rules are in Import components section with introductory remarks in the Component classes section (last paragraph). The point on using custom folders is also mentioned in the Import components section in a sentence under the bullet list).
Most authored components will follow the 3rd bullet point in the Import components section, which is why the example shown in the Partial class support section has the namespace BlazorApp.Pages. It goes by the folder where the component exists.
If the currentCount field is a parameter in the example, the parameter's fully qualified reference is the namespace, class name, and parameter name: BlazorApp.Pages.Counter.CurrentCount. That would the same construction regardless of whether the parameter is located in the partial class (Counter) or in an @code block in the Counter.razor file ... they have the same namespace (BlazorApp.Pages) and class (Counter).
What I'd like to do for this (thus far) is state explicitly in the Component classes section (last paragraph) the typical scenario, which is essentially what the 3rd bullet point states in the Import components section. The Component classes paragraph launches right into using a custom folder without first mentioning that the component namespace is derived from its location (folder).
Did you have a special scenario not covered by the current content, or was there something specifically confusing about the text that I've mentioned?
Even if you're ok now, leave this issue open ... it will close automatically when the PR merges.

Someone from Stackoverflow posted a link where I figured it out.
I had all these namespaces:
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Components.Authorization;
using Microsoft.AspNetCore.Components.Forms;
using Microsoft.AspNetCore.Components.Routing;
using Microsoft.AspNetCore.Components.Web;
But I didn't have the root of AspNetCore.Components:
using Microsoft.AspNetCore.Components;
Thanks for the response, and sorry I posted. All the examples I found used @Code inline on the .razor page.

That shouldn't be necessary with 3.1 to use partial classes. That was a requirement for 3.0. Are you on 3.1?

Yes I am using 3.1. If II create a partial class and do not add a using statement for Microsoft.AspNetCore.Components, I cannot have a [Parameter] tag on the page.
I have the latest Visual Studio installed 16.4.2.
Everything works fine as long as I add the using reference line.

Oh! ... I see what you mean. Yes, ParameterAttribute is in the Microsoft.AspNetCore.Components namespace. Yes, I wonder if it would be best if the templates had that imported.
Let's ping @danroth27 here to see if an engineering issue should be opened for it:
TL;DR The question is should using Microsoft.AspNetCore.Components be added to the templates' _Imports.razor files to make (for example) ParameterAttribute available for partial classes?
Either way on that question: As part of the update for namespaces generally that I was going to make, I'll add the namespace to the partial class example in the topic. I'll do that right now so we get it in quickly.
Leave this issue open, as it will close automatically when the PR merges.

Thanks @DataJuggler ... try the last two commits on the PR to see if that would have avoided the confusion ...
https://github.com/aspnet/AspNetCore.Docs/pull/16098
This issue will close automatically when the PR merges.


Update resin-image-fs, to use a Node 10 compatible ext2fs

Depends on https://github.com/resin-io/reconfix/pull/45 - this won't install until that's merged & released.
Key difference is that it means we depend on a far newer ext2fs (^1.0.0), which will support Node 10 properly, thereby making the CLI easier to install.
Change-Type: patch

Reconfix now merged and pulled in here, so this should be good to go I think. Reviews please @thgreasi @zvin


ci: install rust wasm target first before building

please rename the package to @graphql-codegen/client-preset-swc

@n1ru4l I pushed it before you mentioned it 😛

@saihaj what's this?

@YassinEldeeb unsure we can ignore for now and investigate it later


Add preliminary documentation to create-styles

Missing from this PR:

[ ] ThemeProvider
[ ] CSS Custom Properties (Can we delete this, it was for IE11 support right?)
[ ] Hooks

@sarayourfriend Thank you so much!!! 🙏
This is a wonderful start! We could merge this PR now. We could also wait, if you want to get the other checklist items in.
(And yes, I think we can delete CSS Custom Properties :D)


failure during az containerapp exec

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az containerapp exec Extension Name: containerapp. Version: 0.3.10.
Errors:
The command failed with an unexpected error. Here is the traceback:
'logStreamEndpoint'
Traceback (most recent call last):
  File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/home/jonas/.azure/cliextensions/containerapp/azext_containerapp/custom.py", line 2242, in containerapp_ssh
    conn = WebSocketConnection(cmd=cmd, resource_group_name=resource_group_name, name=name, revision=revision,
  File "/home/jonas/.azure/cliextensions/containerapp/azext_containerapp/_ssh_utils.py", line 56, in __init__
    self._logstream_endpoint = token_response["properties"]["logStreamEndpoint"]
KeyError: 'logStreamEndpoint'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az containerapp exec --subscription {} --resource-group {} --name {} --replica {}

Expected Behavior
Environment Summary
Linux-5.19.0-38-generic-x86_64-with-glibc2.35, Ubuntu 22.04.2 LTS
Python 3.10.10
Installer: DEB

azure-cli 2.46.0

Extensions:
timeseriesinsights 0.2.1
containerapp 0.3.10
kusto 0.5.0
log-analytics 0.2.2

Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

Additional Context

Thank you for opening this issue, we will look into it.

@jsc-di Thanks for reaching out to us and reporting this issue. I see that you are using an older version of containerapp extension. Could you please update the extension by running the below command and check if that helps ?

az extension update -n containerapp

Awaiting your reply.

@navba-MSFT that did it. Thx!
I thought that there are some warning message if I use an older version, though..


OTG Test - RT-5.3: Aggregate Balancing

The OTG on KNE conversion for RT-5.3: Aggregate Balancing.
The PR includes the changes from otgutils internal package raised here #514
IPv6 flow label is not yet supported in OTG....to be added later

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.664%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1462

Relevant Lines:
2012

💛  - Coveralls

Please assign this to me for go readability review if deemed appropriate, unfortunately I cannot review this for content
Following the new procedure from https://github.com/openconfig/featureprofiles/pull/606

Hi, we'll try Marcus for review of content here.  Thanks!

Hi @marcushines. Could you please review this PR?
Thanks!

Hi @liulk. Could you please merge?

@octpetre merged for you


Error: This Suspense boundary received an update before it finished hydrating.

Hey! Thanks again for opening up your ecosystem of tools, this was just what I was looking for - as proper page transitions in React / Next.js with lifecycle hooks are far from trivial :)
I'm currently trying to implement this using Next.js like this:
function DefaultPageTransition({ children }) {
  const router = useRouter();

  // get a more friendly path name (especially for generic pages instead of [...slug])
  const pageName = router.asPath;

  return (
    <PageTransitions
      key={router.isReady ? "hydrated" : "ssr"} // make sure ?querystring parsing works
      pageName={pageName}
    >
      {children}
    </PageTransitions>
  );
}

<DefaultPageTransition>
            <Component key={router.asPath} {...pageProps} />
</DefaultPageTransition>

I get this error however:
Error: This Suspense boundary received an update before it finished hydrating. This caused the boundary to switch to client rendering. The usual way to fix this is to wrap the original update in startTransition.

This happens in PageWrapper.tsx, I've tried the following, but to no avail:
useEffect(() => {
   startTransition(() => {
     store.setState({ suspendedRoute: pathname });
   });
   return () => {
     startTransition(() => {
       store.setState({ suspendedRoute: null });
     });
   };
 }, [pathname]);

I've also tried changing useEffect to useIsoMorphicLayoutEffect, but that won't help either.

I'm on Next.js v14.2.5 using the pages router

Hi @Fabiantjoeaon - sorry for the silcence, I somehow don't get an email on new issues, not sure what setting to adjust 🤔
Not exactly sure why this happens in your case, it's a bit delicate depending on the Nextjs version and order of components it seems.
For isntance, I've had this error happen when calling usePageTransition() as a hook in the layout component outside the  component - but solved it by wrapping it in a functional component instead of just using the hook.
Sorry for not having more ideas!


Regression on v1.0 where there are missing headers when returning request error

In 1.0-preview.1 of the service, when there was a request error, the service was returning valuable information in the headers, like for example: Content-type as application/json which helps the SDK do proper parsing of the message in the body.
We also expose to the customers the apim-request-id as a way to communicate that with the service in case further investigation needs to happen.

We currently don't see this in the v1.0 endpoint and this is causing that the .NET SDK to not be able to present the information to the user.

Service has been informed and they are looking into it

@maririos
can you please provide more info?
i understand the premise, ..
but have couple of quiestions ..

did the service team do the required update?
should we modify our error class to reflect this? and if so, could you provide similar examples (FormRecognizer, or preview 1.0)

Great questions.

did the service team do the required update?

They started the process when I created the issue but I am not sure what the result of that was. An easy way to find out before pinging them could be to enable test https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/translation/Azure.AI.Translation.Document/tests/DocumentTranslationClientLiveTests.cs#L2, run it in live mode, and see if it passes. If it passes then we are good to go and just need to re-record the test and leave it enabled. If not, then we need to ping the service team and see where we are.

should we modify our error class to reflect this? and if so, could you provide similar examples (FormRecognizer, or preview 1.0)

We should just enable and re-record our test as the SDK code is correct.


"Failed to connect" without stacktrace

in MTProtoTcpConnection you catch exception but not show the problem. in my case the Failed to connect if for "NetworkOnMainThreadException"
            } catch(e: Exception) {
                logger.error(marker, "Failed to connect")
                // <----- insert printstacktrack here or throw exception
                try {
                    socketChannel.close()
                } catch (e: Exception) {
                }
            }

Added the exception as a parameter to the log method :)


Support input prompt for bolt command run

Use Case
I'm using Bolt to run tasks concurrently but the tool it is calling has an input prompt
Describe the Solution You Would Like
If there is an input prompt, it would be nice if it could be passed back to the calling user to respond to in the same way it work if calling from the host directly.
Describe Alternatives You've Considered
I considered whether there is a way that I could provide the inputs via an env var or argument. For my use case, this wasn't possible due to limitations of the tool being called.
Additional Context
Currently, if there is an input prompt, it will just hang. An alternative (if there is no desire to support this) it would be nice to show a helpful message that says the command was blocked due to an input required.

I don't think it's possible to detect that a command is waiting for input. The command will print something that a human user can understand as a request for input, but from Bolt's point of view it's just some arbitrary output.
Even if we could detect it, handling the case where multiple targets need input at the same time is extremely difficult. Unfortunately, I'm not sure there's much we can do to help this use case.
You could maybe write a task with #!/usr/bin/expect, depending on what exactly it needs to do.

Closing as won't do


Invalid argument(s): Failed to load dynamic library 'librplayer.so': dlopen failed: library "librplayer.so" not found

I have integrated this package into my project. But it is giving an error that librplayer.so not found.

I have integrated this package into my project. But it is giving an error that librplayer.so not found.

did you found a solution ? please help!


Fix looking at with 180 degree arc

Supersedes/closes https://github.com/godotengine/godot/pull/93653
Fixes https://github.com/godotengine/godot/issues/80249
Fixes https://github.com/godotengine/godot/issues/57457
Fixes https://github.com/godotengine/godot/issues/53793

Have it calculate the perpendicular vectors by the cross product of the constants vector3 (1, 0, 0) or (0, 1, 0) depends on the nearest axis and implement the similar[^1] approach as #93653 in a more simple way. Also, for looking_at() and look_at(), allow this with a warning (and explain it in the document).
[^1]: This PR may cause a breaking compatibility in that the quaternion of the 180 degree rotation will be inverted for consistency in the direction of rotation, but this is almost never a problem unless you use slerp, etc. in the special situation of generating a 180 degree arc. Of you really want to fix it, the user can align the phase of the quaternion, as in Quaternion(Basis(Quaternion(v0, v1))).

We tend to require more help with documentation!


Product list filters NEXT-8591 #880

1. Why is this change necessary?
Fix for issue #880
2. What does this change do, exactly?
Allows the filtering of products on the product list page in the admin panel.
The filter sidebar item is created as its own component and can be used on any list page with by simply passing an array of what filter options is wanted.
3. Describe each step to reproduce the issue or behaviour.
Go to product listing page in the administration panel, before change is was not possible to filter in the result, after change it is.
4. Please link to the relevant issues (if any).
#880
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: NTWEXO:x: Ion-mandenYou have signed the CLA already but the status is still pending? Let us recheck it.

Hello @Ion-manden, could you sign the CLA?

@pweyck accepted it a while back, did it not go through?


Userscript not not working in Adguard

Describe the bug
Oldlander userscript not working with Adguard android which has userscript support.
To Reproduce
Steps to reproduce the behavior:

install Oldlander on adguard
install old reddit redirect
Oldlander not activated

Expected behavior
Oldlander should activate
Screenshots

Desktop (please complete the following information):
Not Applicable
Smartphone (please complete the following information):

Device: [Samsung M12]
OS: [Android 13.1]
Browser:  [Google Chrome]
Version [127]

Reddit page where bug can be easily reproduced
https://old.reddit.com/r/Android/comments/18wm7tp/octonezdoldlander_firefox_addon_to_make_old/
If applicable, add reddit link here where you can easily reproduce bug.
https://old.reddit.com/r/Android/comments/18wm7tp/octonezdoldlander_firefox_addon_to_make_old/
Additional context
Not Applicable

Check latest version - should work correctly.

Check latest version - should work correctly.

Tried the latest version it's working but the preferences are not getting saved.

Preference arent getting saved probably cause AdGuard userscript engine doesn't provide engine. I will try to add fallback for that later on.


Retro Gets Stuck with Pong-Atari Multiplayers

Issue summary
When I launch Pong-Atari with 2 players, it looks like the game gets stuck (never terminate), please render to see it.
[Put a detailed description of the issue here.]
import retro
import matplotlib.pyplot as plt
import gym
#from IPython import display

import gym
import gym.spaces
import numpy as np
import retro

def PongDiscretizer(env):
    """
    Discretize Retro Pong-Atari2600 environment
    """
    return Discretizer(env, buttons=env.unwrapped.buttons, combos=[['DOWN'], ['UP'], ['BUTTON'],])

class Discretizer(gym.ActionWrapper):
    """
    Wrap a gym environment and make it use discrete actions.
    based on https://github.com/openai/retro-baselines/blob/master/agents/sonic_util.py
    Args:
        buttons: ordered list of buttons, corresponding to each dimension of the MultiBinary action space
        combos: ordered list of lists of valid button combinations
    """

    def __init__(self, env, buttons, combos):
        super().__init__(env)
        assert isinstance(env.action_space, gym.spaces.MultiBinary)
        self._decode_discrete_action = []
        self._decode_discrete_action2 = []
        for combo in combos:
            arr = np.array([False] * env.action_space.n)
            for button in combo:
                arr[buttons.index(button)] = True
            self._decode_discrete_action.append(arr)
        # # pkayer 2 : 7: DOWN, 6: 'UP', 15:'BUTTOM'

        arr = np.array([False] * env.action_space.n)
        arr[7] = True
        self._decode_discrete_action2.append(arr)

        arr = np.array([False] * env.action_space.n)
        arr[6] = True
        self._decode_discrete_action2.append(arr)

        arr = np.array([False] * env.action_space.n)
        arr[15] = True
        self._decode_discrete_action2.append(arr)

        self.action_space = gym.spaces.Discrete(len(self._decode_discrete_action))

        print(arr)
        print(self._decode_discrete_action)
    def action(self, act1, act2):
        act1_v = self._decode_discrete_action[act1].copy()
        act2_v = self._decode_discrete_action2[act2].copy()
        return np.logical_or(act1_v, act2_v).copy()

    def step(self, act1, act2):
        return self.env.step(self.action(act1, act2))
        #return self._decode_discrete_action[act].copy()

env = PongDiscretizer(retro.make(game='Pong-Atari2600', players=2))
env.reset()
done = False
steps = 0
total_rew = 0.0
while not done:
    #obs, rew, done, info = env.step(np.random.randint(env.action_space.n))
    #obs, rew, done, info = env.step(steps %3, np.random.randint(env.action_space.n))
    obs, rew, done, info = env.step(1,1 )
#     plt.imshow(env.render(mode='rgb_array'))
#     display.display(plt.gcf())
    #env.render()
    steps += 1
    print(steps)
    if done:
        print("done == True")
        print("info: {}".format(info))
        print('total step ={}'.format(steps))
        print(rew)
        break

System information

[Operating system]
[Python version]
[Gym Retro version]

This looks similar to https://github.com/openai/retro/issues/198
For whatever reason, Atari Pong in retro has two problems:  the button layout is weird, and the game requires that each player hit "BUTTON" after scoring to serve the ball.
2 player atari has 16 buttons available, here are some decent names for those buttons (in order): ["P1_BUTTON", None, "P1_SELECT", "P1_RESET", "P1_UP", "P1_DOWN", "P1_LEFT", "P1_RIGHT", "P2_BUTTON", None, "P2_SELECT", "P2_RESET", "P2_UP", "P2_DOWN", "P2_LEFT", "P2_RIGHT"]
These seem to work fine for FishingDerby-Atari2600, but for Pong-Atari2600, you actually need to use some different buttons:

use P1_LEFT instead of P2_UP
use P1_RIGHT instead of P2_DOWN
use P2_RIGHT instead of P2_BUTTON

Here are the indices for button, up, and down for each player (unless I counted incorrectly)
P1: 0, 4, 5
P2: 15, 6, 7
Your script seems to work fine if I do obs, rew, done, info = env.step(np.random.randint(env.action_space.n), np.random.randint(env.action_space.n))

Closing due to inactivity

any help on this? I got the same issue, and the ball gets stuck

@christopherhesse

Did you try the workaround that I posted?


update mapping of char to signed char

Fixes the current compile error of rmw_connext_dynamic_cpp introduced by ros2/rosidl#211. The rmw impl. still doesn't "work" but at least it is compilable again.

I addressed the style problems. The failing tests are expected.

Whoops, sorry for breaking that. Was there a way for me to know that I'd broken it?

We can enable/disable arbitrary rmw implementation on CI using checkboxes. But we don't test connext_dynamic and opensplice by default on CI so you couldn't know. Most changes don't affect non-supported rmw_implementation. The type of changes affecting all rmw_implementations:

changes in rmw
changes in idl/types (anything related to type support)

Very likely many others but I'd say these are the main ones I faced in the past

I noticed that by compiling my workspace locally. I know that tests don't
run for dynamic (hence the disable button on the farm), but the workspace
should still be compiling.
On Tue, Apr 11, 2017 at 3:02 PM, Mikael Arguedas<EMAIL_ADDRESS>wrote:

We can enable/disable arbitrary rmw implementation on CI using checkboxes.
But we don't test connext_dynamic and opensplice by default on CI so you
couldn't know. Most changes don't affect non-supported rmw_implementation.
The type of changes affecting all rmw_implementations:

changes in rmw
changes in idl/types (anything related to type support)

Very likely many others but I'd say these are the main ones I faced in the
past
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ros2/rmw_connext/pull/222#issuecomment-293413562, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFKzYXrwazAQYlCJ63Yjvq3pb5c06CEOks5ru_hlgaJpZM4M6YEa
.


"Unexpected error" during profiling.

Hi,
Just tried ultra with an app that I'm working on and it failed with a message to console saying
"Unexpected error:An item with the same key has already been added. Key: 4140"
I have both an .etl (920MB) and .etlx (512MB) file in the directory where I launched alpha, but don't know whether it had failed during the etl merge or during the json.gz generation. Unfortunately I can't provide the ETL trace data (private.. work would kill me etc etc), but if there's anything I can do to help identify the root cause then do let me know.
Cheers,
Dean

Hitting the same thing. No idea if this is supposed to work on .NET FW but I was trying to profile Visual Studio itself.

"Unexpected error:An item with the same key has already been added. Key: 4140"
I have both an .etl (920MB) and .etlx (512MB) file in the directory where I launched alpha, but don't know whether it had failed during the etl merge or during the json.gz generation. Unfortunately I can't provide the ETL trace data (private.. work would kill me etc etc), but if there's anything I can do to help identify the root cause then do let me know.

That's ok. If it crashes with that, it is likely during the conversion, so you should have still the ETL files on the disk. You can try to convert the file with some more verbosity and it should hopefully give a more detailled call stack:
ultra --verbose convert --pid xyz your_etl_file.etl

The only annoyance here (that I need to improve) is that you need to open the ETL file originally with a Perfview in order to extract the PID of the process that you were trying to profile.
Or you can run the profile again with --verbose as well. It should work. 🤞
It's an early 1.0.0, so it is likely that there are few bugs like that still around 😅

Hitting the same thing. No idea if this is supposed to work on .NET FW but I was trying to profile Visual Studio itself.

I haven't tested it with .NET Framework, so dunno if it is working. I only tested it with a few simple .NET (Core) applications.

Hi @xoofx,
Extracting the PID and running convert with --verbose gives the following callstack at the point of failure:
Unexpected error: An item with the same key has already been added. Key: 4140
System.ArgumentException: An item with the same key has already been added. Key: 4140
   at System.Collections.Generic.Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
   at System.Collections.Generic.Dictionary`2.Add(TKey key, TValue value)
   at Ultra.Core.EtwConverterToFirefox.Convert(String traceFilePath, List`1 processIds, EtwUltraProfilerOptions options) in
D:\a\ultra\ultra\src\Ultra.Core\EtwConverterToFirefox.cs:line 234
   at Ultra.Core.EtwUltraProfiler.Convert(String etlFile, List`1 pIds, EtwUltraProfilerOptions ultraProfilerOptions) in
D:\a\ultra\ultra\src\Ultra.Core\EtwUltraProfiler.cs:line 338
   at Ultra.Program.<>c__DisplayClass0_3.<<Main>b__16>d.MoveNext() in D:\a\ultra\ultra\src\Ultra\Program.cs:line 213
--- End of stack trace from previous location ---
   at Spectre.Console.Status.<>c__DisplayClass16_0.<<StartAsync>b__0>d.MoveNext() in /_/src/Spectre.Console/Live/Status/Status.cs:line 79
--- End of stack trace from previous location ---
   at Spectre.Console.Status.<>c__DisplayClass17_0`1.<<StartAsync>b__0>d.MoveNext() in /_/src/Spectre.Console/Live/Status/Status.cs:line 120
--- End of stack trace from previous location ---
   at Spectre.Console.Progress.<>c__DisplayClass32_0`1.<<StartAsync>b__0>d.MoveNext() in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 138
--- End of stack trace from previous location ---
   at Spectre.Console.Internal.DefaultExclusivityMode.RunAsync[T](Func`1 func) in /_/src/Spectre.Console/Internal/DefaultExclusivityMode.cs:line 40
   at Spectre.Console.Progress.StartAsync[T](Func`2 action) in /_/src/Spectre.Console/Live/Progress/Progress.cs:line 121
   at Spectre.Console.Status.StartAsync[T](String status, Func`2 func) in /_/src/Spectre.Console/Live/Status/Status.cs:line 117
   at Spectre.Console.Status.StartAsync(String status, Func`2 action) in /_/src/Spectre.Console/Live/Status/Status.cs:line 77
   at Ultra.Program.<>c__DisplayClass0_0.<<Main>b__15>d.MoveNext() in D:\a\ultra\ultra\src\Ultra\Program.cs:line 169
--- End of stack trace from previous location ---
   at XenoAtom.CommandLine.Command.RunAsync(IEnumerable`1 arguments, CommandRunConfig runConfig)
   at XenoAtom.CommandLine.Command.RunAsync(IEnumerable`1 arguments, CommandRunConfig runConfig)
   at Ultra.Program.Main(String[] args) in D:\a\ultra\ultra\src\Ultra\Program.cs:line 244

In my case, our application is a mix of .NET 8 C#, C++/CLR, and Native.
Hope this helps!
Dean

Hopefully 0c4f181 should fixed it, you can grab 1.0.1 in ~ 10 minutes, let's see 🤞

Hi @xoofx ,
Release v1.0.1 fixes it, and I now have a .json.gz that I can load into Firefox. 🎉
Thanks for the (very) quick fix!
Dean

Cool. I just tried it with VisualStudio devenv.exe and it seems to work ok https://share.firefox.dev/4hT3RLk

I was able to get a json.gz, but profiler.firefox.com complains about it "Error: Unserializing the profile failed: SyntaxError: Unexpected end of JSON input". Opening the JSON locally it seems fine visually, appears to terminate, etc.
Regardless, thanks for the rapid fix. I'll have to give it another try later. :-)

@xoofx I tried again, this time attaching to a running process, and it worked! Some tracks like the memory track were missing, which made the profile a lot smaller. Perhaps that made the difference?


Branch 2.1

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@zhuge134, seems mistakenly open. Could you close this please?