Kibana fails to start when using the Network Plugin

Hi,
I tried using the Kibana network plugin on my Ubuntu 18 version by downloading the latest version of the network plugin tar.gz file as the Kibana version on my system is 7.6
But when I start the Kibana service - it fails with the below mentioned error in the logs:
**> FATAL  Error: Optimizations failure.

ERROR in ./plugins/network_vis/public/network_vis.js
Module not found: Error: Can't resolve 'ui/vis/vis_factory' in '/usr/share/kibana/plugins/network_vis/public'
kibana.service: Main process exited, code=exited, status=1/FAILURE
kibana.service: Failed with result 'exit-code'.
kibana.service: Service hold-off time over, scheduling restart.**

Below is the details of package.json file
**> {

"name": "network_vis",
"version": "7.5.2",
"kibana": {
"version": "kibana"
},
"authors": [
"David Moreno Lumbreras<EMAIL_ADDRESS>],
"license": "Apache-2.0",
"repository": {
"type": "git",
"url": "https://github.com/dlumbrer/kbn_network"
},
"dependencies": {
"vis-network": "7.2.0",
"randomcolor": "0.5.0",
"css-element-queries": "0.3.2",
"keycharm": "^0.2.0",
"moment": "^2.24.0",
"vis-data": "^6.2.1",
"vis-util": "^1.1.8"
}
}**

And below is the content in the network_vis.js file
**> import "plugins/network_vis/network_vis.less";

import { KbnNetworkVisController } from './network_vis_controller'
import { visFactory } from 'ui/vis/vis_factory';
import { Schemas } from 'ui/vis/editors/default/schemas';
import { setup as visualizations } from '../../../src/legacy/core_plugins/visualizations/public/np_ready/public/legacy';
import image from './images/icon-network.svg';
import networkVisTemplate from 'plugins/network_vis/network_vis.html';
import networkVisParamsTemplate from 'plugins/network_vis/network_vis_params.html';
import { AngularVisController } from 'ui/vis/vis_types/angular_vis_type';
// register the provider with the visTypes registry
visualizations.types.registerVisualization(NetworkVisTypeProvider);
// define the TableVisType
function NetworkVisTypeProvider(Private) {
// return the visType object, which kibana will use to display and configure new
// Vis object of this type.
return visFactory.createBaseVisualization({
name: 'network',
title: 'Network',
image,
description: 'Displays a network node that link two fields that have been selected.',
visualization: AngularVisController,
visConfig: {
defaults: {
showLabels: true,
showPopup: true,
showColorLegend: true,
nodePhysics: true,
firstNodeColor: '#6F86D7',
secondNodeColor: '#DAA05D',
canvasBackgroundColor: '#FFFFFF',
shapeFirstNode: 'dot',
shapeSecondNode: 'box',
displayArrow: false,
posArrow: 'to',
shapeArrow: 'arrow',
smoothType: 'continuous',
scaleArrow: 1,
minCutMetricSizeNode: 0,
maxNodeSize: 80,
minNodeSize: 8,
maxEdgeSize: 20,
minEdgeSize: 0.1,
springConstant: 0.001,
gravitationalConstant: -35000,
labelColor: '#000000'
},
template: networkVisTemplate,
},
editorConfig: {
optionsTemplate: networkVisParamsTemplate,
schemas: new Schemas([
{
group: 'metrics',
name: 'size_node',
title: 'Node Size',
mustBeFirst: 'true',
min: 1,
max: 1,
defaults: [
{ type: 'count', schema: 'size_node' }
]
},
{
group: 'metrics',
name: 'size_edge',
title: 'Edge Size',
max: 1,
},
{
group: 'buckets',
name: 'first',
icon: 'fa fa-circle-thin',
mustBeFirst: 'true',
title: 'Node',
min: 1,
max: 2,
aggFilter: ['terms']//Only have sense choose terms
},
{
group: 'buckets',
name: 'second',
icon: 'fa fa-random',
title: 'Relation',
max: 1,
aggFilter: ['terms']
},
{
group: 'buckets',
name: 'colornode',
icon: 'fa fa-paint-brush',
title: 'Node Color',
max: 1,
aggFilter: ['terms']
},
{
group: 'buckets',
name: 'colornode',
icon: 'fa fa-paint-brush',
title: 'Node Color',
max: 1,
aggFilter: ['terms']
}
])
},
responseHandlerConfig: {
asAggConfigResults: true
},
// structures the data tables (returned by kibana in resp and in UI Inspect)
hierarchicalData: function (vis) {
return true;
},
});
}
export default NetworkVisTypeProvider;**

I am unable to get a proper resolution for it from anywhere on the google or the digitalocean website.
Can anyone help me here with this issue ? How can this be resolved and what changes will I have to make?

Could you try using the code from the 7-dev branch? thanks!

I tried with that as well and still got the same issue.
Is there any kind of changes which I am supposed to do in the files?

I've also tried running the 7-dev release (and tried the git clone method) with the same error. Running on Windows 10, Kibana 7.6.0.
 FATAL  Error: Optimizations failure.    9267 modules     ERROR in ./plugins/network_vis/public/network_vis.js     Module not found: Error: Can't resolve 'ui/vis/vis_factory' in 'C:\kibana-7.6.0-windows-x86_64\kibana-7.6.0-windows-x86_64\plugins\network_vis\public'

Got the same error with docker.
ELK_VERSION=7.6.0
ARG ELK_VERSION

# https://www.docker.elastic.co/
FROM docker.elastic.co/kibana/kibana:${ELK_VERSION}

# Add your kibana plugins setup here
RUN curl -L https://github.com/dlumbrer/kbn_network/releases/download/7.5.X-1/network_vis-7.tar.gz | tar xz -C /usr/share/kibana/plugins

FATAL  Error: Optimizations failure.
   9267 modules

    ERROR in ./plugins/network_vis/public/network_vis.js
    Module not found: Error: Can't resolve 'ui/vis/vis_factory' in '/usr/share/kibana/plugins/network_vis/public'

@dlumbrer
Any update if we could get a fix for it ?
As I have to use it for one of the project requirement and can only be worked with the Network Plugin.
Please respond ASAP.
Thanks,
Ankit Agarwal

Hey @ankitagarwal10k , @StanGirard , @StuartSmith21 , did you get a work around this issue?
Working with a PATCH on #83  fixed it for me. Try this !

I tried the patch on the windows system but after the checkout step nothing worked for me.

It's weird ! May be we can help you with the patch installation. After the checkout step you have to apply the patch with the git command.
Did you have an error with this step ? Could you give us more details ?


[AutoRelease] t2-dashboard-2022-12-26-04799(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3580
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=4686

issue link:https://github.com/Azure/sdk-release-request/issues/3580


feature request: Make service selectors configurable

Currently, afaik flagger manages the creation of the Service object regarding the exposure of the Deployment to which one wants to have canary or blue-green strategies.
The issue is that the corresponding selectors are not configurable, i.e. the are automatically generated and they are of the form
  selector app=<name>-primary

and
  selector app=<name>

Perhaps it would be convenient to be able to customize this.
For our case, we have a huge codebase of helm templates producing helm releases ending up to helmrelease files following the weaveworks gitops flow and it would require a significant amount of work to adjust them in order to make the best use of flagger

the label selector is configurable see https://docs.flagger.app/faq#label-selectors

Thanks @stefanprodan.
The documentation reads:

Besides app Flagger supports name and app.kubernetes.io/name selectors. If you use a different convention you can specify your label with the -selector-labels flag.

does this mean name is also supported out-of-the-box? (or this also should be passed in the -selector-labels flag?)
And if I may ask where is this flag (-selctor-label) applied?
Within the manifest or e.g. during `kubectl apply -f </path/to/canary.yaml> ?

If both app and name are present in your app deployment template spec, then app will be used. You can edit the Flagger deployment and add -selector-labels=name to the command list.

Make sure that the name value is unique per namespace, otherwise the ClusterIPs will point to multiple deployments.


Confirmation Alerts | Cancel a Credit Validation/Award/Reduction Recommendation |1|

https://trello.com/c/1ot4QX8x/572-confirmation-alerts-cancel-a-credit-validation-award-reduction-recommendation-1
Story:
As Director, I want to receive a confirmation alert when I Cancel a recommended Credit Validation/Award/Reduction so that I don't accidentally cancel the recommendation by mistake.
Scenario:
GIVEN that I am a user
AND my organization is Government
AND my role is Director
AND I have received a recommended Credit Validation/Award/Reduction from a Gov't Analyst
AND the Gov't Analyst has made a mistake in the details of the recommendation
AND I am viewing the recommended Credit Validation/Award/Reduction
WHEN I click on Cancel
THEN I receive a confirmation alert that asks me to confirm that I want to Cancel the Credit Validation/Award/Reduction and send it back to the Government Analyst who initiated the recommendation
AND I can choose to either confirm or close the confirmation alert (i.e. cancel the action)
Demo:
Show the alert when Cancelling a Credit Validation/Award/Reduction
Show that the Director can either confirm the cancellation or close the confirmation alert (i.e. cancel the action)
Show that when the Director confirms the cancellation that the recommendation is returned to the Government Analyst who initiated the recommendation

 Confirmation Alerts | Cancel a Credit Validation/Award/Reduction Recommendation |1|


slimserver: Init at 7.9.0 (pkg + module)

Motivation for this change
Add slimserver package + module. Slimserver is the audio server for the Logitech Squeezebox products.
I have used this PR in my personal setup for the last few days and everything works fine.
I will squash the commits once it's merge ready.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

OK, looks good enough, and isolated enough, so that merging instead of forgetting seems net-positive regardless of what's going on with the service…


Race condition during import with multiple processes on shared filesystem

When multiple processes import keras simultaneously, there is a race condition with reading/writng the ~/.keras/keras.json file here: https://github.com/fchollet/keras/blob/master/keras/backend/init.py#L15-L47. One process can be in the middle of writing the file while the other is trying to read it, leading to a failure when the reading process tries to decode the JSON.

#3498 is a duplicate of this, but that issue was marked closed first.
Fixed in https://github.com/fchollet/keras/commit/a4d191d4f97a8100a561b87802fdd6ab5db0ed26


Improved response, now returns top 3 results, fixed tests.

This looks great, @Ivche1337!  I wonder if you can split out a few small commits for things like the one-line error fix.

LGTM (waiting on build)

merged! thanks @Ivche1337!


New starting balance filter crashes report if no other transactions

Name of Defective Feature
Remove Starting Blance
System Settings (Required)
ToolKit Version: 2.1
Operating System (Mac/Windows/Etc): Win 10 Pro
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: 64.0.3282.186
Other Extensions: Lastpass, Adblock
Export of Toolkit Settings (Bottom Left of Settings Page)

Settings Export

[{"key":"AccountsDisplayDensity","value":"1"},{"key":"AccountsEmphasizedOutflows","value":false},{"key":"AccountsStripedRows","value":true},{"key":"AdditionalColumns","value":true},{"key":"AdjustableColumnWidths","value":true},{"key":"AutoDistributeSplits","value":true},{"key":"BetterScrollbars","value":"0"},{"key":"BudgetProgressBars","value":"0"},{"key":"BudgetQuickSwitch","value":true},{"key":"CalendarFirstDay","value":"0"},{"key":"CategoryActivityCopy","value":false},{"key":"CategoryActivityPopupWidth","value":"1"},{"key":"ChangeEnterBehavior","value":false},{"key":"CheckCreditBalances","value":true},{"key":"CheckNumbers","value":false},{"key":"ClearSelection","value":true},{"key":"CollapseSideMenu","value":true},{"key":"ColourBlindMode","value":false},{"key":"CompactIncomeVsExpense","value":true},{"key":"CreditCardEmoji","value":false},{"key":"CurrentMonthIndicator","value":false},{"key":"CustomFlagNames","value":false},{"key":"DaysOfBuffering","value":true},{"key":"DaysOfBufferingHistoryLookup","value":"12"},{"key":"DisableToolkit","value":false},{"key":"DisplayTargetGoalAmount","value":"0"},{"key":"EasyTransactionApproval","value":true},{"key":"EditAccountButton","value":"1"},{"key":"EnableRetroCalculator","value":true},{"key":"EnlargeCategoriesDropdown","value":true},{"key":"EnterToMove","value":true},{"key":"GoalWarningColor","value":false},{"key":"GoogleFontsSelector","value":"0"},{"key":"HideAccountBalancesType","value":"0"},{"key":"HideAgeOfMoney","value":false},{"key":"HideHelp","value":true},{"key":"HideReferralBanner","value":true},{"key":"ImportNotification","value":"1"},{"key":"IncomeFromLastMonth","value":"1"},{"key":"LargerClickableIcons","value":true},{"key":"MonthlyNotesPopupWidth","value":"0"},{"key":"NavDisplayDensity","value":"0"},{"key":"PrintingImprovements","value":true},{"key":"PrivacyMode","value":"0"},{"key":"QuickBudgetWarning","value":true},{"key":"ReconciledTextColor","value":"1"},{"key":"RemovePositiveHighlight","value":false},{"key":"RemoveZeroCategories","value":true},{"key":"ResizeInspector","value":true},{"key":"RightClickToEdit","value":true},{"key":"RowHeight","value":"0"},{"key":"RowsHeight","value":"2"},{"key":"RunningBalance","value":"1"},{"key":"SeamlessBudgetHeader","value":false},{"key":"ShowCategoryBalance","value":true},{"key":"ShowIntercom","value":true},{"key":"SpareChange","value":false},{"key":"SplitKeyboardShortcut","value":true},{"key":"SplitTransactionAutoAdjust","value":false},{"key":"SquareNegativeMode","value":false},{"key":"StealingFromFuture","value":true},{"key":"SwapClearedFlagged","value":false},{"key":"TargetBalanceWarning","value":false},{"key":"ToBeBudgetedWarning","value":true},{"key":"ToggleMasterCategories","value":true},{"key":"ToggleSplits","value":true},{"key":"ToggleTransactionFilters","value":"1"},{"key":"TransactionGridFeatures","value":true},{"key":"goalIndicator","value":true},{"key":"highlightNegativesNegative","value":false},{"key":"l10n","value":"0"},{"key":"pacing","value":"0"},{"key":"popupCalculator","value":true},{"key":"reports","value":true}]

Steps to Reproduce the Behavior

Add new test account with starting balance and no other transactions
Filter Toolkit Income vs Expense report to limit to test account (I limited to test account, current month
Toolkit crashes

Expected Behavior

Show $0 in/out
Actual Behavior

crash
Apparently you didn't anticipate fools like me wanting to report nothing. Actually, I was just testing to see what actually gets suppressed. Apparently it's "Starting Balance" allocated to To Be Budgeted. Or is it something else? I'm testing because I renamed some starting balance transactions differently to distinguish them for my purposes. If this works, I'll name them as required for them to be suppressed in reports.

Well that's odd. If you change he payee of starting balance to something else what happens? I guess that's not the right fix but it should temporarily fix this issue.

It works when named differently. This is an unlikely event, but possible. I have not tested but the report may work if more than one account is included, even if one has only starting balance.
Chris Cowles
From: Josh Madewell
Sent: Friday, March 9, 2018 09:14
To: toolkit-for-ynab/toolkit-for-ynab
Cc: spammagnet; Author
Subject: Re: [toolkit-for-ynab/toolkit-for-ynab] New starting balance filtercrashes report if no other transactions (#1209)
Well that's odd. If you change he payee of starting balance to something else what happens? I guess that's not the right fix but it should temporarily fix this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I've been able to get the same issue. On starting a new budget, I've simply filled in the starting balances of my accounts. When looking at income/expense report, the toolkit has crashed.

Hey @spammagnet the fix for this should be in our current release. If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!


Larry stopped working due to error in Github API

Describe the bug
Github API returned 502 and we had an exception and Larry stopped running
To Reproduce
Steps to reproduce the behavior:

Simulare error from Github API when fetching repositories

Expected behavior
Even if an error from Github happens, the application should not stop but just log an error
Screenshots

Hi @ezeoleaf been a while ha, I could look into this. Please let me know what other tickets I can look into, will try to work on them in my spare time. Well done on the project so far.


Can't find resolve 'react' after installing react-router-dom

I have installed react-router-dom v 4.2.2 and just imported BrowserRouter in index.js.
I am getting the following error in my screen.
./node_modules/react-router-dom/es/BrowserRouter.js
Module not found: Can't resolve 'react' in '/Users/ankur.gupta/Desktop/repo-name/node_modules/react-router-dom/es'
Version
react-router-dom 4.2.2

This isn't a bug with this library. Check your bundler and npm setup.


Create user data hash

Is your feature request related to a problem? Please describe.
Since the user data is sensitive it should be hashed
Describe the solution you'd like

choose a hashing algorithm (prefered argon2id)
create a canonical json from the user data (name, birthday, aktenzeichen)
hash the data using at least two iterations (tbd) and a secret salt that should be stored in a environment variable
add unit tests
provide the salt and an example hashing code snippet to koblenz

Describe alternatives you've considered
use the existing hashing algorithm
Additional context
It has discussed if we switch to the same hashing algorithm for everything (but this also may need additional resources which are not needed)
https://argon2.online/

It has discussed if we switch to the same hashing algorithm for everything (but this also may need additional resources which are not needed)

There is no real need to use a "slow" hash algorithm for the card info hash (as we add random bytes to it - the pepper). Also we'd have to do it backward compatibly somehow.

It has discussed if we switch to the same hashing algorithm for everything (but this also may need additional resources which are not needed)

There is no real need to use a "slow" hash algorithm for the card info hash (as we add random bytes to it - the pepper). Also we'd have to do it backward compatibly somehow.

Yes I was also thinking about that issues. "It has be discussed" Was what I wanted to write...
And yes password hashing would be difficult and we would have to support two ways of hashing maybe old passwords would never be changed if we don't force it


converted to use ge_objects2  (Work in progress, do not merge)

Current coverage is 77.97% (diff: 100%)

Merging #240 into develop will decrease coverage by 0.02%

@@            develop       #240   diff @@
==========================================
  Files            24         24
  Lines          3792       3792
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           2958       2957     -1
- Misses          834        835     +1
  Partials          0          0

Powered by Codecov. Last update 276f701...ceef755

The correct location would be lib/doekbase/data_api/core.py.
The above code area is older code that should be deprecated.


Support for Media and Attachments

Hi, I have a very quick question.
Does this FHIR Server implementation support media and attachment types?
Many thanks!

@dustinmoris we have full support for all the resource types in STU3 and R4. Let us know if you have other questions


Create Book import sys error

Azure Data Studio Version: 1.26.0-insider
Commit: b10b52e4fed689393414aeb18ceb57494fe0dcbf
Date: 2020-12-08T06:45:10.349Z (2 days ago)
VS Code: 1.48.0
Electron: 9.3.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0
Python Version: 3.6

Steps to Reproduce:

Open the Create Jupyter Book via Notebooks tab.
Run all cells
Error occurs in cell two (import sys)

I included most of the error output and excluded the middle traces.
ERROR:root:Internal Python error in the inspect module.
Below is the traceback from this internal error.

Traceback (most recent call last):
  File "<ipython-input-2-4a2654937271>", line 4, in <module>
    version, version_str = getJupyterBookVersion()
  File "<ipython-input-1-1d9929905f3a>", line 14, in getJupyterBookVersion
    version = tuple(int(x) for x in version_str)
  File "<ipython-input-1-1d9929905f3a>", line 14, in <genexpr>
    version = tuple(int(x) for x in version_str)
ValueError: invalid literal for int() with base 10: '/'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 3343, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-2-4a2654937271>", line 23, in <module>
    raise SystemExit(str(e))
SystemExit: invalid literal for int() with base 10: '/'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/IPython/core/ultratb.py", line 1169, in get_records
    return _fixed_getinnerframes(etb, number_of_lines_of_context, tb_offset)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/IPython/core/ultratb.py", line 316, in wrapped
    return f(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/IPython/core/ultratb.py", line 350, in _fixed_getinnerframes
    records = fix_frame_records_filenames(inspect.getinnerframes(etb, context))
  File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/inspect.py", line 1490, in getinnerframes
    frameinfo = (tb.tb_frame,) + getframeinfo(tb, context)
AttributeError: 'tuple' object has no attribute 'tb_frame'
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-2-4a2654937271> in <module>
      3 try:
----> 4     version, version_str = getJupyterBookVersion()
      5     if version:

<ipython-input-1-1d9929905f3a> in getJupyterBookVersion()
     13             version_str = tuple(x.split(': ')[1].replace('.', ''))
---> 14             version = tuple(int(x) for x in version_str)
     15     return version, version_str

<ipython-input-1-1d9929905f3a> in <genexpr>(.0)
     13             version_str = tuple(x.split(': ')[1].replace('.', ''))
---> 14             version = tuple(int(x) for x in version_str)
     15     return version, version_str

ValueError: invalid literal for int() with base 10: '/'

During handling of the above exception, another exception occurred:

SystemExit                                Traceback (most recent call last)
    [... skipping hidden 1 frame]

<ipython-input-2-4a2654937271> in <module>
     22 except Exception as e:
---> 23     raise SystemExit(str(e))

SystemExit: invalid literal for int() with base 10: '/'

During handling of the above exception, another exception occurred:

TypeError                                 Traceback (most recent call last)
    [... skipping hidden 1 frame]

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/IPython/core/interactiveshell.py in showtraceback(self, exc_tuple, filename, tb_offset, exception_only, running_compiled_code)
   2036                            'the full traceback.\n']
   2037                     stb.extend(self.InteractiveTB.get_exception_only(etype,
-> 2038                                                                      value))
   2039                 else:
   2040                     try:

...

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/IPython/core/ultratb.py in find_recursion(etype, value, records)
    449     # first frame (from in to out) that looks different.
    450     if not is_recursion_error(etype, value, records):
--> 451         return len(records), 0
    452
    453     # Select filename, lineno, func_name to track frames with

TypeError: object of type 'NoneType' has no len()

Hi @VasuBhog - I wasn't able to repro the issue using python 3.6.6.
Let's discuss tomorrow to identify what could be causing the issue 😄

@VasuBhog  can you still repro this?

@VasuBhog  can you still repro this?

Just checked the latest insiders on Mac and got "getJupyterBookVersion" is not defined. Looks to be good on Windows, however.
Will work with @barbaravaldez to see if she can repro.


[BOTS] Market Maker Bot for Base Uniswap, Curve, Aerodrome

As the Paloma Foundation, I want to setup a market-maker bot on any of 3 exchanges for any specific ERC-20 token pool on the exchanges so that the bot will manage the price of the token every second.
Acceptance Criteria

Take input of the Pools for the token
Confirm that the token pools are for the same token
Calculate limit orders for the ERC-20 token
Create limit orders for the ERC-20 Token
Take in data and calculate new limit orders

EPIC: https://www.notion.so/volumefi/EPIC-11-1-24-Paloma-GRAIN-AI-Market-Maker-Bot-for-Base-DEXes-Uniswap-Aerodrome-Curve-12ce002bb62180f18637e8191d85a393

Closing for now.


[Build] BuildPlan: Always discover test modules through their aggrega…

…te products
Motivation:
Fixes a situation when tests were build for both host and target even though they should have been built only for the host (i.e. when one of the modules depends on a macro).
Modifications:

Adjusted BuildPlan.computeDestinations to never add test modules to successor list of a package and allow test products to always iterate their module dependencies instead.

Result:
If one of the test targets depends on a macro we'd never attempt to build others for "target".

@swift-ci please test

@swift-ci please test

@swift-ci please test Windows platform

@swift-ci please test Linux platform


fix(subscriber): only send new tasks/resources/etc over the event channel

Motivation
Currently, there are some rather bad issues that occur when the event
buffer is at capacity and events are dropped.
Completely losing data due to buffer capacity is relatively okay: if
we set a bound on how much memory the console can use, and we don't
record new things that occur when we've reached that limit, this is
correct and acceptable behavior. However, the current design can result
in incorrect data when events are lost due to the buffer being at
capacity.
This is because we currently record things like starting to poll a
task/resource, ending a poll, dropping a task/resource/async op, and
waker operations, as individual events in the buffer. This means that we
can have a situation where the creation of a task was recorded, because
there was buffer capacity at the time, but then when the task ended, the
buffer was full, so we never recorded its termination. That results in
the tasks appearing to run forever and never terminate --- see issue
#230. Similarly, if we record the beginning of a poll, but drop the
end-of-poll event because we're at capacity, this means we will
(incorrectly) record a poll that goes on forever, which is obviously
incorrect. I think this may also be the cause of the false positives
with the lost waker lint (#149), if we record a waker drop but missed a
waker clone that previously occurred.
The change in #212 fixed one category of issue that occurs due to event
buffer capacity --- when a task, resource, or async op's creation
event is dropped due to buffer capacity, we skip any subsequent events
related to that task/resource/op. However, this doesn't fix issues where
the subsequent events are the ones that are dropped.
Solution
This branch proposes a solution to this whole category of event buffer
capacity related issues. Unfortunately, this requires rewriting a lot
of console-subscriber's internals.
In the new approach, we now only send events over the channel when
creating a new task, resource, or async op. Those events now contain an
Arc holding the stats for that entity. Another clone of the Arc is
stored in the tracing_subscriber::Registry's span extensions for the
span corresponding to that entity. When the ConsoleLayer records
subsequent events for a particular entity, such as starting/ending a
poll, it looks up the span by ID, and updates the stats type stored in
its extensions. The aggregator stores its clone of the Arc in a map of
entities, just like it does currently, but no longer handles actually
updating the stats; just building wire format updates from any tracked
entities whose data was updated by the layer.
This should fix all issues where dropping something due to event buffer
capacity results in incorrect data. Once we have successfully recorded
the creation of a task, resource, or async op, any subsequent updates
to its stats are guaranteed to be reliable. If the channel is at
capacity and we fail to record a new resource/task/op, we never create a
stats extension for it, and we won't record anything for it at all.
Otherwise, it will always have correct data recorded.
When possible, the stats in the Arced stats are updated atomically. In
some cases, this isn't easily possible, and some fields of the stats
types are stored in a mutex. In particualr, this is required for storing
timestamps. I don't really love that, but these mutices should be
contented very infrequently. Stats aren't marked as having unset updates
until after the stats inside the mutices have been updated, so the
aggregator will not try to lock the mutex if the layer is currently
updating it; instead, it will simply be included in the next update once
the layer is no longer touching it. Mutices here will only be contended
when multiple threads are updating a task's stats at the same time,
which should occur very rarely...and in most cases, they still won't
have to contend a mutex, since access to most of the mutices are guarded
by an atomic variable for e.g. determining which thread actually was the
last to complete a concurrent poll. The biggest performance downside of
the mutices is probably not actually contention, but the additional heap
allocation required when using std::sync::Mutex. However, since we
have conditional parking_lot support, parking_lot can be used to avoid
requiring additional allocations.
In the future, it's probably possible to make more of this atomic by
converting timestamps into integers and storing them in atomic
variables. I haven't done this yet because both the protobuf timestamps
and std::time timestamps are larger than a single 64-bit number and it
might take a little extra work to ensure we can nicely fit them in an
AtomicUsize...but we can probably do that later.

This is currently a draft PR because there's still a decently large amount of code that's commented out. In particular, we don't currently record attribute updates, because I haven't written that part yet, and basically all of the JSON recording is broken by this change and needs to be rewritten. Also, I haven't cleaned up the code, added comments, or fixed most of the warnings yet.
But, the console's basic functionality mostly works --- you can run this branch and see tasks and their poll durations and so on. I wanted to go ahead and open a PR to get some feedback from @seanmonstar and @zaharidichev so that we can agree that the overall approach here is a good idea.  If you two (or anyone else) has a chance to take a look at this, I'd really appreciate that!

Would be cool to measure the impact of before and after, in case locking and atomic ops hurts the runtime, but mostly just so we are aware...

Yeah, it would definitely be good to come up with a benchmark setup for measuring the console instrumentation's performance cost...we haven't really done this yet, so it would be good to do regardless.
FWIW, using the MPSC channel also involves locking and some atomic ops, so :woman_shrugging:

Okay, I've marked this as "ready for review", as I believe everything should work now and it compiles without warnings. @seanmonstar and @zaharidichev, whenever you two have time to take a look, I'd appreciate a careful review, as this is a kind of large change.
I think it's possible to remove some of the mutexes; the change I discussed in #254 could remove the need for mutexes in timestamps. However, in practice, I think the mutexes should basically never be contended, as the aggregator task won't try to read from them unless the dirty bit is set, and it only gets set when we're done updating the data. The main overhead of the mutexes is probably adding a heap allocation/ptr dereference, which is fixed by enabling parking_lot...

If you run the mutex example you will see that the lock always stays in one state and after you run it in a bit the async ops associated with the Mutex resource just go to zero

Thanks, good catch! I must have missed that because I was mostly testing with the semaphore example. I'll try to figure out what's going on.

Thanks, good catch! I must have missed that because I was mostly testing with the semaphore example. I'll try to figure out what's going on.
@hawkw  I think that has something to do with the fact that you are creating resources,async ops and tasks as not dirty. I wonder whether that is correct. The way you have it now if in an instrumented app you simply create a bunch of resources like locks, they will not show up in the console until they are used in some way. Is that what we want? In any case if you flip the is_dirty flag to true for new async ops, the problem I am describing goes away. I have not yet identified the actual root cause of this though. I assume even if an async op is not created as dirty it needs to be set as dirty at some point when polled...
https://github.com/tokio-rs/console/pull/238/files#diff-096bbe8a08cf82daf24133d20a29936b0288c393537f10858ce731ca1338ba61R497
The other problem with the mutex example where the locked always stays true is a false alarm I think. It just happens so that whenever a mutex guard is dropped, we immediately acquire the lock and we skip issuing the wire update with locked set to false. Does that make sense?

@hawkw I think that has something to do with the fact that you are creating resources,async ops and tasks as not dirty. I wonder whether that is correct.

Oh, that's definitely not right. I'll fix that.

@zaharidichev

The other problem with the mutex example where the locked always stays true is a false alarm I think. It just happens so that whenever a mutex guard is dropped, we immediately acquire the lock and we skip issuing the wire update with locked set to false. Does that make sense?

I'm not totally sure about this; the locked attribute changes on master, so I don't think the current behavior on this branch is correct. I'd like to look a little closer at it...

after adding some printlns, it does appear that @zaharidichev is correct about this; it looks like it just happens that, because the mutex is re-acquired very quickly in the example, we never actually publish an update where it is in the unlocked state. If I modify the example a bit so that the tasks wait a variable amount if time before locking the mutex, and change the main task to not immediately lock the mutex in a loop, we see it in both the locked and unlocked state. I think this is probably fine.

@hawkw Yes just sat down to comment that this is exactly what I did as well to ensure that the update gets propagated. In this example it is just the case that the lock has heavily contended - there is basically always an async op awaiting on it so the moment it gets released, it is acquired again. So yes, all works as expected.


Add hu translation

Hi There,
We was missing the Hungarian version of this one, so i was made it for you!
Hoop you gonna happy with it !
Kind regards
DemirYaman

Big thanks for Gellipapa!


Need Windows Desktop RDC

For some reason, the links/references to the Windows Desktop RDC is no longer referenced.  Should it be referenced back to here or at least it's own sub-page?
https://learn.microsoft.com/en-us/azure/virtual-desktop/whats-new-client-windows

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

ID: 86f7187f-7e2e-da7f-6f46-46568be43db7
Version Independent ID: 49714e7e-ecfb-dba3-f43a-17b6ca677fa6
Content: Remote Desktop clients for Remote Desktop Services and remote PCs - Windows Server
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/remote-desktop-clients.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Hey brmcmill,
Thanks for contacting Microsoft. We value your feedback.
The table has been updated to better clarify the Windows Store version of RDC. The link you're referencing is already part of the green "Tip" block taking users to the Windows Desktop version (which has moved to the Azure repo).
Changes will be live after the next business day. We will now be closing this thread.
Thank you and have a most pleasant day.
#please-close


commit_version_bump expecting changes to "."

I'm getting this error while running commit_version_bump:

Found unexpected uncommited changes in the working directory. Expected these files to have changed: ["Inspirato.xcodeproj/project.pbxproj", "Inspirato/Info.plist", "InspiratoTests/Info.plist", "."]. But found these actual changes: ["Inspirato.xcodeproj/project.pbxproj", "Inspirato/Info.plist", "InspiratoTests/Info.plist"]. Make sure you have cleaned up the build artifacts and are only left with the changed version files at this stage in your lane, and don't touch the working directory while your lane is running. You can also use the :force option to bypass this check, and always commit a version bump regardless of the state of the working directory.

My git status is:
On branch feature/AIR-2055-configure_fastlane
Your branch is ahead of 'origin/feature/AIR-2055-configure_fastlane' by 1 commit.
  (use "git push" to publish your local commits)
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   Project.xcodeproj/project.pbxproj
	modified:   Project/Info.plist
	modified:   ProjectTests/Info.plist

no changes added to commit (use "git add" and/or "git commit -a")

I'm a bit confused how "." would be one of the expected changes? Any advice on this one?

The script looks inside your project configuration to find possible places where a version number might be stored. There are 2 places it looks: a) the project.pbxproj file and b) any Info.plist files.
I would check your project configuration to make sure that there aren't any targets which have a Info.plist File (in Xcode) value set to . Alternatively you can open the .pbxproj file with your favourite text editor and search for INFOPLIST_FILE.

Seeing the same thing:
Found unexpected uncommited changes in the working directory. Expected these files to have
changed:
MyProject.xcodeproj/project.pbxproj
MyProject/Info.plist
MyProjectKit/Info.plist
MyProjectShare/Info.plist
MyProjectTests/Info.plist
MyProjectKitTests/Info.plist.
But found these actual changes:
MyProject.xcodeproj/project.pbxproj
MyProject/Info.plist
MyProjectKit/Info.plist
MyProjectKitTests/Info.plist
MyProjectShare/Info.plist
MyProjectTests/Info.plist
.

The pbxproj and plist files are correct. If I run git diff --name-only HEAD I get:
MyProject.xcodeproj/project.pbxproj
MyProject/Info.plist
MyProjectKit/Info.plist
MyProjectKitTests/Info.plist
MyProjectShare/Info.plist
MyProjectTests/Info.plist

And git ls-files --other --exclude-standard is empty, so I don't see where . is coming from.

Wait... looking at the source, I think the . is just part of the error message. So I'm not sure where the difference is.

Hi @kevboh, I'm going through some old issues. Is this still an issue for you? Let us know!

Hey @JaviSoto—I no longer have access to the code where that was happening (changed jobs), so you should treat this as not reproing unless someone else chimes in.

Thanks for the update 👍


Use rand_s in get_random_info on Windows

Why are you doing this? I can agree that the code looks simpler, but is there another reason to change this?

It provides "proper" randomness (cryptographically secure) and works for Windows UWP targets too, in addition to the code being simpler.


Notifications for duels

Changes

[x] Require authorization (client)
[x] Receive push notification (client / sw)
[x] Send push notification to user when a new duel is available (server)
[x] Redirect the user to the duel when the notification is clicked (sw)
[x] Update tests

Request for authorization for notifications:

Notification (here on macOS ❤ ) when the app is in the background:

Notification when the app is in the foreground:

Would it be possible to change then/catch to await/async for consistency 😃


Provide actions badge svgs

Fix #23688

If we could template the SVGs in gotemplate, that would be even more flexible, for example the user could change the name string via a URL argument, if they so desire.

Also, I suggest lowercasing the status text, and likely it should just mirror the 4 commit statuses we currently have in the displayed string.

https://github.com/essentialkaos/go-badge seems like a package we could use.

If we could template the SVGs in html/template (I asssume it would be suitable), that would be even more flexible, for example the user could change the name string via a URL argument, if they so desire, but in any case, I can accept the current solution as well.

The current method could use CDN and web browser cache better to reduce possible network requests.

Caching can be achieved with proper cache headers as well even when generating those badges dynamically.
BTW, I would personally short the text to CI: success, so I guess many people would want to be able to customize the left side text at least.

Could you share the screenshots, if applicable?

Could you share the screenshots, if applicable?

There is no screenshots. Only svg files on files changed tab.

Caching can be achieved with proper cache headers as well even when generating those badges dynamically.
BTW, I would personally short the text to CI: passing, so I guess many people would want to be able to customize the left side text at least.

This is only for Gitea Actions but not any other integrated third-party CI/CD system. So use Gitea Actions will not make user confusing but CI maybe.

Checking how GitHub does it, they require the workflow name and render it into the SVG. I guess this would be good to have because because if there are multiple workflows, having all of them show "Gitea Actions" would be very confusing.
Example GitHub badges:

I think it can not easily templated with golang because the width of the SVG needs to be dynamically set based on text. So I think https://github.com/essentialkaos/go-badge might still be the best option.


Update rejected_casks.md

Closes https://github.com/caskroom/homebrew-cask/issues/28220.
I think the third point in https://github.com/caskroom/homebrew-cask/issues/28220, namely:

What maintainers feel is too much overhead for the value

is already covered by:

The cask is so difficult to maintain, it constantly breaks. Happened only once so far, with Audacity, because it uses fosshub as a distribution site.

Thanks @vitorgalvao.
Pinging @caskroom/maintainers for any other comments or suggestions before merging.


Reduce framerate drops caused by treble peaks

I think these are primarily caused by the lights associated with the treble peaks, although I think the treble sprites could be a contributing factor.

I think this has been borne out to be sprite driven.


Enforce Plane invariants via private fields & setters

Created during survey of commented code (#377).
There are a number of invariants expected mathematically of the Plane type, that cannot currently be enforced due to the fields being public.
A few issues to consider:

Should validation be performed when getting values from Godot, or should we just trust the engine with those invariants?
What about floating point errors? Should we just accept the drift or attempt to normalize after each operation? It could be useful to see what Godot does in its own source regarding this.

A test should also be added for contains_point_eps.

There are a number of invariants expected mathematically of the Plane type, that cannot currently be enforced due to the fields being public.

This is also true to a lesser extent for Basis, Transform2D and Transform:

Basis vectors should be linearly independent. Not sure if Godot enforces that, but the type is geometrically useless otherwise.
The Rust docs currently call the Transform to represent an "affine transform", but Godot docs don't. In order to be affine, certain mathematical properties need to hold. I'm not sure if that's realistically enforcible, anyway -- but maybe it's just a documentation issue and the transform doesn't need to be affine.

Should validation be performed when getting values from Godot, or should we just trust the engine with those invariants?

Considering this, I'm always asking myself: what do we lose if we validate?

if it's performance we can't afford -> use debug_assert! instead of assert!
if there are valid use cases to violate invariants -> they are not invariants, and thus don't need validation

Also, Godot has the tendency to use ugly workarounds to represent some

The Rust docs currently call the Transform to represent an "affine transform", but Godot docs don't. In order to be affine, certain mathematical properties need to hold. I'm not sure if that's realistically enforcible, anyway -- but maybe it's just a documentation issue and the transform doesn't need to be affine.

I think the presence of affine_inverse(), judging from how its docs are worded, kind of implies that Transform can indeed contain non-affine transforms (emphasis mine):

Returns the inverse of the transform, under the assumption that the transformation is composed of rotation, scaling and translation.

If so, then it should be a documentation error on our side indeed.

if there are valid use cases to violate invariants -> they are not invariants, and thus don't need validation

Not directly related, but GDScript has the tendency to use questionable values to represent "invalid" states, since it doesn't support real error handling, e.g.

This is in fact the main concern of mine regarding the input validation problem: the overall API surface is huge, and there might just be some method that use the types in an undocumented way. In other words, it might be the situation that there's something we think is an invariant but is actually not.
Also related is the problem of floating point drift, where in a series of operations small errors add up and eventually become larger than EPSILON -- in this case we can cause panics in perfectly good code, just because we decided to try to be clever.


Use multiple Entity Managers and Connections

I have multiple Entity Managers and Connections in my Symfony project and all is OK
https://symfony.com/doc/current/doctrine/multiple_entity_managers.html
Now I would like to use multiple Entity Managers in my resolvers or fields but I do not know how to do.
Thanks

If this can help you, in my services.yml
I create new em
em_other: class: Doctrine\ORM\EntityManager factory: ['@doctrine', 'getManager'] arguments: ["other"]
and  new resolver base
`resolver.other:
class: AppBundle\Resolver\AbstractResolver
abstract: true
calls:

[ init, ['@em_other']]`

and edit parent of resolver
resolver.customers: class: AppBundle\Resolver\WstiersResolver parent: 'resolver.other'


Generate builds

Looks good otherwise.

Oh, don't forget we need to update package versions in dynamicDependencies.json if they specify a specific version. I think this is just taco-remote-multiplexer's at the moment.


SCSI Error while formatting disk using Yamaha AW2816 disk utility

Info

Which version of Pi are you using: Raspberry Pi Zero W Rev 1.1
Which github revision of software: master branch v21.12.01
Which board version:68mkla 2.4a
Which hardware sampler is the RaSCSI connected to: Yamaha RS7000 and AW2816

Describe the issue
Yamaha AW2816 is a hard disk recorder with SCSI connectivity for backups. It comes with a tool to format the attached disk, however when it is executed it only reports "SCSI Error". Being a black box, it is hard to say what the actual problem is. RaSCSI logs indicate 05 24h/00h (INVALID FIELD IN CDB) being the problem. I did not manage to establish more details about this issues. I tried changing the termination on RaSCSI but it did not affect it.
On the other hand, RS7000 (audio workstation) also offers SCSI connectivity to store and load samples, patterns, etc. Without looking at RaSCSI logs, the operation seems correct. It is able to reckonise and format RaSCSI disks, and also to save and load the data that is stored there (by RS7000 itself). However, when looking at the logs, one can spot that 24/00 keeps appearing for some reason.
I have looked at #504 but the timeouts in the master seem to be correct and they seem to match Yamaha's SCSI implementation as described in the ticket.
The steps I used to reproduce this issue on AW2816 are:

create a generic 500 mb HDD
attach the disk
switch on the device
format
error is observed

The steps for RS7000:

use the same disk from the previous set of steps
switch on the device
format the disk
mount the disk
load a file from internal memory and save it to SCSI disk
load the same file form SCSI disk

I am attaching the trace files that correspond with the steps outlined above. Due to their length, I divided them as follows:

1-aw2816-format.txt - trace information when formatting is attempted on AW2816
1-rs7000-boot.txt - trace information during the boot process of RS7000
2-rs7000-format.txt - trace information during formatting (RS7000)
3-rs7000-write.txt - trace information during writing a file to SCSI disk (RS7000)
4-rs7000-load.txt - trace information during loading the saved file (RS7000)

Do let me know if I can do something else to proceed with troubleshooting, or if any other information is needed, that I might have missed.

@n4al Can you please re-test formatting with the current release 22.08.02 or the develop branch? Your first log shows that code page $01 was rejected as not being supported. With the current code this code page should definitely be returned.

I retested it with 22.9.1 35bbe97 and the original problem has been solved. The device reports that the format has been successful. I was able to confirm that this is indeed so by mounting the image locally.

@n4al That's good news! I am going to close this ticket, assuming you agree with that.


Shall "Rust web framework comparison" be linked from arewewebyet site?

"Rust web framework comparison" seems to be a maintained comparison of Rust web frameworks (server and client), which I expect to be relevant for arewewebyet visitors, so it may be beneficial to link it from the site.

This is a great repo, I think it would be useful at the Web frameworks page.


Issue with running it on the device

Hey,
I've been trying to use this framework - it is really great.
When I run my project in Simulator everything works marvelously, however when I run it on a real device an error occurs:

dyld: Library not loaded: @rpath/Macaw.framework/Macaw
Referenced from: /private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/ProjectName
Reason: no suitable image found.  Did find:
/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw: code signature invalid for '/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw'
/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw: stat() failed with errno=25
/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw: code signature invalid for '/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw'
/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw: stat() failed with errno=1
/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw: code signature invalid for '/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw'
/private/var/containers/Bundle/Application/78DB2AB3-EB91-478F-AF73-C2FC112D7D08/ProjectName.app/Frameworks/Macaw.framework/Macaw: stat() failed with errno=1

I have found on Stack Overflow and other forums that it might be linked to Embedded Content, however I tried following the steps there, but nothing really worked. Have you had any issues with running projects using Macaw on your devices as well?

Hi, @tomaszosk, no we haven't, which steps did you try?

I built a project from the tutorial by Sean Allen on YouTube, link:
https://www.youtube.com/watch?v=hMyExC9swz8
and it works exactly as in the video on the simulator, however, when I run the same project on the device, the app crashes with 'Thread 1: signal SIGABRT' and the error mentioned above is displayed in the console.
I've read on Stack Overflow that this issue might be connected with a framework being missing and I have to

Add it manually in the Project > General > Frameworks, Libraries, and Embedded Content > '+' and to add Macaw and set it to: Embed & Sign
In Project > Build Phases > Dependences > '+' and add Macaw there as well

Reference: https://stackoverflow.com/a/59421812
I followed these steps, but the the building will fail with this error:

error: /Users/(..)/DerivedData/ProjectName-cevmdzifxaeyjghjdzhiivjcefsc/Build/Products/Debug-iphoneos/Macaw.framework: No such file or directory

I tried different variations (Build Settings > Linking > Runpath Search Paths, Build System > Legacy Build System, or removing Dynamic Frameworks from the Podfile), but nothing really worked

Okay, I'll try to test this out

Which version of cocoa pods do you use? Did you try to clean derived data?

I had the same problem

I solved this problem temporarily：
change use_frameworks! to use_modular_headers! in the podfile file


Update Prerequisites that Helm needs to be installed.

Perhaps obvious to experienced users, but for me it wasn;t immediately obvious why the Chart object was causing a "resource not found" exception when doing pulumi up. After installing Helm, the error went away.
File: docs/guides/crosswalk/kubernetes/app-services.md

The Helm v3 support in Pulumi no longer has this dependency, so best solution is likely to move examples forward to use that.


add a way to track how many mbs/gbs a user has spent

This feature will not be taken into accounting for now, as it is a lot of work and for something that dosen't really isn't that useful since png files don't consume that much space


Do we need @angular-devkit/build-angular

Do we really need @angular-devkit/build-angular as a dependency. It contains protractor, which is deprecated and contains dependencies with vulnerabilities<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

@aljaznuncic Yes - this lib replaces - and internally invokes- the "extract-i18n"-builder from  @angular-devkit/build-angular. Would you have any other idea?
Furthermore any angular project depends on this dependency to build - doesn't it?


Final Capstone Project

In this final capstone project I added many codes for better look.
-Add Javascipt
-Add Css
-Add images
-Add linter check
-Inspire Behance

Hello @Hsuwaddyhtat22
This is a manual review

[ ] Please check the original design again. Everything should be the same except the page of the content.

[ ] I create a video about what you should change. If ı miss any requirement please note that this is a manual review and the next senior can mention what ı didn't mention. So please follow the requirements.

[ ] Most noticeable thing is your page is not fit the screen and the font size is big.
Here is a video

Hi,but upper the reviewers are look up with zomm - .I make up design with
100% zoom.But I think they reduce the size to 67 so I also make my page
with zoom -67%, can u try with it again .You cab see upper the reviewers
are reduce their scree size.
On Thu, Jul 13, 2023, 4:48 PM Ahmet Bozacı @.***> wrote:

Hello @Hsuwaddyhtat22 https://github.com/Hsuwaddyhtat22
This is a manual review

Please check the original design
https://www.behance.net/gallery/29845175/CC-Global-Summit-2015
again. Everything should be the same except the page of the content.
I create a video about what you should change. If ı miss any
requirement please note that this is a manual review and the next senior
can mention what ı didn't mention. So please follow the requirements.
Most noticeable thing is your page is not fit the screen and the font
size is big.
Here is a video
https://www.loom.com/share/66549a486f1e419c8011e132b594b44f?sid=0527d5f6-a89a-40f1-ba44-da32a36bf5b7
—
Reply to this email directly, view it on GitHub
https://github.com/Hsuwaddyhtat22/musical-giggle-capstone/pull/1#issuecomment-1633965140,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A46KXXENS3HYGRMBSKBBCLLXP7DNZANCNFSM6AAAAAA2CAJBIA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi,but upper the reviewers are look up with zomm - .I make up design with 100% zoom.But I think they reduce the size to 67 so I also make my page with zoom -67%, can u try with it again .You cab see upper the reviewers are reduce their scree size.
…
On Thu, Jul 13, 2023, 4:48 PM Ahmet Bozacı @.> wrote: Hello @Hsuwaddyhtat22 https://github.com/Hsuwaddyhtat22 This is a manual review - Please check the original design https://www.behance.net/gallery/29845175/CC-Global-Summit-2015 again. Everything should be the same except the page of the content. - I create a video about what you should change. If ı miss any requirement please note that this is a manual review and the next senior can mention what ı didn't mention. So please follow the requirements. - Most noticeable thing is your page is not fit the screen and the font size is big. Here is a video https://www.loom.com/share/66549a486f1e419c8011e132b594b44f?sid=0527d5f6-a89a-40f1-ba44-da32a36bf5b7 — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A46KXXENS3HYGRMBSKBBCLLXP7DNZANCNFSM6AAAAAA2CAJBIA . You are receiving this because you were mentioned.Message ID: @.>


Panics on trying to take a dispute from a non allowed user

If you try to take the dispute from an account that is not allowed, apparently mostrod received a message with action CantDo and it panics, it looks like mostrod is sending to itself a message with action CantDo.
it panics when it gets here
https://github.com/MostroP2P/mostro/blob/main/src/app.rs#L120

Hi @grunch ,
just replicated this in this scenario:

Created a Sell order --> Taksell order --> Addinvoice --> Payinvoice --> Create dispute -->  Take dispute as Admin

Loop message comes from here in admin_take_disputes.rs :
 info!("Dispute {} taken by {}", dispute_id, event.pubkey);
    // We create a Message for admin
    let message = Message::new_dispute(
        Some(dispute_id),
        None,
        Action::AdminTakeDispute,
        Some(Content::Order(new_order)),
    );
    let message = message.as_json()?;
    // Send the message
    send_dm(&event.pubkey, message.clone()).await?;

Solutions that I propose:

Remove the DM send when is admin taking dispute, just adding a check, but losing the dm message to Admin.
Creating a new Action ( example AdminTookDispute or something like that ) that is discarded in the main loop as you correctly pointed out in the tg chat.

I think second one is better, because is more aligned to mostro behaviour without special cases.
I can try to arrange a new release of mostro-core with the new Action and the fix on mostrod....
Will drop pr after a test.

implemented a quick solution in #237

why don't we discard all messages that are not on the match? something like
_ => ()

Look to pr...i did it with an info!() just to print what is arrived in case...
if we just discard probably we have always the loop because from what I saw Mostro is sending a Action::AdminTakeDispute message to himself triggering again the main fn..sends again and again...we should break this option with another action.

why don't we discard all messages that are not on the match? something like
_ => ()

Probably you're thinking at the case of CantDo that mostrod send to himself, but try the otherscenario where mostrod send to himself admintake dispute command...

What if any other user start sending to Mostro Actions that are not on the match in app.rs?

Lot of info! printed as log at the moment. If you think that could be a bad thing to flood log I agree with you with :+1:
_ => ()
as last match arm to silently discard all wrong actions message.
I was also looking if we can avoid the new action that I triggered when admin takes a dispute, but here we are sending this:
 // We create a Message for admin
    let message = Message::new_dispute(
        Some(dispute_id),
        None,
        Action::AdminAcceptDispute,
        Some(Content::Order(new_order)),
    );

So we are yet using content field to send the order related to dispute, otherwise we could use a TextMessage field to avoid loop and send a message to mostro himself.
If you have better ideas @grunch let me know, open to discussion. This could be a starting point to fix potential loop issue.

great, I think we can leave it this way for now and see how it behaves, thanks buddy
closing it via #237


Stopping monitoring v1 orchestrated and triggered topics

This PR was successfully deployed to production by CircleCI build 162 at 2017-02-13 15:02 GMT.


[MM-844]: Added server testcase for store/welcome_store.go

Summary
Added server testcase for store/welcome_store.go

Created a PR with all these changes on MM remote hence closing this one


[KARAF-5528] Stop refreshed bundles together with updated and deleted…

… ones
This time with if (!noRefresh)

retest this please

retest this please

retest this please

@jbonofre @gnodet I had problem with this issue - stopping bundles that are in toRefresh earlier - together with the ones being updated caused pax-logging-log4j2 bundle to be stopped earlier because it's optionally wired to org.fusesource.jansi which is installed with shell feature. And we loose logging output earlier.
I suggest adding jansi to etc/startup.properties (framework feature) - wdyt?


check mpa

not working in pwa-prompt example, there is no client build!
EDIT: we should check if adding a custom plugin removing the ssr check (on pwa build plugin) works (it should work: we're getting the api reference using mpa flag)

just adding the closeBundle here https://github.com/userquin/vite-pwa-vitepress-integration/blob/main/src/index.ts#L40, checking for api and mpa and call api.generateSW() should work

Mpa support in vitepress seems to be broken...


Preview Mode breaks

Describe the Bug
Same issue as #613.
Error message:

Which package is affected (leave empty if unsure)
No response
Link to the code that reproduces this issue
https://github.com/lukevella/rallly
To Reproduce
On windows 10/11, follow the development guide in https://github.com/lukevella/rallly. At yarn dev, you will hit that issue.
Expected Behavior
No error message.
What's your node version? (if relevant)
18.16.0

Same issue happens on a Kubuntu VM inside virtualbox:

Downgrading react-email in rallly/packages/emails/package.json to 1.7.15 resolves the error on Kubuntu. Haven't tried windows 10.

On Windows 10 it first creates an emails folder with the example emails in the root, but doesn't copy it inside the generated .react-email so it breaks... After manually moving the emails it inside .react-email all works well.

@zanzlender did you move the whole emails folder into .react-email or you copied each individual file?

On Windows 10 it first creates an emails folder with the example emails in the root, but doesn't copy it inside the generated .react-email so it breaks... After manually moving the emails it inside .react-email all works well.

@zanzlender did you move the whole emails folder into .react-email or you copied each individual file?

I just copied the whole folder

This should not happen the same as of<EMAIL_ADDRESS>and should now be fixed.
Check the changelog and our blog post for more info on this.


Fix status code

Status code for successfully adding a role assignment is 201 instead of 200:
https://learn.microsoft.com/en-us/rest/api/fabric/core/workspaces/add-workspace-role-assignment?tabs=HTTP#response

Thanks for your contribution. They actually changed some things on the Core-> Workspace APIs.
I´m including that right now together with some new features and will publish a new version today or tomorrow


Use existing SageMaker enpoints

Add an ability to use existing LLMs deployed on SageMaker rather than deploying new one.

Hi Sergey, really love what you're doing. How can I add e.g. the foundational 7b/70b Llama2 models to the application? And how would I go about adding a finetuned model of these versions to the tool?


ninja: error: unknown target 'clang-format'

The README says that coding style changes can be done automatically by running ninja -C build clang-format. But when I try I get ninja: error: unknown target 'clang-format'.
# meson setup build .
The Meson build system
Version: 1.0.1
Source dir: /mavlink-router
Build dir: /mavlink-router/build
Build type: native build
Project name: mavlink-router
Project version: 3
C compiler for the host machine: cc (gcc 12.2.0 "cc (Debian 12.2.0-14) 12.2.0")
C linker for the host machine: cc ld.bfd 2.40
C++ compiler for the host machine: c++ (gcc 12.2.0 "c++ (Debian 12.2.0-14) 12.2.0")
C++ linker for the host machine: c++ ld.bfd 2.40
Host machine cpu family: aarch64
Host machine cpu: aarch64
Library m found: YES
Library rt found: YES
Run-time dependency threads found: YES
Found pkg-config: /usr/bin/pkg-config (1.8.1)
Run-time dependency systemd found: YES 252
Run-time dependency GTest found: YES (building self)
Compiler for C supports arguments -Wno-inline: YES
Compiler for C supports arguments -Wundef: YES
Compiler for C supports arguments -Wformat=2: YES
Compiler for C supports arguments -Wlogical-op: YES
Compiler for C supports arguments -Wsign-compare: YES
Compiler for C supports arguments -Wformat-security: YES
Compiler for C supports arguments -Wmissing-include-dirs: YES
Compiler for C supports arguments -Wformat-nonliteral: YES
Compiler for C supports arguments -Wold-style-definition: YES
Compiler for C supports arguments -Wpointer-arith: YES
Compiler for C supports arguments -Winit-self: YES
Compiler for C supports arguments -Wdeclaration-after-statement: YES
Compiler for C supports arguments -Wfloat-equal: YES
Compiler for C supports arguments -Wmissing-prototypes: YES
Compiler for C supports arguments -Wstrict-prototypes: YES
Compiler for C supports arguments -Wredundant-decls: YES
Compiler for C supports arguments -Wmissing-declarations: YES
Compiler for C supports arguments -Wmissing-noreturn: YES
Compiler for C supports arguments -Wshadow: YES
Compiler for C supports arguments -Wendif-labels: YES
Compiler for C supports arguments -Wstrict-aliasing=3: YES
Compiler for C supports arguments -Wwrite-strings: YES
Compiler for C supports arguments -Wno-long-long: YES
Compiler for C supports arguments -Wno-overlength-strings: YES
Compiler for C supports arguments -Wno-unused-parameter: YES
Compiler for C supports arguments -Wno-missing-field-initializers: YES
Compiler for C supports arguments -Wno-unused-result: YES
Compiler for C supports arguments -Wnested-externs: YES
Compiler for C supports arguments -Wchar-subscripts: YES
Compiler for C supports arguments -Wtype-limits: YES
Compiler for C supports arguments -Wuninitialized: YES
Compiler for C supports arguments -Waddress-of-packed-member: YES
Compiler for C supports arguments -Wno-inline: YES (cached)
Compiler for C supports arguments -Wundef: YES (cached)
Compiler for C supports arguments -Wformat=2: YES (cached)
Compiler for C supports arguments -Wlogical-op: YES (cached)
Compiler for C supports arguments -Wsign-compare: YES (cached)
Compiler for C supports arguments -Wformat-security: YES (cached)
Compiler for C supports arguments -Wmissing-include-dirs: YES (cached)
Compiler for C supports arguments -Wformat-nonliteral: YES (cached)
Compiler for C supports arguments -Wpointer-arith: YES (cached)
Compiler for C supports arguments -Winit-self: YES (cached)
Compiler for C supports arguments -Wfloat-equal: YES (cached)
Compiler for C supports arguments -Wredundant-decls: YES (cached)
Compiler for C supports arguments -Wmissing-declarations: YES (cached)
Compiler for C supports arguments -Wmissing-noreturn: YES (cached)
Compiler for C supports arguments -Wshadow: YES (cached)
Compiler for C supports arguments -Wendif-labels: YES (cached)
Compiler for C supports arguments -Wstrict-aliasing=3: YES (cached)
Compiler for C supports arguments -Wwrite-strings: YES (cached)
Compiler for C supports arguments -Wno-long-long: YES (cached)
Compiler for C supports arguments -Wno-overlength-strings: YES (cached)
Compiler for C supports arguments -Wno-unused-parameter: YES (cached)
Compiler for C supports arguments -Wno-missing-field-initializers: YES (cached)
Compiler for C supports arguments -Wno-unused-result: YES (cached)
Compiler for C supports arguments -Wchar-subscripts: YES (cached)
Compiler for C supports arguments -Wtype-limits: YES (cached)
Compiler for C supports arguments -Wuninitialized: YES (cached)
Compiler for C++ supports arguments -Waddress-of-packed-member: YES
Header "aio.h" has symbol "aio_init" : YES
Configuring config.h using configuration
Configuring mavlink-router.service using configuration
Program run-clang-tidy found: YES (/usr/bin/run-clang-tidy)
Build targets in project: 10

Found ninja-1.11.1 at /usr/bin/ninja

root@45499d0b1668:/mavlink-router# ninja -C build
... (omitted lots of googletest warnings)...

# ninja -C build clang-format
ninja: Entering directory `build'
ninja: error: unknown target 'clang-format'

Because my host machine is a Mac, I'm doing the build in a Docker container produced by this Dockerfile:
FROM debian:stable

RUN apt-get update
RUN apt-get install -y git build-essential autoconf automake meson ninja-build pkg-config gcc g++ systemd
RUN apt-get install -y googletest clang-tidy

WORKDIR mavlink-router

I then create a Docker container with:
$ sudo docker run -ti --mount type=bind,source="$(pwd)",target=/mavlink-router --name dev-mavlink-router dev-mavlink-router

you probably need to install clang-format in your containers? https://packages.debian.org/search?keywords=clang-format

Thanks, yes, installing clang-format and rerunning meson setup build . did the trick.
I had originally installed clang-tidy but not clang-format, since that's what meson setup build . seems to want:
Program run-clang-tidy found: YES (/usr/bin/run-clang-tidy)

Maybe the README should be updated with the clang-format dependency? And maybe meson.build should check for clang-format too?

can you send a patch with those changes?


maximum recursion depth in NeVA

🐛 Bug
[rank0]:   File "/home/tfogal/dev/thunder/thunder/core/interpreter.py", line 6061, in _impl
[rank0]:     return fn.__func__(fn.__self__, *args, **kwargs)
[rank0]:   File "/home/tfogal/dev/pytorch/torch/nn/modules/module.py", line 1575, in _call_impl
[rank0]:     return forward_call(*args, **kwargs)
[rank0]:   File "/home/tfogal/dev/thunder/thunder/core/interpreter.py", line 6061, in _impl
[rank0]:     return fn.__func__(fn.__self__, *args, **kwargs)
[rank0]:   File "/home/tfogal/dev/nemo/nemo/collections/multimodal/models/multimodal_llm/neva/neva_model.py", line 155, in forward
[rank0]:     return self.replace_media_embeddings(input_ids, words_embeddings, media)
[rank0]:   File "/home/tfogal/dev/nemo/nemo/collections/multimodal/models/multimodal_llm/neva/neva_model.py", line 195, in replace_media_embeddings
[rank0]:     media_features = self.encode_vision_x(media)  # b T F S(eq) H(idden)
[rank0]:   File "/home/tfogal/dev/nemo/nemo/collections/multimodal/models/multimodal_llm/neva/neva_model.py", line 176, in encode_vision_x
[rank0]:     vision_x = self.vision_encoder(vision_x, output_hidden_states=True)
[rank0]: thunder.core.interpreter.InterpreterError: Encountered exception RecursionError: maximum recursion depth exceeded in comparison

Full log of the run which includes the full traceback.
To Reproduce
First apply Kshiteej's patch from #601.
Then install the tfogal/thunder-nemo branch of https://github.com/tfogal/NeMo.  Then run:
HYDRA_FULL_ERROR=1 \
THUNDER_ANNOTATE_TRACES=1 \
NEMO_THUNDER_NEVA=1 \
python3 ./examples/multimodal/multimodal_llm/neva/neva_pretrain.py trainer.precision=16 model.megatron_amp_O2=False trainer.num_nodes=1 trainer.devices=1 trainer.val_check_interval=10 trainer.limit_val_batches=5 trainer.log_every_n_steps=1 ++exp_manager.max_time_per_run=00:00:03:00 trainer.max_steps=20 model.micro_batch_size=2 model.global_batch_size=4 model.tensor_model_parallel_size=1 model.pipeline_model_parallel_size=1 exp_manager.create_checkpoint_callback=False model.data.data_path=./data/multimodal/tiny-neva/dummy.json model.data.image_folder=./data/multimodal/tiny-neva/images model.tokenizer.library=sentencepiece model.tokenizer.model=./data/multimodal/tiny-neva/tokenizer_add_special.model model.num_layers=2 model.hidden_size=5120 model.ffn_hidden_size=13824 model.num_attention_heads=40 model.normalization=rmsnorm model.data.num_workers=0 model.data.conv_template=llama_2 model.mm_cfg.vision_encoder.from_pretrained=openai/clip-vit-large-patch14 model.mm_cfg.llm.from_pretrained=null model.use_flash_attention=false exp_manager.exp_dir=./foo-neva-train

Note you'll need a few files for the referenced ./data directory; you can ping me privately for now, and I'll work on pulling them out from behind the curtain.
Environment
$ nvidia-smi | grep -i cuda
| NVIDIA-SMI 555.42.02              Driver Version: 555.42.02      CUDA Version: 12.5     |
$ python3 -m pip freeze | egrep -i "(nvfuser)|(lightning)|(thunder)|(nemo)"
-e git+ssh://git@github.com/tfogal/lightning.git@8df5db52ead1804f9021bb07caa2d4a7a6ab03a1#egg=lightning
lightning-cloud==0.5.69
-e git+ssh://git@github.com/Lightning-AI/lightning-thunder.git@8c5905fd1a93145e690791a7c7a3c3e10b16b32b#egg=lightning_thunder
lightning-utilities==0.11.2
-e git+ssh://git@github.com/NVIDIA/NeMo.git@c86449e1a93049d2283ebcea8ee4546f2ea241de#egg=nemo_toolkit
# Editable Git install with no remote (nvfuser==0.2.6+git9c5f006)
-e /opt/pytorch/nvfuser
pytorch-lightning==2.3.0

cc @tfogal

Here is a smaller repro, (also this doesn't require the patch from #601):
import torch
import thunder
from transformers.utils.generic import ModelOutput

def fn(x):
    mo = ModelOutput(foo=x)
    return mo["foo"]

# Sanity - eager works
print(fn(torch.randn(3,)))

# Recursion error
thunder.jit(fn)(torch.randn(3,))

Since ModelOutput inherits OrderedDict https://github.com/huggingface/transformers/blob/1c68f2cafb4ca54562f74b66d1085b68dd6682f5/src/transformers/utils/generic.py#L310, I tried OrderedDict instead on a whim, and the thunder.jit worked. just fyi

Thank you @kshitij12345  for the minimal repro and @crcrpar  for the additional info.
The thing we need is to support the wrapper tracking here so we know the mo["foo"] is the same object as the x we put in there. This is a bit tricky due to the multi-inheritance, but so we should check

is the ModelOutput.__post_init__ jitted through or do we have an "opaqueness gap"?
are the .wrapped_items of mo's WrappedValue populated from the postinit function's populating the OrderedDict self?
does the iteration of .items()  (which comes from OrderedDict and should be lookasided) in .__getitem__ give us the same wrapped values back?

If anyone wants to take a stab, don't hesitate to chatting me up. If not I will try to look and write up what I'm doing.
(Additional the infinite recursion is not great, likely somethign funny.)

Thinking about this more: Maybe, just maybe, all this works right and the inf recursion is a problem with the MRO in the lookaside in the __getitem__. We do implement things like iterating over items through __getitem__ and should be careful to call OrderedDict.__getitem__(self,...) from the ordered iteration rather than self.__getitem__.

I have a fix, based on Kshiteej's repro and Masaki's additional analysis. Awesome work, @kshitij12345 and @crcrpar


remove auraria from nbhd level demographics

remove auraria from nbhd level demographics for student demographics - only has 3 students.
auraria row in the dataframe should have NA's across the board.

joe fixed it - all NAs now


What about cross-origin access?

Since my React code is hosted on a static hosting domain, and my PHP is hosted on a non-static hosting domain, I’m running into the CORS issue.
Fetch API cannot load https://myapp.000webhostapp.com/charge.php. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

I was able to specify header("Access-Control-Allow-Origin: *") (temporarily—I know it’s not recommended) inside the https://(myappname).000webhostapp.com/charge.php file to allow for cross-origin resource sharing (CORS). However, stripeToken on the PHP side still doesn’t seem to be recognizing the token.
This is what I specified inside the React component:
fetch('https://(myappname).000webhostapp.com/charge.php', {
	method: 'POST',
	mode: 'cors',
	body: {
		stripeToken: token.id,
		stripeEmail: this.props.data.user.email
	}
})

Here’s my PHP code:
<?php
	header("Access-Control-Allow-Origin: *");
	require_once('../config.php');
	$token	= $_POST['stripeToken'];
	$email	= $_POST['stripeEmail'];
	$customer = \Stripe\Customer::create(array(
			'email' => $email,
			'source'	=> $token
	));
	$charge = \Stripe\Charge::create(array(
			'customer' => $customer->id,
			'amount'	 => 999,
			'currency' => 'usd',
			'description' => 'Life Purpose App',
			'source' => $token
	));
	echo '
Successfully charged $9.99.
';
?>

And, finally, the console.log:

... and if I try to .json() the response, I get an error: SyntaxError: Unexpected token < in JSON at position 0

This is an issue with your server code.

Thanks, @azmenak. In case of interest, I was able to figure out another way how to use react-stripe-checkout<EMAIL_ADDRESS>


Uninstall-AzureRm not found even though AzureRm is installed.

Clean windows 10 install, AzureRM was installed and I need to remove it to get the new Az module, but the pages says to use Uninstall-AzureRm but I get this error:
Uninstall-AzModule : The term 'Uninstall-AzModule' is not recognized as the name of a cmdlet

I had to use this instead from an admin prompt:
Uninstall-Module AzureRM

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

ID: 873500f9-ae68-2bdd-9956-01a5904c5a06
Version Independent ID: b3ce94fa-6e76-4ffe-e70e-07f7a321fcd4
Content: Uninstall Azure PowerShell
Content Source: docs-conceptual/azps-4.6.1/uninstall-az-ps.md
GitHub Login: @mikefrobbins
Microsoft Alias: mirobb

@lovettchris Thank you for the feedback!  We are investigating this and will get back to you shortly.

@lovettchris Thank you for bringing this to our attention!  I wanted to confirm that I'm seeing the same thing you are.  I'm following up internally to make sure that I've got the most up to date instructions for uninstalling AzureRm as this is listed in a couple locations and Uninstall-Module leaves a number of AzureRm modules on the machine.

Yes, looks like you have a repro, thanks for investigating.
Sent from Outlookhttp://aka.ms/weboutlook

From: kobulloc-MSFT<EMAIL_ADDRESS>Sent: Friday, September 11, 2020 12:21 AM
To: MicrosoftDocs/azure-docs-powershell<EMAIL_ADDRESS>Cc: Chris Lovett<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs-powershell] Uninstall-AzureRm not found even though AzureRm is installed. (#1557)
@lovettchrishttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Flovettchris&data=02|01|clovett%40microsoft.com|2049cee17e9d4ca0561f08d856235145|72f988bf86f141af91ab2d7cd011db47|1|0|637354056966030732&sdata=vpbeM4ic3UXVMY7xCCRv6z2zQOlNdvQwF6EO363DCAY%3D&reserved=0 Thank you for bringing this to our attention! I wanted to confirm that I'm seeing the same thing you are. I'm following up internally to make sure that I've got the most up to date instructions for uninstalling AzureRm as this is listed in a couple locations and Uninstall-Module leaves a number of AzureRm modules on the machine.
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F37556655%2F92881858-3665c400-f3c4-11ea-8b1f-67c3ca4e7985.png&data=02|01|clovett%40microsoft.com|2049cee17e9d4ca0561f08d856235145|72f988bf86f141af91ab2d7cd011db47|1|0|637354056966040692&sdata=ABnfZOk%2BOkQVvKdFBq9us0xegYWpuH9RK7fQq7wtAoo%3D&reserved=0
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F37556655%2F92881896-4382b300-f3c4-11ea-9834-e084f6ac8869.png&data=02|01|clovett%40microsoft.com|2049cee17e9d4ca0561f08d856235145|72f988bf86f141af91ab2d7cd011db47|1|0|637354056966040692&sdata=KIKdzwX7eZBgWnknUVbQjvZH5naIg6%2BtQf2JHERHoL8%3D&reserved=0
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F37556655%2F92881930-4aa9c100-f3c4-11ea-833c-ad3b91a507d8.png&data=02|01|clovett%40microsoft.com|2049cee17e9d4ca0561f08d856235145|72f988bf86f141af91ab2d7cd011db47|1|0|637354056966050643&sdata=VD%2BxWuO5gBDa6BD6zldOa0Gvm2wpVRvHCUWFXyYHjes%3D&reserved=0
[image]https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F37556655%2F92882092-7167f780-f3c4-11ea-9307-458e035a61c1.png&data=02|01|clovett%40microsoft.com|2049cee17e9d4ca0561f08d856235145|72f988bf86f141af91ab2d7cd011db47|1|0|637354056966050643&sdata=jt1ubCNIj5OKlxh7vjEjvoofSrs1bh41jA2wK8%2FkS6A%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs-powershell%2Fissues%2F1557%23issuecomment-690923719&data=02|01|clovett%40microsoft.com|2049cee17e9d4ca0561f08d856235145|72f988bf86f141af91ab2d7cd011db47|1|0|637354056966060599&sdata=mxH41JolLO%2BWOx0YqqIonbiGKZzP29e1dwKniKatY%2B8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEOXFKXAY6FZBVHASHH7US3SFHFX7ANCNFSM4RAJ6CSQ&data=02|01|clovett%40microsoft.com|2049cee17e9d4ca0561f08d856235145|72f988bf86f141af91ab2d7cd011db47|1|0|637354056966060599&sdata=F4IYc81LAunMHQTyEIQWkNu%2Bz89RkdCfwI6gGU5xiSI%3D&reserved=0.

@lovettchris Please see the Uninstall the AzureRM module section of the referenced doc.
Simply install the Az PowerShell module as referenced in the doc. The easiest way to accomplish this is to install it from PowerShell using the PowerShell gallery: Install-Module -Name Az.
The Az PowerShell module adds a cmdlet named Uninstall-AzureRM which can be used to remove the AzureRM PowerShell module.
The AzureRM PowerShell module is a wrapper for numerous other modules which is why Uninstall-Module -Name AzureRM doesn't perform a full uninstall. It only uninstalls the wrapper module.
Depending on what state your system is currently in, you may have to use this command to cleanup the AzureRM modules: Get-InstalledModule -Name AzureRM, AzureRM.* | Uninstall-Module

@lovettchris I missed the Az PowerShell module requirement as well.  In the interest of documenting this for anyone else who may run into this issue, this was the shortest path I found to a complete AzureRm install and uninstall (run on Windows 10).
Install AzureRm module
Install-Module -Name AzureRM -AllowClobber
Check to see that AzureRm is installed
Get-InstalledModule
Install Az PowerShell module
Install-Module -Name PowerShellGet -Force
Install-Module -Name Az -AllowClobber -Scope CurrentUser   (This avoids the "Warning: Az module not installed.  Having both the AzureRM and Az Modules installed at the same time is not supported." you encounter from running the script in the documentation.)
Uninstall AzureRm module
Set-ExecutionPolicy RemoteSigned   (This avoids the error: "The 'Uninstall-AzureRm' command was found in the module 'Az.Accounts', but the module could not be loaded.")
Uninstall-AzureRm
Confirm that all AzureRm modules have been uninstalled
Get-InstalledModule
I'll add screenshots for additional clarity.

@lovettchris We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


added Default folder

DEFAULT_PARENT_DIR makes it usefull to specify add a location where the Configuration Files go, and defaulted to ${CMAKE_BINARY_DIR}/thirdparty
all stuff goes into there by default which is cleaner

Thanks for the suggested changes. I understand the motivation behind this pull request, but there are some wider aspects to consider too:

There are already people using this project, so changing the default download location may break their builds and make you a bit unpopular. ;) I'd prefer to keep the default behaviour the same unless there was a good reason to change it.
While one main use case revolves around the external project being a third party project, there's also a common use case for using this for internal projects too. For example, it can be used to bring in other git repositories without having to use git submodules.
You can already control the source and binary directories with the SOURCE_DIR and BINARY_DIR options. You can probably achieve the flexibility you want by explicitly handling a DOWNLOAD_DIR option as well (the name is set by ExternalProject just like SOURCE_DIR and BINARY_DIR) which controls just the directory into which the source is downloaded. Have a look at the ExternalProject documentation to see what these options all map into.

Would you consider modifying your patch to use DOWNLOAD_DIR instead and then just treat this in the same way as SOURCE_DIR and BINARY_DIR? Calling code would still need to set all three, but I think this helps preserve the more direct mapping to ExternalProject, which will make future maintainability easier and more robust.

Actually, I can see it may be a bit more complicated than just adding the DOWNLOAD_DIR option, but explore the general intent and see what you come up with.

As part of fixing the issue identified in #4, I've incorporated a modified version of your code which is a little more consistent with existing ExternalProject keywords. Hopefully it also meets your needs. I'll close this pull request, but if you encounter further related issues, open a bug report or comment on #4. Thanks.


Update hello-eks-anywhere to match upstream configs

Updates for https://github.com/aws-containers/hello-eks-anywhere/pull/3

/lgtm


Debug and Test

the current codes are not free from bugs.
We need to find a way to test them separately.

The whole verify process is running ok see #42


Arithmetic for Number (and JsonValue)?

It shouldn't be too difficult to implement all the arithmetic std::ops for Number, multiplication is already done but commented. Addition and subtraction needs to be able to level the exponent (much the same way PartialEq does it), which should also be abstracted somewhere.
Interesting question is whether or not the results should be normalized (e.g.: if mantissa 150 with exponent -2 should be lowered to mantissa 15 with exponent -1) ?

Required for #66


You need to provide at least one example for "Working with Option"

You need to provide at least one example for "Working with Option"
https://github.com/polkadot-js/docs/edit/master/docs/api/start/types.basics.md

Not sure what this means since the option has descriptions of the specific methods - however leaving it open, there may be a taker for a PR to clarify.

Closing, since we have https://polkadot.js.org/docs/api/start/types.basics#working-with-optiontype


Fix slice op issues #13760 and #14105

Description
This PR fixes nd.slice for erroneous output in the following cases:
(1) begin=end ( fixes #13760 )
(2) end=-1, step<0 ( fixes #14105 )
Added unit test for both cases.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

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

Changes

[x] end=-1 with negative step is a valid input
[x] begin=end throws invalid input error

This PR addresses the comments posted on an earlier PR #14107 (closed).

Please review @reminisce @szha @anirudh2290

@mxnet-label-bot add [Operator, pr-awaiting-review]

There are so many if/else/elseif branches in this piece of code. Could you please find ways to simplify the logic? if not, can you add comment to the end of each branch such as // end of if (s < -1) to make the code more readable.

There are so many if/else/elseif branches in this piece of code. I know it fixes the two cases you mentioned in the description. However, instead of patching, could you please find ways to simplify the logic to fix more general cases? If not, can you add comment to the end of each branch such as // end of if (s < -1) to make the code more readable.

@apeforest Simplified the logic and refactored the code. Also added comments. Please review.


Describe Embulk-announce mailing list in embulk.org and README.md

@muga Can you have a look?

@muga Thanks! Before this, we need to merge #811 to make this change effective. Can you have a look at that?

Let me check soon. Sorry to be late.

Thanks. #811 is merged. Merging this.


Load Balancer port configuration is not applied (update: at all!)

After a successful deployment, running
juju config kubeapi-load-balancer port=8443
does not change anything to /etc/nginx/sites-enabled/apilb. Moreover, it forces / refreshes the old port very often, so it is not possible to manually change it without stoping the Juju agent, which is really not cool.
This conflicts, for example, with the deployment of ingress in environments where masters and workers are colocated, which is a huge (negative) impact for dense environments.

Interestingly the new port is reported by juju status
Model     Controller  Cloud/Region  Version  SLA
k8s-ceph  k8s         maas          2.2.1    unsupported

App                    Version  Status   Scale  Charm                  Store       Rev  OS      Notes
ceph-mon               10.2.7   active       3  ceph-mon               jujucharms   10  ubuntu
ceph-osd               10.2.7   active       3  ceph-osd               jujucharms  244  ubuntu
easyrsa                3.0.1    active       1  easyrsa                jujucharms   12  ubuntu
etcd                   3.1.8    active       3  etcd                   jujucharms   40  ubuntu
flannel                0.7.0    active       9  flannel                jujucharms   20  ubuntu
kubeapi-load-balancer  1.10.3   active       3  kubeapi-load-balancer  jujucharms   16  ubuntu  exposed
kubernetes-master      1.7.0    waiting      3  kubernetes-master      jujucharms   35  ubuntu
kubernetes-worker      1.7.0    error        6  kubernetes-worker      jujucharms   40  ubuntu  exposed

Unit                      Workload  Agent  Machine  Public address  Ports             Message
ceph-mon/0*               active    idle   3        <IP_ADDRESS>                        Unit is ready and clustered
ceph-mon/1                active    idle   4        <IP_ADDRESS>                        Unit is ready and clustered
ceph-mon/2                active    idle   5        <IP_ADDRESS>                        Unit is ready and clustered
ceph-osd/0                active    idle   0        <IP_ADDRESS>                        Unit is ready (1 OSD)
ceph-osd/1                active    idle   1        <IP_ADDRESS>                        Unit is ready (1 OSD)
ceph-osd/2*               active    idle   2        <IP_ADDRESS>                        Unit is ready (1 OSD)
easyrsa/0*                active    idle   3        <IP_ADDRESS>                        Certificate Authority connected.
etcd/0*                   active    idle   3        <IP_ADDRESS>      2379/tcp          Healthy with 3 known peers
etcd/1                    active    idle   4        <IP_ADDRESS>      2379/tcp          Healthy with 3 known peers
etcd/2                    active    idle   5        <IP_ADDRESS>      2379/tcp          Healthy with 3 known peers
kubeapi-load-balancer/0*  active    idle   3        <IP_ADDRESS>      443/tcp,8443/tcp  Loadbalancer ready.
kubeapi-load-balancer/1   active    idle   4        <IP_ADDRESS>      443/tcp,8443/tcp  Loadbalancer ready.
kubeapi-load-balancer/2   active    idle   5        <IP_ADDRESS>      443/tcp,8443/tcp  Loadbalancer ready.
...

So in fact, the configuration of nginx is not rendered at all. Even if the port is changed at deployment, it renders:
upstream target_service {
  server <IP_ADDRESS>:6443;
  server <IP_ADDRESS>:6443;
  server <IP_ADDRESS>:6443;

}

server {
    listen 443 ssl http2;
    server_name _;

    access_log /var/log/nginx.access.log;
    error_log /var/log/nginx.error.log;

    ssl on;
    ssl_session_cache builtin:1000 shared:SSL:10m;
    ssl_certificate /srv/kubernetes/server.crt;
    ssl_certificate_key /srv/kubernetes/server.key;
    ssl_ciphers HIGH:!aNULL:!eNULL:!EXPORT:!CAMELLIA:!DES:!MD5:!PSK:!RC4;
    ssl_prefer_server_ciphers on;

    location / {
      proxy_buffering         off;
      proxy_set_header        Host $host;
      proxy_set_header        X-Real-IP $remote_addr;
      proxy_set_header        X-Forwarded-For $proxy_add_x_forwarded_for;
      proxy_set_header        X-Forwarded-Proto $scheme;
      proxy_set_header        X-Forwarded-Proto-Version $http2;
      proxy_set_header        Upgrade $http_upgrade;
      proxy_set_header        Connection $http_connection;
      proxy_set_header        X-Stream-Protocol-Version $http_x_stream_protocol_version;

      add_header              X-Stream-Protocol-Version $upstream_http_x_stream_protocol_version;

      proxy_pass              https://target_service;
      proxy_read_timeout      90;
    }
}

So now, this is actually not surprising if you look at the template for the config: https://api.jujucharms.com/charmstore/v5/~containers/kubeapi-load-balancer-16/archive/templates/apilb.conf
This has a hard-coded 443 instead of referencing {{ port }} or something like that.

Fixed by https://github.com/kubernetes/kubernetes/pull/48958


WIP: Add weather API

Weather
This PR adds a weather module to Adventure, providing an API for controlling the weather. The current implementation is more of a proof of concept then a final design. I'm not even 100% sure if such a thing fits in Adventure like this, or if it could be its own repo or something completely different. I'm most interested in hearing what everyone thinks!
Holder
Similar to Audience the weather module includes the concept of a WeatherHolder, that being something that can have weather applied and checked. The current weather can be obtained from a holder using WeatherHolder#weather() and can be set using WeatherHolder#weather(Weather).
Additionally, the concept of a ChildWeatherHolder exists. This represents an object that either has its own weather, or has the weather of it's parent. For example, a player could have their own weather overriding that of the world's weather.
Weather
The Weather object itself is constructed using a type and a length. The type is one of rain, clear or thunder, mimicking the /weather command in vanilla Minecraft. The length object either holds an exact duration or is marked as random, mimicking the natural flow of weather in whatever platform you are using.
Examples
Here's some examples of how the system would work.
// clear the weather for a random amount of time
world.weather(Weather.clear());

// nope let's make it rain for exactly 10 minutes
world.weather(Weather.rain(Length.of(Duration.ofMinutes(5))));

// assuming player is a child of world, this would be true
var isTrue = player.weather().type() == Type.RAIN;

// however, we can override the weather for the child and make it permanent
player.weather(Weather.clear(Length.forever()));

// now the child has it's own weather, this will be false
var isFalse = player.weather.type() == Type.RAIN;

I wonder if this is something that might be better suited in another repo, perhaps adventure-extra or something with a different name entirely to handle weather alongside other aspects of Minecraft that could be abstracted into an API like this? Each part would be its own module so that they could be implemented part by part if desired.

Appreciate the rationale. Deciding what's in and out of scope is one of the most challenging parts of maintaining a project. I think though we have some consensus that weather, even in an "extra" project, is out of scope.