Use an Allow list for StargateQuery instead of completely disabling it

I was discussing the need to disable StargateQuery in the discord channel to better understand why they're completely disabled (https://discord.com/channels/737637324434833438/737643344712171600/998624457188987030).
I would like to suggest replacing the current handler with one that contains an allow list. This way any chain that uses wasmd and has the stargate feature enabled, can just customize the allow list for the queries. If the chain takes no action, the behaviour would be equivalent to the current one (https://github.com/CosmWasm/wasmd/blob/7c7840c32a65c79e9b9525ec9488de464cbf78a0/x/wasm/keeper/query_plugins.go#L275). While chains can customize this manually by replacing the stargate querier, I think it feels inconsistent to have StargateQuery on cosmwasm and have it be useless in wasmd.
As a reference, here is the previous DenyList implementation: https://github.com/CosmWasm/wasmd/blob/v0.25.0/x/wasm/keeper/query_plugins.go#L281 (thanks @jhernandezb!)
My interest in this comes from the decision from osmosis to use StargateMsg/StargateQuery for cosmwasm<->osmosis communication instead of custom bindings (https://github.com/osmosis-labs/osmosis/pull/1484#issuecomment-1176960176).
If this is of interest, I can provide a PR.

Also worth to mention the amount of work that requires adding a binding in upstream for external teams
see: https://github.com/CosmWasm/wasmd/pull/903

I'm currently working on this in the osmosis repo for wasmd, can get this upstreamed as I get this finished for reviews!

TL;DR it is not about the work to do so, it is about the danger of introducing a chain halt.

yeah it makes sense to have each separate downstream fork implement it on their own forks, but not support it in upstream level! thanks for the explanation :)

I mean, adding common hooks to wasmd to allow downstream to create an allow list, but keeping it empty by default could work.
I hesitate to enable any by default, but if the downstream wants to, we can make it easier

Thanks for the detailed explanation Ethan!

That means, exposing this may require a deep review if we consider even a patch upgrade of the Cosmos SDK dependency without a hard fork. Did any queries change? Did they introduce maps or other randomness in those?

☝ This is the part I was missing. We still run the same risks when exposing those modules via Custom queries, though, but maybe the extra work needed to write those queries ensures that chain devs do their due diligence here.
I'm starting to think the actual solution here is having a guide of what to look for when exposing modules to cosmwasm. For example, I would expect randomness and arch dependent implementations (i.e.: floats) to be an issue, but I was surprised maps also introduced non-determinism.

We still run the same risks when exposing those modules via Custom queries

Generally not so, as you parse the data from the sdk and produce a limited subset to the contract. If you control that subset, it should be fine. eg. if you have a list, ensure to sort it in your custom querier. Any extra fields would be filtered out.


[tool_crash] SocketException: Failed to create datagram socket, OS Error: Address already in use, errno = 48

Command
flutter attach

Steps to Reproduce

cd path/to/main.dart
flutter attach

Logs
SocketException: Failed to create datagram socket, OS Error: Address already in use, errno = 48
#0      _NativeSocket.bindDatagram (dart:io-patch/socket_patch.dart:981:7)
<asynchronous suspension>
#1      MDnsClient.start (package:multicast_dns/multicast_dns.dart:128:40)
<asynchronous suspension>
#2      MDnsObservatoryDiscovery.query (package:flutter_tools/src/mdns_discovery.dart:66:7)
<asynchronous suspension>
#3      MDnsObservatoryDiscovery.getObservatoryUri (package:flutter_tools/src/mdns_discovery.dart:153:51)
<asynchronous suspension>
#4      AttachCommand._attachToDevice (package:flutter_tools/src/commands/attach.dart:271:11)
<asynchronous suspension>
#5      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#6      AttachCommand.runCommand (package:flutter_tools/src/commands/attach.dart:220:5)
<asynchronous suspension>
#7      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1125:27)
<asynchronous suspension>
#8      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#9      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#10     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:288:9)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:236:5)

[✓] Flutter (Channel flutter-2.5-candidate.5, 2.5.3, on macOS 12.1 21C52 darwin-x64, locale en-CN)
    • Flutter version 2.5.3 at /Users/yianzhou/Documents/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision 18116933e7 (3 months ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[!] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/yianzhou/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 at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.2.1, Build version 13C100
    • CocoaPods version 1.11.2

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

[✓] Android Studio (version 2021.1)
    • 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.11+0-b60-7590822)

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

[✓] Connected device (3 available)
    • iPhone 13 mini (mobile) • 00008110-000119080C42801E            • ios            • iOS 15.1 19B74
    • iPhone 13 (mobile)      • 37399F6A-35C4-4ED8-B460-D6E0C043EBAF • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-2 (simulator)
    • Chrome (web)            • chrome                               • web-javascript • Google Chrome 97.0.4692.99

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: module
Version: 1.0.0+1
Material: true
Android X: true
Module: true
Plugin: false
Android package: com.example.main
iOS bundle identifier: com.example.main
Creation channel: stable
Creation framework version: 18116933e77adc82f80866c928266a5b4f1ed645
Plugins
device_info-2.0.3
flutter_plugin_android_lifecycle-2.0.5
image_picker-0.8.4+4
image_picker_for_web-2.1.5
path_provider-2.0.8
path_provider_android-2.0.11
path_provider_ios-2.0.7
path_provider_linux-2.1.5
path_provider_macos-2.0.5
path_provider_windows-2.0.5
permission_handler-7.1.0
shared_preferences-2.0.8
shared_preferences_linux-2.0.4
shared_preferences_macos-2.0.2
shared_preferences_web-2.0.3
shared_preferences_windows-2.0.4
sqflite-2.0.2

Hi @yianzhou,
There is a similar issue describing this case https://github.com/flutter/flutter/issues/53477, Please follow up there for further updates, Closing this issue as a duplicate. If you disagree feel free to write in the comments and I will reopen it.
Thank you.


Add resourceful-redux implementation

:v:
This project is heavily utilizing the new Ideas & Planning type from #60 :)

Thank you! :v:


storyblok-cli generate-migration should prepend timestamp to migration files

The feature would affect: (check one with "x")

[ ] app.storyblok.com (CMS - Interface)
[ ] api.storyblok.com (CMS - Content Delivery API)
[ ] mapi.storyblok.com (CMS - Management API)
[ ] capi.storyblok.com (Commerce - API)
[ ] Commerce - Interface
[x] Other

Is your feature request related to a problem? Please describe.
Right now when I generate a migration with the storyblok-cli the generated files are missing a unique timestamp. That way we can only have 1 migration per component/field combination. Works fine if I create unique names manually though. Also I'd suggest a more general title for the name of the migration, to enable multiple changes per generated migration.
Describe the solution you'd like
Prepend the timestamp to the generated files, e.g.
20200706101205_change_title_field.js
Describe alternatives you've considered
Manual generation of migration files
Additional context
When going with the readme i quickly encounter multiple migrations with the same name

A more general "enter a title of the migration" would be good, instead of going for the field name, which tells the user only one field can be changed per migration

Hi guys, thanks to send your nice ideas 😄 .
I created a task in our board to implement this nice feature, in next months we can see this deployed, or if you want you can contribute opening a PR to add this feature 😃 .
One more time i want to say thank you so much. And i'm so sorry for the delay to respond this issue.
I will close the issue because we already has the task created. Thanks 💚
cc: @six7 @maoberlehner @monder007


Create RPMs for Fedora 27

Fedora's official release ver is 27.  Will create packages for this version.

Fedora version 1ea608a for i686 and x86_64 up at https://ioncoin.000webhostapp.com/


Typing awareness indicator displays after the message is already sent

Screenshots

Version
v4.15.1
Describe the bug
Typing awareness indicator displays after the message is already sent from the bot.
Steps to reproduce

When using the ShowTypingMiddleware class. By default the indicator will display post message.
When rednering the WebChat we have: sendTypingIndicator: true

Expected behavior
The indicator should display prior to the bot sending the message

Hi @jtenacomm, I'm investigating this issue.

After doing some digging on this issue it looks like the typing indicator by default has a 500ms delay and 3000ms period. If the bot response within this delay the indicator is still sent. This resulted in the behavior of seeing the indicator appear briefly after the message is sent.
Overriding the delay to 0 seems to resolve this issue, but idly it would be able to detect if a message was sent and not send the indicator.

Hey @jtenacomm,
Thanks for the helpful information.
From which source code did you find 500ms delay and 3000ms period?
Did you override the delay in SDK or webchat?

closing due to inactivity. @jtenacomm, please re-open if needed.


feat: DirEnv component

WIP DirEnv Component

docs(README): remove incorrect comment from README.md
feat(projenrc): add dirEnv component, add dirEnvComponent to monorepo workspace deps
feat(projen.component.dir-env): generate files for dir-env projen component
feat(projen.component.dir-env): add DirEnv component

@BradenM I need your help setting marker on the file. Once it's set, we just need to update test snapshots. Thanks


Add Icons to Card Headers

RHCLOUD-31447

Pull icons from backend config
Add icons to card headers in the grid layout and in the widget drawer

Same problem as with the header link: #37 (comment)
Backend is the source of truth for any widget data

Updated to pull the icons from the backend


Add repo template files + CI scripts + service shell

Added all default repo template files + scripts.
Added event consumer + empty handler.

Assumes that the service is triggered from an 'observations-imported' event on the tail end of the existing import process.

I believe we should be using the 1.9.1 tag for Go now.


aws provider authentication - allow IAM user to assume role

What problem are you facing?
Currently the only way to assume a role in the ProviderConfig is to use Injected Identity.  We would like to use an IAM user and assume a role.
Currently the ProviderConfig looks like this for the Injected Identity and assume role feature.
$ cat <<EOF | kubectl apply -f -
apiVersion: aws.crossplane.io/v1beta1
kind: ProviderConfig
metadata:
name: account-b
spec:
assumeRoleARN: "arn:aws:iam::999999999999:role/account-b"
credentials:
source: InjectedIdentity
EOF
How could Crossplane help solve your problem?
Allow the ProviderConfig to use IAM user credentials when assuming a role...  i.e.
$ cat <<EOF | kubectl apply -f -
apiVersion: aws.crossplane.io/v1beta1
kind: ProviderConfig
metadata:
name: account-b
spec:
assumeRoleARN: "arn:aws:iam::999999999999:role/account-b"
credentials:
assumeRoleARN: "arn:aws:iam::999999999999:role/account-b"
source: Secret
secretRef:
namespace: crossplane-system
name: aws-creds
key: creds
EOF

@brianmjones thanks for opening - think we have a Lack of documentation - in general this is possible checkout my original PR https://github.com/crossplane/provider-aws/pull/912

ok, that would be awesome... I'll give it a try and close the issue once verified

Would be great If you have time to enhance the documentation ;)


