visual representation of thematic components

An idea regarding the thematic design: a list (grid?) of themes could be provided near the regional map (see #34). Selecting one could highlight the regional MBONs that are relevant for that theme. EG clicking on "plankton" would highlight only MBONs with plankton data.
This would be easier/harder to implement depending on the choice of tech for #34. Using D3/infographiq would be easy.

The following mockup shows one way to approach representing the thematic components alongside the regional components.

In this view the AP MBON has been clicked on and so the details of that regional MBON are shown in the regional info area to the right of the map. Additionally, the eDNA icon has been selected so the details of eDNA are shown below in the thematic info area. Another result of the eDNA selection is that the icon representing eDNA is shown on the locations of the map that have eDNA.


Restore previous successful build on failure

Idea: Use actions/cache/restore action if the build step fails, with a bogus key (none) and partial restore key html-build-<branchname>, which would match the most recent build on the branch. For technical details see https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#matching-a-cache-key
For an example see: https://github.com/TeachBooks/testable-template/actions/runs/8970098300 Build failed, previous version still live at https://teachbooks.github.io/testable-template/feat-20/intro.html
Resolves #25

This requires for sure a notification in the summary as well (linked to #24 ), otherwise users might be confused that changes are not visible in the build book.
What if the caches isn't available any more (they expire right?)?
In this case nothing is deployed (we call this a cache miss, meaning no restore).

I first thought that you might rebuild the previous commit which was succesful (if that one is available). However, book builds might also fail because a package is updated causes inconsistensies. The requirements.txt might still be the same, but the resulting environment might be different. In that case rebuilding with the previous commit won't work either... So I guess when the cache isn't available any more, make it very clear in the summary that this branch is not published (also related to #24 )

OK! the way I see this: we have a column with either Deployed, Build failed (1), Build failed (2) (and a legend with 1) book published, last published version reused, 2) book not published). If you see a better format, let me know.

OK! the way I see this: we have a column with either Deployed, Build failed (1), Build failed (2) (and a legend with 1) book published, last published version reused, 2) book not published). If you see a better format, let me know.

Sounds good!
I think we should add it as text to the root branch line as well, and as an extra column to the aliases table too

Many commits later, here is what an example summary it looks like: https://github.com/TeachBooks/testable-template/actions/runs/9112678442


Initial drop of tabcmd docs and Jekyll files

The markdown files created from the help.tableau.com html files for tabcmd. Plus other static site files for a Jekyll site on GitHub.

Looks good to me. We'll want to update all the install instructions but it's good to have them there as a reference while building the new installer.


Getting exception at checkout if cart rule is created according to weight.

Framework Version: 0.1.6

Framework Version: 0.1.6

Fixed


[SPARK-15457][MLLIB][ML] Eliminate some warnings from MLlib about deprecations

What changes were proposed in this pull request?
Several classes and methods have been deprecated and are creating lots of build warnings in branch-2.0. This issue is to identify and fix those items:

WithSGD classes: Change to make class not deprecated, object deprecated, and public class constructor deprecated. Any public use will require a deprecated API. We need to keep a non-deprecated private API since we cannot eliminate certain uses: Python API, streaming algs, and examples.

Use in PythonMLlibAPI: Change to using private constructors
Streaming algs: No warnings after we un-deprecate the classes
Examples: Deprecate or change ones which use deprecated APIs

MulticlassMetrics fields (precision, etc.)
LinearRegressionSummary.model field

How was this patch tested?
Existing tests.  Checked for warnings manually.

Test build #59321 has started for PR 13314 at commit cae67d6.

LGTM. I will do one more pass to clear up build warnings before the final release so I might be able to clean up stragglers, if any, later

Test build #59321 has finished for PR 13314 at commit cae67d6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/59321/
Test PASSed.

@srowen willing to help with that too btw :)

Thanks!  I'll merge this with master and branch-2.0 then.

I'll also leave the JIRA open to remind us to check for remaining items.


Unable to get "TokenExpiredError"

Below is the verifyOptions I am using to verify the token sent
verifyOptions: { algorithms: ['HS256'], maxAge: 60 }
When using the below token payload, it does not throw the "TokenExpiredError"
{ iss: 'Online JWT Builder',   iat:<PHONE_NUMBER>,   exp:<PHONE_NUMBER>,   aud: 'www.example.com',.....}
@ryanfitz Could you help me fix this?

Invalid issue, by mistake I was using a token of 1yr expiration time. Closing the issue.


Update page_header helper to correct style

This was missed during the initial branding work, and needed fixing.


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add support for KPIs

I would like to propose a feature request: add support for KPIs
in CatalogItems functions
Currently, it is impossible to setup automatic download/upload for this type of objects.

Have you tried using Write-RsRestCatalogItem and Out-RsRestCatalogItem. I believe these commands can download and upload KPIs.

@martkopecky I was wondering if you had any issues with any of the suggest commands above. I will close this issue for now. If you had any issues, please feel free to re-open this issue.


Cache dom ranges for css elements

Feel free to chime in if you want to take another stab at this.


Flownet2

Notes:
(1) Requirements OpenCV-4.4.0 with DNN support needed.
-DWITH_CUDA=ON -DWITH_CUDNN=ON -DOPENCV_DNN_CUDA=ON -DENABLE_FAST_MATH=1 -DCUDA_FAST_MATH=1 -DCUDA_ARCH_BIN=${CUDA_ARCH_BIN} -DWITH_CUBLAS=1
where the ${CUDA_ARCH_BIN} should be the compute capability of the specific gpu which can be found in the wiki.
(2) The input size is currently limited to a fixed size of 448x320.
(3) The speed is around 4 FPS for a single 130-frame video stream. As a comparison, TV-L1 is around 40 FPS (320x240).


DNSDB Source no longer works

What's the problem (or question)?
The endpoint subfinder requests on DNSDB now always responds with an unexpected response due to their CloudFlare implementation
Do you have an idea for a solution?
Not at the moment.
How can we reproduce the issue?

Visit: https://github.com/subfinder/subfinder/blob/master/libsubfinder/sources/dnsdb/dnsdb.go#L32
Make a valid request to the endpoint with curl:

curl -s 'http://www.dnsdb.org/f/yahoo.com.dnsdb.org/' -v | grep '503'

You'll receive the following response: 503 Service Temporarily Unavailable

Hi @C0RB3N,
DNSDB has added CloudFlare as you said that causes  the tool to fail parsing it.
Thanks for notifying us, should be removed soon. Things have been delayed much due to life :)

It looks like dnsdb exposes an API at https://api.dnsdb.info/, but this requires an API key. It might be worth exploring adding support for that similar to the Shodan source. Or dnsdb can just be removed entirely.

Hey, this is getting deleted in the future. Closing the issue now!


usage

Is this rat ready to use right now?
Please explain me how to compile and get APK file using APK Easy Tool, whenever I select the folder and press 'compile' nothing happens

@Arioshoma yes its completed .... u can use now.......Join this discord server https://discord.gg/zKpPTJ7m and ask doubts there


Implement pyro.markov to recycle enum dims

Resolves #1380
Pair coded with @eb8680
This PR aims to resolve the upcoming dimension limits in PyTorch 1.0, which would otherwise break parallel enumeration of many variables, e.g. examples/hmm.py. The solution is to recycle enum dims and use symbolic names to distinguish colliding dims.
Solution part 1 is to add a tiny bit of annotation pyro.markov to user code, which can be done in a variety of ways. The core idea is to provide a windowed dependency arena, so that when variables go out of context they are assumed conditionally independent, and their enumeration dimensions can be reused. We still keep a unique symbolic name for each dim. The symbolic and numeric dims are managed jointly by a new _ENUM_ALLOCATOR, an outermoset EnumMessenger, and zero or more MarkovMessengers that can be nested inside the EnumMessenger.
Solution part 2  The second part is to switch implementation of contraction machinery from UnpackedLogRing to PackedLogRing with named dimensions. This PR also introduces a pyro.ops.packed module to handle a few packed tensor operations outside of the contract_* functions. (I may pull this out into a separate PR if this PR is too large).
Limitations

MarkovMesenger does not work correctly with poutine.block; @eb8680 may add support for block in a later PR.

Tested

[x] added lots of tests to test_valid_models.py
[x] update tests/infer/test_enum.py to use pyro.markov
[x] update examples/hmm.py to use pyro.markov

Ready to merge as soon as tests pass.

@srush as of this PR you should be able to run CRFs and HMMs in CUDA without hitting dimension limits. E.g. I noticed your EMNLP tutorial was limited to 20 time steps, but by changing your HiddenMarkovModel to use pyro.markov you should be able to use any limit:
- for t in range(T):
+ for t in pyro.markov(range(T)):


cannot pick files in arrive during onboarding

Which at_widget package you find issue with?
at_onboarding_flutter
On what platforms did you experience the issue?
iOS 15.5
What version are you using?
at_onboarding_flutter:
    git:
      url: https://github.com/atsign-foundation/at_widgets.git
      path: at_onboarding_flutter

What happened?

I cannot select my key file<EMAIL_ADDRESS>when using arrive app.
Steps to reproduce
1. Clone arrive
2. Run the app via `flutter run` on iOS simulator
3. Try to onboard by selecting your key file
4. Cannot select key file

Additional info
Using iPhone 13 simulator iOS 15.5
macOS monterey 12.5.1
Apple M2

Relevant log output
No response
Flutter analyze output
No response
Flutter doctor output
jeremytubongbanua@Jeremys-M2-Air arrive % flutter doctor -v
[✓] Flutter (Channel stable, 3.3.0, on macOS 12.5.1 21G83 darwin-arm, locale en-CA)
    • Flutter version 3.3.0 on channel stable at /Users/jeremytubongbanua/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffccd96b62 (10 days ago), 2022-08-29 17:28:57 -0700
    • Engine revision 5e9e0e0aa8
    • Dart version 2.18.0
    • DevTools version 2.15.0

