Fitter may be non-deterministic and giving very different fit values run-to-run for some targets

I ran the same 5 objects via Susan's backend, which calls the fitter given an input period, ephemeris, etc. Below are the values coming out for the same 5 objects for 3 different runs. The first five columns are EPIC ID, Period, Epoch, Duration, and SNR.
Note that for the first object, 210775710, the epoch changes by up to 0.04 days, the duration ranges from 2.97 to 4.92 hours, and the SNR changes from 148 to 202. This is a systematic feature, but it is deep, so I wouldn't normally expect such a big variance fit-to-fit. Seems worrisome.
Some others however, like the second, third, and fourth objects, don't vary at all run-to-run - they are exactly the same. These do look like real transits though.   Was there more than one transit seen in the data for those two, or only one?  I wonder if that’s part of the problem.

On Jul 7, 2016, at 5:09 PM, JeffLCoughlin<EMAIL_ADDRESS>wrote:
I ran the same 5 objects via Susan's backend, which calls the fitter given an input period, ephemeris, etc. Below are the values coming out for the same 5 objects for 3 different runs. The first five columns are EPIC ID, Period, Epoch, Duration, and SNR.
Note that for the first object, 210775710, the epoch changes by up to 0.04 days, the duration ranges from 2.97 to 4.92 hours, and the SNR changes from 148 to 202. This is a systematic feature, but it is deep, so I wouldn't normally expect such a big variance fit-to-fit. Seems worrisome.
Some others however, like the second, third, and fourth objects, don't vary at all run-to-run - they are exactly the same. These do look like real transits though.
Reply to this email directly, view it on GitHub https://github.com/barentsen/dave/issues/15, or mute the thread https://github.com/notifications/unsubscribe/AM5vypQWRifVT8-lYIrqclyrWvcLwriVks5qTZVHgaJpZM4JHndn.

The trapezoid fitter is not guaranteed to be deterministic and find the global minimum.  The trapezoid fitter does fitTrialN (currently defaults to 13) trials of fitting the transit with random starting conditions around the initial results.  The best solution out of these 13 trials is taken as the final result.  I think misshapen and high SNR transits will typically take more trials to find a global minimum.  The only way around this is to (1) bump up fitTrianlN (18? takes more runtime), (2) live with the non-determinicity, or (3) somehow set the random seeds (fixed for all targets or based upon some other unique target identifier) in the trapezoid fitting.
So, I am not surprised to see nondeterminicity.  Is this breaking something?  My preference is (2) do nothing, (1) bump up trials, (3) is quite a bit of work to test to make sure it works as expected and will not improve your chances of finding global minimum while giving the false impression through determinicity that it has.

Ah that makes sense then. It isn't breaking anything - I just noticed it when comparing runs from my local machine and on Hal and investigated it. The large variance in the SNR and duration was worrisome, but given that target was a large systematic, and the transit-like ones don't appear to have the same effects, I think we're fine. We should just keep an eye out in case we ever see important dispositions change from run-to-run as a result.


App.Config??

Hello,
I created the project following these instruction but i do not see a app.config file.  Am i missing a step ?
Johan

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

ID: 3469d675-7452-0f70-3ef1-eb4d353b7163
Version Independent ID: 9e089f76-487e-1e22-c7f5-57db9944f279
Content: Tutorial: Define a Windows Communication Foundation service contract
Content Source: docs/framework/wcf/how-to-define-a-wcf-service-contract.md
Product: dotnet-framework
Technology: dotnet-wcf
GitHub Login: @Lxiamail
Microsoft Alias: wcfsrvt

Hello. We need more information about your environment and projects. I just used Visual Studio 2019 and chose the WCF Service Library template and .NET Framework 4.7.2 and I got the app.config file. What versions are you using?


Basic authentication with passwords is deprecated

Hello
The application is throwing this error on JIRA cloud login:
System.Security.Authentication.AuthenticationException: Response Content: Basic authentication with passwords is deprecated.  For more information, see: https://developer.atlassian.com/cloud/confluence/deprecation-notice-basic-auth/
at Atlassian.Jira.Remote.JiraRestClient.GetValidJsonFromResponse(IRestRequest request, IRestResponse response)
at Atlassian.Jira.Remote.JiraRestClient.d__11.MoveNext()
when using:
jira-export -u<EMAIL_ADDRESS>-p xxxxxxx --url https://my.jira.url --config config-scrum.json --force
Is there a fix for this?

Try https://github.com/solidify/jira-azuredevops-migrator/issues/690

Try https://github.com/solidify/jira-azuredevops-migrator/issues/690


Add github actions to export html and pdf in dist

Hello,
I added a github actions file to export slides to html and pdfs. Exported files are then downloadable as artifacts (top right of the build page, see below).

Looks nice!
I'll go ahead and merge. If there are problems, we can address them later.


Feature Request:  Custom Entry Name

I think it would be nice to have a field in the settings to change the name of the journal entry from "Containers" to something custom that may make more sense to the individual GM.
Also if the Journal Entry option is selected and the custom name filled in maybe have it create the entry if it doesn't exist.

I'll add the first one in the next update, I don't think the auto creation is a good idea tho since it could cause unwanted journal creations.

Great!!   Thanks for considering this, but I agree I can see how that auto creation can get messy.  :)

Works like a charm!!!


Trying=> Downloading wimlib Error: You are using macOS 10.16.

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[ ] ran brew update and can still reproduce the problem?
[ ] ran brew doctor, fixed all issues and can still reproduce the problem?
[ ] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)

What happened (include command output)
Error: You are using macOS 10.16.
We do not provide support for this pre-release version.
You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Discourse, Twitter or IRC. You are responsible for resolving any issues you
experience while you are running this pre-release version.

Command output

What you expected to happen

Step-by-step reproduction instructions (by running brew commands)

Output of brew config and brew doctor commands

You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Discourse, Twitter or IRC. You are responsible for resolving any issues you
experience while you are running this pre-release version.


Initial Codice L1A validation tests

Change Summary
Overview

This PR adds a new test for validation of CoDICE L1a CDFs. We currently don't have validation data from CoDICE, but this sets up the test to be able to compare data values between the CDFs generated by the pipeline and CoDICE validation files. The validation files I am currently using in these tests are just the CDFs that I generated myself. So not a great test, but the goal here is to just set up the framework so that once CoDICE validation files are acquired, they can just be swapped in.
Relates to #251
New Files

imap_processing/tests/codice/data/temp_validation_<descriptor>.cdf

(Temporary) validation files for the currently supported data products: hskp, lo-sw-species-counts, and lo-nsw-species-counts

Updated Files

imap_processing/codice/codice_l1a.py

process_codice_l1a function now returns the xarray dataset in addition to the CDF filename so that the dataset can easily be tested

imap_processing/tests/codice/test_codice_l1a.py

Added test that compares data values for validation CDFs and CDFs generated by the pipeline

@greglucas Do you have any suggestions on how to deal with the temporary validation files? Should I just read them from a local directory?

@greglucas Do you have any suggestions on how to deal with the temporary validation files? Should I just read them from a local directory?

Yes that is definitely an option and then add a pytest.skip() if that isn't found possibly. I guess I'm not sure at the ultimate end goal. Are you anticipating getting CDF files as validation data? I think most of our other instruments have been providing csv or text files, so it isn't as easy as just direct comparisons and we've had to only compare specific variables that were provided in the csv file I believe.

@greglucas Do you have any suggestions on how to deal with the temporary validation files? Should I just read them from a local directory?

Yes that is definitely an option and then add a pytest.skip() if that isn't found possibly. I guess I'm not sure at the ultimate end goal. Are you anticipating getting CDF files as validation data? I think most of our other instruments have been providing csv or text files, so it isn't as easy as just direct comparisons and we've had to only compare specific variables that were provided in the csv file I believe.