Affinity of CRSS  keep changing after restarting pc

Hi
I appreciate your guide very much.
This is just a minor inconvenience that is bugging me while I troubleshoot.
Only fix on I found online is process lasso and I don't want to use it.
Is there a way to permanently set it with process hacker?
Your feedback is appreciated. Thanks

You can save affinities/priorities within Process Hacker (in nightly builds, not 2.x)

or set CSRSS per IFEO:
Windows Registry Editor Version 5.00

[HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Image File Execution Options\csrss.exe]

[HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Image File Execution Options\csrss.exe\PerfOptions]
"CpuPriorityClass"=dword:00000003

and DWM as task:
<?xml version="1.0" encoding="UTF-16"?>
<Task version="1.2" xmlns="http://schemas.microsoft.com/windows/2004/02/mit/task">
  <RegistrationInfo>
    <Date>2021-11-16T21:12:25</Date>
    <Author>spddl</Author>
    <URI>\SetDwmToLow</URI>
  </RegistrationInfo>
  <Triggers>
    <LogonTrigger>
      <StartBoundary>2021-11-16T21:12:00</StartBoundary>
      <Enabled>true</Enabled>
    </LogonTrigger>
  </Triggers>
  <Principals>
    <Principal id="Author">
      <UserId>S-1-5-18</UserId>
      <RunLevel>HighestAvailable</RunLevel>
    </Principal>
  </Principals>
  <Settings>
    <MultipleInstancesPolicy>IgnoreNew</MultipleInstancesPolicy>
    <DisallowStartIfOnBatteries>true</DisallowStartIfOnBatteries>
    <StopIfGoingOnBatteries>true</StopIfGoingOnBatteries>
    <AllowHardTerminate>true</AllowHardTerminate>
    <StartWhenAvailable>false</StartWhenAvailable>
    <RunOnlyIfNetworkAvailable>false</RunOnlyIfNetworkAvailable>
    <IdleSettings>
      <Duration>PT10M</Duration>
      <WaitTimeout>PT1H</WaitTimeout>
      <StopOnIdleEnd>true</StopOnIdleEnd>
      <RestartOnIdle>false</RestartOnIdle>
    </IdleSettings>
    <AllowStartOnDemand>true</AllowStartOnDemand>
    <Enabled>true</Enabled>
    <Hidden>false</Hidden>
    <RunOnlyIfIdle>false</RunOnlyIfIdle>
    <WakeToRun>false</WakeToRun>
    <ExecutionTimeLimit>PT72H</ExecutionTimeLimit>
    <Priority>7</Priority>
  </Settings>
  <Actions Context="Author">
    <Exec>
      <Command>wmic</Command>
      <Arguments>process where name="dwm.exe" call setpriority 64</Arguments>
    </Exec>
  </Actions>
</Task>

Thank you all

Thanks for jumping in @Zusier @spddl

I can save affinity and priority of DWM. However, I can only save the priority for CRSS. It's Affinity keeps defaulting after I restart my pc. I'm running out of thoughts.

CSRSS is a critical windows process so tools like ProcessLasso if I recall (correct me if I'm wrong) don't run at the same privilege level and isn't able to modify the affinities etc.
You could setup a simple login script to use processhacker to make the changes then have the script also unload processhacker dll at the end once the change is made since it's detected as unwanted dll by some anti-cheats.

CSRSS is a critical windows process so tools like ProcessLasso if I recall (correct me if I'm wrong) don't run at the same privilege level and isn't able to modify the affinities etc.
You could setup a simple login script to use processhacker to make the changes then have the script also unload processhacker dll at the end once the change is made since it's detected as unwanted dll by some anti-cheats.

Thank you very much. I will learn how to do that.

That is expected because the process is PsProtectedSignerWinTcb-Light protected. It is unprotected on Windows 7 for reference

Any idea how to bypass/disable the protection?

Process Hacker to bypass and set as mentioned above or katlogic/WindowsD to remove the protection. Beware of anticheat complaints.

I know about WindowsD, just didnt want to use. I meant in powershell if there were anything.
Though I might take a look at SystemInformer/ProcessHacker, if there are commands to be used. It should only be a problem if it's open, for anticheats that is.


⚠ Bofrost.at has degraded performance

In d4d3e10, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1879 ms

Resolved: Bofrost.at performance has improved in d47261c.


more references

look into https://blog.tonari.no/rust-simple-hardware-project
compare, document, reference in README

done with ba8f8b5


Update sql-server-linux-performance-best-practices.md for vm.legacy_va_layout issue

@pradm : Thanks for your contribution! The author, @rgward, has been notified to review your proposed change.

@pradm Thanks! If you click on the files tab, you'll see I made a few minor changes. I probably made them after you checked. :) Hope these are OK.

Sir, Can we make few more changes as below, i have just added word "configuration"
"Linux O/S legacy VM layout configuration is not compatible for SQL. Make sure that vm.legacy_va_layout is set to 0 and Soft Stack limit is set to 8192. For more information, see https://support.microsoft.com/help/4134638/sql-server-2017-terminates-and-generates-a-core-dump-on-rhel-7-4-when."

Sir, Please revert this change back as Bob Ward had suggested this does not fall under performance best practices and it might add confusion. My apologies for wasting your time.

@pradm Thanks. Closing pull request without merging.


Why are you doing this?

Why are you providing a tool to spam Reddit?

why not

Why are you providing a tool to spam Reddit?

the videos linked in the readme explain why but i'll give you the tl;dw: awareness.

Controlling your phone remotely in late 2024 with a web dashboard no ADB or wire needed, a reddit karma bot that mimics human behavior through a learning script for the bot that shadows different humanized ways of managing an account of reddit.
You can check it's commenting features in our repository(https://github.com/Appilot-dev/Reddit-Android-Karma-Farming-Bot)
How you can control the bot’s speed and never get banned again, do tell us by pointing out issues if any by using the bot.


Import Matlab m-files

@pzwang suggested this a few months ago, but I've just come across it.  A very cool Numba (NumbaPro?) feature would be to translate Matlab M-Files and allow imports of existing M-files, probably along these lines:
func = numba.importm('/path/to/func.m')

Some pointers on this from Peter's email:

Matlab parser: https://github.com/ericharley/matlab-parser
PLY: http://www.dabeaz.com/ply/ (probably don't need to point you guys to this)
Some comments from Peter when he brought it up:

This is for both sales and devs, if ever presented with the question of how to interface between Python and Matlab or SAS.
I found this good SO link listing various options:
http://stackoverflow.com/a/23762412/246502
It looks like the top current contenders are:
http://mlabwrap.sourceforge.net/
https://code.google.com/p/pymat2/
https://github.com/ewiger/mlab
https://github.com/jaderberg/python-matlab-bridge
This last one, OMPC, looks interesting because it's actually a Matlab -> Python compiler:
http://ompc.juricap.com/
Maybe this could be a feature in NumbaPro? numba.importm("/path/to/file.m")
:-)

I have not done much work with Matlab, but I would have expected the biggest effort required for translating code would be the translation of calls to functions in Matlab's equivalent of the standard library.  The mappings to existing Python and Numpy functions might be fairly straightforward, but I'm not sure.

Out of curiosity, what's the intended complexity of M-files that would be useful to "import"?

Closing because this is definitely out of scope for Numba.


Identify

[Enter feedback here]

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

ID: c2e83171-fd05-9fe1-3ba7-0fb20161ee63
Version Independent ID: 2e6b2451-5ff8-a277-90ea-49aa70957977
Content: What is Azure Active Directory Identity Protection?
Content Source: articles/active-directory/identity-protection/overview-identity-protection.md
Service: active-directory
Sub-service: identity-protection
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Setup

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

ID: c2e83171-fd05-9fe1-3ba7-0fb20161ee63
Version Independent ID: 2e6b2451-5ff8-a277-90ea-49aa70957977
Content: What is Azure Active Directory Identity Protection?
Content Source: articles/active-directory/identity-protection/overview-identity-protection.md
Service: active-directory
Sub-service: identity-protection
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

I’m new

``

@nanna481 please elaborate on the feedback you have for this specific document.

Hi @nanna481 ,
To create a github issue, please follow the instructions in this guide and include the specific feedback that you have for the document.
If you have a product question or troubleshooting question, please post to Microsoft Q&A .
If you have a feature request, please post to User Voice.
We will proceed to close this thread but if you have further questions please open a new issue with your specific feedback.


Module doesn't link for Android

Hi,
I have React-Native v41.
I downloaded the react-native-fbads with yarn and after that I did "react-native link".
It showed me that the iOS module was link but nothing about the Android module.
How I have to do? Have I to install other things than this to make it working?
When I try to use it on my code, I can't use ith undefined with this line of code :
import { NativeAdsManager } from 'react-native-fbads';
const adsManager = new NativeAdsManager('18025212...');
Sincerely,

When I try to use it on my code, I can't use ith undefined with this line of code :
import { NativeAdsManager } from 'react-native-fbads';
const adsManager = new NativeAdsManager('18025212...');

Hey,
If for some reason react-native link failed for you, please default to manual linking process as described in the official React documentation.

It failes for all people... with recent version so...

You are right that there were already two reports regarding that matter, I'll reopen for now and debug later. If anyone wants to tackle it in the meantime, please do!

If you want I can help to be better in development. It could be an good point as a developer to upgrade a repo. But I don't really know for the moment how to do :/

I found the issue. It seems the problem is that the react-native-fbads folder inside the node_module has one extra folder src which makes react-native miss the gradle file.
I moved all the files inside the src folder (the one that has the android, ios folders) to the main one (react-native-fbads) and react-native link started working.
Now I am finding another issue at compiling but I am not sure it it is related to this project yet.

Alright, this is what I tried:

Update settings.gradle of your project and add this:

include ':react-native-fbads'
project(':react-native-fbads').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-fbads/src/android')

Note: At this point you probably need a Gradle Sync, Clean and Build for the project to come up as module.

Modify the build.gradle file and in the dependencies, add this:
compile project(':react-native-fbads')

Modify the MainApplication.java and add these lines:

import io.callstack.react.fbads.FBAdsPackage;
new FBAdsPackage() in getPackages()

Now it compiles and builds but the APK is not generated due to an error in Duplicated definitions. Still trying to see what is going on.

@sfratini Try
    compile(project(':react-native-fbads')) {
		exclude group: "com.google.android.gms"
    }

Not sure if this is a proper fix though.

@LutherDiaz Yep, actually that is how I solved it. I was also getting the Dex too many methods error but I solved it with Proguard in debug.

Fixed in 4475c36


Should handle untrack dep ignore at setter instead?

Currently it if a dep is initially ignored through ref(), it will always ignore the dep even if a non-ref is assigned afterwards

Done at #164


AdapterPresenter isEmpty() is redundant?

@bagza  I've noticed that we have isEmpty() function is AdapterPresenter. Do we really need that here? Looks like some kind of a helper function, not sure that our core abstraction should know about that, as it pollutes the interface and, potentially, may cause #LSP issues.
What do you think?

fun isEmpty() = getCount() == 0 sounds very much like an extension function

@bagza yeap, guess we can remove it from the abstraction

#8


Issue #1 : Added my portfolio(Jyotimoy Kashyap)

I have checked the commits.
Please review the changes and merge my pull request labelling "hacktoberfest-accepted"

@JyotimoyKashyap  please follow for more such repo

I had one question actually @avinash201199 , I am into open source for the first time, and in my profile of hacktoberfest I can see that those pull requests which is merged it says matures in 14 days, what does that mean? I will be really grateful if you could just give me some idea, how it works.

I'm not sure , may be they will review it or it will take time to reflect.

Oh okay, thank you so much, as this is my first time, maybe I don't want to get disappointed. Thanks for replying, I wasn't expecting such a quick reply, knowing the circumstances that, you must have a lot of pull requests to merge and review in the DSA repo.
Thank You really!

yeah , trying my best .
I have to also make 4 pr as a participant .


chore(readme): update architecture diagram

The diagram preview can be found here

@rootfs please update the diagram preview

preview

@vimalk78 updated, can you check again? thanks
https://github.com/sustainable-computing-io/kepler/blob/609a3ff8d73ed25488b875d751d0f50096e1df86/README.md

/lgtm merging


Firefox has black boxes during transition rendering

I came across this issue when using this in firefox. I already found the solution, however my setup is quite different so I'll just provide the solution instead of sending in a pull request (I didn't generate the updated dist/ file)
Just add visibility: hidden to everywhere you use opacity: 0, and visibility: visible to any lines doing opacity: 1. Firefox seems to render the element anyhow in that distorted animation