[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/jeremytubongbanua/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13F100
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.71.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (3 available)
    • iPhone 13 (mobile) • 9F15E60D-240D-493C-8B30-CCD4E701AE8D • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-5 (simulator)
    • macOS (desktop)    • macos                                • darwin-arm64   • macOS 12.5.1 21G83 darwin-arm
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 105.0.5195.102

[✓] HTTP Host Availability
    • All required HTTP hosts are available

@JeremyTubongbanua Looks like specific top M2 chips. So I have tried this with Cynthia last time in M1 and on mine too and it worked like charm with the same configurations.

Okie I tried with M1, but it still can't pick.

https://user-images.githubusercontent.com/56836874/193356734-95e63f7e-43d3-4da4-9785-adb7ca1936a9.mp4
All you need to do is to long click on the file to get an options dialog, select "get info" and then click on the file.

@stanezeaku Yes, That may or may not work sometimes. And there were fixes made and going to be merged ASAP.

@yahu1031  alright, I will follow up with your fixes.

Thanks @XavierChanth

Pretty sure this one was solved by the file_saver fork (at_file_saver) this can be reopened if not the case.


Added queryType to options for more possibilities

I added the possibility to provide a 'queryType' in the options to also split a css file not just by media queries, but also by other queries (e.g. 'supports'). This is pretty handy to easily split a big css file into smaller files.

Maybe I should try this again with my correct branch as the head. Sorry, I'm new to GitHub and pull requests.


Added Transaction Hash Upgrade note + minor edits

Related to https://issues.trusted.visa.com/browse/WEB-5478

Canceling this PR and the Transaction Hash Upgrade note should be in SDK and not in the Sample Codes.


[firebase_storage] my app gives error and do not open when I add this package to pubspec.yaml

So I am building an app and I want to save some photos of the user in the firebase DB. For this purpose, i added the package however, when i add this package and run the app, the app gives error and do not open.
Here is the error;
E/eglCodecCommon( 2714): glUtilsParamSize: unknow param 0x000082da
E/eglCodecCommon( 2714): glUtilsParamSize: unknow param 0x000082da
E/flutter ( 2714): [ERROR:flutter/lib/ui/ui_dart_state.cc(198)] Unhandled Exception: PlatformException(channel-error, Unable to establish connection on channel., null, null)
E/flutter ( 2714): #0      FirebaseCoreHostApi.initializeCore (package:firebase_core_platform_interface/src/pigeon/messages.pigeon.dart:205:7)
E/flutter ( 2714): <asynchronous suspension>
E/flutter ( 2714): #1      MethodChannelFirebase._initializeCore (package:firebase_core_platform_interface/src/method_channel/method_channel_firebase.dart:29:44)
E/flutter ( 2714): <asynchronous suspension>
E/flutter ( 2714): #2      MethodChannelFirebase.initializeApp (package:firebase_core_platform_interface/src/method_channel/method_channel_firebase.dart:73:7)
E/flutter ( 2714): <asynchronous suspension>
E/flutter ( 2714): #3      Firebase.initializeApp (package:firebase_core/src/firebase.dart:42:31)
E/flutter ( 2714): <asynchronous suspension>
E/flutter ( 2714): #4      main (package:falabella/main.dart:31:3)
E/flutter ( 2714): <asynchronous suspension>
E/flutter ( 2714):

For me, downgrading to firebase_core_platform_interface to 4.4.1 resolves the issue

FirebaseCoreHostApi.initializeCore (package:firebase_core_platform_interface/src/pigeon/messages.pigeon.dart:205:7)

@ysahin20 What version of firebase_core and firebase_core_platform_interface are you using ?
Can you try upgrading firebase_core to latest and firebase_core_platform_interface and see if these help ?
https://pub.dev/packages/firebase_core_platform_interface/changelog

@darshankawar My firebase_core version is 1.12.0 on pubspec.yaml but i do not have the firebase_core_platform_interface. I will try adding firebase_core_platform_interface now.

Well, when i was trying to upgrade a specific plugin, i accidently upgrade all of the plugins and then the code startes without error. Most probably, the version of the firebase_core was the problem. Anyway my problem has solved. Thanks for your comments and suggestions.

Still getting this error, even if I update all firebase packages to last version, and also if I pin firebase_core_platform_interface to 4.4.1 (note that it is dependency_overrides).


⚠ Samsung has degraded performance

In e6f9e00, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2826 ms

Resolved: Samsung performance has improved in 6163c60.


bug(MatMenu): dropdown with menu items pops over menu trigger when there's a scroll bar in the dropdown

Reproduction
Steps to reproduce:

Go to the following StackBlitz URL https://stackblitz.com/edit/angular-xcgrfw
Click on the "Menu" button
Notice how menu items pop over the "Menu" button

Expected Behavior
The expected behaviour is for the menu dropdown to be positioned below the "Menu" button regardless of the number of menu items as in the following example https://stackblitz.com/angular/mnygrqvpejk?file=src%2Fapp%2Fmenu-overview-example.ts
Actual Behavior
The dropdown with menu items pops over the "Menu" button when there's so many menu items that the menu becomes scrollable
Environment

Angular: 9.1.11
CDK/Material: 9.2.4
Browser(s): Google Chrome 83.0.4103.97
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10

@RedasP the StackBlitz you posted appears to be working as expected. Is this the right StackBlitz?

Thanks, I've updated the StackBlitz https://stackblitz.com/edit/angular-xcgrfw

@crisbeto I debugged this one and found the bug- the logic for determining whether something fits with flexible dimenions has this check:
      const verticalFit = fit.fitsInViewportVertically ||
          (minHeight != null && minHeight <= availableHeight);

The problem is that the menu doesn't have a minHeight set, so it never ends up going flexible when it doesn't otherwise fit. This make the position push up.
I think the right thing to do is to change the condition to this:
      const verticalFit = fit.fitsInViewportVertically ||
          (minHeight == null || minHeight <= availableHeight);

We could alternatively, set a minHeight for MatMenu to 48px (the size of one item) to fix this specific issue, but I do think that the position strategy should shrink the overlay in general if no minHeight is given.


Fix after user registration, the page does not redirect to checkout page...

... as url requested bug.
Happens when an anonymous user buying and checking out, the user will be asked to login or register. After the registration, the page will be redirected to account profile page instead of finishing the checkout process.

Thanks! :-)

I just added OSCAR_ALLOW_ANON_CHECKOUT = True in my setting.py file, even after it redirects me to the login page when I click Proceed to Checkout button, I don't understand what is wrong with this, some one please help me out

@MARUTHESH please stop spamming issue threads with support requests. Github issues are not used for support - please use the mailing list for that. Thanks.


 This happend to me when I introduced a typo on one of the store and the entire app crashed.
I was also getting errors from a component that I wasn't even using/importing on any module component. This was very difficult to troubleshoot.
Something along the lines of  store prop is undefined would be helpfull.

Hi Rodry,
That's quite a general statement. Could you elaborate a bit with a small example where you made an error and in which kind of exception it resulted? If you assigned a value to the wrong property somewhere, not that random field assignments can simple not be detected in JS (until proxies are standardized). The best way to prevent such mistakes is by using a static typechecker like flow or typescript (which I really recommend)


Diagram tool for Unity

Hi,
I recently started using IOC and more specifically Unity. As my project grows I think I am getting lost on which has dependency to what.
Is there any tool that analyzes the code and shows the dependency as a diagram or something equivalent?
Thanks

Hi,
the container stores it mappings under the Registrations attribute. Each Registration consists of Name, RegisteredType and MappedToType. You can write a simple dump method that iterates through Registrations and logs beforehand mentioned attributes.
Example:
foreach (var item in container.Registrations) {
var regType = item.RegisteredType.Name;
var mapTo = item.MappedToType.Name;
var regName = item.Name ?? "[default]";
var lifetime = item.LifetimeManagerType.Name;
Logger.Info ($"{regName}: {regType} -> {mapTo}. {lifetime}");

}
Infos about lifetime -> lifetime

@moerwald  thanks for the reply.
I was looking for something which shows the dependency injection tree.
e.g I have following:
IType1, Type1
IType1, Type1A
IType2, Type2
IType3, Type3
IType4, Type4
Type3 has in its constructor IType2 (Type2), IType1 (Type1A)
Type4 has in its constructor IType3
I was looking for a tool that shows the above.
On large projects with hundreds of Contracts it is difficult to keep track the above

A good candidate for an Extension but I don't think I'll have time


Add an explicit test for nested parenthesis

@thysultan this still might be worth merging in as an explicit test for this case, WDYT?

Yes.


JUnit Reports Merger

Is there a way or feature in karate to combine all the executed reports to merge into one final html report.
We have to rerun failed tests multiple times till all tests passed and wanted to combined all into one report :

please ask questions like this on stack overflow. see if this thread helps: https://stackoverflow.com/q/53912893/143475


Support Gradle 7

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
Support Gradle 7
Issues Resolved
Closes https://github.com/opensearch-project/OpenSearch/issues/1246
Check List

[X] New functionality includes testing.

[X] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock finally got it green :D , the Gradle_Wrapper_Validation fails because of Gradle dist changes, otherwise everything seems to be set

@dblock Thanks a lot for the hint, updated Gradle Wrapper bits as well

start gradle check

@dblock Thanks a lot for the hint, updated Gradle Wrapper bits as well

FYI I opened an issue in that private repo to move this workflow into OSS

I've merged this. @dlvenable need your eyes on this to help with helping other plugins on https://github.com/opensearch-project/opensearch-plugins/issues/107 - maybe instructions, etc.

@reta We have an auto-backport mechanism that allows to create backport PRs based on the labels. But since the backport workflow didn't run on this PR with the other checks, this PR would have to backported manually to 1.x.

@VachaShah thanks, on it :)

@reta care to take a look at https://github.com/opensearch-project/opensearch-build/runs/4361547117?check_suite_focus=true please?
Please consult deprecation warnings for more details.
* What went wrong:
795 actionable tasks: 795 executed
Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0

@dblock ah ... I know what is the issue, will send the PR in a sec (old plugin):
> Task :build-tools:publishPluginMavenPublicationToMavenLocal
Execution optimizations have been disabled for task ':build-tools:publishPluginMavenPublicationToMavenLocal' to ensure correctness due to the following reasons:
  - Gradle detected a problem with the following location: '/tmp/tmptny3_u2s/OpenSearch/buildSrc/build/distributions/build-tools-2.0.0-SNAPSHOT.pom'. Reason: Task ':build-tools:publishPluginMavenPublicationToMavenLocal' uses this output of task ':build-tools:generatePomFileForNebulaPublication' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed. Please refer to https://docs.gradle.org/7.3/userguide/validation_problems.html#implicit_dependency for more details about this problem.
Gradle detected a problem with the following location: '/tmp/tmptny3_u2s/OpenSearch/buildSrc/build/distributions/build-tools-2.0.0-SNAPSHOT.pom'. Reason: Task ':build-tools:publishPluginMavenPublicationToMavenLocal' uses this output of task ':build-tools:generatePomFileForNebulaPublication' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed. Please refer to https://docs.gradle.org/7.3/userguide/validation_problems.html#implicit_dependency for more details about this problem. This behaviour has been deprecated and is scheduled to be removed in Gradle 8.0. Execution optimizations are disabled to ensure correctness. See https://docs.gradle.org/7.3/userguide/more_about_tasks.html#sec:up_to_date_checks for more details.
Multiple publications with coordinates 'org.opensearch.gradle:build-tools:2.0.0-SNAPSHOT' are published to repository 'mavenLocal'. The publications will overwrite each other!


Can't enable greenkeeper.

