Fix (#3395) sqlx::test macro in 0.8

Does your PR solve an issue?
This fixes a regression in the sqlx::test macro in version 0.8.
fixes #3395.

@joeydewaal do you mind adding a regression test so we can catch if this happens in the future?

Not sure what the best way is to add a test for a proc macro, but this should work!

I should have mentioned there's already existing test files, one for each supported database:

https://github.com/launchbadge/sqlx/blob/main/tests/mysql/test-attr.rs
https://github.com/launchbadge/sqlx/blob/main/tests/postgres/test-attr.rs
https://github.com/launchbadge/sqlx/blob/main/tests/sqlite/test-attr.rs

It looks like there was a test for mysql that should've caught this. I added a test for postgres and sqlite.

@joeydewaal run cargo fmt?

Yes of course!


Add Nuget Badges

Pull Request Test Coverage Report for Build 40420

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

Totals

Change from base Build 40418:
0.0%

Covered Lines:
4365

Relevant Lines:
8602

💛  - Coveralls


CustomizeProblemDetails section's code example is incorrect

Description
AddProblemDetails needs to be called before builder.Build(), but the code example shows it being called after.
Page URL
https://learn.microsoft.com/en-us/aspnet/core/fundamentals/error-handling?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/error-handling.md
Document ID
38515dfb-91a5-b395-db9d-084bbaf095c8
Article author
@tdykstra

@IGx89 Thanks for reporting that, I'm fixing it.


[enhancement]: Option or knob to control checkout behavior when same repository is specified twice

Describe your feature request here
We have a script that we run against each repo in a microservice system on a nightly basis. Each repo is checked out in to the default s/ dir, then a script iterates over each folder in s/ to do its work. The pipeline takes this form:
steps:
- checkout: git://my-project/repo-1
- checkout: git://my-project/repo-2
# more checkout steps omitted
- bash: |
    cd $(Build.SourcesDirectory)
    for D in $(find . -mindepth 1 -maxdepth 1 -type d -exec realpath {} \;); do
      cd $D
      # do stuff in directory D
    done

I am working on adding additional branches beyond the default to this. I tried this:
steps:
- checkout: git://my-project/repo-1
- checkout: git://my-project/repo-1@other-branch
  path: s/repo-1-other
- checkout: git://my-project/repo-2
- checkout: git://my-project/repo-2@other-branch
  path: s/repo-2-other
# etc

However, in this case checkout will move the already-checked-out repository to the new path and then fetch the specified branch. So in the above example, the first checkout step for repo-1 becomes superfluous because that directory doesn't exist anymore after the second one. This isn't what I want. I want one folder in s/ per checkout step, even if they are from the same repo. This way the existing script can continue to work on a folder-by-folder basis without modifications.
If a knob or setting already exists for this, please let me know what it is, otherwise I am requesting one. Thanks for your consideration.

Hi @lettucemode, Thanks for the idea! As far as I know, we don't have such knobs to control the behaviour. We'll try to implement it in the future, but now we have more prioritized items.

So after some experimentation and digging through the source, it turns out the behavior I want is already present. The two different refs of the repository need to have different aliases (the names that they are referred to in the rest of the YAML) and different paths specified. If a repository resource is not used and you use the shorthand instead, the alias includes the @ref at the end.
Here's the verbose way to do it:
resources:
  repositories:
  - repository: repo-1-main
    type: git
    name: my-project/repo-1
    ref: main
  - repository: repo-1-other
    type: git
    name: my-project/repo-1
    ref: other
  - repository: repo-2-main
    type: git
    name: my-project/repo-2
    ref: main
  - repository: repo-2-other
    type: git
    name: my-project/repo-2
    ref: other

steps:
- checkout: repo-1-main
  path: s/repo-1-main
- checkout: repo-1-other
  path: s/repo-1-other
- checkout: repo-2-main
  path: s/repo-2-main
- checkout: repo-2-other
  path: s/repo-2-other

Or, with the shorthand:
steps:
- checkout: git://my-project/repo-1@main
  path: s/repo-1-main
- checkout: git://my-project/repo-1@other
  path: s/repo-1-other
- checkout: git://my-project/repo-2@main
  path: s/repo-2-main
- checkout: git://my-project/repo-2@other
  path: s/repo-2-other

It's a little odd that omitting the @ref from the shorthand (as in my first post above) causes the overwriting behavior - I expected it could infer the different branch name, but I guess it's fine. My solution isn't that much more verbose to warrant a change, imo anyway.


Ensure we're testing migration between different libvirt versions

What you expected to happen:
In the past (OSP, oVirt), migrations between different versions of libvirt infra caused issues, requiring some libvirt XML changes which allowed VM's to migrate from older to newer but not going back.
Once we support migrations, we should have  tests in place
Some examples of what is supposed to work/not:

RHEL7

This is a good point, I do believe we do not have a strategy for doing upgrades when the VMs are still running. Thanks @iranzo.


Quote Command Arguments with Whitespace

Fixing #69

Two lines of .idea/runConfigurations/MockedTestCases.xml are modified because I am using IntelliJ IDEA and it requires me changing it to build.
I modified verifyExpectations() because I didn't get to manage failing any of my test even though I tried to make it failed.

Thanks.

👍  for also fixing the IntelliJ settings
👍  for getting red tests first, in the process discovering a problem in the fake-class


Update splits parser from Urn to LAST

LAST is an up to date fork based on Urn. The splits format is basically the same, but it has new additional fields: https://github.com/wins1ey/LAST/blob/main/src/last.c

Apparently it's called LibreSplit now.


Ignore failed count when determining whether Job succeeded

The Kubernetes job controller will only set the completion time on a Job
when the job's succeeded count is >= the Job's .spec.completions field
if the field is set, or 1 otherwise.  The controller will retry the job
unless the retries have exceeded .spec.backoffLimit.
Therefore a Job can have a failed count of > 0 and a succeeded count of > 0
while still counting as successful, contrary to the previous
implementation of the object_stats check.
This commit removes the condition of requiring a failed count == 0 to
treat a Job execution as successful, as the presence of a completion
timestamp in conjunction with no active jobs and > 0 completed jobs is
sufficient to determine that a Job completed successfully, see the Job
controller implementation [1].
[1] https://github.com/kubernetes/kubernetes/blob/8211cabfb2bf3b2b531b13589843130cb47df1b1/pkg/controller/job/job_controller.go#L518-L570

Technically, the check could simply be the presence of the completion timestamp, as that field is only set when the Job controller determines that the Job has completed successfully, cf. the linked code.


with Ionic when executing a callback that interacts with map from a child page results in blank map?

Trying to track down this problem.  I'm fairly new to angular and ionic so could behalf the problem but I'm using the nav control to push() (display) a new page passing a callback function from the current page through the push.  On the pushed page I'm then doing some processing and then say want to add a new marker to the map on the previous/parent page I execute the callback function that was passed through which does whatever interactions with the map then use the nav control pop() function to return back to the map which is then blank, doesn't seen to matter what kind of map interaction is done in the callback function results in map being blank when coming back to the page.
Originally I was trying to add a new marker in the callback but changed it to just call animatecamera() to move the map to another position, now I understand that the map is technically not visible or the active page on the screen so don't know if that has anything to do with it and just have to change the logic of my code.  I added plugin.google.maps.environment.setBackgroundColor('green'); to the maps page ionviewdidenter function() which when it comes back to the map page from the pop() it just shows a green background so don't know if that means anything...
I don't know if its because I'm doing camera manipulations that its causing this.  I guess I should try to just add a new marker and see what happens without any camera manipulations or probably the more proper way would be the pass the info back the parent/pop page to process on it.

Too long description. Please write your question more simply.

Main page (with map)  button pressed to show another page passing a callback function from current page.
let params = {
   callback: this.someMapInteractionCallback
};

this.navCtrl.push(SomeOtherPage, params);

someMapInteractionCallback() {
   this.map.animateCamera({
      target: {latitude: 39.8283, longitude: -98.5795},
      zoom: 4
   });
}

SomeOtherPage, execute the callback that was passed through then pop() back to the map page results in map not showing now.
   this.callback().then(() => {
      this.navCtrl.pop();
   });

Use map.setDiv() for detach the map from webview, then map.setDiv(div) for attaching thr map again

I actually do have that, I have this on my main page.
ionViewDidLeave() {
    plugin.google.maps.environment.setBackgroundColor('red');
    this.mapsProvider.map.map.setVisible(false);
    this.mapsProvider.map.map.setDiv(null);
    plugin.google.maps.environment.setBackgroundColor('red');
}

ionViewDidEnter() {
    plugin.google.maps.environment.setBackgroundColor('green');
    this.mapsProvider.map.map.setDiv(this.mapsProvider.map.mapElement.nativeElement);
    this.mapsProvider.map.map.setVisible(true);
    plugin.google.maps.environment.setBackgroundColor('green');
}

Which just results in when I this.navCtrl.pop(); from SomeOtherPage a screen with a green background IF I call any camera manipulation functions in the callback function.

Indeed as I have found that is so; I don't know if that is clearly marked somewhere or just considered common sense :).  I naturally kind of expected it to fail or not work; just silently.  But it just completely breaks if you try too.
Though the word manipulate can be taken multiple ways as you can add markers to a map that is detached which you'd kind of think would be manipulating the map as it adds a marker on the map and what not...  but apparently that's fine and its just when you manipulate the camera position (and who knows what else) that it breaks completely.

Another quick question I have while I'm here... is there a function or something to like refresh/redraw the map?
TL;TR details...
What I mean by that is sometimes in my app when interacting around with things, it's like the screen on the device freezes and looks like it didn't do what you wanted it to do its not until I touch the screen and like move around on the map that things 'kick in' and it shows the page.
I have two main instances where that happens, one is a side menu with buttons that when clicked loads a bunch of markers on the map then uses the bounds of the markers to animate camera to those bounds.  90% of the time it works as should and loads up the markers and goes to the bounds of them all, the other 10% after clicking the buttons you can see in the console log that its processing the action and adding the markers and doing everything as it should accept your screen is still just a blank map still like nothing happened.  It's not until I touch the screen and drag my finger to move around the map that all of a sudden the page/device/map wakes up shows the map all of a sudden with the markers and camera already at the bounds.
Another instance it happens on is related to the same area in my original question here where sometimes when .pop() back to the main/map page again you'll see in the console it did navigate back and its running and processing everything for that page correctly but your device screen is like frozen and still showing the previous page UI you were at and once again touching the screen and dragging finger to interact causes things to kick in and redraw and show the map page.

is there a function or something to like refresh/redraw the map

All functions are documented. If there is no function, answer is no.


Could not find txrudp==0.5.0

I've just pulled the latest version, and pip install -r requirements.txt is failing with the following error:
Could not find a version that satisfies the requirement txrudp==0.5.0 (from -r requirements.txt (line 3)) (from versions: 0.2.0, 0.3.0, 0.4.0, 0.4.1, 0.4.2, 0.5.1)
No matching distribution found for txrudp==0.5.0 (from -r requirements.txt (line 3))

We were testing a new package and while testing pypi I removed 0.5.0. Until we upgrade requirements.txt you can bump to 0.5.1.
pip install txrudp==0.5.1


Dbutson/dashboards

Added interval to all dashboards so that you can control graph visualization when using large time windows.

I can't really figure out what visualization changes have been performed by looking into the json files. Maybe we could figure out some way to make the dashboards reviews easier ... attaching pre/post screenshots or something else?

It's worse than not being able to verify.   Whenever you export a dashboard, it exports in the dashboard version of the release you are using.    So it might not be 100% backward compatible with previous version of grafana.


Fix sub-crate/utility testing

adds a testing step to CICD which performs a cargo test for each individual uu sub-crate
fixes all failing tests and warnings
increases test coverage by 1.79% with minimal change in testing times
adds an additional target ('test-util') to cargo make ... for testing of individual (or all) util(s)

I want to look this over one more time before I release it for a merge.

now, increases test coverage by 5.08%
testing times are increased about 20% (from about 9-12 mins to 11-14 mins)
fixes #1587 with a test modification

I think that this is now ready for review.


Develop

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5f3fabb). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage          ?   10.64%
  Complexity        ?        7