Uhm that's interesting, both my Firefoxes (v39 and Nighlty on Mac OS) render it ok.
Will inspect further. Anyway the workaround you suggest looks very doable, although a bit tricky for me as I can't test if I'm applying the fix properly, being unable to reproduce.
Thanks for chiming in!

OK so the visibility: hidden property made the black boxes occur much less often, but the solution that really worked was adding this property to .mfb-component-list > li { outline: 1px solid transparent; }
See
http://stackoverflow.com/questions/27897413/css-transform-bug-in-firefox-win7-version-not-in-mac

Thanks for this, will publish the suggested fix in the next few days.

I am seeing these black boxes appear in several places with recent updates. It is only Windows that has this issue, macOS doesn't.
Example: https://www.screencast.com/t/F8cHZC9swX
Same site in Win Chrome: https://www.screencast.com/t/2tAei15ja5ON
Interesting thing is in doing research I found that someone said it could be related to extensions so I restarted FF in safe mode and the problem was gone so I thought it was an extension. I restarted normally and removed all extensions thinking I would add back in one at a time to see which was causing it. The problem persisted even with all extensions disabled.
Other sites are doing it as well. This is just WINDOWS. I realize this is a really old thread but it is still open and thought I would add my experience.


ISPN-10040 Embedded and server thread pool defaults should be the same

https://issues.jboss.org/browse/ISPN-10040

Server thread pools now read from embedded

[2019-03-29T08:05:12.636Z] [ERROR] /home/infinispan/workspace/Infinispan_PR-6816/server/integration/infinispan/src/main/java/org/jboss/as/clustering/infinispan/subsystem/ThreadPoolResource.java:39:8: Unused import - org.infinispan.factories.KnownComponentNames. [UnusedImports]

Updated. Sorry, I thought IntelliJ would remove the import when I added the static import for the new method :)

@karesti updated to introduce a constant for the keepalive timeout in BlockingThreadPoolExecutorFactory


Used load_dotenv, bot reponds when mentioned

I have made some changes to the bot.py file, now the bot responds when mentioned, as well as when we use its prefix "." . Also I used "load_dotenv" instead of "dotenv_values", which in my opinion is easier to use rather than "dotenv_values".
A screenshot of the bot -

umm, what do you mean by MSP? main purpose, actually I found some errors in the code so I decided to open a pull request by fixing those changes, although I'll close this PR and submit a new one regarding the issue I was assigned


fix: allow duplicate type names

See #1.

Codecov Report

Merging #2 (c590cd7) into main (9b0a209) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #2   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        13
  Lines          246       246
  Branches        27        27
=========================================
  Hits           246       246

Impacted Files
Coverage Δ

src/verifier.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Matplotlib Issue in Error Messages