Hello,
I want to try with one private repo. After login successfully, I enable greenkeepr  by greenkeeper enable . However, it show the error .
info enable The GitHub slug is: XXXX/XXXX
🌴  info enable Synchronizing your repositories. This might take a while.
🌴  ERR! enable Couldn’t enable a repository with this slug.
🌴  ERR! enable If you want to try your free private repository make sure to grant the necessary rights by running greenkeeper login --force --private
🌴  ERR! enable You have to have a plan for more than one private repository. To verify run greenkeeper whoami
🌴  ERR! enable You need admin access to a repository to enable it.
🌴  ERR! enable If you think this error really shouldn’t appear let us look into it with greenkeeper support

Please help to check it.

Hi @vankhoa011,
have your tried running greenkeeper login --force --private?  This way you can try your private repository.
Let me know if this helped :)

Hello ,
I tried this command "  greenkeeper login --force --private" . However, the
problem is same.
Let me tell more detail.
I work in ABC company. My leader granted admin permission for
ABC/ourcompany_repo .
I logged in github with vankhoa011 account. And I want to enable
ABC/ourcompany_repo by greenkeeper.
Regards,
Khoa Nguyen
On Mon, Apr 11, 2016 at 8:55 PM, Finn Pauls<EMAIL_ADDRESS>wrote:

Hi @vankhoa011 https://github.com/vankhoa011,
have your tried running greenkeeper login --force --private? This way you
can try your private repository.
Let me know if this helped :)
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/greenkeeperio/greenkeeper/issues/165#issuecomment-208354938

--
Thanks and Regards,
Khoa Nguyen.

@finnp: Hello .  Any update on this problem ?

Hey @vankhoa011,
unfortunately you need admin permission to enable a repository. See this issue https://github.com/greenkeeperio/greenkeeper/issues/46 for details of this limitation.
I hope someone in your organization can grant you these rights, or take care of the enabling.
Best,
Stephan

@boennemann  : I had admin permission but still can't enable . :(

@boennemann : Any updated ?

@vankhoa011 We're not sure what's the problem here. It is likely that there is some issue with the permissions. Can you do greenkeeper login --private --force again?
Then you should see a list of you organisations, if there is a button Grant access next to the organisation, it means greenkeeper doesn't have access to it yet.
If you see that button, click it and then finally click Authorize Application.
If that doesn't help, let us know.

@finnp : I worked like a charm. Thanks you very much.


spelling fix

"transisting" isn't a word.  It's "Kerberos Constrained Delegation and Protocol Transition"

Looks good; thanks, @dbrownems


Feature: Extract JSON from Empire page

Codecov Report

Merging #44 into master will decrease coverage by 0.18%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #44      +/-   ##
==========================================
- Coverage   44.24%   44.06%   -0.19%
==========================================
  Files         108      108
  Lines        8812     8848      +36
==========================================
  Hits         3899     3899
- Misses       4807     4843      +36
  Partials      106      106

Impacted Files
Coverage Δ

extracts.go
100% <ø> (ø)
:arrow_up:

extracts_v6.go
80.21% <0%> (-0.54%)
:arrow_down:

prioritize.go
0% <0%> (ø)
:arrow_up:

ogame.go
5.45% <0%> (-0.03%)
:arrow_down:

extractor_v6.go
66.08% <0%> (-0.47%)
:arrow_down:

handlers.go
0% <0%> (ø)
: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 f89bbd8...5ff5c9a. Read the comment docs.

make your PR against the "develop" branch.
Also, I'm not sure what's the json returned by the endpoint, can you create a struct that represent it ?
You can use: https://mholt.github.io/json-to-go

make your PR against the "develop" branch.

I've updated it

Also, I'm not sure what's the json returned by the endpoint, can you create a struct that represent it ?

That struct is 600+ lines long. Let me know if you really want to transform it into such struct then I'll push that change.


Allow unknown transaction response size in python bindings

When using python bindings, there is no way to perform a csp_transaction without knowing the response size beforehand. This patch adds an optional argument, to pycsp_transaction that allows it to accept responses of any size.

Thanks for your contribution.
If I understand correctly, this is an optional argument, and so it should not break the python API?

@johandc that's correct, the argument is optional. If not supplied, the API works in the same way as before
Thanks for merging!


Is it feasible to use CH347 to drive the SPI interface screen?

Sorry for thinking about it, I want to know if CH347 supports three-wire SPI mode (9-bit data)?

I think ch347 support only standard 4-wire SPI mode, you can always check whether some kind of feature availiable in their chip datasheet.

@i2cy It seems that 9bit is not supported
It should be able to use LCD four-wire SPI (CLK, MOSI, CS, DC)?


vm: add new opcodes PUSHT & PUSHF

to align with https://github.com/neo-project/neo-vm/pull/497

Coverage: 80.973% (+0.007%) from 80.967% when pulling 22de9f79deeeff59f6962bac003b9f9fc8837376 on new-opcodes into 12a3ec01f062576cb273636e7a59389ba44d4f5f on master.


Tempotec Sonata HD DAC support

Does the project support this DAC?

If it’s just a driverless USB DAC, I believe it should be supported.

Looks like it's USB based so it should be supported.


UI feedback of 'things happening' is poor

There is no visual concept of locker working in the background to download new data, which is disconcerting to the user.
For example, I can add 4Sq service, and a popup starts to load, and then disappears, and thats it. If I click on photos, no photos- and it is unclear what has happened. (Of course, if I wait 5 secs for locker to start downloading pictures they appear fine.)
Some sort of cue is need for the initial connection, and it might be nice to subtly indicate a refresh of info - something like a spinner spinning on top of the service icon, or a slow fade in and out?

I want to make sure we look for this as we polish the experience, so I'm adding it to MEGA.

Thoughts on this?

and then

We can show the data types w/ icons rather than text, too, which will look prettier for sure.

I like the look of this. A few thoughts-  Provides: instead of Gets:?
Would be nice to have a spinner or something next to the current services that are being installed I think, just to reassure the user that things are happening in between percent complete updates.
The colour of the buttons is tricky IMO- Green is cool for "go! connect your services!" But then they turn grey when connected, which feels a bit like it failed, but then it does say connected, and says what it's installing..
Maybe have them blue to start when they say CONNECT, grey and CONNECTING, ie installing and downloading data from the service, and then green when they say CONNECTED, all data has been downloaded and service is installed and working properly?
This also has room for a red or orange colour for an error state, which could be used for a data error or problem. People on IRC mentioned a profile page with an indication of which services you have already installed etc would be good, this would then maybe also be useful for this?
A connect all button bellow the connected buttons might be nice too, to add all the services with one click.

Here's the account settings project issue: https://github.com/LockerProject/Locker/issues/613

And here's the button states: https://github.com/LockerProject/Locker/issues/630

I feel there's an issue of using any actual progress meter on this page.  It will cause people to stay on this page until it is all the way done.  As soon as data starts syncing the locker is usable, so we want them to quickly move through adding as many services as they want and then get to the exploration phase.  There should definitely be feedback, but not anything that makes them feel restricted.  I was also wondering if there should be anything different in the bottom left corner of the You page until full completion?

Okay great. Splitting this into a different issue for implementation since this covered a bunch of issues. #680 will cover the new account experience.


Improve performance of randomized message queueing

Original issue: https://charm.cs.illinois.edu/redmine/issues/1424

Randomized queue ordering is a great debugging trick to expose race conditions in program logic. However, its implementation makes the cost of dequeueing each message essentially linear in the number of messages. This can trivially be made constant time, by just flattening the structure, picking one message position at random, and swapping the last element into its place. No need to increment iterators arbitrarily or shift elements around.

Original date: 2017-02-17 18:20:43

~~https://charm.cs.illinois.edu/gerrit/2258~~ https://github.com/UIUC-PPL/charm/commit/bb776112d5afd00041bf86cec143eb4f7a0259bb


[Linux, Deb] Cannot start SubutaiTray

I installed subutai-p2p and subutai-tray. Starting subutai-tray gives this:

This should be easy to fix. @Lezh1k is this due to something being missing in deb package?

Changed build to static. Fixed.


postgresql partition table insert id in Eloquent model Undefined offset: 0

Laravel Version: 5.6
PHP Version: 7.2
Database Driver & Version: postgresql 10

Description:
I used the postgresql partition table, when i save a model, i got the notice and can't get the insert id.
PHP Notice:  Undefined offset: 0 in /laravel/framework/src/Illuminate/Database/Query/Processors/PostgresProcessor.php on line 20
and the saved event doesn't fired. but if i using
 DB::getPdo()->lastInsertId(); , I can get the id.
How do i avoid the ?
Now, I want to extend the insertGetId of Builder, but i don't know how to do it ?
I wrote some code like this:
`trait PgsqlPartitionSave
{
public function save(array $options = [])
{
$saved = false;
try {
$saved = parent::save($options);
} catch (\ErrorException $e) {
if (strpos($e->getMessage(), 'Undefined offset: 0') === false) {
throw $e;
}
$saved = true;
}
if ($saved && !$this->id) {
$this->exists = true;
parent::finishSave($options);
$this->id = (DB::select("select currval('{$this->getTable()}_id_seq') as id"))[0]->id;
}
return $saved;
}
}`
but i'm not sure if there will be any bad effects. and i think it's not a good way.
Steps To Reproduce:

make a set of partition tables
using a model->save()

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? If so feel free to reply and we'll try to have a look.
Thanks!


Some pages in  the feature-app sample is  broken.

Following pages in the feature-app sample is broken.
/features-app/osgi/call-class
/features-app/session

Duplicate of https://github.com/wso2/carbon-uuf/issues/170


Upgrade Maven to 3.9.2

Perfect, thanks!

Thanks for maintaining this awesome container over the years! Saved me a lot of trouble

No probs, glad that it has proved useful!


release-1.20: Bump Envoy to 1.21.3

ignoring codegen failures, since they come from the release branch pulling newer CRDs than it should


Updates to produce new, better phenotype comparisons

This currently will generate the IC maps but won't yet pass them to the call to runoak similarity