I planned on removing these temp_validation_*.cdf files once I acquired some sort of CoDICE validation data (whether it were CDF, CSV, txt, etc.) and updating these tests to compare the arrays to whatever the validation data may be. The goal was really to just set up some sort of test that gathered the data arrays and put them in a place ready to be compared to something. (see
Perhaps I can remove these temp files, and just mark the tests as xfail for now?
This also reminds me of your comment on my last PR (https://github.com/IMAP-Science-Operations-Center/imap_processing/pull/530#discussion_r1576669263) -- I still need to add a test for the shapes and lengths of these data arrays

Perhaps I can remove these temp files, and just mark the tests as xfail for now?

That seems reasonable to me and addresses my issue of growing the repository size. I'm not sure if skip or xfail would be better? @pytest.mark.skip(reason="no way of currently testing this")


Add XCTest to the swift-package-manager sample

As a Swift developer, I'd like to see an example of how tests should be configured in Gradle for my project which is currently using the SwiftPM tool to build, so I don't have to figure out how to add them myself.

I half-started this before we got into everything else last week: https://github.com/gradle/native-samples/pull/3

I've added a failing integration test in https://github.com/gradle/gradle/commit/d1f4cf40bb1424591681e87c83199d42b912290f

@eljobe I left you some comments

I merged the sample into native-samples.


Clicking between shared workflows does not change selection

Bug Report
Clicking between shared workflows does not change the selection.
The address bar updates, but the content does not, nor does the selection on the left change.
In the screenshot, I clicked on test2, which shows on the address bar, but test6 is selected.

Problem seems to be getting worse. I see this on develop as well for normal workflows.

FYI, this might have been worked on already

Observed together with both shared and non-shared

Noting duplication with https://github.com/ga4gh/dockstore/issues/1572

@agduncan94
For my part, the mixing up of files between hosted workflows has been resolved on staging.dockstore.org.
@coverbeck
Is the address bar not-changing issue still reproducible for you?
Please close if no longer present

This works for me in staging.


Allow and/or blocks to have 0 or 1 inputs

Change-Id: Ic04587ca376f7ed1bb582d8c273b6f16833ad13d

@mit-cml/core-reviewers This PR now reflects the version I sent out this morning with the tweak to the + operator suggested by @halatmit

logic or doesn't look quite right yet. Identifier is missing after I removed all the inputs in the mutators and there are still two inputs showing on the block.

@ellelili2025 I've addressed your feedback in the following way:

This was due to my introduction of a helper method on logic_operator that I didn't also define on logic_or. This should be fixed in the first commit.

Implementing this wasn't part of my original goal for this PR, but I've gone ahead and added it.

For bitwise-and the identity is -1 (the two's complement of which is all 1s in binary) and for ior/xor it is 0.

Logic or looks fine now. But min with 0 input gives me<PHONE_NUMBER>854775807 and max gives me -9223372036854775807.

Hmm. When I use Do It I get +Infinity and -Infinity, which is what I would expect.


Standardized anchor tag for known and unkown providers

Updated a tag to add target="_blank" and rel="noopener nofollow" if the link is not from a know provider.

JSON of know provider:

{
  "type": "text",
  "marks": [
    {
      "type": "link",
      "attrs": {
        "href": "https://ritesh.scrollstack.com/post/16",
        "title": "The Battle for Dominance"
      }
    }
  ],
  "text": "The Battle for Dominance - US and China fight to rule India's $1 trillion Digital Economy"
}

Expected HTML output:

The Battle for Dominance - US and China fight to rule India\'s $1 trillion Digital Economy

JSON of unknown provider

{
  "type": "text",
  "marks": [
    {
      "type": "link",
      "attrs": {
        "href": "https://en.wikipedia.org/wiki/Zen_of_Python",
        "title": "Python"
      }
    }
  ],
  "text": "Zen of Python"
}

Expected HTML output:

Zen of Python

Let's come up with config system as discussed.


migrate cudaMemcpy to cudaMemcpyAsync+cudaStreamSynchronize

Description
As title.
Plus some refactors to reduce calls to mshadow::Stream<gpu>::GetStream().
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] API Migration from cudaMemcpy to cudaStreamSynchronize

Comments
address #16583

Is it actually necessary to to call cudaStreamSynchronize before launching kernels on the same stream? From what I understand, CUDA calls that are issued on the same stream are executed sequentially. At least, in my experiments with contrib.index_array (#16639), omitting cudaStreamSynchronize doesn't seem to break anything.

Not technically part of this PR so can't comment on the line, but https://github.com/apache/incubator-mxnet/pull/16790/files#diff-8ce45deadbf52bb9d13aa3bb9562dddaR380 is bad. That operator should have a workspace.

Generally src/operator/contrib/multi_proposal.cu seems not great with e.g. kernels using default stream instead of stream given to the operator.

@nickguletskii Some of them are necessary, unnecessary ones have been removed.
@ptrendx agree that those contrib operators should be made better, could be addressed in a future PR.

@ptrendx @nickguletskii Looks good to you guys?


change to use golang 18 and release 3.2.1

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have updated the CHANGELOG.md and version subcommands to the next logical semantic version
[ ] All new and existing tests passed.
[ ] All required Github Actions for PR have passed.

Fix similar issue stated in release-action https://github.com/sanjP10/release-action/issues/7


Improve auth example auth

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[x] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

Why this PR need to be merged?

Closes #86


Testing

@Acibeee Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@Acibeee Thank you for signing the Contributor License Agreement!


Update install-toucan.sh to create /usr/local/bin directory

For machines that are affected by bug #24, these changes create the /usr/local/bin directory.
For this to work on those machines the make command needs to be called with sudo...
sudo make install
...this requires the admin password.

Looks good, thank you for the contribution. 🙏

Thank you. Glad I could help a little. 🙏


[CX-16266] Change with reference to 1a4c164 commit in open source

Due to refactoring of models.py in open source, made this changes instead of cherry picking the commit.

lgtm


Internal Server Error when opening Ballerina Test docs using VS Code View API Docs

Description:

An Internal Server Error occurs when attempting to open the Ballerina Test library documentation within the VS Code View API Docs functionality.

Generated Link: https://lib.ballerina.io/ballerina/test/0.0.0#Config
Working Link: https://lib.ballerina.io/ballerina/test/latest#Config

Suggested Labels:

Suggested Assignees:

Affected Product Version:
2201.8.5
OS, DB, other environment details and versions:
Steps to reproduce:

Open a Ballerina project in Visual Studio Code.
Access the documentation for the test library using the VS Code View API Docs feature.
Open the provided link: https://lib.ballerina.io/ballerina/test/0.0.0#Config

Related Issues:

In here we have to pick the ballerina version and replace with 0.0.0.


set initial value for BehaviorSubject, update connectivity_plus

update connectivity_plus: ^6.0.3

set initial value for BehaviorSubject<SocketConnectionState>

When use toggleConnection will throw at
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: ValueStream has no value. You should check ValueStream.hasValue before accessing ValueStream.value, or use ValueStream.valueOrNull instead.
#0      BehaviorSubject.value (package:rxdart/src/subjects/behavior_subject.dart:146:5)
#1      SocketClient._listenToToggleConnection.<anonymous closure> (package:graphql/src/links/websocket_link/websocket_client.dart:264:40)
#2      _RootZone.runUnaryGuarded (dart:async/zone.dart:1594:10)
#3      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:339:11)
#4      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:271:7)
#5      _MultiStreamController.addSync (dart:async/stream_impl.dart:1101:36)
#6      _MultiControllerSink.add (package:rxdart/src/utils/forwarding_stream.dart:130:35)
#7      _TakeUntilStreamSink.onData (package:rxdart/src/transformers/take_until.dart:13:31)

the crash that you are reporting is something that it is caused by connectivity_plus?
This is unclear to me, sorry!
In addition, can you add the stacktrace in the commit body as out dev guide explain?

@vincenzopalazzo I'm so sorry for confusing you. My PR includes two changes:

Upgrate: connectivity_plus: ^6.0.3 for graphql_flutter package.
Resolve: error log is throwed from graphql package althought it doesn't make any crash.

I think it's simple PR so want to merge into one commit.
Thanks for answering!

Can you add them inside the commit body message?
You should be able to edit it with git commit --amend

@vincenzopalazzo I've updated. Please review it for me.
Thank you very much!

It is missing the stacktrace inside the commit body as suggested before

@vincenzopalazzo please review help me again. Many thanks!

@vincenzopalazzo sorry for late reply. I'm using
dependencies:
  graphql: ^5.2.0-beta.7
  graphql_flutter: ^5.2.0-beta.6
  connectivity_plus: ^6.0.3

but get a error when flutter pub get
Resolving dependencies...
Because * depends on graphql_flutter ^5.2.0-beta.6 which depends on connectivity_plus ^5.0.0, connectivity_plus ^5.0.0 is required.
So, because * depends on connectivity_plus ^6.0.3, version solving failed.

and this is from pub_cache dir on my local

Thanks @hantrungkien now the last thing is dropping the commit that we do not need
https://stackoverflow.com/a/1338758/10854225
You should drop the following commits:

77f3386518e453687404869b67437f827caf6df3
0d0de67467668f9084d0bbbb2ce40db2fcf5557c


Raise TemplateDoesNotExist if template does not exist

When sending templated emails
Closes #3783

Codecov Report

Merging #3784 (be947ee) into master (74272ae) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3784   +/-   ##
=======================================
  Coverage   94.80%   94.80%
=======================================
  Files         442      442
  Lines       48416    48418    +2
=======================================
+ Hits        45903    45905    +2
  Misses       2513     2513

Flag
Coverage Δ

servertests
55.85% <0.00%> (-0.01%)
:arrow_down:

unittests
94.67% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

moto/ses/models.py
96.67% <100.00%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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

This is now part of moto >= 2.0.3.dev7


Paper App always showing Apply Online Button

There was a bug in the conditional to show apply online which meant a paper application only was showing with an apply online button as an option.
QA: Test a listing with different combos of online apps and paper apps in both the preview and published state.

i believe this is fixed


feat: add logion client with authentication.

logion-network/logion-internal#454

In order to succeed to yarn install, I had to do the following:

temporally delete .yarnrc.yml.
corepack enable
yarn init -2
un-delete  .yarnrc.yml
finally run  yarn install

Otherwise it keeps on failing with error: Internal Error: The "yarn-path" option has been set (in $PROJECT/logion-api/.yarnrc.yml), but the specified location doesn't exist ($PROJECT/logion-api/.yarn/releases/yarn-3.2.0.cjs).

Regarding that yarn install procedure, I'll investigate to see if there is another way.


Local changes to .arc manifest not reflected in Begin UI

Is your feature request related to a problem? Please describe.
Note I'm not sure if this is a bug or a missing feature.
Either way, here's my experience. After going though the setup of my first Begin app (really great on-boarding experience btw!) I cloned the app locally and started making some edits. The first was to add a new route. After running ARC_LOCAL=1 npx create I validated the new route was working by running npm start. All looked good. I then pushing the updated code to GitHub and which triggered a new build, upon completion I tried to access the new route via the staging URL, and got the following output: {"message":"Missing Authentication Token"}.
It seems as though these local modifications aren't picked up by the Begin app, and thus the app didn't get the new updates.
After manually adding the previously defined route through the Begin UI the new route worked as expected. Though the files created from ARC_LOCAL=1 npx create and my local modifications were overridden. See this commit: https://github.com/joshgillies/begin-functions-app/commit/d91ea1852bc598350233d811516492be1832e624
Describe the solution you'd like
Ideally the .arc manifest is automatically reflected in the Being UI and is automatically provisioned with a resulting build.
Describe alternatives you've considered
The alternative is to manage all newly defined routes from within the web UI.
Additional context or notes
N/A

Hey @joshgillies, thanks for this!
Fwiw as a workaround, I believe you should be able to create the route in the Begin interface and it should pick up and deploy the functions you already committed. (Please let me know if that didn't work!)
So we've talked about this internally for a while, and I'm going to move this over to our feature request roadmap, but before I do, wanted to get your thoughts. Would you prefer:

Modifying the local arc file and having that modify the Begin app infra upon commit? or
Having a simple Begin CLI tools for doing this, and not managing the arc manifest at all? i.e. begin create get /test, which would spin up the infra, set up the boilerplate function in git, etc.?
(Both?)

Thanks @ryanblock and @brianleroux for the replies. Yes I'd agree both features would be a really good addition to the platform. 👍

heh, this is ooooold and good news! you can now edit your arc file and see changes reflected https://github.com/brianleroux/arc-example-progressive-bundle


Avoid Duplicate Reconcile Logic in Testing Wait Custom Run Controller

Changes
The existing test cases in Wait Custom Run reconciler_test.go are reusing some reconciliation logic in reconciler.go, this may lead some functionality deviation in the future.
This commit rewrites some test cases in reconciler_test.go to reuse ReconcileKind in reconciler.go
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/assign @lbernick

/release-note-none

/kind feature

/approve

/lgtm


Fix replicationgroup update logic

Description of your changes
We check whether an update is due by looking at returned ReplicationGroup and all CacheCluster objects. However, late-init does not compute againts the CacheCluster list and this causes the up-to-date check to return false all the time. I've changed late-init to cover one of the CacheCluster object and this should be fine because late-init is effective only in the first reconcile and all those CacheClusters are meant to converge to the same properties anyway.
Manually tested the fix.
Fixes #277

Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[x] Updated any relevant documentation, examples, or release notes.
[x] Updated the dependencies in app.yaml to include any new role permissions.

@muvaf this should pass CI after https://github.com/crossplane/provider-aws/pull/282 is merged and rebase 👍 Could you give approval over there?


[Reign of Kings] Added New Hook OnPlayerInteract

OnPlayerInteract(InteractEvent Event)
Allow the developer to interrupt the OnInteract event, this allows development of antiloot plugins and more.
Examples of what this effects are Loot Sacks, torches, chests, stations, etc.  almost anything that can be looted.

Anything more you need me to do here Wulf? After all your updates it says it has a conflict that needs resolved, but this .opj file still works with the new oxide build, I have it on my working on my server and have a Anitloot plugin finished and ready for release on Oxide if you want to look at that too.
I can do another pull request for this file if that helps?

I'll likely manually merge it after testing it as the PR is outdated.


typo: succesful -> successful

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Articles/connectors/messages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update dependency org.junit.jupiter:junit-jupiter to v5.10.3

This PR contains the following updates:

Package
Type
Update
Change

org.junit.jupiter:junit-jupiter (source)
test
patch
5.10.2 -> 5.10.3

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Check out this project: https://github.com/7AXEL/Web2APK


Add RunCommands for registering/unregistering a PureStorage plugin

Add RunCommands for registering/unregistering a PureStorage plugin
This PR closes #
The changes in this PR are as follows:
Add RunCommands for registering/unregistering a PureStorage plugin.
The new RunCommands are

Register-PureStorageAvsRemotePlugin
Unregister-PureStorageAvsRemotePlugin

I have read the contributor guidelines and have completed the following:

[x] Formatted the code using VSCode default formatter for PowerShell.
[ ] Tested the code end-to-end against an SDDC.
Tested using on-premise deployment
[x] Documented the functions using standard PowerShell markup and applied AVSAttribute to newly exported functions.

As discussed, this will be in Pure's own package.


Please mention using the Preserve log option.

The section https://learn.microsoft.com/en-us/azure/azure-web-pubsub/howto-troubleshoot-network-trace?branch=main#collect-a-network-trace-in-the-browser-browser-based-apps-only doesn't mention using a Preserve Log option. I understand that this must be selected if issues require multiple page requests to reproduce.
Can you add this?

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

ID: 370e8541-684e-6a57-4805-085c20f89aae
Version Independent ID: 5ffc18f1-bafb-1249-aa8a-3bb6b685ee6c
Content: How to collect a network trace
Content Source: articles/azure-web-pubsub/howto-troubleshoot-network-trace.md
Service: azure-web-pubsub
GitHub Login: @yjin81
Microsoft Alias: yajin1

@wawangmsft I opened this issue per your request.

Firefox:

Safari:

Hi @yjin81 can you please take a look at this feedback and approve the PR

We will now proceed to close this thread since a PR was submitted.  If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Fix issue #2500 Include default requestLog format string in documentation

Picked up issue #2500 and added the default log pattern into documentation.
I did copy it directly from class LogbackAccessRequestLayoutTest, in order to avoid copy/paste errors.
Any feedback welcome.

Thanks!


Images with Dark Mode aren't readable

Can't read the images when using dark mode. The images are using a transparent background instead of white.

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

ID: 33e49268-9863-3be2-218c-c472fe6f3711
Version Independent ID: b104aa79-5b21-cd77-f24e-71a9fd93cf36
Content: A Guide to Query Processing for Memory-Optimized Tables - SQL Server
Content Source: docs/relational-databases/in-memory-oltp/a-guide-to-query-processing-for-memory-optimized-tables.md
Product: sql
Technology: in-memory-oltp
GitHub Login: @MightyPen
Microsoft Alias: genemi

@Phritzy - Thanks Mark! We appreciate you pointing this out. I'll go ahead and change the images.
@MightyPen - https://github.com/MicrosoftDocs/sql-docs-pr/pull/10164

Changes are merge and will go live tomorrow. Go ahead and close when ready. Thanks!

@Phritzy Hi Mark,  The problem you are reporting on is a sudden problem that has arisen in numerous cases.  So we have a couple people who are figuring out what happened, and who should implement a broad fix.
Therefore, I am Closing this valid Issue 2104, as it has plenty of attention now.  :-)
Thank you.


Flet Map

I created Flet Map.

You input longtitude and latitude, show column and row showing images like flutter_map, some issue it have, not directly loading images and you refresh screen.

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

I created Flet Map.
You input longtitude and latitude, show column and row showing images like flutter_map, some issue it have, not directly loading images and you refresh screen.

I suggest renaming control to OpenStreetMap. FletMap is too generic.
Later we could have other implementations such as GoogleMap.

An API key is required as per google maps docs, and it is only for the mobile platforms.
What of flutter_map ? 🤔

If I understand correctly, this version breaks if screenview parameter is not provided. With the addition of Canvas component to flet, this component could be wrapped in it to use the on_resize handler to adjust the tile dimensions automatically to the space available for the map component, removing the need for screenview.

I think https://github.com/flet-dev/flet-contrib would be a better home for this control.
@harunkurtme are you still available to move it to contribs?

I saw new,
Okey master, I will add this contribs.
@FeodorFitsner

Hello, sorry for my comment. I would like to know, is it something new about Flet map support ? It's a very good project, but the map support is a bit lacking for me, :-(

I also think it would be a good function deserves to be developed

Hello all,
I didn't interested in FletMap, you can run at inside this package.
I left a link below, you can check whether it is approved or not via this pull request.
@bibibo-77 @VioletBenin @Insighttful @mikaelho
(Flet-Contrib)[https://github.com/flet-dev/flet-contrib/pull/17#issue-2195578273]


Should we do strong or weak ETag comparison?

CSS communicates an ETag when retrieving a resource:
curl -s -i https://drive.verborgh.org/public/2021/test.txt | grep etag
etag: "1635520727000"

However, Mashlib, sends a slightly modified ETag back when editing the file:
if-match: W/"1635520727000"

resulting in
412 Precondition Failed

Let's find out which of the two is correct and fix accordingly.

Closing, as probably not a CSS bug at the moment.


Question: What's the difference between autofix.ci and an action that applies fixes and push directly to the branch?

What's the difference between autofix.ci and an action that applies fixes and push directly to the branch? I'm not quite understand why a server is needed.

autofix.ci can be used to fix pull requests from untrusted strangers on the internet, doing that with an action would be a security risk.

Ah, thanks for you reply! Love autofix.ci :)


Add an option for disabling the documentation window when autocompleting

When an autocomplete suggestion is hovered over, a preview window appears at the top of the screen showing the member's docstring. This is visually jarring and causes me to lose my place in the file. It would be great if this behavior could be disabled.

If your completeopt setting doensn't include the preview option, the preview won't appear. See :help 'completeopt'.

Thank you!


RTL - LTR Covalent Documentation

Bug Report
RTL - LTR seems to have a bug in the covalent documentation at: https://teradata.github.io/covalent/#/components
If I switch from LTR to RTL and then back to LTR then all the brackets in the example code do not appear correctly, they are reversed/flipped.
Normal appearance:

After switching back to LTR from RTL:

What is the expected behavior?
The brackets should return back to normal on switching back to LTR. I have yet to test it in a test environment.
Which version of Angular and Material, and which browser and OS does this issue affect?
Latest deployed documentation build.

Issue fixed in new documentation


Update Composer dependencies (2021-11-25-00-10)

Visual regression test passed!
View the visual regression test report


Enable arc data services on docker desktop single-node k8s

Currently this feature is unfinished. Current Issues:

mssql container in data controller db pod is failing to start when using local-storage class (error log below)

New Ideas:

attempt to get hostpath to work
setup local NFS

2021/08/10 21:40:05.714842 [mssql-server:stderr] 2021/08/10 21:38:56 [LAUNCH] Starting to run the process "mssql"
2021/08/10 21:40:05.714843 [mssql-server:stderr] ln: failed to create symbolic link '/var/opt/mssql/log/mssql': File exists
2021/08/10 21:40:05.714844 [mssql-server:stderr] 2021/08/10 21:39:00 [LAUNCH] The process "mssql" exists and returns error: The process exited with code '1' and error: exit status 1
2021/08/10 21:40:05.714845 [mssql-server:stderr] 2021/08/10 21:39:00 [LAUNCH] Starting to force stop process "mssql"
2021/08/10 21:40:05.714846 [mssql-server:stderr] 2021/08/10 21:39:00 The process launcher "mssql" exited: Failed to force stop the process "mssql": no such process

2021/08/10 21:40:06.716521 [mssql-server:stdout] 	Copyright (C) 2019 Microsoft Corporation
2021/08/10 21:40:06.716522 [mssql-server:stdout] 	Developer Edition (64-bit) on Linux (Ubuntu 20.04.2 LTS)
2021/08/10 21:40:06.716523 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      UTC adjustment: 0:00
2021/08/10 21:40:06.716524 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      (c) Microsoft Corporation.
2021/08/10 21:40:06.716525 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      All rights reserved.
2021/08/10 21:40:06.716526 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      Server process ID is 380.
2021/08/10 21:40:06.716527 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      Logging SQL Server messages in file '/var/opt/mssql/log/errorlog'.
2021/08/10 21:40:06.716528 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      Registry startup parameters:
2021/08/10 21:40:06.716529 [mssql-server:stdout] 	 -d /var/opt/mssql/data/master.mdf
2021/08/10 21:40:06.716530 [mssql-server:stdout] 	 -l /var/opt/mssql/data/mastlog.ldf
2021/08/10 21:40:06.716531 [mssql-server:stdout] 	 -e /var/opt/mssql/log/errorlog
2021/08/10 21:40:06.716532 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      Command Line Startup Parameters:
2021/08/10 21:40:06.716533 [mssql-server:stdout] 	 -T 3614
2021/08/10 21:40:06.716534 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      Error: 17113, Severity: 16, State: 1.
2021/08/10 21:40:06.716535 [mssql-server:stdout] 2021-08-10 21:39:00.40 Server      Error 87(The parameter is incorrect.) occurred while opening file '/var/opt/mssql/data/master.mdf' to obtain configuration information at startup. An invalid startup option might have caused the error. Verify your startup options, and correct or remove them if necessary.

Environment Info:
Docker Version: v20.10.7
Docker Desktop: 3.5.2
Kubernetes: 1.21.2
WSL2 Integration: enabled
Arc Data Controller Image tag: v1.0.0_2021-07-30
WSL2 Config:
[wsl2]
memory=16GB # Limits VM memory in WSL 2 to 16 GB
processors=4 # Makes the WSL 2 VM use 4 virtual processors

sample pv
apiVersion: v1
kind: PersistentVolume
metadata:
  name: pv1
spec:
  capacity:
    storage: 50Gi
  volumeMode: Filesystem
  storageClassName: hostpath
  accessModes:
  - ReadWriteOnce
  hostPath:
    path: /run/desktop/mnt/host/wsl/arcdata/pv1
    type: Directory

Using rewritten host mount paths accomodating WSL2 docker desktop
https://github.com/docker/for-win/issues/5325

Attempts to use docker desktop hostpath storage class also result in same error.

Attempt to use /mnt/c host path persistent volumes

mssql is able to start and run, however core server fails with crash log

This program has encountered a fatal error and cannot continue running at Wed Aug 11 15:25:15 2021
The following diagnostic information is available:

         Reason: 0x00000006
        Message: Termination of \SystemRoot\system32\AppLoader.exe was due to fatal error 0xC0000001
        Address: 0x3fffbd853014
    Stack Trace:
                 file://package4/windows/system32/sqlpal.dll+0x0000000000298E4C
                 file://package4/windows/system32/sqlpal.dll+0x0000000000296DF5
                 file://package4/windows/system32/sqlpal.dll+0x0000000000253180
                 file://package4/windows/system32/sqlpal.dll+0x0000000000253014
                 file://package4/windows/system32/sqlpal.dll+0x0000000000251E6B
                 file://package4/windows/system32/sqlpal.dll+0x00000000002ACB8B
                 file://package4/windows/system32/sqlpal.dll+0x00000000003927F8
                 file:///windows/system32/AppLoader.exe+0x0000000000004729
                 file:///windows/system32/AppLoader.exe+0x000000000000479B
                 file:///windows/system32/AppLoader.exe+0x00000000000047DB
                 file:///windows/system32/AppLoader.exe+0x000000000000FB5E
                 file:///Windows/SYSTEM32/KERNEL32.DLL+0x0000000000014414
                 file:///windows/system32/ntdll.dll+0x0000000000075541
        Modules:
                 file://package4/windows/system32/sqlpal.dll=303E1B4E4D904BBFA5B49686791254A31
                 file:///windows/system32/AppLoader.exe=39629A68E1594FE59AC543FC257189991
                 file:///Windows/SYSTEM32/KERNEL32.DLL=C715300FB2664729A6126A3F591E6F302
                 file:///windows/system32/ntdll.dll=45137AA3F9814512B3123991067EEE6E2
        Process: 35 - sqlservr
         Thread: 65 (application thread 0x74)
    Instance Id: 881c115d-b2cd-408e-aa2d-f6dfca8d08a2
       Crash Id: 703f2e1e-8ac1-4b72-89a7-f04b5ab7343e
    Build stamp: b259a21007ee0e94d2c7882e92590f605db40ca91948ed68d90a68fea7e09c09
   Distribution: Ubuntu 20.04.2 LTS
     Processors: 4
   Total Memory:<PHONE_NUMBER>6 bytes
      Timestamp: Wed Aug 11 15:25:15 2021

Solution to this problem involved a combination of the following.

creating mount points for each persistent volume that were not of type tmpfs (default for /mnt/wsl).
Using prefixed hostpaths in persistent volumes.

sudo mount --bind /mnt/arcdata/vol1 /mnt/wsl/arcdata/vol1

apiVersion: v1
kind: PersistentVolume
metadata:
  name: pv1
spec:
  capacity:
    storage: 50Gi
  volumeMode: Filesystem
  storageClassName: hostpath
  accessModes:
  - ReadWriteOnce
  hostPath:
    path: /run/desktop/mnt/host/wsl/arcdata/vol1
    type: Directory

pr coming soon


[REEF-258] Writable remote manager

@markusweimer @beysims @jwang98052
This one basically creates WritableRemoteManager.
After this change Wake layer is memory efficient and I will move over to Network Service Layer.
Dhruv

@markusweimer @beysims @jwang98052
I have incorporated RemoteManagerFactory with Writables also. Please have a look. If things are fine please merge.
Dhruv

As we discussed offline, it is probably better to create a new (injectable) Factory for the WritableRemoteManager.

@markusweimer
Any specfic reason why DefaultFactory is in Wake.Impl and not in Wake.Remote.Impl

No. That is a bug. Please move it while you're touching code in that area.

@markusweimer
Pushed the changes. Please merge if things are fine

Looking at it now.

I've done a pass.

@markusweimer
I have done incremental changes suggested by you. Please merge :) ;)

This looks good. I'll merge it.


Document Full List of Docker Labels

I see various Docker swarm service labels used to configure the proxy settings in the tutorials, e.g. com.df.notify com.df.serviceDomain, but I haven't found a list of all the available configuration labels. It would be great to have a quick reference for that.

DFP accepts parameters through HTTP requests. They are documented in http://proxy.dockerflow.com/usage/. DFSL on the other hand, reads service labels and uses them to send requests to other services like DFP.
Please let me know if the list of parameters in usage documentation is what you were looking for.

Closing due to inactivity. Feel free to reopen the issue if you'd like to continue the conversation.


How to reconnect when user closes app?

How to automatically reconnect when user close app and then open app again

https://developer.apple.com/documentation/corebluetooth/cbcentralmanager/central_manager_state_restoration_options
There are some functions to help you use restore identifier on the Central class, check it out


CIS Command not found

Hi Dr. Foster. I am new to compiling COVID FASTQ's from Illumina, and I found your pipeline, and I am hoping to use it.
After downloading the code and installing it (I think), when I went to execute the CIS, I got a Command Not Found. I am thinking I need to change the $PATH in my local installation, but I am not sure where to point the path. I've included a screenshot below this message.
Any thoughts, advice, directions, pointing out what I did wrong is very apprecite.
Thanks,
Brian Delavan

Hi Brian,
I have a feeling you might have missed one of the steps during installation. The steps in order are:

Clone the repository and cd into it
Create the conda environment: conda env create -f environment.yml
Activate the conda environment: conda activate CIS
Install the CIS pipeline as a 'site package' within the environment: pip install .

I think you missed step number 4. Can you try running that command and see if it fixes the installation?
Cheers,
Charles

Hi Dr. Foster. I’ll try it as soon as I can.
Thanks for the quick response. I appreciate it.
Brian
From: charlesfoster @.>
Sent: Thursday, May 12, 2022 5:46 PM
To: charlesfoster/covid-illumina-snakemake @.>
Cc: Brian Delavan @.>; Author @.>
Subject: Re: [charlesfoster/covid-illumina-snakemake] CIS Command not found (Issue #1)
Hi Brian,
I have a feeling you might have missed one of the steps during installation. The steps in order are:

Clone the repository and cd into it
Create the conda environment: conda env create -f environment.yml
Activate the conda environment: conda activate CIS
Install the CIS pipeline as a 'site package' within the environment: pip install .

I think you missed step number 4. Can you try running that command and see if it fixes the installation?
Cheers,
Charles
—
Reply to this email directly, view it on GitHubhttps://github.com/charlesfoster/covid-illumina-snakemake/issues/1#issuecomment-1125485087, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AV36PGWV44RHTIDX7UNFAQDVJWCZ5ANCNFSM5VYXBRBA.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Yup…that fixed it..
@.***
Awesome snake by the way!
Do I close the issue or do you? I can’t recall.
From: charlesfoster @.>
Sent: Thursday, May 12, 2022 5:46 PM
To: charlesfoster/covid-illumina-snakemake @.>
Cc: Brian Delavan @.>; Author @.>
Subject: Re: [charlesfoster/covid-illumina-snakemake] CIS Command not found (Issue #1)
Hi Brian,
I have a feeling you might have missed one of the steps during installation. The steps in order are:

Clone the repository and cd into it
Create the conda environment: conda env create -f environment.yml
Activate the conda environment: conda activate CIS
Install the CIS pipeline as a 'site package' within the environment: pip install .

I think you missed step number 4. Can you try running that command and see if it fixes the installation?
Cheers,
Charles
—
Reply to this email directly, view it on GitHubhttps://github.com/charlesfoster/covid-illumina-snakemake/issues/1#issuecomment-1125485087, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AV36PGWV44RHTIDX7UNFAQDVJWCZ5ANCNFSM5VYXBRBA.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Hi Dr. Foster.
New problem: When I try to run CIS <path/to/fastqs>, I get an error about the database. Any advice and/or thoughts?
P.S. THAT SNAKE IS SO COOL! I LOVE IT!
@.***
From: charlesfoster @.>
Sent: Thursday, May 12, 2022 5:46 PM
To: charlesfoster/covid-illumina-snakemake @.>
Cc: Brian Delavan @.>; Author @.>
Subject: Re: [charlesfoster/covid-illumina-snakemake] CIS Command not found (Issue #1)
Hi Brian,
I have a feeling you might have missed one of the steps during installation. The steps in order are:

Clone the repository and cd into it
Create the conda environment: conda env create -f environment.yml
Activate the conda environment: conda activate CIS
Install the CIS pipeline as a 'site package' within the environment: pip install .

I think you missed step number 4. Can you try running that command and see if it fixes the installation?
Cheers,
Charles
—
Reply to this email directly, view it on GitHubhttps://github.com/charlesfoster/covid-illumina-snakemake/issues/1#issuecomment-1125485087, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AV36PGWV44RHTIDX7UNFAQDVJWCZ5ANCNFSM5VYXBRBA.
You are receiving this because you authored the thread.Message ID: @.***>

Hi Brian,
Glad you like the snake art.
When you experience any problems, it's helpful to copy and paste the error into Github issues for debugging purposes. I'm guessing in this case that you haven't installed a kraken2 database, which is used for analysis of negative controls. You can download a kraken2 database of your choice from here: https://benlangmead.github.io/aws-indexes/k2, then unpack the directory.
To point CIS to your newly downloaded database, use the -b /  --kraken2_db option of CIS, or set the $KRAKEN2_DEFAULT_DB environmental variable, e.g. export KRAKEN2_DEFAULT_DB=/path/to/kraken2_database.
Cheers,
Charles

Dr. Foster.
Thank you for the advice on GITHUB. I am still learning GITHUB, and I appreciate all the advice and comments.
I have CIS working now, thanks to you. I appreciate your help very, very much.
Thanks,
Brian
From: charlesfoster @.>
Sent: Sunday, May 15, 2022 6:59 PM
To: charlesfoster/covid-illumina-snakemake @.>
Cc: Brian Delavan @.>; Author @.>
Subject: Re: [charlesfoster/covid-illumina-snakemake] CIS Command not found (Issue #1)
Hi Brian,
Glad you like the snake art.
When you experience any problems, it's helpful to copy and paste the error into Github issues for debugging purposes. I'm guessing in this case that you haven't installed a kraken2 database, which is used for analysis of negative controls. You can download a kraken2 database of your choice from here: https://benlangmead.github.io/aws-indexes/k2, then unpack the directory.
To point CIS to your newly downloaded database, use the -b / --kraken2_db option of CIS, or set the $KRAKEN2_DEFAULT_DB environmental variable, e.g. export KRAKEN2_DEFAULT_DB=/path/to/kraken2_database.
Cheers,
Charles
—
Reply to this email directly, view it on GitHubhttps://github.com/charlesfoster/covid-illumina-snakemake/issues/1#issuecomment-1127090985, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AV36PGQE3QJTATPRNUWGJD3VKGFTHANCNFSM5VYXBRBA.
You are receiving this because you authored the thread.Message ID: @.***>

Excellent Communication!


feat(floating-label): change literal to constant

related to #827

Codecov Report

Merging #845 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #845   +/-   ##
=======================================
  Coverage   94.47%   94.47%
=======================================
  Files          81       81
  Lines        3475     3475
  Branches      532      532
=======================================
  Hits         3283     3283
  Misses         66       66
  Partials      126      126

Impacted Files
Coverage Δ

packages/floating-label/index.tsx
96.22% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Unable to Login

Was working till morning today, now all of sudden it gives Incorrect Username/Password. Tried changing the password several time but every time got the same above error

Same problem here.

Same here unable
Please fix it bro

Same problem here.

+1

1+

+1
Linux appimage

+1 Windows 7 (x64) too

I think the problem is in udemy api

+1

Thank you all for reporting this. Let me check what is wrong and I will update you shortly.

Thank you sir
Please can we get the version soon enough, cause I need to complete my downloads before this weekend runs off....  Don't have much time on me

Will be glad if done

@Bionic31 Uploading right now.

I have published v1.5.1. Let me know if the problem has been resolved.

Resolved in Win 10. Thank you!

Facing Same Problem Since yesterday on my Windows 10.

Thanks is now working bro
But you could've add to remember on it not anytime we want to access it we need input our details every time

Sir am getting error at a point of the download

What error are you getting? And does this happen for only a particular course?

Thanks Bro It's working

Hi,not working windows 8.1 Please fix it :)

Thanks team ! Issue is now resolved on my laptop (Windows 7) using the latest version 1.5.1

Have fix it bro
Thanks for the care

Login is not working from last 2-3 days, it is saying incorrect username/password but the username and password are working in browser and mobile app.

I have the same issue on Windows 10.

Thank you all for reporting this. Let me check what is wrong and I will update you shortly.
UPDATE
I have identified the cause of this issue and fixed the same. I will soon push the new version and let you know.
For those who want to know what the issue was, actually Udemy has made a slight change in their authentication process where they now have started to check for the origin of the request found in the origin request header. I have the now added the origin header to the login request and hardcoded the same to https://www.udemy.com

Please help to resolve this login issue. I am also unable to login the udemy downloader.


GUACAMOLE-696: In JDBC module, merge effective groups

This pull request overrides the getEffectiveUserGroups() method for the ModeledAuthenticatedUser class such that the user groups are merged between ones passed in from the provider that authenticated the user and the ones present for the same user within the JDBC module.
I don't know if this is the best way to do this, if it fully resolves all of the issues associated with this particular JIRA issue, or if it introduces any adverse behavior, but my testing indicates that it at least clarifies/resolves the issue reported in GUACAMOLE-696.

From what I can tell, retrieval of effective user groups is already guaranteed to produce a set of distinct user groups (no duplicates) as it returns a Set<String>:
https://github.com/apache/guacamole-client/blob/87aa2e6c34d11bb30626c6c27a42796506493ffd/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/base/EntityService.java#L76-L96
Is this not the case?

From what I can tell, retrieval of effective user groups is already guaranteed to produce a set of distinct user groups (no duplicates) as it returns a Set...Is this not the case?

Yes, this is the case; however, the behavior that GUACAMOLE-696 deals with is the following:

User is defined in JDBC module.
User is member of groups in JDBC module.
Groups in JDBC module are assigned permissions to connections, system, etc.
User with matching username authenticates with LDAP
Permissions assigned to the JDBC group are not applied, because no matching group is found in LDAP, and the user did not authenticate through the JDBC module.

What this change does (I think?) is to combine the user groups returned for a user, whether they authenticate through the JDBC module or not, by pulling both the groups for the  AuthenticatedUser (may or may not be JDBC user) and the ModeledUser` object (from the database, matching the name of the logged in user) so that permissions apply to the user from group membership, regardless of whether the user logged in with the JDBC module.
If there's a different/better way I should do this, I'm open to it - this just seems to work and is pretty minimal in terms of changes to accomplish it.

Alrighty. That makes sense to me.


KOGITO-5137: Conscious language initiative

Adjusting our codebase confirming to the conscious language initiative.
@romartin Can you please take a closer look at some constants named ROOT_PATH? They seem like something important :)
Some references to other kiegroup repositories might not work until they update their default branch to main.

Good catch @tiagobento , I see we have a reference in the code to VFS (the ROOT_PATH stuff). It also same thing done in the DMN side.
So @karreiro , you know if we can just drop this code about hardcoded paths in our editors? As far as I see looking at code ,it's only used here [1] in BPMN, and that diagram Metadata I think makes no sense for the kogito stream (it still does on kie-wb stream, but I don't see any usage here)
[1] https://github.com/kiegroup/kogito-editors-java/blob/38cb11270ce519b960f24cc4a2fa98b7e21304ba/kie-wb-common-stunner/kie-wb-common-stunner-sets/kie-wb-common-stunner-bpmn/kie-wb-common-stunner-bpmn-marshalling/src/main/java/org/kie/workbench/common/stunner/bpmn/client/marshall/service/BPMNClientDiagramService.java#L191
WDYT? Can we just drop those references to vfs paths?

@romartin @tiagobento good catch! +1 for removing vfs paths from Kogito stream. I don't see any reason for keeping them neither.

@romartin @karreiro Removed in 70d662b. Please check the commit individually to see if I didn't do anything wrong 😝 .


[21.11] Release bump

Motivation for this change
Release 21.11
Things done
Readme
upgrading-nixos in manual
added date to release notes
To do

cherry pick into master
or

backport this from master to here

Built on platform(s)

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

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

[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

[ ] 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)

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

[x] Fits CONTRIBUTING.md.

since this change causes over 500 rebuilds, do you mind targeting the staging branch
# finding the common merge base will avoid pinging all CODEOWNERs
common=$(git merge-base origin/master origin/staging)
git rebase --onto=$common HEAD~1 # or however many commits you want to pull
git push .. ... --force

then change the base branch in the github PR from master -> staging
See https://nixos.org/nixpkgs/manual/#submitting-changes-staging-branch for more details on staging branch


Unable to view Pull Requests in macOS

I'm not sure if I'm doing something wrong, but I followed the steps to get setup and can't see Pull Requests for any repository.
Pressing the refresh symbol, I see the error command 'pr.refreshList' not found

Let me know if I can provide more information. Also wondering if any other Mac user is having this problem.

After reproducing it once I haven't been able to see it again :/

Thanks @RMacfarlane that fixes this error for me!


update to latest

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

The OS X artifact for arrow-cpp 0.2 hasn't landed yet, so the OS X build here will fail https://anaconda.org/conda-forge/arrow-cpp/files

Actually, I'm not sure. There were some ABI changes so it's possible that the previous arrow-cpp artifact available is ABI compatible

Oops. It will fail because I added a version restriction. Sorry. Should I close this?

Oh neat I guess we got lucky.

@wesm good to go now.


refactor(MutationExecutionStrategy) don't switch strategies during resolution

I found that these lines don't make a difference in the test outcome ... are they used for something?
I found this because I was refactoring SerialExecution in https://github.com/rmosolgo/graphql-ruby/pull/379. I switched from using execution_context.strategy (not reassigned in this code) to using context.execution_strategy (reassigned in this code). After the refactor, I ran the graphql-batch tests against my branch and got a failure:
 1) Failure:
GraphQL::BatchTest#test_mutation_execution [/Users/rmosolgo/projects/graphql-batch/test/batch_test.rb:311]:
--- expected
+++ actual
@@ -1 +1 @@
-{"data"=>{"count1"=>0, "incr1"=>{"value"=>1, "load_value"=>1}, "count2"=>1, "incr2"=>{"value"=>2, "load_value"=>2}}}
+{"data"=>{"count1"=>0, "incr1"=>{"value"=>1, "load_value"=>#<GraphQL::Batch::Promise:0xXXXXXX @state=:fulfilled, @callbacks=[], @value=1>}, "count2"=>1, "incr2"=>{"value"=>2, "load_value"=>#<GraphQL::Batch::Promise:0xXXXXXX @state=:fulfilled, @callbacks=[], @value=2>}}}

I "fixed" the test by ignoring any assignments to context.execution_strategy= after the first (https://github.com/rmosolgo/graphql-ruby/pull/379/commits/497e628380edb47038b23e41b93708ba03e7a590).
Sorry about these bugs from coupling 😩 hoping to take the execution-related concerns "in-house" soon!

Looks like there is a missing test for the lines of code you are trying to remove.
The purpose for those lines is to allow batching for the non-top-level fields in the mutation query.  E.g.
mutation {
  updateProduct(input: $input) {
    product {
      variants {
        image {
          src
        }
      }
    }
  }
}

In this case I only want GraphQL::Batch::Executor.current.wait_all to be called when resolving updateProduct, but I don't want it called when resolving the image or it won't actually batch the loading of the images and cause an N+1.


moved the rss icon

moved the rss icon to the top-right, directly under the header, and aligned with the right of the content

https://github.com/istio/istio.github.io/issues/232


Tram-Lite: update gzipped size and cdn_url

Reduced the listed size for Tram-Lite, and updated the cdn url to be a more generic pointer.
In part, this was a misunderstanding on my part about how assets would be natively gzipped (I didn't know this was a thing 😅), additionally, Tram-Lite now serves a minified asset (next to the default asset).
From testing locally (using the unpkg.com cdn), the size returned is 1.6 kB

@adamghill any chance this update could be merged in, or is there more details / changes that you would want here?

Hey, sorry for missing this before. In my network inspector it says 1.9 kB, but it's close enough for me! :) Thanks for the PR!

Merged and closing, this will get deployed to prod shortly via CI/CD.


Failed to load json schema with self reference in C#

I am trying to self reference json in C# using newtonsoft library:
        String str = @"{""
                               definitions"":{
                              ""Name1"":""Jon Smith"",
                               ""Name2"":""abc"",
                               ""Address1"": {
                               ""$id"": ""1"",
                             ""City"": ""New York"",
                             ""State"": ""NY""},
                             ""AlternateAddress"":{
                               ""$ref"":""#/Address1""}}}";
        Definitions stuff1 = JsonConvert.DeserializeObject<Definitions>(str);
        Console.WriteLine(stuff1.definitions.AlternateAddress);

Expected Behaviour:
Alrenate address should be same as address
Actual Behaviour:
The alternate address here is coming out to be null.
Also, if I want to reference Name2 in Name1, and I do not find a way to directly do this, by not adding a id and changing the type of both.

I've come across a similar problem and it seems to depend to the lack of default constructor in the class. See:  https://github.com/JamesNK/Newtonsoft.Json/issues/1284#issuecomment-355552720


Is it possible to have a single line for 2 events starting/ending on half a day?

Is it possible to have an event starting at the half of the day and ending at the half of the day, and also have the same event on 1 line?  See image(design) for example:

In context:
We have a calendar overview where you can see which holiday home has been booked when and by whom.
When you look at the image you can see that Villa Bona Vista(first event in pink) has been booked from the 26th untill the 30, there is another booking for the same house (Villa Bona Vista) from the 30th untill the 4th(second row).
This is what we have right now:

Here you can see that the two pink events are not aligning on the same row.

I get the idea, but unfortunately this type of event layout is not supported. Perhaps the not yet implemented resource horizontal view will be able to display something similar to what you need. But for now I can’t give any time estimates on when it will be ready.

Oke, good to know. Thanks for the quick response!


[tele5/nexx] azure-http-* don't work (anymore?), produce HTTP Error 400: Bad Request instead

[x] I've verified and I assure that I'm running youtube-dl 2018.09.26

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

Purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Description of issue
not a ytdl bug per se, it seems azure-http-* now require an extra token or something like that
(other formats, e.g. azure-hls-2685 in this case, still work fine)
$ youtube-dl -f azure-http-2500 https://www.tele5.de/mediathek/filme-online/videos/amityville-horror
[Tele5] filme-online/videos/amityville-horror: Downloading webpage
[Nexx] 1552234: Downloading JSON metadata
[Nexx] 1552234: Downloading m3u8 information
[Nexx] 1552234: Downloading MPD manifest
[Nexx] 1552234: Downloading ISM manifest
=>
ERROR: unable to download video data: HTTP Error 400: Bad Request
thanks!

Other websites using nexx.tv I tried seem to work fine.
When opening the website in Firefox with DASH support disabled, the player shows the message Dieses Video ist geschützt und kann hier nicht abgespielt werden. - The video is protected and cannot be played here.
Maybe this is due to configuration on Tele5's side to only allow DASH streaming.
But then it would make sense to have this option reflected in the JSON API. But there was not anything standing out to me in there.

@mrtnmtth thanks for looking into it

Had a further look into this. The only option that stuck out to me was shouldBeCached which is 0 on Tele5 videos and 1 on all other sites I checked.
To confirm or dismiss my suspicion, I need to find at least one other site with shouldBeCached set to 0.

Any update on this? The problem still persists.


Some Junit test failed when use windows system

some junit test is based on linux file system, but failed when use windows.
If they need to be fixed?

Resources should be loaded using ClassLoader, instead of concatenating paths manually.

Use the Path and Paths classes, or use the JUnit TemporaryFolder class to create a test fixture that is automatically set up and torn down. @coder-zzzz would you like to try pr for this improvement?

@vongosling ok

@coder-zzzz  We are also in the process of making existing test cases hermetic. If you want to fix these flaky tests on Windows, ensure the following guidelines are also followed: https://bazel.build/reference/test-encyclopedia#test-interaction-filesystem


Occasional test failure

I'm occasionally seeing test failures from running flutter test that look like this:
00:08 +310: /Users/pasha/github/st-pasha/flame/packages/flame/test/effects/size_effect_test.dart: SizeEffect can be infinite
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following TestFailure object was thrown running a test:
  Expected: a numeric value within <0.01> of <0.0>
  Actual: <82.29216244576264>
   Which:  differs by <82.29216244576264>
Size is not correct

When the exception was thrown, this was the stack:
#0      fail (package:test_api/src/frontend/expect.dart:153:31)
#1      _expect (package:test_api/src/frontend/expect.dart:148:3)
#2      expect (package:test_api/src/frontend/expect.dart:57:3)
#3      expectVector2 (package:flame/src/test_helpers/expect_vector2.dart:11:3)
#4      effectTest (file:///Users/pasha/github/st-pasha/flame/packages/flame/test/effects/effect_test_utils.dart:61:5)
<asynchronous suspension>

The test description was:
  SizeEffect can be infinite
════════════════════════════════════════════════════════════════════════════════════════════════════
00:08 +311 -1: /Users/pasha/github/st-pasha/flame/packages/flame/test/effects/size_effect_test.dart: SizeEffect can be infinite [E]
  Test failed. See exception logs above.
  The test description was: SizeEffect can be infinite

00:08 +327 -1: /Users/pasha/github/st-pasha/flame/packages/flame/test/effects/sequence_effect_test.dart: SequenceEffect can alternate and be infinite
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following TestFailure object was thrown running a test:
  Expected: <true>
  Actual: <false>
Angle is not correct

When the exception was thrown, this was the stack:
#0      fail (package:test_api/src/frontend/expect.dart:153:31)
#1      _expect (package:test_api/src/frontend/expect.dart:148:3)
#2      expect (package:test_api/src/frontend/expect.dart:57:3)
#3      expectDouble (package:flame/src/test_helpers/expect_double.dart:9:3)
#4      effectTest (file:///Users/pasha/github/st-pasha/flame/packages/flame/test/effects/effect_test_utils.dart:56:5)
<asynchronous suspension>

The test description was:
  SequenceEffect can alternate and be infinite
════════════════════════════════════════════════════════════════════════════════════════════════════
00:08 +327 -2: /Users/pasha/github/st-pasha/flame/packages/flame/test/effects/sequence_effect_test.dart: SequenceEffect can alternate and be infinite [E]
  Test failed. See exception logs above.
  The test description was: SequenceEffect can alternate and be infinite

00:08 +332 -2: Some tests failed.

I'm going to try to get to the bottom of this, but posting this as an issue just to keep track.

This happens when the time for an effect overflows and it isn't alternating and it is infinite, since it goes back to the start of the effect when that happens. We could also start using a fixed dt for the update in those tests so that the peak time of the effect is divisible by that dt, then thus shouldn't happen.

Any idea then why this would occur on some runs and not on others?

Any idea then why this would occur on some runs and not on others?

If I rememver correctly dt will differ for each round.

Just ran the tests a hundred times and observed the following failures:
test/effects/combined_effect_test.dart: CombinedEffect can be infinite [seed=1648538475]
test/effects/scale_effect_test.dart: ScaleEffect can be infinite [seed=3997282538]
test/effects/scale_effect_test.dart: ScaleEffect can be infinite [seed=2613699332]
test/effects/sequence_effect_test.dart: SequenceEffect can alternate and be infinite [seed=600606283]
test/effects/size_effect_test.dart: SizeEffect can be infinite [seed=2279967363]
test/effects/size_effect_test.dart: SizeEffect can be infinite [seed=1956599955]

these failures are reproducible with respective seeds.

So, I've looked into this a little, and the problem seems to be due to some roundoff errors when calculating the total time elapsed: the effect class thinks that the total time is just an epsilon bigger than the whole iteration, while the test code thinks that the total time is exactly equal the whole iteration. Thus, the effect rolls over and resets the state, while the test expects the state to be at the end.
There could potentially be several solutions here:

run the test for the total duration of nT - ε, and then check that the final state is close enough (but not equal) to the expected end state;
let the Effect class store total elapsed time, and have the test use that variable to determine how much time have passed;
do not allow discontinuous time jumps in the Effect class. That is, prohibit infinite effects that are not alternating (they probably look visually jarring anyways).

Whatever we choose to do here, it's probably best to wait until #906 is merged in order not to create unnecessary conflicts.

I don't think we should reduce the future set.

Fair point. Just because I can't come up with an example of where it could be useful, doesn't mean nobody else can.

there will be a lot of changes to the effect system in there.

Yep, and there could be some more changes here as well. For example, instead of isInfinite flag we could have numIterations counter, which can support a single iteration, an infinite amount of iteration, or a finite number of iterations. Similarly, instead of isAlternating there could be separate forwardDuration and reverseDuration, which are normally equal, but when the reverseDuration is 0 it would achieve the same effect as isAlternating=false.

Is this still happening after the new test helpers?

I'll check

Yes, it still happens, though it seems like the issue became less frequent. For example, the test "SequenceEffect can alternate and be infinite" fails for seeds<PHONE_NUMBER> and<PHONE_NUMBER> (this is reproducible, at least on my computer).


[Feature Request] use vmlinux from host instead of download

Description
host 1:
Distribution:  Ubuntu
Arch: x86_64
Kernel: 5.15.0-48-generic
host 2:
Distribution:  Ubuntu
Arch: x86_64
Kernel: 6.2.0-1009-azure
On both host 1 and host 2, I found that bpfhack discover always try to download vmlinux-{{kernel-version}} from "https://mirrors.openanolis.cn/coolbpf/btf/". On host 1, bpfhack discover works well, but on host 2, x86_64/vmlinux-6.2.0-1009-azure can not be found in "mirrors.openanolis.cn".
In addition, both host1 and host2 support BTF actually, and the vmlinux file locates /sys/kernel/btf/vmlinux.
So we can see that not all vmlinux file on host format as vmlinux-{{kernel-version}}, I think bpfhack discover should be smarter ~
Proposed Solution
if user assigns the src dir, we check if a filename  called vmlinux or even having the prefix vmlinux- exists in src dir, if so, we copy it instead of downloading. On my host, it will be like that:
bpfhack discover -s /sys/kernel/btf -p /etc/net-exporter/btf

/sys/kernel/btf/vmlinux will be copy to  /etc/net-exporter/btf/vmlinux-5.15.0-48-generic or /etc/net-exporter/btf/vmlinux-6.2.0-1009-azure
I'd be glad to implement this feat.

I didn't notice that https://github.com/alibaba/kubeskoop/blob/289c1c2e8798778e350dd0f63ac0b73faa499675/pkg/exporter/bpfutil/btf.go#L68C22-L68C35, So I think  this issue should be close.


Elapsed time since power-up (controller status)

Maybe start our own timer after getting the timestamp of last message?

resolved with commit 82b6fcf


Unable to view all the team members

On the calendar when I want to add days off. I do not see everyone in the team when selecting

We're closing old issues so we can get back to actively maintaining the extension. Feel free to reopen if this is still an issue for you.


Sabnzb

Hi,
I installed your sabnzb and sonarr addon
EVerything good so far.
However when i try to add the download client into sonarr, it says its not available
I think something is wrong with ip:port
The webui of sonarr has its own ip of homeassistant, and the port 8989
Sabnzb though is unreachable when typing ip of home assistant and port 8080
However the ui is loaded into the home assistant ui , with the home assistant port 8123
How can i make sabnzb to be reachable with the ip and port 8080 so i can point to it in sonarr?
Thx!

You can do it by referencing the add-on's hostname. This is presented on the add-on's info page like so...

You can then just use the standard port of 8989.

Are you also setting the API key? That needs to match what is set in .SABnzbd.

Are you also setting the API key? That needs to match what is set in .SABnzbd.

Yes i copied the api key from sabnzb and put it in sonarr settings, but thats not the problem.
The problem is the ip i dont know what to put in.
Can you give me an example pls?
Normal port of sabnzb  is 8080
So how should the url look like? Cause using the ip of the home assistant device doesnt work

Maybe a screenshot of your settings.
You both have sab and sonarr or radarr running HA OS with the addon right? On same machine...

Using nzbget now without issue


Exception in StrongTypeSystemImpl.functionTypeToConcreteType

I was typing
  AnalysisContext get context => null;

  Index createIndex() {
    return null;
  }

in pkg/analysis_server/test/analysis_abstract.dart.
Somewhere after typing AnalysisContext it throws an exception.
Dart analysis server, SDK version 1.20.0-edge.4d4ebafad3f735b3d69360370b163f4e79d229ba, server version 1.15.0, error: Task failed: VerifyUnitTask for source /Users/scheglov/Source/Dart/sdk.git/sdk/pkg/analysis_server/test/plugin/set_analysis_domain_test.dart
Unexpected exception while performing VerifyUnitTask for source /Users/scheglov/Source/Dart/sdk.git/sdk/pkg/analysis_server/test/plugin/set_analysis_domain_test.dart
#0      AnalysisTask._safelyPerform (package:analyzer/task/model.dart:350:7)
#1      AnalysisTask.perform (package:analyzer/task/model.dart:238:7)
#2      AnalysisDriver.performWorkItem (package:analyzer/src/task/driver.dart:284:10)
#3      AnalysisDriver.performAnalysisTask (package:analyzer/src/task/driver.dart:255:11)
#4      AnalysisContextImpl.performAnalysisTask.<anonymous closure> (package:analyzer/src/context/context.dart:1231:27)
#5      _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212:15)
#6      AnalysisContextImpl.performAnalysisTask (package:analyzer/src/context/context.dart:1229:50)
#7      PerformAnalysisOperation.perform (package:analysis_server/src/operation/operation_analysis.dart:371:37)
#8      AnalysisServer.performOperation (package:analysis_server/src/analysis_server.dart:892:17)
#9      Future.Future.<anonymous closure> (dart:async/future.dart:118)
#10     _rootRun (dart:async/zone.dart:1146)
#11     _CustomZone.run (dart:async/zone.dart:1026)
#12     _CustomZone.runGuarded (dart:async/zone.dart:924)
#13     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:951)
#14     _rootRun (dart:async/zone.dart:1150)
#15     _CustomZone.run (dart:async/zone.dart:1026)
#16     _CustomZone.runGuarded (dart:async/zone.dart:924)
#17     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:951)
#18     Timer._createTimer.<anonymous closure> (dart:async-patch/timer_patch.dart:16)
#19     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:385)
#20     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:414)
#21     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)

Caused by The null object does not have a getter 'isDynamic'.

NoSuchMethodError: method not found: 'isDynamic'
Receiver: null
Arguments: []
#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:44)
#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:47)
#2      StrongTypeSystemImpl.functionTypeToConcreteType.<anonymous closure> (package:analyzer/src/generated/type_system.dart:93:41)
#3      Object&ListMixin.any (dart:collection/list.dart:115)
#4      StrongTypeSystemImpl.functionTypeToConcreteType (package:analyzer/src/generated/type_system.dart:93:23)
#5      StrongTypeSystemImpl.typeToConcreteType (package:analyzer/src/generated/type_system.dart:460:14)
#6      ErrorVerifier._checkForNonAbstractClassInheritsAbstractMember (package:analyzer/src/generated/error_verifier.dart:4731:27)
#7      ErrorVerifier.visitClassDeclaration (package:analyzer/src/generated/error_verifier.dart:465:11)
#8      ClassDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:1847:15)
#9      NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:7688:20)
#10     CompilationUnitImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:2446:21)
#11     RecursiveAstVisitor.visitCompilationUnit (package:analyzer/dart/ast/visitor.dart:1449:10)
#12     ErrorVerifier.visitCompilationUnit (package:analyzer/src/generated/error_verifier.dart:548:38)
#13     CompilationUnitImpl.accept (package:analyzer/src/dart/ast/ast.dart:2439:15)
#14     VerifyUnitTask.internalPerform (package:analyzer/src/task/dart.dart:6454:10)
#15     AnalysisTask._safelyPerform (package:analyzer/task/model.dart:338:9)
#16     AnalysisTask.perform (package:analyzer/task/model.dart:238:7)
#17     AnalysisDriver.performWorkItem (package:analyzer/src/task/driver.dart:284:10)
#18     AnalysisDriver.performAnalysisTask (package:analyzer/src/task/driver.dart:255:11)
#19     AnalysisContextImpl.performAnalysisTask.<anonymous closure> (package:analyzer/src/context/context.dart:1231:27)
#20     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212:15)
#21     AnalysisContextImpl.performAnalysisTask (package:analyzer/src/context/context.dart:1229:50)
#22     PerformAnalysisOperation.perform (package:analysis_server/src/operation/operation_analysis.dart:371:37)
#23     AnalysisServer.performOperation (package:analysis_server/src/analysis_server.dart:892:17)
#24     Future.Future.<anonymous closure> (dart:async/future.dart:118)
#25     _rootRun (dart:async/zone.dart:1146)
#26     _CustomZone.run (dart:async/zone.dart:1026)
#27     _CustomZone.runGuarded (dart:async/zone.dart:924)
#28     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:951)
#29     _rootRun (dart:async/zone.dart:1150)
#30     _CustomZone.run (dart:async/zone.dart:1026)
#31     _CustomZone.runGuarded (dart:async/zone.dart:924)
#32     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:951)
#33     Timer._createTimer.<anonymous closure> (dart:async-patch/timer_patch.dart:16)
#34     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:385)
#35     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:414)
#36     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)

java.lang.Throwable
	at com.intellij.openapi.diagnostic.Logger.error(Logger.java:132)
	at com.jetbrains.lang.dart.analyzer.DartAnalysisServerService$1.serverError(DartAnalysisServerService.java:216)
	at com.google.dart.server.internal.BroadcastAnalysisServerListener.serverError(BroadcastAnalysisServerListener.java:180)
	at com.google.dart.server.internal.remote.processor.NotificationServerErrorProcessor.process(NotificationServerErrorProcessor.java:37)
	at com.google.dart.server.internal.remote.RemoteAnalysisServerImpl.processNotification(RemoteAnalysisServerImpl.java:502)
	at com.google.dart.server.internal.remote.RemoteAnalysisServerImpl.processResponse(RemoteAnalysisServerImpl.java:517)
	at com.google.dart.server.internal.remote.RemoteAnalysisServerImpl.access$600(RemoteAnalysisServerImpl.java:49)
	at com.google.dart.server.internal.remote.RemoteAnalysisServerImpl$ServerResponseReaderThread.run(RemoteAnalysisServerImpl.java:868)

@leafpetersen @jmesserly

Looks like the .type field on a ParameterElement is allowed to be null, and that code (and some other code in that file) assumes that it isn't.  A bit surprising that the field is just null instead of being resolved to dynamic, but it is documented that way.   I'll take a look.

oh dear. elements with null types should not really happen. That's super bad because we rely on ParameterElement having a type in a ton of places.

@bwilkerson Any thoughts?  Given that the documentation explicitly calls out that it might be null I'm hesitant to rely on it being non-null, but it does feel a bit off to me.

so I looked up those docs, that's for VariableElement (and I'm not sure if the doc is still true, but ignoring that). ParameterElement seems to always have a type filled in if you look in resolver.dart. Here's visitSimpleFormalParameter, but the others are similar:
  Object visitSimpleFormalParameter(SimpleFormalParameter node) {
    super.visitSimpleFormalParameter(node);
    DartType declaredType;
    TypeName typeName = node.type;
    if (typeName == null) {
      declaredType = _dynamicType;
    } else {
      declaredType = _typeNameResolver._getType(typeName);
    }
    Element element = node.identifier.staticElement;
    if (element is ParameterElementImpl) {
      element.type = declaredType;
    } else {
      // TODO(brianwilkerson) Report the internal error.
    }
    return null;
  }

(_typeNameResolver._getType returns typeName.type ?? undefinedType so it always returns a non-null value)

and the doc claims:
  /**
   * Return the declared type of this variable, or `null` if the variable did
   * not have a declared type (such as if it was declared using the keyword
   * 'var').
   */

but that's not what the code seems to do anymore, even for variables.

I think it would be better if type were never null anywhere, but I'd have to try to figure out why we allowed it anywhere before I could say it was safe / reasonable.

I found why this happens.
It has nothing to do with strong mode.
Will fix it soon.

https://codereview.chromium.org/2276733003

https://github.com/dart-lang/sdk/commit/23bc00bfc63cbdd599e8075c42de4bb0a420c94c


azure-sdk-for-net PR check in azure-rest-api-specs fails with "permission denied"

The azure-sdk-for net PR check is failing in several PRs in the azure-rest-api-specs repo, e.g.

# 18174
# 18603

@fengzhou-msft  is working on disable .net SDK track1 generation as there is most of failure caused by SDK testing failure and it will be replaced by .net SDK track 2 generation soon.

@fengzhou-msft is working on disable .net SDK track1 generation as there is most of failure caused by SDK testing failure and it will be replaced by .net SDK track 2 generation soon.

The first batch is done with https://github.com/Azure/azure-rest-api-specs/pull/18996
The plan is that we will no longer support Track 1 SDK automation from swagger repo once it shipped a corresponding beta/GA version of Track 2 SDK. Service Team can still generate the Track 1 SDK within the azure-sdk-for-net repo and request exception if they want to release a track 1 SDK.

Track 1 SDK generation failure is resolved by downgrading CI agent to Ubuntu 18.04. A long term solution nis tracked by https://github.com/Azure/azure-sdk-for-net/issues/31839


Fix exception message in DeviceUtils class

Since the DeviceResolvingHandlerInterceptor has been renamed to DeviceResolverHandlerInterceptor, the message should be updated.

@evpaassen thanks! I know it's a minor change, but have you signed the CLA?

Yes, I have contributed in the past.

We appreciate the contribution. Happy Thanksgiving!


feat: initial proposal on how to improve integration chapter

Here is my proposal on how to improve the PACT integration chapter. The relevant changes can be seen rendered in the screenshots below.
I would like to call your attention to:

pCfExcludingBiogenic, fossilGhgEmissions, and fossilCarbonContent (previously completely missing), which all have the same values throughout. The PACT definitions can be found here, here, and here, respectively. Is this ok? (esp. @GabrielaRubioDomingo and @vmatzoros)
biogenicCarbonContent is mandatory (PACT definition here) and therefore my proposal is to set it to "0" for now.
packagingEmissionsIncluded: from what I understood, this should always be false (i.e., also in the case of ShipmentFootprints).

ShipmentFootprint

TOC

HOC

@GabrielaRubioDomingo, thanks for the feedback!
Both fossilGhgEmissions and fossilCarbonContent are mandatory in PACT (please see the links provided for their definitions).
Currently, the iLEAP Tech Specs state the following on how to populate fossilGhgEmissions:

But currently, the iLEAP Tech Specs make no mention of fossilCarbonContent whatsoever, although it is mandatory in PACT.
Therefore, what seemed to me to be the easiest solution, would be to state the same thing for both fossilGhgEmissions and fossilCarbonContent. However, I do not have enough expertise to know if this is a good idea at all...
I believe we should provide some guidance on how to populate fossilCarbonContent, given that it is mandatory.
We can also add a disclaimer about their being updated when the new GLEC version is released, but I thought that applied to biogenic emissions only.

Hi Raimundo,
I think it is important to leave this as to be revised with upcoming GLEC Framework guidance. I am afraid this is linked to the biogenic content. As what is not fossil would be biogenic.
Unfortunately, up until now there is no specific guidance on how to populate those data, so I would make a similar recommendation as with biogenic. (Leave to 0 and revise later)
Sorry for not noticing this before.
Best,
From: Raimundo Henriques @.>
Date: Tuesday, September 3, 2024 at 4:43 PM
To: sine-fdn/ileap-extension @.>
Cc: Gabriela Rubio Domingo @.>, Mention @.>
Subject: Re: [sine-fdn/ileap-extension] feat: initial proposal on how to improve integration chapter (PR #99)
@GabrielaRubioDomingohttps://github.com/GabrielaRubioDomingo, thanks for the feedback!
Both fossilGhgEmissions and fossilCarbonContent are mandatory in PACT (please see the links provided for their definitions).
Currently, the iLEAP Tech Specs state the following on how to populate fossilGhgEmissions:
Screenshot.2024-09-03.at.15.38.55.png (view on web)https://github.com/user-attachments/assets/36ee787d-2352-4477-861a-982d0f96c0fd
But currently, the iLEAP Tech Specs make no mention of fossilCarbonContent whatsoever, although it is mandatory in PACT.
Therefore, what seemed to me to be the easiest solution, would be to state the same thing for both fossilGhgEmissions and fossilCarbonContent. However, I do not have enough expertise to know if this is a good idea at all...
I believe we should provide some guidance on how to populate fossilCarbonContent, given that it is mandatory.
We can also add a disclaimer about their being updated when the new GLEC version is released, but I thought that applied to biogenic emissions only.
—
Reply to this email directly, view it on GitHubhttps://github.com/sine-fdn/ileap-extension/pull/99#issuecomment-2326712510, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFIVUG4EHXVNKRBBDIG65JTZUXDJHAVCNFSM6AAAAABNEK7ME6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRWG4YTENJRGA.
You are receiving this because you were mentioned.Message ID: @.***>

One question:

If what is not fossil is biogenic, then isn't pCfExcludingBiogenic indeed equal to fossilGhgEmissions?

Yes! That is true.
From: Raimundo Henriques @.>
Date: Tuesday, September 3, 2024 at 5:23 PM
To: sine-fdn/ileap-extension @.>
Cc: Gabriela Rubio Domingo @.>, Mention @.>
Subject: Re: [sine-fdn/ileap-extension] feat: initial proposal on how to improve integration chapter (PR #99)
One question:

If what is not fossil is biogenic, then isn't pCfExcludingBiogenic indeed equal to fossilGhgEmissions?

—
Reply to this email directly, view it on GitHubhttps://github.com/sine-fdn/ileap-extension/pull/99#issuecomment-2326806318, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFIVUG4ISLNCHOQHQFGUMNLZUXH7FAVCNFSM6AAAAABNEK7ME6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRWHAYDMMZRHA.
You are receiving this because you were mentioned.Message ID: @.***>

In that case, fossilGhgEmissions was correct.
But by that token, fossilCarbonContent should also be the same as pCfExcludingBiogenic, no?
Here's the PACT definition:

The fossil carbon content of the product (mass of carbon). The value MUST be calculated per declared unit with unit kg Carbon per declared unit (kgC / declaredUnit), expressed as a decimal equal to or greater than zero.

Hi Raimundo,
I think that is a correct assumption! Thanks so much for checking this in detail.
From: Raimundo Henriques @.>
Date: Tuesday, September 3, 2024 at 6:03 PM
To: sine-fdn/ileap-extension @.>
Cc: Gabriela Rubio Domingo @.>, Mention @.>
Subject: Re: [sine-fdn/ileap-extension] feat: initial proposal on how to improve integration chapter (PR #99)
In that case, fossilGhgEmissions was correct.
But by that token, fossilCarbonContent should also be the same as pCfExcludingBiogenic, no?
Here's the PACT definition:
The fossil carbon content of the product (mass of carbon). The value MUST be calculated per declared unithttps://wbcsd.github.io/data-exchange-protocol/v2/#element-attrdef-carbonfootprint-declaredunit with unit kg Carbon per declared unit (kgC / declaredUnit), expressed as a decimalhttps://wbcsd.github.io/data-exchange-protocol/v2/#decimal equal to or greater than zero.
—
Reply to this email directly, view it on GitHubhttps://github.com/sine-fdn/ileap-extension/pull/99#issuecomment-2326895480, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFIVUGYLLQZYI4AGJHSIVM3ZUXMWBAVCNFSM6AAAAABNEK7ME6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRWHA4TKNBYGA.
You are receiving this because you were mentioned.Message ID: @.***>

Hello again, Gabriela,
Thanks for the feedback!
In that case, I think we can safely keep the prior proposal, i.e.,

pCfExcludingBiogenic, fossilGhgEmissions, and fossilCarbonContent should all have the same value.

I think that we can keep the advisements (i.e., the orange boxes) only on the parts concerning biogenic, to avoid creating too much "noise".

Sure, yes, they should have the same value. This is a correct statement. But the problem is, there is no guidance as to which values they should report. So I would keep it as 0 and with the advisement for the moment.
From: Raimundo Henriques @.>
Date: Wednesday, September 4, 2024 at 10:14 AM
To: sine-fdn/ileap-extension @.>
Cc: Gabriela Rubio Domingo @.>, Mention @.>
Subject: Re: [sine-fdn/ileap-extension] feat: initial proposal on how to improve integration chapter (PR #99)
Hello again, Gabriela,
Thanks for the feedback!
In that case, I think we can safely keep the prior proposal, i.e.,

pCfExcludingBiogenic, fossilGhgEmissions, and fossilCarbonContent should all have the same value.

I think that we can keep the advisements (i.e., the orange boxes) only on the parts concerning biogenic, to avoid creating too much "noise".
—
Reply to this email directly, view it on GitHubhttps://github.com/sine-fdn/ileap-extension/pull/99#issuecomment-2328206229, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFIVUG4XJR6FST6FQRBE4E3ZU26NBAVCNFSM6AAAAABNEK7ME6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRYGIYDMMRSHE.
You are receiving this because you were mentioned.Message ID: @.***>

I think I might be missing something here.
If they are the same value as pCfExcludingBiogenic, then, we do have guidance on which values to report, namely, the guidance used to indicate how pCfExcludingBiogenic should be populated. Specifically:

ShipmentFootprint

MUST be set to the total logistics emissions of the shipment, excluding biogenic emissions, calculated by taking the sum of the co2eWTW over all TCEs (tces).

TOC

MUST be set to the logistics emissions intensity of the TOC, excluding biogenic emissions, defined in co2eIntensityWTW.

HOC

MUST be set to the logistics emissions intensity of the HOC, excluding biogenic emissions, defined in co2eIntensityWTW.
Note: The average weight of a container can be considered as 10 tonnes per TEU. For light containers, an estimate of 6 tonnes per TEU is appropriate, while heavy containers can be assumed to average 14.5 tonnes per TEU.

If these are correct for pCfExcludingBiogenic and fossilGhgEmissions and fossilCarbonContent have the same value as pCfExcludingBiogenic, then, we only need to provide guidance for pCfExcludingBiogenic and to state that the other two should be populated with the same value. (And we were already doing this for fossilGhgEmissions. fossilCarbonContent was not mentioned, although it is mandatory.)
Am I missing something?