Describe the bug
The recent set of merges to dev produce a matlotlib issue.
Traceback (most recent call last):
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/matplotlib/__init__.py", line 522, in _get_config_or_cache_dir
    tmpdir = tempfile.mkdtemp(prefix="matplotlib-")
  File "/opt/conda/envs/topsapp_env/lib/python3.9/tempfile.py", line 352, in mkdtemp
    prefix, suffix, dir, output_type = _sanitize_params(prefix, suffix, dir)
  File "/opt/conda/envs/topsapp_env/lib/python3.9/tempfile.py", line 122, in _sanitize_params
    dir = gettempdir()
  File "/opt/conda/envs/topsapp_env/lib/python3.9/tempfile.py", line 291, in gettempdir
    tempdir = _get_default_tempdir()
  File "/opt/conda/envs/topsapp_env/lib/python3.9/tempfile.py", line 223, in _get_default_tempdir
    raise FileNotFoundError(_errno.ENOENT,
FileNotFoundError: [Errno 2] No usable temporary directory found in ['/tmp', '/var/tmp', '/usr/tmp', '/home/ops']
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/opt/conda/envs/topsapp_env/lib/python3.9/runpy.py", line 188, in _run_module_as_main
    mod_name, mod_spec, code = _get_module_details(mod_name, _Error)
  File "/opt/conda/envs/topsapp_env/lib/python3.9/runpy.py", line 147, in _get_module_details
    return _get_module_details(pkg_main_name, error)
  File "/opt/conda/envs/topsapp_env/lib/python3.9/runpy.py", line 111, in _get_module_details
    __import__(pkg_name)
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/isce2_topsapp/__init__.py", line 4, in <module>
    from isce2_topsapp.delivery_prep import prepare_for_delivery
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/isce2_topsapp/delivery_prep.py", line 9, in <module>
    from matplotlib import cm
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/matplotlib/__init__.py", line 965, in <module>
    dict.update(rcParams, _rc_params_in_file(matplotlib_fname()))
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/matplotlib/__init__.py", line 613, in matplotlib_fname
    for fname in gen_candidates():
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/matplotlib/__init__.py", line 610, in gen_candidates
    yield os.path.join(get_configdir(), 'matplotlibrc')
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/matplotlib/__init__.py", line 303, in wrapper
    ret = func(**kwargs)
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/matplotlib/__init__.py", line 556, in get_configdir
    return _get_config_or_cache_dir(_get_xdg_config_dir)
  File "/opt/conda/envs/topsapp_env/lib/python3.9/site-packages/matplotlib/__init__.py", line 524, in _get_config_or_cache_dir
    raise OSError(
OSError: Matplotlib requires access to a writable cache directory, but the default path (/home/ops/.config/matplotlib) is not a writable directory, and a temporary directory could not be created; set the MPLCONFIGDIR environment variable to a writable directory

All the jobs fail rather quickly e.g.
{'job_id': '9eaf520e-698e-46d3-b6a3-fb2b6206b75d',
  'job_type': 'INSAR_ISCE_TEST',
  'request_time': '2023-07-13T20:40:40+00:00',
  'status_code': 'FAILED',
  'user_id': 'cmarshak',
  'name': 'Saudi-Arabia_72_GMAO_0713',
  'job_parameters': {'compute_solid_earth_tide': True,
   'estimate_ionosphere_delay': True,
   'frame_id': 11107,
   'granules': ['S1A_IW_SLC__1SDV_20220125T145707_20220125T145734_041619_04F365_A0A6',
    'S1A_IW_SLC__1SDV_20220125T145732_20220125T145759_041619_04F365_F27F'],
   'secondary_granules': ['S1A_IW_SLC__1SDV_20220113T145708_20220113T145735_041444_04ED75_4324',
    'S1A_IW_SLC__1SDV_20220113T145732_20220113T145759_041444_04ED75_48DD'],
   'weather_model': 'GMAO'},
  'logs': ['https://hyp3-tibet-jpl-contentbucket-81rn23hp7ppf.s3.us-west-2.amazonaws.com/9eaf520e-698e-46d3-b6a3-fb2b6206b75d/9eaf520e-698e-46d3-b6a3-fb2b6206b75d.log'],
  'expiration_time': '2023-07-28T00:00:00+00:00',
  'processing_times': [1.247]}

@mgovorcin - when you ran and generated your sample products on leffe for #142, did you manually set OMP_NUM_THREADS?
I have a feeling this is related to running out of memory on the instance. I am seeing similar issues running the current dev branch on leffe and the bug is challenging to track down. I was wondering too if you knew the disk space required for the new updates.
https://github.com/ASFHyP3/hyp3/blob/3645e0ec8fe685724aaf013dbda3350b7f59317c/.github/workflows/deploy-enterprise.yml#L35-L78
If we were to set this environment variable - do you set the number of threads to 4 for the entire workflow or just for iono? Thanks for your help.

Running the product suggested in #142 (to create the sample product there), I am getting the following error:
Masking upper-iono interferograms IW1/04:  62%|██████▏   | 21/34 [00:20<00:10,  1.24it/s]
Masking upper-iono interferograms IW1/05:  62%|██████▏   | 21/34 [00:20<00:10,  1.24it/s]
Masking upper-iono interferograms IW1/05:  65%|██████▍   | 22/34 [00:21<00:09,  1.29it/s]
Masking upper-iono interferograms IW2/01:  65%|██████▍   | 22/34 [00:21<00:09,  1.29it/s]
Masking upper-iono interferograms IW2/01:  68%|██████▊   | 23/34 [00:21<00:08,  1.26it/s]
Masking upper-iono interferograms IW2/02:  68%|██████▊   | 23/34 [00:21<00:08,  1.26it/s]
Masking upper-iono interferograms IW2/02:  71%|███████   | 24/34 [00:23<00:08,  1.12it/s]
Masking upper-iono interferograms IW2/03:  71%|███████   | 24/34 [00:23<00:08,  1.12it/s]
Masking upper-iono interferograms IW2/03:  74%|███████▎  | 25/34 [00:24<00:08,  1.05it/s]
Masking upper-iono interferograms IW2/04:  74%|███████▎  | 25/34 [00:24<00:08,  1.05it/s]
Masking upper-iono interferograms IW2/04:  76%|███████▋  | 26/34 [00:25<00:08,  1.05s/it]
Masking upper-iono interferograms IW2/05:  76%|███████▋  | 26/34 [00:25<00:08,  1.05s/it]
Masking upper-iono interferograms IW2/05:  79%|███████▉  | 27/34 [00:26<00:07,  1.10s/it]
Masking upper-iono interferograms IW2/06:  79%|███████▉  | 27/34 [00:26<00:07,  1.10s/it]
Masking upper-iono interferograms IW2/06:  82%|████████▏ | 28/34 [00:27<00:06,  1.09s/it]
Masking upper-iono interferograms IW3/01:  82%|████████▏ | 28/34 [00:27<00:06,  1.09s/it]
Masking upper-iono interferograms IW3/01:  85%|████████▌ | 29/34 [00:28<00:05,  1.03s/it]
Masking upper-iono interferograms IW3/02:  85%|████████▌ | 29/34 [00:28<00:05,  1.03s/it]
Masking upper-iono interferograms IW3/02:  88%|████████▊ | 30/34 [00:29<00:04,  1.06s/it]
Masking upper-iono interferograms IW3/03:  88%|████████▊ | 30/34 [00:29<00:04,  1.06s/it]
Masking upper-iono interferograms IW3/03:  91%|█████████ | 31/34 [00:30<00:03,  1.09s/it]
Masking upper-iono interferograms IW3/04:  91%|█████████ | 31/34 [00:30<00:03,  1.09s/it]
Masking upper-iono interferograms IW3/04:  94%|█████████▍| 32/34 [00:31<00:02,  1.06s/it]
Masking upper-iono interferograms IW3/05:  94%|█████████▍| 32/34 [00:31<00:02,  1.06s/it]
Masking upper-iono interferograms IW3/05:  97%|█████████▋| 33/34 [00:33<00:01,  1.09s/it]
Masking upper-iono interferograms IW3/06:  97%|█████████▋| 33/34 [00:33<00:01,  1.09s/it]
Masking upper-iono interferograms IW3/06: 100%|██████████| 34/34 [00:34<00:00,  1.20s/it]
Masking upper-iono interferograms IW3/06: 100%|██████████| 34/34 [00:34<00:00,  1.02s/it]

  0%|          | 0/3612 [00:00<?, ?it/s]
  0%|          | 0/3612 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "/mnt/leffe-data2/cmarshak/mambaforge/envs/topsapp_env/bin/isce2_topsapp", line 8, in <module>
    sys.exit(main())
  File "/mnt/leffe-data2/cmarshak/DockerizedTopsApp/isce2_topsapp/__main__.py", line 350, in main
    sys.exit(process_entry_point.load()())
  File "/mnt/leffe-data2/cmarshak/DockerizedTopsApp/isce2_topsapp/__main__.py", line 232, in gunw_slc
    nc_path = update_gunw_with_solid_earth_tide(nc_path, "reference", loc_data['reference_orbits'])
  File "/mnt/leffe-data2/cmarshak/DockerizedTopsApp/isce2_topsapp/solid_earth_tides.py", line 454, in update_gunw_with_solid_earth_tide
    solid_earth_tide_ds = compute_solid_earth_tide_from_gunw(gunw_path=gunw_path,
  File "/mnt/leffe-data2/cmarshak/DockerizedTopsApp/isce2_topsapp/solid_earth_tides.py", line 184, in compute_solid_earth_tide_from_gunw
    tide_e, tide_n, tide_u = compute_enu_solid_earth_tide(orbit_xmls=orbit_xmls,
  File "/mnt/leffe-data2/cmarshak/DockerizedTopsApp/isce2_topsapp/solid_earth_tides.py", line 352, in compute_enu_solid_earth_tide
    azimuth_time_arr = get_azimuth_time_array(orbit_xmls,
  File "/mnt/leffe-data2/cmarshak/DockerizedTopsApp/isce2_topsapp/solid_earth_tides.py", line 250, in get_azimuth_time_array
    datetime_isce, rng = orb.geo2rdr([lat, lon, hgt])
  File "/mnt/leffe-data2/cmarshak/mambaforge/envs/topsapp_env/lib/python3.9/site-packages/isce/components/isceobj/Orbit/Orbit.py", line 1048, in geo2rdr
    tguess = tguess - datetime.timedelta(seconds = fn/fnprime)
ValueError: cannot convert float NaN to integer

In leffe, the S1-GUNW is not generated yet i.e. the netcdf data is not there, so I believe that this error is misleading in that the workflow is trying to perform the SET correction and cannot.
@mgovorcin - when you have a chance, please re-try your workflow using this dev branch again using the example you provided reinstalling the environment.
Another piece that could be affecting this is the fsspec library we had to add. Maybe that is causing issues with the packaging? I am not sure.

It turned out that packaging completed (i.e. these lines) and the netcdf was generated. However, the SET correction layer did not complete and had an unusual error.

So to be clear:
I ran the container locally on our linux server and found a bug that fixed by #146. Now, I reinstall the environment and rerun the workflow on a test using:
isce2_topsapp --secondary-scenes S1A_IW_SLC__1SDV_20220131T222803_20220131T222830_041711_04F690_8F5F S1A_IW_SLC__1SDV_20220131T222828_20220131T222855_041711_04F690_28D7  --reference-scenes S1A_IW_SLC__1SDV_20220212T222803_20220212T222830_041886_04FCA3_2B3E S1A_IW_SLC__1SDV_20220212T222828_20220212T222855_041886_04FCA3_A3E2  --frame-id 25502 --estimate-ionosphere-delay True

and the process runs to completion.
When I run:
docker run -ti -v $PWD:/home/ops/topsapp_data --entrypoint /bin/bash ghcr.io/access-cloud-based-insar/dockerizedtopsapp:0.2.4.dev160_g5e9e433
conda activate topsapp_env
cd topsapp_data # home directory
<same command as above with username/password>

Ran out of space on device (might be how docker is installed on our linux server). I am not able to get the bug above in 2 seconds as hyp3 suggests.

And furthermore, the test that I reported failed above can be run locally as:
isce2_topsapp --reference-scenes S1A_IW_SLC__1SDV_20220125T145707_20220125T145734_041619_04F365_A0A6 \
                                 S1A_IW_SLC__1SDV_20220125T145732_20220125T145759_041619_04F365_F27F  \
              --secondary-scenes S1A_IW_SLC__1SDV_20220113T145708_20220113T145735_041444_04ED75_4324 \
                                 S1A_IW_SLC__1SDV_20220113T145732_20220113T145759_041444_04ED75_48DD  \
              --compute-solid-earth-tide True \
              --frame-id 11107 \
              --estimate-ionosphere-delay True \
              > topsapp_img.out 2> topsapp_img.err\

This too locally does not fail after a few seconds. I do set OMP_NUM_THREADS=4 as indicated above as a courtesy to other users of the server.

To answer some of the questions I posed above:

The disk space required for many of the jobs with frames is < 100 GB (standard non-frame based GUNWs typically required more ~130 GB). I believe the memory overhead is less because the geocoding step cuts the final 30 meter products generated by ISCE2 to the frame extent which is smaller than a single S1 frame.
I do not know what the memory require to run unwrapping is; it would be surprising that memory was the culprit since we had run ionosphere corrections previously and this did not cause any problems.

@cmarshak , @AndrewPlayer3 on our team offered to try to reproduce the matplotlib error in one of our copies of HyP3 and investigate further today, FYI

@cmarshak, @asjohnston-asf After running some test jobs in one of our instances using the topsapp test image, I was not able to reproduce this error.

I've been able to confirm this is due to the job running out of disk space. Jobs in the hyp3-tibet-jpl deployment are limited to 100 GB or 118 GB of disk space.
We weren't able to reproduce the issue in an ASF environment because we use a different EC2 instance type that gives jobs 237 GB of disk space.
We attempt every job up to three times. The initial attempt fails after running out of disk in ~2 hours. The subsequent two attempts fail immediately with the Matplotlib error because the disk is already full when they start; the retry attempt is starting faster than the docker engine can release the disk from the first attempt. The HyP3 API only reports the log and processing time from the final attempt, which obscured the underlying failure in the first attempt.
I'll forward a log file later today so JPL can review where in the workflow the job is hitting its disk limit.
I recommend reviewing the workflow to reduce it's overall disk space requirement. Investigating whether there's an opportunity to delete intermediate files after they've been used would be a good first step.
We can also provision more disk space for each job if necessary, but that will make the workflow more expensive. I can provide more detail on the tradeoffs if that's an option you'd like to explore.

Thank you so much @asjohnston-asf and @AndrewPlayer3.
I don't think it's a good idea to reduce disk space. One of ISCE2's weakness is the intermediate io and managing this is really a big ask.
If we have been doing the expanded disk space for access, let's do it for Tibet.
Questions:

How did you get the 1st job’s log via AWS? Or did retrieving the log require resubmitting the job and carefully monitoring?
Can you control (or inspect) the disk availability of instances via Hyp3 yml files? If so, can you please link?

Part of my confusion was my inability to answer these questions. Super appreciate your help!

Individual attempts are logged to AWS Cloudwatch. I typically look up the job by ID in the Step Functions console, then click the link to the underlying AWS Batch job from the list of Events. The details of the Batch job will include a list of attempts, each with a link to an AWS Cloudwatch log stream.

AWS Batch only keeps job history for about a week, so the logs get tougher to track down after that

Disk availability is based on the EC2 instance types used by a particular deployment. The three GUNW deployments use a combination of c6id.xlarge and c5d.xlarge instance types, which come with 118 GB and 100 GB of disk space, respectively.

https://github.com/ASFHyP3/hyp3/blob/develop/.github/workflows/deploy-enterprise.yml#L35
https://aws.amazon.com/ec2/instance-types/ -> Compute Optimized -> C6i/C5 tabs

Two more follow up questions:

Can we confirm that the failed jobs are going to the c5d.xlarge? I am not sure how to inspect that.
How did you determine the instance storage available?

The link you shared only says EBS-only for disk space.

If that is indeed the case (and we don't need more storage) I created this PR: https://github.com/ASFHyP3/hyp3/pull/1752
Otherwise, we should find instance with more storage because it's not worth the development to manage disk space for this workflow.

Also, thank you for the clarification - I see the "Job Attempts" tab under the batch with different log streams for each batch. Super helpful.

I can confirm that resubmitting jobs that failed on tibet successfully ran to completion using the ACCESS account (with a few transient exceptions). This means the 18 GB that Andrew cited above would likely fix the issue being reported here.

This was resolved using instances with more disk space.


Consolidate kube-proxy module and sysctl loading

This sets br_netfilter and net.bridge.bridge-nf-call-iptables sysctl from a single play before kube-proxy is first ran instead of from the flannel and weave network_plugin roles after kube-proxy is started

ci check this

@chadswen for what it's worth calico does this automatically. In any case, it will not break any other plugins that we current support. That having been said, I think we should refactor some tasks to a network_plugins/common role, rather than scattering it across preinstall and kubernetes/node role. For now, let's take your approach and then consolidate later

@mattymo Perhaps a kernel-config role? I think I like that, but I also see potential value in setting sysctls/loading modules only in the roles of the components that need them. That was the main reason I favored loading earlier on with this approach. kube-proxy will print warnings when it starts if it doesn't find the module/sysctl configuration that it needs for iptables mode, so I wanted to make sure we ordered the config before starting kube-proxy. A network_plugins/common role would also need to only make changes that run in that same order.

Looks like CI passed on this, but the PR check status wasn't updated? I can see a failure when tearing down GCE nodes on ubuntu-weave-sep, but the job still passed and install/tests worked.


Installing WCSim with Root6 and GEANT4-10.* from homebrew

Hi,
Could someone explain how to "try" to load WCSim using homebrew root and GEANT4? brew installs Root6 and geant4-10+; has anyone successfully configured it and if so, could someone guide me through installation? thanks!

Hi @ghkareem Geant 4.10.1 is the current recommended version, but if you're using 4.10.2 or beyond, reverting #278 might work (it does for 4.10.2, but may not work for the latest version). Note that it is untested, and WCSim has not been tuned to these versions (but some studies have used 4.10.2/3).
I've not tried with root6 - again an older version (5) is the current recommended version. There has been some discussion to move to root6 in HyperK in the nearish future, so if you fancy working out the required changes and putting a PR in, that'd be great and much appreciated!


V0.1.9

V0.1.8:

Remove cloning resource attributes from prior resource (CHEF-3694)
Permanent fix for #8 - remote_file for enterprise edition with authentication
After PR #12 attribute mesh-seed-address-port not needed.
Update cluster.rb recipe for default value and support chef-solo if possible
Restart amc applicatoin if aerospike was restarted
Update tests and rake's tasks

V0.1.9:

Update default attribute configuration
Remove notify for AMC dashsboard as problem was fixed at AMC <IP_ADDRESS>
Disable empty xdr section in config file, wich caused an error for EE (Enterprise Edition)
Update aerospike-cluster::amc for installation AMC EE. Closed #8

@pioneerit  LGTM


It is possible to commit a migration without adding the version to schema_migrations

Environment

Elixir version (elixir -v):

Erlang/OTP 21 [erts-10.0.7] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [hipe]

Elixir 1.7.3 (compiled with Erlang/OTP 21)

Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.):

PostgreSQL 9.6

Ecto version (mix deps):

ecto 3.0.0
ecto_sql 3.0.1

Database adapter and version (mix deps):

postgres 0.14.0

Operating system:

Current behavior
Include code samples, errors and stacktraces if appropriate.
Expected behavior
In the process of upgrading to Ecto 3.0, I found that it is possible for Ecto to commit a transaction whilst not updating the schema_migrations table. This is never desirable, since it means that migrations could be run more than once. Migrations should always be atomic.
The error that caused me to notice this was as follows:
Our schema_migrations table is old and was originally migrated from a MySQL instance. For this reason it has column inserted_at with type timestamptz not timestamp as Ecto expects. This was never a problem in Ecto 2.x but in Ecto 3.x it raises an error.
Here is an example migration run:
mix ecto.migrate
[debug] QUERY OK source="schema_migrations" db=6.5ms
SELECT e0."version"::bigint FROM "schema_migrations" AS e0 FOR UPDATE []
[info] == Running<PHONE_NUMBER>3658 NestDB.Repo.Migrations.UnifyTimestamps.change/0 forward
[info] execute "ALTER TABLE my_table ADD COLUMN this_column_should_never_be_committed"
[info] == Migrated<PHONE_NUMBER>3658 in 12.2s
[error] Could not update schema migrations. This error usually happens due to the following:

  * The database does not exist
  * The "schema_migrations" table, which Ecto uses for managing
    migrations, was defined by another library

To fix the first issue, run "mix ecto.create".

To address the second, you can run "mix ecto.drop" followed by
"mix ecto.create". Alternatively you may configure Ecto to use
another table for managing migrations:

    config :nest_db, NestDB.Repo,
      migration_source: "some_other_table_for_schema_migrations"

The full error report is shown below.

** (DBConnection.EncodeError) Postgrex expected %DateTime{}, got ~N[2018-11-07 08:57:01]. Please make sure the value you are passing matches the definition in your table or in your query or convert the value accordingly.
    /Users/sam/code/nested/local-stack/elixir-umbrella/deps/postgrex/lib/postgrex/type_module.ex:713: NestDB.PostgrexTypes.encode_params/3
    (postgrex) lib/postgrex/query.ex:62: DBConnection.Query.Postgrex.Query.encode/3
    (db_connection) lib/db_connection.ex:1074: DBConnection.encode/5
    (db_connection) lib/db_connection.ex:1172: DBConnection.run_prepare_execute/5
    (db_connection) lib/db_connection.ex:480: DBConnection.parsed_prepare_execute/5
    (db_connection) lib/db_connection.ex:473: DBConnection.prepare_execute/4
    (postgrex) lib/postgrex.ex:167: Postgrex.query/4
    (ecto_sql) lib/ecto/adapters/sql.ex:627: Ecto.Adapters.SQL.struct/10
make: *** [migrate] Error 1

After this migration run, the SQL has been applied but the version is not present in the schema. Subsequent migrations will run the same SQL again.
Furthermore while trying to fix this I discovered it isn't possible to alter the schema_migrations table inside a migration. It hangs forever.
This leads me to conclude that the transaction and update of schema_migrations happens outside of the main migration transaction. This is not a good design, since it can lead to deadlocks and causes the migration to not commit atomically. Ideally, everything would happen inside the same transaction.

Good catch!
I have a fix in hand. It is more a bug rather than a design flaw, as we can fix it by literally moving some lines of code into a block. :) I am writing the tests and I should push a fix shortly.
I am wondering if there is something we can do about the timestamptz bit but I am not sure. Previously Ecto would not care about this but the whole point of Elixir Calendar types is to make the distinction between those clear.

Here is a PR: https://github.com/elixir-ecto/ecto_sql/pull/30
Can you please try it out? I have been a bit too optimistic about the fix but it should be good to go (there were some complexities regarding failure handling with tasks but we have added tests for them too).

Closing in favor of the PR, we can continue the discussion in the other repo.


permissions in v5

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4588  👈
#4587
#4586
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jnsdls and the rest of your teammates on  Graphite


added  2 functions

added all_of() & any_of() functions with link to run code as well

Thank you for your contribution @Pranav-Khurana  :+1:
:relaxed: :relaxed: :relaxed:


Need a Quick Way to View Parse Results & CSO

We've had the need for the developer to review parse results and conversation state object. A quick way to meet this need without too much effort (that's reasonably acceptable) would be to provide a two tiny icons next to the dialog bubbles.

One would link (POST) to take (new tab) the user to the parse API and display the results to the user.
Another would link to the take the user to another pay that shows the CSO contents

@broham89 need some links or tiny icons that would appear in the text bubbles

On it, I'm trying out different effects to click on the chat bubble that shows the links.  I'm thinking that links text like POST and CSO explain more to the user what they are instead of icons.
Currently this is what i'm thinking of doing (image)... But ran into issues like, what would happen if the user only typed 1 word the chatbubble would be tiny and the View links would just visually hang off. Still working on it.

This is another version i'm currently thinking of working on.

I was thinking the options would appear inside the bubble right-aligned (see image). Also, the buttons would be "Parse Results" and "Context" but they're kind long. I wonder if you could have icons for each like a book and a speech bubble? (Need to work on ideas for what iconography are appropriate for those.)

Maybe the wording should say, "Debug: [Parsing] [Context]"?
@wrathagom your preference?

@ewoo @wrathagom  Could you provide some examples snapshots of what Parse and context looks like and what they are used for? Just enough for me to understand how I can start designing the icons for them.

they are both big ass JSON snippets. one has the NLU (intent/entity) information the other has the slots/historical information.

@ewoo why the duplicate tag, is there another issue for this someplace. I'll read through them all here in a minute.

How about these icons? @ewoo @wrathagom

^ here is a simpler look having the JSON brackets to quickly signify to the user too see the code behind it.

I think without text the user will be baffled of what it means. I think a simple view source type text would suffice. Or maybe the icon as an indicator of a debug type area and then several buttons for parse, context, etc.

Things that I edit/added:

js/script.js line 27-30
scss/components/_chat-bubble.scss line 140-190
settings.html div class="source-links" lines 108-115 and 122-129
added img/json.svg

nlu-styles.zip

new ui address the viewing of /parse we con't store the CSO anywhere, so that will have to be addressed later.


handles case when title_lang=wikidata

Codecov Report

Merging #16 into master will decrease coverage by 0.33%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   40.21%   39.87%   -0.34%
==========================================
  Files          16       16
  Lines         470      474       +4
==========================================
  Hits          189      189
- Misses        281      285       +4

Impacted Files
Coverage Δ

mwtext/utilities/words2plaintext.py
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

Looks good.  Nice work.


rfctr(chunking): split oversized chunks on word boundary

The text of an oversized chunk is split on an arbitrary character boundary (mid-word). The chunk_by_character() strategy introduces the idea of allowing the user to specify a separator to use for chunk-splitting. For langchain this is typically "\n\n", "\n", or " "; blank-line, newline, or word boundaries respectively.
Even if the user is allowed to specify a separator, we must provide fall-back for when a chunk contains no such character. This can be done incrementally, like blank-line is preferable to newline, newline is preferable to word, and word is preferable to arbitrary character.
Further, there is nothing particular to chunk_by_character() in providing such a fall-back text-splitting strategy. It would be
preferable for all strategies to split oversized chunks on even-word boundaries for example.
Note that while a "blank-line" ("\n\n") may be common in plain text, it is unlikely to appear in the text of an element because it would have been interpreted as an element boundary during partitioning.
Add _TextSplitter with basic separator preferences and fall-back and apply it to chunk-splitting for all strategies. The by_character chunking strategy may enhance this behavior by adding the option for a user to specify a particular separator suited to their use case.

Thanks @christinestraub! :)


fix pkgconfig freetype2

Here, I don't know why it failed. I have already built all of the packages, and they are all fine.

I restarted the build and the issue did not appear again. I think Github might have overcomitted their servers a bit.


[PLT-8186] add support for ec2 instance profile authentication

Summary
add support for ec2 instance profile authentication
Ticket Link
github: https://github.com/mattermost/mattermost-server/issues/7875
PR in other repos
redux: https://github.com/mattermost/mattermost-redux/pull/400
server: https://github.com/mattermost/mattermost-server/pull/8243
api-reference: https://github.com/mattermost/mattermost-api-reference/pull/336

Looks great to me.

@cpanato This looks good to me but I'll wait for PM approval first. Thanks!

Yeah, I'll test it as soon as the server and redux PRs are merged

@cpanato if you want to point this at the new redux changes, should be ready for PM testing

@jwilander I did not change the redux thing, It always points to master. Should I point to another tag?

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0830f773335d1e2ac.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-0830f773335d1e2ac

@ccbrown Wondering if you'd be interested helping test IAM roles work as expected? Since you're quite familiar with them and was involved in the initial discussions of this feature.

@jasonblais  I also fix the email test button, it has the same behavior. If you use MM in a large monitor and maximized it always put the message beside the button. see attached image for the email.

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spintmint test server creation failed. See the logs for more information.

Thanks @cpanato! We're having trouble with test servers so I'll test the changes soon. Looks like some of the string changes may not have been pushed to your branch though? (at least for the en.json file)

@jasonblais updated forgot to push, my bad

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

@jasonblais
I created a bucket named "mm-spinmint-test" in region us-west-2 and attached an IAM role for it to the spinmint instance. Everything works fine except for this:

If no region is set, Mattermost attempts to get the appropriate region from AWS

Specifying the region gives me this error: "Connection unsuccessful: S3 Region is required". Specifying "us-west-2" works fine. This looks like a server bug though (not part of this PR).

Also, the back ticks here don't look right:

@jasonblais @ccbrown thanks for the review. I added the check for the s3 region and endpoint. but looks like I did it wrong. I can submit a PR to fix that and make it defaults if not set.
shall I make that?

@cpanato Does https://github.com/mattermost/mattermost-server/pull/8453 address the issue?
Also, looks like the IAM role isn't linked yet

@jasonblais no.

Submitted a fix for the backticks in the help text here https://github.com/mattermost/mattermost-webapp/pull/943

make some fixes to address the @ccbrown  comments: https://github.com/mattermost/mattermost-server/pull/8454

@jasonblais rebased to get your changes.

Thanks!  We should also fix the link in the "IAM role" as it wasn't working on the spinmint. After that we should be ready to merge this PR
https://github.com/mattermost/mattermost-webapp/pull/772#issuecomment-372649307

@jasonblais yes, I push this change as well :)

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-02b2c7db8d4b38189.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-02b2c7db8d4b38189

Spinmint test server destroyed

@saturninoabril ready for last dev review

@cpanato Could you please rebase this to latest master and then see to it that all are still good? I recently merged this PR https://github.com/mattermost/mattermost-webapp/pull/905 and then realized that it might cause conflict with this.

Spinmint test server created at: http://i-0a9cb6af78b674de8.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-0a9cb6af78b674de8

Spinmint test server destroyed


IC API stable_read

stable_read: https://docs.rs/ic-cdk/latest/ic_cdk/api/stable/fn.stable_read.html

stable_read is working, there are a couple of lines in stable_read.ts that may need some optimizations
let mut vec_buf: Vec<u8> = vec![0;length as usize];
We need to see if we can make a mult& [] without having to initialize a vec with all zero values.
return Ok(buf.to_vec().azle_into_js_value(_context));
We need to see if to_vec will be efficient enough. If it's better than O(n) then we might be fine, otherwise we will need to figure out a better way of converting it.

Created new issue for the optimization task so we can merge in the code we have since it works https://github.com/demergent-labs/azle/issues/482


Is there a way to generate a token for a user with a custom serializer?

I see that https://django-rest-framework-simplejwt.readthedocs.io/en/latest/creating_tokens_manually.html provides a way to create tokens manually, which is awesome!
However, it looks like https://django-rest-framework-simplejwt.readthedocs.io/en/latest/customizing_token_claims.html can be used to override the views only and not so much this token generation method?
I need to generate a token with custom claims, is this possible using this framework?

Create a custom authentication class and use that in your REST_FRAMEWORK settings.


DKL-DI-0002 (Avoid sensitive directory mounting) seems too strict

https://github.com/goodwithtech/dockle/blob/master/CHECKPOINT.md#dkl-di-0002-avoid-sensitive-directory-mounting

/boot, /dev, /etc, /lib, /proc, /sys, /usr

I'm not sure there can be an attack vector with these volumes, though the runtime might
have some vulns around mounting /dev, /proc, and /sys, potentially .

I misread this article.
https://www.stackrox.com/post/2017/08/hardening-docker-containers-and-hosts-against-vulnerabilities-a-security-toolkit/#volume-mounts
You're right. I will fix it soon. Thanks.


Swiss FINMA lists

Would these two lists, published by the Swiss Financial Market Supervisory Authority (FINMA), be in scope for OpenSanctions?
https://www.finma.ch/en/finma-public/warning-list/
https://www.finma.ch/en/enforcement/enforcement-tools/publication-of-final-rulings/
According to the FINMA website, most but not all of their “final rulings” (the second list) seem to be a type of debarment.

These both seem super relevant - debarment / crime sort of collections. Are you perhaps interested in doing the crawl?

Done :)
https://www.opensanctions.org/datasets/ch_finma_rulings/
https://www.opensanctions.org/datasets/ch_finma_warnings/