On the Jenkins run, the line for building the custom IC map fails due to an SQL error, as below:
09:33:34  + runoak -g hpoa.tsv -G hpoa -i sqlite:obo:phenio information-content -p i --use-associations .all
12:32:40  sqlite3.OperationalError: too many SQL variables
12:32:40
12:32:40  The above exception was the direct cause of the following exception:
12:32:40
12:32:40  sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) too many SQL variables
12:32:40  [SQL: SELECT term_association.id AS term_association_id, term_association.subject AS term_association_subject, term_association.predicate AS term_association_predicate, term_association.object AS term_association_object, term_association.evidence_type AS term_association_evidence_type, term_association.publication AS term_association_publication, term_association.source AS term_association_source
12:32:40  FROM term_association
12:32:40  WHERE term_association.object IN (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,

This may require sqlite to be updated on the build environment.
See also https://github.com/deepset-ai/haystack/issues/588

With changes in https://github.com/INCATools/ontology-access-kit/pull/764 I expect this to work better - will just need to pin OAK version to github

Will test after merging https://github.com/INCATools/ontology-access-kit/pull/759

Getting Jenkins errors on test - unclear why:
Error when executing always post condition:
Also:   org.jenkinsci.plugins.workflow.actions.ErrorAction$ErrorId: 7c6506e2-fc79-47e8-b60a-bfd6d52143dd
org.jenkinsci.plugins.workflow.steps.MissingContextVariableException: Required context class hudson.FilePath is missing
Perhaps you forgot to surround the code with a step that provides this, such as: node
	at org.jenkinsci.plugins.workflow.steps.CoreStep$Execution.run(CoreStep.java:89)
	at org.jenkinsci.plugins.workflow.steps.CoreStep$Execution.run(CoreStep.java:70)
	at org.jenkinsci.plugins.workflow.steps.SynchronousNonBlockingStepExecution.lambda$start$0(SynchronousNonBlockingStepExecution.java:47)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)

The Jenkins error was due to string interpolation in environment variable names not working as expected.

Getting a ModuleNotFoundError: No module named 'semsimian' because semsimian is optional dependency - fixing

Good news: IC map is produced quickly and without errors!
Bad news: Something (OAK or Semsimian) is raising an error:
10:13:21  + runoak -i semsimian:sqlite:obo:phenio similarity --no-autolabel --information-content-file hpoa_ic.tsv -p i --set1-file HPO_terms.txt --set2-file HPO_terms.txt -O csv -o HP_vs_HP_semsimian_-20240516.tsv --min-ancestor-information-content 4.0
10:13:39  Loading custom IC map from: "hpoa_ic.tsv"
10:13:39  Failed to import custom IC map: Error parsing IC value: invalid float literal

That error may be a false positive - when I run a single pair locally and pass the IC map, I don't get that error:
$ runoak -vvv -i semsimian:sqlite:obo:phenio similarity --no-autolabel --information-content-file hpoa_ic.tsv -p i  -O csv -o test.tsv --min-ancestor-information-content 4.0 HP:
0007166 @ HP:0007153
INFO:root:Setting other_languages=()
INFO:root:Settings = Settings(impl=None, autosave=False, associations_type=None, preferred_language=None, other_languages=())
INFO:root:Wrapping an existing OAK implementation to fetch sqlite:obo:phenio
INFO:root:Locator: obo:phenio
INFO:root:Ensuring gunzipped for https://s3.amazonaws.com/bbop-sqlite/phenio.db.gz
INFO:root:Locator, post-processed: sqlite:////home/harry/.data/oaklib/phenio.db
DEBUG:root:Paths to search: [PurePosixPath('model/schema'), PurePosixPath('schema'), PurePosixPath('linkml'), PurePosixPath('src/linkml'), PurePosixPath('src/model'), PurePosixPath('src/model/schema'), PurePosixPath('src/schema'), PurePosixPath('.')]
DEBUG:root:candidate model/schema not found
DEBUG:root:candidate schema not found
DEBUG:root:candidate linkml not found
DEBUG:root:candidate src/linkml not found
DEBUG:root:candidate src/model not found
DEBUG:root:candidate src/model/schema not found
DEBUG:root:candidate src/schema not found
INFO:root:out=test.tsv <class 'str'>
INFO:root:file=<_io.TextIOWrapper name='test.tsv' mode='w' encoding='UTF-8'> <class 'str'>
INFO:root:Splitting terms ['HP:0007166', '@', 'HP:0007153'] on 1
INFO:root:Calculating all-by-all pairwise similarity for 1 objects
[00:00:09] Building closure and IC map: ████████████████████████████████████████ 100%                                                                                INFO:root:Post-processing results from semsimian

This is the relevant function in semsimian:
https://github.com/monarch-initiative/semsimian/blob/e2cba82624dc0633092e0b299d80f3771d01659c/src/utils.rs#L579-L607
Maybe it's just trying to parse EOF? Seems unlikely as there isn't a tab in the last line.
Oh. Wait. No. It's the header.
Semsimian doesn't expect it.
Not sure why that error isn't being raised locally but I'll put a fix for it in here.

Looks like the map is now loading correctly, at least for HP.
12:14:28  + runoak -i semsimian:sqlite:obo:phenio similarity --no-autolabel --information-content-file hpoa_ic.tsv -p i --set1-file HPO_terms.txt --set2-file HPO_terms.txt -O csv -o HP_vs_HP_semsimian_-20240516.tsv --min-ancestor-information-content 4.0
12:14:46  Loading custom IC map from: "hpoa_ic.tsv"
12:14:46  Custom IC map imported successfully.
12:14:55  Warning: The following keys are present in closure_map but not in ic_map:

This is followed by a list of more than 20K CURIEs, including 5952 HP IDs.
Might be excessive to write that to stdout.

I wonder why so many CURIEs are in closure_map but not ic_map?

If the HP ID doesn't show up in the association table at all, it doesn't get an IC score.
So many of these IDs (just based on cursory checking, not exhaustive at all) are rare phenotypes, like HP:0100378 (Absent distal phalanx of the 3rd toe) or high-level terms that just aren't specific enough to get used in associations, like HP:0008008 (Reduced visual acuity) - the latter's child term of HP:0000618 (Blindness) is used 168 times in the HPOA, for example.
Is this the expected behavior, though?
Should all terms receive some sort of baseline score by virtue of existing, then have that score get adjusted proportionally by observed frequency?

Is this the expected behavior, though?

This is expected, but is going to cause problems. I think we will need IC values for everything in phenio

Should all terms receive some sort of baseline score by virtue of existing, then have that score get adjusted proportionally by observed frequency?

We could do that, but I can't think of how to convince OAK to calculate IC like that.
How about this: if a term is not observed, we set the IC to -log(1/number of total counts).
To do this, we could post-process the IC tsv file to set any term in phenio that isn't in there to min(IC score in the IC TSV file). Or, we could do this in semsimian after we read in the IC tsv file.

TODO:

Add option to semsimian + OAK to specify what to do for CURIEs in closure but not in IC map. One option should be to fail. Don't do that by default. Also provide option to infer (assign or use proportional IC.
Save other artifacts to the uploaded output, like the IC maps.

Also something odd happened to the HPvMP and HPvZP uploads. Will check on that

Aha - the issue is that the IC tables for MP and ZP aren't generating what I'd expect.
Running this:
runoak -g mpa.tsv -G hpoa_g2p -i sqlite:obo:phenio information-content -p i --use-associations .all > mpa_ic.tsv

Only outputs 137 ICs, none of them from MP! It looks like the subject_taxon column is being used as the calculation input instead of what's in the object column.
So the hpoa_g2p parser isn't working as expected.
Other parsers are listed here: https://github.com/INCATools/ontology-access-kit/blob/main/src/oaklib/parsers/init.py


Update API endpoint for address lookup

Description

Update address lookup validation API endpoint.
Motivation and Context

https://digital-engagement.atlassian.net/browse/SDPA-5250
How Has This Been Tested?

Test that address lookup will work with new API
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 not work as expected)
[ ] Improvement/refactoring (non-breaking change that doesn't add any features but makes things better)

Checklist

[ ] The PR title is no longer than 64 characters.
[ ] My code follows the code style of this project.
[ ] I've added relevant changes to the documentation (applies to new features and breaking changes in core

Not required as the proxy is managed in AWS Cloudfront distribution https://console.aws.amazon.com/cloudfront/v3/home?region=ap-southeast-2#/distributions/E1EDR7AOOVFJ0G


How to set path?

大佬，问一个愚蠢的问题，怎么设置\lib\test\evaluation\local.py的路径啊？我先运行create_default_local_file.py设置好了各项路径，又在\lib\test\evaluation\local.py中设好各项路径，然后运行video_demo.py依然报错： YOU HAVE NOT SETUP YOUR local.py!!!
Go to "F:\Pycharm Proj/project/project/CVlearning/OSTrack/tracking..\lib\test\evaluation\local.py" and set all the paths you need. Then try to run again.

The information provided is not sufficient. Try debugging to see if local.py is successfully loaded in the env_settings() function.

It's not loaded successfully, error being 'No module named 'test.evaluation'; 'test' is not a package' when running at following step:

Make sure you are running on the correct root path and that /lib\test\evaluation\local.py exists. This may be caused by an incorrect path or a Windows path issue. Since I can't reproduce this error, I can't provide further guidance.

Thanks for your advice. I will try to debug this problem. Thank you so much.


Struggling to get started

Google's Material Design plus React is a fantastic combination, and I strongly believe this is the future. However, despite that I know http servers and static html pages, all these (nodejs + UI) things are new to me. So there is a huge gap for me to follow the http://material-ui.com/#/get-started page.
I managed to get material-ui installed,
  $ npm install material-ui
<EMAIL_ADDRESS>node_modules/react-tap-event-plugin

<EMAIL_ADDRESS>node_modules/react
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
<EMAIL_ADDRESS>node_modules/material-ui
<EMAIL_ADDRESS><EMAIL_ADDRESS>
  $ ls -d1 node_modules/*
  node_modules/material-ui
  node_modules/react
  node_modules/react-tap-event-plugin

but what's next?

Should I move them under my /var/www/? Or start nodejs, or...?
How can I see a simple demo that show everything is working?

All in all, as someone who knows only http servers and static html pages, what's the simplest way for me to see it is working for me in my environment?
Thanks

The simplest way will be to clone the repo, go into the example folder, and npm install that:
https://github.com/callemall/material-ui/tree/master/example
Then you can play around with the simple demo provided, and build your own site on top of that.

Thanks a lot!


drivers/video: Add config for API compatibility

Summary
Add a Kconfig option for supporting old APIs on video driver.
The video_initialize() and video_uninitialize() are re-named to capture_initialize() and capture_uninitialize().
But this modification made break compatibility on user APIs.
So this commit add a option on Kconfig to use old API symbol name.
Impact
No impact except enabling the additional Kconfig option.
Video I/F has an impact when the option is enabled.
Testing
Test spresense:camera, if it can be build and works well.

@SPRESENSE isn't it better to fix the existing applications using the old API instead bringing back those APIs to mainline?
I agree that breaking APIs is not a good thing, but in this case the previous change seems correct: for video capture it is better to use capture_initialize() than video_initialize().
We need to reduce the Kconfig complexity, not increasing it. This patch is increasing complexity for something that should be fixed in the right way (in the external program using this kernel).

But these APIs shouldn't be called from userspace, so we have some flexibility to modify them. BTW, @shizhenghui made this change because v4l2 m2m is added recently, it isn't good to name capture_xxx as video_xxx.

Avoid creating Kconfig entrypoint and just create the macro and include some info that it will be deprecated and the developers should use capture_initialize()

Agreed, but how do we make a noticeable warning (at build-time) that the deprecated API is used? It would be best if messages could be printed at end of compile output.

Avoid creating Kconfig entrypoint and just create the macro and include some info that it will be deprecated and the developers should use capture_initialize()

Agreed, but how do we make a noticeable warning (at build-time) that the deprecated API is used? It would be best if messages could be printed at end of compile output.

There is a deprecated attribute, it is used on Linux kernel.
We could create it for NuttX as well:
#ifdef __GNUC__
#define __deprecated __attribute__((deprecated))
#else
#define __deprecated
#endif

The build on msys2 failed. I restarted the job with debug logging. Let's see if the error is in the codebase or just a spurious error...

Avoid creating Kconfig entrypoint and just create the macro and include some info that it will be deprecated and the developers should use capture_initialize()

Agreed, but how do we make a noticeable warning (at build-time) that the deprecated API is used? It would be best if messages could be printed at end of compile output.

There is a deprecated attribute, it is used on Linux kernel.
We could create it for NuttX as well:
#ifdef __GNUC__
#define __deprecated __attribute__((deprecated))
#else
#define __deprecated
#endif

So we just add:
void old_function(void) __deprecated;

void old_function(void)
{
    // implementation
}

@acassis thank you!
I looked up attribute __deprecated and found this StackOverflow answer which covers GCC, Clang, and MSVC:
https://stackoverflow.com/a/21265197
tl;dr GCC and Clang use attribute((deprecated)) (but we should check for GNUC || clang) while MSVC uses __declspec(deprecated).

Avoid creating Kconfig entrypoint and just create the macro and include some info that it will be deprecated and the developers should use capture_initialize()

Agreed, but how do we make a noticeable warning (at build-time) that the deprecated API is used? It would be best if messages could be printed at end of compile output.

There is a deprecated attribute, it is used on Linux kernel.
We could create it for NuttX as well:
#ifdef __GNUC__
#define __deprecated __attribute__((deprecated))
#else
#define __deprecated
#endif

So we just add:
void old_function(void) __deprecated;

void old_function(void)
{
    // implementation
}

@acassis thank you!
I looked up attribute __deprecated and found this StackOverflow answer which covers GCC, Clang, and MSVC:
https://stackoverflow.com/a/21265197
tl;dr GCC and Clang use attribute((deprecated)) (but we should check for GNUC || clang) while MSVC uses __declspec(deprecated).

Yes, but I think we don't need to implement MSVC since we don't have support for it.

The build on msys2 failed. I restarted the job with debug logging. Let's see if the error is in the codebase or just a spurious error...

the patch need to rebase to pass ci since the fix is provided after this pr: https://github.com/apache/nuttx/pull/12788

Hi,
Thank you for your comments.
I agree that this functions should not be called by userspace.
In nuttx, there is a function board_ioctl() to control board specific devices.
So I think we should define a special command for initializing video device and use it insted.
I will make a patch and force push it on this PR.
And also rebase to newest main branch.


HPC Suite: Execute in Azure only for specialized images

HPC is a specialized SKU in Azure, hence it has to run only on Images specialized for HPC.
"/opt/azurehpc/component_versions.txt" is a file that is present in Azure HPC images, even though it is not publicly documented.
At present there is no other way to identify if the image is an Azure HPC image.

@hcoona FYI and to monitor the discussions

there is a condition check during installing ofed driver based on this file, we need know whether there are customers using RDMA with non-hpc images or not, then decide to skip on non-hpc images or not.

as we discussed, we don't want to miss the test coverage, so close it @adityagesh


added encode option

solves : #1117

 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.


Transparency != Transmission

IMHO the transparencyMap should go into the alpha socket, too.

The reason it is plugged into the transmission socket is because of a few reasons:

We already have one way to achieve alpha transparency. The diffuse alpha channel is used by all current assets that need alpha transparency. Introducing another competing approach would be confusing. Also, having two nodes targeting the same input socket is cumbersone, it'd require a mix node in that case
If we ever want to support things such as IOR in materials, we need something targeting transmission. Alpha transparency isn't influenced by IOR.
MakeHuman makes no use whatsoever of transparencyMap. It's defined in the material class, but it's not then used anywhere else. It is silently ignored. Reusing it for transmission type transparency does no harm.

Since I think we will want transmission type transparency at some point, the alternative I can see is defining a separate key for it. In that case the transparencymapTexture and transparencymapIntensity keys will not be used at all.

I think the transparency map is only necessary for texture maps that cannot handle alpha channels, in particular for jpeg images. It should be "either or" not "and", if an transparency map exists the alpha channel should be ignored.

I took the easy way out and simply changed all uses of transparencymapTexture to a new key "transmissionmapTexture" for mapping against transmission.

In that case we probably will have to update MH, too. Otherwise the information will not survive a round trip from Blender to MH and back...

MH does not need to be modified. The MHAPI plugin might need to be though. This has already been done in https://github.com/makehumancommunity/community-plugins-mhapi/commit/68884aae3f4e327119890e4f0f4f363bbf4012bc


Question on the precomputed GuideScan2 hg38 database

Thanks for developing this great tool!
I am trying to use guidescan2-cli to find some good paired gRNA candidates for a list of genes. I thought your precomputed database could be helpful on saving some running time. Maybe I missed it somewhere, I couldn't find the parameters that were used to generate those precomputed databases. Do you remember what --mismatches and --alt-pam were used?
Additionally, are alternative PAMs weighted in comparison to the NGG PAM in offtarget score calculation? Will it be a good idea to include those NNG and NGA non-canonical PAMs?
Thanks,
Zhibo

Did you find what you need? The --mismatches parameter was set to 3, --alt-pam was NAG, and all off-targets are weighted according to the CFD score introduced by Doench et al. The CFD score incorporates the binding affinity of alternative PAMs, that is, the weighting is implicit.

@schmidt73 Thank you.

@schmidt73 Thank you for the details.
May I ask what is the best approaches to query your precompuated databases? I already have a list of kmers that I want to rank based on specificity and on-target efficiency. I tried to run the enumerate step, which worked pretty well when the list is at the order of 10^3 but significantly slows down when it reaches to the order of 10^5-6.
Do you have a script to query the precomputed bam and only decode the ones on the query kmers?
Thanks,
Zhibo

There are a few available options. You could search directly for the kmers in the BAM database, this will be slow but will scale up. If you have the positions of the kmers, I would use a tool such as BedMap to intersect your list of kmer positions with that in the database. This is very efficient and will scale up to millions of kmers.
If you list is of the order 10^5 or 10^6, you could probably use the indices directly. If you have access to many computers, one way to do this is to split up the list into chunks of 10K-20K and then run a seperate instance of Guidescan2 on each computer. This will be fast and will allow customization on the # of mismatches, alternative PAMs, etc. This sort of approach is detailed in the manual.

@schmidt73 Thank you for the tips. I will try and see which one works better. I have several large sets of kmers but only a workstation. So parallel in large scale is a kind of limited.

Let me know if you need any more help. I think BedMap is the best approach for your usage. I'll close this for now.

@schmidt73 thank you! I will try it.


Code cleanup: Dirty(Comp)

About the PR

Replaced all uses of Dirty(Component) with Dirty(Uid, Component) in content.
Why / Balance

By my count, 138 compiler warnings removed.
Technical details

Some systems had to be modified to pass around uids internally. Notably, the (numerous) systems related to pulling relied on passing and tracking components, so they were modified a bit to be more based around uids. Those systems definitely could still use a refactor, but that was not the intention of this PR. This just changes what needed to be changed to get uids tracked (and maybe a teensy bit more to be consistent).
I did test as many aspects of this as I could, including messing around with pulling with two clients. However, this PR makes (mostly small) changes to a bunch of different systems, so be prepared to do some triage if this gets merged. Everything worked fine for me, and most potential issues would just be client mispredictions, but I'm not going to pretend that I was able to think of and test every possible edge case.
Media

Code.

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

The public APIs for SharedPullingSystem, SharedPullingManagementSystem, and TagSystem have changed to require EntityUids or Entity<T> parameters for most methods. Systems that call these methods will need to be updated.
Changelog

Not today

This should get tested with https://github.com/space-wizards/RobustToolbox/pull/4978 in case one of the uid-component pairs is wrong.

This should get tested with space-wizards/RobustToolbox#4978 in case one of the uid-component pairs is wrong.

I actually added that check to Dirty myself for the purpose of testing this! It did find one mismatch in my testing, which has been fixed. Definitely a good addition to Robust Toolbox.

This has a significant amount of conflicts with https://github.com/space-wizards/space-station-14/pull/24936

This has a significant amount of conflicts with #24936

Oh, I'll bet it does! Glad to see another attempt at a proper rework of pulling. I'll give that some time to get stabilized before fixing up those conflicts.

This has a significant amount of conflicts with #24936

Oh, I'll bet it does! Glad to see another attempt at a proper rework of pulling. I'll give that some time to get stabilized before fixing up those conflicts.

Just nuking the pulling changes is easier realistically.

Yeah, I'll just revert my changes to those files, merge, and do another pass looking for Dirty, in case any slip through.


Lifetime studies

Added lifetime histogram and saved at every bunch passage

Some more cleanup needed. To be re-opened when it is done.


SAMR34 Xplained board: usb_dc_sam0 driver hangs

Describe the bug
I have a SAMR34 XPLAINED PRO board, and I'm trying to bring up USB. I've built the usb_cdc example using the west build command, but I've encountered an issue where the program seems to hang at the line of code https://github.com/zephyrproject-rtos/zephyr/blob/main/drivers/usb/device/usb_dc_sam0.c#L151:
Additionally to board code, I've made some modifications to the device tree files (dts and dtsi) to configure the USB pins as follows:
In .dtsi file:
usb_dc_default: usb_dc_default {
    group1 {
        pinmux = <PA25G_USB_DP>,
                 <PA24G_USB_DM>;
    };
};

In .dts file:
zephyr_udc0: &usb0 {
    status = "okay";
    pinctrl-0 = <&usb_dc_default>;
    pinctrl-names = "default";
};

I inserted and removed the USB, but it doesn't solve the problem. I also tried the example for the board from Microchip Studio USB_CDC, it works correctly and initializes USB.
To Reproduce
Steps to reproduce the behavior:

west build -b atsamr34_xpro zephyr/samples/subsys/usb/cdc_acm
openocd -f board/atmel_saml21_xplained_pro.cfg
gdb-multiarch -iex "target extended-remote localhost:3333" ./build/zephyr/zephyr.elf
load
program hang

Expected behavior
Connection Establishment with host computer
Environment (please complete the following information):

OS: Ubuntu 22.04.2 LTS
Zephyr SDK 0.16.0
v3.4-branch / 3e75c22

Quickly tested on atsamr21_xpro, seems the driver works fine, maybe board configuration issue, clock or something else.

I can confirm this, but unfortunately I don't have a lot of time to look into this right now. Strange, because I thought this was working when I did support for this SoC...
Backtrace from 54efc02:
(gdb) bt
#0  usb_sam0_wait_syncbusy () at /p/zephyr/drivers/usb/device/usb_dc_sam0.c:151
#1  0x000042c4 in usb_dc_attach () at /p/zephyr/drivers/usb/device/usb_dc_sam0.c:248
#2  0x00002bdc in usb_enable (status_cb=status_cb@entry=warning: (Internal error: pc 0x0 in read in CU, but not in symtab.)
warning: (Error: pc 0x0 in address map, but not in symtab.)
0x0 <_vector_table>) at /p/zephyr/subsys/usb/device/usb_device.c:1622
#3  0x00000650 in main () at /p/zephyr/samples/subsys/usb/cdc_acm/src/main.c:168

https://github.com/zephyrproject-rtos/zephyr/blob/54efc028ec366dffc3cffccdb05f96e1aefa346d/drivers/usb/device/usb_dc_sam0.c#L246-L248

I wonder if it might be a power or clock config issue - but I'd be surprised, as far as I know the SAMR34 has SAML21 silicon inside (SiP)... I've skimmed the SAMR34/35 datasheet, but can't see anything calling out differences for USB.
https://github.com/zephyrproject-rtos/zephyr/blob/54efc028ec366dffc3cffccdb05f96e1aefa346d/soc/arm/atmel_sam0/common/soc_saml2x.c#L19-L30

I just tried this on a SAML21-XPro, with the same result on 48c83a1f4268e539afa03abe2ecaed43a0650b7e.
@jfischer-no did you do anything special / any patches?

I just tried this on a atsaml21_xpro, with the same result on 48c83a1.
@jfischer-no did you do anything special / any patches?

"Quickly tested on atsamr21_xpro, seems the driver works fine"
R :smile:

I only have samr21 so I can not reproduce on my side.
@attie-argentum if you donˋt have time to look now we need schedule for the next release and try check using loopback app + Linux.


go-import-add: go exited with status 2

I have error: after call go-import-add func I see go exited with status 2 in status bar.
macos
go version go1.7.1 darwin/amd64
Emacs 25.1(9.0)
Also, go list -e all returns list of packages in console.
Any ideas how to debug this issue?
Thanks.

Fixed problem.


providers/ec2: Fetch EC2 availability zone.

Hi
This change is required in order to mount the AWS EFS endpoint correctly, which uses availability zone as a part of connection string.
I've tested this change on CoreOS 1353.7.0.

This change results in coreos-metadata now fetching more information than just hostname and network addresses, so I'm curious what your motivation is to include fetching this information here as opposed to in some other application.
If you could include a quick example on how you're using it, that would be great.

In order to mount the AWS EFS volume I need to create a systemd unit like this (variable interpolations are from terraform)
[Mount]
What=us-east-1a.fs-${var.efs_id}.efs.${var.region}.amazonaws.com:/
Where=/mnt/efs-galaxy
Type=nfs4
Options=nfsvers=4.1,rsize=1048576,wsize=1048576,hard,timeo=600,retrans=2

[Install]
WantedBy=remote-fs.target

The AWS EFS connection string us-east-1a.fs-${var.efs_id}.efs.${var.region}.amazonaws.com contains the us-east-1a part which is an availability zone. It is required in order to mount the AWS EFS volume correctly. I don't know the AZ before boot because the instance is in the autoscaling group. Therefore, the AZ should be fetched from AWS metadata on boot.

@crawford thoughts on this? Are we fine with extra metadata sliding in for EC2 specifically or do we want to keep the metadata available more consistent across platforms?

This is exactly what coreos-metadata is designed to do. All of the metadata is platform-specific so there's no sense in trying to keep the scope uniform between them.


there is no previous definition of 'file?'

This works:
File.file? "/"

But this does not:
class File
	def self.file?(filename : String | Int)
		previous_def filename.to_s
	end
	def self.exist?(filename : String | Int)
		previous_def filename.to_s
	end
end
File.file? "/"

I'm not actually trying to make these functions accept an Int - I've substituted Int for the actual class I'm using because it doesn't exist here.
Why does the compiler complain that there is no previous definition of 'file?'? Is previous_def not the correct tool to use here? In which case, what should I be using instead?

With self.file?(filename : String | Int) you are defining a new overload, not redefining the previous method, because the previous one just has String as a restriction.
So in your case you have to redefine it for String, use previous_def, and also define a new overload for Int and call whatever you need.
Not sure changing those methods is a good idea, though.

Oh, I see. So...I guess the simplest way is just to write a new function taking Int (not actually int, obviously), and call the base function from there?
Ruby doesn't overload by parameter type, and I keep thinking in Ruby terms ;)

Yes, just define another overload.


SharpDX.Mathematics/Matrix.cs - perspective view matrix calculation seems wrong

I have been porting the software render tutorials series from Microsoft to SDL2,
(https://www.davrous.com/2013/06/13/tutorial-series-learning-how-to-write-a-3d-soft-engine-from-scratch-in-c-typescript-or-javascript/) and using SharpDX code, and I think this led me  to find a bug in: SharpDX.Mathematics/Matrix.cs
(I tried the last two file historical versions and I observed the same behavior)
This method: void PerspectiveFovRH(float fov, float aspect, float znear, float zfar, out Matrix result)
Basically I noticed that my model or mesh would always render upside down.
In my own code is what I ended up using:
`
Matrix4* ComputeFOVProjection( double fov, double aspect, double nearDist, double farDist, bool leftHanded)
{
Matrix4* result = new Matrix4();
result->m11 = 1.0;
result->m22 = 1.0;
result->m33 = 1.0;
result->m44 = 1.0;

if ( fov <= 0 || aspect == 0 )
{
    assert( fov > 0 && aspect != 0 );
    return result;
}

double frustumDepth = farDist - nearDist;
double oneOverDepth = 1 / frustumDepth;

result->m22 = 1 / tan(0.5f * fov);
result->m11 = (leftHanded ? 1 : -1 ) * result->m22 / aspect;
result->m33 = farDist * oneOverDepth;
result->m43 = (-farDist * nearDist) * oneOverDepth;
result->m34 = 1;
result->m44 = 0;
return result;

}`
found here:
http://stackoverflow.com/questions/18404890/how-to-build-perspective-projection-matrix-no-api

Your formula is equivalent to the PerspectiveFovLH only when leftHanded is true, otherwise it is invalid, the StackOverflow method is most likely invalid for RH.
Just check with PerspectiveFovRH on msdn, it should be the same formula used in SharpDX

@xoofx Ok, I see what you mean. Thank you for checking. The Soft Engine tutorial series seems to work better for LH so I am only going to use that formula for LH.  The C# version of the Soft Engine  tutorials code which uses SharpDX actually mixes LH and RH but that seems to be a mistake there. Thanks again.


Feature/asyncio ha discovery extension

This merged request ports #172 to feature/asyncio branch.

:tada: thanks!

@rlehfeld I couldn't keep my fingers off, so I've made a small change, which is that display_name is not explicitly listed as an option, since we can just add name to the ha_discovery dict extra key/values:
digital_inputs:
  - name: motion_sensor
    module: pi_gpio
    pin: 22
    retain: yes
    ha_discovery:
      name: PIR Garage  # optional and defaults to `name` given above
      component: binary_sensor  # optional and defaults to `binary_sensor`
      device_class: motion  # optional
      force_update: true  # or any other key / value pair home-assistant discovery

I shuffled get_common_config() around to support this without the clever, but a bit confusing pop()s (took me a minute to work it out).
def get_common_config(
    io_conf: ConfigType, mqtt_config: ConfigType, mqtt_options: MQTTClientOptions
) -> Dict[str, Any]:
    """
    Return config that's common across all HQ discovery announcements.
    """
    config = dict(name=io_conf["name"])  # Set the default value as the IO conf name
    config = io_conf.get("ha_discovery", {})  # Overwrite the name with one added to `ha_discovery` dict
    config.update(
        dict(
            availability_topic="/".join(
                (mqtt_config["topic_prefix"], mqtt_config["status_topic"])
            ),
            payload_available=mqtt_config["status_payload_running"],
            payload_not_available=mqtt_config["status_payload_dead"],
            device=dict(
                manufacturer="MQTT IO",
                identifiers=["mqtt-io", mqtt_options.client_id],
                name=mqtt_config["discovery_name"],
            ),
        )
    )
    return config


fix: skip testing remote mode in CI

Doesn't work if the contributor isn't from cloudflare as they don't have access to secrets


Isolate XAudio2 driver

Now it's possible to compile for Windows platform if wanted. It's
supported only for Windows 8 or later, so it's not enabled by default.

Updated to use the same shebang line as #6850.


Remove daily limit difference for central/local

Central vs local affects the free text message allowance, not the daily limit.

@quis looks good to me


README.md: add Appveyor's status badge

:+1:
For HarfBuzz, we instructed Travis recently to build on Linux and Mac.  Maybe you can do that here as well?

OK, I'll do that.
On 12 Dec 2015 11:45 a.m., "Behdad Esfahbod"<EMAIL_ADDRESS>wrote:

[image: :+1:]
For HarfBuzz, we instructed Travis recently to build on Linux and Mac.
Maybe you can do that here as well?
—
Reply to this email directly or view it on GitHub
https://github.com/behdad/fonttools/pull/434#issuecomment-164140734.


Adding script for github actions

Change Overview
Please check the type of change your PR introduces:
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[x] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Test Plan
Ran complete CI on code on forked repo. It's working as expected.

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

https://github.com/kanisterio/kanister/pull/827
Link for previously approved PR.
@TribalNightOwl @pavannd1 @PrasadG193 @viveksinghggits
Raised this new PR as my local copy was bit messed up.

The PR needs to be merged manually


[stdlib] [U]Int.bitWidth: Use MemoryLayout.size instead of a direct integer constant

This avoids spurious warnings like the one below:
let adjective = (Int.bitWidth == 32 ? "cozy" : "spacious")
//                                    ^^^^^^
// warning: will never be executed
// note: condition always evaluates to false

rdar://109525235

@swift-ci test

@swift-ci benchmark

@swift-ci test

@swift-ci benchmark

Well, that's unfortunate. In debug builds this does replace the immediate with a load from the value witness, so I expect it isn't good enough for OSLog.
Perhaps it'd be best to simply document that the preferred spelling is if MemoryLayout<Int>.size == 4, not if Int.bitWidth == 32. The most direct way to do this is #if _pointerBitWidth(_32), but there are too many underscores in that...


Add parameter for disabling SSL verification

For r10k users who use the ssl webhook , they will need to disable ssl verification on github to make use of this t&p when using their control repository
I think this is done with the config blob
https://developer.github.com/v3/repos/hooks/#edit-a-hook

Added 'disable_ssl_verify' parameter per your request
Please test this commit to see if this resolves this issue/request:
https://github.com/abrader/abrader-gms/commit/04118814ef61a5cdd9b5ad7dd6ae1188b9a404e1
If all is good, I will tag a release and post to the forge.

@abrader :+1: worked for me after hacking around #5

I'll work https://github.com/abrader/abrader-gms/issues/5 into the code now.  I'll get to https://github.com/abrader/abrader-gms/issues/6 tonight.  Forge release thereafter.  Thanks as always.

v0.0.6 (including this params) now available: https://forge.puppetlabs.com/abrader/gms


[Bug]: Auto-distributing doesn't activate the "Save" button

Apache Hop version?
2.5
Java version?
openjdk version "<IP_ADDRESS>" 2020-11-04 OpenJDK Runtime Environment AdoptOpenJDK (build <IP_ADDRESS>+1) OpenJDK 64-Bit Server VM AdoptOpenJDK (build <IP_ADDRESS>+1, mixed mode)
Operating system
Windows
What happened?
When aligning transforms via Alt +  right arrow, Hop doesn't recognize this as a change. The "Save" button remains greyed out, and when closing it (via the x), it closes without asking if you want to save a change.
Hop should activate the save button or ask if you want to save the changes.  (I.e. same behaviour as when you move a transform manually.)
Issue Priority
Priority: 3
Issue Component
Component: Hop Gui

Link to Mattermost conversation.

Thank you Hans!


with-tailwind example duplicated css

Discussed in https://github.com/vercel/turbo/discussions/2308

Originally posted by oscarsala October 24, 2022
this example causes tailwind to be imported twice duplicating classes, and base selectors styles causing a lot of issues with specificity, has anyone been able to integrate it better?

Reported in #1809

Fixed in https://github.com/vercel/turbo/pull/3557


Multi-line hashes with one key/value are treated like single-line hashes and are not allowed to have a trailing comma

Under this configuration...
Style/TrailingComma:
  EnforcedStyleForMultiline: comma

Rubocop thinks this is a single-line hash and shouldn't have a trailing comma.  Thus, this fails...
_ = {
  key: "value",
}

Is that intentional behavior?
I think this issue could be a duplicate of https://github.com/bbatsov/rubocop/issues/2403 , but I'm creating a new one, since there were multiple problems reported there.

0.35.1 (using Parser <IP_ADDRESS>, running on ruby 2.1.2 x86_64-darwin13.0)

Just verified this still fails in master (8434555b2c62a19e5a0f17e3753e141120fae7b6).

All, I'm pretty sure this is due to #2369  I think there needs to be a check if it's talking about parameters or about a hash.

Thanks for taking a look!  Just realized I didn't include the output, so figure I might as well include that for completeness.
Inspecting 1 file
C

Offenses:

test.rb:2:15: C: Avoid comma after the last item of a hash, unless each item is on its own line.
  key: "value",
              ^

1 file inspected, 1 offense detected


Split between individual and aggregated load profiles is not always practical

Current situation:
Currently for <31 units an individual load profile for electricity has to be selected.
For >30 households an aggregated profile has to be selected.
Problem:
There are cases when I want to assign an aggregated profile to <31 households.
For example when I have different end nodes to represent different stakeholders. But, on the network these nodes are connected to one transformer. In this case the individual profiles are way too spiky.
Possible solutions:

Add a button which can be selected if a user wants to use an aggregated profile on <31 households.
Keep the current interface, but make it possible to assign an aggregated profiles on <31 households when a CSV file is used to define the technologies.

Keep the current interface, but make it possible to assign an aggregated profiles on <31 households when a CSV file is used to define the technologies.

I prefer this option.
@antw @grdw what do you think?

@grdw can you pick this up?

An other option is that the difference between individual and aggregated profiles is just defined in the name of the profile.  So, not all profiles can be used for all numbers of households. Like it is for water heating.
I think I prefer this option.

We could make this way more easy. The easiest way would be getting rid of the < 31 threshold and make it possible to create a separate base_load_edsn and base_load technology. I think it's a bit 'unfair' to only make this possible with a CSV input (in fact I'm not a big fan of the CSV input) and not in the profile matrix editor.

I think it's a bit 'unfair' to only make this possible with a CSV input (in fact I'm not a big fan of the CSV input) and not in the profile matrix editor.

I'm not sure of the best solution, but I absolutely agree with this. We can't apply one set of rules to the normal interface and another to CSVs, otherwise things will break in unpredictable ways when you try to edit these LESes.
I, too, wish we could get rid of the CSV import (export is fine). It might be useful to know in which ways editing a CSV is easier than the normal technology editor, and try to improve it until the CSV import feature is no longer needed. That's a bit off-topic though...

There are cases when I want to assign an aggregated profile to <31 households. For example when I have different end nodes to represent different stakeholders. But, on the network these nodes are connected to one transformer. In this case the individual profiles are way too spiky.

It's worth noting that there is a difference between the individual and aggregated household technologies other than the profiles that may be chosen. Individual households (<30 units) permit use of the "postponing" strategy, while aggregated households use "saving".
An individual household technology (<30 units) with an "aggregated" profile will still using the "postponing" strategy.

The easiest way would be getting rid of the < 31 threshold and make it possible to create a separate base_load_edsn and base_load technology.

I think this may be the right way to go.
When importing a new scenario from ETE we could apply the same approach as we do today: <30 units = individual, ≥30 = aggregated. If the user wants to override that by manually adding an aggregated household with only 20 units, or an individual household with 40, why not let them?

When importing a new scenario from ETE we could apply the same approach as we do today: <31 units = individual, ≥31 = aggregated. If the user wants to override that by manually adding an aggregated household with only 20 units, or an individual household with 40, why not let them?

Sounds good to me

Good ideas everybody!

When importing a new scenario from ETE we could apply the same approach as we do today: <31 units = individual, ≥31 = aggregated. If the user wants to override that by manually adding an aggregated household with only 20 units, or an individual household with 40, why not let them?

Let's do it! 👍

I am currently exploring this issue and @ChaelKruip came with a really great suggestion. He suggested to determine the behavior based on the chosen load_profile. I think this might be a way to approach it.
So that way I assume. There will only be a single base_load technology with both aggregrated and non aggregated profiles. Their curve_type will determine if this profile will either be put in the aggregated or non aggregated basket and this will determine the behavior of the technology. I think this is currently possible... (I am looking at @antw )

I think this is currently possible... (I am looking at @antw )

The exact technology behavior is determined by InstalledTechnology#behavior_with_curve:
base_load = InstalledTechnology.new(type: 'base_load')

base_load.behavior_with_curve(:inflex)
# => "generic"

base_load.behavior_with_curve(:flex)
# => "deferrable" (aka postponing)

# ---------------

base_load_edsn = InstalledTechnology.new(type: 'base_load_edsn')

base_load_edsn.behavior_with_curve(:inflex)
# => "generic"

base_load_edsn.behavior_with_curve(:flex)
# => "optional" (aka saving)

I'm a bit on the fence about this. It would definitely be a easier from the user's perspective to not have to choose between "individual" and "aggregated", but changing the technology behavior by selecting a different profile does feel a bit "magic" and not obvious.
We'd have to add a table/columns so that you could select which strategy applies when uploading base load profiles, and add a special case inside InstalledTechnology to trigger the correct behavior in the network calculation.
At the moment the technology behavior is defined in the YAML file, but now we have to special-case some technologies and profiles? Its easy for a user, but is it simple?

[..]  a different profile does feel a bit "magic" and not obvious.

I also agree with this. ⚖

I also agree with this. ⚖

I'm also not convinced about this solution to be honest.

Allright I think we should keep the distinction between a base_load technology and a base_load_edsn technology. A user can add one or both. We should let the user have the maximum amount of freedom. The first time a user imports their technologies the base_load should 'switch' to base_load_edsn when the threshold is reached. After that automatic edit the user is free to change whatever he or she wants.

After that automatic edit the user is free to change whatever he or she wants.

So, what would happen if someone chooses an individual (base_load) profile for a base_load_edsn technology and uses the save aggregated base load strategy? Will that strategy try to save the 'flexible part' of the individual (base_load) profile?
Or the other way around: would the postpone strategy applied to aggregated (edsn) profiles try to postpone the flexible part of the aggregated profile?
Both wouldn't work as the flexible parts of the two types of profiles are very different.
What I think would be perhaps more robust is that we allow the user to adjust the number of units of either the base_load or base_load_edsn technology importing without applying the threshold automatically. Still keeping the profiles exclusive for each technology.

So, what would happen if someone chooses an individual (base_load) profile for a base_load_edsn technology and uses the save aggregated base load strategy? Will that strategy try to save the 'flexible part' of the individual (base_load) profile?

You can't pick an individual profile for a base_load_edsn technology. You can add a base_load_edsn technology (for which you can pick base_load_edsn profiles) and a base_load technology (for which you can choose base_load profiles). The units however don't matter anymore. The user is free to add for example a base_load_edsn technology, with units = 1. This doesn't make much sense but it is possible.


Fix for error compiling with erlang versions like 19.0.3 or 19.x

This is a patch to fix an error compiling with erlang versions like 19.0.3, it should work with 19 also, and everything else.
diff --git a/get_comp_opts.escript b/get_comp_opts.escript
index b1f4535..68caeaa 100644
--- a/get_comp_opts.escript
+++ b/get_comp_opts.escript
@@ -21,7 +21,7 @@ main(_) ->
     file:write_file("comp_opts.mk", "COMP_OPTS = " ++ CompOpts ++ "\n").

 comp_opts(Version) ->
-    Copts0 = "-DERLANG_VERSION=" ++ Version,
+    Copts0 = "-DERLANG_VERSION=\\\"" ++ Version ++ "\\\"",
     Copts1 = ?IF(Version >= "17", Copts0 ++ " " ++ ?HAS_MAPS_OPT, Copts0),
     Copts2 = ?IF(Version >= "18", Copts1 ++ " " ++ ?FULL_KEYS_OPT, Copts1),
     Copts3 = ?IF(Version >= "19",

Does this handle e.g. <IP_ADDRESS>?

Yes, should work fine.

I will add the fix but I haven't had any problems with htis, 18.3 worked.

OK, I finally see the problem.

Just to leave a record of it, the fix will break when we reach version 99 in erlang, since it relies on string comparison. With string comparison "100" is less than "99". The cases where it breaks are slim within the context of  the current use case, so I don't think there is need to spend more time on it.

If they continue with having about one new major release every year I don't think any of us will have to worry about.

Your fix doesn't.
[james@fanny:~/code/forks/lfe/lfe]$ DESTBINDIR=$HOME/bin MANINSTDIR=$HOME/man make compile
make  erlc-lfec
make[1]: Entering directory '/home/james/code/forks/lfe/lfe'
erlc -I include -o ebin -DERLANG_VERSION=19.0.2 -DHAS_MAPS=true -DHAS_FULL_KEYS=true -DNEW_REC_CORE=true -DNEW_RAND=true -W1 src/lfe_shell.erl
syntax error before: '.': 19.0.2

=ERROR REPORT==== 17-Aug-2016::13:47:17 ===
Error in process <0.48.0> with exit value:
{{nocatch,error},
 [{erl_compile,make_term,1,[{file,"erl_compile.erl"},{line,344}]},
  {erl_compile,parse_generic_option,3,[{file,"erl_compile.erl"},{line,125}]},
  {erl_compile,'-compiler_runner/1-fun-0-',1,
               [{file,"erl_compile.erl"},{line,92}]}]}
Runtime error: {{nocatch,error},
                [{erl_compile,make_term,1,
                              [{file,"erl_compile.erl"},{line,344}]},
                 {erl_compile,parse_generic_option,3,
                              [{file,"erl_compile.erl"},{line,125}]},
                 {erl_compile,'-compiler_runner/1-fun-0-',1,
                              [{file,"erl_compile.erl"},{line,92}]}]}
make[1]: *** [Makefile:71: ebin/lfe_shell.beam] Error 1
make[1]: Leaving directory '/home/james/code/forks/lfe/lfe'
make: *** [Makefile:87: compile] Error 2

That is really weird.  My latest push is 4b58eb016bf782f4a363d1fbced3807c19ed5bfa.
When I compile LFE with it I see:
~/lfe/lfe$ make compile
escript get_comp_opts.escript
/Applications/Xcode.app/Contents/Developer/usr/bin/make  erlc-lfec
erlc -I include -o ebin -DERLANG_VERSION=\"19.0\" -DHAS_MAPS=true -DHAS_FULL_KEYS=true -DNEW_REC_CORE=true -DNEW_RAND=true -W1 src/lfe.erl
erlc -I include -o ebin -DERLANG_VERSION=\"19.0\" -DHAS_MAPS=true -DHAS_FULL_KEYS=true -DNEW_REC_CORE=true -DNEW_RAND=true -W1 src/lfe_bits.erl
erlc -I include -o ebin -DERLANG_VERSION=\"19.0\" -DHAS_MAPS=true -DHAS_FULL_KEYS=true -DNEW_REC_CORE=true -DNEW_RAND=true -W1 src/lfe_codegen.erl
src/lfe_codegen.erl:851: Warning: variable 'St' shadowed in 'fun'
src/lfe_codegen.erl:974: Warning: variable 'St' shadowed in 'fun'
src/lfe_codegen.erl:981: Warning: variable 'St' shadowed in 'fun'
src/lfe_codegen.erl:986: Warning: variable 'St' shadowed in 'fun'
erlc -I include -o ebin -DERLANG_VERSION=\"19.0\" -DHAS_MAPS=true -DHAS_FULL_KEYS=true -DNEW_REC_CORE=true -DNEW_RAND=true -W1 src/lfe_comp.erl

Though github is very slow now, or it might be my net.

No, it was me forgetting to run 'make clean' :)


Crash with latest version  1.12

Issue description:
Crash when entering any other dimension.
What happens:
I set up a small system.  I went to the nether and the end.  After 10 seconds or so game crashes.  And crashes again when relaunched. Unless you can Teleporte out before the 10 seconds.  Crash log points to refined storage.  I'm sorry I'm on my phone and don't have the log.  Will post in a few. I did however find a fix.  Chunk loading the area the system is in stopped the crash
What you expected to happen:
Not crash.
Steps to reproduce:

...
Version (make sure you are on the latest version before reporting):

Minecraft: 1.12
Forge:
Refined Storage: latest

Does this issue occur on a server? [yes/no]
haven't tried. Client only at this point
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]
Again will post the rest of the info soon

Please post your crash report

My bad,  got busy at work
https://pastebin.com/W3xkScUb

The problem is with an importer connected to a inventory. Said inventory has a "null" item assigned to it, which is not allowed and mods shouldn't handle that. So the implementor if that inventory or chest or whatever the importer is connected to has done it wrong.


Sort changes by numerical value

In below snapshot, I sort change column in increasing order. But 28% is listed before 3%, it's sorted by string, not numeric.

I think I know what the issue is. Sorting appears to work fine until there is a --- (means unknown) value in the list. I'll see if making unknowns blank instead preserves the correct sorting.

Fixed in: https://github.com/ursacomputing/conbench/commit/93f999b901889d68f2f489d519126a31d490a745
Before:

After:

Thanks for taking the time to report this @cyb70289!!


Fix label for GO:0061990, molecular_function "beta-ketodecanoyl-[acyl-carrier-protein] synthase

https://github.com/geneontology/helpdesk/issues/103
Close helpdesk ticket when this is done, thx!

I have a PR to fix the label, but I still think the annotations need to be reviewed.

I fixes two labels. Closing this ticket, but leaving the helpdesk ticket open until the annotations are reviewed.


Scalafmt fails on manual type ascription after a {} block

Configuration (required)
Please paste the contents of your .scalafmt.conf file here:
version                                  = 3.6.1
runner.dialect                           = scala213
project.git                              = true
style                                    = defaultWithAlign
docstrings.style                         = Asterisk
docstrings.wrap                          = false
indentOperator.preset                    = spray
maxColumn                                = 120
lineEndings                              = preserve
rewrite.rules                            = [RedundantParens, SortImports, AvoidInfix]
indentOperator.exemptScope               = all
align.preset                             = some
align.tokens."+"                         = [
  {
    code   = "~>"
    owners = [
      { regex = "Term.ApplyInfix" }
    ]
  }
]
literals.hexDigits                       = upper
literals.hexPrefix                       = lower
binPack.unsafeCallSite                   = always
binPack.unsafeDefnSite                   = always
binPack.indentCallSiteSingleArg          = false
binPack.indentCallSiteOnce               = true
newlines.avoidForSimpleOverflow          = [slc]
newlines.source                          = keep
newlines.beforeMultiline                 = keep
align.openParenDefnSite                  = false
align.openParenCallSite                  = false
align.allowOverflow                      = true
optIn.breakChainOnFirstMethodDot         = false
optIn.configStyleArguments               = false
danglingParentheses.preset               = false
spaces.inImportCurlyBraces               = true
rewrite.neverInfix.excludeFilters        = [
  and
  min
  max
  until
  to
  by
  eq
  ne
  "should.*"
  "contain.*"
  "must.*"
  in
  ignore
  be
  taggedAs
  thrownBy
  synchronized
  have
  when
  size
  only
  noneOf
  oneElementOf
  noElementsOf
  atLeastOneElementOf
  atMostOneElementOf
  allElementsOf
  inOrderElementsOf
  theSameElementsAs
  theSameElementsInOrderAs
]
rewriteTokens          = {
  "⇒": "=>"
  "→": "->"
  "←": "<-"
}
project.excludeFilters = [
  "scripts/authors.scala"
]
project.layout         = StandardConvention

Command-line parameters (required)
When I run scalafmt via CLI like this: scalafmt
Steps
Given code like this:
        implicit val um: Unmarshaller[HttpResponse, StorageObject] = Unmarshaller.withMaterializer {
          implicit ec => implicit mat =>
            {
              case HttpResponse(status, _, entity, _) if status.isSuccess() =>
                Unmarshal(entity).to[StorageObject]
              case HttpResponse(status, _, entity, _) =>
                Unmarshal(entity).to[String].flatMap { errorString =>
                  Future.failed(new RuntimeException(s"Uploading part failed with status $status: $errorString"))
                }
            }: PartialFunction[HttpResponse, Future[StorageObject]]
        }.withDefaultRetry

Problem
Scalafmt errors with the following message
org.scalafmt.dynamic.exceptions.PositionExceptionImpl: /Users/mdedetrich/github/incubator-pekko-connectors/google-cloud-storage/src/main/scala/akka/stream/alpakka/googlecloud/storage/impl/GCStorageStream.scala:158: error: [dialect scala213] ; expected but : found
            }: PartialFunction[HttpResponse, Future[StorageObject]]
             ^: /Users/mdedetrich/github/incubator-pekko-connectors/google-cloud-storage/src/main/scala/akka/stream/alpakka/googlecloud/storage/impl/GCStorageStream.scala

Expectation
scalafmt should not error out
Workaround
Removing the type ascription (i.e. : PartialFunction[HttpResponse, Future[StorageObject]]) solves the problem.
Notes
Original problematic source file is at https://github.com/apache/incubator-pekko-connectors/blob/4ebf24a4f996b088b4ed16cee182accc5ac92e1c/google-cloud-storage/src/main/scala/akka/stream/alpakka/googlecloud/storage/impl/GCStorageStream.scala#L149-L159

Thanks for reporting! I think I have a deja vu, I thought we fixed somethings like that in the past :sweat_smile:


Controller support scheduling tasks based on K8s workload status

Current Behavior
based on agent reporting
Proposed Behavior
Controller support scheduling tasks based on K8s workload status

deal with K8s job status

https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.21/#jobstatus-v1-batch
https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/#pod-conditions

retry logic triggered by users
log collection
K8s job cleanup ( manually or https://kubernetes.io/docs/concepts/workloads/controllers/job/#ttl-mechanism-for-finished-jobs)

done

#817
#806
#805
#792
#785
#782


Should angular_test prevent errors from printing to console.error?

Currently, the default ExceptionHandler (BrowserExceptionHandler) is always installed unless overridden, and the default behavior is to forward exceptions to console.error. This is great for debugging, but less useful in a test environment since uncaught exceptions cause test failures anyway (at least in angular_test, they do).
One of the patterns that hurts from this choice is the following:
https://github.com/dart-lang/angular/blob/e81df5fd6733e5cec371b24857ed09ca5b05746e/angular_test/test/frontend/bed_error_test.dart#L110-L127
This isn't the most common code, but it's useful to assert that you expect errors.  Either way not too important, but worth discussing.

Haven't thought about this very long, but here is my initial opinion:
If there are exceptions thrown during a test, that should be failing the test (with the obvious exception that you are asserting in the test that a specific exception was thrown). If these errors already fail the test, then we don't need to also print them to the console.


Composer throws extensions.reduce is not a function

Describe the bug
The useLinks hook is causing an error because the value returned from useRecoilValue(extensionsState) appears to be a string instead of an array.
Version

Browser

[ ] Electron distribution
[x] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[x] macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Pull latest changes from Main
Run yarn && yarn build
Run yarn start:dev - yarn start didn't result in the same behavior..?
Observe error when navigating to Composer in Browser

Screenshots

Additional context
https://github.com/microsoft/BotFramework-Composer/blob/673f961ac3c6765b6598624215c909699188e2f8/Composer/packages/client/src/utils/hooks.ts#L31

@tdurnford I am unable to reproduce this. What is the value of useRecoilValue(extensionsState)?

It's not reproing for me either now. Previously, useRecoilValue(extensionsState) was resolving to a string. Not sure what changed. I'll close the issue for now, and if it happens again I'll reopen it.