=========================================
  Files             ?        5
  Lines             ?      310
  Branches          ?       25
=========================================
  Hits              ?       33
  Misses            ?      275
  Partials          ?        2

Continue to review full report at Codecov.

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


Add static code analysis to projects

We should bring in some static analysis and coding conventions to our projects.
Suggested tools:

JSHint for common JS errors.
JSCS for coding conventions.
polylint for common errors in Polymer code.

Atom has plugins for JSHint, JSCS and polylint, but unfortunately JSCS is still missing the feature to lint JS code inside HTML documents (https://github.com/AtomLinter/linter-jscs/issues/110). Also the polylint plugin for Atom seems to be quite buggy.
Using JSCS from command-line does work with HTML documents via `--extract' flag:
$ jscs --extract vaadin-combo-box.html -p google .
...
One space required after "if" keyword at ./vaadin-combo-box.html :
   253 |
   254 |    _onArrowDown: function() {
   255 |      if(this.opened) {
----------------^
   256 |        this._focusedIndex = Math.min(this.$.selector.items.length - 1, this._focusedIndex + 1);
   257 |      } else {
...

Whether we're using these tools inside editor/IDE or not, we should still add these as part of our build process to catch errors at the latest on Travis build.
TODO:

[ ] JSHint + JSCS for vaadin-combo-box
[ ] Polylint for vaadin-combo-box
[ ] JSHint + JSCS for vaadin-grid
[ ] Polylint for vaadin-combo-box

Played around with Gulp tasks for linting JS code with JSHint and JSCS.
https://github.com/vaadin/vaadin-combo-box/commit/2c623f7c07ceb93d1a5c8d156678296a8e0a127b

https://github.com/vaadin/vaadin-combo-box/pull/64


Automatically run on all files in current directory

I think eclint should automatically run on all non-binary non-vendor files is finds in the current directory
$ eclint check

should equal to something like
$ eclint check '**/*.*' '!node_modules/**/*.*'

coupled with #45 this would make eclint a config-free tool like xo for example

@bfred-it fixed in master branch.

should we release of a beta version first

@gucong3000 I can't actually build master right now because https://github.com/DefinitelyTyped/DefinitelyTyped/pull/14996 isn't yet published. Just waiting on that is all.


Write a Swift program to find roots of quadratic equation

Description
Write a Swift program to find roots of quadratic equation
Use the quadratic formula to find the roots of the quadratic equation
x = $(-b &plusmn \sqrt(b^2 - 4ac))/2a$
The input will be the coefficients of $x^2$, $x$, and the constant $c$.
Input  : 1 2 1
Output : -1 -1

Contributed by - @PavanKumarCB

How to contribute

Save the solution in FindRootsOfQuadraticEquation.swift file
Add FindRootsOfQuadraticEquation.swift file in program/find-roots-of-quadratic-equation folder

!assign


Fix/27165: don't allow go to next step if empty waypoint

Details

In distance request, prevent request money if empty waypoint
Fixed Issues

$ https://github.com/Expensify/App/issues/27383
PROPOSAL: https://github.com/Expensify/App/issues/27383#issuecomment-1718148390
Tests

Request money -> distance
Change /distance in the URL to /participants
Verify that the App redirect to Distance request page

[x] Verify that no errors appear in the JS console

Offline tests

Same above
QA Steps

Request money -> distance
Change /distance in the URL to /participants
Verify that the App redirect to Distance request page

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/141406735/2128634b-047a-42fc-a1a7-3f34d38e65fd

Mobile Web - Chrome

https://github.com/Expensify/App/assets/141406735/198fa9e0-23a8-4e9c-babd-e60432c43280

Mobile Web - Safari

https://github.com/Expensify/App/assets/141406735/c853c79e-65cc-4fe8-bd4e-2e007767acee

Desktop
https://github.com/Expensify/App/assets/141406735/755c6462-cf79-4cfe-93cd-b74e9a132d5d

iOS

https://github.com/Expensify/App/assets/141406735/f2b6739a-6f2b-4240-ab2d-41ed5d297f5e

Android

https://github.com/Expensify/App/assets/141406735/ac9df1d9-9d42-46de-96a7-b7a6fb84ee61

@allroundexperts @tgolen Currently, We have a general bug that relates to shouldReset logic as mentioned in https://github.com/Expensify/App/issues/26946#issuecomment-1740415582 and https://github.com/Expensify/App/issues/28566
Context:

Go to Distance Request -->
Fill start point and finish point -->
Go to Participant Page -->
Go to Confirmation Page -->
Go back 2 times to Distance Request -->
Click Next Button

After step 3, id field in iou will be request${reportID of participant}. When clicking next button at step 6, we miss reportID of previous participant
https://github.com/Expensify/App/blob/c415f7b2758aeab8e06913a634f4079a1c12a2d8/src/libs/actions/IOU.js#L2252
It makes shouldReset is true and iou field is reset, all data is lost

IMO, we should hold this PR until https://github.com/Expensify/App/issues/26538 is resolved

Yeah, either we should hold this PR until my refactorings are done, or merge this PR with the known shouldReset bug. It's up to you.

@tgolen I think we should hold this PR to avoid unexpected behavior for users. If it is ok, please help to hold it. Thanks

Just an FYI that my PR went into review yesterday.

@allroundexperts It seems there are no blocker now, could you help to check? thanks

Bump @allroundexperts Could you help to take a look at this issue?

@allroundexperts It seems some other change makes this PR no longer hold. I can remove the hold label now

@allroundexperts Bump please help to take a look at this PR. Thanks

@allroundexperts Bump could you help to check this PR?

I'm not sure why this was taken off HOLD. The PR that it was held on has not been merged.

Yeah, either we should hold this PR until my refactorings are done, or merge this PR with the known shouldReset bug. It's up to you.

Did you decide to move ahead with merging this with the known shouldReset bug @DylanDylann?

@tgolen We decided to hold this PR because of this reason. But in some days ago, I re-tested this PR and saw that It works well. It seems some recent changes make this PR isn't blocked anymore.

OK, as long as the shouldReset bug isn't an issue, I'm fine with that. Thanks for explaining!

@allroundexperts Bump, please help to check this PR? Thanks

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.3.96-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.96-15 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.97-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


No Cmdlets available for www-data user on Ubuntu

I developped a script that runs fine for my unprivileged user.
Then I wanted to execute it from a PHP script, and I ran into the problem that no Cmdlets are available. Investigfating further, I found out that www-data user cannot run any Cmdlets. The Powershell itself can be loaded though.
Steps to reproduce
Install Powershell for Ubuntu on an Ubuntu machine that has a www-data user created by apache.
Then try this :
some-user $ sudo su -s /bin/bash www-data
www-data  $ /usr/bin/powershell
PowerShell
Copyright (C) 2016 Microsoft Corporation. All rights reserved.

Cannot load PSReadline module.  Console is running without PSReadline.
PS > get-command

Expected behavior
List of all availbale Cmdlets.
Actual behavior
get-command : The type initializer for 'System.Management.Automation.AnalysisCache' threw an exception.
At line:1 char:1
+ get-command
+ ~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Get-Command], TypeInitializationException
    + FullyQualifiedErrorId : System.TypeInitializationException,Microsoft.PowerShell.Commands.GetCommandCommand

Environment data

I am running Powershell for Ubuntu 16.04 on Ubuntu 16.10
PS > $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.0.0-alpha
PSEdition                      Core
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.15
CLRVersion
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

PS > Get-ExecutionPolicy
Unrestricted

Hum!!  I used to get the PSRreadline error on the older PowerShell Alpha version.  Have you try up get PowerShell v6.0.0-alpha.17.
The newer version has lots of fixes.  I'm not getting the PSReadline error now.
Give it a try and post the results.
:)

@MaximoTrinidad Thanks for this ! I indeed upgraded to PowerShell v6.0.0-alpha.17 and this did the trick ! So problem solved.
Cheers

That's Awesome!!  Glad to help!


Add exclude parameter to raw.plot_psd()

Fixes #9441.
This adds an exclude parameter to raw.plot_psd(), which makes it possible to plot the PSD of all channels (including bads) using raw.plot_psd(exclude=[]).
Questions/To Do:

[x] Add changelog in API or new section?
[x] Add .. versionadded :: 0.24.0 tag?
[x] Add note somewhere (docstring?) to show how to plot the PSD for all channels (including bads)?

@drammock any idea what is wrong with the docstring? It ends with a ".", so I don't understand the error message.

@drammock any idea what is wrong with the docstring? It ends with a ".", so I don't understand the error message.

missing space between .. and versionadded

Ultraslow is timing out, otherwise looking good.

thx @cbrnr !

@larsoner there are two unused imports in mne/preprocessing/nirs/_scalp_coupling_index.py introduced in #9522 - do you want me to fix it as part of this PR or do you want a separate one?

Just pushed a hotfix

All green, @drammock or @larsoner feel free to merge.


create custom authorizer within CLI - manually created authorizer will be overwritten

Is your feature request related to a problem? Please describe.
Yes, I am unable to configure a custom authorizer (cognito) with the CLI, and also unable to use the CLI to set my api gateway to use COGNITO_USER_POOLS
Describe the solution you'd like
something similar to the suggestion in this closed (inactivity) issue:
https://github.com/aws-amplify/amplify-js/issues/1702
Authorizer:
Name: CustomerCognitoAuthorizer
Type: COGNITO_USER_POOLS
ProviderARNs:
- !Ref CustomerCognitoPoolARN
IdentitySource: "method.request.header.Authorization"
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
I've tried to manually created the "custom authorizer", but every time I amplify push it will get overwritten/removed.
amplify-API docs mention:
"Cognito User Pools Authorization
You can use the JWT token provided by the Authentication API to authenticate against API Gateway directly when using a custom authorizer."
.
..but I can't find a way to create this custom authorizer in the CLI.
Am I just going about this the wrong way??

thanks @Ashish5591, surely it must be possible to use COGNITO_USER_POOLS with the cli? do you know what I'm doing wrong?

any thoughts @attilah ?

@blomm At the moment, the CLI doesn't support Cognito custom authorizers out of the box. (although you can modify the APIGW Cloudformation tempalte and manage it yourself).
Having said that, the CLI  does support IAM authorization which works well with Cognito Idenity and user pools.

thanks very much @kaustavghosh06 , can you steer me towards some documentation on implementing this?

@kaustavghosh06
I've investigated rolling my own cloudformation template for Custom Authorizer, and it's way too complicated. I'm just going to end up breaking my existing amplify-generated template.
The other option of using IAM is equally complicated, correct me if I'm wrong but I need to:

Create Policy that says what/how a user can query dynamo tables
Create Role,  above policy to this role.
Create Cognito Group, attach above Role to Group
Attach this group to cognito users

I've done all the above, now I'm trying to pass my auth token:
on the frontend I call Auth.currentSession() which returns currentUser
I add to my Headers a Authorization property and attach currentUser.getIdToken().getJwtToken()
But my attempts to call my api-gateway endpoint result in 403's.

Hi @kaustavghosh06,
Do you know if anyone is currently working on supporting custom authorizers (with cognito user pools)?
I'm probably going to need to abandon Amplify and switch to using Serverless (where custom authorizers works very well) until this has been implemented.

We are currently stuck with the same issue. We currently configure the authorizer and the gateway by hand but we have to redo it every time we add a new path as that overwrite the configuration. I'm now investigating editing the cloudformation templates manually. Is there any example for how to do that? Doing it with the API would preferred though to avoid conflicts with changes done by amplify.

Hi @steffengr,
I found this, and started to have have a crack at it, but I've found the amplify cloud-formation stuff to be like a house of cards (the amount of times I've started my backend over again from scratch), so I'm frightened to touch it.
If you figure it out, let me know please, and also as I said, Serverless implements it really easily, a few lines in the yml file, and you're good. So it's possible to just use Amplify for storage or auth and leave the api-gateway / lambda scaffolding work with Serverless.
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-apigateway-authorizer.html

Thanks @kylekirkby , Serverless support this custom authorizer directly from the yml file, so it shouldn't be too big a task, just a matter of setting up the code to generate the cloudformation, and add this as a step in the cli

@attilah @kaustavghosh06 any idea if this is doable? Would be awesome to have Lambda Authorizers added so we can provide a custom lambda function for authenticating users. I've got a SaaS product which creates multiple API keys for users projects and currently I'm doing the lookup of the API key via the lambda function that does the business logic. I'd love to be able to separate this logic out for checking if their API key is valid so this can be re-used easily across functions.

Same issue here. Appreciate sharing any ETA on this.

@attilah  @kaustavghosh06  @powerful23  @dabit3, can you use any of your special powers to get this issue expedited?

Hey @kylekirkby, checking on this now.

It's such a huge issue how is this not a priority?

+1

+1

+1

+1

+1

It's been 3 years, any updates on this?

This issue along with not being able to pull manual console changes is just ridiculous. This project could have been something great.

I know its not a CLI integration but you can create an api override specifically for this issue in this article.
https://docs.amplify.aws/javascript/build-a-backend/restapi/override-api-gateway/


Consider already written rows

Codecov Report

Merging #12 (eea7884) into main (6e3108f) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##                main       #12   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        81        81
===========================================
  Files             11        11
  Lines            236       239    +3
===========================================
+ Hits             236       239    +3

Impacted Files
Coverage Δ

lib/Table.php
100.00% <100.00%> (ø)

lib/TableWriter.php
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Dashboard color standards and better aggregations.

During a past demo, I ended up performing some improvements in our Grafana Dashboard to enhance the data visualization.
The Grafana Dashboard has been updated enforcing a color standard. Now all the graphics related to mutated request are purple, accepted requests are green, reject requests are red.
Furthermore, some queries have been updated to aggregate different time series in one single line.  This is necessary because we have many label in the metrics and each different value generate another time series. Thus, we can leave with many lines in the graph which does not tell use useful info.

As the requests from @ereslibre are small and I've already pushed them. I'm merging this. Thus, we can publish the dashboard in the Grafana.com
@ereslibre if you have more changes, let me know. I'll open another PR for them.


Upgrade python requests to latest version

CVE-2018-18074
moderate severity
Vulnerable versions: <= 2.19.1
Patched version: 2.20.0
The Requests package through 2.19.1 before 2018-09-14 for Python sends an HTTP Authorization header to an http URI upon receiving a same-hostname https-to-http redirect, which makes it easier for remote attackers to discover credentials by sniffing the network.

Fixed as part of  https://github.com/awslabs/scale-out-computing-on-aws/commit/ec279b83a1743cddd5029d68acae2754a895fa02


Error finding configuration file

I just installed with npm install -g bible and when I run bible, I get the following error:
warn  [10:10.51 - 09.07.2015] No configuration file was found. Initing the configuration file.
fs.js:502
  return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
                 ^
Error: ENOENT, no such file or directory 'undefined/.bible-config.json'
    at Error (native)
    at Object.fs.openSync (fs.js:502:18)
    at Object.fs.writeFileSync (fs.js:1103:15)
    at Object.<anonymous> (/usr/local/lib/node_modules/bible/bible.js:43:12)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)

Even if I create the configuration file by hand I still get this error.

Thanks for the heads-up! Indeed, we have to update the dependencies.

Hopefully, this is now fixed! Install it using npm i -g bible@latest. :tada:

That works! Thanks for the quick work.

@aharris88 You're welcome! I noticed that error but waited for the first guy to report it. :smile:


COB

#625  🐛 Bug Report
The new update with the Cob x2 on nightmare doesn't seem to work like it should work. With a x2 boost you'd expect 2 kills on a solo trip. But for me, with 200+ kc, full inquisitor's and Cob, I do a solo trip (1 kill) in 17mins. Without Cob and Inq a solo trip is 19mins..

Should be fixed now.


Support zip in reduce

Codecov Report

Merging #203 into master will decrease coverage by 0.31%.
The diff coverage is 62.5%.

@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   88.09%   87.78%   -0.32%
==========================================
  Files          19       19
  Lines        1210     1195      -15
==========================================
- Hits         1066     1049      -17
- Misses        144      146       +2

Impacted Files
Coverage Δ

src/reduce.jl
82.89% <100%> (-10.53%)
:arrow_down:

src/basics.jl
38.09% <50%> (-5.66%)
:arrow_down:

src/core.jl
72.43% <0%> (-1.45%)
:arrow_down:

src/unordered.jl
93.02% <0%> (-0.6%)
:arrow_down:

src/show.jl
94.87% <0%> (-0.05%)
:arrow_down:

src/library.jl
93.75% <0%> (+0.46%)
:arrow_up:

src/progress.jl
92.77% <0%> (+4.99%)
:arrow_up:

src/interop/blockarrays.jl
100% <0%> (+12.5%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Add onEnter callback

onEnter could be used to retrieve the selected feature data and send it back when enter is hit.

Skip this.


Example persistence for AssetIndex

Would be useful to add a sample/extension that exposes an API that permits to get/query the AssetIndex loaded from the FileSystem.

This will come back with more precise requirement in the next days.


ie10-viewport-bug-workaround.js missing in bower package.

ie10-viewport-bug-workaround.js used in examples is not embedded in the bower package of bootstrap. This makes building combined/minimized/compressed javascript libraries with Grunt harder. Would be great to have it there.

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <[bootstrap] ie10-viewport-bug-workaround.js missing in bower package. (#16943)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.

@websafe I believe that is due to it being a CC3 license since it is in the docs not the MIT license of the core. I may be mistaken as I think there was discussion of changing that. @cvrebert or someone other than me (not on a cell phone) could probably double check that one way or the other.

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <Re: [bootstrap] ie10-viewport-bug-workaround.js missing in bower package. (#16943)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.

@kkirsche OK, I think I've found a solution :-) There is a dedicated bower package containing this file https://github.com/Haixing-Hu/bootstrap3-ie10-viewport-bug-workaround so I'll use it.

@kkirsche We already relicensed it; see https://github.com/twbs/bootstrap/commit/c111c291be426c5e91ceb1b8fe506d0df9818a2d

@cvrebert So can I assume it will be available in the bower package of bootstrap and https://github.com/Haixing-Hu/bootstrap3-ie10-viewport-bug-workaround won't be needed anymore?

No. Like the other user-agent string sniffing browser hacks, we don't consider it part of Bootstrap proper. See also http://getbootstrap.com/getting-started/#support-ie10-width


Adding a very weak AI player for demo purpose and future development

Add a very weak AI player for demo purpose and future development, and had enable it by default. In the next few days I will make the UI compatible with local, network and ai mode, hence players can choose the mode they need.

I also found bugs in the AI player logic when the search space had set a little larger, but I have no time to investigate further.


Fix for UNKNOWN_MESSAGE_TYPE

Pull Request Etiquette

[X] I have checked the PRs for upcoming features/bug fixes.
[X] I have read the contributing guidelines.

Changes

[ ] Internal code
[X] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other:

Closes Issue: NaN
Description
This commit adds the THREAD_STARTER_MESSAGE to the MessageType enum
See
java.lang.IllegalArgumentException: UNKNOWN_MESSAGE_TYPE
        at net.dv8tion.jda.internal.entities.EntityBuilder.createMessage(EntityBuilder.java:1289)

This error happens when retrieving the first message of a thread, because it doesn't recognize the MessageType it errors.

Closed since #1759 has been created , and I'm missing the THREAD_CREATE MessageType


Mark homework 1, exercise 3  of Peigen Zhou

Dear Professor @cecileane ,
I just finished my exercise 3, here is the last commit 503118e410ba3dfbaf1ae08006aad03490777c05
There are three updates

hw1/readme.md
scripts/summaryizeSNaQres.sh
results/summary.csv

For this commit,

log and out files names are all changed.
results/summary.csv has been created

For testing all the scripts work well, you can use following commands from terminal in hw1 directory.
make clean
make

Thanks Peigen! This is my way of asking Irene (@anqis929) and Chelsey (@ChelseyGreen) to try your script and provide feedback on it.

Hi Paigen @BaconZhou @cecileane ,
Great Work !
I especially like how you undo the file names for us so that we do not have to the change the file names back when doing the other peer review.
The way you used "grep" "sed" shows your logic clearly.
The only thing I think you could improve is adding some comments to each command so that people would know what this line of code was used for.
Irene

Hi Paige @BaconZhou @cecileane
I'm going to list out my feedback since that's the easiest way for me to stay organized.
(1) Wow! I'm impressed by your readme file in your HWK1 directory- you've definitely shown me the markdown formatting I should be striving towards! One small request for addition, I don't think I saw it specified from where the script should be run or the hierarchy of folders your script is assuming. Is that what "working directory" maybe means? I'd also like to see a short readme file in each of your other directories to identify the files that are in it - when they were created and by what script, if applicable.
(2) The script works as prescribed. I like that you put the CSV file in a separate directory. I didn't see however, a warning to the user (in your readme or in the actual code) that you'll be creating this file and directory -- it would overwrite any files that have with that name in an existing directory.
(3) Your use of grep, sed, and cut together required a knowledge of each of the function. You can do most of these calls, however just using the sed command, since it is able to replace everything other than what you don't want (using .*). I can show you this use of sed sometime in class if you'd like.
(4) I like that you thought to only return the loglik values before the period for the integer math. I returned the decimal part and then had to strip it off-- your method is more efficient!
(5) You can streamline your code a bit by nesting the if statements within each other. Then if a lik fails the -le 3460 check, it will automatically bypass the le 3450 and 3440 checks. I don't think doing 2 extra if statements will hurt performance much if any especially with the number of values in loglik, but it's a way to make your code a bit more compact.
(6) I'd recommend you add some comments at the top of your code to explain what it does and throughout the code so its easier for a user to understand the steps you have taken.
Please let me know what questions you have on my feedback and suggestions.
Thanks for letting me read through your code!
Chelsey


An ability to resolve references using ASM, without classloading

ClassNodeResolver now has two optimization options, both true by default: asmResolving to enable resolving with ASM (see AsmDecompiler class), and classLoaderResolving to enable the old, classloader-based resolution. Command-line compilers are supposed to turn the latter off.
I haven't observed significant difference in running times for tests or building groovy-core, but compiling against this jar in IntelliJ definitely gives some speedup. I'm not even sure that ASM resolving should be on by default, but as I haven't observed slowdowns on my Linux, I turned it on. Testing on other OSes might be helpful.
A couple unresolved questions:

Java5 treatment of ParameterizedType is incorrect: it doesn't take into account getOwnerType(). I've made ASM decompiler behave the same way for now. I don't know how to represent such types in ClassNode API.
Java5#configureClassNode has some logic concerning package annotations. I strongly suspect that it never works because compiled classes don't have modules (and hence packages). I couldn't find how to make this code work, so I didn't reflect the same logic in ASM decompiler.
I've added only unit tests. Maybe, tests involving full-blown command-line compilation could also be useful, but I haven't found examples of those yet.

Possible improvements:

Lazy MethodNode parameters and return type parsing
Caching ClassStub. One should be careful to avoid false positives when different same-named classes are used.

BTW: how about creating a class/interface with all available optimization options as constants?

Jira issue for this enhancement is https://jira.codehaus.org/browse/GROOVY-4024

Peter, there might be some issues with the PR on Windows that we hadn't noticed before. See the comments on PR#641 and https://jira.codehaus.org/browse/GROOVY-7346. Or it could be unrelated and we are missing something else altogether. If there is an issue, do you have any thoughts on where we might start looking? Thanks.

I have pushed a workaround for that problem: https://github.com/groovy/groovy-core/commit/1a0969697cc311af1902cc2489a7b80f56a6e878
Basically, the problem is that ClassLoader#getResource is case insensitive under Windows (and probably on any jar found on on a case insensitive FS but I couldn't reproduce). So groovy/util/NODE.class matched on groovy/util/Node.class, which is rather unfortunate. I couldn't find any way to make getResource case sensitive, if any of you has a better idea.. /cc @PascalSchumacher

@melix @paulk-asert Thanks for looking into this issue! I wasn't aware of this getResource behavior, sorry. As it turns out, ClassLoader does the same: defineClass also checks if the name in class file equals the name being searched for. So the fix looks right for me.

Excellent, thank you Peter!


Enable banner and point to new blog posts (pint and kvikio)

Trying out @andersy005 's suggestion in https://github.com/xarray-contrib/xarray.dev/issues/223#issuecomment-1232074343

Ok looks good, so merging now


A black block appears in my desktop when I launch and minimize Terminal.

Windows Terminal version
1.15.2875.0
Windows build number
10.0.22621.754
Other Software
No response
Steps to reproduce
Open the Terminal and minimize it.
Expected Behavior
No response
Actual Behavior
A small black square appears in the upper left corner when I open Terminal.

Then, when I minimize that, a small black rectangle appears in the bottom left corner.

Since you closed this issue, I'm guessing that the problem went away? Thanks!

Since you closed this issue, I'm guessing that the problem went away? Thanks!

I realized that a application called "Mica For Everyone" which changeed forms in Acrylic or Mica appearance globally was running in the background after I reported this issue.

In default, nothing appear.

But when I set the Backdrop Type of Mica For Everyone to "Mica", the black block appears.
(Set to "Acrylic", the block is gray.)

Then I use "Pop-up blocking" in Huorong Security application, I found it may cause by openconsole.exe of Terminal.

And then, I terminate Mica For Everyone and restart Terminal, the block disappears, but it still can be found by Pop-up blocking.

I don't know this block was caused by Mica For Everyone or openconsole.exe of Terminal, or why openconsole.exe ran in the desktop invisibly in default. But it can be hidden by Mica For Everyone setting, so I closed this issue.


Task - Hygrogen react components (Accordion, Dialog, Button)

Notes:

The components can be viewed in the H2 Components section in storybook.
Uses the compound component pattern. This pattern allows a component and it's sub-components to share an implicit state, and handle logic together (using the useContext hook). This pattern should help create more declarative and flexible components.

using the useContext hook
👏 Yay! I've been wanting to get used to using this more!

using the useContext hook
👏 Yay! I've been wanting to get used to using this more!

using the useContext hook
👏 Yay! I've been wanting to get used to using this more!

Ya same 😄. Using the compound component pattern with our hydrogen components will make building the larger components much easier to customize and easier to read through.

using the useContext hook
👏 Yay! I've been wanting to get used to using this more!

Ya same 😄. Using the compound component pattern with our hydrogen components will make building the larger components much easier to customize and easier to read through.


Regression detection follow-ons

Once #201 is merged, some things:

[ ] Leaving a new comment each time is pretty noisy, we could modify a single comment instead.
[ ] We should gate a PR on no-regressions || (regressions && regressions-triaged) labels. I tried this with a basic workflow in #201, but it's limited by the fact that workflows can't trigger other workflows.

Suggestion: Comment on all failures, and only comment on passes if it's going from fail -> pass.

Suggestion: Comment on all failures, and only comment on passes if it's going from fail -> pass.

Thanks, done with https://github.com/C2SP/x509-limbo/pull/201/commits/88f27df61f1697bd75a74c52753398bd6200fce0


[Carbon] Submit PR for Ref changes for resizable TableHeader

Our Table Header resize feature is dependent on the base Carbon TableHeader component. We need to submit a PR to Carbon repo with these changes in order to delete our local copy of this file.

I think this is related to #834
Carbon has a column resize PR WIP https://github.com/carbon-design-system/carbon/issues/4751


Upload working docker image to Hub

Issue docker-compose
:
Step 24/41 : RUN apt-get update -yqq   && apt-get install -yqq --no-install-recommends     git     ssh     tar     gzip     ca-certificates     zip     unzip     libicu-dev     libpq-dev     libmcrypt-dev     libfreetype6-dev     libjpeg62-turbo-dev     libpng-dev     mariadb-client   && rm -rf /var/lib/apt/lists
---> Running in b958279a470b
W: Failed to fetch http://deb.debian.org/debian/dists/buster/InRelease  Error writing to output file - write (28: No space left on device) [IP: <IP_ADDRESS> 80]
W: Failed to fetch http://security.debian.org/debian-security/dists/buster/updates/InRelease  Error writing to output file - write (28: No space left on device) [IP: <IP_ADDRESS> 80]
W: Failed to fetch http://deb.debian.org/debian/dists/buster-updates/InRelease  Error writing to output file - write (28: No space left on device) [IP: <IP_ADDRESS> 80]
W: Some index files failed to download. They have been ignored, or old ones used instead.


Icons inside collapse card take the entire row.

I'm trying to display 3 icons horizontally inside a collapse card. I added the icon (i tags) into the card, but they show vertically. For some reason the i element takes the entire line. I tried changing css properties, including:
display: inline;
width: 20px

I also tried changing the parent div to display: flex. All of those did not help.
Here is a Jsfiddle example: https://jsfiddle.net/benCarp/Lz40t409/
These doesn't seem to be the desired behavior of an expansion card. Not sure what causes this unique behavior of the icons inside of the card.

It looks like .card has a flex-direction: column as you can see here.
You can, for example, set the .card's flex-direction to initial or row just to see how it could change.

The html structure is wrong. .card-body should be a child of .card.

At least for 4.1 this is not true. https://getbootstrap.com/docs/4.1/components/card/

I just checked "card" and "card-body" are there on the same div.

We should change that in our docs example.

I see the docs have been updated. Is this still an issue?

Closing since docs were adjusted.


Autocompile/compile key shortcut

Could you introduce either compile keyboard shortcut, and autocompile?

Hello @janat08
From reading #104  we get referenced to the source code, you can indeed recompile with the shortcut CTRL + SHIFT + ENTER.


[WIP] Prototype realm import file upload using Uppy S3 uploader

Not ready for a review.

Closing since this has became quite stale.


Update jaxb-api to 2.2.12

Updates javax.xml.bind:jaxb-api from 2.2.11 to 2.2.12.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "javax.xml.bind", artifactId = "jaxb-api" } ]

labels: library-update, semver-patch

Codecov Report

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

@@           Coverage Diff           @@
##           master     #669   +/-   ##
=======================================
  Coverage   81.13%   81.13%
=======================================
  Files          78       78
  Lines        5163     5163
  Branches      130      130
=======================================
  Hits         4189     4189
  Misses        974      974

Continue to review full report at Codecov.

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


fix: main.py file can not get correct args though workflow

The workflow .yml file should add the $ syntax, then the splitlines method can output the correct value when manual triggering the Github Action 🤣(has verified in the fork repo).
But I do not confirm whether it will be broken when triggered by the shortcut 😅.
can U help to verify it 🙇 ?
FYI:

https://stackoverflow.com/questions/4918067/bash-or-googlecl-new-line-in-a-string-parameter/4918413#4918413
https://stackoverflow.com/questions/26517674/passing-newline-within-string-into-a-python-script-from-the-command-line

Trigger it by action

Diff

before
after

I think it may broke, will try it now.

merge first

I think it may broke, will try it now.

Github action success: https://github.com/yihong0618/iBeats/actions/runs/1465701814

Works fine, Thank you.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_76s560sk84/clone/recipe/meta.yaml:24: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_76s560sk84/clone/recipe/meta.yaml:36: missing_pip_check: For pypi packages, pip check should be present in the test commands
INFO: /tmp/abs_76s560sk84/clone/recipe/meta.yaml:39: invalid_url: http://nbsphinx.rtfd.org/ : URL domain redirect nbsphinx.rtfd.org ->  nbsphinx.readthedocs.io
WARNING: /tmp/abs_76s560sk84/clone/recipe/meta.yaml:43: missing_description: The recipe is missing a description
ERROR: /tmp/abs_76s560sk84/clone/recipe/meta.yaml:43: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_76s560sk84/clone/recipe/meta.yaml:43: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_76s560sk84/clone/recipe/meta.yaml:43: http_url: http://nbsphinx.rtfd.org/ is not https
Errors were found


Incorrect Online status

This bot shows the incorrect Online status of Minecraft servers. When my Minecraft server is Offline, the bot shows that the server is Online and 0/0 are on this server.

Hey?

Hey! Sorry github stopped sending me notifications for this repo for some reason. I will look into this for the august update but seems like our bot is being a bit optimistic when figuring out whether a server is online or not.

"Optimistic" hah. Okay, thanks for response. Good luck bro 🤜


Config Opendatahub components

Hi, I am using K8s GKE on GCP instead of minikube.
Instead of minikube ip, have used GKE cluster ip and couldn't view the keycloak UI on webbroser. (chapter 4)
Also couldn't see any pods of opendatahub after applying the ml-platform.yaml with  changed to GKE cluster ip. (chapter 5)
Can you please guide on how to address the above issues and setup/configure the opendatahub and use Jupyterhub with spark and other associated components.
Regards
Arun

Hi Arun,
In GKE, you can use the actual domain name of your GKE instance instead of .nip.io.
Let us know if this works.
Regards,
Ross

Hi Ross, Thanks for the update.
As advised, changed the .nip.io to domain name (cluster.local) and executed.
The status/results as mentioned below, most of the pods are running and there are 2 issues.
1.the url provided by the ingress is not rendering the ui, it says "This site can't be reached".
2.minio-ml-workshop-n7tj4 and mlflow-5f9bddc57-rnvzg  pods are not running.
Appreciate your help to overcome the above 2 issues enabling to use the ODH components.
results:
kubectl get pod -n ml-workshop
NAME                                          READY   STATUS             RESTARTS        AGE
app-aflow-airflow-scheduler-f7fc5d4cb-vpjmt   2/2     Running            1 (12m ago)     23m
app-aflow-airflow-web-675f4597f9-jw65p        2/2     Running            1 (7m55s ago)   10m
app-aflow-airflow-web-85fcc5bb95-hzg4t        2/2     Running            0               23m
app-aflow-airflow-worker-0                    2/2     Running            3 (9m14s ago)   23m
app-aflow-postgresql-0                        1/1     Running            0               23m
app-aflow-redis-master-0                      1/1     Running            0               23m
flightsdatadb-0                               1/1     Running            0               20m
grafana-867988b65d-n88jt                      1/1     Running            0               23m
jupyterhub-7848ccd4b7-9696x                   1/1     Running            0               23m
jupyterhub-db-0                               1/1     Running            0               23m
minio-ml-workshop-6b84fdc7c4-vn62t            1/1     Running            0               23m
minio-ml-workshop-n7tj4                       0/1     Completed          0               23m
mlflow-5f9bddc57-rnvzg                        1/2     CrashLoopBackOff   8 (4m24s ago)   23m
mlflow-db-0                                   1/1     Running            1 (21m ago)     23m
prometheus-odh-monitoring-0                   2/2     Running            0               19m
prometheus-operator-7bc4c8c5b5-qmbpm          1/1     Running            0               20m
seldon-controller-manager-7f67f4985b-v5t5h    1/1     Running            0               23m
spark-operator-66f8dd959f-gmlgp               1/1     Running            0               23m
kubectl get ing -n ml-workshop
NAME                   CLASS    HOSTS                                    ADDRESS   PORTS     AGE
ap-airflow2               airflow.<IP_ADDRESS>.cluster.local                80, 443   24m
grafana                   grafana.<IP_ADDRESS>.cluster.local                80, 443   24m
jupyterhub                jupyterhub.<IP_ADDRESS>.cluster.local             80, 443   24m
minio-ml-workshop-ui      minio.<IP_ADDRESS>.cluster.local                  80, 443   24m
mlflow                    mlflow.<IP_ADDRESS>.cluster.local                 80, 443   24m
Regards
Arun

Hi Arun,
If you are on GKE. You can use the actual domain name of your GKE instance
instead of .nip.io.
Make sure the ingress controllers in your GKE is correctly configured. Try
running any service and expose it. See if you can access it.
Regards,
Ross
On Sat, Jul 9, 2022 at 9:54 PM arun990 @.***> wrote:

Hi, I am using K8s GKE on GCP instead of minikube.
Instead of minikube ip, have used GKE cluster ip and couldn't view the
keycloak UI on webbroser. (chapter 4)
Also couldn't see any pods of opendatahub after applying the
ml-platform.yaml with changed to GKE cluster ip. (chapter 5)
Can you please guide on how to address the above issues and
setup/configure the opendatahub and use Jupyterhub with spark and other
associated components.
Regards
Arun
—
Reply to this email directly, view it on GitHub
https://github.com/PacktPublishing/Machine-Learning-on-Kubernetes/issues/2,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANHLO36PT2LLXFN2GJFQVLVTGAB5ANCNFSM53DNJTEQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--

Ross Brigoli


MRG: Use h5io and fix tests

Closes #2420.
I also included some fixes I needed to get the test suite to run on my machine without as many warnings.

 Why bother? We also don't do it with any of the other external deps.

Really? Why bother? We also don't do it with any of the other external deps.

Wouldn't it be easier to maintain if we just had to copy and paste the entire package once it is updated upstream?

Looks like they don't copy the test folder, either. Compare:
https://github.com/scikit-learn/scikit-learn/tree/master/sklearn/externals/joblib
https://github.com/joblib/joblib/tree/master/joblib
I assume they copied everything else because they needed it to function. Maybe you didn't notice, but I actually have h5io structured so that _h5io.py is the only file with useful code in it, with no other files needed for it to work. So any updates to h5io will be in that single file, at least for now. This structuring was intentional to make it easy to include in externals in various packages actually...

Assuming that the package in question fullfils its testing duty, this makes sense, I have to admit. We don't want to run these tests.

On 20 Aug 2015, at 20:24, Eric Larson<EMAIL_ADDRESS>wrote:
Looks like they don't copy the test folder, either. Compare:
https://github.com/scikit-learn/scikit-learn/tree/master/sklearn/externals/joblib
https://github.com/joblib/joblib/tree/master/joblib
I assume they copied everything else because they needed it to function. Maybe you didn't notice, but I actually have h5io structured so that _h5io.py is the only file with useful code in it, with no other files needed for it to work. So any updates to h5io will be in that single file, at least for now. This structuring was intentional to make it easy to include in externals in various packages actually...
—
Reply to this email directly or view it on GitHub.

of the tests yet I would put the file in
mne/externals/h5io/_h5io.py
so the import from mne is mne.externals.h5io
thanks @Eric89GXL

@agramfort done


soft lockup - CPU#113 stuck  after terminating training process

It seems a zomby process is generated.

this can be resolved by setting your OS' ulimit to 65535

@zhisbug get it, thanks a lot

maybe refer to this? https://askubuntu.com/questions/875173/nmi-watchdog-bug-soft-lockup-cpu2-stuck-for-23s-plymouthd305

maybe refer to this? https://askubuntu.com/questions/875173/nmi-watchdog-bug-soft-lockup-cpu2-stuck-for-23s-plymouthd305

This seems not work. I currently cannot find why some of the cpus are still occupied in kernel mode after terminating the training process.


move.py - Give more meaningful reports when checksum mismatch

Right now it's baffling and meaningless when checksums fail.

Some extra info has been added but the key information - which files are different has not been added.

fixed in 02c5485


Issue #569 fixed/Introduces 'path' as an alternative to 'raster_path'…

Issue #569 fixed

Thanks, running tests not.

Thanks @Abhishek-kumar0503. As I noted in my overall review comment https://github.com/weecology/DeepForest/pull/617#pullrequestreview-1943695662 there are additional uses of raster_path that need to be updated and we need a test for this. Can you please add the test and then use it to check for other instances of raster_path that need to be changed.

okay @ethanwhite I created the function name def test_predict_tile_with_projection and add it in test
`def test_predict_tile_with_projection(m, raster_path):
# test with projection enabled
m.config["train"]["fast_dev_run"] = False
m.create_trainer()
prediction, projection = m.predict_tile(raster_path=raster_path,
patch_size=300,
patch_overlap=0.1,
return_plot=False,
keep_projection=True)
assert isinstance(prediction, pd.DataFrame)
assert set(prediction.columns) == {"xmin","ymin","xmax","ymax","label","score","image_path"}
assert not prediction.empty

assert isinstance(projection, pd.DataFrame)  # Assert that projection dataframe is returned
`
and check it then it pass the test. Please Review it.

Thanks for adding the test. Have you tried debugging the resulting error? If so, where did you get stuck?

ya it give error in unnecessary data type keep_projection. Solved!

Great! Can you rebase this to remove the merge conflicts and then I'll give it another look

@ethanwhite it give error in check style. Can you help me to solve this issue.

@Abhishek-kumar0503 - try running this locally:
yapf -i --recursive deepforest/ --style=.style.yapf

That should cleanup the style issue.

yeah,thanks it worked

Closing due to inactivity. Feel free to reopen if you return to this.


Odd display artifacts scrolling a long topic list

Perhaps these mean something to a more UI-knowledgable person. They happened while office hour was going on and there were a ton of active topics in the Outreachy stream. I had the full list open but it didn't all fit in my window so I was scrolling up and down through the list of topics. I am using OS X 10.11.6 and Safari 9. I see it both with my accessible minimum font size and without, so it seems to only require that the list of topics be too long to fully display.
First may be nothing actionable, it only happened once and went away soon thereafter. Gray blobs that look like something not entirely redrawn in the top section of the left sidebar.

This is a big black rectangle covering the top of the left sidebar, it flashes off and on when I scroll the list. It's still happening now, sometimes it looks like the first image, sometimes the second.

@feorlen can you try reproducing in another browser?  I'm curious if this is a safari-only bug.

@timabbott I'm not seeing it in Chrome (53.0.2785.116, font size "Very Large" for what that's worth.) Those are the two I have handy with mostly default configurations.

I tried restarting Safari to see if that made any difference, and at first it didn't happen even when I increased the font size and/or made the window shorter. But there were many fewer topics shown in the Outreachy stream. Then a minute later the whole list was back as before, and scrolling gave me the on and off black rectangle. And this too:

So maybe peculiar to having a long list of topics, not only having to scroll.

The topics shown are based on what the browser has cached, so the list will reset to less and then grow if you restart the browser.  Sounds like it's reproducibly problematic on Safari, at least.

Fixed via #2457, thanks for the report @feorlen!

Awesome! I just saw it again today scrolling through a bazillion messages.


Feature/sdpa 749 global alerts and warnings

JIRA issue: https://digital-engagement.atlassian.net/browse/SDPA-749
Changed

Added Alert component.
Updated Template page with alert.

Screenshots

Hi @alan-cole I assigned the ticket to Dori to wait her response. Cheers


Provide dedicated action for dependency graph submission

Please include buildscript (plugin) dependencies or allow as a setting

@ben-manes The new action will use the same mechanism as gradle-build-action, which should capture all dependencies including plugin dependencies.
Projects and configurations can be excluded, but I'd like to make this more configurable and have better defaults.

Wonderful, thanks

A first version of gradle/actions/dependency-submission has just been released: https://github.com/gradle/actions/releases/tag/v3.0.0-beta.6

hmm.. I tried the new action and get an error

Cannot locate tasks that match ':ForceDependencyResolutionPlugin_resolveAllDependencies' as task 'ForceDependencyResolutionPlugin_resolveAllDependencies' not found in root project 'caffeine'.

I assumed this was from an init script but isn't being applied to the root build.

oh! it is because I had setup-gradle run before dependency-submission for the old action. That combination fails in your new set up as the init script must be already configured. Once I removed it then it worked. Seems like an easy thing to warn on.

Thanks for the feedback. I haven't really announced the new dependency-submission action, and I'd say it's not quite ready for the prime-time. The idea is that it should be used in a separate, minimal workflow, ideally with very little user customisation.
But you're right: we should be able to warn if dependency-submission is used after a previous step that uses setup-gradle.

Note that you can continue to use setup-gradle with the dependency-graph option. In that way, setup-gradle should be a drop-in replacement for gradle-build-action.
Can I ask why you're running the help task as part of the "Setup Gradle" step?

yep! no worries that I'm a super early adopter and have zero expectations. just providing feedback as low risk to run on my project and not a priority for you. The old submission action was deprecated and archived, so I thought switching to was a nice friday activity.

oh! you would need to look at the v3 branch. Here is my PR of the process working.
The help was in order to force gradle to be downloaded and set up. Since Caffeine runs so many jobs, the CI would get rate limited with network timeouts. The backoff-and-retry was to minimize that for gradle and jdk set ups.
In the v3 branch, I removed the usage of arguments and that help check since isn't needed. I'm not sure if it ever was.


Crash when traversing over items() of temporary json objects

What is the issue you have?

crash when traversing over items() of temporary json objects
sometimes got null of pair.value() (https://godbolt.org/z/TTQFHW)

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

  const auto obj = nlohmann::json::parse(R"({
    "key1": 0.1,
    "key2": 0.2
  })");

  auto rvalue = [&]() -> nlohmann::json { return obj; };
  for (const auto& pair : rvalue().items()) {
    std::cout << pair.key() << ": " << pair.value() << "\n";  // crash
  }

  auto crvalue = [&]() -> const nlohmann::json { return obj; };
  for (const auto& pair : crvalue().items()) {
    std::cout << pair.key() << ": " << pair.value() << "\n";  // crash
  }

What is the expected behavior?

pair.value() returns 0.1 or 0.2
no crash

And what is the actual behavior instead?

crash on my linux machine: json.hpp:14028: void nlohmann::detail::serializer<BasicJsonType>::dump(const BasicJsonType&, bool, bool, unsigned int, unsigned int) [with BasicJsonType = nlohmann::basic_json<>]: Assertion ‘false' failed.
pair.value() returns null on Compiler Explorer. (https://godbolt.org/z/TTQFHW)

Which compiler and operating system are you using? Is it a supported compiler?

linux/centos
gcc 6.2

Did you use a released version of the library or the version from the develop branch?

latest dev branch
release v3.7.3

When you create rvalue().items(), an iteration_proxy is created that captures the JSON object to iterate by reference. However, the object created by rvalue() is destroyed too soon, and when the iteration proxy is used, it has a reference to a destroyed object. When you use ASAN, you see the details.
I am not sure how to fix this.

However, it works fine when traversing over temporary json array:
  const auto arr = nlohmann::json::parse(R"([
    "key1",
    "key2"
  ])");

  std::cout << "not crash1:" << std::endl;
  auto rvalue = [&]() -> nlohmann::json { return arr; };
  for (const auto& item : rvalue()) {
    std::cout << item << "\n";
  }

  std::cout << "not crash2:" << std::endl;
  auto crvalue = [&]() -> const nlohmann::json { return arr; };
  for (const auto& item : crvalue()) {
    std::cout << item << "\n";
  }

https://godbolt.org/z/5Y-fxW

Could we temporarily disable rvalue().items() and crvalue().items() by overloading with reference qualifier &&?
iteration_proxy<iterator> items() & noexcept
{
    return iteration_proxy<iterator>(*this);
}

iteration_proxy<const_iterator> items() const & noexcept
{
    return iteration_proxy<const_iterator>(*this);
}

iteration_proxy<iterator> items() && noexcept
{
    static_assert(false_v<iterator>, "items() of rvalue is not supported.");
}

iteration_proxy<const_iterator> items() const && noexcept
{
    static_assert(false_v<iterator>, "items() of const rvalue is not supported.");
}

Ref overloads are great, but such fix is a breaking change. Code like
template <typename R>
void iterate(R r);

iterate(get_json().items());

is valid. Though I hope there isn't much code like that.

I found this to be a undefined behavior

If range_expression returns a temporary, its lifetime is extended until the end of the loop, as indicated by binding to the forwarding reference __range, but beware that the lifetime of any temporary within range_expression is not extended.
for (auto& x : foo().items()) { /* .. */ } // undefined behavior if foo() returns by value

This problem may be worked around using init-statement (C++20):
for (T thing = foo(); auto& x : thing.items()) { /* ... */ } // OK

So, is there a way to work around in C++ 11/14/17 ? 😂

Simple workaround exists
{
    auto stored_foo = foo();
    for (auto& x : stored_foo.items()) { /* .. */ }
}

it doesn't answer if items should be ref-qualified or not.

Any ideas how to proceed here? Maybe adding a note to the documentation of items?

I think so. I mean proxy nature of items() is a feature, not a bug.

adding a note to the documentation

I hope so. It would be very helpful, especially for the users of dynamic languages (like Python).


[Bug]: Error updating channel. "Zodios: Invalid Body parameter 'body'"

Contact Details
<EMAIL_ADDRESS>What happened?
When attempting to save changes from the Channels>Edit screen, an error is shown stating "Error updating channel. Check browser console for details".
The browser console shows an error:
Error: Zodios: Invalid Body parameter 'body'
    at F6e.request (index-c6d43532.js:325:9596)
    at async RN.interceptRequest (index-c6d43532.js:330:779)
    at async B6e.request (index-c6d43532.js:330:4794)Caused by: ZodError
    at get error (index-c6d43532.js:321:23047)
    at F6e.request (index-c6d43532.js:325:9649)
    at async RN.interceptRequest (index-c6d43532.js:330:779)
    at async B6e.request (index-c6d43532.js:330:4794) ZodError
    at get error (index-c6d43532.js:321:23047)
    at F6e.request (index-c6d43532.js:325:9649)
    at async RN.interceptRequest (index-c6d43532.js:330:779)
    at async B6e.request (index-c6d43532.js:330:4794) "Zodios: Invalid Body parameter 'body'"

Version
0.5.3
What browser did you experience the the problem on?
Chrome 127.0.6533.120
What operating system are you using?
Windows 11
Relevant log output
Nothing logged to tunarr console.

Saw this recently with somebody in discord, though I thought we solved it. Are you using watermarks?

Yeah this particular channel has a watermark, but even attempting to remove the watermark results in the same error. I'll delete and re-add the channel without a watermark as a workaround.

I see. I will see if I can get some extra logging in there like I’ve added
on some other paths in order to figure out which field is giving a problem.
On Sun, Aug 18, 2024 at 3:39 PM Andy Marshall @.***>
wrote:

Yeah this particular channel has a watermark, but even attempting to
remove the watermark results in the same error. I'll delete and re-add the
channel without a watermark as a workaround.
—
Reply to this email directly, view it on GitHub
https://github.com/chrisbenincasa/tunarr/issues/688#issuecomment-2295367953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMQRX7AJQHZ56SX6E3TNWLZSDZ5TAVCNFSM6AAAAABMWMMZWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJVGM3DOOJVGM
.
You are receiving this because you were assigned.Message ID:
@.***>

I'm getting the same error when attempting to save after adding a filler list to a channel (channel has no watermark).

I can repro the filler collection. Still not sure about the other one yet.

Have an idea what's going on with filler lists and have a potential fix in place locally. Should be able to continue tonight though. Sorry for the inconvenience!

I have a bunch of fixes included in https://github.com/chrisbenincasa/tunarr/pull/689

These fixes are included in the 0.6.0 release, which is going out now. Once that's available, please give it another try and let me know if you have any issues! I have also included additional logging statements on this path so if you do hit another issue, we should be able to diagnose it more quickly than before. Thanks!

Yeah seems to have resolved those issues - thanks for the quick turnaround!

Awesome! Glad to hear it. Thanks for reporting, too!


continuous reboots from webui

Stratux version:  0.8r2  acee114964

Stratux config:
SDR
[ ] single
[ x] dual
GPS
[ x] yes
[ ] no
type: mtk3339
AHRS
[x ] yes
[ ] no
power source:  120vac wall wart
usb cable:

EFB app and version: (e.g., WingX Pro7 8.6.2) na
EFB platform: (e.g., iOS 9.2)
EFB hardware: (e.g., iPad Mini 2)

Description of your issue:
Using a Winblows 7 laptop and Firefox to watch the webui.   Wanted to reboot, so hit the reboot switch on the webui and the RPi3 (and -2) continuously reboots until I shut down the laptop browser page.
severity: critical
Does anyone else get this result?  It happened before with the RPi2 as well.

If possible, enable "Replay Logs", reproduce the problem, and provide a copy of the logs in http://<IP_ADDRESS>/logs/stratux/ and http://<IP_ADDRESS>/logs/stratux.log.

when you click "reboot" did you get the "are you sure" message?
once you say you are sure, did you see the text, "Stratux is rebooting. You may need to reconnect WiFi one it reboots." ?

bradanlane, I really don't remember.  The text you list sounds like just
a reminder and not a checklist item.  Either way, it is weird that the
Pi would continuously reboot based upon a message sent more than boot
ago.  Must be something in the boot shell script that does not get
over-written or replaced at the next boot.  That's just a guess as I
have not seen that particular bit of source code.
(note, if the ... shows up, it is because bradanlane's post did not show
up on github by the time I replied.)
Skypuppy
On 04/13/2016 09:32 AM, bradanlane wrote:

when you click "reboot" did you get the "are you sure" message?
once you say you are sure, did you see the text, "Stratux is
rebooting. You may need to reconnect WiFi one it reboots." ?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/389#issuecomment-209475768

I've been able to reproduce it. the problem is occurring from the browser. since the browser has not received a reply, it keeps trying the last URL. Since that URL is "http://ip_address/reboot" it reboots over and over. I even powered off my Stratux but left my browser open and as soon as Stratux was up and running, it rebooted.

Can we add a reply for boot request back to the webui browser page?
On 04/13/2016 10:00 AM, bradanlane wrote:

I've been able to reproduce it. the problem is occurring from the
browser. since the browser has not received a reply, it keeps trying
the last URL. Since that URL is "http://ip_address/reboot" it reboots
over and over. I even powered off my Stratux but left my browser open
and as soon as Stratux was up and running, it rebooted.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/389#issuecomment-209493788

I'm not that fluent with the oddities of the GO webserver. I'm working on a test to see how to clean up the active browser page. Ideally, I can make the browser go to the "status" page and wait for Stratux to finish rebooting.

An alternative is to remove the "reboot" option from the webui, but that
would interfere with other options, like the need to reboot in flight
until we get the other problems all sorted out.  On the ground, we can
reboot manually until a) the go web is fixed, or b) we drop the go web,
or c) we get all the other workarounds figured out.  :)
Still, one would expect an ack from the Pi to the webserver before
reboot was allowed, simply as a matter of routine protocol.  Or maybe
the webserver IS expecting an ack and not getting it from the Pi?
Still trying to fix things,
Skypuppy
On 04/13/2016 10:03 AM, bradanlane wrote:

I'm not that fluent with the oddities of the GO webserver. I'm working
on a test to see how to clean up the active browser page. Ideally, I
can make the browser go to the "status" page and wait for Stratux to
finish rebooting.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/389#issuecomment-209495792

the most likely "better" solution is for the reboot and shutdown to be AJAX web service calls rather than HTTP POST operations.

Maybe we need to have the reboot function launch another thread that will
sleep for a second or two and then do the reboot.  This would give time to
allow the response to get back to the browser.
Perhaps a simpler solution might be to add a .then method on the $http.post
function and force a flush - or send a config option with a very short
timeout - not sure if either of those would work.
If I could reproduce it that would make it easier to test solutions...
On Wed, Apr 13, 2016 at 10:03 AM bradanlane<EMAIL_ADDRESS>wrote:

I'm not that fluent with the oddities of the GO webserver. I'm working on
a test to see how to clean up the active browser page. Ideally, I can make
the browser go to the "status" page and wait for Stratux to finish
rebooting.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/389#issuecomment-209495792

If I set Wireshark to snoop it, that would not help if the Pi is not flushing the ack (if there even IS one) before the actual reboot occurs.
(And I wonder why these comments are not showing up on github?)  Replying via email leaves the previous text added.  Aside: before I knew the threads were also occurring on github, I was annoyed at some of the posts because there was no context.  :)  Now, I tend to reply only in github (if I can see the post.)
  NOW the posts are showing up on github.
Skypuppy

My guess is that the RPI 3 is just faster so the reboot is happening quicker.  @bradanlane is likely correct in that making this an AJAX call will probably resolve it.
Guess I need to order an Pi3!

We had this problem a while ago from the web UI
(https://www.reddit.com/r/stratux/comments/3ylhyl/video_of_possible_bootloop_problem). It got fixed and I haven't had a problem with reboot loops via my Android tablet. What changed?

I do not know if anything changed or this is a manifestation of a different browser (I can reproduce with Chrome on my Linux Dev system).

it looks like I have a fix.
following the "are you sure" message from a reboot, the WebUI now redirects to the status page and waits for the Stratus to complete its reboot process.
I'll create a branch, commit the code, and submit a PR.

Might I suggest the use shut down his web client when this oddity occurs?
From: bradanlane<EMAIL_ADDRESS>Sent: Wednesday, April 13, 2016 11:20 AM
To: cyoung/stratux<EMAIL_ADDRESS>Subject: Re: [cyoung/stratux] continuous reboots from webui (#389)
the most likely "better" solution is for the reboot and shutdown to be AJAX web service calls rather than HTTP POST operations.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/cyoung/stratux/issues/389#issuecomment-209505839 https://github.com/notifications/beacon/ARCCfeRDG-32nuzUIxLWreHaydJag75Iks5p3QmPgaJpZM4IGQsK.gif

It happens with -2's as well.
On 04/13/2016 10:37 AM, braddward wrote:

My guess is that the RPI 3 is just faster so the reboot is happening
quicker. @bradanlane https://github.com/bradanlane is likely correct
in that making this an AJAX call will probably resolve it.
Guess I need to order an Pi3!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/389#issuecomment-209515539

Yes. I did my debugging, development, and verification on my RPi2.

Yes, Rick, that is what I did to stop the rebooting.  However, a newbie,
up in the air ahd having unidentified problems with his stratux, will
resort to the reboot (we're all trained the Micro$loth way!) and then
his stratux may not come back at all -- and he won't know what to do.
In-flight failure is not going to be fun under any circumstances, much
less when the stuff has hit the fan.  And then, of course, is when stuff
happens to fragile computer gear and interconnects.  We must engineer
as much reliability as we can into a flight instrument, certificated or not.
Skypuppy
On 04/13/2016 11:59 AM, Rick H wrote:

Might I suggest the use shut down his web client when this oddity occurs?
From: bradanlane<EMAIL_ADDRESS>Sent: Wednesday, April 13, 2016 11:20 AM
To: cyoung/stratux<EMAIL_ADDRESS>Subject: Re: [cyoung/stratux] continuous reboots from webui (#389)
the most likely "better" solution is for the reboot and shutdown to be
AJAX web service calls rather than HTTP POST operations.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/389#issuecomment-209505839
https://github.com/notifications/beacon/ARCCfeRDG-32nuzUIxLWreHaydJag75Iks5p3QmPgaJpZM4IGQsK.gif
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/cyoung/stratux/issues/389#issuecomment-209545795


[QUESTION] [documentation] docs for openID connect usage

First check

[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.

Description
Hi,
Would it be possible to add a small manual on how to use openID connect in FastAPI? I know this has already been discussed in #12, but I'm having trouble getting it to work.
I've had mixed success using Authlib, but I'm not able to add this to the openapi spec, nor am I able to use the elegant Depends option to protect my API.
Any help would be much appreciated.
Thanks
M
Additional context

I would like to join this question, i am totally loss

https://docs.authlib.org/en/latest/client/fastapi.html
I managed to find this, it was solid.  That said, I would also love to see docs on how to directly do this in FastAPI.  I liked @tiangolo 's idea about connecting then swapping with a FastAPI generated token, it would be outstanding to see this documented in the way he described.


Fixed 'circular argument reference' warnings

Bow chica wow wow!

Looks great @baselalderi. Thank you.
Could you remove the .idea files from this ? It looks like editor specific stuff that accidentally got committed.
Other than that, should be good to fo.

Easy peasy.

👍


Help pls

I am writing a paper on "Interaction and bayesian graph for continuous data based on submodular factorization of perspective: a study for analysis". However, I don't have time to do a good literature review. How can I train caffe64 to write it for me? I would cite caffe64.s. (Not david fouhey, of course).

Pretty easy!

Collect a related work dataset from Arxiv
Train the network to predict that.
????
Profit

The key is that nobody reads the related work anymore, because in this modern era we only cite future work from Arxiv.


Add scalaPBProtocPath option to scalapblib

Add an option to ScalaPBModule to allow setting the protoc compiler. This is useful for systems that can't run the jar embedded protoc.

Niiiiice, as a NixOS user this would be super useful  :+1:

Thanks.


Unable to open shop GUI or use plugin

When I do /shop, an internal error occurs and I'm unable to perform the command. No GUI pops up. Plugin seems to be unusable. Is there something I'm missing here?
Pastebin link to my latest.log, showing the error : https://pastebin.com/sDwBegTg
Let me know if you need anything else to troubleshoot this, cheers.

I can't reproduce this. I'm guessing there is something wrong with your config files. Please share them in a gist/paste

@Kwikxilver @CACGaming Are you guys running the plugin for the first time or updating? Please let me know so I can resolve this bug

@CACGaming 's report:
I have the same issue:
https://pastebin.com/iku26CxN
This is the first install of the plugin and it fails when trying to open a shop.
Config.yml(Default): https://pastebin.com/tfwShV7c
Shop.yml(Default): https://pastebin.com/CquG6mgk

I have the same issue:
https://pastebin.com/iku26CxN
This is the first install of the plugin and it fails when trying to open a shop.
Config.yml(Default): https://pastebin.com/tfwShV7c
Shop.yml(Default): https://pastebin.com/CquG6mgk

@CACGaming Does the issue persist after a server restart?

@jmanpenilla It persists after restart and deleting the configuration to let it generate a new set

configs.zip
If you try using my config here do you still get the error? (You can ignore the DynamicShop-Graph folder, it is for another plugin)

@jmanpenilla It works just fine with your configs

@jmanpenilla this was my first install of the plugin using the default configs.
After replacing my config folder with the one you installed, everything seems to work fine.
Maybe it's an error with the plugin file you uploaded to spigot?

I don't own the spigot page, 7sat does. He uploaded a build that wasn't ready yet (2.11.8-SNAPSHOT). Would you mind testing if the same issue happens or not with 2.11.7 when starting the plugin with no configs (ie for testing delete the configs and test with fresh 2.11.7 from spigot)? Knowing if the issue is still present/is not present in 2.11.7 will help me determine when the issue was introduced and help me to fix it.

@jmanpenilla It works fine with default configs on version 2.11.7, however upon opening the shop for the first time it is delayed and warns:
[21:38:19 WARN]: Initializing Legacy Material Support. Unless you have legacy plugins and/or data this is a bug!
It isnt another plugin causing that, it only happens after running /shop

Ok, I can reproduce it. Seems like the 2.11.8-SNAPSHOT build is broken. Use 2.11.7, or the 2.11.8 that 7sat will hopefully release soon based on his master branch.

"[21:38:19 WARN]: Initializing Legacy Material Support. Unless you have legacy plugins and/or data this is a bug!"
This warning (should) have nothing to do with SS Dynamic Shop, this comes from spigot/paper on startup.  This warning is not harmful and will not cause sever issues.  It's just spigots way of telling you a plugin or plugins don't have an API version specified in the "plugin.jar" file.  For example all plugins should have the following line:
api-version: 1.15
If the api-version is missing or is anything less than 1.13 it will spit out the above warning message and force spigot into legacy material support mode which should not impact anything.
Best of luck,
Tarlak333

that warning does not matter which you would know if you read it. however I will fix it. #37

This issue should be closed. Thanks


为 MultivariateNormal 分布添加中文文档

为MultivariateNormal 分布，添加中文API文档。

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