Collect Altitude

We should be able to collect altitude of device and store the data under field "altitude" as float.
Example:
"altitude": 0.0,

This is already collected.


Route traffic directly to service

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

Hi,
Just wanted to know that is it possible by any means or configuration, that Service A communicating with Service B both having sidecar enabled, to communicate without sidecar.
I mean Service A hitting Service B, but this communication is not via sidecar but it is direct Service to Service communication.

@rakeshjain1213 see https://preliminary.istio.io/docs/reference/config/annotations.html, there are various excludePort/excludeIP options you can use to partially opt out of sending through sidecar

Hi John Howard,
As per my understanding, using traffic.sidecar.istio.io/excludeInboundPorts or traffic.sidecar.istio.io/excludeOutboundIPRanges requires a change in Deployment yaml.
If we do so, we need to restart the service. I don't want to restart my service and want to achieve the same with only configuration changes.
Is that possible?
The above requirement is for following scenario :
Suppose Service A is communicating with Service B in production, with sidecar enabled.
Now for some reasons, there is some issue with the sidecar attached to Service B. Now since every communication is through sidecar, we wont be able to hit Service B.
So our need is to do some configuration changes, because of which now Service A sidecar will communicate directly to Service B.
So what we are expected to do is only configuration changes and no restarts/redeploy.

it is not possible to do this without a restart. The interception is determined by iptables which only run during init


Null-safety support

Pls add  nullsafety support  to the package. it's super time saving package. Thanks in advance!

Any update here? As this package is discontinued. So is there any alternative to this package?


The _request status variable stays on TIMEOUT

import scratchattach as scratch3
session = scratch3.login("--", "--")
conn = session.connect_cloud("929934092")
client = scratch3.CloudRequests(conn)
@client.request
def ping():
print("ping request received")
return "pong"
@client.event
def on_ready():
print("request handler is running")
client.run()

Met the same problem. Set timeout_after to bigger value doesn't help. Found if I use the project owner account to login, it works even when server is in different machine. Timeout only happens when login using other accounts.

Met the same problem. I make the python script running in a cloud machine (https://scratch.mit.edu/projects/933459770/). Scratch client set timeout_after to bigger value doesn't help. Found if I use the project owner account to login, it works even when server is in different machine. Timeout only happens when login using other accounts. So it means it can't be shared to other people to use.

It looks like the problem is not happening anymore because I went to your project and the problem didn't occur


PR-branch-auto-delete should be configurable to not delete i.e. develop branch

Description
For free accounts that don't have protected branches PR-branch-auto-delete can't control which branches get deleted after PR merge so this deletes i.e. develop all the time and it needs to be restored.
Possible solution: user editable whitelist where you can add regex exceptions for branches
Screenshot
No response
Example URLs
When merging a PR

We don't do feature-specific settings: https://github.com/refined-github/refined-github/wiki/"Can-you-add-this-feature%3F"#3-it-doesnt-require-options
GitHub has a native auto-delete branch feature now, you should ask them to make it configurable.

Protect the branch via GitHub settings so you won’t risk ever deleting.


Also support function

currently
module.exports = {
  scripts: {
    build: 'babel-node scripts/build'
  }
}

expect to support as well:
module.exports = {
  scripts: {
    build(){
      return  'babel-node scripts/build'
    }
  }
}

Cool. Can you explain a use case for doing this?

Sometimes we need to do more complex in function, just gets more flexible.
e.g.  Inject env variable
module.exports = {
  scripts: {
    build: {
      default: {
        script() {
          return  'babel-node scripts/build'
        }
      },

      dev() {
        process.env.BUILD = {isDev: true}
        return  'babel-node scripts/build'
      }

      prod() {
        process.env.BUILD = {isProd: true}
        return  'babel-node scripts/build'
      }
    }
  }
}

in build.js
const {isDev, isProd} = process.env.BUILD

Ah, I see, here are two workarounds for you:
module.exports = {
  scripts: {
    build: {
      default: 'babel-node scripts/build',
      dev: 'cross-env NODE_ENV=development babel-node scripts/build',
      prod: 'cross-env NODE_ENV=production babel-node scripts/build',
    }
  }
}

Then in build.js
const isDev = process.env.NODE_ENV === 'development'
const isProd = process.env.NODE_ENV === 'production'

Note: It's more common to use the NODE_ENV environment variable for things like this, but you could do something similar with a BUILD variable.

Or:
module.exports = function getPackageScripts(input) {
  process.env.BUILD = {isDev: input === 'build.dev', isProd: input === 'build.prod'}
  return {
    scripts: {
      build: {
        default: 'babel-node scripts/build',
        dev: 'babel-node scripts/build',
        prod: 'babel-node scripts/build',
      }
    }
  }
}

Let me know if you have any questions or if you feel like the documentation/examples could be updated to explain this use case better.

I'm starting to think though that implementing what you're suggesting may be a good idea... Not 100% on board yet. Let me know what you think of my work arounds.

Thanks for reply so soon.
For the example 1:
{
dev: 'cross-env NODE_ENV=development'
}

I don't want to do this because Object syntax has better structure if in complex case not just exports as a string
Then the example 2:
Maybe good, is that supporting now?
module.exports = function getPackageScripts(input) { ... }

I think that it could be helpful to have the functionality you suggest. Feel free to makeapullrequest.com :-)

Would a property getter work here?

That should work! Let me know! If it does, could you add it to the examples file?

getter cannot work for me

Well, feel free to makeapullrequest.com :-)

Hmmm... I've actually decided that this overcomplicates the p-s API and I don't want to do that. So I'm going to close this one.


preloadedState is optional

Thank you for the PR!
I don’t know TypeScript so leaving this for @aikoven and @ulfryk to review.
(It should indeed be optional, and we should support omitting both preloadedState and enhancer.)

Isn't the case of omitted preloadedState already covered by the first overload of StoreCreator?

I learned Typescript four days ago, so bear with me :).
I needed to create a store with redux-socket.io middleware. The key line is here:
const store = applyMiddleware(socketIoMiddleware)(createStore)(reducer)

The typescript compiler was erroring out (no matching type definition for call target) until I added the peloadedState argument:
const store = applyMiddleware(socketIoMiddleware)(createStore)(reducer, {})

I thought I tracked the type definition down to the right place, but it is quite possible that I am off a bit. I am pretty new at this.
Let me know if there is something else I can do to verify the change that needs to be made here without it being too much work for you.
I am using Redux v3.5.2 and Typescript v1.8.10.

@raineorshine You are right, your change to StoreEnhancerStoreCreator fixes exactly your issue.
But change to StoreCreator is unnecessary and creates a bit of ambiguity for which overload to use when calling it with exactly one argument.


Support 0.14

Probably it's time to update react version number in package.json?

Ok, there are two pull request about it


docker inspect  returning wrong information (docker v1.8.3)

Description of problem:
I noticed when doing a docker inspect on a Docker Image that the "Image" information returned is incorrect. Although docker images shows a Repository (test) and Tag (1), docker inspect returns "" for "Image"
When image has a Parent, the "Image" field shows the Repository/Tag of the Parent image.
docker version:
Client:
Version:      1.8.3
API version:  1.20
Go version:   go1.4.2
Git commit:   f4bf5c7
Built:        Mon Oct 12 05:27:08 UTC 2015
OS/Arch:      linux/amd64
Server:
Version:      1.8.3
API version:  1.20
Go version:   go1.4.2
Git commit:   f4bf5c7
Built:        Mon Oct 12 05:27:08 UTC 2015
OS/Arch:      linux/amd64
docker info:
Containers: 25
Images: 92
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 142
Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.16.0-4-amd64
Operating System: Debian GNU/Linux 8 (jessie)
CPUs: 4
Total Memory: 15.7 GiB
Name: raven
ID: 32ES:J7WZ:HDAY:OUQH:HGNW:XTPM:ERLW:L3U6:B6RE:KOP2:LGST:7XDH
uname -a:
Linux raven 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt11-1+deb8u5 (2015-10-09) x86_64 GNU/Linux
Environment details (AWS, VirtualBox, physical, etc.):
physical
How reproducible:
Just run docker inspect on a Docker image which has a Repository & Tag set.
Steps to Reproduce:

Just run docker inspect on a Docker image which has a Repository & Tag set.

Actual Results:
For following image I expect "Image" to return "test:1" instead of just ""...:
docker images | grep d8ec
REPOSITORY          TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
test    1   db8ec01b1f1db      About an hour ago    167.5 MB
docker inspect db8ec01b1f1db
[
{
    "Id": "d8ec01b1f1dbb6442bfa4206c951ccb3614f2135eb66b3305e802d7d9486234b",
    "Parent": "",
    "Comment": "Imported from -",
    "Created": "2015-11-07T01:29:22.550951754Z",
    "Container": "",
    "ContainerConfig": {
        "Hostname": "",
        "Domainname": "",
        "User": "",
        "AttachStdin": false,
        "AttachStdout": false,
        "AttachStderr": false,
        "Tty": false,
        "OpenStdin": false,
        "StdinOnce": false,
        "Env": null,
        "Cmd": null,
        **"Image": ""**,
        "Volumes": null,
        "WorkingDir": "",
        "Entrypoint": null,
        "OnBuild": null,
        "Labels": null
    },
    "DockerVersion": "1.8.3",
    "Author": "",
    "Config": {
        "Hostname": "",
        "Domainname": "",
        "User": "",
        "AttachStdin": false,
        "AttachStdout": false,
        "AttachStderr": false,
        "Tty": false,
        "OpenStdin": false,
        "StdinOnce": false,
        "Env": null,
        "Cmd": null,
        "Image": "",
        "Volumes": null,
        "WorkingDir": "",
        "Entrypoint": null,
        "OnBuild": null,
        "Labels": null
    },
    "Architecture": "amd64",
    "Os": "linux",
    "Size": 167510717,
    "VirtualSize": 167510717,
    "GraphDriver": {
        "Name": "aufs",
        "Data": null
    }
}

Expected Results:
"Image", "test:1" instead of "Image", ""
Additional info:
It seems that if image has a Parent, the "Image" field actually displays the Repository/Tag info or the Parent Image...

This is expected behavior.
The ContainerConfig of an image is the container the image was generated from. In the case of your image, it wasn't generated from a container, but from running docker import.

OK....so take a look at this then...
I ran test:1 and did a docker commit creating image with repo:tag test2:
ContainerConfig.Image is then correctly saying "test:1".
But what about Config.Image?? Isn't that in this case supposed to show "test:2"?
Or do I got it all wrong here?
docker images
REPOSITORY          TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
test                2                   236d97cf0087        4 seconds ago       167.5 MB
test                1                   d8ec01b1f1db        3 hours ago         167.5 MB
docker inspect 236d97cf0087
[
{
    "Id": "236d97cf008708800a1245b9b8731515a20b35ebbe4986a35f8d9f2d445aba39",
    "Parent": "d8ec01b1f1dbb6442bfa4206c951ccb3614f2135eb66b3305e802d7d9486234b",
    "Comment": "",
    "Created": "2015-11-07T05:28:52.184635318Z",
    "Container": "0f4db38481e9341a1b80e4bef3b35d6284d53daa1485358de27f68e6163bb131",
    "ContainerConfig": {
        "Hostname": "0f4db38481e9",
        "Domainname": "",
        "User": "",
        "AttachStdin": true,
        "AttachStdout": true,
        "AttachStderr": true,
        "Tty": true,
        "OpenStdin": true,
        "StdinOnce": true,
        "Env": null,
        "Cmd": [
            "/bin/bash"
        ],
        "Image": "test:1",
        "Volumes": null,
        "WorkingDir": "",
        "Entrypoint": null,
        "OnBuild": null,
        "Labels": {}
    },
    "DockerVersion": "1.8.3",
    "Author": "",
    "Config": {
        "Hostname": "",
        "Domainname": "",
        "User": "",
        "AttachStdin": false,
        "AttachStdout": false,
        "AttachStderr": false,
        "Tty": false,
        "OpenStdin": false,
        "StdinOnce": false,
        "Env": null,
        "Cmd": [
            "/bin/bash"
        ],
        "Image": "",
        "Volumes": null,
        "WorkingDir": "",
        "Entrypoint": null,
        "OnBuild": null,
        "Labels": null
    },
    "Architecture": "amd64",
    "Os": "linux",
    "Size": 532,
    "VirtualSize": 167511249,
    "GraphDriver": {
        "Name": "aufs",
        "Data": null
    }
}
]

Or is the Config section only populated for containers and not for images?

Documentation was improved in https://github.com/moby/moby/pull/43461, but we may consider removing the ContainerConfig from the API response if it wasn't used (i.e., when BuildKit was used to produce the image)


Remove revoked Certificates

In case the apple developer certificate was revoked one can remove the certificate from the path C:\Users\<user>\AppData\Local\Xamarin\iOS\Provisioning\Certificates.
I think this is an important information as windows users (like me) search every single place on the computer and local/user certificate manager without success.
Only with a tool (Everything) I was able to find the files because of the most recent changes to them.

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

ID: 6d17e2db-437f-d45f-f78f-f4b32a40c3f3
Version Independent ID: 44ff0294-e19c-a926-1aa2-338c3b61371a
Content: Apple Account Management - Xamarin
Content Source: docs/cross-platform/macios/apple-account-management.md
Product: xamarin
Technology: xamarin-crossplatform
GitHub Login: @davidortinau
Microsoft Alias: daortin

Hi @SeriousM
Good suggestion! I've added it in a PR that will go live later this week.


Updated JSON schema generator

List of changes (maybe if forgot something)

GenerateJSON schema for each XSD and keep it as file, easier to debug smaller JSON schema compared 1 giant merged file
Updated to latest JSON schema version
Added metadata for each XSD, (ns and filename)
Changed names of generated files
Create (limited) test cases
Added logging
Add first version of new rpp.json entrypoint schema for custom root RPP structures, such as "rpp" -> "request" -> "domain_create"
Added detected extensions to epp_readWriteType and epp_extAnyType to allow better validation of extension points
Updated README

NOTE:

I'm using rpp as name in code for now, this is of course subject to change based on the current naming discussion.
Need to discuss what structure to use for the root "rpp" structure. currently  like it's maybe to convoluted.

the, "create" field for example could be dropped and the "domain_create" renamed?
{
  "request": {
      "create": {
        "domain_create": {
          "name": "example.nl",
          "period": {"#value": 1, "@unit": "y"},
          "contact": [
            { "@type": "admin", "#value": "XXX-123"},
            { "@type": "tech", "#value": "XXX-123"},
            { "@type": "billing", "#value": "XXX-123"}
          ],
          "registrant": "XXX-999",
          "ns": { "hostObj": ["ns1.example.nl"]},
          "authInfo": {"pw": {"#value": "my-secret-token"}}
        }
      }
  }
}

Cool improvements.
Sorry I pushed code directly to your repo. I didn't notice and didn't expect I have rights.


Cloning a repository in Dev Home over an existing folder crashes the app

Dev Home version
<IP_ADDRESS>
Windows build number
10.0.22621.1778
Other software
OS Build Version: 10.0.22621.1778.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.16
Steps to reproduce the bug

Select Machine configuration tab in Dev Home
Set-up and Clone a repository
Select a GitHub repository that has already been previously cloned to the system
Enter a path that contains the GitHub repository local copy, with a directory name matching the repository name.

Expected result
Dev Home automatically discovers and adds the existing repository and attempts to install any missing prerequisites.
Actual result
App crash to desktop, no error message
Included System Information
CPU: Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz
Physical Memory: 31.59GB (16.75GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_<IP_ADDRESS>_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_<IP_ADDRESS>_x64__8wekyb3d8bbwe

@cybtachyon please upgrade your DevHome version.  I am aware some older versions of DevHome did crash when cloning to a non-empty directory.
The most recent version in the store does not crash.


Publish artifact on maven.google.com

[X] I have searched existing issues and confirmed this is not a duplicate

Issues and steps to reproduce
The flexbox artifact can only be found on jcenter.
FAILURE: Build failed with an exception.

* What went wrong:
Gradle yields the following error if `jcenter()` is not in the `repositories {}` configuration closure.

Could not determine the dependencies of task ':app:testDebugUnitTest'.
> Could not resolve all task dependencies for configuration ':widgets:debugUnitTestRuntimeClasspath'.
   > Could not find com.google.android:flexbox:1.1.0.
     Searched in the following locations:
       - https://dl.google.com/dl/android/maven2/com/google/android/flexbox/1.1.0/flexbox-1.1.0.pom
       - https://dl.google.com/dl/android/maven2/com/google/android/flexbox/1.1.0/flexbox-1.1.0.jar

Expected behavior
The following gradle repositories configuration should be enough to download the flexbox artifact.
allprojects {
    repositories {
        google()
    }
}

Version of the flexbox library
any version
Please consider this to avoid future problems similar to https://issuetracker.google.com/issues/120759347#comment30

Seems it will be resolved in scope of https://github.com/google/flexbox-layout/issues/566 (which is duplicate, but apparently got more attention 🤷♂)

Duplicate of #566


Randomize name of Ingress sharding svc

There's a problem of CPO distinguishing LBs created by different clusters on a single OpenStack project when they have the same namespace and nam. Normally it's not a problem for tests as test namespaces names are generated randomly, but in case of the should create a TCP %s LoadBalancer when LoadBalancerService ingressController is created on Openshift test, the LB always gets created in the openshift-ingress namespace, so conflicts are possible.
This commit makes sure that name of the IngressController is randomized to avoid that.

interesting. Can this conflict happen in the open?

interesting. Can this conflict happen in the open?

Yes amd it does. I'm looking into fixing this, but it's not trivial when you consider you need a solid upgrade solution too.

2023-08-30T18:12:42.638904615Z I0830 18:12:42.638840       1 event.go:307] "Event occurred" object="openshift-ingress/router-canary-sharding-3bbcd1bb-4a14-408b-8cf7-a03fa0616210" fieldPath="" kind="Service" apiVersion="v1" type="Warning" reason="SyncLoadBalancerFailed" message="Error syncing load balancer: failed to delete load balancer: load balancer 5d747436-6494-4c26-825f-a54a9d60dd3b is in immutable status, current provisioning status: PENDING_CREATE"

Testing cloud issue?

/test test

2023-08-31T09:46:31.124665621Z E0831 09:46:31.124610       1 controller.go:291] error processing service openshift-ingress/router-canary-sharding-d7e3d3b1-bd51-456a-baa0-f7a0f275df71 (will retry): failed to ensure load balancer: load balancer 1386eba4-c7ec-4ee0-800d-bc9feadc0f87 is not ACTIVE, current provisioning status: ERROR

LB gone into ERROR…

/test test

Invalid value: \"router-internal-canary-sharding-d800d53d-5bcf-4c12-87c9-e79e30114906\": must be no more than 63 characters"

Ewww.

/retest
Unrelated.

/retest

/retest
ERROR and PENDING_CREATE LBs.

/retest

/test test

@pierreprinetti, this is ready and passing now.
/cc @rlobillo

/approve
/lgtm

/override ci/prow/test-kuryr

https://github.com/openshift/release/pull/43199

/override ci/prow/test-kuryr
/skip


chore: Comply with eslint and prettier-eslint

Note that the two are not quite the same. The line length in the .prettierrc
is not an absolute ceiling, so this commit reduces it to 78 so that the output
will all comply with the eslint limit of 80.
Resolves #100.
Note that I had to manually move a number of style parameters from the template section to the style section in a few .vue files. For reasons I do not entirely understand, these parameters were no longer being picked up in the browser after the prettier-eslint transformation, but they work OK in the style section, which is probably a better place for them anyway.

Unfortunately this commit will pretty much destroy the the meaningfulness of the "blame" view on Github, but better to do that now rather than wait even longer. Now that formatting will be consistent, there should be very few changes caused by reformatting.

Please test as thoroughly as you have the stomach for, considering the magnitude of the changes (even though they are almost all whitespace or punctuation changes). Thanks!

This passes our minimal test suite. When I tested manually, I noticed that our footer is gone.
In bfd2e2da8b89e7d0ff6956d1290fa3bdb10d493e:

In this PR:

I will add a commit that fixes this momentarily.

@gwhitney, take a look at https://github.com/numberscope/frontscope/commit/fa713301a2d464aed28f19bb6eaf394781b9931f. If all looks good to you, I think we should merge.

Good catch, Liam! Sorry I missed that. I should have done before-and-after screenshots like you did, once I noticed something was wrong. Anyhow, the additional commit looks good to me, and the extra functionality in the footer and the test for it are a +1. So as far as I am concerned, you can merge when you are comfortable.

Hmm... I think my fix didn't get merged. I will open a PR with the fix in just a second.


drivers/flash: flash_get_page_info_by_off uses relative addresses

Describe the bug
In general embedded flash driver uses addresses according to SoC memory map, so absolute address.
This is true for read, write and erase calls but not for flash_get_page_info_by_offs call which uses offset in relation to the flash begin address.
Origin of the bug:
The flash shim API `struct flash_driver_api' allows to get flash layout in shape of run-length encoded description. Because of its nature from such description can't tell what is the absolute offset of the sector.
The flash shim API doesn't allow to transfer information about the flash begin offset.
solution
Introduce flash shim API for fetching flash base address and use it for fixing the call

cc @ioannisg @carlescufi

Looks like i recognize this improperly. Flash driver API uses relative addressing,
This is NOT an issue the.


Use a service account instead of a user account

How would I go about doing this with a service account so I can use my automation process to execute this?  I'm uncomfortable storing expiring user credentials (that may have MFA) in my automation scripts.
Also, is this a one time link?  Can we delete the user id's after I do this setup?

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

ID: 52fe18fe-efc5-3e63-8200-8e20e885b55e
Version Independent ID: 86a4f465-d676-e0e4-7367-2ed9f906340a
Content: Create an Azure virtual network peering - Resource Manager - different subscriptions
Content Source: articles/virtual-network/create-peering-different-subscriptions.md
Service: virtual-network
GitHub Login: @anavinahar
Microsoft Alias: anavin

You can use service principal: https://blogs.msdn.microsoft.com/arsen/2018/11/20/azure-vnet-peering-across-azure-active-directory-tenants-using-service-principal-authentication/
Yes, it is a one time link. Yes.

That video is exactly what I needed.  Can we put some links to multi-tenant service principals documentation and potentially write up the content of the video? or link to it?  Its really changing the focus of the document from a person clicking through the portal to an automation account that I think we should showcase.
Thanks!

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


BUG:  eliminate_zeros raises when indptr.dtype != indices.dtype

Describe your issue.
eliminate_zeros raises when inptr.dtype differs from indices.dtype.

I am aware that only np.int32 and np.int64 are supported data dtypes for intptr and indices, but the problem occurs when one of the datatypes is int32 and one is int64.
This might be a known limitation, but I was unable to find a mention of this in the documentation. Happy to close if it is mentioned somewhere.

Thanks!
Reproducing Code Example
Python 3.11.9 (main, Aug 16 2024, 13:37:14) [GCC 13.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import scipy
>>> import numpy as np
>>> test_array = scipy.sparse.csr_array([1, 2, 3, 0, 4])
>>> test_array.indptr = test_array.indptr.astype(np.int64)
>>> test_array.eliminate_zeros()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".venv/lib/python3.11/site-packages/scipy/sparse/_compressed.py", line 1207, in eliminate_zeros
    _sparsetools.csr_eliminate_zeros(M, N, self.indptr, self.indices, self.data)
ValueError: Output dtype not compatible with inputs.
>>> test_array.indices = test_array.indices.astype(np.int64)
>>> test_array.eliminate_zeros()

Error message
ValueError: Output dtype not compatible with inputs.

SciPy/NumPy/Python version and system information
1.14.1 1.26.4 sys.version_info(major=3, minor=11, micro=9, releaselevel='final', serial=0)
Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /opt/_internal/cpython-3.11.9/lib/python3.11/site-packages/scipy_openblas32/include
    lib directory: /opt/_internal/cpython-3.11.9/lib/python3.11/site-packages/scipy_openblas32/lib
    name: scipy-openblas
    openblas configuration: OpenBLAS 0.3.27.dev DYNAMIC_ARCH NO_AFFINITY Zen MAX_THREADS=64
    pc file directory: /project
    version: 0.3.27.dev
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /opt/_internal/cpython-3.11.9/lib/python3.11/site-packages/scipy_openblas32/include
    lib directory: /opt/_internal/cpython-3.11.9/lib/python3.11/site-packages/scipy_openblas32/lib
    name: scipy-openblas
    openblas configuration: OpenBLAS 0.3.27.dev DYNAMIC_ARCH NO_AFFINITY Zen MAX_THREADS=64
    pc file directory: /project
    version: 0.3.27.dev
  pybind11:
    detection method: config-tool
    include directory: unknown
    name: pybind11
    version: 2.12.0
Compilers:
  c:
    commands: cc
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  c++:
    commands: c++
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 3.0.11
  fortran:
    commands: gfortran
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  pythran:
    include directory: ../../tmp/pip-build-env-k6xvzqr5/overlay/lib/python3.11/site-packages/pythran
    version: 0.16.1
Machine Information:
  build:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
  cross-compiled: false
  host:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
Python Information:
  path: /opt/python/cp311-cp311/bin/python
  version: '3.11'

Thanks for the report. We do not support different index dtypes for the same sparse array/matrix, because doing so would require a significant increase in the amount of native code required.
That said, this issue indicates that we should:

improve the error message for this case
make sure the docs are clear about what we support.

@dschult FYI

That makes sense. Might I suggest asserting this and raising on the setters of indptr and indices?
From my limited understanding, changing the dtype of inptr or indices should (1) not be possible or (2) there should be a 'safe' method to do this. This is somewhat related to https://github.com/scipy/scipy/pull/21858, but for (2) to be possible, the function propose in this PR (or a derivative) would need to make its way into the public API somehow.


Funnel right side labels are not visible properly

Refer above snap.

you can configure the property of AAChartModel named marginLeft to avoid it


Invalid Session problem

I've been brainstorming this for quite a while now. I`m importing information from csv files into SugarCrm and at some point i get an "invalid session" thrown. I checked the the following code:
rescue SugarCRM::InvalidSession => error
@errors << error
old_session = @sugar_session_id.dup
login!
# Update the session id in the request that we want to retry.
json.gsub!(old_session, @sugar_session_id)
send!(method, json, max_retry.pred)
and after some debugging i found out that the session was not updated( old_session == @sugar_session_id after the login! method which should update the session_id). I tried to mess with the code a little bit, tried to logout, then login again but nothing seems to help. Any ideas?

I got the same error, but found when it has been appear. When in string for save/update object exist this one < and &rt; . So my hot fix is string.gsub!(/\</, '') and string.gsub!(/\&rt;/, '')


matterbridge: Update to v1.16.1

Matterbridge upstream cut a new release of Matterbridge with some fixes
for various Slack-related bugs. See the full changelog for more details:
https://github.com/42wim/matterbridge/releases/tag/v1.16.1

@Tjzabel Thanks! :+1: Merging, and I will run the playbook to follow.